MS SQL Server: connect works, basic query fails (RODBC works fine)

Hi there,
I am connecting to a remote SQL Server instance from a windows machine.
My Julia script (below) makes the connection, but fails when making a query - see the Get data section in the script.
For reference, I have included below an almost identical R script that works.
I am running Julia version 0.3.8 on Windows, and ODBC.jl version 0.3.10.
Any ideas?
Cheers,
Jock
The Julia version:
### Init
using ODBC
con = ODBC.connect("my_dsn"; usr = "dsn_username", pwd = "dsn_password")

### Get data
# Both of these lines give the following error:
#     ERROR: [ODBC]: SQLExecDirect failed; Return Code: SQL_DEFAULT_ERROR in error at error.jl:21 (repeats 2 times)
tablenames = ODBC.query("SELECT * FROM INFORMATION_SCHEMA.TABLES WHERE table_type = 'BASE TABLE'", con)
df1 = ODBC.query("select top 10 * from Event", con)

### Clean up
disconnect(con)

The R version:
### Init
library(RODBC)
con <-odbcConnect("my_dsn", uid="dsn_username", pwd="dsn_password")

### Get data
tablenames <- sqlQuery(con, "SELECT * FROM INFORMATION_SCHEMA.TABLES WHERE table_type = 'BASE TABLE'")
df1 <- sqlQuery(con, "select top 10 * from Event")

### Clean up
close(con)

@JockLawrie The error message that you included above is the same as the error message seen in issue #86 that I filed.
  If so, have you set a System DSN entry for your ODBC data source and could you try executing const ODBC.odbc_dm = "C:\\Windows\\System32\\odbc32.dll" prior to executing using ODBC`?

@AndyGreenwell thanks for the suggestion.
I conducted a little further testing with and without your suggested  const odbc_dm = "C:\\Windows\\System32\\odbc32.dll" prior to using ODBC. The results are the same though, and a little bit weird.
For example, if I repeat the query df1 = ODBC.query("select top 10 * from Event", con) several times I get any of the following 3 different responses seemingly randomly:

ERROR: [ODBC]: SQLExecDirect failed; Return Code: SQL_DEFAULT_ERROR in error at error.jl:21 (repeats 2 times)
ERROR: invalid UTF-8 sequence in convert at utf8.jl:163 (repeats 2 times)
valid result (a data frame with the correct data in it)

Perhaps this is related to https://github.com/quinnj/ODBC.jl/issues/67?

should be resolved by 7ae7ebfac60ed3ffb1ccefd552993a3073f8b77b; please re-open if you're still seeing issues


Client progress bars report negative eta when object size can't be determined

Pelican Service:
Client
Describe the bug
An edge case, but progress bar values (eta, MiB/s) dip negative when the object being fetched has no real size. I observed this when testing recent S3 PUT bugfixes that caused an object to be uploaded with no body. At first I thought the 0-length object was the issue, but I'm also observing this in other cases that indicate the client can't determine the object's size:
3000MB-downloaded 194.00 MiB / 0.00 b [-----------------------------------------------------------------------------------] -9s ] 20.95 MiB/s

I'll note this object is coming through an S3 origin, which may have something to do with the undetermined size.

@jhiemstrawisc Is there a reliable way to trigger a transfer where the object size cannot be determined?

I haven't investigated it, but I'm willing to bet the best bet is a download from S3. Otherwise you could always whip up an HTTP server with desired behavior in Python and point the origin at that.


⚠ Jenkins Plugins Artifactory has degraded performance

In 4188f68, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1207 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 4b79e27 after 52 minutes.


NaNs

For example in Jeffery Star's channel (UCkvK_5omS-42Ovgah8KRKtg), the data must be so old that there are NaNs, when pandas tries to reform the type and error is thrown. Will need to print the raw_df and investigate.

This happens because YouTube gives option to stop comments being added to videos. If this is the case, videos will have 'nan' comments which cannot be converted into integers.
WIll need to search for NaNs and turn them into zero, which will be acceptable for what we are trying to achieve with its app.

This has been fixed in the dev branch.
One simple commands:
df = df.fillna(0)

This fill nans with 0. Ok for what we need to do.


[Box] Add types for ref

[x] I have followed (at least) the PR section of the contributing guide.

Fixes https://github.com/mui-org/material-ui/issues/17010

Is this going to be fixed in 4.x?

@christopherjbaker it's not blocking, so no. You can type it as any until you upgrade to v5.


About the zeros-initization of the hash-grid;

Hi
I find the zeros-initilization of the hash-grid in the Sec.4 IMPLEMENTATION is works; (I need those special initialization in my experiment setting)
but i test your code, it not work;
can you explain it? thank you

@Dengzhi-USTC The density output activation function is default to relu, which will make all-zero initialization fail to propagate, while the original implementation uses exponential. However, I haven't succeeded in using exp as the output activation. I recommend you try softplus.

Thank you very much
it works;


Shamir Secret Sharing Scheme functions

oh, wow, how could this be applied? I mean, I assume multiple clients would have to simultaneously come together to execute a secure script?

The use cases are plentiful, the two added functions simply enable the splitting and recovery of data with a K out of N scheme, your creativity will do the rest ;-)

PTAL


Question about SmartRedis clients

Hi SmartSim team,
From your documentation and examples, I noted the C++  Client function does not need the address/port  of the Orchestrator, while the python client does, otherwise running a python client without address/port exits with the next error:
smartredis.error.RedisConnectionError: Could not connect to database. $SSDB not set

I haven't try it yet, but does this implies C++ clients cannot be used as "stand alone" analysis codes, and only can be used as simulation codes executed via an Experiment ?
Thanks,

Hello,
The SmartRedis clients do not have to be used with SmartSim, but SmartSim automates the work of launching a Redis database (either cluster or non-cluster).  If you have launched your own database, the SmartRedis clients can be used without SmartSim.
Regarding the error message that you posted, all of the SmartRedis clients (Python/C/C++/Fortran) read the environment variable SSDB for the address and port in order to connect to the database.  Before executing your code containing the SmartRedis client, you can set the environment variable as follows (substitute in the correct host and port into the command below):
export SSDB="<IP_ADDRESS>:6379"

The python client includes an additional constructor that takes in an address and port which overrides the value of the SSDB environment variable.

Thanks,
I totally missed SSDB reference in the documentation:
https://www.craylabs.org/docs/sr_runtime.html?highlight=ssdb#setting-redis-database-location


feat(portal): add DomPortalHost

R: @hansl @kara
All of the stuff in platform/ is copied from angular/angular and is, for our purposes, the facades necessary for DOM manipulation. (I also had to get rid of implicit any and make it pass tslint... ugh). I talked with Tobias and Misko about how much this sucks, and their answer is https://github.com/angular/angular/issues/7561. For now copying this stuff unblocks progress.
DomPortalHost is the last part of the portal/ code needed for overlays. It's will be used when opening an overlay in the attaching of the overlay element to the document body.
Setting locals for a TemplatePortal attached to a DomPortalHost does not work; I spent quite a while trying to figure out why and came up short. I'm going to follow up with Tobias on that, but in the meantime it's not a blocking issue.

Okay then LGTM with the nit that we should have a proper discussion about using Exception classes rather than always throwing BaseException.

@hansl added exceptions.


make contains validation more readable.

Please provide a description of this PR:
A small optimization to make contains validation more readable.
Unit tests are covered and do not need to be modified.
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[x] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[x] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

Hi @wulianglongrd. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test


CorDebugStringValue internally returning HRESULT.E_INVALID_ARG - triggering error flow

Trying to call public HRESULT TryGetString(out string szStringResult)  goes off the rails in the following snippet:
int cchString = 0;
int pcchString;
char[] szString;
HRESULT hr = Raw.GetString(cchString, out pcchString, null);

if (hr != HRESULT.S_FALSE && hr != HRESULT.ERROR_INSUFFICIENT_BUFFER && hr != HRESULT.S_OK)
  goto fail;

Raw.GetString returns HRESULT E_INVALIDARG (because of the null?), thus triggering the "fail" path.
This happens with the latest build as was provided in https://github.com/lordmilko/ClrDebug/issues/13

Which interface is this for? ICorDebugStringValue?

Yes, that is correct.

ClrDebug assumes that anything that is an array can be retrieved via a two stage process, asking for the length and then creating a buffer with the required length. Some APIs don't actually allow for a two stage query, and require you to figure out the length yourself. I fix rare methods like this as I encounter them. I have confirmed from the implementation of CordbObjectValue::GetString that it requires a valid buffer be specified in the first place, using a length retrieved from ICorDebugStringValue.GetLength. I will update this method to use manual query semantics

Playing around with the interface it looks like the two stage process approach would still works as long as what is being passed in is not null initially, e.g., the following works fine.
CorDebugStringValue stringVal = val.As<CorDebugStringValue>();
char[] charBuff = Array.Empty<char>();
stringVal.Raw.GetString(1024, out int len, charBuff);
charBuff = new char[len];
stringVal.Raw.GetString(len, out int _len, charBuff);
string v = new(charBuff, 0, _len);

But as you mentioned, since Length is correctly populated on ICorDebugStringValue anyway there is probably not even a need for a second pass.


[core] Switch to lodash-es?

To further reduce the bundle size and to continue the strategy of PR https://github.com/callemall/material-ui/pull/8772, the project could switch from the lodash module to lodash-es. lodash-es makes use of ES6 modules (instead of doing everything in one large file) which drastically simplifies tree-shaking.
Migrating shouldn't be that hard, and mostly consist of adapting the import definitions (which can probably be done with a simple search + replace). If you're alright with this, I would line up another PR that does the migration.

We have had some history on the lodash topic: #4380 and #5308.

The lodash-es package is a separate package because it's raw ES6 code which has limited support at the moment.

https://github.com/lodash/lodash/issues/3283#issuecomment-318118820
Sorry, I don't think that it's a good idea. As far as I understand it, it will go against our supported browsers.

But wouldn't Babel take care of compiling lodash down to something ES5-compatible in the regular builds?

Ah nevermind, I understand.

Most people doesn't transpile the node_modules for performance reason.


Fix memos in children; introduce isomorphic spawn

This PR cleans up the task code a bit and introduces a new type of task that runs on the client and during suspense, but does not block suspense. This is useful for two different kinds of futures:

Synchronous responding futures like memo
Server functions that start on the server, but don't block suspense

Looks good, cleaner. Can we add a simplified test to catch the memo thing?


Global Stats

https://sendgrid.com/docs/API_Reference/Web_API_v3/Stats/global.html
Hello.
For the statistic API; we don't have any information about the authentification. What are the parameters to add ? Do we need a username ? An API Key ? Etc.

Hi there,
Please use this documentation for the SendGrid V3 APIs going forward: https://sendgrid.api-docs.io/v3.0/stats
You'll need an API key to use the stats API. Please let us know if you need additional information.
Thanks!

This should be addressed by the updated docs linked in the previous comment. The reported page is removed and redirected.


You always grow from food. But as you get larger, you shrink faster than you can regain mass from food.
Also, when you get larger you see more cells, so efficient food collection can become more resource intensive. This means ignoring food completely when large could also take significantly less CPU cycles.

Right now, it's already setup to ignore small food when no one worthy is around.

yes you're right my bad, it should just be ignored a bit more when you grow larger


Fix Detekt gradle task cache restoration issue (#2180)

Detekt gradle task is marked as @CacheableTask but it's input source files are
marked with PathSensitivity.ABSOLUTE annotation from source property defined in
parent SourceTask.
Having task inputs with absolute path sensitivity effectively disables cache reusability:
https://guides.gradle.org/using-build-cache/#relocatability
Closes #2180

Codecov Report

Merging #2182 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2182   +/-   ##
=========================================
  Coverage     80.59%   80.59%
  Complexity     2028     2028
=========================================
  Files           338      338
  Lines          5850     5850
  Branches       1066     1066
=========================================
  Hits           4715     4715
  Misses          564      564
  Partials        571      571

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 5f41ea5...f055642. Read the comment docs.

Thanks for picking that up, plus the fix!

@3flex is there a way to get a snapshot for gradle-plugin? I don't see it being published here https://oss.jfrog.org/artifactory/webapp/#/artifacts/browse/tree/General/oss-snapshot-local/io/gitlab/arturbosch/detekt

Not sure, I don't have access to publish artifacts. @arturbosch do you know if anything can be done here?
@plastiv consider opening a new issue to track this, thanks!


Update poetry lock

Description
Update poetry lock. This is mainly because of acre update.
Changes

updated poetry.lock file

Only acre module was updated in poetry.lock.

Tested on Centos and Windows and looks good so far.


Add Time Remaining warning

From my observations it can happen that people are not aware of the timer even though it is displayed on the score header itself. I think implementing a time warning such as "30 seconds remaining" and making the timer red/blinking could solve this.
No ideas on the implementation yet. That's to be discussed in the dev channel.

Ideas can also be posted here, doesn'T need to be discussed exclusively in the dev channel. Writing down ideas and information here might also help to keep track of it.


temporary HP bar not showing correctly when max HP is lower than temporary HP

Same as what the title says, temporary HP bar not showing correctly when max HP is lower than temporary HP. The HP bar turns completely blue and keeps extending to the right outside of the box width it normally stays within.

Yes, this behaviour is by design. Because all healthbars get their length as a percentage of the max HP. So the temp HP are 3 times the max HP which means they get a length of 300% of the hp bar.
The alternative would be to have it at max 100% but then people would complain that it shows too little because it should actually be longer.
Do you have a better idea to solve this problem?

I appreciate the response, I understand what you mean and that makes sense. I think you're right, the way this is currently programmed may be the best solution with this situation because it may be unlikely that many people will run in to this issue at all.
I've dabbled in programming in the past but I am no expert- the first idea that comes to mind for a solution here could be to keep the bar the same width it normally is, and invert the colors. So in my example above, the HP bar would be 1/4 red and 3/4 blue (1/4 of the shown current total HP of 12 is equal to max HP of 3 and 3/4 of current total HP is equal to current temp HP of 9)
To put it another way, instead of extending the blue bar further to the right, the blue bar would instead push the red bar smaller as the temp HP goes up, keeping both bars still confined in that width while still visually representing what % of each bar makes up the total combined current HP of both max and temp.
I apologize if this is explained poorly or if I am missing anything obvious, I appreciate your work and this extension is awesome, so I completely understand if this particular issue isn't a priority. I enjoy thinking it through at least!


BUG:Timestamp for this request was 1000ms ahead of the server's time.

有时候会出现委托失败，程序之前下单都是正常的，过了几个小时会这样：
委托失败，状态码：400，信息：{"code":-1021,"msg":"Timestamp for this request was 1000ms ahead of the server's time."}

发错板块了


[CSL-2597] Add to favorites button

Thanks for making the changes, looking way better. Just a couple of things

Misspelling in file name useAddToFavoritesbuttonProps -> useAddToFavoritesButtonProps
Conversion event should only fire if favorites button isn't already clicked
Not sure if it's this PR but single result card is misaligned

Ignore the visual bug. Enes fixed it in another PR @stanlp1


fixed python test error

updated pylint to ignore import-error, since tensorflow recently changes has cause pylint to create false alarm on imports.
pin the tensorflow nightly to 0618, since tfhub seems to have issues with 0605 version.

This change is

Found the source of the breakage: tensorflow-estimator which is a transitive dependency of the converter released 1.14 today https://pypi.org/project/tensorflow-estimator/#history which broke our build because pip lacks lock files...
Pinning tensorflow-estimator==1.13.0 in the requirements.txt will fix that issue.
That way you don't have to disable import-error from pylint and allow pylint to follow imports. WDYT?

Also pinning tensorflow-estimator to an exact version will give us more stability in the future, since our dependencies don't pin it exactly.


Retries

@ibuildcloud Actually, on second thought 5b2a219ad0ebb348c74416e02f47f9e1608900a5 may not be necessary as direct block write I/O is supposed to be atomic on a blocks level. So concurrent 'same write' ops idempotence should still hold...

another thought on coexistence of ping and r/w timeouts:

should we disable the pings in the presence of I/O activity so that it activates only in the absence of I/O in say 2..15s period?


Fail to load PT-eu language

I've noticed an error to get the PT translations.
Error cause:

MR ready


Linker error on zug::meta::bottom conversion operators

I get
CMakeFiles/test-into_vector.dir/into_vector.cpp.o: In function `_ZN3zug4skipIRZNKS_5map_tIZL29____C_A_T_C_H____T_E_S_T____4vEUliE0_EclIRKNS_6last_tEEEDaOT_EUlS9_DpOT0_E_NS_4meta6bottomEJiEEENS_11skip_resultIS8_T0_JDpT1_EE4typeES9_OSI_DpOSJ_':
/home/ltjax/code/zug/zug/skip.hpp:186: undefined reference to `zug::meta::bottom::operator std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >() const'

and
CMakeFiles/test-transducer-eager.dir/transducer/eager.cpp.o: In function `decltype(auto) zug::reduce_nested<zug::last_t const&, zug::meta::bottom, zug::detail::iterator_range<std::reverse_iterator<__gnu_cxx::__normal_iterator<testing::copy_spy<testing::detail::default_copy_spy_base_t>*, std::vector<testing::copy_spy<testing::detail::default_copy_spy_base_t>, std::allocator<testing::copy_spy<testing::detail::default_copy_spy_base_t> > > > >, std::reverse_iterator<__gnu_cxx::__normal_iterator<testing::copy_spy<testing::detail::default_copy_spy_base_t>*, std::vector<testing::copy_spy<testing::detail::default_copy_spy_base_t>, std::allocator<testing::copy_spy<testing::detail::default_copy_spy_base_t> > > > > > >(zug::last_t const&, zug::meta::bottom&&, zug::detail::iterator_range<std::reverse_iterator<__gnu_cxx::__normal_iterator<testing::copy_spy<testing::detail::default_copy_spy_base_t>*, std::vector<testing::copy_spy<testing::detail::default_copy_spy_base_t>, std::allocator<testing::copy_spy<testing::detail::default_copy_spy_base_t> > > > >, std::reverse_iterator<__gnu_cxx::__normal_iterator<testing::copy_spy<testing::detail::default_copy_spy_base_t>*, std::vector<testing::copy_spy<testing::detail::default_copy_spy_base_t>, std::allocator<testing::copy_spy<testing::detail::default_copy_spy_base_t> > > > > >&&)':
/home/ltjax/code/zug/zug/reduce_nested.hpp:47: undefined reference to `zug::meta::bottom::operator testing::copy_spy<testing::detail::default_copy_spy_base_t>&<testing::copy_spy<testing::detail::default_copy_spy_base_t> >() const'

With g++ version 7.4.0. I guess this is because bottom's usage in the any_state ctor is not actually an unevaluated context.

Yeah that is a funny one.   From the errors I do not see how any_state is involved, but maybe you think std::is_base_of is doing something funny in g++?

Here's a pretty nice explanation: https://stackoverflow.com/questions/35088599/what-are-unevaluated-contexts-in-c
Basically, unevaluated contexts are only expressions in typeid, sizeof, noexcept, and decltype. Does not really matter if is_base_of is implemented in terms of those, because this only applies to expressions, I think

I have traced the problems to the line:
auto res   = into_vector(xform, v); in the "into_vector, transduction" test
and
into_vector(reversed, std::move(v)); in the "eager, moves_data_around" test
looks like into_vector is involved

auto v   = std::vector<std::vector<int>>{{1, 2}, {3}, {4, 5, 6}};
auto res = into_vector(mapcat([](int x) { return x * 2; }), v);

Should res be vector of ints? Now it is vector of bottoms.

@ArtemSarmini what compiler and version are you using? That example is extracted from a unit test which is passing...

(Both clang and gcc from nixpkgs)

My bad, IDE warning was false positive.

Thanks for checking!

Confirmed on gcc 7.5, gcc 10.1 (which amazes me), clang 10.0 is not affected.
This stuff is hard. Gcc emits object code for zug::skip with bottom as state, linker tries to fill relocation (for me -pie is default) symbols for conversion operator call before it understands that this function is never called. IMO it's gcc's fault to it this in the first place. I know no technique to forbid this.

It seems I found a solution, but I need to either use constexpr if (which requires bumping std version to 17) or write a custom functor (maybe a couple of them). To put it short, we don't need to call zug::skip to query return type, which result_of in into_vector does and which causes the linking problem.
@arximboldi which path is preferred?

Oh or one can add more overloads and sfinae, 'll try that first.

Or tag dispatch.

I would prefer to not bump to C++17 yet. Thanks a lot for investigating this!


Add love.handlers to love standard

Fixes #161

Codecov Report

Merging #162 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
+ Coverage   97.75%   97.75%   +<.01%
==========================================
  Files          35       35
  Lines        4820     4821       +1
==========================================
+ Hits         4712     4713       +1
  Misses        108      108

Impacted Files
Coverage Δ

src/luacheck/love_standard.lua
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for the patch!


SDD data split in Table I

Hello Nachiket,
Thank you for your wonderful work and open-source code! I am struggling to find SDD data split provided by trajnet, which seems to be expired. Based on the discussion in #2, may I confirm the data files in https://github.com/nachiket92/P2T/tree/main/data/sdd correspond to the data split of Table I, which is essentially the trajnet split for SDD? And if we want to use the split used in PECNet, we have to process the data provided by PECNet github repo, is that true?
Best,
Zhe Huang
Originally posted by @tedhuang96 in https://github.com/nachiket92/P2T/issues/2#issuecomment-1105967142

Hi Zhe, yes, that's correct. The preprocessed data provided corresponds to the data split in Table I.
Table II uses tracks from the trajNet benchmark. This has the same train/val/test split of scenes. However it uses a subset of all tracks, primarily pedestrians. I believe the PECNet/Y-Net repo has this data. I can share my pre processed files as well. I see another issue here requesting them.

I really appreciate your fast response! Have a great day!

Hi Zhe, yes, that's correct. The preprocessed data provided corresponds to the data split in Table I.
Table II uses tracks from the trajNet benchmark. This has the same train/val/test split of scenes. However it uses a subset of all tracks, primarily pedestrians. I believe the PECNet/Y-Net repo has this data. I can share my pre processed files as well. I see another issue here requesting them.

@nachiket92 hi,i wondered what are the 20 scenes used in PECNet,it is not mentioned in the repo, and the data before processing in PECNet is not accessed,could you please share me with the 20 scenes used in PECNet


How to build an array instance ？

I saw in the wiki document that inflate can find the real function address of the generic function,and it's easy to build build a unity string Using Il2Cpp.String.from("")
here's some questions about how to build array instance or list instance
I tried using mscorlib.System.Array.CreateInstance(Type, Int64[]) : Array @ 0x993330
callFunction(0x993330,Il2Cpp.Image.corlib.class("System.String").type.handle,10)
but it's not working ....
Then I tried to look at their instance memory structure.
@arm64 like this :

The first pointer is the type, the fourth pointer position is the size of the array, and the pointer arrangement of the real data continues.
So I tried to directly apply for a space to manually overwrite this area according to its format, but it is not very convenient to obtain the type (At present, we can only consider finding a function with a parameter or a return value with a string[] type, and then parse it to get the handle of the type), so I want to ask if there is a better and more elegant way to build an array or list.
Thanks very much

Uhm, I don't quite understand what the problem is.
You can obtain the System.Array<System.String> class like this:
const SystemString = Il2Cpp.Image.corlib.class("System.String");
const SystemStringArray = SystemString.arrayClass; // System.String[]

I write such a demo should be able to better express my intention:
"Is there a simpler way to implement the above code function?"

The above is the construction of string[], what if it is the construction of list, as for the memory structure of list, it seems that a clear memory construction rule cannot be found simply from hexdump

I think the function you are looking for is this:
https://github.com/vfsfitvnm/frida-il2cpp-bridge/blob/master/src/il2cpp/structs/array.ts#L11
const SystemString = Il2Cpp.Image.corlib.class("System.String");
const array = Il2Cpp.Array.from(SystemString, ["hello", "how", "are", "you?"]);

yes i was looking for it
Is there a legal construct for this type? ↓
System.Collections.Generic.List
System.Collections.Generic.List

Nope, System.Collections.Generic.List<T> is a class like any other (i.e. it doesn't have its own underlying Il2Cpp* struct). You can easily call the methods the class offers!

Il2Cpp.Image.corlib.class("System.String")

Can I create an empty one

Il2Cpp.Image.corlib

If a function has a parameter of type List, and I want to convert this parameter to List to directly access the objects in the list, How could I do?


Have a left right button so users could view more pictures

May want to look at the docs to see how it is structured
docs

Hi, I would like to try to fix this issue :)


[Feature] Chart increase size on hover gets cut due to container width

What problem does this feature solve?
We have implemented an on hover method to increase the radius of chart (snapshot attached). However, it gets cut due to container width and most importantly the overflow of auto generated div of echarts is hidden (snapshot attached). Is there any way to set it to visible and also set the same to the auto generated svg?

What does the proposed API look like?
We have fixed the issue by setting overflow to visible but just from inspect element. It would be good if you can provide any style property of it in series.pie.itemStyle or any other solution.

look like the issue is with zrender lib
https://github.com/ecomfe/zrender/blob/e6d458c1e7bfeeb1949b3462d3a7d48d8dea12fc/src/svg/Painter.ts#L72

This doesn't seem to be a problem with ECharts. If the chart has bigger radius on hovering, then you should probably decrease the default radius to make the hovered radius to be less than 50%.

it's not an issue, but assuming the chart overflow should be hidden is
limiting. There are cases that you want the chart to merge with the page, a
good example of that are dashboards in mobile apps where you don't have a
lot of space to put for padding, but at the same time you want the chart to
take the full space.
On Wed, Aug 24, 2022 at 10:18 AM Wenli Zhang @.***>
wrote:

This doesn't seem to be a problem with ECharts. If the chart has bigger
radius on hovering, then you should probably decrease the default radius to
make the hovered radius to be less than 50%.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/echarts/issues/17558#issuecomment-1225094065,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACETIJ22YV6SIBNJWFXDUWLV2WA75ANCNFSM57LFH5NA
.
You are receiving this because you commented.Message ID:
@.***>

If this is the case, you should set a bigger size for your container and use an outer container to control the overflow.


Devicectl device config

Codecov Report

Merging #151 (f60a796) into prep-release (11e52f6) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Current head f60a796 differs from pull request most recent head 6c95ad7. Consider uploading reports for the commit 6c95ad7 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@              Coverage Diff              @@
##           prep-release     #151   +/-   ##
=============================================
  Coverage         47.74%   47.74%
=============================================
  Files               133      133
  Lines              4784     4784
=============================================
  Hits               2284     2284
  Misses             2500     2500

Impacted Files
Coverage Δ

src/fullctl/service_bridge/devicectl.py
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Typescript migration ?

First of all ; thanks so much for this amazing work!
If you're interested in migrating this repo to Typescript, I would be glad to participate.
I was also thinking about using a linter, as the source code is sometimes missing style consistency and a bit difficult to read.

To be honest we doubt about moving to Typescript at this point and we should think and talk about that a little bit more. But, I agree that Linter will benefit us a lot. I would say go ahead and make pr for that.
I am happy that you are also excited as much as we are and you like to contribute to this project.

@ohpax In my experience, typescript projects are much easier to maintain in the middle-long run. The "cost" is working on types which has it's own learning curve. I've been using Typescript in my projects for 3 years now, and I'm really glad I did.
Anyway, that's of course up to you! On the linter side, do you have presets / preferences ? I'm usually using airbnb javascript guide.

For the Linter I also like the airbnb one, let's go with that.
I don't think we will have any issue with using types because our first language is C# but our main issue is the most react libraries and examples are on plain Javascript and there might be more people who are interested to contribute if its plain JS. I have used some Typescript before and I like it. What do you think @imnapo ?

I can tell you a vast majority of RN libraries offer typings ; and some of them are written / migrating to typescript (example here — and here). Typescript is supported by React Native (you can use the --template=typescript flag with react-native init). However, your concern about contributions is legitimate, even though TS is relatively easy to learn.
We could also offer flow types with tools such as flowgen.

There is no doubt about advantages of using TypeScript but I'm agree with @ohpax about keep this package in javascript, at least for a while until we have more contributors.
but we can support typing like other packages written with JavaScript having type definitions. (like react-native-elements)
For the Linter side I also like the airbnb, so we can go with it.
And also thank you very much for your active participation and contribution.

@imnapo OK ; I'll work on eslint integration :slightly_smiling_face:
I already wrote typings for this package ; I'll wait to test those a bit more before offering a PR.

@imnapo I've started working on eslint ; it seems like there are both yarn and package-lock ; this is probably unwanted ; which package manager you would pick ?

I would personally prefer yarn.


Unable to use package

When trying to use the anonpy package, I get the error:
AttributeError: 'list' object has no attribute 'to_dict'on the anonimize function: anonymize(df, partitions, feature_columns, sensitive_column, max_partitions)

This errors happens even while trying to run the example code on the readme file.

Unfortunately, I get the exact same error when trying to run the example code from the documentation.

I just ran into the same error

I am also running into this issue

same. There seems to be a solution in the other threat (https://github.com/glassonion1/anonypy/issues/1) - would including this in the main branch fix the issue?


Fix Stream example in README

The example of wrapping streams in the README doesn't make sense:

We can wrap Streams:
Failsafe.with(retryPolicy).run(() -> Stream.of("foo").map(value -> value + "bar"));

There is no terminal operation on the constructed stream, so there's nothing to fail and thus nothing to retry. I suggest either removing the example entirely or adding a terminal operation that does something. (Although then it wouldn't be wrapping a stream but rather the result of a stream operation.)
The example following this doesn't actually do anything, but it's less problematic as an example.

Agree. These were meant to hint at what's possible, but that one is just misleading. Removed in 9d2de0ad2a1884bad3f6c595f4ca7f27b91d7938.


:seedling: Simplify the addition of new resources to the replication e2e test

Summary
The original idea was to look at using the dynamic client for code reuse between resources being added to the cache server.
Experimenting with it I felt that creating and manipulating resources is more painful than with the typed client.
In this pull request I tried to address a few things:

to have the needed changes localized in one place for supporting additional resources.
to separate what is resource specific, mainly get, update, delete functions from the plumbing and tests.

The changes should also allow to more easily add tests once and apply them to all the resources. In my eyes it also makes the tests easier to follow but I reckon that others may feel differently.
Related issue(s)
Addresses #https://github.com/kcp-dev/kcp/pull/2381#discussion_r1029443074
Signed-off-by: Frederic Giloux<EMAIL_ADDRESS>

Creating the normal types and just transforming them to unstructured to use one implementation using a dynamic client seems pretty straightforward, and doesn't require the switch/case/overhead in this PR.

doesn't require the switch/case/overhead in this PR.

Which switch/case?

Not sure what you mean - literally all of the code this PR touches is redundant, right? Whether we're doing a switch or having each type mock out its own get/put/delete methods ... if we just take a runtime.Object, transform to unstructured and use the dynamic client we don't need any of it, yes?

literally all of the code this PR touches is redundant, right?

It would be right if the tests were only setting metadata but it is not the case and cannot be the case when fields are mandatory.

if we just take a runtime.Object, transform to unstructured and use the dynamic client we don't need any of it, yes?

The only things that would be impacted are the init functions. All the other functions are already resource type agnostic.
In regards to the the init functions "get" and "delete" would be common (they are currently one liner) but not the data for "create" and "update". The biggest change is that this data would be supplied through yaml strings instead of structures. I can do that but I still think that the gain is low.

/retest

/retest

Closing in favor of #2578


[PoC]worker/depsolve: add a depsolve job type

Allow depsolving to be done in a worker through the job queue rather
than synchronously in composer.
The benefit this might unlock include:

no more blocking calls in the cloud/koji APIs
only workers accessing repositoires

no VPN access from composer
composer not needing to be subscribed to CDN, etc

no dnf cache managment in composer

Potential problems:

the version of composer (so the distro definitions) that
triggered a depsolve, may not be the same that uses the
result to generate a manfiset

Signed-off-by: Tom Gundersen<EMAIL_ADDRESS>This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change

Nice! Btw this will require some fine handling of dnf-json - we need to move it from osbuild-composer to `osbuild-composer-worker.

I thought some more about the problem I highlighted above, and I'm really not sure how to solve it. Imagine this situation:

osbuild-composer-100 generates a PackageSets map to be depsolved for a given image-request
osbuild-worker does as requested and returns the result in the queue to composer
by the time osbuild-composer gets around to calling Manifest() on the result, it has been updated to osbuild-composer-104, and the PackageSets no longer match the Manifest() definition, and things are sad.

A solution would be for the worker to not only do the depsolving, but also the manifest generation, which gets us down the rabbit hole of where manifest generators should live (in the distro would work, on each worker node would be.... confusing?).

One solution is to add a composer version to the job. When it finishes, check if the version is still same. Otherwise, reschedule. Might be troublesome when running multiple composers in k8s/OpenShift.
But I guess the final solution is to move the distro distribution in the distro itself...


Skill Value on SkillGump

Hi there,
I think would be very helpful to have a setting in Sphere.ini where we can decide to show a fixed value for the skillgump. For example, I would like to always show the real value (When you select "Show Real").
Something like:
ShowAlwaysRealSkillValue=1 // 1 = show real - 0 = not enabled
writeInt16((word)(i + 1));
			writeInt16(character->Skill_GetAdjusted((SKILL_TYPE)i));
			writeInt16(character->Skill_GetBase((SKILL_TYPE)i));
			writeByte((byte)(character->Skill_GetLock((SKILL_TYPE)i)));
			if (includeCaps)
				writeInt16((word)(character->Skill_GetMax((SKILL_TYPE)I)));

I hope this helps.
Best.

I don't think it's necessary, as you wrote the show real button already does that, it looks more like an "eye-candy" feature.

If this option exist, i'll delete the show real button.
This button is so useless and some player complain about it

So, I have got the issue (not really an issue).
To get this fixed it's simply needed to remove all bonuses from each skill, and it will returns always the base value.
[SKILL 1]
DEFNAME=Skill_Anatomy
KEY=Anatomy
TITLE=Biologist
PROMPT_MSG=Whom shall I examine?

I hope this helps.


Slime Dirt "spreading" does not work

Minecraft Version
1.18.2
Fabric/Quilt Version
0.14.9
Fabric/Quilt API Version
0.58.0+1.18.2
Tinkers' Construct Version
1.18.2-3.5.1.DEV.85b8f33
Describe your issue
I've never done this before, so I could definitely be doing something wrong, however apparently the way to "duplicate" slime dirt is having a slime fluid flow onto a dirt block, from a slime dirt block.
Crash Report
No response
Other mods
The modpack is Cabricality.
Tried reproducing with just Tinkers?
No
Performance Enchancers
Sodium
Searched for known issues?
Searched open issues

The creator of Tinkers' Construct has explained this is how it works (in a random reddit comment) plus there's images of it working here, but testing in the latest Forge version, this does not seem to happen anymore.


[Quest] - New Reporting Interface for initiating In-App reports for eCBIS Indicators based on FHIR Measures &  MeasureReporting

Description :
The current In-App reports may not conform to the nature of in-app reporting desired by the project teams and previous OpenSRP implementations.
Proposal :
Need to obscure selection of

Date - Start date and End Date
Selection of Subjects
which are present in the mADX approach for measure reporting here  - mADX - https://www.figma.com/file/bH9zok2Yvu6xQ7vGxY8jiC/FHIR-ANC?node-id=1907%3A59174

Reporting page should have

Type of Reports
Months  - Jan through Dec
Then just select a month and then invoke the report

Designs :
Some close designs to fulfil this requirement  - https://www.figma.com/file/Gan8PFBKHYjRd6zSPvfhnQ/WCARO---Design-edit?node-id=614%3A854

@HenryRae @rowo we can plan to have an intro and a check in on this item CC @AnnieMungai

I'd like to join that

Marked as done


codegen/runtime: change codegen to use A => F[Metadata]

Perhaps adding new signatures and deprecate is better as this a breaking change. Wonder why mima is quiet


publish pre-compiled installers/executables for the CLI tool

There's sort of a hierarchy of packaging/availability levels that projects slowly climb. We're currently at the pip install level: python developers know how to do this, but it's not a tool that's particularly accessible outside the python community.
The next level up will be to provide single-file executables, or OS-specific installers. This will use something like py2exe or one of the BeeWare tools to combine a copy of the Python executable with all the necessary libraries. The result will still be a CLI tool (making a proper GUI is a level or two higher on this scale), but it should be available to more users than what we've got now.

Er… Hello? :)

Py2exe supports Python 3 since 2014, perhaps not all of 3.* minor modifications (tutorials: one and two).
Also there are a few related answers on Stackoverflow advocating to try CXFreeze (tutorials: 3.2 and 3.6).


Remove Python2 references

There is still some references to Python2 in cothread. These are contained in the py23.py file. This file should be deleted and the relevant Python3 bits moved to where they are used.

Fixed in #49


add preselected select and multiselect values to test valid open_full_form_modal

Summary
This PR adds fields for a preselected select and multiselect form for the command test valid open_full_form_modal
The PR depends on https://github.com/mattermost/mattermost-mobile/pull/5640
Ticket Link
N/A

@mickmister this PR should either be re-done for the new test app, or closed altogether?

@levb This should be re-done, though the project structure may be so different that we should close and open a new PR


Add ability for compaction-time deletion decisions

Original issue 50 created by<EMAIL_ADDRESS>on 2011-11-02T21:15:59.000Z:
It would be handy if expiration of rows could be done lazily at compaction time. An interface similar to the comparator class instance could return true/false for each key/value pair, deciding its fate.

This is fairly old, so closing. If users still desire this change please submit a pull request.


Callout Align and Used alone

Dear developer, this theme of yours is really very beautiful, using your theme has made me very happy in using Obsidian, thank you for your dedicated development!
However, I'm having a compatibility issue with it and need to ask you to point me in the right direction.
I have a CSS snippet modified from blue topaz that centers the content of a callout whose metadata contains "center". However, in the Border theme, this centering is not working. My code is as follows:
/* === Support right left center Syntax ============ */
/* Supports two ways of writing
    > [!note|right]
    > [!note right]
   */
.callout.callout:is([data-callout-metadata*="left"],
[data-callout*=left]) {
  float: left !important;
  margin: unset;
}
.callout.callout:is([data-callout-metadata*="right"],
[data-callout*=right]) {
  float: right !important;
  margin: unset;
}

.callout.callout:is([data-callout-metadata*="center"],
[data-callout*=center]) {
  display: block;
  margin: auto;
  float: unset ;
  text-align: center;
}

I tried both "|left" and "|right" and both work fine, except "center" which fails. I tried to modify the code to realize this function, but failed. Can you please tell me how to modify the code to realize this function?
Another small requirement I have is that I would like to use a separate callout style for some callouts, while other callouts use the style set in the style setting. I can do this in the AnuPpuccin theme with the callout-metadata function, for example:
> [!note|anp-vanilla-normal] Heading

It is possible to make that callout alone to be displayed in a style like anp-vanilla-normal. I wonder if there is a way for me to do this in the Border theme?
Finally, thank you again for the effort you put into developing such a great theme and I look forward to hearing from you!

"What is the correct effect of the 'center' effect? Could you please provide some visual information? I tried your snippet, and in my environment, it performs as follows. I'm not sure if it's the same as yours."

I know, it's because I turned on the "Justify paragraph text" option that caused the centering in Callout to fail. After turning off this option everything worked fine, thanks for your help!
Also, regarding my latter question about whether it is possible to use a certain callout style on its own, can you please give me some guidance?🥺

Thank you so much! Thanks for taking my advice♥


NODE creation instruction?

Is there another place you have the instructions for creating NODES?
This link seems to be broken:
http://docs.gpilab.com/CoreNodes/
I am looking into the feasibility of using this tool for TensorFlow.

Hi Greg,
Can you tell me where you found that link? -Id like to fix it.
You can find info on writing nodes here:
http://docs.gpilab.com/en/develop/NodeDev/devguide.html
and you may need to setup a library directory with info from here:
http://docs.gpilab.com/en/develop/config.html
There is also a "new node" button in the main menu, that will write a node template to help you get started.
-Nick
Sent from my iPhone

On Aug 12, 2017, at 9:29 PM, Greg Peatfield<EMAIL_ADDRESS>wrote:
Is there another place you have the instructions for creating NODES?
This link seems to be broken:
http://docs.gpilab.com/CoreNodes/
I am looking into the feasibility of using this tool for TensorFlow.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks, I will check it out.  The link is on the top of the main "Code" page:
The core nodes are a collection of basic data manipulation and visualization algorithms. http://docs.gpilab.com/CoreNodes/

Changed the link at the top of the core-nodes repo Code main page.


Add timestamp and silent option in Events

Signed-off-by: Ashish Tiwari<EMAIL_ADDRESS>Notes for Reviewers
This PR proposes the following:

All the events will contain timestamps which can be used to sort them inside an audit log separate from notifications.
All notifications are propagated from events but not all events result in a notification. The events with silent set as true do not bubble a notification.
Audit log will be used to track all the events with their stamps whereas the events with silent set to false will bubble into notifications in realtime.

Note : This PR will require change in adapter-library's proto file as well which will be created once this PR is approved so that changes can be merged together.
Signed commits

[ ] Yes, I signed my commits.

@Abhishek-kumar09  You can make the UI changes on this PR.

Codecov Report
Base: 16.75% // Head: 12.51% // Decreases project coverage by -4.23% :warning:

Coverage data is based on head (9af5b2c) compared to base (284be09).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6646      +/-   ##
==========================================
- Coverage   16.75%   12.51%   -4.24%
==========================================
  Files         115      115
  Lines       11229    11243      +14
==========================================
- Hits         1881     1407     -474
- Misses       8873     9522     +649
+ Partials      475      314     -161

Flag
Coverage Δ

gointegrationtests
?

unittests
12.51% <0.00%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

server/handlers/meshery_application_handler.go
0.00% <0.00%> (ø)

server/handlers/meshery_filter_handler.go
0.00% <0.00%> (ø)

server/handlers/meshery_pattern_handler.go
0.00% <0.00%> (ø)

server/meshes/meshops.pb.go
3.87% <0.00%> (-0.05%)
:arrow_down:

server/meshes/meshops_grpc.pb.go
0.00% <ø> (ø)

mesheryctl/internal/cli/root/system/start.go
1.70% <0.00%> (-56.18%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
1.12% <0.00%> (-53.94%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.17% <0.00%> (-47.06%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.88% <0.00%> (-45.29%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
2.22% <0.00%> (-41.49%)
:arrow_down:

... and 7 more

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

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

I was hoping for something more robust (event levels, like log severity levels) and for reference to the design spec update in the Messaging Framework.
e.g. https://en.wikipedia.org/wiki/Syslog#Severity_level

The audit event will be one of the column according to the proposal, that will keep the time sorted history of audit events.

@Abhishek-kumar09 let me know when this is drafted in the messaging framework design spec.

CloudEvents proto -       https://github.com/cloudevents/spec/blob/main/cloudevents/formats/cloudevents.proto

Will be creating another PR using cloudevents-go-sdk

Will be creating another PR using cloudevents-go-sdk

Excellent


how can I start a newLine in FlowChart

like this

 doesn't work also ,\n,

Did anyone figure out a solution to this?

Hi! Sorry for the slow response.
The project has got good traction but it also means that it is very hard to manage all issues on your spare time. I am trying to get a bigger team together for faster development and better support #866. I am closing this one for now, not out of disrespect but as a part of an effort to clean out stale issues that has been resolved but not closed.
Is this still a problem? If so please reopen it.


Ensure we can change interpreters after trusting a workspace

ConfigSettings wasn't being deregistered properly when extension is deactivated, so it was still using the reference to older singletons before deactivation.

@heejaechang this should address the interpreter change issue you were seeing.


(PUP-6320) Fix error when reporting zero sized array mismatch

Fixes a bug in the TypeMismatchDescriber that caused the type mismatch
reported when passing a zero sized array where a non empty array is
required to report an element type mismatch rather than the size
mismatch.
Also fixes so that array mismatches are reported with correct index
rather than just stating that there's a mismatch between an Array
and a Tuple where some element in the Tuple doesn't match the element
type of the Array.

CLA signed by all contributors.


Increment version for core releases

Increment package version after release of System.ClientModel

API change check
API changes are not detected in this pull request.


Evaluate if it is possible to put the initial setup configuration in yaml files

Will rewrite most of the scripts using Ansible playbooks


chore(github): add dependabot config

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

Project coverage is 70.83%. Comparing base (013ee84) to head (ce95e2f).
 Consider uploading reports for the commit 1b4d73c to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             next      #13   +/-   ##
=======================================
  Coverage   70.83%   70.83%
=======================================
  Files          10       10
  Lines          72       72
  Branches        9        9
=======================================
  Hits           51       51
  Misses         21       21

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


Speed up recursive find for VCF URIs

Use VFS.ls_recursive when recursively searching for VCF files.

@normanb we should look into using ls_recursive for tiledb.cloud.utilities.find. It wasn't immediately obvious how to return an iterator without letting the ls_recursive call finish.


Create test.txt

@pbhandar-amd,
I suppose this PR was opened to verify PR to CR.
If not needed anymore, can we close this?


Add support for X-Couch-Trace header

Overview
Sometime it is useful to be able to trigger open tracing using single header in the request.
For such cases this PR introduces X-Couch-Trace header. This header combined with the following config in default.ini would cause tracing of the request.
[tracing.filters]
all = (#{external := External}) when External == true -> true

Note:

We check for existence of the header and don't care about its value
This feature could allow malicious user to launch resource exhaustion attack on tracing infrastructure (elastic search backend of jaeger). You are absolutely safe if you don't use (#{external := External}) when External == true.

Testing recommendations

update rel/overlay/etc/default.ini as follows

diff --git a/rel/overlay/etc/default.ini b/rel/overlay/etc/default.ini
index 592445c28..69091eab3 100644
--- a/rel/overlay/etc/default.ini
+++ b/rel/overlay/etc/default.ini
@@ -572,15 +572,15 @@ min_priority = 2.0
 ;   - jaeger.thrift over udp
 ;   - jaeger.thrift over http
 ; ## Common settings
-; enabled = false ; true | false
-; app_name = couchdb ; value to use for the `location.application` tag
-; protocol = udp ; udp | http - which reporter to use
+enabled = true ; true | false
+app_name = couchdb ; value to use for the `location.application` tag
+protocol = http ; udp | http - which reporter to use
 ; ## jaeger.thrift over udp reporter
 ; thrift_format = compact ; compact | binary
 ; agent_host = <IP_ADDRESS>
 ; agent_port = 6831
 ; ## jaeger.thrift over udp reporter
-; endpoint = http://<IP_ADDRESS>:14268
+endpoint = http://<IP_ADDRESS>:14268
 [tracing.filters]
 ;
@@ -603,4 +603,4 @@ min_priority = 2.0
 ; corresponding operation name key configured. Thus, users can easily
 ; log every generated trace by including the following:
 ;
-; all = (#{}) -> true
+all = (#{external := External}) when External == true -> true

start jaeger in docker container

docker run -d -p 6831:6831/udp -p 16686:16686 -p 14268:14268 jaegertracing/all-in-one:1.14

Start couchdb dev/run --admin=adm:pass
Create database curl -X PUT -u adm:pass http://<IP_ADDRESS>:15984/test
Query database curl -X GET -u adm:pass http://<IP_ADDRESS>:15984/test/_all_docs -H 'X-Couch-Trace: true'
Open http://localhost:16686/ in the browser and search for spans (you should find some traces)

Checklist

[x] Code is written and works correctly
[ ] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

Why a custom header? W3C has a trace header https://www.w3.org/TR/trace-context/ and it is being supported by many languages and vendors in the OpenTracing/OpenCensus successor https://opentelemetry.io/, including Erlang/Elixir https://github.com/open-telemetry/opentelemetry-erlang-api/

Currently open tracing implementation in CouchDB supports B3 headers (see https://github.com/apache/couchdb/tree/prototype/fdb-layer/src/ctrace#b3-propagation). The B3 headers were invented by zipkin and ended up in some opentracing solutions.
Unfortunately semantics of B3 headers doesn't allow expressing the following:

I want to trace this request but I don't have RootSpan please create it for me

I was not aware that there is a concurrent W3C standard. It is probably quite new. Here is what I found on Jaeger page:
In the future Jaeger will support the upcoming W3C Trace-Context specification.....

I briefly looked into W3C standard this morning. It doesn't look like there is a specific header which can be used to enable tracing without specifying root span at the same time. Although I admit my 15 min reading of the spec could miss it.

Ah ok, now I get what you are doing.
For B3 I think they actually have a "debug" header that you can send without id's that is meant to force a trace. But w3c traceheader I don't think has added such a feature yet.
Probably no reason to depend on it for this simple case of forcing a trace with no root from the client.
But keep an eye on opentelemetry :), the spec and many of the clients (hopefully including the Erlang/Elixir library) will be going into beta in March. Can't tell what you use currently for tracing and reporting traces -- something custom and built-in?

But keep an eye on opentelemetry :), the spec and many of the clients (hopefully including the Erlang/Elixir library) will be going into beta in March.

Thank you this is definitely good to know.


[GSSOC'23]Adding a feature to get notifications in the Whatsapp app

I want to add a feature that will send notifications in WhatsApp. Since many people check WhatsApp daily, this feature would be useful
Can you please assign this project under GSSOC'23

Hello, that would be a pretty major update and a good one too! How are you planning to do that? Because as far as I am aware Whatsapp API is a paid product.

As I will be using the Twilio API, an official WhatsApp Business Solution Provider, I will be able to integrate WhatsApp messaging into my customer engagement workflows. However, due to WhatsApp's policies and guidelines for messaging on its platform, I will need to send the link of the contest in text format, as WhatsApp templates do not allow sending links.

Twilio's whatsapp free plan allows sending 1000 messages per month. Based on the total contest notifications this bot sent in the last 30 days, the free plan can only support sending notifications to max 10 users for free. That's too low a number and as I don't plan to monetize this bot, this would be an added expense I wouldn't like to bear. A cool idea still, you are free to make a whatsapp bot similar to this discord bot on your own if you would like to.


Please advise on how to filter out unwanted content

I studied the source code as a whole and found that I mainly use turndown to convert html into Markdown.
What puzzles me is that the things returned by your interface are very clean and do not contain some useless information on the page. I am very curious about how you do it. Specifically, for example, the navigation bar and comment area of some pages will not appear in the content returned by your interface.
I would like to ask you what kind of processing you have done? Or is there some source code that I haven't seen?

The cleaning part is done by mozilla/readability. Sometime it overdoes, filtering out too much information, and you can set request header to control the levels of details. Please read this section

The cleaning part is done by mozilla/readability. Sometime it overdoes, filtering out too much information, and you can set request header to control the levels of details. Please read this section

Thank you very much for your reply. Let me take a look at the guidance you provided and delve deeper into it


corrected Potion Item display at /jobs info

Now, with material in jobConfig: TIPPED_ARROW:SLOWNESS it will be displayed as Tipped Arrow:SLOWNESS at /jobs info.
Before, it was just "Tipped Arrow" for each potion arrow.
I could not figure out how to respect translatableWords but now we can at least show differences with original potion names.

Now, with material in jobConfig: TIPPED_ARROW:SLOWNESS it will be displayed as Tipped Arrow:SLOWNESS at /jobs info.
Before, it was just "Tipped Arrow" for each potion arrow.

I think it only displayed the name Tipped Arrow because there was no "_" bottom line specified to be equal to the effect name.
The use of materials, entity, etc... names in jobConfig has changed for about 1 year. So, it is recommended to enter all material/enchant names in small letter for the actions, without exception. This is to avoid accidentally some users would not be able to write in capital letters.

Okay, please feel free to change the names in Potion enum and CMIMaterial enum to lowecase.

It's not needed. Every material/entity in upper-cased letter for better check. As you can see we using equalsIgnoreCase which is good for checking for cases.

I made these changes you suggested.

I’ve made some changes that makes these work, so now it pays when we craft a tipped arrow.
Thanks for your PR!


fix ExpansionPanelList merge the header semantics when it is not nece…

…ssart
Description
ExpansionPenalList always wraps its children with MergeSemantics even with canTapOnHeader is false. There is no point for merging header into one semantics if only the ExpandIcon is clickable.
One use case is user expect to be able to have textspan recognizer in the header, but it will cause a crash at the moment.
Related Issues
https://github.com/flutter/flutter/issues/33774
Tests
I added the following tests:
'ExpansionPanelList does not merge header when canTapOnHeader is false'
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I signed the CLA.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.

addressed all coments

Addressed all comments


[ERROR] InnoDB: Operating system error number 13 in a file operation. Persistent Volume using fstab/cifs

I am wondering if you can tell me what I need to set to allow InnoDB permissions to the Persistent Volume using fstab/cifs.  Data is successfully written to the cifs share but when the pod spins up, it errors out because InnoDB doesn't appear to have permissions.  The front end pod spins up without any issues. What is the best way to resolve this or is it an issue with the FlexVolume driver?
apiVersion: v1
kind: PersistentVolume
metadata:
name: mariadb-pv
spec:
capacity:
storage: 1Gi
volumeMode: Filesystem
accessModes:
- ReadWriteMany
persistentVolumeReclaimPolicy: Retain
flexVolume:
driver: "fstab/cifs"
fsType: "cifs"
readOnly: false
secretRef:
name: "cifs-secret"
options:
networkPath: "\\<IP_ADDRESS>\moshpit-str2\Kubernetes\mariadb\"
mountOptions: "dir_mode=0755,file_mode=0755,noperm"

kind: PersistentVolumeClaim
apiVersion: v1
metadata:
name: mariadb-pvc
namespace: drupal
spec:
accessModes:
- ReadWriteMany
resources:
requests:
storage: 1Gi

kind: Deployment
apiVersion: apps/v1
metadata:
name: mariadb
namespace: drupal
spec:
replicas: 1
selector:
matchLabels:
app: mariadb
template:
metadata:
labels:
app: mariadb
spec:
containers:
- name: mariadb
image: mariadb:latest
ports:
- containerPort: 3306
env:
- name: MYSQL_DATABASE
value: xxx
- name: MYSQL_USER
value: xxx
- name: MYSQL_PASSWORD
value: xxx
- name: MYSQL_RANDOM_ROOT_PASSWORD
value: 'yes'
volumeMounts:
- mountPath: /var/lib/mysql/
name: database
resources:
limits:
cpu: '2'
memory: '512Mi'
requests:
cpu: '500m'
memory: '256Mi'
volumes:
- name: database
persistentVolumeClaim:
claimName: mariadb-pvc

kind: Service
apiVersion: v1
metadata:
name: mariadb
namespace: drupal
spec:
ports:
- port: 3306
targetPort: 3306
selector:
app: mariadb
Mysql Pod Logs
2021-04-23 00:32:34+00:00 [Note] [Entrypoint]: Entrypoint script for MySQL Server 1:10.5.9+maria~focal started.
2021-04-23 00:32:34+00:00 [Note] [Entrypoint]: Switching to dedicated user 'mysql'
2021-04-23 00:32:34+00:00 [Note] [Entrypoint]: Entrypoint script for MySQL Server 1:10.5.9+maria~focal started.
2021-04-23  0:32:35 0 [Note] mysqld (mysqld 10.5.9-MariaDB-1:10.5.9+maria~focal) starting as process 1 ...
2021-04-23  0:32:35 0 [Note] InnoDB: Uses event mutexes
2021-04-23  0:32:35 0 [Note] InnoDB: Compressed tables use zlib 1.2.11
2021-04-23  0:32:35 0 [Note] InnoDB: Number of pools: 1
2021-04-23  0:32:35 0 [Note] InnoDB: Using ARMv8 crc32 instructions
2021-04-23  0:32:35 0 [Note] mysqld: O_TMPFILE is not supported on /tmp (disabling future attempts)
2021-04-23  0:32:35 0 [Note] InnoDB: Using Linux native AIO
2021-04-23  0:32:35 0 [Note] InnoDB: Initializing buffer pool, total size = 134217728, chunk size = 134217728
2021-04-23  0:32:35 0 [Note] InnoDB: Completed initialization of buffer pool
2021-04-23  0:32:35 0 [Note] InnoDB: If the mysqld execution user is authorized, page cleaner thread priority can be changed. See the man page of setpriority().
2021-04-23  0:32:35 0 [Warning] InnoDB: Retry attempts for reading partial data failed.
2021-04-23  0:32:35 0 [ERROR] InnoDB: Operating system error number 13 in a file operation.
2021-04-23  0:32:35 0 [ERROR] InnoDB: The error means mysqld does not have the access rights to the directory.
2021-04-23  0:32:35 0 [ERROR] [FATAL] InnoDB: Tried to read 65536 bytes at offset 38400, but was only able to read 0.Cannot read from file. OS error number 13.
210423  0:32:35 [ERROR] mysqld got signal 6 ;
This could be because you hit a bug. It is also possible that this binary
or one of the libraries it was linked against is corrupt, improperly built,
or misconfigured. This error can also be caused by malfunctioning hardware.
To report this bug, see https://mariadb.com/kb/en/reporting-bugs
We will try our best to scrape up some info that will hopefully help
diagnose the problem, but since we have already crashed,
something is definitely wrong and this may fail.
Server version: 10.5.9-MariaDB-1:10.5.9+maria~focal
key_buffer_size=134217728
read_buffer_size=131072
max_used_connections=0
max_threads=153
thread_count=0
It is possible that mysqld could use up to
key_buffer_size + (read_buffer_size + sort_buffer_size)*max_threads = 467871 K  bytes of memory
Hope that's ok; if not, decrease some variables in the equation.
Thread pointer: 0x0
Attempting backtrace. You can use the following information to find out
where mysqld died. If you see no messages after this, something went
terribly wrong...
stack_bottom = 0x0 thread_stack 0x49000
mysqld(my_print_stacktrace+0x30)[0x55587a6a00]
Printing to addr2line failed
mysqld(handle_fatal_signal+0x45c)[0x5558262e2c]
linux-vdso.so.1(__kernel_rt_sigreturn+0x0)[0x7f85f5f7c0]
/lib/aarch64-linux-gnu/libc.so.6(gsignal+0xe0)[0x7f855fa138]
/lib/aarch64-linux-gnu/libc.so.6(abort+0x110)[0x7f855e6d68]
mysqld(+0xcb4fd0)[0x5558644fd0]
mysqld(+0xbf3ba8)[0x5558583ba8]
mysqld(+0xbdbb4c)[0x555856bb4c]
mysqld(+0xbe1f8c)[0x5558571f8c]
mysqld(+0xbe71fc)[0x55585771fc]
mysqld(+0xbe77fc)[0x55585777fc]
mysqld(+0x610654)[0x5557fa0654]
mysqld(+0xb7b92c)[0x555850b92c]
mysqld(_Z24ha_initialize_handlertonP13st_plugin_int+0x78)[0x5558265ce8]
mysqld(+0x70f83c)[0x555809f83c]
mysqld(_Z11plugin_initPiPPci+0x860)[0x55580a0910]
mysqld(+0x646850)[0x5557fd6850]
mysqld(_Z11mysqld_mainiPPc+0x424)[0x5557fdc14c]
/lib/aarch64-linux-gnu/libc.so.6(__libc_start_main+0xe8)[0x7f855e7090]
mysqld(+0x641648)[0x5557fd1648]
The manual page at https://mariadb.com/kb/en/how-to-produce-a-full-stack-trace-for-mysqld/ contains
information that should help you find out what is causing the crash.
Writing a core file...
Working directory at /var/lib/mysql
Resource Limits:
Limit                     Soft Limit           Hard Limit           Units
Max cpu time              unlimited            unlimited            seconds
Max file size             unlimited            unlimited            bytes
Max data size             unlimited            unlimited            bytes
Max stack size            8388608              unlimited            bytes
Max core file size        unlimited            unlimited            bytes
Max resident set          unlimited            unlimited            bytes
Max processes             unlimited            unlimited            processes
Max open files            1048576              1048576              files
Max locked memory         65536                65536                bytes
Max address space         unlimited            unlimited            bytes
Max file locks            unlimited            unlimited            locks
Max pending signals       30215                30215                signals
Max msgqueue size         819200               819200               bytes
Max nice priority         0                    0
Max realtime priority     0                    0
Max realtime timeout      unlimited            unlimited            us
Core pattern: core

I moved to an nfs provisioner.


Multiple jobs for the same segment

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Expected Behaviour

The idea is to have multiple annotators to the same set of images (so we can cross-validate them).
Current Behaviour

Currently we need to create multiple tasks with the same image set, which waste space.
Possible Solution

Two approaches:

One task, two jobs, same frame range for them, one assignee per job. (maybe one option for the user to input which frames this job is linked to)
One task, one job, multiple assignees per job (perhaps like the Gitlab's label selection UI component)

I think the general idea is to provide a manual control over the range of frames per job (the overlap/segment size/start/end would be the automatic version of it). Nowadays we can't have a full overlap (otherwise we have a infinite loop with the current CVAT code).
Steps to Reproduce (for bugs)

Not applicable (I think the issue description is self contained)
Context

Datasets whose subsets can be annotated by multiple users.
 Docker
Operating System and version (e.g. Linux, Windows, MacOS): Linux 5.6.11-1-MANJARO #1 SMP PREEMPT Wed May 6 10:20:32 UTC 2020 x86_64 GNU/Linux
Code example or link to GitHub repo or gist to reproduce problem:

https://github.com/opencv/cvat/issues/125 (similar issue)

Let's close it because it is a dublicate


Change loader styles

My proposition is to change loader styles to center it and fix it on mobile view

works like a charm!


Uniform color styling for all 'Cancel' buttons.

Change the colour of all red Cancel buttons to black (standard colour for all Cancel buttons) in the following:

admin_user_list.handlebars
message_edit_form.handlebars
invite_user.html

In this PR, commit 56eea6a resolves issue https://github.com/zulip/zulip/issues/8548.

@utkarsh23 did you do an audit with git grep for what other cancel buttons there are?

@timabbott yeah I did and I only found these three instances of red buttons

Nice, merged, thanks @utkarsh23!

@utkarsh23 can you also do an audit of the other uses of btn-danger and see if they're in a similar role?  E.g. I just merged 50d8a2436840b4a17558906f4f20b1458d69843f to fix a very similar issue with a "Close" button.

@timabbott I'll submit a PR soon


Doesn't support assertion

2022-01-01 SHELL OIL
    asset:checking           $-8.42 = $11373.17
    expense:transport:gas           $8.42

Formatting on the ledger above panics with
thread 'main' panicked at src/formatter/mod.rs:190:41:
attempt to subtract with overflow
stack backtrace:
   0: rust_begin_unwind
             at /rustc/730d5d4095a264ef5f7c0a0781eea68c15431d45/library/std/src/panicking.rs:662:5
   1: core::panicking::panic_fmt
             at /rustc/730d5d4095a264ef5f7c0a0781eea68c15431d45/library/core/src/panicking.rs:74:14
   2: core::panicking::panic_const::panic_const_sub_overflow
             at /rustc/730d5d4095a264ef5f7c0a0781eea68c15431d45/library/core/src/panicking.rs:181:21
   3: hledger_fmt::formatter::format_content
             at ./src/formatter/mod.rs:190:41
   4: hledger_fmt::main
             at ./src/main.rs:107:25
   5: core::ops::function::FnOnce::call_once
             at /rustc/730d5d4095a264ef5f7c0a0781eea68c15431d45/library/core/src/ops/function.rs:250:5

Changing 2 to 3 at the line solves the problem tho but I have no idea what the magic number does


Use $validators instead of $parsers/$formatters

I have a case where i need to loop through all $validators and set them to valid manually. I have to do this when the input fields are disabled and should therefore not be validated.
My problem is, ng-iban does not use $validators but $parsers/$formatters and sets the validity manually with $setValidity. In this case, the ng-iban validator is not registered in the $validators pipeline.
It would be nice if ng-iban would use the $validtors pipeline.
Please see http://blog.thoughtram.io/angularjs/2015/01/11/exploring-angular-1.3-validators-pipeline.html
Thank you.

Oops, i will have a look at the tests...


Scaper breaks with truncnorm in scipy==1.5.1 (latest)

In the latest Scipy version, Scaper is broken due to an underlying change in the type returned by scipy.stats.truncnorm.rvs. This used to return a float and now returns an array for some reason. Here is a quick example:
❯ pip install -U scipy==1.4.0
Requirement already up-to-date: scipy==1.4.0 in /Users/prem/miniconda3/lib/python3.7/site-packages (1.4.0)
Requirement already satisfied, skipping upgrade: numpy>=1.13.3 in /Users/prem/miniconda3/lib/python3.7/site-packages (from scipy==1.4.0) (1.19.0)
❯ python
Python 3.7.7 (default, May  6 2020, 04:59:01)
[Clang 4.0.1 (tags/RELEASE_401/final)] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import scaper
>>> scaper.util._sample_trunc_norm(1, .1, 0, 5, 0)
1.0122664227574725
>>>
❯ pip install -U scipy
Collecting scipy
  Using cached scipy-1.5.1-cp37-cp37m-macosx_10_9_x86_64.whl (28.7 MB)
Requirement already satisfied, skipping upgrade: numpy>=1.14.5 in /Users/prem/miniconda3/lib/python3.7/site-packages (from scipy) (1.19.0)
Installing collected packages: scipy
  Attempting uninstall: scipy
    Found existing installation: scipy 1.4.0
    Uninstalling scipy-1.4.0:
      Successfully uninstalled scipy-1.4.0
Successfully installed scipy-1.5.1
❯ python
Python 3.7.7 (default, May  6 2020, 04:59:01)
[Clang 4.0.1 (tags/RELEASE_401/final)] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import scaper
>>> scaper.util._sample_trunc_norm(1, .1, 0, 5, 0)
array([1.01226642])
>>>

This causes a lot of downstream errors in formatting in Sox, in the tests that check if things are a real number, etc. Great test suite! The fix should be simple. Convert the output of _sample_trunc_norm in Scaper to return a scalar. I believe this was caused by vectorizing the truncnorm function, though as far as I can tell, the change in return type is somewhat undocumented...
https://github.com/scipy/scipy/issues/11299
I believe the fix should look like this:
def _sample_trunc_norm(mu, sigma, trunc_min, trunc_max, random_state):
    '''
    Return a random value sampled from a truncated normal distribution with
    mean ```mu``` and standard deviation ```sigma``` whose values are limited
    between ```trunc_min``` and ```trunc_max```.

    Parameters
    ----------
    mu : float
        The mean of the truncated normal distribution
    sig : float
        The standard deviation of the truncated normal distribution
    trunc_min : float
        The minimum value allowed for the distribution (lower boundary)
    trunc_max : float
        The maximum value allowed for the distribution (upper boundary)
    random_state : mtrand.RandomState
        RandomState object used to sample from this distribution.

    Returns
    -------
    value : float
        A random value sampled from the truncated normal distribution defined
        by ```mu```, ```sigma```, ```trunc_min``` and ```trunc_max```.

    '''

    # By default truncnorm expects a (lower boundary) and b (upper boundary)
    # values for a standard normal distribution (mu=0, sigma=1), so we need
    # to recompute a and b given the user specified parameters.
    a, b = (trunc_min - mu) / float(sigma), (trunc_max - mu) / float(sigma)
    # old line
    # return scipy.stats.truncnorm.rvs(a, b, mu, sigma, random_state=random_state)
    # new line
    return scipy.stats.truncnorm.rvs(a, b, mu, sigma, random_state=random_state).item()

.item coerces the array to a scalar. This works locally for me.

Per offline discussion with @pseeth, we need to make sure the fix is backwards compatible with scipy 1.4, i.e. if truncnorm.rvs returns a scalar, the code should still work.
Updated proposed solution: cast to array and then back to scalar
return np.array(scipy.stats.truncnorm.rvs(a, b, mu, sigma, random_state=random_state)).item()

The linked issue remarks that there may be a speed hit in Scipy 1.5.1. So I just ran the test suite locally and checked the speed.
1.5.1 -> ran in 150.55 seconds
1.4.0 -> ran in 145.13 seconds

Nothing remarkable there. Just logging it here for posterity.

Updated locally with coercing to array and then back to scalar. Passes tests in 1.4.0 as well as 1.5.1.
.item() exists back to at least numpy 1.13: https://numpy.org/doc/1.13/reference/generated/numpy.ndarray.item.html?highlight=item#numpy.ndarray.item. So we're good there.
Making the PR!

Closed by #108


Expose bit 6 of the glyf first outline flags byte

My 160413-EvenOddTT issue exposes a problem that the macOS Sierra rasterizer sometimes displays overlapping TT (glyf) contours as "white". This behavior can be suppressed if one sets "bit 6 of the first Outline Flag byte to 1". While the official Apple TrueType spec and the OpenType spec treat this bit as "reserved", Apple's introductory chapter to AAT explains that this bit is actually used to control the rendering of overlapping contours in the Apple rasterizer. Currently, no easy-to-use tools expose this bit, so setting it requires binary work.
It would be useful if this bit was exposed to fontTools and/or to the .ttx dump.
I'm not sure how to do it elegantly, especially since the contour.pt structure in fontTools is not a straight binary dump but is a bit prettified. Since the official spec treats the flag as reserved, and the AAT chapter only mentions the first flags byte of a glyph, an option in fontTools could trigger .ttx to dump/merge the following syntax, and of course a .flag6 boolean attribute could be added to the pt object:
<TTGlyph name="O" xMin="59" yMin="0" xMax="768" yMax="702">
      <contour>
        <pt x="674" y="440" on="1" flag6="1"/>
        <pt x="674" y="269" on="1"/>
        <pt x="140" y="269" on="1"/>
        <pt x="140" y="440" on="1"/>
      </contour>
      <instructions><assembly>
        </assembly></instructions>
    </TTGlyph>

A prettier implementation would be to introduce a boolean attribute on a fontTools.ttLib.tables._g_l_y_f.Glyph level, akin to isComposite but named hasOvelaps or hasOverlapsAAT, that would conditionally appear in the .ttx dump as an optional attribute of the <TTGlyph> element (if it is set in the font), and would be coordinated with the bit 6 of the first flag byte of the glyf strucutre for the glyph. The name hasOverlapsAAT would hint at its origin.
<TTGlyph name="O" xMin="59" yMin="0" xMax="768" yMax="702" hasOverlapsAAT="1">
      <contour>
        <pt x="674" y="440" on="1"/>
        <pt x="674" y="269" on="1"/>
        <pt x="140" y="269" on="1"/>
        <pt x="140" y="440" on="1"/>
      </contour>
      <instructions><assembly>
        </assembly></instructions>
    </TTGlyph>

Is there a test font available in which this flag is used? I thought Skia, but as it is a variable font, it doesn’t need to use this flag.

Ah, Skia has indeed have the flag set on Q.

Shall FreeType be extended to expose this bit to the user?    If yes, how?
Unfortunately, the 'natural' solution isn't possible, i.e., being part of the FT_Outline structure, since all additional bits for its tags array are already taken.

I’ve implemented the bit in the simplest way (attached to the actual point that carries the flag in the compiled glyf table): https://github.com/fonttools/fonttools/commit/38563726473e66705d5c83ff929e749efb55ef59
Should add some test data for this ...

Hm, it seems that flags bit 6 is not supported in WOFF2, or at least https://github.com/google/woff2 ignores it.

There was also a discussion about this in the OT Sanitizer thread, and I’m not sure whether ots actually allows it or not https://github.com/khaledhosny/ots/issues/146

Already fixed.


Create DataSet entity

a mirror of current Run, plus linking (1:N) to the original Run

Don't you mean M:1 to the Run association ?

Yes, the datapoints have link to the source Run.


Infer missing a bug for a small example?

Consider the following C program,
#include <stdlib.h>

int main()
{
  unsigned i;
  int* x[10];
  for (i = 0; i < 10; ++i) {
    if (i == 6)
      x[i] = NULL;
    else
      x[i] = (int*)malloc(sizeof(int));
  }

  for(i = 0; i < 10; ++i) {
    *(x[i]) = i;
    free(x[i]);
  }
  return 0;
}

There is a NULL pointer deference bug in this program. However, Infer (v0.9.4) does not catch it with the default setting,
$ infer -- clang test1.c
Capturing in make/cc mode...
Starting analysis...

legend:
  "F" analyzing a file
  "." analyzing a procedure

Found 1 source files in /mnt/local/infer-issues/infer-out
F.

  No issues found

Here is the spec for function main,
Procedure: main
int main()
Timestamp: 3
Status: INACTIVE
Phase: RE_EXECUTION
Dependency_map:
ERRORS:
WARNINGS:
TIME:0.000000 s FAILURE:SYMOPS TIMEOUT (334) SYMOPS:334 CALLS:0,0

So I tried to increase time limit suggested in the issue #572. However, Infer still didn’t catch the bug.
$ infer --iterations 15 --symops-per-iteration 100 -- clang test1.c
Capturing in make/cc mode...
Starting analysis...

legend:
  "F" analyzing a file
  "." analyzing a procedure

Found 1 source files in /mnt/local/infer-issues/infer-out
F.

  No issues found

$ InferPrint infer-out/specs/main.fad58de7366495db4650cfefac2fcd61.specs
Procedure: main
int main()
Timestamp: 3
Status: INACTIVE
Phase: RE_EXECUTION
Dependency_map:
ERRORS:
WARNINGS:
TIME:0.000000 s FAILURE:NONE SYMOPS:462 CALLS:0,0

It seems there was no timeout for this run. Please let me know if I invoked Infer correctly and the result was expected or not.
Thanks for your time and I am looking forward to your reply.

Infer is quite aggressive in abstracting away array values so I'm not surprised it doesn't find this bug. If you want to find out more you can pass --debug to infer:

it will show you if the error was filtered out
if you are curious you can inspect the intermediate values of the analysis in "infer-out/captured/test1.c.*.html" (warning: this displays the internal symbolic execution state of infer which can be quite verbose).

Also, note that this example can crash on dereferencing any of the array indices, not just 6, as malloc() may return NULL. I tried changing your example to exit if malloc() fails, but then infer gets confused about arithmetic and reports an array out of bounds.
Looking at the debug html, it seems that infer only remembers the last value put in the array in this example.

@jvillard Thanks for your reply. I tried to change the program so that it exits when malloc fails as you suggested and got the same result. I was wondering if there is something that I can do to improve the precision.
If it's convenient for you, would you mind taking a look at another example that is a slightly modified version of this one?
#include <stdlib.h>

int main()
{
  unsigned i;
  int* x = (int*)malloc(sizeof(int));
  if (!x)
    return 1;
  for (i = 0; i < 10; ++i) {
    if (i == 6) {
      free(x);
      x = NULL;
    }
  }

  return *x;
}

Here is the result I got,
$ infer -- clang test2.c -c
Capturing in make/cc mode...
Starting analysis...

legend:
  "F" analyzing a file
  "." analyzing a procedure

Found 1 source files in /mnt/local/infer-issues/infer-out
F.

Found 2 issues

test2.c:16: error: NULL_DEREFERENCE
  pointer x last assigned on line 12 could be null and is dereferenced at line 16, column 10
  14.     }
  15.
  16. >   return *x;
  17.   }
  18.

test2.c:16: error: MEMORY_LEAK
   memory dynamically allocated to x by call to malloc() at line 6, column 18 is not reachable after line 16, column 3
  14.     }
  15.
  16. >   return *x;
  17.   }
  18.

Summary of the reports

  NULL_DEREFERENCE: 1
       MEMORY_LEAK: 1

Infer catches the NULL pointer deference bug and reports a false positive on memory leak. Moreover, the error trace suggests some loop iterations are abstracted away. I was wondering if it's possible for Infer to show a complete error trace similar to what we would see using gdb.
test2.c:16: error: NULL_DEREFERENCE
  pointer x last assigned on line 12 could be null and is dereferenced at line 16, column 10
Showing all 12 steps of the trace

test2.c:3: start of procedure main()
1.   #include <stdlib.h>
2.
3. > int main()
4.   {
5.

test2.c:6:
4.   {
5.     unsigned i;
6. >   int* x = (int*)malloc(sizeof(int));
7.     if (!x)
8.

test2.c:7: Condition is true
5.     unsigned i;
6.     int* x = (int*)malloc(sizeof(int));
7. >   if (!x)
8.       return 1;
9.

test2.c:7: Condition is false
5.     unsigned i;
6.     int* x = (int*)malloc(sizeof(int));
7. >   if (!x)
8.       return 1;
9.

test2.c:9: Condition is true
7.     if (!x)
8.       return 1;
9. >   for (i = 0; i < 10; ++i) {
10.       if (i == 6) {
11.

test2.c:10: Condition is false
8.       return 1;
9.     for (i = 0; i < 10; ++i) {
10. >     if (i == 6) {
11.         free(x);
12.

test2.c:9: Condition is true
7.     if (!x)
8.       return 1;
9. >   for (i = 0; i < 10; ++i) {
10.       if (i == 6) {
11.

test2.c:10: Condition is true
8.       return 1;
9.     for (i = 0; i < 10; ++i) {
10. >     if (i == 6) {
11.         free(x);
12.

test2.c:11:
9.     for (i = 0; i < 10; ++i) {
10.       if (i == 6) {
11. >       free(x);
12.         x = NULL;
13.

test2.c:12:
10.       if (i == 6) {
11.         free(x);
12. >       x = NULL;
13.       }
14.

test2.c:9: Condition is false
7.     if (!x)
8.       return 1;
9. >   for (i = 0; i < 10; ++i) {
10.       if (i == 6) {
11.

test2.c:16:
14.     }
15.
16. >   return *x;
17.   }
18.

The complete trace with intermediary states is in the html debug (with --debug): infer-out/captured/*.html.


Update README logo reference

Fixes | Closes | Resolves #

Please remove anything marked as optional that you don't need to fill in.
Choose one of the keywords preceding to refer to the issue this PR solves, followed by the issue number (e.g Fixes # 666).
If there is no issue, remove the line. Remove this note after reading.

Changes introduced in this PR:

Updates readme logo reference

Types of changes
What types of changes does your PR introduce?
Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[ ] Yes
[ ] No

In case you checked yes, did you write tests?

[ ] Yes
[ ] No

Documentation
Does your contribution include breaking changes or deprecations?
If so have you updated the documentation?

[ ] Yes, docstrings
[ ] Yes, main documentation
[ ] Yes, deprecation notices

Further comments (optional)
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 and more.

The CI triggers need to be updated to run based on push/PRs to release/2022.10.1


Add metrics section in ob doc

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

Project coverage is 50.50%. Comparing base (9770bf4) to head (993c39a).

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #543       +/-   ##
===========================================
+ Coverage   39.19%   50.50%   +11.31%
===========================================
  Files         166      166
  Lines       20377    20377
===========================================
+ Hits         7986    10292     +2306
+ Misses      11760     9294     -2466
- Partials      631      791      +160

Flag
Coverage Δ

50.50% <ø> (+11.31%)
:arrow_up:

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

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


Doc Build Failure: ICE: index out of bounds: the len is 1 but the index is 2

Current nightly has an ICE that hits a widely used dependency (time), causing over half of the recent build failures. Once this is fixed we will rebuild all crates affected. First affected nightly was rustc 1.64.0-nightly (c2f428d2f 2022-07-14).

Issue: https://github.com/rust-lang/rust/issues/99261
Fix: https://github.com/rust-lang/rust/pull/99263

@Nemo157 it looks like the ICE was fixed in the meantime, does it?
Which would be the first fixed nightly , coming from the merge time of the fix?
Then I could just query all failed builds for these and requeue them.
Or were you planning on grepping the logs?

Yep, it should be fixed in the 2022-07-17 toolchain. I'm running my scrape-recent-failures 2022-07-13 "index out of bounds: the len is 1 but the index is 2', compiler/rustc_builtin_macros" script right now to grab all the builds affected by this. (I should add a similar command to the binary so we can do the check on the server and queue them straight away instead of going over the web, though I guess it still needs to hit S3 for each build-log).

Queued new builds for:
salvo 0.27.0
salvo_extra 0.27.0
salvo_core 0.27.0
aim 1.5.2
pooly 0.2.1
orly 0.1.3
tracing-init 0.1.0
release_plz_core 0.2.18
clyde 0.1.0
disarray 0.1.11
poem-dbsession 0.3.35
poem-openapi 2.0.5
poem 1.3.35
wasmbus-rpc 0.9.3
rsonpath 0.1.0
wasmcloud-test-util 0.4.1
wasmcloud-control-interface 0.16.2
rsonpath 0.1.0
datapack 2.0.1
cobble-core 0.4.1
mungos 0.2.12


Allow to customise connections arguments

I might not know how to do this, but I think it would be nice if we could allow to remove/change the arguments for relay connections easily.
My current (maybe weird) usecase is that I want to return all items for a connection, so I don't need first, after, before, and last. But I can also see people not wanting before and last :)

Hello, we tried to solve the issue.
This is what we did:
Modify ConnectionExtension to allow customization of relay connection arguments. Add a new connection_arguments parameter to the connection decorator to specify which arguments should be included. Update the apply method to only add the specified arguments.
You can review changes in this commit: https://github.com/wwqwq2313/strawberry-graphql-strawberry-3733/commit/3a3d072fef529bc641b79249278ef242cdc9f46f.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


CFP: Expose URL Path in HTTP metrics

Cilium Feature Proposal
Thanks for taking time to make a feature request for Cilium! If you have usage questions, please try the slack channel and see the FAQ first.
Is your feature request related to a problem?
Hubble L7 metrics currently does not export the http URL path, even though its available in Hubble flows; Not having access to this information is a huge bummer and a large part of why we want L7 visibility in the first place.
The only way to access this information outside of hubble currently is to export the file using --hubble-export-file-path and then parse that information, which isn't ideal...
Describe the feature you'd like
Expose the URL path as labels for the hubble http metrics http_requests_total, http_responses_total and http_request_duration_seconds.
(Optional) Describe your proposed solution
I'm happy to give this a shot if acceptable :)

Take my contribution with caution, I'm new to Cilium

As far as I understand it L7 observability with Hubble monitor each HTTP request sent to your services, meaning that cardinality of such a thing might be infinite.
I'm letting you check out this comment: https://github.com/prometheus/client_golang/issues/491#issuecomment-435872413

Thanks for the CFP @zhilingc. @Joffref is absolutely right: the cardinality of the metrics could literally explose which means we unfortunately cannot expose URL path in HTTP metrics.

@rolinh @Joffref That's regrettable, since it nerfs the out of the box metrics significantly for a large swath of applications, but i see where the decision is coming from. Thanks for the replies :)

This feature is available in istio service mesh, it uses same envoy as cilium does, and it does not contain such metrics out of the box too. So one nice solution would be to let user decide turning these dimensions on.
Could we /reopen?

I understand such metric, in case of a low cardinality in url path, but I'm not sure about implementing this directly inside cilium. Is there any workaround we can think about to enable this dimension without modifying the code base ?

I think if we can consider supporting this, but it would need to support a few things to keep cardinality reasonable:

Needs to be opt in.
The URLs should be "allow listed" with a regex to match, and then remove any parameters from the URL and replace them with placeholders.

Example:
URLs might be:
/user/chance/details
/user/michi/details

And the config/regex would need to support transforming the URL into /user/USERNAME/details within the metrics labels.

/reopen

Can I work on this issue?

@SejalJain1710 I think having a more concrete proposal of how this would be configured would be good to have in the CFP before it's implemented. Feel free to draft something and we can review before you start implementation.


slot type Amazon.Movie doesnt work for en_GB

hi,
im wanting to set this up but i get the following error, which is why i think the beta builder didnt work
Error: There was a problem with your request: the slot type "AMAZON.Movie"are not currently available for en_GB
is there anyway around this?

@phairplay yea sorry, this is a known limitation right now https://github.com/josephschmitt/alexa-libby/wiki//Troubleshooting-Intent-Slots
The Movie intent slot is currently in beta and only works in the US right now. If you want to try to get this working without it, you need to follow the directions for building out a samples dictionary and use the AMAZON.Literal slot.

Dam it
That looks a little over my head
Thanks for the quick reply

So it seems like there’s some movement on this front from Amazon. They have a whole type of intent for video creative works now! https://developer.amazon.com/public/solutions/alexa/alexa-skills-kit/docs/built-in-intent-ref/videocreativework-intents
I'll play with this and see. If it works I think this new type is available in all regions.

Cool!
Really looking forward to testing it out :)

Hi did you have any look with new intent?