Helm should inject its version into the context dict

There currently is no way to check in a template with what version of Helm it is being rendered/installed. As a consequence, it is not possible to safely (without breaking certain users installations) use the lookup function, which was added in Helm 3.1 -- the apiVersion field in the chart metadata only allows marking a chart as "requiring 3.x", and there is no possible way to differentiate Helm 3.1 from Helm 3.0. Also, Go templates do not allow checking if a function is defined.
I understand reservations regarding having templates render differently depending on the version of Helm being used, regarding reproducibility and all. However, in that case the original sin was to change the template API by adding a builtin function without incrementing the required apiVersion of the chart. I also think that way more egregious best practice violations are possible in the current state, and strongly recommending against doing something is a better way than making it impossible. Checking the version could be done to accomplish something as harmless as printing a meaningful error message that this chart requires at least Helm 3.1, instead of erroring out with an obscure error that will require some googling by the user to find out what went wrong.
That ship obviously has sailed for lookup/3.0->3.1, but maybe lookup wasn't the last function to be added to the templating interface.

Helm 3.3.0 shipped with the ability to get the Helm version. The PR is at #7005.
Does that suffice?

Perfect, thank you and apologies for the noise - I had Helm 3.2 installed and didn't realize there was a more recent version. I'll close this.


Frontend With Framework - Javascript Expressions in JSX & ES6 Template Literals

NEED TO CONTAIN

How javascript expression are done using Jsx in React
ES6 Template Literals
Advantage of ES6 template literals

Apart from this any thing relavant about npm can be added.If you guys are making documentation try to include pictures of topic.It will be great if you can
You can take refer from

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Template_literals
https://reactjs.org/docs/introducing-jsx.html#:~:text=Embedding%20Expressions%20in%20JSX&text=You%20can%20put%20any%20valid,into%20an%20element.

Also make sure -
Avoid copy/paste, We will be checking.
Changes should be made inside the Frontend_Web_Development_React_Angular_Vue/ React inside Frontend_Web_Development_React_Angular_Vue branch.
Task will be assigned on first come first serve
Check out Contribution Guidelines
and Code Of Conduct

Can I work on this issue documentation

I would like to work on this

i would like to work on this

@CSAURABH I am assigning this work to you

@CSAURABH are you going to do documentation or video.

@anandhu720 Documentation.

@tanishqchugh2002 @amankumar11 guys if you guys are intrested in making video about it.I can assign you that work

Can I work on the video part of it?

@ShobhanSrivastava Sure,I will assign you this work


Possible bug when changing fan speed

mylesgray, I wanted to start a new topic for this so we can keep the last thread we were discussing it closed.
I was trying to reproduce this issue tonight and realized it might have to do with situations where I change the fan speed on the Blueair unit itself, and the Home app hasn’t updated/synced to the new state. I think when I do this, and then I go change the speed in HomeKit, things get squirrelly.
I usually get unexpected results going from the highest speed 3, to around 30-35% on the slider. The fan will sometimes physically switch to speed 1, and then jump back to 2 right away. It’s hard to say exactly what percent the slider is on but it’s close to 1/3, and that’s when I see this glitch.
As I was testing, if I have the Home app’s fan speed selection on the screen, (let’s say on low), then I change the speed to High by pressing the physical button on the Blueair unit twice, the Honekit GUI won’t update to reflect the new state. It seems the Home app only updates the status if I kill the entire app and reopen it.
I’m not 100% sure yet if the bug I’m experiencing is related to that or not, but I will play around with it more and see if I can reliable reproduce the issue every time. If I can, I’ll post my steps and take a video.
Otherwise it’s really not a big deal as I have to screw around with it outside of a normal way of using it to get it to happen. I think in normal, everyday use, this fan speed jump I am seeing is unlikely to happen.
In any case, I think the issue could stem from the mystery state of the slider switch on the Home app. Because it’s a variable switch and you cannot see a percentage and sort of have to guess, I just can’t tell if I’m misinterpreting something, or if the logic is getting confused.
It would actually be really nice from a user interface perspective if there was a UI switch for a 3 speed selector, instead of a 0-100% sliding switch (intended for a dimmer switch). If there was something like this with 4 hard states (Off, 1, 2 and 3), I think this could solve my issue and possibly simplify the programming logic. Does HomeKit have any configuration settings for this sort of switch with multiple hard states?
I wonder what the HomeKit-enabled Hunter ceiling fans do, since most fans have low, medium, high and off?
Food for thought.

I have seen this behaviour myself actually now you have described it - this is related to #3 - currently the plugin will only poll the device for changed values on the hardware when the app is closed and opened - there is no "push" mechanism available from BlueAir's API that would allow me to automatically change values in the app based on your inputs to the physical device.
As such it will need to be a polling "pull" type method where I will poll the BlueAir API for changes every n seconds or such - however, I don't want to spam the API needlessly.
In addition, some investigation needs to be done about the different methods Homebridge can use to set values of characteristics as in the above issue.
On your question of the 0-100% vs 0, 1, 2 and 3 settings - I agree and that is what I originally wanted to do, however the HomeKit spec doesn't allow for those arbitrary levels and instead only allows me to specify a percentage - as such I had to write some logic to convert the percentage into the 0, 1, 2 and 3 levels the devices supports. See here:
https://github.com/mylesgray/homebridge-blueair/blob/791504d8c21345d037b68ec0c2deeb918c3803df/index.js#L916-L940
So if you want level 1 - anywhere between 1-33%, level 2 is between 34-66% and level 3 is 67-100%.
Happy to be told if there is another way to achieve this - looking at the Fan and Fan v2 services they still use Characteristic.RotationSpeed which is a percentage slider:
https://github.com/KhaosT/HAP-NodeJS/blob/9eaea6df40811ccc71664a1ab0c13736e759dac7/lib/gen/HomeKitTypes.js#L2881-L2919

I did a little looking and found that out of the box HomeKit ceiling fans (like Hunter) also use a 0-100% sliding switch, so that answers that. https://www.google.com/amp/s/9to5mac.com/2017/11/20/review-hunter-simpleconnect-ceiling-fan-homekit/amp/
Just floating another idea - It might not make sense, but what about using thermostat “Mode” state buttons like the Homebridge-Nest plugin uses (Off, Heat, Cool, Auto)? There are 4 buttons, and assuming they can be renamed, could you assign each button to one of the Blueair levels?

Good idea, but the mode state like you get on the heater accessory types aren't supported on the air purifier type we are using - you are also not able to change the labels on them I'm afraid.
The only characteristics I can attach to the air purifier type are here, and as far as I can see - none really make that easier:
https://github.com/KhaosT/HAP-NodeJS/blob/9eaea6df40811ccc71664a1ab0c13736e759dac7/lib/gen/HomeKitTypes.js#L2673-L2692
I myself have just adjusted to setting the fan to 25%-ish if I want level 1, 50%-ish if I want level 2 and 100% if I want level 3.

Got it. Yeah I’m starting to realize that and have also been trying to adjust the slider to somewhere that is visually clear you are in the correct range.
It would be nice if Apple’s UI would just tell you the percentage of wherever the slider lands. I’m sure they will make improvements over time though and offer some better options as new Homekit devices come out.

I think it actually does that on the Mojave version of the Home app, maybe
on iPad too?
Still, I think we are doing pretty alright with how it is working at the
moment :)
On Mon, 26 Nov 2018 at 17:09, ndlaham<EMAIL_ADDRESS>wrote:

Got it. Yeah I’m starting to realize that and have also been trying to
adjust the slider to somewhere that is visually clear you are in the
correct range.
It would be nice if Apple’s UI would just tell you the percentage of
wherever the slider lands. I’m sure they will make improvements over time
though and offer some better options as new Homekit devices come out.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/mylesgray/homebridge-blueair/issues/16#issuecomment-441718824,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAlMTCfLhBmFSJF1BJI3k1HwL-nVgeUXks5uzCBEgaJpZM4Yxti1
.


expose the declaredContents in the context for fixes and checks

i.e., give the user access to the same data that we have when defining checks and fixes;
this will also allow us to modularize the checks better, since we can take them outside of the declaration scope -> i.e., make our default/automatic check+fix functions no different from user defined ones

done now


Reduce TargetSizeRatio from .5 to .49 to avoid HEALTH_WARN

Currently, due to a floating point imprecision in ceph, we get a
"HEALTH_WARN 1 subtrees have overcommitted pool target_size_ratio"
after deplyoing with the TargetSizeRatio of .5 (0.5 + 0.5 > 1.0..).
This will be fixed in future Ceph versions
(https://github.com/ceph/ceph/pull/33035).
In order to avoid the warning until we have the updated Ceph,
this patch lowers the ratio to 0.49.
https://bugzilla.redhat.com/show_bug.cgi?id=1807950
Signed-off-by: Michael Adam<EMAIL_ADDRESS>

/retest ocs-operator-e2e-aws

/test ocs-operator-e2e-aws

/test ocs-operator-e2e-aws

Putting this on hold to reduce load on the CI cluster while #431 merges.
/hold

/hold cancel
/test ocs-operator-e2e-aws

/lgtm


Cesar/small changes

This branch/PR does two small things:

Instead of using two functions that handle the reduce process, now we have just one.
We have updated the documentation regarding the changes we applied to the dispatchers, that are now just simple functions.

LGTM ;-)


predict H chain only?

Dear developer,
How to model one chain only, for example for nanobody? Thanks!
pred = CDR_Predictor(input_path, chains = ("H", "")) #, "L"
It gave me an error:
res = loop[i].split()[5]

IndexError: list index out of range

Hi @QUEST2179
Sorry but ABlooper does not currently support single chain modelling.


Expose the intermediate schemas

Since the schemas can easily be quite convoluted (with a lot of oneOf in particular), it's hard for Nickel to output good error messages when things go wrong, and easy for the LSP to get lost.
To mitigate that, it would be nice to be able to add some contract annotations to intermediate nodes in a config. However, json-schema-to-nickel only exposes the toplevel schema, and there's no way to access the sub-schemas.
As a concrete example: I'd like to annotate this snippet to tell Nickel that it's of type normalJob, but this isn't exposed by the generated code.
What I would like
Expose all the items from the definitions or $defs field of the schema, so that they can be accessed from client-code.

So, if I understand correctly, you want to turn
let internal_definitions = ...defns... in
...schema def...

to something like
{
  definitions = ...defns...,
  Schema = ...schema def...
}

?

I think so, yes

Mh, good question :thinking:
Maybe a CLI flag would make sense, indeed. I guess the most common cases won't care about that


Is there any way we can have Elastic Search integrated with it.

I liked the SQLDatabase integration. If you can let me know what changes I need to do for the elastic search?

There are docs on how to use Elasticsearch as your vector store here:
https://langchain.readthedocs.io/en/latest/modules/indexes/vectorstore_examples/elasticsearch.html
The source code for this is here:
https://github.com/hwchase17/langchain/blob/master/langchain/vectorstores/elastic_vector_search.py
Is this what you're looking for?

I am having all my data in my Elastic Search. When the user gives some text I need it to query from Elastic Search and return all the outputs. I don't need it to summarize the output.

@Rishav-hub did you find any answer?

@Rishav-hub did you find any answer?

@Chirag313 No we have changed the approach, we are not using elastic anymore. Rather than using Weaviate Vector store our vector embeddings.

@Rishav-hub Can you tell me how did that approach pan out?

@Chirag313 Something that would be cool is having a tool (or documentation about) how to handle the "text to elasticsearch DSL" flow.
So for example the input is : "how many apples are sold in average"
Step1: turn NLP into ES query aggregation
Step2: run the ES query and return the aggregations
Step3: Find the answer in the aggregations and return the answer in human language
I wonder if this is possible using chains/tools and how.
Thanks!


Documentation dark mode renders difficult to read images with transparent backgrounds

Describe the bug
The documentation contains images with transparent backgrounds, which can contain (black) text. Using dark mode can make it difficult to read these.
See for example: https://docs.zephyrproject.org/3.3.0/connectivity/networking/net-stack-architecture.html#network-data-flow
Expected behavior
Text embedded in images should always be readable.
Impact
Annoyance

Thanks for reporting this. I'll work on a fix.
FWIW the images on this particular page probably need to be reworked independently of the current fix. The issue is not that they have a transparent background per se, but rather that some of the main elements on them have some level of transparency when they shouldn't. See e.g.


Moves stylelint to peerDependency

closes #67
5f39bc7f34c96f0b9078c2aedf09a15d6d987c94 - removed package-lock.json - yarn.lock was more recent and package.json defines yarn in the volta config
478b5989a6a1a1ae0017e950e61fb58a896a243f - moved styelint to peerDependencies (to fix #67) and devDependencies (to make tests pass)
ed82ef6276a9d8e815a6980d2852a5f57ec0871a - bumps the package.json version - as this is a potentially breaking change. I could not tell if you have another means to automate versioning and publication, so I can remove this commit if you prefer

Thanks @bmuenzenmeyer, this looks good. In terms of the package.json bump, this repo doesn't have any automation on that. Usually just a manual git tag and npm publish.
However I would say that we can delete that commit and publish the 4.0.0 once stylelint has been update to 16.x as per your issue #67

@simonsmith I've dropped the commit that incremented the package.json - this still makes https://github.com/simonsmith/stylelint-selector-bem-pattern/commit/478b5989a6a1a1ae0017e950e61fb58a896a243f a BREAKING change because a user expecting stylelint to come along for the ride will no longer have it if not installed directly. I'll leave the timing to you.

Hey, 4.0.0 going to be published soon?

Apologies, I totally forgot.
https://www.npmjs.com/package/stylelint-selector-bem-pattern/v/4.0.0
I also took the liberty of updating the package to support stylelint 16.x and their move to ESM. Let me know if there are any issues as I'm not actively using this package, I just ensured tests passed 👍🏻


Help users implement the ForwardDiff overload

Thanks. Let me add some docs explaining how to implement the interface, and we can iterate on this!

I have rewritten the docs. Perhaps we could add a concrete example on how to implement the interface for a custom type?

Let's just move this PR to the docs instead of the docstring?

Yes, let's do that. We could have a small section showing how to implement the entire interface for a custom type. I suspect that could be useful, since as we have seen the simple bounds by interval arithmetic seem to become impractical when the function starts to get complicated (e.g. determinants, eigenvalues, etc).

Done


I wonder if there is a way to write this with 3 or less selectors.

https://github.com/rilwis/bamboo/blob/6311e4f486f8b607e5aab6937aa08a99826876b7/src/core.scss#L256
It appears the purpose of this rule is to make input boxes full width if a size is not specified, am I correct?
This rule gets flagged by yellow labs tools for using too many selectors. (anything with more than 3 selectors gets flagged)
I checked modern normalize and sanitize and did not see this rule there, or I would have opened the issue upstream.
Thanks for bamboo!

That's right. The selector targets all inputs except button, submit, reset, checkbox & radio. We set these inputs 100% width by default. It's opinionated and is not in normalize.css or sanitize.css.
Without this selector, we can achieve similar things by writing other input types explicitly, like text, number, search, datetime, date, etc. which I think is longer.

ah ok, thank you. I will try by setting text, number, search, datetime, date with those.
I can use a comma instead of a string of :not so it should not count as so many selectors it may be a little larger of a file but would be under 3 selectors.

After thinking more about this, I think selecting inputs explicitly is better than nagatively ignore some of them. I've added it in a recent commit.


feat: finish off implementation ElasticsearchIntegrationTestExtension

#17
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[x] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[x] Docs related to the changes have been added/updated (if applicable)

Ping


Fixed missing env vars in some test workflow steps

#4878  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on  Graphite


Terminal echo is disabled after building (with zig!?)

After merging #986 so that we're building acton using zig, it seems that my terminal is getting messed up, like I have no echo, and I have to run reset to fix it. I think it is related to zig?
Find cause and fix!

I haven't seen this in quite a while. Going to assume it's fixed. Closing.


Questions: iOS SDK v2 and App Store Server Notifications v2

Hey!
Just 2 quick questions:

I see that there is a v2 version of iOS SDK (and corresponding docs), but that's not the case for the RN one yet. Do you know when it'll be available?
App Store Server Notifications v2 support was supposed to arrive by the end of Q3 2022 (as it says here https://docs.adapty.io/docs/app-store-server-notifications). Is there a new estimation?

Thank you!

Hey, @amberv0! We're already actively working on updates to our SDKs for Flutter, Unity, and React Native, so they'll be rolling out very soon.
As for App Store Server Notifications v2, the team says that this functionality will be available before the end of the year.


Harden parsing of messages from pipe

This bug presents itself when you stage a region in VS Code.
Open a GVFS Repository with VS Code
Make some changes to a file
Select a region of the file in the editor, right click and “Stage selected range”
This is the command run:
/.git/hooks/pre-command.exe update-index --cacheinfo 100644 '7e012a87091a0f76de43fcbd83098b3611791643
' Scripts/ReinstallGVFS.bat --git-pid=20360

Notice the newline in the command.
The InMount process does not like this, and crashes with something similar to (this doesn't necessarily correspond to the instance above):
The parsedCommand is an unexpected length, got: 123 from message: '7580|False|False|123|git update-index --cacheinfo 100644 f7ce833c943b1d4e33fce3c842236c9c044561c9'\r\n

If this is a valid format for a git command, then VFS for Git should be able to parse it.

I believe the root cause of the issue in VFSForGit is that it uses the newline character to separate messages on the pipe (and uses ReadLine to read in new messages).
The hook process is not stripping the newline before passing the command to VFSForGit, resulting in it only reading in a part of the message.

I don't know of one specifically but there could be a situation where stripping the newline out will cause other/different problems. In general, we've had to use a nul to indicate EOL (and a double nul to indicate the end of the list) when communicating with git and other hooks (see the read object hook for an example).


Wrong detection by "organization_has_too_many_admins" check

TL;DR
We have 2 admin users in our GitHub organization but legitify alerts on that as if we have too many admins.
Expected behavior
I would expect that 3 admins and above will be considered too many.
Observed behavior
No response
Version
0.2.6
On which operating system are you using legitify?
Mac OS
Relevant log output
No response
Additional information
No response

Hi @marksarka, thanks for raising this issue!
It's fixed in v0.2.7, please reopen this issue if you see otherwise.


Override * in magit-status-mode-map

* is bound to magit-worktree in magit-dispatch, but that requires opening magit-dispatch before interacting with Git worktrees (e.g. by pressing ?). With this change, * is available anywhere in the magit-status buffer.
The downside is that it can't be used to evil-ex-search-word-forward in magit-status anymore.

Thanks for the PR, but since ? * exists I'd rather not sacrifice evil-ex-search-word-forward. I'll leave it to users to make this choise.


Storage account inaccessible

Common issues that block a Functions access to a storage account are:
The storage account firewall is enabled and function app is provisioned in consumption plan. This is expected behavior as per product design.

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

ID: 75028922-3176-8802-9f85-62c92c381784
Version Independent ID: 1e1c0232-aee5-59c9-fc1e-f8d7a8cf73a8
Content: How to troubleshoot Azure Functions Runtime is unreachable.
Content Source: articles/azure-functions/functions-recover-storage-account.md
Service: azure-functions
GitHub Login: @alexkarcher-msft
Microsoft Alias: alkarche

Hi @pvvdurgaprasad - Thank you for sharing feedback and this insight! Surely, it'll benefit & help others visiting this thread seeking clarity on that scenario. Although I believe this documentation does cover it with: "The storage account firewall is enabled and not configured to allow traffic to and from Functions."

@pvvdurgaprasad We will now proceed to close this thread for now. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

Hi ...I am facing the same issue. Are you able to make function app on non consumption plan ("App Service plan") connect to a storage account? Is there any work around?

Abid, following options seem to be available for function apps:

with consumption plan, you can have storage account in another region for IP whitelisting to work on storage firewall
with app service plan, VNet integration is available to communicate with storage firewall in same region

Thanks for your prompt response Prasad!

Consumption plan with storage account in another region didnt work for me. However, I couldnt find a way to whitelist the IP. Any clues?
2.I did a "vnet integration" of the function app into the same vnet as the storage account - both function app and storage account in the same region, but in two different RGs. It also didnt work.

Any further suggestions?

Hi @abidashaik
It takes up to 12 hours for the storage account to whitelist a virtual network. We just tested this scenario yesterday, so let me know if it begins working after leaving the firewall configured for 12 hours.
In on of my tests it started working in ~30 minutes. In another test it took about 11 hours.
We're working on our side to fix whatever is making this configuration change take so long.

Thanks for your prompt response Prasad!

Consumption plan with storage account in another region didnt work for me. However, I couldnt find a way to whitelist the IP. Any clues?
2.I did a "vnet integration" of the function app into the same vnet as the storage account - both function app and storage account in the same region, but in two different RGs. It also didnt work.

Any further suggestions?

I have verified option 1 recently, it should work.
You will need to copy outbound IP addresses from Function app properties and whitelist all of them in Storage account firewall.
Were you able to do the same.

Thanks for all your inputs .....I was able to make function app talk to a secure storage account by adding it into a subnet of the Vnet, which the storage account is also part of. Both Function App and storage account are part of different Resource Groups.


[Set Audio Device] AirPods missing from device list when connected to other device

Extension
https://www.raycast.com/benvp/audio-device
Description
When my AirPods are connected to another devices, they do not show up in the list of potential audio sources:

But the macOS sound menu shows them, along with other AirPlay devices:

Once I connect them through that menu, they show up in Raycast:

Steps To Reproduce

Connect AirPods to another device
Observe that they are not visible in Raycast menu
Connect them via the macOS sound menu
Observe that they now show up in the Raycast menu

Current Behaviour
AirPods only show up when they are connected to this device specifically.
Expected Behaviour
AirPods should show up when connected to other devices, like they do in the macOS sound menu.

Seems like the AppleScript to get airplay devices is broken on the latest macOS versions. I found this which you should be able to adapt in its place: https://github.com/deweller/switchaudio-osx/issues/9#issuecomment-1520901840

Issue persists

Not stale


Add the HAM hideout trapdoor pick-lock option to the menu entry swapper

It'd be nice for grinding easy clues.

I tried to solve this by adding a swap to MenuEntrySwapperPlugin.java, but I found that the client forcibly orders options of type 3 (open's type) over options of type 1001 (pick-lock's type), so using client.setMenuEntries() here does not work. I am not sure if this is doable with the current API.

I looked at it as well and came to the same conclusion. The current client doesn't allow swapping those 2 specific types.

added in 3fd1ba69500b3cd14ece4e1163e9e7a428d10935


✨ First draft of image repo listing utility

Summary
This PR adds a utility in hack/ for fetching the manifests for all the tags of a given image (repository). This is better than curl or wget of just the first page of tags, and takes the extra step of fetching the associated manifests.
Related issue(s)
Fixes #

This is for use by contributors, which is why I put it in hack/.  It is useful to understand the provenance of images. You may have noticed that the quay.io web UI often ignores requests to go beyond the first page of tags.


Only send 1 tcp packet per message,  saving huge bandwidth

Previously,  every message caused 2 tcp packets to be sent:  one for the header and one for the payload.
This is the header packet:

this is the body packet:

With this pull request,  these 2 packets are combined into one.  This is the result:

Sending 1 message:
before required a total of 68 + 114 = 182 bytes
after requires a total of 116 bytes
This pull request saves 36% bandwidth for 48 byte messages.

Codecov Report

Merging #16 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   77.08%   77.08%
=======================================
  Files           9        9
  Lines         288      288
  Branches       17       17
=======================================
  Hits          222      222
  Misses         56       56
  Partials       10       10

Impacted Files
Coverage Δ

Telepathy/Common.cs
64.47% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4028156...c091fe6. Read the comment docs.


Ft rename readme 184724754

What does this PR do?
This PR is contain more details we should have in our project backend
Description of Task to be completed?

update readme where all team members can status.

How should this be manually tested?

Clone this repo git clone https://github.com/atlp-rwanda/e-comm-team-amigos-bn
Switch your working directory cd e-comm-team-amigos-bn
Create an .env file and update it with .env-example
Install all the dependencies by running npm install
Then run the project by typing npm start

Any background context you want to provide?

Have Node Js installed in your local machine

What are the relevant pivotal tracker/Trello stories?
Story ID: #184724754
Screenshots (if appropriate)
Questions:
N/A

Make your Story ID a hyperLink of your pivotalTracker

This PR has a failing test


reducer exception shown in console

Error thrown in reducer is consumed and only message is shown. This PR adds call to console.error to see a stack trace.

:+1: sorry I left that out!

Should be released in 0.1.3.


A3.06 TNS Kafka Stream

For github original see https://github.com/lsst-uk/lasair/projects/19

At the 28/APR/21 Lasair internal review it was agreed to close this activity.


Vertical ?!

Can I use the animations for vertical scroll  ( I need "Page" animation ) ??

You can set the layout.scrollDirection = .vertical. Let me know if you have more questions.


DCOS-43529: [1.13] Debug deployment time is not correct

Show time since last configuration when a service cannot deploy.
Closes https://jira.mesosphere.com/browse/DCOS-43529
Parent jira: https://jira.mesosphere.com/browse/DCOS-40674
Testing

Start a single-container service.
Wait a few minutes.
Edit it so that it can't start (for example by setting a huge memory).
Wait 5 more minutes.
Verify that the warning message in the Debug tab of the service calculates time since last configuration update, not since the service was created.
Verify that the tooltip for this service in the services table is also using the correct time.
Test the same for multi-container service.

Trade-offs

I noticed that for multi-container services, the configuration displayed is being updated every 2 seconds.

Not sure if this is intended. I still used the time it was last configured for the message. @leemunroe you created the jira, can you tell me if my assumption is correct?

Scale or restart actions don't reset the timer. Again, not sure if this is intended.

Dependencies
None.
Screenshots
Before

After

@GeorgiSTodorov Nice work.

I noticed that for multi-container services, the configuration displayed is being updated every 2 seconds.

This seems like a bug. We should check with Marathon team to see what is going on here for pods. Maybe @nLight knows.

Looking into failing test.

/jenkins/workspace/_dcos-ui-ee-pipeline_PR-854-TKXLFOPQK4GKXQKOWVWLPI4JNS3NVSIWOVFBQGGPFOSC7OUXL2JA/dcos-ui-ee/dcos-ui-plugins-private/secrets/components/SecretActionsModal.js

  87:1  error  Expected indentation of 2 spaces but found 0  indent

This i a rather tricky one. There still could be something in Marathon's queue that it couldn't schedule. We are operating of this information when showing the message. It's been reported to be rather confusing, yes.
Now imagine a scenario: you tried to deploy this huge service for a while so that Marathon ramped up the backoff timeout. Then you change the config and redeploy. Deploy went through, but we still show the indication that there's a stale deployment because the older one is still in the queue.
Related Jira https://jira.mesosphere.com/browse/DCOS-43443
I'm afraid we can't change one without changing another and that would be a bigger change. @leemunroe we may want to have a tab for showing the queue in an explicit way with the button to delete a certain queue item.
I'm saying that this experience should be designed, since resolving one confusion can add more confusion in another place.

@nLight thank you for commenting, let's see what design team has to say.

@nLight @GeorgiSTodorov is this blocked by the linked JIRA? If so we should close this PR and put the JIRA to blocked 😞(We're not going to implement the necessary feature this or next RI…)

I think that it is not blocked. It is a bug fix, that we may not want to merge, because it potentially breaks something else. I guess we are waiting for @leemunroe to say if we want to close this jira.


Require devices information connected to network coordinator  (TZ-749)

Question
I have been exploring the zigbee stack, its behavior in the relationship between coordinator and devices connected to the zigbee network such as routers and end devices. However, for my network coordinator application, I need to obtain information about the devices that are authenticated in the ZigBee network and I cannot find any function in the API that does this.
Can you tell me how I can do this? The objective will be to obtain a list of all network addresses, MAC addresses, capabilities and other information from devices connected to the network.
Best regards
nalves23
Additional context.
No response

@nalves23 ,
The esp-zigbee-sdk currently does not offer the aforementioned support; however, we will consider implementing this feature in the future.

Thankyou @xieqinan.
I will implement in FAT or SPIFFS memory the association devices table and add information of device when it join the network and remove it when they are removed.
Each storage type that you suggest to store this information?
Kind regards
nalves23

Hello @nalves23

Can you tell me how I can do this? The objective will be to obtain a list of all network addresses, MAC addresses, capabilities and other information from devices connected to the network.

The esp-zigbee-sdk v1.3.2 provides APIs to retrieve information about network connections. Please refer to this link. Could you please update the libraries and test it? The issue can be reopened if you encounter any problems with these APIs.


Integrate rocksDB for better runtime performance

Raised https://github.com/UrbanCompass/vinyl/pull/3 for fixing this issue.


[Question]: A question about testing new contributed packages

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I want to rewrite the searchain framework using llama_index. Currently I have completed the source code and am preparing to write test cases. I found that the test cases for other similar packages (such as self-rag) seem to have classes imported directly from the official llama_index package (such as SelfRAGPack). However, I did not submit a pull request, so the searchain framework I wrote must not be included in the official llama_index package. This seems to form a logical closed loop?

（Test case in self-rag, it worked! ! !）

(When I want to write test cases about searchain locally, I fail...)

Instead of testing download_llama_pack, just assume that will work.
Rather, you can just pip install -e <path to root of my package> to install the package (the path should point to the folder where the pyproject.toml is for your pack), and from there you can import it
from llama_index.packs.SearChainPack import SearChainPack or something like that
Then, you can continue on with testing


client GRPC and client.Retries(5) will cause timeouts ?

Hello,
import (
	"github.com/micro/go-micro"
	"github.com/micro/go-grpc"
	"github.com/micro/go-micro/selector/cache"
	clientGRPC "github.com/micro/go-plugins/client/grpc"
)

...

	clientGRPClocal := clientGRPC.NewClient( client.PoolSize(10), client.Selector(cache.NewSelector(cache.TTL(time.Second))), client.Retries(5) )

		VS

	clientGRPClocal := clientGRPC.NewClient( client.PoolSize(10), client.Selector(cache.NewSelector(cache.TTL(time.Second))) )

in first case (when set client.Retries(5)) we catch timeouts (default 5 sec) at 100% of the requests,
at second case (without client.Retries(5)) every thing works as should be.
I can not understand what went wrong and why.
is it our fault or bug in go-micro?

Client retries dictates the number of retry attempts, this is separate from the overall request timeout. There's also an exponential backoff mechanism built in which is why 5 attempts is not the same as 5 seconds. A client should retry until the request timeout expires but we shouldn't do this at the same interval because multiple clients may attempt this, instead a randomly jittered and exponential backoff is required. So client.Retries(5) may not yield 5 retry attempts in 5 seconds.

yes, but service to which we trying to connect is fully available. in case of without client.Retries(5) grpc client works fine, but in case of client.Retries(5) there will be timeout in any request.
steps to reproduce:

prepare grpc client with client.Retries
make request and there will be timeout
prepare grpc client without client.Retries
make request
it works fine

Yes you are right, it's my fault. we are using http 500 code in case of some business exceptions instead of http 200 code


Add a bit realistic parameters to CCID descriptor

Certain CCID drivers (esp. on Windows) will check the descriptor for valid parameters (clock, modes, protocol handling, features, ...) and deny access if the calculated results are out of spec (ISO7816-3).

These feel too specific for TamaGo, I will move them to our main application which uses these (GoKey) to promote application set values.


Extend entry support

Add a basic support of object and array for entry property
accordingly with
https://github.com/GoogleChromeLabs/prerender-loader/issues/3#issuecomment-393671513

Looks good! Before we merge this, I want to double-check that there isn't already a way to handle this via Webpack (SingleEntryPlugin vs MultiEntryPlugin). Otherwise I think this will work great!

From what I see, MultiEntryPlugin works the same way as Single, but we can pass an array there. There is also another like EntryPoint plugin, but I'm not sure how to use it because I'm not familiar how to apply it(constructor doesn't accept arguments and pligin relies only on event), but it handles all type of entry (array, object and string)

I meant https://github.com/webpack/webpack/blob/master/lib/EntryOptionPlugin.js

Does this commit will fix the bug that entry  is an object ?

@vaxilicaihouxian I hope so. https://github.com/GoogleChromeLabs/prerender-loader/pull/8/files#diff-c193db11b3837e38bd24c06b98cb93ffR38 this line uses entry as an object

@havenchyk  But it seems that this commit has not been merge to the master branch.How can I use it?Pull from your fork branch?

Sorry for the delay! I have a project setup to test this easily now.


Whisk object is deprecated going forward replave with NPM

What the title says.
See both:
https://www.npmjs.com/package/openwhisk
https://medium.com/openwhisk/updates-to-openwhisk-node-js-actions-ed5556cd5ae9#.xtcrclic4
For how to change to the OpenWhisk NPM package and promises instead of using the whisk object.

I'll try to integrate it with the latest OpenWhisk promise.

close as fixed


Getting a CORS error in the response.

I'm trying to use this module to build a serverless app and stuck at a point where I can't seem to get the response to have CORS header for a GET request.
I tried adding the header manually but doesn't work. Adding the Lambda handler code below.
exports.handler = (event, context) => {
if (event.headers) {
event.headers["Access-Control-Allow-Origin"] = "*";
event.headers["Access-Control-Expose-Headers"] = "Origin";
}
awsServerlessExpress.proxy(server, event, context);
};
In the express app, I have added below,
app.use((req, res, next) => {
res.header("Access-Control-Allow-Origin", "*");
res.header("Access-Control-Allow-Methods", "GET, PUT, POST, DELETE, OPTIONS");
res.header("Access-Control-Allow-Headers", "Origin, X-Requested-With," +
" Content-Type, Accept");
next();
});
Also, I'm using serverless for deploying the app and have enabled cors in the serverless.yml config file.
Does anyone know of a solution to this problem?

You need to configure on API Gateway enabling CORS. This code just works on Lambda.

Nothing obviously wrong with that. Have you tried using the Express cors middleware instead?

Thanks! It works with the cors middleware. Wondering why it wouldn't work with the above.

Not sure but I'm glad it's working now!


Sizes of tensors must match except in dimension 0. Expected size 7 but got size 8 for tensor number 2 in the list.

Hi @pringwong could you please provide some more context on the issue - specifically, what script you are trying to run as well as the versions of transformers and pytorch that you are using?


adding another ferrari (bburago) models

added names of another ferrari models (bburago brand) to be able to control cars from the newest collection

Hi. I added name of one model from first Ferrari collection (488 GTE) and names of all models from second one.

Btw. Cars from BMW M Motorsport collection are BrandBase (QCAR).

Thanks for PR. I released new version https://github.com/tmk907/RacingCarsController/releases


Uploads should show project name, not just date of upload

A user pointed out that it can be hard to find the right uploaded projects when trying to add multiple to a project--while our naming convention (source + date of acquisition) makes sense for satellites (Landsat, Sentinel, etc.) it doesn't make sense for custom uploads that are most recognizable by their name, not their date of upload.

There was a little bit of discussion of this in-person on Friday that I think resulted in the following decision:

Display Name again
Keep the creation date visible
When a user creates a project with imports:

if there is a single import, the created scene name should equal the project name
if there are multiple imports they should follow the format of <project name> - <sequence>


Make urbit vera 3.0 be the latest on nixos

Make Urbit Vera 3.0 work with NixOS
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Did I do this right? I have never contributed before to Nix

Also you should try to check some of the boxes in the pull request template. Some easy ones are: were you able to build it (and on what platform)? Does the compiled binary work? Did you follow the contributing guidelines? Etc.

Result of nixpkgs-review pr 310890 run on x86_64-linux 1

1 package failed to build:

urbit

Looks like you got the hash for x86_64-linux wrong:
error: hash mismatch in fixed-output derivation '/nix/store/qvynrkgbwmg626v2lqq3cb1brbix5cb8-source.drv':
         specified: sha256-wQY3I/HM816vI2NYAvtaYDga64n3nkUEtNd6G8w3hgU=
            got:    sha256-ip35d9YgwFEkNb+1h+8WYBgUm+QlckvHhlAT69TpeYg=

Those were the hashes I got when I downloaded the zip. What is the best way to get the hash correctly for Nix? I'll go back and I'll do it the right way

Looks like you got the hash for x86_64-linux wrong:
error: hash mismatch in fixed-output derivation '/nix/store/qvynrkgbwmg626v2lqq3cb1brbix5cb8-source.drv':
         specified: sha256-wQY3I/HM816vI2NYAvtaYDga64n3nkUEtNd6G8w3hgU=
            got:    sha256-ip35d9YgwFEkNb+1h+8WYBgUm+QlckvHhlAT69TpeYg=

Those were the hashes I got when I downloaded the zip. @pluiedev What is the best way to get the hash correctly for Nix? I'll go back and I'll do it the right way. I am very new to this language..

There's a bunch of different options, in order of convenience (IMO):

Using nurl tool to prefetch the hash. This works great for things like fetchFromGitHub where it would essentially give you exactly how src should look like, but at the moment it doesn't support fetchzip, so you can't use this in this specific case.
Using nix-prefetch, which is significantly clunkier to use IMO (I still haven't figured out how to use it myself) but it does support a lot more fetchers than nurl.
Using the built in nix-prefetch-url, which is simple to use but quite annoying as it outputs a base-32 hash instead of the more modern base-64 hash, and there's quite literally no way to change that.
And finally, not prefetching at all, and just setting the hash to an empty string, lib.fakeHash, lib.fakeSha256 or sha256-AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA=. All of them are equivalent and just supply a fake hash for Nix to validate against. Nix would then tell you the right hash as it tries to build the package, which you then can paste into the expression.

https://github.com/NixOS/nixpkgs/pull/295278


feat: add cad dashboard playground

adds starter playground for cad dashboard

@ninabauer  I think you have to put this into the deploy/ folder and then re-run make generate to create the openshift template file for it.

/retest

/retest

/retest

/lgtm

/meow


Change detection: Landsat toolbox tools need tool decription

The following tools need tool description

ACCA Cloud Mask
Landsat 7 Gap Filler
LTL Cloud Mask

Tool description written for ACCA cloud mask. Landsat 7 gap Filler and LTK Cloud Mask taken out of project as not referenced in task.

Tools in the pro project are still missing tool documentation.  See dialogs for each tool.
Example 1.

Example2.

This issue was moved to ArcGIS/solutions-templates#19


Processing homepages of news sites.

Hi again @adbar!
Again cheers for a great lib :)
I have been struggling to get any meaningful results from the library for pages such as: https://www.yorkshirepost.co.uk/
Where we mainly have links to other pages, is this the desired and expected behaviour?
Thanks!

Hi @ydennisy, thanks!
Yes, this is expected since there is no main text to be found, not even
loose -elements. The extractors target actual article pages, e.g.
https://www.yorkshirepost.co.uk/news/people/biker-dies-in-collision-with-estate-car-near-ferrybridge-power-station-3201739

Thanks @adbar do you have any suggestions on how one may extract information from a page such as this?
Most homepages are of this style, and have high traffic, so should get some attention :)
I feel capturing the article names is valuable. Would you agree?

Thanks @ydennisy for the input.
Although I'm not primarily interested by such segments some users indeed wish to extract all potentially interesting text parts.
Let's try to address it in a future version (see also #68).

Thanks - that is a possible solution, but why not extend the current functionality where a user can request elements such as comments, to lists also?

Lists are already part of the potential output, there are just absent when there isn't any article-style layout or when they are full of links (as it often means boilerplate parts).
A homepage like in your example combines both characteristics, these are tricky text parts to extract. I guess it's better to implement a recall-based preset which would yield text whatever happens.

You can follow #68 for updated info, I'm closing this issue for now.


report versions of the used tools

Describe the feature
would be great the build-output would show which version of the tools are used.
I guess we are currently running in one of the bugs the recent composer releases had, but were fixed in the subsequent release (we had 2 or 3 releases yesterday).
since I cannot see which version of composer the build is using I cannot confirm my thesis easily.

Version
using the latest v2 tag

@staabm By default it will always setup the latest release version.
I had not done this as some tools have a startup time and checking the version adds to the CI runtime. I will run some tests again.
Currently, I would recommend adding a step after setup-php step to check the tool version.
For cs2pr it will be 1.3.0 release. It shows the versions as 1.1.1-dev though as it picks $version in the script.
- name: Check versions
  run: |
    composer -V
    cs2pr version

Thx for the headsup. Just created a cs2pr issue https://github.com/staabm/annotate-pull-request-from-checkstyle/issues/51


Google Docs performance gets bad with this enabled.

Go to Google Docs.
Start typing.
Disable extension.
Start typing.
Notice less lag.

Hello,
What operating system do you use?
I cannot confirm this problem with Chrome 55 on Ubuntu 16.10.

Not reproducible.


LNK2038 when open AddressSanitizer

Describe the bug
As with the title, when I opened AddressSanitizer, the MSVC compiler encountered error LNK2038. But close AddressSanitizer, the compiler is working properly.
Environment

OS: Windows 11
Compiler: Visual Studio 2022 （17.5.5）

To Reproduce
Compile the project only.
Expected behavior
Compilation succeeded and ran.
Failure logs

Closed as duplicate of https://github.com/microsoft/vcpkg/issues/28627.


#138: Write a unit test for Payments / Get payments endpoint

@rultor merge

@rultor merge

@proshin-roman OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@proshin-roman Done! FYI, the full log is here (took me 2min)


Simon's Algorithm

To resolve Issue #38.

@ecp-rigetti Thanks for reviewing this! I should have covered all the changes that you've suggested; let me know if I've missed anything.

I think I'm happy. :) When we have an issue tracker again, someone should remember to open one about those utility functions.

@tarballs-are-good I made the changes you've suggested.

Be sure to update this branch.

GREAT JOB!

Awesome!!
On Tue, Jul 18, 2017 at 4:27 PM, Robert Smith<EMAIL_ADDRESS>wrote:

GREAT JOB!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rigetticomputing/grove/pull/40#issuecomment-316226507,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD8uAZLSX_hX9RRkZVlOa-Xj74FNBDFfks5sPT9VgaJpZM4OBSbO
.


Fold Lines after the first line

I have an xml document am trying to use the FoldAction
.Contract but its also folding the subitems into one line closing the root node i want the root node to stay open which is at line one of my document but the items within to be collapsed is their a way to say not collapse the root?

In fact it doesnt even coaplse the child items after some testing the folder nodes are open when i click the root node to expand again. I want the child items to remain closed.

I see the code here also for markers but this doesnt appear to work when you have folds as well.
https://github.com/jacobslusser/ScintillaNET/wiki/Markers-and-Bookmarking-Lines

So you are trying to achieve this (?):

Yes also I have an issue with duplication book marks showing yes that correct but i still need to collapse the root node then when it expands the sub items should be still closed exaclty how notepad ++ which is open source and using the same works am programming in c# btw @VPKSoft

Like this

@VPKSoft

I also would like my folder plus minus to show more like this

By that i mean only on the root nodes if that is is possble

The screen captures I took are from a library for creating lexers for ScintillaNET, see ScintillaLexers. The folding for XML is implemented for XML in the library with many others in C#. I don't know about the vertical line though.

Can u still keep the root node with a plus beside it in ur example theirs not @VPKSoft  i wonder how notepad plus plus does it as it using same editor.

I got it sorted only thing is appears to be a gap between my bookmarks and my folding also my book mark is tiny small not same size as notepad++ any ideas, its almost 3 pixels in size real tichy i have line number,book mark, what appears to be a blank space then the folding.

The screen captures I took are from a library for creating lexers for ScintillaNET, see ScintillaLexers. The folding for XML is implemented for XML in the library with many others in C#. I don't know about the vertical line though.

i STILL HAVE not got this fixed but ur docs is very confusing i only want the folding out of your program can you help for xml

For anyone else doing this in another project you have to do something like this.
foreach (var scintillaLine in editorControl1.Lines)
 {
                scintillaLine.FoldLine(FoldAction.Contract);
  }

And to Unfold
foreach (var scintillaLine in editorControl1.Lines)
 {
                scintillaLine.FoldLine(FoldAction.Expand;
  }


Update storage.md

Updating the section in Akord in line with recent updates to our app, API and docs.

Hi @pascaljb, I put in the requested changes without knowing there was a PR open for it (I was given a link to a Google doc with the changes instead). Sorry for any confusion. Closing this PR as no longer necessary.

Hi @lsgunnlsgunn these changes are the latest, in line with very recent updates to the app and API. I believe the Google docs is from a few months ago and the content is out of date. Could we update with this content for the Akord section? Thanks

@lsgunnlsgunn totally, happy to go with whatever you think makes sense on the headings in the context of the styles you have. Thanks!

@pascaljb cool. I'll approve and merge these tomorrow. Thanks for your patience!  There's a lot to keep track of with our fledgling docs site 😆.


chore: supply pre-build docker image

Publish a docker image for each merge and each release for easier deployment for users wanting to host their own server

Thank you! Totally happy to merge this once the conflicts are fixed :grin:


Flashlight button remains orange after turning light off from notification

Good catch, thanks!

@kylecorry31 Does this cover the case when flashlight is turned on/off from OS UI?

@notramo nope - I will look into handling that as well


Support for entity categories

Cannot be merged until next HA release

You will have to increase the required HA version I guess.

Good one, will do :-). I marked it as a draft for that sense. Will add the key soon.


Updated config_l3.1_8b_lora.yaml to be up to date with torchtune 0.3

Issue #, if available:
For sagemaker-distributed-training-workshop/13_torchtune/config/config_l3.1_8b_lora.yaml, the alternative DiskLogger was not updated to torchtune 0.3 and would result in an error when using it instead of WandBLogger
Description of changes:
Changed torchtune.utils.metric_logging.DiskLogger to torchtune.training.metric_logging.DiskLogger
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Added another commit to support single instance fine-tuning in launcher.py and added a qlora config yaml


Docs mention adding role 'cluster-view' but role does not exist

Using OpenShift Origin 1.3.1 (minishift 0.9) and trying to add the cluster-view role to a service account as per the docs but the cluster-view role doesn't exist:
$ oc policy add-role-to-user cluster-view system:serviceaccount:...
Error from server: role "cluster-view" not found

According to the OpenShift docs, the role is called cluster-reader.
Not sure when this changed but the docs should be updated.
Happy to fix it, just confirming the above?

I think that you are right!
@christian-posta: what do you think?

yah, you're right @donovanmuller
wanna send a PR?


Program not load gui.

I installed through .deb file with gdebi. The program don't load the interface. I use Linux Mint 19.1

Run again, or try run in the terminal by typing sudo systemd-manager-pkexec.


fix NormalizedConfig init error message

What does this PR do?

The error message displayed was incomplete. With this addition, the user is reminded that by setting allow_new=True, a new attribute can be added.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Thanks for completing the error message! @PaulQbFeng


Small Readme Bugs . channel!.sendMessage and user!.username

Small Readme Bugs .  channel!.sendMessage and user!.username
Please make sure to check the following tasks before opening and submitting a PR

[ ] I understand and have followed the contribution guide
[ ] I have tested my changes locally and they are working as intended
[ ] These changes do not have any notable side effects on other Revolt projects

As far as I know, these are nullable - the exclamation marks are for TypeScript.

As far as I know, these are nullable - the exclamation marks are for TypeScript.

No

All of the values in question are nullable.

https://www.typescriptlang.org/docs/handbook/2/everyday-types.html#non-null-assertion-operator-postfix-


KAFKA-14758: Extract inner classes from Fetcher for reuse in refactoring

The Fetcher class is used internally by the KafkaConsumer to fetch records from the brokers. There is ongoing work to create a new consumer implementation with a significantly refactored threading model. The threading refactor work requires a similarly refactored Fetcher.
This task includes refactoring Fetcher by extracting out the inner classes into top-level (though still in internal) so that those classes can be referenced by forthcoming refactored fetch logic.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@hachikuji @rajinisivaram @guozhangwang @vvcephei @philipnee This is another PR that "simply" refactors out the inner classes from the Fetcher. I broke it out into a separate PR because it's very noisy for the forthcoming PR.

@guozhangwang wrote:

A meta comment is that for the extracted classes, I think we should add unit test coverages for them (hopefully most of them are already in the FetcherTest...) i.e. one test class for each of the three.

Because CompletedFetch, FetchManagerMetrics, and FetchResponseMetricAggregator were all private inner classes of Fetcher, they are not directly referenced in FetcherTest. As such, there aren't any tests in FetcherTest that I can extract to make CompletedFetchTest, FetchManagerMetricsTest, or FetchResponseMetricAggregatorTest test classes. Any new unit test classes would be net new code.


updated tibber requiment to pytibber v0.29.0 to fix broken tibber integration (issue  #398)

FIX

fix(tibber) changed requirement pytibber to 0.29.0 in manifest.json

Thanks! 👍


Disable defaultHeaders option

Is your feature request related to a problem? Please describe.
I'm getting template names as translatable strings in the pot file, wich I think is unneeded (what's the case for this)?
Describe the solution you'd like
I'd like to not include these strings in the POT file because it is confusing for translators.

Do you have any examples?

Yea so for instance a PHP file with the following lines:
<?php

	/* Template Name: Partner Login template */

Would result into "Partner Login template" as a string in the POT file.
I've traced this back to wp-pot/translation-parser.js - there is some logic in there that detects the Template Name:, 'Plugin Name:' (and some more) strings. This is what I see on line 365-369 of that file:
    if (this.options.metadataFile === filename) {
      this.parseFileHeader(['Plugin Name', 'Theme Name', 'Description', 'Author', 'Author URI', 'Plugin URI', 'Theme URI'], filecontent, filename);
    }

    this.parseFileHeader(['Template Name'], filecontent, filename);

I'm not sure of the use case of this (I don't see a reason why one would want to change the template name, I think is actually in the way of WP when translating sites with for instance WPML, where the template name is inherited when translating a post. Using a different template name I think would break that and you'd have to select the template and re-save for each translated post. I'm not 100% sure of this - I can test this when I have some time if that would help.
In my case anyways this feature is in the way as translators don't know what to do with these strings when I send them the POT file, so for me it would be great to add a flag to be able to disable them (or customize them).
For now I can work around it by just disabling the lines in the wp-pot source. If you want I can submit a PR for this at a later time.

Hey @rasmusbe I've opened a PR for this issue https://github.com/wp-pot/wp-pot/pull/99

Resolved in #99


LiteX submodules update

This PR bumps LiteX submodules and includes generate.py fixups to work with the latest LiteX.
PR can be merged after HW tests using generated bitstreams from litex examples are proven to work

@kowalewskijan can you please rebase to solve the conflicts?


⚠ Bofrost.at has degraded performance

In 8f14b61, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1046 ms

Resolved: Bofrost.at performance has improved in c80b2a2 after 310 days, 2 hours, 27 minutes.


Docs: Missing documentation of assertion functions

Page URL: none yet
Issue: There is no mention of assertion functions in the handbook. The only official documentation of them is in the release notes of TypeScript 3.7: https://devblogs.microsoft.com/typescript/announcing-typescript-3-7/#assertion-functions
Recommended Fix: For the sake of completeness, there should be at least a short section explaining assertion functions somewhere in the handbook. For more detailed examples, it could link to the blog post.

Agreed! I love this feature and often look for it here: https://www.typescriptlang.org/docs/handbook/2/everyday-types.html#type-assertions
...and then get confused on where it is documented

I wrote a blog post filling in this gap https://blog.battlefy.com/refactor-lying-typescript-type-assertions-into-real-assertions


Accept login from approved ips without password

Description:
Accept login from approved ips without password.
Then it will be possible to login from given ips without providing a password.
Pull request in home-assistant.io with documentation (if applicable): https://github.com/home-assistant/home-assistant.github.io/pull/954
Example entry for configuration.yaml (if applicable):
http:
 api_password: cdfahms
 approved_ips:
   - <IP_ADDRESS>
   - <IP_ADDRESS>

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Wouldn't whitelist be a more consistent name? Or does that make it seem like only those IP's can log in at all?

I am guessing it also allows white listing external IPs.

👍 🐬


Thoughts / reviews count on the books

Is your feature request related to a problem? Please describe.
There is no way to see how many people have shared their thoughts and reviews for the book.

Describe the solution you'd like

[ ] Add the reviews count on the all books page

[ ]  On the individual books page as well

@saurabhpatel123456 You can check https://laravel.com/docs/8.x/eloquent-relationships#counting-related-models to get the count of the comments related to the book.


superenv: disable ConstraintEliminationPass on Xcode 16

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

This can result in build failures (e.g. OpenJDK) and silently miscompile
some formulae (e.g. Vim).
See Homebrew/homebrew-core#195325.

Isn't that a super heavy-handed fix? We are aware of two formulas where this bug triggers: vim and openjdk. Compilers will always have various slight optimization bugs, and I think it's a bad road to expect to fix them across the board.

To be clear about the approach I recommend: I suggest enabling the flag for vim and openjdk, and now that the issue is well-identified (thanks to great work on that issue page), see if other reports come in that appear similar.

Maybe, but the way I see it is: we are trading off between the risk of miscompilation vs the risk of missed optimisations. One is much worse than the other.
If that's not actually the tradeoff that we're making, then do correct me, but once you agree that that's the tradeoff then it's hard not to conclude that you'd rather potentially miss some optimisations than potentially miscompile software.
As for compilers always having slight optimisation bugs: this is true, but "optimisation bugs" come in many different forms, and not all of those forms result in miscompilation. (Some, maybe most, just lead in code that isn't really faster.) Wherever we're aware of any that result in miscompilation and can do something about it, we probably should.

I don't think it's just missed optimization, these passes are not strongly tested independently (i.e., most code and most tests just specify one -O value) so there is potential risk that we trigger other optimization bugs by forcing an unusual combination of passes. For the majority of software, where there is no known problem, I'd rather ship with the most tested configuration (including the one that Apple themselves use to compile their own OS and system).
I think the benefit/risk ratio would be different if we had a lot of formulas that crashed or otherwise misbehaved, but we have deployed Sonoma bottles for two months and so far we have found issues in only two cases: openjdk bootstrap, and one very specific code path in vim.

I suggest enabling the flag for vim and openjdk, and now that the issue is well-identified (thanks to great work on that issue page), see if other reports come in that appear similar.

This seems reasonable to me 👍🏻

Original issue was confirmed fixed in Xcode 16.2 beta 3 (clang-16<IP_ADDRESS>), see https://github.com/Homebrew/homebrew-core/issues/195325#issuecomment-2492636277


Unexplained "sqr" function in documentation

Section 4.10.1, listing 15 (https://google.github.io/filament/Filament.md.html#listing_anisotropicbrdf) makes use of a function sqr which is not explained in the text. Please consider changing this code example to be more explicit; my first guess was that it was a typo for sqrt, rather than a utility function for squaring the value.
(Additionally, it looks as though the function is actually called sq in the codebase).

I suggest either pow2 (consistent with pow5), or re-writing the code snippet to avoid its usage altogether.

I'll add a comment in the code snippet explaining what sq does.


x/playground: add ability to resize the output box [automatically]

On small screens like 11-13" laptop's the output box is somewhat small. I would like to be able to adjust its height.
Also when output is big, I want to be able to see all of it without scrolling each time after running the program. This would mean to add some controls like buttons on the right or make the edge between code and output boxes draggable.
As alternative solution would be to define the maximum allowed height of the output box and resize it depending on the output as mentioned here https://github.com/golang/go/issues/15163. That might be a pie in the sky though.
I am sure this was raised before but I could not find it.

Are you talking about the embedded playground in golang.org ?
Then perhaps something similar to the little expander icon at the bottom-right of the text box which you can use to resize the code box. We can apply the same for the output-box.

Ah, you mean like a horizontal expander thing at the border of the yellow and white box ?

@agnivade, yes in the case of UI element. But I wonder how badly do we want to resize output box automatically depending on the size of output.
@andybons

Seems like a solution could be to allow adjusting the split between the code and output. When you hover over the split point it shows a resize cursor (cursor: row-resize) and allows you to adjust it (while not saving between reloads).

the output box on go.dev/play is resizable


npm run start is not work

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

@lebornjose
Can you please explain what you are trying to do and what errors you are getting.
We need you to help us help you


Add relation column average query

Query Builder Get Average from many relationship

@TBlindaruk   can you  accept this ?

Release this as a package first.

How I can do this ?


build: enable again some Lighthouse checks

#438  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


[Bug]: UQI form control components not setting evaluation order

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
evaluation order is not getting set for form inputs in the new components. Likely cause is that some of these paths are created on the fly within these component definitions.
Steps To Reproduce
https://theappsmith.slack.com/archives/C02KTH6EHUG/p1644904919405449?thread_ts=1644833232.546069&cid=C02KTH6EHUG
Environment
Release
Version
Cloud

Related to UQI

Verified as part of regression


Random Events

Overview
I want to play audio files or TTS randomly with a minimum interval only when in practice or race sessions in iRacing. This will be used to add encouraging audio.
Key requirements

Randomised event
Only active during specified iRacing session types
Executes a function e.g. play an audio file or TTS
Metadata is required to determine which audio should be played
Events can originate from random time and specific racing events (randomly selected)

Randomise events

Timer event with a minimum time between events
Event randomly triggered after specified iRacing events e.g. IRacingDriverIncident min 3, max5 will trigger this between 3 to 5 occurrences of IRacingDriverIncident event

I am not sure if this should be done in a script or a plugin.

I think this might help you. It got a timer_talk that is invoked between 10 and 30s. If you press the UI buttons, there is a 50% chance for, that it will invoke triggered_talk. This makes sure that the time_talk is reset, so we dont have two TTS just after eachother.
-- to generate some test events
ui:create_button("relevant")   -- we might triggered_talk
ui:create_button("irrelevant") -- always ignored

-- this is triggered at random times
local timer_talk = function(delay)
	audio:say("Waited " .. tostring(delay) .. " seconds", 1.0)
end

-- this is triggered by relevant events
local triggered_talk = function()
	audio:say("Say something nice", 1)

	-- go back to normal
	timer_talk_setup()
end

function timer_talk_setup()
	local delay = math.random(10,30);

	ui:print("setup_random delay = " .. delay)

	core:debounce("random-wait", function()
		timer_talk(delay)

		-- setup next time
		timer_talk_setup()
	end, delay)
end

function triggered_talk_setup()
	local n = math.random(0,1);

	if n == 0 then
		-- To avoid that we get a "timer_talk" just after event triggered
		-- speech, we use debounce with 0 (immediately) thereby overwriting
		-- the timer_talk with our triggered_talk
		core:debounce("random-wait", triggered_talk, 0)
	else
		ui:print("Ignored event")
	end
end

timer_talk_setup()

function handle(event)
	ui:print(core:event_to_json(event))

	if event.EventType == "UIButtonTriggered" and event.Text == "relevant" then
		triggered_talk_setup()
	end
end

I think this might help you. It got a timer_talk that is invoked between 10 and 30s. If you press the UI buttons, there is a 50% chance for, that it will invoke triggered_talk. This makes sure that the time_talk is reset, so we dont have two TTS just after eachother.
-- to generate some test events
ui:create_button("relevant")   -- we might triggered_talk
ui:create_button("irrelevant") -- always ignored

-- this is triggered at random times
local timer_talk = function(delay)
	audio:say("Waited " .. tostring(delay) .. " seconds", 1.0)
end

-- this is triggered by relevant events
local triggered_talk = function()
	audio:say("Say something nice", 1)

	-- go back to normal
	timer_talk_setup()
end

function timer_talk_setup()
	local delay = math.random(10,30);

	ui:print("setup_random delay = " .. delay)

	core:debounce("random-wait", function()
		timer_talk(delay)

		-- setup next time
		timer_talk_setup()
	end, delay)
end

function triggered_talk_setup()
	local n = math.random(0,1);

	if n == 0 then
		-- To avoid that we get a "timer_talk" just after event triggered
		-- speech, we use debounce with 0 (immediately) thereby overwriting
		-- the timer_talk with our triggered_talk
		core:debounce("random-wait", triggered_talk, 0)
	else
		ui:print("Ignored event")
	end
end

timer_talk_setup()

function handle(event)
	ui:print(core:event_to_json(event))

	if event.EventType == "UIButtonTriggered" and event.Text == "relevant" then
		triggered_talk_setup()
	end
end


Fix/issue 201

Deals with Issue #201 by creating a deferred execution enumerable.

Closing in favor of #317


Add information to the cards

Value Proposition
As a user
I want to see more information on the cards
So that I can see how to perform the exercises
Description

Acceptance Criteria

[ ] the shown cards have an image
[ ] the shown cards have an instruction text

Tasks
(for developers team)

[ ] create branch
[ ] add images to the database file
[ ] add instruction text to the database file
[ ] create a Styled Component for the image
[ ] create a Styled Component for the instruction text
[ ] add the image and text component to the Cardlist Component
[ ] map the data to render it on the page

Complexity
S

Nice 💪🏼
I would just suggest to always display the whole screen/app in your scribble instead of only part of it.


Fix templates not being included into the release

I found out that the templates folder wasn't being included, which is inconvenient, but it actually seems like it should have been included.
The issue I found is that templates/**/* doesn't match the files directly inside templates, it should be just template/*.
I tested it with goreleaser release --snapshot and seems to work perfectly.
Let me know what you think.

Hi @grilix! Thanks for your contribution!
Very interesting! I think this actually should work (iir how globs work correctly). Here's a glob test that shows the result.
You can also test it in zsh: for f in templates/**/*; do echo $f; done. Regardless, you are correct that this needs to be fixed. In the meantime, could you actually add another entry with templates/* instead of removing the deep glob? This is moreso to satisfy my own wants (I plan on including a better panel at some point that will make use of multi level template directories).

Oh, I don't use zsh, but that doesn't work on bash, so maybe that's where this issue comes from.
Just updated the file for the meantime.


Automator: update<EMAIL_ADDRESS>dependency in<EMAIL_ADDRESS>

Generated by Automator - 2023-05-03T16:45:25+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio_release-1.17
cd104290df996434ee5f11454a35ea514b72eb5a
link
unknown
/test integ-pilot-multicluster

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


Please don't hard-code LIB_INSTALL_DIR

In Gentoo Linux x86_64 libs need to go into /usr/lib64 but the current CMakeLists.txt file hard-codes "${CMAKE_INSTALL_PREFIX}/lib"  which will make installation in Gentoo fail because /usr/lib is reserved for x86_32 libraries.

You are right, this is an oversight on my part.
This (and the others) were supposed to be enclosed in if (NOT DEFINED LIB_INSTALL_DIR) blocks, so they can be easily overridden on the command line like cmake . -DLIB_INSTALL_DIR=/usr/lib64/.
set(LIB_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/lib" CACHE PATH "Installation directory for libraries")

I will make a PR for this later, and it will be included in a future minor release.

It would be nice if you would use the GNUInstallDirs module, or at least follow the same naming convention (CMAKE_INSTALL_LIBDIR / CMAKE_INSTALL_FULL_LIBDIR).

Hmm, the current names were chosen to match the ones used by the %cmake macros used in rpm.
The GNUInstallDirs names are the fourth suggested naming convention IIRC. It is obvious there are no right names.
What I do know is the names that are currently used, are used by a lot of applications already.
Only good solution would perhaps be to have detection for different sets of names from the command line, and copy their content over if found.

It could even be possible to add optional support for GNUInstallDirs using command-line option... it would basically convert the variable names to the ones we are currently using...
For example:
cmake -DGNU_INSTALL=YES

I use GNUInstallDirs in minizip-ng and it works good.
https://github.com/zlib-ng/minizip-ng/blob/1593060ecf093411f1f20f655e50992913c6d5b9/CMakeLists.txt#L79

@nmoinvaz But does it handle this parameter properly? cmake . -DLIB_INSTALL_DIR=/usr/lib64/

It think just looking at GNUInstallDirs.cmake it would be something like cmake . -DCMAKE_INSTALL_FULL_LIBDIR=/usr/lib64/

I made a patch for this, it seems to work locally in my limited manual testing. Waiting for CI to catch up before I make a PR.
https://github.com/zlib-ng/zlib-ng/commit/1f03b95321ee3e5b9359c6c841c9cc6e1b2fe902


question about codes

Greetings, Dr.jaeger. This is a student at CU.
At present, we've finished connecting TS w/ CAN.
I changed two places of your code in order to fit TS codes in CCS to support C2000, can you please check are these changes necessary or not?

In ‘src’ folder of 'TS library', for every files, i.e., TS.c, TSbin.c, TS_txt.c, there are code sentence as:

for (unsigned int i = 0; i < n; i++)
It has an error in CCS, because Version of C/C++ GCC support in CCS is not advanced.
Correct it as:
unsigned int i; for (i = 0; i < n; i++)
Do you want to change your code as above? Because it can get a wider usage, e.g. it can support other C standards, and it is easy to change.

In ‘src’ folder, in 'thingset.c', one structure 'ts_get_object_by_id' is defined as:

///
struct ts_data_object *ts_get_object_by_id (struct ts_context *ts, ts_object_id_t id)
{
unsigned int i = 0;
for (i = 0; i < ts->num_objects; i++) {
    if (ts->data_objects[i].id == id) {
        return &(ts->data_objects[i]);
    }
}

return NULL;
}
///
Afterwards,
endpoint = ts_get_object_by_id(ts, id);
W/out this change, 'response' = 0, because return of function is always 0, endpoint=0.
Delete ‘return NULL’, correct it as:
///
struct ts_data_object *ts_get_object_by_id(struct ts_context *ts, ts_object_id_t id)
{
unsigned int i = 0;
for (i = 0; i < ts->num_objects; i++) {
    if (ts->data_objects[i].id == id) {
        return &(ts->data_objects[i]);
    }
}

///
afterwards, endpoint can be correct and can get correct 'response' as e.g. in TS website.

I would be generally open for changes required to switch to C90 compatibility if you submit them as a PR. Is the upfront variable declaration you mentioned the only issue you experienced?
I don't completely understand your 2nd point. We can't just remove the return statement at the end. The function must return a value under all conditions. If the data object is not found it shall return NULL.

Appreciate it for responding, Dr.jaeger.

It is the only occurring error, and it's quite easy to change, in seconds.
Agree function needs to return under all conditions.
Yet, originally code is,

///
`struct ts_data_object *ts_get_object_by_id (struct ts_context *ts, ts_object_id_t id)
{
unsigned int i = 0;
for (i = 0; i < ts->num_objects; i++) {
if (ts->data_objects[i].id == id) {
return &(ts->data_objects[i]);
}
}
return NULL;
}
///`
Afterwards,
endpoint = ts_get_object_by_id(ts, id);
so, if (ts->data_objects[i].id == id, return &(ts->data_objects[i]), it is correct.
Yet, return  NULL is outside 'if' sentence, and outside 'for' sentence.
i.e. under any condition, it'll always run 'return NULL' and get 'return = 0',
if wanting to add condition data object is not found, we think correct one needs to be
`///
struct ts_data_object *ts_get_object_by_id (struct ts_context *ts, ts_object_id_t id)
{
unsigned int i = 0;
for (i = 0; i < ts->num_objects; i++) {
if (ts->data_objects[i].id == id) {
return &(ts->data_objects[i]);
}
else {
return NULL;}
}
}
///`
Add 'else' to represent condition data object is not found, and it needs to be inside of 'for' sentence.

Hello @CUgroup. I have revisited the improvements introduced in C99 and there is much more to it than just declaring variables for for loops outside the loop.
Within this library we make use of the following features of C99:

Standard boolean type bool from <stdbool.h>
Fixed-width integer types [u]intN_t from <stdint.h>
Designated initializers
Variadic macros
Single-line comments with //

So I'm sorry to say we are not going to be able to switch to C90 support. Minimum requirements will be a C99 capable compiler.
Regarding your point 2: Your suggestion won't work. The loop iterates over all elements and it should not return NULL if the first element does not match with the provided id.

Greetings, Dr. Jaeger, appreciate it for your suggestions.
In CCS, one option can change C standard to C99.
After changing standard, code can run.

That's great to hear.
BTW: I've been working on a new version of this library which has a more clean code structure, is better tested and uses an existing CBOR library. Currently it only works with Zephyr, but it won't be too difficult to enable support for other environments. It requires C99 as well.
https://github.com/ThingSet/thingset-node-c
Also the ThingSet specification has been updated to v0.6. The thingset-device-library supports v0.5 only, whereas the new thingset-node-c supports v0.6. See here for a changelog in the spec: https://thingset.io/spec/changelog.html#v0-5-to-v0-6


Image resize no longer working

I just updated docdock and I realized that image resizing doesn't work anymore (see: https://xmpp.mdosch.de/apps/conversations/ ) also I haven't change anything with my source files. Also according to your documentation this function is still available: http://docdock.netlify.com/create-page/page-images/

I'm not why it stopped working in the theme, but Hugo introduced its own image processing in 0.32, so maybe that will work better?

I couldn't figure out how this integrated stuff works. But as I updated the submodules it's working again so it seems you fixed it upstream.


DOCKER-63 Set the number of allowed replicas in sonarqube and sonarqube-lts charts

We only support 1 replica for deployment and statefulset types in sonarqube and sonarqube-lts charts. Therefore, we set 0 and 1 as allowed values for replicas, having 1 as the default value.

We introduce the input validation for replicaCount. Now users specifying a value different from 0 and 1, will get the following error:
Error: UPGRADE FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
sonarqube:
- replicaCount: replicaCount must be one of the following: 0, 1

We also simulated the following upgrade scenario:

Kathy is running version 5.0.1 (it does not include input validation) with replicaCount set to 2 (although this is not recommended, the 2 pods are up and running)
Kathy tries to upgrade to version 5.0.2 (with input validation) using the same values.yaml. The upgrade fails with the above error message. This happens also if they try to set replicaCount via command line (--set replicaCount=2)

Which are your thoughts on that @leo-geoffroy-sonarsource ?

We introduce the input validation for replicaCount. Now users specifying a value different from 0 and 1, will get the following error:
Error: UPGRADE FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
sonarqube:
- replicaCount: replicaCount must be one of the following: 0, 1

We also simulated the following upgrade scenario:

Kathy is running version 5.0.1 (it does not include input validation) with replicaCount set to 2 (although this is not recommended, the 2 pods are up and running)
Kathy tries to upgrade to version 5.0.2 (with input validation) using the same values.yaml. The upgrade fails with the above error message. This happens also if they try to set replicaCount via command line (--set replicaCount=2)

Which are your thoughts on that @leo-geoffroy-sonarsource ?

Looks good to me! I tested it locally and everything works as expected
I think we are good to go after a commit squash :)


Create Terraform code generator

TBD

Not doing this right now, we'll clean up the current changes related to this and may revisit in the future.


feat(Dataset): static exportToValue method + docs examples

Added a new test for the static method, expanded on the example, and added the example to docs/examples


[Rendering] can not render

Target Platforms
Android
SDK Version
1.4
Application Name
Microsoft Teams
Problem Description
can not render Application Card  on Android app
Screenshots

Card JSON
{
  $schema: 'http://adaptivecards.io/schemas/adaptive-card.json',
  version: '1.4',
  type: 'AdaptiveCard',
  body: [
    {
      type: "TextBlock",
      size: "Medium",
      weight: "Bolder",
      text: "👏 Welcome to ONES.com for Teams! Let’s increase efficiency and productivity!",
      wrap: true,
    },
    {
      type: "Container",
      items: [
        {
          type: "TextBlock",
          text: "🔔 Issue notifications",
          wrap: true,
          weight: "Bolder",
        },
        {
          type: "TextBlock",
          wrap: true,
          text: 'Send "notify" to  App, to subscribe and receive notifications you care about.',
        },
      ],
    },
    {
      type: "Container",
      items: [
        {
          type: "TextBlock",
          text: "📜 Help guide",
          wrap: true,
          weight: "Bolder",
        },
        {
          type: "TextBlock",
          wrap: true,
          text: 'Get instructions by sending "help" to App..',
        },
      ],
      separator: true,
      spacing: "Medium",
    },
    {
      type: "Container",
      items: [
        {
          type: "TextBlock",
          text: "👩 Contact us",
          wrap: true,
          weight: "Bolder",
        },
        {
          type: "TextBlock",
          wrap: true,
          text: "Questions, bug reports, feedbacks — we’re here for it all, please [contact us]",
        },
      ],
      separator: true,
      spacing: "Medium",
    },
    {
      type: "Container",
      items: [
        {
          type: "TextBlock",
          text: "👤 Log out your account",
          wrap: true,
          weight: "Bolder",
        },
        {
          type: "TextBlock",
          wrap: true,
          text: 'Send "logout" to App.',
        },
      ],
      spacing: "Medium",
      separator: true,
    },
  ],
}

Sample Code Language
nodejs
Sample Code
No response

@sayali-MSFT

i found the resons:
when i download teams app in google play, it works fine..
but when i download teams app in xiaomi appstore or huawei appstore,  it can not works.
so...  it seems the teams app version in China region is not the latest,
and i found the changlog about teams app,   i can not found the app version "1416/<IP_ADDRESS>1042002" which is available on Xiaomi App Store


Test the Cupcake app: Android Basics in Compose

Add the @Test annotation to the setupCupcakeNavHost() method.

Seems like @Test should be changed to @Before.

@myhency
Thank you for reaching out to us. The typo is fixed and published:


Just reaching out to share our media query parser

See : https://github.com/csstools/postcss-plugins/tree/main/packages/media-query-list-parser
It seems we both created parsers for the same reason :)
Our test suite or some implementation details might be interesting for your package.
You can close this after reading.

Excellent, thanks! Gonna look over it and see if there's anything I missed.
One thing to be aware of in the spec was that there must not be a space between < and = (and > and =); it seemed to be the only place where an explicit check for whitespace is needed

One thing to be aware of in the spec was that there must not be a space between < and = (and > and =); it seemed to be the only place where an explicit check for whitespace is needed

Correct :)
You can observe this in the transpilation of range query syntax.
When there is a space it will be invalid and it will be ignored.
https://preset-env.cssdb.org/playground/#eyJzb3VyY2UiOiJAbWVkaWEgKDQ4cmVtIDw9IHdpZHRoKSB7XG5cdC5mb28ge31cbn1cblxuQG1lZGlhICg0OHJlbSA8ID0gd2lkdGgpIHtcblx0LmZvbyB7fVxufVxuIiwiY29uZmlnIjp7ImJyb3dzZXJzIjpbIj4gMC4yJSBhbmQgbm90IGRlYWQiXSwibWluaW11bVZlbmRvckltcGxlbWVudGF0aW9ucyI6MCwic3RhZ2UiOjIsImxvZ2ljYWwiOnsiaW5saW5lRGlyZWN0aW9uIjoibGVmdC10by1yaWdodCIsImJsb2NrRGlyZWN0aW9uIjoidG9wLXRvLWJvdHRvbSJ9fX0%3D


make env fails with OSError on Mac OS

"/Users/whit537/bin/python2.7" "./vendor/virtualenv-1.11.6.py" \
				--unzip-setuptools \
				--prompt="[gratipay] " \
				--extra-search-dir=./vendor/ \
				--always-copy \
				./env/
New python executable in ./env/bin/python2.7
Also creating executable in ./env/bin/python
Installing setuptools, pip...
  Complete output from command /Users/whit537/perso...om/env/bin/python2.7 -c "import sys, pip; sys...d\"] + sys.argv[1:]))" setuptools pip:
  Ignoring indexes: https://pypi.python.org/simple/
Downloading/unpacking setuptools
  Running setup.py (path:/Users/whit537/personal/gratipay/gratipay.com/env/build/setuptools/setup.py) egg_info for package setuptools

    warning: no files found matching '*.c' under directory 'tests'
    warning: no files found matching '*.pyx' under directory 'tests'
Downloading/unpacking pip
  Running setup.py (path:/Users/whit537/personal/gratipay/gratipay.com/env/build/pip/setup.py) egg_info for package pip
    /Users/whit537/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'entry_points'
      warnings.warn(msg)
    /Users/whit537/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'extras_require'
      warnings.warn(msg)
    /Users/whit537/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'zip_safe'
      warnings.warn(msg)
    /Users/whit537/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'tests_require'
      warnings.warn(msg)
    usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
       or: -c --help [cmd1 cmd2 ...]
       or: -c --help-commands
       or: -c cmd --help

    error: invalid command 'egg_info'
    Complete output from command python setup.py egg_info:
    /Users/whit537/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'entry_points'

  warnings.warn(msg)

/Users/whit537/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'extras_require'

  warnings.warn(msg)

/Users/whit537/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'zip_safe'

  warnings.warn(msg)

/Users/whit537/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'tests_require'

  warnings.warn(msg)

usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]

   or: -c --help [cmd1 cmd2 ...]

   or: -c --help-commands

   or: -c cmd --help

error: invalid command 'egg_info'

----------------------------------------
Cleaning up...
Command python setup.py egg_info failed with error code 1 in /Users/whit537/personal/gratipay/gratipay.com/env/build/pip
Storing debug log for failure in /Users/whit537/.pip/pip.log
----------------------------------------
...Installing setuptools, pip...done.
Traceback (most recent call last):
  File "./vendor/virtualenv-1.11.6.py", line 2338, in <module>
    main()
  File "./vendor/virtualenv-1.11.6.py", line 824, in main
    symlink=options.symlink)
  File "./vendor/virtualenv-1.11.6.py", line 992, in create_environment
    install_wheel(to_install, py_executable, search_dirs)
  File "./vendor/virtualenv-1.11.6.py", line 960, in install_wheel
    'PIP_NO_INDEX': '1'
  File "./vendor/virtualenv-1.11.6.py", line 902, in call_subprocess
    % (cmd_desc, proc.returncode))
OSError: Command /Users/whit537/perso...om/env/bin/python2.7 -c "import sys, pip; sys...d\"] + sys.argv[1:]))" setuptools pip failed with error code 1
make: *** [env] Error 1

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/9724006-make-env-fails-with-oserror-on-mac-os?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

I suggest to upgrade virtualenv and pip and setuptools before getting too deep into their problems.

@techtonik But we include all three in vendor/. Did we recently update those?

Last updated four months ago in https://github.com/gratipay/gratipay.com/commit/8f13d711fd3cb9b1dc71b403416efc3e29d8d463, afaict.

Have I really not rebuilt in that long?

Huh. The problem is in fact that make env is using a newer version of setuptools, one that it finds outside of vendor/ (I recently placed a tarball in ~/src/). We should fix the build to only use setuptools/pip from vendor/.

s/fix the build/instruct virtualenv

Doesn't look like this is possible with the version of virtualenv we're using. However, it appears that newer versions include setuptools and pip:

This option allows you to provide your own versions of setuptools and/or pip to use instead of the embedded versions that come with virtualenv. [emphasis added]

I'm changing this ticket to be upgrading virtualenv/setuptools/pip.

I'll work around the bug locally by moving the newer tarballs out of the way.

This option allows you to provide your own versions of setuptools and/or
pip to use instead of the embedded versions that come with virtualenv.
[emphasis added]

This may not help. Quoting the same page:

virtualenv will look for wheels in the specified directories, but will
use pip’s standard algorithm for selecting the wheel to install, which looks
for the latest compatible wheel. [emphasis added]

This option allows you to provide your own versions of setuptools and/or pip to use instead of the embedded versions that come with virtualenv. [emphasis added]

This may not help. Quoting the same page:

virtualenv will look for wheels in the specified directories, but will use pip’s standard algorithm for selecting the wheel to install, which looks for the latest compatible wheel. [emphasis added]

See.

And we already use --extra-search-dir=./vendor/ option. It looks like it should precedence.

Why do we need setuptools at all? Entry point scripts? I thought that we can just use ones in the root of the project.


Start providing releases

I think the scope of this project has pretty much honed in on Linux with the use of inotify. I'd still think about supporting BSDs and OS X with the use of kqueues and the event API, but I think that's unnecessary for these initial releases.
I am thinking about supporting:
[ ] nix
[ ] apt
[ ] ?
In addition, there should be other ways to build this project rather than just stack, which means, predominantly, cabal support.

Closing due to inactivity.


job: remove duplicate if statement

This PR removes a duplicate if statement I saw while reading ffuf's code.
As there're no unit tests, I built ffuf using Go 1.15 and tip and it worked as usual.
Please, let me know if anything else is needed from my side.
Best,
Miguel

Hi @joohoi,
I added myself to the contributors list. Thanks!


Removed mne installation using newest github version

This PR fixes tests that are crashing in travis build (I changed requiremnts.txt mne version in one of the previous PR but it was defined also in conda environment.yml). I think that we have also a problem with travis build. Travis build is not done for each PRs, so we can merge PRs with failing tests. @agramfort do you have any idea why for a PR we have only circle CI done (it does not run tests)? Is it a problem with travis configuration?

Example of PR with only 2 circle CI checks #101

Ok, I see, that this PR has triggered the travis build, so at this point, I do not understand why there are some PRs without this check.

these days I have seen travis fail to start on certain PRs.
do you have a failing test using mne stable in pypi?

Is the problem with travis permanent? Any idea how to fix that? In this PR it works well.
All tests work when using mne stable in pypi, this PR modifies mne version in the environment.yml to mne from pypi. I missed it in the previous PR #100. I was surprised that our CI was still green - travis should not pass but it was never triggered.

great, merged!


Create Biweekly_meetings_with_Arielle.md

Kicking off our notes - @Arielle-Bennett does this look okay to you?

I think it's great although I don't currently have write access @KirstieJane so can't merge this...

@Arielle-Bennett are you okay with the updates so far? Then we can merge and going forward we're finally on track for the bi-weekly updates 🤞🏻


Header Grid with new Arrows

Changed the header into a grid, changed to new SVG's.  Will probably reference these later to clean up the code but are inline for now.

be sure to rebase with the main branch before continuing, there's been some updates and bugfixes @andrewdeclue