File Structure

Hello, I've translated this page File Structure from FAQ section. Please check it out and if possible merge it. Thanks.

I close this PR due to this file path does not exist in new website.


Remove support for cheap sphere-map reflection option

We have two chunks to evaluate sphere-map reflection:

reflectionSphere.js
reflectionSphereLow.js

The low version is used when there are less than 16 fragment uniforms available., but this no longer compiles as the fragment is not maintained. I suggest we simply remove it and use the reflection of the better quality at all times.
Current issue with the chunk:
vec3 calcReflection(vec3 reflDir, float gloss) {
    vec3 reflDirV = vNormalV;    /////////////// vNormalV is not defined, it was likely renamed in the past.

    vec2 sphereMapUv = reflDirV.xy * 0.5 + 0.5;
    return $DECODE(texture2D(texture_sphereMap, sphereMapUv));
}

Doesn't this suggest we're guaranteed to have a minimum of 64:
https://web3dsurvey.com/webgl/parameters/MAX_FRAGMENT_UNIFORM_VECTORS

DELETE IT! 😆


Add possibility to override serializers

A proposal to make it possible to add own ways how an element is displayed on the screen. Open for discussing how it finally will look like.
Closes #8

@Spoutnik97 I updated this PR. No worries if you have more important things going on, just wanted to ping in case you didn't get notified about it.

Thanks @bitboxer, I missed the github notification :)


Spiderfy at current zoom if all child markers still in single cluster at maxZoom

Current code in _zoomOrSpiderfy spiderfies at current zoom only if all child markers are at the exact same position (i.e. cluster bounds is a point) (or if we are at max zoom already obviously).
That means that if markers are at slightly different positions, but if they are all still in a single cluster at maximum zoom, clicking on the cluster will zoom to that max zoom level, but user will still see the "same" cluster, and no markers. And he/she will need a second click to trigger the spiderfy. This is exactly what the current code was trying to avoid.
Since current MCG does not display any clue about the true markers position anyway (spider legs point at the cluster position, not at true positions), there is no point zooming to maximum zoom in that case either.
This PR checks if all child markers are still within a single cluster at maximum zoom, instead of checking for cluster bounds being a point.
Demo of the 2 different behaviours here: http://jsfiddle.net/ve2huzxw/50/
I also included 2 extra tests in spiderfierSpec test suite to cover these cases.

Good stuff :)


Added TellKontrolWithTimeout Kite Method

In order to allow applications to implement their own application specific Kontrol calls, a TellKontrolWithTimeout method has been added.
cc @fatih

LGTM. I'm going to merge it as it passes both 1.4 and 1.5


swarm join with Etcd fails

I have tried swarm join with the following  etcd protocol syntax only the last syntax worked.
docker run --name swarm-node swarm join --addr=<IP_ADDRESS>:2375 etcd://<IP_ADDRESS>/mod/swarm
time="2015-05-05T16:29:10Z" level=fatal msg="501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]"

$ docker run --name swarm-node swarm join --addr=<IP_ADDRESS>:2375 etcd://<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>/mod/swarm
time="2015-05-05T16:33:36Z" level=fatal msg="501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]"

$ docker run --name swarm-node swarm join --addr=<IP_ADDRESS>:2375 etcd://<IP_ADDRESS>:4001,<IP_ADDRESS>:4001,<IP_ADDRESS>:4001,<IP_ADDRESS>:4001/mod/swarm
time="2015-05-05T16:40:12Z" level=fatal msg="501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]"

$ docker run --name swarm-node swarm join --addr=<IP_ADDRESS>:2375 etcd://<IP_ADDRESS>:4001/mod/swarm
time="2015-05-05T16:32:44Z" level=info msg="Registering on the discovery service every 25 seconds..." addr="<IP_ADDRESS>:2375" discovery="etcd://<IP_ADDRESS>:4001/mod/swarm"

<IP_ADDRESS> is the leader host. Let me know if I'm missing any details while doing swarm join

Hi @smothiki, I encountered this same issue last week using the etcd docker image (coreos/etcd), with the old and new version of etcd (0.4.6 and 2.0). The only way for me to test the discovery locally was to run a single local instance running the etcd binary directly as a standalone server. (Note: even running vagrant-etcd which setups a local cluster instance was returning the error)
So I think it's related to etcd somehow (coreos/etcd#509, GoogleCloudPlatform/kubernetes#3853, GoogleCloudPlatform/kubernetes#3852)
How much memory do you assign to your etcd instances? It seems that other etcd instances are OOM'ing and that only the leader is up in your case. In my case the leader was probably OOM'ing while followers were up. Try to assign more memory to your containers or VM setup, that might probably help although I'm not entirely sure.

Note: I was not even running swarm join but just using curl against the etcd API.

Okay I'm running swarm on Core-OS latest stable release. I'll dig deep on the error

I'm having a similar problem in that I can only get the join to work if I specify the current etcd cluster leader. Perhaps the issue is the client never runs SyncCluster after it's initialized?

Tried to find some more documentation regarding SyncCluster but it's a bit sketchy.

Should we call it when we initialize the store?
Before every operation?
Periodically in a goroutine?

@smothiki Can you try against master? Should have been fixed by #782

There are two problems here I guess:

The first one being the --addr and --bind-addr flags being misconfigured which completely hinders the Sync process. (see: https://groups.google.com/forum/#!topic/confd-users/WM4a4LfPLaY)
The second may be related to calling SyncCluster with the client. (hopefully fixed by #782, but the questions of whether we should call it before each operations remains open)

@aluzzardi @abronan I'm going to get someone from CoreOS to review this, but I think we might want to consider making a requirement that people use etcd 2.x for swarm discovery.
Recent changes in go-etcd may also help with some of the issues listed in this issue.

@aluzzardi If you know your cluster configuration is changing periodically, then you need to call syncCluster periodically accordingly.

@aluzzardi To be clear, cluster configurations in this case would be adding and removing etcd members. Leader elections should not require a cluster sync.

@kelseyhightower @xiang90 Thank you for the clarification. As of now we use v0.4.6 for the integration tests, we might switch to v2.0.6 then. Regarding the sync it is safe then that we add the periodic syncCluster for those who manage their own Etcd cluster. A 10 minute refresh sounds reasonable compared to the cost of having an outdated configuration in the long run.

@kelseyhightower thanks for chiming in, we really appreciate it.
Given the features in our roadmap, moving forward we're going to use etcd for a lot more than just discovery (persistent state storage and leader election are currently being worked on) so it's great if we can get feedback from your side (for instance, make etcd 2.x a requirement as you mentioned).
If you don't mind I'll ping you on relevant PRs going forward

@aluzzardi I am happy to discuss more about etcd with you guys. We also plan to make etcd to store more data and be more reliable.
We have a great roadmap to work on. It would be great to listen your use case and requirements.

@smothiki Could you retry with master?
We have:

Updated our integration tests to run against v2.0.11 #801
Updated our go-etcd library #801
Called SyncCluster #782

Sure. I'll retry with latest code and update in the comments

@aluzzardi @smothiki Just a reminder: I removed the call to SyncCluster in #801 because this was breaking integration tests after updating etcd to v2.0.11. Please tell us if something goes wrong so we can fix this ASAP.

Just tested the syntax is working with just etcd IP

@smothiki So this still does not work with multiple IPs right?

Just tried with the latest swarm code and its not working.
docker run --name swarm-node swarm join --addr=<IP_ADDRESS>:2375 etcd://<IP_ADDRESS>,<IP_ADDRESS>/mod/swarm
time="2015-05-27T03:32:05Z" level=info msg="Registering on the discovery service every 20s..." addr="<IP_ADDRESS>:2375" discovery="etcd://<IP_ADDRESS>,<IP_ADDRESS>/mod/swarm"
time="2015-05-27T03:32:05Z" level=error msg="501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]"

docker run --name manage -p 2395:2375 swarm manage  etcd://<IP_ADDRESS>,<IP_ADDRESS>/mod/swarm
time="2015-05-27T01:14:20Z" level=info msg="Listening for HTTP" addr=":2375" proto=tcp
time="2015-05-27T01:14:20Z" level=error msg="Discovery error: 501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]"
time="2015-05-27T01:14:20Z" level=error msg="Discovery error: Unexpected watch error"

docker -H <IP_ADDRESS>:2395 info
Containers: 0
Strategy: spread
Filters: affinity, health, constraint, port, dependency
Nodes: 0

Thanks @smothiki. I will investigate on this. Which version of etcd were you using? We recently switched to v2.0.11 (latest) but the call to SyncCluster seems to panic for some reasons.
That would be nice if you could also provide us with the complete command you used to launch etcd. That would help to solve the issue with SyncCluster.

I'm using etcd 0.4.9 which comes with latest core-os release
https://coreos.com/releases/#647.0.0

@smothiki we pushed a fix in #867 in time for the rc2. Just tested with a fresh etcd v2.0.11 cluster and it works fine. I haven't tested on older versions of etcd though.

I'm closing this. @smothiki Let us know if you encounter any other related issue and thanks again for reporting!


Add node duration metric to monitoring addon

Description
It would be interesting to track the duration of certain nodes, as we are tracking process instances duration and all work items duration
Implementation ideas
In BPMN and SWF parsers, add a metadata key called "Metric" to those nodes that it will be interesting to track duration for.
In monitoring addon, add a new Distributionsummary object called ""kogito_node_instance_duration_seconds" which will record duration for those nodes flagged with the metadata key.

@elguardian Should I change the BPMN parser to add the metric metadata to certain nodes (in that case we need to dicuss which ones)  or do you prefer to do that with a follow up issue?

there is no need to change the parsec... the editor has already a metadata section. so you just need to take that into account in the runtime

there is no need to change the parsec... the editor has already a metadata section. so you just need to take that into account in the runtime

Sure, I mean, add to the parser logic a call to metadata ("metric", "") for those nodes which duration should be recorded


Issue # 885 fixed hover issue on feedback page in quick links and contact info section

Fixed hover issue on feedback page in quick links and contact info section

Created PR for Issue # 885


Notebook crashing with docker image

OS
Ubuntu 18.04.2 LTS
Docker Image:
docker pull udacity/carnd-term1-starter-kit
Docker Run:
docker run -it --rm --entrypoint "/run.sh" -p 8888:8888 -v ~/Documents/Github/Self\ Driving\ Car\ Engineer/CarND-Camera-Calibration/:/src udacity/carnd-term1-starter-kit
Error Lines:
%matplotlib qt
Error:
Kernel Died

Though we are a little late to respond, we are closing out the issue because it has already been addressed in this issue.
Thank you.


The name 'editor' does not exist

I am a newbie but why do I keep getting "The name 'editor' does not exist in the current context" for NotesPage.xaml.cs?

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin.Forms. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI is the replacement for Xamarin.Forms, and it's documentation can be found here.


How to handle vendor prefixes?

I've gone back and forth on how to handle vendor prefixes here. In the past week, I've made a commit that removed them entirely, then went back and merged a pull request that elegantly put them back in with mixins.
The question is (1) whether we're all cool with mixins as a means for handling vendor prefixes or (2) whether there is another preferred method?
I'm asking the question out in the open because the intent of this is to keep things as unobtrusive as possible. By that, I mean anyone should be able to drop this into their project and have it work. To illustrate that point, I personally prefer using Grunt/Autoprefixer to handle prefixing but know that would require making choices that have an impact on other people's projects and I'd hate for that to happen.
Either way, I'll put it out there. Just want some healthy dialogue and nothing else. :)
Cheers!
Geoff

I think that mixin is probably the best (and idiomatic) way to handle vendor prefixes.
In any case you can add some global config option for mixin to generate only non-prefixed properties and then process generated css with Autoprefixer.
And thanks for work :)

I think the best approach is to leave this to the consumer of the library, using Autoprefixer for example. Otherwise you have to make assumptions about the environment that animate.scss is used in.

I agree with @wmadden. Prefixing is a task best left for the build process.

:+1: +1 @wmadden.

Nice, thanks for the feedback everyone! I think this provides some good ammo for future updates.


Issue 731/add stormglass utils

Nice start @PysarenkoDS-BWT, a few things:

let's add a description to your PR
similarly to sofar.ts, we need our utility functions to return values that are formatted in a way that our API and frontend can understand. For example, data needs to be ready to be added to our forecast table. You can have a quick look at https://github.com/aqualinkorg/aqualink-app/pull/729


Works with .doc / .docx attachments crash viewer / cannot be downloaded

Describe the bug
CORE deposits with docx or doc attachments failed to migrate. (Or I have been unable to find examples of successful migration.)
To Reproduce

SSH to kcworks-prod
Connect to the UI container: docker exec -it 4028a614fad1 /bin/bash
cd /opt/invenio/var/import_data/import_files
ls *.doc / ls *.docx Eg. o_1a40ns08si5p1buq1koc1eai1rb67.doc.applied_blake_miltons_response_to_empire.doc
Find corresponding CORE deposit (usually by intuiting deposit title from filename) Eg. Applied Blake: Milton's Response to Empire
Search for deposit on Works. Eg. https://works.hcommons.org/records/eak7n-gxp03
Click on deposit
Viewer does not appear
Click on Download
Goes to: https://works.hcommons.org/records/undefined/undefined?download=1

Example .docx : https://hcommons.org/deposits/item/mla:103/ / https://works.hcommons.org/records/ewxqw-1hm19
Deposits that seem not to have migrated at all:

https://hcommons.org/deposits/item/hc:57493/
https://hcommons.org/deposits/item/hc:57723/
https://hcommons.org/deposits/item/mla:955/
https://hcommons.org/deposits/item/mla:959/
https://hcommons.org/deposits/item/hc:32859/
https://hcommons.org/deposits/item/hc:32855/

Expected behavior
The work would be viewable and downloadable

The files have migrated properly and can be downloaded directly with api calls. The issue is that the links on the detail page are wrong. This is happening somewhere in the UI data handling, since the 'link' values coming from the back end are correct.


Fix the order of the rex2 registers for r_ib_unsigned_gpr

Fixes #1423
r? @abrown

https://github.com/bytecodealliance/wasmtime/issues/1423#issuecomment-605295620

Could you add an x86_pextr binemit test

Is the last commit ok, or do you want to test more cases?

With this I can finally store SIMD vectors in SSA values, rather than memory. This improves performance for tokei from ~50% overhead compared to cg_llvm to ~30% overhead.

I don't know if this will give you more flexibility, but note that in #1377 I have added support for declaring constants in the function preamble instead of only through vconst. This also should make it easier to use constants of sizes other than 128-bit. Take a look at
64c8cbf and let me know if you can think of any changes to that that would help cg_clif.

I don't think any of that PR will help me. At least for now. Creating SIMD values is currently very slow anyway as at MIR level all lanes get assigned separately, meaning that I have to use vconst 0 + insertlane. I will obviously need to optimize this :)


fix error message thrown

fixes #170

Captures the exception when user inputs an invalid column into the CSV file.
Improve code quality for import function.

Codecov Report

Merging #198 (a4e3ff2) into master (fd95fa0) will decrease coverage by 1.18%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #198      +/-   ##
============================================
- Coverage     65.60%   64.41%   -1.19%
  Complexity      790      790
============================================
  Files           103      103
  Lines          2817     2869      +52
  Branches        400      411      +11
============================================
  Hits           1848     1848
- Misses          806      858      +52
  Partials        163      163

Impacted Files
Coverage Δ

.../main/java/seedu/address/commons/util/CsvUtil.java
0.00% <0.00%> (ø)

...va/seedu/address/logic/commands/ImportCommand.java
33.33% <0.00%> (-2.39%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


--write-thumbnail parameter doesn't work

https://github.com/rg3/youtube-dl/blob/master/README.md#thumbnail-images
With this option, the thumbnail isn't downloaded

This does not seems to work properly with current binary either e.g. ../bin/youtube-dl --write-thumbnail https://www.youtube.com/watch?v=t2oVUxTV4WA would download image and video at the same time.

@yannbertrand have a look at 1.12.2 master and https://github.com/przemyslawpluta/node-youtube-dl/blob/master/example/thumbs.js


Add CSS properties for HTML() into-function

Based on question from StackOverflow: Set condition for innerHTML of a table
If you use HTML() function it is a good idea to give a functionality to change a color or alignment of result cells. Something like:
    alasql('SELECT TOP 10 City \
                    TH {style:{textAlign:"right",textContent:"Town"}}  \
                    TD {color:(CASE WHEN Population > 1000000 THEN "red" ELSE "green" END)} \
               INTO HTML("#res") FROM ? ORDER BY Population DESC',[cities]);

The information, required for table formatting:

Column width
Column header
TH class
TD class

Another option: use Alasql together with Handsontable grid, like:
    alasql('SELECT * INTO HOT("#res") FROM ?',[City]);

"And i want to ultimately achieve the exported excel file's header format like changing color, font style etc,. Is it possible by using alasql to do this. Expecting your valuable response as soon as possible."
I have got great idea how to change Excel file, and will write this later. It will looks like this:
    var x = alasql('SELECT * INTO XLSX() FROM ?'.[data]);
    x.Sheets['Sheet1'].cells['A1'].value = 10;
    alasql.saveBinaryFile(x);

Mixing css and SQL. Hmmm. Would it be better to be able to set class of the table, the header row, the header cells (according to content), the data row and the data cells (according to content)?

I already have some insights this night :))))) To solve this problem (it was requested by many people) I opened another repository, for the library which can export data to XLS file with formats and colors.
https://github.com/agershun/alaxls
This library will work with AlaSQL something like:
    alasql('SELECT a TH {class:"header"} TD {class:IF(a>=0,"positive","negative")} \
             INTO ALAXLS('myfile.xls',{headers:true, headerrange:"B1", rowsrange:"D1" }) FROM ?',[data]);

Plus people can save JavaScript arrays without alasql.js on AlaSQL manner (with SaveFile buttons support):
    alaxls.saveFile('myfile.xls',data,options);

And it can work together with js-xlsx.js library as export function with colors and formats.
Hope it will take day or two to rewrite existing code to the library.

Alternative idea of syntax:
    var opts = {
         sheetid: 'Sheet1',
         thclass: 'header',
         tdclass: 'cell',
         columns: [
              {
                   columnid: 'a', tdstyle: function(row, column, value) {
                         if(value < 0) return "background-color:red; color:white"
                    },
                   width: 200
              } ,
             {
                   columnid: 'b', title: 'Amount', style: 'font-wight: bold'
             }
         ],
         caption: 'My Data',
         caprange: 'A1',
         thrange: 'B1',
         tdrange: 'D1',
         css: '.header {color:yellow} .cell {color:blue} .caption: {font-size:200px}'
    };

    var data = [{a:1, b:10},{a:2,b:20}];
    alasql('SELECT * INTO ALAXLS("myfile.xlsx",?) FROM ?',[data,opts]);

Really like this approach

Agree, it is more universal than to extend SQL and it can live without AlaSQL and js-xlsx.
Plus it can be used outside AlaSQL with one simple function:
    var xml = alaxls(undefined, options); // return XML with data
    var xml = alaxls(options);                   // return XML with data
    alaxls(filename, options);                   // save file to the disk

Where filename is:

path to the file (in Node.js)
in browser it will open a Save File dialog
if starts with # then send result to the proper DIV of DOM

Options:

caprange
thrange
tdrange
caption
capclass
thclass
tdclass
class
capstyle
thclass
tdstyle
columns
width
title
class/style...
format
formatStyle: - Excel format

Each element can be a function or value (string or number).
tdstyle: function(row,columnid, value) {
      return 'color:red';
}

Other options:

format: 'xml'/'html table'
sourcexml: xml-string
query: '#id' - place in source xml where to put table

Another option: use as a source ExcelJS library. Unfortunately, it works with Node.js only.

I think that there is no need to create additional XLSX function. Probably, It is better to:
a) replace js-xlsx for export function (this is not to hard)
b) create an option to use js-xlsx or alaxls, like:
    alasql('SELECT * INTO XLSX("mydata.xlsx",{engineid:"ALAXLS",headers:true})");

or
    alasql.options.XLSX.engineid = "ALAXLS";
    alasql.options.XLSX.engineid = "js-xlsx";
    alasql('SET XLSX = ALAXLS;");

@agershun have you looked at how Excel represents the HTML?  On OSX, we have a neat little tool to inspect the clipboard: https://www.npmjs.com/package/pb -- you can play with it in Excel 2011 by copying a block of cells and running pb html to see the html clipboard.
You don't actually have to generate inline styles for every cell.  Excel actually emits CSS classes and uses them in the code (with names like .xl63 -- that number actually corresponds to the internal style number in XLS and XLSX).  This way, the user could supply his or her own template CSS with standard class names like col1

@SheetJSDev Thank you for the  link!  I need to think about it a little, and write you. I just have finished night run for coloring cells, and have many questions to discuss after day sleep :)
Great news!
Yes, yes, yes, I did it:, I add colors to XLS() function on very primitive way.
http://jsfiddle.net/agershun/95j0txwx/2/
How it works:

XSL() uses own very simple engine with many restrictions and does not use js-xlsx library. Actually it simply saves table in one HTML format without GZIpping, etc.
XLSX() uses more advanced engine (js-xlsx), but it does not support colors and styles (probably, i would like to discuss it with SheetJS developers how to implement this functionality with library)

This is example of style syntax:
    var opts = {
        sheetid: 'My Big Table Sheet',
        headers: true,
        caption: {
          title:'My Big Table',
          style:'font-size: 50px; color:blue;' // Sorry, styles do not works
        },
        style:'background:#00FF00',
        column: {
          style:'font-size:30px'
        },
        columns: [
          {columnid:'email'},
          {columnid:'dob', title: 'Birthday', width:300},
          {columnid:'name'},
          {
            columnid:'name',
            title: 'Number of letters in name',
            width: '300px',
            cell: {
              value: function(value){return value.length}
            }
          },
        ],
        row: {
          style: function(sheet,row,rowidx){
            return 'background:'+(rowidx%2?'red':'yellow');
          }
        },
        rows: {
          4:{cell:{style:'background:blue'}}
        },
        cells: {
          2:{
            2:{
              style: 'font-size:45px;background:pink',
              value: function(value){return value.substr(1,3);}
            }
          }
        }
    };

    var res = alasql('SELECT * INTO XLS("restest257b.xls",?) FROM ?',[opts,items]);

Friends, any feedback is very appreciated!

@SheetJSDev I installed the package. Thank you! Now many things looks more clearly.
In this simple realization I used very lazy approach (without styles, gzipping, etc.). It is rather a simple concept of spreadsheet template.I still didn't learn how to add colors and styles to js-xlsx, but would like to to move this 'temporary realization' back to js-xlsx to reduce support work.

Now AlaSQL can save files to Excel and LibreOffice formats with colors and other
Excel formatting functions. This function also work in IE9. This is a code sample:
    var mystyle = {
      headers:true,
      column: {style:{Font:{Bold:"1"}}},
      rows: {1:{style:{Font:{Color:"#FF0077"}}}},
      cells: {1:{1:{
        style: {Font:{Color:"#00FFFF"}}
      }}}
    };
    alasql('SELECT * INTO XLSXML("restest280b.xls",?) FROM ?',[mystyle,data]);

See the working example in jsFiddle


in_tail: Handle records in the correct order on file rotation

This is the fix for the issue reported by the bug report #1055.
Problem
It turned out that this bug was a matter of the calling order of functions in TailWatcher.on_rotate().
In particular, when TailWatcher detects a log rotation and immediately finds a new file, it performs the following steps:

Create a new TailWatcher for the new file.

In this process, it reads the new file and emit its contents.
This all happens before proceeding to (2).

Dispatch the on_notify() for the archived file.

In this process, it reads the content of the old file and emit its contents.

So naturally, newer log records (written to the new log file) will be emitted before older records in the archived log file.
Solution
This patch fixes it by enforcing flush of older log records before performing the step (1).

Note that this bug occurs only when TailInput is slow to notice the file rotation.
In most cases, it does not occur because a log rotation is detected immediately after
the file rename. In this ordinary case, a file rotation will be handled as follows:

Detect the rename of the log file and set up a new TailWatcher.

In this step, the new file is NOT created yet, so nothing is emitted.

Dispatch the on_notify() for the archived file.
Detect the creation of the new log file.

In this step, the content of the new file gets processed.

Since it handles the content of the old file in the step (2) and the content of the new file
in the step (3), the output order of records is preserved.

Thanks for the investigation and patch. Just merged :)

Thanks for merging it!


[example] Upgrade gatsby to v2

Closes #12320

I did this, too. But I had to change another line in the withRoot() function in withRoot.js:
Old content of the line:
this.pageContext = this.props.pageContext || getPageContext();
This is what I changed there:
this.pageContext = Object.assign(this.props.pageContext ? this.props.pageContext : {}, getPageContext());
The problem was that the page context was not null but an empty object {}. That way, a new page context including the MuiTheme was NEVER created! Material-UI had no chance to use the correct theme and JSS.
After the change, everything was rendering just fine with Gatsby V2.

@mattes3 It's good to know.

This is my temporary solution https://github.com/gatsbyjs/gatsby/issues/6127#issuecomment-409736770
Note that I am using my Layout component above the page component.

I have updated the source code so we support both Gatsby v1 and v2.
@blukai It's a great first pull request on Material-UI 👌🏻. Thank you for giving it a shot!


Config reload and client version information in keepalives

Sean, Hi. I hope you are well. A few things I wanted to discuss before formally raising them:
...
[1.] In a context where standalone checks are automatically generated by a "system", we need to re-start the client to take those (i.e.: new ones, remove discarded, etc...) into account. In this context a feature to automatically reload the configuration would be more than handy. For example, a re-start could, let's say, interrupt the execution of a hundred checks and that would be quite harmful. What do you think?
[2.] It would be really convenient to store and display (i.e.: Redis and Sensu dashboard respectively) the client version and OS type. We currently have a configuration driven way to set the OS and a deployment orchestration framework for mass deployments. (i.e.: version consistency) This being said, that feature won't harm at all; visibility, etc...
Any thoughts?
...
Regards, Andrei

We currently include the Sensu client version in client keepalives, https://github.com/sensu/sensu/blob/master/lib/sensu/client/process.rb#L43

Closing due to inactivity, and because part of this request was resolved w/ Client keepalive client version info. Please feel free to create a new issue if this matter is not resolved. #monitoringlove


Comments are not automatically wrapped.

Single line/multi line comments are not automatically wrapped/checked by csharpier.
Input:
// Test very long line comment.
// Second line of long comment.
var someVariable = someValue;

Width = 10
Output:
// Test very long line comment.
// Second line of long comment.
var someVariable = someValue;

Expected behavior:
// Test
// very
// long
// line
// comment.
// Second
// line
// of
// long
// comment.
var someVariable =
    someValue;

I'm not opposed to this, but it may be a lot of work. Prettier has had the request for 7 years now - https://github.com/prettier/prettier/issues/265

I have a PR for this - will publish to get feedbacks.

.assign

I feel like this would break far too many things, than the few things it would help.


[New API]: Added Social Media API

🎯 Title and Issue Number
Title-[New API]: Social Media API

Closes #143
Description
This API facilitates Social Media Management, providing functionality for users to login, follow each other, post content, comment on posts, and like posts.

🗒 Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x]  I have gone through rules of when PR will get merged.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have followed proper file structure.
[x] I have gone through CONTRIBUTING GUIDELINES.md.

Thank you for contributing!❤
Don't forget to show some love by ⭐ the repository!

Hey @Varad0014, Thank you for creating PR! We will review it super soon. 😇❣
As we continue to evolve and innovate, we encourage you to share your ideas, feedback, and code to help us push the boundaries of what's possible in the world of APIs🚀.
We hope you have followed CONTRIBUTING GUIDELINES.
Thank you for being an essential part of our community. Together, we'll keep building amazing things!
Don’t forget to ⭐ our repository!
Happy Coding!✨
Will See you soon❣

@Varad0014 We're thrilled to inform you that your pull request has been successfully merged into the APIverse🚀🎁
Your hard work, dedication, and expertise have made a significant impact on our project, and we couldn't be more grateful for your contributions.😇
Your code will now be a part of our growing ecosystem, helping developers worldwide to create exceptional APIs.❣🔥
Happy Coding! ✨
Will See you soon❣


fixed issue #2711, Deserializer unwrapped field NPE

fixed issue #2711 , 反序列化含有@JSONField(unwrapped = true)的对象时，解析属性传入了一个null，而在处理的时候却把它当成了一个非空的map，从而会造成空指针。

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #2713 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2713      +/-   ##
============================================
- Coverage     80.45%   80.42%   -0.04%
+ Complexity     7099     7094       -5
============================================
  Files           145      145
  Lines         26704    26704
  Branches       6372     6373       +1
============================================
- Hits          21486    21477       -9
- Misses         3383     3386       +3
- Partials       1835     1841       +6

Impacted Files
Coverage Δ
Complexity Δ

...json/parser/deserializer/JavaBeanDeserializer.java
79.87% <100%> (ø)
299 <0> (+1)
:arrow_up:

...com/alibaba/fastjson/serializer/SerialContext.java
72.91% <0%> (-4.17%)
19% <0%> (ø)

...com/alibaba/fastjson/parser/JSONReaderScanner.java
72.61% <0%> (-1.28%)
44% <0%> (-1%)

...n/java/com/alibaba/fastjson/util/JavaBeanInfo.java
87.88% <0%> (-0.19%)
197% <0%> (-1%)

...java/com/alibaba/fastjson/parser/ParserConfig.java
83.3% <0%> (-0.17%)
202% <0%> (-1%)

.../java/com/alibaba/fastjson/parser/JSONScanner.java
76.24% <0%> (-0.17%)
499% <0%> (-2%)

Continue to review full report at Codecov.

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


Thunderbird 78.2.2 rendering issues

Describe the bug
Thunderbird 78.2.1 f56bda0c0c6ccdd405398a0ab3a5cbc75aaad405 is working.
master cc4a33d78c0fdfc8d64274179c24d8188153287a (Thunderbird 78.2.2) has rendering issues (see screenshot). Most buttons do not react to clicks and the menu is also not reacting.
To Reproduce
Switch to the mentioned commit and rebuild.
Expected behavior
Normally working thunderbird.
Screenshots

Notify maintainers
@edolstra
@lovesegfault
@nbp
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.8.8, NixOS, 21.03.20200914.cc4a33d (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 3.0pre20200829_f156513`
 - channels(root): `"nixos-20.09pre242714.a31736120c5, home-manager"`
 - channels(moritz): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

This was also a bug in Ubuntu, it was fixed there with:
https://bazaar.launchpad.net/~mozillateam/thunderbird/thunderbird.groovy/revision/563
https://bugs.launchpad.net/ubuntu/+source/thunderbird/+bug/1895898
https://bugzilla.opensuse.org/show_bug.cgi?id=1176384#c0

Upstream bug appears to be
https://bugzilla.mozilla.org/show_bug.cgi?id=1664607

Btw temporary fix would be to switch to thunderbird-bin-78 if you are not opposed to binaries


CRIMAP-97 Amend returned application

Description of change
Follow-up to previous preparation PRs, this one implements a barebones "amend" action, with a button in the certificate page so the provider can put back an application into "editable" mode (in_progress) and make changes in order to submit again the application.
Note: there are so many implementation details we are still missing that is not worth it taking this further until we have perhaps the document store.
For now is a bit smoke and mirrors but it serve the purpose of getting an idea how the user experience will look like and we can test this with providers.
I'm awaiting confirmation regarding some of the time stamps on the application, if we have to keep the originals or reset on re-submission. I think we might have to introduce a parent_id at some point, it will keep track of previously submitted applications to create kind of a history of all amendments, but didn't want to add that DB attribute until we have more clarification.
Link to relevant ticket
https://dsdmoj.atlassian.net/browse/CRIMAP-97
Notes for reviewer
Screenshots of changes (if applicable)

How to manually test the feature
To "fake" a returned application, access a submitted application certificate (URL is /completed/applications/{uuid}) and then in the developer tools in the footer click the button Mark as returned. This will take you back to the dashboard and you will see a returned application.
Access again its certificate page (same URL as before) and there should be a blue "Update application" button. Clicking this button will set the application back into in_progress status and take you to the Check your answers where you can change anything and progress all the way to the end to sign the declaration and submit again the application.

@arthurashman

Would it be possible to make the applicant's name link through again once marked as returned?

Not sure I know what you mean by that?
Is it the links for submitted/returned in the dashboard? At the moment these are broken, the PR Chris is working on will resolve them. I'm reluctant to change anything in that page, as there will be conflicts with that other work.
Regarding the copy in the certificate page, you're right, I added a comment in the design, will get any update to this copy if needed 👍

Yeah, just the link from the dashboard 404s. No problem if it isn't possible to do cleanly yet


Switch to pairwise summation for any summations in the statistics

This is more accurate at little to no cost of speed. We can follow the lead Julia and NumPy, which use the same approach by default. For the running sums, this requires a little bit of care to track our position in the recursive tree, but it is still fundamentally online as long as we know the expected depth of the tree.

This was considered, implemented, and reverted. Besides slowing everything down (50-100%, depending on the architecture), I just don't think it's a real problem, especially if we're already using double-precision.
R's summation just uses a naive accumulator, and no one raises a fuss there, so I think it's fine.


Error on intial commit to existing repo

I'm not sure if this is because it's only scanning files added during the last commit (which only contains addition of this GitHub Action):
# This workflow checks out code and scans the content changed or added in the
# last commit for offensive / exclusive terms.
# The scan will provide context on the found terms and alternatvies that can be
# used instead.

name: Inclusiveness Analyser scan

on:
  push:
  workflow_dispatch:

jobs:
  Inclusiveness-Analyser-scan:
    runs-on: ubuntu-latest
    steps:
    - name: Checkout code
      uses: actions/checkout@v3

    - name: Inclusiveness Analyzer
      uses<EMAIL_ADDRESS>      with:
        # Fails the step if a non-inclusive term is found
        failOnNonInclusiveTerm: false
        # Only scan files added or modified in last commit when checkout step has "with: fetch-depth: 2" configured
        excludeUnchangedFiles: true

I'm getting the following error from the action:
Run<EMAIL_ADDRESS>Notice: Inclusiveness Analyzer
Notice: - Scanning files added or modified in last commit
Error: ENOENT: no such file or directory, stat

Hi @jimbudarz
Can you rerun the job with debug logging enabled?

Hi @jimbudarz
Can you rerun the job with debug logging enabled?

##[debug]Evaluating condition for step: 'Inclusiveness Analyzer'
##[debug]Evaluating: success()
##[debug]Evaluating success:
##[debug]=> true
##[debug]Result: true
##[debug]Starting: Inclusiveness Analyzer
##[debug]Loading inputs
##[debug]Loading env
Run<EMAIL_ADDRESS>Notice: Inclusiveness Analyzer
Notice: - Scanning files added or modified in last commit
##[debug]git branch: add_inclusivity_check
##[debug]git commit: f343b975d478807ec46461a210c6adcbc2839050
##[debug]git fetch: +f343b975d478807ec46461a210c6adcbc2839050:refs/remotes/origin/add_inclusivity_check
##[debug]
##[debug]git show:
##[debug]
##[debug]Scanning file:
Error: ENOENT: no such file or directory, stat
##[debug]Node Action run completed with exit code 1
##[debug]Finishing: Inclusiveness Analyzer

Looks like "git show" didn't find the modified or added file.
Is the repository you're working on public? if it is can you share a link to it?
Anyhow the ENOENT is not intended and we need to look into it. (looks like "" string)

Unfortunately the repo in question is Private. Please let me know if I can gather any other information for you.

git show shows me that my latest commit actually only renamed a file, resulting in no added or modified files and an empty git show output.
If you're using git show to list changed files, maybe it would be better not to filter them at all and treat any change as a changed file.

@jimbudarz as we need to properly handle cases where git show returns an empty string I've released v1.0.1.
Can you crosscheck this solves the issue on your end?
Do you still feel that we should treat rename as a changed file and (re)check the content?

I don't mind either way with respect to renamed files.

Thanks @jimbudarz!
waiting for your feedback to see if the new versions solves your issue before closing.

Thanks @jimbudarz! waiting for your feedback to see if the new versions solves your issue before closing.

LGTM!


add nested syntax

Improved symbol list:

Fixed sublime's Goto Definition feature.
Should work for types, functions and variables:

closes #100

closes #53

Looks good, thanks a lot !


Make protodepUp a dependency of PB.generate

This change will ensure protoDepUp runs when running protocGenerate to generate sources, and without compiling.

Nice, thank you!
Please merge master, as I fixed the GH Action that broke on PRs from forks.

Nice, thank you!
Please merge master, as I fixed the GH Action that broke on PRs from forks.


Problem Connecting Debug Dashboard to Websocket Server

Hey I followed your guide/docs for installation and setup.
I run the php artisan websockets:serve command, travel to the debug dashboard url, select my app and click on connect.
I then receive the following error in the console:
WebSocket connection to 'wss://hipm.test/app/24e751527b7716d6c2be?protocol=7&client=js&version=4.3.1&flash=false' failed: Error during WebSocket handshake: Unexpected response code: 404

Same, is because I'm (and you probably) are developing on SSL (I with valet secure) which causes Pusher JS to default to the wss protocol wich has no port set (and also won't work without an certificate set).

Same with me, I as well am using https, I do have a cert set up to use.

This has been resolved now with new instructions added for Laravel Valet (I was having this problem too and now it's working)
https://docs.beyondco.de/laravel-websockets/1.0/basic-usage/ssl.html#usage-with-laravel-valet
However as far as I know this is also causing the statistics/graph not to work and that hasn't been resolved yet

@vesper8 the statistics issue was also resolved in version 1.0.1


Added cache to quickly identify already checked types

Hi, when we used ironmeta in our development and checked it by CPU Profiler on project, we was discovered that the "isAnonymous" method takes a very long CPU time (exact GetCustomAttributes method which used in it). The cache helps to get rid of this problem.
Thanks for IronMeta !

Awesome, thanks!  What are you using IronMeta for?

Generally we use IronMeta for named entity recognition stuff in our NLP tasks. Could you please release an updated version to NuGet?

Done!


Stable version is broken since you forced me to use insiders version that is broken too.

I can't access anything in github.dev, everthing is broken, not usable anymore since you forced me to use insiders crap version.

Version: 1.95.1
Commit: 65edc4939843c90c34d61f4ce11704f09d3e5cb6
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:109.0) Gecko/20100101 Firefox/115.0
Embedder: github.dev

\gifPlease


(DOCS-857) Update benchmark docs with information about running on agent

@samwoods1 this is a first pass at adding docs for running benchmark on an agent using what you described in DOCUMENT-857.
I'm wondering if puppetdb and the necessary pieces to run the benchmark tool are installed on the agent by default, or do you think we need to list doing that as an additional step?

Test PASSed

CLA signed by all contributors.

Test PASSed


Conisder using the .sh install rather than PPA ?

An outstanding bug seems to get in the way of raco installing additional packages from the PPA racket. The .sh installation works around this. Would you consider a PR implementing the suggested work-around?

Yes, I'd love it!


Split the authentication contract into two

Two new types: authentication-password and authentication-oauth.
 What do you mean by "proper permissions"? Also, do you think we will to migrate the old authentication contracts?

@balena-ci rebase

@balena-ci I self-certify!


allow underscore IDs in IAM resources

:nut_and_bolt: Description: What code changed, and why?
Allow underscore char in the IDs of

projects
rules
policies
tokens
roles
teams

:chains: Related Resources

#800

:+1: Definition of Done

underscores are allowed in the aforementioned entities' IDs

:athletic_shoe: How to Build and Test the Change

Graphically, start the stack including the UI,
rebuild components/authn-service (tokens)
rebuild components/authz-service (policies, projects, rules, roles)
rebuild components/teams-service (teams)
and attempt to create a token using different (in)valid IDs:

Finally, o_0 is a valid token ID!

...and one_two a valid project ID,

For projects, use an API request to add one with an underscore in its ID:
[17][default:/src:113]# echo '{"id":"new_id", "name":"new policy", "projects":["one_two"], "statements": [ {"effect": "ALLOW", "actions": [ "iam:*" ], "projects": [ "one_two" ]}] }' | chef-automate dev grpcurl authz-service -- -d @ chef.automate.domain.authz.v2.Policies.CreatePolicy
{
  "name": "new policy",
  "id": "new_id",
  "type": "CUSTOM",
  "statements": [
    {
      "resources": [
        "*"
      ],
      "actions": [
        "iam:*"
      ],
      "projects": [
        "one_two"
      ]
    }
  ],
  "projects": [
    "one_two"
  ]
}

...likewise for roles:
[18][default:/src:130]# chef-automate dev grpcurl authz-service --  -d "$(jq -n '{id: "foo_bar", name: "Foo", projects: ["one_two"], actions: ["svc:type:action"]}')" chef.automate.domain.authz.v2.Policies.CreateRole
{
  "name": "Foo",
  "id": "foo_bar",
  "type": "CUSTOM",
  "actions": [
    "svc:type:action"
  ],
  "projects": [
    "one_two"
  ]
}

:white_check_mark: Checklist

[X] Tests added/updated?
[x] Docs added/updated?

✔ changed the wording, it now fits:

looks like Only will still fit, mind adding it back in? apologies for the back-and-forth on that.

Sure! 😃

Updated the error messages:


Add isAnimated to Switch

Resolves #35

Codecov Report

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

@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   95.23%   95.23%
=======================================
  Files           5        5
  Lines         336      336
=======================================
  Hits          320      320
  Misses         16       16

Continue to review full report at Codecov.

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


Truthy / falsy helpers

To support nicer inputs a truthy helper should be implemented (to be used at the discretion of template authors) that supports the following:
True
y, yes, numeric type != 0, true
False
n, no, 0, false, ""
Invocation forms could be truthy(val) or val.truthy?

Implemented


Create Remove-DbaDbBackupRestoreHistory

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[x] New feature (non-breaking change, adds functionality) #3873
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Implement #3873

a .ps1 is missing here ?! ;-)

I'll update post merge, ty!

Sorry got too excited, I do need to make more changes, I'll ping you on the PR


Add IP Sidekick

Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[ ] Your submissions are formatted according to the guidelines in the contributing guide.
[ ] Your changes are made in the README file, not the auto-generated JSON.
[ ] Your additions are ordered alphabetically.
[ ] Your submission has a useful description.
[ ] Each table column should be padded with one space on either side.
[ ] You have searched the repository for any relevant issues or PRs.
[ ] Any category you are creating has the minimum requirement of 3 items.

Hi @hboon , thanks for the PR. Could you please update Auth so it is just apiKey? Thanks.

@davemachado Done

Thanks!


3.16: didSave breaking change with (Versioned)TextDocumentIdentifier

In rust-analyzer/rust-analyzer#6603 we found that in 3.15 the didSave notification has the following format for its parameters:
interface DidSaveTextDocumentParams {
	/**
	 * The document that was saved.
	 */
	textDocument: TextDocumentIdentifier;

	/**
	 * Optional the content when saved. Depends on the includeText value
	 * when the save notification was requested.
	 */
	text?: string;
}

But in 3.16, the format for didSave's parameters changed with:
-	textDocument: TextDocumentIdentifier;
+	textDocument: VersionedTextDocumentIdentifier;

But VersionedTextDocumentIdentifier requires a version field, making this a breaking change from 3.15 to 3.16.
Is this intended  or should OptionalVersionedTextDocumentIdentifier have been used ?

I broke this when introducing OptionalVersionedTextDocumentIdentifier. I will correct the spec.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 3242 ms

Resolved: Bofrost.at performance has improved in d265653 after 1 hour, 48 minutes.


Automatically lookup arn format.

what

Automatically lookup arn format instead of hardcoding the partition.  This is needed for use in govcloud, for example.

why

Makes the module easier to use.

references

https://github.com/cloudposse/terraform-aws-cloudtrail-s3-bucket/pull/45

/test all

/test all

/test all


Refresh challenge

We could:

[ ] Update Next.js to the pages directory

https://github.com/mui/tech-challenge-full-stack/blob/d7024ec09d831642ebebaf2fb05175063fba8234/pages/_document.tsx#L7

[ ] Upgrade React to v18

https://github.com/mui/tech-challenge-full-stack/blob/d7024ec09d831642ebebaf2fb05175063fba8234/package.json#L21

I'm looking at the new next.js + ts example and honestly, I find it a bit of a setback to the old example.

It's full of UI that belongs in a template, I want a zero UI example, just the setup for next.js + ts + MUI. At most some introduction text with useful links that I can easily delete.
It's kind of opinionated in how it sets up the folders. It would be better to leave the choice to the user whether they want @/... to resolve to the root of the project. IMO this example should just provide a layout.tsx and a page.tsx.

The example should be usable by people that want to migrate to the app dir from an existing project. They'd just want to copy+paste the relevant parts, and it's currently quite hard to find out what is the relevant code with all this template mixed in.

Honestly, I don't want to push candidates into Next.js app dir paradigm. It's so opinionated and aimed at the static website use-case. It has strong constraints on routing, it doesn't support API routes, it's vendor specific. If anything, I think for the future of this challenge we should rather aim towrads getting rid of Next.js and offer something more bare bones.

On

this is something that has already surfaced in the past with @samuelsycamore, @danilo-leal, and @mj12albert. I think there is a general consensus on this. I have added the point to #34905.

Honestly, I'm not sure we should push candidates even deeper into Next.js's paradigms. It's so opinionated and aimed at the static website use-case. It has these strong constraints on routing, it doesn't support API routes, it's all vendor specific and non-standardized. I'd like to gauge candidates capability to understand the basics of React, and the basic anatomy of web servers. If anything, I think for the future of this challenge we could rather aim towards getting rid of Next.js and offer something more bare bones.

Should we go down the Vite + express API route then?

🙂 I typed that comment because I got frustrated with the example. I removed it again after I started playing around with app dir and this challenge and it didn't feel like a good fit. But fair enough, should have signaled this 👍

Should we go down the Vite + express API route then?

I think that would be a good stack for the challenge. The only thing that holds me back from it is https://github.com/vitejs/vite/issues/12423.


HBASE-21110 Remove Illegal reflective access warning for java.nio.Bits.unaligned()

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 32s
master passed

+1 :green_heart:
compile
0m 24s
master passed

+1 :green_heart:
checkstyle
0m 26s
master passed

+1 :green_heart:
shadedjars
4m 43s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 24s
master passed

+0 :ok:
spotbugs
0m 50s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 49s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m  4s
the patch passed

+1 :green_heart:
compile
0m 24s
the patch passed

-1 :x:
javac
0m 24s
hbase-common generated 5 new + 30 unchanged - 0 fixed = 35 total (was 30)

-1 :x:
checkstyle
0m 23s
hbase-common: The patch generated 1 new + 1 unchanged - 0 fixed = 2 total (was 1)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 43s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
15m 50s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 21s
the patch passed

+1 :green_heart:
findbugs
0m 55s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
3m 18s
hbase-common in the patch passed.

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

50m 28s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1113

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 081faf2a9e4a 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1113/out/precommit/personality/provided.sh

git revision
master / 5a3ad6f08f

Default Java
1.8.0_181

javac
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/1/artifact/out/diff-compile-javac-hbase-common.txt

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/1/artifact/out/diff-checkstyle-hbase-common.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/1/testReport/

Max. process+thread count
284 (vs. ulimit of 10000)

modules
C: hbase-common U: hbase-common

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 35s
master passed

+1 :green_heart:
compile
0m 24s
master passed

+1 :green_heart:
checkstyle
0m 26s
master passed

+1 :green_heart:
shadedjars
4m 39s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 24s
master passed

+0 :ok:
spotbugs
0m 51s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 48s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 55s
the patch passed

+1 :green_heart:
compile
0m 24s
the patch passed

-1 :x:
javac
0m 24s
hbase-common generated 5 new + 30 unchanged - 0 fixed = 35 total (was 30)

-1 :x:
checkstyle
0m 24s
hbase-common: The patch generated 2 new + 0 unchanged - 1 fixed = 2 total (was 1)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 39s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
15m 45s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 22s
the patch passed

+1 :green_heart:
findbugs
0m 54s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
3m 20s
hbase-common in the patch passed.

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

50m  8s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1113

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux ca6255373eaf 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1113/out/precommit/personality/provided.sh

git revision
master / 5a3ad6f08f

Default Java
1.8.0_181

javac
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/2/artifact/out/diff-compile-javac-hbase-common.txt

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/2/artifact/out/diff-checkstyle-hbase-common.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/2/testReport/

Max. process+thread count
285 (vs. ulimit of 10000)

modules
C: hbase-common U: hbase-common

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1113/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Should I close this out @infraio  ?

Closing abandoned PR


Crazy Browser

Crazy Browser - https://user-agents.net/browsers/crazy-browser

Coverage remained the same at 100.0% when pulling 35efc75a32430fffe9e1fe6ea52b43511de14eed on ayumi-cloud:Crazy into 4899110cd2f87b01e04ced62dbb9dec541031dee on WhichBrowser:master.


How to run correctly in CI?

Please tell me how to properly assemble with CI help.
For example, for travic-ci:
language: php
dist: bionic

php:
  - 7.3

before_script:
   - vendor/bin/dusk-updater detect --auto-update
   - vendor/orchestra/testbench-dusk/create-sqlite-db
  -  google-chrome-stable --headless --disable-gpu --remote-debugging-port=9222 http://localhost &

script:
  - vendor/bin/phpunit --testsuite=Browser

Why am I doing this instead of checking? I tried a few options and they give slightly different results, which greatly confuses me, for example:

Do I need to add 'addons: chrome: stable' as specified in the laravel documentation?

If you run 'vendor/bin/dusk-updater update ', the result is:
ChromeDriver binary successfully installed for version 78.0.3904.105.

If you check with 'vendor/bin/dusk-updater detect --auto-update ', I get a warning

 ----------------- -----------
  Installed         Version
 ----------------- -----------
  Chrome/Chromium   76.0.3809
  ChromeDriver      78.0.3904
 ----------------- -----------
 ! [CAUTION] ChromeDriver is outdated!
ChromeDriver binary successfully installed for version 76.0.3809.126.

At the same time, I get strange test results, which are very different from the test on the local machine. Any help will be useful.

vendor/bin/dusk-updater update will always download the binaries regardless if you already cached the latest ChromeDriver in your CI.

Hi @tabuna,
Are you able to share the repo that you are having trouble with?
There are a few repo's using testbench dusk with Travis - one here: https://github.com/kontenta/kontour/blob/master/.travis.yml
It's a bit tricky to advise on the strange test results without seeing them.

Hi @Keoghan
Ready to provide everything you need, you helped me a year ago, now I decided to do it again.
Added for this project: https://github.com/orchidsoftware/platform/tree/dusk
The .travis file has additional instructions
Executable log::
https://travis-ci.org/orchidsoftware/platform/jobs/622844991
I get two errors:

testLogout
testEditPage

Both errors are very strange for me, the first I can get on the local machine if it is executed last. If this is indicated explicitly: vendor/bin/phpunit --testsuite=Browser --filter=testLogout. Thats all right
The second one did not repeat on the local machine, I get it only in  Travis/GitHub Action

Thanks @tabuna - I'll try to get some time today to take a look :)

Hi @tabuna,
I don't think these errors are cause by your CI setup with Testbench Dusk. I'm not entirely sure what is causing the first error, but replacing the login sequence with the 'loginAs' appears to resolve the issue with the test breaking, and I think still tests the logout functionality appropriately.
The second feels like the browser hasn't quite loaded as the assertion is run, or there is some sort of delay occuring on the CI system. Perhaps you could try waiting for the specific text to appear before the final assertion?

Hi @Keoghan

I don't think these errors are cause by your CI setup with Testbench Dusk. I'm not entirely sure what is causing the first error, but replacing the login sequence with the 'loginAs' appears to resolve the issue with the test breaking, and I think still tests the logout functionality appropriately.

You are right that this is probably not the problem of this package, I indicated it for the question of a more experienced specialist in this. No problem, we just transfer it to another place.

The second feels like the browser hasn't quite loaded as the assertion is run, or there is some sort of delay occuring on the CI system. Perhaps you could try waiting for the specific text to appear before the final assertion?

Tried to indicate pause before confirming value
//...
->press('Apply')
->waitForText('User was saved.')
->pause(100000)
->assertSee($user->name.'-async-test');
//..

This did not help me. Thank you for your time.


fix searcher factory patch

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-160-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-160/1/index.html


feat: create global notifications

Linear Ticket BAC2-41
Description:
This pull request introduces a new feature for creating global notifications within the application. The feature allows super admins to send notifications to all users, facilitating effective communication regarding important updates, such as scheduled maintenance or new features.
Changes Made:

[ x ] Implemented the endpoint POST /api/v1/notifications/global
[ x ] Added the createGlobalNotification method in the NotificationsService to manage the business logic for notification creation.
[ x ]  Integrated input validation using DTO (Data Transfer Object) to ensure that the request payload is properly validated.
[ x ] Added appropriate HTTP status codes and response messages for success and various error scenarios, including validation errors and unauthorized access.
[ x ] Included unit tests for the NotificationsService to ensure the correctness of the notification creation logic.
[ x ] Updated API documentation to reflect the new global notifications endpoint.

Testing Instructions:

Login Super Admin:

Use the /api/v1/auth/login endpoint with valid super admin credentials to obtain an access token.

Create Notification:
Use the /api/v1/notifications/globals endpoint with the access token in the Authorization header and a valid message in the request body to create a global notification.

Success Response example:
{     "status": "success",     "message": "Notification created successfully",     "data": null }

@Cobidex I added pagination and improved the users fetching by only selecting the id. This is still not optimized for medium to large applications. A better approach can be adding jobs/queue, but for that we'd need to add redis.


Images in build path

do i have to create a new entry in Vendor_files: in build.config.js.
I am using custom images to show on the screens. Please help me on this

Use vendor_files.assets

Thank you. i have put my images in src/assests


Deleting boundaries

You can't delete a Block which has clusters or a District which has blocks

We need a confirmation dialog for Delete - "Are you sure you want to delete this Cluster?" or "Are you sure you want to delete this Block?"  - Add the question mark, please.

moving to beta 1

This is done. I've added an info banner message at the top of the screen to explain why the "delete" button is greyed out. I did this in lieu of showing tooltips/popovers over disabled buttons. If this doesn't work, open a separate bug please with desired UX.

When we delete a school, cluster, block it goes all the way back to the home screen. It should go to the immediate parent's modify screen.
Likely that after deleting the school / cluster/block, the user wants to create a different one in the same hierarchy. This is also how EMS used to behave.

Fixed this.


Unconfirmed user deletion seems broken

Description - What happened? *
Unconfirmed user deletion seems to break under certain circumstances and leaves remainders in the DB, which results in a failing "Users & Roles" download.
Expected Behavior *
Users are cleaned up correctly.
Observed Behavior *
DB deletion fails with the log output below.
Steps to Reproduce
No response
Context Information
No response
Relevant log output
2024-03-12 09:41:58,064 ERROR [io.qua.sch.com.run.StatusEmitterInvoker] (executor-thread-8358) Error occurred while executing task for trigger IntervalTrigger [id=1_de.sovity.authorityportal.web.pages.usermanagement.UnconfirmedUserDeletionService#deleteUnconfirmedUsersAndOrganizations, interval=900000]: java.util.concurrent.CompletionException: org.jooq.exception.IntegrityConstraintViolationException: SQL [delete from "organization" where "organization"."mds_id" in (?, ?, ?)]; ERROR: update or delete on table "organization" violates foreign key constraint "fk_user_organization_id" on table "user"

Screenshots
No response

@jridderbusch
What is this blocked by?


Refactor beam search and update results.

Refactor beam search
As we are using a stateless decoder, the output of the decoder depends only on
the current input, so we can use the current input as a key to the cache to improve the hit rate
and reduce computation.
Update RESULTS.md
After continuing training for more epochs from https://github.com/k2-fsa/icefall/pull/162, we can get a lower WER on both test-clean and test-other.
Before

test-clean
test-other
comment

greedy search
2.77
7.07
--epoch 30, --avg 13, --max-duration 100

beam search (beam size 4)
2.76
6.97
--epoch 30, --avg 13, --max-duration 100

This PR

test-clean
test-other
comment

greedy search
2.69
6.81
--epoch 71, --avg 15, --max-duration 100

beam search (beam size 4)
2.68
6.72
--epoch 71, --avg 15, --max-duration 100

A pre-trained model is uploaded to https://huggingface.co/csukuangfj/icefall-asr-librispeech-transducer-stateless-bpe-500-2022-01-10
You can also find decoding logs in that repo.

Cool!
So the key is how many symbols of left-context, one or two, in this configuration?
Any idea why the 1-best numbers are slightly different?

So the key is how many symbols of left-context, one or two, in this configuration?

In the current configuration with --context-size 2, the key contains two symbols.

Any idea why the 1-best numbers are slightly different?

I have implemented the missing prefix-search step in beam search (see https://github.com/k2-fsa/icefall/pull/178),
but it does not seem to be helpful.
I've no idea why the WER of test-clean is nearly the same for beam-search and greedy-search.


Zoom-to-fill pinch gesture

Let users expand the video to full-screen on devices with 18:9 aspect ratios with a pinch gesture like the YouTube app has.

Sadly, I cannot support this right now as I have no test device.

What's about the Android Emulator?
Might be a slow development, but I could test it for you, if you want.

Such a feature does not only require testing during implementation but for every release build as it might break during development. Right now I cannot support this, because I think other core features and updates are more important. Especially given that Zapp is a one women project at the moment.

The folks from @TeamNewPipe had the same issue fixed a while ago.
Maybe their pulls will help as they use ExoPlayer, too:
https://github.com/TeamNewPipe/NewPipe/pull/1067
https://github.com/TeamNewPipe/NewPipe/pull/1748
https://github.com/TeamNewPipe/NewPipe/pull/1770
I'd be willing to do testing on my 18:9 device.

Thank you for the hint @cimbrico Hopefully I'll get my new phone soon and can continue development.

This issue will be fixed in version 2.2.0, which will hit F-Droid in a view days. Please reopen if you still experience any issues.

Works great for me!
How about adding a toggle for this to the settings menu?

@ghost I disabled pich-to-zoom when the aspect ratio of the video matches the one of the screen. In this case, is a setting really needed? Does the pinch gesture confuse users?
I would like to keep the settings menu as small as possible.

I think your reasoning makes perfect sense @cemrich. An additional option would be redundant.


Add @impl

All examples should have @impl Raxx.Server used correctly through out the project

Fixed with 0.12.2 release


Update gltf-pipeline files

Contains:

https://github.com/CesiumGS/gltf-pipeline/pull/631
https://github.com/CesiumGS/gltf-pipeline/pull/637

Thanks for the pull request @mramato!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.


⚠ Shadow Does Streams has degraded performance

In adf7d7c, Shadow Does Streams (https://stream.everythingbagel.me) experienced degraded performance:

HTTP code: 200
Response time: 3242 ms

Resolved: Shadow Does Streams performance has improved in b91d9fe after 5 minutes.


TTY not properly reset after brew install

brew config output
% brew config
HOMEBREW_VERSION: 3.2.10-78-gf0dbd8d
ORIGIN: https://github.com/Homebrew/brew
HEAD: f0dbd8d9a5e1db3de1fb495039e751a9a816b5ee
Last commit: 5 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 68e4bdd5e77e898f3884376f09e5a9a8dbc3b20a
Core tap last commit: 4 hours ago
Core tap branch: master
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit arm_firestorm_icestorm
Clang: 12.0.5 build 1205
Git: 2.30.1 => /Applications/Xcode.app/Contents/Developer/usr/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 11.3.1-arm64
CLT: <IP_ADDRESS>.1.1623191612
Xcode: 12.5.1
Rosetta 2: false

brew doctor output
% brew doctor
Your system is ready to brew.

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
Trying to install any package (e.g. brew install tree-sitter)
What happened (include all command output)?
TTY flags are modified by brew install and not properly reset. In particular note differences in the output of stty -a before and after running brew install in a fresh terminal:
% stty -a
speed 38400 baud; 49 rows; 99 columns;
lflags: icanon isig iexten echo echoe echok echoke -echonl echoctl
	-echoprt -altwerase -noflsh -tostop -flusho pendin -nokerninfo
	-extproc
iflags: -istrip icrnl -inlcr -igncr ixon -ixoff ixany imaxbel iutf8
	-ignbrk brkint -inpck -ignpar -parmrk
oflags: opost onlcr -oxtabs -onocr -onlret
cflags: cread cs8 -parenb -parodd hupcl -clocal -cstopb -crtscts -dsrflow
	-dtrflow -mdmbuf
cchars: discard = ^O; dsusp = ^Y; eof = ^D; eol = <undef>;
	eol2 = <undef>; erase = ^?; intr = ^C; kill = ^U; lnext = ^V;
	min = 1; quit = ^\; reprint = ^R; start = ^Q; status = ^T;
	stop = ^S; susp = ^Z; time = 0; werase = ^W;
% brew install tree-sitter
==> Downloading https://ghcr.io/v2/homebrew/core/tree-sitter/manifests/0.20.0-1
Already downloaded: /Users/seansullivan/Library/Caches/Homebrew/downloads/caf4df26418c0c4664fed59578fe8257f590731241bbbc342433585a2bab3694--tree-sitter-0.20.0-1.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/tree-sitter/blobs/sha256:cbf44029649cea921fe333454
Already downloaded: /Users/seansullivan/Library/Caches/Homebrew/downloads/e30e408fcc1bd5715be4a06aefd102e2e64d2209249d2fec6f41c7d3d115b0c1--tree-sitter--0.20.0.arm64_big_sur.bottle.1.tar.gz
==> Pouring tree-sitter--0.20.0.arm64_big_sur.bottle.1.tar.gz
🍺  /opt/homebrew/Cellar/tree-sitter/0.20.0: 18 files, 12.6MB
% stty -a

speed 38400 baud; 49 rows; 99 columns;
lflags: icanon isig iexten echo echoe echok echoke echonl echoctl
	-echoprt -altwerase -noflsh -tostop -flusho pendin -nokerninfo
	-extproc
iflags: istrip icrnl -inlcr -igncr ixon -ixoff ixany -imaxbel iutf8
	ignbrk brkint -inpck -ignpar -parmrk
oflags: opost onlcr -oxtabs -onocr -onlret
cflags: cread cs8 -parenb -parodd hupcl -clocal -cstopb -crtscts -dsrflow
	-dtrflow -mdmbuf
cchars: discard = ^O; dsusp = ^Y; eof = ^D; eol = <undef>;
	eol2 = <undef>; erase = ^?; intr = ^C; kill = ^U; lnext = ^V;
	min = 1; quit = ^\; reprint = ^R; start = ^Q; status = ^T;
	stop = ^S; susp = ^Z; time = 0; werase = ^W;

The echonl flag is OFF (-echonl) in the stty -a output before hand and ON afterwards, which has the side effect of an extra newline before the output of each command in the given TTY after running brew install.
It seems like this issue is related to the io/console Ruby gem, which was not used by Homebrew until https://github.com/Homebrew/brew/pull/11914 introduced in 3.2.10 -- the behavior above did not occur prior to this update and by adding logging statement in a local clone of the brew repo, it seems like the line that sets the undesired echonl property is a call to $stdin.cooked! here.
Further, if I create and run a very simple ruby file:
require "io/console"

$stdout.puts `stty -a | grep echonl`
$stdin.raw! if $stdin.tty?
$stdin.cooked! if $stdin.tty?
$stdout.puts `stty -a | grep echonl`

I get the same undesired behavior. So while this is not strictly a homebrew issue, I wanted to flag this to the homebrew maintainers as a possible regression in 3.2.10 for some users. Not sure that this is reproducible everywhere: I was able to reproduce this behavior on my Big Sur and Catalina machines and was NOT able to reproduce on linux.
What did you expect to happen?
The output of stty -a should be the same before and after brew install {package}. I expect this command to fully restore the state of my terminal, in particular it should not modify the echonl or istrip flags.
Step-by-step reproduction instructions (by running brew commands)
brew install {package}

I was able to fix this issue (see https://github.com/sullivan-sean/brew/commit/c88f4c064556b867ec1a3460ab9f8820453cef18) on my machine by using the io.raw method from io/console instead of io.raw!
As mentioned in the documentation for raw! if the terminal needs to be returned to the original state, io.raw should be used. I imagine the current approach might work in some cases but using io.raw will properly reset the tty in all cases and is probably preferable.
Happy to make a PR for this if things make sense and the above diff looks okay.

A PR would be good so your changes could be reviewed more carefully. I think you need to do brew style --fix before opening it, though.

CC @Bo98 who has been working on this recently.

sullivan-sean@c88f4c0

Change looks good to me.


chore(main): release 0.2.0

:robot: I have created a release beep boop
0.2.0 (2022-10-03)
Features

Disassociate Documents and Info Models (#19) (3cefbe9)

Miscellaneous Chores

check: remove explicit name (1417403)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/rotex1800/registration/releases/tag/v0.2.0 :sunflower:


CI test

Background

Changes

Documentation

Test Plan

PR Quality Checklist

[ ] My pull request is atomic and focuses on a single change.
[ ] I have thoroughly tested my changes with multiple different prompts.
[ ] I have considered potential risks and mitigations for my changes.
[ ] I have documented my changes clearly and comprehensively.
[ ] I have not snuck in any "extra" small tweaks changes.
[ ] I have run the following commands against my code to ensure it passes our linters:black .
isort .
mypy
autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports autogpt tests --in-place

You changed Auto-GPT's behaviour. The cassettes have been updated and will be merged to the submodule when this Pull Request gets merged. The pipeline will fail until you merge the challenge scores: https://github.com/merwanehamadi/Auto-GPT/pull/451


Update pub page

First of all, thank you for the awesome project : )
I see you recently commited to the repository, but the pub page hasn't been updated since August.
Could you update it there? Updating CHANGELOG.md would be great too

Hi, I've been very busy for a couple of years :), but the last version was released and all documents updated. Thanks!


Add ruby support

This addresses #38.

The reinstallation fails on my system.
$ gem list | tail -n+1 | sed 's/(/--version /' | sed 's/)//' | xargs -L 1 gem install
ERROR:  While executing gem ... (Gem::Requirement::BadRequirementError)
    Illformed requirement ["default:"]
ERROR:  While executing gem ... (Gem::Requirement::BadRequirementError)
    Illformed requirement ["default:"]
ERROR:  While executing gem ... (Gem::Requirement::BadRequirementError)
    Illformed requirement ["default:"]
Fetching bundler-unload-1.0.2.gem
Successfully installed bundler-unload-1.0.2
Parsing documentation for bundler-unload-1.0.2
Installing ri documentation for bundler-unload-1.0.2
Done installing documentation for bundler-unload after 0 seconds
1 gem installed
ERROR:  While executing gem ... (Gem::Requirement::BadRequirementError)
    Illformed requirement ["default:"]
ERROR:  While executing gem ... (Gem::Requirement::BadRequirementError)
    Illformed requirement ["default:"]
ERROR:  While executing gem ... (Gem::Requirement::BadRequirementError)
    Illformed requirement ["default:"]
ERROR:  While executing gem ... (Gem::Requirement::BadRequirementError)
    Illformed requirement ["default:"]
ERROR:  While executing gem ... (Gem::Requirement::BadRequirementError)
    Illformed requirement ["default:"]
ERROR:  While executing gem ... (Gem::Requirement::BadRequirementError)
    Illformed requirement ["default:"]

Interesting, it was working earlier when I was testing it on my machine, perhaps there was a recent change in gem list's functionality that added the default:  to some version numbers.

Hmm, interesting. I'm using gem 3.1.2. We might need to run a different command to backup / reinstall with gem.

My latest commit fixes the issue, just had to update the backup command.

@tim-coutinho Sorry for taking so long to get around to this. Rebase off master and then I'll merge this.


Fix modal

Fixed this issue https://github.com/thejameskyle/marionette-wires/issues/60
When you initialize bootstrap modal, .modal-dialog element is not exist yet, and bootstrap can't fire event 'shown.bs.modal'. So you need to initialize modal after rendering template or you can remove fade class.

Does this mean that https://github.com/thejameskyle/backbone-service-modals/pull/1 is unnecessary?

Yes, it will work without this PR


(unconfirmed) Changing selector throws

Re-added menu item will not be accepted by its parent because the child already has a parent.

Cannot reproduce, code looks correct.
Nevertheless, code in question should be refactored slightly.


Cleanup EM loading

Add LinuxEnergyModel(EnergyModel) and LegacyEnergyModel(EnergyModel) sublcasses of EnergyModel, so that each of them can implement their own from_target() and check_from_target() methods.
EnergyModel.from_target() autodetects the appropriate subclass to use, so it's completely transparent.
This split can allow separate implementation of other methods as well, notably the ones dealing with idle states, allowing for potential speed-up in LinuxEnergyModel since this class registers a 0 cost for idle states/

Updated with:

PR comments
s/check_from_target/probe_target/ to make it more explicit, and s/_AUTODETECT_ORDER/_PROBE_ORDER/


Do not send emails when recipients list is empty

should we also stop the mailer worker starting if emails is empty, here?


Feedback Research question

Research question
One of your questions is by which garages you can pay by card in 2020, are you also going to look for data before 2020 to see if there is an increase in places where you can only pay by card.
Maybe there are some rare or weird payment methods for parking garages interesting to note.
Datasets needed
I think you have all the essential data you need. Maybe you can find some more interesting variables related to payment and money. Like in this dataset there is information about how expensive parking is: Open-Data-Parkeren-TARIEFDEEL
Assumptions
Some assumptions I made when reading:

There are more places where you can only pay by card than by cash
Parking spots closer to the city centre have more payment options and are more expensive than ones further away

Keep it up and keep on rolling 🤠🙃🙂

@Vuurvos1,
Thanks for your feedback.


Modularization WIP

Pushed just in case anyone is interested. Please ignore for now, but feel free to comment.

I think we can close this now?


fix: v1.1.0 handler

Description
v1.1.0 upgrade handler tries to run updates for modules. But none module registered an upgrade handler, causing panic.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

I don't think it will work.
I think there is an issue because you have created two instances of configurator between RegisterServices and RegisterUpgradeHandlers

Hopefully this does it - otherwise we'll stop 1.1.2 from releasing a binary and try 1.1.3
(apologies for the complications)


Pixel Blurring Issue on Apple M1 Max Internal Display

When running the application on an Apple M1 Max device using the internal screen at native resolution, there is an unexpected visual artifact. While the pixels are rendered as large as intended, there is noticeable blurring around the edges of these pixels. This blurring effect is not desired and may impact the visual clarity of the application.
Environment

Device: Apple M1 Max
Display: Internal screen
Resolution: Native resolution

Visual Evidence

Expected Behavior
The pixels should be rendered with sharp, crisp edges without any blurring or anti-aliasing effects.
Actual Behavior
The pixels are rendered at the correct size but exhibit blurring around their edges, reducing the intended pixelated effect.

Could you provide me with your project?

The project is just a slightly modified ball scene, but even the ball scene suffers from the same problem. It can be perceived in all scenes.

I think I know where the problem is. A PR may be coming soon :)

Excellent! I develop 50% time n an M2 and have never seen this. 🤔

This?
SDL_SetHint(SDL_HINT_RENDER_SCALE_QUALITY, "0");

https://github.com/joshuaskelly/brassengine/pull/40


fix hession bug

Issue Number: close  #200

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wangxiaobin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Ready for Laravel 8 ?

The 8th september, Laravel 8 will be released.
I don't follow the last development.
So, someone knows if we are ready for Laravel 8 ?

For general user, 7.x is good but for pro its very good to new version 8.0.
But i think people need a little bit more time for new version.

My question is about the development of Laravel 8.0, to know if new translations are in this version ( php files and json )

Yet, it is not fixed . Many user are familiar with PHP files.
As a developer,  user are shifting to josn and gaining popularity nowadays.
We can write directly in a single page with less less syntax.

I check the code, and I didn't see update on php translation files. For json, it's most difficult to check.

In the repository, the translation files have not been changed for the 8th version.


microcodeIntel: 20191112 -> 20191113

https://github.com/intel/Intel-Linux-Processor-Microcode-Data-Files/releases/tag/microcode-20191113

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @dtzWill

@GrahamcOfBorg build microcodeIntel


fixed bug: timeToDeath Parameter not being saved when reusing parameters

Please test, that the config parameter timeToDeath is now saved for editing when adjusting existing prameters.
Previously the value and slider state was not restored from the existing config.
(file changed alot due to using formatter accidently :/)

I can't really tell what changed in model/Config.java, but it seems to work from what i can tell.
It looks like configuredIncubationTime is new and that incubation time is now in days not in ticks which makes a lot of sense.

Thanks for testing!
Yes. Because some config values were beinh changed by the programm when starting the simulation.
This obviously made the config values invalid for re-use.


trafilaturan.fetch_url failed to terminate

Hi,
I am a graduate student developing a data pipeline for semantic analysis over news articles and I just encountered this great package. However, after several experiments, there seem to be some problem with trafilaturan.fetch_url with the following link or "http://naplesherald.com/2018/02/07/thousands-african-migrants-protest-israel-deportations/"
After digging into the source code in the downloads.py, I discovered that HTTP_POOL.request('GET', url, headers=_determine_headers(config)) under function _send_request() cannot terminate. I wonder if it is caused by the RETRY_STRATEGY setting because if I fetch the above URL using response = requests.get, the response.status_code is 503. My current workaround is simply to fetch the HTML myself. I wonder if there is any way that I can set a timeout?
python version = '3.9.7'
trafilatura.__version__ = '0.9.3'
urllib3.__version__ = '1.26.7'

Thank you!
Best,
Yuxuan

Hi,
I was able to reproduce the issue
The bug happens with the http:// URL, which actually returns 301 response.
Note that requests.get automatically follows redirects (unless you set allow_redirects=False), so it can mask some redirection issues.
trafilatura -u http://naplesherald.com/2018/02/07/thousands-african-migrants-protest-israel-deportations/ -vv

It's stuck after this line:
DEBUG:urllib3.util.retry:Incremented Retry for (url='/2018/02/07/thousands-african-migrants-protest-israel-deportations/'): Retry(total=1, connect=None, read=None, redirect=None, status=None)

If we use the https:// URL, it stops immediately after the 503 response.
trafilatura -u https://naplesherald.com/2018/02/07/thousands-african-migrants-protest-israel-deportations/ -vv

ERROR:trafilatura.downloads:retries/redirects: https://naplesherald.com/2018/02/07/thousands-african-migrants-protest-israel-deportations/ HTTPSConnectionPool(host='naplesherald.com', port=443): Max retries exceeded with url: /2018/02/07/thousands-african-migrants-protest-israel-deportations/ (Caused by ResponseError('too many 503 error responses'))
DEBUG:trafilatura.cli_utils:No result for URL: https://naplesherald.com/2018/02/07/thousands-african-migrants-protest-israel-deportations/
DEBUG:trafilatura.cli_utils:0 URLs could not be found

The reason it's stuck after the 301 is that the server sends a header Retry-After: 86400 and our retry strategy respects it. So trafilatura would wait for 24 hours before the next retry.
The fix would be to patch urllib3 retry strategy with: respect_retry_after_header=False
I tested and it didn't work out of the box. It seems that we don't pass the retry strategy to all requests.

This seems to be a urllib3 issue. Retry strategy is actually ignored. I opened https://github.com/urllib3/urllib3/issues/2475

Thank you very much for your reply!


Libgcrypt fix link libintl

Describe the pull request

What does your PR fix?
Fixes #27592

Which triplets are supported/not supported? Have you updated the CI baseline?

Same as before

Does your PR follow the maintainer guide?

Yes

The correct fix (for libgpg-error, not libgcrypt) is part of #27643.

closing in favor of #27668


Install a debian server

Install a debian server with:

Nginx
Gunicorn
Postgresql
Supervisord
Data and Database backup and restore scripts
Email Sending
SSH w/ ip limit

Hi all,
I think I can manage this one. Maybe it is necessary to describe in more details the operational needs that this server will support (here, at wiki, next meeting?), because based on the listed software I assume that special care for security should be given too.

Hi kosdar,
Thanks for picking this one up!
The server will support our development needs and temporary a production instance. We need to run dev, stage, production instances on this server thus the listed software. I can immediately give you access to our mediatemple server to get started with a the basic configuration as a first step and then we setup the django instances.
More questions? I'm in #verese on freenode if you prefer live chat ;)
Sounds good?

Oops, that's kosfar sorry for the typo ;)

No probs. I am in #verese channel right now, we may continue from there. Please check if you receive my messages!


No map rendered on KitKat

Note 3
Android 4.4.2
Tangram 0.2.2
Interesting that client data source (pins) are rendered but not the tiles themselves.

Reverting to Tangram 0.2.1 fixes the issue. cc @blair1618 @tallytalwar

Any device logs available @ecgreb?

Trying to grab them now. Getting meaningful logs out of the Note 3 is a bit of a challenge ;)

This appears to be fixed in Tangram 0.3.1. But I am still seeing the other issue where the map renders but other UI elements vanish (menus, text, action bar, etc.) on KitKat.
Note 3/Android 4.4.2
I'll open another issue to track this.


Lazy loading of repositories

Would there be a use for using Lazy in the supervisor is my question?
Would we benefit from using lazy there? Or does DI handle lazily loading these instances? Or would the performance impact be so insignificant that it's not worth it?

Not going to implement