THREESCALE-8508 - /admin/api/account/proxy_configs endpoint for configuration loading

This PR uses /admin/api/account/proxy_configs for the standard configuration loading process, as the default endpoint that APIcast will attempt to fetch the configuration from when the portal endpoint is provided without a path (admin portal endpoint).
With this proposed implementation, APICAST_LOAD_SERVICES_WHEN_NEEDED is deprecated and the configuration is fetched "when needed" by default whenever possible, with the following logic:

THREESCALE_PORTAL_ENDPOINT has no path

APICAST_CONFIGURATION_LOADER=boot

http://${THREESCALE_PORTAL_ENDPOINT}/admin/api/account/proxy_configs/<env>.json?version=version (i.e. all services from proxy_configs endpoint). Using the new endpoint now one request is enough, no need to loop through all services.

APICAST_CONFIGURATION_LOADER=lazy

http://${THREESCALE_PORTAL_ENDPOINT}/admin/api/account/proxy_configs/<env>.json?host=host&version=version (i.e. services that match host from proxy_configs endpoint). This matches the current behaviour when APICAST_LOAD_SERVICES_WHEN_NEEDED is enabled.

THREESCALE_PORTAL_ENDPOINT has a path

APICAST_CONFIGURATION_LOADER=boot

http://${THREESCALE_PORTAL_ENDPOINT}/<env>.json (i.e. all services from master endpoint). Same as the current behaviour.

APICAST_CONFIGURATION_LOADER=lazy

http://${THREESCALE_PORTAL_ENDPOINT}/<env>.json?host=host (i.e. services that match host from master endpoint). Same as the current behaviour.

i.e. the path appended to ${THREESCALE_PORTAL_ENDPOINT} is:

APICAST_CONFIGURATION_LOADER=boot
APICAST_CONFIGURATION_LOADER=lazy

endpoint has no path
/admin/api/account/proxy_configs/${env}.json?version=version
/admin/api/account/proxy_configs/${env}.json?host=host&version=version

endpoint has a path
/${env}.json
/${env}.json?host=host

Update : When any of the following environment variables exist, the gateway will load services one by one.

APICAST_SERVICE_%s_CONFIGURATION_VERSION
APICAST_SERVICES_LIST
APICAST_SERVICES_FILTER_BY_URL

These environment variables are not compatible when pointing to master (when a custom path is provided). Added a check to log error when that incompatible scenario happens.

Code looks ok. Also tests.
Just to verify: APIcast no longer will request proxy configuration for a given service ID. When "lazy" configuration, the endpoint will have a query param for the given "host". Right?
Question: what if two services have the same host?

For managed APIcast deployment (on-prem releases), the THREESCALE_PORTAL_ENDPOINT env var has path configured to /master/api/proxy/configs
https://github.com/3scale/3scale-operator/blob/69c1e2c41b2b9d308278e513d07d405b5d8196e6/pkg/3scale/amp/component/system_options.go#L114
This is still valid or should be changed?

Code looks ok. Also tests.
Just to verify: APIcast no longer will request proxy configuration for a given service ID. When "lazy" configuration, the endpoint will have a query param for the given "host". Right?
Question: what if two services have the same host?

That's fine @eguzki, since it points to master, this PR should have no effect on gateways configured that way.

Is this lost in the ether?

Is this lost in the ether?

No @secretsurf sorry I am supposed to be reviewing and testing this but too many other burning issues I didn't manage to get to this yet.

One thing I concern about this is the paginated endpoint. From the description in the issue
Open Points:

    Currently the endpoint /admin/api/account/proxy_configs/${ENV}

 is paginated, and APIcast does not handle pages, so we would need to
make the pagination optional or to implement pagination on APIcast too before we can proceed with this

and I do not remember to see any pagination implementation. Maybe it was done after my initial review

@pehala @thomasmaas this PR is ready for review

leaving pagination implementation for a (soon) coming PR

This seems like a breaking change, can you enable the old behaviour? Does enabling the old behaviour even make sense?

Basically, we are making an opt-in behavior the default one. Opt-in is no longer needed, hence removal of env var.

This seems like a breaking change, can you enable the old behaviour? Does enabling the old behaviour even make sense?

Basically, we are making an opt-in behavior the default one. Opt-in is no longer needed, hence removal of env var.

Right, and is there a option to opt-out? Does it even make sense to opt-out or is the new behaviour simply superior?

This seems like a breaking change, can you enable the old behaviour? Does enabling the old behaviour even make sense?

Basically, we are making an opt-in behavior the default one. Opt-in is no longer needed, hence removal of env var.

Right, and is there a option to opt-out? Does it even make sense to opt-out or is the new behaviour simply superior?

For me it does not make sense to opt-out. But if for whatever reason there is a need to opt-out from default behavior (i.e. get all services and ask for proxy config for each service), there are ways.

Use APICAST_SERVICE_${SERVICE_ID}_CONFIGURATION_VERSION env var with a not existing ${SERVICE_ID}
Use APICAST_SERVICES_FILTER_BY_URL with a filter that always match.


⚠ Fetchers has degraded performance

In 1398246, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 509 ms

Resolved: Fetchers performance has improved in 3b9b18e.


Better to have slides on proper archival site, get DOI, and just list here

It may be too late for this year, but let me say that I don't think posting all the slides in a GitHub repo is a great idea. It would be much better for authors to deposit their slides in a proper archival site, get a DOI, and use the GitHub repo to simply list the presentations, with complete citation including their DOI.
I, for one, prefer not to add my slides on this repo, as I already have uploaded them to Figshare, so the citation is:

Barba, Lorena A. (2018): Engineers Code: re-usable, open educational modules for engineering undergraduates. figshare. Presentation. https://doi.org/10.6084/m9.figshare.6818279.v1

@labarba I agree and I did the same. I also noticed that a few of the posters in the README are DOI links to zenodo and F1000, so clearly some people are already doing this and got merged.
Do you think linking using the DOI is enough or would it have to be a full citation?

It's always a good idea to remind people how you want your work to be cited (many people don't know!).


Deprecated PHP ini config

the PHP init config assert.active = 0 is deprecated since PHP 8.3
https://www.php.net/manual/de/info.configuration.php#ini.assert.active
should be removed or exchanged

@shyim now the config contains two times zend.assertions first with -1 and second with 0


chart.options undefined after exportChart

Inside a controller we are saving a reference to the chart object as in the following:
var _chart;
// ...
$scope.chartConfig = {
  // ...
  func: function (chart) {
    _chart = chart;
  },
};

After updating the series with data, we export the diagram as in the following:
_chart.exportChart(someExportOptions);

The next time we have to access the chart object, we find that the object is somehow different. What we mean is that the object is still an instance of Chart but loses all of its public attributes. In particular it loses options, but many others as well (see pasted pictures below).
After exporting, the object only contains some jQueryNNNNNNNN property and then a bunch of functions are to be found in the prototype, and those functions do not seem to be the same functions as before.
Chart object before:

Chart object after:

This is exactly the issue I've been banging my head against last few days. I run my export once and it's fine, and the second time it errors.
I'm adding the charts into an array in a service at the end of chartconfig, in the func: callback. Should I be using that getHighcharts funciton there?


capistrano: can't modify frozen object using a github repo

If I use gem "pdfkit", "0.4.6" from rubygems everything works fine.
But when I switch to a github repo like this:
gem "pdfkit", :git => "https://github.com/omsoft/pdfkit.git", :tag => "v0.4.6"
I get these errors:
 **Retrying source fetch due to error (2/3): TypeError can't modify frozen object
 ** Retrying source fetch due to error (3/3): TypeError can't modify frozen object
 ** Unfortunately, a fatal error has occurred. Please see the Bundler
 ** troubleshooting documentation at http://bit.ly/bundler-issues. Thanks!
***  /containers/30028/.rvm/rubies/ruby-1.8.7-p374/lib/ruby/1.8/net/https.rb:138:in `verify_mode='
*** : can't modify frozen object (TypeError)
***  from /containers/30028/.rvm/rubies/ruby-1.8.7-p374/lib/ruby/1.8/net/https.rb:138:in `verify_mode='

I'm using bundler 1.6.5. The command used by capistrano is:
bundle install --gemfile /containers/30028/myapplication_name/releases/20191010104232/Gemfile --path /containers/30028/myapplication_name/shared/bundle --deployment --quiet --without development test
Why this is happening?

It sounds like you're using ruby 1.8.7 and bundler 1.6.5. Those are really ancient, can you upgrade to a newer ruby & bundler version and try again?

No reply from original author. Closing.


Incorrect calculation of the next move

Hi Gavin,
Thanks so much for developing this tiny but powerful engine.
I have been enjoying using it, however, today I noticed that the engine calculates the next move of the below position very much incorrectly.
Position: 2k4r/2p1bpp1/p7/1p2n2b/3pPP2/P2P3q/1PP4N/R3QRK1 b - f3 0 24
I checked it with the two engines below and they didn't suggest that move even with the lowest rating (1300):

https://nextchessmove.com
https://www.365chess.com/play_computer_online.php

I also maxed "thinking time", but got the same result.
I am not sure if you answer this type of question, but thought I ask anyway since it looks like a bug.
Appreciated!

I also maxed "thinking time", but got the same result.

i didn't .. i guess you used the BetaFish version at https://betafish.gavinong.com .. it gives Ng6 at 1 second thinking time, but Qxh2+ at 3,5,10 sec .. could it be different for you because our hardware / connection differs ? I doubt it .. you didn't mention your BetaFish moves .. i just did an analyses with the new and very strong Shashchess GZ NNUE engine, for reference :
indeed, many other moves are better, they even lead to a quick mate, but both Ng6 (11-th) and Qh2+ (12-th) appear in this top-12 list, being clearly winning also .. it seems this position is much better for Black in many variations .. so, should BetaFish choice really be considered a bug ?

To see where Qh2 could lead, I had betafish to play against itself. It turned out that it would lead to checkmate.
So, fair enough; it is not a bug.
However, I still feel this can be improved.
@Strryke Any chance you can have a look?

Hey, thanks for this, it's nice to know people are playing around with Betafish.
I went to take a quick check, basically, it's not a bug — the engine just doesn't look far enough to see the mate in 8 at a lower thinking time and thus doesn't find a better move than Ng6.
I maxed out the thinking time and the engine finds the best move at Depth 7, while at a thinking time of 1 second, the engine only manages to reach depth 4 and decides Ng6 is the best move. I've attached the logs below.

Thanks for the question! I always love revisiting this project.


Feature/material redesign

Updated UI templates to use Material Design, using the material.io library, replacing the existing Twitter Bootstrap theme in order to create a similar design aesthetic to the CAS Management UI application.

Bootstrap is retained, but only the grid/layout utilities, for responsive layouts.
Switched to materialdesignicons.com for user interface icons, as it provides a much larger set (including social brands) than Google's Material Icons, but follows a similar design spec.
Implements normalize.css to provide a baseline stylesheet consistent across browsers.
Once Material.io's "banner" component is released, CAS should update notifications/banners to match.

https://material.io/
https://materialdesignicons.com/
Example screenshots:

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

@rmathis This looks great. Thank you!
One small request: could you outline here the required steps one has to take to turn a CAS HTML page into one that is based on material-design? This would be useful for both developers who might add additional pages, etc and for those who might want to upgrade from existing previous deployments. Granted, most new pages would likely start off with an existing template file as a copy; the instructions are likely more important for those who'd want to migrate.
Nothing too fancy; just an outline and a basic structure should do fine and we can later add it to the documentation if necessary.


Upgrade to Keycloak 22

We would like to upgrade keycloak to the latest version :  22.0.0
Keycloak 22 offers

New bundle release with Quarkus which enhance loading time
Optimization on the security and realm settings.
Complete UI refactoring

Link to 05e616a2e1243a02adee4c86ee3820327bfe73d0

:tada: This issue has been resolved in version 1.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add label scale by distance

Adds a scaleByDistance property to Label and label entities. Fixes #1075, but we should still investigate signed distance fields.

This is spot on.
@mramato does KML have properties that should use this?

@mramato does KML have properties that should use this?

There are no configuration properties per-se, but I would definitely use this to make our KmlDataSource loader better match the view you get in Google Earth.


SDC-12269

modified sql query to avoid corner case per SDC-12269 ticket

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


Start of the "every-n-week" issue?

Describe the bug
I defined the following sensors in configuration.yaml but the Yard Waste which has a starting week defined is not working. What am I doing wrong or is this a bug?
Configuration
YAML or Config Flow (GUI):
########### GARBAGE SENSORS ##########
garbage_collection:
sensors:

name: Compost
frequency: "weekly"
collection_days: "fri"
verbose_state: true
name: Garbage
frequency: "every-n-weeks"
collection_days: "fri"
period: 2
first_week: 2
verbose_state: true
name: Recycling
frequency: "every-n-weeks"
collection_days: "fri"
period: 2
first_week: 1
verbose_state: true
name: "Yard Waste"
frequency: "every-n-weeks"
collection_days: "fri"
period: 2
first_week: 12
verbose_state: true

**Screenshots**

![image](https://user-images.githubusercontent.com/57807544/71785966-e5e7cc80-2fd3-11ea-90df-fb77828a6ad9.png)

The first_week parameter is not meant to "filter" the start of the collection period. That is the function of first_month and last_month. And you can combine it with exclude_dates if necessary.
Using it to set the start of the collection period would make no sense as it only applies to this specific frequency.
The way first_week works is, it is kind of offset (maybe this is what I should have called it). If you say every 2 weeks, with the first week of 1 it will be odd weeks, with 2 it will be even weeks. But the first week is not supposed to be greater than the period. I will add a check for that.
Please use the first_month to set the beginning of the collection period. Thanks.

@bruxy70 , thank you very much for the quick reply and i will incorporate this in a bit and report back. An thank you very much for the great work!

Ok the following worked. I needed the Yard Waste to start on March 20, 2019. The following give me that.
  - name: "Yard Waste"     frequency: "every-n-weeks"     collection_days: "fri"     period: 2     first_month: "mar"     first_week: 14     last_month: "dec"     verbose_state: true
However, your offset idea seems like a better implementation approach since this should be used in conjunction with start and end months.

Ok was able to get it to work with the following config. I needed the yard waste to start on Mar 20th. I like your offset idea. Seems more intuitive.

########### GARBAGE COLLECTION SENSORS ##########
garbage_collection:
sensors:

name: Compost
frequency: "weekly"
collection_days: "fri"
verbose_state: true
name: Garbage
frequency: "every-n-weeks"
collection_days: "fri"
period: 2
first_week: 2
verbose_state: true
name: Recycling
frequency: "every-n-weeks"
collection_days: "fri"
period: 2
first_week: 1
verbose_state: true
name: "Yard Waste"
frequency: "every-n-weeks"
collection_days: "fri"
period: 2
first_month: "mar"
first_week: 10
last_month: "dec"
exclude_dates:

'2020-03-06'
verbose_state: true


Bot Is offline and run.bot says Owner ID not there, but Ive had the owner id there

when i open run.bot it says my owner id isnt there but when i open the config and then options, its there

Nvm

Fixed it

@LuckyZZZZZ then please close the issue.


Small-step-tests are failing

The small steps tests are failing (even in master)
To reproduce cd to the small steps test directory and run the tests.  Here is a transcript.
[nix-shell:~/four/cardano-ledger-specs]$ cd semantics/small-steps-test/
[nix-shell:~/four/cardano-ledger-specs/semantics/small-steps-test]$ cabal test --test-show-details=streaming
Warning: Requested index-state2020-11-15T00:00:00Z is newer than
'hackage.haskell.org'! Falling back to older state (2020-11-14T23:50:07Z).
Resolving dependencies...
Build profile: -w ghc-8.10.2 -O1
In order, the following will be built (use -v for more details):

cardano-prelude-<IP_ADDRESS> (lib) (configuration changed)
goblins-<IP_ADDRESS> (lib) (configuration changed)
small-steps-test-<IP_ADDRESS> (test:doctests) (configuration changed)
cardano-binary-1.5.0 (lib) (configuration changed)
cardano-crypto-class-2.0.0 (lib) (configuration changed)
small-steps-<IP_ADDRESS> (lib) (configuration changed)
small-steps-test-<IP_ADDRESS> (lib) (configuration changed)
small-steps-test-<IP_ADDRESS> (test:examples) (configuration changed)
Configuring library for cardano-prelude-<IP_ADDRESS>..
Configuring library for goblins-<IP_ADDRESS>..
Configuring test suite 'doctests' for small-steps-test-<IP_ADDRESS>..
Preprocessing library for cardano-prelude-<IP_ADDRESS>..
Building library for cardano-prelude-<IP_ADDRESS>..
Preprocessing test suite 'doctests' for small-steps-test-<IP_ADDRESS>..
Building test suite 'doctests' for small-steps-test-<IP_ADDRESS>..
Preprocessing library for goblins-<IP_ADDRESS>..
Building library for goblins-<IP_ADDRESS>..
Running 1 test suites...
Test suite doctests: RUNNING...
doctests: : cannot satisfy -package-id prettyprinter-1.7.0-L9BUdCzYor7LBsek1iasxd
(use -v for more information)
Test suite doctests: FAIL
Test suite logged to:
/home/sheard/four/cardano-ledger-specs/dist-newstyle/build/x86_64-linux/ghc-8.10.2/small-steps-test-<IP_ADDRESS>/t/doctests/test/small-steps-test-<IP_ADDRESS>-doctests.log
0 of 1 test suites (0 of 1 test cases) passed.
cabal: Tests failed for test:doctests from small-steps-test-<IP_ADDRESS>.

seems fixed by the above


Compare Raw Data Files to Processed Data Check via Metadata

Currently, no checks are in place to compare that the number of files in raw-data makes it into processed data. Would create a more robust system.

Useful StackOverflow article. https://stackoverflow.com/questions/31124670/how-to-programmatically-count-the-number-of-files-in-an-archive-using-python

(ingest_raw_data.py) Write out a metadata file for each zip, only if everything finishes, inside processed folder (one meta for raw and one labels). Note on the workflow step in wiki this new output @CarolinaFurtado

RAK: "Currently we're only seeing if a corresponding directory exists in processed data.
one step beyond would be counting the files in raw (may need to unzip) and compare to number in processed.
2 steps beyond would be file name checks. then basically metadata output in processed the check (what was checked and results).
if this metadata exists in processed then skip the check.
basically, if have to unzip the raw each time to compare contents, then we def want to skip if it's been done before.
the metadata will have timestamp and we can always manually ensure that the processed folder hasnt been modified after the metadata check confirmation was created, if we're unsure whether somebody accidentally deleted something
the contents comparison is primarily to ensure nothing was half ingested.


Parallelize Cardano Transaction prover

Content
This PR parallelization of the Cardano transaction prover:

The proof computation by the Merkle map
The block ranges sub Merkle tree computation

Benchmarks

Total Requests
Concurrency
Transactions/Request
Pool (x5): Request/s
Pool (x5) + Parallel proof computation: Request/s

10000
500
1
7913.59
7623.6

10000
500
2
4851.96
4144.61

10000
500
3
3515.48
2779.29

10000
500
4
3101.26
2381.42

10000
500
5
2017.03
1658.07

10000
500
6
1832.84
1524.97

10000
500
7
1528.66
1276.15

10000
500
8
1398.67
1168.99

10000
500
9
1165.36
936.44

10000
500
10
1041.9
920.39

1000
100
20
512.33
440.4

1000
100
30
386.91
313.29

1000
100
40
314.65
254.52

1000
100
50
243.02
195.74

1000
100
60
202.51
163.77

1000
100
70
175.14
141

1000
100
80
149.35
124.07

1000
100
90
129.6
96.62

1000
100
100
115.64
102.17

Running on Linux / 8 cores / 64 GB RAM / 2 TB SSD

Pre-submit checklist

Branch

[x] Tests are provided (if possible)
[ ] Crates versions are updated (if relevant)
[ ] CHANGELOG file is updated (if relevant)
[x] Commit sequence broadly makes sense
[x] Key commits have useful messages

PR

[x] No clippy warnings in the CI
[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

Issue(s)
Closes #1756

As the parallelization work done in the prover does not improve performances, we will not merge this PR.


Koji import

Still in progress and needs non-unit testing.
Almost all the tests from koji_promote are still enabled. I did remove the tests that dealt with tagging for the most part.

tested using
https://osbs.qa.engineering.redhat.com:8443/console/project/osbs-stage/browse/builds/osbs-buildroot-docker-private-mlangsdorf-increm-b2a1b/osbs-buildroot-docker-private-mlangsdorf-increm-b2a1b-32?tab=details
to produce
https://osbs.qa.engineering.redhat.com:8443/console/project/osbs-stage/browse/builds/helloworld-container-master-9bc2a/helloworld-container-master-9bc2a-18
with logs
2017-07-13 12:09:20,523 - atomic_reactor.plugin - DEBUG - running plugin 'koji_import'
2017-07-13 12:09:20,524 - atomic_reactor.plugin - INFO - running plugin instance with args: '{u'kojihub': u'https://brewhub.stage.engineering.redhat.com/brewhub', u'url': u'https://osbs.qa.engineering.redhat.com:8443/', u'koji_keytab': False, u'koji_ssl_certs': u'/var/run/secrets/atomic-reactor/kojisecret', u'verify_ssl': False, u'use_auth': True}'
2017-07-13 12:09:20,525 - atomic_reactor.koji_util - INFO - Using SSL certificates for Koji authentication
2017-07-13 12:09:21,861 - atomic_reactor.util - DEBUG - Parent Config ENV: [u'container=docker', u'PKGM=yum', u'PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/sbin:/usr/sbin']
2017-07-13 12:09:25,893 - atomic_reactor.plugins.koji_import - DEBUG - Build information: {
"build_id": 1015736,
"completion_time": "2017-07-13 12:09:21",
"completion_ts":<PHONE_NUMBER>.0,
"creation_event_id": 13351746,
"creation_time": "2017-07-13 12:09:22.115564",
"creation_ts":<PHONE_NUMBER>.11556,
"epoch": null,
"extra": {
"image": {
"autorebuild": false
}
},
"id": 1015736,
"name": "chromium",
"nvr": "chromium-0.0.1-10",
"owner_id": 3436,
"owner_name": "osbs",
"package_id": 253,
"package_name": "chromium",
"release": "10",
"source": "git://git.engineering.redhat.com/users/mlangsdo/helloworld-container.git#67f8da9b0391499a0cd5de1e6f8a231bd0bef93a",
"start_time": "2017-07-13 12:05:16",
"start_ts":<PHONE_NUMBER>.0,
"state": 1,
"task_id": null,
"version": "0.0.1",
"volume_id": 0,
"volume_name": "DEFAULT"
}
2017-07-13 12:09:26,159 - atomic_reactor.plugin - DEBUG - plugin 'koji_import' finished in 5s

       results = [             {                 'id': 'x86_64-1',                 'container': {                     'arch': 'x86_64',                     'type': 'docker',                 }              },             {'id': 'ppc64le-1',                 'container': {                     'arch': 'ppc64le',                     'type': 'docker',                 }              },         ]
gets reordered between python2 and python3. is there any way to prevent this?

I think this all looks great, and the only reason I haven't approved it is the missing coverage. I think the readability could be improved for 'help' but it looks correct as-is.

Let's make sure to update the docs/plugins.md documentation and also the spec file (to package the plugin in the koji integration subpackage).

2017-07-17 20:11:47,430 - atomic_reactor.plugin - DEBUG - running plugin 'koji_import'
2017-07-17 20:11:47,431 - atomic_reactor.plugin - INFO - running plugin instance with args: '{u'kojihub': u'https://brewhub.stage.engineering.redhat.com/brewhub', u'url': u'https://osbs.qa.engineering.redhat.com:8443/', u'koji_keytab': False, u'koji_ssl_certs': u'/var/run/secrets/atomic-reactor/kojisecret', u'verify_ssl': False, u'use_auth': True}'
2017-07-17 20:11:47,433 - atomic_reactor.koji_util - INFO - Using SSL certificates for Koji authentication
2017-07-17 20:11:48,757 - atomic_reactor.util - DEBUG - Parent Config ENV: [u'container=docker', u'PKGM=yum', u'PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/sbin:/usr/sbin']
2017-07-17 20:11:52,035 - atomic_reactor.plugins.koji_import - DEBUG - Build information: {
    "build_id": 1015875,
    "completion_time": "2017-07-17 20:11:48",
    "completion_ts":<PHONE_NUMBER>.0,
    "creation_event_id": 13352421,
    "creation_time": "2017-07-17 20:11:49.010259",
    "creation_ts":<PHONE_NUMBER>.01026,
    "epoch": null,
    "extra": {
        "image": {
            "autorebuild": false,
            "help": "null"
        }
    },
    "id": 1015875,
    "name": "chromium",
    "nvr": "chromium-0.0.1-16",
    "owner_id": 3436,
    "owner_name": "osbs",
    "package_id": 253,
    "package_name": "chromium",
    "release": "16",
    "source": "git://git.engineering.redhat.com/users/mlangsdo/helloworld-container.git#67f8da9b0391499a0cd5de1e6f8a231bd0bef93a",
    "start_time": "2017-07-17 20:06:55",
    "start_ts":<PHONE_NUMBER>.0,
    "state": 1,
    "task_id": null,
    "version": "0.0.1",
    "volume_id": 0,
    "volume_name": "DEFAULT"
}
2017-07-17 20:11:52,196 - atomic_reactor.plugin - DEBUG - plugin 'koji_import' finished in 4s

Let's make sure to update the docs/plugins.md documentation and also the spec file (to package the plugin in the koji integration subpackage).

I'll take care of this as it also needs to be done for koji_parent plugin.

Release notes updated.
Created https://github.com/projectatomic/atomic-reactor/pull/738 to address packaging and documentation.


Support the SPV_EXT_fragment_shader_interlock extension.

This was straightforward to implement in GLSL. The
ShadingRateInterlockOrderedEXT and ShadingRateInterlockUnorderedEXT
modes aren't implemented yet, because we don't support
SPV_NV_shading_rate or SPV_EXT_fragment_invocation_density yet.
HLSL and MSL were more interesting. They don't support this directly,
but they do support marking resources as "rasterizer ordered," which
does roughly the same thing. So this implementation scans all accesses
inside the critical section and marks all storage resources found
therein as rasterizer ordered. They also don't support the fine-grained
controls on pixel- vs. sample-level interlock and disabling ordering
guarantees that GLSL and SPIR-V do, but that's OK. "Unordered" here
merely means the order is undefined; that it just so happens to be the
same as rasterizer order is immaterial. As for pixel- vs. sample-level
interlock, Vulkan explicitly states:

With sample shading enabled, [the PixelInterlockOrderedEXT and
PixelInterlockUnorderedEXT] execution modes are treated like
SampleInterlockOrderedEXT or SampleInterlockUnorderedEXT
respectively.

and:

If [the SampleInterlockOrderedEXT or SampleInterlockUnorderedEXT]
execution modes are used in single-sample mode they are treated like
PixelInterlockOrderedEXT or PixelInterlockUnorderedEXT
respectively.

So this will DTRT for MoltenVK and gfx-rs, at least.
MSL additionally supports multiple raster order groups; resources that
are not accessed together can be placed in different ROGs to allow them
to be synchronized separately. A more sophisticated analysis might be
able to place resources optimally, but that's outside the scope of this
change. For now, we assign all resources to group 0, which should do for
our purposes.
glslang doesn't support the RasterizerOrdered UAVs this
implementation produces for HLSL, so the test case needs fxc.exe.
It also insists on GLSL 4.50 for GL_ARB_fragment_shader_interlock,
even though the spec says it needs either 4.20 or
GL_ARB_shader_image_load_store; and it doesn't support the
GL_NV_fragment_shader_interlock extension at all. So I haven't been
able to test those code paths.
Fixes #1002.

@cdavis5e not sure how well you know the CFG part of SPIRV-Cross. I can probably continue on the PR and implement the analysis if you want.

@cdavis5e There is nothing particularly urgent to do right now, so picking this up tomorrow unless you've started on the rework already.

I haven't. I made the other change you asked for, and I've thought a bit about what I wanted to do, but if you have nothing better to do, be my guest.

Alright, thanks! Hopefully this will land tomorrow then.

Merged with the fixes on top. 🎉


[PromptLab] New placeholders, support for more browsers, bug fixes

Description
This update to PromptLab adds some new placeholders, improves the placeholder system overall, and fixes some bugs. Changes include:
General

Added a no-view image when no commands/models are installed.
Added support for additional browsers (e.g. Dev/Beta/Canary versions of Chrome, Edge, and Brave, Opera GX, etc.)
Added support for parsing Pages and Keynote documents.
Added support for parsing MS Office and Apple Numbers documents.
Added ability to modify action keybindings in the advanced settings
Added support for models that don't respond using the usual "data: ..." format (e.g. on-device Ollama/Orca).
Fixed bug where trying to obtain the transcript of a YouTube video would result in an invalid URL error (due to a slight change in the YouTube API). Resolves #7711.
Fixed a bug where commands that use file metadata would show a 'No File Found' error if Finder is inactive. Resolves #7898.

Placeholders

Added {{screenContent}} and {{windowContent}} placeholders for getting image vision data of the user's entire screen or the active window, respectively.
Added {{imageHorizon}} placeholder for accessing image horizon analysis data.
Added {{currentAppBundleID}} placeholder for getting the bundle ID of the currently active application.
Added {{elementText:...}}, {{elementHTML}}, and {{focusedElement}} for getting the text/HTML of HTML elements in the active tab of a browser.
Added support for target parameter to {{js:...}} placeholders.

Commands

Added 'Add Placeholder...' submenu when editing a command prompt.
When creating a new command, custom setup fields will now be unlocked by default.
Added horizon detection and corresponding command setting.
Fixed per-command model setting now saving.

Screencast
Summarize screen content:
SummarizeScreenContent-1080.webm
Summarize focused browser element, 'Add Placeholder' action:
SummarizeFocusedElement-1080.webm
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@pernielsentikaer I've switched all the old runAppleScript calls to the Raycast utils method, but I think a more thorough factoring that utilizes parseOutput will be part of the next update (as it already involves a lot of refactoring).
Also, I still use a custom runScript function in order to support callbacks on console logs during the script's execution, which it doesn't look like runAppleScript supports.

Thanks for the explanation, we might add more functionality to the utilities in the future 🙂


Deduplicate common uint methods

Builds on https://github.com/AleoHQ/snarkVM/pull/121, the last 2 commits are new.
This PR deduplicates some common unsigned integer code by introducing the following changes:

use a macro to implement UInt::{negate, rotr} for all unsigned integers
implement the Mul trait for all unsigned integers

Closing due to inactivity and conflicts, happy to revisit if still applicable.


Added Turkish language support

Description
I wanted to let you know that I've contributed to your repository by translating the README file into Turkish. This translation will help the Turkish-speaking community to benefit from your project.
The purpose of translating the README into Turkish is to make the project more accessible to a wider audience. Turkish-speaking users will be able to gain a better understanding of the project's goals and usage.
If you have any feedback or correction requests regarding the translation I've made, please feel free to let me know. I'd be more than happy to make further improvements.
Thank you for providing me with the opportunity to contribute to your project.
Related Issue
Reaching a Wider Audience: By presenting the project in Turkish, you make it more accessible to Turkish-speaking users. This can potentially lead to a broader understanding and use of your project.
Facilitating Local Usage: A Turkish README allows Turkish-speaking users to comprehend and use your project in their native language, making it easier for them to work with your project.
Overcoming Language Barriers: The English README might create language barriers for those who do not speak or have a limited understanding of English. A Turkish README can help overcome this barrier.
User Support and Feedback: Turkish-speaking users may have questions or feedback about your project. A Turkish README makes it easier for these users to communicate and engage with your project.
Contributing to the Turkish Community: Adding a Turkish README demonstrates your commitment to contributing to the Turkish open-source software community and allows you to engage with this community.
Screenshots (if appropriate):

Hey @ensarsnr thank you very much for your contribution,
there must be something wrong with your commit as I can't see any file changing in this PR.
Please let me know once it's fixed as I can't wait to have the library translated into Turkish :)
Teşekkür ederim, dostum


Why avoid optional all properties?

I'm using this feature and json model sent unnecessary properties.
@implementation ProductModel
+(BOOL)propertyIsOptional:(NSString*)propertyName
{
  return YES;
}
@end

here is shorted version my implementation code;
 @"ct" : @"Cities",
        @"tt" : @"Towns",
        @"wa" : @"WorkArea",
        @"kw" : @"Keyword",

and my interface file;
@property (nonatomic, copy) NSString *Cities;

//not in mobile
@property (nonatomic, copy) NSString *Towns;

@property (nonatomic, copy) NSString *WorkArea;

@property (nonatomic, strong) NSString *Keyword;

I just only set "Keyword" property and sent it to HTTP body. But its sent other properties with values like this;
{"ct":"keyword", "tt":"some value", "wa":"", "kw":""}

My question is; setting all properties are optional may caused this result?

Provide all the code please - not just small snippets. If the code is long, you can put it in a Gist.
If JSONModel is serializing a value into an empty string, it is not nil and therefore will be outputted regardless of whether the property is marked as optional.

@billinghamj , Here is interface file;
https://gist.github.com/serhatsezer/6672efa63bafcbe6f061
and implementation file;
https://gist.github.com/serhatsezer/919bbb039b4da482c901
I'm setting only keyword and city field like this;
KNJobListParameter *jobListParameterModel = [[KNJobListParameter alloc] init];
jobListParameterModel.Keyword = // getting from IB
jobListParameterModel.Cities      = // getting from IB
and my json sent it to back-end side like this;
https://gist.github.com/serhatsezer/f5830a121cd848657e54

Something must be instantiating the other properties - there is no other way for them to end up as empty strings.
It would be helpful if you provided a complete example including the actual serialization call.

OK. I will never use this framework again.

Oke 👍🏼

@billinghamj I'm really sorry for this. It's my mistake!

So is the issue resolved? Or do you still need assistance?
You need to keep in mind that the maintainers of open source projects are generally not remunerated in any way. There's no obligation to provide support at all, so it's important to be gracious and polite.

Hi, sorry for replying to this old topic, but setting all properties is ok?

If you want them to all be optional, sure.


Add Riddles

Hey guys, If you want to contribute to this project, make sure to go through CONTRIBUTING.md file🎯
After making an awesome riddle, give us a pull request and we will review it within a day and
merge it if there will be not any problem🙌
Happy hacking🚀

Can you please assign this to me.

Hey @bhavesh1129 anyone can create an Infinite number of Riddle in this project. That's why I have created only one issue. Everyone is assigned by default😊

Ok! Should I write code in Dart?

Yeah, It's a Flutter project, so you have to code in Dart.
Refer Getting Started part from the README.md and CONTRIBUTING.md for contributing to this project🙌

Hey @harshptl14 , I have added Math riddle 3 and given you a PR.
Could you merge it

Hey @harshptl14 , I have added Math riddle 3 and given you a PR.
Could you merge it

Merged🙌

Hey @harshptl14 , I too have added a Math riddle 8 and given you a PR. Please verify and merge it.

Hey @harshptl14, I have done all the changes suggested by you in my previous PR, Please, verify my updated PR.

I have sent a pull request for 11th math riddle.
Please do check and merge. Thank you!

Hi @harsh2201, I have commited riddle and opened a PR for the same. Could you please very and merge it?
Thanks.


No support for setDefaultHandler/setCatchHandler in sw-lib

Library Affected:
sw-lib
You can set a default handler that will be applied to all requests that aren't explicitly matched by other routes via Router.setDefaultHandler().
You can't do that when you use the higher-level sw-lib's wrapper on top of Router.
Ditto for Router.setDefaultHandler() not being exposed.

This is fixed now in master and will be in the next release.


 These buffers are manually allocated using #define meaning that the linker knows nothing about them. As the minispartan6+ needs the firmware to run from this ram (and we are likely to want other things in this ram too) we need to make the allocations visible to the linker.

https://github.com/timvideos/HDMI2USB-misoc-firmware/blob/master/firmware/lm32/pattern.c#L13


Dynamic Properties: UI Chart

I have a use case for this feature; malby I need to contzrol the min  and max value of the y axis of a bar diagram.


Add load and print family of utilities in GD

This PR adds new load, print, and pushError/pushWarning methods to the global scope GD class.
The load function works similarly to that of GD.Load in the C# variant and is a means of being consistent with the GDScript preload function.
Three new print methods are also available, which function similarly to Swift's traditional print function. These methods also handle converting the items accordingly to ensure they get represented properly, as well as converting the resulting string to the Variant type that the normal GD.print methods require.

Thank you for sending this patch!


Win7 MingW

Hi,
I have not the option using Visual Studio but I have the option to use the Win7 SDK or a MingW such as TDM-GCC.
Why is GCC supported on linux platforms but not on Windows?
Thanks

Actually, the main problem with Microsoft tools isn't that they are not free (or something), but that they are completely incompatible with all other C/C++ world. No autotools and no correct gnu-style makefile support means most legacy projects are out of scope and some others need very tough effort to build. And there is always "other way" of doing the same things. Yeah, gyp isn't bad inside, but if you would try to build something that relies on something else...
That would be better if mingw-w64 + msys is supported in some way. That will solve many many small and big problems. For example, I have no idea, why Chromium itself couldn't be built with gnu toolchain on Windows.

This is also terribly inconvenient to me. MSVC should not be a pre-req for gyp on windows...I have a working version of nodejs (and the v8 runtime) built with the gcc toolchain.
There isn't a good reason this isn't supported, other than the node-gyp people are lazy.

@smaudet First and only warning: insults get you banned.  Maintain a professional attitude from now on.


dotnet: november 2024 upgrades and infrastructure changes

Cc: @NixOS/dotnet
Fixes: #355500
Closes: #355642
The big breaking infrastructure changes are:

dotnet: split setup hooks into wrapper for runtime/sdk

This allows the setup hooks for dotnet to be changed without sdk/runtime rebuilds (VMR etc).

dotnet: move dotnet_root to $out/share/dotnet

This is consistent with packaging guidelines.

[x] write some release notes for these.

Things done

Built on platform(s)

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

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

[x] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.

However tests.writers.simple.fsharpNoNugetDeps might be related to this and the logs show no signs of why:
test test-writers-fsharp-no-nuget-deps.fsx failed

@GGG-KILLER none of my nixpkgs-reviews have built tests.writers.simple.fsharpNoNugetDeps.  I'm not sure why that is.  Do you have any idea?

@GGG-KILLER none of my nixpkgs-reviews have built tests.writers.simple.fsharpNoNugetDeps.  I'm not sure why that is.  Do you have any idea?

I don't know, I've looked at nixpkgs-review and there weren't any changes related to running tests either, I wonder if it somehow pulled things from ofborg instead of evaluating manually, but I'm really not sure

I mean it makes sense that it changed.  It's the dotnet-sdk change from 6 -> 8 where it broke.  I just wish I knew why my run didn't catch it.  :thinking:

tests.writers.simple.fsharpNoNugetDeps should be fixed now.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

aarch64-darwin

:fast_forward: 10 packages marked as broken and skipped:

alcom
avalonia
certdump
pablodraw
python312Packages.asteroid-filterbanks
python312Packages.asteroid-filterbanks.dist
python312Packages.pyannote-audio
python312Packages.pyannote-audio.dist
whisper-ctranslate2
whisper-ctranslate2.dist

:white_check_mark: 102 packages built:

ArchiSteamFarm
avalonia-ilspy
azure-functions-core-tools
bicep
bililiverecorder
blendfarm
boogie
btcpayserver
btcpayserver-altcoins
clps2c-compiler
csharp-ls
csharpier
csharprepl
cyclonedx-cli
dafny
dependency-track
depotdownloader
dotnet-aspnetcore
dotnet-aspnetcore_6
dotnet-aspnetcore_7
dotnet-outdated
dotnet-repl
dotnet-runtime
dotnet-runtime_6
dotnet-runtime_7
dotnet-sdk
dotnet-sdk_6
dotnet-sdk_7
dotnetCorePackages.aspnetcore_9_0
dotnetCorePackages.dotnet_8.aspnetcore
dotnetCorePackages.dotnet_8.aspnetcore.man
dotnetCorePackages.dotnet_8.runtime
dotnetCorePackages.dotnet_8.runtime.man
dotnetCorePackages.dotnet_8.sdk
dotnetCorePackages.dotnet_8.sdk.man
dotnetCorePackages.dotnet_8.vmr
dotnetCorePackages.dotnet_8.vmr.man
dotnetCorePackages.dotnet_9.aspnetcore
dotnetCorePackages.dotnet_9.aspnetcore.man
dotnetCorePackages.dotnet_9.runtime
dotnetCorePackages.dotnet_9.runtime.man
dotnetCorePackages.dotnet_9.sdk
dotnetCorePackages.dotnet_9.sdk.man
dotnetCorePackages.dotnet_9.vmr
dotnetCorePackages.dotnet_9.vmr.man
dotnetCorePackages.runtime_9_0
dotnetCorePackages.sdk_6_0_1xx
dotnetCorePackages.sdk_7_0_1xx
dotnetCorePackages.sdk_7_0_3xx
dotnetCorePackages.sdk_8_0_1xx
dotnetCorePackages.sdk_8_0_3xx
dotnetCorePackages.sdk_9_0
famistudio
fantomas
formula
fsautocomplete
garnet
gh-gei
git-credential-manager
github-runner
gitversion
ilspycmd
jackett
jellyfin
juniper
knossosnet
kryptor
libation
lubelogger
marksman
msbuild
nbxplorer
netcoredbg
omnisharp-roslyn
openutau
pre-commit
pre-commit.dist
prowlarr
python311Packages.asteroid-filterbanks
python311Packages.asteroid-filterbanks.dist
python311Packages.clr-loader
python311Packages.clr-loader.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pythonnet
python311Packages.pythonnet.dist
python312Packages.clr-loader
python312Packages.clr-loader.dist
python312Packages.pythonnet
python312Packages.pythonnet.dist
radarr
recyclarr
roslyn
roslyn-ls
seq-cli
smtp4dev
sonarr
torrentstream
upgrade-assistant
vrcadvert
wiseunpacker
yafc-ce

nixpkgs-review result
Generated using nixpkgs-review.

@GGG-KILLER do you have any logs for those failures? whisper-ctranslate2 for example, builds locally for me, and isn't failing on hydra.

This all seems to be a result of pre-commit failing:
       For full logs, run 'nix log /nix/store/6agas29d8hbn70166fnk2r5vwgrqz7qz-pre-commit-4.0.1.drv'.
error: 1 dependencies of derivation '/nix/store/plmm6rfjx0qhdwzw1dc8d0rd8gznny90-python3.11-asteroid-filterbanks-0.4.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/pmara3wrf3bgfg0yissqncr0a5q7b6mw-python3.12-asteroid-filterbanks-0.4.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/dgf6sdif34w6yy201nni6fk573jpkvdz-python3.11-pyannote-audio-3.3.1.drv' failed to build
error: 1 dependencies of derivation '/nix/store/7fhxjf0mz7qghkpp4r0ix8c1qkkc0x6i-python3.12-pyannote-audio-3.3.1.drv' failed to build
run-test-writers-fsharp-no-nuget-deps.fsx> building '/nix/store/bipm7b12fjkqavv4s2j41bz1pdjpvzwf-run-test-writers-fsharp-no-nuget-deps.fsx.drv'
error: 1 dependencies of derivation '/nix/store/c0mhq810dirds3d25ag5pz5kzjyd61a6-whisper-ctranslate2-0.4.6.drv' failed to build

I'll try again, didn't even notice it finished since I just left it running and left to do something else 😅

nixpkgs-review result
Generated using nixpkgs-review.

Yep, seems we indeed have a flaky test on pre-commit @corngood, maybe we should open an issue to notify maintainers or just leave this as a one-off?

Probably a good idea.  Do you have a log of the failure you could attach to it?

Yep, seems we indeed have a flaky test on pre-commit @corngood, maybe we should open an issue to notify maintainers or just leave this as a one-off?

Probably a good idea.  Do you have a log of the failure you could attach to it?

Might have deleted it, but nothing nix-collect-garbage -d and nix build .#post-commit in loop cannot solve.
I'll try to reproduce if I cannot find it and open the issue.

nixpkgs-review usually leaves log files in ~/.cache/nixpkgs-review.

Yeah, the thing is I usually delete that after I run it and everything passes it, but I can't remember if I did it or not. Will check though, thanks for reminding me 😅

Yeah, after 571 attempts I wasn't able to reproduce it. Might be something related to building multiple things in parallel as is usual with nixpkgs-review.
If I run into this again I'll report it, but for now I don't think I have more time to spend on trying to reproduce this.

Any ideas about this, which happens after pulling this in and attempting to upgrade to the net9 SDK? It only happens with a .config/dotnet-tools.json that contains Fantomas; deleting Fantomas but keeping FSharp.Analyzers is fine.
> nix build .#default.passthru.fetch-deps && ./result nix/deps.nix
fetching dependencies for WoofWare.Myriad.Plugins
Running phase: unpackPhase
unpacking source archive /nix/store/23ygy509rwqv4vz47av39xp9safqpvf6-abhkjibr7vgp6nzb907n90pj6qd265jp-source
source root is abhkjibr7vgp6nzb907n90pj6qd265jp-source
Running phase: patchPhase
Running phase: configureNuget
The template "NuGet Config" was created successfully.

Processing post-creation actions...

Running phase: updateAutotoolsGnuConfigScriptsPhase
Running phase: configurePhase
Executing dotnetConfigureHook
Unhandled exception: System.ArgumentException: '' is not a valid version string.
   at NuGet.Versioning.VersionRange.Parse(String value, Boolean allowFloating)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.DeserializeConvertable(JsonConverter converter, JsonReader reader, Type objectType, Object existingV
alue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.ResolvePropertyAndCreatorValues(JsonObjectContract contract, JsonProperty containerProperty, JsonRea
der reader, Type objectType)
[snip]
   at Microsoft.DotNet.Cli.ToolPackage.ToolPackageDownloader.DownloadAndExtractPackage(PackageId packageId, INuGetPackageDownloader nugetPackageDownloader, String packagesRootPath, NuGetVersion packageVersion, PackageSourceLocation packageSourceLocation, Boolean includeUnlisted)
   at Microsoft.DotNet.Cli.ToolPackage.ToolPackageDownloader.<>c__DisplayClass8_0.<InstallPackage>b__0()
   at Microsoft.DotNet.Cli.TransactionalAction.Run[T](Func`1 action, Action commit, Action rollback)
   at Microsoft.DotNet.Tools.Tool.Restore.ToolRestoreCommand.InstallPackages(ToolManifestPackage package, Nullable`1 configFile)
   at System.Linq.Enumerable.ArraySelectIterator`2.Fill(ReadOnlySpan`1 source, Span`1 destination, Func`2 func)
   at System.Linq.Enumerable.ArraySelectIterator`2.ToArray()
   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
   at Microsoft.DotNet.Tools.Tool.Restore.ToolRestoreCommand.Execute()
   at System.CommandLine.Invocation.InvocationPipeline.Invoke(ParseResult parseResult)
   at System.CommandLine.ParseResult.Invoke()
   at Microsoft.DotNet.Cli.Program.ProcessArgs(String[] args, TimeSpan startupTime, ITelemetry telemetryClient)

(This is from https://github.com/Smaug123/WoofWare.Myriad/pull/302 at commit 2712bf2.)


windows1.18.0  HTTPS  Received HTTP code 0 from proxy after CONNECT

It took me several days to compile windows nginx 1.18.0, but the operation reported an error. Please help me. If there is the same issue, please tell me, I can't find it. Thank you
patch -bp1 < proxy_connect_rewrite_1018.patch
auto/configure --with-cc=cl
--builddir=build
--prefix=
--conf-path=conf/nginx.conf --pid-path=logs/nginx.pid
--http-log-path=logs/access.log --error-log-path=logs/error.log
--sbin-path=nginx.exe
--http-client-body-temp-path=temp/client_body_temp
--http-proxy-temp-path=temp/proxy_temp
--http-fastcgi-temp-path=temp/fastcgi_temp
--with-cc-opt=-DFD_SETSIZE=1024
--with-pcre=src/build/lib/pcre
--with-zlib=src/build/lib/zlib
--with-openssl=src/build/lib/openssl
--with-select_module
--with-http_ssl_module
--with-http_sub_module
--add-module=src/build/lib/ngx_http_proxy_connect_module
nginx .conf

# main context
worker_processes auto;

events {
    worker_connections  1024;
}

http {
    #include       mime.types;
    default_type  application/octet-stream;
    sendfile        on;
    keepalive_timeout  65;

 server {
     listen                         3128;

     # dns resolver used by forward proxying
     resolver                       <IP_ADDRESS>;

     # forward proxy for CONNECT request
     proxy_connect;
     proxy_connect_allow            443 563;
     proxy_connect_connect_timeout  10s;
     proxy_connect_read_timeout     10s;
     proxy_connect_send_timeout     10s;

     # forward proxy for non-CONNECT request
     location / {
         proxy_pass http://$host;
         proxy_set_header Host $host;
     }
 }

}

Error

[root@master1 ~]# curl https://www.baidu.com -svo /dev/null -x <IP_ADDRESS>:3128
* About to connect() to proxy <IP_ADDRESS> port 3128 (#0)
*   Trying <IP_ADDRESS>...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 3128 (#0)
* Establish HTTP proxy tunnel to www.baidu.com:443
> CONNECT www.baidu.com:443 HTTP/1.1
> Host: www.baidu.com:443
> User-Agent: curl/7.29.0
> Proxy-Connection: Keep-Alive
>
* Recv failure: Connection reset by peer
* Received HTTP code 0 from proxy after CONNECT
* Connection #0 to host <IP_ADDRESS> left intact

please help me.
thank you !

hi @2777484478
This project is not tested on windows.
If you can enable debug log of nginx, we can try to debug  this request.
From the message from curl, we can know that ur connection to nginx is not established. It may be that ur nginx crashed or ur nginx did not start.
Recv failure: Connection reset by peer

Nginx didn't seem to crash and started normally. Because it seems normal to request HTTP, but HTTPS is not allowed

Cannot debug for current information you provided.
You can configured nginx with --with-debug, and config error_log to debug level. Then log the debug log from error.log with this failed request.
For how to enable debug log, see http://nginx.org/en/docs/debugging_log.html

This issue may be fixed in https://github.com/chobits/ngx_http_proxy_connect_module/pull/248. Note that only nginx-1.17.x ~ nginx-1.23.2 have been fixed. Their corresponding patch is proxy_connect_rewrite_1018.patch and proxy_connect_rewrite_102101.patch.


[Draft] Add tests

Closes #13.
Broadly, now that we have the neural network in PyTorch and Fortran, we need to add appropriate tests to ensure the various components/functions and behaving as we expect.
In this PR, we:

[ ] Add Fortran unit testing framework (PFUnit) and CMake/CTest
[ ] Add instructions for running tests to README.md.
[ ] Add tests for the neural network's forward pass in Fortran.
[ ] Add tests for the neural network in Pytorch.
[ ] Add tests to ensure reporducability between the Fortran and PyTorch models.

Ideally we want to check the fortran NN against the original python to ensure the Fortran NN is doing exactly what the python was trained to do.
At present we only have weights in a netcdf file used for populating the fortran.
Ideally we also want a pickle or pytorch file of trained weights so that we can compare/validate directly to the native trained data.
If we can obtain this data it would allow us to perform this check using an independent ground truth.

Idea of a simple test from meeting: if the NN predicts no change (to the field) then passing it back through the interface should also produce 0 effect (or very small) as a test of the interpolation.

Closing this as it is barely started, stale, and tests now being added as part of #44


Update scalaz-core to 7.2.30

Updates org.scalaz:scalaz-core from 7.2.9 to 7.2.30.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalaz", artifactId = "scalaz-core" } ]

labels: library-update, semver-patch

Codecov Report

Merging #304 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #304      +/-   ##
==========================================
+ Coverage   89.85%   89.88%   +0.03%
==========================================
  Files         133      133
  Lines        2888     2888
  Branches      132      132
==========================================
+ Hits         2595     2596       +1
+ Misses        293      292       -1

Impacted Files
Coverage Δ

...scala/hydra/kafka/ingestors/KafkaTopicsActor.scala
100% <0%> (+3.57%)
: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 f5e55bf...906ec10. Read the comment docs.


palmettopy.exceptions.EndpointDown. How can I access the endpoint please?

The endpoint is available again. Sorry for any inconveniences the down time may have caused.


'You found a resource!' popup

At the moment when I explore the map I collect resources, but there is no feedback to show that happens.  A audible 'collecting' noise and visual representation [+100wheat] would make exploration feel more engaging and let the player know they are collecting resources

Hi @LordSecretive , Can I work on this please?

It's already implemented, just need to enable again


Improve Logging

Currently the logging is rather unhelpful. Would be nice if it could be improved i.e. including the name of the screenshot being taken per step.

Yes, the logging def needs to be improved, will look into that in the next days ... if you have any other suggestions for helpful logs besides the file names feel free to shoot them through :blush:


Suggestion: scripts/webrecon.zsh

Problem:
I'm not sure how you work or how others work, but sometimes when I hit my automation scripts I'm not always in the correct folder to save the output. This has caused me the hassle of having to move folders around or if I'm not paying attention, drop test data randomly on my drive.
Proposed Solution:
So what I have done with some scripts I have is utilise the __ED variable I made within the engagement aliases (#12). This script could benefit from using this variable, or selecting the folder, or even passing a directory to store the output.
This would require a refresh to this script and updating how the tools save and to utilise this more dynamic approach to saving.
Example:
--[SNIP]--
    ############################################################
    # Robots
    ############################################################
    echo -e "${green} [+] Curling... robots.txt ${reset}"
    curl -s -L ${url}/robots.txt -o ${__ED}/tool-output/robots.txt
--[SNIP]--

yeah, that script needs work... definitely needs this added.


Better handling of optional assembly info attributes like AssemblyInformationalVersion

Awesome work 👍 great improvements overall. I'll publish a 1.6 version of the tool soon.
The upside of having all of this in one branch is that I only needed to merge this in order to close all your other PRs too.
Would you mind going over the issues that you created if there are any that are already resolved now?


helm: Generate experimental-install.yaml

Generate experimental-install.yaml in which we can enable experimental/beta
features without changing default values in the Helm chart. This makes
it easier for users to play around with bleeding edge features with a
simple kubectl apply.
Ref #11902
Signed-off-by: Michi Mutsuzaki<EMAIL_ADDRESS>

Coverage increased (+0.04%) to 36.961% when pulling 3aab70453263527d154e1ea115fa260f28bc6cdc on pr/michi/ultimate into 90217c1aa758ea54ffa3aed92b4ec337e35d8eb3 on master.

@michi-covalent I am not very clear on why it's 'ultimate', from the description it sounds like 'experimental' would be more appropriate... Could you explain the intention please?

i like experimental-install.yaml better. will rename

test-me-please


Add a Gitter chat badge to README.md

crossjs/plato now has a Chat Room on Gitter
@crossjs has just created a chat room. You can visit it here: https://gitter.im/platojs/Lobby.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

add badges manual


Why output picture so dark!!! I use the pretrained model. Need any other operation ???

I download the pretrained model and run 'test_Sony.py'.But the output is very dark!

I try to scale the 'output' to the same mean of 'gt_scale', then the problem is more serious! I do not kown what's wrong?

@cchen156 Could you please figure out what's wrong?

I'm also using this pretrained model. The results is interesting.
And you can close this issue if the problem is solved well.

@StudentZhangxu Have you found the 'checkpoint' and 'model.cpkt.index' files?


Vue DevTools: store.subscribeAction is not a function

Version
3.2.30
Reproduction link
None; see comments below
Steps to reproduce
New DevTools updated by Chrome
What is expected?
No errors in the console
What is actually happening?
This error appears when the page loads:
TypeError: store.subscribeAction is not a function
    at backend.js:4836:13
    at addPlugin (backend.js:2810:7)
    at async Object.<anonymous> (backend.js:2016:5)

I am using a custom $store reference (not Vuex) which does not have a subscribeAction.
I can add a no-op and it's fine, but the devtools code should not be making assumptions about what $store is in my app.

fir some reason github won't let me transfer it either, please re-open there.
We'll debug that issue-helper problem


Data repository presentation mobile-friendly

The text and font sizes are optimised for Desktop use.  Could device-detect be  used to restyle the presentation for small form factor devices such as phones and tablet?

It's supposed to be a responsive style sheet. that'll adapt to device characteristics. It comes from the college web team, it should work exactly as well as the college website does. Have to say, it looks just fine on my phone.


Replace authentication_(valid|invalid)? with (has|missing)_credentials?

authentication_valid? is badly named since it only means we had the userid, not
that the authentication was valid.
Calling code "may" have assumed it meant that the authentication was valid.
For example, #1691 shows how client code was duped into thinking authentication_valid?
meant more than has_credentials? and caused workers to continually restart when
the authentications couldn't be validated due to bad password, unreachable systems, etc.
has_credentials? or missing_credentials? is closer to the truth.
The resulting calling code seems to read much clearer now.
Followup to #1691
Rather than keep authentication_valid? and _invalid?, it's probably better to just remove all traces of the confusing method names.

What?  A net-deletion of only 1 line?  :disappointed:

cc @blomquisg @Fryguy @brandondunne @gmcculloug @lfu @roliveri @tenderlove @chessbyte @kbrock @lfu @bzwei  If you've ever used authentication_valid? or authentication_invalid? on our Ems/Hosts/FileDepots, please see above... I'm removing it because it's confusing.
Use this handy cheat sheet for when to use the remaining methods:
has_credentials?  - it has credentials
missing_credentials?  - it is missing credentials
authentication_status_ok? - the last authentication check against the remote system was "Valid"

Yes, only 1 net deletion... :cry:

:100: This whole thing reads much better than before!  Good job

@jrafanie I think we should fix the "wrong" usages of has_credentials in this PR, just in follow up commits, based on input from all the people mentioned above.  I'll comment on what I know.
Also, authentication_status_ok? depends on something coming along and actually verifying the credentials.  For EMSes, I believe this is scheduled.  However, is there something in place for other authentications, like, say, FileDepot?  Even so, I believe that when someone clicks verify in the UI (i.e. when verify_credentials is called), then we should set the authentication_status to ok at that time, since at least at that moment we are sure of it.

@jrafanie I comment on all the parts I recognized.  Additionally, I think all of the .connect methods and .verify_credentials methods are correct as is with the missing_credentials? check, and should not change to authentication_status_ok

Additionally, I think all of the .connect methods and .verify_credentials methods are correct as is with the missing_credentials? check, and should not change to authentication_status_ok

K, this makes sense.  But, we should be using authentication_status_ok in places where we're about to make connections, like before refreshes and scans.

where we're about to make connections, like before refreshes and scans.

Yes, as long as there is a mechanism outside of that code that checks the credentials on some regular basis.  Don't want to get into a situation where it temporarily goes bad, and then it can never connect again.
Incidentally, we probably should immediately mark it as "bad" if during the refresh or scan we detect a bad set of credentials.

Also, authentication_status_ok? depends on something coming along and actually verifying the credentials. For EMSes, I believe this is scheduled. However, is there something in place for other authentications, like, say, FileDepot? Even so, I believe that when someone clicks verify in the UI (i.e. when verify_credentials is called), then we should set the authentication_status to ok at that time, since at least at that moment we are sure of it.

So, I believe we only check if the authentication was changed.  If it was previously "bad" because the system was down and we click validate, it wouldn't change "status".
When you click validate in the UI, it could be using credentials that are not yet persisted since it's very likely someone could be entering a Ems/Host/etc for the first time and wants to validate the creds before saving.    We'd have to change the UI code to call Authentication#authentication_check(type) instead of verify_credentials if we have an authentication.

Yes, as long as there is a mechanism outside of that code that checks the credentials on some regular basis. Don't want to get into a situation where it temporarily goes bad, and then it can never connect again.

Yes, agreed.  Maybe the schedule of once per day is a bit paranoid.

Incidentally, we probably should immediately mark it as "bad" if during the refresh or scan we detect a bad set of credentials.

If we use authentication_check(type) instead of verify_credentials wherever the latter is used, it will mark the creds as bad and return true/false if it was successful.

So, @Fryguy @blomquisg I have applied all of the suggested changes.  I still have to clean up the last commit to add some tests but I think I got everything else.  Did I miss anything?  See the last 6 commits.

@miq-bot add_label needs changelog

Wow, @Fryguy @blomquisg @gmcculloug @kbrock, this is finally passing and I think I addressed all of your comments without chasing too many :sparkles: objects

Looks good to me! :+1: @blomquisg @gmcculloug If you are good with this, please merge.

@blomquisg @gmcculloug please review/merge :bow: :heart:

Nice job @jrafanie  :star:

:tada: Yay!!!

:cake: yay


Clicking a feature with no valid measure makes the timeseries widget buggy

Should be fixed with; https://github.com/kalisio/kMap/issues/127


Remove temporary cli options

Problem:
Temporary CLI options and config options added to s2nc and s2n_config. These options support short term testing, but should be removed before merging the test branch back into mainline.
Proposed Solution:
Look for this issue number in the codebase and remove the corresponding blocks.

This issue had to do with testing client support for HRR and is no longer relevant


Update project AboutUs page

Todos before merging:

[ ] Finalize project roles and responsibities

How to test

[ ] Ensure AboutUs page is completed
[ ] Individual links are correct (e.g. email, linkedin)
[ ] Individual project portfolio pages are up

Preview deployment: https://4a0eca08.tp-b32.pages.dev/
Followup to #7

Codecov Report

Merging #33 (0e2483f) into master (ce4f2cf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #33   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 ce4f2cf...0e2483f. Read the comment docs.


Remove unnecessary _.bind call that seems to contribute to memory/garbage

In analyzing memory usage in rabbot publishing, we found that the _.bind call (which has a lot of complexity inside recent versions of lodash), was contributing significantly to Node memory consumption (not as a leak, but as data in the old section of the heap that is described as not being reclaimed until there is memory pressure).

Fascinating...


Python code

Python translation, based on numpy.
Conversion works for single orientations ...
import numpy as np
import rotations         # need to be in the src/python directory

om = np.eye(3)
print(rotations.om2qu(om))

... and arrays
import numpy as np
import rotations         # need to be in the src/python directory

qu = np.array([np.array([1.,0,0,0])]*10)
print(rotations.qu2om(qu))

an object-oriented, ready to use class with related operations (IPF, rotation, sanity checks) is available in the python library of DAMASK

The code is quite hard to read because of vectorization. Still, the code is carefully checked. I have first translated the Fortran code 1to1 to python and used this as test cases for the vectorized version.
Vectorization is required because loops in python are extremely expensive. If have measured runtime for an IPF calculation making use of the routines. The vectorized version only needs 0.15% of the time compared to using naive loops for 262144 points.


Chaotic CSS

We should reorganize the existing CSS into a file structure that matches the component tree. We should also remove unused CSS.

Fixed in #46


Fix getting Azure instance OS type.

During our test we discovered SKU is only available if the instance was provisioned through a public image. For the one from a private image we can only get the OS type

@bronaghs and @djberg96 please review.

LGTM :+1:

Looks good to me, too.


Support for schedule list query parameter

What was changed

Support for schedule list query parameter
Checklist

Closes https://github.com/temporalio/cli/issues/616

How was this tested:

Added new tests

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

Comment for anyone perusing this: this PR currently needs a newer API version, but the released OSS doesn't yet have the newer API version. Discussions ongoing


connect() should use replaceState()

https://github.com/spoike/refluxjs/blob/master/src/connect.js#L17 should be changed to use replaceState() instead of setState().
If my store has data {a: 5, b: 10} and the data changes to {a: 5, c: 15} the state of the connected components will now be {a: 5, b: 10, c: 15}. I consider this a bug since I expected the connect() function to always sync the store data with the component state.

The function to create a connect mixin has the signature connect(listenable, key). If you specify a key, this.state[key] of the component will contain exactly the contents of the last trigger of your store.
By using replaceState, one could only listen to a single store. It might make some sense when connect is called without key, but I don't find the use-case convincing.

@confiks you are right about that. I only noticed that issue when I used a single store connection. As you can see in #243 I already changed my suggestion to only use replaceState() if no key is specified.

As of React 0.13.0 replaceState() is deprecated, at least from ES6 classes (more at https://github.com/facebook/react/issues/3236).  According to #243 Reflux's trajectory here is to deprecate keyless use of Reflux.connect(), which has an issue here #340.


Add Joseph Frank to Contributors list

@JoeyFrank, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


Unable to change correct disease

Not sure if this is front end or backend. But, I am unable to change the disease ID for this evidence statement:
https://civic.genome.wustl.edu/#/events/genes/25/summary/variants/57/summary/evidence/301/summary
The doid should be 3908 instead of 3909. When I edit evidence I see both "Disease" and "Disease ID" fields. I tried editing both the Disease ID to 3908 and the Disease to Non-small Cell Lung Carcinoma along with some other edits (pubmed and evidence level). The latter edits took but the disease ID revision did not appear. Now when I just trying changing the Disease and/or Disease ID I get a "409 - The attempted update conflicts with an existing record." error. Its like it is not recognizing this as a valid revision.

This is fixed in master!


.pypirc credentials get sent to redirect targets on third-party hosts.

My setup:

pypicloud, configured to redirect package links to signed S3 endpoints
.pypirc file, containing basic auth credentials to my pypicloud instance.

Credentials configured in .pypirc get added to package download requests in package_index.py:open_with_auth. My package server is configured to return a 302 with signed query parameters, pointing to a private s3 bucket. Because of the way setuptools configures its urlopen(…) call, the authorization header destined for my package server gets passed all the way to the signed s3 endpoint, which then responds with a 400, because there are now two authentication methods present.
It looks to me like a potential fix would involve either changing request.add_header("Authorization", …) to request.add_unredirected_header("Authorization", …) or a more extensive refactor making use of urllib.request.HTTPBasicAuthHandler.

this also happens with Artifactory and their "Direct Access" feature which does respond with 302 to S3.

@alkuzad @kurtiss Can you clarify what commands you are using where this is happening? I don't think that we have any maintained (i.e. not deprecated) code that would communicate with an external server and send any credentials.
Is this something that happens with setup.py upload? If so, I recommend updating your setuptools, because setup.py upload was removed several versions ago, you should be using twine to upload your packages to PyPI or any mirrors.

@pganssle this will happen in both python setup.py install or python setup.py test for us. Fetching packages can fail for all or only some packages, because Artifactory does not respond to everything with 302, only these packages that are not cached locally (and cache expires after some time).
I do believe that request.add_unredirected_header will fix that but as JFrog disabled this feature for our server, I can no longer test it there. Simple HTTP server with 302 redirect to S3 could test this too.

@pganssle I actually got local setup with :

Flask server which returns static html for /simple with one package only - pip-tools link to /packages/pip_tools-4.5.0-py2.py3-none-any.whl. It responses with 302 to my own S3 with presigned link

os.getenv controlled if we use request.add_unredirected_header or not

unredirected_header manages to fetch package
standard one returns 400
We can do some quick call on weekend if you want to see it

@pganssle example code is here - https://github.com/alkuzad/setuptools-fix

@alkuzad So I think this is a case where it's only affecting "unsupported" paths, because if I understand correctly using pip install should fix this (I'm a little surprised that python setup.py install is not already doing this, because I thought we replaced easy_install with pip), but the fix looks trivial, so I'm inclined to accept it.
The downside is that I'm about 85% confident that we will break existing peoples' workflow and there will be complaints if we do this, so I think we'll need to be sure that pip install actually works before we make such a change.

@pganssle what unsupported means ? Pip install indeed is working correctly, as it's using more advanced requests library.
Why is there difference with install and test if they both install dependencies ?
In both install.py and test.py I see only references to distutils commands or internal code using easy_install command. Pip is used in installer.py but I don't really know where it's methods are used. This is all using master code.

The use of any package index from setuptools is deprecated and unlikely to be fixed. If this issue is still affecting users, it's an indication that they're reaching an unsupported codepath. Use pip or build to install or build. Use tox or similar to orchestrate tests.


Add Wagtail 1.10 to the testsuite

With Wagtail 1.10 coming up we should add it to the testsuite as soon as released to ensure compatibility.

In version 1.10 wagtail will remove support for Python 3.3 and Django 1.9 and add support for django 1.11 and Python 3.6. We should update our tox configuration to test against the same versions.


config.enable = false no longer works after e6b743ccd4e780bc9cd85b707de67df72eea1a23

Description
Passing enable = false to the table passed to setup(...) no longer works after e6b743ccd4e780bc9cd85b707de67df72eea1a23. All works fine as-of 00fb2f7078ab4a5196d57a6c1c344c9e8489a68e.
Neovim version
N/A
Expected behavior
Context should not appear on startup if user configuration contains enable = false
Actual behavior
Context appears, regardless of configuration having enable = false
Minimal config
-- Shouldn't matter, will add if needed.

Steps to reproduce
Shouldn't matter, will add if needed.
Pass enable = false to configuration and observe the plugin is still enabled.

Haven't done anything beyond a bisect @lewis6991, will look at the commits as time permits to see if the issue is somewhat obvious or not.

Quick glance, I believe it is because plugin/treesitter-context.lua is loaded automatically per rtp stuff, so when setup() gets called with no arguments, it defaults to enabled = true.

Previously there was:
autocmd_for_group('treesitter_context')('VimEnter', function()
  M.setup()
end)

so if the plugin was lazy-loaded, this wouldn't do anything (as far as I can tell it doesn't check vim.v.vim_did_enter later as well). However with the newest changes (and due to the idempotent nature of checking the module-local did_setup), the user actually now cannot specify configuration beyond the defaults it seems.

To fix this we just need to move the did_setup check lower down.
Btw, I wouldn't recommend lazy loading this plugin. There really is no need.

Btw, I wouldn't recommend lazy loading this plugin. There really is no need.

But how else will I get my startup time blazing fast???
\s

They can if they call setup in their init.lua.

After moving/changing did_setup, right?

Ah lazy.nvim calls plugin/ during  init.lua (incorrectly IMO), so this setup() won't work with that plugin manager.


MS.EXO.6.2v1 does not check the Org calendar sharing settings and can produce a false Pass

🐛 Summary
This is specific to MS.EXO.6.2v1 "Calendar details SHALL NOT be shared with all domains". Based on testing and examination of the Rego, the coded policy check does not take into account the value of the Org wide calendar sharing setting which is described in the baseline implementation instructions. I was able to configure a non-compliant value for the Org wide setting and the tool still produced a Pass because it only examines the configuration from the individual domain sharing policies.
To reproduce
Steps to reproduce the behavior:

In Exchange admin center, ensure that all the policies in the individual domain calendar sharing settings are compliant (no calendar sharing with domain "*" or all domains)

Then go to Microsoft 365 admin center > Show All > Settings > Org Settings > Calendar

Make sure the "Let your users share their calendars with people outside of your organization who have Office 365 or Exchange" option is checked and click Save.

Run ScubaGear and you will see that 6.2 shows a Pass

Expected behavior
The code should take the Org wide setting into account when performing the policy check. If there is no Cmdlet to get the Org wide setting then we should discuss a course of action.

@adhilto @buidav Consulted with @schrolla on this. Here is our proposal. For now we plan to remove the implementation instructions referencing the org-level setting since those steps do not align with the intent of the policy which is to limit external sharing to specific approved domains. That said we have issue #469 which will explore both the org-level setting and the individual sharing setting using a hands-on use case testing approach. Once we test those settings out and figure out how they interplay with each other, it could trigger a future update to the baseline including a new policy or revisions to an existing one.
Let me know your thoughts and if I should close this issue?


Upload qc plotdata

Background
Link to issue
https://biomage.atlassian.net/browse/BIOMAGE-517
Link to staging deployment URL
Staging link in UI : https://github.com/biomage-ltd/ui/pull/147
Links to any Pull Requests related to this
https://github.com/biomage-ltd/ui/pull/147
Anything else the reviewers should know about the changes here

plotUuid are still hardcoded in the file. Later on, plotUuid should be emitted directly from the QC without hardcoded values in API.
Does not yet support saving plot configs for multiple samples, as the QC does not yet support multiple samples
Tested locally with inframock, with capability to upload plots-tables table (https://github.com/biomage-ltd/inframock/pull/27)

Changes
Code changes

Added call to DynamoDB in  pipeline response.

Definition of DONE
Your changes will be ready for merging after each of the steps below have been completed:
Testing

[ ] Unit tests written
[X] Tested locally with Inframock
[ ] Deployed to staging

To set up easy local testing with inframock, follow the instructions here: https://github.com/biomage-ltd/inframock
To deploy to the staging environment, follow the instructions here: https://github.com/biomage-ltd/biomage-utils
Documentation updates
Is all relevant documentation updated to reflect the proposed changes in this PR?

[ ] Relevant Github READMEs updated
[ ] Relevant wiki pages created/updated

Approvers

[ ] Approved by a member of the core engineering team
[ ] (UX changes) Approved by vickymorrison (this is her username, tag her if you need approval)

Just before merging:

[ ] After the PR is approved, the unstage script in here: https://github.com/biomage-ltd/biomage-utils is executed. This script cleans up your deployment to staging

Optional

[ ] Photo of a cute animal attached to this PR

LGTM after unit tests are finished.

@marcellp tests added


adds omershapira

Adds omershapira

Please take a look at https://github.com/outcoldman/hackernews-personal-blogs#contributing how to add your blog


Cannot run the example code

Hi, I was trying to run the example code but got an error for Android
* What went wrong:
Could not determine the dependencies of task ':app:preDebugBuild'.
> Could not resolve all task dependencies for configuration ':app:debugRuntimeClasspath'.
   > Could not resolve project :UnityExport.
     Required by:
         project :app > project :@asmadsen_react-native-unity-view
      > Unable to find a matching configuration of project :UnityExport:
          - None of the consumable configurations have attributes.
   > Could not find :unity-classes:.
     Searched in the following locations:
       - file:/Users/chathuri/Downloads/react-native-unity-view-master/example/android/UnityExport/libs/unity-classes.jar
     Required by:
         project :app > project :@asmadsen_react-native-unity-view

Any thoughts? In the ReadMe I can see that it says to copy Unity Export which I cant really seem to find in the example folder either. I'm stuck at this step, as I am not sure how to generate the UnityExport folder

Anyone who is facing this issue? I can get iOS to build (only in device mode) but didnt have a chance to test it out as I dont have an iphone with me.
So I am really trying to run the android project and struggling, as I cant find the UnityExport folder in the /android folder. I am unclear if the whole Build.cs script is running as the apk is created under the UnityProject/Build folder but now the UnityExport folder itself.
I am using Unity 3D 2019.4

you can export from unity , In the Top Menu you can find "ReactNative"  select android

you can export from unity , In the Top Menu you can find "ReactNative"  select android

bu I can't compile the Project also,

bu I can't compile the Project also,

@Superming997 Can you kindly guide about the steps which you have performed after exporting the project.
And how we can build and install using CLI.

@chathuri4 you might have made the same mistake I did.
I put all the root folders/files of my unity project inside /unity instead of my unity project folder.
I realized this when looking in the Build.cs at this line:
static readonly string ProjectPath = Path.GetFullPath(Path.Combine(Application.dataPath, ".."));
I hope it can help.

same issues


Error with crash log

Hello,
I published my app, but it had refused.
When I symbolicate the crash log, I have this line on the exception code :+1:
SMSegment.(title.materializeForSet).(closure #1) (in Honi) (SMSegment.swift:61)
I don't understand, maybe help me please before I post again my app.
Because all test on all device works with TestFlight, so I was surprised by Apple, they say to me "Crash on launch"
Thanks

@AlexLb,
Apologise for this very late reply.
I'm afraid that I couldn't reproduce the issue. Some people mentioned this may relate to protocols in the beta version of Xcode 7. I'm working on a new version of the framework, which deprecates the use of protocols. Hopefully this would solve the issue.


Image in the spine (automatically wrapped in HTML by web view) fails to load

https://readium.firebaseapp.com/?epub=https%3A%2F%2Fcdn.rawgit.com%2Freadium%2Freadium-test-files%2Fmaster%2Ffunctional%2Fsmoke-tests%2FSmokeTestFXL&goto={"idref"%3A"spine-images-001"%2C"elementCfi"%3A"%2F4%2F2%2F2%2C%2F1%3A0%2C%2F1%3A1"}
Step 1: force single page display
Step 2: go to next page (tiger should render with infinite loader)
Step 3: go to next page (orange cat should render with infinite loader)
Step 4: go to next page (blank page with infinite loader)
Step 5: open console, see:
TypeError: Type error
parseFromString — markup_parser.js:27
parseMarkup — markup_parser.js:27
resolveInternalPackageResources — content_document_fetcher.js:47
(anonymous function) — content_document_fetcher.js:40
success — plain_resource_fetcher.js:89

Chrome says:
Failed to execute 'parseFromString' on 'DOMParser': The provided value 'image/jpeg' is not a valid enum value of type SupportedType.


Foreign Keys with cascading deletes

Adding the ability to cascade deletes when creating a foreign key

With regards to failing build, it is due to a random blinking test pertaining to Marten.Testing.Linq.query_with_dates_Tests.can_select_DateTimeOffset_and_will_return_localtime

@mysticmind as discussed, I'll try to tackle that in separate PR, as this blinker is not related to this PR.

@oskardudycz Looks good 👍   Thanks again for finishing this up for me.


[Process Agent][POC] Add status command

What does this PR do?
This PR creates a status command that is meant to eventually deprecate the --info flag.

Motivation
This is part of the process agent stability improvement initiative. It will help us debug issues with customers as they crop up.

Additional Notes

Here is an example output on my machine:
==============================
Process Agent (6.0.0)
==============================

 Status date: Monday, 07-Feb-22 22:00:15 UTC
 Process Agent Start: 9829
 Pid: 52866
 Go Version: go1.16.12
 Python Version: n/a
 Build arch: amd64
 Log Level: debug
 Enabled Checks: [process rtprocess connections]

=========
Collector
=========

  Last collection time: 2022-02-07 22:00:10
  Docker socket: /var/run/docker.sock
  Number of processes: 38
  Number of containers: 0
  Process Queue length: 0
  RTProcess Queue length: 0
  Pod Queue length: 0
  Process Bytes enqueued: 0
  RTProcess Bytes enqueued: 0
  Pod Bytes enqueued: 0

If the process agent is not started the status command will print this:
=============
Process Agent
=============

  The Process Agent is not running

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

To test this command, simply install and run the agent. Then, in another window run process-agent status, and make sure that the output of the status command seems accurate.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The appropriate team/.. label has been applied, if known.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

Currently process_config.expvar_port does not exist in the core agent. At the moment, you need to set a value for process_config.expvar_port, otherwise this will not work. I will wait until after that is merged to bring this out of draft.


Fix validity period of derived SingleDenotations

When running:
val f: ( => Int) => Int = i => i ; f(1)

twice in the REPL, the second time crashed with a ClassCastException.
The difference is that in the second run, the denotation for f.apply is created from the SymDenotation for Function1#apply which already exists and is known to be valid in every phase, but that doesn't mean that the derived denotation for f.apply has the same validity: unlike the SymDenotation it needs to be transformed (in particular to run the ElimByName transformer).
It turns out that there were multiple places in the compiler where we created a new denotation with a validity based on the existing one when this was not legitimate. I've gone through all these places and replaced them by currentStablePeriod.
Fixes #18756.

test performance please

test performance please

test performance please: ff751ddbf7fd1d5638f6c66dce08fb6d48cdd8b9 49571faa98739d209f5c3c6ef88e9c7950c50242

Performance seems to be neutral or better.


Made TS6059 rootDir validation consistent for programmatic usage

When outDir, sourceRoot, and/or mapRoot are specified, the program preemptively called getCommonSourceDirectory, which reports the missing diagnostic if needed. This should still happen if rootDir is enabled without any of the other flags.
Fixes #41020.

For anyone finding their way here after encountering
error TS6059: File '[…]' is not under 'rootDir' '[…]'. 'rootDir' is expected to contain all source files.

when upgrading to TypeScript 4.3: yes, this PR is the breaking change.
You may think your project doesn't specify rootDir and thus shouldn't be affected. But if you use gulp-typescript, it may specify rootDir on your behalf: if rootDir is not specified, gulp-typescript (v6.0.0-alpha.1) tries to mimic TypeScript, which describes the default value as "the longest common path of all non-declaration input files".
gulp-typescript, however, may not use the correct set of "non-declaration input files" when deriving the rootDir. In that case, the solution is probably to explicitly specify rootDir in your tsconfig or Gulpfile. Note that this may affect the emitted directory structure, so other changes may be necessary as well.


WIP: CMake

Todo:

[x] Test on Windows/OS X
[x] slang-llvm binary fetching
[x] complete option handling
[x] document usage (also add presets as done by Falcor)
[ ] test integrating slang into a parent cmake project
[x] cross build (works at the moment if you can run the host code on the build machine, but this should be a staged build)

The main difference between this and the premake build is that all generated files go into the build directory, instead of being generated alongside the source. Also, build products are split appropriately between lib and bin.
It shouldn't step on the toes of the premake build, but I'm not sure about the other way around.
This also integrates slang-glslang and slang-llvm fully into this repo, just build them as normal targets. One can either SLANG_USE_SYSTEM_LLVM to use whatever llvm the OS has installed for us, or have CMake build an appropriate static LLVM for us (suitable for creating a redistributable slang-llvm). I'm planning for the downloaded binary version to be taken from the most recent release in this repo (according to git describe --abbrev=0 --tags --match v*).
Closes https://github.com/shader-slang/slang/issues/3283

Can we move all the test, example and gfx targets behind some optional flag in cmake, so it is possible to include slang in another project without having to deal with all these targets?
Additional maybe it would be nice to detect if a cmake project already included glslang (or other deps) and use these includes instead of slang complaining about double includes (not sure if this could lead to version mismatch problems)


eigenvalues fixed

what python returns as S in SVD method is the square root of the eigenvalues and not the eigenvalues.

I updated the unit tests.


Does httpbeat support dynamic reloading of configuration files?

We use httpbeat to query data from multiple servers. It would be useful to have a feature where the http queries can be added to the httpbeat.yml and Http beat dynamically reloads the new configuration without requiring a restart. All official Elastic beats support this feature.

There is the http module in metricbeat which is a contribution from @christiangalsterer . There you could use the reloading feature: https://www.elastic.co/guide/en/beats/metricbeat/6.0/metricbeat-module-http.html The module is available in 6.0.0-beta1.


[zh] Localize feature-gates/n*.md

Related to issue #44410. Localize these files:

namespace-default-label-name.md
network-policy-end-port.md
network-policy-status.md
new-volume-manager-reconstruction.md
nftables-proxy-mode.md
node-disruption-exclusion.md
node-inclusion-policy-in-pod-topology-spread.md
node-lease.md
node-log-query.md
node-out-of-service-volume-detach.md
node-swap.md
non-preempting-priority.md

/approve


[BUG] Editing tags in the fileclass table breaks the tags field when tags are added

I've got a note with the tags venue, journal, robotics listed in the fileClass Table View:

If I edit this, it looks OK:

I can edit it, add a tag, and it looks fine:

It even shows up in the Obsidian frontmatter OK:

However, if I've edited it directly in the note, so that it uses Obsidians new frontMatter handling, it now shows up in fileClass Table View with each tag on a separate line:

When I edit this (adding another-test), it goes a bit crazy:

I'd guess this is because the tags field is set to be an Input, which doesn't take multiple values. But none of the other fields that take multiple values seem suitable, unless they can accept any tag from the vault.

I get a similar type issue (with #-s popping up and duplicate tags) when tags is defined as a formula field.
Would love to see this fixed, though I'm not sure what would be the best solution here.
My guess is that making the formula output a multiline list (like the obsidian frontmatter editor expects) would fix it, since currently the tags look something like:
tags: tag1 tag2
  - tag1
  - tag2

As if the formula doesn't understand that the list is already there.

I just solved my problem by patching some code!
It seems that when I add FieldType.Formula to the condition in removeIndentedListItems() in src\note\lineNode.ts the problem doesn't occur anymore, this function contains exactly the logic that is needed to remove the indented list.
I also added FieldType.Input to the condition (as part of an unrelated effort to allow multiline values) and it works, so I guess it could work with tags too.
I then encountered a problem with some object list fields disappearing when fields above them are edited, but it seems that changing the indentLevel in the previously-mentioned function from this.field.getAncestors().length to this.field.getIndentationLevel(this) (a function which, if I understood correctly, didn't exist at the time the line was written) fixes the problem!
By the way, I haven't run any tests, I'm just playing around and messing with code, so maybe I've introduced some bugs that I will only notice later 🙈