[LSQ12dlf] duplicate photometry points

The V and r photometry seem to be the same datapoints at an offset. In fact there seem to be 2 copies of the V band data, and of many other bands B, I...  and the r data looks like it is a copy of the V data at an offset.

also this is a SLSN, not a Ic i think

Nice finds! I'm working on fixing the typing for this one. @guillochon may have better insights about the duplicate photometry points.

i just downloaded a bunch of stripped lcvs, i may have more findings! one
suggestion: it would be nice to be able to isolate a photometric band, it
should be possible w bokeh (with the 'tap' tool) although i did not find
any examples of multiple points selection w tap
f
On Sat, Jul 9, 2016 at 9:05 PM, J. Parrent<EMAIL_ADDRESS>wrote:

Nice finds! I'm working on fixing the typing for this one. @guillochon
https://github.com/guillochon may have better insights about the
duplicate photometry points.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/astrocatalogs/astrocats/issues/50#issuecomment-231564600,
or mute the thread
https://github.com/notifications/unsubscribe/ABnkhsMEYMUgm7bnill4RKKHqEsrUF8pks5qUEVYgaJpZM4JIuxQ
.

--
/|/
|*  Please consider the environment before printing this e-mail

This issue was moved to astrocatalogs/supernovae#10

@fedhere I'm moving all these issues to the supernova issue page specifically; the link on the main page wasn't updated to point to the new issue tracker. All three issues are now living here: https://github.com/astrocatalogs/supernovae/issues


Fix iOS warnings in compose-multiplatform template

https://github.com/JetBrains/compose-multiplatform-template (69400e7e863c6f32ee54df81a4634dcbd52f542f)

Open the template in Android Studio on macOS

See warnings:

We should fix them.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Update Schema module and Collection module

#1306
Checklist

[x] the [contributor license agreement][] is signed
[x] commit message follows [commit guidelines][]
[ ] tests are included
[x] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change
As the resumption token has been fixed, this PR attempts to update Schema module and Collection module to use resumption tokens to retrieve data.
Created a new function to list entities that extend BaseEntity, and then updated the two modules to use this function. Also updated mock data and renamed a field of CommonParams.

Thanks @PenghaiZhang
A few bits:

I'm not sure if we need to local functions of getCollections and getSchemas, but I don't mind.
I'm surprised the tests are still fine.... Or is that because there's no resumption token in the mocked data...
Overall though, we need to replace that recursive call - if we have a thousand or more results our stack will be at risk.

Yeah the mocked date do not have resumption tokens, but we should be able to add a mocked implementation to add resumption tokens back.


(reverted) should be more obvious in changes browser

In the following image, the first entry has been reverted, but since the Selector field is clipped you can't tell difference between a reverted method and a method that just has a long selector ...

Yes, this is a problem - looks exactly the same.

@LisaAlmarode @dalehenrich Is it even helpful to retain the reverted item in the list? One way to fix this is to remove it. Thoughts?

Offhand, I think you need a record of what you just did since it could be a big deal if you revert the wrong thing.  Also revert needs an undo/un-revert option since after revert all changes are apparently lost.  Also while changes in the changes browser apply to the project browser immediately, if you re-make a change in the project browser the changes browser does not get updated.

Also revert needs an undo/un-revert option since after revert all changes are apparently lost. Also while changes in the changes browser apply to the project browser immediately, if you re-make a change in the project browser the changes browser does not get updated.

Because of this outstanding feature request, I'm reluctant to make too many changes to this Changes Browser. I would invite a feature list be added to issue #123, though, if RwModifications are ready.

Offhand, I think you need a record of what you just did since it could be a big deal if you revert the wrong thing
Good point.  Given the limitations on this browser, I'll try to make the reverted indicator more visible.


BUG: CLI preview events returns divided by 0

Create preview from compose file

File: Click to expand
services:
  webhook:
    image: uffizzitest/webhooks-test-app:latest
    environment:
      POSTGRES_USER: postgres
      POSTGRES_PASSWORD: postgres
      TEST: test1

x-uffizzi-ingress:
  service: webhook
  port: 80

x-uffizzi-continuous-preview:
  share_to_github: true
  delete_preview_after: 99h
  tag_pattern: uffizzi_request_*
  delete_preview_when_image_tag_is_updated: true
  deploy_preview_when_image_tag_is_created: true
  delete_preview_when_pull_request_is_closed: true
  deploy_preview_when_pull_request_is_opened: true

Run uffizzi preview events DEPLOYMENT-ID

Result: Error divided by 0
Error: Click to expand
~ # uffizzi preview events deployment-7
/usr/local/bundle/gems/thor-1.2.1/lib/thor/shell/basic.rb:168:in `%': divided by 0 (ZeroDivisionError)
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/shell/basic.rb:168:in `block in print_in_columns'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/shell/basic.rb:166:in `each'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/shell/basic.rb:166:in `each_with_index'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/shell/basic.rb:166:in `print_in_columns'
	from /usr/local/bundle/gems/uffizzi-cli-0.3.8/lib/uffizzi/shell.rb:21:in `print_in_columns'
	from /usr/local/bundle/gems/uffizzi-cli-0.3.8/lib/uffizzi/cli/preview.rb:116:in `handle_succeed_events_response'
	from /usr/local/bundle/gems/uffizzi-cli-0.3.8/lib/uffizzi/cli/preview.rb:109:in `handle_events_command'
	from /usr/local/bundle/gems/uffizzi-cli-0.3.8/lib/uffizzi/cli/preview.rb:76:in `run'
	from /usr/local/bundle/gems/uffizzi-cli-0.3.8/lib/uffizzi/cli/preview.rb:55:in `events'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/command.rb:27:in `run'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/invocation.rb:127:in `invoke_command'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor.rb:392:in `dispatch'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/invocation.rb:116:in `invoke'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor.rb:243:in `block in subcommand'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/command.rb:27:in `run'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/invocation.rb:127:in `invoke_command'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor.rb:392:in `dispatch'
	from /usr/local/bundle/gems/thor-1.2.1/lib/thor/base.rb:485:in `start'
	from /usr/local/bundle/gems/uffizzi-cli-0.3.8/exe/uffizzi:5:in `<top (required)>'
	from /usr/local/bundle/bin/uffizzi:25:in `load'
	from /usr/local/bundle/bin/uffizzi:25:in `<main>'

Events for preview was returned by API /api/cli/v1/projects/{project_slug}/deployments/{deployment_id}/events
From API: Click to expand
![image](/uploads/e6718937f60e179efcb02ffda41935a0/image.png)
{
    "events": [
        {
            "first_timestamp": null,
            "last_timestamp": null,
            "reason": "Scheduled",
            "message": "Successfully assigned deployment-7/app-deployment-7-6fcf7dd489-jscj5 to gke-uffizzi-client-sandbox-510ebd16-vmes"
        },
        {
            "first_timestamp": "2022-03-29T14:44:04Z",
            "last_timestamp": "2022-03-29T14:44:04Z",
            "reason": "Pulling",
            "message": "Pulling image \"uffizzitest/webhooks-test-app:latest\""
        },
        {
            "first_timestamp": "2022-03-29T14:44:05Z",
            "last_timestamp": "2022-03-29T14:44:05Z",
            "reason": "Pulled",
            "message": "Successfully pulled image \"uffizzitest/webhooks-test-app:latest\" in 455.548757ms"
        },
        {
            "first_timestamp": "2022-03-29T14:44:05Z",
            "last_timestamp": "2022-03-29T14:44:05Z",
            "reason": "Created",
            "message": "Created container a74ced40d3"
        },
        {
            "first_timestamp": "2022-03-29T14:44:06Z",
            "last_timestamp": "2022-03-29T14:44:06Z",
            "reason": "Started",
            "message": "Started container a74ced40d3"
        },
        {
            "first_timestamp": null,
            "last_timestamp": null,
            "reason": "Scheduled",
            "message": "Successfully assigned deployment-7/cm-acme-http-solver-mbkx2 to gke-uffizzi-client-default-pool-34f79102-s0ri"
        },
        {
            "first_timestamp": "2022-03-29T14:43:53Z",
            "last_timestamp": "2022-03-29T14:43:53Z",
            "reason": "Pulled",
            "message": "Container image \"quay.io/jetstack/cert-manager-acmesolver:v1.6.1\" already present on machine"
        },
        {
            "first_timestamp": "2022-03-29T14:43:54Z",
            "last_timestamp": "2022-03-29T14:43:54Z",
            "reason": "Created",
            "message": "Created container acmesolver"
        },
        {
            "first_timestamp": "2022-03-29T14:43:54Z",
            "last_timestamp": "2022-03-29T14:43:54Z",
            "reason": "Started",
            "message": "Started container acmesolver"
        },
        {
            "first_timestamp": "2022-03-29T14:44:13Z",
            "last_timestamp": "2022-03-29T14:44:13Z",
            "reason": "Killing",
            "message": "Stopping container acmesolver"
        }
    ]
}

The output is displayed in pretty print format

The issue was opened in the UffizziCli project
https://github.com/UffizziCloud/uffizzi_cli/issues/49


Cannot build Foundry from source on Windows 10 - failed to update submodule testdata/lib/forge-std

Component
Other (please describe)
Have you ensured that all of these are up to date?

[ ] Foundry
[ ] Foundryup

What version of Foundry are you on?
No response
What command(s) is the bug in?
cargo install --git https://github.com/foundry-rs/foundry foundry-cli anvil --bins --locked
Operating System
Windows
Describe the bug
Running the  command to build Foundry from source results in the 'no URL configured for submodule' error
input:
cargo install --git https://github.com/foundry-rs/foundry foundry-cli anvil --bins --locked
output:
(dev) PS C:\Users\Skelectric> cargo install --git https://github.com/foundry-rs/foundry foundry-cli anvil --bins --locked
    Updating git repository `https://github.com/foundry-rs/foundry`
error: failed to update submodule `testdata/lib/forge-std`

Caused by:
  no URL configured for submodule 'testdata/lib/forge-std'; class=Submodule (17)
    Updating git repository `https://github.com/foundry-rs/foundry`
error: failed to update submodule `testdata/lib/forge-std`

Caused by:
  no URL configured for submodule 'testdata/lib/forge-std'; class=Submodule (17)
     Summary Failed to install foundry-cli, anvil (see error(s) above).
error: some crates failed to install

Closed by #3774


Fix ServiceTemplate#picture= with models

When creating a service template with a Picture that already exists (done in some of specs in other repos), the previous form of this method would break as create_picture was not relevant for updating pictures that already exist.
This fix #should™ fix that, but more relevant eyes should probably take a look just to make sure...
Where it fails...
Currently fails in the manageiq-api specs:

https://github.com/ManageIQ/manageiq-api/blob/f12cf95d/spec/requests/service_templates_spec.rb#L16-L22
https://github.com/ManageIQ/manageiq-api/blob/f12cf95d/spec/requests/pictures_spec.rb#L26-L36

Links

Introduced in:  https://github.com/ManageIQ/manageiq/pull/18689

Steps for Testing/QA
Try running the API specs with and without this change to the models and confirm that it fails without, and passes with.

Also fixes UI specs:
  1) CatalogController tests that needs all rbac features access #st_upload_image uploads a selected png file
     Failure/Error: expect(assigns(:flash_array).first[:message]).to include('Custom Image file "upload_image.png" successfully uploaded')

     NoMethodError:
       undefined method `first' for nil:NilClass
     # ./spec/controllers/catalog_controller_spec.rb:287:in `block (4 levels) in <top (required)>'

Finished in 0.30847 seconds (files took 7.25 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/controllers/catalog_controller_spec.rb:284

Thanks!

@miq-bot add_label hammer/yes
@simaishi this fixes travis failures coming from https://github.com/ManageIQ/manageiq/pull/18689
So we may want to backport ASAP, to get hammer travis green on ui and api.
(triggered ui hammer travis just to be sure it really fails now: https://travis-ci.org/ManageIQ/manageiq-ui-classic/builds/526392856)

Hammer backport details:
$ git log -1
commit 40a1d86809762479e79f0f654a28b1b8507f4223
Author: Martin Povolny<EMAIL_ADDRESS>Date:   Tue Apr 30 13:31:37 2019 +0200

    Merge pull request #18705 from NickLaMuro/fix_service_template_picture_equals

    Fix ServiceTemplate#picture= with models

    (cherry picked from commit f7fc0cbfff944f8a39466d77f8c5b1f90551c542)

    https://bugzilla.redhat.com/show_bug.cgi?id=1702479

@miq-bot remove_label hammer/yes
(forgot editing comments with miq-bot commands in them leads to this :))

It's not super when super looks like return but can't be used in guard clauses.  Thanks @NickLaMuro!


No charts add-repo command available

Version
0.13.0
Problem
Official docs states that I can add chart repository into the chartfile.yaml
# Add the official Grafana repository
$ tk tool charts add-repo grafana https://grafana.github.io/helm-charts

Well... That's in contradiction to what I have experienced using Tanka:
$ tk tool charts
Declarative vendoring of Helm Charts

Usage:
  tk tool charts [command]

Available Commands:
  init        Create a new Chartfile
  add         Adds Charts to the chartfile
  vendor      Download Charts to a local folder
  config      Displays the current manifest

Flags:
  -h, --help   help for charts

Use "tk tool charts [command] --help" for more information about a command.

If I try to run the command anyways I get the error:
$ Error: unknown subcommand `add-repo`

Hey, good spot. The website is built from current master, and the commit that adds this functionality has not yet been included in any release.
Please wait for the next release, or built it yourself from master :)

Hey, good spot. The website is built from current master, and the commit that adds this functionality has not yet been included in any release.
Please wait for the next release, or built it yourself from master :)

Oh... But wait... Isn't that a matter of pointing to the right tag to build the docs? :smile:

Oh... But wait... Isn't that a matter of pointing to the right tag to build the docs? :smile:

Agreed, this leads to slight inconsistencies, but these don't appear frequent enough to justify the extra complexity it imposes on our build pipeline. If this happens a lot in the future, we might reconsider it :)
Closing as 0.14 is released, so this feature is generally available


Location search

Created a search bar on the map that lets you search for different locations in Italy.

Great job implementing this feature! It will be helpful to find locations quickly


feature request: import google location history

Personal location history can be downloaded from here

Hey, we're working on extracting the last 14 days.
Google doesn't provide an API for this, but we found a trick to extract those locations using a WebView. Stay tuned!

Hi @MiriShalev,
I've also investigated extracting this data using webview with script injection, feel free to look here:
https://github.com/Yshayy/hamagen-react-native/blob/master/src/components/Main/ImportData.tsx
The webview extraction worked and all the data is passed to react-native, I'm in the middle of integrating it to other parts of the app (a bit wrapped in the details of adding historic data).


Add MWM Dust Dist program.

This adds the MWM Dust dist program.  Thisi s the same as the normal MWM dust program, changing only names and the dependence on mwm_galactic_core to mwm_galactic_core_dist.

@jcbird , any interest in reviewing this to make sure I didn't mess up badly?  Here's the relevant diff:
$ diff python/target_selection/cartons/mwm_dust_dist.py python/target_selection/cartons/mwm_dust.py
6c6
< # @Filename: mwm_dust_dist.py
---
> # @Filename: mwm_dust.py
112,116c112,113
< class MWM_Dust_Dist_Carton(BaseCarton):
<     """MWM Dust Dist Carton.
<
<     This is identical to the MWM_Dust_Carton, but depending on the
<     mwm_galactic_core_dist rather than mwm_galactic_core carton.
---
> class MWM_Dust_Carton(BaseCarton):
>     """MWM Dust Carton.
149c146
<     name = 'mwm_dust_core_dist'
---
>     name = 'mwm_dust_core'
152c149
<     program = 'mwm_dust_dist'
---
>     program = 'mwm_dust'
162c159
<                      .where(targetdb.Carton.carton == 'mwm_galactic_core_dist',
---
>                      .where(targetdb.Carton.carton == 'mwm_galactic_core',

This looks good to me. It is ultimately up to @albireox  and @astronomygupta if they are using the derived class name as a unique ID.

This looks initially fine to me. Has the mwm_galactic_core_dist already been implemented and run? We should probably wait until that's done and we can test the _dist code before merging.
The mwm_dust_core carton uses DR3 and removes the TIC, but this one keeps it. Is that something we should change?

I'll let @schlafly confirm, but we should absolutely get rid of TIC and replace with Gaia DR3. Thanks.

Thanks for paying attention.  I did not intend to make any changes wrt TIC.  Honestly, I don't know what TIC is and I would definitely choose Gaia DR3.  Thank you!

This looks initially fine to me. Has the mwm_galactic_core_dist already been implemented and run? We should probably wait until that's done and we can test the _dist code before merging.

Sorry, I don't know the status here.  The goal on my end is to make sure that the dust carton and GG carton are kept in sync; dust_core_dist with galactic_core_dist, dust_core with galactic_core.  Thanks!

Sounds good, I think with your changes we can take care of updating the other bits to using Gaia DR3 the same way we've been doing everywhere else. I'll get to this next week.

It took a while but I implemented both mwm_dust_core and mwm_dust_core_dist. I'm going to close this PR but you can check the updated code in #277. I'll send an email with the location of the test files.


MB-2968 Create tool to generate fake data [MERGE AFTER 2712, 2713]

Description
This PR adds a tool to parse the API files for the Prime and Support APIs using Prance that figures out which data type we want for each field, and then it connects with Faker to generate fake data for the type we selected. It has been implemented with the Prime API tasks create_mto_shipment and create_mto_service_item.
Reviewer Notes
A good order to view these changes in so that you can see the flow of the code would be:
utils/fake_data.py > utils/fields.py > utils/parsers.py > locustfiles/prime.py > tasks/base.py > tasks/prime.py
All the other changes are fairly small and can be mixed in whenever it seems appropriate.
The Support API still has some issues with its configuration -- mostly inconsistent formatting such as using snake-case for keys when we need to be using camel case. I'll have add a ticket to address this issues so this tool can be successfully implemented with the Support API.
Also, this functionality will need to be amended to account for exceptions in new endpoints, and it could use tests to ensure its stability as it gets updated. I will be adding a ticket to add unit tests for our custom tooling in this repo (MB-3237), and we will add additional tickets on a case-by-case basis to update this handling for additional endpoints.
Setup
In the mymove project directory:
make db_dev_e2e_populate && make server_run

In the milmove_load_testing project directory:
make teardown && make setup
make load_test_prime

Enter any number of users and a hatch rate of your choice (e.g. 5 and 1/sec). You should see no exceptions and mostly 200 response codes. The create_mto_service_item endpoint might return some 422 responses due to some custom field validation -- this is acceptable. Code to account for this can be added on a case-by-case basis.
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change.

Also looks like there's some conflicts here that you might need to resolve after merging in #17 and #18

One question that I do have is how much this code will have to change. I noticed in utils/parsers.py there is some custom code around /mto-service-items. Will this be something that needs to happen for every endpoint? If not, when will these need to be added for load tests tasks.

This is a really good question. The quick answer is that we'll have to run each endpoint (as they get added) against the parser and test to see if they need custom handling. If they do, we can make a new ticket to account for that case, and split up the work along the way. I think a field override is a sufficient way to handle avoiding an exception if that's not the work you need to focus on atm, and then just make sure there's a ticket in the backlog to go back and fix it.
I have some tickets I'm going to add to the backlog soon with some of this work. My view was that the base APIParser class should be as static and stable as possible, but the PrimeAPIParser and the SupportAPIParser classes should be in active development along with the Prime and Support tasks. </short answer>
I approached development with the following strategy: If the parser is returning the wrong basic data type for a field, it should not be a custom fix, it needs to be applied on the base data generation level. For example, generating a sentence like "Some cats fly high" for a field that always needs to be in a military time format is a base-level problem.
If it's something like -- oh, an item has to be smaller than the crate it goes in or the audits within the endpoint handler are going to fail, then that's an endpoint-level fix and needs to be custom code.
But also, I can't really account for all of these cases in one go, especially while the API is still in active development. There were a number of reasons for this, but here's a quick (? maybe) summary of my thoughts as I went through this:

Is an exception caused by a problem in the python, or is it a problem with the way we've formatted our .yaml files? If it's the API, it's sufficient to add an override for the field causing the error in the parser and make a ticket to fix our YAML file.

Are we okay with 4xx response codes being an aspect of load testing? Users make mistakes when they enter data and things don't always work out well for them. They could also misinterpret our API using the swagger docs just like the parser could misinterpret it, so those errors might actually be valuable cues into things we need to clarify/fix. This may be something we want to talk about on a team level.

Is it a truly business-rule level error that the user will have to learn and the parser will never be able to intuit? The item vs crate issue fell into this category for me. In this case, we have to write custom code to avoid making crates smaller than the items within them. We also can only find these problems by testing the endpoints with a variety of data, so we have to do this on a case-by-case basis.

The MTOServiceItemBasic issue was the most vague because I think we need that value in the enum options for when we return service items that have that type, but we never want to send it. I couldn't figure out if that was something we could indicate in the swagger or not, so I made some custom code for it! I think doing this will just have to be part of the process of developing against this tool.
I hope that makes sense or helps in anyway haha. But I definitely thought about this a bunch too and would be happy to discuss it any time


Don't show the Phone field when there is no phone number

[x] When there is no location phone number, don't show the phone icon or field.

Currently we are showing what looks like a clickable icon, e.g, at this location. though I think it's just the empty line showing. This is misleading for users so is a high priority fix.

This is done


Overhaul all entity AI

[ ] Lets give every single mob some kind of new AI using the Vanilla AI system instead of my horrible one.
[ ] Redo all bosses as well, convert over to vanilla AI systems.
[ ] Lich too.

Corrupted Pawn as well #70

Note to self: The new Chained Skull entity needs a proper model.


leaflet-geojson-vt plugin added

The leaflet plugin helps to render vector tiles for the GeoJSON data on the fly.

This looks like it overlaps with Leaflet.VectorGrid.Slicer. How is this different from VectorGrid functionality?

Yes, it is a little bit similar to leaflet vectorgrid. But it is very lightweight and very easy to use. Just by adding few lines of code, the geojson vector grid can be generated. It extends the geojson-vt and geojson options.


Added atheism, mlb11, & evals data & inference, multiLines, plot_ss functions

Added data and roxygen data for

atheism (inference for categorical)
mlb11 (simple regression)
evals (multiple regression).

Added functions:

inference and dependencies: chiTail, FTail, and normTail
plot_ss
multiLines

Thanks Albert!


feat: The user's cookie is now stored and passed to the API

Hi everyone!
We can now get the user's cookie via the API.
It allows us to send it with the getProduct() method and thus have different KP depending on the connected user.
The idea is to give access to the Nutri-Score v2.
For now, there are only the EN images in cache.

Codecov Report
Attention: Patch coverage is 11.11111% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 9.51%. Comparing base (4d9c7fc) to head (db6c300).
Report is 77 commits behind head on develop.

Files
Patch %
Lines

..._app/lib/data_models/user_management_provider.dart
12.50%
7 Missing :warning:

...kages/smooth_app/lib/data_models/login_result.dart
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5170      +/-   ##
==========================================
- Coverage     9.54%   9.51%   -0.04%
==========================================
  Files          325     327       +2
  Lines        16411   16579     +168
==========================================
+ Hits          1567    1578      +11
- Misses       14844   15001     +157

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

Thanks.
I forgot to mention it, but you have to log in again to get the cookie.


Updates D3 to v4 RC1, updates Ember-CLI to v2.6

This recreates the PR made by @ivanvanderbyl at https://github.com/spencer516/ember-d3-scale/pull/9. CCing @spencer516 and @taras

LGTM

I've tested this against maximum-plaid https://github.com/ivanvanderbyl/maximum-plaid/pull/28

@ivanvanderbyl thanks very much! @spencer516 any other comments? If not I'll merge this today.

My only comment is above ^^. It'd technically be a breaking change if that stays commented out. Probably worth deciding wether it's a go/no-go on that guidDomainScale. I think it's a nice-to-have; but not really something I'm particularly passionate about. :)

I think this could be useful. We should keep the guid functionality.

yes, I like this functionality as well – I probably won't have time to update the other ordinal scales today but I think I can get to this tomorrow. Thanks!

I think the ordinal scales have this already. So, we'd just need to uncomment that line.

Ok, @spencer516 I uncommented that line - tests for the cat-color-scale are failing, I didn't have time to dig into this too deeply - when using a color scale, the scheme categories from https://github.com/LocusEnergy/ember-d3-helpers/pull/4/files#diff-a2cbf1a835f25bb02cb42e2460dc1893R2 are just strings, and not functions, so https://github.com/LocusEnergy/ember-d3-helpers/blob/master/addon/utils/guid-domain-scale.js#L9 is failing. I think we can convert the string to an ordinal scale, and then pass to guidDomainScale. I can spend more time on this next week, curious if the solution is more immediately apparent to you.

@spencer516 that works perfectly! thanks. Updated the code and merging this now.


User-reported unhelpful error message

Program: https://code.world/#P1AOuuBRfEJa4lYugCC6URg
Message:
program.hs:1:1-20: error:
    Parse error: An equation was expected here.
    Are you missing the equal sign?

Not a serious attempt.


Content Type not matching file type

Storage Explorer Version: 1.12
Build Number: 20200116.10
Platform/OS: Windows 10
Architecture: x64
Regression From: Not sure
Bug Description
When copying a file from file explorer to Azure Storage Explorer, the content type is always application/octet-stream. I've tried this with both a .js and .json file.
Steps to Reproduce

Make a test.js and a test.json file
Copy those files into a Blob container
See that the content type is application/octet-stream and does not match the actual file type

Expected Experience
I expect the content type to be derived from the type of file or file extension of what's being copied over, not application/octet-stream
Actual Experience
When copying files over it always uses application/octet-stream
Additional Context
One odd thing I noticed is that on my mac, this does not happen and it correctly attributes the content type to a file. Here are the specs for that machine:
Storage Explorer Version: 1.4.2
Build Number: 20180917.1
Platform/OS: MacOS Catalina 10.15.4
Architecture: x64

Duplicate of #2267


Add emu support for prefetch instruction

This will need to be updated based on the emu reorg I just pushed in. Can you take a look?

I have rebased electron/emu_prefetch with electron-dev.


Overwrite dnsmasq conf instead of abort

Previously, if dnsmasq conf already exists then the setup command
would return without writing to it, which meant that the DNS may
not be configured properly.
After some discussion (https://github.com/alphagov/govuk-docker/pull/168#issuecomment-519891271)
we've concluded that there is no foreseeable scenario whereby a
developer will need to ever configure this file manually, so we
should be able to overwrite it safely.
'govuk-docker doctor' now fails if /etc/resolver/dev.gov.uk is in
any way different to dnsmasq.conf in govuk-docker.
Arises from #168.
Fixes #153.

CC'ing @benthorner as we discussed this work before you went away. Is this what you had in mind?


one

Why this changes are needed
Test why Deploy on master failed when merged a branch that has no failure on the "Build" workflow.
What are the changes
nothing

experiment completed


Attempting a new "azure_server_farm_with_rich_sku" leads to "Invalid Parameter: allow_pending_state"

What you expected to happen?
A new App Service Plan to be deployed
What happened?
Error: Invalid parameter allow_pending_state(:allow_pending_state)
Error: /Stage[main]/Azuretest/Azure_server_farm_with_rich_sku[resource myappservicename]/ensure: change from 'absent' to 'present' failed: Invalid parameter allow_pending_state(:allow_pending_state)

How to reproduce it?
Attempt to create any new resource using azure_app_server_farm_with_rich_sku
Anything else we need to know?
Looking at the code, it's clearly looking for a parameter called "allowPendingState":
  def self.invoke_create(resource = nil, body_params = nil)
    key_values = self.build_key_values
    Puppet.info("Calling operation ServerFarms_CreateOrUpdateServerFarm")
    path_params = {}
    query_params = {}
    header_params = {}
    header_params["User-Agent"] = "puppetlabs-azure_arm/0.2.1"

    op_params = [
      self.op_param("allowPendingState", "query", "allow_pending_state", "allow_pending_state"),
      .
      .
      .
    ]

Yet nowhere in the documentation is that parameter defined or listed as "required".  Indeed, according to http://superyyrrzz.github.io/NODE/azure-arm-website/azure-arm-website.ServerFarms.html, the parameter is OBSOLETE.
Adding a dummy "allow_pending_state => true"  led to a new error message:
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: no parameter named 'allow_pending_state'

So azure_server_farm_with_rich_sku fails because it requires a parameter it won't allow you to specify.

As a note:  For a quick hack, I commented out this line in two places in  /lib/puppet/provider/azure_server_farm_with_rich_sku/azure_server_farm_with_rich_sku.rb:
self.op_param("allowPendingState", "query", "allow_pending_state", "allow_pending_state"),

I then did a "chattr +i" on that file.  My code deploys now throw up sync-errors, but I was able to deploy my azure_server_farm_with_rich_sku resource just fine.

I created a PR just so I could pull that into my own environment:
https://github.com/puppetlabs/puppetlabs-azure_arm/pull/16

Thank you @Monual for your contribution.Will be taking a look on the issue.Thank you.

@Monual Thanks a lot for raising the Issue and the PR.The fix is available in the master.Apologies for the delay.


[Issue]: <hip/device_functions.h> header automatially padded to <hip/hip/device_function.h> when building rocm for torch extension

Problem Description
as  device_functions.h is located at /opt/rocm/include/hip , so expecting to use it as:
#include <hip/hip_runtime.h>
#include <hip/hip_runtime_api.h>
// Buggy: somehow setuptool will add prefix to <hip/hip/device_functions.h>
// #include <hip/device_functions.h>  // for amd_warp_sync_functions
#include <device_functions.h>

while it's auto padded to  <hip/hip/device_functions.h>,  and to make it work, had to modify as #include<device_functions.h>
is this expected behavior ?
Thanks
David
Operating System
Ubuntu 22.04
CPU
Ryzen
GPU
mi300
ROCm Version
ROCm 6.2.3
ROCm Component
hipBLAS
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

Hi @ZJLi2013. Internal ticket has been created to investigate your issue. Thanks!

Hi @ZJLi2013, can you provide a reproducer for this? I'm not seeing this in a quick test, setuptools seems to process #include <hip/hip_runtime.h>, #include <hip/hip_runtime_api.h>, and #include <hip/device_functions.h> fine when I pass it /opt/rocm/include as an include directory.

hi, @schung-amd , you can try on this repo: https://github.com/ZJLi2013/grouped_gemm/blob/rocm/csrc/permute.hip

I was able to reproduce the mangling with your code and found the root cause. We do a hipify pass in CUDAExtension here which maps CUDA header names to HIP header names. This works fine for files that are named differently, but the corresponding CUDA header is also called device_functions.h and is mapped to hip/device_functions.h here, causing the mangling.
I'm not sure if this workflow is unsupported, or if this simply wasn't tested. The fix is pretty simple, the mapping just needs to map include <device_functions.h to include <hip/device_functions.h instead. I suspect this is also an issue for driver_types.h and library_types.h. For now, as you've noted, you can use device_functions.h as a workaround.


feat: IntermediateField.adjoinRootEquivAdjoin_symm_apply_gen

For #6718

Looks reasonable, but I'm nowhere of an expert on this part of the library. Let's get some more eyes on this:
maintainer merge


ipfs version upgraded to latest version 0.4.9

That constant is the version of the ruby source code in this repo, not of go-ipfs

Oh..I didn't know that..Thanks btw..


Stable merge window for week 42 of 2020

New packages:

chessmarkable 0.4.2-1
whiteboard-hypercard 0.3.0-1

Updated packages:

harmony 0.0.1-15 → 0.0.1-16
~koreader 2020.09.29-2 → 2020.09-53-1~
~mines (previously minesweeper) 0.0.1-14 → 0.0.1-15~
oxide 2.0~beta-3 → 2.0~beta-4
recrossable 0.0.0-1 → 0.0.0-2
remux 0.0.1-18 → 0.0.1-20

Other changes:

Now licensed under MIT.
Added bootstrap script.
Packages listing split in sections.

Do we have a way to mark mines as replacing minesweeper such that it doesn't error out?
Did you solve the version number issue with KOReader, or do we need to patch in a new version number?

Do we have a way to mark mines as replacing minesweeper such that it doesn't error out?

This should be possible using some combination of Replaces and Conflicts (https://www.debian.org/doc/debian-policy/ch-relationships.html#s-replaces).

Did you solve the version number issue with KOReader, or do we need to patch in a new version number?

No, I did not.

I think we should hold mines back until we implement a proper automatic upgrade path with Replaces and Conflicts. As for KOReader, we should probably give it a version number that will be detected as newer.

++holding mines back
we are waiting for October koreader monthly release before switching to it
and technically don't need this upgrade, it's there bc the package wasn't
building due to outdated url

we are waiting for October koreader monthly release before switching to it and technically don't need this upgrade, it's there bc the package wasn't building due to outdated url

Didn't it also change the default to not grab input?

we are waiting for October koreader monthly release before switching to it and technically don't need this upgrade, it's there bc the package wasn't building due to outdated url

Didn't it also change the default to not grab input?

that was in last merge window, i just looked at this pr and it's not changing that line

Done.

lgtm


Wait for sync before backing up seed?

Should i wait for the wallet to synchronice before entering the seed to retreive my raiblocks?
BUG BOUNTY REPORT INFORMATION
In the interest of further improving the security of the network, we have launched the RaiBlocks Bug Bounty Program.
WARNING! Don't disclose any information for the Raiblocks bug bounty. First, please read the article about our Bug Bounty Program.
https://www.reddit.com/r/RaiBlocks/comments/7makm7/announcing_the_raiblocks_bug_bounty_program/

NORMAL BUG REPORT INFORMATION
If you are reporting a new issue, make sure that we do not have any duplicates
already open. You can ensure this by searching the issue list for this
repository. If there is a duplicate, please close your issue and add a comment
to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to
include the BUG REPORT INFORMATION shown below. If you fail to provide this
information within 7 days, we cannot debug your issue and will close it. We
will, however, reopen it if you later provide the information.
-->
Description of bug:

Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
Environment:

logs

Yes

This appears to have been a question that was asked and answered, but I'll add a note that with lazy bootstrapping (#995) the wait should be drastically reduced.


How to generate a string length>5

Now can generate a string with specify a length:
chance.string({ length: 5 })
but I want generate a string length > 5,
any suggestions? thanks!

In this case you can use min parameter.
chance.string({ min: 5 })

It seems that usage doesn't work.
My code is
chance.string({ min: 6 })
but it returns a string length=5

It's really doesn't work.
I created this test and they failed:
`
test('string() can take just a min and obey it', t => {
_.times(1000, () => {
t.true(chance.string({ min: 6 }) >= 6)
})
})
test('string() can take just a max and obey it', t => {
_.times(1000, () => {
t.true(chance.string({ max: 20 }) <= 20)
})
})`
I will check this code here.

I created a pull request to solve this problem #379

Thanks ever so.


Internally reference citations

It would be really nice to internally reference citations that get automagically formatted. For example, the latex internal references.
For that matter, having internal references to sections and subsections would be nice as well.

Could you give an example of what this would look like?

Yes, it basically just means putting a label in the markdown code that inserts a different bit of text in the compiled document. So for references in latex you basically enter the citation information in a separate document (not unlike the references.md file). The format of that references file looks like so (I just copied this from a manuscript of mine):

@article{label1,
title = {Natural Selection on the Influenza Virus Genome},
author = {Y. Suzuki},
journal = {Mol. Biol. Evol.},
volume = {23},
pages = {1902--1911},
year = 2006
}

@article{label2,
author = {S. E. Hensley and S. R. Das and A. L. Bailey and L. M. Schmidt and H. D. Hickman and A. Jayaraman and K. Viswanathan and R. Raman and R. Sasisekharan and J. R. Bennink and J. W. Yewdell},
title = {Hemagglutinin receptor binding avidity drives influenza {A} virus antigenic drift},
journal = {Science},
volume = 326,
pages = {734--736},
year = 2009
}

Then, when you want to cite a the first article, you simply put a tag in the text to indicate that is the citation you want.  In latex, the tag is \cite{label1} or \cite{label2}.  In a sentence in the manuscript, it would look like:

There is a long tradition of research aimed at identifying important regions of the hemagglutinin protein, and by proxy, the sites that determine sequence-cluster transitions \cite{label1}.

Or you could cite two papers like:

There is a long tradition of research aimed at identifying important regions of the hemagglutinin protein, and by proxy, the sites that determine sequence-cluster transitions \cite{label1, label2}.

Then, when the paper is typeset, the typesetting program makes it look like so:

There is a long tradition of research aimed at identifying important regions of the hemagglutinin protein, and by proxy, the sites that determine sequence-cluster transitions (Suzuki, 2006).

Or like this for two:

There is a long tradition of research aimed at identifying important regions of the hemagglutinin protein, and by proxy, the sites that determine sequence-cluster transitions (Suzuki, 2006; Hensley 2009).

In addition, the typesetting program adds the cited paper or papers to the references section as:

Suzuki Y (2006) Natural selection on the influenza virus genome. Mol Biol Evol 23: 1902–1911.

Or for two:

Suzuki Y (2006) Natural selection on the influenza virus genome. Mol Biol Evol 23: 1902–1911.
Hensley SE, Das SR, Bailey AL, Schmidt LM, Hickman HD, et al. (2009) Hemagglutinin receptor binding avidity drives influenza A virus antigenic drift. Science 326: 734–736.

The numbering and appearance happens in order of appearance in the manuscript.
I know it sounds complicated, but I think it is relatively simple to implement. It is conceptually the same thing for internal reference to sections and figures. You could just have some sort of label tag in the markdown that means you don't have to keep track of figure numbers when you're referencing them. Here is a huge document on them.
http://en.wikibooks.org/wiki/LaTeX/Labels_and_Cross-referencing

This has already been implemented in Markdown (and in a very clever way) with pandoc-citeproc. It would be fantastic if it could be integrated here somehow.


Adding Contributing.md

Want to add Contributing.md file to this repo.

@yashchaudhari008 I would like to solve this issue. Please assign it to me.

Hey @preronagit, contribution guidelines are already there in the README file. Are you suggesting to move it in a separate file?

@yashchaudhari008 Yes, I think making a separate file containing contribution guidelines will be better.

@yashchaudhari008
If you could assign this issue to me, I would greatly appreciate it. I am eager to contribute and work on this enhancement for the project.
Steps I will take:
Research Best Practices: Look into common practices for contributing guidelines in open-source projects.
.Draft the Document: Create a draft of the CONTRIBUTING.md file that includes:
.Overview of the project
.Steps to contribute
.Code style guidelines
.Submission process
.How to report issues
.Contact information for questions
.Seek Feedback: Share the draft with the team for feedback and suggestions.
.Make Revisions: Incorporate any feedback received to improve the document.
.Finalize and Commit: Finalize the document and commit it to the repository.

I don't think it is needed for now, hence closing this issue for now


[New Feature]: daac_data_subscriber.py should use a geojson file in the current working directory if not found in S3 opera-ancillaries

Checked for duplicates
Yes - I've already checked
Alternatives considered
Yes - and alternatives don't suffice
Related problems
No response
Describe the feature request
geojson files used in geofiltering are stored and retrieved from opera-ancillaries S3 bucket in the dev account. These files are expected to not change often and should only be a handful of files necessary for historical processing campaigns.
We now have a new use-case during INT and PST activities where we want to create many (10+) of specific geojson files to aid for a specific use-case. These files aren't expected to be used in production. Only a few people have access to opera-ancillaries S3 bucket, by design, because these are ops-critical files.
So to aid the use-case of INT and PST, a simple code change would be for daac_data_subscriber to fall back to looking for geojson file in the current working directory if not found in S3. This would only work when the subscriber is run directly on a Mozart box or a user machine - would not work if running in a verdi worker. But I believe this will satisfy INT/PST needs with minimal development and testing time.

Fixed and merged


issue with prod build

ERROR in vueBundle.js from UglifyJs
Unexpected token: keyword «const» [vueBundle.js:14019,6022]
I am getting this error. I am using Webpack : 4 .I know this component is using  ES6 and UglifyJS does not support ES6.
Do you some suggestion what should I do to resolve it.

Hi @shikharani16,
I found this:
https://github.com/terser/terser#why-choose-terser:
Why choose terser?
uglify-es is no longer maintained and uglify-js does not support ES6+.

terser is a fork of uglify-es that mostly retains API and CLI compatibility with uglify-es and uglify-js@3.

Maybe you can replace UglifyJs with terser-webpack-plugin:
https://www.npmjs.com/package/terser-webpack-plugin

Hi,, yes I tried It yesterday and it was working fine. Thanks

Hi, @shikharani16
I found the library build was not full compile to es5,
so I just publish version v0.13.5 to fixed this.
If you want to know, here is the commit d355657a00fa4793116ee398762e2a4d561c665b

sure will check and let you know. Thank you

Hi, I have updated  to release v0.13.5 and now look like new problem is coming after that.
I am stuck in show/hide. I am using v-model property to do show/hide from outside and inside the component.but may times it is  opening two Modal windows.

Hi,
You should update version to 0.15.0 for the new API.
here is the example:
https://codepen.io/hunterliu1003/pen/dyXyBpR

I tried using 0.15.0 but looks like that created many issues with CSS so I switched back to 0.13.5.


Add the Table of Contents to the subsections to Part 08 in Lessons

Table of contents must be added with appropriate links which connect to the correct subsections in the lesson.

Hi, I'd like to work on this issue

Sure I will assign it to you.

@Kaito117 Any progress on this?

Working on it, will make a pr asap

Please provide feedback on the work done so far -
changes


Deprecate constructor as a block

So far, we have been accepting both a block and the constructor:
keyword argument for providing a constructor. The former will not be
available in 1.0, so we're deprecating it now.
We also prepare the test suite for the change and make naming consistent
using "constructor" where we were using "pre-processor."

Hey @timriley, thanks for your review. It seems your PR does more than this one:

Like this one, it deprecates the constructor as a block.
It also moves default to a keyword argument.

Would it be ok if we split the whole work into two different PR? I'd merge this one, and then I can prepare another one taking what you did to support the new default: keyword argument. If so, I'll update this one to use Dry::Core::Deprecation#anounce as you did (I wasn't aware of its existence).

@waiting-for-dev Happy for it to be done in two PRs, yep :) Let us know when you're ready for the first one to be reviewed.
@solnic @flash-gordon - what do we feel about releasing these API changes? If they can be handled via deprecations and are therefore not a breaking change, I presume we might be more willing for them to go into a new ordinary 0.x.0 release? With the 1.0 release being the one that removes the deprecation shims?

Let us know when you're ready for the first one to be reviewed.

@timriley, it's ready now :rocket:

I change the code to deprecate in the same way that #83 is doing. This way we can already remove Compiler#visit_constructor method.

I change the code to deprecate in the same way that #83 is doing. This way we can already remove Compiler#visit_constructor method.

Nice one, I was actually going to ask about this :)
Have you pushed this change up, though? I can't see it in the diff yet.

Have you pushed this change up, though? I can't see it in the diff yet.

I just left my terminal while it was prompting for the ssh password :sweat_smile: Please, check it out now.

Continues at #111 (created branch in dry-rb/dry-configurable repo instead of my fork)


expose Fold in swc_ecma_visit

Trying to use Fold I hit this error:
error[E0432]: unresolved import `swc_ecma_visit::Fold`
  --> src/rules/no_dupe_args.rs:12:5
   |
12 | use swc_ecma_visit::Fold;
   |     ^^^^^^^^^^^^^^^^^^^^ no `Fold` in the root

error: aborting due to previous error

Oh... I made a mistake. Thanks :)

I noticed that Folder had a bug. I'll fix it today (if possible) or tomorrow (absolutely possible).

@kdy1 can you release swc_ecma_visit when you'll get the moment?

@bartlomieju I fixed it and published it. :)


denemo "GLib-GIO - ERROR   : No GSettings schemas are installed on the system"

Remove ~/.denemo-2.2.0
Start denemo
File -> Open -> Open
Crash. Console says "GLib-GIO - ERROR   : No GSettings schemas are installed on the system".

What's really weird is I found a way to circumvent this by opening an old file I cleaned up from all its previous content and saved in my home dir ($ denemo ~/start.xml.denemo ) and it doesn't crash: https://gist.github.com/SmilingHorse/33ebdaf3cf81f7a18109b77e6287b320#file-start-xml-denemo
No idea what's going :)
Ok so it's what you open denemo from that makes the difference: If I use a file manage like spacefm (that probably already has glib related env variables set) everything seems to be fine. So, using a gtk terminal / file manager / wm / dm to open denemo works while starting it off from some other terminal / file manager / wm / dm means when you try to use the browser popup widget (saving / loading) it crashes.
it's probably something like this: https://github.com/NixOS/nixpkgs/issues/26560
recent nixos stable.

There should be wrapGAppsHook in the nativeBuildInputs, see https://github.com/NixOS/nixpkgs/pull/43150


feat: Support insecure HTTPS with environment variable

Many development servers use self-signed certificates. This change makes it easier to support HTTPS development servers by disabling certificate validity checks.


[SLM] Pre-sharded weight support

The integration with serving layer param loading is missing. @LeshengJin Could you follow up with it?


how to computer the sensitivity

Nice job!
Where can I reference the way that you compute the sensitivity
sensitivity = 2 * self.lr * self.clip / self.data_size + (self.E - 1) * 2 * self.lr * self.clip
By the way, are there any other implementations of dp algo except the Gaussian?

The l2-sensitivity compution formula comes from one of our unpublished manuscripts. We encourage you to modify this part of the code according to your own needs (e.g., derive and compute l1-sensitivity for Laplace noise).
We haven't implemented Laplace or any other DP mechanism yet. You can add some functions to DPMechanisms.py to implement other mechanisms :-)

Thanks for your reply
For the 2rd question, I mean , you've already included so many functions in DPMechanisms.py but only the gaussian_noise_ls is used. So I am wondering what are the use of others or how can I use them?
Actually, I want to compare the effect of different DP algoes in my experiment where I propose a new FL framework and wanna incorporate a privacy preserving mechanism. But I am a newbie to DP so I hope that I can learn some stuff from others' code

Some functions in DPMechanisms.py are not used for DP-FedAVG. For this repository, we aim to provide a basic framework for federated learning (Fed-AVG). Maybe you can find the implementation of other DP mechanisms from other repositories.
If you want to compare different DP alg, I suggest you derive and compute the l1/l2-sensitivity very carefully.
Please do not use sensitivity = 2 * self.lr * self.clip / self.data_size + (self.E - 1) * 2 * self.lr * self.clip directly on your new FL framework. Maybe you can find how to apply different DP mechanisms to FL from some published papers.

many thanks for your suggestions


Issue status nomenclature

In advance of the PyPA sprint at the end of the month, I'd like to go through and do some issue labeling so we can more easily navigate the existing issues and see what work is to be done.
Looking at the issues, I think what we need most is a good indicator of what the status of a current bug is. I'm thinking we should use something like the following nomenclature:

[Needs Triage]: Issues that have not yet been classified
[Needs Repro]: Issues that have not yet been consistently reproduced
[Needs Discussion]: Issues where a general course of action has not yet been determined
[Needs Implementation]: Issues where we know generally what to do, but there's no PR yet

I have not yet used a classification system exactly like this, but this feels like a good start to me.
Before I start labeling every existing issue, I thought I'd run it up the flagpole. Thoughts @benoit-pierre @jaraco?

Since the sprints are in a week, I'm going to go ahead with this scheme so we can start classifying tickets. If we decide to tweak it later hopefully it won't be too difficult to undo (or maybe we can write a script to do it).

:+1:

I've written a little bot to backfill most issues with "Needs Triage".
Some of them probably have already been triaged in some way, but I'll start manually cleaning them up based on some heuristics.


Index page?

is there a page where I can see all the icons and their names? I'm using BuilderX to put together an app and icon picker is unusable.  it would be great to be able to find an icon, see its name and find it by name in BuilderX

https://oblador.github.io/react-native-vector-icons/

awesome!  thanks much


CB-3434 SDX should have a force delete options

Datalake from now sends an optional forced parameter from cloudbreak.

Redbeams force delete needs to come into this PR as well.


How to find the last version of yaml

Sorry, it is not a real issue.
Some yaml files are present in several repo. Can you tell me how to find the most recent version of yaml file ?
I use MD5 checksum of each file to remove same yaml files but after that, in some case, there are file have the same name/id but have different MD5 checksum and I don't know hox to select the more recent file. Use date of creation of file can be use but very sure, size of file ? peraphs, or ...
Tell me if you have an idea, please ?
Thank you for the very good tool !

Hi,
at the moment cent does not check if the template is the most recent or not, it just removes all duplicates.
See here #1 if you want to know more about deduplication.

Closing this as I got no reply
Feel free to open a new issue for any problem.
Thanks!


fix(utils): prevent newId from duplicating ids

Previously, newId would prevent duplicates of ids within the same component, but if you used multiple of the same component on a page, the ids could end up being duplicated. This change should make it extremely unlikely that any two generated ids are the same

Coverage decreased (-0.001%) to 99.071% when pulling 86e5cf558aabb0a249a96a01c6fb41149c4cffbb on bbaker/fix-new-id into 9c1baa8e52de94a84aa37aa28a4f434ee557d21b on master.

:tada: This PR is included in version 3.9.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


FIX default_records are no longer inherited to child classes

good idea :)

Yeah agree with this.

test failures due to bug in simple theme (I think) that is causing 3-branch to fail


Update rubygem license metadata to Apache-2.0

Update the gemspec metadata to reflect the current LICENSE.
Previously the project was licensed under MIT but was changed in 8eb9daeb27cb713b9c398f733c6646c586d5df59 to Apache-2.0

This is relevant for automated tooling that checks licenses and might only view the gemspec metadata.


Personalisation Redaction: Remove the TemplateRedacted class

User Story - Business Need
This ticket is a consequence of vanoity-team 1040.  Its goal is to eliminate technical debt to facilitate the implementation of redactions in notifications.

[ ] Ticket is understood, and QA has been contacted (if the ticket has a QA label).

User Story(ies)
As a Notify developer,
I want not to have extraneous tables in the database
So that the implementation is uncluttered and easier to follow.
Additional Info and Resources
The TemplateRedacted class duplicates information in the Template class.  The tables have a 1-to-1 relationship.
Engineering Checklist

[ ] Remove the TemplateRedacted class from models.py and elsewhere in the app/ and tests/ directories.  You can run grep -rn TemplateRedacted . to identify locations.
[ ] Create a new migration that drops this table.
[ ] Modify or delete unit tests as necessary.

Acceptance Criteria

[ ] All unit tests pass
[ ] The table "templates_redacted" is no longer in the database
[ ] All references to TemplateRedacted and template_redacted have been handled and removed

QA Considerations
For QA to populate. Leave blank if QA is not applicable on this ticket.

[ ] determine if this impacts the expected reponses of any routes, update Pydantic models in QA suite if so

Hey team! Please add your planning poker estimate with Zenhub @babyjunior56 @cris-oddball @EvanParish @k-macmillan @kalbfled @ldraney @nikolai-efimov

@k-macmillan @mjones-oddball Nathan says that Strike queries the table associated with the class, which precludes removing it.


Invalid mac address returned if ip prints to stderr

Function get_ns_veth_mac netns.py#L245 returns invalid result if ip netns ... command printed message in stderr.
ip netns command can print RTNETLINK answers: Invalid argument to stderr and still finish with exitcode 0, which causes next endpoint written to etcd:
{"name": "cali76e50b0684a", "labels": {}, "state": "active", "ipv6_nets": [], "mac": "RTNETLINK answers: Invalid argument\n2a:1b:78:a8:63:a4", "ipv4_nets": ["<IP_ADDRESS>/32"], "profile_ids": ["calico-k8s-network"]}
The root cause of this is that function NamedNamespace.check_output redirects stderr to stdout.
Solution is to read output only from stdout.

Assigning this a low priority on the assumption that the original cause (of ip netns erroring) can be worked around: I can adjust if it can't.  @cutwater did you find out why the command errored?

@matthewdupre We're still investigating the root cause. This issue occurs when there is something wrong with linux namespaces (usually after OpenStack neutron creates dhcp- namespace). Still ip netns works correctly, returns the result as expected and exitcode is 0, except it has such side-effect like writing message to stderr.
The problems begin when NamedNamespace reads stderr, considering it as a result of program execution. To my opinion this is incorrect behaviour, which affects data that is written to etcd.

@djosborne You added the stderr=STDOUT as part of:
65e86e67980e6efe7461b5ea6427067e2f45c332
Author:     Dan Osborne<EMAIL_ADDRESS>AuthorDate: Tue Oct 13 16:27:16 2015 -0700
Commit:     Dan Osborne<EMAIL_ADDRESS>CommitDate: Tue Oct 13 16:27:16 2015 -0700

Removed use of check_call in netns and improved logging

Replaced with check_output, which, when it throws an exception, includes a .output field which can be used to better log encountered errors

Can you remember why?

@neiljerram That was back when I was trying to improve error logging and diags. I followed the example on the python docs:

To also capture standard error in the result, use stderr=subprocess.STDOUT:

subprocess.check_output(
...     "ls non_existent_file; exit 0",
...     stderr=subprocess.STDOUT,
...     shell=True)
'ls: non_existent_file: No such file or directory\n'

When commands failed and printed why to stderr, libcalico could bubble up the reason why to the caller.
As @cutwater has shown, redirecting stderr is a bit overreaching. If we remove it, error messages may become a bit less descript, but we can just find another way to improve that.

Thanks @djosborne.
@cutwater Do you have any more information about the root cause when ip netns exec ... returns 0 but generates stuff on stderr?

@neiljerram here's how you can reproduce it:
mkdir /var/run/netns/zzz
ip netns
RTNETLINK answers: Invalid argument
RTNETLINK answers: Invalid argument
zzz

@adidenko For me that doesn't give the symptoms that this PR is concerned with:
root@smaug:~# mkdir /var/run/netns
root@smaug:~# mkdir /var/run/netns/zzz
root@smaug:~# ip netns
RTNETLINK answers: Invalid argument
RTNETLINK answers: Invalid argument
zzz
root@smaug:~# ip netns exec zzz ip link
RTNETLINK answers: Invalid argument
setting the network namespace "zzz" failed: Invalid argument
root@smaug:~# echo $?
1

Were there some other details to your repro?

That's exactly the case we faced. There were some leftovers in /var/run/netns directory on kubernetes worker nodes and this caused invalid "mac" parameter for newly created PODs on such nodes.
So you can do mkdir /var/run/netns/zzz on any kubernetes worker node and then try to spawn any POD on it.

(1) But why would anyone do that? (Instead of creating a namespace properly with ip netns add.)
(2) The PR discussion above says that part of the problem is that the exit code is 0, but my transcript just above shows that I get an exit code of 1.
I'm afraid I'm quite confused!

for example, some POD/container may mount /run/netns (or /run) from host system and create some netns for its needs. Then after removal of this POD it will leave the netns bind on host system.

ip netns (i.e. ip netns ls) works with exit code 0

$ mkdir /var/run/netns/zzz
$ ip netns ; echo $?
RTNETLINK answers: Invalid argument
RTNETLINK answers: Invalid argument
zzz
0

So if you have a valid netns and a leftover in /run/netns dir, the valid netns works fine but still throws warnings:
$ ip netns add okns
$ mkdir /var/run/netns/zzz

$ ip netns ls ; echo $?
RTNETLINK answers: Invalid argument
RTNETLINK answers: Invalid argument
zzz
okns
0

$ ip netns exec okns ip a ; echo $?
RTNETLINK answers: Invalid argument
1: lo: <LOOPBACK> mtu 65536 qdisc noop state DOWN group default qlen 1
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
0

We do not intend to use this broken zzz netns in Calico. It just causes those RTNETLINK answers: Invalid argument warnings on any ip netns * command execution which breaks networking for Kubernetes.

Thanks @adidenko, I see now, and agree that this is a strong argument for not having stderr=STDOUT in ip netns calls.  It basically means that our code can never reliably use the expected output of such a call.

Fixed by PR #149


[ai_coder] Write a hello world

Let's write a simple python program that outputs the text "Hello world".
Context file: hello_world.py

[ai_coder] Opening PR 8

Worked on it: https://github.com/pereferrera/ai_coder/pull/8

The program does not do anything yet because the print() statement was placed in a comment section. This should be fixed. We should also add a main section.

[ai_coder] Worked on it: https://github.com/pereferrera/ai_coder/pull/8

[ai_coder] Worked on it: https://github.com/pereferrera/ai_coder/pull/8

The file is still not executable. Please remove the """ at the beginning and end of the file.

[ai_coder] Worked on it: https://github.com/pereferrera/ai_coder/pull/8

[ai_coder] Worked on it: https://github.com/pereferrera/ai_coder/pull/8

The main block is defined twice now. Please remove one such block.

[ai_coder] Worked on it: https://github.com/pereferrera/ai_coder/pull/8


Error while reading the file...

Error while reading file **.

Click 'new file'
Click 'save as'
Enter name
Click 'save'
Profit

Thanks, mate. Close issue now. Working great!


Collapsed definitions for single-character spinners, to save memory a…

…nd reduce import time.
Type of changes

[ ] Bug fix
[ ] New feature
[ ] Documentation / docstrings
[ ] Tests
[X] Other - performance and memory reduction

Checklist

[X] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[X] I accept that @willmcgugan may be pedantic in the code review.

(no change made to CHANGELOG as this is purely an internal implementation change.)
Description
Python strings carry a lot of overhead, about 80-bytes each, not counting their contained characters. The _spinners.py definitions for spinners consisting of single characters define their frames as lists of single characters. The spinner logic is equivalent if these frames are collapsed down to single strings consisting of all the single characters.  For instance the initial dots definition can collapse down from
"dots": {
    "interval": 80,
    "frames": ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"],
},

to
"dots": {
    "interval": 80,
    "frames": "⠋⠙⠹⠸⠼⠴⠦⠧⠇⠏",
},

dots8bit is especially large, and so achieves quite a bit of savings.
This transform should also yield some improved speed at import time.

Codecov Report

Merging #2000 (22c26af) into master (e839bfb) will decrease coverage by 0.01%.
The diff coverage is 99.09%.

@@            Coverage Diff             @@
##           master    #2000      +/-   ##
==========================================
- Coverage   99.82%   99.81%   -0.02%
==========================================
  Files          71       71
  Lines        6943     7043     +100
==========================================
+ Hits         6931     7030      +99
- Misses         12       13       +1

Flag
Coverage Δ

unittests
99.81% <99.09%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

rich/_spinners.py
100.00% <ø> (ø)

rich/default_styles.py
100.00% <ø> (ø)

rich/highlighter.py
100.00% <ø> (ø)

rich/markdown.py
100.00% <ø> (ø)

rich/syntax.py
99.27% <97.72%> (-0.34%)
:arrow_down:

rich/pretty.py
99.71% <98.88%> (-0.29%)
:arrow_down:

rich/__main__.py
100.00% <100.00%> (ø)

rich/_inspect.py
100.00% <100.00%> (ø)

rich/color.py
100.00% <100.00%> (ø)

rich/console.py
100.00% <100.00%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


Timestamp Microservice User Story 6 is bad practice

Challenge Timestamp Microservice has an issue.

User Story: If it does not contain a date or Unix timestamp, it returns null for those properties.

For the very first backend project, do we want to be teaching people to return nulls or nonsensical data for errors?  They should be returning error codes.  I recommend changing it to read instead to:
User Story: If the request not contain a date or unix timestamp, an HTTP error 400 is returned.

Вы уже предали тот кого вы даже не знаете и обвинили не обосновано в краже целый день гнобили я может учусь только что правда а Вы за айфон меня обвинили у кого там опера хром  стоит хвастался вот увидимся скоро в суде спасибо

That was ... random.  DG000gl, please don't comment on issues unless you have something helpful to add.

Thanks @chuckadams I am in favor of returning HTTP error codes, just need to debate that should we be providing a link or hint of some sort on why to use HTTP error codes and what they are.
/cc @FreeCodeCamp/issue-moderators

@chuckadams Thanks for your help. We are improving the microservice projects in the new curriculum. I'm closing this and moving the discussion to the related issue in the curriculum expansion repository https://github.com/FreeCodeCamp/CurriculumExpansion/issues/82.


Attack Flow Builder v2.0.1

2.0.1 (2022-12-20)
Bug Fixes

add polyfill for crypto.randomUUID() 3ed71f7
allow copy of DateTimeField text 8cdbf01
enable text copying across application 5ee1ae6
fix issue where undo/redo menu items don't update 556c172
fix TextField update behavior in Firefox 1d8ee9a
make held hotkey behavior mirror browser's 05883b2
prevent tab key from focusing on disabled fields 94a054d

LGTM -- the Test / python_lib target is failing for an unrelated reason so I'm going to merge this as-is.


[Bug] R libraries installed need to be re-installed after stop and re-start the workspace

Describe the bug
R libraries installed need to be re-installed after stop and re-start the SageMaker workspace
To Reproduce
Steps to reproduce the behavior:

Start a SageMaker workspace
Create a new R notebook
Install a library that is not pre-installed
Load the library
Stop the workspace
Start the workspace
Try to load the library
You get an error, the library needs to be re-installed

Expected behavior
That the libraries installed while working on a workspace are available after stopping and re-starting the workspace.
Versions (please complete the following information):

Release Version installed v4.0.1

Additional context
We tried by stopping and starting the workspace directly from the AWS console and the error persist.

@aGutierrezSacristan Is this a new error that came up in latest release or there was a time this was working for you?

Hi @Bingjiling and @jn1119 for your quick response.

It was an error that was also on the previous released
The issue exists on the R notebooks created through SWB and in the ones created through AWS Sagemaker directly too. I tried both.
Yes, this is not an issue in Rstudio workspaces, in which the libraries installed are still there after the workspace is stopped and started again.

I have been trying to narrow down the issue with the SageMaker notebook R library installation trying different things:

When I created the notebook directly on AWS Sagemaker I  did not select any lifecycle configuration but the R installation issue persists, so probably not related to the life cycle configuration.
I also checked where the libraries installed are located, and in SageMaker the libraries installed are in /home/ec2-user/anaconda3/envs/R/lib/R/library, and this folder is "reset" every time the workspace is stopped. How can we know which folders are "reset" and which ones are not? any way to avoid having this folder to be "reset" when the workspace is stopped?
For reference in the Rstudio workspace, when a library is installed is in /home/rstudio-user/R/x86_64-koji-linux-gnu-library/3.4, and this folder remains with all the changes a new libraries added after stop and start.

Thanks!


status page has absolute links instead of relative paths

I noticed that the /status page in AlertManager has broken CSS.  It ended up being because the CSS links are incorrect.
We proxy to /alertmanager with nginx, but I confirmed it also misbehaves talking directly to AlertManager. This happens on at least 0.0.3 and 0.0.4. I don't see anything in the source that would cause /status to be treated differently.
We are running alertmanager -config.file=some_config.conf, not bothering to set any other command-line parameters.

Relative hrefs for /, /alerts and /silences. CSS loads just fine.

curl -s localhost:9093/ |grep default.css
    <link href="static/css/default.css" media="all" rel="stylesheet" type="text/css" />

curl -s localhost:9093/silences |grep default.css
    <link href="static/css/default.css" media="all" rel="stylesheet" type="text/css" />

Absolute hrefs, not correct. Links all broken, no CSS styles loaded.

curl -s localhost:9093/status |grep default.css
    <link href="/static/css/default.css" media="all" rel="stylesheet" type="text/css" />

No proxy here, but the CSS loads correctly for me on all pages, including /status. The paths are absolute in all of them. This is both when building from HEAD as well as when trying out the 0.0.4 tarball.
curl -s localhost:9093/ |grep default.css
    <link href="/static/css/default.css" media="all" rel="stylesheet" type="text/css" />
curl -s localhost:9093/silences |grep default.css
    <link href="/static/css/default.css" media="all" rel="stylesheet" type="text/css" />
curl -s localhost:9093/status |grep default.css
    <link href="/static/css/default.css" media="all" rel="stylesheet" type="text/css" />

You said it also misbehaves without a proxy?

My examples above all were without a proxy, using the release from github, built on your thinkpad! :grinning:
Build Information

branch	master
date	20150909-17:20:43
go_version	1.4.2
revision	e00f5b1
user	julius@julius-thinkpad
version	0.0.4

I'll try building 0.0.4 myself and see what happens.

Ok, that's very strange, as with that exact binary I'm not getting the problem you're seeing. It's a static binary, so we should be getting the same results.
Are you sure that your curl doesn't use a proxy (via environment variable http_proxy or in ~/.curlrc) that does something weird?

I just compiled 0.0.4 in a Vagrant VM, and tested again. Same results. I even zapped web/files/blob.go.
And, no, no funky curl configs.
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ cd /tmp
vagrant@vagrant-ubuntu-trusty-64:/tmp$ rm -fr /tmp/alertmanager/
vagrant@vagrant-ubuntu-trusty-64:/tmp$ git clone https://github.com/prometheus/alertmanager.git
Cloning into 'alertmanager'...
[SNIP]
Checking connectivity... done.
vagrant@vagrant-ubuntu-trusty-64:/tmp$ cd alertmanager/
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ git co -b 0.0.4 tags/0.0.4
Switched to a new branch '0.0.4'
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ make dependencies-stamp
Go version 1.4.2 required but not found in PATH.
About to download and install go1.4.2 to /tmp/alertmanager/.build/go1.4.2
[SNIP]
touch dependencies-stamp
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ rm web/blob/files.go
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ make web
./web/blob/embed-static.sh web/static web/templates | /tmp/alertmanager/.build/go1.4.2/bin/gofmt > web/blob/files.go
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ make
GOROOT=/tmp/alertmanager/.build/go1.4.2 GOPATH=/tmp/alertmanager/.build/gopath /tmp/alertmanager/.build/go1.4.2/bin/go get -d
touch dependencies-stamp
GOROOT=/tmp/alertmanager/.build/go1.4.2 GOPATH=/tmp/alertmanager/.build/gopath /tmp/alertmanager/.build/go1.4.2/bin/go build  -o alertmanager
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ touch alertmanager.conf
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ ./alertmanager &
[1] 3961
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ alertmanager, version  ()
  build user:
  build date:
  go version:
WARN[0000] Couldn't load silences, starting up with empty silence list: open silences.json: no such file or directory  file=main.go line=76
INFO[0000] Replacing aggregator rules (old: 0, new: 0)...  file=manager.go line=246
INFO[0000] Running notification dispatcher...            file=main.go line=151
INFO[0000] listening on :9093                            file=web.go line=76
INFO[0001] Recomputing notification outputs (active alerts have changed, inhibit rules have changed)  file=manager.go line=398

vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ curl -s 0:9093/status  |grep default.css
    <link href="/static/css/default.css" media="all" rel="stylesheet" type="text/css" />
vagrant@vagrant-ubuntu-trusty-64:/tmp/alertmanager$ curl -s 0:9093/silences  |grep default.css
    <link href="static/css/default.css" media="all" rel="stylesheet" type="text/css" />

What the...
Now I'm getting the same (that is, relative paths on everything except /status), although the CSS still loads correctly with both path variants since it's correct both as a relative and absolute path. I have no idea why I didn't get this before, maybe I did something really wrong, even during the copy&paste of the output. Strange.
Looking more.

Aha! in HEAD it's all absolute paths. I noticed that Alertmanager doesn't shut down when it cannot acquire it's port (ugh), so I must've still been running a HEAD Alertmanager in another terminal and didn't notice that the 0.0.4 one didn't start up correctly because it doesn't indicate that properly.
Ok, so something evidently changed between 0.0.4 and HEAD there. Have you given HEAD a try?

Also in general, this current experimental Alertmanager is being replaced by a complete rewrite, of which an Alpha is now out: https://groups.google.com/forum/#!topic/prometheus-developers/AmGnO_Sj7xI
You might already want to use the new one. The old one is experimental and a horrible proof-of-concept codebase, but has seen more practical use. The new one doesn't have as many miles yet, but will be the shape of things to come and is more future-proof and properly built.

Yep, HEAD behaves correctly.
I'll try to have a look at the alpha.
It looks like this ab9348240e5d62c3b6609c4d9f23610dccca7779, insofar as it fixed /silences and /alerts by causing the pathPrefix to now be set.
So, presently, /status is the only path that is working "correctly".

Just to wrap this up, with 0.0.4, if I set -web.external-url and -web.path-prefix everything is working.
Although, it's still the wonky behavior of not always having PathPrefix set:
$ curl -s 0:9093/alertmanager/status|grep default.css
    <link href="/alertmanager/static/css/default.css" media="all" rel="stylesheet" type="text/css" />

$ curl -s 0:9093/alertmanager/|grep default.css
    <link href="static/css/default.css" media="all" rel="stylesheet" type="text/css" />

$ curl -s 0:9093/alertmanager/silences|grep default.css
    <link href="static/css/default.css" media="all" rel="stylesheet" type="text/css" />


[hi] Localize Zero Trust Architecture in Hindi.

Signed-off-by: Aarav238<EMAIL_ADDRESS>Describe your changes
I have localize Zero Trust Architecture in hindi.
Related issue number or link
resolves #1671
Checklist before opening this PR

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

This PR is a duplicate; please see https://github.com/cncf/glossary/pull/1249#issuecomment-1409749859.

closing as this is a duplicate PR for a duplicate issue.


Navbar component?

Hi! I'm trying the components, and i'd find really useful a responsive navbar component like the one on the documentation website. Are you planning to add this?
Thank you!

hi, on the ux.contactlab.com website we aren't really using a navbar component, beacuse we started it out some months ago but we haven't defined a scope (if can be a public component or just internal, if we have to handle multi level navigations, ecc) so at the moment its just a DOM element. If you really need it you can copy the HTML and CSS from the sources of the website.

Moved to https://github.com/orkestra-elements/orkestra-elements/issues/2


Added Install Documentation for Kubernetes

Howdy Technotim,
I decided to add littlelink-server to the k8s-at-home helm repo, you can see it here
I've added a link to it in your readme as well as some starter documentation to help your users that use Kubernetes and helm to get started with it if they are looking at your project and would like to use Kubernetes instead of docker-compose.

Thank you!  Love it!


e2e-metrics should handle case-insensitive for calculation.

Guideline for team is to use ALL Caps for TCID-xxx-xxx-xx for creating TCID.
Use case of a manual error..  (Lets assume only 1 TCID in master and GITLab) Will it show 100% coverage or 0% coverage
TCID-ABC-DEF-GHI (Gitlabjob)
TCID-ABC-DEF-XYZ (master)

@ajeshbaby it will show 0% as coverage

https://github.com/mayadata-io/e2e-metrics/pull/9 will put all test case ids starting with tcid i.e. small cased as deprecated.


change localhost to IP - Swagger

CCXT REST - 2.5.0

Server=http://localhost:3000/

How to change the Swagger URL - localhost is not working for me, since I need an IP-Address to get the calls working.
With Postman no issue - but I would like to use the "/explorer" and the "Try it out"

Server=http://<IP_ADDRESS>:3000/

api/swagger/swagger.yml
there is strict servers list to access, this property need to be removed
servers:
  - url: 'http://localhost:3000/'

Don't have permissions to create PR.


Block rendering update

A single step change, the minimum required to have customized block rendering by mod/block.  Later PR's will enhance this but as it is this is fully functional.

Can one of the admins please verify this patch?

add to whitelist

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/22/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/23/
Hooray Jenkins reported success with all tests good!

Tested around a bit in-game, looked good, with the quick review it seemed fine to merge away. Thanks! :-)

@Cervator, a bit of a side node: that "add to whitelist" command, can be a bit perplexing for people that do not know what goes on under the hood. It may seem a comment for the people involved in the discussion rather than a command for a bot. Could this be improved? I'm thinking along the line of "GooeyHub? Add to whitelist please!" or even "GooeyCmd: add to whitelist".

Oh yes, we have full control of the phrase via RegEx and need to do that. I just haven't taken the time yet and suck at RegEx :D

I figured it out.
"Gooey, add to whitelist please." should already be enough for somebody who doesn't know what's going on: they'd immediately understand that the message is not for them. The "please" at the end is especially important: we all know machines will become more intelligent, will eventually ask voting rights and end up conquering the planet. So, better be nice to them from the beginning.
But apart from that... Gooey (or whatever bot is being addressed).... should answer back!! I.e. "Of course Cervator. Here we go: I added DonaldDuck to the whitelist." To which you should most certainly reply "Thank you Gooey!" and it should follow with a closing "You are most welcome Cervator."
P.S. yes, RegEx. After banging your head on them for enough time they are nice. But it's not clear if they are nice because you finally understood them or because your brain has been mashed to a pulp and in that state eating raw cabbage would taste delicious.


cherry pick PR#547 to release-0.6

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/retest

/retest

/retest

/approve
/lgtm

/approve


tests: Add a couple negative tests for block decoding

Relates to #11.
Add some tests of decoding non-block data.

@piotr-iohk I've fixed @jonathanknowles remarks & added a test to unserialize a block contains MANY (many = 2) transactions.


multi UIWindow not work properly

hi, in my case, I use two uiwindow to manage my application's launch view controller and app view controller, I found the logic in SVProgressHUD when init once view like blow:
dispatch_once(&once, ^{ sharedView = [[self alloc] initWithFrame:[[[UIApplication sharedApplication] delegate] window].bounds]; });
this is use the application's delegate window as show window, but in my case, the window is not my current showing window, so can we change this logic to application's keywindow or their is some other reason can't do this? thanks!

i think this may be my reason cause in swift appdelegate's window property is a readonly property, so i can't reassign it.. i avoid this situation by use another way.. and in most common case people just use one single in window in an app.


Testing

Add tests and infrastructure

@julian-evers take a look at the BPX tests as a simple example of what you need to do

Done in #18


Add support for installing alpha/beta/RC builds

Currently the tool checks for the php versions API, which does not include alpha/beta/RC builds, such as 8.2.0rc2 currently in testing.

With 5a6573d01c14c68d09b5e7bbc4b636f1aa51310d I added the option to pass a --php-git-branch instead of a --php-version. That way it is possible to build git tags as well. So for pre-relase versions you can either specify the current branch (PHP-8.2) or one of the release tags (php-8.2.0RC4).

Sorry, wrong commit referenced, it was this one: d2aacdc35d518a8c96211a96f56d215ba5ec6aab


The length of the visa card is 13, 16, please update

Here is the link I refer to it
https://en.wikipedia.org/wiki/Payment_card_number

Actually see this: https://github.com/jessepollak/payment/issues/24

Actually its been updated to only 13, 16.  I am going to fix it to 16.

I am using the latest card.js from cdnjs. 2.5.3. And it still allows for 19 digits.

yep Card has not been updated yet to Payment 2.4.4

I see, I didn't know they were related.
It would be nice to just have an option to choose which cards are supported. Otherwise great plugin!


What is your suggestion for feature extraction

What is your suggestion for feature extraction? Layer Inception 5b/outout or?

Generally global_pool would work as frame-level feature representations.

Many thanks!


Some CPEs ending with :*:*:*:*:*:*:*:* are missing

Firstly, thank you for your great job!
Recently, I find that some CPEs ending with :*:*:*:*:*:*:*:* are missing, for example, cpe:2.3:a:artifex:ghostscript:*:*:*:*:*:*:*:* is not listed in this yaml. And it's not an isolated case.

I got the CPE cpe:2.3:a:artifex:ghostscript:*:*:*:*:*:*:*:* from this url. It seems NVD itself uses the unreported CPE.

That is not a CPE, that is a Match Criteria string.
According to the CPE API documentation: CPE Match Criteria comes in two forms: CPE Match Strings and CPE Match String Ranges. Both are abstract concepts that are then correlated to CPE URIs in the Official CPE Dictionary. Unlike a CPE Name, match strings and match string ranges do not require a value in the part, vendor, product, or version components. The CVE API returns CPE Match Criteria within the configurations object.

@scanossmining thank you for your explanation.


add samples to jsctrl readme

also explain how to use 'aici jsinit'

samples could be copied from pyctrl readme and translated


remove codec from Service constructor params

Service already takes a ServiceCodecProvider, and we recently made it easier for users to create their own providers.

Fixed by #384


Set minimum charge level

Make it posible to set minum charge level. So that it can be used in automation, would be great so that it could be set higher on Sunny days, to top with solar power

Hi @dkmedic ,
Did you mean to open an issue instead of a PR?
There already is a service (skodaconnect.set_charger_max_current) but it doesn't work with specific amp values (the API rejects it). Only allows values 252 (reduced, = 8A) and 254 (= maximum). I'd also love to have more control but AFAIK this just isn't supported by Skoda at this time. It's also not possible to set this from the car.

Hi @dkmedic ,
Did you mean to open an issue instead of a PR?
There already is a service (skodaconnect.set_charger_max_current) but it doesn't work with specific amp values (the API rejects it). Only allows values 252 (reduced, = 8A) and 254 (= maximum). I'd also love to have more control but AFAIK this just isn't supported by Skoda at this time. It's also not possible to set this from the car.

The old API (for all Skodas except Enyaq) accepts values from 1 to 254 where 252 corresponds to Minimum in the app and 254 to Maximum (API is integers). For Enyaq the API will only report and accept 'Reduced' or 'Maximum'.
For old API the value accepted differs depending on which car and the onboard charger. For my Superb iV I've tested values from 1 to 16 succesfully and it will limit the ampere to that value. I haven't been adventurous and tried values other than 1-16, 252 and 254.

Make it posible to set minum charge level. So that it can be used in automation, would be great so that it could be set higher on Sunny days, to top with solar power

There's a service to set charge level called skodaconnect.set_charge_limit (the minimum charge level).


Tap on push notification but application does not launch [android 12]

Version 4.2.4
When app is background or killed, tapping on a push notification, the app does not lauch. It just happens on Android 12
Please have a look.

Oh, when i changed targetSdkVersion = 30, it works

same, and we are not able to downgrade targetSdkVersion


Failed to import -<EMAIL_ADDRESS>

Failing module

GitHub: https://github.com/ruby/ruby.wasm
npm<EMAIL_ADDRESS>
import {} from "https://esm.sh/<EMAIL_ADDRESS>
Error message
GET https://esm.sh/v135/<EMAIL_ADDRESS>net::ERR_ABORTED 500 (Internal Server Error)

This might not actually be an issue with esm.sh. When I try it on esm.run it also can't bundle it. I can get it to work using the raw file with an import map (jsfiddle) but that defeats the purpose of using esm.sh.

The 2.5.0 includes a workaround fix to avoid this issue. I think you can reproduce it with the exact same version I mentioned in the original description: https://esm.sh/<EMAIL_ADDRESS>

The 2.5.0 includes https://github.com/ruby/ruby.wasm/pull/362 to avoid this issue. I think you can reproduce it with the exact same version I mentioned in the original description: https://esm.sh/<EMAIL_ADDRESS>
nice, i think this is a bug for resolving index.js as your pr metioned, i will take a look! thanks!


Morphing through > 2 images without screen flash?

hi, I created this code:
https://github.com/donundeen/art_image_morpher
which continuously morphs through an almost endless list of images, using facetracking to create the mesh triangles.
It's pretty cool, and you can see it in action here:
http://<IP_ADDRESS>/art_image_morpher/
however, you'll notice between finishing one animation, and starting the next, there's a bit of a flash. I'd like to eliminate that, so the transitions are smoother.
Do you have any idea how that might be accomplished?
Thanks!

update:
by calling morpher.fromJSON(json)
instead of
morpher = new Morpher(json)
$("#morpherHere").html(morpher.canvas)
I can get the transitions without the flashing.
However, now I sometimes get the image resizing before the morph. Because my images are not all the same size.
Previously. with the flashing (re-instantiating morpher each time), this never happened.
I'm guessing that by calling fromJSON the way I did, I've short-cutted some necessary bit of code that sets up source and destination images correctly.
Any ideas?
Thanks!

Could you isolate these problems in something like jsfiddle.net so I could experiment with them easily?
My blind guess is that in both cases the problem lies in preloading images. In case 1 it flickers, because images aren't loaded yet. In case 2 dimensions are applied to canvas before new images are rendered.
You could try creating new Morpher instance for each transition and destroy the previous one after the next one is fully loaded. So it would be the same tactic you would use to create image slideshow with fade animations.

Hi,
I've fixed these problems in most cases by standardizing the width.
Have a new problem now, which I'll create a new issue for.
btw, loving this tool, created this with it:
http://<IP_ADDRESS>/art_object_morpher/


SSO revamp of add-in authentication docs

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/add-ins/authenticate-a-user-with-an-sso-token.md
:warning:Warning

Details

:warning:Warning

Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/add-ins/authenticate-a-user-with-an-identity-token.md
:white_check_mark:Succeeded

docs/add-ins/authentication.md
:white_check_mark:Succeeded

docs/add-ins/images/single-sign-on-graph-permissions.PNG
:white_check_mark:Succeeded

docs/add-ins/images/single-sign-on-web-api-details.PNG
:white_check_mark:Succeeded

docs/add-ins/images/single-sign-on-web-details.PNG
:white_check_mark:Succeeded

docs/add-ins/implement-sso-in-outlook-add-in.md
:white_check_mark:Succeeded

docs/add-ins/inside-the-identity-token.md
:white_check_mark:Succeeded

docs/add-ins/toc.yml
:white_check_mark:Succeeded

docs/add-ins/validate-an-identity-token.md
:white_check_mark:Succeeded

docs/breadcrumb/toc.yml
:white_check_mark:Succeeded

docs/add-ins/authenticate-a-user-with-an-sso-token.md

Line 203: [Warning] Invalid file link:(~/add-ins/implement-sso-in-outlook-addin.md).

[Warning] 'W:\mvnr-s\docs/add-ins/call-a-service-by-using-an-identity-token.md' would be overwritten by redirection rule configured in master redirection file .openpublishing.redirection.json. Please remove the original file to resolve this warning.
[Warning] 'W:\mvnr-s\docs/add-ins/use-the-token-validation-library.md' would be overwritten by redirection rule configured in master redirection file .openpublishing.redirection.json. Please remove the original file to resolve this warning.
[Warning] 'W:\mvnr-s\docs/add-ins/use-php-to-validate-an-identity-token.md' would be overwritten by redirection rule configured in master redirection file .openpublishing.redirection.json. Please remove the original file to resolve this warning.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/add-ins/authenticate-a-user-with-an-identity-token.md
:white_check_mark:Succeeded

docs/add-ins/authenticate-a-user-with-an-sso-token.md
:white_check_mark:Succeeded

docs/add-ins/authentication.md
:white_check_mark:Succeeded

docs/add-ins/call-a-service-by-using-an-identity-token.md
:white_check_mark:Succeeded

docs/add-ins/images/single-sign-on-graph-permissions.PNG
:white_check_mark:Succeeded

docs/add-ins/images/single-sign-on-web-api-details.PNG
:white_check_mark:Succeeded

docs/add-ins/images/single-sign-on-web-details.PNG
:white_check_mark:Succeeded

docs/add-ins/implement-sso-in-outlook-add-in.md
:white_check_mark:Succeeded

docs/add-ins/inside-the-identity-token.md
:white_check_mark:Succeeded

docs/add-ins/toc.yml
:white_check_mark:Succeeded

docs/add-ins/use-php-to-validate-an-identity-token.md
:white_check_mark:Succeeded

docs/add-ins/use-the-token-validation-library.md
:white_check_mark:Succeeded

docs/add-ins/validate-an-identity-token.md
:white_check_mark:Succeeded

docs/breadcrumb/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix CI

Lots of cuda failures so putting this up to avoid too many commits

What caused these? Is it primarily just the _masked additions?

Yeah it looks like that or at least added _masked.X opinfos. It also looks like someone added __getitem__ and a functorch variant but I honestly can't even find that opinfo. I'm going to make an issue for the ctor_inside_grad test because that looks like a pretty bad bug and it might get a little buried