TPM has no attribute self.feature_names_in_

Hi,
I am trying to do TPM analysis on bulk rna sequencing data. This is an example of code I have thus far
import rnanorm

counts = pd.read_csv("gene_counts.csv")
gene_sizes = pd.read_csv("gene_sizes.csv")

tpm = rnanorm.TPM(gene_lengths=gene_sizes)
tpm.fit_transform(counts)

I am getting the following error:
  File "/Users/joshl/miniconda3/envs/como/lib/python3.10/site-packages/rnanorm/methods/within_sample.py", line 119, in _get_gene_lengths
    missing = set(self.feature_names_in_) - set(gene_lengths.index)
AttributeError: 'TPM' object has no attribute 'feature_names_in_'. Did you mean: 'feature_names_out'?

This is the first bit of my counts data frame. It has three columns (naiveB_S3R1, naiveB_S3R2, naiveB_S3R3), with the gene counts as rows. I have extra columns (Entrez gene IDs and ensemble gene IDs) that I removed from the dataframe based on the toy dataset example
       naiveB_S3R1  naiveB_S3R2  naiveB_S3R3
0             1678         1303          343
1              931         1076          355
2               46           88           11
3                0            1            0
4               36            9            3

And this is the first bit of the gene_sizes pandas Series. It is a single column where each row corresponds to the row in the counts dataframe
12883
14949
24273
44636
192073

I appreciate any help you can give!

Having samples as rows may be the root of my issue - thank you for the speedy response! I will be sure to test this today

Closing this as complete because I was able to execute the TPM function with these suggestions


mini + extended mwc-fab

An extended fab appears to be ignoring the miniattribute and remains 48px tall:

It would be useful to be able to make it mini (40px tall):

Hello, I was speaking to the design team and they think extended is incompatible with mini. Do you have any context on where you would be using it?

We use it at the bottom of the page to load more items. Need something more subtle as full sized is a bit too heavy.
All that's needed is to switch the order css height property is applied - first on .mdc-fab--extended and then on .mdc-fab--mini

Also, if extended can only affect the width and mini in this case the height - why would they be incompatible?

Passed your info along. By incompatible I think the designers mean with the design meaning of mini.

This may also be related to density, since this is more related to a "dense" extended FAB instead of an extended mini FAB, which we don't have design guidance for.
We'll put this in our backlog for now until we get more guidance from design on how FAB should handle density.

Like the Customize fab on the default Chrome page

Soon we will be updating fab to use design tokens which should handle this case. There will be a new custom property to handle fab-extended-height

Fixed in M3


fix: splitting crlf lines

#130

@UnderCooled Could you confirm fix? :)

I'm using vim.split(results, '\r?\n') in my "codespell" linter parse function, works fine here. I'm still migrating my null-ls config to guard.nvim.


Allow App to be Hosted at Subdirectory

Problem description
The app fails if it is not hosted at root (i.e. /).
Steps to reproduce the issue

Checkout add-tools-stack branch of aws-cloudformation repo
Run docker-compose up --build
Navigate to http://localhost/phi-deidentifier
You should see a blank page.

Expected behavior
You should see the app, not a blank page.
Extra information
React documentation has a guide for serving React apps from subdirectories

This would offer a greater flexibility but this is no longer needed.


Support multi loader of same extensions

What problem does this solve or what need does it fill?
The bevy_gltf does not support the other gltf extensions. For example, It does not support the MOZ_hubs_components extensions export by blender plugin hubs-blender-exporter.
So I want to add support by my custom asset loader, But when I add a custom asset loader for gltf and glb, It will override the bevy_gltf loader. the AssetServer::add_loader method will override the previous loader. So is it possible to support multi loader for one extension, not just override it.
What solution would you like?
So is it possable to support array loader

With current Bevy, you can set a custom asset loader for myfilename.hubs.gltf and it would not use the builtin gltf loader.
You could also disable the builtin gltf loader to only have yours, that could then work on any gltf file

Maybe it can make the method load_gltf in bevy_gltf loader to be public, so I  can use it in my custom loader.


winget -import support URL endpoints

Description of the new feature / enhancement
Trying to figure out ways to syndicate profiles of applications for associates.  I am sort of hacking it in with Intune Company Portal today by including the JSON in the intunewin file, but maintenance is a bit of a pain.
Be great if associates could provide links to a gist or other raw JSON source (supporting authentication a plus) instead of only a file path.
I can have folks also do a git clone, but git client may be part of the profile so becomes a bit of a chicken and egg.  And curl is another option to localize the file, but be great to have it native.
Also going to open an issue on the restsource repo as if import files were first class endpoints than it helps solve a number of these needs.
Proposed technical implementation details
https://github.com/microsoft/winget-cli/blob/master/src/AppInstallerCLICore/Commands/HashCommand.cpp#L45 basically assumes a local file.  It would need to check if its local or a URL and then use the existing DODownloader to grab the file before creating the inputFile work.
If supporting authenticated destinations things get more complicated.

Duplicate of #1118


Can't input characters any more.

see https://github.com/atom/atom/issues/11529
Seems to be a vim-mode issue, since deactivating all packages but vim-mode made it occur, whereas deactivating all packages it did not occur. Happens while changing from input to normal mode and then back to input. Happens sporadically, like twice per hour.

As stated in the README, this package is no longer maintained and is deprecated. We recommend that people use the vim-mode-plus package instead. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


Word aligned slices?

I have a use case where I’m writing the serialised slice to nvmc, which requires the slice to be word aligned. I’m wondering if it’d be useful then to have a variant of to_slice that provides a word aligned slice. Thoughts? Thanks.

Hey @huntc, you mean the destination needs to be word aligned?
IMO, this is more of a property of the caller, e.g. you should make sure your destination slice is word aligned before calling to_slice, either using a wrapper type, like:
#[repr(align(4))]
struct DestinationBuf<const N: usize> {
    bytes: [u8; N]
}

or some wrapper method that "skips" the first unaligned bytes.
Is there some design you had in mind for how postcard could handle this instead?

Sorry for the slow reply. And thanks for the answer.


Issue#1936 annotation

Turns out I refactored out some support for the multi-line annotations. I wasn't able to recreate the other issue but I tested it pretty thoroughly and found some other small things that snuck by (including one that would have been existing for the previous annotation setup).

Yikes, I think I got all the major hound fixes. I'm not sure if I should change those two camel cases issues because they are from another class and that might make hound examine it as well. A bunch of these comments were from things I didnt even change

@spring-ryanc I'm okay not fixing those changes (Hound is a bit odd). But the tests are failing - could you look into this?

@david-yz-liu You can create a config file based on the defaults to turn things like camel case variables off.

@spring-ryanc Great job!


fix: close  AWS config

Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Can you update the description section of this PR? the testing part as well

@testwill Are you still working on this? Any updates from the last review?

@testwill Are you still working on this? Any updates from the last review?

no

@testwill Are you still working on this? Any updates from the last review?

no


tmuxPlugins.tmux-fzf: unstable-2022-08-02 -> unstable-2023-07-06

Description of changes
tmuxPlugins.tmux-fzf: unstable-2022-08-02 -> unstable-2023-07-06
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2431


start and stop accepting connections

To allow denying new connections before they're established, the transport interface must expose a way to stop accepting new connections, as well as start accepting them again.
First discussed here.
The proposed API here is adding two methods .start() and .stop().
For backwards compatibility, by default, the transport starts in the "accepts connections" state.

I feel this belongs to libp2p-switch. A Transport will always yield the connection to the switch which then handles it to SECIO. Before that happens, the connection can be terminated.

@diasdavid yes, but wouldn't stopping a listener, (at least in some transports), also destroy all the underlying connections?

It doesn't destroy (or shouldn't) it actually waits for them to close and rejects new ones meanwhile.

Here, in the websocket-star protocol, it definitely closes the connection to the relay/discovery server: https://github.com/libp2p/js-libp2p-websocket-star/blob/master/src/listener.js#L77-L85

Which would, AFAIK, terminate all the underlying relayed connections..

@pgte that's a bug then :)

OK, filed issue here: https://github.com/libp2p/js-libp2p-websocket-star/issues/49

Also a bug in libp2p-tcp: https://github.com/libp2p/js-libp2p-tcp/issues/92

Tracking here: https://github.com/libp2p/js-libp2p-switch/issues/251


feat: exposing is_spam flag for the nft

Description
This PR exposes is_spam flag for the NFT in the transaction history.
The new flags object with the boolean flag is_spam is added to the nft_info object of the transaction history response.
The updated history example response is:
{
    "data": [
        {
            "id": "50f3e12794875b59884ddff06629077e",
            "metadata": {
                "operationType": "mint",
                "hash": "0x37f6313c5b0fe3efa4d61c340d6513c9884291a65e5b2f09ccc8e2769b1fe52e",
                "minedAt": "2023-09-19T23:50:30Z",
                "sentFrom": "0xa65f49f207fd07a5c3551d6eea8ab5c4ff977e1d",
                "sentTo": "0xa76ffb17004bab6af34cdf3e0dec6b6215614128",
                "status": "confirmed",
                "nonce": 20667
            },
            "transfers": [
                {
                    "fungible_info": null,
                    "nft_info": {
                        "name": "5 ETH Voucher by Base",
                        "content": {
                            "preview": {
                                "url": "https://lh3.googleusercontent.com/zrM3ThTK4fGQo4KjYSrCZtiGAExqEgFcYjBNIraHraz0bg8KRJk-Dht-RretwRLnhejbYRFxpNcw7ti_whrFr-YVsi_CYxDx3Pk=s250",
                                "content_type": null
                            },
                            "detail": {
                                "url": "https://lh3.googleusercontent.com/zrM3ThTK4fGQo4KjYSrCZtiGAExqEgFcYjBNIraHraz0bg8KRJk-Dht-RretwRLnhejbYRFxpNcw7ti_whrFr-YVsi_CYxDx3Pk=s250",
                                "content_type": null
                            }
                        },
                        "flags": {
                            "is_spam": true // New field
                        }
                    },
                    "direction": "in",
                    "quantity": {
                        "numeric": "1"
                    },
                    "value": null,
                    "price": null
                }
            ]
        }
    ]
}

Resolves #356
How Has This Been Tested?
Dogfooded with the Zerion API transactions result to the HistoryProvider -> ZerionProvider -> get_transactions.
The expected result is successful result deserialization and returns the new object in a blockchain-api response.
Due Diligence

[ ] Breaking change
[ ] Requires a documentation update
[ ] Requires a e2e/integration test update

fyi @enesozturk ;)


Keyboard return to alphabet each time

After clicking on number section on keyboard and then press a number (eg. 1,2,3,...) , it returns to alphabet part each time and it doesn't remain in "number section".

How to disable auto-completion fully to test it? I could find only getter method for it.
version 1.9.1
I think the problem probably occurs at textView shouldChangeTextInRange


Handling of zero magnitude cases in Cartesian3.normalize

Zero magnitude causes divide by zero which causes NaN in place of what should be a number. Those NaNs often times cause a crash further down the road and the crash report only traces about 8 functions back. Hunting down the cause can waste alot of time.
Perhaps have a quick zero magnitude check: if(cartesian.x==0 && cartesian.y==0 && cartesian.z==0)
Some possible actions if true:
console.log("warning: divide by zero");
Or set the result to all zeros instead of all NaNs.
Or set property .valid to false as you can't really turn a zero magnitude vector into unit length.
Perhaps normalize could be renamed to unitize to not be confused with the other definition of normal: a vector perpendicular to a plane.

I suggest putting a wrapper around normalize in your application as we are unlikely to add a check in the low-level code to keep it lean and performant for the common case.

I understand about keeping the functions very lean, particularly ones called quite often and shouldn't involve checks for rare cases. Thanks, I'll be sure to instead wrap the check into another function that in turn calls normalize if non-zero.


how to migrate all exsiting data to the new version?

Context, I'm updating to the latest version. It has a different name. I don't want to lose all my existing data and user accounts. In my honest opinion, there should be a way to auto migrate your data

When launching the new container just keep your previous docker volume name with -v ollama-webui:/app/backend/data in your docker run command.


Update Feedback Page

Helps towards fixing #488
Adds a title to the iframe for screen readers. Also changes the "Report a bug" h1 to an h2, with the same styling, since a page should only have one h1.
#hacktoberfest

Developer Certificate of Origin Version 1.1
Copyright (C) 2004, 2006 The Linux Foundation and its contributors. 1 Letterman Drive Suite D4700 San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or
(b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or
(c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it.
(d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.

Aa


currentdir() fails on long path

here is how it looks like when path length exceeds PATH_MAX - 1:
$ printf 'pwd length: %d\n' "${#PWD}" ; lua -e 'assert((require "lfs").currentdir())'
pwd length: 4100
lua: (command line):1: Numerical result out of range
stack traceback:
	[C]: in function 'assert'
	(command line):1: in main chunk
	[C]: in ?

expected behavior is to try getcwd() with larger buffer if ERANGE thrown

Closed thanks to @n1tehawk's #84 — thanks!


dns resolve error

I had problem today with resolving "ads.google.com".
I figured it out, that --env BLOCK_MALICIOUS=off  solve this.

It's because BLOCK_MALICIOUS also blocks ads related domain names. I might remove it from the ban list for the next update. Thanks for posting!

@ex0nuss sorry that took longer than expected.
I removed the tracking-nl.txt list from the malicious list of urls in https://github.com/qdm12/updated/commit/123a57ffd8a78c7ceebd72feda19ddd44fe9a799 that might solve it. You can try by restarting the container or wait for it to update itself. Thanks for your patience!


activitynet video

hi,
do you mind share video source  ?
Best,
wishes

you can download the org videos like https://github.com/activitynet/ActivityNet/tree/master/Crawler or https://github.com/activitynet/ActivityNet/tree/master/Crawler, it may help you.


Opening the same project twice actually opens the project again

Howdy folks 👋
First of all, thanks a million for the great addon. It has kept me productive for years at this stage 🎉
I noticed the issue I'm having after https://github.com/danielbrodin/atom-project-manager/issues/393 was fixed. Essentially when I run ⌘^p to open the project manager and open a project, then open the exact same project a second time it will open the project twice
I'm pretty sure the behaviour used to be that it would switch to the project that you're requesting to open if it was already open 🤔

This is happening for me as well. I end up with a lot of duplicated projects in different Atom instances. Could it be related to a recent OSX update?

This has always been handled by atom itself since there is no way to know which instances of atom is open. So my guess would be that there have been a change in atom. Maybe something to do with the change that required the #393 fix?
Would be happy to merge a PR if someone solves this.

I see. Too bad :/ Will try to investigate.

The same issue occurs for me with project-manager version 3.3.7 on Atom 1.36.1 on macOS 10.14.4.

I'm also affected (mac 10.14.4, atom 1.36.1).

also affected: linux, atom 1.38.2, project manager 3.3.7

project manager still at version 3.3.7 which has this issue

This bug seems to stem from when there was an Atom issue in 1.36.1 which was subsequently fixed in 1.37.0 see https://github.com/atom/atom/pull/19203.
The pull request #393 was added to fix what was then an atom issue. This doesn't appear to be the case anymore and reverting the change should fix this issue as within the atom package https://github.com/atom/atom/blob/119fead6925c0b6b30e9ce837c551bffaca42989/src/main-process/atom-application.js#L1325 passing the newWindow param will forcibly make atom open a new window, otherwise it will try to find an existing window matching the given paths.

@madrussa fix has been merged and published.


redirectURL is null for 301 redirects with relative Location header

Which version of PhantomJS are you using? Tip: run phantomjs --version.
2.1.1

What steps will reproduce the problem?

Visit a url which redirects, in which the url responding with 301 has a relative location header

The resource.redirectURL is set to null in the onResourceReceived callback handler

Which operating system are you using?
OSX El Capitan

Did you use binary PhantomJS or did you compile it from source?
Binary

Please provide any additional information below.

Phantom should resolve the relative URLs in the location header to absolute urls, and then set the redirectURL property of the resource.


'type()' function not returning anything

As reported on stack overflow...
I'm doing some investigation of my Jupyter Notebook code in VS Code using this snippet:
print('type')
print(type([]))
I get back:
type
Weirdly, if I introduce an error after that snippet I get back:
type
<class 'list'>
I've tested the snippet in another cell and it works fine so there's something odd in my code. Any ideas?
Edit: I've done a little more testing and it looks like the error has to be introduced straight after the snippet. If I allow the code to run all the way through and then introduce the error then I get the original issue.

@gottogethelp, Can you please provide a bit more detail as to exactly how your running this script? We have been unable to reproduce the problem in a VS Code notebook or using the interactive window.

cc/ @rchiodo @IanMatthewHuff @DonJayamanne @DavidKutu @joyceerhl

What info do you need? I've only just started with Python/VS Code in the last month or so. The Notebook itself has nothing 'funky' in it - one class, a few methods, some pandas, some numpy. I could send you the file?


Encoding longer sequences

Is there already any implemented function to encode longer sequences (such as sequencing reads) using their k-mers embeddings?

I have the same question, did you get any reply?

I have the same question, did you get any reply?

No, unfortunately...I guess you should sum the embeddings of all the k-mers of a read to get its final embedding

Did it work to sum the embeddings of different k-mers?

Did it work to sum the embeddings of different k-mers?

I'm sorry for the delay! I didn't try anymore, but I think it can make sense, it would be similar to work with word2vec.

I think need to do something like average pooling for a sequence kmer vecs, may be like this:
import numpy as np
from dna2vec.multi_k_model import MultiKModel

filepath = 'pretrained/dna2vec-20161219-0153-k3to8-100d-10c-29320Mbp-sliding-Xat.w2v'
mk_model = MultiKModel(filepath)

def get_kmer(seq, k):
    # extract kmer function...

seq = "AGCTACG......"

vecs = [ np.array(mk_model.vector(k)) for kmer in get_kmer(sequence, 3) ]
# get the 100-dim average vec for a different length sequence
vec_pool = np.mean(vecs, axis=0)


Operation 'get pacts green'

What does this pull request do?
major refactoring of the pact contract test code - see the commit messages for more details.
there is currently only one functional change in the application code, this was picked up by one of the failing tests, so that's good news.
before:

What is the intent behind these changes?
get the pact tests green

Pact is green - hooooooray

Let's make it required? 😁


Dependencies issue: npm audit found 15974 vulnerabilities

Description
npm audit found 15974 vulnerabilities
Steps to reproduce
npm audit
Screenshot
Critical | Arbitrary Code Execution
Package | eslint-utils
Dependency of | eslint [dev]
...
...
found 15974 vulnerabilities (10 moderate, 15960 high, 4 critical) in 873157 scanned packages
run npm audit fix to fix 15971 of them.
Solution
npm audit fix
npm install
npm test Pass
npm audit
found 2 moderate severity vulnerabilities in 886034 scanned packages
2 vulnerabilities require semver-major dependency updates.
Versions
React-Boilerplate: 4.0.0
node --version v11.5.0
npm --version 6.7.0

Development branch is dev. There is hardly any audit warnings there. Master is currently behind so it is normal to have warnings.
(PS: Also it is not that straight forward to update dependencies...)


adding state within passport authenticate causes a failure

Hey there, I have returned once more,
I am trying to pass additional parameters via my query to the callback function however when I try adding the state
const authenticator =  passport.authenticate('twitter', { scope: ['tweet.read', 'users.read', 'follows.read'], state});
authenticator(req, res, next);
the whole auth fails, any idea how I can resolve this?
Note: i know that twitter uses it's own state param, is there way i can add mine on top of it or add a different one

Hi @VinHylme, I will check this on Monday.

@jnv no probs, thanks :)

@VinHylme Thanks for your patience. So passing the state like this should absolutely work, it's actually implemented by the backing passport-oauth2 library and here's a blog post explaining how to use it: https://medium.com/passportjs/application-state-in-oauth-2-0-1d94379164e
The gist is:

you need to set store: true in the strategy constructor (ie. new TwitterStrategy)
pass the state option into passport.authenticate
the state option should be an object

With these options checked, you can then get back your state from req.authInfo.state after you authenticate the callback request, e.g.:
app.get(
  '/auth/twitter/callback',
  passport.authenticate('twitter', {
    failureRedirect: '/error?login'
  }),
  (req, res, next) => {
    console.log('The state is:', req.authInfo.state);
    next();
  }
);

Mind that this isn't a feature of this library, all of this is handled just by the passport-oauth2.
I am removing the bug label but keep this issue open to add this info into our README.
Let me know if it works for you!

Hey I am having same issue, it is not working when I am adding custom state like this
passport.authenticate('twitter', { scope: ['tweet.read', 'users.read', 'offline.access'],  state: 'twitter' })
I am getting Forbidden error but it is working fine when I am removing state like this
passport.authenticate('twitter', { scope: ['tweet.read', 'users.read', 'offline.access'] })
Twitter OAuth2 strategy within Passport
passport.use(
    new TwitterStrategy(
        {
            clientID: process.env.TWITTER_CLIENT_KEY ?? "",
            clientSecret: process.env.TWITTER_CLIENT_SECRET ?? "",
            clientType: 'confidential',
            callbackURL: 'TWITTER_CALLBACK_URL'
        },
        (accessToken, refreshToken, profile, done) => {
            console.log('Success!', { accessToken, refreshToken, profile });
            const socialMediaPlatform = 'twitter';
            // The user profile obtained from Instagram will be available here
            const user = {
                socialMediaPlatform,
                profile,
            };
            return done(null, user);
        }
    )
);

Hi, thanks for your comment. Just a quick guess. Have you tried passing state as an object?

You might also find this blog post by Jared helpful.

Hi, thanks for your comment. Just a quick guess. Have you tried passing state as an object?
Hey, this is working but I can access the state object on my backend when I am making call to the callback URL, then I can send it on client side, but is it possible to access the state object in the call back URL as a params?

I'm facing this, any fixes?


Only delete the socket file for NioServerDomainSocketChannel

Motivation:
We need to ensure we only delete teh sock file when using NioServerDomainSocketChannel as otherwise it is not possible to connect with multiple clients
Modifications:
Only delete file in NioServerDomainSocketChannel
Result:
Fixes https://github.com/netty/netty/issues/14114

/cc @honglh @yawkat

👍


model: Fix spelling mistake

Signed-off-by: Jean-Christophe Beaupré<EMAIL_ADDRESS>

+1


Implement auto-update software for Agora

As a node operator, I need a software that keeps an automatically updated agora node on my server in order to have seamless integration with the chain.
Definition of Done:

[ ] The auto-updater is separate from Agora
[ ] The auto-updater is the default package pointed to on the website
[ ] The auto-updater can never be updated
[ ] The auto-updater never fails
[ ] The auto-updater can rollback a failing upgrade
[ ] The auto-updater uses multiple mirrors
[ ] The auto-updater supports server-side downtime

We might be able to leverage https://github.com/containrrr/watchtower with a special boot sequence (migration scripts).


Added Collatz Conjecture Algorithm

Hey, i added the Collatz Conjecture Algorithm to algorithms.js and fixed some lint erros;

:+1:
Thanks for contributing!


RSC Example doesn't work

Bug report
The RSC example from the examples folder doesn't work.
Description / Observed Behavior
The RSC example as is, throws an error when running it.
When I await the request (const serverData = fetcher('http://localhost:3000/api/data') in page.jsx it does works.
Expected Behavior
I expect being able run the example without having to await the promise in the page component.
Repro Steps / Code Example
I cloned the RSC example
run yarn install
run yarn run dev
navigate to localhost:3000/rsc
I see TypeError: data.map is not a function.
Additional Context
SWR version: 2.2.5

You can run this example with pnpm dev.

I'm probably doing something run, but when I run it with pnpm dev I still get the same issue.

You might still use the latest version of SWR; the example depends on the feature that is contained in the beta versions.
Did you clean your deps (node_modules/) and run pnpm install and pnpm build on the root before running the app?

weird, I've also tried to "One-Click Deploy" this suspense example, and get the same error during build:

Yeah, this example points to the latest version of SWR, so this won't work.
https://github.com/vercel/swr/blob/main/examples/suspense/package.json#L10
I expect this will work after you update the version to "beta."

Thanks for explaining! I've updated the package.json to use "beta" but that causes this error:
Packages: +24
++++++++++++++++++++++++
Progress: resolved 32, reused 19, downloaded 5, added 24, done

dependencies:
+ next 14.2.3
+ react 18.3.1
+ react-dom 18.3.1
+ swr 2.2.6-beta.4

Done in 3.1s
➜  suspense pnpm build

><EMAIL_ADDRESS>build /Users/k101476/Dev/suspense
> next build

  ▲ Next.js 14.2.3

   Creating an optimized production build ...
 ✓ Compiled successfully
 ✓ Linting and checking validity of types
   Collecting page data  ..file:///Users/k101476/Dev/suspense/node_modules/.pnpm/swr@2.2.6-beta.4_react@18.3.1/node_modules/swr/dist/_internal/index.mjs:1
import { i as isWindowDefined, a as isFunction, m as mergeObjects, S as SWRConfigContext, d as defaultConfig, s as serialize, b as SWRGlobalState, c as cache, e as isUndefined, f as mergeConfigs } from './config-context-client-Ds68J9M9.js';
                                                                   ^
SyntaxError: Named export 'S' not found. The requested module './config-context-client-Ds68J9M9.js' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from './config-context-client-Ds68J9M9.js';
const { i: isWindowDefined, a: isFunction, m: mergeObjects, S: SWRConfigContext, d: defaultConfig, s: serialize, b: SWRGlobalState, c: cache, e: isUndefined, f: mergeConfigs } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:132:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:214:5)
    at async ModuleLoader.import (node:internal/modules/esm/loader:329:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:431:15)

> Build error occurred
Error: Failed to collect page data for /
    at /Users/k101476/Dev/suspense/node_modules/.pnpm/next@14.2.3_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/next/dist/build/utils.js:1268:15 {
  type: 'Error'
}
   Collecting page data  . ELIFECYCLE  Command failed with exit code 1.


HTML

My .vimrc:
" Set compatibility to Vim only.
set nocompatible

" Color terminal
set t_co=256

" Helps force plug-ins to load correctly when it is turned back on below.
filetype on

" Turn on syntax highlighting.
syntax enable
" Turn off modelines
set modelines=0

" Automatically wrap text that extends beyond the screen length.
set wrap
" Vim's auto indentation feature does not work properly with text copied from outisde of Vim. Press the <F2> key to toggle paste mode on/off.
nnoremap <F2> :set invpaste paste?<CR>
imap <F2> <C-O>:set invpaste paste?<CR>
set pastetoggle=<F2>

" Uncomment below to set the max textwidth. Use a value corresponding to the width of your screen.
" set textwidth=79
set formatoptions=tcqrn1
set tabstop=2
set shiftwidth=2
set softtabstop=2
set expandtab
set noshiftround

" Display 5 lines above/below the cursor when scrolling with a mouse.
set scrolloff=5
" Fixes common backspace problems
set backspace=indent,eol,start

" Speed up scrolling in Vim
set ttyfast

" Status bar
set laststatus=2

" Display options
set showmode
set showcmd

" Highlight matching pairs of brackets. Use the '%' character to jump between them.
set matchpairs+=<:>

" Display different types of white spaces.
set list
set listchars=tab:›\ ,trail:•,extends:#,nbsp:.

" Show line numbers
set number

" Encoding
set encoding=utf-8

" Highlight matching search patterns
set hlsearch
" Enable incremental search
set incsearch
" Include matching uppercase words with lowercase search term
set ignorecase
" Include only uppercase words with uppercase search term
set smartcase

" Store info from no more than 100 files at a time, 9999 lines of text, 100kb of data. Useful for copying large amounts of data between files.
set viminfo='100,<9999,s100

" Map the <Space> key to toggle a selected fold opened/closed.
nnoremap <silent> <Space> @=(foldlevel('.')?'za':"\<Space>")<CR>
vnoremap <Space> zf

" Automatically save and load folds
autocmd BufWinLeave *.* mkview
autocmd BufWinEnter *.* silent loadview"

colorscheme dracula

call plug#begin('~/.vim/plugged')
Plug 'vim-airline/vim-airline'
Plug 'vim-airline/vim-airline-themes'
Plug 'tpope/vim-fugitive'
Plug 'tpope/vim-git'
Plug 'moll/vim-node'
Plug 'scrooloose/nerdtree'
Plug 'scrooloose/nerdtree-git-plugin'
Plug 'Valloric/YouCompleteMe', { 'do': './install.py' }
Plug 'pangloss/vim-javascript'
Plug 'jiangmiao/auto-pairs'
Plug 'kien/ctrlp.vim'
Plug 'posva/vim-vue'
" Plug 'leafOfTree/vim-vue-plugin'
Plug 'vim-syntastic/syntastic'
Plug 'othree/html5.vim'
Plug 'sekel/vim-vue-syntastic'
Plug 'alvan/vim-closetag'
call plug#end()

" --- NERDtree ---
map <C-n> :NERDTreeToggle<CR>
" ---

" --- CtrlP ---
let g:ctrlp_custom_ignore = 'node_modules\'
" ---

" --- vim-vue ---
autocmd BufRead,BufNewFile *.vue setlocal filetype=vue.html.javascript.css
" ---

" --- syntastic ---
set statusline+=%#warningmsg#
set statusline+=%{SyntasticStatuslineFlag()}
set statusline+=%*

let g:syntastic_allways_populate_loc_list = 1
let g:syntastic_auto_loc_list = 1
let g:syntastic_check_on_open = 1
let g:syntastic_check_on_wq = 0

let g:syntastic_javascrip_checkers = ['eslint']
let g:syntastic_vue_checkers = ['eslint']
let local_eslint = finddir('node_modules', '.;') . '.bin/eslint'
if matchstr(local_eslint, "^\/\\w") == ''
  let local_eslint = getcwd() . "/" . local_eslint
endif
if executable(local_eslint)
  let g:syntastic_javascript_eslint_exec = local_eslint
  let g:syntastic_vue_eslint_exec = local_eslint
endif
" ---

All HTML on

@sysRQrol Were you able to resolve this? I have a similar issue.

I had a similar problem, try appending this in your .vimrc
autocmd BufRead,BufNewFile *.vue setlocal filetype=vue.html.javascript.css
let g:vim_vue_plugin_load_full_syntax = 1
let g:vim_vue_plugin_debug = 1

This solved my problem, please comment if this work for you.

Can you run the following command with the cursor on the area that isn't highlighted, and paste the result here?
:echo reverse(map(synstack(line('.'), col('.')), 'synIDattr(v:val,"name")'))

The result of the command is:
['htmlTagN', 'htmlTag']

I had a similar problem, try appending this in your .vimrc
autocmd BufRead,BufNewFile *.vue setlocal filetype=vue.html.javascript.css.pug

thanks this solved my problem with pug language,
i am using spacevim with nvim.


Work with a subset of the raster given some extent definition

Now supports nodata values, which is one way to work with an interesting subset of a raster. (But admittedly not the same thing as a coordinate-defined extent window.)

Followup: extent support drafted in d34c04b8db3c06465b50440bf257afd516b59771 with --window (geographic coordinates) and --pixels (raster coordinates). Geographic window option needs more testing as I recall it failed to work with expected good region coordinates in one recent case.

Might have mixed up my easting and northing in that failed case. Coord order is not documented and I often get x/y-lon/lat mixed up if I'm not careful. Documentation should clarify.


TEST: add unit test for docstrings

This merges into 1.1.x – can we change to something like dev/1.1.0rc0 to adopt new conventions? We can create dev/1.1.0rc0 as a direct branch from 1.1.x

This merges into 1.1.x – can we change to something like dev/1.1.0rc0 to adopt new conventions? We can create dev/1.1.0rc0 as a direct branch from 1.1.x

Done!

This merges into 1.1.x – can we change to something like dev/1.1.0rc0 to adopt new conventions? We can create dev/1.1.0rc0 as a direct branch from 1.1.x

Done!

Changed it back as just discussed, could you please review/approve? Thanks!


When do you plan to release SDL Trados Studio plugin?

Hi, Innovators,
I am so excited that you relased 2.0! Will test it shortly.
Matecat folks told that you are redoing API for Matecat and for MMT as they are not stable:
https://github.com/matecat/MateCat/issues/992
My humble questions that are important for many followers:

when are you planning to release SDL plugin?
is MMT API stable?
If SDL plugin is not coming shortly, what is the application for batch  translating XLIFF files ( not using MMT API for every sentence, but translating the whole file). Shall we use Okapi MMT API integration? Is it stable for production use?

Thanks,
Mindaugas

Hi @mindaugasDEV
we are still in the process of releasing the new 2.0 version of MMT (it's not currently available in the releases), but we should complete the process by today.

when are you planning to release SDL plugin?

At the moment there is no deadline for the release of the Trados plugin, but it is definitely something we will work on in the next months.

is MMT API stable?

MMT v2 API should be considered stable, but we haven't published them yet, we are updating the documentation before the official release.

If SDL plugin is not coming shortly, what is the application for batch translating XLIFF files ( not using MMT API for every sentence, but translating the whole file). Shall we use Okapi MMT API integration? Is it stable for production use?

I am currently not aware of the Okapi MMT API so I can't say if they are reliable or not. Regarding XLIFF bulk translation, it really depends if you want to use hosted MMT or the MMT public service (available by end of October). If you have your own installation you should be able to use the translate command like this:
./mmt translate --context <your context> --batch --xliff < myfile.xliff

If you want to use the online service I suggest to use MateCat for translating your XLIFF.

Thank you!!!

Sorry to jump in. @mindaugasDEV I just want to say that I used OKAPI MMT and it works just fine, but it's a bit slow. I tested it with the SMT version of MMT though. You can use it to translate SDL Trados Studio SDLXLIFF files. I would test Davide's method first and see. I believe it would be faster.

Hi, @mzeidhassan
Thanks for jumping into discussion. What tool from Okapi did you use? Tikal command line or Rainbow?
Can you share the commands you use or screenshot?
I could not understand how to use them :(.
Best,
Mindaugas

@mindaugasDEV  you can find details here
http://okapiframework.org/wiki/index.php?title=ModernMT_API_Connector


Draft: proof-of-concept for streamline to precomputed annotation, include CORS server for rendering in local neuroglancer

@balbasty @ayendiki -- starting a branch here for work on trk to precomputed annotation conversion -- Note: I am testing this against google's current neuroglancer, not our fork
To replicate locally/work with if you'd like:
python -m venv venv
source venv/bin/activate
pip install -r requirements.txt
python trk_to_annotation.py

This will produce a few thousand small (1KB-ish) coordinates with the JSON properties necesssary for rendering in neuroglancer -- my current obstacles is getting the precomputed annotation layer, and the nifti layer (sub-I58_sample-hemi_desc-preproc_dwi_FA.nii.gz) in this case to be in the same voxel space
Will keep you posted as things progress -- wanted to get code on GitHub to start the conversation

@balbasty -- a couple mini updates here with the script (things I am trying to tackle)
• It seems that the NIFTI and the precomputed annotations are still offset in neuroglancer -- e.g. they dont overlap when added as layers --
backend.ts:456 Error retrieving chunk [object Object]:3,3,3: RangeError: Offset is outside the bounds of the DataView

• It seems that the empty bytes are failing to be recognized by the precomputed backend for neuroglancer (whether or not we should have "empty" bytes -- I'm not sure?)
backend.ts:891 Uint8Array(8) [0, 0, 0, 0, 0, 0, 0, 0, buffer: ArrayBuffer(8), byteLength: 8, byteOffset: 0, length: 8, Symbol(Symbol.toStringTag): 'Uint8Array']
backend.ts:456 Error retrieving chunk [object Object]:5,5,2: Error: Expected at least 8 bytes

I think you want the tracts coordinate to be in mm RAS, not in voxels. So don't apply the inverse affine, set the axes as 1 mm (not voxel_size) and compute the upper and lower bounds in RAS mm space. There's an example in the Google doc.

I think you want the tracts coordinate to be in mm RAS, not in voxels. So don't apply the inverse affine, set the axes as 1 mm (not voxel_size) and compute the upper and lower bounds in RAS mm space. There's an example in the Google doc.

@balbasty thanks for the guidance -- just so I fully understand, you mean this Google doc? https://docs.google.com/document/d/1TnDNvDigOxpNSuZBjHkve_epp7tbpswcxUUcBOaRV94/edit?tab=t.0
If so, do you mean:
 "dimensions": {"x": [1, "mm"], "y": [1, "mm"], "z": [1, "mm"]},
  "lower_bound": vertices.min(axis=0).tolist(),
  "upper_bound": vertices.max(axis=0).tolist(),

in your info JSON example?

Yes! Although it should really be a "double loop" minimum/maximum (over tracts and edge per tract)

Yes! Although it should really be a "double loop" minimum/maximum (over tracts and edge per tract)

What exactly is accomplished in the "double loop" that wouldn't be accomplished via a "single loop"? I tried to search online to see why, but couldn't really make 100% sense.


list manipulations are not reflected

I gave this package a chance but when I, for example, delete items from the list, the GridViewBuilder is not reflecting those changes.

you can update your list from the onDragAccept callback, let me know if you have any trouble doing this. I will reopen this issue


JS Icon is the React Icon?

[x] I'm sure this issue is not a duplicate

JS Icon is the React Icon?
Firstly, :heart: the extension! Really like your work! :smile:
Context
I got a prompt today that VSCode icons had updated, so I reloaded VSCode. However, it looks like the icon for .js files is now the React icon by default? (See screenshot below)

These files are pure JS files.
Wondering if this is expected behaviour or if the icons are wonky for .js files? 🤔
FYI - I've also tried the Toggle official JS Preset command.
🙂

@abinavseelan Check your settings (Workspace) there should be an entry there that associates the js extension to reactjs. Probably, this is done by another extension you have installed.

Ah ha! That was it! 🎉 😄
Thank you! Closing the issue. 🙇

@abinavseelan, just out of curiosity. Which black theme are you using?

@robertohuertasm
Ayu Dark 😄

@abinavseelan where was that entry? I cant find it on my VSCode settings. Had the same issue after i used create-react-app for the first time

@abinavseelan could you please tell me how did you solve it? I've just opened VS Code and every .js file is associated to react 😄

Maybe you have changed this in your default settings?
it should read
settings.json
{
  "files.associations": {
    "*.js": "javascript",
    "*.jsx": "javascriptreact",
    "*.ts": "typescript",
    "*.tsx": "typescriptreact"
  }
}


Submit project configuration as YAML

Add a file control to the project info page, which allows uploading the entire configuration of plugins as a YAML document.

Posted (with JSON instead of YAML) at<EMAIL_ADDRESS>

Merged.


User wants to have array literal [] to construct arrays

What?
User wants to have array literal [] to construct arrays
How?
let triangles = [
    new Triangle(),
    new Triangle(),
    new Triangle()
];

Nice to have but v2 as you can always do it outside of initialization.
let trinagles = [3];
triangles[0] = new Triangle();
triangles[1] = new Triangle();
triangles[2] = new Triangle();

Moved to v2.0.0
Plan is to reduce calls and do a bulk allocation for array literals.


Fix: System Footer Visibility

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/41184
$ https://github.com/Expensify/App/issues/41135
PROPOSAL: https://github.com/Expensify/App/issues/41184#issuecomment-2081895295
Tests

Log out of Expensify
Navigate to a public room link (https://staging.new.expensify.com/r/2376199970894587)
Verify Message Concierge Footer is not visible

[x] Verify that no errors appear in the JS console

Offline tests
Same as Tests step
QA Steps
Same as Tests step
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[ ] I included screenshots or videos for tests on all platforms
[ ] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

I've asked for a volunteer here.

Waiting for an update here on if we should add a check for isConciergeChat as @mountiny suggested to show it only for system chats (Concierge?)
I'm wondering why would we need to show Message Concierge... on Concierge chat.
I can see The Onboarding Flow is changing here https://github.com/Expensify/App/pull/40678 and for now we don't have read only chats
Since the new flow isn't ready yet, maybe it is cleaner to hide the footer for now?
Waiting for updates before proceeding
cc: @shubham1206agra @mountiny

@neonbhai that is not needed anymore, before we added the onboarding tasks to the concierge chat but now they go to specific system chat.
#Every use will have one and you can identify it by chatType property set to system

You could basically update the code so the canWriteInReport is true unless its chat report with type system

@neobhai @shubham1206agra it would be added here https://github.com/Expensify/App/pull/40678/files
Feel free to add it now

@neonbhai check out the pr j have linked and add the system type. You can also see that OpenApp call returns one chat with this chattype

Finalizing in 30 minutes!

Is this ready for re-review?

It is!
@shubham1206agra please take a look!

It looked like canWriteInReport meant to check REPORT.PERMISSIONS.WRITE of the report:
https://github.com/Expensify/App/blob/c7b6e355009af1d36f70d8b0e21f135f2858d32d/src/libs/ReportUtils.ts#L1201
So instead of mixing with this var:
https://github.com/Expensify/App/blob/c7b6e355009af1d36f70d8b0e21f135f2858d32d/src/pages/home/report/ReportFooter.tsx#L85
I separated it here
This felt cleaner. We could change if you recommend?
@shubham1206agra

Thanks for handling

🚀 Deployed to production by https://github.com/francoisl in version: 1.4.67-7 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.4.67-7 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Post on how we use Tettra

IMPORTANT: blog posts must be approved by two reviewers before they can be published, one of which must be a native English speaker.
[Add any information about this post here. Leave the note above when opening the Pull Request.]
This post links to the one about knowledge bases.

@ABizzinotto Great article!
I think it would be nice to also put some screenshots in the article. What do you think?

@lubc I'm all for screenshots hehe but not sure where I'd add them though. Thoughts?

@lubc I'm all for screenshots hehe but not sure where I'd add them though. Thoughts?

@ABizzinotto maybe you can put one screenshot in some of the sections that you cover, like categories or slack integration.

@lubc done :)


refactor vfs: cleanup & simplify code, better tests, remove old stuff

refactor org.reflections.vfs package

cleanup & simplify code
better tests
remove old stuff

measure: reduce total lines of code, number of classes, code duplications
rebased with 0.10

I can work on this please assign me this issue.

thanks @sgaurav37533 bhai 🙏🏽 I'd rather not assign so that others can work on it if want to
but I can suggest one/some of the following: we can discuss it on mail/zoom for better ctx; you can sketch something and raise pr; or send it over branch/patch for review/discussion; or any other option you wish. feel free to reach out ronmamo at gmail


fixed doc for  resource

Overview
Fixed doc for  resource duplocloud_gcp_scheduler_job resource
This PR does the following:

Fixed example of duplocloud_gcp_scheduler_job resource
...

Testing performed

[ ] Using unit tests
[ ] Manually, on my local system
[ ✓] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-17143 TF:Doc: Fix resource type in example of resource “duplocloud_gcp_scheduler_job”. Change type “duploscheduler_gcp_scheduler_job” to “duplocloud_gcp_scheduler_job”


Added support for refund transactions via Authorize.net CIM and for live validation on customer profile creation

Three small patches for the Authorize.net CIM gateway that I needed:

support for refund transactions

support for passing validationMode in createCustomerProfile action
(useful for triggering errors when creating customerProfile and customerPaymentProfile in one step)

support for validationMode on updateCustomerPaymentProfile action

If there's anything I can do to facilitate these getting merged upstream, please let me know.

If you split these up into separate commits (and avoid including merge commits) I'd be happy to work to get them merged. Thanks!

@ntalbott — thanks for reviewing this pull request. I am no longer using ActiveMerchant's Authorize.net support, so I'm not in a position to update these patches. Hopefully their existence here in their current state will yet prove helpful to someone down the line.
Also, thanks for the feedback... but it left me a little confused. The three bullet points correspond 1:1 with each of the first three commits; would you want these commits split differently? would you want the commits split into 3 separate pull requests? Or is the blocking issue really the merge commit?

I'd definitely need the merge commit cleaned out.
As for splitting them up into separate PR's, that's more debatable. In general I'd personally submit these as at least two separate PR's (refund, and adding validationMode), but honestly I'd probably take the whole lot if the merge was gone and the whole PR applied cleanly.


[Feat] Create a generic design for Stacked bar chart infographic

Hello again!
Here's our take on the generic stacked bar chart slide.

@Mr-DannyDin as in other cases it would be great to have the place holder for the number to include at the top of the bar.


Client side fetch

parameterize es search fetch

@agiannini check this out, if you're ok with these changes let's merge it to main


Phone number has to be msisdn number

In the attributes, the phone number is taken from the account information, but it is not the phone number which is linked to the SIM.
Would be nice to see the msisdn available in the attributes, as this is the number we need.

Thanks for the feedback Geert! It has been updated since release 1.1.2.

Tested it and it is perfect for the call/sms sensor. You need also to change it for the internet sensor (line 263)
Thanks for changing it this swift 👍

Thx, did it too quick ;)


proposal: unicode/utf16: encoding without allocations

Background
utf16.Encode always allocates a []uint16 large enough to fit the UTF-16 encoded sequence, which is really ergonomic but forces one allocation.
Proposal
For those cases that the extra allocation matters, unicode/utf16 could provide an additional encoding function which accepts a pre-allocated (and large enough) backing slice.
The signature would look like this:
// EncodeInto writes into p (which must be large enough) the UTF-16 encoding
// of the Unicode code point sequence s.
func EncodeInto(a []uint16, s []rune) []uint16

Optionally, in order to know the minimum size of the backing array,  unicode/utf16 could provide an additional function which counts the number of code units in a code point sequence.
It would look something like this:
// Countreturns the number of code units in p.
// Invalid encodings are treated as single runes of width 1 byte.
func Count(s []rune) int {
	n := len(s)
	for _, v := range s {
		if v >= surrSelf {
			n++
		}
	}
	return n
}

It worth northing that utf16.Encode could then be implemented using utf16.Count and utf16.EncodeInto.
Examples
My specific use case is to allow x/sys/windows/mkwinsyscall generate syscall wrappers which accept string arguments without allocating, at least for short strings. Check this comment for more context.
If I had utf16.EncodeInto I could implement a non-allocating wrapper as follows:
func Foo(s string) {
	p := []rune(s + "\x00")
	l := utf16.Count(p)
	var a []uint16
	if l < 32 {
		a = make([]uint16, 32)
	} else {
		a = make([]uint16, l)
	}
	a = utf16.EncodeInto(a, p)
	syscall.Syscall6(fnAddr(), 6, 0, uintptr(unsafe.Pointer(&a[0])), 0, 0, 0, 0)
	return
}

Maybe instead we should have Append versions, like #50601 and #51644 ?
If that were the case maybe Count wouldn't be necessary? You could rely on appends growth to size the buffer correctly (across multiple calls to Encode).

Maybe instead we should have Append versions, like #50601 and #51644 ? If that were the case maybe Count wouldn't be necessary? You could rely on appends growth to size the buffer correctly (across multiple calls to Encode).

I like your suggestion a lot. It would match nicely utf8.AppendRune and there is no need for Count, just setting an initial buffer capacity is enough to avoid some allocations.
My syscall example would look like this:
func Foo(s string) {
	p := []rune(s + "\x00")
	a := make([]uint16, 0, 32)
	for _, r := range p {
		a = utf16.AppendRune(a, r)
	}
	syscall.Syscall6(procBCryptGetProperty.Addr(), 6, 0, uintptr(unsafe.Pointer(&a[0])), 0, 0, 0, 0)
	return
}

Prototyped the proposal in CL 409054.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group


Add desc for orders element clarify what fhir resources

Fixes: https://jira.hl7.org/browse/FHIR-42365
and https://jira.hl7.org/browse/FHIR-42678
I considered adding the same blurb to order-dispatch, but I don't think it really needs it.

Also adding a railroad diagram --
Diagram(
  NonTerminal("Open CPOE"),
  Group(Choice(0,NonTerminal("Select product"),"order-select"),"order-select fires at the beginning"),
  Choice(1,
    NonTerminal("Select route"),
    NonTerminal("Select frequency"),
    NonTerminal("Select duration"),
    NonTerminal("Link Order & misc"),
  ),
  Group(Choice(0,NonTerminal("Sign order"),"order-sign"),"order-sign at the end")
  )

here: https://tabatkins.github.io/railroad-diagrams/generator.html#Diagram(
  NonTerminal("Open CPOE")%2C
  Group(Choice(0%2CNonTerminal("Select product")%2C"order-select")%2C%22order-select%20fires%20at%20the%20beginning%22)%2C%0A%20%20Choice(1%2C%20%0A%20%20%20%20NonTerminal(%22Select%20route%22)%2C%0A%20%20%20%20NonTerminal(%22Select%20frequency%22)%2C%0A%20%20%20%20NonTerminal(%22Select%20duration%22)%2C%0A%20%20%20%20NonTerminal(%22Link%20Order%20%26%20misc%22)%2C%0A%20%20)%2C%0A%20%20Group(Choice(0%2CNonTerminal(%22Sign%20order%22)%2C%22order-sign%22)%2C%22order-sign%20at%20the%20end%22)%0A%20%20)


Add the ability to URL encode array of maps

For example the following dict
terms (
    {
    lhs = "blood_glucose_measurement";
    op = gt;
    rhs = 10;
	}
)

should generate the following URL encoded string
type=or&terms[][lhs]=blood_glucose_measurement&terms[][op]=gt&terms[][rhs]=10

I'm working with iOS 5.0 so I couldn't get the tests running or working so I didn't update the test cases to reflect the new functionality. Sorry about that.

Still no joy
➜  RestKit git:(master) ✗ cat uispec.opts
--workspace RestKit.xcodeproj/project.xcworkspace --scheme UISpec --sdk 5.0
➜  RestKit git:(master) ✗ rake uispec:all
(in /Users/bradphelan/workspace/RestKit)
Building project...
xcodebuild: error: The workspace 'RestKit' does not contain a scheme named 'UISpec'.
Exited with status: 65
Failed to build

My XCode Version info is
Developer Information:

  Version:	4.2 (4C177)
  Location:	/Developer
  Applications:
  Xcode:	4.2 (828)
  Instruments:	4.2 (4227)
  Dashcode:	3.0.2 (336)
  SDKs:
  Mac OS X:
  10.6:	(10K549)
  iPhone OS:
  4.2:	(8C134)
  5.0:	(9A5313e)
  iPhone Simulator:
  3.2:	(7W367a)
  4.0:	(8A400)
  4.1:	(8B117)
  4.2:	(8C134)
  5.0:	(9A5313e)

Managed to run the specs from XCode with iOS5. Not sure what is up with the rake tasks? As you see I have now added a spec for the added feature. Hope this aids the pull request.
Regards Brad

Yep, I forgot to mention that. Once you  do
rake 'uispec:server'
Then running the specs from the Xcode build target is the easiest way to go. You should see zero failed tests in your console log at the end of the run
Sent from my iPhone.
On Oct 11, 2011, at 3:39 AM, Brad<EMAIL_ADDRESS>wrote:

Managed to run the specs from XCode with iOS5. Not sure what is up with the rake tasks? As you see I have now added a spec for the added feature. Hope this aids the pull request.
Regards Brad
--
Reply to this email directly or view it on GitHub:
https://github.com/RestKit/RestKit/pull/391#issuecomment-2358988

Looks good. Merged to master. Thanks @bradphelan


Support glob patterns instead of simple fnmatch patterns for add

Today I, for the first time, really tried to use the new (not yet shipped) fnpattern parameter and was pretty disappointed about its real-world utility. After trying different things I settled on the attached code for what I believe to be a much more useful interface:
The pattern parameters now expects a glob instead of a mere fnmatch pattern and can to things such as **/meta/*/zone/**/*.jpg. Also it is possible to pass more than one matching pattern by passing a list of strings instead of a single string parameter. Finally, because it happened to naturally emerge from the implementation, one can also pass fully fledged regular expressions (or anything else implementing .match) along with the glob patterns in that list simply by pre-compiling them.

Oh and I got rid of the top-level fake_dir and tweaked the one test that used it to use test/functional/fake_dir instead.

I apparently missed the notification email for this. I'm so sorry that it took so long for me to notice. Next time this happens, just mention me in a comment and I'll get right on it. I'm looking at the changes today.

I just wanted to start complaining about nobody looking at this, so it's pretty much perfect timing I'd say. :grin:
Stuff like this happens way too often to me as well. Next time I'll try to be more noticeable.

I think this would pass Travis is you rebased on top of master.


[Question] Accepting PRs for injectors for other browser automation solutions?

I was going to open a PR to add an injector for mokr (which I am the author of), but figured I'd check to see if you were open to additional injectors first.

Sure! I'd be happy for support to be added for mokr, it's a very neat package!
However, changes to the injector submodule might be moved to a separate repo soon (#7).


Variable arg shortcuts?

I am looking to add a global argument to my tool to control logging verbosity. I would like to use mytool.exe -v or mytool.exe -vvv for more verbose logging.
I have not figured out how with PowerArgs this can be achieved. Do you have any thoughts or ideas to accomplish it with the current implementation?

Try this:
    [ArgShortcut("vvv")]
    [ArgShortcut("v")]
    [ArgShortcut(ArgShortcutPolicy.ShortcutsOnly)]
    public bool Verbose { get; set; }

Hmmm, that's a good one. You can do something like this.
It's a bit primitive, but hopefully you get the point.


just a minor typo in the readme

i love how simple zx is - this is going to shape some amazing things in an Alpine container.

This is incredibly overdue! I apologize and thank you for the change!


Animated Cards of some special moments of FIFA worldcup 2022 final

Title and Issue number

Title : Messi animated card #77
Animated cutout of Messi when hover will lift worldcup trophy
Issue No. : #77
Close #77

Screnshots

Additional context(if any)
Are you contributing under any Open-source programme?

Apertre

resolve the merge conflict @abhraneel2004


error when using own dataset

Hi, I tried to use the own dataset following the template setting instruction, but it failed when running run.sh like this :
root@mycomputer :~/parawave/egs/cvc/voc1$ sh ./run.sh
./run.sh: 34: utils/parse_options.sh: Syntax error: Bad for loop variable
May I ask is there something I did wrong in the setting? thanks.

What is your environment (OS etc.)?
The script is not made by me (one of the kaldi utils) and I've never met this error.

sorry for my mistake, I run it in bash instead of sh and it works now thanks.


Packages are not specific for import in Node.js

Is it just me or the major 3 classes cannot be imported in node.js i.e THREEx.ArToolkitSource, THREEx.ArToolkitContext, and THREEx.ArMarkerControls. @jeromeetienne it looks like the package was just published to npm without any actual exports of required modules. Is this something you're aware of? Is it open to contributions?

Made a node compatible version here https://www.npmjs.com/package/node-ar.js . @jeromeetienne I'd really love it to be one same thing as the ar.js package so I'll send in a PR and we can discuss how I can collaborate better if there's an issue with how I've structured the code

+1. I would also like to use ar.js in Node.

I have never used A-Frame and I can't say for sure but I can look into that and get it to work for A-Frame

Thanks for replying!
I actually ended up just putting the scripts in the head tag of my site, instead of using node.  That way I was able to use AR.js and AFrame together and not worry about including the modules with node for now.


PR: Move debug buttons to Debugger plugin, depends on #18852

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

Hello @impact27! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/utils/icon_manager.py:

Line 144:80: E501 line too long (98 > 79 characters)
Line 195:80: E501 line too long (85 > 79 characters)


Create OS image for easy test deployment machines

Create an image that we can deploy easily and fast on several machines for testing purposes.
It must be easy and fast to setup everything ("out of the box"):

JRE
maybe some tool / script that allows checking whether specified directories have the same content on all machines?

If native deployment is not possible / feasible, an alternative may be a VirtualBox or VMware container (however, need to configure proper network configuration such that everything works as expected with bootstrapping etc.)

not relevant anymore


http.request api should be improved

Sinks (as well as filters) require response metadata to properly process data. There's a significant difference between receiving a 200 or 404 status code, or whether the Content-Type is application/json, text/html, or text/event-stream.
Without this metadata, I’m forced to rely on unreliable heuristics in the sink/filter code, which is far from ideal.
It's unfortunate because the response metadata is already retrieved but simply isn't accessible within the sink.

Contributions welcome! I don't have much development time for this right now but would be happy to step in as a maintainer role to facilitate merging PRs if somebody tackles this.

Well, at least I have an idea for a backward-compatible improvement:
When a sink is not specified, the first value returned by http.request could be more than just 1. It could be a function that retrieves the body, with the sink passed as an argument.

I'm having similar issues.
I'd prefer not to change the API that much. A simple callback between shouldredirect and shouldrecevicebody fullfills my need.
*** http-orig.lua	2024-09-15 10:18:23.000000000 +0200
--- http.lua	2024-09-15 10:22:48.242167029 +0200
***************
*** 336,347 ****
--- 336,353 ----
      -- at this point we should have a honest reply from the server
      -- we can't redirect if we already used the source, so we report the error
      if shouldredirect(nreqt, code, headers) and not nreqt.source then
          h:close()
          return tredirect(reqt, headers.location)
      end
+
+ 	-- check if headers
+ 	if nreqt.processheaders then
+ 		nreqt.processheaders(headers)
+ 	end
+
      -- here we are finally done
      if shouldreceivebody(nreqt, code) then
          h:receivebody(headers, nreqt.sink, nreqt.step)
      end
      h:close()
      return 1, code, headers, status

My code then looks like this
	:::
	local content_length
	local function processheaders(hdr)
		content_length=assert(tonumber(hdr["content-length"]),"no content-length in header")
	end
	local request=
	{
		method = "GET",
		url = url,
		processheaders = processheaders,
		sink = sink,
	}
	:::
``

@alerque

Contributions welcome!

May I please request that my PR be reviewed?


minor use LOG(INFO) to print s3 client debug log if enabled

Proposed changes
So that user only need to set the BE config aws_log_level to enable/disable s3 client's debug log,
no need to set sys_log_verbose_modules at same time.
It is useful when we only want to debug s3 client.
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


Can't upload a track without surfaces.ini

Describe the bug
When trying to upload some tracks got an error Sorry, looks like that wasn't a track! in the UI.
Some investigations led to the conclusion that tracks without data/surfaces.ini not recognized as tracks by the ACSM front-end. But those tracks work pretty well on other servers.
If I manually add data/surfaces.ini with no content, it uploads and will work for me. But the checksum differs from others.
Broken track example:  https://drive.google.com/file/d/1T-Lq8ZePwOy6YcbmfOX2PJyprSmBquk-/view
I can provide more cases like this if needed.
Screenshots

Desktop (please complete the following information):

OS: linux
Browser chrome
Version 2.1.0-3

Some other tracks without surfaces.ini

Isola 2000 https://www.racedepartment.com/downloads/isola-2000-mountain-road.40090/
Tabasalu Kardirada https://www.racedepartment.com/downloads/tabasalu-kardirada.38262/
Shoubudani https://www.racedepartment.com/downloads/90s-gold-drift-spot-project-3-shoubudani-菖蒲谷.18629/
Tsubaki LINE https://mega.nz/file/0YkAwThA#6Ax7hKed06SkOQv_5U3KVKd6nTyrJMYNjQ1Djpr7Kpw


[On-boarding] Database Schema design

Following the group meeting of 29th Feb, 2020, create the data base schema with the Entity Relationship Design to be presented to the team by the next meeting

The DDL script has been shared with the team members for further review in preparation for setup on cloud services in sprint 2


Days Gone extremely low frame rate.

Name of the game, settings used etc.
Game: Days Gone from GOG.
Issue: Main screen and game play nets 0.2 FPS or lower, also GPU usage is approximately zero.
GPU: Radeon RX 6400
Driver: Mesa 23.2.1
Wine version: wine-8.18 (Staging)
DXVK version: v2.3-12-gf45911a2

Apitrace file(s)
https://drive.google.com/file/d/1L3x3Ih_zvgw-YrFRqxPvhUgURemmXA3j/view?usp=sharing

Almost same effect with
info:  Device properties:
info:    Device : Intel(R) Xe Graphics (TGL GT2)
info:    Driver : Intel open-source Mesa driver 23.2.1
info:  Game: DaysGone.exe
info:  DXVK: v2.3
wine-8.0 (Ubuntu 8.0~repack-4)

Menu fps stays on 20, but I don't expect from integrated graphics much. In game there is 0.2 fps.

I used dxvk-2.0, dxvk-2.1, dxvk-2.2, dxvk-2.3 and got same result.

I also tried 1.9 (where it was supposed to be fixed), 1.92, 1.94, 2.00, 210, 220, 230 and 2.3.12.
I hate to bug Phillip, he's made so much possible, I tried DXVK_ASYNC=1 (or something like that) and all the permutations I could derive from the proton page but to no avail.
I always buy GOG whenever possible, of course if Steam has a Linux port that gets bought as well.  I can wait until this title is working again, preferably under GOG an I can certainly wait until Philip can take a peek, it's only a game after all.

Strange. I had a go at it when it came out on GOG a few days ago and it was buttery smooth on my 3060, 60+ fps, no stutters whatsoever. Quite surprising for a UE4 title, in fact.

Got the game a few days ago - GOG version too. Works fine for me on a low end machine.
GPU: RX 560 2GB
Wine: Staging 8.18
Mesa: 23.2.1
Performance about 35fps using a mix of low/medium, 1280x720 resolution.

So far I've tried it on the following cards: 5700g vega 8, 5600g vega 7, RX 6400, RX 7600, GTX 1650 and they all behaved the same, 0.2 FPS which makes me think my DXVK is somehow broken, I compiled it with mingw  12.2.1-3, however I still got the same thing when I used winetricks dxvk.
On each test, I start with a fresh $WINEPREFIX, then do a symbolic link to 'GOG Games'.   All systems above had 32 GB of RAM except for the one with the GTX 1650 which had 16 GB.  This one is a real puzzler.

Does it work with wined3d?

I have visual issues (it is a mess of purple color) with wine native rendering and it is slow as dxvk.

This is a very old issue, you need to use a version of wine that supports fsync, like proton or wine-ge

@adolfintel Oh, yeah, that makes a lot of sense now. I don't use fsync, but rather the esync patches that come included with Wine Staging. @GurnBlanston Can you try running it with WINEESYNC=1?

@WinterSnowfall You got it, setting WINEESYNC=1 worked!  I was afraid I'd have to put training wheels on my motorcycle to play the game at 0.2 FPS!
It's still a little slow on my RX 6400 but now I have an excuse to use my RX 7600.
Thank you for the solution and thank you to everyone else who helped me resolve this issue.

Use WINEESYNC=1 when launching the game.

Hello, what about platforms that dont have esync?
Reopening, happens on FreeBSD too.
Wine 10.0-rc2
AMD RX 6700 XT
DXVK 2.5.1
mesa-devel 24.3.b.1891

DXVK can't do anything about a game running slow due to missing esync.

I can understand that, but I still find it kind of strange.
For me this would be the first game that would be unplayable without esync.
That it then only runs under one frame... But then completly fine when i open the menu etc...
Scenes that I crawled through before, run better on the 2nd run, a similar behavior as with shader cache.


TypeError: Failed to fetch

I'm having trouble reaching a rest API hosted through a special port.  I don't seem to have issues with any hosts that use the expected 80/443 ports.  I'm not sure what the cause is, but the special port is the only thing I can think of that's different from other hosts that work fine.
Example:
GET https://my.host:8081/status
Displays "Could not send request" error page.  The Chrome developer tools console shows:
extension.ts:230 TypeError: Failed to fetch
at o (chrome-extension://amknoiejhlmhancpahfcfcfhllgkpbld/index.js:2:13)
at N (chrome-extension://amknoiejhlmhancpahfcfcfhllgkpbld/index.js:3:1484)
(anonymous) @ console.js:36
runRequestOnExtension @ extension.ts:230

Since you're using the browser extension, can you make sure that you've added the origin with port number https://my.host:8081 to Hoppscotch Browser Extension's origin list?
Documentation: https://docs.hoppscotch.io/documentation/getting-started/setup#locally-served-apis

Thank you for the suggestion.  I see no change in behavior after adding https://my.host:8081 to the origin list of the extension.  To be clear, my.host is a rest API host, not the Hoppscotch web app host.  I am still using the web app from hoppscotch.io.

I'm also getting this error on both Edge (121.0.2277.106) and Chrome (121.0.6167.139) with the extension version 0.31. It doesn't happen on Firefox (122.0) with version 0.31 of the extension.