python3Packages.graphqlclient: init at 0.2.4

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 130261 at 087972fe run on aarch64-linux 1

2 packages built successfully:

python38Packages.graphqlclient
python39Packages.graphqlclient

2 suggestions:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/python-modules/graphqlclient/default.nix:19:3:
   |
19 |   meta = with lib; {
   |   ^

warning: no-python-tests
Test runner could not discover any test cases: ‘Ran 0 tests in 0.000s’
Near pkgs/development/python-modules/graphqlclient/default.nix:20:0:
   |
20 |     description = "Simple GraphQL client for Python 2.7+";
   | ^

Package does not have a maintainer. Consider adding yourself?

New packages need to have a maintainer.


Add a button to download current content to a file in the left section

Add a button to download current content in the left section.  Once the user edits in half, they want to leave. The temp content should store in some way. Currently, I think I can add a button to download the temp content to a file in the user's computer, and once the user wants to continue editing, they can load the temp content file in the left section. If there is another way to do it more gracefully, please tell me.

Complete.


Added pypy2-slim Dockerfile for autobahn-python

Hi,
I've added a slim pypy2 docker for myself,
you are welcome to publish it as example for the community.
Also, did the same for crossbar itself, let me know if you are interested :)

If users want to run Crossbar.io on a pypy-slim, it's a one line to change the community Dockerfile. However, officially, we only want to have exactly 2 Crossbar.io  images: community and plus


I encounter the following errors when I "make && make install".

CMakeFiles/gamma.dir/build.make:166: recipe for target 'CMakeFiles/gamma.dir/index/gamma_index_ivfpq.cc.o' failed
make[2]: *** [CMakeFiles/gamma.dir/index/gamma_index_ivfpq.cc.o] Error 1
CMakeFiles/Makefile2:75: recipe for target 'CMakeFiles/gamma.dir/all' failed
make[1]: *** [CMakeFiles/gamma.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2

Has it been solved? How to solve the problem

Has it been solved? How to solve the problem


Create a client that gets stock data.

[ ] Create a client which gets real-time Stock information using the Alpaca API - @DarkLaugh
[ ] Research how to run the client as a Background Service  - @ayonchev


feat: add C ndarray API and refactor blas/ext/base/dnanasumors

Resolves none.
Description

What is the purpose of this pull request?

This pull request:

Adds C ndarray API and refactor blas/ext/base/dnanasumors

Related Issues

Does this pull request have any related issues?

No.
Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

The tests for dnanasumors are passing, the failing test is from dnanasum which uses dnanasumors for computing the sum. This should be resolved when dnanasum is refactored.


Refactor services

Some repo refactoring/restructuring around rpc services and data packages. Also code clean up after running make check.
Verification:
$ ./swarm start --min
$ make test

LGTM
`
2016/08/10 20:20:02 connecting to etcd at http://localhost:2379
2016/08/10 20:20:02 connected to etcd at http://localhost:2379
2016/08/10 20:20:02 amplifier is listening on port 50101
=== RUN   TestCreate
--- PASS: TestCreate (0.00s)
=== RUN   TestGet
--- PASS: TestGet (0.00s)
=== RUN   TestDelete
--- PASS: TestDelete (0.00s)
PASS
ok  	github.com/appcelerator/amp/data/storage/etcd	1.012s
=== RUN   TestQuery
--- PASS: TestQuery (0.00s)
PASS


Potential Improvement: Introduce max Request Time in the Backend

Hi,
I just came up with a potential improvement. We have a timeout parameter in the js file. How about adding a timeout to each request in java?
The timeout could be configureable.
Not sure if this makes sense?
Best,
Robin

This is already done on the jetty side. Also we should do this within GraphHopper itself: #473 The current solution via maximum weight is already working but not suitable if the weighting is not seconds nor meters.

I guess a nice solution would monitor all requests and check if one request ran longer than allowed. This could keep the performance overhead rather small.
The default seems to have no time out set.

But limiting the request duration should be possible with jetty, not sure if it kills the thread then but if this is available from jetty I woud not want to implement this. Of course making this easily configurable should be our aim

Thanks, you are right. I misunderstood your last post. I did a quick research but could not come up with an easy solution. The best I found was:
Include the setMaxIdleTime in the GHServer and make it configureable in the config file. But I am not sure if this setting is doing the trick.

Yes, this looks good.

But I am not sure if this setting is doing the trick.

Why not try it ;) ? They say in the docs that this is highly likely delegated to the JVM via setSoTimeout
BTW: this one was also requested and we can make it configurable as well: connector0.setAcceptQueueSize();

Why not try it ;)

I might some day, but not today. BTW: I guess this would be a good first issue ;).

I just tried setting setMaxIdleTime (also soLingerTime and stopTimeout) but none worked as expected. I will keep you updated if I find a good solution.

Will maxVisitedNode not work for your use case?

No I introduced an endless loop for some rare scenarios (which is already fixed). Due to this I revisited this issue.

But once the endless loop is fixed the maxVisitedNodes can be used? Or why can't you use it?

Yes. I was just looking for a prevention.

@boldtrn maybe you try something like connector0.setIdleTimeout(args.getInt("jetty.idletimeout", 30) * 1000); where did you find setMaxIdleTime? Or was this for jetty 8?

Hmmh setIdleTimeout does not seem to work

It is not possible on the jetty level according to this mailing list thread https://dev.eclipse.org/mhonarc/lists/jetty-users/msg06601.html

Seems to be true :(. I also created a StackExchange Thread some weeks ago. http://stackoverflow.com/questions/36642801/embeded-jetty-kill-request-after-a-given-time
The only answer is about writing it myself using Future. Not sure if we should close this one? Were there any problems corresponding to this issue?

Ups, I just created a SO question as well .. will closing it as duplicate of your question :)

Were there any problems corresponding to this issue?

Well I would like to have a simple control mechanism for the GH user for non-CH routes so that they can make sure their user requests do not overload too much their systems. Although this is controllable with maxVisitedNodes now ... hmmh, not sure what to do.

According to the new answers on my question we could only solve this manually, but as we have the maxVisitedNodes in place I would close for now, only if there are still problems for this. Maybe we should set maxVisitedNodes to some limited value already in the config? This won't affect CH but keeps non-CH safer.
Also this low resource filter is interesting: https://www.eclipse.org/jetty/documentation/current/limit-load.html

Maybe we should set maxVisitedNodes

Makes sense. We should set it probably quite high. To theoretically allow non-ch calculations across Europe, that way most people do not have to modify it, but still you sever won't hang up forever, but just (5min?).

Maybe we should set maxVisitedNodes to some limited value already in the config? This won't affect CH but keeps non-CH safer.

So how can potential DOS situations be avoided with CH enabled? I take it the Jetty path of controlling this is a dead end after reading through all of the above...

@arminus

DOS situations be avoided with CH enabled

Well maxVisitedNodes is not used with CH enabled. You have to enable the flexibility mode for that. If you do so, you should be aware of the problems and how to handle them.
Also IMHO, you should check DOS before actually calling your API. There are lot's of ways to do that, e.g. Cloudflare, API Gateways (e.g. Amazon API Gateway), you name it.

I take it the Jetty path of controlling this is a dead end after reading through all of the above...

Yes, one might implement something like this after jetty and before actually calling GH. See: http://stackoverflow.com/a/36643495/1548788

Also IMHO, you should check DOS before actually calling your API.

Actually, I began to do that by calculating line of sight distances between route points and not passing them on to GH of a max distance threshold is passed...

Cloudflare, API Gateways...

For a commercial solution maybe, not for what I do on a small/non-funded scale. Actually, I'm not worried about an intentional DOS attempt but more about users just clicking "somewhere on the map" (e.g. from Hamburg to Naples) which would result in high loads

http://stackoverflow.com/a/36643495/1548788

I understood this approach was dropped above?

Actually, I began to do that by calculating line of sight distances between route points and not passing them on to GH if a max distance threshold is passed...

Sounds like a reasonable approach. Please add it as a PR once it's done. I think this could be beneficial for GraphHopper as well.

intentional DOS attempt but more about users just clicking

Ah ok that is something different.

I understood this approach was dropped above?

Yes because it would be an extra overhead, that is mostly unnecessary.

Please add it as a PR once it's done.

So far I use GH out-of-the-box, so I do this in JS on the client side... (see an example on https://www.xctrails.org/ferrata)

Actually, I began to do that by calculating line of sight distances between route
points and not passing them on to GH if a max distance threshold is passed...

You still should limit max visited nodes to something as otherwise you can still get intended or unintended bad requests

You still should limit max visited nodes to something

as in "somewhere in the GH code"?

No, via a config, see #473

But this is for non-CH only?

But this is for non-CH only?

Yes, you might end up in an endless loop if you don't define them. (Only for non-ch)

Yes, you might end up in an endless loop if you don't define them. (Only for non-ch)

It is not an endless loop but a really large one ;)
Maybe we should also include an additional distance protector. E.g. routing from paris to nyc can be excluded up-front and should not trigger a search even if limited to 1mio nodes.

Well, I have CH enabled and still need to protect the server against paris to nyc like routing attempts

Well, I have CH enabled and still need to protect the server against paris to nyc like routing attempts

This should not be a problem with CH although you can still protect it but the benefit would be a lot lower

This should not be a problem with CH

My "blackbox-observation" doesn't quite confirm this... I just ran a test using a rather powerful server with a GH database covering D/A/CH/I, with
graph.flagEncoders=car,foot|blockFords=false,mtb|blockFords=false
graph.dataaccess=MMAP_STORE_SYNC
prepare.chWeightings=shortest
prepare.minOnewayNetworkSize=100

->
 2016-06-06 15:23:43,486 [qtp131682780-17] INFO  com.graphhopper.http.GHBaseServlet - vehicle=foot&weighting=shortest&point=53.56641415275043,10.01953125&point=38.75408327579141,16.5234375&instructions=true&type=json&key=undefined <IP_ADDRESS> de_DE Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0 [53.56641415275043,10.01953125, 38.75408327579141,16.5234375], took:169.19841, , shortest, foot, alternatives: 1, distance0: 2039668.5970157352, time0: 24675min, points0: 27934, debugInfo: idLookup:4.2552485s; , algoInit:0.06761164s, dijkstrabiCH-routing:129.13196s, extract time:36.1786;, algoInit:0.06761164s, dijkstrabiCH-routing:129.13196s, extract time:36.1786, simplify (39272->27934)
took:169.19841 is a lot less as to what I actually observed for the entire request and all the while, GH sucked up some considerable CPU - which to me is a clear indicator that I have to worry about this, especially on a smaller server.
Of course an AJAX request would time out long before, but the calculation on the server is still continuing (and possibly heaping up, since a user might click again and again...)
So I still think a maxVisitedNodes threshold parameter would also make sense for CH routing, i.e. stop and throw a respective exception rather than continuing long after a client timeout.

You are using MMAP and have probably only a few available RAM (?). Then yes, you would also need such a protection. I meant the defaults (RAM+CH=true)

So I still think a maxVisitedNodes threshold parameter would also make sense for CH routing

The current maxVisitedNodes should influence CH too, if not raise an issue as this would be a bug

BTW: If you have a powerful server, then do NOT use MMAP!

You are using MMAP and have probably only a few available RAM (?).

on the production server, yes

The current maxVisitedNodes should influence CH too

Your last comment On April 29 indicated otherwise, but ok, I just reran the test, with
routing.maxVisitedNodes=2000
Just restarted the server, didn't rebuild the DB, though, do I need to? (that'll take some 24hours...)
->
2016-06-06 15:51:43,633 [qtp131682780-16] INFO  com.graphhopper.http.GHBaseServlet - vehicle=foot&weighting=shortest&point=53.56641415275043,9.931640625&point=38.85682013474361,16.6552734375&instructions=true&type=json&key=undefined <IP_ADDRESS> de_DE Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0 [53.56641415275043,9.931640625, 38.85682013474361,16.6552734375], took:2.7777863, , shortest, foot, alternatives: 1, distance0: 2048202.6742501908, time0: 24777min, points0: 28002, debugInfo: idLookup:0.7286473s; , algoInit:0.017141698s, dijkstrabiCH-routing:1.1349596s, extract time:0.56115097;, algoInit:0.017141698s, dijkstrabiCH-routing:1.1349596s, extract time:0.56115097, simplify (39533->28002)
much faster now (?!), but according to the simplify info, maxVisitedNodes wasn't taken into account?

BTW: If you have a powerful server, then do NOT use MMAP!

Understood, I only run the tests on the server where I build the GH database, not on the smaller server where I deploy it...

BTW: I'm on GH 0.6

much faster now (?!), but according to the simplify info, maxVisitedNodes wasn't taken into account?

Should work on 0.6 I think but not 100% sure.  You'll see an exception if the setting protects you...

Maybe we should set maxVisitedNodes to some limited value already in the config?
This won't affect CH but keeps non-CH safer.

Here I meant that one default setting like 1mio would not make CH safer at the same time as there you have just a few K nodes.

routing.max_visited_nodes on 0.6 doesn't have any effect either - I'll try with the master later. Can it load the 0.6 DB?

routing.max_visited_nodes on 0.6

on 0.6 it should be routing.maxVisitedNodes. Also have a look into the json response, there you see how many visited nodes you have and can just use a bit lower value (restart necessary) to see if it throws an exception.

Ok, this does work in the master branch (java.lang.IllegalArgumentException: No path found due to maximum nodes exceeded...) with routing.max_visited_nodes but I get no such exception when setting routing.maxVisitedNodes in 0.6

Strange. Still we won't have the time to fix this in 0.6 as 0.7 will be out next week ...

That's fine, I also spent almost half the day yesterday to prevent routing in case line of sight dist > some max value on the client side (which will not be entirely in vain once I have 0.7 since it also now properly cancels waypoint drags etc.)

Regarding the "dist > some max value" I've created a new issue: #734

We have been running into memory and performance issues related to this issue.
At scale, we set a large value for maxVisitedNodes to enable routing against sparse data set.
Are there any improvements planned to map matching to detect cyclical graph traversal and prevent infinite loop without having to rely on maxVisitedNodes? It seems that an edge should not be traversed more than once or have a way to configure the number of re-traversals allowed.


Fix anchors, fix naming consistency

I found non working anchors and some naming in consistency. I hope I fixed well. Have a nice day.

Have I done something wrong?
Probably yes. "All checks have failed" it is not good sign, isn't it.
But I don't know what is wrong. Please, if you know, give me a note.
Thank you very much.

@mattez this is due to a build issue that we are working to address in #17050

@kkirsche oh so it's not my foult, = im happy THANK YOU for your response. Good luck with that issue.

Thanks!

With pleasure. I love Bootstrap. A version 4 looks luxuriously smart. I help with what I can. Thank you!


Mystical gears not working

Mystical gears are not usable in the gearbox despite their text.

i tested 10 or so gears, they all appeared to work. need to make sure the iron anxle is pointing into the receiving hole of the gear box for it to work, it wont crank on every gear. in my case the gearbox hole was in the top


package manager (npm, yarn, etc.) is not asked in style (How would you like to use ESLint) option

Two questions (that is install the deps and what package manager) are skipped when choose to style options.

Hello @hitesh-can-code, could you please provide more details about what you mean by 'style options'?

eslint need to ask what package manager first than checking peersdeps

I couldn't reproduce this on my local machine
> bun create @eslint/config
✔ How would you like to use ESLint? · style
✔ What type of modules does your project use? · esm
✔ Which framework does your project use? · none
✔ Does your project use TypeScript? · No / Yes
✔ Where does your code run? · node
✔ How would you like to define a style for your project? · guide
✔ Which style guide do you want to follow? · standard-with-typescript
✔ What format do you want your config file to be in? · JSON
Checking peerDependencies of eslint-config-standard-with-typescript@latest
The config that you've selected requires the following dependencies:

eslint-config-standard-with-typescript@latest <EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>|| ^16.0.0<EMAIL_ADDRESS>typescript@*
✔ Would you like to install them now? · No / Yes
? Which package manager do you want to use? …
❯ npm
  yarn
  pnpm

> bun --version
1.0.29

Did I miss something?

it is because you have npm installed where i only have bun

it was using npm to install any dependencies, it was used to checking the package's peer dependencies:
https://github.com/eslint/create-config/blob/9c4214bc879cc892fd8ba4f1259c7d0686b0d7c2/lib/init/npm-utils.js#L71-L75
for no-npm-installed envs, we could find another way to do it.


panic: runtime error: index out of range [1] with length 0

Your environment.

Version: github.com/pion/ion-sfu v1.9.2-0.20210302184455-3156f57c6cba
Client: ion-sdk-js 1.6.0
Environement: docker swarm
Are you using a TURN server? ion-sfu turn
Other Information
config

sfu:
# Ballast size in MiB, will allocate memory to reduce the GC trigger upto 2x the
# size of ballast. Be aware that the ballast should be less than the half of memory
# available.
  ballast: 0
# enable prometheus sfu statistics
  withstats: false

router:
# Limit the remb bandwidth in kbps
# zero means no limits
  maxbandwidth: 1500
# max buffer time by ms for video tracks
  maxbuffertime: 1000
# Sets the audio level volume threshold.
# Values from [0-127] where 0 is the loudest.
# Audio levels are read from rtp extension header according to:
# https://tools.ietf.org/html/rfc6464
  audiolevelthreshold: 40
# Sets the interval in which the SFU will check the audio level
# in [ms]. If the active speaker has changed, the sfu will
# emit an event to clients.
  audiolevelinterval: 1000
# Sets minimum percentage of events required to fire an audio level
# according to the expected events from the audiolevelinterval,
# calculated as audiolevelinterval/packetization time (20ms for 8kHz)
# Values from [0-100]
  audiolevelfilter: 20

  simulcast:
# Prefer best quality initially
    bestqualityfirst: true
# EXPERIMENTAL enable temporal layer change is currently an experimental feature,
# enable only for testing.
    enabletemporallayer: false

webrtc:
# Range of ports that ion accepts WebRTC traffic on
# Format: [min, max]   and max - min >= 100
  portrange: [5000, 5100]
# if sfu behind nat, set iceserver
  iceserver:
    - urls: ["turn:myhost.example.com:3478"]
      username: "userN"
      credential: "passW"

# sdp semantics:
# "unified-plan"
# "plan-b"
# "unified-plan-with-fallback"
  sdpsemantics: "unified-plan"
# toggle multicast dns support: https://tools.ietf.org/html/draft-mdns-ice-candidates-00
  mdns: true

  candidates:
# In case you're deploying ion-sfu on a server which is configured with
# a 1:1 NAT (e.g., Amazon EC2), you might want to also specify the public
# address of the machine using the setting below. This will result in
# all host candidates (which normally have a private IP address) to
# be rewritten with the public address provided in the settings. As
# such, use the option with caution and only if you know what you're doing.
# Multiple public IP addresses can be specified as a comma separated list
# if the sfu is deployed in a DMZ between two 1-1 NAT for internal and
# external users.
    nat1to1: ["x.y.z.*"]
#    icelite: true

turn:
# Enables embeded turn server
  enabled: true
# Sets the realm for turn server
  realm: "ion"
# The address the TURN server will listen on.
  address: "<IP_ADDRESS>:3478"
# Certs path to config tls/dtls
# cert="path/to/cert.pem"
# key="path/to/key.pem"
  portrange: [5200, 5300]
  auth:
# Use an auth secret to generate long-term credentials defined in RFC5389-10.2
# NOTE: This takes precedence over `credentials` if defined.
#    secret: "secretAwsome"
# Sets the credentials pairs
    credentials: "userN=passW"

log:
  level: "trace"

chat:
  url:
    base: "http://chat:1235"
    access: "/internal/access"
    notify: "/internal/video/notify"

# Rest client
http:
  maxIdleConns: 2
  idleConnTimeout: '10s'
  disableCompression: false

frontend:
  iceserver:
    - urls: ["turn:myhost.example.com:3478"]
      username: "userN"
      credential: "passW"

stacktrace
 myhost chat-video[4803]: [2021-03-02 19:57:39.289] [INFO] [143][main.go][main] => --- Starting SFU Node ---
 myhost chat-video[4803]: [2021-03-02 19:57:39.293] [INFO] [178][main.go][main] => Listening at http://[:7000]
 myhost chat-video[4803]: [2021-03-02 19:57:39.295] [INFO] [114][main.go][startMetrics] => Metrics Listening at :8100
 myhost chat-video[4803]: [2021-03-02 20:00:37.964] [INFO] [198][handler.go][addPeerToMap] => Storing peer for userId=3 to chatId=23
 myhost chat-video[4803]: [2021-03-02 20:00:38.019] [INFO] [147][peer.go][Join] => peer cklsfrers000001179ladu0yz join session chat23
 myhost chat-video[4803]: [2021-03-02 20:00:38.019] [INFO] [160][peer.go][Answer] => peer cklsfrers000001179ladu0yz got offer
 myhost chat-video[4803]: [2021-03-02 20:00:38.022] [DEBUG] [63][publisher.go][func3] => ice connection state: checking
 myhost chat-video[4803]: [2021-03-02 20:00:38.027] [DEBUG] [128][peer.go][func3] => on publisher ice candidate called for peer cklsfrers000001179ladu0yz
 myhost chat-video[4803]: [2021-03-02 20:00:38.028] [DEBUG] [128][peer.go][func3] => on publisher ice candidate called for peer cklsfrers000001179ladu0yz
 myhost chat-video[4803]: [2021-03-02 20:00:38.023] [INFO] [171][peer.go][Answer] => peer cklsfrers000001179ladu0yz send answer
 myhost chat-video[4803]: [2021-03-02 20:00:38.034] [INFO] [204][peer.go][Trickle] => peer cklsfrers000001179ladu0yz trickle
 myhost chat-video[4803]: [2021-03-02 20:00:38.036] [INFO] [204][peer.go][Trickle] => peer cklsfrers000001179ladu0yz trickle
 myhost chat-video[4803]: [2021-03-02 20:00:38.147] [DEBUG] [63][publisher.go][func3] => ice connection state: connected
 myhost chat-video[4803]: [2021-03-02 20:00:38.270] [DEBUG] [101][peer.go][func1] => peer cklsfrers000001179ladu0yz negotiation needed
 myhost chat-video[4803]: [2021-03-02 20:00:38.272] [INFO] [110][peer.go][func1] => peer cklsfrers000001179ladu0yz send offer
 myhost chat-video[4803]: [2021-03-02 20:00:38.273] [DEBUG] [116][peer.go][func2] => on subscriber ice candidate called for peer cklsfrers000001179ladu0yz
 myhost chat-video[4803]: [2021-03-02 20:00:38.274] [DEBUG] [116][peer.go][func2] => on subscriber ice candidate called for peer cklsfrers000001179ladu0yz
 myhost chat-video[4803]: [2021-03-02 20:00:38.290] [INFO] [184][peer.go][SetRemoteDescription] => peer cklsfrers000001179ladu0yz got answer
 myhost chat-video[4803]: [2021-03-02 20:00:38.290] [DEBUG] [57][subscriber.go][func1] => ice connection state: checking
 myhost chat-video[4803]: [2021-03-02 20:00:38.291] [INFO] [204][peer.go][Trickle] => peer cklsfrers000001179ladu0yz trickle
 myhost chat-video[4803]: [2021-03-02 20:00:38.310] [INFO] [204][peer.go][Trickle] => peer cklsfrers000001179ladu0yz trickle
 myhost chat-video[4803]: [2021-03-02 20:00:39.372] [DEBUG] [57][subscriber.go][func1] => ice connection state: connected
 myhost chat-video[4803]: [2021-03-02 20:00:39.828] [INFO] [160][peer.go][Answer] => peer cklsfrers000001179ladu0yz got offer
 myhost chat-video[4803]: [2021-03-02 20:00:39.830] [INFO] [171][peer.go][Answer] => peer cklsfrers000001179ladu0yz send answer
 myhost chat-video[4803]: [2021-03-02 20:00:39.832] [INFO] [204][peer.go][Trickle] => peer cklsfrers000001179ladu0yz trickle
 myhost chat-video[4803]: [2021-03-02 20:00:39.833] [INFO] [204][peer.go][Trickle] => peer cklsfrers000001179ladu0yz trickle
 myhost chat-video[4803]: [2021-03-02 20:00:39.842] [INFO] [204][peer.go][Trickle] => peer cklsfrers000001179ladu0yz trickle
 myhost chat-video[4803]: [2021-03-02 20:00:39.842] [INFO] [204][peer.go][Trickle] => peer cklsfrers000001179ladu0yz trickle
 myhost chat-video[4803]: [2021-03-02 20:00:40.136] [DEBUG] [48][publisher.go][func1] => Peer cklsfrers000001179ladu0yz got remote track id: 4965592e-c32a-4864-a654-4857956911d5 mediaSSRC: 739820034 rid : streamID: e5d85e22-b6e2-4998-85ea-d763ef5b0af7
 myhost chat-video[4803]: [2021-03-02 20:00:40.136] [DEBUG] [180][router.go][AddReceiver] => NewBuffer BufferOptions={1500}
 myhost chat-video[4803]: [2021-03-02 20:00:41.670] [DEBUG] [48][publisher.go][func1] => Peer cklsfrers000001179ladu0yz got remote track id: 61acfdb8-e462-4216-ab6a-4911242f80e1 mediaSSRC:<PHONE_NUMBER> rid : streamID: e5d85e22-b6e2-4998-85ea-d763ef5b0af7
 myhost chat-video[4803]: [2021-03-02 20:00:41.670] [DEBUG] [180][router.go][AddReceiver] => Setting feedback goog-remb
 myhost chat-video[4803]: [2021-03-02 20:00:41.670] [DEBUG] [180][router.go][AddReceiver] => Setting feedback transport-cc
 myhost chat-video[4803]: [2021-03-02 20:00:41.670] [DEBUG] [180][router.go][AddReceiver] => Setting feedback nack
 myhost chat-video[4803]: [2021-03-02 20:00:41.670] [DEBUG] [180][router.go][AddReceiver] => Setting feedback nack
 myhost chat-video[4803]: panic: runtime error: index out of range [1] with length 0
 myhost chat-video[4803]:
 myhost chat-video[4803]: goroutine 200 [running]:
 myhost chat-video[4803]: encoding/binary.bigEndian.PutUint16(...)
 myhost chat-video[4803]:         encoding/binary/binary.go:106
 myhost chat-video[4803]: github.com/pion/ion-sfu/pkg/buffer.(*Bucket).push(...)
 myhost chat-video[4803]:         github.com/pion/ion-sfu@v1.9.2-0.20210302184455-3156f57c6cba/pkg/buffer/bucket.go:59
 myhost chat-video[4803]: github.com/pion/ion-sfu/pkg/buffer.(*Bucket).AddPacket(0xc00044aab0, 0xc000116900, 0x471, 0x471, 0x11297, 0x0, 0xd4c8e0, 0xc00000e018, 0x0, 0xc7956d)
 myhost chat-video[4803]:         github.com/pion/ion-sfu@v1.9.2-0.20210302184455-3156f57c6cba/pkg/buffer/bucket.go:37 +0x279
 myhost chat-video[4803]: github.com/pion/ion-sfu/pkg/buffer.(*Buffer).calc(0xc000301e00, 0xc000116900, 0x471, 0x471, 0x16689ef507d9cdb9)
 myhost chat-video[4803]:         github.com/pion/ion-sfu@v1.9.2-0.20210302184455-3156f57c6cba/pkg/buffer/buffer.go:304 +0x1e5
 myhost chat-video[4803]: github.com/pion/ion-sfu/pkg/buffer.(*Buffer).Bind(0xc000301e00, 0xc000432de0, 0x3, 0x4, 0xc00033af60, 0x1, 0x1, 0x5dc)
 myhost chat-video[4803]:         github.com/pion/ion-sfu@v1.9.2-0.20210302184455-3156f57c6cba/pkg/buffer/buffer.go:169 +0x287
 myhost chat-video[4803]: github.com/pion/ion-sfu/pkg/sfu.(*router).AddReceiver(0xc000227e00, 0xc00033b0e0, 0xc000235e60, 0x0, 0x0, 0xc0002cad00)
 myhost chat-video[4803]:         github.com/pion/ion-sfu@v1.9.2-0.20210302184455-3156f57c6cba/pkg/sfu/router.go:180 +0x4ad
 myhost chat-video[4803]: github.com/pion/ion-sfu/pkg/sfu.NewPublisher.func1(0xc000235e60, 0xc00033b0e0)
 myhost chat-video[4803]:         github.com/pion/ion-sfu@v1.9.2-0.20210302184455-3156f57c6cba/pkg/sfu/publisher.go:49 +0x23b
 myhost chat-video[4803]: created by github.com/pion/webrtc/v3.(*PeerConnection).onTrack
 myhost chat-video[4803]:         github.com/pion/webrtc/v3@v3.0.10/peerconnection.go:451 +0x10f

It worked correctly on my forked repository on commit github.com/nkonev/ion-sfu v1.9.1-0.20210226202827-62c8b234441f
It seems root cause this one https://github.com/pion/ion-sfu/pull/449/files#diff-193890e385023f88670a054946ba694a057953837760a07fa9c4863ce35e436aR59
What did you do?
I open js client then establish webrtc connection. I tested with Safari 14.0 (156<IP_ADDRESS>.9, 15610), Chrome 88.0.4324.192
What did you expect?
Not to see this error
What happened?
Panic

It seems duplicate of https://github.com/pion/ion-sfu/issues/450


Git

This does not work as described with a Git repo

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

ID: 29c747d0-263d-1ed2-1e6a-477a712ef1d3
Version Independent ID: 7d062cbc-c549-fde4-4e9a-d04200c3c3e9
Content: Compare folders
Content Source: docs/tfvc/compare-folders.md
Service: unspecified
GitHub Login: @douge
Microsoft Alias: sdanie

@JuanZamudioGBM This article describes the compare folder process for Team Foundation Version Control, which is a different version control system than Git. Here is an article that describes how to view history and file differences with Git: Review history. Does that info help?


Reduce OPENTHREAD_CONFIG_NUM_MESSAGE_BUFFERS to 32 for nrf5

Problem
OPENTHREAD_CONFIG_NUM_MESSAGE_BUFFERS is 160 as a default for nrf52840 example apps.

Summary of Changes
Change OPENTHREAD_CONFIG_NUM_MESSAGE_BUFFERS to 32

Related #3417

PR
arm-none-eabi-size out/chip-nrf52840-lighting-example
   text	   data	    bss	    dec	    hex	filename
 484244	    524	  80464	 565232	  89ff0	out/chip-nrf52840-lighting-example

master
arm-none-eabi-size out/chip-nrf52840-lighting-example
   text	   data	    bss	    dec	    hex	filename
 483828	    524	  96848	 581200	  8de50	out/chip-nrf52840-lighting-example

Should something like this apply to other platforms too?


Unhandled exception.

Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
at Program.$(String[] args)

Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object. at Program.$(String[] args)

Are you able to describe what causes this issue

Just happens when I open it, idk why

Just happens when I open it, idk why

Did you compile the application yourself, or have you used the pre compiled exe in the latest release?

I used the latest pre compiled exe

Windows 10

Windows 10

Can you send a screenshot of the window that shows the error

If you are still having issues launching.
Please make sure you have met all of the requirements.
Make sure you have .NET 6 installed
Make sure you have the latest version of obs installed / obs virtual camera
If the application launches and asks for administrator permissions, click Yes
If it doesn't launch and ask for administrator permission, then run it as administrator.
Make sure your antivirus is not blocking it,
Mcafee
Avg
And other bad antivirus programs like the above, tend to block many harmless programs.

I have the same problem, I've met all the criteria mentioned above, but still the same problem

Fixed-
releasing Quick patch rn
sorry for the delays.

New version is-
https://github.com/LudoDash/Obs-VCam-Changer/releases/tag/V1.3-Latest-Patch


Bitbucket commit URL invalid

When selecting the View button on a commit, the extension opens the URL of https://bitbucket.org/MyCompany/MyRepo/commit/MyCommitHash which leads to a 404 page. It should be  https://bitbucket.org/MyCompany/MyRepo/commits/MyCommitHash

The correct repo to open issues is:
https://github.com/Sertion/vscode-gitblame/issues
You can avoid the 404 page in bitbucket with the following steps:
Go to Settings, search for Gitblame: Is Web Path Plural and check that option.
More info: https://github.com/Sertion/vscode-gitblame/issues/31


request

can you make it so we can drag and drop

Drag and drop what

drag and drop the youtube video or link. and have it go to the next line automaticlly instead of having to hitting enter to go to next line or shift + enter


Fixing targets

Added missing file to other iOS Targets
Removed swiftlint from Travis CI
Updated the Xcode and Simulator versions for Travis CI
Fixed podspec version causing podspec to not validate

Tests Pass

@akosma thoughts?

Sounds good!


ledger-centrifuge cannot transfer out the CFG token

hi,
according to the official documentation of ledger, i imported ledger-centrifuge to my talisman wallet , everything is going smooth, then, I sent the CFG token to the ledger-centrifuge, ledger-centrifuge wallet balance can also be displayed normally, but I can't transfer it out from the ledger-centrifuge.
When transferring tokens from ledger-centrifuge, the "Approve on Ledger" button pops up, and an error is reported after clicking: this type of transaction is not supported on your ledger. you should check for firmware updates in ledger live before trying again.
please help me to solve this problem.
related information is as follows:
ledger doc : https://support.ledger.com/hc/en-us/articles/11158925739677-Centrifuge-CFG-?support=true
app version:
talisman : 1.17.5
ledger live : 2.62.2
ledger centirfuge : 1.10070.1
ledger firmware version : 2.2.1
hey are all up to date, and it doesn't seem to be an issue with the version.

:link: zboto Link

Hi @lzxeth! I apologize for the inconvenience you're facing. Unfortunately, the Centrifuge app is no longer maintained by our team, and it appears to be broken. We recommend reaching out to the Centrifuge team for assistance or exploring alternative solutions


Fix #1503: ensure child composables are drawn, by using Layout ID rather than SquooshResolvedNode identity.

The previous check (that the SquooshResolvedNodes were equal) failed, because during a transition we generate a new set of SquooshResolvedNodes from the source tree and dest tree. With this change, we consult the layout ID, which is unique within a tree (so still useful for this purpose of identifying the selected node for child rendering).

@rylin8 PTAL - I want to add tests to this; sharing now because I needed it to make a demo work.

So replacement content, components, and nodes with interactions don't render with animations?

So replacement content, components, and nodes with interactions don't render with animations?

Interactions work fine during animation, because those are all Composables that never draw anything at all. Replacement content that draws content (i.e.: all of it) was getting hidden during animations.

LGTM. Let me know if you plan on adding tests to this PR or separately -- or if you'd like me to.


Allow insecure registries anywhere in <IP_ADDRESS>/8 for all cloud providers

<IP_ADDRESS>/8 is with a company's intranet, so it can be insecure.
This is similiar to "For GCE, allow insecure registries anywhere in <IP_ADDRESS>/8"  https://github.com/GoogleCloudPlatform/kubernetes/commit/f8505cd286353b540f28bed32977c1b6247ce616#commitcomment-8879809
Thanks!

This was closed by #2620


Adding/Scanning for newly added disk(s)

How to to scan for new added disks to harvester node. I tried rescan-scsi-bus.sh -a -l and rebooting node.
Also is there a way to use locally attached storage via fiber channel to each node ?

Can the following command work?
for i in `ls /sys/class/scsi_host/*/scan`;do echo "- - -" > $i;done


Opus Audio Support

Pre-emptive pull request to add opus and ogg support via libopus, libopusfile, and libogg.

Unable to properly load libopus-0.dll at this time; investigating potential issues.

Works on Linux; may have to look into alternative DLLs for Windows systems (or it's a conflict in Ruby).


Fix README for merge many wheels

Very tiny fix for the README.
Thanks for the package, btw.

Hi @bp-kelley , thanks for your contribution! I think that is good to go! :)


predictable_random_int() should include script name

predictable_random_int() currently randomises based on $MODULE_DIR - it should also be random based on the script that calls it.  Could use debug_backtrace() to get the script name that called it.  This allows the random ints to be random between scripts within a module
Not that updating this will update all cron times, so it would require a mass update
Do at same time as https://github.com/silverstripe/module-standardiser/issues/21
PR

https://github.com/silverstripe/module-standardiser/pull/39

PR merged


Annotated dataset Card template

There is a current Dataset card template that has"annotations" in the dataset repo (here), it is the annotated version of the dataset card template.
I want to condense the detailed dataset repo template to make an annotated version that resides in the hub docs, similar to the model card annotated template.
Reason/ pro: Working towards updating datasets cards and keeping them inline/similar standard of treatment that model cards have been getting.
(cc'ing @osanseviero @julien-c , @meg-huggingface )

cc @yjernite , who worked on the Dataset Card HOWTO


Automatic Calibration

Hi
I am trying the automatic calibration procedure. I want help for automatic calibration
Bug Reports

Version of VIC that you are using (e.g. VIC.5.0.0)
Name and version of the C compiler you are using
Operating system
A description of relevant model settings
A summary of the bug or error message you are getting

Feature Requests

Description of feature
Description of the problem the feature addresses
Will the proposed feature be backward compatible?
Will the proposed feature change the science results of VIC?

@Riaz3672 - this is a usage question and the best place for it is the VIC users list serve<EMAIL_ADDRESS>


Migrate from docker to podman

Type of change

Enhancement / new feature

Description

Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

Closed because there is no need for this change.


Index_id in group by clause create duplicate records

Index_id & Object_id in group by clause creates duplicate records. Requesting to update the script.

Please provide more details.  Which script and under what conditions are duplicates seen? Is the duplicate record for a partitioned table?

On this sql file FindCCITablesWithLargeDeletedRowCount.sql.
Ideally it should return the schema and table name with more deleted rows, but now it is giving duplicate records for a single table.

I am also seeing this problem with the Synapse/SynapseCCIGuidance/FindSmallCCITables.sql script. It is summing rg.[total_rows], but it is also grouping by rg.[object_id], so each row group gets its own row count, instead of summing the row count for the whole table. There are cases where we have very big tables, but one row group has very few rows, and these row groups are showing up in the query.
Example:
Table1, RowGroup1: 60000000 rows
Table1, RowGroup2: 100 rows
Then query will output - Table1: 100 rows because of the group by object_id. If that were removed, the two groups would be summed up and Table1 would be correctly filtered out as having more than 60 million records.


A variety of warning/buglet fixes revealed by clang-tidy static analysis

Some notable/interesting fixes:

Marking of ImageInput/ImageOutput subclasses as final fixes a no-no that I had overlooked, which is that we usually called close() from within the class destructors. But it's virtual, and it's generally considered unsafe to call a virtual function within a destructor (unless it's a final class) because it might not end up being the version for that level of the class hierarchy.

alloca(0) has undefined behavior, so we change the OIIO_ALLOCA macros to detect that case and return NULL rather than blindly calling alloca(0).

Calling defined() within a macro definition itself is apparently not guaranteed to work with all compilers (though in practice it seems ok).

I'm starting to favor the idiom of specifying the default initializers of non-static struct/class data members in their declaration, rather than in long lists of initializers in the constructors themselves. It's a compact notation, removes redundancy (or inadvertent mismatches) when you have multiple constructors, and if you rearrange structure member ordering, you don't have to chase down the warnings about initializer lists being in the wrong order.

LGTM!


Allow to have arrow keys and cut/copy/paste buttons at the same time

Hello, I'll start by sayingthanks for this awesome app, it is really useful to me.
I prefered a previous version of this app where it was possible to have arrow keys on the top of the keyboard, and have copy/paste buttons. If I remember correctly I think they were where are the Fn (F1, F2, ...) buttons now.
I saw that there is an option Replace arrows with cut/copy/paste buttons but I regularly use arrows and cut/copy/paste. Could you give the possibility to have them both please?
For example, could you add an option to replace Fn buttons with cut/copy/paste buttons?

How about the cut/copy/paste buttons in the new Clipboard section? Pressing Ctrl changes the  "SYM" buttom to a "CLIP" button, so when you click Ctrl, then CLIP, you'll see the cut/copy/paste buttons. Let me know if that works for you.
https://user-images.githubusercontent.com/19718602/127731334-ee2f9626-0be6-4e1b-9d97-53995de78bd1.mp4

Oh, cool! I didn't notice the CLIP button, sorry
Yes this is perfect, thanks!


[wip] JSON error responses

[x] render.* endpoints
[ ] execute endpoint

Current coverage is 84.82%

Merging #307 into master will decrease coverage by -0.12% as of 69094ff

@@            master    #307   diff @@
======================================
  Files           38      39     +1
  Stmts         4232    4264    +32
  Branches       542     544     +2
  Methods          0       0
======================================
+ Hit           3595    3617    +22
- Partial        145     148     +3
- Missed         492     499     +7

Review entire Coverage Diff as of 69094ff

Powered by Codecov. Updated on successful CI builds.


Proper threading and thread-safety with sounddevice?

I'm building a higher-level multi-channel audio playback library built on top of sounddevice that wraps sounddevice and some other libraries (like PySoundFile and python-samplerate) to support easily reading audio files, playing multiple files as once (may require automatic samplerate conversion), adjusting/fading volume, complex channel remapping, and playback event callbacks. I'm a bit confused trying to figure out what is best practice for how many threads to use and what to do on each thread and yet be threadsafe enough to hide this complexity from users of my library who just want to asynchronously play audio files, adjust their volume while they're playing, and be alerted when they've finished without having to actually think about the streaming and so forth.
From examples it seems standard practice to use callback streams rather than blocking streams and to use a queue to pass blocks of samples from the thread that is generating the data to the callback function (running in a thread created by PortAudio, I assume) in order to prevent the callback from being able to block. Doing this, I effectively have 3 threads: the main thread that application would use to request sound files to play, a processing thread in my library that does the work of reading those sound files in chunks and enqueuing them (asynchronously with respect to the main thread), and the stream callback thread.
One question I have is, what's the value of having that separate processing thread instead of doing all the file reading, channel remapping, etc in the callback or just writing to a blocking stream from the processing thread? I know that the callback should do as little as possible so it doesn't block, but if my processing thread can't generate the blocks of audio at the same rate the callback thread consumes them, then I'm going to get audio glitches/drop-outs anyway. Is this extra thread not just overhead if I need to hope I can generate audio fast enough regardless? Is the callback blocking for too long somehow worse than it just returning zeros one frame if the queue is empty? Or is there some reason it needs to return faster than the full amount of time between frames? Or is it simply about using the size of the queue as a buffer so that you won't get dropouts if an occasionally slow to generate frame only slightly drains the queues as long as the average is fast enough?

Sorry for the late response!
First of all, you should consider whether Python is really the right language for your use case.
If you want a reliable library that makes efficient use of multiple CPUs, you might be more successful with a compiled language, such as e.g. Rust or C++ or even in C.
Very recently, I've done a few experiments in Rust that sound kinda similar to what you are trying to do, see https://github.com/mgeier/disk-streaming.
Having said this, as long as you don't need a very short latency, you might be fine doing this with Python (even if the resulting library may be more CPU-hungry than other solutions).

From examples it seems standard practice to use callback streams rather than blocking streams ...

To some extent this may be a matter of taste. I personally prefer using callback streams.

... and to use a queue to pass blocks of samples from the thread that is generating the data to the callback function

Yes, that's pretty common.

... (running in a thread created by PortAudio, I assume)

Exactly.

what's the value of having that separate processing thread instead of doing all the file reading, channel remapping, etc in the callback

Reading data from a file (or opening a file, or writing, ...) might some times take longer than normal, and if you do everything in the audio callback, you might have dropouts because of that.
Of course on average reading from file must be faster than real-time, but by using a queue (as long as it is reasonably filled), you can compensate some delays that happen from time to time.
If you use very long block sizes, you might not need this kind of buffering, but you never know how long your disk (or even network drive!) might block.

... if my processing thread can't generate the blocks of audio at the same rate the callback thread consumes them, then I'm going to get audio glitches/drop-outs anyway.

That's true. On average.
But with a bit of buffering you can avoid occasional glitches, as long as it doesn't happen too often (and there is enough time to re-fill the queue).

Is the callback blocking for too long somehow worse than it just returning zeros one frame if the queue is empty?

No, that's basically the same thing (and it's bad). But the queue should not be empty! If the queue is empty, you have chosen a too short length.

Or is it simply about using the size of the queue as a buffer so that you won't get dropouts if an occasionally slow to generate frame only slightly drains the queues as long as the average is fast enough?

I should probably have read your whole text before answering.
Because that's exactly it. That's basically what I've tried to say above ...

I appreciate the response. I went ahead with my work anyway, and so far things seem fine. I considered deleting my question since I have things working, but I wanted to see if there was anything I was missing.

If you work is open source, please consider posting a link here. I would be interested and probably others, too!

I would love to do that. I'm currently in the process of trying to convince my boss (and through him our investors) to let me open source this (and other python libraries I've been writing for this project).


Potentially not clear enough that you have filters selected on mobile

What
Once you've closed the filter drawer on mobile, there isn't any visual signifier to let you know that you have filters selected. This might be particularly confusing if you've selected a filter, closed the drawer, then entered a different search term (not necessarily expecting the filter to carry across to the new search).
Where
/works at screen width < 600px

Have we tried to find a way to not hide filters on mobile?
Or maybe have the filter list (the thing you can cancel filters from) visible?

I think at the least having some way of seeing that you're doing any sort of filtering at all is important, yeah. Probably one for @DominiqueMarshall and @GarethOrmerod.

at the least having some way of seeing that you're doing any sort of filtering

Seems strange to hide everything and then try to hint at that we're hiding everything... but I am very much aware I am probably missing context and journey we've taken to get here.

A common pattern for this would be for the filter button to have an altered/highlighted state after a selection has been made. I can work something up.

Decision at design review: have Filters(x) as the wording on the button that displays to indicate that filters are in use

After not-particularly-rigorous guerilla testing the decision was updated with having the active filter count contained in a yellow box within the button.


Figure out Expo integration

I was wondering if haul was providing a way to test the mobile app on expo thanks.

@canercandan why closing this issue instead of attaching an enhancement tag?

@hendrul I leave this decision to the project collaborators cc @thymikee thanks

Hey folks, any update on this one?


How to dynamically add card to stack?

I'm creating an initial stack of cards and want to add a new card at the bottom of the stack to replace a card that has been thrown out. Is there a way of achieving this?
Here is the function I'm using. Currently cards are being added at the top of the stack. Using $scope.stack.push instead of unshift also produces the same result.
$scope.addCard = function(name) {
  $scope.stack.unshift({
    'name': name
  });
};

This works for me:
$scope.friends.unshift(friend);

I tried the same as @m10l mentioned, tried with push and also with unshift - it does not work

I found out, that it was because of "track by card.uid". Changed to "track by $index" - now it works as intended!

Track by $index and  $scope.apply() helped me.


Log can of mixed everything

Noticed this wasn't logged, despite the other three items that drop items being logged. Could be more items than this not being logged?

Actually, should the message You think you see a weird thing out of the corner of your eye, but it turns out to be nothing. Which is actually pretty weird, if you think about it, man. be logged as well?

Noticed this wasn't logged

Thanks!

Could be more items than this not being logged?

Probably!

Actually, should the message ... be logged as well?

Don't think so - why would it?


CDN build with media color mode

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
Provide a CDN build where color mode uses media queries instead of the HTML attribute.
Motivation and context
This allows automatic dark mode more easily for prototypes and examples which use the CDN builds.

You can just use a tiny 1 line JavaScript in the head which does the same thing:
<script>
// Switch to dark mode if user has it set
  if (window.matchMedia && window.matchMedia("(prefers-color-scheme: dark)").matches) {
   document.documentElement.setAttribute("data-bs-theme", "dark");
  }
</script>

I agree but it would be nice to have it work automatically through the media query without the need for JavaScript.


Can not create an organization without default_environment

default_environment is an optional argument of resource awx_organization. However if you don't set the variable, the provider still tries to associate the environment with id=0 to the organization, which result in a fail if it does not exist.
  # awx_organization.orga will be created
  + resource "awx_organization" "orga" {
      + id        = (known after apply)
      + max_hosts = 0
      + name      = "Organization name"
    }

Plan: 3 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

awx_organization.orga: Creating...
╷
│ Error: Unable to create Organization
│
│   with awx_organization.orga,
│   on organizations.tf line 1, in resource "awx_organization" "orga":
│    1: resource "awx_organization" "orga" {
│
│ Unable to create Organization got Errors:
│ - default_environment: [Invalid pk "0" - object does not exist.]

Thanks for the contribution @quentinleclerc! The build is currently failing due to some changes in golang-ci lint, which subsequently fired off some other lint fixes I will need to make. I'll approve this then submit another PR to clean up the linting changes.

ok, v1.0.10 released which should include your changes.

Perfect, thanks for quick answser!


Introduce release branches

Hi, thanks once more for just-the-docs.
Currently, I'm using it like this: remote_theme: pmarsceill/just-the-docs
I would like to use it like this: remote_theme<EMAIL_ADDRESS>I.e. I'm asking if it would be possible to create "release branches", so that we can use them as remote_themes, gaining stability, while avoiding the need to fork.
They don't need to be something formal or frequent. Even a small "stable branch" vs "development branch" will do.
3-4 times so far, I tried to push something to the sites where I use just-the-docs (ltsp.org and epoptes.org), and build failed, and I had to spend some time to re-sync my site immediately, while I would prefer to do that e.g. once per year, to sync with the next "release".
Or is this a bad idea and users should just fork and use them as remote_themes and update them when they like?

@alkisg It is indeed a good idea to pin the theme version on sites that are infrequently updated.
As indicated in a previous issue , the version to use can be specified in _config.yml, e.g.:
remote_theme<EMAIL_ADDRESS>
Note that the v above is obligatory.
For local building, the installed version of the theme gem should be the same, e.g.:
gem "just-the-docs", "0.3.3"

(no v here).
The available release versions are listed at https://github.com/pmarsceill/just-the-docs/releases

Thank you @pdmosses, I didn't know tags were also supported and actually already in use, I'll do it that way. Closing!


typicalPersonsAddressBook.xml: remove @code tag for header comment

@code tag is used for javadocs and not .xml files.

For clarification, the offending line is this:
https://github.com/se-edu/addressbook-level4/blob/95611eb437b8e84e51f4b0555fa483fbad05a16b/src/test/data/XmlSerializableAddressBookTest/typicalPersonsAddressBook.xml#L3


Get a Property with dots in the name from a JSON Payload

I just love this piece of software , upon using it , i came across a use case that i couldn't get to work , which is having the following JSON Payload :
"properties": {
    "sonar.analysis.buildnumber": "1567096760967"
  }

and i want to send this build number as an argument to the command , but couldn't get that to work.
i tried using the full name , referencing it with the index but in vain.
Thank you

Good question.  That's not currently supported since we naively treat dots as parent-child boundaries (see hook.GetParameter).
How are you trying to reference that value in your hook config?  Or how do you propose to access the value?
Quick Braindump
Option 1: Quote property value
      {
        "source": "payload",
        "name": "blah.properties.\"sonar.analysis.buildnumber\""
      }

We'd have to write a string parser for this or use regex.
Option 2: Make the delimiter configurable
    "object-delimiter": "|",
    "pass-arguments-to-command":
    [
      {
        "source": "payload",
        "name": "blah|properties|sonar.analysis.buildnumber"
      }

This would be simple and flexible.

Both suggestions landed the same error :
[webhook] 2019/09/03 10:19:30 [8ed707] error extracting command arguments: couldn't retrieve argument for {Source:payload Name:project|name EnvName: Base64Decode:false}
[webhook] 2019/09/03 10:19:30 [8ed707] error extracting command arguments: couldn't retrieve argument for {Source:payload Name:properties|sonar.analysis.buildnumber EnvName: Base64Decode:false}
I managed to create a workaround , if i get the whole properties value , i get the following result :
[map[sonar.analysis.buildnumber:1123324545]
So using Sed in the shell file , i was able to extract the value that i want

The options I gave before are only ideas for enhancing webhook functionality.  I didn't mean for you to try them.  😃
I'm glad you found a work-around, but I'd like to keep this enhancement request since I believe it's an important improvement.

I like the idea with the object separator, but the issue would still persist if the user explicitly specifies it as ..
We could also try to modify the hook.GetParameter to check if the full property path exists, and return it, otherwise it should continue, breaking it down until it reaches the exact match?
@moorereason unrelated to this issue, I was thinking of writing a spec for the next version (v3?) which  would tackle some of the issues such as:

easing up the configuration on users (probably creating a more expressive DSL).
This would allow us to extend webhook with new features more easily, and get rid of the ugly JSON. YAML wasn't the right move to solving the configuration readability.
adding rate limiting option
introducing middleware plugins (so we can allow for a bit of payload manipulation, customized validators etc...)
querying the status of async hook executions

What do you think?

@adnanh,
I considered that option, too, but didn't include it for whatever reason.  I think it may work and be more intuitive.
As to v3, all of that sounds pretty straight-forward with the exception of the DSL.  The current config is tedious (mainly the trigger rules), but I'm curious what you're thinking of in terms of a DSL.  I generally prefer existing formats to DSLs for config files.  DSLs can be more concise and expressive, but they come with their own cognitive load and learning curve (and maintenance cost).  I look forward to see your proposal.  😉


Started to translate Basic HTML and HTML5 in French

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Hi,
If you want to translate the entirety of the curriculum into French and are actively interested in maintaining it, please open an issue.

Hi,
If you want to translate the entirety of the curriculum into French and are actively interested in maintaining it, please open an issue.

Thanks for the information but I already did it.
How can I merge my commit now ?

Hi @aenkirch
Sorry about the delay in getting back to you. I have updated the issue here: https://github.com/freeCodeCamp/freeCodeCamp/issues/36877
We are incredibly grateful for your interest in translating. We recommend you take a look at other help wanted issues if you are interested.
Thanks.

If you want to translate the entirety of the curriculum into French and are actively interested in maintaining it, please open an issue.

That's fine. No one needs to maintain the entirety of the curriculum, you can ignore these comments. :)


Skew is double-declared in mixins.less

removed wrong Skew-Function

added translate3d to mixins.less

Merged manually into 2.0.1-wip. Thanks!


[webapi][XWALK-3459] Update common script for NFC API

Add checking for testharness to avoid outer script duplicate

Impacted tests(approved): new 0, update 58, delete 0
Unit test platform: [IVI]
Unit test result summary: pass 58, fail 0, block 0

Hm, let's avoid document.write. Please remove them from the support file and update test files in need.

@zqzhang
I have updated it following your comments.

Replace String.search() with String.indexOf()
Remove language attribute on script

I still keep document.write.  If we update common script with others way like append(),  the load of outer script is asynchronous. It may lead to that testharness.js/testharnessreport.js have not been loaded successfully while test process has begin.


feat(Scalar.AspNetCore): support multiple scalar endpoints

This PR introduces support for registering multiple MapScalarApiReference endpoints. To achieve this, I decided to clone the options before invoking them in the action. This approach ensures that the options can be invoked multiple times without side effects.
Closes #3901

@xC0dex Will this also enable Scalar to pull OpenApi specs from remote hosts?
Imagine you have 5 microservices on their own URLs, each exposing an OpenApi spec. Then you have one documentation site that wants to consume all of those OpenApi spec documents and show them in a single scalar UI.
This is possible in Swagger using the swaggerOptions.SwaggerEndpoint option, but I can't seem to find anything similar for Scalar.

I think I'll close this PR and open a new one once I have a suitable solution for it.


Brainstorm KPIs/Metrics for the OBPS System

Key Performance Indicators (KPIs) allow us to measure how well we're delivering our service. In ideating our KPIs, we can come to common understanding of our definition of success for our new system. Useful KPIs are often specific, measurable, attainable, and time-bound -and should be evaluated & re-evaluated over time.
Acceptance Criteria:

[ ] Add to me! (how should we go about doing this work?)

Goal of the ticket:

Ideate what success looks like in the OBPS system
Brainstorm metrics that can be collected to evaluate our success over time

@patriciarussellCAS we can close this I think, #47 is a more fleshed out version of this.


Latest version cannot be downloaded by Cocoapods

The latest version of Cuckoo on Cocoapods is v0.7.0. But this does not include the generator update which added --no-class-mocks option. Can you suggest how do I get the new generator?

Hi, you can specify Cuckoo repository and commit/branch you want to use. We are now converting our internal project that use Cuckoo to Swift 3 and after we're done and sure Cuckoo for Swift 3 is ready, we'll be releasing it to CocoaPods. This should happen in the next week.
In the meantime, you can do: pod 'Cuckoo', :git => 'https://github.com/SwiftKit/Cuckoo.git', :branch => 'master' (depending on what you need)

Thanks for your input. But I'm in a bit of a different state -- My project is in Swift 2.2 but I need the generator option (--no-class-mocks). Is this possible?

I see. You can use branch swift-2.2 then. The --no-class-mocking should be implemented there.

Thank you so much! This is a very promising tool.

Harshad

On 19-Oct-2016, at 5:47 PM, Tadeas Kriz<EMAIL_ADDRESS>wrote:
I see. You can use branch swift-2.2 then. The --no-class-mocking should be implemented there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Create one account for each orderer

Every orderer would sign with a unique submit key, and one operator key.

Having a distinct operator per-orderer means that each orderer maintains their own hbar cryptoaccount used to pay for transactions to the topic.
There may be a desire later on for either the submitKey and/or adminKey to be keylist/threshold keys, but having distinct operator/payer key is a good first step.

No code change is needed, verified with the first-network sample configured to have distinct operator for different orderer instance.


References view should reveal items on focus

Hi,
References view is not respecting enablePreview setting.
It makes References view inconsistent with other views like e.g. Search.
In e.g. Search view when we are navigating with keyboard (up, down arrows) then preview editor pops up.
Below video shows the problem.
screen-recording(3).webm

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Investigate API usage irregularities

See this slack conversation here.
It looks like the sales dashboard is overstating the API usage statistics.

This was determined to be a 30 day rolling window updating less frequently than the ground truth reporting.


Tall flower world interaction recipe wrongly matches botania tall flowers

prank recipe not real
cc @jopejoe1

Fixed this in #74


Fix toplevel constant warning in Azure refresh spec

@djberg96 @blomquisg @bronaghs Please review.
This fixes the following warning that appears in the refresh spec:
/Users/jfrey/dev/manageiq/spec/models/manageiq/providers/azure/cloud_manager/refresher_spec.rb:150:
warning: toplevel constant SecurityGroup referenced by ManageIQ::Providers::Azure::CloudManager::SecurityGroup

@Fryguy - LGTM 👍

👍


lang.NumFmt Numbers greater than or equal to 5 are not rounded up correctly

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.68.3/extended darwin/amd64

Does this issue reproduce with the latest release?
Yes it can be reproduced with the latest release with the following numbers:
{{ lang.NumFmt 2 927.675 }} -> 927.67 (expected 927.68)
{{ lang.NumFmt 2 1927.675 }} -> 1927.67 (expected 1927.68)

{{ lang.NumFmt 2 2927.675 }} -> 2927.68 (as expected)

$ hugo version
Hugo Static Site Generator v0.69.2/extended darwin/amd64

Rounding also does not work correctly for the following numbers:
{{ lang.NumFmt 2 535.325 }} -> 535.33 (as expected)
{{ lang.NumFmt 2 3352.825 }} -> 3352.82 (expected 3352.83)

/cc @moorereason


How to calculate xG for a single shot in a local match?

Hi. How can I calculate xG with your model for a single goal chance in a local match.
There is no data available for our game. Only video of the game is available. I want to watch the video and manually write the data and then, use your model to calculate xG of that.

I think you are opening the issue on a wrong repo. Feel free to reopen if this isn't the case.

Where can I find solution to my problem if this repo is not the right one?
Can you help me?


vlc.MP4Handler doesn't worked to compiling. hxcodec.VideoHandler does worked to compiling

Describe your bug here.
i cant compiling when it's vlc.MP4Handler, and i try to compiling with using MP4Handler, and it doesn't work. and please use it VideoHandler from hxcodec because it works at all, i try compiling with using VideoHandler, and it works for me. please fix it thanks

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Nope

haxelib remove hxCodec
haxelib install hxCodec 2.5.1

but that videohandler works like a same

haxelib remove hxCodec haxelib install hxCodec 2.5.1

i try it but it doesn't works normally

did you even revert it

did you even revert it

no, MP4Handler it doesn't work. but Only VideoHandler it works because this is worked to using


DO NOT SUBMIT: Insert a failure in the Java 8 test.

See
https://github.com/jspecify/jspecify/pull/270#issuecomment-1223179417

OK, good.
I'll close this, and I'll carve off that second commit into another PR.


Generating PDF documents with React js

Topic Suggestion
Generating PDF documents with React js
Proposal Submission
Proposed title of the article
Generating PDF documents with React js
Proposed article introduction
Business applications often need to produce ready to print documents to communicate transaction information. For example, You may send a customer a printable receipt after they buy your products. PDF documents are supported by many platforms and retain their structure when printed, this makes them one of the best way to present business documents.
In this tutorial we'll generate a PDF invoice, on both the client and server using the React PDF library. We'll also discuss the library limitations and how to work around them.
Key takeaways

Understand CSS Flexbox and absolute positioning
React pure components
Separating business logic from view logic

Article quality

This week I did a task that required generation of PDF documents. The guides that I found on the internet were either outdated (didn't work with React 18) or covered basic documents. Due to the limitations of the library, I needed to adapt to a certain mental models in  order to deliver the document the way I envisioned them.

References

https://react-pdf.org
https://css-tricks.com/understanding-flex-grow-flex-shrink-and-flex-basis/

On it. Thank you

Just a heads up @vinmugambi rules regarding article payout changed.


Buildroot

When I tried running make on buildroot after copying its config file, I was getting the following error:
cp: cannot stat 'riscv32_beluga/riscvbin/ar/libc.a'

Wow, I didn't expect someone to actually compile this project anymore (or ever, to be honest) haha.
Few things come to mind:

Have you checked out the submodules successfully? You need to run git submodule update --init --recursive at least once.
Check the Makefile in the repository root, it contains some other potentially cross-dependent build steps
riscv32_beluga should not be in use at all, actually - I think I remember that being there to test something, not to actually be a part of the build chain, so something funky might be going on in general


Use static getDervicedStateFromProps instead of componentWillReceiveProps

Is it possible to replace componentWillReceiveProps because is deprecated to remove React 16+ warning.

cWRP isn't deprecated until React 16.4. As of this moment, 16.3 isn't even out. We're looking at larger refactors around async stuff in React 17 in #890.

cWRP seems to be depreacted.

@BenoitClaveau Which version of react do you use? It shouldn't happen until 16.4.

This started showing up after updating to react-native 0.54.0.

downgrading to react 16.2.0 doesn't remove the warnings

I guess the problem is react-native. AFAIK they published package with react alpha which had this problem. Try to downgrade it.

under "react": "16.2.0",

Downgrading shouldn't be the option, since the latest valid version of react-native does deprecated these functions. Is there a ETA on the change?

getDerivedStateFromProps is easy enough using the compat thing which isn't a breaking change.
A breaking change is gonna be the context replacement which has no compat thing so I guess a new major version will be released when React 17 is released.

@timdorr I would like to make a PR migrating react-redux to new lifecycles using react-lifecycles-compat because I don't want StrictMode to warn me about my Connected components. Does it make sense to you?

Yes, the purpose of 16.3 was to let libraries start migrating before the deprecation warnings.

@gaearon I also think it would make a lot of sense for you to include a link to create-react-context polyfill same as you did for react-lifecycles-compat because otherwise people don't even know that they can transition their libraries to the new context without breaking React 16 users (though breaking all older Reacts).
Specifically I had to google a lot until I accidentally came across that create-react-context polyfill thing. It would make life easier for a lot of component mainterners if you included that link as a sidenote in the announcement blog post.

Updating the title to better reflect what this is about. I'll put up a failing PR in a sec that others can build off of.

OK, phew, got all that done. We're now on React 16, Jest, and react-test-renderer.
Here's a failing test: #918

@Hypnosphi Yeah... create-react-context only polyfills React.createContext you are still limited to the rendering features of the versions of React you are compatible with.

has this fix been released? which version?
I'm using:
    "react": "^16.3.2",
    "react-redux": "^5.0.6",
and still getting the error in strict-mode.
For example:
 componentWillReceiveProps: Please update the following components to use static getDerivedStateFromProps instead: Connect(ProfileHandlerHoc)
componentWillUpdate: Please update the following components to use componentDidUpdate instead: Connect(ProfileHandlerHoc)

i've updated to react-redux 5.0.7 and also getting the same warnings as @packetstracer

still throwing these errors after update
componentWillReceiveProps: Please update the following components to use static getDerivedStateFromProps instead: Connect(DemoSideBarNavContainer), Connect(LanguageSelectorContainer), Connect(ProfileHandlerHoc)

componentWillUpdate: Please update the following components to use componentDidUpdate instead: Connect(DemoSideBarNavContainer), Connect(LanguageSelectorContainer), Connect(ProfileHandlerHoc)

package.json content
    "react-redux": "^5.0.7",
cat node_modules/react-redux/package.json
{
  "name": "react-redux",
  "version": "5.0.7",
  "description": "Official React bindings for Redux",

Any clue on what could be?

So what is the alternative to use until the PR was merged to fix the warnings?

@slim-hmidi are you building a library or an app? If the latter, you shouldn't need StrictMode for now

@Hypnosphi  I'm building an application.

Hey folks, any update on the PR being merged? Thanks!

Why was it closed? It's still an issue. Using strict mode or not is developer's decision. Whichever he/she chooses, there's no reason not to fix depracation warning. At least this issue should not be closed, as it's not yet resolved.
What's the PR, @timdorr?

Anyone coming here with this problem, you can use react-redux@next with npm (resolved to 5.1.0-test.1 as of now).

Sure. Though either #995 or #1000 is going to solve it.

Now that #1000 was merged, please make sure it works with StrictMode as well before releasing! 🙌 cc @timdorr @markerikson

Just tested 6.0.0-beta.2, it did fix the warnings! :) thanks.

Clooooosing out!


TextEditor.get is deprecated.

TextEditor.prototype.displayBuffer has always been private, but now
it is gone. Reading the displayBuffer property now returns a reference
to the containing TextEditor, which now provides some of the API of
the defunct DisplayBuffer class.
TextEditor.get (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:105:14)
<unknown> (.atom/packages/perfectionist/index.js:44:76)

ATOM created the above content (quite conveniently) and I am not sure whether this library is still being maintained, but thought I would raise the issue. Kind regards

This was already fixed more than a month ago: https://github.com/sindresorhus/atom-perfectionist/commit/58c19e87fe3d2e4fdd7e8facf14c7438ffb9e543


Android - Attachment - Unable to scroll PDF files in preview

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

Action Performed:

Launch New Expensify app
Go to any chat
Click on the PDF file
Scroll down the file

Expected Result:
User is able to swipe to scroll down the file.
Actual Result:
User is unable to scroll down the file, unless pinching in while scrolling is performed.
Workaround:
Unknown
Platforms:

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

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
https://expensify.testrail.io/index.php?/tests/view/3497642
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/bc876db3-b721-43c8-bca3-0e979a644b79
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

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

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

given the size of the deploy and the urgency to get this out this is one of two/three remaining blockers. I am not sure if its big enough to hold on this if its just android and user can downlaod the PDF

I want to clarify the reasoning behind not passing the disabled state into react-native Pressable component. When pressable gets disabled={true} we are loosing the possibility to alter the cursor appearance on hovering (so we cannot show disabled cursor). This said I was opting for not passing disabled state directly, but to check the disabled state both, when passing, and also inside, the event handlers.

Will review this today

Bump @abdulrahuman5196!

@hoangzinh  proposal is working on android. But will check further to be double sure and provide review tomorrow.

@hoangzinh 's proposal here https://github.com/Expensify/App/issues/20867#issuecomment-1595795537 looks good.
And their hypothesis about Pressable and PanResponder is correct as well. Since we can't directly disable PressableWithFeedback as per this https://github.com/Expensify/App/issues/20867#issuecomment-1596867538. The current proposal is the best path ahead.
🎀👀🎀
C+ Reviewed
cc: @deetergp

Not overdue waiting on proposal confirmation by @deetergp  here https://github.com/Expensify/App/issues/20867#issuecomment-1604469050

PR review underway

Still in review

I don't think this issue should have penalty. We found an another issue during implementation and was discussing and waiting on @deetergp 's confirmation here - https://github.com/Expensify/App/pull/21714#issuecomment-1632650015

BZ Checklist:

The PR that introduced the bug has been identified. Link to the PR:
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

https://github.com/Expensify/App/pull/20671#issuecomment-1646592304

Determine if we should create a regression test for this bug.
If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

No

@greg-schroeder BZ checklist is complete here https://github.com/Expensify/App/issues/20867#issuecomment-1646592428

Thanks @abdulrahuman5196! Taking care of payment now (not applying penalty/bonus)

Offer sent @hoangzinh

Accepted the offer @greg-schroeder

Payments complete:
@hoangzinh $1000 - C
@abdulrahuman5196 $1000 - C+

@hoangzinh @abdulrahuman5196 thoughts on the above?

@greg-schroeder This is not that the PDF not able to scroll., but the scrolling is not smooth. This issue of scroll being not smooth has been present for long time. I also noticed and mentioned the same during the review. The PDF is scrolling but clashing with horizontal and vertical scroll. We should create a separate issue to improve the scroll experience.

Okay, I thought as much. thanks


Compilation fix for !HAVE_HTTP_CLIENT in 2.4

Hello,
This pr wraps two call to CanSyncWithCrowdin inside an #ifdef HAVE_HTTP_CLIENT.  That functions is not available if the http client is disabled, so it wouldn't compile.  Noticed while I was updating the OpenBSD port, since there the http client is disabled (C++ rest is not yet available as package).
Lightly tested, as I'm not sure how to exactly trigger that codepath, but it shouldn't introduce regressions.
Thanks for developing this, keep up the good work :)

Hi. I ran into the similar problem when updating the package for OpenMandriva Cooker (clang compiler).
edframe.cpp:1780:19: error: use of undeclared identifier 'CanSyncWithCrowdin'
                   !CanSyncWithCrowdin(m_catalog));
                   ^
edframe.cpp:2542:80: error: use of undeclared identifier 'CanSyncWithCrowdin'
              if (lang.IsValid() && plForms != lang.DefaultPluralFormsExpr() && !CanSyncWithCrowdin(m_catalog))

Your pull request solve my problem. I don't see any problems with poedit now. Package works fine.

Thanks for spotting this and submitting the PR! I opted for a different fix because it is less verbose and more robust against future accidental breakages, see 29ff4b4e, but it is functionally equivalent and either is suitable for packaging patches.

I think I'll take the patch from 29ff4b4 until the next release than, thanks :)
Btw, sorry for the terrible pr title!

I'm sorry to bother you again, but the latest release with 29ff4b4 added on top still does not build:
In file included from welcomescreen.cpp:30:
./crowdin_gui.h:127:32: error: unknown type name 'CatalogPtr'
inline bool CanSyncWithCrowdin(CatalogPtr) { return false; }
                               ^
./crowdin_gui.h:128:46: error: unknown type name 'CatalogPtr'
inline bool ShouldSyncToCrowdinAutomatically(CatalogPtr) { return false; }
                                             ^

This is because catalog.h is included only if HAVE_HTTP_CLIENT.
--- src/crowdin_gui.h.orig
+++ src/crowdin_gui.h
@@ -26,9 +26,10 @@
 #ifndef Poedit_crowdin_gui_h
 #define Poedit_crowdin_gui_h

+#include "catalog.h"
+
 #ifdef HAVE_HTTP_CLIENT

-#include "catalog.h"
 #include "cloud_sync.h"
 #include "customcontrols.h"

Thank you (and sorry for the screw-up)!
Committed your patch in b551f7d572ccaed8a870ba8ad7bc779275770b43 with attribution.

No problem :)
Cheers!


Creating a cheat sheet page

Created a cheat sheet page and got it started with some good ones.
Also cleaned up some broken links in the free-programming-books-en.md

our (new) cheatsheet page is at free-programming-cheatsheets.md. please merge there.


Improve usage of deployType for Loads

The deployTypes are crucial for a proper deployment into the execution environment, like a post deployment action like NEWCOPY.
While DBB has the capability to understand it's type (isSQL, isCICS, isDLI etc.), can we include this information into the deployType?
See
https://github.com/IBM/dbb-zappbuild/blob/02c9a7b88e3bd3d1b66d44d4255512156692483d/languages/Cobol.groovy#L288-L296

This is resolved by #125


Fixing user base url within public dashboard

Without the base_url variable, model endpoints will not add the desired /u/:username in order to make correctly the requests.
Fixes #4903.
@matallo @juanignaciosl

Basically, putting back this
https://github.com/CartoDB/cartodb/commit/7b1b0a34fab0fd15c6208e7b18d2e335d4e41713#diff-881062bd28607952eb5b4458fcebbec8L94

retest this please

LGTM


8 restrict internal only resource visibility

@joadan This contains all 3 of the internal suggestions noted in #217. Additionally, I updated ChartSerializer to be both static and internal. You will also see that any of the internal classes are now under a folder of the same name.

Thank you!


Error restoring the pre-trained resnet models.

Hi,
Great job providing the code and the pre-trained resnet models.
However, I have a problem loading/restoring the torrent models.
with tf.Session() as sess:
    dataSaver = tf.train.import_meta_graph('ResNet-L50.meta')
    dataSaver.restore(sess, 'ResNet-L50')

I get the error:

tensorflow.python.framework.errors.DataLossError: Unable to open table file ResNet-L50.meta: Data loss: not an sstable (bad magic number): perhaps your file is in a different file format and you need to use a different restore operator?

Any suggestion/help is much appreciated.
Best,
Silvia

dataSaver.restore(sess, 'ResNet-L50.ckpt') workd for me

Hi,
Thanks a lot! Indeed, it seems the .ckpt extension is mandatory.
This tutorial got me a bit confused (https://www.tensorflow.org/versions/r0.9/how_tos/meta_graph/index.html).
Best,
Silvia

Hi, Silvia
Can you provide a complete file of restoring the model?


fix(getting-started): remove the update username feature

There were the following issues with your Pull Request

Commit: c0b6015f090dccee731672dc0eab7f5629ccea66
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com


Too many Auburn colors

Describe the bug
A clear and concise description of what the bug is.
The shear amount of Auburn colors in this application is absolutely astounding. This amount of matching orange and blues could cause irreparable harm to people's eyes that are unfamiliar with dealing with unfathomable disgusting  images. If this were to continue, we could potentially face extremely expensive hospital charges, lawsuits, and psychological harm. This should be taken out immediately!!!
To Reproduce
Steps to reproduce the behavior:

Go to any screen
Click on anything
Scroll down to anything
See terrible colors everywhere

Expected behavior
A clear and concise description of what you expected to happen.
My eyes shouldn't bleed by looking at the screen.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: anything

Additional context
Add any other context about the problem here.

scientific proof

No.


Running "flutter pub get" problem

To Reproduce (i.e. user steps, not code)
Steps to reproduce the behavior:
Go to ' terminal make flutter upgrade be sure the Version is 2.10.4  on channel stable'
Click on 'run flutter pub get'
See error
Error messages
What went wrong:
Because every version of flutter_test from sdk depends on collection 1.15.0 and xml >=5.4.0 depends on collection ^1.16.0, flutter_test from sdk is incompatible with xml >=5.4.0.
And because flutter_native_splash >=1.3.0 depends on xml ^5.3.1, if flutter_test from sdk and flutter_native_splash >=1.3.0 then xml >=5.3.1 <5.4.0.
And because xml >=5.3.0 <5.4.0 depends on petitparser ^4.3.0 and json_path 0.4.0 depends on petitparser ^5.0.0, one of flutter_test from sdk or flutter_native_splash >=1.3.0 or json_path 0.4.0 must be false.
And because no versions of json_path match >0.4.0 <0.5.0 and flutter_project depends on json_path ^0.4.0, flutter_test from sdk is incompatible with flutter_native_splash >=1.3.0.
So, because flutter_project depends on both flutter_native_splash ^2.0.4 and flutter_test from sdk, version solving failed.
pub get failed (1; So, because flutter_project depends on both flutter_native_splash ^2.0.4 and flutter_test from sdk, version solving failed.)
Expected behavior
i am Expecting my app to run without any problem
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
OS: [e.g. MacOS Big Sur + 11.6.2]
Browser [e.g. chrome + 99.0.4844.82]
Smartphone (please complete the following information):
Device: [e.g. Pixel 3 a]
OS: [e.g. Android Tiramisu]
Flutter SDK version
environment:
sdk: ">=2.16.1 <3.0.0"
Flutter 2.10.3 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 7e9793dee1 (3 weeks ago) • 2022-03-02 11:23:12 -0600
Engine • revision bd539267b4
Tools • Dart 2.16.1 • DevTools 2.9.2
insert output of "flutter doctor -v" here
[✓] Flutter (Channel stable, 2.10.4, on macOS 11.6.2 20G314 darwin-x64, locale en-TR)
• Flutter version 2.10.4 at /Users/imac/flutterDevelopment/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision c860cba910 (7 days ago), 2022-03-25 00:23:12 -0500
• Engine revision 57d3bac3dd
• Dart version 2.16.2
• DevTools version 2.9.2
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
• Android SDK at /Users/imac/Library/Android/sdk
• Platform android-32, build-tools 32.1.0-rc1
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
[✓] VS Code (version 1.66.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.36.0
[✓] Connected device (4 available)
• SM A125F (mobile)      • R58R21ZSWAR                          • android-arm64  • Android 11 (API 30)
• iPhone 13 Pro (mobile) • 4326275E-EB0C-4C06-A7B2-B30861DE3A38 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
• macOS (desktop)        • macos                                • darwin-x64     • macOS 11.6.2 20G314 darwin-x64
• Chrome (web)           • chrome                               • web-javascript • Google Chrome 100.0.4896.60
! Error: iPhone is not connected. Xcode will continue when iPhone is connected. (code -13)
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!
Additional context
I am trying to use json_path on last version of flutter
Because every version of flutter_test from sdk depends on collection 1.15.0 and xml >=5.4.0 depends on collection ^1.16.0, flutter_test from sdk is incompatible with xml >=5.4.0.
And because flutter_native_splash >=1.3.0 depends on xml ^5.3.1, if flutter_test from sdk and flutter_native_splash >=1.3.0 then xml >=5.3.1 <5.4.0.
And because xml >=5.3.0 <5.4.0 depends on petitparser ^4.3.0 and json_path 0.4.0 depends on petitparser ^5.0.0, one of flutter_test from sdk or flutter_native_splash >=1.3.0 or json_path 0.4.0 must be false.
And because no versions of json_path match >0.4.0 <0.5.0 and flutter_project depends on json_path ^0.4.0, flutter_test from sdk is incompatible with flutter_native_splash >=1.3.0.
So, because flutter_project depends on both flutter_native_splash ^2.0.4 and flutter_test from sdk, version solving failed.
pub get failed (1; So, because flutter_project depends on both flutter_native_splash ^2.0.4 and flutter_test from sdk, version solving failed.)
is there any solution
Note: when deleting one of json_path from pubspec.yaml the app run without any problem

Hi. Seems to be a version resolution issue. Have you tried downgrading to json_path 0.3?

Closing for now. Feel free to reopen.