text:
Alphabet for English rec model is not matching OCR on image

Thank you so much for providing code to convert paddle models to onnx format and be able to use these converted models with rapid OCR.
I used paddleocr_convert to convert the latest english models as follow:
paddleocr_convert -p  https://paddleocr.bj.bcebos.com/PP-OCRv4/english/en_PP-OCRv4_rec_infer.tar -o paddle_models  -txt_path https://raw.githubusercontent.com/PaddlePaddle/PaddleOCR/release/2.7/ppocr/utils/en_dict.txt

paddleocr_convert -p  https://paddleocr.bj.bcebos.com/PP-OCRv3/english/en_PP-OCRv3_det_infer.tar  -o paddle_models

For testing these models, I use the following image

I use rapid ocr to get OCR:
from rapidocr_onnxruntime import RapidOCR

engine = RapidOCR(rec_model_path="paddle_models/en_PP-OCRv4_rec_infer/en_PP-OCRv4_rec_infer.onnx",
det_model_path="paddle_models/en_PP-OCRv3_det_infer/en_PP-OCRv3_det_infer.onnx")

img = "c8fa66439d63041e.jpg"
result, elapse = engine(img)

and I get the following result (I only included the first few predictions):
[[[[551.0, 72.0], [952.0, 83.0], [949.0, 177.0], [548.0, 166.0]],
  'DlizAly',
  0.8711231350898743],
 [[[540.0, 211.0], [978.0, 221.0], [978.0, 250.0], [540.0, 241.0]],
  'ulD ADluAplAo jluABy',
  0.9064718882242838],
 [[[65.0, 280.0], [207.0, 277.0], [207.0, 293.0], [65.0, 297.0]],
  "B!QR%%'R}N}   T",
  0.6442324388772249],
 [[[220.0, 286.0], [270.0, 283.0], [270.0, 297.0], [220.0, 300.0]],
  'B!QV&P"%R',
  0.7667727172374725],
...

Whereas the default models produce more matching results
from rapidocr_onnxruntime import RapidOCR

engine = RapidOCR()

img = "c8fa66439d63041e.jpg"
result, elapse = engine(img)

result -->
[[[[545.0, 72.0], [954.0, 84.0], [951.0, 179.0], [542.0, 167.0]],
  'WEBSTER',
  0.8526573106646538],
 [[[539.0, 210.0], [975.0, 218.0], [975.0, 250.0], [539.0, 243.0]],
  'NEW TWENTIETH CENTUR',
  0.9136510831969125],
 [[[63.0, 280.0], [209.0, 277.0], [209.0, 295.0], [63.0, 299.0]],
  'Underrte, r.aa ferg',
  0.6786298751831055],
 [[[220.0, 285.0], [271.0, 282.0], [271.0, 298.0], [220.0, 301.0]],
  'Undlacove',
  0.7574679881334305],

This suggests I am perhaps using the wrong dictionary. However, the only dict I could find in the paddle git repo is https://raw.githubusercontent.com/PaddlePaddle/PaddleOCR/release/2.7/ppocr/utils/EN_symbol_dict.txt.
Do you happen to know if there is some step I am doing wrong here?
thank you so much,
Chris

I am closing this issue. After posting the issue, I have noticed that I was using a local dict that was slightly different from the one available with paddle ocr

repo: RapidAI/PaddleOCRModelConvert


text:
ImportError: cannot import name 'Deque'. when I type rasa init

**_C:\Users\Maadhav\rasa>_** rasa init
Traceback (most recent call last):
  File "C:\Users\Maadhav\AppData\Local\Programs\Python\Python36\Scripts\rasa-script.py", line 11, in <module>
    load_entry_point('rasa', 'console_scripts', 'rasa')()
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\pkg_resources\__init__.py", line 489, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\pkg_resources\__init__.py", line 2852, in load_entry_point
    return ep.load()
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\pkg_resources\__init__.py", line 2443, in load
    return self.resolve()
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\pkg_resources\__init__.py", line 2449, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "c:\users\maadhav\rasa\rasa\__init__.py", line 8, in <module>
    from rasa.run import run
  File "c:\users\maadhav\rasa\rasa\run.py", line 7, in <module>
    from rasa.core.lock_store import LockStore
  File "c:\users\maadhav\rasa\rasa\core\__init__.py", line 5, in <module>
    from rasa.core.train import train
  File "c:\users\maadhav\rasa\rasa\core\train.py", line 11, in <module>
    from rasa.core.domain import Domain
  File "c:\users\maadhav\rasa\rasa\core\domain.py", line 11, in <module>
    import rasa.utils.common as common_utils
  File "c:\users\maadhav\rasa\rasa\utils\common.py", line 7, in <module>
    import rasa.core.utils
  File "c:\users\maadhav\rasa\rasa\core\utils.py", line 27, in <module>
    from sanic import Sanic
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\sanic\__init__.py", line 2, in <module>
    from sanic.app import Sanic
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\sanic\app.py", line 18, in <module>
    from sanic.asgi import ASGIApp
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\sanic\asgi.py", line 28, in <module>
    from sanic.websocket import WebSocketConnection
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\sanic\websocket.py", line 12, in <module>
    from websockets import (  # type: ignore
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\websockets\__init__.py", line 3, in <module>
    from .auth import *
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\websockets\auth.py", line 15, in <module>
    from .server import HTTPResponse, WebSocketServerProtocol
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\websockets\server.py", line 49, in <module>
    from .protocol import WebSocketCommonProtocol
  File "c:\users\maadhav\appdata\local\programs\python\python36\lib\site-packages\websockets\protocol.py", line 18, in <module>
    from typing import (
ImportError: cannot import name 'Deque'

Thanks for the issue, @tabergma will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

Sorry, for the late reply.
Do you still face the issue? What Rasa version are you using?

you need to update to python >= 3.6.1

source: gharchive/issue

authors: Maadhav sara-tagger tabergma tmbo

repo: RasaHQ/rasa

license_type: permissive

license_source: github-api


text:
Add concurrent Redis lock store option

Proposed changes:
TL, DR: Add a concurrent version of Redis lock store.
Problem Description: The current RedisLockStore implementation does not protect against a race condition when separate Rasa instances are trying to acquire the lock at the same time.
Solution:

Implement a new subclass of LockStore based on Redis that can handle concurrency, named ConcurrentRedisLockStore

Issue deprecation warning for RedisLockStore to be removed in 4.0.0

ConcurrentRedisLockStore implementation of save_lock includes:

persisting serialised individual tickets to the lock store, set to a key which is a combination of key_prefix, conversation_id and ticket number.
persisting last issued ticket number
persisting the serialised TicketLock to maintain current implementations of delete_lock and get_lock

Add a new method increment_ticket_number in LockStore with default implementation currently used by InMemoryLockStore and RedisLockStore and remove this line in Lock.issue_ticket()

The new ConcurrentRedisLockStore overrides increment_ticket_number to make use of Redis atomic transaction INCR. If there is no last issued ticket number persisted in the Redis store, or the last issued ticket by TicketLock is not yet persisted in the store, then the function will return the ticket lock last issued ticket number + 1 (this is for edge cases such as when the lock store is empty and needs to handle a first message).

Status (please check what you already did):

[x] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

how complex is that db migration that we need? Can we, without too much effort, just do the migration in place when constructing a ConcurrentRedisLockStore potentially?

@losterloh I've raised my concerns about an online migration here. TL,DR: in order to migrate we need destination db details so I can't see how we could have both old db and new db details without making breaking changes to endpoints.yml. Unless we create a new CLI command? 🤔

wait, why do we need a different database? can't we just use different keys? I am not sure we even need to migrate existing entries, there is little value in them after a restart.

why do we need a different database?... I am not sure we even need to migrate existing entries

Very good points, we are actually using the same db and we also do not need to migrate the persisted lock, I don't know redis well enough if any action needs to be taken in this case?

source: gharchive/pull-request

authors: ancalita tmbo


text:
add RabbitMQ message header to "rasa export"

Proposed changes:

fixes #5317

Status (please check what you already did):

[x] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

Ok, just saw the telemetry issue. Then my questions are answered 👍 Good to go!

authors: ricwo wochinge


text:
Add error message for rasa validate when text is null in a response

Proposed changes:

Fix #7908

Status (please check what you already did):

[X] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[X] reformat files using black (please check Readme for instructions)

Ok so I looked a bit deeper into the pykwalify code... and I think the problem is with our extension. Because we raise it kicks out of the validation and doesn't add it to the errors. However if we were to return the error message instead, it would be collected as an error and end up in the final message.

Ok so I looked a bit deeper into the pykwalify code... and I think the problem is with our extension. Because we raise it kicks out of the validation and doesn't add it to the errors. However if we were to return the error message instead, it would be collected as an error and end up in the final message.

@joejuzl Yup, that's what I understood as well, but didn't know you can return a Error 😄  so try changing L20 in pykwalify_extensions.py to a return and remove the check I have for msg ?

authors: ancalita joejuzl


text:
test codeowners #1

Proposed changes:

test PR to make sure CODEOWNERS functions correctly
should tag Enable

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

unsuccesful test.


text:
Update 2.x with changes from 2.2.x and 2.3.x

Description
The goal of this PR is to show how changes made in a minor version would be documented in a "Docs for major only" setup. More information in this document.
How to use this PR

you can run do cd docs && yarn && yarn develop to run the docs locally
skip changelog.mdx
skip sources/ and reference/ folders
look at different kind of changes:

changes introduced in the docs because there were changes in the codebase (new feature, changes in behaviour, deprecations)
changes in the docs to improve it, independent of codebase change

💡 Pro-tip, filter files in the PR on ".mdx" only

Next steps

[ ] get reviews
[ ] remove the 2.2.x version of the docs in this branch
[ ] remove the 2.3.x version of the docs in this branch
[ ] merge this branch into documentation-full
[ ] repeat the process for all minors until 2.8

@desmarchris I went ahead and merged 2.2.x and 2.3.x in this PR. Can you take a look when you have the chance? Changes were easier to make this time, so I did 2 versions in a row

looks good!

@desmarchris probably a good idea to approve the changes if the requested changes are minor. That way I can incorporate the changes and then merge the PR without overriding your change request (here I assume that everything is good apart from the minor change). Not a big deal :)

authors: desmarchris m-vdb


text:
Dictionary look up

Is it possible to see the defiinition of words other than the first in a sentence? A way to scroll through them or something? I've just been deleting text to get to them but I don't know if there's another easier way to do it

It is possible now! In the new update you can tap the first character of the word you want to look up, just like Yomichan, if you ever used that


text:
🛑 GiteeSite is down

In a347992, GiteeSite (https://ravelloh.gitee.io) was down:

HTTP code: 0
Response time: 0 ms

Resolved: GiteeSite is back up in b364573.

repo: RavelloH/sitemonitor

license: MIT


text:
🛑 GiteeSite is down

In 1494f9a, GiteeSite (https://ravelloh.gitee.io) was down:

HTTP code: 0
Response time: 0 ms

Resolved: GiteeSite is back up in bb8b6fc.


text:
Cannot log in.

Running in docker.
Everything works as it should, but I cannot log in to my created admin account.
The "Login" button under "Accounts" in the settings pane does not do anything at all on 3 different browsers.
Am I missing something?

If you created account and have user/users.json file, you can enter your registered admin name to login input, and then you should see password input, if your name is same and registered user exists.

Oh, I see the problem, users.json is reset every time I restart the container...

After a few /addAdmin -> restart container attempts, it finally stuck.

Are tou runned it from cli, as in readme, or some other way?

docker run -d -it -p 4200:4200 -v /[...]

docker run --rm -it -p 4200:4200 -v /[...] makes the config stick.
The thing is that as soon as the /exit command is entered, the container shuts down.
I just want it to run in the background.
It seems the only way to get users.jsonto stick, is to do it this way,
then /exit, and then run again with -d

Thanks for info and words, will try to repro it later and check if there is some other docker flags to keep state on exit with detached mode. Maybe i write users file only on exit from memory, and exit signal is not sended in this corner case, hm.
I mostly use this project as screen -S synctube node build/server.js, so docker setup is not much tested, and was added by users initiative, but i want to keep it in good state, since containers is useful too.

I tested it like this:
docker run -d --rm -it -p 4200:4200 -v ${PWD}/user:/usr/src/app/user synctube
docker attach active_container_id_here

And /exit after /addAdmin shows Saving state... in output and saves things correctly, is this same for you?
Or, if you got state.json/users.json removed on next container start, then it looks like problem with attached --volume folder.

authors: RblSb ellidi

repo: RblSb/SyncTube


text:
New Subscribe Methods Using AnySubscriber as an Arg

I've never liked all the boilerplate code for multi-Subscriptions--ie. subscriptions instantiated in a viewController that implement StoreSubscriber. I've managed to code a solution that would allow devs to use a shortcut notation using a keyPath like so:
handleSubscription( \.deviceState.applicationState ) { [weak self] state in
    if let error = state as? ApplicationState {
          // do work here
     }
   }

versus the standard way:
private lazy var applicationStateSubscriber
     = AnonymousStoreSubscriber<DeviceState> { [weak self] state in
       self?.newApplicationState(in: state)
     }

   store.subscribe(applicationStateSubscriber) { subscription in
     subscription
       .select { $0.deviceState }
       .skip { $0.applicationState == $1.applicationState }
   }

By implementing an AnySubscriber like this:
public class ManySubscriber: AnyStoreSubscriber {

    public init(onNewState: @escaping (Any) -> Void) {
        self.onNewState = onNewState
    }

    public func _newState(state: Any) {
        self.onNewState(state)
    }

    public var onNewState: (Any) -> Void
}

We can add this ManySubscriber to a base viewController for managing subscriptions :
 var subscribers = [ManySubscriber]()

  func handleSubscription<SelectedState: Equatable>(_ keyPath: WritableKeyPath<AppState, SelectedState>,
      onNewState: @escaping (Any) -> Void ) {

      let subscriber = ManySubscriber { _ in }
      subscribers.append(subscriber)
      subscriber.onNewState = onNewState
      store.subscribeAny(subscriber, keyPath: keyPath)
  }

  deinit {
      subscribers.forEach { store.unsubscribe($0) }
  }

I've intentionally called the new method subscribeAny() to avoid confusion with the original subscribe() method.
I am aware that this approach means the Value passed in the KeyPath will not be typed.
UPDATE:  I've added a second overloaded subscribeAny() call which takes 2 closures as args--a subscription and newState handler. It is invoked liked so:
handleSubscription( { subscription in
          subscription
              .select { $0.state1 }
              .skip { $0.counter1 == $1.counter1 }
      }) { [weak self] state in
          if let state = state as? State1 {
              self?.counterLabel.text = "\(state.counter1)"
          }
      }

In this call we conveniently keep together in one place WHAT WE ARE SUBSCRIBING TO and HOW WE HANDLE IT.

Hi Jeff, this is a common extension that is used for multiple subscribers, and can be achieved in a type safe manner using a type such as BlockSubscriber. Here is a simple set of extensions that you can use in your projects:
We've considered adding this sort of thing to the project but at the moment we're holding off. Its simple enough to include yourself when needed.
class BlockSubscriber<S>: StoreSubscriber {
    private let block: (S) -> Void

    init(block: @escaping (S) -> Void) {
        self.block = block
    }

    func newState(state: S) {
       block(state)
    }
}

extension StoreType {
    func subscribe<SubState>(
        select: @escaping (State) -> SubState,
        newState: @escaping (SubState) -> Void
    ) -> BlockSubscriber<SubState> {
        let subscriber = BlockSubscriber(block: newState)
        subscribe(subscriber) { $0.select(select) }
        return subscriber
    }

    func subscribe<SubState: Equatable>(
        select: @escaping (State) -> SubState,
        newState: @escaping (SubState) -> Void
    ) -> BlockSubscriber<SubState> {
        let subscriber = BlockSubscriber(block: newState)
        subscribe(subscriber) { $0.select(select) }
        return subscriber
    }
}

class Foo {
    private var subscribers: [AnyStoreSubscriber] = []

    init(store: Store<Bar>) {
        subscribers = [
            store.subscribe(select: \.someState) { state in
                print(state)
            },
            store.subscribe(select: \.otherState) { state in
                print(state)
            }
        ]
    }
}

authors: jeffhandy mjarvis

repo: ReSwift/ReSwift

license: mit

license_source: bigquery


text:
index.js: 1437 Warning: Please use `require("react-router-dom").withRouter

You cannot use this statement
import {withRouter} from "react-router-dom/withRouter";
because it gives this error
index.js:1437 Warning: Please use require("react-router-dom").withRouter instead of require("react-router-dom/withRouter"). Support for the latter will be removed in the next major release

We are getting rid of releasing a package with multiple modules in favor of a single package. This means that everything will have to be imported from react-router-dom; you can no longer cherry pick imports from a nested module. Right now, we have helper modules that allow you to use the cherry picking syntax, but with warnings that this is going away.
import { withRouter } from 'react-router-dom';

authors: pshrmn yosvani

repo: ReactTraining/react-router


text:
Two-way-binding?

Which would be the best approach to handle a two-way-binding by using ReactiveKit?
Also I've opened up a question on integrating CouchbaseLite with ReactiveKit on StackOverflow; but unfortunately I wasn't able to select the tag "ReactiveKit". So if you could help there too, this would be awesome!
http://stackoverflow.com/questions/33973446/how-to-implement-two-way-binding-with-swift

Oh, right, we'll need someone with 1.5k reputation to create us a tag...  I answered your question there.

Thank you very much!! Feel free to close the issue here :)

Hope it helps :)

authors: marbetschar srdanrasic

repo: ReactiveKit/ReactiveKit


text:
Another attempt at defining a Flowable.

Using intrusive_ptr<> for efficiency; support for movable as well as
unique_ptr subscribers; generalized subscriber construction.

Have open ... starting to look.

Struggling with the history on this one, will just close it and open a new PR.

authors: benjchristensen vjn

repo: ReactiveSocket/reactivesocket-cpp


text:
Rename scan to fold?

I don't have high hopes for this proposal, since due to conservatism there is a strong gravity to keep scan, but here goes nothing.
I've always scratched my head with "why is scan called scan?". The name doesn't convey its semantics properly.
scan is really just like fold from Functional Programming. fold is a very well establish name in FP community. Also Elm's scan is foldp as in "fold from the past". Fold just makes sense. If you imagine the Observable as a timeline of events, if you fold that timeline like you would with origami, it's gonna reduce the events.
So, since we're still in alpha, now would be the best time to rename it. We already have a lot of drastic renames such as flatMap => mergeMap, so why not fix others as well, like scan => fold?
PS: same amount of characters. ;)

@staltz because it's exactly the same as Haskell's scanl and that's where we got the name.

Ah, that's where it came from. Thanks, I finally know.
Anyway, since Haskell's foldl is JavaScript's reduce, why can't Haskell's scanl be JavaScript's fold?

I think scan is probably one of the more widely understood operators, honestly. I'd hate to rename it. When I've heard other people talk about Rx, they usually mention it in terms of "a reduce that emits its accumulated value every time".

@staltz please, let's not fork the name to get people confused as to something that is well understood in other languages such as Haskell, Scala, F#

Also, FWIW: flatMap will remain as an alias forever, as far as I'm concerned, because it's pretty idiomatic. mergeMap is so named because it makes it easier for people to understand and it's more conventional.

Oh well, this won't proceed, so carry on folks.

authors: blesh mattpodwysocki staltz

repo: ReactiveX/RxJS


text:
feat(switchFirstMap): rename switchMapFirst to switchFirstMap

Renaming switchMapFirst to switchFirstMap to match the naming scheme of <mergingStrategy>Map
related #915

quickly :+1: by PR only contains name change. :)

authors: blesh kwonoj


text:
first operator does not accept default value only signature with null-value predicate

RxJS version:
latest
Code to reproduce:
Rx.Observable.range(0, 10).first(null, null, true);
Expected behavior:
Actual behavior:

error TS2345: Argument of type 'null' is not assignable to parameter of type '(value: number, index: number, source: Observable) => boolean'.

Additional information:
This seems bit weird since default signature allows optional values for predicate / resultselector (https://github.com/ReactiveX/rxjs/blob/69d051bc56cb337148db30515029363c1e78b90d/src/operator/first.ts#L76) and test case exists (https://github.com/ReactiveX/rxjs/blob/69d051bc56cb337148db30515029363c1e78b90d/spec/operators/first-spec.ts#L46) - not sure why it's being caused. Maybe signature ordering causes this.

I tested this with rxjs@5.2.0 and typescript@2.2.1 and it compiled without any problem. What Typescript version are you using?

2.1 / 2.2 both seems have issues.
https://travis-ci.org/kwonoj/rxjs-first-signature-demo/builds/209822410#L172
https://github.com/kwonoj/rxjs-first-signature-demo/blob/f971b3426f8c5e497b219632bb27e57c0b70ec6b/index.ts#L3

null is indeed not assignable to that type, even if it's optional. What you're looking for is this:
Rx.Observable.range(0, 10).first(undefined, undefined, true);

Yes, I am aware this can be avoided. Still type definition need to be looked into.

Does something like this look valid? I could create a PR then ...
export function first<T, R>(this: Observable<T>, predicate?: (value: T, index: number, source: Observable<T>) => boolean | undefined | null,
                            resultSelector?: ((value: T, index: number) => R) | void | undefined | null,
                            defaultValue?: R): Observable<T | R> {
  return this.lift(new FirstOperator(predicate, resultSelector, defaultValue, this));
}

I'm looking at this again after more than a year and I think this should be the expected behavior. Even when the method is defined as optional it shouldn't accept null in my opinion. If I want to skip any of the parameters I should use undefined.

This is no longer an issue.

authors: benlesh jsayol kwonoj martinsik seveves

repo: ReactiveX/rxjs


text:
getting "Types of property 'lift' are incompatible." with typescript 2.4.0

RxJS version: 5.1.0
Code to reproduce: linting a code using Subject
Expected behavior: no errors
Actual behavior:
Types of property 'lift' are incompatible.
Type '<R>(operator: Operator<T, R>) => Observable<T>' is not assignable to type '<R>(operator: Operator<T, R>) => Observable<R>'.
Type 'Observable<T>' is not assignable to type 'Observable<R>'.
Type 'T' is not assignable to type 'R'.

Additional information:
The bug popped up after typescript updated to 2.4.0
see https://github.com/angular/angular/issues/17800 also

dupe of https://github.com/ReactiveX/rxjs/issues/2699 , https://github.com/ReactiveX/rxjs/issues/2539, https://github.com/ReactiveX/rxjs/issues/2700 . https://github.com/ReactiveX/rxjs/issues/2701

If kwonoj would list the problem that lists the solution, that would be helpful. Kwonoj's practice of listing umpteen different dupes only confuses the problem.

There is open issue to track this one and it is on discussion. I'm closing all dupues on behalf of opened one - it doesn't have meaning to open issue without searching dupes.

I understand that you only should track the problem on one ticket. How about specifying on which ticket the problem is being tracked instead having the user go through all four tickets to find the issue tracking the problem?

I'd rather suggest people search issue first before filing redundant issue that I have to just close.

OK I will do it for you. This issue is being tracked here: https://github.com/ReactiveX/rxjs/issues/2539

uhm, so what is the canonical issue for this? None of the dupes you mention seem canonical.

All of them are same dupe issue and latest update resolved it.

Same here. I'm searching for the fix on this issue and can't find it anywhere...


text:
drag and drop bug.

RxJS version:
5.0.3
Code to reproduce:
jsbin
Expected behavior:

mouseup event only be added once.
after mouseup event, subscribe won't be called.

Actual behavior:

every time, just after drag and drop, mouseup event be added.
You can play it around, drag one over another, repeat it. You will notice that mouseup event does not stop the drag

Additional information:

are you saying mousedrag::completed is not called when mounseup emitted for after mouseup event, subscribe won't be called. or otherwise?

@kwonoj I know the completed won't be called. what I mean is after mouseup event. the mousemove event shouldn't drag the element. But that's not always true. You can play the demo around,

I just tried your demo, only add console.log(pos) in subscription and confirmed subscription is not called once up event fired. Check above screen record.

and reason it keeps creating up is each time mousedown event emits, you're creating new observable internally adds eventlistener in up event. I believe this is user level code, not library side issue - closing for now. If you believe this is bug in library, please share isolated code snippet causes growing addition to eventlistener.
Lastly, for event binding you can use fromEvent / fromEventPattern instead of creating observable directly.

If you just drag one element , then every is fine.You should drag one over anther, and repeat it. Like drag target1 over 'target2, drag target2overtarget1.  BTW, have you notice there are multiup` logs ? Is it a bug ?

I just commented above - it's not library, you're creating multiple observable keeps adding eventlistener into target.

Please don't close It . Could you tell me what software are you using to make screen shot to git. I can take a screen shot there will be a bug like what I said.

@lili21 I noticed multiple up, and digging into it as commented your code keep creating eventlistener. This issue repo is to triage / report library side issue, while this is user code.  As issue template suggests, it is better for you and for us that this question is placed in StackOverflow or some chat channel instead.

actually I don't care the multiple up very much, I care more about the drag and drop work as expected. which does not.

and why my code keep creating eventlistener ? because of takeUntil ?

It keeps creating an eventlistener because each mouse down maps to a mousemove.takeUntil(mouseUp) combo. The problem is this:

  var mouseup   = Rx.Observable.create(ob => {
    target.addEventListener('mouseup', e => {
      console.log('up')
      ob.next(e)
    })
  })

As was pointed out you would be better off using the fromEvent pattern as the code above does not clean up the subscription on unsubscribe. Thus you keep generating new mouseup handlers everytime there is a mouse down event.

@paulpdaniels Thanks. understand that now.
Unfortunately I can't use fromEvent in my real application code.

@kwonoj  after mouseup event,

figured that out, when dragging red block "over" black block, the red block is below black block because of the default z-index value. so the mouseup event is fired on black block ,not red block.

authors: kwonoj lili21 paulpdaniels


text:
Observable.from prioritizes array over observable symbol

Bug Report
Current Behavior
When an object that is "array like" and has Observable Symbol, Observable.from uses array like Observable.from mechanism before the Observable Symbol interop.
This is a problem in Microstates because the Observable Symbol provides the proper mechanism to make a Microstate observable. Treating it like an array does something very funky.
Reproduction

REPL or Repo link:
https://runkit.com/taras/creating-an-observable-from-array-not-working

const assert = require('assert');
const { from } = require('rxjs');
const Microstates = require('microstates').default;

let ms = Microstates.from([{hello: 'world'}]);

from(ms).subscribe(next => assert(next[0], 'Should have an array with one item'))

Expected behavior
I would expect the Observable Symbol interop to be used before any other.
Environment

Runtime: [Node & Browsers]
RxJS version:
6.1

Possible Solution
Check for Observable Symbol interop before all other options.
Related https://github.com/microstates/microstates.js/issues/130

I agree. I think this is a bug. If it's not an RxJS Observable, we should check to see if it has the interop point before checking other types. Thanks, @taras.
cc @kwonoj

@benlesh I can create a PR if it helps.

@taras feel freely go for it.

@kwonoj here you go

authors: benlesh kwonoj taras


text:
Uncaught AjaxError - React Native / Android

I'm getting the following error on React Native:

I'm using redux-observable with the following code snippet:
const tokenEpic = (action$, state$) => action$.pipe(
  ofType(UPDATE_DEVICE_TOKEN),
  mergeMap((action) => {
    const updatePath = `${API}/users/${action.id}`;
    const updateBody = {
      action: "update_token",
      new_token: action.token,
    };
    const settings = {
      url: updatePath,
      method: "PATCH",
      body: updateBody,
      timeout: 1000,
    };
    return ajax(settings).pipe(
      map((result) => {
        const data = result;
        return updateTokenSuccess(data.response.data.token);
      }),
      retry(3),
      catchError(error => of(showApiCallFailed())),
    );
  }),
);

Observations:

React Native (Android Emulator) instantly produces this error whenever the server responds with a HTTP 501 error
It does not even seem to retry before throwing that uncaught error
If there is no server responding to the HTTP call, it takes 75 seconds (a timeout?) to produce the same error, regardless of the timeout config
If I run React Native's javascript in the debugger (Debug JS Remotely - I think it's running on Chrome V8), not on the emulator:

this uncaught error does not appear to be thrown - showApiCalled() gets produced as expected
the long timeout (75 sec) still applies regardless of the config
if retry(3) is there, the timeout is 5 minutes before producing showApiCallFailed()

All in all, it seems to have something to do with how JS is run on Android in React Native.
Versions:
react-native@0.55.3
redux-observable@1.0.0
rxjs@6.2.1

dupe of https://github.com/ReactiveX/rxjs/issues/3738 .

authors: kwonoj tuomohopia


text:
docs(CONTRIBUTING): fix spelling and link to submit an issue

Description: Fixed broken path and spelling error inside CONTRIBUTING.md

I think I did it correctly. Please tell me if something is incorrect and I will try to fix it.

I think change's legit. I'll check this in around late today to tomorrow if I can't find anything. Huge appreciate for effort!

Found one more typo.

I came to think it'd be good if there's way to run spell check as similar to run lint over codes per test / CI. :)

I  would love too help look for a solution for that.

Merged with 074b780, appreciate @AdamWendell take effort to bring this. Thanks!

authors: AdamWendell kwonoj


text:
.NET WinForms cannot load Neo4jClient

"Cannot find source code: ResponseBuilder.cs is undefined"
Win 8.1/64 Visual Studio 2013, compiling with .NET 4.5. Installed: ReSharper 10.0.2, PostSharp
Installed Neo4j Community today. version 2,3,2 Server app is open now.
Created a new WinForms project.

used NuGet to install latest Neo4jClient
// other necessary usings present
 using Neo4jClient;
 private GraphClient client;
 private void Form1_Load(object sender, EventArgs e)
 {
     client = new GraphClient(new Uri("http://localhost:7474/db/data"));
     client.Connect();
 }

tried variations of creating the Client using both forms of username/password entry: same error.

thanks, Bill

Hi Bill,
Are you doing any querying at all?
I've just created a new WinForms project, added Neo4jClient and my Form1 code looks like:
using System;
using System.Linq;
using System.Windows.Forms;
using Neo4jClient;

public partial class Form1 : Form
{
    private IGraphClient _client;
    public Form1()
    {
        InitializeComponent();
    }

    private void Form1_Load(object sender, EventArgs e)
    {
        _client = new GraphClient(new Uri("http://localhost.:7474/db/data"));
        _client.Connect();
    }

    private void button1_Click(object sender, EventArgs e)
    {
        var query = _client.Cypher.Match("n").Return(n => n.Count());

        var result = query.Results.Single();
        MessageBox.Show($"Got {result} nodes");
    }
}

I've also added a button to perform a query, and this works as well - I presume you're using 1.1.0.32 of Neo4jClient.
Do you get the error from the .Connect() method call?

Thanks for your response !  I installed the Neo4j client via NuGet, and it's showing up as run-time version v4.0.30319, version 1.1.1.0.
I've tried both declaring the Client as GraphClient, and IGraphClient, but the error occurs when trying to instantiate the 'client variable. Strangely, if I set a breakpoint just before the 'client is instantiated, I get a different error:  "GraphClient.cs not found" rather than the error message I report in my first post (with no breakpoint).
thanks, Bill

I'm closing this as it's the same as #171 and we've discussed it more there.

authors: billwoo cskardon

license: MS-PL


text:
Telegram 群：C++ 众

http://tech.acgtyrant.com/C-众/
如果您用 Telegram 且有兴趣加入并一起讨论，留言申请即可。
审核标准是「教养」，比如对事不对人，不轻易评论甚至攻击不熟悉的事物，少谈主义多研究问题等。
该 issue 一周后我会自行关掉。

不用急于关闭，可以一直放这儿。

@Mooophy 好，很高兴你加入。

网站打不开了呃。。。就在这儿申请好了。。。

我怎么知道你的 ID?

@xenonz 应该是这个，

应该是全小写，一不小心at我自己了。。。

@acgtyrant 申请加入，ID @courier_ma

@acgtyrant 申请加入, ID @conanforever22

@acgtyrant  申请加入，ID @chestnut

@zct Telegram 上搜不到 @chestnut

@zct Telegram 上搜不到 @chestnut

不好意思，之前没设置，用 @orangeNut 看下？

申请加入， ID @digmouse

@acgtyrant 申请加入，ID @digmouse

repo: ReadingLab/Discussion-for-Cpp


text:
Wrong cli command description

$ sls

Commands
* Serverless documentation: http://docs.serverless.com
* You can run commands with "serverless" or the shortcut "sls"
* Pass "--help" after any <command> for contextual help

config credentials ............ Configures a new provider profile for the Serverless Framework
create ........................ Create new Serverless service
install ....................... Install a Serverless service from GitHub
deploy ........................ Deploy a Serverless service
deploy function ............... Deploy a single function from the service
deploy list ................... List deployed version of your Serverless Service
invoke ........................ Invoke a deployed function
invoke local .................. Invoke function locally
info .......................... Display information about the service
logs .......................... Output the logs of a deployed function
metrics ....................... Show metrics for a specific function
remove ........................ Remove Serverless service and all resources
rollback ...................... Rollback the Serverless service to a specific deployment
slstats ....................... Enable or disable stats
swagger ....................... Build an ECS cluster
                                ^^^^^^^^^^^^^^^^^^^^

"Build an ECS cluster" doesn't seem OK :)

Oh, thanks :D
On 7 April 2017 at 23:24, Iñaki Abete notifications@github.com wrote:

$ sls
Commands* Serverless documentation: http://docs.serverless.com* You can run commands with "serverless" or the shortcut "sls"* Pass "--help" after any  for contextual help
config credentials ............ Configures a new provider profile for the Serverless Framework
create ........................ Create new Serverless service
install ....................... Install a Serverless service from GitHub
deploy ........................ Deploy a Serverless service
deploy function ............... Deploy a single function from the service
deploy list ................... List deployed version of your Serverless Service
invoke ........................ Invoke a deployed function
invoke local .................. Invoke function locally
info .......................... Display information about the service
logs .......................... Output the logs of a deployed function
metrics ....................... Show metrics for a specific function
remove ........................ Remove Serverless service and all resources
rollback ...................... Rollback the Serverless service to a specific deployment
slstats ....................... Enable or disable stats
swagger ....................... Build an ECS cluster
^^^^^^^^^^^^^^^^^^^^
"Build an ECS cluster" doesn't seem OK :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Reckon-Limited/serverless_swagger/issues/3, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABu5hjWT_TrNsgiRsU8OUlHCoQ1ey6Vks5rtjj4gaJpZM4M25ER
.

authors: inakiabt tobyhede


text:
Update versioning

Description
Current versioning is always set to 1.0.0, while setuptools_scm is listed as dependency.
This change will provide a similar versioning as the one used in our other Python repositories
Type of change

Refactor (refactoring code, removing useless files)

Testing steps
Locally run python setup.py sdist to check the generated version`
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

lgtm

Just FYI, there's some warnings when running setup.py --version about the usage of dashes (-) instead of underscores (_) in the setup.cfg file.

lgtm

Just FYI, there's some warnings when running setup.py --version about the usage of dashes (-) instead of underscores (_) in the setup.cfg file.

Already fixed in the last version. TY!

Codecov Report
Base: 90.79% // Head: 90.79% // No change to project coverage :thumbsup:

Coverage data is based on head (4ceb57a) compared to base (c5afbac).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #39   +/-   ##
=======================================
  Coverage   90.79%   90.79%
=======================================
  Files          18       18
  Lines         489      489
=======================================
  Hits          444      444
  Misses         45       45

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter epapbak joselsegura

repo: RedHatInsights/insights-ccx-messaging


text:
Fine-tune microdnf commands used while building images

Same as https://github.com/RedHatInsights/notifications-backend/pull/2928.

/konflux

repo: RedHatInsights/notifications-gw


text:
fix(RHINENG-3363): Alter permissions for change settings button

Fixes https://issues.redhat.com/browse/RHINENG-3363.
This changes the required inventory permission to inventory:hosts:read.
Description
'inventory:*:read' is no longer available for the administrator roles. This was causing the change settings button to be always disabled regardless user's roles.
How to test the PR
TBA
Before the change
After the change
Dependent work link
Checklist:

[ ] The commit message has the Jira ticket linked
[ ] PR has a short description
[ ] Screenshots before and after the change are added
[ ] Tests for the changes have been added
[ ] README.md is updated if necessary
[ ] Needs additional dependent work

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (19964db) 51.67% compared to head (69fd95a) 51.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #270   +/-   ##
=======================================
  Coverage   51.67%   51.67%
=======================================
  Files          88       88
  Lines        1252     1252
  Branches      264      264
=======================================
  Hits          647      647
  Misses        605      605

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: codecov-commenter gkarat

repo: RedHatInsights/sed-frontend


text:
Fix alias file path when installed from pypi - 37

[Auto generated]
Number: [#37]

must-gather-explorer[pull_request][a7f2-11ef-82cb-0c96c574aaf5][PR 37]: Closing issue for PR: Fix alias file path when installed from pypi.
PR was closed.

repo: RedHatQE/must-gather-explorer


text:
Simplify must_gather_shell, get all args from user - 56

[Auto generated]
Number: [#56]

must-gather-explorer[pull_request][dfd49090-b547-11ef-8b15-42d0501ea0a6][PR 56]: Closing issue for PR: Simplify must_gather_shell, get all args from user.
PR was closed.


text:
{ BUG } Learn More Button on Windows Download Page is Broken

Bug Description
The "Learn More" button on the page located at: https://www.redotengine.org/download/windows is pointing to https://docs-latest.redotengine.org/about/introduction#about-redot-engine instead of https://docs.redotengine.org/en/latest/about/introduction#about-redot-engine which seems to be the correct URL.
Steps to Reproduce

Visit redotengine.org on a Windows computer.
Click the "Download" button located in the Hero at the top of the page.
Scroll to almost the bottom of the page, look for the heading "What is Redot Engine?"
Click the "Learn More ->" button.

Expected behavior
The introduction page in the latest version of the docs should be loaded.
Additional Details
Sorry my first submission is so lame. I'm just getting around to checking out Redot and noticed it while I was browsing. :)
Thx to the devs for all the hard work. The page looks absolutely amazing.
Operating System
Windows 11 - Release Channel - Current

The old website will no longer be updated; a new version will be released by the end of the year.

This is the repository for the new website; we will take this into account when implementing the download page.

This is the repository for the new website; we will take this into account when implementing the download page.

repo: Redot-Engine/redot-website


text:
github workflow should use .net 7.0

After a bump to .net 7.0 the project fails to build in github workflow. Fixed it to also use .net 7.0 to build and publish a nuget package.
Also excluded sample project from release "any cpu" configuration build to speed workflow up.

Thanks, updated to 8

authors: Redth dance


text:
negative values not handled

the sample code does not take into account negatives. when a negative value is entered it shows off grid. Is it possible to get the bar chart to also handle a negative value?
Example:
var randomScalingFactor = function () { return Math.round(Math.random() * 100) };
var barChartData = {
labels: ["January", "February", "March", "April", "May", "June", "July"],
datasets: [
{
fillColor: "rgba(220,220,220,0.5)",
strokeColor: "rgba(220,220,220,0.8)",
highlightFill: "rgba(220,220,220,0.75)",
highlightStroke: "rgba(220,220,220,1)",
data: [-129, 52, 7, 42, 16, 0, 3]
},
{
fillColor: "rgba(151,187,205,0.5)",
strokeColor: "rgba(151,187,205,0.8)",
highlightFill: "rgba(151,187,205,0.75)",
highlightStroke: "rgba(151,187,205,1)",
data: [randomScalingFactor(), randomScalingFactor(), randomScalingFactor(), randomScalingFactor(), randomScalingFactor(), randomScalingFactor(), randomScalingFactor()]
},
{
fillColor: "rgba(240,73,73,0.5)",
strokeColor: "rgba(240,73,73,0.8)",
highlightFill: "rgba(240,73,73,0.75)",
highlightStroke: "rgba(240,73,73,1)",
data: [randomScalingFactor(), randomScalingFactor(), randomScalingFactor(), randomScalingFactor(), randomScalingFactor(), randomScalingFactor(), randomScalingFactor()]
}
]
};
window.onload = function () {
var ctx = document.getElementById("canvas").getContext("2d");
window.myBar = new Chart(ctx).StackedBar(barChartData, {
responsive: true
});
};

Hi @wirlygig,
Sorry for the late response!
If you need to handle negative values I suggest not to use this extension, since this particular way of visualizing data is not sufficient for your use case.
For negative values you'd rather use a non-stacked bar chart or line chart, which allows negative values for comparison.

authors: Regaddi wirlygig


text:
v0.4.0 - opacity in widget extension got key as positioned parameter

Opacity method in widget extension got key as positioned parameter in 0.4.0 leading to compile errors for existing code.

fixed in hotfix


text:
设置小于 12px 的字体

实现一

使用transform缩放属性-webkit-transform:scale(0.5);
注意-webkit-transform:scale(0.75)收缩的是整个元素的大小，这时候，如果是内联元素，必须要将内联元素转换成块元素

实现二
使用图片：如果是内容固定不变情况下，使用将小于 12px 文字内容切出做图片，这样不影响兼容也不影响美观

repo: ReinerLau/Blog


text:
Does not accept login credentials

Using Pyload in Vivaldi to access a locally hosted Pyload instance running on docker.
I have copied and pasted the url and port number to the YAPE options page however when I click login and add my credentials I constantly get a message saying 'Login failed, invalid credentials'.
These credentials work fine when I access the Pyload URL directly.
I have tried logging in and out of Pyload before entering credentials in YAPE but it makes no difference.
Default credentials of pyload/pyload do not work either (as expected).
My docker instance is using HTTP however I have tried toggling HTTPS support in YAPE too with no effect.

Hi @souperM, thank you for opening an issue !
I have updated the option page so that it gives you a feedback of your login status. Can you pull the latest version and let me know what is displayed ? The extension should use your browser session cookie so if the origin (IP, port and protocol) are correct it should work automatically.

Hi @souperM !
The store version of Yape has officially been released, it will be easier for you to check if your issue is solved. You can get it here. Let me know how it goes.

authors: RemiRigal souperM

repo: RemiRigal/Yape


text:
Data type not aligned

Dear author, during training the data type of img embedding is float16, but the input type for llama is bf16, I am wondering if this misalignment leads to data precision loss during training? Since float16 is much more precise than bf16.

Hi, thanks for your interest.
I thinks it's ok to use fp16 for img embedding since it can reduce GPU memory, and it's a common practice for other MLLMs like video-llama. However, it would be possiable to improve performance by using bf16.

authors: KKKLeon RenShuhuai-Andy

repo: RenShuhuai-Andy/TimeChat


text:
Increase security on site

As I have had the site up and running for over five days now, I have noticed bots trying to attack my site looking for potential PHP exploits. Now, as this is an express project, this isn't an issue, however, I have now noticed that they are asking my site to fetch other sites and using it proxy themselves.
I simply proof of concept is the following:
telnet localhost 3000

GET http://www.google.com.au HTTP/1.1

Now, my site should return a 400 as I don't own any content from google.com.au, however, the site returns the following:
GET http://google.com.au HTTP/1.1

HTTP/1.1 200 OK
X-Powered-By: Express
Content-Type: text/html; charset=utf-8
Content-Length: 4753
ETag: W/"1291-yai5bwQd+/W4//cwKMH/4Q"
Date: Tue, 18 Jul 2017 01:21:40 GMT
Connection: keep-alive

Now as we are using the default middleware as supplied by express generator, there might be a fix this already or another package to install to avoid this.

Okay, a quick solution might be to follow these steps:
https://www.nginx.com/blog/5-performance-tips-for-node-js-applications/

Hmm, I am reopening this issue as it requires further investigation.

Okay, I have a final verdict on this.
Currently the same call can be made to the server and it will return a 200.
I am not sure why this is the case considering it isn't found on my server, I believe this would be an issue with the express middleware.
I am going to downgrade this from a security issue to a bug in express.

repo: RenegadeTech/site


text:
Fix ISO 8601 format

Date from RS: 2016-10-25T16:16:16.236+0100
Date from RK: 2016-04-26T07:00:00-07:00

given that you are importing Joda Time, there are good news: the toString() method of DateTime is formatted as ISO8601 format (yyyy-MM-ddTHH:mm:ss.SSSZZ).

repo: ResearchStack/ResearchStack


text:
Scholar into main

Description
Created models to save user preferences for a website PR #87
Get publications using scholarly PR #83
Categorizing publications PR #81
Type of change

[ ] Bug fix
[X] New feature
[X] Refactoring
[ ] Documentation

Problem
#87 : No way to associate user website preferences, so added models to do that.
#83  : Before we had to scrape the results using a node package but that wasn't ideal so we decided to do it in python instead where scholarly does the scraping for us.
#81 : As per the client request, each publication should have a field indicate whether it's conference /journal and what the title of the conference/ journal.
Solution description
See individaual PR
Tests
All unit tests pass? N/A
Builds successfully? Yes
Attach screenshot of the expected result (e.g. UI of the website/ postman api call result/ database changes)
#87 Themes, templates and team model

#83 See PR for more detailed information to reproduce and expected result
#81 New publication form

Different layout implement

Relevant document/ link (if any)
#87 https://docs.google.com/document/d/1U68bJV0POrh_wDXem_hZ_PrivoVT6qZCPC8vw7Dg5uQ/edit#heading=h.p3p1n0br5dbe
Risk
High for #81 Create and update operation for publication might not work properly if bugs exist as the publication form has been modifed
Low for #83 and #87

Everything looks good from what I've seen and had no issues when running it. The only concern I have is with some of the checks that have failed, in particular the unit tests. Not really sure what the fail is for but better to have this sorted before doing the merge into main.

@ssha0054 The failed static analyzer / unit tests is on the PR stage rather than the push which is weird because they would have the same content anyway. It seems to be some internal git issue, which is out of our hands. Rest assured the two commits with the failed tests is just me commenting out two lines in a python file and then putting them back in again.

repo: Researchify/Researchify


text:
Separate Blockchain/Snapshot database files from Wallets files

Is your feature request related to a problem? Please describe.
Every time it is required to import a new Snapshot or for some reason the database gets corrupted, it is needed to add all of the wallets again using their private keys, which is really tedious.
Describe the solution you'd like
Separate those two functionalities so that it is possible to import a snapshot or even start from block 0, without having to re-add all wallets.
Describe alternatives you've considered
Have two different folders, one for Wallets, and another for the Databases.

This is how snapshots work. The reason being that the wallet file will contain transactions. If you want to start from block 0 you simply just don't use a snapshot. Snapshots in order to work must start with a fresh instance. If this is not suitable then simply don't use and you can sync from 0.

Also just to add the wallet files are separated from a snapshot but they are not agnostic nor should they be.

repo: ReserveBlockIO/ReserveBlock-Core


text:
Fixed building for Cocoa/Mac

Cross-platform code contained a few lines of iOS specific code.

Thank you sir

authors: blakewatters ivucica

repo: RestKit/RestKit


text:
Kinetic & Gazebo7: Video streaming fails

Baxter's camera streaming fails using Gazebo7:
[ WARN] [1486153355.581155367]: GazeboRosVideo plugin missing <robotNamespace>, defaults to "".
[ WARN] [1486153355.581207155]: GazeboRosVideo Plugin (ns = ) missing <topicName>, defaults to "image_raw".
[ WARN] [1486153355.581218927]: GazeboRosVideo Plugin (ns = ) missing <height>, defaults to 240.
[ WARN] [1486153355.581227121]: GazeboRosVideo Plugin (ns = ) missing <width>, defaults to 320

Camera streaming is working for me using the following three test commands:
rosrun image_view image_view image:=/cameras/head_camera/image
rosrun image_view image_view image:=/cameras/right_hand_camera/image
rosrun image_view image_view image:=/cameras/left_hand_camera/image

And your related PR that was merged in.
Though I am also seeing those warnings in the console. You mentioned having more patches, do you have a fix for this already or should I create one?

This bug has already been reported on Gazebo issue tracker

Seems like an issue with the SDF parser for URDFs, I spent considerable time on this today but have not solved it

Ah, I'm sorry Dave, I think I did get to the root of this last week, but this week I've had my hands full doing a bunch of infrastructure upgrades. I'll follow up on the two issues Camera streaming and Image displaying using gazebo_ros_pkgs. The camera issue seems to have to do with namespacing, and the Image display issue seems to be caused by Gazebo7 injecting extra SDF tags when parsing the Video plugin from URDF. I have patches for both, but they'll need OSRF feedback.

Oops my last two comments were in regards to the image display, and should have been reported in a different issue (https://github.com/RethinkRobotics/baxter_simulator/issues/95).
The problem with the camera streaming seems fixed, and the warnings only needs this PR merged: https://github.com/RethinkRobotics/baxter_common/pull/86

authors: IanTheEngineer davetcoleman


text:
Error When Key Generated

Trying to generate a key for an old MD3200 and I got this error after compiling with VS2019:
ee the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.
************** Exception Text **************
System.FormatException: Could not find any recognizable digits.
at System.ParseNumbers.StringToInt(String s, Int32 radix, Int32 flags, Int32* currPos)
at System.Convert.ToByte(String value, Int32 fromBase)
at DellPowerVaultFeatureKeygen.Utility.HexStringToByteArray(String hex) in C:\Users\PowervaultKeygen-master\PowervaultKeygen-master\DellPowerVaultFeatureKeygen\Utility.cs:line 25
at DellPowerVaultFeatureKeygen.MainForm.Generate(Object sender, EventArgs e) in C:\Users\PowervaultKeygen-master\PowervaultKeygen-master\DellPowerVaultFeatureKeygen\Forms\MainForm.cs:line 26
at System.Windows.Forms.Control.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ButtonBase.WndProc(Message& m)
at System.Windows.Forms.Button.WndProc(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
************** Loaded Assemblies **************
mscorlib
Assembly Version: 4.0.0.0
Win32 Version: 4.8.4150.0 built by: NET48REL1LAST_C
CodeBase: file:///C:/Windows/Microsoft.NET/Framework/v4.0.30319/mscorlib.dll

Hi, no error here with VS 2017 Community.
Just had to install Dot Net Framework 4.7.2 Dev Pack, and build was OK, in 2 seconds.
Key generated, it's amazingly simple. Thank you very much, @ReverseAllTheThings

I tried VS 2017 community and still got the same error when i tried to enable the feature using the identified of "SSD Cache" as describe in the config files.
"Could not find any recongnizable digits"
Is there a list of parameters or names and values that work for the FEature Enable Identifier?

The fix apparently is to use the NUMBER of the init and not the text value which then generates the proper key output. I wish this was made more clear somewhere.  I'll have to test and confirm the key is working when my array is reassembled.

I didn't select anything, just put the number the soft was waiting for, and generate. That's all. And the generated key was containig all premium features (didn't know we can choose)

What was the number you used? I don't see any list of how to add all the features at once without generating multiple keys.

I don't need to use any "number". After build, there was 3 files, located in /bin/Debug :

then I started the .exe :

I put my number from Dell Powervault Modular Disk Storage Manager :

and keygen gives me a .key file, and when I download it to MD3220i with button next to my number, it was recognized as licence for all.

repo: ReverseAllTheThings/PowervaultKeygen


text:
Add Dark Mode Color Scheme

Dark Mode has already been implemented in the css. I just need to come up with an appropriate color scheme.

#13

authors: GerritBergen RianBergen


text:
Speed up git mode by doing a shallow clone

Speed up git mode by doing a shallow clone: --depth 1
I guess this would be a breaking change, since git has not supported the --depth flag forever. (From what I can tell, it was introduced in git 1.5.0 which was released 13 years ago.)

I guess this would be a breaking change

degit only supports Node 8+, which has only been released 3 years ago, which means that "git with --depth" is also definitely expected.

@ArtskydJ degit development appears to have been ceased for over a year now. However, this pull request and others have been merged into the community driven fork: https://github.com/tiged/tiged

authors: ArtskydJ fregante nake89

repo: Rich-Harris/degit


text:
Include a captain section

It would be something like
Captain
This project is currently captained by @userxyzabc. For information about current state and goals, see the captain log.

I think this should be int he Maintainers section. I have been doing this:
## Maintainers

Captain: @hsanjuan.

Listing the information about what a captain is is something we can do. But I don't think that every README should have a Captain, so I don't think this is a good section. It also privileges Captains over other maintainers. The linter I've been using for IPFS repos checks for a captain the maintainer section; that should solve your need here.

Ah, sorry, i didnt realize there was a maintainers section already!

It should be there, actually. Opened a PR.

Fixed. Updated.

authors: RichardLitt hsanjuan

repo: RichardLitt/generator-standard-readme


text:
Fix for CMake errors if parent project also provides gtest

I'd argue that for ftl to be less intrusive, bringing in a relatively-common third-party target should also ensure that it doesn't break an existing implementation/CMake project.
Let me know if you'd rather the gtest_force_shared_crt flag be set regardless. Maybe that's safer if ftl requires it?

So, this is actually order-dependent from the POV of the parent project - it needs to add the gtest target (e.g. via ADD_SUBDIRECTORY) before adding ftl for this check to work, but it at least provides some way of preventing the conflict that isn't requiring the parent project to depend on whether or not ftl's tests are enabled.
If that was the way it worked, the parent project could have tests enabled but ftl's disabled, and then gtest wouldn't have been added and all hell would break loose.
Interested in any other solutions.

No longer relevant, since we switched to catch2. Thanks for the contribution though :)

authors: BrodyHiggerson RichieSams

repo: RichieSams/FiberTaskingLib


text:
Multiple instances don't work

I'm building a style guide and would like to use this for many instances across the page. Currently when dragging one, all the others resize and when releasing the mouse, the first stays active. How can this be fixed?
Here is a pen using the demo and showing the issue: https://codepen.io/danlewski/pen/rwzrvP

If you want different behavior from those two separate panels you'll have to name them differently. Use a unique ID for left and right panels and the splitter.
The issue here is there's no way to discern directly which item the sizing applies to, because the library doesn't know the relationship between the splitter that's firing the events and the target other than the selector you give it.

Ah, ok. That makes sense. The only problem is that I'm going to have 50+ instances and there's no way to maintain that. Could targeting the one being clicked on with this possibly work? So resize this on mouseDown, reset on mouseUp?

@dlewand691, I have the same requirement to maintain multiple split panes on one page.
I fixed it by finding the correct handle for each resizeable pane and binding the events only to that specific handle.
I made some changes to the getHandle function to get this to work:
function getHandle(selector, $el) {
        if (selector && selector.trim()[0] === ">") {
            selector = selector.trim().replace(/^>\s*/, "");
            return $el.find(selector);
        }

        return selector ? ($el).parent().find(selector) : $el;
    }

authors: ArnoutCreemers RickStrahl dlewand691

repo: RickStrahl/jquery-resizable


text:
Support for request url in exception for Typescript

It would be helpful to add the request url into the SwaggerException in Typescript client.
We handle these exceptions through some logger, message box .. and request url is important information for better determine what is wrong on backend side.

It would be a nice feature to have. Is there any plan for implementing it ?

Any update on this?

Any update on this?

authors: MirKml ahocquet supermar1010

repo: RicoSuter/NSwag


text:
Use OpenApiTagAttribute of class to set tags for those methods where such attribute is not set explicitly

Currently  OpenApiTagAttribute (and OpenApiTagsAttribute as well) has targets of both AttributeTargets.Class and AttributeTargets.Method.
But setting it as class attribute has no effect because OperationTagsProcessor uses only attributes from method and if no such attributes are present - processor uses controller class name.
Suggested change adds following OperationTagsProcessor logic: if no OpenApiTag atrtributes present at method, then try to get OpenApiTag (and OpenApiTags as well) from class, and if they are not present - then use controller class name as usual.
This can be helpful when controller contains a lot of methods and all of them should have the same tag(s).

Please also add a test

Please also add a test

I've added OperationTagsProcessorTests. Since OperationTagsProcessor is in NSwag.Generation assembly I have to add new test project NSwag.Generation.Tests

Please also add a test

@RicoSuter  ping ;)

authors: RicoSuter korneyev


text:
🛑 AnimeThemes is down

In 768c353, AnimeThemes (https://animethemes.moe) was down:

HTTP code: 0
Response time: 0 ms

Resolved: AnimeThemes is back up in d0df258.

Resolved: AnimeThemes is back up in d0df258.

repo: RitsuProject/ritsu-status


text:
Changelog compilation?

I'm not sure how to interpret "ERROR: Changelog was not compiled.". What am I missing?
https://github.com/troelsbjerre/Bottleneck/issues/73

Better error message required for this check (https://github.com/Roang-zero1/factorio-create-release-action/blob/master/entrypoint.sh#L33-L36)


text:
InstanceNorm not working

Hi! Thanks for this library :)
I'm trying to use InstanceNorm and it appears there's a bug.
When I run the following
irreps = BalancedIrreps(3, 20)
norm = InstanceNorm(irreps)
x = torch.randn(9, 20)
batch = torch.zeros(9, dtype=torch.long)
norm(x, batch)

I get the following error:
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Cell In[7], line 8
      6 x = torch.randn(9, 10)
      7 batch = torch.zeros(9, dtype=torch.long)
----> 8 norm(x, batch)

File /usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py:1501, in Module._call_impl(self, *args, **kwargs)
   1496 # If we don't have any hooks, we want to skip the rest of the logic in
   1497 # this function, and just call forward.
   1498 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
   1499         or _global_backward_pre_hooks or _global_backward_hooks
   1500         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1501     return forward_call(*args, **kwargs)
   1502 # Do not call functions when jit is used
   1503 full_backward_hooks, non_full_backward_hooks = [], []

File /usr/local/lib/python3.8/dist-packages/segnn/segnn/instance_norm.py:85, in InstanceNorm.forward(self, input, batch)
     82 # For scalars first compute and subtract the mean
     83 if ir.l == 0:
     84     # Compute the mean
---> 85     field_mean = global_mean_pool(field, batch).reshape(-1, mul, 1)  # [batch, mul, 1]]
     86     # Subtract the mean
     87     field = field - field_mean[batch]

File /usr/local/lib/python3.8/dist-packages/torch_geometric/nn/pool/glob.py:63, in global_mean_pool(x, batch, size)
     61     return x.mean(dim=dim, keepdim=x.dim() <= 2)
     62 size = int(batch.max().item() + 1) if size is None else size
---> 63 return scatter(x, batch, dim=dim, dim_size=size, reduce='mean')

File /usr/local/lib/python3.8/dist-packages/torch_geometric/utils/scatter.py:81, in scatter(src, index, dim, dim_size, reduce)
     78 count.scatter_add_(0, index, src.new_ones(src.size(dim)))
     79 count = count.clamp(min=1)
---> 81 index = broadcast(index, src, dim)
     82 out = src.new_zeros(size).scatter_add_(dim, index, src)
     84 return out [/](https://vscode-remote+ssh-002dremote-002bgatr.vscode-resource.vscode-cdn.net/) broadcast(count, out, dim)

File /usr/local/lib/python3.8/dist-packages/torch_geometric/utils/scatter.py:21, in broadcast(src, ref, dim)
     19 size = [1] * ref.dim()
     20 size[dim] = -1
---> 21 return src.view(size).expand_as(ref)

RuntimeError: The expanded size of the tensor (10) must match the existing size (9) at non-singleton dimension 1.  Target sizes: [9, 10, 1].  Tensor sizes: [1, 9, 1]

It appears this is because global_mean_pool from pytorch geometric does not support more than 2 dimensions. The solution could  be to replace global_mean_pool(field, batch) with global_mean_pool(field.view(-1, mul), batch).
Cheers,
Pim

Hi,
Thanks for alerting us to this, I will fix it in the near future.
Cheers,
Rob

@RobDHess Hi!
Are there any changes about this bug? I also have it
Cheers,
Ivan

authors: RobDHess pimdh z4-qqq


text:
Error temp measure

The thermistor I have installed was measuring the temperature correctly when connected to the board. With the OrbitertoolO2, and using the same configuration (replacing the pin with the tool's pin), it shows a very low temperature, around 3 degrees and a bit, when the ambient temperature is about 20 degrees.

Have you configured the correct pull up resistor?

Ok, That was the problem. I had crossed that line. Thnaks

authors: RobertLorincz dimicel

repo: RobertLorincz/Orbiter-Toolboards


text:
Alternate model definition proposal

Hello,
I managed to reduce a bit of boiler plate with a simple type definition:
export type ModelAttrs<T> = Omit<T,  keyof Model>

// example

// not required
export interface BaseModelAttrs{
  id: number
  createdAt: Date;
  updatedAt: Date;
}

export type ModelAttrs<T> = Omit<T,  keyof Model> & BaseModelAttrs

export default class BaseModel<T> extends Model<ModelAttrs<T>, Partial<ModelAttrs<T>>>{}

@Table
class Person extends BaseModel<Person> {
  @Column
  name!: string

  @Column
  birthday!: Date
}

this gives the proper auto complete when creating a new instance:

without the attrs you get all the model props

unfortunately this requires a base class to work

Im not exactly sure how this could be best integrated into the library (if at all) but here are some options.
Im happy to pr them but just wanted to get your feelings first.
Options:

docs: just a some docs and people can copy and paste the snippets
TypedModel base class

export abstract class TypedModel<T,C=T> extends Model<ModelAttrs<T>,ModelAttrs<C>>{}

This has actually been improved recently in Sequelize, and can be used in sequelize-typescript too, see InferAttributes & InferCreationAttributes here https://sequelize.org/docs/v6/other-topics/typescript/
It should get even better in a future major release: https://github.com/sequelize/sequelize/pull/14091

repo: RobinBuschmann/sequelize-typescript


text:
Typescript sequele where model given include

I'm trying to do this
 return Company.findOne({
      where: { id },
      attributes: ["id", "name", "email"],
      include: [
        {
          model: Box,
          attributes: ["id", "name", "shortUrl", "moderator"],
          include: [
            {
              model: Employee,
              where: { id: Box.moderator}
            }
          ]
        }
      ]
    }).then(data => data);

How can I get Employee given Box moderator ? can someone help?
Thanks

already solved


text:
Cannot able to find index.html. Need to add script tag in it.

I wanted to add one of my math library script in my electron app, for that i need to add the library in script tag, can you guide me how to resolve that issue ?

I suggest using StackOverflow for questions related to these kinds of things since this is not related to this library specifically, I will close this.

authors: Robinfr yashvekaria

repo: Robinfr/electron-react-typescript


text:
Fix compile error: #12

Fixed: #12
This was broken after an optimization commit: https://github.com/RobloxResearch/SIMDString/commit/849525764519d973e2976e947dc30868f3a9c5c0

This was fixed in 0ea70431d143b09a3ead6499be84a568de74125d

authors: BlackMATov linwansong

repo: RobloxResearch/SIMDString


text:
Barebones Team-vs-Team-Proof of Concept in Docker containers

It's fairly basic but if you have Open Source GPU Drivers or nvidia-docker set up correctly you can run
TEAM_Y=001 TEAM_B=002 ./run-docker.sh

and webots will run (on your host's Xorg-session but without internet access for the container) one 10-minute-Halftime while returning the referee-log-json and mp4-recording of the game to ~/webots_video_out/

This has been superseded by the docker-tournament-run-branch

repo: RoboCupJuniorTC/rcj-soccer-sim


text:
Train YOLOv8 model to detect mallets and water bottles

One of the tasks in the autonomous missions dictate that we need to be able to identify mallets and water bottles in the environment. We want to fine-tune a YOLOv8 model to do this task. This includes curating a dataset from manual capture or open-source sets and writing infrastructure to train and run inference using the model.

@abhiramg2021 please add the links to the online datasets that you found here. Thanks!

@abhiramg2021 please add the links to the online datasets that you found here. Thanks!

Here are the three datasets I found on roboflow.

https://universe.roboflow.com/mallet-and-water-bottle/mallet-and-water-bottle/model/5?webcam=true
https://universe.roboflow.com/wvu-urc-2024/wvu-urc-object-detection-data
https://universe.roboflow.com/michigan-mars-rover/mallet-bottle-detector/model/3

repo: RoboJackets/urc-software


text:
Director cannot handle collections messages at very high rate

Not going to fix it now, reporting anyway:
Director locks up when receiving collection messages (from pronto in debug mode) at more than 300Hz or so. Need to disable collections in startup

It's likely because of these lines:
https://github.com/RobotLocomotion/director/blob/master/src/python/director/lcmcollections.py#L207
The way collections was implemented in Director (by following lcmgl.py's implementation) put the lcm subscribers in Python.  But as we know, python cannot decode messages at a fast rate.  The best fix would be to move the lcm subscribers to threaded c++.  If you feel strongly about it I can help.
If you removed the lines calling setNotifyAllMessagesEnabled(True) then it might resolve the lock up issue, but it means some messages will be skipped.

This is used for analysis - so allowing messages to be skipped is not a suitable option.
I think the correct option is to move decoding to c++. I'll come back to this when I have the cycles.

authors: mauricefallon patmarion

repo: RobotLocomotion/director


text:
Top-left logo is unchangeable

I am not sure if this is a thing i am unaware about or is it actually not present.
What would you like to be added:
Currently the top-left logo of Rocketseat is set by default and it is only possible to change it by changing the node module files of @rocketseat.
I think there should be a option to add that top-left logo from the config settings.
Why is this needed:
i faced this issue and i had to change the module files in order to change the top-left logo
this was the file i had to change: @rocketseat/gatsby-theme-docs/src/components/Logo.js
Is your enhancement request related to a problem? Please describe.
yes it is a problem while making a documentation using rocketseat, as everyone would most probably like to have their own logo at that position.
Additional context
the top-left logo

@vinayaksh42 The docs (https://rocketdocs.netlify.app/usage/shadowing) clearly mention how it's supposed to be done. The short answer is Shadowing in Gatsby.

@vinayaksh42 The docs (https://rocketdocs.netlify.app/usage/shadowing) clearly mention how it's supposed to be done. The short answer is Shadowing in Gatsby.

Thanks alot for the information. Will check it

repo: Rocketseat/gatsby-themes


text:
Produce error for unknown file type

Currently, the editor does nothing when a file type is not recognised.

The file dialog supported various file types that the file reader didn't. I disabled all except .wool files, so this problem should no longer occur.

repo: RoessinghResearch/wool


text:
Dresden Files Accelerated Edition DFA.html bugs

Bug 1: Approach roll buttons
"Mantle"/Customisable approach button is currently tied to @{intelligence} rather than @{mantle-approach} as it should be. So when you hit the button, it rolls for intelligence, not whatever's in the mantle box.
Bug 2: Refresh
Refresh isn't updating.  Whenever I input new refresh costs into either conditions or stunts, @{refresh-spent} resets to value=NaN. There is no listed value for @{refresh-spent}, but I don't THINK that's the issue.
3 things I see that MIGHT be involved (or might not be issues, in which case I have no idea what the problem is)
1 - attr_stunt-cost does not have a title, unlike attr_condition-cost, so var 'stuntCost' is trying to retrieve a non-existent value.
2 - Lines 247-259 sets var 'stressCost', which relies on "repeating_stress", a value that doesn't exist anywhere else.
3 - refresh-spent relies on summing up conditionCost, stuntCost, and stressCost. I assume this translates back to @{refresh-spent} which might be why the value messes up (if it's pulling blank values for stuntCost and stressCost)

Hello,
I'm working on closing up the older issues on the Github. Can you let me know if this is still an issue?
Thanks,
Cassie

@clevett this issue can be closed as solved, as mentioned here.

Thanks for the update.

authors: Anduh clevett rentarded


text:
Conan_2d20_Adventures_In_An_Age_Undreamed_Of ... New Feature ... Verision 1.4 update includes language translation.

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x ] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x ] The pull request makes changes to files in only one sub-folder.
[x ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[x ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.


text:
AD&D1e Revised update

Small adjustment to the roll templates that allows text after a forced line break in a macro to align centered in the header and subheader keys. ie {{name}} and {{subtag}}

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.


text:
New Sheet Submission

Ironsworn RPG Character Sheet

Hi @dedurrett, I'm personally super hyped to get an Ironsworn character sheet on here, I've been playing it myself and was definitely this close to making one over the weekend.
That said, I can't accept this PR as-is; please remove the ACKS sheet from this pull request and put it in a separate PR to be reviewed.  Normally I would make an exception but this is a complete overhaul of a sheet someone else very recently authored and indicated that they wanted to improve upon, so it needs to undergo some further, detailed review.
Thanks,
Jeff

Thanks Jeff!
I'm still new to this github stuff, so it was a mistake that these both ended up in the same request in the first place. Hopefully I corrected it? I just reverted the last commit which was the ACKS related stuff. Let me know if I missed any steps.

Looks good!  Also, whoa, you must be quite an early user to have such a low user id.  Glad you're still with us! :D  Can't wait to play some Ironsworn on Roll20.  I'll merge this in and hopefully get it live soon.
Jeff

authors: Solarswordsman dedurrett


text:
[Coriolis] Issue fixes

Changes / Comments
These are changes for issue #5756
Corrected 2 of the 4 points the original poster mentioned.

Modifier query button for all rolls
Xd6>6 armor buttons

The other points I can't fix due to lack of familiarity with the game rules.
Sheet appearance was unaffected, though new items were added to the translation file.
New sheet attribute default to a value that will cause no effect to existing characters, the modifier query will be turned off.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

If this PR should close that issue, you can auto-close it by editing your comment to say fixes #5756.
Example of a PR that closed the related issue is this one

authors: Anduh JBeighel


text:
Update Winget version for existing WAU deployments

While #77 fixes/updates Winget at WAU install time, there's no functionality currently to update an existing WAU install with an updated version of winget if a minimum version number wasn't met. We could utilise the same Get-AppxProvisionedPackage check as the installer and perhaps keep a version number in an xml?
I'm going to investigate writing another function that could potentially achieve this, as while installing preview versions may be occasionally useful, it would be good to be able to revert to a milestone version if/when available.

The thing is, we didn't want to force users to use "preview" version when WAU was already installed. So only new install are concerned. As Winget should get updated by itself with MSStore, we expected Winget Team to release a new stable version soon.
But maybe we should use these latest preview version as it embeds new features

Totally agree, but there is the potential to want to change the installed version if there's a breaking bug or feature (like this) that could be massively beneficial, and being able to control that functionality could be really useful. I did some initial work pulling out the WAU update links to a new update.xml file in the config folder and I'm working on a function for the winget version. The change to the links file makes forked testing or custom deployments easier.

authors: Romanitho SkipToTheEndpoint

repo: Romanitho/Winget-AutoUpdate


text:
eth_sign requires 32 byte message hash

error: JSON-RPC error -32000: eth_sign requires 32 byte message hash

  @override
  Future<String> sign(String message, String address) async {
    // TODO: implement sign
    final result = await connector.sendCustomRequest(
      method: 'eth_sign',
      params: [address, message],
    );

    return result;
  }

error: JSON-RPC error -32000: eth_sign requires 32 byte message hash

Did you use MetaMask as provider? seems something related to this:
https://github.com/ChainSafe/web3.js/issues/4780

import 'dart:convert';
import 'package:crypto/crypto.dart';

// ...
  const messageToSign = "TEST";
  final bytes = utf8.encode(messageToSign);
  final messageHash = sha256.convert(bytes);
  final signature = await provider.sign(
      message: '0x$messageHash', address: session.accounts[0]);

Using personal_sign instead solves the problem, please check:
https://github.com/RootSoft/walletconnect-dart-sdk/pull/39/files

https://github.com/RootSoft/walletconnect-dart-sdk/issues/38#issuecomment-1122386757

authors: AllLuckly HaoCherHong RootSoft

repo: RootSoft/walletconnect-dart-sdk


text:
Update version and fix

good day
Conducted a number of revisions:

Added support for new versions single_php_filehost.
fixed the error of 413 (Request Entity Too Large)

P.S. Some observations. PHP-FPM really works faster PHP-CGI. Also nginx no worse Lighttpd

Works! Thank you.
Yeah I'd agree with those observations.

authors: MuratovAS Rouji

license: ISC


text:
Поправить минимап для мобильной версии

Обнаружили, что минимпап разъезается на мобильном устройстве. Скорее всего, это произошло из-за смены шрифтов или изменения позиционирования лейблов.
Для демо пока вообще скрыли минимап, но надо бы пофиксить asap.

Я уменьшал шрифт и даже прятал последний лейбл, но не помогло — поэтому и спрятал.
Т.е. дело точно не в шрифтах и может быть не в лейблах.

Я думаю, что проблемы мобильной версии растут откуда-то из сочетания viewport и script.js.
Например, у минимапа на мобиле есть еще одна проблема: если сделать анзум, чтобы страница

Упс, случайно закрыл да еще и коммент отправил, который, обдумав, посчитал ошибочным :)

authors: RusPosevkin krauklish

repo: RusPosevkin/evolution


text:
[WIP] Implement array::_array_reconstructor

Test cases:

[x] ArrayReconstructorTest.test_error
[x] ArrayReconstructorTest.test_numbers
[ ] ArrayReconstructorTest.test_unicode

[x] 'UTF-16-LE'
[x] 'UTF-16-BE'
[ ] 'UTF-32-LE'
[ ] 'UTF-32-BE'

whitespace-ignoring review link: https://github.com/RustPython/RustPython/pull/2937/files?w=1

Hi, @coolreader18, I'm working with the UTF32 encodings/decodings. There are already some enc/dec functions in Lib/_pycodecs.py.
There is a comment says that it comes from PyPy, but I cannot find the source in PyPy's code base, but only find this file. It is very different from _pycodecs.py.
Did you remember where the _pycodecs.py come from? I might quickly add the enc/dec for UTF32, and maybe rewrite them in Rust in the next PR.

@youknowone No problem. Let me resolve the conflicting files first.

One last thing - could you separate this into 2 commits? One with reformatting array.rs to be mod array { ... }, the other with the changes you made for _array_reconstructor? Hopefully that shouldn't be too hard :grimacing:

Oh, sad! The indents in marcos of the first commit is not correct, let me fix it.

I left 2 more comments and other parts looks good.

authors: coolreader18 whjpji youknowone

repo: RustPython/RustPython


text:
getLangDictionary does not work for "und"

See: https://github.com/RusticiSoftware/TinCanJS/blob/2a204f300343a9d3cddfcfcc91862f54f138d0a7/src/Utils.js#L221
Should instead be:
if (typeof langDict["und"] !== "undefined") {
    return langDict["und"];
}

Related issue: getLangDictionaryValue

My mistake.

repo: RusticiSoftware/TinCanJS


text:
✖ Instance of 'BrickIncompatibleMasonVersion'

While i tried to build a feature, i'm getting following error
✖ Instance of 'BrickIncompatibleMasonVersion'

What should i do? Thanks in advance

@Uvais-Iocod I have bumped mason versions and tested this. Should be good to go now. You would need to update the version to 0.3.0

Its working fine after updated version to  0.3.0. Thanks @RyanDsilva for your valuable support.

authors: RyanDsilva Uvais-Iocod


text:
画像合成の計算量軽減

基本的にバウンディングボックス内の小さな画像の情報だけ保持する。
random_place時は、まず小さな画像をランダムに回転させ、PILのpaste()とかで貼り付ける（またはtorch.roll）元画像全体・マスク全体に対する処理をなるべくしない。
https://note.nkmk.me/python-pillow-paste/

Done.

repo: RyotaUshio/chestnut-detection


text:
Init should return a bool

The init function should return a bool (and probably false if it failed to init).
This is currently not checked in usbh.c, so it currently doesn't hurt. But in the future it may be checked and cause your driver to fail loading.
Also, please mark unused parameters like the rhport in open as unused like adding "(void) rhport;" to the begin of the function.
Other than that: The driver works for me :-) Thanks a lot!

Thanks! youre right
Closed by  https://github.com/Ryzee119/tusb_xinput/commit/0f9cbc3953431c602376d7e7f3c7a6ff2149c544


text:
Unique subtitles for every guild

#10
While the use of a single text file, i.e, subreddit.txt is efficient and reliable, we do have to remember that this bot is made with the vision of extending its services to more than one server.
So, the use of a single text file for storing subreddit names for all servers is inconsistent, mainly because adding/removing/modifying the list will cause the changes to be reflected across all guilds.
So the proposed solution is to use a .json file where the key is the guild id in string and value will be an array of subreddit names to be used in that guild.
Ex :

{ "<guild_1_id>" : "[<subreddit_1>,
                                <subreddit_2>,
                                <subreddit_3>,....]",
"<guild_2_id>" :  "[......]",
}

@Rajrup911 can you take this up?

yeah


text:
Safety: Make pages

Modify Lab Manual: Safety section to have separate pages for: (1) Lithium Ion Batteries and (2) Field Safety (field safety is currently a placeholder, which is ok for now).

Done

repo: SAEL-SWFSC/SAEL-lab-manual


text:
How to automatically switch service to access to shared table between HANA and SQLite?

Dear team,
Thanks for your great contribution to share multitenancy application and instruction.
I am implementing multitenancy application based on your repository and I have question for shared database container.
In your explanation, we need two CDS for both HANA for production and SQLite for local testing.
Currently I extended your service to access to shared table and manually change CDS file /code/db/sqlite/data-model to /code/db/hana/data-model before deployment to Cloud Foundry.
Is there smarter way to switch between two CDS files?
I could not find good example of this use case. So I am helpful if you advise me.

Please check this example: fiori/package.json#L42

Hi @gregorwolf,
Thanks for your answer. That is helpful and I already checked this code sample.
But my question is how to refer to db CDS from service CDS.
For example, when I want to add Shared entity to public-service.cds, now I am writing like below.
using {susaas.common as common} from '../../db/hana/data-model';
...
service PublicService @(path : '/catalog/PublicService', impl: 'srv/public-service') {

  entity Shared as projection on common.Shared
  ...

Hence I am specifying db/hana/ directory, I cannot use it in for local SQLite.
How to automatically switch path in service CDS, or something alternatives?

Hi Dear @kaztakata san,
Thank you very much for your contribution!
To be honest, when we designed this "common container" approach, I personally only considered the database layer usage.
But your requirement is valid and makes sense. I think it might be useful for the rest of our valued susaas community members too.
I have a nice solution proposal for the problem you have.
This solution proposal should not change the behavior of the repository and should also fix the problem you have.
The main reason we have two files is that Shared  entitys DB artifacts are already created on deployment; therefore, we would like to mention it to HANA to avoid errors, but for sqlite we ne need to create the tables.
Can you please confirm if the changes on the fix/cc-localdev branch fix the problem you have?
I also linked the relevant PR to this issue.

Hi @alperdedeoglu san,
Thanks for your solution. I think extend CDS is effective for this situation.
I used your code and changed package.json to switch between production and development profiles like previous answer by Gregorwolf then working.

authors: alperdedeoglu gregorwolf kaztakata

repo: SAP-samples/btp-cap-multitenant-saas


text:
Need hints on consuming on-prem systems from App Studio

During the premiere of the Workflow topic exercise 03/04 videos yesterday, Vignesh S V asked if it was possible to connect OData services in SAP on-prem systems with the SAP Business Application Studio.

The answer is yes, and the following resources may help:

SAP Business Application Studio: Connecting to External Systems (SAP Help Portal)
The tutorial Create an SAP Fiori App Using SAP Business Application Studio which is part of the tutorial group Develop an SAP Fiori App Using SAP Business Application Studio


text:
CDS Plugin Concept in CAP

Is there any source code that you want to share as part of your demo?

Done

authors: jung-thomas qmacro


text:
Doc: Minimum extensions to run spartacus

could be based on our min recipe for testing
https://stash.hybris.com/projects/C3PO/repos/devconfigs/browse/spartacus/build.gradle
or our yenv recipe
https://stash.hybris.com/projects/INFRA/repos/yenv/browse/templates/spartacus

minimal recipe extension.txt

repo: SAP/cloud-commerce-spartacus-storefront


text:
Remove spartacus demo doc from repo

We don't need it. Remove spartacus_demo.md

Duplicate / already done.

authors: Xymmer gladius-mtl


text:
Variant selection doesn't work

Environment Details

Spartacus:  1.5.0-next.2

Steps to Reproduce
Go to any PDP that has multiple variants.
Click on one of the variants
Observed Results
The user is sent to the home page.
An error is logged No configured path matches all its params to given object.  Route config:  {paths: Array(1), paramsMapping: {…}} Params object:  {code: "96977_white"}
Expected Results
The user should be sent to the PDP of that variant
Additional Information
This is caused by missconfiguration of the router config.
Router config expects both name & code on the product, yet the variant-selector only passes a code.
The variant selector should pass both the code & the name to avoid sending user to a PDP without the name and therefore indexing both PDP with and without name.

QA steps:

Go to app.module.ts and change the line:
paths: ['product/:productCode/:name', 'product/:productCode'],
to
paths: ['product/:productCode/:name'],
Start the app
Go to http://localhost:4200/apparel-uk-spa/en/GBP/product/300738133/Asterisk SS youth washed orange L
Change the style or size variant.
Check that the URL of the reloaded page contains both product code AND product name

QA done, URL's are working as expected.

authors: CarlEricLavoie Matejk00 mszczepaniak