Runmetadata and graph definitions are not mached

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
source
TensorFlow version (use command below):
1.6
Python version:
Python 2.7.12
Bazel version (if compiling from source):
0.13.0
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
9.0, 7.0
GPU model and memory:
TitanXP, 12G

You can collect some of this information using our environment capture script
You can also obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the current behavior
I was trying to match information between graph definition and timeline but they didn't match.
For example, below is a node in graph definition.
node {     name: "AutoParallel-Replica-0-Accum-Apply/model/gradients/model/lm/sampled_softmax_loss/embedding_lookup/Gather_3_grad/Reshape"     op: "SparseAccumulatorApplyGradient"     input: "AutoParallel-Accum/model/gradients/model/lm/sampled_softmax_loss/embedding_lookup/Gather_3_grad/Reshape"     input: "Const_35"     input: "AutoParallel-Replica-0/model/gradients/model/lm/sampled_softmax_loss/embedding_lookup/Gather_3_grad/Reshape_1"     input: "AutoParallel-Replica-0/model/gradients/model/lm/sampled_softmax_loss/embedding_lookup/Gather_3_grad/Reshape"     input: "ToInt64_107"     device: "/job:worker/task:0/device:CPU:0"     attr {       key: "_class"       value {         list {           s: "loc:@AutoParallel-Accum/model/gradients/model/lm/sampled_softmax_loss/embedding_lookup/Gather_3_grad/Reshape"         }       }     }     attr {       key: "dtype"       value {         type: DT_FLOAT       }     }     attr {       key: "has_known_shape"       value {         b: true       }     }   }
However, Runmetadata contains below node stat for the node. The inputs in timeline_label are different with information in the graph node. I want to know the policy or rules to match two different information.
node_stats {       node_name: "AutoParallel-Replica-0-Accum-Apply/model/gradients/model/lm/sampled_softmax_loss/embedding_lookup/Gather_3_grad/Reshape"       all_start_micros:<PHONE_NUMBER>138092       op_start_rel_micros: 10       op_end_rel_micros: 12405       all_end_rel_micros: 12418       memory {         allocator_name: "cpu_rdma_bfc"         total_bytes: 2574336         peak_bytes: 2574336         allocator_bytes_in_use: 260048896         allocation_records {           alloc_micros:<PHONE_NUMBER>142050           alloc_bytes: 2574336         }         allocation_records {           alloc_micros:<PHONE_NUMBER>150477           alloc_bytes: -2574336         }       }       timeline_label: "[cpu_rdma_bfc 2.5MB 2.5MB] AutoParallel-Replica-0-Accum-Apply/model/gradients/model/lm/sampled_softmax_loss/embedding_lookup/Gather_3_grad/Reshape = SparseAccumulatorApplyGradient(AutoParallel-Accum/model/gradients/model/lm/sampled_softmax_loss/embedding_lookup/Gather_3_grad/Reshape, Const, AutoParallel-Replica-0/model/lm/sampled_softmax_loss/embedding_lookup/DynamicPartition:3, AutoParallel-Replica-0/model/gradients/model/lm/sampled_softmax_loss/embedding_lookup_grad/Gather_3_G818, ToInt64_2)"       scheduled_micros:<PHONE_NUMBER>138009       memory_stats {       }     }
Describe the expected behavior
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Exact command to reproduce

Some of the fields are missing. You can use the latest TensorFlow version then compare graph definition with timeline again.

Closing as this is resolved, free to reopen if problem persists.


Update gfx942 NT/TN/NN FP8/BF8/BF16 Equality

1, Ticket: https://ontrack-internal.amd.com/browse/SWDEV-502660

Only F8 or BF8 input needs Scale A/B

done

All the local CI tests were passed.
[----------] Global test environment tear-down
[==========] 50058 tests from 13 test suites ran. (2350152 ms total)
[  PASSED  ] 50058 tests.
hipBLASLt version: 1200
command line: ./build/release/clients/staging/hipblaslt-test --gtest_output=xml --gtest_color=yes --gtest_repeat=1


Travis ci

Hi !
Here is a PR for travis CI ! It assures code compiles for each platforms and provide a way to release for each platform (just by creating a release in github, the tag will be triggered, as you can see here).
You should provide some environment variable to your travis :

ANDROID_NDK_ROOT: you should set it to /home/travis/build/alexfonseka/android-sdk/ndk-bundle. (it should be readable in console)
FMODUSER: A fmod user to download fmod api. (should not be visible in console)
FMODPASS: the password corresponding to FMODUSER. (should not be visible in console)
TRAVIS_TOKEN: a github personal access token to deploy release. (should not be visible in console)

Hope you'll enjoy it !

Magnificent. I will set this up as soon as I have the time. Distributing compiled versions of the engine as part of the release seems like a great idea so users (specially sound designers and composers) won't have to go through the process of recompiling the engine.

Hi !
I did not get any problem on my side using the script : https://travis-ci.com/piiertho/godot-fmod-integration.
It seems it does not go after the first request:
TOKEN=$(curl -X POST -u $1:$2 https://www.fmod.com/api-login | jq -r '.token')

Did you set valid Fmod account travis environment variables FMODUSER and FMODPASS in travis ci interface ?
It also seems there are some compilation errors on Linux, MacOS, and Android ARM64 since master merge.

Hey there.

Did you set valid Fmod account travis environment variables FMODUSER and FMODPASS in travis ci interface ?
Yeah it's set up in my repository settings on Travis. I'm guessing Travis doesn't make them available for forks and PRs.
It also seems there are some compilation errors on Linux, MacOS, and Android ARM64 since master merge.
Yep it seems like a data type issue. I'll push a fix for it soon.

Hi Alex !
I had same data type issue on GDNative side, I used static cast to handle it.
Maybe it could be simpler if we see CI with skype or discord ?

Pushed a fix 2e8dff5 to address the type issue on Linux and Android64.

Merged and build passed! All good.
Thanks for setting up Travis! 👍


More go-ish and works with go get

Nice work on the package :) We made few adjustment to make it more go-ish and make sure it work with the standard go toolset (i.e.: go get https://github.com/leesper/go_rng would work if you merge)
Removed the pkg dir, moved the file at the root, made sure the test still pass.
Thanks!
Dany

Many thanks to you, Dany ! I really appreciate your help, actually I'm a newbie in golang, well it's quite amazing and surprising haha  ^o^, I will working hard to keep up with you !


feat: Added like feature frontend

Summary of the like functionality (frontend)
Should close #305

feat: Added like feature frontend commit

Added like button and likesCount span element to the Project.tsx file.
Update the ProjectSkeleton.tsx file.
Update Project.interface.ts file.
Update ProjectList.tsx file.

Added liked and likesCount property to the IProject interface commit

hi @zvolcsey could you fix the conflicts and resolve the build issue

@zvolcsey could you please resolve conflicts?
also,

it's not showing any counts in here, can we just hide this when count is 0, rather then showing 0 it would be nice if we doesn't show it.

Sorry, the issue is closed now. But I'm working on it and will push my fixed version soon.

@zvolcsey if possible please make the frontend and backend changes in the same PR so we can test it well.

@rohitdasu Okay. I will open a new PR with frontend and backend.
I created a new frontend design. If it is good, I'll add this to the new PR. Maybe the like button next to the contribution button is better. So the long titles will also look good.

@rohitdasu Okay. I will open a new PR with frontend and backend.
I created a new frontend design. If it is good, I'll add this to the new PR. Maybe the like button next to the contribution button is better. So the long titles will also look good.

looking great! one suggestion, could you please show the likes count inside the like button itself. we have room for the number. no need to show the count above the button. if 0 likes, then don't show it.


Meta refactor

This is an attempt to improve how we grab the metadata for our test runs, knowing that it is possible and likely for the file structure of these test runs to change.
I have created three sets of functions, one for each our buckets, that pull in all the necessary metadata for a run: the bucket and job, the time of the run, the semver of k8s used in the test and the commit of the k8s/k8s repo used, and the list of audit log links for that given bucket and job.
This change required a bit of a refactor across our files, but the aim is to have more consistent errors, and for the functions in general to be shorter and to the point.

Thank you for the work @zachmandeville

Would be great if @zachmandeville and @BobyMCbobs can do full code review next week.


Validators Can Receive ICS-29 Fees Instead of Relayers

Summary of Bug
When the fee middleware is activated, and all validators are rational, an unexpected behavior can occur where all ICS-29 fees are allocated to the validators rather than the relayers.
This issue arises because a validator proposing the next block can replicate and front-run all the MsgRecvPacket, MsgTimeout, and MsgAcknowledgement messages that any relayer may submit. As a result, all the ICS-29 fees are directed to the proposing validator without performing the corresponding work.
Expected Behaviour
The intended behavior is for the wallet running relayer software, which initially submits the MsgRecvPacket, MsgTimeout, or MsgAcknowledgement, to receive the ICS-29 fee. This fee acts as compensation for the relayer's efforts.
Version
This issue affects all ibc-go versions that include the fee middleware.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

(Copying my replies from a slack thread)
I agree this is a problem. I classify it in general as "arbitrage stealing"
I don't think this is a problem worth solving today.
Why? Because:

It will be discoverable and social mechanisms will handle it
Iterated game theory makes the base not so bad
Censorship resistance solutions also fix this "for free".
Economic value here is low that I doubt we will get meaningful proposer thievery in 2024.

When we move to zkp's of things, we can easily start having the proofs "tagged" for the submitting relayer address to prevent this.
The "rational acting block proposer" logic is a bit flawed (its how a lot of early MEV thinking went) for the following reasons:

It assumes relayers and proposers are unable to coordinate/communicate
It does not take into account iterated games

Lets still ignore off-chain relayer/proposer collusion. (Even though it is the end state of such a market with no protocol changes)
Even then, the iterated game theory of this will not lead to all proposers stealing all IBC relays
why? Because then the profit for stealing goes to 0.
Theres a classic game theory argument then (if there were only 2 parties), that says that suppose the minimum relayer R is willing to send a packet for, is payment P. Tip is T.
We'll reach an equilibrium where Miner takes T - (P + epsilon), Relayer gets P + epsilon
(Imagine this as either probabilistic thievery, or kickbacks from proposer to relayers every so often)
I think the multi-validator-ness will make this play out a bit differently then the classic 2 party game theory result (Supposing profit will be vastly higher than it will really be. Cumulative profit per chain here across all proposers is going to be like 100k/year tops in 2024, so realistically its not worth software development time for proposers)
When some proposers start this attack, you'd get relayers defensively keeping the info private if next block is by that proposer. (As the proposer schedule is deterministic)

Thank you for your thoughtful response, @ValarDragon. I understand the reasons you've laid out for considering this issue as not immediately pressing. Social mechanisms are indeed in place to prevent such behaviour in most chains, and I acknowledge their value. However, it's worth noting that these mechanisms and iterated game theory are not bulletproof.
So I'd like to address a couple of points:

Iterated Game Theory: While I appreciate the game-theoretical perspective, it's based on a set of assumptions that might not hold universally. It assumes a degree of rationality and cooperation that might not always exist among relayers and validators.

Tip Equilibrium: I'm skeptical of the equilibrium scenario you've described. In the existing system, the miner doesn't have the luxury of taking a part of the tip; it's an all-or-nothing situation, which undermines the potential for a balanced game theory outcome.

Proposer Schedule: If relayers start to withhold information based on the proposer schedule, it inherently impacts the efficiency and speed of the IBC. This seems counter-intuitive, especially since one of the goals of fee middleware is to expedite these processes, not slow them down.

While it might not be a top priority right now, this issue still represents a potential bug that could have broader ramifications as the ecosystem evolves. Perhaps we could consider some intermediate solutions that don't require an immediate, full-scale overhaul but would still mitigate some of the risks?

Although this is a valid issue, if we assume that tips from fee middleware will be similar to the amount paid in fees, for the Cosmos Hub this would be around $3k / month and for Osmosis around $1k / month (ref). As to whether we do see validators taking fees for packets they did not relay, or the game theory model plays out and validators don't do this for fee middleware users, the amount at stake is still very small.
The solutions to this problem should be around preventing this form of MEV and these solutions should be lower down in the stack and seem a bit beyond the scope of ibc-go - e.g. threshold decryption or using zkps. We considered an allow list to only enable certain relayer addresses to receive the incentives, but the this doesn't really solve the core of the problem and creates a privileged position for certain relayers. It would also result in some overhead for an authority managing this list and an end user would only care about their packets being delivered, not who ends up getting the fee.
Therefore, as Serdar tagged, we will put this issue on hold for now.

I generally agree that this isn't the area of concern of IBC-go. (Though we should request ZKP-based client proofs to add in tagging mechanisms imo, its a pretty cheap win)
I also don't agree with the proposer schedule being a long term problem here. Already in progress plans decentralize block production,

Using vote extensions to have more validators include txs than just the proposer per block
Bullshark consensus algorithms (I imagine all TM chains switching to Bullshark at some timescale)


Update GNU licenses SPDX license identifiers

Some of the SPDX license identifiers of the GNU licenses currently still
in use in this plugin have been deprecated:

Release 3.0 [of SPDX License List on 28th December 2017] replaced previous
Identifiers for GNU licenses with more explicit Identifiers to reflect
the "this version only" or "any later version" option specific to those
licenses. As such, the previously used Identifiers for those licenses are
deprecated as of v3.0.

(from: SPDX license list, see as well: SPDX IDs: How to use)
The following SPDX IDs could be found that are affected by the deprecation:
"LGPL-3.0", "LGPL-2.1", "GPL-2.0", "GPL-3.0"

Also for the GNU license IDs the Affero type of license was missing so I
added it.

@psliwa If there is anything I can do to improve or clarify this PR, or anything you feel uncomfortable with it please let me know. Just in case there is something blocking you from pulling this in. Also if you've got questions, please ask or comment.

This PR is ok. I merged it and release soon. Thank you.

@psliwa  You're welcome, and thank you for having me here.


Adapt Python SDK to Typescript (E2B-30)

Thanks @jzanecook and congratulation for being our first contributor! Ran quick test and seems to be working. Good job!

Thanks!

Is there a reason to prefer this over OpenAPI generator? Did you try this generator?

No, I have not. Should I?

@ValentaTomas correct me, if I'm wrong. The reason for static method is to have one global Agent wrapper and you can then conveniently import it from different files in the codebase without the need to pass it as an argument. At the moment it isn't that important, because there isn't any functionality why would you need importing the Agent yet.

I made the agent use static methods, but ran into issues when both were static, I referenced this article https://stackoverflow.com/a/64733963 and did it that way.

Let's name the package agent-protocol. I have the name reserved.

Done.

Can you add script for generating the boilerplate (models, routes, etc.)

I didn't use a script to generate it, I guess that's part of the OpenAPI generators?

What is the purpose of json-schema-to-typescript in Python SDK?

This was with the pydantic2ts package and should be removed.

When running npm run fix I am getting this errors

This rule requires the `strictNullChecks` compiler option to be turned on to function correctly  @typescript-eslint/strict-boolean-expressions

Fixed, I added this compiler option.

You have prettier among scripts, but it isn't among dev dependencies

I had been using the installed prettier, not the package prettier, fixed now.

Remove jzanecook-e2b-sdk-test from dependecies

Whoopsies.

When trying to build I am getting this

Entry module "src/index.ts" is using named and default exports together. Consumers of your bundle will have to use `agentSdk["default"]` to access the default export, which may not be what you want. Use `output.exports: "named"` to disable this warning

This was a lot more effort than I was expecting lol, but at the end what I had to do was build CJS and ESM separately, reference the build script. I also ran into issues with the typings going into dist/src and erroring, so I added a little extra to the build script to make that go away.

It would be nice to have linted examples/ as well

Added the examples to the linting process in tsconfig, also removed them from the build.

I checked microbundle and it doesn't support default and named exports in one file. So I would prefer not mixing or using tsup. This looks too complicated and probably it's bypassing their design decision to not support it.

That's correct, it was from their github issues I found the resolution with the separated build. You're right though, it's too complicated. I wasn't able to get the tsup build to work at all, though.

Is there a reason to prefer this over OpenAPI generator? Did you try this generator?

No, I have not. Should I?

I would prefer it. This way you would depend on the python package. If any change happens in the spec, you will have to change adjust python code and after that you can adjust js code.

Well that's a shame, I was going in the wrong direction the whole time...

I made the agent use static methods, but ran into issues when both were static, I referenced this article https://stackoverflow.com/a/64733963 and did it that way.

I added some code suggestions to copy the python behavior.

Code suggestions?

Something like this line. When we change the spec, you will just need to run the command to regenerate the code and maybe do some small adjustments. But it should make it much more maintainable.

Okay, I'll take a look and see what I can come up with.

Can we separate these comments out into individual issues so that we can isolate them and clarify the requirements a little easier?

Can we separate these comments out into individual issues so that we can isolate them and clarify the requirements a little easier?

Sure, let's get this done. I'll create an issue for generating the types and endpoints. This works and that's the most important thing! :tada:

So just address the suggestions, rebase on master and let's get it out 💪🏻

Pretty sure I rebased properly here, I also:

Made sure that the build with tsup (which I'm glad worked) also built the types so that if I ran the minimal example with ts-node it wouldn't yell at me.
Removed index.cjs.ts since it's no longer needed.
Added the proper typing to the static function in the Agent class so that eslint wouldn't yell at me.
Successfully verified that nothing is yelling at me! :tada:

PS: It should be noted that eslint and prettier conflict. I've been pushing after prettier, but if you run eslint without prettier it will cause changes in all of the files. We may consider putting the eslint check and the prettier fix into a singular command, or tweaking the eslint to match prettier.

Just last few nit picks and we'll merge it 🎉

Thanks @jzanecook for your contribution!


Error using CommonUtils.addExtendedEmail() in my own job

I'm trying to use CommonUtils.addExtendedEmail() in a builder of my own. Here's the code:
class QuMatrixJobBuilder {
...
    Job build(DslFactory factory) {
        factory.matrixJob(name) {
            it.description this.description
            CommonUtils.addDefaults(delegate)
            CommonUtils.addExtendedEmail(delegate, emails)
        }
    }
}

And here's the error when I run the seed job:
ERROR: (CommonUtils.groovy, line 41) No signature of method: javaposse.jobdsl.dsl.jobs.MatrixJob.extendedEmail() is applicable for argument types: (java.lang.String, jenkins.automation.utils.CommonUtils$_addExtendedEmail_closure2_closure7) values<EMAIL_ADDRESS><EMAIL_ADDRESS>...]

Note that CommonUtils.addDefaults() works just fine
Now, if I try it outside the context of a custom builder, and just adding it to an existing job created with one of the JAC builders, I get the same error:
new QuTaskBuilder(
        name: "qu-virtualenv-tester",
).build(this).with {
    CommonUtils.addExtendedEmail(delegate, Config.emails)
}

Same error as above:
ERROR: (CommonUtils.groovy, line 41) No signature of method: javaposse.jobdsl.dsl.jobs.FreeStyleJob.extendedEmail() is applicable for argument types: (java.lang.String, jenkins.automation.utils.CommonUtils$_addExtendedEmail_closure2_closure7) values: [emails redacted, ...]

this was a pebkac. I needed to add it inside a publishers{} block


Added a basic menu for mobile

To try and alleviate the issue presented in #66, I added simple show/hide menu js functions, added a mobile menu icon, and added CSS rules to properly display based on whether the user is on mobile or desktop.

Waitwait! It turns out we actually want to be able to show and hide these on Desktop, too. Also, I've got some feedback for the code itself. I'll review it now

Actually, after merging so I could open it on my phone for real instead of Chrome dev tools, clicking the menu bar does nothing on iOS (iPhone 7)

Yeah the hamburger menu doesn't actually work in mobile Safari

^ we crossed comments

Also I'm not sure testing by merging into a branch that's automatically pushed to production is a sustainable workflow @RitwikGupta

@brlodi I came, I merged, I regretted

Veni, insedi, paenitui

I was only using chrome's device toolbar to test, my bad!

I'm gonna give this all a try in a few hours, thanks for the feedback @vonbearshark


fix: check for ModelBuilder set PseudoColumns

PseudoColumns must not be escaped but the check using modelbuilder provided information did not work.

Hello @mrt181, thank you for the fix! Could you please rebase the target to release/6.5 branch and also add a note for 6.5.2 in the changelog?

@tomasfabian do you want me to cherry-pick this commit on to the release/6.5 branch?

@tomasfabian do you want me to cherry-pick this commit on to the release/6.5 branch?

Yes, please.

Ok, did that


V3

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

wtf?

That's what I said.  My apologies.
I really like this project vue-form-generator.  I was using this in my
project and found a few things I felt was missing.  I am going thought the
code to see if I can integrate them.  I also found instances where the date
picker will not work properly with the environment we are on.  Well I will
look into that also.  Let me first get familiar with the code.
I was just trying to pull down the v3 code but kept getting v2.3.4.
Thanks you.
On Wed, Feb 27, 2019 at 6:21 PM David Higgins<EMAIL_ADDRESS>wrote:

wtf?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vue-generators/vue-form-generator/pull/599#issuecomment-468110150,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADhoeExudeCylAecwDK4vHHWr7OK1RLlks5vRz0QgaJpZM4bVmJp
.

Closing - this appears to be a mistaken PR


chore: update urql to v3 and add peer dependencies

What does this PR do?

updates urql packages to newly released ^3.0.0
adds urql, wonka, and graphql as peerDependencies (closes #13)
updates other dependencies to latest versions

Why has typescript been updated?

graphql >= 16.0.0 only supported typescript >= 4.7.4

Why the resolutions field?
As graphql requires typescript >= 4.7.4, but tsdx has problem with peer dependencies,
there is a workaround to use the new typescript version:
https://github.com/jaredpalmer/tsdx/issues/926#issuecomment-751936109

@clentfort Hi, thanks a lot for this awesome extension! I'm using urql and your package, and I would love to use the newly released version of<EMAIL_ADDRESS>It'd be nice of you to have a look at this PR. (FYI: Tests run successfully)

It looks like wonka, graphql and urql/core are still listed in dependencies. Ithink they should be moved to devDependencies, otherwise package managers will install those and the peerDependencies won't have an effect, right?

I suppose you're right 😄 Done!

Released as version 1.0.0


chore: Move dev dependencies into dev dependencies

I noticed these deps were being installed when I was working with the
SDK recently.
I think save-dev was added
due to an errant npm i save-dev since it is a holding package so I
removed it (sorry if I added it haha!).
@types/dotenv is no longer needed since the dotenv package pulls
this in independently. It was pulling on dotenv as a peerDep (I think)
so this just moves to depending on the package itself.
The typedoc generator is also used only for development / ci (as far as
I know) so moving.

A few things I noticed:

npm test fails because of missing credentials. Are there instructions on how to set them up? Maybe having a .example.env file with placeholers for orkes credentials might help
The name in the package is conductor-javascript; is that what we are publishing under?

I also noticed the CI action didn't run on this PR. Not sure why, maybe it's due to github actions and secrets?


Output folder naming

Hello @hongyeehh,
One thing we need in order to make the component compatible with any other components is the ability to define custom output names.
When I run this tool I can see how the folder names goes as original-datasetnetworkdate?.
dtepr_mhsa_1716103576/
dtepr_mhsa_1716108417

Would be possible to add a config field in which this folder name can be declare? Otherwise, is there any way to know which folder name is going to have an output?

Hey, maybe this is a good opportunity to discuss about unifying the interfaces of our methods to make it easier for you and potential users? Cheers, S

Sure, that would be really helpful. So far, right now the interface that we are using from the ODTP perspective is describe here in the parameters section. Those parameters can be declared as environment variables in the docker image and tool will be executed with them. We are still working on the data-input/data-output declaration.
However, at this point for us would be enough to have the option to define the output folder naming in order to complete the first working version of the component.

The latest commit fixed this issue with the config file's new run_name parameter. A folder with run_name will be created (if not existing) under run_save_root.
By the way, I am not able to access the link @caviri; maybe the provided repo is private?

Hello @hongyeehh, thanks for the changes. I'll include them in the next version of the component.
I forgot to make public the repository, now you should have access to it. I tested it in different platforms including ARM64, AMD64 (with and without Nvidia GPU).

Thanks for the link! I'll close the issue as completed.


Create consistent performance iscsi disk support

As SL no longer provided the ordering of legacy iscsi disk, softlayer-go needs to make corresponding changes to support consistent performance iscsi disk.
In order to enhance the efficiency of calling softlayer rest api, I created a new kind of http request in softlayer client. Several query methods in softlayer services had been added to leveraging filter function. For example, in production environment, there are a fair mount of storage, itemprice, blockdevicetemplates, etc. So it is better to filtering objects than traversing the result set multiple times.
New http request with objectFilter parameter in softlayer_client.go:
func (slc *softLayerClient) DoRawHttpRequestWithObjectFilterAndObjectMask(path string, masks []string, filters string, requestType string, requestBody *bytes.Buffer) ([]byte, error) {
	url := fmt.Sprintf("https://%s:%s@%s/%s", slc.username, slc.apiKey, SOFTLAYER_API_URL, path)
	url += "?objectFilter=" + filters
	url += "&objectMask=filteredMask["
	for i := 0; i < len(masks); i++ {
		url += masks[i]
		if i != len(masks)-1 {
			url += ";"
		}
	}
	url += "]"
	return slc.makeHttpRequest(url, requestType, requestBody)
}

SoftLayer_Product_Package_Service.go
+ GetItemPricesBySize(packageId int, size int) ([]datatypes.SoftLayer_Item_Price, error)
SoftLayer_Account_Service.go
+ GetIscsiNetworkStorageWithFilter(filter string) ([]datatypes.SoftLayer_Network_Storage, error)

@jianqiu,
In future, please use ``` to format any code in comments. You can also use single quote like this for simple one line or code inline.
For multiple line, use three ` like this:
multiple
line
as
code

All, too many issues. Please address and make sure all errors (I mean, all errors) are dealt with. This means never ever use _ unless you have a very good reason---which you should document as a comment.
There are only two acceptable cases:

you check for error and recover right then in there
you check for error and return it so it can be dealt with by the caller if the method or function.

I will be extra vigilant in will use @cppforlife vetting extension to vet previous code. Seems that I missed some of the old storage code. Let's correct that.
Thanks for you cooperation. Best,
@maximilien

Since there are lots of changes and fixes to be addressed, I am closing this.
Please address all comments and update and resubmit.
Thanks.

@maximilien
Thanks for your explanation in a great detail, I had updated the codes, and submitted a new PR


Use NWXCMake version info

PR Type

[ ] Breaking change
[ ] Feature
[x] Patch

Brief Description
Use the version info from NWXCMake. Also updates the C-CPP workflow to recent changes in CMaize.
Not In Scope
PR Checklist

[x] Is documented
[x] Is tested
[x] Adheres to applicable organization standards

🚀 [bumpr] Bumped! New version:v0.0.3 Changes:v0.0.2...v0.0.3


[Orthography] Check NititaKhatbol profile

There are cases like saxᵐbuɣut -> s a xⁿ b u ɣ u t, so the segmentation is wrong here xᵐb should be x ⁿb.

@LinguList so, in this case, would we simply delete the xⁿ segment from the profile?
Line 104: xᵐ	xⁿ	1

Yes, this normally does the trick!


Fix attribute name mismatches from mlops sdk

Overview

Fix some name mismatches between the mlops sdk and the agent:

model_name -> modelName
numerical -> numeric
type -> feature_/label_type
value -> feature_/label_value
event name -> inferenceData

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop-scikitlearn@ad05014). Click here to learn what that means.
The diff coverage is n/a.

@@                  Coverage Diff                   @@
##             develop-scikitlearn     #845   +/-   ##
======================================================
  Coverage                       ?   75.53%
======================================================
  Files                          ?      202
  Lines                          ?    20215
  Branches                       ?     3440
======================================================
  Hits                           ?    15269
  Misses                         ?     4021
  Partials                       ?      925

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


drag and drop would be nice

I was looking for moving a method from a class to its superclass

moved to webside


Sign In Page is Not Responsive

Seems like the issue didn't autotrigger, but it should've been resolved as of #2770


[Component Proposal] RuleBuilder page

Part of #1445
What package is this for?

[x] React
[ ] Angular

Summary
This Is the containing page for the Rule builder functionality that will live alongside the rule list table. This component is responsible for managing state.

This page should contain the following:

[x] Rule title
[x] Metadata text under title
[x] action bar for star, share, trash, and save
[x] Separation line between header and body
[x] Tab groups. One for building the filter/rule, the other for sharing and updating title and tags
[x] Footer section with three buttons. Cancel, preview filter, and apply filter

Additional context
Invision prototype

closed by #2066


v1.8.1

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.72%. Comparing base (89ad5c2) to head (56cb023).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1808   +/-   ##
=======================================
  Coverage   99.72%   99.72%
=======================================
  Files          87       87
  Lines        5895     5895
  Branches      500      500
=======================================
  Hits         5879     5879
  Misses         13       13
  Partials        3        3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Sub-national regions should link to country

When we recognize sub-national regions - e.g., states in the US, provinces in Canada, we should also provide links to the entity id of the country.

We can add an extra attribute to entities of provinces/states along with the original attribute of wikilink. This will look like:
for ent in doc.ents:
    if ent.label_ in ['US_STATE', 'CANADIAN_PROVINCE']:
        print(f'{ent.text}, type: {ent.label_}, {ent._.wikilink}, parent: {ent._.parent} [{ent.start_char}, {ent.end_char}]')
    else:
        print(f'{ent.text}, type: {ent.label_}, {ent._.wikilink} [{ent.start_char}, {ent.end_char}]')

Or we can just tack on the link after the matching process is done by checking the label type of the entities:
for ent in doc.ents:
    if ent.label_ == 'US_STATE',:
        print(f'{ent.text}, type: {ent.label_}, {ent._.wikilink}, parent: {USA_WIKILINK} [{ent.start_char}, {ent.end_char}]')
    elif ent.label_ == 'CANADIAN_PROVINCE':
        print(f'{ent.text}, type: {ent.label_}, {ent._.wikilink}, parent: {CANADA_WIKILINK} [{ent.start_char}, {ent.end_char}]')
    else:
        print(f'{ent.text}, type: {ent.label_}, {ent._.wikilink} [{ent.start_char}, {ent.end_char}]')

@lintool which option seems better? I think adding an extra attribute is more consistent.


[doc] Reorganize sidebar, rewrite overview page and rename sections

I'm following some of @snario's suggestions.
The idea is to:

define our target audience better
expand the scope of the documentation from nitro-protocol to the whole statechannels stack

Once this stuff is settled, it should be easier to see where the holes are, that need to be filled in before we push this out / publicise it.

Preview https://5f181ca6e8517e019d88d909--nitro-protocol.netlify.app

Suggest that this changeset be accompanied by

URL change to docs.statechannels.org
link from statechannels.org is named 'docs' instead of 'protocol docs'


Gemini integration

We still need to have lots of more integration:

Multimodal features
Notebook: more examples, including RAG, competition, etc.
Robust test cases
Function calls and other advanced features
Blog post about the Gemini's integration.

Why are these changes needed?

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
Attention: 114 lines in your changes are missing coverage. Please review.

Comparison is base (e18dc33) 26.63% compared to head (ce5eaf7) 0.41%.

Files
Patch %
Lines

autogen/oai/gemini.py
6.66%
98 Missing :warning:

autogen/oai/client.py
30.76%
9 Missing :warning:

autogen/token_count_utils.py
0.00%
7 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #978       +/-   ##
==========================================
- Coverage   26.63%   0.41%   -26.23%
==========================================
  Files          28      27        -1
  Lines        3777    3883      +106
  Branches      858     920       +62
==========================================
- Hits         1006      16      -990
- Misses       2700    3867     +1167
+ Partials       71       0       -71

Flag
Coverage Δ

unittests
0.41% <8.80%> (-26.17%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Move into Microsoft/autogen/gemini branch. Close this PR and please visit: https://github.com/microsoft/autogen/pull/979


Http server parsing post parameters

Parsing on disk
Http server parsing post parameters failed with "file exists already" error
Steps to reproduce
Post empty value for a field
Netty version
I am using 4.1.6.
I RESOLVED IT WITH OVERRIDING AbstractDiskHttpData->setContent
public void setContent(ByteBuf buffer) throws IOException {
    if (buffer == null) {
        throw new NullPointerException("buffer");
    }
    try {
        size = buffer.readableBytes();
        checkSize(size);
        if (definedSize > 0 && definedSize < size) {
            throw new IOException("Out of size: " + size + " > " + definedSize);
        }
        if (file == null) {
            file = tempFile();
        }
        if (buffer.readableBytes() == 0) {

////////////////////// CHANGE BEGIN ////////////////////////
if(file.exists())
return;
////////////////////// CHANGE END ////////////////////////
// empty file
if (!file.createNewFile()) {
throw new IOException("file exists already: " + file);
}
return;
}
FileOutputStream outputStream = new FileOutputStream(file);
try {
FileChannel localfileChannel = outputStream.getChannel();
ByteBuffer byteBuffer = buffer.nioBuffer();
int written = 0;
while (written < size) {
written += localfileChannel.write(byteBuffer);
}
buffer.readerIndex(buffer.readerIndex() + written);
localfileChannel.force(false);
} finally {
outputStream.close();
}
setCompleted();
} finally {
// Release the buffer as it was retained before and we not need a reference to it at all
// See https://github.com/netty/netty/issues/1516
buffer.release();
}
}

@yusufbulentavci please open a PR with the proposed fix.

@yusufbulentavci ping

Pong @normanmaurer  https://github.com/netty/netty/pull/7697


feat(adpater-node): support all esbuild build options

This PR allows changing the options passed to esbuild build which allows to customize the server generation like target node version, sourcemap, inject, and more.
It contains one braking change: out is renamed to outdir to follow the esbuild API. It is possible to avoid a breaking change by making out an alias for outdir. I'm happy to adjust the PR.
Before submitting the PR, please make sure you do the following

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts

I think it would be clearer if these options were nested under single esbuildOptions param field instead of spread amongst the base param obj.

I think it would be more clear if these options were nested under a single esbuildOptions param field instead of spread amongst the base param obj.

I have adjusted the PR to use a esbuildOptions parameter. I changed the default options to use outdir instead of outfile to support code splitting. This still creates a single build/index.js file because the entrypoints filename is index.js. If the outfile option is set outdir will not be used because esbuild expects either outfile or outdir.

Commit message has a typo.

Commit message has a typo.

Sorry, I do not know what you mean.

https://github.com/sveltejs/kit/pull/1692/commits/c6075d7f54a464ff8be743fbce0effb0d38cc432
I assume the title is supposed to be "adapter" and not "adpater"

Same thing in the commit message. (Don't know if you will fix it later or overlooked it)

c6075d7
I assume the title is supposed to be "adapter" and not "adpater"

Same thing in the commit message. (Don't know if you will fix it later or overlooked it)

I can not fix that now. But I believe that PRs are squashed and use the PR title (which is fixed) as commit message.

Just thought I would raise before we adopt esbuild more significantly, but the initial usage of esbuild was a suggested soln

Mostly just playing around for now, but this is intended to solve the perennial problem of dependencies behaving weirdly, by allowing adapters to determine what gets bundled.
-Rich Harris in #1091

Has there been more thought on alternative solutions or is esbuild usage likely to stick around? @Conduitry @benmccann

@jthegedus it's not an area I understand well, so I will defer to Rich and Conduitry

It looks like this PR needs to be rebased

It looks like this PR needs to be rebased

Done.

There's a related PR for adapter-vercel here: #1732

I do not know if using config.kit.vite().esbuild is the best way here. In my specific use-case I want to use a different esbuild config (mainly target and format).
Maybe a hybrid approach:
{
	...config?.kit?.vite()?.esbuild,
	...esbuildOptions,
}

But that would make reasoning about the resulting config difficult.
What is your take on this?

Is there anything I can do to progress this PR?

Since the common js output format is also available in this PR. I'm wondering if we can check if  __dirname and require already exist before shim it, so the configuring is easier. I tried building it to cjs before and it's throwing an error in the runtime because import.meta.url is undefined.
But regarding how to check if __dirname exists, I can only think of some hack like this:
var __dirname;
__dirname = __dirname ?? dirname(fileURLToPath(import.meta.url))

Since the common js output format is also available in this PR. I'm wondering if we can check if  __dirname and require already exist before shim it, so the configuring is easier. I tried building it to cjs before and it's throwing an error in the runtime because import.meta.url is undefined.
But regarding how to check if __dirname exists, I can only think of some hack like this:
var __dirname;
__dirname = __dirname ?? dirname(fileURLToPath(import.meta.url))

(Sorry I spamed the notification. accidently deleted the previous comment)

Since the common js output format is also available in this PR. I'm wondering if we can check if __dirname and require already exist before shim it, so the configuring is easier. I tried building it to cjs before and it's throwing an error in the runtime because import.meta.url is undefined.
But regarding how to check if __dirname exists, I can only think of some hack like this:
var __dirname;
__dirname = __dirname ?? dirname(fileURLToPath(import.meta.url))

(Sorry I spamed the notification. accidently deleted the previous comment)

I use define + inject:
shim-node-cjs.js
import { pathToFileURL } from 'url'

export const shim_import_meta_url = /*#__PURE__*/ pathToFileURL(__filename)

And the in the esbuild config:
    // Or something like path.relative(import.meta.url, './shim-node-cjs.js')
    inject: [require.resolve('./shim-node-cjs.js')],
    define: {
      'import.meta.url': 'shim_import_meta_url',
    },

This works quite well for single file outputs.

I'm also fine with an example in the doc. I just thought making it work out of the box would be a little bit easier for the user.

Why are you wanting to change target and format?

Why are you wanting to change target and format?

We are bundling the server and the assets using pkg into an executable. pkg needs cjs as format and we can use a higher target (v16).

And we're hosting with Windows IIS with iisnode, which doesn't seem to support ESM.
People using other hosting platforms that don't already have an official adapter may want to leverage the node adapter. And since some other adapters also ship common js. I think it would a good idea to allow configuring node adapter to output common js.

@sastan it looks like this PR will need to be rebased

I do not know if using config.kit.vite().esbuild is the best way here

I agree. It's better to keep them separately configurable


Update generated Go source code for proto libraries [phab:D873]

Created by schroederc at 2016-05-03 18:44:08:

shahms accepted this revision at 2016-05-03 18:44:40

schroederc closed via commit cbfc1fa41ce08394d1a5c18941b9fde61f4b13a9 at 2016-05-03 18:49:13


raspberrymatic addon (homeassistant) is blocking USB ZigBee Device

Describe the issue you are experiencing
I would like to use the raspberrymatic addon on a homeassistant yellow with the HmIP-RFUSB
But I cannot figure out how to use the Device without blocking the Zigbee USB Stick I am using.
switching to the integrated zigbee device of the homeassistant yellow is not an option as it is not working well with my zigbee devices
Describe the behavior you expected
an option to use a specific /dev/ttyUSBxx device would be good I think
Steps to reproduce the issue

...
What is the version this bug report is based on?
<IP_ADDRESS>30506
Which base platform are you running?
ha-addon (HomeAssistant Add-on)
Which HomeMatic/homematicIP radio module are you using?
HmIP-RFUSB
Anything in the logs that might be useful for us?
no logs available

Additional information
No response

Please be more verbatim. Thus, state which Zigbee USB stick you are using and show potential error messages / output from the home assistant error log where one can see that zigbee is not available at all when the raspberrymatic addon is running. Just stating "this does not work" is unfortunately too less information.

Maybe CUxD is installed as AddOn inside RM-HA-AddOn and grabs (and locks) all USB-Devices.

Maybe CUxD is installed as AddOn inside RM-HA-AddOn and grabs (and locks) all USB-Devices.

That's a good idea to check. So please @marc4s check/state if you have the CUxD addon installed within the RaspberryMatic HA Add-on which could explain that the Zigbee USB device will be grabbed and thus is unavailable to the host system (Home Asssistant) when RaspberryMatic is running.

some more information:
I had a RPI3 Homematic before and I restored a backup which had already the TTYPARAM=none in CUxD
and after restore the backup (with this config) it was blocked. So I stopped the addon... After starting it again it was not blocked anymore. so it is solved. but why it needed to be restarted I do not know. maybe CUxD was started before the config was rolled out or so... does not matter.


Enhancement request: Add utility classes for responsive behaviour

It will be helpful if some helper utility classes (for responsiveness) are added, something as follows:

hide-for-medium-up (hides the element for screen sizes 'medium width' and up)
show-for-large-down (shows the element only for screen sizes of large width and below).

Something to keep in mind (WHEN possible) while writing the styles the default should always consider smallest possible screen sizes first (mobile-first-approach), the modifiers should only modify the behaviour for targeted screen-sizes and up.

@sambhav-gore I've got a 1st version. I have one issue with it is that to target a specific size only (e.g hide only for medium) you have to use two classes (one hide from, one show from). Maybe we should have classes that target one specific size? e.g. dc-hide-for-medium

@gabrielhl I like the idea of targeting specific size as well.


Fixed Grid lines issues.

Fixed Issue: Grid rows were treated as columns and columns were treated as rows due to which grids don't fill up whole image when image isn't square.

Good catch; thanks!


autoprefixer can not find grid areas when using @media

Versions
Angular CLI: 1.7.2
Node: 9.4.0
OS: linux x64
Angular: 5.2.7
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.7.2
@angular-devkit/build-optimizer: 0.3.2
@angular-devkit/core: 0.3.2
@angular-devkit/schematics: 0.3.2
@ngtools/json-schema: 1.2.0
@ngtools/webpack: 1.10.1
@schematics/angular: 0.3.2
@schematics/package-update: 0.3.2
typescript: 2.5.3
webpack: 3.11.0

Repro steps
Having @media tag prevents it from finding associated grid-area.
✗ The following does output:
WARNING in ./src/app/core/footer/footer.component.scss (Emitted value instead of an instance of Error) autoprefixer: /.../footer.component.scss:28:6: Can not find grid areas: social
footer.component.scss
@media (min-width: 550px) {
  footer {
      grid-template-areas:
        "social ..."
        "...";
    }
}

.social {
  grid-area: social;
}

✓ The following does NOT output any warning:
footer {
    grid-template-areas:
      "social ..."
      "...";
}

.social {
  grid-area: social;
}

Desired behavior
@media should not prevent SCSS from finding grid-area outside its scope.
Other
Related: https://github.com/angular/angular-cli/issues/9502

I have the same issue.

Same here
Angular CLI: 1.6.7
Node: 8.9.4
OS: win32 x64
Angular: 5.2.3
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cdk: 5.2.0
@angular/cli: 1.6.7
@angular/material: 5.2.0
@angular-devkit/build-optimizer: 0.0.42
@angular-devkit/core: 0.0.29
@angular-devkit/schematics: 0.0.52
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.9.7
@schematics/angular: 0.1.17
typescript: 2.5.3
webpack: 3.10.0

anyone found a solution to this ?

We've been using a combination of the two answers found here as a workaround: https://stackoverflow.com/questions/48588493/angular-5-and-css-grid-cannot-find-grid-areas-warning

Is this related? https://github.com/postcss/autoprefixer/issues/954

@krokofant interestingly enough I just added 8.6.2 to our project and that didn't seem to fix it. Is it possible the CLI is using a different version than what's in my node_modules?

@mcblum Try to hack the autoprefixer package that is used by @angular-devkit/build-angular (node_modules@angular-devkit\build-angular). That might show if it works or not.
In my current setup "autoprefixer": "^8.4.1" is used. I think that takes precedence.

@bbonczek If you look at the autoprefixer repo you'll see that this is mentioned there.

Haha, always IE who breaks the party.

@krokofant true, haven't seen this before. In that case I assume there is no bug in angular-cli then?

Remember it's just a warning. IE does actually have support, but it's partial support according to https://caniuse.com/#feat=css-grid

@bbonczek I have added the browserslist to my package.json, I am still seeing the warnings.  Is browserslist a top level property in package.json?

Yes, I think most of us don't care about IE compat. Either way the browserlist file should work and did... until I updated the CLI. Currently at 6.2.1
Browserlist contains
last 1 version

5%
not dead
not ie > 1
not ie_mob > 1


Export all core extensions and add nav section type

Export all core extensions
Since we want to allow apps to utilize fully all types we defined, let's export them.
New navigation section
To fully support all navigation features we need to add nav section as well.

/assign @vojtechszocs

/lgtm
In future I'd pick types we need to expose for SDK consumers, for now we can expose everything.


Testing framework native envoy support dynamic listener

We post-processing pilot's discovery response, https://github.com/istio/istio/blob/master/pkg/test/framework/runtime/components/apps/agent/pilot_agent.go#L597

Trim away the dynamic listeners.
Rewrite the clusters for A->envoy->envoy->B. Due to lack of the iptable.

This issue is to address the first one. Potential solutions,

Configure app using different label. So that app can be distinguished.
Convert that to static config and then append.

To get permissive e2e test working, more stuff is needed:

generate localhost/127.0.0,1 cert locally.
configure authn config and then wait for that being propagated to sidecar. (it takes some delay)


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/29633

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/35979

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.


Enable Istio resources redeployment after install and upgrade

Description

enable Istio resources redeployment after install and upgrade

Links
See: https://github.com/kyma-incubator/reconciler/issues/247

/retest


Actor metrics - incorporate feedback, add more tests, put some metrics tests to be common

Description
This tackles the comment on the PR https://github.com/dapr/dapr/pull/6255#discussion_r1176001516 that was merged.
Issue reference
Please reference the issue this PR will close: #6032
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[x] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/cc @mukundansundar

milestone tracked by linked issue


BOM-1512 : Replace deprecated loadstatic and admin_static syntax

Relevant JIRA : https://openedx.atlassian.net/browse/BOM-1512

Tested in https://github.com/edx/credentials/pull/766


Make some pep8 fixes

@mambocab to review

Small request for changes in quoting. Other than that, +1. Pretty metal.


Tooltipster not working if elements having same id

I have update the latest version of tooltipster(tooltipster-master 4.1.5) in my project.If elements are having same id,then tooltipster works first element only.For other elements it doesn't work.
Check the simple example : https://jsfiddle.net/g3quuqe1/7/
Note : In older versions(3.3.1), it's working fine.
We are using id for some purpose.

Hi, in HTML, two elements should never have the same id.

Use classes instead

@louisameline Now i got elements which are having title attributes.And i have added the toolsipter for these elements using below quotes

$(document).ready(function() {
$('[title]').tooltipster();
});

this also not working.Check the same https://jsfiddle.net/g3quuqe1/8/
It works well in older versions.And I hope it's an issue in the latest version(tooltipster-master 4.1.5).

I have answered this. In HTML, two elements should never have the same id.

If you write bad HTML, don't be surprised when bad things happen.


Incompatible types with JDK 8

The basic use of the library informed by the site of simple mail would be this:
`Email email = new Email();
email.setFromAddress("Michel Baker", "m.baker@mbakery.com");
email.addRecipient("mom"<EMAIL_ADDRESS>RecipientType.TO);
email.addRecipient("dad"<EMAIL_ADDRESS>RecipientType.TO);
email.setSubject("My Bakery is finally open!");
email.setText("Mom, Dad. We did the opening ceremony of our bakery!!!");
new Mailer("server", "25", "username", "password").sendMail(email);`
However, there are no more references from the RecipientType in JDK 8, and I could not find an alternative to this either.

I'm a little confused by your report.
RecipientType is not part of the JDK, but JavaMail, which is stated as dependency of Simple Java Mail. Further, there are many more types used from JavaMail which are not in JDK, so you should get errors for those too.
        <dependency>
            <groupId>com.sun.mail</groupId>
            <artifactId>javax.mail</artifactId>
            <version>1.5.5</version>
        </dependency>


Move to bluebird

Compared to Q which we are using now, bluebird has better performance and compatibility.

Use native promises by default

@Jokero planning for this.

@shernshiou @Jokero +1 for using native promises that come easily with Node v4+

Closed in favor of native promise #65


Add UI tests

Provide intent tests for the requirements you support. Deliver the test code to your source repository. If you have test data files, also include those. Test data should be realistic.

I can write UI tests for registration

I can write the UI tests for the books

@siyuamymi what part of the books? I was planning to do that but I can do something else instead

I'm just starting now so if you've already started, I can do something else like maybe profiles

Yeah, I already started a bit. I should have said something
If you could do profiles that would be great ^^

No prob, ill do profile rn

I can write UI test for the borrower section.

#101 partially completes this

Honestly, I think that we should only have UI tests for adding/viewing/deleting books as well as viewing/editing the profile since those are fairly self contained. @Kaho-Lee @MarcusBoay  We should not have UI tests for requests/borrowing since they require you to make requests on books that are already in the database. It would be hard to make those tests very repeatable and accurate since they depend on other data in a live production instance.

Looking at some past projects, they didn't have many UI tests so I don't think we should worry about them too much. I think signin/book/profile tests should be good enough.

I think we have enough UI tests for this part. I'm going to close this issue.


Observable to track visibility changes

I see that there is a consumer to set the view's visibility but I couldn't find an observable variant for the same. I'm trying to do something like this (for now I used a workaround but ideally an observable that emits visibility changes would be the best for this job):
secondItem.visibility()
  .subscribe ({
    if(it == View.VISIBLE) {
      (itemWhichShouldFillLayout.layoutParams as LinearLayout.LayoutParams).weight = 0.65f
    } else {
      (itemWhichShouldFillLayout.layoutParams as LinearLayout.LayoutParams).weight = 0.8f
    }
})

What listener would you use without RxJava for this?
On Sat, Feb 29, 2020, at 4:18 PM, Saifur Rahman Mohsin wrote:

I see that there is a consumer https://github.com/JakeWharton/RxBinding/blob/master/rxbinding/src/main/java/com/jakewharton/rxbinding3/view/ViewVisibilityConsumer.kt to set the view's visibility but I couldn't find an observable variant for the same. I'm trying to do something like this (for now I used a workaround but ideally an observable that emits visibility changes would be the best for this job):

mainLayout.visibility()   .subscribe ({      if(it == View.VISIBLE) {       (itemWhichShouldFillLayout.layoutParams as LinearLayout.LayoutParams).weight = 0.65f     } else {       (itemWhichShouldFillLayout.layoutParams as LinearLayout.LayoutParams).weight = 0.8f     } })

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/JakeWharton/RxBinding/issues/543?email_source=notifications&email_token=AAAQIEKOMZECMITT6UR4QLDRFF5Q7A5CNFSM4K66MFNKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IRNS3NQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAQIELH7VCB2YHQ6VKWXB3RFF5Q7ANCNFSM4K66MFNA.

Like I said, I wrote a workaround to solve this for me without using any listener coz my specific application needed to check visibility only when a button was clicked so I coded it to recheck visibility every time the button click click happened and make the relevant views hide or show based on an if condition. But I’m sure it would be good to have a hot observable in the view class for this.
But as per the Android docs I believe there is one onVisibility Changed listener for this.

Like I said, I wrote a workaround to solve this for me without using any listener coz my specific application needed to check visibility only when a button was clicked so I coded it to recheck visibility every time the button click occurred and made the relevant views visible or gone based on an if condition. But I’m sure it would be good to have a hot observable in the view class for this.
But as per the Android docs I believe there is one onVisibility Changed listener for this.

Same issue. This listener is a protected function cannot write a kt extension to observe the change. Any ideas?

I see that there is a consumer to set the view's visibility but I couldn't find an observable variant for the same. I'm trying to do something like this (for now I used a workaround but ideally an observable that emits visibility changes would be the best for this job):
secondItem.visibility()
  .subscribe ({
    if(it == View.VISIBLE) {
      (itemWhichShouldFillLayout.layoutParams as LinearLayout.LayoutParams).weight = 0.65f
    } else {
      (itemWhichShouldFillLayout.layoutParams as LinearLayout.LayoutParams).weight = 0.8f
    }
})

Hola
_@hola _


Android Camera Permission

What is the specific issue that causes to need a rooted device?

This part:
su
setenforce 0
lsusb
chmod 0777 /dev/bus/usb/<Bus number>/<Dev Number>

from AndroidJavaApp.md.
There are several alternatives, including building a custom image as proposed in some pull-requests (#2489). We are still evaluating this method, but from my personal experience, building Android image from source is not something I as a developer would do at home.
Some people work-around this issue by requesting permissions at Java level and passing the resulting file-descriptor to libusb (see UVCCamera). We evaluated this method and chose not to endorse it, for two reasons -

There is no guaranty Google wont shut-down this method at some point.
This is not a built-in capability of libusb, thus requires maintaining a custom version of libusb.

Since librealsense is permissively licensed and open-source, you are free to pursue this approach, including commercially, but we can't offer support.
We are also looking into other methods.
Intel also has a dedicated implementation of the camera HAL it shares with its partners, but this is also not really suitable for dev-kits.

I'm also interesting with this question. appreciate too much @dorodnic 's detailed elaboration.
is there any planning or roadmap of librealsense's supportive with Android ?
or what's the planning or roadmap of librealsense's supportive with Android?
we both know that Intel's Realsense Camera is an imagination and creative product and the librealsense is also a excellent c++ library. and we also know that there are huge Android based devices in the mobile market or IoT market, it will be very helpful if we could get more clear planning/roadmap of librealsense.

I think the approach for handling communication for android, is with UsbManager class, instead of libusb, besides it should stop using libuvc at all.
To properly support android, this means replacing libuvc, and by doing that, you can't use stream class directly, therefor all the content of libuvc folder has to be reimplemented in android. Then in the native lib, orchestrate an input/out flow.

Thanks the detailed expatiate.  I have following questions:

could we maintain a custom libusb for librealsense?  then invoke the standard Android Java API in native layer to obtain the required infos for librealsense.

the following is a simple example to illustrate the possibility of the above proposal:
the apk would preview bag file on non-rooted Android phone:
https://github.com/zhouwg/irsa/blob/master/img-folder/irsa_example_on_non_rooted_phone.jpg

it seems there are two major issues with librealsense on Android device:  patch Android kernel; root privilege to access /dev/video* and /dev/bus/usb/*.
librs could running well on any non-rooted Android device  if Google could provide a customize  AOSP which add the librealsense kernel patch in Android kernel and add some SELinux  "door" for librealsense.


Fixes #1412 (removalPredicate now uses unification instead of and simplification)

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock

Decided not to pursue this further in this iteration.


Client Allocation Request for: himelight

Client Allocation Questions
Core Information

Name: himelight
Website / Social Media: https://www.himelight.com/org/59a7d564c809aa474f41089c
Region: Asia excl. Greater China
DataCap Requested: 10TiB
Addresses to be Notarized: f1wwuty54bkquibpqor4xkuam334xme2rqwzmojqq
Notary Requested: ozhtdong

@himelight Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Thanks for your application.
I want to know more basic information of you and your company. Please answer following question:
1.What's your name? What's your position?
2.What's your company's business? Do you have any other social media link?
3.Could you show me your company's business license?

Kaijie Zhan. I am a legal person.
It is a team of capital workers who want to use the technology platform to benefit themselves, add interest, and change the habitual mode. Jidianshu is committed to developing the most interesting and spoof regular customer management platform. It digitizes the traditional marketing tools such as gathering points, coupons, birthday coupons, etc. to achieve customer management and big data analysis. Through gathering points, it records customer consumption profiles. Achieve better marketing efficiency and stream smart retail for stores.
https://www.himelight.com/org/59a7d564c809aa474f41089c
This is our official website.
Yes, I will show you via email.

Hi. This is our business license.
On Sat, Sep 18, 2021 at 3:34 PM ozhtdong @.***> wrote:

Thanks for your application.
I want to know more basic information of you and your company. Please
answer following question:
1.What's your name? What's your position?
2.What's your company's business? Do you have any other social media link?
3.Could you show me your company's business license?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/filecoin-project/filecoin-plus-client-onboarding/issues/881#issuecomment-922233395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVV4ADBTCOACZKLZMGDB5DTUCQ6JBANCNFSM5EIXWIJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thanks for your quick reply and detailed information.
I have further more questions about data and storing.
1.What kind of data do you want to store and how much? Could you please send me some demos of your data?
2.Have you/your company ever used cloud? Which platform do you use? What's your usage volume and frequency?

Our data is the information of the merchant discount plans. I will email you too.
No we haven’t use any cloud before.

Here are demos.
On Sat, Sep 18, 2021 at 5:48 PM ozhtdong @.***> wrote:

Thanks for your quick reply and detailed information.
I have further more questions about data and storing.
1.What kind of data do you want to store and how much? Could you please
send me some demos of your data?
2.Have you/your company ever used cloud? Which platform do you use? What's
your usage volume and frequency?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/filecoin-project/filecoin-plus-client-onboarding/issues/881#issuecomment-922250518,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVV4ADBAMWMTO4D67T764J3UCROABANCNFSM5EIXWIJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thanks for your information.
The last step, please show me your company searching consequence in the local business license searching website.

Check it please.
On Wed, Sep 22, 2021 at 10:48 AM ozhtdong @.***> wrote:

Thanks for your information.
The last step, please show me your company searching consequence in the
local business license searching website.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/filecoin-project/filecoin-plus-client-onboarding/issues/881#issuecomment-924537390,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVV4ADAKKOH7NE2WJ2D23PTUDE7YTANCNFSM5EIXWIJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzaceb342ggbpxwi6ci7jqlvdlsxpkgqn66eugnd6ruujca3kv23dkrb4

Address

f1wwuty54bkquibpqor4xkuam334xme2rqwzmojqq

Datacap Allocated

10TiB

You can check the status of the message here: https://filfox.info/en/message/bafy2bzaceb342ggbpxwi6ci7jqlvdlsxpkgqn66eugnd6ruujca3kv23dkrb4


Hugo prints escape chars when running in Windows shell

When Hugo runs under Windows 10 powershell (or legacy command shell) it prints non sense ANSI escape sequences before and after "Building sites" message
e[K25lBuilding sites … e[?25h

Since Windows 10.1607 it looks like we need to set ENABLE_VIRTUAL_TERMINAL_PROCESSING  using SetConsoleMode if we want ANSI support.
Previous builds had it enabled by default (Windows 10.1511) and older versions did not support it.
Here is some of the docs I found on this:

https://docs.microsoft.com/en-us/windows/console/console-virtual-terminal-sequences
https://docs.microsoft.com/en-us/windows/console/getconsolemode#ENABLE_VIRTUAL_TERMINAL_PROCESSING


sommelier: init at 2020-08-10-be4e16f

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build sommelier

What are you doing with it?

For testing:
sommelier --master  --display=wayland-0 --socket=wayland-1
sommelier -X --display=wayland-1 xterm

I am still getting following errors:
xdg_surface@61: error 3: xdg_surface has never been configured

This is most likely to be fixed with: https://patches.videolan.org/patch/20875/

@colemickens using for lightweight secure dev environments, have several iterations how i want to test it (locally with multiple users sharing same root compositor, docker dev containers, kata containers, just a few ideas)

@colemickens thanks for initial package

@offlinehacker I'm very interested in those use-cases, I'd love to know if you get cool stuff working. You might be interested in https://spectrum-os.org/ and #spectrum on freenode if you're not already familiar.
@offlinehacker also, no worries, I didn't know any of it was useful. I think I had (tried to?) implemented xdg stable but never got anything to actually work...

I was also trying with drm support using:
sommelier --master  --display=wayland-0 --socket=wayland-1 --shm-driver=dmabuf --drm-device=/dev/dri/card0
sommelier --display=wayland-1 alacritty

Getting these errors:
MESA-LOADER: failed to open iris (search paths /run/opengl-driver/lib/dri)
failed to load driver: iris
MESA-LOADER: failed to open kms_swrast (search paths /run/opengl-driver/lib/dri)
failed to load driver: kms_swrast
MESA-LOADER: failed to open swrast (search paths /run/opengl-driver/lib/dri)
failed to load swrast driver
error: couldn't get display device

I have iris driver, but i did not dig too much yet, why it does not work.

Does my Sommelier package work? chromiumOSPackages.sommelier in https://spectrum-os.org/git/nixpkgs

Not sure what you mean about "official"
source -- I fetch from Google Gitiles just as you do.

Sorry I have mistaken with this fork from @colemickens https://github.com/colemickens/platform2-sommelier

Looks like they just reverted the update to latest xdg shell, probably same issues with xdg_surface. https://chromium.googlesource.com/chromiumos/platform2/+/32050c0ea6c00c16999915856b40a6a6b8b41bb9


add initiate_chats

Why are these changes needed?

Adding an initiate_chats API so that one can construct a sequence of chats with different senders (comparing to the initiate_chats method of a particular ConversableAgent).
Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
Attention: 37 lines in your changes are missing coverage. Please review.

Comparison is base (899b250) 37.79% compared to head (1071dcc) 20.31%.

Files
Patch %
Lines

autogen/agentchat/chat.py
14.70%
29 Missing :warning:

autogen/agentchat/utils.py
62.50%
3 Missing and 3 partials :warning:

autogen/agentchat/conversable_agent.py
50.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1638       +/-   ##
===========================================
- Coverage   37.79%   20.31%   -17.49%
===========================================
  Files          50       50
  Lines        5704     5745       +41
  Branches     1295     1412      +117
===========================================
- Hits         2156     1167      -989
- Misses       3377     4428     +1051
+ Partials      171      150       -21

Flag
Coverage Δ

unittests
20.31% <33.92%> (-17.49%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Looks like a really useful interface.
I need to test it this week on my workflows, but I think it's a natural extension to the agent's initiate_chats mechanism from the week prior.
One natural question this PR raises though is: Do we still need initiate_chats in conversable_agent? It seems like this could completely replace it.

The test failed: https://github.com/microsoft/autogen/actions/runs/7881102985/job/21504193590?pr=1638


Added pull_request_template + CODE_OF_CONDUCT

Added pull_request_template + CODE_OF_CONDUCT

 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.


Ensure errors received from device prevent creation of LB components

Currently, if you try to do something like lbaas-loadbalancer-create on a device that isn't ready yet (you just rebuilt the appliance in stack, for example), the driver throws an error but the elements are still created in Openstack.  You can't delete it because it doesn't exist on the device so you're forced to manually delete the record from the DB (--good).  We need to ensure that said error condition is propagated to the caller so it knows not to create DB records/etc for stuff that's broken.

If the device isn't ready yet, the command should probably be retried again later. Is there a way to signal to neutron that the command should be re-queued? Once a record is created (for whatever reason) and there isn't a corresponding object on the device, deleting the object in neutron should succeed.

We should at least set the status_description when commands fail, so the reason is visible in the neutron commands as well as in the logs.

Returning and recording nice error messages is #179


Filter multiple valid turn server entries in Edge

Description
Edge throws an error when using multiple valid turn servers (https://github.com/webrtc/adapter/issues/451). This is an attempt to fix it by filtering out additional turn servers.
It filters servers in .url, multiple servers in .urls and in separate ice server entries.
Purpose
Filter out any additional turn servers in the ice servers array to prevent Edge from throwing an error when creating a RTCPeerConnection.

I'll take a look at the test case you posted.
Do you need me to squash everything into a single commit?

@Fippo, we allow "turn", not "turns". For turn, we allow both "udp" and "tcp", however, limit to only one turn per type, i.e. one "udp" and one "tcp".

@kapetan looks like ?transport=tcp is filtered in another place and that needs to be removed before the filter is improved.
I think we can merge your code as is, feel free to squash yourself or @KaptenJansson will.

LGTM, will squash


tests: add helm upgrade tests

What this PR does / why we need it:
This PR adds some basic scaffolding for helm upgrade tests
Which issue this PR fixes
Part of https://github.com/Kong/gateway-operator/issues/155
Special notes for your reviewer:
~In order to perform update form pre 1.2 to 1.2+ we need to be able to disable ControlPlane and Gateway controllers because their respective CRDs (ControlPlane and GatewayConfiguration) got a breaking change in 1.2 (bump from v1alpha1 to v1beta without a KGO version that would allow a smooth transition).~
~This requires a change in the chart which would allow setting args because only KGO 1.2+ supports configuration via env. https://github.com/Kong/charts/pull/1058 implements that.~
The upgrade from pre 1.2 to 1.2 and up is not done as helm chart (used in this test suite) only contains the CRDs from 1.2 and since helm still doesn't provide CRD management there's no easy way to achieve that (well apart from applying the CRDs manually but we'd need to add manual steps for that).
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[ ] the CHANGELOG.md release notes have been updated to reflect significant changes

In order to perform update form pre 1.2 to 1.2+ we need to be able to disable ControlPlane and Gateway controllers because their respective CRDs (ControlPlane and GatewayConfiguration) got a breaking change in 1.2 (bump from v1alpha1 to v1beta without a KGO version that would allow a smooth transition).
This requires a change in the chart which would allow setting args because only KGO 1.2+ supports configuration via env. https://github.com/Kong/charts/pull/1058 implements that.

Have you tested it with KGO EE? It seems it should work, but it's better to be sure 😉

Good point. I've created https://github.com/Kong/gateway-operator-enterprise/pull/137 to test this

Have you tested it with KGO EE? It seems it should work, but it's better to be sure 😉

Good point. I've created Kong/gateway-operator-enterprise#137 to test this

@programmer04 https://github.com/Kong/gateway-operator-enterprise/pull/137#issuecomment-2096470896


[WIP] Add an option to not use unsafe send and require answering pending prompt

Add an option to not use unsafe send and require answering pending prompt

I can't progress on this until https://github.com/crossbeam-rs/crossbeam/issues/1096 gets fixed


Border color only

This is a problem I've had with the original extension for a long time. I'd like the border of a line to be colored, but keep the original text color.
Right now the text becomes white when I turn the text color off with a border.
Can this version help there?

Hmm, this is an interesting one.  I've had a go myself and it seems to do the same: add the border OK, but changes the colour of the text within it to an unexpected colour.
Please can you post the "todohighlight.keywords": ... and `` parts of your configuration, and which keyword trigger you're using? And are you using a dark or a light theme?  I can then have a go.

Here's my full config:
  "todohighlight.isEnable": true,
  "todohighlight.keywords": [
    {
      "text": "NOTE:",
      "color": "#ff0000",
      "backgroundColor": "white",
      "overviewRulerColor": "grey"
    },
    {
      "text": "TEMP:",
      "color": "#c98436",
      "backgroundColor": "transparent",
      "border": "1px solid #ff8800",
      "isWholeLine": true,
      "overviewRulerColor": "#ff8800"
    },
    {
      "text": "TODO:",
      "color": "white",
      "backgroundColor": "#ff00ff",
      "overviewRulerColor": "#ff00ff"
    }
  ],
  "todohighlight.isCaseSensitive": false,
  "todohighlight.defaultStyle": {
    "color": "white",
    "backgroundColor": "#ff00ff",
    "overviewRulerColor": "pink",
    "borderRadius": "2px",
    "cursor": "pointer",
    "border": "none",
    "isWholeLine": false
  },
  "todohighlight.include": [
    "**/*.js",
    "**/*.jsx",
    "**/*.ts",
    "**/*.tsx",
    "**/*.html",
    "**/*.php",
    "**/*.css",
    "**/*.scss",
    "**/*.vue"
  ],
  "todohighlight.exclude": [
    "**/node_modules/**",
    "**/bower_components/**",
    "**/dist/**",
    "**/build/**",
    "**/.vscode/**",
    "**/.github/**",
    "**/_output/**",
    "**/*.min.*",
    "**/*.map",
    "**/.next/**"
  ],
  "todohighlight.maxFilesForSearch": 5120,
  "todohighlight.toggleURI": false,

The keyword in question is the TEMP: one. Now it does orange text & border but I'd like it to be only orange border but intellisensed text color.
I'm using the Dark+ theme.
Thanks!

OK, I think we're narrowing down the issue. I've realised there are three different ways text can be coloured here:

the label itself (e.g. "TEMP:")
any other text included in a regex term
the whole line, if isWholeLine is set to true.

I think you're hitting problems on type 3, which I can reproduce. Even if you take out the "color": "#c98436" from the keyword and the  "color": "white" from the defaultStyle, it doesn't render other intellisense colours, but makes all the text on the line blue, whatever the theme in use.
I've tracked down this blue in the code, I think to util.js line 26 in the definition of DEFAULT_STYLE, and also in extension.js.  Sadly, though, as I didn't write any of this code, and hardly know Javascript, I can't understand the main extension.js code, so can't fix this.

Thanks for trying! I have zero extension experience so I can't really help here

set color to empty string
    {
      "text": "TEMP:",
      "color": "",
      "backgroundColor": "transparent",
      "border": "1px solid #ff8800",
      "isWholeLine": true,
      "overviewRulerColor": "#ff8800"
    }

Thanks! That actually works perfect!


feat: drops geolocation

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #9460
[x] Requires deployment https://github.com/kodadot/private-workers/pull/78

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

Please note multiple cities are for testing purposes only (Denver, Jakarta)

Can we test?

Can we test?

sure, let me merge PR (and deploy) on private-workers first

@yangwao @prury ready to test, lmk if you need to change the location

First ok I guess

Second probably not as I'm in Denver and I see where is issue

when I'm at - https://whatismyipaddress.com/ip/<IP_ADDRESS>

it shows

@yangwao, what do you get if you visit this one: https://geolocation.kodadot.workers.dev/. We can use the value from that

I think region Colorado should be good?

We can use the value from that

Geolocation: Hello World!
You now have access to geolocation data about where your user is visiting from.

Colo: DFW

Country: US

City: Dillon

Continent: NA

Latitude: 39.59150

Longitude: -105.97860

PostalCode: 80435

MetroCode: 751

Region: Colorado

RegionCode: CO

Timezone: America/Denver

I think region Colorado should be good?

updated

using vpn:

@preschian i'm getting the error below on this PR when trying to mint, can you verify?

@yangwao
i know you are busy, but i ran out of KSM on Assethub on the testing account
would exchange with my personal account but i also ran out of assethub ksm there
address: CmWHiv7h4m9tEzKD94DH4mqwGTvsdYQe2nouWPF7ipmHpqA
i need this to test this PR and others

yup got this error again:

also:

yup got this error again:

also:

Could you compare it with canary? I think both of them are unrelated to the PR
sent the link for the image error. It can be safe to ignore if the image tag tries to load a non-image link (such as html/video)

@prury Oh, maybe the PR is not synced with the updated main branch. updated now

@prury Oh, maybe the PR is not synced with the updated main branch. updated now

all solved now ty

Thanks!
pay 80 usd

😍 Perfect, I’ve sent the payout
💵 $80 @ 8.34 USD/DOT ~ 9.592 $DOT
🧗 1xjvRADwdJcnmUCLWerEHR4iGCT5EBTm4D4fzLLg4LcAC9p
🔗 0xb8c9df67541f0306205ee1abb40a153326484088763323be82adb68a2730c0c7
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Create BlockPathRegex Middleware

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
Since golang and goregexp does not support lookaheads and readbehinds, issues come into play when we want to expose all except a specific path. For example, PathPrefix: /, but NOT /bacon.
There should be configurable response codes for the block match.
By having this coded in middlewares, it could be re-used, and shared between providers.

duplicate of #4812


Open telemetry

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕
If you write sample code, please follow the samples format.

@maitrimangal: Is there a reason this was closed without merging any of the changes? It would be awesome to see the addition of OpenTelemetry support within the Google PubSub java library.


Error in table

The table has two rows for LPCSTR. The first of these rows should be for LPSTR.

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

ID: be7907e0-45f6-2333-d4b0-09c2206f2917
Version Independent ID: ba9200fd-d49e-8d88-0d21-faa82db22c36
Content: Calling Native Functions from Managed Code
Content Source: docs/dotnet/calling-native-functions-from-managed-code.md
Product: visual-cpp
Technology: cpp-cli
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Thank you. Fixed.
The fix should be live by Monday 3/30


[0.5/dx12] essential fixes to depth views and command allocation

Fixes #2656
Tested on vange-rs and wgpu-rs examples
With these fixes, we run wgpu-rs stuff without validation errors, and terminate successfully :tada:

Thanks for the review!
bors r=msiglreith


Set x-ms-app HTTP request header in Monitor Query library

Library name
Azure.Monitor.Query
Please describe the feature.
The Azure Monitor Log Analytics service team stores the caller information for their REST APIs. To support tracking HTTP requests from the Azure Monitor Query library, send an x-ms-app request header. The header's value should be set to AzureMonitorQuerySDK. The metrics team uses the user agent to identify the caller. Send this header ONLY for logs queries.

After reviewing with the arch board, a decision was made to not make this change. As a workaround, the Draft service team can run a contains operation on the user agent string to identify the caller. This workaround has been communicated to Dan Hadari.


Dupe all doesnt quite work it only dupes the top bit

that means your ping is low. you can try ping spoof on a client or use a vpn to slow your internet down

This is a ping problem, sometimes this can happen under some conditions, I would recommend trying again as it is usually random.


Can't use " to access registers and can't type special characteres like ç ..

Hello, this Ui looks promising, but I can't access my registers or type ç,ã,õ... inside nvui

Hi! I need to diagnose this problem a little more, could you tell me your

Operating System
Keyboard Layout / Language

Also, by "can't access my registers", do you mean that it is not typing characters like the double quote (")? If so, when you try and type these characters what text is it displaying as?

my bad
OS: Linux
Keyboard conf:
❯ setxkbmap -query
rules:      evdev
model:      pc105
layout:     us
variant:    intl

Also, by "can't access my registers", do you mean that it is not typing characters like the double quote (")? If so, when you try and type these characters what text is it displaying as?

Nothing, it just ignores it
https://user-images.githubusercontent.com/5401584/131389495-c9294a26-d8fb-465a-bfc2-e6db6f904c3b.mp4

Hey, sorry for the late update. I think that #68 might fix your problem. Builds are here. Let me know if you are still experiencing issues.


command context menu doesn't work if terminal is dragged and dropped into the editor area

enable shell integration
run some commands
drag and drop the terminal in the editor area
click a decoration
no context menu 🐛

the inverse is also broken -

drag from editor area to panel
click a decoration
no context menu 🐛

The context menu is working for me now, but the alignment is not correct

original issue was a duplicate of #143703
Renaming to address the alignment issue


[REQUEST] current item of a progress

when creating progress bars for logical items (that are then supported with additional progress pars,
i would consider it helpful if it was possible to add a name/render able for the current item, and to push those in updates
i`m not yet sure how this is best expressed/implemented

Not sure I follow. You can add arbitrary fields in add_task, which could be renderables.

For example progress.track will not update a task with the current item, i can Elaborate more once back to the computer

More details please.

for example in the progress configured in https://github.com/pytest-dev/regendoc/blob/19cfd54ff9b5353a58b11a2dbafb73a3ba37fb5b/src/regendoc/init.py#L76-L82
i progress files in https://github.com/pytest-dev/regendoc/blob/19cfd54ff9b5353a58b11a2dbafb73a3ba37fb5b/src/regendoc/init.py#L113-L116
instead of just writing processing files, i'd like to add to the bar something that which which is the current file
for that having a task.current_item (which is set by progress.track) would be a nice help

You can add arbitrary fields like this:
progress.update(task_id, filename=name)

And you can refer to those in the columns with task.fields.filename.
Does that help?


Refactored multivariate normal example (trivial)

Use of dstack seems slightly more concise than declaring empty array with tuple concatenation, etc.

Merged, thank you!


[Feat] Advanced Entity Listing endpoint for TA

A request/response example
this feature MAY be implemented in spid_cie_oidc.authority.
GET /federation_adv_list HTTP/1.1
Host: registry.spid.gov.it

200 OK
Last-Modified: Mon, 17 Dec 2018 11:15:56 GMT
Content-Type: application/json

{
 "iss": "https://registry.spid.gov.it/",
 "iat":<PHONE_NUMBER>,
 "entities": [
   {
    "https://rp.example.it/spid/": {
       "iat":<PHONE_NUMBER>,
   },
   {
    "https://rp.another.it/spid/": {
       "iat":<PHONE_NUMBER>,
   },
   {
    "https://rp.it/spid/": {
       "iat":<PHONE_NUMBER>,
   },
   ... # many other entries
 ],
"page": 1,
"total_pages": 2,
"total_entries": 189,
"next_page_path": "/federation_adv_list/?page=2",
"prev_page_path": ""
}

Closed by
https://github.com/italia/spid-cie-oidc-django/pull/132


Related youtube videos

This is rather a question than an issue, I would be very thankful for any input.
I noticed that on your demo page you don't have any youtube's suggestions  - "More videos" when the video is paused. But since Youtube doesn't allow to remove related videos with rel=0 anymore, I am wondering what your secret is. Not finding anything related to it in the source code, I decided to ask here.

It seems to be simply the width of the player. I'm not sure what the exact breakpoint is, but the demo app is 480px wide and doesn't show suggestions. If I make it wider then it does.


Create INSTALL instructions

for python and cmake imstalls.

Instructions for:

python3 setup.py install
pip install
cmake
Add CI tests for cmake and setup.py installs


Where can I get VITON-HD 512?

Thank you for your great work.
I followed the tutorial and download the VITON-HD dataset from the link you provided (https://github.com/shadow2496/VITON-HD), however the dataset image size is 1024*768.
Your pretrained model is 512, could I know there to get the 512 dataset?

@kasim0226 how you solve this


queryScalar - wrong count of rows

What steps will reproduce the problem?
Having query with group by and limit:
$query = Products::find()->joinWith('units')->groupBy('id')->limit(10);

What is the expected result?
Total count of all rows
What do you get instead?
Total count of limited rows (10)
COUNT query is:
SELECT COUNT(*) FROM
(SELECT `products`.* FROM `products`
   LEFT JOIN `units` `u` ON `products`.`unit_id` = `u`.`id
   GROUP BY `products`.`id` LIMIT 1
) `c`

instead of
SELECT COUNT(*) FROM
(SELECT `products`.* FROM `products`
   LEFT JOIN `units` `u` ON `products`.`unit_id` = `u`.`id
   GROUP BY `products`.`id`
) `c`

Additional info
Related issue where code was created: https://github.com/yiisoft/yii2/issues/2490
I think you just need to execute query first and then re-assign select, limit and offset

This is expected result.
You should not set limit for the count query or unset it before like it is done at ActiveDataProvider:
https://github.com/yiisoft/yii2/blob/master/framework/data/ActiveDataProvider.php#L165

facepalm.jpg

In all places you need to do this routines instead of executing it in one point in framework
In framework code you still set limit and offset as null before creating command and re-assign it after creating. You do it for first part of condition

  if (empty($this->groupBy) && empty($this->having) && empty($this->union) && !$this->distinct) {
            return $command->queryScalar();

but I have no idea why you don't want to do the same for the second part.
Very low quality of code.