Show and focus URL bar when creating a new tab.

This also shows and focuses the URL bar when creating a new window,
since a new window immediately creates a new tab.
I elected to allow dispatching lists (Arrays) of actions to save renders
and set a pattern for avoiding races in the future.  In spirit, this is
similar to https://twitter.com/dan_abramov/status/656074974533459968 and
https://github.com/tshelburne/redux-batched-actions.  However, this
approach works with our thunk middleware (and those don't).

@bgrins you've been mostly owning this.  Take a look?  This should address #340 and #341.  However, I don't know how to test that the focus has been set.  Actually, I now see ui-state.js handling these.  I'll add tests for these properties.

Coverage increased (+0.3%) to 42.722% when pulling c2683be5d06d499de21413e92d00264bd2ae444a on focus-new-tab into 62c589554bcfbfefc723d9b3e7c89f66f3bacb25 on master.

I was actually able to produce a crash with this applied by doing: Open New Window, Close the window, Add tab on the original window, close tab on the original window. Looks like it's referencing this call, although only on the generated source: bw.webContents.send('focus-url-bar')
Error: Object has been destroyed
    at Error (native)
    at Function.<anonymous> (/tofino/lib/main/hotkeys.js:73:551)
<IP_ADDRESS> - - [24/May/2016:21:51:04 +0000] "POST /v1/session/end HTTP/1.1" - - "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.102 Electron/1.1.0 Safari/537.36"
Build failed.
Error: Exited with exit code 1
    at ChildProcess.<anonymous> (/tofino/build/utils.js:95:18)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:204:12)

Oh wait, that crash seems to happen even without this commit applied

Coverage increased (+0.4%) to 42.891% when pulling 6871c8918446d1cf437c099fc9ac12b4d69ad0b9 on focus-new-tab into afee003cdfd7e148051c9df6c845ce1158e7c3e3 on master.

If I open up the autocomplete box on the first tab by starting to edit it, then close it, then open a new tab, then the autocomplete box pops back up.  Off hand, I think if we did a clearCompletions() action when focusing and showing the URL bar that would fix this issue

@bgrins -- my guess is you're seeing something like #527.  That commit may address your crash.
Is it reasonable to clearCompletions during CREATE_TAB?  That is, should creating a new (selected) tab also dispatch a "reset UI state" command?

Is it reasonable to clearCompletions during CREATE_TAB? That is, should creating a new (selected) tab also dispatch a "reset UI state" command?

If the tab is going to become selected, I think so.  Same as how awesomebar becomes closed when opening a new tab in Firefox.

@bgrins, @linkclark -- sorry for the force-push; hopefully earlier comments are still present.  I built on top to ease review.  How do you feel about this implementation?

Coverage increased (+0.6%) to 42.356% when pulling dfcd32cc25bbb37d0c91defb619b382e5c7278aa on focus-new-tab into ac6fa2bf190f7d879908fe34b49bcb0f1dacb421 on master.

Just tested this out locally, LGTM

This landed


Auto Run Plugins

May be tricky.
Allow config for the following:
Auto run:

On init - don't wait for input, just run when the plugin is launched
On change - any change (key press) triggers the plugin to run
On paste - if something is pasted into the input editor, trigger the plugin
On blur(?) - when any form field  (or the editor) loses focus

Needs:

Combination - although "on change" is a superset of "on paste", "on init" is separate
Debouncing - if you type (for on change) you don't want to trigger too often
Ability for the user to disable the behaviour somehow

Maybe on paste should be the default, which can be disabled by the plugin.
If it's a true transform it should be idempotent, so it should be fine, and likely to be the preferred outcome (people are likely pasting to do a transform of that data)

autoRun: {
  onInit: true, // default: false
  onPaste: false, // default: true
  onType: false, // default: false
  onBlur: true, // default: false
}

This would run initially when the user opens the plugin, then again only if the user changes focus from any form field or the text editor.


Support running iTerm

WIP
To run tests in iTerm, set the strategy to "iterm":
Menu > Preferences > Settings
"phpunit.strategy": "iterm",

Notes
I've added .no-sublime-package to unpack the package on install, because we now have an iterm executable to run the tests in the iterm strategy,

If including executables or shared libraries, add a .no-sublime-package file. Adding this file to the root of your repository will prevent Package Control from keeping your package packed as a .sublime-package file in ST3.  https://packagecontrol.io/docs/submitting_a_package

Close #86

Coverage decreased (-0.8%) to 46.092% when pulling 9dfcaf7e421903544bbe118fa44f6f63391dd4da on osx into 1c741612f4c7ede434ba63767605721fa62cc274 on master.


Use YAML for attribution file format

Right now, an attribution file is just a text file with expected keywords but no format. This is an excellent YAML use-case, where an actual schema could be used to organize the data into lists, similar to the Q&A YAML files.

This was chosen by the lawyers from the Project. We cannot change this without dealing with the legal team and what they are looking for.
As much as this from a technical conversation is a great idea, this is out of our hands as this project goes.

Ok, thanks for the quick response!


Restore eMMC

Removed at https://github.com/jadonk/beaglebone-blue/commit/75f1e67d3ee9893e94103e41b84841ae77a6b699
Sandisk provided info on routing by using N/C pins.

Restored at 9f8d1503517d18915054b3e56a3a5148aa323ae9


Issue with output path

Hi. Thanks for the package, this is exactly that I need. Currently i'm having an issue then running it on my machine.
I wanted to extract audio:
audio_path = f"./temp/temp.mp3"
extract_audio(input_path=str(file_path), output_path=audio_path)

The output_path dir exists on my system, but the library tries to create it anyways:
Traceback:
  File "/run/media/dvarkless/LinuxData/Files/PythonProjects/some_dir/venv/lib/python3.10/site-packages/audio_extract/ffmpeg.py", line 16, in extract_audio
    result = validator.validate()
  File "/run/media/dvarkless/LinuxData/Files/PythonProjects/some_dir/venv/lib/python3.10/site-packages/audio_extract/validators.py", line 31, in validate
    self._validate_output_path()
  File "/run/media/dvarkless/LinuxData/Files/PythonProjects/some_dir/venv/lib/python3.10/site-packages/audio_extract/validators.py", line 59, in _validate_output_path
    os.makedirs(folder_path)
  File "/home/dvarkless/.pyenv/versions/3.10.11/lib/python3.10/os.py", line 225, in makedirs
    mkdir(name, mode)
FileNotFoundError: [Errno 2] No such file or directory: ''

I noticed, that the separators in the code are in Windows style '\':
# ./audio_extract/validators.py
    def _validate_output_path(self):
        output_path = self.output_path
        if output_path.endswith("/") or output_path.endswith("\\"):
            output_path += "audio." + self.output_format

        output_path = os.path.abspath(output_path)
        output_parts = output_path.split('\\')    # Here
        filename = output_parts[-1]
        folder_path = "\\".join(output_parts[:-1])  # And here
     ....

I think it doesn't work on my linux system.
As stated in the documentation, os.sep is system independent and should be used instead of explicitly typing separators.

Hey, Thanks for reporting this because i would have never foundout since i use windows. I made the change and pushed it but i can't really test it so pleaes let me know if the issue is fixed for you.

Looks like the issue with output path is gone, thanks

The command arguments seems to be wrong
  File "/run/media/dvarkless/LinuxData/Files/PythonProjects/some_dir/src/bot/action_handlers.py", line 42, in video_handler
    extract_audio(input_path=str(file_path), output_path=audio_path)
  File "/run/media/dvarkless/LinuxData/Files/PythonProjects/some_dir/venv/lib/python3.10/site-packages/audio_extract/ffmpeg.py", line 39, in extract_audio
    raise Exception(f"Failed : {error}.")
Exception: Failed : Use -h to get full help or, even better, run 'man ffmpeg'.

But then I run the ffmpeg itself, the same arguments seems to be work just fine:
ffmpeg -i ./Video/somevideo.MOV -ss 0 -f mp3 -y ./Video/test.mp3

...
>>> size=    1840kB time=00:01:57.70 bitrate= 128.1kbits/s speed=58.6x
>>> video:0kB audio:1840kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 0.018098%

I'll run the debuger and comment if I can figure something out

I fixed the problem by removing argument shell=True from the subprocess call. Now the function works fine.
# ./audio_extract/ffmpeg.py
# Line 34
result = subprocess.run(command, stdout=subprocess.PIPE, stderr=subprocess.PIPE)

I don't know why this works, but it works for me

I dont really remember why I had it there to begin with it might have been for debbuging.
I think on linux it needs permissions to run with shell=True but if it works without it then ts not needed.
Anyway i removed it and thanks a lot once again for the feedback.


Help with keymapping RK61 Keyboard

New to coding, so got a starter mechanical keyboard, however a lot of the keys I need to use are either really poorly placed or don't exist on the keyboard (backslash for example).
I would LOVE to remap smoother features and a full key set, if it wasn't for the poor software that comes with the RK61 (That doesn't work for my non-RGB version) then this keyboard would be a perfect startup keyboard!
Could anyone help me with this so I can make full use of QMK? Thanks!

I also have a keyboard by Royal Kludge (model RK71) and would like to know if it could be added to the QMK firmware.

Based on this article the RK84 uses the BYK916 MCU. Unfortunately, there isn't much information about the PCB nor the MCU. If someone is able to find the schematics for this PCB/MCU (which is also used by the Feker PCBs), it'll definitely be easier to write the firmware.

Here you go.
https://github.com/0xYuzko/RK61-PCB

The is a PR for rk71, I believe we can ref to it
rk71 draft


Make 'removeNumericPathSegments' method substitutable

optionally override the default 'removeNumericPathSegments' functionality by providing a function that does the redaction.
natchez-extras should fall back to the current implementation when the function is not provided to keep backward compatibility

sbt test passed locally.
The following test failed on Github workflow:
==> X com.ovoenergy.natchez.extras.slf4j.Slf4jSpanTest.should Log cancelled tasks  0.131s munit.ComparisonFailException: /home/runner/work/natchez-extras/natchez-extras/natchez-extras-slf4j/src/test/scala/com/ovoenergy/natchez/extras/slf4j/Slf4jSpanTest.scala:47
46:      }
47:    )
48:  }
values are not the same
=> Obtained
List(
  "foo started",
  "foo cancelled"
)
=> Diff (- obtained, + expected)
 List(
-  "foo started",
   "foo cancelled"

I think, it's a flaky test since I have only made a change in natchez-extras-http4s module and I haven't touched anything in slf4j. How can I re-run the failed github workflow? Do I need a github permission to do so? or can you re-run it for me 🙏

Test failed for some reason. Closing and re-open it to run the test

Test failed for some reason. Closing and re-open it to run the test


Fixed typo in name of address field for Person record in Chapter 7.

In the code, the address field for the Person record is called homeAddress, but the text in the book refers to address.

I also fixed issue #80 as it was in the same block of code...

Thanks anyway, but please see CONTRIBUTING.md, I don't accept PRs here.


huy-reactjs-starter-kit

Sorry, now I'm even more confused. There are 63 file changes in this pull request. What is this about?

Chillax dude, I was taking your course on Pluralsight (I'm a loyal
subscriber) and click on the wrong button in the wrong repo.
On Fri, May 27, 2016 at 10:08 AM, Cory House<EMAIL_ADDRESS>wrote:

Sorry, now I'm even more confused. There are 63 file changes in this pull
request. What is this about?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coryhouse/react-slingshot/pull/149#issuecomment-222155707,
or mute the thread
https://github.com/notifications/unsubscribe/AE4wOdsiqUhq8fpQ5vNVKgUEOVrLfQkcks5qFvrEgaJpZM4Iofk6
.


{acl: 'public-read'} is hard-coded in CamaleonCmsAwsUploader, no way to change

Here it is - https://github.com/owen2345/camaleon-cms/blob/master/app/uploaders/camaleon_cms_aws_uploader.rb#L92
Sure I've worked it around by setting public read access to the bucket, but AWS is complaining of this and, indeed, this would be good to be configurable.

Hi @texpert
You can customize it using the hook "on_uploader", sample:
def my_hook_listener(settings)
  settings[:aws_settings][:aws_file_upload_settings] = lambda { |file_settings| file_settings.merge({ .. custom_file_settings })  }
end

BTW: Sorry for the bad code, the project  is too too old.

Thanks a lot, @owen2345 , worked like a charm!!
However, I have just discovered, that downloading of private files is possible only via presigned URL. Not a word about it in AWS docs, until I went to their forums :(


visual code

Hello, do you have a visual code

Hi @lshssel,
I do not, please see some previous issues where people posted their own
Best,
Orr


host: Remove unused port allocator

It became obsolete with the move to per-container IPs, and the last use is an unused code path in the layer 0 bootstap runner.

LGTM, pending CI


Docs.MS ToC Constriction Script

Intended to be used as part of Docs.MS CI. For example we could update preview ToC to match the content (where appropriate) to latest. This will allow us to keep our ToCs consistent without manual effort.

[x] Handle Arbitrary Construction of the ToC.yml. Still maintain only the nodes that have generated namespaces
[x] Updates to readmes of a suffix provided
[ ] Need to update the uid to include a preview tag or otherwise prevent duplicate UID across generated ToCs
[ ] More bugfixes will arrive as I apply this to each preview ToC

@Azure/azure-sdk-eng
@maggiepint

/check-enforcer reset

/check-enforcer reset

/check-enforcer override


Crash in windowed mode in the hangar

Hello,
Justagai has reported an issue:

I forgot to mention Reimar's ddraw in windowed mode outright crashes the game in the hangar. I'll try to get some event viewer logs uploaded later.

See http://www.xwaupgrade.com/phpBB3/viewtopic.php?p=156268#p156268
Faulting application name: XWINGALLIANCE.EXE, version: <IP_ADDRESS>, time stamp: 0x3765a9b7
Faulting module name: USER32.dll, version: 10.0.17134.376, time stamp: 0xbeaa4463
Exception code: 0xc00000fd
Fault offset: 0x00026fbb
Faulting process id: 0x22f0
Faulting application start time: 0x01d49a9ac72ad9ef
Faulting application path: D:\GOG Games\Star Wars - X-Wing Alliance\XWINGALLIANCE.EXE
Faulting module path: C:\WINDOWS\System32\USER32.dll
Report Id: de2060d5-b4ac-4f55-8160-d2b10b05b3e9
Faulting package full name:
Faulting package-relative application ID:

Thanks, fixed.


About the experiment results

Hello! Thank you for your wonderful work！
I notice that in your readme ,when input is xyz+color+landmark, the accuracy(test) is 0.320. But when I use your pre-trained model or train from the scratch, the result is about 0.272 , lower than your checkpoints.

So I wonder whether you are using a validation set or a test set to evaluate your model?
Many thanks for your time.
Regards.

The accuracy(test) is the performance of the test set, not the val set.


Using new protobuf lib

Replaced python3-protobuf (an outdated for of the official protobuf lib) with the official protobuf lib.
It only required re-compiling protobuf files.
There were integration tests here. Some of them doesn't work (and didn't work before my changes) due to usage of non-standard riak bucket types, but most of them succeed.
There are also a couple of tests in our aioriak fork. They work as well.
I'm going to live-test the change:

[x] salesforce-integration on staging
[ ] salesforce-integration on canary?
[ ] Some other live service

Can you briefly explain how did you approach this change (e.g. maybe you re-used some existing draft open-source PR), and how you tested it?

@le-r-on added a better description


fix: typo

这个 PR 实现什么功能/修复什么问题:
拼写错误修复
是否需要 backport 到之前的 release 分支:

release/3.1

/area region
/cc @yousong

Codecov Report

Merging #5682 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5682   +/-   ##
=======================================
  Coverage    6.97%    6.97%
=======================================
  Files         624      624
  Lines      105534   105534
=======================================
  Hits         7358     7358
  Misses      97600    97600
  Partials      576      576

Flag
Coverage Δ

#aFlag
6.97% <ø> (ø)

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 66de347...cc28924. Read the comment docs.


support flag detail display

detail:
https://en.wikipedia.org/wiki/FLAGS_register

https://github.com/cch123/asm-cli/commit/b6e650194ab5fd1ca2801e989f11efd16275ace7


Update retrain.py to be compatible with Python 3

It's PROBABLY backwards compatible with Python 2 but I haven't fully tested it
Fixes https://github.com/tensorflow/tensorflow/issues/1274

Can one of the admins verify this patch?

I signed it! As Arterys, Inc.

Updated commit e-mail address to match my CLA e-mail address (Arterys, Inc.) and force pushed.

Can you go to https://cla.developers.google.com/clas and check that the corp CLA for Arterys shows up as "CLAs I am covered by"?

@martinwicke sorry for the confusion; now I see the CLA listed at https://cla.developers.google.com/clas.

Thanks, can you update to master?  Then we can test.

@vrv Rebased onto master

@tensorflow-jenkins: test this lease

*test this please

@tensorflow-jenkins: test this please.

Our mac tests are broken, but this otherwise lookss good. merging.


Notebook has no Title in docs if sub-title was not specified

I'm running nbdev 0.2.17
When building docs I noticed a  warning  Warning: Some of your Notebooks use the same title
After fiddling around, I assume that nbdev does not parse a notebook's title if it has no subtitle.
See first screenshot: notebooks have no subtitles, and they do not appear in the warning (Foo and Bar are just Title, use colors as guide)

Then add subsitles, and it all works like a charm.

So I suggest either making subtitles mandatory, or changing the warning text, or changing parsing rules.
p.s. sorry for reporting a bug instead of MR, nbdev is amazing and completely transforming how I work!

Should be fixed now. Thanks for flagging!


circle CI runner type support

Our list of supported runner types is missing CircleCI.
This task is about doing something similar we did for Jenkins #88.

Looking after this

https://circleci.com/docs/variables/


Jpg to png convertor using pillow Module in Python

Issue #40
About PR:
This script converts JPG images in a folder to PNG images and gets stored in another folder using Pillow module of Python

@amanmehara Please review PR and merge for "Haktoberfest"

@amanmehara Please review


GAL-2507 temp fix for baseurl

Description
baseUrl isn't working as expected on prod, and causes the link copied by the Share button to have localhost instead of gallery.so
getting the baseurl from the next env vars didn't seem to be working as expected on deployed environments, so this is a temp solution to use window.origin until we figure out why

legendary


397,398,399,400 Add special events to Pycon site

model
admin (in Django admin)
detail page that looks like a CMS page
include in program export

Fixes #397. Fixes #398. Fixes #399. Fixes #400.

Merge conflicts here.

Merge conflicts and a test failure fixed.

The Special Events model differs very much from the other things in the PyCon admin category. Maybe it deserves its own app?

With that said, the model's verbose name probably won't need to differ from the default.

Does #400 refer to the schedule export? I thought that program export was intended.

On Wed, Aug 12, 2015 at 6:20 PM, Rebecca<EMAIL_ADDRESS>wrote:

Does #400 https://github.com/PyCon/pycon/issues/400 refer to the
schedule export? I thought that program export
https://github.com/PyCon/pycon/blob/develop/pycon/program_export.py was
intended.

Good question...  maybe it should be both?

I'll ask about the exports on today's client call.

On Wed, Aug 12, 2015 at 6:10 PM, Rebecca<EMAIL_ADDRESS>wrote:

The Special Events model differs very much from the other things in the
PyCon admin category. Maybe it deserves its own app?

I'm reluctant to add another app (we already have so many), but maybe it
would fit better in a different app than where I've put it?  Possibly
under Conference?

Our call yesterday got canceled; I just asked on Basecamp about the exports.

Ewa says program export (https://basecamp.com/1849796/projects/9507827/messages/46536642#comment_320492231), not schedule export.  I'll update the code.

@rebecca-caktus I've updated this.

Changes look good. I've suggested some minor admin customizations & display logic when the event starts & ends on the same day. Then :ship:


Run scanner end to end

Currently Scanner is not integrated well with the visitors, so integrate it with them and add integration test for it

Done


⚠ Bofrost.hr has degraded performance

In cb783b5, Bofrost.hr (https://www.bofrost.hr) experienced degraded performance:

HTTP code: 200
Response time: 2897 ms

Resolved: Bofrost.hr performance has improved in 135fa37 after 6 minutes.


force genesis constants config to top level

Summary
@mrmr1993 Had previously consolidated the Genesis_constants module to define and manage a large amount of config:

type t = <some record of relevant genesis constants>
module Constraint_constants
module Proof_level

He started the work to prohibit accessing these "compile time config" values directly, instead requiring each value/function that accesses them to take them as an argument. This PR finishes that effort, pushing this strategy all the way up to top level executables and tests.
Even with this work, we are still using the compile time config values rather than reading them in via cli/json/etc. I vote to change this in a follow up PR because there are still some choices to make and this PR is already large enough. For the time being, I have simply pushed everything up to the same level where we are accessing CLI arguments, since that's where it will likely be handled. Concretely, I am suggesting something like what is in this ROUGH poc. For tests/etc which have no CLI args I have pushed it up to main, we will need to decide how that is handled as well.
Changes

aside from accessing Genesis_constants.For_unit_tests for some unit tests, no function/value outside of executables should access compile time config managed in Genesis_constants (Eventually we might want to change this too). In other words, you should only find access to the module Genesis_constants.Compiled at the top level.
delete the genesis_constants.compiled package, as it is no longer needed

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

Let's also get rid of mina_compile_config module (in this PR). It has only 21 usages now (a very small amount) with only the following fields used:

itn_features
Fee constants

default_transaction_fee_string
default_snark_worker_fee_string
minimum_user_command_fee_string
let's unite them in a fee_constants structure and pass to usages, sourcing from runtime config

zkapp_cmd_limit
vrf_poll_interval_ms
network_id (just remove it, it's already sourced from runtime config)
compaction_interval_ms

Let's move these to runtime config and pass as named parameters down the function call stack.

!ci-build-me

!ci-build-me

!ci-build-me

Let's also get rid of mina_compile_config module (in this PR). It has only 21 usages now (a very small amount) with only the following fields used:

itn_features

Fee constants

default_transaction_fee_string
default_snark_worker_fee_string
minimum_user_command_fee_string
let's unite them in a fee_constants structure and pass to usages, sourcing from runtime config

zkapp_cmd_limit

vrf_poll_interval_ms

network_id (just remove it, it's already sourced from runtime config)

compaction_interval_ms

Let's move these to runtime config and pass as named parameters down the function call stack.

@georgeee We are going to get there, but I would like to kick that can to the next PR. Even though there are "21" uses, the changes can easily cascade up the stack to create a much larger diff than 21 edits.

!ci-build-me

Also:
./src/lib/crypto/snarky_tests/snarky_tests.ml has needs to be ocamlformat-ed

Please run make reformat to fix it

!ci-build-me

!ci-build-me

!ci-nightly-me

!ci-build-me

!ci-nightly-me

!ci-build-me

!ci-nightly-me

!ci-nightly-me

coverage: 100.0% (+66.6%) from 33.437%
when pulling 45c12f7edf0a8ff26ab510c13e2e953b279bfd1f on martin/force-config-to-toplevel
into 3d9a1971577cf5afd548c7504cf56a3d5dd45f51 on compatible.

!ci-build-me

!ci-build-me

!ci-build-me

!ci-nightly-me


Method 'get_CustomField' does not have an implementation

Describe the bug
Using Manatee.Trello libraries to access Trello data from a c# program.  Just upgraded to latest stable versions.  Now the program fails.  Thanks for any help you can give.
Here are the error details
System.TypeLoadException
HResult=0x80131522
Message=Method 'get_CustomField' in type 'Manatee.Trello.ManateeJson.Entities.ManateeActionData' from assembly 'Manatee.Trello.ManateeJson, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=fbb8336f4f768592' does not have an implementation.
Source=Manatee.Trello.ManateeJson
StackTrace:
at Manatee.Trello.ManateeJson.ManateeSerializer.InitializeAbstractionMap(JsonSerializer serializer)
at Manatee.Trello.ManateeJson.ManateeSerializer..ctor()
at Project1.Form1.button2_Click(Object sender, EventArgs e) in C:\Users\Bruce Pike Rice\Documents\Source\Project1\Project1\Form1.cs:line 34
at System.Windows.Forms.Control.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ButtonBase.WndProc(Message& m)
at System.Windows.Forms.Button.WndProc(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
at System.Windows.Forms.UnsafeNativeMethods.DispatchMessageW(MSG& msg)
at System.Windows.Forms.Application.ComponentManager.System.Windows.Forms.UnsafeNativeMethods.IMsoComponentManager.FPushMessageLoop(IntPtr dwComponentID, Int32 reason, Int32 pvLoopData)
at System.Windows.Forms.Application.ThreadContext.RunMessageLoopInner(Int32 reason, ApplicationContext context)
at System.Windows.Forms.Application.ThreadContext.RunMessageLoop(Int32 reason, ApplicationContext context)
at System.Windows.Forms.Application.Run(Form mainForm)
at Project1.Program.Main() in C:\Users\Bruce Pike Rice\Documents\Source\Project1\Project1\Program.cs:line 19
Here is a small winforms app to show the problem:
using System;
using System.Windows.Forms;
using Manatee.Trello;
using Manatee.Trello.ManateeJson;
namespace Project1
{
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}
    private void Form1_Load(object sender, EventArgs e)
    {

    }

    private void button2_Click(object sender, EventArgs e)
    {
        // Manatee configuration
        var serializer = new ManateeSerializer();
        TrelloConfiguration.Serializer = serializer;
        TrelloConfiguration.Deserializer = serializer;
        TrelloAuthorization.Default.AppKey = "MyAppKey";
        TrelloAuthorization.Default.UserToken = "MyUserToken";

        string BrdId = "5ab159c6e49acdc7d971134d";
        var Brd = new Board(BrdId);
        string BrdNm = Brd.Name;

    }
}

}
To Reproduce
Please add test code or steps to reproduce the behavior:

Open In Visual Studio
Replace MyAppKey and MyUserToken with valid values.
start program as debug.  Click on button.
See error at line 24

Expected behavior
Program to complete with no error.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
.Net Target (Fwk 4.6.1)
.Net Fwk Version 4.7.03056

Manatee packages installed

Manatee.Json v9.9.0
Manatee.Trello v3.0.10
Manatee.Trello.ManateeJson v2.3.0
Manatee.Trello.WebApi v2.0.3

@BruceP99166 what version of Manatee.Trello are you using?  If you're using version 3+, then you don't need to do the JSON serializer setup.

Just added the Manatee versions.


Jesse: Watch Corey[\W_]*+Wilson

Jesse requests the watch of the watch_keyword Corey[\W_]*+Wilson. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Corey[\W_]*+Wilson has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Rejected by Jeff Schaller in Charcoal HQ. Reason: 'doesn't have anchoring for a username match; FP for existing users on SO'


Attempting to run in Docker

trying to run in a docker container. taking the exact same steps as i did on the host os. encountering error code 3
isaac@6e5f08a713d3:~/projects/webdash-test$ npx webdash serve --port 3000
Webdash running on http://localhost:3000!
(node:3905) UnhandledPromiseRejectionWarning: Error: Exited with code 3
    at ChildProcess.cp.once.code (/home/isaac/projects/webdash-test/node_modules/opn/index.js:84:13)
    at Object.onceWrapper (events.js:317:30)
    at emitTwo (events.js:126:13)
    at ChildProcess.emit (events.js:214:7)
    at maybeClose (internal/child_process.js:925:16)
    at Socket.stream.socket.on (internal/child_process.js:346:11)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at Pipe._handle.close [as _onclose] (net.js:567:12)
(node:3905) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:3905) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
'''

unfortunately i get
'localhost didn’t send any data.'
when i visit the page.
but if i run webdash on the host os in the exact same folder it will work. id love to look into it myself as well but im just starting my trip down the docker rabbit hole.

ah okay
I've never worked with Docker but isn't this about Docker port mapping/forwarding/exposing?

ive double checked the setup for the ports. i can for example host an angular app through that port.

so you're sure it's port 3456 right?
if not, then are you passing -p 8000 for example?

yes, i have verified that all the ports match up and that i am attempting to use webdash through the exposed port. even set up a container with webdash's default port as the exposed port. im thinking that when it breaks it is somehow blocking further processing.

If possible, please try manually commenting out the opn line (and the try catch) in index.js and see if it works (although I doubt it)

you were correct. commenting those out didnt affect it. it seems to be connecting because the error page is saying 'didn't send any data' instead of couldn't be reached.
this instance i set the docker up to use webdash's default port. i can verify the port is being listened to
if anyone else wants to attempt
using docker
docker pull ubuntu
docker run -it -p 3456:3456 ubuntu
apt update
apt install curl
curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.33.9/install.sh | bash
follow last instructions from nvm to get it added to path in existing terminal
nvm install 8

create a project folder and enter the folder
follow webdash instructions https://webdash.xyz/

Closing for inactivity, feel free to add more information in case it's still an issue


Add card details to docs

I ordered the wrong cards at first not realizing Brainstorm sells a variety of card types. Needless to say, the kids were thoroughly unimpressed with my first magic cards demo 🙄.

OH NOOO! Email them, the guy there is pretty nice. I'm sure he'll take them back for a swap.
Thanks for adding the clarification.

NP! Thank you for putting together such a polished/well-documented repo! My 3-yo (and probably everyone who receives royalties for streaming Moana) is thrilled with the result.


CARDS-1337: Enable patient authentication

What's included:

a service for generating tokens
an authenticator that accepts tokens for login
a patient user that is used for all patients
access rights that only allow a visiting patient read access to the Visit and Patient subjects, and read-write access to existing forms for the visit
a temporary helper script that can be used for manually generating tokens; this should be removed after testing

How it works:

start with ./start_cards.sh --project cards4proms --dev --permissions trusted
as admin, create two Patient subjects (alice, bob), each with two Visit sub-subjects (a1, a2, b1, b2)
create a Visit Information form for a1, leave the visit date empty
create a Visit Information form for a2, set a visit date in the future
create a Visit Information form for b1, set a visit date in the past
do not create a Visit Information form for b2
create other forms for some of these visits
to create a token for a visit, access the Visit subject, and in the URL remove /content.html and append .token.html at the end, e.g. http://localhost:8080/content.html/Subjects/0542e731-575d-4e06-bc83-f79f7f300166/7a96f652-cff6-4701-9b27-55a2c2409034 becomes http://localhost:8080/Subjects/0542e731-575d-4e06-bc83-f79f7f300166/7a96f652-cff6-4701-9b27-55a2c2409034.token.html
the token will be printed in the page
tokens are valid for: a1 and b2 10 minutes from creation, a2 until the visit date specified in the form, b1 should already be expired and not work at all
to log in using a token, append ?auth_token=the token to the URL, e.g. http://localhost:8080/?auth_token=dbbc84ba-aaf0-4d75-8581-27ff194d7c99_08fa82bc722d100424db2f52c3b90c374e8cdb33c7007bc0
once logged in with a token, you should:

appear logged in, a P avatar indicating you are patient
have access to the homepage, but no data is shown, Error - Access to data is pending the approval of your account is displayed instead
have access to the correct Patient and Visit subjects (by manually copy-pasting the URL), but not to other visits or other patients

on the patient/visit pages, it's OK to have Error obtaining subject data: 404 Not Found

have access to the correct forms (by copy-pasting the URLs), can view, edit, save, re-edit
I didn't test, since there are no such questions in proms mode, but do vocabularies work?
after ten minutes, trying to access using the expired token should fail, re-displaying the login page

using the proms UI (/Proms.html/Cardio) does not work at the moment

Things to do:

[x] If a token is expired, right now we just show the login page; is it worth spending time to instead display a message about the expired link? - CARDS-1548
Logging in with an expired token will delete it, but there's no way to periodically delete expired tokens. - CARDS-1549
[x] Looks like logging out doesn't work, needs to be fixed in this PR
We should prevent access to the regular homepage if a patient is logged in, they should only have access to the Proms UI once it's actually working - CARDS-1339
When sending emails to the patient, call tokenManager.create(date of the visit + 2hours, Collections.singletonMap("cards:visit", "uuid of the visit node")).getToken() to get the login token, and append it to the URL. - to be considered in CARDS-1422 + CARDS-1424.
The proms UI needs to be reworked to actually get the data from the server instead of trying to create new forms for the fake test patients. - the backend support for this is in CARDS-1349 + CARDS-1484; UI will be adjusted in CARDS-1550.
[x] Disallow changing the subject (deny write access to form/subject)

I cannot sign out when I'm logged as patient.

I cannot sign out when I'm logged as patient. Should change password option be available for patients?

Patients will not have access to this UI, this will be ensured by properly set permissions in https://phenotips.atlassian.net/browse/CARDS-1339 .

After login with ?auth_token=the token, should I be able to create a new questionnaire for the given subject?
It allows me to select the questionnaire and the subject for which I created token, but then I get the login dialog.

When I click on statistics it gives me the data related to the subject to whom I created a token. However, when I click on the graph I get this:

I created a token for patient who has two visits. Each visit has data that should contribute to the Stats. When I click on Statistics I get 2 blank graphs. I am assuming this is correct, as the token is not applicable to the visits.

I didn't test, since there are no such questions in proms mode, but do vocabularies work?
Yes. Tested.

After login with ?auth_token=the token, should I be able to create a new questionnaire for the given subject?
It allows me to clik on the (+) button, select questionnaire and only the subject for which I created token, but then I get the login dialog.

No, patients can only edit existing forms. They cannot create any new subjects/forms, and they should not be able to delete an existing form that they have access to. One thing that is allowed but shouldn't be is changing the subject. I'll fix that.

Also, the login dialog appears after attempting to delete the subject.

The delete button should not be present. This is kind of a known bug, our UI doesn't take into account the actual rights that the user has, and always displays edit/delete buttons, even if the user cannot actually do that.
But this is irrelevant, the patients will not be able to access this UI at all once we properly implement the Proms UI and restrict access to the dashboard.

When I click on Statistics it gives me the data related to the subject to whom I created a token.
However, when I click on the graph I get this (white screen):

Patients should not have access to statistics at all, but it's not a big deal if they can fetch the stats data, if that contains only the accessible forms (the ones for the current visit).
And the statistics UI will not be accessible to patients.

I created a token for patient who has two visits. Each visit has data that should contribute to the Stats. When I click on Statistics I get 2 blank graphs. I am assuming this is correct, as the token is not applicable to the visits.

Correct that the stats should not be shown.
But tokens should not be created for anything other than a visit. I did not prevent this in the code, though, since we may want to add other kinds of token usecases in the future, and the manual token generation will not be available, all tokens will be created automatically when sending emails to the patient.

Question: Dose token created for patient ever expire?

In ten minutes.

https://user-images.githubusercontent.com/23243244/147788587-9c80071b-41ff-48a7-af45-59ee86645f60.mp4

Not sure if the grammar is correct here. Perhaps "The link has expired." or "The token has expired.".

use token to login
open Visit form and
click on Change subject
click on the NEW button in the Change subject dialog
login dialog pops up

With the visit a1 token login I am unable to access visit a1

however I can access all visit a1's forms.

Attempting to access any other form gives me 404 err. However, attempting to access any other visit seem to get me stuck.

Stats are available just for the visit for which the token was created. When I click on stat's graph, I get this:

Suggestion: Disable Change subject and Delete form options.

The whole UI will be inaccessible to patients, only the PROMS UI will be available to them, and that one is not yet working.
But that may also affects regular users depending on their rights, so it will have to be fixed at some point, but not in this PR.

With the visit a1 token login I am unable to access visit a1 page ( this was expected afair)
however I can access, view and edit all visit a1's forms.
Attempting to access any other form gives me correctly an 404 err.

Yep, as expected (rightswise, and the UI errors should be ignored).

However, attempting to access any other visit seem to get me stuck.

Another "this is not the UI for patients" symptom, not a problem for now.

Stats are available just for the visit for which the token was created. When I click on stat's graph, I get this:

Expected, patients are not allowed to perform queries.


Allow any capitalization for commands

If a command ask gets registered, then %ask something and %ASK something and %aSK something should also lead to that command. There have been some cases of people using %ASK A CAPITALIZED QUESTION, so I think it makes sense.

This is indeed not intentional, see #36.

no problem


Account persistence after docker container restart

Hello,
first of all - thank you very much for that project!! :heart:
I'm currently running the version 0.10.1 within a docker container.
Everything works fine except the persistence of user accounts, when the container gets restarted.
Is there a way to persist that on a volume?
In which file / location are the account data stored?
Thank you in advance!
Kind regard,
Andreas

@gean-xyz Did you ever get this persistence issue sorted out?


Problems with gzipped sites?

I'm having issues making this work with sites on Amazon S3 with gzipped content. I get a "Finished! 0 links found." message back. Other sites without gzip work perfectly though.
Is this a known issue and are there plans to support sites that use gzip?

It should be supported. I'll have to add tests to find out.

Thanks for the response. I just published a test gzipped version of my personal site: http://test.alexmeub.com
Running blc http://test.alexmeub.com results in:
Getting links from: http://test.alexmeub.com/
Finished! 0 links found.

Running blc http://alexmeub.com works just fine (my live non-gzipped site). So it seems gzipped sites don't work (at least if hosted on Amazon S3).

bhttp doesn't support gzip. This is another reason to switch away from it.

FWIW, I locally forked broken-link-checker and replaced bhttp with request-promise. I needed to be able to have cookies and gzip support. From what I can tell, it also addressed the redirects-reported-as-broken issue in #47.

I tried to add these tests today using a module called broken-link-checker AKA blc. This tool seemed promising at first, but I ran into an issue where redirects for URLs with trailing slashes were wrongly being reported by blc as 404s. Others have run into this as well, and it seems the library is no longer actively maintained: https://github.com/stevenvachon/broken-link-checker/issues/47
Next I think I'll give check-links a try...

forked broken-link-checker and replaced bhttp with request-promise.

@MrDys I tried this but didn't have much luck. Can you share a link to your working fork?

https://github.com/request/request/issues/3142

Fixed i v0.8.0 branch.


Re-enable taffy send+sync assert

Objective
Fix up a FIXME comment
Solution
It looks like the linked issue has been resolved and we can re-enable the "assertion."

@TimJentzsch @Weibye for review please :)

bors r+


Robbed of 50+ points!! - What happened?!

Suddenly my honor just dropped by 50 points?
That's 2 days of effort down the drain :-(
I was able to capture a screenshot from the leader- board showing the discrepancy. But now even the leader=board shows to lower number

Please reinstate my points,

Hi, this happens time to time. This shows the correct honor points (check Total at the bottom of the page).
Regards,
suic


Remove unwanted warnings during training

During training with image logs, you can receive this error:
/nydl/code/open-metric-learning/oml/metrics/embeddings.py:292: RuntimeWarning: More than 20 figures have been opened. Figures created through the pyplot interface (`matplotlib.pyplot.figure`) are retained until explicitly closed and may consume too much memory. (To control this warning, see the rcParam `figure.max_open_warning`).
  fig = plt.figure(figsize=(30, 30 / (n_instances + N_GT_SHOW_EMBEDDING_METRICS + 1) * len(query_ids)))

To remove it, we need to add plt.close() in a callback.

Solved, after 1+ year of waiting...  xD
cc @b0nce


Bad URL in Tenable.io content solution manifest and package

Required items, please complete
Change(s):

Bad URL in Tenable.io content solution manifest and package (documentation)
issue #6209

Reason for Change(s):

Corrected URL

Version Updated:

NA

Testing Completed:

YES

Checked that the validations are passing and have addressed any issues that are present:

YES

@sreedharande : Seems some issue with Contributor License Agreement acceptance, please try to sign Contributor License Agreement again to accept your contribution

Main template deployed successfully

Arm-ttk validation run successfully

CreateUI definition fine

@sreedharande : Seems some issue with Contributor License Agreement acceptance, please try to sign Contributor License Agreement again to accept your contribution
If you sign in, we will approve and can be good to merge.

@sreedharande : Seems some issue with Contributor License Agreement acceptance, please try to sign Contributor License Agreement again to accept your contribution
If you sign in, we will approve and can be good to merge.

Re-verified, Looking good.


Nougat: app crashes on Twitter sharing

App crashes when we try to share image on Twitter in NOUGAT.

this is a known issue check #Issue 814 this is happening because of New File scheme changes of Android  7.0 . I will try to give a PR soon.


cip: Social Accounts Definition

PR for CIP-23
Discussion: https://github.com/ceramicnetwork/CIP/issues/55

@oed @PaulLeCam I made some comments


DB queue is > 50 (@298); try increasing --db-threads

what is this, how can I solve it?

use mysql and sex max_connections to 5000
https://pgm.readthedocs.io/en/develop/extras/mysql.html


https://developer.mozilla.org/ko/docs/Gecko_DOM_Reference

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
The doc is under the old URL. Please move the page to /ko/docs/Web/API/Document_Object_Model. 😭

Moved.


Debian 8 CFFI failure Kibana import during install

Starting Kibana
Importing Kibana Index Patterns
Importing /home/user/sweetsecurity/kibana/patterns/sweet_security_alerts.json
Traceback (most recent call last):
File "setup.py", line 220, in
kibana.install(chosenInterfaceIP)
File "/home/user/sweetsecurity/install/kibana.py", line 85, in install
importIndexMapping(os.path.join(patternPath, file))
File "/home/user/sweetsecurity/install/kibana.py", line 117, in importIndexMapping
from elasticsearch import Elasticsearch
File "/usr/local/lib/python2.7/dist-packages/elasticsearch/init.py", line 17, in
from .client import Elasticsearch
File "/usr/local/lib/python2.7/dist-packages/elasticsearch/client/init.py", line 4, in
from ..transport import Transport
File "/usr/local/lib/python2.7/dist-packages/elasticsearch/transport.py", line 4, in
from .connection import Urllib3HttpConnection
File "/usr/local/lib/python2.7/dist-packages/elasticsearch/connection/init.py", line 2, in
from .http_requests import RequestsHttpConnection
File "/usr/local/lib/python2.7/dist-packages/elasticsearch/connection/http_requests.py", line 4, in
import requests
File "/usr/lib/python2.7/dist-packages/requests/init.py", line 68, in
_attach_namespace(urllib3, 'requests.packages')
File "/usr/lib/python2.7/dist-packages/requests/init.py", line 63, in _attach_namespace
module = import(name)
File "/usr/lib/python2.7/dist-packages/urllib3/contrib/pyopenssl.py", line 55, in
import OpenSSL.SSL
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/init.py", line 8, in
from OpenSSL import rand, crypto, SSL
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/rand.py", line 10, in
from OpenSSL._util import (
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/_util.py", line 18, in
no_zero_allocator = ffi.new_allocator(should_clear_after_alloc=False)
AttributeError: 'FFI' object has no attribute 'new_allocator'

issue seems mroe upstream and generic with CFFI and openSSL python libs. pip --update cffi also reports same broken-ness
workaround: download 1.10.0 from https://pypi.python.org/pypi/cffi and build/install

Thanks for the workaround, I'll look into if this is something that needs to be added to the installer with changed packages.

Python-PIP is also broken and breaks the install in a similar manner when it reaches the Kibana import parts of your python installer. My workaround for that was to use the get-pip.py script and install pip that way, then the installer runs fine.

1. install cffi lib:
sudo apt-get install libffi-dev
2. install these packages manually:
sudo pip install elasticsearch
sudo pip install requests
sudo pip install flask-mail
sudo pip install flask_wtf
sudo pip install cryptography --upgrade
sudo pip install pyopenssl --upgrade

if there's no more error, run setup.py again.
as you can see at line 29 & 48 of this file: SweetSecurity/install/packages.py
os.popen('sudo pip install elasticsearch 2>&1').read()
os.popen('sudo pip install requests 2>&1').read()
os.popen('sudo pip install flask-mail 2>&1').read()
os.popen('sudo pip install flask_wtf 2>&1').read()
os.popen('sudo pip install cryptography --upgrade 2>&1').read()
os.popen('sudo pip install pyopenssl --upgrade 2>&1').read()

there's no guarantee to install these packages successfully because they're non-blocking tasks. and there's no log to show the result of installation.
I guess your failure is caused by the dependence issue when packages.py runs
os.popen('sudo pip install cryptography --upgrade 2>&1').read()
or
os.popen('sudo pip install pyopenssl --upgrade 2>&1').read()
and if my memory is correct, libffi-dev is a dependence of cryptography or pyopenssl


DatePicker /w Date-FNS

Create a solution that enables the consumers of SQForm to uses Date-FNS with our date picker components. We still need to support moment.js for backward compatibility.

Can we enhance our existing components without adding complexity?
Or will it be less complex to create SQFormDatePickerWithDateFNS.js?

We would also need to update our documentation to show users how to add Date-FNS to the context provider that is required to wrap any application that uses SQFormDatePicker's.
The end goal is consumers no longer need to install moment to use SQFormDatePicker components

I haven't set up this type of thing before, but from what I'm seeing in the Mui datepicker docs and in the SQForm codebase, it seems that all the users would need to do to make the SQForm datepickers work with date-fns is give the DateFnsAdapter to the LocalizationProvider instead of MomentAdapter. Then they'd need to migrate the use of moment to date-fns in their project.
Unless I'm missing something... ?
The only place SQForm uses moment is in src/utils/datetime.js, which actually isn't used anywhere.
       summary               unimported v1.11.0
─────────────────────────────────────────────────────────────
       entry file          : src/index.js

       unresolved imports  : 0
       unused dependencies : 1
       unimported files    : 1

─────┬───────────────────────────────────────────────────────
     │ 1 unused dependencies
─────┼───────────────────────────────────────────────────────
   1 │ moment
─────┴───────────────────────────────────────────────────────

─────┬───────────────────────────────────────────────────────
     │ 1 unimported files
─────┼───────────────────────────────────────────────────────
   1 │ src/utils/datetime.js
─────┴───────────────────────────────────────────────────────

@codyarose That's the hope! We attempted this with dayjs awhile back and ran into bugs with the library, specific to dayjs. Can we use MCE to test this out?

Can we use MCE to test this out?

Sure! My use of moment is pretty minimal there, so it shouldn't take much to switch. It would be a small test of date-fns typescript support as well.


anchor evaluation is not compatible with ytt directives on the same node

When converting a YAML file that used ERB templating to ytt we experienced an issue with anchors.
We were able to replicate it in this simple case.
Given output.ytt:
nullz: &nullz #@ "asdf"
asdf: *nullz

I expected the output to be:
nullz: asdf
asdf: asdf

But what I got was:
nullz: asdf
asdf: null

If this is a Works as Expected™, please document it clearly. Anchors are such a big part of YAML that this simple behaviour through us off in our concourse pipeline.

documented as known limitation here: https://github.com/k14s/ytt/blob/develop/docs/known-limitations.md. will close for now, and mark as "deferred".


COMMITTED write stream don't seem to work.

I tried to the BigQuery Write Storage API with COMMITTED stream. While there's no errors when writing data in the stream, the data doesn't seem to be inserted; doing a SELECT * FROM table returns 0 result. Using the same code, but with PENDING write stream and adding a call to BatchCommitWriteStreams works just fine.
Looking at the tests, it seems like the WriteStream API is only tested using PENDING stream.

Thank you for reporting this issue. While we cannot promise an immediate fix, any additional information you can provide would be incredibly helpful. If possible, please provide a failing test case, or reproducible code or steps. This will greatly assist in our investigation and potential resolution of the issue.
Thank you


Limiting the number of instances of an application

I'd like to know if it's possible to limit the number of instances registered at a given time.

@andreldm there are no limits (either on total number of instances, or number of instances per application) currently. Is this some new functionality you are looking at?

I was trying to limit the number of instance to only one at a time. Now I'm convinced this defeats the whole point of service discovery, so Eureka should not be ideal to my case.

@andreldm one of the tenants of eureka is that instances are abstracted away and application talk to each other over some fungible set of physical instances. If your use case require limiting it to just 1 instance, then yes eureka doesn't make too much sense.


fix: user Route and Google Oauth

#41
closes #41
👷🏻 Changes made

Added a new parameter usertype in Google Oauth and Google Callback in Auth Js which will set the type of user before creating a new account that will allow the user to open user profile or club profile easily.
Fixed the User route. Now individual can access their accounts easily.
Fixed the Display js. Now admin can see all the clubs presents

📸 Screenshots

This change will allow the user to add individual or club usertype from user for Google Signup. So I have created an option for this in frontend. So after it is merged I will pull request in Milan
Ref issue: #1091


testowa zamian

dadada

This change is

Review status: 0 of 1 files reviewed, 1 unresolved discussion

boards/arm/96b_carbon/96b_carbon.dts, line 109 at r1 (raw file):

	 * For more information, see:
	 * http://docs.zephyrproject.org/devices/dts/flash_partitions.html
	 *zmaina1

co to ma być?

Comments from Reviewable

Review status: 0 of 1 files reviewed, 1 unresolved discussion

boards/arm/96b_carbon/96b_carbon.dts, line 110 at r3 (raw file):

	 * http://docs.zephyrproject.org/devices/dts/flash_partitions.html
	 *zmaina1 zmina dfholaswhfo
	 *zmain2

gsgsgs

Comments from Reviewable


I am unable to get NTLM Hashes using Inveigh using WinPwn

I did check the directory but no file is created. Nothing that shows the ntlm hashes. I tried editing the invoke-Inveigh command with FileOutput -Disabled but still couldn't make it work.
Are there any logs that I can share? I did see that the Inveigh module of WinPwn would open a new PS Script process but going through some earlier issues (posted in Github), I learned that AMSI bypass was not getting applied to Inveigh. So I assume that Inveigh loads and runs on the existing PS session but still I am unable to get the NTLM hashes. No output file in the Directory.
Any help would be greatly appreciated!

It’s not loaded in a new Powershell process anymore. I did change that, so it’s running in the current process where the AMSI bypass definitely was applied before.
I cannot troubleshoot that for you, as I don’t know if any hashes were gathered at all. 🤷♂ Maybe there were no incoming connections?

Thanks for replying. I did start 2 inveigh sessions, one from Robertson's repo and the other from Winpwn. The screenshot is from a Win 10 PC - MNPC1 (<IP_ADDRESS>) and on a domain controller - MNDC (<IP_ADDRESS>), I tried accessing MNPC1 in 2 ways from MNDC:

from run window --> \<IP_ADDRESS>
from a browser window --> http://<IP_ADDRESS>

I did get NTLM hashes on Inveigh session from Robertson, but not on the WinPwn session:

Please let me know if I can share any other details.

What if you use -ConsoleOutput No in Inveigh? Because I’m using that in WinPwn, as you can see on the screenshot the output directory is one sub directory of your desktop folder. The hashes should be there in a text file.

WinPwn just doesn’t print the hashes out in the console window. Can you verify that the hashes are in a text file in the screenshot folder?

Thanks for taking the time. Yes, I did check the output directory for WinPwn but could not find any text files. As per your instructions, I did use the same configuration on Inveigh.ps1:
Invoke-Inveigh -ConsoleOutput N -NBNS Y -mDNS Y -HTTPS Y -Proxy Y -FileOutput Y
I got the text files, please check the screenshot:


Add ingress messages reference

Adds a reference document explaining ingress messaging.
If you are submitting a Pull Request for adding or changing content on the ICP Developer Documentation, please make sure that your contribution meets the following requirements:

[x] Follows the developer docs style guide.
[x] Follows the best practices and guidelines.
[x] New documentation pages include document tags.
[x] New documentation pages include SEO keywords.
[x] New documents are in the .mdx file format to support the previous two components.
[x] New documents are registered in /sidebars.js, otherwise, it will not appear in the
side navigation bar.
[x] Make sure that the .github/CODEOWNERS file is
filled with new documents that you added. This way we can ensure that future Pull Requests are reviewed by the right people.

Thanks for the additional feedback and input @roelstorms and @yvonneanne ! I addressed everything.

@roelstorms since you approved I took the liberty to resolve your comments after addressing them. But if you have anything to add please re-open them!
@yvonneanne could you PTAL? If you're happy with the page I'd proceed and merge this :-)


export summit proposals from CiviCRM for review by committee

looks like @jessicacoates got this.

:+1:


The "master-cs" branch should be deleted

From the README file:

All C# scripts must be submitted to a *-cs branch, e.g. master-cs, 3.1-cs, etc. This is to ensure that users who aren't using the Mono-enabled version of Godot do not have C# scripts present.

The master-cs branch is unmaintained and does not actually contain any C# scripts anyway. Therefore I propose deleting it and removing all references to it.
If in the future people do wish to add node extensions with C# scripts, I think that having a separate folder would be better, either /addons/godot-next-cs or /cs/addons/godot-next-cs. The current approach of having different branches is a bad idea. It leads to increased difficulty to maintain everything and/or the branches going unmaintained.
Additionally, perhaps some of the other branches should be looked at too.

@aaronfranke Sounds like very sound reasoning. It is more difficult to maintain precisely because it is a separate branch. I think it's a good idea.

Additionally, perhaps some of the other branches should be looked at too.

Yeaaaaa. A lot of them are WIP branches of mine. I wonder if I can fork my own repository and make another repository on my own account. Then I can just keep them all there and delete the various branches that clutter up the repo.

Done.
As for the other branches, I'm thinking of an alternative solution that would help people associated with different repositories avoid similar issues. Should have more info soon.


Update NEOS to MPB0.4

[Current coverage][cc-pull] is 94.18%

No coverage report found for master at d4a890d.

Powered by Codecov. Last updated by [d4a890d...6e6f72d][cc-compare]
[cc-base-branch]: https://codecov.io/gh/odow/NEOS.jl/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/odow/NEOS.jl/compare/d4a890d3f7aaa7f8af9aa168101c5ff316e3c624...6e6f72dd170a5c2e0a08e18c037239fab90ff010
[cc-pull]: https://codecov.io/gh/odow/NEOS.jl/pull/4?src=pr

Changes Unknown when pulling 30b86c1a010c388f829ac978fd6d5d241ca711c6 on updatesformpb0.4 into ** on master**.


Starts on system boot

This is a total nogo, I dont know the reason but when I start my system, among all other apps this script also starts, with a flatpak update (slowing it down already quite a bit) and then an rpm-ostree update, making it sluggish as hell.
The performance effect of that is extreme. Can you set the timer to only go off at 20pm or something?

A perfectly reasonable suggestion and PR. However, would you mind making a change? Allow me to explain...
Issue #1 asked about setting the update interval. I can see someone wanting a different start time. I addressed the request in issue #1 with instructions to change the interval. Maybe the solution here is better instructions from me (or both of us). Instead of removing OnBootSec, we should leave it for those who want their update to run on boot. Would you modify your PR so that both options are present but one commented out? Then one or both of us edit README.md so that the first step of installation is to customize the unit file and then run the installer script. Does that sound reasonable? Thanks for contributing!

Oh, yes. Add yourself to the copyright.

I just pushed a new README.md.


Plugin system

I just merged https://github.com/request/request/pull/1400 and https://github.com/request/request/pull/1406 here + refactored all of the 4 plugins to use similar structure.

I like the concept.  I'm a little worried about unexpectedly breaking things though, like #1407.
I have been swamped lately (just accepted a job with Automattic!) so I'll need at least a few days to do a proper review.  @mikeal @FredKSchott @seanstrom etc. any help appreciated.

I'm really not changing that much at this stage, just moving code around, the instance bug with auth is very silly, but apparently not caught by any of the tests.

I agree, I feel like I could have caught that in reviewing though, so I'd like to be more thorough this time.

OK, my point in https://github.com/request/request/issues/1409 was that it's about code modified in this PR + I already merged two other PR's here, so I don't want to push even more things here :)

I don't have time to do a full review but I will say that I like this approach. I'll be impressed if you can actually work the forwarding logic in to a plugin, that logic always seemed to infect parts of the code I never thought I'd have to worry about :)

@mikeal I'm not familiar with the entire code base in details tbh, but still the refactoring should start from somewhere.

@nylen off topic, but congrats on the new job!! :)

I'll be doing a thorough review today :)

Thanks @seanstrom take a look at this PR https://github.com/request/request/pull/1406 too, as most of my latest ideas are there. Also not really that much changed at this point, I'm really just moving code into separate scopes as I see fit. For example the last one - the redirect one is just something that I thought it deserves it's own scope, and in fact except two of the fields in this newly created class are used externally, so it looks like a separate entity after all.


[wc] Simplify extending renderer templates

The RenderTemplate declares optional properties to load external dependencies and additional CSS styles
It is a little of pitfall that one wishing to extend another template by decorating has to explicitly reassign those properties so that the renderer can find and call them.
To make extending templates easier, a simple decorator function could be provided which would do that automatically
import { decorate, PropertyTemplate } from '@hydrofoil/shaperone-wc/templates'

export const property = decorate<PropertyTemplate>((decorated: PropertyTemplate) => {
  return const decorator: PropertyTemplate = (ctx, arg) => {
    return decorated(ctx, arg)
  }
})

And any additional loadDependencies and styles should also be combined
import { css } from 'lit-html'
import { decorate, PropertyTemplate } from '@hydrofoil/shaperone-wc/templates'

export const property = decorate<PropertyTemplate>((decorated: PropertyTemplate) => {
  const decorator: PropertyTemplate = (ctx, arg) => {
    return decorated(ctx, arg)
  }

  decorator.styles = css`More styles`
  decorator.loadDependencies = () => import('my-component')

  return decorator
})

closed by #126


cargo install does not install v2.3.2 or more

Because https://github.com/rust-onig/rust-onig does not publish new releases on crates.io.
We have to avoid build error by explicitly specifing revision here https://github.com/greymd/teip/commit/82779d5b7900c74dde5115615f1c277c93c9e36e

If rust-onig is not actively updated, we will consider creating fork of rust-onig and publish it with appropriate manner stated below.
https://users.rust-lang.org/t/crate-how-to-publish-a-fork/57234


updating atlas game

Added atlas game with all countries.
Added new country finding logic, binary searching.
.....
Please test and send me any inconsistencies arising with the code
Thank you

The Atlas Program is working properly


sama5d2_xult: USB Host worked.

Signed-off-by: Takeyoshi Kikuchi<EMAIL_ADDRESS>

@centurysys Looks good to me! :+1:

@centurysys I'll test host with mass storage using a USB Flash drive tomorrow, as per Greg's suggestion. If that works I'll create a PR to get this into NuttX's main repo (crediting you too).


Create repository for solutions and implement them

We want to have a solution repository which is private and contains the solutions of the examples.

added new branch release/solutions.


Bug with Moco on validation step

🐛 Bug
I was using pytorch-ligntning and especially MocoV2 in the scope of my undergraduate thesis which I do at JetBrains Research, and I found the implementation of the validation stage strange.
As was shown in the following issue https://github.com/PyTorchLightning/lightning-bolts/issues/615 , currently in MocoV2 the validation step is absolutely identical  to the train step, and in connection with this the following two actions occur:

The key encoder is updated on every validation step https://github.com/PyTorchLightning/lightning-bolts/blob/5577453a6d7072724d9ae24184daf8f45d4baff7/pl_bolts/models/self_supervised/moco/moco2_module.py#L223
The queue is filled with both train and validation samples https://github.com/PyTorchLightning/lightning-bolts/blob/5577453a6d7072724d9ae24184daf8f45d4baff7/pl_bolts/models/self_supervised/moco/moco2_module.py#L254

Both of these events lead to the data leak, which affects both the train and validation process.
To solve this issue we can do the same thing as was done in BYOL, where the callback was implemented, and the update only happened at the end of the training batch https://github.com/PyTorchLightning/lightning-bolts/blob/5577453a6d7072724d9ae24184daf8f45d4baff7/pl_bolts/callbacks/byol_updates.py#L42
I can implement this feature and create a pull request with it since Moco is a very important part of my bachelor thesis. Also, I would tag @williamFalcon since he was stated in the docs as an author of this module and also @annikabrundyn who implemented BYOL. What do you think about it?

@maximzubkov Thank you for reporting the issue. It should be addressed in #631.


unload() does not clear error message

I noticed this problem in Tobira, a single page app. When I visit a video with invalid URL, the "error loading video" message appears as expected. But when I then click on another video, the message does not disappear. Clicking on another video causes: unload() to be called on the current Paella object, and then a new Paella object is created on the same DOM node.I guess unload just need to clear the error message and everything else from the tree as well.

That use case occurs on the official paella player website, and there are three examples of errors that are recovered in a SPA website:
https://paellaplayer.upv.es/#/demos/manifest-not-found
Once loaded, try another video. Some of the video sources don't work, but that's because I have to spend some time reviewing the demos. You can try Dual stream video or HLS Multi Audio, those do work.
The paella player site is SPA, but there is one case in which the entire web is reloaded: if you try to load another video during a transition between two states, it's impossible to "cancel" that transition, and the only way to load another manifest without wait until the transition is done is reloading the page. For example, if you try to load a video that takes a long time, such as Triple Stream, and before it finishes loading you try to load another one, in that case the page reloads completely, because otherwise you have to wait for the previous video to finish loading.
You can check how it is implemented at paellaplayer.upv.es at github:
https://github.com/polimediaupv/paellaplayer.upv.es/tree/paella-7-site
It is implemented with Svelte. Check the src/Player and src/Demos components for the implementation, in case it helps. You can also refer to the paella player lifecycle documentation. If you find any inconsistency between the documentation and the behavior of the player, please let me know so I can fix it.
https://paellaplayer.upv.es/#/doc/life_cycle.md
There is one case where an error may occur when clearing the error message: if the new video manifest does not have the metadata.preview field set, the error is not cleared properly. Note that this is NOT A BUG. The metadata.preview field is mandatory in paella core, so if it is not present the player behavior is unpredictable.

Oh, we did in fact not set metadata.preview, so that's likely the problem. I'd argue that a preview image should not be mandatory though. But that's a different topic.
We solved it in another way, by forcing React to not reuse the DOM node.


Text element: Resizing/rotating should be possible in edit mode

Feature Description
By original design, the Movable features (such as resizing and rotation) were disabled for a Text element in edit mode. The design has now been updated and it should be possible to scale and change the width of a text element in edit mode as well.
See Figma.

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

It should be possible to scale and change the width of a Text element in editor mode
It should be possible to rotate an element in edit mode
The font size should be adjusted while resizing, just as in the display mode

QA Instructions

Tested in PR#2699
When I start typing content immediately after adding the text element and THEN grab a handle, the text entered is automatically selected
Video: https://drive.google.com/file/d/1_pIXpYWt7na7j-7OYnin6nWeG8mn7oW3/view

Maybe we can merge this and create a separate issue for this specific edge case. I've tried disabling pointer events from the edit mode and it still happens -- not sure why the selection happens but perhaps it's not worthwhile to keep the full PR blocked for that. It also seems to happen on the main branch when dragging the mouse in the upper right corner while in edit mode.
@barklund WDYT?

I agree, let's move file in separate issue as it's a bit esoteric to re-create.

Follow-up issue: https://github.com/google/web-stories-wp/issues/3303


GFL8KFQC-euc Can't export library

Support log ID: GFL8KFQC-euc
I can't seem to export my library - it loads and then nothing happens.

I don't see an export happening in GFL8KFQC-euc. What exporter did you use?

A simple restart of the computer was all I needed. Sorry for the convienience and thank you again for an amazing plugin!


Wrap text by default when users go to text mode

Summary

How did you test this change?

[ ] Test(s) added
[x] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

https://user-images.githubusercontent.com/87973126/214066559-8c135052-3ec6-4c4b-88e2-6a51af24de5c.mp4

@beekemarie


Lecture 10 related question

Description
Hi, may I ask what does the number (3,3,5) in the return value of Thread.currentThread.getName() mean? Thanks!
Screenshots

it uniquely identifies the worker thread in the ForkJoinPool, to help understand of how concurrent tasks are managed and executed.(cf2 is directly chained to cf1 using thenRun(), which means cf2 will execute immediately after cf1 on the same thread. however, cf3 use thenRunAsync(), which means it will run asynchronously, so that the worker thread is different) Also, notes that thread could be reused, so the number u see in one execution may differ in another due to reuse and the dynamic state of the thread pool at the time of execution.

it uniquely identifies the worker thread in the ForkJoinPool, to help understand of how concurrent tasks are managed and executed.(cf2 is directly chained to cf1 using thenRun(), which means cf2 will execute immediately after cf1 on the same thread. however, cf3 use thenRunAsync(), which means it will run asynchronously, so that the worker thread is different) Also, notes that thread could be reused, so the number u see in one execution may differ in another due to reuse and the dynamic state of the thread pool at the time of execution.

Oh I see thank you!


Add v1beta3 files to update-demo

I hesitate to start actually talking about v1beta3 in the README since this is supposed to be an example for a complete beginner.

You'll also need to update the e2e test that runs through this example since you're changing the file location. See https://github.com/GoogleCloudPlatform/kubernetes/blob/master/test/e2e/kubectl.go#L47

Fixed, PTAL. I ran that e2e test to verify that it still runs.

LGTM. Will merge once travis completes.

I think the example test is no longer passing, I'll fix later.

Fixed, PTAL

LGTM


pkgsStatic.openssh: fix build

Remove reference to libredirect for cross builds, as for those the tests are disabled:
https://github.com/NixOS/nixpkgs/blob/4818f809a92b2ef8d8c002c10a29f1080106ee81/pkgs/stdenv/generic/make-derivation.nix#L103

Motivation for this change
Closes: #154048
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

cc @KAction

@GrahamcOfBorg build pkgsStatic.openssh


Allow { in defines in .te files

Currently
define(`something', `{ thing1 thing2 }')

leads to an parser error because the curly braces are not allowed in the
second argument of the definition. As this seems to work in practise,
this seems like a parser problem to me. This commit allow the curly braces
to be used in these definitions.

Looks great, thanks!


Build error when adding crate protocol library to anchor project

I created a project with anchor init and added only crate as follows, but it does not build. Can you gays tell me what the problem is?
[dependencies]
anchor-lang = "0.23.0"
crate-token = { version = "0.5.0", features = ["cpi"] }

This needs to be resolved by the crate-token crate and is not an Anchor issue.


Updated smartling retry configuration

Updating the Smartling retry back to a max back off of 60 seconds to better handle Smartling rate limiting exceptions.
See https://help.smartling.com/hc/en-us/articles/1260805145550-Rate-Limits for further information.

Sorry to barge in, but I noticed the comments in the tests weren't updated to reflect the increase in retry count. Perhaps instead of updating them, they could be removed? Or replaced with a general "1 more than number of retries"?

Thanks @tedyoung, I've raised a PR to remove the comments in https://github.com/box/mojito/pull/746


Filter threads

@apangin This is a prototype implementation of the feature that I was talking about in the #237 issue. The implementation is immature and has a lot of place for improvement but I still wanted to get your opinion.
I have tested the following scenarios and it behaved as it was supposed to

Changing thread name
Exiting from running threads
New thread creation
High load of threads ~ 10k threads

I decided to divide the changes into small pr-s so if all goes well I will implement it for the CPU profiler as well.

It seems to me that the CI is using  g++ version 4.3 and there were no regular expressions implemented back then. Could you change the CI g++ version or should I somehow change my implementation ?

Here is a simpler version where it basically checks if the given word is a substring of the thread name. Still a simple and premature implementation but I would appreciate your feedback ))

@apangin Did you have a chance to look at the new changes )))?

@apangin Did you have chance to take a look at the changes )))?

Did you have chance to take a look at the changes )))?

Not yet, sorry. I'm a bit busy this week, but I'll definitely take a look later.

Sorry, I have been busy lately, in the upcoming week will submit the new version.

What's the status on this?  If @guser21 is no longer interested in this feature, I'm happy to take a swing at getting the PR into an acceptable state.

The work on filtering continues in https://github.com/jvm-profiling-tools/async-profiler/tree/thread-filter

Closing the PR as obsolete.


Delay 0 should show loading state in the current event loop

When delay is set to 0, loading state should show up immediately. Currently showing the loading state is delayed until the next event loop due to setTimeout.

Just FYI your git config is messed up so your GitHub profile isn't tied to this commit. I'll give you a chance to update it if you want.

@thejameskyle thanks! I've fixed the author info.


Invoke plugin methods after permission is granted

This PR resolves the scenario where a plugin method is invoked for the first time and permission has not yet been granted. In this scenario, the permission prompt will come, but the API that was requested will never be invoked.
The fix is to invoke the API method after permission has been granted.
My Swift is not strong so please offer any suggestions or fixes (particularly I am not positive on self vs weakSelf usage with closures).
Note: it is easier to look at this PR if you ignore whitespace changes with ?w=1 parameter (https://github.com/codeborne/cordova-plugin-contacts/pull/5/files?w=1).

@l3ender Thank you for the contribution. Currently we don't actively maintain this plugin.
Probably we will review pull requests when we get back from quarantine.


RequestClientCredentialsAsync error Newtonsoft.Json.JsonReaderException: Error reading JObject from JsonReader

Hi.
I had a project original in .Net Core 2.0, and updated to .Net Core 2.1.
The project runs perfectly when it is in a localhost, but once I put it in the server I am getting an error when I request the token in this line "var tokenResponse = await tokenClient.RequestClientCredentialsAsync("api1");"
It says: tokenResponse.Error: Error reading JObject from JsonReader. Path '', line 0, position 0.
So I created a solution from scratch, following the documentation and the examples in identityserver4 quickstart, and I am getting the same exactly error once is in my host.
Here is what the tokenResponse.Exception.StackTrace says:
at Newtonsoft.Json.Linq.JObject.Load(JsonReader reader, JsonLoadSettings settings)
at Newtonsoft.Json.Linq.JObject.Parse(String json, JsonLoadSettings settings)
at IdentityModel.Client.Response..ctor(String raw) in C:\local\identity\model\IdentityModel2\src\IdentityModel\Client\Messages\Response.cs:line 22
And this is the short program that I am running to test the idsvr4:
` public class Program
{
public static void Main(string[] args) => MainAsync().GetAwaiter().GetResult();
    private static async Task MainAsync()
    {
        // discover endpoints from metadata
        //var disco = await DiscoveryClient.GetAsync("https://localhost:44366/");
        var disco = await DiscoveryClient.GetAsync("https://www.luisbragagnolo.com/");
        if (disco.IsError)
        {
            Console.WriteLine(disco.Error);
            return;
        }

        // request token
        var tokenClient = new TokenClient(disco.TokenEndpoint, "client", "secret");
        var tokenResponse = await tokenClient.RequestClientCredentialsAsync("api1");

        if (tokenResponse.IsError)
        {
            Console.WriteLine(tokenResponse.Error);
            return;
        }

        Console.WriteLine(tokenResponse.Json);
        Console.WriteLine("\n\n");

    }
}`

The server still is in my host in case your would like to test.
So I am lost....any help would be appreciated.
Thanks

Check your logs in your token server. Also, check the network traffic and see what content type or payload is returned. It might be html being returned (due to an error and an error page), not JSON.

Hi brockallen.
I tried with different tools and I am having the "HTTP/1.1 400 Bad Request" Which I don't understand why. It should be easy and straight forward.

Yes, the logs will make it easy and straight forward to know what the problem is.

I understand, I found some logs from IIS in my private server, I hope it helps.
#Version: 1.0
#Date: 2018-07-31 18:16:06
#Fields: date time s-ip cs-method cs-uri-stem cs-uri-query s-port cs-username c-ip cs(User-Agent) cs(Referer) sc-status sc-substatus sc-win32-status time-taken
2018-07-31 18:16:06 <IP_ADDRESS> GET /.well-known/openid-configuration - 7000 - <IP_ADDRESS> - - 200 0 0 1693
2018-07-31 18:16:06 <IP_ADDRESS> GET /.well-known/openid-configuration/jwks - 7000 - <IP_ADDRESS> - - 200 0 0 58
2018-07-31 18:16:06 <IP_ADDRESS> POST /connect/token - 7000 - <IP_ADDRESS> - - 400 0 0 420
2018-07-31 18:16:34 <IP_ADDRESS> GET /.well-known/openid-configuration - 7000 - <IP_ADDRESS> - - 200 0 0 3
2018-07-31 18:16:34 <IP_ADDRESS> GET /.well-known/openid-configuration/jwks - 7000 - <IP_ADDRESS> - - 200 0 0 7
2018-07-31 18:16:34 <IP_ADDRESS> POST /connect/token - 7000 - <IP_ADDRESS> - - 400 0 0 120
2018-07-31 18:23:21 <IP_ADDRESS> GET /.well-known/openid-configuration - 7000 - <IP_ADDRESS> - - 200 0 0 28
2018-07-31 18:23:21 <IP_ADDRESS> GET /.well-known/openid-configuration/jwks - 7000 - <IP_ADDRESS> - - 200 0 0 3
2018-07-31 18:23:21 <IP_ADDRESS> POST /connect/token - 7000 - <IP_ADDRESS> - - 400 0 0 65
2018-07-31 18:26:45 <IP_ADDRESS> GET /.well-known/openid-configuration - 7000 - <IP_ADDRESS> - - 200 0 0 5
2018-07-31 18:26:45 <IP_ADDRESS> GET /.well-known/openid-configuration/jwks - 7000 - <IP_ADDRESS> - - 200 0 0 12
2018-07-31 18:26:45 <IP_ADDRESS> POST /connect/token - 7000 - <IP_ADDRESS> - - 400 0 0 67

Found the problem!!!
It was the connection to the database, I had a few places as "debugconnection" and "defaultconnection" but I forgot to check if the environment was debug or not.
Thanks and please feel free to close it.


Add option to indent query separator ";" on a final new line

A new conflig flag might be overkill or this could be yet another non opiniated option, but sharing in case there someone knows a smarter way to propose the same functionalities with as less friction as possible.
Flag:
--indent-query-separator
Indent query separator ";" on a final new line
e.g.
If set to true:
  SELECT * FROM foo;

the ending ';' will be indented on its own line:
  SELECT
    *
  FROM
    foo
  ;

Instead of:
  SELECT
    *
  FROM
    foo;

Coverage increased (+0.001%) to 99.816% when pulling f3fb2323591adf87283ec721bc5f382d55441f41 on romainr:master into 3cd6d98d2ab1b05e9588f0f80f30067b54326ef9 on zeroturnaround:master.

This feature has now been incorporated to sql-formatter. There's now a new config option named newlineBeforeSemicolon.
Thanks for the PR.


[SPARK-14254][Core]Add logs to help investigate the network performance

What changes were proposed in this pull request?
It would be very helpful for network performance investigation if we log the time spent on connecting and resolving host.
How was this patch tested?
Jenkins unit tests.

Test build #54462 has started for PR 12046 at commit 769cc14.

cc @nongli

Test build #54462 has finished for PR 12046 at commit 769cc14.

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

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

Merged build finished. Test PASSed.

LGTM

Merging to master


New solution for estimate_symbolic_duration

Adressed issue #371
Introduced two new clauses that collapse the prediction to empty symbolic duration to avoid wrong values.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.29%. Comparing base (3cd1e41) to head (0e77947).
Report is 180 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #372      +/-   ##
===========================================
- Coverage    84.11%   82.29%   -1.82%
===========================================
  Files           82       88       +6
  Lines        14506    15463     +957
===========================================
+ Hits         12202    12726     +524
- Misses        2304     2737     +433

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thank you for the quick fix! I added some minor details to the estimation but I think it works nicely now. Since I added code, feel free to review and merge.


Updated validations.py python script.

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

i  am adding new features.


Check if a Binary Tree is BST

I would like to add code for checking if a given binary tree is BST or not. And explaining my approach with comments.

I would love to solve this issue...Please assign this to me.

Please assign this issue to me.

This repo has some issue so please make a request on my other repo https://github.com/ravi-singh-100/DSA-Hacktober2022 .

I would love to solve this issue...Please assign this to me.

This repo has some issue so please make a request on my other repo https://github.com/ravi-singh-100/DSA-Hacktober2022 .

Would you, Please assign this issue to me ,,,,,

I would love to solve this issue...Please assign this to me.

This repo has some issue so please make a request on my other repo https://github.com/ravi-singh-100/DSA-Hacktober2022 .

Is this not related to hacktoberfest?Because even if i have contributed i didnt get any progress


[3.1.x] Fixed #32713, Fixed CVE-2021-32052 -- Prevented newlines and tabs from being accepted in URLValidator on Python 3.9.5+.

In Python 3.9.5+ urllib.parse() automatically removes ASCII newlines
and tabs from URLs [1, 2]. Unfortunately it created an issue in
the URLValidator. URLValidator uses urllib.urlsplit() and
urllib.urlunsplit() for creating a URL variant with Punycode which no
longer contains newlines and tabs in Python 3.9.5+. As a consequence,
the regular expression matched the URL (without unsafe characters) and
the source value (with unsafe characters) was considered valid.
[1] https://bugs.python.org/issue43882 and
[2] https://github.com/python/cpython/commit/76cd81d60310d65d01f9d7b48a8985d8ab89c8b4
Backport of cfe6b8bdd63890e4bb6482a806f3a9065045291e from main.

Merged in afb23f5929944a407e4990edef1c7806a94c9879.


⚠ Bofrost.fr has degraded performance

In 1918bf0, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1113 ms

Resolved: Bofrost.fr performance has improved in 00d6b29 after 48 minutes.


Compile distinct browser source code

There should be a possibility to substitute arbitrary source files with the browser-side implementation of its functions.
Currently neither Webpack nor Jest can handle distinction among of Node or Browser functionalities at build.
The proposal is stick with NodeJS implementation by default and where is necessary create a [X].browser.ts file which fully implements functions of [X].ts file.

Closed by #25


Feat: added Preloader/VerticalTiles

Fixes #249

@hari @sudhashrestha 🙌🏻

Triggering workflow by closing and opening the PR.