Feature request - ESMValTool html reports

For the past two years, we've been using BGC-val to evaluate the marine component of the spin up and CMIP6 runs of the UK Earth System Model (UKESM). One of the main selling points of BGC-val, was that it can summarise the results of its metrics in a simple, clear, shareable and mobile-friendly html website. Here's an older example report that we showed at EGU2017.
The report maker tool was crucial for the uptake in use of the tool and it allowed several users to share monitoring and evaluating the model. Similarly, the ILAMB  interface is very popular among land surface modellers for this reason, among others. I'd like to implement a similar feature for ESMValTool. I've discussed this with @valeriupredoi a few times, and he recommended starting a discussion issue on github.
This is how I envisage the interface with ESMValTool:

A MakeReport Boolean will need to be added to the global config.
The html will be added to a new directory, report, alongside plots, preproc, run and logs. This new directory will contain an index.html file, an images folder and an assets directory. The index.html file will contain the bulk of the report. The report directory would be fully portable, and would contain everything needed to show the report.
The report maker would have to run run after the diagnostics have completed, and search recursively through the plots folder of the output directory, adding every figure to the index.html page.
Each image would be shown alongside some metadata in a table. The metadata would include information like original dataset, scenario, ensemble member, preprocessors, diagnostic scripts, etc.... In some cases, metadata can be added directly into the image file metadata. However, this might not be compatible with all supported image formats. In either case, we'd need to make a wrapper for pyplot.savefig/iris.savefig which adds metadata into the image.
The report maker would likely not be compatible with tools that already do this, like autoassess, but would instead function as a "native" ESMValTool version of such tools.

I'm happy to reuse the code that we had in BGC-val. It would need a bit of tweaking, but wouldn't be a major issue. The HTML was initially taken from html5up.net and was under a creative commons licence. This was a html5 ready, mobile friendly, scalable website template. However, I'm not 100% confident that this would be the wisest option. It may be more sensible to built a new report template from scratch. Do we have any html experience in ESMValTool?
I'm not the best html coder, nevertheless, I'm happy to take a crack at this. However, I'd like to have the discussion first, to determine whether there is support for the idea, but also whether it would be feasible and worthwhile.

Do ESMValTool developers think this would be worthwhile? Would people use it?  Is there something like this already implemented?
Would there be licensing issues with the html5up licence?
Is it sensible or possible to run a python tool after the diagnostics have completed?
How can we associate image metadata with individual figures? What metadata would we want to include?
Can we add a ESMValTool wrapper for pyplot.savefig?
Is the format that we showed at EGU2017 appropriate or do you think we could make some improvements?

Incidentally, point 5 here  may be a good way to enforce a specific ESMValTool image format and style.  This would be a great way to build an ESMValTool standard image style, which would help with brand recognition. Or it would be a way to make sure our images are appropriate for AR6 or specific journals with style requirements.

Related to #405 and #328.

@nielsdrost This might be an interesting issue for you to have a look at.

Do ESMValTool developers think this would be worthwhile? Would people use it? Is there something like this already implemented?

In the past (v1), there had been several situations where such a report/website was needed and implemented, e.g. see C3S_511 (pdf-report, based on sphinx) and cmip6 (website).

Would there be licensing issues with the html5up licence?

I would be cautious about integrating external themes that are licensed other than Apache2.

Is it sensible or possible to run a python tool after the diagnostics have completed?

This could maybe be just another diagnostic that takes the others as ancestors.

How can we associate image metadata with individual figures? What metadata would we want to include?

The provenance system that @bouweandela is implementing in https://github.com/ESMValGroup/ESMValTool/tree/version2_provenance passes all relevant information to the final image file, so that a report can make use of this when showing the images, see #240, #649.
In general such a reporting is needed. It should be flexible regarding output format (pdf-reports, static websites, etc.). In case of very large numbers of output files a static website might be not sufficient (then a database or a database like file might help).

@LisaBock you might like to join here?


Support for Firefox

I created a plugin for Firefox, which changes the theme over an extension, switches the settings and dev tools theme and lets other websites and extension see if you are using a dark theme.

Wow great work I was working on something similiar but never managed to finish it, is there any chance to port this extension to chrome also?


Hubitat2prom does not parse errant string values for battery statuses correctly

I came across a small issue when setting up some battery monitoring. I have a couple of devices that were offline and the battery value was not a numeric but a string “unknown”, this causes an exception and the rest of the JSON doesn’t get parsed
{"battery":"unknown","dataType":"STRING"}
---> System.FormatException: Either the JSON value is not in a supported format, or is out of bounds for a Double.
Would it be possible to handle none numeric types, perhaps convert to -999 to indicate an issue?

https://community.hubitat.com/t/release-prometheus-device-metrics-hubitat2prom-in-c/90365/4

I will update hubitat2prom to parse "unknown" as -1. I'll take a quick look to check whether this is a problem across the board (with other attributes and devices), and the battery attribute just happened to trigger the error.

[ ] Find and fix the cause of the bug
[ ] Check whether this is a systemic issue in how hubitat2prom parses attributes
[ ] Add one or more tests to cover the bug


override bootstrap CSS to ensure text is black against white background

fixes #1576
when rendering drop-down menus in the LookupComponent, we should be sure the text is black (since the background is white), so we can read it

LGTM; will approve once merge conflict is resolved.

rebased and force pushed


Asynchronous config values

Is there a preferred (or at least possible) way to populate configuration values asynchronously? I was hoping to be able to do this in config/boostrap, but it looks like before the bootstrap file runs Sails expects the database connection config to be available.
My particular use case is that some of my app's configuration data is encrypted, and the decryption is an asynchronous process. Ideally I'd be able to setup my data before anything internal to Sails expects to be ready.
An alternative solution would be to handle this setup in the host environment/start script and provide the data as environment variables, but if possible I think it would be simpler/better to do it within the app.
Is this possible within Sails?

I also tried using hooks to solve this, but I ran into the problem where another hook (orm) is trying to access the configuration data in parallel. There is sails.after to allow hooks to wait for other hooks. I tried to find something like sails.before to force other hooks to wait for mine, but it looks like that doesn't exist.

For now I've modified the app.js file that sails generated to load my configs before calling sails.lift, then pass the result into rc. This seems to work well enough. I'll leave this open in case there is a better idea. Feel free to close this issue if that is actually the best way to do it.

For now I'm modifying the app.js file that sails generated to load my config and pass it into sails.lift().

@evansolomon Great question.  At the moment the answer is no, Sails doesn't support asynchronous configuration, so your workaround is the best solution.  Definitely something we'll keep in mind.  Thanks for using Sails!

Okay, thanks for letting me know. Would definitely be a nice thing to support eventually, even some kind of reallyBeforeEverythingEvenDatabaseConnectionsYoureOnYourOwnWildWest place to hook into the app starting process would probably get most of the job done.


[decoupled-execution] change linkedlist to ordered dictionary

Replace the underlying data structure from double-linkedlist to an ordered dictionary

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/diem/diem/tree/main/developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

/land


Pipeline parametrization and running in docker interact in weird ways

I have  .popper.yml file that looks like this:
badge-server-url: http://badges.falsifiable.us
metadata:
  access_right: open
  license: CC-BY-4.0
  publication_type: article
  upload_type: publication
pipelines:
  my-pipeline:
    envs:
      local/my-environment:
        args: !!set
          --ipc=host: null
          --runtime=nvidia: null
    parameters:
    - par1: var1
    - par1: var2
    - par1: var3
    - par1: var4
    path: pipelines/my-pipeline
    requirements: {}
    stages:
    - train
    - test
    timeout: 0

When I run this, something weird happens. The popper run inside the Docker environment does each Job (for var1:4), but then the popper OUTSIDE the docker (which launched it in the first place), goes through and does them all again. You end up with a total of 16 jobs run. Further, the popper run inside the docker keeps overwriting the 0 directory in popper/local_my-environment/0. Once all the parameters have been run inside the docker, it then moves onto the 1 directory and does it all over again. Strange! Not sure if I'm doing something wrong. Let me know if I am!
Thanks!

hey @pseeth, thanks for reporting this. I'll take a look and report back on what I find


@import with '../*' cannot be resolved

I wrote a CustomImporter to retrieve a SASS file URL. It all works well. But there is no way to support paths starting with "../" in import statements, as I don't have any clue what the URL of the importing SASS file is at this point.
dart-sass-java should give a reference to the importing document, like in jsass: https://gitlab.com/jsass/jsass/-/blob/master/src/main/java/io/bit3/jsass/importer/Importer.java

From sass.embedded_protocol.EmbeddedSass.InboundMessage.CompileRequest you can reconstruct this information and forward it to the Importer:
    private String getCompileSource(CompileRequest compileRequest) throws IOException {
        final CompileRequest.InputCase inputCase = compileRequest.getInputCase();
        try {
            if (inputCase == CompileRequest.InputCase.PATH) {
                return new File(compileRequest.getPath()).toURI().toURL().toString();
            }
            if (inputCase == CompileRequest.InputCase.STRING) {
                return compileRequest.getString().getUrl();
            }
        } catch (MalformedURLException mue) {
            throw new IOException(mue);
        }
        return null;
    }

Can you provide a complete example which reproduces your problem?


Remove the in-app documentation

Currently there is a page in EQUELLA which is built by downloading doco from maestro.equella.com. I suspect we just want to get rid of this as the doco will be accessible from github.
So the places in which doco needs to be removed:

Downloads tab (server administrator)

IntegrationDownloadTab.java
The "integrationDownloadSection" extension point and anything that extends it

REST API docs

swagger.ftl

For now just removing it is ok, eventually we probably want to put back in a link somewhere to github docs (perhaps based on a tag if our docs are versioned)

Sounds very reasonable.

Also in the "Integration Pack" (the code samples etc)

I've removed the tab and all that was generating it... but I don't know if I should go beyond and touch something in the /com.tle.integration.standard project... or just leave it there.

As the code has been merged, I close this one. We can reopen if something more arises.


Fix eq

https://github.com/goblint/analyzer/pull/260/files#r662117738

Fixed. I made eq and ne more precise as suggested by Michael.


Remove RGW Objectstore Unable to create buckets

this issue was fixed by software update

@skroczek  here now as a new commit

Looks good. Thank you for your contribution


Refactoring: Split greenpak4_counters pass into a tech-independent "find counters" pass followed by techmapping of count cells to GP_COUNTx

Will be sending a PR shortly.

Fixed in #399.


Make index.js executable

Adds a shebang so you can the script via shell by name. e.g.
eslint-rules-mapper ./path/to/.jshintrc > ./.eslintrc

Looks good! Thanks


What is the license of this work?

Hello EUCA team,
Huge work with real-life studies, templates, structured analyses, etc. Congrats!
However, I have not found any mention of a license for this content. Maybe I just missed it.
From the way it is presented, I assume that it looks like a Creative Commons licence, but would you mind clarifying that aspect?
It would help me a lot on knowing how and what I can use, among the templates you provides, etc.
Best,

Hi datajms,
Thanks for your message! I've added the cc license to the work. Feel free to provide any comments or feedback:)

@weinajin, Great, thanks for clarifying!
I may use some part of it, or ideas in future project. I'll try to give feedbacks then!


Allow custom subs

here's a gif that should be pretty easy to emulate

[x] add custom subtitle text options (how?)
[ ] add custom subtitle fonts
[ ] add custom subtitle placement

I added the ability to have a single caption for the whole gif, but it's a serious UI challenge to have a sensible way on the command line to have multiple captions. It basically requires a whole new programming language.
Thoughts:

I could do it by allowing people to specify ASS files (which name will never not be funny to me, btw)
I could do it by allowing people to specify complex filter pipelines (parts of pipelines?) on their own
I could write my whole own DSL
I could tell people to use -v, copy the command line, and write it their own selves
I should provide examples of how I would do it! maybe in a document in the repo.


fix: sig anchors in the example section

Not sure if this was intended but some anchors were annotating the name of the function without the parameters as a sig, and some took all of the signature inside a sig anchor. So I made it consistent, and fixed a missing /.

Thanks for the PR! The examples and the ANCHOR definitions are used to render the tutorial. Changing the name of the anchor, for example, will break the building of the page. If you're interested in changing the examples, I'd encourage you to build the documentation using mdbook and make sure the examples look fine before we can merge the changes.

Closing this for now since there is no updates/response. @kayagokalp if you get a chance to update the PR, feel free to reopen


resolveld bug shutting down terminals even though keep_terminals flag…

… is True
*Issue #, if available: 26
*Description of changes: Added needed logic to prevent unwanted shutdown in case that existing terminals should be kept alive because "keep_terminals"-flag is set to True.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Merging in #28


[FEATURE]: Add mode: mariadb to mysql

Describe what you want
Hello,
We use mariadb at work. I don't know why, but seniors told me that "it's just a fork of mysql because of licensing". So I used it as any mysql. With mysql2 driver. While using Drizzle, I found out that mariadb doesn't support lateral derived tables, because my SQL for db.query failed on LEFT JOIN LATERAL.
I don't think that it would be proper to treat mariadb as different db, because in my experience, it feels closer to mysql than Planetscale (at least it has fkeys). So I wanted to ask, what do you think about having a new mode: mariadb for mysql in Drizzle?

+1, any updates?

+1, any update?

While not being quite the same, someone made a PR a MariaDB driver.
https://github.com/drizzle-team/drizzle-orm/pull/1692

While not being quite the same, someone made a PR a MariaDB driver, but yet to be merged. #1692

Just letting everyone know that my PR only partially solves this issue, as the RQB syntax (db.query) doesn't work due to MariaDB not supporting lateral joins. However, it would support the core API syntax (db.select).
The idea of a mode for MariaDB has only been briefly discussed with the team (that I'm not officially a part of), and the RQB API would need to be extended and tested extensively (at least, I think that would be the case).

There are valid reasons to use MariaDB over MySQL. I hope this gets implemented too 🙏
https://mariadb.org/how-mariadb-and-mysql-performance-changed-over-releases/


Rational mode operations on complex numbers should use rational values for calculations

Things like: 7i/(7+9i) = (49i+63)/130 or (63/130, 49/130) in our notation.

This is a HUGE change, and makes parsing complex numbers from strings EVEN HARDER than it was before.
Also, we're going to make the change that even without rational mode being active, if a complex number was defined with fractions, they will remain that way, and continue to do rational calculations regardless of the mode.

Consider setting up the fractions as "always proper" to make the presentation a little nicer. And maybe 1 / 2 ought to be 1/2 in this case (we decided a while back that this was kinda hard to read in the straight fraction case, so maybe we need yet another flag here). Then we would need another flavor of BigFraction.ZERO which is "proper" (or maybe every use of it could be flagged as proper because we almost never present it anyway??)

Trying to actually compute my example above:
complex('7/1i')/complex('7/1+9/1i')
Error: Negative exponent.

It's doing this at the moment (note: two problems here):
> c1=complex(frac(7))
c1 = complex (frac (7)) -> ( 7, 0 )
> c2 = complex(frac(7), frac(9))
c2 = complex (frac (7), frac (9)) -> ( 7 / 1, 9 / 1 )
> c1/c2
c1 / c2 -> ( 0.3769230769230769230769230769230769, -0.4846153846153846153846153846153846 )
> c1=complex('7/1')
c1 = complex ('7/1') -> ( 7 / 1, 0 / 1 )
> c1/c2
Error: Negative exponent.

In case you missed it, the first c1 value should look like: ( 7 / 1, 0 / 1 ) but it doesn't, while the second problem is ... obvious ...

ComplexNumber.valueOf is screwed up in the way it recognizes (or doesn't) fractional values... getDecimal is first ...

Okay, the "Negative exponent" error comes from MathUtil.tenPower(-1) because the denominator is 130, which has a scale of -1. There is a note at line 299 in BigFraction that asks: "what about negative scale?" ... and here we have it.

Test case:
c1 = complex('7/1')
c2=complex(frac(7), frac(9))
c3 = c1/c2
c3 * c2

Note: the actual result here is different than above (63, -49)


tests: fix macos test

Rebased after #219, I think I have a clearer picture now on how to fix some tests now


The input line is too long -- affects R plugin and perhaps others

Seems to be a classic cmd bug in 1.3.1. The problem is with set CLASSPATH being too long. This problem exists for the R plugin and may exist elsewhere:
C:\Users\aolne_000\beaker-notebook-v1.3.1-0-g3072e5e-windows\config\plugins\eval\r>bin\r
The input line is too long.
The syntax of the command is incorrect.
C:\Users\aolne_000\beaker-notebook-v1.3.1-0-g3072e5e-windows\config\plugins\eval\r>

This bug prevents the R plugin from working on my machine (Win 8.1 Pro x64). The offending line is line 72. I just changed it to:
set CLASSPATH=%APP_HOME%\lib\*

which works because all the jars on the offending line were in this directory anyways.
This change makes the R plugin work for me. It also has the possible advantage of being easier to read/maintain.

Interesting, thanks for the report.  Sounds like a gradle issue as this file is generated by our java build tool.  Looks similar to https://discuss.gradle.org/t/filename-too-long-in-windows/9222
there is a suggested workaround there...

FWIW here is a SO thread that talks about various ways of addressing this:
http://stackoverflow.com/questions/201816/how-to-set-a-long-java-classpath-in-windows

https://issues.gradle.org/browse/GRADLE-2992


[youtube] add support for multiple subtitles with the same lang_code

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[x] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Improvement
[x] New feature

Description of your pull request and other information
As title...
Taken from here: https://github.com/ytdl-org/youtube-dl/pull/26112
Some youtube videos have multiple subtitle for the same language. Right now yt-dl only takes one. This PR solves that.
It probably requires to add more tests and fix some to pass.
this one as an example: https://www.youtube.com/watch?v=wsQiKKfKxug
Before:
[youtube] wsQiKKfKxug: Downloading webpage
Available subtitles for wsQiKKfKxug:
Language formats
en       vtt, ttml, srv3, srv2, srv1, json3

After:
[youtube] wsQiKKfKxug: Downloading webpage
Available subtitles for wsQiKKfKxug:
Language name                           formats
en       English                        vtt, ttml, srv3, srv2, srv1, json3
en       English - French Speakers Only vtt, ttml, srv3, srv2, srv1, json3

Why though? Isn't https://github.com/pukkandan/yt-dlp-dev/commit/0162ae4e76d6278be9dea46ad60b744ecf123afb enough?
$ yt-dlp --list-subs wsQiKKfKxug jc5DlOkOcU4
[youtube] wsQiKKfKxug: Downloading webpage
Available subtitles for wsQiKKfKxug:
Language       formats
en             vtt, ttml, srv3, srv2, srv1, json3
en-1JJtFDtHuZg vtt, ttml, srv3, srv2, srv1, json3

We could also add the name to --list-subs so that user can easily understand which is which. But other than that, I don't see the point of the other changes

in the minimal approach you did the language code en-1JJtf.... is an invalid language code that should be two letter. IMHO.
and I think it creates problem when you embed the subs in the video and the lang code is non-standard.
I think the info about the substitle name is useful especially when embedding the subs because it makes the sub more recognizable
Example in mediainfo and mpc:

For all the code, I just took the code from the linked PR and adapted for the slightly different code of yt-dlp.
It has code for managing the added name to the subtitle, the multiple subs with the same lang code in ffmpeg embedding or converting and I added the fixed tests that failed.

The "language code" is not a standardized language code, but only an id for the subtitle.
    subtitles:      The available subtitles as a dictionary in the format
                    {tag: subformats}. "tag" is usually a language code, and
                    "subformats" is a list sorted from lower to higher
                    preference, each element is a dictionary with the "ext"

"tag" is usually a language code

Many extractors don't write the 2-letter lang code as the tag. This tag is passed through ISO639Utils.short2long when embedding. So as long as the first 2 characters of the tag is the lang-code, the correct language code will be written to the video.

I think the info about the substitle name is useful especially when embedding the subs because it makes the sub more recognizable

I agree, hence why I said "We could also add the name to --list-subs"

not only that... try --write-subs and see what happens with your minimal code. It downloads only one of the subs, not both.
yt-dlp does not support multiple subtitle with the same lang code ATM. the minimal code is a workaround to "list" them but it doesn't do anything else with it.
Hence the extra code written in the PR that fundamentally extends the subtitles handling by yt-dl.
At first, when I saw the code, I too was of the impression that it seemed a little bit excessive and my solution was very similar to yours. But then I realized, while testing it, that everything else besides listing the subtitles simply did't work. Downloading, embedding, converting etc...
Which is why the code is so much, it changes the way subtitles are handled by yt-dl.

try --write-subs and see what happens with your minimal code. It downloads only one of the subs, not both.

That is the intended behavior. It should only write the "best" subs with the default options. If you want both subs, you need to specify so using --sub-langs. In this case, that would be --sub-langs en,en-1JJtFDtHuZg or even easier --sub-langs en,en.+

yt-dlp does not support multiple subtitle with the same lang code ATM

If there are multiple subtitles with the same tag, the assumption is that they are the same subs in different formats

everything else besides listing the subtitles simply did't work. Downloading, embedding, converting etc...

They all work as long as the user requests both subs

it changes the way subtitles are handled by yt-dl.

That is exactly why I am reluctant to merge it

I've added the name as well. https://github.com/pukkandan/yt-dlp-dev/commits/multi-sub
Let me know what you think

I've added the name as well. https://github.com/pukkandan/yt-dlp-dev/commits/multi-sub
Let me know what you think

@nixxo
I was simply asking whether you still think there is any benefit to this more complex approach than my implementations. I did not expect you to force push the entire PR to that branch. But since that is what you did, I assume you think my approach has no disadvantages over this? If so, I can merge those commits separately.
Pushing it to this PR serves no purpose other than simply making it impossible for anyone to follow this conversation in the future. So I have pushed it back to its original state

both solutions solves the problem of recognizing all subtitles available.
To me is indifferent which one is implemented.
The solution in this PR assumed, and I did as well, that the lang_code needed to be a two-letter identifier and so having multiple variants with the same lang code required extra code.
But, has you pointed out, the lang code is just a generic identifier for the languare and so it's not necessary to create "variants" for the same lang code with all the extra code.
Your solution is simpler and cleaner and it will allow to be maintained easily I guess.


It is better change the "python" to "python2"

Arch linux has chosen  python3 as default python interpreter, thus this script do not work at my machine .
I suggest use /usr/bin/env python2 instead  of /usr/bin/env python.

sorry , I find that there are some pull requests have solve the problem....


Add Polars backend

Closes #96

Thanks for this! I'm away from my computer for a little while but I'll give you a review when I get back

Updated - I moved the whole class definition under the try.  This makes pylint/mypy a bit happier + avoids needing all the if not none checks

Benchmark results suggest it may be the fastest impl:

Records
Built-In DataTable
FastDataTable (Arrow from Parquet)
FastDataTable (Arrow from Records)
FastDataTable (Polars from Parquet)

lap_times_100.parquet
0.008s /   1.304s
0.006s /   1.406s
0.007s /   1.441s
0.023s /   1.145s

lap_times_1000.parquet
0.050s /   1.749s
0.007s /   1.933s
0.005s /   1.866s
0.005s /   1.098s

lap_times_10000.parquet
0.339s /   1.985s
0.006s /   1.909s
0.009s /   1.862s
0.005s /   1.107s

lap_times_100000.parquet
3.171s /   4.755s
0.012s /   1.895s
0.053s /   1.932s
0.013s /   1.096s

lap_times_538121.parquet
17.126s /  18.921s
0.025s /   1.933s
0.272s /   2.138s
0.033s /   1.131s

wide_10000.parquet
2.216s /   5.913s
0.009s /   2.362s
0.023s /   2.291s
0.007s /   1.086s

wide_100000.parquet
22.291s /  26.005s
0.023s /   2.300s
0.184s /   2.552s
0.011s /   1.091s

The benchmark results are exciting -- the optional import stuff still needs some work, though.


Extra curly brace in first flutter app codelab

This can be found on https://codelabs.developers.google.com/codelabs/first-flutter-app-pt2/#5
I found the following block of code to have an extra closing curly bracket at the end of the block:
void _pushSaved() {
    Navigator.of(context).push(
      MaterialPageRoute<void>(
        // NEW lines from here...
        builder: (BuildContext context) {
          final tiles = _saved.map(
            (WordPair pair) {
              return ListTile(
                title: Text(
                  pair.asPascalCase,
                  style: _biggerFont,
                ),
              );
            },
          );
          final divided = ListTile.divideTiles(
            context: context,
            tiles: tiles,
          ).toList();

          return Scaffold(
            appBar: AppBar(
              title: Text('Saved Suggestions'),
            ),
            body: ListView(children: divided),
          );
        }, // ...to here.
      ),
    );
  }
}

@sfshaza2 Is this something that belongs in the web site repo?

I am a first timer here. I would like to work on this issue.

It's not an extra closing curly bracket, it's a closing bracket of class _RandomWordsState extends State<RandomWords>.


internal/graph: use a dispatcher without goroutines at concurrency 1

Handled by https://github.com/authzed/spicedb/commit/a3b0cac15a03e6ad1aef325f705589594e54f5c9


meta: update CLIAPIVersion to 8

Update CLIAPIVersion to 8 because of the newly introduced compression enhancement.
Longhorn/longhorn#5376

CLI has been changed due to https://github.com/longhorn/longhorn-engine/commit/8e096311 for backup create.


Development: Update Apollon to v2.12.7

Checklist
General

[x] I chose a title conforming to the naming conventions for pull requests.

Motivation and Context
Update version of Apollon from v2.12.6 to v2.12.7
Description
This bugfix release of Apollon fixes the alignment of elements to grid issue.
As Apollon only supports scaling factors with numbers having one decimal place (0.7, 0.8, 1,...) the scaling factor is now changed from 0.75 to 0.8.
Changes include:

Set the default scale factor of Apollon to 0.8 instead of 0.75
Set the moving/resizing factor of elements to 10 (previous value: 20)
Fix alignment issue with following elements:

Rectangle in syntax tree
Fork nodes
Terminal, Process and Function Call in Flow chart

Steps for Testing

Prerequisites:

1 Instructor

Create a modeling exercise
Observe all the elements (especially the one mentioned above) are aligned perfectly with the grid

Screenshots
Before

After

Tested on TS2. Tested all elements. Most elements work but I encountered a few issues:

Flowchart diagram

Perinet: Transition (same as @timo07k)

Updated the Apollon issue. As mentioned, it will be fixed in a future release of Apollon.


Roadmap

I plan to follow this roadmap to implement new features and improvement, please comment:
Beta 1 - 0.1.0

[ ] - Complete documentation - UiArea
[ ] - Complete documentation - UiGrid
[ ] - Complete documentation - Menus
[ ] - Memory managament

Beta 2 - 0.2.0

[ ] - Implement UiTable
[ ] - Implement various enums
[ ] - Bump nbind to 0.3.0: nullable arguments, enhanced type checking, fixed callbacks arguments

Beta 3 - 0.3.0

[ ] - Precompile binaries with node-pregyp and publish them to github
[ ] - Application packager - probably in its own repo

Two year later, this is completely exceeded by reality, so I close it.


CMakeLists.txt:   Imported target "aws-cpp-sdk-s3" includes non-existent path

I am having a problem to compile including libraries from CMakelist.txt
Error: CMake Error in CMakeLists.txt:
Imported target "aws-cpp-sdk-s3" includes non-existent path
"/opt/iway/deps/aws-sdk-cpp/aws-cpp-sdk-s3/include"
in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

The path was deleted, renamed, or moved to another location.
An install or uninstall procedure did not complete successfully.
The installation package was faulty and references files it does not
provide.

CMakelist.txt: I only show what represents aws
if (DEFINED ENV{AWS_SDK_PATH})
set(AWS_SDK_PATH "$ENV{AWS_SDK_PATH}/build")
else()
set(AWS_SDK_PATH "${CMAKE_CURRENT_SOURCE_DIR}/aws-sdk-cpp/build")
endif()
set(aws-sdk-cpp_DIR "${AWS_SDK_PATH}")
find_package(aws-sdk-cpp)
target_link_libraries( myproyect
aws-cpp-sdk-s3
)

ENV:   AWS_SDK_PATH=/opt/iway/deps/aws-sdk-cpp

Screenshot Path:

that could be happening?

Compiler version, OS version, SDK version, any other relevant information would be helpful.
I'd like to see your CMake command as well.

gcc (Ubuntu 4.8.4-2ubuntu1~14.04) 4.8.4
cmake version 3.1.3
OS centos6 and ubuntu
SDK version: I made git clone today
CMake: I made a script
#!/usr/bin/env bash
cd /opt/iway/deps
LIBS_WE_NEED="core;s3"
if [ ! -d aws-sdk-cpp ]; then
echo "AWS SDK Source: Not found"
git clone --depth 1 https://github.com/aws/aws-sdk-cpp aws-sdk-cpp
else
echo "AWS SDK Source: Found"
fi
pushd aws-sdk-cpp
if [ ! -d build ]; then mkdir build; fi
pushd build
cmake
-DBUILD_ONLY="${LIBS_WE_NEED}"
-DCMAKE_BUILD_TYPE=Release
-DENABLE_TESTING=OFF
-Wno-dev
..
make -j2

can you upload the output of this script?

Sorry I did not understand what you need, if you want the log of the script or the compiled folder that you weapon?

By the doubts I raise the two
https://drive.google.com/file/d/0BxCNpdIwzL9zM2NMdGRacTY1Z2c/view?usp=sharing
log.txt

did you run make install?

nevermind you are doing this via cmake export targets.

Can you upload the generated aws-sdk-cpp exports file?

This is the aws file that was generated
https://drive.google.com/file/d/0BxCNpdIwzL9zM2NMdGRacTY1Z2c/view?usp=sharing
Yes, make install was executed.

There are news of this topic?

This problem should have been fixed, reopen if you still have.


Include .trim() in cli-color directly.

Currently we need require the cli-color/trim to support trim. And it is a little redundant, for me. Why I'll require a feature (trim) inside a major feature (cli-color)? It's like I need to include cli-color/width to get the console width, because cli-color is focused in color.
Currently:
var cli = require("cli-color"),
    trim = require("cli-color/trim");

console.log(trim(cli.blue("hello")));

Suggestion:
var cli = require("cli-color");

console.log(cli.trim(cli.blue("hello")));

Version:

0.4.x Implements this feature;
1.0.x Remove support to cli-color/trim way;

It may make sense, as indeed it's not large utility. I'll schedule it for v1.

Addressed and published with v1.0.0. Still trim was also renamed to strip


[Bug] Console not reporting ANSWER even though memory contains correct statement

Describe the bug
When trying multistep reasoning example from OpenNARS, the system is able to derive the correct conclusion but it is not reported in Console as ANSWER but as regular OUT.
To Reproduce
Steps to reproduce the behavior:
maxeeem@Maxims-Air PyNARS % python3 -m pynars.Console
PyNARS version v0.0.2 pre-alpha
Setup: rand_seed=137
NARS: Init...
NARS: Run...
NARS: Console.
Input: <a --> b>.
 0.80  0.50  0.95  IN    : <a --> b>.  %1.000;0.900%
Input: <b --> c>.
 0.80  0.50  0.95  IN    : <b --> c>.  %1.000;0.900%
Input: <c --> d>.
 0.80  0.50  0.95  IN    : <c --> d>.  %1.000;0.900%
Input: <a --> d>?
 0.90  0.90  1.00  IN    : <a --> d>?
 0.18  0.50  0.71  OUT   : (--, <b-->c>).  %0.000;0.900%
 0.25  0.50  0.74  OUT   : <c --> b>.  %1.000;0.474%
Input: 2000
                   INFO  : Run 2000 cycles.
 0.13  0.50  0.71  OUT   : (--, <c-->d>).  %0.000;0.900%
 0.17  0.50  0.74  OUT   : <d --> c>.  %1.000;0.474%
 0.80  0.80  0.50  OUT   : <d --> a>?
 0.80  0.80  0.50  OUT   : <a --> c>?

 ...

0.96  0.90  0.63  ANSWER: <a --> d>.  %1.000;0.269%

...

 1.00  0.90  0.86  OUT   : <a --> d>.  %1.000;0.729%

Expected behavior
Higher confidence statement should be the best available answer.

@bowen-xu this is related to the conversation here. When writing it as a test case it actually passes since it looks at all of the derived tasks but Console isn't reporting it so somewhere in the control part there is a broken link.
def test_multistep_1(self):
    tasks_derived = process_two_premises(
        '<a --> b>.',
        '<b --> c>.'
    )
    tasks_derived.extend(process_two_premises(
        '<c --> d>.',
        '<a --> d>?',
        200
    ))
    self.assertTrue(
        output_contains(tasks_derived, '<a --> d>. %1.00;0.73%')
    )

@bowen-xu Hm... I just tried ConsolePlus and got the right answer....
Which is the right console to use? Console, ConsoleMC or ConsolePlus? Should we continue maintaining all of them or updating the basic one since there's clearly some inconsistency?

@maxeeem Console is the default one. ConsoleMC was implemented by Tangrui@MoonWalker1997 for MultiChannel, and that is outdated. ConsolePlus was implemented by @ARCJ137442 as an optional one.

@bowen-xu Hm... I just tried ConsolePlus and got the right answer....
Which is the right console to use? Console, ConsoleMC or ConsolePlus? Should we continue maintaining all of them or updating the basic one since there's clearly some inconsistency?
EDIT:
Actually not really consistent results for ConsolePlus. Seems to report both and alternate. Something strange with the control mechanism I think.
after running the first 2000 cycles...
.....

Input: <a --> d>?
 0.90  0.90  1.00 IN    :<a --> d>?
 And if you set compositional_enabled to False, then KanrenEngine also produces a result. However, this alone does not explain why the answer isn't produced with that enabled even thought there is a statement with the correct truth value, or why ConsolePlus can give the right answer in some cases but not in others. I still think the control part requires our attention if the goal is to migrate to the new inference engine.
I would recommend reviewing inference_step code and seeing if there is some issue we can identify.

@bowen-xu I have another theory. Because of the many more derived statements produced by the new engine, there are many more possible combinations and some relevant items may be pushed out of memory since we default to n_memory = 100 in Console.py run_nars method. If you increase it to 1000 for example and run the inference for 11000 steps like in OpenNARS test case, we get the expected result of 73% confidence.
So to me this once again points to a control thing more than the inference engine i.e. how we pick what to pass to the inference engine at every cycle and how we allocate resources.

ConsolePlus was implemented by @ARCJ137442 as an optional one.

As @bowen-xu said, ConsolePlus is an improved version of the default Console written by myself.
You can get more details and enhancements in PR#27.

Which is the right console to use? Console, ConsoleMC or ConsolePlus? Should we continue maintaining all of them or updating the basic one since there's clearly some inconsistency?

It's already an issue #35 which is still open yet.
I'm not sure how these 'Console' implementations will be handled, maybe they will eventually be merged into one.

Actually not really consistent results for ConsolePlus. Seems to report both and alternate. Something strange with the control mechanism I think.

I checked codes in
https://github.com/bowen-xu/PyNARS/blob/6ee80358254ec14764e51c1da95e20136ec9f731/pynars/Console.py#L160-L166
and
https://github.com/bowen-xu/PyNARS/blob/6ee80358254ec14764e51c1da95e20136ec9f731/pynars/ConsolePlus.py#L520-L550
One of the difference in the reasoning effect between the two consoles lies in the default values of the parameters n_memory and capacity when building the inference engine. The default parameters in Console is smaller than ConsolePlus in n_ roomy and ConsolePlus (100 vs 500)
I also checked codes in
https://github.com/bowen-xu/PyNARS/blob/e5b56a511985607bbb23fc09964e276040b8ade3/pynars/Console.py#L124-L133
and
https://github.com/bowen-xu/PyNARS/blob/e5b56a511985607bbb23fc09964e276040b8ade3/pynars/Interface.py#L414-L422
where the NARSOutput in definited as
https://github.com/bowen-xu/PyNARS/blob/e5b56a511985607bbb23fc09964e276040b8ade3/pynars/Interface.py#L43-L59
and used as
https://github.com/bowen-xu/PyNARS/blob/e5b56a511985607bbb23fc09964e276040b8ade3/pynars/Interface.py#L253-L269
https://github.com/bowen-xu/PyNARS/blob/e5b56a511985607bbb23fc09964e276040b8ade3/pynars/Interface.py#L98-L189
(the code here is a little complicated, and the main purpose is to adapt to some terminals that do not support ANSI escape sequences).
I think there is no functional difference between the two Consoles in printing outputs.


Fix up FlowContainer conditional not properly accounting for fit fill…

Fixes Drawings crashing because this is being used in the case of GroupTeam which is auto-sized in the y-axis but the flag is fillmode-fit inside it.

Can you describe your thought process behind how the if statement looks like now, potentially updating the comment above it? I have a hard time grasping what it is supposed to mean with this change.

In the case that a child is relatively-sized on any of the parent's auto-sizing axes:

If the child is fitting with relative size = both

The exception is not triggered. This is as per the comment above.

If the child is fitting with relative size != both, then the "fit" size won't actually be calculated due to https://github.com/ppy/osu-framework/blob/master/osu.Framework/Graphics/Drawable.cs#L689

The exception should be triggered if the following conditions are also met.

If the child is not fitting, same as above.

To be honest I'm not entirely sure what the conditions after this actually prevent, so that's my thought process purely on the fit change.
The other possibility is to remove the relativechildsize checks, which seem wrong in the first place, but I'm not sure if they had a use somewhere.
In-fact this has also led me to realize the AutoSizeAxes check is also in the incorrect place.

@Tom94 I've removed the RelativeChildSize checks from the conditional, and hopefully simplified things a bit. Let me know if you agree with these changes, because as far as I can test this does not result in an autosize feedback loop in the following case:
Add(new FillFlowContainer
{
    Anchor = Anchor.Centre,
    Origin = Anchor.Centre,
    AutoSizeAxes = Axes.Y,
    Width = 60,
    Children = new Drawable[]
    {
        new Box
        {
            RelativeSizeAxes = Axes.Both,
            FillMode = FillMode.Fit,
            Size = new Vector2(1.5f)
        },
        new SpriteText
        {
            Text = "Testing!!!"
        }
    }
});

Can't the child be larger than the parent, even if it has FillMode.Fit if one of its relative axes is larger than Parent.RelativeChildSize? This would cause a feedback-loop, so I believe the checks which you removed are actually necessary.


[Blazor] Improve E2E test infrastructure

The complexity in our infrastructure is going a bit out of hand as we iterate on server-side blazor.
For multiple-component entry points we needed to change the entry-point for the test as server-side components can't be rendered from static HTML no more.
This has complicated the infrastructure a bit more to the point it starts becoming unmanageable
We want to take some time to come up with a better strategy and clean this up, as it impacts our ability to create net E2E tests and maintain existing ones.

One proposal could be to break the test hosts into multiple projects. Right now Components.TestServer is taking all the burden of supporting ServerSide and ClientSide blazor. We could break it into two Components.ClientTesHost, Components.ServerTestHost and have a set of shared base class for startup both inherit from and that contains most of the shared logic, with the exception of the logic to map the app entry point.

That way we get rid of using #server to discriminate the server app.
We simply use one host on the server and a different one on the client.
We don't need any special mechanism to make the apps run.

👍 x a million for the idea of trying to simplify things. I think all of these ideas need some debate and discussion, but I want to make sure we invest in this, and I doubly want to make sure we pick techniques that meet all of our needs.
If possible I'd like to reduce the number of projects down to a really small number. It's possible to use different startup classes to fork on this kind of thing rather than different projects.

Moving this to 3.1 just to get it out of our faces for 3.0. We should still treat this as a priority.


Make pex recognise py310 wheels

pep425.py - version is a string that looks like 38 or 310 (it does not include patch version). Taking just version[1] was forcing py 3.10 to match for 3.1 instead. Including the whole of remaining string as minor version to fix this.
intepreter.py - Not exactly sure where this code is used - perhaps not in our flows since we always specify the interpreter - but fixing the regex to match 2 digit minor versions regardless.

jenkins merge


Documentation is missing a section for CNI

Probably mean moving much of the docs from github/calico-cni into the main docs reference section.  Then add the standard header about where to find user face docs (see github/felix for example) to the calico-cni repo.

#266 and https://github.com/projectcalico/calico-cni/pull/220 and https://github.com/projectcalico/k8s-policy/pull/61 will fix this.


feat(v2): home page for docs

🚀 Feature

When I navigate to https://v2.docusaurus.io/docs, I see empty page, not even a sidebar on the left. It would be great if we have a way to define content for this page e.g. docs/index.md

Refer to https://v2.docusaurus.io/feedback/p/home-page-for-docs
Have you read the Contributing Guidelines on issues?
Yes
Pitch
This can be specified via an option on the docs plugin.

I think we should make a list with the latest 5 versions of docs, so it will be easy to navigate between them, and also write an introduction about the whole doc, and some feature of the last shipped version of the doc too

Not every docs have versioning. People can already built it themselves like how we do it on https://v2.docusaurus.io/versions

oh okay, just let me know what we need to render on this page, and I gonna take this issue and submit the PR asap @yangshun

Maybe instead it should just redirect to the "real" home page https://v2.docusaurus.io/docs/introduction

Hello everybody, what do you guys think about creating a custom page with the currents docs that have in the website instead a dropdown button?

I created this simple design thinking of being the home page for the docs page. for the user to select the version he wants to use. could it be a solution?


Add UTF-8 and UTF-16 specific tests for hosts to verify correctness of transcoding in the transport layer and add code comments do document the behavior

See https://github.com/Azure/azure-sdk-for-cpp/pull/897#discussion_r517808859
Verify that WideCharToMultiByte and MultiByteToWideChar are working as expected.

This would be a data corruption bug if its wrong.  We need to test cases to cover this scenario before GA


Skip styler workflow for PRs coming from forks

Because auto-commit workflow fails otherwise.

No longer relevant (cf. https://github.com/easystats/easystats/issues/357)


Matching-Collaboration-History Integration

Probably going to add a finalise button instead as it is easier than marking question as done when the code passes the test cases.

do we need to mark questions as done? Currently just setting the question as attempted upon any run code

Up to you I guess. I think adding a button to mark as done is better since if you run the code and fail, then you get an attempt but you have not completed the question, so you might want to get it again. But then again you might want them to get a completely unseen problem.


Add Docker in Production to Scalability

High-quality learning resource about techniques to get the most out of Docker and managing applications in production.
Docker in Production - different techniques to get the most out of Docker

Looking at the title, I was expecting some insights on running Docker in production and how to tune it for production workload.
Whereas guides featured on this page are some introductions to running Docker swarm


Occasional social.exceptions:AuthStateMissing

Hi guys,
I get this exception for about 1-3% of daily users. I am not sure what the problem is.
Setup:
django 1.8
google-oauth
Things I tired so far:

Originally I thought it was caused by sessions missing in the database. I deleted all the session to see if I can get a 100% repro. But i could not get it to repro for me.
I traced creation of state in the session table. I can see that the state variable is set before google auth redirection and then it is read back again. It works on my machine consistently. So I can not repro the exception.
I have read all the related bug on AuthStateMissing but none of the solutions worked for me.

It seems that the problem is persistent. I can see the use attempting to login several times and gets this error and then gives up. Few hours later another user attempts several times and gets several of these exceptions. I am not 100% sure that after several errors the user gets though or the user gives up. I bet is that the user gives up.
Here is is the stack trace:
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/newrelic-<IP_ADDRESS>/newrelic/api/web_transaction.py", line 717, in iter
File "/usr/local/lib/python2.7/dist-packages/newrelic-<IP_ADDRESS>/newrelic/api/web_transaction.py", line 1093, in call
File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/wsgi.py", line 189, in call
File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 132, in get_response
File "/usr/local/lib/python2.7/dist-packages/newrelic-<IP_ADDRESS>/newrelic/hooks/framework_django.py", line 499, in wrapper
File "/usr/local/lib/python2.7/dist-packages/django/views/decorators/cache.py", line 57, in _wrapped_view_func
File "/usr/local/lib/python2.7/dist-packages/django/views/decorators/csrf.py", line 58, in wrapped_view
File "/usr/local/lib/python2.7/dist-packages/social/apps/django_app/utils.py", line 51, in wrapper
File "/usr/local/lib/python2.7/dist-packages/social/apps/django_app/views.py", line 28, in complete
File "/usr/local/lib/python2.7/dist-packages/social/actions.py", line 43, in do_complete
File "/usr/local/lib/python2.7/dist-packages/social/backends/base.py", line 41, in complete
File "/usr/local/lib/python2.7/dist-packages/social/utils.py", line 229, in wrapper
File "/usr/local/lib/python2.7/dist-packages/social/backends/oauth.py", line 375, in auth_complete
File "/usr/local/lib/python2.7/dist-packages/social/backends/oauth.py", line 88, in validate_state

Same problem here

Issues #577 #830 #702 may be relevant

An update on this. I never figured out what the problem was. At some point it came up to 1% of daily auths.
But then it went down dramatically on its own. Now it happens very rarely. A few times a week.
I get about 1%  Authentication process canceled  . Maybe it gets masked in those somehow. But for now it is low enough that I am no worried about it.

There was an incorrect check for the state value on error responses hiding the real error happening. Fixed by https://github.com/python-social-auth/social-core/commit/4d2903c1a26cc433731dcc103fd211a7a4716b96


Is there a way to use this to proxy all requests?

That is, something like everything coming into localhost/api goes to myserver/ ?

Yes!
You could use {**catchall}.
app.UseProxy("api/{**catchall}", async (args) => {
    return $"http://myserver/{args["catchall"]}";
});

You could also do it in a controller if you use the ** catchall.  All of the routes use the ASP.NET standard routing, and the catchall is one of the built-in helpers.

@pkellner, this work for you?

With the UseProxy extension the query string parameters are lost by default. I've ended up using something like this:
app.UseProxy("api/{**catchall}", async (context, args) => {
    return $"http://myserver/{args["catchall"]}{content.Request.QueryString.ToUriComponent()}";
});

Is that the best way to do it?
I feel like it would have been better if the query string was appended by default, and maybe there was an overload of getProxiedAddress which had the querystring as a parameter in case anyone wanted to do logic on it.

@waxingsatirical, good catch.
I am also in the midst of a large redesign, and I am adding a RunProxy helper that will catch everything.  So, that will be nice.
The problem with the "catchall" method right now is that it is an ASP.NET route type that I have no control over.  Maybe they should add a ***whatever route that catches the whole path any query string.
An overload of getProxiedAddress is not a bad idea either.  If you are interested, you seem to have dug around enough to help out on my redesign PR.  When I finally get around to posting it, any interest in providing feedback?

Yes, I see that you are pretty much tied to the MapMiddleWareRoute extension and therefore you need to take the ASP.Net template as a parameter. Hmmn.
Happy to have a look, although I would not describe myself as a  "power user". I just use this one method to proxy everything to another domain, tacking on a client certificate for authentication.

@waxingsatirical, cool!  Is that open sourced at all?  Would be cool to have a good example of how you do the client certificate work. :)

Open source! No, we're not very progressive like that.
I'm reluctant to share my implementation, because it's very heavy handed, but essentially you have to provide another implementation of the IHttpClientFactory and add the client certificate in the CreateClient method.
services.AddScoped(IHttpClientFactory, CertificateHttpClientFactory>();

Haha, cool.  Yeah.  That is what I guessed you did.

set http_proxy=http://myproxy
dotnet run

https://docs.microsoft.com/en-us/dotnet/api/system.net.http.httpclient.defaultproxy?view=netcore-3.1


How does one add bounded constraints to ARD parameters for a simple regression?

RBF kernel, two ard parameters. I've searched the docs but nothing seems to work.
I've tried   model['rbf_lengthscale'].constrain_bounded(a,b),  but then get an error saying that there is no lower for a logistic.

Can you share the full error message? The following should work:
 model.rbf.lengthscale.constrain_bounded(a, b) where model is e.g. a GPy.models.GPRegression with ARD kernel.


Should the library decode the response body?

Hello,
I'm currently trying to use your library in one of my projects, so thanks for your work.
I've seen that the lib is not automatically decoding the response body even if Poison is one of the current dependencies. Should it be done eventually by the library?

Hello,
Sorry, I missed the notification!
Thanks for trying
What do you mean by decoding? Transform from a json to an elixir hash? It could be a good pr. If so i will make the change soon


Forge script crashes when two script contracts have the same name.

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (63c71b4 2022-09-12T00:05:14.033477Z)
What command(s) is the bug in?
forge script
Operating System
macOS (Apple Silicon)
Describe the bug
When two 'script contracts' have the same name, and forge script is run for any function in the contract, forge crashes with this error:
➜  contracts git:(script-test) ✗ forge script src/deploy/Deploy.t.sol --sig "deploy()"--private-key <>
[⠰] Compiling...
[⠒] Compiling 1 files with 0.8.10
[⠘] Solc 0.8.10 finished in 1.13s
Compiler run successful
The application panicked (crashed).
Message:  called `Option::unwrap()` on a `None` value
Location: cli/src/cmd/forge/script/executor.rs:36

This is a bug. Consider reporting it at https://github.com/foundry-rs/foundry

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.
[1]    6716 abort      forge script src/deploy/Deploy.t.sol --sig "deploy()" --rpc-url  --private-ke

It is likely that having two contracts with the same name in different files is not ideal (or possibly not valid, they compiled fine). However, if that is the case then forge should detect this and print a friendly error.
Thanks!

are both contracts actually contracts or is one an interface?

The application panicked (crashed). Message:  called Option::unwrap()on aNone` value
Location: /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/revm-1.9.0/src/subroutine.rs:378
This is a bug. Consider reporting it at https://github.com/foundry-rs/foundry
Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.
`
I am writing tests with fork of 2 different chains. Without fork, there is no error but when I use fork, I am getting this error when I include this code:
uint256 id = (uint256(uint160(address(counter))) << 96) | (chainId1 << 80) | (chainId2 << 64) | 0;
The test works fine when I comment this line.

I ran into this today. Basically what I did was clone a script (Deploy.s.sol) and renamed it (to Mint.s.sol), changed the content, but forgot to change the name of the new contract from DeployScript -- leading to the above error and no hint as to why I was seeing it.

Optimistically marking as resolved
Recreating a setup with a duplicate CounterScript now returns:
Error:
Multiple contracts found with the name `CounterScript`

Running against a direct path forge script script/Counter2.s.sol yields no errors


bumblebee: backport critical libkmod.patch

Motivation for this change
Backport change which replaces modprobe execution with libkmod2 library calls. This is a must for anyone who has an issue where Bumblebee can't disable discrete graphics card. Was already backported into Debian
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

So this fixes https://github.com/NixOS/nixpkgs/issues/20009?

@joachifm No it's unrelated, so I created a new issue https://github.com/NixOS/nixpkgs/issues/20047

Seems good to me; I'll test it ASAP and merge if things are okay.

@abbradar Thanks. I'm on 16.09 and use Bumblebee nix module, so tested my change following this guide: http://anderspapitto.com/posts/2015-11-01-nixos-with-local--checkout.html

Sorry, completely forgotten about this! I've tested it and it works okay for me. Thank you for your contribution!


[FEATURE REQUEST] Write the open-im-server/config/openim-push.yml configuration to env

Why this feature?
Convenient modification of push service
Suggested Solution
Read env configuration first, so it won't be lost after restart
Additional Information
No response

Please refer to the following for instructions on modifying the configuration files:
https://docs.openim.io/zh-Hans/guides/gettingStarted/dockerCompose


Reapply "gss: re-enable tests"

This reverts commit e850cbb4f8b236d5c54af6be631783299a4f5ec8.
Follow-up of #335319 to apply this to staging instead of master.
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The commit message doesn't explain at all why this is being re-applied.  Can you put something in there like you did in the PR body?

@alyssais Done


[Typescript] withStyles error

This typescript 2.6 code for material-ui@next compiles with ERROR:
type ClassKey = 'red' | 'blue'
const styles: StyleRulesCallback<ClassKey> = theme => ({
  red: { color: 'red' },
  blue: { color: 'blue' },
})

const app: React.SFC<WithStyles<ClassKey>> = props => <div/>
const App = withStyles(styles)(app)

const element = <App classes={{ red: 'someClass'}} />
==ERROR: Property 'blue' is missing in type '{ red: string; }'==

The solution is to add Partial modifier to WithStyles.classes property in styles\withStyles.d.ts:
export interface WithStyles<ClassKey extends string = string> {
  classes?: Partial<ClassNameMap<ClassKey>>
  theme?: Theme
}

Maybe better solution (inspired by https://github.com/callemall/material-ui/issues/8928) is to cast app props:
const App = withStyles(styles)<{}>(app)

Closing issue...


Support for mb-15 G1

I have an eluktronics MB-15 (very similar to the G2) with an 8291 keyboard but aucc has one of 3 effects. (Either the light turns off, or it returns to the default blue state,or nothing changes.) What can I do to help add support? I have some coding experience but very limited RE abilities

What is the revision of your chip?

Sorry despite looking several times I just noticed it's rev .02

in readme have a link for project starbeat, try it


database query condition (one-off)

https://github.com/pombase/fypo/issues/4104
For this, I would like to know which sessions use the condition
"glucose MM,  proline nitrogen source"

Here are the sessions.  The first column is the number of times "glucose MM, proline nitrogen source" is used in the session.
      8 72f780a0a113152a
      2 64464dca7a13a142
      1 c3bdc1e547b6330a
      1<PHONE_NUMBER>786b20
     10 650374502e2a8566
      1 4847e0de3cb01075
     10 07fb3e57038e972d

Thanks!


protobufc: fix compatibility with new protobuf

Description of changes
Fixes failure caused by bumping protobuf: https://github.com/NixOS/nixpkgs/pull/338885
https://github.com/protobuf-c/protobuf-c/issues/709
protobuf-c/protobuf-c#711
cc @trofi @NickCao
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Fixed protobufc build for me. Thank you!


User roles

Implement roles for users functionality. Now flexible. Users has role field where by default standuper is added with "user" role. New slash command /comedianaddadmin adds user with admin privileges which give access to slash commands.
From the very beginning only super admin (set in env variables) is able to configure Comedian.
This PR also has translation implemented for messages related to user roles.

Coverage decreased (-1.7%) to 70.945% when pulling 9464e5a2050f1a8b7f30193c3c46f9c74bac53e5 on userRoles into 92b61c355dbba102166f39a2688c2e04774297e9 on master.


pngout: 20150319 → 20200115

Motivation for this change
Update pngout. Also include support for additional architectures: aarch64-linux, armv7l-linux, and x86_64-darwin.
Things done

Built on platform(s)

[x] i686-linux
[x] x86_64-linux
[x] aarch64-linux
[x] armv7l-linux
[x] x86_64-darwin
[ ] aarch64-darwin

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

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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 144958 run on x86_64-darwin 1

2 packages built:

image_optim
pngout

Other platforms are coming. The devices I have for them are just really slow.

 Is there a standard? I’ve seen it both ways in the nixpkgs commit history.

Done. Is there a standard?


added fix for issue 959

fix for issue #959

Hi, thanks for this PR. I suggest to divide the test in two, instead of evaluating two different specs in one test. @r-sreesaran


publisher chatflow doesn't check if the domain name is unavailable

before reserving the subdomain, it should check if it is free.
refer to:
https://github.com/threefoldtech/js-sdk/blob/development_capacitypool/jumpscale/packages/tfgrid_solutions/chats/solution_expose.py#L140

verified


Kube DNS error count defintion incorrect?

According to the documentation here, the kube dns error count is the number of dns request errors, however when digging through the kube-dns repo, it seems that the error metric actually being reported is the number of times that kube-dns attempts to read its own metrics and fails.
https://github.com/kubernetes/dns/blob/c85a61ffaa3dfe2d237f2a141d279a08a5237ab6/pkg/sidecar/server.go#L65
Is the documentation here incorrect?

Any update here? I am only pinging this issue because it is being marked stale.

It doesn't seem DataDog monitors this repository. I'd suggest emailing their support to get some action<EMAIL_ADDRESS>

Hi @ctrlbrett thanks for reaching out.
I believe the metric you pointed to is about the dnsmasq subsystem: https://github.com/kubernetes/dns/blob/1.14.10/pkg/sidecar/metrics.go#L83-L85
It's not the counter we're collecting for our kubedns.error_count metric: https://github.com/DataDog/integrations-core/blob/9c354c3e4a85aae763278caf26c5cf55a92a61f9/kube_dns/datadog_checks/kube_dns/kube_dns.py#L23
The HELP line in the prometheus output we look at confirms it:
# HELP skydns_skydns_dns_error_count_total Counter of DNS requests resulting in an error.
# TYPE skydns_skydns_dns_error_count_total counter
skydns_skydns_dns_error_count_total{cause="nxdomain",system="auth"} 25470


remove gentelella (and possibly other) dependencies from our code base

gentelella is embedded in our code-base: /public/gentelella
it should be installed as an external dependency and not be part of our code, should be easy to do using npm or bower: https://github.com/puikinsh/gentelella

will be handled as part of #36 - which will use gentelella


[4.x][Bug] Image field remove button does not work if crop = false

If the image field does not have crop set to true the remove button does not do anything.
Fixed in this pull request: #2197

You're totally right, of course! Thank you for spotting this, and big thanks for putting together the PR!


bumped version to 0.3.1

Totally spectacular changes. Ship it.

Coverage remained the same at 100.0% when pulling b0ef3dbf0d482a146f45d786e073d18449102f39 on bumped-version-to-0.3.1 into d885b1f5e1313bf846416c9b3648d89a1b9968ed on master.


Unfreeze panes in Excel

How can I unfreeze panes in datasets exported as xls files?

Please retest with the newest release and comment if it's still a problem. Thanks!


[CB-11617] (ubuntu) confusion between vertical and horizontal accuracy

Platforms affected
Ubuntu
What does this PR do?
Fixes the accuracy of the information returned,
What testing has been done on this change?
Checked that the values are correctly set on Ubuntu devices
Checklist

[X ] ICLA has been signed and submitted to<EMAIL_ADDRESS>[X ] Reported an issue in the JIRA database
[ X] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
[ ] Added automated test coverage as appropriate for this change.

Cordova CI Build has completed successfully.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 8.1 Store
Link
Link
Link

Windows 10  Store
Link
Link
Link

Windows 8.1 Phone
Link
Link
Link

iOS
Link
Link
Link

Android
Link
Link
Link


[Bug] Set Notification - duplicate menu item

Open Adar and do narrow screen
Look at screen - "Set notifications" is duplicate

Tested on DEV


Fix punctuation mark typo

What's changed, or what was fixed?

Fix a typo of punctuation mark

[ ] This has been reviewed and approved by (NAME)

[ ] I have run npm test locally and all tests pass.

[ ] I have added the appropriate type-something label.

[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

thank you.

:tada: This has been pushed live to https://developers.google.com/web/


Update for 2015-10-16 API Version

Upgrading DSP to 2015-10-16 going to be a pretty massive upgrade and likely require lots of model changes and rethinking as we are currently only on 2012-11-07.
The bulk of the logic is currently contained as methods on models in models.py.  It's already a pretty large module and it's a little overwhelming to think about updating this module with all the updated bits in stripe since November 7, 2012.
All of the models in pinax-stripe are designed to be non-editable. They are merely a cache of the data that is in Stripe. The methods are wrappers around making API calls and updating the cached data (mostly).
I think to make this project easier to maintain and for new developers to reason about, we need to separate the business logic from the storage of the API data.
Some options:

Proxy Models for containing all the logic leaving the models doing the storage without any logic.
Service Layer where we pull out the methods off the models and create functions that perform actions.  This could be something like creating an actions module with functional areas that map to areas of the API as submodules (e.g. actions/subscriptions.py, actions/transfers.py, etc.).

I think these two options are largely the same, with the exception that the Proxy Model will have the data already and the Service Layer you'd need to pass in the required data. I think my preference is the Service Layer right now.
I'd love to hear feedback from the community on ideas for addressing this issue.

@brosner I'd especially love to know your thoughts on this.
@jtauber Your input is something I'd also love to hear if you have anything to add.

One thing I really like about the separation, especially in creation of a service layer, is we can more rapidly maintain 1:1 the model-cache layer with objects in the API as they are just simple bags of data.
Likewise, have functions that serve as the API for views/forms and external projects/apps, can more easily be maintained and constructed and to some degree provide some stability in API changes that might occur going forward.
Furthermore, as Stripe continues to grow, having these modules broken out could ease having distinct areas that different maintainers might surface (e.g. someone using ACH functionality in their project might be better able and have interest in maintaining the actions/ach.py module).

I like the direction. No opposition from me. Defining narrow interfaces is a good plan in my mind. I don't have a particular preference over which direction you go. Proxy models feel more like idomatic Django, but there isn't much value in exposing the underlying data since you said it is treated as a local cache. If I were to approach this problem, I would want to identify the common tasks being performed and wrap them up in an interface. This way the interface implemetation can change based on the underlying Stripe API. Different versions of the interface can crop up to support newer features, but still enabling us to maintain compatibility with older APIs (based on some deprecation policy we can establish).
At this point code is key. We can discuss design until the end of time so it's best to have code we can reason about.

Going to refactor the views to deal with multiple subscriptions and right now just going for something RESTful:
/subscriptions/                       # list subscriptions
/subscriptions/create/                # subscribe to a plan
/subscriptions/(?P<pk>\d+)/delete/    # cancel
/subscriptions/(?P<pk>\d+)/update/    # change plan

/payment-methods/                     # list current payments methods
/payment-methods/create/              # add a new payment method
/payment-methods/(?P<pk>\d+)/delete/  # delete a payment method
/payment-methods/(?P<pk>\d+)/update/  # update a payment method

/invoices/                            # list all invoices (payment history)
/invoices/(?P<pk>\d+)/                # view details of the invoice```

I am certain once we have this working we will want something a little more polished so the the UX is more fluid.

#192 closes this issue


Wolves can be eldritch

When a player tames an eldritch wolf its HP is reduced down to vanilla max.  They shouldn't spawn eldritch at all.

This is an intentional feature. However, the latest update lets you choose which mob can be a special mob by adding or removing the entity's namespace from the eldritch_allowed.json file.


fix(plot): not working with "xaxis=sim_time" option

Description
Fixed a problem with Plot not working correctly with the sim_time option.

Related links

Notes for reviewers

Pre-review checklist for the PR author

[x] I've confirmed the contribution guidelines.
[x] The PR is ready for review.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR has been properly tested.
[ ] The PR has been reviewed.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] (Optional) The PR has been properly tested with CARET_report verification.
[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@rokamu623
With this PR, exception fault issue is fixed, but the result of plot.show(xaxis_type='sim_time') is not correct.
Could you check it please?
I tested with the attached trace data and script, which are created following the tutorial for sim_time.

The original frequency of all topics are 10 Hz
The rosbag is played with -r 0.2
message_flow works as expected

When sim_time=False, the frequency of topics are 2 Hz
When sim_time=True, the frequency of topics are 10 Hz. and beginning time is the same as rosbag

However, create_frequency_timeseries_plot doesn't work as expected

When sim_time=False, the frequency of topics are 2 Hz
When sim_time=True, the frequency of topics are 2 Hz. It's should be 10 Hz !!!

test.zip

@takeshi-iwanari
The bug was caused by calculating the frequency before converting to simtime (same for the latency and the period).
https://github.com/tier4/CARET_analyze/blob/e68cc31932ce07f77d4bb26975b0b839be6a8a2c/src/caret_analyze/plot/timeseries/frequency_timeseries.py#L102
@rokamu623
Can I ask for an implementation fix to address this bug?

Problem solved with conversion to sim_time after Frequency calculation.
Please review once. @atsushi421 @takeshi-iwanari
This has resulted in correct behaviour when sim_time is specified in Plot.create_frequency_timeseries_plot.
Fixes are in progress for period and latency.

@rokamu623
I fixed the source code and updated the ipynb file.
Could you please review and confirm?

test.zip

@tamegaictc2

Commit conflict has occurred.

Please merge main branch into rokamu623:fix/bug_sim_time branch to resolve the conflict.

When you commit, please remember to add the --signoff option.

Please include the results without the xaxis="sim_time" option for better understanding.

@tamegaictc2

Commit conflict has occurred.

Please merge main branch into rokamu623:fix/bug_sim_time branch to resolve the conflict.

When you commit, please remember to add the --signoff option.

Please include the results without the xaxis="sim_time" option for better understanding.

@rokamu623
After making the necessary fixes in the commit.(f7058603810036b1080ed17c2e98753a4d614e64)
I retryed test the ipynb file.
please confirm?

test.zip

@atsushi421
This PR appears to be causing conflict with #291 .
Is it safe to merge it as it is?

@atsushi421 @nabetetsu
As review from me have been completed, please review it.

I was changing the rate of ros2 bag play command to see if it works with same results, but the median period results displayed in the graph created by Plot.create_period_timeseries_plot() appear to be different.
Please give me a few moments to check it out deeply.

@nabetetsu
Problems with changing the rate of ros2 bag play will be treated in another PR.
We would like to close this PR on the condition that it does not cause syntax errors.


Add post hook on site generation

I would find it useful to have a post hook after the site is generated, especially with the watch functionality where the site is generated on every change (which is very useful!).  for example, a script could open or refresh a browser window or call sync files, etc.

@spf13 you said back in 2013 that you would "take a stab at it in a couple days". Now it is more than a few days later; what is the status?

This issue has been automatically marked as stale because it has not been commented on for at least four months.
The resources of the Hugo team are limited, and so we are asking for your help.
If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still valuable, please open a proposal at https://discuss.gohugo.io/.
This issue will automatically be closed in four months if no further activity occurs. Thank you for all your contributions.

Note/Update: This issue is marked as stale, and I may have said something earlier about "opening a thread on the discussion forum". Please don't.
If this is a bug and you can still reproduce this error on the latest release or the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.

I would love to have this feature for a unique usecase of mine - along with pre-build hooks too.
Has there been any interest in this lately, or would you still be open to it, @bep and/or @spf13 ?


Event handler messes up  rendering

Version
3.0.0-beta.14
Reproduction link
https://github.com/sneppy/vue-table-bug.git
Steps to reproduce
Start dev server and navigate to localhost:8080
What is expected?
Table is rendered correctly and clicking the first row triggers a 'clicked' message in the console window:
(1, 1)	(1, 2)	(1, 3)
(2, 1)	(2, 2)	(2, 3)
(3, 1)	(3, 2)	(3, 3)

What is actually happening?
Click event works, but table is not rendered correctly:
(1, 1)	(1, 2)	(1, 3)
(2, 1)(2, 2)(2, 3)(3, 1)(3, 2)(3, 3)

Removing the '@click` event handler seems to solve the rendering issue.

https://github.com/vuejs/vue-next/pull/1230


Fix GH action to trigger sync job in ODH

This GH action  failed to run during the last release of the CF stack. The action had to be run manually.
This needs to be fixed

cc @Bobbins228


Paging and Update

Hi,
i really liked this library on top of dapper among other options.
I have seen mapping instruction which allows to update specific properties. is it possible to update properties from anonymous object or provide a list of properties at runtime?
.Update({Id= 1, Name= "ABC"});
.Update(user, statement => statement.onlyfields(u => u.Name));
this approach will be simple as OrmLite approach but that library is no more free to use.
For Paging, i have not seen any helper  .Paging(1, 10, sql)
I have a plan to use this library but my project update same entity in different ways and having custom mappings for every case is not a case so is it possible to add this as feature in future if it makes sense?

I don't think you'll be seeing partial updates directly in the Update method. The idea behind default and custom entity mappings is that the SQL can be created once and reused in subsequent operations, reason why the overhead introduced by this library versus Dapper is minimal.
For paging, you've got Top and Skip. See the wiki for some examples.

dbConnection.Find(statement => statement
.Where($"{nameof(Person.FirstName):C}=@FirstNameParam")
.OrderBy($"{nameof(Person.LastName):C} DESC")
.WithParameters(new {FirstNameParam: "John"}));
there is no .Select method to select particular columns?

WithEntityMappingOverride is what you're looking for. See this topic in the wiki.


Google connect does not meet Google branding style and will be rejected on API request

Relates to "Google connect: use "google" icon instead of "google plus" icon #2343"
Just to re-iterate, the Google connect module is still broken and does not meet Google branding so new users will find they can not get authorization for it. This is the email Google sent me:
Dear Developer,
Thank you for submitting an OAuth App Verification request. To proceed with the verification process, please update your project queersociallogin-1xx7396x18x7 to comply with our requirements.
Please update the Google sign-in button on your project’s homepage URL so that it complies with the Google sign-in branding guidelines. Once your project is updated to reflect these requirements, please reply to this email to confirm your compliance. If you have additional questions, you can review the OAuth Application Verification FAQ or respond to this email.

Thank you, but there is already an open ticket for this:
https://github.com/unaio/una/issues/2347


⚠ Bofrost.de has degraded performance

In f08ab94, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1740 ms

Resolved: Bofrost.de performance has improved in 8427295 after 19 minutes.


help with relative urls rebasing

I'm trying to set a correct relative url path using gulp-minify-css 1.2.1 with clean-css 3.4.6
The initial file is located in:
assets/js/vendor/fancybox/jquery.fancybox.css
Here's one of the url:
background-image: url('fancybox_loading@2x.gif');
I'm currently using these parameters:
{
    relativeTo: path.join(process.cwd(), "css"),
    root: path.join(process.cwd(), "img", "fancybox")
}

The final css is stored in:
css/style.min.css
The images are stored in:
img/fancybox
I would like that url to become:
background-image: url('../img/fancybox/fancybox_loading@2x.gif');
Is this possible by using the relativeTo and target parameters?

You should be able to achieve that by using root: path.join(process.cwd(), "css") too. Please let me know if it works.

Just tried, the resulting url is:
background-image:url(/fancybox_loading@2x.gif);

Ah, those don't share a common root - please give target a try instead, see https://github.com/jakubpawlowicz/clean-css#how-to-rebase-relative-image-urls

I've set the parameters to:
target: path.join(process.cwd(), "img", "fancybox"),
relativeTo: path.join(process.cwd(), "css"),

The result is:
<EMAIL_ADDRESS>
Is there a way to force a base path to all the urls in a file?

There's no way to do that. I see that problem from time to time when it comes to plugins.
It could be tricky if you are moving files during build process - is<EMAIL_ADDRESS>in the same folder as your input CSS file and is it not after the build?

Yes, that's the case.

@maiolica I think you have the settings swapped:
new CleanCSS({
  relativeTo: path.join(process.cwd(), "img", "fancybox"),
  target: path.join(process.cwd(), "css")
}).minify("background-image: url('fancybox_loading@2x.gif');").styles;

"background-image: url(../img/fancybox/fancybox_loading@2x.gif);"

Please also see discussion in https://github.com/jakubpawlowicz/clean-css/issues/854#issuecomment-274776447.
Note that 4.x has only rebaseTo option which is similar to target in 3.x.
I'm closing this issue now due to inactivity.
Feel free to comment it further if needed.


Dropdown will close when the search input is focused

Bug Report
Steps

Expected Result
Dropdown should open.
Actual Result
Dropdown closed.
Version
0.88.2
Testcase

https://codesandbox.io/s/vj3f6?module=/example.js
https://react.semantic-ui.com/modules/dropdown/#types-search-in-menu

@lc-soft please update to the latest version. I think this has been resolved there. If still present, reply back and we can reopen.

@lc-soft please update to the latest version. I think this has been resolved there. If still present, reply back and we can reopen.

It is not resolved, have you followed the steps of gif animation in the Dropdown document page to reproduce this bug?
https://react.semantic-ui.com/modules/dropdown/#types-search-in-menu
This bug can be reproduced on Chrome and FireFox browsers, I suggest you upload a gif animation to prove that this bug has been fixed.
Environment:
OS: Ubuntu 18.04.3 LTS
Chrome 78.0.3904.108
FireFox 76.0.1

Thanks for the detailed description and filling the issue. This is a duplicate https://github.com/Semantic-Org/Semantic-UI-React/issues/1593#issuecomment-295010142

To solve it please use .stopPropagation() on Input: https://codesandbox.io/s/thirsty-https-bneim?file=/example.js
<Input
  icon="search"
  iconPosition="left"
  className="search"
  onClick={e => e.stopPropagation()}
/>

@brianespinosa It is not resolved, have you followed the steps of gif animation in the Dropdown document page to reproduce this bug?

@lc-soft nope. We get a LOT of reports on this repository. If we see someone reports an issue from an old version, it is common we will ask for testing again with the latest version first. In the limited time some of us dedicate to open source work, we don't have time to chase down issues on old versions. If it turns out something is still present on the current version, we are always ready to reopen a reported issue, as noted above.


[19.07] let passwd using busybox to avoid failure when shadow-utils is installed

If the package shadow-utils is installed, passwd will not be provided by busybox. In this case, changing the password on the web page will fail.
The PR fixed this problem by calling the busybox passwd subcommand.
This has been successfully tested on the openwrt-19.07 branch. In theory, this also applies to the master branch.

Would be interesting to know why it fails. Is (echo secret; sleep 1; echo secret) | passwd not working with shadow-passwd? What's the error when running this command?

Would be interesting to know why it fails. Is (echo secret; sleep 1; echo secret) | passwd not working with shadow-passwd? What's the error when running this command?

@jow- As far as I know, shadow-passwd receive information from stdin.
(But only some old version of passwd can. For example CentOS 6, passwd --stdin can receive passwords from stdin)

@jow- As far as I know, shadow-passwd receive information from stdin.
(But only some old version of passwd can. For example CentOS 6, passwd --stdin can receive passwords from stdin)

I'm also hitting this problem now with current master. And busybox passwd helps.
Is there a problem merging this PR ?

@egorenar
On master, there are some problems on shadow's hash algorithm. See comments of this commit https://github.com/openwrt/packages/commit/adf9c249498fe526e2d6afd19bf7985f116c8531.


refactor(e2e): extract list of tokens to be looped

Motivation
We are going to create more tests based on what happens in each token page, or for each token that we would test. That is why we extract a re-usable token list for the tests.
Note
We don't use directly the list of tokens constants, but their symbol. That is because in the tokens we are importing the SVG icons that are not correctly recognized by playwright (see workflow). This is a know issue with playwright apparently.

apparently the issue with importing static assets with playwright is more profound, and not hackable in this PR unfortunately. For now, we keep hard-coding all the tokens we want to test, and not loop them


Enable SyncBatchNorm For LightningLite

What does this PR do?

Fixes https://github.com/Lightning-AI/lightning/issues/13749
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

LGTM ! Mind adding a test ?

Sure. I was stuck by something else in the past weeks, and I will add it soon

@awaelchli, mind checking this one? :otter:

@Borda I replied already on the linked issue https://github.com/Lightning-AI/lightning/issues/13749#issuecomment-1250146805

@lsy643 Thanks for your contribution. This is blocked by other larger pieces of work so we should revisit this after we can do it. Closing for now, let's continue discussing in the linked issue.


Type not match in generated code

rust 1.62.0
flutter 3.0.2
ffigen 5.0.1

What is that "Dir"? please share the code.
In addition, it would be great to add a reproducible test case at https://github.com/fzyzcjy/flutter_rust_bridge/blob/master/frb_example/pure_dart/rust/src/api.rs and https://github.com/fzyzcjy/flutter_rust_bridge/blob/master/frb_example/pure_dart/dart/lib/main.dart. In other words, add a few lines there, and make a PR, and the PR's CI should fail and show those errors.

I'm trying to make reproducible test case, but the codegen doesn't generate the *.freezed.dart file.

FYI
flutter_rust_bridge_codegen --rust-input rust/src/api.rs --dart-output ./dart/bridge.dart

api.rs
#[derive(Debug)]
pub enum LinkType {
    File,
    Dir { include_all: bool },
    Http,
    Git,
}

#[derive(Debug)]
pub struct Link {
    pub inner: String,
    pub kind: Box<LinkType>,
}

pub fn print_link(link: Link) -> anyhow::Result<()> {
    println!("{:#?}", link);
    Ok(())
}

but the codegen doesn't generate the *.freezed.dart file.

Have you add dependency in yaml

You forgot dev_dependencies

Hmm please show full code, probably just fork this repo, add code in api.rs and .dart, and make a PR. Then we can discuss in the PR and look at CI etc.
No worries to PR before fully implemented a feature - just mark PR as draft

https://github.com/fzyzcjy/flutter_rust_bridge/pull/550


starship: fix fish integration syntax

The logic is fine, but the syntax is not valid fish and must have been
untested. I've tested the fixed version, and it works as expected.
Description
The current fish integration for starship uses parts of POSIX-esque test syntax. fish does not use this syntax. It also uses -n instead of -z to check for an unset variable. This patch fixes these issues.
Testing these integrations would be difficult, as it would probably require setting up a shell and monitoring the output somehow.
Checklist

[x] Change is backwards compatible.

[x] Code formatted with ./format.

[x] Code tested through nix-shell --pure tests -A run.all.

[ ] Test cases updated/added.

[x] Commit messages are formatted.

This PR doesn't initiate a module.

Thanks! Rebased to master in dbf6b2d2abe0b9966d657f301ea32e4d5364b030.


Update LLVM to llvm/llvm-project@b13592219c421820b

Update LLVM to llvm/llvm-project@b13592219c421820b (https://github.com/llvm/llvm-project/pull/85376) Changes done to resolve mlirbc issue in https://github.com/iree-org/iree/issues/19498 through updating/regen of input IRs in azure and SHARK-TestSuite to work with latest mlir-opt.
This PR also carries the following reverts:
https://github.com/llvm/llvm-project/pull/120115
The main issue with this PR is it breaks matvec codegen generating scf.if instead of scf.for(s). An issue will be pushed up for repro.

@ScottTodd In order to drop the reverts and not keep it for too long, I updated the mlirbc of open-llama in SHARK-TestSuite and kept in in a private branch
 iree-opt open-llama-3b-v2-f16.mlirbc -o open-llama-3b-v2-f16.mlir
rm open-llama-3b-v2-f16.mlirbc
iree-opt open-llama-3b-v2-f16.mlir --emit-bytecode -o open-llama-3b-v2-f16.mlirbc

LMK what you think!

@raikonenfnu let's revert the tile and fuse change and land it.
@Groverkss you can add your changes and undo the revert