[FEATURE] ASPNETCORE UI (MVC)

Hello @mkeymolen ,
Please could you add a ASPNETCORE MVC user interface sample where users have the ability to SignIn, SighUp, ResetPassword and Signout?

@NicoCG i can do that, but you still need to configure some stuff in Azure Ad/B2c . i'll try to implement the feature and add some documentation

Hi @mkeymolen,
That would be great I excited to see this sample.

Hi @mkeymolen, Sorry to bother you but is there and ETA on this implementation on this sample.
Do not want to sound needy haha just need it for a project 🙈

Hi @mkeymolen, Sorry to bother you but is there and ETA on this implementation on this sample.
Do not want to sound needy haha just need it for a project 🙈

Sorry, very busy at the moment. I'll see what i can do asap

@mkeymolen Thank you, this means a lot to me.

Hi @mkeymolen,
I just want to confirm is this sample related to this issues request?
If so may I suggest why not merge Library A with B so you have one library Hexiron.Azure.ActiveDirectory to maintain which caters for UI and API?
If not the merge suggestion then maybe rename the library Hexiron.AspNetCore.Authentication.AzureAdMixed to Hexiron.Azure.ActiveDirectory.B2C maybe?

@nicotravassos i updated the READMES and added a HowTo in the wiki.
This library only contains the basics and the API connectors. The Hexiron.AspNetCore.Authentication.AzureAdMixed library contains extensions on the current Microsoft.AspNetCore.Authentication library that enables you to use both AzureAD and Azure AD B2C in the same host.
With the extensions in that library you can accept JWT tokens issued by either an Azure AD or Azure B2C tenant.
However, i also added howTo's in the wiki and a UI sample (as you noted above) where i show how you can activate Azure AD B2C authentication.
See How to Authenticate with Azure AD B2C in AspNetCore
This should fix this feature request.


Child pid killed by signal 11

Upgrading sslsplit inside Debookee from 0.5.4 to 0.5.5, sslsplit stops with signal 11 as soon as it sees first traffic.
Debug says :
Connecting to [<IP_ADDRESS>]:993
tcp <IP_ADDRESS> 51907 <IP_ADDRESS> 993
TCP connected to [<IP_ADDRESS>]:993
TCP connected from [<IP_ADDRESS>]:51907
Checking for a client hello
Peek found ClientHello
Child pid 25352 killed by signal 11

Configuration:
On macOS 10.14.6, non-root Email traffic is redirected to interface lo0 and then port 6971 through redirections and NAT rules:
pass out on en0 route-to lo0 inet proto tcp from any to any port = 993 user != 0 flags S/SA keep state
pass out on en0 route-to lo0 inet6 proto tcp from any to any port = 993 user != 0 flags S/SA keep state
rdr pass on lo0 inet proto tcp from any to any port = 993 -> <IP_ADDRESS> port 6971
rdr pass on lo0 inet6 proto tcp from any to any port = 993 -> ::1 port 6971

For bug reports, please supply:

Output of sslsplit -V

./sslsplit -V
SSLsplit 0.5.5 (built 2020-08-27)
Copyright (c) 2009-2019, Daniel Roethlisberger<EMAIL_ADDRESS>https://www.roe.ch/SSLsplit
Build info: OSX:10.14.6 XNU:4903.241.1:fallback:4903.278.28 V:FILE HDIFF:0 N:83c4edf
Features: -DHAVE_PF
NAT engines: pf*
Local process info support: no
compiled against OpenSSL 1.0.2 22 Jan 2015 (1000200f)
rtlinked against OpenSSL 1.0.2 22 Jan 2015 (1000200f)
OpenSSL has support for TLS extensions
TLS Server Name Indication (SNI) supported
OpenSSL is thread-safe with THREADID
OpenSSL has engine support
Using SSL_MODE_RELEASE_BUFFERS
SSL/TLS protocol availability: ssl3 tls10 tls11 tls12
SSL/TLS algorithm availability: SHA0 RSA DSA ECDSA DH ECDH EC
OpenSSL option availability: SSL_OP_NO_COMPRESSION SSL_OP_NO_TICKET SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION SSL_OP_DONT_INSERT_EMPTY_FRAGMENTS SSL_OP_NO_SESSION_RESUMPTION_ON_RENEGOTIATION SSL_OP_TLS_ROLLBACK_BUG
compiled against libevent 2.1.11-stable
rtlinked against libevent 2.1.11-stable
compiled against libnet 1.1.6
rtlinked against libnet 1.1.6
compiled against libpcap n/a
rtlinked against libpcap 1.9.1
4 CPU cores detected

Output of uname -a

Darwin Pep 18.7.0 Darwin Kernel Version 18.7.0: Mon Feb 10 21:08:45 PST 2020; root:xnu-4903.278.28~1/RELEASE_X86_64 x86_64

Exact command line arguments used to run sslsplit
sudo ./sslsplit -k ~/mitmproxy-ca.pem -L ~/logAllData.log -l ~/logConnections.log autossl <IP_ADDRESS> 6971

List of failing unit tests in make test output

make test
100%: Checks: 145, Failures: 0, Errors: 0

make sudotest
shell-init: error retrieving current directory: getcwd: cannot access parent directories: Permission denied
job-working-directory: error retrieving current directory: getcwd: cannot access parent directories: Permission denied
100%: Checks: 147, Failures: 0, Errors: 0

Can you disable (comment out) the call to the function bufferevent_openssl_set_allow_dirty_shutdown() in pxy_conn_autossl_peek_and_upgrade() in the file pxyconn.c, and then recompile and try if it is going to crash still? That seems to be one of the main differences in autossl between 0.5.4 and 0.5.5.
Btw, I guess the command line you have provided is not the actual command line you have used, because it is missing the -c option. Otherwise, sslsplit would quit with an error like "no CA cert specified (-c)". And I am surprised that the version of OpenSSL on osx is so old.

Same problem here. I commented out the line and the problem persists!

I tried with "ssl" and it stops also:
Initialized 16 connection handling threads
Started 16 connection handling threads
Starting main event loop.
SNI peek: [n/a] [complete]
Child pid 12268 killed by signal 11

Can you try the underlying-bevs-issue#303 branch and report back please? It has a couple of fixes for autossl at least.
Otherwise, enabling the DEBUG_PROXY switch in GNUmakefile can probably give us more info.

This is with DEBUG_PROXY switch in GNUmakefile enabled. Let me know if you require more info:
Initialized 16 connection handling threads
Started 16 connection handling threads
Starting main event loop.
0x55ae931a0200             pxy_conn_ctx_new
            0x55ae931a0470 pxy_bufferevent_setup
Connecting to [<IP_ADDRESS>]:12697
0x55ae931a0200 0x55ae931a0470 eventcb dst connected
            0x7fa81c000cd0 pxy_bufferevent_setup
tcp <IP_ADDRESS> 50021 <IP_ADDRESS> 12697
TCP connected to [<IP_ADDRESS>]:12697
TCP connected from [<IP_ADDRESS>]:50021
0x55ae931a0200 0x7fa81c000cd0 src readcb
Checking for a client hello
Peek found ClientHello
Child pid 4101 killed by signal 11

Unfortunately, those debug logs do not help either (I guess you have started sslsplit with the -D option, right?). The debug logs in sslsplit are not verbose.
So the other option are:

Use gdb: You can generate a dump file to use with gdb, and obtain a backtrace on the same system that sslsplit crashes. This is supposed to give us the exact location in the source code where the crash happens.
Try SSLproxy: SSLproxy supports the split mode of operation similar to sslsplit with the same proxyspec syntax, and provides very verbose debug logs, so if it crashes too we may get more info. Plus, if sslproxy does not crash, so we know it is really sslsplit. And other possibilities.
macOS: I should gain access to an osx test machine, reproduce those crashes, and try to fix myself (these crashes happen on an osx, right?). But this seems unlikely.

Btw, I don't know your xnu version, but another possibility is that this may be about the header files under xnu. Please see the xnu folder in the sources.

If this issue is on macOS Mojave 10.14.6 (which is the system the OP was using), can you please try the xnu-4903.270.47 branch? Since I don't have a macOS machine, this is a stab in the dark.

Update:
I've cloned github repository for sslsplit and now it's working fine in Kali Linux
Output of sslsplit -V for the working scenario:
# /home/kali/Downloads/sslsplit/sslsplit -V
SSLsplit 0.5.5-12-ge17de84 (built 2022-07-22)
Copyright (c) 2009-2019, Daniel Roethlisberger<EMAIL_ADDRESS>https://www.roe.ch/SSLsplit
Build info: V:GIT
Features: -DHAVE_NETFILTER
NAT engines: netfilter* tproxy
netfilter: IP_TRANSPARENT IP6T_SO_ORIGINAL_DST
Local process info support: no
compiled against OpenSSL 3.0.3 3 May 2022 (30000030)
rtlinked against OpenSSL 3.0.3 3 May 2022 (30000030)
OpenSSL has support for TLS extensions
TLS Server Name Indication (SNI) supported
OpenSSL is thread-safe with THREADID
OpenSSL has engine support
Using SSL_MODE_RELEASE_BUFFERS
SSL/TLS protocol availability: tls10 tls11 tls12
SSL/TLS algorithm availability: !SHA0 RSA DSA ECDSA DH ECDH EC
OpenSSL option availability: SSL_OP_NO_COMPRESSION SSL_OP_NO_TICKET SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION SSL_OP_DONT_INSERT_EMPTY_FRAGMENTS SSL_OP_NO_SESSION_RESUMPTION_ON_RENEGOTIATION SSL_OP_TLS_ROLLBACK_BUG
compiled against libevent 2.1.12-stable
rtlinked against libevent 2.1.12-stable
compiled against libnet 1.1.6
rtlinked against libnet 1.1.6
compiled against libpcap n/a
rtlinked against libpcap 1.10.1 (with TPACKET_V3)

Output for the non-working one:
┌──(root㉿kali)-[/etc/sslsplit]
└─# sslsplit -V
SSLsplit 0.5.5 (built 2021-12-26)
Copyright (c) 2009-2019, Daniel Roethlisberger<EMAIL_ADDRESS>https://www.roe.ch/SSLsplit
Build info: V:FILE HDIFF:1 N:83c4edf
Features: -DHAVE_NETFILTER
NAT engines: netfilter* tproxy
netfilter: IP_TRANSPARENT IP6T_SO_ORIGINAL_DST
Local process info support: no
compiled against OpenSSL 1.1.1n  15 Mar 2022 (101010ef)
rtlinked against OpenSSL 1.1.1o  3 May 2022 (101010ff)
OpenSSL has support for TLS extensions
TLS Server Name Indication (SNI) supported
OpenSSL is thread-safe with THREADID
OpenSSL has engine support
Using SSL_MODE_RELEASE_BUFFERS
SSL/TLS protocol availability: tls10 tls11 tls12
SSL/TLS algorithm availability: !SHA0 RSA DSA ECDSA DH ECDH EC
OpenSSL option availability: SSL_OP_NO_COMPRESSION SSL_OP_NO_TICKET SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION SSL_OP_DONT_INSERT_EMPTY_FRAGMENTS SSL_OP_NO_SESSION_RESUMPTION_ON_RENEGOTIATION SSL_OP_TLS_ROLLBACK_BUG
compiled against libevent 2.1.12-stable
rtlinked against libevent 2.1.12-stable
compiled against libnet 1.1.6
rtlinked against libnet 1.1.6
compiled against libpcap n/a
rtlinked against libpcap 1.10.1 (with TPACKET_V3)
6 CPU cores detected

Seems like I was using a pretty old version and wasn't aware of it.


Fix k8s panic

Fixes #877
Fixes #869
DO NOT merge before validation made in https://github.com/containous/traefik/issues/877

Should'nt this be enought? Whats the motives to use "third" party libs in stead of just channels and goroutines directly?
@@ -270,9 +271,9 @@ func (c *clientImpl) watch(url string, labelSelector string, stopCh <-chan bool)

        go func() {
                defer close(watchCh)
-               defer close(errCh)
                go func() {
                        defer res.Body.Close()
+                       defer close(errCh)
                        for {
                                var eventList interface{}
                                if err := json.NewDecoder(res.Body).Decode(&eventList); err != nil {

@jonaz I don't know what you are calling "third party libs" here.
safe.Go is taken from Traefik directly. The goal is to avoid Traefik to panic and stop. So it is not useless ;)

I mean some external package thats not part of go. Just thought the fix was a bit to complicated for what was needed. But i'll admit i did not thoroughly read the safe packge from traefik.

Is'nt there a possible goroutine leak aswell?
	safe.Go(func() {
		EndCh := make(chan bool, 1)
		defer close(watchCh)
		defer close(errCh)
		defer close(EndCh)
		safe.Go(func() {
			defer res.Body.Close()
			defer func() {
				EndCh <- true
			}()
			for {
				var eventList interface{}
				if err := json.NewDecoder(res.Body).Decode(&eventList); err != nil {
					if !strings.Contains(err.Error(), "net/http: request canceled") {
						errCh <- fmt.Errorf("failed to decode watch event: GET %q : %v", url, err)
					}
					return
				}
				watchCh <- eventList
			}
		})
		<-stopCh **// Wont we listen here forever in case of error in Decode?**
		safe.Go(func() {
			cancel() // cancel watch request
		})
		<-EndCh
	})

provider/kubernetes.go seems to check for error channel and send stop to it. But i dont think a client should be dependant on the usage not to leak goroutines.
DISCLAMER: This maybe should belong in another issue on github.

@jonaz goroutine leak has been tested quite intensively during the RC phase.
Besides, we are in the process of moving to the official k8s client in the next release.

Good. Official client should sufficient! :D


Tracker blocking opt in experiment

Reviewer: @jdorweiler
CC: @russellholt @bbraithwaite @bsstoner @chrismorast @MariagraziaAlastra
Depends on: https://dub.duckduckgo.com/duckduckgo/ddg/pull/15986
Description:
This is part of a larger effort to measure the impact of making tracker blocking opt-in in the extension and apps. This PR contains a minimal experiment framework based on atb variant, a content script that is injected into the post install page, changes to the popup to show an 'activate tracker blocking' screen when it's disabled, and a lot of functionality around when tracker blocking is turned on or off.
Steps to test this PR:

Follow testing steps 1-10 in https://dub.duckduckgo.com/duckduckgo/ddg/pull/15986 to get the extension installed in the experiment group.
There should be a red badge on the extension icon when tracker blocking is disabled.
There should also be an 'activate tracker blocking' CTA in the extension popup when tracker blocking is disabled.
Trackers should be tallied in popup but not blocked when tracker blocking is disabled.
There are four places where tracker blocking can be activated: in the extension popup, on the post install page (button in middle of page, button in header that appears when you scroll down on post install page, toggle on extension settings page).
When tracker blocking is turned on via any of these methods, all extension icons should update to hide the red badge and all popups should update to hide the activate tracker blocking CTA. When tracker blocking is enabled while on the post install page via either in page buttons or the popup, a tba cookie should be set. This is to remember that tracker blocking has been enabled so the post install page shows the success state. We do this so that if a user installs then opens the popup on the post install page and activates tracker blocking, the post install page reloads and shows the success state. If you want to test other CTAs on the post install page, make sure to delete this cookie first.
When tracker blocking is enabled via the popup CTA, the current page should reload after a few seconds.
When tracker blocking is disabled, 'report broken site' button should not appear in the hamburger menu in the popup.
All of this should only affect new installs in the experiment group (va=k). Make sure nothing here affects existing users. Test with control group (va=l) and other or no variant / atb variant to make sure none of these changes impact those groups.
Check pixels if you have time, for reference see https://app.asana.com/0/0/1122176086265607/f

Automated tests:

[ ] Unit tests
[ ] Integration tests

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR Author Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications

👍


[Bug]: Invisible chests, ender chests and shelker boxes

Describe the bug
Invisible chests, ender chests and shelker boxes
To Reproduce

Use pure modpack
Open and close chest repetitively
Invisible chest occurs

Expected behavior
Chest types shouldn't be invisible
Screenshots
None.
Type of the modpack
All edition
Operating System Name/Version
Windows 11 23H2
Modpack Version
QuantuMiz 3.1.1
Minecraft Version
1.21.1
Name of Launcher you are using
Modrinth App
Additional context
Temporary Workaround: Disable dynamic resources on Mod Menu -> ModernFix
3.1.1b patch is coming later (soonish) if I have time to fix it but not until friday
Gimme some time

Fixed in https://github.com/QuantuMiz/QuantuMiz/releases/tag/v3.1.1b


Parameters schematron missing

Describe the bug
Trying to validate a Parameters resource using the schematron validator fails due to missing schematron file.
To Reproduce
See minimal reproduction project: https://github.com/MFAshby/parametersbug
The validation fails with an error Failed to load schematron for resource 'Parameters'
Note that validation of a different resource (e.g. Observation) works as expected, this bug appears to be restricted to Parameters resources only.
Expected behavior
The Parameters resource should be validated according to the schematron (available here http://hl7.org/fhir/parameters.sch)
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

HAPI FHIR Version = 3.7.9
OS: [e.g. iOS] = Linux
Browser [e.g. chrome, safari] = N/A

Additional context
Add any other context about the problem here.

On further investigation, the file parameters.sch appears to be missing from the STU3 release of FHIR: http://hl7.org/fhir/STU3/parameters.sch returns 404, wheras other resource schematrons are present in STU3 http://hl7.org/fhir/STU3/observation.sch.
The Parameters schematron is present in version 4 http://hl7.org/fhir/parameters.sch, might it be worth backporting this to 3?


Task is complete


Rename the package to tinker-engine - [merged]

In GitLab by @waxlamp on Aug 10, 2020, 17:50
Merges rename -> master
The word "framework" and phrase "the framework" (referring to this
entire software package) have been converted to appropriate usages of
"Tinker Engine" (referring to the full package), "tinker" (referring to
the name of the executable), or left as "framework" in its generic sense
of a software system that other things can hook into.
This includes renaming the Python package directory to tinker-engine,
the console script associated with the executable version to tinker,
and updating other references in code, comments, and elsewhere as well.
This leaves the documentation and Dockerfile intact, as those will be
updated in future efforts. However, these changes are enough to change
the name of this GitLab repository.
Closes #37.
TODO

[x] change the name of the repository to tinker-engine

Related to:

sail-on-client !26
evm_based_novelty_detector !79

In GitLab by @waxlamp on Aug 10, 2020, 17:51
@cfunk1210 please review

In GitLab by @waxlamp on Aug 10, 2020, 17:51
@as6520 please review

In GitLab by @waxlamp on Aug 10, 2020, 17:51
@eswears please review

In GitLab by @waxlamp on Aug 10, 2020, 18:18
changed the description

In GitLab by @waxlamp on Aug 11, 2020, 09:11
mentioned in merge request !36

In GitLab by @as6520 on Aug 11, 2020, 09:41
LGTM

In GitLab by @as6520 on Aug 11, 2020, 09:41
approved this merge request

In GitLab by @as6520 on Aug 11, 2020, 09:41
mentioned in merge request darpa-sail-on/sail-on-client!26

In GitLab by @as6520 on Aug 11, 2020, 10:11
mentioned in merge request darpa-sail-on/evm_based_novelty_detector!79

In GitLab by @cfunk1210 on Aug 11, 2020, 13:31
Is this a good idea right before the evaluation?  Should we have staged MR's for both sailon and learn to update them at the same time?

In GitLab by @waxlamp on Aug 12, 2020, 09:12
(Eran has declined to review this, so I'm resolving this thread on his behalf.)

In GitLab by @waxlamp on Aug 12, 2020, 09:14

Is this a good idea right before the evaluation?

We can wait until after the evaluation, so I'll open a new blocking thread with a note to wait until after the 27th.
I'd still like your review before that, though.

Should we have staged MR's for both sailon and learn to update them at the same time?

Yes, and @as6520 actually has opened some already. @ameya, would you mind editing the MR description to add links to those MRs? @cfunk1210, could you compile a list of other repos that will require such staged MRs?

In GitLab by @waxlamp on Aug 12, 2020, 09:14
This MR should not be merged until after 2020-08-27.

In GitLab by @as6520 on Aug 12, 2020, 09:18
changed the description

In GitLab by @as6520 on Aug 12, 2020, 09:18
Done, I think the only other repo that would require the changes would be learn

In GitLab by @waxlamp on Aug 28, 2020, 09:36
resolved all threads

In GitLab by @waxlamp on Aug 31, 2020, 13:55
added 12 commits
d49232c9...14736a83 - 11 commits from branch master33e09b18 - Merge remote-tracking branch 'origin/master' into rename
Compare with previous version

In GitLab by @cfunk1210 on Aug 31, 2020, 15:51
My main concern is that this will break the learn framework locally.  Roni, can you do an MR on learn so we can do that at the same time?
Also, we should have a tag for the release right before this (just in case).

In GitLab by @waxlamp on Aug 31, 2020, 16:34
Sure, I can work on this.

In GitLab by @waxlamp on Aug 31, 2020, 16:40
Is there already an MR in progress for Learn?

In GitLab by @waxlamp on Aug 31, 2020, 16:50
mentioned in merge request learn!90

In GitLab by @waxlamp on Aug 31, 2020, 16:50
See https://gitlab.kitware.com/darpa_learn/learn/-/merge_requests/90.

In GitLab by @cfunk1210 on Sep 1, 2020, 10:49
resolved all threads

In GitLab by @cfunk1210 on Sep 1, 2020, 10:49
approved this merge request

In GitLab by @waxlamp on Sep 1, 2020, 17:04
marked the task change the name of the repository to tinker-engine as completed

In GitLab by @waxlamp on Sep 1, 2020, 17:05
mentioned in commit 738c8c55dd4f9ad6eb5ecf2d47595a58850dd1fc

In GitLab by @waxlamp on Sep 1, 2020, 17:05
merged


3609 Inventory out should include items allocated to kits. Inventory in should include kits allocated

Resolves #3609
Description:
It's just a quick draft PR to see if I am in the right direction with my approach
I created one model 'allocations' with enum allocation_type: ['inventory_in', 'inventory_out']
so far code added for allocation only, once got some feedback I will also add code for de-allocation
about 'inventory_out'
at time of kit allocation, I am creating one record for 'allocations' with type 'inventory out' with line_items equal to kit's line_items
having quantity in negative, quantity is  based number of kits added
so in-case we add another kit with the same kit_id, it will just find this allocation record and adjust quantity of line_items
about 'inventory_in'
at time of kit allocation, creating one record for allocations with type 'inventory_in', then adding only one line_item with kit.item.id and kit quantity,
in-case user add another kit with same kit_id it will find the allocation record with kit_id and type 'inventory_in'
and adjust the first line_item's quantity
Question:
in-case invetory_in there is always one line_item for one record in allocation
so having has_many association for line_items doesn't makes sense here
instead of making itemizable type model we can have model similar to invetory_item for both invetory_in and invetory_out, wdyt?

@dorner some specs are failing, I checked they are also failing in main branch where I have made no change, any thoughts?

@dorner after re base with main now all specs are passing, please review

Thanks @zeeshan-haidar -- we pushed this to production this morning -- there are still some things to be done, though -- please check out the re-opened issue.


Error when running in github actions

We seem to get an error when running yarn fauna-schema-migrate apply all in github actions.
The migrations seems to complete successfully though.
The error message suggests adding some checks and fallbacks to prevent this.
https://github.com/vadimdemedes/ink/#israwmodesupported

Does it work with the --legacy flag?
I figured that different environments might struggle with ink which is why I added that.
If it doesn't. I'll take a look next week.

Yes, it works as expected when the --legacy flag is added.
Thank you, I'll close the issues


updated scale factors so that odometry works

put in resetOdometry and getOdometry

We tested on Gladys and had reasonable numbers for X,Y, rotation. Also used a straight edge to align the steering angles a little better.


Show more fix

Whenever user clicks on 'Show More' in Add Expense, we persist the data in localStorage so that user doesn't need to click again and again.

Use storageService, recentLocalStorageItemsService is sort of a persistant cache in local storage which we use to store recently used projects, categories, etc. recentLocalStorageItemsService is built on top of storageService and has additional checks which are not needed here

I have changed the service


The draggable dom structure is diffrent from the sample

First thank you for this component, it's very useful !
Then I have something confused, when I use this component just as the "basic sample". I got the draggable dom differently.
Normally it should be like:

(The .react-grid-item is on the same div with the class .react-draggable)
But actually my page like this:

span class="react-draggable"...
+----  div class="react-grid-item" ...
This makes the css is very hard to wirte.
What should I do about this? Do I have missed some tips?
I use npm installed the "react-grid-layout": "0.10.0-beta1",
and my src:

var BasicLayout = React.createClass({
  propTypes: {
    //onLayoutChange: React.PropTypes.func.isRequired
  },
  getDefaultProps() {
    return {
      className: "layout",
      items: 3,
      rowHeight: 30,
      cols: 6,
      isDraggable: true,
      isResizable: true,
      autoResize:false,
      useCSSTransforms:false,
    };
  },
  getInitialState() {
    var layout = this.generateLayout();
    return {
      layout: layout
    };
  },

  generateDOM() {
    var rows = [];
    for (var i=0; i < this.props.items; i++) {
        rows.push(< div key={i}> < span >11111111111< / span > < / div >);
    }
    return rows;
  },

  generateLayout() {
    var rows = [];
    for (var i=0; i < this.props.items; i++) {
        var y = 4;
        rows.push({x: i * 2 % 12, y: Math.floor(i / 6) * y, w: 2, h: y, i: i});
    }
    return rows;
  },

//uncomment this will have error:  Uncaught TypeError: this.props.onLayoutChange is not a function
  //onLayoutChange: function(layout) {
  //  this.props.onLayoutChange(layout);
  //},

  render() {
  	let page_style = {
  		backgroundColor: 'rgba(210, 230, 159, 0.0745098)',
  		marginLeft:'50px',
  		marginRight:'auto',
  		paddingLeft:'20px',
  		paddingRight:'20px',
  		maxWidth:'800px'
  	};

    return (
      < div id="content" {...this.props} style={Object.assign(page_style, this.props.style)} >
      	< ReactGridLayout layout={this.state.layout} onLayoutChange={this.onLayoutChange}
          {...this.props}>
          {this.generateDOM()}
        < / ReactGridLayout>
      < / div>
    );
  }
});

I'm also curious about this. For one it's not correct to place a div within a span, for another it broke my layout.

The <span> is effectively used for grouping, but it could probably be eliminated.
In reality, a <span> can certainly contain a <div>, and this happens all the time in React projects for one reason of convenience or another.

Yeah, I just heard that inline elements shouldn't contain block elements, perhaps it's puritan. For HTML validation I usually just use the different css display-properties on divs to avoid warnings / errors, but the end result should be the same.

Do you have any advice on how to go about removing the wrapping <span> tags? The presence of the tag is making the draggable functionality very janky. Items can still be rearranged but they snap immediately to the spot of their placeholder instead of moving along with the mouse.

I think this is a known issue, if you look at https://github.com/STRML/react-grid-layout/issues/129.

This is what the DOM looks like in 0.10:

You should be good to go if you upgrade.


S3 download links stop working early with IAM roles

I have pypicloud set up using the default local SQLite database and with S3 package storage.  Rather than putting the AWS credentials in plain text in the pypicloud.ini file, the system running the server has an IAM role that allows it to access the S3 bucket with my packages.
I'm frequently seeing a problem where 'pip install' reports a 400 HTTP error.  If I manually try to fetch the redirect error, I get back an Amazon error saying "The provided token has expired."  This is with the default settings (expire_after=86400 seconds, buffer_time=600 seconds), and both the URL and the expiry data stored in SQLite suggest the URL should be valid for another 17 hours or so.
I have a vague suspicion that the IAM role is introducing an unfortunate level of indirection, where the credentials boto comes up with actually expire after a relatively short period of time, but I haven't been able to prove this yet.
pypicloud==0.2.7 (with boto==2.34.0, if that's relevant) on Python 2.7.6 in an Ubuntu 14.04 Docker container on an Ubuntu 14.04 host.

Great! I'll merge it in and cut a new release soon

Fixed in 0.2.10 release


Remove cause attribute from RespondDecisionTaskFailedRequest

DecisionTaskFailedCause doesn't make sense when coming from a decider.
https://github.com/uber/cadence/blob/master/idl/github.com/uber/cadence/shared.thrift#L908

at least the workflow execution reset logic use this cause as indication that workflow is resetted


How to get image features of my own dataset

Thanks for contributing such interesting and meaningful work!
Your codes work great, and I wonder how I can use the code for my own dataset. I noticed you have processed vitb32.hdf5 for each dataset, which contains a 512-dimensional ViT feature for each image. So could you give some official instructions to create a vitb32.hdf5 for my own dataset?

Thanks for asking! I just updated the script to extract CLIP features here.
Hope it helps :)

Thanks for the prompt response.
I also wonder if there is a script to get images from input texts using VQGAN or OFA.

We were using the following official implementations to render images from text with VQGAN & OFA.

VQGAN+CLIP: https://github.com/nerdyrodent/VQGAN-CLIP
OFA: https://github.com/OFA-Sys/OFA

Thanks very much for your kind help!!! I'll have a try!
Good day!

Good luck! I'll close this issue for now


Date confusion

The article states, in a note, that "[t]he A8, A9, A10 A11 VMs are planned for retirement on 3/2021." It seems that by the date of the article, 01/02/2022, the planned retirements could have been confirmed and the note changed to say that the models were retired. Or one might argue that the pending (at the time of the original article) retirement is now, after the fact, a moot point.

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

ID: fc49ab2c-f3f6-b6d1-5715-80377ec0ac2b
Version Independent ID: 9289adae-4e8c-12dc-ec26-2a5f2ffe5cf7
Content: Azure VM sizes - General purpose - Azure Virtual Machines
Content Source: articles/virtual-machines/sizes-general.md
Service: virtual-machines
Sub-service: vm-sizes-general
GitHub Login: @brbell
Microsoft Alias: brbell

@brucelangston Thank you for bringing this to our attention!  Now that's it's been a year, I'll confirm that we are ready to remove this note.

@brucelangston Thank you again for bringing this to our attention!  We're still waiting for official word from the doc author on removing the note, but a PR has been created to address that these sizes have been retired until that call is made.  Thank you for helping to improve the docs!

@brucelangston I spoke with the doc author and we are going to remove the note entirely.  I'll get a PR out for that shortly.  Thank you again for helping to improve the Microsoft Docs!


[Doc] Add Supported LLMs to README

Hi Astra,
Greetings from 01.AI. Thank you for building this wonderful project and integrating Yi! 😘
As discussed with @cyfyifanchen today, we both think adding a "Supported LLMs" section to Astra's README is a good idea, as it 1) showcases Astra's thriving ecosystem 2) helps users locate desired info more quickly
Example:
Supported LLMs List of LlaMA-Factory

If this suggestion is accepted and implemented by Astra, could you please add "Yi" to the "Supported LLMs" list?
Feel free to let me know if you have any questions or concerns. Thank you!  🙌

The very first issue from the community, gotta love it.

Now it has a ready-to-use extensions to inform users available extensions.


feat: adds support for channel priming and refreshing

It contains the downstream changes of https://github.com/googleapis/gax-dotnet/pull/704 to enable channel priming and refreshing.

Hi @jskeet and @amanda-tarafa,
Please have a look at it and share your feedback.
Thanks,

Closing this as the feature is put on hold.


 Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

re #247
feat: runtime lazy eval scope infra

2. What is the scope of this PR (e.g. component or file name):

kclvm/compiler/src/codegen/llvm/context.rs
kclvm/compiler/src/codegen/llvm/emit.rs
kclvm/compiler/src/codegen/llvm/module.rs
kclvm/runtime/src/eval/**

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[x] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

All grammar tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

79 of 212 (37.26%)  changed or added relevant lines in 9 files are covered.
10 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.1%) to 71.748%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kclvm/runtime/src/_kclvm_addr.rs
0
5
0.0%

kclvm/runtime/src/context/mod.rs
0
10
0.0%

kclvm/runtime/src/context/api.rs
0
57
0.0%

kclvm/runtime/src/eval/mod.rs
0
61
0.0%

Files with Coverage Reduction
New Missed Lines
%

kclvm/runtime/src/context/mod.rs
1
9.57%

kclvm/compiler/src/codegen/llvm/context.rs
2
75.59%

kclvm/compiler/src/codegen/llvm/node.rs
7
89.34%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
44570

Relevant Lines:
62120

💛  - Coveralls


Please add error code 0x0111000A to the table of SPMT error codes

Please add error code 0x0111000A to the table of SPMT error codes as it is not clear that the SPMT cannot process files greater than 100Gb.
Result category
SCAN FAILURE
Message
Scan File Failure:File size exceeds limit
Error Code
0x0111000A
Recommended action
SPMT is unable to upload files greater than 100Gb (see release notes) . The only option is to use OneDrive for Business.

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

ID: cfa08336-690e-0d21-6491-3f14f6d13391
Version Independent ID: 8927975a-7956-fc72-055e-36603e4eddaf
Content: Troubleshoot SharePoint Migration Tool - Migrate to Microsoft 365
Content Source: migration/troubleshooting-common-spmt-issues.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

@buildbod Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hello @buildbod how are you?
I tested and could verify, adding the information thru a pr.
Thanks for your feedback

Great thanks for confirming and making the change
Simon

From: Andres Mariano Gorzelany @.>
Sent: 15 July 2021 22:58
To: MicrosoftDocs/OfficeDocs-SharePoint @.>
Cc: Simon Denton @.>; Mention @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SharePoint] Please add error code 0x0111000A to the table of SPMT error codes (#2852)
Hello @buildbodhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbuildbod&data=04|01|simon.denton%40mottmac.com|fde2326241214ed334bb08d947dbbd76|a2bed0c459574f73b0c2a811407590fb|0|0|637619831362274690|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=di7CbDid44S8VJRobCzPa6mFytqOxS4J8EDr3UkhC98%3D&reserved=0 how are you?
I tested and could verify, adding the information thru a pr.
Thanks for your feedback
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SharePoint%2Fissues%2F2852%23issuecomment-881038775&data=04|01|simon.denton%40mottmac.com|fde2326241214ed334bb08d947dbbd76|a2bed0c459574f73b0c2a811407590fb|0|0|637619831362274690|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=hL8j4EqpFxXgfpJDpc0dbdf69JZJdnYgU4nTUquqkrY%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADKCNOEVX334KT5XFAGNNZTTX5KZ3ANCNFSM5ALO42DA&data=04|01|simon.denton%40mottmac.com|fde2326241214ed334bb08d947dbbd76|a2bed0c459574f73b0c2a811407590fb|0|0|637619831362284645|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=iHti4RHXLDDsMQPoQ%2BiNKPItkwn7VwC9RvkCQ3%2Fngtk%3D&reserved=0.


Add option for FileManagerStrategy discard stale data

Closes #61.
AWS PutLogEvents will not upload a batch log if the logs in the batch are older than 14 days. Therefore we will allow the user to specify an option in FileManagerStrategy to automatically discard 14 day old logs from the batch.
This PR will expose the discard_stale_data option to the user.
PR to add logic and testing for the option has been opened in cloudwatch-common repo:
Allow user to automatically discard CloudWatch logs in batch that are > 14 days old #56
Signed-off-by: Jesse Ikawa<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The Travis builds currently do not pass since FileManagerStrategyOptions has not been updated with the new discard_stale_data option from cloudwatch-common PR.


fix(http1): force always-ready connections to yield after a few spins

Adds a YieldNow helper, since I believe there may be a couple other places that could make use of this, like the accept loop in SpawnAll.
Closes #1828
cc @ry

Thanks @seanmonstar, that seemed to have fixed the problem!

(From https://deno.land/benchmarks.html)


Removed tkn-pac resolve message from generate command

Changes
Addressed https://github.com/openshift-pipelines/pipelines-as-code/pull/1028#discussion_r1035750484

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

let's hold on on it as I am working to fix tkn pac resolve with git auth secret..


Image location not matching with imagery and OpenStreetMap Data

Image location mismatch

http://mapbox.github.io/osm-navigation-map/#18.33/49.23247/-123.01175
Here the location of the image showing in Mapillary layer is not matching with the OSM data present around. Seeing the same difference in both iD and JOSM editors.
@gyllen It would be great if you have a look at this and see what is making this difference of location.
cc @planemad

Similar to the above issue, I have observed that Mapillary signage location is misplaced on OSM Navigation map.

Detected no_u_turn sign at the bottom Link to OSM naavigation map
The image associated is located at a different junction. I have checked on Mapillary website by pasting the image key from Navigation map.

As seen on Mapillary website. Link
As you can see from the above images,the signs on OSM navigation map are located in between 'Fernside Ct' and 'Parkview Avenue' streets but the actual location on the Mapillary webpage is at the junction of 'Bayview Avenue' and 'Sheppard Avenue E'
cc: @planemad

Tracking on new ticket


[5449] Allow admins to delete records

Context
https://trello.com/c/uog5bcBM/5449-l-add-feature-to-allow-admins-to-delete-records
Changes proposed in this pull request

New journey to soft delete non-awarded traineea

Guidance to review

Log in as admin
Pick a non-awarded trainee and remember the name for later
Click the admin table and click "Delete this trainee" link at the bottom
Follow the journey and play around looking for bugs
If everything goes as expected the trainee should be delete and not show up on the trainees index page

Screenshots

Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

Looks good John. Confirmed you can't delete awarded trainees. Please can we also block 'QTS RECOMMENDED' trainees from being deleted?
Content here: 'Trainee already QTS' needs to be changed to 'Trainee already has QTS'.
@edwardhorsford @johndoates should we be warning Support this will mean they will not be able to access or view this record anymore and neither will the provider?

NB: we already block editing on 'finished' records for non admins - recommended, awarded, and withdrawn trainees - should we use the same scope for limiting which records can be deleted?
@claire-hughez I don't think saying anything extra is necessary (or likely to be read?).
Further bug. When zendesk url isn't provided, can this show Not entered as a fallback?

NB: we already block editing on 'finished' records for non admins - recommended, awarded, and withdrawn trainees - should we use the same scope for limiting which records can be deleted?
I don't think we should block deleting withdrawn records. I think many providers will have withdrawn records instead of deleting them.


cmd/compile: longtest builder failing on TestNexting/gdb-opt-hist

After CL 174617, the longtest builder is failing on TestNexting/gdb-opt-hist.
Here's a sample of the log:
--- FAIL: TestNexting (13.35s)
    --- FAIL: TestNexting/gdb-opt-hist (2.98s)
        debug_test.go:245: step/next histories differ, diff=
            --- testdata/hist.gdb-opt.nexts	2019-04-30 21:30:30.000000000 +0000
            +++ /workdir/tmp/debug_test382328161/test-hist.gdb-opt.nexts	2019-04-30 22:37:43.423960976 +0000
            @@ -28,8 +28,6 @@
             i = 1
             81:			hist = ensure(int(i), hist)
             82:			hist[int(i)]++
            -74:		for scanner.Scan() { //gdb-opt=(scanner/A)
            -scanner = (bufio.Scanner *)
             75:			s := scanner.Text()
             76:			i, err := strconv.ParseInt(s, 10, 64)
             77:			if err != nil { //gdb-dbg=(i) //gdb-opt=(err,hist,i)
            @@ -38,8 +36,6 @@
             i = 1
             81:			hist = ensure(int(i), hist)
             82:			hist[int(i)]++
            -74:		for scanner.Scan() { //gdb-opt=(scanner/A)
            -scanner = (bufio.Scanner *)
             75:			s := scanner.Text()
             76:			i, err := strconv.ParseInt(s, 10, 64)
             77:			if err != nil { //gdb-dbg=(i) //gdb-opt=(err,hist,i)
            @@ -48,8 +44,6 @@
             i = 1
             81:			hist = ensure(int(i), hist)
             82:			hist[int(i)]++
            -74:		for scanner.Scan() { //gdb-opt=(scanner/A)
            -scanner = (bufio.Scanner *)
             75:			s := scanner.Text()

When I compile src/cmd/compile/internal/ssa/testdata/hist.go both before and after this change, with both -S and -S=2, I don't see any line number differences. This is the extent of the difference of the -S output:
$ diff old new
1611c1611
< 	0x0020 02 07 01 0e 02 12 01 6e 02 06 01 1c 02 08 01 04  .......n........
---
> 	0x0020 02 07 01 0e 02 12 01 6e 02 02 01 20 02 08 01 04  .......n... ....

This difference is part of the go.isstmt."".test SDWARFMISC size=0 symbol.
So somehow my change messed up the statement marks? Is there a way to display those differences?
@dr2chase , any ideas?

Dup of #31263?

Working on it...
go test debug_test.go -v -args  -f -n

produces
...
=== RUN   TestNexting/gdb-opt-hist
( /Users/drchase/work/go-quick/bin/go build -o testdata/test-hist.gdb-opt '-gcflags=all= -l' -ldflags=-compressdwarf=false testdata/hist.go )
( TERM=dumb gdb -nx -iex 'add-auto-load-safe-path /Users/drchase/work/go-quick/src/runtime' -ex 'set startup-with-shell off' testdata/test-hist.gdb-opt )
tbreak main.test
run
# Tag for above is gdb-opt
...

Redo good and bad compilations prepending GOSSAFUNC=test to both of them.
Also compare dwarfdump -debug-line for both, search forward for testdata:
"good"
..................     74      0      2   0             0  is_stmt
..................     82      0      2   0             0  is_stmt
..................     82      0      2   0             0
..................     81      0      2   0             0
..................     74      0      2   0             0
..................     81      0      2   0             0
..................     74      0      2   0             0  is_stmt
..................     74      0      2   0             0

"bad"
..................     74      0      2   0             0  is_stmt
(missing)
..................     82      0      2   0             0
..................     81      0      2   0             0
..................     74      0      2   0             0
..................     81      0      2   0             0
..................     74      0      2   0             0  is_stmt
..................     74      0      2   0             0

It works okay with delve, I am still searching through the ssa output to see where the difference appears.  It's in blocks 17 and 19, v264 and v275.

I have a CL coming that doesn't fix this, but it at least stops us from relying on dumb luck to make gdb happy.  I have not yet figured out what would make gdb happy; a naive reading of the marked lines here (the loop in question, after coming CL) shows more than enough is_stmt=1 lines.

If the fix isn't going to land today, please add a skip so that the longtest builder will be able to proceed past TestNexting again. (The build dashboard is still unpleasantly red and I don't want to mask any more regressions.)

If the fix isn't going to land today, please add a skip so that the longtest builder will be able to proceed past TestNexting again. (The build dashboard is still unpleasantly red and I don't want to mask any more regressions.)

How do you feel about me changing it to use Delve instead?

@dr2chase, you're asking the wrong people. We don't know what this is testing. We just know it's red & harmful to people trying to get a useful regression signal out of this builder, and making people get used to broken builds.

Should have a fix (switch to delve, tested for 100+ repetitions on Linux and Mac) later today.
Tests running now will I had off to a talk.


a11y : Fix search tab underline

Closes https://github.com/denoland/dotland/issues/2354

Remove use of CSS({}) and text-decoration
Use bottom-border to give an underline

Works on Chrome, Firefox, Opera

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please format

Please format

Formatted the code, and merged the k === kind conditions in that block.


Loading large XLSX files failed due to unddefined split function

Action: read xlsx file with morde then 100000 rows without value for Range
Error message:
Uncaught TypeError: Cannot read property 'split' of undefined alasql.min.js:7 (anonymous function)alasql.min.js:3 utils.loadBinaryFile.xhr.onload
Note: happens only, if no range is definied
Your code snippet:
if(workbook.Sheets[sheetid][range])range=workbook.Sheets[sheetid][range]}var rg=range.split(":");

Please ignore. It is a issue for xlsx. Problem of country specific formats.


Why can't I use @use with this?

I've implemented this according to the docs ... but whenever I use @use nothing works. but if I use the old @import everything is fine.
Now, HOW do I make sure that I use the right (Dart Sass – which support @use) version and NOT LibSass? I've installed Dart Sass globally npm i -g sass, and also locally (otherwise it won't run). So, basically I'm confused ...

Can you post an example of what isn't working? A GitHub repository, CodeSandbox or something else?
gulp-sass is just a thin wrapper around sass, and I use @use without problems myself. Most likely there's a problem with your setup, but you don't give much detail about it.

Now, HOW do I make sure that I use the right [...] version

In your project:
npm install sass gulp gulp-sass --save-dev

In your gulpfile:
const gulp = require('gulp');
const sass = require('gulp-sass')(require('sass'));

Same here, I had to manually wrap sass and it works like charm...
while passing the importers when calling gulp-sass didn't work!


[WIP] FBX pivot support

Last thing to do on monday:

fix duplicate lookup so it works via stack :)
remove extra argument added when testing


Cancellation of styled components

For me, as a developer, the appearance of styled components was very significant. I fell in love with this library and used it on many projects, followed its development, told colleagues.
But the addition of the postinstall script by the author in version 5.3.3 offended and discredited me. I believe that open source is not for political slogans and agitations.

Too bad


Application Exception: Error on GetAllItems

This is an issue created automatically by GrammarNazi when an exception was captured.
Date (UTC): 04/16/2021 23:03:30
Exception:
System.InvalidOperationException: Error on GetAllItems
---> System.Threading.Tasks.TaskCanceledException: The request was canceled due to the configured HttpClient.Timeout of 180 seconds elapsing.
---> System.TimeoutException: The operation was canceled.
---> System.Threading.Tasks.TaskCanceledException: The operation was canceled.
---> System.IO.IOException: Unable to read data from the transport connection: Operation canceled.
---> System.Net.Sockets.SocketException (125): Operation canceled
--- End of inner exception stack trace ---
at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error, CancellationToken cancellationToken)
at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.GetResult(Int16 token)
at System.Net.Security.SslStream.ReadAsyncInternal[TIOAdapter](TIOAdapter adapter, Memory1 buffer)    at System.Net.Http.HttpConnection.FillAsync(Boolean async)    at System.Net.Http.HttpConnection.ReadNextResponseHeaderLineAsync(Boolean async, Boolean foldedHeadersAllowed)    at System.Net.Http.HttpConnection.SendAsyncCore(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)    --- End of inner exception stack trace ---    at System.Net.Http.HttpConnection.SendAsyncCore(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)    at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpClient.SendAsyncCore(HttpRequestMessage request, HttpCompletionOption completionOption, Boolean async, Boolean emitTelemetryStartStop, CancellationToken cancellationToken)    --- End of inner exception stack trace ---    --- End of inner exception stack trace ---    at System.Net.Http.HttpClient.SendAsyncCore(HttpRequestMessage request, HttpCompletionOption completionOption, Boolean async, Boolean emitTelemetryStartStop, CancellationToken cancellationToken)    at Firebase.Database.Http.HttpClientExtensions.GetObjectCollectionAsync[T](HttpClient client, String requestUri, JsonSerializerSettings jsonSerializerSettings)    --- End of inner exception stack trace ---    at GrammarNazi.Core.Repositories.FirebaseRepository1.ExecuteFirebaseQuery[TResult](Func1 action, String callerMemberName) in /src/GrammarNazi.Core/Repositories/FirebaseRepository.cs:line 115    at GrammarNazi.Core.Repositories.FirebaseRepository1.GetAllItems() in /src/GrammarNazi.Core/Repositories/FirebaseRepository.cs:line 92
at GrammarNazi.Core.Repositories.FirebaseRepository1.GetFirst(Expression1 filter) in /src/GrammarNazi.Core/Repositories/FirebaseRepository.cs:line 64
at GrammarNazi.App.HostedServices.DiscordBotHostedService.GetChatConfiguration(SocketUserMessage message) in /src/GrammarNazi.App/HostedServices/DiscordBotHostedService.cs:line 189
at GrammarNazi.App.HostedServices.DiscordBotHostedService.OnMessageReceived(SocketMessage socketMessage) in /src/GrammarNazi.App/HostedServices/DiscordBotHostedService.cs:line 88
at GrammarNazi.App.HostedServices.DiscordBotHostedService.b__8_0(SocketMessage eventArgs) in /src/GrammarNazi.App/HostedServices/DiscordBotHostedService.cs:line 75

Firebase was down, now seems to be up. I'm considering migrating the database...


Great job

I am working with Vaadin and OSGI for many years now.
And this is the best implementation about concerns I now!
Congrats ;)
I think your impl should become the "Vaadin OSGI addon".
Best florian

May I propose some ideas about becoming "Vaadin OSGI addon"?


Port randomization fixes

There is an issue wrt. destination port randomization that conflates subtly different booleans controlling whether port randomization is enabled for a given connection.

[x] Port randomization consistency for Unidirectional Registration - (w/ and w/o phantom support for randomization)
[x] Port randomization consistency for Bidirectional Registration - (w/ and w/o phantom support for randomization)

it is not clear that the proper param object is being forwarded to the station to ensure that the client and station always agree on the expected destination port.

[x] Copy Params from pointer instance instead of using provided pointer instance in set Params otherwise issues with values changing between connections causes unpredictable behavior.
XX IPv6 Phantom Selection bug that allows selection of IPv4 phantom when the number of IPv4 phantoms is sufficiently small.

XX changing this requires a new client Library version as it changes the phantom selection and we will need to support the old version still in order to be backward compatible.

Sessions using bidirectional registrars do have consistency between the client and the station wrt port randomization as  regprocessor.go -- which handles the intake of bidirectional registrations, and generating the RegistrationResponse that is both sent to the client and attached to the registration forwarded to the -- handles port randomization properly. When a bidirectional registrar is used the stations use the port in the attached registration response which the client  is supposed to use as well.
When picking the destination port the regprocesser uses the station side transport interface and provides the transport parameters which is where the client indicates whether port randomization is enabled or not.

It turns out that making a copy of the params shared on init was not enough for item 3 because subsequent dials would still overwrite parameters within the Transport object if it is re-used, which we allow it to be.
The real solution was to add a public Parameters that is set by the user, which should be treated as immutable, unless it is explicitly set again using SetParams(). And then use an mutable internal parameter object that deep clones  the user parameters on each dial. In this way internal state only persists for one session and each dial will start with consistent parameters matching caller expectations.
For things like bidirectional registration where it is necessary to overwrite some parameters within the session the function SetSessionParams() is used.

In general I think this maybe motivates a different interface for Transports more similar to Tor's pluggable transports where you have a Transport, but then when dial is called a function like transport.ClientFactory() is called to build a new instance of the implemented protocol with/from the original immutable config.

The 4th todo is not directly related to these fixes and should be handled separately as it is not required to get the fixed port randomization rolled.  See #240


Ion-tabs when scrolling down the screen does not hiding navbar in iOS Safari

HTML structure is as follows:
ion-view
ion-content
ion-tabs
ion-tab ...
The problem is that the project is necessary when scrolling down to hide the upper and lower standard panels Safari. But they did not hide, as I believe is happening inside the tab scrolling and Safari does not understand what is happening to scroll the screen, thereby hiding the standard browser panel. Perhaps someone faced with this problem?

Greetings @rubinchyk!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Hello! Thanks for opening an issue with us, we just need a little more info from you so that we can better help you with your issue. Would you mind posting a code sample, plunkr or codepen that we can use to reproduce your issue? Also, would you mind running ionic info and posting the output? Thanks again!

Hello! Thanks for opening an issue with us! Since it has been a little while since there was any activity on this issue i will be closing it, but feel free to reopen this issue if needed!


Allow a string value as a default for a Decimal field

A simpler alternative to #7.

Nice work :shipit:


Clean up tinkerbell stack

Description of changes:
Clean up the stack based on some of the comments on this PR https://github.com/aws/eks-anywhere/pull/2274
Testing (if applicable):
Created a cluster and verified that the changes didn't break anything
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


blueprints.orientdb.lightweightEdges is false by default

docs say it should be true by default http://orientdb.com/docs/last/Graph-Database-Tinkerpop.html#using-indices
https://github.com/orientechnologies/orientdb/blob/master/graphdb/src/main/java/com/tinkerpop/blueprints/impls/orient/OrientConfigurableGraph.java#L35

Thanks, fixed.


I have a question

I couldn't find anything like getLength.
I'd like to know the length of the effect.
Is there a way?
thank you

There is  no way. Because a effect can use random to change a length.


merging exceptions in one package

moved exceptions in emc-metalnx-services to emc-metalnx-core

Invalid pull request to master. It shoud be merging into development.


Add options to KeyboardDragListener to override movement direction keys

For components that should use KeyboardDragListener but are constrained to 1D motion it would be helpful to specify the keys for each direction like
rightMovementKeys: [ KeyboardUtils.KEY_RIGHT_ARROW, KeyboardUtils.KEY_D, KeyboardUtils.KEY_W, KeyboardUtils.KEY_UP_ARROW ]

It is very difficult (impossible?) with the current API to override behavior of certain movement keys.

@pixelzoom suggested

It might even be better to have:

type KeyboardDragDirection: 'leftRight' | 'upDown' | 'both';

// KeyboardDragListenerOptions
keyboardDragDirection?: KeyboardDragDirection

// default option value
keyboardDragDirection: 'both'

and then standardize how those behave.  And option like rightMovementKeys would not be standardized.

Just so I am clear, KeyboardDragListener is emplyed for custom draggable objects like the Yellow and Green Balloons in Balloons and Static Electricity, the Chemistry Book in Friction, and the Bar Magnet in Faraday's Law, the Both Hands interaction in Ratio and Proportion, etc. AND, more generally, it is used for other draggable interactive objects. Is that the case?

@terracoda that is correct, it implements dragging with a keyboard. As of right now, if it isn't an <input type="range"> (using AccessibleSlider) in the PDOM then it uses KeyboardDragListener.

Done in the above commit, @pixelzoom would you mind reviewing and testing in your use case?

I used the new keyboardDragDirection option in ph-scale, and it's working nicely. See the above commit.
The implementation looks nice. It has the added benefit of not allocating an array on on every call to updatePostion.
@jessegreenberg I do see one unrelated issue, not a new problem.  In updatePosition:
      if ( this.leftMovementKeysDown() ) {
        deltaX = -delta;
      }
      if ( this.rightMovementKeysDown() ) {
        deltaX = delta;
      }
      if ( this.upMovementKeysDown() ) {
        deltaY = -delta;
      }
      if ( this.downMovementKeysDown() ) {
        deltaY = delta;
      }

Shouldn't these be else if?  Or is it necessary for some reason to call all 4 cases every time?  Keep in mind that this is in critial code, where a user interaction is being processed.

And thanks for the quick turnaround!

Yes, that is better thanks! How is https://github.com/phetsims/scenery/commit/b8a6f884e6b5278fa444f40809d1574c869d82c5?

👍🏻 closing.


CrateFile2 is not usable on Windows 7

Encountered in https://github.com/gluon-lang/gluon/issues/777 . It appears that https://github.com/ogham/rust-ansi-term/commit/23456d5fd32179858a827fd4db4861a624e302a2#diff-0d46108ce4a3c312f023e721b094d40c makes ansi_term no longer work on Windows 7.
This might not be something that you want to support but I figure I should report it.
I am working around it by simply not including ansi_term on Windows

Fixed in #52.

PR#52 is merged, so this should be fixed in master (now released as v0.12.1).


rework treatment of extra compiler flags

Use BLT__FLAGS{_<DEBUG|RELEASE>}

:+1: looks good to me!

One subtle issue folks may miss:
CMake uses "Fortran" vs "FORTRAN for its settings vars used to configure Fortran (including compiler flags)
For example:
CMAKE_Fotran_FLAGS vs CMAKE_FORTRAN_FLAGS
For BLT, I think its good to be consistent with using all caps for our variable settings, so I used:
BLT_FORTRAN_FLAGS
BLT_FORTRAN_FLAGS_DEBUG
BLT_FORTRAN_FLAGS_RELEASE

I like that, however, people still need to remember to enable Fortran as a language, rather than all caps?

@davidbeckingsale  we have "ENABLE_FORTRAN" as our option, however they do need to set CMAKE_Fortran_COMPILER ...

That's fine, I think. I was more getting at using project(blah LANGUAGES Fortran) to tell CMake that the project needs Fortran. Do you think it's worth adding the corresponding enable_language call in SetupCompilerOptions in case the user hasn't done it already?

Yes, I'll add something like this:
################################
# Invoke CMake Fortran setup
# if ENABLE_FORTRAN == ON
################################
if(ENABLE_FORTRAN)
    enable_language(Fortran)
endif()

I thought it was already there, but I was confusing with another cmake setup.

Hadn't considered it, but purpose of ENABLE_FORTRAN allows us to optionally enable fortran -- so we can guard fortran features when the compiler doesn't exist.
If you have a fortran only project, I think you can avoid C and CXX with your own project statement:
project(myname Fortran)
However you will sill have to set ENABLE_FORTRAN = ON to trigger the right blt paths.
Don't know of a use case where we optionally want C and CXX? Maybe we can revisit ...

We were just dealing with a situation recently, where, depending on the platform, we might need some additional linker flags.
I think we should also have user-supplied BLT_LINKER_FLAGS -- I would think that these need to be language dependent (e.g. BLT_C_LINKER_FLAGS, BLT_CXX_LINKER_FLAGS, ... ).
Should this be part of the same PR, or in a new PR?

@kennyweiss  can we tackle linker stuff in a new pr?

Sure -- let's handle the linker flags separately.

Do we need the conditionals checking for the current build type for every set of flags?
I think we could always add BLT_C_FLAGS_DEBUG to CMAKE_C_FLAGS_DEBUG (for example) regardless of build type, since they will only be used based on the type of the current build.

@davidbeckingsale . yes -- good catch, we only needed those when we were doing logic to combine debug and release flags for the RelWithDebInfo case.
I however keep the guard for (NOT CMAKE_CONFIGURATION_TYPES) to avoid doing this logic in visual studio

Sounds good!

@kennyweiss  updated to use a loop over the types as you suggested

@kennyweiss @davidbeckingsale  can you give a quick review again? I tested the paths a bit manually, but another set of eyes would be good.

Looks good (based on a visual inspection)

Yeah, looks good to me!

thanks!


Update SETUP.md

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
The docker-compose command needs to be updated to V2
Steps To Reproduce
Go to - https://github.com/CircuitVerse/CircuitVerse/blob/master/SETUP.md
Expected Behavior

using docker compose instead of docker-compose

Screenshots
it might give error while trying to setup the project locally if using docker-compose

Device Information [optional]
- OS:
- Browser:
- version:

Additional context
No response
Are you working on this issue?
Yes

If approved, Can i work on this?

If we try to set up the project using Gitpod this command work fine. If you try this command in any terminals it will show error like this. As they mention this command usage in the Gitpod set up only there is nothing wrong in that I guess !!!!

If we try to set up the project using Gitpod this command work fine. If you try this command in any terminals it will show error like this. As they mention this command usage in the Gitpod set up only there is nothing wrong in that I guess !!!!

But that's not under Gitpod, it's under Docker

Could you please let me know on which terminal and OS version you are doing ??

Could you please let me know on which terminal and OS version you are doing ??

I am working on Linux

the hyphen matters.
The docker-compose up was the command for version 1 (v1), which is deprecated, and docker compose up is for version 2 (v2).Also, you can read more about it here. It's mentioned at the very top of the page. I am not sure about this DO cross check once. Thankyou !!
https://docs.docker.com/compose/


Update README with learning goals

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

yo


Docs: Popover Styling ClassName

I had to review the source code for Popover to figure out that className is passed via statelessProps, I realise that there's not really much wrong with that but it's at best faux-intuitive without any mention in documentation.
DOCS:
https://evergreen.segment.com/components/popover/
SRC:
https://github.com/segmentio/evergreen/blob/37d141457a82297fc2de14b252c68fa1a5e965f1/src/popover/src/Popover.js#L382

Bigger issue:
The following works and adds class:
  <Popover
    statelessProps={{
      className: "TEST"
    }}

This CSS module via JS does not add any class to Popover :(
    statelessProps={{
      className: layout.popMenu
    }}

Going to compare to another implementation that is working for me.
So far I have found that:
Popover passes statelessProps (where className is provided) to PopoverStateless.
PopoverStateless takes its propTypes from layers/Card
layers/Card takes its propTypes from Pane
Pane takes its propTypes from Box (LOWEST LEVEL FOR ASSIGNMENT)
The the lowest level for passing className
// /src/layers/src/Pane.js
const className = cx(
    props.className,
    glamorCss({
      ...css,
      ...hoverElevationStyle,
      ...activeElevationStyle
    }).toString()
  )

  return (
    <Box
      ..............
      {...props}
      className={className}
    />
  )

This works, I'm going leave another comment after this one comparing the lowest level className assignment for Icon versus Popover (Box?)

Okay, so:
Icon extends PureComponent but it does not have any props passed using the spread operator (which includes the className from Box in the case of Popover - I believe)
Button also works for passing a className, on button it has a className props type on its own class (non-inheritance)
https://github.com/segmentio/evergreen/blob/37d141457a82297fc2de14b252c68fa1a5e965f1/src/buttons/src/Button.js#L79
There is some fiddling with it regarding themedClassName but this obstructs nothing for me.
https://github.com/segmentio/evergreen/blob/37d141457a82297fc2de14b252c68fa1a5e965f1/src/buttons/src/Button.js#L126
Then className assignment to component is quite straightforward:
https://github.com/segmentio/evergreen/blob/37d141457a82297fc2de14b252c68fa1a5e965f1/src/buttons/src/Button.js#L163

I haven't done any testing but would adding the following propType to Popover and escaping statelessProps (or the same thing within statelessPopover instead) allow the className to be passed?
https://github.com/segmentio/evergreen/blob/37d141457a82297fc2de14b252c68fa1a5e965f1/src/buttons/src/Button.js#L79

Another difference would be that the css module is usually passed as
className={MODULE.RULENAME}
Whereas for popover to pass to statelessProps it has to be within an object, surely this should not matter? (Need to do some console.logging)
statelessProps={{
      className: MODULE.RULENAME
}}

Updated evergreen and its working now 🤷🤷♂🤷🤷♂🤷🤷♂🤷🤷♂🤷🤷♂🤷🤷♂🤷🤷♂🤷

@xAvarice do you know which version you were on before?


Plugin events are never fired

[x] I am on the latest Hyper.app version

[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: macOS Sierra 10.12.1

Hyper.app version: <IP_ADDRESS>8

Link of a Gist with the contents of your .hyper.js: Here

Relevent information from devtools (CMD+SHIFT+I on Mac OS, CTRL+SHIFT+I elsewhere):

(re)loading renderer plugins
index.js:5 [hyper-expose-app] Plugin loaded
index.html:34 total init time<PHONE_NUMBER>0000003
bundle.js:32264 Unimplemented DEC Private Mode: 1034

The issue is reproducible in vanilla Hyper.app: One local plugin

Issue
It seems that the onApp and onWindow events are never getting fired. The full console output is above.

I'm also having this issue. Any more information or workarounds available?
OS version and name: macOS Sierra 10.12.3
Hyper.app version: <IP_ADDRESS>4

The lack of these events firing really prevents the ability to meaningfully extend Hyper. I've also tried disabling all plugins and get the same behavior.

OS version and name: Windows 10
Hyper.app version: 1.3.1

Im having this issue as well.
Hyper.app version: Hyper 1.3.3
OS Version and name: Ubuntu 17.04

exports.decorateConfig = config => {
...
}```
is also not getting fired for me

Upgraded to Hyper 1.4.3 (stable). Still can't get onApp events.

Hyper 2.0.2 still does not fire onWindow.

Let's fix it @LoganDark

"Let's"? Meaning me?
How am I supposed to fix it? I don't even know how Hyper works.

I disable all plugins, even deleted the local plugin folder, remove .hyper.js, re-install the fresh one, still get this issue.
macOS 10.12.6
node 8.6.0

The devs might as well remove this "feature" from the documentation since it hasn't worked in 10 months and over 20 releases.

At least it works in dev mode... https://github.com/zeit/hyper/issues/3013

Maybe it is the bundling process that blocks onWindow to fire.

Sometimes it works! But when I try to add a new feature it suddenly doesn't work.

Same issue, i am trying to create an autocomplete for hyper, but the onWindow event is not called on plugin


404 Everywhere

I deploy NOJ as the docs said, just change .env file for database.
But finished all steps, I can noly visit the index page, when I click the label and visit other pages, it returns 404
I use bt panel to install some environment such as PHP, Mysql, Redis, is it wrong?
Platform:
Ubuntu 18.04 (tencent CVS)
Mysql 5.7
PHP 7.3
Redis 6.0

Looks like you are having a rewrite problem.

Try using .htaccess for Apache or .conf for Nginx, here is one rewrite example:
location / {
    try_files $uri $uri/ /index.php?$query_string;
}

Here is one Apache .htaccess (NOJ has that included) example:
Options +FollowSymLinks -Indexes
RewriteEngine On

RewriteCond %{HTTP:Authorization} .
RewriteRule .* - [E=HTTP_AUTHORIZATION:%{HTTP:Authorization}]

RewriteCond %{REQUEST_FILENAME} !-d
RewriteCond %{REQUEST_FILENAME} !-f
RewriteRule ^ index.php [L]

The ultimate goal is to redirect the majority of the requests to index.php.

If your problem persists, comment below.

Problem resolved, thanks!


Reduce polyfill.js or remove all together

I have noticed you are importing the polyfill.js into this package which bloats the file size when you use it in a project. I would much prefer giving people the option to use their own polyfills if they need them for their projects.
At least remove the promise polyfill which adds 10kb. Add to the readme about the need to use your own promise polyfill.
side note
Ideally there will be better support for having ES5 and ES2015 npm modules together. Something like rollup has. So once webpack v2 is released then we can all do tree-shaking.

@andykenward, thanks for the suggestion.
Will work on removing them.


APEXMALHAR-2075 enhance jdbcinputpojooperator to support date/time/ti…

…mestamp fields in POJO

Hello @sandeepdeshmukh @bhupeshchawda @DT-Priyanka  Could you please review.

If there are no further comments, I will merge this in a day.


Fix for depreciated method and corrected status variable

This method is being depreciated this month.
http://googleadsdeveloper.blogspot.com/2015/05/announcing-v201505-of-dfp-api.html
Also, data.rval.reportJobStatus returns undefined. data.rval now returns what we're looking for.

Thanks!


feat: get model modes and detect norag mode.

fix #33
It requires the albert-api version >= 02b989d to support the new /models endpoints.
PS: Just a remark, but !sources could be present event in a norag mode. It will just show the semantic search results associated to the last query, which can be usefull in some scenario.

LGTM!

@bolinocroustibat just a hint ti be sure, but the albert-api staging needs to be rebuilt after my last patches. I say that because  I am not sure it was done in the last deployement as we just rebuilt the chunks.


Select requires props onFocus, onBlur

🐛 Bug Report
The select component now requires onFocus and onBlur properties in version 4.4.0.
To Reproduce
Steps to reproduce the behavior:
<Select              data={this.programOptions}              label="Program"              onSelect={option => this.onSelect(option)}  />
Expected behavior
Before version 4.4.0 onFocus and onBlur were not required by the Prop types for the select component. onFocus and onBlur should be optional props.
Link to runnable example or repository (highly encouraged)
😇
UI Kitten and Eva version

Package
Version

@eva-design/eva
1.3.0

@ui-kitten/components
4.4.0

Environment information
react-native run-ios

I meet the same i end up by overriding selectui-kitten  SelectProps: type MySelectProps = Omit<SelectProps, 'onFocus' | 'onBlur'>

The fix is already present in master https://github.com/akveo/react-native-ui-kitten/pull/877 and will be included in the next release

@artyorsh thank you!


fixed image not centering issue for mobile devices in scenes.html page

Fix for Issue #135

@all-contributors please add @Shubhangam333  for code, design, and ideas


Quantified constraints for Show

You can create a show instance for AST sym sig, if you if you say that forall sig. Show (sym sig).
That can be achieved by Show1 or ForallF Show:
instance ForallF Show dom => Show (AST dom sig) where
  showsPrec p = \case
    Sym s ->
      showParen (p >= 11) (showString "Sym " . showsPrec 11 s) \\
        instF @Show @dom @sig
    f :$ x ->
      showParen (p >= 10) (showsPrec 10 f . showString " :$ " . showsPrec 10 x)

Now any symbol domain that has a show instance parametric in its semantic type can be shown, are there any benefits to doing it like this?
data NUM a where
  NUM :: Int
      -> NUM (Full Int)
  ADD :: NUM (Int :-> Int :-> Full Int)
  MUL :: NUM (Int :-> Int :-> Full Int)
deriving instance Show (NUM a)

>>> Sym ADD
Sym ADD
>>> Sym ADD :$ Sym (NUM 1)
Sym ADD :$ Sym (NUM 1)
>>> Sym ADD :$ Sym (NUM 1) :$ Sym (NUM 2)
(Sym ADD :$ Sym (NUM 1)) :$ Sym (NUM 2)

Nice! Where is ForallF defined?

It's defined in the constraints package (Data.Constraints.Forall) which is already a dependency of syntactic.

Eq1, Show1, ... just got added to base in the GHC 8 release: to Data.Functor.Classes and they are very similar, compare
class Eq1 f where
    liftEq :: (a -> b -> Bool) -> f a -> f b -> Bool

class Equality e where
    equal :: e a -> e b -> Bool

ForallF Eq would not allow comparing elements of different argument types, so I'm not sure how this all fits together to be honest (obviously Eq1 doesn't have a hash :: e a -> Hash method and Show1 doesn't have renderArgs) but I thought I'd mention it


Linking the library with a React-Native module

Thank you so much for the good job of compiling the library making it very easy to use in a project.
I have struggling with an issue the last three days.
I am building react-native module to consume in a react-native project using https://github.com/brodybits/create-react-native-module
Am not sure if am placing the "include" directory in the right place.
am getting the error when i import lbtdjson in a .swift file.
: error: no such module 'libtdjson'
Your help will be highly appreciated.

Hi, I wonder you didn't create the module_map file for swift. Please check this: https://github.com/up9cloud/ios-libtdjson#use-it-as-module-ios-swift

Thank you so much for your reply.
Actually i did, and it works well when i use it directly in my react-native application.
But when i try to extract it on an npm module that i can easily install, Im stuck. I really dont know what am doing wrong.
/Users/bernardgaitho/tests/cJamboTest2/node_modules/react-native-tdx/ios/TdJsonClient.swift:10:8: error: no such module 'libtdjson' import libtdjson

Well, I guess it's the same thing. after you installed the pod, you need to create the module file and configure it for swift. I don't think "create-react-native-module" will do it for you.
Did you check the pod folder which create-react-native-module installed for your project?

I meant the create-react-native-module is actually doing installing pod and wrapping. You have to check the pod folder  they installed for you.

In my example project, https://github.com/gaithoben/react-native-tdx.git, i have adjusted the .podpecs file so that the library is installed in the projects 'ios/Pods/libtdjson' folder.
I guess configuring at the module is whats giving me a headache.

Your help will be highly appreciated.

In my example project, https://github.com/gaithoben/react-native-tdx.git, i have adjusted the .podpecs file so that the library is installed in the projects 'ios/Pods/libtdjson' folder.
I guess configuring at the module is whats giving me a headache.

I think there are 2 ways to solve this:

to figure out why -l"tdjson" -fmodule-map-file="${PODS_ROOT}/Headers/Public/libtdjson/libtdjson.modulemap" didn't work as expectation. (have to check the .xcconfig)
download the libs and build it as a xcframework then manually add it to your lib in xcode (you might want to try https://github.com/leoMehlig/TDJSON/releases/tag/v1.6.1 )

Thank you for your reply. Let me test your suggestion and i will keep trying and researching...
I have never built a native project in native swift or objective c, i guess thats the reason why i find it difficult.
It was not difficult to make it work in android. But to be honest this is not easy for me in iOS, if you can spare sometime and make a working example package for react-native, i will be ready to facilitate that.  My email is gaithoben \at \gmail .com.
Thanks in advance.


Upperbound version restrictions for Flutter SDK are ignored

Steps to Reproduce

Specify a build environment for flutter with an upperbound version restriction in the project's pubspec.yaml.

environment:
  sdk: ">=2.13.0 <3.0.0"
  flutter: ">=2.2.0 <2.3.0"

Run flutter pub get to reload the pubspec.yaml using Flutter 2.2.2.
In pubspec.lock the upperbound version is now recorded differently:

sdks:
  dart: ">=2.13.0 <3.0.0"
  flutter: ">=2.2.0"

Now, when upgrading Flutter to version 2.5.1 (stable) and running flutter pub get again, no error message is visible that the build environment does not match.

Logs

Logs

$ flutter pub get
Downloading Dart SDK from Flutter engine 91c9fc8fe011352879e3bb6660966eafc0847233...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  199M  100  199M    0     0  20.9M      0  0:00:09  0:00:09 --:--:-- 28.8M
Building flutter tool...
  ╔════════════════════════════════════════════════════════════════════════════╗
  ║ A new version of Flutter is available!                                     ║
  ║                                                                            ║
  ║ To update to the latest version, run "flutter upgrade".                    ║
  ╚════════════════════════════════════════════════════════════════════════════╝

Downloading package sky_engine...                                  355ms
Downloading flutter_patched_sdk tools...                           249ms
Downloading flutter_patched_sdk_product tools...                   223ms
Downloading linux-x64 tools...                                   1,183ms
Downloading linux-x64/font-subset tools...                         149ms
Running "flutter pub get" in project...                       1,121ms
$ flutter upgrade
Upgrading Flutter to 2.5.1 from 2.2.2 in /home/ivar/Android/flutter...
Downloading Dart SDK from Flutter engine b3af521a050e6ef076778bcaf16e27b2521df8f8...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  2  221M    2 5518k    0     0  10.1M      0  0:00:21 --:--:--  0:00:21 10.1M
 14  221M   14 31.4M    0     0  20.5M      0  0:00:10  0:00:01  0:00:09 20.5M
 26  221M   26 58.3M    0     0  23.0M      0  0:00:09  0:00:02  0:00:07 23.0M
 38  221M   38 84.6M    0     0  23.9M      0  0:00:09  0:00:03  0:00:06 23.9M
 50  221M   50  111M    0     0  24.5M      0  0:00:09  0:00:04  0:00:05 24.5M
 60  221M   60  133M    0     0  24.1M      0  0:00:09  0:00:05  0:00:04 25.6M
 72  221M   72  159M    0     0  24.4M      0  0:00:09  0:00:06  0:00:03 25.6M
 83  221M   83  184M    0     0  24.4M      0  0:00:09  0:00:07  0:00:02 25.2M
 93  221M   93  205M    0     0  23.8M      0  0:00:09  0:00:08  0:00:01 23.7M
100  221M  100  221M    0     0  24.0M      0  0:00:09  0:00:09 --:--:-- 23.5M
Building flutter tool...

Upgrading engine...
Downloading package sky_engine...                                  376ms
Downloading flutter_patched_sdk tools...                           297ms
Downloading flutter_patched_sdk_product tools...                   248ms
Downloading linux-x64 tools...                                   1,274ms
Downloading linux-x64/font-subset tools...                         351ms

Flutter 2.5.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision ffb2ecea52 (9 days ago) • 2021-09-17 15:26:33 -0400
Engine • revision b3af521a05
Tools • Dart 2.14.2

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.1, on Ubuntu 20.04.3 LTS 5.4.0-81-generic, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.2)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.1)
[✓] Connected device (1 available)

• No issues found!
$ flutter pub get
Running "flutter pub get" in project...                       1,808ms
The plugins `flutter_jailbreak_detection, flutter_privacy_screen` use a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to see if these plugins support the Android V2 embedding. Otherwise, consider removing them since a
future release of Flutter will remove these deprecated APIs.
If you are plugin author, take a look at the docs for migrating the plugin to the V2 embedding: https://flutter.dev/go/android-plugin-migration.

[✓] Flutter (Channel stable, 2.2.2, on Linux, locale en_US.UTF-8)
    • Flutter version 2.2.2 at /home/ivar/Android/flutter
    • Framework revision d79295af24 (4 months ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/ivar/Android/Sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /home/ivar/Android/Sdk
    • Java binary at: /snap/android-studio/101/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /snap/android-studio/101/android-studio
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.2)
    • IntelliJ at /snap/intellij-idea-ultimate/311
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.4746.57

[✓] IntelliJ IDEA Ultimate Edition (version 2021.1)
    • IntelliJ at /snap/intellij-idea-ultimate/289
    • Flutter plugin version 55.0.5
    • Dart plugin version 211.6693.108

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 94.0.4606.61

• No issues found!

@mit-mit @csells Do we have documentation about this behavior anywhere?

The documentation of the SDK version constraints can be found here.
I also tried to specify a specific SDK version, like use exactly version 2.2.2, but that one is also converted to >= 2.2.2.

Hi @ivard
I can reproduce the issue in Fluttter. However, this also reproduces in a dart project with Flutter SDK and dart pub get
dart create hello
name: hello
description: A simple command-line application.
version: 1.0.0
# homepage: https://www.example.com

environment:
  sdk: '>=2.12.0 <3.0.0'
  flutter: ">=2.2.0 <2.3.0"

# dependencies:
#   path: ^1.8.0

dev_dependencies:
  lints: ^1.0.0

# Generated by pub
# See https://dart.dev/tools/pub/glossary#lockfile
packages:
  lints:
    dependency: "direct dev"
    description:
      name: lints
      url: "https://pub.dartlang.org"
    source: hosted
    version: "1.0.1"
sdks:
  dart: ">=2.12.0 <3.0.0"
  flutter: ">=2.2.0"

This looks like an intended behavior, these changes are made in https://github.com/dart-lang/pub/pull/2780 and https://github.com/dart-lang/pub/issues/2696
Change

That's what my question above was regarding. It should certainly be documented, it's just a question of where, and if it already is somewhere.

Was there a special reason why it was deprecated? We see a lot of contributers struggling with finding the right Flutter version to build our project. Having a newer SDK version of Flutter might also lead to issues, for example if a dependency has no support for the newer Flutter version yet.
Of course we can also solve this by mentioning it in the project's README, so it is not strictly necessary. However, enforcing a Flutter SDK version in the pubspec.yaml appeared to me as a nice extra way to help people setting up a project.

This looks like the tracking issue with context: https://github.com/flutter/flutter/issues/68143

Having a newer SDK version of Flutter might also lead to issues, for example if a dependency has no support for the newer Flutter version yet.

Flutter, unlike the rest of the ecosystem, does not follow semver, so there is no way of setting a meaningful upper constraint in advance. Is the use case here an actively maintained project that doesn't support current versions of Flutter?

Yes right. In our case it concerns a mobile app. We get issues like this one, because people are using the wrong SDK version.
This issue was also related to the fact that we provided an outdated Makefile, but even then a clear error that someone is using a wrong SDK version would help.

We are also running into this issue. Flutter 3.8.0-0.0.pre introduced a breaking change for us.
Before Flutter 3.8.0-0.0.pre we could call WidgetsBinding.attachRootWidget like this:
binding.attachRootWidget(MyApp());

However, as of Flutter 3.8.0-0.0.pre, we have to call it like this:
binding.attachRootWidget(binding.wrapWithDefaultView(MyApp()));

I reported this issue. See #121180.
Using an upper bound constraint on the flutter sdk version would have been useful. I wanted to ship a version of our package using an upper bound on the flutter sdk, like this:
name: monarch
version: 3.0.1
...

environment:
  sdk: '>=2.12.0 <4.0.0'
  flutter: '>=2.12.0-4.1.pre <3.8.0-0.0.pre'

And then another version of our package using 3.8.0-0.0.pre as the lower bound:
name: monarch
version: 3.3.0
...

environment:
  sdk: '>=2.12.0 <4.0.0'
  flutter: '>=3.8.0-0.0.pre'

I expected the upper bound constraint to work. However, it is ignored. Users that upgrade to Flutter 3.8 may end up using an incompatible version of our package.

I wanted to ship a version of our package using an upper bound on the flutter sdk, like this: [...]
And then another version of our package using 3.8.0-0.0.pre as the lower bound:

The upper bound wouldn't really do anything useful in that scenario; package resolution will give the highest viable version, so clients on a version compatible with your 3.3.0 would get that, and everyone else would get 3.0.1, even without the upper bound.

Users that upgrade to Flutter 3.8 may end up using an incompatible version of our package.

That's true regardless. Presumably you had at some point in the past released a version 3.0.0 without an upper bound, which someone updating to Flutter 3.8.0 can continue to use regardless of what else you release. To prevent that you would need retroactive constraints.
Which is exactly why the current behavior exists: you had no way to know when you published 3.0.0 what upper bound it would need, because Flutter doesn't use semver, which makes upper bounds useless in packages.

That's true, but if people download the Flutter SDK, they always get the latest stable version. This means that if a project is not compatible with that version yet, they will get into trouble. There is no nice way to communicate this now. The only thing we can do is to mention the version issue in a README and then hope that people see it.

@ivard I was responding to a comment about a package; your use case is an app, which as I said above is very different.

@stuartmorgan:

The upper bound wouldn't really do anything useful in that scenario; package resolution will give the highest viable version, so clients on a version compatible with your 3.3.0 would get that, and everyone else would get 3.0.1, even without the upper bound.

I see what you are saying. That makes sense now. I don't need an upper bound in this scenario.

... because Flutter doesn't use semver...

What do you mean by that? It certainly looks like Flutter uses semver. Also, is it ok to use the package pub_semver to compare Flutter versions?

... because Flutter doesn't use semver...

What do you mean by that?

I mean that Flutter versioning doesn't follow the requirements of semver. Most importantly in this context, it doesn't follow this one:

Major version X (X.y.z | X > 0) MUST be incremented if any backwards incompatible changes are introduced to the public API.

which is why there's no way to ever set a useful upper bound for Flutter in advance, only after the fact. Unlike ecosystem packages that follow semver, where (in theory at least), setting a ^ range on packages avoids later incompatibility.

It certainly looks like Flutter uses semver.

If Flutter followed semver, only Flutter 2 and Flutter 3 would have any entries here.

+1 for the app use case, specifically setting an exact version.

Have run into issues where a teammate pulls in newer version w breaking change.
Want to ensure builds use an exact sdk version
sdk version upgrades are intentional.

pubspec seems the best place to specify this, without introducing extra tooling.

Hope a solution comes through soon. I just encountered this issue, where even after specifying sdk: '>=2.12.0 <4.0.0' for using leveldb: ^7.0.0, I get the error The current Dart SDK version is 3.4.1. Because timers_cascade depends on leveldb >=2.0.1 which doesn't support null safety, version solving failed. The lower bound of "sdk: '<2.0.0 or >=2.0.0 <3.0.0'" must be 2.12.0 or higher to enable null safety..
Looking forward to a day when Users would no longer have to specify such things, and an AI would just select and download the necessary dependencies.


cwltoil.py fails in resolve_indirect_inner for Workflow if substep is a workflow and some inputs are optional and empty

I have a Workflow job that uses workflow for the substep. If some inputs for the substep are optional and empty, it fails resolve_indirect_inner() in cwltoil.py:
def resolve_indirect_inner(d):
    if isinstance(d, IndirectDict):
        r = {}
        for k, v in d.items():
            if isinstance(v, MergeInputs):
                r[k] = v.resolve()
            else:
                r[k] = v[1][v[0]]
        return r
    else:
        return d

The problem is at Line 103:
         r[k] = v[1][v[0]]

Thank you for this fix @liuxf09


Fix for #129. Added additional handling of LambdaExpression target.

Fixed regression with Invokation.
Fixes #129

Superseded by https://github.com/scottksmith95/LINQKit/pull/131


Allow non-exact dates

We have some data that has a non-exact date, e.g. for a composer, Born: circa 1860, Died: circa 1920.
This information should be able to be stored in the CE.

@alastair do you have a suggestion about how this data could be stored?

@musicog might have some ideas about how these kinds of dates are normally stored in linked data

Sorry for the delayed response, I missed this originally.
Unfortunately I don't have any satisfying response here -- approximate / uncertain dates are still somewhat an open problem for Linked Data; and it's a big problem, particularly in early publishing.
In a previous project around publishing the Early English Books Online (EEBO) corpus as Linked Data, we ended up defining our own precise- and exact- properties, e.g. eeboo:precise-birth and eeboo:precise-death for birth and death dates; see https://github.com/oerc-elephat/preprocessed-elephant/blob/master/eeboo-ontology.ttl
What makes it more difficult is that different data sets have a huge variety of ways of expressing precise and approximate dates -- here's the (large) set of regular expressions we wrote to capture them from the EEBO corpus (p == precise, a == approximate, r == range)
https://github.com/oerc-elephat/preprocessed-elephant/blob/master/regex-author-year.csv
https://github.com/oerc-elephat/preprocessed-elephant/blob/master/regex-publication-date.csv
The other approach is to cheat -- e.g., dbpedia does things like specifying "1551-0-0" to mean "sometime in the year 1551", see the death date for Richard Sherry on this page: http://dbpedia.org/page/Richard_Sherry
Ideally, this is something that we should think about carefully, with input from a (music) domain expert.

Duplicate #91

Not quite the same, #91 talks about "Born in 1790-04, but we don't know the day", and this one is "Born some time around 1790 (maybe some years before or after)", but I agree that they're closely related and we could track them in one ticket. If we do so, we should add notes from this ticket to the other one.


test(BehaviorSubject): add more tests for BehaviorSubject

Add tests that verify BehaviorSubject's behavior with multiple subscribers, also when it is
subscribed after it has completed.

Merged with 6d81d32... thanks @staltz. Great work.


Null pointers should not be dereferenced

This pull request is focused on resolving occurrences of Sonar rule squid:S2259 - Null pointers should not be dereferenced
You can find more information about the issue here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S2259
Please let me know if you have any questions.
Kirill Vlasov

Hi @kirill-vlasov , can you open a Jira for this issue.
Thanks a lot

I can create the JIRA for you, but you have to sign the CLA so that we can integrate your fix,

@vladmihalcea this fix can be merged without the CLA. Thanks!

I added HHH-10711 for this issue, and it can be integrated right away.

Applied PR on master.


Operator Streaming API in L4LB

JIRA: https://jira.mesosphere.com/browse/DCOS_OSS-5464
DC/OS PR: https://github.com/dcos/dcos/pull/6480

all dc/os integration tests have passed.