Fix: Repair the logic for obtaining the configuration file path.

Fixed the logic for obtaining the configuration file path to ensure that config.json is used as the default when no command line arguments are provided.

感谢完善！


vscode unocss plugin doesn't work well with the transformer

I found that vscode unocss plugin doesn't work well with the unocss-preset-weapp/transformer, below is the reason:
First, the vscode unocss plugin use the getMatchedPositionsFromCode  function to extract all positions which need to be styled with a dashed underline:
async function getMatchedPositionsFromCode(uno, code, id = '') {
  const s = new MagicString(code)
  const tokens = new Set()
  const ctx = { uno, tokens }

  const transformers = uno.config.transformers?.filter(i => !ignoreTransformers.includes(i.name))
  for (const i of transformers?.filter(i => i.enforce === 'pre') || [])
    await i.transform(s, id, ctx)
  for (const i of transformers?.filter(i => !i.enforce || i.enforce === 'default') || [])
    await i.transform(s, id, ctx)
  for (const i of transformers?.filter(i => i.enforce === 'post') || [])
    await i.transform(s, id, ctx)
  const hasVariantGroup = !!uno.config.transformers?.find(i => i.name === '@unocss/transformer-variant-group')

  const result = await uno.generate(s.toString(), { preflights: false })
  console.log(result.matched)
  return getMatchedPositions(code, [...result.matched], hasVariantGroup, false, uno)
}

Result.matched is all class names that catched by unocss core engine.
Before unocss core engine generate the result, transformers are applied.
So, if the source code is <template><view class="text-#f00">123</view></template>, transformer will transform it to <template><view class="text-_wn_f00">123</view></template>.
However, text-_wn_f00 isn't in original source code which vscode use to find the positions.
This is why.
I found a workaround, but I doesn't know whether it should be adopted into this repo, so I didn't create a pr.
The workaround is:
When we transform the code, we also inject a comment, the comment's content is all class names collected while transforming. The comment's format depends on that the source code is .vue file or .jsx? file.
Here is the key code:
function transformCode(code, rules = defaultRules) {
  const classNames = getClass(code)

  classNames.forEach((c) => {
    let currentClass = c[0]
    c.slice(1).forEach((selector) => {
      currentClass = currentClass.replace(selector, transformSelector(selector, rules))
    })
    code = code.replace(c[0], currentClass)
  })

  const classSet = new Set(classNames.map(x => x[1]).filter(x => x).flatMap(x => x.split(' ')))
  return preserveOriginalClass(code, [...classSet])
}

function preserveOriginalClass(code, uniqueClassList) {
  const injectStr = uniqueClassList.join(' ')
  if (/<template>/.test(code)) {
    return code.replace('<template>', `<template>\n<!-- ${injectStr} -->\n`)
  } else {
    return `/* ${injectStr} */\n` + code
  }
}

Do you mean it should not be displayed as c_lfl__wn_157_lfr__60 ?

I understand what you mean

So it's because I wrote c-[#157]_60 twice in the demo, so I didn't notice this issue

Yes!


locationDelegation/PermissionRequestActivity extends Activity

Make PermissionRequestActivity extends Activity insteads of AppCompatActivity.
Also change minSdkVersion to 19.

@andreban PTAL Thanks


Instructions?

I think I have completed everything I was supposed to for PCM (short of issue #179 on which @danscime and I are still waiting for a reply) and I am having difficulty thinking of something to do next.  Are there any suggestions or should I just try to find work for myself?

Take a look in Drasil.SRS.   See that intro combinator?  There should be one like it for every section of an SRS.  And then those should be used.

Okay, found it but I am not really clear on where I need to use it.  Clarification?

In every example, every section starts with section (titleize' foo) [...].  That should be replaced with a combinator foo, and then use foo [...] directly.

We think we need to adjust the intro 'combinator' to work with the 'section' function instead of the 'Section' type.  Is it alright if we make the change?

Yes please.

Okay know what to do.  Thank you.  On this topic though, I am missing most of the sections to PCM and so have VERY little to change in this regard.  Is there anything else I can do after this?

Do PCM, and then Dan will be out of his current meeting and give you more.

Alright, thank you.


wait for lambda to become active during deployment

Many parts of the deployment process (e.g., scheduling certain events, setting up alb target groups) will fail if the lambda function is not in a ready state, this pr attempts to alleviate that by using a boto waiter during the lambda creation function.

+1 I can't deploy with an ALB because of this..

Yeah, I've been using this for a few weeks now over which time I've probably done a good 10+ deploys. No issues thus far although I am using ALB and have APIGateway disabled.

What's the status of the PR?
I really need it :pray:

I need to make the above fixes, it’s been on my mind so will try to get it sorted soon but the real blocker is the CI which is defunct right now, nothing can merge until we get that sorted imo.
Sent from my iPhone

On 16 Sep 2021, at 17:29, Sebastian Illing @.***> wrote:

What's the status of the PR?
I really need it 🙏
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

…I wonder if it is worth looking at moving the CI to GHA?  Thanks!


The SDK version is incompatible?

When I build fabric app, I got this error.
`
github.com/hyperledger/fabric-sdk-go/internal/github.com/hyperledger/fabric/core/operations
C:\Users\95762\go\pkg\mod\github.com\hyperledger\fabric-sdk-go@v1.0.0\internal\github.com\hyperledger\fabric\core\operations\system.go:225:23: not enough arguments in call to s.statsd.SendLoop
have (<-chan time.Time, string, string)
want (context.Context, <-chan time.Time, string, string)
`

When I build fabric app, I got this error. `
github.com/hyperledger/fabric-sdk-go/internal/github.com/hyperledger/fabric/core/operations
C:\Users\95762\go\pkg\mod\github.com\hyperledger\fabric-sdk-go@v1.0.0\internal\github.com\hyperledger\fabric\core\operations\system.go:225:23: not enough arguments in call to s.statsd.SendLoop have (<-chan time.Time, string, string) want (context.Context, <-chan time.Time, string, string) `

Hello, has this problem been solved？

I had the same issue.  It is due to go-kit 0.9.0 using context.  I made a local patch on fabric-sdk-go v1.0.0  in the file internal/github.com/hyperledger/fabric/core/operations/system/go by replacing go s.statsd.SendLoop(s.sendTicker.C, network, address) with go s.statsd.SendLoop(context.Background(), s.sendTicker.C, network, address).
Now it compiles again, even with go-kit v0.9.0.  I did not run the test suite on the fabric-sdk-go.  I just run my test suites on my apps that worked fine,
Of course, I updated the go mod to go-kit v0.9.0.

When I build fabric app, I got this error. `
github.com/hyperledger/fabric-sdk-go/internal/github.com/hyperledger/fabric/core/operations
C:\Users\95762\go\pkg\mod\github.com\hyperledger\fabric-sdk-go@v1.0.0\internal\github.com\hyperledger\fabric\core\operations\system.go:225:23: not enough arguments in call to s.statsd.SendLoop have (<-chan time.Time, string, string) want (context.Context, <-chan time.Time, string, string) `

Hello, has this problem been solved？

Yes, clear the compilation environment, and recompile.The problem has been solved.


see my events

I have some doubts. There is a way to delete an event? And which icon should I put on the sidebar to represent "my events"(I put the icon-check just to differentiate it from the "events")?


OutOfMemoryError

When a few videos are in the download queue,android Profiler in android studio shows 322.16 MB has been used and the app stops.
java.lang.OutOfMemoryError:Failed to allocate a 48 byte allocation with 3968 free bytes and 3KB until OOM.

@sn32247 What version of Fetch are you using?. Can you replicate? Share logs?. What devices were you testing on?


Poolable Components

Poolable Components are currently added to objects at runtime during instantiation by the Pooler.

This affects when references to the component are available.
Can't be on Awake!
Currently at OnEnable.

Choices:

Leave as is. It's not a real problem yet.
We could add the Component to the Prefabs of all Poolable objects:

In the Editor.
In the source code using [RequireComponent (typeof (Poolable))] attribute.

Reference: [RequireComponent (typeof (...))]

This would allow more flexibility in where initializations take place.

Option 3: Don't burden your components with logic regarding pooling. It would be cleaner if you could just call say RemoveFromGame or some similar simple and descriptive name, which deals with the plumbing in the background.
Before
After (base class deal with details)

You're getting ahead of me with these event shenanigans! :smiley: :
Good idea and use of events! :+1:
Your rewrite of Poolable.cs helps make your solution and comment  all clear to me now: it's event driven! :mega:
Link to Poolable.cs
A snippet:
public interface PoolableAware : IEventSystemHandler
{
    void PoolableAwoke(Poolable p);
}

void Awake()
{
        InstantiationGuard();
        ExecuteEvents.Execute<PoolableAware>(gameObject, null, (script, ignored) => script.PoolableAwoke(this));
}

Game objects now have a base behaviour class GameBehaviour.
They all become "aware" of their poolable status through events, executed on Awake in Poolable.
public class GameBehaviour : MonoBehaviour, PoolableAware, Recyclable
{
    Poolable poolable;
    void PoolableAware.PoolableAwoke(Poolable p) { poolable = p; }
    public void RemoveFromGame()
    {
        if (poolable)
            poolable.Recycle();
        else
            RequestDestruction();
    }
.
.
.


Final polishing script needed

After doing the semi auto alignment we need to then run a routine to cross match sources in the images (which should now be very accurate) and do one final wcs determination. This is necessary because the manual alignment can't take distortion into account when we apply an initial guess.
For the f/2 camera this seems to be essential.

Done in three different ways.
Users can use Astrometry.net, Scamp, or the built in gaia alignment. This will be available in the next release.


Log class name of class if failed to be instantiated

Currently if Cuppa fails to instantiate a test class it does not log the name of that class, only the fact that it failed to instantiate a test class. Logging the name of the test class would add in debugging issues.

Coverage remained the same at 89.222% when pulling ec3947a825c5549912448d5562be273d0daab9a8 on phillcunnington:add_test_class_that_failed_to_be_instantiated into eddd35501b62ed3670fd7f80c8a6ff0beb585b2b on cuppa-framework:master.

Ship it


List type and uniqueness of modules/dependencies

When I specify a list of modules, I could do it like:
let foo = { name = "foo", contents = "none" }
in { package = "bar"
    , modules = [ foo, foo, foo ]
    , dependencies = [ foo, { name = "foo", contents = "othercontents" } ] }

How should these cases be handled?
I inquired about introducing a Set datatype to dhall but that doesn’t play well with the non-existing equality checks.

Good question. I don't yet have an answer.
Originally, I wanted modules to be a homogeneous record, but I couldn't type it. A map, basically. Going to do some research.


How do I rip private SubReddits?

How do I rip private SubReddits?

As far as I know, that is not possible currently.


Can different cloudrun instances of the same application talk to each other?

Let's say I create a cloudrun application and it scales to 2 containers. Is there some metadata env vars available like in k8s that would allow the container to determine its own IP address so that it could advertise it and let the other instance to connect to it?

If sync messages, why not just use the app's host URL?  For async I would recommend Pub/Sub (if you need throughput and speed) or Cloud Tasks (if you need control over rate limit). The documentation recommends that you avoid background processes, though.

in my case I have some long lived connections opening and then some incoming requests need to be routed to the app instance that has the connection open from a device. It's mostly SSH tunnelling but building more on this functionality.
In k8s what I would do:

Once a device connects to the server, I put the device ID and pod IP into Redis
Once user tries to connect to the device and if I know that the device is not connected to the current instance that has accepted the connection, do a TCP forwarding to another pod that I know has the connected device (retrieve that info from Redis)

Not sure how would this work in CloudRun. I suspect that they must have some internal IPs available. Otherwise I would need a separate deployment for these long lived connections from devices.

There's no individual addressing of containers on Cloud Run.
We also don't show you individual container instances.


fc_run on SLURM system, crashing at daligner-split step

Operating system
Which operating system and version are you using?
CentOS Linux 7 kernel 3.10.0-957.21.2.el7.x86_64
Slurm scheduler
Package name
PB-Assembly 0.0.6
Conda environment
What is the result of conda list? (Try to paste that between triple backticks.)
# packages in environment at miniconda3:
#
# Name                    Version                   Build  Channel
asn1crypto                0.24.0                   py37_0
blas                      1.0                         mkl
bzip2                     1.0.6                h470a237_2    conda-forge/label/cf201901
ca-certificates           2019.5.15                     0    anaconda
certifi                   2019.3.9                 py37_0    anaconda
cffi                      1.12.2           py37h2e261b9_1
chardet                   3.0.4                    py37_1
conda                     4.6.14                   py37_0    anaconda
cryptography              2.7              py37h1ba5d50_0    anaconda
cudatoolkit               10.0.130                      0
cudnn                     7.6.0                cuda10.0_0
cycler                    0.10.0                     py_1    conda-forge/label/cf201901
dbus                      1.13.2               h714fa37_1    anaconda
expat                     2.2.5                hfc679d8_2    conda-forge/label/cf201901
fontconfig                2.13.1               h65d0f4c_0    conda-forge/label/cf201901
freetype                  2.9.1                h6debe1e_4    conda-forge/label/cf201901
gettext                   <IP_ADDRESS>             h5e8e0c9_1    conda-forge/label/cf201901
glib                      2.56.2               h464dc38_1    conda-forge/label/cf201901
gst-plugins-base          1.14.0               hbbd80ab_1    anaconda
gstreamer                 1.14.0               hb453b48_1    anaconda
icu                       58.2                 hfc679d8_0    conda-forge/label/cf201901
idna                      2.8                      py37_0
intel-openmp              2019.4                      243
jpeg                      9c                   h470a237_1    conda-forge/label/cf201901
kiwisolver                1.0.1            py37h2d50403_2    conda-forge/label/cf201901
libedit                   3.1.20181209         hc058e9b_0
libffi                    3.2.1                hd88cf55_4
libgcc-ng                 8.2.0                hdf63c60_1
libgfortran-ng            7.3.0                hdf63c60_0
libiconv                  1.15                 h470a237_4    conda-forge/label/cf201901
libpng                    1.6.36               ha92aebf_0    conda-forge/label/cf201901
libstdcxx-ng              8.2.0                hdf63c60_1
libtiff                   4.0.10               h2733197_2
libuuid                   2.32.1               h470a237_2    conda-forge/label/cf201901
libxcb                    1.13                 h470a237_2    conda-forge/label/cf201901
libxml2                   2.9.8                h422b904_5    conda-forge/label/cf201901
matplotlib                3.0.2            py37h8a2030e_1    conda-forge/label/cf201901
matplotlib-base           3.0.2            py37h20b835b_1    conda-forge/label/cf201901
mkl                       2019.4                      243
mkl_fft                   1.0.12           py37ha843d7b_0
mkl_random                1.0.2            py37hd81dba3_0
ncurses                   6.1                  he6710b0_1
ninja                     1.9.0            py37hfd86e86_0
numpy                     1.16.4           py37h7e9f1db_0
numpy-base                1.16.4           py37hde5b4d6_0
olefile                   0.46                     py37_0
openssl                   1.1.1                h7b6447c_0    anaconda
pandas                    0.24.2           py37he6710b0_0    anaconda
pcre                      8.43                 he6710b0_0    anaconda
pillow                    6.0.0            py37h34e0f95_0
pip                       19.0.3                   py37_0
pthread-stubs             0.4                  h470a237_1    conda-forge/label/cf201901
pycosat                   0.6.3            py37h14c3975_0
pycparser                 2.19                     py37_0
pyopenssl                 19.0.0                   py37_0
pyparsing                 2.3.0                      py_0    conda-forge/label/cf201901
pyqt                      5.6.0            py37h8210e8a_8    conda-forge/label/cf201901
pysocks                   1.6.8                    py37_0
python                    3.7.3                h0371630_0    anaconda
python-dateutil           2.7.5                      py_0    conda-forge/label/cf201901
pytorch                   1.1.0           py3.7_cuda10.0.130_cudnn7.5.1_0    pytorch
pytz                      2019.1                     py_0    anaconda
qt                        5.6.3                h8bf5577_3    anaconda
readline                  7.0                  h7b6447c_5
requests                  2.21.0                   py37_0
ruamel_yaml               0.15.46          py37h14c3975_0
setuptools                41.0.0                   py37_0
sip                       4.18.1           py37hfc679d8_0    conda-forge/label/cf201901
six                       1.12.0                   py37_0
sqlite                    3.27.2               h7b6447c_0
tk                        8.6.8                hbc83047_0
torchvision               0.3.0           py37_cu10.0.130_1    pytorch
tornado                   5.1.1            py37h470a237_0    conda-forge/label/cf201901
urllib3                   1.24.1                   py37_0
wheel                     0.33.1                   py37_0
xorg-libxau               1.0.8                h470a237_6    conda-forge/label/cf201901
xorg-libxdmcp             1.1.2                h470a237_7    conda-forge/label/cf201901
xz                        5.2.4                h14c3975_4
yaml                      0.1.7                had09818_2
zlib                      1.2.11               h7b6447c_3
zstd                      1.3.7                h0b5b093_0

Describe the bug
A clear and concise description of what the bug is.
I am running falcon assembler via fc_run.  Job is crashing at 0-rawreads/dalinger-split step
My fc.cfg file is:
#### Input
[General]
input_fofn=input.fofn
input_type=raw
pa_DBdust_option=
pa_fasta_filter_option=streamed-median
target=assembly
skip_checks=false
LA4Falcon_preload=false
use_tmpdir=false

#### Data Partitioning
pa_DBsplit_option=-x500 -s200
ovlp_DBsplit_option=-x500 -s200

#### Repeat Masking
pa_HPCTANmask_option=
pa_REPmask_code=0,300;0,300;0,300

####Pre-assembly
genome_size = 800000000
seed_coverage = 40
length_cutoff = -1
pa_HPCdaligner_option=-v -B128 -M24
pa_daligner_option= -k18 -e0.75 -l1200 -h256 -w8 -s100
falcon_sense_option=--output-multi --min-idt 0.70 --min-cov 4 --max-n-read 200
falcon_sense_greedy=False

####Pread overlapping
ovlp_HPCdaligner_option=-v -B128 -M24
ovlp_daligner_option=-k24 -e.92 -l1800 -h600 -s100

####Final Assembly
length_cutoff_pr=1000
overlap_filtering_setting=--max-diff 100 --max-cov 100 --min-cov 2
fc_ovlp_to_graph_option=
length_cutoff_pr=10000

[job.defaults]
job_type=slurm
pwatcher_type=blocking
JOB_QUEUE =short
NPROC = 6
njobs = 10
submit = sbatch -p ${JOB_QUEUE} -J ${JOB_NAME} -o ${JOB_STDOUT} -e ${JOB_STDERR} --cpus-per-task=${NPROC} --wait ${JOB_SCRIPT}

[job.step.da]
NPROC=4
njobs=10
[job.step.la]
NPROC=4
njobs=10
[job.step.cns]
NPROC=8
njobs=10
[job.step.pda]
NPROC=4
njobs=10
[job.step.pla]
NPROC=4
njobs=10
[job.step.asm]
NPROC=24
njobs=1

Error message
Paste the error message / stack.
The last few lines of stderr are, the key is SyntaxError:more than 255 arguments :
[INFO]Num satisfied in this iteration: 6
[INFO]Num still unsatisfied: 1
[INFO]About to submit: Node(0-rawreads/repc/rep-gathered)
[INFO]Popen: '/bin/bash -C /software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/pwatcher/mains/job_start.sh >| /project/ag100pest/Scal/Falcon/0-rawreads/repc/rep-gathered/run-P6dfaabcd965906.bash.stdout 2>| /project/ag100pest/Scal/Falcon/0-rawreads/repc/rep-gathered/run-P6dfaabcd965906.bash.stderr'
[INFO]recently_satisfied:
{Node(0-rawreads/repc/rep-gathered)}
[INFO]Num satisfied in this iteration: 1
[INFO]Num still unsatisfied: 0
[INFO]Num unsatisfied: 2, graph: 3014
[INFO]About to submit: Node(0-rawreads/repc/rep-combine)
[INFO]Popen: '/bin/bash -C /software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/pwatcher/mains/job_start.sh >| /project/ag100pest/Scal/Falcon/0-rawreads/repc/rep-combine/run-P165057521ca462.bash.stdout 2>| /project/ag100pest/Scal/Falcon/0-rawreads/repc/rep-combine/run-P165057521ca462.bash.stderr'
[INFO](slept for another 0.0s -- another 1 loop iterations)
[INFO](slept for another 0.30000000000000004s -- another 2 loop iterations)
[INFO](slept for another 1.2000000000000002s -- another 3 loop iterations)
[INFO]recently_satisfied:
{Node(0-rawreads/repc/rep-combine)}
[INFO]Num satisfied in this iteration: 1
[INFO]Num still unsatisfied: 1
[INFO]About to submit: Node(0-rawreads/daligner-split)
[INFO]Popen: '/bin/bash -C /software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/pwatcher/mains/job_start.sh >| /project/ag100pest/Scal/Falcon/0-rawreads/daligner-split/run-P2f034f6e535eee.bash.stdout 2>| /project/ag100pest/Scal/Falcon/0-rawreads/daligner-split/run-P2f034f6e535eee.bash.stderr'
[INFO](slept for another 1.4s -- another 4 loop iterations)
[INFO](slept for another 2.0s -- another 5 loop iterations)
[INFO](slept for another 5.7s -- another 6 loop iterations)
[INFO]recently_satisfied:
{Node(0-rawreads/daligner-split)}
[INFO]Num satisfied in this iteration: 1
[INFO]Num still unsatisfied: 0
Traceback (most recent call last):
  File "/software/7/apps/pb-assembly/0.0.6/bin/fc_run", line 11, in <module>
    load_entry_point('falcon-kit==1.4.4', 'console_scripts', 'fc_run')()
  File "/software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/falcon_kit/mains/run1.py", line 706, in main
    main1(argv[0], args.config, args.logger)
  File "/software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/falcon_kit/mains/run1.py", line 73, in main1
    input_fofn_fn=input_fofn_fn,
  File "/software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/falcon_kit/mains/run1.py", line 293, in run
    daligner_split_script=pype_tasks.TASK_DB_DALIGNER_SPLIT_SCRIPT,
  File "/software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/falcon_kit/mains/run1.py", line 524, in add_daligner_and_merge_tasks
    dist=Dist(NPROC=4, MB=4000, job_dict=daligner_job_config),
  File "/software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/falcon_kit/pype.py", line 194, in gen_parallel_tasks
    dist=Dist(local=True, use_tmpdir=False),
  File "/software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/pypeflow/tasks.py", line 60, in gen_task
    validate_dict(inputs)
  File "/software/7/apps/pb-assembly/0.0.6/lib/python3.6/site-packages/pypeflow/tasks.py", line 56, in validate_dict
    collections.namedtuple('validate', list(mydict.keys()))
  File "/software/7/apps/pb-assembly/0.0.6/lib/python3.6/collections/__init__.py", line 429, in namedtuple
    exec(class_definition, namespace)
  File "<string>", line 12
SyntaxError: more than 255 arguments

To Reproduce
Steps to reproduce the behavior. Providing a minimal test dataset on which we can reproduce the behavior will generally lead to quicker turnaround time!
Expected behavior
A clear and concise description of what you expected to happen.

The conda-env that you've listed does not include any pb-* packages, leaving open the possibility that you are out-of-date.
python3.6 -- We support only python3.7. But I see python 3.7.3 in your conda-env. So I think you are running Falcon from elsewhere, not from your conda-environment.
That means this is not a "Slurm" problem at all. You should always run locally (on a small dataset) before wasting any time at all with job-distribution. If there is still a bug, feel free to re-open this.


Migrate Admin Tests

LGTM! I think the usage of t.Parallel() is good, given that the tests use different clusters & namespaces.


Fixed bug where parent fields could not be merged before saving.

Add last modified date and user when init user table.
Fixed bug where parent fields could not be merged before saving.

getClass().getDeclaredFields() could not resolve parent fields.

LGTM


public pages: pagination first item goes to org. profile instead of user

when user is part of an organization, if you go to next pages, i.e.
https://team.cartodb.com/u/matallo/page/2
and then want to come back to first page, the href in the link goes to org. profile instead of user (find attached)

I think this is duplicated :)

oop, sorry then


Requested Changes with in GraphicsOverlay

Hi there,
we have 2 problems using GraphicsOverlay,

Its Sealed - meaning we cant extend it and add our own stuff in it. for example property Name  Can it be change?
The Id property is not raising PropertyChange. Any change to add PropertyChange into it?

Thanks,
Shahar.

The ID is meant as an internal indexer for finding a overlay, not as a name in a ToC item.
I'd suggest you create a view model around your graphics overlays that accommodates the features you need for binding into your view, and allow you to do anything that provides model properties optimized for binding into a view.
Ie something along the lines of:
public class GraphicsOverlayCollectionViewModel : IEnumerable<GraphicsOverlayViewModel>
{
         public GraphicsOverlayCollectionViewModel(GraphicsOverlayCollection overlays)
         {
            // ...
         }
}

Feedback is noted though, but I'll close it here, as this repo is specific for the samples.


Labels & Context filters refactoring

Impact

[x] Bug fix (non-breaking change which fixes expected existing functionality)
[x] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Fixes #5175

Tickets discovered:

https://github.com/liquibase/liquibase/issues/5458
Doc update required: https://datical.atlassian.net/browse/PD-4084

Things to be aware of

Things to worry about

Additional Context

@MalloD12 this one has been stalled since January.. do you plan to still working on that?

Hey @filipelautert,
This refactoring is pretty big and could be a breaking change. It would still require some more testing and probably some extra code change. That's why since it doesn't seem to be something urgent I leave it in the backlog. Do you think this is something worth we should plan and addressing in some upcoming sprint?
Thanks,
Daniel.


Do not enable TLSv1 if it is not a supported protocol

Motivation
Our application runs in an Android 4.2 environment where TLSv1 has been disabled for security compliance. The recent code change in #14245 enables TLSv1 in all environments, without first checking to see if that version of the protocol is supported. This causes an exception:
FATAL EXCEPTION: OkHttp Dispatcher java.lang.IllegalArgumentException: protocol TLSv1 is not supported                                                                              at org.apache.harmony.xnet.provider.jsse.NativeCrypto.checkEnabledProtocols(NativeCrypto.java:569)                                                                              at org.apache.harmony.xnet.provider.jsse.OpenSSLSocketImpl.setEnabledProtocols(OpenSSLSocketImpl.java:782)                                                                              at com.facebook.react.modules.network.TLSSocketFactory.enableTLSOnSocket(TLSSocketFactory.java:74)                                                                              at com.facebook.react.modules.network.TLSSocketFactory.createSocket(TLSSocketFactory.java:49) <... remaining stack omitted for brevity...>
Test Plan
I rebuilt the app with a patched version of React Native 0.50, and it runs successfully.
Related PRs
#14245
Release Notes
[ANDROID] [BUGFIX] [Network] - Enable TLSv1 only if supported by the OS

@urbane Fair enough. Is it common that "TSLv1.1" is not supported also?

@newyankeecodeshop I agree that it probably isn't a common concern at this time, but the problem is that it would break if the device didn't support either.  Given some of the vendors I've worked with, I wouldn't be terribly surprised if they randomly decided to enable or disable support.
An intersection is more expensive, but I'm not aware of another solution that would work in every environment.

Hey @newyankeecodeshop , Is this issue resolved for you?


Use old function for the old php version on the server.

Can we update the php version lol? Or will that break everything

it won't break everything, but it is more invasive.

we can definitely upgrade to php 8 if we really, really wanted to, but i definitely don't really, really want to.  :D


Update vitest config

Resolves #134
Make a few changes to our vitest config:

Added a test:coverage script to run tests with coverage. This shows the coverage report in the terminal and generated a html report in coverage folder
Removed c8 and replaced it with istanbul which will be used to generate the coverage. This was mainly done because there were issues getting the default c8 to properly generate coverage as no files were displayed in the coverage. Even with the all files setting, none of the files showed anything being covered by tests
Added coverage folder to .gitignore
Updated watchExclude to exclude our mysql-data folder
Removed redundant include in vitest.config

As discussed in the meeting, I think it would be beneficial to see what files are not covered in app/models by unit tests.

As discussed in the meeting, I think it would be beneficial to see what files are not covered in app/models by unit tests.

I have filed #420 to add coverage for playwright tests, and I am planning to update this coverage based on that


Feature/alg 558 ssd embed anchors

It's safer to store anchors inside of a checkpoint and more correct to convert preds to boxes and scores inside of a mode

On hold, waiting for some fixes yamls to arrive from master. Want to see everything training with the proposed changes before we merge it, but for now it cannot be trained even in master


Graph not generated

Hi
I am using your image to deploy my vesta server. Other working great but not showing the image on the Graph section.
Thanks

Did you check your folder permissions.

Check my Screenshot. It works out of the box.

http://vikramrao.com/screenshot.jpg

Hi Vikram,
Thank for the response, which folder nee dto check, in user folders or /usr/local/vesta/rrd/ ?

Use this shell script. It will handle everything for you. In case you are having trouble installing it on your current configured server, just add '-f' (without quotes) at the end of install command. Works perfectly.

Wich script?

http://vestacp.com/pub/vst-install.sh

This issue is valid and related to the docker container.  Don't run VikramRao command.  Run this one instead:
sudo sed -i -e "s/if (\$dir_name/\/\/if (\$dir_name/g" /usr/local/vesta/web/list/rrd/image.php

Hi Noogen,
No, i am aware that the script for automatic fresh instalation.

sudo sed -i -e "s/if ($dir_name///if ($dir_name/g" /usr/local/vesta/web/list/rrd/image.php

using that the graph was shown.
Thank you

Also we may need a documentation to remove the unused package since every user have diffrent preference.


Fix UDP ports mapping

[x] I understand that contributing to this repository will require me to agree with the CLA

Description
The default port type is tcp, so explicitly specify udp in config.

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

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 tech-docs
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

I have read the CLA Document and I hereby sign the CLA

@any contributor @CzBiX infra


Merge pull request #3 from openaps/master

Updating to latest

Swap around - you should pull FROM openaps TO your master :)

Yep, I caught it.  Hehe


Add HEAD type

I was working with itty-router and tried adding a HEAD route when I was getting a type error, itty-router seemed to handle the request fine for me and based on this comment https://github.com/kwhitley/itty-router/issues/87#issuecomment-1034023188 . I believe it's just missing in the types

Gonna merge this so you get into the contributors, but v3.x will not be subject to further release.
I've also added HEAD to the v4.x/next branch as well, so when that releases (hopefully in the next couple days), expect that functionality :)


Band-aid fix for restart song crash

There has to be a better way to fix this, right??? I didn't find one though, but this should fix this for now. Please pull this; the crash is genuinely irritating

This PR has an issue; base game stages won't load if you specifically use the RESTART SONG option. Still, it's better than nothing


Accessibility plugins

~~https://github.com/maranran/eslint-plugin-vue-a11y~~
https://github.com/vue-a11y/eslint-plugin-vuejs-accessibility
https://github.com/evcohen/eslint-plugin-jsx-a11y

This is better suited on application level.


Get datasources dropdown content as part of every page's query instead of making a separate query

Use a fragment. Pass the fragment up to the Frame.

With the production data the datasources query is a bit slower, and the dropdown appears after other components have loaded. The effect is disconcerting. It's better to not allow the user to interact with the page until everything is ready.


Move RFCs into the documentation site

Fixes

Fixes #1162 by @zackkrida
Description

Adds copy buttons to Sphinx codeblocks
Moves rfcs/ to the docs site in /documentation/projects/proposals
Delete the /docs and /rfcs directories; remove from CODEOWNERS
Fixes small docs errrors:

Switch :reverse: to :reversed: in changelogs ToCs
Remove some odd ↖ Table of Contents strings at the top of some documents (is this for something, and I missed it?)
Fix heading hierarchy of some docs with multiple h1s

Update every implementation plan and project proposal title to include the date and type of proposal

Todos

[ ] Check for GitHub flavored markdown contents in proposals and update them to MyST/Commonmark+ syntax

Testing Instructions

Ideally, check out the built docs preview once it exists 😄
https://docs.openverse.org/_preview/1164/projects/index.html
Checklist

[ ] My pull request has a descriptive title (not a vague title like
Update index.md).
[ ] My pull request targets the default branch of the repository (main) or
a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible
errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I support this move, but looking at the table of contents generated based on the current approach with headings, it seems pretty unruly, especially imagining the numerous documents we'll be adding just this year:

Rather than using a glob pattern, each project directory could have an index.md, that way we still get the nested, project-name based headings to navigate.

@sarayourfriend that's an excellent idea. Then we could also link to the project in GitHub in the index file as well. I definitely thought this looked cluttered but also did want to show a list of all projects. I'll add the index files today!

@AetherUnbound I think to get that separation we'd need to have a single toctree that points only to the guides/index and proposals/index and then have each of those have toctrees that list the relevant documents for them.
Alternatively, the Furo theme allows doing some manual futzing about with the sidebar listing but it's listed as an unstable feature: https://pradyunsg.me/furo/customisation/sidebar/
I was looking at the code for this last week or the week before and even the code has some comments to that effect...


How to create mysql foreign key

Operating system (distribution) and version : maxosx
DBeaver version : <IP_ADDRESS>101032114

I am editing a mysql table and I want to add a foreign key to this table, but I find that the confirm button is grayed out, what's the matter?

Please select a column for foreign key (left column on your photo). The button will become active after it.

Please select a column for foreign key (left column on your photo). The button will become active after it.

Great, I know how to use it

Great, I know how to use it

Great, I know how to use it

Can you please tell me how you resolved it, I am also facing the same issue.
Please reply as soon as possible.

Great, I know how to use it

Can you please tell me how you resolved it, I am also facing the same issue. Please reply as soon as possible.

Maybe not a bug but definitely sub-optimal UI.  Once you've selected a table in "Referenced table" you then need to click in the "Column" column in the "Columns" table 😑


Network october release

Can one of the admins verify this patch?

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

@AutorestCI regenerate azure-sdk-for-go

@lmazuel LGTM, re-ran failed job, now looks ready to be merged

@MikhailTryakhov Please let me know when this can be merged. Thanks!

@jianghaolu please merge

@AutorestCI regenerate azure-sdk-for-java


Fee Estimate Email Generation

As an applicant
I want to receive my fee estimate information electronically
so that I can easily pay for my fee

Assumptions & Scope
This story is the email that will be automatically delivered to the applicant upon putting a request on hold, WHEN the applicant has an email address included with their request.
This will be a basic version of the email sent to the applicant - and will include the file uploaded as part of the on hold state change.
What is IN scope?
The email that will be generated and sent to the applicant
What is NOT in scope?
The email service that will be used and set up
The actual payment of the fee itself
Edits or additions to the email sent
Acceptance Criteria
Scenario 1: Email Details

GIVEN a request has been put on hold
WHEN the email is generated
THEN the email will be sent to the applicant's email address on file

Scenario 2: Attachments to email

GIVEN a request has been put on hold due to fees
WHEN the email is generated
THEN the document uploaded during the change of state to on hold will be attached to the email to the applicant.

Scenario 3: Content of Email

GIVEN a request has been put on hold
WHEN the email is generated
THEN the contents of the message will include a hyperlink to pay the fee estimate
NEED DESIGN

Scenario 4: Send Email

GIVEN a request has been put on hold
WHEN the email is generated
AND the attachments have been added
AND the content has been generated
THEN the email will send to the applicant on file.

Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
https://jacklyn808742.invisionapp.com/console/share/RSBDPXJGKAW/922730561
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

Keeping this one down on the list and marked as incomplete AC - will need some more research/unpacking into what the letter should entail before we move ahead on this one.

Story updated per the updated flow of having a letter to upload instead of auto-generating the letter.
Thinking we will need a design @JHarrietha-AOT - since it is just an email, I could offer to write up a mock-one instead? Thoughts?
cc: @lmullane @KaraBeach @arielleandrews

Basic content of email:
"Dear ,
Please see the attached regarding your FOI Request.
If you would like to pay your estimate online, please click on this link.
Thank you,

Will create a separate design ticket for templating considerations

Per discussion at refinement, need a word doc to act as a template for the email. Include tags like contact info/other to be pulled into the email itself

@arielleandrews - will work with @JHarrietha-AOT on this

@m-prodan @arielleandrews I am going to remove the design link from this ticket to avoid confusion with the devs. @aparna-aot I will tag you here with the word doc

@aparna-aot @JHarrietha-AOT - email template in word document here:
https://bcgov.sharepoint.com/:w:/r/teams/03521/Shared Documents/General/Design Assets for Web App/Letters and Templates/Fees/Draft Email - Online Pay.docx?d=w0ea79f549c554d678d12512ac09363f3&csf=1&web=1&e=EObum3
cc: @lmullane @KaraBeach @arielleandrews

@m-prodan Just confirming, if this email doesn't have a footer similar to FOI Request Confirmation Email.

Good catch - we do have something similar with our email templates, but not the same as that one. I have updated it. The letter attached will have most of that contact information in it anyways, but is good to have a consistent footer

updated story to include test email details, and assumptions around new mailbox needing to be ordered.
cc: @lmullane @KaraBeach @arielleandrews

Testing completed, works as expected. Some notes for future changes/enhancements:

Currently all attachments to the email have "fee.pdf" as the filename. Need tech debt ticket to have this pull actual filename of the file
Need to allow for upload of two files (fee letter and invoice) during state change transition from Fee Estimate to On Hold, and attach both files to email to applicant
Update design of email template to incorporate BC government logo


Stack overflow: check for infinite recursion error when awaiting tween.finished

Godot version
4.1.1.stable
System information
Windows 10 - Godot 4.1.1.stable - Vulkan Forward +
Issue description
In my current project I am writing some code so that an artist can drag and drop some ProceduralSkyMaterial resources into a WorldEnvironment and have the corresponding sky materials be used for specific points in a day/night cycle.
To do this I have a basic loop in my game where sunrise gives way to day, day to sunset, and sunset to night which goes to sunrise again.
During this loop I tween at specific points to the values defined in the resource but for some reason when awaiting the tween finished signal it just skips over it seemingly.
Now if I create timers and wait for those to complete then I don't get the recursion error. But I thought I could just await tween.finished and it would wait for all the tweens to be completed...
Steps to reproduce
Open the provided zip project in Godot. Enter the UtilsTest.tscn, and play the current scene.
Uncommenting the lines with the timer in them in the day, night, sunrise and sunset functions makes the script work for some reason.
Minimal reproduction project
GodotUtils.zip

Wait I forgot to
await tween_to_random_sky(MorningSkies)

I'm so used to other languages where if you don't await they tell you.
closing issue as this now works


fix: upgrade provider to 1.54.0

Description
Bump provider version to pick up bug fix for auth policies in 1.54.0
Types of changes in this PR
Changes that affect the core Terraform module or submodules

[x] Bug fix
[ ] New feature
[ ] Dependency update

Changes that don't affect the core Terraform module or submodules

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other

Release required?
Identify the type of release. For information about the changes in a semantic versioning release, see Release versioning.

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
Update IBM provider to v1.54.0
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Gradle Error:Could not find com.borax12.materialdaterangepicker:library:1.8.

Gradle Error:Could not find com.borax12.materialdaterangepicker:library:1.8.
http://gradleplease.appspot.com/#borax12
https://bintray.com/search?query=borax12
Anyone knows how to include this library?

This works though.
compile 'com.borax12.materialdaterangepicker:library:1.6

same error happens to me..

I have the same problem, I can't download 1.8 version. Last working version is 1.7

Check out my comment in another issue. https://github.com/borax12/MaterialDateRangePicker/issues/50#issuecomment-247498454

The jar is not included somehow in the 1.8 version , you will have to use the aar binary . I will be making sure the 1.9 version gets released in jar too


pip-compile doesn't seem to support subpackages

Working with Airflow, which uses subpackages, e.g. pip install airflow[s3, postgres]. I cannot seem to find a cannon way to work around this with pip-tools, so far.
Steps to replicate

Create a new virtualenv
Install pip-tools
Create a requirements.in containing only the line airflow[s3, postgres]
pip-compile requirements.in

Expected result
At a minimum, the compiled requirements.txt should pass the subpackages along to the pinned line, e.g.:
airflow[s3, postgres]==<IP_ADDRESS>
In my most favorite world, this would also pin the expanded deps of those submodules (at least boto and psycopg2 in this example).
Actual result
Subpackages are omitted entirely, e.g.:
airflow==<IP_ADDRESS>
And so are their deps.

This is probably a less articulate version of #408.

This should be fixed by #410 and be supported from pip-tools 1.8 onwards, which will be released shortly.

Brilliant, thanks!


Upgrade to latest grpc

We currently depend on 1.10.1, but 1.12.0 is available now.

@garrettjonesgoogle Can I just bump it manually or should this be done by codegen?

Bump it manually, and run integration tests locally to make sure they pass (since CI doesn't run integration tests on PRs)


Unable to extract nsig function code

Checklist

[ ] I'm reporting a broken site support issue
[X] I've verified that I'm running youtube-dl version 2021.12.17
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar bug reports including closed ones
[X] I've read bugs section in FAQ

Verbose log

% ./youtube-dl -v 'https://youtube.com/shorts/Z4js1seup0o?si=fGTXOUdYnNas1xlw'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://youtube.com/shorts/Z4js1seup0o?si=fGTXOUdYnNas1xlw']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Single file build
[debug] Python 3.13.0 (CPython x86_64 64bit) - Linux-6.11.5-300.fc41.x86_64-x86_64-with-glibc2.40 - OpenSSL 3.2.2 4 Jun 2024 - glibc 2.40
[debug] exe versions: ffmpeg 7.0.2, ffprobe 7.0.2
[debug] Proxy map: {}
[youtube] Z4js1seup0o: Downloading webpage
[youtube] Z4js1seup0o: Downloading player baafab19
ERROR: Unable to extract nsig function code (caused by RegexNotFoundError('Unable to extract \x1b[0;34mInitial JS player n function name\x1b[0m; please report this issue on https://github.com/ytdl-org/youtube-dl/issues , using the appropriate issue template. Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose option and include the complete output.')); please report this issue on https://github.com/ytdl-org/youtube-dl/issues , using the appropriate issue template. Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose option and include the complete output.
Traceback (most recent call last):
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1637, in _decrypt_nsig
    jsi, player_id, func_code = self._extract_n_function_code(video_id, player_url)
                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1686, in _extract_n_function_code
    func_name = self._extract_n_function_name(jscode)
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1660, in _extract_n_function_name
    func_name, idx = self._search_regex(
                     ~~~~~~~~~~~~~~~~~~^
        # new: (b=String.fromCharCode(110),c=a.get(b))&&c=nfunc[idx](c)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    ...<5 lines>...
            (?P<nfunc>[a-zA-Z_$][\w$]*)(?:\s*\[(?P<idx>\d+)\])?\s*\(\s*[\w$]+\s*\)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        ''', jscode, 'Initial JS player n function name', group=('nfunc', 'idx'))
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/common.py", line 1101, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player n function name; please report this issue on https://github.com/ytdl-org/youtube-dl/issues , using the appropriate issue template. Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose option and include the complete output.
Traceback (most recent call last):
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1637, in _decrypt_nsig
    jsi, player_id, func_code = self._extract_n_function_code(video_id, player_url)
                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1686, in _extract_n_function_code
    func_name = self._extract_n_function_name(jscode)
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1660, in _extract_n_function_name
    func_name, idx = self._search_regex(
                     ~~~~~~~~~~~~~~~~~~^
        # new: (b=String.fromCharCode(110),c=a.get(b))&&c=nfunc[idx](c)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    ...<5 lines>...
            (?P<nfunc>[a-zA-Z_$][\w$]*)(?:\s*\[(?P<idx>\d+)\])?\s*\(\s*[\w$]+\s*\)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        ''', jscode, 'Initial JS player n function name', group=('nfunc', 'idx'))
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/common.py", line 1101, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player n function name; please report this issue on https://github.com/ytdl-org/youtube-dl/issues , using the appropriate issue template. Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose option and include the complete output.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/YoutubeDL.py", line 875, in wrapper
    return func(self, *args, **kwargs)
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/YoutubeDL.py", line 971, in __extract_info
    ie_result = ie.extract(url)
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/common.py", line 571, in extract
    ie_result = self._real_extract(url)
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 2104, in _real_extract
    self._unthrottle_format_urls(video_id, player_url, dct)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1721, in _unthrottle_format_urls
    n_response = decrypt_nsig(n_param)(n_param, video_id, player_url)
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1614, in inner
    raise ret
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1606, in inner
    self._player_cache[cache_id] = func(*args, **kwargs)
                                   ~~~~^^^^^^^^^^^^^^^^^
  File "/home/corey/git-trees/youtube-dl/./youtube-dl/youtube_dl/extractor/youtube.py", line 1639, in _decrypt_nsig
    raise ExtractorError('Unable to extract nsig function code', cause=e)
youtube_dl.utils.ExtractorError: Unable to extract nsig function code (caused by RegexNotFoundError('Unable to extract \x1b[0;34mInitial JS player n function name\x1b[0m; please report this issue on https://github.com/ytdl-org/youtube-dl/issues , using the appropriate issue template. Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose option and include the complete output.')); please report this issue on https://github.com/ytdl-org/youtube-dl/issues , using the appropriate issue template. Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose option and include the complete output.

Description

In the last couple of months, I've tried to download a youtube videos on a several occasions, and ran into this error.  At least I think it was this same error.  It doesn't seem to matter which video I try to download.

Please review #30839 to check whether your version is expected to work with YT (if it's actually 2021.12.17 it isn't) and #32905 regarding work-arounds temporarily required with the latest nightly version.

It seems I made an incorrect assumption that cloning the git tree from this repo and checking out the master would give me the latest stable code.
I just now tried the nightly build and it worked fine.  Sorry for opening this issue.

The master branches in the main and nightly repos should be the same apart from code that is specific to the nightly build (naming, etc), except for a few hours after a commit in the main repo, or if the sync action has to be disabled temporarily (it hasn't).
If you checked out the 2021.12.17 tag from the master repo rather than the latest version, you would indeed get the logged behaviour. The nightly repo is a better target because it shows accurate version details.

Thanks for the clarifications!
As far as I remember, I just did a straight-up clone, and didn't specify a branch or tag.  Each time I tried it and ran into an error, I did a pull, and it kept saying my tree is up to date.  I suppose it's possible I tried checking out a particular release, thinking the top of the tree might be broken, and then forgot that I had done that later. In any case, I deleted the entire tree so there's no way to know anymore.  I'm currently tracking the nightly repo.


typo fixed: "someting" to "something"

Please note that the 2nd, 3rd and 4th screenshots in readme.md may need updating.


Issue Fixed #6886 (Locale Dropdown Overlapping on Mobile View issue fixed)

Issue Reference

#6886
Description

Issue Fixed => Locale Dropdown Overlapping on Mobile View
How To Test This?

Steps To Test--

Log in as an admin on mobile.
Click on products inside the catalog tab.
Try to change the app locale.

Check Here (Issue Fixed)

Documentation

[ ] My pull request requires an update on the documentation repository.

@lex111 Check now latest commit, Seems Good?

I'm not the maintainer of this repo, but I think it's definitely better.

@lex111 Yes I know, I just ask for you checking for more suggestion.
Anyway, Thanks for guiding for optimizing my solution of fixing this Issue.

@lex111 Yes I know, I just ask for you reviewing that its seem like that as you suggested.
Anyway, Thanks for guiding for optimizing my solution of fixing this Issue.

@Sanjaybhattwebkul Please check this PR.

@JunaidJaved248
Resolve the conflicts.

@JunaidJaved248
This is working fine, please find the screenshot Here
Thanks for your contribution.


customMetaData

Hello,
I would like to use the customMetaData property of the multiple-cucumber-html-reporter module with this plugin. My thought would then be that this is connected to the metadataKey. If this would be defined by the user to take over the metadata setup that would be defined there instead of the default that is always used right now.
Regards,
Rob

Hi @Rob-Meijeren
Isn't this what you mean?

Hi Wim,
No that wasn't what I meant as that is creating an extra block and I would like to edit an existing block.
Currently I have the following as my metadata (I added to the config of the plugin that it should look for deviceProperties instead of metadatakey):

When I run my tests with this I have the following report:

But I would like to see it like this:

Now the reason this is different is because of instead of having a platform property in my metadata I have a os property with the same data. Since this module uses multiple-html-reporter I would like to use the option of them described here https://github.com/wswebcreation/multiple-cucumber-html-reporter#custommetadata. by setting the boolean and then adding the metadata the user has defined in their metadata (in this case deviceProperties) I should get a proper report without the question marks.

Hi @Rob-Meijeren
Sorry for my late response, I'm a bit busy the last weeks. If you like you may create a PR for it, else I pick this up in the upcoming weeks

Added this option in version 1.5.0.
I'm going to close it for now, if it's still an issue please reopen this one or open a new issue


Using CKEditor to upload images causes CSRF token authenticity failure

When trying to create/update a record after uploading an image using ckeditor, I'm getting this error: WARNING: Can't verify CSRF token authenticity
The process to replicate is this:

Upload image via the ckeditor (using the latest for Rails 3.1 --> 3.7.0.rc2)
This uploads the image, but if you look at the log, it has the error CSRF token error in there.
Now when you insert the image and update/create the record, you get redirected to a sign in.

Any ideas to solve this?  Let me know if more info is needed to diagnose.
Thanks guys!

We had a similar issue in resolve/refinerycms#628 and updating jquery-rails fixed it (or jquery-ujs at the time). Maybe it's something like that?

Can't reproduce with a fresh bundle update / Carrierwave

I was able to reproduce it with paperclip.  I attached a minimal project below and a ss of the log.  The event model in the project has a description field with ckeditor configured on it.  That error happens as soon as you hit "Send to server".
One other thing I noticed, was that in order to make rails_admin and the latest ckeditor work together, I had to copy the ck js files from the gem to public/javascripts.  The latest ckeditor wants to use sprockets.
Project: http://cl.ly/2I2a1H1u0G360H1y220V
Log SS: http://cl.ly/2q0O2K2J2x2t211R0K0z
BTW, the latest UI changes look really good!

Awesome.  Works great in test app.


f

Description
Briefly describe the issue
ChefDK Version
Tell us which version of the ChefDK you are running. Run chef --version to display the version.
Platform Version
Tell us which operating system distribution and version ChefDK is running on.
Replication Case
Tell us what steps to take to replicate your problem. See How to create a Minimal, Complete, and Verifiable example for information on how to create a good replication case.
Stacktrace
Please include the stacktrace.out output or link to a gist of it, if there is one.
NOTE: CHEFDK BUGS ONLY
This issue tracker is for the code contained within this repo -- chefdk.

Server issues
Chef-client issues
Cookbook Issues (see the https://github.com/chef-cookbooks repos or search Supermarket or GitHub/Google)

does not look like a valid issue, closing. Please feel free to create a new bug with further details if necessary.


SubtypesFactory generator

What does the pull request do?
Use a source generator to replace the old runtime reflection behavior, to make it AOT-compatible and fast.
What is the current behavior?
We use runtime reflection to gather all Easing types.
What is the updated/expected behavior with this PR?
All Easing types will be generated in compilation time, no runtime reflection is required. Note that other similar scenarios can also make use of the SubtypesFactoryGenerator for code generation.
How was the solution implemented (if it's not obvious)?
Implement a source generator for subtypes instance generation.
For the Easing case, we put an attribute on the partial method:
[SubtypesFactory(typeof(Easing), "Avalonia.Animation.Easings")]
private static partial bool TryCreateEasingInstance(string type, [NotNullWhen(true)] out Easing? instance);

then the generated code:
using System;
using System.Collections.Generic;

namespace Avalonia.Animation.Easings
{
    partial class Easing
    {
        private static partial bool TryCreateEasingInstance(string type,out Easing? instance)
        {
            var hasMatch = false;
            (hasMatch, instance) = type switch
            {
                "BackEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.BackEaseIn()),
                "BackEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.BackEaseInOut()),
                "BackEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.BackEaseOut()),
                "BounceEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.BounceEaseIn()),
                "BounceEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.BounceEaseInOut()),
                "BounceEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.BounceEaseOut()),
                "CircularEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.CircularEaseIn()),
                "CircularEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.CircularEaseInOut()),
                "CircularEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.CircularEaseOut()),
                "CubicEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.CubicEaseIn()),
                "CubicEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.CubicEaseInOut()),
                "CubicEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.CubicEaseOut()),
                "ElasticEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.ElasticEaseIn()),
                "ElasticEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.ElasticEaseInOut()),
                "ElasticEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.ElasticEaseOut()),
                "ExponentialEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.ExponentialEaseIn()),
                "ExponentialEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.ExponentialEaseInOut()),
                "ExponentialEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.ExponentialEaseOut()),
                "LinearEasing" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.LinearEasing()),
                "QuadraticEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.QuadraticEaseIn()),
                "QuadraticEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.QuadraticEaseInOut()),
                "QuadraticEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.QuadraticEaseOut()),
                "QuarticEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.QuarticEaseIn()),
                "QuarticEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.QuarticEaseInOut()),
                "QuarticEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.QuarticEaseOut()),
                "QuinticEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.QuinticEaseIn()),
                "QuinticEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.QuinticEaseInOut()),
                "QuinticEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.QuinticEaseOut()),
                "SineEaseIn" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.SineEaseIn()),
                "SineEaseInOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.SineEaseInOut()),
                "SineEaseOut" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.SineEaseOut()),
                "SplineEasing" => (true, (Avalonia.Animation.Easings.Easing?)new Avalonia.Animation.Easings.SplineEasing()),
                _ => (false, default(Avalonia.Animation.Easings.Easing?))
            };

            return hasMatch;
        }
    }
}

Alternative solution
Write generated code directly without a source generator.

Now it's ready for review.

You can test this PR using the following package version. 0.10.999-cibuild0019704-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019743-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

Any thoughts on this PR?
I think "gathering all types under a namespace" is potentially useful not only for Easing, so I create this source generator.

You can test this PR using the following package version. 0.10.999-cibuild0019937-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

@hez2010 From my perspective this change looks good. I was curious about perf so I've benchmarked this against current master.

Method
Mean
Error
StdDev
Gen 0
Allocated

CreateOld
178.4 ns
4.39 ns
12.88 ns
0.0880
184 B

CreateNew
100.1 ns
2.25 ns
6.57 ns
0.0880
184 B

It is a lot faster despite losing caching in that dictionary (compiler outputs decent code for this large switch, although there is one extra string comparison in there still).

As for today it shouldn't be a problem to have source generators in the main repo. I'm fine with it.
Probably it won't make much sense to generate same code with xaml compiler either.

You can test this PR using the following package version. 0.10.999-cibuild0019962-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019966-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019972-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019980-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

Please update the source generator to use the IIncrementalGenerator interface instead of ISourceGenerator with an ISyntaxReceiver. Syntax receivers have a really bad performance cliff that can completely destroy the IDE experience. Incremental generators (while harder to author initially), have no such issues.

I've updated the source generator to implement IncrementalGenerator.

You can test this PR using the following package version. 0.10.999-cibuild0020001-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


cl21: Fix test spirv_new - define test cache path

Fix issue #373

Meanwhile we can read something :) https://attractivechaos.wordpress.com/2018/08/31/a-survey-of-argument-parsing-libraries-in-c-c/


Incorrect handling of UnicodeBOM in GraphQL grammar

UnicodeBOM must be ignored everywhere in a document not just at the beginning.

And this is how Apollo just ignores UTF-8 BOM after opening curly:
$ echo -ne '{"query": "{\xEF\xBB\xBF__typename }"}' | curl -X POST -H "Content-Type: application/json" --data-binary @- http://localhost:4000
{"data":{"__typename":"Query"}}


How to stop the FTP server?

I was able to successfully implement your library in my app, but one essential thing is missing for me: How to stop the server?
Do you have an idea?
Thank you for this good work.

This feature is not implemented yet.

The feature is implemented now.
You can stop the FTP server at runtime like this:
https://github.com/hxcan/hxftpserver/commit/9986d76048041ff8ea8c15acddc28bec95350396#diff-b9a19f3b2f9a51896a8fe8a0ca05b1723c5557557a68123cf8cdd81458256889R125
    ftpServer.stop();

You have to use the latest version "2023.10.24".

The feature is implemented now.
You can stop the FTP server at runtime like this:
https://github.com/hxcan/hxftpserver/commit/9986d76048041ff8ea8c15acddc28bec95350396#diff-b9a19f3b2f9a51896a8fe8a0ca05b1723c5557557a68123cf8cdd81458256889R125
    ftpServer.stop();

You have to use the latest version "2023.10.24".


Any way to add a nested object?

I'm trying to build up a JSON document hash starting from {}. I keep ending up here when I add a nested path. I see the problem -- you can't know whether applying the patch {"op": "add", "path": "/foo/0", "value": "bar"} to {} should yield
{"foo": {"0": "bar"}}

or
{"foo": ["bar"]}

However, for paths with all non-numeric segments there should be no confusion.
Is there a way to add a nested element with Hana that I can't see? Is it reasonable to think of adding support for this?

Is there a way to add a nested element with Hana that I can't see?

No.  Not that I know of.

Is it reasonable to think of adding support for this?

Yes, I think so. But add the test here so that all implementations will get the behavior.


Allow dataset implement .take

Feature request
I want to do:
dataset.take(512)

but it only works with streaming = True
Motivation
uniform interface to data sets. Really surprising the above only works with streaming = True.
Your contribution
Should be trivial to copy paste the IterableDataset .take to use the local path in the data (when streaming = False)

    dataset = IterableDataset(dataset) if type(dataset) != IterableDataset else dataset  # to force dataset.take(batch_size) to work in non-streaming mode

hf discuss: https://discuss.huggingface.co/t/how-does-one-make-dataset-take-512-work-with-streaming-false-with-hugging-face-data-set/50770

so: https://stackoverflow.com/questions/76902824/how-does-one-make-dataset-take512-work-with-streaming-false-with-hugging-fac

Feel free to work on this. IterableDataset also supports skip, so we should add this method to Dataset, too


use fibonacci hash for memory pixmap

How's this?

Pretty good, interesting feature to visualize the banking.


Upgrade to Modular Build and STM8 eForth 2.2.24.pre3

There is an new solution, the modular build, to which I'll eventually update this project.

Originally posted by @TG9541 in https://github.com/TG9541/W1209/issues/26#issuecomment-621800043

@tracing-home: FYI


Setup Kaban Board For React-Redux

Hi Reviewer :wave:
The link to my github project
Note: We are two in this group in our group project.

Hi @calebchris000 and @Gmanlove,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because the link to the project is broken and i can not review it
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

@addod19 Thanks for the correction. It's fixed now.

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights 💯

Descriptive info ✔
Tasks distributed equally ✔

Optional suggestions
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Some feature not working on android in release mode

When I debug the flutter app, everything is ok. But when I run in release mode flutter run --release -d xxxxxx, below feature is not work for me:
FlutterStatusbarManager.setStyle(StatusBarStyle.DARK_CONTENT);
FlutterStatusbarManager.setNavigationBarStyle(NavigationBarStyle.DARK);

Call the manager just in app build:

env:

device: Google Pixel 2 XL running Android 9
flutter doctor ouput:

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.2.1, on Mac OS X 10.14.3 18D109, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.3)
[✓] VS Code (version 1.32.3)
[!] Proxy Configuration
    ! NO_PROXY is not set
[!] Connected device
    ! No devices available

Well, when I wrap these calling in a async function, it worked again in both debug and release mode.

FlutterStatusbarManager functions should been called synchronously


It's an awful lot of code for a hello world

Tried looking up DEEP, and it was hard to get the idea of how to work with it, so a hello world should  make this all clear, or so I thought.
Just wanted you to reconsider and make the hello world example as canonical as possible, as it is quite off-putting in its current state for new developers.
Is really a concept of models necessary? And tests? And validation, front-end etc?
Not for a hello world, but sure would be nice with those examples too, in separate repositories as soon as a basic hello world is understood.

@kennethlynne Thank you very much for your feedback! We are really sorry that current state of the platform is quite complex and we would like to improve it (reduce the complexity). Indeed, we are considering to simplify hello world repository, as well as open source other repositories that would make it easier for others to understand and engage with DEEP.
To answer your questions, I would agree that concepts of validation, tests, and similar is not necessary in hello world, but front-end + back-end + data (tiers) are vital to seamless functioning of this platform. So, again, let us work in the next couple of days on reducing the codebase, as well as improving the documentation.

We have worked to simplify the codebase. Everything you need to know is summarized here: https://github.com/MitocGroup/deep-microservices-helloworld/tree/master/src/DeepHelloWorld
We are now working to simplify the documentation and create some more tutorials + videos that will show the value of this approach.
We'll close this issue now, but feel free to reopen or create new ones if you see other problems.


No System.js or Angular.io support out of the box

It'd be nice to have some docs about how to use this in angular with System.Js.  It is not clear to me if only with this project are we able to get a yes/no authentication, or we need to install additional packages, like the aws-sdk package.

FWIF it does work with Angular (4.x), though I've only tested with Webpack. I've only installed amazon-cognito-auth-js package through NPM, and imported CognitoAuth using import { CognitoAuth } from 'amazon-cognito-auth-js/dist/amazon-cognito-auth';. Hope that helps.

thkx @dinvlad , I decided to import this in index.html:
<script src="app/aws-cognito-sdk.js"></script>
<script src="app/amazon-cognito-auth.js"></script>

and use it globally.  I would have preferred to have it contained inside angular as a Module, but it was taking me too long to figure out how to do it.  I've seen that most modules include typings and can be installed through npm.  Until amazon-cognito-auth-js gets there, I think this is the only way to go while using system.js

Typings are not needed here, the module already contains ES5 classes to support TypeScript out of the box.

Is there a working example for angular 4? I am still finding my feet with Angular atm.

@yhedaoo not for this project, at least not using system.js. It seems @dinvlad has tested with Webpack and I got it working with a global approach. (I am using system.js).

At the risk of citing a competitor solution, there's a very thorough guide from Auth0 on how to configure Angular (2+.x) with their library [1]. I followed it almost to the letter to configure CognitoAuth. The only difference is in how you instantiate this class, parse tokens in the callback, and get the session (arguably, it is actually easier now with CognitoAuth). Here's a rough sketch of my AuthService class, where all of CognitoAuth hides:
import { Injectable } from '@angular/core';
import { Router } from '@angular/router';

import { CognitoAuth } from 'amazon-cognito-auth-js/dist/amazon-cognito-auth';

export class AuthConfig {
  constructor(
    public readonly clientId: string,
    public readonly domain: string,
  ) {}
}

@Injectable()
export class AuthService {
  private readonly auth: CognitoAuth;

  constructor(config: AuthConfig, private readonly router: Router) {
    this.auth = new CognitoAuth({
      ClientId: config.clientId,
      AppWebDomain: config.domain,
      TokenScopesArray: [],
      RedirectUriSignIn: this.callbackUrl,
      RedirectUriSignOut: this.callbackUrl,
    });
    this.auth.userhandler = {
      onSuccess: session => this.onSuccess(session),
      onFailure: () => this.onFailure(),
    };
  }

  // have to construct callbackUrl from "scratch",
  // as Angular doesn't appear to offer a native way to do it;
  // '/login' is my callback route
  private get callbackUrl() {
    return window.location.href.split('/').slice(0, 3).concat('login').join('/');
    // or window.location.origin + '/login';
  }

  // gets called by the callback component
  login() {
    this.auth.parseCognitoWebResponse(this.router.url);
    this.auth.getSession();
  }

  // used to determine access in AuthGuard
  isAuthenticated() {
    return this.session && this.session.isValid();
  }

  private session;

  private onSuccess(session) {
    this.session = session;
    this.router.navigateByUrl(this.guardedUrl);
  }

  private onFailure() {
    this.session = undefined;
  }

  get accessToken() {
    return this.session && this.session.getAccessToken().getJwtToken();
  }

  // save and restore initial route upon redirect from Cognito-hosted page

  set guardedUrl(url: string) {
    localStorage.setItem('guardedUrl', url);
  }

  get guardedUrl() {
    return localStorage.getItem('guardedUrl') || '/';
  }
}

Everything else may be done exactly the same as in [1].
[1] https://auth0.com/docs/quickstart/spa/angular2/01-login

the redirect back to localhost:4200/login seems to fail when I try it.. do you have a boilerplate angular4 application set up that I can possibly look at? Thanks

@yhedaoo I had the same problem, please look at this recommendations:

Cognito Auth only works with HTTPS, so your localhost must be configured for this secure access.
Make sure you enable both "code" and "implicit" authentication methods in the Cognito application settings.
Double check your redirect urls, they must all match client & server side.
Make a screenshot of the error selecting "login" instead of "favicon.ico" Could provide more insight.
Good luck!

@yhedaoo yes, everything @rcfrias said needs to fall into place (although "code" method is not needed for Angular, unless it's server-side rendered). You do need to configure a /login route for redirect, or alternatively redirect to the base of your app but then put your login() method into ngOnit of your AppComponent. Unfortunately I'm still in the process of open-sourcing my applications, but once it's ready I can share a link.

Does any of the suggestions work? @yhedaoo

Hi @dinvlad, could you help me understand a few things from your example?
I am able to go to the AWS created page and log in using facebook or google. I am returned a access_token in my callback, but I am not sure how I am supposed to capture this token? My callback is localhost:4200/callback, but as soon as my application goes to this route, it re-routes back to localhost:4200 and removes the hash.
Neither the onSuccess nor the onFailure functions are fired in my authService. Sorry if this is obvious stuff. I am fairly new to Angular.

@iamcootis sorry, not sure how to help. Generally, your Angular Router should be set up to direct your callback route to the Auth component that will invoke authService.login() when hit.
Btw to anyone concerned, unfortunately I'm no longer using this library (we moved to Firebase). I'd defer to Amazon staff here for further help.

Hi @iamcootis, You need to call "auth.parseCognitoWebResponse(curUrl); ", which would have extracted the tokens from the url and invoked onSuccess()/onError() callbacks.
You can see the example in our sample line 97 to 112. https://github.com/aws/amazon-cognito-auth-js/blob/master/sample/index.html
Here you can see that you need to initialize auth object first(line 194 to 216), then call "auth.getSession()" for sign-in, then call "auth.parseCognitoWebResponse(curUrl);" to parse the callback url containing tokens or code.

@yuntuowang The problem, and I think @dinvlad is having the same issue, is that your userhandler is not working in angular.
this.auth.userhandler = {
      onSuccess: session => this.onSuccess(session),
      onFailure: () => this.onFailure(),
    };

This part of the code never gets triggered. Do you have a solution for that? Or an working angular example?

@marc101101 : Yes indeed. Same problem, userhandler is not working in angular, onSuccess and onFailure never being called

Fixed this problem. You have to init the handler bevore calling:
this.auth.parseCognitoWebResponse(this.router.url);

I still struggled to get the handlers firing. What made it click for me was the following:
The Component that will be loaded by the Router when the RedirectUriSignIn url is activated should call the this.auth.parseCognitoWebResponse(this.router.url) method, eg in the ngOnInit method. That url will have the token/code added in the query string, and parsing that will trigger the onSuccess handler, which in turn will give you your session object.
So while anything can call the this.auth.getSession() to go to the AWS Cognito hosted web ui page, the callback Component (the Component linked by the Router to the RedirectUriSignIn url you have configured) should parse the url to get the session.
Hope this helps someone.


Make it possible to override addDisposableListener, addStandardDisposableListener etc.

Hello,
We wrap the Monaco editor in an Angular component. But key and mouse events will trigger Angular's change detection, which is totally unnecessary and can lead to slow downs while editing (e.g. holding down the Shift key for while would make Angular trigger its change detection constantly until the key is released).
In our code, we monkey-patched esm/vs/base/browser/dom's addDisposableListener, addStandardDisposableListener, addDisposableNonBubblingMouseOutListener and addDisposableThrottledListener to run outside of Angular's zone.
This works as long as we compile our modules using "es5" as a target. If we change the target to "es2015" for example, JavaScript is much more strict and gives a runtime error disallowing the modification of these functions.
The feature request is to make it possible to override these functions, so that in our case the overrides would call NgZone.runOutsideAngular((node, type, handler, useCapture) => super.addDisposableListener(node, type, handler, useCapture)) and so on.
Thanks!

any update on this? I need to be able to override the default paste listener and I can't find a good way to do it.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


🚀 Feature:

Bug Report Checklist

[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
A few people have reported annoyance/difficulty seeing the Philly JS or Boston TS websites on super wide or zoomed out monitors. The content gets placed in the far corners, which is kind of weird.

Let's add a very large max-width to the page content.
Additional Info
Co-authored-by: @git-megan

@allcontributors add @git-megan for ideas

So we paired on this and it's actually kind of complex 😂. Unassigning and hoping one of us can take some time to look into it ... later. Or someone else. Good luck!

Talked about it with @git-megan and... well, we figured out how to do it (right: calc(50vh - var(--max-page-width) / 2))... but we don't see a strong use case for it. Folks can always zoom in or out. Wontfixing unless/until someone pushes back with a real use case.


peerDependencies not updated at npm repo

Hi,
Just noticed that the peerDependencies are still the previous ones to #503 on npm.
Just installed the package inside an empty directory (after clearing npm cache) and the peerDependencies section of installed react-scroll package.json shows:
"peerDependencies": {
    "react": "^15.5.4 || ^16.0.0 || ^17.0.0",
    "react-dom": "^15.5.4 || ^16.0.0 || ^17.0.0"
  },

I'm not sure why, maybe the package version should be bumped in order to get it updated at npm?
Thanks :)

@martabitbrain You're right, I've completly forgotten to publish the latest changes.
It's now published under v1.8.7


Make testing.py executable

The testing.py script does not have the execution permission and hence it is not runnable out of the box. chmod +x will gladly fix it. :hammer:

Oh, I forgot to set that, I'm doing it now

After an hour i managed to upload the executable version

Great, you could have closed the issue as it has been effectively resolved. I will do so.


[Bug] Vue 3 HMR does not work inside a default slot using PrimeVue

Vue version
3.2.45
Link to minimal reproduction
https://github.com/martinszeltins/vue-hmr-not-working-slot
Steps to reproduce
I don't know if this is an issue with Vue or Vite or PrimeVue... but since it has to do with HMR, I figured it must be a Vue issue.
Here are the steps to reproduce the bug:

Clone and run the project.

$ git clone git@github.com:martinszeltins/vue-hmr-not-working-slot.git
$ cd vue-hmr-not-working-slot
$ npm install
$ npm run dev

Now open App.vue and change the default slot of TabPanel (line 9). Then save the App.vue file and look at the browser - HMR did not work unless you manually reload the page.

All I did was take the Vite starter template and added PrimeVue.
$ npm create vite@latest my-vue-app -- --template vue
$ cd my-vue-app
$ npm install
$ npm install primevue@^3 --save
$ npm install primeicons --save

Then I added PrimeVue to main.js like this:
import { createApp } from 'vue'
import './style.css'
import App from './App.vue'
import PrimeVue from 'primevue/config'
import AppModal from 'primevue/dialog'

const app = createApp(App);

app.use(PrimeVue);
app.component('AppModal', AppModal);
app.mount('#app')

And then I added the component in App.vue as you can see from the reproduction repo. I didn't change anything else. Something is wrong.
What is expected?
I was expecting HMR to work and auto refresh the content inside the default slot but it is not working unless I reload the whole page.
What is actually happening?
HMR is not working as expected.
System Info
System:
    OS: Linux 5.15 Ubuntu 20.04.5 LTS (Focal Fossa)
    CPU: (8) x64 Intel(R) Core(TM) i5-10300H CPU @ 2.50GHz
    Memory: 9.56 GB / 31.26 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 18.12.1 - ~/.nvm/versions/node/v18.12.1/bin/node
    Yarn: 1.22.19 - /usr/bin/yarn
    npm: 8.19.2 - ~/.nvm/versions/node/v18.12.1/bin/npm
  Browsers:
    Chrome: 107.0.5304.121
    Firefox: 107.0
  npmPackages:
    vue: ^3.2.41 => 3.2.45
    primevue: 3.20.0
    vite: 3.2.3

Any additional comments?
No response

Is it possible that this is the problem？

Duplicate of #7155