std::array doesn't have a default printer.

Hi.
For std::array comparison I get:
REQUIRE( {?} == {?} )
I believe you support this for other containers. Can you add this when/if you have the time?

@DenisYaroshevskiy This isn't supported for any container from the STL - Catch2 does that. I might provide a second header with extensions (because the main header will never include any STL/whatever header) and provide stringification for the most common containers/STL Types.
A related issue is printing ENUM values which will also go into such an extension header - see this: https://github.com/onqtam/doctest/issues/121

@onqtam Oh - I see.
You do rely on ADL though for looking up types <=> for std ones it's not nice to do so.
Can I specialize smth instead?

How about for generic  test? I don't know what your minimal language requirements, but if you do it based on having nested iterator type, you should be OK even for 98.
What if 1 million? Good point. Well - dump all is a very decent default if you could override it.

C++11 is the requirement for doctest 2.x (no longer C++98).
I think this is how Catch2 detects ranges.
I might implement matchers at some point and in that case when comparing containers you would get automatically the index that points to the different elements - this would be very hard to do with the current stringification of the values.
But currently I'm focusing on finishing the XML reporter.


Alias issues on join

I have a simple setup with the following configuration and tables:
roapi.yml
addr: <IP_ADDRESS>:8080
tables:
  - name: "countries"
    uri: "countries.csv"

  - name: "cities"
    uri: "cities.csv"

countries.csv
ID,COUNTRY
1,Germany
2,Sweden
3,Japan

cities.csv
ID,CITY,COUNTRY_ID
1,Hamburg,1
2,Stockholm,2
3,Osaka,3
4,Berlin,1
5,Göteborg,2
6,Tokyo,3
7,Kyoto,3

If I then perform the following JOIN request without any issues:
$ curl -X POST -H 'Accept: application/csv' -d "SELECT c1.CITY, c2.COUNTRY FROM cities AS c1 JOIN countries AS c2 ON c1.COUNTRY_ID = c2.ID" http://<IP_ADDRESS>:8080/api/sql

CITY,COUNTRY
Stockholm,Sweden
Göteborg,Sweden
Osaka,Japan
Tokyo,Japan
Kyoto,Japan
Hamburg,Germany
Berlin,Germany

But if I then try to additionally select the city ID things doesn't goes so well:
$ curl -X POST -H 'Accept: application/json' -d "SELECT c1.ID, c1.CITY, c2.COUNTRY FROM cities AS c1 JOIN countries AS c2 ON c1.COUNTRY_ID = c2.ID" http://<IP_ADDRESS>:8080/api/sql

{"code":400,"error":"query_execution","message":"Failed to execute query: Error during planning: The left schema and the right schema have the following columns with the same name without being on the ON statement: {Column { name: \"ID\", index: 0 }}. Consider aliasing them."}%

To me this seems as if the aliasing is not working as expected. I would expect the query engine to be able to distinguish between those two fields since I've aliased the to tables (c1 and c2 respectively). Using e.g. sqlite this query works as expected:
sqlite> SELECT c1.ID, c1.CITY, c2.COUNTRY FROM cities AS c1 JOIN countries AS c2 ON c1.COUNTRY_ID = c2.ID;
1|Hamburg|Germany
2|Stockholm|Sweden
3|Osaka|Japan
4|Berlin|Germany
5|Göteborg|Sweden
6|Tokyo|Japan
7|Kyoto|Japan

Tested on 0.3.4 and main+gf72299f.

Thanks @johanols for the detailed bug report! This is caused by https://github.com/apache/arrow-datafusion/blob/db305dadea458a26723fa53d85463155247d39f9/datafusion/src/physical_plan/hash_utils.rs#L92. From a quick glance, it looks like we are indeed being too strict on join conditions. I will work on a fix for it this weekend.

This should be addressed by https://github.com/apache/arrow-datafusion/pull/1023

FYI, cutting a 0.4.1 release to pick up this fix.

Wow! That was fast! 😲
Just tried it out with the 0.4.1 release. Works like a charm 👍
Thank you!


ERR_UNKNOWN_FILE_EXTENSION with css inside dependency

Describe the bug
When a npm dependency imports CSS, vitest's resolution fails with node error ERR_UNKNOWN_FILE_EXTENSION, even with the css option is enabled. The same works fine when the imported CSS file is first-party, so I think it's an error in vitest's module resolution. vite is able to build this fine, but vitest fails.
Previous suggestions to this problem were to use deps.inline, but as this has now been deprecated, I'm looking for better solutions which ideally do not involve whitelisting the dependencies that have CSS imports.
Reproduction
Repo: https://github.com/silverwind/vitest-css.
Dependency vitest-css-test-module is published to npm from the subdirectory.
git clone https://github.com/silverwind/vitest-css
cd vitest-css
npm install
npx vitest

Will fail with
 DEV  v0.34.1 vitest-css

 ❯ index.test.js (0)

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Failed Suites 1 ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯

 FAIL  index.test.js [ index.test.js ]
TypeError: Unknown file extension ".css" for node_modules/vitest-css-test-module/index.css
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
Serialized Error: { code: 'ERR_UNKNOWN_FILE_EXTENSION' }
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/1]⎯

 Test Files  1 failed (1)
      Tests  no tests
   Start at  14:30:10
   Duration  343ms (transform 36ms, setup 0ms, collect 0ms, tests 0ms, environment 0ms, prepare 99ms)

System Info
System:
    OS: macOS 13.4
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 84.09 MB / 16.00 GB
    Shell: 5.9 - /usr/local/bin/zsh
  Binaries:
    Node: 20.3.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.7.1 - ~/.npm-global/bin/npm
    pnpm: 8.6.2 - ~/.npm-global/bin/pnpm
  Browsers:
    Chrome: 113.0.5672.126
    Chrome Canary: 114.0.5686.0
    Firefox Nightly: 113.0a1
    Safari: 16.5
  npmPackages:
    vitest: 0.34.1 => 0.34.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Previous suggestions to this problem were to use deps.inline, but as this has now been deprecated, I'm looking for better solutions which ideally do not involve whitelisting the dependencies that have CSS imports.

Use deps.optimizer.

Use deps.optimizer.

Such an option does not exist. You mean optimizeDeps.disabled? I just tried it, same error.

Such an option does not exist

https://vitest.dev/config/#deps-optimizer

I think this is defintely a bug in vitest because the resolution works with vite.

This is not a bug, but a performance optimization. We cannot run all dependencies inside vite-node because your tests will be running x10 slower. If you want identical handling, use browser tests. Vitest does a lot of hacks to be performant, but we cannot handle all cases.

I just want to elaborate on the idea with the optimizer (currently in the reproduction it doesn't change anything). When .css is used in a dependency, it is usually some kind of a components library. For performance reasons, it's always better to bundle it (the reason is described in the documentation).
These files are then imported using the native Node.js mechanism (not the vite-node wrapper that is used for source code, it is very slow on large amounts of files). Unfortunately with assets (.png, .svg, ...) and .css files it doesn't really work that well. You can still use deps.server.inline to inline only needed dependencies.
Vitest will provide an alternative option to allow the processing of specific files without inlining them (so, they will still run in Node.js, and not in slow vite-node).

I tried this

There is no "disable" option in the optimizer. It's called enabled and it overrides optimizeDeps.disable. Also, I said in my previous message:

I just want to elaborate on the idea with the optimizer (currently in the reproduction it doesn't change anything).

I don't really mind slower performance if the outcome is correct and works resolves like vite.

I do mind it :)
As I said before, you can still use server.deps.inline for your usecase.

There is no "disable" option in the optimizer. It's called enabled and it overrides optimizeDeps.disable

Would you mind posting a full config for the repro that works? The vitest docs link to vite's optimizeDeps which shows a disabled. This is the latest thing I tried:
import {defineConfig} from "vitest/config";

export default defineConfig({
  test: {
    environment: "jsdom",
    deps: {
      optimizer: {
        web: {
          include: [],
        },
      },
    },
  },
});

Generally, I do think removing this implicit CSS import and moving it to an explicit @import in the consumer's CSS is the way to go as I see value in having such frontend modules importable in Node. Still I would be interested in how it can be solved with deps.optimizer, I tried many variations of the supposed config like with empty include array but nothing works. The docs on this option are really lacking, they don't specify what options web or ssr can hold, some examples would be great there.

Okay, feel free to close unless you want to keep it open for the mentioned bug. I will definitely remove these CSS imports as I don't feel like relying on such advanced/unstable config options is a good idea.
I do wonder one thing thought: What if the module would export a .jsx file? Node can't load these either. Would vitest fail similarily? I think the problem is not directly related to JS, but all things node can't load (but vite can).


Add rarity tags to Archetype journals and add missing rarity to several entries

Adds rarity to several archetypes entries that were missing this and used rarity tags:

I think we can remove the hr after the tag

I think we can remove the hr after the tag

Changes made


[AIRFLOW-3455] add region in snowflake connector

Jira

[ ] My PR addresses the following Airflow Jira issues and references them in the PR title.

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[ ] Passes flake8

Codecov Report

Merging #4285 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4285      +/-   ##
==========================================
+ Coverage   78.07%   78.08%   +<.01%
==========================================
  Files         201      201
  Lines       16458    16458
==========================================
+ Hits        12850    12851       +1
+ Misses       3608     3607       -1

Impacted Files
Coverage Δ

airflow/models.py
92.33% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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


Bigger viewport scale for Desktop version

We need to be able to support:

smaller viewport for iPhone 5 SE
larger viewport for desktop

Also, I feel like while making these additions, more concise logic can be used to accomplish the goal.


Timd

I found this word timd in this quotation. "Moba a chedaol el timd el ouetkeu er a rokui el odam." ("Greet all the brethren with a holy kiss.") This seems to be from the word melimd, which is currently defined as "sip." Has anyone else seen this word used for "a kiss"?

Should timd be added as a new word?
Does the word melimd also mean "to kiss" and thus could be added to the definition?
Is there any inappropriate connotation of this word?
Is there a Palauan word that means "a kiss" (noun) with neutral implications? I assume most Palauans just say "kiss."

melemtimd:timd: ketimd-all part of kissing.
Melimd: sip water.
On Tue, Oct 18, 2022 at 9:04 PM ChrisPerrette @.***>
wrote:

I found this word timd in this quotation. "Moba a chedaol el timd el
ouetkeu er a rokui el odam." ("Greet all the brethren with a holy kiss.")
This seems to be from the word melimd, which is currently defined as
"sip." Has anyone else seen this word used for "a kiss"?

Should timd be added as a new word?
Does the word melimd also mean "to kiss" and thus could be added
to the definition?
Is there any inappropriate connotation of this word?
Is there a Palauan word that means "a kiss" (noun) with neutral
implications? I assume most Palauans just say "kiss."

—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/356, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF2DJJCXRFJL33NQKO2GXCTWD5XNJANCNFSM6AAAAAARIWM2AU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Tools.isBrower doesn't support Content Security Policy

The use of new Function() within tools.ts isBroswer() results in an eval function being used which means that any project using ng-sidebar cannot use a restrictive CSP header.
To reproduce the problem:
ng new csp-test
Edit index.htm to include:
<meta http-equiv="Content-Security-Policy" content="default-src 'none'; script-src 'self'; style-src 'self' 'unsafe-inline'; img-src 'self'; connect-src 'self' ws://localhost:4200; font-src 'self';">
Reference ng_sidebar and use AOT:
ng serve --aot
You will get errors in the browser when it blocks the eval function.
I do not understand the function or would make a fix & pull request but for my local version have just used return true; to sidestep the issue.  Perhaps you know of another way to determine if the code is running inBrowser without the use of new Function()?

The function was written that way to avoid it crashing in Node.js contexts (i.e. SSR), but also written before Angular provided a way of checking the context.
It'll need to be updated to make use of PLATFORM_ID from @angular/core / isPlatformBrowser from @angular/common.
I'll try to get around to it within the next few days, but I'll also happily accept PRs. :)


Allow to check whether a document has a specific field

If I try to access a field that is not present, I get an exception.  It would be great if there would be a way to check whether a document has a specific field, e.g. with
"some_field" in doc

why can't you just do if (doc.some_field) ?

Is this because objects that get passed in through params are of type java.util.HashMap instead of a proper javascript object? typeof doesn't work either. if (doc.some_field) or if (doc['some_field']) works fine.

Something like that, based also on this: https://github.com/elastic/elasticsearch-lang-javascript/issues/26
I actually decided to rewrite our script in Groovy since JS support doesn't seem to be tested very well.

Same issue, some objects come in as java.util.HashMap.
{Object here} "of class java.util.HashMap where it expected String, Number, Boolean or Scriptable instance. Please check your code for missing Context.javaToJS() call."

Closing as this plugin does not live here anymore and we recommend using painless instead.


Final Metric Calculation in Code

Hello,
I hope you're doing well! I've been working with the code and have a couple of queries I'd like to discuss.
It appears that the current code calculates metrics on a per-activity basis. Could you confirm if the MOF metric is averaged per activity, or if it's calculated as the overall mean of correctly classified frames over total frames?
If the calculation involves averaging MOF scores per activity, I've observed some discrepancies between the BF and YTI datasets. Notably, the differences in the YTI dataset are quite significant.
I'd appreciate your insights on these points. Clarifying the intended methodology and addressing any inconsistencies would greatly assist in ensuring the accuracy of our results.

Hi,
The MoF metric is calculated as the overall mean, not averaging MoF per activity. I did this manually instead of in the code. This is also what we report for the reproduced version of CTE in the paper.

Closed.


Leaflet-Plugin: LatLng does only work with flat data

I ran into the problem, that I cant use my data with the plugin. The config object provides a field for "latField" and "lngField", however these are access via entry[latField] in the plugin code.
My data looks like the following:
[{      temp: 30,      geoPos: { latitude: 50, longitude: 20 } }]
Access the value will not work since entry['geoPos.latitude'] cannot be resolved.
Would be great, if it would be possible to use this plugin without the need to process the data beforehand.

I'm sorry, data processing will be necessary w/ the default leaflet plugin implementation as the data schema heatmap.js won't change. Too much flexibility would confuse people who just started. BUT you could adapt the heatmap-leaflet plugin so it works for your specific data format


[Model Deprecation]: essincident version 1.0.0 deprecate

Update Reason

essincident version 1.0.0 shall be set to deprecate because new version 2.0.0 is not compatible to version 1.0.0. As version 1.0.0 is not in use by anyone it should be deprecated as soon as version 2.0.0 is released.
MS1 Criteria

[x] The model that should be deprecated exists and is not deprecated already
[x] The respective model does not affect existing use cases (aspect is either used in higher versions or replaced by a different aspect)
[x] A shared aspect, which will be deprecated, is not used by any released aspect model

@agg3fe can you take over and create an PR to change status to deprecated?


Add s390x arch support

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Can you add s390x support for Traefik? Is it planned ?
https://github.com/traefik/traefik-library-image/issues/20
PR - https://github.com/traefik/traefik/pull/8729

Closed by #8729.


Running a vibe-d app with --DRT-gcopt=profile:1 fails

I can't pass --DRT-gcopt to my vibe-d application.
Unrecognized command line option: --DRT-gcopt=profile:1

This is using a custom main function with:
	if (!finalizeCommandLineOptions())
		return;
	lowerPrivileges();
	runEventLoop();

Hmm, removing if (!finalizeCommandLineOptions) allows the options to pass down to the runtime.
That function might need to recognize D runtime specific options.

Will be fixed by https://github.com/vibe-d/vibe.d/pull/1944


Consider moving images form /lib/images/ to the assets folder /static/images in de root of your project

That's what the static folder is for

I have remove the images in the lib
https://github.com/lisagjh/voorhoede/tree/feature-member-overviewpage/src/lib

Now it is in static the images
https://github.com/lisagjh/voorhoede/tree/feature-member-overviewpage/static


feat: Add button to disable ModSettings mod

In preparation for Northstar releasing merged Mod Settings mod.

@EladNLG Hi :)

hello yes what do youk want

you should probably also add a button to disable all mods that depend on ModSettings.

you should probably also add a button to disable all mods that depend on ModSettings.

Do we need one? Cause AFAIK the merged code mirrors ModSettings functions so one can simply drop ModSettings mod, no?

you should probably also add a button to disable all mods that depend on ModSettings.

Do we need one? Cause AFAIK the merged code mirrors ModSettings functions so one can simply drop ModSettings mod, no?

I though it was different ¯\_(ツ)_/¯

Do we need one? Cause AFAIK the merged code mirrors ModSettings functions so one can simply drop ModSettings mod, no?

Not necessarily, there are two cases where they will break:
If a mod relies on pre 2.0 modsettings (should be very rare now, and not a lot we can do about it)
If a mod uses dependency constants. this will break the mod in some way, depends how it is made. Some mods will prompt the user to download modsettings, which is going to be confusing since it wont work


Adding libraries needed to build Krita

What does your PR fix?
This PR tries to package the libraries into vcpkg that we use to build Krita. I am still pretty new to packaging libraries for vcpkg, so still learning. I took the KF5 portfiles from #4652 and #7075.

Which triplets are supported/not supported? Have you updated the CI baseline?
For now I am testing this on x64-windows but I have to check on other systems for compatibility. Yes I have updated the CI baseline.

Does your PR follow the maintainer guide?
Yes

@hellozee
Thanks for your update.
It seems that there are still some other review suggestions that haven't done yet.
Could you please address them again?  Such as adding homepage, update the deprecated functions or variables and so on.
Also need to look into the failures on CI tests.

@NancyLi1013  ahh yes, I was working on that too, it is just the number of ports is a bit hard to keep track of. Could I get a look in error logs? Cause on my side they build fine on both Windows and Linux, unfortunately I don't have a Mac to test it out.

@hellozee
Thanks for your efforts for this PR. It's really a huge project.
quazip failed on x64-windows-static, due to this:
   Creating library quazip5d.lib and object quazip5d.exp
Qt5Cored.lib(qglobal.obj) : error LNK2019: unresolved external symbol gethostname referenced in function "public: static class QString __cdecl QSysInfo::machineHostName(void)" (?machineHostName@QSysInfo@@SA?AVQString@@XZ)
Qt5Cored.lib(qglobal.obj) : error LNK2019: unresolved external symbol WSAStartup referenced in function "public: __cdecl QWindowsSockInit::QWindowsSockInit(void)" (??0QWindowsSockInit@@QEAA@XZ)
Qt5Cored.lib(qglobal.obj) : error LNK2019: unresolved external symbol WSACleanup referenced in function "public: __cdecl QWindowsSockInit::~QWindowsSockInit(void)" (??1QWindowsSockInit@@QEAA@XZ)
Qt5Cored.lib(qlocale_tools.obj) : error LNK2019: unresolved external symbol "public: static void __cdecl double_conversion::DoubleToStringConverter::DoubleToAscii(double,enum double_conversion::DoubleToStringConverter::DtoaMode,int,char *,int,bool *,int *,int *)" (?DoubleToAscii@DoubleToStringConverter@double_conversion@@SAXNW4DtoaMode@12@HPEADHPEA_NPEAH3@Z) referenced in function "void __cdecl qt_doubleToAscii(double,enum QLocaleData::DoubleForm,int,char *,int,bool &,int &,int &)" (?qt_doubleToAscii@@YAXNW4DoubleForm@QLocaleData@@HPEADHAEA_NAEAH3@Z)
Qt5Cored.lib(qlocale_tools.obj) : error LNK2019: unresolved external symbol "public: double __cdecl double_conversion::StringToDoubleConverter::StringToDouble(char const *,int,int *)const " (?StringToDouble@StringToDoubleConverter@double_conversion@@QEBANPEBDHPEAH@Z) referenced in function "double __cdecl qt_asciiToDouble(char const *,int,bool &,int &,enum StrayCharacterMode)" (?qt_asciiToDouble@@YANPEBDHAEA_NAEAHW4StrayCharacterMode@@@Z)
Qt5Cored.lib(qregularexpression.obj) : error LNK2019: unresolved external symbol pcre2_config_16 referenced in function "public: void __cdecl QRegularExpressionPrivate::getPatternInfo(void)" (?getPatternInfo@QRegularExpressionPrivate@@QEAAXXZ)
Qt5Cored.lib(qregularexpression.obj) : error LNK2019: unresolved external symbol pcre2_match_context_create_16 referenced in function "public: struct QRegularExpressionMatchPrivate * __cdecl QRegularExpressionPrivate::doMatch(class QString const &,int,int,int,enum QRegularExpression::MatchType,class QFlags<enum QRegularExpression::MatchOption>,enum QRegularExpressionPrivate::CheckSubjectStringOption,struct QRegularExpressionMatchPrivate const *)const " (?doMatch@QRegularExpressionPrivate@@QEBAPEAUQRegularExpressionMatchPrivate@@AEBVQString@@HHHW4MatchType@QRegularExpression@@V?$QFlags@W4MatchOption@QRegularExpression@@@@W4CheckSubjectStringOption@1@PEBU2@@Z)
Qt5Cored.lib(qregularexpression.obj) : error LNK2019: unresolved external symbol pcre2_match_context_free_16 referenced in function "public: struct QRegularExpressionMatchPrivate * __cdecl QRegularExpressionPrivate::doMatch(class QString const &,int,int,int,enum QRegularExpression::MatchType,class QFlags<enum QRegularExpression::MatchOption>,enum QRegularExpressionPrivate::CheckSubjectStringOption,struct QRegularExpressionMatchPrivate const *)const " (?doMatch@QRegularExpressionPrivate@@QEBAPEAUQRegularExpressionMatchPrivate@@AEBVQString@@HHHW4MatchType@QRegularExpression@@V?$QFlags@W4MatchOption@QRegularExpression@@@@W4CheckSubjectStringOption@1@PEBU2@@Z)
Qt5Cored.lib(qregularexpression.obj) : error LNK2019: unresolved external symbol pcre2_compile_16 referenced in function "public: void __cdecl QRegularExpressionPrivate::compilePattern(void)" (?compilePattern@QRegularExpressionPrivate@@QEAAXXZ)
Qt5Cored.lib(qregularexpression.obj) : error LNK2019: unresolved external symbol pcre2_code_free_16 referenced in function "public: void __cdecl QRegularExpressionPrivate::cleanCompiledPattern(void)" (?cleanCompiledPattern@QRegularExpressionPrivate@@QEAAXXZ)
Qt5Cored.lib(qregularexpression.obj) : error LNK2019: unresolved external symbol pcre2_pattern_info_16 referenced in function "public: class QStringList __cdecl QRegularExpression::namedCaptureGroups(void)const " (?namedCaptureGroups@QRegularExpression@@QEBA?AVQStringList@@XZ)
Qt5Cored.lib(qregularexpression.obj) : error LNK2019: unresolved external symbol pcre2_match_data_create_from_pattern_16 referenced in function "public: struct QRegularExpressionMatchPrivate * __cdecl QRegularExpressionPrivate::doMatch(class QString const &,int,int,int,enum QRegularExpression::MatchType,class QFlags<enum QRegularExpression::MatchOption>,enum QRegularExpressionPrivate::CheckSubjectStringOption,struct QRegularExpressionMatchPrivate const *)const " (?doMatch@QRegularExpressionPrivate@@QEBAPEAUQRegularExpressionMatchPrivate@@AEBVQString@@HHHW4MatchType@QRegularExpression@@V?$QFlags@W4MatchOption@QRegularExpression@@@@W4CheckSubjectStringOption@1@PEBU2@@Z)
Qt5Cored.lib(qregularexpression.obj) : error LNK2019: unresolved external symbol pcre2_match_16 referenced in function "int __cdecl safe_pcre2_match_16(struct pcre2_real_code_16 const *,unsigned short const *,int,int,int,struct pcre2_real_match_data_16 *,struct pcre2_real_match_context_16 *)" (?safe_pcre2_match_16@@YAHPEBUpcre2_real_code_16@@PEBGHHHPEAUpcre2_real_match_data_16@@PEAUpcre2_real_match_context_16@@@Z)
Qt5Cored.lib(qregularexpression.obj) : error LNK2019: unresolved external symbol pcre2_match_data_free_16 referenced in function "public: struct QRegularExpressionMatchPrivate * __cdecl QRegularExpressionPrivate::doMatch(class QString const &,int,int,int,enum QRegularExpression::MatchType,class QFlags<enum QRegularExpression::MatchOption>,enum QRegularExpressionPrivate::CheckSubjectStringOption,struct QRegularExpressionMatchPrivate const *)const " (?doMatch@QRegularExpressionPrivate@@QEBAPEAUQRegularExpressionMatchPrivate@@AEBVQString@@HHHW4MatchType@QRegularExpression@@V?$QFlags@W4MatchOption@QRegularExpression@@@@W4CheckSubjectStringOption@1@PEBU2@@Z)

vc  failed on arm64-windows, due to this:
FAILED: CMakeFiles/Vc.dir/src/support_x86.cpp.obj
"C:\PROGRA~2\Microsoft Visual Studio\2019\Enterprise\VC\Tools\MSVC\14.26.28801\bin\Hostx64\arm64\cl.exe"   /TP -D_CRT_SECURE_NO_WARNINGS -D__AVX2__ -D__AVX__ -D__BMI2__ -D__FMA__ -D__SSE2__ -D__SSE3__ -D__SSE4A__ -D__SSE4_1__ -D__SSE4_2__ -D__SSSE3__ -ID:\buildtrees\vc\src\1.4.1-c20f654517 /nologo /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP  /wd4290 /wd4800 /wd4996 /wd4244 /wd4146 /wd4227 /wd4722 /wd4748 /D_DEBUG /MDd /Z7 /Ob0 /Od /RTC1    -DVc_COMPILE_LIB /showIncludes /FoCMakeFiles\Vc.dir\src\support_x86.cpp.obj /FdCMakeFiles\Vc.dir\Vc.pdb /FS -c D:\buildtrees\vc\src\1.4.1-c20f654517\src\support_x86.cpp
D:\buildtrees\vc\src\1.4.1-c20f654517\src\support_x86.cpp(49): error C2065: '_XCR_XFEATURE_ENABLED_MASK': undeclared identifier
D:\buildtrees\vc\src\1.4.1-c20f654517\src\support_x86.cpp(49): error C3861: '_xgetbv': identifier not found

D:\buildtrees\vc\src\1.4.1-c20f654517\src\cpuid.cpp(140): error C3861: '__cpuid': identifier not found
D:\buildtrees\vc\src\1.4.1-c20f654517\src\cpuid.cpp(143): error C3861: '__cpuid': identifier not found

libheif failed on x64-linux, due to this:
/mnt/vcpkg-ci/buildtrees/libheif/src/v1.7.0-efb1ff6752/libheif/heif_encoder_x265.cc:36:10: fatal error: x265.h: No such file or directory
 #include <x265.h>
          ^~~~~~~~
compilation terminated.

libde265 failed on x64-uwp, due to this:
4>D:\buildtrees\libde265\src\v1.0.5-4449139dc1\libde265\encoder\algo\tb-transform.cc(125): error C4703: potentially uninitialized local pointer variable 'residual' used
libde265 failed on arm64-windows and arm-uwp, due to this:
D:\buildtrees\libde265\src\v1.0.5-4449139dc1\libde265\x86\sse.cc(45): error C3861: '__cpuid': identifier not found
As for the failure on x64-windows-static for libde265, I didn't find any error info from CI test.

@NancyLi1013 ahh thanks a lot, that cleared a lot of doubts for me, though from first look, it might take a bit of time from my side to get them all resolved, feel free to avoid this till I request a review, 😃

@NancyLi1013 so, I couldn't understand why the build for x64-windows-static is failing, I poked around the azure pipelines and found out the artifacts. The windows build is fine, I don't know why it is complaining, nothing fails. For the linux one, I can't reproduce that locally for now.

As for the failure on x64-windows-static:
-- Performing post-build validation
DLLs should not be present in a static build, but the following DLLs were found:

    D:/packages/libde265_x64-windows-static/bin/libde265.dll
    D:/packages/libde265_x64-windows-static/debug/bin/libde265.dll

There should be no bin\ directory in a static build, but D:\packages\libde265_x64-windows-static\bin is present.
There should be no debug\bin\ directory in a static build, but D:\packages\libde265_x64-windows-static\debug\bin is present.
If the creation of bin\ and/or debug\bin\ cannot be disabled, use this in the portfile to remove them

    if(VCPKG_LIBRARY_LINKAGE STREQUAL "static")
        file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/bin" "${CURRENT_PACKAGES_DIR}/debug/bin")
    endif()

We need to add this to portfile.cmake.
if(VCPKG_LIBRARY_LINKAGE STREQUAL "static")
        file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/bin" "${CURRENT_PACKAGES_DIR}/debug/bin")
endif()

Okay, seemingly both linux and osx fail with the same logs. I think would try to replicate that in a clean box then, :)

@NancyLi1013, tried both with a native linux install and the new wsl2, can't reproduce, probably the CI is doing something which I am not quite aware of, 😫

/azp run

Hi @hellozee
I have tested this on my machine and it can be reproduced now.
libheif build failed due to this:
/Users/vagrant/Data/buildtrees/libheif/src/v1.7.0-efb1ff6752/libheif/heif_encoder_x265.cc:36:10: fatal error: 'x265.h' file not found
#include <x265.h>

I checked the source code and found these codes in CMakeLists.txt
if (UNIX)
  find_package(PkgConfig)
  pkg_check_modules(LIBDE265 libde265)
  pkg_check_modules(X265 x265)
  pkg_check_modules(AOM aom)
endif()

Since x265 cannot be found via find_package(PkgConfig) in vcpkg. We need to use find_library() and find_path().

Okay, so probably ubuntu comes with pkg-config preinstalled. Dang, now I feel stupid to not even notice that it calls pkgconfig, nonetheless it seems to be buildling fine now, 😄

@hellozee
Thanks for your so quick update. It seems nice now except for the newest review suggestions.

Hard work, thanks for this PR!

@ras0219-msft updated the ports with the suggestions. Sorry for the delay, unfortunately playing limbo with graduation is not allowing me to code, 😞

The failures on osx are like this:
CMake Error at /Users/vagrant/Data/installed/x64-osx/share/cmake/Qt5Core/Qt5CoreConfig.cmake:95 (message):
  Library not found: zstd
Call Stack (most recent call first):
  /Users/vagrant/Data/installed/x64-osx/share/cmake/Qt5Core/Qt5CoreConfig.cmake:285 (_qt5_Core_process_prl_file)
  /Volumes/data/work/1/s/scripts/buildsystems/vcpkg.cmake:441 (_find_package)
  CMakeLists.txt:16 (find_package)

This should be fixed in PR #12342.
Could you please merge master to this PR?

The osx port still fails to build, but it is something else this time.

LGTM now. Thanks for your hard work. @hellozee


Accept all-features flag

Many cargo subcommands accept an --all-features flag, which enables all of a crate's features. I have a crate that conditionally includes files if features are set (which subsequently aren't formatted when running cargo fmt). It'd be great if cargo-fmt could accept this flag

Hello. I'm interested in starting to contribute to rustfmt, and I saw that this issue didn't seem to have anyone already looking.
While I'm familiar with git, I haven't gone through the workflows on GitHub lately. If there's anything I've missed or done incorrectly, just let me know and I'll be happy to fix it.
Thanks!
-Angel

AFAIK, rustfmt formats files behind #[cfg(feature = "...")] by default, but files behind cfg_if! are partially supported (#3763 should hopefully fix it, though).

I think that this is already fixed via the support of cfg_if!. Please feel free to re-open the issue if this is not the case.


Fixed openstack/standalone-cinder/pkg/provisioner/provisioner.go:L115 linter error impacting build

I'm seeing a bunch of failures in CI related to this linter error across multiple PRs:
Verifying /home/travis/gopath/src/github.com/kubernetes-incubator/external-storage/repo-infra/verify/go-tools/verify-gometalinter.sh
openstack/standalone-cinder/pkg/provisioner/provisioner.go:115::error: arg err for printf verb %f of wrong type: error (vet)
FAILED   /home/travis/gopath/src/github.com/kubernetes-incubator/external-storage/repo-infra/verify/go-tools/verify-gometalinter.sh	0s

For example, https://travis-ci.org/kubernetes-incubator/external-storage/jobs/288716995

/assign @wongma7
CC: @msau42 @dhirajh

Here's another example, https://travis-ci.org/kubernetes-incubator/external-storage/jobs/288675837

FYI: @aglitke - looks like something passed by the linter on this commit - https://github.com/kubernetes-incubator/external-storage/commit/fbf95b867034b849b04512d21f7d2fdbc500f0ff

/lgtm

Thanks for fixing this @ianchakeres !  Does anyone know how this passed the linter when it was merged?


Tag

Area
UI Components
The problem
Create a Tag component.
Tag is a visual adornment for Cards that is somewhat like a Pill (visually) and enables you to highlight keywords, or related metadata/tags
The solution
Publish to Core components.
Alternatives and examples
n/a
Are you a JPMorgan Chase & Co. employee?

[x] I am an employee of JPMorgan Chase & Co.

### Dependency
- [ ] #2523

Zazu Goal: S(3)

component is in Core
docs yet to be published to core
tag in a card example to be created

Outstanding Question

overriden styles required

Closed by:
https://github.com/jpmorganchase/salt-ds/pull/3515
https://github.com/jpmorganchase/salt-ds/pull/3605
https://github.com/jpmorganchase/salt-ds/pull/3608
https://github.com/jpmorganchase/salt-ds/pull/3624


[stdlib][cmake] OpenBSD target requires -lpthread.

Like FreeBSD, OpenBSD needs -lpthread for a target link. This is
necessary to get the swift-reflection-test to link and build correctly.

@swift-ci Please test

Have you manage to build swift on ether FreeBSD or OpenBSD?

Yes, but there are a few more changes to upstream and caveats before Swift builds on OpenBSD out of the box.

@3405691582 that's awesome. Could you provide a instruction of building on FreeBSD? I could make a port for it, so it's easier for others.

Let's avoid unrelated comments and notifications on the PR. Feel free to look through my commit history; I will document everything when things are more closer to being complete.


Ledvance smart plug fetching and pairing issue

Hello,
I paired Ledvance smart+ wifi plug with its own app on android. Its working ok. I tried tuyapower py. It scans find and outputs all the parameters including plugkey something like "keyj5dhdbjch". When i try to get device info with given parameters, it says no power data. I suppose plugkey is not the actual key?
I reset the device and tried to pair with both smart life and tuya smart app but it fails. During manual pairing with smart life or tuya smart app,  device's led stops blinking but app fails to pair with timeout even though device connects to wifi router.
Appreciate helps. Thanks

Hi @devmet34, thanks for opening this issue. Can you also post a link to the specific device in case others see this?
You are correct.  PlugKey (aka Product Key, Product ID) is essentially a device SKU but is not the local security KEY used to encrypt communication.  When you set up a device using the App (e.g. Smart Life or Tuya Smart), a local key is created and the device is paired with the Tuya Cloud.  That key is stored on the device and in the Tuya Cloud.  The instructions to fetch the key is not simple, unfortunately, but if you have a Tuya compatible plug and you follow the steps it does work: https://github.com/jasonacox/tinytuya#setup-wizard---getting-local-keys
However, I'm not sure your device is compatible.  You could still try the instructions to see if the Ledvance app would work. Unfortunately, it is likely not going to work.  A quick search and I see several post in reddit.com about people not getting Ledvance devices to work with the Tuya echo system.  I do wonder what they are doing.
Has anyone else had any luck with these devices?

Hey Jason, thanks for the quick reply. The model name and link given below. I'll try that method and post the result. If it doesnt work, which i too think it wont work, is there any workaround for getting the key from device. It's probably encrypted so cant sniff it. Maybe ledvance support can help? Or is there any other way to get power info like using ifttt, web api, mqtt etc. with given ledvance cloud credentials?
Device model: AC27825
Official link: https://www.ledvance.com/consumer/products/smart-home/smart-home-products-with-wifi-technology/smart-home-components/plug-with-smart-socket-to-control-non-smart-devices-with-wifi-technology--pairable-with-remote-controller-c6482

Thanks @devmet34 - appreciate the link!
I think you have some good ideas here.  I don't have any of these devices so I don't have a way of testing or helping.  I wish there was a better way to get these KEYs (even for traditional Tuya devices).  Some thoughts I have:

The local KEY may exist on your Android phone so there may be a path to pick up that file in the phone file system if you have a jail-broken android.
You could do some investigation to see if the Ledvance cloud provides API access.  I would search their websites and contact their support to see if they do.  They may even have a way to get the local KEYs similar to Tuya.  I would also ask them if there is a way to use SmartLife App instead and if not, suggest they add that as a feature request.

Thanks again for opening this and keeping us all updated on your discovery!

Ok got the answer it seems like yes its working :)

Awesome!  Thanks @devmet34 !  Can you share the method you used to get the keay?

Thanks @devmet34 !  Great tip.  And just to confirm, you are able to get status and control this Ledvance device with TinyTuya once you had those keys?

Yes, i can get power values with the key.  Cheers


New installer/app icon

For consistency with the TPG brand (and to make it slightly more visually appealing), started work on a new icon in a separate branch https://github.com/ThePacielloGroup/aviewer/tree/new-app-icon
Work is not complete here, as I believe the following still need to be resolved:

[ ] convert all png files into a multi-image ico
[ ] recompile the 32bit and 64bit exe files to use the new icon
[ ] modify the iss to grab the new ico file when creating the final installer

@jun325 if you have any experience in the creation of appropriate multi-image ico files, give me a shout and we can coordinate

For reference, adding the large version and 16x16 version here.

seem to have managed to create a multi-image ico file, but not sure if it's correct https://github.com/ThePacielloGroup/aviewer/commit/4d01995d14ee018c42280876cb3bb9560e713c68

Tested the ico by creating a shortcut to an exe and manually assigning the ico to this shortcut, then resizing the icons in file explorer. At list size, it correctly shows the custom 16x16 version, so assuming the multi-image ico is correct. I've now merged the separate branch into Develop ... @jun325 could you recompile the 32bit and 64bit versions using the new Aviewer_Icon.ico file in the development branch? This should then hopefully result in the nice multi-image version being used for the executable at different sizes, and the custom 16x16 to appear in the application window title bar, I assume. Then I can package this up for next release :)

(fair warning that I'll do a new icon for CCA-Win as well in the coming days)

I changed it.  https://github.com/ThePacielloGroup/aviewer/commit/225fb63f43ed2ca176ec61c0fdea3d868d84d4f4

Perfect, thanks @jun325


I configure push.mp3 in the res/raw directory, server set setSound(push) set SetChannel(channeId), but after receiving push notification.Mp3 is not playing, expect to play push.Mp3 didn't notice the sound of the phone system. In the foreground application, I could use onMessageRecieved, but in the background application, what should I do because the ring tone configured by the server has not been playing,

I configure push.mp3 in the res/raw directory, server set setSound(push) set SetChannel(channeId), but after receiving push notification.Mp3 is not playing, expect to play push.Mp3 didn't notice the sound of the phone system. In the foreground application, I could use onMessageRecieved, but in the background application, what should I do because the ring tone configured by the server has not been playing,

@wangyangke please follow the issue template when filing an issue so that we can help you.  If you can provide all the information in the template, please file a new issue.  If not I would suggest asking on StackOverflow or some other general forum.


python310Packages.flask-restful: remove condition from patch

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Why do you revert 9d32e349d27ade285b468477dc8ca4f0a7244875 just 15 days later and don't even ping the commit author???

You didn't even remove the werkzeug argument.

Why was nixpkgs-review not run???


Project edit:  Error needs a reload ?

Edit a project and introduce an error (missed the closing tag)

The project is forced to reload

The project fails to load and on reload you will get right error message.

Is it possible to inform the error without reload?

@jinujoseph, there is a red squiggle that appears before the file is saved, showing the error. Once the file is saved, it has overwritten the project file on disk. so there's not a heck of lot we can do there. The only way we could make this work is there was an MSBuild api to evaluate a given file and return whether or not it's a valid project.

Dup'ing this against: https://github.com/dotnet/roslyn-project-system/issues/1570 which is tracking this.


Tesseract doesn't recognize multiple languages

If I were to run tesseract page356.png page356 -l eng+osd+ell pdf
It would only recognize the English characters, but produce no errors about other language recognition
If I run tesseract page356.png page356greek -l ell
It recognizes the Greek fine, but now there is no English
If I run tesseract page356.png greekandenglish356 -l ell+eng+osd pdf I get this pdf
greekandenglish356.pdf
only recognizes English
I ran apt-get install tesseract-ocr-all
and I'm experiencing this issue on multiple linux distros
Here is a sample image

AFAIK multiple language support is only for version 4+, not necessarily available in the default repo of the distro

Multiple languages are supported on v3.x

Share a sample image please.

It gives very poor results, but Tesseract 4 is producing a mix of Greek & English. This is also true in PDF output. Problem is not reproducing for me.
$ tesseract -l ell+eng example.png - -

[...]
6- 5- 4- 3- 2- α- β- N- Entry Name
[...]

And I even see some Greek in your attached PDF. There is a β in there.

Changing order of languages usually changes the results.
OP had used eng+ell
On Thu 17 May, 2018, 4:41 AM jbreiden<EMAIL_ADDRESS>wrote:

And I even see some Greek in your attached PDF. There is a β in there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/1579#issuecomment-389694263,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2_ow-mGqTl97cXBdbfL_Ybeda4A8-wks5tzLIPgaJpZM4UBqgR
.

Also, try with the script trained data
https://github.com/tesseract-ocr/tessdata_best/blob/master/script/Greek.traineddata
It should have both Greek and English.

Could be issue be closed?
There is no reaction from original reporter for several months....

zdenop, 30/09/2018 17:19:

There is no reaction from original reporter for several months....

What information is needed? I have the same experience with tesseract
3.05.02, but I don't understand from the responses above whether it's
considered expected result until version 4.

Reported claims that there are only English characters. jbreiden but see there greek alphabets...
Shreeshrii provided several suggestion for testing... No reply...
If reporter does not care why we should? We are not paid for this. If reporter does not value our time and cooperate than I will eliminate such report (if it does not have some value for this project)

Same problem reported in the forum today, but with a custom traineddata for Thai and English.
https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/tesseract-ocr/YLvnrS-01kI/x0PUNGsGBAAJ

As mentioned by the OP in the forum if the input image have both language(eng+thai) in the same line it will read only in 1 language but when having single language in that line it will read in correct language
script/Thai.traineddata seems to give correct result.
ubuntu@tesseract-ocr:~/TEST$ bash ./en_th.sh

 *****  ./en_th.jpg LANG tha+eng TESSDATA tessdata OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
1โล!10 สวัสดีจ้า
This is a test.
นี่คือการทดสอบ

 *****  ./en_th.jpg LANG tha+eng TESSDATA tessdata_best OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
1โล!1๐0 สวัสดีจ้า
This is a test.
นี่คือการทดสอบ

 *****  ./en_th.jpg LANG tha+eng TESSDATA tessdata_fast OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
1โล!10 สวัสดีจ้า
This is a test.
นี่คือการทดสอบ

 *****  ./en_th.jpg LANG eng+tha TESSDATA tessdata OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
1โล!10 สวัสดีจ้า
This is a test.
นี่คือการทดสอบ

 *****  ./en_th.jpg LANG eng+tha TESSDATA tessdata_best OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello aaa
This is a test.
นี่คือการทดสอบ

 *****  ./en_th.jpg LANG eng+tha TESSDATA tessdata_fast OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello ayaa
This is a test.
นี่คือการทดสอบ

 *****  ./en_th.jpg SCRIPT Thai TESSDATA tessdata_best OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello สวัสดีจ้า
This is a test.
นี่คือการทดสอบ

 *****  ./en_th.jpg SCRIPT Thai TESSDATA tessdata_fast OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello สวัสดิจ้า
This is a test.
นี่คือการทดสอบ
DONE

Last(?) change from Ray regarding multi-language mode seems to be
https://github.com/tesseract-ocr/tesseract/commit/b453f74e0194f2cf08e9251b1846a0132657c4f8

Any updates on this issue. I am using Tesseract v4 to detect text "Bœuf Stroganoff" using German and French traindata. Text detection doesn't work when I use traindata for multiple languages together.

You can try Latin.traineddata from best/fast.

@amitdo Tried that. It doesn't work as well. The command I used :
tesseract <input image> <output file> -l lat

-l lat is Latin language.
Use script/Latin which is Latin script and has been trained using all
languages using that script.
On Sat, 6 Oct 2018, 19:02 Nawab Hussain<EMAIL_ADDRESS>wrote:

@amitdo https://github.com/amitdo Tried that. It doesn't work as well.
The command I used :
tesseract   -l lat
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/1579#issuecomment-427612285,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2_o1DBTo47XF2soIHl-5S6W28SQdmjks5uiTZpgaJpZM4UBqgR
.

Please provide a test image. We need to test whether this is a regression .

 This image shows the character set I am targetting
 This image shows the text I was experimenting with as mentioned in the previous comments
Any input on these would be highly appreciated.

Please post an representative image, not ABC demo.

https://github.com/tesseract-ocr/langdata/issues/83#issuecomment-375027879
theraysmith commented on Mar 21

I did have an idea for a better multi-language implementation that would cleanly use models from multiple languages at once, but that depends on getting rid of the old code, and moving the multi-language functionality into the beam search. Until the old code is gone, that would be very messy. …

I can replicate this.
it seems  to me that œ is only trained for French. However, it is not being recognized when French is listed second.
*****  ./euro.png OEM 1 PSM 6 LANG deu+fra TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra+deu TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG eng+fra TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra+eng TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG script/Latin TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bouf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

Results with all three repos, tessdata,  tessdata_fast and tessdata_best
*****  ./euro.png OEM 1 PSM 6 LANG deu+fra TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG deu+fra TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG deu+fra TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra+deu TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra+deu TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra+deu TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG eng+fra TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG eng+fra TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG eng+fra TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra+eng TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra+eng TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra+eng TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG script/Latin TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bouf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG script/Latin TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bouf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG script/Latin TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
*****  ./euro.png OEM 1 PSM 6 LANG fra TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

You already summarized the behavior when lstm is activated.

As mentioned by the OP in the forum if the input image have both language(eng+thai) in the same line it will read only in 1 language but when having single language in that line it will read in correct language

'in 1 language' -> 'in the first given language'

Also, try with the script trained data
https://github.com/tesseract-ocr/tessdata_best/blob/master/script/Greek.traineddata
It should have both Greek and English.

My OCR even became faster by using Devanagari.traineddata, is there any reason for this to happen, also hin+eng was converting a lot of the hindi text to english

@sirius0503 Devanagari was trained with hin+san+mar+nep+eng so it is better at recognition, plus only one traineddata file is used rather than two diff ones.

@Shreeshrii : Surprisingly, it is faster than hin.traineddata

That's due to a difference in the net-spec between the two, which makes Devanagari's network smaller than hin's network.
In addition, like Shree said, in the case of hin you add another network, eng. Devanagari has just one network for both languages.

That's due to a difference in the net-spec between the two, which makes Devanagari's network smaller than hin's network.

@amitdo : When I am using only -l Devanagari instead of  -l hin , I get better speed ( not even hin + eng), Can you explain more about the net-spec difference

https://github.com/tesseract-ocr/tesseract/wiki/VGSLSpecs
https://github.com/tesseract-ocr/tesseract/issues/1404#issuecomment-374680492

Lang
Repo
Height
Lfys
Lfx,Lrx
Lrx

Devanagari
best
48
64
64
512

hin
best
48
64
96
512

Devanagari
fast
36
48
96
192

hin
fast
48
64
96
384

I am struggling to get it to talk to laser. It states cannot find path? Which part am I missing, please?

Shree, can you retest the eng+tha / tha+reng, best/fast, with code from the mater branch?
.

Using the same image used for test in https://github.com/tesseract-ocr/tesseract/issues/1579#issuecomment-426351989 and master code built with disable-legacy:
ubuntu@tesseract-ocr:~/TEST$ bash en_th.sh
tesseract 5.0.0-alpha-473-g6d171
 leptonica-1.78.0
  libgif 5.1.4 : libjpeg 8d (libjpeg-turbo 1.4.2) : libpng 1.2.54 : libtiff 4.0.9 : zlib 1.2.11 : libwebp 0.4.4 : libopenjp2 2.3.0

 *****  ./en_th.jpg LANG tha+eng TESSDATA tessdata OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello สวัสดีจ้า
This is ล test.
นี่คือการทดสอบ
1.66user 0.02system 0:01.70elapsed 99%CPU (0avgtext+0avgdata 81408maxresident)k
0inputs+0outputs (0major+1568minor)pagefaults 0swaps

 *****  ./en_th.jpg LANG tha+eng TESSDATA tessdata_best OEM 1 PSM 3 ****
Warning: Parameter not found: segsearch_max_futile_classifications
Warning: Parameter not found: language_model_ngram_on
Warning: Parameter not found: language_model_ngram_space_delimited_language
Warning: Parameter not found: chop_enable
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello สวัสดีจ้า
This is ล test.
นี่คือการทดสอบ
2.00user 0.03system 0:02.06elapsed 98%CPU (0avgtext+0avgdata 64768maxresident)k
0inputs+0outputs (0major+1936minor)pagefaults 0swaps

 *****  ./en_th.jpg LANG tha+eng TESSDATA tessdata_fast OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello สวัสดีจ้า
This is ล test.
นี่คือการทดสอบ
1.23user 0.00system 0:01.24elapsed 99%CPU (0avgtext+0avgdata 25408maxresident)k
0inputs+0outputs (0major+666minor)pagefaults 0swaps

 *****  ./en_th.jpg LANG eng+tha TESSDATA tessdata OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello สวัสดีจ้า
This is ล test.
นี่คือการทดสอบ
1.64user 0.02system 0:01.67elapsed 99%CPU (0avgtext+0avgdata 78144maxresident)k
0inputs+0outputs (0major+1527minor)pagefaults 0swaps

 *****  ./en_th.jpg LANG eng+tha TESSDATA tessdata_best OEM 1 PSM 3 ****
Warning: Parameter not found: segsearch_max_futile_classifications
Warning: Parameter not found: language_model_ngram_on
Warning: Parameter not found: language_model_ngram_space_delimited_language
Warning: Parameter not found: chop_enable
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello สวัสดีจ้า
This is ล test.
นี่คือการทดสอบ
1.95user 0.02system 0:01.97elapsed 99%CPU (0avgtext+0avgdata 59136maxresident)k
0inputs+0outputs (0major+1667minor)pagefaults 0swaps

 *****  ./en_th.jpg LANG eng+tha TESSDATA tessdata_fast OEM 1 PSM 3 ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Estimating resolution as 376
Hello สวัสดีจ้า
This is ล test.
นี่คือการทดสอบ
1.30user 0.02system 0:01.33elapsed 99%CPU (0avgtext+0avgdata 25472maxresident)k
0inputs+0outputs (0major+625minor)pagefaults 0swaps
DONE
ubuntu@tesseract-ocr:~/TEST$

Hello is now being recognized correctly.
a in This is a test is now being recognized as a Thai character.

Interesting, thanks.

Results for test case in https://github.com/tesseract-ocr/tesseract/issues/1579#issuecomment-428432287 are also changed.

ubuntu@tesseract-ocr:~/TEST$ bash euro.sh

 *****  ./euro.png OEM 1 PSM 6 LANG deu+fra TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG deu+fra TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG deu+fra TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG fra+deu TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG fra+deu TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG fra+deu TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG eng+fra TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG eng+fra TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG eng+fra TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG fra+eng TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG fra+eng TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG fra+eng TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Boeuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG script/Latin TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bouf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG script/Latin TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bouf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG script/Latin TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG fra TESSDATA tessdata ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG fra TESSDATA tessdata_best ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff

 *****  ./euro.png OEM 1 PSM 6 LANG fra TESSDATA tessdata_fast ****
Warning: Invalid resolution 0 dpi. Using 70 instead.
Bœuf Stroganoff
DONE

https://raw.githubusercontent.com/tesseract-ocr/langdata_lstm/master/fra/fra.wordlist
has these words:
Boeuf, boeuf, BOEUF, bœuf, but not Bœuf.
https://raw.githubusercontent.com/tesseract-ocr/langdata_lstm/master/eng/eng.wordlist
has boeuf, but not Boeuf.


Backport OCM-2.4: Fix azure e2e

Managed DNS feature for azure is not working currently. This is a temporary workaround to fix the e2e
Backports #1620

Beauty
/lgtm


Edit variable name in PowerShell code

blobStorageAccountKey variable name without $ sign in PowerShell code caused error. Fixed.

@VuPhong : Thanks for your contribution! The author, @tamram, has been notified to review your proposed change.

@VuPhong - Thanks for finding that!
#sign-off


Changes for daemon-helper to work with systemd

Use systemd when cluster is setup with ceph-deploy or ceph-ansible,  still in testing..
Signed-off-by: Vasu Kulkarni<EMAIL_ADDRESS>

@ktdreyer @athanatos once this works we can replace half of the install task with ceph-deploy and other half with ceph-ansible that will do more customer like testing  for thrash testcases
Currently holding on http://tracker.ceph.com/issues/17050 to be fixed as well, also have to check on what the wait function does once i can reach that stage..

Neat!

@vasukulkarni  i just closed http://tracker.ceph.com/issues/17050 as "rejected".

@tchaikov thanks, that makes sense, will make changes and rerun.

@zmc this is ready for merge and tested.

@vasukulkarni, any ETA on this one?

internal task pr should be ready for monday, soon after that I will pick this one.

@vasukulkarni and I agreed that I should help out with this PR. I'll rebase it and make changes as needed. In the meantime I will make this a DNM as it's not been tested.

@vasukulkarni I've cleaned up the branch. Note:

It is still enabled by default on machines that use systemd. I need to think of how to make it configurable in a sane way
It will probably not work just yet (needs testing), so I'm less immediately concerned about its default state
It needs the changes in https://github.com/zmc/ceph/tree/wip-master-init to run

@zmc awesome thanks!
for 1) we can try to use config at higher level eg: overrides: disable-daemon-helper: true or something so that its easier to run suites with overrides

These are the first 10 tests that I ran with these changes and I have checked logs for few and dont see any  issues 💯 ,  if/when we enable the option to configure it from yaml we could starting enabling it per suite basis.
http://pulpito.ceph.com/vasu-2017-09-29_22:02:40-smoke-luminous-distro-basic-ovh/
2017-09-29T22:23:17.464 INFO:tasks.ceph:Starting mon daemons in cluster ceph...
2017-09-29T22:23:17.464 INFO:teuthology.orchestra.run.ovh081:Running: 'which systemctl'
2017-09-29T22:23:17.620 INFO:teuthology.orchestra.run.ovh081.stdout:/bin/systemctl
2017-09-29T22:23:17.620 INFO:tasks.ceph.mon.a:Restarting daemon using systemd
2017-09-29T22:23:17.620 INFO:teuthology.orchestra.run.ovh081:Running: 'ps -ef | grep "ceph-mon.*--id a" | grep -v grep | awk {\'print $2\'}'
2017-09-29T22:23:17.806 INFO:tasks.ceph.mon.a:starting a non-running daemon
2017-09-29T22:23:17.806 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo systemctl start ceph-mon@a'
2017-09-29T22:23:17.993 INFO:teuthology.orchestra.run.ovh013:Running: 'which systemctl'
2017-09-29T22:23:18.096 INFO:teuthology.orchestra.run.ovh013.stdout:/bin/systemctl
2017-09-29T22:23:18.097 INFO:tasks.ceph.mon.b:Restarting daemon using systemd
2017-09-29T22:23:18.097 INFO:teuthology.orchestra.run.ovh013:Running: 'ps -ef | grep "ceph-mon.*--id b" | grep -v grep | awk {\'print $2\'}'
2017-09-29T22:23:18.255 INFO:tasks.ceph.mon.b:starting a non-running daemon
2017-09-29T22:23:18.256 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo systemctl start ceph-mon@b'
2017-09-29T22:23:18.444 INFO:tasks.ceph.mon.c:Restarting daemon using systemd
2017-09-29T22:23:18.445 INFO:teuthology.orchestra.run.ovh013:Running: 'ps -ef | grep "ceph-mon.*--id c" | grep -v grep | awk {\'print $2\'}'
2017-09-29T22:23:18.604 INFO:tasks.ceph.mon.c:starting a non-running daemon
2017-09-29T22:23:18.605 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo systemctl start ceph-mon@c'
2017-09-29T22:23:18.773 INFO:tasks.ceph:Starting mgr daemons in cluster ceph...
2017-09-29T22:23:18.773 INFO:tasks.ceph.mgr.x:Restarting daemon using systemd
2017-09-29T22:23:18.773 INFO:teuthology.orchestra.run.ovh081:Running: 'ps -ef | grep "ceph-mgr.*--id x" | grep -v grep | awk {\'print $2\'}'
2017-09-29T22:23:18.900 INFO:tasks.ceph.mgr.x:starting a non-running daemon
2017-09-29T22:23:18.900 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo systemctl start ceph-mgr@x'
2017-09-29T22:23:19.068 INFO:tasks.ceph.mgr.y:Restarting daemon using systemd
2017-09-29T22:23:19.068 INFO:teuthology.orchestra.run.ovh013:Running: 'ps -ef | grep "ceph-mgr.*--id y" | grep -v grep | awk {\'print $2\'}'
2017-09-29T22:23:19.182 INFO:tasks.ceph.mgr.y:starting a non-running daemon
2017-09-29T22:23:19.183 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo systemctl start ceph-mgr@y'
2017-09-29T22:23:19.352 INFO:tasks.ceph:Setting crush tunables to default
2017-09-29T22:23:19.353 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo ceph --cluster ceph osd crush tunables default'
2017-09-29T22:23:28.929 INFO:teuthology.orchestra.run.ovh081.stderr:adjusted tunables profile to default
2017-09-29T22:23:28.953 INFO:tasks.ceph:Starting osd daemons in cluster ceph...
2017-09-29T22:23:28.953 INFO:teuthology.orchestra.run.ovh081:Running: "python -c 'import os; import tempfile; import sys;(fd,fname) = tempfile.mkstemp();os.close(fd);sys.stdout.write(fname.rstrip());sys.stdout.flush()'"
2017-09-29T22:23:29.128 INFO:teuthology.orchestra.run.ovh081.stdout:/tmp/tmpELMXtd
2017-09-29T22:23:29.128 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo cp /var/lib/ceph/osd/ceph-0/fsid /tmp/tmpELMXtd'
2017-09-29T22:23:29.291 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo chmod 0666 /tmp/tmpELMXtd'
2017-09-29T22:23:29.717 DEBUG:teuthology.orchestra.remote:ovh081:/tmp/tmpELMXtd is 37B
2017-09-29T22:23:30.094 INFO:teuthology.orchestra.run.ovh081:Running: 'rm -fr /tmp/tmpELMXtd'
2017-09-29T22:23:30.197 INFO:teuthology.orchestra.run.ovh081:Running: "python -c 'import os; import tempfile; import sys;(fd,fname) = tempfile.mkstemp();os.close(fd);sys.stdout.write(fname.rstrip());sys.stdout.flush()'"
2017-09-29T22:23:30.354 INFO:teuthology.orchestra.run.ovh081.stdout:/tmp/tmpjm5VFo
2017-09-29T22:23:30.355 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo cp /var/lib/ceph/osd/ceph-1/fsid /tmp/tmpjm5VFo'
2017-09-29T22:23:30.500 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo chmod 0666 /tmp/tmpjm5VFo'
2017-09-29T22:23:30.926 DEBUG:teuthology.orchestra.remote:ovh081:/tmp/tmpjm5VFo is 37B
2017-09-29T22:23:31.305 INFO:teuthology.orchestra.run.ovh081:Running: 'rm -fr /tmp/tmpjm5VFo'
2017-09-29T22:23:31.412 INFO:teuthology.orchestra.run.ovh013:Running: "python -c 'import os; import tempfile; import sys;(fd,fname) = tempfile.mkstemp();os.close(fd);sys.stdout.write(fname.rstrip());sys.stdout.flush()'"
2017-09-29T22:23:31.540 INFO:teuthology.orchestra.run.ovh013.stdout:/tmp/tmpRkW24A
2017-09-29T22:23:31.541 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo cp /var/lib/ceph/osd/ceph-2/fsid /tmp/tmpRkW24A'
2017-09-29T22:23:31.689 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo chmod 0666 /tmp/tmpRkW24A'
2017-09-29T22:23:32.110 DEBUG:teuthology.orchestra.remote:ovh013:/tmp/tmpRkW24A is 37B
2017-09-29T22:23:32.451 INFO:teuthology.orchestra.run.ovh013:Running: 'rm -fr /tmp/tmpRkW24A'
2017-09-29T22:23:32.555 INFO:teuthology.orchestra.run.ovh013:Running: "python -c 'import os; import tempfile; import sys;(fd,fname) = tempfile.mkstemp();os.close(fd);sys.stdout.write(fname.rstrip());sys.stdout.flush()'"
2017-09-29T22:23:32.679 INFO:teuthology.orchestra.run.ovh013.stdout:/tmp/tmpyKCeG6
2017-09-29T22:23:32.680 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo cp /var/lib/ceph/osd/ceph-3/fsid /tmp/tmpyKCeG6'
2017-09-29T22:23:32.827 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo chmod 0666 /tmp/tmpyKCeG6'
2017-09-29T22:23:33.256 DEBUG:teuthology.orchestra.remote:ovh013:/tmp/tmpyKCeG6 is 37B
2017-09-29T22:23:33.596 INFO:teuthology.orchestra.run.ovh013:Running: 'rm -fr /tmp/tmpyKCeG6'
2017-09-29T22:23:33.698 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo ceph --cluster ceph osd new 7acbb9d6-5c31-4e38-b263-d10e47c81467 0'
2017-09-29T22:23:34.162 INFO:teuthology.orchestra.run.ovh013.stdout:0
2017-09-29T22:23:34.181 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo ceph --cluster ceph osd new cea06024-b985-4071-8f31-6255ec64aeb1 1'
2017-09-29T22:23:34.597 INFO:teuthology.orchestra.run.ovh013.stdout:1
2017-09-29T22:23:34.618 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo ceph --cluster ceph osd new 908f5340-b3c8-4077-afd7-fb39d049848e 2'
2017-09-29T22:23:35.025 INFO:teuthology.orchestra.run.ovh013.stdout:2
2017-09-29T22:23:35.043 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo ceph --cluster ceph osd new 699ab681-3549-4950-8111-733baf792edd 3'
2017-09-29T22:23:35.429 INFO:teuthology.orchestra.run.ovh013.stdout:3
2017-09-29T22:23:35.443 INFO:tasks.ceph.osd.0:Restarting daemon using systemd
2017-09-29T22:23:35.443 INFO:teuthology.orchestra.run.ovh081:Running: 'ps -ef | grep "ceph-osd.*--id 0" | grep -v grep | awk {\'print $2\'}'
2017-09-29T22:23:35.561 INFO:tasks.ceph.osd.0:starting a non-running daemon
2017-09-29T22:23:35.561 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo systemctl start ceph-osd@0'
2017-09-29T22:23:35.776 INFO:tasks.ceph.osd.1:Restarting daemon using systemd
2017-09-29T22:23:35.776 INFO:teuthology.orchestra.run.ovh081:Running: 'ps -ef | grep "ceph-osd.*--id 1" | grep -v grep | awk {\'print $2\'}'
2017-09-29T22:23:35.924 INFO:tasks.ceph.osd.1:starting a non-running daemon
2017-09-29T22:23:35.924 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo systemctl start ceph-osd@1'
2017-09-29T22:23:36.142 INFO:tasks.ceph.osd.2:Restarting daemon using systemd
2017-09-29T22:23:36.142 INFO:teuthology.orchestra.run.ovh013:Running: 'ps -ef | grep "ceph-osd.*--id 2" | grep -v grep | awk {\'print $2\'}'
2017-09-29T22:23:36.232 INFO:tasks.ceph.osd.2:starting a non-running daemon
2017-09-29T22:23:36.232 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo systemctl start ceph-osd@2'
2017-09-29T22:23:36.445 INFO:tasks.ceph.osd.3:Restarting daemon using systemd
2017-09-29T22:23:36.445 INFO:teuthology.orchestra.run.ovh013:Running: 'ps -ef | grep "ceph-osd.*--id 3" | grep -v grep | awk {\'print $2\'}'
2017-09-29T22:23:36.603 INFO:tasks.ceph.osd.3:starting a non-running daemon
2017-09-29T22:23:36.603 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo systemctl start ceph-osd@3'
2017-09-29T22:23:36.829 INFO:tasks.ceph:Waiting for OSDs to come up
2017-09-29T22:23:36.830 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo systemctl show ceph-osd@1 | grep -i state'
2017-09-29T22:23:36.944 INFO:teuthology.orchestra.run.ovh081.stdout:LoadState=loaded
2017-09-29T22:23:36.944 INFO:teuthology.orchestra.run.ovh081.stdout:ActiveState=active
2017-09-29T22:23:36.944 INFO:teuthology.orchestra.run.ovh081.stdout:SubState=running
2017-09-29T22:23:36.945 INFO:teuthology.orchestra.run.ovh081.stdout:UnitFileState=disabled
2017-09-29T22:23:36.945 INFO:teuthology.orchestra.run.ovh081.stdout:StateChangeTimestamp=Fri 2017-09-29 22:23:36 UTC
2017-09-29T22:23:36.945 INFO:teuthology.orchestra.run.ovh081.stdout:StateChangeTimestampMonotonic=687976204
2017-09-29T22:23:36.945 INFO:teuthology.orchestra.run.ovh081:Running: 'sudo systemctl show ceph-osd@0 | grep -i state'
2017-09-29T22:23:37.095 INFO:teuthology.orchestra.run.ovh081.stdout:LoadState=loaded
2017-09-29T22:23:37.096 INFO:teuthology.orchestra.run.ovh081.stdout:ActiveState=active
2017-09-29T22:23:37.096 INFO:teuthology.orchestra.run.ovh081.stdout:SubState=running
2017-09-29T22:23:37.096 INFO:teuthology.orchestra.run.ovh081.stdout:UnitFileState=disabled
2017-09-29T22:23:37.096 INFO:teuthology.orchestra.run.ovh081.stdout:StateChangeTimestamp=Fri 2017-09-29 22:23:35 UTC
2017-09-29T22:23:37.096 INFO:teuthology.orchestra.run.ovh081.stdout:StateChangeTimestampMonotonic=687612366
2017-09-29T22:23:37.096 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo systemctl show ceph-osd@3 | grep -i state'
2017-09-29T22:23:37.215 INFO:teuthology.orchestra.run.ovh013.stdout:LoadState=loaded
2017-09-29T22:23:37.215 INFO:teuthology.orchestra.run.ovh013.stdout:ActiveState=active
2017-09-29T22:23:37.215 INFO:teuthology.orchestra.run.ovh013.stdout:SubState=running
2017-09-29T22:23:37.216 INFO:teuthology.orchestra.run.ovh013.stdout:UnitFileState=disabled
2017-09-29T22:23:37.216 INFO:teuthology.orchestra.run.ovh013.stdout:StateChangeTimestamp=Fri 2017-09-29 22:23:36 UTC
2017-09-29T22:23:37.216 INFO:teuthology.orchestra.run.ovh013.stdout:StateChangeTimestampMonotonic=683024266
2017-09-29T22:23:37.216 INFO:teuthology.orchestra.run.ovh013:Running: 'sudo systemctl show ceph-osd@2 | grep -i state'
2017-09-29T22:23:37.370 INFO:teuthology.orchestra.run.ovh013.stdout:LoadState=loaded
2017-09-29T22:23:37.371 INFO:teuthology.orchestra.run.ovh013.stdout:ActiveState=active
2017-09-29T22:23:37.371 INFO:teuthology.orchestra.run.ovh013.stdout:SubState=running
2017-09-29T22:23:37.371 INFO:teuthology.orchestra.run.ovh013.stdout:UnitFileState=disabled
2017-09-29T22:23:37.371 INFO:teuthology.orchestra.run.ovh013.stdout:StateChangeTimestamp=Fri 2017-09-29 22:23:36 UTC
2017-09-29T22:23:37.371 INFO:teuthology.orchestra.run.ovh013.stdout:StateChangeTimestampMonotonic=682640410
2017-09-29T22:23:37.372 INFO:teuthology.orchestra.run.ovh081:Running: 'adjust-ulimits ceph-coverage /home/ubuntu/cephtest/archive/coverage ceph --cluster ceph osd dump --format=json'

I will also schedule a full smoke suite today

I think we are probably missing some setting, need to check with @jdurgin  on what needs to be done for this to pass
http://qa-proxy.ceph.com/teuthology/vasu-2017-10-03_18:00:35-smoke-luminous-distro-basic-ovh/1699376/teuthology.log
2017-10-03T18:32:34.026 INFO:teuthology.orchestra.run.ovh028:Running: 'sudo adjust-ulimits ceph-coverage /home/ubuntu/cephtest/archive/coverage timeout 120 ceph --cluster ceph osd last-stat-seq osd.0'
2017-10-03T18:32:34.219 INFO:teuthology.orchestra.run.ovh028.stderr:2017-10-03 18:32:34.193614 7f45c6cfd700 -1 WARNING: all dangerous and experimental features are enabled.
2017-10-03T18:32:34.241 INFO:teuthology.orchestra.run.ovh028.stderr:2017-10-03 18:32:34.215396 7f45c6cfd700 -1 WARNING: all dangerous and experimental features are enabled.
2017-10-03T18:32:34.434 INFO:teuthology.orchestra.run.ovh028.stdout:0
2017-10-03T18:32:34.434 INFO:tasks.ceph.ceph_manager.ceph:need seq<PHONE_NUMBER>7
 got 0
 for osd.0

b) http://tracker.ceph.com/issues/21671

I will fix the current cluster configuration to properly test with systemd without overloads and rerun, not really great run with current cluster config here http://pulpito.ceph.com/vasu-2017-10-03_18:00:35-smoke-luminous-distro-basic-ovh/

@zmc  this is ready to be merged!
Tested here: http://pulpito.ceph.com/vasu-2017-10-14_17:53:49-smoke-luminous-distro-basic-ovh/
the cephfs failures are due to distro kernel
I also used the qa branch at : https://github.com/zmc/ceph/tree/wip-master-init , so we should be able to merge both to master.

A better run http://pulpito.ceph.com/vasu-2017-10-17_04:01:21-smoke-luminous-testing-basic-ovh/

@vasukulkarni is there a run that mirrors one of those but uses master branches?

@zmc scheduled here now, just used stable branch before http://pulpito.ceph.com/vasu-2017-10-17_22:00:28-smoke-master-testing-basic-ovh/

@vasukulkarni that's not using master for teuthology :-/

@zmc its not clear what you are asking in comment,  I dont understand why 'master' branch of teuthology needs to be tested?

@zmc ok if you need for comparision here from the nightly runs http://pulpito.ceph.com/teuthology-2017-10-17_05:02:02-smoke-master-testing-basic-ovh/
Additional 2 rgw is due to own qa branch that does not have equivalent branch in s3tests git repo.

@vasukulkarni ah, that's definitely helpful, thanks!

Can t wait for this to be merged! , Please merge when you are done reviewing, I can merge the qa part.

@vasukulkarni because of the way we've done this PR, it won't let you formally review. So if you could give a thumbs up or something, I can review and then we can merge.

:+1:  :+1:  LGTM  and is ready for merge along with qa branch at https://github.com/zmc/ceph/tree/wip-master-init

@vasukulkarni - just filed the ceph PR

@zmc thanks, will merge both once the qa is green.


Add instructions for building on Windows (pull request #236)

Thanks

@championswimmer I think the corresponding patch for build.gradle is still not merged.
Ideally, It should have been a single PR.

@suheb Yeah I have created the pull request for the same here https://github.com/fossasia/open-event-android/pull/236
It wasn't made into a single PR as Arnav said that changes to code and Readme should be in separate PRs


Confusing help text in Pylint extension settings about how to set Pylint args

Repro steps:

Install Pylint extension in VS Code
Open extension settings
Navigate to the "Pylint: Args" section

Result:

Helper text reads:

"Arguments passed to Pylint for linting Python files. Each argument should be provided as a separate string in the array. Examples:
"pylint.args": ["--rcfile="]
"pylint.args": ["--disable=C0111", "--max-line-length=120"]

However, adding "pylint.args": ["--disable=C0111"] and hitting "Add Item" will break the extension. It looks like the cause is that it will inject it into the command line in this way: python -m pylint --reports=n --output-format=json "pylint.args": ["--disable=C0111"] --clear-cache-post-run=y which is malformed and will throw exceptions.InvalidArgsError

Solution:

There are two confusing aspects: one is that you're already in the pylint.args section, so there's no need to specify pylint.args, and the other is that it's expecting each argument as a separated unquoted string
It seems that the hint should instead suggest adding each argument as an unquoted string like --disable=C0111 instead of "pylint.args": ["--disable=C0111"] or "--disable=C0111" so that it's injected into the commandline arguments correctly

@ilya-aby This has nothing to do with Pylint extension. This is more of an issue on the VS Code settings UI, and how the UI expects strings to be provided.
Moving this to VS code for further investigation.


Update test_queue.py as a stateful test

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.66%. Comparing base (91e9f4a) to head (1d43be2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
+ Coverage   88.40%   92.66%   +4.25%
==========================================
  Files          46       46
  Lines        1216     1308      +92
  Branches      161       62      -99
==========================================
+ Hits         1075     1212     +137
+ Misses        133       88      -45
  Partials        8        8

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


Added info about We did not receive the submission

Tried to call out customer gaps which is mostly leading this error condition and thus escalation to us.

@chrisda could you please approve this for me?


Persist our canvas state to Google Realtime API

Yeah the PR isn't ready yet, but I wanted to see the net files changed
across my commits.
Once I clean it out to the point that it's ready for review, I'll comment
to let you guys know.
In the meantime, feel free to check it out though.
On Thu, Dec 22, 2016 at 10:39 PM Nati Tessema<EMAIL_ADDRESS>wrote:

@nati22 commented on this pull request.
In src/client/app/graph/canvas/canvas.component.html
https://github.com/tessa3/uxgraph/pull/130#pullrequestreview-14267219:

@@ -4,5 +4,7 @@
xmlns:xlink="http://www.w3.org/1999/xlink"
(mousedown)="this.onMousedown($event)"
(mousewheel)="this.onMousewheel($event)">

<g uxg-card *ngFor="let card of this.canvasService.cards" [card]="card">

<g uxg-card

*ngFor="let card of (this.canvasService.cards2 | toIterable)"

cards2?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tessa3/uxgraph/pull/130#pullrequestreview-14267219,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvJ7RqWJX_0yOG-vFV9oD7MpGWq-btJks5rK2yFgaJpZM4LUkw_
.

Sounds good. @eyuelt and I will keep lurking

For future reference, you can do this without making a PR. Just add /compare to the repo url, https://github.com/tessa3/uxgraph/compare, and enter the branches or commits you want to compare.
Example: https://github.com/tessa3/uxgraph/compare/master...collaborative-state

Ahh Eyuel how do you understand both Critique and GitHub?? They're both so
confusing 😶
On Thu, Dec 22, 2016 at 11:07 PM Eyuel Tessema<EMAIL_ADDRESS>wrote:

For future reference, you can do this without making a PR. Just add
/compare to the repo url, https://github.com/tessa3/uxgraph/compare, and
enter the branches or commits you want to compare.
Example: master...collaborative-state
https://github.com/tessa3/uxgraph/compare/master...collaborative-state
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tessa3/uxgraph/pull/130#issuecomment-268949252, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABvJ7X31ZBxBFpj2T005-wKG5GqXTBDJks5rK3NMgaJpZM4LUkw_
.

Okay, this PR should be ready for review now.
We can go over it together next time we meet up so that I can really walk it through.

Yeah I think a walkthrough would be helpful for this PR in particular

This PR will close #124, #125 and #129


Cookies fetched/stored incorrectly when path is specified

Hey
We have recently updated to turbolinks but had to revert back.
The problem is we stories cookies on our site with the same name by specifying the path. So colPreference is stored on accounts page with /accounts as a path, and on campaigns page colPreference is stored with /campaigns as a path. So when we fetch cookie we just look up the cookie with colPreference and that works as it always picks the one most apt (basically with the right path).
I am not sure what's happening after turbolinks integration - Is it storing the cookie with the right path, or fetching it incorrectly.
Any help on how to get around this?
P.S - I'm also using jquery turbolinks.

I have the same issue.
I'm using the Bootstrap Tables with cookie extension, and it stores cookies wrong way in everywhere in application:

But it should be only one path: /administrator/users
Without turbolinks it works well. Any idea?

@Shoroh are you resetting cookiePath on page:update? https://github.com/wenzhixin/bootstrap-table/tree/master/src/extensions/cookie#cookiepath

Oops, I didn't. Thanks!


Adds instrumentation bom for pinning a version

Thanks @jorgheymans for the start on this.
Please check my advice is valid cc @shakuzen @devinsba @marcingrzejszczak
Fixes #619

The pickiest of nits: add xml for prettier code highlighting.

good catch anyway!

Good stuff, is  the reporter bom also needed still?

Not sure really. We can wait until someone asks for that and then do it in
zipkin-reporter.
On 27 Feb 2018 3:48 pm, "Jorg Heymans"<EMAIL_ADDRESS>wrote:

Good stuff, is the reporter bom also needed still?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/brave/pull/624#issuecomment-368776392, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD612QJ72ekZap-4acrzvaXiLI8S3QEks5tY7M-gaJpZM4SUVr3
.


Support for weird polymorphic relationship format

Given { some_belongs_to_id: 42, some_belongs_to_type: 'some_type' },
transforms it be conform to what the json-serializer expects, ie
{ some_belongs_to_id: { id:42, type: 'some_type'}
Fixes #18
cc/ @wecc @igorT @lessless

Left a couple of comments, other than that :+1:

@wecc @fivetanley I took the comments in account :smile:


Review All Instances of Invariant("... :C"); Set Correct Cash Symbol

Expected Behavior
Avoid serialization bugs and set the correct string cash symbol manually.
Actual Behavior
In trades tab we're seeing serialization errors from dotnet's automatic currency format.
Potential Solution
Review to use currency helper and source currency from order/order-events.
https://github.com/QuantConnect/Lean/blob/master/Common/Messages/Messages.Orders.cs#L355C70-L355C70
Reproducing the Problem
View backtest json result.

Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

And please review the Order Tags to see which tags we should even fix! In this case above the Order Tag is redundant now and was added to the order in a very early version of LEAN before we had the nice / new orders table


Inline enumerations

Would be nice to have a way to type lua enums which aren't actual enums and just rely on some restricted set of values. For example imagine following lua function:
function sayHello(language)
  if language == 'en' then
    print('Hello')
  elseif language == 'ru' then
    print('Привет')
  end
end

Of course we can provide simple typing like following:
declare function sayHello(language: 'en' | 'ru'): void

But it would be nice to be able to define something like this:
/** @inlineValues */
export enum Language {
  English = 'en',
  Russian = 'ru',
}

declare function sayHello(language: Language): void

https://typescripttolua.github.io/docs/advanced/compiler-annotations#compilemembersonly

https://typescripttolua.github.io/docs/advanced/compiler-annotations#compilemembersonly

That's not it if I understand correctly. My example should emit following code:
sayHello('en')

when used as:
sayHello(Language.English)

but compileMembersOnly will result in:
English = 'en'
sayHello(English)

That's critical difference given the fact that currently there is no way to depend on external typescript libraries (#432) and all external typings must be fully abstract

That's not it if I understand correctly. My example should emit following code:
sayHello('en')

when used as:
sayHello(Language.English)

but compileMembersOnly will result in:
English = 'en'
sayHello(English)

That's critical difference given the fact that currently there is no way to depend on external typescript libraries (#432) and all external typings must be fully abstract

Sounds like you are looking for const enums, see the following example on the playground: https://typescripttolua.github.io/play#code/MYewdgzgLgBApmArgWxgGQIZgOaI9uGAbwCgYYBRHAGwEsIALGAXhgHIE2AaMmAJUQQItLC3YAnRNxIBfEiQAmcYNQzjCAM0RhgUWuBgQMATwAScatRAAKVTjwEAXOiy58cAJTOAbiFoL5IzMLK2tMe3cAOipsOkYPAG4gA

Sounds like you are looking for const enums, see the following example on the playground: https://typescripttolua.github.io/play#code/MYewdgzgLgBApmArgWxgGQIZgOaI9uGAbwCgYYBRHAGwEsIALGAXhgHIE2AaMmAJUQQItLC3YAnRNxIBfEiQAmcYNQzjCAM0RhgUWuBgQMATwAScatRAAKVTjwEAXOiy58cAJTOAbiFoL5IzMLK2tMe3cAOipsOkYPAG4gA

@Perryvw :bow: my bad, not really familiar with typescript :smile:

@Perryvw :bow: my bad, not really familiar with typescript :smile:


Can't fork thang "burl"

If I try to fork "burl" it says "Error saving Burl2: 401: Unauthorized"
and another message in red: "Error: Uncaught Syntax Error: Unexpected Token M
Check the console for more"
console says:"WebSocket connection to 'ws://codecombat.com:9485/' failed: Error in connection establishment: net::ERR_CONNECTION_TIMED_OUT
jquery.js:8623 POST http://codecombat.com/db/thang.type 401 (Unauthorized)
CocoModel.coffee:149 going to log an error message
CocoModel.coffee:150 Error saving burl2 undefined
burl:1 Uncaught SyntaxError: Unexpected token M"
if it fails to save/fork because im unauthorized, there's no problem

I will close this, because the issue has been fixed - @chess9876543210 , you should still be able to reply to @nwinter 's question.


Update train.py

Changed line 56 to log_loss to fix hptune error once deployed

@paulleroyza What kind of issue is this trying to resolve?
Also, since there are multiple files using the same module and parameter, all should be updated if there is a common issue.
https://github.com/search?q=repo%3AGoogleCloudPlatform%2Fasl-ml-immersion SGDClassifier&type=code
Thanks!

Hold off on this for now, it is affecting multiple labs (at least three of
the kfp ones and the basic one), but there is another issue that breaks the
pipeline run which this doesn't solve. I'll work through this an try to
solve both issues.
On Mon, Aug 28, 2023, 03:39 Takumi Ohyama @.***> wrote:

@paulleroyza https://github.com/paulleroyza What kind of issue is this
trying to resolve?
Also, since there are multiple files using the same module and parameter,
all should be updated if there is a common issue.
https://github.com/search?q=repo%3AGoogleCloudPlatform%2Fasl-ml-immersion SGDClassifier&type=code
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/asl-ml-immersion/pull/387#issuecomment-1695453051,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHZHFIIOC5IWJP7AAN334CTXXRYMXANCNFSM6AAAAAA3657LOI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@paulleroyza
Thanks!
I'd appreciate it if you could share the detail of the issue (with error messages if possible).

Moved to Pull https://github.com/GoogleCloudPlatform/asl-ml-immersion/pull/398, there was an issue with my CLA, I have reverted it back to this user now, but it failed the check.


bug fix:when swiped, the component's props.index will not changed,but…

… state.currendIndex is changed,this bug cause view not updated as expected:
for example,
1.set view's props.index to 3
2.scroll to 2
3. set props.index to 3 again
result: the view will be  in page 2  instead of scrolling to 3
so nextProps.index should be compared to state.indexCurrent

You need to control the component (using onChange). Acception this PR will create unpredictible changing index.

This PR shouldn't have passed the test suite. I'm gonna add a test.

I have added a new test regarding this behavior 208bdd9d609a6ab98ce09455b3c8ad62e0d6bf22.