[Task][Go SDK]: Respond to Control FnAPI messages per-Bundle ordering, not globally.

What needs to happen?
At present the Go SDK harness responds to FnAPI Control in order of receipt, other than ProcessBundleRequests which are processed in their own thread.
This may be causing contention on the control GRPC thread as all other messages are happening sequentially as it's also the "grpc" receipt thread. This includes handling "keepalive" responses on the GRPC connection and other maintenance messages.
https://github.com/apache/beam/blob/master/sdks/go/pkg/beam/core/runtime/harness/harness.go#L168
Per Bundle Processing spec (Link TBD), the response ordering only needs to be strictly ordered based on the specific bundle instruction. Eg. requests to split, get progress, etc.
A simple solution A would be to remove the response processing from the network thread, allowing that to drain, and having a separate processing thread. This avoids contention from the network thread to some buffered channel limit.
A more elaborate solution B would be to spawn an additional "side car" goroutine for each in flight bundle to handle these ordered messages, similarly unblocking the network thread and allowing the different side messages to be processed in order. Each would get it's own buffered channel to some reasonable value, and allow the unblocking to occur.
The concern with B is in "high concurrent bundle" cases which may be unnecessary memory overhead per bundle, but this is speculation and would need to be validated through profiling.
In either case, the same single "response" channel will be used to serialize the responses to the runner half since that should remain a low return effort reply. (subject to profiling).
cc: @lukecwik @lostluck
Issue Priority
Priority: 2
Issue Component
Component: sdk-go

#23583 removes a long operations from out of the lock, but another action would be to separate the lock into either map specific locks, or into control message usage specific locks. That could also avoid losing process on the control channel handler goroutine.

The SDK should do a keyed de-dupe to make a single in-flight for the same requests
That would have benefits for when the descriptor is large, as we'd save the deserialization time, if not the plan construction time.
This would have more benefits for streaming jobs than batch, but it's a good move nevertheless, as it would reduce wire congestion.

Not likely the issue the user is facing, but Bundle Finalization seems to be doing a fair amount of work in the control lock critical section that could be moved out.
https://github.com/apache/beam/blob/master/sdks/go/pkg/beam/core/runtime/harness/harness.go#L413
and accessing the plans map doesn't appear to be locked here too:
https://github.com/apache/beam/blob/master/sdks/go/pkg/beam/core/runtime/harness/harness.go#L505


DOP-371: Correctly represent page flags

[DOP-371] Fixes a small issue that's been bothering me for a while! Page flags are currently represented as empty strings rather than booleans. E.g. :noprevnext: becomes this:
{'noprevnext': ''}

instead of this:
{'noprevnext': True}

This is possible because of #220, when the page options object was modified to allow all SerializableType data. Will open a small corresponding front end PR when this is merged in! Thanks 😀

Closing in favor of a more deterministic approach!


LineageOs on IPFS

If I understood this repo correctly, it's for pitching proof of concepts and applications of novel ideas accompanied by a blog post. So, here we go.
LineageOS on IPFS
Basically, I created a Go daemon to read LineageOS' API in search of new builds and I pinned the latest build of every device.
Proof of concept :

https://lineageos-on-ipfs.com/
https://lineageos-on-ipfs.com/ajax/devices.php

Sources :

https://gitlab.com/NatoBoram/lineageos-on-ipfs.com
https://gitlab.com/NatoBoram/LOSGoI

Conclusion : https://github.com/ipfs/ipfs-cluster/issues/783

Really cool!


UDFRegistration.registerJava() not whitelisted

Hi Splink team,
This function is not whitelisted in my databricks environment.
org.apache.spark.sql.UDFRegistration.registerJava()
Unfortunately, the latest version of Splink uses it to import some distance metric UDFs. (line 277 of spark/linker.py).
Would it be possible to please add a parameter so that I can choose to bypass this line? (since I don't need the distance metric UDFs).
Otherwise, I think I will need to revert to an older version of Splink.
Thanks in advance.
Jonny

Hi Jonathan,
Yes, I think we should be able to do this.
If you're interested in contributing to Splink,  you'd want to add an additional argument here e.g. register_udfs_automatically:
https://github.com/moj-analytical-services/splink/blob/26950eabce0f7f8d9e8cc13155c01c08d6e0ec61/splink/spark/linker.py#L97
The argument would then activate/deactivate this line of code from running:
https://github.com/moj-analytical-services/splink/blob/26950eabce0f7f8d9e8cc13155c01c08d6e0ec61/splink/spark/linker.py#L191
Probably the new argument wants a default value that means the self._register_udfs_from_jar() runs, but if the user sets the value of register_udfs_automatically false,  then self._register_udfs_from_jar() is not run.
We'll try to get around to doing this sometime next week hopefully if you're not able to add it.
Thanks,
Robin

Hi Robin,
Thanks for the reply. Yes, that's exactly what I was thinking.
I'd love to contribute to Splink! It'll be next week, but I'll have a go at this.
Cheers,
Jonny

Linting errors now fixed, please try and merge again.
Thanks!

Closed by #1774


Specify YouTube URL

What is the purpose of your issue?

[ ] Bug
[x] Feature Request
[ ] Question
[ ] Other

Description

I wish there was a way to specify what video you wanted to pull the audio from, that was different from the -m option. something like this:
spotdl -s spotify:track:1qxcyvBOcu3wnZpxlihnBf -y https://www.youtube.com/watch?v=3QHX7Q9rfno
I have done this song with the -m option, and the right song doesn't show up so I end up having to tweak the search terms in the config.yml. It would be great to be able to choose the song from Spotify for the metadata, and also choose the YouTube video.

Duplicate of #500.


Improve Bundle Size?

Hey, Rebass is awesome! It is pretty heavy (which makes sense) at 172.7kb raw as measured by the vscode plugin: import cost. If you think it is somewhat stable with it's usage of styled-components and your helper styled- libraries now, maybe trying to trim down the bundle size is worth doing now.
I will try to compile a list of potential low hanging fruit within the dependencies.

https://github.com/jxnblk/palx/issues/6: Potential 10kb savings: replace chroma with color in palx

Of course, feel free to close if you don't think that it is that big of a deal for rebass, you plan for more substantial iteration, or whatever else. Perhaps rebass is for when bundle sizes don't matter and csx is for when they do.

BTW, import-costs is a lie because it doens't take in consideration the rest of your dependencies.
A better metric<EMAIL_ADDRESS>

Closing this due to the v2 rewrite


Iterable

Hi,
I've added the Iterable interface, as desired in https://github.com/pavlunya/Bitmask/issues/1
Also completed https://github.com/pavlunya/Bitmask/issues/4
And a couple of other tweaks
Hope this is all okay with you!

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             master     #5   +/-   ##
=======================================
  Coverage       100%   100%
- Complexity        0     32   +32
=======================================
  Files             2      2
  Lines            35     62   +27
=======================================
+ Hits             35     62   +27

Impacted Files
Coverage Δ
Complexity Δ

src/Mask.php
100% <100%> (ø)
29 <12> (+29)
:white_check_mark:

src/MaskException.php
100% <100%> (ø)
3 <1> (+3)
:white_check_mark:

Continue to review full report at Codecov.

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

@antriver
Thanks a lot for your help! Tomorrow I'll release new version with your PR as is.
But I see one thing to improve.
protected $flags = [];

Variable $flags is like $mask but in another form. So I think only one should exist and we should iterate through $mask (I don't know exactly how right now).
I'll try to implement this soon but I can't say exactly when.

@pavlunya agreed about the duplication. I thought of $flags as more of a cache to avoid having to find the flags in set in $mask on every call to getFlags (which would have to be called on every foreach iteration).
It could use just the $flags array, and use isset() to see if the flag is set. But that defeats the purpose of using a bitmask at all I think.


Add annotations to capture file format

Adds a new annotation block type to the capture file format.  The annotation consists of a string with a name/label for the annotation and a string with the annotation data.  The data can be represented as plain text, XML, and JSON.
Adds a new AnnotationHandler interface.  Custom handlers can be created for processing annotation data.
Adds support to the FileProcessor for processing annotation blocks that are read from the capture file, passing the annotation name and data to an AnnotationHandler.

This looks like a really good start!


search node

Hi !
I've loaded with the apis up to 400 devices... and I lost lot of time to search device in the device list :
Will it be possible to imagine have search capabilities, or at least the possibility to change the number of devices displayed at each page, to improve the user experience ?

Sounds like a good feature :-) I guess being able to search on name and / or DevEUI (prefix?) would make sense in this case.

cool !

@brocaar Are you working on this or do you need any help? I'm implementing it on my fork, so I wouldn't mind adding it to yours and creating a PR when ready.

@brocaar @gillespilloudwyres I forked from the latest version and added the node search functionality by either name or DevEUI, and matching any substring given, not the whole field. You can check it at https://github.com/iegomez/lora-app-server. If you think it's a good addition, I'll make a pull request.
Cheers!

@iegomez Cheers! I was out of the country for a conference. Will take a look at it this or next week :-)

@brocaar I saw your comment on the pagination issue: "Regarding the search solution I need to do some more research around performance (e.g. what changes this would require on the DevEUI column in order to avoid table scans)."
I looked up some ways to deal with LIKE or ILIKE queries avoiding the full scan, and it seems the most common solution is using the pg_trgm extension to create a gin index on the search column, as the query will take the index in consideration (from Potgres 9.1 and above). You can find a great summary at https://niallburkley.com/blog/index-columns-for-like-in-postgres/
More detailed information here: http://0x80.pl/articles/sql-ngram-index.html
That said, I'm not really sure if this would work when applying text() to the dev_eui byte array, I guess we'd have to analyze the query plan matching only dev_eui and not name, with and without the extension, to check if there's any difference.

I tried defining a gin index on text(dev_eui), but it made no difference on the select plan, in both cases (with and without index) a sequencial scan was made. Any ideas?

A sequential scan will still be faster up to a lot of row.

I was just testing with ~2000 to see if there was any difference, as Orne was concerned about scans. With that amount the search is fast, about 12 ms, but the concern still stands. I'm no DB expert, so I'm not sure if there's a way to deal with this.

12 ms doesn't means much :)
You need to consider in what machine are you running, SSD or HD, did your query actually hit the disk, what was the load of the machine at the moment and so on...
There is much to explore in this problem but most will come down to a simple trade-off that includes how many node is necessary to support.
A quick example.
In my database the dimension of a row in the node table is of roughly 300 bytes (some arrive to 900bytes, but are less than 300).
A page in postgres is of 8kB so on a single page will fit roughly 25 nodes.
It also means that a sequential scan on 2500 nodes will need to fetch (very worst case scenario) 100 pages (most will be in cache/memory already).
How long does it take to fetch a page depends on your hardware, cache-friendlyness of the layout and so on and so forth.
On the other side with an index what will be slower will be the write time, this is extremely messy to compute, but image a simple binary tree (GIN index are actually B+ tree) and image to put a new node in the tree, every time that you hit an intermediate node you do a page access (an uncached one).
However when you do a search you don't hit anymore every single node in the table but only the one in tha path between the root and your target.
At the end of the story it depends if we prefer fast insertion (no index) or fast look up (index), however, as long as the table is small it won't make any difference.

In your query you will need two GIN index, one for devEUI and one for name.
:)

I'm aware how btrees work and the complexity of select/insertion (didn't know about those page sizes though, nice info), and yes, I think fast look up is what @gillespilloudwyres an @brocaar aim for. What I don't know is if Postgres has a way of using an index over text(dev_eui) for a select with ILIKE and wildcards (my test shows that at least the gin index approach doesn't change the sequencial plan), or if there's another option (does ILIKE or something similar allow to search against bytea instead of text with wildcards or something like that? would an index work in such option?) or a better approach (is it a valid option to create another column that holds a text value for the dev_eui -which should benefit from the gin index, as name does- only for search purposes?.
I think these are the questions that need answer for the search feature to be correctly implemented, specially if it would make it to the default release, because you can't assume the number of nodes any given user will have. In my case the sequencial look up is good enough, but that may not be the case for someone else.

Great :)
Have you try to use an EXPLAIN.
It will tell you whenever the index would be useful.
A naive EXPLAIN will clearly show the sequential scan, if you disable the scan it should help understamd if the index is actually useful.
:)

Sorry for the late response, I've been quite busy lately. I used explain analyze in the test I mentioned early. Now I followed your suggestion and disabled scans, but it doesn't seem to make a significant difference (nor does it mention using the gin index, only the node id index). Of course, ~10000 records aren't much, so maybe the planner just omits the index because a sequential scan is good enough. I'll need to test further.
Thanks for the help!

Hi,
that is weird, 10k rows should be enough to start using indexes.
There are two main thing that I would like to be sure, assuming that the query you are running is the one in your commit history: https://github.com/brocaar/lora-app-server/commit/d149b2241033841e43790ff2a83d325463af5ba3#diff-1b3165fff3fcd132a3b217e13372ee83R342
None of the indexes are used? Neither the one on the column "name"?
How did you create the index on the column "dev_eui"? Did you create it on "dev_eui" or on "text(dev_eui)"?
Speaking about "text", why did you use it? Isn't "encode/decode" more appropriate? Couldn't find much documentation on "text"
If you could share the database I could try to work on it too... Or if you have some kind of script that quickly populate it...

The query I was testing doesn't consider the name part, as a varchar field is not the issue. So there's no index to be used for the name column in the test, and the only index that got used is the application_id one when disabling seq scans. Here's an example run:
explain analyse select * from node where application_id = 1 and text(dev_eui) ilike '%02%' order by name;
                                                                  QUERY PLAN
----------------------------------------------------------------------------------------------------------------------------------------------
 Sort  (cost=703.99..708.99 rows=2000 width=137) (actual time=15.282..15.325 rows=567 loops=1)
   Sort Key: name
   Sort Method: quicksort  Memory: 175kB
   ->  Index Scan using idx_node_application_id on node  (cost=0.29..594.33 rows=2000 width=137) (actual time=0.124..14.104 rows=567 loops=1)
         Index Cond: (application_id = 1)
         Filter: ((dev_eui)::text ~~* '%02%'::text)
         Rows Removed by Filter: 9435
 Planning time: 0.578 ms
 Execution time: 15.441 ms
(9 rows)

The index was created on text(dev_eui), which I used because the dev_eui field is a bytea one, so I couldn't use a gin index over it (or use ilike fot that matter). Why text instead of encode/decode? It was just my first idea, I'll try using the other option.
For the records I just through a simple func that created nodes with random DevEUIs and some default options and called it in main. Anyway, if you'd like to have the same specific records, I'll dump the ndoes table and send it later.

I created an index on encode(dev_eui, 'hex'):
create index trgm_encode_idx_node_dev_eui on node using gin (encode(dev_eui, 'hex') gin_trgm_ops);
The result was pretty much the same:
explain analyse select * from node where application_id = 1 and encode(dev_eui, 'hex') ilike '%02%' order by name;
                                                                  QUERY PLAN
----------------------------------------------------------------------------------------------------------------------------------------------
 Sort  (cost=678.99..683.99 rows=2000 width=137) (actual time=17.133..17.176 rows=567 loops=1)
   Sort Key: name
   Sort Method: quicksort  Memory: 175kB
   ->  Index Scan using idx_node_application_id on node  (cost=0.29..569.33 rows=2000 width=137) (actual time=0.214..15.910 rows=567 loops=1)
         Index Cond: (application_id = 1)
         Filter: (encode(dev_eui, 'hex'::text) ~~* '%02%'::text)
         Rows Removed by Filter: 9435
 Planning time: 0.682 ms
 Execution time: 17.333 ms
(9 rows)

To compare, I created a gin index on name and run a similar query but on name, with a pretty different result:
explain analyse select * from node where application_id = 1 and name ilike '%ode%';
                                                            QUERY PLAN
-----------------------------------------------------------------------------------------------------------------------------------
 Bitmap Heap Scan on node  (cost=101.51..444.52 rows=10001 width=137) (actual time=4.037..14.684 rows=10001 loops=1)
   Recheck Cond: ((name)::text ~~* '%ode%'::text)
   Filter: (application_id = 1)
   Heap Blocks: exact=193
   ->  Bitmap Index Scan on idx_gin_node_name  (cost=0.00..99.01 rows=10001 width=0) (actual time=3.993..3.993 rows=10001 loops=1)
         Index Cond: ((name)::text ~~* '%ode%'::text)
 Planning time: 0.793 ms
 Execution time: 15.226 ms
(8 rows)

Any insights on this?

@iegomez , you are in the back of my mind, I am really trying to figure out why it doesn't use the index.
If you could dump the table it would be amazing!

@siscia I'll send you the dump on wednesday, we are on national holidays right now.

@siscia You can download the table dump from https://expirebox.com/download/98168bdba97c57e91f5d6abae90df8bf.html. It will expire in 48 hours.

@iegomez  Get it :)
I don't make any promises!

@iegomez  I believe that the problem is quite "trivial"...
pg_tgrm is based on tri-grams, your query use a bi-gram and cannot use the index (seems a quite educate guess reading online, but honestly I am not so expert)
What I would suggest is to try the exact same query using a bigger (more than 2, so at least 3) char in the query.
Let's compare a query that use ilike with two chars:
postgres=# explain analyse
postgres-# select * from node
postgres-# where application_id = 1 and
postgres-# encode(dev_eui, 'hex') ilike '%10%' order by name;
                                                  QUERY PLAN
--------------------------------------------------------------------------------------------------------------
 Sort  (cost=477.69..482.69 rows=2000 width=161) (actual time=61.312..62.040 rows=543 loops=1)
   Sort Key: name
   Sort Method: quicksort  Memory: 169kB
   ->  Seq Scan on node  (cost=0.00..368.04 rows=2000 width=161) (actual time=0.137..57.276 rows=543 loops=1)
         Filter: ((application_id = 1) AND (encode(dev_eui, 'hex'::text) ~~* '%10%'::text))
         Rows Removed by Filter: 9459
 Planning time: 0.324 ms
 Execution time: 62.572 ms
(8 rows)

Time: 63,734 ms

And a similar query that use 3 char with ilike
postgres=# explain analyse
postgres-# select * from node
postgres-# where application_id = 1 and
postgres-# encode(dev_eui, 'hex') ilike '%103%' order by name;
                                                                  QUERY PLAN
-----------------------------------------------------------------------------------------------------------------------------------------------
 Sort  (cost=232.39..233.39 rows=400 width=161) (actual time=0.757..0.799 rows=43 loops=1)
   Sort Key: name
   Sort Method: quicksort  Memory: 36kB
   ->  Bitmap Heap Scan on node  (cost=15.10..215.10 rows=400 width=161) (actual time=0.084..0.496 rows=43 loops=1)
         Recheck Cond: (encode(dev_eui, 'hex'::text) ~~* '%103%'::text)
         Filter: (application_id = 1)
         Heap Blocks: exact=39
         ->  Bitmap Index Scan on trgm_encode_idx_node_dev_eui  (cost=0.00..15.00 rows=400 width=0) (actual time=0.045..0.045 rows=43 loops=1)
               Index Cond: (encode(dev_eui, 'hex'::text) ~~* '%103%'::text)
 Planning time: 0.273 ms
 Execution time: 0.967 ms
(11 rows)

You can see that with at least 3 chars the query use the index.
If it is possible, and reasonable, it could be worth to make a db call when there are at least 3 chars in query, or after that the user had waited a little bit. (It is a realtime search or press a button and then make the query?)

 You're totally right, @siscia, I tested it and it works fine, either on name or on encode(dev_eui, 'hex') (with text(dev_eui) it works fine too).
To address the case in question, I did some more tests using a multi-column index on name and dev_eui using gin:
create index multi_idx_node_name on node using gin (name gin_trgm_ops, encode(dev_eui, 'hex') gin_trgm_ops);
When running a query against dev_eui, it uses it:
explain analyse select * from node where application_id = 1 and encode(dev_eui, 'hex') ilike '%0202%';
                                                          QUERY PLAN
------------------------------------------------------------------------------------------------------------------------------
 Bitmap Heap Scan on node  (cost=20.62..171.59 rows=80 width=137) (actual time=0.116..0.157 rows=2 loops=1)
   Recheck Cond: (encode(dev_eui, 'hex'::text) ~~* '%0202%'::text)
   Rows Removed by Index Recheck: 3
   Filter: (application_id = 1)
   Heap Blocks: exact=5
   ->  Bitmap Index Scan on multi_idx_node_name  (cost=0.00..20.60 rows=80 width=0) (actual time=0.071..0.071 rows=5 loops=1)
         Index Cond: (encode(dev_eui, 'hex'::text) ~~* '%0202%'::text)
 Planning time: 0.573 ms
 Execution time: 0.257 ms

When running a query against name only it also uses the index:
explain analyse select * from node where application_id = 1 and name ilike '%ode%';
                                                             QUERY PLAN
-------------------------------------------------------------------------------------------------------------------------------------
 Bitmap Heap Scan on node  (cost=101.51..444.52 rows=10001 width=137) (actual time=5.928..37.951 rows=10001 loops=1)
   Recheck Cond: ((name)::text ~~* '%ode%'::text)
   Filter: (application_id = 1)
   Heap Blocks: exact=193
   ->  Bitmap Index Scan on multi_idx_node_name  (cost=0.00..99.01 rows=10001 width=0) (actual time=5.704..5.704 rows=10001 loops=1)
         Index Cond: ((name)::text ~~* '%ode%'::text)
 Planning time: 0.885 ms
 Execution time: 39.491 ms

This seems odd, as I thought that a multi-column index on (A, B) should work for both together and A independently, but not for B independently, rendering a unique index on A useless and one on B necessary. That doesn't appear to be the case here. Maybe the latest versions of Postgres have changed this behaviour, or maybe I was wrong all along. Do you know about this?
Finally, when querying with an OR condition, with seq scan enabled and disabled, the results were the following:
appserver=> SET enable_seqscan = OFF;
appserver=> explain analyse select * from node where application_id = 1 and (encode(dev_eui, 'hex') ilike '%0202%' or name ilike '%ode%');
                                                                QUERY PLAN
-------------------------------------------------------------------------------------------------------------------------------------------
 Bitmap Heap Scan on node  (cost=124.61..517.65 rows=10001 width=137) (actual time=3.571..27.501 rows=10002 loops=1)
   Recheck Cond: ((encode(dev_eui, 'hex'::text) ~~* '%0202%'::text) OR ((name)::text ~~* '%ode%'::text))
   Filter: (application_id = 1)
   Heap Blocks: exact=193
   ->  BitmapOr  (cost=124.61..124.61 rows=10002 width=0) (actual time=3.521..3.521 rows=0 loops=1)
         ->  Bitmap Index Scan on multi_idx_node_name  (cost=0.00..20.60 rows=80 width=0) (actual time=0.095..0.095 rows=5 loops=1)
               Index Cond: (encode(dev_eui, 'hex'::text) ~~* '%0202%'::text)
         ->  Bitmap Index Scan on multi_idx_node_name  (cost=0.00..99.01 rows=10001 width=0) (actual time=3.424..3.424 rows=10001 loops=1)
               Index Cond: ((name)::text ~~* '%ode%'::text)
 Planning time: 1.155 ms
 Execution time: 28.064 ms
(11 rows)

appserver=> SET enable_seqscan = ON;
appserver=> explain analyse select * from node where application_id = 1 and (encode(dev_eui, 'hex') ilike '%0202%' or name ilike '%ode%');
                                                          QUERY PLAN
------------------------------------------------------------------------------------------------------------------------------
 Seq Scan on node  (cost=0.00..393.04 rows=10001 width=137) (actual time=0.057..20.715 rows=10002 loops=1)
   Filter: ((application_id = 1) AND ((encode(dev_eui, 'hex'::text) ~~* '%0202%'::text) OR ((name)::text ~~* '%ode%'::text)))
 Planning time: 0.784 ms
 Execution time: 21.272 ms
(4 rows)

The query will not use the multi-column index when seq scan is on, but I found that to be related with the specific query string 'ode'. My test records all have 'node' in their name, so the index won't be used, but if I change it to match a small group of nodes (in particular, I have one called 'gw-simulation-node'), it uses the index whether enable_seqscan is on or off and  works great. The latter applies to name only queries too:
appserver=> explain analyse select * from node where application_id = 1 and (encode(dev_eui, 'hex') ilike '%0202%' or name ilike '%sim%');
                                                             QUERY PLAN
------------------------------------------------------------------------------------------------------------------------------------
 Bitmap Heap Scan on node  (cost=32.65..183.84 rows=81 width=137) (actual time=0.132..0.191 rows=2 loops=1)
   Recheck Cond: ((encode(dev_eui, 'hex'::text) ~~* '%0202%'::text) OR ((name)::text ~~* '%sim%'::text))
   Rows Removed by Index Recheck: 3
   Filter: (application_id = 1)
   Heap Blocks: exact=5
   ->  BitmapOr  (cost=32.65..32.65 rows=81 width=0) (actual time=0.087..0.087 rows=0 loops=1)
         ->  Bitmap Index Scan on multi_idx_node_name  (cost=0.00..20.60 rows=80 width=0) (actual time=0.070..0.070 rows=5 loops=1)
               Index Cond: (encode(dev_eui, 'hex'::text) ~~* '%0202%'::text)
         ->  Bitmap Index Scan on multi_idx_node_name  (cost=0.00..12.01 rows=1 width=0) (actual time=0.015..0.015 rows=1 loops=1)
               Index Cond: ((name)::text ~~* '%sim%'::text)
 Planning time: 0.938 ms
 Execution time: 0.303 ms
(12 rows)

So, to sum up, I think a multi-column index on name and encode(dev_eui, 'hex') using the pg_trgm gin is the way to go. I also agree to search only with 3 or more characters by default. Maybe doing auto search with >= 3 chars and explicit search (i.e., pressing a search button) for 1 or 2 chars is a good mix, though removing chars under the 3 threshold may be weird (should it do an immediate search? should it wait to see if the user was erasing the whole search input?). Anyway, @gillespilloudwyres, @brocaar, @siscia, what do you guys think?

@iegomez , don't worry, I felt very dumb too when I couldn't understand why with only 2 chars it didn't use the index :smile:
GIN stands for Generalized Inverted Index, when it is decided what we want to index that part is written along with the column where it appears.
Suppose you want to use trigram GIN index on the world lorawan  the first step is to create the trigram that are: __l, _lo, lor, ora, raw awa, wan, an_, n__.
At this point at each of these trigrams is associate the tuple the contains the world lorawan.
(Note how GIN and trigram are actually different piece of software, the trigram indexes uses the GIN abstraction)
In this scenario the composite index is even difficult to define, it is not the classical direct B+-Tree index and I could believe that the behaviour we are seeing is justified.
I would try to keep a consistent behaviour everywhere, and I would use a simple "Search" button to avoid query the DB when not really necessary.

Hi, sorry I haven't commented anymore, I was a bit busy. If I get any time in the next few days, I'll change the auto search behaviour to a explicit search button and then generate a PR.
Note that in order for search to work the extension pg_trgm must be created as a pre-step so that the indexes may be created, so the PR would include this on the DB creation part of the docs:
-- connect to db
 \c loraserver_as

 -- create the pg_trgm extension for search capabilities
 create extension pg_trgm;

Sadly, anyone upgrading from a past version could miss this, so I was thinking that maybe there should be a default fallback in the index migrations if the extension isn't found. On the other hand, normally lora-app-server would just crash and log a migration failure involving the index creation, so that could be enough warning to get users to create the extension. Any thoughts?

@iegomez , sorry I got now the notification and I remembered of the issues about the extension.
Are you sure that it need to be create together with the database and it cannot be a simple new migration?

@siscia You need superuser privileges to create the extension, so unless you grant your appserver user with superuser privileges, it needs to be created earlier.

Thanks to @iegomez we almost have the search in place :-) 👍

This feature has been merged into master and will be included into the next release.

Thanks again @siscia and @iegomez. I might actually consider using the pg_trgm now. I'm currently working on a global search page (e.g. type in a deveui), to search trough all accessible organizations, applications, devices and gateways.
The advantage of the pg_trgm extensions is that it also provides functions to calculate the similarity between two arguments (https://www.postgresql.org/docs/9.6/static/pgtrgm.html), which is useful to order the search results.

What do you think?

I like it, and pg_trgm is really easy to use for that kind of ordering.
Have you thought of the best way to introduce this dependency (because of the superuser restriction)?

I think there is no other option than to document it, next to the create database commands + add a warning in the changelog that the extension needs to be enabled before upgrading.
Unfortunately it is not possible to do it using the migrations as they will fail when the user does not have the needed permissions. Even create extension if not exists pg_trgm fails when already enabled (else it would be possible to include this in the migrations).

thanks a lot, the deveui search is very helpful and now I can find easily all of my devices !


[WIP] Moving entities into content state #185

This is just a very quick, hacky PR to showcase the general direction of how I want to approach things for #185. I mainly want to gather some feedback so we can discuss whether we want to keep DraftEntity and just use that to perform transactions on the state or whether we want to go a different route.
I have updated most of the tests to pass. One is failing due to me not having implemented DraftEntity.mergeData and two other ones are failing because they rely on an implementation which requires a contentState argument and I didn't refactor anything there yet.
I'm totally down to refactor anything here or to go a different direction. Just wanted to get the ball rolling. But one way or another: this is going to be a breaking change ;)

Status update possible? Is there anyway we can help?
Thanks for taking the initiative!

@amireh I am waiting for initial feedback from @hellendag regarding this.

Thanks for your patience. :)
I think this is on the right track. A couple of comments here. Mostly I'm thinking we can just move everything directly into ContentState and throw out the DraftEntity module altogether.

@hellendag I was on vacation last week. I hope I'll be able to take a look at this soon! :)

Hey @johanneslumpe and @hellendag, this is awesome and in my opinion one of the more important things that need to be implemented atm. Not having entity state changes refresh the content state is a big pain point that gave me loads of headaches while developing plugins for draft-js-plugins, at the point were I change both component state and entity state to force atomic editor components to rerender, since forcing a rerender by setting selection doesn't work reliably.
Is there anything you need help with? Has there been any further discussions about this PR internally @hellendag?

@bkniffler Unfortunately I still didn't have time to revisit this - got some personal things to deal with right now. I will get back on this eventually though!

@hellendag the only thing left to fix is the DraftPasteProcessor or rather convertFromHTMLtoContentBlocks, which creates LINK entities. In order to do that I think we either have to give it access to the content state. But then again we won't be generating chunks/fragments but rather a new/updated contentState whenever we parse HTML. There is only a single call site to convertFromHTMLToContentBlocks though inside the DraftPasteProcessor and that again is only called in the editOnPaste handler:
if (html) {
      var htmlFragment = DraftPasteProcessor.processHTML(
        html,
        this.props.blockRenderMap
      );
      if (htmlFragment) {
        var htmlMap = BlockMapBuilder.createFromArray(htmlFragment);
        this.update(insertFragment(this.props.editorState, htmlMap));
        return;
      }
    }

So instead of having a fragment here, we could just return a new contentState and then call this.update with the new state. What are your thoughts on this? Do you see any issues with that?

@hellendag I went ahead and committed 012ea41 - it's one way to do it. we're generating additional objects now when parsing html now, but I'm not sure that that's a huge issue. Let's discuss what you think about it :)

I like this a whole lot and I think it's really close.

@johanneslumpe Whats the current state? Is there anything missing or is it ready for merge? I tried your branch on a local project and didn't encounter any issues.

@bkniffler I asked some questions to @hellendag and I've been waiting on answers since. This actually works well as of now. There were potential changes to src/model/decorators/DraftDecoratorType.js but I haven't gone ahead with those because I wanted to discuss them first.
But this is working!

We're going to get this merged today. Taking another pass now.

Okay, let's go ahead and do this, then refine as needed.
Thanks again for your patience! :)


Mention Jetbrains support in Readme

Jetbrains have offered support for our project. We should also support them by adding a link to them in Readme.

Yes, 100%. 👍
Thanks for chasing!
Will raise a PR soon!

Available in version 1.3.10 and higher.


Should expose scrollingContainer

scrollingcontainer
Necessary to fix scroll jumping bugs when Quill is set to auto grow its height, and another ancestor container is responsible from the scrolling.

Would love for this!

Finally i find the answer form your issue！


mhmapspotter database duplicates mice table

mhmapspotter has a mice table, mhhunthelper has a mice table. Both have an id column and both have generated ids so they may not match. The mouse names should match. Proposal - use the mhhunthelper mice table.
Updates required:

mapping needs to be created. mice already has mhhh_id, we can make sure this is populated for all mice.
update map_mice to set mouse_id to the mice.mhhh_id instead of mice.id
update aggregation event to use the mhhunthelper.mice table
update PHPs (mapintake, searchbyitem) to use mhhunthelper.mice table
drop/rename mice table

just fyi, this is blocked by the #119 since the map spotter sucked in a whole bunch of scavenger map items, and I don't want to accidentally suck them into mhhh db mouse table while testing/fixing that ticket. @AardWolf fyi


Add side event information of Ground floor

Issue

close #644

Overview (Required)

Fixed the display of event information on the Ground Floor. The data shown is provisional.

Links

Screenshot

Before
After

Thank you for your contribution! LGTM!


[HOLD for payment 2024-08-02] [$250] mweb-LHN-Mark as unread a message after deleting self mention displays green dot in LHN.

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

Version Number: 1.4.63-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail: n/a
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: applause internal team
Slack conversation:
Action Performed:

Go to https://staging.new.expensify.com/
Open self DM report
Mention self and send it
Send another message
Delete the previous DM message
Navigate to LHN
Long press the report in LHN and select mark as unread

Expected Result:
Mark as unread a message after deleting self mention must not display green dot in LHN.
Actual Result:
Mark as unread a message after deleting self mention displays green dot in LHN.
Workaround:
unknown
Platforms:

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

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/77db8ab9-2874-4a85-8b47-1e8fa36d5a98

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e421f5653f9e5c16
Upwork Job ID:<PHONE_NUMBER>128035328
Last Price Increase: 2024-04-24
Automatic offers:

hungvu193 | Reviewer | 0
tienifr | Contributor | 0

Issue OwnerCurrent Issue Owner: @twisterdotcom

@twisterdotcom FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think this bug might be related to #vip-vsb

A mobile web bug for an uncommon flow, this is niche.

I'll review this one tomorrow.

@tienifr 's main solution here looks good to me! 🎀 👀 🎀  C+ reviewed.

Bump @neil-marcellini

Bump @neil-marcellini

Sorry somehow I lost track of this one. Looking now.

@tienifr 's main solution here looks good to me! 🎀 👀 🎀 C+ reviewed.

I agree, let's do it. Let's also use the plan in 3.b

3. b. Or look for the latest mention report action that mentions this user, and optimistically set the lastMentionedTime to that one instead. If there's no such mention report action, set lastMentionedTime to null/empty

@tienifr please let me know via DM if backend changes are needed too, it sounds like they probably are.

Still working on the PR.

I'll try to squeeze this in today

I got a draft PR up, still a lot of work to do though

I got an automated test passing in the backend for this case. My next steps are to test manually with the contributor's App PR. After that I want to add an automated test to make sure it works when there are multiple mentions. Then I want to manually test some more complex cases in a report with multiple users and varying mentions. If I find real problems there I will make the backend handle that too.
Right now it's only update the lastMentionedTime for the user who deletes the comment, but I'm pretty sure deleting a comment could effect all users mentioned in it.

Notes to self:
Manually testing this exact case in app worked well and it's fixed, even without a frontend PR. I tested some more complicated cases like mentioning another user in a public report and I verified that it causes the green dot to show incorrectly. I also encountered an error with OpenReport after these cases, and fixed that.
Next time I can start writing some automated tests for these more involved cases, then work on a query to make it all work. I'm thinking I won't send the Onyx updates until a user marks it as unread, but I will save the data right away for consistency.

Yesterday I wrote a test for a slightly more complicated case and updated the query to set the last mentioned time for all users in the mention. Finding the last read time and doing the update now happens in one query. I'm also changing it to only send the Onyx update when a user marks the report as unread, because that's the only flow where it's used in App right now and OpenReport will update lastMentionedTime as well.
Before I write more tests I should get the query timed to make sure it's going to be fast enough. I'm concerned that it will not be since I think it involves scanning over all report actions. If it's too slow we might need a totally different approach towards mention data storage.

How is this going @neil-marcellini?

I've been concerned that the query to fix this on the backend won't scale, so I brought that up in #engineering-chat today. After some discussion we decided that the only way to make this efficient is to change the storage format for mentions, and agreed that it's not worth doing so to fix this edge case. So we're going to do nothing on the backend.
However, I think we can still go ahead with the frontend fix because it looks like it solves this exact case and only has a bug in an even more extreme edge case described here.

@tienifr @hungvu193 would you please get the conflicts resolved, retest the PR, and get it ready to merge?

@neil-marcellini Conflicts resolved, can you take a look at the PR please?

Taking a look!

Turns out it's a backend bug though, and we already said above we're not going to fix that. Hopefully we can get this merged soon.

The PR was merged today, finally!

[x] The PR that introduced the bug has been identified. Link to the PR: I can't really find the offending PR as this is an extremely edge case. See this comment.
[x] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
[x] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x] Determine if we should create a regression test for this bug: Yes

Regression Test:

Open App.
Open self DM report.
Mention yourself and send it.
Send another message.
Delete the previous DM message.
Navigate to LHN.
Long press the report in LHN and select mark as unread.
Verify mark as unread a message after deleting self mention must not display green dot in LHN.

Payment Summary:

@hungvu193 paid $250 via NewDot Manual Request (Reviewer)
@tienifr paid $250 via NewDot Manual Request (Contributor)

$250 approved for @hungvu193

$250 approved for @tienifr


Tracking time spent on each page of the pdf

Is it possible to track the time spent on each page of a pdf by the user?

Hi @indirapimpalkhare
sorry for my late reply.
To answer your question: It depends on how you define "spent time".


Update schema for schema viz

Remove "Biospecimen" as a dependency for "ImagingLevel2" (so the logic becomes: "ImagingLevel2" depends on "ImagingLevel1", and "ImagingLevel1" depends on "Biospecimen")

Thanks @linglp. For some reason this is showing in the diff as a complete replacement of the csv and the action to generate the JSON-LD is failing - can you recommit to just have the single line change required in the ImagingLevel2 component DependsOn in the csv. The action will then generate the JSON-LD.

Thanks @linglp!
We have a few other PRs open waiting approval or ready to merge.
@elv-sb it probably makes sense to try and get these into the data model in sequence and hopefully minimise conflicts along the way.

Related to the issue here


v4l2loopback feature like for windows user

Anyone want to make a v4l2loopback feature for windows user like me, for the reason that I wanted to use this feature on OBS like on linux. Hoping soo much for this feature.

Also I found this Issue #776, and your reply to this
ref #662
https://github.com/CatxFish/obs-virtual-cam
Originally posted by @rom1v in https://github.com/Genymobile/scrcpy/issues/776#issuecomment-527134659
is unavailable.

I also want to be able to do this on Windows.


IPython console execution lag related to graphical stuff and Anaconda?

From<EMAIL_ADDRESS>on 2013-10-18T14:41:32Z
Spyder Version:  2.2.5
Python Version:  2.7.5
Qt Version    :  4.8.5, PySide 1.2.1 on Linux
pyflakes >=0.5.0:  0.7.3 (OK)
pep8 >=0.6      :  1.4.6 (OK)
IPython >=0.13  :  1.0.0 (OK)
rope >=0.9.2    :  0.9.4 (OK)
sphinx >=0.6.6  :  1.1.3 (OK)
matplotlib >=1.0:  1.3.0 (OK)
sympy >=0.7.0   :  0.7.3 (OK)
pylint >=0.25   :  None (NOK)
Ever since I started using Spyder via Anaconda, Spyder has been acting really strangely.  As far as I can determine, I think it is related to graphics (PySide)...  can't reproduce the problem unless I make a figure or something.  I've copy pasted an example bugged session at the end of this description.
The problem is an apparent lag between submitting a command in the console and having the command evaluated.  The example below demonstrates that I have to press enter many times before the value of x is finally printed.  Similarly, had I tried to launch a new figure during the laggy part, no figure would appear until I presed enter a bunch of times.  Waiting (and not pressing enter a bunch of times) does nothing.
This bug only occurs in the regular Python console.  Other things seem wrong in the IPython console, but it could just be my unfamiliarity with it (I've never tried the IPython console until now).
The bug does still occur when using F9 to run a selection.  Also occurs using F5 to runfile.
I can find no helpful information in any sort of logs or other output (is there a flag I can use to enable more verbose messages?).  I've tried uninstalling then reinstalling Spyder.  Same deal for Anaconda.  I've tried deleting the .spyder2 folder, it didn't help.  I've long since removed any trace of the old Spyder that was on my system prior to Anaconda.  I'm currently in the process of removing anything python related that is not from Anaconda... so perhaps it is some old package sneaking in and causing issues?
If I disable the option "Install Spyder's input hook for Qt", the problem goes away.  However, plots no longer work.  The figure windows appear, but are totally empty (no plot buttons, no axes, just grey except for the titlebar and window boarder).
My operating system is OpenSUSE 12.3
Python 2.7.5 |Anaconda 1.7.0 (64-bit)| (default, Jun 28 2013, 22:10:09)
[GCC 4.1.2 20080704 (Red Hat 4.1.2-54)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
Imported NumPy 1.7.1, SciPy 0.12.0, Matplotlib 1.3.0
Type "scientific" for more details.
>>> figure();plot(arange(5))
<matplotlib.figure.Figure object at 0x2732d90>
[<matplotlib.lines.Line2D object at 0x3ad1690>]
>>> x=5
>>> x
5
>>> x
5
>>> x
5
>>> x
5
>>> x
5
>>> x
5
>>> xx
Traceback (most recent call last):
File "", line 1, in <module>
NameError: name 'xx' is not defined
>>> x
5
>>> x
5
>>> x
5
>>> x
5
>>> x
x5
>>>
x>>>
x>>> x
5
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> x
>>>
>>>
>>>
>>>
>>>
5
>>>
>>>
>>>
>>>
>>> x
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
5
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1611

From<EMAIL_ADDRESS>on 2013-10-18T12:58:58Z
I just tried removing the Spyder that came with Anaconda and installed 2.3.0beta1.  Same result.

From<EMAIL_ADDRESS>on 2013-10-18T12:54:43Z
I just removed the .spyder2 folder, uninstalled Anaconda, deleted everything relating to older Python installations, reinstalled Anaconda.  Nothing but my own code is present in the PYTHONPATH.  As before, IPython figures don't work unless I have the Qt hook option enabled.  And with that enabled, I quickly encounter the strange lag issue.
I don't know if it is related, but I've also been encountering multiple unrecoverable hangs per day.  Though this seems to have more to do with the editor and perhaps the clipboard...
Is anyone else using Spyder via Anaconda?

From ccordoba12 on 2013-10-18T14:57:29Z
If you are on OpenSuse, are you running Spyder under KDE by any chance?

From<EMAIL_ADDRESS>on 2013-10-21T06:11:29Z
Yes, KDE

From<EMAIL_ADDRESS>on 2013-10-24T06:30:43Z
I have the same issue (random lag apparently related to Qt; it usually works fine the first time I run the programm though) with Spyder 2.3.0beta1 running on python 3.3 with Anaconda.
I tried with WinPython <IP_ADDRESS> (running Spyder 2.3.0dev6 with Python3) and I have the same problem.
But I have no problem with Spyder 2.24 running with Anconda on Python 2.7.
I am running windows 7-64bits.

From<EMAIL_ADDRESS>on 2013-10-21T08:09:11Z
Well, that was enough of a hint to find a post of yours on the Anaconda message boards: https://groups.google.com/a/continuum.io/forum/#!msg/anaconda/YzzgzXq3QHo/Iu6ZfakojH4J I tried updating to qt 4.8.5 (was easier than downgrading to 4.7.4, as someone else had suggested)... however there was no improvement.  I removed Anaconda's PySide and reinstalled it via Yast, still no improvement.
I briefly considered downgrading to 4.7.4, but it would have broken way too many dependencies.

From<EMAIL_ADDRESS>on 2013-10-25T06:22:46Z
Oops, just noticed the title is wrong.  to be clear, problem occurs in the regular Python console.  IPython console does not have the problem (and can be viewed as a workaround).

From<EMAIL_ADDRESS>on 2013-10-31T18:15:15Z
I am experiencing the same thing on spyder2.3.0dev6, running python3 in win7 64 bit.

From<EMAIL_ADDRESS>on 2013-12-03T14:20:55Z
updated Anaconda to 1.8.0 64-bit, problem remains (Spyder version 2.2.5 at the moment).

From<EMAIL_ADDRESS>on 2014-10-17T02:23:26Z
I have exactly the same problem in two systems of mine, both running Ubuntu 64bit. One is using Gnome3 and the other is on a virtual machine that is being used only through X-forwarding (so the question of KDE vs Gnome seems to be irrelevant to this bug).  Using Spyder 2.3.1.
Also, why is the status "Won't fix"? And what does "Hope this is finally solved in the next Anaconda release" mean? Did we figure out the cause?

From ccordoba12 on 2014-05-05T06:11:22Z
Hope this is finally solved in the next Anaconda release
Status: WontFix


look up additional metadata about journal for a submission during submission workflow

We need to get these things from NIH's eUtils api:

ISSN's pub-type
Journal's NLMTA value

Given an ISSN, this information can be acquired by first hitting the below search end point and capture the <id> field value
https://eutils.ncbi.nlm.nih.gov/entrez/eutils/esearch.fcgi?db=nlmcatalog&term=0006-2952[issn]
Given the above <id> value, use this end point to look up associated ISSNs, their pub-type and NLMTA value for that journal
https://eutils.ncbi.nlm.nih.gov/entrez/eutils/esummary.fcgi?db=nlmcatalog&retmode=json&rettype=abstract&id=101032
(note that we're only interested in print and electronic issn pubtype and NLMTA is captured as "medlineta" in the request's response. )

the above information need to be presented in the metadata blob of the submission for the deposit service to consume. the fields name should be nlmta, ISSN and pub-type. some data structure might be needed to capture the combo of ISSN and its associated pub-type in the case of multiple ISSNs. Please sketch out the intended locations/names of the fields in the blob ahead of time to give deposit service folks time to implement the parsing.
Bonus point for being able to display these fields during the workflow, but it's not critical

First link will be interesting: &term=<your-issn-here>[issn] number needs to be labeled as ISSN in square brackets


Obfuscate script output when running scenario.

Describe the issue
When we run a scenario, the output displays the commands for setting up the scenario, which gives excess information to the user, and could help them shortcut the scenario.
To Reproduce
Run any scenario and observe the output
Expected behavior
When running a scenario, there should be no information displayed which could unintentionally give information to help solve the scenario.

This needs more details as to the issue this is addressing

@s-irvine this looks like it can be closed, based upon the changes you made to perturb's output


Consider setting AddAlternateKeyPaths to true in the conversion settings

APIs such as List members of a directory role now support alternate keys:
GET /directoryRoles/{role-id}/members
GET /directoryRoles(roleTemplateId='{roleTemplateId}')/members

We should consider setting AddAlternateKeyPaths to true in the CSDL to OpenAPI conversion settings at https://github.com/microsoftgraph/msgraph-metadata/tree/master/conversion-settings to support the alternate key in our SDKs.
As part of this change, we should monitor the net effect of this setting on the growth of the number of paths.

I will do a path count diff of the two files with the setting true and false and share the results here.

/cc @baywet Does Kiota currently handle this kind of alternate path syntax?
@irvinesunday  I think this issue should be transferred over to the OpenAPI.OData  repo.

@darrelmiller yes, you'll get a directoryRolesByRoleTemplateId method in the fluent API surface.
Since we already have the setting in the conversion lib is merely a matter of updating the configuration files here, no action is needed in the conversion lib unless I missed something. So I'd argue against moving this issue.

@irvinesunday how did this go?

Setting AddAlternateKeyPaths increases the paths for v1.0 from 6237 - 6241 paths. This is quite an insignificant increase.


'constructor' not allowed as a variable name

To reproduce: type
let constructor = 3

in a notebook (e.g. http://andrewray.github.io/iocamljs/min.html).  The cell freezes after the 'o'.

Confirmed.  Seems to be a problem with the codemirror editor as it can be also seen here;
http://codemirror.net/mode/mllike/index.html
codemirror/CodeMirror#3106

Fixed in repo and online.

Thanks!


run_pending_github_builds on launch doesn't seem to work

Since https://github.com/fastlane/ci/pull/381

It's fixed 🎉


Timeout without a "Zappa event"

Context
When I send a request to my Zappa page I get an Internal server error, but my logs simply say:
[1553875874136] Instancing..
[1553875879150] 2019-03-29T16:11:19.138Z b3b3a6b7-6ee5-4ea6-a5f3-f811264d1ddd Task timed out after 5.00 seconds

Your Environment

Zappa version used: 0.48.2
Operating System and Python version: Docker image python:3.6 (Python 3.6, I believe stretch)
The output of pip freeze: My requirements file:

django==2.1.7
django-crispy-forms==1.7.2
django-recaptcha==2.0.2
Pillow==5.4.1
django-cleanup==3.1.0
dj-database-url==0.5.0
psycopg2-binary==2.7.7
sentry-sdk==0.7.4
libsass==0.17.0
django-compressor==2.2
django-sass-processor==0.7.2
requests==2.21.0
django-ipware==2.1.0
django-maintenance-mode==0.13.0
django-constance[database]==2.3.1
django-grappelli==2.12.2
django-storages==1.7.1
collectfast==0.6.2

Your zappa_settings.json:

{
  "staging": {
    "django_settings": "orangetools.settings",
    "profile_name": null,
    "project_name": "orange-tools",
    "runtime": "python3.6",
    "s3_bucket": "zappa-pe5li99w9",
    "aws_region": "eu-west-2",
    "exclude": [
    ],
    "memory_size": 256,
    "timeout_seconds": 5,
    "domain": "staging.orangetools.xyz",
    "keep_warm": false
  }
}

This used to work fine but for the last day, I have been having this issue and have tried to redeploy it but still having issues.

I would try removing the exclude and profile_name and try again. Also double-check that the permission role the function is using can access the database (lack of permissions will cause the timeouts when trying to access the database).

Still failed with those changes and I haven't touched my permissions. If it's the database then that wouldn't really make sense since right now it's a public URL.

I just undeployed and deployed, still getting Internal Server Error and no Zappa Event showing in the logs


Release '0.1.0'

Signed-off-by: Yevhen Vydolob<EMAIL_ADDRESS>

Codecov Report

Merging #279 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #279   +/-   ##
=======================================
  Coverage   69.86%   69.86%
=======================================
  Files          42       42
  Lines        3106     3106
  Branches      570      570
=======================================
  Hits         2170     2170
  Misses        936      936

Continue to review full report at Codecov.

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


implement ThemeProvider

Closes #192

PR-Codex overview
Detailed summary

Deleted useTheme.ts file.
Added ThemeProvider.tsx file.
Updated main.tsx file to use ThemeProvider component.
Updated ThemeSwitcher.tsx file to use useThemeContext instead of useTheme.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Codecov Report
Patch and project coverage have no change.

Comparison is base (9201c67) 61.89% compared to head (8828dbd) 61.89%.

:exclamation: Current head 8828dbd differs from pull request most recent head a1ca5f3. Consider uploading reports for the commit a1ca5f3 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff            @@
##             master     #200   +/-   ##
=========================================
  Coverage     61.89%   61.89%
  Complexity      207      207
=========================================
  Files           100      100
  Lines          1774     1774
  Branches        118      118
=========================================
  Hits           1098     1098
  Misses          567      567
  Partials        109      109

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


auth: more verbose errors

The failure on master is caused by etcd using API from go1.5 - os.LookupEnv, needs to be pinned to 1.4-compatible release,

👍


Synchronous grpc - blocking or polling?

I was wondering if grpc calls implement thread blocking or polling.
Assume a situation where a client makes a synchronous grpc call to a server, then will the thread making this call go to a "blocked" state while waiting for the response from the server (and might then be woken up by some underlying network kernel thread), or does it continuously poll for the response?

As per the "view from above" our API, the thread that makes the synchronous call is blocked.
Under the hood, gRPC is free to implement that as it likes. We could make the calling thread be the polling thread, which would involve blocking on a syscall like epoll_wait. Or we could have some internal polling thread for that, which would wake up the user thread (waiting on a gpr_mu) when the right bytes come off the wire.
The reason I am leaving this vague is because this system has been changing a lot recently.


Unregister bespoke-theme-fancy

Hi,
I'd like to know if you can unregister the bespoke-theme-fancy package for me.
I actually registered it with a wrong repo URL ('https://github.com/markdalgleish/generator-bespokeplugin'), but it should have been https://github.com/fegemo/bespoke-theme-fancy.git.
If I try to unregister the package, I get bower EFORBIDDEN    Must have push access to view repository collaborators.
Thanks and sorry for the inconvenience.

@markdalgleish Do we have your permission to unregister bespoke-theme-fancy?

Just unregistered it for you.

Thanks, Mark. Sorry for the trouble :smile:


Question on List of terms from older APIs (Rate)

[Enter feedback here]
Hi,
In "List of terms from older APIs", in "Old term", there is a reference to "Rate".
I don't find that field in my report generated from Consumption Usage Details API.
In the other side, I have ResourceRate part of that report and I have not found any migration rule for this term.
Is it replaced by EffectivePrioce?
If no, can you detail which field is replacing ResourceRate in new API?

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

ID: d79af6b6-535b-fe98-2bf4-670b0cc1e71f
Version Independent ID: e75818d6-3f51-dd56-04b4-a92a523906d5
Content: Understand usage details fields - Microsoft Cost Management
Content Source: articles/cost-management-billing/automate/understand-usage-details-fields.md
Service: cost-management-billing
Sub-service: cost-management
GitHub Login: @bandersmsft
Microsoft Alias: banders

@ANS-salee
Thanks for your feedback! We will investigate and update as appropriate.

@ANS-salee Are you using any of the older APIs? The Cost Management engineering team wants me to remove the List of terms from older APIs section from the article so that readers won't get confused by it.

@ANS-salee - Can you please provide the requested information as requested by @bandersmsft ?


Closes #5812 add underline on event name on mouse hover

Fixes Issue
Adds a visual effect on hover
Closes #5812
Changes proposed

Add an underline on the Event name when mouse hover

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots
Before Change
https://user-images.githubusercontent.com/116902573/229454158-175b1bd9-5db7-456f-a875-f325707c7e65.png
After Change
https://user-images.githubusercontent.com/72187009/229775180-09d0b1f0-188c-42b9-af42-36fc8523143a.png
Note to reviewers
It's just a simple change that adds an underline to the event name when mouse is hovered over it

Yes, I am sorry but I made some mistake while pushing the code and some other code was pushed. But I have made a new PR with the requested changes. Sorry for the inconvenience

ok no problem 👍


Unique Wither Skeleton Entity

allows it to work properly in powered spawner, plus it's really annoying

Please add a config option to disable this.

It checks for the config before registering the handler
https://github.com/hilburn/EnderIO/commit/77a9c0f42ac7162738155269ad6dfb9b7840ca92#diff-be8d31552bdfd3b446a181f80c4e724bL565

Right. Derp.


fix: Update Dockerfile

Added apk packages to allow node-gyp to build with python

Sure. I was trying to run the Portal as described in the Readme and it failed due to missing python package in the used base image.
After installing the package in the image, the build went fine.


Improvement for IPv6 Limitation Information/Documentation for AppGw

Hello Team.
Is it possible to improve the current documentation which clarifies the IPv6 routing limitation on AppGw?
We should have a section on the AppGw Microsoft Docs which shows all the "Limitations" for this product, specially with IPv6.
Today we only have the IPv6 information like a FAQ:
https://learn.microsoft.com/en-us/azure/application-gateway/application-gateway-faq#does-application-gateway-support-ipv6

Examples: For Azure Load Balancer we have a section "Limitation" for a lot of topics and I think this make the information clearer and organized to the customer and even to the Support/Customer Engineer which would help the customer.
https://learn.microsoft.com/en-us/azure/load-balancer/skus
https://learn.microsoft.com/en-us/azure/load-balancer/cross-region-overview
Thank you for checking on that.
Aline Dutra

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

ID: 88466307-81b8-1e5b-7f2c-191d8e818071
Version Independent ID: 08ad49f3-6a57-b925-794a-37215e985aba
Content: Frequently asked questions about Azure Application Gateway
Content Source: articles/application-gateway/application-gateway-faq.yml
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@alinedutrasilva
Thanks for your feedback! We will investigate and update as appropriate.

@alinedutrasilva
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

This is being tracked as internal content issue: 91197
#please-close


no need Conv2d bias for Conv2d+BN module.

In REBNCONV module, I find Con2d+BN combination. If BN is used after Conv2d, Conv2d bias is unnecessary. We can set bias=False in Conv2d.
https://github.com/xuebinqin/U-2-Net/blob/71aa5a8bef19967ac0def9276bd0ef6cfb4273d8/model/u2net.py#L5-L18

Hi, WuZhe, thanks for your suggestion. you could fork and create your own
branch and try that out. Currently, we probably keep that there, otherwise
the pretrained weights are not able to be loaded correctly. In addition,
without further experiments we are not sure if diable bias makes sense.
Usually pure theoretical equalization doesn't mean practically
effective, especially in deep learning.
On Fri, Jul 22, 2022 at 2:07 AM WuZhe @.***> wrote:

In REBNCONV module, I find Con2d+BN combination. If BN is used after
Conv2d, Conv2d bias is unnecessary. We can set bias=False in Conv2d.
https://github.com/xuebinqin/U-2-Net/blob/71aa5a8bef19967ac0def9276bd0ef6cfb4273d8/model/u2net.py#L5-L18
—
Reply to this email directly, view it on GitHub
https://github.com/xuebinqin/U-2-Net/issues/323, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSGORJRIO5YLLMRIMIPNXTVVJQDZANCNFSM54KUB6GQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Xuebin Qin
PhD
Department of Computing Science
University of Alberta, Edmonton, AB, Canada
Homepage: https://xuebinqin.github.io/


add anyOS flag to force showing details for all platforms

addresses issues #118 and #96

You can show details regardless of the platform/os on which you are running Invoke-AtomicRedTeam as follows:
Invoke-AtomicTest T1016 -ShowDetails -anyOS Invoke-AtomicTest T1016 -ShowDetailsBrief -anyOS


Start influxdb instance in default recipe

Otherwise calls to influxdb_* LWRPs will fail because influxdb isn't running yet.

So I agree with @tjwallace that influx should start by default and those that don't want that behavior should roll this up in a wrapper cookbook and just use the custom resources.  I'd like to see this change in the next release as long as @bdangit agrees and you can fix up the conflicts.

this is already done on default.rb master branch.
https://github.com/bdangit/chef-influxdb/blob/master/recipes/default.rb#L57
reopen if it's not the case.


Fix issue #1309 | Easy Issue 4 (C++)

Pull Request Type
What kind of change does this PR introduce?

Solved an issue/task

What is the current behavior?

Issue Number: https://github.com/fork-commit-merge/fork-commit-merge/issues/1309
What is the new behavior?

C++ program that reverses a given string and prints it out to the console.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


rwx volume: use softerr instead

Commented out softerr accidentally.
Longhorn/longhorn#6655

@Mergify backport v1.4.x v1.5.x

what's the default value here? I thought @roger-ryao already closed those backport issues .. :thinking: We need to reopen them as well and see if test steps need to be updated.

Wait...it seems my environment issue.
False alarm, sorry.


KAFKA-8488: avoid kafka array list grow

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
see https://issues.apache.org/jira/browse/KAFKA-8488
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

oneliner, no tests ;)


fix: discard role attributes that require superuser from dump

What kind of change does this PR introduce?
Bug fix https://github.com/supabase/cli/issues/933
What is the current behavior?
Roles with nosuperuser and noreplication attribute cannot be restored as non-superuser.
What is the new behavior?
Discard these role attributes from pg_dump as these the are default values anyway.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 62.935%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4486

Relevant Lines:
7128

💛  - Coveralls


Rebase project to variable font?

I wonder if there is a plan to rebse this project to variable font version of Source Han Sans?

There should be a plan. But the developer is busy with something else.


Axios Error 504

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
After Installing on my system using WSL Ubuntu and Helm, starting the Cluster with minikube and trying to access via kubectl port forwarding I get the following message:
AxiosError: Request failed with status code 504
I noticed following problem when running kubectl get pods:
NAME                                                 READY STATUS     RESTARTS     AGE
devlake-grafana-66fcd84bcb-wjsgn   0/1      Running   0                   4m21s
devlake-lake-7b6f7d647f-jdjvk           0/1      Running   1 (55s ago)   4m21s
After reinstalling:
NAME                                                READY STATUS   RESTARTS     AGE
devlake-lake-7455659445-8nh8v      0/1     Running   4 (61s ago)   5m1s
and a few minutes later:
NAME                                                READY STATUS                   RESTARTS           AGE
devlake-lake-7455659445-8nh8v      0/1     CrashLoopBackOff   6 (3m31s ago)   12m
The other devlake service run fine, so maybe that could be part of the problem.
I tried to reinstall but it didn't help.

What do you expect to happen
Open Apache DevLake UI as usual.
How to reproduce
Install as documented on WSL Ubuntu with Helm.
Anything else
Happens everytime.
Version
v1.0-beta1
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please share the pod log for diagnosis.

Here are the logs:
ubuntu@dds-t14-007:~$ kubectl logs devlake-lake-7b6f7d647f-jdjvk
Defaulted container "lake" out of: lake, waiting-database-ready (init)
time="2024-07-02T08:36:58Z" level=info msg="no [.env] file, devlake will read configuration from environment, please make sure you have set correct environment variable."
Version:  v1.0-beta1@60faf14
time="2024-07-02 08:36:58" level=info msg="migration initialized"
time="2024-07-02 08:36:58" level=info msg="plugin loaded ae"
time="2024-07-02 08:36:59" level=info msg="plugin loaded azuredevops_go"
time="2024-07-02 08:36:59" level=info msg="plugin loaded bamboo"
time="2024-07-02 08:36:59" level=info msg="plugin loaded bitbucket"
time="2024-07-02 08:36:59" level=info msg="plugin loaded bitbucket_server"
time="2024-07-02 08:36:59" level=info msg="plugin loaded circleci"
time="2024-07-02 08:36:59" level=info msg="plugin loaded customize"
time="2024-07-02 08:36:59" level=info msg="plugin loaded dbt"
time="2024-07-02 08:37:01" level=info msg="plugin loaded dora"
time="2024-07-02 08:37:01" level=info msg="plugin loaded feishu"
time="2024-07-02 08:37:02" level=info msg="plugin loaded gitee"_
ubuntu@dds-t14-007:~$ kubectl logs devlake-ui-6c9777d847-xw6f2
2024/07/02 08:30:45 [notice] 10#10: using the "epoll" event method
2024/07/02 08:30:45 [notice] 10#10: nginx/1.25.4
2024/07/02 08:30:45 [notice] 10#10: built by gcc 12.2.0 (Debian 12.2.0-14)
2024/07/02 08:30:45 [notice] 10#10: OS: Linux 5.10.207
2024/07/02 08:30:45 [notice] 10#10: getrlimit(RLIMIT_NOFILE): 1048576:1048576
2024/07/02 08:30:45 [notice] 10#10: start worker processes
2024/07/02 08:30:45 [notice] 10#10: start worker process 11
2024/07/02 08:30:45 [notice] 10#10: start worker process 12
<IP_ADDRESS> - - [02/Jul/2024:08:30:50 +0000] "GET /health/ HTTP/1.1" 200 0 "-" "kube-probe/1.30" "-"

The log looks ok, seems like a k8s problem, please check your setup.


Allows integration tests to skip verifying exception

Codecov Report

Merging #3476 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3476      +/-   ##
==========================================
- Coverage   40.13%   40.12%   -0.02%
==========================================
  Files        2175     2175
  Lines       71136    71134       -2
  Branches     9803     9802       -1
==========================================
- Hits        28549    28540       -9
- Misses      39848    39855       +7
  Partials     2739     2739

Impacted Files
Coverage Δ

...a/com/navercorp/pinpoint/test/PluginTestAgent.java
0% <0%> (ø)
:arrow_up:

...navercorp/pinpoint/collector/util/PacketUtils.java
0% <0%> (-22.23%)
:arrow_down:

...t/collector/receiver/tcp/RequestPacketHandler.java
63.33% <0%> (-6.67%)
:arrow_down:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
80.59% <0%> (-2.99%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.37% <0%> (-1.75%)
:arrow_down:

...inpoint/profiler/sender/AsyncQueueingExecutor.java
76.08% <0%> (-1.09%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
85.36% <0%> (+2.43%)
:arrow_up:

...p/pinpoint/profiler/sender/AbstractDataSender.java
100% <0%> (+2.94%)
:arrow_up:

Continue to review full report at Codecov.

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


Reorder Environments

Is your feature request related to a problem? Please describe.
I want to be able to reorder my list of environments because I'm starting to have a lot and right now they're just ordered on how I added them
Describe the solution you'd like
Show a way to reorder the environments. For example: Drag-n-Drop or up/down arrows
I would even settle for editing the config manually and restarting the app (although I have no idea where the config is nor if it's even in a text format I can edit)
Describe alternatives you've considered
It seems my only option is to remove/overwrite in the order I want them which is very tedious
Additional context

Thanks for reporting this! I'd look into it as soon as I can.

Please keep considering this


fix for softether

They must have changed something, regardless I've updated my fork to fix it
https://github.com/blindpet/MediaServerInstaller/commit/255bab1162212b4cb79a456ecf8eac6c133bb2ef

Could be. This scripts need some update ... busy elsewhere :(

I started on dev branch:
https://github.com/igorpecovnik/Debian-micro-home-server/commits/dev

Yea I know you're busy, trying to help where I can. Here's a fix for Varnish
wget -O - https://repo.varnish-cache.org/GPG-key.txt | apt-key add -
cat > /etc/apt/sources.list.d/varnish-cache.list<<EOF
deb-src https://repo.varnish-cache.org/debian/ jessie varnish-4.1
EOF
apt-get update
apt-get build-dep varnish -y
cd /tmp
apt-get source varnish -y
rm varnish_*.dsc
rm varnish_*.orig.tar.gz
rm varnish_*.diff.gz
cd varnish-4*
./configure --prefix=/usr
make -j$(ncpu)
make install
cp debian/varnish.init /etc/init.d/varnish
chmod +x /etc/init.d/varnish
cp debian/varnish.default /etc/default/varnish
update-rc.d varnish defaults
mkdir -p /etc/varnish
cp etc/example.vcl /etc/varnish/default.vcl
dd if=/dev/random of=/etc/varnish/secret count=1
service varnish start


⚠ Bofrost.at has degraded performance

In 0350c44, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1471 ms

Resolved: Bofrost.at performance has improved in ed949fb after 22 minutes.


hotfix that fixes the rotation issue with structured templates

this is a quick hot-fix that fixes the block building in structured templates. A freeformfamily is now considered a side-defined family. this can be verified by building a structured template shape with stairs and verifying that the stairs are in the right direction.

Hooray Jenkins reported success with all tests good!

Tested out in GooeysQuests and MedievalCities - looking good! Thank you so much :-)


[BUG] Azure Functions (Python V2 model) doesn't seem to support Session-based Service Bus Queue triggering

Investigative information
Please provide the following:

Timestamp: 2023-04-05 10:04
Function App name: hoiaipocnonprdeuwfacdc
Function name(s) (as appropriate): cdc
Core Tools version: 4.0.5095

Repro steps
Provide the steps required to reproduce the problem:

Package Name: azure-functions
Package Version: 1.13.3
Operating System: WSL Ubuntu (in VSCode) and Windows 11
Python Version: 3.10.4

I'm trying to create a Azure Service Bus Queue - triggered function using the Python v2 programming model, but I'm running into a bug.
I'm mainly following the steps outlined in this article: https://learn.microsoft.com/en-us/azure/azure-functions/functions-bindings-service-bus-trigger?tabs=python-v2%2Cin-process%2Cextensionv5&pivots=programming-language-python
The only difference being that the Queue I have created is Session-aware.
I'm trying to figure out how to specify the Session-id, but can't seem to find it back in the documentation, nor anywhere online.

Expected behavior
Screenshots
Full screenshot from the logstream:

Expected behavior
Provide a description of the expected behavior.
That I'm either able provide the Session-id as a decorator parameter, or that the functioning of the Queue-trigger for session-aware queues is agnostic, and can take in messages normally.
Actual behavior
Provide a description of the actual behavior observed.
The error message is as follows:
[Error]   ReceiveBatchAsync Exception: System.InvalidOperationException: It is not possible for an entity that requires sessions to create a non-sessionful message receiver.

It works fine for a non-Session-aware Queue trigger, but it starts throwing the error if I make it Session-aware. I also can't seem to find any parameter for the decorator that would allow me to specify the Session-id.
Contents of the requirements.txt file:
Provide the requirements.txt file to help us find out module related issues.
azure-functions
azure-identity
azure-servicebus

@gavin-aguiar  pls comment  and validate

I'm having the same errors with Python 3.10, v2 programming model. The examples here don't help either. Is this a bug? Or has anyone found a more complete example that works?

@TDehaene I've been experiementing Service Bus sessions recently. Not sure  if the below code could help you.
import json
from typing import List

from azure.servicebus import ServiceBusMessage
from azure.servicebus.aio import ServiceBusClient

client = ServiceBusClient.from_connection_string("SB-CONN-STRING")

results_df = pd.DataFrame(results_list)
results_df = results_df.sample(frac=1).reset_index(drop=True)

async def sample_session_send_receive_with_pool_async(
            pay_load,
            session_id,
        ):
    async with client.get_topic_sender(topic_name='nextstep-service-bus') as sender:
        await sender.send_messages(
            ServiceBusMessage(
                body=json.dumps(pay_load, default=str),
                session_id=session_id,
                subject='AI_SESSIONS_DEMO'
            )
        )

Please check this for more examples: https://github.com/Azure/azure-sdk-for-python/tree/main/sdk/servicebus/azure-servicebus/samples/sync_samples


[BUG] Automatically installed scoop-search detected as malware

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions
[X] This issue is about a bug (if it is not, please use the correct template)

Describe your issue
scoop-search which WingetUI automatically installs without asking the user is currently being detected as malware by some vendors and was put into my antivirus's quarantine. This could be a false positive, but if not, then not having asked the user to install this would be even worse.
Steps to reproduce the issue
No response
WingetUI Log
-

Package Managers Logs
-

Relevant information
Not sure how WingetUI will behave after scoop-search has wrongly or rightly been removed from the system. Maybe WingetUI falls back to native (slow) Scoop searching?
Hopefully the situation resolves itself.
Screenshots and videos
No response

For visibility: https://github.com/shilangyu/scoop-search/issues/44#issuecomment-1896263779

Scoop-search is the search backend for scoop.
I will try to build some backend for scoop search if the library is not present.

@marticliment sfsu, although experimental, looks to be a more performant (~5x), more actively developed Scoop search (and other commands) drop-in replacement, which may be of interest. Also not currently flagged by any major AVs.

Partly in response to the AV false-positives, scoop-search's rewrite in Zig is done, released as v1.4.0, and is available via Scoop's main bucket as of a few hours ago.
https://github.com/shilangyu/scoop-search/releases/v1.4.0

Done for three reasons, in order of importance:

Use a language without a runtime to hopefully stop being incorrectly flagged as malware by anti-viruses
I was informed there are even faster tools
Have fun rewriting in Zig

scoop-search should behave exactly as before, but is now even faster.

https://github.com/ScoopInstaller/Main/commit/d65328c0cb46b9d615ed12a8f4c15b107717cbbd

scoop-search: Update to version 1.4.0

sfsu looks interesting.

@marticliment The rewrite of scoop-search has resolved this issue. No longer flagged as malware.


Replace libc with rustix in some backends

Replaces libc with rustix in the epoll, event ports and poll-based backends. This ensures that libc is entirely bypassed when making system calls.
We can't do kqueue until bytecodealliance/rustix#605 is released.
Supersedes #91

...and since bytecodealliance/rustix#605 is released now, kqueue can be implemented with rustix now too! (Or almost all of it). Thanks, @sunfishcode!


EnclosingClass.super.methodCall() doesn't seem to work properly

EnclosingClass.this.methodCall() is fine but super generates code that complies with a warning and crashes with does not recognise selector

I'm unable to reproduce any issue. With your example code the "run" method produces the following translation, which executes correctly.
- (void)run {
  ((void (*)(id, SEL))[Test instanceMethodForSelector:@selector(methodCall)])(this$0_, selector(methodCall));
}


Test Cancel Workflow in Dispatcher Service

[ ] I have added the tests to cover my changes.
[ ] I have updated the documentation and CHANGELOG accordingly.
[ ] I have read the CONTRIBUTING document.

@Emmanuel289 this issue may be related: https://github.com/AgnostiqHQ/covalent-staging/issues/333

@Emmanuel289 this issue may be related: AgnostiqHQ/covalent-staging#333

Alright. I will link it up. Though it has an empty scope

@kessler-frost Please let me know if I'm on the right track for the test_cancel_workflow_execution function or if I've missed something... I'll complete the bodies of the remaining two test functions by today.

@Emmanuel289 Yeah, I think you're on the right track.

@Emmanuel289 Yeah, I think you're on the right track.

Thanks!


Client don't reregister in some cases

Hi guys we are having an issue, that client stop sending register, after connection was lost for 1-2 minutes.  This don't cause WS connection to close, i see it opened in devtools, and we receive OPTIONS after connections is back.  What i have noticed that i will re-register if it didn't received OPTIONS after ISP is back, and if OPTIONS was received after connect, client stops send register every 60 sec (as we configured) at all.
Here is those OPTIONS:
Tue Oct 27 2015 14:43:23 GMT+0200 (EET) | sip.transport | received WebSocket text message:

OPTIONS sip:dml4e1gs@<IP_ADDRESS>;transport=wss SIP/2.0
Via: SIP/2.0/WS SERVER_IP:5060;branch=z9hG4bK73ce1bc5;rport
Max-Forwards: 70
From: "unknown" <sip:unknown@SERVER_IP>;tag=as726f625e
To: <sip:dml4e1gs@<IP_ADDRESS>;transport=wss>
Contact: <sip:unknown@SERVER_IP:5060;transport=WS>
Call-ID: 32d0b8be3a5c0aed5583338a1d316428@SERVER_IP:5060
CSeq: 102 OPTIONS
User-Agent: OUR_COMPANY
Date: Tue, 27 Oct 2015 12:42:28 GMT
Allow: INVITE, ACK, CANCEL, OPTIONS, BYE, REFER, SUBSCRIBE, NOTIFY, INFO, PUBLISH, MESSAGE
Supported: replaces, timer
Content-Length: 0

bundle.js:19907 Tue Oct 27 2015 14:43:23 GMT+0200 (EET) | sip.transport | sending WebSocket message:

SIP/2.0 200 OK
Via: SIP/2.0/WS SERVER_IP:5060;branch=z9hG4bK73ce1bc5;rport
To: <sip:dml4e1gs@<IP_ADDRESS>;transport=wss>;tag=r984l0kpfu
From: "unknown" <sip:unknown@SERVER_IP>;tag=as726f625e
Call-ID: 32d0b8be3a5c0aed5583338a1d316428@SERVER_IP:5060
CSeq: 102 OPTIONS
Allow: ACK,CANCEL,BYE,OPTIONS,INFO,NOTIFY,INVITE
Accept: application/sdp,application/dtmf-relay
Supported: outbound
User-Agent: SIP.js/0.7.1
Content-Length: 0

bundle.js:19907 Tue Oct 27 2015 14:43:23 GMT+0200 (EET) | sip.transaction.nist | Timer J expired for non-INVITE server transaction z9hG4bK73ce1bc5
bundle.js:19907 Tue Oct 27 2015 14:43:23 GMT+0200 (EET) | sip.transport | received WebSocket text message:

SIP/2.0 401 Unauthorized
Via: SIP/2.0/WSS <IP_ADDRESS>;branch=z9hG4bK442602;received=CLIENT_IP;rport=51264
From: <sip:USER_NAME@OUR_DOMMAIN_NAME>;tag=ju51htu5kt
To: <sip:USER_NAME@OUR_DOMMAIN_NAME>;tag=as5e801060
Call-ID: 5scc2a805rpsu0pprm8kej
CSeq: 85 REGISTER
Server: OUR_COMPANY
Allow: INVITE, ACK, CANCEL, OPTIONS, BYE, REFER, SUBSCRIBE, NOTIFY, INFO, PUBLISH, MESSAGE
Supported: replaces, timer
WWW-Authenticate: Digest algorithm=MD5, realm="020000A3C076", nonce="7cb0b1fc"
Content-Length: 0

Is this something known?

Could you provide a gist of a full trace of this happening and recovering?

You can close this one, the issue was on our server side, sorry for disturbing you.


How does this compare to Magellan?

How would you compare ava to https://github.com/testarmada/magellan - is ava for paralellizing unit tests versus magellan for functional, selenium-powered tests? Are there any plans (or how easy would it be) to extend ava to support port management, https://github.com/FormidableLabs/rowdy, etc..?
thanks!!

Not sure if I'm fully understanding the question, but ava is a library more akin to tape/mocha and is built for unit testing, while magellan looks like it's more of an integration suite for browser testing. Is that what you're getting at? If so, these are different libs for different testing types and at least as far as I know there aren't plans to move ava in the integration direction :) happy for anyone else to chime in though, I'm pretty new to watching/helping w/ava

I see... it sounds like this is a test framework with parallelization built-in as a feature, as opposed to a parallelization framework that works to parallelize test suites written in other frameworks (mocha, jasmine, etc)?

@mattfysh pretty much, yeah — parallelization is a feature, not the feature. I suppose you could use the runner to run other tests, but really then you're creating a modified version of the gorilla-banana problem with no extra inherent benefit :)

@markthethomas gave a good answer on this one, I think it's safe to close.
@mattfysh If you have some questions regarding AVA, feel free to chat in our gitter room.


What a documentation

[Enter feedback here]
The next statement  was found in the page:
???? If you examine this key, you can see how to specify different languages. ????
The code to examine was like the following
// Visual Basic
[$RootKey$\Languages\CodeExpansions\Basic\Paths]
"HelloWorldVB"="$PackageFolder$"
When I "EXAMINED" the code I found that, when the language is Visual Basic it's translated to \Basic.
Oh yes then I'm very able to know when it's C#, razor, html and so on. Of course I've to know, it's evident.

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

ID: 5c60cdeb-c3af-ebb1-80e7-1ec682896734
Version Independent ID: 04010067-02f9-548e-467b-fa4efa391860
Content: Distribute code snippets as an extension - Visual Studio (Windows)
Content Source: docs/ide/how-to-distribute-code-snippets.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@GeorgeSaeed -- thank you for your feedback.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@TerryGLee -- please look into this issue.


Fix Sortablejs throwing an error when Livewire is not present

Fix for #27. I don't see a problem with the fact that there will no longer feedback if Livewire is not instantiated. Either Livewire is just not installed in which case you shouldn't use this package or Livewire is not used on a page in which case error suppression is wanted

Thanks!