Component detection fails for certain rules when using this within a stateless function

This took me awhile of digging around to figure this out.
Here is a mockup of something we had in our React components:
import React from 'react';

const onClick = () => {};
const MyComponent = ({ myVar }) => {
  return (
    <button onClick={onClick.bind(this)}>{myVar}</button>
  );
};

MyComponent.propTypes = {};

With this rule configured ('react/prop-types': [2],), eslint failed to find anything wrong with this code. However, it should be erroring because myVar is a prop that isn't defined in .propTypes
I dug into the source code and figured out what's going on. The fix is to remove the .bind(this) and change it to anything other than this.
The root cause is this logic which affects all attempts to list this component with a confidence level >= 2
Obviously, the this usage is an oversight and can be easily addressed but it's worrisome that the linter is potentially skipping certain rules because of it.
Any insight or thoughts?

Yeah, functional components detection is pretty tricky so we have different strategies to detect them, and it can happen that some of them are unfortunately ignored.
I'll try to be less restrictive on the this keyword usage in functional components.


Increased sqlite timeout.

With previous change (sending reads/writes to db to different threads), we need higer timeout, because database could be easily locked for more than 5 seconds.

LGTM


Can anyone give an example containning related files  to use this ?

Hello,
I am trying to use this software to extract coverage. But failed.  Does anybody can upload an example to try.

IT would be more than a little helpful if you listed the steps you took so
someone can figure out what went wrong.
That will also help us improve our docs if is a documentation failure.
On Thu, Sep 7, 2023 at 11:12 PM monk907 @.***> wrote:

Hello,
I am trying to use this software to extract coverage. But failed. Does
anybody can upload an example to try.
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-software-src/riscv-isac/issues/75, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJX3BCTNBKNOXVIEGPTXZKZNRANCNFSM6AAAAAA4P4LKKY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

IT would be more than a little helpful if you listed the steps you took so someone can figure out what went wrong. That will also help us improve our docs if is a documentation failure.
…
On Thu, Sep 7, 2023 at 11:12 PM monk907 @.> wrote: Hello, I am trying to use this software to extract coverage. But failed. Does anybody can upload an example to try. — Reply to this email directly, view it on GitHub <#75>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHPXVJX3BCTNBKNOXVIEGPTXZKZNRANCNFSM6AAAAAA4P4LKKY . You are receiving this because you are subscribed to this thread.Message ID: @.>

Hi Allen,
Thanks for your reply. I followed steps from Quickstart. And I use below command:
riscv_isac  coverage -t test.log --parser-name spike  -o coverage.rpt  -c rv32i.cgf
I don't add elf file here because the usage descriptions don't say it is required. The outputs of this command are as below:

INFO | ****** RISC-V ISA Coverage 0.18.0 *******
INFO | Copyright (c) 2020, InCore Semiconductors Pvt. Ltd.
INFO | All Rights Reserved.
Traceback (most recent call last):
File "/home/zenos/.local/bin/riscv_isac", line 8, in
sys.exit(cli())
File "/usr/lib/python3/dist-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/usr/lib/python3/dist-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/usr/lib/python3/dist-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/lib/python3/dist-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/lib/python3/dist-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/home/zenos/.local/lib/python3.8/site-packages/riscv_isac/main.py", line 136, in coverage
isac(output_file,elf,trace_file, window_size, expand_cgf(cgf_file,int(xlen),int(flen),log_redundant), parser_name, decoder_name, parser_path, decoder_path, detailed, test_label,
File "/home/zenos/.local/lib/python3.8/site-packages/riscv_isac/cgf_normalize.py", line 565, in expand_cgf
cgf = utils.load_cgf(cgf_files)
File "/home/zenos/.local/lib/python3.8/site-packages/riscv_isac/utils.py", line 163, in load_cgf
safe_yaml.dump(dict(safe_yaml.load(fp)),string_stream)
File "/home/zenos/.local/lib/python3.8/site-packages/ruamel/yaml/main.py", line 426, in load
return constructor.get_single_data()
File "/home/zenos/.local/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 111, in get_single_data
node = self.composer.get_single_node()
File "_ruamel_yaml.pyx", line 706, in _ruamel_yaml.CParser.get_single_node
File "_ruamel_yaml.pyx", line 724, in _ruamel_yaml.CParser._compose_document
File "_ruamel_yaml.pyx", line 775, in _ruamel_yaml.CParser._compose_node
File "_ruamel_yaml.pyx", line 889, in _ruamel_yaml.CParser._compose_mapping_node
File "_ruamel_yaml.pyx", line 775, in _ruamel_yaml.CParser._compose_node
File "_ruamel_yaml.pyx", line 889, in _ruamel_yaml.CParser._compose_mapping_node
File "_ruamel_yaml.pyx", line 775, in _ruamel_yaml.CParser._compose_node
File "_ruamel_yaml.pyx", line 889, in _ruamel_yaml.CParser._compose_mapping_node
File "_ruamel_yaml.pyx", line 743, in _ruamel_yaml.CParser._compose_node
ruamel.yaml.composer.ComposerError: found undefined alias
in "", line 15, column 11

I am not sure which step goes wrong.  The sample cgf file  comes from this website: https://github.com/riscv-software-src/riscv-ctg/blob/master/sample_cgfs/rv32i.cgf


Performance Bench Mark

Rasa NLU version  :  0.10.0a5
Used backend / pipeline :   spacy_sklearn,
Operating system (windows, osx, ...): ubuntu
Issue:

Are there any performance bench mark available for rasa nlu (preferably with spacy pipeline)
Further, given a certain hardware configuration, I would like some guidance on improving performance.

Content of configuration file (if used & relevant):
language ="en"

You can use the rasa_nlu/evaluate.py script to evaluate your trained model on data file. There are papers around that compare the performance between different services (e.g. api.ai, rasa_nlu,...) if you are interested in that let me know and I'll look it up.

Thanks so much for your response. I was looking for the result of  rasa bench mark test for the following parameters.

Maximum number of intents and utterance supported
How much time did it take to test the maximum number of Intents and utterances.
What is the hardware requirement to support the bench mark test
Was there an impact on the confidence of the identified intents (with the maximum number of intents/utterance).
How can the performance be improved with a given hardware.
Please help me if you have the data for the above questions.

The biggest thing we talk about when these request come up are the training data sources to support such tests. If you have some in the Rasa format or another compatible format we'd be interested in them. But as it is many of the data sources that are being used by community members are considered proprietary I've never found a great dataset to do such testing.


fe connect be error connected time out

Search before asking

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

Version
2.0.3 x64 no avx2
What's Wrong?
broker error log TProtocolException: Bad version in readMessageBegin
What You Expected?
connect normal
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Do you use be and fe with same versions?


Can 't  disable the Switch

Currently there is no way to disable the switch. Or is there any?

@ak274 its available in version v1 https://github.com/Aslam97/react-fancy-switch?tab=readme-ov-file#typescript-support


Filter out WebM sources by default, since ytdl.downloadFromInfo misbehave when converting it to audio.

Fixes https://github.com/ytb2mp3/youtube-mp3-downloader/issues/53;
It seems ytdl.downloadFromInfo() isn't accepting certain values for quality when the highest available quality is a webm, so for instance, trying to download this video QjZwuimMb3M with 'highestaudio' would cause an infinite load, due to the fact that the yielded stream won't trigger 'response' nor 'error';

Thanks for the PR. Merging....


Install Failed on CentOS 5.6 + Python 2.4

[root@raullen ~]# easy_install requests
Searching for requests
Reading http://cheeseshop.python.org/pypi/requests/
Reading http://python-requests.org
Reading http://cheeseshop.python.org/pypi/requests/0.9.3
Best match: requests 0.9.3
Downloading http://pypi.python.org/packages/source/r/requests/requests-0.9.3.tar.gz#md5=b13b6fbfa8fc3fc3c25bae300748053f
Processing requests-0.9.3.tar.gz
Running requests-0.9.3/setup.py -q bdist_egg --dist-dir /tmp/easy_install-jyVdnT/requests-0.9.3/egg-dist-tmp-8CYvTK
Traceback (most recent call last):
File "/usr/bin/easy_install", line 7, in ?
sys.exit(
File "/usr/lib/python2.4/site-packages/setuptools/command/easy_install.py", line 1670, in main
with_ei_usage(lambda:
File "/usr/lib/python2.4/site-packages/setuptools/command/easy_install.py", line 1659, in with_ei_usage
return f()
File "/usr/lib/python2.4/site-packages/setuptools/command/easy_install.py", line 1674, in
distclass=DistributionWithoutHelpCommands, **kw
File "/usr/lib/python2.4/distutils/core.py", line 149, in setup
dist.run_commands()
File "/usr/lib/python2.4/distutils/dist.py", line 946, in run_commands
self.run_command(cmd)
File "/usr/lib/python2.4/distutils/dist.py", line 966, in run_command
cmd_obj.run()
File "/usr/lib/python2.4/site-packages/setuptools/command/easy_install.py", line 211, in run
self.easy_install(spec, not self.no_deps)
File "/usr/lib/python2.4/site-packages/setuptools/command/easy_install.py", line 446, in easy_install
return self.install_item(spec, dist.location, tmpdir, deps)
File "/usr/lib/python2.4/site-packages/setuptools/command/easy_install.py", line 471, in install_item
dists = self.install_eggs(spec, download, tmpdir)
File "/usr/lib/python2.4/site-packages/setuptools/command/easy_install.py", line 655, in install_eggs
return self.build_and_install(setup_script, setup_base)
File "/usr/lib/python2.4/site-packages/setuptools/command/easy_install.py", line 930, in build_and_install
self.run_setup(setup_script, setup_base, args)
File "/usr/lib/python2.4/site-packages/setuptools/command/easy_install.py", line 919, in run_setup
run_setup(setup_script, args)
File "/usr/lib/python2.4/site-packages/setuptools/sandbox.py", line 26, in run_setup
DirectorySandbox(setup_dir).run(
File "/usr/lib/python2.4/site-packages/setuptools/sandbox.py", line 63, in run
return func()
File "/usr/lib/python2.4/site-packages/setuptools/sandbox.py", line 29, in
{'file':setup_script, 'name':'main'}
File "setup.py", line 6, in ?
File "/tmp/easy_install-jyVdnT/requests-0.9.3/requests/init.py", line 26
from . import utils
^
SyntaxError: invalid syntax
I will probably take a look on that!

Python 2.4 is not supported.

If this is the case, it would be better to mention it explicitly in the README or other instructing sheets to avoid confusion.


Please Update Podspec

Thanks for fantastic control!  Much Appreciated!!

Yes, the pod needs to be updated.  It's currently at 1.0.0.  Thanks for this nice widget!

When will pod be updated?

@rlaferla I plan on updating the spec this evening.

Thank you.  I'm not trying to be annoying.  I'm just anxious to use your nice widget!

I've finally gotten around to updating the podspec to the latest, which is version 1.0.3.


[json-schema] Added schema for device geo location

added device geo_location schema

generated python API for this schema

please, bump also minor version + add entry in the changelog


[wip] docs: restructure

Quick lil restructure of docs; should make them a bit more readable by delegating some of the stuff to the guide. Thanks!
Changes

docs: moved contributing section to a separate file
docs: removed common actions so they can be made part of the workshop (should we call it handbook perhaps?)
added an issue template
moved around some pieces of FAQ

Todo

update API docs to reflect new stuff; seperate PR perhaps?

👍 for calling workshop -> handbook

Cool; fully rebased against 3.0 and incorporated comments - think it should be good now :grin: - merging!

Oh PS added a lil deploy section to the end of the example; should address @toddself's concern about es2020 :D - also moves people away from trying to build boilerplates and stuff I reckon, hah

Wouldn't you put es2020 in the "older browser support" section? Those few things es2020 transpiles are supported in modern browsers, aren't they?

Problem is people thing safari is modern. Yet it supports none of this :(

On Jul 4, 2016, at 15:45, Tim Wisniewski<EMAIL_ADDRESS>wrote:
Wouldn't you put es2020 in the "older browser support" section? Those few things es2020 transpiles are supported in modern browsers, aren't they?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

yeah, also super sure that my phone's browser doesn't work without es2020 - it's like barely two years old; lil tweak might save people a lot of boilerplate hah :sparkles:


Add global parameters

From AutoRest :
(https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/creating-swagger.md)

##Global parameters Swagger allows for parameters to be defined separately from the operation where they are used. By convention, AutoRest treats global parameter definitions as Client properties. For example, almost all Azure Resource Manager APIs require subscriptionId and api-version. These are defined as global parameters and become properties of the client.

Adding global parameters would allow better integration with AutoRest and conform with swagger standard

https://github.com/domaindrivendev/Swashbuckle.AspNetCore/pull/403
Turns out that having a $ref on NonBodyParameter was missing to be able to ref global parameters included with DocumentFilters.


⚠  No Changeset found

          ###  ⚠  No Changeset found

Latest commit: 57fa4f0bec6a36f4358f5f6e04467a993f11a2e6
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
Originally posted by @changeset-bot[bot] in https://github.com/primer/react/issues/5480#issuecomment-2565626905

Hi. This issue doesn't appear to contain a question. Documentation on how to add a changeset when contributing is available here: https://github.com/primer/react/blob/main/contributor-docs/CONTRIBUTING.md#adding-changeset-to-your-pull-request


Support import of ListenBrainz recommendations playlists

Proposal
The proposed branch adds support for playlists returned by the playlists created for endpoint (recommendations playlists) from ListenBrainz API.
AFAICT the returned playlists are made of suggestions taken from previously listened tracks.
Implementation
The playlist descriptions are fetched by the existing Mopidy frontend during setup.  Mopidy-Local database is then searched for matching tracks and Mopidy playlists are created using a trivial Mopidy backend dedicated to those playlists management.
The playlist import must be enabled using a new configuration key import_playlists (default to false).
Playlists are periodically updated (each week, on monday).
Possible improvements

Import all user playlists, not only recommendations
Make user playlists editable and synchronized with Listenbrainz
Import playlists from other users, but doesn't make much sense if one doesn't have a huge local library... What do you think?

Test
python -m pip install --upgrade --break-system-packages git+https://github.com/orontee/mopidy-listenbrainz.git@feature/playlists
Tried with both Argos and Iris.

Something is wrong with current implementation because .m3u8 files are created; It looks like the created playlists are handled by Mopidy-M3U and not the introduced backend...
It could explain why deletion and edition works.

Thanks for the PR! Sorry for being so late, for some reason github did not notify me about it.

FYI, there's a regression in Iris, https://github.com/jaedb/Iris/issues/953

It looks good but I can't test it super well because I don't have the songs in my local backend. If you wanted to add configuration (here, in another PR) for that it would be awesome but I can still merge this one without it.

Good point. I'll do it here, next week seems reasonable.


Venus "Settings > Enery meters" is empty and 3rd party script don't read AC Loads

Hi,
I have this script running and it looks like it's working as expected. On the home screen I see Grid and AC Loads with correct values.
Once I go into the Venus Settings and scroll down to "Energy meters" it's empty. Is this a problem?
I'm now trying to combine my Venus OS with https://github.com/Telekatz/venus.dbus-hoymiles and this script has a "Setup > Power Meters" section which says "Unknown". Sounds similar to the "Enery meters" in the Venus Settings? It seem to read from com.victronenergy.acload https://github.com/Telekatz/venus.dbus-hoymiles/blob/89568b623ee798afa108b93a317de1868b239a1f/HMpvinverter.py#L1162 ?
Do you know if venus.dbus-hoymiles is reading it wrong or do you miss something while publishing the AC loads?

Hello,
it is normal that the Energy Meter is empty, since this is a grid meter. Since this driver is intended to be a grid meter and not energy meter it‘s not pushing the data to com.victronenergy.acload. I also didn‘t find documentation about this in the dbus wiki https://github.com/victronenergy/venus/wiki/dbus
I will take a look in the next days.


Impossible to start Prometheus with storage.tsdb.path on a SeaweedFS mount

What did you do?
I have a SeaweedFS setup, where I'd like to store my persistent data. When I start Prometheus, with the --storage.tsdb.path="/nomad/prometheus", it errors. This /nomad is a seaweedfs mount point.
What did you expect to see?
Prometheus should just start-up fine, and store the metrics in /nomad/prometheus.
What did you see instead? Under which circumstances?
Prometheus failed to start with all the default configuration, except for the --storage.tsdb.path option.
Environment

System information:
Linux 3.10.0-957.12.2.el7.x86_64 x86_64

Prometheus version:

prometheus, version 2.18.0 (branch: HEAD, revision: a12e96299dcd159ea09b260f1a21e7e4b86e011d)
  build user:       root@7fbcff55abdb
  build date:       20200505-14:26:04
  go version:       go1.14.2

Prometheus configuration file:

# my global config
global:
  scrape_interval:     15s # Set the scrape interval to every 15 seconds. Default is every 1 minute.
  evaluation_interval: 15s # Evaluate rules every 15 seconds. The default is every 1 minute.
  # scrape_timeout is set to the global default (10s).

# Alertmanager configuration
alerting:
  alertmanagers:
  - static_configs:
    - targets:
      # - alertmanager:9093

# Load rules once and periodically evaluate them according to the global 'evaluation_interval'.
rule_files:
  # - "first_rules.yml"
  # - "second_rules.yml"

# A scrape configuration containing exactly one endpoint to scrape:
# Here it's Prometheus itself.
scrape_configs:
  # The job name is added as a label `job=<job_name>` to any timeseries scraped from this config.
  - job_name: 'prometheus'

    # metrics_path defaults to '/metrics'
    # scheme defaults to 'http'.

    static_configs:
    - targets: ['localhost:9090']

Logs:

[root@server1 prometheus-2.18.0.linux-amd64]# ./prometheus --storage.tsdb.path=/nomad/prometheus
level=info ts=2020-05-07T16:12:09.995Z caller=main.go:302 msg="No time or size retention was set so using the default time retention" duration=15d
level=info ts=2020-05-07T16:12:09.995Z caller=main.go:337 msg="Starting Prometheus" version="(version=2.18.0, branch=HEAD, revision=a12e96299dcd159ea09b260f1a21e7e4b86e011d)"
level=info ts=2020-05-07T16:12:09.995Z caller=main.go:338 build_context="(go=go1.14.2, user=root@7fbcff55abdb, date=20200505-14:26:04)"
level=info ts=2020-05-07T16:12:09.995Z caller=main.go:339 host_details="(Linux 3.10.0-957.12.2.el7.x86_64 #1 SMP Tue May 14 21:24:32 UTC 2019 x86_64 server1.homelab (none))"
level=info ts=2020-05-07T16:12:09.995Z caller=main.go:340 fd_limits="(soft=1024, hard=4096)"
level=info ts=2020-05-07T16:12:09.995Z caller=main.go:341 vm_limits="(soft=unlimited, hard=unlimited)"
level=error ts=2020-05-07T16:12:09.996Z caller=query_logger.go:71 component=activeQueryTracker msg="Failed to read query log file" err=EOF
unexpected fault address 0x7fe0d51a2000
fatal error: fault
[signal SIGBUS: bus error code=0x2 addr=0x7fe0d51a2000 pc=0x4687e8]

goroutine 1 [running]:
runtime.throw(0x26b4471, 0x5)
        /usr/local/go/src/runtime/panic.go:1116 +0x72 fp=0xc0008c7340 sp=0xc0008c7310 pc=0x434af2
runtime.sigpanic()
        /usr/local/go/src/runtime/signal_unix.go:692 +0x443 fp=0xc0008c7370 sp=0xc0008c7340 pc=0x44b5e3
runtime.memmove(0x7fe0d51a2000, 0x26b1946, 0x1)
        /usr/local/go/src/runtime/memmove_amd64.s:148 +0x108 fp=0xc0008c7378 sp=0xc0008c7370 pc=0x4687e8
github.com/prometheus/prometheus/promql.NewActiveQueryTracker(0x7ffe88d11723, 0x11, 0x14, 0x2e940a0, 0xc00062ff50, 0x2e940a0)
        /app/promql/query_logger.go:120 +0x3a3 fp=0xc0008c74d0 sp=0xc0008c7378 pc=0x1dbb573
main.main()
        /app/cmd/prometheus/main.go:368 +0x4ed8 fp=0xc0008c7f88 sp=0xc0008c74d0 pc=0x1f59888
runtime.main()
        /usr/local/go/src/runtime/proc.go:203 +0x212 fp=0xc0008c7fe0 sp=0xc0008c7f88 pc=0x437172
runtime.goexit()
        /usr/local/go/src/runtime/asm_amd64.s:1373 +0x1 fp=0xc0008c7fe8 sp=0xc0008c7fe0 pc=0x467521

goroutine 36 [select]:
go.opencensus.io/stats/view.(*worker).start(0xc000093bd0)
        /app/vendor/go.opencensus.io/stats/view/worker.go:154 +0x100
created by go.opencensus.io/stats/view.init.0
        /app/vendor/go.opencensus.io/stats/view/worker.go:32 +0x57

goroutine 39 [chan receive]:
k8s.io/klog/v2.(*loggingT).flushDaemon(0x42271e0)
        /app/vendor/k8s.io/klog/v2/klog.go:1107 +0x8b
created by k8s.io/klog/v2.init.0
        /app/vendor/k8s.io/klog/v2/klog.go:416 +0xd6

goroutine 78 [select]:
github.com/prometheus/prometheus/pkg/logging.(*Deduper).run(0xc00012afc0)
        /app/pkg/logging/dedupe.go:75 +0x1db
created by github.com/prometheus/prometheus/pkg/logging.Dedupe
        /app/pkg/logging/dedupe.go:61 +0xcf

goroutine 98 [chan receive]:
github.com/prometheus/prometheus/storage/remote.(*WriteStorage).run(0xc00043b030)
        /app/storage/remote/write.go:84 +0xb2
created by github.com/prometheus/prometheus/storage/remote.NewWriteStorage
        /app/storage/remote/write.go:77 +0x1a9

Thank you for your report.
NFS and other distributes filesystems are not supported by Prometheus. We strongly encourage our users to use local storage which meets POSIX compliance.
We have mechanisms like federation / remote write to scale out if you need to.


$request->input returns array|string|null, string expected

As stated in #112, Laravel's method signature for Illuminate\Foundation\Http\FormRequest::input() is array|string|null. From Laravel's perspective, this will be used together with the method rules which allows the developer to state the requirements for an input field (e.g. array or string).
This pull request instructs phpstan to consider the return type as dynamic instead. Since this is my first-ever contribution to a phpstan extension, I have a couple of concerns to highlight:

The method actually comes from trait Illuminate\Http\Concerns\InteractsWithInput. Should the extension target the trait or only concrete classes?
This pull request only targets FormRequest, however Illuminate\Http\Request will have the same issue as it also uses the InteractsWithInput trait. Should there be one extension for each class or is there a way to have a single extension that targets both?

Any feedback is well appreciated and thanks for this wonderful tool, both larastan and phpstan!

Here's a MVP of the problem: https://phpstan.org/r/708f863bb376f24259086d30ee8be72e
By doing any string manipulation, PHPStan will complain that you might be manipulating string with a possible array. Or, if you also use it in an array function, PHPStan will tell you that you're using a possible string where an array is expected. By changing from array|string|null to mixed, PHPStan will not complain anymore.

Would it be possible to statically read the rules and find out whether the method returns array|null OR string|null?

@ondrejmirtes I would say no, unless you have a valid Laravel container. The Rules method is invoked via the Container and can have any dependency in it's signature. Also, any class implementing the Rule interface can act as a ruler, which makes it harder to figure out which output will be.
Also, from the HTTP definitions, I don't think it's possible to build anything other than array or string. Since laravel will convert empty string / missing keys into null, that gives us only those 3 scenarios: array, string or null.

Maybe you should think about using BenevolentUnionType of array/string/null instead of mixed. That means that if you use it as something completely else (like stdClass), PHPStan will tell you, but it won't complain in most situations normal UnionType complains.

Thanks! 🏄♂

@deleugpn I am open for more extensions like that.


Update tx service delegate methods

Closes https://github.com/safe-global/safe-cli/issues/425

A new release of safe-eth-py is needed as the method that calculates the delegate signature in the current version is incorrect. Anyway, the task work in safe-cli would be ready.

coverage: 87.308% (+0.03%) from 87.279%
when pulling 82ef773b8620a1cbbb685b35fa381e1ff01b8819 on update_tx_service_delegate_methods
into a9427b222f74866642a6a37ee1a2f5df19826bd2 on main.

coverage: 87.497% (+0.03%) from 87.47%
when pulling ca3750a77842798c63476748f3dfb70a15ae27fb on update_tx_service_delegate_methods
into ddd66d66093ddebbe1a9301319e1b8121d21a81d on main.

coverage: 88.593% (+0.02%) from 88.571%
when pulling 95e874c40f32bda088045438123bc9b225c0b2a8 on update_tx_service_delegate_methods
into f343bb8c4020de06108c453e290327166d29afe2 on main.

coverage: 88.602% (+0.03%) from 88.571%
when pulling 465f46eb4fd4700f296c7285c563fac2bfbd625c on update_tx_service_delegate_methods
into f343bb8c4020de06108c453e290327166d29afe2 on main.

Added a blocked label to wait for a new version of safe-eth-py with the signature EIP712 fixed

coverage: 88.602% (+0.02%) from 88.58%
when pulling ab20b376ba7c6343dd5efcbe44ecda9f3b1e1cd9 on update_tx_service_delegate_methods
into d59dc929a03de12bb50693ffd7a461c040dec79d on main.

coverage: 88.602% (+0.02%) from 88.58%
when pulling 58f2ecec4180af378c58e02f1b92ab15b7b67dd6 on update_tx_service_delegate_methods
into d59dc929a03de12bb50693ffd7a461c040dec79d on main.


⚠ Bofrost.at has degraded performance

In ac410ad, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1328 ms

Resolved: Bofrost.at performance has improved in b47c220.


phoebemirman/ch11603/depth-chart-depth-header-value-not-shown

Clubhouse Story
Submitter checklist

[ ] Test covering functionality added/fixed in
Check the linked story includes the following:

[ ] Steps to reproduce
[ ] Screenshot (If applicable)

Reviewer Checklist

[ ] Test/lint pass

@sharkcrayon see comment on CH task, maybe we just want to remove DEPTH header to make more room?


[Bug Report]: Message: Trying to access array offset on value of type bool

What operating system are you using?
Linux
What version of PHP do you use?
PHP 7.x
Where did you get the CMS code?
Github
Where are you trying to install the CMS?
Localhost
Which emulator are you using and which version?
TrinityCore
Describe the bug
After filling out the information to log into the databases i get several errors.
Error 1:
A PHP Error was encountered
Severity: Notice
Message: Trying to access array offset on value of type bool
Filename: core/MY_Lang.php
Line Number: 97
Backtrace:
File: /home/painwithin/public_html/register/application/core/MY_Lang.php
Line: 97
Function: _error_handler
File: /home/painwithin/public_html/register/index.php
Line: 320
Function: require_once
Error 2:
A PHP Error was encountered
Severity: Notice
Message: Trying to access array offset on value of type bool
Filename: core/MY_Lang.php
Line Number: 109
Backtrace:
File: /home/painwithin/public_html/register/application/core/MY_Lang.php
Line: 109
Function: _error_handler
File: /home/painwithin/public_html/register/index.php
Line: 320
Function: require_once
Error 3
A PHP Error was encountered
Severity: Notice
Message: Trying to access array offset on value of type null
Filename: core/MY_Lang.php
Line Number: 109
Backtrace:
File: /home/painwithin/public_html/register/application/core/MY_Lang.php
Line: 109
Function: _error_handler
File: /home/painwithin/public_html/register/index.php
Line: 320
Function: require_once
and Error 4:
A PHP Error was encountered
Severity: Warning
Message: Cannot modify header information - headers already sent by (output started at /home/painwithin/public_html/register/system/core/Exceptions.php:271)
Filename: core/MY_Lang.php
Line Number: 119
Backtrace:
File: /home/painwithin/public_html/register/application/core/MY_Lang.php
Line: 119
Function: header
File: /home/painwithin/public_html/register/index.php
Line: 320
Function: require_once
How can I reproduce the problem?
I am unsure how i can produce a way for someone else to get these errors, i simply filled in the form for the configuration of the database and then these errors pop up, im not sure if its something im somehow doing wrong or not.
Screenshots
No response
To add some additional information, my WoW Server is on a Windows VM and the web server is on a linux VM "i use proxmox" so i put the IP information in accordingly to the auth and the database i made for the site.

Do you have communication between the two machines? I ask just in case, because it may be a port issue. I don't know in which part of the site generates the error, when entering to which functionality?

Yes I do. I use another reg page atm and it works just not sure why this one isnt.

Also, does this error mean there is an issue with it communicating with the database? if so then i will have to dig deeper into what the issue is as it does not give an error message other then the ones i posted.

What version of PHP 7 are you using? Just in case, try to use version 7.3 or 7.4. At the moment version 8 is not supported, so try to use the latest available version 7. Please let me know if the problem persists, to see what we can find out about it.


Add Activesupport 7.1 support

Merge #289 first
This PR is based on #289 and add ActiveSupport 7.1 testing and fix.

Too much unrelated code, I will open another cleaner PR.


Update your bad code

The code for https://github.com/pwnbus/scoring_engine/pull/111 is bad, and I feel bad. I should update it to make it good.

So, I actually went back and the code doesn't look that bad, not sure what I was talking about haha.


Avoid generated files

Generated source files are not generally expected to follow proper Go style. For example, running lint on a protobuf-generated Go file reports hundreds of lint issues.
With the upcoming generator disclaimer headers, we should consider having lint ignore these types of source files: golang/go#13560

Is it jumping the gun to say that the consensus is look for one of "DO NOT EDIT" or "Code generated by" in the first comment block?

Now that the code generation header is formalized via https://golang.org/s/generatedcode, this seems tractable.

I'm still experiencing linting alerts for generated files. I'm running go 1.8, my $GOPATH.../lint repo's master references c5fb716 which appears current.
I have two files generated by protoc-gen-go whose header block matches the pattern// Code generated ...\n\n... DO NOT EDIT."
I expected that when I run golint in the folder with these, that i would not receive warnings, but actually I still get the warnings.
Ditto for running golint from vim-go.
Unless I'm missing something, the isGenerated function is broken. repro'd here: https://gist.github.com/therealplato/27e1c3ea716b25674ca158c9b9496cd2

You must have an old version of protoc-gen-go. This was fixed in https://github.com/golang/protobuf/commit/47eb67eaf5cab63c58956d4d4ce86b03ad5eaa03

Thanks @dsnet. The code was certainly generated before that commit

"go.lintFlags": [
    "--exclude=\"\bexported \\w+ (\\S*['.]*)([a-zA-Z'.*]*) should have comment or be unexported\b\""
]


IE compatibility

You use Error#stack quite a bit in the code which was only supported by IE10+

Old IE don't have stack info, there's no way to get custom stack.

Thanks, fixed.


Move provider scripts into cmd directory

All entry scripts will be under cmd.

@kaifeng looks good, just some merge conflicts to resolve.


Introduce complete JNA implementation of cipher and cryptorandom

Before merging this we should benchmark it and compare performance to JNI openssl binding

Thank @salyh for your great work.
I prefer the JNA implementation if performance does not decrease too much. I will do some performance test on it.

The implementation works OK for many cases, but the Cipher class crashes the JVM if

init() is called after close()
close() is called twice

Commenting out the call to OpensslNativeJna.EVP_CIPHER_CTX_free(context); in close() fixes the problem, but that means the context is never freed.

this seems to be merged somehow with https://github.com/apache/commons-crypto/commit/b6006e563f63c745c964a87fae459200ae24573b

Thank @salyh for your contribution.


Ability to query API resources that match any or multiple words of a search term

Currently, it is not possible to implement a keyword-based search. For example, if "professional development" is searched, a partial-match query does not return results for "professional" and/or "development". It tries to match "professional development" entirely.
This the query that I've attempted to use to implement a keyword-based search:
https://github.com/restarone/violet_rails/blob/master/test/models/concerns/jsonb_search/query_builder_test.rb#L33

adding a bit more detail:

we want the system to given a query like "Professional Development" and have the ability to search for keywords (in this case, delineated by spaces) like "Professional" and "Development"  and show results that include "Professional" and/or "Development"


add VEP 105 CSQ FIELDs

make VEP_CSQ_FIELDS a dictionary to contain both 101 & 105 fields, but make VEP_CSV_HEADER default to 105. Double checked vep_struct_to_csq(), no exceptions need to be added.
remove the missing fields: MINIMISED, SWISSPROT|TREMBL|UNIPARC from VEP 105 CSQ fields. ancestal & context were not in the CSQ fields of VEP 101.

@mike-w-wilson I wasn't sure if I should also change line 391. My intension was to keep both 101 & 105 fields.

Yes, please change the linked line 391 VEP_CSQ_FIELDS. You can have it default to CURRENT_VEP_VERSION

@mike-w-wilson, I added VEP_VERSIONS & CURRENT_VEP_VERSION.


(#877) Remove usage of Cake.Figlet

Use the built in aliases that are part of Spectre.Console.
Fixes #877

This will need to be rebased once this PR has been completed: https://github.com/cake-contrib/Cake.Recipe/pull/876


Holograms not saving; Keep reverting back after restart

Proof:
https://gyazo.com/893f2c2076b25a21dd8826c9bc6d558a

I am having the same problem as well.

Please confirm this behavior is still happening with the changes made in cecc517


Error: An item with the same key has already been added

I am not sure if it's a paket bug but as it have something with the restore target and dependency management I am posting it here for now.
I have a project with this structure:
Fulma.fsproj:

Is a netstandard2.0 fsproj
TargetFramework: netstandard1.6

Fulma.Extensions.fsproj:

Is a netstandard2.0 fsproj
TargetFramework: netstandard1.6
It include a reference over Fulma via paket.references

Docs.fsproj:

Is a netstandard2.0 fsproj

TargetFramework: netstandard2.0

Include references on the twos others project via ProjectReference attributes
<ItemGroup>
   <ProjectReference Include="../src/Fulma/Fulma.fsproj" />
   <ProjectReference Include="../src/Fulma.Extensions/Fulma.Extensions.fsproj" />
 </ItemGroup>

When running dotnet restore on Docs.fsproj I got this error:
docs feature/conversion *$ = $ dotnet restore
  Paket version 5.92.0-beta003
   - C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\docs\obj\docs.fsproj.netstandard2.0.paket.resolved created
   - C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\docs\obj\docs.fsproj.paket.clitools created
   - C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\docs\obj\docs.fsproj.paket.props created
   - C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\src\Fulma\obj\Fulma.fsproj.netstandard1.6.paket.resolved created
   - C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\src\Fulma.Extensions\obj\Fulma.Extensions.fsproj.netstandard1.6.paket.resolved created
  Performance:
   - Disk IO: 56 milliseconds
   - Runtime: 2 seconds
  Restoring packages for C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\src\Fulma\Fulma.fsproj...
  Restoring packages for C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\docs\docs.fsproj...
  Restoring packages for C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\docs\docs.fsproj...
  Restoring packages for C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\src\Fulma.Extensions\Fulma.Extensions.fsproj...
C:\Program Files\dotnet\sdk\2.0.0\NuGet.targets(102,5): error : An item with the same key has already been added. Key: (Fulma, 0.2.0-beta-002) [C:\Users\mange\Workspace\Github\MangelMaxime\Fable.Elmish.Bulma\docs\docs.fsproj]

I tested it with this version of paket:

5.92.0
5.92.0-beta003

You can reproduct this issue with this project on the branch feature/conversion
Steps:

Clone the repo on branch feature/conversion
Run .paket/paket.exe install
cd docs
dotnet restore

Also, when I was building the libs to use netstandard2.0 as fsproj format and TargetFramework it's was working as expected.

I forget to tell something.
If I change
<ItemGroup>
   <ProjectReference Include="../src/Fulma/Fulma.fsproj" />
   <ProjectReference Include="../src/Fulma.Extensions/Fulma.Extensions.fsproj" />
 </ItemGroup>

to
<ItemGroup>
   <ProjectReference Include="../src/Fulma.Extensions/Fulma.Extensions.fsproj" />
 </ItemGroup>

before the running dotnet restore it's succeed to restore the projet.

Fulma.Extensions is already making a reference to Fulma Nuget package so I guess you cannot have the same .fsproj and Paket reference at once. It doesn't look as a Paket error to me, though I'm not sure if MSBuild/Nuget could handle a situation like of this or if the error message could be more informative.

We probably would accept a PR if it either:

Improves the error message to clearly indicate what @alfonsogarciacaro has already said
Filters out all packages with the same name as a project reference

No matter which one is chosen we probably need a test.


Error when building using cmake through Bazel on OSX

Hi there,
I'm getting an error when building the library from source with Bazel using an external dependency rule. I believe that this problem is related to bazel defining DEBUG to be 1, which they claim is the correct behavior  on MacOSX. The word DEBUG is used in at least one place in the code base and this creates problems. Here is the error:
/var/tmp/_bazel_grantreszczyk/0c7662858b5d6ee977c0c3f6d357e89a/sandbox/darwin-sandbox/12/execroot/__main__/external/glfw_src/tests/glfwinfo.c:361:39: error: expected identifier
    enum { CLIENT, CONTEXT, BEHAVIOR, DEBUG, FORWARD, HELP, EXTENSIONS, LAYERS,
                                      ^
<command line>:2:15: note: expanded from here
#define DEBUG 1
              ^
/var/tmp/_bazel_grantreszczyk/0c7662858b5d6ee977c0c3f6d357e89a/sandbox/darwin-sandbox/12/execroot/__main__/external/glfw_src/tests/glfwinfo.c:463:18: error: duplicate case value: 'CONTEXT' and '1' both equal '1'
            case DEBUG:
                 ^
<command line>:2:15: note: expanded from here
#define DEBUG 1
              ^
/var/tmp/_bazel_grantreszczyk/0c7662858b5d6ee977c0c3f6d357e89a/sandbox/darwin-sandbox/12/execroot/__main__/external/glfw_src/tests/glfwinfo.c:449:18: note: previous case defined here
            case CONTEXT:
                 ^
2 errors generated.
make[2]: *** [tests/CMakeFiles/glfwinfo.dir/glfwinfo.c.o] Error 1
make[1]: *** [tests/CMakeFiles/glfwinfo.dir/all] Error 2
make: *** [all] Error 2

When I compile with optimization flags, the library builds as expected.

I find it strange that the development environment would rely on a non-reserved identifier this way, but it's easy enough to work around on our end.
Note that it's only the "glfwinfo" test program that triggers the issue, not the library itself. It should be possible to configure your dependency (sorry, I don't know anything about Bazel) to skip building the test and example programs if all you want is the library itself.


axios vuln - CVE-2020-28168

Description
bolt-js is currently utilising axios v0.19.0 which is affected by CVE-2020-28168 (SNYK-JS-AXIOS-1038255) which is present in all versions of axios prior to v0.21.1.
Vuln was fixed in v0.21.1 - https://github.com/axios/axios/pull/3410
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version: 2.5.0
node version: 15.0.1
OS version(s): MacOS 11.1
Steps to reproduce:

Scanned via Snyk<EMAIL_ADDRESS>
Expected result:
N/A
Actual result:
N/A
Attachments:
CVE Entry: https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2020-28168
Snyk Vuln DB: https://app.snyk.io/vuln/SNYK-JS-AXIOS-1038255

some realtime thoughts:

This project is only vulnerable to this issue because in App.ts we're spreading the user-supplied clientTls option directly into the call for axios.create(), allowing for an unintended means of use. The intention is that clientTls should only contain keys pertaining to SecureContextOptions from node, but in theory a user could add the proxy option to that object to trigger the insecure behavior. So an alternative fix could be to filter out the acceptable keys in clientTls.

Why should the above even matter when we can just bump the dependency? I need to look closer, but if my memory serves correctly, there were some undesirable changes (perhaps breaking changes or known bugs) in later versions of axios that we intentionally opted out of. As long as that's not the case with the latest releases, and there's no breaking change that cannot be just as easily resolved, I agree think bumping the version is the best path forward.

some realtime thoughts:

This project is only vulnerable to this issue because in App.ts we're spreading the user-supplied clientTls option directly into the call for axios.create(), allowing for an unintended means of use. The intention is that clientTls should only contain keys pertaining to SecureContextOptions from node, but in theory a user could add the proxy option to that object to trigger the insecure behavior. So an alternative fix could be to filter out the acceptable keys in clientTls.

Why should the above even matter when we can just bump the dependency? I need to look closer, but if my memory serves correctly, there were some undesirable changes (perhaps breaking changes or known bugs) in later versions of axios that we intentionally opted out of. As long as that's not the case with the latest releases, and there's no breaking change that cannot be just as easily resolved, I agree think bumping the version is the best path forward.

the @slack/web-api package is not vulnerable since we've fixed the proxy option to false.
https://github.com/slackapi/node-slack-sdk/blob/2e9ad5804709c9548342d0da967b4ef706994729/packages/web-api/src/WebClient.ts#L123-L141

the @slack/web-api package is not vulnerable since we've fixed the proxy option to false.
https://github.com/slackapi/node-slack-sdk/blob/2e9ad5804709c9548342d0da967b4ef706994729/packages/web-api/src/WebClient.ts#L123-L141

This has been fixed in version 2.6.0! Thanks!


border_mode = 'same' in pooling layers still changes the dimentions

I am trying to use a pooling layer (both 1D and 2D) with border_mode = 'same' but it seems to change the dimensions anyway

1D: stride=(pool_length/2)
2D: stride=(pool_size[0]/2, pool_size[1]/2)
Also, this is entirely expected behavior - pooling is generally used to decrease output size, and definitely will when you use the default stride.

Yes but border_mode = 'same' should ZeroPad back to the original dimensions.

Yes, if stride == poolSize / 2. By default, stride == poolSize
-----Original Message-----
From: "Yam Peleg"<EMAIL_ADDRESS>Sent: 2/19/2017 11:23 PM
To: "fchollet/keras"<EMAIL_ADDRESS>Cc: "Pat York"<EMAIL_ADDRESS>"Comment"<EMAIL_ADDRESS>Subject: Re: [fchollet/keras] border_mode = 'same' in pooling layers stillchanges the dimentions (#5448)
Yes but border_mode = 'same' should ZeroPad back to the original dimensions.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

can't i use a different stride size? as far as i know, strides are used to "skip" cells when applying the pooling filter, am i right?

Yes, and if you skip cells (more than poolSize/2) you reduce dimensionality.
Draw a picture of a simple example (4x4 image, 2 pool and 2 stride) see what size you end up with when you do no padding, 1 (same) padding, and 2 (full padding). You'll see the dimensionality is reduced in the first 2 cases, but not in the 3rd. However, in the third you'll find that the output has a border of padding, which is essentially the output of border_mode=valid followed with a Padding2D((1,1)) layer. In other words, sure the dimensionality is technically the same, but its because it was padded with zeros..
-----Original Message-----
From: "Yam Peleg"<EMAIL_ADDRESS>Sent: 2/19/2017 11:29 PM
To: "fchollet/keras"<EMAIL_ADDRESS>Cc: "Pat York"<EMAIL_ADDRESS>"Comment"<EMAIL_ADDRESS>Subject: Re: [fchollet/keras] border_mode = 'same' in pooling layers stillchanges the dimentions (#5448)
can't i use a different stride size? as far as i know, strides are used to "skip" cells when applying the pooling filter, am i right?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you, this is what i was asking. If this is intentional, i just wanted to pad the output with zeros even though it is not at the same dimensions.
I'll just use zero padding layers for this.
However i still think automatic zero padding should be considered to be added as a border mode.

With border mode=full it is exactly what you are asking for: An artificially inflated output size. I don't know if keras allows that border mode in pooling layers, because it is only available in Theano and is equivalent to just padding afterwards. You're right that a subsequent padding layer will accomplish what you need. However, I recommend you still draw a picture so that you understand why the border mode, pool size, and stride interact in the way they do.
Please close this issue, and direct any future implementation/non-bug questions to the Slack channel or Stackoverflow.
-----Original Message-----
From: "Yam Peleg"<EMAIL_ADDRESS>Sent: 2/19/2017 11:42 PM
To: "fchollet/keras"<EMAIL_ADDRESS>Cc: "Pat York"<EMAIL_ADDRESS>"Comment"<EMAIL_ADDRESS>Subject: Re: [fchollet/keras] border_mode = 'same' in pooling layers stillchanges the dimentions (#5448)
Thank you, this is what i was asking. If this is intentional, i just wanted to pad the output with zeros even though it is not at the same dimensions.
I'll just use zero padding layers for this.
However i still think automatic zero padding should be considered to be added as a border mode.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Deferred when request is closed

I have an http-server request handler that holds onto some resources until an action is complete. If the request is prematurely closed by the client, I'd like to release those resources. Is there a way to determine if a client has closed a request? It would be useful to have this event in a deferred.

If you'd like to simply check if a connection is closed, you can do:
(-> req ^io.netty.channel.Channel .ch .isOpen)

If you'd like it as a deferred that's realized when the channel is closed, you can do:
(-> req ^io.netty.channel.Channel .ch .closeFuture aleph.netty/wrap-future)

Hope that helps.

This is exactly what I was looking for. Thanks!


Authors should be created and notified when articles are

Article creation now cascades author creation.
This issue can be expanded to allow for authors to be notified on how to use the AMS.


Update bibtex for GPflow paper.

@jameshensman we should probably update the reference to the paper to the peer reviewed version, no?

Codecov Report

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

@@           Coverage Diff           @@
##           master     #443   +/-   ##
=======================================
  Coverage   99.83%   99.83%
=======================================
  Files          28       28
  Lines        2376     2376
=======================================
  Hits         2372     2372
  Misses          4        4

Continue to review full report at Codecov.

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


fix: typo in storybook verbiage

Simple change. While debugging https://github.com/trussworks/react-uswds/issues/869, I noticed a typo in our stories.

Warnings

:warning:
This PR does not include changes to tests, even though it affects source code.

  Generated by :no_entry_sign: dangerJS against 9d4622cf32c93e6c8a650b10cdacfd55fb3b4ad2


Tab title is wrongly cropped

VSCode Version: 1.42.0
OS Version: Ubuntu 19.10

Steps to Reproduce:

Open a file named model.pxd
Look at its tab title:

Does this issue occur when all extensions are disabled?: Yes
I'm not 100% sure but this might be a regression in 1.42.0.
Settings that may be relevant:
    "window.zoomLevel": 0.70,
    "window.menuBarVisibility": "toggle",

My screen is hidpi, scaling factor is 2x, font is default Ubuntu one.

This does not look new to me. I think italic fonts are not pushing the parent div large enough for some reason. Up for help if someone has clues how to resolve this, ideally with CSS and no hacks.

This seems to only happen on Linux and seems to be related to using overflow: hidden. I have reported this as https://bugs.chromium.org/p/chromium/issues/detail?id=503426


跑infer.sh样例报错

跑样例报错
(ms) root@v4ae1f9ac727411f9a25ff403f1244d0-task0-0:/code/minddiffusion/vision/stablediffusionv2# bash scripts/infer.sh
workspace /code/minddiffusion/vision/stablediffusionv2
WORK DIR:/code/minddiffusion/vision/stablediffusionv2
Loading model from models/stablediffusionv2_512.ckpt
LatentDiffusion: Running in eps-prediction mode
making attention of type 'vanilla' with 512 in_channels
Working with z of shape (1, 4, 32, 32) = 4096 dimensions.
making attention of type 'vanilla' with 512 in_channels
param not load: (['first_stage_model.encoder.down.3.downsample.conv.weight', 'first_stage_model.encoder.down.3.downsample.conv.bias', 'first_stage_model.decoder.up.0.upsample.conv.weight', 'first_stage_model.decoder.up.0.upsample.conv.bias'], ['cond_stage_model.transformer.transformer_layer.resblocks.23.attn.attn.in_proj.bias', 'cond_stage_model.transformer.transformer_layer.resblocks.23.attn.attn.in_proj.weight', 'cond_stage_model.transformer.transformer_layer.resblocks.23.attn.attn.out_proj.bias', 'cond_stage_model.transformer.transformer_layer.resblocks.23.attn.attn.out_proj.weight', 'cond_stage_model.transformer.transformer_layer.resblocks.23.c_fc.bias', 'cond_stage_model.transformer.transformer_layer.resblocks.23.c_fc.weight', 'cond_stage_model.transformer.transformer_layer.resblocks.23.c_proj.bias', 'cond_stage_model.transformer.transformer_layer.resblocks.23.c_proj.weight', 'cond_stage_model.transformer.transformer_layer.resblocks.23.ln_1.beta', 'cond_stage_model.transformer.transformer_layer.resblocks.23.ln_1.gamma', 'cond_stage_model.transformer.transformer_layer.resblocks.23.ln_2.beta', 'cond_stage_model.transformer.transformer_layer.resblocks.23.ln_2.gamma'])
Data shape for PLMS sampling is (8, 4, 64, 64)
Running PLMS Sampling with 50 timesteps
Traceback (most recent call last):
  File "txt2img.py", line 294, in <module>
    main()
  File "txt2img.py", line 269, in main
    x_T=start_code
  File "/code/minddiffusion/vision/stablediffusionv2/ldm/models/diffusion/plms.py", line 120, in sample
    unconditional_conditioning=unconditional_conditioning,
  File "/code/minddiffusion/vision/stablediffusionv2/ldm/models/diffusion/plms.py", line 167, in plms_sampling
    old_eps=old_eps, t_next=ts_next)
  File "/code/minddiffusion/vision/stablediffusionv2/ldm/models/diffusion/plms.py", line 230, in p_sample_plms
    e_t = get_model_output(x, t)
  File "/code/minddiffusion/vision/stablediffusionv2/ldm/models/diffusion/plms.py", line 195, in get_model_output
    e_t_uncond, e_t = ops.split((self.model.apply_model(x_in, t_in, c_in)), 0, 2)
  File "/root/miniconda3/envs/ms/lib/python3.7/site-packages/mindspore/ops/function/array_func.py", line 5259, in split
    raise ValueError(f"For split, the value of 'split_size_or_sections' must be more than zero, "
ValueError: For split, the value of 'split_size_or_sections' must be more than zero, but got 0.

猜想可能是mindspore版本接口不适配了。mindspore用的是2.0rc1

2.0接口变了一些，要重新修改脚本的，暂时我们还没适配2.0版本。


add needsUpdate for AnchorSystem

heads up @michaelthatsit - forgot to note this on the pr, adding it in since it's a small fix to the merged version - it's a change based on the optimization from https://github.com/Volumetrics-io/mrjs/pull/211
im going to merge this to get it going, but just wanted to give u a heads up

Wait how are we indicating when something needs an update? Anchors can move anytime so its one of those things that needs to be updated every frame to keep transforms in sync with the real world anchor.

Wait how are we indicating when something needs an update? Anchors can move anytime so its one of those things that needs to be updated every frame to keep transforms in sync with the real world anchor.

yea this is to make sure the needsUpdate always runs as true - otherwise it'll rely on if the registry is set or not per: https://github.com/Volumetrics-io/mrjs/blob/main/src/core/MRSystem.js#L75 , so adding in this change lets it still run every frame (ie the return true)
lmk if this makes sense - can hop on a quick call to explain as well if that helps

updated the comment in the function for better clarity as well - https://github.com/Volumetrics-io/mrjs/pull/238

Ah ok. I'm out rn but I'll take a closer look when I get back. But sounds good to me.


runtime: External tests not run with the correct compiler

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go 1.7rc4

What did you do?

$ cd go/
$ ./src/make.bash
$ ./bin/go test -short runtime

What did you expect to see?
Tests run with ./bin/go

What did you see instead?
Tests run with the first "go" in $PATH (in this case, the "go" on $PATH was broken, so the tests failed.)

There's already this in testenv:
// GoToolPath reports the path to the Go tool.
// If the tool is unavailable GoToolPath calls t.Skip.
// If the tool should be available and isn't, GoToolPath calls t.Fatal.
func GoToolPath(t *testing.T) string {
        MustHaveGoBuild(t)

There's probably stuff not using it.

Excellent, that's new to me.  We should use GoToolPath instead of the MustHaveGoBuild + "go" combo.
Same for MustHaveGoRun, I guess.


Create abstract base class for common fields

Fixes #41
Introduces base.models.BasePageFieldsMixin which includes image and introductionfields.
Currently utilized in:

base.models.GalleryPage
blog.models.BlogIndexPage
breads.models.BreadsIndexPage
locations.models.LocationsIndexPage

Review other pages for use of the mixin:

[x] BlogPage
[ ] BreadPage
[x] LocationPage

Hey wow, I had actually done half of this work last night, was going to finish today, but you beat me to it. Thanks! Code looks great, and I pulled it down and it migrated and works fine. My only question is: Shouldn't every page of the site have a large hero image and intro? (both index and detail pages)? If so, then this should also be applied to BlogPage, BreadPage, and LocationPage, not just the index pages. If you agree, can you do a 2nd commit on this PR to add those? Or do you disagree?

Apologies! I was going under the notion that if a ticket was unassigned, it
was free for the taking.
I'll review the other pages; you are likely correct!
On Sat, Feb 18, 2017, 6:35 PM Scot Hacker<EMAIL_ADDRESS>wrote:
Hey wow, I had actually done half of this work last night, was going to
finish today, but you beat me to it. Thanks! Code looks great, and I pulled
it down and it migrated and works fine. My only question is: Shouldn't
every page of the site have a large hero image and intro? (both index and
detail pages)? If so, then this should also be applied to BlogPage,
BreadPage, and LocationPage, not just the index pages. If you agree, can
you do a 2nd commit on this PR to add those? Or do you disagree?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wagtail/bakerydemo/pull/47#issuecomment-280883337, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AARNaWNMs8M8xKlSV2eZS3jdQUckPe26ks5rd4A1gaJpZM4MFF--
.

Of course you're right that unassigned tickets are up for grabs - I should have thought to assign it to myself. All yours now :)

@shacker I would like some feedback as to how to best approach BlogPage and BreadPage:
BlogPage
This model is currently differentiated by the addition of a subtitle field and how the content panels are configured:
        MultiFieldPanel([
            FieldPanel('subtitle'),
            FieldPanel('introduction'),
        ]),
        ...

BreadPage
BreadPage currently has no introduction field. It does have a description StreamField:
    description = StreamField([
        ('heading', blocks.CharBlock(classname="full title")),
        ('paragraph', blocks.RichTextBlock()),
    ])

How would you suggest that we reconcile the above with the BasePageFieldsMixin?

For the BlogPage, I don't think we need both subtitle and introduction. Intro is important for smart truncation of blog posts on index pages so keep that, but subtitle isn't actually something I've seen on blogs before. I'd ditch that. For the content panels of BlogPage, I'd show the inherited ones first, then the specific ones (so introduction after the two inherited ones).
On the BreadPage, I don't see the point of a heading CharBlock since we already have the bread's title. With that gone, we could just use body from elsewhere, rather than introduction (to maximize consistency).
Agree/disagree?

@shacker when you have a change, if you could take another look at this PR; I implemented your suggestions.

@shacker and @daaray the subtitle came from the design, and is the text that's appearing visually under the main title on the BlogPage. If it's going to make lives difficult then I'd say ditch it, but it does help with the design by giving an extra element to play with. (Not perhaps relevant but On UK news websites it's also a pattern that's used regularly to have title, standfirst then introduction)
The rest looks great!

@heymonkeyriot @shacker I restored subtitle as it is a design requirement.  It also allows us to illustrate the malleability of the content_panels (they are 'just' lists and can be interacted with as such):
    content_panels = BasePageFieldsMixin.content_panels + [
        StreamFieldPanel('body'),
        FieldPanel('date_published'),
        InlinePanel(
            'blog_person_relationship', label="Author(s)",
            panels=None, min_num=1),
        FieldPanel('tags'),
    ]
    # Inject subtitle panel after title field
    title_index = next((i for i, panel in enumerate(content_panels) if panel.field_name == 'title'), -1)  # noqa
    content_panels.insert(title_index + 1, FieldPanel('subtitle'))

This should be ready for final review.

That's a very interesting technique, using a list comprehension to manipulate the content_panels dynamically.


Add support for Sasmung 49in monitors

Good day Microsoft VSCode team. I recently purchased a 49in ultrawide Samsung Monitor for coding. The VSCode does not seem to support Samsungs 'Easy Setting Box' software which is used to snap applications on to portions of the screen. Your other Microsoft products such as Office 365 (Outlook, Word, Excel etc) support this, however the VSCode app does not.
Can you build in support for this?
Here is a link to the monitor I purchased:
Samsung: https://www.microcenter.com/product/511198/samsung-c43j890-434-4k-uxga-120hz-hdmi-dp-curved-led-monitor
Samsung Easy Setting Software: https://displaysolutions.samsung.com/solutions/monitor-solution/easy-setting-box
Much thanks!
TechGuyChris (Christopher)

Hi there,
Just got my own ultrawide, and not happy to see that this isn't supported.
I'm an IoT embedded systems developer working with three different platforms simultaneously - two hardware, one software. My current development schedule has me working more than 80 hours a week, developing code while managing a team of 3.
My previous solution of having three 27 in monitors, while functional, noticeably strains my neck. The ultrawide has relieved this issue, but your lack of support here makes setting up and rearranging my work environment a real pain.
Could you please rethink this? I'm not a streamer or hobbiest; I'm a professional that bought this solution specifically to solve a real problem.

Hi there,
Just got my own ultrawide, and not happy to see that this isn't supported.
I'm an IoT embedded systems developer working with three different platforms simultaneously - two hardware, one software. My current development schedule has me working more than 80 hours a week, developing code while managing a team of 3.
My previous solution of having three 27 inch monitors, while functional, noticeably strains my neck. The ultrawide has relieved this issue, but your lack of support here makes setting up and rearranging my work environment a real pain.
Could you please rethink this? I'm not a streamer or hobbiest; I'm a professional that bought this solution specifically to solve a real problem.

If you are using windows 11, you can use win + z to arrange the windows

If you are using windows 11, you can use win + z to arrange the windows

Hey there,
Thanks for the reply, but I'm using win10 and will not be moving to win11.


Is there a way to remove the unminified react-native javascript in the .ipa file?

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
Why is there both a minified and unminified version of my javascript in my .ipa file? And further, can I remove the unminified version of the javascript?
For security purposes, I don't want someone to be able to download my .ipa file from the appstore and see all of my javascript source code (I believe they can extract the .ipa with a jailbroken phone).
Question Description
I am successfully generating builds and uploading them to TestFlight. However, when I run a local build (Fastfile seen below) to try and inspect the files that I'm uploading to TestFlight, I get an .ipa file that I open and can see all of my unminified javascript in a /src/ folder (blue box) in addition to minified javascript (red box):

Is there a way to remove this /src/ folder from the .ipa output?
Fastfile command that generates the output .ipa file:
platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :local do
    match(type: "adhoc")
    build_app(workspace: "mobile.xcworkspace", scheme: "mobile", export_xcargs: "-allowProvisioningUpdates", export_method: "ad-hoc")
  end
end

fastlane env
✅ fastlane environment ✅
Stack

Key
Value

OS
10.15

Ruby
2.6.3

Bundler?
false

Git
git version 2.21.0 (Apple Git-122)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.15 (19A602)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib

OpenSSL Version
LibreSSL 2.8.3

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
11.0

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane

default_platform(:ios)

platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :beta do
    increment_build_number(xcodeproj: "mobile.xcodeproj")
    match(type: "appstore")
    build_app(workspace: "mobile.xcworkspace", scheme: "mobile", export_xcargs: "-allowProvisioningUpdates")
    upload_to_testflight
  end

  lane :local do
    match(type: "adhoc")
    build_app(workspace: "mobile.xcworkspace", scheme: "mobile", export_xcargs: "-allowProvisioningUpdates", export_method: "ad-hoc")
  end

  lane :release do
    increment_build_number(xcodeproj: "mobile.xcodeproj")
    match(type: "appstore")
    build_app(workspace: "mobile.xcworkspace", scheme: "mobile", export_xcargs: "-allowProvisioningUpdates")
    upload_to_app_store

  end
end

`./fastlane/Appfile`
app_identifier("REDACTED") # The bundle identifier of your app

itc_team_id("REDACTED") # App Store Connect Team ID
team_id("REDACTED") # Developer Portal Team ID

# For more information about the Appfile, see:
#     https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.134.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.3.0

slack-notifier
2.3.2

atomos
0.1.3

CFPropertyList
2.3.6

claide
1.0.3

colored2
3.1.2

nanaimo
0.2.6

xcodeproj
1.13.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.6.0

terminal-table
1.8.0

plist
3.5.0

public_suffix
2.0.5

addressable
2.7.0

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.7.0

tty-cursor
0.7.0

tty-spinner
0.9.1

babosa
1.0.3

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

excon
0.67.0

faraday
0.17.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.13.1

fastimage
2.1.7

gh_inspector
1.1.3

json
2.1.0

mini_magick
4.9.5

multi_xml
0.6.0

rubyzip
1.3.0

security
0.1.3

xcpretty-travis-formatter
1.0.0

dotenv
2.7.5

bundler
1.17.2

naturally
2.2.0

simctl
1.6.6

jwt
2.1.0

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.2

mime-types-data
3.2019.1009

mime-types
3.3

multi_json
1.14.1

signet
0.12.0

memoist
0.16.0

os
1.0.1

googleauth
0.6.7

httpclient
2.8.3

google-api-client
0.23.9

google-cloud-env
1.2.1

google-cloud-core
1.3.2

digest-crc
0.4.1

google-cloud-storage
1.16.0

emoji_regex
1.0.1

forwardable
1.2.0

logger
1.3.0

stringio
0.0.2

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.1.0

strscan
1.0.0

date
2.0.0

fileutils
1.1.0

etc
1.0.1

io-console
0.4.7

zlib
1.0.0

libxml-ruby
3.1.0

rexml
3.1.9

psych
3.1.0

mutex_m
0.1.0

generated on: 2019-10-29

That is not really a fastlane question - it just uses Xcode to build your app, so the result so should be identical to when you build with Xcode itself.

I see, thank you. I was curious if there were any build options that fastlane accepts that would achieve the minification I'm looking for. I couldn't find anything in the docs. I'm guessing the answer is no, but I don't know if xcode has any kind of option like that either.

If Xcode has one, fastlane also supports this.

It turned out that we had a configuration in our build that was explicitly moving the /src/ folder into the build via Xcode's Build Phases > Copy Bundle Resources. So I removed that configuration. Note for anyone in the future that when I did that, our images were no longer able to be resolved, so I had to add --assets-dest='./ios' to the bundle command that creates our  main.jsbundle, and then add that folder to Copy Bundle Resources.
This can be closed.


Get HammerJS on the Google CDN

Do we want hammer.js or hammer-time.js? See http://hammerjs.github.io/

Hammer JS :D

https://developers.google.com/speed/libraries/#hammerjs


support jep394's Pattern Matching for instanceof

Problem description
public class Main {

  public void abc(String a) {
    if (a instanceof Object d) { // jep394
    }
  }

}

Spec: JEP 394: Pattern Matching for instanceof
Expected parse tree output (optional)
Actual parse tree output (optional)
program [0, 0] - [7, 0]
  class_declaration [0, 0] - [6, 1]
    modifiers [0, 0] - [0, 6]
    name: identifier [0, 13] - [0, 17]
    body: class_body [0, 18] - [6, 1]
      method_declaration [2, 2] - [5, 3]
        modifiers [2, 2] - [2, 8]
        type: void_type [2, 9] - [2, 13]
        name: identifier [2, 14] - [2, 17]
        parameters: formal_parameters [2, 17] - [2, 19]
        body: block [2, 20] - [5, 3]
          if_statement [3, 4] - [4, 5]
            condition: parenthesized_expression [3, 7] - [3, 30]
              instanceof_expression [3, 8] - [3, 27]
                left: identifier [3, 8] - [3, 9]
                right: type_identifier [3, 21] - [3, 27]
              ERROR [3, 28] - [3, 29]
                identifier [3, 28] - [3, 29]
            consequence: block [3, 31] - [4, 5]

Seems fine on master


Updated for 2.002 multiplicative egg behavior

Just a heads up, usually CookieMonster.js is the "compiled" file from Combine.bat.  The changes you made should have been in src\Sim.js.  It's fine, I'll probably merge, then back port it to src\Sim.js.

Alright, I'll keep that in mind! Thanks :-)

Looking at your profile, you don't seem to be that new to coding..  I guess you just missed the source files?  Lol

Well, I'm a mathematician, but I've done some programming in Java and
Python (mostly to solve some Project Euler problems for fun). I don't
really know anything about Git or how to help others maintain and improve
their code. However I want to learn as much as possible and am eager to
help improve Cookie Monster in any way I can. It's an awesome add-on!
Alex
On Tue, Jun 28, 2016 at 9:09 PM, Aktanusa<EMAIL_ADDRESS>wrote:

Looking at your profile, you don't seem to be that new to coding.. I guess
you just missed the source files? Lol
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Aktanusa/CookieMonster/pull/95#issuecomment-229229308,
or mute the thread
https://github.com/notifications/unsubscribe/ANeh29lxTsPD8iYiGCRgbKbyPOMHUzUIks5qQcW1gaJpZM4JAlMm
.

Naive question: how exactly is
http://aktanusa.github.io/CookieMonster/CookieMonsterBeta.js generated? I
thought that perhaps merging my code would change that file, but it seems
to have no effect.
Alex
On Tue, Jun 28, 2016 at 9:18 PM, Alex Provost<EMAIL_ADDRESS>wrote:

Well, I'm a mathematician, but I've done some programming in Java and
Python (mostly to solve some Project Euler problems for fun). I don't
really know anything about Git or how to help others maintain and improve
their code. However I want to learn as much as possible and am eager to
help improve Cookie Monster in any way I can. It's an awesome add-on!
Alex
On Tue, Jun 28, 2016 at 9:09 PM, Aktanusa<EMAIL_ADDRESS>wrote:

Looking at your profile, you don't seem to be that new to coding.. I
guess you just missed the source files? Lol
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Aktanusa/CookieMonster/pull/95#issuecomment-229229308,
or mute the thread
https://github.com/notifications/unsubscribe/ANeh29lxTsPD8iYiGCRgbKbyPOMHUzUIks5qQcW1gaJpZM4JAlMm
.

Thanks for your kind words.  If you really want to help out, as a mathematician, tell me what is the formula to find the best thing to buy at any given time?  That is what I liked solved, if possible, lol.  Everything @ http://aktanusa.github.io/ is served from the the gh-pages branch.  This is why master branch doesn't have the beta JS and why only gh-pages branch has the beta JS.  I would like to (try) and finish all the preliminary changes before pushing a new beta JS to gh-pages.

That is definitely a challenging problem, and I will think about it!
Alex
On Tue, Jun 28, 2016 at 9:30 PM, Aktanusa<EMAIL_ADDRESS>wrote:

Thanks for your kind words. If you really want to help out, as a
mathematician, tell me what is the formula to find the best thing to buy at
any given time? That is what I liked solved, if possible, lol. Everything @
http://aktanusa.github.io/ is served from the the gh-pages branch. This
is why master branch doesn't have the beta JS and why only gh-pages branch
has the beta JS. I would like to (try) and finish all the preliminary
changes before pushing a new beta JS to gh-pages.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Aktanusa/CookieMonster/pull/95#issuecomment-229232201,
or mute the thread
https://github.com/notifications/unsubscribe/ANeh24RNzoO_oBT9DWOSI1JToZf5ZiHcks5qQcqcgaJpZM4JAlMm
.

Btw, not sure you've noticed, but your fix is live now.


Application crashes using manager.authorize('twitter')

Same here, no matter if you provide consumer_key and consumer_secret or not.

experiencing the same thing


ng build throws Cannot read property 'startTag' of null exception

I want to publish my Angular 8 app to IIS. Then I've started to follow the deployment instructions. When I build my project on prod mode with base-href, operation throws an error. If I build in development mode(just ng build), there is no error but I have to build in production mode. --base-href causes the error. What can I do now?
Here is error:

Cannot read property 'startTag' of null TypeError: Cannot read property 'startTag' of null at Object.augmentIndexHtml (D:\ReklamUssuProjects\DevBranch\RULink\RULink.UI\Angular\node_modules@angular-devkit\build-angular\src\angular-cli-files\utilities\index-file\augment-index-html.js:126:55) at compiler.hooks.emit.tapPromise (D:\ReklamUssuProjects\DevBranch\RULink\RULink.UI\Angular\node_modules@angular-devkit\build-angular\src\angular-cli-files\plugins\index-html-webpack-plugin.js:70:60)

I've opened the same issue on stackoverflow link below:
https://stackoverflow.com/questions/57754746/ng-build-throws-cannot-read-property-starttag-of-null-exception?noredirect=1#comment101946252_57754746

Can you please share the contents of your index.html?

It's an MVC app, so index.html file is empty. But I can share index.cshtml which is under DashboardController
`@{
ViewBag.Title = "Index";
}
@Styles.Render("~/Content/Angular")

@Scripts.Render("~/bundles/Angular")
`

An index file that contains a minimum of html, head and body tags is required.
This issue is also related to https://github.com/angular/angular-cli/issues/11816

Man... Thank you very much. Build is now successful and the problem is so easy...


Change the label "Manuscript, Book" to "Manuscript, Archival Item" for single-item PULFA records

Description
We designate single-item archival objects as leader 6/7 “t” and “m”: manuscript language materials and monograph/item. Based on those values, Orangelight applies the label “Manuscript/Book.”
For PULFA records with that leader, we’d want “Book” to be “Archival Item.”
Example
https://catalog.princeton.edu/catalog/9945578583506421
Implementation
PULFA records have a 035$a that starts with (PULFA) and a 040$e set to dacs.
We can filter by those so we don't apply the label to actual books:
For any records with

035$a starting with "(PULFA)" and
040$e being "dacs" or "appm" or being absent, and
leader 6 "t" and
leader 7 "m" :
label for leader 7 should be "Archival Item" (not "Book")

Acceptance criteria:

[ ] PULFA records matching those selection criteria display as "Manuscript, Archival Item", e.g. this record
[ ] non-PULFA records continue to display as "Manuscript, Book", e.g. this record
[ ] PULFA records not matching those selection criteria continue to display as they did before, e.g. this record would continue to display as "Manuscript"
[ ] The format facet should display the same icon as "Manuscript" for the new format "Archival Item"

Before merging to prod:
index a sample of data for review
Related Slack thread

I've updated the above criteria for the $040 to include appm as well as an absent 040.


Debug panel is unreadable

Take a look at image, when i try to inspect my collections it's going to be very hard (very long scroll), maybe you can stylize it better somehow?

This is just how the debug bar works: http://phpdebugbar.com/
Perhaps create an issue there?

@heihachi88 I agree but it's an external tool so yes, politely request an improvement at phpdebugbar :)


Feature: Plugins (for clips and chapters)

Which Mux Elements/Packages would you like a new feature for?

mux-video
mux-audio

Describe the feature you'd like to request
I would like being able to show clips (parts of a Mux VOD file) and display chapters in the timeline.
I could also just have a good way to add/use plugins, so that we can write one (and maybe pull request).
Describe the use cases the feature will solve

Show short recordings from long live stream (without have to clip the file it self).
Guiding a viewer through long videos with chapters

How are you using Mux Elements?
I'm not right now,
Describe the solution you'd like to see
For clips we're now using videojs-offset, which works fine.
For chapters I like de design of youtube and would like to be able to set chapters using JSON or JavaScript (not a vtt file)
For plugins I'd like a guide and a forkable project that includes testing (not to break anything)
Describe alternatives or workarounds you've considered
Using VideoJS, but the mux player looks much nicer!

I can media-chrome has a <media-clip-selector> to create a clip, but does Mux Player support that? (Can't find a way)


stored procedure created insead of plain sql statement

We do have a model that has string based field which are used in where Statement to filter data.
If we do this, EF7 emitts a stored procedure instead of a plain SQL. Example:
iChats.Where(Function(C) C.SenderName.Contains(filter))
SenderName is a string field in the Chat. According documentation the SenderName.Contains creates a LIKE, as expected.
But unfortunately, the full Statement is enclosed in as
exec sp_executesql .....
Having this, the retreivel of data from SQL is magnitudes slower than with a plain SQL Statement.
Why does EF7 create the stored procedure?
Additionally, in case of this:
iChats.Where(Function(C) C.SenderName.Contains(filter) or C.Sender.Contains(filter))
no LIKE is generated at all and the whole table is loaded and filtered in Memory and not in the database.

Could you show some SQL statements?

Yes,
the emitted SQL for a LIKE case as above is:
exec sp_executesql N'SELECT TOP(100) [Entry].[ID], [Entry].[AgentID], [Entry].[AgentName], [Entry].[ChannelType], [Entry].[ConversationID], [Entry].[End], [Entry].[LanguageID], [Entry].[PreviewText], [Entry].[Sender], [Entry].[SenderName], [Entry].[SkillID], [Entry].[Start], [Entry].[TicketID]
FROM [Chat] AS [Entry]
WHERE [Entry].[Sender] LIKE (''%'' + @$VB$Local_Sender_0) + ''%''',N'@$VB$Local_Sender_0 nvarchar(4000)',@__$VB$Local_Sender_0=N'test'
Running this is very slow, tried in SQL Management Studio.
I would expect a stetment like this
SELECT TOP(100) [Entry].[ID], [Entry].[AgentID], [Entry].[AgentName], [Entry].[ChannelType], [Entry].[ConversationID], [Entry].[End], [Entry].[LanguageID], [Entry].[PreviewText], [Entry].[Sender], [Entry].[SenderName], [Entry].[SkillID], [Entry].[Start], [Entry].[TicketID]
FROM [Chat] AS [Entry]
WHERE [Entry].[Sender] LIKE '%test%'
which executes much faster.
First one  took abot 3 seconds, second one some milliseconds.
Does this help?

Does the first one also take 3 seconds the second time you run it?

yes, every time :-(

What is the create table statement for the Entry.Sender column?

It is a Code first model. So, the line creating that column is:
Key .Sender = table.Column(Of String)(nullable:=True),
the Index for that column is created like this
migrationBuilder.CreateIndex("Chat.Sender", "Chat", "Sender")
Do you Need anything else?

This is actually done by SQL Client and is transparent to EF. I believe it is always done when parameters are present.

This is not true.
When I create a where that holds Parameters the resulting SQL I see in the SQL Server is a pretty normal SQL Statement that has a where clause as expected.
Only in case I do have a where that Looks up a varchar or nvarchar in the database that has something like
ColA.Contains("Bla") or ColB.Contains("Bla")
than EF emits a procedure.
In case I use only something like
ColA.Contains("Bla")
EF emits the expected SQL where clause and no procedure is created.
The Impact on Performance is considerably high, so it would be much better to have a Standard SQL where insead of a proceure.
So, this might be something the should be improved.

Whether or not we create a parameter depends on how the value is passed. In RC1, no parameter is created for this:
context.Blogs.Where(p => p.Id == id);

SELECT [p].[Id], [p].[Url]
FROM [Blog] AS [p]
WHERE [p].[Id] = 5

But one is created for this:
var id = 5;
context.Blogs.Where(p => p.Id == id);

exec sp_executesql N'SELECT [p].[Id], [p].[Url]
FROM [Blog] AS [p]
WHERE [p].[Id] = @__id_0',N'@__id_0 int',@__id_0=5

BTW you can see this behavior in SQL Client without EF in the picture by running the following code:
var conn = new SqlConnection(@"Server=(localdb)\mssqllocaldb;Database=Sample;Trusted_Connection=True;");
var cmd = conn.CreateCommand();
cmd.CommandText = "SELECT Count(*) FROM dbo.Blog WHERE Id = @p0";
cmd.Parameters.Add(new SqlParameter("p0", 5));

conn.Open();
var count = cmd.ExecuteScalar();
conn.Close();

The query sent to the database by SQL Client is:
exec sp_executesql N'SELECT Count(*) FROM dbo.Blog WHERE Id = @p0',N'@p0 int',@p0=5


[Feature Request]: Add field for document name in LabelledRagDataset

Feature Description
Add info about document to LabelledRagDataset - from which document was each question created from.
Having an attribute .metadata would also allow users to add their own extra info as dictionary keys.
There is implementation in Ragas
for document in documents:
    document.metadata['file_name'] = document.metadata['source']

https://docs.ragas.io/en/stable/getstarted/testset_generation.html#get-started-testset-generation
RagDatasetGenerator could possibly add this info when creating questions.
Reason
No response
Value of Feature
Knowing the source dokument for questions allows to get more detailed info, such as metrics and failed questions for individual document.  And finding the documents with lowest score.

Hi, also some evaluation utilities still use EmbeddingQAFinetuneDataset instead of LabelledRagDataset, but the former cannot be created from the latter, because LabelledRagDataset does not retain any information about nodes once created.


settings_org: Extract settings_components module

(27, 6) → (25, 5).
This resolves all cycles through settings_org except one: settings_realm_user_settings_defaults.set_up → settings_org.register_save_discard_widget_handlers → settings_org.discard_property_element_changes → settings_org.update_dependent_subsettings → settings_realm_user_settings_defaults.realm_default_settings_panel. That could be resolved in a few ways, such as by moving set_up or realm_default_settings_panel.
Discussion.

A few other functions that could readily move:

get_time_limit_dropdown_setting_value plus set_time_limit_setting
set_dropdown_list_widget_setting_value
set_property_dropdown_value
Perhaps get_sorted_options_list
get_realm_time_limits_in_minutes plus update_custom_value_input

I think it's possible that the remaining import cycle you mention could be addressed by moving update_dependent_subsettings and settings_notifications.set_enable_digest_emails_visibility to settings_components.js; all of those functions seem to be little wrappers around settings_components functions.
I don't love the separation-of-concerns boundary that creates, and ultimately the right long-term fix is going to be having some process for these other modules registering components with settings_components.js... but I think that would suffice as a checkpoint that removes this import cycle.

I don't have any objections to the actual changes in this PR, though, so feel free to merge and submit a follow-up and just do additional code moves in a follow-up PR if that feels nicer.

A few other functions that could readily move:

get_time_limit_dropdown_setting_value plus set_time_limit_setting
set_dropdown_list_widget_setting_value
set_property_dropdown_value
Perhaps get_sorted_options_list
get_realm_time_limits_in_minutes plus update_custom_value_input

Moved these all.

I think it's possible that the remaining import cycle you mention could be addressed by moving update_dependent_subsettings and settings_notifications.set_enable_digest_emails_visibility to settings_components.js; all of those functions seem to be little wrappers around settings_components functions.

That still leaves the same cycle, just now with settings_components instead of settings_org: settings_realm_user_settings_defaults.set_up → … → settings_components.update_dependent_subsettings → settings_realm_user_settings_defaults.realm_default_settings_panel.

OK, a better plan for that is that the panel object isn't very important to that code. Fundamentally, container is just #realm-user-default-settings or a different constant, and for_realm_settings is just true|false, so I think set_enable_digest_emails_visibility should just accept a string and a bool and we can hardcode the values in update_dependent_subsettings for the two calls. i.e.
set_enable_digest_emails_visibility("#user-notification-settings", false);
set_enable_digest_emails_visibility("#realm-user-default-settings", true);

Nice, that does the trick: #27132.

OK, marked this to merge once CI passes, thanks @andersk!