fix ketch scenario

Tests for delivery/ketch fail and need to be updated.

What happened is that while ketch had been updated recently, it has not been updated for microservices. The current tests can be fixed if and after we land #142. Related: #79
Ideally we should just tackle #79, but at first glance it's not obvious how ketch handles multi-service apps. Docs here: https://learn.theketch.io/docs/application-management.


Updated details on maintenance updates schedule

Update documentation to explain the schedule of optional and mandatory updates.
slack thread here.

Thanks for reviewing this PR. I added this based on the scenario @fingon described and updated the rest of the details as per @staceysalamon-aiven 's suggestion.

Service quarterly patch updates are always mandatory with 6 months advance notice.

Content LGTM now but I'm sure @staceysalamon-aiven can come up with better way to say it :)


⚠ ClamAV has degraded performance

In 1558195, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 430 ms

Resolved: ClamAV performance has improved in 05e3b0a.


Topic/enhancement/topic updating

Enhancements include:

Creating resources and mapping them to topics
Updating topics (name, resource array)
Updating suneditor title, content

Save button is used to save topic changes (name, resource array). All other changes should save to BE once completed (ex. creating resources, updating suneditor title/content)

Make sure to pull main into this branch, there is a conflict in the topic view.


Fix broken migration link

PR checklist

[X] Addresses an existing issue: #2364
[ ] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update

Overview of change:
Fix broken link
Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:

thanks @irfanhudda!


fix crash in ios due to rapid tap on tab

Summary
This PR is related to this issue https://github.com/callstack/react-native-pager-view/issues/458. (react-native-view-pager) and https://github.com/satya164/react-native-tab-view/issues/1277 (react-native-tab-view).  IOS App Crash due to Rapid Tap on Tab Navigation. This issue seems due to racing conditions of animation. So in this PR I solve the issue by counting the tap action and only allowing new action when all existing actions are completed.
Test Plan
What's required for testing (prerequisites)?

https://github.com/satya164/react-native-tab-view

What are the steps to reproduce (after prerequisites)?

Create Tab Navigation (I this case I'm using https://github.com/satya164/react-native-tab-view)
Rapid tap on "Tab" Navigation.

Compatibility

OS
Implemented

iOS
✅

Checklist

✅ I have tested this on a device and a simulator

I tested this out with patch-package and it worked as expected thanks

Could this PR be merged?

Thanks, this PR works.

Please try this fix https://github.com/callstack/react-native-pager-view/releases/tag/v5.4.10

@RZulfikri can we close this MR, since the issue has been resolved ?


Add GCP check to ensure that GKE node service accounts are overridden

👋
Just adding a check to ensure that service_account is overridden in google_container_cluster and google_container_node_pool inside the node_config block.
Checks Added

GCP012: Checks if node_config in google_container_cluster or google_container_node_pool has a service account defined which is recommended by Google (see below)

Reference: https://cloud.google.com/kubernetes-engine/docs/how-to/hardening-your-cluster#use_least_privilege_sa

By default, nodes are given the Compute Engine default service account, which you can find by navigating to the IAM section of the Cloud Console. This account has broad access by default, making it useful to wide variety of applications, but it has more permissions than are required to run your Kubernetes Engine cluster. You should create and use a minimally privileged service account to run your GKE cluster instead of using the Compute Engine default service account.

Thanks for this @maaslalani  - It's prompted me to add a new helper function which is just going through then will finish review


nixos/pgbouncer: init

Description of changes
Hopefully a helpful and usable service for pgbouncer!
Will follow up with prometheus-pgbouncer-exporter upon merge.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@ofborg test pgbouncer

Thank you, tested.


allow for pvc to be added to multiple workbenches

https://issues.redhat.com/browse/RHOAIENG-12803
Description
This PR implements the functionality to allow a PVC to be added to multiple workbenches. The changes include:

Updating the UI to allow selection of multiple workbenches when adding or editing a PVC.
Modifying the logic to handle attaching a PVC to multiple workbenches.

I also updated some storage class logic and manifests
https://github.com/user-attachments/assets/a25b7f9a-7bef-4eb4-b5e2-b1531aba3805
How Has This Been Tested?

create two workbenches

one with its own new pvc
the second one should be able to use the previous pvc

go to cluster storage tab
create a new pvc and connect one workbench
save and re edit pvc and add another workbench

Test Impact

Cypress tests for adding a new PVC and attaching it to multiple workbenches

Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/retest

@andrewballantyne Should the bug you mentioned above be fixed in my ticket? RHOAIENG-7680 It's all about the validation of the storage path, I will post a PR later this afternoon.

/lgtm
As long as https://github.com/opendatahub-io/odh-dashboard/pull/3231#discussion_r1768950814 and https://github.com/opendatahub-io/odh-dashboard/pull/3231#discussion_r1768952289 are addressed next week, this PR is ready to go.
The bug mentioned in https://github.com/opendatahub-io/odh-dashboard/pull/3231#pullrequestreview-2318816652 will be fixed in https://github.com/opendatahub-io/odh-dashboard/pull/3236.

/approve


Add displayed code samples

Show me

Provide screenshots/animated gifs/videos if necessary.

 I want displayed code samples on the demo page for each example.

@jgerigmeyer Curious to how you see this coming along so far. I've added plain language notes for each (working) example along with relevant code. I hid the notes and code for the two examples that I think aren't working yet?
I think the intro paragraph could use another sentence or two explaining the demo? @sanajaved7 and @oluoluoxenfree if you have any feedback on the introduction/example code/and explanation notes that would be great too.

@dvdherron Looks good to me! I made a few minor wording edits, made one change to the positioning of the scroll example (so that the elements are visible initially without scrolling), and commented out the non-supported examples entirely (so that users aren't confused by them).
It'd be awesome if we could automatically pull the sample CSS from the actual CSS files, so we don't have to remember to keep the demos up-to-date if we change the CSS... But probably not a top priority for now. 👍

@jgerigmeyer Your changes look good and make sense. Good call on hiding the non-supported examples completely for now.
And I agree it would be ideal to have the CSS update automatically. Something we can look into later. 👍🏽

@sanajaved7 I prepped this for a v0.0.1 release, so once you review it I'll merge and do a new release.


Could not proxy react website to hyper-express API

I'm making a website with React and a API in hyper-express, but when trying to proxy the react requests to api, i receive this error in react console:
Proxy error: Could not proxy request /api/session from localhost:3000 to http://localhost/.
See https://nodejs.org/api/errors.html#errors_common_system_errors for more information (ECONNREFUSED).

Proxy error: Could not proxy request /public/assets/favicon.ico from localhost:3000 to http://localhost/.
See https://nodejs.org/api/errors.html#errors_common_system_errors for more information (ECONNREFUSED).

But when i do the same thing, but with express, the code runs without error, and making the requests normally... What can i do to run this with hyper-express?
This is my hyper-express API code (typescript)
import HyperExpress from 'hyper-express';
import { Routes } from './Routes';

const server = new HyperExpress.Server({ fast_abort: true, fast_buffers: true, trust_proxy: true });
const methods: Array<"get" | "use" | "post" | "delete" | "patch"> = ["get", "use", "post", "delete", "patch"];
const port = 80;

for (const route of Routes) {
    for (const method of methods) {
        if (route[method]) {
            if (method == "use") route.pattern ? server.use(route.pattern, route.use!) : server.use(route.use!);
            else server[method](route.pattern, route[method]!)
        }
    }
}

server.listen(port)
    .then(x => console.log(`[API] - I'm online on port ${port}`))
    .catch(e => console.log(`[API] - Error: I can't open on port ${port}`));

This is why I think the error is with hyper-express:
This is network debugger with Express:

This is network debugger with Hyper-Express:

OBS: When i try to access the api with Chrome, the api runs whitout problems, the problem is only to work with the proxy

This seems to be an implementation error in your implementation rather than Hyper-Express. I see that the session requests are sending back an HTTP 500. Check your logs for any errors from hyper-express or your own logic that may be sending the 500 error there? Please analyze the HTTP 500 errors and provide the error traces logged which are causing the HTTP 500 response codes.


Use today's date as the default for onetime's date

LGTM :)

Hey @skanehira , how are the releases created? Seems like this change is not released yet


Polyfill fromEntries for older versions of Node

First, thanks for such an awesome library! I love how this simplifies communicating with Figma.
I have a use-case at work where we are locked to an older version of Node and was curious if you would be willing to add an Object.fromEntries polyfill? I can add it to our cli tool, but figured it could be nice for anyone else on earlier versions. I can put together a PR if this sounds good, just let me know :)

hey @souporserious, sure I think that's fine! I'm happy for you to contribute, really appreciate the initiative!


Equinor Developer Conference

One would think that when clicking the plus-icon in the epic column that you would get an epic, but no…


Support for CINEMA 4D

Support for CINEMA 4D
Issue details
I use CINEMA 4D tool to create fbx in cm. In fbx-converter, the fbx can be show, but in the code—desktop, it is no exist or alpha. when I use Blender tool to create ,it can be show.
And my issue is why not support for CINEMA 4D!!!!
Reproduction steps/code
Version of LibGDX and/or relevant dependencies
last version
Stacktrace
//Please provide the stacktrace if applicable

Please select the affected platforms

[1 ] Android
[ ] iOS (robovm)
[ ] iOS (MOE)
[ ] HTML/GWT
[ ] Windows
[ ] Linux
[ ] MacOS

the version(1.9.9, now 1.9.10? but I think even update, it can not change):

the code I'm using to render the mode:

the fbxs(I try to use different size, you can open it to see ):
blue9.zip

@mbjgit i think your FBX files are too recent for fbx-conv, i made a PR that check file version. Current fbx-conv fail silently and output empty G3D files, i exported your files to G3DJ and it output an empty scene so i suspect your files are too recent. Current fbx-conv supports FBX 2014.2 and master supports FBX 2015.1.
If you can't choose a suitable format version in Cinema4D, a workaround is to re-export them from Blender.

Thank！But i already kown  that current fbx-conv supports FBX 2014.2 , and it can conv to g3db. finnaly, it can not show in java-desktop.
Now I re-conv C4D-fbx to Blender-fbx. you kown, it is rather time-consuming and the compatibility is also one problem.

Yeah i understand your point. However, i suggest you to create a feature request in fbx-conv issue tracker and close this issue since it's not a libgdx issue.

feature request already exists actually, see https://github.com/libgdx/fbx-conv/issues/114


Enable screen readers for visually impaired to read text

Use proper markdown strikethrough (~): “special” unicode characters often are not recognized by screen readers.

screen readers treat special characters differently to the standard
alphabet. This can cause issues as assistive technologies cannot always ‘read’ them. Sometimes they will skip them altogether.

https://business.scope.org.uk/article/accessibility-screen-readers-special-characters-and-unicode-symbols/

Hey, thanks a lot! Feel free to also suggest some other improvements that improve accessibility, also in the form of issues. Improving UX for impaired people is definitely worth some work :)
I'll try to deploy it soon alongside some other changes


["Problem"]

need to rehost make case studies app now that we dont have posit connect
https://rsconnect.biostat.jhsph.edu/MakeCaseStudies/

once we rehost we want to remove the note about it not being active


Feat: Make Contact Page Responsive

Describe the feature
Currently, Our Contact page is not yet responsive for smaller screens......I would like to add responsiveness to it
Add ScreenShots
https://github.com/mdazfar2/HelpOps-Hub/assets/115717746/875183ad-8637-4096-b12d-63cbeb4906d8
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSoC'24 contributor
[X] I want to work on this issue

@mdazfar2 Please assign me this issue


Feature request: Add ability to center the display in Zen Mode

VSCode Version: 1.18 Insiders
OS Version:  OSX 10.12.6 Sierra

Steps to reproduce: Activate Zen Mode - Cmd K + Z
At the moment, in order to achieve the desired visual effect, I have to use the split editor feature and open a dummy file. It would be really good if there was a configurable way of getting the Zen Mode option to center/left pad the display.

Duplicate of #15684


Replace deprecated tempdir crate with tempfile crate.

Solves #134

Sorry for being a bit impatient, I'll do the requested change myself so I can push out a bugfix release today!

Merged in d2b5614f1a4d91ed4c1c2135105b9f176cfe8759.


Abstractalph

Codecov Report

Merging #9 into bitindex-improvments will increase coverage by 0.09%.
The diff coverage is 75%.

@@                   Coverage Diff                    @@
##           bitindex-improvments       #9      +/-   ##
========================================================
+ Coverage                  58.4%   58.49%   +0.09%
========================================================
  Files                        56       56
  Lines                      3166     3173       +7
========================================================
+ Hits                       1849     1856       +7
  Misses                     1317     1317

Impacted Files
Coverage Δ

src/mutablesequences/hash.jl
29.85% <ø> (ø)
:arrow_up:

src/shortsequence/shortsequence.jl
57.14% <ø> (ø)
:arrow_up:

src/biosequences/predicates.jl
81.08% <ø> (ø)
:arrow_up:

src/mutablesequences/conversion.jl
50% <ø> (+25%)
:arrow_up:

src/iterators/eachskipmer.jl
0% <ø> (ø)
:arrow_up:

src/refseq/refseq.jl
67.14% <ø> (+0.94%)
:arrow_up:

src/mutablesequences/mutableseq.jl
34.61% <ø> (+4.61%)
:arrow_up:

src/biosequences/conversion.jl
33.33% <ø> (-66.67%)
:arrow_down:

src/biosequences/printing.jl
45.16% <0%> (-6.7%)
:arrow_down:

src/trait_definitions/alphabet.jl
62.71% <100%> (+0.64%)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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


Skeletal Mesh Doesn't Update at all

I followed the steps to enable the plugin in a basic project, added the construction script and enabled the IK Body Component, I also made sure that my Skeletal Mesh has the ABP_IKBody, however when trying it on Oculus Quest 2 I don't see the result intended, I followed the same tutorial you had in Blueprint and it used to work so I'm certain it's not the hardware, did I miss something in the steps?
NB: I don't use finger IK for this project.
I tested quickly a build on Windows and the Body inside the IKBody is valid, but not the case when using it in a VR headset.

Fixed:
For those who encountered the same problem, the plugin wasn't supporting Android for some reason, so in the UnrealBody.uplugin make sure this line is as follows:
"WhitelistPlatforms" : [ "Win64","Android" ]


Checker Workflow icon is not present on parent entry in the "My Workflows" page

Spinning out from: https://github.com/dockstore/dockstore/issues/4436, which has good details and discussion.
Describe the bug
When a parent entry links to a Checker workflow that a user does not own, the My Workflows page does not properly display any icons:

To Reproduce
Steps to reproduce the behavior (easiest done locally):

Get a copy of the database locally.
Find a workflow entry that you own
Update the row so that it links to a checker workflow that you don't own
View the parent workflow in the UI, and see no icon

Expected behavior
Perhaps a 'The checker workflow is private' if the user does not have permissions to view it? If the linked checker workflow is public it should be viewable from any screen.
This is a fairly niche case, so it probably isn't a priority.
Additional context
Webservice - 1.11.5
UI - 2.8
Compose Setup - 1.11
Deploy - 1.11.0
┆Issue is synchronized with this Jira Story
┆Issue Number: DOCK-1901
┆Sprint: Backlog
┆Issue Type: Story

Possible workaround, use Discover button


jQuery upgrade

Jquery is a javascript library for DOM traversal and manipulation, event handling, animation, and Ajax.
When text/javascript responses are received from cross-origin ajax requests not containing the option dataType, the result is executed in jQuery.globalEval potentially allowing an attacker to execute arbitrary code on the origin.

This project requires jQuery == 2.2.0, and unfortunately every version-2 jQuery lib is vulnerable.
Vulnerable: >=1.4.0 <=1.11.3 || >=1.12.4 <=2.2.4
Patched: >=3.0.0
I hope it won't be too hard to update the lib.
More information available here : https://nodesecurity.io/advisories/328

The project itself does not use cross-orign requests.

Absolutely agree with you.
I'll keep an eye on this issue.
Have a nice day


Webcam API to detect faces

Created an API which lets you upload an image from Swagger, turns on the webcam and looks for that face for a particular amount of time and determines if a person is found or not.

Thanks for sharing. This is a great example that will help people who want to do something like this. But it's a little more complicated than a basic library example and might be better as something you host in your own github repository so you can keep it updated and get feedback and stuff.

Sure.

I wouldn't mind adding a section at the bottom of the README called "Projects using this library" with a link to your project if you wanted to do something like that.


MicroProfile GraphQL 1.1 Specification Review

Specification issue template
When creating a specification project release review, create an issue in the MicroProfile-WG repository repo with the content defined as follows.

[x] Specification name and version
[x] Add the label Release Review
[x] Naming conventions for artifacts:

[x] Specification PDF in the form of microprofile-project-spec-version.pdf
[x] Specification HTML in the form of microprofile-project-spec-version.html
project is the microprofile specification short project name (config, health, ...)
version is the two digit x.y version of the specification

[x] The Nexus Staging links (orgeclipsemicroprofile-NNNN where NNNN is the staging repository id) (eg, https://oss.sonatype.org/content/repositories/orgeclipsemicroprofile-NNNN/org/eclipse/microprofile/config/) which contain all the binaries and relevant documentation:

[x] Staging Repo
[x] Spec PDF
[x] Spec HTML
[x] Javadoc
[x] API Jar
[x] TCK Jar

[x] A link in the Eclipse downloads sections is still available for specs and apidocs:

[x] https://download.eclipse.org/microprofile/staging/microprofile-graphql-1.1/
[x] An apidocs directory containing the javadoc associated with the API jar.

[x] Summary that a Compatible Implementation is complete, passes the TCK, and that the TCK includes sufficient coverage of the specification.

[x] [Compatible Certification Request] (https://github.com/eclipse/microprofile-graphql/issues/367)

[ ] Release Review

[x] Follow governance process and create/update release record

Component Release Record

[x] Email to PMC (more info here)
[x] Start release review by emailing EMO (more info here)(Track the bug issue created by EMO after the email)
[x] A member of Steering Committee initiates the ballot by sending an email to [microprofile-wg]<EMAIL_ADDRESS>(more info here
[ ] After the voting has ended, email [microprofile-wg]<EMAIL_ADDRESS>with the ballot result (more info here

We are missing the EMO issue.

I added it

Thanks!

Closing this one, since we were unable to recover all required file following the staging repository delete. The spec team may follow up with another release as they see fit. Thanks!


runtime: add a comment which briefly mentions where (or how) g_stackguard0 is defined

Am I missing something obvious? I attempted but failed to locate the definition of the symbol with grep and ack.
A brief comment will certainly save time for people who are trying to learn the runtime's internal.

It looks like it's just a local variable name.
/cc @ianlancetaylor

@bradfitz It looks like the symbol is generated by dumpasmhdr  (/cmd/compile/internal/gc/export.go), which is equivalent to offsetof(g, stackguard0).

You are asking where the assembler constant g_stackguard0 is defined?  As you say, it's defined in go_asm.h, which is a generated file.  It's not just g_stackguard0; it's all the TYPE_FIELD symbols used by the assembler files--g_sched, gobuf_sp, g_m, etc., etc.  I just checked, and go_asm.h defines 1634 constants.  I'm not sure how to make a comment for that that will help people looking for a specific symbol.

@ianlancetaylor @bradfitz Indeed, adding a comment does not help much as I thought (as I brought up the issue before I found out the answer). Maybe the assembler deserves a more capable language which won't need such kind of cleverness in the future.
Thanks for your responses and feel free to close the issue.

I don't think we could make the assembler more capable to eliminate the
auto-generated header unless we embed a copy of cmd/compile inside cmd/asm.

I guess https://golang.org/doc/asm could/should mention asmhdr? Would give
people a chance to figure out what's going on at least.

Yeah, we could document go_asm.h in doc/asm, but my concern is that it will
not help much as it's such a rarely used feature that I don't know any
package besides runtime uses it.


Only maintain the warehouse switch information in other version.
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[x] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:disappointed: fail : 7 / 12 (58.33%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/loadv2/LoadMgr.java
0
1
00.00%
[243]

:large_blue_circle:
com/starrocks/server/NodeMgr.java
0
1
00.00%
[437]

:large_blue_circle:
com/starrocks/backup/BackupJob.java
0
1
00.00%
[609]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadJob.java
3
5
60.00%
[1563, 1564]

:large_blue_circle:
com/starrocks/load/loadv2/InsertLoadJob.java
4
4
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 5 / 7 (71.43%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/service/FrontendServiceImpl.java
0
1
00.00%
[2142]

:large_blue_circle:
com/starrocks/backup/BackupJob.java
0
1
00.00%
[609]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadJob.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/loadv2/LoadMgr.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/server/NodeMgr.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
1
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 5 / 7 (71.43%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/service/FrontendServiceImpl.java
0
1
00.00%
[2142]

:large_blue_circle:
com/starrocks/backup/BackupJob.java
0
1
00.00%
[609]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadJob.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/loadv2/LoadMgr.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/server/NodeMgr.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
1
100.00%
[]


Default PostingsFormat lost the SPI extension point in the Codec class

Description
Lucene70Codec had:
private final PostingsFormat defaultFormat = PostingsFormat.forName("Lucene50");

In the Lucene80Codec PostingsFormat instantiation was moved to the constructor. Presumably to pass the additional fstLoadMode parameter?
https://github.com/apache/lucene/commit/28e8a30b536a39e5539ac6e8b9407d31724c8857#diff-3a74c1b72ab52e54dfcdc9de142b4331b372c11bcca87842b001c30f89ce58ebR98
In a subsequent commit the code was reverted to the default PostingsFormat constructor:
https://github.com/apache/lucene/commit/651f41e21bd3df98f70d2673295db29506e3d2e6#diff-3a74c1b72ab52e54dfcdc9de142b4331b372c11bcca87842b001c30f89ce58ebR96
But the SPI call PostingsFormat.forName() was not restored. And is still missing in Lucene95Codec:
https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/codecs/lucene95/Lucene95Codec.java#L121
Restore the SPI extension point to allow overriding the PostingsFormat without having to override the Codec?

Hi,
the SPI should only be used when READING indexes. When you create a codec for IndexWriter the codec version hardcodes its postings formats and other subtypes. As you see it is the same for docValuesFormat and other parts.
This allows to read any index, but when you write an index it will use the exact codec as specified.
Basically we made the decission to hardcode the correct classes when writing indexes, but load the codecs dynamically when open an existing index.

I see. Thank you for the explanation.
The commits that change this behavior did not say anything about this.
So it looked like this SPI extension point loss was an unwitting side-effect of a sequence of refactorings.

So it looked like this SPI extension point loss was an unwitting side-effect of a sequence of refactorings.

No, the SPI fromName does not allow you to change the implementation, as there can only be one "Lucene50" implementation on classpath. If you want to have another codec it must have a new name and therefore for (new) indexes passed via IndexWriterConfig using a new codec.

private final PostingsFormat defaultFormat = PostingsFormat.forName("Lucene50");

This does not allow you to overwrite the format, you still need to subclass codec, as the name "Lucene50" is part of Lucene core and can't be replaced. So it will always load the hardcoded 5.0 codec.

For the FST loading mode mentioned above, the codec does not need to be changed, you can tell DirectoryReader to use FST load modes using attributes.

The general rule is: If you want to change the index postings format (but nothing else like codec itsself) when writing a new index, you need to subclass default codec. By that it keeps its name and code reading the index will look it up. If you add a completely new postings format, subclass abstract base class, give it a new name and register it in SPI. If you just want to change settings you can reuse the postings format by instantiating it in the codec with different settings.

The codec classes are final.
Besides having two implementations (Lucene84PostingsFormat in lucene-core and MyLucene84PostingsFormat) with the same "Lucene84" name will likely result in a lookup error?

The postings format classes are final. Besides having two implementations (Lucene84PostingsFormat in lucene-core and MyLucene84PostingsFormat) with the same "Lucene84" name will likely result in a lookup error?

Exactly and because of that its final. Your postings format needs a new name.
If you want to use it as default you can subclass the Codec as this is the main entry point: https://github.com/apache/lucene/blob/df8745e59ee65f276ccaefa87480e1fd85facb56/lucene/core/src/java/org/apache/lucene/codecs/lucene95/Lucene95Codec.java#L55
I don't understand what your problem is:

Write your own postings format with its own name. You can't subclass. But you can use a FilterPostingsFormat to wrap the default postings format
Subclass https://github.com/apache/lucene/blob/df8745e59ee65f276ccaefa87480e1fd85facb56/lucene/core/src/java/org/apache/lucene/codecs/lucene95/Lucene95Codec.java#L55 and return/inject your codec there, an alternative which is not so statically depending on the exact version is to use new FilterCodec(Codec.getDefault()) { ... override method returning postingsFormat()... }

Besides having two implementations ... with the same "Lucene84" name will likely result in a lookup error?
Exactly and because of that its final.

I just do not understand then how your suggestion to "subclass default codec. By that it keeps its name and code reading the index will look it up" would work?
Having the "official" Lucene95Codec and MyCustomCodec sharing the same "Lucene95" name will result in lookup error, won't it?
So I have to give my custom codec a new name. And then "plug" it in using some configuration property, I suppose?
Or find a way to ensure "my" META-INF/services appears on the classpath before lucene-core.jar


Readme should point you to the weekly all-hands calls

The Readme in this repo should prominently display info about how to find, propose agenda items & participate in the weekly calls.

👍 +1:

This was done on https://github.com/ipfs/pm/pull/136/files by @flyingzumwalt


add: arquivos do projeto css

https://br.pinterest.com/pin/738308932675929156/

1 ano depois…
kkkkkk
Em ter., 21 de fev. de 2023 às 10:21, Lay @.***>
escreveu:

Closed #17 https://github.com/Layssaa/turing-desafios-css/pull/17.
—
Reply to this email directly, view it on GitHub
https://github.com/Layssaa/turing-desafios-css/pull/17#event-8570933208,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHQXTFIYOZRZB2GJ6WK5IJTWYS6PNANCNFSM5WPLHKHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Table View of [PSCustomObject]

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
This issue is a continuation of #19107 and #18492, which are closed as resolved. But this is not true.
The pscustomobject table representation of non-integer numbers rounds them up or down, regardless of the user's preference.
Values are rounded to two or three decimal places, depending on the System.Globalization.CultureInfo.CurrentCulture used.
For example user want to see five digits after decimal separator of property one and one digit of property two:
[pscustomobject] @{

     one = [math]::Round(0.1234567,5)
     two = [math]::Round(1.1234567,1)
     }

 one  two
 ---  ---
0.12 1.10

As you can see, the result was shocking. The user can format the output himself (as a string), but this effort looks strange.
Expected behavior
- The output values should match the user's preferences.
- The user does not have to make additional efforts to obtain the expected result.

Actual behavior
The user is forced to change the type of the resulting values and reformat their output to obtain the desired result.

Error details
It's not a bug. This is a change in the behavior of the classic table view.

Environment data
Name                           Value
----                           -----
PSVersion                      7.3.2
PSEdition                      Core
GitCommitId                    7.3.2
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Please just close this one as well.   @237dmitry  Enough. Please, your wasting your own time and that of normally patient people

It is working as designed.
Format table should not be making exceptions for a single type if round is being used to format strings. If the object is commonly used it should be labelled with a type and given proper formatting, if it is a 1 off, format what you want to display as a string.
(1/2), (2/3), (3/4) , (4/5), (5/6)| % {    [pscustomobject][ordered]@{"Scale"=$_; "Diameter"=($_*144); "Circumference"=($_*144*[math]::pi) }}  should not produce a mess like

            Scale Diameter    Circumference
            ----- --------    -------------
              0.5       72<PHONE_NUMBER>58465
0.666666666666667       96<PHONE_NUMBER>4462
             0.75<PHONE_NUMBER>06587698
              0.8    115.2<PHONE_NUMBER>93544
0.833333333333333<PHONE_NUMBER>18430775

It is working as designed.

No, not working. See my example. And this is yours, no difference, the mess of trailing zeroes:
PS > & {
     (1/2), (2/3), (3/4) , (4/5), (5/6)| % {
     [pscustomobject][ordered] @{
                "Scale"=$_
                "Diameter"=($_*144)
                "Circumference"=($_*144*[math]::pi)
     }}}

Scale Diameter Circumference
----- -------- -------------
0.500   72.000       226.195
0.667   96.000       301.593
0.750  108.000       339.292
0.800  115.200       361.911
0.833  120.000       376.991

User have to format to strings:
PS > & {

   $res =
   (1/2), (2/3), (3/4) , (4/5), (5/6) | % {
   [pscustomobject][ordered] @{
              "Scale"= [string] [math]::Round($_,2)
              "Diameter"= [string] [math]::Round(($_*144),2)
              "Circumference"= [string] [math]::Round(($_*144*[math]::pi),2)
   }}

       $res | ft @{ e='Scale'; a='right' },@{ e='Diameter'; a='right' },@{ e='Circumference'; a='right' }
   }

Scale Diameter Circumference
----- -------- -------------
  0.5       72        226.19
 0.67       96        301.59
 0.75      108        339.29
  0.8    115.2        361.91
 0.83      120        376.99

And now suppose that the user needs the table only as an option, for example, so that he can receive only one parameter, moreover, of a given type, which means that, in addition to the table, he will need to organize branching in the script. But, sorry, Powershell is a scripting language, which means it should be as efficient as possible for writing scripts in “three lines”.

Just about anywhere in the world, the decimal points, 1s, 10s and 100s are expected to line up. What you have shown is unacceptable.
As I said before, stop using round() as a string formatter, then you'll be fine.

stop using round() as a string formatter

I want the user to get the result he expects.
Difference:

I also want a user to get what he or she expects that decimal points line up, and multiple rows have the same number of decimals.
Single and Double always store the same number of bytes for the mantissa and exponent in your screen show round is converting 1.11111100000000 to 1.11110000000000  (14 places of decimals)
Your left screen shot is showing 1.11110000000000  correctly.
Your example of what you expect only works if you know the values going it,  tell me  what this will produce
 [pscustomobject] @{ "aa" = [math]::round($a,4) ; "bb" = [math]::round($b,5)}
Go on. How many decimal places be shown in columns  aa and bb display ?    (Tip try [math]::round($null,2))
If you type  "11.1111" and "22.22222" and want to print  "11.1111" and "22.22222" put quotes round them :-)  If you don't know what the number will be, you don't
In my example about if $a is 5.1 it doesn't show to 4 places.  As long use cite round, I'll keep repeating  that round() is not a string formatter as asking to round 5.10000000000000  shows.
The smart thing to request would be NOT applying this formatting to the decimal type.
1.10d  is stored as a different number from 1.1d.
The whole column is NOT formatted the same by format-table if one psobject contains an INT and one contains a float, but formatting all floats the same is the least wrong way to deal with things - it is not perfect, "N" format is is a poor choice for numbers bigger than say 10^20 or smaller than 10^-2
Not formatting decimals they are already preformatted (but numbers not strings) would seem like a valid thing to do.

If you type "11.1111" and "22.22222" and want to print "11.1111" and "22.22222" put quotes round them :-) If you don't know what the number will be, you don't

Again. One more time. Especially for you. And for them who are 20 years in a bomb shelter. I WANT AS IS, as the user decides, and should be output to the console. Is it really incomprehensible to you, or maybe you are interested in something, maybe you are paid extra for closed issues? I do not understand YOUR personal interest.
For an ordinary person, my request is absolutely obvious and understandable. I just want to get the result and not look for workarounds to implement it. I think this is a normal human desire. But you did everything to make life difficult.

I'm not paid anything, I left Microsoft in November 2010.  And learn some manners, or at some point you'll get banned from commenting here.  You've ignored everything you have been told - including how to frame a request for a change which could work - and now are just being obtuse and rude.  There's an old saying "Don't try to teach a pig to sing - it wastes your time and annoys the pig". Well my time is being wasted, you're getting annoyed, so I'll not to respond to you in future.
What you say about "as the user decides" is wrong, and your use of Round() for string formatting is also wrong   [pscustomobject] @{ "aa" = [math]::round($a,4) ; "bb" = [math]::round($b,5)}
Does not output 4 decimal places if $a is e.g, 3.1  If I want two decimal places and with aligned decimal points and do it your way, what do I get in PS5 ?
 > [pscustomobject] @{ "aa" = 1.000 ; "bb" = 2.300};    [pscustomobject] @{ "aa" = 3.450; "bb" = 4.567}

  aa    bb
  --    --
   1   2.3
3.45 4.567

It's amazing we had rows of output NOT aligning the columns of numbers for so long.
Your argument is because you typed the value of $a as 1.23 it should be printed as 1.23;  it doesn't work if you type 1.00 though, does it ?  Round doesn't do that, does it.?  If the number is  not something you typed and it might internally be 1.0000000000, 1.2000000000, or 1.2300000000, you're saying that users  want
Row     Value
-----    ------
1             1
2           1.2
3          1.23

Not only do they not want that, dozens and dozens of types through the history of PowerShell have had to create format files just to apply the "N" formatting.  .NET does have not have ability excel has to say "Display a digit but NOT a trailing 0", so you can't align the digits in 1, 1.2 and 1.23 without the trailing 0s.
I'm sure you'll want the last word. I'm going to try not to be goaded into replying.

My suggestion is not to find a workaround, but to find a classic table view. This is important for backwards compatibility and common sense.
And yet, if you do not want to answer me, then I will survive it.

@237dmitry I understand you're frustrated, but please keep the discussion professional.  I think you'll find it easier to convince others if they don't feel like they are being attacked (and the same would go for others as well).
Formatting changes are NOT consider breaking changes as the original values are not being modified, just how they are displayed on screen.  If you know the shape of the object, you can already define your own custom formatting however you'd like, so we're only discussing the default formatting.
Using CurrentCulture to affect decimal formatting makes sense as users would expect that even within the console as that is the purpose of that configuration.
I guess the question is whether it makes sense to consider a way to either suppress using CurrentCulture or apply decimal formatting "globally" in a table rather than require defining it for each property.  Unless there's a larger part of the user base that wants such a feature, I don't know if it's worth building it when we could be fixing other issues.

but please keep the discussion professional.

I'm already tired, how long can I endure? Everyone is looking for some workarounds to solve the problem. But I myself know perfectly well how to get around them. My request is not about workarounds, but about the very essence of the problem.

Formatting changes are NOT consider breaking changes as the original values are not being modified, just how they are displayed on screen.

This is exactly what I'm trying to get you to change this behavior, revert everything back to how it was up until version 7.3+. If I'm not mistaken in the versions.

so we're only discussing the default formatting.

So you are suggesting that everyone rewrite their scripts so that their work is no different? An excellent approach, you just warn in advance, otherwise you get situations similar to $PSStyle.OutputRendering. Who here said that tables are only for the console? Hello?
Please answer this question -- why are the displayed values in a tabular form rounded both up and down? Why do you do experiments like this on people?

Can someone detail when/where this was changed? I can't seem to find the responsible PR...

Can someone detail when/where this was changed?

I think from 7.3.1, maybe early.

The PR was https://github.com/PowerShell/PowerShell/pull/17650, to address the issue #17544.
conceptually, I think it makes sense to adhere to the decimal formatting configured in the current culture.


HDDS-8436. Support setSafeMode(), isFileClosed() FileSystem API

What changes were proposed in this pull request?
Implement new Hadoop file system APIs to support HBase.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8436 Support setSafeMode(), isFileClosed() FileSystem API
How was this patch tested?
Unit tests

BTW, is it possible to move the setSafeMode method to Basic RootedOzoneFileSystem?

We can't, because Hadoop 2 does not have SafeMode interface (introduced in Hadoop 3.3.6). So I can only put it where Ozone is not run with Hadoop 2 runtime.

But as it turns out, we test MR jobs with Hadoop 3.1 3.2 and 3.3 runtime, and the MR job tests fail with Hadoop 3.1 and 3.2 (due to the lacking of SafeMode and LeaseRecoverable interface). So either we find a workaround or remove the Hadoop 3.1 3.2 tests.


Use getConfigInMultipleLangs() instead of getInt() as it's deprecated

Questions
Answers

Description?
As the getInt() method of Configuration is deprecated and could be removed into next major version of PrestaShop, we update this method. (Cfr. https://github.com/PrestaShop/PrestaShop/pull/26306)

Type?
improvement

BC breaks?
no

Deprecations?
no

How to test?
Nothing really to test.

Hello, it seems this PR is now stale.
I close this PR, following our review guidelines "Pull Requests may be closed after 30 days of inactivity".
If however you are still interested in it, please ping us 😉  we can reopen.
Thank you for your contribution.


[WIP] upgrade db to postgres 15.1 + postgis 3.3.2

work in progress, don't merge

@OriHoch what's missing here in order to merge?

need to upgrade the DB before merging


Do not assume dictionaries exists in footer

This is my first Rust PR, please let me know if I could do this in a more idiomatic way!
Which issue does this PR close?
Closes #1335, on the arrow-rs side at least, Julia's Arrow.jl also writes mismatched versions (apache/arrow-julia#320).
What changes are included in this PR?
IPC reader no longer assumes footer exists.
Are there any user-facing changes?
No.

There are some format error.

Codecov Report

Merging #1631 (86cf000) into master (7d00e3c) will increase coverage by 0.00%.
The diff coverage is 73.07%.

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

what do you mean by format errors?

You can run cargo fmt locally to check the format. There are also some clippy error. You can click the CI check links to see the details too.

thank you, should be good now!

❤


repo.yml: Add balena-yocto-scripts upstream source

This allows to add nested changelogs from the balena-yocto-scripts repository
from version >= 1.14.1
Change-type: patch
Changelog-entry: Add balena-yocto-scripts upstream source
Signed-off-by: Alex Gonzalez<EMAIL_ADDRESS>

Looks good to me, though it appears I don't have write access to this repository (or at least the ability to review PRs).

@balena-ci rebase

@resin-jenkins retest this please

@balena-ci retest

@resin-jenkins retest this please

@balena-ci rebase


Implement FromWasmAbi for Result<i32|u32, JsValue>

It looks like there's no FromWasmAbi implementation for Result<u32, JsValue> and Result<i32, JsValue>
The feature is required by #1379, since there are return types like Result<i32, JsValue> in Atomics
https://github.com/rustwasm/wasm-bindgen/blob/master/crates/js-sys/src/lib.rs#L528

It's intended that Result can be returned from functions but for FromWasmAbi that typically means it's an argument I think? Are you trying to take a Result as an argument? Or is there a different situation where the requirement for Result: FromWasmAbi is coming up?

Here is an example
pub mod Atomics {
    use super::*;

    #[wasm_bindgen]
    extern "C" {
        #[wasm_bindgen(js_namespace = Atomics, catch)]
        pub fn add(typed_array: &JsValue, index: u32, value: i32) -> Result<JsValue, JsValue>;

When I run wasm-pack test --headless --safari in crates/futures folder, it fails with an error
error[E0277]: the trait bound `std::result::Result<wasm_bindgen::JsValue, wasm_bindgen::JsValue>: wasm_bindgen::convert::traits::FromWasmAbi` is not satisfied
   --> crates/js-sys/src/lib.rs:522:5
    |
522 |     #[wasm_bindgen]
    |     ^^^^^^^^^^^^^^^ the trait `wasm_bindgen::convert::traits::FromWasmAbi` is not implemented for `std::result::Result<wasm_bindgen::JsValue, wasm_bindgen::JsValue>`

The error points here https://github.com/rustwasm/wasm-bindgen/blob/master/crates/js-sys/src/lib.rs#L519
Originally the return type was Result<i32, JsValue>, but changing i32 to JsValue didn't help. I wonder, what could cause the error

Sorry I think I still may not be quite following, could you perhaps gist a standalone example that exhibits the error?

Found it! I was confused by error message.
use wasm_bindgen::prelude::*;

pub mod Namespace {
    use super::*;

    #[wasm_bindgen]
    extern "C" {
        #[wasm_bindgen(js_namespace = Namespace)]
        pub fn example() -> Result<JsValue, JsValue>;
    }
}

If we try to run wasm-pack build, the error will be
error[E0277]: the trait bound `std::result::Result<wasm_bindgen::JsValue, wasm_bindgen::JsValue>: wasm_bindgen::convert::traits::FromWasmAbi` is not satisfied
 --> src/lib.rs:6:5
  |
6 |     #[wasm_bindgen]
  |     ^^^^^^^^^^^^^^^ the trait `wasm_bindgen::convert::traits::FromWasmAbi` is not implemented for `std::result::Result<wasm_bindgen::JsValue, wasm_bindgen::JsValue>

The mistake in code is missing catch here:
#[wasm_bindgen(js_namespace = Namespace)]

and the following code compiles:
#[wasm_bindgen(js_namespace = Namespace, catch)]

Hm I'm unfortunately still having difficulty reproducing this and I don't know why, @ibaryshnikov can you perhaps send a PR adding a test to crates/macro/ui-tests/*.rs which produces the desired error?

@alexcrichton I tried to add the code to crates/macro/ui-tests/ but it doesn't trigger an error
I created a repo with minimal example, you can checkout it and run wasm-pack build to see the error
https://github.com/ibaryshnikov/fromwasmabi


Request: Adding an option so magic prompt and get lucky does not affect negative prompts

I think it would be useful if there was an option so magic prompt, get lucky and attention grabber would not add any term or modify the user negative prompt, because now some positive terms like "trending on artstation" or "artgerm" are regularly added usually lowering the quality of the end result picture. Thank you.

one thing you can do is set "ignore negative prompts" in advanced settings. Not a perfect solution if you are still planning on using your own prompts in the negative prompts box.

Thank you for your response, I am doing that now and that is, in part, the reason i posted the bug #167 that you fixed earlier, in fact i misunderstood the tool tip text and though that "Disable negative prompt" functionality was what i am proposing here. I know it is not something critical, even so, I still think that it could be a nice feature to have.

Is there an option to enable magic prompt/get lucky/attention grabber for only the positive prompt? Ideally I would like to allow them to alter the positive prompt, while leaving my negative prompt alone.

Not right now.

I've pushed a new version (v2.3.4) that addresses this.

Thank you for looking into it. I have tested this new version and it worked like a charm.


Gem does not support Ruby 3.0

General information:
This gem has a dependency of paypal-checkout-sdk which has a dependency of paypal-http, both of these gems do not support Ruby 3.0 yet. This is causes issues as seen here: https://github.com/solidusio-contrib/solidus_paypal_commerce_platform/issues/132
Example:
The above issue highlights that the incompatiblity causes issues with our execute method, which is a core method in the gem. This trickles down to URI.escape method used in paypal-http.
This has been "deprecated" for 10 years and removed in Ruby 3.0
paypal-http has an issue for the Ruby 3.0 incompatibility : https://github.com/paypal/paypalhttp_ruby/issues/12
The mentioned dependencies might not be the only dependencies that does not support Ruby 3.0, so this should be investigated further.
Proposed solution:
As soon as the PayPal dependencies are compatible with Ruby 3.0 we should attempt to make the gem compatible with it as well.
Temporary solution:
The required_ruby_version in the gemspec  has been rollbacked to pre 3.0 to prevent developers having the same issue.

Why is this not fixed after almost a year, and why does bundler not recognize the conflict for me? My Gemfile states:
ruby '>= 3.0'
gem 'solidus_paypal_commerce_platform'

And it installs without complaint... What do I do now, I want to start using the store in a few days?

apparently the upstream issue is fixed, so maybe this can be resolved by updating dependencies?

Can this be closed now @cpfergus1 and @elia?
https://github.com/solidusio/solidus_paypal_commerce_platform/pull/161

@RyanofWoods for sure, master is testing against ruby 2.7, 3.0, and 3.1 👍👍👍


NuGet Version 2.12?

MonoDevelop 5.10
openSuse Tumbleweed Lastest Version & Updates
Hi,
Trying to add the main Rx package, but I need the NuGet 2.12 version:

Adding System.Reactive...
The 'System.Reactive 3.0.0' package requires NuGet client version '2.12' or above, but the current NuGet version is '<IP_ADDRESS>'.

Is there an update?

Not for MonoDevelop 5.10. MonoDevelop 6.0.2 supports NuGet 2.12.
For MonoDevelop 5.10 you may be able to just replace the NuGet.Core.dll and the Microsoft.Web.Transform.Xml.dll that ships with MonoDevelop 5.10 with the NuGet 2.12 versions available from GitHub. Wherever MonoDevelop is installed on your machine the NuGet addin is installed in the AddIns/MonoDevelop.PackageManagement subdirectory. I would take a backup of the existing .dll files before replacing them with the new versions.

Thanks for the reply!  Okay, I will make the changes and by the way, what is the best distro for MonoDevelop.  I'm trying to create a software development OS, so I don't have to wait for an update.

I do not have any recommendations for the best distro to use for MonoDevelop. I have used both OpenSuse and Ubuntu.

How can I update to the current version without installing Windows as a VM?

For MonoDevelop v6 you will either have to compile from source or you can use FlatPak which is currently available as a preview.

Okay, yea, I saw that, so I'll give it try...  Thanks!...


Adafruit Metro M0 Express onboard neopixel shines brilliant while running program.

First, check if this repo is the right one to file the bug report.
micro:bit

Issue tracker Microsoft/pxt-microbit.
Have a question? Check out the micro:bit Support Forums.

Adafruit - Circuit Playground Express

Issue tracker Microsoft/pxt-adafruit.
Have a question? Check out the Adafruit Support Forums.

Minecraft

Running into issues with Code Connection, MakeCode for Minecraft, or Minecraft Education Edition? Post your question on Minecraft EE Tech Support.
For help on setting up MakeCode and Minecraft, follow the setup guide.
File bug report in this repo.

Chibitronics - Chibi Chip

Issue tracker Microsoft/pxt-chibitronics.

I'm a developer!

Have a question? Ask it on stack overflow, we monitor the makecode tag.
Want to speak directly to the team? Ask your question on

Bug Template
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
Add screenshots to help explain your problem. You can copy paste the screenshot in the github report. The .gif screen recording is very useful as well.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

After upload of program to Metro M0 Express, when program does not include any mention of control of on-board neopixel (green- the one that shines when re-set button is pressed prior to flashing code) then glows very brilliant green while code is running for the duration of program.

Please open bug in github.com/microsoft/pxt-maker

did yesterday after Peli's recommendation.


Add engine_adapter_callback

Description

Adds engine_adapter_callback that is executed after the SQLAlchemy engine is constructed
This allows developers to apply any arbitrary statements before subsequent queries are issued
In particular, I want to be able to run SET workload = "olap"; to connect to a Planetscale DB without bumping into 100k rows returned system limits.

Related Issues

Fixes #...
Closes #...
Resolves #...

Additional Context

@umar-ahmed sql_database and sql_table accept Engine instance in credentials. So you can create your own, run anything on it and then pass it to dlt. OFC having this callback is more convenient because you do not need to handle credentials yourself. if you plan to continue this PR we will also need a simple test where you verify that adapter receives engine instance


Update servingruntime ootb configuration

#340 for 1.27

My bad, this is not targeted for 1.27, I'm sorry for the confusion


[SPARK-23942][PYTHON][SQL] Makes collect in PySpark as action for a query executor listener

What changes were proposed in this pull request?
This PR proposes to add collect to  a query executor as an action.
Seems collect / collect with Arrow are not recognised via QueryExecutionListener as an action. For example, if we have a custom listener as below:
package org.apache.spark.sql

import org.apache.spark.internal.Logging
import org.apache.spark.sql.execution.QueryExecution
import org.apache.spark.sql.util.QueryExecutionListener

class TestQueryExecutionListener extends QueryExecutionListener with Logging {
  override def onSuccess(funcName: String, qe: QueryExecution, durationNs: Long): Unit = {
    logError("Look at me! I'm 'onSuccess'")
  }

  override def onFailure(funcName: String, qe: QueryExecution, exception: Exception): Unit = { }
}

and set spark.sql.queryExecutionListeners to org.apache.spark.sql.TestQueryExecutionListener
Other operations in PySpark or Scala side seems fine:
>>> sql("SELECT * FROM range(1)").show()

18/04/09 17:02:04 ERROR TestQueryExecutionListener: Look at me! I'm 'onSuccess'
+---+
| id|
+---+
|  0|
+---+

scala> sql("SELECT * FROM range(1)").collect()

18/04/09 16:58:41 ERROR TestQueryExecutionListener: Look at me! I'm 'onSuccess'
res1: Array[org.apache.spark.sql.Row] = Array([0])

but ..
 I'm 'onSuccess'
[Row(id=0)]

>>> spark.conf.set("spark.sql.execution.arrow.enabled", "true")
>>> sql("SELECT * FROM range(1)").toPandas()

18/04/09 17:53:26 ERROR TestQueryExecutionListener: Look at me! I'm 'onSuccess'
   id
0   0

How was this patch tested?
I have manually tested as described above and unit test was added.

Test build #89053 has started for PR 21007 at commit db1987f.

Test build #89053 has finished for PR 21007 at commit db1987f.

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

Merged build finished. Test PASSed.

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

cc @cloud-fan and @viirya (from checking the history).

@BryanCutler @ueshin
add test in https://github.com/apache/spark/blob/master/python/pyspark/tests.py?

Yup, will add. I was just hesitant because It needs some complexity for testing out (as described in the PR description) whereas the fix could be quite straightforward.

got it. sorry I missed the last sentence. maybe a jvm only test?

Will give a shot first for the Python one to show how it looks like. I have an incomplete one in my local.

Will address the comments together soon.

Test build #89108 has started for PR 21007 at commit edb5eea.

Test build #89109 has finished for PR 21007 at commit e865c88.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #89127 has started for PR 21007 at commit e865c88.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2156/
Test PASSed.

Test build #89108 has finished for PR 21007 at commit edb5eea.

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

Merged build finished. Test PASSed.

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

Test build #89127 has finished for PR 21007 at commit e865c88.

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

Merged build finished. Test PASSed.

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

will address the comments soon.

Test build #89163 has started for PR 21007 at commit deacb17.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2184/
Test PASSed.

Merged build finished. Test FAILed.

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

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2193/
Test PASSed.

Merged build finished. Test PASSed.

Test build #89173 has finished for PR 21007 at commit deacb17.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SQLTestUtils(object):
class ReusedSQLTestCase(ReusedPySparkTestCase, SQLTestUtils):
class QueryExecutionListenerTests(unittest.TestCase, SQLTestUtils):

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2200/
Test PASSed.

Test build #89182 has started for PR 21007 at commit deacb17.

Test build #89182 has finished for PR 21007 at commit deacb17.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SQLTestUtils(object):
class ReusedSQLTestCase(ReusedPySparkTestCase, SQLTestUtils):
class QueryExecutionListenerTests(unittest.TestCase, SQLTestUtils):

Merged build finished. Test PASSed.

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

Test build #89193 has started for PR 21007 at commit 1a52dbe.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2208/
Test PASSed.

Merged build finished. Test PASSed.

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

Test build #89267 has started for PR 21007 at commit 7c1b3c6.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2265/
Test PASSed.

Test build #89267 has finished for PR 21007 at commit 7c1b3c6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #89279 has started for PR 21007 at commit 7c1b3c6.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2275/
Test PASSed.

Test build #89279 has finished for PR 21007 at commit 7c1b3c6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #89290 has started for PR 21007 at commit 7c1b3c6.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2283/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2285/
Test PASSed.

Test build #89290 has finished for PR 21007 at commit 7c1b3c6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #89304 has started for PR 21007 at commit 7c1b3c6.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2289/
Test PASSed.

Merged to master.
Thanks for reviewing this @felixcheung, @viirya and @BryanCutler.

@HyukjinKwon @BryanCutler @viirya @felixcheung The first sentence of this PR really scares me. After reading the PR description. Since the PR description will be part of our change log. Please be careful to ensure they are right.

What's wrong in the description and PR title, and what to document? Do you mean the first sentence This PR proposes to add collect to a query executor as an action. is wrong because this sentence doesn't mention Pyspark specifically? I think the PR title and other words explain what this PR changes.
It's already documented - https://github.com/apache/spark/blob/bd4eb9ce57da7bacff69d9ed958c94f349b7e6fb/sql/core/src/main/scala/org/apache/spark/sql/util/QueryExecutionListener.scala#L44
Should we document collect in PySpark is now recignised as a query execution action in a query execution listener?

@gatorsmile I think the PR description here is great and very detailed, what exactly is wrong and scary?


Move content to CMS

The content should be edited in a cloud CMS or similar. Currently everything is hard-coded.

We could use Contentful in the beginning.
We would at least need the following content types:

Page (rich text with images, lists, etc + main image for the page
Organizers (photo, bio, Twitter, GitHub, LinkedIn, etc)
Sponsors (name, website, Twitter, Facebook, Twitter, LinkedIn, etc)
Posts (for Frontpage & archive)


STL-1490 | Added react component for session note step

Short Description

Short description of this pull request.
Provide any additional context as needed and explain reasons behind your implementation choices.
Add any links to tickets, specs, documentation, external resources, etc. that may be relevant.

Changes

What changes does this pull request introduce?
Are there any additional changes (e. g. refactoring) that are part of this pull request?

Feedback

Which parts of the code would you like feedback on?
Questions that are open are also good (e. g. "Do you see any tests missing for class x?")

How to review this Pull Request
Link to Confluence (internal)

I forgot! We could actually remove the template basis/display_standard_with_list.html since it was only being used to the session note component, right?

I would resolve the minor merge conflicts after PR has been reviewed and approved.

I forgot! We could actually remove the template basis/display_standard_with_list.html since it was only being used to the session note component, right?

Should be okay :+1:


Add output parameter with packages updated and release notes

First, thank you very much for creating this super useful action!!
Maybe I'm biased because I saw first https://github.com/marketplace/actions/swift-package-dependencies-check, but I think it would be super useful being able to get access to updated packages release notes in the PR description. So, before merging, you can check if you really want to merge those package updates or not.
It's a nice to have, but really nice 😉

Hi Carlos, I can only apologise for ignoring this issue for so long - I've not been watching this repository on GitHub and so wasn't notified on new comments. I appreciate the kind words all the same.
The ability to get release notes is something which requires a fair bit of extra logic even in the package provided above (in fact, it's a whole bunch of extra dependencies!). If it supported Xcode projects then I could be tempted to add it but at this time I feel it would add too much to the project to support.
Having said that, I am looking to work on https://renovatebot.com/ which is a fantastic tool much like Dependabot for updating dependencies automatically with far more functionality than this kind of action can supply. As part of that integration I'd love to ensure it has full support for release notes 🙂
I'll keep this issue open as I think it's a super valuable idea, but for this action I think it would be a little too much work at this time. Hope that makes sense!

Ok, I understand. Thanks for asking. I look forward to see Swift in renovatebot.com !!
By the way, I'll open a new issue now that I got your attention ;-)


Support masking layers from Figma, to enable better Dials & Gauges

Moving to 0.21, because we missed the cut-off for 0.20 with the PR.


TypeScript Server Error - Cannot auto import get code fixes

VSCode Version: 1.36.0-insider
OS Version: macOS Mojave 10.14.4
Typescript version: tested on 3.5.1, 3.5.2, 3.6.0-dev.20190618

Auto import does not work. Typescript server errors out when trying to auto import either by intellisense completion, cmd+. or hover and click Quick Fix

tsserver.log error
Err 1197  [14:49:8.716] Exception on executing command {"seq":28,"type":"request","command":"getCodeFixes","arguments":{"file":"/Users/donovanhiland/src/lx/src/shared/components/onboarding/RestoreSessionScreen/RestoreSessionScreen.tsx","startLine":58,"startOffset":12,"endLine":58,"endOffset":24,"errorCodes":[2304]}}:

    Debug Failure.

    Error: Debug Failure.
        at Object.assertDefined (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:2080:24)
        at getDefaultExportInfoWorker (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115542:80)
        at getDefaultLikeExportInfo (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115523:24)
        at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115506:35
        at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115650:21
        at forEachExternalModule (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115663:21)
        at forEachExternalModuleToImportFrom (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115648:13)
        at getExportInfos (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115504:13)
        at getFixesInfoForNonUMDImport (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115490:57)
        at getFixesInfo (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115431:50)
        at Object.getCodeActions (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:115193:28)
        at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:112841:121
        at Object.flatMap (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:573:25)
        at Object.getFixes (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:112841:23)
        at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:122417:35
        at Object.flatMap (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:573:25)
        at Object.getCodeFixesAtPosition (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:122415:23)
        at IOSession.Session.getCodeFixes (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:131471:64)
        at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:130221:61)
        at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:131665:88
        at IOSession.Session.executeWithRequestId (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:131656:28)
        at IOSession.Session.executeCommand (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:131665:33)
        at IOSession.Session.onMessage (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:131687:35)
        at Interface.<anonymous> (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:132978:27)
        at Interface.emit (events.js:182:13)
        at Interface._onLine (readline.js:290:10)
        at Interface._normalWrite (readline.js:433:12)
        at Socket.ondata (readline.js:149:10)
        at Socket.emit (events.js:182:13)
        at addChunk (_stream_readable.js:283:12)
        at readableAddChunk (_stream_readable.js:264:11)
        at Socket.Readable.push (_stream_readable.js:219:10)
        at Pipe.onStreamRead [as onread] (internal/stream_base_commons.js:94:17)

Steps to Reproduce:

Open vscode
Try to auto import something via the methods described above

Does this issue occur when all extensions are disabled?: Yes
I realize this might not be very much information. This does not happen on an extremely simple test project with the same tsconfig. Please let me know what if anything else I can do to provide more information.

Can you share your project or some example code that demonstrates this issue?

Unfortunately I can't share the project. I'm working on creating some example code with the same config but I can't reproduce. I'm wondering if it has something to do with the size of the project? The codebase where this is manifesting is decently big. It also looks like it's related to the allowSyntheticDefaultImports option being set to true. Setting it to false resolves the issue.
If this still doesn't help I can keep trying to reproduce.

Doing a search for getDefaultExportInfoWorker led me to #29227 which turns out to be the issue. We have a file re-exporting a non-existent default export from another file which resulted in the crash.

I have a similar problem, but auto-import is broken after creating a new file.
tsserver.log doesn't have any errors.
My workflow looks like this:

write some code
create a new file (react component)
auto-import is broken
restart TS server from the command menu

After restarting, auto-import is not broken if I create another file. But only after some time, and always if I create a new file. I must restart the TS server every 20-40 min.

Closing as issue was solved er https://github.com/microsoft/TypeScript/issues/31975#issuecomment-506466520


Add ability to apply templating to header & footer files

Closes #230
Check the following

[ ] Keep 100% code coverage
[ ] Be properly formatted
[ ] Have no linting errors
[ ] Documentation changes are included
[ ] Include a change file if expected

Additional context
This is based on PR #232 and as such includes its changes.
@miniscruff - Here is my take on how to close 230.  I'm open to any feedback you have.  The tests will NOT pass because I changed the signature of BatchPipeliner interface to make this work.  I'm happy to fix up the tests and adjust the documentations if you think this change is the right way to go about doing this.
IMO this allows the for the most flexibility on the end user's part because the header and footer can just be plain files OR they can be templates AND allows it to be configured as ephmeral or persistent.  This also means we could do a follow on which adds a new flag & config body-path / bodyFormat which could allow for the batch body to be completely built up using a template too for maximum flexibility for a power user to fully leverage sprig templates.
Also then going 1 step further we could completely remove the BatchPipeliner.WriteChanges function by converting it's logic to the default bodyFormat by leveraging sprig to provide as an example of how powerful and configurable the template format can be.

Also then going 1 step further we could completely remove the BatchPipeliner.WriteChanges function by converting it's logic to the default bodyFormat by leveraging sprig to provide as an example of how powerful and configurable the template format can be.

I think this sounds interesting but I am a little hesitant to go all in on sprig just yet, I would like to try it out a bit and see how it goes.

I think this looks good. Obviously some tests are gonna fail and probably linting issues, will definitely need some doc updates as well. I can help out with any of that if you are busy as well, but yea no complaints on the code really.

Alright @miniscruff - I fixed up the tests and got to a passing state locally but its only
ok  	github.com/miniscruff/changie/cmd	3.406s	coverage: 99.3% of statements

Would love any help with getting that back to 100% for you but i'm not sure what to do.

Alright @miniscruff - I fixed up the tests and got to a passing state locally but its only
ok  	github.com/miniscruff/changie/cmd	3.406s	coverage: 99.3% of statements

Would love any help with getting that back to 100% for you but i'm not sure what to do.

If you fix linting I can get that last little bit, or run make coverage and look at the html for the missing line. We also should mention in the docs that templates have extra functions.

Moved the work over to #239 as I was not able to work directly from the branch. Will probably get it done tomorrow.


Disable tooltips on mobile to prevent them to stay after clicking

This fixes #585. Yeah, I know, this is less than ideal. But this is somewhat blocking 2.0.0 release. And none of the other options are ideal either (some discussed options will simply not work).
On v1.5.0, these buttons didn't have any tooltips, so they won't be missed on mobile. Plus any hovering effect on mobile is kind of a lost battle anyway.
We have an open issue on Primer's tooltips: https://github.com/primer/tooltips/issues/4
I suggest we hide them on mobile to freely deploy 2.0.0, and see to work them back in when we have a better solution.

Fine by me for now, not that I think the tooltips are particularly useful. This might catch browsers as well on small screens in combination with zoom, but the fixed annoyance outweights the benefits for me. 👍


Was there at one point a 1.13.5 release?

I installed Customizer via the Kanboard plugin directory, not through the CLI, back around January/February. My Kanboard states that version 1.13.5 of Customizer is installed, but the latest release on here is 1.13.4. When upgrading plugins via the CLI, it says it's up-to-date, and doesn't get 1.13.4, which has a more recent release date. Was there at one point a 1.13.5 version that's since been removed?
Thanks, I really appreciate this plugin!

1.13.5 is the masters current version, you would have had to install using the master to get that version. The latest official release is 1.13.4.


Check 0 values observation

Remove observation with observed zeros values. Eg.; pm2.5 = 0

Solved. Data generated without 0s at VM.


gerrit-300802-2

Current coverage is 59.33% (diff: 14.28%)

Merging #88 into master will decrease coverage by 3.36%

@@             master        #88   diff @@
==========================================
  Files           265        263      -2
  Lines         50105      49392    -713
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          31416      29307   -2109
- Misses        18689      20085   +1396
  Partials          0          0

Powered by Codecov. Last update f8f8814...aad3afe


Re_botcmd_fullmatch: Add re_botcmd_fullmatch

Closes https://github.com/coala/corobo/issues/402
Reviewers Checklist

[ ] Appropriate logging is done.
[ ] Appropriate error responses.
[ ] Handle every possible exception.
[ ] Make sure there is a docstring in the command functions. Hint: Lookout for
botcmd and re_botcmd decorators.
[ ] See that 100% coverage is there.
[ ] See to it that mocking is not done where it is not necessary.

test coverage ;)


savings-account: missing prerequisites

The savings-account exercise requires the student to know the following types, which are not in the prerequisites:

The type keyword
The float32 keyword (numbers prerequisite)

This issue should be fixed together with https://github.com/exercism/go/issues/1583.

Included in other issue.


add accessibility section for Tooltip

Closes #1553
Description
Added accessibility documentation for Tooltip.
Is this a breaking change (Yes/No):
No
Additional Information
None

@setodeve
Please set order to front matter.


Add tests for 2D manifold in 3D

Tests for #25 should be added, covering the mesh generation, container constructors, and solvers.

Completed in #30.


X11-forwarding doesn't work

Either using a local or remote console I'm not able to visualize any GUI-based program from the console. Everything seems to work fine and no error is reported like the display output is redirect to some null source.

I make a component upgrade an it now works.


BUG: ComputeRidgeness checks for tangent direction

Before computing ridgeness, roudness, curvature and levelness, the
ComputeRidgeness function now checks for the tangent and normals
direction and reorder them if needed.

Looks good to me!


New rule: no-bind

To circumvent re-rendering by creating a new  function each time when binding this to functions
https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-bind.md
"jsx-no-bind": [<enabled>, {
  "ignoreRefs": <boolean> || false,
  "allowArrowFunctions": <boolean> || false,
  "allowBind": <boolean> || false
}]

I found that the performance impact may not even be the real issue this would be helpful with, but rather TypeScript's lenient typing of bind (bind(this: Function, thisArg: any, ...argArray: any[]): any;), which results in parameter-mismatches not being caught.

@tp Great point! I didn't notice that, here is the link https://basarat.gitbooks.io/typescript/content/docs/tips/bind.html

will be available in v2.6.0


Will re-created pod still on same node after Helm upgrade?

Hi,
We are using Helm and want to know some details about Helm upgrade.

During our "helm upgrade", pod will be re-created. We'd like to know if this re-created pod will still on same node, or it may be re-scheduled to other nodes?
If it may be re-scheduled to other nodes, how should we do (during helm upgrade) to keep it on previous node?

Thanks in advance.
Output of helm version:

Client: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}

Output of kubectl version:

Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.3", GitCommit:"435f92c719f279a3a67808c80521ea17d5715c66", GitTreeState:"clean", BuildDate:"2018-11-26T12:57:14Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.3", GitCommit:"435f92c719f279a3a67808c80521ea17d5715c66", GitTreeState:"clean", BuildDate:"2018-11-26T12:46:57Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
Kubernetes Vanilla

The recreation of pods is handled by Kubernetes and not Helm.  Helm asks the Kuberentes API server to perform operation by giving it rendered manifest files from the chart.
From Kubernetes perspective, maybe take a look at assigning pods to nodes.

@zhanglijingisme Is there any further information that you need?

@hickeyma Thanks for yoru help. There is no other question for me, and I'll close this.


'guitablabel' should only be set if user hasn't set it in vimrc.

Right now, the user has to override 'guitablabel' in gvimrc instead of vimrc because MacVim overrides the setting, but this change makes it so that MacVim only sets the defaults if the user hasn't set it already.

I would have thought that gvimrc and not vimrc is where settings governing the GUI should go.

In other words, what is the problem with having to override guitablabel in gvimrc?  That's where one would expect to override it.

That's the best practices as described in the docs (gui.txt), yes, but I think there are people who also just set gui options in vimrc using has("gui_running") checks. Feels to me if the user has already set the option, MacVim's own gvimrc should try to respect that and not stomp the custom value (you can set this option in regular Vim/Gvim's vimrc and it will work, for example).
My understanding of gvimrc is that there are certain GUI work and initialization that happens after vimrc is sourced, which is why you need another file, but in this case there's no technical limitation on why we should stomp the user's provided value.

Oh actually, I made a mistake in the docs then. You need to still override the gvimrc to get the default tab labels back. I'll submit PR for that.

but I think there are people who also just set gui options in vimrc using has("gui_running") checks.

There are (though I suspect it's very few), but they're not supposed to be doing that.
I don't see the point of this unless one is desperately trying to avoid needing to have a ~/.gvimrc in the first place.  If you already have a ~/.gvimrc, there is no reason not to put your guitablabel setting in it, and there is no reason to put that setting in ~/.vimrc.

Feels to me if the user has already set the option, MacVim's own gvimrc should try to respect that and not stomp the custom value

This would be true for options that have nothing to do with the GUI (e.g., &textwidth).  But I can't see why that shouldbe presumed to hold for GUI specific options.

Hmm, I don't really have too strong an opinion of this. I do think it's nice to respect options that was set in vimrc if the user set it somehow but yes they should have been set on the gvimrc instead. There are also other existing vimrc settings you can set like macvim_skip_colorscheme in order to not having to override them in gvimrc. Pinging @splhack  in case he has an opinion on this.

I don't have any opinion about that :) and I can revert it #637


Fix edge case where PF returns undefined

occasionally PF will return an array with a single undefined member when it doesn't find anything. This handles that error.

I wish I understood what causes the issue this fixes, but I'm seeing something along these lines in my test environment and this edit seems to fix it.


[Bug]: Incorrect type symbol for effective type

Description
Consider the following program
type A readonly & Pet;

type Pet record {
    string name;
};

If we get the effective type symbol of A we get a TypeReferenceTypeSymbol instead of RecordTypeSymbol
Steps to Reproduce
No response
Affected Version(s)
No response
OS, DB, other environment details and versions
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

The behaviour of showing type-reference symbol is correct but we get readonly & Pet as the name which needs to be fixed. This has driven to an IllegalArgumentException exception when calling definition() API in type-reference-type symbol


WebAssembly

The bulk of the UI is still built with GopherJS.
The WebAssembly compiler might be ready.

the standard golang WASM uses a ton of ram.
I have been using tinyGO that has  WAMS support.
https://github.com/aykevl/tinygo/tree/master/src/examples/wasm
Tinygo is amazing... Really should try it.

For now I'm more concerned that GopherJS might become unusable - e.g. https://github.com/gopherjs/gopherjs/issues/855 means that to rebuild the client one can't be doing things the module way. It's another command that needs to be installed locally, but Go WASM has the benefit of being built into Go.
TinyGo is promising. I think some Go features are in use that TinyGo isn't supporting yet, so we'll see.

A few days ago I started experimenting in the wasm branch. After many minor code fixes, it compiles and runs up to the point of displaying a graph. Sadly UI interaction is broken after that point; the client crashes inside github.com/johanbrandhorst/protobuf which is still based on GopherJS. From a very quick look, Johan's packages don't look too difficult to port to syscall/js...

Good thing I didn't keep going down that path, because of course github.com/johanbrandhorst/grpcweb-wasm-example exists already.


Get markdown report from JSON

When I run an nvbench-based benchmark, like:
thrust.bench.transform.basic.base -d 0 --stopping-criterion entropy --json baseline.json

it prints a nice markdown report at the end, summarizing the run benchmarks, times, bandwidths etc. All data is also stored into baseline.json. I archive the JSON file for later use, e.g., comparing it with other results. The terminal output is discarded. However, the comparison script nvbench_compare.py later only shows the runtimes of the baseline and comparison run, not the bandwidth and %SOL. The data should be in the JSON file tough.
Is there a way to have the comparison script print the bandwidth as well?
Is there a tool/script to print the markdown report given a JSON file?
Both features would be very useful to easily print the achieved bandwidth and %SOL for results stored as JSON. Otherwise, I would always need to capture and archive the markdown report as well, which is annoying and seems redundant.

Is there a way to have the comparison script print the bandwidth as well?

The data is in the JSON and we can modify the comparison script to print it (maybe an --additional-fields option or similar?).

Is there a tool/script to print the markdown report given a JSON file?

No, the existing markdown printer reads from the runtime data structures, not the JSON archive. This would require a new implementation.

I would warmly welcome both additions :)


Typings: missing prop "item" for Dropdown

Steps
<Dropdown item/>

It occurs
[ts] Property 'item' does not exist on type 'IntrinsicAttributes & IntrinsicClassAttributes<Component<DropdownProps, ComponentState>> & Readon...

From documentation: item{bool}: A dropdown can be formatted as a Menu item.
Version
0.64.4

@layershifter when you hit Dropdown's typings, can you add this?  Thanks!

Ignore me, didn't see the PR was already opened :)


Profile application that runs on top if Kubernetes?

Hello there,
Do you know how to run the profiler for an app that runs on kubernetes? I have tried from within the container but failes with:

"Perf events unavailble. See stderr of the target process."

And also tried from within a kubernetes node with error:

Failed to inject profiler into 74053 	linux-vdso.so.1 =>  (0x00007fff48f78000) 	libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007f914ce02000) 	libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f914cbe5000) 	libstdc++.so.6 => /usr/lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007f914c863000) 	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f914c55a000) 	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f914c344000) 	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f914bf7a000) 	/lib64/ld-linux-x86-64.so.2 (0x00007f914d6b0000)
Anyone did this with success?
Thank you
G.

Extended Troubleshooting section with more details.
Added a section about profiling in a container.
Implemented itimer profiling mode to workaround various restrictions.


feat: add Datadog plugin card in plugin hub

Fixes: #[Add issue number here]

add Datadog plugin card
update the icon

Screenshots of the change:

cc @bzp2010

wait #715

The datatog's plugin link is https://apisix.apache.org/docs/apisix/next/plugins/datadog
@1502shivam-singh Do you have any ideas on this? I'd prefer using this way:

Keep the https://apisix.apache.org/plugins/ page using the stable plugins link;
Adding a mark or note on Datadog, and saying this plugin is in the Next Version of Apache APISIX.

Keep the https://apisix.apache.org/plugins/ page using the stable plugins link;
Adding a mark or note on Datadog, and saying this plugin is in the Next Version of Apache APISIX.

Right, makes sense. We could put a beta tag over such plugin cards, like a notification badge, until that particular APISIX version releases.

I have added a beta tag to the plug-in card and added a comment on the hover, Also updated with links.
If there is still something that needs to be corrected, please help me to point it out. Thanks!

cc @1502shivam-singh @bzp2010 @bisakhmondal


Use of Java Servlets

Chnages the addFunFact function in .js file so that it now fetches a random quote from the server and s
displays it on our home page

Could you send a link to the running demo instance with each review, please? It's much easier to verify whether things are working as expected when we can test them.


[Crash Report] Crash when I place Crystal Growth Chamber

http://pastebin.com/jmXqtnhE
I'm using:
appliedenergistics2-rv3-beta-5
bdlib-<IP_ADDRESS>-mc1.7.10
ae2stuff-<IP_ADDRESS>-mc1.7.10
Game crash with and without Optifine

If you are using AE2 rv3 alpha/beta you need the -rv3 builds of AE2 Stuff.
http://minecraft.curseforge.com/projects/ae2-stuff/files/2270295


Android-Kotlin : 19.01 Overview of MVVM architecture in Android.

Description :scroll:

Give a step-by-step tutorial in Kotlin along with a brief and clear explanation.
Required Documentation or Video or Audio.

Domain of Contribution :bar_chart:

[x] Android Dev (kotlin)

References :

https://www.geeksforgeeks.org/mvvm-model-view-viewmodel-architecture-pattern-in-android/#:~:text=Model — View — ViewModel (MVVM,logic%20part%20of%20the%20application.

Please Read it Carefully Before Contributing

The file structure should look like Android_Development_With_Kotlin  -> 19 Architecture -> <file name>.
You have to push your PR to the Android_Development_With_Kotlin branch only.
You can take help to the references but don't simply try to copy it. Just explain in your own word.
Your code should be written only in Kotlin.
This issue is for GWOC'21 contributors only.
You need to push in a single commit. If there exists more than one commit then you need to squash your commits into one commit.
Will assign you on a first come first serve basis.
Please try to complete your assigned task within 7 days. If you need some more time then, please inform mentors and supervisors.
If you have two previously assigned issues on Android-Kotlin pending then please first complete them.

Supervisor @rahulsain

Happy Contributing. Best of Luck.:sparkling_heart:

I'd like to contribute the documentation for MVVM. Please assign this to me!

Any update? @Agamjyot08

Can i work on this for documentation @rahulsain @sitamadex11 ?


As shown in the figure, to process remote logs, we need to make this jar package and put it in our project, or under Maven dependency?

Just pack it and put it under your project. Subsequent versions will be deployed to the official pom.

需要打包一下放到你的项目下就好。后续的版本会deploy到正式的pom上。