Clarify licenses of external dependencies

The README currently only mentions (very briefly) that the library is licensed under the MIT license. However, the repository also contains external code (primarily in the extern folder, I believe?) which is covered by separate licenses. This needs to be clarified. I would suggest a License header in the README clarifying what code is covered by the license, and what code is covered by separate licenses, as is customary in other projects. For example, perhaps something along the lines of:
# License

External dependencies are covered by separate licensing terms.
See the `extern` folder for the code and respective licensing terms of each dependency.

The SPlisHSPlasH library code is licensed under the MIT license. See `LICENSE` for details.

Good point. I added this info.


feat: Rebrand pandevice to pan-os-python

Description
Full rebrand of this library...
Package friendly name:  Device Framework -> PAN-OS SDK for Python
Package pythonic name: pandevice -> pan-os-python
Import statements: import pandevice -> import panos
Motivation and Context
The name pandevice is confusing and awful. It originally came from the first class in the library which was class PanDevice(object): but since the library has grown by orders of magnitude since then, this name is no longer descriptive or applicable.
At the same time as this rebrand, we are releasing this package as a stable 1.0.0 release and dropping python2 support.. New name, new support model, new release schedules.
How Has This Been Tested?
All tests run and passed.
Docs build successfully.
No changes to the library except for the naming.
Types of changes

Breaking change (all imports must change from pandevice to panos)
Documentation (changes to all names across all documentation)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

Overriding review requirement. This has been extensively examined and tested.


pulse effect animation

Pulse effect animation

Close #4701

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have created a helpful and easy to understand README.md

@MohitGupta121 @TusharKesarwani Kindly review this PR.

@MohitGupta121 @TusharKesarwani Kindly assign labels to this PR.


Fix npm lint script

Adjust the npm lint script to disable globbing (set -f) as this should be handled by eslint. I also adjusted it to only lint src/js by default.

⛵ 💯 ✨

mesosphere-ci Retest this please


Feat: Improve the Navbar DropDown in the Mobile View

Describe the feature
on exploring this website I saw it in the mobile view then I noticed that the The Toggle button (light/Dark) is not visible perfectly so I would like to fix it . So can u please assign this to me under @gssoc'24
Add ScreenShots

Record

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

this issue already taken and assigned #250 , first always confirm the issue in issue list already mentioned or not.


Azure Servicebus gave System.ServiceModel.AddressAlreadyInUseException

Hi Support,
My azure service bus giving me below exception when listener trying to connect it:
Service cannot be started. System.ServiceModel.AddressAlreadyInUseException: The specified address already exists.Address sb://ServiceBusName.servicebus.windows.net/ is already in use by an existing listener with different settings. TrackingId:2fa03e51-934c-4d40-b8ce-e65f1d0d3883_G17, SystemTracker:NoSystemTracker, Timestamp:7/3/2018 11:37:46 AM
Error occurred: The specified address already exists.Address sb://ServiceBusName.servicebus.windows.net/ is already in use by an existing listener with different settings. TrackingId:2fa03e51-934c-4d40-b8ce-e65f1d0d3883_G17, SystemTracker:NoSystemTracker, Timestamp:7/3/2018 11:37:46 AM
Service bus can connect with multiple listener with different settings.
We used NetTcpRelayBinding in the listener to connect with service bus.
Exception comes when "ServiceHost.Open(new TimeSpan(0, 0, 30)); method is called.
Even I can't see how many listener are connected with service bus.
I googled it and found a image where they specify the listener count, but same is not appeared on my azure machine. Can you help me to find out this, how I can locate the no. of listener?
https://i.stack.imgur.com/nBIr4.png
Looking forward a quick reply.

@SdnaMunish this repository is for documentation related feedback and issues.  I would suggest you ask your question in this forum:
https://social.msdn.microsoft.com/Forums/azure/en-US/home?forum=servbus
If you have a support plan, you can open a case directly with support as outlined here:
https://docs.microsoft.com/en-us/azure/azure-supportability/how-to-create-azure-support-request


Add Kepler maps to the Maps section

Related issue
https://github.com/CartoDB/cartodb/issues/15487

I fully agree with both observations. I do not know if in some leapfrog I can take the time to make a  refactor and fix this. 👍


Is it possible to change the color on a white background?

I tried something like
odoo.default({
  el:".spin",value:"£400" , "color" : "black", "background-color" : "green"
})

But it always shows up as white on white ...

It's a bug. Thanks for reporting this. Color should be handled by fill attribute in css (as it is a svg).

Hi, now you can change the color with css. As it is a svg element you need to use fill instead of color. The cool thing is that you could use some additional attributes like stroke.
.js-odoo {
  fill: black;
  background-color: green;
}


Migrate article: Automated Debezium Kafka PostgreSQL test environment deployment

Migrate the content from this article to fit the documentation portal format https://help.aiven.io/en/articles/5278616
(Assign this issue to yourself before you start, and remember to mention the issue number when you open your pull request)

@lornajane Where would this article best fit at? It's an integration with 3 services:

Kafka
Kafka Connect
PostgreSQL

It also requires Terraform as well and since it's an automated deployment, I am inclined to place it as a separate page under Aiven Terraform Provider HowTo section.
Let me know if that sounds good or if it's better placed somewhere else and I can go with that.

@bcrisologo Yes please, the Terraform howto looks like a good place for it! Please verify the steps are still valid by testing the process when migrating.

Thanks for this @lornajane! I'll test out the article, but will probably follow the instructions on the official github repo, update that repo to be up-to-date, then migrate the article accordingly with the new changes that makes it work.  I noticed some of the files are outdated at least on the tf side.

I haven't been able to successfully complete the automation testing as there were things failing in the script last I tried this out.  I think we should just close this for now as the repo linked in my previous comment hasn't been updated in a while.


Change state from script trigger?

How can I call/change a state from a SCRIPT trigger?

Hey, sorry for the two weeks delay.
Normally, you trigger a change of state by returning true from the is_triggered function. You can't directly set the next state you want. Script triggers are used to determine whether or not a transition occurs.
If you need more help, please post your code and fsm graph.


Generated DDL against SQL Server adds column width to certain datatypes though not applicable

System information:

Microsoft Windows 10
DBeaver Version :- 7.1
Additional extensions

Connection specification:

Database Version :- Microsoft SQL Server - v2017
Driver name :- Microsoft JDBC v 4
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
When Dbeaver generates a DDL against a table it will sometime add column widths.
In some cases column widths can not be explicitly stated for certain datatypes.
Datatypes that do not support column widths include

picture
timestamp

Steps to reproduce, if exist:
DDL - Original ( Hand Entered )
use [tempdb]
go
create schema [datatype] authorization [dbo]
go
create table [datatype].[personImage]
(
[id]                int not null  identity(1,1)
, [tag]              binary(20) not null
, [fingerPrint]  varbinary(4000) null
, [signature]    binary null
, [picture]        image
, [ts]                timestamp not null
)
go
DDL Generated
CREATE TABLE tempdb.datatype.personImage (
id int IDENTITY(1,1) NOT NULL,
tag binary(20) NOT NULL,
fingerPrint varbinary(4000) NULL,
signature binary(1) NULL,
picture image(16) NULL,
ts timestamp(8) NOT NULL
) GO;
Include any warning/errors/backtraces from the logs
Error 1
Msg 102, Level 15, State 1, Line 8
Incorrect syntax near 'GO'.
Error 2
Msg 2716, Level 16, State 1, Line 7
Column, parameter, or variable #5: Cannot specify a column width on data type image.
Error 3
Msg 2716, Level 16, State 1, Line 7
Column, parameter, or variable #6: Cannot specify a column width on data type timestamp.

Thank you for the bug report.

Fixed

still reproducible for image type

Umm, it is image, not picture.
Fixed

verified


[Issue-23171] SSL_add_dir_cert_subjects_to_stack(): Return values undocumented

…
In the man page for SSL_add_dir_cert_subjects_to_stack(), the functions returning int have undocumented return values.
https://www.openssl.org/docs/man3.2/man3/SSL_add_dir_cert_subjects_to_stack.html
#23171

Checklist

[x] documentation is added or updated
[ ] tests are added or updated

@t8m - Build fuzzers is failing from long time. Request your guidance for the same.

@t8m - Build fuzzers is failing from long time. Request your guidance for the same.

Please just ignore it for now.

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This pull request is ready to merge

Squashed and merged to the master, 3.2, 3.1 and 3.0 branches. Thank you for your contribution.


I can't access my project

Describe the bug
I need to be able to access my project.
If I can't access this I will be pressing a lawsuit against you guys.
Link to the Bolt URL that caused the error
https://bolt.new/~/bolt-nextjs-shadcn-7dgegv
Steps to reproduce
It isn't loading from my history
Expected behavior
It isn't loading from my history
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 128
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 372235681,
  "usedJSHeapSize": 353541493,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = programmer22
Chat ID = c76bdf9ede3c

Threatening with a lawsuit makes me laugh so much

@gvasilakiss not sure what's so hilarious, I also lost a project today.
Luckily it was only one day of work.
@programmer22 if it's any consolation, I cancelled my subscription when it happened to me!

Appreciate the support @davidmoshal  I did some background checking on @gvasilakiss and he goes to a college rated a 2.1 out of 5 stars, calls himself a computer expert which is language only a rookie uses, and dosen't have any good  experience in programming. I laughed when I read his profile

I lost about 5 different projects from yesterday too so I know what it is like but I don't go posting online for lawsuits lol.
Also its called University not college and I don't call myself Blockchain Maestro at least:)

Love that I ruffled your feathers ;)

@gvasilakiss Last one because i've been laughing so hard reading your profile and watching this video https://www.youtube.com/watch?v=Bs9hC-olVlE&t=47s

Enjoy it, at least I'm someone here and achieved a lot:)

You have achieved nothing lol just from reading your profile, watching your youtube video, and doing more research, I know you don't do much you don't have to buff up your resume for your own self conscious to make yourself feel better :D
I also have more contributions than you on github their just private because why would I want to share any of my work with someone like you?
  Student complaints: useless. Course: waste of money.
But then again, this reflects the state of the UK's education system as a whole.
Hahaha you lost this convo @gvasilakiss but if you want to keep exposing yourself as a fool go right ahead, i love being a troll on here :)

This reflects the kind of developer you will be @gvasilakiss:
Rated 1 out of 5 stars
Jun 7, 2024
Just when you think that this foul and fetid institution can’t possibly sink any lower - they always sink lower than rock bottom.
Absolutely despicable practices going on here.
AVOID AT ALL COSTS.

I won, admit your loss move on (pat on the back)

FYI I've blocked OP from all @stackblitz repos, this guy is toxic af and has no place here

Thanks @EricSimons congrats on the webinar today too!


The picker appears twice in 1.0.7 on Firefox

Firefox v45.0.2 on Windows 7.1

See #70 - I believe this is a duplicate

Released v1.1.0


Write unit tests according to the test plan

MUST test:

[x] add.js
[ ] chunk.js
[x] difference.js
[x] divide.js
[x] filter.js
[x] isDate.js
[ ] isEmpty.js
[ ] reduce.js

SHOULD test:

[ ] ceil.js
[x] countBy.js
[x] defaultTo.js
[ ] map.js

COULD test:

[ ] at.js
[ ] capitalize.js
[ ] castArray.js
[ ] endsWith.js
[ ] get.js
[ ] keys.js
[ ] toFinite.js
[ ] toNumber.js
[ ] toString.js
[ ] upperFirst.js
[ ] words.js

Comments:

add.js:
difference.js: fails in even the most basic use cases
divide.js: fails in even the most basic use cases
filter.js: works partially:

predicate function must take care not to cause Exceptions e.g. by referencing nonexistent attributes, otherwise the filter leaks the occurred Exception to caller.
Inserts an extra empty Array to the result Array, if given an empty parameter.

isDate.js ok. Notice, that it returns true for all Date-objects, even if the date itself is insane.


Autosave error when file not named

If autosave is turned on with let g:tmux_navigator_save_on_switch = 1 and a vim instance in opened, ie running just vim and not vim <path-to-file> then navigating away from the tmux pane containing this unnamed (for lack of a better word) vim instance results in the following error:
Error detected while processing function <SNR58_TmuxWinCmd..<SNR>58_TmuxAwareNavigate:
line 11:
E32: No file name

Here's a screenshot of the error.
EDIT: fix formatting

Thanks for the report.
That might be easy to fix by just catching this Vim error there, i.e. nothing will be saved, but you won't see an error.

That'd be awesome :)
Hope it's easy to repro this issue.

..and depending on the code afterwards, this should be skipped maybe.

All I needed was a hint ;)
I've literally never written vimscript. Hopefully I'll get a PR out for this later today/early tomorrow.


Global onBeforeAction  hooks not working

Hey, I have a problem using the global onBeforeAction hooks.
The global hook Router.onBeforeAction(filters.isLoggedIn, {only: ['home']}); doesn't work but the one defined in the route does.
this.route('home', {
        path: '/',
        yieldTemplates: {
            'home': {to: 'content'}
        },
        onBeforeAction: function (pause) {
            if (!Meteor.user()) {
                this.render('login', {to: 'content'})
                pause();
            }
        }
    });

Thanks in advance!

@JohnAllen I have the same problem. Did you solve yours? I realize that the waiton() function interferes with the onBeforeAction.


npm fails to install dependand packages for coder

npm ERR! Error: failed to fetch from registry: express/3.1.0
npm ERR!     at /usr/share/npm/lib/utils/npm-registry-client/get.js:139:12
npm ERR!     at cb (/usr/share/npm/lib/utils/npm-registry-client/request.js:31:9)
npm ERR!     at Request._callback (/usr/share/npm/lib/utils/npm-registry-client/request.js:136:18)
npm ERR!     at Request.callback (/usr/lib/nodejs/request/main.js:119:22)
npm ERR!     at Request. (/usr/lib/nodejs/request/main.js:212:58)
npm ERR!     at Request.emit (events.js:88:20)
npm ERR!     at ClientRequest. (/usr/lib/nodejs/request/main.js:412:12)
npm ERR!     at ClientRequest.emit (events.js:67:17)
npm ERR!     at HTTPParser.onIncoming (http.js:1261:11)
npm ERR!     at HTTPParser.onHeadersComplete (http.js:102:31)
npm ERR! You may report this log at:
npm ERR!     http://bugs.debian.org/npm
npm ERR! or use
npm ERR!     reportbug --attach /home/coder/coder-dist/coder-base/npm-debug.log npm
npm ERR!
npm ERR! System Linux 3.10.18
npm ERR! command "node" "/usr/bin/npm" "install"
npm ERR! cwd /home/coder/coder-dist/coder-base
npm ERR! node -v v0.6.12
npm ERR! npm -v 1.1.4
npm ERR! message failed to fetch from registry: express/3.1.0
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/coder/coder-dist/coder-base/npm-debug.log
npm not ok

Thanks for reporting this. I've just published the latest release, which contains a more recent version of node and is more explicit about which packages it supports.


Sanddance-app in React

Hi Team,
How to consume the package sanddance-app in another React Application? There is no separate docs for sanddance-app package.

Hi @praveen5959 , sanddance-app is not a component meant to be consumed. feel free to make a copy of it and modify for your needs.


Change printing to logging

Makes it so we can filter the cuda setup messages at least

Is this not happening? I see the print statement is still there
https://github.com/TimDettmers/bitsandbytes/blob/main/bitsandbytes/cuda_setup/main.py#L147
On Sun, May 7, 2023 at 7:42 PM Tim Dettmers @.***>
wrote:

Closed #303 https://github.com/TimDettmers/bitsandbytes/pull/303.
—
Reply to this email directly, view it on GitHub
https://github.com/TimDettmers/bitsandbytes/pull/303#event-9185158276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYU34PA4T2EDIDBPH7Q5EDXFAXODANCNFSM6AAAAAAW3AU5HI
.
You are receiving this because you authored the thread.Message ID:
@.***>

It's literally a one-second change to make
On Mon, May 8, 2023 at 2:38 PM Jules Gagnon-Marchand <
@.***> wrote:

Is this not happening? I see the print statement is still there
https://github.com/TimDettmers/bitsandbytes/blob/main/bitsandbytes/cuda_setup/main.py#L147
On Sun, May 7, 2023 at 7:42 PM Tim Dettmers @.***>
wrote:

Closed #303 https://github.com/TimDettmers/bitsandbytes/pull/303.
—
Reply to this email directly, view it on GitHub
https://github.com/TimDettmers/bitsandbytes/pull/303#event-9185158276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYU34PA4T2EDIDBPH7Q5EDXFAXODANCNFSM6AAAAAAW3AU5HI
.
You are receiving this because you authored the thread.Message ID:
@.***>

@TimDettmers


Axon 4 - Saga run again after JVM restart

I am using Axon4 with Spring boot. Created simple saga. Its works well during JVM in flight. However once it restarts, it re-runs the Saga again.
Tried to to  JpaSagaStore  for persistance, did not work. Below are code snippet. Please help.
@Configuration
public class AxonConfig {
@PersistenceContext
private EntityManager entityManager;

@Bean
public SagaStore sagaStore() {
	return JpaSagaStore.builder().entityManagerProvider(new SimpleEntityManagerProvider(entityManager)).build();
}

}
@Saga(sagaStore = "sagaStore")
@Slf4j
public class OrderSaga {
@Autowired
private transient CommandGateway commandGateway;

private UUID orderId;

private boolean passed;

@StartSaga
@SagaEventHandler(associationProperty = "orderId")
public void on(OrderPlacedEvt evt) {
	log.debug("handling {}", evt);
	if (!passed) {
		orderId = evt.getOrderId();
		UUID shipmentId = UUID.randomUUID();
		associateWith("shipmentId", shipmentId.toString());
		commandGateway.send(new PrepareShipmentCmd(shipmentId, evt.getDestination()));
	}
}

@SagaEventHandler(associationProperty = "shipmentId")
public void on(ShipmentPreparedEvt evt) {
	log.debug("handling {}", evt);
	log.debug("orderId: {}", orderId);
	commandGateway.send(new RegisterShipmentForOrderPreparedCmd(orderId, evt.getShipmentId()));
}

@SagaEventHandler(associationProperty = "shipmentId")
@EndSaga
public void on(ShipmentArrivedEvt evt) {
	log.debug("handling {}", evt);
	log.debug("orderId: {}", orderId);
	commandGateway.send(new RegisterShipmentForOrderArrivedCmd(orderId, evt.getShipmentId()));
}

}

Let me give you a little back ground first @pms-github.
A Saga instance will receive it's from an Event Processor.
The default Event Processor in Axon 4 is the TrackingEventProcessor - this will keep track of which events it has handled. This allows you to start and stop your application, without events being handled a new, as well as enabling replays for your Query Model.
The TrackingEventProcessor for this 'keeps track' of which events it has handled in a TrackingToken, which is stored in a TokenStore.
My hunch is that you're using an InMemoryTokenStore - this means that on every restart, the TrackingEventProcessor used to provide events to your saga will start from the beginning of time.
Thus to resolve this you should ensure that the application running your Saga has a database to store tokens in.
Lastly, I'll be closing this as a 'Question', and would like to note that typically questions are meant for the Axon Framework User Group or for Stack Overflow (with the tag axon).

I got that but somehow not able to configure JPA persistant store properly. If you can help me with the ideal configuration to achieve persistant axon saga would be great :). Below is the configuration I tried.
@configuration
public class AxonConfig {
@PersistenceContext
private EntityManager entityManager;
@Bean
public SagaStore sagaStore() {
return JpaSagaStore.builder().entityManagerProvider(new SimpleEntityManagerProvider(entityManager)).build();
}

Spring data JPA

Then you should have the JPA TokenStore by default. Are you using an embedded database?

Spring data JPA
On Fri, Feb 1, 2019, 6:59 PM Allard Buijze<EMAIL_ADDRESS>wrote:

Do you use Spring Data JPA or configure JPA ‘manually’?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AxonFramework/AxonFramework/issues/983#issuecomment-459722625,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARJtu1IFX0ZWImTAD53WV_tN6OPkAoFqks5vJEEygaJpZM4adqiz
.

I am using Postgres database and default axon server docker image.
DB conf -
spring.datasource.username=orderdemo
spring.datasource.password=secret
spring.datasource.driver-class-name=org.postgresql.Driver
spring.jpa.properties.hibernate.dialect=org.hibernate.dialect.PostgreSQL95Dialect
spring.jpa.properties.hibernate.jdbc.lob.non_contextual_creation=true
spring.jpa.properties.hibernate.hbm2ddl.auto=create
spring.jpa.properties.hibernate.implicit_naming_strategy=org.springframework.boot.orm.jpa.hibernate.SpringImplicitNamingStrategy
spring.jpa.properties.hibernate.physical_naming_strategy=org.springframework.boot.orm.jpa.hibernate.SpringPhysicalNamingStrategy
On Fri, Feb 1, 2019, 8:09 PM Allard Buijze<EMAIL_ADDRESS>wrote:

Then you should have the JPA TokenStore by default. Are you using an
embedded database?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AxonFramework/AxonFramework/issues/983#issuecomment-459743392,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARJtuzuY0TSii7UUzGpzyaNAO2gO9Bf-ks5vJFGSgaJpZM4adqiz
.

The line
spring.jpa.properties.hibernate.hbm2ddl.auto=create
is the problem. create means that the database is cleared from any data on startup. Since AxonServer will still contain its events, the application is made to believe it has a lot of catching up to do.
You probably want
spring.jpa.properties.hibernate.hbm2ddl.auto=update
which creates tables if they don't exist, and will create columns if entities declare attributes that can't be mapped to existing ones.

It was freak mistake under my nose ! its working now with spring.jpa.properties.hibernate.hbm2ddl.auto=update , thank you !
Cheers


[shaders] Skybox is flipped vertical

Hi,
I had an issue with the skybox.c example running on my Windows 10 with the following configuration:
INFO: Initializing raylib 3.0
INFO: DISPLAY: Device initialized successfully
INFO:     > Display size: 1920 x 1080
INFO:     > Render size:  1920 x 1080
INFO:     > Screen size:  1920 x 1080
INFO:     > Viewport offsets: 0, 0
INFO: GLAD: OpenGL extensions loaded successfully
INFO: GL: OpenGL 3.3 Core profile supported
INFO: GL: OpenGL device information:
INFO:     > Vendor:   NVIDIA Corporation
INFO:     > Renderer: GeForce GTX 1060/PCIe/SSE2
INFO:     > Version:  3.3.0 NVIDIA 442.94
INFO:     > GLSL:     3.30 NVIDIA via Cg compiler
INFO: GL: Supported extensions count: 388
INFO: GL: DXT compressed textures supported
INFO: GL: ETC2/EAC compressed textures supported
INFO: GL: Anisotropic textures filtering supported (max: 16X)
INFO: GL: Mirror clamp wrap texture mode supported
the skybox appears v-flipped. I added to skybox.fs the support to flip the texture passing a bool.
I attach a .zip containing the example with shaders and assets.
SKYBOX.zip

@peppemas Please, could you send a PR with the improvement? Thanks!


<EMAIL_ADDRESS>decompress error

Scoop Configuration

//# Your configuration here

7-Zip 21.07 (x64) : Copyright (c) 1999-2021 Igor Pavlov : 2021-12-26
Scanning the drive for archives:
1 file, 21629864 bytes (21 MiB)
Extracting archive: C:\Users\sei\scoop\apps\grammarly-app\<IP_ADDRESS>\cosi.7z
WARNING:
C:\Users\sei\scoop\apps\grammarly-app\<IP_ADDRESS>\cosi.7z
Cannot open the file as [7z] archive
The file is open as [PE] archive
--
Path = C:\Users\sei\scoop\apps\grammarly-app\<IP_ADDRESS>\cosi.7z
Open WARNING: Cannot open the file as [7z] archive
Type = PE
Physical Size = 21629864
CPU = x86
Characteristics = Executable 32-bit NoRelocs NoLineNums NoLocalSyms
Created = 2021-09-26 03:27:46
Headers Size = 1024
Checksum = 21636316
Image Size = 610304
Section Alignment = 4096
File Alignment = 512
Code Size = 27136
Initialized Data Size = 186880
Uninitialized Data Size = 2048
Linker Version = 6.0
OS Version = 4.0
Image Version = 6.0
Subsystem Version = 4.0
Subsystem = Windows GUI
DLL Characteristics = Relocated NX-Compatible NoSEH TerminalServerAware
Stack Reserve = 1048576
Stack Commit = 4096
Heap Reserve = 1048576
Heap Commit = 4096
Image Base = 4194304
Comment =
{
FileVersion: <IP_ADDRESS>
ProductVersion: <IP_ADDRESS>
CompanyName:
FileDescription: Grammarly for Windows
LegalCopyright: Copyright Ac 2009-2022 Grammarly Inc.
ProductName: Grammarly for Windows
}
Path = [0]
Size = 21298048
Packed Size = 21298048
Virtual Size = 21298048
Offset = 324096
Path = [0]
Type = Nsis
Physical Size = 21298041
Tail Size = 7
Method = Deflate
Solid = -
Headers Size = 36630
Embedded Stub Size = 0
SubType = NSIS-3 Unicode
Sub items Errors: 1
Archives with Errors: 1
Sub items Errors: 1
ERROR: Data Error : Uninstall.exe.nsis

(base) PS C:\Users\sei> scoop install grammarly-app
Installing 'grammarly-app' (<IP_ADDRESS>) [64bit]
Loading GrammarlyInstaller.exe from cache
Checking hash of GrammarlyInstaller.exe ... ok.
Extracting cosi.7z ... ERROR Exit code was 2!
Failed to extract files from C:\Users\sei\scoop\apps\grammarly-app\<IP_ADDRESS>\cosi.7z.
Log file:
~\scoop\apps\grammarly-app\<IP_ADDRESS>\7zip.log
Please try again or create a new issue by using the following link and paste your console output:
https://github.com/gregwen/grewen-scoop/issues/new?title=grammarly-app%4<IP_ADDRESS>%3A+decompress+error

config.json scoop
{
"lastupdate":  "2022-05-22T21:47:32.9292020+05:30",
"SCOOP_REPO":  "https://github.com/ScoopInstaller/Scoop",
"SCOOP_BRANCH":  "master",
"MSIEXTRACT_USE_LESSMSI":  true
}


Why was the extension unpublished?

Marketplace page says:

This extension is now unpublished from Marketplace. You can choose to uninstall it.

I noticed that I got a message
We gave uniunstatlled 'cesium.gltf-vscode' which was reported to be problematic
I'm curious what issues there might have been (there's a chance I start dabbling with some multi-media-ish vscode extensions, so idly wondering what kind of general issues there might be).

See https://github.com/AnalyticalGraphicsInc/gltf-vscode/issues/132

Thanks @lexaknyazev.  Closing as duplicate with #132.

We're back on the air now.  But, users who launched VSCode today will have to manually reinstall.


doc: clarify that scanners are scanners and not inspectors

Differences between "hook set / scanner" and "inspector" are following :
A hook set is a collection of prebuilt hook - no more -  which can be load/unload during analysis. Each hook set has its purpose such bypass root detection. It was previously used to import hook redacted by community.
An inspector is a plugin able to:

register new buit-in hooks
change Dexcalibur component behaviors
emit own event
perform analysis on hook message
generate hook
perform own analysis over Dexcalibur internal DB
extend GUI
and so

"Scanner" or "Hook set" are probably not the better words. I created it before inspector concept. The idea is to make something close to Frida's "CodeShare" accessible directly from Dexcalibur's UI.

Thanks for the explanation.
What puzzles me is that we have hooks both on the Dashboard page and on the Hook-set pages, while they seem to be the same.
The only differences I see are:

Hooks are in folder inspectors while hook-sets/scanners are in src/scanner.
Hooks are enabled by default while are  hook-sets/scanners disabled by default.

What's the conceptual difference between inspector/hook of page Dashboard and hook-sets/scanners of page Hook Set?
Would it make sense to have them on the same page?

Short anwser :
Hooks defined into src/scanners should be moved into inspectors disabled by default, and Dashboard page should includes a tab panel with a tab for : built-in hook, not loaded hooksets,  custom hooks.
An Inspector is an HookSet and several others things deployed at startup.
An HookSet has been primarly created for offering "one-click security mechanism bypass" and Inspector concept as a plugin system. But it is not still true.
Long answer:
In fact, when you select a method and you click "probe", when you define a hook into an inspector, or when you define a list of hooks into a "hook set",  Dexcalibur creates HookPrimitive object.  It is like a hook template. When you "deploy" a hook set or an inspector, you ask to HookManager to generate the source code of each hook for the current context. The results is the editable source code displayed into Dashboard.
Dashboard page is the UI of the HookManager component, and display only . If you search a method  and click on "probe" button, the hook manager creates a new Hook

Inspectors define built-in hooks , already visible into Dashboard page. When, you start Dexcalibur, all inspectors are loaded before any analysis. An inspector can customize own hooks with data gathered by later analysis or previous hooking sessions or when a particular event is emitted by another inspector.

Hook-set contains simple  hooks, loaded on-demand through scanner page.


feat: new feat 6

:tada: This PR is included in version 2.2.0-stage.4 :tada:
The release is available on:

npm package (@stage dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[context] Move ServiceConfigCallData to arena based context

Pausing this until #36776 lands -- because this one depends on that one


ui.PlatformDispatcher.instance.sendPlatformMessage parameter reply , different value.

Hello, I get error from MethodChannel, invokeMethod, and webview.
today, I Debugged my app and created new app only for error function.
and I got this result,  inside flutter/lib/src/services/binding.dart
this ->>>  Future<ByteData?> send(String channel, ByteData? message)
and inside this, -->> ui.PlatformDispatcher.instance.sendPlatformMessage(channel, message, (ByteData? reply)
this method got different value of reply param.
see the below results,
this one is fine for param reply, which is created today for only function

and this one is my project but reply param is null

same computer, same function, but...why..?

@TrustOne
Can you provide flutter doctor -v and a complete minimal reproducible code sample that shows the reported behavior ?


chore: bdd-spring-1583470372 to 0.0.1

chore: Promote bdd-spring-1583470372 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


the process occupies 1.0T memory

it's so weird

$ ps aux |grep greenclip
7321    0.3    0.0<PHONE_NUMBER>   3104   pts/2   S+     18:17     0:00   greenclip daemon

sorry for the screenshot link http://paste.opensuse.org/6abd9150

Hello,
This is normal and not general to greenclip but to all static haskell program.
For more info go look here: https://downloads.haskell.org/~ghc/master/users-guide/8.0.1-notes.html#runtime-system
and here https://en.wikipedia.org/wiki/Virtual_memory
For your information, Virtual memory is not what your program is really consuming, it the space that has been requested by the program to the operationg system.
Operating system are smart enough to not allocate all the ram requested by the program, and allocate it chunk by chunk lazily.
If you want to know what a program is really using in RAM, check the RES/RSS column in htop, it is the residential memory, the one actually used.
In your case greenclip use  3.1 Mo

THX
and is it possible to obfuscate history cache content instead of plain text?

You can encrypt the file/directory with https://wiki.archlinux.org/index.php/ECryptfs and mount the partition at startup


#454 - reminder refactor

Should close #454.
Refactored reminder about upcoming ohs trainings or medical exams for users. Updated tests.
Should be opened after #451.

When I click "Anuluj" it redirects to request page
What happens when someone has overdue ohs training and admin adds him a new one? I'm not sure if this query will work properly

fixed


[pkg] Switch to composer-installers-extender

Closes: #5391

Coverage remained the same at 88.49% when pulling 8bb6782396628529096e7dfaa50a45732b442486 on vietduc01100001:develop into 3683efa8942cd68b03fd699b1dceb9c3af628dac on moment:develop.

Thanks for this.


bloom-rclc-18 correction

there is now a 'foxy' branch in ros2/rclc repository. I updated the version with the bloom-release tool, however, the version was not updated in foxy/distribution.yaml. Therefore this manual PR.
see previous PR: https://github.com/ros/rosdistro/pull/28481

@JanStaschulat Please rebase this PR on the latest; in particular, it shouldn't have the redundant change of changing the version tag to 1.0.0-1 (that was already done by #28481).  The change to the source branch is fine, but you probably also want to update the doc branch while you are at it.

@JanStaschulat Friendly ping. Please rebase this PR such that we only see the change to the source entry.

@jacobperron Sorry, for the delay. I rebased PR and updated doc version as well.


feat(explorer): use well-known oracle data on explorer oracle by id page

chore(markets): export extra components from market lib to aid consistency
feat(explorer): add oracle logos and verified status to oracle by id page

Related issues 🔗
Closes #6630
Description ℹ
Reuses some of the well-known oracle functionality from Trading/markets lib to display a logo/verification status on the oracle page in explorer.
Demo 📺

Looks good to me


[BUG] ModuleNotFoundError: No module named 'google.cloud.location'

Description
Since I've upgraded crewai to the latest version:
No matter which crewai script i want to run, i always get the same error
i've tried downgrading the version, etc... nothing works for me...
this is the complete outupt in the terminal:
/Users/user111/Library/Python/3.11/lib/python/site-packages/pydantic/_internal/_config.py:341: UserWarning: Valid config keys have changed in V2:
* 'allow_population_by_field_name' has been renamed to 'populate_by_name'
* 'smart_union' has been removed
  warnings.warn(message, UserWarning)
Traceback (most recent call last):
  File "/Users/user111/Documents/AIAgent-Sales/main.py", line 2, in <module>
    from crewai import Agent, Task, Crew, Process
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/__init__.py", line 1, in <module>
    from crewai.agent import Agent
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/agent.py", line 13, in <module>
    from crewai.agents import CacheHandler, CrewAgentExecutor, CrewAgentParser
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/agents/__init__.py", line 2, in <module>
    from .executor import CrewAgentExecutor
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/agents/executor.py", line 19, in <module>
    from crewai.agents.agent_builder.base_agent_executor_mixin import CrewAgentExecutorMixin
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/agents/agent_builder/base_agent_executor_mixin.py", line 4, in <module>
    from crewai.memory.entity.entity_memory_item import EntityMemoryItem
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/memory/__init__.py", line 1, in <module>
    from .entity.entity_memory import EntityMemory
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/memory/entity/entity_memory.py", line 3, in <module>
    from crewai.memory.storage.rag_storage import RAGStorage
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/memory/storage/rag_storage.py", line 9, in <module>
    from crewai.utilities.paths import db_storage_path
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/utilities/__init__.py", line 4, in <module>
    from .instructor import Instructor
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/crewai/utilities/instructor.py", line 3, in <module>
    import instructor
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/instructor/__init__.py", line 79, in <module>
    from .client_vertexai import from_vertexai
  File "/Users/user111/Library/Python/3.11/lib/python/site-packages/instructor/client_vertexai.py", line 5, in <module>
    from vertexai.preview.generative_models import ToolConfig  # type: ignore[reportMissingTypeStubs]
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/vertexai/__init__.py", line 17, in <module>
    from google.cloud.aiplatform import version as aiplatform_version
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/google/cloud/aiplatform/__init__.py", line 24, in <module>
    from google.cloud.aiplatform import initializer
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/google/cloud/aiplatform/initializer.py", line 33, in <module>
    from google.cloud.aiplatform import compat
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/google/cloud/aiplatform/compat/__init__.py", line 18, in <module>
    from google.cloud.aiplatform.compat import services
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/google/cloud/aiplatform/compat/services/__init__.py", line 18, in <module>
    from google.cloud.aiplatform_v1beta1.services.dataset_service import (
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/google/cloud/aiplatform_v1beta1/__init__.py", line 21, in <module>
    from .services.dataset_service import DatasetServiceClient
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/google/cloud/aiplatform_v1beta1/services/dataset_service/__init__.py", line 16, in <module>
    from .client import DatasetServiceClient
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/google/cloud/aiplatform_v1beta1/services/dataset_service/client.py", line 64, in <module>
    from google.cloud.location import locations_pb2  # type: ignore
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ModuleNotFoundError: No module named 'google.cloud.location'

Steps to Reproduce

Go to Visual Studio Code
run any crewai script

or run it from terminal
Expected behavior
i used to get results from crewai
Screenshots/Code snippets
.
Operating System
macOS Sonoma
Python Version
3.11
crewAI Version
0.51.0
crewAI Tools Version
.
Virtual Environment
Venv
Evidence
.
Possible Solution
.
Additional context
.

i solved it by installing vertexai (pip install vertexai)


add onto skip button to include "meh" message

its annoying to see skips not being shown to public, even with logs its tedious to find out who skipped
ima add this thing cos of that reason, this should be easy clueless emoji

added onto it and also made it play a same sound as when you click "meh"


chore: restructured project, compatible with sequelize v6

as requested in issue #4, the project was restructured, compatible with sequelize v6.

hey guys, I created un repository with as updates mentioned in this commit.
node-sequelize-noupdate-attributes

@daviddossantos feels like you should create a new repo for this? This repo is likely dead.

@daviddossantos @mmahalwy Yeah sorry, Life as usual stepped in the road and I had to focus on other things, making me unable to dedicate time to a project I didn't used frequently anymore.
If you wanna be promoted to manage this repo just tell me, I will be more than happy to see that even a simple utility like this is maintained and useful for other people.

@daviddossantos out of curiosity what exactly wasn't compatible with sequelize v6? The changes in this restructure appear to be semantically equivalent.

hello guys, it's been so long since I did this that I don't remember anymore.


Add an option to waits a minimum amount of time before showing a state

Inspired by the behavior of ContentLoadingProgressBar this would allow to prevent content flashing.
Ideally, this should be set per states. This requires some amount of work to pull-off and they might be some tricky edge cases.
For references:
https://developer.android.com/reference/androidx/core/widget/ContentLoadingProgressBar

What about debouncing calls to showState(..) with a default debounce duration of 0 millis ?
Developers interested in making states remain for at least X millis (e.g. to avoid blinking) will simply change the debounce duration to X.

That could be a good solution 👍. They might be some edge-case (like handling the real target state and not the previous one).
You're welcome to create a PR 😉

Actually a debounce would have introduced a delay in every state change, not good.
PR created with a better behavior :

If requests are chained (almost zero milliseconds in between) then all will be ignored except for
the latest one.

If one or many requests are sent during the [minimalTimeBetweenRequests] after a request has been
handled then only the most recent one will be forwarded to the handler.

Fixed by #85


healt says dead while I am alive

Description
so I am doing something an bang my health says dead can't take anything or inspect but I am alive still can move an open doors if I relog the bug is gone for some seconds an it comes back
Reproduction
do something an bang health says dead
Screenshots

dupe


Fix interpolation extremities on exponential and elastic curves

Resolves #3015.
Summary
Elastic and Expo curves due to the formulae used would not stop exactly at 0 or 1 at time 0 or 1, as pointed out in tests in c84b164. Resolve this discrepancy by introducing small linear factors to the appropriate easing curves.
The linear factors offset the differences at the extremities that were wrong, and then fade to 0 at the other ends which were correct. A special case are the InOut easings, in which the factor will be fully included at both ends and then fade out linearly to 0 at time = 0.5.
Since the adjustments involved are minuscule (less than 4 significant decimal places), they should not be noticeable during transitions, but fix the jumping at extreme points seen before.
Remarks
Aside from the unit tests originally written by @peppy (sorry for taking co-credit on that commit, I don't quite know why that happened - might have been due to me rebasing a bit to clean up history before push) I also included a test scene that allows for previewing all available curves for comparison reasons.
To check the visual effects you can either run the test scene or see the attached videos: before, after. The difference is quite subtle and will mostly be apparent on large sizes, and at the very ends of the transitions.

I'm not certain how good the JIT optimizations are, but it might make sense to bake all the constants into static variables to make sure they're not re-computed all the time. Pow is relatively expensive, after all. Perhaps worth a test.

Yeah, I figured it might be worth it. Will benchmark and report back.

Benchmarking results (see benchmark code):
Before extractions to constants

Method
Easing
Mean
Error
StdDev

ApplyEasing
InExpo
101.4 us
2.00 us
3.71 us

ApplyEasing
InElastic
114.5 us
2.19 us
2.34 us

ApplyEasing
OutElasticHalf
128.5 us
2.55 us
4.05 us

ApplyEasing
OutElasticQuarter
127.6 us
2.55 us
3.22 us

ApplyEasing
InOutElastic
134.7 us
3.95 us
11.65 us

After extractions to constants

Method
Easing
Mean
Error
StdDev

ApplyEasing
InExpo
46.87 us
0.635 us
0.594 us

ApplyEasing
InElastic
64.19 us
1.146 us
0.957 us

ApplyEasing
OutElasticHalf
62.81 us
0.950 us
0.889 us

ApplyEasing
OutElasticQuarter
59.69 us
0.533 us
0.499 us

ApplyEasing
InOutElastic
65.64 us
0.658 us
0.616 us

Yup, seems like a definite win performance-wise; just pushed that in.


Allow specifying packages for ambiguous modules

Fixes #9.
Still working on tidying this up. I was surprised to find that the ghc package does not have a parser for PackageNames in it. I think the best place to get that from is Cabal-syntax.

Note that this requires the PackageImports language extension, which is a little surprising. Anyway, here's a complete example. Starting with this module:
{-# OPTIONS_GHC -fplugin=Imp #-}
main = print Data.SemVer.initial

The import is ambiguous:
Main.hs:2:14: error: [GHC-45102]
    Ambiguous module name ‘Data.SemVer’.
    it was found in multiple packages:
    semver-<IP_ADDRESS> semver-range-0.2.8
  |
2 | main = print Data.SemVer.initial
  |              ^^^^^^^^^^^^^^^^^^^

So we can enable this new feature to disambiguate:
{-# OPTIONS_GHC -fplugin=Imp
  -fplugin-opt=Imp:--package=Data.SemVer:semver #-}
main = print Data.SemVer.initial

But that bumps into the extension being required:
Main.hs:3:14: error: [GHC-10032]
    Package-qualified imports are not enabled
    Suggested fix: Perhaps you intended to use PackageImports
  |
3 | main = print Data.SemVer.initial
  |              ^^^^^^^^^^^^^^^^^^^

So we can add the extension:
{-# LANGUAGE PackageImports #-}
{-# OPTIONS_GHC -fplugin=Imp
  -fplugin-opt=Imp:--package=Data.SemVer:semver #-}
main = print Data.SemVer.initial

And now it works:
Version {_versionMajor = 0, _versionMinor = 0, _versionPatch = 0, _versionRelease = [], _versionMeta = []}


DOPS-4261 Enabled bulk insert and improved exception debugging

@MiroslavArbor You can add this version of SDK to DI project in following ways:

run following commands locally or form Docker container (recomended way)
composer remove arbor-education/arbor-sdk-php
composer require arbor-education/arbor-sdk-php:dev-improvement/DOPS-4261-enabled-bulk-insert-and-improved-exception-debugging

by using symlink

clone SDK project to your computer
checkout branch improvement/DOPS-4261-enabled-bulk-insert-and-improved-exception-debugging
delete folder vendor/arbor-education/arbor-sdk-php in your DI project
symlink location of SDK project source code with folder of SDK package in vendor directory of DI project by using relative path (due to using Docker containers)
ln -s ./relative-path/to/sis-sdk-folder ./relative-path/to/di-folder/vendor/arbor-education/arbor-sdk-php

Looks like it's high time for this PR to come out of shadows and see some day light 😀
DI team continued work from the point where Dutek left with ticket DOPS-4261 and changes made on SIS and SDK side. We also have some basic banckmarks which can be found here.
@AnthonyPorthouse @aaronlinley @ronappleton  As project currently doesn't have CODEOWNER file I'm tagging you all so I can grab you attention and get some approvals. If you have any suggestion(s) and need some clarification on the work we done, we can setup a meeting and go over all those things.


Share game url copy button is not working

Hey, buddy! Could you give us a little more information? Do we need to look in CreateGame.vue (func add player) and PlayersList.vue?

Hey, thanks for showing interest. The share game button and text area is in a component: https://github.com/platput/filmemo/blob/main/filmemo-ui/src/components/ShareGame.vue
Currently it is working when locally tested but in production it is not.

I have figured out the issue and I am trying to fix it. The clipboard api will not work if the page is not served through https. So I am converting the site to use https and it will most likely solve the issue. Will keep this thread updated.

Got it. Did this solve your problem?

I have figured out the issue and I am trying to fix it. The clipboard api will not work if the page is not served through https. So I am converting the site to use https and it will most likely solve the issue. Will keep this thread updated.

Yes, I was using aws s3 to serve the front end which doesn't have https.
For https we can add cloudfront and that solved the issue. Thanks for asking.


Update completion to work with racer 2.0

Instead of the directory where the file resides, Racer 2.0 expects the third argument of complete to be the path to the original file. This patch performs that change.

Thanks for catching this.


Added extra information for minifabric update

When updating minifabric, it does not show the updated
image id and data time, this PR will add the image ID
and created at date time after update
Signed-off-by: Tong Li<EMAIL_ADDRESS>

Changed my mind on where this function should be added. I want to add this to stats command instead.


Added an argument allowing to force generation of diff file

As said, I added an argument to ocaml-mdx which forces the generation of a corrected file, even without diff.
It allows to really on the file in dune rules (e.g this example)

Thanks!


Change text image dir

https://github.com/boostcampaitech2-happyface/DALLE-pytorch/blob/b19883424dee08a042f430aa96e79a1df003bc27/train_dalle_gpt_kakao.py#L57-L62
Change the path to all cropped image(110k images), after 11k images for testing

Solved with 0e6744f1bb16963fde4daa0945fd0385557fb5ba commit


Update slack action to allow users to supply username and icon URL

override_slack_username_and_icon: default value is true
username: default value is fastlane
icon_url: default value is https://s3-eu-west-1.amazonaws.com/fastlane.tools/fastlane.png

The default value of override_slack_username_and_icon is true for backwards compatibility reasons. The current implementation always overrides the username and icon_url settings.
If override_slack_username_and_icon is true, the username and icon_url options are used. These both use default values to maintain backwards compatibility.

hey @codykimberling this is an awesome feature! thanks for submitting the PR! i made a couple comments above, curious to hear your thoughts! :octopus:

Hey @codykimberling, just wanted to check if you found the time to work in the last changes? :+1:

Thanks for the feedback!
The reason that an additional ConfigItem, override_slack_username_and_icon, was added is to give the user the option to fallback to what they configured in Slack's incoming webhook settings.
Slack lets you configure both a name and an icon that the webhook will use by default. I wanted to make sure that users of this action could configure the webhook at this level and this action would not override it.
By passing false to the override_slack_username_and_icon option, a user can fallback to the default Slack settings, which is achieved by setting the variables to nil. If it is true or not included (it is true by default), then the action will take the provided username or icon_url, if available, or default to fastlane and/or https://s3-eu-west-1.amazonaws.com/fastlane.tools/fastlane.png.
I realize that this implementation is a little confusing, but I wanted to ensure that we gave users an option to fall back to the Slack settings and, more importantly, I did not want to break any configurations currently in use.
Let me know what you think or if you have any suggestions. Thanks!

Thanks for the clarification @codykimberling! That seems like a legit proposal, would you mind if we are a bit pedantic here and try to change some of the wording so it is more clear? What would you think of changing the override_slack_username_and_icon option to something like use_webhook_configured_username_and_icon and it can default to false to make sure that people who are not using this option will keep their existing behavior. Definitely open to more conversation about this.

@ohwutup, good suggestion! I've updated the PR.
Thanks!

Thats awesome! Thanks @codykimberling !! Last request: Would you mind just squashing your commits down into one and rebasing off master to fix the merge conflicts? Then we will be :100: to :ship: !

@ohwutup squashed and rebased. Thanks!

 We really like to merge PRs with just one commit and somehow we got 250+ added on to this PR. I am going to pull down the branch and see if I can clean it up a bit, or if you're familiar with the process it would be great if you could reset and get back to just your commits. Thanks!

@ohwutup Yikes, sorry about that. I'll take a look and let you know if I run into any problems.

No worries! I just cleaned it up here https://github.com/fastlane/fastlane/pull/1456 but unfortunately, it gave me credit for the commit so I would rather merge one directly from you. So it might be best if you can just recreate the diff on top of master with just the necessary code change. Whatever is easiest for you! :octopus:

@ohwutup I created a new PR with just those changed here. Thanks!

Awesome! Thanks @codykimberling !!! :rocket:


Metal-poor abundances

For the giant star training set we were forced to include very metal-poor stars from Ruchti, Fulbright, etc so that our abundances didn't taper off at the edge of the training set. However, we didn't readily have individual abundances for those stars, so I just guessed them to follow galactic chemical evolution (e.g., [alpha/Fe] ~ +0.4).
It turns out that was a really bad thing to do, because now all metal-poor stars in that regime end up with those values. You can see this structure in the galactic chemical evolution plot.
We either need to:

collate individual abundances of (O, Mg, Al, Si, Ca, Ni) from the literature for those metal-poor stars (heterogenous sources are OK, we just need to keep track of where things came from -- it is more important for us to be able to identify stars with [Fe/H] ~ -4);
admit that our individual abundances for extremely metal-poor giants are probably all wrong (and perhaps, remove the individual abundances for stars below [Fe/H] ~ -2); or,
initialise them with random values around the central values we have already adopted.

Given time constraints, we probably cannot run too many experiments, because the analysis of 500,000 spectra with a 9 label regularized model takes about ~6 hours or so in wall clock time, unless I branch out to other CPU resources (which is possible).
I want discussion on this amongst co-authors. @mkness @davidwhogg @HWRix @keithhawkins @amkunder thoughts?

And @amkunder!

I am in favor of admitting and flagging and moving on. At this stage we can't extend scope

Ps we can explicitly flag any star at or beyond our safe zone

I agree with Hogg. To obtain results+paper by Gaia release we will need to
limit somewhat the scope. I would be worried about using heterogeneous
sources unless we can prove that they are systematically in line with the
primary source we are taking the labels from.
It may be worth exploring this later though.
On Mon, Sep 5, 2016 at 8:53 AM, David W. Hogg<EMAIL_ADDRESS>wrote:

Ps we can explicitly flag any star at or beyond our safe zone
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AnnieJumpCannon/RAVE/issues/13#issuecomment-244740100,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIDDq2lP_NYZ9pKNNVl_fuAYCz9eJcFVks5qnBC7gaJpZM4J05x9
.

I've admitted that our individual abundances for extremely metal-poor giants are probably all wrong. There will either be a QC flag and QC_ABUNDANCES that indicates whether results are knowingly erroneous or not, or I will just remove those results which I don't believe.


Result push trigger

[ch8142]
This is the internal implementation of result push. What's missing from this PR and will be implemented under separate tickets:

Limiting the number of rows
The actual HTTP API (in slamdata-backend)
A function/module to convert results to CSV. This PR depends on a passed function to accomplish this. I believe results are of type Stream[F, MimirRepr]
Some sort of integration tests. I have added unit tests but there are so many moving pieces I'd like to add something to SDBESpec.

Labeled as breaking since this changes the destination API slightly. Nobody is using that API yet, but still.

@wemrysi this should be ready for another look. The last problem with this PR is the use of @unchecked to find a ResultSink.Aux[F, ResultType.Csv[F]] in NonEmptyList[ResultSink[F]].
I'm convinced the cast is safe but I don't know how to convince the Scala compiler of the same.

@jsantos17 Looks good, if you want to remove the @unchecked, one way is to add a value of type cats.evidence.Is to the Csv ResultType case, like so
  final case class Csv[F[_]]() extends ResultType[F] {
    type T = (List[TableColumn], Stream[F, Byte])
    val isT: Is[T, (List[TableColumn], Stream[F, Byte])] = Is.refl
  }

then when you pattern match on c @ Csv() you can use c.isT to apply the type refinement to other instances of resultType.T.

Unfortunately it doesn't work even with isT.  I'm doing this:
  object Csv {
    def unapply[F[_]](sink: ResultSink[F]): Option[ResultSink.Aux[F, ResultType.Csv[F]]] =
      sink.resultType match {
        case (c @ ResultType.Csv()) => Some(new ResultSink[F] {
          type RT = ResultType.Csv[F]
          val resultType = ResultType.Csv[F]()

          def apply(dst: ResourcePath, result: resultType.T): F[Unit] =
            (c.isT.substitute[Function2[ResourcePath, ?, F[Unit]]](sink.apply))(dst, result)
        })
        case _ => None
      }
  }

i.e for the Scala compiler sink and sink.resultType are not related in any way. The error is:
found   : (quasar.api.resource.ResourcePath, sink.resultType.T) => F[Unit]
required: (quasar.api.resource.ResourcePath, c.T) => F[Unit]

It's almost the same type, but not quite

@jsantos17 Drat.
Do we use ResultType for anything else? If not, what if we just get rid of it and push the types into ResultSink
sealed trait ResultSink[F[_]]

object ResultSink {
  final case class Csv[F[_]](run: (ResourcePath, List[ColumnType], Stream[F, Byte]) => F[Unit]) extends ResultSink[F]
}

We do use it elsewhere, but I can fix that. I'll push another commit eliminating the existentials. Hopefully it's not too hard.

@wemrysi I have removed the type members and the build is green so this should be ready for another look.


Validation is to fast.

Hi Daniel Moncada,
I have a validation bug, as seen in the screenshot; when the end time is lower than the start time, the validation instantly changes the start time to the end time input. For instance I have 11:15 as start time and want to type with keyboard 15:15 for end time, the start time jumps to 01 because the "1" of "15" is lower than 11.
I think in the past with angular 7 I didn't have this issue, but when I have upgraded to angular 12 this happens.  How can I delay the input validation? Or how can I solve this issue?
I'm using the current packages:
@angular/animations": "~12.2.14",
"@angular/cdk": "^12.2.13",
"@danielmoncada/angular-datetime-picker": "^13.1.1",
"@danielmoncada/angular-datetime-picker-moment-adapter": "^1.1.0",
code:
<owl-date-time
#dtPickerRef
[pickerType]="'timer'"
[hour12Timer] = false
[stepMinute]="15"

@danielmoncada Is there any update on that subject?


New package: radj307.volume-control version 6.0.5

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-pkgs/issues/107639


Add h$isBoolean to jsbits. Req'd by GHCJS.Foreign

Sorry, this got closed due to cleaning up branches.  Please reopen against master.


add static analysis for allocations

To verify that the controller is guaranteed not to allocate

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

Comparison is base (88db5cd) 97.82% compared to head (53e65aa) 98.07%.
Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   97.82%   98.07%   +0.25%
==========================================
  Files           1        1
  Lines          46       52       +6
==========================================
+ Hits           45       51       +6
  Misses          1        1

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


feat(db): add SQLiteMigrator to perform db migrations

What type of PR is this? (check all applicable)

[x] Refactor
[x] Feature
[ ] Bug Fix
[x] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[x] Yes
[ ] No, because:

Description
This PR enhances our SQLite database with migration logic.
SQLiteMigrator class
The new SQLiteMigrator class handles safely running database migrations. It is initialized in the SqliteDatabase class's init, and immediately runs all database migrations.
Migration class
Migrations are reprsented by a Migration class, which has 3 attributes:

db_version: int: The database version this migration results in.
app_version: str: The semver app version this migration is run for.
migrate: Callable[[sqlite3.Cursor], None]: A function that performs the migration. It receives a cursor only, but can do anything it wants to do. A convention is established for these functions.

All schema-creating SQL now lives in a migrate function. We haven't needed to make any data migrations yet, but when we do, this will also be handled within one of these callbacks.
Migration Flow
First, migrations are registered with SQLiteMigrator with it's register_migration method. This performs some basic checks of the migration version.
After registering all migrations, they are run with the run_migrations method. This does a few things:

Creates a version table in the DB, if it doesn't already exist. This table has db_version INTEGER, app_version TEXT and migrated_at DATETIME columns.
Sort the migrations by their db_version.
Do some checks to see if we need a migration.
Backs up the database (if it's a file database). The migration bails out if this fails.
Runs each migration. If there is a problem, restore from backup.

Included Migrations
Migrations are in invokeai/app/services/shared/sqlite/migrations.
migrate_1.py
All* schema SQL up to 3.4.0post2 is in migration_1.py. Running only this migration should result in a database that is identical to the one you get from starting up 3.4.0post2.
SQL in this migration is idempotent (same as it was when the SQL was spread across the various services).
migrate_2.py
Schema changes through 3.5.0 (the upcoming release) are in migration_2.py.
SQL in this migration is not idempotent. Future migrations need not be idempotent, as the migration logic ensures each will only be run once.
*Caveat - ItemStorage
This class provides a generic document-db-like interface for storing objects. Our graph_executions and graphs tables are created and managed by this service. This PR does not touch this class and therefore does not touch either of those two tables.
We can decide how to handle those tables in the future as the need arises.
Change to Model Manager Metadata table
I noticed that there is a model_manager_metadata table which included the app version, and whose version property wasn't accessed outside the service.
I believe the new version table fulfills the purpose of this table, and have removed it.
@lstein Please let me know if this is not right.
QA Instructions, Screenshots, Recordings

Case 1 - Upgrade

Back up your 3.4.0post2 database
Run this PR
It should upgrade your database and everything should work exactly like it did before

Case 2 - New Install

Move your database out of the invoke root so that when the app starts, it creates a new one
Run this PR
It should work just like a new install

Case 3 - With an In-Memory Database

Enable the in-memory memory database (set use_memory_db under Paths in invokeai.yaml to true)
Run this PR
It should work just like a new install

Added/updated tests?

[x] Yes: Fairly comprehensive tests are added for the SQLiteMigrator.
[ ] No : please replace this line with details on why tests
have not been included

Oh, fun, the tempdirectory is failing github actions on windows :(

This looks neat. I'll start reviewing tomorrow.

@lstein Thanks for reviewing!
I thought a bit more about this since I created the PR and think my design has a major flaw.
Migrations currently just have a resultant db_version, and execution is based on inequality (less-than/greater-than) comparison between version numbers. There is an implicit assumption there about db version numbers which may be false.
Instead, the execution should be based on equality comparison. Migrations should have should have a from_db_version in addition to the to_db_version, and they only run if current_db_version == from_db_version.
Does my thinking here make sense? If so, I'll incorporate this and your other feedback and revise this PR

@RyanJDick @lstein
I've incorporated the your feedback, made some improvements and added a data migration for the workflow library:

Move the migrator and its calls out of the SqliteDatabase class
Use the safer backup strategy suggested by @lstein, where the migration is run on a copy of the database. If successful, the original database is renamed and retained as a backup, and the copy renamed to the original database's name.

We get free catastrophic failure detection from this. If a copy exists when the app starts up, it means the previous migration was interrupted. We just try again in this case, logging a warning.

Revise the Migration class to use from_version and to_version. This gives us much more confidence that we never run the wrong migrations.
Clean up the handling when using an in-memory db - we don't need to make backups and such
Add support for pre- and post- migration callbacks
Add the workflow library data migration as a post-migration callback. This data migration populates the new has_workflow column in the db. It needs access to a logger and image file service. These dependencies are provided by making a partial.
More comments and cleanup

Appreciate another round of reviews here.

@lstein Thanks for reviewing!
I thought a bit more about this since I created the PR and think my design has a major flaw.
Migrations currently just have a resultant db_version, and execution is based on inequality comparison between version numbers, e.g. current_db_version < migration.db_version. There is an implicit assumption there about db version numbers which may be false.
Instead, the execution should be based on equality comparison. Migrations should have should have a from_db_version in addition to the to_db_version, and they only run if current_db_version == migration.from_db_version.
Does my thinking here make sense? If so, I'll incorporate this and your other feedback and revise this PR

Yes, I think this is the right test to perform.

@RyanJDick @lstein Thanks for your reviews. I think I'm in final draft territory now.
SQLite transactions > backups
As @RyanJDick pointed out, SQLite has excellent safety around transactions and the backing up the database to a file is largely unnecessary.
I've removed the logic around this, which makes the migrator a lot cleaner. However, I wonder if it makes sense to back up the database before releases anyways - what do you think? This could be outside the migrator, or a very simple function within the migrator.
Migration dependencies
The v3.5.0 migration requires access to the image files service to check if they have an embedded workflow. I had handled this by manually creating partials of the migrate callbacks and handling them as "post" callbacks, but this was messy and error-prone.
I've revised this with a more resilient (and hopefully elegant) design. Migrations may declare dependencies at instantiation, which then must be provided at runtime. This enabled the 3.5.0 migration to use the image files service after it has been initialized.
I've added documentation here:
https://github.com/invoke-ai/InvokeAI/pull/5227/files#diff-cf84e4dd7ecaa30bf7862e256fe4723f7d7ec0b9b362d65d41cb54cf85da872aR51-R106
Hopefully this just needs one more round of reviews.

@RyanJDick
I've implemented your suggestion for handling dependencies - great idea. Comments updated and the dependency tests are removed as there's nothing to test.
I made some other small changes:

Minor improvement to db initialization. That is now in an init_db function, which is called both by the app and test suite. This ensures the we keep the tests and app using the same set of migrations. A mock is used for the image files dependency.
I renamed "SQLiteMigrator" to "SqliteMigrator" to be consistent with the rest of the app
Renamed Migration.migrate_callback to simply callback
Updated some comments

This this is ready

Let it be done.


Unable to receive incoming mails

Hello, I am able to send emails from my postal but I am not able to receive the reply mails (Incoming Mails).
postal gives the following error:
Connection reset by peer - SSL_connect
Support Ref: PBFAPVPN

Is this the error you're seeing in Postal or on another service?

The error is seen on the postal. When I try to send an email from anotherdomain.com to mydomain.com through postal, the ssl_connect error is seen. The mydomain.com has MX record pointed to my postal mail server. So mydomain will send and receive the emails via postal mailing server. However I am able to send email via postal mail server, but cany receive the reply or incoming mails.

Have you enabled the SMTP TLS settings?
https://github.com/postalhq/postal/blob/2595481b266199d8e29347dea2764f2c10202805/config/postal.defaults.yml#L75-L77
You can verify this from your terminal openssl s_client -connect yourpostalserver.whatever.com:25 -starttls smtp

@willpower232
here are my postal settings for SMTP Tls
smtp:
host: mydomain.com.np
port: 2525
username: # Complete when Postal is running and you can
password: # generate the credentials within the interface.
from_name: Postal
from_address<EMAIL_ADDRESS>smtp_server:
port: 25
tls_enabled: true
tls_certificate_path: /etc/letsencrypt/live/mydomain.com.np/fullchain.pem
tls_private_key_path: /etc/letsencrypt/live/mydomain.com.np/privkey.pem
tls_ciphers: TLSv1_1
ssl_version: TLSv1_1
proxy_protocol: false
log_connect: true
strip_received_headers: false
max_message_size: 14

smtp is for allowing Postal to send its own emails like password resets so isn't part of this problem.
It looks like you have everything correct but I'm not sure about tls_ciphers: TLSv1_1, if you set this as blank and restart Postal, does that fix the problem?

Suppose I have an email address<EMAIL_ADDRESS>how will I configure the address endpoint and routing such that the mail received from any address will be received on<EMAIL_ADDRESS>What will be on the Name field on below Image, also i have configured the address point to be<EMAIL_ADDRESS>

The endpoint will need to be either an email address (like your gmail) or another system that can handle email (like some custom code).
If you want an inbox for<EMAIL_ADDRESS>then that is not Postals job.

@willpower232 I have my domain setup in Cpanel, so I am using Roundcube as my email client. I am sending emails from roundcube with email address<EMAIL_ADDRESS>via mail server = postal.  When I send emails from<EMAIL_ADDRESS>from roundcube, i want all the replies/incoming emails to<EMAIL_ADDRESS> Is it possible ?
Also are there any specific firewall rules for postal. For now i have allowed port 443,25,465,5672.

You should be able to set the reply address to your email at some point, I'm not sure how Roundcube works. The reply address is different to the return path address.
You only need to open 80 and 443 for the web UI and 25 for SMTP. All other traffic would be internal to the server so would not be a concern for your firewall.


chore(api-builder): support API doc language agnostic URIs

This PR is in parts (and best explained and reviewed that way, in part because of the rename):
chore(api-builder): rename linkDevGuide to linkDocs
Rename the specialized API docs @linkDevGuide tag to @linkDocs since it is meant to be used to refer to any angular.io chapter, not just the developer guide.
chore(api-builder): let linkDocs links resolve to proper language
The old linkDevGuide required URIs to be given in the form 'ts/latest/guide/architecture' which forced all language versions of the API pages to refer to the ts versions of the doc pages they linked to.
This PR fixes that problem. URIs are now given in the form 'guide/architecture' and then rendered properly for each language. Of course, when needed, it is still possible to give a language specific URI.

I will squash and merge (no one cares about the steps)


Support additional custom prefixes

Currently, FlandersBOT only supports a single custom command prefix per server. It would be good to be able to add/remove more custom prefixes.

Added with: d42dedaa991271656b23824ffaf9e418623f2c2a
With the migration from json to DB, FlandersBOT now supports up to 10 custom prefixes per server.
Prefixes are now also more case friendly. For example, if you added a custom prefix of example, FlandersBOT will now respond to the following messages as a prefix:

example<command>
example <command>
example-<command>
Example<command>
Example <command>
Example-<command>
EXAMPLE<command>
EXAMPLE <command>
EXAMPLE-<command>

Note: prefixes are NOT case-insensitive. For example, FlandersBOT will still not respond to EXamPlE <command>


[PrintAsObjC] Fix handling of swift_name and swift_newtype (#2695)

Explanation: PrintAsObjC didn’t correctly take swift_name (for structs and enums) or swift_newtype (for typedefs) into account. This led to crashes in some cases and incorrect generated headers in others.

Scope: Depends on how widely adopted swift_newtype and swift_name are.

Issue: rdar://problem/26372925.

Risk: Medium-low. While it is unlikely that this will cause new crashes, it’s possible that it will result in generated headers that are incorrect in some other way.

Testing: Added compiler regression tests, verified that the fix-it for the original test case is correct.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.
[ ] Get a review.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

Proactively testing this commit in advance of a review, to get the ball rolling.
@milseman, @DougGregor, or @rjmccall, mind reviewing this? (same as the other request)
@swift-ci Please test

LGTM


[iOS] App crash on websocket close if app was in background for long-term

Steps to Reproduce

Run flutter create wscrash
Replace main.dart with next code:

import 'dart:async';
import 'dart:io';

import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter WebSocket Crash',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key? key}) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('WebSocket Crash'),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text(
              'After "Connect to WebSocket" move app to background, then wait 3 min and open. Boom!',
            ),
            TextButton(
              onPressed: () async {
                print('>>> wscrash connecting');
                WebSocket ws =
                    await WebSocket.connect('ws://echo.websocket.org');
                print('>>> wscrash connected');
                Timer(Duration(seconds: 30), () async {
                  print('>>> wscrash closing');
                  await ws.close();
                  print('>>> wscrash closed');
                });
              },
              child: Text('Connet to WebSocket'),
            ),
          ],
        ),
      ),
    );
  }
}

Run the app in release mode flutter run --release
Press Connect to WebSocket button
Move the app to background
Wait for 3-5 min (period depend from some internal iOS logic)
Tap on the app icon to reopen it

Expected results: the app is open then timer handler triggered and close WebSocket without error
Actual results: the app is open then timer handler triggered but on close WebSocket it crashed

Logs

$ flutter run --release
Multiple devices found:
iphone11 (mobile) • 11111111-2222222222222222            • ios            • iOS 14.6
iPhone 12 (mobile)   • 33333333-4444-5555-6666-777777777777 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
Chrome (web)         • chrome                               • web-javascript • Google Chrome 91.0.4472.101
[1]: iphone11 (11111111-2222222222222222)
[2]: iPhone 12 (33333333-4444-5555-6666-777777777777)
[3]: Chrome (chrome)
Please choose one (To quit, press "q/Q"): 1
Launching lib/main.dart on iphone11 in release mode...
Found saved certificate choice "Apple Development<EMAIL_ADDRESS>(XXX)". To clear, use "flutter config".
Signing iOS app for device deployment using developer identity: "Apple Development<EMAIL_ADDRESS>(XXX)"
Running Xcode build...
 └─Compiling, linking and signing...                        13.6s
Xcode build done.                                           48.7s
Installing and launching...                                        14.0s

Flutter run key commands.
h Repeat this help message.
c Clear the screen
q Quit (terminate the application on the device).

Application finished.

$ flutter analyze
Analyzing wscrash...
No issues found! (ran in 2.0s)

$ flutter doctor -v
[✓] Flutter (Channel stable, 2.2.1, on macOS 11.4 20F71 darwin-x64, locale en-UA)
    • Flutter version 2.2.1 at /usr/local/Caskroom/flutter/1.22.4/flutter
    • Framework revision 02c026b03c (2 weeks ago), 2021-05-27 12:24:44 -0700
    • Engine revision 0fdb562ac8
    • Dart version 2.13.1

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/ych/Library/Android/sdk
    • Platform android-30, build-tools 28.0.3
    • ANDROID_HOME = /Users/ych/Library/Android/sdk
    • Java binary at: /Users/ych/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7351085/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/ych/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7351085/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.1.2)
    • IntelliJ at /Users/ych/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.56.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.23.0

[✓] Connected device (2 available)
    • iPhone 12 (mobile) • 8DEBEAD6-B758-4696-A656-498B2E5245D3 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 91.0.4472.101

! Doctor found issues in 1 category.

Some lines from app Console log
...
default	20:51:12.987155+0300	Runner	flutter: >>> wscrash closing
default	20:51:12.990753+0300	SpringBoard	[application<com.example.wscrash>:2147] Workspace connection invalidated.
default	20:51:12.991018+0300	SpringBoard	[application<com.example.wscrash>:2147] Now flagged as pending exit for reason: workspace client connection invalidated
default	20:51:12.993191+0300	SpringBoard	Application process state changed for com.example.wscrash: <SBApplicationProcessState: 0x28255cca0; pid: 2147; taskState: Suspended; visibility: Foreground>
...
default	20:51:13.012008+0300	mediaserverd	-CMSessionMgr- CMSessionMgrHandleApplicationStateChange: Client com.example.wscrash with pid '2147' is now Terminated. Background entitlement: NO ActiveLongFormVideoSession: NO WhitelistedLongFormVideoApp NO
default	20:51:13.016213+0300	SpringBoard	[com.example.wscrash] com.example.wscrash application state changed to ForegroundRunning
default	20:51:13.016408+0300	SpringBoard	[com.example.wscrash] Ignore becoming foreground for application without push registration
default	20:51:13.016702+0300	SpringBoard	[com.example.wscrash] com.example.wscrash application state changed to Terminated
default	20:51:13.016963+0300	SpringBoard	[com.example.wscrash] Ignore becoming background for application without push registration
default	20:51:13.022229+0300	runningboardd	[application<com.example.wscrash>:2147] termination reported by launchd (2, 13, 13)
default	20:51:13.022264+0300	runningboardd	Removing process: [application<com.example.wscrash>:2147]
default	20:51:13.022476+0300	runningboardd	Removing launch job for: [application<com.example.wscrash>:2147]
default	20:51:13.022644+0300	runningboardd	Removed job for [application<com.example.wscrash>:2147]
default	20:51:13.022700+0300	runningboardd	Removing assertions for terminated process: [application<com.example.wscrash>:2147]
default	20:51:13.022766+0300	runningboardd	Removed last relative-start-date-defining assertion for process application<com.example.wscrash>
default	20:51:13.032129+0300	runningboardd	Calculated state for application<com.example.wscrash>: none (role: None)
default	20:51:13.032272+0300	runningboardd	Calculated state for application<com.example.wscrash>: none (role: None)
default	20:51:13.032545+0300	SpringBoard	[application<com.example.wscrash>:2147] Process exited: <RBSProcessExitContext| specific, status:<RBSProcessExitStatus| domain:signal(2) code:SIGPIPE(13)>>.
default	20:51:13.032710+0300	SpringBoard	[application<com.example.wscrash>:2147] Setting process task state to: Not Running
default	20:51:13.032776+0300	SpringBoard	[application<com.example.wscrash>:2147] Setting process visibility to: Unknown
default	20:51:13.033128+0300	SpringBoard	[application<com.example.wscrash>:2147] Invalidating workspace.
...

As a temporary solution works suggestion from https://github.com/flutter/flutter/issues/47203#issuecomment-590834018 (which based on section Handle or disable SIGPIPE from Apple Avoiding Common Networking Mistakes).
Also this #71979 seems related.

The minimal workaround is adding signal(SIGPIPE, SIG_IGN) to application function call in AppDelegate.swift:
import UIKit
import Flutter

@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
  override func application(
    _ application: UIApplication,
    didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?
  ) -> Bool {
    signal(SIGPIPE, SIG_IGN);
    GeneratedPluginRegistrant.register(with: self)
    return super.application(application, didFinishLaunchingWithOptions: launchOptions)
  }
}

/cc @aam - we should make sure that we suppress SIGPIPE on sockets.

I was unable to replicate after keeping app in background for around ~3 to 5 and ~8 to 10 mins. Labeling it based on above comment.

@darshankawar did you see the next line in Console app on macOS (when iPhone connected via cable)?
default	13:47:45.511503+0300	runningboardd	[application<com.example.wscrash>:4107] Shutdown sockets (ALL)

Also, I found how to speed up the crash reproduction - lock the screen of the iPhone (sometimes several times).
And I need to mention that this issue reproduces on a real device. I am not sure about the simulator.
Here is how it looks like:
https://user-images.githubusercontent.com/552025/121885042-5a23b500-cd1c-11eb-8acf-b86e4817d79f.mp4

/cc @aam - we should make sure that we suppress SIGPIPE on sockets.

We already do that on a server side of a socket https://github.com/dart-lang/sdk/blob/master/runtime/bin/socket_macos.cc#L149, but not on a client side.

Two things needs clarification:

we already ignore SIGPIPE on macos/ios(as well as linux) https://github.com/dart-lang/sdk/blob/master/runtime/bin/platform_macos.cc#L48. It's not clear why we still see SIGPIPE affecting process execution.
if I remove code that ignores SIGPIPE(from platform_macos/platform_linux) I expect to see failures in some test. I can't seem to write a test(pair of client/server apps) that would start crashing unexpectedly if other side closes socket pipe. But I don't see crashes, everything works as before.

Of course, ios might be special(than mac/linux), but that is puzzling.

@aam I'd guess that Platform::Initialize is not actually called from engine code.

ah, true @mraleph , it only seems to be invoked from standalone embedder/tester/gen_snapshotter.
Second question remains - it would be nice to understand how to repro this on mac/linux.

@aam as for removing code that ignores SIGPIPE and absent crashed, maybe it is because of setsockopt with SO_NOSIGPIPE in Socket::CreateBindDatagram and ServerSocket::CreateBindListen.

sorry @ycherniavskyi on macos for my tests I removed that code too. On linux we don't do this setsockopt call.
Also, that code only affects server side of the socket, so even if that code is in place client should fail(terminate) if the server drops connection(like you see happening on ios).
I think dart code responsible for writing into socket(socket_patch.dart) normally keeps track of socket state and normally prevents attempts to write into dropped closed socket. The way how paused app on iOS gets resumed though probably throws wrench into that state tracking, so such write-after-close does sneak through before the socket gets notified it was closed.

@aam to be precise I want to mention that in my case the app crashed on close, but I just check add and the behavior is the same.
() async {
  print('>>> wscrash add');
  ws.add('test');
  print('>>> wscrash delayed');
  await Future<void>.delayed(Duration(seconds: 3));
  print('>>> wscrash closing');
  await ws.close();
  print('>>> wscrash closed');
}

App output default	10:17:56.918102+0300	Runner	flutter: >>> wscrash add  in Console and then crashed.

Yes @ycherniavskyi , similar to ws.add(...) ws.close() results in closing data sequence being sent out, which can't go through if socket pipe was closed https://github.com/dart-lang/sdk/blob/master/sdk/lib/_http/websocket_impl.dart#L743

The fix rolled into flutter master in https://github.com/flutter/flutter/commit/e57d97afd8dd434665eba6ede1446f62ebb201fc


[FLINK-23868][Client] JobExecutionResult printed when suppressSysout is on

What is the purpose of the change
Environments prints job execution results to stdout by default and provides a flag suppressSysout to disable the behavior. This flag is useful when submitting jobs through REST API or other programmatic approaches. However, JobExecutionResult is still printed when this flag is on, which looks like a bug to me.
Brief change log
Skip printing JobExecutionResult if suppressSysout is on.
Verifying this change
This change is a trivial rework/code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduces a new feature?  no
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

The idea is sound but I'm wondering if we really want to change this behavior after 2 years (it was introduced in 1.10 by FLINK-15072).

On the other hand, one could question how many people rely on the JobExecutionResult being printed to the cluster .out files when using the application mode or when submitting a job via the web ui. I didn't know that this is a behaviour of Flink. I'd be fine with correcting it.

@link3280 I think the background here is where suppressSysout introduced. It seems we suppress sysout on application mode. @kl0u @aljoscha may I ask the reason introducing this bool flag?

Thanks a lot for your inputs! @zentol @tillrohrmann @tisonkun
In addition to the background, I was trying to submit jobs through Flink client interfaces in a gateway process (like Ververica's Flink SQL gateway) and found unexpected outputs in the process's stdout. I think suppressSysout was meant for this case when jobs are submitted in a process where the stdout is not (like CliFrontend does) directly showed to end users. @tisonkun

Merging...
Thanks for your contribution @link3280 !

Thanks @link3280 for this contribution and thanks @tisonkun for merging this PR.


Removed unnecessary printing during parsing

Relevant issue: https://github.com/igumnoff/shiva/issues/97


Review inlining and naming choices in OpenAPI

#463 refactors how we decide what to inline and what names to use and puts it all in one place, but it doesn't change behavior except to fix obvious bugs.
Bit of a brain dump here. I will think on it and try to build a proposal.
Briefly, we currently:

Inline anonymous types, including arrays, tuples, anonymous unions, anonymous models, etc.
Inline template instantiations with no @friendlyName,
We effectively provide a default @friendlyName of Type_Arg1_...ArgN if the args are not inlined and not themselves template instantiations.
Strip off root Cadl and service namespace

Issues:

friendlyName drops namespace
friendlyName can easily cause collisions (we will now produce an error if there's a collision)
Weird that Foo<string> is inlined but Foo<Bar<string>> is not (absent any explicit friendly names). This feels like the sort of quirk that will bite us when we go to change it and break someone.
Stripping Cadl alone still leaves lots of Rest.*. Maybe the namespaces to filter should be bigger and/or configurable, or maybe we should filter all namespaces unless there's a collision?
It's not really possible to use friendlyName with more than one type parameter. How do you refer to more than one name?
We report duplicate type names but only on the occurrences after the initial occurence. We should report all occurences as we do for symbols and endpoints. We should also consider along above lines if we can make collisions less likely or impossible in certain cases where they can occur today.

Mini Proposal:

Drop full qualification with namespace unless there is an actual collision. This might be tricky to implement, but I think it gives the cleanest experience.

Append namespace to friendly name also if friendly name collides. Friendly name collisions are still possible in other cases (say two types in same namespace with same friendly name). Fix the error reporting we have laready to report both locations that caused a collision, but keep the

3 Remove default friendly name with underscores. The conditions where we do this are, IMHO, too fragile and complex. We should do one of two things instead:
a. Inline generics unless they have a friendly name.
b. Emit generic instantiations with angle brackets and URL-encoded refs.
I could go either way on these. In both cases, people will dislike the emit sometimes but model is and @friendlyName allow the user to override things to their liking. For completeness of user control, consider adding an @inline decorator that the user can use to force inlining if we choose (b).

Augment the syntax for decorators with string substitutions to allow referencing more than one argument.

@friendlyName("Map{0:name}To{1:name}", K, V)
model Map<K, V>`

This can be made backwards compatible, if there is no ':' '0:' is implied. We could also consider allowing {<number>} as being equivalent to {<number>:name} since .name property is the overwhelmingly common case and then you could also do @friendlyName("Map{0}To{1}, K, V).
Also disallow undefined property just formatting into the name as undefined with a proper diagnostic.

Filed #562 to do 3a, which was the only conclusion in the design meeting today. We will need to re-discuss the other points in a future design meeting.

Need to still decide 1, 2, and 4 design