Fix hasFile Method to Accurately Identify Files in Request

Summary
This pull request addresses an issue in the Illuminate\Http\Client\Request class where the hasFile method could incorrectly identify non-file data as a file when checking for multipart form data. The fix enhances the logic in the hasFile method to ensure that only actual file uploads are recognized as files, preventing false positives.
Problem
The hasFile method relied on checking whether the contents of a request's data matched certain criteria, but it lacked a robust mechanism to differentiate between actual file uploads and plain string data. This led to scenarios where non-file data with the same key as an expected file could be misinterpreted as a file, resulting in inaccurate behavior.
Solution
The proposed fix introduces the following improvements:

Resource Check: The method now includes a check using 'is_resource($file['contents'])' to determine if the 'contents' of the request data are a valid file resource (e.g., a file handle).
Enhanced Validation: The existing 'json_encode($file['contents']) !== false' check is retained to cover binary data scenarios.
Test Coverage: New test cases have been added to verify that 'hasFile' correctly distinguishes between non-file data and actual file uploads, ensuring that false positives are avoided.

Impact

Backward Compatibility: This change does not introduce any breaking changes and is fully backward compatible.

Accuracy Improvement: The method now more accurately identifies file uploads, leading to better reliability in handling multipart form data in HTTP requests.

I am not sure if I follow the reasoning.
The component being changed is the lluminate\Http\Client\Request, which allows sending requests out of the application, right?
Currently, if one needs to send a simple string as an attachment, per a 3rd party API constraint, for example, one could just call PendingRequest@attach() with a string (usually proxied by the factory object), and if needed use hasFile() to check if the pending request, has a pending file attachment, right?
If this PR gets merged, one would need to create either a real or a temp file, and attach its handler, to have the hasFile() working? For a request that is pending to be sent later by Guzzle?
What is the actual benefit?

Although there is no change in the method signature, the behavior is hugely changed, IMO it is a breaking change.
It is not an improvement per se, or a bug fix, but a behavior change. It might be the desired behavior, and if so, should be merged. But nonetheless, it seems like a behavior change that can cause previous code to stop working.

I am not sure if I follow the reasoning.
The component being changed is the lluminate\Http\Client\Request, which allows sending requests out of the application, right?
Currently, if one needs to send a simple string as an attachment, per a 3rd party API constraint, for example, one could just call PendingRequest@attach() with a string (usually proxied by the factory object), and if needed use hasFile() to check if the pending request, has a pending file attachment, right?
If this PR gets merged, one would need to create either a real or a temp file, and attach its handler, to have the hasFile() working? For a request that is pending to be sent later by Guzzle?
What is the actual benefit?

Thank you for your questions and for taking the time to review the proposed changes.
You're correct that Illuminate\Http\Client\Request is responsible for sending requests out of the application. The hasFile() method is designed to check if a request contains a file attachment, which is useful in certain scenarios, especially when interacting with APIs that require file uploads.
The problem with the current implementation of hasFile() is that it does not accurately distinguish between actual file uploads and simple string data. For instance, if a request contains a string value that is expected to be a file, hasFile() might incorrectly return true, even though the string is not a file. This could lead to unexpected behavior, especially when the application or a third-party API expects a real file upload.
The key benefit is that this change improves the accuracy and reliability of the hasFile() method. It eliminates the risk of false positives when dealing with file uploads, ensuring that the method only confirms the presence of real files. This is particularly important when working with APIs that have strict file upload requirements.
In scenarios where you're attaching a string due to third-party API constraints, this change encourages more explicit handling of file uploads by requiring a file resource. This makes the code more predictable and reduces the potential for errors when working with file uploads in HTTP requests

Although there is no change in the method signature, the behavior is hugely changed, IMO it is a breaking change.
It is not an improvement per se, or a bug fix, but a behavior change. It might be the desired behavior, and if so, should be merged. But nonetheless, it seems like a behavior change that can cause previous code to stop working.

You’re correct that this could be considered a breaking change since it modifies how hasFile() interprets what constitutes a file. Code that previously passed a string and expected hasFile() to return true might now fail unless an actual file resource is used.
The primary reason for this change is to improve the accuracy of the hasFile() method in identifying genuine file uploads. The current behavior can lead to false positives, which could result in bugs, especially when interfacing with APIs that strictly require files and not strings.
In conclusion while the change is intended to improve the robustness and accuracy of file handling, it’s clear that this could impact existing applications that rely on the current behavior. If this is the desired direction, we should clearly communicate the change to users and consider the best way to introduce it without causing disruption.

The problem with the current implementation of hasFile() is that it does not accurately distinguish between actual file uploads and simple string data. For instance, if a request contains a string value that is expected to be a file, hasFile() might incorrectly return true, even though the string is not a file.

Sure, but if all one wants is to attach some string to be sent as a file, on a multipart request body, and later check if they have attached it, the current implementation has them covered up.
As this object is constructed by the application developer, and not by parsing external input such as its Illuminate\Http\Request counterpart, I don't see the benefit of the additional check, granted it still allows simple strings to be sent as file attachments.
But let's wait for the maintainers' verdict.

Can you describe what is actually going wrong in your production app with the current method and how to reproduce?

Please mark as ready for review when the requested changes have been made.

Can you describe what is actually going wrong in your production app with the current method and how to reproduce?

sure, create a form with multiple non-file fields (e.g., text, arrays, or JSON data).
Submit the form without any actual file attachments.
Use Request::hasFile('some_field') to check for file uploads.
Expected outcome: Request::hasFile should return false when no files are uploaded.
Actual outcome: Request::hasFile returns true, incorrectly identifying non-binary fields as files.

Also, how are you accessing the Illuminate\Http\Client\Request instance? Since it is manually created internally by the Illuminate\Http\Client\PendingRequest instance before dispatching it to guzzle?
And therefore the Illuminate\Http\Client\Request@hasFile() is inaccessible? Since it is not exposed nor by the Illuminate\Http\Client\Request class, not by the HTTP Façade?
Is it from a Guzzle Middleware?
https://laravel.com/docs/11.x/http-client#guzzle-middleware

Yeah a simple copy and paste reproduction would be helpful.


Python2.7 build with GCC 5.4.0

Hi guys,
First, thanks a lot for this great project! Solved a lot of confusions and headaches for me and team around AWS Lambda.
We are building and deploying python projects in AWS Lambda and one of the dependencies in Pystan. Installing the latest version of pystan (2.19.0) requires gcc 5 compared to gcc 4.8.3 on the lambci python2.7 images.
Is there a way to prep a build image with, say, gcc 5.4.0? I'm happy to do the legwork but need some guidance.

You'd need to build gcc 5.4.0 for Lambda, store that tarball somewhere (eg, S3) and then download and unpack it before your build.
The main thing to be aware of is that /tmp is the only writeable directory in a Lambda, so you'd need to compile your gcc in a way that it references all of its dependencies from /tmp.
You should be able to use Amazon Linux or a docker-lambda build image to do this: https://hub.docker.com/r/lambci/lambda/


Roll engine 7d3e722349e7..3c4dbe223cfe (17 commits)

https://github.com/flutter/engine/compare/7d3e722349e7...3c4dbe223cfe
git log 7d3e722349e7923115f0b8800022a08c8dd795b3..3c4dbe223cfefaedc6c13a80b3cdbc0f0c90aed5 --no-merges --oneline
3c4dbe223 Revert " Roll src/third_party/dart 67ab3be10d...43891316ca (#9670)" (flutter/engine#9673)
5e596f2ac  Roll src/third_party/dart 67ab3be10d...43891316ca (flutter/engine#9670)
46a2239b4 Roll src/third_party/skia 5b52c52141ac..11eb847a2080 (6 commits) (flutter/engine#9671)
b84f89b44 Allow embedders to add callbacks for responses to platform messages from the framework. (flutter/engine#9655)
8dac2e979 Begin separating macOS engine from view controller (flutter/engine#9654)
d3616c7a3 Roll src/third_party/skia 0e0113dcbd9a..5b52c52141ac (8 commits) (flutter/engine#9665)
cea2c3617 Mutators Stack refactoring (flutter/engine#9663)
6a8782fe0 Roll src/third_party/skia 93eeff578b08..0e0113dcbd9a (9 commits) (flutter/engine#9662)
791143f17 ExternalViewEmbedder can CancelFrame after pre-roll (flutter/engine#9660)
d637f291a External view embedder can tell if embedded views have mutated (flutter/engine#9653)
ceee3d733 Roll src/third_party/skia 38ae3f42fec1..93eeff578b08 (1 commits) (flutter/engine#9659)
d75729006 Roll src/third_party/skia febc162c7898..38ae3f42fec1 (3 commits) (flutter/engine#9658)
58133abce Roll src/third_party/skia 3de5c6388142..febc162c7898 (2 commits) (flutter/engine#9657)
29342dd30 Roll src/third_party/skia 1e2cb444e0c1..3de5c6388142 (8 commits) (flutter/engine#9656)
b5473560d Pipeline allows continuations that can produce to front (flutter/engine#9652)
8306ee68c Move the mutators stack handling to preroll (flutter/engine#9651)
511b9f23a Roll src/third_party/skia 215ff3325230..1e2cb444e0c1 (12 commits) (flutter/engine#9650)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6393136501227520

Commit queue failed; closing this roll.


Add support for role attribute querying to CLI

Do we want to commit to master?
This will cause what is documented as 0.13 functionality will be in 0.12.x.

Do we want to commit to master?
This will cause what is documented as 0.13 functionality to be in 0.12.x.

I'll squash all the release notes into a single doc and give these a point release note. I can't guarantee their correctness unfortunately. Ordinarily I'll bump the minor, but that amounts to a major bump now. I can't guarantee that the autogenerated build number will line up exactly, if someone else merges around the same time as I do, or if a smoke build flakes.
I may just bump the minor anyway. If all the release notes are together, it doesn't feel like as big a deal to bump the number.

Merged all release notes into single file.
Updated PR target to release-0.13 so we can stage more changes for the next release


Can't chose window size

I like the idea of sliding a window along a read to check for a drop in quality, but sickle seems to be missing a crucial option, how can one choose the size of the window ?

For the record, PR #36 lets you do this.


aws-sdk dependency version updated to avoid sigV4 error

I was constantly getting "unable to upload error non file stream objects are not supported with sigv4" error.
When I updated aws-sdk version, it worked fine for me.

this is never going to be merged?!

@shreyawhiz , could you create a new package for everyone to use? :)

Seems like this is dependency is not going to be updated. You can use my fork if you're experiencing this issue:
https://www.npmjs.com/package/@monolambda/s3


use try and except block around StopIteration or avoid StopIteration completely

StopIteration is deprecated from python3.6+, https://docs.python.org/3.6/whatsnew/3.6.html#deprecated-python-behavior
As per https://www.python.org/dev/peps/pep-0479/ we should not use raise StopIteration on generators.
See #636

@znation @TobyRoseman Please assign this ticket to me as I would like to try fixing it and contribute. Thanks in advance!


[DISCO_F746NG] Add support of this target in RTOS config files

We still need to add files to support for M7 or the way to reuse M4 files as it seems they should be compatible thus reusable


is incorrectly inserted as

As title. As time goes by, the formatting will become weird because we can have something like

    some text

Can we force  always by itself, not enclosed within any other tag?

Browser-level bug, I do not want to try to fix that in version 2
Wait for Trumbowyg 3 :)


data type

Please align to other style guides in 80% of all CSP guides? (remember this goes to a Global Audience)
A valid data point is published by Open Mobile Alliance (which Microsoft adopted MDM DM client formats from) as:
String
String (XML file)
Date and time
Integer
Floating point
Boolean
Base64 (file)

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

ID: 167fe82b-002b-39a5-ffc4-e77e10e23302
Version Independent ID: 6d416711-11b7-8231-0ced-343cbd7103f1
Content: SurfaceHub CSP
Content Source: windows/client-management/mdm/surfacehub-csp.md
Product: w10
GitHub Login: @MariciaAlforque
Microsoft Alias: maricia

@officedocsbot assign @mypil

@Tdiwestyguy - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@Tdiwestyguy - From our understanding, the issue has been resolved but it may take a few days for the merged content to appear in the article. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


fix: updating Select/StyledValueContainer to fix text color in dark mode

Description
In dark mode color of the input blends with the background , so one can't see what is the value

Scope

[X] Patch: Bug Fix
[ ] Minor: New Feature
[ ] Major: Breaking Change

uhm,  not sure why probot/pr-label  and screener are not happy. Any suggetions?

Hi @DianaSuvorova - thanks a lot for the fix!
It seems like screener is unhappy because this commit changed the visual of the datepicker:

can you take a look?


Modify pip command in CreateJupyterBook notebook code cells (#12808)

Update command for installing jupyter-book in notebook to prevent issues with pip running on a different python installation.

Pull Request Test Coverage Report for Build 296305824

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 38.63%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
74.52%

Totals

Change from base Build 296128843:
-0.01%

Covered Lines:
19247

Relevant Lines:
45431

💛  - Coveralls


added support for es7 decorator (a la angular2)

fixes https://github.com/Microsoft/TypeScript-TmLanguage/issues/8
followed from https://github.com/Microsoft/TypeScript-Sublime-Plugin/issues/476
typescript with es7 descorators should like this now:

Hi @orizens, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

@anubmat  can you take a look.

@orizens , Thank you for your contribution, However, there are two modifications I would like you to make before I can review these changes:
First, I see you have made changes to the pList file - "TypeScript.tmLanguage". There is a YAML file "TypeScript.YAML-tmLanguage" that can be converted to this pList file. It is easy to make changes to the YAML and convert it to pList. We do not want the YAML to be stale, so they must remain in sync. I can help you out here if you need help with this.
Secondly, there is another YAML file - "TypeScriptReact.YAML-tmLanguage" that also converts to pList file by the name: "TypeScriptReact.tmLanguage". Corresponding changes need be made to this file so that TypeScript and TypeScriptReact remain in sync.
So in total, there must be 4 files changed - 2 YAMLs and 2 pLists

hi @anubmat
I can update the yaml - no probs.
Regrading conversion - sublime can now read YAML with "sublime-syntax" extension.
Which tool you're using to convert to tmLangauge?

Hi @orizens , You can use sublime text's package dev after installing package control. Here is a link to the repo and details on how to use it - https://github.com/SublimeText/PackageDev. It will generate a tmLanguage (pList) from the YAML.

hi @anubmat
I updated the yaml file and generated a tmLanguage file.

Hi @orizens , can you please check why some tests are failing, by running npm test from tests folder, maybe you have to fix some baselines. Also as mentioned earlier, can you also make changes to TypeScriptReact.YAML-tmLanguage and its corresponding tmLanguage file. Also it will be great if you can add some testcases.

hi @anubmat
i updated the TypescriptReactYML.
however - i can't figure the tests failures.

Refer to this link  for details on how to do the testing locally. Some of your tests failed because you introduced new scopes, which is okay but you may have to go over these tests, find differences, compare results in test/generated folder against test/baselines folder and either update the files in test/baselines or improve some of your changes to pass existing baselines.

hi @anubmat
sorry - I don't understand how the tests for this package works.

I am looking at the test failures, you are failing 14/33 tests. I am pointing out a few places, where changes can be made.
Just for your reference here is the process to test:
from the folder 'TypeScript-TmLanguage\tests', run:
npm install
npm test
It will list failed cases. The test case files are present in 'TypeScript-TmLanguage\tests\cases'. In these test typescript files, '^^' is used to test the scope of the following word/code.
Every testcase is run with npm tests and generates files in 'TypeScript-TmLanguage\tests\generated' folder. These are compared against existing files ( or baselines) in 'TypeScript-TmLanguage\tests\baselines' folder.
The differences in these files is what leads to failing tests. You can always compare these files, and if you think the ones generated have better scopes, you can change the corresponding baseline and check-in the updated baseline. If on the other hand you feel your generated files give unacceptable scopes, you should update these grammar files to correct those scopes.

i'm not sure either.
i think i'll start from scratch.

Hey @orizens! If you do send a follow-up PR, just a few comments based on what you've sent so far:

Do not base this work off of other repos. It appears that you adapted the work from the Babel TmLanguage file, where the license is currently not compatible and the adapted portions don't contain attribution. I don't think it's a good idea to try to "meld" the files together like this anyway.
Keep versions and project names (like "ES7" and "flowtype") out of scope names.

Feel free to ping us if you have any questions.

hi @DanielRosenwasser
notes your suggestions.
I opened a new pr - #201
There were too many confusions with this one.
With the new PR - I also added html syntax highlight in template strings - which is a much needed feature .
You're right about adopting the es7 decorators from the babel package - i noted that on the new pr and will add it the yml as well.
I'm not sure about licensing and using bable's MIT code in this project - that's currently beyond me.
It took me few tries to make decorator highlighting work with typescript's yml - and i'm working with this for about 3 months now - without any side effects (before i opened this pr).


Switch from an absolute ::dioxus to the relative dioxus_core

This resolves an edge-case that may arise if you (for some weird reason) rename the dioxus dependency with the Cargo.toml package = ... key.
Consider a Cargo.toml with:
dioxus_but_named_wrong = { version = "0.5", features = ["web", "router"], package = "dioxus" }

And a main.rs with:
use dioxus_but_named_wrong::prelude::*;

If you then use the Routable derive macro, the compiler will error with:
error[E0433]: failed to resolve: could not find `dioxus` in the list of imported crates
 --> src/main.rs:6:17
  |
6 | #[derive(Clone, Routable, Debug, PartialEq)]
  |                 ^^^^^^^^ could not find `dioxus` in the list of imported crates
  |
  = note: this error originates in the derive macro `Routable` (in Nightly builds, run with -Z macro-backtrace for more info)

By switching this quote!'d return type from the root-level ::dioxus namespace to the relative dioxus_core, we allow the generated code to use whatever dioxus_core symbols are imported from the dioxus_you_can_name_this_however_you_want::prelude::*.

Thanks! We need to support other names for a dioxus-like package in case you are using dioxus-lib


Can't change location of postcss.config.js

Postcss-loader permits us to choose a custom directory for postcss.config.js
https://github.com/postcss/postcss-loader#config
But in css-loader-config.js checks only webpack context and breaks this feature.
const postcssConfig = findUp.sync('postcss.config.js', {     cwd: config.context   })
Postcss config may calls as .postcssrc.js as well.

Hi, thanks for creating an issue. We currently recommend using https://nextjs.org/docs/basic-features/built-in-css-support as the plugins have been deprecated in favor of the built-in support.


IHostApplicationLifetime bug

IHostApplicationLifetime ApplicationStopping is not waiting events
when registering IHostApplicationLifetime.ApplicationStopping event, event is called, but application stops before registered event is finished. According to documentation, application should wait for events (to be fully executed).
Steps to reproduce
1.) create new "asp.net core web application" using standard VS template.
2.) inject "IHostApplicationLifetime" into "configure" method and register "applicationStopping" event:
`
    public void Configure(IApplicationBuilder app, IWebHostEnvironment env, IHostApplicationLifetime hostApplicationLifetime )
    {
        app.UseRouting();
        app.UseAuthorization();
        app.UseEndpoints(endpoints =>
        {
            endpoints.MapControllers();
        });
        hostApplicationLifetime.ApplicationStopping.Register(() =>
        {
            var _i = 0; // place debug breakpoint here!
            _i++;
        });

    }

`
3.) start application in debug mode using "console" start. After starting, console window appears.
4.) set breakpoint inside "register" method. Then click "close"("x" on right upper corner of console application). You will notice that "applicationstopping" event is called, this is correct. But if you just wait on a breakpoint for 5-10 secons, you will notice that application has stopped without waiting for event to be executed.
`

The event is likely only triggered if you perform a graceful shutdown. @Pilchie

event is triggered, but it does not block application termination.

This issue was moved to aspnet/Extensions#2827


new form

Summary | Résumé

created the new form for building permit

Needs review from @nmakuch . tx


[ feature ] code completion: ide / text editor plugins

User Story

As an AI developer / AI-ops engineer
I want to code completion when working on resource (YAML) files
So I can accurately define specable attributes when configuring my agents

Acceptance Criteria

Given that I want to customize or need to configure an agent
When I have an Eidolon plugin for my IDE / text editor

IntelliJ
VSCode

Then I only see the allowed attributes and sub-attributes

In the following example, it's hard to understand why there's no implementation directly under apu. If there were apu > implementation, would llm_unit be indented? Unless you have someone (or some tool) helping you, it's difficult to apply custom logic.
apiVersion: server.eidolonai.com/v1alpha1
kind: Agent
metadata:
  name: your_agent_name
spec:
  apu:
    llm_unit: # from /docs/components/llmunit/overview
      implementation: OpenAIGPT
      model:
        name: gpt-4o-mini # from /docs/components/llmmodel/overview
        human_name: "GPT-4o Mini" # optional

Challenge: agent templates can expose arbitrary attributes.

https://claude.ai/chat/eec684b3-0ab6-4d8f-bc6f-de7d8164169c
^ this is the project we are looking for. By contributing here we are automatically supported

Also look into creating a complete Kubernetes CRD. Requires investigation how to be picked up automatically by k8 plugins

@LukeLalor the claude chat link has expired or otherwise not available. When you have a sec make a note of the project we'll be contributing to.

updated


connection https druid cluster

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Stack Trace
2018-02-22 20:54:10,051:DEBUG:urllib3.connectionpool:Starting new HTTP connection (1):
DNS.NAME.com
Error while processing cluster 'druid'
('Connection aborted.', BadStatusLine('\x15\x03\x03\x00\x02\x02P',))
2018-02-22 20:54:10,062:ERROR:root:('Connection aborted.', BadStatusLine('\x15\x03\x03\x00\x02\x02P',))
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/superset/cli.py", line 171, in refresh_druid
    merge_flag=merge)
  File "/usr/lib/python2.7/site-packages/superset/connectors/druid/models.py", line 123, in refresh_datasources
    self.druid_version = self.get_druid_version()
  File "/usr/lib/python2.7/site-packages/superset/connectors/druid/models.py", line 113, in get_druid_version
    return json.loads(requests.get(endpoint,verify=False).text)['version']
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 72, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 58, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 513, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 623, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/adapters.py", line 490, in send
    raise ConnectionError(err, request=request)
ConnectionError: ('Connection aborted.', BadStatusLine('\x15\x03\x03\x00\x02\x02P',))
Refreshed metadata from cluster [omni druid dev]

so I looked at the models.py and it seems that it only support http connection
def get_pydruid_client(self):
cli = PyDruid(
'http://{0}:{1}/'.format(self.broker_host, self.broker_port),
self.broker_endpoint)
return cli
how can I make druid connection to https can work ... ?
Superset version
superset version
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Superset 0.22.1
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Expected results
Connecting to Druid (allow https connection to druid)
Actual results
The Stacktrace is there
Steps to reproduce
I tried to use https instead of http , but it still has SSL HANDSHAKE error...     appreciated if you can help me out on this issue.. The druid we setup only support https ...

Python 2.7.5


Replaces non ascii characters to dashes so that it's consistent with …

Fix client/server theme name consistency

closes https://github.com/TryGhost/Ghost/issues/7313, refs https://github.com/TryGhost/Ghost/pull/7340
replace non ascii characters in theme filenames with dashes so that it's consistent with how the server encodes filenames

@disordinary: would you mind reformatting your commit message to match our guidelines? Something like...
Fix client/server theme name consistency

closes https://github.com/TryGhost/Ghost/issues/7313, refs https://github.com/TryGhost/Ghost/pull/7340
- replace non ascii characters in theme filenames with dashes so that it's consistent with how the server encodes filenames

Also, this needs a rebase now as there were some changes merged earlier that removed the mime-type checks. Thanks! 😄

git commit --amend is the magic ticket, just in case 😉

It should be good to go?

This PR still has 4 commits, normal workflow is to leave it with just 1. I think the squash and merge tool can be used to fix it up though?

👍 LGTM

Tested with https://github.com/TryGhost/Ghost/pull/7380/files.
Works.


add ScrollToTop component & fix TnC page

Description

Added a ScrollToTop component to the project to ensure the window scrolls to the top whenever a new page is navigated to.
Fixed minor bugs in TnC page.
Fixes and closes https://github.com/MAVRICK-1/Nest-Ondc/issues/98

How Has This Been Tested?

Tested by navigating between different routes in the application.
Tested on local development server.

Screenrecording :
https://github.com/MAVRICK-1/Nest-Ondc/assets/88451512/c1fede08-fe53-4ccb-b2a4-02772cc5a115
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I'm currently having my exams. I'll make the requested changes after my exams are over.

okay


Be able to select what branch we want to use for a project

Context: We already have a project editor, with a lane selection. We should think about how we want to do the branch selection. Each project is specific to a repo and a lane (you can only run one lane per project, to run more lanes, you create multiple projects, e.g. nightly, release, etc.)
Branch is not just selecting a branch, we should think about how we do it in a bigger way.
Some use-cases (we don't have to cover all of them right away)

Run builds only on master
Run builds only for PRs and master
Run builds for every commit (even no PRs)
Run builds for master, develop

We don't have to build the whole UI and everything to support all of this, however we should think about supporting those long-term, so ideally we come up with a way to store this setup in our persistence layer already.
I'd also be ok with just providing a list of branches and storing that, and ignoring the PR or no PR thing for now, as this could be a separate toggle.

We can figure out a vocabulary model to define the rules that apply to a project. Early idea: "@master&@PR&!develop" use & to concatenate rules, operator+branch to define which branch or other reserved words to build out not. I think that'd be easy to formalize and implement. What do you guys think? 😁

For now, let's make it super basic, and just allow a single branch? Are there any standards other CI systems have already?

So in Jenkins is the most rough way, where you use refs/<branch> or wildcards like /refs/*. Other CI I know are more like three main options: 1) build all new branches by default (you can later disable manually new branches created). 2) automatically build all PRs. 3) automatically build all tags.

I've been thinking about this and I actually don't know if we need to do anything here. If we do all configuration via fastfile, and in the fastfile we specify what branch we're using, do we really want to define what branch we're using in fastlane.ci too? That means we have to override the fastfile in the project, which doesn't seem like a good idea. Instead, maybe in the future you can modify your fastfile, or have alternate fastfiles.

If we do all configuration via fastfile, and in the fastfile we specify what branch we're using, do we really want to define what branch we're using in fastlane.ci too?

Yes, I think we still need to. One example is: I want to run fastlane release_to_app_store only on the release branch. fastlane.ci will be used for testing on probably all (*) branches, but quite often I think people will use it for e.g. weekly beta deployments or App Store deployments also.

Ah, ok, so maybe the implementation detail could be that we pass the branch information as a parameter to the lane? That way we keep all branch management for lanes in the Fastfile

I think that will cause issues long term. One thing that comes to mind is that the Fastfile might/will be different for each branch. From what I see, it has to be on a different layer (being the CI). Luckily there is not a lot of things where that's the case, but the Git URL, branch, lane to run are some of them

Good point ☝

At this point, we have the following:
Project (repo + lane)

Triggers
Each trigger can have a different branch

So when you select a branch, you'll want to update the appropriate trigger (or create a new one) with the branch selected 🎉

Correct me if i'm wrong here.
In summary, in order for users to be able to perform different tasks on different branches we'll allow them in the projects json to specify which branch and which lane to use. And should they not be provided we'll use master, and the first lane respectively?

A project is actually a repo + lane(a lane can be thought of as a task)
A trigger can act on a specific branch (master being default)
Lanes can range from "build and test" to "build, test, sign, distribute". So while a lane represents a task, a task can be a single action, or it can be multiple actions.
Since lanes can be super complicated, we wanted to have a single project represent just a single lane.

Is this still a WIP? Note that I just created a new card that's kind of related: https://github.com/fastlane/ci/issues/236

This should be done now
🚀


git clean with sudo where possible

Depending on whether the building user has sudo privileges, it's possible to write files in a build that Jenkins can't clean up, breaking builds for everyone else on that node. Stashbot sudo cleaning would avoid this.

@j-baker - thanks for the suggestion!  I'm not sure running sudo "willy nilly" is a good idea.  FIrst off, this will cause a ton of spam in audit logs if passwordless sudo isn't enabled for the user.  Also, people might like to know if any root files are being written, this just clobbers them all with no feedback if passwordless sudo is enabled.
A much better solution I'd recommend is having your build run a build script like this:
Cmd: ./bin/verify.sh
verify.sh:
#!/bin/bash
set -e
function cleanup {
  # Your cleanup code here
}
trap cleanup EXIT

And of course, your cleanup code can be sudo chown ... or whatever.  I would not be suprised if you are using docker?  docker is a very common usecase that sorta "craps root-owned files everywhere".  It is also possible to run as a user in your docker container whose ID is the same, and chown the files in there during your build.  Long story short there are lots of fixes but the one I've suggested here is most flexible because devs can edit the script and it doesn't force other users of stashbot to run sudo, or add yet another configuration option to the config screen.
Does that make sense?  Is there a usecase my suggestion doesn't cover?
Thanks!

(also, feel free to pop into #stashbot on irc.freenode.org if you want to chat about this further, I am cmyers there)

Yes, we were using docker. Our failure mode was that we left a few docker-created files there, then all of the build-nodes became corrupted because the clean command had insufficient privileges to remove the files, and all of our builds failed. We then had to have someone with access clear off the boxes we'd soiled, which took considerable time as we are UK based.
We have a script like that now, but created this PR to preemptively help others from having this problem - because it's not at all clear that you should need such a cleanup script.

Yeah, if your build writes root-owned files, and fails to clean them up, I don't feel that should be stashbot's concern (and is in fact important to be a separated concern).  Even if we added this to stashbot, it wouldn't have helped you unless the user running your builds had sudo access, and if it did, you could have checked in a script that ran "sudo " and got yourself unblocked without delay that way as well...right?

I generally agree with Carl that it seems a bit odd to put 'sudo' invocations into stashbot.  However, users can't actually get themselves unblocked because stashbot always runs 'git reset --hard && git clean -fdx' before any user-defined script, so it'll fail and quit before users have any chance to do anything.
We could switch this around slightly and have the command be
git reset --hard && { git clean -fdx || sudo git clean -fdx; }
For cases where people don't have root-owned files spewed over their tree, that would basically behave as before (because the reset --hard and clean -fdx would succeed and the '||' would short-circuit).  Only when the normal git clean -fdx fails would the attempt to sudo be invoked.
But that might still be slightly odd.  In the docker case where the user has sudo rights, it cleans up for us nicely.  But when the problem is that someone did a 'chmod -R a-w somedir' (and we have seen stuff like that) but doesn't have sudo rights, then it's just weird and doesn't help solve the problem.
And on that note...
Do we also need to do a 'chmod -R a+w .' here?  And 'find . -type f | xargs chattr -i'?  And maybe a git rebase --abort? (and merge --abort and cherry-pick --abort and am abort and bisect reset)?  And ...
The number of ways people can screw up their builds when we reuse them seems pretty large...

Yeah, you're definitely right, at the core this issue is caused by reusing build slaves. However, the whole reason for the git clean in the first place is presumably because we use such build slaves, and because it's an easy way to handle 95% of the cases we care about.
Your modification (no sudo unless required) would increase the currently failing cases with minimal additional complexity. It seems that these Docker cases are generally problematic - the fact there are instructions on how to set uid bits and recommendations to specifically git clean yourself before stashbot does is evidence of that.

Hmm, that is an exceedingly good point, @newren - I'm going to think about this a bit over the next day or two.  At a minimum I would want this to be a not-enabled-by-default setting at the server level maybe, to accept it into my fork, I think.  But I'll think on it more before I make a "final ruling" =)

@terabyte: An alternative way that might be cleaner is adding a check for the existence of some global system-defined pre-job or post-job script, and if it exists, run it.  That way palantir could define a global pre-job script of '/usr/local/itools/bin/palantir-cleanup.sh' and we could do the ugly, gross, site-specific stuff that tends to hit many of our internal projects (e.g. cleaning out all tags and refetching them, or sudo chown -R jenkins:jenkins ., or maybe running a setuid program that calls docker-cleanup or some chattr nonsense or whatever).  It'd make it really easy for the site-admins to add new cleanup stuff this way without resaving all existing jobs (just edit the palantir-cleanup.sh script), and also has the benefit that company-specific stuff doesn't clutter the main stashbot.  The only trick would be allowing the script to be defined in the global stashbot configuration page, and having that script be sucked into all the jenkins templates.  (With a big warning in the UI that if you change the script name, you have to re-save all jobs.)
Or something along those lines...

@newren I would greatly prefer that approach.  That seems like a much cleaner and more flexible way of solving this and other problems.  I'd make a text field in the jenkins config settings that is a path to a global cleanup script which is run, if it exists, in each job.  Shouldn't be too difficult a change to make.


Add Allstart github app

We should add some safeguards and security checks in the repo to show good practices are in place
What do you think about adding https://github.com/ossf/allstar ? @azu

I think that Allstar is for a orgnization, but it is useful for checking.
Also, We can move this repo to a org like security-alert/security-alert.
Maybe, it is better that move this repo to org if we will use allstar app.
Because, GitHub App installation requires owner permission of the orgnization or the repository.
It means I am the only one who can install GitHub Apps.(It is limitation of personal repository)
If we have moved to org, orgnization owners(I and you) can install GitHub Apps.
It will avoid SPOF(Single Point of Failure).

OK. I will create security-alert and transfer this to the org.


Please support for .obj .dae models with cannon.js

For developing any serious game with three.js or cannon.js we are going to be required to load models.
From my research so far it seems that with cannon.js we can only use three.js json models.
Converting a simple 1mb .blend file results in a 29mb .json model.
This file size is impractical for any online game.
I have opened a issue for three.js
Blender to json export file size #6455
https://github.com/mrdoob/three.js/issues/6455
Not sure what the response will be but if the cannon folks can support the .obj or .dae that would help taking three.js and cannon.js forward.

SceneVR uses .obj models with cannon.js. You can convert the mesh data (that is loaded using any loader) into a cannon.js 3d model.

Based on my experience, I don't think either format is worth the effort.

@anothercoder2 @bnolan @ioquatix  How do you convert the mesh data to a cannon js model. There is nothing in there docs that have a custom type http://schteppe.github.io/cannon.js/docs/  What custom types or classes are needed to make this happen? Thanks


Update mac osx doc icon with actual time

It would be awesome if the app also updated the dock icon with actual time left so I wouldn't have to click the icon to know how much time has passed.

What if you have multiple timers on? Which one would show up?

i didn't even know you could have multiple timers! when there are multiple timers, you could show an option on each timer that would designate it as the one which would show up in the doc icon.

Okay lovely I'll see if I can implement this? You may have to wait a little while as I'm backed up with exams and don't have a lot of time.

@akarki15
My latest commit now adds a badge to the dock tile:
https://github.com/DummybugStudios/timer-app/commit/5cde4397817d9eb1d945a0dad1368189a44b5a1a
However, if you have multiple timers which one it will choose is still a little random so I won't be opening a pull request until I have sorted that out.
In the meantime - feel free to build that branch and enjoy the badge :)

@akarki15 I have opened a pull request for the badge.
You have to right click teh window and choose "show in dock" to show the timer in the dock.


[python] SparseNDArray bounding box inconsistent -- shape vs. used coordinates

The current python "soma bounding box" for sparse nd array is inconsistent in its semantics in the write path:

for Tensor objects, it utilizes the shape of the tensor to compute the new bounding box.  From the code:

    nr, nc = values.shape
    bounding_box = self._compute_bounding_box_metadata([nr - 1, nc - 1])

for Table objects, it scans for the max coordinate, and writes the 'used' bounding box.  From the code:

    maxes.append(pacomp.max(coords).as_py())
    bounding_box = self._compute_bounding_box_metadata(maxes)

This will have different results when the tensor is sparse (empty) in the max coordinates.  Code that will otherwise work fine breaks the bounding box -- for example, these two writes produce entirely different results:
m = sparse.coo_matrix(([1,2], ([0,1],[0,1])), shape=(10,10))

# writes bounding box of (0,10),(0,10)
A.write(pa.SparseCOOTensor.from_scipy(m))

# writes bounding box of (0,1),(0,1)
A.write(pa.Table.from_pydict({'soma_data': m.data, 'soma_dim_0': m.row, 'soma_dim_1': m.col})

It is not entirely clear which is correct, and depends on downstream uses.  For example if the expectation is that a sparse matrix, with the last few rows set to zero, can be saved and reconstituted with the same shape, this difference will be an issue.
If the semantics are "used coordinates", as implemented in the Table code branch, we would be better off using the underlying nonempty_domain implementation (resolving bug #1969).  If the semantics intend shape, then the Table code path needs some means of determining the caller's intended shape.
CC: @mojaveazure @johnkerl

@bkmartinjr thank for filing this.
The semantics of bounding-box we hammered out in several spec meetings and a design  doc was definitely mean for intended shape. There were three:

non-empty domain which is the actually used -- e.g. 100x200 but rows 0-3 and 98-99 are completely unoccupied;
bounding box would track the numbers 100 and 200 regardless of whether first/last rows/columns of a sparse array were entirely unoccupied or not;
"shape" as available domain with room for growth later -- maybe billions in each dimension

So if we want to keep bounding box as intended shape we need to provide a way to get those numbers like 100 and 200 actually wired all the way from the user data to the on-disk bounding-box metadata.
And if we want to jettison the bounding box entirely then we just need to use non-empty domain for the performance issue raised in #1933.
Personally I can go either way. But the latter wins in a tie-breaker, for two reasons:

Parsimony -- all else being equal, I'd rather have two shapish accessors than three
Correctness -- the bounding box has correctness issues with simultaneous open-open-write-write-close-close as tracked in #1969

Pending #2407


About Top 1 and Top 5? What are they?

Code version (Git Hash) and PyTorch version
Dataset used
Expected behavior
Actual behavior
Steps to reproduce the behavior
Other comments
In accuracy, there are Top 1 and Top 5. So, I would like to know what is the difference between them and what do they represent? Although I read about your paper, I am not clear about these these values. Could you explain that? And after training, how to test custom video using trained model weights one by one?

While training a single video clip may fall under more than one class but would have annotated only for one class.
So we try to get the top 5 predictions while testing.


Implement armor stand entity and ArmorEquipable

Only a few methods tested but the methods are rather repetitive in their function so should work for all.
There's some issues with methods calling themselves due to same signature as shadowed method, so comment those methods out in development environment until @Mumfrey has a way to fix this.

so comment those methods out in development environment until @Mumfrey has a way to fix this.

There already is a way to fix this, it's explained at length in the mixin examples.

so comment those methods out in development environment until @Mumfrey has a way to fix this.

There already is a way to fix this, it's explained at length in the mixin examples.

Ahh yes, forgot about @Overrite, changed it

Ahh yes, forgot about @Overrite, changed it

I have updated this PR to make use of a mixin's ability to target multiple classes. MixinArmorEquipable attaches onto all entity classes that have an ArmorEquipable interface. This is a useful technique to avoid copying the same implementation from mixin to mixin.

Merged thanks!


Added www direct to nginx block

I assume most people want www.mysite.com and site.com to go to the same place, so I've added extra code/ explanation to the nginx block to allow them to do that for static sites.

I mean to add www, but this is actually a suboptimal way to do it (even though I think I did it in a video).
This will cause a little lower SEO because the site is mirrored at a different URL. It's better to have www 301 redirect to the bare domain or vice versa.


Cannot Make an account for weChat

I cannot make an account for weChat there is some problem with login.

I have asked @Manveer695 also and he got same problem. If anyone from China or any person who will be able to make acount can make account for us so that I can work on WeChat bot then it will be helpful.

Hello back! I'm currently in China, but I could not make the account either. It requires a Chinese ID document, and I moved to the United States three years ago, so I no longer have a Chinese ID card. We should try to sign up a organisation account, should be fairly simple task.


added user tests

Added tests for the Profile model, login function, edit profile
Fixes: #155

Hey @Akshi149 please attach a screenshot showing the test cases run

here it is


Update Deprecated Libraries

npm WARN deprecated<EMAIL_ADDRESS>babili has been renamed to babel-minify. Please update to babel-minify
npm WARN deprecated<EMAIL_ADDRESS>babili has been renamed to babel-minify. Please update to babel-preset-minify
npm WARN deprecated<EMAIL_ADDRESS>Please update to the latest object-keys

Fixed with b2a490cf5d19b71e595fcf0061281192ccc16ced


Uncaught Error: Setting active pane that is not present in pane container

[Enter steps to reproduce:]

Open Atom
Open a recently opened project folder
Open/reopen tree pane a few times
Click/right-click tree pane a few times

Atom: 1.18.0-beta2 x64
Electron: 1.3.15
OS: Mac OS X 10.12.5
Thrown From: Atom Core
Stack Trace
Uncaught Error: Setting active pane that is not present in pane container
At /Applications/Atom Beta.app/Contents/Resources/app/src/pane-container.js:253

Error: Setting active pane that is not present in pane container
    at PaneContainer.didActivatePane (/Applications/Atom Beta.app/Contents/Resources/app/src/pane-container.js:253:21)
    at Pane.module.exports.Pane.activate (/Applications/Atom Beta.app/Contents/Resources/app/src/pane.js:875:20)
    at Pane.module.exports.Pane.focus (/Applications/Atom Beta.app/Contents/Resources/app/src/pane.js:269:25)
    at atom-pane.<anonymous> (/Applications/Atom Beta.app/Contents/Resources/app/src/pane-element.js:51:25)

Commands
     -0:36.7.0 window:toggle-full-screen (input.hidden-input)
     -0:32.2.0 tree-view:toggle (atom-workspace.workspace.scrollbars-visible-when-scrolling.theme-atom-material-syntax.theme-one-dark-ui)
     -0:21.8.0 application:reopen-project (div.tool-panel.tree-view)
  4x -0:17.1.0 tree-view:toggle (atom-workspace.workspace.scrollbars-visible-when-scrolling.theme-atom-material-syntax.theme-one-dark-ui)

Non-Core Packages
atom-beautify 0.29.26
atom-clock 0.1.8
atom-material-syntax 1.0.6
atom-material-ui 2.0.4
atom-terminal-tab 0.2.0
build 0.68.0
build-make 0.13.0
clojure-indent 0.2.0
django-atom 0.2.4
file-type-icons 1.3.4
fizzy 0.21.0
git-time-machine 1.5.9
hyperclick 0.0.40
hyperlink-hyperclick 1.3.4
improved-chester-atom-syntax 0.3.0
ink 0.6.5
language-crystal-actual 0.35.0
linter 2.1.4
linter-ruby 1.2.5
linter-ui-default 1.6.0
one-tab 0.10.0
proto-repl 1.4.20
release-notes 0.53.0
scroll-through-time 0.2.0
symbol-gen 1.3.1
tab-title 0.3.3
tree-ignore 0.6.2
typewriter 0.7.0
unfancy-file-icons 0.11.0
wakatime 7.0.3

Thanks for the report!  Can you confirm if the error is consistently reproducible and if so, is it reproducible in safe mode (atom --safe)?

Preamble: I'm new to development and whatnot. Trying to help and get help but let me know if I'm just adding noise or how to improve 🙏 😄
In case it helps, I re-encountered this error where the tree sidebar is missing only for this one particular project and even running in atom --safe and when I clicked project in the bottom left corner, I got a magical link to this issue.
Oddly, all other projects seem to open fine with the tree sidebar intact. My atom settings
Screenshot of error and link that brought me here below

Alright so as an update, I got this sorted out on my end back to normal by doing the following

Close all open Atom projects
Run atom --clear-window-state saw on this reference
Run atom --safe on the repo that I had issues with (file tree now shows up)
Close that repo project in Atom
Run atom . on the same repo (file tree still shows up)
🎉

i meet it so

Thanks for the info @epan :v:  Seems like some of the saved state maybe related to that one project was causing the trouble, good to know that clearing the saved state fixed things up for you.
If you still get the same error in safe mode @ron-wolf, clearing saved state is a good next step to try (make sure to save any unsaved work first).

I run into the same issue and @epan's solution works, thanks.

Having this same issue.  Closed the editor in the project I was having an issue.  Reopened atom with --clear-window-state.  Project fixed.  Opened another project that was working fine.  It had a couple tabs with unsaved content.  All that project's tabs are gone too, including the unsaved ones.  Posting because it isn't clear from the above solution that running --clear-window-state will destroy all content in all unsaved tabs in all projects with no warning.  Think I just lost a bunch of project-related phone call notes in other projects.
Posted a question about how to fix it or at least prevent it here.  If anyone knows plugins to get back untitled file contents after --clear-window-state, or a plugin to dump them to a file to prevent future loss, I'd greatly appreciate the help.
The launch stories underlying the issues are listed here

Sorry for the data loss @a-laughlin, but --clear-window-state permanently destroys any saved state across all projects.
I tried reproducing this issue but was unable to.

sorry about that @a-laughlin and I've edited my original suggestion based on your experience

[Enter steps to reproduce:]

open atom
open a folder
open tree view
click around empty tree view

Atom: 1.17.2 x64
Electron: 1.3.15
OS: Mac OS X 10.12.5
Thrown From: Atom Core
Stack Trace
Uncaught Error: Setting active pane that is not present in pane container
At /Applications/Atom.app/Contents/Resources/app/src/pane-container.js:253

Error: Setting active pane that is not present in pane container
    at PaneContainer.didActivatePane (/Applications/Atom.app/Contents/Resources/app/src/pane-container.js:253:1)
    at Pane.module.exports.Pane.activate (/Applications/Atom.app/Contents/Resources/app/src/pane.js:875:1)
    at Pane.module.exports.Pane.focus (/Applications/Atom.app/Contents/Resources/app/src/pane.js:269:1)
    at atom-pane.<anonymous> (/Applications/Atom.app/Contents/Resources/app/src/pane-element.js:51:25)

Commands
     -0:19.8.0 tree-view:toggle (input.hidden-input)

Non-Core Packages
atom-beautify 0.29.26
atom-clock 0.1.9
atom-django 0.3.2
atom-material-ui 2.0.4
atom-ternjs 0.18.3
atom-typescript 11.0.3
auto-update-packages 1.0.1
autocomplete-paths 1.0.5
autocomplete-python 1.8.64
busy-signal 1.4.3
color-picker 2.2.5
django-templates 0.5.0
emmet 2.4.3
ex-mode 0.15.0
file-icons 2.1.7
git-control 0.9.0
git-plus 7.9.3
git-time-machine 1.5.9
highlight-line 0.12.0
highlight-selected 0.13.1
intentions 1.1.2
language-babel 2.65.1
linter 2.1.4
linter-eslint 8.2.1
linter-gcc 0.7.1
linter-javac 1.9.4
linter-pycodestyle 2.0.2
linter-tslint 1.6.0
linter-ui-default 1.6.1
merge-conflicts 1.4.5
minimap 4.28.2
minimap-highlight-selected 4.6.1
minimap-linter 2.0.0
monokai 0.24.0
multi-cursor 2.1.5
nucleus-dark-ui 0.12.3
pigments 0.39.1
platformio-ide-terminal 2.5.1
project-manager 3.3.5
Remote-FTP 0.10.7
seti-syntax 1.1.3
sync-settings 0.8.1
todo-show 1.11.0
tool-bar 1.1.0
tool-bar-almighty 0.8.2
vim-mode-plus 0.93.0

epan's quick instructions fixed the bug, thanks!

@AHL96 - you mentioned you can't find a pattern for when it happens so just to clarify, is the error not reproducible for you?  Or do the steps you shared always recreate the error?

@rsese reproduct steps like this:
open atom
open a project folder and open tree view
remove the  project folder A
open project folder B
remove the  project folder B
at this time the tree-view panel is closed
open project folder A by menu command
at this time the project folder A is not present in tree-view,but the window title is normal
left/right click around  tree-view times
my config like this:
"*":
"atom-autocomplete-php":
binComposer: "C:\ProgramData\ComposerSetup\bin\composer.phar"
"atom-beautify":
general:
_analyticsUserId: "88fadf73-5736-4566-9b11-f75af24ee819"
php:
cs_fixer_path: "C:\Users\netmou\AppData\Roaming\Composer\vendor\fabpot\php-cs-fixer\php-cs-fixer"
"atom-terminal": {}
"autocomplete-dcd": {}
"autocomplete-php": {}
"autocomplete-plus":
autoActivationDelay: 300
strictMatching: true
"busy-signal":
itemsToShowInHistory: 1
"clipboard-plus":
maximumTextLength: 32
"color-picker":
preferredFormat: "HEX"
core:
autoHideMenuBar: true
closeDeletedFileTabs: true
disabledPackages: [
"language-clojure"
"language-coffee-script"
"language-csharp"
"language-gfm"
"language-go"
"language-mustache"
"language-objective-c"
"language-perl"
"language-property-list"
"language-python"
"language-ruby"
"language-ruby-on-rails"
"language-toml"
"open-on-github"
"wrap-guide"
"timecop"
"spell-check"
"php-twig"
"language-java"
"language-make"
]
openEmptyEditorOnStart: false
projectHome: "e:\"
telemetryConsent: "no"
themes: [
"one-light-ui"
"one-light-syntax"
]
editor:
backUpBeforeSaving: true
fontFamily: "Source Code Pro"
fontSize: 18
invisibles: {}
scrollPastEnd: true
showCursorOnSelection: false
showIndentGuide: true
showInvisibles: true
softTabs: false
softWrap: true
useShadowDOM: false
zoomFontWhenCtrlScrolling: false
emmet: {}
"exception-reporting":
userId: "94525e00-8cce-ee60-d3fc-3fdcd2da473d"
"file-icons": {}
"git-diff": {}
"git-plus":
commits: {}
experimental:
stageFilesBeta: false
general:
_analyticsUserId: "9ce739be-4a08-43fd-b4f9-cce53e5b6447"
"highlight-selected": {}
linter:
lintOnChangeInterval: 800
"linter-htmlhint": {}
"linter-jshint": {}
"linter-php": {}
"linter-ui-default":
panelHeight: 64
useBusySignal: false
minimap:
displayPluginsControls: false
plugins:
"git-diff": true
"git-diffDecorationsZIndex": 0
"highlight-selected": true
"highlight-selectedDecorationsZIndex": 0
selection: true
selectionDecorationsZIndex: 0
scrollAnimation: true
"one-dark-ui": {}
"one-light-ui": {}
"rest-client":
tab_key_behavior:
soft_tab_length: 4
"simple-drag-drop-text":
copyKey: "ctrl"
"spell-check": {}
"split-diff":
diffWords: true
ignoreWhitespace: true
leftEditorColor: "red"
rightEditorColor: "green"
scrollSyncType: "Vertical + Horizontal"
syncHorizontalScroll: true
"status-bar":
cursorPositionFormat: "%L 行：%C 列"
selectionCountFormat: "(选中：%L 行，%C 字符)"
"symbol-gen": {}
"symbols-view": {}
tabs:
enableVcsColoring: true
usePreviewTabs: true
"terminal-plus":
core: {}
"tree-view":
hideIgnoredNames: true
welcome:
showOnStartup: false
whitespace:
ensureSingleTrailingNewline: false
ignoreWhitespaceOnCurrentLine: false
keepMarkdownLineBreakWhitespace: false
removeTrailingWhitespace: false
".babel.regexp.source":
editor:
tabLength: 4
".basic.html.text":
editor:
tabLength: 4
".d.source":
editor:
tabLength: 4
".diff":
editor:
tabLength: 4
".diff.source":
editor:
tabLength: 4
".html.php.text":
editor:
autoIndent: true
".js.jsx.source":
editor:
tabLength: 4
".source.sql":
editor:
tabLength: 4

sorry this is the right step
1.open a project folder A and open tree view
2.remove the project folder A
at this time the tree-view panel is closed
3.open project folder B by menu command
4.remove the project folder B
5.reopen project folder A by menu command
at this time the project folder A is not present in tree-view,but the window title is normal
6.left/right click around tree-view times

Sorry for the confusion. When the error occurs, those steps are usually what i did. But idk if those steps are what is causing it to happen because it doesn't happen every time i do those steps,  So it seems

Using Project Manager package and Shift+Enter-ing into a package (open in same window) also makes this bug happen. Fixed after a restart of Atom.

Same problem, same cause here -- Clearing the window state didn't work for me, but what finally did was removing the contents of the /storage directory in my user's Atom folder

Thanks for confirming @AHL96 - if you do come across specific steps to reproduce let us know.
@netmou - thanks for sharing specific steps.  Can you confirm if you can reproduce in safe mode (atom --safe) and can you also share the specific commands for your steps?  If you can share the exact menu item names that you're using, that would be helpful when trying to reproduce.

I had a project open from the other day I was working in, and then wanted to reference another project from the past so I opened a new window and got this error - thought nothing of it. Restarted my computer and tried booting up my working project and I am getting the same error. I tried clearing window state in the past and it worked, but nothing this time.

Fresh install, (deleted ~.atom folder) on OSX, working in a few files, did a global search and replace in a project folder (for '#!' ajax url functions if that matters), and closed atom with the search box still open.  Upon re-opening, tree-view is broken.  I can toggle it but not put the same project folder in it, it remains blank.
I can put a different folder in it and have it show up.

Thanks @RNCTX - to confirm, what version of Atom are you running and where does the error show up in your steps?  Also, is the error reproducible in safe mode (atom --safe)?

@rsese 1.18.0 on OSX Sierra.
The error is strange, after I posted that I was able to get the original folder to re-appear in the file tree by deleting it from the 'recents' history.
My steps were...

fresh install without packages/configs
toggle file tree
open project folder from local drive
search/replace from the root of that project folder
close atom

Upon re-opening, I could open a different project folder but opening the same one resulted in a blank file tree and the error that traces back to this bug report.
After clearing the offending project folder from the 'recents' I was able to open it again and everything seems normal.
So I would suspect this is somehow related to the recent projects list and/or the global project search/replace functionality.

The same for me, but tracing refers line 257 instead of 253 on Win8.1 x64:
trying figure the steps out
Atom: 1.19.1 x64
Electron: 1.6.9
OS: Unknown Windows version
Thrown From: Atom Core
Stack Trace
Uncaught Error: Setting active pane that is not present in pane container
At C:\Users\User1\AppData\Local\atom\resources\app\src\pane-container.js:257

Error: Setting active pane that is not present in pane container
    at PaneContainer.didActivatePane (~/AppData/Local/atom/resources/app/src/pane-container.js:257:21)
    at Pane.module.exports.Pane.activate (~/AppData/Local/atom/resources/app/src/pane.js:911:20)
    at Pane.module.exports.Pane.focus (~/AppData/Local/atom/resources/app/src/pane.js:290:25)
    at HTMLElement.<anonymous> (~/AppData/Local/atom/resources/app/src/pane-element.js:51:25)

Commands
     -0:53 core:cancel (atom-notification.fatal.icon.icon-bug.native-key-bindings.has-detail.has-close.has-stack)
  2x -0:38.9.0 tree-view:toggle (input.hidden-input)

I can repro this issue by doing the following:
Repro steps:

Open Atom with a project.
Rlight Click Close pane in the tree-view,  Notice nothing happens
right click and remove the project.
Tree-view disappears,  Toggle it back open
right click and click add folder back
Tree view remains empty and if you open dev tools you see error from https://github.com/atom/atom/issues/14454#issuecomment-322984675
close tree-view and open it back up and left click on the tree view.

Thanks @ungb and thanks again everyone for the reports - added your steps to the issue body Bryant :bow:

Another way to reproduce it:

Open Atom (no project or folder select, just open it)
File | Add Project Folder | Select your project
Notice that no directory structure is shown on the left hand "Project" side bar/menu
Click on the left hand bar/menu/pane (not sure how it's called)
Hiding/showing the left hand tree view pane doesn't show anything/refresh content

Atom: 1.19.3
Windows 7 (sadly)

How I came to this error.
Restart Atom after package update.
Project pane is empty.
Attempt to load a project, pane is still empty.
Click in Project pane and error appears.

Yes, I encounter the issue under the same circumstances as gbittman.

This occurred to me when I opened two atom windows, clicked a file in tree view on one window, and dragged it's tab on to the second window.

This occurred to me when I opened two atom windows, clicked a file in tree view on one window, and dragged it's tab on to the second window.

Same to me.
Atom version: 1.21.1

I got the same error on Windows 7 in version 1.21.1 x64. I managed to get the project back into view by starting Atom in Safe mode in the project folder (from cmd, run atom --safe), then closed it, then opened it normally. The error did not appear the second time and I could not make it occur again.
Steps taken when the error occurred: Opened Atom normally, from the Start menu. The window did not have the Project tree view. Split the main editor pane left. Opened a file in the left side of the pane. Closed the right half. On closing the right half of the pane, the error appeared and I could not make the project appear in the tree view upon loading it,

Atom: 1.22.1 ia32
Electron: 1.6.15
OS: Microsoft Windows 10 Pro
Thrown From: Atom Core
Stack Trace
Uncaught Error: Setting active pane that is not present in pane container
At %AppData%\Local\atom\app-1.22.1\resources\app\src\pane-container.js:257

Error: Setting active pane that is not present in pane container
    at PaneContainer.didActivatePane (~/AppData/Local/atom/app-1.22.1/resources/app/src/pane-container.js:257:21)
    at Pane.activate (~/AppData/Local/atom/app-1.22.1/resources/app/src/pane.js:1002:46)
    at Pane.focus (~/AppData/Local/atom/app-1.22.1/resources/app/src/pane.js:393:23)
    at /app.asar/src/pane-element.js:51:25)

Commands
     -4:25.6.0 application:add-project-folder (ol.tree-view-root.full-menu.list-tree.has-collapsable-children.focusable-panel)
  2x -2:00.6.0 tree-view:toggle (atom-pane.pane.active)

Non-Core Packages
acid-syntax 1.0.0
activate-power-mode 2.6.0
acute-syntax 1.0.0
aesthetic-ui 4.0.0
afterglow-monokai-syntax 1.3.7
align-regexp 0.4.0
ample-syntax 0.3.0
atom-beautify 0.30.6
atom-material-syntax 1.0.7
atom-material-ui 2.0.6
atom-monokai 0.10.9
atom-numbers 0.7.0
atom-terminal 0.8.0
autoclose-html 0.23.0
ax-monokai-syntax 0.2.0
bleen-syntax 0.3.1
browser-plus 0.0.98
cake-syntax 1.0.2
classy-syntax 1.1.1
cmyk-syntax 0.2.0
color-picker 2.2.5
dark-bint-syntax 0.10.0
darya-syntax 0.2.0
dawn-syntax 0.1.0
doc-green-syntax 1.1.0
edm-light-syntax 0.1.2
emmet 2.4.3
firewatch-syntax 1.1.1
genesis-ui 0.5.0
git-control 0.9.0
gruvbox-plus-syntax 1.3.1
html-entities 0.4.0
html-to-css 0.2.2
hugs-syntax 1.0.0
idle-syntax 0.1.0
juicy-syntax 1.0.5
katana-syntax 0.1.0
ker0x-syntax 0.1.3
language-angelscript 0.1.1
language-batch 0.4.0
language-htaccess 0.1.1
language-markdown 0.25.1
language-vb 0.1.0-pre1
language-vbscript 0.9.0
leet-syntax 1.0.0
line-ending-converter 1.3.2
lines 0.13.1
Lux-Syntax 2.0.0
markdown-preview-plus 2.4.16
minimap-git-diff 4.3.1
monokai 0.24.0
monokai-faded 0.0.4
monokai-light 0.4.1
monokai-plus 0.1.3
monokai-seti 0.9.0
monokai-syntax 0.6.0
Neka-Syntax 1.0.1
nerv-ous-syntax 1.0.0
now-syntax 0.2.4
one-sharp-syntax 3.0.0
open-terminal-here 2.3.1
php-cs-fixer 4.1.1
pigments 0.40.2
plain-syntax 0.3.0
polymer-snippets 3.2.1
power-effects 0.4.0
prism-syntax 1.4.1
rad-syntax 0.4.0
revert-buffer 0.6.0
sass-autocompile 0.13.3
save-session 0.15.7
sequential-number 0.5.0
simple-align 0.2.0
simple-drag-drop-text 0.3.6
sub-monokai 1.4.0
text-manipulation 0.6.0
trailing-spaces 0.4.0
valign 1.1.3
vivid-syntax 2.1.0
warm-syntax 1.0.0

I was having the same problem as @epan.
The steps in his comment fixed that issue!
Thank you!

Just ran into this issue as well
http://www.flowpress.com/screenshots/screenshot_2017-12-18_at_11.08.21AM.png


DAL: pq: syntax error at or near "MODIFY"

Conclusion
This can also happen if you click on resolve after adding one/multiple fields
Reproduction steps

Resolved.
@katrinDY do note that the way you've configured it doesn't really make sense so it'll probably cause issues down the line.
You've used values for the generic values JSON (the first field uses it), then you've made the second field use that values column as an integer -- that won't fly


Implement "Need Support?" page

Context:
https://docs.google.com/document/d/1BbOXsuikpHlaI9ChjrUlltbpPAmgnOafnWEeqEC8B74/edit#heading=h.efs7ggobb664
With that context,
0: Implement the support page and have it render at https://explorer.hiro.so/support
1: Swap the current "Community Support" link with "Need Support?" (as the label) and the link pointing to https://explorer.hiro.so/support
2: Remove "Submit bug or feature request"; it is already included in the support page. That is sufficient.

Design mockup pending

Related https://github.com/hirosystems/explorer/issues/1480

Figma link: https://www.figma.com/file/31q1g27k69z1yXb6r6vrRD/Stacks-Explorer-Working-File?type=design&node-id=7492%3A66570&mode=dev&t=eHaS1nGj2AtHTgRN-1
Mock up:

This one is done :)


[Bug][runner] DirectRun failed to generate encKey automatically

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
According to @keon94 , when he tried to launch Github with DirectRun, it failed due to encKey being empty.
What you expected to happen
encKey should be generated automatically when it was empty in DirectRun mode.
How to reproduce
Not sure.
Anything else
No response
Version
main
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@keon94
I try to generate encKey automatically on DirectRun,but there is a contradiction:
As follows

We can see that connectionId was required if you want DirectRun on github plugin.
So it means you have to set at least one connection before do DirectRun.
If you had already create one conntion,that must have a enckey.And if you lost the enckey,It is useless to generate encKey automatically.
So you have to post an connection at first before you try DirectRun.
I created an issue  #2362 For if you want create connection by DirectRun on feature.


Pytest fails

Note: If this is a general question or difficulty in installing or using the software, please ask in the Discussions section.
Describe the bug
Please give a short and clear description of what the bug is and what you expected to happen. Please provide instructions and data to reproduce the problem (model, jcl, etc.). Please provide the smallest working example that reproduces the problem.
The Gui test show the following error message
app = QApplication([])
RuntimeError: Please destroy the QApplication singleton before creating a new QApplication instance.

Closed by #45


Fix unpkg url

The example of using library via unpkg.com pointed to payment.js instead of stpayment.js. Trying to load payment.js returns this error instead of the script:
Cannot find module "/dist/payment.js" in package<EMAIL_ADDRESS>

Hi! Thank you so much for get this one :)


Hey, I just wanted to know if you tried to change the paw holding the mouse. I don't want to make any preasure, I  just want to know if you could help me to change it. I actually just want like a stickman hand you know? And if you could solve this problem many people would be very grateful.

I would like to know aswell!!


Flickr plugin only retrieves MAX 30 photos.

Hi,
The Galleria Flickr plugin only retrieves max 30 photos.
Is there a way to adjust this - I have set the max to 100.
http://www.barterphotography.net
Thanks for your time.
Cheers,
Stephen McCormack
MWeb Solutions

Seems to be working OK

Is working fine - did not know there were more settings you could add. Would be great for other users to add samples of how this is done on the website. Thanks for a great product David. Cheers. Steve.


Optimize Set#&, Set#clone and Set.new

UPDATE:
I also init the size on a Union now (as the size has to be at least the Max size of the two sets, this has similar gains as changes to clone and new)
Changes to clone and new involve setting the initial capacity. Always in the case of clone, and an overload for Indexable(T) in the case of new for access to an efficient size.  (I also expect this to generate less garbage, so the real world gain from should be even better)
0.25.0
new 100 items 243.51k (  4.11µs) (± 7.58%)
new 1 000 000 items  22.69  ( 44.08ms) (± 3.39%)
clone 100 items 239.94k (  4.17µs) (± 5.79%)
clone 1 000 000 items  21.42  ( 46.69ms) (± 3.56%)

PR
new 100 items 282.04k (  3.55µs) (± 9.10%)
new 1 000 000 items  25.95  ( 38.54ms) (± 5.07%)
clone 100 items  280.8k (  3.56µs) (± 5.76%)
clone 1 000 000 items  24.84  ( 40.26ms) (± 4.88%)

For & it's the same optimization as what intersects? does: iterating over the smaller of the two:
0.25.0
sparse & small  17.54k ( 57.01µs) (± 2.78%)
dense with small  26.63k ( 37.56µs) (± 2.99%)
small with sparse    9.4M (106.34ns) (± 3.39%)
small with dense   9.34M (107.01ns) (± 2.34%)

PR
sparse & small    7.8M ( 128.2ns) (± 2.59%)
dense with small   8.03M (124.61ns) (± 2.46%)
small with sparse   7.85M (127.32ns) (± 2.88%)
small with dense   8.07M (123.98ns) (± 2.38%)

small is 1 item, and both sparse and dense are in the 10K range.

Am i reading this correct for &?
Sparse & small 17.54k vs 7.8M is 444 times faster 😮 ?

@sdogruyol
small = Set.new([1])
large = Set.new(Array.new(1_000_000) {|i| i})

In the the existing code, if you did small & large you'd only iterate through smalls single item and check its existence in large. However, if you did large & small you'd iterate all million items and check all million of them. But in the end, both give the same result.
With the PR, it will always let the small one "lead". The cost is figuring out which is smaller (self.size < other.size)...and while intersecting 1 to 1_000_000 is extreme,  the gain should be fairly linear relative to the size difference.

@karlseguin thanks for the detailed explanation 🙏

Excellent work @karlseguin, let's wait for dev team to review! :smile_cat:

Thank you again! 💚


Adapt cert tests for CNTI certification V2.0

Ref: #1931
Ref: #1932
Description
(name of issue/change)
Issues:
Refs: #NNN
How has this been tested:

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Verified all A/C passes

[ ] develop
[ ] master
[ ] tag/other branch

[ ] Test environment

[ ] Shared Packet K8s cluster
[ ] New Packet K8s cluster
[ ] Kind cluster

[ ] Have not tested

Types of changes:

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

Checklist:
Documentation

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] No updates required.

Code Review

[ ] Does the test handle fatal exceptions, ie. rescue block

Issue

[ ] Tasks in issue are checked off

PR looks good. Triggered a build here - https://github.com/cnti-testcatalog/testsuite/actions/runs/8524764982

Updated, tested by executing cert tests.
Ready for review and potential merge, github actions can be run.

I triggered a build with the new changes from this PR.
The cnf-config tag specs are failing.
https://github.com/cnti-testcatalog/testsuite/actions/runs/8596519498

New build running here - https://github.com/cnti-testcatalog/testsuite/actions/runs/8615039901

Looks like I wasn't pushing the latest changes for previous build.
I made a new branch and pushed the changes again. The build is running here - https://github.com/cnti-testcatalog/testsuite/actions/runs/8616979925


UI - UI allows for LB services to be created with no listening ports.

Server version - v0.31.0-rc4
Add a Load Balancer service with no "LISTENING PORTS".In this case we have 1 blank entry for port in the UI.
Adding LB succeeds. LB agent gets created with no ports.  Since we do not allow for editing the listening ports , this LB service needs to be deleted and recreated .
UI does restrict LB services creation with no listening ports , when there is no  entry under "LISTENING PORTS"  (when the blank entry gets removed). Following message is presented to the user - "One or more listening ports are required".
The same behavior is expected when there is 1 blank  entry for  "LISTENING PORTS" ,which is the default view for adding LB rule.

Tested with server version - v0.33.0-rc3
User is now provided the following message - "Choose one or more ports to listen on" , when there is 1 blank entry for port in the UI for "LISTENING PORTS and user chooses to create LB.


Carbon/DateTime is not being converted when I save it in an array

Hi,
So my problem is that a Carbon date is not automatically converted to a MongoDate (like it says in the docs) if I store that date inside an array.
My code is this:
$product->stock = [
    'amount'       => $amount,
    'lastUpdate' => Carbon::now(),
];

$product->save();

and I get this on my document for the lastUpdate field:
lastUpdate:Object
date:"2017-07-27 09:42:00.000000"
timezone_type:3
timezone:"UTC"

It works just fine if I do this:
'lastUpdate' => new \MongoDB\BSON\UTCDateTime(new \DateTime('now'))
becomes:
lastUpdate:2017-07-27 11:49:47.000
The documentation says that my code should work:

Eloquent allows you to work with Carbon/DateTime objects instead of MongoDate objects. Internally, these dates will be converted to MongoDate objects when saved to the database. If you wish to use this functionality on non-default date fields you will need to manually specify them as described here: http://laravel.com/docs/eloquent#date-mutators

and it actually does... but not inside an array.
By the way, I also defined this:
    protected $dates = [
        'stock.lastUpdate',
    ];

So is this a bug? Or I am doing something wrong?

How to define $dates attribute for array of documents. For example I want following structure.
versions => [
  [
    updated_at => 'some date',
    created_at => 'some_date'
  ],
  [
    updated_at => 'some date',
    created_at => 'some_date'
  ],
]

Is there any way to achieve this without explicitly converting date like below
versions => [
  [
    updated_at => new \MongoDB\BSON\UTCDateTime(new \DateTime('now')),
    created_at => new \MongoDB\BSON\UTCDateTime(new \DateTime('now'))'
  ],
  [
    updated_at => new \MongoDB\BSON\UTCDateTime(new \DateTime('now'))',
    created_at => new \MongoDB\BSON\UTCDateTime(new \DateTime('now'))
  ],
]

I seem solved this problem.It's very useful to add a data attribute.
<?php

namespace Modules\Project\Entities;

use Jenssegers\Mongodb\Eloquent\Model;
use Jenssegers\Mongodb\Eloquent\SoftDeletes;

//use Illuminate\Database\Eloquent\Model;
//use Illuminate\Database\Eloquent\SoftDeletes;

class Project extends Model
{
    protected $connection = 'mongodb';

    use SoftDeletes;

    protected $dates = ['start_date_at', 'online_date_at'];

}

@dark-wind Can I cast DateTime format in nested attributes. I tried one example below, but it seems not working
protected $dates = ['start_date_at', 'online_date_at', 'some_array.start_date',  'some_array.end_date'];

How to define $dates attribute for array of documents. For example I want following structure.
versions => [
  [
    updated_at => 'some date',
    created_at => 'some_date'
  ],
  [
    updated_at => 'some date',
    created_at => 'some_date'
  ],
]

Is there any way to achieve this without explicitly converting date like below
versions => [
  [
    updated_at => new \MongoDB\BSON\UTCDateTime(new \DateTime('now')),
    created_at => new \MongoDB\BSON\UTCDateTime(new \DateTime('now'))'
  ],
  [
    updated_at => new \MongoDB\BSON\UTCDateTime(new \DateTime('now'))',
    created_at => new \MongoDB\BSON\UTCDateTime(new \DateTime('now'))
  ],
]

Hey, did you get the solution for such attributes? I am also facing the same
"history" : [
{
"action" : "initiate",
"offer_price" : 450000.0,
"action_by" : "buyer",
"action_date" : ISODate("2021-03-12T06:20:51.000Z")
},
{
"action" : "counter",
"offer_price" : 470000.0,
"action_by" : "seller",
"action_date" : ISODate("2021-03-12T06:22:18.000Z")
},
How can I define action_date?

+1 - no matter what I do, only created_at and inserted_at are saved as strings.

I solved my problem creating a new model for the child object...
class ArticleReport extends Model
{
    protected $collection = 'articleReports';
    public function reports(){
        return $this->embedsMany(FlagReport::class);
    }
}

class FlagReport extends Model {
    protected $primaryKey = null; //Remove id
    protected $dates = [
        "datetime"
    ];
}

Then, I can use it like this:
$report = new ArticleReport();

$flagReport = new FlagReport();
$flagReport->userId = $userId;
$flagReport->flag = $flag;
$flagReport->datetime = Carbon::now();

$report->reports()->associate($flagReport);

$report->save();

As mentioned, $casts is the solution.


Force close when running FastCell Examples

Hello george and thanks for the good job.
When i run FastCell Example i got a force close in android.
Specificlly when running complex fastcell i got a force close:
06-09 13:20:32.796 I/MonoDroid( 1451): UNHANDLED EXCEPTION:
06-09 13:20:32.796 I/MonoDroid( 1451): System.NullReferenceException: Object reference not set to an instance of an object
06-09 13:20:32.796 I/MonoDroid( 1451): at Xamarin.Forms.Platform.Android.Platform.GetRenderer (Xamarin.Forms.BindableObject) <IL 0x00006, 0x0002d>
06-09 13:20:32.796 I/MonoDroid( 1451): at Xamarin.Forms.Platform.Android.ListViewAdapter.GetView (int,Android.Views.View,Android.Views.ViewGroup) <IL 0x00249, 0x00dfb>
06-09 13:20:32.796 I/MonoDroid( 1451): at Android.Widget.BaseAdapter.n_GetView_ILandroid_view_View_Landroid_view_ViewGroup_ (intptr,intptr,int,intptr,intptr) [0x0001a] in /Users/builder/data/lanes/monodroid-mavericks-monodroid-5.1-series/d419c934/source/monodroid/src/Mono.Android/platforms/android-21/src/generated/Android.Widget.BaseAdapter.cs:509
06-09 13:20:32.796 I/MonoDroid( 1451): at (wrapper dynamic-method) object.62d82a1e-955d-4c59-b503-2617c412b9b0 (intptr,intptr,int,intptr,intptr) <IL 0x00023, 0x00047>
06-09 13:20:32.804 I/dalvikvm( 1451): Could not find method java.lang.Throwable., referenced from method md52ce486a14f4bcd95899665e9d932190b.JavaProxyThrowable.
06-09 13:20:32.804 W/dalvikvm( 1451): VFY: unable to resolve direct method 9507: Ljava/lang/Throwable;. (Ljava/lang/String;Ljava/lang/Throwable;ZZ)V
06-09 13:20:32.804 D/dalvikvm( 1451): VFY: replacing opcode 0x70 at 0x0000
06-09 13:20:32.804 D/AndroidRuntime( 1451): Shutting down VM
06-09 13:20:32.804 W/dalvikvm( 1451): threadid=1: thread exiting with uncaught exception (group=0xa4c0a648)
An unhandled exception occured.
06-09 13:20:37.292 D/dalvikvm( 1451): GC_CONCURRENT freed 453K, 6% free 10498K/11084K, paused 68ms+0ms, total 72ms
06-09 13:20:37.292 D/dalvikvm( 1451): WAIT_FOR_CONCURRENT_GC blocked 1ms
06-09 13:20:37.292 I/dalvikvm-heap( 1451): Grow heap (frag case) to 10.473MB for 143392-byte allocation
06-09 13:20:37.292 E/AndroidRuntime( 1451): FATAL EXCEPTION: main
06-09 13:20:37.292 E/AndroidRuntime( 1451): java.lang.RuntimeException: java.lang.reflect.InvocationTargetException
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:553)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at dalvik.system.NativeStart.main(Native Method)
06-09 13:20:37.292 E/AndroidRuntime( 1451): Caused by: java.lang.reflect.InvocationTargetException
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at java.lang.reflect.Method.invokeNative(Native Method)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at java.lang.reflect.Method.invoke(Method.java:525)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:737)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	... 2 more
06-09 13:20:37.292 E/AndroidRuntime( 1451): Caused by: md52ce486a14f4bcd95899665e9d932190b.JavaProxyThrowable: System.NullReferenceException: Object reference not set to an instance of an object
06-09 13:20:37.292 E/AndroidRuntime( 1451): at Xamarin.Forms.Platform.Android.Platform.GetRenderer (Xamarin.Forms.BindableObject) <IL 0x00006, 0x0002d>
06-09 13:20:37.292 E/AndroidRuntime( 1451): at Xamarin.Forms.Platform.Android.ListViewAdapter.GetView (int,Android.Views.View,Android.Views.ViewGroup) <IL 0x00249, 0x00dfb>
06-09 13:20:37.292 E/AndroidRuntime( 1451): at Android.Widget.BaseAdapter.n_GetView_ILandroid_view_View_Landroid_view_ViewGroup_ (intptr,intptr,int,intptr,intptr) [0x0001a] in /Users/builder/data/lanes/monodroid-mavericks-monodroid-5.1-series/d419c934/source/monodroid/src/Mono.Android/platforms/android-21/src/generated/Android.Widget.BaseAdapter.cs:509
06-09 13:20:37.292 E/AndroidRuntime( 1451): at (wrapper dynamic-method) object.62d82a1e-955d-4c59-b503-2617c412b9b0 (intptr,intptr,int,intptr,intptr) <IL 0x00023, 0x00047>
06-09 13:20:37.292 E/AndroidRuntime( 1451):
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at md5530bd51e982e6e7b340b73e88efe666e.ListViewAdapter.n_getView(Native Method)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at md5530bd51e982e6e7b340b73e88efe666e.ListViewAdapter.getView(ListViewAdapter.java:40)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.widget.HeaderViewListAdapter.getView(HeaderViewListAdapter.java:220)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.widget.AbsListView.obtainView(AbsListView.java:2161)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.widget.ListView.makeAndAddView(ListView.java:1840)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.widget.ListView.fillDown(ListView.java:675)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.widget.ListView.fillGap(ListView.java:639)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.widget.AbsListView.trackMotionScroll(AbsListView.java:4970)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.widget.AbsListView$FlingRunnable.run(AbsListView.java:4127)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.view.Choreographer$CallbackRecord.run(Choreographer.java:749)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.view.Choreographer.doCallbacks(Choreographer.java:562)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.view.Choreographer.doFrame(Choreographer.java:531)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:735)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.os.Handler.handleCallback(Handler.java:730)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.os.Handler.dispatchMessage(Handler.java:92)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.os.Looper.loop(Looper.java:137)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	at android.app.ActivityThread.main(ActivityThread.java:5103)
06-09 13:20:37.292 E/AndroidRuntime( 1451): 	... 5 more
06-09 13:20:37.320 D/dalvikvm( 1451): GC_FOR_ALLOC freed 14K, 6% free 10623K/11228K, paused 24ms, total 24ms
06-09 13:20:37.332 D/skia    ( 1451): --- SkImageDecoder::Factory returned null
06-09 13:20:37.356 D/skia    ( 1451): --- SkImageDecoder::Factory returned null
06-09 13:20:37.380 D/dalvikvm( 1451): GC_CONCURRENT freed 535K, 6% free 10593K/11268K, paused 2ms+1ms, total 8ms
06-09 13:20:37.464 D/dalvikvm( 1451): GC_CONCURRENT freed 439K, 5% free 10737K/11288K, paused 2ms+0ms, total 9ms
06-09 13:20:37.468 D/dalvikvm( 1451): WAIT_FOR_CONCURRENT_GC blocked 6ms
06-09 13:20:37.476 D/dalvikvm( 1451): WAIT_FOR_CONCURRENT_GC blocked 11ms
06-09 13:20:37.484 D/dalvikvm( 1451): GC_FOR_ALLOC freed 261K, 6% free 10844K/11428K, paused 5ms, total 5ms
Thread finished:  #9
Thread finished:  #9
The thread 'Unknown' (0x9) has exited with code 0 (0x0).

additional notes: i tested it on a moto X, and in simulator (nexus 4) android 4.4

you need to update xamarin forms to 1.4.3
I should add this to the notes..
1.4.3 completely hoses breakpoint debugging. but on android, there's a crash in cell re-use which compeltely breaks the fastcell, unless you have 1.4.3 running.
please try : upgrading all the projects to 1.4.3 and deleting the 1.4.2 forms folders from your packages folder. You should find that works just fine.
Please let Xamarin know that you need the breakpoint debugger fixed in 1.4.3 - https://bugzilla.xamarin.com/show_bug.cgi?id=29861

i will test it. and when i have a response i will do it. thanks george.

George, i've update to forms 1.4.3pre2 and the fastcell is working as expected.
regarding the breakpoints, i've placed a breakpoint in every method on FastCellComplexSamplePage and all the breakpoints worked fine on XAM studio. didn't tested it on VS. did you have problems with breakpoints?
note: i only tested the app in IOS

Yes, whenever I have breakpoints set they don’t stop/step correctly.
I’ll try it again when I get a chance; but as you can see from the issue, I’m not the only one who get’s undefined breakpoint behaviour.
Perhaps the latest XS fixes it. If it works fine here, I’ll update the project to use 1.4.3
Please be warned that this is WIP and I’ve not done a great deal of work on memory management yet. Be sure to clear out the caches, as per the examples.
Let me know if issues/any fixes you have. Xamarin are dragging their heals on improving virtualiation performance, so the FastCell is quite important for now : I want to make it as good as I can.

On Jun 10, 2015, at 7:52 AM, zeferino<EMAIL_ADDRESS>wrote:
George, i've update to forms 1.4.3pre2 and the fastcell is working as expected.
regarding the breakpoints, i've placed a breakpoint in every method on FastCellComplexSamplePage and all the breakpoints worked fine on XAM studio. didn't tested it on VS. did you have problems with breakpoints?
note: i only tested the app in IOS
—
Reply to this email directly or view it on GitHub https://github.com/twintechs/TwinTechsFormsLib/issues/4#issuecomment-110739402.

here the breakpoints are working. im using this:
=== Xamarin Studio ===
Version 5.9.3 (build 1)
Installation UUID
Runtime:
Microsoft .NET 4.0.30319.0
GTK+ 2.24.22 (MS-Windows theme)
GTK# 2.12.26
=== Xamarin.Android ===
Version: 5.1.3 (Business Edition)
Android SDK:
Supported Android versions:
2.3   (API level 10)
4.0.3 (API level 15)
4.4   (API level 19)
5.0   (API level 21)
Java SDK: C:\Program Files (x86)\Java\jdk1.7.0_71
java version "1.7.0_71"
Java(TM) SE Runtime Environment (build 1.7.0_71-b14)
Java HotSpot(TM) Client VM (build 24.71-b01, mixed mode, sharing)
=== Xamarin Android Player ===
Version: <IP_ADDRESS>
Location: C:\Program Files\Xamarin Android Player\DeviceManager.exe
=== Build Information ===
Release ID: 509030001
Git revision: 5a524e1726ed103fdd4fe37e0356f2b35466ce9d
Build date: 2015-06-02 16:19:11-04
Xamarin addins: 51957cfbd06be911b212671ad05c2c6221ac90f9
=== Operating System ===
Windows 6.2.9200.0 (64-bit)
i now will use fastcell in a projet and i will tell you soon how it is working there.
thanks again, and hope xam devs can fix the pre2 issues.

zeferino, fyi - I've confirmed that my issues go away with 1.4.3-pre2 on XS 1.9, at least in this sample project, so I've updated the project to use that.

thumbs up!


Set-up issue with Paperproof and Mathematics in Lean book

Hiya,
I'm following the instructions provided by both Mathematics in Lean and Paperproof and it seems to be completely broken at the moment. After adding require Paperproof from git "https://github.com/Paper-Proof/paperproof.git"@"main"/"lean"  to the lakefile.lean file, and import Paperproof to one of the .lean files, and then restarting the file as indicated by VS Code, I get the following error:
[1980/1980] Building Paperproof
Some build steps logged failures:
- Building CheckIfUserIsStillTyping
error: build failed

I have tried re-installing everything and have even wiped my entire set of elan toolchains but nothing seems to help. Can anyone here shed some light on this?

Hi Rick!
Looks like the new lean version breaks something in our CheckIfUserIsStillTyping.lean.
Can you take a look at what your Lean version is?

Hi Evgenia! I'm running the very latest release candidate,  v4.8.0-rc1, so that would explain the issue. Is there a workaround for this or should I just downgrade Lean for the time being?
Just as a quick aside, it might be worth updating the list of supported versions since Paperproof's Reservoir page lists this version as fully supported

Is there a workaround for this or should I just downgrade Lean for the time being?

No easy workaround until we rewrite that portion of the code to be compatible with the new Lean, I'll try to look at it this weekend.
Right now you can downgrade to Lean (version 4.0.0-nightly-2023-06-20, commit a44dd71ad62a, Release) - this is my version of Lean, and it was certainly working with Paperproof last time I checked.

Made Paperproof work with leanprover/lean4:v4.8.0-rc1 (fixed in this commit).
Just running lake update Paperproof should update Paperproof to the version that works with v4.8.0.

Latest Paperproof works with the following Lean versions:

4.0.0-nightly-2023-06-20
v4.8.0-rc1

And I assume with everything in between (though I couldn't check that, switching Lean versions takes ~20 minutes even when everything goes smoothly).

Thanks for the report @RickDW, please feel free to reopen if you have some further issues.

Thanks for the quick update!
The error I saw has disappeared after I updated Paperproof, but it seems like something is still broken. All I can see in the Paperproof window inside VS Code is a completely white background. I'm not sure if this is related to the previous build error?

Huh could reproduce it!
I didn't have this error just two days ago, but I did get it today.
This needs further exploration, but what I believe happened is vscode-lean4 extension got updated in the past few days, bringing about the "asUri is not a function" error. Probably something related to this PR https://github.com/leanprover/vscode-lean4/pull/437. (To be clear - this is not a bug in vscode-lean4, it's a bug in how Paperproof interacts with vscode-lean4).
This can be fixed in the following way:

Follow these instructions to rollback lean4 extension to version v0.0.140.
Fully close vscode, open vscode.
Try opening Paperproof again.

Made Paperproof work with the latest lean extension (vscode-lean4 v0.0.140).
Updating to vscode extension Paperproof v1.3.0 should fix the blank screen.
Again - thanks for the report!


[ssr] Design doc for SSR test utility

Tracking issue for writing a design doc for https://github.com/lit/lit/issues/2351

Design doc has been written and reviewed with core team members.


Move import validation to business error checker

Please make sure the below checklist is followed for Pull Requests.

[ ] Checks are green
[ ] Tests are added where necessary
[ ] Documentation is added/updated where necessary
[ ] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

Excellent


Failed to write to pager: exec: "less": executable file not found in %PATH%

Whenever I invoke cheat, I get the following error: failed to write to pager: exec: "less": executable file not found in %PATH%.
For example:
$cheat -c -s ssh
failed to write to pager: exec: "less": executable file not found in %PATH%
$cheat -l
failed to write to pager: exec: "less": executable file not found in %PATH%
$cheat ssh
failed to write to pager: exec: "less": executable file not found in %PATH%

My environment:
echo $PSVersionTable

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

Installed via scoop.

The default config from cheat --init does contain the pager key. I suggest removing the default pager configuration, using the PAGER env variable, or adjusting the error message to indicate that pager is configured explicitly in the conf.yml file.

Appears to duplicate #571

@ondrasek, thanks for reporting this. I'm going to close this and track this issue along with other Windows-only issues in #665, but I can confirm this behavior.
I believe I have a fix for this working locally. It should ship with the next version of cheat, hopefully later today.


hasUpMigration method name ambiguous

hasUpMigration seems a bit ambiguous because it's not clear whether that's referring to the up migration having already ran or if the up migration exists for the name in the migrations directory.  What about a method name like "isUpMigrationApplied" or "hasUpMigrationBeenApplied" ?
If a change is desired I can do it as part of my pull request with a hasUpMigration method in place for backwards compatibility.

I tend to prefer "hasUpMigrationBeenApplied".