update OMIM xrefs - obsolete OMIM terms

The terms in this spreadsheet are obsolete in Mondo.
These need to be marked as MONDO:equivalentObsolete in Mondo (live class in Mondo and an obsolete class in the source ontology.)

[ ] @sabrinatoro  please use the unmerge/replace workflow to update these source annotations?
[x] please add the documentation on this workflow to our editors guide

@nicolevasilevsky
Can you please confirm that the current annotations in the spreadsheet have the OMIM annotation as "MONDO:equivalent"?
The workflow documentation is here: https://mondo.readthedocs.io/en/latest/editors-guide/bulk-removal/

@nicolevasilevsky
How did you get this list?
In order to update the annotation, I need to know what the annotations originally are. It looks like many of these annotations are currently "MONDO:ObsoleteEquivalent", and some (~20) are "MONDO:equivalentTo"
TO DO:
-[ ] get the updated list with the current source
-[ ] update the new source (I think that "MONDO:ObsoleteEquivalent" would have to be changed to MONDO:obsoleteEquivalentObsolete)

@sabrinatoro - it is from here:
https://github.com/monarch-initiative/mondo-ingest/pull/108/files#diff-b0d295ecd7d05fde88526ba07223a9bfbc08cc572d5ead04804bd3d49d2741b5
the file is: src/ontology/reports/omim_mapped_deprecated_terms.robot.template.tsv

Just spot checking this, I think the MONDO classes have been obsoleted, as well as the OMIM classes, so I think they need this annotation on the OMIM xref: MONDO:obsoleteEquivalentObsolete

Thank you @nicolevasilevsky
In order to update the annotations, I still need to know what the current source is. I will do some sparql query (etc) and figure it out.

@nicolevasilevsky : Looking into this more carefully, the situation is much more complicated than expected. For whatever reasons, a lot of these MONDO terms (I check about 10 of them, obsolete or not obsolete) have the OMIM x-ref twice, once with "MONDO:equivalentObsolete", and once with either "MONDO:obsoleteEquivalentObsolete" or "MONDO:equivalentTo".
This is a sign (imo) that there was a problem with the slurp.
This should be cleaned up:  we need to ensure that there is only 1 same omim id per MONDO term (though I am surprised this wasn't caught up by the QC).
CCing @matentzn and @joeflack4 as it looks very suspicious and they can advice on the best way forward (maybe a new issue in the other tracker?).
Some screenshots:

I can help manually review these and clean them up. I think if we run
sh run.sh make NORM
mv NORM mondo-edit.obo

it should merge the duplicate xrefs - I will try that. And then I think QC will catch it.

hm, no, that didn't do anything....
Aside from manually reviewing these and fixing them, I don't know what else to do.

UPDATE:
when the OMIM x-ref has more than 1 source, the removal and update of the annotation does not work.
--> need to update manually

thanks for doing this @sabrinatoro !


Extract blockOnGc utilities that wait for weak references to clear.

Fixes #975
/cc @lambcode

Removed awaitility - I can remove the polling too if we're nervous about it.


Play() should be declared as async as it is its default behavior.

When Play() is invoked it runs asynchronously.
I like it but it might be nice to declare it as an asynchronous function so it can be awaited.
https://user-images.githubusercontent.com/2603287/188267318-574c7492-06ab-46c0-9c71-ee3702cbf2c2.mp4

From the Video above.
_payerBeep.Play() method is async as you can see the soundwave running in parallel to the animations in the video.
Although I can't make the function call, part of the WhenAll statement.
On the other hand, TextToSpeech acts erratically in this condition (the reason why I commented it out).
Plugin.Maui.Audio deserves to be in the Microsoft.Maui.Media. (more than TextToSpeech for sure)
Sample can be found at:
https://github.com/pierre01/TimerFaceMockup

I think we could quite easily wrap the current player up inside an async wrapper class. I think we would end up with something like (written entirely here so will likely need some work):
public class AsyncAudioPlayer
{
    private readonly IAudioPlayer audioPlayer;

    public Task PlayAsync(CancellationToken cancellationToken)
    {
        var taskCompletionSource = new TaskCompletionSource<bool>();

        try
        {
            audioPlayer.PlaybackEnded += (o, e) => taskCompletionSource.SetResult(true);

            return taskCompletionSource.Task;
        }
        catch (OperationCanceledException oce)
        {
        }
    }
}

I did something like this:
var audioPlayer1 = audioManager.CreatePlayer(await FileSystem.OpenAppPackageFileAsync("sound1.mp3"));
audioPlayer1.Play();
await Task.Delay(Convert.ToInt32(audioPlayer1.Duration));
await Task.Delay(3000);  // time between the sounds
var audioPlayer2 = audioManager.CreatePlayer(await FileSystem.OpenAppPackageFileAsync("sound2.mp3"));
audioPlayer2.Play();
await Task.Delay(Convert.ToInt32(audioPlayer2.Duration));
I know that is not the best code, but it works for me!


Bug in event handling for carbon_v1 pools

Issue described in: https://github.com/bancorprotocol/fastlane-bot/issues/419

After testing I found that this code functions as-is. The issue is somewhere else in the code. Closing PR.


Fix off-by-one error in maxWaitQueueLimit

We're not counting ourselves against the maxWaitQueueLimit when enqueuing a waiter, which results in us going one past the maxWaitQueueLimit.

So, 'procedure syntax' actually means 'trailing commas' in this case :)

I wish there were a Scala 2.11 plugin to support trailing commas.  I accidentally leave them all over the place, and actually like them for diff reduction.  Thanks for the fix!


[question] Tattoos free for first character creation?

Is there a way to make applying tattoos free for the very first time someone has joined and is making their initial character?
I've found the getNewCharacterConfig() function but I'm unsure if I what I need to add goes in there/what to put to void the Config.TattooCost.. Is literally just Config.TattooCost = 0 in that function? or is it more complicated than that?

Figured it out. No longer needed.


pytorch的数据集只支持coco格式吗？我现在的yolo格式（yolov8)如何进行处理？

pytorch的数据集只支持coco格式吗？我现在的yolo格式（yolov8)如何进行处理？
主要是觉得转起来有点麻烦。数据集目录都要完全重构？

是的

：《  好的，肯定早就start了啊

this is no problem , you can use  yolo2coco.py and  find it in mmdet.

this is no problem , you can use yolo2coco.py and find it in mmdet.

hello~ I cannot find this yolo2coco.py script in mmdet folder, can you specify which directory it is under? thank u very much!

Hi, I'm very sorry for the misunderstanding, you can now find this file in mmyolo, it's located in mmyolo/tools/yolo2coco.
/dataset_converters/yolo2coco.py, you can see how he uses it Have a nice day!

Hi, I'm very sorry for the misunderstanding, you can now find this file in mmyolo, it's located in mmyolo/tools/dataset_converters/yolo2coco.py, you can see it

I found it! thank u very much!hope you have a nice day~


upgrade to more recent sesame / rdf4j release

Dear,
In connection with my previous bug report, a suggestion: the sesame 2.7.x release is a rather old release. I suggest to consider an upgrade to the latest release rdf4j.org
kr,
Bert

Indeed this is our plan:
https://github.com/ontop/ontop/issues/84

The next release will target the latest release of rdf4j


Bluetooth: GATT: Add function to get UUIDs for adv data

Add a function that can help an application get get a
partial or complete list of UUIDs for advertisement purposes.
Signed-off-by: Emil Gydesen<EMAIL_ADDRESS>fixes https://github.com/zephyrproject-rtos/zephyr/issues/33777

Rebased to solve merge conflicts

I am not convinced this is right thing to do.
Most of the GATT Services don't need to to be advertised. There might be many circumstances under which service should not be advertised e.g. depending on privacy, whether the advertisement is connectable or not etc. My feeling is that should be left to the user to build advertising data (like it is now).

I don't disagree with this, which is why this is just a helper function to retrieve that data, and then it's up to the application to determine whether to include it all, or some. However, perhaps this should return an array of UUIDs instead of the data preformatted for adv data?

Alternatively, bt_gatt_service could be extended with some callbacks that are called by some advertisement data builder, so that the service could decide whether it should be included in advertisement or not (depending on advertising type, privacy, etc).

That seems to be the opposite of what you write above, isn't it? IIRC there is no requirement for any service to have it's UUID advertised. Some services/profile may, however, require that service data (which is different from this) shall be advertised. For service data I could see a benefit of such a callback.

Rebased to solve merge conflict

@MariuszSkamra @asbjornsabo please re-review
@jori-nordic @alwa-nordic @MarekPieta @KAGA164 @jhedberg can you please take a look?

Closing based on feedback. The idea may be revisited at some later point if we have more requests for it.


Need support for ReactNative 0.64+

Hi team,
Currently, we are trying to run app using ReactNative 0.64 with Gradle version 6.9.
We are experiencing various conflicts between Java, Gradle, ReactNative, NodeJS, etc. We are not able to match these versions nor able to run the example given in the library.
Could you please help on the same?
Some of the errors that we are facing after every fix that we are trying:

Can you please help us understand what is the minimum and maximum versions of Java, ReactNative, Gradle and NodeJS required to run?
Thank you.

@jjerrythimothyj  did you find any solution please let me its very helpfull to me...


Cannot find module "meyda"

Trying to import or require the module after installing (tried both yarn and npm) gives me Cannot find module "meyda".
And indeed, the package.json for this module says "main": "./dist/node/main.js", but there is no such folder. This is probably related to https://github.com/hughrawlinson/meyda/issues/112. I don't have a dist folder in my node_modules/meyda. Also, the suggested unpkg link confirms that the package isn't distributed with the dist folder (which is suggested in the Wiki).

The current workaround is to navigate to the node_modules/meyda, and run npm i && npm run build. This makes everything work as expected.

Hey @lazarljubenovic - @2xAA and I will work on fixing this today :)

@lazarljubenovic this should be fixed in v4.0.1 which we've just published - could you check it out and confirm?

Thanks for quick responses! Looks like it's getting there but still missing
some deeper deps.
Module not found: 'jsfft-for-meyda' in
/home/.../node_modules/meyda/dist/node

On Sat, Apr 15, 2017 at 3:00 PM, Hugh Rawlinson<EMAIL_ADDRESS>wrote:

@lazarljubenovic https://github.com/lazarljubenovic this should be
fixed in v4.0.1 which we've just published - could you check it out and
confirm?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hughrawlinson/meyda/issues/196#issuecomment-294291886,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHTnkYhA5nayr_uWVK8xk9lmY3uHnobHks5rwL-AgaJpZM4M-R78
.

@lazarljubenovic could you try v4.0.2?

Error in ./~/meyda/dist/node/main.js
Module not found: [CaseSensitivePathsPlugin] `/home/lazar/pi-proj/node_modules/meyda/node_modules/jsfft-for-meyda/dist/fft.js` does not match the corresponding path on disk - File does not exist.

:(

Can you share more about your webpack setup? /node_modules/meyda/node_modules/jsfft-for-meyda looks like the wrong path.

Also, can you try rm -rf node_modules/meyda && npm install meyda

It's a create-react-app setup, so webpack is hidden away.
And yup, I am nuking and re-installing the package again every time. Still no luck.
I think I'm onto something, though. The meyda/dist/node/main.js has the require('jsfft-for-meyda') which is probably why it's looking deeper into node_modules.
If I change my import to import Meyda from 'meyda/dist/web/meyda.js', it loads, but logging console.log(Meyda) is an empty object, so I'm getting runtime errors like _meyda2.default.createMeydaAnalyzer is not a function.
If I try import * as Meyda from 'meyda/dist/web/meyda.js' or its equivalent const Meyda = require('meyda/dist/web/meyda.js'), I get an object with the only property default, which is again an empty object.

I think I'm onto something, though. The meyda/dist/node/main.js has the require('jsfft-for-meyda')which is probably why it's looking deeper intonode_modules`.

This doesn't sound right - whichever implementation of require you have should be looking at the top level node_modules.
Could you check that<EMAIL_ADDRESS>is a dependency of your project?
Also, meyda/dist/web/meyda.js is for use in script tags, and exports a global Meyda object. For commonjs imports you should import meyda/dist/node/main.js. You can set this to happen automatically in your webpack.config.js as shown in this commit

Interesting. I was deleting just meyda from node_modules (either with npm command or with rm -rf) and re-installing, but the issue remained. When I nuked the whole node_modules, I got it to work. I have to import from mayda/dist/node/main.js explicitly now though, but I don't mind it.
I don't have access to webpack.config.js, as I don't want to eject the create-react-app (it hides away the build process, and I don't want to fiddle with it).
So as far as I'm concerned, this is solved (since you provided the webpack configuration thing for people who have access to it). :ok_hand:
Now on to figuring out how to extract features! :dancer:

Great, thanks for helping us figure this out! I'll close this issue as you say it's solved, but I'll open another issue to track the require via browser property. It seems that webpack expects the file in package.json/browser to be a commonjs module, which is weird to me, but if more people report this, we'll take that to be the correct behaviour.
Enjoy the feature extracting, please let us know when you've built something cool!

I'm still getting this error through the client. Here's what I did:
$ sudo npm install -g meyda
$ meyda a b

module.js:557
    throw err;
    ^

Error: Cannot find module '../dist/node/main.js'
    at Function.Module._resolveFilename (module.js:555:15)
    at Function.Module._load (module.js:482:25)
    at Module.require (module.js:604:17)
    at require (internal/module.js:11:18)
    at /usr/lib/node_modules/meyda/bin/cli.js:27:15
    at Object.<anonymous> (/usr/lib/node_modules/meyda/bin/cli.js:222:3)
    at Module._compile (module.js:660:30)
    at Object.Module._extensions..js (module.js:671:10)
    at Module.load (module.js:573:32)
    at tryModuleLoad (module.js:513:12)

$ node --version
v9.3.0
$ npm --version
5.5.1

@erezsh this is actually related to #225 (a npm deployment issue) and I'd really like to fix it but I don't currently have permissions for the meyda npm account. paging @meyda/owners in case anyone has 10 min to look into it?

Thanks! It can now be require'd right from a standard npm install meyda, awesome.


Update JSON-RPC schema to v0.3.0

https://github.com/starkware-libs/starknet-specs/blob/v0.3.0-rc1/api/starknet_api_openrpc.json
Should resolve #401 (and close #412 ?)

We'll work on this on Software Mansion side.

Hey, fyi I'm working on this, you can see the progress here: https://github.com/bartekryba/starknet-devnet/pull/2
I'm hoping on opening pr for review later this week.

Any update?

Any update?

We're in progres of internally reviewing the linked PR with RPC update.

I just opened a PR https://github.com/0xSpaceShard/starknet-devnet/pull/467


Paged Resume

This PR extends example "resume" by adding a paged version. It addresses how to get a paged document in combination with a customized rendering.
The original example is unchanged, except that a preamble containing a running element has been added to resume.doc.
README.md has been updated to show what this demo is all about and how both version are generated.
This PR also removes examples/resume/cv.pdf which appears to have no function any longer.

Hi, https://github.com/ggrossetie/asciidoctor-web-pdf/blob/main/examples/resume/cv.pdf is still live. What's worse is that this version does not contain the svg icons for phone or email, it makes it look like there is a problem with the output asciidoctor-web-pdf creates. Best to remove it!


implement function that finds surface blocks out of all blocks inside polygon

this should go in polygonal area functionality, surface is defined as the first block that is walkable, bukkit disponibilises a api to do this in world.getHighestBlockAt(x, z)

Folks at the papermc discord told me that I should check the chunks are loaded first maybe using async? first implementation shouldn't worry about this though


pool: support more user-friendly search

It would be nice to have search function that returns list of oid.ID in the pool.
In most service applications that use sdk we have to define the similar function:
func searchObjects(ctx context.Context, sdkPool pool.Pool, cnrID *cid.ID, filters object.SearchFilters) ([]oid.ID, error) {
	res, err := sdkPool.SearchObjects(ctx, *cnrID, filters)
	if err != nil {
		return nil, fmt.Errorf("init searching using client: %w", err)
	}

	defer res.Close()

	var num, read int
	buf := make([]oid.ID, 10)

	for {
		num, err = res.Read(buf[read:])
		if num > 0 {
			read += num
			buf = append(buf, oid.ID{})
			buf = buf[:cap(buf)]
		}

		if err != nil {
			if errors.Is(err, io.EOF) {
				break
			}

			return nil, fmt.Errorf("couldn't read found objects: %w", err)
		}
	}

	return buf[:read], nil
}

Is there a need for flexible handling of the resulting slice, or is it mostly required to iterate over the elements? if so, I would recommend making the function an iterator for memory efficiency.

Most common use cases:

Iterate over the elements to get/head objects
Ensure that the resulting slice contains exactly one element and get/head it

If we implement a wrapper, there must be an argument limiting the number of OIDs in the returned result. We need this to prevent long time waiting|processing large search results.

@KirillovDenis could you provide a link to where this is used?
We can either have (*ResObjectSearch).ReadAtMost or (*ResObjectSearch).Iterate depending on what is more suitable.

https://github.com/nspcc-dev/distribution/blob/main/registry/storage/driver/neofs/neofs.go#L476
https://github.com/nspcc-dev/restic/blob/master/internal/backend/neofs/neofs.go#L139
https://github.com/nspcc-dev/rclone/blob/master/backend/neofs/neofs.go#L714

We need to decide what method is more basic - Read or Iterate - and implement other operation as a function over the basic.

I think the Iterate is more basic for search

@KirillovDenis in the first 2 links exactly one search result is expected. I don't think this check belongs to SDK as it to enforce some business logic invariants. Is it necessary to check this in the code you linked?

@KirillovDenis in the first 2 links exactly one search result is expected. I don't think this check belongs to SDK as it to enforce some business logic invariants. Is it necessary to check this in the code you linked?

It seems I didn't understand you question about links. I thought you need to see most common use cases (so I included different patterns of use).
If we speak about iterating and making some actions the following links are more appropriate:

https://github.com/nspcc-dev/distribution/blob/main/registry/storage/driver/neofs/neofs.go#L630
https://github.com/nspcc-dev/restic/blob/master/internal/backend/neofs/neofs.go#L164

done in #156


[WIP] Refactor ContainerLabelTagMapping

This is the first part of an effort to support AWS tag mapping in ManageIQ. The current code was basically hard coded for Kubernetes' containers instead of a generic fashion which resulted in several problems:

Poorly named table
No way to distinguish mappings between providers
Hard coded mappable types (Container types only)
Hard coded tag prefix (always "kubernetes")

The first part of this PR is to move the previously hard-coded MAPPABLE_TYPES array into its own table with two fields - name (e.g. ContainerNode) and provider e.g. (amazon). Then I populate the table with the current array values, along with one entry for Amazon. Presumably more will be added later.
In addition, I've moved the ContainerLabelTagMapping table to a more generic name of ProviderLabelTagMapping, which contains the original code (minus the hard-coded constants). To help with backwards compatibility, the original class was then subclassed from this. I also added the relationship model relationships.
Note that this PR will necessitate an update on the classic-ui side as well. Not only will they not be able to use the constants that I removed, a drop-down selection of providers needs to be selected. Based on which provider the user selects, the list of mappable types will then be visible.

@blomquisg @Fryguy Please let me know what you think.

@cben can you take a look at these changes, too?

 Closing for now.

:-/
If you have figured out what you want to achieve (eg. how it would interact with editable AWS labels?), please share with me & containers team...
Perhaps some simplifications could make it easier:

We can likely drop specific-value mappings (unless you have a use for them).  They were never documented nor exposed in UI.

Perhaps we could drop/simplify mappable types!  (Already exposed in UI, need to figure out at least partially compatible migration behavior, but let's entertain the idea...)

What if users could only map a label name in same way for all providers / models?
True, a "foo" label on kubernetes ContainerNodes and vs amazon Vms are probably completely unrelated, but is the harm of some unnecessary mapping worth the complexity?
When doing policies or reports based on tags, you'd only deal with one model at a time anyway...

If you need per-model, perhaps you don't care about provider?  Then it can remain 1 field of model base name.
If you need per-provider, perhaps you don't care about model?

And perhaps it can be per provider family (cloud / infra / container)?

Note that current support for mappable types is partial and surprising, not quite what it sounds like :-(

If you have completely different meanings for "foo" on ContainerNodes vs ContainerReplicators, and want to map them to independent categories, cool.
BUT if your goal was simple whitelisting — map "foo" on ContainerImages and ContainerGroups identically, same way as if you chose "" just not for other entities — then you can't.

For simplicity, especially of "garbage collecting" mapped tags when one deletes a mapping, the current UI maintains 1 category per mapping.  We didn't want to rule out 2 mappings of same name with different entities, so we made the UI create separate categories  — kubenetes:container_node:foo vs kubenetes:container_image:foo.  But it's not at all clear that's useful to users...


ci: Duplicate crate check in codecheck.py

Code to detect duplicate crates (i.e. crates pulled in as deps in multiple versions).
Applied to partity-scale-codec(-derive)?.
Draft to see how if it works in Github CI.

Related issue: #590


markdown help generation

in a past was added logic to generate description of cli args in markdown format as table and was leaved as is with hardcoded paths. It was used for our documentation in a past
I refactored it. Let discuss do we need it now and do we want to add it to repo

Thank you, let's keep this flag for now (we can remove it anytime).


thread authn provider specific plugins into nodeagent

https://github.com/istio/istio/issues/9035

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: quanjielin
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS [quanjielin]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
9a2990f79f792d98f71310c724f061d2a8644046
link
/test istio-integ-k8s-tests

prow/e2e-mixer-no_auth-mcp.sh
9a2990f79f792d98f71310c724f061d2a8644046
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
9a2990f79f792d98f71310c724f061d2a8644046
link
/test istio-integ-k8s-tests

prow/e2e-mixer-no_auth-mcp.sh
9a2990f79f792d98f71310c724f061d2a8644046
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-multicluster-e2e.sh
9a2990f79f792d98f71310c724f061d2a8644046
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: incfly, quanjielin
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS [incfly,quanjielin]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
9a2990f79f792d98f71310c724f061d2a8644046
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-multicluster-e2e.sh
9a2990f79f792d98f71310c724f061d2a8644046
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
0da32654bd8398a40dbaa36e643bbd619908c409
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
0da32654bd8398a40dbaa36e643bbd619908c409
link
/test istio-integ-k8s-tests

prow/e2e-mixer-no_auth-mcp.sh
0da32654bd8398a40dbaa36e643bbd619908c409
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Create Stream from file on Android

A Little help please, how can I create an Stream from a file using the DW.Android.Helpers unit?
I tried ...
              var jis : JInputStream;
              var uri: Jnet_Uri;

              Result:= TMemoryStream.Create;
              var Data: Jnet_Uri;
              var lFile: JFile;
              lFile := TJFile.JavaClass.init(StringToJString(SourcePath));
              var AFile:= TAndroidFileStream.Create(TAndroidHelper.JFileToJURI(lFile));
              Result.CopyFrom(AFile,AFile.Size);

              ASize:= Result.Size;
              Result.Position:= 0;

where source path is "/data/user/0/com.xpsistemas.HVPPortalMovil/files/CER_TEGJ780722QQ8.cer"

You should be able to do just this:
  var AFile := TAndroidFileStream.Create(SourcePath);

..and treat AFile like any other TStream descendant. There should be no need to copy it to another stream, or use any other Android Java classes. You should also be able to do this:
  AFile.SaveToFile(SomeFileName);

Where SomeFileName is a filename in a folder that your app can write to

It's correct... thank you very much!


fix for  < character

Fixes #39

@Ice3man543 This was a fast "fix" in order to give at least the reason of failure. I suppose it's not necessary in others, I debugged the issue and it happens just in this module. I plan to change also every passive source to be totally async and reduce the usage of long living objects to minimum and meanwhile I will also fix these not working modules, I can't allocate a lot of time right now, so it will take some time

@Mzack9999 Awesome work. Really appreciate you contributing to this tool. No need to hurry. Just, create a branch and make one source asynchronous. Others I can do following what you do to one. What are those not working modules?

I saw this one, and there was another one that produced weird results (probably an issue related to regex or parsing), later on filtered out by the final validation. I said generically "not working modules" to mean that I will try to fix these issues along the way if it happens I work on such modules


Fix updates of target networks

Description
Fixes the updates of target networks. The variable was assigned to the new target
networks and then used. Order was reverted.

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "enforcerprofile",
    "component": "enforcerd",
    "pr-id": "701",
    "commit-sha": "5d92a0174cd9cd75e3a4d78dbf3200a682610258"
  },
  {
    "project": "enforcerprofile",
    "component": "trireme-lib",
    "pr-id": "578",
    "commit-sha": "e7eea9fb96649a649a87be9852c5dbc48ff4faf8"
  }
]


Add package docs and readme

Add documentation and readme to top level and subpackages

(may want to merge https://github.com/apache/datasketches-go/pull/9 before - this branch is based on top of it)


git clone https://github.com/devbutlazy/Blum-AutoClicker cd blum-auto-clicker

    self.url = "https://web.telegram.org/k/#@BlumCryptoBot"

https://web.telegram.org/k/#@BlumCryptoBot


add sivatd bio

This pull request adds a bio for @githubteacher for the issue #16
Please review @jaw6

Will do

This looks good


Updated Icons on sidebar menu

Dashboard - import SpaceDashboardSharpIcon from '@mui/icons-material/SpaceDashboardSharp';
Pit Inventory - import HomeRepairServiceSharpIcon from '@mui/icons-material/HomeRepairServiceSharp';
Loaner Program - import ConstructionSharpIcon from '@mui/icons-material/ConstructionSharp';
Whomp Whomp Report - import CarCrashSharpIcon from '@mui/icons-material/CarCrashSharp';
Checklists - import ChecklistSharpIcon from '@mui/icons-material/ChecklistSharp';

is the Whomp Whomp Report on the figma?


chore: Bump to version 1.0.3

Codecov Report

Merging #10 (dea386d) into main (29c3ce7) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #10   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           56        56
  Branches         7         7
=========================================
  Hits            56        56

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add support for 3d graphics

← Arrows to the left

So - the idea is an abstract pipeline implementation that just leverages bgfx. We'll provide utility classes and methods for all the useful stuff, and a default 2d pipeline. There's no reason why you couldn't add 3D on top, its a very light framework as opposed to a full-fat engine (we believe most of the logic will live in your game, anyway.)
For now our priorities will be 2D (on 3D pipeline) but we'll create the project in a way it could potentially be extended in future.

Hello. (author of gfm and various D libs).
Very interesting project! Very cool that you are supporting D and I very mich agree with building upon bgfx.
I'm a recent user of Godot engine and some things have strike me in that engine:

first of all, Godot uses OpenGL directly which is a trap imho
Godot's Transform is very good with readable names, however it has no "shearing", or perspective deformation allowed. When Transform is converted to a row-major 4x4 matrix, the last line is forced to be [0, 0, 0, 1]. Though maybe understandability wins.
Godot duplicate a lot of functionality between 2D and 3D (navmeshes, pathfinding, etc.), despite 2D being mostly a subset of 3D. I agree 2D is easier to author and even make an engine for, but I think supporting 3D "only" still allows 2D games.

So I'd initially class this as implemented in terms of project direction.
Whilst we lack almost all major functionalities, the architecture is starting to shape up to be something that transparently handles 3D or 2D. For now we have basic Entity type which at minimum stores some positions. We'll extend them through tagging, and for example, have a SpriteRenderer looking for texture information. Theoretically we could support a MeshRenderer with the same tagging system and support a data driven entity design.


TableCommit builder's build method should be pub

Hi,
Since Catalog trait is pub, any struct used in the fns in the trait should be publically buildable.
But TableCommit's builder's builder method's scope is explicitly set to pub(crate).
So, the code out of the crate cannot call the update_table() fn since TableCommit cannot be built.
Is there any reason to limit the scope? Or is there any other way to build it?
If not, TableCommit builder's build method should be pub.
Thanks.

Hi, @liurenjie1024, I see. Thank you!

@liurenjie1024  Sorry, but there is another question.
I want to add a Snapshot to a Table but Transaction does not provide a fn to do it (append_updates is private). Is there another method to do that?
Thank you.

@liurenjie1024 Sorry, but there is another question.
I want to add a Snapshot to a Table but Transaction does not provide a fn to do it (append_updates is private). Is there another method to do that?
Thank you.

Hi, @zeodtr It's in roadmap but not implemented yet.

@liurenjie1024  Thank you for your quick answer.
I'm currently implementing my own Catalog to handle my own Iceberg-compatible storage. If adding a snapshot cannot be done using Catalog interface, maybe I should do it without the Catalog interface for now.
Thank you.


OTVCA issue

Hello,
I am trying to implement the OTVCA function.
The documentation says that it would return a 3D matrix but it returns a 2D tuuple.
It would be great if you could help me implement it.

Hello, what you are seeing is expected actually, OTVCA returns both the denoised HSI and the extracted features.
you can see this in the documentation here: https://github.com/Helmholtz-AI-Energy/HyDe/blob/main/src/hyde/transform_domain/otvca.py#L35
if there are any other issues please don't hesitate to ask

Hello,
Yess I see that now.
I was expecting the output like the matlab code.
But this worked.
Thank you so much :)
Best Regards,
Rupsa
Dr. Rupsa Chakraborty
@.***
CASUS - Center for Advanced Systems Understanding,Untermarkt 20, 02826 Görlitz
https://www.casus.science &
HIF- Helmholtz Institute Freiberg for Resource Technology, 09599, Freiberg
Helmholtz-Zentrum Dresden-Rossendorf e.V. (HZDR)

On 24. Aug 2023, at 09:59, Daniel Coquelin @.***> wrote:
Hello, what you are seeing is expected actually, OTVCA returns both the denoised HSI and the extracted features.
you can see this in the documentation here: https://github.com/Helmholtz-AI-Energy/HyDe/blob/main/src/hyde/transform_domain/otvca.py#L35
if there are any other issues please don't hesitate to ask
—
Reply to this email directly, view it on GitHub https://github.com/Helmholtz-AI-Energy/HyDe/issues/26#issuecomment-1691197352, or unsubscribe https://github.com/notifications/unsubscribe-auth/BB5U2LM37FP2SW4CT3EG3WTXW4CU3ANCNFSM6AAAAAA34SHMSM.
You are receiving this because you authored the thread.


#1329 Not fixed and causes an additional bug

The issue still exists that the popup gets displayed but is empty.
Also it created a new bug, that when all comments are filtered that it also does not show the description despite there being one. It shows an empty popup like above when it should display the description in this case.

It appears that the issue is that the preset being used internally to determine if the comments are filtered is different from the one used by Comment Filters. Can you check if you have multiple presets with the same name in the popup? Try deleting the duplicate ones and seeing if it works now, I don't know why it's duplicating presets...
As for the second bug, are you sure it's not just the same description of a previous giveaway being filtered? Can you link an example of a giveaway where it happens so I can try reproducing?


[consensus] cleanup EpochManager, RecoveryData

Motivation
Three commits inside:

unify EpochManager initial start path with epoch start path
unify LedgerRecoveryData with RecoveryData
fix the unverified SyncInfo in SyncProcessor

It's preparation for leader reputation work as we want to start a new EventProcessor upon state sync to propagate the latest commit info to ProposerElection.
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
Existing tests.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

@bors-libra delegate+

:v: @zekun000 can now approve this pull request

@bors-libra r=aching

:pushpin: Commit f66f30e has been approved by aching

:lock: Merge conflict

@bors-libra retry

@bors-libra r=aching

:pushpin: Commit 17b2ca6 has been approved by aching

:hourglass: Testing commit 17b2ca6984787e951236f4b31806085b380cee82 with merge f2eb03aa3022d693ebfc85f5867b5e28b1d87d66...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit 17b2ca6984787e951236f4b31806085b380cee82 with merge 0ee35dd6fefe0eca07e9d6bae5e40d3643615b8b...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: aching
Pushing 0ee35dd6fefe0eca07e9d6bae5e40d3643615b8b to master...


Atoencoder code is uploaded.

Coverage remained the same at 100.0% when pulling 9879fb5f9003677e0f700a33138f9ab7ad6cb6ce on hadikazemi:master into f91e19a2dfcc37a497d4edbc75bf94d16e7e3d92 on astorfi:master.

Thanks!


fix(atomic): improved search box accessibility for macOS VoiceOver

https://coveord.atlassian.net/browse/KIT-1908
Follow-up to https://github.com/coveo/ui-kit/pull/2286
In this PR, I changed the instructions for screen readers so they reflect the correct controls for macOS.
Also, I updated an assertive aria-live whenever a suggestion is selected, since VoiceOver on macOS doesn't notify of those by default.

Woops, I used the wrong JIRA. I meant to use https://coveord.atlassian.net/browse/KIT-1908


Enable devcontainers in repo.

Description
This is an initial PR to enable devcontainer support in the repo. The goal is to enable the various remote development scenarios (Codespaces, SSH, etc) so that we can test them. This is important because we've added logic to the apphost that can do things like detect it is running in Codespaces and using the Aspire repo under Codespaces is a good way of casually validating everything is working.

Outstanding issues

[ ] Dashboard URL shows as localhost:port/login?t=; whilst Codespaces will automatically remap that URL for us when we click on it, the redirect flow (GitHub auth) looses the token in the URL

Fixes # (issue)
Checklist

Is this feature complete?

[ ] Yes. Ready to ship.
[x] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[ ] Yes
[x] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

/azp run

@mitchdenny one place where we don't detect the codespace and print the right url is here:

We should change this too.

Yeah I was looking at this I made some tweaks last night as an experiment. I can put up a PR.

Also it seems like only certain playground examples work?


Deepworld Login / New Player

Good afternoon,
I just wondered how we would be able to fix the "Couldn't connect to the search service. Please try again in a bit." I'm currently doing this off of steam on a Windows 10 Computer. I would like to revisit the memories and things that happened with Deepworld.
Is there a fix to this?

Could be multiple things.
Are you sure you patched your client correctly?
Are you sure the server is on?

Inactive, feel free to re-open if you require more assistance.


Add function to drop sources and detectors

What does this implement/fix?
Add function to drop sources and detectors
Additional information
Modeled on https://mne.tools/stable/generated/mne.io.Raw.html?highlight=drop#mne.io.Raw.drop_channels

Codecov Report

Merging #379 (b51dbaa) into main (94d13f1) will decrease coverage by 0.22%.
The diff coverage is 87.69%.

@@            Coverage Diff             @@
##             main     #379      +/-   ##
==========================================
- Coverage   93.69%   93.47%   -0.23%
==========================================
  Files          51       51
  Lines        1841     1900      +59
  Branches      237      251      +14
==========================================
+ Hits         1725     1776      +51
- Misses         70       76       +6
- Partials       46       48       +2

Impacted Files
Coverage Δ

mne_nirs/channels/_channels.py
80.95% <76.47%> (-5.26%)
:arrow_down:

mne_nirs/channels/__init__.py
100.00% <100.00%> (ø)

mne_nirs/channels/tests/test_channels.py
100.00% <100.00%> (ø)

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 94d13f1...b51dbaa. Read the comment docs.


docs/bug: setting Gi suffix as given in docs causes installation to fail down the line

Short Description of the issue
Setting the node_limits.memory as given in the docs here
interlink_ip: <IP_ADDRESS>
interlink_port: 30443
interlink_version: 0.2.1-patch2
kubelet_node_name: my-civo-node
kubernetes_namespace: interlink
node_limits:
    cpu: "10"
    memory: 256Gi
    pods: "10"
...

and then generating a values.yaml by running ./interlink-install. (generates the values file below)
virtualNode:
  CPUs: 10
  MemGiB: 256Gi
  Pods: 10
  HTTPProxies:
    HTTP: null
    HTTPs: null

and then running the helm command
helm --debug upgrade --install --create-namespace -n interlink multipass-docker oci://ghcr.io/intertwin-eu/interlink-helm-chart/interlink  --values values.yaml

causes failure in deployment with the below logs
k logs -f deployment/multipass-docker-node
                                                                                ─╯
Defaulted container "vk" out of: vk, refresh-token
time="2024-10-29T10:26:01Z" level=info msg="Loading Virtual Kubelet config from /etc/interlink/InterLinkConfig.yaml"
panic: invalid memory value 256GiGi

goroutine 1 [running]:
main.main()
        /app/cmd/virtual-kubelet/main.go:119 +0x1e98

This is either a bug or a error in docs. This can be resolved with a change in the docs.
Environment

Operating System: Ubuntu
Other related components versions: 0.3.1-patch

Steps to reproduce
-- given above
Summary of proposed changes
update docs and interlink-install --init command which generates the .interlink.yaml file

should be fixed in 0.3.4


add back opt flag in pipeline class

Hi, Could you share how to use （add back opt flag in pipeline class）in python

Hey @pengjianlinbs
Do you mean by the flag for a pose-graph optimization at the end of sequence?

Closing due to no response, feel free to reopen with a comment.


Godot 4 port

Don't know If this is of any kind of interest yet but made a quick port, unfortunately gut has yet not been ported.
Most changes is:

Changing Reference to RefCounted (https://github.com/godotengine/godot/pull/49312)
Updating setget syntax
Updating calls to super init to new syntax

This could potentially still need some more work.

Ping @quentincaffeino

Sorry for taking so long to respond. I will try to review this pr soon.
One thing I see right away is new syntax. Is it mandatory to use new setget syntax with godot 4? Reason I'm asking is because I'm trying to support  older versions of godot in this project. Hence why such complicated solution for callbacks was implemented, callbacks weren't possible before.
If old syntax is deprecated and deleted from godot then I might think of having two branches for some time, since godot 4 will take some time to be widely adopted.
If old syntax is still available I'd ask you to revert those changes related to syntax.

Sorry for taking so long to respond. I will try to review this pr soon.
One thing I see right away is new syntax. Is it mandatory to use new setget syntax with godot 4? Reason I'm asking is because I'm trying to support  older versions of godot in this project. Hence why such complicated solution for callbacks was implemented, callbacks weren't possible before.
If old syntax is deprecated and deleted from godot then I might think of having two branches for some time, since godot 4 will take some time to be widely adopted.
If old syntax is still available I'd ask you to revert those changes related to syntax.

No problem, we all got a life outside github right? 😅
But regarding the the setget syntax to my understanding to old syntax is deprecated, but I could have misinterpreted.
There is info about it here: https://github.com/godotengine/godot-proposals/issues/844
Perhaps 2 branches would be nice, and just to clarify I'm wouldn't be confident to say that this I ready to merge. First of all I have only used this repo lightly so I might have missed stuff and also gut isn't ready so that sure is something that holds it back as well.

Godot 4 Beta 1 Error;
res://addons/quentincaffeino/console/src/Type/Vector2Type.gd:29 - Parse Error: Function "str2var()" not found in base self.
Kind regards

Beta 1 Error;
res://addons/quentincaffeino/console/src/Type/Vector2Type.gd:29 - Parse Error: Function "str2var()" not found in base self.
Kind regards

This could be a bug in godot but nothing prevents us from doing explicit conversion by doing something like float(point[0])

It's not a bug, var2str and str2var has been renamed to var_to_str and str_to_var. Should probably redo this pr since it's been quite some changes, might have time this weekend or next week.

You're right, however using var.to_float() would be better than str_to_var imo

Should be some-what working now, haven't tested everything yet

What current situation? I don't know if this would be part of the thread, but wouldn't it be of interest to also put the godot4 branch in the asset library?

I plan to put it on asset lib after godot 4 is released

Another question would be that this pr is not with the wrong target branch?

Totally possible, I haven't yet thought through how I'll manage development for multiple versions. Probably godot 4 will go in dev and I'll keep a separate branch for godot 3 to simplify dropping it when godot 3 becomes too old or something like that.
Also I will prob drop dev branch and everything will go directly to master. Dev is kinda useless in this project case.

@scriptsengineer right now if you are willing to try out godot 4 version I'd be glad if you will report any bugs you find :)

Also working on ability to use package manager like npm (or hopefully any npm-compatible) because asset lib is pretty vanilla

So I came from Unity and I agree, it lacks some kind of management, looking forward to some kind of response in the proposals made here
https://github.com/godotengine/godot-proposals/issues/2262
https://github.com/godotengine/godot-proposals/issues/2263

About godot4 update I think it is extremely necessary since there is a lot of demand in this version.
I will work on this, currently investigating the godot3 to godot4 update script, for some reason the godot4 branch has some corrupted scenes that still keep old version of godot3 ids

Update: The mentioned corrupted scene godot issue #69177 is due to a bug with godot 4 beta 6 that has now been fixed.
Testing with godot 4 beta 5 version everything works normally. This pull request can be sent to a master, if the idea is to keep the godot 3 version on a separate branch.

What's the status on this? I'd suggest making the develop branch compatible with Godot 4 and create a 3.x branch from the current develop branch.


can't run masakari-hostmonitor

hello I try to run the masakari-hostmonitor on the centos7 and there are some errors.I read the shell file :

I don't know how to configure the cib .

Hi,
Please help me to understand how far you setup the environment.
In the compute nodes,
[1] setup pacemaker cluster and check whether its work fine.
[2] Install the masakari-hostmonitor, instancemonitor, and at the end processmonitor.
In your last comment "I don't know how to configure the cib", I guess you need info for how do [1].
If you are familiar with chef, you may refer to chef recipes on,
https://github.com/ntt-sic/masakari-deploy/blob/master/cookbooks/pacemaker/recipes/default.rb
And config files of corosync can be found in,
https://github.com/ntt-sic/masakari-deploy/tree/master/cookbooks/pacemaker/templates/default
If you are not familiar with chef, I can give provide some guide to how to setup and configure the cib.


0.10.2 introduce PDF to cut off on the left

Hi. We did upgrade to 0.10.2 because of security issues, but PDF is broken now. Can you please help us?
I did downgrade and issue is gone, but we don't wanna to downgrade because of xmldom security issues.
Correct PDF (0.9.0 - 0.9.3 version)
Napoleon Maclovio Sandy Saavedra_20 Fundo_2024 (2).pdf
Incorrect PDF (0.10.2 version)
Napoleon Maclovio Sandy Saavedra_20 Fundo_2024 (3).pdf
Some configs:
// function that returns config - start

  const doc_width = 11.69; // A4 measures 210 × 297 millimeters or 8.27 × 11.69 inches
  const doc_height = 8.27;
  const dpi = 120; // targeting ~1200 window width
  const img_width = doc_width * dpi;
  const img_height = doc_height * dpi;
  const win_width = img_width;
  const win_height = img_height;

  const html2canvasOpts = {
    width: img_width,
    height: img_height,
    windowWidth: win_width,
    windowHeight: win_height,
    scale: 2,
    letterRendering: true,
    useCORS: true,
    removeContainer: false,
  };

  const jsPDFOpts = {
    orientation, // 'landscape'
    unit: 'in',
    format: [doc_height, doc_width],
  };

  const pdfConfig = {
    filename: `${fileName}.pdf`,
    image: { type: 'jpeg', quality: 0.7 },
    html2canvas: html2canvasOpts,
    jsPDF: jsPDFOpts,
  };

// function that returns config - end

const HTMLToPDF = async (html: string[], fileName: string, output?: string) => {
  const a4PdfConfig = getA4PdfConfig('landscape', fileName); // returns pdfConfig

  // Initialize a PDF document using html2pdf library
  let doc = html2pdf()
    .set(a4PdfConfig) // Set the PDF configuration to A4 size
    .from(html[0]) // Convert the content of the first element to PDF
    .toPdf();

  // Convert the HTMLCollection to an array for easier manipulation
  const elementsArray = Array.from(html);

  // Loop through additional elements starting from the second one
  for (let j = 1; j < elementsArray.length; j += 1) {
    doc = doc
      .get('pdf') // Retrieve the existing PDF document
      .then((pdf: any) => {
        pdf.addPage(); // Add a new page to the PDF document
      })
      .from(elementsArray[j]) // Convert the content of the current element to PDF
      .toContainer()
      .toCanvas()
      .toImg()
      .toPdf();
  }

  // Use Promise.all to wait for all pages to be converted
  await Promise.all(elementsArray.map(element => doc.from(element).toPdf()));

  if (output) {
    return doc.output(output);
  }

  return doc.save();
}

I did noticed that it cut off when you change screen (window) width

@eKoopmans
Hello. Do you have any ideas?

Did figure out. The issue was because of CSS + windowWidth: win_width, windowHeight: win_height for html2canvas.
We are generating A4 size and using the same size for windowWidth and windowHeight what is wrong and it did move content somehow to the left. So I removed windowWidth and windowHeight and starting to use constant size at CSS for HTML container.

I had the same issue, what I did was create a wrapper with fixed dimensions based on A4 portait.


Subscribing to candles on 2 different timeframes (or intervals)

Hi @Graeme22 thank you for this great project.
I'm struggling to learn asyncio well enough to do some more advanced things.  I've read the advanced section here https://tastyworks-api.readthedocs.io/en/latest/data-streamer.html#advanced-usage.  I've also read through the tests/test_streamer.py.
My question is how would you go about getting candles on 2 different timeframes (or intervals).  At first I thought 2 different streams, then I decided 1 stream with 2 subscriptions was the right way.
async def get_candles():
    session = as_tasty_session_token()
    dt = datetime(2024,7,12,7,0,0)
    async with DXLinkStreamer(session) as streamer:
        await streamer.subscribe_candle(['SPY'], '15m', dt)
        await streamer.subscribe_candle(['QQQ'], '30m', dt)
        async for c in streamer.listen(EventType.CANDLE):
            print(c)

if __name__ == "__main__":
    asyncio.run(get_candles())

That code seems to get wedged on the "async for" so I'm guessing multiple .subscribe_candle() on one stream isn't supported?
I'm just wondering if there is a way to do this within the same process?  If multiple streams is the answer, how many streams can you have without trouble?  And is there a best practice for having 2 streams running in one process?
Thanks you!

Hey! Thanks for opening this issue. This is actually a bug in the SDK, the code you posted should work correctly! I fixed it and it will work in the next release.

Can you confirm this is fixed for you in v7.9?

@Graeme22, I tested v7.9 and it works exactly as it should -- both timeframes are coming through at the same time.  Thank you!

Great!


test: string utils

Summary

Please summarize your changes, describing what they are and why they were made.

...
Changelog
- Please write a list of changes

Testing
Platforms

Please select any platforms where your changes have been tested.

Desktop

[ ] MacOS
[ ] Linux
[ ] Windows

Mobile

[ ] iOS
[ ] Android

Instructions

Please describe the specific instructions, configurations, and/or test cases necessary to test and verify that your changes work as intended.

...
Checklist

Please tick the following boxes that are relevant to your changes.

[ ] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[ ] I have verified that new and existing unit tests pass locally with my changes
[ ] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

Edited .filter((n): n is RegExpMatchArray => !n) to .filter((n): n is RegExpMatchArray => n !== null) because it was causing the tests Test check to fail


Does sass @at-root work in styled-component

Is there any option to run sass @at-root in styled-component  ? Currently  selecting parent element with & works fine but unfortunately @at-root not seems to be working.
export const ButtonContainer = styled.div`
  display: block
  ${(props) => props.fixed ? `
    position: fixed
    right: 0
    bottom: 0
    left: 0
    z-index: 100
    width: 100%

    @at-root body {
      background-color: tomato;
    }
    ` : ''
}

I'm not sure what that does, but it's definitely not something we want to encourage from looking at your example :wink:

@mxstbr Looking for the solution to apply for @at-root feature https://sass-lang.com/documentation/file.SASS_REFERENCE.html#at-root

@athimannil I think this is what you want: https://codesandbox.io/s/m73v75rz09
const Box = styled.div`
  height: 100px;
  width: 100px;
  background: blue;

  body & {
    border: 1px solid red;
  }
`;

@probablyup Not really
I just need to apply properties for parent class. Somethings like this http://sassbreak.com/getting-back-to-our-roots/

Oh I see. Yeah we can't support that for the moment... the createGlobalStyle helper in v4 should make it a little easier to arrive at this functionality with something like this though:
const Comp = styled.div`
  color: red;
`;

const GlobalStyle = createGlobalStyle`
  body ${Comp} {
    color: blue;
  }
`;

etc


Fix EasyRtcAdapter send/broadcastData to use WebRTC when possible.

EasyRtcAdapter's sendData method was using easyrtc.sendPeerMessage which doesn't use WebRTC data channels when they are connected. The broadcastData method was also using WebSockets via easyrtc.sendWS.
This fix makes the EasyRtcAdapter's sendData and broadcastData methods use easyrtc.sendData which uses WebRTC when available and WebSockets as a fallback.

Great catch! Merging now and will release as 0.3.1 since this is very incorrect behaviour.


Stop some callers from bypassing the CurrentDataType cache

This PR concerns SetDataType(...):
In the log, I was surprised to see so many unneeded "TYPE I" commands, even though there is a caching mechanism (similar to the one for "CWD" / "PWD").
It turns out that some callers (rightly) go in at ...NoLock(...), thereby bypassing the cache.
This PR moves the caching code down into the no-lock part of the code. The diff looks crazy, I know, maybe lay the two versions side by side :-)

Thanks for finding this and fixing it!

Thanks for finding this and fixing it!

Published.
https://www.nuget.org/packages/FluentFTP/35.2.3

Published.
https://www.nuget.org/packages/FluentFTP/35.2.3


Support dotted JSONPath notation for partitioner key

We need that for partitioning cirrusSearchElasticWrite job since it's key is inside a subject.

Coverage increased (+0.1%) to 88.589% when pulling 8e53211eeb92c6ccb0701ab52aaaa451fa303636 on Pchelolo:partitiner_key into 3ce1ac5e2fb55f5cde7f3aee62cf07c976445b32 on wikimedia:master.


Apple doesn't allow for this wording

For this to work in apps that go on the App Store, you cannot have "allow" to be on the buttons that prompt the user to initiate permissions...this wording should either be changed or made editable

tutorial just didn't indicate to use ALT button var


Support javascriptreact filetype

I've recently updated VIM, and jsx files start being recognized as javascriptreact instead of javascript.
On my distribution, Arch Linux, ftplugin, syntax and indent folders have a javascriptreact.vim which load javascript configuration:
For example, ftplugin/javascriptreact.vim:
" Placeholder for backwards compatilibity: .jsx used to stand for JavaScript.
runtime! ftplugin/javascript.vim

Is there a way to achieve the same thing in syntastic? (If not, I could send the PR copying all javascript syntax_checkers with the javascriptreact name, but I don't feel the react approach).
SyntasticInfo output:
:SyntasticInfo
Syntastic version: 3.9.0 (Vim 801, Linux, GUI)
Info for filetype: javascriptreact
Global mode: active
Filetype javascriptreact is active
The current file will be checked automatically
Available checkers: -
Currently enabled checkers: -

Thanks in advanced!

Please read :h 'syntastic_filetype_map'.

Thanks, I couldn't find that configuration!


Contributors in README.md?

Can I add contributors list in README.md?

Hey @priyanshumaitra
I think you can make a .md file listing all the contributors with pictures and GitHub links
and then link that file in the Readme.md
PS:- I'll assign this to you only and only if you promise to keep updating the new contributors to the list, I'll wait for your confirmation
Thanks for considering to contribute to this Repo
Really means a lot! 💖💖

It will be updated automatically. And will show something like this below 👇

Alright @priyanshumaitra work on it
And ya just make it in any other file rather readme because it'll cluster it and link that file in the readme
Thank you for considering to contribute in this repo
Really means a lot!!
💖💖


tool: add cargo xflowey custom-vmfirmwareigvm-dll

To quote the tool's CLI help:

Encapsulate an existing pre-built IGVM file into unsigned
vmfirmwareigvm.dll resource DLL.
Unlike build-igvm, this tool will NOT build OpenHCL from scratch. This
tool streamlines the process of building the in-tree vmfirmwareigvm_dll
crate (which requires setting various env vars, installing certain
dependencies, etc...).
NOTE: This tool is primarily intended for use by Microsoft employees, as
open-source deployments of OpenHCL typically load the IGVM file directly
(rather than being encapsulated in a resource-DLL).

Unfortunately, I still don't quite know what magic is required to properly cross-compile the vmfirmwareigvm_dll crate from WSL2 / Linux, as the dependency on rc.exe makes things a bit more complicated than the typical windows cross-compile case. In the meantime, I've added some checks to ensure folks are only running this tool on native windows.

Thanks for this! Would you add usage instructions to the guide?

Thanks for this! Would you add usage instructions to the guide?

Given that this is a Microsoft-employee oriented tool, I'm inclined to say that the documentation in the CLI is sufficient in the context of open-source. That said, I'll send out a PR which updates our Microsoft-internal guide to include a pointer to this new tool, as well as some extra docs on how to actually use the DLL internally.


feat: check for blank/missing agency_id

Summary:
This PR provides changes to verify:

agency_id is provided in all cases (WARNING if agency.txt contains an unique row)
agency_id is not blank when provided (WARNING)

New notices

AgencyIdBlankNotice
AgencyIdMissingNotice

Expected behavior:
A notice should be issued if:

agency_id is blank
agency_id is not provided when agency.txt  only contains 1 row

Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Run the unit tests with gradle test to make sure you didn't break anything
[x] Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
[x] Linked all relevant issues
[ ] Include screenshot(s) showing how this pull request works and fixes the issue(s)

Closing. We will not continue work on this PR since the parsing process makes this use case not relevant: blank and empty Strings are nullified so that a GtfsAgency will never be created with agency.agency_id = "   "

Closing. We will not continue work on this PR since the parsing process makes this use case not relevant: blank and empty Strings are nullified so that a GtfsAgency will never be created with agency.agency_id = " "


Doc bug in rb-nutshell

From RT #131269:

On https://docs.perl6.org/language/rb-nutshell, it says:

No space allowed before the opening parenthesis of an argument list.
foo (3, 4, 1); # Not right in Ruby or Perl 6 (in Perl 6 this would
               # try to pass a single argument of type List to foo)
foo(3, 4, 1);  # Ruby and Perl 6
foo 3, 4, 1;   # Ruby and Perl 6 - alternative parentheses-less style

This doc was probably written before the GLR, because even if the ‘foo (’ version does produce a
single argument of type List, then the “Single argument rule” makes it behave the same as the others.
Another bug, further down the the file is:

perl6 example.p6 --length=abc
   Usage:
     c.p6 [--length=<Int>] [--file=<Any>] [--verbose]

Clearly that’s a copy-pasto, since the file can’t be named c.p6 and example.p6 both at the same time.

Thanks. Fixed in https://github.com/perl6/doc/commit/f897a9c7ab0dc732ddf2a826b505f8949d1c75dc

then the “Single argument rule” makes

The OP is confused about the rule. The example is right.


Animation inside setTimeout function not firing

Hi, I'm having issues firing a animation inside a setTimeout function. My code looks like this:
function resize_functions() {
    resize_timeout_ID = window.setTimeout(function () {
        anime({
          begin: function () {
            $('.floating-button').removeClass('resizing-hide');
          },
          targets: '.floating-button',
          opacity: [0.2, 1],
          duration: 100
        });
        // other code
    }, 150);
}

So the code is supposed to show a semi-hidden button when the window has stopped resizing. resizeFunctions() is called each resize via a resize event listener and the timeout is cleared if the window is still resizing. However, the animation is not firing when it is inside the setTimeout function. Does anyone have any idea what could be the problem? Is this a known issue? (Great library btw, love it so far)
Regards,
jox

I think the issue may stem from instantiating the animation inside of your event handler. A better approach might be to instantiate the animation outside the handler and then use the included playback controls to start the animation.

Thank you for the tip, it solved the problem. A heads up for anyone who encounters a similar issue in the future: If the animation is not firing the best approach would be to use playback controls or if that is not possible, you could also use a slight delay with the setTimeout function. Thanks again @alexchantastic


Docs warn navigator ios

ping https://github.com/facebook/react-native/issues/795#issuecomment-139895307

lgtm cc @foghina

cc @sahrens

Hi there! Please rebase this and I will merge :)

@facebook-github-bot shipit


Upgrade to<EMAIL_ADDRESS>

<EMAIL_ADDRESS>incorporates changes in substack/node-trumpet#57 that fix the issue with live reloading.
Close #75.

Thanks!


Adding APIs to expose valid data size information at a store level

We currently have no API in ambry to provide space consumption statistics. This is a problem because statistics like valid data size per log segment is needed by compaction. Current valid data size per serviceId and containerId is also needed by quota management and containerization of ambry in the future.
Additions to BlobStore

StatsEngine: this class is responsible for most of the business logic for the creation and maintenance of the data structures needed to forecast valid data size per serviceId and containerId for some time into the future. The data structures will also provide valid data size per segment information for compaction purposes. A scan of the index will be performed here and construct the necessary data structure to forecast expiry of blobs for some period of time depending on the memory allowance. New puts and deletes that happens after the scan can be captured by BlobStoreStats to keep the valid data size information up to date.

BlobStoreStats: this class is mainly responsible for handling requests for valid data size by extracting requested information from previously constructed data structures. The BlobStoreStats is also responsible for maintaining the data structures by taking appropriate action for new puts and deletes that arrive after the data structure is built.

Data structure classes

Bucket (time slot): this is used to consolidate the data changes in the store over some time span into a single data point. For example, if a 10MB blob is expired at 1:15PM, a 20MB blob is expired at 1:30PM and a 35MB blob is expired at 1:50PM then a Bucket could be constructed with a value of -65MB. The Bucket will also have a time stamp, in our example the time stamp could be 2:00PM indicating that by the end of 2:00PM a total of 65MB worth of blobs have expired.

StatsEvent: this class is used for the purpose of handling new puts and deletes during a scan. StatsEvent captures/buffers all needed information of puts and deletes that happen while a scan is in progress. Once the scan is finished these buffered StatsEvent will then be processed and added to the data structures accordingly.

@xunyin8 Could this be added #530 instead of creating a new issue?

@vgkholla I think this is different enough to #530 that it will be a complete rewrite. Do you still want me to do that?


UI Upgrade for kanye.rest API

The public API page for kanye.rest can use a UI upgrade as well which would be a bit more funky and aesthetic to Kanye West. #2107

@sayonnnnnn Thanks for opening this issue ;)
We are not maintainers of this API. You must indicate an update to the UI of the page in their repository: https://github.com/ajzbc/kanye.rest


High performance issues in FlatList

I have a SectionList where is render DropShadow component with text and images. On Android 12 the performance very low comparing to View component, where infinite loading hundreds of items is smooth.
I didn't change anything else in my very complex code, just replaces it by View and i feel 100% performance boost.
Do you have any measurement or ideas why it was so slow?

Have you tried following example?

I already used a different solution, but if anyone would have performance issue, i hope it will help.
CellRenderrerComponent={DropShadow}

@radekzz Hey man, is the problem solved?

I ended up with stop using the component and use my own styles unfortunately