SELECT * FROM metrics should return metric names, descriptions, etc.

@betodealmeida would you be able to add a bit of detail on this one? Even when selecting explicitly all metrics I get an error like Metrics ... have non-shared parents which I think makes sense to me but then this issue would not make sense

Yeah, this one definitely needs more context. I added it here so I wouldn't forget.
The SQL API allows you to fetch data from metrics by running queries like:
SELECT
    "basic.num_comments"            -- a metric node
  , "basic.dimension.users.gender"  -- a column from a dimension node
FROM metrics                        -- a virtual table that encapsulates all metrics
WHERE
    "basic.dimension.users.country" != 'BR'
GROUP BY
     basic.dimension.users.gender

The query will be translated into SQL that actually be executed. It happens in this function, and will produce SQL that looks like:
SELECT
    COUNT(*) AS "basic.num_comments"
  , gender AS "basic.dimension.users.gender"
FROM
    comments
JOIN
    users
ON
    comments.user_id = users.id
...

It would be nice to expand the abstraction of the metrics table, and allow queries like:
SELECT name, description FROM metrics

But I think it requires some discussion first of how we want those queries to look like. Additionally, it would be nice to improve the SQL API so that it can also run the SQL used to define nodes. For example, the node basic.num_comments has this SQL:
SELECT SUM(user_id) FROM basic.source.comments

Currently, we can't send that SQL to the SQL API, since today it can only be used to query the metrics virtual table. It would be nice to have that functionality, since eventually we could have a web UI where people can run the SQL for their nodes as they're developing them.

@betodealmeida thinking about SELECT name, description FROM metrics it seems like it could be confusing. Do you think there should be a differentiation like metrics.metadata? (credit to @samredai for this convention which makes a lot of sense to me). Either that or I suppose allow queries directly against the actual table so something like select name, description from node where type='METRIC'

Yeah, I think there are 3 separate cases for the SQL API:

The original one, returning data for an arbitrary expression referencing metric nodes, that I implemented for Apache Superset.

SELECT "metric_node", "dimension_node.column"
FROM metrics
GROUP BY "dimension_node.column"

Returning metadata about nodes, metrics, databases, etc.

SELECT name, description FROM XXX

Evaluating arbitrary SQL from a node, to allow users to interactively test their nodes while authoring them:

SELECT COUNT(*) FROM some_node WHERE condition

i think the challenge here is making all these 3 uses cases unambiguous by introducing some kind of namespace for the FROM clause. Eg:

SELECT metric_node FROM dj.metrics
SELECT name, description FROM dj.metadata
SELECT COUNT(*) FROM some_node

For something like this we would need to prevent nodes from being created in a dj directory, to keep the cases distinct.
@agorajek @samredai thoughts here?

Agreed that we should think about how to make these different behaviors unambiguous because I think there are even more cases (or sub-cases?) to consider. For me, it's a bit hard to organize them all in my head. For example:
Joining nodes that share a common set of dimensions?
SELECT "metric_node_a", "metric_node_b", "dimension_node.common_column"
FROM metrics
GROUP BY "dimension_node.common_column"

Getting the history of a node that's been changed?
SELECT name, description, version FROM dj.metadata

Getting the raw SQL string from a transform node?
SELECT query FROM transform_node

I really like the idea of using namespacing to make this more intuitive and clear and @CircArgs also mentioned the idea of having a dj namespace which feels right and is probably the most flexible. Would everyone else find it helpful to have a docker compose setup that includes a superset container so we can play around with building visual components from these queries?

Joining nodes that share a common set of dimensions?
SELECT "metric_node_a", "metric_node_b", "dimension_node.common_column"
FROM metrics
GROUP BY "dimension_node.common_column"

This is possible today if metric_node_a and metric_node_b have the same parents.

Getting the history of a node that's been changed?
SELECT name, description, version FROM dj.metadata

I like this one... we might want to introduce versioning early rather than later, since it's easier. This is another one of those cases where we might be over-engineering, but it's much easier to do it early.

Getting the raw SQL string from a transform node?
SELECT query FROM transform_node

This one doesn't make sense from the current API perspective (or my proposal). If we're not using the dj namespace then the query is treated as a data query, so the query above would only make sense if transform_node has a column called query, and it would return its data.
We could do something similar, like:
SELECT query FROM dj.metadata WHERE name = 'transform_node'

I really like the idea of using namespacing to make this more intuitive and clear and @CircArgs also mentioned the idea of having a dj namespace which feels right and is probably the most flexible. Would everyone else find it helpful to have a docker compose setup that includes a superset container so we can play around with building visual components from these queries?

Let me know if you need help connecting Superset to DJ!

I think it may be easier to reason about all these metadata queries if we simply agree on the data model of these DJ metadata tables. I think making dj namespace restricted is a good start. Think of it like the catalog schema of any famous database, say Postgres, Oracle etc.
Next let's have a look at some possible tables in there:
-- databases
dj.database (
  description STRING,
  uri URI,
  read_only BOOL,
  cost INT
)

-- all node
dj.nodes (
  description STRING,
  type STRING fk to dj.node_types.name, -- this maybe goes too far but to show the idea
  name STRING,
  query STRING,
  columns ARRAY<STRING>,
  attributes MAP<>,
  downstream_nodes ARRAY<STR>, -- list of all downstream nodes
  upstream_nodes ARRAY<STR>, -- list of all upstream nodes
)

-- dj.metric_nodes (similar to node, but specifically for metric type)
-- dj.transform_nodes (etc...)
-- dj.source_nodes (this one could be offering all the info we know about through the SQL interface, wow... this is so cool!)

With the above data model, there is no ambiguity what your queries should do:

If you query thing from the dj.* only, you should be getting results only from the DJ metadata space. I don't think we need an extra dj.metadata table btw, unless we come up with some special fun use for it. Example:

SELECT description, name FROM dj.nodes
-- or from dj.metric_nodes, dj.transform_nodes, or dj.source_nodes

When you run query on the "other" objects in the DJ database, then you should be getting the user data, say:

SELECT * FROM basic.dimension.users
-- should return all the data from that dimension

We could also offer some "hidden columns" on all the user nodes. Example:

SELECT __name__, __type__, __query__, user_id FROM basic.dimension.users
-- which would repeat the same values on all the __*__ columns cause they are all the same

There is definitely also a place for combining queries on the dj metadata with the queries on the user data, but I don't have a good example in mind yet.


Bedrock is not properly reporting exit code 1 on test failure

https://github.com/digitalbazaar/bedrock/compare/evergreen-tests?expand=1
Successful CI action reported there, but there was indeed a test failure shown here:
https://github.com/digitalbazaar/bedrock/runs/4451334012?check_suite_focus=true
Inspect the 'Run test with Node.js 14.x' section.
This was evidently broken by the recent signal handlers patch released in v4.4.2:
https://github.com/digitalbazaar/bedrock/blob/main/CHANGELOG.md#442---2021-11-04
If the bad test is applied to v4.4.1 return code of 1 is properly returned and CI properly reflects a failure:
https://github.com/digitalbazaar/bedrock/compare/4.4.1-bad-test?expand=1
The improper 0 return code can also be seen by doing.  A 1 is expected when the test suite fails.
npm test
echo $?

Addressed by #95.


LOH fragmentation when user post form with section content more than 85KB

Post a multipart form with content more than 85KB in each section would cause heavy LOH fragmentation in MVC app.
Post a multipart form with 100 sections, each containing random sized content with an average of 120KB.
After the first request:
_         segment             begin         allocated              size
0000021c56c50000  0000021c56c51000  0000021c582fa2a0  0x16a92a0(23761568)
Total Size:              Size: 0x1f0bcb8 (32554168) bytes.
Statistics:
              MT    Count    TotalSize Class Name
0000021c46a5d8e0      **107         3204      **Free
00007ffbbe854418        9       146136 System.Object[]
00007ffbbe84e210       16      2097536 System.Byte[]
00007ffbbe862498      ** 81     21514154 **System.String
_

After the 2nd request:
_Large object heap starts at 0x0000021c56c51000
         segment             begin         allocated              size
0000021c56c50000  0000021c56c51000  0000021c5983f700  0x2bee700(46065408)
Statistics:
              MT    Count    TotalSize Class Name
0000021c46a5d8e0      **194         5814      **Free
00007ffbbe854418        9       146136 System.Object[]
00007ffbbe84e210       22      2884112 System.Byte[]
00007ffbbe862498      **162     43028308 **System.String
_

After the 3rd request:
_Large object heap starts at 0x0000021c56c51000
         segment             begin         allocated              size
0000021c56c50000  0000021c56c51000  0000021c5983f700  0x2bee700(46065408)
Total Size:              Size: 0x362f488 (56816776) bytes.
Statistics:
              MT    Count    TotalSize Class Name
0000021c46a5d8e0     ** 276         8274      **Free
00007ffbbe854418        9       146136 System.Object[]
00007ffbbe84e210       23      3015208 System.Byte[]
00007ffbbe862498     **<PHONE_NUMBER>2 **System.String_

So for each request we are adding the large string in LOH for each section (field) of the form. Also, look at the "Free" type. They are mostly 30 bytes in LOH accompany each large string. It may be some string interning.
For the string object, basically we are reading each section (field) into a string:
_                        else if (HasFormDataContentDisposition(contentDisposition))
                        {
                            // Content-Disposition: form-data; name="key"
                            //
                            // value

                            var key = HeaderUtilities.RemoveQuotes(contentDisposition.Name);
                            MediaTypeHeaderValue mediaType;
                            MediaTypeHeaderValue.TryParse(section.ContentType, out mediaType);
                            var encoding = FilterEncoding(mediaType?.Encoding);
                            using (var reader = new StreamReader(section.Body, encoding, detectEncodingFromByteOrderMarks: true, bufferSize: 1024, leaveOpen: true))
                            {
                                **var value = await reader.ReadToEndAsync**();
                                formAccumulator.Append(key, value);
                            }
                        }_

This can be easily explored to cause the server to run OOM. Security concern as well.

@SajayAntony @sivagms @halter73 @Tratcher @blowdart

@yuwaMSFT Does the LOH fragmentation cause OOM or crash?

Related: https://github.com/aspnet/HttpAbstractions/issues/583

It causes the memory to accumulate. At some point it will cause OOM (and the app would crash on exception of memory allocation or killed by OOM killer before that happens).
The LOH looks like below:
0000021c5a90d830 0000021c46a5d8e0       30 Free
0000021c5a90d850 00007ffbbe862498   353606
0000021c5a963d98 0000021c46a5d8e0       30 Free
0000021c5a963db8 00007ffbbe862498   289318
0000021c5a9aa7e0 0000021c46a5d8e0       30 Free
0000021c5a9aa800 00007ffbbe862498   474146
0000021c5aa1e428 0000021c46a5d8e0       30 Free
0000021c5aa1e448 00007ffbbe862498   345570
0000021c5aa72a30 0000021c46a5d8e0       30 Free
0000021c5aa72a50 00007ffbbe862498   297354
0000021c5aabb3e0 0000021c46a5d8e0       30 Free
0000021c5aabb400 00007ffbbe862498   208958
0000021c5aaee440 0000021c46a5d8e0       30 Free
0000021c5aaee460 00007ffbbe862498   112526
0000021c5ab09bf0 0000021c46a5d8e0       30 Free
0000021c5ab09c10 00007ffbbe862498   305390
0000021c5ab54500 0000021c46a5d8e0       30 Free
0000021c5ab54520 00007ffbbe862498   249138
0000021c5ab91258 0000021c46a5d8e0       30 Free
0000021c5ab91278 00007ffbbe862498   241102
0000021c5abcc048 0000021c46a5d8e0       30 Free
0000021c5abcc068 00007ffbbe862498   305390
0000021c5ac16958 0000021c46a5d8e0       30 Free
0000021c5ac16978 00007ffbbe862498   216994
0000021c5ac4b920 0000021c46a5d8e0       30 Free
0000021c5ac4b940 00007ffbbe862498   233066
0000021c5ac847b0 0000021c46a5d8e0       30 Free
0000021c5ac847d0 00007ffbbe862498   136634
0000021c5aca5d90 0000021c46a5d8e0       30 Free
0000021c5aca5db0 00007ffbbe862498   257174

Related to #583 but different issue. This is easier to be explored as a security concern.

Ouchie :(

@blowdart need to fix this?

Yes. If it's spiking memory so badly the app pool will recycle then it's a DoS

@yuwaMSFT Do you know if this is still an issue?

I checked the latest code. It seems to still have the same issue for multipart form data content.
We seems to only apply KeyLengthLimit and ValueLengthLimit (which is 4MB anyway) to urlencodedform but not multipart-form. The only guard for multipart-form is the MultipartBodyLengthLimit which is by default 128MB. So I believe the same issue still exists.


scss float: left on .feature__item--right .archive__item causes problem with embedded svg

What happened?

Minimal Mistakes version: 4.24.0
Ruby gem or remote theme version: 4.24.0
Jekyll version: 3.9.2
Git repository URL: https://github.com/sums-sta/sums
Hosted on GitHub Pages (if yes provide URL to site): https://sums-sta.github.io/sums
Operating system: Linux

Expected behavior
I embedded a svg using feature_row aligned to the right (specifically {% include feature_row id="signup_row" type="right" %}), but the svg is not properly aligned (see attached screenshot).

Using Firefox's devtools, I noticed that in the <div class="archive__item"> html element, there is a float:left in .feature__item--right .archive__item. Turning it off fixes the problem (see attached screenshot) and brings about the expected behavior.

Digging through some scss, I found out that it is controlled by line 346 in /_sass/minimal-mistakes/_archive.scss.
https://github.com/mmistakes/minimal-mistakes/blob/8a67ce8e41ec850f2d7c373aa47739b2abfee6f1/_sass/minimal-mistakes/_archive.scss#L346-L348
Removing these lines fixes the problem.
Aligning feature_row to the left also does not work properly. This seems to be controlled by line 305 in /_sass/minimal-mistakes/_archive.scss.
https://github.com/mmistakes/minimal-mistakes/blob/8a67ce8e41ec850f2d7c373aa47739b2abfee6f1/_sass/minimal-mistakes/_archive.scss#L305-L307
I also noticed that this is part of the fix for Algolia search link positioning (#1904), but I am not able to verify if this is actually needed for the fix because I don't really know how to use Algolia search. If anyone else could help me verify this, I would really appreciate it.
Steps to reproduce the behavior
This is a styling issue so there's not many steps to reproduce except just runing bundle exec jekyll serve.
Log:
$ bundle exec jekyll build --trace --verbose
To use retry middleware with Faraday v2.0+, install `faraday-retry` gem
  Logging at level: debug
Configuration file: /home/chend/sums-website2/_config.yml
      GitHub Pages: github-pages v227
      GitHub Pages: jekyll v3.9.2
             Theme: jekyll-theme-primer
      Theme source: /home/chend/.local/share/gem/ruby/3.0.0/gems/jekyll-theme-primer-0.6.0
         Requiring: jekyll-github-metadata
Dotenv not found. Skipping
         Requiring: jekyll-seo-tag
         Requiring: jekyll-paginate
         Requiring: jekyll-sitemap
         Requiring: jekyll-gist
         Requiring: jekyll-feed
         Requiring: jemoji
         Requiring: jekyll-include-cache
         Requiring: jekyll-redirect-from
         Requiring: jekyll-coffeescript
         Requiring: jekyll-commonmark-ghpages
         Requiring: jekyll-github-metadata
         Requiring: jekyll-relative-links
         Requiring: jekyll-optional-front-matter
         Requiring: jekyll-readme-index
         Requiring: jekyll-default-layout
         Requiring: jekyll-titles-from-headings
         Requiring: jekyll-remote-theme
   GitHub Metadata: Initializing...
            Source: /home/chend/sums-website2
       Destination: /home/chend/sums-website2/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
             Theme<EMAIL_ADDRESS>      Theme source: /tmp/jekyll-remote-theme-20221229-5535-fhe147
      Remote Theme: Using theme mmistakes/minimal-mistakes
      Remote Theme: Downloading https://codeload.github.com/mmistakes/minimal-mistakes/zip/4.24.0 to /tmp/jekyll-remote-theme-20221229-5535-7b11nb.zip
      Remote Theme: Unzipping /tmp/jekyll-remote-theme-20221229-5535-7b11nb.zip to /tmp/jekyll-remote-theme-20221229-5535-fhe147
       EntryFilter: excluded /Gemfile
       EntryFilter: excluded /Gemfile.lock
           Reading: _posts/2021-01-01-new-year.md
           Reading: _posts/2022-12-28-progress.md
        Generating: JekyllRedirectFrom::Generator finished in 0.003327903 seconds.
        Generating: JekyllOptionalFrontMatter::Generator finished in 0.000113721 seconds.
        Generating: JekyllReadmeIndex::Generator finished in 8.3365e-05 seconds.
        Generating: Jekyll::Paginate::Pagination finished in 0.000259797 seconds.
        Generating: Jekyll::JekyllSitemap finished in 0.000296609 seconds.
       Jekyll Feed: Generating feed for posts
        Generating: JekyllFeed::Generator finished in 0.000557599 seconds.
        Generating: JekyllRelativeLinks::Generator finished in 0.000109122 seconds.
        Generating: JekyllDefaultLayout::Generator finished in 0.000325328 seconds.
        Generating: JekyllTitlesFromHeadings::Generator finished in 2.1511e-05 seconds.
         Rendering: _posts/2021-01-01-new-year.md
  Pre-Render Hooks: _posts/2021-01-01-new-year.md
  Rendering Markup: _posts/2021-01-01-new-year.md
         Requiring: kramdown-parser-gfm
  Rendering Layout: _posts/2021-01-01-new-year.md
     Layout source: theme
         Rendering: _posts/2021-01-01-new-year.md/#excerpt
  Pre-Render Hooks: _posts/2021-01-01-new-year.md/#excerpt
  Rendering Markup: _posts/2021-01-01-new-year.md/#excerpt
   GitHub Metadata: Generating for SUMS-StA/sums
   GitHub Metadata: No GitHub API authentication could be found. Some fields may be missing or have incorrect data.

 Auto-regeneration: disabled. Use --watch to enable.
      Remote Theme: Cleaning up /tmp/jekyll-remote-theme-20221229-5535-fhe147

Other
No response

Most likely a copy-paste error in early years. Now fixed as shown above.


Updates: https://github.com/swaywm/wlroots/issues/1640 (Atomic output updates issue)
See also: https://github.com/swaywm/wlroots/pull/1762 (Atomic output updates original PR)
See also: https://github.com/swaywm/wlroots/issues/1780 (Issue caused by atomic output updates)
See also: https://github.com/swaywm/sway/issues/4419 (Issue caused by atomic output updates)
See also: https://github.com/swaywm/wlroots/pull/1781 (Revert PR)

Can you point out the changes to this which fix the bug we reverted this for in the first place?

Sway fails to build with this:
../sway/config/output.c: In function 'set_mode':
../sway/config/output.c:209:10: error: void value not ignored as it ought to be
  209 |   return wlr_output_set_custom_mode(output, width, height, mhz);
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../sway/config/output.c:229:9: error: void value not ignored as it ought to be
  229 |  return wlr_output_set_mode(output, best);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../sway/config/output.c: In function 'apply_output_config':
../sway/config/output.c:270:19: error: void value not ignored as it ought to be
  270 |   modeset_success = wlr_output_set_mode(wlr_output, mode);
      |                   ^
../sway/config/output.c: In function 'set_mode':
../sway/config/output.c:230:1: warning: control reaches end of non-void function [-Wreturn-type]
  230 | }
      | ^

This is now ready for review.
The previous failures were due to wlr_output_attach_render being called without wlr_output_commit in Sway. A buffer was registered in the pending state, submitted in the next commit. The leftover buffer was making the commit fail (because all buffers were busy).
To fix this:

A new function, wlr_output_rollback, allows to discard the pending state and cancel a wlr_output_attach_render call
wlr_output is now more strict and refuses to commit a buffer or modeset a disabled output

Thanks!


Revert "Advanced filters"

Reverts wheresrhys/fetch-mock#346

Coverage increased (+0.3%) to 97.814% when pulling 22b202a6d2a5057aa8fee358c0a896bb91909c07 on revert-346-advanced-filters into 13a2525c659dadcbe42746a234671a99f8e6ebf4 on master.


Unit label incorrect for specific humidity plots

The units label for specific humidity RMSE (and MAE, MSE etc) plots is "g/kg" but the values are roughly 1e-3 which suggests the data being shown is actually in units of "kg/kg".
For example:

Thanks for pointing this out. I will try to remember to change this when I update the plots next.


Pass apex environment as tfvar to terraform

Pass apex environment as tfvar to terraform. This will allow terraform modules that are shared across all environments to take use the targeted environment at plan/apply time.

Simple PR included.


Extraction of EIA Data

Changes proposed in this Pull Request

In this PR I implement logic to extract data from the EIA API. My thoughts are we are starting to have a bunch of different data come from the EIA, and standardizing this would be useful.
@ktehranchi your thoughts on this would be great!! :)
Example:

See module header for example
The code shows how I envision the implementation (but am very open to other implementations!)
Extraction of fuel costs by fuel (gas and coal) and industry is implemented
Gas imports and exports by point of entry is implemented

Benefits:

Proposed code is extensible to add new API calls for specific data. The user really just needs to add the API call and raw data formatting logic.
Control of output data formats. Using the API, it is much easier to standarize dataframe structures compared to direct URL dowload (where formatting seems much more all over the place)
Separation of EIA data from the PyPSA-USA workflow. This allows us to easily test our EIA data processing code.
Calling of data will be easier - for example, getting time dependent fuel costs for different years will be cleaner, as all this logic is held in the totally independent eia.py module

Drawbacks:

Requires an API key. While this is free and very quick to get through the EIA (see here), it is an extra step.
Transferring existing EIA data to this will should be done. I see the following that can be extended to this:

Historical load data
Gas storage and production facilities
Powerplant capacity

API calls are limited to 5000 lines. So we will need to sort out a way to extract hourly data - this is a very solvable problem using offset in the api call, just needs to be implemented.

Checklist

[ ] I tested my contribution locally and it seems to work fine.
[ ] Code and workflow changes are sufficiently documented.
[ ] Changed dependencies are added to envs/environment.yaml.
[ ] Changes in configuration options are added in all of config.default.yaml.
[ ] Changes in configuration options are also documented in doc/configtables/*.csv.

I do also totally understand that this is getting towards what PowerGenome does, however, this is just limited to EIA data and therefore lighter weight.

It may be nice to move the year argument to the get_data() method call, but thats an implementation detail I guess.

I agree this will be useful for historical fuel flow/price data. I am actually going to move away from EIA data for the historical demand, and back to the GridEmissions (https://github.com/jdechalendar/gridemissions/pull/20#issuecomment-1894595311). I was planning on getting this done today or tomorrow so we can probably remove the need for historical demand in this.
I spent some time with the API in the past to get this data, but found the 6 mo files more simple solution.


Move the README.MD content somewhere else!

This documentation getting soo big and somehow ugly to maintain!
So far I'm thinking of Github Wiki I'm not a big fan of it, so we're for suggestions to a better alternatives.
Nice feature to have in the doc tool:

Supports versioning, in case Docker made an upgrade and we had to change some commands like in the past.
Easy to find and replace words in all pages, not having to open every single page.
Allows user to search all pages for any keyword.

Nice Github WIKI's for inspirations:

https://github.com/snowplow/snowplow/wiki
https://github.com/mcMMO-Dev/mcMMO/wiki
https://github.com/Netflix/Hystrix/wiki/Configuration

@Mahmoudz I like the format used by Netflix (not a big gamer so ... :-)
This one does look complete (https://github.com/Netflix/Hystrix/wiki/Configuration)
In addition, might want to consider airbnb style guide format (https://github.com/airbnb/javascript)
I am willing to help upgrade documentation

@mikeerickson sounds great. I like those docs.
What about the tool, any recommendations!? I'm really trying to stay away from the Github WIKI unless I have no other choices! I've tried it with other projects in addition to the missed features mentioned above it's somehow ugly! and hard to navigate in it unless you create your own navigations in each page..
Thanks!

@mahmoudz I am not sure on the tools, but I will do some research and get back to you.

We can use github page. We can keep the doc in this repository (so you have the documentation when you clone the repository) or use an other repository. Search can use lunr.js.

https://github.com/blog/2233-publish-your-project-documentation-with-github-pages
https://github.com/blog/1939-how-github-uses-github-to-document-github

I would love to contribute on this. @Mahmoudz @mikeerickson. Do keep me in the loop once the documentation format is finalized.

@dannysood @mikeerickson I was looking for some tools today and I found that Jekyll plays well with Github Pages. And while looking for themes I found this theme, it seems perfect for a documentation.
So I was testing it and I created a new repository to host the docs files only. Main reason why new repo and not /docs folder on this repository is to get this nice URL laradock.github.io/documentation instead of "laradock.github.io/laradock".
So guys feel free whenever you have some free time to migrate all the docs there. It's pretty easy just will take couple hours to move everything there. Check the readme of the new documentation repo for some help.
Note: the current documentation hosted is the demo site of that theme. I made no changes at all but adding README.md file to the root. So we need to clean all the default markdowns files from pages/ and modify the _data/sidebars/ yaml files to setup our own sidebar...

@Mahmoudz Funny you mention Jekyll, I am looking at this right now :-)
I have always considered using it for my repos, but didnt want the overhead as none of them are highly used projects.

@Mahmoudz If you want to add me, I will do the forking dance and get started with contribution :-)

@Mahmoudz you can use a /doc folder as a source for github website and I think get a nice URL. See the end of this documentation: https://github.com/blog/2233-publish-your-project-documentation-with-github-pages

@mikeerickson it looks like the Github team like this tool! anyway feel free to start anytime, and try to migrate section by section, each on it's own commit so we can see the changes and probably work with you in parallel or suggest editing.. Please also look at some existing examples to get inspired for naming the pages..
@gagarine seems by default Github Pages creates the following link {organization-name}.github.io/{repository-name} but not sure if we can change the default behavior! So if I create the docs here in this repo I'll get laradock.github.io/laradock which is not as cool as laradock.github.io/documentation.

@Mahmoudz might be worthwhile to buy a domain name, we can then use it for the github pages. laradock.io could be an idea?

vote laradock.io domain and we can use hugo Static Site Generator.

@Mahmoudz the coolness of the domain shouldn't force you of splitting repository if it make more sens to keep the doc with the code. Their is advantage of having one repository: pull request can contain the code and corresponding doc change. I can't see an advantage of a second repository for a small project like laradock. For extra fun, like @philtrep  we can use a custom domain.
@appleboy jekyll works out of the box on  GitHub and is widely used. Hugo is also popular but I can't see the advantage for a documentation over Jekyll. With Jekyll peoples don't even have to install any tools on their machine and can simply edit markdown file right on the repository using the GitHub editor (I don't clone locally the repository for a small text change)
@Mahmoudz for moving the documentation: first migrate (move the all readme on one page in a simple copy past), then improve (move section...).

@gagarine Agree. Thanks for your explanation.

@philtrep I just got that domain today :D
@appleboy Hugo is a another great popular tool. I'm not sure how well it works with Github! however we can always change the tool in the future if we found the need to.
@gagarine I get your point for making contribution less complex for users, the domain can actually solve the url little issue, so I'm gonna merge them in this repo now.

@dannysood @mikeerickson kindly note I've moved the docs to this repository inside the /docs folder and deleted the other one. The domain pointing there is laradock.io.

@laradock/admins Kindly note I've moved the docs from the readme.md to /docs. I've used Hugo as website generator tool, just because I think it's a little cleaner, faster and I like that theme. Also I've included steps for how to contribute to the documentation here.

@Mahmoudz Good to hear using Hugo as website generator tool. 👍

Happy to see the doc moved :) but I keep my position "against" Hugo for documentation mainly because we are going to commit generated HTML instead of clean markdown. But I'm curious to see how it works.

@gagarine I do agree on that, that's why I started playing with Jekyll first, but after giving Hugo a chance, I found it more elegant! and this theme we're using is really cool :D
I don't mind at any point later switching to another alternatives, as I myself forget to regenerate the site couple times, before pushing my changes.


Stream directly to mpv.exe on Windows 10?

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.09.26. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.09.26

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

Description of your issue, suggested solution and other information
How can I stream youtube-dl directly to my custom mpv.exe on Windows 10?
The "stdin" and the command is for linux only and does not work on Windows
Fully download takes long time and i don't want to wait.

Download youtube-dl.exe from the official download page and place it in the same directory as mpv.exe. After you have done that, running mpv.exe <link to the video> in command prompt from that directory should work as intended.

https://github.com/rg3/youtube-dl#how-do-i-stream-directly-to-media-player

It actually works. Thank you very much!


[ISSUE-806][REFACTOR] Remove ResourceConsumption out of ControlMessage

What changes were proposed in this pull request?
Remove ResourceConsumption out of ControlMessage
Why are the changes needed?
Because this class doesn't belong to ControlMessage, it is a normal class bean, should be put with Quota and QuotaManager, DefaultQuotaManager together under org.apache.celeborn.common.quota.
What are the items that need reviewer attention?
Related issues.
#806
Related pull requests.
How was this patch tested?
/cc @AngersZhuuuu
/assign @main-reviewer

ping @pan3793 @FMX


Fix PSSA Issues in MSFT_xExchPowershellVirtualDirectory (Fixes #85)

Fixing the PSSA Issues in MSFT_xExchPowershellVirtualDirectory.
This fixes #85.
Running Invoke-ScriptAnalyzer with the following rules excluded no longer has any output:
PSUseShouldProcessForStateChangingFunctions
PSDSCDscTestsPresent
PSDSCDscExamplesPresent
These rules have been approved to be ignored.

This change is

Hi @kwirkykat, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Katie Keim). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Looks good to me.


Invalid comma for CacheFlag enum

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the [latest JDA version][download].
[x] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report
There are an invalid comma inside CacheFlag file after MEMBER_OVERRIDES item
https://github.com/DV8FromTheWorld/JDA/blob/9e8408894409bb723d8a0c4adf53f82e753eabfe/src/main/java/net/dv8tion/jda/api/utils/cache/CacheFlag.java#L60
Expected Behavior
Remove this invalid comma and put the semicolon instead
Code Example
N/A
Exception or Error
N/A

This comma is valid syntax.

Considering that the code is still compiling without any errors can you assume that this comma isn't invalid.

True, but you can remove it without any issues

we need ocd label


Send token popup does not show ProfileComponent of receiver

Re use ProfileComponent.vue here - only call it when the address set is valid (v-if=isValidAddress(address))


Fix tab navigation adding extra route in history

Currently, when navigating to /preview/<uuid> (or other pages with tabs) you first land on the route, then the javascript clicks on the a tag of the first tab. This causes an extra route in the browser history so you have to hit back twice to go back to the previous page.
This fixes that by using replaceState instead of clicking on the a tag to navigate you to the hash of the first tab without adding another route to history.
Also changes the target prop for Preview on watch overview as it causes inconsistent behavior when opening preview. Edit opens in current browsing context so I believe Preview should as well. Along with the changes to the tab navigation it creates a better experience for navigating around the UI.

This would be fantastic, but I get this error in chrome
caught TypeError: window.location.replaceState is not a function
    at tabs.js:16:25

window.history.replaceState maybe?

third problem, it doesnt trigger the first tab state, so the first tab is not set

please fix and open a new one :)


1.0.0 Beta

Generating of GitHub login URL
Changed Github user access Scopes
Add basic HTTP redirect handler (#5)
Change codectrl-protobuf-bindings source and bump tag
Update Rust bindings to prevent compile error, please clippy
Fix incorrect run_server usage, delete old log server

More to come.
Credit to @z9fr.
Had to recreate because I accidentally merged when trying to merge changes from main into gh-auth.

Had to work to fix some conflicts.

Renaming this to a general-purpose 1.0.0-beta tracking PR.

Merging -- not releasing 1.0.0-beta yet


test: De-flake xds server_e2e_test

De-flake xds server_e2e_test by:

return nil error from xDS server after context cancel
wait for stream server error check to be done before completing test
return io.EOF upon channel close from stream_test mock
eliminate xDS CacheUpdateDelay
eliminate duplicate SendRequest in TestRequestStaleNonce
increase xDS test stream timeout

See commit messages for rationale for each step.
Fixes: #31855

/test

@sayboras How far you think we should backport this? The code touched here has barely changed, so it should be easy to backport, and recuding flakes in release builds may be valuable as well.

How far you think we should backport this? The code touched here has barely changed, so it should be easy to backport, and recuding flakes in release builds may be valuable as well.

Yes, xds code is not changed much for a long time, we can backport this till 1.13 imo.


mujoco display irregularity

Hi,
I'm a student and I'm trying to use MuJoCo for RL training
I'm looking for some help with display issue
I tried mujoco 200, 237, 315, None of them work properly on my laptop :(
All of them are only showing a little bit of the image in the top left corner as shown below
It seems to be just a display issue, I can even manipulate the options in this small area
And I use a computer with an amd cpu with core graphics and a nvidia graphics card, I don't know if that helps.
I'm hoping to find a way to get it to work on my computer, thank you very much for your reply!

Never mind.I updated my core graphics card driver and everything seems to be working fine! :)

For those of you who also using amd cpu with Radeon 680M core graphics, updating to 31.0.24031.5001 will solve your problem!

After I restarted the laptop, the strange thing happened again

I really don't have a clue.

Same issue there also. Did anyone found any solutions for this issue?

Hmm there used to be an open issue about problems related to AMD GPUs but I can't seem to find it...
@saran-t can you find it?

is there any way to solve this gui interface issue with AMD GPUs? Any advice or hack or solve is appreciated.

https://github.com/google-deepmind/mujoco/issues/639

Let's keep track of AMD related issues in #639.


Only python 2.7 is being tested by travis

I ran into a few surprising python 3.x incompatibilities when trying to use smarty in https://github.com/open-forcefield-group/open-forcefield-tools.
It looks like only python 2.7 is actually being tested in the travis branches for other python versions. I'll fix this in my PR #104.

I think this was fixed in #104 correct?

Yes!

So it's ok to close it right?


⚠ Bofrost.fr has degraded performance

In 4f7512f, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1365 ms

Resolved: Bofrost.fr performance has improved in 851ddab.


Adding comments in between the code

Hello, i would like to add comments in between the code to make it more readable as it would be difficult to comprehend if there are no proper comments in between the lines of code.
please assign me this task under GSSOC'24.
Thank you!

I would like to add comments in the code to make the code more understandable
Please assign this task to me under GSSOC'24

hi ### @shrawani21 ,
please label this issue i want to contribute in this project please assign it to me under the GSSOC'24
adding comment between the code is helpful to understand the code, it increase the code readability , it reduce the complexity of programme

I kindly request permission to enhance code readability with comments. I am eager to contribute meaningfully and ensure code is understandable for all. Please consider assigning this task to me.Thank you.

I am Thrilled about your project on Dots and Boxes ! My skills in aligning comments perfectly. Eager to learn & contribute! Available to start ASAP! #GSoC24

HII I AM REALLY EXCITED ABOUT SOLVING THIS ISSUE , I AM PROFICIENT IN PYTHON
KINDLY ASSIGN ME THIS TASK UNDER GSSOC'24

Hi, I want to do this. I am good at Python. Please assign me this task

Hi @shrawani21
I'm eager to contribute to the project. I've been looking at the issue regarding adding comments to the codebase, and I believe it would be a great opportunity for me to get involved and learn more about the project. Kindly assign me this issue.

I am interested in this task. Pls assign it to me under GSSOC'24 so that I can contribute to this project.


MSD command start_bl.act puts K64F to MAINTENANCE mode

I tried to put K64F to BOOTLOADER mode, by copying start_bl.act to DAPLINK drive like told in DAPLink/docs/MSD_COMMANDS.md. But instead device goes to MAINTENANCE mode. I tried
still copying daplink 244 to drive, it reseted but not mounting then at all. Only available mode is now maintenance (instead of bootloader by connecting with reset button on).
This has happened to 2 devices. Originally I had daplink 243 and automation mode on.

Hi @ErkkiMehtala, MAINTENANCE mode is bootloader mode. The drive name was changed from BOOTLOADER to MAINTENANCE when the DAPLink project was created from CMSIS-DAP.


Problem with connor - ValueError: After pruning, no terms remain

I encountered an issue while using the connor tool to organize my documents. The command I executed was:
$ connor run --path $(xdg-user-dir DOCUMENTS)
--------------------------------------------------------------------------------
To customize default settings instead run the command <connor settings -h>
folder_name_length: 2
reading_word_limit: 100
similarity_threshold: 50%
--------------------------------------------------------------------------------
Folder '/home/denis/Документы' is being organized...
Traceback (most recent call last):
  File "/home/denis/.local/bin/connor", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/home/denis/.local/pipx/venvs/connor-nlp/lib/python3.11/site-packages/connor/main.py", line 29, in main
    cli_tool.organize_folder(args.path)
  File "/home/denis/.local/pipx/venvs/connor-nlp/lib/python3.11/site-packages/connor/connor_nlp/command.py", line 58, in organize_folder
    data_vectorized = vectorizer.fit_transform(words[1] for words in self.file_list)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/denis/.local/pipx/venvs/connor-nlp/lib/python3.11/site-packages/sklearn/feature_extraction/text.py", line 2091, in fit_transform
    X = super().fit_transform(raw_documents)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/denis/.local/pipx/venvs/connor-nlp/lib/python3.11/site-packages/sklearn/base.py", line 1473, in wrapper
    return fit_method(estimator, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/denis/.local/pipx/venvs/connor-nlp/lib/python3.11/site-packages/sklearn/feature_extraction/text.py", line 1385, in fit_transform
    X = self._limit_features(
        ^^^^^^^^^^^^^^^^^^^^^
  File "/home/denis/.local/pipx/venvs/connor-nlp/lib/python3.11/site-packages/sklearn/feature_extraction/text.py", line 1237, in _limit_features
    raise ValueError(
ValueError: After pruning, no terms remain. Try a lower min_df or a higher max_df.

Expected Behavior:
I believe it would be beneficial if the connor tool could:
Allow sorting of documents regardless of their content.
Handle empty documents without throwing errors.
This would enhance the usability of the tool, providing a more seamless experience when organizing files.
A HUGE THANKS FOR YOUR INCREDIBLE FILE ORGANIZER!!!

Commit 40cc86f solves this issue.
I have also added user confirmation to go through with the organization processes. Snippet from CLI:
...
The above directory tree explains how the folder will be organized.
Do you want to continue? [y/n] y
Folder 'path/to/folder' organized successfully.

...
The above directory tree explains how the folder will be organized.
Do you want to continue? [y/n] n
Folder organization aborted. The files in 'path/to/folder' were left untouched.


bumping pillow version after security updates

Have we tested cifar with this new version of pillow? I think in the past we set it to a specific version because of some weird conflict/error with certain versions. Maybe that's not an issue anymore though.

I was able to train and see the expected accuracies. I'm not sure if pillow is used in some other post processing that is not included in that pipeline, though.

Awesome, that's good enough for me. I recall last time it caused an immediate error and wouldn't even run so that sounds good!


Let new users complete their account setup

This PR adds the final piece to #271: letting newly-approved users receive an email with their setup instructions, with a link to choose a password and come on in. As with the previous PRs in this chain, testing it is a little bit clunky until we complete #273.
Testing:

Start Rails and Sidekiq
Prerequisite: a newly-created applicant that has confirmed their email address and been approved by an admin, and then been turned into a user. Reminder of those steps:

Go to http://localhost:3000/apply and create a new applicant (with an email address not matching any existing user)
Check your email, click the link, confirm their email address
Switch to the Rails console: rails c
Find that applicant (a = Applicant.find(…)) and approve them: a.approve
Create a user from that now-approved applicant: u = User.create_from_applicant(a)

Send setup instructions to that user: u.send_setup_instructions
Check your email, click the link, and choose your password
✅ You should be logged in!

Testing failing cases:

Try bad passwords
Try to setup an account a second time

Closes #271

Can we add a User to seeds.rb that takes care of the prerequisite steps? We could also just add a bit of code that runs through the steps when we run rails db:seed. It seems as if that's going to be important to have hanging around from here on out for all testing purposes.

Can we add a User to seeds.rb that takes care of the prerequisite steps?

🤦 Of course! Great idea. On it.

@cguess Okay, I have updated the seeds file with a whole host of new applicant and user states. The three old users remain, but now we can target each stage of the process.
So for the purposes of this PR, you could skip the prereq section and just:
u = User.find_by(email<EMAIL_ADDRESS># Stop! Update the user to have an email address that you can receive at. Then, continue…
u.send_setup_instructions

Note that you need to update the user to have an email address where you can actually receive mail, or else you'll never get the setup link.

should we require a bit more password protection? Maybe longer than 6 characters?

We certainly can! Here I was just mimicking Devise's defaults. I'll open a discussion issue.


Support for "long vector" types

Elements not covered by the initial support we intend to upstream:

Constant expressions, cf https://github.com/google/clspv/pull/655/files#r510348777
Additional LLVM intrinsics, cf #666
Phi nodes, cf #686
More complex forms of GEP, cf #694
Additional bitcasts, cf #732

Original description below.

Because "long vector" types (i.e. any vector type of arity 8 or 16) cannot be directly represented in Vulkan SPRI-V, their usages are currently rejected by the frontend.
We (arm) have been working on a prototype that lowers these vector types into equivalent aggregate of scalars and massages OpenCL C builtin function calls and other LLVM instructions to produce valid, equivalent Vulkan SPIR-V. The core functionality is implemented as an LLVM module pass. It is, in some ways, similar to the LLVM scalariser but it selectively scalarises instructions and handles further builtin functions as well as function parameters.
The benefit of this pass is to increase support for arm compute library (#349). Before providing our implementation, we wanted to give a high level description of how it works and make sure that the proposed approach looks sensible to you.
Here is an example of what the prototype currently supports:
static size_t getId() { return get_global_id(0); }
static float8 load(global float *ptr) { return vload8(getId(), ptr); }
static void process(float8 a, private float8 *b) { *b = fma(a, 4, 7); }
static void store(global float *ptr, float8 value) { vstore8(value, getId(), ptr); }

kernel void k(global float *in, global float *out) {
  float8 x = load(in);
  process(x * 2, &x);
  store(out, x);
}

Long vector types are supported as variable types and parameters types for user-defined functions but not for kernel parameters. It might be possible to support the latter case, too, but we left it out for now and an appropriate error message is provided to the user. Pointers to these types are supported as well. The pass will rewrite user-defined functions, when needed. E.g. the load function shown above is rewritten as follows in LLVM IR:
define internal spir_func { float, float, float, float, float, float, float, float } @load.1(float addrspace(1)* %0) #1 {
entry:
  %call.i = call spir_func i32 @getId() #5
  %1 = shl i32 %call.i, 3
  %2 = getelementptr float, float addrspace(1)* %0, i32 %1
  %3 = load float, float addrspace(1)* %2, align 4
  %4 = or i32 %1, 1
  %5 = getelementptr float, float addrspace(1)* %0, i32 %4
  %6 = load float, float addrspace(1)* %5, align 4
  %7 = or i32 %1, 2
  %8 = getelementptr float, float addrspace(1)* %0, i32 %7
  %9 = load float, float addrspace(1)* %8, align 4
  %10 = or i32 %1, 3
  %11 = getelementptr float, float addrspace(1)* %0, i32 %10
  %12 = load float, float addrspace(1)* %11, align 4
  %13 = or i32 %1, 4
  %14 = getelementptr float, float addrspace(1)* %0, i32 %13
  %15 = load float, float addrspace(1)* %14, align 4
  %16 = or i32 %1, 5
  %17 = getelementptr float, float addrspace(1)* %0, i32 %16
  %18 = load float, float addrspace(1)* %17, align 4
  %19 = or i32 %1, 6
  %20 = getelementptr float, float addrspace(1)* %0, i32 %19
  %21 = load float, float addrspace(1)* %20, align 4
  %22 = or i32 %1, 7
  %23 = getelementptr float, float addrspace(1)* %0, i32 %22
  %24 = load float, float addrspace(1)* %23, align 4
  %25 = insertvalue { float, float, float, float, float, float, float, float } undef, float %3, 0
  %26 = insertvalue { float, float, float, float, float, float, float, float } %25, float %6, 1
  %27 = insertvalue { float, float, float, float, float, float, float, float } %26, float %9, 2
  %28 = insertvalue { float, float, float, float, float, float, float, float } %27, float %12, 3
  %29 = insertvalue { float, float, float, float, float, float, float, float } %28, float %15, 4
  %30 = insertvalue { float, float, float, float, float, float, float, float } %29, float %18, 5
  %31 = insertvalue { float, float, float, float, float, float, float, float } %30, float %21, 6
  %32 = insertvalue { float, float, float, float, float, float, float, float } %31, float %24, 7
  ret { float, float, float, float, float, float, float, float } %32
}

(Technically, this is the output after the new pass and InstCombine. The lowering of long vectors introduces many instructions that are trivial to remove and do not impact, AFAIK, the quality of the generated Vulkan SPIR-V.)
As you can already see from the above snippet, vload8 was lowered into 8 getelementptr + load pairs of instructions and the resulting float8 object is represented using an aggregate of 8 floats.
Similarly, the multiplication x * 2 and the call to fma are replaced by 8 fmul instructions and 8 calls to @_Z3fmafff, respectively. We currently have support for a limited set of builtin functions using a simple mapping between mangled names. We appreciate this solution has some serious shortcoming but we feel confident that it can be replaced later on with a better alternative that scales to support all builtin functions without impacting the design of the overall lowering pass.
If this approach looks okay to you we will produce PRs over the next couple of weeks to share our code, after we have finished some minor cleanups. We can either do it in one go with a larger PR or in small chunks, as you prefer. For reference, the current bulk of the implementation is roughly 1K LOC, not counting the tests.

Why are kernel parameters not supported? I would have thought that would be necessary to handle most code using long vectors.
Does parameter handling include support for pointers to long vectors? Is that the case that is preventing kernel parameters from being supported? You don't have an example here of that.
I'm not sure I understand the problem with the builtins? Which calls are problematic to identify? The builtin demangler should be able to recognize all the functions.

Thanks for your reply.

Why are kernel parameters not supported? I would have thought that would be necessary to handle most code using long vectors.

I personally believe they could be supported, but it implies lying to the runtime and I don't have proof that it will work with any runtime. As a first step, I suggest we leave this as unsupported for now. It can be revisited in the future without major redesign I think.
Supporting these is not strictly necessary because long vectors can be loaded from memory using vload8/16, but I agree that it would be a nice feature.

Does parameter handling include support for pointers to long vectors? Is that the case that is preventing kernel parameters from being supported? You don't have an example here of that.

Currently, both long vector types and pointer to long vector types are rejected when used as kernel parameters using a check in the frontend for the reason mentioned above.
However, both are supported as user-defined functions. For example, process above is rewritten as follows:
define internal spir_func void @process.2(
    { float, float, float, float, float, float, float, float } %0,
    { float, float, float, float, float, float, float, float }* %1
) #1 {
...

(Functions are renamed when they need rewriting but I'm hopeful this won't be the case in the future although it doesn't matter when they are internal functions.)

I'm not sure I understand the problem with the builtins?

There isn't really a problem. To clarify what I meant earlier, we took the simplest and cheapest solution we could find for this prototype and focused our attention to trickier areas. We took a test-driven approach to ensure the builtin functions were converted to the expected SPIR-V and therefore decided to limit the support of these builtins to the ones we have tests for. We identified this as a sub-problem that can be progressed independently once the core functionality is implemented.

The overall direction sounds reasonable. I agree it's good to implement this incrementally. We can improve the conversion later if necessary. Smaller PRs are generally preferable to single large ones.

I've created #638 to introduce the first elements to support long-vectors. Please do let me know if at any point you feel the PRs are too small/too big. I'll submit sometimes multiple commits within the same PR but I'm happy to provide them separately instead if you prefer.
I would like that acknowledge and thank two colleagues at Arm, Pedro Olsen Ferreira and Anna-Mariia Koltsova, for their help and collaboration on this project.

With #732 merged, this completes our (Arm) prototype for long vector support. Some elements are known to be missing (cf non-exhaustive list at the top of the description + TODO in tests/code), but they can be addressed independently by anyone we expect.
If desired, I can open one issue per element on the list in the description and close this issue. Let me know your preference.
A personal thanks for all the good review comments!

We can leave this issue open at least until someone starts tackling the remaining work. Thanks for all the contributions!


Please add support for H5082

What issue do you have? Please be as thorough and explicit as possible.
Recieving this error while trying to use Govee H5082 devices
Details of your setup.

Do you use Homebridge (with config-ui?) or HOOBS?

config-ui

Which version of Homebridge/HOOBS do you have?

homebridge 1.3.4

Which version of this plugin (homebridge-govee) do you have? Has the issue started since upgrading from a previous version?

homebridge-govee v4.7.0

Which Govee devices do you have that are causing issues? Please include product models if applicable.

H5082 dual plug smart plugs
Please paste any relevant logs below.

[Dual Smart Plug #4] is not currently supported, do create a Github issue with the following info:
{
   "device":"DA:F6:9C:04:A0:08:F9:F9",
   "deviceName":"Dual Smart Plug #4",
   "model":"H5082",
   "httpInfo":{
      "device":"DA:F6:9C:04:A0:08:F9:F9",
      "sku":"H5082",
      "spec":"0",
      "versionHard":"1.02.00",
      "versionSoft":"1.00.09",
      "deviceName":"Dual Smart Plug #4",
      "deviceExt":{
         "deviceSettings":"{"wifiName":"IoT_all_the_things_2G","address":"9C:04:A0:08:F9:FA","bleName":"ihoment_H5082_F9FA","topic":"GD/7ba3f046bb1189803d2526173e5d93d7","wifiMac":"9C:04:A0:08:F9:F9","pactType":1,"pactCode":1,"wifiSoftVersion":"1.00.09","wifiHardVersion":"1.02.00","ic":0,"secretCode":"wqa0AvGnyAo=","subDevices":{"sub_0":{"name":"Outlet 2"},"sub_1":{"name":"Outlet 1"}},"sku":"H5082","device":"DA:F6:9C:04:A0:08:F9:F9","deviceName":"Dual Smart Plug #4","versionHard":"1.02.00","versionSoft":"1.00.09"}",
         "lastDeviceData":"{"online":false}",
         "extResources":"{"skuUrl":"","headOnImg":"","headOffImg":"","ext":"","ic":0}"
      },
      "pactType":1,
      "pactCode":1,
      "goodsType":50
   },
   "isAPIDevice":false
}

I have sent you a dm on discord


Dep ensure -update indicates out-of-sync when files in sync; proj isn't

Dep Version
v0.3.0, darwin-amd64 - Downloaded from GitHub releases page
What dep command did you run?
$ dep ensure -v -update github.com/akutz/gofig
Gopkg.toml and Gopkg.lock are out of sync. Run a plain dep ensure to resync them before attempting to -update

What did you expect to see?
I expected that the gofig dependency would be updated.
What did you see instead?
Gopkg.toml and Gopkg.lock are out of sync. Run a plain dep ensure to resync them before attempting to -update

Addendum
The fact is that the Gopkg.toml and Gopkg.lock files were in sync. I even checked them both out from upstream/master to ensure it. I then had the idea to run dep status, and that's when I figured out what appears to have been the issue:
$ dep status
Lock inputs-digest mismatch due to the following packages missing from the lock:

PROJECT                     MISSING PACKAGES
github.com/sirupsen/logrus  [github.com/sirupsen/logrus]

This happens when a new import is added. Run `dep ensure` to install the missing packages.

The above output indicates that the dependency github.com/sirupsen/logrus is missing from the lock. That's true, but it's also not in the toml file. It is in the project as I'd already updated the Go import statements to reflect the new case of sirupsen.
It appears to me that an out-of-sync error is given when it's not the files that are out of sync, but the project that includes imports for packages not reflected either in the toml or lock file. Is this the intended behavior? Even if so I think a better error could be provided to indicate it's the project that is out of sync.

According to dep model:

The first input is both Gopkg.toml and the project imports (from code). The error message should reflect that the input-hash could change in both (and therefore Gopkg.lock becomes out of sync).


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1351 ms

Resolved: Bofrost.de performance has improved in 58d7bdf.


Retrieve old podcast entries

Hi! I would like to know if it is possible to implement an option to search and play old episodes of a given podcast, that is, let's say I want to play an episode of 2016 to take notes and process the information. At the moment PodNotes only shows the episodes of the last six months.
Maybe it has something to do with the platform from which I collect the information from.
I attach the loml of the podcast: https://www.ivoox.com/feed_fg_f136870_filtro_1.xml
I congratulate you for your work with this plugin. Thank you very much,
I look forward to your comments.
Best regards.

Hi @EhuddR,
Thank you!
Looks like the feed link only provides episodes going back to Sat, 16 Apr 2022.
I looked around a bit, but couldn't find any providers that had any episodes going further back, so I assume it's just the podcast host that doesn't provide them.
Best regards,
Christian


New class request: Argon-39

xref https://github.com/EnvironmentOntology/envo/issues/904
Hi CHEBI team - could you please add this isotope of Argon, used in the Essential Ocean Variables.
Annotation properties are textbook, nothing fancy here.

An entry for argon-39 (CHEBI:155827) has now been created. It should be visible on the ChEBI website by tomorrow.


StoreDescriptor rename temp file failed

fileStream not close, rename temp file failed.

@duckshine do you have a stack trace? Also please describe when this happened.

I'm work in windows, when ambry server start.
StoreDescriptor.java line 82.


Navigation Menu

Implement the multi-level navigation menu on both desktop and mobile per the design: https://www.figma.com/design/2BTJg00ikyVZMj82I9sSYB/Lume-JAN

Here @pcfreak30 we need to better define what the new navigation will be. We can adapt a component but in terms of layout theres no clear reason to have multi level navigation

Here @pcfreak30 we need to better define what the new navigation will be. We can adapt a component but in terms of layout theres no clear reason to have multi level navigation

Consider the following:

Services

S5

Potential subsection

IPFS

Potential subsection

Future Service

API Keys
Account
...


Could you add into the FAQ comparison to pytorch and tensorflow

Isn't this comparison table enough?
I think it is a bit difficult to access.
https://docs.chainer.org/en/stable/comparison.html

Thanks. I  aggree. So it seems to me there is no sense to prefer it over pytorch.


Jeff Schaller: Watch lichamduong

Jeff Schaller requests the watch of the watch_keyword lichamduong. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

lichamduong has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Eliah Kagan in Charcoal HQ


Redesign build for TeamCity build server with InvokeBuild module

Is Your Feature Request Related To A Problem? Please describe.
The build we currently handle doesn't do any script signing or formatting passes, and isn't very easily usable in our TeamCity environment because we don't really have a great way to define build task dependencies.
Describe The Solution. Why is it needed?
Rewrite the build in Invoke-Build, pulling in tasks for signing the script(s) and running PSSA formatting lints (see #67).
This should let us much more easily pull this build into our TeamCity build server.
Additional Context.
Invoke-Build quickstart here: https://github.com/nightroman/Invoke-Build/wiki/Script-Tutorial
Related Issues

Issue #67

:tada: This issue has been resolved in version 0.3.0 :tada:
The release is available on:

GitHub Release
NuGet Package

Your GitReleaseManager bot :package::rocket:


(PUP-598) Add gpasswd support to Group provider

CLA signed by all contributors.

@peiriannydd In order to get this merged I'd like to see your commits squashed, with a message that follows the contributing.md file, and the syntax changed to a proper if statement as mentioned above.

@peiriannydd have you had a chance to respond to @branan 's comments?

Sorry, haven't had time. Will try to look this week.
On Tue, Apr 14, 2015 at 2:23 PM, Melissa<EMAIL_ADDRESS>wrote:

@peiriannydd https://github.com/peiriannydd have you had a chance to
respond to @branan https://github.com/branan 's comments?
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3687#issuecomment-93007491.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

Hi @trevor-vaughan, thanks again for the contribution!  Thought I'd check in on the status of the feedback above during our weekly PR triage meeting.  Thanks!

I'll be looking at this tonight.
On Wed, Apr 22, 2015 at 1:18 PM, Peter Huene<EMAIL_ADDRESS>wrote:

Hi @trevor-vaughan https://github.com/trevor-vaughan, thanks again for
the contribution! Thought I'd check in on the status of the feedback above
during our weekly PR triage meeting. Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3687#issuecomment-95270239.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

The commits have been squashed and the message has been updated.
On Wed, Apr 22, 2015 at 3:44 PM, Trevor Vaughan<EMAIL_ADDRESS>wrote:

I'll be looking at this tonight.
On Wed, Apr 22, 2015 at 1:18 PM, Peter Huene<EMAIL_ADDRESS>wrote:

Hi @trevor-vaughan https://github.com/trevor-vaughan, thanks again for
the contribution! Thought I'd check in on the status of the feedback above
during our weekly PR triage meeting. Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3687#issuecomment-95270239.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

@trevor-vaughan It looks like either Struct::Group needs to be stubbed so that the spec can run on Windows or need to exclude it from running on Windows.  That's the source of the above AppVeyor failure.

Interesting. I don't actually have a working Windows test system right now.
Would you mind providing the stub?
Thanks,
Trevor
On Thu, Apr 23, 2015 at 2:53 PM, Peter Huene<EMAIL_ADDRESS>wrote:

@trevor-vaughan https://github.com/trevor-vaughan It looks like either
Struct::Group needs to be stubbed so that the spec can run on Windows or
need to exclude it from running on Windows. That's the source of the above
AppVeyor failure.
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3687#issuecomment-95684528.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

@trevor-vaughan We should probably just prevent the relevant spec from running on Windows entirely.  There are examples in other specs where we confine to having the windows? feature present.

Ah, OK
On Wed, Apr 29, 2015 at 1:27 PM, Peter Huene<EMAIL_ADDRESS>wrote:

@trevor-vaughan https://github.com/trevor-vaughan We should probably
just prevent the relevant spec from running on Windows entirely. There are
examples in other specs where we confine to having the windows? feature
present.
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3687#issuecomment-97511896.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

@trevor-vaughan Sorry for the confusion: was that something you could push an update for?  Adding :unless => Puppet.features.microsoft_windows? to the specs should do the trick.

Sorry, just been busy.
On Wed, May 6, 2015 at 12:39 PM, Peter Huene<EMAIL_ADDRESS>wrote:

@trevor-vaughan https://github.com/trevor-vaughan Sorry for the
confusion: was that something you could push an update for? Adding :unless
=> Puppet.features.microsoft_windows? to the specs should do the trick.
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3687#issuecomment-99532289.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

Sorry this took so long. Added the restriction and pushed.

CLA signed by all contributors.

Hi @trevor-vaughan.  Have you had a chance to review @joshcooper's feedback yet?  Thanks!

Review, yes, fix no...sorry :-/
Trevor
On Mon, May 18, 2015 at 3:03 PM, Peter Huene<EMAIL_ADDRESS>wrote:

Hi @trevor-vaughan https://github.com/trevor-vaughan. Have you had a
chance to review @joshcooper https://github.com/joshcooper's feedback
yet? Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3687#issuecomment-103177556.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

Hi @trevor-vaughan.  Just checking in again from PR triage :)

sigh
Hi Peter....
On Tue, Jun 2, 2015 at 2:02 PM, Peter Huene<EMAIL_ADDRESS>wrote:

Hi @trevor-vaughan https://github.com/trevor-vaughan. Just checking in
again from PR triage :)
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3687#issuecomment-108034571.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

Over 6 year old bug: https://projects.puppetlabs.com/issues/1942

@joshcooper to review with @branan PR

All of these should have been fixed in the last PR update, did that come through?

@branan is going to be looking at this issue in our next sprint.

I'm guessing that we won't have time for this for another sprint or two. That doesn't make me happy but it's probably realistic.

Marked the associated ticket for grooming so we can hopefully pick this up again in the next few weeks/months.

@jhelwig The upstream gpasswd module has had additional updates that need to be integrated into this PR. Not sure when I'll have time to update this, but the gpasswd code can be found over at https://github.com/onyxpoint/puppet-gpasswd and I have 'Allow commits from maintainers' if anyone wants to make an update.

The updates definitely address some of the concerns I have. The "successful, but not really" if managing any of the group members fails is really the biggest thing that concerns me about this. https://github.com/onyxpoint/puppet-gpasswd/blob/a265ddeb8e43faf4f88e9456716ccfc6ab73c8b4/lib/puppet/provider/group/gpasswd.rb#L88-L97
I'm still not sure what the best way to handle this would be, but I don't think that letting the resource "succeed" is it. I'll see if I can set aside some time to poke at this to find a better option.

Closing this PR since nobody has had time to come back and address the changes necessary, and the upstream gpasswd module has moved on significantly and needs to be re-integrated into this work.


pulse moving non-width-defined elements side to side

pulse moving non-width-defined elements side to side, is there a way I can fix it?

wait, I think i just solved it.
.pulse {   -webkit-animation-name: pulse;   animation-name: pulse;        transform-origin: 0%;     -webkit-transform-origin: 0%; }


Semantic Versioning Documentation, for drud/ddev #352

The Problem:
We committed to support this and we just need some additional documentation outside the issue queues to both 1) make this more obvious to an end-users and 2) make this as part of our release processes.
The Fix:
See verbiage in PR.
Related Issue Link(s):
OP drud/ddev/issue/352

@rfay Added you as reviewer to get the official +1


setLang does not work quite righ

Hello, I have a big project with maps, pm is registered globally and because of this, setLang does not work quite right, I mean that in fact the merge of the object happens, and not clone , if I use fallback lang , then return to the original i I can't. Only after reloading the page.

Without your code we can't help you much. Please create a demo: https://jsfiddle.net/o1dwu2vg/
Are you doing following?
// from default en to de
map.pm.setLang('de');

and later:
// from de to en
map.pm.setLang('en');

map.pm.setLang('custom', { buttonTitles: { drawMarkerButton: 'Поиск в точке', drawLineButton: 'Поиск по кривой', drawRectButton: 'Поиск в прямоугольнике', drawPolyButton: 'Поиск в полигоне' } }, 'ru')
and later:
map.pm.setLang('ru')
don't work.The custom translation will remain

I second this. I have my own custom language and switching to it works just fine:
map.value.pm.setLang('lv', geomanTranslation, 'en');
However, afterwards the switch back to English does not work:
map.value.pm.setLang('en');
Instead of switching back to English, I tried some other languages like de and ru - both actually worked just fine. Seems that there is a particular problem with switching back to English.


[5.5] Add replace method to the Collection class

Simple implementation of a replace method which replace all occurrences of an item to another.
I kept this implementation simple but it's possible to add a strict mode, an optional callback for the replacement and other features.

Really nice @ThibaudDauce , what about adding an extra $strict = true parameter ?

@lucasmichot because I didn't want to add complexity to the PR and I didn't find a clever solution to avoid the additional condition ^^
if ($strict) {
    return $item === $search ? $replace : $item;
}

return $item == $search ? $replace : $item;

We could need a new helper:
function is_equals($a, $b, $strict)
{
    // if ($a instanceof Model AND $b instanceof Model) {
    //     return $a->is($b);
    // }

    if ($strict) {
        return $a === $b;
    }

    return $a == $b;
}

$operator = $strict ? '===' : '==';

return eval("$a $operator $b');

I'd prefer to stick with the conditional though, not sure it's worth the performance boost.

Up to @taylorotwell on that
BTW, eval() is hell 😈

@ThibaudDauce Any reason for leaving out nested collections? :)

@morloderex what behaviour do you expect from nested collections?
collect([
    collect([1, 2, 1]),
    collect([2, 1, 2]),
])->replace(1, 3);

// [ [3, 2, 3], [2, 3, 2] ]

I would just use map directly in your own code.


MM-34977 - add the correct text to the purchase modal button when trialing

Summary
Add the correct text to the purchase modal button when trialing
Ticket Link
https://mattermost.atlassian.net/browse/MM-34977
Screenshots

@pablovelezvidal I'm still seeing "Upgrade Mattermost Cloud" in the button on https://mattermost-webapp-pr-7939.test.mattermost.cloud/admin_console/billing/subscription

yes, but I don't see that the server has enabled the free_trial option

Yes, this subscription is not created in trial mode yet, possibly because the trial changes were not in the test portal yet when the test server was created in this PR.
Adding the tag again to recreate.

/update-branch

Mattermost

Test server is now updated and in trial mode. @stevemudie you can test again now

/cherry-pick cloud

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-webapp
   ccacddc96..bff425d1c  cloud      -> upstream/cloud
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-webapp-#7939-upstream-cloud-1619426165
Switched to a new branch 'automated-cherry-pick-of-mattermost-webapp-#7939-upstream-cloud-1619426165'
Branch 'automated-cherry-pick-of-mattermost-webapp-#7939-upstream-cloud-1619426165' set up to track remote branch 'cloud' from 'upstream'.

+++ About to attempt cherry pick of PR #7939 with merge commit 9e06dd3b697050f7401213eef974f0606f472bfb.

error: could not apply 9e06dd3b6... MM-34977 - add the correct text to the purchase modal button when trialing scenario (#7939)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

+++ Conflicts detected:

UU i18n/en.json
Aborting.

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.

/cherry-pick cloud

Cherry pick is scheduled.


how to make it works?

I got different errors reported by different sbt version.  How can I make it works after download it.
Should I install scala, spark before set up this package 'sbt package'?

Hi @fine1x,
Have you installed spark 2.2.0 already? You can get it in here.
After that you can follow the tutorial in here. If you have additional questions, let me know.
Best wishes,
Heitor

I am closing this, get in touch again if you need assistance.


Updated SRTM URL to kurviger.de

Since I renamed my motorcycle route planer to Kurviger I had to change the base URL to Kurviger.de. The old url will be still valid for the foreseeable future.

Somehow I'm getting in a branch CertificateException: No subject alternative DNS name matching srtm.kurviger.de found for the https version (and also when trying to connect via the browser this fails). Should we just use http?

Sorry, should be fixed.


[5.3] Date: Add distance(to:) and advanced(by:) introduced in macOS 10.15

(cherry picked from commit 65ca91c724dc744da4fe1f241dec04eb1df93531)

@swift-ci test

cc @drexin

cc @millenomi

cc @tomerd


Tests fail due to a branch in simple-server that doesn't exist

Tests fail due to using a branch (gh96) from simple-server repo that doesn't exist.
https://github.com/steveklabnik/doxidize/blob/master/Cargo.toml#L39
Build failure : https://travis-ci.org/steveklabnik/doxidize/builds/359890935
There is already a PR to use Travis CI #89 . Feel free to close this if it's not necessary.
Thanks

See #92

On Apr 1, 2018, at 8:42 AM, Xtreak<EMAIL_ADDRESS>wrote:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Sorry about that. I overlooked the issue thinking it was windows specific. Closing it in favour of #92 and #91 .
Thanks


Remove dependence on unittest from Adapter tests

Description
Remove dependence of the adapter tests on unittest (and instead depend on pytest).  Also updates some lagging references to the shared adapter libraries, that were still pointing at dbt-core.
Checklist

[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt-databricks next" section.

@jackyhu-db when I went to add tests per your request on the other PR, I saw that this test suite had not been updated to our latest standards, and figured I would do that refactoring separate from adding new tests.


FileNotFoundError when running make duck_cloudfront command

Description
Running the make duck_cloudfront command described in the README results in a FileNotFoundError. The error occurs in the duck.py script which is trying to read the  CC-MAIN-2024-22.warc.paths.gz which is not created yet
Suggested Fix

Add the  CC-MAIN-2024-22.warc.paths.gz  file to the repository as mentioned in the makefile

Thank you, I took your suggestion in a recent PR.


The provider is not registered

Hello,
I try to work with the provider "Zadarma", entered the SIP server, user name and password but the
provider is not registered. What could be the problem? how to proceed?
Thanks!
Yuval

You should use static ip registration.


Proof of Concept of Styles at a Global Scale

Description
As a user of OSCAL viewer, I would like to see a simple demonstration/concept of how styles can be implemented at a global scale.
Acceptance Criteria

A working proof of concept is created to demonstrate styles being used at a global scale

Proposed Solution
One or two styled elements are supported in this proof of concept, such as a primary button and a single header.
Additional Notes
This should not be a full implementation and a formal strategy doesn't need to be determined yet.
Implementation

[ ] I can (or plan to) submit a pull request to implement this
[ ] Implementing this may result in a breaking change

After configuring a bit of our theming/styling, here's a strategy I propose going forward. I believe there are three layers of implementing our styles from a global and more reusable standpoint:

Top/App level: A theme directory is created with general styling to pass to App.js at oscal-viewer/src.

This styling is more global/broad to edit common elements like headings and buttons by overriding material UI defaults. I think it will make sense to start with a single theme file starting off AppTheme.js file will work for now, although this will set us up for adding any other themes, such as a dark mode.
This article explains an implementation for theming and using multiple themes: Theming and Theme Switching with React and styled-components | CSS-Tricks - CSS-Tricks

Component level: The most common pattern I'm seeing is either to keep styling within the component or split Styled components into separate style files. To follow a more reusable pattern for our styled components, I believe the latter approach will be better.

For example, if OSCALCatalogGroup.js has styled components, these styled components would be placed in OSCALCatalogGroupStyle.js. If we see these components being reused by more than a single component we can place this styling at a parent level component styling that would be shared between the two files. With all of our components, I'm thinking to make our file structure a little more clean, these style files would be placed in a styles or styled-components directory within oscal-react-library/src/components.
An article discussing styled component organization: Organizing CSS with React Styled Components

One off edits within a component that aren't reused: Use sx inline styling.

It is a common practice to add these in cases where edits are small and there is no reusability. This follows with CSS practices as well, as typically inline styling is only used when one attribute is edited, otherwise they are placed in classes.
For more details see here: How to customize - Material UI (mui.com)

To view how theming is implemented, please review the Proof of Concept PR I've created: https://github.com/EasyDynamics/oscal-react-library/pull/780. I don't plan to do any major refactoring to implement the component level changes in this proof of concept, but I imagine any styled(<component>) will just be moved to it's own respective …Style.js/…Style.ts file. We also use sx occasionally to do inline styling, although some of these can be reused and should probably can be placed in a `Style.js file or may be seen as unnecessary with future theme changes - I imagine it will be more of an iterative process to fix these.

Looking further into theming, it seems like it has the capabilities to handle a lot of global styling, which is demonstrated within the PR. It can override styling components fairly easily, including buttons. I'm thinking an Emotion file doesn't seem necessary as theming can handle the needs of the stylesheet.

Looking further into theming, it seems like it has the capabilities to handle a lot of global styling, which is demonstrated within the PR. It can override styling components fairly easily, including buttons. I'm thinking an Emotion file doesn't seem necessary as theming can handle the needs of the stylesheet.

Considering this a little more, I still think that having a global styles file (Emotion) would be nice to have to style common components through the application. For example say we have a secondary button styling that we don't want to have directly impacted by theming or have greater flexibility to customize and style common components at a global scale.
Below you can see the before and after screen shots of some initial style changes. I've changed the font family to Source Sans Pro, and the MUI buttons have been updated to Emily's style guide:
Before:

After:

Pushing to Sprint 46

In the proof of concept, I've started a basic implementation of the Emotion file (GlobalStyles.js), which will handle global styling unrelated to palette colors and fonts.
For demonstration purposes I've taken the general body and code styles which were previously in CSS and converted them into CSS in JS code. This makes it very simple to pass our themes, which I've shown in the proof of concept by altering the background color to be the secondary main palette theme (which will not be in the final product):


Error in IOS Build/Products/Debug-iphonesimulator/AEPAssurance/AEPAssurance.modulemap' not found

I am facing a problem AEPAssurance.modulemap not found. I problem comes up suddenly, it was working fine but suddenly this issue comes, I have been tried all the solutions but not able to fix it, kindly help me to fix this issue.
Xcode version 15.2, I have been tried on older version as well but not able to fix it.
ruby version: 2.7.6
pod version: 1.12.1
node version: 18.5.0
React Native Version: 0.71.4

I am getting the same issue.
pod 1.14.3
node v19.9.0
react-native 0.72.5
ruby 3.2.2


VSCode does not set a user agent when attempting to retrieve $schema JSON Schemas

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.79.2
OS Version: Darwin x64 22.5.0

Steps to Reproduce:

Start a webserver at http://localhost:8080 serving a single schema at foo.json containing {"required": ["bar"]} (the precise contents don't really matter)
Open a file bar.json, and paste in:

{
  "$schema": "http://localhost:8080/"
}

Place the cursor between the quotes just after the trailing slash
Type foo.json, and look at the webserver logs, observing no user agent was sent

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.80.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I suggest to verify this in code: https://github.com/microsoft/vscode/pull/190726/commits/ac8936a9b24f350f1d28dd688b1f44940acb0a57


Builder doesn't have correct generic when initialized through class-string

Larastan Version: 2.9.2
Laravel Version: 11.1.1

Description
When you initialize a Builder through the ::query() method on a class-string, the generic on the Builder isn't correctly deduced from the generic of the class-string, and defaults to Model. This can be easily tested by trying to access a local scope on the query, as demonstrated by the code below. This results in a PHPStan error: Call to an undefined method Illuminate\Database\Eloquent\Builder::test()..
It's debatable whether this is a bug or rather a feature request.
Laravel code where the issue was found
<?php

namespace App\Models\Example;

use Illuminate\Database\Eloquent\Builder;
use Illuminate\Database\Eloquent\Model;

class Example extends Model
{
    public function scopeTest(Builder $query): void {}
}

/** @var class-string<Example> $className */
$className = Example::class;
$className::query()
    ->test()
    ->get();

:+1: I'll try to get to this in the next week or two
A similar issue I've noticed is calling static::query() does not handle the static instance:
class AbstractModel extends Model
{
    public static function new(): static
    {
        return static::query()->create(); // this fails because returns AbstractModel instead of static(AbstractModel)
    }
}

class User extends AbstractModel
{
}

I try help for first part of this issue, but currently can't understand how fix this:

 return static::query()->create(); // fails, returns AbstractModel instead of static(AbstractModel)


Project fails to build due to isnan issue in MADP

The project fails to build if MADP is downloaded automatically, with this error trace:
prob.h:531:30: error: 'isnan' was not declared in this scope, and no declarations were found by argument-dependent lookup at the point of instantiation [-fpermissive]                      if( isnan( _p[i] ) )
This is a common issue when the older code is not up to new C++ standards, and the issue was resolved in the MADP with the latest commit (MADPToolbox/MADP@a6c1bb7), however this project downloads an earlier release of the MADP.
A dirty hack is to edit MADP source files after the download, however the best solution would be to autodownload the latest version od MADP code. At the very least, this information could be added to README of this project.

Thanks, I updated the project to download a version of MADP that should compile with newer compilers.


Update stephan-dev branch so that demo is up to date and stable

[x] Add "to be completed" or similar to the cite button
[x] Update readme in main/packaging/stephan-dev to include new demo link: https://datalad.github.io/datalad-catalog
[x] Make sure all .at() functionality is replaced
[x] Add poster to "About" page
[x] Fix links pointing to old "data-browser" repository
[x] Update

These updates have now been made and the current demo resides at: https://datalad.github.io/datalad-catalog
It builds from the stephan-dev branch, which should probably be cleaned up and renamed to a demo or similar in future.


Maintenance

Is this project still being maintained?

It is not

you can find a fork with up to date dependencies here: https://github.com/vstirbu/cloak


Error: Column start (x) greater than line length (y)

I get these errors all the time in various languages. What causes them and how can I stop them?

They are almost always caused by the linter program (eg, stylelint) providing an invalid column. It could also potentially be the provider (linter-stylelint) not translating the ranges properly, but I don't know of any plugin that currently exhibits that bug.
In your screenshot it looks like linter-stylelint is the provider currently causing that to be thrown, can you put that .scss file up somewhere so we can verify where the issue is?

Sorry, I closed the errors and proceeded to modify the sheet a lot. Next time I get one of these errors I can post the file and the exact spot it's happening.

The error in your screenshot was triggered because stylelint reported an error as starting at column 13... on a line that only has 12 characters. Since this is past the end of the line it couldn't possibly be a valid value, so atom-linter.rangeFromLineNumber threw an error.

@Arcanemagus and @1000ch (main contributors) i get this error often and want say (we need to fix it in linter-stylelint) that we fixed this error once and forever(i hope) in eslint-plugin-html by this commit and here is original issue, maybe it helps to solve it.

lol, the issues. XD

@arturparkhisenko The error is merely saying the linter program (be it stylelint or eslint + eslint-plugin-html) is feeding invalid data back to us. The cause of that varies widely, from your issue of not properly handling line endings, to things like not stopping pulling lexemes in when trying to tokenize a string when you hit an error state (esprima based JS linters), and coincidentally stylelint in this case if I remember right.

@Arcanemagus ow, but when i change line endings i didn't get this error, thats why i get there with my idea :) anyway thanks for explanation, i'm agree with you about that.

Ah, sorry, thought you were the author of eslint-plugin-html till I just looked at those again in some detail :stuck_out_tongue:.

Should this be closed?

There hasn't been a rash of reports as the initial splurge of issues have either been fixed or silently ignored till the linter providers update, so this doesn't really need to serve as a catch-all any longer.


chore: install extras from ragbits index package

Description

Extras from subpackages are now included in the ragbits package.
Modified scripts/update_ragbits_package.py to sync extras versions in ragbits with those declared in subpackages.

When selecting package to update as ragbits-core the extras logic is not triggered.
Usually we run release on core, as it's updates all the packages which depend on it + updates ragbits package, so it's important to run extras logic also in this case.


Fmt

Current dependencies on/for this PR:

main

PR #57   👈

PR #54

PR #55

PR #56

This stack of pull requests is managed by Graphite.


svlint: 0.4.16 -> 0.4.18

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/dalance/svlint/releases.
meta.description for svlint is: "SystemVerilog linter"
meta.homepage for svlint is: "https://github.com/dalance/svlint"
meta.changelog for svlint is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.4.18 with grep in /nix/store/24408vplglrpq7w6k125f3z7iazybwbl-svlint-0.4.18

directory tree listing: https://gist.github.com/fde82ae87fbb0f8e496e031286f0c3a5

du listing: https://gist.github.com/551e49cf85ad04030f4e9b2b288b9577

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
svlint

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/24408vplglrpq7w6k125f3z7iazybwbl-svlint-0.4.18 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A svlint https://github.com/r-ryantm/nixpkgs/archive/a7e2baebaaf6e46e1cc3c5a48d2d198557e1ab9a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/24408vplglrpq7w6k125f3z7iazybwbl-svlint-0.4.18
ls -la /nix/store/24408vplglrpq7w6k125f3z7iazybwbl-svlint-0.4.18/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

svlint

Maintainer pings
cc @trepetti for testing.

Result of nixpkgs-review pr 116201 1

1 package built:

svlint


allow Authority as relationship name

required for: https://github.com/jhipster/generator-jhipster/issues/11062
Please make sure the below checklist is followed for Pull Requests.

[ ] Checks are green
[ ] Tests are added where necessary
[ ] Documentation is added/updated where necessary
[ ] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

Where are we on this?

@MathieuAA : still missing unit tests I think

I'll will proceed this topic this week. Just back from winter vacation

@xetys nice! thanks

sry, took some time, hope the very basic tests inspired by the user relation tests are fine
If something is missing, feel free to ask for changes

Thanks @xetys :)