Add Api::ProjectsController

The Api::ProjectsController endpoints behavior is very similar to
Api::TomatoesController endpoints behavior.
Actions:

GET #index
GET #show
POST #create
PATCH/PUT #update
DELETE #destroy

Get the current user's list of projects:
GET /api/projects?token=123
GET /api/projects?token=123&page=2

Get a current user's project:
GET /api/projects/123?token=123

Create a new project:
POST /api/projects

{
  "token": "123",
  "project": {
    "name": "Web app",
    "tag_list": "one, two",
    "money_budget": 1200,
    "time_budget": 120
  }
}

name is a required attribute.
money_budget is an integer value that represents the amount
of money allocated for the project expressed in user's default
currency
time_budget is an integer value that represents the amount of
time estimated for the project expressed in hours

Update an existing project:
PATCH /api/tomatoes/123

{
  "token": "123",
  "project": {
    "name": "Web app updated",
    "tag_list": "one, two"
  }
}

Delete an existing project:
DELETE /api/projects/123?token=123

The #index action returns a paginated list of projects, it includes
pagination information such the current page, the total number of
pages, and the total number of resources:
{
  "projects": [
    {
      "id": "57f9c9377c8402dd306d1c8b",
      "name": "Web app",
      "created_at": "2016-10-09T04:36:07.787Z",
      "updated_at": "2016-10-09T04:36:07.787Z",
      "tags": [
        "one",
        "two"
      ],
      "money_budget": 1200,
      "time_budget": 120
    }
  ],
  "pagination": {
    "current_page": 1,
    "total_pages": 1,
    "total_count": 1
  }
}

The JSON representation of a project resource is:
{
"id": "57f9c9377c8402dd306d1c8b",
"name": "Web app",
"created_at": "2016-10-09T04:36:07.787Z",
"updated_at": "2016-10-09T04:36:07.787Z",
"tags": [
"one",
"two"
],
"money_budget": 1200,
"time_budget": 120
}

A request to `POST #create` creates a new project associated to the
current user. It returns `201 Created` and a JSON representation of
the resource created in case of success. It returns `422 Unprocessable
Entity` and a JSON representation of the validation errors in case of
failure.

Example response body (failure):

```json
{
  "name": [
    "can't be blank"
  ]
}

A request to PATCH #update updates an existing project associated to
the current user. It returns 200 Ok and a JSON representation of the
resource created in case of success. It returns 422 Unprocessable Entity and a JSON representation of the validation errors in case of
failure. See POST #create to see a failed operation response example.
A request to DELETE #destroy removes an existing tomato associated to
the current user. It returns 204 No Content in case of success.

You are including the token on example responses (sometime): I think this is a copy/paste issue, please remove it.


Allow using Rust arrays as GraphQL lists (#918)

Fixes #918
Allows usage of Rust arrays as GraphQL lists.

@LegNeato the main challenge imposed by this PR is that for input values Rust array validation is stricter than for vectors and slices, because we have to ensure that input GraphQL list length matches the one of the declared Rust array.
Unfortunately, current juniper architecture keeps input validation and user defined types far appart. Everything we declare implmenting GraphQLType, GraphQLValue (and others) comes into play after the input query was validated, and the current validation mechanism has no knowledge about expected list sizes, which results in panicking here and there when Rust arays are used as inputs.
Idealy, to overcome this, it would be nice to have an ability to participate in validation for custom defined types (like visitor pattern, for example), but that goes far beyond theme of this PR and requires quite a massive refactoring of ast/validation layer.
For now, I've just tweaked the ast/meta/validation layers with an optional expected_size information for lists. Which is None for vectors and slices, but drives in for arrays.
Additionally, gradual Rust array initialization requires some additional unsafe squats in the current stable Rust. I've tried my best to provide enough // SAFETY: comments to explain what is happening and why.

I had a change for this but looks like you beat me to the punch! I'll take a look soon, but at quick glance I didn't see Input Coercion , but perhaps I missed it:
https://spec.graphql.org/June2018/#sec-Type-System.List

@LegNeato additionally, I've reimplented unsafe squats a bit in FromInputValue implementation. The previous one was not panic safe.

If this change could be released, that'd be great :)


[SPARK-35887][BUILD] Find and set JAVA_HOME from javac location

What changes were proposed in this pull request?
This PR aims to find and set JAVA_HOME from the javac location.
Why are the changes needed?
Since SPARK-35850, maven compile fails with Java8 when there is no JAVA_HOME and java path is JRE java.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manually run mvn in vanilla ubuntu.

cc: @HyukjinKwon @dongjoon-hyun could you review this please?

Test build #140298 has started for PR 33075 at commit 7e09fb1.

Since it's verified with build/mvn, I'll merge this to recover the effected downstream, @HyukjinKwon .
Merged to master.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44830/

Thx!

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44830/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44830/

Test build #140298 has finished for PR 33075 at commit 7e09fb1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140298/


[Feature]: Spring authorization server access to Github third-party login

Function description and necessity description
Provide third-party access convenience
Suggestions and solutions
No response

complete


Fedora 26 - Unable to start rai_wallet

Description of bug:
I have downloaded the latest version rai-9.0.0-Linux.
I'm getting the following error:
Illegal instruction (core dumped)

Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
Environment:
Kernel
Linux fedorahost 4.14.11-200.fc26.x86_64
Node version
v6.12.0

logs

Wtf? Issues referencing goes in a loop with no answer... check issues #365 #316 #398 and it is almost not the same issue, as the node here doesn't even start to crash: it says Illegal instruction (core dumped)


New Name for LabVIEW-CLI?

There is now a lot of confusion with NI LabVIEW-CLI. I think perhaps we should consider a new name. There were a few ideas on twitter - please comment below with other ideas.
They were:
"Command Line Interface for LabVIEW" or "CIFL"
"LabVIEW Automation Interface"

Other obvious would be G-CLI.
I should say, at this point we should probably still use labview-cli as the command for compatibility but think about a longer term migration, probably supporting both

I still like CIFL (Command Line Interface for LabVIEW) best. But maybe, it doesn't convey the fact that it's not the LabVIEW native one but a separate product.
How about "Open-Source Command Line Interface for LabVIEW" or "osCIFL"?

I think I'm going to go with G-CLI. 'G' has tended to be synonymous with independent LabVIEW ("OpenG", "GDevCon") so it seems appropriate (and keeps the command short!). Next steps:

Update the launch and add a redirect for the old command.
Update source project names.
Update VIPM Name
Prepare for tools network.
Resubmit to tools network under new name

Changed the code. New command which is g-cli with a batch file that means that labview-cli command still works. Only likely break will be that the VIPM file wont recognise the change of name as the same package.


Fix duplicated deploy explanation

And added a feature branch explanation on how to remove the merged old branch.
The deployment command was duplicated and could lead to missunderstanding (IMHO), so I'm keeping only one of the two same commands and explaining how to delete the old branch optionally! 😸

Thanks @garciasdos for this!


Could you share your monobehavior component that is showing this issue?

Incidentally, multi-edit is a very experimental feature and you might want to disable it if it is giving you trouble.
This is a very important feature, but I don't see it receiving the love it needs anytime soon.
If you want to disable it, just put this code in a cs file somewhere:
using FullInspector;
using FullInspector.Internal;

public sealed class FullInspectorSettingsProcessor: fiSettingsProcessor {

    public void Process() {
        fiSettings.EnableMultiEdit = false;
    }

}

The monobehavior is nothing special, just a plain old color field. The multi-select must happen as prefabs in the project view though. Let's me try to make a project that recreates the bug


Including IP and TCP source and dest address and port info in filename

Thank you for the stream splitter. I added some IP info to the filename to make it useful for my use cases, sending you a pull request in case you want to include it.

Thank you, it's useful for cases other than automatic analyze :)


Testing/5.7.0

Sorry for the repeat PRs, I realized I forgot to update a few occurrences of the old version number

To fix the issue on Linux, you have to add the following lines to the test_package's CMakeLists.txt:

Just after the add_executable(helloworld main.cpp) line:

target_compile_features(helloworld PRIVATE cxx_nullptr)

Just after the add_executable(helloworld2 main.cpp) line:

target_compile_features(helloworld2 PRIVATE cxx_nullptr)

It will activate the support for c++11.

For the Windows build, it's an issue I didn't have the time to investigate: git needs the sh.exe executable to exist in order to run the submodule command. However, cmake refuses to work if it finds sh.exe in PATH. I made a very ugly fix (in 5.6.2 branch) that should help for the moment.

Thank you for the feedback, I made those changes.

I merged your PR. Don' t pay attention to the "Revert": I made a mistake when merging and I had to do manual changes.
Thanks for your help.


fix: Corrected example use

I have corrected the example use in the Readme. It now uses your class SpreadsheetManager to create a new managedSpreadsheet variable.

Thanks


Broken setup.py

The setup.py scripts try to read the old readme file.
It's fixed in PR #22.

Thanks for the report and the fix.


added KMeans to models.py

I'm a fan of this!  Just for fun I added in a K-means implementation.  Its not as concise as yours, but does the trick.  Congrats on a cool project.

That's great! Thanks.

I did some extra trimming and added an example. Thanks again!


The configuration cache entry should be discarded on build logic validation issues

Expected Behavior
The configuration cache discards the configuration cache entry on build logic validation issues, not creating a potentially incorrect entry.
This would require to be able to distinguish a build failure because of build logic validation vs. an actual build failure such as test failure.
 The next build will reuse the configuration cache entry, possibly using wrong task inputs.
Context
This issue was extracted out of https://github.com/gradle/gradle/issues/16485
Steps to Reproduce
See https://github.com/gradle/gradle/issues/16485

@eskatos, could you clarify what you mean by build logic validation issues?

@bamboo, IIRC, it was about failures in task properties validation (e.g. ValidatingValueCollector) where the first run with CC storing was failing but running right after with CC loading would succeed because we stored a state, replacing a property with its serialized form that somehow passes validation.
See https://github.com/gradle/gradle/issues/16485 from where this issue was extracted for a discussion around this and a reproducer.


How to activate office 2019 using kms

Hi Team,
my Situation: I am able to download the office 2019 and set up using below commands
setup.exe /download configfile.xml
setup.exe /setup configfile.xml  (Till here I am successful)
now
I would like to activate office 2019 using kms  (kms server was set up and no issue with that), whenever we create a new vm in my environment (scvmm 2016).
Right now we are using office 2016 (in the office 2016 folder, we have ospp.vbs, so we use cscript pathof ospp /act command) to activate.
problem: I am unable to find the ospp.vbs script in 2019 folder. Where I can get it?
is it right process for office 2019? If not please help me on this.
Thanks

@sam1228 -- Try c:\program files\Microsoft Office\office16. That's where the ospp.vbs script file is in my installation of Office Professional Plus 2019. Hope that helps.

@DHB-MSFT: Thanks for your reply.
So we don't get office19 folder when we download office using ODT. Right?

@sam1228 -- Volume licensed versions of Office 2019 (Professional Plus and Standard) now use Click-to-Run instead of Windows Installer (MSI) for the installation/update technology so installation sources files (which you get when you do a /download) and the file folder structure when you install Office (when you use /configure) are different than before.

@DHB-MSFT: Thanks for sharing the info.
I read about that one. I was downloaded office 2019 in that way and installed successfully. Here I can enter my subscription credentials and make vm template, but the team will create new VM use my credentials.  I don't want to use my credentials for the team.
Now I want to activate using kms.
as per our conversation, I understood, I have to go to an older office16 folder and try to activate it(It will activate office using kms key. ( as per the key belongs  (2016 or 2019)). right?

any suggestions

@sam1228 . yes it will activate


Updated vision api url

The url in the existing version returns a 404 error.

This fixed my app, as well. Thanks!

/csharp/ResizeImage/run-module5.csx also needs to be changed


Updated readme with new logo

Updated the logo with the work performed by Sébastien Bakirci

Changed logo

Changed logo


problem with parsing large HTML files

Hi.
I'm trying to parse this page.
But unfortunately I can't get all items from page.
My simple test code is as following
require 'open-uri'
require 'nokogiri'

html = Nokogiri::HTML open('http://www.pro-medic.ru/index.php?ht=246&perpage=all')
p html.css('ul.products-grid-compact li .goods_container').count

It returns only 83 items as long as real count is about 186.
I've thought that problem can be in open function, but it seems that function read all html page correctly.
Can someone help me?
Thanks.

Hello!
Thanks for asking this question! Your request for assistance using
Nokogiri will not go unanswered!
However, Nokogiri's Github Issues is reserved for reporting bugs or
submitting patches. If you ask your question on the mailing list, Team
Nokogiri promises someone will provide you with an answer in a timely
manner.
If you'd like to read up on Team Nokogiri's rationale for this policy,
please go to http://bit.ly/nokohelp.
Thank you so much for understanding! And thank you for using Nokogiri.


Performance issue with maximumLineLength

It seems that the maximumLineLength rule is making the CLI interface take 7.7x as long to run on a file, and the Node interface 33x times as long to run.
The configuration used was a copy of the current airbnb.json. The maximumLineLength rule was commented out using /* */ (no difference was found for simply removing it).
Tested with<EMAIL_ADDRESS>running through Node.js v5.11.0 on the CLI and through Atom v1.8.0-beta3 for the "Node API" (equivalent to Node v5.1.1).
The file used to test this can be found in here: linter-jscs_GH241-2.zip
The raw data can be found here: https://docs.google.com/spreadsheets/d/1HGXcBMeRCgOEvAuvgNZEgtZUdEAsPwm24BUYtS_x_cI/edit?usp=sharing
This was originally filed as part of https://github.com/AtomLinter/linter-jscs/issues/241.
If you need any additional information please let me know.

Replaced with https://github.com/jscs-dev/node-jscs/issues/2246

You referenced this issue as the one replacing it... not sure what you meant there :stuck_out_tongue:.

Ah, probably https://github.com/cst/cst/issues/113, as you referenced this from there.

Oh yeah, sorry, indeed i meant cst/cst#113

Oh yeah, sorry, indeed i meant cst/cst#113

I don't understand how this issue with too many long lines being reported has anything to do with memory consumption. Setting the maximumLineLength to 200 (in my case) made the problem disappear.
The lines verified by JSCS are not very long, so I am surprised that you classified this as a memory issue. Do you need additional source files which exhibit the problem?

Perhaps things get more clear if you would read issue description?


After upgrade to 1.6.0, set font-family 'URW Gothic L' fail

VSCode Version: Code 1.6.0 (e52fb0bc87e6f5c8f144e172639891d8d8c9aa55, 2016-10-10T18:30:01.819Z)
OS Version: Linux x64 4.4.0-42-generic
Extensions: DotJoshJohnson.xml-1.6.0, HookyQR.beautify-0.4.0, NoHomey.theme-realgithub-1.0.4, Nuuf.theme-hackershaze-1.2.1, PKief.material-icon-theme-1.1.1, RoscoP.ActiveFileInStatusBar-1.0.2, abusaidm.html-snippets-0.0.16, be5invis.vscode-icontheme-nomo-dark-1.1.4, bernardodsanderson.theme-material-neutral-0.6.0, chenxsan.vscode-standardjs-1.0.1, donjayamanne.python-0.4.0, eg2.tslint-0.5.39, eg2.vscode-npm-script-0.0.20, gerane.Theme-Blackboard-0.0.2, karyfoundation.theme-karyfoundation-themes-5.2.0, liuji-jim.vue-0.1.3, mohsen1.prettify-json-0.0.3, ms-vscode.Theme-1337-0.1.2, ms-vscode.js-atom-grammar-0.1.9, msjsdiag.debugger-for-chrome-1.1.0, samverschueren.yo-0.8.5, shinnn.stylelint-0.17.0, stevencl.addDocComments-0.0.8, tht13.python-0.2.3, vsmobile.cordova-tools-1.2.3, xabikos.JavaScriptSnippets-1.2.0

Steps to Reproduce:

@Toybreak What is your exact "editor.fontFamily" setting. Do other fonts work for you?

@alexandrudima Hi,  this is my setting

"editor.fontFamily": "'URW Gothic L','Microsoft Yahei'"

used to work well, till this upgrade. Other fonts work well

We have updated Electron and with it came a newer version of Chromium. I do not know what Chromium changed w.r.t. font family names and their resolving, anything they would have changed would impact any website.
@Tyriar have you heard anything regarding this (on linux) ?
@Toybreak Can you please try if any of the browsers you have pick up the correct font when you go to http://jsbin.com/yaxehunoco/edit?html,output

Haven't seen anything like this personally, I'd guess it's related to the font string being incorrect or being installed incorrectly?

hi @alexandrudima i've tried in chrome and chromium, both not working. i think i installed it correctly

it works in firefox.

@Toybreak Thank you for following up on this.
In this case, may I please ask you to search for an existing issue / create a new issue against Chromium and link to it in the comments -- https://bugs.chromium.org/p/chromium/issues/list
If Firefox can find your installed font, then IMHO Chromium should be able to find it too.

sorry, recently i can not recreate this bug. so i will close this issue


Feature request- Support additional private endpoint for aks apiserver

Is there an existing issue for this?

[X] I have searched the existing issues

Community Note

Please vote on this issue by adding a :thumbsup: reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
in AKS private cluster scenario, it will be useful if we could create additional private endpoint for aks api endpoint. this removes the necessity of vnet peering between aks and devops agent vnet. Setup is described in MS documentation.
https://learn.microsoft.com/en-us/azure/aks/private-clusters#use-a-private-endpoint-connection
New or Affected Resource(s
azurerm_private_endpoint
Potential Configuration file
# new private_endpoints block in aks

   private_endpoints = {
      pe1 = {
        name               = "aks-pe"
        #lz_key             = "" # for vnets created in different lz
        vnet_key           = "spoke_aks_re1"
        subnet_key         = "private_endpoints"
        private_service_connection = {
          name                 = "aks-psc"
          is_manual_connection = false
          subresource_names    = ["management"]
        }
      }
    }

### References

_No response_

closing as merged


Add AllNoFailMethod

AllNoFail (except will fail if everything fails :) )  Not sure I love the method name....let me know

I had to add some JavaDoc to get the build to run...

Try not to format the license blocks, makes it look like the wording has changed.

I would also like to see a simple test case added for this


Further Code Clean Up

Added ToDo's
Fixed moons-String.
Optimized Code further (C#7 conventions)
Changed some code to LINQ

I made several commits, to integrate your advises
I look how I can merge them with this pull request.

Hey @StollD , is that okay now? :)

I also added an Info about the code conventions, you want in this project. :)

Looks ok to me. Merging.
Thanks for your contribution!

Perfect :)
And I'm glad, that you are happy about my contribution :D


Hamburgermenu Styles not included with MahApps?

Are the "Creators Style" and "Ripple Effect" styles not included as styles in the Mahapps.Metro package ?
I feel like they could be included as optional styles - ready to use of of the box.

@Swiftpaws "Creators Style" maybe, "Ripple Style" not, because it depends on MDIX library.

@Swiftpaws I added the Creators Update styles to the themes

MahApps.Styles.ListBoxItem.HamburgerMenuItem.CreatorsUpdate
MahApps.Styles.HamburgerMenu.CreatorsUpdate


Mention IRB console in readme

I also took the liberty to format markdown tables in the readme (through my editor extension 😛 ), which will make them easier to read in the raw view.
For example:

Thank you. Could you consider to add about that remote debugging doesn't support it?


incomprehensible answers from Gemma:7b

What is the issue?

Mistral:7b works fine, so I suppose that the issue is realted to the model
OS
macOS
GPU
Apple
CPU
Apple
Ollama version
0.1.32

This should work fine. Here's my output on a MBP:
% ./ollama run gemma:7b
>>> hi there
Hi there, and welcome to my chat! 👋

I'm glad you decided to talk to me today. What would you like to talk about?

Two things:

how much memory do you have, and is there memory pressure?
can you paste the output for ollama ls | grep gemma:7b?

The latter should look like:
% ./ollama ls | grep gemma:7b
gemma:7b                                430ed3535049    5.2 GB  3 weeks ago

The latter should look like:
% ./ollama ls | grep gemma:7b
gemma:7b                                430ed3535049    5.2 GB  3 weeks ago

Running Ollama 0.1.32, Mac OS Sonoma with 16 GB, can vouch that the latest version - a72c7f4d0a15, works without any modifications.
ollama ls | grep gemma:7b
gemma:7b                             	a72c7f4d0a15	5.0 GB	7 days ago

ollama run gemma:7b

>>> hi there
Hi there! 👋

It's great to hear from you. What would you like to chat about today? 😊

@pdevine

how much memory do you have, and is there memory pressure?
can you paste the output for ollama ls | grep gemma:7b?

Mac Mini M2 8GB. apparently it should be able to run since mistral 7b can
gemma:7b  	a72c7f4d0a15	5.0 GB	35 seconds ago

I realize I had the outdated version. :-D I think it's almost certainly a memory pressure issue w/ metal cc @mxyng

@pdevine but can it be the problem here? [...]
it doesn't refuse to generate a reply or takes long to generate, but it just returns a bunch of "unused" tags, as if it's something that needs to be decoded. also I might have misunderstood your response

I think maybe? It tends to do weird things when it runs out of memory. It's weird that mistral:7b works fine though.

@mkmohangb the team had been talking about this earlier today and we suspected that this might be the issue. Good catch.
There is a fix in the 0.1.33 pre-release which may fix this. @kukidevs (or anyone commenting on the issue) would you be able to try it out?

@pdevine I tried Gemma with 0.1.33-rc5 version. It works now but is slow. I see in the server logs that not all the layers are sent to the GPU. How do you decide the upper limit for this?

@pdevine I tried Gemma with 0.1.33-rc5 version. It works now but is slow. I see in the server logs that not all the layers are sent to the GPU. How do you decide the upper limit for this?

We compute the graph of the memory footprint for the model and see if you have enough memory.
I'll go ahead and close the issue. The workaround for now is to /set num_gpu to a lower number, but otherwise just upgrade to 0.1.33.


adding a new page to display contributors✨

I want to add a page to showcase all the contributors of this website.
@ANSHIKA-26 can you assign me this issue.

Pls assign this task to me

it already exists in readme and it doesnt make sense to add one to the project , sorry


argocd-apps: use api chart v0.32.0

use the fixed api chart in argocd https://phabricator.wikimedia.org/T375268

hm I don't think I understand, the chart version changes here only because we needed to update the api chart with the recent redis fix, usually something like this wont be required (for new image releases)


Added Bluemix support and simplified application.

This PR adds all the necessary files to push the application to Bluemix. It also makes significant changes to main.swift by removing all the different code examples that can be put in docs and focusing only on serving static content, GET requests, and starting the server.

I think we should split this into two PRs.  One for the Bluemix changes, and another for the simplification.
Personally, while the Bluemix support is important (and so is Heroku), I'm not sure we should be removing functionality from this application.  We should consider refactoring it so that visiting the default route provides Kitura education etc. and gives links to other routes that demo specific functionality.

@ianpartridge Here’s the context around this pull request. As part of our deliverables for the next month, the Kitura-Sample should be updated so it can be pushed/deployed to Bluemix using a CF buildpack. In addition to this, during the SCRUM earlier today, the team discussed the idea of simplifying the Kitura-Sample so that it showcases the main aspects of the Kitura framework as opposed to attempting to showcase every single aspect that Kitura supports (e.g. templating). We are thinking that separate examples could be used to showcase templating, redis, cloudant, etc. instead of having all of them bundled up under one single app. Kyle volunteered to help us with this and submitted this PR after making these changes. The templating sample code was removed in this PR. We have a separate repo that showcases how to use stencil: https://github.com/IBM-Swift/Kitura-SampleStencil.

It should also be noted that the templating was breaking breaking the Bluemix routes. Which is another reason I chose to remove it when adding Bluemix support.
Granted, I removed a lot more than just templating with this PR. I stripped the application down to showing essentially the bare minimum for a Kitura application that serves static content and has a basic REST interface.
I could make a separate PR that adds Bluemix support and only removes templating while keeping the rest of the examples in main.swift, but I feel that this overcomplicates the sample for a starting Kitura developer and that those examples should be shown in the Kitura docs.
Would you still prefer I do that? Then do the rest of the simplification in another PR.

That commit adds back in most of the examples. Notable examples that are still missing:

Basic Auth Middleware
Multiple Handler Blocks
Custom "Not Found"
Handling errors that get set

And of course:

Templating

the Kitura-Sample should be updated so it can be pushed/deployed to Bluemix using a CF buildpack.

Totally agree - this is really important.

In addition to this, during the SCRUM earlier today, the team discussed the idea of simplifying the Kitura-Sample so that it showcases the main aspects of the Kitura framework as opposed to attempting to showcase every single aspect that Kitura supports (e.g. templating). We are thinking that separate examples could be used to showcase templating, redis, cloudant, etc. instead of having all of them bundled up under one single app.

I'm not so sure about this.  The Kitura-Sample app is already quite simple.  Rather than have lots more repos (making it harder to find information) I would prefer to structure the Kitura-Sample app so that the different pieces of functionality are easy to find.
For example, we could have separate routes to showcase templating, redis, cloudant etc. The code could be split into separate files.  The main file/route would link off to different sub-topics.

This PR has been moved to a new project: https://github.com/IBM-Swift/Kitura-Starter-Bluemix.


feat(new construct): aws-fargate-kinesisstreams

Issue #, if available:
README and architecture diagram for the new aws-fargate-kinesisstreams solutions construct.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: ae3b069af9ba23dd0aa630bade332445582209bd
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: githubautobuild-for-cdk-v2
Commit ID: e13b2d454b3f3a6bd3dc6ae9be02414411b0b25d
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: githubautobuild-for-cdk-v2
Commit ID: b7ff22c8f014737b668d97f8efbd3dc3c65b990a
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: githubautobuild-for-cdk-v2
Commit ID: b382007ad2604c96db414d360ded472688b7568a
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: githubautobuild-for-cdk-v2
Commit ID: ff3b846711c40b6ef97d155bba22294ca758af4b
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: githubautobuild-for-cdk-v2
Commit ID: 4e5d05d73f2e0c0fdaf74fa65930d7194860850e
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: b8067fc9110dc7de065d50e8a8d170fe20db3afe
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 0e5a01d0806220cd438f335e8973460031362975
Result: SUCCEEDED
Build Logs (available for 30 days)

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


Question: Equivallent of Response_End in Global.aspx.cs

Hi,
What is equivalent of Response_End (available in Global.aspx.cs in earlier versions of Asp.NET MVC)?
MVC app with Asp.NET 5 is not event driven. In this case what should I implement to trigger post response processing?
Any help would be appreciated.

@Tratcher what do you recommend here?

What kind of work do you want to do?
Have you tried HttpContext.Response.OnCompleted?

Or would simply implementing custom middleware work for your scenario?

How do I implement custom middleware? Is there any guideline or demo code available? Thanks.
Basically I have unit of work pattern and single repository implemented. I would like to save context during response end. I would like to do it globally so that I can also handle errors and other scenarios in a single place. I have done this in Global.aspx.cs earlier. I would like to port the same application on newer framework.

The middleware docs are under review here: https://github.com/aspnet/Docs/pull/348/files

If you're specifically using MVC 6, you can use an ActionFilterAttribute and override OnActionExecuted to do cleanup work. There are other types of filters in MVC that might be more suited to your scenario, depending on when you want to do the cleanup work.
Writing middleware is also very easy. Here's a blog post I found that shows how to do it: http://www.mikesdotnetting.com/article/269/asp-net-5-middleware-or-where-has-my-httpmodule-gone
Your middleware can be a simple class that's just a few lines of code, and then you register it in your Startup class's Configure method.


App says "view file" instead of "view channel" when you type a channel name with the full lbry:// prefix

This should be a simple one so I'm not doing the whole form thing.
When you type @MH (say) in the top bar, it says "view channel" next to the suggestion that would just take you to that location, because it's a channel. But when you type it as lbry://@MH, it rightly doesn't offer to search, but says says "view file". It should say "view channel".

Nice catch. Confirmed this is a bug.


Update Protobuf to v21.12

Upgrading Protobuf to match the onnx community. In order to upgrade Protobuf we have to include the abseil dependencies as well or else we will receive failures. I am hoping by upgrading Protobuf it will help us with ONNX 1.15.0 upgrade.

Jenkins Linux ppc64le Build #12469 [push] Update Protobuf to v21.1... started at 13:24

Jenkins Linux s390x Build #13475 [push] Update Protobuf to v21.1... started at 13:15

Jenkins Linux amd64 Build #13448 [push] Update Protobuf to v21.1... started at 12:15

Jenkins Linux amd64 Build #13448 [push] Update Protobuf to v21.1... passed after 2 hr 37 min

Jenkins Linux s390x Build #13475 [push] Update Protobuf to v21.1... aborted after 3 hr 17 min

Jenkins Linux ppc64le Build #12469 [push] Update Protobuf to v21.1... aborted after 3 hr 17 min


License ?

Hello, under what license is this project released under ? I would like to study it to learn from it. Thank you.

Hello, I have the same question? What is the license?

Sorry for late response, will add a MIT license.

Sorry for late response, will add a MIT license.

Added a pull request with it for you ;)


Add initial tests to auth command

Nice work @lricoy! So glad to finally have some tests. I'll try to review this one over the weekend.
Also curious to see if @aurelienshz has any feedback.

This looks to be enough for testing the auth module. In a close future I really think we should write a decent mock for the fs module, since we are testing a lot of things related to reading from / writing to the disk, and actually hitting the disk can be slow and unreliable. But I think for now using tmp for auth is fine.
This looks pretty nice, but when running the test it hangs after the end and never gives back the prompt. Have you encountered this problem?

Apparently, the issue is that the server that gets started by miniOauthServer hangs indefinitely even after the end of the test. In #85 I mock it and I also mock the function from googleapis that allow to hit Google's token endpoint, and this allows us to perform assertions on the obtained token (and also get rid of the timeout with 1000ms delay :slightly_smiling_face: ).


Auto switch between dark and light theme

Hello.
First of all, thank you for your work. I think your theme is really beautiful
Second, I've been looking for how to automatically change the theme from light to dark, and I think I've found something interesting.
Although I could make a pull request on this, I'd like to let you know first.
I found the code of the auto theme
And I tried to apply the same.
So i create a theme-auto-lugit.css (in public/assets/css) with:
@import "./theme-light.css" (prefers-color-scheme: light);
@import "./theme-dark.css" (prefers-color-scheme: dark);

And at least it works for me.
I think it would be cool if you could add this as well

That's actually a great idea!
I would probably create a third theme file called "auto.scss" here. So that we can keep all "3" themes as an option (dark, light, auto).
I'll look into it tomorrow.
Thanks for the idea! Cheers!

Thanks for the idea! Done!
Cheers


Update subgraph urls (moving away from hosted service)

Closes #580
Main changes:

Update subgraph and gaugesSubgraph config url for all chains

Questions:

do we still need to support blockNumbersSubgraph? -> used to calculate accumulated swapFees on the last 24h
do we need to push those changes to the master branch and perform a latest release?
I didn't add subgraph info for Mode because we don't have any config info for it - do we need to add it?
I didn't update subgraph info for Goerli because we don't have a new one - should we remove all config info for Goerli and drop support for it?

do we still need to support blockNumbersSubgraph? -> used to calculate accumulated swapFees on the last 24h

I'm not sure but if you need here are the endpoints you can use.
https://api.studio.thegraph.com/query/48427/ethereum-blocks/version/latest
https://api.studio.thegraph.com/query/48427/avalanche-blocks/version/latest
https://api.studio.thegraph.com/query/48427/arbitrum-blocks/version/latest
https://api.studio.thegraph.com/query/48427/optimism-blocks/version/latest
https://api.studio.thegraph.com/query/48427/polygon-blocks/version/latest
https://api.studio.thegraph.com/query/48427/gnosis-blocks/version/latest

I didn't add subgraph info for Mode because we don't have any config info for it - do we need to add it?

Mode is already there right? 🤔 and it looks because it already uses the Studio endpoint.

I didn't update subgraph info for Goerli because we don't have a new one - should we remove all config info for Goerli and drop support for it?

I suggest removing it. Goerli has stopped producing blocks and it's not useful anymore.

Thanks for the feedback!

if we have endpoints for blockNumbersSubgraph, I'll simply update them 👍
didn't see Mode there - ignore the comment 😅
I'll go ahead and remove Goerli

Not sure it's related, but there are a few failing APR tests.
It seems to be related to token price repository still using coingecko as their main source of data. On other areas of the code we already moved to the API as source for token prices.
Not sure this is something we'd like to spend time on, so will put this on hold for now.
Questions we'd need to answer:

is someone using the SDK as source of data for this info? or is this already being taken care somewhere else (e.g. api)?
is it ok to simply skip those tests for now and eventually remove APR content from the old SDK?

Another alternative is to keep the APR functionality but update to make it a proxy to the API. Would keep the functionality incase anyone is using it. I think this would be fairly easy to do and would probably allow us to remove a bunch of dependecies.

Hi @brunoguerios @johngrantuk! When can we expect to be merged in?
The hosted subgraphs used by the SDK have been made unavailable since 8:30 CET breaking most of its features. I'll cherry pick this branch for now to solve the issues we're having because of this.

Hi @brunoguerios @johngrantuk! When can we expect to be merged in?
The hosted subgraphs used by the SDK have been made unavailable since 8:30 CET breaking most of its features. I'll cherry pick this branch for now to solve the issues we're having because of this.

@peterpolman - Sorry about that, just released now in<EMAIL_ADDRESS>Please open a ticket or reach out if there are still issues.
Also worth noting, this SDK will be deprecated in favour of a new one with narrower focus. Any depreacted functionality (e.g. APRs) should be available from the API: https://api-v3.balancer.fi/, which should give you a better UX.

Thanks for the patch! Tested and solves the issues🙏

Aware of the new version as well. Will schedule a task for a migration.
Not aware of the API tbh. How can I query for the APR and token pair prices of my pool?

Thanks for the patch! Tested and solves the issues🙏
* Aware of the new version as well. Will schedule a task for a migration.

* Not aware of the API tbh. How can I query for the APR and token pair prices of my pool?

The API is running as a graphql server and is deployed at https://api-v3.balancer.fi/, its self-documenting so you can click around the link to explore.
This example query gets pools details including APRs:
{
  poolGetPool(id: "0x7f2b3b7fbd3226c5be438cde49a519f442ca2eda00020000000000000000067d", chain:MAINNET) {
    id
    name
    type
    version
    allTokens {
      address
      name
    }
    displayTokens {
      ...on GqlPoolTokenDisplay {
        symbol
      }
    }
    dynamicData {
      totalLiquidity
      apr {
        swapApr
        nativeRewardApr {
          ...on GqlPoolAprTotal {
            total
          }
        }
        thirdPartyApr {
          ...on GqlPoolAprTotal {
            total
          }
        }
        items {
          title
          apr {
            ...on GqlPoolAprRange {
              min
              max
            }
            ...on GqlPoolAprTotal {
              total
            }
          }
        }
      }
    }
  }
}

If you confirm what you mean by token pair prices (and maybe how you currently do that) I could try to find the best query for you.

Thanks! I appreciate that 🙏
This method is responsible for updating price information periodically using the SDK:
https://github.com/thxprotocol/monorepo/blob/develop/apps/api/src/app/services/BalancerService.ts#L109-L130
The USD value of the token pair 20USD-80THX, our token THX and USDC.e are consumed by our frontend. Additionally we fetch the latest BAL USD value from Coinbase, but preferably we fetch this from the API as well.
20USD-80THX:
https://app.balancer.fi/#/polygon/pool/0xb204bf10bc3a5435017d3db247f56da601dfe08a0002000000000000000000fe

Thanks! I appreciate that 🙏
This method is responsible for updating price information periodically using the SDK: https://github.com/thxprotocol/monorepo/blob/develop/apps/api/src/app/services/BalancerService.ts#L109-L130
The USD value of the token pair 20USD-80THX, our token THX and USDC.e are consumed by our frontend. Additionally we fetch the latest BAL USD value from Coinbase, but preferably we fetch this from the API as well.
20USD-80THX: https://app.balancer.fi/#/polygon/pool/0xb204bf10bc3a5435017d3db247f56da601dfe08a0002000000000000000000fe

You can use:
{
  tokenGetCurrentPrices(chains: [POLYGON]) {
    chain
    address
    price
  }
}

and filter for the token addresses.


Navbar buttons in local are not properly highlighted

Looking forward for reporting a security issue:
Please report security issues by sending an email to<EMAIL_ADDRESS>instead of raising a GitHub issue.
Describe the bug
In thelocal setup of fcc, the navbar buttons for curriculum, forum and news have a small white dash popping out of the white box.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots

could not reproduce :(

@ahmadabdolsaheb I meant it happens when you hover the cursor over the buttons, sry if I was not clear before.

Couldn't repro either on a Windows 10 VM using Chrome, although I do see a border on the bottom when you hover.

@Nirajn2311 which OS and browser are you using?
I tried to reproduce in local as well as the production site, I was unable to reproduce. I am on Windows 10 Home and using Chrome 71.0.3578.98

I cannot reproduce @Nirajn2311, feel free to raise a PR though. If it is effecting you, it will be the same for others

@ahmadabdolsaheb Strange, I could reproduce this only in Chrome(v72.0.3626.96).

@Nirajn2311 it might be a webkit specific styling. I wonder if you could track down that dash in the devtools by selecting the that element and moving up and down the dom tree and checking out the styles at the same time. That might be helpful :/

@Nirajn2311 Are you working on it? If needed, I can help.

@ValeraS Were you able to reproduce the issue?

@Nirajn2311 I can reproduce it when zooming page.

@ValeraS I couldn't figure out how to fix it, so you can start working on it.


patchelf hook: propagate patchelf failure

Otherwise we could succeed while producing bad output (in pratice). https://matrix.to/#/!RROtHmAaQIkiJzJZZE:nixos.org/$7P4dd-uuZHFp7pz-d82FGDrV2v4lWcxGcauEBPZ-CMs?via=nixos.org&via=matrix.org&via=nixos.dev
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

How do we guarantee we don't call this for non-ELF now? Do we guard all callsites by "isELF" ?

It is guarded by isELF just two lines above.

It is guarded by isELF just two lines above. EDIT: maybe I misunderstood what you really meant.

No I think you are right, I just wondered if "isELF" doesn't have any false positives

Uh well, I don't know
       > shrinking RPATHs of ELF executables and libraries in /nix/store/byw2fqcvhqhcdg9mjzqd8ycy44ssnywk-xgcc-12.3.0
       > shrinking /nix/store/byw2fqcvhqhcdg9mjzqd8ycy44ssnywk-xgcc-12.3.0/lib/gcc/aarch64-unknown-linux-gnu/12.3.0/crtendS.o
       > patchelf: wrong ELF type


module 'pydantic.fields' has no attribute 'ModelField' in awswrangler[modin,ray]==3.2.1

Describe the bug
When trying to invote wr.engine.initialize() with "awswrangler[modin,ray]==3.2.1", following error reported
Traceback (most recent call last):
  File "//wr_init.py", line 8, in <module>
    df = pd.DataFrame({'col1': [1, 2], 'col2': [3, 4]})
  File "/usr/local/lib/python3.10/site-packages/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/modin/pandas/dataframe.py", line 179, in __init__
    distributed_frame = from_non_pandas(data, index, columns, dtype)
  File "/usr/local/lib/python3.10/site-packages/modin/pandas/utils.py", line 71, in from_non_pandas
    new_qc = FactoryDispatcher.from_non_pandas(df, index, columns, dtype)
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/dispatching/factories/dispatcher.py", line 173, in from_non_pandas
    return cls.get_factory()._from_non_pandas(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/dispatching/factories/dispatcher.py", line 116, in get_factory
    Engine.subscribe(cls._update_factory)
  File "/usr/local/lib/python3.10/site-packages/modin/config/pubsub.py", line 217, in subscribe
    callback(cls)
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/dispatching/factories/dispatcher.py", line 158, in _update_factory
    cls.__factory.prepare()
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/dispatching/factories/factories.py", line 447, in prepare
    from modin.core.execution.ray.implementations.pandas_on_ray.io import (
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/io/__init__.py", line 16, in <module>
    from .io import PandasOnRayIO
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/io/io.py", line 43, in <module>
    from ..dataframe import PandasOnRayDataframe
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/dataframe/__init__.py", line 16, in <module>
    from .dataframe import PandasOnRayDataframe
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/dataframe/dataframe.py", line 16, in <module>
    from ..partitioning.partition_manager import PandasOnRayDataframePartitionManager
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/partitioning/__init__.py", line 16, in <module>
    from .partition import PandasOnRayDataframePartition
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/partitioning/partition.py", line 28, in <module>
    class PandasOnRayDataframePartition(PandasDataframePartition):
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/partitioning/partition.py", line 178, in PandasOnRayDataframePartition
    _iloc = execution_wrapper.put(PandasDataframePartition._iloc)
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/common/engine_wrapper.py", line 111, in put
    return ray.put(data, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/auto_init_hook.py", line 18, in auto_init_wrapper
    return fn(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/client_mode_hook.py", line 103, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/worker.py", line 2612, in put
    object_ref = worker.put_object(value, owner_address=serialize_owner_address)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/worker.py", line 693, in put_object
    serialized_value = self.get_serialization_context().serialize(value)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/worker.py", line 618, in get_serialization_context
    context_map[job_id] = serialization.SerializationContext(self)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/serialization.py", line 151, in __init__
    serialization_addons.apply(self)
  File "/usr/local/lib/python3.10/site-packages/ray/util/serialization_addons.py", line 58, in apply
    register_pydantic_serializer(serialization_context)
  File "/usr/local/lib/python3.10/site-packages/ray/util/serialization_addons.py", line 21, in register_pydantic_serializer
    pydantic.fields.ModelField,
AttributeError: module 'pydantic.fields' has no attribute 'ModelField'

The requirements.txt from awswrangler[modin,ray]==3.2.1 as below:
aiohttp==3.8.4
aiohttp-cors==0.7.0
aiosignal==1.3.1
annotated-types==0.5.0
async-timeout==4.0.2
attrs==23.1.0
awswrangler==3.2.1
blessed==1.20.0
boto3==1.27.0
botocore==1.30.0
cachetools==5.3.1
certifi==2023.5.7
charset-normalizer==3.1.0
click==8.1.3
colorful==0.5.5
distlib==0.3.6
filelock==3.12.2
frozenlist==1.3.3
fsspec==2023.6.0
google-api-core==2.11.1
google-auth==2.21.0
googleapis-common-protos==1.59.1
gpustat==1.1
grpcio==1.51.3
idna==3.4
jmespath==1.0.1
jsonschema==4.17.3
modin==0.20.1
msgpack==1.0.5
multidict==6.0.4
numpy==1.25.0
nvidia-ml-py==11.525.131
opencensus==0.11.2
opencensus-context==0.1.3
packaging==23.1
pandas==1.5.3
platformdirs==3.8.0
prometheus-client==0.17.0
protobuf==4.23.3
psutil==5.9.5
py-spy==0.3.14
pyarrow==12.0.1
pyasn1==0.5.0
pyasn1-modules==0.3.0
pydantic==2.0.1
pydantic_core==2.0.2
pyrsistent==0.19.3
python-dateutil==2.8.2
pytz==2023.3
PyYAML==6.0
ray==2.5.0
requests==2.31.0
rsa==4.9
s3transfer==0.6.1
six==1.16.0
smart-open==6.3.0
typing_extensions==4.7.1
urllib3==1.26.16
virtualenv==20.21.0
wcwidth==0.2.6
yarl==1.9.2

How to Reproduce
Good baseline of modin+ray without pydantic errors

Create a simple ray_init.py

import ray
ray.init()

import modin.pandas as pd
df = pd.DataFrame({'col1': [1, 2], 'col2': [3, 4]})

Create the first dockerfile with Python3.10 base image

FROM python:3.10-slim as build

ARG FUNCTION_DIR

RUN apt-get update
RUN apt-get install -y --no-install-recommends \
	build-essential gcc

RUN pip3 install modin==0.20.1 ray==2.5.0
COPY ray_init.py .

Build and run the container

% docker build --no-cache -t wr_test -f ./Dockerfile .
% docker run --rm --shm-size=4G wr_test python3.10 ray_init.py
2023-07-05 01:03:40,427 INFO worker.py:1636 -- Started a local Ray instance.
UserWarning: When using a pre-initialized Ray cluster, please ensure that the runtime env sets environment variable __MODIN_AUTOIMPORT_PANDAS__ to 1
UserWarning: Distributing <class 'dict'> object. This may take some time.

Error with awswrangler

Create a new wr_init.py

import awswrangler as wr
import modin.pandas as pd

wr.engine.initialize()
print(f"Execution Engine: {wr.engine.get()}")
print(f"Memory Format: {wr.memory_format.get()}")

df = pd.DataFrame({'col1': [1, 2], 'col2': [3, 4]})

Update the Dockerfile to install "awswrangler[modin,ray]==3.2.1"

FROM python:3.10-slim as build

ARG FUNCTION_DIR

RUN apt-get update
RUN apt-get install -y --no-install-recommends \
	build-essential gcc

RUN pip3 install modin==0.20.1 ray==2.5.0
COPY wr_init.py .

FROM build
RUN pip3 install "awswrangler[modin,ray]==3.2.1"

Build and run the container

% docker build --no-cache -t wr_test -f ./Dockerfile .
% docker run --rm --shm-size=4G wr_test python3.10 wr_init.py
2023-07-05 01:16:37,943 ERROR services.py:1207 -- Failed to start the dashboard , return code 1
2023-07-05 01:16:37,943 ERROR services.py:1232 -- Error should be written to 'dashboard.log' or 'dashboard.err'. We are printing the last 20 lines for you. See 'https://docs.ray.io/en/master/ray-observability/ray-logging.html#logging-directory-structure' to find where the log file is.
2023-07-05 01:16:37,944 ERROR services.py:1276 --
The last 20 lines of /tmp/ray/session_2023-07-05_01-16-35_556700_771/logs/dashboard.log (it contains the error message from the dashboard):
  File "/usr/local/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/usr/local/lib/python3.10/site-packages/ray/dashboard/modules/log/log_manager.py", line 8, in <module>
    from ray.util.state.common import (
  File "/usr/local/lib/python3.10/site-packages/ray/util/state/__init__.py", line 1, in <module>
    from ray.util.state.api import (
  File "/usr/local/lib/python3.10/site-packages/ray/util/state/api.py", line 17, in <module>
    from ray.util.state.common import (
  File "/usr/local/lib/python3.10/site-packages/ray/util/state/common.py", line 120, in <module>
    @dataclass(init=True)
  File "/usr/local/lib/python3.10/site-packages/pydantic/dataclasses.py", line 139, in dataclass
    assert init is False, 'pydantic.dataclasses.dataclass only supports init=False'
AssertionError: pydantic.dataclasses.dataclass only supports init=False
2023-07-05 01:16:39,212 INFO worker.py:1636 -- Started a local Ray instance.
Execution Engine: EngineEnum.RAY
Memory Format: MemoryFormatEnum.MODIN
UserWarning: When using a pre-initialized Ray cluster, please ensure that the runtime env sets environment variable __MODIN_AUTOIMPORT_PANDAS__ to 1
Traceback (most recent call last):
  File "/app/wr_init.py", line 8, in <module>
    df = pd.DataFrame({'col1': [1, 2], 'col2': [3, 4]})
  File "/usr/local/lib/python3.10/site-packages/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/modin/pandas/dataframe.py", line 179, in __init__
    distributed_frame = from_non_pandas(data, index, columns, dtype)
  File "/usr/local/lib/python3.10/site-packages/modin/pandas/utils.py", line 71, in from_non_pandas
    new_qc = FactoryDispatcher.from_non_pandas(df, index, columns, dtype)
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/dispatching/factories/dispatcher.py", line 173, in from_non_pandas
    return cls.get_factory()._from_non_pandas(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/dispatching/factories/dispatcher.py", line 116, in get_factory
    Engine.subscribe(cls._update_factory)
  File "/usr/local/lib/python3.10/site-packages/modin/config/pubsub.py", line 217, in subscribe
    callback(cls)
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/dispatching/factories/dispatcher.py", line 158, in _update_factory
    cls.__factory.prepare()
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/dispatching/factories/factories.py", line 447, in prepare
    from modin.core.execution.ray.implementations.pandas_on_ray.io import (
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/io/__init__.py", line 16, in <module>
    from .io import PandasOnRayIO
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/io/io.py", line 43, in <module>
    from ..dataframe import PandasOnRayDataframe
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/dataframe/__init__.py", line 16, in <module>
    from .dataframe import PandasOnRayDataframe
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/dataframe/dataframe.py", line 16, in <module>
    from ..partitioning.partition_manager import PandasOnRayDataframePartitionManager
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/partitioning/__init__.py", line 16, in <module>
    from .partition import PandasOnRayDataframePartition
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/partitioning/partition.py", line 28, in <module>
    class PandasOnRayDataframePartition(PandasDataframePartition):
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/implementations/pandas_on_ray/partitioning/partition.py", line 178, in PandasOnRayDataframePartition
    _iloc = execution_wrapper.put(PandasDataframePartition._iloc)
  File "/usr/local/lib/python3.10/site-packages/modin/core/execution/ray/common/engine_wrapper.py", line 111, in put
    return ray.put(data, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/auto_init_hook.py", line 18, in auto_init_wrapper
    return fn(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/client_mode_hook.py", line 103, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/worker.py", line 2612, in put
    object_ref = worker.put_object(value, owner_address=serialize_owner_address)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/worker.py", line 693, in put_object
    serialized_value = self.get_serialization_context().serialize(value)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/worker.py", line 618, in get_serialization_context
    context_map[job_id] = serialization.SerializationContext(self)
  File "/usr/local/lib/python3.10/site-packages/ray/_private/serialization.py", line 151, in __init__
    serialization_addons.apply(self)
  File "/usr/local/lib/python3.10/site-packages/ray/util/serialization_addons.py", line 58, in apply
    register_pydantic_serializer(serialization_context)
  File "/usr/local/lib/python3.10/site-packages/ray/util/serialization_addons.py", line 21, in register_pydantic_serializer
    pydantic.fields.ModelField,
AttributeError: module 'pydantic.fields' has no attribute 'ModelField'

Expected behavior
No error or as per the output before installation of AWSWrangler
OS
python3.10 base image on aarch64
Python version
3.10
AWS SDK for pandas version
N/A

Thanks for raising this @kangks, it seems that there is a major issue opened a couple of days ago in Ray about this:
https://github.com/ray-project/ray/issues/37019
The current workaround is to force a lower version of Pydantic as it's currently conflicting with Ray:
pip install "pydantic<2"

After downgrading pydantic I was able to run your code snippet without issues

I can confirm the same, using ray[tune]==2.5.1 and pydantic > 2 yielded the same error.

Thanks for raising this @kangks, it seems that there is a major issue opened a couple of days ago in Ray about this: ray-project/ray#37019
The current workaround is to force a lower version of Pydantic as it's currently conflicting with Ray:
pip install "pydantic<2"

After downgrading pydantic I was able to run your code snippet without issues

Did also do that, restricting my poetry package manager accordingly.

confirm that ray>=2.3 isn't compatible with pydantic>=2

Having the same issue with after poetry add awswranger[modin,ray] with pip list showing:
awsrangler==3.4.0
modin==0.23.1
pydantic==2.4.2
pydantic_core==2.10.1
ray==2.3.1


Bug with lockitron plugin example

I used the lockitron plugin example as a template to create my own. I kept getting a seemingly random homebridge crash with a fatal error:
/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/lib/util/once.js:11
      throw new Error("This callback function has already been called by someone else; it can only be called one time.");
      ^

I took me a while to figure this out, but the reason is because the example I copied binds the "get" event twice! The culprit is line 25 of example-plugins/homebridge-lockitron/index.js:
 this.service
    .getCharacteristic(Characteristic.LockCurrentState)
    .on('get', this.getState.bind(this));

  this.service
    .getCharacteristic(Characteristic.LockTargetState)
    .on('get', this.getState.bind(this))
    .on('set', this.setState.bind(this));
}

I suggest removing line 25 to prevent some other poor soul from repeating my mistake :)

Sorry, I don't grok it. I read two characteristics (current and target) and two distinct events (get and set).
Can you elaborate a little further what the culprit is supposed to be?

@snowdd1 You are right, I totally missed that they are different characteristics until you just pointed it out!


Call OnServiceInstall after successful service installation

Reference Issues/PRs
None.
What does this implement/fix?
This fix ensures that the OnServiceInstall Action is called after successful service installation.
Some line ending inconsistencies have also been straightened out.
Any other comments?
No.

Thanks for your contribution. I will release a new version in a bit.

Released as 2.0.12

@PeterKottas thank you for releasing, much appreciated.


can flux nodes communicate with each other easily / natively over flux?

not easily, they would have to pickup each other's ip and nodes may go down so this is a bad idea to attempt communication


Clean up PR42 - add support for CC3220SF

@flit I don't see the button to add reviewers for some reason. Please review this PR.


fix: Ensuring that useQuery's isFetching becomes true after query changes

An issue was found where if the query or parameters change for useQuery, there would be single render cycle where isFetching was false before it became true.
For example:
1. isFetching: false // query changed but the variable is still false
2. isFetching: true // state has been updated
3. isFetching: false // query has finished re-executing

This fix ensures that isFetching is correctly set to true as soon as the hook executes due to a query change.
1. isFetching: true // query change causes hook to execute, compare current query to new query -
track with internal `shouldFetch` variable.
2. isFetching: true // state has been updated
3. isFetching: false // query has finished re-executing

In our case, we had multiple renders with isFetching being incorrectly false in combination with the old stale data, probably due to us having multiple useQuery hooks firing and re-rendering things.
I have tested @Chriztiaan's changes locally, and the latest version in this PR resolves the issue for us.


Verify behavior around pre-existing service accounts

We don't currently have a known pattern around how to handle pre-existing service accounts. We should test what happens and decide on a defined policy.

reported by @thejosephstevens

Hey all! The super quick feedback I'd give on the functionality here is, as a consumer I don't actually much care what the outcome is as long as it's in the docs. Part of the reason I landed in this spot is the existing documentation talks at length about generating all of the bindings at runtime, but it doesn't actually mention (unless I missed a section) anything about management of the subjects. Since there was no reference I assume it was BYO, and that turned out to work for our original installs. It also seemed to follow from the notion that it would set up bindings with other resources (like groups) that don't have a visible kube resource backing them.
FWIW, the spec I was running when I ran into my issues was this:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: node-labeler
  namespace: infra
imagePullSecrets:
 - name: image-pull-creds
---
apiVersion: rbac.authorization.k8s.io/v1beta1
kind: ClusterRole
metadata:
  name: node-labeler
rules:
- apiGroups: [""]
  resources: ["nodes"]
  verbs: ["list", "get", "patch"]
---
apiVersion: rbacmanager.reactiveops.io/v1beta1
kind: RBACDefinition
metadata:
  name: node-labeler
rbacBindings:
  - name: node-labeler
    subjects:
      - kind: ServiceAccount
        name: node-labeler
    clusterRoleBindings:
      - namespace: infra
        clusterRole: node-labeler

With this config, rbac-manager logged errors on creating the service account due to missing namespace (prior to investigation of this, it was hooking up to an existing SA just fine, so it may have been throwing an error without me noticing).

Further updates, it appears that if you reference an existing subject it doesn't cause a failure as I previously thought, the binding actually does get created correctly. However it does result in the rbac-manager throwing errors at runtime. What I would suggest from a user perspective here is, attempt to create the subject, but if the subject is already found, go ahead and adopt the discovered subject. That way you can get a best of both worlds where in my deployment processes I can deploy SAs ahead of time (so deploys can reference them and not run into race conditions waiting for rbac-manager to resolve state), but still get the stability guarantees that if I delete an SA, rbac-manager will create one and get me back to a stable state.

My only hangup with adopting a service account as it's own when the SA exists beforehand is that if you remove the RBACDefinition some time down the road, the service account will be deleted too. That would most likely be an undesirable outcome.

My vote is to fail on the pre-existence of a serviceAccount.  This would simplify the issue and ensure that rbac-manager remains the source of truth for those service accounts.  This would also tie nicely into #103

I added more documentation in this section of the README in #128
I did not change any behavior currently, so rbac-manager still logs an error if the serviceaccount is pre-existing but the permissions are indeed granted. Also rbac-manager does not delete the pre-existing service account when removing the RBACDefinition.
Happy to discuss other ideas for this in a new issue if this doesn't satisfy requirement, but I'll close this issue for now.


Rails install generator

[ ] Create a Phlex initialiser for configuration
[ ] Configure Tailwind if installed
[ ] Configure autoload paths to include app/views, namespaced as Views

I can work on this, but I have two questions:

Create a Phlex initialiser for configuration

Do we agree there is nothing to configure yet? (lib/phlex/configuration.rb is empty)

Configure Tailwind if installed

Do you have something specific in mind for this?

Yeah, there's no configuration yet. I think we'll need to add some for the compiler soon but I don't think we need to do that yet. For Tailwind, we should update the configuration to include Ruby files in app/views.
Here's the default configuration in config/tailwind.config.js.
const defaultTheme = require('tailwindcss/defaultTheme')

module.exports = {
  content: [
    './public/*.html',
    './app/helpers/**/*.rb',
    './app/javascript/**/*.js',
    './app/views/**/*.{erb,haml,html,slim}',
    './app/content/**/*.{erb,haml,html,slim,rb}'
  ],
  theme: {
    extend: {
      fontFamily: {
        sans: ['Inter var', ...defaultTheme.fontFamily.sans],
      },
    },
  },
  plugins: [
    require('@tailwindcss/forms'),
    require('@tailwindcss/aspect-ratio'),
    require('@tailwindcss/typography'),
  ]
}

Ideally, we should at rb to this list './app/views/**/*.{erb,haml,html,slim}'. Though I think it might be more reliable to simply add another item to the content array ./app/views/**/*.rb'.


Add support for Russian language

Officially, Google Assistant supports Russian language, but SDK doesn't have support for it. Will SDK support Russian language later?

Is this for the service API, or the library?

+1 wait for russian lang support for Python library (already soldering raspberry assistant, but cant understant him)

It's for Google Assistant Service

I am waiting Russian language for library too.

Is it difficult to implement?

Пока еще нет поддержки. Хотя в Google Assistant app уже появился русский язык для автоматизации дома на моем iPhone. Тоже жду поддержки русского языка.
Обещали к концу 2018.

15 months later, still waiting...

still waiting!

Still waiting! Guys, seriously 😳


Update stable version in repo from 1.2.0 to 1.3.0

Think about update stable version to 1.3.0
Version: stable=1.2.0 (67 days old) => latest=1.3.0 (19 days old)
Installs: stable=24 (29.63%), latest=5 (6.17%), total=81
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 1.3.0 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

Is there any reason to delay updating stable any longer?
Please drop a commit if you know any reason to refrain from updating.
If there is no response until 14.07.2024 we could decide to update anyway.

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 1.3.0 but the current stable version is already 1.3.0.
So this issue should be closed.
@mcm1957 for evidence


[Release-1.24] Install test images are Out-of-Date

Backport tracking for https://github.com/k3s-io/k3s/issues/6377

Closing as we generally do not test CI changes (CI tests them itself 🙂 )


update monitoring logic for packages in CLI for uTKG Cluster creation/upgrade

(This is used to request new product features)
Describe the feature request
For uTKG, CLI needs to update the packages monitoring logic to depend on ClusterBootstrap (CBT) custom resource (CR) status conditions
Describe alternatives you've considered
Affected product area (please put an X in all that apply)

[ ] APIs
[ ] Addons
[X] CLI
[ ] Docs
[ ] IAM
[ ] Installation
[ ] Plugin
[ ] Security
[ ] Test and Release
[ ] User Experience

Additional context

Use the ClusterBoostrap status condition reference - https://github.com/vmware-tanzu/tanzu-framework/blob/main/apis/run/v1alpha3/clusterbootstrap_types.go#L28
https://github.com/vmware-tanzu/tanzu-framework/issues/2388
https://github.com/vmware-tanzu/tanzu-framework/pull/2743

new issue created https://github.com/vmware-tanzu/tanzu-framework/issues/2780, hence closing this.


Remove Prettier config files

Since linters, formatters, etc. are really specific to the person writing the code, I find that it's better to keep their configuration out of the repository root.

Using Prettier in this case will improve the readability of code. Whenever someone commits to the repo, the code will be formatted which is better
Why do you think that Prettier should be removed?

You make a fair point. My thinking was that it seems unnecessary to make it repo-wide, especially since a number of people prefer to use other linters/formatters.

I agree with you about people having different preferences.
I personally don't know about other formatting options than prettier but prettier is the most popular one and there can be another way to use a different formatter.
But I don't think removing prettier completely will make sense because majority people don't use any formatter.

enforcing a coding style is very beneficial. It makes the code way easier to read, aswell as organizing the whole project. That way all code is similar across files. Adding ESlint to enforce this, is also beneficial since it can enforce better coding styles.  https://eslint.org/docs/rules/

Yeah. Fair enough.
Closing.


ARROW-13286: [CI] Require docker-compose 1.27.0 or later

We need it for "extends".
See also:

https://issues.apache.org/jira/browse/ARROW-13199
https://github.com/apache/arrow/pull/10611
https://github.com/docker/compose/pull/7588

A failed case: https://app.travis-ci.com/github/apache/arrow/jobs/522634908

+1
Worked: https://app.travis-ci.com/github/apache/arrow/jobs/522672412


Refine readme

this one only required lil changes. :)
Would it make sense to rename the repo to simply heroku-addon? So it is uniform with the general naming style?

Would it make sense to rename the repo to simply heroku-addon? So it is uniform with the general naming style?

It would!
Done! 😄


Fix NoClassDefFoundError in RewriteTest on jgit diff()

Fixes https://github.com/openrewrite/rewrite/issues/3243

Since I am facing the problem myself right now, I went ahead and merged it.

I can also confirm that this is fixed now.


Switch dev-tools integration pipeline to use dynamic workers

This switches the dev-tools integration test pipeline to use dynamic jenkins workers instead of the old static workers that we have used so far. The idea is to ensure that the dynamic workers can be used without risking too much.

cool it worked :dancers:
/lgtm


Add /latest/ URL to access the docs

We need to be able to publish https://docs.jmix.io/jmix/1.0/studio/subscription.html link (e.g. in Store's email templates)
like https://docs.jmix.io/jmix/latest/studio/subscription.html
to automatically use latest available documentation version.

Now https://docs.jmix.io/jmix/latest/* redirects to https://docs.jmix.io/jmix/1.0/*


German translation of Basic JavaScript

[x] Comment your JavaScript Code Complete
[x] Declare JavaScript Variables Complete
[x] Storing Values with the Assignment Operator Complete
[x] Initializing Variables with the Assignment Operator Complete
[ ] Understanding Uninitialized Variables Complete
[ ] Understanding Case Sensitivity in Variables Complete
[ ] Add Two Numbers with JavaScript Complete
[ ] Subtract One Number from Another with JavaScript Complete
[ ] Multiply Two Numbers with JavaScript Complete
[ ] Divide One Number by Another with JavaScript Complete
[ ] Increment a Number with JavaScript Complete
[ ] Decrement a Number with JavaScript Complete
[ ] Create Decimal Numbers with JavaScript Complete
[ ] Multiply Two Decimals with JavaScript Complete
[ ] Divide one Decimal by Another with JavaScript Complete
[ ] Finding a Remainder in JavaScript Complete
[ ] Compound Assignment With Augmented Addition Complete
[ ] Compound Assignment With Augmented Subtraction Complete
[ ] Compound Assignment With Augmented Multiplication Complete
[ ] Compound Assignment With Augmented Division Complete
[ ] Convert Celsius to Fahrenheit Complete
[ ] Declare String Variables Complete
[ ] Escaping Literal Quotes in Strings Complete
[ ] Quoting Strings with Single Quotes Complete
[ ] Escape Sequences in Strings Complete
[ ] Concatenating Strings with Plus Operator Complete
[ ] Concatenating Strings with the Plus Equals Operator Complete
[ ] Constructing Strings with Variables Complete
[ ] Appending Variables to Strings Complete
[ ] Find the Length of a String Complete
[ ] Use Bracket Notation to Find the First Character in a String Complete
[ ] Understand String Immutability Complete
[ ] Use Bracket Notation to Find the Nth Character in a String Complete
[ ] Use Bracket Notation to Find the Last Character in a String Complete
[ ] Use Bracket Notation to Find the Nth-to-Last Character in a String Complete
[ ] Word Blanks Complete
[ ] Store Multiple Values in one Variable using JavaScript Arrays Complete
[ ] Nest one Array within Another Array Complete
[ ] Access Array Data with Indexes Complete
[ ] Modify Array Data With Indexes Complete
[ ] Access Multi-Dimensional Arrays With Indexes Complete
[ ] Manipulate Arrays With push() Complete
[ ] Manipulate Arrays With pop() Complete
[ ] Manipulate Arrays With shift() Complete
[ ] Manipulate Arrays With unshift() Complete
[ ] Shopping List Complete
[ ] Write Reusable JavaScript with Functions Complete
[ ] Passing Values to Functions with Arguments Complete
[ ] Global Scope and Functions Complete
[ ] Local Scope and Functions Complete
[ ] Global vs. Local Scope in Functions Complete
[ ] Return a Value from a Function with Return Complete
[ ] Assignment with a Returned Value Complete
[ ] Stand in Line Incomplete
[ ] Understanding Boolean Values Complete
[ ] Use Conditional Logic with If Statements Complete
[ ] Comparison with the Equality Operator Incomplete
[ ] Comparison with the Strict Equality Operator Incomplete
[ ] Comparison with the Inequality Operator Incomplete
[ ] Comparison with the Strict Inequality Operator Incomplete
[ ] Comparison with the Greater Than Operator Incomplete
[ ] Comparison with the Greater Than Or Equal To Operator Incomplete
[ ] Comparison with the Less Than Operator Incomplete
[ ] Comparison with the Less Than Or Equal To Operator Incomplete
[ ] Comparisons with the Logical And Operator Incomplete
[ ] Comparisons with the Logical Or Operator Incomplete
[ ] Introducing Else Statements Incomplete
[ ] Introducing Else If Statements Incomplete
[ ] Logical Order in If Else Statements Incomplete
[ ] Chaining If Else Statements Incomplete
[ ] Golf Code Incomplete
[ ] Selecting from many options with Switch Statements Incomplete
[ ] Adding a default option in Switch statements Incomplete
[ ] Multiple Identical Options in Switch Statements Incomplete
[ ] Replacing If Else Chains with Switch Incomplete
[ ] Returning Boolean Values from Functions Incomplete
[ ] Return Early Pattern for Functions Incomplete
[ ] Counting Cards Incomplete
[ ] Build JavaScript Objects Complete
[ ] Accessing Objects Properties with the Dot Operator Incomplete
[ ] Accessing Objects Properties with Bracket Notation Incomplete
[ ] Accessing Objects Properties with Variables Incomplete
[ ] Updating Object Properties Complete
[ ] Add New Properties to a JavaScript Object Complete
[ ] Delete Properties from a JavaScript Object Complete
[ ] Using Objects for Lookups Incomplete
[ ] Testing Objects for Properties Incomplete
[ ] Introducing JavaScript Object Notation (JSON) Incomplete
[ ] Accessing Nested Objects in JSON Incomplete
[ ] Accessing Nested Arrays in JSON Incomplete
[ ] Record Collection Incomplete
[ ] Iterate with JavaScript For Loops Complete
[ ] Iterate Odd Numbers With a For Loop Complete
[ ] Count Backwards With a For Loop Complete
[ ] Iterate Through an Array with a For Loop Incomplete
[ ] Nesting For Loops Incomplete
[ ] Iterate with JavaScript While Loops Complete
[ ] Profile Lookup Complete
[ ] Generate Random Fractions with JavaScript Complete
[ ] Generate Random Whole Numbers with JavaScript Complete
[ ] Generate Random Whole Numbers within a Range Complete
[ ] Sift through Text with Regular Expressions Complete
[ ] Find Numbers with Regular Expressions Complete
[ ] Find Whitespace with Regular Expressions Complete
[ ] Invert Regular Expression Matches with JavaScript

I'm working on it.

@5k9m any progress on this?

@bugron Unfortunately not as much as I would like - have been swamped with work for the past 3 weeks... Hope to finish the translation until the middle of June!

How is it going? @5k9m

Issue moved to FreeCodeCamp/TranslationExpansion #82 via ZenHub

@codingHahn @5k9m please continue collaboration on the new issue thread.

var a = 3;
var b = 17;
var c = 12;
// Only modify code below this line
a += 12;
b += 9;
c += 7;


 I don't think it's proxy issue, but hover bumpversion doesn't consider usage inside China.
So, anyone could help?

 I don't think it's proxy issue, but hover bumpversion doesn't consider usage inside China.
So, anyone could help?

hover bumpversion is missin the environment variable if I see it right.
https://github.com/go-flutter-desktop/hover/blob/1e7668b522986374c33efe2ec2dd23f2618e6196/cmd/bumpversion.go#L56

hover bumpversion is missin the environment variable if I see it right.
https://github.com/go-flutter-desktop/hover/blob/1e7668b522986374c33efe2ec2dd23f2618e6196/cmd/bumpversion.go#L56

I was wrong it should be passed through

I was wrong it should be passed through

@jld3103  Envrioment also not work.

@jld3103  Envrioment also not work.

 If yes remove https://github.com/go-flutter-desktop/hover/blob/1e7668b522986374c33efe2ec2dd23f2618e6196/cmd/bumpversion.go#L57 then it should work. You can then create a PR

Are you able to build hover yourself? If yes remove https://github.com/go-flutter-desktop/hover/blob/1e7668b522986374c33efe2ec2dd23f2618e6196/cmd/bumpversion.go#L57 then it should work. You can then create a PR

@jld3103 So, would it consider usage inside China? As somewhat reason you know.....

@jld3103 So, would it consider usage inside China? As somewhat reason you know.....

@jld3103 Let me have a try.

@jld3103 Let me have a try.

https://github.com/go-flutter-desktop/hover/pull/195 Submitted.

https://github.com/go-flutter-desktop/hover/pull/195 Submitted.

So it works?

So it works?

You shouldn't delete the whole statement. Just that single line

You shouldn't delete the whole statement. Just that single line

@jld3103 sorry

@jld3103 sorry

Fixed.
Please wait of a new release of hover. Or compile hover from the source.

Fixed.
Please wait of a new release of hover. Or compile hover from the source.


Regenerator runtime problem

Should add new plugin
npm install --save-dev @babel/plugin-transform-runtime
npm install --save @babel/runtime

.babelrc
{
  ...
  "plugins": [
    ...
    ["@babel/plugin-transform-runtime"]
  ],
  "sourceType": "unambiguous"
}

https://babeljs.io/docs/en/babel-plugin-transform-runtime#regenerator
[You'll want to set the sourceType option to either "unambiguous" or "script" if your files aren't ES modules. The docs for the option explain, but essentially because the type is wrong, it will insert import statements into your CommonJS file, causing the error you're seeing.](https://github.com/babel/babel/issues/8900#:~:text=You'll want to set the sourceType option to either "unambiguous" or "script" if your files aren't ES modules. The docs for the option explain%2C but essentially because the type is wrong%2C it will insert import statements into your CommonJS file%2C causing the error you're seeing.)

Will check

Closed due to ESM usage


Add SureSites/User Assistance Links

Link to SureSites (http://utahsuresites.com/) in prominent place.
Add contact information somewhere to contact Kelleigh if a user wants assistance with site selection. I discussed this one with Aaron for awhile, he's going to get us ideas on the design of that.

Same thing, from what I gather, Aaron isn't going to send us anything else.

Closed with 805e278190c84249cb9709e87479de4f4e44d9ac


Translation in Hindi

Hindi is the 4th most spoken language in the world and I am willing to translate the documentation in hindi if you guys want.

We are open to community contributions on new languages. This is what we did for Chinese translations. Go for it.


Add tdgc role

Add new role for cloning a repository into a temp directory.
Useful for automating commits to another git repository.

:+1:


Update IC_GameManager_Class.ahk

Game.GameInstance.Controller.Area.activeMonstersListSize

Thanks for the suggestion.
Your pull request was a bit incomplete without the function and EGS offsets but the idea was sound so it was implemented in a branch with updated offsets yesterday. Unfortunately the branch still had issues with some offsets that weren't corrected until today but it has now been merged..
The relevant function is ReadActiveMonstersCount().


Allow providing SchemaValidatorFactory

This PR adds support to provide custom SchemaValidatorFactory.
At the same time, I've added SchemaFormModule.forRoot() wich returns module with default providers (providing both WidgetFactory and SchemaValidatorFactory)

Thanks :)

required in string widget is not working ("ngx-schema-form": "2.4.3")


Firefox on Windows 10 (v43 and above): codemirror Editor is truncated. and there is a black bar on the gutters and horizontal scroll bar

Firefox on Windows 10 (v43 and above): codemirror Editor is truncated. and there is a black bar on the gutters and horizontal scroll bar.
This happens on FF v43 and above when the number of lines in the editor is above 602.(total lines i have is 800).
As you can see in the attached screenshot, the XML after line 602 is not visible and the black bar is visible in place of line numbers.
Additional css that I have done are :-
.CodeMirror {     height: auto !important; }  .CodeMirror-scroll {     overflow-y: hidden !important;     overflow-x: auto !important; }

Don't mess with the overflow properties on CodeMirror-scroll. That's not supported.


Visualization is not correctly centred on iPhone8 plus

Issue seen on current app store version - not related to recent changes for iPhoneX support.
Visualization is not correctly centred.

How odd - let's get this fixed in the iPhone X update.

Apparently this was broken for weeks according to @nbrooke - we'll need to prioritize "Plus" testing better in the future.


[FLINK-6918] [tests] Harden AbstractOperatorRestoreTestBase by retrying CancelWithSavepoint messages

The problem is that a StreamTask can be in state RUNNING without internally being running.
As a consequence checkpoint message will be discarded. This problem will be solved once
FLINK-4714 has been addressed. Until then, we harden the test case by retrying the
CancelWithSavepoint message.

+1 to merge.

Thanks for reviewing @zentol. Merging this PR.


Use eval for parsing patterns

Seems I broke things with #4 🤔 This goes back to using eval, as it will handle more things and I don't think the security implications are concerning.
Also adds some basic tests to avoid repeating this.

Still doesn't really gracefully handle parse errors.

I settled with the same solution: https://github.com/jasonkarns/bats-mock/pull/18
So +1 for this

I feel like we should pool our efforts @Flamefire :D

Sure :)

To get this going I'd suggest to get this into a better shape. I started with that in my fork: https://github.com/Flamefire/bats-mock
I added all my fixes and enhancements as well as npm packaging and CI to the master branch. We should decide, which repo should become the "official" one and add collaborators so it doesn't die. @jasonkarns seems to be gone and there are many forks around with similar fixes. There is even almost a rewrite at https://github.com/norwoodj/bats-mock which I like for its clean code, but it seems also inactive. But I'd like to incorporate the ides from there. IMO all these indirect variables are a pain to work with


wip - update OpenAI workflow

Why are these changes needed?

Related issue number

Checks

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

Codecov Report

Merging #21 (7c8f19d) into main (f619ecc) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
- Coverage   34.51%   34.35%   -0.16%
==========================================
  Files          17       17
  Lines        1921     1921
  Branches      420      420
==========================================
- Hits          663      660       -3
- Misses       1207     1209       +2
- Partials       51       52       +1

Flag
Coverage Δ

unittests
34.35% <ø> (-0.11%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

Can you by the way, remove this line
# %pip install --quiet flaml"[autogen]"~=2.1.0

from the first code block of this notebook: https://github.com/microsoft/autogen/blob/main/notebook/agentchat_teaching.ipynb

Can you by the way, remove this line
# %pip install --quiet flaml"[autogen]"~=2.1.0

from the first code block of this notebook: https://github.com/microsoft/autogen/blob/main/notebook/agentchat_teaching.ipynb

I'll make that change.

This PR doesn't trigger OpenAI test. Do you know how to trigger it?

pull_request_target event runs in the context of the base repository of the pull request, rather than in the merge commit. This is designed specifically for the purpose of providing a safe execution environment for workflows that originate from forks.

That's why, you probably need to merge it to main so the rest of PR will be able to trigger that workflow
That actually reminds me that I forget to checkout to the merge commit to run the test. Can you not merge this PR until I add that checkout step?

pull_request_target event runs in the context of the base repository of the pull request, rather than in the merge commit. This is designed specifically for the purpose of providing a safe execution environment for workflows that originate from forks.

That's why, you probably need to merge it to main so the rest of PR will be able to trigger that workflow
That actually reminds me that I forget to checkout to the merge commit to run the test. Can you not merge this PR until I add that checkout step?

The PR should trigger the test based on the rules. Do you know why it doesn't trigger? I'd like to test it before merging.

@sonichi
The workflow with pull_request_target needs to be merged to the base repo for testing. It won't be triggered by a pull request.
This is because pull_request_target event runs in the context of the base repository of the pull request, rather than in the merge commit. For example, the base repository of this pull request is microsoft:main, and there's no code change detected. Therefore, the openai workflow won't be triggerd.
In order to run the test, we need to checkout to the merge commit so the change from pull request will be applied. I haven't add the step to checkout to merge commit in openai workflow yet, so please don't merge this PR

Could you make this a draft until it's ready?


[LLVM][RUNTIME] Make ORCJIT LLVM executor the default one

This PR propose to make ORCJIT the default LLVM runtime executor (engine).
Notes

Currently ORCJIT is a optional executor under llvm -jit=orcjit flag, introduced here.
Targets like RISCV, and possible others too, will not work on MCJIT see some discussions here .
As a live example, similar project Halide already use ORCJIT completley replacing the old MCJIT engine.

TVM disscussions and recent issues on the subject:

https://discuss.tvm.apache.org/t/run-tvm-software-stack-on-risc-v/17683/4
https://github.com/apache/tvm/issues/17508

Since ORCJIT was poorly exposed within TVM until now (being optional) any rising issues will need attention.

This is ready for review.
Cc: @masahi , @vinx13, @tqchen , @ekalda , @lhutton1 , @quic-sanirudh , @kparzysz-quic , @Lunderberg
Cc: @katebern-grovety , @PhilippvK, @JieGH

@cbalint13 can you dig a bit into windows issue? seems related to a missing symbol

@cbalint13 can you dig a bit into windows issue? seems related to a missing symbol

@tqchen , Yes I am on it, it will take a little bit of time.

@tqchen ,

The MSWIN __ImageBase issue is fixed using COFF workaround flags via explicit JIT linkage pass.
For track of record, the reference LLVM example was used here, where mentioned flags are showcased.

If this goes to main will help track & tackle issues as they arise, I hope for the best that there will be none at all.

@tvm-bot rerun


Push MDT key for new user on Coral Dev Board

Description
I have created a new user and I would like to be able to mdt shell to connect as that user. However when I try to push the key I get Unable to push keys to the device: Authentication failed. What should I be doing differently?
This is the process I am using so far:
mdt set username mendel
mdt set password mendel
mdt exec "sudo useradd -m newuser"
mdt exec "echo -e 'newpassword\nnewpassword' | sudo passwd newuser"
mdt exec "sudo adduser newuser sudo"
mdt exec "sudo cp -a /home/mendel/. /home/newuser/"
mdt exec "echo 'newuser ALL=(ALL:ALL) NOPASSWD: ALL' | sudo tee /etc/sudoers.d/newuser"
mdt exec "sudo rm /home/mendel/.ssh/authorized_keys"
mdt resetkeys <IP_ADDRESS>
mdt set username newuser
mdt set password newpassword
mdt pushkey

Click to expand!
Issue Type
Support
Operating System
Mendel Linux
Coral Device
Dev Board
Other Devices
No response
Programming Language
No response
Relevant Log Output
No response

@bsteiner-dandy Please try the below steps:
mendel@deft-goose:~$ sudo adduser test
Adding user `test' ...
Adding new group `test' (1004) ...
Adding new user `test' (1001) with group `test' ...
Creating home directory `/home/test' ...
Copying files from `/etc/skel' ...
New password:
Retype new password:
passwd: password updated successfully
Changing the user information for test
Enter the new value, or press ENTER for the default
        Full Name []:
        Room Number []:
        Work Phone []:
        Home Phone []:
        Other []:
Is the information correct? [Y/n] y

mendel@deft-goose:~$ sudo vim /etc/passwd
modify the test user line as : test:x:1000:1000:,,,:/home/mendel:/bin/bash
mendel@deft-goose:~$ sudo vim /etc/ssh/sshd_config
Add the belwo line at the end of /etc/ssh/sshd_config file:
AllowUsers test
mendel@deft-goose:~$ sudo reboot now
mdt set username test
mdt set password test
mdt shell

I tried this process, but in the mdt shell, I ran whoami and it reports "mendel"

Please try echo "$USER". The output of the whoami command is slightly different from $USER because whoami outputs the username that the user is working under, whereas $USER outputs the username that was used to log in
mendel@deft-goose:~$ echo "$USER"
test
mendel@deft-goose:~$ whoami
mendel

https://en.wikipedia.org/wiki/Whoami#:~:text=On Unix-like operating systems,was used to log in.

I see, thank you