build: bump deps clap/parquet/arrow

What are included in this PR

Bump parquet/arrow to 38.0.0

Bump clap to 4.2.7

In parquet,  ArrowReader and ParquetFileArrowReader are deprecated and replaced with ArrowReaderBuilder and ParquetRecordBatchReaderBuilder

In clap, multiple_values = true is replaced with num_args = 1..

Use #[arg(short, long)] instead of #[clap(short, long)]

The behavior of record_batch_to_json_row() has been changed and thus our integration test for pqrs cat --json needs to be changed too(CAT_JSON_OUTPUT needs to be updated)
# old output
$ pqrs cat data/cities.parquet --json

#########################
File: data/cities.parquet
#########################

{"continent":"Europe","country":{"name":"France","city":["Paris","Nice","Marseilles","Cannes"]}}
{"continent":"Europe","country":{"name":"Greece","city":["Athens","Piraeus","Hania","Heraklion","Rethymnon","Fira"]}}
{"continent":"North America","country":{"name":"Canada","city":["Toronto","Vancouver","St. John's","Saint John","Montreal","Halifax","Winnipeg","Calgary","Saskatoon","Ottawa","Yellowknife"]}}

# new output
$ ./target/debug/pqrs cat data/cities.parquet --json

#########################
File: data/cities.parquet
#########################

{"continent":"Europe","country":{"city":["Paris","Nice","Marseilles","Cannes"],"name":"France"}}
{"continent":"Europe","country":{"city":["Athens","Piraeus","Hania","Heraklion","Rethymnon","Fira"],"name":"Greece"}}
{"continent":"North America","country":{"city":["Toronto","Vancouver","St. John's","Saint John","Montreal","Halifax","Winnipeg","Calgary","Saskatoon","Ottawa","Yellowknife"],"name":"Canada"}}

code format with cargo fmt

Since this is the successor of #32, let's close it.
Closes #32

This looks great! Thanks a lot making these updates @SteveLauC - really appreciate it!


[API] Players SteamIDs should be converted on /api/v1/team/create

https://github.com/FlowingSPDG/get5-web-go/commit/29939a8f0a5ace4af423ff6618ae2d7621102366


Update image quay.io/mkovarik/ose-cli-git:4.11

PR https://github.com/openshift-pipelines/pipeline-service/pull/867 needs kustomize cli, so updating the image to get more cli and also its mkovarik personal quay image

/retest

I think CI won't pass, e2e tests for build-definitions are currently failing due to a broken secret, which nobody hasbeen able to fix because apparently the build-templates-e2e namespace has no admins

I think CI won't pass, e2e tests for build-definitions are currently failing due to a broken secret, which nobody has been able to fix because apparently the build-templates-e2e namespace has no admins

ok but the change in pr https://github.com/openshift-pipelines/pipeline-service/pull/867 will fail until we have this fix. Is there any workaround in this case to merge this pr?

I think CI won't pass, e2e tests for build-definitions are currently failing due to a broken secret, which nobody has been able to fix because apparently the build-templates-e2e namespace has no admins

ok but the change in pr openshift-pipelines/pipeline-service#867 will fail until we have this fix. Is there any workaround in this case to merge this pr?

The image is configurable, in this PR you're only changing the default. You can pass the SCRIPT_IMAGE param to the task in your pipeline definition

FYI https://issues.redhat.com/browse/RHTAPBUGS-1043 was opened to move off of quay.io/mkovarik/ose-cli-git:4.11
@prietyc123 you should mark whatever Jiras you have that depend on this as blocked by that bug to help @redhat-appstudio/build-team prioritize
Wit respect to @tisutisu 's comment at https://github.com/redhat-appstudio/build-definitions/pull/685#pullrequestreview-1747883166 is not correct, as the RHTAP override of the default tekton SA has the credentials linked.
In fact, there is already use of registry.redhat.io in this very task, namely the default for GIT_IMAGE
@prietyc123 is simply changing the default for SCRIPT_IMAGE
In fact, you see use of registry.redhat.io in other both PaC based jobs in this repo, namely
https://github.com/redhat-appstudio/build-definitions/blob/main/.tekton/pull-request.yaml
As well as well used tasks like init and git-clone
Now, all that good news, said, there is the question of verifying this new image works.
@prietyc123 can you elaborate here on how you've tested, so we have a record here (vs. where you may have reported elsewhere)

@prietyc123 can you elaborate here on how you've tested, so we have a record here (vs. where you may have reported elsewhere)

@gabemontero Thanks for your detailed input here. There is another problem with the image is that multiple team using this image. So changing it needs proper analysis. I don't think I have bandwidth for that. So for now maybe I will let someone else pick up this task. I will close this pull request for now.


Added etherscan to gas prices

Hi, thanks for creating a PR!
I have a few questions:

Could you perhaps use the etherchain endpoint that uses mempool prediction, which is more accurate? https://www.etherchain.org/api/gasnow, and make the link point to https://www.etherchain.org/tools/gasnow (the mempool gas price prediction page)?
Could you also resubmit the pr without all the linting changes that your editor probably made automatically? It makes reading it a mess, and there's no point in changing all those single quotes to double quotes and all that ^^
Also tweak the pr title from "etherscan" to "etherchain" :)

Thank you! Happy to merge it afterward!


Minor changes to SIRS demo

Added visual tweaks to match GD and ND versions.
The visual size of the grid is now fixed for all N values, capped at 300x300 which is when the drawing throttles the fps anyways despite numdot performing well. Not sure how to improve this.
Changed convolution kernel to not include the site itself.

Thanks!


User name is available when registering, but password entry gives "Invalid Credentials"

Hello,
I was able to install OpenNote with MySQL.  When I register and enter the username, the application changes from "Username must be at least four characters" to "Available" after the fourth character, but when I attempt to enter the password and click Register, I get "Invalid Credentials."  Any help would be greatly appreciated.  Thank you.

It is returning a HTTP/1.1 500 Internal Server Error whenever I try to register.  If anyone can provide help, I would appreciate it.  Thank you.

Please see https://github.com/FoxUSA/OpenNote/issues/115


Implement Volunteer Hours Log

https://www.figma.com/file/h85P5QFeMPJlRy9piBQQxH/Wireframes?node-id=969%3A44987

/cib event pagination

Branch 100-event-pagination created!

/cib volunteer hours log page

Branch 100-volunteer-hours-log-page created!


Pimple service return different instance

Hi,
In my application, every time a service is used, it's constructor called. If all works right, the constructor should be only called once, right?
My pimple version: ~3.0
PHP version: 5.6
Could anyone can help with it, thanks.
JL

Can you provide a reproducing case ?


Power consumption does not update when using a template sensor.

Hello, thanks for your hard work!
I'm trying to monitor an humidifier power consumption based on the fan rotation speed, but the value power (watt) doesn't update.
I've tried:
powercalc:
  scan_interval: 00:00:10
  create_utility_meters: true
  utility_meter_types:
    - daily
    - weekly
    - monthly
    - yearly

sensor:
  - platform: powercalc
    entity_id: humidifier.humidificateur
    standby_power: 2.3
    fixed:
      power: >
        {% if states('sensor.humidificateur_motor_speed')|int >= 990 %}
        4.4
        {% elif states('sensor.humidificateur_motor_speed')|int >= 900 %}
        3.9
        {% elif states('sensor.humidificateur_motor_speed')|int >= 800 %}
        3.5
        {% elif states('sensor.humidificateur_motor_speed')|int >= 700 %}
        3.3
        {% elif states('sensor.humidificateur_motor_speed')|int >= 600 %}
        3.1
        {% elif states('sensor.humidificateur_motor_speed')|int >= 500 %}
        2.9
        {% endif %}

and
powercalc:
  scan_interval: 00:00:10
  create_utility_meters: true
  utility_meter_types:
    - daily
    - weekly
    - monthly
    - yearly

sensor:
  - platform: powercalc
    entity_id: humidifier.humidificateur
    standby_power: 2.3
    fixed:
      states_power:
         mode|Silent: 2.6
         mode|Medium: 3.3
         mode|High: 4.4
         mode|Auto: >
           {% if states('sensor.humidificateur_motor_speed')|int >= 990 %}
           4.4
           {% elif states('sensor.humidificateur_motor_speed')|int >= 900 %}
           3.9
           {% elif states('sensor.humidificateur_motor_speed')|int >= 800 %}
           3.5
           {% elif states('sensor.humidificateur_motor_speed')|int >= 700 %}
           3.3
           {% elif states('sensor.humidificateur_motor_speed')|int >= 600 %}
           3.1
           {% elif states('sensor.humidificateur_motor_speed')|int >= 500 %}
           2.9
           {% endif %}

It only update when changing fan mode. Thanks in advance!

Try setting entity_id to sensor.humidificateur_motor_speed as this will make powercalc listen to state changes of sensor.humidificateur_motor_speed.

You're right, so obvious that I couldn't see it! It works! Thanks a lot!

I think I can make some modifications that the power sensor also changes state whenever one of the referred entities in the template changes.
Than your initial configuration which you reported this issue with should work. And will also track humidifier.humidificateur as well as sensor.humidificateur_motor_speed.
This needs some work, but I will see what I can do.

Thanks for your commitment! It will be great and useful for a lot of cases. I love your work!

Resolved by #629
Are you able to test this by installing the master branch?

Yup via HACS, let me try and get back to you.

Doesn't work :'(
Only change when I change Mode or switch ON/OFF!
By the way, I know it's not the right place, but I made a PR for supporting a new bulb earlier, and I made a mistake.
The bulb is sold as branded "Lexman" but it's manufactured by "ADEO".

Doesn't work :'( Only change when I change Mode or switch ON/OFF!

Hmm ok. which configuration did you use now. And you are sure you are on latest master?

By the way, I know it's not the right place, but I made a PR for supporting a new bulb earlier, and I made a mistake.
The bulb is sold as branded "Lexman" but it's manufactured by "ADEO".

You could also create a PR to add an alias here https://github.com/bramstroker/homeassistant-powercalc/blob/master/custom_components/powercalc/const.py#L103

yup, sadly I'm in master branch, and only work when changing mode.

sensor:
  - platform: powercalc
    entity_id: humidifier.humidificateur
    standby_power: 2.3
    fixed:
      states_power:
        mode|Silent: 2.6
        mode|Medium: 3.3
        mode|High: 4.4
        mode|Auto: >
          {% if states('sensor.humidificateur_motor_speed')|int >= 990 %}
          4.4
          {% elif states('sensor.humidificateur_motor_speed')|int >= 900 %}
          3.9
          {% elif states('sensor.humidificateur_motor_speed')|int >= 800 %}
          3.5
          {% elif states('sensor.humidificateur_motor_speed')|int >= 700 %}
          3.3
          {% elif states('sensor.humidificateur_motor_speed')|int >= 600 %}
          3.1
          {% elif states('sensor.humidificateur_motor_speed')|int >= 500 %}
          2.9
          {% endif %}

Could you try your first configuration? I only added support for tracking templates on fixed->power
Will see if I can also do that on fixed->states_power

Yup, it's working with fixed->power !
Thanks you so much!

See #632
I have also added support for tracking templates when you use templates in states_power
Has been merged into master. Will be in next release.

Confirmed working! Both ways! Thanks.


Initial implementation for the JVM and JavaScript.

The Scala Native version is not there yet, as Scala Native does not have yet the necessary mechanisms to implement the API.
The only discrepancy between JVM and JS is the behavior of protected constructors. The JVM version considers that they are visible, because they are compiled to public in the bytecode, while Scala.js explicitly excludes them. It seems that the most reasonable fix would be to allow access to protected constructors in Scala.js' core Reflect mechanism.

Ready for review, mainly @gzm0.
@densh It would be good if you could have a look as well, mostly to tell us whether any of it is unreasonable for Scala Native. But from our earlier discussions, it should be fine.

Updated with fixes to make the CI pass on 2.10 and 2.11 (I had misconfigured Travis so I wouldn't actually test with those versions).
And with an additional commit that uses macros on the JVM to get the caller's class loader.


Roads under construction

The style should show roads that are under construction and possibly some major proposed roads. Otherwise, there can be prominent, otherwise inexplicable gaps or dangling roads.
Québec Autoroute 35 is being built in phases.
This bridge across the Ohio River has been closed for repairs long enough to be tagged as being under construction too.
Precedents
The predominant convention among North American maps is to depict a road under construction as a broken line:

This convention extends to maps published in Canada or for tourists from continental Europe:

The essence of this treatment is that the casing is also broken, not just the fill, and the fill has alternating transparent segments, not white segments. I prefer this treatment to what openstreetmap-carto does, which evokes a barber pole or candy cane:

In many cases, inaccessible or unpaved roads get almost the same treatment. It seems the general idea is for a broken or dashed line to warn of a bad day:

Some maps call them “future” roads and include roads still in aspirational planning stages, while others distinguish between proposed roads and roads under construction:

The Ohio Department of Transportation at one point used a translucent red dotted overlay to indicate future roads, just like openstreetmap-carto’s access=no treatment:

Most maps in my collection, even city-scale maps only show limited-access highways under construction but omit less important roads under construction:

This could be because they only depict freeways for which a right of way has already been cleared, leaving a major gap. But it could also be because smaller roads tend to get completed more rapidly than paper maps can reliably keep up with.
Some maps do show multiple levels of roads under construction; these maps generally don’t already use dashes to convey surface conditions:

I think it would benefit mappers if Americana shows any road under construction, regardless of classification, but if the complexity of supporting yet another facet of road classification proves too complex, at least there would be precedent for only showing the most major construction projects.
Since not-yet-completed freeways are relatively rare, most maps annotate them with a special label explicitly stating that they’re planned or under construction, even giving an expected date of completion if the DOT is confident enough about it:

I would suggest avoiding display of roads that are just proposed - it encourages mapping them and it is often filled with overenthusiastic plans, blatant campaign promises, mapping several variants, mapping personal fantasies and so on.
Completely absurd projects are being mapped.

Let's scope this issue to under-construction roads, which are non-controversial and supported by OpenMapTiles.  Any discussion of proposed roads should be a separate ticket.  I tend to agree that proposed roads have a pretty serious curation issue that would need to be explored if there is ever a serious initiative to render them.

it encourages mapping them and it is often filled with overenthusiastic plans, blatant campaign promises, mapping several variants, mapping personal fantasies and so on.

Incentives can flow in either direction – what if we hide proposed roads or make them look obnoxious unless they have some additional tags like source and opening_date? Maps typically annotate proposals with “(Expected Completion 2030)” or somesuch. That’s not to say that the date will be an accurate reflection of anything, but it would raise the bar a bit. That said, if there’s a sense that no proposed highway is ever valid, then that’s a different story.
In any case, #215 only deals with roads under construction.

no proposed highway is ever valid

I would be actually in this camp, but I am aware that it is not a consensus position.
But "highway=proposed should be limited to serious proposals, not all official proposals are serious and all highway=proposed should have source in source tag or changeset data" seems matching consensus position

In Openstreetmap Carto, the current railway and subway tunnel rendering is similar to the USA map examples for under-construction or proposed roads.
Do you have thoughts about how this would work with subways and other rail tunnels? I understand they are not yet rendered here yet.
Would under-construction railways be eventually rendered?

Good question. In #101, we were considering rendering railroad tracks with ties rather than an alternating fill, at least for freight and commuter rail. Ties seem to be an uncommon treatment for light rail and subways, but they probably wouldn’t be depicted with an alternating fill in any case. We’re currently rendering tunnels with a faint but solid fill and casing, so it should be possible to combine that with the broken line treatment for tunnels. However, so far in https://github.com/ZeLonewolf/openstreetmap-americana/pull/215#issuecomment-1140313010, I’ve shied away from indicating tunnels or bridges on roads under construction. I had roads in mind, but I suppose the presence of a tunnel would be more notable for a subway under construction.
In the event of a conflict in line treatments, we can also lean on the “(Under Construction)” gloss as seen in #215, since large construction projects tend to be the exception rather than the rule.


perf: remove intermediate contract deserialization

Codecov Report

Merging #1050 (ac85336) into main (2d25af1) will decrease coverage by 0.07%.
Report is 2 commits behind head on main.
The diff coverage is 86.66%.

@@            Coverage Diff             @@
##             main    #1050      +/-   ##
==========================================
- Coverage   90.02%   89.96%   -0.07%
==========================================
  Files          49       49
  Lines       13165    13030     -135
==========================================
- Hits        11852    11722     -130
+ Misses       1313     1308       -5

Files
Coverage Δ

...c/core/contract_address/sierra_contract_address.rs
91.66% <ø> (-0.65%)
:arrow_down:

src/serde_structs/mod.rs
100.00% <100.00%> (ø)

.../api/contract_classes/deprecated_contract_class.rs
82.33% <100.00%> (-0.24%)
:arrow_down:

src/core/contract_address/casm_contract_address.rs
93.50% <0.00%> (-1.24%)
:arrow_down:

src/execution/execution_entry_point.rs
96.99% <0.00%> (-0.23%)
:arrow_down:

... and 8 files with indirect coverage changes


Logger send elastic one value at a time

A few questions before you begin:

Is this an issue related to the Serilog core project or one of the sinks or
community projects.
This issue list is intended for Serilog Elasticsearch Sink issues. If this issue relates to another sink or to the code project,
please log on the related repository.  Please use Gitter chat and Stack Overflow  for discussions and questons.

Does this issue relate to a new feature or an existing bug?

[ ] Bug
[ ] New Feature

What version of Serilog.Sinks.Elasticsearch is affected?  Please list the related NuGet package.
What is the target framework and operating system? See target frameworks & net standard matrix.

[ ] netCore 2.0
[ ] netCore 1.0
[ ] 4.7
[ ] 4.6.x
[ ] 4.5.x

Please describe the current behavior?
Please describe the expected behavior?
If the current behavior is a bug, please provide the steps to reproduce the issue and if possible a minimal demo of the problem

public  Logger()
{
_loggerConfiguration = new LoggerConfiguration();
_loggerConfiguration.Enrich.WithMachineName().Enrich.WithProcessId().Enrich.WithThreadId()
.MinimumLevel.Information();
    }

    public static void CreateLogger()
    {
        Log.Logger =_loggerConfiguration.CreateLogger();

    }

    public static void EnableElasticSearch(string url = "")
    {
        _loggerConfiguration.WriteTo.Elasticsearch(new ElasticsearchSinkOptions(new Uri("http://localhost:9200/"))
          {
            AutoRegisterTemplate = true,
            AutoRegisterTemplateVersion = AutoRegisterTemplateVersion.ESv6,
            IndexFormat = "serilog",
            CustomFormatter = new ExceptionAsObjectJsonFormatter(),

        }).MinimumLevel.Information();

    }

     public static void LogInformation(int LogId ,string msg, string functionname)
     {

            LogRecord record = new LogRecord
            {
                LogId = LogId,
                Message = msg,
                FunctionName = functionname,
                LogLevel = Level.INFORMATION,
                LogLevelId = Level.INFORMATION.GetHashCode()
            };

            Log.Information("{@holder}", record);

    }

from main iam calling
Logger log = new Logger();
Logger.EnableElasticSearch();
Logger.CreateLogger();
100 times
Logger.LogInformation(123, "sadfdf", "asdad");
but only one value is coming.
if i use other function  only first time log is coming
Logger.LogInformation(123, "sadfdf", "asdad");
Logger.LogError(123, "sadfdf", "asdad");
Logger.LogWarning(123, "sadfdf", "asdad");
Logger.LogFatal(123, "sadfdf", "asdad");
Logger.LogInformation(123, "sadfdf", "asdad");
Logger.LogInformation(123, "sadfdf", "asdad");

Did you dispose the logger? You need to close and flush the sinks at the end/closing of application. It has an internal buffer so it needs some time or an explicit flush to write them to ES.

Op 1 mrt. 2020 om 12:32 heeft Lucifer-Elf<EMAIL_ADDRESS>het volgende geschreven:

public Logger()
{
_loggerConfiguration = new LoggerConfiguration();
_loggerConfiguration.Enrich.WithMachineName().Enrich.WithProcessId().Enrich.WithThreadId()
.MinimumLevel.Information();
}

public static void CreateLogger()
{
    Log.Logger =_loggerConfiguration.CreateLogger();

}

public static void EnableElasticSearch(string url = "")
{
    _loggerConfiguration.WriteTo.Elasticsearch(new ElasticsearchSinkOptions(new Uri("http://localhost:9200/"))
      {
        AutoRegisterTemplate = true,
        AutoRegisterTemplateVersion = AutoRegisterTemplateVersion.ESv6,
        IndexFormat = "serilog",
        CustomFormatter = new ExceptionAsObjectJsonFormatter(),

    }).MinimumLevel.Information();

}

 public static void LogInformation(int LogId ,string msg, string functionname)
 {

        LogRecord record = new LogRecord
        {
            LogId = LogId,
            Message = msg,
            FunctionName = functionname,
            LogLevel = Level.INFORMATION,
            LogLevelId = Level.INFORMATION.GetHashCode()
        };

        Log.Information("{@holder}", record);

}

from main iam calling
Logger log = new Logger();
Logger.EnableElasticSearch();
Logger.CreateLogger();
100 times
Logger.LogInformation(123, "sadfdf", "asdad");
but only one value is coming.
if i use other function only first time log is coming
Logger.LogInformation(123, "sadfdf", "asdad");
Logger.LogError(123, "sadfdf", "asdad");
Logger.LogWarning(123, "sadfdf", "asdad");
Logger.LogFatal(123, "sadfdf", "asdad");
Logger.LogInformation(123, "sadfdf", "asdad");
Logger.LogInformation(123, "sadfdf", "asdad");
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

okay i have added close and flush in close of this File but i have a simple task .
calling below function from main
Logger.LogInformation(123, "sadfdf", "asdad");
Logger.LogError(123, "sadfdf", "asdad");
Logger.LogWarning(123, "sadfdf", "asdad");
Logger.LogFatal(123, "sadfdf", "asdad");
Logger.LogInformation(123, "sadfdf", "asdad");
Logger.LogInformation(123, "sadfdf", "asdad");
and these should be in elastic search correct?
but insead of  6 entry i always get 1st entry . please check my functions are public static

Logging.zip
please check with my test app and do let me know.

I advice you to look at the sample located here: https://github.com/serilog/serilog-sinks-elasticsearch/blob/dev/sample/Serilog.Sinks.Elasticsearch.Sample/Program.cs
You introduce a very complicated way of applying configurations and logging while there is so much simplicity already available out of the box. The destructor usage might also not be optimal.
To further troubleshoot; add a simple console sink and see if that does log. Keep in mind that the ES sink is a periodicbatching sink, so it buffers to max 50 items and 2 seconds before it flushes (and then sends it to ES). In a console app this means your app is closed before it has time to flush. See that explicit line here: https://github.com/serilog/serilog-sinks-elasticsearch/blob/dev/sample/Serilog.Sinks.Elasticsearch.Sample/Program.cs#L69

Hi Mivano, thanks I tried this one it works fine. But the project which I have created it works fin the console message come as the data but only elastic search data doesn't have all data.
Is there any way to flush and not to close the connection . I have simple I should configur once Logger and use the static function to Log message in elastic search based on the log type.

is there is any way to flush all the data and not to close the connection? or after flush also to log data

There is already a static Log.Logger available, so use that one. For the buffer size, you can configure that to flush after each message, but that is not recommended.


Fix DMatrix feature names/types IO.

Added type hints.

Close https://github.com/dmlc/xgboost/issues/6452 .

Codecov Report

Merging #6507 (2f08000) into master (afc4567) will increase coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #6507      +/-   ##
==========================================
+ Coverage   80.23%   80.38%   +0.15%
==========================================
  Files          13       13
  Lines        3551     3558       +7
==========================================
+ Hits         2849     2860      +11
+ Misses        702      698       -4

Impacted Files
Coverage Δ

python-package/xgboost/core.py
80.79% <100.00%> (+0.16%)
:arrow_up:

python-package/xgboost/callback.py
79.95% <0.00%> (+0.24%)
:arrow_up:

python-package/xgboost/tracker.py
95.11% <0.00%> (+1.12%)
:arrow_up:

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 afc4567...3162656. Read the comment docs.


refactor: expose parsed Header fields

This PR exposes the parsed Header fields.
Motivation
Additional validations at the application layer. In particular, we have a use case where the same keyId should be referenced in a custom header that is included in the signature's covered content.
Signed-off-by: George Aristy<EMAIL_ADDRESS>

ping @YaroslavPodorvanov @igor-pavlenko


Import perfect-scrollbar from node_modules instead of bower_components

As @ludalex noted that it will be good to import perfect-scrollbar from the npm package instead of bower and ember-cli is also moving away from bower, this issue should be addressed.

now that my pr is merged; how about getting a new tag, publishing it and closing this issue? I can be a maintainer as I already indicated in another issue if you like :)

@feanor07 good point! By the way I aded you as a maintainer.


Strengthen neutral congruence for Π / Σ types.

We assume that the term is self-convertible as a neutral. This also implies being a whne once we have the generic typing properties, so we remove this field, although it makes some proofs slightly more annoying later on.
cc @MevenBertrand it might allow simplifying some of the stuff in the bidir dev?

Yes, I need to bake in as much inversions on negative introduction rules as possible.

(Note also that at the level of the generic API, we don't know yet that typing implies self-convertibility.)


PAB-328: post-award compatibility

https://trello.com/c/tQ3rDYzh/328-implement-authentication-changes-into-front-end-auth-utils-and-docker-runner
Change description
Enables other FSD applications (namely for now, post-award-frontend) to make use of the SSO and Session capability of authenticator. This is enabled by passing a return_app query param to authenticator when requesting SSO authentication, so that the app can return back to other applications other than pre-award-assessment. These apps are defined in config and map to return URLs and authorisation BPs.

[x] Unit tests and other appropriate tests added or updated
[x] README and other documentation has been updated / added (if needed)
[X] Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

How to test
If manual testing is needed, give suggested testing steps
Screenshots of UI changes (if applicable)

DRAFT - tests still to come.

Minor additional change to make once the accompanying fsd_utils changes go in


Deis builder container removed

I'm trying to deploy new apps in my Deis platform. Previous apps were deployed successfully. Now, when I run 'git push' the action is not performed. I checked 'deisctl list' and the builder component doesn't appear. In the node, I see the builder component image, but the container doesn't exist in 'docker ps neither 'docker ps -a'.
Now I want to re-run the builder component, I tried 'deis install builder' but it gives a timeout, and 'deisctl restart builder' returns 'Could not find unit: deis-builder.service'.
In journalctl I see:
Feb 23 16:39:43 ip-10-21-2-117.eu-west-1.compute.internal fleetd[714]: ERROR reconcile.go:79: Unable to determine agent's current state: failed fetching     unit states from UnitManager: dbus: connection closed by user
Feb 23 16:39:44 ip-10-21-2-117.eu-west-1.compute.internal fleetd[714]: ERROR generator.go:65: Failed fetching current unit states: dbus: connection     closed by user
Feb 23 16:39:45 ip-10-21-2-117.eu-west-1.compute.internal fleetd[714]: ERROR generator.go:65: Failed fetching current unit states: dbus: connection     closed by user
Feb 23 16:39:46 ip-10-21-2-117.eu-west-1.compute.internal fleetd[714]: ERROR generator.go:65: Failed fetching current unit states: dbus: connection     closed by user
Feb 23 16:39:47 ip-10-21-2-117.eu-west-1.compute.internal fleetd[714]: ERROR generator.go:65: Failed fetching current unit states: dbus: connection     closed by user

Why I getting this error?
How can I run the builder image?

Judging from the logs I would assume that fleet is down. Can you confirm that etcd/fleet is responding correctly?

Fleet, etcd and systemd are running properly : /

It's a Fleet problem. I close this thread.
Thanks.


add estimated value class OBI:0002749 for #1189

#1189
Have also added an entry to the spreadsheet!

I'll mark this as a draft until I hear back about my comment on #1189.


Gedis client bug

start threebot server, it starts gedis by default on port 8901
client = j.servers.threebot.local_start_default(web=True)

it returns the client threebot. Do a ping, it pings fine.
JSX> client.ping()
True

get the same client again but give it the wrong port
client = j.clients.gedis.get("threebot", port=8900)

the client is returned without crashing, even though nothing is running on 8900
and when you try to ping, it pings just fine.
It's actually still pinging using the old port 8901.
if you check client._redis you'll see it's using port 8901
On top of this bug, if you have a gedis client pointing to a port where nothing is running, if you try to do a get it will fail, because the init of the object calls the reload which uses redis and the connection will fail. So even if you pass new ports, it will still fail because the object is loaded first then updated. If you try to delete it, it will also fail, because the delete does a get.

Added a small fix to reset redis when the instance is saved https://github.com/threefoldtech/jumpscaleX_core/commit/ce763592013ec9076d22707e2bfeb7821259100b however, if someone changes the port property on the object cl.port = newvalue and doesn't save, redis instance wont be reset.

however, if someone changes the port property on the object cl.port = newvalue and doesn't save, redis instance wont be reset.

I think in this case, _update_trigger of base class can be used, see for zos client as en example.

Added triggers to handle both property change and save https://github.com/threefoldtech/jumpscaleX_core/commit/247286cd7e661f3c403b7ce26b5f08ba156aaa99

great elegant fix, well done


Error with runner.generate in TensorRT-LLM 0.14.0 for Qwen Example

Environment
•	Docker Image: nvcr.io/nvidia/tritonserver:24.10-trtllm-python-py3
•	TensorRT-LLM Version: 0.14.0
•	Run Command:

python3 ../run.py
--input_text "你好，请问你叫什么？"
--max_output_len=50
--tokenizer_dir /data/models/Qwen1.5-7B-Chat/
--engine_dir ./tmp/qwen/7B/trt_engines/fp16/1-gpu/
•Example Code: examples/qwen/run.py (from README)
Description
While running the run.py script as described in the README of the examples/qwen/ directory, the following error occurs when invoking runner.generate:
Error Traceback
Traceback (most recent call last):
File "/triton/TensorRT-LLM-release-0.14/examples/qwen/../run.py", line 887, in
main(args)
File "/triton/TensorRT-LLM-release-0.14/examples/qwen/../run.py", line 711, in main
outputs = runner.generate(
File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/runtime/model_runner_cpp.py", line 624, in generate
requests = [
File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/runtime/model_runner_cpp.py", line 625, in
trtllm.Request(
TypeError: init(): incompatible constructor arguments. The following argument types are supported:
1. tensorrt_llm.bindings.executor.Request(input_token_ids: list[int], *, max_tokens: Optional[int] = None, max_new_tokens: Optional[int] = None, streaming: bool = False, sampling_config: tensorrt_llm.bindings.executor.SamplingConfig = SamplingConfig(), output_config: tensorrt_llm.bindings.executor.OutputConfig = OutputConfig(), end_id: Optional[int] = None, pad_id: Optional[int] = None, position_ids: Optional[list[int]] = None, bad_words: Optional[list[list[int]]] = None, stop_words: Optional[list[list[int]]] = None, embedding_bias: Optional[torch.Tensor] = None, external_draft_tokens_config: Optional[tensorrt_llm.bindings.executor.ExternalDraftTokensConfig] = None, prompt_tuning_config: Optional[tensorrt_llm.bindings.executor.PromptTuningConfig = None, lora_config: Optional[tensorrt_llm.bindings.executor.LoraConfig] = None, lookahead_config: Optional[tensorrt_llm.bindings.executor.LookaheadDecodingConfig] = None, logits_post_processor_name: Optional[str] = None, encoder_input_token_ids: Optional[list[int]] = None, client_id: Optional[int] = None, return_all_generated_tokens: bool = False, priority: float = 0.5, type: tensorrt_llm.bindings.executor.RequestType = RequestType.REQUEST_TYPE_CONTEXT_AND_GENERATION, context_phase_params: Optional[tensorrt_llm.bindings.executor.ContextPhaseParams] = None, encoder_input_features: Optional[torch.Tensor] = None, encoder_output_length: Optional[int] = None, num_return_sequences: int = 1)
Invoked with: kwargs:
input_token_ids=[151644, 8948, 198, 2610, 525, 264, 10950, 17847, 13, 151645, 198, 151644, 872, 198, 108386, 37945, 56007, 56568, 99882, 99245, 11319, 151645, 198, 151644, 77091, 198],
encoder_input_token_ids=None,
encoder_output_length=None,
encoder_input_features=None,
position_ids=None,
max_tokens=50,
num_return_sequences=None,
pad_id=151643,
end_id=151645,
stop_words=None,
bad_words=None,
sampling_config=<tensorrt_llm.bindings.executor.SamplingConfig object at 0x7f000502f830>,
lookahead_config=None,
streaming=False,
output_config=<tensorrt_llm.bindings.executor.OutputConfig object at 0x7f0001cca270>,
prompt_tuning_config=None,
lora_config=None,
return_all_generated_tokens=False,
logits_post_processor_name=None,
external_draft_tokens_config=None
Additional Context
The engine and tokenizer paths are configured as follows:
•	--tokenizer_dir: /data/models/Qwen1.5-7B-Chat/
•	--engine_dir: ./tmp/qwen/7B/trt_engines/fp16/1-gpu/
The engine appears to load successfully, as indicated by the log output:
[TensorRT-LLM][INFO] Engine version 0.14.0 found in the config file, assuming engine(s) built by new builder API.
...
[11/18/2024-02:33:18] [TRT-LLM] [I] Load engine takes: 12.188158512115479 sec
However, the error seems to indicate a problem with the argument types for the tensorrt_llm.bindings.executor.Request class, particularly with sampling_config and output_config.
If more logs or information are needed, please let me know! Thank you!

Hellow. I saw same error when I coverted llama3.0 using trt-llm-0.14. So I change trt-llm-0.13, and then I can covert llama3.0 to trt model.

Thanks !! cool, man I also used a similar method to solve this current
problem. I used the run.py file from the previous version of the code, but
I'm not sure of the version number, but the problem was also resolved, it
should be a small bug in the latest version.
yspch2022 @.***> 于2024年11月19日周二 08:54写道：

Hellow. I saw same error when I coverted llama3.0 using trt-llm-0.14. So I
change trt-llm-0.13, and then I can covert llama3.0 to trt model.
—
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/TensorRT-LLM/issues/2452#issuecomment-2484483093,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARD4MCRWGRZABRYINEL765T2BKD5LAVCNFSM6AAAAABR6T5UAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBUGQ4DGMBZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fault Tolerance Guarantees

Fault Tolerance Guarantees About Siddhi

Please open a new issue with your exact query, closing this, since this doesn't provide any context.


[Idea] Implement success rate in logging

Proposal:
In addition to reward, having an overview of the current success rate can be useful in many envs. This can be a very important metric in envs that have a clear goal (e.g. successfully landed for the 3DLander env, sucessfully parked for the 3DCarParking env, etc.).
It seems we could support this with SB3 by implementing:.
https://stable-baselines3.readthedocs.io/en/master/common/logger.html#rollout

success_rate: Mean success rate during training (averaged over stats_window_size episodes, 100 by default), you must pass an extra argument to the Monitor wrapper to log that value (info_keywords=("is_success",)) and provide info["is_success"]=True/False on the final step of the episode

Needs to be considered:
It would be great if this can be added in a way that doesn't affect previous envs (e.g. they either report always true, false, or don't show this statistic).

Info sending/receiving: Some modifications would be needed to the plugin and Python env code to send / receive info, optimally preserving compatibility with older envs that don't send info. Once we enable info sending, we can later also set the truncated/terminated flags.

Usage / plugin side changes:

func end_episode(final_reward = 0, success = true):
	reward += final_reward
	done = true
	needs_reset = true
	episode_successful = success

(Just a potential usage example, the end episode method is implemented in the env code, not plugin, although we can consider simplifying the process with something like https://github.com/edbeeching/godot_rl_agents_plugin/pull/20, however, that does break compatibility with existing envs)
For compatibility, possibly the simplest way would be to always report episode success as true by default, unless set by the user.
Optionally, we could also add a boolean arg to the sb3 example script that sets the monitor to report this stat or not.

I can help implement this, since I also wanted to expose the info dictionary to Python, such that we can later populate it with more useful info.
If I got it correctly from the documentation (the environment info dict must contain an is_success key to compute that value) we just need to add the "is_success" key to the Godot info. I think we can do this based out of whether the environment ended before or after the X simulations steps you set in the environment.

I can help implement this, since I also wanted to expose the info dictionary to Python, such that we can later populate it with more useful info.
If I got it correctly from the documentation (the environment info dict must contain an is_success key to compute that value) we just need to add the "is_success" key to the Godot info. I think we can do this based out of whether the environment ended before or after the X simulations steps you set in the environment.

Thanks, we can work on implementing this together. For now, a few things to consider are that some of these will be SB3 specific (the success flag), but we can highlight that with a comment somewhere in the plugin and make it optional, and other stuff like custom info should be framework agnostic (no issues there). There's an additional thing I wanted to implement at some point, and that's truncation. These are separate issues, but of course make sense to consider when we implement the changes.
For success: I think we should let the user decide the criteria for success. An episode that reaches the restart timer is not necessarily unsuccessful (e.g. one env's goal could be to collect as many items as possible before the env restarts, there's not necessarily a clear success here, but the user could define an arbitrary success threshold for tracking if wanted). What do you think?

Sounds good. I'll start working on a draft sometime this week and I'll ping you so we can ping-pong some ideas.
When you say truncation, you mean terminating all agents once one of them is done?


Errors when using model path from Comfyui

I am running SwarmUI and ComfyUI on the same node, a windows host. I already have models, loras and all the things downloaded and in use by ComfyUI, so in Server -> Server Configuration i followed the instructions and provided absolute path for my ModelRoot which is in my case "D:\ComfyUI_windows_portable\ComfyUI\models", all other directories (checkpoints, loras, etc) are subfolders of this root.
SwarmUI is rejecting the change and in the logs i see this:
10:06:31.985 [Warning] User local changed server settings: paths.modelroot, paths.sdmodelfolder, paths.sdlorafolder, paths.sdvaefolder, paths.sdembeddingfolder
10:06:31.989 [Error] Failed to create one or more directories: The filename, directory name, or volume label syntax is incorrect. : 'D:\SwarmUI\Model paths settings are invalid, rejected change.
Model paths settings are invalid, rejected change.
D:\ComfyUI_windows_portable\ComfyUI\models\checkpoints'

Did you miss the readme migration notice? https://github.com/Stability-AI/StableSwarmUI?tab=readme-ov-file#migration-notice
https://github.com/mcmonkeyprojects/SwarmUI/discussions/2

I have just installed it for the first time why would i need to migrate if there's nothing to migrate?

This is the outdated repository, that's why :)

I'm sorry, i got my tabs mixed up and raised the issue in the wrong repo. I have indeed cloned the updated one. Will raise the issue there. Apologies for the confusion.


Build fail on ubuntu 20.04

I'm trying to build velox on Ubuntu 20.04 using the provided script setup-ubuntu.sh
These are my steps:
git clone --recursive https://github.com/facebookincubator/velox.git
cd velox
git submodule sync --recursive
git submodule update --init --recursive
cd scripts/
sudo ./setup-ubuntu.sh

Unfortunately, the build fails with the following here due to folly

I tried using different releases of folly, to no avail.
I don't know how to fix this problem, can anyone please help me ? =)

i got same issues on wsl2.

@giulio98 @jiangzhx
Removing "--masm=intel" from  the COMPILER_FLAGS could solve this problem.
@mbasmanova Is this flag necessary to build velox, generating the assembly code in Intel syntax?
There seems to be a PR working on removing this flag in the setup-macos.sh
https://github.com/facebookincubator/velox/pull/1336

Removing "--masm=intel" from the COMPILER_FLAGS could solve this problem.
Is this flag necessary to build velox, generating the assembly code in Intel syntax?

I haven't tried this. Maybe try to see what happens.
@kgpai @mshang816 @Yuhta Krishna, Michael, Jimmy, would you know how to unblock compilation on ubuntu 20.04?

What I did: https://github.com/facebookincubator/velox/pull/1505/commits/136ef7f97f9cc6a466eef9a98238fc6a8d688a60#diff-c388f86bda09fbfc530dba49b80d1c22912c27c22a8b412d51b405304ccb8810L509-R509

@Yuhta Thank you for sharing a workaround. Just to clarify, the change is to replace  __builtin_ia32_tzcnt_u64 call with __builtin_ctzll. Is that right?

CC: @fbalicchia Would you like to try the suggestion above from @Yuhta ?

@Yuhta Thank you for sharing a workaround. Just to clarify, the change is to replace __builtin_ia32_tzcnt_u64 call with __builtin_ctzll. Is that right?

Yes exactly


ImportImages should take in an array of io.Reader

func ImportImages(rs io.ReadSeeker, w io.Writer, imgFiles []string, imp *pdf.Import, conf *pdf.Configuration) error
takes in imgFiles []string which is paths in the filesystem to the images that are being imported.
It should instead take in an array of Readers.

Yes I am aware.
pdfcpu is relying on some magic of the image package in order to detect the image file type.
The image file type controls the PDF filter that needs to be applied in order to import an image into the PDF stream. We'll have to look for a workaround.


[Question] Can i send a request that without 'Host' header?

Hi guys. I want to send an HTTP request that without Host header for debugging of my server. Can I do it?
--- Thanks for this great App. Cheers

Hi,
Thank you for the question.
Well, basically you can't when using this app. Although it is possible (according to the spec) to send the Request with an absolute URI but the app will always split it into the path and host header. This is how the transport library was made.
So it always gonna be something like:
GET /get HTTP/1.1
HOST: httpbin.org

instead of
GET httpbin.org/get HTTP/1.1

It could be changed in the transport library but this change is quite big and I can't afford to do it right now. I hope you understand.
It may change in the future but as for now I'm not planning to change this behavior.


Bug in DependencyGraph._tree

Running the dependencygraph doctests produces an error, probably due to recent changes in DependencyGraph. It shows up in https://github.com/nltk/nltk/blob/develop/nltk/parse/dependencygraph.py#L265
@dimazest – would you be able to look at this please?
Traceback (most recent call last):
  File "dependencygraph.py", line 627, in <module>
    demo()
  File "dependencygraph.py", line 421, in demo
    malt_demo()
  File "dependencygraph.py", line 451, in malt_demo
    tree = dg.tree()
  File "dependencygraph.py", line 281, in tree
    return Tree(word, [self._tree(i) for i in deps])
  File "dependencygraph.py", line 281, in <listcomp>
    return Tree(word, [self._tree(i) for i in deps])
  File "dependencygraph.py", line 265, in _tree
    word = node['word']
KeyError: 'word'

There's a related problem for the non-projective dependency parser:
Traceback (most recent call last):
  File "nonprojectivedependencyparser.py", line 766, in <module>
    demo()
  File "nonprojectivedependencyparser.py", line 724, in demo
    nonprojective_conll_parse_demo()
  File "nonprojectivedependencyparser.py", line 743, in nonprojective_conll_parse_demo
    for parse_graph in npp.parse(['Cathy', 'zag', 'hen', 'zwaaien', '.'], ['N', 'V', 'Pron', 'Adj', 'N', 'Punc']):
  File "nonprojectivedependencyparser.py", line 465, in parse
    g_graph.connect_graph()
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/nltk-3.0.0-py3.4.egg/nltk/parse/dependencygraph.py", line 110, in connect_graph
    node1['deps'].append(node2['address'])
AttributeError: 'collections.defaultdict' object has no attribute 'append'

I'll try to have a look by the end of the week.

Thanks. It's one of the few things to resolve before 3.0.2 can be released.

There's another downstream problem with the projective dependency parser:
  File "projectivedependencyparser.py", line 557, in <module>
    demo()
  File "projectivedependencyparser.py", line 468, in demo
    projective_rule_parse_demo()
  File "projectivedependencyparser.py", line 487, in projective_rule_parse_demo
    for tree in trees:
  File "projectivedependencyparser.py", line 195, in parse
    dg = DependencyGraph(conll_format)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.2/lib/python3.2/site-packages/nltk-3.0.0-py3.2.egg/nltk/parse/dependencygraph.py", line 71, in __init__
    cell_separator=cell_separator,
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.2/lib/python3.2/site-packages/nltk-3.0.0-py3.2.egg/nltk/parse/dependencygraph.py", line 245, in _parse
    "The graph does'n contain a node "
nltk.parse.dependencygraph.DependencyGraphError: The graph does'n contain a node that depends on the root element.

This is holding up 3.0.2.

I just saw the PR. Thanks @dimazest – I hope that solves the problems.

Unfortunately there are still problems with our dependency parsers, which seem to have arisen after #822 was merged. @dimazest I'd be grateful if you would look into it some more please.

right, i've noticed problems with nltk.parse.nonprojectivedependencyparser.ProbabilisticNonprojectiveParser but didn't see a clear solution right on spot. I'll have a look later this week.

@longdt219 would you please take a look at these problems soon? They're holding up 3.0.2. I'd rather not have to revert @dimazest's PR #822.

yes I will @stevenbird

Thanks for #869 @longdt219

These issues have been resolved in #870 and #872.

We still seem to have a problem stemming from #822:
Traceback (most recent call last):
  File "/scratch/jenkins/python/python-2.6.9-x86_64/lib/python2.6/unittest.py", line 279, in run
    testMethod()
  File "/scratch/jenkins/workspace/nltk/TOXENV/py26-jenkins/jdk/oraclejdk8/nltk/test/unit/test_corpora.py", line 137, in test_parsed_sents
    '.'
  File "/scratch/jenkins/python/python-2.6.9-x86_64/lib/python2.6/unittest.py", line 350, in failUnlessEqual
    (msg or '%r != %r' % (first, second))
AssertionError: Tree('fortaleci\xf3', ['hoy', 'considerablemente', Tree('aumento', ['El', Tree('del', [Tree('\xedndice', [Tree('de', [Tree('desempleo', ['estadounidense'])])])])]), '.', Tree('al', [Tree('euro', [Tree('cotizaba', [',', 'que', Tree('a', ['0,9452_d\xf3lares']), Tree('a', [Tree('15.35', ['las', 'GMT'])]), Tree('en', [Tree('mercado', ['el', Tree('de', ['divisas']), Tree('de', ['Fr\xe1ncfort'])])]), Tree('frente_a', [',', Tree('0,9349_d\xf3lares', ['los', Tree('de', [Tree('ma\xf1ana', ['esta'])])])]), 'se'])])])]) != Tree('fortaleci\xf3', [Tree('aumento', ['El', Tree('del', [Tree('\xedndice', [Tree('de', [Tree('desempleo', ['estadounidense'])])])])]), 'hoy', 'considerablemente', Tree('al', [Tree('euro', [Tree('cotizaba', [',', 'que', Tree('a', [Tree('15.35', ['las', 'GMT'])]), 'se', Tree('en', [Tree('mercado', ['el', Tree('de', ['divisas']), Tree('de', ['Fr\xe1ncfort'])])]), Tree('a', ['0,9452_d\xf3lares']), Tree('frente_a', [',', Tree('0,9349_d\xf3lares', ['los', Tree('de', [Tree('ma\xf1ana', ['esta'])])])])])])]), '.'])

https://nltk.ci.cloudbees.com/job/nltk/TOXENV=py26-jenkins,jdk=oraclejdk8/lastCompletedBuild/testReport/nltk.test.unit.test_corpora/TestCoNLL2007/test_parsed_sents/

The dependencies doctest is also broken by this PR:
cd nltk/test
make dependency.errs

@dimazest this is blocking the 3.0.2 release

I'll have a look as soon as I get to my laptop.


typelevel dependency

For moving this package to purescript-contrib, @thomashoneyman has requested that the package has no dependencies outside of purescript-contrib, and the purescript-typelevel package does not belong to purescript-contrib.
The BytesPerValue class with the dependency on purescript-typelevel was introduced in this redesign by @athanclark .
https://github.com/jacereda/purescript-arraybuffer/pull/23
Do you have any advice about this @jacereda @athanclark ? How would you approach this?

From purescript-typelevel we only use the toInt' method of the Nat class on types D1, D2, D4, and D8.
Here's one thing which I think we could do which I think would completely eliminate the need for purescript-typelevel.
We could replace the BytesPerValue typeclass and replace it with something like this:
class BytesPerType (a :: ArrayViewType) where
  byteSize :: (Proxy a) -> Int

instance bytesPerTypeInt8 :: BytesPerType Int8 where byteSize _ = 1
instance bytesPerTypeInt16 :: BytesPerType Int16 where byteSize _ = 2
…

Then we would also be able to remove the Nat type constraint on a lot of public API functions, which would be a win in my opinion.

Resolved in v11.0.0 .


docs: pattern guidance for multi-task form

Added guidance for using the multi-task form pattern
View preview
Checklist
Preflight

[x] Prefix the PR title with the slug of the package or component - e.g. accordion: Updated padding or docs: Updated header links
[x] Describe the changes clearly in the PR description
[x] Read and check your code before tagging someone for review
[x] Create a changeset file by running yarn changeset. Learn more about change management.

Documentation

[x] Create or update documentation on the website

It will be better to add a live code snippet rather than an image for the return to partial completion example


Use elementPlaceholder instead of shadowRoot

Could you just create a separate demo instead? I can add it to the docs
page as well.
On Tue, Sep 22, 2015, 9:26 AM Justin Ridgewell<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/google/incremental-dom/pull/142
Commit Summary

Use elementPlaceholder instead of shadowRoot

File Changes

M demo/customelement.html
https://github.com/google/incremental-dom/pull/142/files#diff-0 (5)
M demo/define_component.js
https://github.com/google/incremental-dom/pull/142/files#diff-1 (17)

Patch Links:

https://github.com/google/incremental-dom/pull/142.patch
https://github.com/google/incremental-dom/pull/142.diff

—
Reply to this email directly or view it on GitHub
https://github.com/google/incremental-dom/pull/142.

While the demo doesn't do it, using elementPlaceholder wouldn't allow you to pass children. Shadow DOM is coming soon (the last browser, safari is working on it.
Ultimately, I believe web components are the right way forward and I'm not going to suggest to people, through a demo or otherwise to use a different approach.


Added resources to mongodb deployment

Codecov Report

Merging #2677 (9cb544a) into master (af5632d) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #2677      +/-   ##
==========================================
- Coverage   36.09%   36.07%   -0.03%
==========================================
  Files         161      161
  Lines        9242     9248       +6
  Branches      195      195
==========================================
  Hits         3336     3336
- Misses       5371     5377       +6
  Partials      535      535

Flag
Coverage Δ

moduleA
22.94% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...uation-details/ktb-evaluation-details.component.ts
11.92% <0.00%> (ø)

mongodb-datastore/handlers/events.go
23.38% <0.00%> (-0.32%)
:arrow_down:

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 04e08cc...9cb544a. Read the comment docs.


Add ArgoCD provider and rework the static one

Add an ArgoCD provider that allows to discover projects to check from ArgoCD application resources. The daemon is reworked to take providers as parameter
Create a static provider to manage static projects

#sre

Lint timeout, could you check?


Map , Area and Item

Create a Map section where the user will see all the items and areas registered around his position in 10km.
Make a page to create a new item (trails, hole, water, plant.. ) and a new area (national conservation area, urban area ....)
Leaders of brigades must be able to approve those items, so make a page to see items to approve.
For now make an alert to admin when new item is created and inform the user that this will be approved (send a email to user when approve)

Imlemented  the maps, link to add a new item, process of approval.. sending email to user when approved as well


I'm trying to override ./themes/anake/_default/baseof.html with layouts/_default/baseof.html but it seems to not be working. Changes in layouts/_default/baseof.html aren't reflected in the build while changes in the themes baseof.html are.
According to Hugo's lookup order I think this should work. The doc says:
/layouts/_default/baseof.html
/themes//layouts/_default/baseof.html
What am I doing wrong?

Hi @mbernico Sounds like you're doing it right. Can't say more without seeing your code. You'll also get more eyeballs on it by asking in the forums.

I agree with @budparr. Could you post some examples of your code or perhaps fork the repo? I'm curious to look into this.

@budparr @susodapop This was my dumb mistake guys, I'm sorry to bug you. I named the folder _defaults instead of _default and had been staring at it way too long.


feat: added support for mssql v11

refs https://jsw.ibm.com/browse/INSTA-8736

LGTM
Please consider my suggestion: https://github.com/instana/nodejs/pull/1190/files#r1647083061
Otherwise, feel free to merge

Considering that we still support node versions 16 and 14, and the latest MSSQL doesn't support these versions, I intend to retain the tests for versions 10 and 11.


Update enum values for travelHistory.purpose per spec

Still TBD whether we're putting enums in the schema so will put it for now but happy to pare down the schema later

 Updates to schema based on standard spec

Codecov Report

Merging #283 into master will increase coverage by 22.12%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master     #283       +/-   ##
===========================================
+ Coverage   71.17%   93.29%   +22.12%
===========================================
  Files          33       15       -18
  Lines         666      194      -472
  Branches      156       29      -127
===========================================
- Hits          474      181      -293
+ Misses        192       13      -179

Impacted Files
Coverage Δ

data-serving/data-service/src/model/travel.ts
100.00% <100.00%> (ø)

...tion/curator-service/ui/src/components/Profile.tsx

verification/curator-service/ui/src/setupProxy.js

...ation/curator-service/ui/src/components/Navbar.tsx

.../curator-service/ui/src/components/NewCaseForm.tsx

...i/src/components/new-case-form-fields/Location.tsx

...r-service/ui/src/components/CompletenessCharts.tsx

...ator-service/ui/src/components/FreshnessCharts.tsx

verification/curator-service/ui/src/index.tsx

...urator-service/ui/src/components/LinelistTable.tsx

... and 9 more

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 3ef4b5e...661dfc9. Read the comment docs.


GLSL > MSL imageAtomicAdd error

Version: 1.0.26
This GLSL code:
...
layout(binding = 15, r32ui) uniform uimageBuffer gLightsCounter;
...
uint _449 = imageAtomicAdd(gLightsCounter, 0, 1u);

generates wrong MSL code:
uint _449 = atomic_fetch_add_explicit((volatile thread atomic_uint*)&(gLightsCounter, 0), 1u, memory_order_relaxed);

The error:
[MoltenVK ERROR] VK_ERROR_INITIALIZATION_FAILED: Shader library compile failed (error code 3):
Compilation failed:
program_source:170:87: warning: expression result unused
uint _449 = atomic_fetch_add_explicit((volatile thread atomic_uint*)&(gLightsCounter, 0), 1u, memory_order_relaxed);
^~~~~~~~~~~~~~
program_source:170:85: error: cannot take the address of an rvalue of type 'int'
uint _449 = atomic_fetch_add_explicit((volatile thread atomic_uint*)&(gLightsCounter, 0), 1u, memory_order_relaxed);
^ ~~~~~~~~~~~~~~~~~

There is no "right" code for this, because Metal doesn't support this. This is known to Apple (rdar://41859236); dunno when it'll be fixed.

But when it is fixed in Metal, we'll still be generating incorrect code for this. But that's an issue with SPIRV-Cross, which MoltenVK uses to handle shader modules.

Is there anything to be done on this issue for now...or can we close it?


Little improvement of stack.yaml

$ nix-shell --run ./scripts/build/cardano-sl.sh throws exception
You cannot have packages and a shell-file filled at the same time in your nix-shell configuration.
because ~/.stack/config.yaml contains
nix:
  enable: true
  packages: [zlib.dev, zlib.out]

The patch below would solve the issue.
diff --git a/stack.yaml b/stack.yaml
index 306ea7fad..8d333a1d3 100644
--- a/stack.yaml
+++ b/stack.yaml
@@ -119,6 +119,7 @@ packages:

 nix:
   shell-file: shell.nix
+  packages: []

 extra-deps:
 - ekg-core-<IP_ADDRESS>                # https://github.com/tibbe/ekg-core/pull/21

I recommend not having global packages specified, since you'll have to override each project that uses the shell-file.

Sure, it's up to you


Cannot shrinkwrap with 3.3.0

I can no longer run a shrinkwrap when nock is added as a devDependency. This seemed to happen > 3.0 but I haven't looked too deeply into exactly which version caused it.
The error looks like:
npm ERR! Darwin 15.0.0
npm ERR! argv "node" "/usr/local/bin/npm" "shrinkwrap"
npm ERR! node v0.12.0
npm ERR! npm  v2.14.11

npm ERR! Problems were encountered
npm ERR! Please correct and try again.
npm ERR! extraneous<EMAIL_ADDRESS>/Users/jakecraige/gitrepos/nock-shrinkwrap/node_modules/nock/node_modules/changelog

I've created a repo to reproduce https://github.com/jakecraige/nock-shrinkwrap
Steps to reproduce:

Clone repo git clone https://github.com/jakecraige/nock-shrinkwrap.git && cd nock-shrinkwrap
npm install
npm shrinkwrap

This happens if I use NPM v2.11.3.
If I use NPM v3, it works.
This looks like an NPM bug, since NPM shouldn't care about dev dependencies..
Please reopen if you think this is something that nock should solve.

I am having the same issue.
If you download the latest package you can see that the extraneous package changelog, which shrinkwrap complains about is actually included in the tarball.
This is not a bug in npm, this is a problem with the published package.
http://registry.npmjs.org/nock/-/nock-3.3.2.tgz

(This might help: https://docs.npmjs.com/misc/developers#keeping-files-out-of-your-package)

ok I did some more investigation. Turns out this might be a bug in npm directly but not when installing modules but rather when packaging them up.
I have run npm pack with various different patterns to explicitly try and exclude the changelog module without any luck. From my understanding there is no reason npm should include it in the first place though even with the current configuration.
The only way I could get this to not be included in the package is to literally remove the dependency. It looks like it's a module that prefers to be installed globally anyways.
What is your publishing flow? Is this something that could be run in dev machines or do you publish as part of a CI pipeline?
For reference I'm on npm 2.14.4

I can use  global install to generate the changelog.
Removed the devDependency.
"Fix" landed on v3.4.0.

Unfortunately this is still an issue with nock 3.4.0.
Notice the changelog package which is bundled with the tarball...
➜  /tmp  wget http://registry.npmjs.org/nock/-/nock-3.4.0.tgz
--2015-12-15 13:44:06--  http://registry.npmjs.org/nock/-/nock-3.4.0.tgz
Resolving registry.npmjs.org (registry.npmjs.org)... <IP_ADDRESS>
Connecting to registry.npmjs.org (registry.npmjs.org)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 541202 (529K) [application/octet-stream]
Saving to: ‘nock-3.4.0.tgz’

nock-3.4.0.tgz                        100%[==========================================================================>] 528.52K  3.27MB/s   in 0.2s

2015-12-15 13:44:06 (3.27 MB/s) - ‘nock-3.4.0.tgz’ saved [541202/541202]

➜  /tmp  tar xzvf nock-3.4.0.tgz
package/package.json
package/.npmignore
package/README.md
package/index.js
package/assets/reply_file_1.txt
package/assets/reply_file_2.txt.gz
package/.travis.yml
package/CHANGELOG.md
package/.istanbul.yml
package/coverage/coverage.json
package/coverage/lcov-report/prettify.js
package/coverage/lcov-report/sorter.js
package/coverage/lcov-report/base.css
package/coverage/lcov-report/index.html
package/coverage/lcov-report/nock/index.html
package/coverage/lcov-report/nock/index.js.html
package/coverage/lcov-report/nock/lib/match_body.js.html
package/coverage/lcov-report/nock/lib/back.js.html
package/coverage/lcov-report/nock/lib/delayed_body.js.html
package/coverage/lcov-report/nock/lib/index.html
package/coverage/lcov-report/nock/lib/intercept.js.html
package/coverage/lcov-report/nock/lib/common.js.html
package/coverage/lcov-report/nock/lib/mixin.js.html
package/coverage/lcov-report/nock/lib/recorder.js.html
package/coverage/lcov-report/nock/lib/request_overrider.js.html
package/coverage/lcov-report/nock/lib/scope.js.html
package/coverage/lcov-report/nock/lib/socket.js.html
package/coverage/lcov-report/prettify.css
package/coverage/lcov-report/sort-arrow-sprite.png
package/coverage/lcov.info
package/examples/_log.js
package/examples/delay-connection.js
package/examples/delay-response.js
package/examples/net-connect-default-no-mock.js
package/examples/net-connect-default-other-mock.js
package/examples/net-connect-disabled-different-host.js
package/examples/net-connect-mock-same-host-different-path.js
package/examples/socket-delay-abort.js
package/examples/socket-delay-no-abort.js
package/.jshintrc
package/lib/back.js
package/lib/common.js
package/lib/delayed_body.js
package/lib/intercept.js
package/lib/match_body.js
package/lib/mixin.js
package/lib/recorder.js
package/lib/request_overrider.js
package/lib/scope.js
package/lib/socket.js
package/.coveralls.yml
package/tests/test_url_encoding.js
package/tests/test_browserify.js
package/tests/test_common.js
package/tests/test_data.js
package/tests/test_dynamic_mock.js
package/tests/test_basic_auth.js
package/tests/test_https_allowunmocked.js
package/tests/test_intercept.js
package/tests/test_ipv6.js
package/tests/test_isomorphic_fetch.js
package/tests/test_net_connect.js
package/tests/test_back_2.js
package/tests/test_nock_off.js
package/tests/test_back.js
package/tests/test_recorder.js
package/tests/test_aws_dynamo.js
package/tests/test_redirects.js
package/tests/test.js
package/tests/test_request_promise.js
package/tests/test_s3.js
package/tests/test_encode_querystring.js
package/tests/fixtures/browserify-script.js
package/tests/fixtures/goodRequest.json
package/tests/fixtures/wrongUri.json
package/tests/browserify-public/browserify-bundle.js
package/tests/browserify-public/.gitkeep
package/tests/browserify-public/index.html
package/node_modules/changelog/package.json
package/node_modules/changelog/.npmignore
package/node_modules/changelog/README.md
package/node_modules/changelog/LICENSE
package/node_modules/changelog/Gruntfile.js
package/node_modules/changelog/.travis.yml
package/node_modules/changelog/CHANGELOG.md
package/node_modules/changelog/bin/changelog.js
package/node_modules/changelog/bin/examples.sh
package/node_modules/changelog/examples/express.png
package/node_modules/changelog/examples/gulp.markdown
package/node_modules/changelog/examples/node.json
package/node_modules/changelog/lib/changelog.js
package/node_modules/changelog/lib/cli.js
package/node_modules/changelog/lib/log.js
package/node_modules/changelog/lib/datasrc/github.js
package/node_modules/changelog/lib/datasrc/npm.js
package/node_modules/changelog/lib/output/markdown.js
package/node_modules/changelog/lib/output/terminal.js
package/node_modules/changelog/lib/util/bullet.js
package/node_modules/changelog/lib/util/userAgent.js
package/node_modules/changelog/lib/util/versionFilter.js
package/node_modules/changelog/.idea/libraries/changelog_node_modules.xml
package/node_modules/changelog/.idea/.name
package/node_modules/changelog/.idea/dictionaries/dgreene.xml
package/node_modules/changelog/.idea/dictionaries/dylan.xml
package/node_modules/changelog/.idea/encodings.xml
package/node_modules/changelog/.idea/jsLibraryMappings.xml
package/node_modules/changelog/.idea/changelog.iml
package/node_modules/changelog/.idea/misc.xml
package/node_modules/changelog/.idea/modules.xml
package/node_modules/changelog/.idea/scopes/scope_settings.xml
package/node_modules/changelog/.idea/vcs.xml
package/node_modules/changelog/.idea/workspace.xml
package/node_modules/changelog/templates/changelog.md
package/node_modules/changelog/templates/readme/api.md
package/node_modules/changelog/templates/readme/cli.md
package/node_modules/changelog/templates/readme/how-it-works.md
package/node_modules/changelog/templates/readme/screenshots.md
package/node_modules/changelog/templates/readme/tests.md
package/node_modules/changelog/test/changelog.test.js
package/node_modules/changelog/test/github.test.js
package/node_modules/changelog/test/npm.test.js
package/node_modules/changelog/test/versionFilter.test.js
package/node_modules/changelog/test/fixtures/github-dylang-changelog.json
package/node_modules/changelog/test/fixtures/npm-changelog.json


Compare changes across branches, commits, tags, and more below

Q
A

Fixed Issues
Fixes #1, Fixes #2

Patch: Bug Fix?

Major: Breaking Change?

Minor: New Feature?

Tests Added/Pass?

Spec Compliancy?

License
MIT

Doc PR

Any Dependency Changes?

Codecov Report

Merging #551 into master will decrease coverage by 3.50%.
The diff coverage is 96.34%.

@@            Coverage Diff             @@
##           master     #551      +/-   ##
==========================================
- Coverage   91.78%   88.27%   -3.51%
==========================================
  Files         433      433
  Lines       27412    26765     -647
  Branches     8250     8102     -148
==========================================
- Hits        25161    23628    -1533
+ Misses       2221     2184      -37
- Partials       30      953     +923

Impacted Files
Coverage Δ

packages/babel-parser/src/parser/expression.js
98.18% <ø> (-0.80%)
:arrow_down:

packages/babel-parser/src/parser/lval.js
99.05% <ø> (-0.50%)
:arrow_down:

packages/babel-parser/src/parser/util.js
93.51% <ø> (-2.82%)
:arrow_down:

packages/babel-parser/src/plugins/estree.js
89.38% <ø> (-5.18%)
:arrow_down:

packages/babel-parser/src/plugins/flow.js
95.76% <ø> (-1.51%)
:arrow_down:

packages/babel-parser/src/plugins/jsx/index.js
97.33% <ø> (-1.05%)
:arrow_down:

...kages/babel-parser/src/plugins/typescript/index.js
92.87% <ø> (-3.86%)
:arrow_down:

packages/babel-parser/src/util/class-scope.js
97.56% <ø> (-2.44%)
:arrow_down:

packages/babel-parser/src/util/scope.js
100.00% <ø> (ø)

...proposal-logical-assignment-operators/src/index.js
95.23% <ø> (-4.77%)
:arrow_down:

... and 279 more

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 15d6da0...426acf3. Read the comment docs.


Project site usage is outdated: Radium.wrap

The project site details usage via React.createClass(Radium.wrap({...})).  This seems to have been removed and throws "not a function".
The README usage is module.exports = Radium(Component) or Component = Radium(Component) and works.
Thanks for cranking on this.

@alexlande Do we just have to update the gh-pages branch to update the website?

@ianobermiller yup

Done!

Nice.


Release?

@willow-ahrens can I release this and perhaps Finch depend? It's very clunky otherwise (I'd like Finch.jl to be an extension package to my packages right now, so I can't depend on Finch for it. But I'd like this in a variety of places!)

Added to JuliaSparse as owner, should be enough?

Hmm, I know you can construct one and do stuff with it (bcasting, multiplication, etc). I'm mostly trying to use it to set up solvers right now, but I think just some basic tests with SparseMatrixCSC should be fine.
As a test dep only though. I can do that in a bit.

Ok! I'm now happy to add CIndex as a Finch.jl dep as soon as it's released.


[WIP] Replace images used by machine readable metadata component

What
Replace the images below used by the machine readable metadata component:

govuk-logo.png
govuk-schema-placeholder-1x1.png
govuk-schema-placeholder-4x3.png
govuk-schema-placeholder-16x9.png

Why
Update the images to use the new tudor crown logo
Trello card
Visual Changes
The visual changes can be previewed on this commit - https://github.com/alphagov/govuk_publishing_components/pull/3797/commits/05b9b36519b1e29dc1b2d33f545d6453eaa075c6

Closing as the changes have been copied to https://github.com/alphagov/govuk_publishing_components/pull/3852


User is not authenticated on page refresh.

Description
User logs in with loginWithRedirect function. But on refreshing the page useAuth0 returns isAuthenticated as false.
But on calling longinWithRedirect user gets authenticated without any credentials.
Reproduction
I am using google oauth.
Environment

**Version of this library used:**1.0.0

Hi @a-c-sreedhar-reddy
It sounds like you're using the developer keys to test logging in with social identity providers. These developer keys have some limitations - in your case they prevent silent auth from working, which is why your login is not persisted between page refreshes.
If you change the keys of your Google social connection in the Auth0 dashboard, this should resolve the issue https://manage.auth0.com/#/connections/social
More info here: https://auth0.com/docs/connections/social/devkeys
Closing for now, feel free to reopen if you have any other questions

Hi @adamjmcgrath. I have added the google keys. Even then on page refresh isAuthenticated is false.
I have added the fetch requests below.
This one gets called when I refresh.
fetch("https://mydomain.us.auth0.com/authorize?client_id=**clientid**&redirect_uri=http%3A%2F%2Flocalhost%3A3000&scope=openid%20profile%20email%20offline_access&response_type=code&response_mode=web_message&state=UmxzMXRoZ0w4LnhBSDBaa0ExRDhrbU8tckNHfkRySW9wQ3RaSFpmdGZWaQ%3D%3D&nonce=T2UwLUFkWmc2Sy5kMGg3Wl9uWVA1cDMzfkUyLlE4VUprSEl4M005bWJPTw%3D%3D&code_challenge=gBfcHonM-5E2rhD6Q79zvvpgqWGczhrOFUVOvtmceHo&code_challenge_method=S256&prompt=none&auth0Client=**auth0Client**", {
  "headers": {
    "accept": "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9",
    "accept-language": "en-GB,en-US;q=0.9,en;q=0.8",
    "sec-fetch-dest": "iframe",
    "sec-fetch-mode": "navigate",
    "sec-fetch-site": "cross-site",
    "upgrade-insecure-requests": "1"
  },
  "referrer": "https://mydomain.us.auth0.com/",
  "referrerPolicy": "no-referrer-when-downgrade",
  "body": null,
  "method": "GET",
  "mode": "cors",
  "credentials": "omit"
});

This one gets called when  loginWithRedirect is called.
fetch("https://mydomain.us.auth0.com/authorize?client_id=**clientid**&redirect_uri=http%3A%2F%2Flocalhost%3A3000&scope=openid%20profile%20email%20offline_access&response_type=code&response_mode=query&state=aDh6cnNtWkFtVEpyZTBjR04zYV9NNTF1QzJJWjlCRXVSQ1JuZHZuNXUwNg%3D%3D&nonce=bTliSG5RUmdURE5GYm5QdnFGUHZacUpHdGpRQjFzN1lvQVV2Y3MySDVYYw%3D%3D&code_challenge=f9-iNbG8u0HCNlXfghT4UlbkPRZNVd7rdaiBOpKu3DM&code_challenge_method=S256&auth0Client=**auth0Client**", {
  "headers": {
    "accept": "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9",
    "accept-language": "en-GB,en-US;q=0.9,en;q=0.8",
    "sec-fetch-dest": "document",
    "sec-fetch-mode": "navigate",
    "sec-fetch-site": "cross-site",
    "sec-fetch-user": "?1",
    "upgrade-insecure-requests": "1"
  },
  "referrerPolicy": "no-referrer-when-downgrade",
  "body": null,
  "method": "GET",
  "mode": "cors",
  "credentials": "include"

Hi @a-c-sreedhar-reddy - can you let me know which browser are you using?

Hi @adamjmcgrath I am using Brave browser. I am sorry for not mentioning it in the issue.

Hey @adamjmcgrath. Just now checked in chrome. In chrome it is working fine.

Hey @a-c-sreedhar-reddy - you're running into the Intelligent Tracking Prevention (ITP) browser restrictions of Safari and Brave (and soon Chrome)
Your 2 options are to either use Rotating Refresh Tokens (see https://auth0.com/docs/libraries/auth0-spa-js#use-rotating-refresh-tokens) or a Custom Domain (See https://auth0.com/docs/custom-domains)
Closing for now, feel free to reopen if you have any more questions

I am having the same issue. I tried https://github.com/auth0/auth0-react/issues/49#issuecomment-654466353 but no success. I also tried enabling Rotating Refresh Tokens as mentioned in a previous comment, but still no avail.
I have documented my issue here before I came across this issue.

Hi @a-c-sreedhar-reddy
It sounds like you're using the developer keys to test logging in with social identity providers. These developer keys have some limitations - in your case they prevent silent auth from working, which is why your login is not persisted between page refreshes.
If you change the keys of your Google social connection in the Auth0 dashboard, this should resolve the issue https://manage.auth0.com/#/connections/social
More info here: https://auth0.com/docs/connections/social/devkeys
Closing for now, feel free to reopen if you have any other questions

If this is an expected behavior, then I suppose my auth is "working as expected". However, I'm still getting this error for Samesite cookies. That's what made me look deeper into this issue.


Once instance has previously been poisoned

Hi Prisma Team! My Prisma Client just crashed. This is the report:
Versions

Name
Version

Node
v16.14.2

OS
windows

Prisma Client
in-memory

Query Engine
da41d2bb3406da22087b849f0e911199ba4fbf11

Database
postgresql

Logs
prisma:tryLoadEnv Environment variables not found at
prisma:tryLoadEnv Environment variables not found at
prisma:tryLoadEnv No Environment variables loaded
prisma:client dirname C:\Users\petar\Documents\nordoyane.no\prisma
prisma:client relativePath
prisma:client cwd C:\Users\petar\Documents\nordoyane.no\prisma
prisma:client clientVersion: in-memory
prisma:client clientEngineType: library
prisma:client:libraryEngine internalSetup
prisma:client:libraryEngine:loader loadEngine using C:\Users\petar\Documents\nordoyane.no\node_modules\prisma\engines\da41d2bb3406da22087b849f0e911199ba4fbf11\query_engine-windows.dll.node
prisma:client:libraryEngine sending request, this.libraryStarted: false
prisma:client:libraryEngine library starting
prisma:client:libraryEngine sending request, this.libraryStarted: false
prisma:client:libraryEngine library already starting, this.libraryStarted: false
prisma:client:libraryEngine sending request, this.libraryStarted: false
prisma:client:libraryEngine library already starting, this.libraryStarted: false
prisma:client:libraryEngine library started
prisma:client:request_handler {"clientVersion":"in-memory"}
prisma:client:request_handler {"clientVersion":"in-memory"}

Client Snippet
// PLEASE FILL YOUR CODE SNIPPET HERE
npx prisma studio

Schema
// PLEASE ADD YOUR SCHEMA HERE IF POSSIBLE
// This is your Prisma schema file,
// learn more about it in the docs: https://pris.ly/d/prisma-schema

generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

model Location {
  id String @id @default(cuid())
  name String
  latitude Float
  longitude Float
  content String
}

Prisma Engine Query
{"X":{}}

Potenitally related
https://github.com/prisma/prisma/issues/14864

Hey @petmil123 could you maybe give us a small reproduction that triggers this error?

Internal: Making implicit knowledge from discussion explicit: Error message is probably from https://doc.rust-lang.org/std/sync/struct.PoisonError.html

Mostly likely happens due to a panic here:
https://github.com/prisma/cuid-rust/blob/05bdf4c4bac6cd340bf59af2f48bcb502371a1ba/src/lib.rs#L29

Duplicate: https://github.com/prisma/prisma/issues/14870


chore: turn on no-unsafe-optional-chaining rule again

Description Of Changes
Addressed a comment in https://github.com/sendbird/sendbird-uikit-react/pull/526#discussion_r1188223665 to bring back no-unsafe-optional-chaining rule again which is useful to have.
As I mentioned here,  I found there're several places(actually many) we're doing optional chaining unnecessarily. Some of them are caught by lint(fixed in this PR) but others are still there. We'd better revisit them too.

@HoonBaek Thanks!


Removed the underscore.js dependency

This pull request is a fix for #107. It gets rid of the underscore.js dependency while preserving compatibility with IE9+.

Hero! Amazing! Thank you!


Add WorldGuard API hook for land claim prevention

You should add worldguard support to Husktowns.
For example, allowing the ability to prevent claiming in certain regions for people that create regions as safezones can use this to prevent players from doing so.
You could use worldguard api to create a flag that can prevent this. Would be a very handy feature

Added in #435


It would be cool if one could access self-hosted generative models (Llama, WizardLLM, etc.)

Either over the network via Llamacpp/Kobold AI interfaces or running locally on your machine.
With the subscription prices being as they are, hosting your own model can be cheaper.

I've forked it and added some of that functionality. Its not complete, so I'm not making a PR for it yet, and I'm not sure he wants the changes :P

It'd be great if we could host and run our own models with Ollama (https://github.com/ollama/ollama).
They also host lots of models (https://ollama.com/library), it would be great if they would have a finetuned model for Godot 4.x code generation ^^
@minosvasilias I have just seen that you have several Godot code finetunes from about a year ago! :D Would you recommend them over newer models like LLaMA 3? Are you planning on finetuning any of the recently launched models? (and maybe adding it to the Ollama model library?) I'm somewhat baffled that the official Godot devs aren't supporting this initiative more actively ^^U By lowering the entrance barrier and enabling more people to code in Godot, we could all benefit!

I've forked it and added some of that functionality. Its not complete, so I'm not making a PR for it yet, and I'm not sure he wants the changes :P

That's amazing @Efoi! Will follow and star your fork - even if the PR won't be accepted here, I'll just use the fork.


dclexperience: Carousel at the spawn fountain is not working

Explorer Alpha build version:
Issue Description:

Clicking on the Places or Events billboards doesn't play shiny "popup" animation
Switching via E/F is not working
When switching billiboards via right/left 3d-arrows carousel often leaves E/F 3d-buttons in improper places (stuck with old billboard)

Steps to reproduce:

/goto dclexperience.dcl.eth
try to click on the Places or Events billoboards - Observe nothing happens
Press E/F - observe it does nothing
Press Right/Left 3d-arrow buttons - observe E/F buttons not changing position to proper places

Legend to image:

at the left you see big 3d arrow button
at the right you see small F 3d-button

Expected behaviour:

Bilboard should move in front of you with shiny animation (compare with web-explorer)
pressing E/F on keyboard should rotate carousel
E/F 3d-arrows should be on the focused billoboard after usage of the 3d big arrows

Reproduction rate:

[ ] < 10%
[ ]  < = 25%
[ ]  50%
[x]  < = 75%
[ ]  100%

Severity:

[ ] Cosmetic
[ ] Minor
[ ] Medium
[x] High
[ ] Critical
[ ] Blocker

Priority:

[ ] High
[ ] Medium
[ ] Low

Platforms / OS:

[ ] Windows 11
[ ] Windows 10
[ ] Mac Sonoma
[ ] Mac (specify the version on Additional Notes)

Evidence:
Additional Notes:

At build 0.3.402-g7a23dc7e-main Currently, it only works by clicking the arrows, the rest of the buttons (E/F clicking and on keyboard) are not working.

The carrousel is working, but there are some visual issues, also, the slots are not disabled, just hidden from view, causing issues when clicking on between carrousel cards on other carrousels...
Also the  buttons are not located correctly consistently...


Parallelization Support for Bidding and Auction Services

Will the Bidding and Auction Services support parallelization with the "contextual" ad request, as discussed in https://github.com/WICG/turtledove/issues/385?

Thanks.
We have proposed unification of Contextual and FLEDGE flow with Bidding and Auction Services. Refer https://github.com/privacysandbox/fledge-docs/blob/main/bidding_auction_services_api.md#unified-contextual-and-fledge-auction-flow-with-bidding-and-auction-services for more details.

We support the unified flow / unified ad request.
Seller's ad server may call B&A while the contextual auction is running. You may read more about the flow here- https://github.com/privacysandbox/fledge-docs/blob/main/bidding_auction_services_api.md#flow

Parallelizing Contextual / RTB and B&A auctions would be a further latency optimization.
We do plan to work with different SSPs to incorporate further parallelization of contextual / RTB auction and Bidding and Auction services, so that B&A can be called earlier. However, this needs to happen without impacting traffic shaping in seller's ad server.
We will keep you posted. Thanks.

This issue has been migrated to the new WICG/protected-auction-services-discussion repository.
Please continue the discussion in the migrated issue here: https://github.com/WICG/protected-auction-services-discussion/issues/12


Refactor to switch to snapshotter interface

This commit adopts snapshotter interface in install, reset and upgrade commands. The change implies changes to the respective specs, grub configuration and dracut modules.
This commit also changes the behavior of recovery system upgrades. Now recovery upgrades are an optional step of a system upgrade. Recovery image can't be upgraded without upgrading the active system.
Finally build-disk command is also changed to be better aligned with upgrade and install procedures. Expandable disks are an unprivileged build and non expandable ones require privileges as they relay on snapshotter.

Codecov Report
Attention: 211 lines in your changes are missing coverage. Please review.

Comparison is base (0bf2c64) 73.76% compared to head (00e7851) 72.57%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

pkg/action/upgrade.go
66.89%
41 Missing and 8 partials :warning:

pkg/action/reset.go
55.55%
27 Missing and 9 partials :warning:

pkg/action/install.go
80.00%
19 Missing and 3 partials :warning:

pkg/action/build-disk.go
71.01%
14 Missing and 6 partials :warning:

pkg/mocks/utils.go
53.12%
10 Missing and 5 partials :warning:

pkg/types/v1/config.go
70.73%
6 Missing and 6 partials :warning:

pkg/elemental/elemental.go
88.88%
7 Missing and 1 partial :warning:

pkg/snapshotter/loopdevice.go
41.66%
6 Missing and 1 partial :warning:

pkg/config/config.go
88.46%
4 Missing and 2 partials :warning:

pkg/types/v1/snapshotter.go
70.00%
4 Missing and 2 partials :warning:

... and 7 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1906      +/-   ##
==========================================
- Coverage   73.76%   72.57%   -1.19%
==========================================
  Files          71       72       +1
  Lines        7749     7818      +69
==========================================
- Hits         5716     5674      -42
- Misses       1612     1690      +78
- Partials      421      454      +33

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

We should do a code walk-through to get this merged 🤔

We could also reconsider setting constants.MaxSnaps = 2 as discussed yesterday.

Yes I have to apply these little changes yet. As soon as I manage to manually upgrade an elemental cluster from current staging to a Dev including this PR I'll mark this PR ready to be merged and arrange this little details we discussed yesterday.

Finally got green run on the CI here and managed to run a regular UI based upgrade from staging to this current head.


Query for Tamworth Agricultural Institute: Cropping system

[x] In the paper (Schwenke et al. 2018), crop sequence was mentioned from 2009 to 2012, but the Table 1  shows no crop after 2011. Can you please clarify?
[x] Need a clarification of soil moisture unit.
[x] Very high CO2 fluxes.

Tamworth min, max temperature fixed .


[Desktop / Windows] BUG: Black Screen / Broken Navigation after Forwarding Image

Tested on: Windows 11 Microsoft Store BlueBubbles Client
Version: <IP_ADDRESS>_Windows
Steps to reproduce: Select image from a chat and action to "forward" the image to another chat recipient. Attempt to back out of that chat (doesn't matter if you actually complete forwarding or not), and you end up on a black unusable screen that is only rectified by reloading the application.
Why this is a problem: Irritating and not a good user experience to have to relaunch the application.
Screenshot of the problem:

Thanks, team! I love the software, and this is a small issue that I figured I could help contribute by raising. I may even look into it myself.

Hi, this will be fixed in the upcoming v1.11 beta for desktop. Thanks for reporting


COMPVIS-3547

This pull request has 2 primary additions / modifications:
1.a. The addition of the function fetch_geometry_info() to image.py; this function returns a tuple of geometry data ('orientation', 'position', and 'spacing') from a CASTable. If the CASTable does not have geometry data columns the function returns a tuple of empty tuples.
1.b. Unit tests for evaluating fetch_geometry_info(). The unit tests check the return values for images with and without geometry data.
2.a. The  get_image_array_from_row() function now sets the type of the NumPy array directly. This addition is to insure the data type of the returned NumPy image arrays match the CASTable types (in addition to insuring the types are what we expect it prevents NumPy from using a more memory inefficient data type).
2.b. Added a unit test to check all possible data types in the CASTable -> NumPy array conversion in get_image_array_from_row().

I don't think I have write access to sassoftware/python-cvpy to merge in the request. If you could give me permissions I'll merge and close this out!


When I ban a user, there are no restrictions

Hint: The file where the user is banned can be found here - https://github.com/lazarnarcis/chat/blob/master/ban.php

Basically he bans him but he still has access to chat until he relogin.


Update booting-on-openstack.md

booting-on-openstack: Update some of the command args to openstack CLI
It seems that some command arguments changed over time. This updates a couple of them.

LGTM, thanks.


docs: markdown-content: use defineConfig consistently

What kind of changes does this PR include?
Minor content fixes (broken links, typos, etc.):

use defineConfig consistently
use the code "title" when not used

I find it confusing, when the docs sometimes use one pattern and other times something else. This changes it to use defineConfig everywhere on this page.

I just wanted to let you know that a PR tackling this was merged, hence the conflicts - #2635.
Thanks for considering contributing though, hope to see you around 🙂


Include serverId for reporter blocking functionality

We should include server id as well, so that it isn't a global block, but rather a guild based one.
Originally posted by @Jasius in https://github.com/Microsoft-community/Dotsimus/pull/114#r697809799

Completed in #128


lock_manager: polish waiter manager

What have you changed?

If more than one waiters are waiting for the same lock, only the oldest one will be waked up immediately when the lock is released. Others will be waked up after wake_up_delay_duration to reduce contention and make the oldest one more likely acquires the lock. If the lock is released again, the second oldest will be waked up immediately, and so on.
Each waiter has a maximum lifetime -- wait_for_lock_timeout to prevent starvation.
Use lock hash as the key of WaitTable.
Some refactor.

What is the type of the changes?

Improvement (a change which is an improvement to an existing feature)

How is the PR tested?

Unit test
Manual test (add detailed scripts or steps below)

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?

LGTM

/run-all-tests

/merge

/merge


File Upload Bug When Display And View Page

Package
filament/tables
Package Version
v3
Laravel Version
v10
Livewire Version
v3
PHP Version
8.2
Problem description

after uploading then error
but image success uploaded to dir
Expected behavior
no
Steps to reproduce
not displayed on list and edit page
Reproduction repository
https://github.com/DeviJv/error-select-form-loop-json-array-filament
Relevant log output
No response

Not a good enough reproduction. It is not minimal. We do not even have all the files required to run it. Please open a discussion instead, it's probably not a bug.


Compilation error on Alpine Linux

Describe the bug
Unable to complete the compilation of the binary on Alpine Linux due to errors (mostly "async trait functions are not currently supported". Stop at crunchy-cli-core (lib) d
To Reproduce
Steps / command to reproduce the behavior:
$ export RUST_BACKTRACE=full
$ apk add git cargo pkgconfig openssl openssl-dev ffmpeg nano

$ git clone https://github.com/crunchy-labs/crunchy-cli
$ cd crunchy-cli
$ git checkout v3.2.2
$ cargo build --release

Expected behavior
A complete compilation
Client (please complete the following information):

OS: Alpine Linux
Version 3.19
running on ProxMox LXD [Linux crunchyroll 6.5.11-5-pve #1 SMP PREEMPT_DYNAMIC PMX 6.5.11-5 (2023-11-27T19:52Z) x86_64 Linux]
rustc 1.72.1 (d5c2e9c34 2023-09-13) (Alpine Linux 1.72.1-r0)
cargo 1.72.1

Additional context

Version 3.1.1 can compile
Note: the commit  https://github.com/crunchy-labs/crunchy-cli/commit/283a3802b22d556eade5b8fca6114025efc6f879 is the one where 'async-trait' library is no longer in use.

Logging::
error[E0706]: functions in traits cannot be declared async
--> crunchy-cli-core/src/utils/filter.rs:80:5
|
80 |     async fn finish(self, input: VecSelf::T) -> ResultSelf::Output;
|     -----^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|     |
|     async because of this
|
= note: async trait functions are not currently supported
= note: consider using the async-trait crate: https://crates.io/crates/async-trait
= note: see issue #91611 https://github.com/rust-lang/rust/issues/91611 for more information
error[E0706]: functions in traits cannot be declared async
--> crunchy-cli-core/src/lib.rs:31:5
|
31 |     async fn execute(self, ctx: Context) -> Result<()>;
|     -----^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|     |
|     async because of this
|
= note: async trait functions are not currently supported
= note: consider using the async-trait crate: https://crates.io/crates/async-trait
= note: see issue #91611 https://github.com/rust-lang/rust/issues/91611 for more information
error[E0658]: use of unstable library feature 'io_error_other'
--> crunchy-cli-core/src/utils/rate_limit.rs:57:30
|
57 |                     .map_err(io::Error::other)
|                              ^^^^^^^^^^^^^^^^
|
= note: see issue #91946 https://github.com/rust-lang/rust/issues/91946 for more information
Some errors have detailed explanations: E0658, E0706.
For more information about an error, try rustc --explain E0658.
error: could not compile crunchy-cli-core (lib) due to 31 previous errors
warning: build failed, waiting for other jobs to finish...
error: could not compile crunchy-cli-core (lib) due to 31 previous errors

Your Rust version is too old. Please use at least Rust 1.75.0. You can use rustup.rs to manually install Rust.

Thank you bytedream. It worked.
For information.I use
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh
And probably use apk build-basen openssl-dev
export OPENSSL_INCLUDE_DIR=/usr/include


BUG: SparseDataFrame indexing may return normal Series

[x] related to #4400 (not close yet)
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

Found a below bug in SparseDataFrame indexing.
# NG, must be SparseSeries
df = pd.DataFrame([[1, 2], [np.nan, 4]]).to_sparse()
type(df.loc[0])
# pandas.core.series.Series

Rebased, and now green.

thanks!