Avalanche v1.4.0 connection stalls

playing in CuteChess (v1.3.0-beta4) GUI the newest compiled Avalanche v1.4.0 has an error :

White just played Qh8+ and Black has only one move, but its connection stalled .. the last move 24.Qh8+ isn't even displayed by CuteChess !?
[ i'm on Xubuntu 22.04 ]

btw. sorry i posted this Issue several times .. i got a 500 error while submitting and tried again .. i closed those extra Issues now ..

and here's the ZIPped log file :
CuteChess-GUI-log.zip

It's alright! I haven't used cutechess for a while so thank you for the issue. I'll take a look when I get home.
Do the compiled binaries in release work? I develop the engine on a Mac (aarch64) and I cross-compiled to other architectures. I haven't implemented a "bench"-like command so the current CI doesn't really test the engine either... so thanks a lot for the issue.

sorry, i mixed up : the error came while using your Avalanche-x86_64-linux-haswell-<IP_ADDRESS> asset .. i'm not able to compile your ZIG code because i have 0.11.0-dev.3031+f40539e5d installed .. i remember in some other Issue you explained how to use a different ZIG version, but i didn't try yet ..

Oh ok. New zig versions don't work because they migrated to a self-hosted compiler after 0.10. I will investigate the Linux binary as it seems the CCRL team ran the windows binaries fine. Thanks 👍

i often try to compile new versions of chess engines .. programmers use many languages .. ZIG is unknown to me, so i prefer to use a default ZIG version on my Linux installation .. same for Rust Nightly : i prefer a default Rust, otherwise i pick the asset binary (if available).
i guess this error is hard to reproduce .. i gut feeling : Avalanche gets 'confused' when having only 1 legal move.

another error : i just let the same Avalanche v14.0 NN play a second game with the same opponent :

here's the ZIPped log : CuteChess-GUI-log2.zip
this time Avalanche had more than one legal moves !?
btw. this game i set ponder ON

Avalanche doesn't support pondering as of right now. Thanks for a second log. Will take a look when I get home.

I cannot reproduce this on my mac. I inputted the same commands and it worked normally... I'll try again in a few days on my WSL.

I'm only using Linux .. these Issues came while using CuteChess v1.3.0-beta4 but i will use v1.2.0 (last stable) to test some more .. maybe this is the cause, i don't know .. until now i had no problems with the CuteChess beta version.

I also just discovered that Avalanche v1.4.0 is not working properly in SCID (v5.0.0) : the engine stops after depth 1 .. i encounter more engines fail to run in SCID .. it seems their UCI protocols don't match .. CuteChess has less such failures ..

There is an issue with the UCI protocol for v1.4.0. I think it is fixed now. I'll release next version as soon as I finish threading.

Sounds good !
I guess it's hard / impossible to reproduce the error .. did my log files help ?

They probably will once I get WSL working. Thanks.

i'm not familiar with WSL .. i just read it's "Windows Subsystem for Linux" .. OK, but what about the "issue with the UCI protocol" ? Do you think the errors are related to Linux ? I don't think so ..

Hi,
Can you try again with the latest release? I fixed a few communication issues and tested on Linux and windows. Sorry for the delay.

having ZIG v0.11.0 .. i just figured out how to use the older ZIG version v0.10.1, and now compiling of the newest cloned source went well : i get a v1.5.0 binary of 2.5 Mb and it runs fine CuteChess !

Great to hear! Thanks for trying out my engine.


Please add the option to close the  component when clicking outside of it

Duplicates

[x] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
it can be simple props hideOnClickOut for .

if this option is specified, than if user click out of  his open props will take the value false

Examples 🌈
No response
Motivation 🔦
No response

Can I work on it? @exp-of-betrayal

It is unclear which drawer you are talking about, but the temporary drawer of both Material UI and Joy UI (in which temporary drawer is the only variant in Joy UI so far), already behaves the way you are looking for.
Material UI drawer: https://mui.com/material-ui/react-drawer/#temporary-drawer
Joy UI drawer: https://mui.com/joy-ui/react-drawer/#introduction


The hash in db/server/users.json stopped working after update from 4.0 to 5.1

Hello together!
I just updated my instance of wireguard-ui from v4.0 to v5.1.
After restarting the webinterface, it is only accessable with the standard-password "admin".
Under version 4.0 I used successfully a hashsum which I put under db/server/users.json like here:
{         "username": "admin",         "password": "",         "password_hash": "HERE IS MY HASHSUM" }
after the update to 5.1 it stopped working
What was changed here?

The DB folder structure was changed. For user db, they are stored under ./db/users/ directory. In your case, it should be something like ./db/users/admin.json which has the following content
{
	"username": "admin",
	"password": "",
	"password_hash": "THE HASH",
	"admin": true
}


route.getLaneletMap() - object has no attribute

Hi,
We're following the examples here - https://github.com/fzi-forschungszentrum-informatik/Lanelet2/tree/master/lanelet2_routing - trying to get routing working in python but the route.getLaneletMap() line is throwing the following error:
AttributeError: 'Route' object has no attribute 'getLaneletMap'

We tried checking what other properties are available and there is a laneletMap property but calling this returns the following:
TypeError: No to_python (by-value) converter found for C++ type: std::shared_ptr<lanelet::LaneletMap const>

Any suggestions?
Thanks,
trav.

Looks like we didn't actually need laneletMap. We're up and running with
shortestPath = route.shortestPath()

But that was a valid issue. I updated the README and fixed the python binding for laneletMap. Thanks!

I have the same issue, what could be problem?


Issue in autocommenting on merged pr #255

I updated the post method in the code and I have also tested this on my local repository and it worked.

can i know how to add links in workflow yml files

We can Simply write like this and it will work


Training on GPU

I had an issue when trying to perform a training run on the GPU, which appeared to be caused by reference and predicted data being stored on different devices leading to errors like RuntimeError: indices should be either on cpu or on the same device as the indexed tensor (cpu).
I can fix this by explicitly allocating the reference data (energies, forces and coords) to the GPU (https://github.com/SimonBoothroyd/descent/blob/92a139604f4b166a6ab040e5e8e8b8a70fa719d8/descent/targets/energy.py#L110):
        energy_ref = entry["energy"].cuda()
        forces_ref = entry["forces"].reshape(len(energy_ref), -1, 3).cuda()

        coords = (
            entry["coords"]
            .reshape(len(energy_ref), -1, 3)
            .detach()
            .requires_grad_(True).cuda()
        )

but likely something smarter is needed that can deal with CPU/GPU runs.

Ah solved by this PR by @jthorton I believe: https://github.com/SimonBoothroyd/descent/pull/72


Deleting cached resizing images

Hi!
I was wondering how does cached image cleanup happen in this implementation. I saw a automaticdelete lambda but Im not exactly sure what this does. If I delete an image in my original source bucket, do I need to delete the corresponding folder of images in the image optimization bucket as well?

From the README:

The transformed image is stored in S3 with a lifecycle policy that deletes it after a certain duration (default of 90 days) to reduce the storage cost.

If cached images are deleted automatically after some time, I wouldn't worry about cleaning up when source object gets deleted, unless it was a business requirement (privacy etc.)

I would like to hear from you your use case before answering. Let's assume that the generated image was corrupted or that you have completely changed the original image and tou want to reflect it to users. In this case, you need to remove all generated images of the image (under same folder), and then invalidate the cache.
If you update an image, but publish it with a new URL (versioning approach), no action is needed from your side. The old image variants will fall of the CloudFront cache, and the corresponding assets in the generated image folder will be deleted by the S3 lifecycle policy to avoid incurring cost.

@achrafsouk As of right now, I change the original image in the bucket, so I guess I would need to remove all the generated images and invalidate the cache. However, I'd prefer not to do that so I think Ill switch it so that I publish it with a new url and delete the old image from the source bucket. Thanks!

Awesome!


Filtering redundant annotations is very slow

The "filtering redundant annotations" step of the Chado load (for filtering GO annotation) is very slow - it takes more than 2 hours.
Original comment by: kimrutherford

@kimrutherford is this still an issue?

is this still an issue?

Yep.

Ah, talking to yourself...

Yep.

Nope.
Fixed by 2009421d2b5f075c in 2014.  :-)


metrics: allow suppressing curl logs

Thank you for contributing to the Cohere Toolkit!

[ ] PR title: "area: description"

Where "area" is whichever of interface, frontend, model, tools, backend, etc. is being modified. Use "docs: ..."  for CI changes.
Example: "deployment: add Azure model option"

[ ] PR message: Delete this entire checklist and replace with

Description: a description of the change
Issue: the issue # it fixes, if applicable
Dependencies: any dependencies required for this change

[ ] Add tests and docs: Please include testing and documentation for your changes

[ ] Lint and test: Run make lint and make run-tests

AI Description

The pull request introduces changes to the src/backend/services/metrics.py file, specifically to the report_metrics function.
The updates involve adding a check for the METRICS_LOGS_CURLS environment variable before logging the signal curl. This enhancement ensures that the signal curl is logged only when the METRICS_LOGS_CURLS variable is set to "true".
Code Changes:

Addition of METRICS_LOGS_CURLS = os.getenv("METRICS_LOGS_CURLS", None) to capture the environment variable.
Modification of the report_metrics function to include a conditional check for METRICS_LOGS_CURLS before calling log_signal_curl.

Codecov Report
Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 85.99%. Comparing base (9125660) to head (a5a7e6e).
Report is 9 commits behind head on main.

Files
Patch %
Lines

src/backend/services/metrics.py
66.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #372      +/-   ##
==========================================
- Coverage   86.46%   85.99%   -0.47%
==========================================
  Files         178      178
  Lines        6873     7014     +141
==========================================
+ Hits         5943     6032      +89
- Misses        930      982      +52

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


Question about the find_ancs.py？

Hello, after runing the find_ancs.py, I'm not sure the texts in atrs.json is sorted in ascending order or descending order？Could you tell me the details？

Hello, the higher the ranking, the higher the possibility of entanglement. For example, the 1st one is the predicted most possibly entangled attribute.

Sorry, I also want to know whether the texts I get in the atrs.json is sorted in ascending order or descending order？

The 1st text is the 1st possibly entangled attribute, the 2nd is the 2nd possibly entangled attribute, and so on...
Hope this resolves your issue.


DISCOVERY: Restructure doc (basely)

@azinit: Decompose if needed!

Description

[ ] Swap specific articles
[ ] Merge/Split specific articles
[ ] Update redirects
[ ] Update both locales (ru, en)
[ ] Reorder some items if needed (e.g. - concepts by difficulty)
[ ] Group some items if needed (promo & history & mission ...)
[ ] (Maybe) Add old decompose view for compatability

TODO

Legend:

🔎 TUTORIAL (get started)
🛠 GUIDE (guides & examples)
🧩 EXPLANATION (concepts)
📚 UNDERSTANDING (reference)
🍰 ABOUT

🔎 int/Promotion at company =>  🍰 /promo/Promotion at company
🔎 int/Integration at team, 🔎 int/Partial application  =>  🍰 /promo/Integration at team
🔎 int/Cheatsheet, 🔎 Decompose logic => 🔎 Cheatsheet
🔎 int/Adv & Disadv => 🍰 /Adv & Disadv
🔎 tut/Decompose logic, 🔎 tut/Decompose design layouts => 🔎 Decomposing logic
🔎 tut/QuickStart => 🔎 QuickStart
🔎 Alternatives/* => 🍰 /Alternatives/*
🔎 Motivation => 🔎 Motivation (basely),  🍰 Motivation (completely)
🧩 Needs-driven => 🧩Domain-first, 🧩Needs-driven
🧩 Naming adaptivity => 🧩Naming
🧩 App splitting => 🧩Decomposition
🛠 Decouple entities =>  🧩  Decouple entities (FIXME)
🛠 Low Coupling & High Cohesion =>  🧩  Isolation
🛠 /White-labels =>🛠 /examples/White-labels
🛠 /Monorepo =>🛠 /examples/Monorepo
🛠 Handbook/* => 🧩 Handbook/* (TODO: and change name)
🍰 Community, 🍰 Team => 🍰 Team

Хотя уже понятно, что в процессе придется щас перелопатить чутка реструктуризацию 🤔


No background color for file selection

Loving the theme btw! I don't seem to see any background color when trying to select or link to a file. E.g., when linking to a new note, the autocomplete that pops up doesn't show which file I am hovering over or when using arrow keys so it's hard to select the right note. Or when using the cmd+O file switcher.

Fixed in latest commit, I'm now using the highlight color. Thanks for the suggestion.


Add dask (e.g. via Dask delayed) as a possible workflow engine

What new feature would you like to see?
Or at least prove it can't be done easily...

So, all @jobs are equivalent to @delayeds, and @flow isn't needed by Dask. The issue is what to do with subflows.
import dask
from dask.distributed import Client
from dask import delayed

client = Client()

@delayed
def add(a, b):
    return a + b

@delayed
def make_more(val):
    return [val] * 3

@delayed
def add_distributed(vals, c):
    return [add(val, c) for val in vals]

delayed1 = add(1, 2)
delayed2 = make_more(delayed1)
delayed3 = add_distributed(delayed2, 3)
dask.compute(delayed3)

This will return a Tuple[List] where List contains 3 Delayed object instead of the desired [6, 6, 6].


Develop

:tada: This PR is included in version 2.1.9 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update GojekPay.php

fixing error login must wait 12 hours

the device id is manual change by user, not by script, i know about limitation, so you must change it manually

is there any example how to change device id manually?

you can manual generate by calling self::uuidv4()
then copy to script
or use online generator
https://www.uuidgenerator.net/


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

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

Project coverage is 95.69%. Comparing base (afc0822) to head (1a75af5).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #915   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

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


chore(defaults): Update webpack-defaults and move it to devDependencies

This PR move webpack-defaults to devDependencies so consumer of this plugin don't have the npm script added to their package.json by the post install script.
While I was there, I took the opportunity to bump it to its last version. Tell me if it's something you don't want and I'll ditch this part.
Thanks for the plugin in any case!

Oh wow I'm so sorry I'm still catching up with my emails here, I just added @samccone as a Collab so he might be able to help review etc. Otherwise, I'll

No need to apologize. Thanks for all the good work!


Program fails to build if windows.h is included after toml++.

Environment
toml++ version and/or commit hash:  dca69453f61700761c6798cb0b8b685861e01b2a

Compiler:  MSVC 19.28.29915

C++ standard mode (e.g. 17, 20, 'latest'):  latest/20

Target arch (e.g. x64):  x64

Library configuration overrides:  -

Relevant compilation flags:  -

Describe the bug

Including windows.h after toml++/toml.h (but not the other way around) causes the build to fail:
Error C2733: MultiByteToWideChar/WideCharToMultiByte: you cannot overload a function with extern "C" linkage.
Steps to reproduce (or a small repro code sample)

#include <toml++/toml.h>
#include <windows.h>

int main()
{
	return 0;
}

Library is added as a cmake subproject and built at the same time.
Additional information

The issue happens in this header: toml_default_formatter.hpp, at line 205. I managed to get around this error by placing the declarations in the root namespace and guard them with a macro:
#ifndef _WINDOWS_
extern "C"
{
    int __stdcall WideCharToMultiByte(unsigned int CodePage, unsigned long dwFlags, const wchar_t* lpWideCharStr, int cchWideChar,
                                        const char* lpMultiByteStr, int cbMultiByte, const char* lpDefaultChar, int* lpUsedDefaultChar);
    int __stdcall MultiByteToWideChar(unsigned int CodePage, unsigned long dwFlags, const char* lpMultiByteStr, int cbMultiByte,
                                    const wchar_t* lpWideCharStr, int cchWideChar);
}
#endif

Though I'm not sure how it'd be fixed without removing the winapi namespace.

Ah, good catch! I think your solution is a good one; them being in a namespace isn't overly critical.

Fixed in master. Thanks @ImTouchk!

@marzer I don't understand how what you did here fixes the problem?
For me the build for my project now fails with error C2375 while it worked flawlessly before this change:
1>------ Build started: Project: TOMLConfig, Configuration: Debug Win32 ------
1>TOMLConfig.cpp
1>C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\um\stringapiset.h(126,1): error C2375: 'MultiByteToWideChar': redefinition; different linkage
1>C:\Work\Projects\PS\CameraService\3rdParty\toml++\toml.hpp(12113): message : see declaration of 'MultiByteToWideChar'
1>C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\um\stringapiset.h(141,1): error C2375: 'WideCharToMultiByte': redefinition; different linkage
1>C:\Work\Projects\PS\CameraService\3rdParty\toml++\toml.hpp(12103): message : see declaration of 'WideCharToMultiByte'
1>Done building project "TOMLConfig.vcxproj" -- FAILED.

Your definition for those functions is also incorrect as it omits both SAL annotations and WINBASEAPI qualifier (which expands to __declspec(dllimport)), resulting in the above error.
I would prefer if you don't copy function definitions from system includes, but instead include appropriate header file and rely on their existing guards to avoid function redefinitions or if you simply state in documentation and code examples that <windows.h> should be included before toml++.hpp on Windows platform.

@levicki

I don't understand how what you did here fixes the problem?

It doesn't fix your problem. It fixes the problem identified in the issue as reported by @ImTouchk. The thing you're experiencing is a different (but related) issue.

Your definition for those functions is also incorrect as it omits both SAL annotations and WINBASEAPI qualifier (which expands to __declspec(dllimport)), resulting in the above error.

Not including SAL annotations has no impact on it's correctness as far as C++ is concerned. They're preprocessor-only hints for tooling (and for the poor humans forced to read them). You're right that it should have WINBASEAPI specifier, though. That is the actual problem here. Interesting that this never came up as an issue for me, even though I use the library on Windows too. Perhaps compiler differences are to blame? I'd need to dig into it to figure out why. That's very odd.

I would appreciate if you do not copy function definitions from platform header files but instead include appropriate platform header file and rely on their existing guards to avoid function redefinitions or if you simply state in documentation and code examples that windows.h should be included before toml++.hpp on Windows platform.

I'd rather not, if I can avoid it. If the library must always be paired with windows.h on Windows, then the library should just include Windows.h itself, as it did originally. The idea behind the change was to avoid the enormous size of Windows.h; I've seen that in a 'blank-slate' project simply including Windows.h can cause compilation speed slowdowns of 500ms - 1s per translation unit, which is not ideal. If there's a way to avoid that for the 2 functions I need from Windows.h then I'd like to make use of it. Given that the vast majority of Windows.h is a very very stable API, in most cases it's generally safe to do this sort of thing (as long as it's actually correct, and mine clearly is not!).

If you are concerned about namespace pollution, there are a lot of macros that can be defined prior to including windows.h that will reduce it

Those are useless for a library; if I use them then either my library is the first one to include Windows.h and anyone including it after me would be getting the minimal version of Windows.h that is likely to step on what APIs they might need themselves, or my library isn't the first to include Windows.h and they accomplish nothing.

Not including SAL annotations has no impact on it's correctness as far as C++ is concerned.

If I am not mistaken SAL annotations are there to enable compiler to verify correctness of the code calling the function at compile time? If you are sure your code doesn't need them that's good for you.
However, introducing SAL-less definitions that replace windows.h original function definitions will prevent correctness verification in every other code file which also calls those functions where your header file is included. Are you sure that's a good idea?

You're right that it should have WINBASEAPI specifier, though. That is the actual problem here. Interesting that this never came up as an issue for me, even though I use the library on Windows too. Perhaps compiler differences are to blame? I'd need to dig into it to figure out why. That's very odd.

I disagree with what you think the actual problem is -- actual problem is not including windows.h when you need it in your code, and the requirements for using those functions are very clear:

The idea behind the change was to avoid the enormous size of Windows.h; I've seen that in a 'blank-slate' project simply including Windows.h can cause compilation speed slowdowns of 500ms - 1s per translation unit even on a very modern PC, which is not ideal.

Is that with or without using precompiled headers?
Regardless, most projects on Windows platform will have to include windows.h anyway, so the slowdown associated with it cannot really be completely avoided. I think this change was a pointless "optimization" -- you can already avoid recompilation of stable header files by using precompiled header create and use compiler options (/Yc and /Yu) which work very well.
If you are really not comfortable with including windows.h in toml.hpp despite the clear requirements, then why not simply do this:
#ifndef _WINDOWS_
#error Please include windows.h before toml.hpp, preferrably in your project's precompiled header file (stdafx.h or pch.h).
#endif

Given that the vast majority of Windows.h is a very very stable API, in most cases it's generally safe to do this sort of thing (as long as it's actually correct, and mine clearly is not!).

On the contrarry, given that it is a stable API it belongs in the project's stdafx.h or pch.h file. It's not your problem that inexperienced Windows developers don't know that, and IMO you shouldn't pander to them.

Those are useless for a library; ...

I agree, that's why I said I wouldn't recommend that approach.

However, introducing SAL-less definitions that replace windows.h original function definitions will prevent correctness verification in every other code file which also calls those functions where your header file is included. Are you sure that's a good idea?

No it won't. Attributes, annotations etc are cumulative. A forward declaration can omit them, and have them added later by a full definition. My code uses those two functions correctly so I don't benefit from the annotations, but me doing this won won't impact SAL for anyone who is including Windows.h for their own use since the SAL will.

If you are really not comfortable with including windows.h in toml.hpp despite the clear requirements, then why not simply do this [...]

Because that would impose an ordering requirement on #include directives which is almost always a bad idea.

I disagree with what you think the actual problem is -- actual problem is not including windows.h when you need it in your code, and the requirements for using those functions are very clear

I've already outlined my reasoning here and do not agree with you about the rest of it. I'll fix the issue by adding the missing declaration.

@levicki your issue should now be fixed in ba75446. Additionally you can use the (undocumented) switch TOML_INCLUDE_WINDOWS_H to override; set it to 1 if you want to explicitly include windows.h, or 0 if you want to use the (now fixed) explicit forward declarations. Works locally, but evidently your mileage may vary. Let me know if encounter any issues.

You complain that people don't read documentation, and then you cater to those same people by removing the need to read via hacky workarounds such as this one.  I find that really disappointing because it is teaching them the wrong thing.
The proper solution for Windows platform is #error directive that instructs them to include windows.h in a precompiled header which is always included first in any compilation unit so it will naturally come before toml.hpp, and which dramatically reduces compile time as a bonus.
That would teach them to use the platform toolset properly, while this is teaching them that it's OK to copy & paste stuff from random header files (as if copying from Stack Overflow wasn't bad enough).
In this particular case it might be benign because API is (mostly) stable, but people extrapolate weirdest things and next thing you know they will copy declarations from STL or Boost or God knows what other 3rd party library where API isn't that stable and when someone else on a team later updates said library all hell will break loose. Your fix is not technically wrong, but IMO it is setting a bad example.

Good for you, but it's my library so it's up to me to decide the nature of the issue.

Now I feel bad for putting you on a defensive.
I apologize for even bringing this up, obviously I was mistaken in my belief that you value my input so I will refrain from reporting and / or commenting on any further issues.

@levicki I value input, but your way of communicating it has come to be oddly entitled somehow, as though you're my customer and I owe you something, and it is very demotivating. Your initial communications with me were great but more recently it feels like you've come to act as though you have some ownership of this project, and be more pushy/insistent accordingly. If you can drop that nonsense I'll be more engaged, otherwise please do move on.

@marzer I apologize if I am coming through like that, I can be rather passionate when discussing the future of something I use and like (and I do like your library a lot so let me use this opportunity to thank you again for writing it and allowing us to use it).
However, even if I contributed code instead of just ideas (and opinions like in this case), I would have never considered myself as having any special rights to your work, not even customer relationship, much less ownership. If that's what you are getting from my feedback then either I am not communicating clearly enough, or there must be some sort of language barrier given how English is not my primary language, or perhaps it is just difference in our temperaments.


fix position access error

position is a subproperty of _value,cash and now_account_value belongs to  _value.position
maybe error since:https://github.com/microsoft/qlib/pull/1120

Thanks for your contributions!
Please fix the CI error.

Thanks for your contribution.
This reflex another flaw of our CI that it can't detect the errors in the report.
So I think it would be better to add such a check in our CI. (It may be some command like jupyter nbconvert  workflow_by_code.ipynb)
Would you like to contribute to this feature in this PR?
Otherwise, @SunsetWolf lf will help with this feature.

I have fix the CI error.
@you-n-g

Thanks for your


Game Complete - Screen

AC:

Countdown timer is now showing "Game Over"
Winner modal is displayed

Zeplin:
https://zpl.io/1hcgHT

Looks good based on the AC, moving to design review.


Create a Potion object

Create tests for the Potion object.
The Potion object will be used to give the player stat boosts.
If no stat is provided, the stat should be randomly selected.

Integrated under: commit 269670d315003a30182544bb2db6bf56c20d8f7f .


Error in data$construct() :    lgb.Dataset.construct: cannot create Dataset handle

Tried solutions described here: https://github.com/Microsoft/LightGBM/issues/571
Did not work. Re-installed lightgbm, re-started R, used lgb.unloader and rec'd same error.  Any help is appreciated--thanks!
Environment info
Operating System: Windows 7 Professional
CPU/GPU model: Intel(R) Core(TM) i3-4150 CPU @ 3.50 GHz 3.50 GHz
C++/Python/R version: RStudio Version 1.1.456
Error message
Error in data$construct() :
lgb.Dataset.construct: cannot create Dataset handle
Reproducible examples
require(lightgbm)

require(methods)
data(agaricus.train, package = "lightgbm")
data(agaricus.test, package = "lightgbm")
train <- agaricus.train
test <- agaricus.test
print("Training lightgbm with sparseMatrix")
[1] "Training lightgbm with sparseMatrix"
bst <- lightgbm(data = train$data,

            label = train$label,

            num_leaves = 4,

            learning_rate = 1,

            nrounds = 2,

            objective = "binary")

Error in data$construct() :
lgb.Dataset.construct: cannot create Dataset handle

lgb.unloader(wipe = TRUE)
library(lightgbm)
bst <- lightgbm(data = train$data,

            label = train$label,

            num_leaves = 4,

            learning_rate = 1,

            nrounds = 2,

            objective = "binary")

Error in data$construct() :
lgb.Dataset.construct: cannot create Dataset handle
Steps to reproduce

ping @jameslamb and @Laurae2

Thank you for the reproducible example! I will see if I can reproduce it on my machine and look for a fix

Great! Thanks for your help.
Dave
On Sun, Oct 7, 2018, 7:32 PM James Lamb<EMAIL_ADDRESS>wrote:

Thank you for the reproducible example! I will see if I can reproduce it
on my machine and look for a fix
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/LightGBM/issues/1730#issuecomment-427696379,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AmUoCH03EJIDMf6WmsoDkdOpUcWARjT4ks5uio8JgaJpZM4XGRLi
.

Looking into this today. Will update shortly

Ok I ran the code from above on my Mac and could not reproduce the issue.
library(lightgbm)
library(methods)

data(agaricus.train, package = "lightgbm")
data(agaricus.test, package = "lightgbm")

train <- agaricus.train
test <- agaricus.test

print("Training lightgbm with sparseMatrix")

bst <- lightgbm(
    data = train$data,
    label = train$label,
    num_leaves = 4,
    learning_rate = 1,
    nrounds = 2,
    objective = "binary"
)

I got this result:
[LightGBM] [Warning] Starting from the 2.1.2 version, default value for the "boost_from_average" parameter in "binary" objective is true.
This may cause significantly different results comparing to the previous versions of LightGBM.
Try to set boost_from_average=false, if your old models produce bad results
[LightGBM] [Info] Number of positive: 3140, number of negative: 3373
[LightGBM] [Info] Total Bins 128
[LightGBM] [Info] Number of data: 6513, number of used features: 107
[LightGBM] [Info] [binary:BoostFromScore]: pavg=0.482113 -> initscore=-0.071580
[LightGBM] [Info] Start training from score -0.071580
[1]:	train's binary_logloss:0.198597
[2]:	train's binary_logloss:0.111535

Thanks for looking into this for me--any idea of what my issue might be or
what I should try next? Thanks!
On Sat, Oct 13, 2018 at 12:40 PM James Lamb<EMAIL_ADDRESS>wrote:

Ok I ran the code from above on my Mac and could not reproduce the issue.
library(lightgbm)
library(methods)
data(agaricus.train, package = "lightgbm")
data(agaricus.test, package = "lightgbm")
train <- agaricus.train
test <- agaricus.test
print("Training lightgbm with sparseMatrix")
bst <- lightgbm(
data = train$data,
label = train$label,
num_leaves = 4,
learning_rate = 1,
nrounds = 2,
objective = "binary"
)
I got this result:
[LightGBM] [Warning] Starting from the 2.1.2 version, default value for the "boost_from_average" parameter in "binary" objective is true.
This may cause significantly different results comparing to the previous versions of LightGBM.
Try to set boost_from_average=false, if your old models produce bad results
[LightGBM] [Info] Number of positive: 3140, number of negative: 3373
[LightGBM] [Info] Total Bins 128
[LightGBM] [Info] Number of data: 6513, number of used features: 107
[LightGBM] [Info] [binary:BoostFromScore]: pavg=0.482113 -> initscore=-0.071580
[LightGBM] [Info] Start training from score -0.071580
[1]:	train's binary_logloss:0.198597
[2]:	train's binary_logloss:0.111535
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/LightGBM/issues/1730#issuecomment-429556287,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AmUoCGQVlDRNNqRFTPZb0AO80i_1hxj1ks5ukheZgaJpZM4XGRLi
.

@dwh2102 Now that I've got it working on my Mac, the next thing I'll try (probably tonight) is to build on a Windows machine.
I was looking through the construct() code last night and there are about different places this could have failed. If nothing else, I'll add logging in there so we can see which block executes for you. Then if that doesn't help us find an answer we can pull in @Laurae2 to help.
We won't abandon you! Thanks for your patience.

Great! Thank you so much.
On Mon, Oct 15, 2018 at 11:34 AM James Lamb<EMAIL_ADDRESS>wrote:

@dwh2102 https://github.com/dwh2102 Now that I've got it working on my
Mac, the next thing I'll try (probably tonight) is to build on a Windows
machine.
I was looking through the construct() code last night and there are about
different places this could have failed. If nothing else, I'll add logging
in there so we can see which block executes for you. Then if that doesn't
help us find an answer we can pull in @Laurae2
https://github.com/Laurae2 to help.
We won't abandon you! Thanks for your patience.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/LightGBM/issues/1730#issuecomment-429902882,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AmUoCDwbxvRL4LU7YrnLg9cfdMEYPgpoks5ulKsFgaJpZM4XGRLi
.

@jameslamb Windows 10, R 3.5.0, LightGBM master:
R version 3.5.0 (2018-04-23) -- "Joy in Playing"
Copyright (C) 2018 The R Foundation for Statistical Computing
Platform: x86_64-w64-mingw32/x64 (64-bit)

R is free software and comes with ABSOLUTELY NO WARRANTY.
You are welcome to redistribute it under certain conditions.
Type 'license()' or 'licence()' for distribution details.

R is a collaborative project with many contributors.
Type 'contributors()' for more information and
'citation()' on how to cite R or R packages in publications.

Type 'demo()' for some demos, 'help()' for on-line help, or
'help.start()' for an HTML browser interface to help.
Type 'q()' to quit R.

> library(lightgbm)
Loading required package: R6
> library(methods)
>
> data(agaricus.train, package = "lightgbm")
> data(agaricus.test, package = "lightgbm")
>
> train <- agaricus.train
> test <- agaricus.test
>
> print("Training lightgbm with sparseMatrix")
[1] "Training lightgbm with sparseMatrix"
>
> bst <- lightgbm(
+     data = train$data,
+     label = train$label,
+     num_leaves = 4,
+     learning_rate = 1,
+     nrounds = 2,
+     objective = "binary"
+ )
Loading required package: Matrix
[LightGBM] [Warning] Starting from the 2.1.2 version, default value for the "boost_from_average" parameter in "binary" objective is true.
This may cause significantly different results comparing to the previous versions of LightGBM.
Try to set boost_from_average=false, if your old models produce bad results
[LightGBM] [Info] Number of positive: 3140, number of negative: 3373
[LightGBM] [Info] Total Bins 128
[LightGBM] [Info] Number of data: 6513, number of used features: 107
[LightGBM] [Info] [binary:BoostFromScore]: pavg=0.482113 -> initscore=-0.071580
[LightGBM] [Info] Start training from score -0.071580
[1]:	train's binary_logloss:0.198597
[2]:	train's binary_logloss:0.111535
> sessionInfo()
R version 3.5.0 (2018-04-23)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows >= 8 x64 (build 9200)

Matrix products: default

locale:
[1] LC_COLLATE=English_United States.1252  LC_CTYPE=English_United States.1252    LC_MONETARY=English_United States.1252 LC_NUMERIC=C
[5] LC_TIME=English_United States.1252

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] Matrix_1.2-14  lightgbm_2.2.1 R6_2.2.2

loaded via a namespace (and not attached):
[1] compiler_3.5.0    magrittr_1.5      tools_3.5.0       yaml_2.1.19       grid_3.5.0        data.table_1.11.6 jsonlite_1.5      lattice_0.20-35

The plot thickens!
@Laurae2 I'm going to make the PR to improve logging anyway, then maybe if @dwh2102 can run and reproduce the problem, we'll be able to form a better hypothesis about where something went wrong.

Hi, I have the same problem.
System: windows 7
library: I compile the source code with VS 2017 to generate a .dll file and then compile to a zip file
Then I install the package through the zip file from RStudio, and run the code
`> library(lightgbm, quietly=TRUE)

data(agaricus.train, package = "lightgbm")
train <- agaricus.train
dtrain <- lgb.Dataset(train$data, label = train$label)
Loading required package: Matrix
lgb.Dataset.construct(dtrain)
Error in dataset$construct() :
lgb.Dataset.construct: cannot create Dataset handle
`
Maybe there are some problems during the compiling and installation of the package. I remember after I run the R CMD INSTALL command, it shows "DONE", however  I cannot find any function in the lightgbm library (I use lightgbm:: and there are only 3 datasets without functions). Then I make the directory into zip file and then install the zip file, and then find the functions at last. However, when I run the code above it shows the problem.

Could you use my .zip file and check what the problem is? Thank you,
lightgbm_2.2.2.zip

Hi, I have the same problem.
System: windows 7
library: I compile the source code with VS 2017 to generate a .dll file and then compile to a zip file
Then I install the package through the zip file from RStudio with Windows R Open 3.5.1
platform x86_64-w64-mingw32
arch x86_64
os mingw32
system x86_64, mingw32
status
major 3
minor 5.1
year 2018
month 07
day 02
svn rev 74947
language R
version.string R version 3.5.1 (2018-07-02)
nickname Feather Spray
, and run the code
> library(lightgbm, quietly=TRUE)
> data(agaricus.train, package = "lightgbm")
> train <- agaricus.train
> dtrain <- lgb.Dataset(train$data, label = train$label)
Loading required package: Matrix
> lgb.Dataset.construct(dtrain)
Error in dataset$construct() :
  lgb.Dataset.construct: cannot create Dataset handle

Maybe there are some problems during the compiling and installation of the package. I remember after I run the R CMD INSTALL command, it shows "DONE", however I cannot find any function in the lightgbm library (I use lightgbm:: and there are only 3 datasets without functions). Then I make the directory into zip file and then install the zip file, and then find the functions at last. However, when I run the code above it shows the problem.
For another case, I use code from
https://www.kaggle.com/nschneider/gbm-vs-xgboost-vs-lightgbm
it shows:
> lgb.bench = microbenchmark(
+   lgb.model <- lgb.train(
+     params = params.lgb
+     , data = lgb.train
+     , valids = list(test = lgb.test)
+     , learning_rate = 0.1
+     , num_leaves = 7
+     , num_threads = 2
+     , nrounds = 500
+     , early_stopping_rounds = 40
+     , eval_freq = 20
+   )
+   , times = 5L
+ )
[LightGBM] [Fatal] Check failed: num_data > 0 at d:\lightgbm\src\io\dataset.cpp, line 27 .

Error in data$construct() :
  lgb.Dataset.construct: cannot create Dataset handle

Could you use my .zip file and check what the problem is? Thank you,
lightgbm_2.2.2.zip

Now I solve this problem by compiling the source code again to generate the .dll with command
cmake -DCMAKE_GENERATOR_PLATFORM=X64 -DUSE_R35=1 ..
cmake --build . --target _lightgbm --config Release

Then I copied the .dll files to the libs directory of R-package
and add the directory to a zip file, then install it from RStudio
I've uploaded the workable lightgbb.zip and you can use it directly for Windows.
lightgbm.zip

@jameslamb @Laurae2 any updates here?

@aaliwei91 Using your lightgbm.zip file on my side, it works.
Did you make sure to not have LightGBM loaded when installing LightGBM? Make sure to:

Compile/Write the package to the right .libPaths() folder
Install LightGBM from a fresh R session
Try using Rgui as you are using Windows. RStudio has issues sometimes with installing packages with specific dependencies.

Log using your lightgbm.zip:
> library(lightgbm)
Loading required package: R6
Warning message:
package ‘lightgbm’ was built under R version 3.5.1
> data(agaricus.train, package = "lightgbm")
> train <- agaricus.train
> dtrain <- lgb.Dataset(train$data, label = train$label)
Loading required package: Matrix
> data(agaricus.test, package = "lightgbm")
> test <- agaricus.test
> dtest <- lgb.Dataset.create.valid(dtrain, test$data, label = test$label)
> params <- list(objective = "regression", metric = "l2")
> valids <- list(test = dtest)
> model <- lgb.train(params,
+                    dtrain,
+                    100,
+                    valids,
+                    min_data = 1,
+                    learning_rate = 1,
+                    early_stopping_rounds = 10)
[LightGBM] [Info] Total Bins 137
[LightGBM] [Info] Number of data: 6513, number of used features: 116
[LightGBM] [Info] Start training from score 0.482113
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[1]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[2]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[3]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[4]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[5]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[6]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[7]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[8]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[9]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[10]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[11]:	test's l2:6.44165e-17

@aaliwei91 any news?

@dwh2102 Are you still encountering this issue?

No longer an issue for me. Thanks!

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/LightGBM/issues/1730#issuecomment-481665517,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AmUoCPFvbzPfrY8nBaWhRK1Yqc6J1AjSks5vfdXHgaJpZM4XGRLi
.

Hi, I have some questions about the lightGBM package.
system: Windows10
I downloaded the lightGBM package from the internet. And then I run the code to use the package. But there is one error. I don't know how to solve it. Can you help me have a look at the code and check what is the problem. Thanks a lot!
Here is the R-package I used.
R-package.zip
And this is my code.
install.packages("E:\LightGBM\R-package", repos = NULL, type="source",lib = "E:\R3.6.2\library",dependencies = TRUE)
library(lightgbm)
bia1 <- data.matrix(traindata[,-22]) # change class(Vars) into numeric
bia2 <- traindata$REAP.CD272.ACCGAACA
dtrain <- lgb.Dataset(data=bia1,label=bia2)
bia3 <- data.matrix(testdata[,-22]) # change class(Vars) into numeric
bia4 <- testdata$REAP.CD272.ACCGAACA
dtest <- lgb.Dataset.create.valid(dataset=dtrain,data=bia3,label=bia4)
valids <- list(test=dtest)
params <- list(objective = "regression", metric = "l2") # L2 not twelve
lgb1 <- lgb.train(params=params,
data=dtrain,
valids = valids,
min_data =1, # min data in a group
learning_rate=0.1, # smaller,slower,maybe more accurate
nrounds = 300,
early_stopping_rounds = 20) #if not better than last 20 rounds,stop
But the result is this.
Error in reference$get_colnames() :
dim: cannot get dimensions before dataset has been constructed, please call lgb.Dataset.construct explicitly

@Josie-xufan can you please open a new issue? (https://github.com/microsoft/LightGBM/issues)
Your issue looks different from the others in this thread, and I think it would be best to talk in a new issue. We also need some other information which you'll see requested in our issue template, including your lightgbm version.
Once you open the issue I'll be happy to help out!


Unifying the repo with other E+ Python repos

@JasonGlazer, this makes a few tweaks to the repository and package to unify it with other EnergyPlus Python apps.  Before this gets any wide support, I think it is worthwhile to get it looking nice with the other apps.
The main change causing changes here is using energyplus_ as the source code prefix.  This is what is now done in all the other apps:
energyplus_pet
energyplus_launch
energyplus_iddidf
energyplus_regressions
energyplus_transition

So as larger workflows are built up, it looks really professional to have this one do the same.  I imagine users building up workflows with all this at the top:
from energyplus_iddidf import IDFProcessor
from energyplus_api_helpers import EnergyPlusAPiHelper
from energyplus_rmd.translator import Translator

That looks just beautiful :)
Anyway, that's the big change, I'll comment up the other changes.  I feel it's important to do this now even if it's a mild disruption for a couple people.  It will be way harder to do later.

Making changes like this makes sense now, I think we only have one user :-)

Making changes like this makes sense now, I think we only have one user :-)

Agreed.  This is all clean on CI and ready to go in once you are happy with the changes.

@JasonGlazer any issue with these updates?  If so I'd like to merge it down.


Add zoom stops for scene.background.color.

I had a requirement to alter the background color based on zoom level.
This pull request adds a zoom stop specification for scene.background.color and applies it
to the rendering of the map.
scene:
   background:
      color: [[1,blue],[5,orange],[15,black]]

The colors are interpolated between the zoom stops.
Its a pretty nifty effect.
You can still use the default, such as a color in name, hash, or rgb sequence.

Neat!

Yeah, it can be pretty trippy. :)
On Thu, Sep 27, 2018 at 3:15 PM Nathaniel V. KELSO<EMAIL_ADDRESS>wrote:

Neat!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tangrams/tangram-es/pull/1885#issuecomment-425210686,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAlEQffT3Y_L9345Bbg1JGYAof3FXqBks5ufSO1gaJpZM4W838A
.

If we merge this, please add a corresponding issue in Tangram JS.

One behavioral note though: this looks like it will smoothly interpolate and update the background color in between zooms (per frame); is this correct? That's straightforward implementation-wise, and can provide some nice styling effects.
However, it's different from how all other styling color interpolation with zoom stops works, where the interpolation is only done on integer zoom levels (corresponding to when new tiles are built). We could choose to restrict the background color to interpolate on integer zooms to match.
Either is possible, but we should just consider if it would be confusing or undesirable if we have different interpolation behavior for different color properties. (Side note, we DO have variable zoom interpolation for camera properties for focal_length/fov, but they require that by nature to achieve a proper continuous zoom, and don't conflict with other similar properties.)
cc @nvkelso @sensescape @meetar

Maybe an option for stepwise interpolation? I agree it would be odd if it was the only color changing between the integer zooms…

Definitely not trivial to make arbitrary properties interpolate continuously between zooms, but this specific case is pretty easy to interpolate - as the diff shows.
I just discovered that in Tangram ES you can make this effectively step-wise by writing your stops like:
color: [[1, blue], [5, blue], [5, orange], [15, orange], [15, black]]

Not sure if this is advisable though because the color values at exactly zoom 5 and zoom 15 are not very well-defined.

Yep, not suggesting pursuing making other properties interpolate
continuously!
I just think that there's potential liability in having inconsistent
behavior only for this color parameter.
On Thu, Oct 4, 2018 at 10:47 PM Matt Blair<EMAIL_ADDRESS>wrote:

Definitely not trivial to make arbitrary properties interpolate
continuously between zooms, but this specific case is pretty easy to
interpolate - as the diff shows.
I just discovered that in Tangram ES you can make this effectively
step-wise by writing your stops like:
color: [[1, blue], [5, blue], [5, orange], [15, orange], [15, black]]
Not sure if this is advisable though because the color values at exactly
zoom 5 and zoom 15 are not very well-defined.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tangrams/tangram-es/pull/1885#issuecomment-427230292,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABBXRisI3sL-e9x3zy4Ry9Y8L4c50zLks5uhshAgaJpZM4W838A
.

@bcamper
I am inquiring on the path forward for this PR. Are you looking for this request to be modified to use integer zoom levels before it is accepted? What external action, if any, needs to happen before this can be merged? Thank you.

So, where else is color interpreted for zoom levels? Is it for foreground color, such as text color? And why is it only for integer zoom levels, when we travel zoom levels in a decimal fashion? I was under the impression that you could use decimals in a zoom stops specification, such as [[2,blue], [3.25, orange]].  Maybe, I'm wrong?
In any case, my only requirement was to have different colors for certain zoom levels, since a single color for all zoom levels is quite limiting. I only did the color interpolation because the function was there, and I didn't have to write one. :)
The effect of the pull request implementation is that when the zooms are in between numbers (not necessarily integers), when smoothly zooming through the levels, the color change isn't very abrupt, or should I say, it doesn't have to be abrupt. As @matteblair mentions above, you can specify it to be abrupt. If zoom level specification went to the level of only integers, this restriction may leave the user with startling changes. Especially true when doing large swaths of area such as background color, as opposed to a text foreground color.  I think that the foreground/background distinction may be relevant here.
Regardless, of how it goes, I wouldn't like to see the changes only restricted to integer values, because then at least, you can still specify somewhat of a gradient with decimal zoom stops.

Color is interpreted for zoom levels for properties such as color values for feature rendering for polygons, lines, points, etc., and also for font (and font stroke) color properties. Those are all only evaluated at integer zooms, because they are sampled when new tile geometry is built. The color values that are computed for the zoom get baked into the GL buffer data for the geometry (which is why they are currently fixed for the entire zoom range at each integer level).
I believe you can specify decimal values inside zoom stops, and they will be taken into account accurately when the interpolation is sampled -- but, that sampling will still only occur at each integer zoom change, if that makes sense.
My comment on the smooth interpolation here was simply that I think it would be confusing to have different color interpolation behavior for this one property. It's not because I think it's better! :) More that I think consistency may be more important. Open to more feedback though.
While it would be possible to add capability to interpolate those color values through fractional zooms, it's not trivial because it involves new vertex attributes and/or shader logic; this was @matteblair's point. So it's not something we're actively pursuing, though I think it is definitely worth keeping in mind for future functionality.

Ah, so it's an efficiency issue. I'm not too familiar with the GL shading logic, and now I assume that the continuous non-integer feature incurs a big hit. Is there anything I can do to make it less so?

Updated this with the master branch and modified the logic to evaluate background color stops at integer zooms. Using a single background color over an integer zoom level is likely to be more useful for designing a background to match the styling of tiles at each zoom level. For gradual changes in color the instant change isn't too jarring.


When data is downloaded from the site, it doesn't have a separate column for Country

Overview
When data is downloaded from the site, it does not have a separate column for Country. This does not match the upload format.
Expected Behavior
When downloading data from the site, Country information should have its own separate column.
Actual Behavior
When downloading data from the site, the file contains the following columns:

nameID (the column title that has been given to the OAR ID number)
longitude
latitude
name
address

Steps to Reproduce

Search the site in order to return a set of results
Click "download" button

Demo
Here's an example of data downloaded from the site:
https://drive.google.com/open?id=1BLm1VnPhKej8Aefi9QIkzMg5SQhK5LRJ
Additional context
I also think the title of the OAR ID number column being "nameID" is unhelpful. Could this be renamed to "OAR ID"?
A lot of organisations organise their supplier lists by country, so having this information in a separate column, which people can filter by, is crucial if it's going to be useful to users of the site.

Shares a point with #127


[🐴] Recover from initial failed firehose state

We've had some reports of chats failing to update. One possible culprit I've thought of so far is what this PR addresses: if the event bus/firehose is in an error state, starting a new chat will not restart the polling.
So if the user:

opens app
event log fails for any reason
they nav to a chat or start a new one
send a message
we never poll for messages, and so never get a response back from their recipient

It's not clear why the event log is failing yet, but we have had issues with session agents that could cause failures like this, and fixes have been merged already.
What this PR does is handle a dispatched UpdatePoll event when the firehose is in an error state, and uses that to basically restart from 0 and attempt a recovery. If it fails again after this, the Convo screen will show its normal error.

This is totally a flaw of course.


deps: bump SDK to v0.43.0

Looks like @fedekunze beat me to the merge. I've still got e2e tests running locally, but expect that to be 100% fine


ticdc: fix detecting kafka version

What problem does this PR solve?

Issue Number: close #11047
What is changed and how it works?
Expose requestVersion param and use the low version by default.
After getting version information, set kafka version as high as possible.
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

Please update this section in the PR description:
Check List
Tests
Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Please add the test result in pr body.

/test all

We need to consider default-kafka-version. It should be set to the lowest version supported by TiCDC. @3AceShowHand @flowbehappy  https://github.com/pingcap/tiflow/blob/1c7c9aec24454baa8624193e0499666fe629a5f3/pkg/sink/kafka/sarama.go#L32

/retest

/retest

/retest

/retest

/retest


Can not access the konga

When start up the service, found that I and a few colleagues can access the service, but there are also some other colleagues can't access.This should have nothing to do with the browser.
Do konga have to limit access to IP or amount, or is this a bug?

Hi @koyuhau , Konga has no limit access to IP.
I will need more information on the issue since from what i understand it could be network related.
When you say "cannot access" what do you mean?
Can't access the server?
Can all the PCs ping the server where Konga is deployed on port 1337?

"cannot access" means that there is no reaction using the browser to access port 1337.
And all the PCs can ping the server.

If you open the javascript console, are there any logs?

It nothing and just say ERR_CONNECTION_REFUSED

That means that the connection is rejected.
As far as i can tell this is network related.
Is there a chance that a firewall is blocking the connection to the specific port?
Try

# telnet {your-server-ip} 1337

I restart the konga，there is no problem.
This problem is a bit strange, it's not the first time.Sometimes can access, the second day cannot access again.

Thank you for your help, again appear this kind of circumstance, I telnet 1337 again.


Suspected logic error in: test("echo requests - each concurrent client gets back what it sent")

I suspect a logic error in:  test("echo requests - each concurrent client gets back what it sent") .
I am a FS2 beginner, so pardon me if I do not understand something subtle.
If I read correctly, all of the clients are sending the same message "fs2.rocks".
If there is any cross-wiring how is it detected?  That is, if the server reads
client-2 twice instead of client-2 then client-3, how is this detected?
Client-3 gets back what client-2 sent but the results are indistinguishable.
A fix makes the test harder to implement, but I believe it would live up to
its intent if each client added something like an Int (fs.rocks-2).

"So there should not be any cross-wiring between connections."
Thank you for the explaination, it will help in my tracing the "errno 107".
I concur that there should not be any cross-wiring, especially at the
level you describe. However, if the implementation below is doing something
untoward, how can a test which sends the same data for each client detect
any cross-talk?  Each connection must have distinguishable data in order to do that.
Again, this is a test, where it has to assume that the world it is testing is prone
to fail.

how can a test which sends the same data for each client detect
any cross-talk? Each connection must have distinguishable data in order to do that.

That's a good point. We change the test so it sends unique data for each client. Would you like to give that a try? :)

Oh, now I see what you are getting at 😆
"echo requests - each concurrent client gets back what it sent"
You are absolutely right, the test is not really checking this :) I copied this test out of FS2 proper, we should probably fix it there as well.

Would you like to give that a try? :)

Thank you for the vote of confidence.  Can we queue that attempt until after we have
dispatched the "errno 107"?  I think the Issue is worth capturing but I think the "errno 107"
is telling us something.
Your call.

Oh sure, I thought this would help with the errno 107. If not then we can backlog this :)


Add support for Pydantic v2

pydantic made a breaking change in their major version update to how Optional fields work with FastAPI. This PR updates the OpenAI interface to reflect this change, while ensuring backward compatibility with Pydantic v1.
A few things worth calling out:

sample_client and sample_openai have both been tested with pydantic v1 and pydantic v2, and are expected to run successfully.
langchain has a dependency on pydantic v1, so sample_langchain will not work with pydantic v2. This is reflected in inline comments within the sample code. However, it is expected to run successfully with pydantic v1.
There is now a dependency on Python 3.10 for the REST API to run successfully.

Fixes https://github.com/mlc-ai/mlc-llm/issues/590 and https://github.com/mlc-ai/mlc-llm/issues/581.

CC @comaniac


add a watchdog

Fixes #1851
Extract watchdog responsibilities to an interceptor. This removes the feedback loop in ResumingStreamingResultScanner which should fix the race conditions. The watchdog is inspired by the implementation in gax, but is scoped for read rows.

This is ready for review. In a follow up PR we should merge ResumingStreamingResultScanner and ResponseQueueReader since ResumingStreamingResultScanner is empty now

PTAL

all feedback should be addressed. PTAL

All feedback should be addressed. PTAL


Page doesn't render if timezone not set.

If the user doesn't have the timezone set the rangeslider page doesn't render with follow stack.

Uncaught (in promise) RangeError: Invalid time zone specified:
at Date.toLocaleString ()
at SplunkTimeRangeSliderInput.render (Rangeslider.js?42c4:2854:51)
at finishClassComponent (react-dom.development.js?4c72:17160:1)
at updateClassComponent (react-dom.development.js?4c72:17110:1)
at beginWork (react-dom.development.js?4c72:18620:1)
at HTMLUnknownElement.callCallback (react-dom.development.js?4c72:188:1)
at Object.invokeGuardedCallbackDev (react-dom.development.js?4c72:237:1)
at invokeGuardedCallback (react-dom.development.js?4c72:292:1)
at beginWork$1 (react-dom.development.js?4c72:23203:1)
at performUnitOfWork (react-dom.development.js?4c72:22154:1)

In Dashboardselector.jsx:

.then((data) => {
console.log(data.entry[0]['content']);
this.setState({ tz: data.entry[0]['content']['tz'] });
this.setState({ realname: data.entry[0]['content']['realname'] });
})

end of data payload:

search_use_advanced_editor: true
type: "Splunk"
tz: ""
username: "admin"
[[Prototype]]: Object

Maybe check tz is populated or add a default like gmt or use something like
const tz = Intl.DateTimeFormat().resolvedOptions().timeZone;
Maybe add a warning to let them know a default was used because they haven't set a timezone.

Great ideas! Can you possibly share me the URL that was generated for your dashboard? If you can't share the FQDN, at least just the path and query params. I'm just trying to understand if we were able to find a TZ for the user running this or not. That will help me confirm where I need to apply the fix

Adding this message if no TZ is set.

rangeslider?dashboardid=test&rangeStart=&rangeEnd=&timeinterval=15min&theme=&timerangetype=relative&relativetime=1h&tz=

It's this endpoint that is returning tz: "".

/splunkd/services/authentication/current-context


[CBR-304] Track block metadata during ApplyBlock

Description
Implement BlockMeta as per the wallet spec
Linked issue
https://iohk.myjetbrains.com/youtrack/issue/CBR-304
Type of change

[x] 🛠 New feature (non-breaking change which adds functionality)
[x] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc

Testing checklist

[x] All new and existing tests passed.

But this lacks any and all tests.

I thought the policy was that all new code should be accompanied by tests :)

I thought the policy was that all new code should be accompanied by tests :)

I also thought you were arguing that we shouldn't leave branches open for too long. This doesn't affect any existing code, and there is a new ticket for the tests.

My problem is less with long running branches and more with the difficulty of reviewing large PRs containing large chunks of work. I am also a fan of code and tests for that code being developed at the same time.


Update README.md

Amusingly the pipeline OP "breaks" the readme formatting because Markdown likes the | character a bit too much. Escaping the | fixes this and makes the readme display properly.
Just a single character README change.

cool thanks 😛!!

This has got to be a bug with Github's markdown.

Where do we report that? could email<EMAIL_ADDRESS>


Add more documentation for (styled) primitives

There currently is no documentation for the individual primitive structs, except for a single example per primitive type. And the existing examples only cover the use as styled primitives and no other example for the "raw" primitives, like hit testing or creating a Points iter.
The primitive docs should also contain an explanation how the shape is interpreted by e-g (see #182).
Should we mix styled and raw primitive docs or is there a better place to document styled primitives? The styled primitive docs would include information like how lines with equal start end end point  are drawn which doesn't apply to raw lines (discussed in #353).

I think it might be best to mix the styled/non-styled docs as they're then all in one place under the primitive itself. A potential issue with this though is a very long doc page for certain primitives if a lot of examples and functionality is demonstrated.
Another option might be to add type aliases for the Styled primitives and add doc comments to those. For example
/// Test doc comment
pub type StyledLine<C> = Styled<Line, PrimitiveStyle<C>>;

Renders as:

It could certainly work, but I think discoverability is reduced somewhat even with a link to StyledLine from Line.


Expose workload cluster capabilities/attributes

A cluster that is being added to KCP, may have existing capabilities and inherent attributes. During the life of a synctarget, it may gain new capabilities and possibly attributes. These can provide valuable information to API Providers that need to make decisions based on underlying attributes.
Example Attributes:

Geo (what geo  was this workload placed into)
Cloud (which cloud was this workload placed onto)

Example Capabilities:

Some additional workload / operator has been installed on the physical cluster  that provides a new capability
An API/Service provider has installed a workload that provides a capability to a location in response to an API object being created within a workspace. Other workspaces using this location should be able to leverage this capability

Use Case
As an API provider, my service needs to make decisions based on key attributes of a chosen synctarget.
Examples:

I am provisioning a data base and based on what Geo a workload is synced to will determine where I should create that database instance.
I am configuring DNS and want to setup Geo based responses, I need to know which Geo(s) the workload has been placed into
I provide an API that allows the user to weight traffic to go to one cloud provider over another, I need to know which cloud providers a workload has been synced to.

Options

Expose a set of supported attributes that can be set on the sync target and become available when a workload is placed. These would be a well known set of key values similar to those applied to nodes https://kubernetes.io/docs/reference/labels-annotations-taints/#topologykubernetesioregion

Something we experimented with locally was set these attributes directly on the workload keyed by synctarget. This is similar to how the status works. We set these manually on the Ingress object during the experiment
 experimental.attributes.workload.kcp.dev/5MivhNIs7DjM7dK95I2K7TpWe7aUGMU4WHqjWn: {"geo":{"continent":"EU", "countryCode":"IE"}}
Another option may be to provide a read only view within a virtual workspace to the synctarget(s) a workload has been placed on and provide an API to look that up from a placed resource.

@sttts I have tried to capture use cases for exposing capabilities
cc @ncdc @chirno

Please compare against https://github.com/kcp-dev/kcp/pull/1084, the MVP for location API.

So on location we have an AvailableSelectorLabels but this looks to be populated by "hand" and not based on capabilities expressed by the synctarget.
Would it make sense for a synctarget to support capability labels that can be aggregated to the location?
I am thinking about a use case where a controller needs to understand something about the synctarget and a developer needs to understand something about a location.
So in the case of GLBC, we are looking at allowing more control over how traffic is weighted and exposing an API around that.
More detail can be found here:
https://github.com/Kuadrant/kcp-glbc/issues/138

What we can do is to define conventions and then the syncer supports reporting the decided labels to the SyncTarget object.

Right this makes sense and had thought that might be the way to do it, so when installing the syncer configure it with a set of label values that meet the convention and those get added to the corresponding synctarget

Would rather say that the placement could have more detailed info about the chosen synctarget. But we must be careful not to leak information that isn't meant to be public. Also note that a user must note be aware of SyncTargets at all. The location is the concept the user should know and use.

Yes I think this could work. So once a synctarget is chosen, then any attributes that are set and meet the required convention could be added to that placement object and this would allow a controller that had perms to read the placement to make a decision based on those attributes.

No. A location comes first and stays (it's the user API). SyncTargets come and go. A location is an abstraction of sync targets. A location must be configured in such a way that it supports the properties (AvailableSelectorLabels) that should be exposed for scheduling

This one is interesting to me. If the syncer can add a set of labels to the synctarget, I have to wonder why those wouldn't inform the availableSelectorLabels especially if they have to meet a specified convention. This would seem to help avoid a location that is advertising something but actually no synctargets in that location support that anymore? IE one place to manage it rather than two

Think of availableSelectorLabels as a allow-list of what of SyncTarget's labels to expose to the user.

Think of availableSelectorLabels as a allow-list of what of SyncTarget's labels to expose to the user.
Not to get bogged down on this one thing, but would a convention not also work as an allow list? E.G kcp/location/attribute?

If we did reflect these attributes onto the placement object would it be keyed by cluster similar to how we do the status?

@davidfestal I have updated this issue to make it more up to date with current thinking and requirements

/transfer-issue contrib-tmc


"TokenTransfer" is an alias of "TokenAmount" (and both are DTOs)

TokenTransfer is an alias of TokenAmount (both are DTOs).
We should consider TokenAmount as the more important class, and TokenTransfer as a mere alias. Reason: TokenAmount is also usable for representing amounts that are being held by an account, not only amounts being transferred (name is more inclusive).

Maybe we should consider dropping TokenTransfer and only keep TokenAmount since it's more inclusive.

Indeed, we should drop TokenTransfer.
https://github.com/multiversx/mx-sdk-specs/issues/78


An example for implementing the package

Thanks for sharing useful tips
Can you give an example for implementing this package? Thanks

Sorry for the delay. I refactored some implementations add two example with simple testing. It satisfys my demands.
There is a basic usage for reference, and you can other examples here.
To be noticed, the pkg may have some breaking-changes and corresponding tests as soon as possible.

Thanks @zheeeng . It's working fine now. thanks so much. Thanks for sharing useful tip :)

@chipv-crystal Hello, just then I released a fresh version, focus on simplifying usage. You can take a look at README.md or Go Doc.
Wish you enjoy it and I look forward to having your feedbacks.


Race condition with fast containers leads to timeouts

Description
I'm running sam local start-api, and then a subsequent test script that executes hundreds of requests against the lambda function. If I don't add a sleep in the startup of the lambda under cold start, I get Function 'func' timed out after 30 seconds.
Steps to reproduce
It seems like there's a race condition between when the container starts, is invoked, and when the "attach" command is run to attach to the stdout of the container.
Observed result
Functions randomly timeout in a CI environment, adding timeouts to the startup of the lambda executing resolves the flakiness.
Expected result
Functions complete without timeouts and report their results to the Flask API gateway shim.
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)

OS: Ubuntu
sam --version: 0.15.0

Is the usecase around load-testing? Could you elaborate a bit more on why the test scripts executes hundreds of requests?

Also note that SAM CLI's local execution is not optimized for tons for parallel requests. We have made some design decisions that intentionally trade off performance for user experience because it is a development tool. We use Flask as a cheap mult-threaded web server which is not really the intended use case.
There could be two root causes:

With hundreds of invokes, your Docker server might be coming to a grinding halt slowing down everything. I don't know much about Docker's scalability. Is it possible that Docker is unable to handle hundreds of running containers on the same host?

Or, SAM CLI's was coming to a grinding halt and not responding as fast as Docker.

SAM CLI slowdown hypothesis
From a quick glance at the code, I don't see how there can be a race. In fact if there was a race, I would expect SAM CLI to return immediately while the container is running. Based on the symptom, I think there is an overall slowdown of the Python process (may be due to GIL) to a point where the attach iterator did not get a chance to complete before the timer went off.
With hundreds of threads all spawning containers and reading sockets, I expect SAM CLI to be heavily IO-bound. Timers, which is a CPU bound operation, probably ran at the precise time and killed the before the IO bound stuff completed.

I'd add to the chorus that this probably isn't a good use of sam local currently. We're continuously making performance improvements, including in some of the areas mentioned, so it may get better over time at this, but I'd recommend running against an actual live AWS Lambda deployment for parallel invocations.
Closing since the feedback is noted and this seems to be asked and answered, but feel free to reopen if you have more details you want to cover.


Handle Press Error ModalDropdown

I got this issue
I'm try to install these version

v0.7.0
"react-native-modal-dropdown": "git+https://github.com/siemiatj/react-native-modal-dropdown.git",
but, still has an error
please, help me

https://gyazo.com/bb5be5715bc81cd25d8d20557d4de434

Can you show the code, that is resulting in that error?


Video recording extension

Use Seleniumn Grid extra as your grid. You will get video recording automatically....I am using Ubutubu + grid extra +  xvfb...Everything works great.


Update .gitignore to ignore IDE files

Ignores IntelliJ and Eclipse files as well as the generated log.txt from running unit tests

Good catch, @melloware! Thanks! See diff: https://github.com/cleydyr/dart-sass-maven-plugin/compare/f73717e...49bf024


await available profiles before creating a terminal

@Tyriar see https://github.com/microsoft/vscode/commit/c765e70cf92bc4d2f08bdaa11ed1231063b7aa39#commitcomment-56021309


CIT-425: Fix copy-paste error in mqtt sub help message

Proposed changes
Change the help message of tedge mqtt sub to show that the topic is for subscription, not for publication.
Types of changes
What types of changes does your code introduce to thin-edge.io?
Put an x in the boxes that apply

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactorings that doesn't explicitly fix a bug or add any new functionality)
[X] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue
https://cumulocity.atlassian.net/browse/CIT-425

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report

Merging #341 (3855ec3) into main (315f010) will decrease coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #341      +/-   ##
==========================================
- Coverage   66.24%   66.14%   -0.11%
==========================================
  Files          67       67
  Lines        3990     3990
==========================================
- Hits         2643     2639       -4
- Misses       1347     1351       +4

Impacted Files
Coverage Δ

tedge/src/cli/mqtt/cli.rs
50.00% <ø> (ø)

mapper/tedge_mapper/src/collectd_mapper/batcher.rs
80.70% <0.00%> (-1.76%)
:arrow_down:

Continue to review full report at Codecov.

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


For windows 7, sougou pinyin IME not work in Textfield or CupertinoTextfield.

flutter version: stable 2.10.0  dart 2.16.0
only windows 7
only input Chinese
word disorder and insert letter
video

https://user-images.githubusercontent.com/62126718/152928611-48064740-501a-46c6-9219-e4097bba6c3a.mp4
6.code
class _MyHomePageState extends State<MyHomePage> {

  @override
  Widget build(BuildContext context) {
    // TODO: implement build
    return Center(
      child: Column(
        children: [
          TextField(),
          CupertinoTextField(),

        ],
      ),
    );
  }
}

flutter doctor  -v
[√] Flutter (Channel stable, 2.10.0, on Microsoft Windows [Version 10.0.19043.1466], locale zh-CN)
    • Flutter version 2.10.0 at D:\fluter
    • Upstream repository https://gitee.com/mirrors/Flutter.git
    • FLUTTER_GIT_URL = https://gitee.com/mirrors/Flutter.git
    • Framework revision 5f105a6ca7 (6 days ago), 2022-02-01 14:15:42 -0800
    • Engine revision 776efd2034
    • Dart version 2.16.0
    • DevTools version 2.9.2
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at C:\Users\liping\AppData\Local\Android\sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: D:\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.3)
    • Visual Studio at D:\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31424.327
    • Windows 10 SDK version 10.0.19041.0

https://user-images.githubusercontent.com/62126718/152930929-7ed5e980-d9b9-4830-b241-79cb694e0644.mp4

Hi @liqping, Thanks for filing the issue. If I am understanding it correctly when you enter some characters and delete the cursor ends up in the middle of the text is that correct?
Also if the characters are wrongly entered, could you please specify what English Keys needs to be pressed and what characters do you expect in the TextField?
Thanks

Such problems will occur when you use input methods such as Sogou. You can add the following code to improve it,
TextField(         keyboardType: TextInputType.multiline,         maxLines: null,       ),

flutter version: stable 2.10.0  dart 2.16.0
only windows 7
only input Chinese
word disorder and insert letter
video

Rec.0001.mp4
6.code
class _MyHomePageState extends State<MyHomePage> {

  @override
  Widget build(BuildContext context) {
    // TODO: implement build
    return Center(
      child: Column(
        children: [
          TextField(),
          CupertinoTextField(),

        ],
      ),
    );
  }
}

flutter doctor -v
[√] Flutter (Channel stable, 2.10.0, on Microsoft Windows [Version 10.0.19043.1466], locale zh-CN)
    • Flutter version 2.10.0 at D:\fluter
    • Upstream repository https://gitee.com/mirrors/Flutter.git
    • FLUTTER_GIT_URL = https://gitee.com/mirrors/Flutter.git
    • Framework revision 5f105a6ca7 (6 days ago), 2022-02-01 14:15:42 -0800
    • Engine revision 776efd2034
    • Dart version 2.16.0
    • DevTools version 2.9.2
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at C:\Users\liping\AppData\Local\Android\sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: D:\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.3)
    • Visual Studio at D:\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31424.327
    • Windows 10 SDK version 10.0.19041.0

flutter version: stable 2.10.0  dart 2.16.0
only windows 7
only input Chinese
word disorder and insert letter
video

Rec.0001.mp4
6.code
class _MyHomePageState extends State<MyHomePage> {

  @override
  Widget build(BuildContext context) {
    // TODO: implement build
    return Center(
      child: Column(
        children: [
          TextField(),
          CupertinoTextField(),

        ],
      ),
    );
  }
}

flutter doctor -v
[√] Flutter (Channel stable, 2.10.0, on Microsoft Windows [Version 10.0.19043.1466], locale zh-CN)
    • Flutter version 2.10.0 at D:\fluter
    • Upstream repository https://gitee.com/mirrors/Flutter.git
    • FLUTTER_GIT_URL = https://gitee.com/mirrors/Flutter.git
    • Framework revision 5f105a6ca7 (6 days ago), 2022-02-01 14:15:42 -0800
    • Engine revision 776efd2034
    • Dart version 2.16.0
    • DevTools version 2.9.2
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[√] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at C:\Users\liping\AppData\Local\Android\sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: D:\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.3)
    • Visual Studio at D:\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31424.327
    • Windows 10 SDK version 10.0.19041.0

Such problems will occur when you use input methods such as Sogou. You can add the following code to improve it,
TextField(  keyboardType: TextInputType.multiline,  maxLines: null,  ),

Such problems will occur when you use input methods such as Sogou. You can add the following code to improve it.
TextField(  keyboardType: TextInputType.multiline,  maxLines: null,  ),

@maheshmnj

It works fine in windows 10. It happend only on windows 7.
Microsoft pinyin and google pinyin works fine , but It happend such as Sougou,baidu IME  on windows 7.
It happend any english  characters , but not specify.

@wch1618 thanks! I try~

@wch1618 It worsk fine only maxlines == null. How to limit lines? I need single line.

@liqping  Add limit logic in onChanged

@wch1618 thanks!

@wch1618 It is very inconvenient and Tedious！It is a solution now only.

It works fine in windows 10. It happend only on windows 7.
Microsoft pinyin and google pinyin works fine , but It happend such as Sougou,baidu IME  on windows 7.
It happend any english  characters , but not specify.

Thanks for the info labeling this issue for further investigation.

cc @cbracken

google pinyin  and shift+ctrl

the same question

https://user-images.githubusercontent.com/69191321/173213143-cea43183-f08e-4982-87fb-5f8ed603cc82.mp4

Flutter assets will be downloaded from https://storage.flutter-io.cn. Make sure you trust this source!
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.0.2, on Microsoft Windows [版本 10.0.22000.675], locale zh-CN)
[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Enterprise 2022 17.0.6)
[√] Android Studio (version 2021.1)
[√] VS Code (version 1.68.0)
[√] Connected device (3 available)
[√] HTTP Host Availability


Merge clang and gcc wrappers

Currently gcc and clang are wrapped in almost identical fashions, with only minor differences and a lot of duplication. We should merge these, with any necessary divergence being built on top of a common base. Waiting for stdenv-updates.

This seems to have been fixed in 48f63c2f2e3752df2a0b73cad9b9bba9eda8ac60, Can someone close it?

True.


net: only js/wasm returns an error on writes to closed connections

If a localhost connection is closed by the peer, all other GOOS seem to allow writes to happen without error, while js/wasm returns write tcp <IP_ADDRESS>:XXX-><IP_ADDRESS>:1: Socket is not connected with XXX changing.
I admit it does feel like something that should return an error, and I don't know if there is a precise requirement to do otherwise, but js being the only GOOS to do it is very inconsistent, and not even NaCl on the playground with its fictitious network returns an error.
This came up because js/wasm were the only broken TryBots in some crypto/tls CL: https://storage.googleapis.com/go-build-log/ca02321c/js-wasm_d63aa3d8.log
https://play.golang.org/p/lRhq44NJXTr (this specific program will actually panic on js/wasm due to #28649, but it proves the log.Print is being reached)
/cc @neelance

Unfortunately, the fake implementation of TCP-like byte-sequence full-duplex pipe on JS is broken. Also, it doubles the maintenance cost by having each implementation on NaCl and JS. I have a CL that may solve the issues: https://go-review.googlesource.com/c/go/+/120958, but it will probably land in Go 1.13 or above.
Can you please skip the roadblock test cases for now?

Yeah I adapted the tests, so not urgent on my side. Feel free to re-milestone as you wish.
What is blocking that CL?

What is blocking that CL?

Just for congestion control; the plan for Go 1.12 had AIX and Fuchsia ports that need to churn up the runtime, os and net packages. Like other communication protocols, congestion control is reviewee/sender side's responsibility (and flow control is reviewer/receiver side's responsibility.)


Update members.yaml

update spelling error

@tgorochowik do you know why the comment GH action is failing? Note preview worked fine on my PR, so is this an issue with Kendall's account permissions?

Something must have changed with the privileges settings again. It is most likely caused by the fact that you pushed your branch to the chipsalliance org, while @wendallkaters pushed to their own fork.
We need to take a look at that if we want external contributions to also be able to publish the previews.

@tgorochowik ok that is what I thought @wendallkaters is a PM coming on to the project so I can give her the same repo permissions that I have. No need to look at adding external contributions.

OK i invited kendall to this repo, future PRs will be made against this repo rather than a personal branch.


[INFINITY-2075] Fix DefaultSchedulerTest.testConfigurationUpdate

Wait for task to be starting before sending TASK_RUNNING status as that is what happens in the real world.

@benclarkwood given this only touches the SDK unit tests (which passed), should we merge or rerun the failed tests?

@NimaVaziri I see no reason to break protocol for this. Kicked off re-runs.

Fair. Just reminded me of https://github.com/mesosphere/dcos-commons/pull/1286, save for the breakage.

True. That was at peak testing frustration. I want to get a sense for where we are at now (I think we are better than we were then).


ci(repo): Migrate to Vite to optimize the build time for libraries

Summary of changes
Provide a detailed description of proposed changes.

[x] Migrate to Vite to optimize the build time for libraries.
[x] Moved the Tailwind preset to tailwind.preset.js at the root level. This change makes it reusable across dApps and libraries. The file is required for configuration and is referenced during build time, so it cannot reside in an Nx library.

[x] Additionally, I imported this file into the tailwind-preset library for potential npm publication.

[x] Removed npm workspaces from the package.json file to avoid conflicts with the Nx workspace.
[x] Improved SVG file handling across dApps and libraries by converting them into React components using svgr. Additionally, I leveraged the resource query feature in webpack 5.
[x] Fixed builds on the repository after the migration (which took the longest time 😢) to ensure compatibility between Webpack-based dApps, Next.js dApps, TypeScript libraries, Vite libraries, and Storybook.
[x] Removed the deprecated Scroll Testnet.
[x] Bumped all dependencies.

Proposed area of change
Put an x in the boxes that apply.

[x] apps/bridge-dapp
[x] apps/hubble-stats
[x] apps/stats-dapp
[x] apps/tangle-dapp
[x] apps/testnet-leaderboard
[x] apps/faucet
[x] apps/zk-explorer
[x] libs/webb-ui-components

I’m going to merge this pull request into the ESM Migration PR. After that, you all can review and perform QA.


Mistake in this page

I think the second C# here is supposed to be F#:

Some features of XML documentation in C# and other .NET languages are not supported in C# ⚠.

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

ID: 1a8fbc01-942f-ac15-e06b-2e50c45e8183
Version Independent ID: 788235a1-2b43-1607-b24b-57c0552fd890
Content: XML Documentation - F#
Content Source: docs/fsharp/language-reference/xml-documentation.md
Product: dotnet-fsharp
GitHub Login: @cartermp
Microsoft Alias: phcart

Fixed by https://github.com/dotnet/docs/pull/22256

Fixed by https://github.com/dotnet/docs/pull/22256


chore: remove opokornyy from reviewers

What this PR does / why we need it:
chore: remove opokornyy from reviewers
Ondrej has not contributed to this project for a long time. Thanks for all the work you did!
Release note:
NONE

@opokornyy Thanks a lot for all your contributions!
/lgtm


Track Assembly and Resource issues

From @ryanbrandenburg on Tuesday, July 5, 2016 11:56:48 AM
This is a tracking issue for dotnet/corefx#9766 and dotnet/coreclr#6123 which were found due to #257 and had work arounds in aspnet/Localization#262.
Copied from original issue: aspnet/Localization#263

@hishamco was correct, both issues were already resolved (I thought one of them was still around). This can be closed now.


Add stylelint linter

it doesn't look like you're testing the changes, given that the code doesn't even run. You need to add the linter config to linter/init.lua to expose it to guard.nvim just like all the other linters do.
This is an unecessary step I will code a fix up for right now, since you shouldn't have to. Sorry about that.
In the meantime, I think you should use the find flag again to not queue the linter according to the specs.

The find here is a bit tricky because of:

Stylelint expects a configuration object, and looks for one in a:
stylelint property in package.json

We may or may not want to stop at a package.json. I could add a find for the other stylelint specific files, I'll do that.

@xiaoshihou514 there was an unresolved issue here.


In kodi 20 Video has audio stutters and takes a long time to start, downgrade to 19 solves the issue

Netflix add-on version
1.20.6+matrix.1
Operative systems used
Linux (Ubuntu / Mint / ...)
Kodi version used
Kodi 20 (Nexus)
Description of the bug
Hi,
I'm running kodi using the flatpak version on fedora kinoite.
When kodi was upgraded to 20 bij flatpak the netflix videos started to give issues

long time to start
audio stutter
downgrading to the last kodi 19 flatpak version solved the problem
For those who wonder which version:

flatpak update --commit=e15d861c08f40a370b16efe0a999b59f85727032077f6c4bdc7102336387154d tv.kodi.Kodi

there are also a lot of the following errors in the logs:
error : CDVDAudio::AddPacketsRenderer - timeout adding data to renderer
and even more warnings:
warning : ActiveAE - large audio sync error: -25496.991293
The workaround is to  downgrading the kodi version back to 19.1
Everything works smoothly again after that.
Rob
Steps to reproduce the behavior
install fedora kinoite on a nuc
install flatpak kodi
install netflix plugin from castagnait
try to view any movie (will work but the audio will be very annoying)
downgrade to kodi 19.1
flatpak update --commit=e15d861c08f40a370b16efe0a999b59f85727032077f6c4bdc7102336387154d tv.kodi.Kodi

everything will work fine again
Debug log - mandatory
https://paste.kodi.tv/opulirizaz.kodi
Possible fix
downgrade to kodi 19.1 as a workaround
Additional context
No response
Screenshots
No response

this is not an addon problem:
2023-02-19 19:47:41.551 T:17      error <general>: ActiveAE::InitSink - failed to init
2023-02-19 19:47:41.551 T:17    warning <general>: CActiveAE::StateMachine - signal: 11 from port: OutputControlPort not handled for state: 2
2023-02-19 19:47:41.551 T:17    warning <general>: CActiveAE::StateMachine - signal: 12 from port: OutputControlPort not handled for state: 2

you have to ask to kodi community


Backport: Error while trying to create a cluster with AWS Asia Pacific (Hong Kong) Region 'ap-east-1'

https://github.com/rancher/rancher/issues/20474

Can be tested with v2.2.4-rc16

Tested with rancher/rancher:v2.2.4-rc19 from May 30
Steps:

Create an AWS node template
Region is ap-east-1
Instance type is t3.medium
Create EC2 cluster for region ap-east-1
Modify the count of cluster nodes
Delete cluster

Result:
Node template create successfully.
Cluster create/modify/delete successfully.
Verified fixed.


Anerdw: Watch<PHONE_NUMBER>147(?# invite code)

Anerdw requests the watch of the watch_number<PHONE_NUMBER>147(?# invite code). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Dragonrage in Charcoal HQ