@storybook/angular-4.0.0-alpha-18 issues with angular 6

If you are reporting a bug or requesting support, start here:
Bug or support request summary
Issue 1:
when npm run storybook picks up angular default projects webpack config ..
and in default project of mine is of type angular application & options.style property which takes in style files ,
angular cli/builder expects the path to the file be relative to workspace ..
but story book appends '<project>.rootpath to every style file inoptions.style.. which mean if i  add style path relative torootstorybook will not work & if i add style path relative toroot` ng serve does not work
Issue 2
when npm run storybook when transpiles it does not seam to exclude node_modules
in presence of @angular-devkit/core
it transpiles<EMAIL_ADDRESS>and fails to run
Steps to reproduce
npm run storybook with below angular.json & packages mentioned below
angular.json application json
"app1": {
      "root": "apps/app1/",
      "sourceRoot": "apps/app1/src",
      "projectType": "application",
      "prefix": "app",
      "schematics": {
        "@schematics/angular:component": {
          "styleext": "scss"
        }
      },
      "architect": {
        "build": {
          "builder": "@angular-devkit/build-angular:browser",
          "options": {
            "outputPath": ".output/dist/apps/app1",
            "index": "apps/app1/src/index.html",
            "main": "apps/app1/src/main.ts",
            "polyfills": "apps/app1/src/polyfills.ts",
            "tsConfig": "apps/app1/tsconfig.app.json",
            "assets": [
              "apps/app1/src/favicon.ico",
              "apps/app1/src/assets"
            ],
            "styles": [
              "apps/app1/src/styles.css"
            ],
            "scripts": []
          },
          "configurations": {
            "production": {
              "fileReplacements": [
                {
                  "replace": "apps/app1/src/environments/environment.ts",
                  "with": "apps/app1/src/environments/environment.prod.ts"
                }
              ],
              "optimization": true,
              "outputHashing": "all",
              "sourceMap": false,
              "extractCss": true,
              "namedChunks": false,
              "aot": true,
              "extractLicenses": true,
              "vendorChunk": false,
              "buildOptimizer": true
            }
          }
        },
        "serve": {
          "builder": "@angular-devkit/build-angular:dev-server",
          "options": {
            "browserTarget": "app1:build"
          },
          "configurations": {
            "production": {
              "browserTarget": "app1:build:production"
            }
          }
        },
        "extract-i18n": {
          "builder": "@angular-devkit/build-angular:extract-i18n",
          "options": {
            "browserTarget": "app1:build"
          }
        },
        "test": {
          "builder": "@angular-devkit/build-angular:karma",
          "options": {
            "main": "apps/app1/src/test.ts",
            "polyfills": "apps/app1/src/polyfills.ts",
            "tsConfig": "apps/app1/tsconfig.spec.json",
            "karmaConfig": "apps/app1/karma.conf.js",
            "styles": [
              "apps/app1/src/styles.css"
            ],
            "scripts": [],
            "assets": [
              "apps/app1/src/assets"
            ]
          }
        },
        "lint": {
          "builder": "@angular-devkit/build-angular:tslint",
          "options": {
            "tsConfig": [
              "apps/app1/tsconfig.app.json",
              "apps/app1/tsconfig.spec.json"
            ],
            "exclude": [
              "**/node_modules/**"
            ]
          }
        }
      }
    }

web pack appends to app1.root to every style element below .( angular expects it is relative to workspace & not project root)
"styles": [
          "apps/app1/src/styles.css"
        ],

Please specify which version of Storybook and optionally any affected addons that you're running
"@storybook/addon-actions": "4.0.0-alpha.18",
"@storybook/addon-links": "4.0.0-alpha.18",
"@storybook/addon-notes": "4.0.0-alpha.18",
"@storybook/addons": "4.0.0-alpha.18",
"@storybook/angular": "4.0.0-alpha.18",
"babel-loader": "^8.0.0-beta.6",
"@babel/core": "^7.0.0-rc.4",
"@angular-devkit/architect": "^0.7.5",
    "@angular-devkit/build-angular": "~0.6.1",
    "@angular-devkit/build-ng-packagr": "^0.7.5",
    "@angular/cli": "~6.1.2",
    "@angular/compiler-cli": "^6.1.0",
    "@angular/language-service": "^6.1.0",

Acceptance criteria
npm start storybook

should not pick files from node_module for transpile
should be able to pick styles like ng serve

Who to contact
Arvind Nama<EMAIL_ADDRESS>

Issue 1: was fixed in #4058 (not released yet)
Issue 2": storybook uses your default src/tsconfig.app.json file (which makes me think now that we actually should take the one is provided in angular.json 🤔), so it's related to what is written there. You can add a custom tsconfig.json in the .storybook dir, extend it from whatever tsconfig.json you want and override the exclude/include fields. Here is an example in the our testing app.

Issue1: when would this be released?
Issue2: I ve node_modules excluded from tsconfig.json, yet i get issues npm run storybook

when would this be released?

Soon. CC @shilman.

Issue2

what are the errors now ?

i added a custom tsConfig.json in  .storybook
{
  "extends": "../tsconfig.json",
  "include": [
    "**/*.ts"
  ]
}

and i get this error .
WARNING in /Users/namaumaa/repos/tumor-board/tumor-board-microservice/tumor-board-ui/.storybook/tsconfig.json
[tsl] ERROR
      TS18003: No inputs were found in config file 'tsconfig.json'. Specified 'include' paths were '["**/**/*.ts"]' and 'exclude' paths were '["../node_modules","../tmp"]'.
 @ ./libs sync .stories.ts$ ./dia-angular/navify-ui/stories/template/topnav.stories.ts
 @ ./.storybook/config.js
 @ multi<EMAIL_ADDRESS><EMAIL_ADDRESS>./.storybook/config.js ./node_modules/webpack-hot-middleware/client.js?reload=true ./apps/tumor-board/src/styles.scss

How does your file structure look like?

I was able to solve it ( by adding dedicated include paths to app & libs)  .. thanks..
i was trying to set up SASS loader as none of the assets were getting resolved
Added this to the webpack.config.js
const path = require("path");

module.exports = {
  module: {
    rules: [
      {
        test: /\.scss$/,
        loaders: ["style-loader", "css-loader", "sass-loader"],
        include: path.resolve(__dirname, "../")
      }
    ]
  }
};

i get these errors ::

ERROR in ./libs/***i/stories/helper/mixin-style/mixin-style.component.scss
Module build failed (from ./node_modules/sass-loader/lib/loader.js):

$mixin-indent : 1.3rem;

Don't you use angular-cli? sass should be supported by default

so .. i used default angular.json and my default project's config looks like this
(the build & serve json)
i am was assuming, npm run storybook will pick the default projects cli info but when i read ur above comment , i though i will need to add a custom webpack.config.js will scss-loader hence i did something above ^ ..
but no luck ,,
Basically, I want the npm run to resolve styles and assets without having to build the styles lib & move them to node_modules.
angular.json : Build/serve
"build": {
          "builder": "@angular-devkit/build-angular:browser",
          "options": {
            "outputPath": ".output/dist/apps/tumor-board",
            "index": "apps/tumor-board/src/index.html",
            "main": "apps/tumor-board/src/main.ts",
            "polyfills": "apps/tumor-board/src/polyfills.ts",
            "tsConfig": "apps/tumor-board/tsconfig.app.json",
            "assets": [
              "apps/tumor-board/src/favicon.ico",
              "apps/tumor-board/src/assets"
            ],
            "styles": [
              "apps/tumor-board/src/styles.scss"
            ],
            "scripts": [],
            "stylePreprocessorOptions": {
             "includePaths": [
               "libs/dia-angular/navify-ui/src/lib/scss"
             ],
             "importer": ["node-sass-import-once"],
             "importOnce": {
               "css": true
             }
           },
"serve": {
          "builder": "@angular-devkit/build-angular:dev-server",
          "options": {
            "browserTarget": "tumor-board:build",
            "stylePreprocessorOptions": {
             "includePaths": [
               "libs/dia-angular/navify-ui/src/lib/scss"
             ],
             "importer": ["node-sass-import-once"],
             "importOnce": {
               "css": true
             }
           }
          },
          "configurations": {
            "production": {
              "browserTarget": "tumor-board:build:production"
            }
          }
        }

@arvindnama, did you manage to solve the problem? Or you can share a reproduction repo?


Issue using this with raw JS and no build step

It would be really great to get an example of using this using raw JS with no esbuild step. My code works in Node 20, but fails if attempting to use this layer on the 2023 image that has no language runtime, even with the arm64 code added as a layer.
arn arn:aws:lambda:us-east-1:667977005188:function:sandbox-pnc-gateway-nodejs
Just trying to see if NodeJS can operate even close to where Python 3.11 image is operating

Hi @LewisCowlesMotive. I'm not following, are you trying to use sandbox-pnc-gateway-nodejs layer and run with LLRT?

Hi Richard, Not quite, although it does run under nodeJS. If I switch the runtime and have used the arm64 zip (it's an arm64 lambda) nodejs code doesn't "just run"; so I'm wondering if there might be lower-level steps to use this with say a hello-world or one of the standard AWS examples that talk to dynamoDB, or other network services.
The key thing is. Given some JS that works. Without esbuild or typescript. What does someone need to do, to test if this has any performance impact at all?

Hi Richard, Not quite, although it does run under nodeJS. If I switch the runtime to the 2023 amazon that does not have nodeJS, and have used the arm64 zip (it's an arm64 lambda) nodejs code doesn't "just run"; so I'm wondering if there might be lower-level steps to use this with say a hello-world or one of the standard AWS examples that talk to dynamoDB, or other network services.
The key thing is. Given some JS that works. Without esbuild or typescript. What does someone need to do, to test if this has any performance impact at all?

Ah I see. I would recommend to checkout our SAM examples. That should give you a pretty good idea what to do. In summary:

Download LLRT for your target platform llrt-lambda-arm64.zip
Add your source code together with the bootstrap file from the zip above.
Upload both and use Amazon Linux 2023 inside Lambda.

It's the bootstrap from the zip above I'm missing. we're doing the other stuff. So Bootstrap will take our code and launch it via the different runtime?

Kind of, I recommend reading here:
https://docs.aws.amazon.com/lambda/latest/dg/runtimes-custom.html


Add supported runtimes section and Cloudflare import note

This pull request adds two pieces of information to the Remix Auth README:

A "Supported runtimes" section that clearly shows which environments Remix Auth supports. This addition helps users quickly determine if Remix Auth is compatible with their deployment environment.
A note about importing from @remix-run/cloudflare when deploying to Cloudflare. This information is crucial for users deploying to Cloudflare, as it prevents potential import errors.

Latest version has no dependencies so it works on any runtime that supports Request object


Server cannot be reached

When I opened the playground from the content sidebar, facing issues with the server cannot be reached.
Error is:
{
  "error": "Response not successful: Received status code 400"
}

Please help ASAP

Can you check if the API key you defined for the app, has access to the environment you try to query?

Yes. I checked. Please find attached a screenshot.
On Thu, Jul 1, 2021 at 5:05 PM Stefan Judis @.***>
wrote:

Closed #3
https://github.com/stefanjudis/contentful-graphql-playground-app/issues/3
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stefanjudis/contentful-graphql-playground-app/issues/3#event-4965395777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFVB5J4XHYP4MIMY3BWVBTTVRHJRANCNFSM47UGONWA
.

There's no screenshot though. :)

Alright, please provide more information, then. It's hard to help without context. :)
Useful information: the query, the selected environment, and the potential alias.
In the meantime: you can always use Contentful's default GraphiQL interface to write your GraphQL queries.

Alright, please provide more information, then. It's hard to help without context. :)
Useful information: the query, the selected environment, and the potential alias.
In the meantime: you can always use Contentful's default GraphiQL interface to write your GraphQL queries and not be blocked. :)

I'm confused. Does it work now?

I'm confused. Does it work now?

Nop... Just trying to add a screenshot which shows query and environment details

Okay, then let's not close the issue. :)
I'll have a look at it when I have a moment. In the meantime, can you try changing the URL in the playground from .../environments/develop (that's the environment) to .../environments/master and see if that resolves it?

Okay, then let's not close the issue. :)
I'll have a look at it when I have a moment. In the meantime, can you try changing the URL in the playground from .../environments/develop (that's the environment) to .../environments/master and see if that resolves it?

Already tried that but no luck.... :(

Are you 100% sure you have the correct API key stored in the Playground config?
Can you use the default GraphiQL and see if you can replicate the issue?

If the API key was wrong then it throws an error like 401 right?

I've also seen 400 for incorrect API keys.

Can you use the default GraphiQL and see if you can replicate the issue?

Can you please use the default tooling. And then compare what the playground app does differently.
And report all information in a single comment so that I can investigate? :)

Yes, in short, it returns 4XX if the key is invalid, not 5XX.
I already tried it using Postman and got the exact same response, I will install graphiql and share the complete issue in one comment

Thank you. :) Let's figure that one out. 💪

Issue explanation:
I have two environments - develop and master and a content model with few basic fields and one reference field. The required API access token has been generated with the permission of CDA and CPA APIs, and both the env.
GraphQL API throws an 500 ('internal server error') while trying with the following query (the same can be found in the attached screenshot)
query gtiStoreEntryQuery {   gtiStore(id: "<id>") {     items{             name     }   } }
Error:  Server cannot be reached
{     "errors": [         {             "message": "Union type PageContent can only include type PageLanding once."         }     ] }

What I have tried so far:

reEnabled the API key to have access to all environments, In fact, I have two environments only.
Tried API for both the environments
Tried API in different tools like Contentful's integrated graphql playground, Postman, graphiql and cURL

@zeelmavani Thanks for the detailed explanation.
I think this is now a different problem, and not a problem with the Contentful GraphQL Playground app. The app makes the request accordingly and shows the correct API response.
It's then rather a problem with the GraphQL API and/or the content you want to access.
Depending on your pricing plan, please reach out to support (contentful.com/support) and/or maybe join the community slack (contentful.com/slack) to seek help from the community.


Merge aes.hpp into aes.h

any updates on this? it seems a very simple review

Actually, I think the include statements should also be corrected to use cstdint and cstddef inside the C++ guard with an #else branch including the original .h files:
#ifdef __cplusplus
#include <cstdint>
#include <cstddef>
extern "C" {
#else
#include <stdint.h>
#include <stddef.h>
#endif


PR: Extend Find in Files to select other search directories

Fixes #4814
Preview

@ccordoba12 I have some problems choosing a directory using qtpy.compat.getexistingdirectory. When the folder selection window is closed, it keeps reappearing instead of closing, this happens for both Cancel and Accept options. After that, Spyder cannot be closed unless is killed with a SIGKILL signal.

@ccordoba12 Does it look right?

I like it, a lot!
@jnsebgosselin, @avylove, what do you think?

Questions:

How many paths are we saving in the drop-down menu?
Are those paths saved among sessions?

In point 2. I meant, are those paths saved when Spyder is closed and open again?

On a single session we're storing all paths as the user would like, however, between sessions, we're just storing the 15 most recent entries.

Looks pretty good @andfoy :+1:

On a single session we're storing all paths as the user would like

That could cause problems. Please limit that to the 15 most recent paths too.

@andfoy do we have a 'search in current project' option ?

@goanpeca Yes, I forgot to show it on the preview GIF. The option is only enabled when a project is opened

@andfoy This looks great and very intuitive. I like it. Good job.

@andfoy Looks great! Good solution!

@andfoy, this requires now a merge with 3.x to fix conflicts.

@ccordoba12 I think this one is ready


resolving link inside material-ui tooltip

This would possibly resolve the issue #897
Detailed explanation is explained at my comment in the ticket, https://github.com/VEuPathDB/web-eda/issues/897#issuecomment-1050119406
I expected that Dave would make this kind of PR with my suggestion but he might have been hectic so I made this instead 😄

This is also related to #905


Add a hostedZoneName field to Cloud DNS

What this PR does / why we need it:
Analog to Route53 and azure this adds a zone field for DNS01 validation to help people with specific zone setups that our normal SOA probe cannot resolve.
Which issue this PR fixes:
can be used as a workaround in #2968
Special notes for your reviewer:
Release note:

Add a hostedZoneName field to Cloud DNS

tested and it works as expected :smile:

/hold
working to add this into v1beta1

/unhold

Thanks for the review @wallrj
Implemented the feedback!

/kind feature

/retest


Mohave County, AZ, USA: updated address data url + conform

If it's easy to find, can you add website too?


GUACAMOLE-1159 / GUACAMOLE-1160 : Add/Update French translations.

Hi,
I have added/updated French translations for both the application and the extensions. I have used the English translation as the base.
The associated JIRA cards were already existing, so I referenced them.
Regards

This PR needs to be limited to a single JIRA issue. I've closed 1159 as I believe its scope is completely contained by 1160, so let's just go with GUACAMOLE-1160 for now.


interactive visualization

Dear,
Could scanpy provide some interactive visulaization tools ? So that non-bioinformaticans can also look up the results outuputed by scanpy, just like the following link:
https://scDissector.org/martin

@Marius1311 might be able to tell you some more about his interactive plotting library. This is however more targeted at interactive plotting in notebooks rather than servers. Cellxgene from CZI is also worth taking a look at for communicating results to non-bioninformaticians.

@wangjiawen2013 feel free to check out our interactive plotting package for scanpy mentioned by @LuckyMD above (https://github.com/theislab/interactive_plotting) and let me know if you have any questions.

There is already a discussion at https://github.com/theislab/scanpy/issues/253. I think we can close this issue.


Unable to load mopar component on Home Assistant 103

Home Assistant release with the issue:
Home Assistant 0.103.0
Last working Home Assistant release (if known):
None
Operating environment (Hass.io/Docker/Windows/etc.):
Running in Docker on Synology
Integration:
Have mopar uConnect account with password and pin -
NOTE: works through Alexa
Description of problem:
When rebooting home assistant I have the config added to configuration.yaml
mopar:
username...
I am seeing the following
2019-12-11 16:22:57 ERROR (MainThread) [homeassistant.setup] Error during setup of component mopar | stderr
simplejson.errors.JSONDecodeError: Expecting value: line 2 column 1 (char 1) | stderr

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

@jlvandusen could you please add the entire log to this issue? Where does this error occur e.g. in which line?

Any update on this?  I see the same issue... got it working twice, but getting same type of errors.

@springstan
Here is what is produced in my logs as well...
2020-02-03 11:56:40 ERROR (MainThread) [homeassistant.setup] Error during setup of component mopar Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/setup.py", line 174, in _async_setup_component     component.setup, hass, processed_config  # type: ignore   File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/src/homeassistant/homeassistant/components/mopar/__init__.py", line 68, in setup     data.update(now=None)   File "/usr/src/homeassistant/homeassistant/components/mopar/__init__.py", line 113, in update     self.tow_guides[index] = motorparts.get_tow_guide(self._session, index)   File "/usr/local/lib/python3.7/site-packages/motorparts/__init__.py", line 92, in wrapped     return function(*args)   File "/usr/local/lib/python3.7/site-packages/motorparts/__init__.py", line 106, in wrapped     return function(session, *args)   File "/usr/local/lib/python3.7/site-packages/motorparts/__init__.py", line 164, in get_tow_guide     'vin': profile['vehicles'][vehicle_index]['vin']   File "/usr/local/lib/python3.7/site-packages/requests/models.py", line 897, in json     return complexjson.loads(self.text, **kwargs)   File "/usr/local/lib/python3.7/site-packages/simplejson/__init__.py", line 525, in loads     return _default_decoder.decode(s)   File "/usr/local/lib/python3.7/site-packages/simplejson/decoder.py", line 370, in decode     obj, end = self.raw_decode(s)   File "/usr/local/lib/python3.7/site-packages/simplejson/decoder.py", line 400, in raw_decode     return self.scan_once(s, idx=_w(s, idx).end()) simplejson.errors.JSONDecodeError: Expecting value: line 2 column 1 (char 1)

I believe its expecting the index value now...maybe?

I believe its expecting the index value now...maybe?

Not sure what is causing this issue, but it seems like this is a reoccurring problem with this library. See https://github.com/happyleavesaoc/python-motorparts/issues/1 and #16379.
I would recommend to open a new issue in python-motorparts since this needs to be fixed in the library instead of HA.

Opened issue as suggested: https://github.com/happyleavesaoc/python-motorparts/issues/4

Issue is located here: https://github.com/happyleavesaoc/python-motorparts/issues/4

No updates?

Resolved for me with 105.5.

What about the others?

Testing now on 105.5 I had it commented.

Still erroring when loading mopar
Wed Feb 19 2020 05:30:03 GMT-0500 (Eastern Standard Time) Error during setup of component mopar Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/setup.py", line 174, in _async_setup_component     component.setup, hass, processed_config  # type: ignore   File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/src/homeassistant/homeassistant/components/mopar/__init__.py", line 68, in setup     data.update(now=None)   File "/usr/src/homeassistant/homeassistant/components/mopar/__init__.py", line 113, in update     self.tow_guides[index] = motorparts.get_tow_guide(self._session, index)   File "/usr/local/lib/python3.7/site-packages/motorparts/__init__.py", line 92, in wrapped     return function(*args)   File "/usr/local/lib/python3.7/site-packages/motorparts/__init__.py", line 106, in wrapped     return function(session, *args)   File "/usr/local/lib/python3.7/site-packages/motorparts/__init__.py", line 164, in get_tow_guide     'vin': profile['vehicles'][vehicle_index]['vin']   File "/usr/local/lib/python3.7/site-packages/requests/models.py", line 897, in json     return complexjson.loads(self.text, **kwargs)   File "/usr/local/lib/python3.7/site-packages/simplejson/__init__.py", line 525, in loads     return _default_decoder.decode(s)   File "/usr/local/lib/python3.7/site-packages/simplejson/decoder.py", line 370, in decode     obj, end = self.raw_decode(s)   File "/usr/local/lib/python3.7/site-packages/simplejson/decoder.py", line 400, in raw_decode     return self.scan_once(s, idx=_w(s, idx).end()) simplejson.errors.JSONDecodeError: Expecting value: line 2 column 1 (char 1) Connection lost. Reconnecting…

The Mopar integration is actually violating our architectural design records (ADR-0004), since it uses web scraping.
This means the integration needs to be updated or should be removed. However, it currently doesn't have a code owner.

What to do when services like this offer no options other than web scraping? :(


Support for Fastify

Thanks @rqbazan this looks good, I'll be able to take a closer look and get this released soon.


Add VTS priorty to counties

#662

[x] added an csv importer in admin tools for county vts data (csv to use for testing: https://docs.google.com/spreadsheets/d/1vFjVRD_m0URWlrgDJ5NG7RXPA-xYcG5vWRnCRN9_giE/edit#gid=0)

[x] added county vts priority to location admin and list view with ordering

[x] added county vts priority to county summary in report admin

As for the mypy error, I'm not sure why its erroring (the display decorator is imported correctly?) - @simonw should I ignore it or do you have more insight 🤔

I'm going to merge this and test things on staging!

One thing to note, is that I had to edit the original spreadsheet to prepend zeros to the FIPs code, which excel/sheets does not treat as significant automatically. I'll add a note about this in the admin panel in a separate PR.
I was able to upload all 3142 counties provided's codes. 🥂


Slider without gesture

Describe the bug
In the home page there is a slider to show images of the court, you can navigae in them thank to arrows (next/prev) or with pagination bullet, BUT NOT with gesture for mobile
Expected behavior
Add gesture for mobile

Fixing with add gesture for any devices


Add simulation tests for app Import-Export

The simulation app has different commands.
We currently have 1 simulation test for AppStateDeterminism
The goal of this issue is to add test for AppSimulationAfterImport/AppImportExport or both.
This will allow us to detect any problems that we might run into if at some point we need to export our app state and start a new chain with the exported genesis.
Also include snapshots for wasmd (the last time(3-4months ago) we checked, snapshots weren't working as expected, but it wasn't wasmd specific problem)

Done in 57
Created issue to track the creation of github workflow for this newly added simulation test (issue)


[Pulsar Scheduler] Fix Endless Reschedule Bug

This changes the Pulsar scheduler behaviour so that we only delete jobs from the in memory DB once a terminal state message has been received.  Previously we deleted them as soon as we had generated a terminal message, but this lead to cases where an update was received in the interim and we treated the job as completely new.

Codecov Report
Base: 47.40% // Head: 91.87% // Increases project coverage by +44.47% :tada:

Coverage data is based on head (8e99859) compared to base (6c19460).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #2097       +/-   ##
===========================================
+ Coverage   47.40%   91.87%   +44.47%
===========================================
  Files         234       18      -216
  Lines       31224      665    -30559
===========================================
- Hits        14801      611    -14190
+ Misses      15323       54    -15269
+ Partials     1100        0     -1100

Flag
Coverage Δ

airflow-operator
89.13% <ø> (ø)

armada-server
?

python-client
93.83% <ø> (ø)

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

Impacted Files
Coverage Δ

internal/scheduler/scheduler.go

internal/executor/domain/resources.go

internal/armada/server/submit_from_log.go

internal/lookoutv2/repository/getjobrunerror.go

...nternal/scheduler/schedulerobjects/requirements.go

internal/jobservice/repository/sql_job_service.go

internal/armada/repository/job.go

...ernal/executor/podchecks/container_state_checks.go

internal/scheduler/scheduling_algo.go

...nternal/scheduler/schedulerobjects/reporting.pb.go

... and 206 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Trigger

Closes #80

@tchiotludo, the rockset test is not working I don't know why, but the PR is ready to me reviewed.
There is a lot of changes as I had to refactor things to share the conf between Query and Trigger, it should be easier to review commit by commit.


Update to Rust master

The usual, won't pass Travis until the next nightly is cut, but DList::append takes a &mut DList now.

Oh whoops, I actually meant to patch this method, but kept getting distracted. It's actually just incorrect.
See this branch for fixes: https://github.com/Gankro/collect-rs/compare/blist (I was planning to rebase out the first commit, but spent the past few days reinstalling my OS).

OK, cool.  I'll just close this then so you can land yours :D
Let me know if you want me to do something else instead.


I want to use google art and culture art selfie api

I want to use google art historic selfie API match who looks like you in china, I don't have any commercial or never earn money by this action, Just want to this technology to share china, Think about people will like it.

Hello,
This repo is related to the Art-Palette experiment from Google Arts & Culture.
The Art Selfie App is using different technics and is not related to Art Palette.
Thanks for your interest!

Hi - I have been using this to match images to a target palette. What should the distance be for an image with itself? I get 100, yet i do find images that have closer distances? Cheers!

Hello,
The embedding is using machine learning to speed up the distance
calculation.
If you want the exact distance you can calculate the exact LAB colors
https://en.wikipedia.org/wiki/CIELAB_color_space distance : first order
the colors in the palette vectors and then calculate the distance between
each ordered vectors.
If you have lot of elements to compare and want a fast response you can use
the python code to calculate the distance
https://github.com/googleartsculture/art-palette/blob/master/palette-embedding/python/palette_embedding.py#L257
and
get the nearest neighbors.
Simon
Le lun. 6 juil. 2020 à 18:47, zamia-git<EMAIL_ADDRESS>a écrit :

Hi - I have been using this to match images to a target palette. What
should the distance be for an image with itself? I get 100, yet i do find
images that have closer distances? Cheers!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/googleartsculture/art-palette/issues/1#issuecomment-654349298,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACJXADQ235DIYLEA54KPV3R2H53ZANCNFSM4JSJWMFQ
.

--
Simon Doury
http://www.simondoury.com

ah - perfect - cheers!


Splashes and effects on sustains.

What feature do you want to get added on the base engine?
You know how in the newest FNF charting teaser there is an effect for holding down sustains and an extra little splash at the end of the hold? You can probably do this with Lua, but I feel like this should be a base feature. You could also add getting extra points for timing your release of the sustain or sum', idk.
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

No, I'm not going to copy stuff from teasers, that's disrespectful to the devs.

Fair enough ig.


feat(course): create shared types lib for firebase data

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING and other relevant documents.

Description

Added a shared lib for all types which will be defined in the firebase data structure and used in the frontend.

Yeah, you're right. My fault 🦖


fix: remove build path from git url

As the title.
Test Plan
internal

Current dependencies on/for this PR:

main

PR #14   (this PR)

This comment was auto-generated by Graphite.

I think it's fine to temporarily remove this, but I do think we should be able to fix the functionality with a tweak - commented on linear task with details.


[Autosuggest] Client context has incorrect base URI, so requests fail

Package Name: @azure/cognitiveservices-autosuggest
Package Version: 6.9.0
Operating system: Windows 10 Enterprise 1803
[x ] nodejs

version: 10.16.0

[x] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
AutoSuggestClientContext (https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/cognitiveservices/cognitiveservices-autosuggest/src/autoSuggestClientContext.ts) does not set the base URI correctly (see line 42), so calls to the service fail.
This is the same issue as https://github.com/Azure/azure-sdk-for-js/issues/4160.
To Reproduce
Steps to reproduce the behavior:

Set environment variables AUTOSUGGEST_SUBSCRIPTION_KEY and AUTOSUGGEST_ENDPOINT to a valid Autosuggest subscription key and the corresponding endpoint, respectively.
Install @azure/ms-rest-js and @azure/cognitiveservices-luis-authoring.
Run the following code with Node.

"use strict";

/* To run this sample, install the following modules.
 * npm install @azure/cognitiveservices-autosuggest
 * npm install @azure/ms-rest-js
 */
var Autosuggest = require("@azure/cognitiveservices-autosuggest");
var msRest = require("@azure/ms-rest-js");

const key_var = 'AUTOSUGGEST_SUBSCRIPTION_KEY';
if (!process.env[key_var]) {
    throw new Error('please set/export the following environment variable: ' + key_var);
}
const subscription_key = process.env[key_var];

const endpoint_var = 'AUTOSUGGEST_ENDPOINT';
if (!process.env[endpoint_var]) {
    throw new Error('please set/export the following environment variable: ' + endpoint_var);
}
const endpoint = process.env[endpoint_var];

const creds = new msRest.ApiKeyCredentials({ inHeader: { 'Ocp-Apim-Subscription-Key': subscription_key } });
const client = new Autosuggest.AutoSuggestClient(creds, { endpoint: endpoint + "/bing/v7.0/Suggestions" });

async function quickstart() {
    var result = await client.autoSuggest("xb");

    var groups = result.suggestionGroups;
    if (groups.length > 0) {
        var group = groups[0];
        console.log("First suggestion group: " + group.name);
        var suggestions = group.searchSuggestions;
        if (suggestions.length > 0) {
            console.log("First suggestion:");
            console.log("Query: " + suggestions[0].query);
            console.log("Display text: " + suggestions[0].displayText);
        }
        else {
            console.log("No suggestions found in this group.");
        }
    }
    else {
        console.log("No suggestions found.");
    }
}

try {
    quickstart();
}
catch (error) {
    console.log(error);
}

Output:
(node:1292) UnhandledPromiseRejectionWarning: Error: Resource not found
    at new RestError (C:\Users\<user>\node_modules\@azure\cognitiveservices-autosuggest\node_modules\@azure\ms-rest-js\dist\msRest.node.js:1399:28)
    at C:\Users\<user>\node_modules\@azure\cognitiveservices-autosuggest\node_modules\@azure\ms-rest-js\dist\msRest.node.js:2494:37
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:1292) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)
(node:1292) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Expected behavior
Expected output:
First suggestion group: Web
First suggestion:
Query: xbox
Display text: xbox

Additional context
I am able to resolve the issue and produce the expected output by making the following change to my local copy of the autosuggest package.
<EMAIL_ADDRESS>Change line 872 from:
_this.baseUri = "{Endpoint}";

to:
_this.baseUri = "{Endpoint}/bing/v7.0";

Hi @v-jaswel  thanks for sending the bug over. I've tagged this to get it routed to the right team.

The library here needs to be re-generated using the latest code generator which should fix the problem with the baseuri

Hi @ramya-rao-a, I have tried the 2.0 version but getting a "Resource not found" error. The endpoint seems to be correct through most of the msrest library, but somehow fails in there somewhere.
It's the autoSuggest() API call, the result comes back as null. So far this issue has been found with the Java and the Python SDK too.
var result = await client.autoSuggest("xb", null);
(node:6532) UnhandledPromiseRejectionWarning: Error: Resource not found
    at new RestError (C:\Users\wazure\Desktop\Autosuggest_Node\node_modules\@azure\ms-rest-js\dist\msRest.node.js:2400:28)
    at C:\Users\wazure\Desktop\Autosuggest_Node\node_modules\@azure\ms-rest-js\dist\msRest.node.js:3544:37
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async quickstart (C:\Users\wazure\Desktop\Autosuggest_Node\auto_1.js:24:18)
(node:6532) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)
(node:6532) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Thanks @wiazur, Can you please log a new issue and link the corresponding Java and Python SDK issues if any?


Request to add an UsageCategory to page 5096 "Task List"

Hi,
There is no Task overview page which can be found via search. Hereby a request to add a UsageCategory to page 5096 "Task List" so it can be found via search as well.
Thanks in advance.
Regards, Don

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52844.


is linux sqlite.so also work?

Does this library work on Linux?

It ought to, although I have not tested it. Paths to the .so would need to
be adjusted, of course.
On Sun, 8 Sept 2024 at 00:29, Sec Wang @.***> wrote:

Does this library work on Linux?
—
Reply to this email directly, view it on GitHub
https://github.com/xpqz/dyalog-sqlite3/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUQ6JDHU6XI6AAUYS57FUTZVOD6VAVCNFSM6AAAAABN2NNDD2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYTEMJRGUYTOMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Stefan Kruger @.***>


Support serializing and sending complex data to the server

Currently it's not supported to send messages like this:
client.Send(new { message = "something" })

Currently, the client only supports sending simple data structures and lists of simple data structures.


Troubles about creating hulls

Hello,
I've translated the functions in geometryutil.js into C# for a course project. The problem is that sometimes the polygonHull function may lose several parts in the process. I've been captured that overlapping points with the one on the upper half of A and the other is in the lower cause the problem. Still, there are lots of problems cases I cannot handle.
I wonder if there is another feasible and reliable mean to create the hull of the parts?
thx in advance

ah, sorry about that. That function was used in a previous version but I haven't gotten around to removing it. You might notice that it's not actually referenced anywhere else in the code.
there are a lot of edge cases that this function fails to handle, unfortunately. I don't know of any robust implementations for this problem, but I think a better way to approach it would be to first get the convex hull, then iteratively adding points on the convex edges.

I've noticed that there is a clipper class library in the util file. Is the SVGnest using this library to connect the NFPs to iteratively placing parts?

yes the boolean operations are done with jsclipper

it seems that double coordinates have to be trasferred into int before being processed in jsclipper?


Any guideline to set I18N for front-end?

Hi,
Right now, I know that set the web site language to be Chinese via config.i18n.default_locale = 'zh-CN' at the file /config/application.rb after I install kms into my rails app. But I cannot find any guideline for the front-end. Is there anyone can help?
Thanks.

OK, I read the code, and found via config.i18n.default_locale = 'zh-CN', it works for front-end.
Issue closed.

Hello @AbelLai.
Yes, actually you're responsible for frontend, there's no translations approach there

It should work. Did you do assets precompilation for production?

Yes, I did assets precompilation for production environment.

Hm, then I don't have suggestions, need more information. Maybe you could share your entire project?


Support optional custom function when See More pressed

Hello,
I am using this component in a project and it works perfectly!
There is one use case I needed and had to use patch-package to add for my project, and that is the ability to use a custom function when "See More" is clicked.
For example, when "See More" is clicked, instead of expanding text, open up a modal.
Here is an example video of this functionality.
Here is the patch that I applied to achieve this.
I can open up a PR if you believe this is functionality that would improve the package.
Thanks.

Hi @ChrisLFieldsII I like the idea. Also I reviewed the patch file and the video. Looks like a nice addition we can have.
I'd just say the name onToggle isn't a best fit. It should be something like blockedToggle or onSeeMoreBlocked or something which will indicate the purpose in a better way.
Having said that a PR would be highly appreciated!!

@ChrisLFieldsII wonderful.
Once you're about to raise a PR, just make sure to update these:
1_ In package.json update version 2.3.5 to 2.3.6
2_ Update the newly added prop in readme.md
Thanks,

Thanks @ChrisLFieldsII , the feature is published. Ver 2.3.6

Awesome! @
Thanks for the package, saved us quite a bit of time @fawaz-ahmed


[docs] fix 404 url

Describe the pull request

What does your PR fix?
Fixes docs packaging-zipfiles.md

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

Does your PR follow the maintainer guide?

Yes / I am still working on this PR

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Close and reopen this PR to trigger CLA check.


Added 'enabled' function

This will allow users of the library to quickly check if cookies are enabled on the client without having to maintain an external function to do it for them.
$.cookie.enabled //returns a boolean to check

A boolean value check was also added to the unit tests

Travis CI build failed due to a failure in loading saucelabs

Thanks for the PR. navigator.cookieEnabled isn't implemented cross-browser though, thus not going to merge.


Display error message for bad password

Part of #570: Ensures that error message is displayed when a user enters bad credentials on email login. @frankwiles or @gregnewman wanted to chat with you about this before merging -- not sure if it's the right approach, moving the errors to the messages framework?
(still an issue: the error message loads, but when the page reloads, it's back to the social login screen)

@williln in my opinion I feel like this should show the error outside of the messages framework and placed close to the form.

@gregnewman  Yeah, I agree. I'll continue debugging why it's not showing up like it should in the template.


bugfix: parameter misspelling

Traceback (most recent call last):
File "examples/multi_instance.py", line 192, in
num_episodes=args.num_episodes,
AttributeError: 'Namespace' object has no attribute 'num_episodes'

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

@CrabAss Thank you for your contribution!

Cherry-picked into develop with https://github.com/huawei-noah/SMARTS/commit/778271a80a771d7dd7867710d7e6b0b417be25b7 to avoid issues with changing the base.


One time Linux didn't add to Active Directory

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
Run terraform-azurerm-vnet-app
Any log messages given by the failure
Created symlink /etc/systemd/system/multi-user.target.wants/smbd.service → /lib/systemd/system/smbd.service.
samba-ad-dc.service is a disabled or a static unit, not starting it.
Processing triggers for ufw (0.36.1-4ubuntu0.1) ...
Processing triggers for man-db (2.10.2-1) ...
Processing triggers for dbus (1.12.20-2ubuntu4.1) ...
Processing triggers for libc-bin (2.35-0ubuntu3.1) ...
NEEDRESTART-VER: 3.5
NEEDRESTART-KCUR: 5.15.0-1041-azure
NEEDRESTART-KEXP: 5.15.0-1041-azure
NEEDRESTART-KSTA: 1
NEEDRESTART-SVC: packagekit.service
NEEDRESTART-SVC: unattended-upgrades.service
Cloud-init v. 23.2.2-0ubuntu0~22.04.1 running 'modules:final' at Tue, 22 Aug 2023 00:35:27 +0000. Up 428.54 seconds.
20230822T0035297908Z : NuGet Powershell Package Provider version <IP_ADDRESS> is already installed...
20230822T0035309337Z : Setting PSGallery installation policy to 'Trusted'...
20230822T0035322430Z : Installing PowerShell Az module...
20230822T0036109823Z : PowerShell Az.Automation version 1.9.1 is installed...
/var/lib/cloud/instance/scripts/configure-vm-jumpbox-linux.sh: line 30: $log_fileprintdiv: ambiguous redirect
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
tkey query failed: GSSAPI error: Major = No credentials were supplied, or the credentials were unavailable or inaccessible, Minor = SPNEGO cannot find mechanisms to negotiate.
id<EMAIL_ADDRESS>no such user
Cloud-init v. 23.2.2-0ubuntu0~22.04.1 finished at Tue, 22 Aug 2023 00:36:15 +0000. Datasource DataSourceAzure [seed=/dev/sr0].  Up 476.86 seconds
Cloud-init v. 23.2.2-0ubuntu0~22.04.1 running 'init-local' at Tue, 22 Aug 2023 01:35:07 +0000. Up 7.05 seconds.
Cloud-init v. 23.2.2-0ubuntu0~22.04.1 running 'init' at Tue, 22 Aug 2023 01:35:09 +0000. Up 9.74 seconds.
ci-info: ++++++++++++++++++++++++++++++++++++++Net device info+++++++++++++++++++++++++++++++++++++++
ci-info: +--------+------+-----------------------------+---------------+--------+-------------------+
ci-info: | Device |  Up  |           Address           |      Mask     | Scope  |     Hw-Address    |
ci-info: +--------+------+-----------------------------+---------------+--------+-------------------+
ci-info: |  eth0  | True |           <IP_ADDRESS>          | <IP_ADDRESS> | global | 00:0d:3a:8d:34:c6 |
ci-info: |  eth0  | True | fe80::20d:3aff:fe8d:34c6/64 |       .       |  link  | 00:0d:3a:8d:34:c6 |
ci-info: |   lo   | True |          <IP_ADDRESS>          |   <IP_ADDRESS>   |  host  |         .         |
ci-info: |   lo   | True |           ::1/128           |       .       |  host  |         .         |
ci-info: +--------+------+-----------------------------+---------------+--------+-------------------+
Expected/desired behavior
to see jumplinux1 in the computers container in the mysandbox.local active directory.
OS and Version?

Linux Ubuntu 22.04

Versions
Mention any other details that might be useful
I destroyed the environment and re-deployed and it worked. Just wanted to report the issue.

@esaadat thanks for reporting the issue you encountered. What release of #AzureSandbox were you using when you provisioned vnet-app? There were several bugs associated with upgrading to Ubuntu 22.04 that were fixed in subsequent releases. The most recent release is v2.6.5.

@esaadat , regarding provider registration, see #4440. I'll add this to known issues in case anybody else hits it. Thanks for calling this out. Regarding the other issue (jumplinux1 not showing up in AD) I'm glad a redeploy worked for you. If it happens again please open a new issue and keep the existing environment in place so we can pull some logs etc.

Closing this issue due to inability to reproduce failure.

Reopening this issue. This is a transient failure and I have a repro on v2.6.5 of Azure Sandbox. The following errors were in /var/log/cloud-init-output.log and were generated during execution of configure-vm-jumpbox-linux.sh
tkey query failed: GSSAPI error: Major = No credentials were supplied, or the credentials were unavailable or inaccessible, Minor = SPNEGO cannot find mechanisms to  negotiate.
id<EMAIL_ADDRESS>no such user

As a temporary mitigation to avoid having to redeploy use the following steps:

Use Bastion to ssh into jumplinux1 as bootstrapadmin using the bootstrapadmin-ssh-key-private key vault secret
Re-run the jumplinux1 configuration script using the following commands:

git clone https://github.com/Azure-Samples/azuresandbox
cd azuresandbox/terraform-azurerm-vnet-app
chmod 755 configure-vm-jumpbox-linux.sh
sudo ./configure-vm-jumpbox-linux.sh

Closing this as duplicate of issue [2}(https://github.com/Azure-Samples/azuresandbox/issues/2)


refactor(focus-lock): separate logic of FocusLock component

Closes #3353 (Separation of UI and Logic in the FocusLock Component)
Description
Separate the UI and logic of FocusLock component
Current behavior (updates)

New behavior

Is this a breaking change (Yes/No):
No
Additional Information

You don't need to merge the main branch. There are too many changed files.

I've changed the main branch to v2.0, what should I do next?

@hong-nguyen13
If this PR is incorporated first, conflicts will occur in other work, so close this PR.
Please submit this PR again after the other work is completed.


Fix issue with NH3 CCS technologies becoming cheaper than non-CCS counterpart

Very small PR to change how NH3 CCS technologies in materials module are mapped
@ywpratama noticed during ScenarioMIP review process that costs for NH3 CCS technologies were becoming cheaper than the non-CCS NH3 technologies. This issue happened before in #186, due to CCS technologies being mapped to a different WEO technology than its non-CCS counterpart, as well as having different cost reduction assumptions. At the time, only energy technologies existed within the costs tool, and this new issue pertains to materials/industry technologies.
In this case, because the NH3 CCS technologies were mapped to MESSAGE technology igcc_ccs and the non-CCS counterparts were mapped to igcc, this led to issues of their costs crossing. As a fix, now the NH3 CCS and non-CCS technologies are mapped to MESSAGE energy technology igcc.
Because this issue is not ScenaripMIP-specific, I think this small fix should best be treated as a standalone PR, merged to main. Then, additional ScenarioMIP-specific changes related to the SSP cost reduction narratives for these technologies can be added in a separate branch (using the functionality added in #255).
How to review
@ywpratama and/or @macflo8: please check if issue of costs for NH3 CCS still persists
For @khaeru and/or @glatterf42 : Read the diff and note that the CI checks all pass.
PR checklist

[ ] Continuous integration checks all ✅
[x] ~Add or expand tests; coverage checks both~ ✅
[x] ~Add, expand, or update documentation.~
[x] Update doc/whatsnew.

Wrote a small script to test with the materials wrapper that calls create_cost_projections() with a pre-defined config. It generates the cost output for each SSP (+ LED) and asserts that the difference between the respective ccs and non-ccs NH3 technology is greater than 0. I can confirm that this runs without an AssertionError on this branch.
@glatterf42  Something similar could be integrated into the test suite for all CCS technologies maybe.
import ixmp
import message_ix
from message_ix_models.model.material.data_util import gen_te_projections

mp = ixmp.Platform("<platform-name>")
scen = message_ix.Scenario(mp, f"<scenario-name>", "<model-name>")

# SSP1-5
for i in range(1, 6):
    inv, fix = gen_te_projections(scen, f"SSP{i}", method="gdp")
    for tec in [i + "_NH3" for i in ["biomass", "coal", "gas", "fueloil"]]:
        non_ccs = (
            inv[inv["technology"] == tec]
            .drop(columns=["unit", "technology"])
            .set_index(["node_loc", "year_vtg"])
        )
        ccs = (
            inv[inv["technology"] == tec + "_ccs"]
            .drop(columns=["unit", "technology"])
            .set_index(["node_loc", "year_vtg"])
        )
        assert ccs.sub(non_ccs).le(0).all().all() == False
# LED scenario
inv, fix = gen_te_projections(scen, "LED", method="gdp")
for tec in [i + "_NH3" for i in ["biomass", "coal", "gas", "fueloil"]]:
    non_ccs = (
        inv[inv["technology"] == tec]
        .drop(columns=["unit", "technology"])
        .set_index(["node_loc", "year_vtg"])
    )
    ccs = (
        inv[inv["technology"] == tec + "_ccs"]
        .drop(columns=["unit", "technology"])
        .set_index(["node_loc", "year_vtg"])
    )
    assert ccs.sub(non_ccs).le(0).all().all() == False

@measrainsey —the doc/whatsnew conflict arises from the changes in #225 that I mentioned in Slack. If you rebase you can resolve the conflict by putting your added line from this PR into this section: https://github.com/iiasa/message-ix-models/blob/c5e275c87832d4c6b9de6d8cf05fc60579981b07/doc/whatsnew.rst?plain=1#L54-L65
Please ping me if that's not clear.

Thanks @khaeru ! I rebased and added my edit to doc/whatsnew in the new section -- hope that looks okay!
Regarding merging, I quickly chatted with @glatterf42 and we discussed me trying to see if I can add tests similar to the code @macflo8 showed above. I will spend a little time tomorrow to see if I can (or should) add this test -- will report back if I think this can be saved as a later PR or if I want to add it onto this PR!

hi all, i've added a short test based on @macflo8's code that checks costs for CCS technologies vs their non-CCS variants. and perhaps good thing too because turns out the test failed for the cooling module 😅 so i've also pushed a commit to change the mapping of the CCS-based cooling technologies (i can check with @adrivinca if this is okay and change the cooling technologies' inputs later on if necessary but for now don't consider this a blocker to pass this PR)
if this new test looks good to all, then please consider all done and ready on my end, and if we can move forward with merging that'd be great -- thanks!
that being said, before the latest push, the macOS python 3.13 tests were failing here:
FAILED message_ix_models/tests/model/transport/test_report.py::test_plot_simulated[stock-ldv] - genno.core.exceptions.MissingKeyError: required keys (<fuel economy:nl-m:ldv+ref>,) not defined

all tests are passing now so if everything looks good, would be great if we can move forward with merging 🙏🏻

This appears all checked, approved, and green, so I'll merge.


Iteration Plan for April 2019 [DRAFT]

This plan captures our work in April. This is a 6-week iteration. We will ship our April Update in early May. You'll notice that we list less work than usual in an iteration that is longer than usual. If you wonder what we are up to: One of the most commented issues is about WSL and led us to explore what we can do (#63155). It made us realize that with a bit of enablement in the core, we can build extensions that greatly improve the WSL experience and "a bunch of other stuff" :smiley:. It's a model comparable to what happened a year ago with the LiveShare extension. So, this month we keep the focus on those extensions and lift the curtain in early May. It's going to be cool, stay tuned.
Oh, and the Swiss part of our team needs to set a :snowman: on :fire:.
Endgame

May 6, 2019: Endgame begins
May 10, 2019: Endgame done

The endgame details for this iteration are tracked in TBD.
Plan Items
Below is a summary of the top level plan items.
Legend of annotations:

Mark
Description

:runner:
work in progress

:hand:
blocked task

:muscle:
stretch goal for this iteration

:red_circle:
missing issue reference

:large_blue_circle:
more investigation required to remove uncertainty

:black_circle:
under discussion within the team

UX

[ ] :runner: Iconography re-design https://github.com/Microsoft/vscode/issues/8017 @misolori
[ ] :runner: Explore Extensions Management UX https://github.com/Microsoft/vscode/issues/68527 @misolori @sandy081
[ ] :runner: Explore new VS Code icons https://github.com/Microsoft/vscode/issues/71827 @misolori

LSP

[ ] :runner: LSP support for call hierarchies PR #420 @dbaeumer
[ ] :runner: LSP support for progress reporting PR #261 @dbaeumer

JavaScript/TypeScript

[ ] Adopt TypeScript 3.4.x @mjbvz

Extension Contributions

[ ] Contributions to the Vetur extension https://github.com/vuejs/vetur/milestone/7 @octref

API

API finalization: query
API proposals: query

Exploration

[ ] :runner: Explore support for developing on WSL https://github.com/Microsoft/vscode/issues/63155 @alexandrudima team
[ ] :runner: Explore supporting Electron 5.0 default renderer configuration https://github.com/Microsoft/vscode/issues/68302 @bpasero @sbatten team

Deferred Items

Electron 4.0 upgrade https://github.com/Microsoft/vscode/issues/61787 @bpasero
Explore flatting folder hierarchies in the explorer https://github.com/Microsoft/vscode/issues/41627 @isidorn @joaomoreno
LSP support for type hierarchies PR #426 @dbaeumer

Really excited for the tease of WSL support 😀

Don't forget to pin when ready! @kieferrm :)

I'm hoping my old issue from 2 years ago can be closed ! https://github.com/Microsoft/vscode/issues/22663

I googled "endgame release date", 'cause I though it was a bit earlier than May 6, 2019. And it was.
Only then I understood that you did not take 4 days of iteration just because everyone could see the newest Avengers movie.... It's just an endgame for the iteration 😂

Every month I'm hoping for a line about at least exploring tree-sitter :'(

@kieferrm no plans for #50062 on horizon?

@mtxr not this iteration.

Should we create a hope(wish) list?

@NateLing excatly! :)

Can we please get a temporary solution to #68001? Even a dropdown to toggle the setting, as indicated in the comments.

Should we create a pinned issue for Features Wish List?
Developers can vote for the features they need most.
Then consider implementing it and join into the iteration plan.

The list is already there-
https://github.com/Microsoft/vscode/issues?q=is%3Aopen+is%3Aissue+label%3Afeature-request+sort%3Areactions-%2B1-desc
And everyone can vote!

No plans for #45385 ? It would help make autocomplete much more robust, while extending possibilities for extension devs. @kieferrm , @mjbvz , anyone?

What is the status of WSL supporting? The task is still not checked, so will it come to the release of this month? (Anxiously waiting 😟)

@MichaelTong seems your wish was granted: https://code.visualstudio.com/blogs/2019/05/02/remote-development

It exists, but hasn't been linked yet (there's actually a link to it just a few comments above)
https://github.com/microsoft/vscode/issues/73175

How's endgame looking? Typical details page missing.
Also, for anyone unfamiliar, WSL = Windows Subsystem for Linux.

Be sure to target WSL2, which embeds an actual Linux kernel (4.19 to start with) into Windows (including some hardware device access), and not the initial WSL "compatibility layer" (Cygwin-like) effort.

What's an early May?


Update: Remove metric requirement validation

Description
Fili is removing the requirement to choose a metric. We will be too. this will allow dimension only reporting.
Update ember data model for request to require selecting a metric.
Make sure alerts is good. make sure adapter handles empty metrics correctly.
Proposed Changes

Remove request model validation that requires at least one metric.

Screenshots

The last two screen shots show chart visualization available. Do they break?


ci: add test that runs on windows-latest

Codecov Report
Patch and project coverage have no change.

Comparison is base (9ea6f94) 82.15% compared to head (1d24ba4) 82.15%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage   82.15%   82.15%
=======================================
  Files          73       73
  Lines        5087     5087
=======================================
  Hits         4179     4179
  Misses        908      908

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


add GHA workflow - freetext > dropdown

We have 5K+ repositories in our org and I am presented with a drop down menu.

Yes, this is a known problem. Of course CCMenu could fetch all repositories, or the user could actively say "fetch more", but then a pop-up isn't a great UI for that. I wanted to use a search-box with drop-down but that UI element doesn't exist in Swift UI and it proved to be quite difficult to implement it. Will return to this sooner or later.

Im not familiar with swift, but it looks like you save the state in json files? Where is the files located? I could problably add the repositories manually?

Yes, that's a possible workaround. That said, CCMenu stores its settings in the macOS database but you can export and import pipeline settings, and those are in JSON format. So, you can create a JSON file with your pipeline (or pipelines even) and then import that from the menu on the pipeline window in CCMenu. The format is relatively straightforward. Here's an example for CCMenu itself:
[
  {
    "feedName" : "",
    "feedType" : "github",
    "feedUrl" : "https://api.github.com/repos/ccmenu/ccmenu2/actions/workflows/build-and-test.yaml/runs",
    "name" : "ccmenu2 | Build and test"
  }
]

Note the outer angular brackets, the file must be an array of pipeline objects. The feedName field is always empty for GitHub pipelines.


Feature/initial

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@605fedf). Click here to learn what that means.
The diff coverage is 84.25%.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   84.25%
=========================================
  Files             ?        3
  Lines             ?      127
  Branches          ?       29
=========================================
  Hits              ?      107
  Misses            ?       20
  Partials          ?        0

Impacted Files
Coverage Δ

src/DropDownFilter/components/DropDownFilter.ts
74.35% <74.35%> (ø)

src/DropDownSort/components/DropDownSort.ts
88.46% <88.46%> (ø)

src/TextBoxSearch/components/TextBoxSearch.ts
88.88% <88.88%> (ø)

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 605fedf...8c2d3a9. Read the comment docs.


Add list of nearby places without photos

Implement #73

Conflict it seems? Would you mind rebasing or pulling from master?

Otherwise, the code looks good so feel free to merge :-)

Oof, there are 71 files that have merge conflicts, mostly the various strings.xml files for different languages. :/ Do I really need to go through all of them and resolve them manually?

You changed 24 files, so there will only be 24 conflicts at most.
On Fri, Sep 2, 2016 at 3:03 PM, Josephine Lim<EMAIL_ADDRESS>wrote:

Oof, there are 71 files that have merge conflicts, mostly the various
strings.xml files for different languages. :/ Do I really need to go
through all of them and resolve them manually?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/commons-app/apps-android-commons/pull/244#issuecomment-244290680,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGFBgDvwow_OmWFw0vUhSNNoKQLoVPeks5ql7xNgaJpZM4JzTHU
.

The conflicts aren't in the files that I changed though. For instance in values-te/strings.xml the conflicting lines of code are
 <string name="app_name_beta">వికీమీడియా కామన్స్ బీటా</string>
<string name="provider_campaigns">ప్రచారాలు</string>

And so on and so forth for other languages

Very strange.
To solve the problem quickly: Download the current master as a zip, replace all of your language files (except the English one) with those from the ZIP, and git add all of them and commit/push

(after merging the real conflicts of cours ^^)

Odd, that doesn't work either, now I just have 130 conflicts (because the error.xml in all the language folders are conflicting now). Is it okay if I just put the language folders in .gitignore?

Okay, I think this might be what's causing the issue:
2016-09-03 16:58:41.6702|WARN|thread: 1|StandardUserErrors|Showing user error Failed to create a new commit.
GitHub.IO.ProcessException: fatal: pathspec 'app/src/main/res/layout/activity_campaigns.xml' did not match any files

I checked, and before I even pulled from commons-app/master, these xml files did not exist in my layout directory. And the PR that I'm trying to merge is removing those files. So I can't merge a commit that removes a file that I didn't already have, I guess?
Although I'm not sure why they are in commons-app/master. I thought I had already removed them some time ago, hmm.

Edit: I think I should have mostly fixed it via command line. Will test and merge shortly.


Different cookie name when saving all or saving selection

Hello and sorry for my English from the google translator !!
When you save the cookie settings, cookies are saved with different names depending on whether you use the accept selection or accept all button.
If you accept the selection, it stores the cookie with the name "cookie_consent_targeting" and if you accept all, it stores the cookie with the name "cookie_consent_cookie_targeting".
This implies that when checking the cookie you have to check the two names
@if(isset($_COOKIE[config('cookies_consent.cookie_prefix') . 'cookies_consent_targeting']) || isset($_COOKIE[config('cookies_consent.cookie_prefix') . 'cookies_consent_cookie_targeting'])
and that in the cookie policy you have to specify too many cookies.
I don't know if it's a small error or if it's programmed like this for some reason, but I hope you can help me.
Thank you very much and greetings

Hello! Thanks for opening this issue.
Indeed, the bug you referred to exists. We are addressing this issue in today's release (v1.0.1). Can you please update the plugin version and let me know if it is fixed for you?

Sorry for not answering sooner, but I was on vacation.
I have already checked the update and it is perfect , now it stores the
selection with the same name.
Thank you very much for the solution and best regards
El vie, 17 mar 2023 a las 17:06, Paul Isaris @.***>)
escribió:

Closed #4 https://github.com/scify/laravel-cookies-consent/issues/4 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/scify/laravel-cookies-consent/issues/4#event-8780073473,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMBFWMLGXYKVZWKYRB4S6O3W4SDZRANCNFSM6AAAAAAVTU2ZKU
.
You are receiving this because you were mentioned.Message ID:
@.***>


How to give initial guess to /diff_drive_controller/odom

ISSUE TEMPLATE ver. 0.4.0
Thank you for contributing to Science, you're great!
I want to plot the topics of /diff_drive_controller/odom and /ground_truth via plotjuggler. I changed the x_pose from gazebo.launch.py to -2.0 to match with the environment but i can see the diff drive controller x position is at 0.0 but the ground truth is at  -2.0. How can i set -2.0 for diff drive controller too as initial starting positiono to then follo ground truth and my extended kalman filter?

Which TurtleBot3 platform do you use?

[ ] Burger

Which ROS is working with TurtleBot3?

ROS2 humble

Which SBC(Single Board Computer) is working on TurtleBot3?

simulation solo on virtual machine for windows

Which OS you installed on SBC?
Ubuntu 22.04

Hi, @PeppePercival
I followed your explanation step-by-step.
However, I found that the initial value of /odom aligns exactly with the Gazebo simulation's ground truth value.
Can you tell me more situation you indeed?

@PeppePercival Apologies for the delayed response. This issue will be closed due to prolonged inactivity. If the problem persists, feel free to reopen it, and we will address it promptly. Thank you for your understanding!


Ignore my previous comment

I just found the discussion on parameterised queries on a later page, so please ignore my previous comment.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a641b52c-39cb-fb2d-ed45-ded0ffafdb66
Version Independent ID: a432ca73-d567-ae0b-7fe8-772556c8bdbe
Content: WHERE clause in Azure Cosmos DB
Content Source: articles/cosmos-db/sql-query-where.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @timsander1
Microsoft Alias: tisande

No worries. One other thing to keep in mind is that Cosmos DB supports read-only SQL queries. You can't change data or drop containers with a SQL query in Cosmos DB.


Add provider - senhasegura DevOps Secrets Management (DSM)

Add senhasegura DevOps Secrets Management (DSM) provider
Let me know if I missed something in the provider implementation

Thanks @lfraga! I did not know about this provider. We are a bit worried about starting to add too many providers to our list, and with that, keep increasing the maintenance load.
With that said, I want to ask you, can we count on you to respond to inquiries regarding this provider? Just cause we had some people contributing bit things in the beginning and vanishing after a while :)
Also, if we need to test something, do you have an account handy to test it? If you could include a bit more info about how to create the secret and all that in the docs would be also nice!
Thanks again for the contribution!

Hi @knelasevero  thanks for your reply
Yes, I can help with maintenance and questions about the senhasegura provider, I've been using ESO and senhasegura for a while
I'll add details in provider docs
About tests, do you have a reference guide to help me create the mocking for tests ?

Hi @gusfcarvalho
Thanks for you review, I will work on that and update the PR

Hey @lfraga just wanted to check your availability to finish this PR :)

Hey @lfraga just wanted to check your availability to finish this PR :)

Hey @knelasevero , yes i'm available and working on this PR locally, thanks :)

Hey @lfraga this might be relevant to you as well :)
https://github.com/external-secrets/external-secrets/pull/967

Hey @gusfcarvalho and @knelasevero , could you review again? I made the requested implementations/fixes, added improvements in the documentation, provider tests and other minor changes.
I believe the only pending task is mock for tests non-provider (go pkg dsm and auth), right ?

Thanks @lfraga! I did not know about this provider. We are a bit worried about starting to add too many providers to our list, and with that, keep increasing the maintenance load.
With that said, I want to ask you, can we count on you to respond to inquiries regarding this provider? Just cause we had some people contributing big things in the beginning and vanishing after a while :)
Also, if we need to test something, do you have an account handy to test it? If you could include a bit more info about how to create the secret and all that in the docs would be also nice!
Thanks again for the contribution!

Hi @knelasevero , I have several accounts on senhasegura that can be used to test DSM, tomorrow I'll send in another location to you and @gusfcarvalho  details about authentication parameters :wink: (post auth info here is not a good idea)

Thanks @lfraga! I did not know about this provider. We are a bit worried about starting to add too many providers to our list, and with that, keep increasing the maintenance load.
With that said, I want to ask you, can we count on you to respond to inquiries regarding this provider? Just cause we had some people contributing big things in the beginning and vanishing after a while :)
Also, if we need to test something, do you have an account handy to test it? If you could include a bit more info about how to create the secret and all that in the docs would be also nice!
Thanks again for the contribution!

Hi @knelasevero , I have several accounts on senhasegura that can be used to test DSM, tomorrow I'll send in another location to you and @gusfcarvalho details about authentication parameters wink (post auth info here is not a good idea)

:partying_face: Nice! If you wanna use an ots service (+ instant messaging) to send it to one of us: https://ots.uncloud.do/ Are you on kubernetes slack?

(Not mandatory)
We have a Automatically generated change log, (that sometimes we manually edit). Would you please squash some of your intermediary commits so we have them showing up cleanly in the change log?


[Bug] No manga found when searched

Device information

Tachiyomi version: <IP_ADDRESS>
Android phone & version: Pixel 4 XL & Android 11

Steps to reproduce

Search for a known manga that exists on mangadex without using any filters.

Expected behavior
Results displayed for the searched term.
Actual behavior
Page shows "No results found."
Other details
Additional details and attachments.

clear your cookies in advanced settings and relogin

Worked perfectly. Thanks for the quick reply. Love your app to pieces.


Design token value map

Is your feature request related to a problem? Please describe.
Switching a theme should be as easy as swapping out one design token lib for another. The problem is that the SCSS for the auro-button element's SCSS uses tokens directly as the value per CSS property.
  // auro-button--tertiary
  &--tertiary {
    background-color: var(--auro-color-background-lighter);
    border-color: var(--auro-color-ui-default-on-light);
    box-shadow: inset 0 0 0 3px var(--auro-color-ui-default-on-light);
  }

Describe the solution you'd like
A proposed solution is to create a mapping from any pre-defined list of design tokens to a more generic set of values. A possible solution is to use fallback values.
  // auro-button--tertiary
  &--tertiary {
    background-color: var(--auro-color-background-lighter, var(--airstream-color-background-lighter));
    border-color: var(--auro-color-ui-default-on-light, var(--airstream-color-ui-default-on-light));
    box-shadow: inset 0 0 0 3px var(--auro-color-ui-default-on-light, var(--airstream-color-ui-default-on-light));
  }

While this simplifies the addition of the different themes, this also means that we need to update individual element Scss if there are changes.
Another way to consider this is to have a single map.scss file that uses a fallback method to redefine the value of generically used tokens. t is also recommended that the mapping of these values is not contained per custom element. This should be a global resource that is subscribed to.
It is assumed that we could create this from the tokens repo.
:root {
  --color-background-lighter: var(--auro-color-background-lighter, var(--airstream-color-background-lighter))
  --color-ui-default-on-light: var(--auro-color-ui-default-on-light, var(--airstream-color-ui-default-on-light))
}

Then the SCSS would be the following.
  // auro-button--tertiary
  &--tertiary {
    background-color: var(--color-background-lighter);
    border-color: var(--color-ui-default-on-light);
  }

Exit criteria
For this issue to be closed, a user must be able to subscribe to an alternate set of design tokens and HTML custom elements within the scope of that project will change themes based on those tokens.
Dependency

https://github.com/AlaskaAirlines/AuroDesignTokens/issues/88

Link to ITS -> Auro Design Token Mapping

After discussion with @jason-capsule42, we feel as though auro-input would be a better component to use for design token value mapping. The reason behind this is that auro-button is an older component that uses the imports for Sass properties and variables to build out a CSS variable file. While auro-input directly imports the Custom CSS tokens directly onto its demo page, easily allowing for us to switch between this import and any other theming imports with simple JS.

Refer to this new ticket in auro-input to continue the design token value mapping discussion:

https://github.com/AlaskaAirlines/auro-input/issues/224


Tabs with Header

hope, Materialize have one Tabs style with Header like https://developers.google.com/fonts/

You can make a navbar or any div above the tabs and make them the same color. There is no need for us to do this for you when you have the tools to make it yourself.


Resize on upload cant be turned off

Description
If I disable Resize on upload the setting doesn't get saved:

Steps to reproduce

Go to the settings page
Turn of Resize on upload on a specific asset resource
press Save

Additional info

Plugin version: 1.0.1
Craft version: 2.6.3002

The latest 1.1.0 release fixed this.


MatSelect respects disabled input with reactive form control while MatInput does not

Reproduction
Here's the issue on Stackblitz: https://stackblitz.com/edit/angular-ivy-mdin4v?file=src%2Fapp%2Fapp.component.html
Steps to reproduce:

I'm using the formInputs array to dynamically generate form fields using an *ngFor directive in the html
The [disabled] property on the matInput doesn't get applied for normal inputs. It does however get applied for other components like <mat-select>.

Expected Behavior
The matInput should be disabled
Actual Behavior
The disabled property wasn't applied on the input field (matInput).
Environment

Angular: 9.1.4
CDK/Material: 9.2.4
Browser(s): Brave
Operating System (e.g. Windows, macOS, Ubuntu): Windows

Thanks for the issue. I think this is working as expected as with reactive form controls, the disabled input is supposed to be a noop. This is the same for native inputs. See the form control logic. The warning is also printed in the StackBlitz.
Interestingly enough though, mat-select behaves differently as you noted. This is because select implements a control value accessor and does not prioritize the value from potential controls (as done in MatInput or native inputs) It might be worth making this consistent.

The fix for this ended up breaking a lot of apps internally and it appears to be against the convention in the ReactiveFormsModule based on this warning https://github.com/angular/angular/blob/master/packages/forms/src/directives/reactive_directives/form_control_directive.ts#L75. Closing since reactive form controls should be disabled using the form control itself, not the disabled binding.

@crisbeto Can you reopen this issue?
In https://github.com/angular/components/pull/19705 you stated

It ended up being quite breaky and it looks like the forms module logs a warning against usages like this

This warning message is occurring with how the MatSelect component is currently structured; that PR solves the warning message and keeps it from appearing.
Please correct me if I am wrong, but there is currently no way to have the FormControl disable the MatSelect.
A separate binding to MatSelect's [disabled] input allows us to disable the MatSelect.
<mat-select formControlName="selectControl" [disabled]="false">
    <mat-option value="option1">Option 1</mat-option>
    <mat-option value="option2">Option 2</mat-option>
</mat-select>

...but as we've seen, this throws the "disabled attribute with a reactive form directive" warning message.
What the reactive forms module wants you to do is take that [disabled] binding out, like this:
<mat-select formControlName="selectControl">
    <mat-option value="option1">Option 1</mat-option>
    <mat-option value="option2">Option 2</mat-option>
</mat-select>

...but now we can't disable the MatSelect because MatSelect doesn't care/honor if its backing NgControl is disabled.
You can obviously work around this by manually enabling/disabling the matSelect.disabled  property in code, but that is a pain.
The changes in https://github.com/angular/components/pull/19705 look like exactly what needs to be done, especially considering this is how MatInput works.
Can you provide an example(s) of the kind of code this change is breaking?
As a related sidenote, I might put in a PR to just completely remove this warning message. I feel like it was added with zero thought put toward [disabled] being an input property to a directive on the same HTML element as this NgControl and just assuming the user was always trying to bind to the disabled HTML attribute .
The disabled input on the NgControl directive should really be this:
@Input('disabled', 'But only if `disabled` is not an input for any other directives or if *all* these other directives respect the disabled state of this NgControl which I obviously can't check for')
set isDisabled(isDisabled: boolean) {
	if (typeof ngDevMode === 'undefined' || ngDevMode) {
	  console.warn(disabledAttrWarning);
	}
}

The whole vibe of the warning message is:
"You don't know what you're doing!"
When in fact, the answer is:
"Yes, I do. Unfortunately, the MatSelect component doesn't check if you are disabled, which is why I am providing this input in the template. Please get off my back and leave me alone."

Please correct me if I am wrong, but there is currently no way to have the FormControl disable the MatSelect.

I'm not sure what you mean by this, because the select has respected the disabled state of a FormControl since we first released the component. Here's an example: https://stackblitz.com/edit/angular-n4azj3?file=src%2Fapp%2Fselect-overview-example.html

@crisbeto It appears you are correct. Ignore my comments above about MatSelect.
I forked your example and made it closer to what we were doing, but it still works as you stated.
https://stackblitz.com/edit/angular-n4azj3-nfpryi?file=src%2Fapp%2Fselect-overview-example.ts
I'll just have to dig into our code further to figure out why the MatSelect isn't being disabled. Thanks for providing that example.


NPM and Bower Submission

Thank you for this excellent library. What about adding module loading support and publishing it to npm and bower?

Thanks for your kind words @nileflowers!
I already registered it in bower, can it be improved in some way?
I come from the java world, so I need some directions here. Can you give me any hint?
Also, feel free to do it youself and send me a pull request.

I think I know just enough to get by. Let me try to get the module loading support started in the next couple of days.
I'll keep you posted on my progress.
Thanks!

Great!

the package.json is missing a version, without it, nom refuses to recognize it as a package.
npm ERR! addLocal Could not install /var/folders/11/y4k_165n06lg_tn2nb02nbjr0000gn/T/npm-81078-f0bc005d/git-cache-0b28cc08c116a9ae00185274b10f0903/86350eb237305cbab6c33a607560a709d21281db
npm ERR! Darwin 15.2.0
npm ERR! argv "/usr/local/Cellar/node/5.3.0/bin/node" "/usr/local/bin/npm" "install" "--save" "https://github.com/brutusin/json-forms.git"
npm ERR! node v5.3.0
npm ERR! npm  v3.3.12

npm ERR! No version provided in package.json
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/dh/Projects/Eventured/node_booking_api/npm-debug.log

That last comment has been solved by PR https://github.com/brutusin/json-forms/pull/14

@nileflowers I think this issue has been solved right?

Sorry @idelvall, @e110c0 for the delay.
@idelvall
I think that's needed for sure, but I was thinking more along the lines like you see in my commit that I just pushed. I basically added support so the module could work more seamlessly with the latest-generation module bundlers like webpack and jspm. Basically removing the assumption that it'll always run in a browser.
I am not ready to create a pull request yet because I haven't tested this. I am thinking about running my own manual tests as well as adding tests using a test harness. Haven't gotten to this part yet.

@nileflowers that sounds great!
Yeah, I assumed since the beginning the application being executed in a browser since these were my needs.
I know there is a whole world around node.js, but I am complete newbie in it, so I leave this matter to you.
Thanks mate!

Hi,
'brutusin-json-forms' is not in the npm registry.
npmjs.com

Yes @dawadam, this issue is still open.
Maybe you can help us and send a PR that solves the issue?

Hi,
is 'brutusin-json-forms' now in the npm registry ?

@idelvall
I think we can't send a PR to add the package to the npm registry.


Nuxt.js support

Summary
Integrating tinacms with Nuxt.js
Motivation
Since Nuxt.js is a framework for building websites in vue.js (like next for react), it would be interesting to have a package for it.

My company would be willing to sponsor this feature. LMK if that’s of interest, maybe a few companies can pool their resources and get this done.

My company would be willing to sponsor this feature. LMK if that’s of interest, maybe a few companies can pool their resources and get this done.

I'm also interest in that, and I can join the development, if anyone want to start that.

+1
And it begs the overarching question: is Tina intended just for integration with Next? Or are other stacks on the roadmap?

+1
We use Nuxt (and Forestry) pretty heavily, I also agree that it would be great to have Nuxt support in Tina!

+1

My company would be willing to sponsor this feature. LMK if that’s of interest, maybe a few companies can pool their resources and get this done.

Seconded! Our company, which primarily uses nuxt as well for static sites, is good to co-sponsor this feature as well. Tina looks great, but we are definitely happier using Vue/Nuxt as opposed to React/Next.
We're actively avoiding migrating from Forestry to Tina because we need things to stay in Vue.

@@jamespohalloran, I noticed you removed the wontfix label last time. Is this issue effectively resolved in the new v1.0?

In the early days of TinaCMS, there was a high bar for getting started. TinaCMS is now easier than ever to setup.

It looks like this will create previews of the generated site with an iframe. Any idea if this feature indirectly allows for Nuxt support as the above quote suggests? I imagined that generating any site in a frame would only solve half the problem?


[Windows custom ops] Compiled c++ and imported dll successfully, but custom ops are not visible from python?

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Win 7 64
TensorFlow installed from (source or binary): sources
TensorFlow version (use command below): 1.3.1
Python version: 2.7.13
Bazel version (if compiling from source): NA
GCC/Compiler version (if compiling from source): Visual Studio 15
CUDA/cuDNN version: 8.0/7.5
GPU model and memory: NA
Exact command to reproduce: Please see below.

Describe the problem
Trying to build a library available at https://github.com/optas/latent_3d_points.
It has custom ops implemented in c++.
I have successfully built the package therein specifically structural_losses from https://github.com/optas/latent_3d_points/tree/master/external/structural_losses with changes for compiling with Visual Studio 2015.
In python, I am also able to load the ops dlls e.g. using
approxmatch_module = tf.load_op_library(osp.join(base_dir, '_approxmatch.pyd')).
However, I don't see the operators approxmatch_module.approx_match,approxmatch_module.match_cost, etc. from the c++ getting exposed to the python tensorflow.
I am getting the following error:
AttributeError: 'module' object has no attribute 'approx_match'
When I do the following
from structural_losses import approxmatch
dir(approxmatch.approxmatch_module)

it lists only the following items
['LIB_HANDLE', 'OP_LIST', '_InitOpDefLibrary', '__builtins__', '__doc__', '__name__', '__package__', '_collections', '_common_shapes', '_op_def_lib', '_op_def_library', '_op_def_pb2', '_op_def_registry', '_ops']
where the actual operations from c++ are missing.
What am I missing from proper compilation?
Or is it that in windows custom ops is unavailable/not implemented?

@mrry might know more about Windows status.

@cooltopics Are you certain that the custom library was built and linked correctly? One potential cause for this problem is if the static initializers for Op and Kernel registration are "optimized" out of the final DLL. The /WHOLEARCHIVE option to link.exe might be useful here.

@mrry Losing hope, I have downloaded Tensorflow version 1.7.1 sources and built anew for Python 3.6 win64. Using this versions combination, it is working. I propose to close this issue as I no longer have the builds for the particular combination mentioned in this issue.

Hi, @cooltopics
Have you solved the problem?
I try to build a tensorflow user op under Window 10. Following the Official Docs, I successfully generate a zero_out.dll using VS2017, But when I try to load and use it in python,
import tensorflow as tf
zero_out_module = tf.load_op_library('./kernel/zero_out.dll')

use_cpu = True
device_count = {"GPU": 0} if use_cpu else {"GPU": 1}
with tf.Session(config=tf.ConfigProto(device_count=device_count)):
  zero_out_module.zero_out([[1, 2], [3, 4]]).eval()

it produces an error with
AttributeError: module 'f21c708d1ddc75dcce283dd13fe531f7' has no attribute 'zero_out'

Do I miss something or make any mistakes? Do you have any suggestion?
Thanks a lot.

Hi all,
I just wanted to write that I had the same problem under Mac OS X, and after a lot of dead ends (including building tensorflow from source...!) I realised that I hadn't copied part of the example to my file. In particular, I was missing the REGISTER_OP part right at the top of the file. The following file works for me:
`#include "tensorflow/core/framework/op_kernel.h"
#include "tensorflow/core/framework/op.h"
#include "tensorflow/core/framework/shape_inference.h"
using namespace tensorflow;
REGISTER_OP("ZeroOut")
.Input("to_zero: int32")
.Output("zeroed: int32")
.SetShapeFn([](::tensorflow::shape_inference::InferenceContext* c) {
c->set_output(0, c->input(0));
return Status::OK();
});
class ZeroOutOp : public OpKernel {
public:
explicit ZeroOutOp(OpKernelConstruction* context) : OpKernel(context) {}
void Compute(OpKernelContext* context) override {
// Grab the input tensor
const Tensor& input_tensor = context->input(0);
auto input = input_tensor.flat();
// Create an output tensor
Tensor* output_tensor = NULL;
OP_REQUIRES_OK(context, context->allocate_output(0, input_tensor.shape(),
                                                 &output_tensor));
auto output_flat = output_tensor->flat<int32>();

// Set all but the first element of the output tensor to 0.
const int N = input.size();
for (int i = 1; i < N; i++) {
  output_flat(i) = 0;
}

// Preserve the first input value if possible.
if (N > 0) output_flat(0) = input(0);

}
};
REGISTER_KERNEL_BUILDER(Name("ZeroOut").Device(DEVICE_CPU), ZeroOutOp);
`
You guys may have a Windows-specific problem, but I thought I'd pass this on anyway.

@mrry Losing hope, I have downloaded Tensorflow version 1.7.1 sources and built anew for Python 3.6 win64. Using this versions combination, it is working. I propose to close this issue as I no longer have the builds for the particular combination mentioned in this issue.

Hi @cooltopics  Could  you share some information how you solved the problem? To build the user defined op in Windows, do you need to install the tensorflow from source and put the defined c++ in the tensorflow/core/.. ?  Looking forward to your comments!

Hi, @cooltopics
Have you solved the problem?
I try to build a tensorflow user op under Window 10. Following the Official Docs, I successfully generate a zero_out.dll using VS2017, But when I try to load and use it in python,
import tensorflow as tf
zero_out_module = tf.load_op_library('./kernel/zero_out.dll')

use_cpu = True
device_count = {"GPU": 0} if use_cpu else {"GPU": 1}
with tf.Session(config=tf.ConfigProto(device_count=device_count)):
  zero_out_module.zero_out([[1, 2], [3, 4]]).eval()

it produces an error with
AttributeError: module 'f21c708d1ddc75dcce283dd13fe531f7' has no attribute 'zero_out'

Do I miss something or make any mistakes? Do you have any suggestion?
Thanks a lot.

Hello,

Hi, @cooltopics
Have you solved the problem?
I try to build a tensorflow user op under Window 10. Following the Official Docs, I successfully generate a zero_out.dll using VS2017, But when I try to load and use it in python,
import tensorflow as tf
zero_out_module = tf.load_op_library('./kernel/zero_out.dll')

use_cpu = True
device_count = {"GPU": 0} if use_cpu else {"GPU": 1}
with tf.Session(config=tf.ConfigProto(device_count=device_count)):
  zero_out_module.zero_out([[1, 2], [3, 4]]).eval()

it produces an error with
AttributeError: module 'f21c708d1ddc75dcce283dd13fe531f7' has no attribute 'zero_out'

Do I miss something or make any mistakes? Do you have any suggestion?
Thanks a lot.

Hello, could you solved this problem?

Hi, @cooltopics
Have you solved the problem?
I try to build a tensorflow user op under Window 10. Following the Official Docs, I successfully generate a zero_out.dll using VS2017, But when I try to load and use it in python,
import tensorflow as tf
zero_out_module = tf.load_op_library('./kernel/zero_out.dll')

use_cpu = True
device_count = {"GPU": 0} if use_cpu else {"GPU": 1}
with tf.Session(config=tf.ConfigProto(device_count=device_count)):
  zero_out_module.zero_out([[1, 2], [3, 4]]).eval()

it produces an error with
AttributeError: module 'f21c708d1ddc75dcce283dd13fe531f7' has no attribute 'zero_out'

Do I miss something or make any mistakes? Do you have any suggestion?
Thanks a lot.

Hi~
Could you solved this problem?
I have the same problem.

Hi all,
I just wanted to write that I had the same problem under Mac OS X, and after a lot of dead ends (including building tensorflow from source...!) I realised that I hadn't copied part of the example to my file. In particular, I was missing the REGISTER_OP part right at the top of the file. The following file works for me:
#include "tensorflow/core/framework/op_kernel.h"

#include "tensorflow/core/framework/op.h"
#include "tensorflow/core/framework/shape_inference.h"

using namespace tensorflow;

REGISTER_OP("ZeroOut")
    .Input("to_zero: int32")
    .Output("zeroed: int32")
    .SetShapeFn([](::tensorflow::shape_inference::InferenceContext* c) {
      c->set_output(0, c->input(0));
      return Status::OK();
    });

class ZeroOutOp : public OpKernel {
 public:
  explicit ZeroOutOp(OpKernelConstruction* context) : OpKernel(context) {}

  void Compute(OpKernelContext* context) override {
    // Grab the input tensor
    const Tensor& input_tensor = context->input(0);
    auto input = input_tensor.flat<int32>();

    // Create an output tensor
    Tensor* output_tensor = NULL;
    OP_REQUIRES_OK(context, context->allocate_output(0, input_tensor.shape(),
                                                     &output_tensor));
    auto output_flat = output_tensor->flat<int32>();

    // Set all but the first element of the output tensor to 0.
    const int N = input.size();
    for (int i = 1; i < N; i++) {
      output_flat(i) = 0;
    }

    // Preserve the first input value if possible.
    if (N > 0) output_flat(0) = input(0);
  }
};

REGISTER_KERNEL_BUILDER(Name("ZeroOut").Device(DEVICE_CPU), ZeroOutOp);

You guys may have a Windows-specific problem, but I thought I'd pass this on anyway.

Hi ~,I have met the same question on ubuntu. But the library complied by g++ works well.
How did you solve this problem?

@ZhuXinyan @sc1991327 @sunglowzhang @VVingerfly
It is likely that the building of the shared library did not include the header file correctly.
If this particular line is incorrect, the custom op interface will not find the method or attribute correctly.


[FEATURE] ESM and CJS build

Is your feature request related to a problem? Please describe.
Inserting script tags into the head tag isn't very organized especially for somewhat complex projects.
Describe the solution you'd like
An ESM build along with a CJS build.
Describe alternatives you've considered
None.
Additional context
I can do this in a PR just checking what people are thinking about this.

Just pushed builds for cjs, esm, and iife, may need some testing to make sure it's compatible to use with bundlers