transport: native sockets on iOS/Android

Using native sockets on iOS/Android is the recommended approach for performing low-level networking. We should investigate:

What benefits we can get from using native sockets/APIs
The work required to use these APIs, both within Envoy Mobile and upstream Envoy

Current proposal:

Envoy uses BSD sockets for sending/receiving data. On iOS, there are known issues that arise with this approach (such as these with gRPC). To work around these problems, Envoy will need to support using the native networking stack to send/receive data in place of the built-in mechanism
Note: We’ll need to take into account TLS support (we should likely use TLS from Envoy instead of native)
Implement a socket provider on iOS that will replace the existing BSD socket usage in Envoy. This will need further investigation, but can likely be accomplished through CFStream, Network.framework, etc.

Potentially related, here are some key announcements Apple made regarding networking at WWDC earlier this month: https://github.com/Blackjacx/WWDC#advances-in-networking-part-1 (parts 1 and 2)

You'll definitely want to use Network.framework as the socket provider on iOS – it works across all of Apple's platforms. BSD sockets do not work on Apple Watch, for example.

an even simpler test: iirc, BSD sockets don't handle cellular/wifi transitions, but Open Transport and Network.framework do. Other areas to check might be multipath TCP support and VPNs.

For watchers the main work items here are:

Slightly refactor the TLS transport socket to allow for swapping out the actual read/write/connection APIs
Create a new iOS transport socket that derives from the upstream TLS transport socket and uses appropriate iOS APIs for the underlying connections.

This should be pretty straightforward given existing Envoy extensibility mechanisms.

Detailed write-up of the current state of the transport layer issues on iOS in this comment: https://github.com/lyft/envoy-mobile/issues/128#issuecomment-516260951

Current proposal doc: https://docs.google.com/document/d/1vb6qyDLyUi0MpItFEsXcJ6oKD61B3DPVtowN7CoH6ls/edit?usp=sharing

Related: https://github.com/lyft/envoy-mobile/issues/541

https://docs.google.com/document/d/16jq5YO542BbAnBGwldgPfaJel9H3K6ufWD7Lqvrx0Bg/edit?usp=sharing
The above doc captures the current state of the world with copious linked references and examples. In brief, our current usage of POSIX sockets on both platforms, with our current workarounds, presents no serious general-case issues. There are some somewhat niche cases (e.g. on-demand VPN on iOS) which could be better-addressed.
In the medium term, implementing socket providers using the newer platform APIs would yield some incremental benefit, but more importantly, help to future-proof the library.

While there are no general issues, Multipath TCP could yield pretty significant gains:

Last year we announced that in addition to Siri, we'd also enabled Multipath TCP for Apple Music. Since then, we've seen a 13 percent reduction in music stalls and in the event that a stall occurs, there has been a 22 percent reduction in the stall duration. https://developer.apple.com/videos/play/wwdc2020/10111/

(It might even be more pronounced for Lyft because users are leaving/entering WiFi range all the time)


Dev compare and equal constraint types

Codecov Report
Base: 96.86% // Head: 96.70% // Decreases project coverage by -0.15% :warning:

Coverage data is based on head (a55126d) compared to base (e2f9afd).
Patch coverage: 93.81% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
- Coverage   96.86%   96.70%   -0.16%
==========================================
  Files          36       36
  Lines        2230     2310      +80
==========================================
+ Hits         2160     2234      +74
- Misses         64       68       +4
- Partials        6        8       +2

Impacted Files
Coverage Δ

collections/iterators.go
99.01% <ø> (ø)

collections/rb_tree.go
92.91% <50.00%> (-2.21%)
:arrow_down:

collections/maps.go
96.00% <83.33%> (-4.00%)
:arrow_down:

collections/array_stack.go
100.00% <100.00%> (ø)

collections/circular_array.go
99.03% <100.00%> (+<0.01%)
:arrow_up:

collections/deques.go
100.00% <100.00%> (ø)

collections/graphs.go
100.00% <100.00%> (ø)

collections/lists.go
100.00% <100.00%> (ø)

collections/multimaps.go
100.00% <100.00%> (ø)

collections/queues.go
100.00% <100.00%> (ø)

... and 2 more

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

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


Restore CQL Worker

This PR allows the CDS Dashboard to leverage the latest update to Encender's cql-execution branch, which restores the use of CQL Worker for CQL Execution

Restore WorkerFactory in applyCDS()
Update package-lock.json to point to latest commit of Encender's cql-execution branch (https://github.com/ccsm-cds-tools/encender/pull/15)

This will be subsumed by https://github.com/ccsm-cds-tools/ccsm-cds-dashboard/pull/69, and can be closed once that pull request is merged.

https://github.com/ccsm-cds-tools/ccsm-cds-dashboard/pull/69 was merged, which contains the commit in this PR. This PR can now be closed.


[GITHUB-5] Update to Latest Standards

Flatten variables
Convert to molecule testing (test for installed package only due to docker limitations)

LGTM 👍


Add predictions to train_step

return predictions of train step here to allow users to calculate any other metric
https://github.com/flowerpot-ai/stabilizer/blob/master/stabilizer/trainer.py#L14

Yes that should be useful. I will do it and create a PR


Lab 2 Wordpress critical error

Some students get There has been a critical error on this website when installing the Wordpress role. Fix this issue in the role itself or write a note about this problem in the assignment.
The solution for this problem is to install the package php-json.

I think I'm not going to keep maintaining that role. Since the wordpress package was deleted from the EPEL repo, it's much more complicated to reliably install Wordpress. But then, Wordpress should be replaced with something equivalent, preferably with a database backend.


add dob field

closes: #143
Add a DOB field in profile and profile diff.
For reasons Please use the above issue

Are there any tests for Profile service so that we can test for this modification.


Alfred gets disconnected and doesnt reconnect by itself.

Happened on an external server.
strace say :
strace -p 25539
Process 25539 attached
select(4, [3], [], [], {15, 312840})    = 0 (Timeout)
clock_gettime(CLOCK_MONOTONIC, {7504257, 407523298}) = 0
open("/usr/share/gotham/modules.d/save/alfred.xxx.fr.save.tmp", O_WRONLY|O_CREAT|O_TRUNC, 0700) = 6
write(6, "{\n\t\"me\":\t{\n\t\t\"features\":\t{\n\t\t\t\"x"..., 5888) = 5888
close(6)                                = 0
rename("/usr/share/gotham/modules.d/save/alfred.xxx.fr.save.tmp", "/usr/share/gotham/modules.d/save/alfred.xxx.fr.save") = 0
select(4, [3], [], [], {29, 972715})    = 0 (Timeout)
clock_gettime(CLOCK_MONOTONIC, {7504287, 411634697}) = 0
open("/usr/share/gotham/modules.d/save/alfred.xxx.fr.save.tmp", O_WRONLY|O_CREAT|O_TRUNC, 0700) = 6
write(6, "{\n\t\"me\":\t{\n\t\t\"features\":\t{\n\t\t\t\"x"..., 5888) = 5888
close(6)                                = 0
rename("/usr/share/gotham/modules.d/save/alfred.xxx.fr.save.tmp", "/usr/share/gotham/modules.d/save/alfred.xxx.fr.save") = 0
select(4, [3], [], [], {29, 968603})    = 0 (Timeout)
clock_gettime(CLOCK_MONOTONIC, {7504317, 412073087}) = 0
open("/usr/share/gotham/modules.d/save/alfred.xxx.fr.save.tmp", O_WRONLY|O_CREAT|O_TRUNC, 0700) = 6
write(6, "{\n\t\"me\":\t{\n\t\t\"features\":\t{\n\t\t\t\"x"..., 5888) = 5888
close(6)                                = 0
rename("/usr/share/gotham/modules.d/save/alfred.xxx.fr.save.tmp", "/usr/share/gotham/modules.d/save/alfred.xxx.fr.save") = 0
select(4, [3], [], [], {4, 176185})     = 0 (Timeout)
clock_gettime(CLOCK_MONOTONIC, {7504321, 593980921}) = 0
write(2, "ERR<25539>: ecore_con.c:694 ecor"..., 52) = 52
write(2, "safety check failed: svr->delete"..., 43) = 43
write(2, "\n", 1)                       = 1

Last 3 lines tells us a bit the error, but its not enough to understand clearly

netstat tells there isnt any socket connected to the server.

This seems to have led the  xmpp library to an endless error state.

A fix should be written in either ecore or shotgun. Keeping this issue opened until fix is done.

And then follows the original error :
ERR<28322>: ecore_con.c:694 ecore_con_server_send() safety check failed: svr->delete_me is true

This is probably the ping from shotgun (sending a single space) that creates the error.

So the bug lies in ecore itself when using dns.c as a dns resolver.
Disabling ipv6 would switch to fork+getaddrinfo, fixing the issue.
So either change dns resolver, either fix the use of dns.c.
When using dns.c, we will jump to this condition : https://github.com/gfriloux/ecore/blob/master/src/lib/ecore_con/ecore_con.c#L1658
   if (!net_info) /* error message has already been handled */
     {
        svr->delete_me = EINA_TRUE;
        goto error;
     }

Bug the comment is wrong when dns.c is used, cause the error will not have been handled.


Solution - Jeff VanDyke, Vibration Research

Sorry. Your solution doesn't give the correct answer. Please, feel free to fix an resubmit.


tsc compiler option for --newLine NEWLINE Use newline: 'dos' or 'unix'.

I have an patch to TypeScript compiler options to solve issues with different newlines as in this item:
https://typescript.codeplex.com/workitem/258
It adds this option to tsc, and uses it in program.ts / createCompilerHost() / getNewline():
--newLine NEWLINE Use newline: 'dos' or 'unix'.
I've signed the Contributor License Agreement (CLA) and I'm working on test cases but what is the next step to be allowed to push a branch and/or create a pull request?
Thanks to all for TypeScript, and I hoping to eliminate the newline conflicts at the source for mixed development environments of Mac, Linux/UNIX, Windows.

Hey @kmashint, thanks for checking in with us. Here's some steps you might want to check:

Read our CONTRIBUTING.md.
As described in the above, make sure you follow the Coding Guidelines and in general try to stay canonical with our codebase.
In your PR body (not title), clearly specify what issue you're aiming to resolve/fix (i.e. #1693), and potentially link to this issue for reference as well.

For this feature specifically, ~2-3 tests for both CRLF and LF line endings each might be decent. I'd put the tests in tests/cases/compiler and I'd name the tests something like newlineFlagWith[BlahBlah]CRLF.ts/newlineFlagWith[BlahBlah]LF.ts (e.g. newLineFlagWithTemplateStringsCRLF.ts.
In general, if there's an existing issue and the reception looks positive (or you just want to show a proof-of-concept and don't mind if it doesn't get pulled in), you can send out a PR. We'll try to give feedback on it and work with you.

i don't really like 'dos' and 'unix' as the names.  Why not just be explict with things like  cr, crlf, and lf ?

i don't really like 'dos' and 'unix' as the names.

Agreed, use CRLF and LF as option names (don't include CR as an option).

The open issue that is marked as accepting Pull Requests is here: https://github.com/Microsoft/TypeScript/issues/1693

Thanks, I took a survey and git is the only tool that refers to crlf and lf combinations.  I'll change to use --newLine CRLF (or LF, lower-case also acceptable) although git is far from the best example of command-line options (what's the opposite of push?  fetch, not pull... hmm...)
UNIX tools: dos2unix, unix2dos
vi, vim: set ff=dos, set ff=unix
Eclipse: workspace line delimiter Windows or UNIX
Notepad++ (Win): EOL: UNIX Format, Windows Format, Mac Format (old CR-only)
WinMerge: carriage return differences (DOS/UNIX/MAC)
git: options refer to crlf, lf, cr combinations

Git is nice, modern, and precise.  For example, say that Windows moves to LF at some point.  Now 'Windows' or 'Dos' is a complete misnomer.  This is not speculative, look at how this happened with Apple.  It used to be CR only for them, but then they moved to LF.
CRLF and LF is precise and indicate exactly what you'll be getting.

I hear you on git, but somewhere they lost track of the first part of the adage to "keep simple things simple and the complex possible".


Can't seem to figure out if i can embed objects (binary files) in MS Word document

Is it possible using this package to embed binary objects (i.e., OLE objects) into a MS Word document and have an icon represent them like in the sample image below? I'm looking for a cross-operating system compatible way of generating .docx files containing an unknown number of small embedded files.

Not presently. That feature does exist in python-pptx however, if you're flexible as to the document format.


Update 0046-first-label.md

Update "Impact on Existing Code" section to have slightly better wording and fix one spelling error.

incorporated into #204


Add coingeckoId to ChikinCoin

Here is the initial commit for ChikinCoin
d01e051b2b3ce96a23e823f0b294c4ab6cea65f2

Here is the coingecko url https://www.coingecko.com/en/coins/chikincoin


npm unpublish old google books repo

Considering that the github repo for this package no longer exists, perhaps you should consider running npm unpublish on it so people on NPM don't start using a package that won't be supported.

Thanks for the heads-up, it has now been removed from NPM.


Quarkus 3 Upgrade

Should this be tested / released as alpha with Quarkus 3?
I believe this is a precondition to do some work on https://github.com/quarkiverse/quarkus-tika/issues/79
At first sight it seems to be working with minimal changes:
https://github.com/tpodg/quarkus-poi/commit/2b3a812b426f520a38a94197316aa2d8adaac2b3

The integration tests are not published in central, so I'd say that it works in Quarkus 3. Let's wait until 3.0.0.Final is released to update

Actually, I think we can update to Quarkus 3.0.0.Alpha4 with your commit and release a 2.0.0.Alpha1, WDYT?

Sounds good. A new branch is probably required though?

Yeah, instructions are here: https://github.com/quarkiverse/quarkiverse/wiki/Migrating-to-Quarkus-3.x#implement-the-migration

Thank you for the link. I can prepare everything described, except for the branch in this extension repo due to permissions.

@tpodg not a problem, I added you as a committer in https://github.com/quarkiverse/quarkiverse-devops/commit/cb86529cc934d1f55670fbe125bb7d3037841fb0

@all-contributors add @tpodg for code and maintenance

Thank you


Enforce Login on Admin Pages

Admin Pages Auth intergration
https://spaceyatech.com/admin/events/add-event

Login has been set up.


Missing Video Metadata

Hello, the recording videos are missing some metadata like length and frame width/height. VLC can play them normally, but if you try to look for videos details in windows 10, adobe premiere, and other video players. They can't read the metadata and some players cannot read the files.
It could be an encoding problem. I'm comparing  a scrycpy video (test.mp4) and  adb recording (adb video.mp4)
https://imgur.com/a/IuKqsTn
I hope this helps. Thanks!

the recording videos are missing some metadata like length and frame width/height

These metadata are present for both files in your screenshot.
I see nothing important missing.
The only difference is the codec id, which is isom on the left and mp42 on the right, but both should be ok.

In the windows explorer, you can see the windows 10 is able to preview the file and read the metadata.

The video files fail on:

Windows Media Player
Movies & TV
Quicktimes
Adobe Premiere
Video Player Android 8.1
It works fine on VLC. I have to transcode the video with Handbrake to make it work.
It seems that some quick-access metadata is missing. If you go to properties on Windows, therea are not video details.
Thank you for your work!

Thank you for the details.
I will investigate when I have some time (and currently I don't have access to any of the software that you listed which fail).
For now, could you paste the result of scrcpy -v please?
In particular, I would like to know which #ifdef branch is executed here. If it's the "new version" branch (which uses codecpar), I am interested in the result with the "deprecated" version (which uses codec):
diff --git a/app/src/recorder.c b/app/src/recorder.c
index 2e846e9..b13bb5a 100644
--- a/app/src/recorder.c
+++ b/app/src/recorder.c
@@ -63,25 +63,11 @@ SDL_bool recorder_open(struct recorder *recorder, AVCodec *input_codec) {
         return SDL_FALSE;
     }

-// In ffmpeg/doc/APIchanges:
-// 2016-04-11 - 6f69f7a / 9200514 - lavf 57.33.100 / 57.5.0 - avformat.h
-//   Add AVStream.codecpar, deprecate AVStream.codec.
-#if    (LIBAVFORMAT_VERSION_MICRO >= 100 /* FFmpeg */ && \
-        LIBAVFORMAT_VERSION_INT >= AV_VERSION_INT(57, 33, 100)) \
-    || (LIBAVFORMAT_VERSION_MICRO < 100 && /* Libav */ \
-        LIBAVFORMAT_VERSION_INT >= AV_VERSION_INT(57, 5, 0))
-    ostream->codecpar->codec_type = AVMEDIA_TYPE_VIDEO;
-    ostream->codecpar->codec_id = input_codec->id;
-    ostream->codecpar->format = AV_PIX_FMT_YUV420P;
-    ostream->codecpar->width = recorder->declared_frame_size.width;
-    ostream->codecpar->height = recorder->declared_frame_size.height;
-#else
     ostream->codec->codec_type = AVMEDIA_TYPE_VIDEO;
     ostream->codec->codec_id = input_codec->id;
     ostream->codec->pix_fmt = AV_PIX_FMT_YUV420P;
     ostream->codec->width = recorder->declared_frame_size.width;
     ostream->codec->height = recorder->declared_frame_size.height;
-#endif
     ostream->time_base = (AVRational) {1, 1000000}; // timestamps in us

     int ret = avio_open(&recorder->ctx->pb, recorder->filename,

It seems the If block code is executed, libavcodec is 58.18.100
>scrcpy -v
scrcpy v1.5

dependencies:
 - SDL 2.0.9
 - libavcodec 58.18.100
 - libavformat 58.12.100
 - libavutil 56.14.100

@Gapplay Could you tell me whether this video has metadata issues, please? test.mp4

@rom1v that video is working correctly on all video players/programs I've tried.

Great, then I have a fix :tada:

Fixed on dev branch.

Thanks to my colleague @robUx4 who helped me a lot during the FOSDEM to fix this issue (and others which prevented recording to MKV, now it also works :tada:)

Do you have an ETA for when this might be released?


Transcript fix

Fixed empty transcript menu, added gene-viz when only gene entered, added gene-viz to other model tracks, showExon now displays correctly for other tracks, added terms of service glyph to nav bar, only drawing gene-viz once, and updating when data changes,

This pull request has been updated with the recommended changes after review

@WilliamRichards2017 is this deployed somewhere I can test it? dev.gene.iobio.io?

deployed to http://25e8c9ec.ngrok.io/

Looks good to me for merging

@WilliamRichards2017 @mvelinder

I am going to merge this pull request, but we still have problems with showing the correct coverage for Mother and Father exons.  I'll create a new issue.


Added Travis to Slack notification

@frankscholten conflicts, and one comment

@frankscholten please resolve the conflict.

@pidster @vlal I resolved conflicts

Coverage remained the same at 24.121% when pulling fbbf9ff6bae5406d8cc829876be1b1fa5d20f42b on enhancement/travis-slack-notification into e1e3c65b17f1b5204d320a0d8e50ddc917a728cd on master.


Fixed #ISSUENUM of Check if two trees are Mirror in C++

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Description
(Write your answer here.)
Checklist

[ ] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[ ] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

We don't accept PRs without any corresponding Issue.

@HarshCasper  I was assigned by this issue.

@rk05-lenavo
This is why we have this section in clear words:

We ask you to share the Issue Link you are assigned. You cannot expect us to skim through 300+ open Issues and find the one you are assigned to. Be kind on us, so that we can be kind on you.


One knative service to one endpoint

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #460
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

This change is

/cc @rakelkar @yuzisun @animeshsingh @ellis-bigelow

Cool. Nice little refactor and looks like you fixed a couple bugs with status conditions. Can you add tests for this?

Sure, Done

/test kubeflow-kfserving-presubmit

/test kubeflow-kfserving-presubmit

/test kubeflow-kfserving-presubmit

/test kubeflow-kfserving-presubmit

Nice work @hougangliu !
/lgtm
/approve


WIP: Bring prefs interface up to Layer 7 version

Fix for Layer 7 compatibility. Currently compiles but doesn't register command correctly.
Fixes #11 (eventually)

Withdrawing PR, grown-ups taking care of patch.


ConfigAgent: move BounceDatabase(

Change-Id: I3955b90fc8a60462325348ad3a4d99a06765cc0d

/retest

Looked over the PR while rebasing and noticed that the req struct had another arg and should be used as a full struct to keep the args structured.
Also reviewed all previously merged PRs to confirm that this same oversight didn't occur.


Rewrite FeedForwardCharacterization to add acceleration

Now uses Moore-Penrose pseudo-inverse instead of Polynomial Regression, allowing us to be able to also calculate kA.
Special thanks to team 3061 :)
This should not be merged until tested on real robot.

Update: this will not work unless we switch the swerve modules to deal with meters per second. This will be done in another pr


Keyboard settings lock up and change bindings

VSCode Version: 1.9.1
OS Version: Windows 10 (Surfacebook)

Just installed the latest updates for VS Code on my Surfacebook. I typical use the UK layout because when I touch type I don't look at keys and naturally go for the layout I grew up with. On VS Code I switch into the US layout (because I can't use backslash without switching to US).
Whilst using the new version of VS Code if I change pages, the keyboard settings lock up, the keybindings change and when I press certain keys they correspond with neither US nor UK layouts nor Romanian (The only other keyboard I have active).
Steps to Reproduce:

Open VSCode, switch to US Keyboard layout from default UK Layout
Type in a file
Switch to another file
Attempt to type (try using stuff like quote marks - alphanumerical characters seems unaffected)
Also note that with VS Code still in focus (important, don't switch to another program), the keyboard switcher (in the bottom right corner) freezes and doesn't work.

The keyboard doesn't switch in any other programs (including other text editors and IDEs), it just seems to have started in VS Code.

In 1.11, on Windows, we now detect keyboard layout changes and refresh our keybindings tables appropriately


Find bar not following accent color

I chose this accent color in Windows,

And my findbar at the bottom looks like this as a result (is there a way to follow accent color?)

in addition, if the difference mode is set to lighter, the findbar is unreadable.

Yes, that's a pretty nasty bug!

Fun fact: If Unfocused theme is set to Light titlebar, findbar is displayed correctly only when inactive.

Yup. I think it is related to the new theme API introduced:
https://developer.mozilla.org/en-US/Firefox/Releases/60

Checking the release notes of FF 61 https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/61#WebExtensions , now the toolbar attributes apply also to the find bar.
When the window is selected, a default value is applied to the toolbar. That's what is causing the bug.

@martinguti95 Yes, and that snippet of code in one of the commits solves the issue, however there are problems with it as you will find out. If you see another way to fix this, I'm all ears.


New version AnimateJS

Hi! A new version of animate.js has been released, I think it's time for you to update too!

Yes, just tested it, it doesn't work with the new version of the animate.css. The name convention of the CSS has been changed like this animate__animated animate__heartBeat

Hello，我是林昕，我已收到来信~

It woud be nice to update the defaults, but it still works using the animateClass option:
import wowjs from "wow";

const wow = new wowjs.WOW({
  animateClass: "animate__animated"
});

document.addEventListener("DOMContentLoaded", function() { wow.init(); });


Better optimized PNGs

I ran all the screenshots (and logo) through ImageOptim, which selects the best lossless encoding method for PNGs. It managed to shave 250KB off of the 12MB of images in the folder, with an average per-image saving of 3.5%. All images should look exactly the same (no compression artifacts or anything), and take up less space.

Can you tell which files resulted in the most savings? Most of them have already been run through ImageOptim once. I suspect much of the 250kb savings comes from a small number of images.

All images should look exactly the same

I might just be hyper vigilante to it, but I see noticeable color shifting on some of the images.

Here is a list of all the files sorted by their % savings. I'd be happy to selectively optimize the worst offenders.
Would you mind sharing which images you noticed the color shifting on? I'll take a closer look at them and see if there's a pattern. I know Imageoptim strips out color profiles (since they shouldn't be used on the web), but it still shouldn't be super noticeable.


Move requirements.txt into provisioning directory.

Move requirements.txt into provisioning directory.
Do not merge until JJG-Ansible-Windows pull #16 is merged and the version in this repo is updated from upstream.

Had to rewrite the patch because of other changes, but thanks, this makes the root directory that much nicer!


Possible to search packages?

Hi!
Is it possible to also search for references in packages? For example, I select a file in a package and want to know where it is used within that package.
Thanks!
Chris

Nevermind! I found the solution: I had to remove the Packages/ entry from the exclude list in the settings.


Task list for tvOS support

Task list for tvOS support
Documentation

[ ] Installation
[ ] Limitation

[ ] Storage
[ ] App size
[ ] Encryption
[ ] Inter process notification

[ ] Use cases

[ ] Bundled data
[ ] Inteligent Cache

Unit tests

[x] Logic tests (on Simulator)
[ ] UI tests on device? (If possible)

Sample code

[ ] Cache sample?

Build/Release Automation

[ ] Core
[ ] Realm/RealmSwift
[ ] CocoaPods
[ ] Carthage

I'd like to get your feedback.
/cc @jpsim @mrackwitz @timanglade @bigfish24

@bigfish24 was kind enough to lend his tv for our CI system until Realm can buy one when they go on sale on Monday. I've updated our CI setup to automatically attach the tv to a worker VM, which is tagged with appletv in Jenkins.
We now need to create a build.sh command similar to test-ios-devices that will pick up a TV if it's available and run the unit tests on that.

Unit testing on simulator available on https://github.com/realm/realm-cocoa/pull/2721

tvOS support has been released in 0.97


Add Java Operator SDK to operator frameworks

Java Operator SDK allows to implement Kubernetes Operators in Java. Already used for various services in production, a running project for more then two and half years. This PR adds it to the list of tools that helps implementing Kubernetes operators.

/easycla

Hi @csviri,
Thanks for proposing this change! But please limit this PR to content/en/docs/concepts/extend-kubernetes/operator.md only.
Once the change to content/en is merged, you can open other PR to update each languages, or let others to do it.

/approve


dbw_polaris_ros: 1.0.1-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository dbw_polaris_ros to 1.0.1-1:

upstream repository: https://bitbucket.org/DataspeedInc/dbw_polaris_ros.git
release repository: https://github.com/DataspeedInc-release/dbw_polaris_ros-release.git
distro file: noetic/distribution.yaml
bloom version: 0.10.7
previous version for package: 1.0.0-1

dbw_polaris

No changes

dbw_polaris_can
* Bump firmware versions
* Add reserved bits
* Improve printing of license info
* Add user control of alert
* Contributors: Kevin Hallenbeck

dbw_polaris_description

No changes

dbw_polaris_joystick_demo

No changes

dbw_polaris_msgs
* Add user control of alert
* Contributors: Kevin Hallenbeck

Holding for Noetic sync


Add -convert-elementwise-to-affine

Second part to fixing #143 by adding -convert-elementwise-to-affine which can be used prior to -polynomial-to-standard to rewrite polynomial operations over tensors of polynomials to loops over those tensors instead.
It's probably a good idea to throw in some loop unrolling/fusion after using this pass, as it currently creates one affine.for per (tensor-operand) polynomial op.
My idea was that it's probably relatively easy to adapt LowerElementwiseToLinalg to generate affine.for instead of linalg.generic. However, it turns out the final pass implemntation is in fact rather different from the linalg version, since linalg.generic doesn't need to make the operands explicit, the biggest change is that we now need to generate explicit tensor.extract and tensor.insert operations

As far as I can tell, this now works as expected and is ready to review - I rebased it current main to remove the dependency on PR 508 as they're in fact completely orthogonal (just both relevant to #143).

As part of integrating this into the polynomial pipeline, I changed how the --heir-polynomial-to-llvm pipelines handles the actual "to llvm" part from a bunch of manually assembled passes to using the ConvertToLLVM framework (that automagically lowers everything that has the appropriate interface). This should (hopefully) make that part of the pipeline more robust to future changes in higher-level lowerings 😃
With this, we now also have a bgv->polynomial->llvm path (-bgv-to-polynomial -heir-polynomial-to-llvm), though given the performance overhead of the generic polynomial-to-standard lowering, I wouldn't recommend trying to actually run the generated code for anything except toy programs 😉

Fixes are in & PR is squashed :)

Great work! Do you think there'd be an appetite for this upstream?

Yes, I think this (and your structural pattern conversions stuff + the tensor of tensor logic) really seem like common patterns that MLIR should support out of the box (but any upstream PR is a todo for after Toronto 😉).


Add Tools 26 parsing

This Closes #2.
I implemented a new parser for build-tools > 26. I also noticed sdkmanager finally has a --version argument, which I'm using to decide which parser to call, since this would be a breaking change.
If you think this should be done differently (by an option in settings, for example), feel free to suggest the changes. The boring part of parsing is done anyway.
What do you think?

@gep13 @Redth would someone be able to review and maybe accept this PRs?


Send to Interactive doesn't correctly handle indented code

Highlighting code and hitting "Send it to Interactive" is an extremely important feature to have working robustly.
However, the handling of indentation is extremely buggy.
One of the easiest demonstrations is the following: put this in a python script
def something_that_has_indented_code():
    x = 1

    y = 1

    # A comment
    z = 3

    return

now highlight starting from the line above y=1 to the end of the line y=1. You'll get randomly either

or

Now send it to interactive.
>>>     y = 1
  File "<stdin>", line 1
    y = 1
    ^
IndentationError: unexpected indent

Now highlight the comment and z-line:

Send to interactive
>>> # A comment
...     z = 3
...
  File "<stdin>", line 2
    z = 3
    ^
IndentationError: unexpected indent

Loops don't execute properly because of indentation handling. Test code:
def something_that_has_indented_code():
    x = 1

    y = 1

    # A comment
    z = 3

    while(True):
        if x == 10:
            raise Exception('paste fail')
            break

        # this is a comment
        x = x + 1
        bob = 'bob'

        # ------ Hello ------ #

        # some comment
        # another
        print('Counter is %s' % x)

        if x == 4:
            break

    a = 2

highlight the following

Send to interactive
>>> x = 1
>>> y = 1
>>> # A comment
... z = 3
...
>>> while(True):
...         if x == 10:
...             raise Exception('paste fail')
...             break
...         # this is a comment
...         x = x + 1
...
Traceback (most recent call last):
  File "<stdin>", line 3, in <module>
Exception: paste fail
>>> bob = 'bob'
>>> # ------ Hello ------ #
...
...
... # some comment
... # another
... print('Counter is %s' % x)
...
Counter is 10
>>> if x == 4:
...
  File "<stdin>", line 2

    ^
SyntaxError: unexpected EOF while parsing
>>> break
  File "<stdin>", line 1
SyntaxError: 'break' outside loop
>>> a = 2
>>>

All of this demo was in VS 2019 16.1.0 preview 2.0. These problems are very old (years).

Hello again. I know the PTVS team is very busy. I would just like to reiterate what an annoyance this bug must be for everyone relying on PTVS. I would say my workflow is about 20% slower because of avoiding highlighting comments in code when sending to interactive, and the delay associated with accidentally sending comments to the interactive session causing it to choke on a snippet of code.

@joseortiz3, thank you for reporting this issue. I'm going to take a look at it.

@RaymonGulati1 Please do. This is still a big time waster for python development. I write a lot of scripts and there's a lot of "Send to Interactive" happening in my workload, sure other people do too.

@joseortiz3 I’ve looked into this and everything is working as expected. You are encountering those issues because of the way you are selecting code and sending it to interactive.
In your example #3 and 4, you are only partially selecting the first line of code . Since Python is indentation sensitive, partially selecting the first line means that the indentation for that line isn’t sent to Python which results in the errors you see in VS. This can be easily fixed by selecting the entire line before sending it to the interactive window. See examples below.
In your example #1 and #2, you are sending an additional space character (the line above “y = 1”). Instead, you should only send the line of code you want to execute in the interactive window (put cursor on that line and press "Control + Enter").
Side note: If you send multiple lines of code and all of them are indented, then VS will ignore those indentations before sending it to Python. This is what allows the interactive window to execute code that is already indented (such as the while loop in the example below)”

I don't think the current behavior is expected behavior. Handling indentation should be smarter than that.


Unable to control fan speed on GPD WIN 4

Although the current fan speed is being displayed correctly, attempting to create and apply a fan profile does not appear to result in any change to the fan speed.
OS: ChimeraOS 41, BIOS: 3.03

This problem is known for now.It seems to be caused by the ec of the fan speed not writing properly.The EC reading and writing method used in Win4 is different from other handheld.I don't have this device so I'll need some time to collaborate with others who have this device to test and fix it.

This problem has been fixed

The fan control has been fixed on the main branch, but it seems like this change has introduced a new issue. Sometimes, after using the device unplugged for a while, the battery indicator disappears from MangoHud overlay. A simple reboot doesn't fix it; I have to reset the device by holding down the power button. The battery indicator is also missing when rebooting to Windows when this happens.

I've tested it many times and it doesn't show up like this
You need to consider whether the battery is dead first
Or the battery is protected

The fan control has been fixed on the main branch, but it seems like this change has introduced a new issue. Sometimes, after using the device unplugged for a while, the battery indicator disappears from MangoHud overlay. A simple reboot doesn't fix it; I have to reset the device by holding down the power button. The battery indicator is also missing when rebooting to Windows when this happens.

I apologize for the late reply, as I have been quite busy recently. According to the feedback from my testing partner, the plugin is working well on his Win4. He mentioned that a similar issue had occurred before, but it was resolved after returning and changing the battery. Therefore, I want to make sure this is not a hardware issue first. Are you certain that this issue did not occur before the fan control was fixed? If so, you can try uninstalling the plugin and see if the issue still persists. If it is confirmed to be caused by my plugin, I will do my best to find the problem and fix it within my abilities.

I haven't encountered this issue with a version of this plugin without fan control (https://github.com/oliverchang/PowerControl), or without the plugin.

Controlling the fan speed with EC.RamRead and EC.RamWrite seems to fix the battery issue on my device.


Feature: link benchmarks to Go version

@bobheadxi : panic: runtime error: index out of range [-1] is the error.

@bobheadxi : are you fixing that upstream? Thanks for looking along.

@hazcod see https://github.com/bobheadxi/gobenchdata/commit/e21a86ba8cfd511b29b592415237628293e00c4d and https://github.com/bobheadxi/gobenchdata/releases/tag/v1.0.1

😅 unfortunately I would advise not pushing results to the same branch - https://github.com/bobheadxi/gobenchdata/issues/43#issuecomment-647562271


Infinite redirect

Install the latest version of the package and also install the latest Laravel but getting continuously redirect and unable to redirect home route after authenticated. Please help how can fix this.

Package Version: v17.1
Laravel Version: v8.54
PHP Version: v8.0

I got the same problem on fresh install of Laravel and laravel-shopify. It infinite redirects between authenticate/token and home controller.I tried with WAMP and with Laravel built-in server, still the same problem.

Laravel-Shopify v17.1
Laravel framework v7.30.4
PHP v7.3.29

Downgrading Laravel-Shopify package to v16.0.6 resolved the problem for me.

@vlkf Thanks, it's also working for me with v16.0.6, but I read an article where Shopify reject the app which is created on v16

@ClaraLeigh I have done the changes as you mentioned but still getting 302 infinite redirect loops on my local machine but it's working fine server.

@sandyrewa I don't think its in any of the releases yet, only master.
To get it working for me, I had to essentially clone the VerifyShopify middleware, and the SessionToken.php, make the changes there and use my custom middleware instead. I think I made a couple of other changes too.... I just had to get it out the door essentially, I've been so busy lately.
I'll probably make some more PRs later but I'm pretty busy till Jan

Thanks @ClaraLeigh! Your solution worked for us :-)
It took us few hours to figure out it is still not in the release, we were sure by now it's there so we didn't check.
I wonder it is not in any release yet.
Amos

@amosmos Can you please tell us what exactly needs to be done, it's still not working for us?
Thanks

@sandyrewa
Do you use one app for many stores with different api_key for each store?
I'm asking because this is the problem that this fix is fixing.
If you only have one api_key then i think you probably have a different problem and that's perhaps why you still have it after making this change.
Amos

@amosmos
Thanks for the quick reply, I'm creating a public app that uses only one api_key.

@sandyrewa So I think you have a different issue because this issue is only caused on you rely on a specific store to get a specific api_key. The code in this fix shouldn't make any effect on your case.
Just guessing here, but check these things:
Are you sure you saved the environment parameters correctly for the api_key and api_secret?
Also make sure you disabled the server environment for X-Frame-Options "SAMEORIGIN" (look here: https://github.com/osiset/laravel-shopify/wiki/Installation#appbridge)
Also, are you using your own route for the home route? If so make sure you follow the instructions to properly set the route and the view, and if you're not, make sure you remove the default home route you get from laravel, look here: https://github.com/osiset/laravel-shopify/wiki/Installation#routing
Lastly, I don't know if it's related or not, but perhaps the CSRF setting is also important for the flow (I'm not sure because the request supposed to be GET but I didn't check), so if nothing else works it's worth a shot, look here: https://github.com/osiset/laravel-shopify/wiki/Installation#csrf

@amosmos ok, thanks for your help will check

Could this be related to #1179?


Channel Master: use selected channels for copy/paste keyframes

now channel copy/paste is on selection by default
to copy/paste all channel needs to be set in the configuration


Store playground state in url, for easy sharing

Hi! Thanks so much for working on squiggle, I really like this project!
What is possibly my favorite feature is the playground, to easily play around with probability distributions without a need for dev environment.
A thing I really like about the Typescript Playground is that you can very easily share a link with your code, with no need to log in, so that everyone can very easily send snippets for debugging or showing examples.
I think it would be even more useful for Squiggle, as it would allow people to play around with models linked in posts without the need to copy paste the model, and to easily share the model in chats.
This PR is an approach for doing that, following what the TS playground does by putting the code in the location hash.
With this change the url would look something like https://www.squiggle-language.com/playground/#%7B%22showTypes%22%3Atrue%2C%22showControls%22%3Afalse%2C%22showSummary%22%3Atrue%2C%22showEditor%22%3Atrue%2C%22initialSquiggleString%22%3A%22cost%20%3D%2010%20to%2011%5Cnvalue%20%3D%200%20to%20100%5Cnvalue_over_cost%20%3D%20value%20%2F%20cost%22%7D
It also encodes some settings in the hash, since I find summary statistics very convenient to show, and every time I would share a model I would need to add to the link the instructions "click View Settings and turn on Show summary statistics".
There are probably better ways to do this, and I'm happy to make any changes. Thanks again for the awesome project!

Thanks! This has been on our list for a while.

Nice!
I now can write e.g. "Compare this with this ... "

Kudos for submitting a functional PR! Exciting to see work on this.
We had previous functionality listed here:
https://github.com/quantified-uncertainty/squiggle/issues/452

How do you actually get the string? I take it we'll need a button or something?
How about we use lz-string encoding or similar? https://github.com/pieroxy/lz-string. I'd want to get an encoding decided on before we publish something publicly, as changes will break earlier links.

With the typescript playground I just copy-paste the URL, but it might make sense to add a button as an affordance
Sure, kroki.io uses zlib e.g. via pako. I don't think we really need it, unless we expect huge stuff in the playground. But happy to implement, should be very easy

I don't think we really need it, unless we expect huge stuff in the playground. But happy to implement, should be very easy

I think that file sizes will increase over time, and also, smaller urls are just more managable for copying and pasting and stuff.

Is it ok to add a depencency on pako?

Is it ok to add a depencency on pako?

Yep, go ahead.
Eventually we'll probably make sure that people can download the non-Playground components.

I'm not really convinced about the compression. For longer models it might be handy, but for short models it produces much longer urls than base64. Especially when settings get baked in.

I think it's just the settings and the base64-ing, and the compression helps even with small models.
For the minimal code normal(0, 12):
> json = '{"showTypes":true,"showControls":false,"showSummary":false,"showEditor":true,"initialSquiggleString":"normal(0,12)"}'
> btoa(json).length
156
> b64.fromByteArray(pako.deflate(json, {level: 9})).length
132

We can probably save a bunch of characters, but I'm not sure if it's necessarily worth it

I'm not really convinced about the compression. For longer models it might be handy, but for short models it produces much longer urls than base64. Especially when settings get baked in.

This seems important to get right, whichever way we go. (Because changing would be difficult for users). I'll spend some time on it in the next few days, also curious to get other thoughts on the best compression scheme.

I guess one obvious way to increase compression: maybe we could use shorter keys for the compressed hashes? Like, we don't need the full word "initialSquiggleString", something short like "iss" could be fine. Also, if the settings are the same as the defaults, those don't need to be saved.

I agree with @Recursing, let's add key/value format to the hash, if we'll make it possible to change the format later then it doesn't matter much which one we use for now.

Cool, I'll merge if checks pass.


Push.appCollection still empty

I read the other issues on this topic and tried all of the alternatives but I still can not get this to work for GCM.
I have:
-Enabled GCM API for android
-Created the Push.config.json file with the API key and project number
However when I try pushing with meteor shell I get:
I20150714-13:37:51.687(-7)? Push: Send message "hello" via query {}
I20150714-13:37:51.689(-7)? Push: Sent message "hello" to 0 ios apps 0 android apps
I20150714-13:37:51.689(-7)? Push, GUIDE: The "Push.appCollection" is empty - No clients have registred on the server yet...
Client.js:
Push.addListener('token', function(token) {
console.log('Token: ' + JSON.stringify(token));
alert('Token: ' + JSON.stringify(token));
});
I do get a token on startup but I still don't get any notifications even after logging in.

yep I have battled this one for the past 8 hours to no avail.  Please let me know if anybody has any new ideas to debug this as my app is completely dependent on notifications.

try setting the --mobile-server flag - the client might not be able to connect to the server

I've tried --mobile-server : and without the  many times and it still does not work. I've tried many of the solutions from the other issues but I still can not get it to work. JSON.stringify reveals that I receive a token but it still does not register the client.

I meant to update this earlier but I guess I only updated another thread.  It turned out to be something with my phone and I still would like to know what it was exactly, myself.  In googling things like "can't receive Google GCM" and the like, I came across an app which you can easily find on Google Play, called Push Notification Tester.  Now, I downloaded that app, ran it, and the notification worked.  I then sent a test notification with my app and this raix package, no code changes whatsoever, and it just worked.  What can have changed?  Note that before running this tester app, I received notifications fine from, eg. meetup.com and other apps.  I do keep things like Facebook or Twitter notifications turned off, and I wondered if some per-app setting of "Off" might have been affecting my own meteor-w-raix-push app somehow...  I pretty much spent an extremely long day debugging ghosts.

Hi Chris, I downloaded the app you mentioned and tried it and I get a notification from that app but I am still unable to receive notifications on my phone with the package using meteor shell.

Shoot.  I'm trying to remember now any other little snags I hit.  I believe config.push.json needs to be at root level.  Dbl-check your api settings.  My config looks like this:
{   "gcm": {     "apiKey": "foo",     "projectNumber": 1234   },   "badge": true,   "sound": true,   "alert": true,   "production": false }
At one point someone in IRC suggested I add this (in /client), although I don't know that any of this is necessary:
`
Meteor.startup(function () {
Push.allow({
send: function(userId, notification) {
// Allow all users to send to everybody - For test only!
return true;
}
});
Push.debug = true;
/* do something with push token*/
Push.addListener('token', function(token) {
console.log('Token: ' + JSON.stringify(token));
});
Push.addListener('notification', function(notification) {
console.log('notification: ' + JSON.stringify(notification))
});
Push.addListener('startup', function(notification){
console.log('Routing Push: ' + JSON.stringify(notification));
});
});
`
Lastly, I am mup-deploying to a DO server.  IIRC, I only ever got things to work when running with --mobile-server to my mup-deployed server, and I was able to do some debugging with "DEBUG=* mup logs" after the "mup deploy".
Hopefully that can give you something new to try.  It was a very long day for me but in the end the problems seemed to be on my end, nothing with the package itself.

I've also been deploying with digital ocean and running with that mobile server, did you do anything in DO?

Nothing special really.  I use nginx proxy.  I have had the firewall turned
off since some other thing I was debugging.  I haven't tried turning the
firewall back on and tuning that yet.
On Mon, Jul 27, 2015 at 12:38 PM tltim<EMAIL_ADDRESS>wrote:

I've also been deploying with digital ocean and running with that mobile
server, did you do anything in DO?
—
Reply to this email directly or view it on GitHub
https://github.com/raix/push/issues/89#issuecomment-125320819.

And yes I "mup deploy" to a minimal $5 DO server.  Everything else with
your app is working on DO--just not notifications yet?
On Mon, Jul 27, 2015 at 12:41 PM Chris Cortese<EMAIL_ADDRESS>wrote:

Nothing special really.  I use nginx proxy.  I have had the firewall
turned off since some other thing I was debugging.  I haven't tried turning
the firewall back on and tuning that yet.
On Mon, Jul 27, 2015 at 12:38 PM tltim<EMAIL_ADDRESS>wrote:

I've also been deploying with digital ocean and running with that mobile
server, did you do anything in DO?
—
Reply to this email directly or view it on GitHub
https://github.com/raix/push/issues/89#issuecomment-125320819.

yeah everything works fine it's only notifications it also may have something to do with accounts? Because I get the token before logging in but not after. But even before logging in the Push.appCollection.find().fetch() is still empty.

I can't think of what it could be with accounts.  I'm using only
accounts-password and ian:accounts-ui-boostrap-3.  What did you see when
you ran the "DEBUG=* mup logs"?
On Mon, Jul 27, 2015 at 12:51 PM tltim<EMAIL_ADDRESS>wrote:

yeah everything works fine it's only notifications it also may have
something to do with accounts? Because I get the token before logging in
but not after. But even before logging in the
Push.appCollection.find().fetch() is still empty.
—
Reply to this email directly or view it on GitHub
https://github.com/raix/push/issues/89#issuecomment-125324551.


fix property naming in the cognito event docs

Description of changes:

just fixed a small typo :)

Description of how you validated changes:

you can see the correct property name in the line above

Checklist:

[ ] Write/update tests
[ ] make pr passes
[ ] Update documentation
[ ] Verify transformed template deploys and application functions as expected
[ ] Add/update example to examples/2016-10-31

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

Codecov Report

Merging #1331 into master will decrease coverage by 0.62%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1331      +/-   ##
==========================================
- Coverage    95.3%   94.67%   -0.63%
==========================================
  Files          78       77       -1
  Lines        5171     4129    -1042
  Branches     1190      815     -375
==========================================
- Hits         4928     3909    -1019
+ Misses        113      108       -5
+ Partials      130      112      -18

Impacted Files
Coverage Δ

samtranslator/swagger/swagger.py
92.57% <0%> (-2.91%)
:arrow_down:

samtranslator/plugins/api/implicit_api_plugin.py
95.57% <0%> (-2.66%)
:arrow_down:

samtranslator/plugins/exceptions.py
83.33% <0%> (-2.39%)
:arrow_down:

samtranslator/model/eventsources/push.py
90.61% <0%> (-1.43%)
:arrow_down:

samtranslator/plugins/globals/globals.py
99.02% <0%> (-0.98%)
:arrow_down:

samtranslator/model/api/api_generator.py
96.72% <0%> (-0.92%)
:arrow_down:

...el/preferences/deployment_preference_collection.py
91.13% <0%> (-0.91%)
:arrow_down:

samtranslator/open_api/open_api.py
93.56% <0%> (-0.83%)
:arrow_down:

samtranslator/model/apigateway.py
97.79% <0%> (-0.66%)
:arrow_down:

samtranslator/translator/arn_generator.py
86.95% <0%> (-0.55%)
:arrow_down:

... and 23 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 93b6ac0...614aca8. Read the comment docs.

Closing this PR as this was fixed in #1233


Output channels are not fully disposed

VSCode Version: 1.9.1
OS Version: OSX

Steps to Reproduce:
  const outputChannel = window.createOutputChannel(`A channel`);
  outputChannel.show();
  setTimeout(() => {
    outputChannel.dispose()
  }, 1000);

The channel is left in the output channels list in the UI:

I don't have a way (anymore?) to remove an output channel

@jrieken I connected it end to end with the api. Please take a look and let me know if it works for you.

Looks good. Thanks 👏

@isidorn,
It will be included into the release 1.10, won't it?

@KalitaAlexey yes.

@isidorn,
That's cool.
Thank you.

Disposing an output channel switches to another channel in the tab select box, but it still shows the content of the disposed output channel.

re https://github.com/Microsoft/vscode/issues/20703#issuecomment-283581882 fyi @isidorn

Created new issue https://github.com/Microsoft/vscode/issues/21734


[BUG] Timber\Menu, current item classes not working/showing

Expected Behavior
If they are on the current page, for example: "Home", then in the menu for the current element, put the default classes of wp_nav_menu current element, for example: "menu-item menu-item-type-post_type menu-item-object-page menu-item-26 current-menu-item"
Actual behavior
Now, even if I am on the home page in the menu, the active class is not displayed but only: "menu-item menu-item-type-post_type menu-item-object-page menu-item-26"
Steps to reproduce behavior
Just have a menu with Timner\Menu(), i dont know how to reproduce, it simply dont working as expected
Notes
I even dezactivate all other plugins like: acf pro, acfe, permalink manager pro, queyr monitor, yoast, the CLASS of current menu item not working, if we use {{ fn('wp_nav_menu') }} then the current menu item class working without problems.. i even downgrade the plugin to 1.20.0 and still not working
What version of Timber are you using?
1.22.1
What version of WordPress are you using?
6.2.2
What version of PHP are you using?
7.4.33
How did you install Timber?
Installed or updated the Timber plugin from the WordPress plugin repository

Hi @fgdaniel ,
Can you provide us the code that you used to setup the Timber/Menu() and how you are displaying it inside your twig file? Can you also let us know if you used a menu name, menu id or a menu location to setup the menu class?

I dont know if its from my code or a simply bug, why i say that, because i have another sites with almost the same code, same plugins and same timber version and works fine, this problem its only on two of my client sites and of course same code of menus and initiated them
header.twig the menu.twig now its commented because wp_nav_menu() working fine.

			{% include "components/site-logo.twig" with {
				class: 'py-[33px]',
				imageClass: 'max-w-full w-auto h-[84px]'
			} %}
			{% include "components/mobile-hamburger.twig" with {
				class: 'flex flex-col items-center justify-center w-8 h-8 ml-auto text-2xl leading-3 outline-none md:hidden focus:outline-none'
			} %}

			{# {% include 'components/primary-menu/menu.twig' with {
				items: primary_menu.items,
				class: 'primary-menu list-unstyled mb-0',
				breakpoint: 768
			} %} #}
			{{ fn('wp_nav_menu', {
				menu: 'primary_menu',
				menu_id: 'primary-menu',
				menu_class: 'primary-menu',
				container: false,
			}) }}

```

menu.twig
```{% if items %}
	{% set class = class|class_unique %}

		{% for item in items %}
			{% include "components/primary-menu/item.twig" with {
				index: item.id,
				item: item
			} %}
		{% endfor %}

{% endif %}```

item.twig
```{% set classes = item.classes %}
{% if item.children %}
	{% set classes = classes|merge(['has-dropdown']) %}
{% endif %}
{% if item.current %}
	{% set classes = classes|merge(['active']) %}
{% endif %}

	{{- item.title -}}
	{% if item.children %}
		<button class="dropdown-toggle-button">

				+

				-

		</button>
		{% include 'components/primary-menu/dropdown.twig' with {
			index: item.id,
			items: item.children,
			level: 1
		} %}
	{% endif %}

```

dropdown.twig
```

	{% for item in items %}
		{% include 'components/primary-menu/dropdown-item.twig' with {
			index: item.id,
			item: item,
			level: level + loop.index
		} %}
	{% endfor %}

```

dropdown-item.twig
```{% set classes = item.classes|array_filter %}
{% if item.children %}
	{% set classes = classes|merge(['has-dropdown']) %}
{% endif %}
{% if item.current %}
	{% set classes = classes|merge(['active']) %}
{% endif %}

	{{- item.title -}}
	{% if item.children %}
		<button class="dropdown-toggle-button">

				+

				-

		</button>
		{% include 'components/primary-menu/dropdown.twig' with {
			index: index,
			items: item.children,
			level: level
		} %}
	{% endif %}

```

theme.php where i set supports and register menus

```<?php

// Exit if accessed directly.
if ( ! defined( 'ABSPATH' ) ) {
	return;
}

/**
 * Set up the theme.
 *
 * Loads the text domain, adds theme support for various features,
 * registers menus, and defines image sizes.
 */
function theme_setup() {

	// Load text domain
	// This function is called once to prevent conflicts during translations
	load_theme_textdomain( TEXT_DOMAIN );

	// Let WordPress manage the document title.
	// This theme does not use a hard-coded <title> tag in the document head.
	// WordPress will provide it for us.
	add_theme_support( 'title-tag' );

	// Switch default core markup for search form, comment form, and comments to output valid HTML5.
	add_theme_support( 'html5', array(
		'search-form',
		'comment-form',
		'comment-list',
		'gallery',
		'caption',
		'script',
		'style',
	) );

	// Enable support for Post Formats.
	add_theme_support( 'post-formats', array(
		'aside',
		'image',
		'video',
		'quote',
		'link',
		'gallery',
		'status',
		'audio',
		'chat',
	) );

	// Add support for site logo.
	add_theme_support( 'custom-logo' );

	// Add support for full and wide align images.
	add_theme_support( 'align-wide' );

	// Add support for responsive embeds.
	add_theme_support( 'responsive-embeds' );

	// Add support for menus.
	add_theme_support( 'menus' );

	// Enable support for Post Thumbnails on posts and pages.
	add_theme_support( 'post-thumbnails' );

	// Register Primary Menu.
	register_nav_menu( 'primary_menu', __( 'Primary Menu', TEXT_DOMAIN ) );

	// Register image size.
	add_image_size( 'featured-image', 540, 380, false );
}

add_action( 'after_setup_theme', 'theme_setup' );

/**
 * Registers style and scripts for the theme.
 */
function theme_add_style_scripts() {
	wp_enqueue_script("jquery");
}
add_action( 'wp_enqueue_scripts', 'theme_add_style_scripts' );

/**
 * Registers an editor stylesheet for the theme.
 */
function theme_add_editor_styles() {
	add_editor_style( 'editor-style.css' );
}
add_action( 'admin_init', 'theme_add_editor_styles' );```

context.php where menu it set to global context

```/**
 * Add to global context
 */
add_filter( 'timber/context', function ( $context ) {
	$context['google_map_api'] = 'AIzaSyBGcxscdlCRwgDx8rE1oxGgLaBAghH0iqs'; // Api de Dev
	$context['theme']          = context_theme();
	$context['logo']           = context_theme_logo();
	$context['primary_menu']   = new Timber\Menu('primary_menu');
	$context['options']        = context_options();
	// $context['formular_inscriere'] = array(
	// 	'perioade' => context_formular_inscriere_perioade()
	// );

	return $context;
} );```

This might be caused because both the menu name and location are called primary_menu it seems. Can you change, for example, the location name, and see if that solves your issue? @fgdaniel

This might be caused because both the menu name and location are called primary_menu it seems. Can you change, for example, the location name, and see if that solves your issue? @fgdaniel

It still doesn't work, I renamed primary_menu to primary in the register_nav_menu() function ex: register_nav_menu( 'primary', __( 'Primary Menu', TEXT_DOMAIN ) );, and in the global context it remained 'primary_menu', but I made it work in a different way, I had to make a function for the menu and include it in twig functions, you have the php and twig code below
twig.php
<?php

// Exit if accessed directly.
if ( ! defined( 'ABSPATH' ) ) {
	return;
}

function theme_twig_menu($id) {
	$menu = new Timber\Menu( $id );

	return $menu->get_items();
}

/**
 * Add to global twig
 */
add_filter( 'timber/twig', function ( $twig ) {
	$twig->addFunction( new Timber\Twig_Function( 'Menu', 'theme_twig_menu' ));
	/* rest of code *
	return $twig;
} );

header.twig

			{% include "components/site-logo.twig" with {
				class: 'py-[33px]',
				imageClass: 'max-w-full w-auto h-[84px]'
			} %}
			{% include "components/mobile-hamburger.twig" with {
				class: 'flex flex-col items-center justify-center w-8 h-8 ml-auto text-2xl leading-3 outline-none md:hidden focus:outline-none'
			} %}

			{% include 'components/primary-menu/menu.twig' with {
				items: Menu('primary_menu'),
				class: 'primary-menu list-unstyled mb-0',
				breakpoint: 768
			} %}
			{# {{ fn('wp_nav_menu', {
				menu: 'primary_menu',
				menu_id: 'primary-menu',
				menu_class: 'primary-menu',
				container: false,
			}) }} #}

I don't know why it does that, it was supposed to work like that... I think something is happening with the global context and I can't figure out why it doesn't pick up certain values from wordpress, like I can't put the value of get_query_var('paged') in the global context even though query_var paged exists (this is an example) it might be related in a bizarre way

Hi @fgdaniel ,
Instead of using primary_menu.items could you try using primary_menu.get_items? I thing this is a shorter way for you to do this. From what I understand from the Menu class is that .items should work, but for some reason it doesn't. I will take a deeper dive locally in the future to see what causes this.
{# {% include 'components/primary-menu/menu.twig' with {
				items: primary_menu.get_items,
				class: 'primary-menu list-unstyled mb-0',
				breakpoint: 768
			} %} #}

Hi @fgdaniel ,
Instead of using primary_menu.items could you try using primary_menu.get_items? I thing this is a shorter way for you to do the same without using the extra mapped function. From what I understand from the Menu class is that .items should work, but for some reason it doesn't. I will take a deeper dive locally in the future to see what causes this.
{# {% include 'components/primary-menu/menu.twig' with {
				items: primary_menu.get_items,
				class: 'primary-menu list-unstyled mb-0',
				breakpoint: 768
			} %} #}

Unfortunately, it doesn't work no matter what I do, it only works through my method with the Menu function in twig.. I also tested the 'Timber Starter Theme' and it works there without problems, I made the theme I use to be almost identical to the 'Timber Starter Theme', you have a link to my default theme here (I don't know if it's allowed) -> https://github.com/fgdaniel/wp-timber

I tried to reproduce your issue @fgdaniel, but I wasn't able to do so.
What I did to reproduce:

Setup a starter timber theme on PHP8.0 with Timber installed via composer, instead of a plugin. This should not make a difference in this regard.
register a menu location with the slug primary_menu
add 'primary_menu' => new Menu('primary_menu') to the context
Created a new menu via the wordpress interface and linked it to the new location.
dumped the menu in index.twig

            {{ dump(primary_menu.items) }}

I see classes popping up like: current-menu-item
So I am pretty sure it must have something to do with your codebase. I would advise you to make the issue as small as possible. Remove all extra hooks/filters from your theme and then start testing.

try another menu name in WordPress
link it to another location, etc.

Hopefully you will be able to find out what the issue might be.

I tried to reproduce your issue @fgdaniel, but I wasn't able to do so.
What I did to reproduce:

Setup a starter timber theme on PHP8.0 with Timber 1.22.1 installed via composer, instead of a plugin. This should not make a difference in this regard.

register a menu location with the slug primary_menu

add 'primary_menu' => new Menu('primary_menu') to the context

Created a new menu via the wordpress interface and linked it to the new location.

dumped the menu in index.twig

            {{ dump(primary_menu.items) }}

I see classes popping up like: current-menu-item
So I am pretty sure it must have something to do with your codebase. I would advise you to make the issue as small as possible. Remove all extra hooks/filters from your theme and then start testing.

try another menu name in WordPress

link it to another location, etc.

Hopefully you will be able to find out what the issue might be.

Sorry for spam.. i tried to disable all my custom filters/functions and still dont work, i will try again to reconstruct my starter thene from your starter theme to see if this world, and the last thing, you you have time and you want to git clone my starter theme and test with it to see the problem, but thanks you all for this beatiful plugin i love it🥰

Your theme is no longer publicly available so we cannot take a look at it. That said, it's a bit out of the scope of the support for Timber anyway. I will close this ticket for now. If you still run into it, make a repo with the codebase as small as possible so we know for sure we are looking at a Timber related issue.
An another note, I would highly recommend switching to the composer based version of Timber as soon as possible so you are ready for Timber 2.
Best of luck!

@fgdaniel,
You posted your API key here in the initial post. I removed it since, but it is advised that you lock down/secure/rotate that API key if still in use.


fix: Use silent nonTTYRenderer to avoid dupl output

The error output, if any, was being printed thrice in non-TTY consoles. Use of the silent renderer for non-TTY consoles fixes the issue partially and should be helpful at least for some use-cases. If --debug flag is passed or if the renderer option is set to verbose in the lint-staged config(option not documented?), the output will still be duplicated. Not sure how that could be fixed.
Closes #308.
Related to #142.

This hides useful logs like Running tasks for *.js [started] 😞

@sudo-suhas I think tests are relying on it to be verbose so we'll need to update tests to make it explicit.

I saw that. I am not sure if we should land this fix. The following won't get printed :
LOG Running tasks for *.js [started]
LOG Running tasks for *.js [skipped]
LOG → No staged files match *.js

Closed via #421.


into(Spark/HDFS, SQL DBs)

migration from blaze
https://github.com/ContinuumIO/blaze/issues/582
reposting for ease of use:
from @chdoig
As Spark is becoming a popular backend, it's a common use case for people to want to transfer their datasets in DBs to a Spark/HDFS cluster.
It would be nice to have an easy interface for end-users to transfer their tables in DBs to a Cluster.
into(Spark/HDFS, SQL DBs)
A lot of people are talking now about tachyon, maybe worth taking a look:
http://tachyon-project.org/
http://ampcamp.berkeley.edu/big-data-mini-course/tachyon.html
This might be related with @quasiben work on SparkSQL. Maybe a barrier for people to star using SparkSQL is how they should make that transfer since:

A SchemaRDD can be created from an existing RDD, Parquet file, a JSON dataset, or by running HiveQL against data stored in Apache Hive.

But I'm not able to find how you make that connection from existing SQL DBs:
http://spark.apache.org/docs/latest/sql-programming-guide.html
cc: @mrocklin

FWIW, Spark can use the JDBC connector to do computation. It looks like this:
sc = SparkContext(...)
sql = HiveContext(...)
df = sql.load("jdbc", url="jdbc:postgresql:dbserver", dbtable="schema.tablename")

I don't think this would be that tricky. The function would look something like this:
@append.register(SQLContext, sa.Table)
def sql_to_sparksql(ctx, tb, **kwargs):
    url = connection_string_from_engine(tb.bind)  # <- not implemented, but i think easy
    dbtable = [tb.name]
    if tb.schema is not None:
        dbtable.insert(0, tb.schema)
    return ctx.load('jdbc', url=url, dbtable='.'.join(dbtable))

here's the spark docs on using jdbc
https://spark.apache.org/docs/1.3.0/sql-programming-guide.html#jdbc-to-other-databases

If anyone is interested in implementing this, you'll have to install libpostgresql-jdbc-java for testing on Travis-CI

The postgres connection string syntax can be found here:
https://jdbc.postgresql.org/documentation/91/connect.html
one really annoying thing is that it looks like jdbc connection strings aren't standard and can be different for different vendors
here's oracle's:
http://docs.oracle.com/cd/B28359_01/java.111/b31224/urls.htm#BEIJFHHB


New version: BugReporting v0.2.5

Registering package: BugReporting
Repository: https://github.com/JuliaLang/BugReporting.jl
Created by: @maleadt
Version: v0.2.5
Commit: 3d97b2edfebb5fdc6a1c31039467ae370df4f480
Reviewed by: @maleadt
Reference: https://github.com/JuliaLang/BugReporting.jl/commit/3d97b2edfebb5fdc6a1c31039467ae370df4f480#commitcomment-79443485
Description: Streamlines bug reporting for julia

I missed that the previous version hadn't successfully been tagged: https://github.com/JuliaRegistries/General/pull/64920


Add the ability to switch backends

Currently the backend is automatically chosen in the following order of preference.
CUDA  > OpenCL > CPU
Make it possible for users to choose a backend of their choice.

Fixed in master.
af.backend.set(name) # name can be 'cpu', 'cuda', 'opencl'

An exception is thrown if the one tries to change backend after an af.Array() is created. This can can be silenced by using the following:
af.backend.set(name, unsafe=True)

Be careful when toggling unsafe = True. Passing arrays created on different backend to the current library will cause issues.


Missing questions in Amazon article

Coding round and technical interview round questions are almost non-existent in this article. Anyone taking this issue should add some questions to all the aforementioned or blank sections while keeping contributing guidelines in mind. Geeksforgeeks has lot of articles regarding Amazon. It would be great if someone properly enhance the article rather than symbolic contributon of 1 question or fixing typos.

I would like to do this issue

I would like to work on this. Can you please assign this to me?

Hey! I would love to work on this issue.

Hey! I would love to work on this issue.

Assigned you the issue. Do create a good article. Happy Coding✌


Refactor partysocket to use native EventTarget

This is an alternative to #171

This has broken the tests because the events ws emits aren't the same instance as the jsdom Events and so dispatchEvent fails...
Not sure how best to solve this tbh.

Also just struggling with these tests in general... Are they marked as flaky because of port in use errors? If so I think I have an idea to fix that.

yeah the tests were disabled because they're flaky af :(

This has broken the tests because the events ws emits aren't the same instance as the jsdom Events and so dispatchEvent fails...

ughhh

Btw I rebased this branch on #172 and all the tests pass ☺

love to see it

did you mean to leave this as a draft? happy to review and land if you'd like

Only because the tests were failing before, ready now!

LFGGGG you're a legend @tom-sherman!

sadly this breaks support for react-native, where the DOM Event classes are not defined 😭

Oh whoops. Any recommended workarounds/fixes?

You can try shimming it: https://www.npmjs.com/package/event-target-shim
React Native are working towards wider support for web standards but I couldn't find an issue for this API.

@brentvatne could you explain how this worked without the shim? https://github.com/expo/todo-sync-example/pull/2 Does expo add the shim?


Is it forbidden to deploy a branch name has slashes?

https://github.com/Unitech/pm2-deploy/blob/a3398fdd56afb39a43039a7864635688843b39e6/deploy#L210
I have a branch named as  origin/deploy/test . When I want to setup for a new server. It keeps complain:
Cloning into '/data/web/xxxx/source'...
warning: Could not find remote branch test to clone.
fatal: Remote branch daiwhea not found in upstream origin

Can I deploy a branch which name containes slashes[/] ?
Change the code above to local branch=${ref#*/} will be ok.
Thanks.

Feel free to make a PR so we can publish the fix :)

yep, i think we can fix this.


How can we use versioning?

https://laravel.com/docs/5.6/mix#versioning-and-cache-busting

Why would you version pug files?

Closing due to no reasonable scenario where one would version a blade view.


Unexpected character '`'  problem

Hi,
thank u first, npm run build problem

ERROR in static/js/vendor.45cf410bef1d1ee48a7d.js from UglifyJs
Unexpected character '`' [./~/vue-masked-input/src/MaskedInput.js:7,0][static/js/vendor.45cf410bef1d1ee48a7d.js:61918,12]

what is the problem ? how to fix ? Ie not working app

I have same problem

same here!

What version can we use now?

+1

Fork: Solving unexpected character '`' problem in webpack's uglifyjs

Fixed in 0.5.0, please update

@niksmr do you remember how did you fix it ? I have the same issue on another project and I do know how to resolve it. I would like to make a pull request but I have no idea was is causing the issue :/

@victorboissiere do you use 0.5.0^ version? The reason for the issue was the use of ES6-template literals. Starting with version 0.5.0, they are no longer there.

I was talking about the same error happening on a completely different package. I guess you just removed the ES6-template literals to fix it! Thank you and sorry to ask here I was so desperate :/


Any way to show multiple hits in the same note when querying / searching?

The query tool is a nice way to find content in notes. I like the card view which shows the queried string in the context of the note.
However, it can be difficult to find what you want when a note contains the string more than once, since Tangent only shows the first 'hit' in each file and gives no indication there are other hits later in the file.
Consider the following Markdown file as an example:
There is some content here.

### FINDME here.

Lorem ipsum dolor sit amet, consectetuer adipiscing elit.

Maecenas porttitor congue massa.

Fusce posuere, magna sed pulvinar ultricies, purus lectus malesuada libero, sit amet commodo magna eros quis urna.

Nunc viverra imperdiet enim.

Fusce est.

Vivamus a tellus.

Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas.

### FINDME here too.

Proin pharetra nonummy pede.

Mauris et orci.

Aenean nec lorem.

When I query for the string FINDME, Tangent shows clearly the first sentence containing it, but there's no indication that a 2nd sentence also contains FINDME in the note:

Of course I can scroll down within the note while in query view to find the other FINDME reference highlighted in green, but this doesn't work well when there are many notes to look through (or long notes).
As an example of other software that does this nicely, I mention Zettlr. Using the same query approach there, I see:

Clicking on the line in Zettlr's query sidepanel jumps to the line in the note itself, so it's very easy to find all times a string appears in a note and jump to it.
Perhaps Tangent could consider enhancing the query results view to include the case for multiple hits within a single file. Maybe each card (i.e. file) could have a number on the preview showing "number of hits", or even a forwards/backwards button so the user can cycle through them quickly while previewing the little card? Just an idea.

Yes, definitely a limitation. I like your notion of having some way of moving between highlighted sections. That's a good idea.
One eventual improvement I want to make to queries is being able to display things at a sub-note level, e.g. Lines with .... A significant piece of work, but it would multiply the utility of the query system.

Thanks and yes, that proposed sub-note level querying would be an amazing feature!


allowfullscreen on iframe to work on firefox

hello, i needed to add "allowfullscreen" on iframes to work on firefox.
"iframe src="multiplay.html?me=u1" height="300" width="300" allowfullscreen></iframe"
thanks for your work.

Ah, cool - thank you! Will put it in the code.

Actually I found a better solution: Firefox doesn't need to switch to Fullscreen anymore, if you need Pointerlock. So I commented out the whole fullscreen part, which removes a whole lot of workaround code.
Again: thanks for bringing this to my attention!


Add back missing javadoc

Issue Description
Examples:
https://github.com/eclipse/deeplearning4j/commit/4766032444de8e0c2c3389270576bb6e7c466211#diff-49067a9948bb62443545e65cce8559402d08265c92932b7db650598e9418c816
https://github.com/eclipse/deeplearning4j/blob/c715aea405980eb043420af86c671032bbb78ec6/nd4j/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/lossfunctions/impl/LossMultiLabel.java
Please describe our issue, along with:

expected behavior
encountered behavior

Version Information
Please indicate relevant versions, including, if relevant:

Deeplearning4j version
Platform information (OS, etc)
CUDA version, if used
NVIDIA driver version, if in use

Additional Information
Where applicable, please also provide:

Full log or exception stack trace (ideally in a Gist: gist.github.com)
pom.xml file or similar (also in a Gist)

Contributing
If you'd like to help us fix the issue by contributing some code, but would
like guidance or help in doing so, please mention it!

I would like to contribute to this.

@RealYusufIsmail follow this guide on how to submit pull requests: https://www.freecodecamp.org/news/how-to-make-your-first-pull-request-on-github-3/#:~:text=Go to your repository on,Congratulations!
Start with understanding how to clone a repository, fork it, and make a small change then send a pull request from your fork to this base repository with your changes. if you need other help let me know.

@RealYusufIsmail follow this guide on how to submit pull requests: https://www.freecodecamp.org/news/how-to-make-your-first-pull-request-on-github-3/#:~:text=Go to your repository on,Congratulations!
Start with understanding how to clone a repository, fork it, and make a small change then send a pull request from your fork to this base repository with your changes. if you need other help let me know.

I know how to use Github. What do I need to change. There is not enough detail in the description


[patch] Allow log prefixes to be properly disabled when using custom logger

See issue #14 EDIT: Moved to sails#4036.
These changes create the following behavior (I made sure to test each scenario described below):
Without a custom logger, everything should work the same, except that it only sets the default prefix if options.prefix is not false or null.
With a custom logger and no prefix defined, no prefixes are set and relies on the custom logger to handle its on prefixes (probably preferred behavior with most custom loggers).
With a custom logger and options.prefix defined to true it adds the default prefixes before passing the message to the custom logger (this option might result in duplicate prefixes like described in the issue if the custom logger has its own prefixes, but it would be useful if the custom logger did not have it's own prefixes).
With a custom logger and options.prefix is defined as a string, the string prefix is prepended to the message sent to the custom logger.

@felixmc is it possible to add a test for that scenario?

👍

@RWOverdijk see https://github.com/balderdashy/captains-log/issues/14#issuecomment-171563335 for more recent discussion on it

Update:  I'm leaving this open pending https://github.com/balderdashy/captains-log/issues/14#issuecomment-171563335.  If anyone would like to do that, please go ahead!

@mikermcneil I think this one was pending a change in sails-docs that is already done. Probably can be merged?


Create remote configurations fail at developer portal

Describe the issue:
Error while submitting the configuration detail and fail the submit
blob:https://localhost:9443/96af06c8-2c41-406e-8459-2d6e86e35bca:54 Uncaught TypeError: Cannot read property 'startsWith' of undefined
    at _.forEach.i (blob:https://localhost:9443/96af06c8-2c41-406e-8459-2d6e86e35bca:54)
    at Array.forEach (<anonymous>)
    at Object.ft [as httpRequest] (blob:https://localhost:9443/96af06c8-2c41-406e-8459-2d6e86e35bca:54)
    at self.onmessage (blob:https://localhost:9443/96af06c8-2c41-406e-8459-2d6e86e35bca:54)

How to reproduce:

Log in to console
Navigate to the developer portal and the Remote repo config
Click to add new remote repository config button
Add configuration details and click "Create configurations' button

Expected behavior:
Configuration should be submit.
Environment information

Product Version: [e.g., IS 5.11.0-M31]

Optional Fields
Related issues:

Suggested labels:

This feature was temporarily hidden[1] due to ongoing backend improvements.
[1] https://github.com/wso2/identity-apps/pull/1190

This feature is enabled again.
If you select the webhook option, it fails with the error,
{"code":"UE-10000","message":"Invalid Request.","description":"Provided request body content is not in the expected format.","traceId":"a58b1809-2143-4a97-8210-0f4bcad81ea3"}
Failed request body: webhook
{
  "actionListener": {
    "attributes": {
      "frequency": null
    },
    "type": "WEBHOOK"
  },
  "configurationDeployer": {
    "attributes": {},
    "type": "SP"
  },
  "isEnabled": true,
  "remoteFetchName": "ApplicationConfigurationRepository",
  "repositoryManager": {
    "attributes": {
      "branch": "master",
      "directory": "test",
      "uri": "https://github.com/wso2/identity-apps"
    },
    "type": "GIT"
  }
}

Success request body: polling,
{
  "actionListener": {
    "attributes": {
      "frequency": 60
    },
    "type": "POLLING"
  },
  "configurationDeployer": {
    "attributes": {},
    "type": "SP"
  },
  "isEnabled": true,
  "remoteFetchName": "ApplicationConfigurationRepository",
  "repositoryManager": {
    "attributes": {
      "accessToken": "dasdaasdad",
      "branch": "master",
      "directory": "test",
      "uri": "https://github.com/wso2/identity-apps",
      "username": "wso2carbon"
    },
    "type": "GIT"
  }
}

So the issue can be due to accessToken and username attributes missing in the request.

{
  "actionListener": {
    "attributes": {
      "frequency": null
    },
    "type": "WEB_HOOK"
  },
  "configurationDeployer": {
    "attributes": {},
    "type": "SP"
  },
  "isEnabled": true,
  "remoteFetchName": "ApplicationConfigurationRepository",
  "repositoryManager": {
    "attributes": {
      "branch": "master",
      "directory": "test",
      "uri": "https://github.com/wso2/identity-apps"
    },
    "type": "GIT"
  }
}

This will work. The issue is WEB_HOOK, not WEBHOOK. You can find the constant here.
https://github.com/wso2-extensions/identity-fetch-remote/blob/179437a3aaabacfac44f3ee0491fc25d0b35c89b/components/org.wso2.carbon.identity.remotefetch.common/src/main/java/org/wso2/carbon/identity/remotefetch/common/RemoteFetchConstants.java#L44


Doesn't Work

I just installed this, it's my favorite theme on all my editors. But When I try it with codeoss it doesn't stick. I'm not sure why, but when I click "use theme" it takes, but when I switch back to my code editor it switches right back. Any ideas? The included themes are aweful. This one is the only one I've ever liked;

Hi @ty-meador. It's great to hear that you enjoy using my theme, and I appreciate you taking the time to provide feedback.
I'm sorry to hear that you're experiencing issues with Codeoss, and I want to let you know that I have only tested the theme on VSCodium and VSCode.
Unfortunately, I'm unable to dedicate time to improving it further at the moment due to other commitments.
That being said, I plan to rename the theme and make it available for use on different editors in the future.
Rest assured that I will address this issue once I make those changes.
Thank you for your understanding and patience.


feature: Add a duplicating option for slides, so that it direclty creates a copy of the slides in notion.

Problem:
When downloading the slides from dailcode, some part of the code  is clipping.
Solution:
Adding a feature, which duplicates the slides direclty into notion site, so that the notes persist in user's notion account.
How To Add:
Open the below notes link, you will find out the duplicating option built by notion, add the same option to slides on dailycode.
https://quickest-juniper-f9c.notion.site/Cohort-2-0-FullStack-Open-Source-6b6c2a9f1282499aba4782b88bf7e204
https://github.com/code100x/daily-code/assets/52690028/05cc7389-bf87-46b2-b29d-64c0672003b6

sorry for offtopic question
im getting this error after running the repo please help, not able to run the code due to this error

after running this command chmod +x ./setupDB.sh
./setupDB.sh

@ayesparshh have installed all the dependencies?
Just follow the set-up steps in readme...

yes after yarn install, cd packages/db then chmod +x ./setupDB.sh
./setupDB.sh ,this command doesn't work on migrating the db, but successfully start a container

@ajaykasturi so the feature would be like the duplicate to notion option right next to the Download button?
And after click it copies all slides into user's notion doc?

@ajaykasturi so the feature would be like the duplicate to notion option right next to the Download button? And after click it copies all slides into user's notion doc?

Yes

have to link with the notion account then also somehow

@ajaykasturi  https://quickest-juniper-f9c.notion.site/Cohort-2-0-FullStack-Open-Source-6b6c2a9f1282499aba4782b88bf7e204
In this example the Notion documents are only available for viewing and are not saved to personal accounts. To make the documents available in personal accounts, you first need to create a master account where all the slides are uploaded. From there, you can use Notion's "Duplicate" feature to allow users to copy the slides into their own accounts. If you are aware of any alternative methods, please share them. Otherwise, I will proceed with this implementation.


Cannot run postfix test interactively

Describe the bug
The service postfix-setup fails but only when run from the interactive test driver.
Steps To Reproduce
$ nix-build -A nixosTests.postfix
[...succeeds...]
$ nix-build -A nixosTests.postfix.driverInteractive
$ result/bin/nixos-test-driver
>>> test_script()
[...]
unit "postfix.service" is inactive and there are no pending jobs
>>> print(*machine.execute("systemctl status postfix-setup"))
3 × postfix-setup.service - Setup for Postfix mail server
     Loaded: loaded (/etc/systemd/system/postfix-setup.service; linked; preset: enabled)
     Active: failed (Result: exit-code) since Wed 2022-11-16 17:06:28 UTC; 36s ago
    Process: 691 ExecStart=/nix/store/87dymimi5wnwkz00sj9vdab66cxynzi3-unit-script-postfix-setup-start/bin/postfix-setup-start (code=exited, status=1/FAILURE)
   Main PID: 691 (code=exited, status=1/FAILURE)
         IP: 0B in, 0B out
        CPU: 48ms

Nov 16 17:06:26 machine systemd[1]: Starting Setup for Postfix mail server...
Nov 16 17:06:27 machine postfix-setup-start[727]: postalias: fatal: open database /var/lib/postfix/conf/aliases.db: Permission denied
Nov 16 17:06:27 machine postfix/postalias[727]: fatal: open database /var/lib/postfix/conf/aliases.db: Permission denied
Nov 16 17:06:28 machine systemd[1]: postfix-setup.service: Main process exited, code=exited, status=1/FAILURE
Nov 16 17:06:28 machine systemd[1]: postfix-setup.service: Failed with result 'exit-code'.
Nov 16 17:06:28 machine systemd[1]: Failed to start Setup for Postfix mail server.

Expected behavior
Test should succeed, regardless whether it's run interactively or not.
Screenshots
N/A
Additional context
Probably related but doesn't have a lot of context and is closed without comment: https://github.com/NixOS/nixpkgs/issues/25113
Notify maintainers

@mweinelt @globin @dotlambda @nlewo
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

system: "x86_64-linux"
host os: Linux 5.15.0-52-generic, Ubuntu, 20.04.5 LTS (Focal Fossa), nobuild
multi-user?: no
sandbox: yes
version: nix-env (Nix) 2.8.1
channels(henri): "nixpkgs"
nixpkgs: /home/henri/.nix-defexpr/channels/nixpkgs

The problem only occurs on single-user installations. I cannot reproduce on NixOS. As mentioned in the older issue this is a permission problem.


use BoundingBox for geo queries instead of distance / radius

Currently, using instant-meilisearch combined with a map view, we have some issues related to the use of geo distance queries. Because the resulting shape is a circle, the shape of the map shown to users will either not show all results, or not show some of the results. This becomes a problem when the users can filter the results by moving the map.
The proper solution here is to use a bounding box, instead of geo radius queries. These are not yet officially supported by meilisearch, but are planned for the 1.1 release in march.
I think that when 1.1 is released, instant-meilisearch should be updated to use bounding box queries by default. This probably involves making changes to  geo-rules-adapter.ts.

Hello @joepio
Thank you very much for the feedback. Indeed this feature is in the works and will be released with v1.1, as you've mentioned above. The team is well aware of this, and there will be changes to ensure compatibility with instant-meilisearch.
I hope this helps. Thanks again.


HTTP code is 502 which indicates error: undefined - ContainerInspect failed

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

No information provided - Closing


Fix taskrelation link path

The link path was changed a while ago. Perhaps the new path was lost in some conflict

I noticed tasks.controller.ts also has a link with /tasks/ still in it, can you add that to this PR too

Good catch, added!


Closes #49

Hi @marlycormar,
This fix notes that in the 1st printed edition of Exercise 2.8.5 (Book Exercise 1.5) the typo bug (summry) was corrected so there are only two bugs to catch.
Please let me know if you need anything else or if I messed up in the PR.
Thanks again.
Kurtis

Thanks for letting me know, @jonthegeek, and no problem. If there's something I can help with, please let me know.


Update guidelines-for-app-settings.md

Removed outdated Windows 8.x introduction screenshot.

Updated "Color mode" settings section for terminology consistency with Settings > Personalization > Color. "App mode" is the official and most straightforward term.

Corrected reason for "Windows color settings" link. Theme is different from app mode, as the former, respectively, is a combination of tunes, background images, color settings (except for app mode), and mouse pointers.

About & feedback section - clarified ideal implementation and added guidelines for "Legal" subheader to further improve organization and consistency.

@mijacobs  : PR passed review checklist. Please evaluate proposed changes further and consider for merging.


Testing: Waiting for store.dispatch() to complete

I'm writing some instrumented tests where I need to set up an initial store state, then run an action, and then check the store's state after the action completes to ensure that the correct state got set afterward.
I thought that by ensuring all of the state dispatchings were on the main thread would essentially make this synchronous, but it appears my understanding may have been incorrect.
Is there a way to write code like the following (simplified)?

Thanks!


[doom-gruvbox] Add contrast options?

The original gruvbox has 3 contrast for both dark and light versions: hard, medium, and soft. This makes 3 * 2 = 6 variations in total.
As far as I'm concerned, it is simply the change in the background colour -- all other colours are not touched.
Here are the colours I've extracted, if anyone is interested in adding the option. IMHO, there's no need to "create a new theme", we just have to have a variable that controls the contrast. (Something similar to (setq doom-solarized-dark-brighter-text t))
* Just the background colours:
---------------------------------
dark-hard:              #1d2021
dark-medium (default):  #282828
dark-soft:              #32302f
light-hard:             #f9f5d7
light-medium (default): #fbf1c7
light-soft:             #f2e5bc

Here are the colour references: https://github.com/morhetz/gruvbox

I've experimented a bit with 2-way boolean values:
(defcustom doom-gruvbox-contrast-dark-hard nil
  "If non-nil, background will be darker, turning the theme into gruvbox-dark-hard."
  :group 'doom-gruvbox-theme
  :type 'boolean)

Let me know what's the best practice to use multiple choices like 'hard 'soft 'medium and I'll create a PR.

That feature is already implemented for Gruvbox light! As @jsoa told me, with the dark version it's slightly more difficult but not that much. If you could create a PR it would be great! I've been kinda busy with work recently.

@ema2159 Cool! ### I'll check out the implementation for gruvbox-light and send a PR for gruvbox dark soon :-)
P.x. Good luck with your work!

Done! PR here: https://github.com/hlissner/emacs-doom-themes/pull/475
Now the users can set
(setq doom-gruvbox-dark-variant "hard")
;; or
(setq doom-gruvbox-dark-variant "soft")

P.s. For before/after screenshots, please kindly check out the PR conversation here: https://github.com/hlissner/emacs-doom-themes/pull/475

I think it's safe to close this issue as well as #458 now :+1:  Thanks Ian!

You're right, closing issue now!


Menu clipped in settings editor with few results

Have large window
Open settings editor and search for a modified setting
With only 1 result and a huge window, there should be a lot of whitespace below
Click gear menu for setting

🐛 => menu is clipped by the list container not occupying the full space of the window. this is caused as the dropdown menu is a child of the gear icon element for focus reasons.

Duplicate of #164602


ENH: Dark theme with light Stanford logo and default to browser preference

Generated a white logo from Stanford branding and cropped it to match the current logo.
I don't really understand why this isn't working. Chrome continues to show the dark logo, while Firefox only shows the light logo. Will probably need Javascript to actually make this work, which seems bad when HTML supports this.
Marking this as a draft in case anybody wants to figure this out. Will mark it ready to review if/when I get around to doing this.

Converted to draft, as sharing this with Quarto devs led one to point out that dark mode can produce flashes on load, which might trigger epilepsy: https://github.com/quarto-dev/quarto-cli/discussions/2588#discussioncomment-6092667
Will probably try to play around with it more sooner or later. I really don't like light themes...