pretty rtype logo ?

I listened to this podcast recently:

https://devchat.tv/js-jabber/205-jsj-shasta-with-eric-schoffstall

In it, @contra credits having a catchy logo as part of the success of his open-source projects. He even suggests something like https://www.squarespace.com/logo/ is a great way to get started with a logo.
Any ideas?

I used to be logo designer.

Ill do one this month just for fun. It's been a while since Iv touched Illustrator.

😄

We could also have a logo-treatment of some actual rtype text. We could use fun, frivolous examples of rtype for this:

fn (rtype: Spec) => 😀

That's easy :stuck_out_tongue_winking_eye:
JavaScript

TypeScript

RType

Looks like a logo for a natural language search engine.

How about these for a couple of concepts? Further derivations at https://rlogo-jslog.c9users.io/js-jslog.github.io/index.html

What does the B/8/∞/tape/goggles/mask represent?
I also have a logo ready. I just don't have time right now to vectorize it.

@js-jslog Looks too much like a B, I think.

@ericelliott agreed. I've added a few more ideas here which are more clearly R shaped. Out of interest, what is the meaning behind the name rtype?
@Mouvedia The idea is that the 'type' is preceded by a colon (as you would find in the rtype syntax) and the R is built around that colon. Neither the R nor the colon were clear enough so I have tried to remedy that. Let me know what you think. Here's one example, there are a few more at http://js-jslog.github.io/.
If anyone wants to develop on this idea at all I would welcome that.

r-type inspired the rtype name. =)

I'm pleased to hear that


fix: JS option missing for some style items in table widget v2

Description
The JS toggle option was missing in Text formatting section of table widget v2 for:

Emphasis
Text Align
Vertical Alignment

Enabled the options in styleConfig
Fixes #2906
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manually tested

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] 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
[ ] New and existing unit tests pass locally with my changes

/ok-to-test sha=7f71c46

/ok-to-test sha=3ecdbe1

/ok-to-test sha=617420f

Tested and verified.


Create Controller and Views

In this milestone, complete the list of requirements as given below:

Generate the controllers for the Splash Page, Home Page and Transaction Page.
Add the views by adding the actions into the controller.
Set the routes as per the requirements in order to see the page.

Feature Implemented #13


Alpha of ARM packages

@ahmetalpbalkan I had to create a new PR. Sorry.
Signed-off-by: Brendan Dixon<EMAIL_ADDRESS>

Hi @brendandixon, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@ahmetalpbalkan I will merge this and fix any issues that remain to get around the bootstrapping problem with CI.

This PR was green. Means, we are cool. :-)


[WIP] Remove PF_Ring Submodule

Issue
Using PF_Ring as a submodule and compiling the detector using static libraries is limiting.
Solution
Install PF_Ring as a dependency on the station and then link against libpfring/libpcap shared object files. This way we can more easily package and distribute station code without managing pf_ring kernel module.
TODO

[x] Compile using PF_Ring shared object files
[ ] Test which PF_Ring versions are compatible with current station code
[ ] Modify .github/workflows/build.yml to use new build process.

[ ] Update Readme for install instructions
[ ] Create a test release

PF_Ring provides packaged installation options from 7.5 onward. Current default installed by apt is 7.8
Tested successful station compilation:

[x] 6.4.0-stable (from source)
[x] 7.2.0-stable (from source)
[x] 7.4.0-stable (from source)
[x] 7.8.0-stable (from package)


Calculate perplexity

Hello,
I want to evaluate the log probability (really the perplexity) of a sentence of text. Can i use your library for this?

Hello,
Yes: first apply token_logprobs with add_bos=True, end_of_prompt="", then drop the first None in each list, and then apply agg_log_probs.
But you don't need CAPPr for this computation. See this HuggingFace tutorial.
You may want to use CAPPr if your data looks like this—
text 1: {same_prompt} {different_completion_1}
text 2: {same_prompt} {different_completion_2}
.
.
.
text n: {same_prompt} {different_completion_n}

In that case, use predict_proba or predict_proba_examples and pass normalize=False to get the log-probability of every completion. CAPPr caches same_prompt to save time.

Thank you for the quick reply! The reason I came across your package is because I am on a short deadline and I have to use a model other than GPT 2 since I used them previously. If this is something you can help with, we can add you to the acknowledgments section of this paper. Do you have other models that work with your code (or another approach you may have the code for?)
https://arxiv.org/abs/2308.14132

Both CAPPr and the HuggingFace tutorial should work for any model that can be loaded via AutoModelForCausalLM.from_pretrained, e.g., GPT-J, Llama, Mistral, Bloom.
In the HuggingFace tutorial, change
from transformers import GPT2LMHeadModel, GPT2TokenizerFast

device = "cuda"
model_id = "gpt2-large"
model = GPT2LMHeadModel.from_pretrained(model_id).to(device)
tokenizer = GPT2TokenizerFast.from_pretrained(model_id)

to
from transformers import AutoModelForCausalLM, AutoTokenizer

device = "cuda"
model_id = "your-autoregressive-lm"  # for example, "EleutherAI/gpt-j-6B"
model = AutoModelForCausalLM.from_pretrained(model_id).to(device)
tokenizer = AutoTokenizer.from_pretrained(model_id)

and you should be good to go

@galonpy lmk if you need more help. If you're all set, feel free to close the issue


Update Call.html

Include two typo fixes..
There's one change I made -> Calls -> Calls, cuz it be clear to the reader what call we are talking about..and that 's' added with the code tag seems kind of vague...What do you say about it?

Just realised that it be good if my commit message would be 📝Fixed typos
Will take care next time!


How to get all the vector dof lists for the fespace?

I am not clear about the use of
void GetVDofs(int vd, Array<int> &dofs, int ndofs = -1) const;
what does the vd mean?
Thanks,
Adam

A vdof is associated with "vector dimensions". vd denotes the "dimension". Take for example a velocity field in 2D. The size of the vector is dim * ndofs, so in this case vdim = spatial_dim.


Op 10608

We prefer small, well tested pull requests.
Please refer to Contributing to Spinnaker.
When filling out a pull request, please consider the following:

Follow the commit message conventions found here.
Provide a descriptive summary for your changes.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Add inline code comments to changes that might not be obvious.
Squash your commits as you keep adding changes.
Add a comment to @spinnaker/reviewers for review if your issue has been outstanding for more than 3 days.

Note that we are unlikely to accept pull requests that add features without prior discussion. The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.
https://devopsmx.atlassian.net/browse/OP-10608

This PR  triggered this build in this execution, please merge to master only if this succeeds:  https://spin.prod.opsmx.com/#/applications/gcp-gitprtrigger/executions/details/01FR5FN33VNYNY68KETPT7X5EV


generate_schemas.py script should use textwrap instead of forking an fmt process

When generating the embedded text representing the JSON schema of a protocol element, the generate_schemas.py script pipes the text through fmt to wrap the text. The textwrap module from the standard library provides a much better way to do this.

working on this

Fixed in #164


Using newline element returns 'build_left_prompt:11: command not found: prompt_newline'

In my zshrc, adding newline to POWERLEVEL9K_LEFT_PROMPT_ELEMENTS between dir and rbenv does not work properly. Instead, upon opening zsh the message build_left_prompt:11: command not found: prompt_newline is returned. I've tried this with and without any preexisting settings involving multiple lines, but the same result is given.
For reference, these are all the powerline settings currently in my zshrc.
POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(context root_indicator dir rbenv vcs vi_mode)
POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=(history time)
POWERLEVEL9K_PROMPT_ON_NEWLINE=true
POWERLEVEL9K_RPROMPT_ON_NEWLINE=true
POWERLEVEL9K_PROMPT_ADD_NEWLINE=true

POWERLEVEL9K_MULTILINE_FIRST_PROMPT_PREFIX=""
POWERLEVEL9K_MULTILINE_SECOND_PROMPT_PREFIX=" ↳ "

POWERLEVEL9K_DIR_DEFAULT_FOREGROUND='white'
POWERLEVEL9K_DIR_HOME_FOREGROUND='white'
POWERLEVEL9K_DIR_HOME_SUBFOLDER_FOREGROUND='white'
POWERLEVEL9K_ROOT_INDICATOR_FOREGROUND='black'
POWERLEVEL9K_ROOT_INDICATOR_BACKGROUND='yellow'
POWERLEVEL9K_VI_MODE_NORMAL_BACKGROUND='112'
POWERLEVEL9K_VI_MODE_NORMAL_FOREGROUND='022'
POWERLEVEL9K_VI_MODE_INSERT_BACKGROUND='white'
POWERLEVEL9K_VI_MODE_INSERT_FOREGROUND='027'

Hey @jcoolrace - I copied your settings, and am unfortunately not able to reproduce your problem. This is what I see:

Can you share your entire .zshrc?

As soon as you said that, I realised my clone of the repo hasn't been updated for a very long time! Running git pull fixed all my problems. My apologies for any confusion you might've had.


Page is out of date with current VS2022 17.8.3

in 17.8.3 aspnetcore-https.js no longer exists in the project. The screenshots still show preview on the project template which is gone. And the extra settings for publishing al already done in the template
It's still usuable but maintenance is key.

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

ID: c924f1c1-823e-3e29-bf09-405fc4c6bbb8
Version Independent ID: 585c2426-5f9b-8499-8eda-0e56bd64c93c
Content: Create an ASP.NET Core app with Vue - Visual Studio (Windows)
Content Source: docs/javascript/tutorial-asp-net-core-with-vue.md
Product: visual-studio-windows
Technology: vs-javascript
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@bzuidgeest Thanks for the feedback! Updating the article.


Added .eslintignore file | Suppress Eslint rules | Changed noopener to noreferrer

This built fine in the deploy preview, and I didn't see any obvious problem when going through the deploy preview site, so I'm going to merge the PR and let the site build in staging proper.


select options not showing

Hello,
I have this piece of code along with other inputs and checkboxes, but I do not know why select does not show the options within my array.
`{
className: "col-xs-8",
key: "dropdown",
type: "select",
templateOptions: {
options: [
{
"name" : "Option-A",
"value": "OptionA",
                           }
                ]
            }
        }

`
If I set another set of name/value I can see my dropdown having two empty options, but no text.
Can anyone please tell me what Im doing wrong?
thanks a lot

@mptorrejon  Hi, did you solve your problem?

@turlife see https://github.com/formly-js/ng-formly/blob/master/demo/hello.ts#L159-L169


Added possibility to use projectId as param instead of project name

Hey!
We took a look again on this ticket and we have some new info here: The project id is not constant across project boundaries (for example a new id is generated for a project if you export and import it). Hence we wouldn't see project id usage as a good practice and instead encourage usage of the project name.
So we cannot accept this pr, I'm sorry.
Kind regards,
Hannes


Support collecting worker metrics in collectMetrics command

What changes are proposed in this pull request?
#14966 : Support collecting worker metrics in collectMetrics command.
Why are the changes needed?
Support collecting worker metrics in collectMetrics command
Does this PR introduce any user facing changes?
None

@beinan Thanks for your review. PTAL.

will it take too much time if there are hundreds of alluxio workers?  Is it possible to add a command argument/flag to enable/disable the metrics collecting from workers?

will it take too much time if there are hundreds of alluxio workers? Is it possible to add a command argument/flag to enable/disable the metrics collecting from workers?

@beinan Thanks for your review once again. I have added a command flag only-master to disable the metrics collecting from workers.

I've added command flag. @beinan PTAL. Thanks

alluxio-bot, merge this please.


chore: remove/replace SLSA1 and SLSA2 tests

Description
We deprecated SLSA1 and SLSA2 collections/configurations/pipelines, this removes those from e2e tests.
Issue ticket number and link
https://issues.redhat.com/browse/EC-195
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
In CI :)
Checklist:

[X] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/retest

/test redhat-appstudio-e2e


Remove hdwallet provider usage

@sirpy pls review

@fernandomg any idea why snapshots fail again? what causes this?

Removing the async and the await renders the functionality of "ready()" redundant. since the purpose of ready is to return a promise which is resolved once the wallet is ready.
Do we need a ready method? Are any of the methods generating the addresses, async?

@sirpy, If I understand what you're referring to, in this commit ready was removed https://github.com/GoodDollar/GoodDAPP/pull/53/commits/2605045cbd55f209ea3a3a680276b0aad93f5edf

testing slack

testing slack. @hadarbmdev

testing again. @hadarbmdev

@hadarbmdev
again.sorry.


Foreign currency amounts mess up statement parsing

The parser does not handle foreign currency transactions which result in an extra item in the PDF text array e.g.
[ "0000 0000 0000 0000", "Jun 15", "Foreign transaction", "11111111111111111111111", "-100",  "Foreign Amount 123456" ]

This currently skews the transactions that are read in.

Updated statement parsing to handle international transactions.


Reason for eval echo?

Handy script! I have enjoyed using it. Thank you for creating it.
Today, I was using dotenv-shell and encountered a problem when my .env file contained a value that contains an ampersand. Example: KEY=val&ue
When running dotenv against this key value pair example, I get a -bash: ue: command not found error message as the val&ue is getting split on the ampersand.
While browsing the source code for this repo, I noticed this eval echo line. If I removed this 1 line and then change the next line line to eval export "$key='$temp'";, the script behaves as I expect it to and exports the key value pair as I typed it in the .env file.
With all this being said, what is the reason for the eval echo line? I ask as I am not advanced with bash and assume that I would be missing out on functionality if this code is removed.
Thank you!

After playing around with the test script, I am understanding the dotenv script more and more and now understand the need for the eval command.
Closing


fix action not processing metric

Description
Since the metric is a gauge, it will continue reporting the value once an action starts not processing and will never recover from the metric's perspective even if the action has actually recovered. Setting the gauge back to 0 should handle recovery without requiring a restart.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[X] Scheduler
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[X] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[X] I signed an Apache CLA.
[X] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[X] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #5395 (70162e3) into master (cb3b64f) will decrease coverage by 72.36%.
The diff coverage is 0.00%.

:exclamation: Current head 70162e3 differs from pull request most recent head e9f29cd. Consider uploading reports for the commit e9f29cd to get more accurate results

@@            Coverage Diff             @@
##           master   #5395       +/-   ##
==========================================
- Coverage   76.87%   4.52%   -72.36%
==========================================
  Files         240     240
  Lines       14597   14600        +3
  Branches      620     634       +14
==========================================
- Hits        11221     660    -10561
- Misses       3376   13940    +10564

Impacted Files
Coverage Δ

...e/openwhisk/core/scheduler/queue/MemoryQueue.scala
0.00% <0.00%> (-82.36%)
:arrow_down:

... and 204 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Ensure we have test coverage for Secrets API

This covers:

[x] Ensure that the existing tests in the repo are running agains the new API shape.
[ ] Add additional tests for the API shape
[ ] Add support for record and playback for the tests.

Here's my current plan:

Upgrade the keyvault tests to reflect the API changes.
a. The API changes can be seen in examples/singleService.ts.
b. Make sure to use ms-rest-nodeauth instead of ms-rest-js.
c. Make a Pull Request.
Add more tests.
a. List all the tests available in other packages (including implementations in other languages or implementations made by the community) as long as they are not reflected in the currently available tests within the keyvault folder.
b. Make pull request for each one of the new tests.
Add support for record and playback for the tests.
a. Check this PR: https://github.com/Azure/azure-sdk-for-js/pull/2227/files and the related issues.

Closed on favor of:
https://github.com/Azure/azure-sdk-for-js/issues/3642
https://github.com/Azure/azure-sdk-for-js/issues/3643
https://github.com/Azure/azure-sdk-for-js/issues/3644
https://github.com/Azure/azure-sdk-for-js/issues/3645
https://github.com/Azure/azure-sdk-for-js/issues/3646


Support for new dlib 19.7 stuff

Bump version to 1.1.0
Require dlib 19.7 and face_recognition_models 0.3.0
Support the new slimmer 5-point face post model when creating face encodings. This speed things up slightly (a few fps in some cases)
Update docs/Dockerfile everywhere to say dlib 19.7 instead of dlib 19.5

See http://blog.dlib.net/2017/09/fast-multiclass-object-detection-in.html for context on dlib speed-ups
Now that redundant face pose models are being loaded, it makes sense to start lazy loading them to save ram (especially on an RPi). But I didn't do that yet because I haven't tested how it might interact with python's different multiprocessing modes when using more than one thread.

Great work as always. Two suggestions: The user is currently not able to set the type of model used in the face_encodings function. Also the use of "model" might be a bit ambigious because of "cnn" and "hog". Maybe use landmark_model instead?

@lolstatsguy this should answer your question https://github.com/ageitgey/face_recognition/issues/169

@eloelo22 I am very familar with dlib and these model-typed. I am refering to the changes is this commit. One model is used for detection the other one for landmarks. It might be better to change this in the code for better understanding. So "Hog" and "Cnn" may be renamend to "detection_model" and "small" and "large" to "landmark_model" because they are used very different in this libary which might be confusing

ow i am sorry i thought we had the same issue ✌

@lolstatsguy the user can't set the model there because the new model is just a slimmer version of the existing model - it just returns less points for each face. It's mainly useful to speed up face recognition where not all points are needed. It's not really that useful on it's own (unless you are doing something pretty special and then might just use dlib directly anyway).


vertex version increment when ridBag.embeddedToSbtreeBonsaiThreshold set to -1

OrientDB Version: 2.2.17
Java Version: 1.7.0_91
OS: centos06
Expected behavior
expected the version numbers on a give vertex to not change when adding edges if the config option was set. As described in the documentation here http://orientdb.com/docs/2.2/SQL-Delete.html (see Controlling Vertex Version Increments section)
Actual behavior
version number incremented on the associated vertices when an edge was added/deleted
Steps to reproduce
I added the following line to my config file orientdb-community-2.2.17/config/orientdb-server-config.xml
<entry value="-1" name="ridBag.embeddedToSbtreeBonsaiThreshold"/>

I then verified this was being set by going into the console and checking the value after typing config. It showed what I expected, -1
Next I tested created two vertices
@rid  @version
#10:1  1
#9:1    1

then I created an edge between them
create edge from #10:1 to #9:1

but listing the vertices again I see the following
@rid  @version
#10:1  2
#9:1    2

Am I interpreting the documentation wrong? I expected changes to edges not to affect the node version numbers

hi @creisle
that should be for the first time that requires to initialize the collection
do the version change if you re-execute this ?
create edge from #10:1 to #9:1

no it doesn't. I think perhaps I misunderstood how this worked. So does this mean that the node version will continue to increment as edges are added/update/removed unless it is a "multiple" edge or another edge between the same two nodes? Am I getting that right? because the versions still increment if I create a third node and add edges to it
If so that wasn't what I understood from the docs
PS I got the link wrong above its seems. should have been http://orientdb.com/docs/2.2/SQL-Delete-Edge.html

Hi @creisle
as long as the bonsai tree structure is created in the records involved, the version is not updated.
But if you create an edge between 2 records where for that type of relationship the tree is not yet created then the version is incremented
You can find more about RidBags here
http://orientdb.com/docs/2.2/RidBag.html
I guess the docs it's a bit unclear about that.
I will mark this as documentation
Thanks


Get all instances does not return symbols that are used as override

For a plugin that I am writing I need to check the master.id before importing a symbol. This is currently not possible and I have to import all symbols from a library before I can check the master ID. This issue has already been reported here: https://github.com/BohemianCoding/SketchAPI/issues/203
One of the side effects is that I need to clean up the imported symbol masters that are unused. I was hoping that this would happen automaticly on save but it does not appear to do so.
What I currently do is the following:
document.getSymbols().forEach((symbolMaster) => {
    if (symbolMaster.getAllInstances().length == 0) {
        symbolMaster.sketchObject.removeFromParent();
    }
});

One of the unlucky side effects is that all the nested symbols are also unlinked.

I could implement the following code:
https://github.com/Ashung/Automate-Sketch/blob/master/automate-sketch.sketchplugin/Contents/Sketch/Symbol/Remove_Unused_Symbols_New.cocoascript

But this means another loop over all my symbols and their overrides which I would like to prevent
Would it be possible to let get all instances also return instances that are used as override? Then I would not have this probem.
https://developer.sketchapp.com/reference/api/#get-all-the-instances

I think for us exposing the symbolsMaster.id on the importable objects would prevent the extra looping over the document for us. As we then can only import symbol master we actualy need, and therefore keep the file clean. a symbolMaster.isUsedIn(document) or saveTodelete would be nice though, but if it would be simular to our implementation then I don't mind using the same as wer currently do.


fix: cleanrooms - pass boto session to wait_query

Feature or Bugfix

Bugfix

Detail

When querying AWS Clean Rooms (cleanrooms.read_sql_query) with a boto session the library tries to call boto3.cleanrooms.get_protected_query with the default credentials instead of the session.

This PR passes the boto3_session to wait_query.
Relates

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 68c930c1cf1ebaa75e61f88f125aa35af0bd364d
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 68c930c1cf1ebaa75e61f88f125aa35af0bd364d
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Superseded by #2381, thanks


added:: Missing Meta Tags, robots.txt, sitemaps for SEO Enhancement

Issue #7

@yashksaini-coder Please add labels for gssoc and level tag. Without these gssoc will not consider this.


Fix the pinning of Rubocop version

This PR pins Rubocop to the last version that's supported by Ruby 2.5. I think it was improperly pinned/Rubocop's versioning went different than expected.

The lint step was running Ruby 2.5 and as a result was using Rubocop 1.28.2.
In the gemfile, gem "rubocop", "~> 1.21" allows for any versions < 2.
As a result, there's most likely drift between people's Rubocop version and the one used in the lint step.
Rubocop 1.29.0 is the first version that sets the requirement for Ruby 2.6+.

Some PRs are experiencing unrelated linting issues. I don't have the context to safely make these changes, but this change should unblock folks.

It's not clear why the lint step is failing:
https://github.com/httprb/http/runs/6890611978?check_suite_focus=true
Run bundle exec rubocop --format progress --color
Your RubyGems version (<IP_ADDRESS>) has a bug that prevents `required_ruby_version` from working for Bundler. Any scripts that use `gem install bundler` will break as soon as Bundler drops support for your Ruby version. Please upgrade RubyGems to avoid future breakage and silence this warning by running `gem update --system 3.2.3`
Error: unrecognized cop or department Gemspec/DeprecatedAttributeAssignment found in .rubocop_todo.yml
Did you mean `Gemspec/OrderedDependencies`?
Error: Process completed with exit code 2.

Locally this passes for me
➜  httprb-gem git:(drwl/fix-rubocop-issues) ✗ bundle exec rubocop --format progress --color
Inspecting 80 files
................................................................................

80 files inspected, no offenses detected

@tarcieri I'm not able to request reviewers, but this should fix the failed linting step in various PRs.

cc @ixti

Thanks!


How to unpack list parser to another list parser

Parser<Token> expressionStart = ...;
Parser<Token> expressionEnd = ...;
Parser<List<Token>> expressionBody = ...;
Parser<List<Object>> expression = expressionStart & expressionBody & expressionEnd;

and unpacking expression value to flat list
some thing like  expressionStart & ...expressionBody & expressionEnd will be good
Parser<List<Token>> expression = expressionStart & expressionBody.unpack() & expressionEnd;

Not sure what you expect unpack() to do, but there is (pick)[https://github.com/petitparser/dart-petitparser/blob/master/petitparser/lib/src/parser/action/pick.dart] that returns the n-th element of a list, i.e.
Parser<List<Token>> expression = (expressionStart & expressionBody & expressionEnd)
    .pick(1);

my task is getting flatten list of tokens Parser<List<Token>> expression.
expressionStart and expressionEnd needed too.
Parser<List<Token>> expressionBody = ...; list of token
and i can't assign
Parser<List<Token>> expression = expressionStart & expressionBody & expressionEnd
(sorry for my poor english ☹)

This also includes the start and end tokens:
Parser<List<Token>> expression = (expressionStart & expressionBody & expressionEnd)
    .map((parts) => [parts[0], ...parts[1], parts[2]]);


[webpack] Module not found: Error: Can't resolve 'serialport/package'

ERROR in ./node_modules/modbus-stream/lib/driver/serial.js
Module not found: Error: Can't resolve 'serialport/package'

What version are you using and in what system are you trying?

windows 10
webpack@^3

I'm not sure I can help as I don't use Windows. Are you able to just create a folder and install serialport package there?

I have installed<EMAIL_ADDRESS>but the error still occur.

Never tried above version 6.0.0 and never done it on Windows, can't help you 😢

fixed: add this config to webpack config:
module.exports = {
  resolve: {
    extensions: ['.json', '.ts', '.js'],
  },
}

Because this line does not use .json explicitly.


Preview rotated by 90 degrees.

When I add the preview layer to a view while the the physical device is in landscape the preview layer  is rotated by 90 degrees.
If I change the _currentVideoOrientation() function to
    fileprivate func _currentVideoOrientation() -> AVCaptureVideoOrientation {         return .portrait     }
everything seems to work as expected, but that fix just seems very hacky, so I was wondering if this may be a bug or if I'm doing anything wrong!?
I should probably mention that I only support the portrait orientation in my app and I'm pretty sure this has something to do with it.

I have a similar problem, I support only portrait in whole app except inside the "video recorder" section, where i support both landscape orientations. If i set camera.shouldRespondToOrientationChanges = true the camera follows the orientation when the device is portrait. I think (but not completely sure) the AVCaptureVideoOrientation should be derived from UIInterfaceOrientation instead of UIDeviceOrientation inside _currentVideoOrientation() -> AVCaptureVideoOrientation

_currentVideoOrientation() -> AVCaptureVideoOrientation implementation should be
switch UIApplication.shared.statusBarOrientation {
case .landscapeLeft: return .landscapeLeft
case .landscapeRight: return .landscapeRight
default: return .portrait
}

instead of
   switch UIDevice.current.orientation {
        case .landscapeLeft:
            return .landscapeRight
        case .landscapeRight:
            return .landscapeLeft
        default:
            return .portrait
   }


fix: allow multiple connections to work with the runner

fix: allow multiple websocket connections in parallel (custom threading / multiprocessing is initialized before patching for use with gevent)
feat: send events (during execution) to all connected (listening) websocket connections
Queries (placeholder_query) is still using a request-response pattern and does not multicast to all connections
green thread pool size for connections is limited to 8 for now

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

v0.5.0
GitHub release

Your semantic-release bot :package::rocket:


Integrate Gulp and Browserify

Browserify is a javascript utility that allows you to write JavaScript using modules. It works by compiling all of your javascript into a single file. Gulp is an automated build process which will automatically compile the javascript code. Integration of both these utilities will make a big difference in terms of creating code readability and reduction of code complexity.
I already have working versions of both these utilities on my personal repositories, so integrating them should be simple. The difficult part will be ensuring that salt correctly installs these utilities and builds the javascript.

This will also allow for the integration of javascript automated unit testing.

I think that the package.json and gulpfile here will do most of what we need it to do: https://github.com/brittainhard/fuzzyjson

Hi @brittainhard - what benefit will this provide us other than code readability. I've never used this tool and I'm weary of introducing things that make it so that folks like me can't contribute here or there. I'd like to know more about why we should be spending time on this, rather than adding new features. Thanks for listening and for letting me know.

@chrismattmann I think you bring up some good points. The use of Gulp and Browserify is not all that mainstream, and most people who have written javascript code don't have and knowledge of these tools. There is also a conceptual hurdle to overcome when using these tools. I think that in my situation, because of my familiarity with the tools, that the benefits far outweigh the technical debt that we might incur.
I should point out that the use of these tools does not put constraints on any other javascript development. Javascript files can still be loaded as static files apart from the build process, and only a trivial amount of knowledge of Django and the DOM is required to include new javascript staticfiles in the templates.
The use of Gulp and Browserify would greatly improve my development process. It would allow me to easily add new dependencies and makes the managing of static files trivial, since it compiles the selected files into one file. It would also allow me to write with modules, and I don't think I could overstate the benefits of this.
I will certainly take your points into consideration though before I start working on this feature.

Closing this. I think that the difficulties outweigh the benifits.


noob experience

I thought it might be nice to read about the experience of a java noob diving into the dropwizard documentation.

http://www.dropwizard.io/1.1.0/docs/getting-started.html#creating-a-configuration-class What is the purpose of this configuration class? With methods of setDefaultName it looks like a model to me, isn't this stuff suppose to go into entities?
When this class is deserialized from the YAML file which yaml file? Why is it deserialized? Why are we now talking about yaml when the configuration file itself is litered with JsonProperty annotations?

in IRC dreamreal (Joseph Ottinger. Used to be the editor of Java Developer Journal and then TheServerSide.com) has to say the following about it
<flip111> what's the purpose of this Configuration class?  http://www.dropwizard.io/1.1.0/docs/getting-started.html#creating-a-configuration-class   i see in the example methods like setDefaultName .. seems more like a model to me ..
<dreamreal> it's a model for the *application*
<flip111> don't really want to model my application at this moment
<flip111> it also says  "When this class is deserialized from the YAML file, "  which YAML file ?
<dreamreal> you're going to write a YAML file.
<dreamreal> Well, default name is for the app to spit out by default. So that's your "application model."
<dreamreal> Like if your app is to say "Hello, world" where "world" is paramaterized, "world" is the "default name."
<flip111> spit out where ?
<dreamreal> I don't know what the tutorial does - but given dropwizrd, I'm assuming it's going to start a webapp where you hit http://localhost/something
<dreamreal> and yes, it'll be http://localhost:port/hello-world
<dreamreal> and yes, it'll be http://localhost:8081/hello-world based on the rest of the configuration
<flip111> ok that's not so obvious yet ..
<flip111> i'll make notes
<dreamreal> It is a tutorial that actually progresses as it goes along. I don't understand tutorial writers who do that, though.
<dreamreal> I mean, why don't they SAY "this is what we're working toward" at the very beginning?
<dreamreal> "Getting Started will guide you through the process of creating a simple Dropwizard Project: Hello World." ... no "We're going to say 'hello world' through an HTTP endpoint, allowing us to change defaults through configuration and using a data representation model that we can change to fit our needs..."
<dreamreal> that way, the "configuration" makes more sense and you know you're going to be creating something on the web...

Loading templates and default values from YML seems not really necessary to get going, would be better if this was more obvious in the code itself. Later i would not put my html templates in yaml files anyway so i don't understand this choice for the tutorial.

-- edit and add more later maybe

@flip111 Thanks for your feedback!

http://www.dropwizard.io/1.1.0/docs/getting-started.html#creating-a-configuration-class What is the purpose of this configuration class? With methods of setDefaultName it looks like a model to me, isn't this stuff suppose to go into entities?
When this class is deserialized from the YAML file which yaml file? Why is it deserialized? Why are we now talking about yaml when the configuration file itself is litered with JsonProperty annotations?

I think these questions are answered in the very first paragraph of that section:

Each Dropwizard application has its own subclass of the Configuration class which specifies environment-specific parameters. These parameters are specified in a YAML configuration file which is deserialized to an instance of your application’s configuration class and validated.

@joschi it's not clear what is meant here with "environment" and why methods like setDefaultName should go there. I mean "specifying environment variables" is a mean to what ends?

Why do we need such a configuration anyway? Think about a simple REST API that just returns stuff from JAX-RS resource classes. For me it is unclear why I want to pass default values through a configuration class? A minimal example should work without such a configuration class?!

If every time you start your app, a field will always have the same value, then you are correct that you can do away with the configuration aspect of that field, and leave it hardcoded in the project.
It's only when you expect this field to take on different values on start up (think of accessing a DB on localhost vs a remote host) that it is helpful. The default value of the field could be "localhost" so that when the DB is deployed on localhost one can omit: db_location: localhost in their yaml file. Yet when the DB is moved to a different box, instead of having to recompile the application with the new location (if we had hardcoded it), one can just add db_location: <ip> to the configuration to override the default.

Closing this as the original question appears answered. If not, and this is an issue with the Dropwizard project, let me know and I can re-open the issue.


Integrate jtreceval

WIP, addresses #2247, replaces #2276
Edit: command for running trec_eval:
java -jar target/trec_eval.jar qrels.txt runfile.trec

Looking to mirror what's done in pyserini, basically adding more checks for command line and runfile formatting.
Also looking to integrate built-in qrels from anserini-tools.
Let me know if this is what you had in mind @lintool

Pulled branch, did a simple test on my iMac Pro (intel):
(pyserini-dev3) jimmylin@iMac-Pro anserini % tools/eval/trec_eval.9.0.4/trec_eval -m map -m P.30 tools/topics-and-qrels/qrels.adhoc.401-450.txt runs/run.disk45.robust04.bm25
map                   	all	0.2515
P_30                  	all	0.3560
(pyserini-dev3) jimmylin@iMac-Pro anserini % java -jar target/trec_eval.jar  -m map -m P.30 tools/topics-and-qrels/qrels.adhoc.401-450.txt runs/run.disk45.robust04.bm25
map                   	all	0.2515
P_30                  	all	0.3560

Works as expected... so far, so good!

I think we should remove all instances of the 9.0.4 binaries so we don't get confused?

I think we should remove all instances of the 9.0.4 binaries so we don't get confused?

Good idea, I've removed them

I've pulled the latest branch and ran the following tests:
https://github.com/castorini/anserini/blob/master/docs/regressions/regressions-disk45.md
nohup python src/main/python/run_regression.py --index --verify --search --regression disk45 >& logs/log.disk45 &

# old way
tools/eval/trec_eval.9.0.4/trec_eval -m map -m P.30 tools/topics-and-qrels/qrels.adhoc.401-450.txt runs/run.disk45.robust04.bm25
tools/eval/trec_eval.9.0.4/trec_eval -v

# new way (1)
java -jar target/trec_eval.jar -m map -m P.30 tools/topics-and-qrels/qrels.adhoc.401-450.txt runs/run.disk45.robust04.bm25
java -jar target/trec_eval.jar -v

# new way (2)
java -cp target/anserini-0.24.1-SNAPSHOT-fatjar.jar trec_eval -m map -m P.30 tools/topics-and-qrels/qrels.adhoc.401-450.txt runs/run.disk45.robust04.bm25
java -cp target/anserini-0.24.1-SNAPSHOT-fatjar.jar trec_eval -v

Everything works as expected on Linux (linux.cs.uwaterloo.ca), Mac Intel (my iMac Pro), and Mac ARM (my Mac Studio).
👍


Consider switching to quantikz for latex_source drawing

What is the expected enhancement?
As per the title, the quantikz library for tikz, available here, seems to have better drawing options for circuits. Also, the development of qcircuit seems to have stalled, while the author of quantikz is very quick to respond and integrate changes in the library.

This issue is related to #763

@maddy-tod actually this is an separate independent issue. #763 is about rewriting the python logic in the latex module to use a python library to handle actually writing out the latex code. (although I've been meaning to rewrite that issue to just be a general rewrite of the latex backend to be more clear, using a python lib or not) This issue is about changing the library we use in the LaTeX output generated by the python code.
@tigerjack I'm not sure I see an advantage for switching everything to use quantikz instead of qcircuit. While qcircuit doesn't have many patches in the last year, we haven't had any real issues with it. When we do (see adding barrier support for example) the maintainers of qcircuit are also very responsive to our needs. Do you have any concrete examples where quantikz provides an advantage over qcircuit for our use case?

@mtreinish from a programmatic perspective, I don't really know if they can be any advantage, I have not looked at the code to generate the latex in qiskit. I can only brought my personal experience with both of them, and I had a much easier lifetime in using quantikz wrt qiskit.
In the qiskit specific example, I think that exporting a simple circuit to quantikz could allow user to better personalize the circuit after qiskit have generated it.
Something like this

or this (look at the output bundle)

and even this

are not easy, if not impossible, to obtain in qcircuit.
It also allows to personalize the style of lines and gates, which could be useful also directly in qiskit in a similar way to what happens now with the matplotlib, with the main difference being that the latex source exported from qiskit can be directly modified. Indeed, it is a library of the well known tikz package, which is extensively used to generate figures and diagrams in Latex.
Also, and maybe most important of all, the syntax is pretty similar to qcircuit.

But again, it's only my experience, and maybe others have been widely using qcircuit without any problems.

So I can definitely see the advantage of using a tikz based package over an xymatrix/xypic based one like qcircuit. Tikz does provide a bit more flexibility in how you can interact with it, like in the first example you copied from the docs. The other 2 (minus the blue shading in the 3rd) are actually natively supported by qcircuit and in the docs. But, as you mention don't that provide an advantage for the way qiskit uses it, it would be more for people consuming the latex_source output to manually annotate. As for the style options the xypic and xymatrix packages support these things to a certain degree but they're not integrated into qcircuit which makes it difficult to modify the style of a circuit. If this is something people see value in we can probably easily add this to qcircuit.
We could do a side by side feature comparison, but it looks like they're mostly comparable and the author of quantikz based it off of qcircuit. But, after looking through the quantikz manual pdf the only thing really missing which is a blocker for any potential switch is support for barriers. The \slice command is close, but not quite all the way there. It only supports inserting the dashed lines through all bits.
That all being said I'm not particularly tied to using a specific library over the other. But, I prefer to avoid the code churn because generating latex programmatically like this is always going to have a set of quirks and by switching libraries we're going to trade a set of known quirks for new ones. I don't think we can even consider doing this until after 0.8 is released. But even then I'm not super inclined to rock the boat and switch unless there is a reason why qcircuit isn't working for us which we would fix by switching to quantikz (which I don't think there is).

@mtreinish yeah, just thought it was worth considering the switch since qcircuit seems to be a bit staled and most importantly quantikz is actively developed and based on the well-known tikz.
For the second picture, by the way, there is no way that I'm aware of to group the output bits (see https://github.com/CQuIC/qcircuit/issues/32). But again, this is usually needed only in post-processing the latex_source.
Regarding the barrier, I think you're right, I don't see any command to select the number of involved wires.

@1ucian0 If you think this is worth a shot, I'd like to give it a try :slightly_smiling_face:

I would encourage rewriting the latex circuit drawer to leverage the yquant library:
https://arxiv.org/pdf/2007.12931.pdf
https://github.com/projekter/yquant
It is based on TikZ and pure TeX, and is extremely flexible and easy to use. I recently typeset all the figures in here using this library: https://arxiv.org/pdf/2104.14722.pdf
There are some features of Qiskit drawer such as "justify" or "show_idle" or "show_barrier" or drawing "layout" that need to be maintained, but I'm hoping that the bulk of the code should become leaner as this library does the heavy lifting.

I would encourage rewriting the latex circuit drawer to leverage the yquant library:
https://arxiv.org/pdf/2007.12931.pdf
https://github.com/projekter/yquant
It is based on TikZ and pure TeX, and is extremely flexible and easy to use. I recently typeset all the figures in here using this library: https://arxiv.org/pdf/2104.14722.pdf
There are some features of Qiskit drawer such as "justify" or "show_idle" or "show_barrier" or drawing "layout" that need to be maintained, but I'm hoping that the bulk of the code should become leaner as this library does the heavy lifting.

In that case, wouldn't it be better to add an optional argument like backend="qcircuit" in the circuit_drawer option, only considered when the output argument is set to latex or latex_source, so that the user can choose between qcircuit, quantikz and yquant?
The rationale for this would be to let the user choose the framework they want to work with once the circuit has been exported to the LaTeX source. On the bright side, this only means adding code rather than changing/removing some, adds flexibility for the end-user and allow to deprecate more easily a method, were it not to be maintained anymore. Of course, that'd mean that the code would be harder to maintain, so I'm not sure whether that would be worth the effort.

It sounds like a lot of code to maintain for marginal utility.
Why not writing those drawers as external tools on top of qiskit-terra? We can work on a common plugin interface, if that's needed.

@1ucian0 I'm not sure I understand what you mean, can you elaborate on this please?

A circuit drawer is a rather independent module. It would be possible to write a drawer as a fully independent package on top of qiskit-terra. There is no need to modify terra.

Closing this for now, as a I think it should be implemented independently of terra. Please reopen (and tag as discussion) if you disagree.


what is "vt" status? what is the "t"?

First, I have to say "govendor" README.md/FAQ etc.. are really hard to understand for a newbie like me. I tried to find a tutorial or a book about govendor by google but failed. Whatever....
When I do govendor list, I see many vt status. I tried very hard to read everything about govendor in github and google, but I failed to find a single place where it mentions what the status "t" is. Thank you very much for your help.
PS: Could anyone please write a good documentation for govender? for example, I have no idea what the difference is between govendor install +local and go install.

t stands for "tree". It says the package is vendored (copied locally) and not just that package, but all folders and files under it too.
Normal vendoring in govendor just grabs a single package.

@kardianos Thank you very much for your quick reply. Just two more newbie questions:
(1) What is the best practice to vendor the required sub-package?
For the code:
import (
    "github.com/jinzhu/gorm"
    _ "github.com/jinzhu/gorm/dialects/sqlite"
)

govendor fetch github.com/jinzhu/gorm will not work. It requires the sub-folder dialects/sqlite.
(a) govendor fetch github.com/jinzhu/gorm/^ works, but some people say this is not good to do because it fetches more than I need. I am not sure if this is the best practice.
(b) govendor fetch +outside works only if I have the import codes. Is this the best practice? What if I don't write any codes yet and want to fetch what I need before writing any codes?
(c) any more better options?
(2) Can you please explain what the difference is between govendor install +local and go install?
Thank you very much.


Create NonNeg class, use NonNeg for ConicSolver, major web doc update

Well this took forever and a day, but here we go. The motivation for this PR was just to update web documentation. In the process it became clear that ConicSolver.format_constraints had been returning problem data for the nonnegative orthant, when it was labeled as the nonpositive orthant. The problem data was always correctly passed to solvers, so this isn't a correctness issue per-say, but it did mean that writing a new solver interface would involve a major headaches.
In the process of correcting the above issue I introduced a NonNeg class within the file cvxpy/constraints/nonpos.py (maybe we should change the file name). I then modified all conic solver interfaces to use NonNeg constraint objects rather than NonPos constraint objects -- again, this didn't actually mean changing logic in the solver interfaces. Since the data was already been passed as though it were NonNeg, changes to most solver interfaces were a matter of find-and-replace.
I moved one of the ConeDims classes to cone_matrix_stuffing.py, since the data returned by ParamConeProg  (defined in that file) can only be understood when using that ConeDims class. I moved another ConeDims class to qp_matrix_stuffing.py (the file was previously defined in solver.py, but only ever imported in the QP code path). I modified the ConeDims field to have NonNeg instead of NonPos for conic programs, but left the ConeDims class for QPs unchanged.
Somehow in a previous PR I failed to drop SuperSCS as we had intended. This PR also fixes that.

@SteveDiamond the ECOS_BB PR is here: https://github.com/cvxgrp/cvxpy/pull/973. That PR mentions dropping SuperSCS as part of an add-on for the ECOS_BB issue, which I somehow neglected to follow-through on. We discussed the user-displayed error message and documentation changes as part of that PR.

Thanks for pointing out the PR to drop ECOS_BB. Not sure why it slipped my mind. Anyway, the PR looks fine if you add tests for NonNeg.


Configuration Overwrite Keys Application Fix and VS2019 Build Fix

Hi!
I have been having a good look at the eCAL library and experimenting with this library on Windows 10 with Visual Studio 2019 (open folders) as the build environment.  I have made a few minor enhancements that facilitate using this impressive library in this environment and I would like to offer these for merging back upstream.
The first enhancement is a workaround for the following Visual Studio 2019 (16.6.2) cmake integration bug:
https://developercommunity.visualstudio.com/content/problem/1085862/cmake-build-reports-error-when-there-isnt-one.html
The file "thirdparty/cmake_functions/git/git_revision_information.cmake" uses the word "error" in a diagnostic message which is unconditionally emitted.  Whilst this has been reported to Microsoft, who have triaged the issue, it prevents successful builds so the workaround is necessary on current and older Visual studio releases.
The second enhancement relates to the application of configuration overwrite keys to the default eCAL configuration:
Presently the overwrite keys are only applied if an ini file is successfully opened.  In the scenario where an application is calling the "int Initialize(std::vectorstd::string args_, const char *unit_name_, unsigned int components_)" function and specifying an empty string for the "--ecal-ini-file" option, the ini file fails to load as expected and the default configuration is used as expected; however, the overwrite keys are subsequently ignored.  This use-case is for when an application configures itself and does not use an ini file on disk.  The enhancement provided in this pull request re-orders the "CConfigImpl::AddFile" function such that, regardless of the success or failure of loading an ini file, overwrite keys are always applied to the configuration.  It seems prudent and a light-touch to alter the behaviour in this way as opposed to attempting to handle a non-ini-file use-case in a different way more invasive way.
The last enhancement is a minor adjustment to the warning message emitted in the "CConfigImpl::AddFile" function.  This message has been re-directed via the eCal logging infrastructure rather than being emitted to the console (which may not be accessible or visible depending on the application).

Double checked the logging, the global logging isn't initialised when the ini file is loaded so the PR is incorrect so I closed the PR until I can correct this error on my part.

Hi John, thank you for that nice issue report. Great ! If you setup a new PR we can merge this easily.

@rex-schilasky I have squashed the commits and tested the changes.  The new PR is: #61

Thank you .. CI is ongoing. I reviewed your changes and it's all fine. Will merge soon ...


Access MeetupAPI to Pull Information about Our Events

GIVEN that the user wants to see the upcoming Events for our group
WHEN the user clicks on the Our Events link on the Nashville FCC website
THEN the user is presented with detailed information regarding our upcoming events (that is accessed from the Meetups API)

Awesome. I'll get to work on it as soon as I can.

just do some research on it and see if you can get some stuff pulled down using postman or something like it. thanks for responding so fast!

No problem! I'll see what I can find.

https://secure.meetup.com/meetup_api/console/
This is a wonderful way to test out Meetup's API. So thoughtful of them!

Also, we need to make this group an authentic member in order to test in Postman and use it in the site.

ya i was talking to laura about that last week. we are thinking there shouldnt be too many fetch requests to go over the free amount. i might just make a quick email to use for that.

Fair enough!
If we are looking for events based on freeCodeCamp, the best fetch call to do it is:
https://api.meetup.com/freeCodeCamp-Nashville/events
From there, we can add specifications to the way we list our card components on the website.

@JamesMcClarty  completed a Mockup API for accessing data.  Still need to access the real Meetup information. (FYI -- @Hellrungj )

Rebooting this ticket with #24


Added parser.Split()

I'd like to have access to the Split method in my Go projects.
Added invocation of parser.Split() and build an array of slices from its results.

Hey @lfittl ! Would you mind taking a look at this? This would enable us using this lib in some projects. Thanks a lot!

Hello @lfittl ! Sorry for the delay but I have pushed some changes based on your suggestions. I have also added SplitWithScanner. I'd appreciate if you have a moment to take a look. Cheers!

Hello @lfittl ! Sorry for the delay but I have pushed some changes based on your suggestions. I have also added SplitWithScanner. I'd appreciate if you have a moment to take a look. Cheers!

Excellent, thank you for making those changes!


Minimum RMSD does not work

I found an wrong behavior about the RMSD argument.
Here is my test sample, fragment.sdf, fragment_docking.in. Other data is the same as in testdata directory.
Though I set POSE_RMSD=10 but the RMSD of outputs seems much lower than the threshold. Could you check the bug?
fragment_docking.in
INNERBOX 10, 10, 10
OUTERBOX 20, 20, 20
BOX_CENTER 0.3826, 81.705, 109.195
SEARCH_PITCH 1, 1, 1
SCORING_PITCH 0.25, 0.25, 0.25
MEMORY_SIZE 8000
RECEPTOR testdata/2HU4_A_r.pdbqt
LIGAND testdata/fragment.sdf
OUTPUT testdata/fragment_docked.sdf
GRID_FOLDER testdata/grid
NO_LOCAL_OPT false
POSE_RMSD 10
POSES_PER_LIG 10

fragment.sdf
ligprep_1.smi:1
  -OEChem-10092314143D

 20 21  0     1  0  0  0  0  0999 V2000
    1.9431   -0.9280   -0.0056 C   0  0  0  0  0  0  0  0  0  0  0  0
    1.0078    0.0264   -0.4042 C   0  0  0  0  0  0  0  0  0  0  0  0
    3.2402   -0.8999   -0.5217 C   0  0  0  0  0  0  0  0  0  0  0  0
    1.3651    1.0144   -1.3210 C   0  0  0  0  0  0  0  0  0  0  0  0
    3.5995    0.0893   -1.4406 C   0  0  0  0  0  0  0  0  0  0  0  0
    2.6642    1.0414   -1.8369 C   0  0  0  0  0  0  0  0  0  0  0  0
    4.9496    0.1853   -2.0314 C   0  0  0  0  0  0  0  0  0  0  0  0
    3.0654    2.0739   -2.8058 C   0  0  0  0  0  0  0  0  0  0  0  0
    6.0000   -0.8247   -1.6340 C   0  0  0  0  0  0  0  0  0  0  0  0
    5.2908    1.0994   -2.8850 N   0  0  0  0  0  0  0  0  0  0  0  0
    4.3666    2.0151   -3.2564 N   0  0  0  0  0  0  0  0  0  0  0  0
    2.2639    2.9276   -3.1737 O   0  0  0  0  0  0  0  0  0  0  0  0
    1.6614   -1.6963    0.7089 H   0  0  0  0  0  0  0  0  0  0  0  0
    0.0000    0.0001    0.0002 H   0  0  0  0  0  0  0  0  0  0  0  0
    3.9437   -1.6575   -0.1909 H   0  0  0  0  0  0  0  0  0  0  0  0
    0.6226    1.7493   -1.6191 H   0  0  0  0  0  0  0  0  0  0  0  0
    5.9511   -1.6694   -2.3119 H   0  0  0  0  0  0  0  0  0  0  0  0
    5.8091   -1.1511   -0.6175 H   0  0  0  0  0  0  0  0  0  0  0  0
    6.9771   -0.3578   -1.6898 H   0  0  0  0  0  0  0  0  0  0  0  0
    4.6571    2.7208   -3.9258 H   0  0  0  0  0  0  0  0  0  0  0  0
  1  2  2  0  0  0  0
  1  3  1  0  0  0  0
  2  4  1  0  0  0  0
  3  5  2  0  0  0  0
  4  6  2  0  0  0  0
  5  6  1  0  0  0  0
  5  7  1  0  0  0  0
  6  8  1  0  0  0  0
  7  9  1  0  0  0  0
  7 10  2  0  0  0  0
  8 11  1  0  0  0  0
  8 12  2  0  0  0  0
 10 11  1  0  0  0  0
  1 13  1  0  0  0  0
  2 14  1  0  0  0  0
  3 15  1  0  0  0  0
  4 16  1  0  0  0  0
  9 17  1  0  0  0  0
  9 18  1  0  0  0  0
  9 19  1  0  0  0  0
 11 20  1  0  0  0  0
M  END
$$$$

The result says min_rmsd = inf


Sound only notification

It will be great to consider adding a sound-only alarm, in many cases no need for text notification

Hi! Sorry for taking so long to respond to your issue on GitHub. I was focused on releasing the new 0.7.0 version of awesome_notifications and awesome_notifications_fcm and i didn't had time enough to do both. But now I can answer all your questions.
So now i'm asking you to recreate this topic using the new issue template. There's a lot of missing informations that i need to understand your problem, and all the instructions are already in the new issue template. Also, remember to check beforehand if your issue was posted by another user.
So I will automatically close all previous issues so far.
Sorry for the inconvenience and i will be waiting for your new issue request.
Thank you so much for your support!


Missing comma in lambda orchestrator

Semmle found a bug in the lambda code:
https://lgtm.com/projects/g/Netflix-Skunkworks/cloudaux/?mode=list&tag=external%2Fcwe%2Fcwe-665

Implicit string concatenation. Maybe missing a comma?

Coverage remained the same at 34.151% when pulling 66cd55ac3dc13b9cbe2cf81e43e267b3bd0ce1b0 on scriptsrc:patch-1 into 9c807d04b83b6db76c0ec12bd381e76983e05d01 on Netflix-Skunkworks:master.

Good find.  Can you also update the version number in __about__.py?


Add shift operators

Please add shift operators on left and right to allow fast multiplication and division.

Please read the documentation, they've been there for a long time :)
https://xc-basic.net/doku.php?id=v3:shl
https://xc-basic.net/doku.php?id=v3:shr

I read it and couldn't find it easily. I expected to find them among the operators, but they weren't there.

It would make sense to put links to these functions on the Operators page. Will do that.


Add Z axis support for spatial averaging

Discussed in https://github.com/xCDAT/xcdat/discussions/591

Originally posted by tomvothecoder January 29, 2024
Is your feature request related to a problem?
I'm running into a case where I need to average over the Z axis. The CDAT code being used is cdutil.averager(tvar, axis="z"), which I need to replace.
xCDAT's spatial averager, which is based on cdutil.averager(), only supports rectilinear grids ("X" and "Y"). However, we do mention that the get_weights() method can be extended to support other axes:
https://github.com/xCDAT/xcdat/blob/fbf1db68fd6328d5a534e0c1e57e22e6fdd70c76/xcdat/spatial.py#L246-L252
Are there are any possible answers you came across?
No response
Describe alternatives you've considered
No response
Additional context
For the short-term, I might be able to use xarray.DataArray.weighted directly. However, I still need to generate the weights xr.DataArray beforehand.
datarray.weighted(weights).mean("height")

@tomvothecoder - FYI - I thought this might only be a couple lines and took a look. It ended up being a little more than I thought, but I committed a working prototype because I thought it might be helpful (needs validation, review to make sure the docs are updated correctly, and unit tests). It could be possible to combine some of the _get_XYZ_weights functionality, but maybe it is fine as-is. We can delete the branch if you have something in progress already. Or I can open a PR.

@pochedls Thanks for taking a stab at this so quickly! You can open a PR and tag me for review.
As a temporary workaround in e3sm_diags, I created a _get_z_weights() function that is based on xCDAT logic. I am planning on replacing it whenever xCDAT supports generating weights for the Z axis, which now seems sooner rather than later.


Add , and # support to multiline config

This config
> 1%,
last 2 versions

leads to Unknown browser query ""
https://browsersl.ist/?q=>+2%25%2C%2Clast+2+version

Now we have a temporary solution
https://github.com/browserslist/browsersl.ist/pull/435
The idea of the solution by @ai
https://github.com/browserslist/browsersl.ist/pull/435#issuecomment-1218990106

@AleksandrSl no, it is not in a progress and PR is welcome


Worker naming error when tag supplied

Supplying -c 0.3.0 as a commit to Brig will cause an error:
message: 'Pod "cross-compile-1516809571731-0.3.0" is invalid: [spec.volumes[0].name: Invalid value: "cross-compile-1516809571731-0.3.0": a DNS-1123 label must consist of lower case alphanumeric characters or \'-\', and must start and end with an alphanumeric character (e.g. \'my-name\',  or \'123-abc\', regex used for validation is \'[a-z0-9]([-a-z0-9]*[a-z0-9])?\'), spec.containers[0].volumeMounts[0].name: Not found: "cross-compile-1516809571731-0.3.0"]',

This is because the generated pod is not a valid DNS label. I think we have a fix in the pipeline for this, but I wanted to make sure to capture the error.

Resolved in #307


zenith: 0.10.0 -> 0.10.1

Motivation for this change
Things done

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

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

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

/nix/store/cp0wngf83ryhvnsx1ybja66kdip7wiz8-zenith-0.10.0         34.1M
/nix/store/d6r9rnryjlxdld21g1n3bffcqxh36bda-zenith-0.10.1         33.6M

would you mind removing the platforms from meta? the issue referenced is closed

@GrahamcOfBorg build zenith

would you mind removing the platforms from meta? the issue referenced is closed

done. thanks

It looks like it still doesn't build on aarch64 so I put back the platforms part.
I did a PR to fix it once https://github.com/NixOS/nixpkgs/pull/88616 but I gave up because of lack of interest. Feel free to reimplement it.

I see, thanks anyway.

Thanks


[ENTESB-5028] Create quartz persistent store test

Can one of the admins verify this patch?

https://github.com/wildfly-extras/wildfly-camel/commit/543797f65793edd17f40f3e82f986400dccb8210


CaseOfCase kind mismatch error fix

Closes #5922 by fixing order of type arguments as they are extracted.

most importantly, this function is in some general module, how can you be sure you're not breaking something else by reversing the order of the result?
I don't see any way around going through all transitive dependencies and ensuring you aren't breaking things elsewhere

Besides CaseOfCase transform I found only one place where this function extractTyArgs is used - in the PlutusIR.Analysis.Builtins.defaultUniMatcherLike and that one is only used in the CaseOfCase.
The tests for CaseOfCase transformation are all green so this gives me the confidence.

There's no comment on what that function is supposed to return, no examples etc, maybe that's how it's supposed to work for some other use case that isn't case-of-case.

I've added comment and test cases to explain the semantics as I understood them. I'd be grateful to @michaelpj  for checking those case cases making sure that I derived the intended behavior correctly.

please document the offending function, so that it's clear what it returns to reduce the possibility of us screwing things up the same way again

Done.

please add the test that caught the bug as a unit/golden test to make sure we never regress back

It was the property test that caught the bug, and its already there. I think that unit tests for the extractTyApp are enough to make sure we never have this bug again.

So the analysis is that this would only happen when we had case-of-case on a type that had two type parameters, is that right? That does indeed seem like something we might not have tripped up on until now. Hooray for the generators!

I have recorded the offender as a golden test ;-)


https://warpplus-05522e68.cloudflareaccess.com/warp

          https://warpplus-05522e68.cloudflareaccess.com/warp

The default connection protocol for this team is MASQUE.
Open an issue or send an e-mail to<EMAIL_ADDRESS>to let us know when it's full.
Originally posted by @0x24a in https://github.com/0x24a/WarpPlusKeyGenerator-NG/issues/11#issuecomment-2461303436
this link is full. please provide us the new link.

https://github.com/0x24a/WarpPlusKeyGenerator-NG/issues/11#issuecomment-2467210288


Milestone 1 Feedback

A+
A+

don't need 522 in the repo name
the licence should be copyrighted to your names not MDS (it is your work)

A-

Your question is not quite clear to me. What exactly is the supervised learning model going to predict? I.e. what are the class labels? Given a neighbourhood/time/etc are you going to predict whether a crime will happen? or what type of crime will occur (given that you know some crime has occurred)?

A+, A

so is zip_file_name the name of the unzipped file?

A+, A-, A, A- (a,q,v,r)

I don't know what "peak of data" means in this context
format dates in figures (remove commas)
Is midnight the placeholder for "no time recorded"?
You've found the neighbourhoods with the most crime and the most common crimes, how will this be used in your model?

A+

Thank you for your feedbacks! Following are changes made according to your suggestions

Removed 522 from the repo name
Updated names in the LICENSE to team members names
Updated the final report to clarify the research question and the analysis
Updated download_data.py by changing zip_file_name to mandatory
Corrected typos in EDA (e.g., peak to peek)
Formatted date in figures by removing commas, added explanations to date axis in EDA
Added explanations on how we will use some figures to support our analysis in EDA


[Bug] generation benchmark failed on llama2-chat-7b-w4

Checklist

[x] 1. I have searched related issues but cannot get the expected help.
[x] 2. The bug has not been fixed in the latest version.

Describe the bug
session 1 stats:
[[0.0, 2048, 0.0], [0.0, 2048, 0.0], [0.0, 2048, 0.0], [0.0, 2048, 0.0], [0.0, 2048, 0.0], [0.0, 2048, 0.0], [0.0, 2048, 0.0], [0.0, 2048, 0.0], [0.0, 2048, 0.0], [0.0, 2048, 0.0]]
--------------------------------------------------
profile_generation.py:143: RuntimeWarning: divide by zero encountered in scalar divide
  throughput = np.sum(stats[:, 1], axis=0) / np.sum(stats[:, 2],
--------------------------------------------------
concurrency: 1, input_tokens: 0, output_tokens: 2048
elapsed_time: 0.12s
first_token latency(min, max, ave): 0.00s, 0.00s, 0.00s
token latency(min, max, ave): 0.00s, 0.00s, 0.00s
throughput: inf token/s

and the following error returned.
concurrency: 8, input_tokens: 0, output_tokens: 2048
elapsed_time: 0.18s
first_token latency(min, max, ave): 0.00s, 0.01s, 0.01s
token latency(min, max, ave): 0.00s, 0.01s, 0.01s
throughput: -3200.00 token/s

Reproduction
python3 -m lmdeploy.serve.turbomind.deploy     --model-name llama2     --model-path ./llama2-chat-7b-w4     --model-format awq     --group-size 128
Error traceback
No response

#507 has resolved this issue.


alot: support contact completion

I would like to make contact completion easier, or at least have a valid configuration out of the box.
I am unsure about the interface, ideally the user should be able to pass anything. Should I provide a submodule or freeform attributes with a reasonable default ?
In alot-accounts.nix, I also wanted to refer to pkgs.notmuch, not sure how to cleanly achieve that though, callPackage seems to bring in a HM specific package.

Sorry about the delay! I had a look at the alot documentation and I suspect it would be a bit tricky to produce a sufficiently flexible interface using a submodule. So perhaps best to use a plain attribute set. I.e., contactCompletion having the type attrsOf str or possibly attrsOf (either str bool). Could default it to the empty set and use
      ++ optionals (alot.contactCompletion != {}) (
        [ "[[[abook]]]" ]
        ++ mapAttrsToList (n: v: n + "=" + v) alot.contactCompletion
      )

The option could have a fairly large example from the linked page and then I think most would see how it would work. I.e., something like this:
example = literalExample ''
  {
    type = "shellcommand";
    command = "abook --mutt-query";
    regexp = "'^(?P<email>[^@]+@[^\t]+)\t+(?P<name>[^\t]+)'";
    ignorecase = "True";
  }
''

So I think it's not necessary to pull pkgs.notmuch into the module. But had it been necessary then I believe the most straight forward way would be to add an initial pkgs argument to the module file. I.e., to put pkgs: first in the file and then in email.nix change
(import ../programs/alot-accounts.nix)

to
(import ../programs/alot-accounts.nix pkgs)

This is a wise approach. Will do as you say.

it does import pkgs, but then won't find config.notmuch. I believe modules have their own way of passing arguments but not sure how.

hopefully it should be ok now.

Thanks! Rebased to master in 81dae2f88eaff306f36b9072d2cb752f82dbc60a.

Thanks I am really happy to see this merged.
It seems like you removed the contactCompletion.defaultText I had used to prevent the manual from changing. Is that a mistake or voluntary ?
      defaultText = literalExample ''{
        type = "shellcommand";
        command = "'notmuch address --format=json --output=recipients  date:6M..'";
        regexp = ''\'''\''\[?{"name": "(?P<name>.*)", "address": "(?P<email>.+)", "name-addr": ".*"}[,\]]?''\'''\'''\';
        shellcommand_external_filtering = "False";
      }'';

It was intentional. When testing the module before merging I also checked whether this was a case where the documentation builder could detect and fix the issue automatically.  Turned out that it was was, so it was safe to remove the defaultText attribute 🙂
On February 20, 2019 3:06:39 AM GMT+01:00, Matthieu Coudron<EMAIL_ADDRESS>wrote:

Thanks I am really happy to see this merged.
It seems like you removed the contactCompletion.defaultText I had used
to prevent the manual from changing. Is that a mistake or voluntary ?
     defaultText = literalExample ''{
       type = "shellcommand";
command = "'notmuch address --format=json --output=recipients
date:6M..'";
regexp = ''\'''\''\[?{"name": "(?P<name>.*)", "address":
"(?P<email>.+)", "name-addr": ".*"}[,\]]?''\'''\'''\';
       shellcommand_external_filtering = "False";
     }'';

--
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/rycee/home-manager/pull/565#issuecomment-465389954

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Great because the regexp escaping was not very sexy !


EV batteries and Household batteries show rather different network impact in iNET scenarios

@ChaelKruip and I were preparing the iNET meeting on Tuesday evening and found some remarkable differences in how EVs and HH batteries affected the network peak loads at different grid levels. We had expected these batteries to have a similar effect, since they are both connected to the LV grid and EV-flex behaviour or availability is currently not coupled to their charging behaviour (@ChaelKruip has assured me).
Example: scenario: https://pro.energytransitionmodel.com/saved_scenarios/9131
Lots of surpluses and also 25% curtailment of all kinds of solar PV capacity.
I start by setting EV to 0% in Flex >> Storage. Have a look at grid impact on LV and LV/MV:

Then is show the impact of 20% EV (5.9 GW). This is the scenario's original setting. EV-batteries in 1st place in the flex merit order:

Compare this to the impact (almost) the same installed capacity of HH batteries has (1st place in flex merit order):

I struggle to explain this difference. @ChaelKruip had already noted that the batteries 'leakage' for EVs and HH batteries seem exactly a factor 10 different.
Unfortunately I do not have time to prepare this issue further, but perhaps this is enough to see what is going on, @michieldenhaan and @antw ?

More background:
We are trying to install a certain amount of LV batteries to lower the peak lead on that grid level, without causing lots of power to flow from higher grid levels to the LV-level batteries. This is undesirable. We don't want to overload the grid by storing offshore wind power in HH batteries, after all.

@ChaelKruip had already noted that the batteries 'leakage' for EVs and HH batteries seem exactly a factor 10 different.

Yes. But upon inspection of the node source analyses, the difference is not exact and both figures are based on different sources. That rules out a simple typo but still raises the question of these sources are reliable.

I think the answer is that somewhere there has been a misunderstanding about what reserved_fraction means.

ETEngine / Merit and the EV flexibility slider use reserved_fraction to change the size of the EV buffer available for flexibility purposes. Assuming electric vehicles have a 20kWh battery:

EV slider
Reserved for driving
Available for flex

0%
20kWh
0kWh

20%
16kWh
4kWh

50%
10kWh
10kWh

0%
0kWh
20kWh

The query which shows installed EV capacity in the flexibility order uses reserved_fraction to reduce the input capacity.

The ETEngine implementation is correct; at least based on what we originally planned to implement: (emphasis mine)

The 'availability'-slider which quantifies which fraction of EVs is connected to the grid on average. Both capacity and volume of the batteries are to be multiplied by this fraction.
The 'reserved fraction'-slider which quantifies which part of the volume is reserved for driving and cannot be discharged for load balancing. The volume of transport_car_using_electricity needs to be multiplied by 1.0 - reserved_fraction

We don't have an EV availability slider – only a reserved fraction slider (which is confusingly called transport_car_using_electricity_availability).
I created a local copy of the above scenario. With P2P batteries at 20% and EV at 13%, here are the input capacities:

Technology
Capacity in ETEngine and Merit
Capacity in flex order query

P2P batteries
5,466.352941176471
5,466.352941176471

Electric vehicles
29,522.6071
5,904.521419

The capacity of electric vehicles in Merit is controlled by the number of EVs in the scenario, not the EV flexibility slider.
The reason for EVs having significantly different results on the network is that they have much higher capacity, and there therefore much more "spikey".
To make matters worse, the query used to show EV flexibility in the LV demand chart is using the wrong node; it refers to the "driving demand" node rather than the flexibility node. Corrected, the difference between P2P and EV flexibility is quite obvious:

P2P @ 20%
EV @ 13%

bravo, @antw ! So i guess that means there was a bug and then some misunderstanding. I am still not sure I really understand the issue or even how EVs work fully. How do we report back to iNET about the way batteries work in the scenarios, @ChaelKruip ?

Thanks @antw!
So if I understand this correctly, the following is the case:

The available EV battery capacity for flex is always 100% of total EV capacity, regardless of whether the user chooses to use EV batteries for flex or not
By default, 100% of the available battery volume is 'reserved' for driving, so effectively EV batteries aren't used for flex unless the user changes the default.

The slider allows users to change the latter, the reserved fraction.
Example:
Suppose a scenario has 1 million cars. ETM default is 100 kWh per car with a charging capacity of 3.7 kW per hour.
This means that we have 3.7 GW installed battery capacity with 100 GW volume.
Suppose the user sets the EV flex slider to 10% -> The 3.7 GW batteries now have 10 GWh available for flex.
Suppose the user changes the slider to 50% -> The 3.7 GW batteries now have 50 GWh available for flex.
So the installed capacity stays the same, only the volume of the battery changes.
@antw Is this interpretation correct?

To make matters worse, the query used to show EV flexibility in the LV demand chart is using the wrong node; it refers to the "driving demand" node rather than the flexibility node. Corrected, the difference between P2P and EV flexibility is quite obvious

Are you sure the original query was wrong? As far as I can see the chart shows total electricity demand on the LV grid. So I would expect the 'electric vehicles' series to show "driving demand", not flex demand. Shouldn't EV flex rather be included in the "Flexibility" series (together with households P2P)?

So the installed capacity stays the same, only the volume of the battery changes.
@antw Is this interpretation correct?

Indeed.

Are you sure the original query was wrong? As far as I can see the chart shows total electricity demand on the LV grid.

You're absolutely right. I've reverted my change to this query and will add EV P2P to the flexibility series.

I'm ready to fix this. We have two options:

Use a single slider which sets both the available storage volume and the available input capacity for flexibility.
Use separate sliders, one which sets the available storage volume, and one which sets the input capacity.

I strongly favour (1) from a code point of view. It's simple to have a slider set the availability of the flex node, and have this affect both the volume and capacity simultaneously. It and means we can get rid of the reserved_fraction attribute, which was only used on this one node. (2) means having to treat input capacity and volume differently in code: input capacity affected by availability, but volume not affected.
I also think option 1 is clearer for users: with option 2, users may be confused why they moved one of the sliders and didn't see any P2P occurring.
With option 1, if the user sets the EV flexibility slider to 20% this means that 20% of the buffer, and 20% of the EV capacity will be available for flex:
Before (Passenger EV share = 100%)

Slider position
EV flex capacity in MO
Avail. for flex

20%
29,522 MW
159,581 MWh

50%
29,522 MW
398,954 MWh

100%
29,522 MW
797,908 MWh

After (Passenger EV share = 100%)

Slider position
EV flex capacity in MO
Avail. for flex

20%
5,904 MW
159,581 MWh

50%
14,761 MW
398,954 MWh

100%
29,522 MW
797,908 MWh

I strongly favour (1)

Me too! Also discussed with @michieldenhaan and he agrees. Let's go for it!


ModuleNotFoundError: No module named 'paystack.urls'

Hello, i have done everything in the ReadMe and still get this error, i am using a virtual environment and i have also swapped ' paystack ' to ' paystack.frameworks.django ' in the settings.py. I have also started a new project and used pipenv to install instead of pip in the documentation but i still get this error. Thanks in advance

Check this issue https://github.com/gbozee/pypaystack/issues/16


Bugfix for zero-longitude crossing trajectories leaving domain.

After 2 month of successful operational application, it seems safe to merge this branch to main.


CAF image background transparency fixes

@MikeWasson Please review. This PR is intended to fix PNG image transparencies that result in difficult to read graphics in dark mode. This update is limited to images under the /cloud-adoption folder.

OPS Build status updates of commit c15e5a8:
:clock10: Preparing: average preparing time is 46 sec(s)

OPS Build status updates of commit c15e5a8:
:clock10: Incremental building: average incremental building time is 23 sec(s)

OPS Build status updates of commit c15e5a8:
:warning: Validation status: warnings

File
Status
Preview URL
Details

toc.json
:warning:Warning

Details

bread/toc.json
:warning:Warning

Details

topics/high-performance-computing/TOC.json
:warning:Warning

Details

building-blocks/extending-templates/toc.json
:warning:Warning

Details

topics/high-performance-computing/breadcrumb/TOC.json
:warning:Warning

Details

topics/high-performance-computing/context/hpc-context.json
:warning:Warning

Details

docs/cloud-adoption/_images/adoption-phases.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/discovery-guide-sdn.png
:white_check_mark:Succeeded

docs/cloud-adoption/_images/formula-gain-from-investment.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/formula-roi.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/calculator-enterprise.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/calculator-small-to-medium-enterprise.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/cost-management.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/deployment-acceleration.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/governance-evolution-large.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/governance-evolution.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/governance-mvp-implementation-flow.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/governance-mvp.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/identity-baseline.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/incremental-governance-example.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/large-enterprise-resource-organization.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/mid-market-resource-organization.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/resource-consistency.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/governance/security-baseline.png
:white_check_mark:Succeeded
View

docs/cloud-adoption/_images/hybrid-consistency.png
:white_check_mark:Succeeded
View

toc.json

[Warning] Error happen when converting toc.json to Pdf. Details: Could not find file 'T:\ww3j\toc.json'.

bread/toc.json

[Warning] Error happen when converting bread/toc.json to Pdf. Details: Could not find a part of the path 'T:\ww3j\bread\toc.json'.

topics/high-performance-computing/TOC.json

[Warning] Error happen when converting topics/high-performance-computing/TOC.json to Pdf. Details: Could not find a part of the path 'T:\ww3j\topics\high-performance-computing\TOC.json'.

building-blocks/extending-templates/toc.json

[Warning] Error happen when converting building-blocks/extending-templates/toc.json to Pdf. Details: Could not find a part of the path 'T:\ww3j\building-blocks\extending-templates\toc.json'.

topics/high-performance-computing/breadcrumb/TOC.json

[Warning] Error happen when converting topics/high-performance-computing/breadcrumb/TOC.json to Pdf. Details: Could not find a part of the path 'T:\ww3j\topics\high-performance-computing\breadcrumb\TOC.json'.

topics/high-performance-computing/context/hpc-context.json

[Warning] Error happen when converting topics/high-performance-computing/context/hpc-context.json to Pdf. Details: Could not find a part of the path 'T:\ww3j\topics\high-performance-computing\context\hpc-context.json'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat: add prototype package, clone feature

feat: add prototype package, clone feature

feat: add prototype package, clone feature


[CCLEX-166]: Changed timestamp format in 'Synced at' and 'Created at' fields

PR Checklist
Check if done, remove if not applicable:

[x] New feature or bug fix is mentioned in the CHANGELOG.

👀


K8s load tests fail when there is lock in the execution flow

Description:
During a load test in Kubernetes, an error rate was observed for higher concurrent users (20 and above) when there is a single or multiple lock syntax in the execution flow. After a few minutes (2-3 minutes) the tests are started, there is an increasing error rate, and Read timed out logs get printed in the JTL output.
Here are the summary test specifications and the observations:
Test duration: 15 mins
Payload size: 50 bytes
Container specifications (c2c properties):
[cloud.deployment]
min_memory="256Mi"
max_memory="512Mi"
min_cpu="200m"
max_cpu="1000m"

Concurrent users
Errors

5
NO

10
NO

20
YES

30
YES

Steps to reproduce:

Clone and build the sample project.
https://github.com/ldclakmal/ballerina-samples/tree/master/load-tests/locks
Build and push the Docker image.docker build -t ldclakmal/ballerina-locks:latest target/docker/ballerina_locks
docker push ldclakmal/ballerina-locks:latest

Deploy it in k8s.kubectl apply -f ./target/kubernetes/ballerina_locks
kubectl expose deployment ballerina-locks-deployment --type=NodePort --name=ballerina-locks-svc-local

Check the NodePort of the service.kubectl get svc

Run the load test (assign the NodePort for the -Jport argument).jmeter -n -t ./resources/http-post-request.jmx -l original.jtl -Jusers=30 -Jduration=600 -Jhost=localhost -Jprotocol=http -Jpath=foo -Jresponse_size=50 -Jpayload="./resources/50B.json" -Jport=<NodePort>

The JMeter output summary log will keep running for a few minutes (2-3 minutes) and then it can be seen with an increasing error rate. Refer to the following output samples.
JMeter Log
Creating summariser <summary>
Created the tree successfully using http-post-request.jmx
Starting standalone test @ Tue Nov 23 11:08:58 IST 2021<PHONE_NUMBER>606)
Waiting for possible Shutdown/StopTestNow/HeapDump/ThreadDump message on port 4445
Warning: Nashorn engine is planned to be removed from a future JDK release
summary +      1 in 00:00:05 =    0.2/s Avg:  5237 Min:  5237 Max:  5237 Err:     0 (0.00%) Active: 3 Started: 3 Finished: 0
summary +    764 in 00:02:27 =    5.2/s Avg:  2795 Min:     8 Max: 120029 Err:    17 (2.23%) Active: 30 Started: 30 Finished: 0
summary =    765 in 00:02:32 =    5.0/s Avg:  2798 Min:     8 Max: 120029 Err:    17 (2.22%)
summary +     30 in 00:02:00 =    0.2/s Avg: 120010 Min: 120004 Max: 120020 Err:    30 (100.00%) Active: 30 Started: 30 Finished: 0
summary =    795 in 00:04:32 =    2.9/s Avg:  7221 Min:     8 Max: 120029 Err:    47 (5.91%)

JTL Output
The following logs can be seen in the .jtl output file:
timeStamp,elapsed,label,responseCode,responseMessage,threadName,dataType,success,failureMessage,bytes,sentBytes,grpThreads,allThreads,URL,Latency,IdleTime,Connect
1637324448823,2496,HTTP Request,200,OK,Users 1-3,text,true,,179,938,4,4,http://localhost:31123/foo,2486,0,1
1637324445002,6317,HTTP Request,200,OK,Users 1-1,text,true,,179,938,4,4,http://localhost:31123/foo,6308,0,25
...
...
1637324557882,103,HTTP Request,200,OK,Users 1-11,text,true,,179,938,30,30,http://localhost:31123/foo,103,0,0
1637324557809,120018,HTTP Request,Non HTTP response code: java.net.SocketTimeoutException,Non HTTP response message: Read timed out,Users 1-14,text,false,Test failed: text expected to contain /50/,2620,0,30,30,http://localhost:31123/foo,0,0,0
1637324557809,120018,HTTP Request,Non HTTP response code: java.net.SocketTimeoutException,Non HTTP response message: Read timed out,Users 1-24,text,false,Test failed: text expected to contain /50/,2620,0,30,30,http://localhost:31123/foo,0,0,0
1637324557802,120025,HTTP Request,Non HTTP response code: java.net.SocketTimeoutException,Non HTTP response message: Read timed out,Users 1-4,text,false,Test failed: text expected to contain /50/,2620,0,30,30,http://localhost:31123/foo,0,0,0
1637324557809,120018,HTTP Request,Non HTTP response code: java.net.SocketTimeoutException,Non HTTP response message: Read timed out,Users 1-6,text,false,Test failed: text expected to contain /50/,2620,0,30,30,http://localhost:31123/foo,0,0,0
1637324557807,120020,HTTP Request,Non HTTP response code: java.net.SocketTimeoutException,Non HTTP response message: Read timed out,Users 1-9,text,false,Test failed: text expected to contain /50/,2620,0,30,30,http://localhost:31123/foo,0,0,0
...
...
...

Sample output logs can be found: https://github.com/ldclakmal/ballerina-samples/tree/master/load-tests/locks/resources/logs
Tested Version:
Ballerina SwanLake Beta4-RC3
OS, DB, other environment details and versions:
MacOS Monterey
Version 12.0.1
MacBook Air (M1, 2020)

IS this only happen in K8s. I mean can we reproduce the same in the local machine.

@ldclakmal Is this only happen in K8s. I mean can we reproduce the same in the local machine.

I didn't try to reproduce in local machine. But I hope this can be reproduced for a higher number of concurrency users depending on the specifications of the machine.
On a separate note, this behaviour was there for sometime after the Swan Lake lock implementation AFAIR. Because, we observed the same behaviour in AWS VM performance tests as we were using locks for ballerina/cache implementation.

This is the load test results done in my local linux machine.
environment :
Ubuntu 20.04.3 LTS
openjdk 11.0.8 2020-07-14
minikube version  v1.21.0
kubectl version
Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.2", GitCommit:"092fbfbf53427de67cac1e9fa54aaa09a28371d7", GitTreeState:"clean", BuildDate:"2021-06-16T12:59:11Z", GoVersion:"go1.16.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.7", GitCommit:"132a687512d7fb058d0f5890f07d4121b3f0a2e2", GitTreeState:"clean", BuildDate:"2021-05-12T12:32:49Z", GoVersion:"go1.15.12", Compiler:"gc", Platform:"linux/amd64"}

properties :
duration - 900s
protocol - http
response-size - 50 bytes
json payload - {"size":"50B","payload":["0123456789","ABCDEFGH"]} (As in the file /resources/50B.json)
Results (error rate) :

Concurrent users
beta 4
beta 5
beta 6

5
0%
0%
0%

10
0%
0%
0%

20
0%
0%
0%

30
0%
0%
0%

40
0%
0%
0%

50
0%
0%
0%

Could this be specific to MacOS? @ldclakmal

Did a test on Azure K8S cluster and could not reproduce the issue. This can be related to MacOS, will check and confirm again. Hence closing the issue.