Posix message queues leak.

During testing, I notice that I sometimes get MSGQ test failures
despite having the "queues per process" configuration set to
a high enough value, and rebooting fixes the issue.
After one spate of reboots, I went and did some digging, and we
do seem to be actively leaking Posix Message Queues, at least
during testing: posix message queues have identities, and can
stick around when nobody has them open, and of course there is
a finite limit to them in the system.
Fortunately, /dev/mqueue shows us all of the queues that exist
in the system, and if you remove the entry in /dev/mqueue, the
message queue itself is removed.
I have implemented code in my testing scripts that aggressively
watches for message queue leaks during testing.
What I am seeing so far:

The CFE "Run the Core until the deadman timer trips" test
can leave behind many queues.

The OSAL "queue timeout test" can leave behind a queue.

It would be worthwhile to assure that only operations unique
to unit testing (such as "kill -9 $CFECORE" ...) leak these.

Trac comment by abrown4 on 2016-01-14 13:08:47:
I also hit this during development of a CFS app using Eclipse in Linux.  Eclipse kills the process rather than sending SIGINT - which is trapped and used for a (more-orderly?) cleanup.  It was time-consuming to track down and other Linux-based development (even with other IDEs) will likely hit it.
It would be preferable that unit tests don't leak either b/c leaks in the test environment would generate false test failures.
Leaks are bad in general.  Can this get plugged?

Imported from trac issue 99.  Created by glimes on 2015-09-10T16:45:02, last modified: 2019-08-14T14:11:46

Trac comment by dcmccom3 on 2018-04-25 06:47:56:
Proposed fix, 1-25-2018 email from Thad Fleming (JSC-EG)[ODYSSEY SPACE RESEARCH LLC]:
Looks like the posix-ng OSAL has a fix for the mqueue leak bug. It's
extremely simple: just call mq_unlink in OS_QueueCreate after the
queue is created successfully. The message queue will remain open
while the process runs, but it won't appear in /dev/mqueue.
For bonus credit, go ahead and remove the other call to mq_unlink in
OS_QueueDelete, since that will always fail.

Trac comment by jphickey on 2019-03-26 09:07:13:
Adding this to the group of bugs that are fixed by using the "NG" architecture. (this proactively unlinks the persistent name so the queue does not persist if the creator process dies)


Fix incorrect error logging in Stats indexer and OAuth2

Fixes two cases of bad logging:

Handle ErrAlreadyInQueue in stats indexer
Fix incorrect logging in oauth2.go

Fix #12380
Fix #11945
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Please send backport

I've just sent the backport in https://github.com/go-gitea/gitea/pull/12387


Keyboard focus is lost on tab navigating into ComboBox with no selection

Describe the bug

If you tab to a ComboBox which doesn't have a selection and you press Enter, the keyboard focus is lost and requires you to press down arrow key in order to get focus on the first element.
This causes another issue if you have many items in the ComboBox (more than the MaxDropDownHeight), then when you open the ComboBox the middle of the ComboBox item list is visible rather than the first item.
Steps to reproduce the bug

Steps to reproduce the behavior:
The issue can be reproduced on Xaml Controls Gallery by commenting out this line https://github.com/microsoft/Xaml-Controls-Gallery/blob/22c5fd1b7d72b02b9a7885232c22bcb3e92d3251/XamlControlsGallery/ControlPages/ComboBoxPage.xaml.cs#L77

Tab to the ComboBox without selection
Press Enter
Expected behavior

Keyboard focus should be on the first element as there is no selection
Screenshots

Focus issue:

Middle of the list visible by default and not the top:

Version Info

NuGet package version:

Microsoft.UI.Xaml 2.5.0-prerelease.200923002
Can also be reproduced with winRT Xaml Islands

Windows app type:

UWP
Win32

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2020 Update (19041)
Yes

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context

The issue can be reproduced on PowerToys Keyboard Manager https://github.com/microsoft/PowerToys/issues/7070

Does this happen on the Touch Keyboard?

I have not tried it with the Touch keyboard, but it happens on normal physical keyboards as well as the On-Screen keyboard. I added the use of On-Screen keyboard in the gif so that the repro steps are clear.

Will be fixed in the next version of WinUI 3.

@llongley is that 1.1?

Yes, it will be in 1.1-preview3


Mass by volume percentage calculator

Closes #614

@S-ishita resolve the conflict

done @Srijita-Mandal ..kindly increase the difficulty level


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #27.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Do we close this and try again from master?

Sure, I think that's better.


run: validate metadata

Validate metadata before running the gadget

#2512 changed the way how the matadata was validated (all validation was almost removed). I think it's still a good idea to validate the whole metadata before processing the gadget and in the same place. This will allow to use the same logic to validate the metadata also when building the gadget.

A deeper discussion will be the role of the metadata now that we agreed to make the macros to define tracers and other mandatory in #2560. Perhaps things like MapName and StructName aren't needed anymore now that those are contained in the eBPF program itself?.
Any thoughts on this?

Perhaps things like MapName and StructName aren't needed anymore now that those are contained in the eBPF program itself?.

Yeah, the metadata file would have only the struct and eBPF params sections (BTW, we can now drop the params defaultValue field). We don't need to associate the structs to the data sources anymore, right?
In other words, with the macros we define the behaviour of the data sources and params. With the metadata, we configure only the "style" and documentation things.


Use secure standardized network rpc interface.

Currently RPCUDP is used for the network RPC interface, this is inadequate for the following reasons.

UDP does not guarantee order, congestion control or delivery.
None of the calls are authenticated.
The traffic is not securely encrypted.

The following should be done:

A standard network interface should be chosen and added to the storjspec.
RPCUDP should be replaced with a compatible SecureRPC lib.

For encryption ECIES with this python implementation.
ECDH could be used for the key exchange.


Has nav2djs library been discontinued? Unable to link it using any CDN link.

I'm trying to use nav2d in my React JS project with ROS2. I'm unable to get nav2d work since it's CDN link isn't working. Has it been discontinued? Or is there any alternative?

CDN是挂了 不过你可以通过引入源文件的方式加载
比如那个ros2djs  https://github.com/RobotWebTools/ros2djs


Avoid use of global Chart inside function

This is to support environments which may not have access to the global Chart object anymore after the initial setup (chartjs-node-canvas).

Thanks, LGTM


A few inconsistencies...

I found a few inconsistencies between the implementation and the paper...

Eq. 5 is not the same as line 33
Eq. 13 is not the same as line 49

and in Fig. 6, you are actually plotting S', defined in Eq. 12, instead of the original S, defined in Eq. 8...

@SubtleMuffin you are right, thanks for spotting the bugs.
Ad .1. in line 33 should be (n-1) instead of n. I coded this operation differently.
Ad. 2. I omitted the parentheses in eq. (13). The code is fine.

Cool :)


Rename Astronomy curriculum in curriculum list to indicate that it is Data Carpentry

As shown in https://github.com/datacarpentry/astronomy-python/issues/133, the astronomy curriculum shows up in the curriculum list as "Foundations of Astronomical Data Science" and is not marked as belonging to the Data Carpentry lesson program as are all other DC curricula. This is a low priority issue, but I wanted to put on the list for when there is capacity or for the next time this list is updated.

Wow! That was fast! Thank you!


Increase icon size + improve ability to recognize icons in Jupyter extension

Icons are currently very small and hard to discern what they represent because of their size and possibly because of the number of elements (lines, shapes, etc) in the icon. That means that a user might frequently have to hover over an icon to understand what it does. Attached are some screenshots of the icons I noticed. I'm wondering if it would be possible to increase the size of the icon / use thinner lines for the icons / simplify them for improved UX.

Thanks for the feedback. We're changing several icons already, for example the variable explorer will be changed to
 and in the new native notebooks interface, clear cell outputs will be changed to

We'll pass that feedback along regarding the export to Python script icon.
I'm wondering if the size of the icons we use should follow the editor.fontSize setting. That would let users configure how large the icons are, but it would potentially cause size mismatches with the rest of VS Code, which doesn't seem to use that setting for icons.

Thanks for your feedback. This issue will be addressed once VS Code adds native support for editing notebooks. Please see https://github.com/microsoft/vscode/issues/91987. You can experiment by using VS Code insiders, which defaults to the new notebook experience.


Manual promotion to maven cetral via gh

Right now to promote the staged package to maven central, we have to manually close repo via nexus gui as per https://central.sonatype.org/publish/publish-gradle/#distribution-management-and-authentication
There are some plugins to gradle, however, that could be used to move this flow to github actions (like https://github.com/gradle-nexus/publish-plugin/).
The objective of this task is to assess possibilties to automate the promotion of the package, and create a new gh actions flow that would handle this upon manual user interaction (https://docs.github.com/en/actions/managing-workflow-runs/manually-running-a-workflow).

also docs should only be deployed (built?) after a successfull promotion of a new release


Updated Exit script in CDTS v5.0.0

Do you have any suggestions for executing server side activities, such as closing a secure session, when a user clicks on an external link?
If I configure the Exit plugin to use the popup dialogue, there doesn't appear to be any opportunity to execute any server side code if the  user clicks on the Yes button of the dialogue.
Using the intermediary page, I can present a warning message like the one from the popup option.  I don't do any session clean up until the client confirms that they wish to exit.  It appears the only access to the destination link is to use the following code:

This renders an anchor tag on the page.  If the user clicks on that anchor on the page, I don't get the opportunity to close the session.  There doesn't appear to be any mechanism that makes the destination URL available to be processed on the server so that exit processing can occur before redirecting the user.
While the previous version was vulnerable, you at least had the opportunity to sanitize and validate the destination URL and close the user session prior to redirect.

I modified the script to add an additional condition statement and use this with the modal popup configuration.  I load this script on each page.  If you have other modal popups on the page then this may need to be refined a bit more.  So far it's been working good in testing.
    const elem = e.target;
    if (!elem || elem.tagName.toUpperCase() !== 'A') return true;

    // additional condition for the modal popup Yes button.
    if (!elem.matches(".popup-modal-dismiss")) return true;

    const linkLocation = new URL(elem.href);


fix: config maps can now be used in setup if they already exist

Description
Config maps were trying to be created more then once
How Has This Been Tested?

Run e2e tests with no tests enabled
bike-rentals-test-data config map should be created
Run e2e tests with no tests enabled and there should be no error

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/approve
/lgtm
This resolved the issues where running make go-test against an existing test model failed due to an existing test-data ConfigMap


Incompatible with Chrome on Android

I'm running Lollipop with Chrome 39.02171.59 and the picker instantly opens and closes, making it unsable. Demo GIF below:

Hello,
Thanks for reporting issue.

DateTimePicker opens and closes
The issue of closing DateTimePicker on Mobile devices is fixed, you can check Demo

Issue while focusing DateTimePicker with TAB
Support for TAB-based navigation is added. You can check Demo and Keyboard Navigation

Thank You.


simplify named! macros with the :vis macro arg type

issue introducing :vis for visibility identifiers
suggested in #566 by @derekdreery

There might be more visibility modifiers now, or in the future, that would be supported by using :vis.


fixed dummy manifest path not being correctly parsed on windows

Hello, while trying to use the crate on Windows, the dummy manifest path was being passed incorrectly to the Toml parser, causing an error to occur. As paths in Windows use \ instead of /, it interpreted it as an escaped character. Due to this, I added an OS verification which adds an extra \ in case the OS is Windows.

Thanks! Does this mean that you always have to write all Windows paths in the Cargo.toml with a double \ even when writing the file manually?

That is what it seems. I couldn't find any actual documentation on this behavior, but from a couple of Stack Overflow posts people recommend using double inverted slashes for paths in Cargo.toml.

Hmm ok, this still seems strange to me.
I'm currently working on creating a test framework for this crate and it seems like it still errors on Windows even with your fix.

I got another error after this one, but it was unrelated to this fix. I thought at the time it was due to me using the crate in a [no-std] environment, so I didn't explore further. I will give it another try later today in a separate crate and so some more testing.

Ok, thank you! For now I'm going to revert this PR since it also seems to break the build on Linux.
I'll set up a test framework in a moment so that we can automatically check a new PR for this on all major platforms.

I reverted this PR in #13 and created the pull request for adding a test framework in https://github.com/phil-opp/cargo-post/pull/12. The Windows issue that this PR intended to fix is reproducible on CI, so it would be great if you could open a new pull request for this. For now, I'm gonna merge the test framework PR even though it doesn't pass on Windows so that we have automatic checks for future PRs.


Visually wide character support

It is currently assumed a rune is exactly 1 column wide. Drawing japanese hiragana for example results in only half of the runes in View.lines to be copied to View.viewLines (visually) because the x offset of each character, as passed to v.setRune, is incremented by 1 instead of 2.
Same goes for wrapping and most methods in editor.go.
What does the nul byte in editor.go's writeRune do? It breaks unicode code points that contain that byte, which results in every hiragana character to be (visually) separated with a space after a call to View.Buffer. This should be fixed in View.Buffer and View.ViewBuffer but since I'm not sure of its use, you should double check my 'fix'.
I think most changes should not break existing code that uses gocui except for MoveCursor which now moves entire runes instead of columns. It would be trivial to rename this though and expose the original moveCursor again.
I have only done some really basic testing with demo.go and my own cli-app that uses gocui.
ps: I'm no unicode nor mattn/go-runewidth expert by any means so forgive me if I did something stupid ;)
And thank for your great work on gocui btw.

forgot to mention moveCursor still needs work wherever v.ox is referenced, if line wrapping is disabled and the origin is shifted, it does not take wide characters into account and overflows by runeWidth - 1

First of all, thanks for your contribution! :)
I'm in the process of rewriting the edition mode (issue #60) and I'd prefer to finish it before considering to merge PRs that can be incompatible with this refactoring. Also, this rewrite should fix several issues like the null byte one that you mentioned.
Of course, I'll take into account your PR because I'd definitely like to support variable width runes in the new version.

@jroimartin
It seems #60 was already closed.  How do you feel about this PR?

@jroimartin Will reconsider including this please? Thanks for your library and it works almost well but missing wide characters support is very critical issue. For example, jesseduffield/lazygit depends on this library and commit messages including wide characters mess up. Thanks.

@itchyny I may as well say this here :)
I'm going to merge this PR into my gocui fork because it seems to do the trick. If it ends up being merged into the main gocui repo down the line I'll revisit this and try to stay at parity with the main repo.
@jroimartin Thankyou for all your effort so far with gocui it's been a massive help to me :)


Fastpack?

https://github.com/fastpack/fastpack

Fastpack
Webpack
Parcel*

initial build
2.148s
6.113s
24.32s

- persistent cache
0.176s
N/A
14.88s

- watch mode
0.074s
0.612s
0.354s

@denji is it intended to be used for production? It reminds me Pax which is for development only. I can see a line in their README saying that Fastpack doesn't support production yet.


fix a few links

Describe the change

PR checklist

[ ] I have added description of the change I'm proposing in the OpenMS Documentation.
[ ] I have read and followed OpenMS Documentation Contributing guidelines.
[ ] I have attached a screenshot of the relevant area after this change.
[ ] CHANGELOG.md is updated.
[ ] I have added my name in CONTRIBUTING.md.

any update here?

19 link errors vs 33 in current. Will merge to make progress.


fix: support export async function

Hey, really like this plugin.
We noticed that actions and loaders declared as async functions are not being correctly exported by the routes generator.
Looked into the code and seems like it was because the regex is not matching for async function.
Not sure if there is a reason for this, but created a small PR to support it.
Let me know if there is anything else that needs to be done to solve this.

Published at v1.7.20


Remove misleading crontab guru suggestion from docs

Since the underlying syntax is non-standard, it is very confusing and misleading to suggest using a tool with different syntax!

https://sysctl.io/crontab-generator
https://www.freeformatter.com/cron-expression-generator-quartz.html
https://www.programmertools.online/generator/cron_expression.html

Your first link gives me a 404

Yeah, it is working in my general browser (maybe cached), but not working in other or in incognito. I hope it is temporary issue.
Anyway, it is self-hosted it-tools project. The same is deployed at https://it-tools.tech/.


cohesin exit gate opening MF

Hi David,
On the recent  Skype we discussed this as a MF and you thought that we did not know the mechanism, so it could not be included.
I still think there is enough for a molecular function term here...
It has been suggested that Wapl facilitates cohesin dissocia- tion from chromosomes by opening a DNA exit gate between the kleisin N terminus and Smc3 (Chan et al., 2012; Huis in ’t Veld et al., 2014). We therefore tested whether Wapl indeed causes dissociation of the Rad21 N terminus from fission yeast Psm3. We used purified cohesin tetramers in which one of the two separase cleavage sites in Rad21 was changed for the recognition sequence of TEV protease (Murayama and Uhlmann, 2014; Tomonaga et al., 2000). Following TEV cleavage, both the N- and C-terminal cleavage products (Rad21N and Rad21C, respectively) remained associated with the cohesin tetramer, which was immunopurified from the reaction via an epitope tag on Psm3. Addition of Pds5-Wapl resulted in loss of Rad21N, but not Rad21C, from the cohesin complex, demonstrating that Pds5-Wapl indeed displaces the kleisin N terminus (Figures 4B and S4A). Rad21N displacement occurred in the presence of ATP or ATP-g-S, but not in the presence of ADP or in the absence of nucleotide. In contrast, Mis4-Ssl3 in the presence or absence of ATP and DNA did not achieve Rad21N displace- ment (Figure S4B). These results confirm that Pds5-Wapl opens the cohesin ring interface between Rad21N and Psm3. The requirement for ATP, but not its hydrolysis, further suggests that the Rad21N-Psm3 interface only opens if the SMC head dimer is occupied by ATP and therefore closed.

A tentative definition
ATP-dependent catalysis of the dissociation of a kleisin molecule  from cohesin to enable cohesin unloading from DNA.
If you still think unsuitable can we capture this as a process so it is a subpart of 'unloading'.Its a shame not to capture this specific detachment of Kleisin as its such an elegant experiment, and the MF almost certainly occurs.....I don't want to lose the info....

I just noticed I have an older ticket for this. I think this one is clearer so  I'll close the other one:
https://github.com/geneontology/go-ontology/issues/12531

So the 2 steps of unloading are

SMC head disengagement
Kleisin (Rad21N) disengagement (this would be a synonym for  cohesin kleisin exit gate opening)

" SMC head disengagement. After this, DNA exit remains dependent on the presence of ATP, but not its hydrolysis, consistent with the characteristics of Rad21N disengagement."

But what is the molecular function really? It seems like what we really want is something like the opposite of protein binding. Protein dissociation activity??? I think we should make a model and bring this up on an annotation call. It seems like we should be able to come up with a generic molecular function for this type of dissociation since I suspect we will encounter it over and over again. A term like cohesin exit gate opening seems like a bit of a fudge to me because it isn't classifiable in the current structure. It's just a descriptive way of describing what is happening in this special case.

Don't worry about this one David, it predated this one which was opened when I made the LEGO model! http://noctua.berkeleybop.org/editor/graph/gomodel:57c4867e00000026?barista_token=wr9rgbigvb5dlcbmnral
I have this ticket, pending reading future papers
https://github.com/geneontology/go-ontology/issues/12623
This will be a while, I have put my mitotic chromosome segregation project on hold to tackle G/2 M transition  because it is more directly relevant to cell cycle project.


Build script

[x] --version
[ ] man page generation?

I think man-page should be manual since it requires a dependency. --version has been implemented in #63


"All com.android.support libraries must use the exact same version specification"

Hello, thanks for your great library !
I get the error in my build.gradle on this line compile 'com.android.support:appcompat-v7:25.2.0' :

All com.android.support libraries must use the exact same version specification (mixing versions can lead to runtime crashes).

I can still run the project, but the compile line is underlined in red and it annoys me a bit ...
Do you have any suggestion ? Thanks !

This is because the library has not been updated to use the new support libraries, presently it is using:

com.android.support:appcompat-v7:25.1.0
com.android.support:design:25.1.0

The fix for now would be to use 25.1.0 libraries for your project, or to download and build the project yourself using the new libraries.

Hello, fellow programmers!
Yes, as @Aiden-Ziegelaar comments the reason for the warning is that Dexter is using an old version of the appcompat library. We are planning to release v4.0.0 today, we will make sure that version is completely updated. However, if you don't feel like updating there you can always exclude the appcompat dependency from Dexter to use your own. To do so, you have to load Dexter in your gradle file as follows:
testCompile('com.karumi:dexter:4.0.0') {
        exclude group: 'com.android.support', module: 'appcompat-v7'
        exclude group: 'com.android.support', module: 'design'
}

Remember that you should provide those dependencies by yourself or the project won't even compile.
Thanks for reporting and helping us to improve!

I'm closing this issue for now, we will release the updated version (#122)

Great ! I'm looking forward to see the new version !

Was this solved?

Yes, you have to use version 4.0.0 of the library.

I still experience an error after the changes.

compile 'com.android.support:appcompat-v7:25.1.0'
testCompile 'junit:junit:4.12'
compile 'com.google.firebase:firebase-messaging:11.0.4'
compile 'com.google.firebase:firebase-auth:11.0.4'

compile 'com.android.support:appcompat-v7:25.1.0'
testCompile 'junit:junit:4.12'
compile 'com.google.firebase:firebase-messaging:11.0.4'
compile 'com.google.firebase:firebase-auth:11.0.4'
is correct or not
tell me

For those who still getting some error is because of android-support conflicting versions
Solution :
https://youtu.be/UDLqFbat7n0
This will solve your problem.

For those who still getting some error is because of android-support conflicting versions
Solution :
https://youtu.be/UDLqFbat7n0
This will solve your problem.

Thank you for sharing, it worked for me too.

Go to project/.idea/libraries folder on your file system and see which libraries are different.
You will have to manually include these libraries with the same version in your build.gradle file.
Then, sync your project.
E.g.:
Ex Here

Go to project/.idea/libraries folder on your file system and see which libraries are different.
You will have to manually include these libraries with the same version in your build.gradle file.
Then, sync your project.
Example Here


can not compiled successed.

I have follow the tutorials step by step,when I run

yarn start-dev
some errors accurred:

Can anybody give me some advice to solve it?

What model did you convert in the "Download and convert pre-trained model" step? I found that certain h5 model files don't properly convert when I run it through tensorflowjs_converter. I ran into the issue you were hitting when I manually downloaded a mobilenet model (https://github.com/fchollet/deep-learning-models/releases/download/v0.6/mobilenet_1_0_224_tf.h5). After converting this model and trying to use it, I would hit the Cannot read property 'model_config' of null error you are hitting.
However, when downloading through the model through the Keras API in the download_model.py script located this repository, seems that the h5 file generated through that would convert correctly and can be used. This might have to do with some differences in the h5 formatting for older saved models.

So strange！Thanks a lot！ I will have a try~


Height of Query results is not retained when results reshown

Azure Data Studio Version: 1.10.0 (User Setup) on Windows

Steps to Reproduce:

Execute a query
Resize the Query Results Window (e.g. make it shorter)
Press WIN + SHIFT+R to toggle the query results window, it is now hidden
Press WIN + SHIFT+R to toggle the query results window again, it is now visible, but at the default height, not with the dimensions set in step 2

any update on this being fixed? this issue is opened since 2019.


Clarification of limitation on app-service-linux-intro

Rewording to make it clearer that it's a cannot mix (limitation) rather than do not mix (recommendation)

@alexjs : Thanks for your contribution! The author, @msangapu, has been notified to review your proposed change.

#sign-off


[BACKLOG-23398] Add metadata injection to set field value to a constant

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/setvalueconstant/SetValueConstantMeta.java
  engine/src/main/resources/org/pentaho/di/trans/steps/setvalueconstant/messages/messages_en_US.properties
  engine/src/test/java/org/pentaho/di/trans/steps/setvalueconstant/SetValueConstantMetaInjectionTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.checksum.CheckSumMeta

Branch Change: -1.3% :small_red_triangle_down:
Complexity Change: -1.4% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.4% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/setvalueconstant/SetValueConstantMeta.java
  engine/src/main/resources/org/pentaho/di/trans/steps/setvalueconstant/messages/messages_en_US.properties
  engine/src/test/java/org/pentaho/di/trans/steps/setvalueconstant/SetValueConstantMetaInjectionTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.9% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -1.8% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.7% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change: -0.1% :small_red_triangle_down:


need help with installation (cuDNN issue)

Hi Talmo,
I am trying to set up LEAP on a new machine. I'm SUPER close, but I am having trouble after the fast training step. The issue seems to do with the cuDNN.. I've attached the output. Do you have any idea what it could be or what I could try? I downloaded cuDNN v7.0.5 , and copied the files into the correct folders, but it doesn't seem to be enough.
name: GeForce GTX 1080 Ti major: 6 minor: 1 memoryClockRate(GHz): 1.645 pciBusID: 0000:65:00.0 totalMemory: 11.00GiB freeMemory: 9.08GiB 2019-02-21 12:37:32.191564: I C:\tf_jenkins\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\common_runtime\gpu\gpu_device.cc:1312] Adding visible gpu devices: 0 2019-02-21 12:37:35.326578: I C:\tf_jenkins\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\common_runtime\gpu\gpu_device.cc:993] Creating TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 8794 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1080 Ti, pci bus id: 0000:65:00.0, compute capability: 6.1) 2019-02-21 12:37:36.966472: E C:\tf_jenkins\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\stream_executor\cuda\cuda_dnn.cc:385] could not create cudnn handle: CUDNN_STATUS_NOT_INITIALIZED 2019-02-21 12:37:36.967063: E C:\tf_jenkins\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\stream_executor\cuda\cuda_dnn.cc:389] error retrieving driver version: Unimplemented: kernel reported driver version not implemented on Windows 2019-02-21 12:37:36.968154: F C:\tf_jenkins\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\kernels\conv_ops.cc:717] Check failed: stream->parent()->GetConvolveAlgorithms( conv_parameters.ShouldIncludeWinogradNonfusedAlgo<T>(), &algorithms)
Thank you!!
Konstantin
p.s. you can close that previous issue that is still open.

Hey Konstantin,
This error can happen under several circumstances but the one I've seen most often is due to having an older GPU driver version. Try updating it from the NVIDIA website and giving it another go.
Talmo

Hi Talmo,
Thanks. I updated the GPU driver and now I have a new error.
'name: GeForce GTX 1080 Ti major: 6 minor: 1 memoryClockRate(GHz): 1.645
pciBusID: 0000:65:00.0
totalMemory: 11.00GiB freeMemory: 9.11GiB
2019-02-24 16:19:04.258147: I C:\tf_jenkins\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\common_runtime\gpu\gpu_device.cc:1312] Adding visible gpu devices: 0
2019-02-24 16:21:06.767107: I C:\tf_jenkins\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\common_runtime\gpu\gpu_device.cc:993] Creating TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 8818 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1080 Ti, pci bus id: 0000:65:00.0, compute capability: 6.1)
2019-02-24 16:21:10.888156: E C:\tf_jenkins\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\stream_executor\cuda\cuda_dnn.cc:378] Loaded runtime CuDNN library: 7102 (compatibility version 7100) but source was compiled with 7003 (compatibility version 7000).  If using a binary install, upgrade your CuDNN library to match.  If building from sources, make sure the library loaded at runtime matches a compatible version specified during compile configuration.
2019-02-24 16:21:10.890164: F C:\tf_jenkins\workspace\rel-win\M\windows-gpu\PY\36\tensorflow\core\kernels\conv_ops.cc:717] Check failed: stream->parent()->GetConvolveAlgorithms( conv_parameters.ShouldIncludeWinogradNonfusedAlgo(), &algorithms) '
From reading around on the internet, it seems like it is using a 7.1 version of CUDNN. The library that I have recently downloaded (and whose files I copied into the CUDA folders) is version 7.0.5.
Is it OK that I upgraded to the newest driver for my GPU, but am trying to use CUDA v9?
Thanks,
Konstantin

Hey Konstantin,
Sorry for the delay -- did you figure this out? The exact versions I've confirmed to work are:
CUDA 9.0:

https://developer.nvidia.com/cuda-toolkit-archive
cuda_9.0.176_win10_network.exe

cuDNN v7.3.1 (Sept 28, 2018), for CUDA 9.0:

https://developer.nvidia.com/rdp/cudnn-archive (need to login)
cudnn-9.0-windows10-x64-v<IP_ADDRESS>.zip
Extract to: C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v9.0

Let me know if you're still having issues.
Talmo


trying to shorten scraping code

@BenGriffith cannot share gpt thread with images in them yet ("Sharing conversations with images is not yet supported")
Can you check accuracy?
Your initial version:
(Pdb) len(united_states_tollways)
29
(Pdb) len(united_states_tollways.values())
29
(Pdb) sum(len(a) for a in united_states_tollways.values())
170
My version:
(Pdb) len(united_states_tollways)
30
(Pdb) sum(len(a) for a in united_states_tollways.values())
190

And could use a walrus :)
https://twitter.com/bbelderbos/status/1720502899422142557

I added a small check to remove h2 that is not a state/territory from being processed. Code was picking up the h2 'Jurisdictions without toll roads' at the very end causing the count difference of 1 between 29 and 30.

Great, that explains, thanks.
What about
(Pdb) sum(len(a) for a in united_states_tollways.values())
170
vs
(Pdb) sum(len(a) for a in united_states_tollways.values())
190

Reviewed webpage and updated results (190) a few minutes and 190 is the correct value. I had only done a few spot checks on code producing 170 so possible had a bug wasn't aware of.

nice, thanks


[PoC] decouple AbstractSuccessfulDocument from Serializer and Response

This PR is to illustrate what I try to explain in #53
As you can see, everything work like before, the main JsonApi class is always able to return PSR7 Response and the document is not reponsible anymore to generate it. The reponder use the serializer to do it.
What is missing :

do the same thing for errors.

WDYT ?

This is very nice a work :) I am in favor of this solution for sure! My only problem is that codes like below will break:
https://github.com/woohoolabs/yin-middleware/blob/master/src/Middleware/JsonApiErrorHandlerMiddleware.php#L53
I am a bit uncertain if anyone else uses this middleware or direct response generation from documents at all besides me :P Semantic versioning would dictate to consider this a breaking change... :S Do you have any good idea how to manage the situation?

With or without this middleware the change I made are a BC break and force you to bump the major version of the library. Doing this your middleware can still use version 1.0 if you want.
Another solution (and I'm not really fan) can be to cohabit getResponse* and getContent methods , deprecate getResponse* method and remove them when a new major version will be released.
If you want to create a new major version, I can suggest you to bump the php version too and require at least php 7.0. And I can submit other enhancement that will not respect BC promise like this one https://github.com/lunika/yin/commit/d6955813bc7bc1839794ff445a75d222902e94d1 (I can open an other issue to explain this commit)

Yes, I also considered as a less-than-ideal solution to leave the original iterface as-is and add your changes to it. And yes, I planned to increase the major version during the summer but a major version bump is a good reason to do it sooner.
I am looking forward to getting the explanation for the other change you made :)
And by the way: what about passing the Responder to the jsonApi class what we were talking about initially? I might saw something that you don't need it anymore.

And by the way: what about passing the Responder to the JsonApi class what we were talking about initially? I might saw something that you don't need it anymore.

Hmm I think it's not possible, the responder has no interface so it's not possible to replace it by an other one because there is no contract between you and a developer that use the library. If I implement my own responder, which method I have to implement ?

I mean one could fine-tune the responer after extending the original Responder class and passing it to the JsonApi. Until now I haven't thought that anyone would need this. And as the same can be done externally after the document got transformer, I won't change the current behaviour.
And concerning the PR, I'll accept it during the following days when I start working on Yin 2.0. :)


Deprecate LookupHash

Desc
Per discussion in Slack. LookupHash vindexes should be deprecated in favor of Lookup.

LGTM


Jsonpath

Coverage remained the same at 100.0% when pulling 5f4bbddfae73f6f037275ad48288014effed36ae on jsonpath into 10917d93e936309e804b6f6729ad6e0c929d2b5f on master.


Atom language plugin?

The current markdown language packages for Atom don't use tree-sitter. I would be great if this package could be used as an Atom Language package.

Feel free to raise an issue in the corresponding Atom Language repository or  create your own Atom Language package. Let me know if you have any question regarding this package!


New protocol version / migration for lazy mux and block sizes

We need to define a new protocol version to accommodate a couple changes in the network and consensus.
The new version will support one flag InitiatorOnly, which indicates that the peer that started the version negotiation is not running the responder side of the miniprotocols. This change may not be able to use the protocol versioning mechanism, needs further review.
The consensus layer also needs to instantiate the chain sync to include the block size (needed to provide that info to block fetch). This might be done by a breaking change, or by using the protocol versioning.

Consensus is ready after https://github.com/input-output-hk/ouroboros-network/pull/1632, we just need to enable it.


chore: Updating the UI for scrollbars all over the product to use the same color and width

Description
Updating the UI for scrollbars all over the product to use the same color and width
Fixes #19340
Media:
https://cjrc9tc9nt.vmaker.com/record/0Wabub8xKvzaL2Ac
Type of change

Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?

Manual

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/build-deploy-preview env=release

/build-deploy-preview skip-tests="true"

/build-deploy-preview skip-tests="true"

/build-deploy-preview skip-tests="true"

/build-deploy-preview skip-tests="true"

/build-deploy-preview skip-tests="true"

/build-deploy-preview

/build-deploy-preview

/build-deploy-preview skip-tests="true"


Preliminary exploration of artifact generation through loc/conf in the controller

Let's see what it would take for the localization and configuration in ocm-controller to produce an artifact.

Most of what's required is in https://github.com/open-component-model/ocm-controller/pull/498.
Otherwise, I don't think this ticket now adds anything of value really.


Upgrade is failing with flake does not provide attribute

Describe the bug
Upgrade is failing with flake does not provide attribute
To Reproduce
nix run "https://getfleek.dev/latest.tar.gz" -- update
 [*]  Update package index
From github.com:yanns/home
 * branch            main       -> FETCH_HEAD
Already up to date.
 [i]  Updating flake sources
Everything up-to-date
error: flake 'git+file:///Users/yannsimon/.local/share/fleek' does not provide attribute 'apps.aarch64-darwin.fleek', 'packages.aarch64-darwin.fleek', 'legacyPackages.aarch64-darwin.fleek' or 'fleek'
  ERROR
  ERROR   Error: exit status 1
  ERROR
  ERROR
  ERROR   Error:
  ERROR
  ERROR   There was an internal error. Run with FLEEK_DEBUG=1 for a detailed error message, and consider reporting it at https://github.com/ublue-os/fleek/issues

Expected behavior
Upgrade should work
Environment

OS: macOS Ventura
Shell: zsh
fleek version: 0.9.23

@yanns can you show me the first 30-40 lines of ~/.local/share/fleek/flake.nix?
I wonder if you upgraded in the magic moment while I was releasing 0.9.24 and there was a 30-60 second window where the new version existed but flakehub still pointed to the old version.
You should be able to fix this locally by running nix run "https://getfleek.dev/latest.tar.gz" -- write which is a command hidden from the cli's help screens that we use internally to update the flake sources before applying the changes locally.  But please upload your current flake.nix before you run this so I can see if I missed an edge case of some sort on the upgrade logic.
Thanks for reporting

Thank you @yanns, can you confirm if the write command above resolved your problem?

The write is successful.
The new content:
───────┬──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
       │ File: /Users/yannsimon/.local/share/fleek/flake.nix
───────┼──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
   1   │ {
   2   │   # DO NOT EDIT: This file is managed by fleek. Manual changes will be overwritten.
   3   │   description = "Fleek Configuration";
   4   │
   5   │   inputs = {
   6   │     # Nixpkgs
   7   │     nixpkgs.url = "github:nixos/nixpkgs/nixos-unstable";
   8   │
   9   │     # Home manager
  10   │     home-manager.url = "https://flakehub.com/f/nix-community/home-manager/0.1.tar.gz";
  11   │     home-manager.inputs.nixpkgs.follows = "nixpkgs";
  12   │
  13   │     # Fleek
  14   │     fleek.url = "https://flakehub.com/f/ublue-os/fleek/*.tar.gz";
  15   │
  16   │     # Overlays
  17   │
  18   │
  19   │   };
  20   │
  21   │   outputs = { self, nixpkgs, home-manager, fleek, ... }@inputs: {
  22   │
  23   │      packages.aarch64-darwin.fleek = fleek.packages.aarch64-darwin.default;
  24   │
  25   │     # Available through 'home-manager --flake .#your-username@your-hostname'
  26   │
  27   │     homeConfigurations = {
  28   │
  29   │<EMAIL_ADDRESS>= home-manager.lib.homeManagerConfiguration {
  30   │         pkgs = nixpkgs.legacyPackages.aarch64-darwin; # Home-manager requires 'pkgs' instance
  31   │         extraSpecialArgs = { inherit inputs; }; # Pass flake inputs to our config
  32   │         modules = [
  33   │           ./home.nix
  34   │           ./path.nix
  35   │           ./shell.nix
  36   │           ./user.nix
  37   │           ./aliases.nix
  38   │           ./programs.nix
  39   │           # Host Specific configs
  40   │           ./Yanns-MacBook-Pro.local/yannsimon.nix
  41   │           ./Yanns-MacBook-Pro.local/custom.nix
  42   │           # self-manage fleek
  43   │           {
  44   │             home.packages = [
  45   │               fleek.packages.aarch64-darwin.default
  46   │             ];
  47   │           }
  48   │           ({
  49   │            nixpkgs.overlays = [];
  50   │           })
  51   │
  52   │         ];
  53   │       };
  54   │
  55   │     };
  56   │   };
  57   │ }
───────┴──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

The update is now also successful.
There is one error with the apply:
Activating installPackages
nix profile remove /nix/store/pamazja69k0kr57n9yl405nzqm8f1c75-home-manager-path
removing 'home-manager-path'
error: An existing package already provides the following file:

         /nix/store/rz8nw4hpwg37yd1500z155cb9w6mawqj-fleek-0.9.23/bin/fleek

       This is the conflicting file from the new package:

         /nix/store/c7grwh7ammna3jnjhw4y3g7b72v831v2-home-manager-path/bin/fleek

       To remove the existing package:

         nix profile remove github:ublue-os/fleek#packages.aarch64-darwin.default

       The new package can also be installed next to the existing one by assigning a different priority.
       The conflicting packages have a priority of 5.
       To prioritise the new package:

         nix profile install /nix/store/c7grwh7ammna3jnjhw4y3g7b72v831v2-home-manager-path --priority 4

       To prioritise the existing package:

         nix profile install /nix/store/c7grwh7ammna3jnjhw4y3g7b72v831v2-home-manager-path --priority 6

Oops, Nix failed to install your new Home Manager profile!

Perhaps there is a conflict with a package that was installed using
"nix profile install"? Try running

    nix profile list

and if there is a conflicting package you can remove it with

    nix profile remove {number | store path}

Then try activating your Home Manager configuration again.
  ERROR
  ERROR   Error: exit status 1
  ERROR
  ERROR
  ERROR   Error:
  ERROR
  ERROR   There was an internal error. Run with FLEEK_DEBUG=1 for a detailed error message, and consider reporting it at https://github.com/ublue-os/fleek/issues

Should I open a new issue for that?

Yes, just so I can track the need to force removal of older versions of fleek that were installed outside the flakehub installation method.
You can resolve this by following the nix profile list & nix profile remove message at the bottom of the error.

Problem fixed, thanks a lot.
For the issue with the homemanager: https://github.com/ublue-os/fleek/issues/288

Reopening until there's a software fix without any workarounds

Upgrade instructions documented here:
https://getfleek.dev/docs/upgrade


Add rule set for Youtube to open inside Yattee App

@lilaelephant Thanks a lot for your rule!
I'd like to merge it now, but 1 thing to confirm: the rules in this repository are currently under construction for a library feature that will be released in a few months. Are you OK with your rule being used for that purpose? It would look like this and will be a free feature:
https://github.com/mshibanami/redirect-web/assets/1333214/43607d28-3559-4735-904e-ca4e7d2aacf7

Hi @lilaelephant, I've released the new version 6, which includes the Library feature as I announced! Are you OK with the app displaying your rule in the library?
(For safety, I won't merge this pull request unless I get your reply. 🙂)

@lilaelephant I will merge this PR now. But if you have any concerns, please let me know, then I will roll it back as soon as possible!


table

Hi laiiihz,
I have Sunmi V2 Pro. Could you please tell me how to print table and Image

im working on a new version of sunmi service,and i will add this feature this week (if i have time 😂 ) @Ahmedmedhat187

try the new version of this package 😆  @Ahmedmedhat187
printColumnsText(
    text: [ 'a', 'b', 'c' ],
    width: [ 4, 4, 8 ],
    align: [ 1, 1, 1 ]
)
printBitmap(image)

im closing this issue🤣


[cli][docs] adding doc updates to reflect the --processor cli arg changes

to: @chunyong-lin
cc: @airbnb/streamalert-maintainers
size: small
resolves N/A
Background
Previous changes to the CLI updated the manage.py lambda deploy and rollback commands to use a list of functions passed to the --processor arg. This PR updates the test command and also updates docs to go with previous changes.
Changes

Updating the manage.py lambda test command to support multiple args.
Updating docs to depict the usage of CLI --processor flag which now accepts a list of functions to test/deploy/rollback.
The previous method of using multiple --processor flags is no longer supported and will result in only the last --processor flag having any affect.

Testing
Ran tests with various options to ensure the proper behavior occurs.

Coverage remained the same at 95.157% when pulling c5d268b24176aa2edbca0377e68bae1b391b72da on ryandeivert-minor-doc-updates into affb216d45f957adfb5007a26226e667cc20cfa1 on master.

Coverage remained the same at 95.109% when pulling 95634add3e8a2840461a56ebcfac06a7301631bc on ryandeivert-minor-doc-updates into 7e2ec1857c76fdb27ac4d60b556bb227d0ae5a6c on master.

hi @chunyong-lin I pushed a commit with a code comment

Coverage remained the same at 95.075% when pulling 715f77d1c6a450ce5a64ad6fc1ba8a1be178e4e5 on ryandeivert-minor-doc-updates into 3cf8b22dbc6b42a0b10db96a6a66d346c98995dc on master.


python37Packages.uproot-methods: 0.4.3 -> 0.4.4

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.7-uproot-methods/versions.
meta.description for python37Packages.uproot-methods is: '"Pythonic mix-ins for ROOT classes"'.

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.4.4 with grep in /nix/store/l5cjszx1g93wbp3iicjais0k1dwhj9rs-python3.7-uproot-methods-0.4.4
directory tree listing: https://gist.github.com/13ae0abb6d240fbbdc21222df160786b
du listing: https://gist.github.com/ec05f63fadd19011ac6f227ae577a883

Rebuild report (if merged into master) (click to expand)

16 total rebuild path(s)
4 package rebuild(s)
4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
python27Packages.uproot
python27Packages.uproot-methods
python37Packages.uproot
python37Packages.uproot-methods

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l5cjszx1g93wbp3iicjais0k1dwhj9rs-python3.7-uproot-methods-0.4.4 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A python37Packages.uproot-methods https://github.com/r-ryantm/nixpkgs/archive/f54b90c2f7c1445bdff56d5bda8c8be940c3a8b4.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l5cjszx1g93wbp3iicjais0k1dwhj9rs-python3.7-uproot-methods-0.4.4
ls -la /nix/store/l5cjszx1g93wbp3iicjais0k1dwhj9rs-python3.7-uproot-methods-0.4.4/bin

cc @costrouc for testing.

nix-review shows no issues

should build now on aarch64
@GrahamcOfBorg build python37Packages.uproot-methods  python27Packages.uproot-methods


chore: replace github.com/ghodss/yaml with sigs.k8s.io/yaml

Currently we are using two YAML packages:

github.com/ghodss/yaml v1.0.0
sigs.k8s.io/yaml v1.3.0

The github.com/ghodss/yaml package is no longer actively maintained. sigs.k8s.io/yaml is a permanent fork of github.com/ghodss/yaml. It is actively maintained by Kubernetes SIG, and also widely used in K8s projects.
The notable change is that github.com/ghodss/yaml uses gopkg.in/yaml.v2 v2.2.2, but sigs.k8s.io/yaml uses gopkg.in/yaml.v2 v2.4.0. Changes can be seen here v2.2.2...v2.4.0, mostly bug fixes.

/auto-cc

/auto-cc

/assign @soltysh
Hi, sorry for the explicit mention. I would be grateful if you could take some time to review the changes and provide me with your feedback. If there are any specific concerns or questions you have about the changes, I am more than willing to help address them. Thank you :smile:

/assign @deads2k
/assign @ingvagabund
Hi, I would be grateful if you could take some time to review the changes and provide me with your feedback. If there are any specific concerns or questions you have about the changes, I am more than willing to help address them. Thank you :smile:

sigs.k8s.io/yaml was forked ~6 years ago. The original repo merged only comments, test changes and a new function. sigs.k8s.io/yaml continued to develop the code. There's a bunch of vendored deps updates. With creating yaml.v3. Looks like it all looks good.

/ok-to-test

/lgtm

@deads2k Friendly ping :smile:

@deads2k Friendly ping :smile:

Rebased to resolve conflicts

/retest

/retest

/approve

/lgtm

@deads2k @ingvagabund Any idea on why the bot is not applying the label? The bot seems to be acting weird today :thinking:

/refresh

/lgtm


Could not download Krakend CE version 2.0.4 using curl

Not able to download this using curl as part of Docker image build. Only 2.0.5 is could be downloaded
https://repo.krakend.io/bin/krakend_2.0.4_amd64.tar.gz

Hi @krishna-kunderu
The download should work now!


Bypass Certificate Verification on Development

Hi
I'm trying to use this package on a local Keycloak instance that uses a self-signed certificate. Since python requests verify the certificate by default, creating a FastAPIKeycloak wrapper to the Keycloak instance always raises an SSLVerificationError.
Is it okay if a verify server certificate parameter will be included to the FastAPIKeycloak wrapper so we can temporarily bypass certificate verification?
Thank you!

Hi @edgarfelizmenio,
We'd rather not open this package to vulnerabilities due to testing purposes. Sorry.
If you really need that functionality, you could either monkey patch the requests method or utilize a context manager to override its configuration (Check https://stackoverflow.com/questions/15445981/how-do-i-disable-the-security-certificate-check-in-python-requests).
Both is neither recommendable nor beautiful, but that's mostly due to the fact that using Keycloak with self-signed certificates isn’t either


Config 'minimal' JSON option

Current TopSim configuration files - specifically, the 'cluster' component - get unwieldy when we have large systems.
Having a flag in the 'header' that allows us to unroll the entire system topology without having to print 100+ compute nodes (often with the same exact numbers) would improve readability of TopSim simulation configuration.
e.g.
            "header": {
                "time": "false",
                "minimal":"true",
                "generator": "",
                "architecture": {
                    "cpu": {
                        "GenericCPU": "num_nodes",
                        "resource": {
                        "flops": 48000000.0,
                        "memory":64,
                        "rates": 10.0
                        }
                    },
                    "gpu": {}
                }
            },

This would then be checked and unrolled in Config.parse_cluster_config

An additional feature for this: have the config class update any older files to the new structure if it's detected.


Remove chartconfig usage from Tiller metrics

Towards giantswarm/roadmap#30
I missed removing some chartconfig code from the Tiller metrics. Getting it sorted.
E 03/10 19:50:29 failed collecting metrics | exporterkit/collector/set.go:92
	/go/src/github.com/giantswarm/chart-operator/vendor/github.com/giantswarm/exporterkit/collector/set.go:92:
	/go/src/github.com/giantswarm/chart-operator/vendor/github.com/giantswarm/exporterkit/collector/set.go:83:
	/go/src/github.com/giantswarm/chart-operator/service/collector/tiller_max_history.go:84:
	the server could not find the requested resource (get chartconfigs.core.giantswarm.io)

Tested on ghost.


Benchmark

I was curious how well the proposed sorting functions for three, four or five integers would perform on my hardware, so I wrote a quick benchmark. It seems like there is almost no difference between AlphaDev and a simple sorting network, but perhaps someone finds the results interesting anyway, so I thought I'd share.
Benchmark results
Sort5AlphaDev: 17.367800 cycles on average, checksum -1224191569
Sort5Naive: 17.378000 cycles on average, checksum -1224191569
AlphaDev is 0.058729 % faster than a naive sorting network

Sort5AlphaDev: 17.384000 cycles on average, checksum -1224191569
Sort5Naive: 17.381600 cycles on average, checksum -1224191569
AlphaDev is -0.013806 % faster than a naive sorting network

Sort5AlphaDev: 17.505800 cycles on average, checksum -1224191569
Sort5Naive: 17.371000 cycles on average, checksum -1224191569
AlphaDev is -0.770031 % faster than a naive sorting network

Benchmark code
// clang -O3 main.c -o main && ./main
#include <stdio.h>
#include <stdint.h>
#include <assert.h>
#include <stdlib.h>

// N_SORT can be 3, 4 or 5
#define N_SORT 5

void Sort3AlphaDev(int* buffer) {
  asm volatile(
      "mov 0x4(%0), %%eax            \n"
      "mov 0x8(%0), %%ecx            \n"
      "cmp %%eax, %%ecx              \n"
      "mov %%eax, %%edx              \n"
      "cmovl %%ecx, %%edx            \n"
      "mov (%0), %%r8d               \n"
      "cmovg %%ecx, %%eax            \n"
      "cmp %%r8d, %%eax              \n"
      "mov %%r8d, %%ecx              \n"
      "cmovl %%eax, %%ecx            \n"
      "cmovle %%r8d, %%eax           \n"
      "mov %%eax, 0x8(%0)            \n"
      "cmp %%ecx, %%edx              \n"
      "cmovle %%edx, %%r8d           \n"
      "mov %%r8d, (%0)               \n"
      "cmovg %%edx, %%ecx            \n"
      "mov %%ecx, 0x4(%0)            \n"
      : "+r"(buffer)
      :
      : "eax", "ecx", "edx", "r8d", "memory");
}

void Sort4AlphaDev(int* buffer) {
  asm volatile(
      "mov 0x8(%0), %%eax            \n"
      "mov (%0), %%ecx               \n"
      "mov 0x4(%0), %%edx            \n"
      "cmp %%eax, %%ecx              \n"
      "mov %%eax, %%r8d              \n"
      "cmovl %%ecx, %%r8d            \n"
      "cmovl %%eax, %%ecx            \n"
      "mov 0xc(%0), %%r9d            \n"
      "cmp %%r9d, %%edx              \n"
      "mov %%r9d, %%eax              \n"
      "cmovl %%edx, %%eax            \n"
      "cmovl %%r9d, %%edx            \n"
      "cmp %%eax, %%r8d              \n"
      "mov %%eax, %%r9d              \n"
      "cmovl %%r8d, %%r9d            \n"
      "cmovge %%r8d, %%eax           \n"
      "mov %%r9d, (%0)               \n"
      "cmp %%edx, %%ecx              \n"
      "mov %%edx, %%r8d              \n"
      "cmovl %%ecx, %%r8d            \n"
      "cmovge %%ecx, %%edx           \n"
      "mov %%edx, 0xc(%0)            \n"
      "cmp %%r8d, %%eax              \n"
      "mov %%r8d, %%ecx              \n"
      "cmovl %%eax, %%ecx            \n"
      "cmovge %%eax, %%r8d           \n"
      "mov %%r8d, 0x8(%0)            \n"
      "mov %%ecx, 0x4(%0)            \n"
      : "+r"(buffer)
      :
      : "eax", "ecx", "edx", "r8d", "r9d", "memory");
}

void Sort5AlphaDev(int* buffer) {
  asm volatile(
      "mov (%0), %%eax               \n"
      "mov 0x4(%0), %%ecx            \n"
      "cmp %%eax, %%ecx              \n"
      "mov %%eax, %%edx              \n"
      "cmovl %%ecx, %%edx            \n"
      "cmovg %%ecx, %%eax            \n"
      "mov 0xc(%0), %%r8d            \n"
      "mov 0x10(%0), %%ecx           \n"
      "cmp %%r8d, %%ecx              \n"
      "mov %%r8d, %%r9d              \n"
      "cmovl %%ecx, %%r9d            \n"
      "cmovg %%ecx, %%r8d            \n"
      "mov 0x8(%0), %%r10d           \n"
      "cmp %%r10d, %%r8d             \n"
      "mov %%r10d, %%ecx             \n"
      "cmovl %%r8d, %%ecx            \n"
      "cmovle %%r10d, %%r8d          \n"
      "cmp %%ecx, %%r9d              \n"
      "cmovle %%r9d, %%r10d          \n"
      "cmovg %%r9d, %%ecx            \n"
      "cmp %%eax, %%r8d              \n"
      "mov %%eax, %%r9d              \n"
      "cmovl %%r8d, %%r9d            \n"
      "cmovle %%eax, %%r8d           \n"
      "cmp %%edx, %%ecx              \n"
      "mov %%edx, %%eax              \n"
      "cmovl %%ecx, %%eax            \n"
      "cmovle %%edx, %%ecx           \n"
      "mov %%r8d, 0x10(%0)           \n"
      "cmp %%eax, %%r10d             \n"
      "cmovle %%r10d, %%edx          \n"
      "mov %%edx, (%0)               \n"
      "cmovg %%r10d, %%eax           \n"
      "cmp %%r9d, %%ecx              \n"
      "mov %%r9d, %%r8d              \n"
      "cmovl %%ecx, %%r8d            \n"
      "cmovle %%r9d, %%ecx           \n"
      "mov %%ecx, 0xc(%0)            \n"
      "cmp %%r8d, %%eax              \n"
      "cmovle %%eax, %%r9d           \n"
      "mov %%r9d, 0x4(%0)            \n"
      "cmovg %%eax, %%r8d            \n"
      "mov %%r8d, 0x8(%0)            \n"
      : "+r"(buffer)
      :
      : "eax", "ecx", "edx", "r8d", "r9d", "r10d", "memory");
}

void sort2(int *a, int *b){
    if (*a > *b){
        int tmp = *a;
        *a = *b;
        *b = tmp;
    }
}

void Sort3Naive(int *a){
    sort2(a, a + 1);
    sort2(a, a + 2);
    sort2(a + 1, a + 2);
}

void Sort4Naive(int *a){
    sort2(a, a + 1);
    sort2(a + 2, a + 3);
    sort2(a, a + 2);
    sort2(a + 1, a + 3);
    sort2(a + 1, a + 2);
}

void Sort5Naive(int *a){
    sort2(a, a + 3);
    sort2(a + 1, a + 4);
    sort2(a, a + 2);
    sort2(a + 1, a + 3);
    sort2(a, a + 1);
    sort2(a + 2, a + 4);
    sort2(a + 1, a + 2);
    sort2(a + 3, a + 4);
    sort2(a + 2, a + 3);
}

#define NUM_TESTS 10000
#define CONCAT2(a, b, c) a##b##c
#define CONCAT(a, b, c) CONCAT2(a, b, c)

uint64_t test_AlphaDev(){
    uint64_t total = 0;
    int checksum = 0;

    int a[N_SORT];
    for (int k = 0; k < NUM_TESTS; k++){
        for (int i = 0; i < N_SORT; i++){
            a[i] = rand();
        }

        uint64_t start = __builtin_readcyclecounter();

        CONCAT(Sort, N_SORT, AlphaDev)(a);

        uint64_t end = __builtin_readcyclecounter();

        // Check correctness
        for (int i = 1; i < N_SORT; i++){
            assert(a[i - 1] <= a[i]);
        }

        // Compute checksum to prevent compiler from optimizing away the loop
        checksum += a[0] + a[1] + a[2];

        total += end - start;
    }

    printf("Sort%dAlphaDev: %f cycles on average, checksum %d\n", N_SORT, (double)total / NUM_TESTS, checksum);

    return total;
}

uint64_t test_Naive(){
    uint64_t total = 0;
    int checksum = 0;
    int a[N_SORT];
    for (int k = 0; k < NUM_TESTS; k++){
        for (int i = 0; i < N_SORT; i++){
            a[i] = rand();
        }

        uint64_t start = __builtin_readcyclecounter();

        CONCAT(Sort, N_SORT, Naive)(a);

        uint64_t end = __builtin_readcyclecounter();

        // Check correctness
        for (int i = 1; i < N_SORT; i++){
            assert(a[i - 1] <= a[i]);
        }

        // Compute checksum to prevent compiler from optimizing away the loop
        checksum += a[0] + a[1] + a[2];

        total += end - start;
    }

    printf("Sort%dNaive: %f cycles on average, checksum %d\n", N_SORT, (double)total / NUM_TESTS, checksum);

    return total;
}

int main(){
    for (int k = 0; k < 100; k++){
        srand(0);
        uint64_t cycles_alphadev = test_AlphaDev();
        srand(0);
        uint64_t cycles_naive = test_Naive();

        double percent = cycles_naive * 100.0 / cycles_alphadev - 100.0;

        printf("AlphaDev is %f %% faster than a naive sorting network\n\n", percent);
    }

    return 0;
}

I changed your code to pre-generate a large array and then sort all the data at once, which gave a completely different result (AlphaDev is now significantly faster). I think it should be related to caching.
New Result
Sort5AlphaDev: 9.321600 cycles on average, checksum -1224191569
Sort5Naive: 68.231400 cycles on average, checksum -1224191569
AlphaDev is 631.970906 % faster than a naive sorting network

Sort5AlphaDev: 9.330000 cycles on average, checksum -1224191569
Sort5Naive: 68.377600 cycles on average, checksum -1224191569
AlphaDev is 632.878885 % faster than a naive sorting network

Sort5AlphaDev: 9.325800 cycles on average, checksum -1224191569
Sort5Naive: 70.886600 cycles on average, checksum -1224191569
AlphaDev is 660.112805 % faster than a naive sorting network

New Code
uint64_t test_AlphaDev(){
    uint64_t total = 0;
    int checksum = 0;

    int a[NUM_TESTS][N_SORT];
    for (int k = 0; k < NUM_TESTS; k++){
        for (int i = 0; i < N_SORT; i++){
            a[k][i] = rand();
        }
    }

    uint64_t start = __builtin_readcyclecounter();

    for (int k = 0; k < NUM_TESTS; k++){
        CONCAT(Sort, N_SORT, AlphaDev)(a[k]);
    }

    uint64_t end = __builtin_readcyclecounter();

    for (int k = 0; k < NUM_TESTS; k++){
        // Check correctness
        for (int i = 1; i < N_SORT; i++){
            assert(a[k][i - 1] <= a[k][i]);
        }

        // Compute checksum to prevent compiler from optimizing away the loop
        checksum += a[k][0] + a[k][1] + a[k][2];
    }

    total += end - start;

    printf("Sort%dAlphaDev: %f cycles on average, checksum %d\n", N_SORT, (double)total / NUM_TESTS, checksum);

    return total;
}

// Same change in test_Naive, emitted

Perhaps test_AlphaDev() --> test_AlphaDev(number_of_sorts, number_of_tests) to make tests easier to tweak.
Does the upper limit on number_of_sorts need to be five?  Could it be 5,000 instead?

Does the upper limit on number_of_sorts need to be five? Could it be 5,000 instead?

It could be up to 8.
https://github.com/deepmind/alphadev/blob/1a6eac1544c1075ef27814f09f9223cb84537c59/sort_functions_test.cc#L280
I guess higher numbers would be more difficult due to limited number of registers.


Updating script to be able to update the EBI Copyright every year. No…

…w excluding the .git directory from the file search as this was corrupting the git repos
@dzerbino and @magaliruffier updated the script last year so that we now display a copyright like the following in our scripts:

Copyright [1999-2015] Wellcome Trust Sanger Institute and the EMBL-European Bioinformatics Institute
Copyright [2016] EMBL-European Bioinformatics Institute

I've updated the script so that we can update the EMBL-EBI part every year (similar to what the script was doing in the past). For this year, I believe we would want something like:

Copyright [2016-2017] EMBL-European Bioinformatics Institute

I've also excluded the ".git" directory from the grep as this was corrupting the Git repos :-/

I guess that's ok. Does this script deal with alternate expressions, like license in a comment instead of a POD block?

Coverage remained the same at 81.548% when pulling bb38ec4d621a1bf4f47ff6b5cbdc48cff176c466 on thomasmaurel:annual_copyright_updater into 8d91a4e0bf620fb833e75be63c5dba34ae894ff7 on Ensembl:master.

Well spotted @nerdstrike, there was a bug for anything different than a POD block.

Coverage remained the same at 81.548% when pulling 2ab02629613de70a3df0baddb867731feccdc207 on thomasmaurel:annual_copyright_updater into 8d91a4e0bf620fb833e75be63c5dba34ae894ff7 on Ensembl:master.

Excellent, we don't need this coming back to haunt us again.


examples appear to be broken

Example contains two pvc's instead of a pv and pvc.
Also was not able to get the s3 storageclass to function.  Controller logs did not appear to have anything watching storageclass.

Hey @acelinkio,
Thanks for using the module and providing feedback. The examples section is indeed broken and outdated.
I recommend that you refer to these ones instead to run your tests https://github.com/awslabs/mountpoint-s3-csi-driver/blob/main/examples/kubernetes/static_provisioning/static_provisioning.yaml


Use ConceptID is Query Suggester

Wheat search: xylan OR "Xylan biosynthesis" OR "glucuronoxylan glucuronosyltransferase activity of Homologues of GUX2 like (LOC_Os03g08600) [Golgi lumen] glucuronoxylan [Golgi lumen] (1->4)-beta-D-xylan (n+1) [Golgi lumen]" where the 3rd phrase was added from the query suggester breaks the search (server offline error).
Need to a) avoid complex long phrases (usally for phenotypes, BioProc, traits) with [ - > () ] in query suggester, b) filter search keywords before sending to server to remove or add \ before certain characters and also c) check keyword box for wildcards like * . always put after alphabets and not in empty or start of search term.

@AjitPS to add UI filtering etc and @KeywanHP to maybe think of better ways to refine suggestions coming from backend API (in future).

I have an idea! The query suggester should add the ConcepID: to the search box instead of the concept name. In fact, it would be exactly the same functionality as what we already have in in Evidence View (+/-) buttons. This would solve all above issues with handling special characters and be actually more accurate. @AjitPS @marco-brandizi

That's good if the user is fine with it, else, it could have an agreed structure like: Concept:"Seed size"(1234) and it could be easily pre-filtered and turned into the ConceptID:1234 format.

The API endpoint synonyms to create the Query Suggestor content returns the ConceptID in the last column. The client JS code can use the id instead of the string. @manlikleo
https://knetminer.com/araknet/synonyms?keyword="seed dormancy"

This was implemented by @manlikleo - see PR: https://github.com/Rothamsted/knetminer/pull/614


Indicate that etag option default is weak

cf. code of application.js

Looks good :)

@hacksparrow @dougwilson, please, how should such modifications be ported to the 5.x branch? Is it "your" job, as maintainers, to merge it? Or should any contributor to expressjs.com take care of the 5.x branch too?
Cheers

@madarche it is open for everyone to open PRs to this repo. Feel free to do so, if you have anything on your mind.

@hacksparrow I mean: What is the recommended workflow that has been devised to make sure that all fixes/features contributed in the 4.x branch actually end up in the 5.x branch too? Thanks

The last commit on 5.x was on 24th July. We'll have to review the commits / PRs to 4.x in the meantime, and merge accordingly.

OK. Thank you!

@madarche are you planning on doing that, btw? 5.x release is quite near now, we need to start merging.

@hacksparrow I would be glad to help on that. But with limited time I'm focusing on documenting other things (some more PRs to come) rather than working on this 5.x merge. So to conclude: I was asking 1st to be sure that my work would not be lost when 5 is released, and 2nd so my work fits in well with the global "publication workflow" (because an another option could have been that contributors should provide both a PR for "stable" branch and a PR for "next" branch).

@madarche great, looking forward to the PRs. The PR-to-both-branches approach could potentially cause conflicts and everyone might not be keen on doing two PRs. Since we have to review everything anyway, a PR to gh-pages should serve the purpose.


Move engine-api to docker/docker/api

Remove all dependencies to engine-api and start using docker/docker/api.
Signed-off-by: Jana Radhakrishnan<EMAIL_ADDRESS>

LGTM, waiting for the CI green..


Not listed yet: www.sammobile.com

The website at this url is unlisted: https://www.sammobile.com/

This website is now in the support list and will be in the next release of the GCM web extension on addons.mozilla.org
Thanks!


fix: extending context

Coverage increased (+0.2%) to 93.684% when pulling 880926df59e6904076c5d37adafc0e1d1c249adb on fix-context into 9bbd74dd96420c5efe44123f4f6a01869bda5f36 on master.


[Auto Bench] Scripts of Running Benchmarks with Nginx

Closes #2826 .

This PR:

Adds the script for benchmarking when using nginx.
Raise up runnable tx size to 1e7 bytes. With 50 nodes and 5 da committee it can get 2.37MBps. With tx size of 2e8 and 10/5 nodes, it can get 15.56MBps. More results in results_init_run.csv.
Supports loop-over-tuples-in-bash for parameter assignment.
Changed some debug and error log to println, since our Datadog does not support info(not sure why) and debug, and it takes println as info.

This PR does not:

Guarantee large total nodes (> 100) and large tx size (> 1e7 bytes) could finish runs successfully, I still meet lots of time outs using nginx.

Key places to review:
scripts/aws_ecs_nginx_benchmarks.sh
How to test this PR:
Follow the testing steps here: https://github.com/EspressoSystems/HotShot/pull/2793 .
And change the script to ./script/aws_ecs_nginx_benchmarks.sh.
Remember to run nginx following the step mentioned in the script.

Coverd by https://github.com/EspressoSystems/HotShot/pull/2858


Codechef  Bitwise tuples(BITTUP).cpp solution

I want to add the solution of a great question on CODECHEF i.e. Bitwise Tuples.
https://www.codechef.com/JUNE21C/problems/BITTUP
I want to work on this issue.
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

I want to work on this issue using C++, Kindly assign this issue to me.


Custom "disk offerings"with "custom disk size" not working after upgrade from 4.16.1

ISSUE TYPE

Bug Report

COMPONENT NAME

Management Server

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION

OS / ENVIRONMENT

Ubuntu 20.04
SUMMARY

After upgarding to <IP_ADDRESS> from 4.16.1 my cutome disk offerings with "custom disk size" option aren't working anymore.
Errormessage:

No log entries in the management-server.log
STEPS TO REPRODUCE

Create a disk offering with option "custome disk size" in  CS <IP_ADDRESS>
Use disk offering for creation of a new instance
Enter value for disk size
launch instance

EXPECTED RESULTS

Instance is generated and started

ACTUAL RESULTS

Error in GUI; see screenshot above

@Hudratronium this has been fixed with https://github.com/apache/cloudstack/pull/6586.
You may use API to deploy such VMs with <IP_ADDRESS> packages. You may also try latest nightly packages, https://download.cloudstack.org/testing/nightly/latest/

Thanks for this info.
I am just looking but do we have already a planned releasewindow for 4.17.1?
As this disables a "customer" faceing function, a remark on this in the releasenotes for 4.17.0 / <IP_ADDRESS> might be usefull...

@Hudratronium https://markmail.org/message/jjbxfv6tlx6s2tj2

Thanks - for me this is solved looking forward to the release of 4.17.1


Switch to isotope-packery, or add option for isotope-packery.

We're using vue.isotope but are running into a problem, vueisotope uses isotope-layout which doesn't have the packery layout option. Ideally there should be some way to use isotope-packery (v2) instead.

you can change layout, see this fiddle:
https://jsfiddle.net/dede89/d117mj5u/
vue-istope-chemistry-example - JSFiddlehttps://jsfiddle.net/dede89/d117mj5u/
jsfiddle.net
Test your JavaScript, CSS, HTML or CoffeeScript online with JSFiddle code editor.
You may have install it with bower or npm.

De : bcwp<EMAIL_ADDRESS>Envoyé : jeudi 6 octobre 2016 02:26
À : David-Desmaisons/Vue.Isotope
Objet : [David-Desmaisons/Vue.Isotope] Switch to isotope-packery, or add option for isotope-packery. (#10)
We're using vue.isotope but are running into a problem, vueisotope uses isotope-layout which doesn't have the packery layout option. Ideally there should be some way to use isotope-packery (v2) instead.

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/David-Desmaisons/Vue.Isotope/issues/10, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAgBSZwftsfu-N7hFPsKpDx9u3qHg3FPks5qxFxigaJpZM4KPe0G.


Mock reconciliation service

Close #2398
Now the reconciliation service used by WikitextImporter is successfully mocked.

BTW, PreviewWikibaseSchemaCommandTest.testValidSchema makes call to Wikidata during inspection, and it's the last test that depends on external services. Should I open a new issue for that?


Add debug section to Contributing.md

[ ] Did you write/update appropriate tests
[ ] Release notes updated (if appropriate)
[ ] Appropriate logging output
[ ] Issue linked
[ ] Docs updated (or issue created)
[ ] New package licenses are added to ThirdPartyNotices.txt (if applicable)

Very rough, this has been my experience so far! Please fell free to fill in alternative methods @timrogers @dylan-smith

I resisted writing docs like this in the past because it feels like generic "how to debug a dotnet project" guidance that I hope exists elsewhere at a more authoritative source that we could link to.

I resisted writing docs like this in the past because it feels like generic "how to debug a dotnet project" guidance that I hope exists elsewhere at a more authoritative source that we could link to.

If there is a more authoritative document please let me know! We can pop them in. I think facilitating this for devs would make it one step easier to start contributing IMO

2. https://code.visualstudio.com/docs/editor/debugging

Cool! How would folks feel about linking these docs then?

IMO the dotnet CLI is the bare minimum ppl should be familiar with and probably is the very first thing. For example dotnet run, dotnet test, dotnet build, etc. So I don't think that we should document them.
Regarding how to pass command args, it may be useful to add a link.
Those being said, maybe we can add a Useful links section to the end of the doc and add link for both .NET CLI docs and the vscode debugging doc

@ArinGhazarian I really like your suggestion of adding a helpful links section! I also think what I wrote out is something that could be helpful (perhaps written in a more robust/elegant way?).
My vision for writing this is enabling newer engineers, even level 1 engineers who have little to no C# experience to get a command going and be able to set some breakpoints to explore the codebase 😄

@ArinGhazarian I really like your suggestion of adding a helpful links section! I also think what I wrote out is something that could be helpful (perhaps written in a more robust/elegant way?).
My vision for writing this is enabling newer engineers, even level 1 engineers who have little to no C# experience to get a command going and be able to set some breakpoints to explore the codebase 😄

I see value in both having a quick command and how to run examples and also should have useful links reference. I wonder why we are so anti-docs in this repo? It shouldn't be so much pushback for a change that will be useful to some (and the others like yourselves who are familiar can just ignore). I definitely recall having to explicitly ask how to run things when I did some dev work here and would have found these quick references useful.


TextInput > Placeholder prop does not re-render

Consider the following example:
<TextInput
  placeholder={this.state.placeholder}
/>

The placeholder will not re-render with new text, even if state changes.

I just gave this a test and was unable to repro in Snack with SDK 19 (RN 0.46.1). Could you provide more information?
https://snack.expo.io/Byxte-gvZ

Hey, thanks for reporting this issue!
It looks like your description is missing some necessary information, or the list of reproduction steps is not complete. Can you please add all the details specified in the template? This is necessary for people to be able to understand and reproduce the issue being reported.
I am going to close this, but feel free to open a new issue with the additional information provided. Thanks!


Original enum

I rewrote enum.dt
It works similar to the original enum version again, where you had to write (enum value) instead of just value when it was defined like this (def enum (enum intern uintt ((value 0))))
This is useful, because some enums may be big and contain many already used names (when they are similar to the c-names of objects, but without prefix for example) or when multiple enums contaion the same objects (see #149).
If the enum is defined in another namespace, the operations are also defined in this namespace and you have to write enum-namespace.+ to sum 2 enums for example.
Now def-enum defines the new type in the current namespace, and creates a namespace, where the variables of the enum are defined. it will also create a macro similar to the original version of the macro.
so there are 3 ways to use the enum variables:
(enum value) (expands to (using-namespace enum value)) (also works with a expression using value)
enum.value
(using-namespace enum …value…) (this may be most useful when enum values should be used as global variables (because they don't conflict with other symbols normally))
Currently this does not work correctly since the subnamespaces of the namespace, where the enum is defined is not inherited and only referenced by name. this could be solved with #154

I'm not really sure, if there isn't a better solution

Another option may be defining the functions of the enum in the default namespace. I think this is the better solution

I think, the other solution would be preferable


Revert "Use platform appropriate filepaths"

Reverts flutter/flutter#44221
This introduced https://github.com/flutter/flutter/issues/46163

Looking to add a test.

Going to close this and re-open against a different branch.