Helm chart : User specified namespace + Grafana with persistency enabled => rendered PVC lacking namespace declaration

Describe the bug
When using the Helm chart with a user specified namespace and the Grafana persistency enabled, the generated PersistentVolumeClaim does not have the namespace declaration, leading in Grafana to be stuck in the Pending state.
Expected behavior
Everything to start up and running well 😸
Steps to reproduce the bug
Render the helm chart and try to deploy it :
helm template install/kubernetes/helm/istio --name istio --namespace istio-system --set grafana.enabled=true --set grafana.persist=true
Version
Istio 1.0.5release from github
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.5", GitCommit:"753b2dbc622f5cc417845f0ff8a77f539a4213ea", GitTreeState:"clean", BuildDate:"2018-12-06T01:33:57Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.5-eks-6bad6d", GitCommit:"6bad6d9c768dc0864dab48a11653aa53b5a47043", GitTreeState:"clean", BuildDate:"2018-12-06T23:13:14Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Installation
Rendering the chart through helm template and applying it kubectl apply -f -
Environment
AWS EKS cluster
Cluster state
istio-dump.tar.gz

@douglas-reid to review so confirm this can be merged into 1.1

Merged. Marking as Fixed.

Thanks! Just came across this problem


Fail to create quad mesh for many holes

Dear Prof. DanielZint,
Thank you for providing so powerful quad mesher RatRace. I tested it with some simple shapes and it works well. However, when I tested it with a shape with many holes, it failed to create correct quad mesh. Could you please help me to take a look at it?
Thanks,
Tang Laoya
transition4to1.off.gz

Dear Tang Laoya,
The mesh you sent is indeed a tough one. The good news is, I could generate a quad mesh on it. The bad news is, there are low quality quads, which might be a problem for you.
One thing that you should keep in mind when you use the code is that it uses a cache (see the 'cache' folder in the project that is generated when running the code for the first time). If you use the same name for a different mesh it will use the cached information for the mesh generation. Therefore, you should have unique names for your meshes.
My quality output for your mesh looks like this:
qMin = 0
qMean = 0.756099
Quality 0 - 0.1        |       11
Quality 0.1 - 0.2      |       4
Quality 0.2 - 0.3      |       5
Quality 0.3 - 0.4      |       74
Quality 0.4 - 0.5      |       281
Quality 0.5 - 0.6      |       7992
Quality 0.6 - 0.7      |       73000
Quality 0.7 - 0.8      |       114009
Quality 0.8 - 0.9      |       33665
Quality 0.9 - 1        |       34065
Valences: 32124 < 4 | 102223 == 4 | 31477 > 4
singular vertices [%]: 38.3545
Again, there is at least one degenerated quad with quality of 0. I just checked and the problem is that Blossom-Quad already generates qMin=0. That means the degenerated quad appears somewhere on the boundary. Unfortunately, I cannot give you any post processing step to resolve that issue. I am pretty sure it is possible but I would have to take a much closer look to figure something out.
If your goal is just to produce a valid quad mesh, that can be achieved by generating a hybrid mesh and then subdividing it. Here is a modified main.cpp where a subdivision is used instead of triangle merging or blossom quad:
main.zip
Nevertheless, you should be aware that this method produces much more quads than the others.
I hope this helps you somehow. If you have further questions, don't hesitate to ask me.
Best regards,
Daniel Zint
P.S.: I am not a professor so no need  for that formality ;-)

Dear Daniel,
Thank you very much for your kindly reply. Yes, the simplest method is to subdivide triangles, but as you said, it will generate much more quads. I am also seeking for less quads with good quality.
In fact, I am trying to implement the algorithm by this paper:
https://cims.nyu.edu/~ag6961/GIULIANI_quad.pdf
But I am failed in the 'edge coloring' algorithm. I can't let one color removed from the boundary edges if number boundary edges is even. Do you have any good suggestion?
In addition, the idea in your paper is excellent. You use blossom to generate quad at boundary, and then user your fast method to generate meshes inside. My question is: how do you control the blossom to let it just generate quad for the triangles near boundaries?
Thanks,
Tang Laoya

Dear Daniel,
Thank you very much for your kindly reply. I have got a copy of your paper. I need to further study and try to understand everything you said in your paper. I will also study the 'cross-field methods' you mentioned.
BTW: could you please also provide a CPU version of your RatRace? I tried to modify your code to let it work on CPU configure since there is no GPU card on the computer, maybe there is some error in my modified code.
Thanks,
Tang Laoya

Hey,
the part that is CUDA Code is just the smoothing step in the post-processing. The method is called Discrete Mesh Optimization (DMO). Luckily, I wrote a version of it that also runs on CPU so yes, I can provide a RatRace Version that is CPU only. But I don't know when I will find the time to do so. I will try next week but there is a lot going on. I will let you know when I managed to update the code.

Dear Daniel,
Thank you very much for your kindly reply. This weekend I tried to study the 'cross-field methods' you mentioned, but I still don't know what it can help me in my structure.
Thanks,
Tang Laoya

Dear Daniel,
Thank you very much for your kindly reply. This weekend I tried to study the 'cross-field methods' you mentioned, but I still don't know what it can help me in my structure.
Thanks,
Tang Laoya

My knowledge on cross-field methods is also rather restricted. People often show beautiful results in their cross-field based methods but I don't know if there is any open source project that you could use to generate some quad mesh. Implementing them yourself is quite tough.
I won't be able to update the RatRace code this week. Next week I should have more time.

Dear Daniel,
Thank you very much for your kindly reply. Yes just as you said, it is too difficult for me to study the cross-field method:(
I only need to generate quad mesh by converting the triangles into quads. I also require that boundary nodes don't change (if number of boundary nodes is odd, I will split the boundary triangle with maximal boundary edge length). Therefore, your hybrid method is best for me.
Thanks,
Tang Laoya

Hi,
I finally found some time to add the templated version of DMO to the code. It is added as a submodule so be aware that you need to clone the Git repository now with --recurse.
I could not check if the code runs on a machine without Cuda installed but as DMO does, I don't see a reason why this project should not. However, it might be that I made some mistake. In that case, please let me know.
Best regards,
Daniel

Dear Daniel,
Thank you very much for your kindly update. Yes, the latest code works. For some simple models it can generate very beautiful quad meshes. However, for the one that I uploaded the first time, it still can't generate final quad mesh. The following message displayed:
done [0.000 secs]. 0 grows, 0 expands, 0 shrinks
expands: [0.000 secs], shrinks: [0.000 secs], dual updates: [0.000 secs]
perfect matching with 0 nodes and 0 edges
starting init...done [0.000 secs]. 0 trees
.
done [0.000 secs]. 0 grows, 0 expands, 0 shrinks
expands: [0.000 secs], shrinks: [0.000 secs], dual updates: [0.000 secs]
I20220413 07:36:06.449043 138236 RatRace.cpp:322] Greedy triangle merge on interior
I20220413 07:36:07.304776 138236 main.cpp:144] Start RatRace
I20220413 07:36:07.369886 138236 RatRace.cpp:332] Number of triangles = 64
I20220413 07:36:07.818861 138236 main.cpp:151] Finish RatRace
I20220413 07:36:08.635948 138236 SizeGrid.h:46] Read SizeGrid from cache file 'E:\company\mesh\quadmesh\RatRace\RatRace-main\cache\transition4to1_SizeGrid_1000_1000_float.bin'
I20220413 07:36:08.657505 138236 RatRace.cpp:350] Removed interior val2: 394
I20220413 07:36:08.691980 138236 RatRace.cpp:353] Splitted boundary val2: 29
I20220413 07:36:08.714291 138236 RatRace.cpp:354] Removed boundary val2: 8
I20220413 07:36:08.738458 138236 RatRace.cpp:355] Removed boundary val2: 0
Could you please help me to take a look at it?
Thanks,
Tang Laoya

The code break down in function optimizeHierarchical:
The error message is below:
Run-Time Check Failure #2 - Stack around the variable 'oneRing' was corrupted.

Dear Daniel,
The crash problem is solved after simply changed MAX_ONE_RING_SIZE from 16 to 200. However, the output mesh is not correct. For example, the section I put here contain a large triangle with it's one edge overlapped with several boundaries. Could you please help me to take a look at it?

Thanks,
Tang Laoya

The code break down in function optimizeHierarchical:
The error message is below: Run-Time Check Failure #2 - Stack around the variable 'oneRing' was corrupted.

I added a warning to the DMO code that tells the user that MAX_ONE_RING_SIZE might be too small. However, you don't need to increase it to 200. I think when you go to 32 you should be larger than any reasonable configuration. Actually, 16 is already a really big value. If your mesh has vertices with more than 16 neighbors, then the output is usually of low quality.
I also tried to check how the mesh quality can be improved but I cannot give you any fast answer to that. One thing I noticed is that your input mesh is not really good for making it a quad mesh. You could try to improve the quality of the triangle mesh and that might then also give a better quad mesh. Otherwise, I am afraid that I am not able to help you. Sorry!


Feature request: new cli option "--folder" for standalone version

Hi,
First of all, thanks for the great tool 🚀, I like it a lot.
I want to propose a new command line option --folder to the standalone variant of the ABAP cleaner.
Background would be to easily run the cleaner over a large amount of ABAP files (e.g. a working copy of an abapGit repository).
At the moment I was able to do the same thing with a powershell script, which is fine, but slow if you have a lot of files to format (~40s for 50 files). I am guessing that this is because of the eclipse runtime that has to be loaded for each formatting call.
By introducing the --folder option the eclipse runtime would only be loaded once and could speed up the whole formatting immensly.
Let me know what you think about that.
Regards,
Ludwig

Hi Ludwig,
as mentioned on your pull request, that's a great idea, and indeed there will be a huge performance gain with this, because ABAP cleaner (and esp. the RND Parser in it) then won't be initialized for each single file.
And thanks so much for right away contributing the implementation – actually the first code contribution to ABAP cleaner since its Open-Source release! Assigning you to this issue to document this is already being worked on.
Kind regards,
Jörg-Michael

Hi Ludwig,
meanwhile, I tested the new --sourcedir option, works like a charm!
I must admit, though, that the getHelp() documentation as I suggested it, was rather confusing now, so I split it into two scenarios:
Usage for single file:
    .\abap-cleanerc.exe {--sourcefile sourcefile / --source sourcecode } [--linerange linerange]
                        [{ --profile profile / --profiledata profiledata }] [--release release]
                        [--targetfile targetfile [--overwrite]] [--partialresult]
                        [--stats] [--usedrules]

Example for single file:
    .\abap-cleanerc.exe --sourcefile "CL_ANY_CLASS.txt" --linerange "20-35" --profile "team profile.cfj" --release "757" --targetfile "result\CL_ANY_CLASS.txt" --overwrite --stats --usedrules

Usage for multiple files:
    .\abap-cleanerc.exe --sourcedir sourcedir [--filepattern filepattern] [--recursive]
                        [{ --profile profile / --profiledata profiledata }] [--release release]
                        [--targetdir targetdir [--overwrite]]
                        [--stats] [--usedrules]

Example for multiple files:
    .\abap-cleanerc.exe --sourcedir "C:\temp\source" --filepattern "*.txt" --recursive --profile "team profile.cfj" --release "757" --targetdir "C:\temp\target" --overwrite

(then followed by the section with all the options of both cases).
Kind regards,
Jörg-Michael

Hi Jörg-Michael,
great. Certainly makes the whole thing clearer for newbies 😄.
Meanwhile I am working on a first version of a GitHub Workflow to automatically run ABAP Cleaner
Kind regards,
Ludwig

Hi Ludwig,
thank you so much for this idea and contribution! I just released version 1.6.0, so the new command line options are now available to everyone!

Meanwhile I am working on a first version of a GitHub Workflow to automatically run ABAP Cleaner

Wow! Please keep us updated, so we could e.g. add a section in the documentation on how such a workflow can be set up.
Kind regards,
Jörg-Michael

I agree, the GHA workflow is an cool idea, I look forward to seeing it!


register_shutdown_function() causes major memory leak during testing

Expected behavior
Symfony tests should run under phpunit without major memory leaks.
Alternatively, provide a bootstrapping process which can be modified for testing. Currently, the hard-coding of the Bugsnag\Client class makes that very difficult.
Observed behavior
Each request/sub-request during testing adds 2MB-4MB of permanent memory consumption.
Steps to reproduce
class TestCase extends \Symfony\Bundle\FrameworkBundle\Test\WebTestCase
{
    public function testMemory()
    {
        echo memory_get_usage() . PHP_EOL;
        $client->request('GET', '/');
        echo memory_get_usage() . PHP_EOL;
        $client->request('GET', '/');
        $client->request('GET', '/');
        $client->request('GET', '/');
        $client->request('GET', '/');
        $client->request('GET', '/');
        echo memory_get_usage() . PHP_EOL;
    }
}

Version
1.4.0/1.5.0
Additional information
I have confirmed that removing the register_shutdown_function([$this, 'flush']); from the Client constructor resolves the issue. Unfortunately, that encloses the state of $this until the process shuts down, which on large test suites is usually when it crashes from memory problems.

This is a functional work-around stub for testing, but very sad.
config_test.yml or test/bugsnag.yaml
bugsnag:
    client: App\Bugsnag\TestClient
    factory: App\Bugsnag\TestFactory

TestClient
use Bugsnag\Breadcrumbs\Recorder;
use Bugsnag\Client;
use Bugsnag\Configuration;
use Bugsnag\HttpClient;
use Bugsnag\Pipeline;
use Bugsnag\Request\BasicResolver;
use Bugsnag\Request\ResolverInterface;
use Bugsnag\SessionTracker;
use GuzzleHttp\ClientInterface;

class TestClient extends Client
{
    public function __construct(
        Configuration $config,
        ResolverInterface $resolver = null,
        ClientInterface $guzzle = null
    ) {
        $this->config = $config;
        $this->resolver = $resolver ?: new BasicResolver();
        $this->recorder = new Recorder();
        $this->pipeline = new Pipeline();
        $this->http = new HttpClient($config, $guzzle ?: static::makeGuzzle());
        $this->sessionTracker = new SessionTracker($config);
    }
}

TestFactory
use Bugsnag\BugsnagBundle\DependencyInjection\ClientFactory;
use Bugsnag\BugsnagBundle\Request\SymfonyResolver;
use Bugsnag\Client;
use Bugsnag\Configuration;

class TestFactory extends ClientFactory
{
    /**
     * @return Client
     */
    public function make()
    {
        return new TestClient(new Configuration(''), new SymfonyResolver(), new \GuzzleHttp\Client());
    }
}

Fixed in v1.6.0


Assign timeouts to various protocols of Ouroboros-Network

This is a follow up on #1395, and this one is just to assign concrete, production ready timeouts rather than finish the architecture for it.

Fixed in #1813

The current timeouts are quite coarse. Fine tuning is left for another ticket.


fix: top level spans have db and collection tag for streaming requests

The top level tracing spans for streaming requests didn't have the db and collection tags (they are set in SendMsg and ReceiveMsg).

Codecov Report
Base: 29.71% // Head: 29.69% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (b3e5705) compared to base (3dc41fd).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #536      +/-   ##
==========================================
- Coverage   29.71%   29.69%   -0.02%
==========================================
  Files          89       89
  Lines       10445    10448       +3
==========================================
- Hits         3104     3103       -1
- Misses       7051     7054       +3
- Partials      290      291       +1

Impacted Files
Coverage Δ

server/metrics/tracing.go
59.72% <0.00%> (-1.98%)
:arrow_down:

server/middleware/tracing.go
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


packaging request: chromium-bsu (arcade game)

Project description

Chromium B.S.U. is a fast paced, arcade-style, top-scrolling space shooter.

This package is available in the Gentoo package repository: https://packages.gentoo.org/packages/games-action/chromium-bsu; ebuild (the ebuild isn't for the latest released version, tho)
Metadata

pname = "chromium-bsu"
homepage URL: http://chromium-bsu.sourceforge.net/
source URL: mirror://sourceforge/${pname}/${pname}-${version}.tar.gz
gentoo url, nixos has an equivalent to the sourceforge mirrors, but I don't know how to translate it.
see also: http://sf.net/projects/chromium-bsu/files
license: (according to gentoo) Clarified-Artistic, (according to project homepage) The "Artistic License"
platforms: unix. requires OpenGL

Please do the honors, @zseri


[wip] Improve tutorials and fix minor bugs

Checklist:

[ ] Add tests to cover the fixed bug(s) or the new introduced feature(s) (if appropriate).
[x] Update the API documentation is a new function is added or an existing one is deleted.
[x] Added a news entry.

copy news/TEMPLATE.rst to news/my-feature-or-branch.rst) and edit it.

xref #5
blocked by https://github.com/maclandrol/graphormer-pretrained/pull/1
blocked by https://github.com/conda-forge/staged-recipes/pull/22357
ping @hadim

Here is the proposed plan in case GF and fairseq do not get merged before the release:

graphormer-pretrained: use the conda artifact from staged-recipes and upload it to the maclandrol anaconda channel as public (makes more sense since the repo is at maclandrol name).
fairseq: I don't think we should package it ourselves. Instead I propose to keep the pin on numpy and we'll just remove it once the conda package has been merged.

Let me know what you think.

Getting this in the docs for one of the notebooks

I had that before, but I don't remember how I solved it.

Getting this in the docs for one of the notebooks

I had that before, but I don't remember how I solved it.
@cwognum @hadim

I don't recall how to fix that. Maybe it's a CSS fix?

Getting this in the docs for one of the notebooks

I had that before, but I don't remember how I solved it.
@cwognum @hadim

@maclandrol It's because of having two code blocks after one another without anything in between. To fix, explicitly add a title (e.g. # My fancy title) or add some filler text in between the two code blocks. Not sure why this happens!

Could you also add the DOI in the documentation homepage (index.md)?

@maclandrol It's because of having two code blocks after one another without anything in between. To fix, explicitly add a title (e.g. # My fancy title) or add some filler text in between the two code blocks. Not sure why this happens!

I added the title in my early attempts to fix that, did not change anything.  There is no code block before. I also clear output and rerun the notebook, but no changes

I just made the changes locally and everything worked.

There is no code block before
It's because of the python codeblock installing torchinfo followed by the print of the model info.

Can you commit your changes here then ?

When tests are green, can be merged


Docs page buttons

I noticed "Contents", "Get in touch", and "Dark Mode" are buttons (or appear to be buttons) on the docs page. Did you intend on them having a specific functionality?

They were links at first but they weren't going anywhere so I think I made them buttons to add some CSS rules. Should I style them differently and make links instead?

I think since we will be making the Docs/Resources page just one page, we will just want them to be subheadings (not buttons or links), with a similar styling to what UBC Launchpad uses on their docs section.


Suggestion: Group based hijack ability

As is superusers always have the hijack ability and you can enable staff via the settings. However that enables it for all staff. Having the ability to add a user to a group that gives them the ability to hijack would allow a much higher degree of control of who can do it.
user.groups.filter(name__iexact='Hijacker Allowed').exists()
When I have time I'll dig deeper into the code and see if I can figure it out myself and possibly create a PR if no one else thinks this is worth their time. Thanks.

Please check the Custom authorization function if the standard behavior doesn't fit your needs.
That gives you full control and full flexibility to implement most use cases.


InvalidArgumentError, Found Inf or NaN gradient(global norm).

NO,I am seeing the same error. I also used the same function(tf.clip_by_global_norm),but I found learning rate and function are not the key reasons. when i generate Vocab,i set the size is 4682,and the vocab_size is 4682 in train.py,too. as the same,i do not know whether decrease the batch size is useful.
I look at an answer that it might be related to the vanishing/exploding gradient? I do not have any methods.
In train.py

the error is

and the error is found here

i found a questionable point is GPU:0. I think it may be related with my GPU,so i try to add the code like this

i do not know whether it is useful,i want to try.

Have you solved the mistake


Improve terms.

Improve terms and correct localized strings.

Thanks for the PR and for reviewing the translations.


Button doesn't seem to get updates when there's a textfield in alert content

Hi, thank you for the great framework. I run into some issues where the alert buttons didn't get updated. Here's the code:
      .customAlert("Enter access code",
                     isPresented: $isShowingAccessCodeAlert,
                     content: {
            TextField("Access Code", text: $accessCode)
                .onSubmit {
                    submitCode()
                    isShowingAccessCodeAlert = false
                }
        }, actions: {
            MultiButton {
                Button(role: .cancel) {
                } label: {
                    Text("Cancel")
                }

                Button("Submit", action: submitCode)
                    .disabled(accessCode.trimmingCharacters(in: .whitespacesAndNewlines).count == 0)
                    .bold()
            }
        })

it doesn't seem like the button disable state is getting updated when the user starting to enter text. What did I do wrong? Thanks!

Hi, well the current version doesn't support this due to a limitation of WindowKit which is used to display the alert.
I found a way around this limitation in WindowKit 2.1.0+ but for reasons unknown to me at the moment, the button don't update, even though they get redrawn.
I will continue to investigate this issue, hopefully I can find a solution, because this behavior seems very useful.

I fixed the issue by introducing a env variable and force the button to update. It's not great, but it does the trick case by case.

Hi, this should be fixed in 3.5.0

Thank you


The module could not use in new version of Openresty.

in openresty v.1.9.* can build and use stream and ngx_lua code.(use the v.0.0.1 stream-lua-nginx-module which u give the link to get this.)
in openresty v.1.11.* can not use stream ,when I test the nginx conf,it echo "段错误".(use v.0.0.1 or master).

@caichudong Yes, the latest ngx_stream_lua_module requires at least the NGINX 1.13.4 core to work properly while the current OpenResty release is <IP_ADDRESS> which is based on NGINX 1.11.2.
The plan is to release a new OpenResty version with a new 1.13.x NGINX core (and also the ngx_stream_lua_module) this month or so.

@agentzh Thank u, now I had used the old version to solve this problem.I'm looking forward for the new version.

@caichudong
you can use openresty-<IP_ADDRESS>.tar.gz, and use the newest ngx_stream_lua_module in the master branch , then use nginx-1.13.6.tar.gz to instead of the one in the openresty bundle, I did this, and it works for me.
we can use this way to try the new feature of ngx_stream_lua_module. And use the stard version once its release.

@idevz ok, thanks for ur advice. I'll try it.

@idevz Better avoid replacing nginx core in the openresty bundle since it may break compatibility. Better use the following OpenResty preview version instead:
http://openresty.org/download/openresty-<IP_ADDRESS>rc0.tar.gz

@agentzh thanks a lot, I just want to try the newest ngx_stream_lua_module for test, so didn't think much more. thanks for your opinion

@caichudong you can use this preview version instead too.
http://openresty.org/download/openresty-<IP_ADDRESS>rc0.tar.gz

when I install openresty-<IP_ADDRESS>rc0.tar.gz , I got this:
cd /media/psf/g/idevz/code/src/openresty-<IP_ADDRESS>rc0/../pcre-8.39 \
&& gmake libpcre.la
gmake[3]: 进入目录“/media/psf/g/idevz/code/src/pcre-8.39”
 cd . && /bin/sh /media/psf/g/idevz/code/src/pcre-8.39/missing automake-1.13.4 --gnu
/media/psf/g/idevz/code/src/pcre-8.39/missing:行81: automake-1.13.4: 未找到命令
WARNING: 'automake-1.13.4' is missing on your system.
         You should only need it if you modified 'Makefile.am' or
         'configure.ac' or m4 files included by 'configure.ac'.
         The 'automake' program is part of the GNU Automake package:
         <http://www.gnu.org/software/automake>
         It also requires GNU Autoconf, GNU m4 and Perl in order to run:
         <http://www.gnu.org/software/autoconf>
         <http://www.gnu.org/software/m4/>
         <http://www.perl.org/>
gmake[3]: *** [Makefile.in] 错误 1
gmake[3]: 离开目录“/media/psf/g/idevz/code/src/pcre-8.39”
gmake[2]: *** [/media/psf/g/idevz/code/src/openresty-<IP_ADDRESS>rc0/../pcre-8.39/.libs/libpcre.a] 错误 2
gmake[2]: 离开目录“/media/psf/g/idevz/code/src/openresty-<IP_ADDRESS>rc0/build/nginx-1.13.5”
gmake[1]: *** [build] 错误 2
gmake[1]: 离开目录“/media/psf/g/idevz/code/src/openresty-<IP_ADDRESS>rc0/build/nginx-1.13.5”
gmake: *** [all] 错误 2

and then, I update the Makefile generate under the pcre-8.39 dir, remove the version info from
ACLOCAL = ${SHELL} /media/psf/g/idevz/code/src/pcre-8.39/missing aclocal-1.13.4
AMTAR = $${TAR-tar}
AM_DEFAULT_VERBOSITY = 0
AR = ar
AS = as
AUTOCONF = ${SHELL} /media/psf/g/idevz/code/src/pcre-8.39/missing autoconf
AUTOHEADER = ${SHELL} /media/psf/g/idevz/code/src/pcre-8.39/missing autoheader
AUTOMAKE = ${SHELL} /media/psf/g/idevz/code/src/pcre-8.39/missing automake-1.13.4

to
ACLOCAL = ${SHELL} /media/psf/g/idevz/code/src/pcre-8.39/missing aclocal
AMTAR = $${TAR-tar}
AM_DEFAULT_VERBOSITY = 0
AR = ar
AS = as
AUTOCONF = ${SHELL} /media/psf/g/idevz/code/src/pcre-8.39/missing autoconf
AUTOHEADER = ${SHELL} /media/psf/g/idevz/code/src/pcre-8.39/missing autoheader
AUTOMAKE = ${SHELL} /media/psf/g/idevz/code/src/pcre-8.39/missing automake

and It's work for me.
so, @agentzh this is because of the pcre version or any other reason? I found that the pcre use in the new OpenResty Installation is using pcre-8.40, my net access was forbidden, and I can't get pcre-8.40 from ftp.csx.cam.ac.uk for try this :(.

@idevz It's strange that it triggers automake in the pcre repo. This should never happen. I wonder if your system's clock is screwed up?

@agentzh may be you're right, I did this in my centos VM, which clock is always more later then the host mac. Thanks a lot.

OK, a new openresty preview version is provided for testing:
https://openresty.org/download/openresty-<IP_ADDRESS>rc0.tar.gz
It includes the latest nginx 1.13.6 core.

10 x thanks

BTW, this openresty <IP_ADDRESS> rc0 preview version already includes and enables ngx_stream_lua_module by default.

Using openresty <IP_ADDRESS> rc0 to handle motan(weibo RPC framework) request in weibo, we can do many common things such as validation, filter and so on at preread phase through preread_by_lua_file, its so awsome and cool. Thanks @agentzh, @dndx

@idevz Good to know!

when is planning to release stable openresty with nginx 1.13.* ?

@henriquechehad OpenResty <IP_ADDRESS> RC1 will be out in the next couple of days.

@agentzh has recently released OpenResty <IP_ADDRESS> RC1:
https://groups.google.com/forum/#!topic/openresty-en/pVPxLB5jfl4

Consider it resolved.


un-hardcode english localized control strings. Make timeline/scrubber accessible and keyboard navigable

fixes #127

@hozefaj would love some feedback on this PR. Happy to make changes!


Deep importing .module.scss file compiles in dev but not build

Describe the bug
Description
When importing .module.scss files directly from a dependency package (in my reproduction case @acme/ui is the "external" package, but you could imagine any dependency instead) in a Vite project, the imports work correctly in development mode but none of the styles are applied/compiled during production build. This does not work both when importing a module from another module scss file or directly from the application code.
Reproduction
Project structure:
apps/
  vite-project/
    src/
      App.tsx
packages/
  ui/
    src/
      button.module.scss
      button.tsx

Code Examples

apps/vite-project/src/App.tsx

import<EMAIL_ADDRESS>// This import works in dev but fails in build

packages/ui/package.json

{
  "name": "@acme/ui",
  "exports": {
    ".": {
      "types": "./dist/src/index.d.ts",
      "default": "./src/index.ts"
    },
    "./*": {
      "types": "./dist/src/*.d.ts",
      "default": [
        "./src/*.ts",
        "./src/*.tsx"
      ]
    }
  }
}

Expected Behavior

SCSS Modules imports from workspace packages should work consistently in both development and production builds even when deeply imported from dependency
Styles should be properly processed and included in the final bundle

Actual Behavior

Development mode (pnpm dev): CSS Modules imports work correctly

Production build (pnpm build): Build does not apply styles with module resolution errors for deep imports of .module.scss files from node modules

Environment

Vite: 6.0.1 (IMPORTANT: Also not working in 5.4.10)
Node.js: 20.18
OS: MacOS Latest
Package Manager: pnpm (also appears using npm, yarn)

Additional Context
This issue appears to be related to how Vite handles CSS Modules imports from node_modules during production builds. While the development server can resolve these imports through the workspace, the production build seems to have different module resolution rules.
Would appreciate guidance on the recommended approach to handle SCSS Modules imports from both workspace packages and dependencies in Vite projects.
Reproduction
https://github.com/moishinetzer/vite-scss-modules-reproduction
Steps to reproduce
Compare pnpm dev with pnpm build
System Info
System:
    OS: macOS 14.7.2
    CPU: (8) arm64 Apple M2
    Memory: 60.22 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.18.1 - ~/.nvm/versions/node/v20.18.1/bin/node
    Yarn: 1.22.22 - ~/.nvm/versions/node/v20.12.2/bin/yarn
    npm: 10.8.2 - ~/.nvm/versions/node/v20.18.1/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 131.0.6778.140
    Safari: 18.2

Used Package Manager
pnpm
Logs
VITE v6.0.3  ready in 115 ms
➜  Local:   http://localhost:5173/
➜  Network: use --host to expose
➜  press h + enter to show help
Deprecation Warning: The legacy JS API is deprecated and will be removed in Dart Sass 2.0.0.
More info: https://sass-lang.com/d/legacy-js-api
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

More smaller reproduction: https://stackblitz.com/edit/vitejs-vite-kakz5upv?terminal=dev
This is caused by #16051.
The workaround is to add this plugin: https://github.com/vitejs/vite/pull/16051#issuecomment-2076616816.


Fix for AttachmentCollection validate

In AttachmentCollection.validate(), type checking added to casting of Attachment to FileAttachment as fix for #18.

Hi @barisaydinoglu, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

@barisaydinoglu it would be great if you also would add some unittests for the behavior

I've fixed if conditions.

@barisaydinoglu are you planning any changes on this PR?


Adding BasedOn Style on App.xaml is crashing on App() { InitializeComponent(); }

Adapting an project to Template10, I go into inherited styles in App.xaml are crashing.
My App.xaml.cs

My App.xaml

Great question!, but it's the kind of question better served on http://StackOverflow.com. Use tag "Template10".


clickhouse does not get built by Hydra

clickhouse fails to build on Hydra, but succeeds to build on my big machine on current commit 45fc7d4a35c5343e58541a7847f6415654ccbb37: https://hydra.nixos.org/build/147474186

2021-04-26 Last successful build
2021-05-08 First broken build
2021-07-09 This build

Because of this, the plausible NixOS cannot be easily used, as clickhouse is a large build.
It says there Status: Output limit exceeded, but I don't know yet what to do about that yet.
CC @Ma27 @happysalada

This means the log is too long, iirc.

The passing build is 3.6 MB uncompressed, 50 KB gzip-compressed as delivered by Hydra.
The failing build https://hydra.nixos.org/build/147474186 doesn't have any log file, not even a cut-off one.

I've restarted it, to check if the error can be reproduced.

Well, it is repeatable. Maybe someone familiar with hydra can help. @grahamc?

According to Hydra, the issue is not the build itself, but fetching the source:

I tried building it, and I'm noticing it is fetching Many Git submodules, including grpc, llvm, icu, brotli, and ... well, dozens: https://github.com/ClickHouse/ClickHouse/tree/v<IP_ADDRESS>-lts/contrib
This source directory ends up being<PHONE_NUMBER> bytes (2.12024 gigabytes) which is above the 2 gigabyte maximum. Of this source result, almost all of it<PHONE_NUMBER> bytes) is the contrib directory.
I wonder if there is a way to reduce this somehow?
Also, side note the broken attribute should be moved to meta:
https://github.com/NixOS/nixpkgs/blob/5e9429689a988bac273a98b3f85cb71d1aa15eb0/pkgs/servers/clickhouse/default.nix#L12

I wonder if there is a way to reduce this somehow?

A big submodule of clickhouse is llvm, which according to their docs:

Allows to bundle LLVM inside ClickHouse repository. This is intended for reproducible builds, that continue to work regardless to changes of compile options and ABI changes.

The usage of this repository is not necessarily.

Not compiling llvm as part of clickhouse would also save a lot of build time.

How can we exclude an individual submodule from cloning?

Looks like this will be the first use case for that. Might need to add a sort of "postClone" hook to the fetchgit script, where we can do all sorts of dirty stuff before fetching submodules, LFS or whatnot. Not sure if that's compatible with pre-fetch scripts but that's not a loss; just lacking a feature in a corner case.

I self-host a Hydra anyways, so I can take a look today :)

Note that it can use more than 2G of space during the build, but it must be below 2G at the end of the build.

I'm very sorry, but I'm going to stop working on this. My main motivation was to get plausible building on Hydra again, but my knowledge on clickhouse internals is far too limited.
The thing is, with a system-llvm you have to set some additional flags (https://github.com/ClickHouse/ClickHouse/blob/v<IP_ADDRESS>-lts/cmake/find/llvm.cmake#L47-L53) which in turn cause some linking errors: https://gist.github.com/Ma27/d8ee175e8581286b75ca59198fa772af.
WIth the latest stable (v<IP_ADDRESS>-stable) we can't even use a third-party llvm anymore: https://github.com/ClickHouse/ClickHouse/blob/v<IP_ADDRESS>-stable/cmake/find/llvm.cmake#L12-L14
Of course, we could disable the features that require llvm, btu I don't think that this is a viable option.
My hackery can be found at https://git.mbosch.me/ma27/clickhouse-experiments in case someone's interested.

Looks OK now on master/unstable and 21.11.


Ashuffle stops when a file is not found

I am experimenting with a web radio so I have a continuous broadcast of a large library playing at random with ashuffle.
Sometimes a file is not found (did not find the reason yet, maybe just need an mpc update to run).
In such cases, the playlist is started over and replayed. If I delete the unfound file and play the last track on the playlist, then ashuffle takes over again.
I would expect/like ashuffle to skip through the unfound file and pick another one instead, and in any case to keep on going.
I use ashuffle with a queue buffer of 1.

This appears to be a duplicate of #94. Closing this. I will follow-up in #94 instead.


Constrain optax to 0.1.x

This assists downstream users of scenic (example: see https://github.com/google-research/perch/issues/613) by constraining scenic's dependency on optax to

a specific minor version: as optax is still a 0.y.z project we might assume that the 0.1.z to 0.2.0 change contains breaking changes.
actual releases of optax, not a github  URL

Disclaimer: I don't know this history behind the extant dependency specification, e.bg. maybe there was a reason scenic needs to rely on bleeding-edge pre-release code.  Would the scenic authors be amenable to my change?

I think this is still the right thing to do, and all contributors are covered under a CLA.  Can this be merged?


fix(configpicker): close button regression

Fix #34

:tada: This PR is included in version 5.0.2 :tada:
The release is available on:

GitHub release
v5.0.2

Your semantic-release bot :package::rocket:


Overload Load.largeIcon to take a resource id

Fixed sample throwing NullPointerException
Changed build.gradle to build against the local library instead of remote dependency
Overloaded Load.largeIcon method, with updated SamplePugNotification and README.md

Hello,
After a long vacation time I am again with all gas to develop new features for the library.


Changing pin to use

Hey Matty, is there a way to change the pin that is used? I tried changing the line OneWire ds(12); to OneWire ds(13); and I'm using slayton's cartridge writer but the program seems to have a problem if I change the pin. I would normally think that the problem would be with the cartridge writer program but as far as I can tell the onewireProxy script is what is actually interfacing with the board. Is this an issue with eepromTool or should I be looking at the cartridge writer you think? Any tips?

Nevermind, it turns out I have a bad pin 13 on that particular arduino, it works with other ports and on another board it works with pin 13. Doh!

It seems you know me somehow.  I hope you find this tool useful!


unable to resolve typings

typings ERR! message Unable to resolve typings
typings ERR! caused by Unexpected token '/' at 1:1 in C:\Users\AKidanganMon\angu
lar2-demo\typings.json
typings ERR! caused by //core-js: It brings ES2015/ES6 capabilities to our ES5 b
rowsers.
typings ERR! caused by ^
typings ERR! cwd C:\Users\AKidanganMon\angular2-demo
typings ERR! system Windows_NT 6.1.7601
typings ERR! node -v v7.10.0
typings ERR! typings -v 0.6.10
Prerequisites

[ ] Did you check the FAQ?
[ ] Did you search open and closed issues to see if your issue has already been reported?

Description
[Description of the bug or feature]
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

Expected behavior: [What did you expect to happen?]
Actual behavior: [What actually happened?]
Versions
[From typings -v]

able to resolve


[Feature request] Add recursive comparison support to do the value resolution only by field (reflection)

Hi,
Summary
If you have a getter that modifies the value or only returns a subset of the value, the current resolving strategy is error prone / unusable for recursive comparison tests (see example below).
Perhaps it's possible to add the support to do the value resolution only by field via reflection? Perhabs a boolean configuration could override the current implementation?
Currently, we use the recursive comparison for all our tests and love the fact that new properties of a class are always tested immediately. Thanks for the new kind of testing 👍🏼
Example
Note: This is only an example to illustrate the problem. The easy solution would be the usage of "clean" getters. But if you have a public API you couldn't just change the internal behaviour…
public class Example
{
    private Collection<String> value;

    public Example(Collection<String> value)
    {
        this.value = value;
    }

    public String getValue()
    {
        if (value == null)
        {
            return null;
        }
        return value.iterator().next();
    }
}

class ExampleTest
{
    @Test
    void testRecCom() // should fail
    {
        Assertions.assertThat(new Example(List.of("A", "B")))
                .usingRecursiveComparison()
                .ignoringAllOverriddenEquals()
                .withStrictTypeChecking()
                .isEqualTo(new Example(List.of("A")));
    }
}

Greetings!

The best way to go would be to specify in the configuration to only use fields, something like: comparingOnlyFields() and change the implementation to introspect fields only.

Hello there. What about adding a bit logic to introspection/PropertyOrFieldSupport/getSimpleValue such that if comparingOnlyFields is set to true then we skip the property part and look straight into fields?
public Object getSimpleValue(String name, Object input) {     if (comparingOnlyFields){ //         // try as a field, then map key         ...     }else{         // try as a property, then field, then map key         ...   }
I can implement this with tests if you'd like.
Frankly speaking, we are a group of university students working on an assignment for the course Software Engineering, which requires us to contribute to the community by fixing a number of issues. We are willing to help with things we are capable of. You can let us know if there are other issues you want to fix but are too busy to do so.
Best regards.

Hello there. What about adding a bit logic to introspection/PropertyOrFieldSupport/getSimpleValue such that if comparingOnlyFields is set to true then we skip the property part and look straight into fields?
public Object getSimpleValue(String name, Object input) {
    if (comparingOnlyFields){ //
        // try as a field, then map key
        ...
    }else{
        // try as a property, then field, then map key
        ...
    }
    ...
}

I can implement this with tests if you like.
Frankly speaking, we are a group of university students working on an assignment for the course Software Engineering, which requires us to contribute to the community by fixing a number of issues. We are willing to help with things we are capable of. You can let us know if there are other issues you want to fix but are too busy to do so.
Best regards.

@sustc11810424  thanks! we still need to decide the best way to go on this one.
look for issues with label "good first issue", these are the ones that are not too complex to tackle.

@fegbers , we have finally got the time to address  #2189 which is why I'm closing this issue.
I actually wrote a test using a ComparingFields introspection strategy that addresses this issue.
I'm planning to expose the ComparingFields introspection strategy implementation in AssertJ so that it's available out of the box for users like yourself (we'll likely do the same with ComparingProperties).
We need to discuss in the team to switch the default introspection strategy implementation to ComparingProperties, I'm closing this issue and might create another one to decide on the the default introspection strategy.
Feel free to comment on the issue if you feel it's not fully addressed and we'll be happy to reopen it.


Add netkan files for new TestFlight configs.

This PR adds .netkan files for the major TestFlight config packs in Starstrider42/TestFlight-Configs.
Local testing shows that while the dependency metadata allows/forbids the correct combinations of mods, the conflict between TestFlightConfig-Stock-S42 and the original TestFlightConfigStock is not handled consistently -- the latter sometimes displays on the relationship tab as "not indexed" (despite being blocked correctly), and CKAN won't let you switch between the two config packs without uninstalling TestFlight first. 😲 I'm not sure if that's a bug in CKAN's dependency resolution or in the .ckan files.

I understand this right, that TFC-KerbalAtomics, TFC-NFPropulsion and TFC-NFSpacecraft should be installed on top of another testflight config, like TestFlightConfigStock, TestFlightConfig-Stock-S42 or the one of RO, yes?

They're designed to be mixed and matched at will; any of the config packs will work with only Generic as a dependency. While I can't imagine somebody wanting one of the mod-specific configs but not stock, I don't see any reason to enforce it.
Note that TestFlightConfigStock does exclude other configs (probably because the developer assumed people play either stock or RO), and RO is unlikely to work well with stockalike part mods like Kerbal Atomics or Near Future. But that seems like something that should be handled in the primary mods' configs, not the TestFlight packs.

While I can't imagine somebody wanting one of the mod-specific configs but not stock, I don't see any reason to enforce it.

I do. Because if not, users might just install one of the mod specific extensions and assume this also includes configs for the stock parts, but it doesn't.
I think we should remove the provides: ["TestFlightConfig"] for those three extension configs, because

they don't provide a "full" config, only extensions that need any of the both stock configs to be installed, but which isn't enforced or indicated right now.
This should solve the siimavs stock config conflicting with them, whilst keeping the conflicts between TestFlightConfigStock <-> TestFlightConfig-Stock-S42 <-> TestFlightConfigRO / RO, to ensure they don't get installed side by side.

I'm sorry, I don't quite follow what you mean by "full" config. A TF config pack adds TF modules to certain parts, no more, no less. Can't you just as well argue that a config for the stock parts isn't a full config because it doesn't support parts from mods (either these, or something that hasn't been covered at all yet, like SpaceY)?
If you want to restrict provides: ["TestFlightConfig"] to only config packs that support stock or total conversion mods (which seems to be what you're saying), is there some other mechanism to link single-mod configs to the TestFlight mod? I'd rather that config authors not be expected to add a suggests block to TestFlight itself; something that centralized would almost certainly get forgotten.

is there some other mechanism to link single-mod configs to the TestFlight mod?

Currently not. There's always been the wish for conditional recommends, but unfortunately they are not a thing yet.
So I understand your considerations. I would go with your suggestion and change the conflicts of the original TestFlightConfigStock to TestFlightConfig-Stock-S42, RealismOverhaul, the counterpart for what you did here in TestFlightConfig-Stock-S42.netkan.
This will allow the parallel installation of the mod-specific configs to the "base"/stock game config, as you wanted.
To point out that these three configs do not modify the stock parts (some might think this is the case), could you add a sentence like
This config does not modify stock parts. To use the TestFlight functionality for those, you have to install one of the TestFlight configs for stock to the abstract of these three? (You are free to word it better than me)
This should sort that out too.

Thanks! Let's try if it works out.


feat(TCOMP-469): Add an endpoint to filter the definition by execution engine.

This endpoint will allow us to get the component definition only compatible
to a given execution engine
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
Add possibility to query the enpoint /comp/definition/components with the the option executionEngine
Ex:
http://components.weave.local:8989/tcomp/definitions/components?executionengine=BEAM
http://components.weave.local:8989/tcomp/definitions/components?typology=TRANSFORMER&executionengine=BEAM
Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:

Pull request test result: success :star:

Pull request test result: success :star:


Update logback-classic to 1.4.6

About this PR
📦 Updates ch.qos.logback:logback-classic from 1.4.4 to 1.4.6
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (1.4.4).
You might want to review and update them manually.
modules/multilanguage-csharp-sdk/Surge.csproj

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.qos.logback", artifactId = "logback-classic" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "ch.qos.logback", artifactId = "logback-classic" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Codecov Report
Patch coverage has no change and project coverage change: -0.04 :warning:

Comparison is base (73ed185) 59.77% compared to head (0dc77a7) 59.74%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #350      +/-   ##
==========================================
- Coverage   59.77%   59.74%   -0.04%
==========================================
  Files         152      152
  Lines        3254     3254
  Branches       92      105      +13
==========================================
- Hits         1945     1944       -1
- Misses       1309     1310       +1

see 3 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Superseded by #356.


Use operator EQUIV instead of OR cost more cpu usage ?

According to Query Rewriting document section here, I test the yql below
select * from index where field contains "A" or field contains "B";

select * from index where field contains equiv ("A",  phrase("B", "C"));

When using the second yql, the cpu usage become much higher than the first one. Is it normal and how to reduce the cpu usage?

The vespa version I use is 7.0

The vespa version I use is 7.0

If you compare apples versus apples and comparing or with equiv with the same type of query terms I believe the performance should be roughly the same as measured by latency and cpu util
This is an apple versus apple comparison where we replace OR with equiv which acts as an OR for retrieval but threats the two terms as equal during ranking.
This is an apples versus apples comparison:
select * from index where field contains "A" or field contains "B";

and compare with
select * from index where field contains equiv("A","B");

I expect latency and cpu util to be comparable.
This query however introduces a phrase term which changes the complexity:
select * from index where field contains equiv ("A",  phrase("B", "C"));

The above involves a phrase term which is more costly to evaluate as positions of term B and C needs to be intersected which involves more cpu processing.

If you compare apples versus apples and comparing or with equiv with the same type of query terms I believe the performance should be roughly the same as measured by latency and cpu util
This is an apple versus apple comparison where we replace OR with equiv which acts as an OR for retrieval but threats the two terms as equal during ranking.
This is an apples versus apples comparison:
select * from index where field contains "A" or field contains "B";

and compare with
select * from index where field contains equiv("A","B");

I expect latency and cpu util to be comparable.
This query however introduces a phrase term which changes the complexity:
select * from index where field contains equiv ("A",  phrase("B", "C"));

The above involves a phrase term which is more costly to evaluate as positions of term B and C needs to be intersected which involves more cpu processing.

So, in conclusion using a phrase is inherently more expensive and you need to consider if it is worth it for you.
An alternative could be to search for A or (B and C) and do the more accurate processing as ranking (by using a ranking function where  documents with B adjacent to C get a higher rank score).

So, in conclusion using a phrase is inherently more expensive and you need to consider if it is worth it for you.
An alternative could be to search for A or (B and C) and do the more accurate processing as ranking (by using a ranking function where  documents with B adjacent to C get a higher rank score).

@bratseth @jobergum Thanks for you reply. As I mentioned in https://github.com/vespa-engine/vespa/issues/15876, the OR and EQUIV behave different when calculating features/scores. Any way to implement like equiv (A, B and C) which doesn't requires the positions?

@bratseth @jobergum Thanks for you reply. As I mentioned in https://github.com/vespa-engine/vespa/issues/15876, the OR and EQUIV behave different when calculating features/scores. Any way to implement like equiv (A, B and C) which doesn't requires the positions?

HI @luoyetx, could you be more specific here with examples? I'm confused so please be patient :)
It's correct that (A OR C) produces different feature score for multiple ranking features (e.g nativeRank(field), bm25(field)) than equiv(A,C).
I don't see the connection with #15876 directly since it did not not mention equiv but repeating a term several times.

Any way to implement like equiv (A, B and C) which doesn't requires the positions?

The amount of information loaded from posting lists depends on the ranking features which are used in the ranking profile, but phrase search will always need to load position information from postings lists.

HI @luoyetx, could you be more specific here with examples? I'm confused so please be patient :)
It's correct that (A OR C) produces different feature score for multiple ranking features (e.g nativeRank(field), bm25(field)) than equiv(A,C).
I don't see the connection with #15876 directly since it did not not mention equiv but repeating a term several times.

Any way to implement like equiv (A, B and C) which doesn't requires the positions?

The amount of information loaded from posting lists depends on the ranking features which are used in the ranking profile, but phrase search will always need to load position information from postings lists.

@jobergum sorry for the confusion, let me explain a use case.
If we rewrite term "cat" with "hello kitty", what we want to retrieve is the document with term "cat" or terms "hello" and "kitty"(the distance  between these two terms should be 0). If we use title contains "cat" or title contains phrase("hello", "kitty"), the score of the document(only with term "cat") can be smaller then title contains "cat"(mentioned in #15876 ). So we use equiv instead of or. the yql becomes title contains equiv("cat", phrase("hello", "kitty")) and the cpu usage is unacceptable. (we didn't test the cpu usage of first yql, maybe it also increase a lot of the cpu usage)
As @bratseth said, the phrase harms the  performance, actually for term "hello" and "kitty", ignore the distance when matching is acceptable, if we retrieve a document with title "hello xxx yyy zzz kitty", we can manually design some distance feature and using ranking model to deal with the problem. So I ask for a way to implement something like equiv("cat", "hello" and "kitty").

@jobergum sorry for the confusion, let me explain a use case.
If we rewrite term "cat" with "hello kitty", what we want to retrieve is the document with term "cat" or terms "hello" and "kitty"(the distance  between these two terms should be 0). If we use title contains "cat" or title contains phrase("hello", "kitty"), the score of the document(only with term "cat") can be smaller then title contains "cat"(mentioned in #15876 ). So we use equiv instead of or. the yql becomes title contains equiv("cat", phrase("hello", "kitty")) and the cpu usage is unacceptable. (we didn't test the cpu usage of first yql, maybe it also increase a lot of the cpu usage)
As @bratseth said, the phrase harms the  performance, actually for term "hello" and "kitty", ignore the distance when matching is acceptable, if we retrieve a document with title "hello xxx yyy zzz kitty", we can manually design some distance feature and using ranking model to deal with the problem. So I ask for a way to implement something like equiv("cat", "hello" and "kitty").

Thanks for the clarification and sorry for the late reply.

If we use title contains "cat" or title contains phrase("hello", "kitty"), the score of the document(only with term "cat") can be >smaller then title contains "cat"(mentioned in #15876 ).

Could you clarify the problem here. It's also unclear what exactly score you mean? Are we talking about nativeRank? In this case yes, the score is different but this is ranking which is ordering results and the two queries are different in the case where you use OR documents with both cat and "hello kitty" will be ranked higher. It's also unclear if the increased cpu usage is related to this change, if you track total number of hits and this increases after this change then it could be explained as doing query expansion either by OR or using equiv might increase the total number of hits.
I don't have a clear simple solution to this, but I'll give a few thoughts on this.
So rewriting the query in this way classifies as query expansion which I believe you do to try improve recall as you metadata per video is limited. The alternative to query side expansion is document side expansion which is cheaper. An interesting approach for document side expansion is document to query using a sequence to sequence generative model (e.g T5) which predicts queries for the given document text.  See e.g https://github.com/vespa-engine/sample-apps/tree/master/msmarco-ranking and how we train a LightGBM model combining multiple features over matching in multiple fields and where the expanded document queries are indexed in a separate field and where we use features like elementCompletness in the trained expression. I think that your organization have a lot of experience with ML and lots of training data so this might be an alternative to rule based query expansion to improve recall.
It is not possible to do equiv("cat", "hello" and "kitty") as equiv only works on terms and phrases and I don't see a work around to other than in the index use a query expansion field using word/exact matching to avoid phrase search.

Couple of examples using MS Marco Passage Ranking dataset
The phrase "hello kitty" occurs in 138 passages

Expanding "hello kitty" with the more general "cat" using OR brings the total hit count to 47440 passages, this increase in number of hits which is exposed to ranking does impact serving performance.

Both examples uses bag of of word BM25. Similar if using equiv the total number of hits increases, phrase or not so this might be the reason you see performance impact but difficult to assess based on the information to this ticket.

@jobergum Thanks for your detail reply. We already use OR instead of equiv, so i believe the performance is harmed by phrase. The score returned is based on nativeFieldMatch (seems not much difference with nativeRank). Why we need the score to be comparable is because we want to get multi query result and rank them with a limited number.
You also mentioned the document side expansion, but i believe it's easier to expanse query side because you don't need to update all documents when updating the generative model.
Thanks for your kindly reply, we may need to modify the score calculation logic by ourself to satisfy the demand.

Oki, using BM25 which is also a bag of words model is slightly faster than nativeFieldMatch ranking feature but the phrase constraint is the same.
I have also earlier talked about retrieval using weakAnd which also can be combined with the rank() operator to production additional features and where those features are not evaluated.
rank(weakAnd(,..), phrase(,...)) will be faster as the phrase term in this case will only be evaluated for those documents making it into the top-k result of the weakAn


ASP.NET Core 2.2  Exception

System.InvalidOperationException:“A path base can only be configured using IApplicationBuilder.UsePathBase().”

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

ID: 14fce65f-1337-58cb-2193-8d8e0e62792d
Version Independent ID: 59222460-a971-e76e-4fe9-23d4d7d768ee
Content: Get started with Swashbuckle and ASP.NET Core
Content Source: aspnetcore/tutorials/getting-started-with-swashbuckle.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @zuckerthoben
Microsoft Alias: scaddie

sorry ,my code wrong!in launchSettings.json i lost "applicationUrl"


[Request] Upload source code

Right now the only file is a readme file. If you uploaded the source to github people could help design features and fix bugs. The correct license would also copyright the work.

I will upload the source code for the plugins of Macro Deck 2.


TypeError: setting a property that has only a getter

Version 0.8.12 brings a change that causes this error. This does not happen in v0.8.11.

This is happening for me on Firefox (v53.03), but not on Chrome (v54.0.2840.59).
The third stack trace element is this:

It seems to me this issue is related to #788.

@chris13524 , thank you for reporting the issue, I will check it.

This occurs in the default installation of Angular:
sudo install -g @angular/cli
ng new angular-zonejs-test
cd angular-zonejs-test
ng serve

I get the error: TypeError: setting a property that has only a getter.
I'm on Ubuntu v17.04, Firefox v54.0 (64-bit), NPM v5.2.0, and Node.js v8.2.1.
Here's the zipped default app (I removed node_modules due to GitHub's 10 MB upload limit).
angular-zonejs-test.zip

@chris13524 , ok I will test it. But I have no Ubuntu, so I will check in mac first.


[TC-TM-1.1] Temperature Mesurement cluster AttributeList attribute value mismatch

Summary:
As per testplan TC-TM-1.1 AttributeList showing value mismatch error when executed .
expected value [0,1,2,3] but got more than that.
Testplan screenshot:

Execution screenshot:

TM_AttributeList.txt

@manjunathambiger23 The test plan is wrong. AttributeList always includes the global attributes.


Code for reproducible bug reports

Hi,
As discussed on https://github.com/raxod502/straight.el/issues/470, moving .emacs.d around to create bug reports is not really easy. There are way arounds, but it should be possible to offer some code so that users do not have to figure them out themselves.
I would imagine something like this:
(setq user-emacs-directory (make-temp-file ".emacs-" t))

(message "Starting emacs in the temporary directory %s" user-emacs-directory)

(setq straight-repository-branch "develop")
;; Insert other necessary straight settings here

(defvar bootstrap-version)
(let ((bootstrap-file
       (expand-file-name "straight/repos/straight.el/bootstrap.el"
	       user-emacs-directory))
      (bootstrap-version 5))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

;; Insert other necessary code here

It can then be called with emacs -q -l <file>.el, and should be equivalent to starting emacs in an empty .emacs.d (but a temporary folder instead).

I think #574 takes care of this for now. Closing this issue.


adding teamtreehouse

Please consider updating the PR to reflect the recently added formatting guidelines:  CONTRIBUTING

@sdmg15 updated


User Guide and API documentation

The README will shortly show its limits, so we want to develop a proper documentation.
From a bird's eye view, we need:

Content: at minimal, an expanded version of the README, and the API reference.
Hosting: a place to host the documentation, GH pages?
Version control: make it so future documentation is available as pre-release, and also host older versions (just like scikit-learn docs).

I think Material for MkDocs is a good starting point (used by FastAPI)

This is a good suggestion.
Just note that we should do a quick due diligence and compare it to sphinx (actually, the theme used by scikit-learn), and collect pros and cons. I know some people in the team would like us to follow unified conventions, but we can break them if we know there are a good reasons.

We should use the same theme and system as scikit-learn (PyData theme for sphinx).

Closed by #553 #582 #584.


Create New button on PCC_BF2_Work_Monograph template does not work

The Create New button which appears with the In series RT on PCC_BF2_Work_Monograph template does not function as intended. When I click on the button, I'm prompted to select a template from a drop-down list. In most cases, a Serial Work template would be the logical choice (either the LD4P Serial Work or PCC Serial Work). No matter what I select, Sinopia instead sends me to a blank Monograph Work template, and the monograph work I was working on is gone. There is an error message that flashes between the time of selecting from the menu and the blank Work template coming up, but it goes by too fast to read.
I am using Sinopia Stage in Firefox 82.0.3.
This is possibly related to issue #2712 .

The list of templates offered under Create New is as expected (see below) but you're right, it's not opening the template that you do select from the Create New dropdown.
As expected:
"Create New" shows you a list of any resource template of type Work. (Since the lookup is to Sinopia Works, Sinopia assumes you want to create a new Work. Sinopia has no idea what kind of Work.)
Bug:
When you do select something from the dropdown, the selected template should open in a new tab, leaving the one you were working on alone.
We need to fix this bug.
As an enhancement, we could consider letting the template creator define what the button does. For example, in the "In Series" field, one would configure the template so that "Create New" did not give the cataloger a choice but instead opened a new "Serial Work" template. This could be really useful for Contributor too, to open a new template for a new Agent.

I'm not seeing a Create New button anymore. Here's my test template with a field that's a lookup to Sinopia Works.
https://stage.sinopia.io/editor/testing123:1

in this template on stage https://stage.sinopia.io/editor/pcc:bf2:Monograph:Work the "in series" field has a create new button and it seems to be working properly now.
but the "has BIBFRAME instance" field does not have a Create New button, even though it is a lookup to "Sinopia BIBFRAME Instances". Shouldn't that produce a Create New button with a list of templates of class Instance?
also, the test template above https://stage.sinopia.io/editor/testing123:1 has 2 fields both lookups to "Sinopia BIBFRAME work resources" but it is not showing a Create New button either.

in this template on stage https://stage.sinopia.io/editor/pcc:bf2:Monograph:Work the "in series" field has a create new button and it seems to be working properly now.
but the "has BIBFRAME instance" field does not have a Create New button, even though it is a lookup to "Sinopia BIBFRAME Instances". Shouldn't that produce a Create New button with a list of templates of class Instance?
also, the test template above https://stage.sinopia.io/editor/testing123:1 has 2 fields both lookups to "Sinopia BIBFRAME work resources" but it is not showing a Create New button either.


.includes is called before polyfills are loaded

Examples bug report
Example name
With Polyfills
Describe the bug
This is a polyfill example but it is using .includes before it loads the polyfills. .includes itself is not available before ES2016 and so depending on how far back you go in testing this would crash I believe.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Run this example on any ES5 browser

Expected behavior
The polyfills are loaded and the example works as expected
Additional context
This line specifically, it seems like it should be using .indexOf(...) < 0 since this is runs before the polyfills load.
https://github.com/zeit/next.js/blob/canary/examples/with-polyfills/next.config.js#L9

This line specifically, it seems like it should be using .indexOf(...) < 0 since this is runs before the polyfills load.
/examples/with-polyfills/next.config.js@canary#L9

next.config.js should never be loaded in the browser, at least it won't by default 🤔

I see, ok thanks. I didn't have time to fully vet this but wanted to double check the reasoning about it.


serde_derive with pre-compiled binary does not work when building via Bazel with rules_rust

When building some crate which using or depending serde_derive 1.0.172 via Bazel using rules_rust on x86_64 GNU Linux the build will fail with:
error: proc-macro derive panicked
 --> main.rs:3:10
  |
3 | #[derive(Serialize, Deserialize, Debug)]
  |          ^^^^^^^^^
  |
  = help: message: failed to spawn process: Os { code: 2, kind: NotFound, message: "No such file or directory" }

It does not find the precompiled binary used in the derive macro.
Would it be possible to add an opt-out option to not use the precompiled binary for serde_derive which can be set until a proper solution is found?
Issue in the rules_rust repository: https://github.com/bazelbuild/rules_rust/issues/2071
Somehow related issue: https://github.com/serde-rs/serde/issues/2526

You would need to sort this out with other Bazel users.
I am using serde_derive successfully from Buck which is a conceptually similar system, so it is definitely usable.

I would also like to mention that it does not work with cargo2nix either.
I don't understand how I can configure serde to compile from source and have not found any documentation about it yet :-(

I'm also a nix user, investigating the problem. It's not that hard to make the binary from crates.io visible but it just segfaults straight away on my CPU with "invalid instruction". So whatever fixes, including whatever suggested in #2526 are done, this is just not going to work.
Presumably there should simply be a feature to opt-into from-source code path instead of picking based on platform and crossing fingers.

Presumably there should simply be a feature to opt-into from-source code path instead of picking based on platform and crossing fingers.

In meantime we'll just be maintaining a fork with a patch to just use the source version I guess...
https://github.com/tsurucapital/serde/commit/45a3539b1ed5104a857f4512381eaf8cb7b736eb

Presumably there should simply be a feature to opt-into from-source code path instead of picking based on platform and crossing fingers.

In meantime we'll just be maintaining a fork with a patch to just use the source version I guess...
tsurucapital@45a3539

Thank you for this. I've been looking for a fork of some kind to use.


Bind console log method so it works in the browser

It's illegal to call unbound console.log in the browser.


unable to compile to DOIT esp32 DEV KIT

I have been trying to compile the code for esp32 and enc28j60 as an ftp client to receive the data. but throws the following error
Using board 'esp32doit-devkit-v1' from platform in folder:
In file included from c:\Users\Admin\Documents\Arduino\libraries\FTPClient_Generic-main\src/FTPClient_Generic.h:41,
                 from C:\Users\Admin\Desktop\wificlienteps32\FTPClient_Generic-main\examples\Ethernet\FTPClient_DownloadFile\FTPClient_DownloadFile.ino:17:
c:\Users\Admin\Documents\Arduino\libraries\FTPClient_Generic-main\src/FTPClient_Generic_Impl.h: In member function 'void FTPClient_Generic::OpenConnection()':
c:\Users\Admin\Documents\Arduino\libraries\FTPClient_Generic-main\src/FTPClient_Generic_Impl.h:246:50: error: no matching function for call to 'EthernetClient::connect(char*&, uint16_t&, uint16_t&)'
   if ( client.connect(serverAdress, port, timeout) )
                                                  ^
In file included from c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/Ethernet_Generic_Impl.h:68,
                 from c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/Ethernet_Generic.h:90,
                 from C:\Users\Admin\Desktop\wificlienteps32\FTPClient_Generic-main\examples\Ethernet\FTPClient_DownloadFile\defines.h:409,
                 from C:\Users\Admin\Desktop\wificlienteps32\FTPClient_Generic-main\examples\Ethernet\FTPClient_DownloadFile\FTPClient_DownloadFile.ino:15:
c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/EthernetClient_Impl.h:94:5: note: candidate: 'virtual int EthernetClient::connect(IPAddress, uint16_t)'
 int EthernetClient::connect(IPAddress ip, uint16_t port)
     ^~~~~~~~~~~~~~
c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/EthernetClient_Impl.h:94:5: note:   candidate expects 2 arguments, 3 provided
c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/EthernetClient_Impl.h:69:5: note: candidate: 'virtual int EthernetClient::connect(const char*, uint16_t)'
 int EthernetClient::connect(const char * host, uint16_t port)
     ^~~~~~~~~~~~~~
c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/EthernetClient_Impl.h:69:5: note:   candidate expects 2 arguments, 3 provided
In file included from c:\Users\Admin\Documents\Arduino\libraries\FTPClient_Generic-main\src/FTPClient_Generic.h:41,
                 from C:\Users\Admin\Desktop\wificlienteps32\FTPClient_Generic-main\examples\Ethernet\FTPClient_DownloadFile\FTPClient_DownloadFile.ino:17:
c:\Users\Admin\Documents\Arduino\libraries\FTPClient_Generic-main\src/FTPClient_Generic_Impl.h: In member function 'void FTPClient_Generic::InitFile(const char*)':
c:\Users\Admin\Documents\Arduino\libraries\FTPClient_Generic-main\src/FTPClient_Generic_Impl.h:403:55: error: no matching function for call to 'EthernetClient::connect(IPAddress&, uint16_t&, uint16_t&)'
   if (dclient.connect(_dataAddress, _dataPort, timeout))
                                                       ^
In file included from c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/Ethernet_Generic_Impl.h:68,
                 from c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/Ethernet_Generic.h:90,
                 from C:\Users\Admin\Desktop\wificlienteps32\FTPClient_Generic-main\examples\Ethernet\FTPClient_DownloadFile\defines.h:409,
                 from C:\Users\Admin\Desktop\wificlienteps32\FTPClient_Generic-main\examples\Ethernet\FTPClient_DownloadFile\FTPClient_DownloadFile.ino:15:
c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/EthernetClient_Impl.h:94:5: note: candidate: 'virtual int EthernetClient::connect(IPAddress, uint16_t)'
 int EthernetClient::connect(IPAddress ip, uint16_t port)
     ^~~~~~~~~~~~~~
c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/EthernetClient_Impl.h:94:5: note:   candidate expects 2 arguments, 3 provided
c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/EthernetClient_Impl.h:69:5: note: candidate: 'virtual int EthernetClient::connect(const char*, uint16_t)'
 int EthernetClient::connect(const char * host, uint16_t port)
     ^~~~~~~~~~~~~~
c:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main\src/EthernetClient_Impl.h:69:5: note:   candidate expects 2 arguments, 3 provided

Using library SPI at version 2.0.0 in folder: C:\Users\Admin\AppData\Local\Arduino15\packages\esp32\hardware\esp32\2.0.5\libraries\SPI
Using library Ethernet_Generic at version 2.6.2 in folder: C:\Users\Admin\Documents\Arduino\libraries\Ethernet_Generic-main
Using library EthernetWebServer at version 2.2.4 in folder: C:\Users\Admin\Documents\Arduino\libraries\EthernetWebServer-master
Using library Functional-Vlpp at version 1.0.2 in folder: C:\Users\Admin\Documents\Arduino\libraries\functional-vlpp-master
Using library FS at version 2.0.0 in folder: C:\Users\Admin\AppData\Local\Arduino15\packages\esp32\hardware\esp32\2.0.5\libraries\FS
Using library FTPClient_Generic at version 1.3.0 in folder: C:\Users\Admin\Documents\Arduino\libraries\FTPClient_Generic-main
exit status 1

Compilation error: exit status 1

HI @KvarTechnologies
The library currently is not supporting ESP32 using Ethernet (W5x00 or ENC28J60).
I'll modify and release a new version to add that support within today.
Best Regards,

Hi @KvarTechnologies
The new FTPClient_Generic v1.4.0 has just been published in both Arduino IDE and PIO. Your contribution is noted in Contributions and Thanks.
Please test and continue reporting any more issue
Best Regards,

Releases v1.4.0

Add support to ESP32/ESP8266 using Ethernet W5x00 or ENC28J60 with Ethernet_Generic or EthernetENC library
Add astyle using allman style. Restyle the library
Update Packages' Patches

Notes
You can only use EthernetENC library for ENC28J60, not the old and deprecated UIPEthernet
Just changing these lines from
https://github.com/khoih-prog/FTPClient_Generic/blob/36e9fb792c256d9e364460e270102dde05d203b4/examples/Ethernet/FTPClient_DownloadFile/defines.h#L363-L367
to
// Only one if the following to be true
#define USE_ETHERNET_GENERIC  false
#define USE_ETHERNET_ESP8266  false
#define USE_ETHERNET_ENC      true
#define USE_CUSTOM_ETHERNET   false

I just had a test using ESP32 + W5500 with the result here
https://github.com/khoih-prog/FTPClient_Generic#13-FTPClient_DownloadFile-on-ESP32_DEV-with-W5x00-using-Ethernet_Generic-Library

The example file is compiling and able to upload to esp32_ dev board. As suggested USE_ETHERNET_ENC enable by the True, but serial monitor stops after printing following
Starting FTPClient_DownloadFile on ESP32_DEV with ENC28J60 using EthernetENC Library
FTPCLIENT_GENERIC v1.4.0
[EWS] =========== USE_ETHERNET_ENC ===========
[EWS] Default SPI pinout:
[EWS] MOSI: 23
[EWS] MISO: 19
[EWS] SCK: 18
[EWS] SS: 5
[EWS] =========================
[EWS] ESP32 setCsPin: 5

I suggest you test your board and ENC28J60, with the basic examples using EthernetENC library first.
Post the successful results here before moving on with this library.

FYI, I'm testing here and OK. Possibly your wiring, boards, etc.
Starting FTPClient_DownloadFile on ESP32_DEV with ENC28J60 using EthernetENC Library
FTPCLIENT_GENERIC v1.4.0
[EWS] =========== USE_ETHERNET_ENC ===========
[EWS] Default SPI pinout:
[EWS] MOSI: 23
[EWS] MISO: 19
[EWS] SCK: 18
[EWS] SS: 5
[EWS] =========================
[EWS] ESP32 setCsPin: 5
[EWS] =========================
[EWS] Currently Used SPI pinout:
[EWS] MOSI: 23
[EWS] MISO: 19
[EWS] SCK: 18
[EWS] SS: 5
[EWS] =========================
Using mac index = 7
Connected! IP address: <IP_ADDRESS>
Max Free Heap: 116724
[FTP] Connecting to:  <IP_ADDRESS>
[FTP] Command connected
[FTP] Send USER =  ftp_test
[FTP] Send PASSWORD =  ftp_test
[FTP] Send CWD
Creating new file helloworld.txt
[FTP] Send TYPE Type A
[FTP] Send PASV
[FTP] outBuf = 227 Entering Passive Mode (192,168,2,112,66,27).

[FTP] Data port:  16923
[FTP] _dataAddress:  <IP_ADDRESS> , Data port:  16923
[FTP] Data connection established
[FTP] Send STOR
[FTP] Write File
[FTP] Close File
[FTP] Send TYPE Type A
[FTP] Send PASV
[FTP] outBuf = 227 Entering Passive Mode (192,168,2,112,183,31).

[FTP] Data port:  46879
[FTP] _dataAddress:  <IP_ADDRESS> , Data port:  46879
[FTP] Data connection established
[FTP] Send APPE
[FTP] Write File
[FTP] Close File
[FTP] Send TYPE Type A
[FTP] Send PASV
[FTP] outBuf = 227 Entering Passive Mode (192,168,2,112,90,251).

[FTP] Data port:  23291
[FTP] _dataAddress:  <IP_ADDRESS> , Data port:  23291
[FTP] Data connection established
[FTP] Send RETR
[FTP] Result:  150 Opening ASCII mode data connection for helloworld.txt (75 bytes).
226 Transfer complete.

The file content is: Hi, I'm a new ASCII file created @ millis = 6681
Added text @ millis = 7490
[FTP] Send TYPE Type A
[FTP] Send PASV
[FTP] outBuf = 227 Entering Passive Mode (192,168,2,112,231,117).

[FTP] Data port:  59253
[FTP] _dataAddress:  <IP_ADDRESS> , Data port:  59253
[FTP] Data connection established
[FTP] Send LIST
[FTP] Result:  150 Here comes the directory listing.
226 Directory send OK.

hello_world.txt

helloworld.txt

octocat.jpg

CloseConnection
[FTP] Connection closed

Thanks for the support, your library is now compatible with the ESP32 Dev  board. A small suggestion, it will be great if there would have been a flag to state the connected or disconnected, it waits for a quite a long time for reconnecting. Function to start a connection, if not connected don't move for the further process


Refactor BotStateManager into User and Conversation components

Addresses #156
Addresses #134
Deleted BotStateManager
Created UserStateManagerMiddleware
Created ConversationStateManagerMiddleware
Updated Tests.
Updated Samples.
Outstanding questions (for a different issue/PR) : Are we getting any value out of the "BotState" class? Seems this should just go away and we provide a Conversation and User object handing off BotContext. If folks want to add their own, they should be able to using a more robust mechanism.

Doing the merge dance now.


Add landing page for lessons

Move lessons directories into a top-level lessons folder
Add an index.html file that provides links to all relevant lessons in that folder and associated subfolders

@colinsauze - is this something that's already implemented in https://github.com/carpentriesoffline/carpentriesoffline-installer?

@colinsauze confirmed via Slack that this should go in this project.
So @garezana - you're all set to proceed to putting this together when you have the chance.


Translations not working for 5.0.0-beta1 release

Note: I don't think we should be running translations during a release, we often seem to have drama. We should be doing this outside of releases and raising PRs to the next-minor branches
I do have tx installed locally
cow release:translate -vvv --skip-fetch-tags 5.0.0-beta1 silverstripe/recipe-kitchen-sink
[translations] Pulling sources from transifex for silverstripe/framework (min %10 delta)
RUN  'find /home/myuser/Releases/release-silverstripe_recipe-kitchen-sink-5.0.0-beta1/vendor/silverstripe/framework/lang -type f ( -name ".yml" ) -exec touch -t<PHONE_NUMBER>19.45 {} ;'
ERR  sh: 1: exec: find /home/myuser/Releases/release-silverstripe_recipe-kitchen-sink-5.0.0-beta1/vendor/silverstripe/framework/lang -type f ( -name ".yml" ) -exec touch -t<PHONE_NUMBER>19.45 {} ;: not found
ERR
RES  127 Command did not run successfully

relevant:

https://github.com/silverstripeltd/product-issues/issues/399
https://github.com/silverstripeltd/product-issues/issues/563

@emteknetnz No PRs linked?

Sorry my bad, have linked

We also should have a PR that pulls the translation logic out of cow. We don't want to maintain two sets of this thing.

All done


Multiple Instances Issue

Hello again.
I have two diffrent sheet and I would like to show them with diffrenet pages.
I'ım using MMM-Pages module for this.
Page 1 - > Module for  Sheet 1  display
Page 2 - > Module for  Sheet 2  display
Could you please help me about this issue ?

@emrahaslan I unfortunately cannot help with other modules. I would suggest posting to that module's repo if you are having issues with their module. There is no reason you cannot make multiple instance of this module however as long as both of the configurations are correct.

Hi Ryan, I think there is missunderstood.
Could you please tell how can i duplicate this module?
I would like to display my “Sheet1” into bottom_left and my “Sheet2” into bottom_right. Is that possible?

I need to two different instance like,
MMM-GoogleSheets
MMM-GoogleSheets_2

@emrahaslan yes it is possible. You can just add them as two different instances in your config file, each with a separate location
{     module: "MMM-GoogleSheets",     header: "First Instance",     position: "bottom_left",     config: {         url:"url",         sheet: "Sheet1",         range: "range" },{     module: "MMM-GoogleSheets",     header: "Second Instance",     position: "top_left",     config: {         url:"url",         sheet: "Sheet1",         range: "range" }
You'll need to make sure the module name is not changed (if that's what you were trying with your most recent comment)


(docs) Fiber.abort can be passed an object

There are some redundant whitespace changes in this PR.

We need an .editor_config and some agreement on how excess whitespace is handled (in general) so we can fix this once and for all... trailing whitespace issues is a unnecessary frustration when just trying to make a PR for a small fix.

Related to #982

editorconfig

#846 !  I agree with removing all the trailing spaces.

I took another pass at this.

Better now.


Fix NPM audit

Could you update the dependencies so that npm audit successfully passes? Current output of npm audit:

# npm audit report

postcss  <8.4.31
Severity: moderate
PostCSS line return parsing error - https://github.com/advisories/GHSA-7fh5-64p2-3v2j
No fix available
node_modules/addons-linter/node_modules/postcss
  addons-linter  <=6.14.0
  Depends on vulnerable versions of postcss
  node_modules/addons-linter
    web-ext  >=1.0.0
    Depends on vulnerable versions of addons-linter
    Depends on vulnerable versions of sign-addon
    node_modules/web-ext
      vite-plugin-web-extension  *
      Depends on vulnerable versions of web-ext
      node_modules/vite-plugin-web-extension

request  *
Severity: moderate
Server-Side Request Forgery in Request - https://github.com/advisories/GHSA-p8p7-x288-28g6
Depends on vulnerable versions of tough-cookie
No fix available
node_modules/request
  sign-addon  *
  Depends on vulnerable versions of request
  node_modules/sign-addon

tough-cookie  <4.1.3
Severity: moderate
tough-cookie Prototype Pollution vulnerability - https://github.com/advisories/GHSA-72xf-g2v4-qvf3
No fix available
node_modules/tough-cookie

7 moderate severity vulnerabilities

Some issues need review, and may require choosing
a different dependency.

Originally posted by @danielptv in https://github.com/aklinker1/vite-plugin-web-extension/issues/149#issuecomment-1824841779

I've enabled dependabot and will be fixing all the issues mentioned there. That should cover the NPM audit as well.

Version upgrades released in v4.0.0


Don’t require("crypto") in browsers.

Same fix as tensorflow/tfjs#355.

This change is

 Thanks! If crypto is ignored here,  then users of tfjs-core (e.g. tfjs) won't have to explicitly ignore it, right?

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Awesome, thanks Mike! I think you're right, since we ignore it we don't have to do it for all dependencies of core.