Add reference links support

I think it could be possible using RichTextLabel.scroll_to_line()

Hey I was about to use dialogic's markdown parser  but I found your addon and it's exactly what I need! I just want to say that dialogic implemented the markdown 'sections' that can be linked to. I haven't seen it in action but you might find another solution there.


langchain-visualizer on clean env got the error

pydantic.errors.PydanticImportError: BaseSettings has been moved to the pydantic-settings package. See https://docs.pydantic.dev/2.3/migration/#basesettings-has-moved-to-pydantic-settings for more details.
from langchain_visualizer import visualize_embeddings

if "__main__" == __name__:
    visualize_embeddings()

Sorry, I can't reproduce this... This project doesn't use pydantic anywhere. Can you try again on v0.0.30 and see if the problem still persists?

Same on my end.

Ah thanks for providing the stack trace! It turns out this is caused by https://github.com/oughtinc/ice/issues/319
I've set this to explicitly depend on pydantic v1 instead while the above issue is still open. Please try seeing if v0.0.31 of langchain-visualizer works for you. Feel free to reopen this if it's still failing :)


Update Chinese translation of how-to-contribute.md

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

I found a few strange sentences in the Chinese translation of how-to-contribute.md when I am reading it. I happened to be free, so I fixed them.

Thanks so much for your first contribution!  You rock, @aeilot!

16qwwLpSDngpgkf3cBZbRPW39h9x3dvdMk

16qwwLpSDngpgkf3cBZbRPW39h9x3dvdMk


Change default author (@rocketseat)

Description of bug
If I change the default author of the theme, he remains persistent in finding the author @rocketseat.
To Reproduce

Change the name '@rocketseat'
Configure gatsby-config.js to set the new author name
Open the preview

Expected behavior
Accept other author folder (name).

Exception or Error
Welcome, this is the default page of @rocketseat/gatsby-theme-docs from Rocketseat! To change this page just create a file named index.mdx inside @rocketseat/gatsby-theme-docs/text and place there your text. Also, if you need help, feel free to open an issue with a description of the problem you're facing or, if you prefer, you can chat with us on our Discord Community 👋🏻
Environment:
OS: Windows 10 (Gatsby running in Debian WSL)
Browser: Chrome 83.0.4103.116
Node v12.16.3

Additional context

Hey @OfficialMarinho, the siteAuthor is used for SEO reasons and is not related to folder names.
I believe you were expecting to change the folder name you shadow, but it is not possible. Since you’ve installed @rocketseat/gatsby-theme-docs and want to customize the index.mdx, you need to replace the theme’s original file @rocketseat/gatsby-theme-docs/text/index.mdx, with your own file to make any changes you need. You can't change the prefix @rocketseat, because this folder (and all its files) are included on the Webpack bundle.

Ohhhh, that's sad. I don't knew about that.
Anyway, thanks for the answer. 👍


Bug - button export csv sort auto by id DESC

Hello,
I didn't see that before but I get an error while pushing the button for export in csv my data, because it is sorted by id in DESC, which is not the identifier I chose, nor in my entity identifier, nor in my listView sortField and sortDir. I don't know how to resolve this.
Thanks

use the .sortField() and .sortDir() methods to choose the sorting field.


[SPARK-32800][SQL] Remove ExpressionSet from the 2.13 branch

What changes were proposed in this pull request?
This PR is a followup on #29598 and removes the ExpressionSet class from the 2.13 branch.
Why are the changes needed?
ExpressionSet does not extend Scala Set anymore and this class is no longer needed in the 2.13 branch.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Passes existing tests

Merged build finished. Test PASSed.

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

Test build #128295 has started for PR 29648 at commit 7f08aaf.

Test build #128300 has started for PR 29648 at commit d176b3a.

Merged build finished. Test PASSed.

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

Test build #128295 has finished for PR 29648 at commit 7f08aaf.

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

Merged build finished. Test PASSed.

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

Test build #128300 has finished for PR 29648 at commit d176b3a.

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

Merged build finished. Test PASSed.

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

Merged to master.


[Enhancement] Add option to use Nijiro scoring

I have written a guide for my best guess at how Nijiro scores are calculated: https://gist.github.com/joshuacwnewton/6410cb591a4c8a2c8a9ab4ac7fdef31b#2-computing-nijiro-scores-without-an-arcade-machine
I will reproduce the relevant text below just in case the Gist link ever goes away:

2 Computing Nijiro scores without an arcade machine
I've dug into a handful of posted videos of people playing the Nijiro Arcade version, most from teamPGX, to try to understand how Nijiro scores are computed. I believe I've come up with a rough formula that gets it right 95% of the time. Below, I'll demonstrate the steps needed to compute a Nijiro score.
2.1 Computing points per GOOD hit
First, we need to determine the number of points that you should get for a "GOOD" hit. The naive way of computing this value would be to take upper end of the range of scores (1,000,000) and divide it by the number of notes in the song. (As the number of notes times the number of points per GOOD will give the maximum "All GOOD"/"Donderful combo" score.)
n_points_GOOD_naive = 1_000_000 / n_notes

The first problem with this approach is that, in Taiko no Tatsujin, "points per GOOD" are scored in terms of 10s of points. To fix this, we need to round up to the nearest 10.
n_points_GOOD_naive = math.ceil((1_000_000 / n_notes)/10)*10

(Notably, this results in some fuzziness in scores, causing the base score to now be 1,000,000 plus or minus some accumulated rounding errors.)
2.2 Adjusting points per GOOD to compensate for drumrolls
The second (and bigger) problem is that this method only computes the point value for regular notes, and does not account for drumrolls. Drumrolls are worth a static value of 100 points, and are added on top of the value of regular notes. So, under this method, a perfect playthrough on song with 200+ drumroll hits will result in a score of roughly 1,020,000 points, while a perfect playthrough on song with only 50 drumroll hits will result in a score of 1,005,000.
To compensate for drumrolls (to allow comparing scores between songs of varying drumroll lengths), Nijiro scoring slightly decreases the "points per GOOD" amount depending on the length of the drumroll sections in a song.
The most straightforward way of computing the total length of the drumroll sections in a song is to play the song on "AUTO" mode, where the computer will play the song perfectly for you. You can then take the number of drumrolls that the computer hit, and map them as follows:
# Case-by-case method
if 0 <= n_drumrolls_cpu < 50:
   n_points_drumroll_adjustment = 0
elif 50 <= n_drumrolls_cpu < 150:
   n_points_drumroll_adjustment = 10
elif 150 <= n_drumrolls_cpu < 250:
   n_points_drumroll_adjustment = 20
# etc...

# Rounding method
n_points_drumroll_adjustment = round(n_drumrolls_cpu/100)*10

We can then combine the "drumroll adjustment" with the previously computed "points per GOOD" to get the adjusted points per GOOD:
n_points_GOOD = n_points_GOOD_naive - n_points_drumroll_adjustment

2.3 Computing points per OK
We almost have everything we need. The last thing that is necessary is to determine how many points should be provided for an "OK" hit. Thankfully, this is very easy; we just take the number of points per GOOD, and divide it by 2, rounding down to the nearest 10.
n_points_OK = math.floor((n_points_GOOD_adjusted/2)/10)*10

2.4 Points per BAD and drumroll
To repeat: "BAD" notes are worth 0 points, and drumroll hits are worth 100 points:
n_points_BAD = 0
n_points_drumroll = 100

2.5 Computing the total score
With all of these point values determined we can finally compute the total score:
# Compute points per GOOD
n_points_GOOD_naive = math.ceil((1_000_000 / n_notes)/10)*10
n_points_drumroll_adjustment = round(n_drumrolls_cpu/100)*10
n_points_GOOD = n_points_GOOD_naive - n_points_drumroll_adjustment

# Compute points per OK
n_points_OK = math.floor((n_points_GOOD_adjusted/2)/10)*10

# Compute points per BAD and drumroll
n_points_BAD = 0
n_points_drumroll = 100

# Compute total score:
n_points = (  (n_GOOD     * n_points_GOOD)
            + (n_OK       * n_points_OK  )
            + (n_BAD      * n_points_BAD )
            + (n_drumroll * n_points_drumroll))

2.6 Avoiding the need for computer drumroll count
There is one flaw with this formula: It's time-consuming to determine the number of drumrolls performed by the computer player for multiple songs, since you have to wait through the AUTO performance again and again. So, there is a simple shortcut: Estimate how fast you (or another player) performs drumrolls relative to the computer.
For example, if the computer gets 149 drumrolls for a given song, but you can drumroll faster, and get about 229 drumrolls on average for that song, then you can just take the ratio (149/229=0.65), and use it to remove the dependency on the CPU's drumroll count. (Note, however, that this will introduce some inaccuracy into the computations, as every now and then, the estimate will be off by a little bit, and throw the "adjusted GOOD score" into the wrong "adjustment bracket". But, it's good enough for 95% of cases.)
I found that in testing, a good number for typical "pro level scores" (e.g. teamPGX) is 0.535 computer drumrolls per 1 pro player drumroll. Therefore, we make the following adjustment:
# Compute points per GOOD
n_points_GOOD_naive = math.ceil((1_000_000 / n_notes)/10)*10
-n_points_drumroll_adjustment = round(n_drumrolls_cpu/100)*10
+n_points_drumroll_adjustment = round(n_drumrolls_pro_player*0.535/100)*10
n_points_GOOD = n_points_GOOD_naive - n_points_drumroll_adjustment

3 Example songs
To test the formula, as well as the "pro player estimate", here are some simple examples using two very similar 10* difficult songs:
3.1 Revenge of Meta-Knight Medley

1000 notes (all GOOD)
241 "pro player" drumrolls
teamPGX video

# Compute points per GOOD
n_points_GOOD_naive = math.ceil((1_000_000 / n_notes)/10)*10
                    = math.ceil((1_000_000 / 1000)/10)*10
                    = 1000
n_points_drumroll_adjustment = round(n_drumrolls_pro*0.535/100)*10
                             = round(241*0.535/100)*10
                             = 10
n_points_GOOD = n_points_GOOD_naive - n_points_drumroll_adjustment
              = 1000 - 10
              = 990

# Compute points per OK
n_points_OK = math.floor((n_points_GOOD_adjusted/2)/10)*10
            = math.floor((990/2)/10)*10
            = 490

# Compute points per BAD and drumroll
n_points_BAD = 0
n_points_drumroll = 100

# Compute total score:
n_points = (  (n_GOOD     * n_points_GOOD)
            + (n_OK       * n_points_OK  )
            + (n_BAD      * n_points_BAD )
            + (n_drumroll * n_points_drumroll))
         = (  (1000 * 990 )
            + (0    * 490 )
            + (0    * 0 )
            + (241  * 100))
         = 1_014_100

3.2 8OROCHI (Ura)

1001 notes (all GOOD)
73 "pro player" drumrolls
teamPGX video

# Compute points per GOOD
n_points_GOOD_naive = math.ceil((1_000_000 / n_notes)/10)*10
                    = math.ceil((1_000_000 / 1001)/10)*10
                    = 1000
n_points_drumroll_adjustment = round(n_drumrolls_pro*0.535/100)*10
                             = round(73*0.535/100)*10
                             = 0
n_points_GOOD = n_points_GOOD_naive - n_points_drumroll_adjustment
              = 1000 - 0
              = 1000

# Compute points per OK
n_points_OK = math.floor((n_points_GOOD_adjusted/2)/10)*10
            = math.floor((1000/2)/10)*10
            = 500

# Compute points per BAD and drumroll
n_points_BAD = 0
n_points_drumroll = 100

# Compute total score:
n_points = (  (n_GOOD     * n_points_GOOD)
            + (n_OK       * n_points_OK  )
            + (n_BAD      * n_points_BAD )
            + (n_drumroll * n_points_drumroll))
         = (  (1001 * 1000 )
            + (0    * 500 )
            + (0    * 0 )
            + (73   * 100))
         = 1_008_300

As you can see, despite these two songs having very different drumroll counts, they both end up being about the same (a few thousand points above 1,000,000).
3.3 Zouo to Shuuaku no Hanataba (Ura)
For one last demonstration, we can try a trickier example, with lots of OKs and a few BADs:

1170 notes (738 GOOD, 385 OK, 47 BAD),
96 "pro player" drumrolls
チャンネルみくぴー video

# Compute points per GOOD
n_points_GOOD_naive = math.ceil((1_000_000 / n_notes)/10)*10
                    = math.ceil((1_000_000 / 1170)/10)*10
                    = 860
n_points_drumroll_adjustment = round(n_drumrolls_pro*0.535/100)*10
                             = round(96*0.535/100)*10
                             = 10
n_points_GOOD = n_points_GOOD_naive - n_points_drumroll_adjustment
              = 860 - 10
              = 850

# Compute points per OK
n_points_OK = math.floor((n_points_GOOD_adjusted/2)/10)*10
            = math.floor((850/2)/10)*10
            = 420

# Compute points per BAD and drumroll
n_points_BAD = 0
n_points_drumroll = 100

# Compute total score:
n_points = (  (n_GOOD     * n_points_GOOD)
            + (n_OK       * n_points_OK  )
            + (n_BAD      * n_points_BAD )
            + (n_drumroll * n_points_drumroll))
         = (  (738 * 850 )
            + (385 * 420 )
            + (47  * 0 )
            + (96  * 100))
         = 798_800

The only tricky part is analyzing a song to determine n_drumrolls, then computing the how much the "points per GOOD" should be deducted using n_drumrolls. That part I am not 100% sure on yet, I am only approximating in my "Example songs" section, but my approximation works 95% of the time.


Restrictions on usage of model

Hi,
Just wanted to confirm that there are no restrictions on how we query the model ie: the model can only give the confidence on the top n predicted tokens, etc.

@akul-goyal @pluskid
There are unnecessary restrictions on the query .... creating unnecessary tokens.
It is the first step towards a universal grammar for query programming.
Disable the following (and tidy upstream coding):
https://github.com/google-research/lm-extraction-benchmark/blob/a36cbfab1f7719e71667aedc7fbf7f27b57ab823/baseline/simple_baseline.py#L69
I remain interested.

Correct; you can query however you like.
@heitikei I'm sorry I don't understand what you're trying to say. Can you please raise a new issue if there's some problem?


Addresses thread safety, GitHub issue #769.

See also: discussions in PR #852 and #786.
In addition to improving thread safety, it removes 8 bytes of heap per node.

@jpap for context: we have an internal device lab that will run benchmarks on physical Android and iOS devices. As much as I would like everyone to be able to run benchmarks, this will not become a publicly available resource.
Some numbers:

The synthetic benchmark that measures allocation, style assignment, layout calculation, layout reads, and deallocation recessed by ~11% (Android)
calculation in isolation recessed by ~11% on Android, ~8.5% on iOS
allocation/deallocation in isolation recessed by ~35% (Android)
The losses are smaller when using the Java API, as there’s fixed JNI overhead

These numbers don’t reflect real-life scenarios accurately (we will get there in the coming months), but they give an indication about the performance characteristics of changes.

The other approach in #852 suffers from not being able to concurrently perform a layout of a given tree, and any number of cloned subtrees. This is because per-layout state is stored in each (shared) node, and the generation count is different for each layout run. (If this is not of practical concern, it could be documented as a limitation.)

For now this is an acceptable limitation, and I agree that it should be documented. Shared subtrees are cloned lazily during layout calculation, and the only client making use of that is React Native. In their case, that’s thread safe, as the two highest ancestors sharing a node are never identical, and React Native treats subtrees as immutable.
That being said, it would be nicer to evolve Yoga into a direction where we can separate the layout outputs from the nodes, but this involves migrating client frameworks as well, including Litho, ComponentKit, React Native, React Native Windows, and internal projects at Facebook and other companies.
I think the short term goal should be to make parallel calculations on different trees possible.
Apologies if I am not always super responsive on Github. I really appreciate the input, though!


[libjpeg-turbo] Upgrades 2019.05.08

Upgrade to revision 2.0.2, related to PR https://github.com/microsoft/vcpkg/pull/6382

Ignore the failure here, vxl failed without creating any logs since openjpeg installed on the test machine before vxl. Similar for butteraugli.

x64-windows-static
test
notes

guetzli
Skip
Missing port dependency

butteraugli
Fail
Regression

x64-linux
test
notes

vxl
Fail
Regression

/azp run


Merge op params and query string params in GET operations

Params passed in GET operation are now available via Rack::Request params method.
Uses URI.parse instead of url.split("?") to parse the op url

@arsduo, can we get this PR merged?

It doesn't work for me. I don't know why.


[Abide] Password type fields couldn't passed the required rule

How can we reproduce this bug?

Use a type "password" into input fields with required attribute
Put a value to the password input
Submit the form. It's always throws an error and form can't pass

What did you expect to happen?
It should pass the validation since It's already has a value
What happened instead?
Did not pass the form validation
Give us a link to a CodePen or JSFiddle that recreates the issue.

#7407 fixed this and many other abide issues.

Should be fixed with #7590, please comment and submit pr's to the abide branch until 6.1 release.


Same BCU version on 2 machines Win 10 x64 shows 2 different versions to same binary

Same BCU version on 2 PCs running Win 10 x64 shows 2 different versions to same installed binary
BCU ver. <IP_ADDRESS>773 (on both PCs) installed on two PCs running windows 10 x64 - in both machine the same binary (https://github.com/clsid2/mpc-hc/releases/download/1.8.4/MPC-HC.1.8.4.x64.exe) is installed.
In 1 machine this binary shows as correctly as version 1.8.4
but on another machine it shows up as version 1.7.13 !!!!!!!!!!!
Same binary on Both PCs was a clean install (any previous version uninstalled by BCU complete with all registry keys deletion etc.)
Both binaries were MD5 compared (using CyoHash) and are identical (see attached)

Make sure that MPC didn't automatically update itself, or the installer doesn't automatically download the newest version.
If it's the exact same files installed, please select the entry in BCU and click File > export on both systems and send the files.


Gate diffuse and specular transmission behind shader defs

Objective

Address #10338

Solution

When implementing specular and diffuse transmission, I inadvertently introduced a performance regression. On high-end hardware it is barely noticeable, but for lower-end hardware it can be pretty brutal. If I understand it correctly, this is likely due to use of masking by the GPU to implement control flow, which means that you still pay the price for the branches you don't take;
To avoid that, this PR introduces new shader defs (controlled via StandardMaterialKey) that conditionally include the transmission logic, that way the shader code for both types of transmission isn't even sent to the GPU if you're not using them;
This PR also renames STANDARDMATERIAL_NORMAL_MAP to STANDARD_MATERIAL_NORMAL_MAP for consistency with the naming convention used elsewhere in the codebase. (Drive-by fix)

Changelog

Added new shader defs, set when using transmission in the StandardMaterial:

STANDARD_MATERIAL_SPECULAR_TRANSMISSION;
STANDARD_MATERIAL_DIFFUSE_TRANSMISSION;
STANDARD_MATERIAL_SPECULAR_OR_DIFFUSE_TRANSMISSION.

Fixed performance regression caused by the introduction of transmission, by gating transmission shader logic behind the newly introduced shader defs;
Renamed STANDARDMATERIAL_NORMAL_MAP to STANDARD_MATERIAL_NORMAL_MAP for consistency;

Migration Guide

If you were using #ifdef STANDARDMATERIAL_NORMAL_MAP on your shader code, make sure to update the name to STANDARD_MATERIAL_NORMAL_MAP; (with an underscore between STANDARD and MATERIAL)

I'm not sure how much that really matters. I suspect the overhead of the if statement would be worse than any potential extra work on average. It's not a big deal either way, but it's probably worth removing still.

Okay, after thinking about it for a bit, I ended up being convinced that it will be a rare enough situation (having a texture to control the transmission levels, and having the majority of it not be transmissive) that's probably not worth it to give the much more common scenario a small penalty every time just for this unlikely benefit. Removed the if statements.


Breaking sifters voids meshes

Breaking a sifter doesnt return the mesh, wasting the material especially if misplaced

Fixed and released!
Thx!


Hide only pageList buttons, if data is placed on one page

Hello! I tried version 3. And it is very cool. But I would like to be able to hide only pageList button if data is placed on the one page (and SizePerPage and Total are visible). I have not found a way to do it now.
For example as a parameter in the options like hideSizePerPage. It would be great. Thank you for your work!

@jmorozov, sorry we dont support this config to hide the pagination list if the only 1 page there. BTW, did you have custom the pagination panel by options.paginationPanel, if you did, I think you can hide the pagination list by yourself, because it provide full ability for you to custom.
Anyway, I think I can just support this feature by a options, but it'll be release with v3.0.0 instead of v2.x, I've stop all the development on v2.x and ready to release v3.0.0 to production.
Thanks

Fixed on v3.1.0, use options.hidePageListOnlyOnePage and assign true will hide page list if only one page here.


Explore slimNT documentation

Explore documentation and link relevant information. Try to update if possible.

Script is straightforward conceptually. Familiarizing myself with nextflow to better understand the steps.

Most updated version of slimNT on HIVE 1  is slimNT_20240123. This version has been tested and works as expected.


rollback problem of different DB

hi everyone
i have a question about fescar rollback for transcation
for this code ：
`	@Autowired
private DemoFeignClient demoFeignClient;
@Autowired
private DemoFeignClient2 demoFeignClient2;
@GlobalTransactional(timeoutMills = 300000, name = "spring-cloud-demo-tx")
@GetMapping("/getdemo")
public String demo() {

	// 调用A 服务  简单save  数据在A数据库的test表中
	Integer result = demoFeignClient.insertService("test",1);

	// 调用B 服务(B数据库中的 test表中)。报错测试A 服务回滚
	demoFeignClient2.saveService();

	System.out.println("this is -------port:9091");
	return "data";
}`

the problem is  B service run fail, but A service can not rollback , it is still save success,
please help me .and  i need other Configuration?

Is fescar's starter referenced? Please print xid at the provider

@slievrly   i was imported fescar-starter to my project service A & B . i can print to xid in my service project A & B . bug it isn't still rollback ,

For resolving your problem.please create a github repo for your example.so we can debug it locally.

@xingfudeshi  i have find it  by  slievrly help ,it is  a bug for fescar . but i don't konw how to fixed it and waiting for fixed it by other fescar's developer

@vikenlove ok.will fix it later.

@xingfudeshi  which is version fixed it？ 0.4.0 GA？

ERROR 12172 --- [imeoutChecker_1] c.a.fescar.core.rpc.netty.TmRpcClient    : eureka client is not register,do this operation had to be register!
2019-03-20 09:30:44.369 ERROR 12172 --- [imeoutChecker_1] c.a.fescar.core.rpc.netty.TmRpcClient    : no available server to connect.
@xingfudeshi  does fescar‘s developer  not  fixed this bug at 0.4.0 version？

@vikenlove please dingtalk ping  me。


📝 CA Workshop Marketing PPT

Split the original Clean Architecture PPT from the retreat into smaller PPT specific for Workshops

@william-liebenberg - is this the workshop marketing PPT (i.e. NOT the workshop content PPT)?

Correct :)

@william-liebenberg - I thought this had already been split?  What is still outstanding?

The PPT above IS the one that got split out from the original big kahuna PPT.
@matt-goldman and I should review the slides this week and make final changes.

@william-liebenberg - we'll also need to update the event agenda: https://www.ssw.com.au/ssw/Events/Training/Clean-Architecture-Workshop.aspx
Hopefully this information could be included in the marketing PPT so we can give to Camilla.

@william-liebenberg @matt-goldman - I've added the agenda as new slides.  Can you please review?
https://sswcom.sharepoint.com/:p:/s/CleanArchitecturev2/ERvFIRuzgs5BpORoeYKqn5gBQoGijXzS7sIk61XnvpEpZQ?e=3Aw5N5&nav=eyJzSWQiOjM2MywiY0lkIjoxMDM5ODUxODQ5fQ

@danielmackay - reviewed and left a couple of new comments

@matt-goldman - did you end up working on this last Friday?  What else is remaining here?

All looks good to me know


Fix libXScrnSaver requirement

openSUSE and some others distributions uses a different name for package libXScrnSaver, it's called libXss1.
This fix uses the name of the module provided by the package, this will make dependency independent of package name.
https://github.com/keybase/keybase-issues/issues/2445

Thanks for the PR!  I'm sorry we missed this.  CC @oconnor663 to help review, fixes the RPM on OpenSUSE.

When I try this on my 64-bit Fedora VM, it looks like it tries to install a bunch of 32-bit packages:
[jacko@localhost ~]$ sudo dnf install libXss.so.1
keybase                                                                             41 kB/s | 5.1 kB     00:00
Dependencies resolved.
===================================================================================================================
 Package                          Arch                 Version                         Repository             Size
===================================================================================================================
Installing:
 glibc                            i686                 2.24-4.fc25                     updates               4.0 M
 libX11                           i686                 1.6.4-4.fc25                    updates               623 k
 libXScrnSaver                    i686                 1.2.2-10.fc24                   fedora                 28 k
 libXScrnSaver                    x86_64               1.2.2-10.fc24                   fedora                 28 k
 libXau                           i686                 1.0.8-6.fc24                    fedora                 34 k
 libXext                          i686                 1.3.3-4.fc24                    fedora                 43 k
 libcrypt-nss                     i686                 2.24-4.fc25                     updates                43 k
 libxcb                           i686                 1.12-1.fc25                     fedora                227 k
 nss-softokn-freebl               i686                 3.29.3-1.0.fc25                 updates               221 k

Transaction Summary
===================================================================================================================
Install  9 Packages

Total download size: 5.3 M
Installed size: 17 M
Is this ok [y/N]:

It looks like some obscure parentheses syntax can fix this?
[jacko@localhost ~]$ sudo dnf install 'libXss.so.1()(64bit)'
Last metadata expiration check: 0:00:19 ago on Wed Apr 19 11:33:24 2017.
Dependencies resolved.
===================================================================================================================
 Package                       Arch                   Version                         Repository              Size
===================================================================================================================
Installing:
 libXScrnSaver                 x86_64                 1.2.2-10.fc24                   fedora                  28 k

Transaction Summary
===================================================================================================================
Install  1 Package

Total download size: 28 k
Installed size: 39 k
Is this ok [y/N]:

But I'm not able to find any actual documentation on how the parentheses work. Do you know if this would be the right thing to do on 64-bit systems? Will it be portable across other RPM-based distros?

openSUSE 42.2
sudo zypper install libXss.so.1
Loading repository data...
Reading installed packages...
'libXss.so.1' not found in package names. Trying capabilities.
Resolving package dependencies...

The following 5 NEW packages are going to be installed:
  libX11-6-32bit libXau6-32bit libXext6-32bit libXss1-32bit libxcb1-32bit

"libXss1' providing 'libXss.so.1()(64bit)' is already installed"
sudo zypper install 'libXss.so.1()(64bit)'
Loading repository data...
Reading installed packages...
'libXss.so.1()(64bit)' not found in package names. Trying capabilities.
'libXss1' providing 'libXss.so.1()(64bit)' is already installed.
Resolving package dependencies...

Nothing to do.

@oconnor663 I think it will be portable. Thanks by report the installation of 32-bit packages.

 I know that tons of packages do have that requirement but I never understood why.
This is the description of the package in question and I seriously don't get why keybase would depend on that:

libXss1 - X11 Screen Saver extension client library
The X Window System provides support for changing the image on a display screen after a user-settable period of inactivity to avoid burning the cathode ray tube phosphors. This extension allows an external "screen saver" client to detect when the alternate image is to be displayed and to provide the graphics.

@michbona Electron won't start without linking against this library. It's possible there might be workarounds for this problem that I don't know about yet though?

OK, if it's an Electron thing, I understand the link to a library that protects cathode ray tubes ;-) Thanks for the info. I have no experience with Electron, so I can't help with a workaround. Making the dependency distri independent seems to be the way forward.

We get it from Electron, Electron gets it from Chromium, and Chromium uses it as one of many inputs for determining whether the system is currently idle.

I ignore the dependencies on my openSUSE Tumbleweed installaion for libXScrnSaver and keybase launches dandily. Is this really required now? Or am I missing feature?

@cjb any chance Electron started statically linking it?

@oconnor663 Possible, but even if they had, we don't take on new Electron versions quickly, so I don't expect that's it.
Perhaps openSUSE has this dep but provides it in a way that doesn't satisfy our ask?  Dunno.

Sorry, this discussion seems be happening in Issue #2445 as well, but since the proposed patch is here, I will add my 2 cents here. @cjb : I appreciate that new features like Teams are great and take a lot of attention, but can we please solve this pesky little bug for some early supporters? It's really hard to believe that a security-focused app like keybase cannot provide auto-updates on a major Linux distro. And I am not talking about the Zypper bug, this one is on your package.

@michbona Sorry, we're a small team, someone who actually uses openSUSE will need to help fix it.  The proposed patch here doesn't work -- it installs 32-bit libs.  And when we switched to it to asking for 64-bit libs, it seemed like it didn't install anything.

It's really hard to believe that a security-focused app like keybase cannot provide auto-updates on a major Linux distro.

The very latest change to this PR (just now, as I'm writing this comment) probably does the right thing, with an if statement that specifies different dependencies for different architectures.
Testing is the real issue here, and that's why we didn't just land a fix like this when the problem was first reported. The variance between RPM implementations in particular is really wide -- take a look at the yum/yast/urpmi switches all over this file (mostly copied from Chrome's packaging). Various versions of various distros tend to have maddening little differences that have burned us before. Hardcoding a library filename feels like precisely the kind of thing that will break someone somewhere someday.
If anyone with RPM packaging experience could suggest a less-dangerous-sounding workaround, that would be a huge help for us getting this in quickly.

@cjb @oconnor663 I understand the different priorities and the complexity of the problem. And I appreciate every minute you spend solving the problem or at least communicating. But @marceloatie has offered to help, submitted a patch and then had to almost beg for it to be considered. And while I am happy to test stuff, it's really hard to test something that is not in the package. Looking forward to some quicker turnaround times in the futures. After all, I am sure you're not the first to have that particular problem - especially if it's really caused by Electron.

I've put this ticket at the top of our sprint for next week. Apologies for all the delay.

Just merged these changes as part of https://github.com/keybase/client/pull/9996. The next public release probably won't be until after the holidays though.


Items must be properties or removing them doesn't cause re-render

Fixes #739

Fixes #739

This sould also fix #736


GFSocialButton: Unable to load asset: "icons/facebook.png".

Describe the bug
The following code
class SignInPage extends StatelessWidget {
  const SignInPage({super.key});

  @override
  Widget build(BuildContext context) {
    return SafeArea(
        child: Container(
      padding: const EdgeInsets.all(25.0),
      child: Column(
        mainAxisAlignment: MainAxisAlignment.center,
        crossAxisAlignment: CrossAxisAlignment.center,
        children: [
          GFSocialButton(
            type: GFSocialType.facebook,
            buttonType: GFSocialButtonType.solid,
            onPressed: () {
              signInWithFacebook();
            },
          ),
          GFSocialButton(
            type: GFSocialType.google,
            buttonType: GFSocialButtonType.solid,
            onPressed: () {
              signInWithGoogle();
            },
          ),
        ],
      ),
    ));
  }
}

produces the following error:
Restarted application in 990ms.

════════ Exception caught by image resource service ════════════════════════════
The following assertion was thrown resolving an image codec:
Unable to load asset: "icons/facebook.png".

Exception: Asset not found
When the exception was thrown, this was the stack
#0      PlatformAssetBundle.loadBuffer
#1      AssetBundleImageProvider._loadAsync
#2      AssetBundleImageProvider.loadBuffer
#3      ImageProvider.resolveStreamForKey.<anonymous closure>
#4      ImageCache.putIfAbsent
#5      ImageProvider.resolveStreamForKey
#6      ScrollAwareImageProvider.resolveStreamForKey
#7      ImageProvider.resolve.<anonymous closure>
#8      ImageProvider._createErrorHandlerAndKey.<anonymous closure>
<asynchronous suspension>
Image provider: AssetImage(bundle: null, name: "icons/facebook.png")
Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#fc06d(), name: "icons/facebook.png", scale: 1.0)
════════════════════════════════════════════════════════════════════════════════

════════ Exception caught by image resource service ════════════════════════════
Unable to load asset: "icons/google.png".
════════════════════════════════════════════════════════════════════════════════
D/EGL_emulation( 6586): app_time_stats: avg=631.67ms min=13.07ms max=33210.80ms count=54

To Reproduce
Steps to reproduce the behavior:
Try the code above.
Desktop (please complete the following information):

OS: [e.g. iOS] Linux
Browser [e.g. chrome, safari] Chrome
Version [e.g. 22] latest

Am currently having this issue right now
Have you fixed it

I just switched to using a different plugin (sign_button) for generating social login buttons.

is it fixed? if not can assign to me

@gantanikhilraj  our team is fixing it.

I found solution which work for me
Step 1: Create "icons" directory in the root folder and Add required icons to "icons" directory
Step 2:
Update pubspec.yaml file assets section
assets:
- icons/

Hi, we have fixed this issue, we have merged the changes
for now you can use this package like below until we update the version number.

Is the issue with icons not loading in the docs related to this?


Migrate to Postgresql as DB in place of sqlite

Migrated the back end DB to postgres and made data model changes to support this change. Also updated ansible playbooks to automate postgresql DB setup.

Coverage decreased (-0.05%) to 76.273% when pulling 8665dc174d5304dc58f14b2a9c040f0403fa2541 on postgres into bbc007ceff5021d5fec58c0caed475b81eb1e2a7 on master.

Coverage decreased (-0.05%) to 76.273% when pulling 8665dc174d5304dc58f14b2a9c040f0403fa2541 on postgres into bbc007ceff5021d5fec58c0caed475b81eb1e2a7 on master.


Is there a support for Braintree billing?

The readme and docs don't mention Braintree. Is it supported? What would be needed to implement that?

Technically out of the box I'm not too sure. I've never used it. I would think you would have to switch it in composer, and the database details and just try and see where it fails. I would think they're roughly compatible, at least in the basic sense. The billing kit is rather simple and just sets up basic stuff for subscriptions and single purchases. The BillingService which handles all the other stuff should be compatible as well. If you're up for it I'd give it a shot, and offer PRs for anything that isn't out of the box compatible. Thanks.


Handling generic descriptive terms for activities

Issue
There have been several activities submitted for addition to the list that include terms that could potentially apply to many activities. Examples include:
Low Impact Exercise for Health: Low-impact can refer to any activity that doesn’t place a lot of strain or weight through the joints.
SEND Activities: SEND stands for "Special Educational Needs and Disabilities", and refers to activities which are inclusive of children and young people with learning disabilities.
Adaptive Games: Activities which are adapted to make them inclusive of participants with specific needs.
GP Referral: Activities which are specifically for people who have been referred by their GP.
Pups on SUPs: Activities where participants are welcome to bring animals. In this case, stand up paddle boarding with dogs.
The primary aim of the activity list is to facilitate search, and we try to include terms that will support effective searching, though we should consider the implications of the list becoming significantly longer.
Options
Option 1: We include ‘versions’ of activities in the activity list as separate concepts.
For example, we add concepts for ‘SEND rugby’, ‘SEND football’, ‘SEND cricket’, etc as they are requested or as they emerge as common search terms.
This may simplify search, allowing users to use specific terms to hone in on appropriate activities.
However, it will extend the length of the list significantly over time. We are interested to hear from data publishers and data consumers about potential issues this may cause.
Option 2: These ‘versions’ or attributes of an activity are reflected elsewhere in the OpenActive specifications.
As with age, level, and gender restrictions, there may be alternative approaches to reflect the intended audience for an activity.
The oa:category property can be used to provide a set of tags that help categorise and describe an activity...
"category": [
    “Low Impact”
  ]
"category": [
    “GP Referral”
  ]
"category": [
    “Dog-friendly”
  ]

The oa:accessibilitySupport and oa:accessibilityInformation properties can be used to specify the types of disabilities or impairments that are supported or to provide additional, specific documentation for participants about how disabilities are, or can be supported.
There is also the more detailed extension of the Accessibility Support datatype proposed by Tim Hill for use in cases where accessibility provision is good and data-collection practices are strong.
Again, interested to hear from data publishers and data consumers on the challenges associated with this approach.
For example, can publishers envisage processes to populate the above fields? Do consumers feel they can present such information in search interfaces in a useful format to improve search?
Also keen to hear ideas for any alternative approaches.

@howaskew option 2 sounds more sensible to me. There are many scenarios where additional tagging would be useful and this is a good time to utilise it.
We use 'collections' in a number of activity finders, which are a useful way of grouping activity types together. However, it would be useful to be able to create collections for things like 'GP referral' which could have different parameters for what meets those requirements outside of activity type.
From a data publishers perspective, any additional steps when listing an activity (a) creates work for publishers, (b) adds another step to the providers listing experience, I would aim for working with the data that is already present (maybe through keywords in descriptions).
From a data consumers perspective, more flexible categories and a search experience would be a nice to have - although I would say that this could be handled through other mechanisms (like keyword search, collections etc) that did not add additional work for publishers or providers.
Have added some commentary to those specific examples below;
Low Impact Exercise for Health - there is already a collection for 'Low Impact' which groups together activity types that are deemed 'low impact' like Walking, Seated Exercise etc. We originally created a collection for this - which you can see a reference to in the 'Walking' concept , although the link no longer works
SEND Activities: This is an important one for us, although I feel that this could be something that is handled by 'accessibility'? However, more flexible collections could enable a collection to be created using various accessibility data points and activity types / providers for example.
Adaptive Games: This seems the most like a genuine activity type which would be useful to include in the list, it also may be possible to create a collection based on other data points that already exist.
GP Referral: Would be useful to see an example of this type of activity and a bit more info about the use case?
Pups on SUPs: This is just a name of a session, not an activity type. The activity type is Stand up Paddle boarding. Could be better handled through a category.


duplicated_computernames widget fix

Fix for name and report.

Thanks

No problem. Hopefully this is the first of many.
Sent from my iPhone

On Apr 16, 2015, at 10:55 AM, Arjen van Bochoven<EMAIL_ADDRESS>wrote:
Thanks
—
Reply to this email directly or view it on GitHub.


merging structs with opiton "WithOverride" not consistent

Hi,
In my opinion merging nested structs isn't really consistent when using the option "WithOverride". If you nest a struct "by value" or "by pointer" the behaviour of merge with override is different. (Possibly a duplicate of #14)
Example Code:
package main
import (
        "fmt"
        "log"

        "github.com/imdario/mergo"
)

type PtrTest struct {
        Str    string
        Number int
        Nested *Nested
}

type Test struct {
        Str    string
        Number int
        Nested Nested
}

type Nested struct {
        NestedStr    string
        NestedNumber int
}

func main() {
        dstPtr := &PtrTest{
                Str: "This is a non empty test string",
                Nested: &Nested{
                        NestedStr:    "This is a non empty string in a nested struct",
                        NestedNumber: 42,
                },
        }

        srcPtr := &PtrTest{
                Str:    "This is also a string",
                Number: 4,
                Nested: &Nested{},
        }

        if err := mergo.Merge(dstPtr, srcPtr, mergo.WithOverride); err != nil {
                log.Fatal(err)
        }
        fmt.Printf("Result with 'Nested' as pointer: %v\n", dstPtr.Nested)

        dst := &Test{
                Str: "This is a non empty test string",
                Nested: Nested{
                        NestedStr:    "This is a non empty string in a nested struct",
                        NestedNumber: 42,
                },
        }

        src := &Test{
                Str:    "This is also a string",
                Number: 4,
                Nested: Nested{},
        }

        if err := mergo.Merge(dst, src, mergo.WithOverride); err != nil {
                log.Fatal(err)
        }
        fmt.Printf("Result with 'Nested' as value: %v\n", dst.Nested)
}

This code prints the following two lines:
Result with 'Nested' as pointer: &{ 0}
Result with 'Nested' as value: {This is a non empty string in a nested struct 42}

Quote from the documentation: WithOverride will make merge override non-empty dst attributes with non-empty src attributes values.
I expected that behaviour also from nested structs (by pointer). Maybe this is just a misunderstanding of mine, because in my example the pointer to the nested struct isn't empty, but the element to which the pointer points is.
Actually if you remove the pointer to the empty element the example works like I'd expected it. However I got a use case which this solution isn't really applicable :smiley:.
Without looking at your code I think it should be possible to make a consistent use of "WithOverride" possible (also with my example). However I think it won't be very easy and maybe adds a bit of complexity.
If there is interest I can take a look at it and try to implement it. If not I think it should at least be documented for other people.
Please let me know your opinion about that.
Thanks for your work and regards,
Thomas

It took some time but I'm checking it now. I'm having problems to understand the full reach of your modification because the PR has smaller edits to unrelated things. Anyway, I'll do my best and merge as soon as possible.

Ok, I think now I get it. Sorry, I can't accept it as is. It doesn't feel safe to change behavior in such way.
Also, I ran your example code and I must say that it works as expected. Please follow my explanation in #52:
[...] As per MergeWithOverwrite's documentation:

MergeWithOverwrite will do the same as Merge except that non-empty dst attributes will be overriden by non-empty src attribute values.

[...] time.Time can be semantically empty (as in IsZero() returns true) but it is a struct. Structs in Go don't have a zero value by themselves, as described in the official documentation about zero value:

When storage is allocated for a variable, either through a declaration or a call of new, or when a new value is created, either through a composite literal or a call of make, and no explicit initialization is provided, the variable or value is given a default value. Each element of such a variable or value is set to the zero value for its type: false for booleans, 0 for integers, 0.0 for floats, "" for strings, and nil for pointers, functions, interfaces, slices, channels, and maps. This initialization is done recursively, so for instance each element of an array of structs will have its fields zeroed if no value is specified.

In other words, an empty struct isn't an empty value per se.
I'll accept your PR but I will merge it removing your edits in merge.go.


Adding more clarity of used configuration

Most users of this package do not care about all possible options, they are ok with any sane defaults.
However, it is not clear from the README that these defaults actually exists, neither what they are exactly.
I added some additional information to fulfill this gap.

Q
A

Is bugfix?
No

New feature?
No

Breaks BC?
No

Fixed issues
-

👍


joint sample VCF

Hi,
Thanks for the nice tool. Do you have some method (or future plan) for joint calling?
I tried merge to combine multiple samples' VCFs but am not sure whether this is a correct way to get reference homozygous genotypes..

I have a similar question. Is there a genotype option? To re-genotype all samples on the merged VCF? This is critical for larger cohorts.

Thanks for the suggestion, I agree this should be implemented. As suggested by @brentp, I think the option to re-genotype from an input vcf (from previous dysgu run or other SV caller) would be nice. I will try and get this implemented in the near future

FYI, dysgu now has a --sites option that can be used to genotype a list of candidate sites. Thanks for the suggestion.


Collection insert return null

Normally in mongodb after a record insert or delete it will return something like insertedId or id;
but here how do i return such a value after insert thanks!

I'll will try to solve asap


[Bug]: curl module provides non-functional http_only flag

What happened?
I want to use the curl module with https.  But default this module disables https support, but it provides a http_only flag:
https://github.com/bazelbuild/bazel-central-registry/blob/46c3c225b3f95ff91809eca5f86d77bf16b984e4/modules/curl/8.7.1/patches/add_build_file.patch#L24-L32
However, the flag has no effect. Whatever its setting https support is disabled.
Version
Development (host) and target OS/architectures:
For now Linux on x86. Eventually I need this to work on Windows, macOS, and many Linux variants.
Output of bazel --version:
bazel 7.2.1

Version of relevant rules from the WORKSPACE or MODULE.bazel file:
Language(s) and/or frameworks involved:
C++
How to reproduce
This program should work, but returns an error:
#include <iostream>
#include <curl/curl.h>

int main() {
  auto* curl = curl_easy_init();
  if (curl == nullptr) {
    std::cerr << "Error in curl_easy_init()\n";
    return 1;
  }

  curl_easy_setopt(curl, CURLOPT_URL, "https://pki.google.com/roots.pem");
    curl_easy_setopt(curl, CURLOPT_VERBOSE, 1L);

  auto res = curl_easy_perform(curl);
  if (res != CURLE_OK) {
    std::cerr << "Error in curl_easy_perform()\n";
  }
  curl_easy_cleanup(curl);
}

You also need a BUILD file:
package(default_visibility = ["//visibility:public"])

licenses(["notice"])  # Apache 2.0

cc_binary(
    name = "hello",
    srcs = ["hello.cc"],
    deps = ["@curl//:curl"]
)

And a MODULE.bazel file
module(name = "bzlmod-test")

bazel_dep(name = "curl", version = "8.7.1")

Any other information?
It seems to me that this module should have a dependency on boringssl to get https support.  The BCR patch  should also add USE_OPENSSL and HAVE_BORINGSSL and probably may another macros to local_defines.

The current issue should be fixed by either:
https://github.com/bazelbuild/bazel-central-registry/pull/2233
https://github.com/bazelbuild/bazel-central-registry/pull/1943
or https://github.com/bazelbuild/bazel-central-registry/pull/1917
the http_only flag is about disabling things like ftp, not about http vs https


fix(curriculum): Add missing quote in Python list example

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

This PR fixes a typo in the Python list example in the lambda functions lesson.
A missing quotation mark was added to correctly format the nested list example.

Hey @DFwJZ,
Is this PR related to some existing issue, which should be closed by this PR? If so, please link it correctly.
Here's the pull request template for reference
Checklist:

<!-- Please follow this checklist and put an x in each of the boxes, like this: [x]. It will ensure that our team takes your pull request seriously. -->

- [ ] I have read and followed the [contribution guidelines](https://contribute.freecodecamp.org).
- [ ] I have read and followed the [how to open a pull request guide](https://contribute.freecodecamp.org/how-to-open-a-pull-request/).
- [ ] My pull request targets the `main` branch of freeCodeCamp.
- [ ] I have tested these changes either locally on my machine, or GitPod.

<!--If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.-->

Closes #XXXXX

<!-- Feel free to add any additional description of changes below this line -->

Thank you, just updated.

Hi @DFwJZ. Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 📝


[ERROR] Read1 length is shorter

Hi,
I am obtaining this error when running CITE-SEQ-COUNT
[ERROR] Read1 length is shorter than the option you are using for Cell and UMI barcodes length. Please, check your options and rerun.
My barcodes are 12-bp length
UMI 8-bp length
I have checked R1.gz for the 12-bp barcodes and they are contained in these fq files
The following I am running which would be correct but is generating the above error messge:
CITE-seq-Count -T 4 -R1 R1.gz -R2 R2.gz -t tags.txt -cbf 1 -cbl 12 -umif 13 -umil 20 -cells 264 -wl whitelist.tsv -o samples_adt.out
Any suggestions would be great!
Thank you
Jennifer

Hello @jmosullivan and please forgive me the late response.
This is really strange. Can you double-check that your sequencing was done as expected?
Did you by any chance to some filtering prior to running CSC?

Closing this. Feel free to reopen if you have updates regarding this issue.


[BUG] Can't see city name when city has default image

Can't see city in FindCityInfoActivity when slider showing the default image
Bug screenshot

Expected behavior
See city name in any image color in the slider
Suggestion
Add TextView with good color for example colorPrimary to show city name
Suggestion screenshot

Would you like to work on this issue?
Yes

My suggestion would be: to add black contrast to background image, like in the Utilities section. The cards have reduced transparency with black background. White text will look good in this case.
Reason: your suggestion won't look good when I fully scroll up the screen.

Hey Guys
I don't think this is a gub put just FontColor issue, The city font color is just merging with the image color. We can change the image  or generate color according to the image

Can I fix this issue?

Can I fix this?

@Swati4star You are right i will implement your suggestion

@HeinKhantZaw Sorry bro i am working on it

Oh! Sry, I didn't see that you want to work on this issue.


NAs produced by integer overflow

Hi, I tried to use the function decontX passing as input a SingleCellExperiment object.
I got the following error:
--------------------------------------------------
Starting DecontX
--------------------------------------------------
Thu May 14 09:22:00 2020 .. Analyzing all cells
Thu May 14 09:22:00 2020 .... Converting to sparse matrix
Error: subscript contains NAs
In addition: Warning messages:
1: In (ans - 1L) * dim[, along] : NAs produced by integer overflow
2: In (ans - 1L) * dim[, along] + aind[, along] :
  NAs produced by integer overflow

thanks a lot for any help

Hi Nicola,
Can you send me the commands and ideally the SCE object you used to run decontX? Without the exact SCE object to reproduce this error, I might not be able to help you more.
Zhe
------------------ Original ------------------
From:  "Nicola<EMAIL_ADDRESS>Date:  May 14, 2020
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject:  [campbio/celda] NAs produced by integer overflow (#287)
Hi, I tried to use the function decontX passing as input a SingleCellExperiment object.
I got the following error:
-------------------------------------------------- Starting DecontX -------------------------------------------------- Thu May 14 09:22:00 2020 .. Analyzing all cells Thu May 14 09:22:00 2020 .... Converting to sparse matrix Error: subscript contains NAs In addition: Warning messages: 1: In (ans - 1L) * dim[, along] : NAs produced by integer overflow 2: In (ans - 1L) * dim[, along] + aind[, along] :   NAs produced by integer overflow
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

thanks for the fast reply.
Not sure if I can share the SCE (let me check this please).
Meanwhile these are commands that I used:
# create SCE object
sce <- read10xCounts(samples = cellranger_h5.file, type = 'HDF5',col.names = T)
# run decontX
dec <- decontX(x = sce)

Hi Nocola,
I am not able to reproduce the error you reported using the example data from read10xCounts documentation. My guess is the dataset you are trying to process is quite large and it failed to be loaded to memory during the conversion to dgCMatrix step.
Can you try this after you run read10xCounts:
assay(sce) <- methods::as(assay(sce), "dgCMatrix")
If this works maybe you can successfully run decontX next.
Best,
Zhe
------------------ Original ------------------
From:  "Nicola<EMAIL_ADDRESS>Date:  May 14, 2020
To<EMAIL_ADDRESS>Cc:  "Zhe<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject:  Re: [campbio/celda] NAs produced by integer overflow (#287)
thanks for the fast reply.
Not sure if I can share the SCE (let me check this please).
Meanwhile these are commands that I used:
create SCE object sce <- read10xCounts(samples = cellranger_h5.file, type = 'HDF5',col.names = T) # run decontX dec <- decontX(x = sce)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


Discussion: naming convention for ciReference

We should define what the git tag should be formed when we set it on the and used in the official release for input ciReference
Since they will be used for reproducible build, unless we decided to use SHA instead of tag in the build, we should never move tags around. create new one if needed
Suggestion: v<YY>.<MM>.01
e.g for 2022 Oct CPU release, v2022.10.01 if any hot bug fix, we can create v2022.10.02 and so on.
DoD:

what the name should be for tag
be clear the process: who will apply such tag and when it should be done pre-release
update https://github.com/adoptium/temurin-build/blob/master/RELEASING.md#steps-for-every-version

Ref: https://github.com/adoptium/temurin-build/issues/3155

@andrew-m-leonard @smlambert @sxa @karianna any thoughts on this?

Maybe really spell the tag out a bit more for CPU releases? I like it when it is clear what the numbers means, eg:
BuildScripts_2022.Oct_CPU
BuildScripts_2022.Oct_CPU.01

we will use below format as a start:

for trail test: vYYYY.MM.beforereleasetest
for official tag:  vYYYY.MM.0X  (when X starts with 1 and can be increased for patch build)


Add Object.values to lib declaration

Resolves #1449
Changes:

simply add Object.values() signature to Object definition.

@bassettsj Thanks! There was some previous work done on this over in #1335. That PR includes a comment from me with tests around this behavior. Do you think you could base your change off that one, and include tests?

@samwgoldman yeah, I think I can help. I have updated the current tests for the changes in the expected output from adding the two lines from #1335 and now I just need to work on covering the test cases from your comments.
What are the steps for writing the tests here, do I just simply provide the example cases where I would expect type errors to be thrown and then update the expected output from flow in the tests? I didn't see much documentation around writing the unit test here :grin:

do I just simply provide the example cases where I would expect type errors to be thrown and then update the expected output from flow in the tests?

Yes, precisely. Add the tests, verify the errors are what you'd expect, and update the .exp file (copy from .out or apply the .diff). If we make a change that alters the expected output, we check to see if that's a regression.

Great, maybe we should add those steps to the readme in a future PR :-)

add those steps to the readme

Yes!
Also, and I'm sorry for not being more clear about this, but the commits in #1335 are subtly wrong. See my comment there for all the details.
Thank you for contributing!


Switch to OpenStack Ussuri Release

Signed-off-by: Lance Albertson<EMAIL_ADDRESS>

!bump minor ~

Jenkins has merged this PR into master and has automatically performed a minor-level version bump to v5.4.0.  Have a nice day!


Amplitude automatic Version tracking

Expected Behavior
Amplitude Kotlin SDK should be tracking app Version and Start Version automatically as per doc: https://help.amplitude.com/hc/en-us/articles/215562387-Appendix-Amplitude-User-Property-Definitions
Current Behavior
Currently Amplitude Kotlin SDK which is being initialized like this:
 Amplitude(
            Configuration(
                apiKey = AMPLITUDE_KEY,
                context = androidApplication()
            )
        )

Is not traking versions automatically:

Environment

Unity Plugin Version: 1.1.1
Device: Pixel 6
Device OS and Version: Android 12

I have checked AndroidContextPlugin
It does have correct version name in context provider but amplitude gets this payload in the end:
{
  "$insert_id": "8e6b0c5a-793d-499c-9b12-bc099a05effd",
  "$row_source": "realtime",
  "$schema": 12,
  "_time":<PHONE_NUMBER>678,
  "adid": null,
  "amplitude_attribution_ids": [
    "6843d9761a0bc3373a5f4f0f12cfc408"
  ],
  "amplitude_event_type": null,
  "amplitude_id":<PHONE_NUMBER>67,
  "app": 332009,
  "city": null,
  "client_event_time": "2022-08-10 06:30:24.678000",
  "client_upload_time": "2022-08-10 06:30:23.421000",
  "country": "Ukraine",
  "data": {
    "group_first_event": {
    },
    "group_ids": {
    }
  },
  "device_brand": "google",
  "device_carrier": "lifecell",
  "device_family": "Google Phone",
  "device_id": "a2b14d66-bd79-4f5a-a555-fedc3f9fad6aR",
  "device_manufacturer": "Google",
  "device_model": "Pixel 6",
  "device_type": "Google Pixel 6",
  "display_name": "[Amplitude] Start Session",
  "dma": null,
  "event_id": 2,
  "event_properties": {
  },
  "event_time": "2022-08-10 06:30:24.678000",
  "event_type": "session_start",
  "group_properties": {
  },
  "groups": {
  },
  "idfa": null,
  "ip_address": null,
  "is_attribution_event": false,
  "language": "English",
  "library": "amplitude-analytics-android/1.1.1",
  "location_lat": null,
  "location_lng": null,
  "os": "android 12",
  "os_name": "android",
  "os_version": "12",
  "partner_id": null,
  "paying": null,
  "plan": {
  },
  "platform": "Android",
  "region": null,
  "sample_rate": null,
  "server_received_time": "2022-08-10 06:30:23.421000",
  "server_upload_time": "2022-08-10 06:30:23.422000",
  "session_id":<PHONE_NUMBER>344,
  "start_version": null,
  "timeline_hidden": false,
  "user_creation_time": "2022-08-10 06:30:24.678000",
  "user_id": null,
  "user_properties": {
    "system_theme": "light"
  },
  "uuid": "eb1cccdf-1875-11ed-8ea9-595a90196bcb",
  "version_name": null
}

Thank you for your feedback @gazanfarov . I'll check and fix the issue.

Hi @gazanfarov Please check out the v1.1.2 version for fix of this issue. Please let me know if you have any other questions.

Hey @bohan-amplitude thank you for update
Version is fine now but we do not also see user's city

Same thing with Region:

Also start version seems to be always writing the current one and not the one which was actually at the start of user life period

Why am I waisting our teams time on this half finished sdk
Can you guys please specify explicitly on the version of the sdk that it's a BETA release and it's not stable?

Hi @gazanfarov , sorry for the confusion. City, Region, Country and DMA are usually set by our backend service based on IP address using MaxMind database if not explicitly set in event body. We have observed that some IP addresses don't have City and Region in the MaxMind database. Please feel free to use other geo services and set these events attributes value during instrument. For Start version, it is actually a derived property which is set on the user when the first analytics event is ingested which contains a value for the Version user property. It's affected by this missing Version issue.

Thank you for your answer
We will get back to this SDK a little bit later 🙏


Add status, fix message ordering, added full executeRequest unit test

For #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

:exclamation: No coverage uploaded for pull request base (ds/rawKernel@01e1a64). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             ds/rawKernel   #10698   +/-   ##
===============================================
  Coverage                ?   60.54%
===============================================
  Files                   ?      585
  Lines                   ?    31750
  Branches                ?     4502
===============================================
  Hits                    ?    19223
  Misses                  ?    11542
  Partials                ?      985

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 01e1a64...bfdff86. Read the comment docs.

            session: sessionId,

Are you sure this doesn't have to match the session id from the very beginning? How else does the kernel now which session the message is a part of?
See the comments here:
https://jupyter-client.readthedocs.io/en/stable/messaging.html#general-message-format

Refers to: src/test/datascience/raw-kernel/rawKernel.functional.test.ts:93 in f813dc0. [](commit_id = f813dc0badd5a08c3fc95458ccf8389e50cc8e03, deletion_comment = False)

🕐

sessionID is the client session ID. It's not relevant to your functional test. Your test is just looking for a reply on the ZMQ layer that will happen regardless of what the session is. That session ID is only checked a level up at the kernel level.

🕐

sessionID is the client session ID. It's not relevant to your functional test. Your test is just looking for a reply on the ZMQ layer that will happen regardless of what the session is. That session ID is only checked a level up at the kernel level. @rchiodo

Ah okay, you just have to ensure it's always the same when creating your messages. The lower layer doesn't care.

🕐

sessionID is the client session ID. It's not relevant to your functional test. Your test is just looking for a reply on the ZMQ layer that will happen regardless of what the session is. That session ID is only checked a level up at the kernel level. @rchiodo

Ah okay, you just have to ensure it's always the same when creating your messages. The lower layer doesn't care.

@rchiodo . Yup, before I was accidentally passing in a session ID to ZMQ. But that identity is just a ZMQ identity, so I removed it in this PR.

Functional tests did timeout, but they passed before a minor change and the Linux ones passed. So I think this is just Windows timing issues. So pushing.


use and label lead Next to think no response was sent

Describe the bug
Any async endpoint that I pass middleware to using use or label ultimately responds to the front-end as expected, but it leads Next to log the below string in development.
API resolved without sending a response for /api/<rest of uri>, this may result in stalled requests.

To Reproduce
Steps to reproduce the behavior:

Create a function withMiddleware as shown below;
Create a basic asynchronous handler.
Run the application
Call the endpoint
Observe the logs in the terminal

const withMiddleware = label({
  development: [morgan("dev") as ExpressMiddleware],
});

export default withMiddleware(process.env.NODE_ENV)(
  async (req: NextApiRequest, res: NextApiResponse) => {
    await wait(0);
    res.status(200).send([{ value: BookmarkStatus.Saved }]);
  }
);

// helper function
function wait(ms: number = 0) {
  return new Promise(res => setTimeout(res, ms));
}

Expected behavior
I expect everything to work the way it does, except that Next should process that the handler is working.
You can see the expected behavior by removing the call to wait from the sample code.
Additional context
I haven't tried any other methods, so I am explicitly calling out the two methods that I have tried

Hi @Tbhesswebber – I just attempted to reproduce this example. I think there is a typo in the wait function that could be causing this to fail.
This helper function doesn't actually call the promise resolve argument.
function wait(ms: number = 0) {
  return new Promise(resolve => setTimeout(resolve, ms));
}

If you modify this function locally to invoke the resolve argument, the API handler and middleware execute correctly.
function wait(ms: number = 0) {
  return new Promise(resolve => setTimeout(() => {
    resolve();
  }, ms));
}

Do you still see an issue after updating the wait function?

Here's a sandbox that still has the issue after the change to the wait function (if that change had fixed things, we'd have found a bug in JavaScript, which would have been cool!).
https://codesandbox.io/s/flamboyant-wind-8ed1e?file=/pages/api/hello.js

Serendipitously, I happened to randomly see this issue right after browsing the Nextjs API documentation. I noticed that the Nextjs API docs for a Custom config happens to include this snippet:

externalResolver is an explicit flag that tells the server that this route is being handled by an external resolver like express or connect. Enabling this option disables warnings for unresolved requests.

It seems possible that this issue is related to that externalResolver config option and, if so, the warning should be ignored in the context of using this middleware. How Nextjs can detect that middleware is being used is not clear. It's also not obvious to me how wrapping an API handler in middleware would produce any detectable difference from the framework's perspective.
Side note:
function wait(ms: number = 0) {
  return new Promise(res => setTimeout(res, ms));
}

I think there is a typo in the wait function that could be causing this to fail. This helper function doesn't actually call the promise resolve argument.

Incorrect. See the definition of setTimeout.  setTimeout(res, ms) will call the res function after a ms delay. res is just a function in the same way that () => res() is just a function.

How Nextjs can detect that middleware is being used is not clear. It's also not obvious to me how wrapping an API handler in middleware would produce any detectable difference from the framework's perspective.

To me, this would imply that Next is expecting a promise to be returned (even if that promise resolves to undefined via an implicit return).
I did know about the external resolver property, but part of the reason I went the route of using this package rather than leveraging an express server was because I didn't want to move to a fully managed API and lose any potential optimizations that Next might offer in the future.

To me, this would imply that Next is expecting a promise to be returned (even if that promise resolves to undefined via an implicit return).

I also thought this might be the case, but I believe the example hello.js API route that is created when you initialize a new Nextjs app does not return a promise (and does not log a warning). Additionally, if you change the hello.js function into an async function (i.e. force it to return a promise), Nextjs does not log any warnings to the console.

This is all based on assumption, so I may be wrong (and will gladly admit it if someone who knows can tell me).
When a handler isn't async, Next knows if it threw an error because of the single threaded nature of JavaScript. When you make it async, it returns a promise by default (the equivalent of Promise.resolve(undefined)) and Next is able to wait for that promise to resolve or reject and handle it appropriately. It's when you get into more complex situations (like piping requests and responses through an array of middleware), things get tricky.

I took a crack at refactoring how middleware gets executed, and I believe this may handle callback-style middleware and async middleware correctly.
Would you mind cloning and seeing if it resolves the error you've encountered? @Tbhesswebber @jorroll
PR: https://github.com/htunnicliff/next-api-middleware/pull/12

@htunnicliff sorry for the miscommunication. I've never encountered this error with next-api-middleware. I simply noticed this issue and was reminded of something I read in the nextjs API docs so figured I'd share the information.

@Tbhesswebber this bug should be resolved in v1.0.0.


[FEATURE] add focusIn and focusOut events to element control

I'm in the process of building a datepicker control in the same way that you've implemented your addon for ember-power-select.
In doing so, I wanted to render a bs-form/element/control/input  and listen to the focusIn event to open my datepicker and on focusOut close it. These event's aren't public on either the element or it's control, though in my case I'm interested in setting it on the control itself.
Obviously we'd need to ensure that the event on the control is still handled as expected by the wrapping element...

If I understand you right, you want to create a bs-form/element/control/datepicker (or similar) component and use bs-form/element/control/input internally in its template, right?
As the focusin event bubbles up the DOM tree, you should be able to listen to this event in your bs-form/element/control/datepicker, with a simple event handler method like this:
...
focusIn(e) {
  // handle the e event
}

Hope that works for you!?

@simonihmig thank you for coming back to me - I've actually resolved this like you suggest by following the example here
Is this way of defining custom control officially supported as it's not documented.

Good to hear @fran-worley! :)

Is this way of defining custom control officially supported as it's not documented.

Good question! It is not really public API yet, and thus not documented. But it is used (not only be me! 😉), so it won't go away, at least not without a prior deprecation and a clear alternative. So it should be "ok-ish" to use! Hope that's good enough for you!? 😬


bootstrapping using environment variable credentials fails if a shared credentials file exists

bootstrapping using credential env variables (AWS_ACCESS_KEY_ID, ...) always fails if a shared credentials file (~/.aws/credentials) exists with this error:
AssumeRoleCredentialPlugin cannot obtain credentials for role cdk-writeRole
canProvideCredentails for read role: false
canProvideCredentails for write role: false
 ❌  Environment aws://111111111111/eu-central-1 failed bootstrapping: Error: Need to perform AWS calls for account<PHONE_NUMBER>11, but the current credentials are for<PHONE_NUMBER>22, and none of these plugins found any: AssumeRoleCredentialPlugin

Turns out the credentials given in the env variables are not used due to the order of the credential provider chain.
https://github.com/aws-samples/cdk-assume-role-credential-plugin/blob/77066554a05e58fb1cf94e45a8a6791fc385095b/lib/assume-role-source.ts#L227-L242
While I understand from the comment that there is a specific reason to reorder the credential provider chain concerning the priority of AWS.ECSCredentials() I think it would be less surprising to stick to the default order for the rest of the chain.

I am no longer planning on maintaining this repository and will be archiving it soon. I've updated the README to indicate that I no longer recommend using this plugin since the functionality has been made the default in CDK v2.


First DM Stack tutorial notebook

Make a first tutorial notebook to exercise DM Stack (and hence check that people are set up correctly). This could be “Hello Sky” (#13 ) or something else - perhaps one of the v15 HTML tutorials? Assigning to Alex, who wanted to write a notebook :-) Belongs to epic #15

The "first notebook" desire was satisfied through lsst-sqre/notebook-demo#2. I think this could be closed, though we need to decide how to share tutorials across the groups.


Made lines for tables and linebreaks lighter

#13 mentioned lines for tables and linebreaks being too dark for viewing. I made them lighter to fix this by changing the background-modifier-border in the _base.scss file and compiling it
Example here

Right on, thanks for your contribution!


Qualification tool backport: Check for metadata being null (#3285)

clean cherry-pick of https://github.com/NVIDIA/spark-rapids/pull/3285 commit https://github.com/NVIDIA/spark-rapids/commit/59e214fb2f20d31a4815eebb71f99bb1c8b0440b back into 21.08.
Signed-off-by: Thomas Graves<EMAIL_ADDRESS>

build


Will swagger-core support kotlin class?

This is an expected feature, not a issue ^_^
I expect swagger annotations(typically @ApiModel, @ApiModelProperty) could be used with kotlin class, currently these annotations are not compatible with kotlin class, many properties could not be recognized.
I'm wondering if there is any plan for this?

It's hard to say without more details. I suggest looking at the 3.0.0-rc2 branch and reporting what doesn't work

Hi, thanks for reply.
It is my mistake, I defined both @ApiModel(value = "xxx"), where xxx is identical in both classes. after modifying the value, the swagger output is fine.
you may close this issue.


Improvement make sync rf size work when need_local_merge

Proposed changes
make sync rf size work when need_local_merge
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Use single json field tag name

Currently, the field tag has multiple names that increase the risk of conflicts with other field tags from other packages. Breaking change.
Currently
type Req struct {
  B int `path:"b"`
  C int `query:"c"`
  D int `body:"json"`

  E []int `query:"e,implode,pipeDelimited"`
}

Next
Use single field tag "oas" (OpenAPI Specification). All settings are comma-separated as before. Additionally, a new origin setting is added:

path
query
body

type Req struct {
  B int `oas:"b,path"`
  C int `oas:"c,query"`
  D int `oas:",body,json"` // if name is provided, it is ignored

  E []int `oas:"e,query,implode,pipeDelimited"`
}

I'd rewrite
  D int `oas:"d,body,json"`

I.e. [name][where][options]

I don't want to enforce the sequence, it makes it harder to use. The setting names do not conflict, they can be used in any order.

In principle, the syntax is <name>,<settings> where <settings> is a comma-separated list of settings.


Permissions error

Describe the bug
Trying to run an expo project throws error
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-xsmdau
Steps to reproduce
npm ERR! code EACCES
npm ERR! syscall mkdir
npm ERR! path /usr/local/lib/node_modules/expo-cli/node_modules/@babel/runtime
npm ERR! errno -13
npm ERR! EACCES: permission denied, mkdir '/usr/local/lib/node_modules/expo-cli/node_modules/@babel/runtime'
npm ERR!  [EACCES: permission denied, mkdir '/usr/local/lib/node_modules/expo-cli/node_modules/@babel/runtime'] {
npm ERR!   code: 'EACCES',
npm ERR!   errno: -13,
npm ERR!   path: '/usr/local/lib/node_modules/expo-cli/node_modules/@babel/runtime',
npm ERR!   syscall: 'mkdir'
npm ERR! }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator.
npm ERR! A complete log of this run can be found in: /home/.npm/_logs/2024-10-08T16_24_20_193Z-debug-0.log
jsh: spawn npm EACCES
~/projects/sb1-xsmdau 10s
Expected behavior
Run the app
Screen Recording / Screenshot
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Hey @GonzaloSP! Looks like an issue with WebContainer. Thanks for the link to the project!
I've added this to our internal tracker.

Thank you, @Nemikolh ! Does this mean that no Expo project is currently working? Do you have an estimate for when the fix might be available?


There was a problem sending your payment. FinalExpiryTooSoon - 23 satoshis

There was a problem sending your payment. FinalExpiryTooSoon

23 satoshis
Can someone clarify what this message means exactly, resp. how to fix it.

Hi @wdey .
This error comes directly from lnd and happens when the last HTLC of a route (exit hop) has an expiration height that is deemed too soon by the final destination of the HTLC.
if lastHtlc.cltvExpiry < expectedHeight.
See https://github.com/lightningnetwork/lnd/blob/42c45979215e3242ef2e162ccd9de3cbb0ed28a1/htlcswitch/link.go#L2286-L2300 for implementation details.


Test

https://docs.microsoft.com/en-us/azure/devops/repos/git/pullrequest?view=azure-devops

@AllAppRegression1  please, next time put a more explanatory title in the PR! Thanks!


locale(ro): New translations added for Romanian

Thanks for the PR. LGTM, we'll see about getting this merged before next update.

Thanks for the PR.


Docz

@fernandofleury @felixjung I will need your approval to proceed. The build was not working Travis CI, so I had to revert the commit, rebase, and push further changes. (file name capitalisation conflicts and updated .travis.yml)
You can see here that it builds docz successfully:
https://travis-ci.org/sumup/circuit-ui/builds/421602115


Documentation License

This git repo for documenting the Xilinx 7-series bit-stream format says in two places (in readme and contributing) that "all code in the Project X-Ray repository is licensed under the very permissive ISC Licence." (emphasis added.) The license itself also says permission is being granted for the software (emphasis also added.) Does that not include the documentation of the Xilinx 7-series bit-stream format that's being made? Why only the software and not everything?

The database itself is available under CC0-1-Universal here.

Thanks but I mean the non-code/non-software stuff in this one. An example being the docs directory. Since it's not code I take it that it doesn't fall into the ""all code in the Project X-Ray repository is licensed under the very permissive ISC Licence." Why only the software and not everything in this git repository? The vtr-verilog-to-routing repo makes it quite clear by comparison that it includes everything and not just the code/software itself.


Fix test: hash value depends on endianness and bitness.

The test fails on 32-bit systems and on big-endian systems since Rust 1.44.
Fixes #10004.

@bors r+

:pushpin: Commit aa00def843c8b994ce6e0df71a7b5d4bd00b9612 has been approved by joshtriplett

:hourglass: Testing commit aa00def843c8b994ce6e0df71a7b5d4bd00b9612 with merge e11cd814f8ec5d11a65ae3aea7c3976f250d36a6...

:sunny: Test successful - checks-actions
Approved by: joshtriplett
Pushing e11cd814f8ec5d11a65ae3aea7c3976f250d36a6 to master...


While klipper starts following error rises

Error loading template 'gcode_macro HEAT_BED_ZONE:gcode': jinja2.exceptions.TemplateSyntaxError: Encountered unknown tag 'zone_temps'. Jinja was looking for the following tags: 'elif' or 'else' or 'endif'. The innermost block that needs to be closed is 'if'.

Thanks for reporting! I finally got to put it into klipper. Haven't actually tested the macro yet, but at the least it won't prevent klipper from starting!

Forgot to close :)


remove Fragment support

Removes the fragment specific navigation module, the fragment codegen and
fragment annotation. With this there are a lot of things that can be
reorganized or further improved but I'll keep that to other prs.

Current dependencies on/for this PR:

main

PR #636   👈

PR #637

PR #638

This stack of pull requests is managed by Graphite.


HIVE-26681:Upgrade dom4j: flexible XML framework for Java to safe ver…

…sion due to critical CVEs

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@nrg4878 please see this


[Problem tutorial ja] "Header Parameters" highlight lines mismatch

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[ ] I searched the FastAPI documentation, with the integrated search.
[ ] I already searched in Google "How to X in FastAPI" and didn't find any information.
[ ] I already read and followed all the tutorial in the docs and didn't find an answer.
[ ] I already checked if it is not related to FastAPI but to Pydantic.
[ ] I already checked if it is not related to FastAPI but to Swagger UI.
[ ] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to one of:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Example
As is: https://fastapi.tiangolo.com/ja/tutorial/header-params/#_2
To be: https://fastapi.tiangolo.com/tutorial/header-params/#automatic-conversion
Description
As is: In fastapi/docs/ja/docs/tutorial/header-params.md
hl_lines="9"
To be: hl_lines="10"
like fastapi/docs/en/docs/tutorial/header-params.md

Thanks! I think this was fixed at some point. It seems to be correct now, right?

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


firefox: terminal characters overlapping

I'm getting a weird issue viewing the code-server terminal via Firefox. This just started happening recently.
code-server: latest Docker image
Firefox v88.0.1: character overlap
Chrome v90.0.4430.212: no character overlap
Steps to try and resolve:

Cleared cache & cookies in FF
Restarted the Docker container
Tried resizing the window & terminal

fixed itself!


Revert "Removed duplicate rishiqing entry"

Just read the comment asking for further changes on this PR. reverting

Thanks, @alichtman!
I will clean up the Chinese version in a few hours.


Will submit a dirty inefficient fix for now.

Why don they have to be strings? I use and return arrays in qtc. No problem what so ever. But maybe I just don't get the problem.

It's about the timestamps-keys of the dictionaries in both World_Trace.trace and World_QSR_Trace.trace. It only affects time-dependent QSRs (like QTC) depending on whether you are calling the get_sorted_timestamps and you parse via it; it has no effects on single time QSRs (like RCC3). The effect is easier to show with the following MWE:
def bad_previous_sort(l):
    return sorted(timestamp_keys)

def good_new_sort(l):
    ret = l
    foo = ret[0]
    ret = [int(i) for i in ret]
    ret = sorted(ret)
    if type(foo) is str:  # if they were str make them strs again
        ret = [str(i) for i in ret]
    return ret

timestamp_keys = ["2", "1", "20", "5", "4", "22", "9", "6", "7", "8", "11", "10"]
print "bad_previous_sort:\t", bad_previous_sort(timestamp_keys)
print "good_new_sort:\t\t", good_new_sort(timestamp_keys)

bad_previous_sort:    ['1', '10', '11', '2', '20', '22', '4', '5', '6', '7', '8', '9']
good_new_sort:        ['1', '2', '4', '5', '6', '7', '8', '9', '10', '11', '20', '22']

Would not have been a problem if they were numerics instead of strings.


CUDA out of memory

I have 24GB of ram, and i tried to generate a 512 by 512 image and i still get this!
Traceback (most recent call last):
File "F:\OmniGen\venv\lib\site-packages\gradio\queueing.py", line 624, in process_events
response = await route_utils.call_process_api(
File "F:\OmniGen\venv\lib\site-packages\gradio\route_utils.py", line 323, in call_process_api
output = await app.get_blocks().process_api(
File "F:\OmniGen\venv\lib\site-packages\gradio\blocks.py", line 2018, in process_api
result = await self.call_function(
File "F:\OmniGen\venv\lib\site-packages\gradio\blocks.py", line 1567, in call_function
prediction = await anyio.to_thread.run_sync(  # type: ignore
File "F:\OmniGen\venv\lib\site-packages\anyio\to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "F:\OmniGen\venv\lib\site-packages\anyio_backends_asyncio.py", line 2441, in run_sync_in_worker_thread
return await future
File "F:\OmniGen\venv\lib\site-packages\anyio_backends_asyncio.py", line 943, in run
result = context.run(func, *args)
File "F:\OmniGen\venv\lib\site-packages\gradio\utils.py", line 846, in wrapper
response = f(*args, **kwargs)
File "F:\OmniGen\app.py", line 22, in generate_image
output = pipe(
File "F:\OmniGen\venv\lib\site-packages\torch\utils_contextlib.py", line 116, in decorate_context
return func(*args, **kwargs)
File "F:\OmniGen\OmniGen\pipeline.py", line 278, in call
samples = scheduler(latents, func, model_kwargs, use_kv_cache=use_kv_cache, offload_kv_cache=offload_kv_cache)
File "F:\OmniGen\OmniGen\scheduler.py", line 162, in call
pred, cache = func(z, timesteps, past_key_values=cache, **model_kwargs)
File "F:\OmniGen\venv\lib\site-packages\torch\utils_contextlib.py", line 116, in decorate_context
return func(*args, **kwargs)
File "F:\OmniGen\OmniGen\model.py", line 387, in forward_with_separate_cfg
temp_out, temp_pask_key_values = self.forward(x[i], timestep[i], input_ids[i], input_img_latents[i], input_image_sizes[i], attention_mask[i], position_ids[i], past_key_values=past_key_values[i], return_past_key_values=True, offload_model=offload_model)
File "F:\OmniGen\OmniGen\model.py", line 338, in forward
output = self.llm(inputs_embeds=input_emb, attention_mask=attention_mask, position_ids=position_ids, past_key_values=past_key_values, offload_model=offload_model)
File "F:\OmniGen\venv\lib\site-packages\torch\nn\modules\module.py", line 1553, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "F:\OmniGen\venv\lib\site-packages\torch\nn\modules\module.py", line 1562, in _call_impl
return forward_call(*args, **kwargs)
File "F:\OmniGen\OmniGen\transformer.py", line 156, in forward
self.get_offlaod_layer(layer_idx, device=inputs_embeds.device)
File "F:\OmniGen\OmniGen\transformer.py", line 52, in get_offlaod_layer
self.evict_previous_layer(layer_idx)
File "F:\OmniGen\OmniGen\transformer.py", line 43, in evict_previous_layer
param.data = param.data.to("cpu", non_blocking=True)
RuntimeError: CUDA error: out of memory
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.

I don't know how you installed, because repo doesn't have proper installation instructions, but very likely installation error
i made 1-click Python 3.10 installers and works as low as 5.5 gb : https://github.com/VectorSpaceLab/OmniGen/issues/86

Cool I will try yours
On Sun, 3 Nov 2024, 5:23 pm Furkan Gözükara, @.***>
wrote:

I don't know how you installed, because repo doesn't have proper
installation instructions, but very likely installation error
i made 1-click Python 3.10 installers and works as low as 5.5 gb : #86
https://github.com/VectorSpaceLab/OmniGen/issues/86
—
Reply to this email directly, view it on GitHub
https://github.com/VectorSpaceLab/OmniGen/issues/90#issuecomment-2453390992,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3IB2HS5BIYRLRNK5XB5QXLZ6YBTTAVCNFSM6AAAAABRCTVA26VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGM4TAOJZGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Multiple destinations in loop fails

A project I am working on requires a number of themes, compiled into different directories, which inherit from a master theme. When compiling a single directory, gulp-ruby-sass works fine, however mapping the output for each individual theme into it's own directory breaks. (Using 1.0.0-alpha)
var gulp = require( 'gulp' ),
    sass = require( 'gulp-ruby-sass' ),
    fs = require('fs'),
    path = require('path');

var config = {
  default: 'default',
  skin_dir: 'skin/frontend/theme'
};

function getFolders(dir) {
    return fs.readdirSync(dir)
      .filter(function(file) {
        return fs.statSync(path.join(dir, file)).isDirectory();
      });
}

gulp.task('styles', function() {
    // Multiple values do not work
    var dirs = getFolders(config.skin_dir);
    // But a single value does work
    // var dirs = ['default'];
    var compile = dirs.map(function(dir){
        var theme_dir = config.skin_dir + '/' + dir;
        return sass( theme_dir + '/scss/style.scss',
            {
                sourcemap: true,
                loadPath: config.skin_dir + '/' + config.default + '/scss',
                style: 'expanded'
            })
            .on('error', function (err) {
              console.error('Error', err.message);
            })
            .pipe(gulp.dest( theme_dir + '/css' ));
    });
});

gulp.task('watch', function() {
    gulp.watch(config.skin_dir + '/**/*.scss', ['styles']);
});

gulp.task('default', ['styles', 'watch']);

The errors I get vary, which leads me to believe that this is a problem with synchronicity.
Is this the preferred way to accomplish this? If not, how should this be accomplished?

What are some of the errors? Remember to use the container option.

A common error is:
fs.js:696
  return binding.stat(pathModule._makeLong(path));
                 ^
Error: ENOENT, no such file or directory '/var/folders/5h/xcq6y2pn6_g3ccgwz4q8v3g80000gn/T/gulp-ruby-sass/style.css.map'
    at Object.fs.statSync (fs.js:696:18)
    at Object.<anonymous> (/www/node_modules/gulp-ruby-sass/index.js:158:12)
    at Object.immediate._onImmediate (timers.js:372:16)
    at processImmediate [as _immediateCallback] (timers.js:354:15)

Can you try adding container: somestring to options, where somestring is a var that's set per directory. Maybe use forEach instead of map and then use the index?

@robwierzbowski container worked like a charm (facepalm) - thanks for your help!

No problem! Figured it was a race condition with container deletion.


Deprecation "A property of... was modified inside the didInsertElement hook." after upgrading to ember 1.13

Hi,
I am getting the following deprecation warning from the component after upgrading to ember 1.13

A property of... was modified inside the didInsertElement hook. You should never change properties on components, services or models during didInsertElement because it causes significant performance degradation.

Have same deprecation notice with latest-ember branch

:+1:


Flicker if using PVR

Tried your addon.
It works for music (milkdrop) and movies, but if I use PVR (DVBLink), the screen is flickering extremely.
Do you already know of this problem?

Hello there,
Your issue lacks a bit details, could you add a bit more such as which kodi versions / hue plugin version / dvblink versions, what platforms (arm/x86/win-lin-android?).
I don't know if logs would help but the more the better :)

Hi.
I use the latest Kodi 17.1 but I had the same problem with older version (tested the addon some time ago from michaelrcarroll.
The Hue plugin version ist your version from here.
DVBLink is the version which is included in Kodi (3.4.2).
I use Kodi with Windows10 x64 at a ZBOX ID92 (Intel i5 Proz. with Intel 4600 grafik).
The problem only occure if using LiveTV.
If I can help with additional infos (photo of the screen, etc.) then I will.
Thanks for help.

Hello,
I'm just trying to help where I can, I'm not the plugin author nor maintainer, just a modest contributor ;)
Anyway, I remember liveTV worked properly for me but I was using "iptv simple" and online channels (my ISP provides a .m3u playlist) so your problem looks like a conflict between the hue plugin and dvblink.
May I ask you to test if the bug also happens with another liveTV source ? (like another plugin)
It might also be a good thing to open an issue here : https://github.com/kodi-pvr/pvr.dvblink/issues
As they might know what is causing this and/or have good suggestions.

Unfortunately I myself don't have anything LiveTV capable, thanks @Moyster for the tips provided. Leaving this open - maybe someone will figure out the root cause.

I also have the problem while watching Live-TV. I'm using latest Kodi 17.4 with your version (downloaded yesterday).
Kodi is using TVHeadend as PVR. Movies are working like a charm but Live TV is like a stroboscobe Green/Image/Green/Image.
Until this issue could be resolved: Maybe you can add a "disable for Live TV"-Option?

I also had this problem, i found this on the kodi forum:
https://forum.kodi.tv/showthread.php?tid=222245
Both solutions mentioned there work.
Now i have interlacing turned off for all videos.
No more green flicker.