Could GraphQLRequestEntity make setRequestMethod as public method?

Our team use this framework to compose query of GraphQL API call. I saw setRequestMethod method in GraphQLRequestEntity is access only package and GraphQLTemplate call it and change request method. For our program, we use Feign+Hystrix as http call. That means we only use Nodes to compose query.
For my understanding, it is not enough to change setRequestMethod as public method. All initial process finish in GraphQLRequestEntity constructor method.
could expose set up requestMethod (query or mutate) in builder?

should expose builder method to set up requestMethod (query or mutate)?

Adding this to the builder sounds like an excellent idea!

Could I give pull request or you could make it change and release new version?

I will absolutely take the change in a pull request! Otherwise I can make the change fairly quickly either tomorrow or Saturday

Alright, just released v0.4.0 which should allow you to set the request method in the builder or on the GraphQLRequestEntity instance so you can build the correct query without executing the fetch


Broken for Godot 4.3

I tried importing this project and converting it to 4.3, but it seems to be broken.  Would like to see an update to this project

Hello @dsol-cpu  there is already a branch for Godot 4.x, https://github.com/hgouveia/godot-toon-water-shader-port/tree/godot-4 please give it a try, i didnt test it yet on 4.3, please let me know if worked for you

I gave it a go and got these dependency issues with the inherited model scenes

I gave it a go and got these dependency issues with the inherited model scenes

Hello @dsol-cpu,
I just tried it myself and opened it without any issues on Windows. However, I was able to reproduce the issue using Linux.
I'm not really sure what is causing this issue; it seems to be a bug in Godot 4.3. But I've found a workaround for the meantime.
If you try to import it first using Godot 4.2.2-stable, and once it finishes importing, you can then try to open it again with Godot 4.3, and it works, at least in my tests.

Thank you for your speedy reply!  I'll try your workaround and I'll update if that works for me too.

I tried importing with 4.2.2 and it worked, but the shader and some textures were not applied or didn't exist.  I dragged and dropped the textures that did exist and got this result.

Hi @hgouveia,
I tried importing with 4.2.2 and it worked, but the shader and some textures were not applied or didn't exist. I dragged and dropped the textures that did exist, changed the DEPTH_TEXTURE variable as directed by Godot 4.2.2 to:
uniform sampler2D DEPTH_TEXTURE : hint_depth_texture, filter_linear_mipmap;
and got this result.

Yeap probably you just need to apply the Shader Material and fill the values and texture, this shader use NoiseTexture to generate most of the effect,
but i actually think i've found the issue,  it seems the issue is due the newer FBX import, if you use the old way , by using FBX2gltf, it should work directly with Godot 4.3
so first, open your Godot 4.3, setup the FBX importer, it will show you a window and a link to download the importer, after it setup, download again the sample code form my repo in the same godot-4 branch since i removed some unnecessary folder that might be causing the issue as well, and import it

I added the old fbx importer to Godot 4.3 and added the textures back, but the shader is still white for me.  What am I missing?

I added the old fbx importer to Godot 4.3 and added the textures back, but the shader is still white for me. What am I missing?

everything seems ok, my only guess is that probably your GPU do no support that shader version of Vulkan, do you tried another shaders before and worked? what if you try to convert the project to Compatibility Mode or Mobile? could you also try the Godot 3 version with GLS3 ?

I tried Compatibility and it worked wow.  I'm using an RTX 4060 so I think it's the driver that doesn't support it--I'll try again on Windows, but all in all thanks for the help!

yeap maybe, but is hard to debug, on my test using a Virtual Machine of Ubuntu, worked fine with Forward+, maybe can be a the driver you are using in linux, did you have a chance to test it over on Windows?


Link Styles are not available

Brief bug description
First of all: Thank you for the great richtext editor integration. It is very extendable and customizable.
We tried to define custom link styles as described in the Froala documentation (https://froala.com/wysiwyg-editor/examples/link-styles/) and the button to select those styles is simply not showing up when clicking into a link. Other custom configurations and custom plugins are working as expected.
Repro steps

Add linkStyles to the richtext default configuration.
In the richtext widget insert a link.
Klick on the link in the text and a pop-up menu with "open", "edit", and "unlink" appears.

Expected behavior
Inside of the pop-up menu a "style" selection should be visible where the configured styles can be selected.
Test environment

Browser Chrome, Firefox, Edge (old and new)
Version Kentico.EMS12.MvcComponents.Widget.RichText 1.2.1

Hi @spitzfaust,
in our default configuration we removed the 'linkStyle' button from the linkEditButtons option, therefore, to make the button visible again, please also provide a linkEditButtons option in your custom toolbar configuration and add back the 'linnkStyle' button like so:
{
  ...
  linkEditButtons: ['linkOpen', 'linkStyle', 'linkEdit', 'linkRemove'],
  linkStyles: { /* Your link styles */ },
  ...
}

Thank you this worked like a charm :+1:


Primary clipboard / pasting broken

OS: Kubuntu 18.04.1 LTS
Version of st: latest
When I paste, it doesn't paste anything at all, pasting my primary clipboard works in other applications (and urxvt) but not in st. Now if I select some text in st and try to paste it will past that text perfectly fine, but if I close the current st and open up a new one then it won't paste the previously selected text.

I've made an update (to the man as well) which clarifies some of copy/pasting shortcuts. There was a period where the bindings weren't properly reflected in the manual, so you may've been pressing the wrong keys. Now it's:

alt-c for copy
alt-v or shift-insert for paste
alt-p to input primary selection

I will say that the fact that your pasting is non-persistent after you close a window is a general property of X See the answer to this for example: https://unix.stackexchange.com/questions/450508/how-to-keep-clipboard-content-from-xterm-after-closing-it-like-firefox-or-le
There are probably ways of bypassing this problem and I'm sure you could look it up, but it's not a aspect specific to this program.


[2.0][CPP] C++ inference core dumped with tensorRT enabled

Description
Tracking c++ inference issue in master/cpp-package
https://github.com/apache/incubator-mxnet/issues/19550#issuecomment-841583103
Error Message
root@6da7899c2de8:/work/mxnet/cpp-package/example/inference# ./imagenet_inference --symbol_file "./model/Inception-BN-symbol.json" --params_file "./model/Inception-BN-0126.params" --batch_size 16 --num_inference_batches 500 --benchmark --enableTRT
[01:51:07] ../cpp-package/example/inference/imagenet_inference.cpp:302: Loading the model from ./model/Inception-BN-symbol.json

[01:51:07] ../src/nnvm/legacy_json_util.cc:209: Loading symbol saved by previous version v0.8.0. Attempting to upgrade...
[01:51:07] ../src/nnvm/legacy_json_util.cc:217: Symbol successfully upgraded!
[01:51:07] ../cpp-package/example/inference/imagenet_inference.cpp:317: Loading the model parameters from ./model/Inception-BN-0126.params

[01:51:07] ../src/storage/storage.cc:199: Using Pooled (Naive) StorageManager for CPU
[01:51:09] ../src/storage/storage.cc:199: Using Pooled (Naive) StorageManager for GPU
terminate called after throwing an instance of 'dmlc::Error'
  what():  [01:51:09] ../cpp-package/include/mxnet-cpp/symbol.hpp:268: MXNetError: Error in operator TensorRT0: [01:51:09] ../src/operator/subgraph/tensorrt/tensorrt.cc:61: Check failed: idx_g.input_nodes().size() == in_shapes->size() + params_map.size() (55 vs. 1) :

Aborted (core dumped)

To Reproduce
$ ./imagenet_inference --symbol_file "./model/Inception-BN-symbol.json" --params_file "./model/Inception-BN-0126.params" --batch_size 16 --num_inference_batches 500 --benchmark --enableTRT

@Kh4L FYI

The unit test in question has comments that indicate it was modeled after mxnet/contrib/tensorrt.py, which is long gone and replaced with optimize_for("TensorRT") in mxnet/python/symbol.py. Looks like something should be added in symbol.hpp to introduce optimize_for that would make call to MXOptimizeForBackend.


Parent Selection with HTTP/2 has performance issue

Summary
We found weird performance issue with parant.config. Using h2load, requests with HTTP/1.1 seems ok, but some requests (2 ~ 9%) with HTTP/2 is faild.
Conditions

Origin server is running on <IP_ADDRESS>:8001.
ATS has below configs

HEAD of 7.1.x branch (eeae4118315283dc188249a4f3c7826836a4b4cb)
records.config and ssl_multicert.config has some changes to enable HTTP/2 on 4443
parant.config

dest_domain=. parent="localhost:8001" round_robin=consistent_hash

remap.config

map / http://<IP_ADDRESS>:8001/

100KB content is cached

Tests
HTTP/2
$ h2load -n 10000 -m 100 -c 2 https://<IP_ADDRESS>:4443/100k
starting benchmark...
spawning thread #0: 2 total client(s). 10000 total requests
TLS Protocol: TLSv1.2
Cipher: ECDHE-RSA-AES256-GCM-SHA384
Server Temp Key: ECDH P-256 256 bits
Application protocol: h2
progress: 10% done
progress: 20% done
progress: 30% done
progress: 40% done
progress: 50% done
progress: 60% done
progress: 70% done
progress: 80% done
progress: 90% done
progress: 100% done

finished in 5.87s, 1661.31 req/s, 162.37MB/s
requests: 10000 total, 10000 started, 10000 done, 9744 succeeded, 256 failed, 256 errored, 0 timeout
status codes: 9744 2xx, 0 3xx, 0 4xx, 0 5xx
traffic: 952.34MB (998598258) total, 105.15KB (107676) headers (space savings 96.39%), 951.56MB (997785600) data
                     min         max         mean         sd        +/- sd
time for request:      954us       1.90s    112.14ms    160.98ms    96.13%
time for connect:     5.43ms      8.64ms      7.03ms      2.27ms   100.00%
time to 1st byte:   440.40ms    444.99ms    442.69ms      3.24ms   100.00%
req/s           :     808.84      966.39      887.62      111.40   100.00%

HTTP/1.1
$ h2load --h1 -n 10000 -m 100 -c 2 https://<IP_ADDRESS>:4443/100k
starting benchmark...
spawning thread #0: 2 total client(s). 10000 total requests
TLS Protocol: TLSv1.2
Cipher: ECDHE-RSA-AES256-GCM-SHA384
Server Temp Key: ECDH P-256 256 bits
Application protocol: http/1.1
progress: 10% done
progress: 20% done
progress: 30% done
progress: 40% done
progress: 50% done
progress: 60% done
progress: 70% done
progress: 80% done
progress: 90% done
progress: 100% done

finished in 2.90s, 3453.07 req/s, 338.46MB/s
requests: 10000 total, 10000 started, 10000 done, 10000 succeeded, 0 failed, 0 errored, 0 timeout
status codes: 10000 2xx, 0 3xx, 0 4xx, 0 5xx
traffic: 980.18MB<PHONE_NUMBER>) total, 3.01MB (3160000) headers (space savings 0.00%), 976.56MB<PHONE_NUMBER>) data
                     min         max         mean         sd        +/- sd
time for request:      622us    366.62ms     51.13ms     59.50ms    88.65%
time for connect:     3.47ms      3.52ms      3.50ms        39us   100.00%
time to 1st byte:     4.04ms      4.97ms      4.50ms       660us   100.00%
req/s           :    1726.73     2223.60     1975.17      351.34   100.00%

I couldn't try this with latest master by another bug.

another bug == #3101

When requests faild, TS sends RST frame and I'm seeing below error log.
[Feb  9 15:23:34.824] Server {0xb0016000} DEBUG: <Http2ConnectionState.cc:202 (rcv_headers_frame)> (http2_con) [1] [937] Received HEADERS frame
[Feb  9 15:23:34.824] Server {0xb0016000} DEBUG: <Http2ConnectionState.cc:1565 (send_rst_stream_frame)> (http2_con) [1] [937] Send RST_STREAM frame

[Feb  9 16:02:59.870] Server {0xb0016000} ERROR: HTTP/2 stream error client_ip=<IP_ADDRESS> session_id=1 recv headers creating inbound stream beyond max_concurrent limit

But I'm not sure, how parent.config effects concurrent stream number.

I add debug logs on increment/deceement client_streams_in_count. (https://paste.apache.org/vCwi)
And I counted some logs until first “Send RST_STREAM FRAME” appears.
 752  ++client_streams_in_count
 652  --client_streams_in_count
 657  Delete stream
 753  Received HEADERS frame
   1  Send RST_STREAM frame

This means delete_stream is already called (client will open new stream), but the client_streams_in_count is not decremented. Currently the client_streams_in_count is decremented by release_stream, so it's possible if event processor is too busy.
So we have 2 issues.
a) client_streams_in_count is not decremented correctly when event processor is busy.
b) the settings of parent.config make ATS slow

Still wondering why the parent.config trigger the problem.
@vmamidi @jrushford Could you take a look?

parent.config and dns reverse lookup
When some rules are written in parent.config, it triggers SM_ACTION_DNS_REVERSE_LOOKUP regardless value of proxy.config.http.parent_proxy_routing_enable in records.config.
https://github.com/apache/trafficserver/blob/e4aa18e3337929bb1c9dc4272f369c86a1325c61/proxy/http/HttpTransact.cc#L1701-L1705

HTTP/1.1 vs HTTP/2
As for HTTP/1.1, only one transaction is proceeded on one connection. So only one dns query is scheduled.
As for HTTP/2, some transactions (up to 100) are proceeded on one connection simultaneously. So many dns queries are scheduled in same time. IMO this made event processor busy and triggered the problems. Below is metrics of eventloop against master. You can see huge spike of HTTP/2 with  the parent.config.


Add/Edit page

View with main data, page attributes, page details aka page variants.
Edit page should give access to page structure view.
Parent ticket #4

Closes via #26


Clean up of VS2015 warnings - Part 3

Changes from SageService Launcher applied to MiniClient Launcher
Resolve Compile and Static Analysis warnings
Change to MS safe/secure functions

Looks like some of the indentation got screwed up

And in the future...please limit your pull requests to much smaller changes. For stuff like this, ideally it'd be one file at a time (unless it's just a couple line change). It's too easy for me to start to glaze over when doing a review like this and miss something.

And in the future...please limit your pull requests to much smaller changes. For stuff like this, ideally it'd be one file at a time (unless it's just a couple line change). It's too easy for me to start to glaze over when doing a review like this and miss something.

Yeah, sorry about that, and noted. I started out trying to just do the MiniClientLauncher with the changes that had been done for the ServiceControlLaunch and what I thought was going to be just a couple of lines in some other files,  but then I got trapped in SageTVWin32DLL.cpp :)

Apologies for all the back and forth and thanks once again for all the time you've spent walking me through this.

No problem, thanks for the cleanups. :)


Typed errors

Add a type field to the error response.

Error now have type and status code.
Limitation: no statusCode for errors thrown by GraphQL, like querying data not in schema.


add validation to occ types to protect from invalid csvrs

Nicely done. Software handles bad occ type data nicely now! Post compute:


Multiple items

Hello
There is a possibility to add array items or dynamically (multiple) to snap targets?

Hi @szybki8562, thank you for the comment.
Yes of course you can change options including the snap dynamically.

can u help me how ? im new in js
THX

For example:
https://jsfiddle.net/yjmhdr2a/

thanks for the answer, but it was not what I was talking about
I have random id elements added while the application is running. I asked if it can be added to the current snap or from whole array svg

It seems that the example is what you want.
You can specify snap targets even if those have random IDs.
See line 4.

i cant write this way
draggable.snap = [     **_document.getElementById('box-a'),     document.getElementById('box-b')_**   ];
i dont have this elements at start and i dont know what will the id
elements that I want to attract, adds to the array with time

I merged the example and your code:
https://jsfiddle.net/yjmhdr2a/1/
BTW, the TEMPARRAY and gravity option are not needed.

`ARRAY = [{id: 'box-a'}, {id: 'box-b'}];
var TEMPARRAY = ARRAY.map(function(target) { return document.getElementById(target.id);});

alert(TEMPARRAY);`

in my proj alert is null in jsfiddle OK

Sorry, my English is poor.
What do you mean?

TEMPARRAY  have empty

Did you try this?
https://jsfiddle.net/yjmhdr2a/2/

ARRAY.map propably this not working

something is wrong with my project your fiddle is OK
THX for yr time

The null is different from "empty".
You should clear up that.
If that is "empty", the getElementById couldn't get elements.
If that is null, your code has problem.

i have comma

For example:
var ARRAY = [{id: 'box-a'}, {id: 'box-b'}],
  TEMPARRAY = ARRAY.map(function(target) { return document.getElementById(target.id); });

This code gets elements box-a and box-b.
In other words, that gets nothing if there are not box-a and box-b.

What is "comma"?

,

I know "comma".
What do you mean "i have comma"?

array.map return only comma in alert before map is ok have values

Please don't change comments that were read because we can't have conversation.
So, could you explain the current progress?

I have no value after the array map

In console, try this:
document.getElementById('box-a')

array map needs some library?

What is your environment? browser and version?

vs code  jquery 3.3.1 opera

Which version of Opera?

pure html with  same result
opera new
chrome and firefox developer same result

opera 56.0.3051.36

I see. The Array#map in your environment doesn't need a library because ECMAScript 5 supports that.
So, did you that?
https://github.com/anseki/plain-draggable/issues/11#issuecomment-429697715

where?

Try this in console of browser:
https://github.com/anseki/plain-draggable/issues/11#issuecomment-429697715

var numbers = [4, 9, 16, 25];
alert(numbers.map(Math.sqrt));
this map ok

Yes, of course I know.
So, did you try the code that I indicated?

document.getElementById('box-a') ?

Yes.

i dont know how in console this 'document.getElementById('box-a')' must be a return of one value

`ARRAY.map(function(target) {
alert(target.id); OK
});
ARRAY.map(function(target) {
alert(document.getElementById(target.id)); NO OK
});
`

Sorry, my English is poor.
Do you mean that you don't know how to execute the code in console?

return null

?

i dont have that element i only check array draggable working

Do you mean that this code returned null?
document.getElementById('box-a')

yes i dont have element with box-a id

ok i understand im very sleepy sorry for my bad

i got null or nothing because i wanted getelement

I see.
Your code has problem.
Did you read this?
https://github.com/anseki/plain-draggable/issues/11#issuecomment-429694673

You must specify correct ID to both ID of the element and getElementById.

hehe did not pay attention to getElementById when i delete this then ok

yes my id's have in another array

for (var it in WALLS) {
ARRAY.push([{id: WALLS[it].id}]);
}
but this not working

Try this:
console.log(ARRAY);

in console have value

What value?

my id and some another values in json propably

What ID?

from my loop elements

but TEMPARRAY length = 0

What ID?
Could you tell me that?

"2WITS" this is id. and i want this id assign to draggable

Ok, try this:
document.getElementById('2WITS')

return my svg path

<path d="M750,310 L730,290 L730,440 L750,420 Zi" stroke="none" fill="#666" stroke-width="1" stroke-linecap="butt" stroke-linejoin="miter" stroke-miterlimit="4" fill-rule="nonzero" id="2WITS"></path>

I see.
It seems that the ID or your code is obviously wrong.
At least, when the draggable is constructed, the 2WITS element doesn't exist yet.

You can add snap target after that element was added.
Before the element is added, of course you can't.
This is JavaScript, not PlainDraggable.
See: https://github.com/anseki/plain-draggable/issues/11#issuecomment-429694673

i have elements.
add element -> add to array -> array to plaindraggable
from the beginning
no elements no array

You must make the TEMPARRAY after the elements were added.

yes

but TEMPARRAY is null as image
https://github.com/anseki/plain-draggable/issues/11#issuecomment-429704608

The null that returned by getElementById means that the element doesn't exist.

there are elements for sure

This works fine:
https://jsfiddle.net/yjmhdr2a/2/
Because the elements exist.

For test, insert this line in to your code, before ARRAY.map.
console.log('2WITS: ' + document.getElementById('2WITS'));

GUH23: [object SVGPathElement]

I see, replace this:
ARRAY.map(function(target) { return document.getElementById(target.id); }

with:
ARRAY.map(function(target) { console.log(target.id + ': ' + document.getElementById(target.id)); return document.getElementById(target.id); }

undefined: null

That means the ARRAY is incorrect.

for (var it in WALLS) {          ARRAY.push([{id: WALLS[it].id}]);  }
this is wrong
added id by hand is ok

Insert this before for (var it in WALLS) ...:
console.log(WALLS);

ok got 5 elements

What was shown?

ARRAY.push([{id: WALLS[it].id}]); This is fine?

What was shown by console.log(WALLS);?

Click the left of 0.

Ah, ok.
Replace this:
for (var it in WALLS) { ARRAY.push([{id: WALLS[it].id}]); }

with:
for (var it in WALLS) { console.log(WALLS[it].id + ': ' + document.getElementById(WALLS[it].id)); ARRAY.push([{id: WALLS[it].id}]); }

https://www.w3schools.com/js/tryit.asp?filename=tryjs_array_object
GREAT THANKS FOR HELP

The both for and ARRAY.push are not needed.

Everything working

I'm glad if I could help you. :smile:
So, could you close this issue?

YES Thanks

Could you close this issue if it was solved?


Add Rack::Attack for additional hardening against abusive clients

Consider adding Rack::Attack to harden the BadgeApp against attack, in particular to throttle rates.
Here's an example configuration: https://github.com/kickstarter/rack-attack/wiki/Example-Configuration
Another article:
https://kickstarter.engineering/rack-attack-protection-from-abusive-clients-4c188496293b
We'll have to ensure that the IP addresses being considered are from the client, and not from the Heroku router.

One complication is that I don't believe the cache is shared, so the counts won't be shared either.  That doesn't make it useless, it just means that it won't trigger unless the attacks are really bad :-).

I investigated the caching. Looks like we'll be fine, but it would be wise to insert a comment next to the cashe and configuration in case we change it later.  I have already added some documentation about it.

Here is a useful video about the gem:
https://m.youtube.com/watch?v=m1UwxsZD6sw#

I'm concerned about getting the actual IP address on Heroku. It is not clear that rack will give us the correct value. Here are some sources that suggest Solutions..
https://github.com/quipper/real_ip/blob/master/README.md
https://stackoverflow.com/questions/26276775/ruby-app-thinks-clients-ip-address-is-always-the-same
https://devcenter.heroku.com/articles/http-routing

I'm still trying to determine how to securely get client IP address.
The Rack documentation: https://www.rubydoc.info/github/rack/rack/Rack/Request/Helpers#
Shows this as the code that implements "req.ip" in file 'lib/rack/request.rb', line 256:
def ip
  remote_addrs = split_ip_addresses(get_header('REMOTE_ADDR'))
  remote_addrs = reject_trusted_ip_addresses(remote_addrs)

  return remote_addrs.first if remote_addrs.any?

  forwarded_ips = split_ip_addresses(get_header('HTTP_X_FORWARDED_FOR'))

  return reject_trusted_ip_addresses(forwarded_ips).last || get_header("REMOTE_ADDR")
end

So it prefers "REMOTE_ADDR", but I don't know that I can trust REMOTE_ADDR.  I may have to just write my own method that returns the last entry fo HTTP_X_FORWARDED_FOR when available.

I have pushed to the "master" tier and run some tests.
GOOD NEWS: It appears that figuring out the externally-visible client IP address is easy, just take the next-to-last value of "HTTP_X_FORWARDED_FOR" (which is from the HTTP header X-Forwarded-For).  Because everything goes through Fastly, and then through the Heroku router, the Heroku router's "last value" is always the one from Fastly, and Fastly already adds to the end whatever it got.  A client can always set X-Forwarded-For and try to spoof something; those entries are not lost, but they are always earlier in the list (so we can easily ignore them).
Below are the test results that I believe confirm this.
====
I first used https://www.myexternalip.com/ to confirm my externally-visible IP address   In the test, it reported as <IP_ADDRESS>.
Ran some commands:

curl https://master.bestpractices.coreinfrastructure.org/debug

DEBUG /debug req.ip=<<IP_ADDRESS>>, HTTP_X_FORWARDED_FOR=<<IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>>, HTTP_X_REAL_IP=<>, REMOTE_ADDR=<<IP_ADDRESS>>

The system seems to remember failed connections, so I have to do a "heroku restart --app master-bestpractices" after each test.  Okay, did that.  I then did:

curl -H 'X-Forwarded-For: <IP_ADDRESS>' https://master.bestpractices.coreinfrastructure.org/debug

DEBUG /debug req.ip=<<IP_ADDRESS>>, HTTP_X_FORWARDED_FOR=<<IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>>, HTTP_X_REAL_IP=<>, REMOTE_ADDR=<<IP_ADDRESS>>

A quick lookup on https://mxtoolbox.com/arin.aspx reports that IP address <IP_ADDRESS> is owned by Fastly.
There's really no need to pay attend to the rest.

We've done it!


please add - interior gpt

InteriorGPT: Transform Your Space with AI 🤖. Discover innovative interior and exterior design concepts, detailed floor plans, and personalized reimaginations of your environment, all powered by Generative AI tech. Elevate your space with a touch of generative design 🏠🎨📸
link - https://chat.openai.com/g/g-MMsjdW9vb-interiorgpt

Added


Cannot get files of a torrent without unnecessary priorities and wanted fields

Description
Currently, it is impossible to get files of a torrent (with the .get_files() interface) unless you fetch the priorities and wanted fields.
This is due to trying to directly reference these fields here: https://github.com/trim21/transmission-rpc/blob/57c064f6b740b67be1ed7c5afad5ce18476616f4/transmission_rpc/torrent.py#L436-L437
It currently returns a key error if you try to use this:
    priorities = self.fields["priorities"]
                 ~~~~~~~~~~~^^^^^^^^^^^^^^
KeyError: 'priorities'

This doesn't really seem necessary and forces fetching these fields onto users for use-cases that may not need them.
I would propose updating this function or providing an alternative interface to get the files of a Torrent class
(as a side note, why are getting files a function interface while everything else on the class is a property? It seems a bit weird/inconsistent)

(As a note to others who find this issue before it's fixed)
A hacky workaround to this without actually fetching the additional fields is to just manually set the internal field state of the torrent yourself before calling .get_files() - i.e.:
torrent.fields["priorities"] = [0] * len(torrent.fields["files"])
torrent.fields["wanted"] = [True] * len(torrent.fields["files"])
# get_files() should now work
torrent.get_files()

ref https://github.com/trim21/transmission-rpc/pull/446

A bit weird to close this issue because it's definitely not fixed. Unless you're saying you won't fix?
Should probably separate #446 out since there's 2 different issues:

Trying to call .get_files() if the files field wasn't fetched should return an error
Trying to call .get_files() if the priorities or wanted fields aren't fetched should not fail

2 will be fixed in PR #446

Shouldn't this issue remain open until at least it's merged then?

Yes, both your suggestions on splitting PR and keep issue open sounds reasonable to me. But I'm don't want to spend time on that PR just split it into 2 PR and resolve conflicts when I plan to merge it, so I'll just only re-open this issue.

Yeah sounds good. It's just helpful to track this as an open issue. I appreciate it!

for tracking, I want to avoid unnecessary breaking change so I will merge PR and make new release when new transmission version is released.

you can use pip to install zip of PR if you need it now.


[Integration] implement DataStore for MongoDB

load whole documents from Mongo.. what about store?
use some particular collection or all docs in DB? it'll be interesting to get joins for Mongo docs :).
will we need BSON DataFormat for this too?

implemented, tested. done?

Ok, good enough for the first release


Center map on South America

We discussed this in the group meeting. The map is centered on South America on initial opening. I didn't change the bounds, so no functionality is removed. Would it be a good idea to reduce the bounds to only South America (or a subset of it) if that's the only area we're focusing on?

Hmm, on looking at it more, it seems like this resets the view every time Leaflet re-renders = when the map state changes = when a new pin is selected. I'll look into an alternate solution that only changes the view on initial load. The problem is that useMap() only works in components that are children of .

Fixed using useEffect(). I'll merge that with the larger map pin pull request I'm cooking, though.


openssh: Fix cross-compile regression from c99c499

Motivation for this change
Recent refactoring from c99c499 into a "common.nix" dropped some unusual details that allowed cross-compilation. See https://github.com/NixOS/nixpkgs/pull/107606 for more details on the original patch.
Things done
This patch reinstates those, and adds comments for posterity linking to cross compilation discussion.

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

/rebase staging-next

https://github.com/openssh/openssh-portable/commit/32dd2daa56c294e40ff7efea482c9eac536d8cbb the underlying issue is now fixed, so we can depend on kerberos normally next release.


[pybind11] bumped to 2.4.3

[pybind11] bumped to 2.4.3

What does your PR fix? Fixes issue #

Which triplets are supported/not supported? Have you updated the CI baseline?
I have tested on a windows machine with a custom triplet for windows static release
Haven't touched CI

Does your PR follow the maintainer guide? Yes

Obsolete and duplicate


Feature/12 seed data

Summary
Added seeds for interview questions, changed documentation to mention how to update database
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Refactoring (non-breaking change to code and structure)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Please explore the option of how to exclude seeds from migration. It would be useful for Prod.

I would expect seed to be also executed manually if needed. There will be a need for adding more data to the seed so having it in migration would complicate things. 🤔

Ignore sonar cloud duplicity warning, its not relevant in this case.

I will merge this one due to permission to merge if the check fails.


BCF-2685: move types from core to support Solana plugin command extraction

https://smartcontract-it.atlassian.net/browse/BCF-2685
Supports:

https://github.com/smartcontractkit/chainlink-solana/pull/550
https://github.com/smartcontractkit/chainlink/pull/10973

Merged to unblock the growing stack of PRs, but I'm happy to circle back on anything.


lambdatest-upload 3.0.0

What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[ ] I will not move an already shared step version's tag to another commit
[ ] I read the Step Development Guideline
[ ] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[ ] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[ ] I read and accept the Abandoned Step policy

@tothszabi We have made few changes to the older version. Please review this PR. Thankyou.

Can you please review this once?

Looks good @rohithmone-lt, I'm merging it now. Sorry for the delay.

Thankyou @ofalvai


amazon-connect-streams is not compatible with javascript strict mode

TypeError: Cannot set property 2011-06-15 of # which has only a getter
We received this error when importing stream library into stencilJS component for building custom CCP. This issue is because the Library is not compatible with JavaScript's STRICT MODE. StencilJS  requires strict mode when importing external library: https://stenciljs.com/docs/module-bundling#strict-mode
The root cause in the source code is in aws-client.js.
In aws-client.js, on line 2869, '2011-06-15' is defined as a property of apiLoader.services['sts'] with a getter but no setter. On line 21104, there is an attempt to set the property. As it stands now, defining the property on line 2869 is pointless as it's value is guaranteed to be overwritten,  commenting out the latter can resolve this issue. Please provide fix for this issue.

Streams 2.3.0 has just been released which should fix this issue.


Unable to use slate node 8.8.1

Problem
Slate command line not working on node 8.9.1 npm 5.4.2
Replication steps

Install slate globally npm install -g @shopify/slate
run slate

More Information
Returns
$ slate
/Users/juanpablosolano/.nvm/versions/node/v8.8.1/bin/slate: line 1: use strict: command not found
/Users/juanpablosolano/.nvm/versions/node/v8.8.1/bin/slate: line 3: syntax error near unexpected token `('
/Users/juanpablosolano/.nvm/versions/node/v8.8.1/bin/slate: line 3: `var _fs = require('fs');'

Runing npm install -g @shopify/slate a second time has fixed the issue

When did you do this yesterday? You may have coincidently npm install'd when I mistakingly had @shopify/slate pointing to the wrong version of the repo 😅


Join queries

Description
Perform a join query between “individual” and “biosample” or any other two or more entity
Proposed solution
New API endpoint to support queries involving multiple entities.
Definition of Done
Availability of new API endpoint

Hi @costero-e
Please let me know if join queries are possible.
Cheers
Mustafa

Hi @mustafahsyed, by join queries do you mean aggregating data from two endpoints and return it in a single record or applying filters to multiple endpoints at once? We are planning to develop multiple endpoints filters, e.g. show me variants for a specific position for individuals that are male (specifications do accept this), but we are not planning to return data aggregated from different endpoints.

Hi @costero-e
Ideally both filtering and aggregating data but we can begin by just having option to filter data using multiple endpoints. Please let me know when such join multiple endpoints filters will be available?
Excellent work!
Thanks

Hi @mustafahsyed,
today I have finished implementing "cross queries", which can be used to apply a filter to a collection not belonging to the final collection you want to get the documents from.
Here is one example you can try:
curl \
  -H 'Content-Type: application/json' \
  -X POST \
  -d '{
    "meta": {
        "apiVersion": "2.0"
    },
    "query":{ "requestParameters": {
        },
        "filters": [
{"id":"NCIT:C20197", "scope":"individual"} ],
        "includeResultsetResponses": "HIT",
        "pagination": {
            "skip": 0,
            "limit": 0
        },
        "testMode": false,
        "requestedGranularity": "count"
    }
}' \
  http://localhost:5050/api/g_variants

Aggregating data is not in our roadmap though right now, as it is not part of the spec.
Let me know if this new implementation fits your needs or is what you expected.
Best,
Oriol

@costero-e
Join query now works great! Appreciate adding this very useful update.
One more related question:
If I like to pass mutiple filter IDs for the same data element, eg., query diseases.diseaseCode NCIT:C3270 or NCIT:C16576. How should my payload should look like?
Cheers
Mustafa

@costero-e
Above payload pulls any record with both terms, NCIT:C3270 and NCIT:C16576. What I am looking for is all records with diseaseCode either NCIT:C3270 or NCIT:C16576.
Payload you suggested above performs "AND" operation, I am looking for "OR" operation. Please let me know how I can have an OR query.

Hi @mustafahsyed , this is still not in the standards but I know there is a plan for having it included in the short future.
Once it is accepted, I will develop "OR" operations for RI as well and I will let you know.
Best,
Oriol


”Bereni”, ”Bâra”, ”Cându”, ”Drojdii” are listed in the drop-down menu

Description:
When user writes ”Bereni”, ”Bâra”, ”Cându”, ”Drojdii” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Bereni"

Expected results
The location appears in the drop-down menu as "Sat Bereni, MUREȘ (Bereni)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bâra"

Expected results
The location appears in the drop-down menu as "Sat Bâra, MUREȘ (Bereni)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Cându"

Expected results
The location appears in the drop-down menu as "Sat Cându, MUREȘ (Bereni)".

Step 7
Write in the search bar "Drojdii"

Expected results
The location appears in the drop-down menu as "Sat Drojdii, MUREȘ (Bereni)".


Events when expand/collapse finished

In addition to onExpand/onCollapse events that fire when expand/collapse animation starts, it would be great to get onExpandFinished/onCollapseFinished events that fire when the corresponding animation finishes.

#19


[stable/rabbitmq] [stable/rabbitmq-ha] Merge charts discussion

Currently there are 2 charts for rabbitmq. Until #4591, only rabbitmq-ha supported multiple replicas. Now both charts are very similar. At this point work on one chart is likely either already done on the other or would need to be duplicated there. I'd like to get together and merge the charts.
Toward that effort, what features from each do we need to make sure to support in the 1 unified chart going forward:

Feature
rabbitmq
rabbitmq-ha

Ingress for management  port
☑
#4749

Amqps
No
☑

Persistence
partial, but not using statefulset templates?
full with #4610

Image
bitnami
official

Affinity
customizable
soft or hard on hostname

Customizable probes
☑
#4749

Additional plugin ports
No
For supported plugins

@bartimar @laghoule @etiennetremel @vtuson @novcn @Antiarchitect
As recent committers on one chart or the other, I'm interested to hear if there are any features I'm missing that are different between the two charts.
Can we all agree we'd like to only have 1 rabbitmq chart and then work from there on how we get to that?

Moving to a single Chart with all the features combined is much preferred, imho.
Feel free to assign me to PRs related to this change and I'll review them.

hi @coreypobrien - I am not sure sure that rabbitmq chart was ever designed to only support one node (that seems kind of mad). I agree that they are both similar as they are both based on the upstream example: https://github.com/rabbitmq/rabbitmq-peer-discovery-k8s/tree/master/examples/k8s_statefulsets
To merge into a single chart, instead of duplicating more features (https://github.com/kubernetes/charts/pull/4749 ) wouldn't make more sense if you made PRs to rabbitmq with what you think is missing from rabbitmq-ha.
Happy to jump on slack or hangout if that will help.

@vtuson I think we're agreeing in principle, I just saw rabbitmq-ha as the more complete so was PRing in the other direction. If #4749 merges to rabbitmq-ha, is there something missing from rabbitmq-ha that would prevent you from using it?

/cc @etiennetremel @prydonius @tompizmor @carrodher

Good call, I'm also in favor of having both merged together.

@coreypobrien actually, I don't see a problem with having multiple charts. They might look similar now (specially with your new PRs) but they might diverge again into different topologies. Having choices is always a good thing.
I understand that we might be harder for people to choose where to contribute but that is part of the beauty of open source communities :)

@vtuson While having multiple charts in the open source community is a good thing, I don't agree that having multiple in this project is anything other than confusing. Can you elaborate more on the multiple topologies? Is there a specific pair of topologies that you think are useful to implement that couldn't be implemented as two options in the same chart?

Can all of these changes please be made under unstable label/directory? Some of us are trying to use these charts in production and keeping up is becoming tiring.

Can all of these changes please be made under unstable label/directory? Some of us are trying to use these charts in production and keeping up is becoming tiring.

Unfortunately there isn't a great way to make backwards-incompatible changes obvious to users, but we do try to ensure semver is followed for the chart version. #4591 should have bumped the major version, sorry that wasn't caught. We're definitely open to other ideas around how we can make changes more visible to users.

@prydonius Possibly something like https://github.com/semantic-release/semantic-release would help.

After discussing this with @etiennetremel, we decided it makes sense to merge efforts on rabbitmq and rabbitmq-ha together and have a single chart. We'll work on getting features combined and eventually deprecating rabbitmq-ha.

@coreypobrien I think that table is a good start, @tompizmor also came up with a list of things that would need to be done. We were thinking it makes sense for the combined chart to be called rabbitmq instead of rabbitmq-ha, is there any reason you'd pick rabbitmq-ha?

@tompizmor would you mind sharing your list here?
@prydonius Since I was working off the idea the rabbitmq-ha needed no changes, I was leaning toward the least effort to get to a single supported chart.
Is there any precedent for removing a chart from this repo or will we always have 2 charts now that there are 2?

Will there be a migration path either way?

Sorry guys, I have been quite busy these day and it will continue for the next few weeks, if anyone want to pick this up to speed up the process, be my guest.
As I mentioned to @prydonius, apart of not using the official RabbitMQ image, I'm more than willing to merge both charts.

@etiennetremel When you say "apart of not using the official RabbitMQ image" do you mean "I think the final chart should use the bitnami image by default" or "I think the final chart should support the bitnami chart as an alternative"?

I would prefer using official images, for any chart, not only rabbitmq. But I understand that Bitnami bring some automation to keep the chart up to date which is also interesting, so there is a trade off to be done.

Hey everyone! Are there any updates on the effort to merge rabbitmq and rabbitmq-ha charts? I'm in the process of adding a rabbitmq subchart in a chart I'm developing. Any suggestions as to what is the safest, most future-proof option at this point?

just ran into this.. rabbitmq-ha is what i tried first, and it worked lovely.. but i wanted to experiment with rabbitmq as well.. then i noticed that there are subtle difference between the two values.yaml and this has lead me to waste many an hours trying to get the same arguments that work with rabbitmq-ha to work with rabbitmq..
can you please either re-open this to keep this work going, or just re-sync rabbitmq from rabbitmq-ha (and preferably just get rid of one)..
there's no clear explaination why there are two charts, when rabbitmq claims to have HA features (yet hard-code the cluster-domain :-/ )

This kind of issues will be solved once the distributed repositories are implemented definitively, so I think it's a matter of time. In the future, you can use your favorite chart from the repository you choose, and in case there are duplications inside the same repository, the owner of the repository should provide a solution/answer.

Hi All, since it's already 2020. Is there any progress or decisions about future of  RabbitMQ vs RabbitMQ-ha? Is there any sort of recommendations where should typical user go?  Using RabbitMQ or -HA chart? I see some advantages in both charts but they are still quite different. Thanks!

Hi, since Helm 3 release this stable repository as we know it right now will disappear sooner than later: https://github.com/helm/charts#deprecation-timeline
In the near future, there will be different charts for the same application hosted in different registries and maintained for different people (all of them listed in the Helm hub) users should use the one that best meets their needs, but the figure of stable as a source of truth will disappear. That is what I understand for the community comments and discussion, not sure if some maintainers can add more information about this topic.


add support for outlook web application

this should also be the same on windows but again have not tested it. this is also another web application that could eventually go into a web folder.

Few minor notes, similar to proton:

Until someone vets these on Windows or Mac, I'd vote to add
os: linux
i do wonder if the win.title is specific enough? Should it be some regex of mail.outlook.com or whatever?
if you've time to try tags on v1276, i'd say use tag: browser instead of listing the apps.

(This is beyond the scope of this, but i think we may be able to usefully consolidate the grammars between messengers/rich text/email clients some down the road. It's been on my list for a bit, but I don't have access to my work stuff currently)

i've confirmed tags work well, i also added the linux filter. similar to the protonmail problem the window title doesn't come from the actual url, so what i used is about as granular we can get at the moment

Sounds good, thanks for this!


New package: SudachiEmu.Sudachi version 1.0.2

Pull request has been created with komac v2.3.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\SudachiEmu.Sudachi__DefaultSource\QtWebEngineProcess.exe returned exit code: -1073741515
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\SudachiEmu.Sudachi__DefaultSource\sudachi-cmd.exe returned exit code: -1073741515
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\SudachiEmu.Sudachi__DefaultSource\sudachi.exe returned exit code: -1073741515
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\sudachi.exe returned exit code: -1073741515

Automated error analysis suggests -1073741515 may mean STATUS_DLL_NOT_FOUND - possibly due to a dependency on Microsoft.VCRedist.2015+.x64 (or .x86).
(Automated response - build 886.)

HI @xiSage,
Does this package have any dependencies? (The message mentions Microsoft.VCRedist.2015+.x64 (or .(x86) because they are the vast majority of dependencies.)

HI @xiSage,
Does this package have any dependencies? (The message mentions Microsoft.VCRedist.2015+.x64 (or .(x86) because they are the vast majority of dependencies.)

HI @xiSage,
Does this package have any dependencies? (The message mentions Microsoft.VCRedist.2015+.x64 (or .(x86) because they are the vast majority of dependencies.)

I think this package should have only one dependency on Microsoft Visual C++ Redistributable.

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\SudachiEmu.Sudachi__DefaultSource\QtWebEngineProcess.exe returned exit code: -1073741515
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\SudachiEmu.Sudachi__DefaultSource\sudachi-cmd.exe returned exit code: -1073741515
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\SudachiEmu.Sudachi__DefaultSource\sudachi.exe returned exit code: -1073741515
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\sudachi.exe returned exit code: -1073741515

Automated error analysis suggests -1073741515 may mean STATUS_DLL_NOT_FOUND - possibly due to a dependency on Microsoft.VCRedist.2015+.x64 (or .x86).
(Automated response - build 886.)

I think this package should have only one dependency on Microsoft Visual C++ Redistributable.

I agree - the issue is that this manifest currently has zero dependencies, when that one is needed.

Unfortunately, the symlink doesn't work.

Verify from full link:


Updated node-hid to 0.4.0

@andrew Looks good here (OSX 10.10.3, node v0.12.0).
No errors now, only 2 warnings:
child_process: customFds option is deprecated, use stdio instead.
  CC(target) Release/obj.target/hidapi/hidapi/mac/hid.o
../hidapi/mac/hid.c:255:20: warning: comparison of integers of different signs: 'CFIndex' (aka 'long') and 'size_t' (aka 'unsigned long') [-Wsign-compare]
                if (chars_copied == len)
                    ~~~~~~~~~~~~ ^  ~~~
../hidapi/mac/hid.c:295:20: warning: comparison of integers of different signs: 'CFIndex' (aka 'long') and 'size_t' (aka 'unsigned long') [-Wsign-compare]
                if (used_buf_len == len)
                    ~~~~~~~~~~~~ ^  ~~~
2 warnings generated.

These warnings were present before 0.4.0 though.

:ship: https://github.com/andrew/node-xbox-controller/releases/tag/v0.6.0

Thanks @andrew!


Updated to support daisyui 4.0.0+ version.

Hey @LeekJay!
Are you still using this package? Running the /playground, I noticed an infinite loop of requests for hmr:

Related to unocss/issues/2039. Seems that cssnano is the culprit but I'm not sure where it's coming from.

Hey @LeekJay!
Are you still using this package? Running the /playground, I noticed an infinite loop of requests for hmr:

Related to unocss/issues/2039. Seems that cssnano is the culprit but I'm not sure where it's being invoked.
Update: On closer inspection, removing the daisy.handler solves the hmr issue, so I'll investigate further.

Already fixed.unocss-preset-component/pr/1

Hey @LeekJay!
Are you still using this package? Running the /playground, I noticed an infinite loop of requests for hmr:

Related to unocss/issues/2039. Seems that cssnano is the culprit but I'm not sure where it's being invoked.
Update: On closer inspection, removing the daisy.handler solves the hmr issue, so I'll investigate further.

Fixed.unocss-preset-component/pr/1
This issue has been identified:
Array.reverse() is an in-place operation, changing the original array.
If this array is in the module's global scope, its state (including the global scope's array state) will be retained when the module is read and cached by Vite.
Then when using Array.reverse(), since this is an in-place operation, the array state will be changed. This will be detected by Vite, which will consider the module to have changed, thus triggering a hot update.
When a hot update is triggered, Vite will not re-read and parse the module file, but will use the cached module state, which has been changed by Array.reverse().
Therefore, after each hot update Array.reverse() will be run again, and the array state changes again, which triggers a hot update, forming an infinite loop.
To solve this problem, avoid using Array.reverse() directly in the global scope. We can copy an array before each use and then use Array.reverse() so that the original array will not be changed. For example, use Array.from(array).reverse() or [...array].reverse().

Wow, really great find, thanks a lot!
P.S. there seems to be another call to parents.reverse() on line 115 but it doesn't seem to trigger the loop..

Hey @LeekJay!
Are you still using this package? Running the /playground, I noticed an infinite loop of requests for hmr:

Related to unocss/issues/2039. Seems that cssnano is the culprit but I'm not sure where it's being invoked.
Update: On closer inspection, removing the daisy.handler solves the hmr issue, so I'll investigate further.

Fixed.unocss-preset-component/pr/1
This issue has been identified:
Array.reverse() is an in-place operation, changing the original array.
If this array is in the module's global scope, its state (including the global scope's array state) will be retained when the module is read and cached by Vite.
Then when using Array.reverse(), since this is an in-place operation, the array state will be changed. This will be detected by Vite, which will consider the module to have changed, thus triggering a hot update.
When a hot update is triggered, Vite will not re-read and parse the module file, but will use the cached module state, which has been changed by Array.reverse().
Therefore, after each hot update Array.reverse() will be run again, and the array state changes again, which triggers a hot update, forming an infinite loop.
To fix this problem, we can avoid using Array.reverse() directly in the global scope. Or we can use Array.from(array).reverse() or [...array].reverse(), copy the array before each use, and then use Array.reverse(). This way the original array will not be changed.

Wow, really great find, thanks a lot!
P.S. there seems to be another call to parents.reverse() on line 115 but it doesn't seem to trigger the loop..

Wow, really great find, thanks a lot!
P.S. there seems to be another call to parents.reverse() on line 115 but it doesn't seem to trigger the loop..

Yes, maybe unocss already handles that in rules, but preflights doesn't.


track with site-parameter using the magento "website" value

if this is enabled (should be possible to disable it) all requests should contain a "site" parameter

not relevant anymore


Thread exception when printing Object description with relationships

Goals
Wish to log a resolved realm object description on a separate thread.
Expected Results
descriptionWithMaxDepth: to appropriately return description.
Actual Results
IncorrectThreadException raised.
Code Sample
Create Object subclass which has at least one relationship.
class Group: Object {

    dynamic var id: String = ""
    dynamic var name: String = ""

    dynamic var site: Site?

Now create an instance of the object & give it a relationship object, in this case an instance of Site.
Now create a ThreadSafeReference.
On a new thread create an instance of Realm, resolve the thread reference & then try logging:
let realm = try! Realm(configuration: Realm.Configuration(inMemoryIdentifier: "MyInMemoryRealm"))
realmObject = realm.resolve(resourceThreadRef)
print("realmObject: \(realmObject)")

Version of Realm and Tooling
ProductName:	Mac OS X
ProductVersion:	10.12.3
BuildVersion:	16D32

/Applications/Xcode.app/Contents/Developer
Xcode 8.2.1
Build version 8C1002

/usr/local/bin/pod
1.2.0
(not in use here)

/bin/bash
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin16)

/usr/local/bin/carthage
0.15.2
(not in use here)

/usr/bin/git
git version 2.10.1 (Apple Git-78)

Thanks for reporting this @markst, I'm looking into it.

@jpsim I think this is largely to do with Realm not complaining when setting a relationship from a different realm. i.e:
let group = Group(value: ["id":NSUUID().uuidString])
group.site = self.site

Resolution for above exception is:
group.site = realm.create(Site.self, value: self.site, update: true)

This also appears to happen if making an unmanaged copy of a realm object which has relationships:
let unmanagedUser = User(value: self.user!)
print("unmanagedUser: \(unmanagedUser)")

Haven't been able to delve too deep, but could be due to RLMIsObjectValidForProperty() not correctly identifying whether it's a valid object for property.
https://github.com/realm/realm-cocoa/blob/master/Realm/RLMUtil.mm#L151
Otherwise the correct duplication of array would occur here:
https://github.com/realm/realm-cocoa/blob/master/Realm/RLMObjectBase.mm#L95

This is bizarre & seems to happen even if creating unmanaged copies of the relationships also:
// Create unmanaged if necessary
var fitting: Fitting = self.fitting.realm == nil ? self.fitting : Fitting(value: self.fitting)
fitting = self.fittingInformationViewController!.fittingInformationView.updateFitting(fitting: fitting)

// Update test relationship with umanaged if necessary
let test: Test = fitting.test?.realm == nil ? fitting.test! : Test(value: fitting.test!)
fitting.test = self.testResultViewController!.testResultView.update(test: test)

// Update group relationship with umanaged if necessary
let group: Group = fitting.group?.realm == nil ? fitting.group! : Group(value: fitting.group!)
fitting.group = group

My solution for now is to return nil for description:
- (NSString *)description
{
<EMAIL_ADDRESS>%@ self: %@ description: %@",self.class.className,self,self.realm);
    return nil;
}

The problems mentioned in https://github.com/realm/realm-cocoa/issues/4747#issuecomment-287668924 and https://github.com/realm/realm-cocoa/issues/4747#issuecomment-289986583 seem at first glance like they stem from an expectation that Object(value:) will perform a deep copy of the passed-in object, when in fact only a shallow copy is performed. This means that any objects referenced via relationships (either properties that are of an Object subclass type or List<T>) would still refer to the instances from the original Realm (and thus from a different thread if the copied object is passed to a different thread). As @jpsim mentioned at https://github.com/realm/realm-cocoa/issues/4882#issuecomment-296810821, we have #3381 tracking adding a built-in way to perform a deep copy in the future.
The initial comment mentions seeing this problem when using a ThreadSafeReference, which must have a different cause. It'd be worth seeing if we can reproduce that.

@markst, can you please share self-contained steps to reproduce the problem when you're able (keeping in mind my point above about Object(value:) only performing a shallow copy)?

Given the age of this issue and the lack of movement, I am closing it. If you see this problem again with the latest version of Realm and are able to build a repro case (or at least offer reproduction steps), please feel free to re-open this ticket or to file a new one.


Not working

I got the public key and put it in as the guide says, but nothing is showing up.
My format for the data is : datasources["ABC"]["123"][13]

Hey, sorry. I'm not using this actively anymore. You may want to check the console of the browser to see if there are any errors which may help.


merge from konsum-frontend-svelte,template-svelte-app

package.json

chore(deps): add ^6.12.51 remove ^6.12.48 (devDependencies.npm-pkgbuild)

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

v1.29.25
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Refactor WorkspaceInitialPage to no longer use withFullPolicy

@arosiclair Please review
Details
Refactor WorkspaceInitialPage to no longer use withFullPolicy and use the FullPageNotFoundView instead.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/218369
Tests/QA

Create a Workspace if you do not have one already
Click on Settings -> [Workspace]
Make sure you can interact with the page fully and without any errors:
https://user-images.githubusercontent.com/4741899/185338389-1b69cfd1-4bf9-424d-934d-06635a312134.mp4

Using the URL bar above, change the workspace ID to something else.
Make sure you see that the workspace is shown to be "Not found":
https://user-images.githubusercontent.com/4741899/185338667-4492d192-998d-4676-a362-42a92a39e2d4.mp4

Using the JS console, switch the network tab to off
Navigate to the workspace, make sure you are still able to and you load the page fully like before.

[ ] For all of the above make sure there are no JS console errors while you do any of this.

Updated and retested!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @arosiclair in version: 1.1.89-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


tag system for protocols?

I got an idea based on a question from Emily Veltjen. The current categorization of protocols is rigid (generic, soil, water, ...) and not easily extendable. Sometimes, it is not 100% clear to which category a protocol should belong. For instance "Generic sampling of a biological matrix for molecular research" could go into category "generic" or category "species".
The INBO tutorials website has this distinction between categories and tags, maybe we can use this for the protocols website too? Tags allow for more flexibility because multiple tags can be associated with one protocol and they need not come from a predefined list. I don't know how easy / difficult it would be to implement something similar for the protocols website.

Honestly I don't know the current protocol workflow very well to judge the implications.  I know functions automatically place protocols in different folders, which won't be necessary anymore, but I don't know how much coding is behind it.  In any case, as this has big implementations for the storage of the protocols, I don't think it is a good idea to postpone it to 'later': it should either be done before starting adding protocols, or we should consider restarting with a new repo when implementing it later on.

Interesting idea IMHO, although some new challenges arise:

literally using the term 'tag' for this causes confusion with the 'general' and 'specific' tags we will use for released protocols, which effectively are git tags.

you probably want consistency among chosen tags between protocols, perhaps by presetting available options (e.g. soil, species, ...) – effectively making it collapse with the idea of (fixed) 'categories', which themselves would probably be closely related to combination of protocol type (f, a, i, o, p) and theme (0, 1, 2, 3, 4, 5) and perhaps extending it a little.

it will cause a hybrid situation:

since we still must stick with protocol types (f, a, i, o, p) and themes (0, 1, 2, 3, 4, 5) – that approach is already long embedded in the protocol codes of available protocols – such tagging will not avoid the dilemma of having to choose the type and theme that figures in the protocol code (and determining filepaths)
the additional 'category' system (avoiding 'tag' here :stuck_out_tongue_winking_eye:) would overlap with the aim of protocol type and theme, and that may cause confusion too, e.g. then best also repeat the actual protocol type/theme as a category of the protocol, potentially next to other ones. So this makes it quite complex (it already was).

A simplification would be achieved if we would only go with the categories system, and drop the current protocol code approach :smile:; however I assume that is not desired (it's deeply integrated in workflows and tools) and it may present other disadvantages (e.g. protocols get sorted regardless of categories).

FYI @Eveltjen00

We didn't pursue this idea further for the reasons @florisvdh gave. So I'm going to close this issue. Feel free to re-open if there is still a need for this.


Floating version selection explained

Simple, we need to have better examples for floating versions.
There are few and far in between today:
Floating versions should not be called wildcard versions

Specifically we should capture the *-* once it's released.

https://github.com/NuGet/docs.microsoft.com-nuget/blob/main/docs/concepts/Dependency-Resolution.md could be expanded upon.


Jeet.gs and Rupture in Stylus, how configure?

Anybody know if Is possible add this features (Jeet.gs and Rupture) on the Stylus that is used with Angular-Cli?

@nvrossett -- the GitHub issue tracker is for project bugs and project feature requests as noted in CONTRIBUTING.md. For better support, consider creating a question on StackOverflow with all the details.

We don't currently have a way to configure CSS proprocessors.

Hi,
Is there a way to configure rupture with angular cli now in 2017?

Do you have a solution about it @nvrossett?

Hi @LFeh =)
Nothing, even stopped studying Angular 2 because I did not feel comfortable with the workflow without these tools, in fact today without Rupture
I do not know how you are today, but to this day I do not know any solution.
Today I am venturing with Vue.js and enjoying a lot, for me is what the Angular 2 should have become, has several similarities with AngularJS and React.
For me it joins the best of the 2 worlds

Nada, até deixei de estudar Angular 2 por que não me senti confortável com a workflow sem essas ferramentas, na verdade hoje sem o Rupture
Não sei como está hoje, mas até hoje não conheço nenhuma solução.
Hoje estou me aventurando com Vue.js e curtindo muito, para mim é o que o Angular 2 deveria ter se tornado, tem varias similaridades com o AngularJS e React.
Para mim une o melhor dos 2 mundos


Use override keyword for overriden class member functions

This helps to prevent errors when you change base class interface and descedant member function is not called after that.
Also,  I've made the function GetHDChain in keystore.h virtual and the function with the same name in the wallet\crypter.h overriden, I think it is more correct if we have functions with the same name and signature in ancestor and descendant classes.

Pinging @codablock - will this interfere with bitcoin backports you are doing?

Yeah, this touches quite a lot of methods. I'll stumble on this when doing a rebase after this is merged...probably in really many of my commit. I would suggest to leave this PR open until we have catched up with Bitcoin

ok, postponing till later then

Agree to hold off on touching bitcoin code for now, for this and other C++11 updates.
src/governance-exceptions.h is Dash-specific so it can be changed, but it also has a deprecated throw() dynamic exception specification that I was looking at changing...
Note that the src/drafted files are not used and will be removed at some point.
So I'd leave src/drafted/governance-classes.h

As we have catched up with Bitcoin 0.14 now, we can probably finish this one. I'll handle merge conflicts for 0.15 later on-the-fly while I backport the PRs.

Needs rebase


Update javascript dependencies

Update javascript dependencies
fix reported vulnerabilities
renamed babel plugin
add repo link and license to package.json

Awesome :)


To make an AI CHESS GAME

PR Description 📜
I have added an AI Chess game using HTML,CSS and JavaScript.
Issue#1730

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you  @AnkitaSikdar005 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Throw an exception when attempting to read columnar encrypted Parquet files on the GPU [databricks]

fixes https://github.com/NVIDIA/spark-rapids/issues/5246
Since Spark 3.2, columnar encryption is supported for Parquet tables with Apache Parquet 1.12+
CUDF doesn't support reading columnar encrypted parquet files, the files are only read on the executor side in the tasks so we can't easily fallback to the CPU.
To work around this, we check for the parquet encryption configs and if we see them we null them out and let the parquet footer filter checks throw. (Caused by: org.apache.parquet.crypto.ParquetCryptoRuntimeException: Trying to read file with encrypted footer. No keys available). The NATIVE reader is a bit different and we can check for the encrypted magic number PARE.   We attempt to catch those failures and then throw our own more useful message.
Message thrown looks like:
Caused by: java.lang.RuntimeException: The GPU does not support reading encrypted Parquet files. To read encrypted or columnar encrypted files, disable the GPU Parquet reader via spark.rapids.sql.format.parquet.read.enabled.
I manually tested v1/v2 readers, Native and java footer types and then perfile, coalescing and multithreaded readers.  see https://spark.apache.org/docs/latest/sql-data-sources-parquet.html#columnar-encryption for information used on testing. I wrote a columnar encrypted file using the CPU and then tested reading via GPU and made sure it threw.
Added automated test that relies on the parquet hadoop tests jar.  So we download it and rely on it being there for those versions that have parquet > 1.12.

build

build

build

Ok, added a test.  It will run in our regular nightly tests but I filed a follow to support in our integration test runs taht will need to download and ship the parquet hadoop tests jar.  I had to add a version check for CDH because it ships with older parquest version.

build

thats is weird the build for this passed in jenkins... kicking again

build


Impersonate System for initial build config creation

Tests failed.

Tests failed.

retest this please

Tests failed.

retest this please

Tests failed.

[merge]


Inject additional js in html output of to_html()

Adds functionality to the html version of the visualization:

Click on data point to toggle persistence of hover-box
Click on text-line in hover-box to copy the value of the line into clipboard.

Signed-off-by: Buech Holger<EMAIL_ADDRESS>

@sfalkner: Want to test drive this feature? I'd be curious about your feedback.


No methods named "Get..." should return unique_ptrs. It should be Extract... or Create...

No methods named "Get..." should return unique_ptrs. It should be Extract... or Create... - Review SDK for that.
Originally posted by @antkmsft in https://github.com/Azure/azure-sdk-for-cpp/issues/1865#issuecomment-797087985
std::unique_ptr<Azure::Core::IO::BodyStream> GetBodyStream() - Extract?
std::unique_ptr<RawResponse> GetRawResponse(...) - Create?
std::unique_ptr<CurlNetworkConnection> GetCurlConnection(...) - Create?
std::unique_ptr<RawResponse> GetResponse() - Extract?
std::unique_ptr<Azure::Core::Http::RawResponse> GetResponse() - Extract?
std::unique_ptr<Azure::Core::Http::RawResponse> GetResponse() - Extract?

@antkmsft Please update the issue describe with all offending instances.

Updated.


0.11.0 release

Pull request
Description
0.11.0 release
Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

Performance
no code changes

Coverage remained the same at 83.185% when pulling dd44cd20996895056d14bfb2ad0e82de7044e102 on rel-0.11 into c50eb02818e2ce23ae8d32fc87a8890c67e3d869 on main.


DROOLS-733 - UI fixes for KieServer, added possibility to update serv…

…er instances status from UI

@mswiderski Unfortunately tests BoxPresenterTest.testSetupLoadingServerRef() and BoxPresenterText.testSetupLoadingServer() fail with this PR.

thanks @manstis it was more for local try outs and more improvements are on the way. Will update this PR when they are ready and make sure all tests pass.

Thanks.. and if we're really following the law of the land.. I shouldn't
really accept this PR until the documentation has been updated too.. but
assuming this is to be done when the Kie Server refactor is complete I can
let this slip... for now... :)
On 2 September 2015 at 16:35, Maciej Swiderski<EMAIL_ADDRESS>wrote:

thanks @manstis https://github.com/manstis it was more for local try
outs and more improvements are on the way. Will update this PR when they
are ready and make sure all tests pass.
—
Reply to this email directly or view it on GitHub
https://github.com/droolsjbpm/kie-wb-common/pull/73#issuecomment-137136014
.

@manstis tests fixed so this PR is good to go from my point of view
docs will be updated once we have stable/satisfying kie server hopefully within a week or so (thanks a lot for trying it out and finding issues:))

(master) http://github.com/droolsjbpm/kie-wb-common/commit/eefae8912
(6.3.x) http://github.com/droolsjbpm/kie-wb-common/commit/6713f6d7b


Classes on a HTML SELECT are removed

Describe the bug
The classes are removed from the HTML select on the popup.
To Reproduce

run the jsfiddle
the classes of the select on the popup (sweetalert library) are removed

Expected behavior
The select on the popup should have the class MYCUSTOMCLASS
Reproducible example
https://jsfiddle.net/k0x169oe/
Additional context
I noticed that deleting the simplebar library solves the problem (try on the jsfiddle). More specifically, i noticed that by removing HTMLSelectElement from the domIterables variable, the problem no longer occurs.
// iterable DOM collections
// flag - `iterable` interface - 'entries', 'keys', 'values', 'forEach' methods
 domIterables = {
	  CSSRuleList: 0,
	  CSSStyleDeclaration: 0,
	  CSSValueList: 0,
	  ClientRectList: 0,
	  DOMRectList: 0,
	  DOMStringList: 0,
	  DOMTokenList: 1,
	  DataTransferItemList: 0,
	  FileList: 0,
	  HTMLAllCollection: 0,
	  HTMLCollection: 0,
	  HTMLFormElement: 0,
	  ---> HTMLSelectElement: 0,
	  ....
};

I think you posted this on the wrong repository!


http: only close connector if it is available

Stuff like asyncio-retry offer their own clients which proxy the original ClientSession, and they do not bind stuff like _connector directly. Since this is a private-attribute, it might be nicer if we access it optionally and only try to close if we can find it.

For py 3.6, I think you need loop.run_until_complete

Ah, right. I'll just skip these under 3.6 (as we do in test_asyn, https://github.com/intake/filesystem_spec/blob/e9bd63f795d84c35898bef3223fc7e055d49c774/fsspec/tests/test_async.py#L56)

That's another solution :)

I am running the tests again, but the windows async failure is a little worrying, even if it was a one-off.

I recall seeing this error before too. Would you mind explaining what test_async_other_thread does?

what test_async_other_thread does

The filesystem's loop is running in another explicit thread, and we manipulate the loop object directly to get it to do things (i.e., not using async/await). This is actually quite similar in effect to specifying no loop/asyncronous flags at all and calling the blocking functions.

Perhaps try changing
loop = asyncio.get_event_loop()
to
loop = asyncio.new_event_loop()

(don't know why this should show up just now...)

The only thing that I guess might affect is the order of tests, let me temporarily move the test to the end and see if it passes or not.

OK, well it passed - we can revisit if it happens again.


[liblzma] Build error on x64-windows-static-md

Package<EMAIL_ADDRESS>Host Environment

Host: x64-windows
Compiler: MSVC 19.29.30151.0
vcpkg-tool version: 2024-03-14-7d353e869753e5609a1f1a057df3db8fd356e49d
vcpkg-scripts version: 1daec834f 2024-04-02 (5 hours ago)

To Reproduce
vcpkg install pcl:x64-windows-static-md
Failure logs
-- Downloading https://github.com/tukaani-project/xz/archive/v5.4.4.tar.gz -> tukaani-project-xz-v5.4.4.tar.gz...
[DEBUG] To include the environment variables in debug output, pass --debug-env
[DEBUG] Trying to load bundleconfig from D:\share\github\repo\vcpkg\vcpkg-bundle.json
[DEBUG] Failed to open: D:\share\github\repo\vcpkg\vcpkg-bundle.json
[DEBUG] Bundle config: readonly=false, usegitregistry=false, embeddedsha=nullopt, deployment=Git, vsversion=nullopt
[DEBUG] Metrics enabled.
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] Feature flag 'dependencygraph' unset
Downloading https://github.com/tukaani-project/xz/archive/v5.4.4.tar.gz
[DEBUG] Trying to hash D:\share\github\repo\vcpkg\downloads\tukaani-project-xz-v5.4.4.tar.gz.2700.part
[DEBUG] D:\share\github\repo\vcpkg\downloads\tukaani-project-xz-v5.4.4.tar.gz.2700.part has hash 03a20c9e1f751370d57a92b66ba3a1bd383919f68ac2ea02d27d30c3736827b6a28f2c66426a302449531ef1c426ced68b1a280af3335e774f4f59227f92e866
error: Failed to download from mirror set
error: File does not have the expected hash:
url: https://github.com/tukaani-project/xz/archive/v5.4.4.tar.gz
File: D:\share\github\repo\vcpkg\downloads\tukaani-project-xz-v5.4.4.tar.gz.2700.part
Expected hash: c28461123562564e030f3f733f078bc4c840e87598d9f4b718d4bca639120d8133f969c45d7bdc62f33f081d789ec0f14a1791fb7da18515682bfe3c0c7362e0
Actual hash: 03a20c9e1f751370d57a92b66ba3a1bd383919f68ac2ea02d27d30c3736827b6a28f2c66426a302449531ef1c426ced68b1a280af3335e774f4f59227f92e866
[DEBUG] D:\a\_work\1\s\src\vcpkg\base\downloads.cpp(1031):
[DEBUG] Time in subprocesses: 0us
[DEBUG] Time in parsing JSON: 3us
[DEBUG] Time in JSON reader: 0us
[DEBUG] Time in filesystem: 6060us
[DEBUG] Time in loading ports: 0us
[DEBUG] Exiting after 390 ms (374494us)

CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):

      Failed to download file with error: 1
      If you are using a proxy, please check your proxy setting. Possible causes are:

      1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
         to `https://address:port`. This is not correct, because `https://` prefix
         claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
         , etc..) is an HTTP proxy. Try setting `http://address:port` to both
         HTTP_PROXY and HTTPS_PROXY instead.

      2. If you are using Windows, vcpkg will automatically use your Windows IE Proxy Settings
         set by your proxy software. See https://github.com/microsoft/vcpkg-tool/pull/77
         The value set by your proxy might be wrong, or have same `https://` prefix issue.

      3. Your proxy's remote server is out of service.

      If you've tried directly download the link, and believe this is not a temporary
      download server failure, please submit an issue at https://github.com/Microsoft/vcpkg/issues
      to report this upstream download server failure.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:270 (z_vcpkg_download_distfile_show_proxy_and_fail)
  scripts/cmake/vcpkg_from_github.cmake:106 (vcpkg_download_distfile)
  ports/liblzma/portfile.cmake:1 (vcpkg_from_github)
  scripts/ports.cmake:175 (include)

same #37919

Duplicate of https://github.com/microsoft/vcpkg/issues/37893.


Feature/fix cursor timeout

This is a weird one; we have a collection with 51 million records that we need to mirror to postgres and we kept hitting the CURSOR_NOT_FOUND exception.  As recommended on https://github.com/mongodb/mongo-ruby-driver/wiki/FAQ#i-keep-getting-cursor_not_found-exceptions-whats-happening I tried adding the :timeout => false parameter, which did not change anything.  However, once I additionally removed the "with_retries" around it everything works.  I don't know why that would break it, I'm not much of a ruby guy, but it doesn't seem to be doing anything near as important as getting all entries in a large collection, so unless you have a suggestion for another way to fix this I suggest this fix.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: add toc in definition pages

refactor faq and definitions to use the same basetype and loadAll function
parse markdown statically instead of using react-markdown
automatically add slug id to headings, retrieve headings info and add them to IArticle type
add toc in definition pages
remove react-markdown
add heading anchor link
Add a summary ui component

We should  create some IMarkdown types useful to type markdownProcessor signature and for your props

Plus one thing that would really be lovely would be to add visual anchors like I did in https://api.gouv.fr/les-api/api-particulier#a-quoi-sert-l’api-particulier-?
I always felt contrained by react markdown so I did not do it but I find these anchor very helpful for support

Plus one thing that would really be lovely would be to add visual anchors like I did in https://api.gouv.fr/les-api/api-particulier#a-quoi-sert-l’api-particulier-?

Really simple to do so with this refacto, even if it means adding a new dependency : https://github.com/rehypejs/rehype-autolink-headings

Is nt a dependency a bit overkilled for just a few line of code ?

Ok I choose to add a new dependency, rehype-autolink-heading for heading headlines :

It's the easiest and clearest way of doing it inside remark / rehype pipeline
If I wanted to to without, and still use unified, I would have to install other utils dependencies anyway to transform the AST
The other possibility is to move to react world and use react-remark to parse markdown and add transformation directly with react utilities (find headings in children, etc.)

The drawback, is the possibility to add this logic on the client side if we are not careful with our boundary, which would be a big overhead

I'm not fond of adding a lot of dependencies on a project either. But I think in that case, it's justified. If it becomes a hassle to upgrade, we can go back to react-remark

Also : if you rebase with main, preview app should work


Getting incorrect x,y frame for image captured using camera ios

I am using on device text detection ,MLVisionTextModel.
in which I am capturing image using camera sending image to textdetector with metadata of image orientation.
When I print result , it shows in incorrect y position. line which is on top of other line has greater value than of line below it.
where as if I directly embed image into the project architecture and load it for textdetection then it works well.
Not sure what is going wrong here.

var textDetector: VisionTextRecognizer!
lazy var vision = Vision.vision()
textDetector = self.vision.onDeviceTextRecognizer()
let visionImage = VisionImage(image: image)
let metadata = VisionImageMetadata()
metadata.orientation = self.detectorOrientation(in: image)
visionImage.metadata = metadata
textDetector.process(visionImage) { (features, error) in
if features != nil
{
self.processResult(from: [features!], error: error)
}
}

//Detects orientation of the selected or captured image
func detectorOrientation(in image: UIImage) -> VisionDetectorImageOrientation {
    switch image.imageOrientation {
    case .up:
        return .topLeft
    case .down:
        return .bottomRight
    case .left:
        return .leftBottom
    case .right:
        return .rightTop
    case .upMirrored:
        return .topRight
    case .downMirrored:
        return .bottomLeft
    case .leftMirrored:
        return .leftTop
    case .rightMirrored:
        return .rightBottom
    }
}

Closing since this has staled. Please comment here and I'll reopen.


👀 vue-codemirror + vite4 = nothing

Describe the bug

    "@codemirror/lang-java": "^6.0.1",
    "@codemirror/lang-markdown": "^6.0.5",
    "@codemirror/state": "^6.1.4",
    "@codemirror/theme-one-dark": "^6.1.0",
    "@codemirror/view": "^6.7.1",
    "@intlify/unplugin-vue-i18n": "^0.8.1",
    "@zebing/vite-plugin-css-modules": "^1.1.0",
    "ant-design-vue": "^3.2.15",
    "auto-changelog": "^2.4.0",
    "codemirror": "^6.0.1",
    "esbuild": "^0.16.6",
    "jsonlint": "^1.6.3",
    "jsonlint-mod": "^1.7.6",
    "pinia": "^2.0.28",
    "postcss-load-config": "^4.0.1",
    "vite-plugin-implicit-css-modules": "^0.0.3",
    "vite-plugin-vsharp": "^1.3.0",
    "vite-plugin-vue-static-css-modules": "^3.0.3",
    "vue": "^3.2.45",
    "vue-codemirror": "^6.1.1",
    "vue-codemirror6": "^1.1.1",
    "vue-cookie-next": "^1.3.0",
    "vue-demi": "^0.13.11",
    "vue-i18n": "^9.2.2",
    "vue-router": "4.1.6",
    "vue-toastification": "^2.0.0-rc.5"
           ...
   "ts-node": "^10.9.1",
    "typescript": "^4.9.4",
    "unplugin-auto-import": "^0.12.1",
    "unplugin-vue-components": "^0.22.12",
    "vite": "^4.0.1",

Hi, I did everything according to the guide, as a result, nothing works.
no code highlighting
there is nothing
Even the out of the box option doesn't work.
Reproduction
<template>
   <div>
   <Codemirror
            v-model="value"
            basic
            wrap
            tab
            :dark="dark"
            :lang="lang"
            :extensions="extensions"
            :options="cmOption"
          ></Codemirror>
        </div>
        //....
</template>
<!-- SCRIPT -->
<script lang="ts" setup>
  import Codemirror  from 'vue-codemirror6';
  import { markdown as md } from '@codemirror/lang-markdown';
  import type { LanguageSupport } from '@codemirror/language';
  import type { Extension } from '@codemirror/state';
  import type { ViewUpdate } from '@codemirror/view';
  import { oneDark } from '@codemirror/theme-one-dark';

  console.log(`[LOG] oneDark`, `<${typeof oneDark}>`, oneDark);

  const value = ref(`
  adw
  da
  widthwad
  awda
  widthawd
  awdawdawdwd

  `);

  /** Dark mode **/
  const dark = ref(window.matchMedia('(prefers-color-scheme: dark)').matches);

  /**
   * CodeMirror Language
   *
   * @see {@link https://codemirror.net/6/docs/ref/#language | @codemirror/language}
   */
  const lang: LanguageSupport = md();
  const extensions = [md(), oneDark];
  const cmOption = ref({
    lineNumbers: true,
  });
        </script>

System Info
System:
    OS: Windows 10 10.0.22000
    CPU: (12) x64 Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz
    Memory: 2.38 GB / 15.83 GB
  Binaries:
    Node: 19.2.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - C:\Program Files\nodejs\yarn.CMD
    npm: 9.1.3 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22000.120.0), Chromium (108.0.1462.46)
    Internet Explorer: 11.0.22000.120

Used Package Manager
yarn

There is no prop definition called options in vue-codemirror6.
It seems that there is no special usage, so I think it will work with the following code.
<script setup lang="ts">
  import Codemirror  from 'vue-codemirror6';
  import { markdown as md } from '@codemirror/lang-markdown';
  import { oneDark } from '@codemirror/theme-one-dark';
  import type { Ref } from 'vue';

  console.log(`[LOG] oneDark`, `<${typeof oneDark}>`, oneDark);

  const value: Ref<string> = ref(`
  adw
  da
  widthwad
  awda
  widthawd
  awdawdawdwd
  `);

  /** Dark mode **/
  const dark: Ref<boolean> = ref(window.matchMedia('(prefers-color-scheme: dark)').matches);
</script>

<code-mirror
  v-model="value"
  basic
  wrap
  tab
  :dark="dark"
  :lang="md()"
  :theme="oneDark"
/>

thanks i will check later


1.20.3 and 1.20.4 support.

Tested in my Roulette plugin, working correctly.

Thanks! I'll merge tomorrow!

Thank you for your contribution! You did well even if you're new to git :)


Background is white and can't be changed

I've tried both ly 0.4.0 and the latest git version (installed from the AUR packages ly and ly-git, respectively) under Manjaro Linux + i3 and Sway. In both cases, the background is almost always white, although the line bg=0 is commented out in /etc/ly/config.ini (which should default to black, I assume?). This makes the text somewhat hard to read.
I’ve tried uncommenting that line and I get the same results.
Also, I’ve found an example of setting the background and foreground in #80, but from my understanding, it is mentioned there that this option wouldn't work under tty. I don’t know if the situation has changed since then and if it has, what format/value is accepted in the current version.

I hit the similar situation; In my case the bg and the fg were always black and white respectively and config did not seem to affect. I don't know what I'm missing, but I found a workaround finally.
I edited the /lib/systemd/system/ly.service appending a ExecStartPre line to modify tty's color pallet.
[Unit]
Description=TUI display manager
After=systemd-user-sessions.service plymouth-quit-wait.service
<EMAIL_ADDRESS>
[Service]
Type=idle
ExecStartPre=/usr/bin/printf '%%b' '\e]P0FFC5C4\e]P7FFFFFF\ec'
ExecStart=/usr/bin/ly
StandardInput=tty
TTYPath=/dev/tty2
TTYReset=yes
TTYVHangup=yes

[Install]
Alias=display-manager.service

The above example sets color0(bg) to #FFC5C4 and color7(fg) to #FFFFFF.

I found that term_reset_cmd in /etc/ly/config.ini also has to be modified to keep the colors after logging-in and while in second time login screen after logging-out again.
term_reset_cmd = /usr/bin/tput reset; /usr/bin/printf '%b' '\e]P7F1E9C7\e]P0090602\ec'

@takagiy I tried the steps of your answer but I didn’t succeed here :/

I found that term_reset_cmd in /etc/ly/config.ini also has to be modified to keep the colors after logging-in and while in second time login screen after logging-out again.
term_reset_cmd = /usr/bin/tput reset; /usr/bin/printf '%b' '\e]P7F1E9C7\e]P0090602\ec'

P7F1E9C7 <----- is this color code?


Add illustration for lazy storage field

@xgreenx Wdyt?

Maybe it would be more clear Lazy<MyType, ManualKey<0x12345678>>. Because otherwise, a reader will have a question about why 0x12345678=) Or maybe you want to add a note that 0x12345678 is not real and only to show that the key is different.
Also, maybe it is better to write the storage cell key near each field(I don't know how to do that in the best way) to show that a, b, and c will have 0x0000000 while Lazy 0x12345678.


support nested messages and enums; support multiple module loading regardless of load order

Hi, we'd really like to see the infinite-loop issue fixed and this PR looks good and has tests. Unfortunately it's 1.5 years old at this point.
@hchasestevens Is this project still active? Any thoughts on moving this PR forward?

Hi @charlesnicholson ,
Thanks for reaching out. The PR has a few conflicts that would need to be solved before merge. I don't know if @igorpotapyev has plans to do so; I would also happily accept a PR from yourself if you wanted to rehabilitate this one/otherwise fix this bug.


Add ability to preload fonts

Issue
When a Google font is used the first time, the text shows in the fallback font initially, and then shows the actual Google Font as shown in the attached recording.
This behavior appears in both cases:

Font is available in assets and GoogleFonts.config.allowRuntimeFetching = false.
Font is fetched from network/cache and not available in asset.

The recording is based on the second case in a Flutter Web app, and the font is most likely coming from cache. For the first case, the font-switch is quicker but it is still apparent.
https://user-images.githubusercontent.com/46427323/109412438-0da8a980-79b9-11eb-9779-e098d210985d.mov
Request
It'd be nice to have an asynchronous function that can be called and waited for to preload the fonts for a given list of fonts or from all the fonts in the assets. This will allow avoiding this undesirable font-switch from showing up.  An example API for the suggested request would be something like this:
final fonts = <String>['Caveat', 'Roboto', ....... , 'Lato'];
await GoogleFonts.preloadFonts(fonts);

// or
await GoogleFonts.preloadAssetFonts();

Additional Context
The following was previously suggested by @clocksmith in #103 but the issue is closed:

When fonts are coming from assets, they still need to be loaded by the FontLoader, which is very fast when loading from assets, but not necessarily done before the first screen. We might need to make a preload function available to support this case (something that calls loadFontIfNecessary).

Would be very useful. Came here to create exactly the same issue.

I came to report the same. +1 👍

For this solution, would be it ok to block on the splash screen while the fonts are loaded, causing a slightly delayed start up time?

@clocksmith I think that would be the best.

No. This way if the app supports multiple languages, the splash screen will be dramatically delayed, even though the user does not necessarily needs all those fonts for multiple languages.
It would be much better if we had an async function using which we could preload specific fonts.

I also found this issue when drawing some text to the canvas. And since the font was not loaded when the canvas was drawn, the font remains unchanged. Unless I triggered repainting the canvas by for example increase the font size.

I also found this issue when drawing some text to the canvas. And since the font was not loaded when the canvas was drawn, the font remains unchanged. Unless I triggered repainting the canvas, it shows the font properly.

This is the same issue as we are facing with google fonts in Flame too, since we are drawing the text directly on the canvas.

Wanted to create a same issue. Its a mandatory feature.

+1 👍

I also found this issue when drawing some text to the canvas. And since the font was not loaded when the canvas was drawn, the font remains unchanged. Unless I triggered repainting the canvas, it shows the font properly.

This is the same issue as we are facing with google fonts in Flame too, since we are drawing the text directly on the canvas.

Also having issues using it in flame because it is not preloaded. But in our usecase we are not loading from assets and we are loading the fonts dynamically. It would be nice to await preloading specific fonts right before they are we get them.
Something like
await GoogleFonts.getFontAsync()

Made a feature request here
@spydon seems to be working with flame components also and it is possible to make multiple calls from multiple components to preload a font.

same issue
+1

Update: this will be done this quarter
Googlers, see go/preloading-fonts-flutter

@guidezpl
quarter is almost up! haha j/k couldn't resist the saying. hope to see this implemented soon.
thanks

Hello, Is there any way to do it while it is being implemented?
For example is there a way to know if a font was loaded in the FontLoader?
I want to know if a google font is already loaded, just checking if the font family already exists, if not I know I should loaded and I wait a delay until is loaded.
Thank you

🎉 🎊 🎈

This is still not done ??

This is a freely available package I work on when I have time. If you are unsatisfied, I encourage you to contribute. Beyond that, I closed this with #195.