Getting Module not found error

Hey @IGassmann I am getting below error even after I installed the module.
Module not found: Can't resolve 'remark-directive-rehype'
  14 | });
  15 | // import remarkDirectiveRehype from "remark-directive-rehype";
> 16 | const remarkDirectiveRehype = dynamic(() => import("remark-directive-rehype"), {
     |                                                                               ^
  17 |   ssr: false,
  18 | });

Package.json
"remark-directive": "^2.0.1",
"remark-directive-rehype": "^0.4.0",

Can you please help me out here ?

might be related to https://github.com/syntax-tree/unist-util-map/issues/6

This is now fixed since https://github.com/IGassmann/remark-directive-rehype/releases/tag/v0.4.1

Having similar issue:
Module not found: Can't resolve 'remark-directive-rehype' in


Update to support Null-safety; added new parameters

Update to support null-safety
add new parameters:

tabBarHeight (double)
tabBarColor (Colors)
tabSize (double)
tabIconSize (double)
tabIconSelectedSize (double)
tabIconSelectedColor (Colors)

@therezacuet Please review, approve, and update Pub.dev package.

Please refer to the below for the updates:
https://github.com/kimmanwky/Motion-Tab-Bar
https://pub.dev/packages/motion_tab_bar_v2


Small corrections to Getting Started with Pub Article

This issue was originally filed by<EMAIL_ADDRESS>
I have some small corrections for (http://pub.dartlang.org/doc):
The table of contents doesn't match the list of sections. "Installing
dependencies" is there twice.

This issue has been moved to dart-lang/pub#246.


FbI work on 11.0.x ?

you know FbI work in 11.0.x with homebrew laucher ?
thanks
Mataka.

Sorry for my bad english i'm swiss.

nope

Any tutorial on how i can downgrade? i dont really want to hardmod...

Lots on GBA temp. This site isn't a support forum. And you either need another hacked 3DS, or you need to hard mod.


Best way to use of extract_clap_text.py

In gereneral script generate image from text like "The sound of something".
What best way of represent something? For example: I do classification of meow. There labels like "bitish", "persian", "siberian", etc.
The text will be "The sound of british" but I presume right text must be "The sound of british cat"? And what I get if laion/larger_clap_general model dont known about my specific cat? Random cat meow?

Hi! Thanks for your interest in our work. From my experiences, the clap model may not have the ability to distinguish different types of cats. I think just using "the sound of a cat" or "an audio clip of a cat meow" works the best for the current model.


Multiple loss functions

DS-2145
Tonks currently expects all tasks to be multi-class and has a number of hard coded, the main focus of this was to allow for multi-label tasks using Pytorch's implementation of binary cross entropy loss.
The main thing ive done is removed hard coded aspects of the learner switched it over to reference a dictionary of tasks with their associated losses and a number of other previously hard coded features... For a custom loss function (not cross entropy or binary cross entropy) a user has to specify the different functions to populate that loss dictionary.
I am not sure on the best way to handle this so im open to suggestions.
Pull Request Checklist

[x] All tests in the tests folder pass with a local build
[x] Pull request includes a description of why we are doing this
[x] Init files import new capabilities to appropriate level of package (if applicable)
[x] CHANGELOG has been updated
[x] Version in _version.py has been updated
[x] README has been updated (if applicable)

Cool! Can you help me make sure I understand the difference between multi-label and multi-task in terms of (1) use cases and (2) network architecture?
I think in terms of use cases, the difference is that you would need to go multi-task rather than multi-label when you have some data that has labels for one task but not the other. Is that right?
I think in terms of network architecture, the difference is that a multi-task network splits for the different tasks, whereas a multi-label network just has one head that returns a single vector with values for different labels in different positions. Is that right?

I think in terms of use cases, the difference is that you would need to go multi-task rather than multi-label when you have some data that has labels for one task but not the other. Is that right?
yeah so TECHNICALLY we could make all of our multi-task things a gigantic multi-label thing, but like you said we dont have labels for all of them, or it might make sense to break out parts. But if we can build multi-label datasets for specific problems then we can allow multiple values for things like color/occasion/style which we are limited on currently since we have to force like jeans into a single category with our multi-class setup.

I think in terms of network architecture, the difference is that a multi-task network splits for the different tasks, whereas a multi-label network just has one head that returns a single vector with values for different labels in different positions. Is that right?
yes I think that is correct in general. So while people could then use tonks to train a large multi-label network if they wanted with this current setup I still want the added flexibility of mixing and matching additional tasks which can either be multi-class or multi-label where we can train with additional datasets and such per usual in Tonks.

idk if that clears things up. Happy to also chat in person for this

I think that seems fair. I didn't have a chance to get to it today, will try and work through some details tomorrow, but I agree with you.

Tagging myself so I'll be notified when this merges @peterskipper :) 👋

Ok so most recent commit gets us to a point similar to keras... Basically keeping some set of supported losses and accuracy functions but also allowing users to input custom functions as they see fit.
So oddly we're now back at a point where we can support custom loss functions but kinda stumbled back into a better way of doing it.


Improvement on control flow docs

Adds some missing documentation on for :reduce, minimal example on it's use, plus some general tidying up.

Also added a missing section for dofor based on my quick experiment with it.


add autodocs

@whoburg, ready for review.

Let's auto-generate these (via the sphinx Makefile?)

It is done!

(and pushed to master so I could confirm it was working...)


System.Net.Sockets.SocketException

Hello. Thanks for this beautiful library!
I'm try to read data from TCP socket, but...
Client.ReadStream.Read(..) generate exception:
System.IO.IOException: Unable to read data from the transport connection: Operation on non-blocking socket would block. ---> System.Net.Sockets.SocketException: Operation on non-blocking socket would block   at System.Net.Sockets.Socket.Receive (System.Byte[] buffer, System.Int32 offset, System.Int32 size, System.Net.Sockets.SocketFlags socketFlags) [0x00016] in <9624f2cab6ac4ffc9c31e9469d40591a>:0    at System.Net.Sockets.NetworkStream.Read (System.Byte[] buffer, System.Int32 offset, System.Int32 size) [0x0009b] in <9624f2cab6ac4ffc9c31e9469d40591a>:0     --- End of inner exception stack trace ---
But when I try use Async method client.ReadStream.ReadAsync(..) - I have deadlock... no return from await.
P.S. ReadTimeout - not -1 :) v2.0.2 stable
P.S.S. Sorry for my English.

Howdy,
That's a strange one that I haven't heard of before - can you give any more information about the platforms involved (iOS, Android, other?), and the nature of the socket you are connecting to?
Your english is good :)

I found the reason of System.IO.IOException My TCP server ( hardware module RM04 work in mode - TCP <-> UART-arduino) is..not answering after sometime! It's hardware bug. I think Exception just mean no bytes to read after Timeout. Tested work on Android.
However issue with ReadAsync is remained.

Ah I understand the hardware issue - if you are getting a deadlock it is almost certainly due to the calling code - if you post some of that we may be able to investigate 👍


Missing right side of image in ocr result

Hi,
I have a big problem, that tesseract often only interprets the left side of the image.

All Numbers from the right side are missing in the result.
Any ideas?
Regards
Ingo

Please respect guidelines for posting issue - use tesseract user forum for asking support.


Tag new release

As the main concourse repo includes v1.0.0 of this lib the fix from #3 is unfortunately still not available. This is a first step, I'll also open an isuse in the main repo to bump the version there as well.

tagged as v1.0.1 - no need to open an issue on the main repo, pretty sure dependabot will open a PR for this automatically (though there could be some delay)
thanks!


Unnecessary check

Could you please explain why this check is needed ?
(parseInt(req.get('x-forwarded-port'), 10) === parseInt(req.get('x-forwarded-port'), 10))

This is to makes sure that req.get('x-forwarded-port') is an int. NaN is not equal to itself as a rule so this will account for that case. Let me know if you have any further questions.


Cannot run Vue3 app with Vite

After following the standard-installation I can't run the app, the error is
Error: Cannot find module 'pug'
According to the same Getting started doc, pug and sass are needed only for partial setup, but for now I want to include everything.
My package.json and main.js looks more or less the same as in https://stackblitz.com/edit/waveui-3 (apart different versions).
Is there anything I'm missing?
The versions I use:
"vue": "^3.4.15",
"wave-ui": "^3.11.0"

Thanks!

Apparently the problem was in importing wave components in mine. Without the import everything works perfect.


Fix incorrect use of JNIEnv. It is not supposed to be cached.

Hello,
I came across this problem when we started to use jblas in our distributed computation clusted in a multithreaded environment. JVM started to crash constantly giving us some weird crash dumps jblas was mentioned there very often but crash dumps were all over the place and we couldn't pin point the problem. After looking at the JNI wrapper I noticed that xerbla() callback uses cached JNIEnv* which is not recommened.  I made a multithread test for xerbla which was just calling SanityChecks.checkXerbla() in multiple threads simultaneously. When I set a good mount of threads ~ 100 the test made jvm to crash every time.
Here is proposed fix for this. After this change my test passes (no crashes) with any amount of concurrent threads for extended periods of time. I didn't include this test in PR because I was not sure it will be desirable to have long running unit tests.
Please consider including this fix into the main branch. Jblas is a very useful library it would be great to make more reliable.
Thank you.

Hi azenkov,
aaah, I see what I did there. Yes, that obviously wouldn't work. A bit swamped with work right now, but I'll fully review & merge the PR in the next few days.
Thanks for your efforts!
-M

Hi, mikiobraun,
Thanks a lot for Jblas. And does it merge in the new one?  Since I might run Jblas in the same situation.
Songpeng

Hello azenkov,
finally found the time to look into this. Thanks a lot for your contribution! I will merge it, then do a little bit of cosmetic clean-up.
Reading this: https://stackoverflow.com/questions/9642506/jni-attach-detach-thread-memory-management it seems that one should not call AttachCurrentThread all the time as it potentially create a new instance, as well as Detaching afterwards. I will fix both these things afterwards.
Best,
-M


Since 2.3.0 Android build is broken

Hello,
I am working on a cross-platform app using:

<EMAIL_ADDRESS>At the beginning I've started Lottie integration with version 2.3.2
On iOS it worked ok.
Then I tried to build the project on Android and obviously something went wrong:

* What went wrong:
A problem occurred configuring project ':app'.
> Could not resolve all dependencies for configuration ':app:_debugApkCopy'.
   > Could not resolve project :lottie-react-native.
     Required by:
         project :app
      > Project :app declares a dependency from configuration '_debugApkCopy' to configuration 'default' which is not declared in the descriptor for project :lottie-react-native.

I've looked everywhere with no solution. Then I tried to rollback to older versions: 2.3.1/0 are broken too.
Passed to 2.2.7 and it builds. Unfortunately at the moment I decide to show an animation then it does not appear and get a Warning of type:
Failed prop type: undefined is not an object(evaluating 'Object.keys(flattened)') in LottieAnimation ...
any idea?

We had the same problem, but managed to resolve it by changing the path in settings.gradle:
project(':lottie-react-native').projectDir = new File(rootProject.projectDir, '../node_modules/lottie-react-native/src/android') to src (previously lib)


Problem with MagicAPI

Hello, I have a problem with the MagicAPI.
Can  you contact me on Skype ?
My Skype: dr.blamo
Thanks.

I don't get on Skype or do live chat (sorry!) but I'd be happy to help here if you can elaborate.

When I configure the wand to add a property It doesn't work.
The property appear in the Wand description only, but has no actual effect.
(With the MagicAPI ==> configure)

If it shows in the lore it should be active- lore is dynamically generated from the wand properties.
Could you please paste the code you're using and let me know the effects and what you expect to see?

    public void mana(Player p) throws SQLException {

	int volontecount = instance.getArmorLores(p, "Volonté");

	if (volontecount == 0) return;

	float mana = (float)volontecount * (1.f/24.f); p.sendMessage("0");
	instance.mage.Configure(p, "xp_max_boost", mana);

}

Hm- I don't think there is a "configure" method on Mage, does that code compile?
If you want to configure the wand the player is holding, it'd be like mage.getActiveWand().configure().
Also "xp_max_boost" doesn't work on wands, that's only for armor- and there isn't an API to get armor, I don't think. You'd have to get the ItemStack from the Player inventory, then wrap it in a Wand and use configure on that.
If you just want to raise the max mana on a wand it'd be "mana_max"

Ow.. because it's working when I execute the command IG.
/wand configure xp_max_boost 100

That will show up in lore as a boost on the wand, but I don't think it actually has any effect.
That command also does the equivalent of what I wrote above- mage.getActiveWand().configure()

No, the effect is ok.
For instance when I run this command :
/wand configure xp_max_boost 1
I have 400 mana, I can cast a spell of 200 mana twice

@NathanWolf ?

I'm not really sure where you're at right now, sorry. Are you using mage.getActiveWand() ?
Calling configure() on a Wand instance is exactly what the "/wand configure" command does, so if you're not getting the same results that is very odd.

I'm using getWand, with the WandActivatedEvent

it could be something weird with that event- have you tried calling setName() or some other method that would create a really obvious effect?

Even though the wand description (lore) changes, so that we can SEE the boost written in the description, that boost is in fact not active. It's just written, and that's it.
Without setName().

What I was trying to tell you before is boosts don't work on wands, that's only for armor. Shouldn't make a difference how you set it.
At least, it's not intentional for it to work on wands. If you want to increase max mana of a wand, just increase the max mana.
Boost is meant for armor- so for instance you can wear the full set of Wizard's Robes and your wand will get a boost to max mana and regeneration.

Yes but the effect work when I execute the command in-game :

/wand configure xp_max_boost 1

Not working with :

Configure(p, "xp_max_boost", 1.f);

I have no idea why that would be, just that I know it's not intended to work on wands. The behavior may be unpredictable.
If you think it's working the way you want when using the configure command, maybe try calling Wand.deactivate() before configure, and activate() after. Looking at the code for the "configure" command, that's all it does differently (all of the commands use the API):
https://github.com/elBukkit/MagicPlugin/blob/master/Magic/src/main/java/com/elmakers/mine/bukkit/magic/command/WandCommandExecutor.java#L887
I do want to stress yet again that it's not intentional if it does work that way. I really didn't think the code even looked at the boost attributes on wands at all.

It's working when I deactivate / activate, but it's flooding the event :/
//Mage mage = instance.getMagicAPI().getMage(p);

//instance.getMagicAPI().getWand(p.getItemInHand()).deactivate();
instance.mage.Configure(p, "xp_max_boost", 1.f);
//instance.getMagicAPI().getWand(p.getItemInHand()).activate(mage);

Oh yeah.. since an activate would fire the event :\
Um- maybe can you tell me what exactly you're trying to accomplish? Maybe there's a simpler way.

Well, I want to create a plugin that allows players to customize their armors with gems. For short, you can put "intelligence gems" on your armor to boost your maximum mana and your spell damage. You can put up to six gems on a single piece of armor. In such a case, each gem must increase the maximum mana of the player by five (3 gems = 15 max mana).

Might be interesting to do that with crafting- I'm not sure if you could simply config it in Magic (maybe), but making a custom crafting plugin that adds mana boosts to armor would be pretty easy.
If you're applying the bonuses to armor, I don't think you have to worry about the activate event, or activating/deactivating a wand. Especially if the armor is in a workbench when it gets upgraded.
If you want to update a piece of armor while a player is wearing it, there is no API method for updating but I could add one. Mage.armorUpdated is the method you'd want to call, but it's not currently exposed in the API.

The thing is, you don't actually craft the armor (in a workbench), you go see a NPC that pops up a menu where you put the armor and the gem, the plugin takes them and give you the updated armor. However, the armor may be updated while the player is wearing it by many ways : the armor breaks, the player goes in a world with different inventory, the player joins a mobarena, etc. In thoses cases, the armor of the player is removed without any actual action from him.

That sounds cool!
And .. I may be missing something, but this sounds a lot easier than you're making it out to be.. maybe!
So when the NPC is giving the player their armor, you could do this like:
MageController controller = magicAPI.getController();
Wand enchantedArmor = controller.getWand(theArmorItemStack);

// It occurs to me that I could add some accessor methods here to make this easier..
Map<String, Object> properties = new HashMap<String, Object>();
properties.set("mana_max_boost", 2.0);
enchantedArmor.configure(properties);
// The itemstack *may* be changed here, though probably not.
theArmorItemStack = enchantedArmor.getItem();
// Then give this ItemStack to the player

They will then have a piece of armor that, when worn, will increase their mana. If they lose the armor, they lose the mana boost.
Does that sound like what you want?

I'll try that a bit later, I have to go now, but that sounds like something that could work with what I want ! I will keep you updated as soon as I'll try it.
Thanks you !

I tried your solution, but your plugin remove lores.
And my plugin use lores to add some abilities.
Is there a way to keep lores ?

Not really, no. I don't think you're going to be able to make wands useable by a plugin that needs specific lore.
You can mess with the description, but that's just one line.

Could you add the possibility to add a lore with your plugin, that stays ? That would be very useful, with something like : magicAPI.lore.add(String text);

No, not easily. The lore is all dynamically added based on Wand NBT data. It'd be extremely non-trivial to add arbitrary lore. Sorry!

Or a possibility to add xp_max_boost on a wand ? :/

I still don't get why you need that? It makes very little sense- a boost is meant to boost a wand via an accessory item. Wands themselves have max mana that you can just increase if you want more mana on it.

The boost will apply a bonus of mana, so I don't have to get the mana of a wand, and players could compare the base mana with the armor mana.

Hm, alright. Well looking at the code I may be mistaken, it looks like boosts on wands ought to work, so long as it's their active wand.
Mana boost calculation starts with the wand's boost and adds in any boosts from the armor.
It looks like you were seeing this working, so maybe the only problem is that you have to deactivate+reactivate the wand while configuring, and you can't really do that during the wand activate event? It should work fine if you're doing some sort of "give your wand to an NPC to get a boost" thing though.

Not with an NPC, if they unequip their armor, they can save the mana and try glitch.
The only problem, it will deactivate / reactivate when the player call the WandActivatedEvent.

I'm extremely confused about what you're trying to do.
If you want a mana boost when the player is wearing certain armor, the mana boost goes on the armor, not on the wand.
Magic will handle applying the boost when the player wears and removes armor. There is no need to mess with the wand data yourself.

I want to apply a mana boost on the wand, to prevent glitches that would happen if it was applied to the armor. Furthermore, with xp_max, it wouldn't be possible to add directly mana on the wand, otherwise I would need to find a solution to save the base mana of the wand. I must use xp_max_boost to achieve what I want to do.
The boosts will be updated each time the player calls the event WandActivatedEvent, because it's not Magic which will define the % boost, it will be the lores on the armor, via my plugin, wich will apply certain Magic effects.

Ah so the part I was missing is that you're trying to use your own lore-based plugin so you can't use the feature that is, sadly, made specifically for what you're trying to do.
Oh well...
Offhand it looks like if you switch to the PreActivateEvent it should work. Try this with just configure(), no need to deactivate and reactivate.

But I can't find the PreActivateEvent

https://github.com/elBukkit/MagicPlugin/blob/master/MagicAPI/src/main/java/com/elmakers/mine/bukkit/api/event/WandPreActivateEvent.java

Make sure you're building against the latest Magic, that event is relatively new (like a few months old)- 6.2.2 is the latest version, Maven should be able to find it.

Yes but my server is still on 1.8.9 version :/
The WandPreActivateEvent is working but there is a bug, the method that is configuring the wand need to be fired twice to refresh correctly the wand.

So does that mean you found a workaround and got it working? Or no?
The 1.8 builds of Magic aren't going to be getting any major updates.

Yes it's working but I have to trigger event at two times.
instance.mage.Configure(p, "xp_max", mana);
instance.mage.Configure(p, "xp_max", mana);

Looking at the code, one ugly work-around for this might be to call Wand.armorUpdated after you call Configure.
You'd need to cast the API Wand to com.elmakers.mine.bukkit.wand.Wand since this method is not exposed to the API.

Okey, I'll try thanks :)
I wanted to know to if I can make a MagicItem unstackable.

@NathanWolf ?

Um.. I don't think I have anything in the configuration to allow that directly. As far as I know the only way to do that is to put some random NBT data on the item.

Okey, thanks :)

And can't we add a green colored, bold, name on a magic item ?

@NathanWolf ?

Hey sorry I never responded to this - but yes you can use & escape codes to do color and formatting in magic item and wands names. Hope you figured it out!

I'll try thanks


Hello

Hello

See: https://twitter.com/malgorithms/status/1173785764544569345

And just had to turn off new GitHub Space Drop registrations also, due to bad actors. Even after all these years, the Internet can find a way to let ya down. http://keybase.io/airdrop

???
Отправлено из Mail.ru для Android вторник, 17 сентября 2019г., 19:30 +09:00 от Rafal W<EMAIL_ADDRESS>:

See:  https://twitter.com/malgorithms/status/1173785764544569345

And just had to turn off new GitHub Space Drop registrations also, due to bad actors. Even after all these years, the Internet can find a way to let ya down.  http://keybase.io/airdrop
—
You are receiving this because you authored the thread.
Reply to this email directly,  view it on GitHub , or  mute the thread .


Fix snapshots caching in Hermes

Resolves https://github.com/AppScale/appscale/issues/2827
https://ocd.appscale.com:8080/job/Daily Build/5222/

This PR resolved https://github.com/AppScale/appscale/issues/2803.


Added Partial Serialization support + Missing Default & Debug derives

Allows using serde to serialize rigidbodies among other things.

I updated this to work with the latest version of nphysics: https://github.com/Laaas/nphysics

@Laaas Thank you for updating this! Would you mind porting your changes in this PR, or perhaps open a new one?
Also, could you please double-check that the serde-serialize feature of nalgebra is really not enabled if ncollide's serde feature isn't enabled either. If I remember correctly there is some kind of Cargo bug that unifies all the features independently from the target.cfg statements. One way of checking this is to compile (in verbose mode) ncollide without the serde feature and see if the command line compiling nalgebra includes the serde-serialize feature or not.

You are unfortunately right.

@Laaas Thank you for checking. I guess one way of working around this is to add a feature serde-serialize = [ "serde", "nalgebra/serde-serialize" ] to ncollide instead of all those target.cfg.

@sebcrozet what should I do with this PR then?

You can make a pull request over my fork and it will update this pull request.


JGRP-2477 Reintroduce support for configuring a JChannel via URL

See: https://issues.redhat.com/browse/JGRP-2477
This reintroduces the constructor org.jgroups.JChannel#JChannel(java.net.URL) to ease library usage.
Infinispan 9.4.x (9.4.19) uses a URL to load the JGroups configuration from the infinispan configuration, when an external jgroups stack configuration file is used:
See: "Using an external JGroups file" in https://infinispan.org/docs/9.4.x/user_guide/user_guide.html#using_an_external_jgroups_file
When creating the JGroups stack, infinispan uses the constructor in org.jgroups.JChannel#JChannel(...) which takes a URL as parameter.
See: https://github.com/infinispan/infinispan/blob/9.4.x/core/src/main/java/org/infinispan/remoting/transport/jgroups/JGroupsTransport.java#L586
However this constructor is only present in versions up-to including 4.0.22 https://github.com/belaban/JGroups/blob/4.0.22/src/org/jgroups/JChannel.java#L121
and was removed in later versions.
Because of this users of infinispan 9.4.x, who use the "Using an external JGroups file" configuration approach cannot upgrade to new JGroups versions, like 4.2.3. Some that some applications, are currently bound to Infinispan 9.4.x which prevents upgrading to a newer Infinispan version. It would be great if one could at least upgrade to newer JGroups versions in that case.

Superseded by #494


Update to Griptape v.0.28.0

Update Griptape ComfyUI nodes to work with Griptape v0.28.0

In progress - LMStudio driver isn't working with this latest version - so I'm working on fixing that.

Done!


Extraneous log messages within validate function

Description
We are integrating protovalidate for Protobuf validation in a Python service and noticing thousands of strange log messages coming from within validate(request).
Steps to Reproduce
We have the following in requirements.txt:
protovalidate==0.3.1
And in our middleware.py:
from protovalidate import validate, ValidationError
...
    try:
        parsed_request = ParseDict(json.loads(body), request_proto_class())
        validate(parsed_request)
    except ValidationError as validation_exc:
        logger.exception(validation_exc)
        raise HTTPException(status_code=422, detail=str(validation_exc.violations))

Expected Behavior
Validation function should not log extraneous messages.

Additionally, we're finding that the validate function hangs for > 10 seconds.
Screenshots/Logs

Environment

Operating System: Initially found the issue in one of our testing environments in Kubernetes, but the issue is reproducible on our local macOS machines.
Protobuf Compiler & Version: Buf 1.29.0
Protovalidate Version: 0.3.1

Possible Solution

Additional Context

Hey @carolyn-lyra, I suspect those logs are coming from protovalidate-python's usage of the cel-python library — there's a related issue on that repo about extraneous logging; can you try out the code snippet in the comment to see if it fixes the extra logging? And let us know if the hanging still occurs with that change in place.

Hey @carolyn-lyra, I suspect those logs are coming from protovalidate-python's usage of the cel-python library — there's a related issue on that repo about extraneous logging; can you try out the code snippet in the comment to see if it fixes the extra logging? And let us know if the hanging still occurs with that change in place.

Thanks, this fixed the logging issue for us! We'll deploy to a test environment to see if the performance issue still persists.

Hey @stefanvanburen - unfortunately, we're still seeing the latency increase issue even after deploying your suggested logging fix. The validate step is taking on average over 5 seconds to complete, and sometimes spiking over 10 seconds. For additional context, our service's payloads are often times quite large (a few thousand fields per payload), but this wasn't an issue when we were using protoc-gen-valdiate. Has your team seen latencies increase with this new library vs. the deprecated protoc-gen-validate library?

Hey @carolyn-lyra, I don't think we've seen similar increases so far. Just to confirm, you're re-using the same exported validate func across your middlewares, correct?
I suspect that most of the slowness can be traced down to cel-python (one related issue in https://github.com/cloud-custodian/cel-python/issues/68, and I've asked about a new release as there have been some performance improvements since the last one). However, any additional details you can provide us from your end in order to reproduce the slowness would be great!

Hi @stefanvanburen -
Yes, we are using the validate func from from protovalidate import validate in our middleware.
Thanks for linking the issues for context. I've pasted a scrubbed version of the Protobuf message below with the latency regression. Typically, the repeated field repeated DataSchema data = 1; repeats for 100-200 times per request. A few of the nested repeated fields in DataSchema can also have >30 items. One of the sample payloads we pulled for example had over 7k fields total. Let me know if you need any additional clarification on the message below!
syntax = "proto3";

import "buf/validate/validate.proto";
import "google/protobuf/wrappers.proto";

message Type1 {
  string value = 1 [(buf.validate.field).string.pattern = "^[a-zA-Z0-9]{8}-(?:[a-zA-Z0-9]{4}-){3}[a-zA-Z0-9]{12}$"];
}

message Request {
  message DataSchema {
    double field_1 = 1 [
      (buf.validate.field).ignore_empty = false,
      (buf.validate.field).double = {
        gte: 0,
        lte: 100
      }
    ];
    google.protobuf.DoubleValue field_2 = 2;
    double field_3 = 3 [
      (buf.validate.field).ignore_empty = false,
      (buf.validate.field).double = {
        gte: -1000,
        lte: 1000
      }
    ];
    Type1 field_4 = 4 [(buf.validate.field).required = true];
    google.protobuf.Int32Value field_5 = 5 [(buf.validate.field).int32 = {
      gte: 1,
      lte: 10
    }];
    google.protobuf.Int32Value field_6 = 6 [(buf.validate.field).int32.gte = 0];
    google.protobuf.BoolValue field_7 = 7;
    google.protobuf.BoolValue field_8 = 8;
    google.protobuf.Int32Value field_9 = 9 [
      (buf.validate.field).ignore_empty = false,
      (buf.validate.field).int32 = {
        gte: 0,
        lte: 730
      }
    ];
    google.protobuf.DoubleValue field_10 = 10;
    google.protobuf.DoubleValue field_11 = 11;
    string field_12 = 12;
    Type2 field_13 = 13; // "Type2" is an enum
    google.protobuf.Int32Value field_14 = 24 [(buf.validate.field).int32 = {
      gte: 1,
      lte: 120
    }];
    repeated string field_15 = 15;
    google.protobuf.BoolValue field_16 = 16;
    google.protobuf.BoolValue field_17 = 17;
    repeated string field_18 = 18;
    repeated string field_19 = 19;
    google.protobuf.BoolValue field_20 = 20;
    Type3 field_21 = 21; // "Type3" is an enum
    google.protobuf.Int32Value field_22 = 25 [(buf.validate.field).int32 = {
      gte: 1,
      lte: 120
    }];
    google.protobuf.DoubleValue field_23 = 23;
    Type4 field_24 = 26; // "Type4" is an enum
  }
  repeated DataSchema data = 1;
}

Hi @stefanvanburen, following up on this - any updates on the performance issue?

hey @carolyn-lyra, no current updates; I still suspect most of the perf gains we can expect come from upstream improvements and a release. It'd be nice to convert your case above into a benchmark that we can look to confirm that intuition, but I'm not sure when I'll have time to take a look at that. Open to a PR or a demo repo for further investigation, though!


Populate device_info with 'Unknown' for unrecognised device

No driver installed for a BCM94360CS2.

I've been avoiding this as returning "Unknown" is not going to be very helpful for people. I have a potential fix but I'm waiting for testers, but if that doesn't work I'll revisit this in the future.


Feature/294 display unzipped file sizes

Related Issues:

EQ-294

Main Changes:

Added csv_size, gz_size, and zip_size to the mv_profile_stats table.
Updated the national downloads page to display the zipped and unzipped file size.
Fixed a "skipped header" accessibility issue on the national downloads page.

Steps To Test:

Verify your app and etl are pointed to your local db.
Create the csv_size, gz_size, and zip_size columns. To do this you can either drop your current database, so the etl gets a fresh start or you can run the query below to add and pre-populate the columns. The pre-populate query leaves the new column values as nulls.
Copy the ready.json and status.json files from S3 into your local app/server/app/national-downloads folder.
Run the etl
Start the app
Navigate to http://localhost:3000/national-downloads
Verify the table now displays both the zipped and unzipped file sizes.
Run accessibility checkers
Verify "skipped header" issue is fixed and verify no new issues introduced from table changes

TODO:

[x] - Add these new columns to the dev database prior to merging.
[ ] - Add these new columns to the stage database after merging and testing on dev.

BEGIN;

ALTER TABLE logging.mv_profile_stats RENAME CONSTRAINT mv_profile_stats_pkey TO mv_profile_stats_pkey_old;
ALTER TABLE logging.mv_profile_stats RENAME TO mv_profile_stats_old;

CREATE TABLE IF NOT EXISTS logging.mv_profile_stats
(
	profile_name VARCHAR(40) NOT NULL,
	schema_name VARCHAR(20) NOT NULL,
	num_rows INTEGER NOT NULL,
	last_refresh_end_time TIMESTAMP WITH TIME ZONE NOT NULL,
	last_refresh_elapsed VARCHAR(20) NOT NULL,
	csv_size BIGINT,
	gz_size BIGINT,
	zip_size BIGINT,
	creation_date TIMESTAMP WITHOUT TIME ZONE NOT NULL,
	PRIMARY KEY (profile_name, schema_name)
);

INSERT INTO logging.mv_profile_stats (profile_name, schema_name, num_rows, last_refresh_end_time, last_refresh_elapsed, creation_date)
SELECT profile_name, schema_name, num_rows, last_refresh_end_time, last_refresh_elapsed, creation_date FROM logging.mv_profile_stats_old;

DROP TABLE logging.mv_profile_stats_old CASCADE;

GRANT SELECT ON TABLE logging.mv_profile_stats TO eq;
GRANT ALL ON TABLE logging.mv_profile_stats TO postgres;

COMMIT;

Great, thanks for making those changes!


Windows title is not santized (xtitle)

Describe the issue
While I was debugging polybar, I had a notepad opened to note some strings and I noticed that my main bar (not the one I was debugging) was freezing. After investigation I found that the culprit was the title of my notepad  "%{l}%{A1:xkeyboard/switch:}fr  %{u#e60053 +u B#e60053"
Expected behavior:
The title of my window should be displayed without having my bar freezing and the tags must be ignored.
Actual behavior:
Polybar freeze or if the tag formatting is correct, the tag actually change the xtitle module.
To Reproduce
A minimal but complete config with which the problem occurs:
[module/title]
type = internal/xwindow

format = <label>
label = %title%

[bar/example]
monitor = eDP-1
monitor-fallback =
monitor-strict = false
override-redirect = false
bottom = true
fixed-center = true
width = 100%
height = 20

font-0 = "Ubuntu Condensed:size=10;2"
font-1 = "icomoon\-feather:size=10;2"
font-2 = "xos4 Terminus:size=12;1"
font-3 = "HanaMinA:style=Regular:size=13;3"

modules-left =
modules-center = title
modules-right =

Having a window title which have a tag in it

Polybar Log
Post everything polybar outputs to the terminal when you run it and the issue occurs below
* Loading config: /home/loma/Code/repositories/polybar/doc/config
- config: Current bar section: [bar/example]
* Loaded monitor eDP-1 (1920x1080+0+0)
* Bar geometry: 1920x20+0+1060; Borders: 0,0,0,0
- bar: Attach X event sink
- bar: Attach signal receiver
- controller: Install signal handler
- controller: Setup user-defined modules
* Starting application
- controller: Main thread id = 1
* Starting module/title
* Eventqueue worker (thread-id=3)
- module/title: Thread id = 2
* Entering event loop (thread-id=1)
- bar: Create renderer
- controller: Attach config watch
- renderer: Get TrueColor visual
- renderer: Allocate colormap
- renderer: Allocate output window
- renderer: Allocate window pixmaps
- renderer: Allocate graphic contexts
- renderer: Allocate alignment blocks
- renderer: Allocate cairo components
- renderer: Load fonts
* Configured DPI = 96x96
* Loaded font "Ubuntu Condensed:size=10" (name=Ubuntu Condensed, offset=2, file=/home/loma/.local/share/fonts/Ubuntu-C.ttf)
* Loaded font "icomoon\-feather:size=10" (name=icomoon-feather, offset=2, file=/home/loma/.local/share/fonts/icomoon-feather.ttf)
* Loaded font "xos4 Terminus:size=12" (name=DejaVu Sans, offset=1, file=/usr/share/fonts/TTF/DejaVuSans.ttf)
* Loaded font "HanaMinA:style=Regular:size=13" (name=HanaMinA, offset=3, file=/usr/share/fonts/TTF/HanaMinA.ttf)
* Bar window: 0x1800001
- bar: Reconfigure window
- bar: Set window WM_NAME
- bar: Set window _NET_WM_WINDOW_TYPE
- bar: Set window _NET_WM_STATE
- bar: Set window _NET_WM_DESKTOP
- bar: Set window _NET_WM_PID
- bar: Map window
- bar: Draw empty bar
- bar: Setup tray manager
* Disabling tray manager (reason: missing `tray-position`)
* module/title: Rebuilding cache
- bar: Force update
* Redrawing bar window
- renderer: flush(2 geom=289x20+816+0, falloff=0)
- bar: Received expose event
* module/title: Rebuilding cache
* Redrawing bar window

Screenshots
Look it is fun I can inject colors in polybar with the title of my window by having a window with this name "%{c}%{u#e60053 +u B#e60053}polybar [~/Code/repositories/polybar] - .../src/components/parser.cpp%{B- -u u-}" :smile:

Environment:

WM: i3 version 4.16.1 (2019-01-27)
Distro: ArchLinux 5.0.2-arch1-1-ARCH x86_64
Output of polybar -vvv:

polybar 3.3.0

Features: +alsa +curl +i3 -mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) -render -damage -sync +composite +xkb +xrm +xcursor

Build type: RelWithDebInfo
Compiler: /usr/bin/c++
Compiler flags: -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -Wall -Wextra -Werror -Wno-noexcept-type -pedantic -pedantic-errors
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

We already had a first approach for this in #1065 but the author is no longer around. If you want you can pick up that PR.

Hum, okay I will do that, but I disagree with some decisions in this PR.
I think the string should be sanitized by default on some user input, I made a list of all the tokens which should be sanitized by default in my opinion (see below). Some should be sanitized for security purposes while others should be sanitized to avoid bugs.
Maybe my list is over abused, but we all know that some users won't update they configuration, so I think it's better to disable the tag by default in some cases.
Bspwm module

label-{focused, occupied, urgent, empty} -> %name%
label-monitor -> %name% (not really sure about this one even if the monitor can be renamed)

Filesystem module

label-{unmounted, mounted} -> %mountpoint%
label-{mounted} -> %fsname%

i3 module

label-mode -> %mode%
label-{focused, unfocused, visible, urgent} -> %name%, (%output% same as for the label-monitor of bspwm)

Mpd module

label-song: %artist%, %album-artist%, %album%, %title%

Network

label-connected -> %essid%, (%ifname% not sure if this is necessary)

xwindow module

label -> %title%

xworkspace

label-monitor -> %name%
label-{active, occupied, urgent, empty} -> %name%

I think sanitizing specific tokens is a good idea.
Do you have an idea how the configuration could look like to enable/disable sanitization for specific tokens?
Also, if we do it at the token level, we could probably do the sanitization without the X tag by just replacing % with %%.

Hum, I propose something like:
[module/i3]
[…]
label-focused = %name (%monitor%)
; Disable tags escaping for the token listed in the list below
label-focused-whitelisted = %name% %monitor%

The load_label functions should be modified to load this list of tags and so the label would be responsible to escape the tags that are not in the whitelist. Though it is required to manually add the defaulted whitelisted tags to each label. However for the other modules, I don't think any modification would be needed.


Added ability to specify custom attributes in testing.conf.

I tested a slightly hacked merge of this and a few other PRs on cori and that mess worked, so while I haven't tested exactly this PR, I think thats close enough to merge this.


Raider Tropo Correction

This is brings in new raider developments into the GUNW for tropospheric delay computation. This is still a draft and is branched from #74 so that should be reviewed and merged first.
To run raider successfully, I had to update the environment to >=3.10. Using now both isce2 and isce3.

@dbekaert - confirming that we want to perform tropospheric correction on reference SLC only. Or am I missing something important for time-series analysis here.

So we compute the delay for reference and we compute one for secondary.
We compute the differential delay just as we do for the insar pairing.
we convert it from delays in m unites to 2-way path delay so you are missing a lambda/(4 pi) factor

@jhkennedy

You suggested offline that we could have 2 docker containers within a plugin is that suggested? Or should we have separate plugin. Your input would be greatly appreciated. There are some considerations:

Hitting the ERA5 API takes a significant amount of time and is the biggest bottleneck for processing
If we need to separate the plugins, there is some accounting to get insar pair metadata e.g. acq datetimes, extent, etc.
Packaging - will literally be a few lines in xarray - assuming there is no more layers beyond this, I am ok with the SLC portion being disjointed, but might be good to have it cleaner for burst portion

Integration test - I can do this - but just wanted to get your thoughts


Support DAPR_HTTP_ENDPOINT and/or DAPR_GRPC_ENDPOINT

Describe the proposal
Allow consistent way to configure endpoint for Dapr's sidecar, while enabling remote sidecar configuration.
Child of https://github.com/dapr/dapr/issues/6035
SDK should parse these in a consistent way, so operators have a consistent environment variable to set for Dapr apps. Right now, each SDK has their own way of configuring the endpoint for Dapr's sidecar.
If SDK allows gRPC communication from SDK to sidecar, then DAPR_GRPC_ENDPOINT must be supported.
If SDK allows HTTP communication from SDK to sidecar, then DAPR_HTTP_ENDPOINT must be supported.
DAPR_HTTP_ENDPOINT and DAPR_GRPC_ENDPOINT and not exclusive, meaning it is acceptable to support both, like in the Java SDK, if the SDK supports both protocols.
Also, companies might decide to offer different endpoints per protocol, so there is not an unique environment variable for both.

Copying from the parent issue"

FYI the JavaScript SDK already has this configuration embedded into the API. For instance, one could do
const daprHost = "<IP_ADDRESS>"; // Dapr Sidecar Host, could also be https://example.com
const daprPort = "3500"; // Dapr Sidecar Port of this Example Server
// HTTP Example
const client = new DaprClient({ daprHost, daprPort });

Having extra environment variables is not required, but it can be there and the API options can take precedence over env var.

These new environment variables can exist in JS-SDK, and daprHost/daprPort will take precedence.

/assign


Promote the provenance field in status

Changes
/kind feature
Fixes https://github.com/tektoncd/pipeline/issues/6309.
Prior, the provenance field in status was gated by the dedicated feature flag named enable-provenance-in-status.
This PR moves the field out of the feature flag.

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)

Release Notes
Promote `provenance` field to beta.

/assign

@chuangw6 this PR needs a rebase, added it to v0.48 milestone

/retest

/retest

/retest

/retest-required

/retest

cc @chitrangpatel PTAL. Thanks!

/retest

/retest

/retest

Thank you @jerop for the review. I've addressed all the comments. Please take a look.
Regarding the feature flag, we will keep the dedicated feature flag but enable it by default. please see the https://github.com/tektoncd/pipeline/issues/6309#issuecomment-1553458310 for details.

/lgtm


Please add LUIS Bot Multilingual Sample

On this page of the document. Could you add more details about how to make Bot with LUIS to support multi-language? I am seeing an error when trying to Start the Bot saying that missing {Botname}.es file.
Thank you.

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

ID: 51e3f9d9-a593-40a3-bd68-1235966cca23
Version Independent ID: 6984f534-80f3-6bd8-0392-b1cedf86393a
Content: Multiple language support - Bot Composer
Content Source: composer-docs/how-to-use-multiple-language.md
Service: bot-composer
GitHub Login: @zxyanliu
Microsoft Alias: kamrani

Hi @anniexudan thanks for reporting the issue.
Can you please share more details about your error message? If you follow the instructions covered in this doc, you should be able to make it work. :-)
FYI. We are releasing our docs for Composer 1.4 release. There will be a lot of updates coming.

Hi @zxyanliu , thanks for reaching out. I was following the doc you mentioned. Works fine. However, the challenge and issue is that when the Bot has LUIS intents it does not work. It would be nice to have a Blog about how to build a Multi-Lingual Bot in Composer with LUIS or QNA.

hi @zxyanliu , I found where the issue is. I tested in 'zh-cn' with LUIS intent Bot and it works fine. However, when it is 'zh', it does not work. Maybe in the documentation, we can add what languages are supported?

Yes, that's a nice catch. This is the updated content in the PR.
The content should go live soon.

Feel free to reach out to me on Teams via:
<EMAIL_ADDRESS>

Closing this issue for how as the content has been updated and published to live.


yii\widgets\MaskedInput removes custom classes and attributes applied via inputOptions

What steps will reproduce the problem?
Using ActiveForm Create a Masked Input and add a class my-control to it via inputOptions in field() and provide a style attribute with display:none like below
<?=
        $form->field($model, 'mobile_phone', [
            'inputOptions' => ['class' => 'my-control', 'useSessionFlash' => true,'style'=>'display:none'],
            'template' => '
{label}{input}
{error}',
        ])->widget(\yii\widgets\MaskedInput::className(), [
            'mask' => '999-999-9999'
        ])->label(null, ['class' => 'form-label']);
        ?>

What is the expected result?
The rendered input should have the class applied in the class attribute and should have a style attribute defined with display:none
<input id="leadform-mobile_phone" class="my-control" name="LeadForm[mobile_phone]" data-plugin-inputmask="inputmask_7b93eb48" type="text" style="display:none">

What do you get instead?
The renderd input shows the default class form-control on the input and there isnt any style attribute defined where as it should have the custom class my-control it renders the input like below
<input id="leadform-mobile_phone" class="form-control" name="LeadForm[mobile_phone]" data-plugin-inputmask="inputmask_7b93eb48" type="text">

If we remove he MaskedInput widget everything works as normal
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
5.6

Operating system
Ubuntu

inputOptions in ActiveField are not passed to widget. Pass the options to widget directly
<?=
        $form->field($model, 'mobile_phone', [
            'template' => '
{label}{input}
{error}',
        ])->widget(\yii\widgets\MaskedInput::className(), [
            'mask' => '999-999-9999',
            'options' => ['class' => 'my-control', 'useSessionFlash' => true,'style'=>'display:none'],
        ])->label(null, ['class' => 'form-label']);
        ?>

i love you @SilverFire

To achieve it, ActiveField should blindly try to pass its inputOptions to widget that is not a good idea as for me


support mapping actions together onto an alert

alert -> ceph info -> map(ceph-repair, concurrency=2)

I believe that Mjolnir will now allow this, but needs a working example to verify


Add FullObjects option, retrieving full objects on listing

Description
I think title says enough already ^^'
This also fixes some other small things, neatly in their own commits.
Release Note
Release note for CHANGELOG:

* NONE (updating unreleased code)

References

generic client introduced in #56
SYSENG-684

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Codecov Report

Merging #79 (5254a18) into main (84f0ef0) will increase coverage by 0.11%.
The diff coverage is 70.27%.

@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
+ Coverage   49.98%   50.10%   +0.11%
==========================================
  Files          88       88
  Lines        3969     4000      +31
==========================================
+ Hits         1984     2004      +20
- Misses       1674     1682       +8
- Partials      311      314       +3

Flag
Coverage Δ

integration
38.56% <16.21%> (-0.24%)
:arrow_down:

unittests
28.07% <70.27%> (+0.28%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/lbaas/loadbalancer/loadbalancer.go
50.79% <ø> (ø)

pkg/api/api_implementation.go
95.53% <64.70%> (-1.67%)
:arrow_down:

pkg/api/pagination.go
97.39% <68.75%> (-2.61%)
:arrow_down:

pkg/api/internal/options.go
100.00% <100.00%> (ø)

pkg/api/options.go
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 84f0ef0...5254a18. Read the comment docs.

There are some tests missing for Get operation made by PageInfo or ObjectChannel returning an error

lgtm


adoptopenjdk: add 17.0.0

Motivation for this change
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 via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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.

/cc @SuperSandro2000

Maybe we should add separate temurin (and maybe semeru) package.

Add temurin (and maybe semeru) package.
Add adoptopenjdk-hotspot-bin-17 and adoptopenjdk-jre-hotspot-bin-17 to pkgs/top-level/aliases.nix that throw an error.

If it is burden, I will do it in the next weekend.


Speed up fetchnvd

There is a possibility that delete-insert is slow.
Make a note of how to improve it.
(We need to check the performance before we fix it)

Opt.a. Make all tables have fetch_meta_id and delete one table at a time with one delete statement.
Opt.b. Delete from * where fetch_meta_id in (1, 2, 3, ...) https://gorm.io/docs/delete.html#Delete-with-primary-key

fixed https://github.com/kotakanbe/go-cve-dictionary/pull/214


Add English as a language option

This would be used as the base for translating to other languages.

VubbiScript is now available in English!


Aztec barcode support

Is there any chance that this will support Aztec barcode in the future ?

@munge83, did you find an alternative for Aztec?


fix: showing errors in a dialog.

Changes  done

In home_view.dart and settings.dart, all the error shows in a dialog box from the package trash.

This is not fixing #89  at all; you are just adding the support to display the component in a dialogue and nothing more.
Now all the view should use it


Module path should be "github.com/pouriyajamshidi/tcping", not "tcping"

Hi,
Thank you for this program.
Do you have plans to make this go gettable by changing the module path from tcping to github.com/pouriyajamshidi/tcping in the go.mod file?
 🕙 05:58:54 ❯ go get github.com/pouriyajamshidi/tcping
go: downloading github.com/pouriyajamshidi/tcping v1.2.0
go get<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: tcping
	        but was required as: github.com/pouriyajamshidi/tcping

Hi,
You  are very welcome!
Yes, I will apply the modification and make it go gettable soon!
Stay tuned.

Should be fine now. Could you please confirm?

Thank you. I confirm it is working with a go get now.

You are welcome and thanks for bringing this to my attention.


Update wmi-and-the-power-management-tab.md

Clarify Power Management settings for devices. @tedhudek could you please review or redirect.
@AndrewYangMSFT

Looks fine to me @Shyamal-V.  Do you want Andrew to have a look before I complete?

commit 2 looks good to me.

@tedhudek is good to go with the latest commit. thanks.

Thanks guys!


opensearch.index_df stops without error but does not complete full index

I'm trying to write a dataframe to AWS opens search but it seems to fail on specific rows without error.
Is there any way to debug what the issue is?
wr.opensearch.index_df(     client,     df=project_df,     index="project_index",     id_keys=["index_no"],     bulk_size=1000, #number of documents )
Result:
'''
Indexing:   0% (0/8540)|                                 |Elapsed Time: 0:00:00
Indexing:  11% (1000/8540)|###                           |Elapsed Time: 0:00:30
Indexing:  23% (2000/8540)|#######                       |Elapsed Time: 0:01:15
Indexing:  35% (3000/8540)|##########                    |Elapsed Time: 0:01:34
Indexing:  46% (4000/8540)|##############                |Elapsed Time: 0:01:48
{'success': 4000, 'errors': []}
'''
There should be  8k rows.

@chris-wynne could you please turn on logging to obtain more details on what goes wrong

@jaidisido thanks for that. It returns error 413, so the content is tool large.

Understood - you might want to tweak some of the default input arguments to the method if that helps (e.g. bulk_size, chunk_size...)

Thanks, a combination of setting bulk_size and stripping some of the content out worked.


Log in button not working with X11

Describe the bug
Log in does not work, button seems to do nothing
To Reproduce
Steps to reproduce the behavior:

Open Spot
Fill in your credentials
Click on "Log in"

Expected behavior
Log in box closes and user is logged in
Actual behavior
Nothing happens, log in dialogue still open
General information:

Distribution: Ubuntu 20.04 with Plasma 5.21.2
Installation method: installed from Flathub
Version: 0.1.9 stable
Device used: Desktop

Stack trace:
Console just prints: "home", no error, no warning

I just switched from X11 to Wayland and it worked instantly. Gnome X11 didn't work either.

@kocher the login form doesn't actually give any visual feedback when a login fails. Could it have been needed to use your email like (#26) or had your password wrong? Also if you still have the broken version would you mind trying pressing enter on the keyboard while the username or password field is selected to see if that works.

Probably unrelated to X11 vs Wayland indeed!
Yeah login sucks right now, I am waiting for the next major version of librespot which will make auth failures much easier to manage. But maybe something can be done in the meantime indeed

Password is correct, since I copy it from a password manager - and it works from my Wayland session. Neither username nor email work. Pressing enter doesn't work, neither does navigating to the button with keyboard and pressing enter.
I had errors with password before in the console before so I know how that looks, but now console is dead silent.
After restarting the computer I am back on X11. App requires log  in again even though I haven't logged out. I can see my playlist in the app in the background but the log in form doesn't work.

Really weird, don't know what could be happening :/ I don't have Plasma installed atm, might be an issue related to focusing the login dialog under X11? Could you post a screenshot of the login dialog?
I don't know which keyring Ubuntu is using (seahorse aka GNOME Keyring?), but you could try wiping Spot credentials from here.

I have the same issue. I am using Pop_OS 20.10, which is basically Ubuntu 20.10 with a few extensions and modified APT repositories.
I tried it on X11 and Wayland,  with email and username, pressing enter and pressing the button. I also tried disabling all gnome extensions and nothing.
When I run spot from the terminal I see the following:
$ flatpak run dev.alextren.Spot

Gtk-Message: 11:12:48.116: Failed to load module "appmenu-gtk-module"
Gtk-Message: 11:12:48.318: Failed to load module "canberra-gtk-module"
Gtk-Message: 11:12:48.322: Failed to load module "canberra-gtk-module"

There's no more information when trying to login. I'm not sure if there's a way to get more in-depth logs?

Hi! Thanks for the detailed write-up! I fear that if the form isn't even submitting there is not much happening and therefore not much to log :/
As for the gtk modules, canberra is no big deal, appmenu not so sure, I'll have to dig into it.
Would you mind sharing screenshots of the login dialog? My current understanding is that it might not have proper focus in some DEs, if that's the case there should be subtle visual cues of this.

Here they are. I'm not sure what focus means in the context you are using it, but the green border means that the windows is being focused, as it would if I alt-tabbed through them. I have zero knowledge about this sort of thing, but if there's any tests or logs that you think might help let me know.
Cheers

That was absolutely what I meant, guess that's a wrong hypothesis then :D Thanks for the screenshots
I'm out of ideas for now, I'll probably upload a build with more logs when I have the time!

Hey, so a little after I send that message I broke my system by interrupting a distro update. I restored my system with timeshift, re-ran my updates and now it works. So....yea, that's a thing
I can't really help you because, as far as I know, my system should be basically the same as before I restored it. I still get errors about the missing modules so that wasn't it. Good luck and thank you

Well I'm certainly glad it worked for you (at what cost though! :smile: ), I am not sure how to approach the issue for now :/


feat(master): trigger builds for the master branch

based on:

https://buildkite.com/docs/apis/rest-api/builds#create-a-build
https://developer.github.com/v3/activity/events/types/#pushevent

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


Detecting collisions between rigid bodies

Set Solver/Contact Monitor
Player.dead signal connect to game_over() func
Test cases:

Player collision to rock

Player lives count minus by 1
invulnerable during 2 seconds

Game over when player collision to rock three times.

https://github.com/user-attachments/assets/5568b3db-a6e7-4d7d-a4aa-cf15dba87d6c


feature: throttle sandbox authentication requests

A clear and concise description of what you want the system to do.

Implement controls against:

Password spray attack (track by source IP address)
Password brute-force attack (track by user identity)

Upon the detection of the above attack, the offending IP/identity should be blocked for a period of time, e.g. 15 minutes.

Record username and source IP address.

Block source IP address that failed auth 10+ times within 2 minute period across multiple usernames
Block username who failed auth 5+ times within 5 minute period having same source IP address

Closed in favor of https://github.com/greenpau/caddy-security/issues/19


[codelab]mdc 103 failing to build (even complete)

Running npm install, npm start produces an error when following the tutorial.
I also ran it against the 'complete' version and the error is there too.

    ^
          Can't find stylesheet to import.
      ╷
    1 │ @forward "@material/animation" as mdc-animation-*;
      │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      ╵
<EMAIL_ADDRESS>1:1  @import
      stdin 5:9                                              root stylesheet
          in<EMAIL_ADDRESS>(line 1, column 1)
     @ multi (webpack)-dev-server/client?http://localhost:8080 ./home.scss

Hey,
I got the same problem, but I have fixed the issue by following this sugguestion on issue #139


Add JWK support to importKey.

You are welcome to take anything you like from: https://github.com/transmute-industries/jose-hpke/blob/main/src/hpke.ts
You might also want to review:

https://github.com/panva/jose/blob/main/src/key/generate_key_pair.ts
https://github.com/panva/jose/blob/main/src/runtime/node/key_to_jwk.ts

Thanks for your infromation and using this lib!
I'm very interested in jose-hpke and its use cases!
Additionally, if you propose JOSE-HPKE as a standard, I will fully support it! Although my support might be a bother for you...

I'm not bothered by strong opinions.
Never apologize for having them :)


Add OpenAI API compatible assistant

PoC - do not merge.
Details

Download https://huggingface.co/Mozilla/Mistral-7B-Instruct-v0.2-llamafile/blob/main/mistral-7b-instruct-v0.2.Q5_K_M.llamafile (or host your own model locally)
export BASE_URL=http://localhost:8080 (or wherever your model is available locally)
Add assistants = [  "ragna.assistants.OpenAIApiCompatible", ] to ragna.toml

Superseded by #425


Fix condition with readme container

This is fix for bug I found  with PR https://github.com/NuGet/NuGetGallery/pull/8406
Expect legacy readme work when uploading package without embedded readme
Without changes: legacy readme container invisable at upload package page if we upload package without embedded readme

After changes:   legacy readme container display when package has legacy readme

No problem with embedded readme
Addresses https://github.com/NuGet/Engineering/issues/3507

Could you provide more explanations about how it works and why it was broken? I don't very understand this fix, and how it deals with the "Submit" button. Thanks!

Could you please provide more explanations about how it works and why it was broken? I don't very understand this fix, and how it deals with the "Submit" button. Thanks!

@zhhyu, please let me know if my explanationis not clear.
In PR below, In order to preview image when we upload package with embedded readme, ReadmeFileContents was changed to type of RenderedMarkdownResult instead of string.
https://github.com/NuGet/NuGetGallery/pull/8406/files#diff-289b3f52256d09316200396f1a20bfd6526bcb73b9bbc9e6a1501e3dae8454c8R258
               if (model.ReadmeFileContents.Content) {
                    $('#import-readme-container').addClass('hidden');
                } else {
                    $('#import-readme-container').removeClass('hidden');

Here, ReadmeFileContents are NULL, it will never go to if statement, which readme container disapear.
https://github.com/NuGet/NuGetGallery/pull/8406/files#diff-af1ecc85bc3f873d40ff9cc74d6335b402a60ed36fe472cf693d2ad8f47106c0R114

I am still curious about why the submit button is associated here?

@zhhyu , there is exception threw for ReadmeFileContents is null, initialization for upload page will be hold, which means submit session won't be initailzed becuase of 404, please see screenshoot below


getAll missing from API doc

I was going to edit however your package.json scripts include a docs build so was unsure if this is automated or not.

Yes. It's kind of automatic, you can look at update-docs.sh
You can edit docs, but only I can publish it


Update all collapse triggers with .collapsed class and aria-expanded

Fixes #15741.
Closes #15751 by merging it.
CC: @hnrch02 for final re-review

LGTM otherwise.


Upgrade Go version to v1.20.5

This commit upgrades Go to the latest released version.

Thanks!!

@caarlos0 Can you tag a new release?

Thanks!. on the way


compstate[insert]="automenu" missing from click 8

In click 7.x the ZSH script used compstate[insert]="automenu" (https://github.com/pallets/click/blob/7.x/src/click/_bashcomplete.py#L73) and in click 8.x this is no longer supported (https://github.com/pallets/click/blob/main/src/click/shell_completion.py#L125).
Without this, you can no longer to in comparisons on autocompletes.

Without this, you can no longer to in comparisons on autocompletes.

I don't see how this follows from the first statement.
I am at my limit of familiarity with shell completion, so I won't personally be doing more work on the scripts. If you'd like to submit a PR, and can guarantee that everything else continues working, then I'll be happy to review it.

@davidism I am also on the edge ;)
For my case I am getting around it with:
_SOURCE_ZSH = """\
#compdef %(prog_name)s

%(complete_func)s() {
    local -a completions
    local -a completions_with_descriptions
    local -a response
    (( ! $+commands[%(prog_name)s] )) && return 1

    response=("${(@f)$(env COMP_WORDS="${words[*]}" COMP_CWORD=$((CURRENT-1)) \
%(complete_var)s=zsh_complete %(prog_name)s)}")

    for type key descr in ${response}; do
        if [[ "$type" == "plain" ]]; then
            if [[ "$descr" == "_" ]]; then
                completions+=("$key")
            else
                completions_with_descriptions+=("$key":"$descr")
            fi
        elif [[ "$type" == "dir" ]]; then
            _path_files -/
        elif [[ "$type" == "file" ]]; then
            _path_files -f
        fi
    done

    if [ -n "$completions_with_descriptions" ]; then
        _describe -V unsorted completions_with_descriptions -U
    fi

    if [ -n "$completions" ]; then
        compadd -U -V unsorted -a completions
    fi
    compstate[insert]="automenu"

}

compdef %(complete_func)s %(prog_name)s;
"""

@add_completion_class
class FCZshComplete(ZshComplete):
    source_template = _SOURCE_ZSH

I don't know if this will work for everyone, but it seemed to work prior to 8.x

It's not clear from your description why this is required.

I can't reproduce an issue based on what you've provided. I tried the following, and was able to see a menu of completions.
# example.py
import click

@click.command()
@click.argument("x", type=click.Choice(["apple", "arugula", "bagel"]))
def cli(x):
    click.echo(x)

# setup.py
from setuptools import setup

setup(
    name="example",
    version="0.1.0",
    install_requires=["click"],
    py_modules=["example"],
    entry_points={"console_scripts": ["example = example:cli"]
)

$ python3 -m venv example-venv
$ . ./example-venv/bin/activate
$ pip install .
$ eval "$(_EXAMPLE_COMPLETE=zsh_source example)"
$ example apple
apple
$ example a<tab>
apple  arugula

from click.shell_completion import CompletionItem, add_completion_class, ZshComplete

# _SOURCE_ZSH = """\
# #compdef %(prog_name)s
#
# %(complete_func)s() {
#     local -a completions
#     local -a completions_with_descriptions
#     local -a response
#     (( ! $+commands[%(prog_name)s] )) && return 1
#
#     response=("${(@f)$(env COMP_WORDS="${words[*]}" COMP_CWORD=$((CURRENT-1)) \
# %(complete_var)s=zsh_complete %(prog_name)s)}")
#
#     for type key descr in ${response}; do
#         if [[ "$type" == "plain" ]]; then
#             if [[ "$descr" == "_" ]]; then
#                 completions+=("$key")
#             else
#                 completions_with_descriptions+=("$key":"$descr")
#             fi
#         elif [[ "$type" == "dir" ]]; then
#             _path_files -/
#         elif [[ "$type" == "file" ]]; then
#             _path_files -f
#         fi
#     done
#
#     if [ -n "$completions_with_descriptions" ]; then
#         _describe -V unsorted completions_with_descriptions -U
#     fi
#
#     if [ -n "$completions" ]; then
#         compadd -U -V unsorted -a completions
#     fi
#     compstate[insert]="automenu"
#
# }
#
# compdef %(complete_func)s %(prog_name)s;
# """
#
#
# @add_completion_class
# class FCZshComplete(ZshComplete):
#     source_template = _SOURCE_ZSH

def auto_complete(ctx, args, incomplete):
    items = [CompletionItem(value="apple", help="Big Apples"),
             CompletionItem(value="green-apple", help='Big green apple'),
             CompletionItem(value="red-apple", help='Big red apple'),
             CompletionItem(value='pear', help="Just a pear")]

    if incomplete:
        results = []
        for item in items:
            if incomplete in item.value or incomplete in item.help:
                results.append(item)

        return results
    return items

@click.command()
@click.argument("x", shell_complete=auto_complete)
def cli(x):
    click.echo(x)

example app<tab>

This will erase the subcommand. If you uncomment the FCZshComplete and try again you will get a menu.


FGBS-001, wrong status in MQTT Client start-up

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Z-Wave JS UI version
8.6.0.f033ee8
ZwaveJS version
10.3.1
Describe the bug
My MQTT client (Domoticz) is not showing the right status for my My FGBS001, at start-up.
probably the status is correct in ZwaveJS UI
My NAS is rebooted every day and I run ZwaveJS UI, Mosquitto and Domoticz on separate containers that start automatically.
When just restarting only the Domoticz container, the status remains "open" and is (most probably) not consistent with the status in ZwaveJs-UI.
To Reproduce
Start the host, then 3 containers (ZwaveJS UI, Mosquitto and Domoticz)
in Domoticz, the device corresponding to the FGBS 001 is shown as open and a couple of seconds/minutes after withtout doing anything , to close (correct status)
If restarting only the DZ container, the DZ device will remain "open" and never get back to the correct status.
My workaround is to refresh the Zwave Node value in ZwaveJS,
Expected behavior
Having directly the correct status at start-up
Additional context
No response

Domoticz uses mqtt, if the status is correct on zui but not on domotics it means mqtt isn't connected or broker is down

but all others devices are correct, on DZ and ZUI sides....

probably the status is correct in ZwaveJS UI

COuld you verify this?

the device status is CORRECT in ZUI and stay "wrong in DZ"...

And if you connect to broker using mqtt explorer does the status is correct?

Hello.
First time this morning that the problematic device did change status at start-up. I don't know why, I did not change anything....
Could it helps that I have just noticed that my lights have a status "off" at DZ container start-up, even if the light was already ON ?

This is hard to debug without testing on your side. You have to know at least how MQTT works, when a value changes on zwave I get an event on zui and I will publish a retained message to the valueId topic with the value. This means that when Domioticz connects to the broker and subscribes to that valueId it will always receive latest value sent to that topic, if you see the correct value on zui it means that it also sent the value to MQTT broker so the problem could be that for some reason Zui or DZ is not connected to broker or DM is not picking up the value on connection for some reason.


enable usage of upstream images

Added changes to enable usage of upstream images for velero and its plugins
documented changes in readme

Issue fix https://github.com/konveyor/oadp-operator/issues/15


Image stream detection not active with files in local directory

By default we try to detect image streams (so a video). Looks like it does not work when it is in local directory, we need to check that.


Periodic boundary conditions

We need periodic boundary conditions for

Idealised setups
For regional setups with a zonal periodicity (e.g. the southern ocean)
For global "tri-polar" setup with a fold at the north pole

The main tasks are to

[ ] Implement periodicity in the dynamics and advection code
[ ] Make sure information about which borders are periodic flows between the restart file and the dynamics

We'll do this as part of the MPI communication.


feat: allow prefixed env vars

For label_predicates, description_predicates etc. this commit allows you
to set the values as prefixed values. for example
# .env
LABEL_PREDICATES='["schema:headline", "dcterms:title"]'

this should work for any of the default bound prefixes in rdflib

[!WARNING]
i had to change the test cases becuase this now considers "1" to be an invalid prefix
this may or may not be a breaking change actually, so please consider carefully.

I'd prefer not to do this as it relies on us knowing what namespace the prefix is mapped to, in common cases this may be clear but even for say schema: is that referring to the http or https version of schema, and for anything less common where would the mapping be kept, and how would you guarantee the prefix:namespace mappings are 1:1.
I realise perhaps counter intuitively the prez settings use the RDFLib Namespaces so can be written in the prefixed form


tmux.nvim copy_sync related crash

Hi there,
I am using the newest version of this plugin, with Neovim on nightly, and for the past few weeks, I have been experiencing a crash of this plugin while inside Neovim. It manifests after around 10-15 minutes of being open.
     4 stack traceback:
     5 >       [C]: in function 'assert'
     6 >       ...ocal/share/nvim/lazy/tmux.nvim/lua/tmux/wrapper/tmux.lua:27: in function 'execute'
     7 >       ...ocal/share/nvim/lazy/tmux.nvim/lua/tmux/wrapper/tmux.lua:70: in function 'get_buffer_names'
     8 >       ...veres/.local/share/nvim/lazy/tmux.nvim/lua/tmux/copy.lua:29: in function <...veres/.local/share/nvim/lazy/tmux.nvim/lua/tmux/copy.lua:20>
     9 07:36:18 PM msg_show ...ocal/share/nvim/lazy/tmux.nvim/lua/tmux/wrapper/tmux.lua:27: unable to execute: [ tmux -S /tmp/tmux-1000/default  list-buffers -F "#{buffer_name}" ]

UPDATE:
Tmux version: 3.3a-7.20230918gitb202a2f.fc39
Evidently, I am using Fedora Workstation 39.
Additionally, I am getting an error: Failed to start process: Too many open files for xclip or for e.g., zsh when wanting to open a terminal.

Was there a tmux update as well?
Can you please try to execute the listed call after the plugin crashed? I suspect, this behavior is caused by tmux because the assert checks if the call succeded.

Heho,
sry for the late response. It seems the call against tmux fails/times out. Because the error occurs in the assert. The plugin itself is not crashing/bugging out.
Since im not actively dev the plugin anymore im still maintaining it. I would be glad if you can debug and pr a fix.
Kind regards
Alexander

Hi @aserowy, no worries, in the meantime I found out, that some change in Fedora 39 probably reduced the user limit of open files, so the crash has nothing to do with your plugin! Sorry for the confusion and thanks for the plugin
SOLUTION:
Globally update the limit of open files in sudoedit /etc/security/limits.conf by appending this line:
* soft nofile 8192

Which was originally 1024.

Nice that you found a solution! It was a nice experience to write even if im not using it anymore. Neovim is <3 :D


MO1: App rating and feed back

This is part of MO1, please refer to MO1: https://trello.com/c/1SUC3KXe
Functional scope:

Ability to rate the app/ give feedback. The user will be prompted every 5 logins and asked if they like the app or want to give a feedback. If they like the app, they are asked to rate the app on the app store, this opens an in-app feature supported by the OS. If the user wants to send a feedback, email opens up to<EMAIL_ADDRESS>
This issue covers the end-to-end implementation including screens, navigation based on UX flow, actions, reducers, Sagas as well as update to storage, wallet image and upgrade script.

Screens names with xd links to follow

screens
https://xd.adobe.com/view/6e89fcde-5787-4d1a-a1ba-c284e770fdad-3bda/
More
Settings
Settings – Write a Review
Success Screen – Review Submitted

The functional scope described here seems out of sync with the designs.
The current designs show the user navigating to a dedicated form within Wallet Settings where we manually handle the submission of a "title" and "review" text.
The functional scope described above entails rendering an OS-specific alert based upon the login frequency, and then either opening the OS-specific app store where they would leave their rating, or opening the user's preferred email client for having them send feedback to Bithyve.
@AliMeer @antuz123 @Arpanjainson


Add getLines method.

Adds a function getLines() which first clones the original string, replaces \r\n in the cloned string with \n due to platform differences, and then splits the cloned string with splitToString(), appending each string to an ArrayList and then returning the ArrayList as an owned slice.
I'm pretty new to Zig (just learning it now) but saw this was missing and implemented it as I needed it for something else so feedback appreciated :)

I like the idea. A change I would like would be to rename it to lines() instead of getLines(). It would be more consistent with the other function names in the library.

Done! Was also thinking of making a more general method to split it into slice of slices or slice of strings on the delimiter as that might be handy too, not sure what names fit though.

You should definitely try your idea and make a PR. Thank you for the contribution!


fix https://github.com/mapbox/node-sqlite3/issues/890

<EMAIL_ADDRESS>has an issue (https://github.com/mapbox/node-sqlite3/issues/890) so i updated dependency to fixed sqlite3 version.

Hi
Many thanks ofr your pull request
I will update it to regenerate the shrinkwrap file and push a patch version
Thanks again !
Regards


Move the adapter module to the detail namespace

No elements of the adapter module appears in the public API so it should be moved in the detail namespace and directories. Some parts of it belong into redis/detail while other in redis/resp3/detail.

Hi, sorry for being so late. Before the inclusion in Boost connection::async_exec used to take an adapter instead of a response parameter
# Present
conn.async_exec(request, response, token);

# Before Boost
conn.async_exec(request, adapt(response), token);

In the Boost review however somebody raised the question about why I have to call adapt()? and I decided to hide its call here.  This resulted in a simpler API that is however harder to type-erase as you correctly noted (type-erasure occurrs here btw).
There is however a simple fix for that: Move the adapter out of the inner layer to expose it to the user as originaly thought. That would result in another async_exec overload, namely
# Current overload
template <class Response, class CompletionToken>
auto async_exec(request const& req, Response& resp, CompletionToken&& token);

# Overload that takes the currently internal type-erased adapter.
template <class CompletionToken>
auto async_exec(request const& req, adapter_type adapter, CompletionToken&& token);

By using asio::any_completion_handler the implementation of this overload can be completely moved to the .ipp file as I did for async_run and you wouldn't need any more work on your side.
For family reasons however I don't have much time to dedicate to this feature in the upcomming weeks (second baby comes in a week) but if you are lucky some Asio expert such as @anarthal might get interest in implementing it. If you want to folow this path however, please open a new ticket with reference to this one, thanks.

If we're sure removing adapt is not considered a breaking change (because it was never part of the public API in the first place) we should do that ASAP before people end up using it.

By using asio::any_completion_handler the implementation of this overload can be completely moved to the .ipp file as I did for async_run and you wouldn't need any more work on your side.

This is true. @GoldRobot is still going to have to pimpl away the connection, as including <asio/ssl/...> takes the order of seconds on itself. He may be able to do the same without the use of any_completion_handler, as long as we provide a way to type-erase the adapter type. @GoldRobot , which completion style are you using? (callbacks, yield_context, C++20 coroutines...)
@GoldRobot I found myself in a similar need with Redis, and came up with this scheme. Essentially, you build a service class (using pimpl or virtuals) where you expose a repository-like API. The style is a little old-fashioned, but gets the job done, achieves a pretty good level of separation of concerns, and is very well suited to run unit tests using mocks.
Another option (not ideal, but also gets the job done) is to use explicit template instantiations:
// your pimpl connection.hpp
class pimpled_connection {
    unique_ptr<impl_t> impl_;
public:
    template <class Response>
    asio::awaitable exec(const redis::request& req, Response& res); // not defined here
};

// in a .cpp file that uses pimpl connection
asio::awaitable handle_request(pimpled_connection& conn) {
    redis::request req; // compose the request
    std::vector<std::string> res; // response to use
    co_await conn.exec(req, res);
}

// in pimpl connection.cpp,
// define your template
template <class Response>
asio::awaitable pimpled_connection::exec(const redis::request& req, Response& res) { /* ... */ }

// explicitly instantiate your template
template asio::awaitable pimpled_connection::exec<std::vector<std::string>>(const redis::request&, std::vector<std::string>);

Even if a generic type-eraser is the ideal, I'd avoid it for now until we understand whether boost_redis_adapt is going to be hidden in the next release - otherwise your code will stop working when you upgrade Boost.
Moving forward with the Redis issue. What you're calling adapter_type is probably better named any_response_ref, as from the user's point of view, you're just doing type-erasing.
I need to understand a couple of things before doing anything:
a. Reading adapter code (like this), it looks like adapters have data members keep intermediate parsing state. This suggests me that you've got support for stream parsing, where the message is received in chunks, so the parser can't rely on having the entire message from the beginning.
b. However, looking at how it is used, it looks like no stream parsing in happening.
c. Is stream parsing really happening or not? If it's not, we can refactor parsing so that adapters don't keep any state, which makes them much easier to type erase (just a couple of pointers).
If you're happy with this change, I can implement it and PR (might be big depending on what you answer).
Also, if you're going to be unavailable to maintain Boost.Redis, I can probably dedicate some time to doing it moving forward.

as long as we provide a way to type-erase the adapter type.

The internal connection type (connection_base) uses a
std::function<void(node_type const&, system::error_code&)>;

to type-erase it.

Moving forward with the Redis issue. What you're calling adapter_type is probably better named any_response_ref, as from the user's point of view, you're just doing type-erasing.

The adapter is lightweight handle to the response and knows how to fill it as chunks of RESP3 arrive on the socket.

I need to understand a couple of things before doing anything:

a. Reading adapter code (like this), it looks like adapters have
data members keep intermediate parsing state. This suggests me that
you've got support for stream parsing, where the message is received
in chunks, so the parser can't rely on having the entire message
from the beginning.

Correct, Boost.Redis implements a sans-IO stream parser. It could be even moved into a separate library to complement Boost.Serialization and Boost.Json.

b. However, looking at how it is used, it looks like no stream
parsing in happening.

The connection class will pass incomming data to the parser until it is done with the message and after that it is reset.

c. Is stream parsing really happening or not? If it's not, we can
refactor parsing so that adapters don't keep any state, which makes
them much easier to type erase (just a couple of pointers).

Yes, we need it. There is no way of knowing the size of e.g. agreegate message types in advance.

If you're happy with this change, I can implement it and PR (might
be big depending on what you answer).

Yes, that is fine. I expect something along the lines of my previous message, where another async_exec is provided, which takes an adapter as parameter.

Also, if you're going to be unavailable to maintain Boost.Redis, I
can probably dedicate some time to doing it moving forward.

That would be nice, thanks. I am not going to say I will be completely unavailable, but only that I won't engage in larger features. Help is much appreciated.

A simpler overview about how the parser works can be seem here (which is only suitable for whole messages) and it usage is shown here.

Looks like I completely misread the code involving stream parsing, thanks @GoldRobot for your detailed analysis.
Looks like we're all agreeing on exposing a type-erased response adapter as the best solution. I'll try to work that out and PR you.


Advisory Database: Dart language support

Summary
The Dart programming language allows developers to write both web and mobile apps. It uses a package manager called pub. This feature will add support for publishing security advisories against the pub package manager.
Intended Outcome
This is a necessary step to help protect the software supply chain for developers that use the Dart programming language.
How will it work?
The Advisory Database has a list of known ecosystems. We will add Dart pub packages to that list.

this has shipped to dotcom! 🚢<EMAIL_ADDRESS>closing out this issue

Reopening this issue as so far Dart is only supported on Dependabot version updates, but not in the Advisory Database. Release of support to the Advisory Database is coming soon.

This has shipped to support the Advisory Database: https://github.blog/changelog/2022-10-06-githubs-supply-chain-features-now-support-dart/.


Add npm run build to travis

Add npm run build to travis and fix failing linter in pf4 TagView

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Setting theme doesn't change anything

So I tried setting the style, however after some fiddling I found out that I need to set handler for on_column_resize for the style to even get applied. But it gets applied on the divider only, and it still gets applied from the global theme, and not from the theme I set. Is there any workaround without breaking my global theme? I haven't found any.

Indeed, same issue here.


[Feature]: Implement routes for /v1/files

Contact Details(optional)
No response
What feature are you requesting?
Please check OpenAI reference of files API and the link for files object

[ ] #72
[ ] #73
[ ] #74
[ ] #75
[ ] #76

Add the path, for example ./files, into Makefile follow the way of adding env vars on the top of makefile, and volume bind in docker-compose.yaml and docker-compose-dev.yaml.

Hey, would like to help with this if you could assign

Hi @fduffoo welcome to contribute.


Problems with hotspots

Running:
python ./protein_generator/inference.py \
    --num_designs 1 \
    --out result/prot_gen_fold \
    --pdb 5KQVpart.pdb \
    --contigs 4-153,0 231-231,0 \
    --hotspots E88,E96,E64 \
    --T 25 --save_best_plddt \
    --dssp_pdb /content/7DET_B.pdb

raises:
Traceback (most recent call last):
  File "/content/./protein_generator/inference.py", line 238, in <module>
    main()
  File "/content/./protein_generator/inference.py", line 235, in main
    S.generate_sample()
  File "/content/protein_generator/utils/sampler.py", line 814, in generate_sample
    self.setup()
  File "/content/protein_generator/utils/sampler.py", line 568, in setup
    self.make_hotspot_features()
  File "/content/protein_generator/utils/sampler.py", line 159, in make_hotspot_features
    for n,x in enumerate(self.features['mappings']['complex_con_ref_pdb_idx']):
KeyError: 'complex_con_ref_pdb_idx'

My fault, should have used:
--contigs A4-153,0 231-231


setup

please check in the dev branch for the setup. I tried submitting that link but it is not being accepted in the request link.
Here is a link to the setup dev branch.
Click to Setup

STATUS: INVALID!!!  :x: :x: :x:
Hi @jerryowusu,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code, please make sure your implement the following changes before you resubmit for review.
Required Changes ♻
Check the comments below.

[ ] Please kindly raise a pull request, from a feature branch to the dev branch before resubmitting for review since you are to follow [Gitflow](https://github.com/microverseinc/curriculum-transversal-skills/blob/main/git-github/articles/gitflow.md) for this particular project. The link is not being accepted because you have not raised a PR. Kindly raise a pull request and submit the pull request link.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.

Thanks


Notification - allow executing services other than notifier

Checklist

[x] I checked for similar existing requests (both open and closed) before posting.
[x] My request is generic, other users may benefit from it too.

Proposal
Right now, the notification settings only allow selecting a notifier service, even when i go to YAML mode and manually change the service to other (for example zwavejs.set_value) im getting an error and cannot save the notification.
It would be nice to be able to use different services as notification - not only performing zwave actions, but it would also be possible to make play a sound on a speaker or use TTS.

Could you share your YAML code?
This sounds more like a bug than a feature request.
The notification editor should natively support notify and tts services (although custom settings have to be done via YAML), others are configurable via YAML mode.
The editor should not block you from creating a custom action.


Extend docs on git+ssh auth

What does this do / why do we need it?
You can configure dep to auth via ssh by using the manifest file instead of overriding global git config.  Just adding an alternative way to do things

@googlebot I signed it!

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


chore(main): release 0.9.0

:robot: I have created a release beep boop
0.9.0 (2022-10-21)
Features

add instructions for installing hybrid (16cf357)
better yaml splitting for sealed secrets in hybrid demo (9e17cc3)
configurable machine type for hybrid cluster (f02db36)
hybrid example with reduced infra costs (cedd1ff)
hybrid-basic: deploy sealed secrets as helm chart (b88ab1c)
move cert manager to helm chart (2aff39d)
replace csplit with awk for better multi-platform support (a9b636b)

This PR was generated with Release Please. See documentation.

/gcbrun

:robot: Release is at https://github.com/apigee/terraform-modules/releases/tag/v0.9.0 :sunflower:


Add Potsdam

https://github.com/woog-life/potsdam-scraper

Should be called "Potsdamer Havel", right?


updating documentation

Issue #, if available: N/A
Description of changes:

Updating documentation to align with these changes: https://github.com/awslabs/amazon-ec2-instance-qualifier/pull/10 ; therefore, this PR should be reviewed+merged after

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

should add a defaults table similar to AEMM


Fix react native app

-add missing peer deps to app and web
-add missing @babel/core peer dep
-update ts and concurently
-fix metro bundler
-fix styled components
-fix native package
-xcode build ok
-improve gitignores
-udpate rn to 0.59.3, rename app to com.foton.app, add .DS_Store to gitignore
-rm minimum flow version to allow more flexibility
-add missing xcode script in @foton/app

thanks to @arthurdandrea for fixing nohoist globs


Task: Discuss database schema change procedure

With the way we embed the current database schema in our cif.form_change table, making changes to the database schema that are not addition of columns becomes tricky.
Since creating a new revision copies the old form_change records from the previous revision, we risk seeding a new revision with an outdated database schema, and this will result in a failure to save that revision.
Options are:
-updating all the form_change records according to the new schema, through a migration - but then we'd be messing with our app's audit trail
-versioning our database schema and write specific migration functions between versions, so that when a new project revision is created, we can update the duplicated form_change records to the new schema (and update the revisions not yet committed) - this sounds cumbersome and high-maintenance
-disallow database schema changes - not an option
-rewrite the create_project_revision mutation to not rely on the previous committed revision, but on the actual database schema - but then what do we do with the revisions and form_changes that are not committed?
-...any more ideas?

@pbastia keeping existing fields and changing titles, was another option?! if we only want to change the title!

In that case yes - although keeping the database schema as close to the business domain is preferable. But cases where the schema actually changes could me more hairy, for example moving a column from a table to another

@mikevespi @dleard @matthieu-foucault @Sepehr-Sobhani @BCerki @gurjmatharu When is a good time for a discussion about this topic?

I'm away thursday Friday this week, anything other than that works for me

@mikevespi @dleard @matthieu-foucault @Sepehr-Sobhani @BCerki @gurjmatharu When is a good time for a discussion about this topic?

Flexible - whenever works for everyone.

Except for this Friday, my flex day, I'm flexible.

Maybe tomorrow after backlog refinement?

I'd go for

save json schema in the db
json schema records can specify a parent json schema, and can provide a function to migrate data from the previous to the new json schema.
breaking json schema changes should be fairly rare. The example of moving a column from one table to another sounds unlikely, assuming that we have an adaptor pattern to remove the strict coupling between forms and database tables

Document:

How things work right now (diagram)

Limitations

Cost in terms of development and maintenance

Potential solutions (look at #373 and #867)

Estimate the impact and the time (including migrating the production data)


Could you provide a Docker image?

Could you provide a Docker image?

We will provide pip install support in these two days and then the dockerfile

Could you provide a Docker image?

We have provided an image on DockerHub, which can be pulled using docker pull approachingai/ktransformers:0.1.1. Using can be found on the Docker.md. We look forward to your feedback.