[BC] Feature request - Integration of the route /subscriptions

https://developer.bigcommerce.com/api-reference/storefront/storefront-subscriptions/subscription/postsubscriptions
(It seems that it is not available in the BC side for graphql (I'll search more and re-open it if needed)

Super easy to implement on your own. In framework/bigcommerce create api endpoint subscribe and make it somewhat similar to this
const { data } = await config.storeApiFetch('/v3/customers/subscribers', {       method: 'POST',       body: JSON.stringify(         {           ...(firstName && { first_name: firstName }),           ...(lastName && { last_name: lastName }),           email,         },       ),     })
I made name and last name conditional because BC API doesn't require them
Then just make a use-subscribe hook
`import { useCallback } from 'react'
import type { MutationHook } from '@commerce/utils/types'
import { CommerceError } from '@commerce/utils/errors'
import useSubscribe, { UseSubscribe } from '@commerce/customer/use-subscribe'
import type { SubscriberHook } from '../types/subscribe'
export default useSubscribe as UseSubscribe
export const handler: MutationHook = {
fetchOptions: {
url: '/api/subscribe',
method: 'POST',
},
async fetcher({ input: { email, firstName, lastName }, options, fetch }) {
if (!(email)) {
throw new CommerceError({
message:
'An email is required to subscribe',
})
}
return fetch({
  ...options,
  body: { email, firstName, lastName },
})

},
useHook: ({ fetch }) => () => {
return useCallback(
  async function subscribe(input) {
    const data = await fetch({ input })
    return data
  },
  [fetch]
)

},
}
`
For my project I made types for everything extra so thats why they are imported but you can take those out I guess if you need this functionality urgently. Also you will need to add it to the provider files and commerce folder

What is in your @commerce/customer/use-subscribe ? Do you have a working exemple somewhere ?

What is in your @commerce/customer/use-subscribe ? Do you have a working exemple somewhere ?
I wrote the code bellow, inspired from the use-signup.
import { useHook, useMutationHook } from '../utils/use-hook'
import { mutationFetcher } from '../utils/default-fetcher'
import type { HookFetcherFn, MutationHook } from '../utils/types'
import type { SubscriptionsHook } from '../types/subscriptions'
import type { Provider } from '..'

export type UseSubscribe<
    H extends MutationHook<SubscriptionsHook<any>> = MutationHook<SubscriptionsHook>
    > = ReturnType<H['useHook']>

export const fetcher: HookFetcherFn<SubscriptionsHook> = mutationFetcher

const fn = (provider: Provider) => provider.subscriptions?.useSubscriptions!

const useSubscribe: UseSubscribe = (...args) => {
    const hook = useHook(fn)
    return useMutationHook({ fetcher, ...hook })(...args)
}

export default useSubscribe

Yeah mine is super similar to yours. Unfortunately my only example is in a project I'm using for work so I can't share it.
import { mutationFetcher } from '../utils/default-fetcher'
import type { MutationHook, HookFetcherFn } from '../utils/types'
import type { SubscriberHook } from '../types/subscribe'
import type { Provider } from '..'

export type UseSubscribe<
  H extends MutationHook<SubscriberHook<any>> = MutationHook<SubscriberHook>
> = ReturnType<H['useHook']>

export const fetcher: HookFetcherFn<SubscriberHook> = mutationFetcher

const fn = (provider: Provider) => provider.subscribe?.useSubscribe!

const useSubscribe: UseSubscribe = (...args) => {
  const hook = useHook(fn)
  return useMutationHook({ fetcher, ...hook })(...args)
}

export default useSubscribe

This is what mine looks like. Just don't forget to create those types for the hook, the input, result, etc and add it to the provider

I understand, thanks for your help though.
Yes they are similar :/ I am facing the error triggered due to these lines I guess
const useSubscribe: UseSubscribe = (...args) => {
    const hook = useHook(fn)
    return useMutationHook({ fetcher, ...hook })(...args)
}

Where it says that TypeError: hook.useHook is not a function in framework/commerce/utils/use-hook.ts :/

Hey there! Thank you for opening this issue. We have decided to take Next.js Commerce in a new direction and will be closing out current PRs and issues due to this change. Please see this PR for more details: https://github.com/vercel/commerce/pull/966


is .net aspire available for vs 2022 community version?

Type of issue
Missing information
Description
[Enter feedback here]
is .net aspire available for vs 2022 community version?

Page URL
https://learn.microsoft.com/en-us/dotnet/aspire/get-started/aspire-overview
Content source URL
https://github.com/dotnet/docs-aspire/blob/main/docs/get-started/aspire-overview.md
Document Version Independent Id
1b7d0d68-ea99-ae77-0b30-0bafb0195096
Article author
IEvangelist

You need 17.9 preview 1:

https://learn.microsoft.com/en-us/dotnet/aspire/setup-tooling?tabs=visual-studio#install-net-aspire

Close as resolved.


Release builds for Scala 2.12

Hi! I'm currently upgrading my application and one of the dependencies that does not yet have published artifacts for Scala 2.12 is your scala-faker. Could you release a build for 2.12 please?

+1

Because apparently this isn't of any priority to the original author, I've published this package to my own repo based on the fork/pull request by Eldius:  . Hope someone finds it useful:
"qwiksense-misc" at "https://packagecloud.io/priv/7d7c2536054b6bb2b0ff747b23e415f2e86af5ceca4d1470/qwiksense/misc/maven2", // scala-faker
"se.qwiksen" %% "scala-faker" % "0.5-QWIKSENSE" % Test

Hi

Agreed it seems the original author doesn't have time to maintain this repo. I found the faker gem very useful in my ruby jobs so I'd like to see it continue to be useful in scala land.
I've merged all the outstanding PRs (#4, #6, #8, #9) to a fork at bitblitconsulting and done some general updating to the build to bring it from sbt-0.12 to sbt-0.13 standards.
I've also cut a 0.4 release and published on sonatype/maven central.
Ruby's faker has undergone a lot of developent since this repo was last maintained; my intention is to bring the new categories of fakes over here too - contributions and pull requests will be gladly accepted!

+1


Add a Composer script for autofixing PHP code style issues

I'm really not sure about that.
Won't take make contributors run this script, and then not pay attention to whatever the script changed, and just blindly commit?

Won't take make contributors run this script, and then not pay attention to whatever the script changed, and just blindly commit?

I don't see this as a real problem. Autofixing style issues is a real time saver, and independent of how someone makes some changes, they still need to read their code before (or when) they create a PR for these changes.
(Apart from that, in my experience phpcbf so far never has changed the code in a way that was a problem.)

I really have a mixed feelings about those "autofixed" solutions.
First of all, why is it even named "fixed"? Is there something broken, that requires fixing? I don't think so. What it does, is conforming the current codebase to the arbitrarily set rules. Shouldn't it be named conform-to-codestyle-rules then? Certainly not "fix"/"autofix". I hope you know what I mean? (I don't mean to be too bold).
Second of all, are you sure we should empose rules on the code, that are so hard to maintain that you require a script to follow them for you?
I mean, it's hard to follow the intention here. The point of the code-style/code-check is to find potential problems/bugs that humans failed to see. That's right, right? And then... we introduce a tool, that can be used by programmers to bypass the very same tool we introduced before, to "autofix" it. The tool of code-style/code-check, at least I respected, because it would allow me to find the mistakes I made. I could use it. It could help me make the library better. But the "autofixed" is there to fool me. It will hide any mistakes I could've done from me and edit the codebase according to the style. If there was indeed a real bug or a real mistake, I will never find it.
I'm sorry, perhaps I'm missing some key-features. Maybe you can help me see it? Maybe, as you're the more experienced user of these tools can help me see the overall picture?
But the way I see it, I have a choice:

1% chance of missing a bug, that the autofixed will hide from me
99% chance of inconvenience, annoyance, and time spent manually adapting code according to the code-style

for me it isn't a chocie at all.
PS: This autofixer idea in my head is very similar to the idea of autotester, which would automatically write a unit test for a logic, to match the coverage. And that's of course none-sense.

Then let's close this PR. (I don't really want to spend so much time discussing this. If you'd like me to explain my reasoning and how you might have misunderstood some things, please let me know. Maybe a call would also work better then communicating via GitHub issues. ;-) )

(I'd be happy to explain this to you, but only if you're interested in hearing it.)

@oliverklee Sure, I'd be interested in hearing the explaination.
Things that don't quite add up to me:
If using autofixer is good and safe, why do we need the checker and why do humans should even know the rules. surely, if the autofixer can be relied on, then we can trust it to introduce any kind of change the autofixer is programmered to do. In that case, checkers and rules should be out of our control. However, if you don't find that true, that autoprefixes isn't 100% reliable, then why use it. And if it's not reliable, then humans would have to find occurrances of that unreliablness.

Okay. Should we have a call?
I propose we chat in order to find a good date and time. I'm at the Symfony Devs and the TYPO3 Slack.

@oliverklee I created danon.t-regx user in TYPO3. I think I need an invitation to join the slack.

@Danon Now that you have a typo3.org account, you can log in at https://my.typo3.org/ and navigate to https://my.typo3.org/profile . Under the heading "Connected services", there should be an option to connect your account to the TYPO3 Slack workspace.

@Danon Please let's coordinate a good date and time for a call via Slack.


mobile screen overflow fixed

Hi,
On the demo I discovered there is an overflow on mobile screens. I fixed this issue by addind w-full class into StepsFeatures component flex div.
Have a good day :^)

Thanks @berkaycubuk


map not displaying in the div

Hi,
I followed the basic example, and using my API key (with the chrome extension to check it works) and billing setup appropriately with a restriction only to my site and localhost. I see the function like Map, Satellite, fullscreen option, the drop pin option.. BUT the map itself is not displaying.
How to resolve this?

@fsoudiere Thank you for opening this issue. 🙏
Please check out these other resources that might be applicable:

Support Console - If you have a support contract.
Discord - chat with other developers
StackOverflow - use the google-maps tag
Google Maps Platform issue trackers

This is an automated message, feel free to ignore.

Please reopen with a minimally reproducible example. There is not enough detail here to be actionable.

import React, { useEffect, useRef, ReactElement } from "react";
import { Wrapper, Status } from "@googlemaps/react-wrapper";
const render = (status) => {
switch (status) {
case Status.LOADING:
return ;
case Status.FAILURE:
return failed;
case Status.SUCCESS:
return ;
}
};
function MyMapComponent() {
const center = { lat: 102, lng: 23 };
const zoom = 8;
const ref = useRef();
useEffect(() => {
new window.google.maps.Map(ref.current, {
center,
zoom,
});
});
return <div style={{ height: '100vh', width: '100%' }} ref={ref} id="map" />;
}
function Map() {
const apiKeyMap = process.env.NEXT_PUBLIC_GMAP;
return (

)
}
export default Map

const center = { lat: 102, lng: 23 }; are not valid coordinates.

Its always the most obvious thing.. I m sorry thank you!


Implement automatic key colors based on scale size

refs #53

Such extreme scales are not intended. I'll check if using fifth-based scales helps.

Changed to fifth/fourth based construction. Mavila edos still get only two black keys (as they should IMHO), but 17 gets more now.

Works well, looks good to merge


Update ssh_keys.md

Small typo updated for this page: https://buildkite.com/docs/agent/v3/ssh-keys

Thanks for catching this and fixing it, @iggyjs! I'll run the linting, approve, and then merge for you. 😊

@mbelton-buildkite Thanks!


[SUPPORT] When enable bulk insert, executing insert overwrite table will delete the table and then recreate a table

A clear and concise description of the problem.
When hoodie.sql.bulk.insert.enable is enabled, executing insert overwrite will delete the table and then recreate a table, so time travel cannot be performed.
I don't know if this is a reasonable design or if it's a bug.
Steps to reproduce the behavior:

create table hudi_cow_test_tbl (
id bigint,
name string,
ts bigint,
dt string,
hh string
) using hudi
tblproperties (
type = 'cow',
primaryKey = 'id',
preCombineField = 'ts',
'hoodie.sql.insert.mode' = 'non-strict',
'hoodie.sql.bulk.insert.enable' = 'true'
);

insert into hudi_cow_test_tbl select 1, 'a1', 1001, '2021-12-09', '11';

insert overwrite hudi_cow_test_tbl select 3, 'a3', 1001, '2021-12-09', '11';

Environment Description

Hudi version : 0.10.1

I tried to solve the problem by just deleting fs.delete(tablePath, true) in HoodieSparkSqlWriter.
But after that,  execute the query, the old data is not overwritten
By looking at the code, I found that the WriteOperationType of bulk insert overwrite is BULK_INSERT, not INSERT_OVERWRITE. At this time, only commit will be executed, but replace will not be executed.
I don't know good way to solve this problem, maybe add a new type BULK_INSERT_OVERWRITE? Or first determine whether it is OVERWRITE, and then determine the type of insert operation?

yes, insert_override should be honored irrespective of operation type. we might need to put in a fix.

@minihippo : Do you want to take up this fix. its a minor bug fix on the sql layer.

@nsivabalan No problem. I will take it

Created https://issues.apache.org/jira/browse/HUDI-4755 for it. please do follow up. thanks.
CC @xushiyan for the ask and any thoughts you have around this on spark-sql layer.


first installation

k8s 1.6.2:
helm install --name ldap helm-openldap/ --tls
Error: YAML parse error on openldap/charts/ltb-passwd/templates/ingress.yaml: error converting YAML to JSON: yaml: line 5: mapping values are not allowed in this context
I only changed storage-class and size in values.
Also setting both phpldapadmin and ltb-passwd to false:
Error: validation failed: unable to recognize "": no matches for kind "StatefulSet" in version "apps/v1beta2"        ( It's apps/v1 in k8s 1.6.2)
Having this done, the pods crashing (interesing enough - only two are visible) with
read_config: no serverID / URL match found. Check slapd -h arguments

Hi @smirnov-mi ,
Just updated the chart with the templatisation of api version to avoid this issue.
Now ltb-passwd isn't working well so you should disable it.
You don't need to pass tls argument, ldaps is already configured. This is only to pass a custom certificate.
So could you please retry without the tlsand tell me if you have still the issue ?

Hi @kpc31 ,
the   --tls    refers to my HELM, so it's OK. I don't know how to tell helm to use tls by default, so I have to add "--tls" manually every time.
I tried the updated chart now,
one little thing - I had to set  apps/v1  in the phpldapadmin/templates/deployment.yaml.
One big thing:    ldap pod still crashes, phpLDAPadmin crashes as well, logs below. Does it require some sort of initial ldif or something?
openldap container logs:
*** CONTAINER_LOG_LEVEL = 3 (info)
*** Search service in CONTAINER_SERVICE_DIR = /container/service :
*** link /container/service/:ssl-tools/startup.sh to /container/run/startup/:ssl-tools
*** link /container/service/slapd/startup.sh to /container/run/startup/slapd
*** link /container/service/slapd/process.sh to /container/run/process/slapd/run
*** Set environment for startup files
*** Environment files will be proccessed in this order :
Caution: previously defined variables will not be overriden.
/container/environment/99-default/default.yaml
/container/environment/99-default/default.startup.yaml
To see how this files are processed and environment variables values,
run this container with '--loglevel debug'
*** Running /container/run/startup/:ssl-tools...
*** Running /container/run/startup/slapd...
Database and config directory are empty...
Init new ldap server...
Backing up /etc/ldap/slapd.d in /var/backups/slapd-2.4.47+dfsg-3~bpo9+1... done.
Creating initial configuration... done.
Creating LDAP directory... done.
invoke-rc.d: could not determine current runlevel
invoke-rc.d: policy-rc.d denied execution of restart.
Start OpenLDAP...
Waiting for OpenLDAP to start...
Add bootstrap schemas...
config file testing succeeded
Add image bootstrap ldif...
Add custom bootstrap ldif...
Add TLS config...
Add replication config...
Stop OpenLDAP...
Configure ldap client TLS configuration...
Remove config files...
First start is done...
*** Set environment for container process
*** Remove file /container/environment/99-default/default.startup.yaml
*** Environment files will be proccessed in this order :
Caution: previously defined variables will not be overriden.
/container/environment/99-default/default.yaml
To see how this files are processed and environment variables values,
run this container with '--loglevel debug'
*** Running /container/run/process/slapd/run...
5df6c98d @(#) $OpenLDAP: slapd  (Feb  9 2019 17:02:42) $
Debian OpenLDAP Maintainers<EMAIL_ADDRESS>5df6c98d read_config: no serverID / URL match found. Check slapd -h arguments.
5df6c98d slapd stopped.
5df6c98d connections_destroy: nothing to destroy.
*** /container/run/process/slapd/run exited with status 1
*** Killing all processes...
phpldapadmin container logs:
2019/12/16 00:07:45 [INFO] received CSR
2019/12/16 00:07:45 [INFO] generating key: ecdsa-384
2019/12/16 00:07:45 [INFO] encoded CSR
2019/12/16 00:07:45 [INFO] signed certificate with serial number<PHONE_NUMBER>6946035950657881504992111099161697437
Link /container/service/:ssl-tools/assets/default-ca/default-ca.pem to /container/service/ldap-client/assets/certs/ldap-ca.crt
*** Running /container/run/startup/phpldapadmin...
Set apache2 http config...
Bootstap phpLDAPadmin...
tr: write error: Broken pipe
tr: write error
sed: -e expression #1, char 79: unterminated `s' command
*** /container/run/startup/phpldapadmin failed with status 1
*** Killing all processes...

Hi @smirnov-mi
I think i found your issue, the chart has to be deployed in the chart name namespace, openldap in your case.
I'll investigate why .
I just updated the chart with phpldap running and ltb-passwd

Hey @kpc31 ,   thank you very much!
Looks like it works now. Have to verify if the sync really works, but I see 3 pods and phpLDAPadmin as well.    I have a question: what is the LDAP-conf then with 3 masters ?  Do they have each two more servers configured, to sync with?    Or  does it make more sence to use only 2 pods, so they sync each other?
I will enable ltb-passwd now and try it.

Hi @smirnov-mi  , they all see eachother.
There are a variable configured LDAP_REPLICATION_HOSTS  for 3 hosts the variable looks like :
#PYTHON2BASH:['ldap://helm-openldap-github-0.helm-openldap-github-headless','ldap://helm-openldap-github-1.helm-openldap-github-headless','ldap://helm-openldap-github-2.helm-openldap-github-headless']

If you need to scale up or down then you have to update the chart with the number of wanted replica.
The replication configuration is :
LDAP_REPLICATION_CONFIG_SYNCPROV: "binddn=\"cn=admin,cn=config\" bindmethod=simple credentials=$LDAP_CONFIG_PASSWORD searchbase=\"cn=config\" type=refreshAndPersist retry=\"60 +\" timeout=1 "
  LDAP_REPLICATION_DB_SYNCPROV: "binddn=\"cn=admin,$LDAP_BASE_DN\" bindmethod=simple credentials=$LDAP_ADMIN_PASSWORD searchbase=\"$LDAP_BASE_DN\" type=refreshAndPersist interval=00:00:00:10 retry=\"60 +\" timeout=1 "

I will update the chart to add some info in the values , like the interval and the timeout

OK, cool. Sorry, i'm new to k8s and to multi-master ldap as well .
ltb-passwd wouldn't start. Describe pod gives:
Environment:
  LDAP_SERVER:           ldaps://openldap.openldap
  LDAP_BINDDN:           <set to the key 'BINDDN' in secret 'openldap'>  Optional: false
  LDAP_BINDPASS:         <set to the key 'BINDPW' in secret 'openldap'>  Optional: false
  LDAP_STARTTLS:         false
  LDAP_BASE_SEARCH:      dc=example,dc=org
  SECRETEKEY:            LDAP_ADMIN_PASSWORD
  LDAP_LOGIN_ATTRIBUTE:  cn

Events:
Type     Reason   Age                    From                     Message

Warning  Failed   27m (x211 over 77m)    kubelet, worker-01  Error: couldn't find key BINDDN in Secret ldap/openldap

Hi @smirnov-mi ,
Indeed you have to create two Key in the secret defined in binduserSecret .
Per defaults its openldap , so you have to add the Key BINDDN and BINDPW  to the dn of the user admin ans its password.
Or any users allowed to query the database

I'm trying to do it, no luck yet....  Upgraded my HELM to v3 in the meanwhile...

Hi @jp-gouin , hope you're doing well.
I ended up having it installed somehow last time (2x openldap + phpLDAPadmin). Now I'm trying to recreate it and no luck. I also tried to increase number of replicas to 3 and the 3rd pod can not start as well.
I'm on k8s 1.7.2 now.  Start of the first pod is always like this. (BTW it always starts with loglevel 3):
*** CONTAINER_LOG_LEVEL = 3 (info)
*** Search service in CONTAINER_SERVICE_DIR = /container/service :
*** link /container/service/:ssl-tools/startup.sh to /container/run/startup/:ssl-tools
*** link /container/service/slapd/startup.sh to /container/run/startup/slapd
*** link /container/service/slapd/process.sh to /container/run/process/slapd/run
*** Set environment for startup files
*** Environment files will be proccessed in this order :
Caution: previously defined variables will not be overriden.
/container/environment/99-default/default.startup.yaml
/container/environment/99-default/default.yaml
To see how this files are processed and environment variables values,
run this container with '--loglevel debug'
*** Running /container/run/startup/:ssl-tools...
*** Running /container/run/startup/slapd...
Database and config directory are empty...
Init new ldap server...
Backing up /etc/ldap/slapd.d in /var/backups/slapd-2.4.48+dfsg-1~bpo10+1... done.
Creating initial configuration... done.
Creating LDAP directory... done.
invoke-rc.d: could not determine current runlevel
invoke-rc.d: policy-rc.d denied execution of restart.
Start OpenLDAP...
Waiting for OpenLDAP to start...
Add bootstrap schemas...
config file testing succeeded
Add image bootstrap ldif...
Add custom bootstrap ldif...
Add replication config...
Stop OpenLDAP...
Remove config files...
First start is done...
*** Set environment for container process
*** Remove file /container/environment/99-default/default.startup.yaml
*** Environment files will be proccessed in this order :
Caution: previously defined variables will not be overriden.
/container/environment/99-default/default.yaml
To see how this files are processed and environment variables values,
run this container with '--loglevel debug'
*** Running /container/run/process/slapd/run...
5e959d85 @(#) $OpenLDAP: slapd  (Jul 30 2019 16:24:19) $
Debian OpenLDAP Maintainers<EMAIL_ADDRESS>5e959d85 read_config: no serverID / URL match found. Check slapd -h arguments.
5e959d85 slapd stopped.
5e959d85 connections_destroy: nothing to destroy.
*** /container/run/process/slapd/run exited with status 1
*** Killing all processes...


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

AWS CodeBuild CI Report

CodeBuild project: DDKIntegrationTestsD8472609-5SCHWjldmWcf
Commit ID: 0600a88ea71376049a5daf6a9e1284ca35fbf075
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[chore] Separate website/elements/apps from library?

The current structure has served us well but we're at the point where we need to separate the two.
In terms of tests, presumably the JS tests will stay with the library, while the HTML tests (old and new) will live with the website.
Both will still deploy to the web, I’m thinking dev.colorjs.io for the library repo.

Where would the API docs live? Presumably they need access to the source, but perhaps that can be done by installing color.js as an npm package on the website repo and pointing typedoc to that?

Where would the API docs live? Presumably they need access to the source, but perhaps that can be done by installing color.js as an npm package on the website repo and pointing typedoc to that?

That should work if we want to keep all website stuff together. Alternatively there could also be a Git submodule that links back to this repo, but that might be overcomplicating things.

Yeah, my opinion on this is that submodules are plague and must be avoided at all costs 😛

I’ve been thinking about this and I think it makes sense to also separate apps and elements into separate repos / separate npm packages / separate subdomains.
Especially as I plan a bit of a revamp of these, what do you think about separating them?
We could create an npm namespace and publish them as @colorjs/apps and @colorjs/elements.
Later down the line we may even want to separate elements into separate packages, but let’s cross that bridge when we get to it.

As an update, I have split elements and apps into their own repos and websites (https://apps.colorjs.io and https://elements.colorjs.io respectively). The website will be harder to split out though…

Ok, thinking about this some more, I have a plan:

We split into two repos: core (which we can deploy, e.g. to dev.colorjs.io) and colorjs.io (names TBB, suggestions welcome)
core will contain test, tests, src, types, scripts, CONTRIBUTING.md, .editorconfig, ESLint config, tsconfig, LICENSE, README.md
colorjs.io will contain: Notebook, 11ty files, assets, get/
Unsure: benchmarks, Typedoc,

Thoughts?

I think it'd be ideal to have tests/benchmarks alongside the source code, but I'm not sure how practical it'd be to host them on the website if we do that. If there's an easy way to set up the website to use content from multiple repositories (besides Git submodules which we've decided against), then I'd opt for putting those directories with the source code. Otherwise I'd say they should go with the rest of the website content.

Depends on what you mean by "the website". We can easily deploy the code repo to a separate subdomain (e.g. dev.colorjs.io)

That'd be the best way to do it then imo


Django 1.8 Warning "get_query_set should be renamed get_query_set"

positions/managers.py:31: RemovedInDjango18Warning: PositionManager.get_query_set method should be renamed get_queryset.
class PositionManager(Manager):
This is running Django 1.7.4

#33


Makefile for creating a dynamic library

Hey Max,
I really like your library and I wanted to make a C++ dynamic library out of it to link to locally.
I had to merge the C and the C++ files since I didn't figure out how to make a library out of them separately.
Maybe you could maintain a Makefile as part of the repository, to ease this process for the future?
P.S. I am really enjoying your algorithms course on openedu.ru, I really hope there will be a second advanced class in the future!

Hi Yechiel,
I have just added a makefile (check the misc directory) which produces a shared library. It is also equipped with a basic sanity test. (And, thanks to your request, edx-io.h now has extern "C" so that linking the object files made by gcc and g++ is possible).
Could you please check whether it fulfils your needs?

Checked it and it works, thanks so much.

Thank you for the response, I am closing the issue now.
On the openedu.ru course. Unfortunately, I have no idea for whether there will be a follow-up course. I was notified recently that, according to the recent data analysis of the logs, the course is typically felt as overcomplicated. One of the goods solutions is to split the course into two, where the first one is prepended by a few weeks dedicated to the basics of programming, and the second one is augmented by more topics (such as graphs and computational geometry). However, I doubt I am qualified enough to make up the easy part, and I also doubt that those who are qualified will agree to participate.


Add helpers for dealing with timestamps

created_at doesn't need anything from us, we can just have DEFAULT current_timestamp. updated_at requires a trigger, but we shouldn't require our users to write that. We can use the same procedure for pretty much every table:
CREATE FUNCTION set_updated_at() RETURNS trigger AS $$
BEGIN
    IF (
        NEW IS DISTINCT FROM OLD AND
        NEW.updated_at IS NOT DISTINCT FROM OLD.updated_at
    ) THEN
        NEW.updated_at := current_timestamp;
    END IF;
    RETURN NEW;
END
$$ LANGUAGE plpgsql;

We should provide that so users don't have to write it repeatedly

Unsure if this is too much, but I think I'd also like to have something like diesel_manage_updated_at(table_name), because I can never remember the trigger syntax.

Did you want to set updated_at upon INSERT as well?

We can handle that via DEFAULT NOW()

Alright, and you wanted a procedure like this?
CREATE OR REPLACE FUNCTION diesel_manage_updated_at(_tbl regclass) RETURNS VOID AS $$
BEGIN
  EXECUTE format('CREATE TRIGGER set_updated_at BEFORE INSERT OR UPDATE ON %s FOR EACH ROW EXECUTE PROCEDURE set_updated_at()', _tbl);
END;
$$ LANGUAGE plpgsql;

I guess it should just be BEFORE UPDATE if we assume updated_at has a default.

Oh yeah, had that one written prior to your response haha. Apologies if this is obvious, but where do you imagine these helpers existing? Should the updated_at/created_at -> Timestamp pairs be reserved and handled with this procedure automatically within the migration code, or exposed to the user so they can decide whether or not diesel should manage it for them? Or is there a plan to implement a migration DSL in Rust, akin to Rails (e.g. add_timestamps)?

There's currently no Rust DSL, though we may add one in the future as use cases arise. I'm fine with creating this as part of the migrations code, as long as it has a sufficiently specific name.
The actual code for creating these should probably live on the Connection, so we can easily adapt it for SQLite once #152 is merged.

I've been looking into what exactly would it take to implement this, constrained by the SQL format of migrations. If they wanted to, a user could write valid SQL with alternating capitals and one word per line, and it'd be accepted by Pg. So implementing a check for the phrase updated_at TIMESTAMP and then parsing the table_name would most likely mean collapsing all whitespace, and then using the regex crate's capture groups.
I also thought about a pseudo-DSL using SQL comments, but a good design of that basically means implementing the foundation of a DSL entirely.

We shouldn't do any sort of magic around this, the user should set the trigger themselves. We just need to provide the function for them.


Tvs test branch

Describe your changes
Issue number
#000
Checklist before requesting a review

[ ] The pr title can be used to describe what this pr did in CHANGELOG.md file
[ ] I have executed pre-commit on my machine
[ ] I have passed pr-check on my machine

Thanks for your cooperation!

Hi @tjgitdh thanks for opening this pr! May I ask what's this pr's purpose? Could you please give us a description with more details? Thanks!

I'm closing this pr since no response from the author, please feel free to reopen it if you think you're ready to contribute new feature or bug fix, thanks @tjgitdh!


Feature request -- prevent unsupported browsers from fb/goog

Description
We have been plagued by various issues from users coming from iOS Facebook (inc. Messenger) and iOS LinkedIn. The main problem is that Firebase does not support Google or FB auth from within these non-standard in-app browser-ish thing that the FB and LinkedIn iOS apps use. There does not seem to be a workaround as it's tightly prohibited by Google/Firebase, and the "solution" has changed direction several times but the latest approach is described below.
Related items
(from Jira CSR-134)
#34 and #37 should probably be done first so we have the button structure in place. Then it will be easier to add the "goog/fb login..." message below the buttons.
Resolution

[x] Undo existing setup: remove unsupported message that appears when symptoms or signup modal are opened.
[x] For unsupported browsers, add "Google/Facebook login available in web browser only" below buttons
[x] Disable the FB/Google buttons for those users.

Note that this does not include the proposed related changes to the splash screen, nor the suggestion to show an iOS screenshot explaining how to open a browser.


Clarify what was the product removed from

What

Clarify what was the product removed from. You could otherwise believe you have deleted the product from your local history or from the server
"Product removed from comparison"

Part of

#1284

Screenshot

Hi @teolemon ,
All we have to do is update the snack bar?

Hi @teolemon,
Please assign this issue to me.

Please close this issue as this is fixed by #1345


Direct alloc/dealloc implementations for ecma_number_t

As a result of earlier developments, there remained no other
instatiations of the ALLOC, DEALLOC, and DECLARE_ROUTINES_FOR
templates. So, it's simpler to write the alloc/dealloc routines
directly for ecma_number_t as well.
JerryScript-DCO-1.0-Signed-off-by: Akos Kiss<EMAIL_ADDRESS>

Note: The proposed change gives equivalent code to the currently existing implementation. I've tried adding the inline and JERRY_ATTR_ALWAYS_INLINE modifiers to the functions the same way they are used at the other allocs/deallocs, however, that resulted in no speed improvement but increased the binary size 133440 -> 133752.


GPU support

We current run the workflow in a GPU-less container. From what I have tested, this could get GPU support by:

basing on a CUDA image, e.g. FROM nvidia/cuda:10.0-cudnn7-runtime-ubuntu18.04, and
running docker run --gpus all

It should still work on CPU-only systems.

8eba477b205e666f55e6141d8a269b6d53fb975b introduced rudimentary support. This could be a bit cleaner though, i.e. no duplication in Dockerfile-core*

8eba477b205e666f55e6141d8a269b6d53fb975b introduced rudimentary support. This could be a bit cleaner though, i.e. no duplication in Dockerfile-core*

Now the question is instrumentation, i.e running the containers with docker run --gpus=all.

Always run with --gpus=all

This seems to be possible, with no effect, even on non-GPU systems

Use CUDA_VISIBLE_DEVICES to support disabling running on GPU

We need that sometimes to make use of the CPU ressources available

TL&DR: Use GPU by default, fall back to CPU if necessary or told by the user

Related: https://stackoverflow.com/questions/59008295/add-nvidia-runtime-to-docker-runtimes

https://github.com/NVIDIA/nvidia-container-runtime#nvidia_visible_devices


Github Action is available

I created an public github action to build linux factory. The action tests agaisnt this repo with an successful build here. However, live testing the system on a proxmox VM fails to start the live system without using fail-safe mode, and even the fail-safe mode gets stuck on a gdm login screen. I am unable to confirm if this is an result of the action or the result of the current repo, as latest gitlab builds are failing and I can't seem to find an alternative iso anywhere to compare images.

Sometimes you have to play around with different settings, specifically the debianInstallerDistribution as the debian team is always pushing out new changes to it.
https://github.com/clayrisser/linux-factory/blob/main/os/config.yaml#L20
I'll try a new build soon to test it again.
One suggestion is to try turning off most of the overlays to help pinpoint where the issues are brought in.


upgrade protobuf to v3.0.0-alpha-3

Some commits have been added to the alpha-3 release since we last upgraded:
https://github.com/google/protobuf/commits/v3.0.0-alpha-3

Duplicate of #1810.

ah, it's so hard to keep track ;-)

No worries.
I am testing the prebuilt packages for it, so the merge should happen soon.
On Fri, May 29, 2015 at 2:28 PM, Jan Tattermusch<EMAIL_ADDRESS>wrote:

ah, it's so hard to keep track ;-)
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/1825#issuecomment-106940235.


How to choose what field to process at runtime

I have a data structure and visitor set up to parse streaming data for handling multi-gigabyte files. It looks like the following, so I can read the "features" property of massive JSON files:
#[derive(Deserialize, Serialize)]
struct CollectionWrapper {
    #[serde(
        deserialize_with = "deserialize_stream",
        rename(deserialize = "features")
    )]
    features_processed: u64,
}

I use it to read JSON files that look like:
{
  "ignored_property": "whatever",
  "more": …,
  "features": [{name: "something"}, …]
}

This works well, however: I would like to make the field I am deserializing configurable. For example, I would like to pass a flag at runtime to specify the name of the "features" property in case I have some different JSON. Right now, I can only see how to change this at compile-time, and it doesn't feel right to edit source and recompile just to handle slightly different data layouts.
Is it possible to tell serde the name of fields at runtime? How might I approach that?
I tried driving a visitor that implemented both visit_map and visit_seq. While I was able to find out dynamically whether a map entry had a key matching what I passed in, I wasn't able to figure out how to proceed with streaming parsing after that point. The visitor doesn't have a reference to the deserializer and seems to load the whole value into memory as part of the map visiting.
Thanks!

The closest I've gotten to specifying at runtime is with the visitor implementation below. Is there a way to deserialize the next_value (which I know to be an array) in a stream?
        fn visit_map<M>(self, mut map: M) -> Result<Self::Value, M::Error>
        where
            M: MapAccess<'de>,
        {
            while let Some(key) = map.next_key()? {
                let key: String = key;
                if key == self.collection_name {
                    // could this perhaps descend into our visit_seq?
                    // how do I stream from here?
                    let v: serde_json::Value = map.next_value()?;
                    println!("value found: {:?}, {}", v, v.is_array());
                } else {
                    map.next_value::<IgnoredAny>()?;
                }
            }
            Ok(0)
        }

I'd love to replace the next_value handling with something like:
let val: SeqAccess = map.next_value()?;
self.visit_seq(val);

That way I could handle the sequence nicely, but SeqAccess can't be instantiated as it's a trait.

// could this perhaps descend into our visit_seq?
// how do I stream from here?

You would need to call map.next_value_seed with a visitor for processing the value. Something like:
struct FeaturesVisitor {...}

impl<'de> DeserializeSeed<'de> for FeaturesVisitor {
    type Value = ??;
    fn deserialize<D>(self, deserializer: D) -> Result<Self::Value, D::Error>
    where
        D: Deserializer<'de>,
    {
        deserializer.deserialize_seq(self)
    }
}

impl<'de> Visitor<'de> for FeaturesVisitor {
    type Value = ??;
    fn visit_seq ...
}

map.next_value_seed(FeaturesVisitor {...})

The pseudocode you wrote is nicer but would be blocked on GAT.


[AutoRelease] t2-quota-2022-11-09-21486(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3390
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=3083

issue link:https://github.com/Azure/sdk-release-request/issues/3390


updated /tutorials/security/seccomp.md

Mainly removed an extra { on line 66.
content/en/docs/tutorials/security/seccomp.md

Preview: https://deploy-preview-35902--kubernetes-io-main-staging.netlify.app/docs/tutorials/security/seccomp/

/lgtm

Adding a hold while the comments above are replied to.
/hold

Adding a hold while the comments above are replied to.
/hold

Ok, noted and modified, thanks.

/lgtm
/unhold

/lgtm
/approve


Document symbols should include imported modules

In the VS Code Python extension we are sending a textDocument/documentSymbol request and using the result to position code lenses.  The data we get back is mostly correct.  However, some symbols are missing.
For example, using the following Python code:
import unittest

class SpamTests(unittest.TestCase):
    def test_all(self):
        self.assertTrue(False)

I got the following JSON response from the language server (formatted for clarity):
[{
    "name":"SpamTests",
    "detail":"SpamTests",
    "kind":5,
    "deprecated":false,
    "range":{
        "start":{"line":2,"character":6},
        "end":{"line":2,"character":15}
    },
    "selectionRange":{
        "start":{"line":2,"character":6},
        "end":{"line":2,"character":15}
    },
    "children":[{
        "name":"test_all",
        "detail":"test_all",
        "kind":12,
        "deprecated":false,
        "range":{
            "start":{"line":3,"character":4},
            "end":{"line":4,"character":30}
         },
         "selectionRange":{
             "start":{"line":3,"character":4},
             "end":{"line":4,"character":30}
         },
         "children":[{
             "name":"self",
             "detail":"self",
             "kind":13,
             "deprecated":false,
             "range":{
                 "start":{"line":3,"character":17},
                 "end":{"line":3,"character":21}
             },
             "selectionRange":{
                 "start":{"line":3,"character":17},
                 "end":{"line":3,"character":21}
             },
            "children":[],
            "_functionKind":""
        }],
        "_functionKind":"function"
    },
    {
        "name":"assertTrue",
        "detail":"assertTrue",
        "kind":13,
        "deprecated":false,
        "range":{
            "start":{"line":0,"character":0},
            "end":{"line":0,"character":0}
        },
        "selectionRange":{
            "start":{"line":0,"character":0},
            "end":{"line":0,"character":0}
        },
        "children":[],
        "_functionKind":""
    }],
    "_functionKind":"class"
}]

The symbol for the "unittest" module is missing.

Unless it is a user request, won't fix.


Agent supports the collection of JVM arguments and jar dependencies information.

[x] Update the CHANGES log.

If the frontend is optimized, it will be more friendly.

@kezhenxu94 @Ax1an This is a thing I keep waiting for. Once we merge this, InstanceTraffic could combine the jvm arguments and dependency libs changes, which should be considered as a JVM level event. Redeploying with codes/setting upgrading is a very common case for a performance issue, such as -XMX change.

We should add a config list at the backend, to indicate which instance properties should be identified as an instance event.

@wu-sheng @kezhenxu94 I'll take a moment to learn about the logic of skywalking's events.

@wu-sheng @kezhenxu94 I'll take a moment to learn about the logic of skywalking's events.

I think you could finish this PR first if you want this included in the 8.6.0 release. It is coming. Event mechanism is a backend feature.

Sorted results:

@wu-sheng According to your question, I made some adjustments.

@wu-sheng @kezhenxu94 I've refactored the code and tested it.

@wu-sheng I adjusted the way to get the classloader list. And got the same test results as above.

We have conflicts, please fix.

@Ax1an Tests are failing.

@kezhenxu94 I'm a little confused about this. Can I get more detailed information?

@Ax1an this error typically means the OAP cannot start up successfully in Docker, you can simply build the Docker image locally and docker run it, it should crash somehow

@kezhenxu94 The feature is good to me. Please check the codes, and especially the tests.

@kezhenxu94 I found that this test case has failed since the main branch was merged.

@kezhenxu94 I found that this test case has failed since the master branch was merged.

Yes never mind this case. It's not a blocker. Once all others pass we can merge this


Darwin: MTRDeviceControllerFactory improvements

Handle cleanup after failed factory start in the same dispatch_sync
Also remove some unnecessary OOM handling at the framework level, treat allocation failure as fatal.

Fast-tracking Darwin-only change.


Robot load: Running on the same sample twice results in the sample being unloaded and reloaded

Likely a GDA issue. When asking to do a hyperion robot load + data collection on a sample which is already loaded, the sample will be unloaded by the robot, and then reloaded. In this case, the robot shouldn't do anything at all
Acceptance Criteria

Implement fix and test on beamline

Discuss with @ndg63276 what the behavior of the robot is when asked to load the same position. We think it should be checking.

Talk with Mark about how we want this to work


Use Custom attributes & alpine js

@CherphAlt you'll have to be a bit more specific with what this issue means. 😊

How can i add custom attributes to the html elements like data-this or use custom elements

Have a look at the docs at https://pkg.go.dev/github.com/maragudk/gomponents#Attr .


PrimeNG integration

Does anyone have experience with integrating the primeNG library in a webpack based project?
Trying to integrate it in this starter results in the following error:
Error: No Directive annotation found on Button     at new BaseException (http://localhost:3000/main.bundle.js:1318:24)     at DirectiveResolver.resolve (http://localhost:3000/main.bundle.js:11928:16)     at RuntimeMetadataResolver.getDirectiveMetadata (http://localhost:3000/main.bundle.js:32492:52)     at http://localhost:3000/main.bundle.js:32549:63     at Array.map (native)

there are two versions of Angular 2 installed. they need to update their angular2 dep or set as a peerDep

see https://github.com/primefaces/primeng/pull/86

Merged now, thank you.
https://github.com/primefaces/primeng/issues/88

@cagataycivici thanks

@bPopovska did you get it running? We have basics running but datepicker makes trouble :(

@Sicaine Still running into issues. Can't get primeng to work when source maps are enabled.

@bPopovska Are the widget styles issues you talk about related to the images urls of the style? i'm using sass-loader and as it just process the import statement as if that theme content were copied to the importing file, it breaks the images urls since now that content is in a different folder.

I've done a fork to add PrimeNG for the UI;
https://github.com/cagataycivici/angular2-webpack-starter

@cagataycivici Does dialogs work in yours? the errors above i fixed using the url loader but now i have issues when using the Dialog and seems related to jquery.

Yes, it works for me in my fork above.

I don't use assets for Css I just do a require on node modules in app.ts
Envoyé de mon iPhone

Le 21 avr. 2016 à 23:12, Çağatay Çivici<EMAIL_ADDRESS>a écrit :
I've done a fork to add PrimeNG for the UI;
https://github.com/cagataycivici/angular2-webpack-starter
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

@cagataycivici Maybe my error is related to the way jquery is loaded due to me using the NgUpgrade package to try and update an Ng1 app? i'll take a deeper look at your starter and see what's the issue. 😄

@cagataycivici I made a fork of yours and made some changes in helpers, however when i go to pull request it shows me the original AngularClass repo, why yours not showing in the pull request options?


BREAKING CHANGE: make getLatestBlockhash async

Description
"so i was thinking about how we get the poller out of the JS SDK.  the main issue is that litNodeClient.getLatestBlockhash(); is not async.  if we make it async, then we can remove the poller, and just get the latest block hash when they call that function.  this is a breaking change, but def worth it IMO"
Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] yarn test:e2e:nodejs --filter=test-latest-blockhash.mjs

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

this makes sense to me and looks good.  i think we should add caching?  if the block hash is less than 30s old, no need to talk to the nodes and get the latest?  probably a good idea to prevent developers that like, accidentally call getLatestBlockhash() on every react re-render from hitting the nodes more than necessary.

This is re-implementing a lot of existing logic from lit-core, where we do this exact process already during the connect() flow. In fact, functionally it is the same, as it handshakes with all nodes, and even rebuilds the connectedNodes, but in its current state it leaves connectedNodes in an inconsistent state for the duration of this process -- while we're waiting on handshakes to return, the set will be mutated 'in place' - calling connect() no longer does that as of LIT-2560 -- and this could possibly run multiple concurrent handshakes if the method is called multiple times close together, which connect() will no longer do.

Can we move this code to live in core, since it is managing the same state as core does anyway?
Can we avoid re-implementing 'get most common value' and error handling/logging, by instead adding the method core that calls this.connect() and then returns this.latestBlockhash?

We're already updating the latestBlockhash property during connect() execution anyway, using another implementation that already gets the 'most common string' in the returned values from the nodes
We already have error handling and logging on fetching the blockhash from each node and an error is thrown if we fail to get the blockhash from any node

This method in lit-node-client can then be delegated to the core implementation. When we find a better place to source latestBlockhash from, we can replace the connect() usage in lit-core to be lighter-weight.

Great points! Made relevant changes :)

We're just about there I think!

I think there are a couple of instances in the apps directory that need still need updating - here and here
Also, not a blocker for this PR I assume, but I think we'll also need a PR against [docs](repo:LIT-Protocol/docs getLatestBlockhash) to update any references there

Otherwise this LGTM 👍!
NOT blockers for this PR, but the changes to .vscode/settings.json made me consider: Since we're trying to improve code hygiene and consistency, and eliminate eslint errors and unnecessary reformat commits...

It would be helpful if you can test enabling format-on-save using prettier and automatic ESLint fix on save for vscode users and commit those changes either in this PR or in a follow-up since it's not directly related to this PR :).
Prettier: Check out # 2 in this post: https://stackoverflow.com/a/70242338
ESLint: https://www.masrinastudio.com/post/eslint-formatting-vscode-guide/
It would also probably provide nice auto-complete for folks in vscode using conventional commits to just enter in all of the conventional commit scopes at once rather than only adding a few at a time :)

I'll commit some editor configuration for Webstorm that does the same things, since that's what I'm using :)

Thanks for being attentive to all required changes for this! Made relevant changes! I will look into the testing prettier and automatic ESLint fix in a follow-up PR :)

We're just about there I think!

I think there are a couple of instances in the apps directory that need still need updating - here and here
Also, not a blocker for this PR I assume, but I think we'll also need a PR against [docs](repo:LIT-Protocol/docs getLatestBlockhash) to update any references there

Otherwise this LGTM 👍!
NOT blockers for this PR, but the changes to .vscode/settings.json made me consider: Since we're trying to improve code hygiene and consistency, and eliminate eslint errors and unnecessary reformat commits...

It would be helpful if you can test enabling format-on-save using prettier and automatic ESLint fix on save for vscode users and commit those changes either in this PR or in a follow-up since it's not directly related to this PR :).
Prettier: Check out # 2 in this post: https://stackoverflow.com/a/70242338
ESLint: https://www.masrinastudio.com/post/eslint-formatting-vscode-guide/
It would also probably provide nice auto-complete for folks in vscode using conventional commits to just enter in all of the conventional commit scopes at once rather than only adding a few at a time :)

I'll commit some editor configuration for Webstorm that does the same things, since that's what I'm using :)

Thanks for being attentive to all required changes for this! Made relevant changes! I will look into the testing prettier and automatic ESLint fix in a follow-up PR :)

Merge conflicts occurred after another PR was rebased and merged to master

@Ansonhkg I assume we want this to land in this week's release w/ other breaking changes -- LMK as soon as the conflicts are resolved and I'll re-add approved review so you can get it merged :). It sure will be nice when we get to the point that we're not modifying package.json in our feature PRs anymore :)

@MaximusHaximus conflicts are resolved can you re review?


requestWithCredentials not using origin header

Not sure if I'm misunderstanding how the "requestWithCredentials" option works but I'm having a bit of trouble. When uploading an image, the initial POST sends my cookie and an origin header. The subsequent request to get the image data to insert into the WYSIWYG contains the cookie, but not the origin header. The authorization built into the backend I am using requires the origin header for authorization. Attempted to set it via the "requestHeaders" option, but since it is an unsafe header I am unable to monkeypatch. Are there any known workarounds for such a scenario?
Thanks.

Looks like the second request is made by the browser (in the image src attribute) and isn't made by a jQuery XMLHttpRequest in the tool like I thought. Closing this one.

Hi,
I am facing the same issue, did you come up with a workaround.
Thanks


document app.css

https://github.com/angular/material-start/blob/master/app/assets/app.css
Where did this file come from?
How does it apply to material design?
Why does it seem to be so responsible for the look and feel?
Why isn't it scss?
I guess my confusion is more around why this isn't just part of angular/bower-material/angular-material.css? It doesn't feel very re-usable and feels kind of hidden in this starter project.

I think there are quite a few styles in there that can be removed once https://github.com/angular/material/pull/1985 is merged. But I think the general idea is that you should have a file with your specific overrides in it. If there are that many look-and-feel styles in app.css even when angular-material.css is included, it points out holes in our CSS API.

Just commenting out the app.css, it feels like there is quite a bit of style being declared that is helpful at a framework level.
http://take.ms/b9oDT vs. http://take.ms/X7Ivw


Each disco node sees itself as the master

Hi,
I have disco running with 7 nodes, master included. With the nodes with no disco process running I start the master, and the master starts the nodes like in the tutorial. The process on the node is like:
"disco    14960  0.0  0.0<PHONE_NUMBER>6 ?        Sl   10:24   0:00 /usr/lib64/erlang/erts-5.8.5/bin/beam.smp -K true -- -root /usr/lib64/erlang -progname erl -- -home /home/users/disco -- -noshell -noinput -noshell -noinput -master disco_8989_master@dtn-cn -sname disco_8989_slave@hulk -s slave slave_start disco_8989_master@dtn-cn slave_waiter_6 -connect_all false -pa /usr/lib/disco/master/ebin/ -pa /usr/lib/disco/master/deps/mochiweb/ebin -pa /usr/lib/disco/master/deps/lager/ebin -pa /usr/lib/disco/master/deps/plists/ebin -f:
There is no log on the nodes, but in the master "disco -v"
[disco@dtn-cn ~]$ disco -v | grep dtn-cn
DISCO_JOB_OWNER =<EMAIL_ADDRESS>DISCO_MASTER = http://dtn-cn.mydomain.com8989
DISCO_MASTER_HOST = dtn-cn.mydomain.com
DISCO_TEST_HOST = dtn-cn.mydomain.com
Disco master at http://dtn-cn.mydomain.com:8989
In the salve "disco-v"
[disco@hulk disco]$ disco -v | grep hulk
DISCO_JOB_OWNER =<EMAIL_ADDRESS>DISCO_MASTER = http://hulk.mydomain.com:8989
DISCO_MASTER_HOST = hulk.mydomain.com
DISCO_TEST_HOST = hulk.mydomain.com
Disco master at http://hulk.mydomain.com:8989
When I try to run a job in a node the final output is:
disco.error.CommError: Unable to access resource (http://hulk.mydomain.com:8989/disco/job/new): couldn't connect to host (is disco master running at http://hulk.mydomain.com:8989?)
Is there a way to change the master in the nodes? what can I do?

Outside of the actually running of the job on the worker, I do not think that the worker is tied to a particular master machine.
When running a Job the worker does know which master it is serving.  Otherwise it would not be able to save the results back to DDFS (The save_results worker parameter implemented here https://github.com/discoproject/disco/blob/a89e37843bad58f744a29d642f84686ae467a6c0/master/src/job_coordinator.erl#L393)
If you want to specify a Disco master node on the nodes for the purpose of running disco/ddfs commands from the node then you can use the DISCO_MASTER_HOST setting (http://disco.readthedocs.io/en/latest/lib/settings.html), for example by setting the DISCO_MASTER_HOST in the appropriate shell file when you login to the nodes.


Show pull link for agit pull request also

example:

Codecov Report

Merging #18235 (6545408) into main (ba9e4e2) will increase coverage by 0.00%.
The diff coverage is 58.33%.

@@           Coverage Diff           @@
##             main   #18235   +/-   ##
=======================================
  Coverage   45.62%   45.62%
=======================================
  Files         830      830
  Lines       92024    92048   +24
=======================================
+ Hits        41983    41999   +16
- Misses      43300    43303    +3
- Partials     6741     6746    +5

Impacted Files
Coverage Δ

routers/private/hook_post_receive.go
62.40% <58.33%> (-0.90%)
:arrow_down:

models/asymkey/gpg_key_common.go
59.67% <0.00%> (-4.84%)
:arrow_down:

services/pull/pull.go
41.70% <0.00%> (-0.41%)
:arrow_down:

modules/queue/workerpool.go
48.85% <0.00%> (+1.52%)
:arrow_up:

modules/queue/queue_channel.go
96.66% <0.00%> (+5.00%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba9e4e2...6545408. Read the comment docs.

What does "agit" refer to?

AGit-flow , eg https://git-repo.info/en/2020/03/agit-flow-and-git-repo/

@a1012112796 Hi, I'm trying out git-repo tool with the latest commit of Gitea, after I run git pr and edit the PR message, there's an error:
----------------------------------------------------------------------
[FAILED] ./              master
       (cannot find helper 'git-repo-helper-proto-gitea')

Can you tell me what's missing? Thanks!

Please create a new issue.


sumsug android 14 after PlatformView from background  to foreground, it will become to disappear, because surface has been release , it need to recreate.

Steps to reproduce

setup an PlatformView
start app. and let app turn to background
after a while, turn app to foreground
the platformview will become to disappear

Expected results
platformview don't  disappear
Actual results
platformview disappear
Code sample
like FlutterWebviewPlugin with Surface mode.
Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Closing this as a duplicate of https://github.com/flutter/flutter/issues/139039.
Kindly subscribe to the original issue for updates.
Thank you


mongo_db_plugin: It takes at least 15 minutes to update a transaction to an irreversible state.

It takes at least 15 minutes to update a transaction to an irreversible state.
And during this time, there is no block num and block id in the transaction data.

It is normally around 330 block time for a new block to became irreversible, which is around 160 seconds, or 2.5 minutes. And irreversible time has nothing to do with mongod_db_plugin. Why do you experience a 15 minute delay?

The version of nodeos is 1.2.5, and the last_irreversible_block_num is 17378133.
db.transactions.findOne({trx_id:"582b73ddeb5459df6fc4d052acc512d6dd7cd87b0f20c073c3949452a1a6cdc0"});
{
	"_id" : ObjectId("5ba35a223b1a73144b9746c6"),
	"trx_id" : "582b73ddeb5459df6fc4d052acc512d6dd7cd87b0f20c073c3949452a1a6cdc0",
	"accepted" : true,
	"actions" : [
		{
			"account" : "eosio.token",
			"name" : "transfer",
			"authorization" : [
				{
					"actor" : "hexiaofei124",
					"permission" : "active"
				}
			],
			"data" : {
				"from" : "hexiaofei124",
				"to" : "eosbiggame11",
				"quantity" : "0.7000 EOS",
				"memo" : ""
			},
			"hex_data" : "4044706a51e3ba6a1082928631773055581b00000000000004454f530000000000"
		},
		{
			"account" : "eosio.null",
			"name" : "nonce",
			"authorization" : [ ],
			"data" : "153743209798178832"
		}
	],
	"context_free_actions" : [ ],
	"context_free_data" : [ ],
	"createdAt" : ISODate("2018-09-20T08:28:18.901Z"),
	"delay_sec" : 0,
	"expiration" : "2018-09-20T08:29:17",
	"implicit" : false,
	"max_cpu_usage_ms" : 0,
	"max_net_usage_words" : 0,
	"ref_block_num" : 10481,
	"ref_block_prefix" : 508163308,
	"scheduled" : false,
	"signatures" : [
		"SIG_K1_Kjc7xdpENqw4HJaLW7vJGaUihqVHLYxQV3P2NdJn5SnyR5RHjPg2LvsBZLpPD7DncT6ox1V1odc8fRcyLn9MhHpsLNaqNh"
	],
	"signing_keys" : {
		"0" : "EOS83xcyYqYEdmsFVmgt55AJdA133CpDz6M2xoLH8ZTfCTNSR6e2D"
	},
	"transaction_extensions" : [ ]
}

There is no block id and block num in the transaction , and I don't known why.

same error

same error still here,when I update to v1.5.2.
refer to #6528


ListView displays in reverse order

This can be observed in UIExplorer.

I have been experiencing that same issue since the first time at tried
react native desktop too.
On Fri, Sep 30, 2016, 14:30 Lucas Still<EMAIL_ADDRESS>wrote:

This can be observed in UIExplorer.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ptmt/react-native-macos/issues/120, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAGKgbkGYHUN48jnQ7SzA1pVO2MLCYoRks5qvQDngaJpZM4KLCUC
.

In /Libraries/CustomComponents/ListView/ListView.js
a) The prop autoScrollToBottom is a misnomer
b) If I switch the order in the "prepend items" section, it appears to render correctly
if (this.props.autoScrollToBottom) {   bodyComponents.push(row); } else {   bodyComponents = [row].concat(bodyComponents); }

You answered quicker than I have fixed. : ) Yes, this is because of autoScrollToBottom property (there is the other line too with reverse index).

This hack was introduced because ios solution to achieve "chat window reverse order" (translateY: -1) doesn't work well on macOS. It seems broken after a pretty inaccurate merge with latest RN and hasn't got properly tested. Thank you again for reporting.

I can't even begin to explain how much of a positive performance impact this has made on my app. Now listviews only render rows once. :)

Don't hesitate to open as many issues as you want here and feel free to reach me out in any way. I don't have many hours dedicated to RN macOS right now, but I still have big plans and willing to fix bugs and improve performance. And sometimes a bug can turn out into a simple fix as this : )

A console.log accidentally got left in the commit/fix.

fixed
On Fri, 30 Sep 2016 at 17:28, Lucas Still<EMAIL_ADDRESS>wrote:

A console.log accidentally got left in the commit/fix.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ptmt/react-native-macos/issues/120#issuecomment-250758655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9SU8IDmpxH1eXQwP611IElYO_RtIPXks5qvRyFgaJpZM4KLCUC
.


Add note about firewalld on CentOS 6

Since firewalld does not exist for CentOS 6, the Drupal VM firewall_disable_firewalld setting, which defaults to true, must be overridden in config.yml and set to false.
Also, this commit organizes the CentOS 6 notes into bullets.

Looks great, thanks!


Feat auth supported for clients

proxy supported AUTH cmd for clients

Codecov Report

Merging #111 into master will decrease coverage by 0.19%.
The diff coverage is 44.64%.

@@            Coverage Diff            @@
##           master     #111     +/-   ##
=========================================
- Coverage   49.47%   49.27%   -0.2%
=========================================
  Files          62       62
  Lines        6280     6307     +27
=========================================
+ Hits         3107     3108      +1
- Misses       2826     2849     +23
- Partials      347      350      +3

Impacted Files
Coverage Δ

proxy/config.go
25% <ø> (ø)
:arrow_up:

proxy/proxy.go
0% <0%> (ø)
:arrow_up:

proxy/handler.go
0% <0%> (ø)
:arrow_up:

proxy/proto/redis/request.go
47.05% <0%> (-2.33%)
:arrow_down:

proxy/proto/redis/cluster/proxy_conn.go
60.6% <0%> (ø)
:arrow_up:

proxy/proto/redis/proxy_conn.go
72.63% <54.34%> (-8.81%)
:arrow_down:

proxy/proto/pipe.go
82.64% <0%> (-0.83%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb1e8a6...fe617d5. Read the comment docs.


[G000549] Rep. Jim Gerlach

Rep. Jim Gerlach:
http://gerlach.house.gov
Likely contact form URL to start with:
http://www.house.gov/writerep

YMAL file not found. I am working on it.


Feature Request

Is it possible to make it deployable on (https://vercel.com/)?
Thanks for your hard work on this project.

Hello, Vercel is suitable for deploying front-end projects. However, we also have back-end project that may not run well on Vercel.


nixos/all-firmware: make wirelessRegulatoryDatabase default clearer

Description of changes
sometimes people set the following maybe thinking that the the wireless db is not being loaded despite that it is actually being loaded. Lets update the defaulText to better reflect that.
hardware = {
  enableRedistributableFirmware = true;
  wirelessRegulatoryDatabase = true;
};

alternatively we could change the default value to be depending on the two config options and drop the mkMerge
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Already part of https://github.com/NixOS/nixpkgs/pull/262565

Good catch!


Adding custom blocks to the Creative inventory makes the game crash when the inventory is opened (BL 1.11 stable)

I tried adding a custom block to the Creative inventory with Player.addItemCreativeInv() in the latest Blocklauncher (1.11 stable) and this causes a crash everytime I try to open the Creative inventory.

So this only happens with blocks, not items, right?

Yes, items work perfectly. Adding blocks causes this crash.

@zhuowei you should've released BlockLauncher 1.11 in December so that you could fix all bugs

Should be fixed in the latest beta - give it a try?

In the latest beta (1.11 beta 3) this bug doesn't seem to be fixed. Items work while adding just a simple block makes the game crash when opening the inventory.
Tried with two blocks with this code:
// cubes
const CUBE_NORMAL_ID = 232;
Block.newBlock(CUBE_NORMAL_ID, "Cube", "cubenormal");
Block.setDestroyTime(CUBE_NORMAL_ID, 1);

const CUBE_COMPANION_ID = 233;
Block.newBlock(CUBE_COMPANION_ID, "Companion Cube", "cubecompanion");
Block.setDestroyTime(CUBE_COMPANION_ID, 1);

And using Player.addItemCreativeInv() with these two blocks (and some items too) in newLevel.

Forgot to say that Block.newBlock it's a custom function to catch texture pack errors. It just act like Block.defineBlock:
Block.newBlock = function(id, name, textureNames, sourceId, opaque, renderType)
{
	try
	{
		Block.defineBlock(id, name, textureNames, sourceId, opaque, renderType);
	} catch(e)
	{
		// user hasn't installed the texture pack
		if(!textureUiShowed)
			pleaseInstallTextureUI();

		Block.defineBlock(id, name, "enchanting_table_top", sourceId, opaque, renderType);
	}
}

@zhuowei I tried your piston mod and it add blocks in creative without any crash. But I don't know why my blocks make the game crash. Tried also to move the code from newLevel to outside any function like you did, but it didn't work.

This is a little test mod I did with only one block and it make the game crashes.
Really don't know why https://gist.github.com/Desno365/697d58021d8f3f3c5229

After 100 and more tries, I figured out that adding a block to the creative inventory without setting the item category makes the game crash. So the code that I just needed to add was Item.setCategory(CUBE_COMPANION_ID, ItemCategory.TOOL);

there is no latest beta linked @zhuowei


package.json missing?

Why there is no package.json in this repository?

Is it necessary? NPM should download it remotely from here:
https://www.npmjs.com/package/node-binance-api
Here is the contents:
{
  "name": "node-binance-api",
  "version": "0.2.11",
  "description": "Binance API for node https://github.com/jaggedsoft/node-binance-api",
  "main": "node-binance-api.js",
  "dependencies": {
    "request": "^2.83.0",
    "ws": "^3.2.0"
  },
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "repository": {
    "type": "git",
    "url": "git://github.com/jaggedsoft/node-binance-api.git"
  },
  "keywords": [
    "binance",
    "api"
  ],
  "author": "Jon Eyrick<EMAIL_ADDRESS>(https://nextlocal.net)",
  "license": "MIT",
  "bugs": {
    "url": "https://github.com/jaggedsoft/node-binance-api/issues"
  },
  "homepage": "https://github.com/jaggedsoft/node-binance-api#readme"
}

The thing is that using an API wrapper where I will put API keys is already quite a potential security flaw. Having a package.json would allow people to easily fork the repository and use their version of the code, which is guaranteed to be secure, as they could review it. It is slightly more difficult to verify that the version of this module published to NPM is actually secure/not malicious.
Also, I just don't understand why not to follow the convention everyone follows regarding NPM modules. If this is an NPM module, not sure why the package.json is not added to the repo.


why windows service start is error 1053

my service have a "json config" file eg: appsetting.json.
but
my service is installed in D://myservicefolder/
so i start windows service is error 1053 error
so must config file in  c:/windows/system32

In windows, services don't have a configurable working directory (always
system32). so you need to use "os.Executable" to get the current executable
path, then join to the application config.
On Tue, Jun 29, 2021 at 10:47 AM zhouhao @.***> wrote:

my service have a "json config" file eg: appsetting.json.
but
my service is installed in D://myservicefolder/
so i start windows service is error 1053 error
so must config file in c:/windows/system32
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kardianos/service/issues/280, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFYLMPRR73N7COF6NI4AIDTVHTKFANCNFSM47QNZS4Q
.

Tks,I have a try, tks...
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Daniel Theophanes @.>
发送时间: Tuesday, June 29, 2021 11:53:37 PM
收件人: kardianos/service @.>
抄送: zhouhao @.>; Author @.>
主题: Re: [kardianos/service] why windows service start is error 1053 (#280)
In windows, services don't have a configurable working directory (always
system32). so you need to use "os.Executable" to get the current executable
path, then join to the application config.
On Tue, Jun 29, 2021 at 10:47 AM zhouhao @.***> wrote:

my service have a "json config" file eg: appsetting.json.
but
my service is installed in D://myservicefolder/
so i start windows service is error 1053 error
so must config file in c:/windows/system32
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kardianos/service/issues/280, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFYLMPRR73N7COF6NI4AIDTVHTKFANCNFSM47QNZS4Q
.

―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/kardianos/service/issues/280#issuecomment-870718436, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH56OJVQENPYHAUXWR6HP7TTVHUADANCNFSM47QNZS4Q.


Optimize waveform creation in store

load spike after spike instead of cluster after cluster (to preserve IO contiguity)

YAGNI (HDD-only)

Fetch waveform subsets regularly from the raw data during the creation of the cluster store before storing them cluster-by-cluster. cc @nsteinme

in fact @nsteinme doing waveforms_excerpt_size = None in your user_settings.py should suffice no?

Sorry to be slow but I didn't test this yet...

@nippoo will do a prerelease and you can test it then


When the cumulative number of user connections reaches a certain amount, guacd will report an error. "RDP server closed/refused connection: Security negotiation failed (wrong security type?)”

Local host: | Windows host
-- | --
Windows host
1.5.4
rdp
When the cumulative number of user connections reaches a certain amount, guacd will report an error: "RDP server closed/refused connection: Security negotiation failed (wrong security type?)"Feb  4 16:51:48 guacd[2038]: Creating new client for protocol "rdp"
Feb  4 16:51:48 guacd[2038]: Connection ID is "$514ed1c3-2756-4d9f-8153-6388354873cc"
Feb  4 16:51:48 guacd[991760]: Security mode: RDP
Feb  4 16:51:48 guacd[991760]: Resize method: reconnect
Feb  4 16:51:48 guacd[991760]: No clipboard line-ending normalization specified. Defaulting to preserving the format of all line endings.
Feb  4 16:51:48 guacd[991760]: User "@5259dffd-eb02-42df-9b82-18de1b87457f" joined connection "$8ea080a2-1da5-4221-a828-823160906743" (1 users now present)
Feb  4 16:51:48 guacd[991760]: Loading keymap "base"
Feb  4 16:51:48 guacd[991760]: Loading keymap "en-us-qwerty"
Feb  4 16:51:48 guacd[991764]: Security mode: RDP
Feb  4 16:51:48 guacd[991764]: Resize method: reconnect
Feb  4 16:51:48 guacd[991764]: No clipboard line-ending normalization specified. Defaulting to preserving the format of all line endings.
Feb  4 16:51:48 guacd[991764]: User "@3814f8a9-f362-4b1e-9eb1-30fe6ed7c62b" joined connection "$514ed1c3-2756-4d9f-8153-6388354873cc" (1 users now present)
Feb  4 16:51:48 guacd[991764]: Loading keymap "base"
Feb  4 16:51:48 guacd[991764]: Loading keymap "en-us-qwerty"
Feb  4 16:51:48 guacd[991764]: RDP server closed/refused connection: Security negotiation failed (wrong security type?)
When you try to access again, an error occurs: "segfault at 20 ip 00007ff86708f848 sp 00007ff83affc9c0 error 4 in libdl-2.28.so[7ff86708f000+1000]"Feb  4 11:16:51 guacd[2507120]: Creating new client for protocol "rdp"
Feb  4 11:16:51 guacd[2507120]: Connection ID is "$35290a32-8188-4bbb-9d7d-5e181be83e25"
Feb  4 11:16:51 audit[3804338]: ANOM_ABEND auid=0 uid=0 gid=0 ses=8 pid=3804338 comm="guacd" exe="/usr/local/sbin/guacd" sig=11 res=1
Feb  4 11:16:51 kernel: [143626.086546] guacd[3804338]: segfault at 85 ip 00007f5a1d96a77d sp 00007f5951f929c0 error 4 in libdl-2.28.so[7f5a1d96a000+1000]
Feb  4 11:16:51 kernel: [143626.086550] Code: c3 0f 84 96 00 00 00 8b 53 04 48 8b 43 18 85 d2 0f 84 9d 00 00 00 48 85 c0 74 1f 48 8d 3d a2 08 00 00 b9 0e 00 00 00 48 89 c6 <f3> a6 0f 85 7b 01 00 00 48 c7 43 18 00 00 00 00 48 8b 04 24 e9 63
Feb  4 11:16:51 guacd[2507120]: Connection "$35290a32-8188-4bbb-9d7d-5e181be83e25" removed.

he fix has been merged to git. If you want to try making use of it now, you can build from source and test.
We hope to be able to use Guacamole version 1.5.4 soon.
1.5.4 is a point in time - it's unchanging. The fix for this and any other issue in 1.5.4 would be a future release. In this case, the fix is part of upcoming 1.5.5. There is no specific timeline for 1.5.5 except that it is nearing completion and expected soon.


Bug Report: crash on Freeplay (DiffCalc)

What version of the game are you using? Look in the bottom left corner of the main menu.
1.5.4 84d7b68af77fa02b2396558304f3d6277aff73e4
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
open Freeplay Menu
Please describe your issue. Provide extensive detail and images if possible.
The game crashes on https://github.com/KadeDev/Kade-Engine/blob/84d7b68af77fa02b2396558304f3d6277aff73e4/source/DiffCalc.hx#L41

anything else? cuz it doesn't crash for me so uh.

well I could try again

ok, apparently it works now


[DO-NOT-MERGE] Investigate PIP package failure

What changes were proposed in this pull request?
Seems like PIP packaging test is being failed. See https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/120218/testReport/
Why are the changes needed?
Does this PR introduce any user-facing change?
How was this patch tested?

Test build #120230 has started for PR 27995 at commit db53aa5.

Merged build finished. Test PASSed.

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

Test build #120230 has finished for PR 27995 at commit db53aa5.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120231 has started for PR 27995 at commit 857cdb4.

Merged build finished. Test PASSed.

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

Test build #120231 has finished for PR 27995 at commit 857cdb4.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

Test build #120232 has started for PR 27995 at commit fa9b0e5.

Test build #120232 has finished for PR 27995 at commit fa9b0e5.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120234 has started for PR 27995 at commit 37c32d1.

Merged build finished. Test PASSed.

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

Test build #120234 has finished for PR 27995 at commit 37c32d1.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120237 has started for PR 27995 at commit aa5647f.

Merged build finished. Test PASSed.

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

Test build #120237 has finished for PR 27995 at commit aa5647f.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120238 has started for PR 27995 at commit 570a08e.

Merged build finished. Test PASSed.

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

Test build #120238 has finished for PR 27995 at commit 570a08e.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120245 has started for PR 27995 at commit cee6d51.

Merged build finished. Test PASSed.

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

Test build #120245 has finished for PR 27995 at commit cee6d51.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120248 has started for PR 27995 at commit a1fdad8.

Seems a bug from the latest setuptools.

Merged build finished. Test PASSed.

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

Damn, okay. it was because of new setuptool release, specially this https://github.com/pypa/setuptools/commit/e7f29530df7595a876e4995f2011784ac9f43086.

Test build #120248 has finished for PR 27995 at commit a1fdad8.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120252 has started for PR 27995 at commit f5a60c6.

Merged build finished. Test PASSed.

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

Test build #120252 has finished for PR 27995 at commit f5a60c6.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120255 has started for PR 27995 at commit 01a1cad.

Merged build finished. Test PASSed.

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

Test build #120255 has finished for PR 27995 at commit 01a1cad.

This patch fails PySpark pip packaging tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #120256 has started for PR 27995 at commit 0c83437.

Merged build finished. Test PASSed.

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

Test build #120256 has finished for PR 27995 at commit 0c83437.

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

Merged build finished. Test PASSed.

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

Okay .. it passed. I'll clean up and merge to unblock other PRs.

Merged to master, branch-3.0, and branch-2.4.
I am not going to resolve the JIRA yet - this PR is a temp workaround at this momoent.

Test build #120259 has started for PR 27995 at commit b8926bd.

@nchammas, @srowen, @dongjoon-hyun, @BryanCutler, FWIW, I think this is one example of the trade-off we discussed at https://github.com/apache/spark/pull/27928.
setuptools is dev-only dependency but seems it found a pretty critical issue. Currently seems users can't use pip-installed PySpark assuming from pypa/setuptools#2041. However, of course it broke the PR builders.

Test build #120259 has finished for PR 27995 at commit b8926bd.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test PASSed.

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

Was there an issue in 46.1.0? wouldn't pinning to 46.0.0 have been a good thing then? or else I misunderstand what broke.

Hi, @HyukjinKwon .
This broke branch-2.4 due to the following. I'll revert this from branch-2.4.

https://amplab.cs.berkeley.edu/jenkins/view/Spark QA Test (Dashboard)/job/spark-branch-2.4-test-sbt-hadoop-2.6/804/

UnsatisfiableError: The following specifications were found to be in conflict:
  - python=3.5
  - setuptools=46.0.0 -> python[version='>=3.8,<3.9.0a0'] -> openssl[version='>=1.1.1d,<1.1.2a']
Use "conda search <package> --info" to see the dependencies for each package.

Since Apache Spark 2.4 still supports Python 2.7+/3.4+, setuptools 46.0.0 seems to be incompatible to that. Please make a separate PR for branch-2.4.

Perhaps the requirement should instead be of the form setuptools < 46.1.0, to allow flexibility for older versions of Python to still find a release of setuptools that satisfies their requirements.

Yeah, maybe I should better set upperbound instead. Thanks @dongjoon-hyun and @nchammas

@srowen, seems this is the problem in client side with setuptools.So, using lower version now fixed the test but the issue itself persists.

I'm still confused. This is an argument for pinning versions, right? Because a new version of setuptools caused a problem:

But we found that issue from the broken PR builder, which seems to be pretty critical.

I simply meant this is an example of the trade-off. It caused a problem and broke the PR builder vs it found a problem early.


Add initial mkdocs documentation website

As discussed with @axelson on Slack, this is the first version of the ElixirLS documentation website built using Mkdocs
At the moment, it contains everything a user requires to learn how to use and install ElixirLS for any supported editor, as well as intuitive navigation, links and universal search.

The GitHub Actions for automatic publishing of the website to Github Pages will be added in a future PR, as well as more screenshots and GIFs showcasing the features.
At the moment, the website can be manually served using mkdocs serve (for development) and built using mkdocs build.

@axelson added the changes you requested. Regarding the automated publishing of changes to the site - it looks like there is a new Github Action for this that greatly simplifies this, should be very easy to setup: https://github.com/jamesIves/github-pages-deploy-action

That new GitHub action looks like it'll be good. Let's add it to .github/workflows/release-asset.yml as a new job

@axelson updated with your comments. I set Github Actions to deploy the build website into the branch mkdocs. This is the one that you have to configure Github to present from.

I think next we can add the link to the site somewhere in the repo description, or maybe also in the org home. It might also be worth revisiting the README and removing a bunch of stuff that was migrated to the site anyway, and just leave a quick QuickStart guide and a link to the site.


Support HTMX or other popular html-enhanced libs

html_of_jsx is quite handy, but some people use the turbo cringe htmx, so it might be a good idea to support it.
I'm thinking on something like this (where htmx attributes are kebab_cased):
<button hx_post="/clicked" hx_swap="outerHTML">
  Click Me
</button>

Currently we support dataWhatever -> data-whatever transformation (camelCase to dashed-case) and most aria attributes work this way.
Which makes it a little rare, but I believe most htmx users wouldn't mind.
I'm thinking also of enabling htmx with a flag (or a separate ppx)

Support for htmx and other engines is on https://github.com/davesnx/html_of_jsx/pull/5


Add none

Now we'd have all, any, and none.  What would be a nice name for the
fourth and still missing variant?
Addresses #90

Codecov Report

Merging #91 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #91   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         736    739    +3
=====================================
+ Hits          736    739    +3

Impacted Files
Coverage Δ

dist/partial.lenses.cjs.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 508b71d...7484062. Read the comment docs.


Add function support for evaluating a dynamic path against a payload.

Add a function to support evaluating JSON path against JSON payloads, XPath against XML payloads, and perhaps Regex against String payloads. Make it generic to handle extension in future for other payload types.
Discussed briefly on Slack with @sanjiva

Hi @rbiw, thanks for the suggestion and we will consider this for one of our future release. In case you are interested, you can easily implement it by writing native functions which wraps java code. Then you can publish it to Ballerina central so that everyone can benefit from it.
Please refer following links where you can get an idea about how to implement native functions.
https://github.com/ballerina-platform/ballerina-lang/blob/master/stdlib/ballerina-builtin/src/main/ballerina/crypto/natives.bal
https://github.com/ballerina-platform/ballerina-lang/blob/master/stdlib/ballerina-builtin/src/main/java/org/ballerinalang/nativeimpl/crypto/Hash.java
Also please refer the contributing guide as well.
Please join our Slack channels so you can ask any question you have from Ballerina developers.

In addition to these it would be nice to support the query languages XPath and JSONiq for XML and JSON.

Any progress on this effort?

What is the slack link?
Showing 404 for https://ballerina.io/community/slack/

@dhayalramk Try this link: https://ballerina.io/community/slack

@pubudu91  Thank you :)


fix: fixed for the issue (#18)

1. Description

1.1. Checklist

[x] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[x] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[x] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

1.3. Related Issues

Codecov Report

Merging #81 (4bc8dae) into main (c874e88) will increase coverage by 0.50%.
The diff coverage is 100.00%.

:exclamation: Current head 4bc8dae differs from pull request most recent head 1f2eaa5. Consider uploading reports for the commit 1f2eaa5 to get more accurate results

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
+ Coverage   69.35%   69.85%   +0.50%
==========================================
  Files          85       85
  Lines         832      846      +14
==========================================
+ Hits          577      591      +14
  Misses        255      255

Impacted Files
Coverage Δ

lib/src/service/entities/featured_tag.dart
100.00% <ø> (ø)

...b/src/service/v1/accounts/accounts_v1_service.dart
97.93% <100.00%> (+0.22%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


c3-completions: docs: update to reflect recent changes

There have been some changes to c3-ctl which were yet to be reflected
in the bash completions or the documentation. Follow through with
these changes to keep things current.

Add foreground, interactive and attach options to start
Add rename/mv command
Remove --peer and --nested options no longer available to start
Remove restart command from documentation

Signed-off-by: Mark Asselstine<EMAIL_ADDRESS>

merged. will be pushed Monday morning with the consolidated changes.


Bad file permisson pin-utils-0.1.0/LICENSE-APACHE

My build failed because of:
19:10:33  + cargo build --all-targets --offline
19:10:33  error: failed to calculate checksum of: /cargo/vendor/pin-utils-0.1.0/LICENSE-APACHE
19:10:33
19:10:33  Caused by:
19:10:33    failed to open file `/cargo/vendor/pin-utils-0.1.0/LICENSE-APACHE`
19:10:33
19:10:33  Caused by:
19:10:33    Permission denied (os error 13)

I had to chmod it

This is a cargo bug. see also #37.


[image_picker] add image limit

for example:
picker.pickMultiImage(
 limit:20
)

Hi @Kizat, See
https://github.com/flutter/flutter/issues/85772#issuecomment-884137114
&
https://github.com/flutter/flutter/issues/85772#issuecomment-887614127
Due to platform limitations this feature cannot be implemented atm.
Closing as this is not an issue with image_picker plugin but a limitation of other underlying platforms.
Thank you


Annotations in geojson with same image

The annotations are associated with the same image. issue with the loop

fixed


Fix double execution of LeftClickCommand

This PR fixes an issue where the LeftClickCommand was executed twice. The redundant handling of the WM_LBUTTONUP message caused the command to be triggered multiple times. By removing the WM_LBUTTONUP message handling, the LeftClickCommand is now executed only once when the notification icon is clicked, also ensuring the correct behavior with NoLeftClickDelay enabled.

@DarkPro1337 your changes have been merged, thanks for your contribution 👍


fix(wizards/connectedap) Scope Address for ConnectedAP more narrowly, closes #1123

Thank you for the review.


Sending Email for TOTP is not working

Version: 5.10.0
When we try send email with TOTP as mentioned here https://is.docs.wso2.com/en/latest/learn/configuring-totp/ is not working

Workaround:
If we want to send an email for totp, Need to make two changes.

Need to remove ‘&’ character infront of sessiondataKey in https://github.com/wso2/identity-apps/blob/master/apps/totp-authentication-portal/src/main/webapp/js/scripts.js#L50

Need to change the TOTP template in management-console
Change {{user.claim.givenname}} to {{user-name}}

hi @piraveena
I think step 2 only is sufficient on WSO2 5.10.0
Thank you for the tips!

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!