Disconnecting ISO mid-send giver error in hci_num_completed_packets

Describe the bug
If we are in the middle of sending ISO data on an iso channel (i.e. we have sent data, but have not yet received the num completed packets) when an ISO (or an ACL) disconnects (i.e. we get a disconnect complete event from the controller), we see the following fail:
		conn = bt_conn_lookup_handle(handle);
		if (!conn) {
			BT_ERR("No connection for handle %u", handle);
			continue;
		}

There's nothing stopping us from deallocating iso channels while there is an ongoing write.
To Reproduce
Steps to reproduce the behavior:

Create ACL and CIS connection between two devices with a low ISO interval
Reset or move one of the device out of range to trigger a disconnect
See the error in the log

Expected behavior
The expected behavior would be to set the iso channels in the BT_ISO_DISCONNECT state until we have received the num complete packet of a pending write, and then release/deallocate the iso channel.
Impact
Low; the only problem caused is that the log will contain and error message.
Logs and console output
If applicable, add console logs or other types of debug information
e.g Wireshark capture or Logic analyzer capture (upload in zip archive).
copy-and-paste text and put a code fence (```) before and after, to help
explain the issue. (if unable to obtain text log, add a screenshot)
Environment (please complete the following information):

Commit SHA or Version used: 36abc60b5109b08a270c7df4e829bf67ecf3b96a

Additional context
Add any other context about the problem here.

I couldn't (haven't yet) figured out how this is done for ACL connections, but I would assume that a solution would require taking a reference count when sending and dereference it when sent.
@Vudentz @jhedberg @joerchan Do you have any suggestions, or input on how this is done for ACL connections?

@Thalley it's handled here: https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/bluetooth/host/conn.c#L1454

I'm taking a look at this issue right now

@jhedberg @joerchan Been looking at this issue for a while. I can confirm that the state handling of ISO connections works similar to ACL connections, and that process_unack_tx is called (as @jhedberg referenced in https://github.com/zephyrproject-rtos/zephyr/issues/36928#issuecomment-879229964).
However, I don't see how that function, or any other function anywhere, ensures that all bt_hci_evt_num_completed_packets are processed before we set the state of the connection to BT_CONN_DISCONNECTED.
The issue is basically that we get a bt_hci_evt_num_completed_packets event after we get the disconnect complete event. I am starting to wonder if this is correct behavior of the controller (OBS! a 3rd party controller in this case!), since the bt_hci_evt_num_completed_packets contains a connection handle, and the host typically will consider a handle invalid after the disconnect complete event.
I can't find anything in the core spec that explicitly states whether this is okay for the controller to do, or not.
Do you guys have any input?

Update: Found this little nifty sentence in the core spec:

The HCI_Number_Of_Completed_Packets event shall not specify a given Connection_Handle before the event indicating the corresponding connection has been created or after an event indicating disconnection of the corresponding connection.

So my conclusion is that this is an issue with the controller. If no objections, I'll close this issue.

Confirmed controller issue, closing.


Same cached component with different params

So how this mimics stack based routing is by caching pages. But when we're hitting the same page but with different params, the same cached component is being displayed/resumed.
This becomes a serious issue when let's say we have a page that displays information related to car based on id /car/:id. When we click to Honda X, the same old Tesla Y information from the cache is being displayed.
What I suggest, is to add an option to the StackRouter that caches page against complete URL string (including params). In this way, cached entries will consider params too.

The behavior you're describing is a compromise I made to port the concept of stack-based navigation to the web. The problem I encountered is that, while a native App has a maximum stack size imposed by the navigation rules, the same is not true on the web, where a user doesn't usually go back.
I'm wondering if you could solve the issue by writing a custom handler in your page component: using an onBeforeLoad handler, you could check if the params match the same car and if they don't you could temporarily show a loading state while you fetch new information.
If the above solution isn't what you are looking for, I can try to add an optional caching strategy, keeping the default memory-saving strategy, but allowing for the alternative you are describing.

Another option could be to add setResumable(false) to the car page, but I don't know if that's the behavior you're trying to achieve.

setResumable option won't resume the page regardless of same or different params.
I currently solved this issue by making my params reactive and setting a reactive function onParamsChange that reloads data on change of parameters. This isn't a good solution but works
If stack limit is the issue, we can add that option aswell to the StackRouter options and define an apt preset limit. Shouldn't be that big of an issue.

What you did (reacting on params change) is exactly what I usually do, it's the pattern I had in mind during the implementation, in fact you can find this in the readme:

a cached page is uniquely identified by its route template

I'm thinking about the implication of what you describe.
The new behavior would be different, because as of right know you have the guarantee that every cachable entry stays in the cache and doesn't get destroyed for the entire lifetime of the browsing session, but having a limit would remove this guarantee. It would probably be a good idea to add a priority flag to determine the order of destructions of items in the cache, for example setting the maximum preservation priority to an infinite scroll page.
Let me know if you'd like to work on that, otherwise I'll give it a try in a few weeks.

Instead of setting a flag, the most common practice is to use an LRU cache. Surprisingly, it outperforms all caching strategies. Apart from this, setting the stack size will be in our hands as well. In my opinion, we won't even reach the set stack limit in normal cases unless we're dealing with a huge app, and in that case, it would be optimal for the developer to alter the stack size according to his/her need.
I'm too busy at the moment, but i'll have a look at it. If I'm able to manage time for this, I'll open a PR. You should probably assign a 'Feature-Request' label to this issue.

Thanks, let me know if you manage to dedicate some time to it, otherwise I can give it a try in a few weeks.


MoveRange  (issue #311)

PR for #311

@atifaziz  all adjusts done (except that about ternary operator, because I agree with @fsateler ).

@atifaziz  All adjusts done! Thanks for show me about "test case source", I didnt know about it.

any problem with this PR ?

@atifaziz done!

take a look at InsertAt too, I think it may ship in the 2.8v

@leandromoh I'll take a final look after @fsateler has cleared his review.


strip tags (i.e., Word kill)

We have an application in which visitors submit scientific papers.  Only a limited amount of formatting is allowed - bold, italics, super, sub using b, i, sub, sup.  The "clean" data must export to XML for importing into inDesign.  Problem is that the submitters either don't read the instructions or are lazy and just paste directly from Word.
It seems to me with jQuery's abilities to bind to events we should be able to run a regex against the content and clean it up.  Ideally this would fire off with any change to the content (but could be a performance killer), or alternatively when the cursor leaves the field.
Can you add this, or give me some hints to hack it?

Just catching up on this issue now, has @christhomas's solution been merged?


adding script to reject hits

This won't let you reject only a subset of assignments for a HIT. Still seems useful though.


improve init command to create v2/v1 yaml file

sqlc init now would generate v2 format yaml file by default, user also can use --v1 to generate v1 format if needed.
Create an empty sqlc.yaml settings file

Usage:
  sqlc init [flags]

Flags:
  -h, --help   help for init
      --v1     generate v1 config yaml file
      --v2     generate v2 config yaml file (default true)

Thanks! This has been on the todo list for a while.


Colors of logo don't match

This relates to #1 and #10. (@lucasvazq)

The colors of the logo do not match. The icon is #188DBF and the text below it is #4990E1. To fix this, the color of the text should be changed to match the color of the icon.

Please comment below before you start working on this, and when you are done, please make a PR targeting the development branch.

Mind if I take a moment and fix it?

@AndrewJMD Go ahead!

Lool, HAHA 😂, 🙇♂🤦🏻♂

Opening this back up


Update README.md

Docker engine flag changed to --authorization-plugin

should be closed, no ?
the README was already updated by @npcode PR

Thanks @doronp, fixed in #15


Fix wrong release date for 1.4.13

What this PR does / why we need it:
1.4.13 is supposed to be released in 2023, not 2022. :slightly_smiling_face:
What type of PR is this?
/kind documentation
Does this PR introduce a user-facing change? Then add your Release Note here:
NONE

Documentation:
NONE

/lgtm
/approve


Known issue: Scalar Value mutator mutates unwanted junk IR nodes

This is issue is best demonstrated by this Pull Request: https://github.com/mull-project/mull-integration-tests/pull/26.
We have to disable GetElementPtrInst and ArraySubscriptExpr because otherwise both valid and junk nodes get mutated:
/opt/mull-integration-tests/tests/mutations/scalar_value/14_array_index_access/sample.cpp:17:20: warning: Replacing scalar with 0 or 42
  int first = (int)string[13];
                   ^
/opt/mull-integration-tests/tests/mutations/scalar_value/14_array_index_access/sample.cpp:17:20: warning: Replacing scalar with 0 or 42
  int first = (int)string[13];
                   ^

This is how GetElementPtrInst looks in the LLVM IR:
%4 = getelementptr inbounds [61 x i8], [61 x i8]* %1, i64 0, i64 13, !dbg !21

i64 0 is junk, i64 13 is valid
This is what we do at the moment:
/// TODO: we might want to reconsider this because some mutations can be
/// part of this instruction.
if (llvm::dyn_cast<llvm::GetElementPtrInst>(&instruction)) {
  return false;
}

and
// TODO: Not implemented on the LLVM IR level.
if (const clang::ArraySubscriptExpr *arraySubscriptExpr =
  parent.get<clang::ArraySubscriptExpr>()) {
  *mutationLocation = arraySubscriptExpr->getExprLoc();
  *astNodeType = ASTNodeType::ARRAY_SUBSCRIPT_EXPRESSION;
  return false;
}

https://github.com/mull-project/mull-integration-tests/pull/26

The current decision is to avoid the usage of the ScalarValue operator. Additionally, this particular case should be filtered on the AST level.
We might solve the scalars' problem altogether when Mull will support AST-level mutations.

The current decision is to avoid the usage of the ScalarValue operator. Additionally, this particular case should be filtered on the AST level.
We might solve the scalars' problem altogether when Mull will support AST-level mutations.


Option to always use the polyfill instead of native implementation

I've come across a strange situation where Firefox's native scrollIntoView implementation behaves strange and unnatural, whereas the polyfill works perfectly.
Thus I think there should be an option to always override this behaviour (for some special cases).

You're right about the scrollIntoView issue, the implementation has a bug and was already reported to Mozilla. This is actually a good idea to incorporate.

Yes please do this 👍  firefox is unusable with scrollIntoView without this

I made a simple demo where Firefox's smooth scrolling doesn't work while regular scrolling does http://output.jsbin.com/wunofejamu

Feature added on v0.4.0, go to the README to find out how to force polyfill. Thanks for the contributions y'all!


Code Quality: Duplication in DetailsLayoutBrowser, ColumnViewBase and GridViewBrowser

Description
Some code is duplicated between DetailsLayoutBrowser.xaml.cs, GridViewBrowser.xaml.csandColumnViewBase.xaml.cs`. Sometimes it's entire methods, sometimes it's only partial bits.
These classes are already inheriting from BaseLayout.cs. The duplicated code may be better hoisted inside this class.
Concerned code

BaseLayout.cs
ColumnViewBase.xaml.cs
DetailsLayoutBrowser.xaml.cs
GridViewBrowser.xaml.cs

Gains

Better maintainability.
No need to do the same modifications thrice when we want to add a functionality to the three views.

Requirements

Hoist the duplicated code into BaseLayout.cs

Comments
No response

@lukeblevins do we want to move ahead with removing duplicate code in this area? It might help future refactoring efforts but I'm not sure if it conflicts with your current work.

The current plan of action for the refactorings are as follow:

Create a standard layout for the currently existing views ( Done in #10996 )
Create a common layour for Details and Grid views to fuse their common code.
See what code should be brought down from BaseLayout to StandardLayout (i.e. the code that has nothing to do in the ColumnBlades)
Review the logic of the ColumnView and the ColumnBlades to see if the blades logic could be separated from the BaseLayout.

If it can be done, see if the existance of the BaseLayout is still required.

@QuaintMako Do you think we should create ViewModels of those layout mode Views? I introduced(brought from App.Interacts) BaseLayoutViewModel and found some codes can be isolated from code-behind.


In-app broser DApp connections status

Show connection information on in-app browser like the extension version

Update follow new UI: https://www.figma.com/file/2nYhrm0rlJtfJoMwrT51Ep/SubWallet-Apps?type=design&node-id=3647-409422&t=Qx4ke3j8wTGtNYjo-4
Resolve in https://github.com/Koniverse/SubWallet-Mobile/issues/692


Fixed format string error

:smile:

Derp, thanks.
Now the real question is, who is writing contract parameters without default constructors?

Clearly me!  I had one that is special because it gets blown away before
being written too there persistence file. I fixed it, but I'm still getting
the error, so I'll have to recompile cw+ to know which one it's complaining
about.

Okay, when I looked at it, I could see why it would fail for me.  One was the "special" one I mentioned which has a generic parameter, the other one was an abstract class that I use for several parameters.  So I added exclusions for that.


Support package.json's exports in @tamagui/font-inter

My _layout.tsx imports Tamagui font files:
function RootLayoutNav() {
	const [fontsLoaded, fontError] = useFonts({
		Inter: require('@tamagui/font-inter/otf/Inter-Medium.otf'),
		InterBold: require('@tamagui/font-inter/otf/Inter-Bold.otf'),
	})
...

I'm experimenting with Metro config field unstable_enablePackageExports, docs https://reactnative.dev/blog/2023/06/21/package-exports-support.

When no match is found in "exports", Metro will log a warning and fall back to resolving modules without considering "exports".

Metro warns me that the font dependency should have exports.
warning: Attempted to import the module<EMAIL_ADDRESS>which is not listed in the "exports" of "/Users/guillem/programacio/codi/cites/xiroi/xiroi/node_modules/@tamagui/font-inter". Falling back to file-based resolution. Consider updating the call site or asking the package maintainer(s) to expose this API.
warning: Attempted to import the module<EMAIL_ADDRESS>which is not listed in the "exports" of "/Users/guillem/programacio/codi/cites/xiroi/xiroi/node_modules/@tamagui/font-inter". Falling back to file-based resolution. Consider updating the call site or asking the package maintainer(s) to expose this API.

fixed in 43127313c1e35653f3ad81a74adddeebeeee31bd will go out next release.


removes newline from the message

Hi,
I've noticed that an HTML multiline message will be rejected beacuse it will produce a wrang JSON object.
You are appending <br/> at the end of each lines but not removing the \n
This small fix will removes all the \n and \r in the message before sending it.
I've tested it on OS X 10.9 and CentOS 7

+1
Is anybody working on this?

I believe PR #30, now merged, addresses this issue (although in a slightly different way).  If not, please reopen!  Thanks!


fix(snap): Refactor to avoid conflicts with readonly config provider directory

This PR makes changes to incorporate the following fixes:

https://github.com/canonical/edgex-snap-hooks/pull/83
https://github.com/canonical/edgex-snap-hooks/pull/82

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

For testing instructions, please refer to https://github.com/edgexfoundry/device-mqtt-go/pull/535
New Dependency Instructions (If applicable)

Codecov Report

Merging #194 (7bed824) into main (96327e0) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@          Coverage Diff          @@
##            main    #194   +/-   ##
=====================================
  Coverage   3.05%   3.05%
=====================================
  Files          6       6
  Lines        687     687
=====================================
  Hits          21      21
  Misses       666     666

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


Secrets not being cleaned up

Describe the bug:
We've been seeing a lot of errors in our logs like this:
cert-manager/secret-mapper "msg"="unable to fetch certificate that owns the secret" "error"="Certificate.certmanager.k8s.io \"example-tls4\" not found" "certificate"={"Namespace":"default","Name":"example-tls4"} "secret"={"Namespace":"default","Name":"example-tls4"
I believe cert-manager is looking at the secret and trying to do something(?) with it. However the resource which owned the secret has been deleted.
In this case only the secret remained. Nothing under kubectl get certificates.
Would it not make sense for cert-manager to automatically clean up these secrets once the resource/ingress has been deleted?
Expected behavior:
Orphaned secrets would be garbage collected/deleted
Steps to reproduce the bug:
Delete an ingress resource, the secret will remain.
Anything else we need to know?:
Environment details::

Kubernetes version (e.g. v1.10.2):
v1.12.8
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc):
AWS/kops
cert-manager version (e.g. v0.4.0):
v0.8.1
Install method (e.g. helm or static manifests):
static

/kind bug

This is impacting me as well. We strive for the cluster to be in the same pre apply and post delete. Right now the secrets are the only artifact left over.

This is supported, however is not the default (as it has the downside of taking down your production services if you accidentally delete any CRDs).
To enable it, add the --enable-certificate-owner-ref flag to your controller: https://github.com/jetstack/cert-manager/blob/f1d591a5317fda693a8df755e4c9ceaece998dbb/cmd/controller/app/options/options.go#L275-L277
Hope that helps! 😄

Script to clean these up if you dont have that flag set:
https://github.com/richstokes/k8s-scripts/tree/master/clean-orphaned-secrets-cert-manager

@richstokes you just saved my ass! I found so little information search for the errors that certmanager was throwing and finally came upon this using the logs from the injector pod

Ha. Glad you find it helpful!

I had the same error, but after the above script showed no orphaned Secrets, I examined the cert via:
kubectl -n <namespace> get certificates
kubectl -n <namespace> describe certificates <yourcert>
and I saw the actual error about being rate limited.

Thanks @richstokes , that greatly increased my understanding of how all of this works.

Thanks so much for your script, @richstokes ! It was very useful.
In our case, the secrets and certs don't have the same names, so I made an alternative script to check the secret's metadata.annotations.cert-manager.io/certificate-name and compare that to the certificate names instead. I'll leave it here in case anyone else comes along searching for the answer to "unable to fetch certificate that owns the secret". 😄
https://gist.github.com/lisawolderiksen/8c6026ef55f04e4f5d6a288b5e53214b


v0.2.0

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #14   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           4      4
=====================================
  Hits            4      4

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 8245989...3788ea6. Read the comment docs.

Can you stage a draft of the release notes and add a link to them here?

@JustinBeckwith no problem! https://github.com/googleapis/nodejs-dataproc/releases/tag/untagged-3ec076a239bd05c4d62f


Regarding position embeddings

Firstly, thank you for making the code public, it's neatly written and quite understandable. I think I am a bit confused about (partly because of my lack of knowledge about PyTorch) is position embeddings:
self.position_embeddings = nn.Parameter(torch.zeros(1, n_patches, config.hidden_size))
It seems that it is the equivalent of tf.variable in TensorFlow, which is initialized with zeros - (possibly) to be learned during training, though I am not 100% sure about the assumption. What I also don't understand is:

Why not initialize it sequentially from 0 to n_patches - 1 then perhaps pass it through an embedding layer?
Why is it two-dimensional and taking into account the hidden_size as well?

Thanks in advance :)

I would like to ask if you have solved this problem, what is the significance of applying the initializing all-zero matrix in the code, and how to use it? If you have solved this problem, We sincerely hope to get your help!
thank you!

Well, there is nothing to be solved there. It is about two competing ideas: relative positioning vs. absolute positioning. And one can experiment with both. Using learnable variables for positions means relative positioning and constants mean absolute/fixed positioning.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

Well, there is nothing to be solved there. It is about two competing ideas: relative positioning vs. absolute positioning. And one can experiment with both. Using learnable variables for positions means relative positioning and constants mean absolute/fixed positioning.
Thanks for your reply.
Wish you all the best!


Form submission blocked by clear button

When placing the Geocoder component inside a form, the form cannot be submitted using the Enter key. This issue arises because the clear button lacks a type="button" attribute, causing it to default to type="submit". Consequently, it is triggered during form submission and cancels the submission within the  _clear function.

A simple workaround is to set the attribute after adding the Geocoder control to the map (or another element).
container.querySelector('.maplibregl-ctrl-geocoder--button').setAttribute('type', 'button');


[ASK] Proxy with Authentication

Summary
What is the proper format in proxy file list for authentication for private proxies?

Hi, @hazanasec. It's simply, the workaround is: http://username:password@IP:PORT.


Using in separated npm package

How to use as npm package.
I have a problem with not being able to set the path to the sprite file in production mode

I am not sure to understand the problem you have. This plugin is made to work with ViteJS like this:
// vite.config.js / vite.config.ts
import VitePluginSvgSpritemap from '@spiriit/vite-plugin-svg-spritemap'

export default {
  plugins: [
    VitePluginSvgSpritemap('./src/icons/*.svg', {
      styles: 'src/scss/spritemap.scss'
    })
  ]
}

And use like this in your app. ViteJS will transform /__spritemap to your vite assets directory.
<svg>
  <use xlink:href="/__spritemap#sprite-spiriit"></use>
</svg>

You will need aditional setup if you have a backend like wordpress/drupal/laravel...
What is your environment?

Thx for replying.
I am using this package to generate the svg sprite. I dev mode everything works fine because dev environment provides the correct path to the svg sprite. But I don’t understand how to provide path to the svg sprite in dev mode.
<svg>   <use xlink:href="/__spritemap#sprite-spiriit"></use> </svg>
__spritemap string  will replace with correct path to the svg by dev env. But how to use it in production mode?

On production, the plugin replace /__spritemap with the vitejs assets directory.
Can you provide a small reproduction to understand what is not working for you? Do you have a particular vitejs configuration?

I think I might be facing the same issue. I'm building a library component in vue, to be used in another app.
When building, the spritemap does get put out to the dist/assets folder, but when importing my component (which uses vite-plugin-svg-spritemap for icons), the url in the build points to nothing (/assets/spritemap.4bdde031.svg).
When I manually copy the spritemap.4bdde031.svg to the apps public/assets folder, it works.

@vaaski did you find any solution?

@ve-esquela unfortunately not. development on the project i needed this for is paused.


TypeError: Cannot read property 'cursor' of undefined

I'm submitting a ...
[x ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
Do you want to request a feature or report a bug?
What is the current behavior?
Does not compile
What I do is:
start running project in Visual Studio
I currently use Webpack.
What is the expected behavior?
to compile
Please tell us about your environment:
dragNdrop-ng2.4.9
I get and error that starts with the following whenever I start running the application:
"Exception: Call to Node module failed with error: Error: Uncaught (in promise): TypeError: Cannot read property 'cursor' of undefined
TypeError: Cannot read property 'cursor' of undefined"
It seems to only occur when starting a component that features draggable components. If I load the app without them it works fine.
Angular version: 2.4.4
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

same here

same here

Any update on this ?

I decided to revisit this and after lots of digging around i have found what my issue was. It was because I was setting the dnd-sortable-container directive on an <ng-container>. Changing it to a <div> fixed it.
I assume the issue is something to do with the type of element an ng-container is and nativeElement as the recent change that broke it for me introduced it was https://github.com/akserg/ng2-dnd/commit/6c08842989139430516e142cca74165fc0d36d1b#diff-f55efd0d37300ccf1f787abc6aa003af
Hope this helps you guys out @natilivni, @imraqes, @NateGriffin


Malware kdevtmpfsi using the entire CPU arising from the php-fpm container

Description:
When I deployed the production version, there was malware that used up the entire CPU.
This problem is probably coming from the php-fpm container.
When I stop the php-fpm container, the CPU is freed.

Is redis used?
#2451

Is redis used?
post your docker-compose ps result
#2451

Yep, i used Redis but I have set a password for it.

The photo above is the result of docker-compose ps.
Thanks!


refactor: move data process logic to utils

Description
data converting logic better not to be inside components code
also:
i added some more tests here for data converting util
screenshots
<-- paste here -->

No prob, i will revert the move to utils.
Will only leave the tests
On Wed, Oct 11, 2023, 17:32 NoamGaash @.***> wrote:

@.**** requested changes on this pull request.
Hi,
Thanks - I appreciate the good will,
but I don't agree with the perception that all "logic" should be put in
the utils file.
Code should be arranged by context, and abstraction should serve code
readability.
What do you try to achieve here?
In src/pages/components/utils/index.ts
https://github.com/hasadna/open-bus-map-search/pull/111#discussion_r1355104218
:

@@ -38,3 +39,32 @@ export const mapColorByExecution = (planned: number, actual: number) => {
return 'red'
}
}

+export type HourlyDataList = HourlyData[]

// Convert gapsList into HourlyDataList structure
+export const processData = (gapsList: GapsList): HourlyDataList => {

having this name is a bad encapsulation - please refactor.
when someone reads the code, they should understand what this function is
doing without having to read the entire implementation
In src/pages/useGapsList.ts
https://github.com/hasadna/open-bus-map-search/pull/111#discussion_r1355105735
:

import { GapsList } from 'src/model/gaps'

-export const processData = (gapsList: GapsList): HourlyData[] => {
when processData appears in the context of the useGapsList hook, it's
easier to understand what it's purpose
In src/pages/useGapsList.ts
https://github.com/hasadna/open-bus-map-search/pull/111#discussion_r1355109769
:

const [hourlyData, setHourlyData] = useState<HourlyData[]>([])

useEffect(() => {
const fetchData = async () => {
try {
const gapsList: GapsList = await getGapsAsync(fromDate, toDate, operatorRef, lineRef)

   console.log(gapsList[0])

remove
—
Reply to this email directly, view it on GitHub
https://github.com/hasadna/open-bus-map-search/pull/111#pullrequestreview-1671396670,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKGMHWQYQ7G3VQPX4X4TXTX62UXLANCNFSM6AAAAAA54BKOP4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Azure Resource explorer

Azure Data Studio Version: 1.1.3

Steps to Reproduce:
Request to add active directory integrated authentication since windows auth is not supported by azure SQL.

We're tracking AAD support with this issue https://github.com/Microsoft/azuredatastudio/issues/13.  This is being actively worked on and should land in either Nov or Dec.  To use Azure Resource Explorer you can add an Azure account using the Account icon in the status bar.


zephyr: samples: Fix possible unbootable without bootloader

It may cause the text section linked into wrong place when the
application do not support bootloader.
Signed-off-by: Ding Tao<EMAIL_ADDRESS>

Just to be clear, I think unless I'm misunderstanding your purpose that this pull request should be closed.
If you're looking for a hello world sample that does not assume it's booted by mcuboot, please use the one in the Zephyr tree: https://github.com/zephyrproject-rtos/zephyr/tree/master/samples/hello_world

Hi @mbolivar thank you for the comment.
I'm try to run mcuboot on my application, but, sometimes I don't want the application to support bootloader. For test purpose or some other reasons I guess?.
To do that I can simply disable the CONFIG_IMG_MANAGER in Kconfig or application config file. However, the dts.overlay file is still working, which will make the application code to be load at address of SLOT0 at link stage (defined in board spec .dts) rather than load the application to the start of flash address.

While I can understand why that is convenient for you, it makes the application itself more complicated for everyone else to read.
One of the main reasons we include this application is to provide the smallest example we can for starting an application which uses mcuboot. The application itself doesn't actually do anything useful. It just prints hello world. So I don't think there's a good reason to add extra features like "boots directly without mcuboot", because the application isn't doing any other useful things except being loaded by mcuboot.
If you want to use an application which isn't loaded by mcuboot, then you can use the hello world which is already provided by Zephyr and which I linked to above.
So I'm sorry but this is not a patch which mcuboot should accept in my opinion.

Haha, you are right about it. I'm noticed that the dtc support conditional compilation, it is convenient for me to flash application without write any other scripts by add this commit.
Thank you take your time to review this.


[Blobs] Migrate existing Samples to samples/ folder

[x] Verified Docs in sphinx are generated proprly
[x] Got completely rid of test framework dependency
[x] All the samples are inside the samples/ folder and not anywhere else

Can one of the admins verify this patch?


Add a teal lexer

This PR adds a lexer for teal (https://developer.algorand.org/docs/reference/teal/specification/), the language used for smart contracts for the algorand blockchain.
I've tried to follow the coding standards used elsewhere in this project as closely as possible, but if anything's wrong please don't hesitate to let me know. Hope this can help.

Thanks! A few more things: You have to move all examples into /examplefiles/teal (i.e. also tests/examplefiles/example.teal), and it looks like you're missing the golden test files (you can generate them using py.test --update-goldens)

oops, thank you

Thanks. Just to make sure,<PHONE_NUMBER>777777777777777777777777777777777777777774MSJUVU is really supposed to be a single number?

Yeah, it's a crypto account address

Merged, thanks!

Thank you!


Frontend with Framework: React 3.3  Basic Hooks useState and useEffect

Description
Description
Contributors are required to create documentation and explain about Hooks useState and useEffect which should cover questions like :-

What is useState in React ?
When do we use useState ?
What is useEffect in React ?
When do we use useEffect ?
How to implement useState and useEffect with code (also give theory explanation of the code) ?
You can take 2-3 examples of each with codes and different types to explain this etc....
and also add some more points related to this topic.

Note :

Plagiarism is strictly prohibited.
Changes should be made inside the Frontend_Web_Development_React_Angular_Vue/ dir inside Frontend_Web_Development_React_Angular_Vue branch.
Files need to be added into the Week3 folder only.

Domain
Frontend Dev React/Angular/Vue
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

I would like to work on this issue.Kindly assign me this.

/assign

I would like to work on this issue. Kindly assign me this.

Go for it @shreya024

/assign

Is this issue still live? I'd love to contribute to this!

it is assigned to @shreya024

@shreya024 what is the update till now on this issue.

I have written the codes,output and overall documentation.Only explanation bit is left.


Improve diagnostic messages

This is a first pass on the text of diagnostics to make them follow the style convention: https://github.com/apple/swift/blob/master/docs/Diagnostics.md
Some errors are still defined using the FixableError protocol. That's my next step.

@swift-ci please smoke test


Upgraded the CRC crate from 1.x to 3.x

CRC 1.x has not been updated for over 4 years, this PR upgraded it to 3.x.

Codecov Report

Merging #930 (b6d17da) into main (78ffd0a) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #930      +/-   ##
==========================================
+ Coverage   73.89%   73.97%   +0.08%
==========================================
  Files         292      292
  Lines       31764    31768       +4
==========================================
+ Hits        23471    23500      +29
+ Misses       8293     8268      -25

Flag
Coverage Δ

unit-tests
73.97% <100.00%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

crates/primitives/src/forkid.rs
96.30% <100.00%> (+0.05%)
:arrow_up:

crates/net/network/src/peers/manager.rs
83.42% <0.00%> (+0.18%)
:arrow_up:

crates/transaction-pool/src/pool/txpool.rs
59.37% <0.00%> (+0.50%)
:arrow_up:

crates/net/eth-wire/src/capability.rs
63.10% <0.00%> (+0.97%)
:arrow_up:

crates/transaction-pool/src/test_utils/mock.rs
59.70% <0.00%> (+6.59%)
:arrow_up:

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


Compatibility Issue with OpenAI Python Package (Version 1.1.1)

Description
Compatibility issue with the Langchain library due to the recent changes in the OpenAI Python package (version 1.1.1). The Langchain library relies on certain structures and imports from the OpenAI package, which have been modified in the new version. Specifically, the issue seems to be related to the following changes:

In the Langchain code, the error handling imports in langchain/llms/openai.py at line 90 were based on the older structure of the OpenAI package. In the newer version, these imports have been restructured and are available in openai._exceptions.

In langchain/llms/openai.py at line 266, values["client"] = openai.Completion is no longer valid in the new version of OpenAI (version 1.1.1).

!pip install langchain openai

from langchain import OpenAI
import os

os.environ["OPENAI_API_KEY"] = "key"

llm = OpenAI(
    model_name="text-davinci-003",
    temperature= 0.2,
    max_tokens= 64,
    openai_api_key=os.environ["OPENAI_API_KEY"],
)

Also

Note:
To avoid the above error, users should downgrade the OpenAI package to version 0.28.1.
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
!pip install langchain openai
from langchain import OpenAI
import os
os.environ["OPENAI_API_KEY"] = "key"
llm = OpenAI(
model_name="text-davinci-003",
temperature= 0.2,
max_tokens= 64,
openai_api_key=os.environ["OPENAI_API_KEY"],
)
Expected behavior
Langchain should work without errors when using OpenAI version 1.1.1.

Can confirm, openai 1.1.1 breaks langchain completely.  It seems the "openai.Completion"  calls being deprecated are the culprit.

As an adhoc Running pip install "openai<1.0.0"  helps mitigate the issue.

I can confirm that the app we have built is using the LLMChain (Legacy) class and we have this issue... its annoying as we started buidling at the start of the langcahi project and this is causing havoc! Thanks @kmishra1204 for your advice there I will look at doing that and hope LC can patch this !

We all want compatibility with the last version of the openai python library

pip install openai==0.28.1 resolved the issue.

You should use the Langchain extra package to install OpenAI:
pip install langchain[openai]
This will pull a compatible version.

I see that multiple newer versions of langchain has been released in the last weeks but it is not clear if any has provided full support for openai v 1.x.
Is there a plan for langchain to fully migrate to that, or do the newer releases already take care of this?

Newer versions of LangChain indicate that OpenAI v1 is supported.
openai = {version = "<2", optional = true}

Thanks for your reply. Sorry if this is a silly question but where do I look for that?
I downloaded the source code of release v0.0.340 Latest (https://github.com/langchain-ai/langchain/releases/tag/v0.0.340) and I checked the poetry.lock file, which still shows openai = ["openai (>=0,<1)", "tiktoken (>=0.3.2,<0.6.0)"] under [package.extras].

And yet, you are not going to be able to install this with conda, for example.
    ├─ langchain is installable and it requires
    │  └─ openai >=0.0.0,<1.0.0 , which can be installed;
    └─ openai >=1  is not installable because it conflicts with any installable versions previously reported.

... that's a dependency conflict...
If you use a lib that depends on a certain version of a library, it's normal to reject another version.
That's what dependency resolution is for. Otherwise you'll get issues at runtime, which this issue is exactly about.

Should be fixed, if anyone still sees issues with the latest versions of langchain-openai and the openai python sdk let us know

I'm running to bad request error by langchain embedding using openai. I confirmed calling openai embedding API directly works fine. Anyone know the root cause for bad request error? Any ideas?
Here are the versions:
Python 3.12.7
Name: openai
Version: 1.57.2
Name: langchain
Version: 0.3.11
Name: langchain-core
Version: 0.3.24
Name: langchain-openai
Version: 0.2.12
Name: langchain-chroma
Version: 0.1.4
from langchain_openai import OpenAIEmbeddings
embeddings = OpenAIEmbeddings(
model="text-embedding-3-large",
)
try:
vector = embeddings.embed_query("hello")
print(vector[:3])
except Exception as e:
print(f"An error occurred: {str(e)}")
An error occurred: Error code: 400 - {'error': {'message': 'invalid input type', 'type': 'api_error', 'param': None, 'code': None}}


Navigation from Favourites to /hi/:id with params results in 404

I modified the example code a bit and ran into this issue:
`
@main
struct ExampleApp: App {
var body: some Scene {
    WindowGroup {
        Router.tabbed(tabs: [
            TabRoute(
                path: "/",
                builder: { _ in HomeView() },
                labelBuilder: { Label("Home", systemImage: "house") },
                children: [
                    Route(path: "details", builder: { _ in NamedView(text: "Details")}),
                    Route(path: "hi/:id", builder: { _ in NamedView(text: "Details")}, isModal: true), // add parameter
                    Route(path: "modal/:id", builder: { _ in ModalRouter() // ignore this

                    }, isModal: true),
                ]
            ),
            TabRoute(
                path: "/favorites",
                builder: { _ in FavoritesView() },
                labelBuilder: { Label("Favorites", systemImage: "star") },
                children: []
            )
        ])
    }
}

}
`
`
struct FavoritesView: View {
@Environment(Navigator.self) var navigator
var body: some View {
    VStack(spacing: 16) {
        Button("open \"/details\"") {
            navigator.go(to: "/details")
        }
        Button("say \"/hi\"") {
            navigator.go(to: "/hi/123123") // added params
        }
        Button("open unknown page") {
            navigator.go(to: "/url/that/does/not/exist")
        }

        Button("open modal") { // ignore this
            navigator.go(to: "/details/123123123")
        }
    }
}

}
`

Thanks for reporting the issue!
This is kinda related to the first issue you reported, so I updated the tests and the example app as well.


Add tests for parsing submission message classes

Add unit tests for parsing submission message classes.

Closed in #26.


Fix corelation between text setOutlineThickness() and getLocalBounds()

This fixes text crawling explained in issue #2129
Thanks a lot for making a contribution to SFML! 🙂
Before you create the pull request, we ask you to check the follow boxes. (For small changes not everything needs to ticked, but the more the better!)

[X] Has this change been discussed on the forum or in an issue before?
[X] Does the code follow the SFML Code Style Guide?
[X] Have you provided some example/test code for your changes?

Description
Please describe your pull request.
This PR is related to the issue #2129
Tasks

[ ] Tested on Linux
[X] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
Example code provided in the issue linked above

Seems reasonable, would like to see a before/after picture in a noticeable case with a large outline. Also, would we need to do the same for shapes?

I'm not sure what you mean? You have to run the example code provided to understand what it's about. The screenshots won't tell you the story I'm afraid, as they will look the same.

I think the example is okay to demonstrate the issue + fix.
For me the main question is, whether there's something that speaks against including it, just so we don't "fix" something that was intentionally designed that way. But if nobody knows about it, then it should be fine for me as well.

SFMLBuildMaster: Build this please.

Do you just want to fix this on master or also for 2.6.x?

Yes, also on 2.6.x please.

Do you want to retarget it or shall I? 😄

Please do. I'm not going to be in front of my PC today.

SFMLBuildMaster: Build this please.

Thanks for the fix! 🥳


How can I combine all the images to one single docker image?

Has anyone tried to create a single docker image from ION before? If so, what's the best way to do it? There is a pool of several public IPs on the host and I would like to assign ION as a standalone image to a static IP with all the necessary services in it.
Thanks :)

you can use：
docker-compose pull
docker-compose up


Reverse Only Letters

Accept Please!

Hey thanks for your contribution ❗. Keep up the good job and leave a star if you liked the repo .
Happy hacking 😉


NERDTreee err

When I open the vim, on the left I have an open NERDTree window, when I hide it with a combination of ctrl-t, the lightline disappears. Probably the lightline is tied to one window of vim. How to repair it?
https://drive.google.com/drive/folders/1_kFMc-z24Qd7GmTkrPdVJpZN1XA-G5Mk?usp=sharing

How about set laststatus=2 as described in README.md#introduction?

How about set laststatus=2 as described in README.md#introduction?

Yes, it helped. sorry for bothering ... This is because everything looked as it should, and then I discovered this error and forgot about this advice which is described in the way.


memory error

I am using the git cloned version of faucet as of 3/29/2017.  Faucet logs shows:
Mar 29 16:46:18 faucet.exception ERROR    Unhandled exception, killing RYU
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/faucet/valve_util.py", line 52, in __koe
    func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/faucet/faucet.py", line 420, in _packet_in_handler
    pkt = packet.Packet(msg.data)
  File "/usr/local/lib/python2.7/dist-packages/ryu/lib/packet/packet.py", line 67, in __init__
    self._parser(parse_cls)
  File "/usr/local/lib/python2.7/dist-packages/ryu/lib/packet/packet.py", line 76, in _parser
    proto, cls, rest_data = cls.parser(rest_data)
  File "/usr/local/lib/python2.7/dist-packages/ryu/lib/packet/icmpv6.py", line 130, in parser
    msg.data = cls_.parser(buf, offset)
  File "/usr/local/lib/python2.7/dist-packages/ryu/lib/packet/icmpv6.py", line 367, in parser
    options.append(option)
MemoryError

I don't have instructions to replicate it yet.  It is happening on my machine :-(  Will investigate more and add to this bug report.  But, I did want to make sure it is recorded.

That looks like the Ryu packet parser crashed trying to parse a packet in.
A tcpdump (of the OF channel) will be required to figure it out. We could
also catch and discard the packet if the parser fails in a way we can catch.
On Mar 29, 2017 5:24 PM, "shivarammysore"<EMAIL_ADDRESS>wrote:

I am using the git cloned version of faucet as of 3/29/2017. Faucet logs
shows:
Mar 29 16:46:18 faucet.exception ERROR    Unhandled exception, killing RYU
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/faucet/valve_util.py", line 52, in __koe
func(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/faucet/faucet.py", line 420, in _packet_in_handler
pkt = packet.Packet(msg.data)
File "/usr/local/lib/python2.7/dist-packages/ryu/lib/packet/packet.py", line 67, in init
self._parser(parse_cls)
File "/usr/local/lib/python2.7/dist-packages/ryu/lib/packet/packet.py", line 76, in parser
proto, cls, rest_data = cls.parser(rest_data)
File "/usr/local/lib/python2.7/dist-packages/ryu/lib/packet/icmpv6.py", line 130, in parser
msg.data = cls.parser(buf, offset)
File "/usr/local/lib/python2.7/dist-packages/ryu/lib/packet/icmpv6.py", line 367, in parser
options.append(option)
MemoryError
I don't have instructions to replicate it yet. It is happening on my
machine :-( Will investigate more and add to this bug report. But, I did
want to make sure it is recorded.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/REANNZ/faucet/issues/489, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEgZ1_f7pcIKFo1QaFn5JQgTmUX1Tuvpks5rqvY2gaJpZM4Mtt-v
.

One more data point is that this controller was managing a switch with DPDK ports.

That should not matter; the traceback shows a Ryu parser crash.
On Mar 29, 2017 5:29 PM, "shivarammysore"<EMAIL_ADDRESS>wrote:

One more data point is that this controller was managing a switch with
DPDK ports.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/REANNZ/faucet/issues/489#issuecomment-290266497, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEgZ1weJ3avrVIyTQm4GGWzswjQbl0r9ks5rqvd6gaJpZM4Mtt-v
.

Which version of ryu is in use here?
Also as per the Python documentation, basically the interpreter has run out of memory which may give you some clues:

exception MemoryError
Raised when an operation runs out of memory but the situation may still be rescued (by deleting some objects). The associated value is a string indicating what kind of (internal) operation ran out of memory. Note that because of the underlying memory management architecture (C’s malloc() function), the interpreter may not always be able to completely recover from this situation; it nevertheless raises an exception so that a stack traceback can be printed, in case a run-away program was the cause.

Ryu Version: 4.11.  I just upgraded it to 4.12 and will keep an eye on this.
# free -m
              total        used        free      shared  buff/cache   available
Mem:           7897         368        7084           1         445        7228
Swap:          8107          92        8015

# mpstat
Linux 4.8.0-22-generic (allbirds) 	03/29/2017 	_x86_64_	(2 CPU)

05:41:19 PM  CPU    %usr   %nice    %sys %iowait    %irq   %soft  %steal  %guest  %gnice   %idle
05:41:19 PM  all   23.13    0.01    0.55    2.13    0.00    0.03    0.00    0.00    0.00   74.15

Hi @shivarammysore just checking in whether you saw this issue again?
If not I opt we close the ticket since we don't have a repro case.

It is still occurring.   Attached is the TCP dump and the faucet.log file.  I think there is a memory leak ...
Also, top output:
top - 18:35:33 up 11:21,  2 users,  load average: 1.12, 1.20, 1.16
Tasks: 150 total,   2 running, 148 sleeping,   0 stopped,   0 zombie
%Cpu(s): 37.4 us,  0.6 sy,  0.0 ni, 59.9 id,  2.1 wa,  0.0 hi,  0.0 si,  0.0 st
KiB Mem :  8087516 total,   118792 free,  7895504 used,    73220 buff/cache
KiB Swap:  8302588 total,  5706432 free,  2596156 used.    18700 avail Mem

  PID USER      PR  NI    VIRT    RES    SHR S  %CPU %MEM     TIME+ COMMAND
 5525 root      20   0 10.920g 7.297g   2316 R 100.0 94.6  24:04.02 ryu-manager
    1 root      20   0   55856   4440   3248 S   0.0  0.1   0:04.14 systemd
    2 root      20   0       0      0      0 S   0.0  0.0   0:00.00 kthreadd
    3 root      20   0       0      0      0 S   0.0  0.0   0:00.06 ksoftirqd/0
    5 root       0 -20       0      0      0 S   0.0  0.0   0:00.00 kworker/0:0H
    7 root      20   0       0      0      0 S   0.0  0.0   0:02.99 rcu_sched

memory_error.zip

Looks like Ryu itself is getting upset by something.
The of channel dump doesn't seem to line up - it just has HELLOs over and
over with no other activity, and has timestamps for 1pm, but faucet.log has
timestamps for 6pm.
Is it possible please to get matching faucet.log and a pcap?
On Thu, Apr 6, 2017 at 6:37 PM, shivarammysore<EMAIL_ADDRESS>wrote:

It is still occurring. Attached is the TCP dump and the faucet.log file. I
think there is a memory leak ...
Also, top output:
top - 18:35:33 up 11:21,  2 users,  load average: 1.12, 1.20, 1.16
Tasks: 150 total,   2 running, 148 sleeping,   0 stopped,   0 zombie
%Cpu(s): 37.4 us,  0.6 sy,  0.0 ni, 59.9 id,  2.1 wa,  0.0 hi,  0.0 si,  0.0 st
KiB Mem :  8087516 total,   118792 free,  7895504 used,    73220 buff/cache
KiB Swap:  8302588 total,  5706432 free,  2596156 used.    18700 avail Mem
PID USER      PR  NI    VIRT    RES    SHR S  %CPU %MEM     TIME+ COMMAND
5525 root      20   0 10.920g 7.297g   2316 R 100.0 94.6  24:04.02 ryu-manager
1 root      20   0   55856   4440   3248 S   0.0  0.1   0:04.14 systemd
2 root      20   0       0      0      0 S   0.0  0.0   0:00.00 kthreadd
3 root      20   0       0      0      0 S   0.0  0.0   0:00.06 ksoftirqd/0
5 root       0 -20       0      0      0 S   0.0  0.0   0:00.00 kworker/0:0H
7 root      20   0       0      0      0 S   0.0  0.0   0:02.99 rcu_sched
memory_error.zip
https://github.com/REANNZ/faucet/files/904313/memory_error.zip
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/REANNZ/faucet/issues/489#issuecomment-292392670, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEgZ19ewuK0_iJkr4JkzqwWv4jRNLMhTks5rtZM_gaJpZM4Mtt-v
.

Thanks for looking at this so quickly.  I will get it tomorrow.  I am running a overnight test and can't disturb the current setup now.
/Shivaram

On Apr 6, 2017, at 6:54 PM, Josh Bailey<EMAIL_ADDRESS>wrote:
Looks like Ryu itself is getting upset by something.
The of channel dump doesn't seem to line up - it just has HELLOs over and
over with no other activity, and has timestamps for 1pm, but faucet.log has
timestamps for 6pm.
Is it possible please to get matching faucet.log and a pcap?
On Thu, Apr 6, 2017 at 6:37 PM, shivarammysore<EMAIL_ADDRESS>wrote:

It is still occurring. Attached is the TCP dump and the faucet.log file. I
think there is a memory leak ...
Also, top output:
top - 18:35:33 up 11:21, 2 users, load average: 1.12, 1.20, 1.16
Tasks: 150 total, 2 running, 148 sleeping, 0 stopped, 0 zombie
%Cpu(s): 37.4 us, 0.6 sy, 0.0 ni, 59.9 id, 2.1 wa, 0.0 hi, 0.0 si, 0.0 st
KiB Mem : 8087516 total, 118792 free, 7895504 used, 73220 buff/cache
KiB Swap: 8302588 total, 5706432 free, 2596156 used. 18700 avail Mem
PID USER PR NI VIRT RES SHR S %CPU %MEM TIME+ COMMAND
5525 root 20 0 10.920g 7.297g 2316 R 100.0 94.6 24:04.02 ryu-manager
1 root 20 0 55856 4440 3248 S 0.0 0.1 0:04.14 systemd
2 root 20 0 0 0 0 S 0.0 0.0 0:00.00 kthreadd
3 root 20 0 0 0 0 S 0.0 0.0 0:00.06 ksoftirqd/0
5 root 0 -20 0 0 0 S 0.0 0.0 0:00.00 kworker/0:0H
7 root 20 0 0 0 0 S 0.0 0.0 0:02.99 rcu_sched
memory_error.zip
https://github.com/REANNZ/faucet/files/904313/memory_error.zip
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/REANNZ/faucet/issues/489#issuecomment-292392670, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEgZ19ewuK0_iJkr4JkzqwWv4jRNLMhTks5rtZM_gaJpZM4Mtt-v
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

This bug seems to happen inconsistently only when I manage a OVS switch with DPDK enabled interfaces (tested with OVS v2.7 with DPDK).  I will close the bug for now and reopen when I can get more detailed logs than the one I have provided so far.


Set backgroundImage's fillMode to 'repeat' doesn't make it cover the whole area.

Platform
What platform is your issue or question related to? (Delete other platforms).

UWP

Author or host
Host
Version of SDK
AdaptiveCards.Redering.Uwp  3.0.2
Issue
I  want to use an image to fill the whole background of my adaptive card. By setting 'fillMode' to 'repeat',  I assume no matter the how small the image is, it will cover the whole area of the adaptive card. But seems this is not true. The image only cover part of the area.
Here is the template and data.
        const string templateJson = @"
{
    ""type"": ""AdaptiveCard"",
    ""version"": ""1.5"",
    ""backgroundImage"": {
            ""url"": ""${BackgroundImageUrl}"",
            ""fillMode"": ""repeat""
            },
    ""body"": [
        {
            ""type"": ""Container"",
            ""verticalContentAlignment"": ""bottom"",
            ""items"": [
                {
                    ""type"": ""Container"",
                    ""backgroundImage"": {
                        ""url"": ""${BackgroundImageUrl2}"",
                        ""fillMode"": ""repeat""
                        },
                    ""items"":[
                        {
                        ""type"": ""TextBlock"",
                        ""text"": ""${Name}"",
                        ""size"": ""medium"",
                        ""wrap"": true,
                        ""maxLines"": 2,
                        ""horizontalAlignment"": ""center""
                        }
                    ],
                    ""bleed"": true
                }
            ],
            ""height"":""stretch"",
            ""spacing"": ""none""
        }
    ],
    ""selectAction"": {
                ""type"": ""Action.OpenUrl"",
                ""url"": ""${Url}""
            },
    ""$schema"": ""http://adaptivecards.io/schemas/adaptive-card.json"",
    ""minHeight"": ""200px"",
    ""verticalContentAlignment"": ""top""
}";

var myData = new
            {
                Name = "hello world",
                Url = "https://office.com",

                // This iamge has larger dimension
                //BackgroundImageUrl = "https://neonstorage1.blob.core.windows.net/adaptivecardresources/orange.png",

                // This image has smaller dimension
                BackgroundImageUrl = "https://neonstorage1.blob.core.windows.net/adaptivecardresources/green.png"
            };

Here is the rendering result:

Thanks for the report! This looks like a recent regression. We'll investigate as soon as we can.

@lcmsft the problem is that the image you supply is very large by default (750 x 1350) :) I assume you wanted your image to be just the green rectangle, but instead if you open the link, you will see that your picture is white background with a little green rectangle in the top left corner. You will need to adjust your image to get desired behavor.

@beervoley You are right. Sorry I didn't notice it before.

@beervoley I just created a new small image
https://neonstorage1.blob.core.windows.net/adaptivecardresources/green2.png
I then use this image to render a 800x800 adaptive card.

This is what I get. The light green is image covered area.

@lcmsft can I ask what behavior are you trying to achieve?
I'm using a sample weather card and these are the possible outcomes (with the picture you provided) for the backgroundImage:
fillMode : repeat

fillMode: repeatHorizontally

fillMode: repeatVertically

@beervoley My understanding is no matter how large the adaptivecard is, backgroundImage should cover the whole area when fillMode is repeat.  But I didn't see it on my UWP sample.

@lcmsft can you check the size of your Rendered Adaptive Card?
It seems that though Grid is 600x600 the adaptive card is much smaller :)

@beervoley After I change minHeight from 200px to 600px, backgroundImage covers the whole area. So minHeight actually determines the real height, is my understanding right?

@beervoley Yes, you are right. I think it is because the card's height is smaller.


Package into nilearn-like, pip-installable program

Define classes
Decide on and configure input parameters
Generate and output regressors
Output log files, QC html

This has been pretty much addressed already


Stop Giving pre 2021, suggestions and replies for FLUTTER and DART programming, Kindly use at least Flutter 3.0, as all the suggestions and answers are obselete now

Its very frustrating to see and get answers which are not working at all

Duplicate https://github.com/microsoft/vscode-copilot-release/issues/513


Update CODE_OF_CONDUCT.md

Overview
TL;DR

Questions
<If relevant, write a list of questions that you would like to discuss related to the changes that you have made.>
Next Steps
<If incomplete, create a task list of items that are still being worked on within the Pull Request.>
Review
<If complete, or ready for :eyes:, use @mentions for quick questions, specific feedback, and progress updates.>

Thank you @pwasik371!

For those who don't want to peek at the diff, this PR is a small change to the grammar of the code of conduct. It doesn't change the meaning.


Everything's coming up blank. #17

Overhaul of API to support JavaScript loading of new webpage.
Docker creates standalone chrome container to parse page after Javascript has been loaded.
Lots has changed in this commit so some features may not be working the same as before due to changes in webpage
All configuration variables have been moved to .env for easier configuration
See readme.md about new docker command.

Hey, I see you are using remote Chrome instance for Selenium. Is it hosted by you or some third party? Will its address be changed?

it is part of the docker solution and can be run by anyone. If the pull
request is accepted hopefully the app can be hosted on heroku like before
with minimal changes to the end user.
I don't have plans to do any public hosting for it but the code is running
on my NAS right now and it's working great
On Fri, May 8, 2020 at 4:29 AM Arpit Choudhary<EMAIL_ADDRESS>wrote:

Hey, I see you are using remote Chrome instance for Selenium. Is it hosted
by you or some third party? Will its address be changed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/appi147/thepiratebay/pull/18#issuecomment-625771756,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYJPOX5XXE7VVJYE532XBDRQPUJFANCNFSM4M3XHGYQ
.

I'll do a test to see if I can deploy this on Heroku and update the branch with the needed files so that it's more straightforward.
Looking at the Heroku documentation for deploying dockers it's less intuitive than I imagined.

@appi147 I've tested a deploy on Heroku. By using buildpacks you can avoid any need to deploy multiple docker images or host the standalone-chrome anywhere.
By setting the following Environment variables and buildpacks the API can be launched as normal.

The app can be seen running at https://test-heroku-tpb.herokuapp.com/
I am using free dynos so I will not be leaving it up after this pull request has been merged.

Shutting dynos off for heroku, let me know if you want them back up to see them or if you want ownership.
Thanks.


Update changelog for EKS-A v0.15.3

Update changelog for EKS-A v0.15.3.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Drop python27 from travis.yml

Now that we have dropped support for python2, remove py27 from .travis.yml.
It is removed from tox.kni, but not from travis. So it gives build failing message, and reduces trustworthiness of library for newcomers.

Done


Documentation page references incorrect env property SENTRY_DSN

The documentation page https://docs.sentry.io/clients/php/integrations/laravel/ instructs you to use the .env value SENTRY_DSN but the 0.9 config file expects SENTRY_LARAVEL_DSN.
Having SENTRY_DSN set in .env on my local environment still worked correctly but when running on a staging environment, issues weren't visible in my dashboard, but an error ID was generated.

Setting SENTRY_DSN should still work since the PHP package should read that out correctly from the env variables (unless you use config caching), but yes that should be updated, will close this issue when it's done.

Ah yes that's the difference. The remote staging environment caches its config. Error IDs were still being generated though and the user feedback form was showing up ( app()->bound('sentry') && !empty(Sentry::getLastEventID()) were true) so it was behaving as if it was configured correctly.
I think the config change needs to be highlighted or or a warning shown if SENTRY_LARAVEL_DSN isn't set. In the case where you're on 0.8, upgrade to 0.9 and test locally without cached config files everything will work. You deploy to a cached config environment, no errors are shown and sentry seems to work but issues aren't submitted correctly. It could take a long time to realise that the config isn't correct.

Yes 100%, the doc change will be merged and probably live tomorrow, so this should not happen anymore.
Thanks for keeping me sharp 👍

Docs are updated and live, thanks again 😄


EC2: add support for encrypted volumes

It would be nice if we could create encrypted volumes. It shouldn't be too hard as https://github.com/mitchellh/goamz already supports this.

@rienvalkenaers Thank you for the request. We use https://github.com/goamz/goamz which does not include the encrypted support yet, they are similar packages though so porting wouldn't be a problem. We need to investigate this further to figure out where it would fit in the model since today we specify volumeType, IOPS, and availabilityZone for volumes. It would seem to make sense to declare it as another create option for encrypted for volume operations and the actual volume object itself. The other thing to consider is whether storage platforms in general would consistently provide this feature, which I believe would be an accurate statement.
We have also been considering the in-guest fuse based encryption method. This could be complimentary if applied to the mount operations where the volume map would specify it being attached/mounted including encryption.

Addressed by codedellemc/libstorage#355


AzureResourceManagerTemplateDeployment@3 task supports Bicep

Besides deploying ARM templates, this task now also supports deploying Bicep. See readme AzureResourceManagerTemplateDeploymentV3.
The ability to deploy Bicep is not yet described in this reference.
Please describe at parameter csmFile that besides an ARM template also a Bicep file can be used. And make a mention in the Remarks about the possibility of using a Bicep file as input (provided Az CLI version >= 2.20.0 is installed).

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

ID: 39f9c731-0172-2061-b972-3870d69ac358
Version Independent ID: 39f9c731-0172-2061-b972-3870d69ac358
Content: AzureResourceManagerTemplateDeployment@3 - ARM template deployment v3 task
Content Source: task-reference/azure-resource-manager-template-deployment-v3.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-tasks
GitHub Login: @steved0x
Microsoft Alias: sdanie

@steved0x  Is it known when the change will be made? And when it will be visible on the page in learn.micosoft.com?

@A-AontwSR I am publishing an updatetoday  which includes Bicep support for the task, in the inputs for templateLocation, csmFile, csmParametersFile, and a general note in the task remarks. This will be live later today. Thank you for letting us know.


Error while running the site of fedora

I am getting following error while running the site:
[nodemon] 1.12.1
[nodemon] to restart at any time, enter rs
[nodemon] watching: .
[nodemon] starting node server.js
app running on 5000
[Browsersync] Access URLs:
      UI: http://localhost:3001

UI External: http://<IP_ADDRESS>:3001
[Browsersync] Watching files...
events.js:183
throw er; // Unhandled 'error' event
^
Error: watch /home/devesh/Downloads/setups_electron/electronjs.org/node_modules/electron-api-historian/electron/vendor/boto/boto/sdb ENOSPC
at _errnoException (util.js:1024:11)
at FSWatcher.start (fs.js:1374:19)
at Object.fs.watch (fs.js:1400:11)
at createFsWatchInstance (/home/devesh/Downloads/setups_electron/electronjs.org/node_modules/chokidar/lib/nodefs-handler.js:37:15)
at setFsWatchListener (/home/devesh/Downloads/setups_electron/electronjs.org/node_modules/chokidar/lib/nodefs-handler.js:80:15)
at FSWatcher.NodeFsHandler._watchWithNodeFs (/home/devesh/Downloads/setups_electron/electronjs.org/node_modules/chokidar/lib/nodefs-handler.js:228:14)
at FSWatcher.NodeFsHandler._handleDir (/home/devesh/Downloads/setups_electron/electronjs.org/node_modules/chokidar/lib/nodefs-handler.js:407:19)
at FSWatcher. (/home/devesh/Downloads/setups_electron/electronjs.org/node_modules/chokidar/lib/nodefs-handler.js:455:19)
at FSWatcher. (/home/devesh/Downloads/setups_electron/electronjs.org/node_modules/chokidar/lib/nodefs-handler.js:460:16)
at FSReqWrap.oncomplete (fs.js:153:5)
[nodemon] app crashed - waiting for file changes before starting...

@dost4frnd I'm going to close this but feel free to comment if you are still experiencing this issue.


StatusBar color disabled

On Android 5.1.1 (Nexus 5)
Using io.palaima.debugdrawer:debugdrawer:0.1.1
The StatusBar color is disabled
Here's my code :
@Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        ButterKnife.inject(this);
        stock();
        if (BuildConfig.DEBUG) {

            mDebugDrawer = new DebugDrawer.Builder(this).modules(
                    new DeviceModule(this),
                    new BuildModule(this),
                    new NetworkModule(this),
                    new SettingsModule(this)
            ).build();
        }
    }

@Override
    protected void onStart() {
        super.onStart();
        if (mDebugDrawer != null) {
            mDebugDrawer.onStart();
        }
    }

    @Override
    protected void onStop() {
        super.onStop();
        if (mDebugDrawer != null) {
            mDebugDrawer.onStop();
        }
    }

The issue seems to be in the my stock() function where i'm setting a java android.support.v7.widget.Toolbar . If i remove the toolbar everything is fixed.
Thanks,

+1

Thanks for report.
I'll try to dig what causes this problem

Check demo app. I have changed actionbar with toolbar. Status bar color is changed to colorPrimaryDark and is not disabled. Also tried change status bar color programmatically (See MainActivity.java), works fine.

After checking again , the issue happens because i also use the com.mikepenz.materialdrawer library for my main navig drawer
Will try with an other lib

Same here. I'm using com.mikepenz.materialdrawer too. Let me know if you find a solution.
Thanks!

@nalmada  FYI, I set com.mikepenz.materialdrawer's Drawer with .withTranslucentStatusBar(false) and that resolved the problem for me. Hope it helps!

Yes it works (even if then the nav drawer doesn't follow material design guidelines).
Thanks anyway.


[Desktop][Mobile] Add Checkboxes to accept Terms & Conditions and Privacy Policy on Create Individual Profile Page.

As a user,
Given that I am on the Create your Profile page,
Then I should have an option (checkboxes) to accept Terms & Conditions and Privacy Policy.
Desktop:

Mobile:

Isn't this issue already resolved?? @tejasm2 where did you get this from?
Where are you checking??

Closing this as this already exists.

Seems like it is still an issue for the user profile! We have this already for ORG profile!

@idjevm thank you for clarifying this issue will be resolved with the #860 PR. I'm going to remove the "MVP-Critical"  tag


Customize the application name

Currently it just says "Playing PyCharm Community version" in discord. I would love a feature where I can somehow change it to something I want ?

Duplicate of #114


Fix #26

Fix #26.
Use clean config for each daterangepicker.
Fix not cleaning up after itself when scope is destroyed.
The original code did not clean up after itself when switching views, and also extended a global variable which is then applied to all daterangepickers. This PR fixes that.

@tcooc thanks for the fix, it definitly makes sense from a code perspective, however for me it does not fix #26, I still have the same issue. After the single view has been used, it still shows a single view for the next usage that is configured as range. I also get an exception in some cases that the remove is called on an undefined element, I haven't been able to figure out when and when not though, could be related to being in a modal.
I double checked the initialization, my problem seems to be related to the case that customOpts is actually undefined after it is parsed ...

@elm- Sorry to hear that. I had 2 separate daterangepickers on 2 different pages, one being a single daterangepicker, and this fixed it for me. It would be helpful if you could provide an example, since my use case sounds the same as yours.
(Also, just in case, did you update to this latest change? You'll have to do bower instal git@github.com:fragaria/angular-daterangepicker.git#ce00c1047f2abd504b3b1995e33bf95eeba9a81a since these changes haven't been released yet)

@tcooc I did not use bower, I just replaced the JavaScript in the downloaded component and verified that it was compiled into my vendor scripts, should do the trick.
I have my date pickers placed in modals, that might potentially be an issue as I know Angular-UI  has some issues with scoping in that case. I'll try to reduce it to a simple case (it is part of a 10k lines coffeescript app, there are probably other factors involved).
What I observed so far is that the customOpts variable evaluates to undefined and not the options array. That could be a reason.

Hello,
I merged your PR and released new version with it. @elm- you can try the bower version now.
Thank you!

@krtek also happening with 0.1.11 from bower, but I Just noticed that the instance is actually not cleaned from the body stack. every time I re-open a modal with the date range picker it is added to the HTML body. I'll investigate there.

Ok, I meant 0.1.12 not 0.1.11. The problem is that $destroy fails with, I think this is related to the modal where the directive is used in.
TypeError: Cannot read property 'remove' of undefined
    at http://localhost:9000/assets/js/vendor.js:58440:44
    at Scope.$broadcast (http://localhost:9000/assets/js/vendor.js:23669:28)
    at Scope.$destroy (http://localhost:9000/assets/js/vendor.js:23327:14)
    at http://localhost:9000/assets/js/vendor.js:60409:34
    at afterAnimating (http://localhost:9000/assets/js/vendor.js:60455:13)
    at HTMLDivElement.<anonymous> (http://localhost:9000/assets/js/vendor.js:60439:13)
    at HTMLDivElement.jQuery.event.dispatch (http://localhost:9000/assets/js/vendor.js:4641:9)
    at HTMLDivElement.elemData.handle (http://localhost:9000/assets/js/vendor.js:4309:46)

@elm- Did this fix the issue with the 2 daterangepickers having the same config? This change should have fixed that.
I understand that this doesn't cleanup properly when using modals, but everything should still work. I'm working on a fix for cleanup issue.

@tcooc yes, it fixes it. I had some time to look at it today and noticed that the options passed to the date range picker are actually the wrong ones, so there was also a bug on our side.
The way this happened: We have a global default template for the date range picker which we clone and customize for the individual usage. At one usage extend instead of copy was used, which resulted in the template being modified and therefore being applied to all other parts.
Offtopic: We started making all our constants and templates actual constants by providing them through functions, so there is no way to modify their global state, this one was not covered yet. I'm really looking forward to freeze / immutability support on JavaScript, maybe then I can get warmer with the whole frontend stuff =)
Thanks for your help, really appreciate it!


Warn on post table version 3

Currently it is a pass. Should warn on it:
http://typedrawers.com/discussion/1945/truetype-based-fonts-and-post-table-version-3-0

And that warning would be based on what spec?

Apple's. post v3 is clearly a problem with some rendering engine; and Apple's did clearly forewarn it.
I hope this is not arguing for the sake of arguing. AFAIC, any issues which is clearly a problem with some/any common rendering engines, should have a warning. Regardless of spec.

So fonts that is fully spec-complaint would produce warnings because some program some where is buggy? It is much easier to reduce this to a single warning “This is a font file, some application somewhere will handle some aspect of it incorrectly. You have been warned”, and call it a day.

Khaled, kindly I agree with hin-tak, warning for non spec issues is fine.
Erroring for spec issues is appropriate
On Jan 22, 2017 4:18 PM, "Khaled Hosny"<EMAIL_ADDRESS>wrote:
So fonts that is fully spec-complaint would produce warnings because some
program some where is buggy? It is much easier to reduce this to a single
warning “This is a font file, some application somewhere will handle some
aspect of it incorrectly. You have been warned”, and call it a day.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/Font-Validator/issues/47#issuecomment-274360348,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP9y4urSBQRnfohTzgvz0JI65OeGOd-ks5rU8eLgaJpZM4LqDm7
.

Firmware upgrades for printers (some program) is not a sensible demand to put upon users. Telling users that their printers have buggy firmware isn't practical, nor is asking users to upgrade/change OSes (again, some program) to get a better or correct rendering.
I was hoping this is a simple issue for contributors to have a go.  Anyway, it is quicker for me to just make the change, than to respond to trolling.
Since FontVal runs stand-alone and just produces a report (and don't really do any actually rejecting/etc like otc), I'd just like the report to be as detailed as possible. It is up to the person reading the report to decide which parts are relevant to him/her; in the extreme case, he/she should stop reading the report if he/she does not see any item of interest.

The printing stuff is red herring, it is just an outdated note in a decade old spec that Apple is not known for keeping it up to date.
The Apple issue that actually triggered this (http://typedrawers.com/discussion/1945/truetype-based-fonts-and-post-table-version-3-0) has nothing to do with printing. It is an undocumented autohinting feature that seems to depend on an unknown number of variables  and its extend is not even clear. Jumping quickly to just trigger warning here s premature at the best and even harmful on the long run as excessive unnecessary warnings tend to cause people to ignore them altogether and actually useful warnings become a needle in a haystack.
Not to mention that checking for post table version is not even enough, as evidenced from the forum discussion above, having glyph names other than what Apple expects (which is not clear either) will trigger the hinting issue. You can even have version 1 post table with 0 glyph names or partial coverage of the glyphs in the font, which will pass the check promoted here but still potentially trigger the autohinting issue (and the mythical printing issue).

Good point Khaled, a post table version 1 can cause the bug just as well.

I don't think so. The other post table version explicitly encodes the glyph names. Post table version 3 does not. hence the issue with non-conformant/unexpected glyph names.
Anyway, I am fine with adding more warnings on post table version 1 too, if you can identify such issues.
As I have repeated said a few times... I fully intend people to skip over and ignore part of the reports they don't like or think irrelevant to their usage. The warning message clearly says "Apple said...", so if you intend your fonts to work on microsoft platforms only or whatever, you are free to ignore it. Not sure why this is even worth going over and over about. The reports are meant to be as detailed as possible, as far as I am concerned. The reader decides what they want to do with that information.


something wrong

Output
lchdeMacBook-Pro:~ lch$ phpbrew switch 7.1.3
You should not see this, if you see this, it means you didn't load the ~/.phpbrew/bashrc script, please check if bashrc is sourced in your shell.

i have add [[ -e ~/.phpbrew/bashrc ]] && source ~/.phpbrew/bashrc to
.bashrc

plz read README.md
did you execute phpbrew init?

i did ,but not helpful
and i have give it up ,so sad

@yygxlin My understanding is you'd want:
phpbrew switch php-7.1.3

The switch command takes the name of the install, rather than just the number. Run phpbrew list to see the installed PHPs.

phpbrew switch can work with and without the php- prefix same as phpbrew use.

The issue is outdated and will be closed.


Added select tag feature

It is possible use select tags in formFields

Hey thanks for the PR!
Before reviewing it, could you please expand a bit more about the issue this is solving? It's not be clear to me at the moment, even after reading the #5 ticket.

Hello,
In fact tis PR was made to solve another issue i had (use select/option). As it was my first PR on github, i tried to solve an easier problem. I'm haven't figured out how solve the #5 ticket.

Hi!
I've left some comments on the PR that would need either correction or clarification in order to merge it.
Here are a couple of additional changes that this PR needs in order for me to merge it:

PR's title describes a problem, rather than a change that fixes or adds something. The latter should be preferred.
It seems that this PR provides multiple unrelated features/fixes. It would be better to split each feature/fix into a separate PR. Ideally separate issues describing problems or possible improvements and separate PRs linked to them.
I've just added feross/standard compliance to the project. This adds some styling rules for new code. Please rebase against master and make sure that the code comply to the feross/standard rules.

This are rules I'd like to enforce on my repositories to make contributions easier to manage.


update ME32 rule violation message to include violating measures

TOPS-839

I can only imagine how frustrating it is for a TM to have to ask a DE / Dev to find out which measure is overlapping when faced with an ME32 message but no other information.

Why

All errors should give enough information for a TM to process and update the workbasket in order to resolve a rule violation. This is the first attempt to expose key information that will make TMs work more autonomous

What

Really simple change, adding the measure SIDs to the end of the violation message
Example : There may be no overlap in time with other measure occurrences with a goods code in the same nomenclature hierarchy which references the same measure type, geo area, order number, additional code and reduction indicator. Clash with Measure SID 20198035.

Checklist

Requires migrations? No
Requires dependency updates? No

Links to relevant material
Ticket : TOPS-839 (https://uktrade.atlassian.net/browse/TOPS-839)

Looks good to me and makes sense. All for better error messages in the UI.

Codecov Report
Base: 93.52% // Head: 93.52% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (1f5adbe) compared to base (cc77933).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #733   +/-   ##
=======================================
  Coverage   93.52%   93.52%
=======================================
  Files         345      345
  Lines       23091    23095    +4
  Branches     1786     1787    +1
=======================================
+ Hits        21596    21600    +4
  Misses       1160     1160
  Partials      335      335

Impacted Files
Coverage Δ

measures/business_rules.py
87.25% <100.00%> (+0.12%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Raised error 2 while making

I cloned the repo and ran the make command, then error 2 was raised. What does this mean? Please take a look.
I have the latest go version 1.16.6.

$ make
go build -ldflags '-X github.com/vesoft-inc/nebula-br/pkg/version.GitSha=8e994dc -X github.com/vesoft-inc/nebula-br/pkg/version.GitRef=master' -o bin/br main.go
go: downloading github.com/olekukonko/tablewriter v0.0.5
go: downloading github.com/vesoft-inc/nebula-go/v2 v2.0.0-20210608032721-3e7bf449d35f
go: downloading golang.org/x/net v0.0.0-20201110031124-69a78807bb2b
go: downloading golang.org/x/crypto v0.0.0-20200622213623-75b288015ac9
# runtime
/usr/local/go/src/runtime/mgcsweepbuf.go:87:80: memstats.gc_sys undefined (type mstats has no field or method gc_sys)
/usr/local/go/src/runtime/mgcsweepbuf.go:106:102: memstats.gc_sys undefined (type mstats has no field or method gc_sys)
make: *** [build] Error 2

$ bin/br version
-bash: bin/br: No such file or directory

maybe you can check this. it seems problem of go environment.


Multiple issues with binary package AND build dependencies appear to no longer be maintained.

Download and initial execution of framework functions, but no prompt is displayed once within the shell framework. The only thing visible is an indented cursor. i.e ----> | . This makes navigation difficult.
During attempt to use the account.checker module, received error can't load website listing.. Could not readily discover exactly how to remedy this.
During use with phone.* modules encountered difficulty with providing PATH for vcard.
Went to attempt to build binary to hopefully resolve encountered issues. Url for glide dependency "cli' has changed for manifest.
Glide's website is no longer active, thus the installation script for glide is not accessible. Project appears to not have been touched in over three years.

Hello,
Thank for this report.
I work on new version of operative framework written in rustlang, i will push it very soon.
I think this version, solve many error.

@graniet Oh nice, exciting news! I have used it nearly a year ago without any issues before, and I really liked the CLI interface and structure.  Very robust framework. I look forward to the next release.
Looks like I need to begin learning rust myself, it is exploding in popularity.


Returning Oauth2 tokens

Hello,
I've been struggling with this for a while, so pardon me if it is documented or I've missed it.
I have an application which uses Oauth2. However, djang-rest-auth seems prepared to return single (and most importantly, unique per user) tokens only, is that correct?
I have tried to use the REST_AUTH_TOKEN_MODEL setting to set the model of the RefreshToken (currently oauth2_provider.models.RefreshToken, a well known Oauth2 toolkit for Django), and the TOKEN_SERIALIZER setting to a serializer where I return the given RefreshToken, plus the access token assigned to it. I believe I still would need to set REST_AUTH_TOKEN_CREATOR however this seems unclear to me and it appears the toolkit is managing tokens just fine - either way this setting is unneeded until I fix the issues.
TLDR
Even by using the settings provided to use a custom token system, they seem lacking when using non-unique-per-user tokens such as Oauth2 - the error I get is the following:
  File "/usr/local/opt/pyenv/versions/xxxxx/lib/python3.4/site-packages/rest_auth/views.py", line 83, in post
    self.login()
  File "/usr/local/opt/pyenv/versions/xxxxx/lib/python3.4/site-packages/rest_auth/views.py", line 59, in login
    self.token = create_token(self.token_model, self.user, self.serializer)
  File "/usr/local/opt/pyenv/versions/xxxxx/lib/python3.4/site-packages/rest_auth/utils.py", line 15, in default_create_token
    token, _ = token_model.objects.get_or_create(user=user)

oauth2_provider.models.MultipleObjectsReturned: get() returned more than one RefreshToken -- it returned 2!

Is there a workaround around this? Am I doing something wrong? Or would the project need to further evolve?
All I can think off is to create a custom proxy class that manages the token logic for me and leaves it all ready for django-rest-auth.
Thank you for your time, even with this issue, your project is saving me time :)

Regardless of this, I'll just use the Oauth2 provider API for authentication, and django-rest-auth for the rest of the calls


Fix typos in LSP specification

Fix typos in the specification.

@microsoft-github-policy-service agree

@aeschli Can you please approve this PR?


FailedScheduling when installing on fargate

Describe the solution you'd like
external secrets should be able to install on was fargate or at least better documentation regarding integration using fargate vs ecs
What is the added value?
Better documentation for the long run.
Give us examples of the outcome
When installing external-secrets through helm, the install goes perfectly but when I check the events for the external-secrets namespace, I'm presented with the error below. Looks like a fargate specific taint but I haven't been able to find any information about it.
❯ k get events -n external-secrets
LAST SEEN   TYPE      REASON              OBJECT                                                   MESSAGE
31s         Warning   FailedScheduling    pod/external-secrets-6bbdcfc778-d5bjt                    0/5 nodes are available: 5 node(s) had taint {eks.amazonaws.com/compute-type: fargate}, that the pod didn't tolerate.
3m13s       Warning   FailedScheduling    pod/external-secrets-6bbdcfc778-zsmbz                    0/5 nodes are available: 5 node(s) had taint {eks.amazonaws.com/compute-type: fargate}, that the pod didn't tolerate.
31s         Normal    SuccessfulCreate    replicaset/external-secrets-6bbdcfc778                   Created pod: external-secrets-6bbdcfc778-d5bjt

Observations (Constraints, Context, etc):
Cluster is on k8s version 1.22 and running on AWS Fargate
Installed with the following commands
helm repo add external-secrets https://charts.external-secrets.io
helm install external-secrets external-secrets/external-secrets -n external-secrets --set installCRDs=true

Hi @ultratin ! Have you set any fargate profiles for your installation? I believe they are needed if you intend to run pods on it (see more at https://docs.aws.amazon.com/eks/latest/userguide/fargate-profile.html)

I see, that makes much more sense as a lot of people have mention this repo as the best way to mount secrets to fargate. I'll take a look into it, would this be a good candidate for docs update?

we always welcome PR to docs! I think you can put it into the examples session :)

Some guides for it: You need to add your .md files to docs/ folder, then add a reference to it in hack/api-docs/mkdocs.yml


Resetting the password does not work.

I am running the plone/volto on http://localhost:3000/ but after registration for Login when I go to password reset page . The reset button is not working and I wouldn't got any password reset email to my gmail account.

@vinay72 this is a known issue that is non-trivial to solve. Thanks for reporting!

See https://github.com/kitconcept/kitconcept.voltodemo/pull/1 for progress on this issue as it needs to be fixed/patched on the backend for now.

We should write a Cypress test in kitconcept.volto to fully test the password reset. This is an important feature and we have to make sure it works. We can copy over the test and the code to Plone 6 once it is ready.

The first issue and sort of solved last year was overriding the e-mail which is sent, we needed a volto.domain setting for this in the Plone backend.
I got stuck last year at the 2019 PloneConf sprint on:

fixing multilingual/i18n in the frontend reset password pages.
resetting a password based on providing your e-mail address or your user-id. Plone/restapi expects only user, the frontend form now asks for e-mail.

Hi @tisto,
Can I pick this issue as my first contribution ?

@aryan7081 please read and follow First-time contributors, especially Things not to do, Contributing to Plone, and Contributing to Volto.


[C++] Fix LZ4 support build for VS 2019 & 2022

This fixes building arrow/parquet with LZ4 support on Windows for Visual Studio 2019 & 2022.
Changes:

PlatformToolset=v140 requires Windows SDK 8, which is not normally installed with VS 2019 or 2022. I changed it to use the toolset available for the current build system, which will be v142 or v143 in recent VS versions.
The LZ4 library project doesn't have "Relwithdebinfo" configuration, so when trying to build arrow/parquet with relwithdebinfo, it fails. I added a condition to use Release for LZ4 when the parent arrow/parquet is being built using with relwithdebinfo.

With above changes, I could create arrow/parquet dlls with LZ4 support to be used internally at Microsoft.

LZ4 provides .sln for Visual Studio 2017 https://github.com/lz4/lz4/tree/dev/build/VS2017 and CMake files https://github.com/lz4/lz4/tree/dev/build/cmake . How about using CMake files like Zstandard  https://github.com/apache/arrow/blob/master/cpp/cmake_modules/ThirdpartyToolchain.cmake#L2319 when CMake is 3.7 or later?

@pykello Could you perhaps open a JIRA ticket for this?  See https://arrow.apache.org/docs/developers/bug_reports.html for how to do this.

@pykello Would you like to follow up with this and address the comments above?

@kou Since the PR author is responsive, I don't know if you would like to take this up.

@pitrou sorry for not being responsive. Closing so I reopen it properly in future when I have more time.

No problem. I've opened a JIRA issue for this: ARROW-17348