Fixed #28077 -- Added PostgreSQL opclasses to Index

Continued from https://github.com/django/django/pull/9332.

Please forgive my ignorance, but why the requirement:

Index.name is required when using opclasses.

At least in the case of GIN, is there a reason why the generated index name could not be used the same way when using the default opclasses? Not specifying opclasses in this case is actually opclasses=['jsonb_ops']
-- default opsclass:
CREATE INDEX "a_c_c_9cdf5c_gin" ON "a_c" USING gin ("c");
-- is same as:
CREATE INDEX "a_c_c_9cdf5c_gin" ON "a_c" USING gin ("c" jsonb_ops);
-- using a non-default opclass:
CREATE INDEX "a_c_c_9cdf5c_gin" ON "a_c" USING gin ("c" jsonb_path_ops);

https://www.postgresql.org/docs/10/static/datatype-json.html#JSON-INDEXING

This is because auto-generating an Index name is a legacy feature only supported for backwards compatibility.

ah, i was not aware of this at all. The official documentation is not even hinting at it:
https://docs.djangoproject.com/en/2.1/ref/models/indexes/#name
I did not know it will be going away, i have always found it working well and convenient.
Is it possible to read more about this somewhere?

@minusf I can't speak for the deprecation decision but it surely has a been a pain point and a source of bugs and frustration to maintain in the past.

The fact that automated index names are based of the table and column names forces Django to perform an action when the model or fields it's defined for are renamed to keep both in-sync. Since index renaming can be a costly operation on some backends (or at least it uses to be) Django skips it (or uses to).

The algorithm used to rtrim index names also happened to change during South to Django migration transition.

In this PR case considering opclasses when generating the name would have changed the expected names of indexes created for CharField(db_index=True) on PostgreSQL.

All of these issues caused indexes names to drift from what Django expects them to be over time and required adding complex introspection code to determine whether and index with a mismatched name actually exists. This is also problematic when you want to write statements that refers to such index names (e.g. MySQL USE INDEX/IGNORE INDEX, PostgreSQL ON CONFLICT) because you must introspect the database to determine what's the current name of the index instead of using something along Index.name because it might have drifted.

Thanks for the examples.  I have worked on some projects in the past where handpicked index names were let's say "less than ideal", so in that sense the django autogenerated ones were very sensible and more importantly consistent (maybe except the random string at the end :}). But these could work only in the simpler use cases. index_together always felt a bit like a kludge..
I definetely agree that having explicit index names would get rid of any ambiguity and heuristics on django's part.
The indexes meta looks like the correct approach, it's easy to see even at first glance all the indeces, and it can easily deal with the more exotic ones like GIN subtypes.
I guess I was just surprised that there isn't stronger language in the docs about this deprecation coming, as this is quite a big change from the status quo, and the sooner people start seeing it, the sooner they can start getting used to the idea (and actually read up on indexing :})

I don't know if the legacy behaviour is officially deprecated, but new functionality isn't required to support it.

Thanks for bringing up the question, @minusf! I don't think there exists any formal process of "we deprecate automatic index name generation". Mostly, because there's no (suitable) way for people to migrate to them when they started with very early versions of Django (speaking of the 0.9x age). Back then the index name even changed between runs.
My idea is to, once all supported databases have a RENAME INDEX operation, write a management command that inspects your existing indexes and renames them using a stable naming system. But until now, RENAME INDEX isn't supported across the board.

i heard somewhere that "explicit is better than implicit" :}
I am afraid that notes burried in the docs about what is supported in which circumstances (and especially differences on the same index Class) will generate more cognitive overhead for me, than listing all indeces all the time... I also realized that a couple of times I forgot to add composite indeces or more esotheric ones because django was "hiding" this aspect of the model and was "just working". That is probably not ideal for such a crucial part of the database.  One less argument also for all the "there's too much magic in django" crowd, win-win :}
postgresql has ALTER INDEX, so mission complete for my use case, and i see that mysql 5.7 has something similar too, but mariadb does not...  There is always DROP INDEX and ADD INDEX i guess :} Having seen full table rebuilds after ADD COLUMN, index migrations are probably a smaller issue...
At least index renamings are an operation on existing data so it can be always re-run (unless the database is huge, in which case all migrations will be very difficult)...


Network Error Logging Support in OkHttp

Hey OkHttp folks,
Was wondering if there was any plans to support Network Error Logging natively in OkHttp.  It's supported in cronet through experimental features but in talking to a handful of app teams, it would be great to be able to see reachability data via okhttp as well.
Thanks so much!

A few thoughts

This doesn't feel like something we would turn on by default for all clients.
Similar complexity additional features tend to be implemented as separate libraries like https://github.com/babylonhealth/certificate-transparency-android
Detecting the specific errors is really awkward across so many different platforms (Android versions, JDK versions, Providers).

On the last point it seems like a lot of work to correctly root cause every error type automatically, this is a look at just one class of errors
https://github.com/square/okhttp/issues/6411
And with Conscrypt - we've asked for ways to cleanly understand causes of handshake failures but it isn't something cleanly supported at the moment https://github.com/google/conscrypt/issues/847
If we could even do 3) reliably, I think we could wrap more exceptions into ones with something like a public API for errors, even without the reporting aspect.  Make it possible for clients to catch those and explain to end users why the connection is faulty.

Yeah I think you could probably do this with OkHttp, but I don’t think it’s a good fit within the library.

But one thing we could do, which would be pretty nice, is to report our errors using the taxonomy here:
https://www.w3.org/TR/network-error-logging/#predefined-network-error-types

Let's reopen and define not as an interceptor but a gradual move to errors containing enough structured data to sometimes report these.
Maybe work with Conscrypt folk to make this happen a layer lower also.

Put an exploration in the PR above.

Also wondering how it could relate to https://www.chromium.org/developers/design-documents/network-stack/netlog
Which are effective as a much more user friendly (cooperative) version of wireshark pcaps. chrome://net-export/

Also wondering how it could relate to https://www.chromium.org/developers/design-documents/network-stack/netlog
Which are effective as a much more user friendly (cooperative) version of wireshark pcaps. chrome://net-export/

Or if we wait long enough
https://tools.ietf.org/html/draft-marx-qlog-main-schema-02#section-4.3.3
https://tools.ietf.org/html/draft-marx-qlog-event-definitions-quic-h3-02

Or if we wait long enough
https://tools.ietf.org/html/draft-marx-qlog-main-schema-02#section-4.3.3
https://tools.ietf.org/html/draft-marx-qlog-event-definitions-quic-h3-02

Closing as low priority and unlikely to be picked up.
WireShark is working with JDK 11


Add Apache Incubator-shenyu Ui Dependencies Licenses

Task
A clear and concise description of Task
Task List:

[x] add apache shenyu ui dependencies licenses

Please read the Contribution Guideline before submitting the PR

complete!
Zhang Yonglun @.***> 于2021年7月9日周五 下午2:48写道：

Closed #1741 https://github.com/apache/incubator-shenyu/issues/1741.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-shenyu/issues/1741#event-4998634801,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALAFSKRMZFWKOBAUMTZEPKLTW2LSHANCNFSM5AB4BA7A
.


Support jsInteropMode "JS_RC"

Are you planning on supporting JS_RC?
I did notice that latest 2.8.0-SNAPSHOT was working properly w/ "JS" mode, however not in dev mode. Unsure if related to these changes or not.
Thanks; this is a great utility!

Yes we are planning support for it.
Duplicated #66


Inconsistency in the night-battle prediction

When I doing E-1 I noticed unlike when I was doing doing day battles where sowed the current HP with a bar under it, that it would instantly show my ships final hp without showing what their hp was before the night battle started. This seems to be inconsistent with how the visual theme of KC3Kai is supposed to be and is most probably a bug.

since this also applies even to single fleets, cannot merge this with #578 so will not mark duplicate, but is heavily related.
will check later on latest Battle Prediction if still not working

Before Fix
Before the fix, we can see that night battle starts with the predicted HP as its official HP
See Kaga, who was still green after day battle, but upon starting night, she instantly became red. Wherein that red HP was supposed to be the prediction for "after-night"

After Fix
On the fix, the prediction bars work again, and the original HPs stay as the "grayed" bars

My report was specifically about the default theme rather than natsuiro would you mind checking it?

supposedly it will work for both themes


HP Regen ticker

Same as the special attack one but then for HP would be usefull for NMZ

Added in #923


Jenkins CI: Updated postman collection to get tokens

@kailash I think the changes made on the master branch- PR https://github.com/datakaveri/iudx-gis-interface/pull/53
will be required in the 3.5.0 branch as well.

retest this please


bugfix: resolve crash with renderer for demo

I have a swap chain with 5 buffers in vulkan? could the definition be expanded with a minimum looks like you're querying for the supported number of swap chain and then requesting the max number of images in the swapchain?
using the MAX_FRAMES_IN_FLIGHT against the current frame should be safe. we're just iterating over 5 frame buffers but are bound to two active frames in flight.

I think the API vulkan provides allows us to specify a minimum (we pick minimum driver supports + 1 which ensures non-blocking behavior.) I don't think we get to decide anything but the minimum count. If the driver gives us 5 images we have to roll with it.
We may be able to have the sync primitive list be exactly MAX_FRAMES_IN_FLIGHT + 1 elements long regardless of number of images. That may be the more correct fix here.

I think the API vulkan provides allows us to specify a minimum (we pick minimum driver supports + 1 which ensures non-blocking behavior.) I don't think we get to decide anything but the minimum count. If the driver gives us 5 images we have to roll with it.
We may be able to have the sync primitive list be exactly MAX_FRAMES_IN_FLIGHT + 1 elements long regardless of number of images. That may be the more correct fix here.

this is under rafx api, is MAX_FRAMEs_IN_FLIGHT exposed there?
using the min amount of frames specified by surface capabilities. this is the minimum number of frames that the device supports, you can specify less then this number?

Past me (2+ years ago) was trying to avoid MAX_FRAMES_IN_FLIGHT being in rafx_api as a public symbol because it felt like an opinionated choice. But present me thinks it's probably better to just have it in rafx_api in one place like you're suggesting. It's proven too useful for various systems and trying to "hide" it is counterproductive. And pragmatically I think the only values that would ever make sense there are either 1 or 2 (double/triple-buffering.) So I'd be on board with moving it to rafx_api and removing the other occurrences if you wanted to do that. I probably will at some point if you don't, but I've had a busy few weeks and may not get to it right away.
By the way what was it exactly that was crashing? I'm surprised to hear that a driver is providing 5 images, but I wouldn't necessarily expect that to cause a crash?

Past me (2+ years ago) was trying to avoid MAX_FRAMES_IN_FLIGHT being in rafx_api as a public symbol because it felt like an opinionated choice. But present me thinks it's probably better to just have it in rafx_api in one place like you're suggesting. It's proven too useful for various systems and trying to "hide" it is counterproductive. And pragmatically I think the only values that would ever make sense there are either 1 or 2 (double/triple-buffering.) So I'd be on board with moving it to rafx_api and removing the other occurrences if you wanted to do that. I probably will at some point if you don't, but I've had a busy few weeks and may not get to it right away.
By the way what was it exactly that was crashing? I'm surprised to hear that a driver is providing 5 images, but I wouldn't necessarily expect that to cause a crash?

your render graph has 2 pipelines and I'm cycling through 5 images in my swapchain so it kind of implodes.

Do you have a line of code or stack trace where the crash occurs?

I may be encountering a similar issue where rotating_frame_index is 3 but static_resources.tonemap_debug_output contains only 3 values. This is when running demo with the vulkan backend.
thread 'main' panicked at rafx-plugins/src/pipelines/modern/graph_generator/mod.rs:265:46:
index out of bounds: the len is 3 but the index is 3

adding a modulus operator allows the demo to run correctly.

The mesa discussion here is informative: https://gitlab.freedesktop.org/mesa/mesa/-/issues/5350
In my case, minImageCount is 4 and maxImageCount is 0. With rafx adding + 1 to minImageCount, so 5:0.

Thanks for the helpful comments and link. I think #257 could be a good fix for this. I think just adding the %3 would be perfectly fine functionally. However I think the underlying issue was having a sync primitive per swapchain image rather than per frame in flight and I think that will take care of the crash reported here.
I haven't tested this on anything but Mac + MoltenVK. If you confirm that #257 fixes the crash you saw, that would be helpful. Or otherwise I'll at least confirm #257 doesn't break windows/vulkan. I don't have a linux machine handy to test myself (and if I did it's very unlikely to be the same configuration as you have.)

Closing in favor of #257


[Feature Request]: adding more social icons in hamburger

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description

want to add more social icons here
Use Case
adding icons
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@dhairyagothi can you describe

in mobile view side bar contains only github icon i think there should be some more social icons

@dhairyagothi, this issue might not be ideal for now. If we can reference our social profiles, like adding GitHub icons, it could be beneficial, but it's not the best time for this.

So, please close this issue and work on other

Okay


bfloat16 Training

Thank you for releasing code for these inspiring works!
I tried to use bfloat16 for model parameters, and manually converted images and labels from float32 to bfloat16 before feeding them for training, but noticed that training slowed down by about 3 times. Also, the performance becomes noticeably worse. I'm wondering if it is wrong to use bfloat16 in this way?
Thank you very much for your help.

You mean that you have 3x slower step time? Or is it 3x slower to target accuracy? The first would be unexpected, but I wouldn't know why that is the case without examining the training with a profiler.
In general, you could not expect to have the same performance when going from float32 to bfloat16. With ViTs we found that  the first Adam momentum can safely be kept in bfloat16 (example config), but the second moment and the model weights need to be kept in float32.


ENH implement sin, sqrt and tan on CSR and CSC matrices

I implemented a few more arithmetic operations on sparse matrices, so that
 np.sin(X)
 np.sqrt(X)

and
np.tan(X)

work when X is CSR or CSC.

@dlaxalde: new version pushed with --force. Sorry about that.

I like the idea, but I'm not in favor of this pull request as it stands, because it is incomplete.  If it is a good idea to implement sin, tan, and sqrt for sparse matrices, then an even better idea is to implement all the unary operators for which func(0) = 0, and all the binary operators for which func(*, 0) = func(0,*) = 0.  That is a more ambitious project, but it is the right way to do it.  (See http://docs.scipy.org/doc/numpy/reference/ufuncs.html#available-ufuncs for ndarray functions.)
There is a problem with this code:
def _apply_arith(self, op):
    X = self.copy()
    op(X.data, X.data)  # out keyword not supported by older Numpy
    return X

The dtype of the result is the same as the dtype of the matrix. This is not how the ndarray versions work. E.g.
In [20]: a
Out[20]: array([1, 2, 3, 4])

In [21]: sin(a)
Out[21]: array([ 0.84147098,  0.90929743,  0.14112001, -0.7568025 ])

The sparse version should have the same behavior.

It's early and I'm probably missing something trivial, but why only the ufuncs with func(0) = 0?

@rgommers: These are the functions that will preserve sparseness.  But perhaps this suggestion is too protective.  Should all the ufuncs be implemented?  If so, what should the return type of, say, cos(a), be when a is a sparse matrix?  The same as a, even though the matrix is no longer sparse?  Or should it return a dense matrix instance?

I think it is reasonable to expect that func(a.todense()) gives the same result as func(a).todense(). That would not be the case if, for example, cos were implemented and it only acted on the nonzero elements of a.
This https://gist.github.com/1657681 is a modified version of scipy/sparse/data.py that adds the numpy unary ufuncs for which func(0) = 0 to the _data_matrix class.  If the idea looks sound, I can make a real pull request to implement this.
Implementing the binary operators is a bigger task that I don't think I'll have time for (although most of the foundation for this might already exist in the _cs_matrix class).

You're right, forget about ufuncs with func(0) != 0.
Your gist looks OK (haven't tried it yet). Where is this ufunc behavior of looking at methods with the same name actually documented?

This https://gist.github.com/1657681 is a modified version of scipy/sparse/data.py that adds the numpy unary ufuncs for which func(0) = 0 to the _data_matrix class.  If the idea looks sound, I can make a real pull request to implement this.

It is definitely a good idea and your code works fine.

I just went through the ufuncs documentation in the refguide again, but can't find this feature. Let's make sure that this doesn't work only by accident before merging.

... Let's make sure that this doesn't work only by accident before merging.

Good idea.


Add CSV ColumnConfig Format for ColumnConfig.json

A useful feature to have to could to provide a JSON to CSV convertor which will enhance readability and wider dissemination of reporting (for example, the ColumnConfig.json) when performing feature-level distribution or sanity checks.
Use-case for above requirement is as follows — Often, we need to generate summary statistics (mean, median, standard deviation, missing rate etc.) on time-level data (could be day by day or week by week depending on what is the most granular level for the driverset). This helps in discarding data for specific days or weeks (or feature-level discarding) if the distribution deviates significant from baselines. For now, we run SHIFU’s stats step on snapshot data and combine all ColumnConfig.json’s resulting from each run (week or day) into a single CSV (see adhoc generation sample code in nsharma1@<>:~/statsgen).

"time","columnFlag","columnName","columnNum","iv","ks","max","mean","median","min","missingCount","missingPercentage","stdDev","totalCount","weightedIv","weightedKs","weightedWoe","woe","co
lumnType","finalSelect","version"
"06","None","AUTHORIZATION_ID","0","0.051183","6.634072","1530514432.0","1452439644.77","1452439644.77","1388684695.0","0","0.0","24039249.366","60665","0.0511831972732","6.63407168107","-1
.1911665815","-1.1911665815","N","False","0.2.0"
"06","None","trans_id_num","1","0.024123","2.902147","1.78549703475e+16","1.55562673689e+12","1.55562673689e+12","-38143237441.0","0","0.0","1.62066977578e+14","60665","0.0241228792421","2.
90214700112","-1.1911665815","-1.1911665815","N","False","0.2.0"
"06","None","trans_id_char","2","0.023935","2.902147","1.78549703475e+16","1.56555940953e+12","1.56555940953e+12","30246395327.0","0","0.0","1.62066881931e+14","60665","0.023935305689","2.9
0214700112","-1.1911665815","-1.1911665815","N","False","0.2.0"
"06","None","transaction_ts","3","0.0","0.0","0.0","0.0","0.0","1.79769313486e+308","60665","1.0","4e-06","60665","0.0","0.0","-1.1911665815","-1.1911665815","N","False","0.2.0"
"06","None","ts_year","4","0.000202","0.06694","2015.0","2014.002291","2014.002291","2013.0","0","0.0","0.050496","60665","0.000201596204613","0.0669400809998","-1.1911665815","-1.191166581
5","N","False","0.2.0"
"06","None","ts_month","5","0.035871","6.32181","12.0","6.39161","6.39161","1.0","0","0.0","1.45473","60665","0.0358713401673","6.32180975066","-1.1911665815","-1.1911665815","N","False","0
.2.0"

Done in 0.2.6


[SUGGEST] Default value of top_p is 0 in OpenAIServiceConfig, which may result BadRequestError

Is your feature request related to a problem? Please describe.
I locally deployed a OpenAI-Compatible server based on Qwen-1.5-Chat model by vllm, then use scripts/llm_api_test.py to test the api. The response gave an error:
openai.BadRequestError: Error code: 400 - {'error': {'message': "OpenAIException - Error code: 400 - {'object': 'error', 'message': 'top_p must be in (0, 1], got 0.0.', 'type': 'BadRequestError', 'param': None, 'code': 400}", 'type': None, 'param': None, 'code': 400}}

At first,I was unfamiliar with taskweaver, so I thought there was something wrong with the model calling. I tried to rebuild the link based on LiteLLM and vllm.
I met the same problem two hours later.
Finally, I found that the default value of top_p in OpenAIServiceConfig is 0.
self.top_p = self._get_float("top_p", 0)

Describe the solution you'd like
Please change the default value of top_p or further improve the document.
Describe alternatives you've considered
None
Additional context
None

And I am confused about how to set the top_p parameter without changing the default value.
I guess maybe I can set a parameter like llm.top_p in taskweaver_config.json？

@BarryRun ! Could you try to set llm.openai.top_p as a non-zero value?

@zhangxu0307 Yes it works, it would be better if you update the document  :)

close inactive issues


Production-Ready

Is legend-state production-ready? :)

Yes. We are using it in production for http in a Next.js app and a React Native app.
As of version 0.19 there are no known major bugs or plans for big changes. We're just doing some more stress testing and documentation on the way to a version 1.0 now.


Full documentation for edpm_network_config role endpoint

Each variable now has a type, default value and basic info.
Several variables were removed, as they are not used by the role.
Namely:

edpm_network_config_with_ansible

Typo in the edpm_os_net_config module was fixed.

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

/override rdoproject.org/github-check


Cover Image bleeding into playlist, when enhanced

As the title states, the cover image is bleeding into the Liked Songs and only the liked songs. It seems only to happen when Enhanced is active.
Also, not sure if this is supposed to be the functionality, the banner for liked songs changes to the current song image regardless if you are playing a song from liked songs or if the song is not liked and in that playlist.
Spotify: 1.2.0
Spictify: 2.15.0

Its working fine for me in liked songs page aswell as enhanced songs in a playlist

can u share ur config? ( run spicetify config and send the SS)

try using this latest version of spotify and after installing run spicetify backup apply. it should prob fix it

That didn't fix it. I uninstalled spicetify and Spotify, but it's still happening.
Initially, I downloaded the theme from the marketplace, but after seeing others with the same issue and your recommendation to get the one using PowerShell, I uninstalled the marketplace one and installed the other.
Is there a place I can check for residual files that might still be around from the marketplace version?

Is there a place I can check for residual files that might still be around from the marketplace version?

No
Can you check is there any errors from nord in spotify devtools?

Also what page is this liked songs? playlist?

yea it is liked

I dont have a enhance button in liked songs page, but i have that that button in playlist page.
can you check once again you have a enhance button in liked songs page?
because enahced songs in other pages causes this issue and i fixed it, i dont know liked songs page has a enhance button

It's in the experimental features

It's in the experimental features

Fixed It 🎉, Never knew this setting existed
If your issue is fixed feel free to close this issue 👇
Thanks for reporting and and being patient till I understood the problem ❤

Thx, it's fixed! No worries, I've been using the setting for a while. I forgot I needed to enable it.


Thefuck alias resolving seems to be broken as of the new version

I could have swore that in earlier versions my aliases where interpreted correctly and now in the new version they are not.  I mean, what's in your .zshrc regarding TheFuck?
What does alias output? I Imagine g is among these aliases.

Aliases: https://gist.github.com/deiga/b6f397e89ed2a7939a3e
And my .zprofile includes eval $(thefuck --alias)

I had the same problem with 3.4
but with 3.6 it works fine for me again

Yeah, something in newer versions fixed this.


correct the version per github issue 624

Docs Build status updates of commit 7359d9c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/bingads-13/bulk-service/downloadcampaignsbyaccountids.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/downloadcampaignsbycampaignids.md
:white_check_mark:Succeeded

Advertising/bingads-13/guides/services.md
:white_check_mark:Succeeded

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


feat: basepath should be case-insensitive

hello, @molefrog
The wouter matcher is case-insensitive by default, so the basepath should be case-insensitive default too ?
keep pathname and basepath with the same default behavior, so we can reducing user confusion.
Of course, we can also provide a sensitive parameter, but it should be applied to matcher and basepath at the same time, otherwise, we will ignore case by default.
Do you have any suggestions？

@molefrog done. please review again.

Codecov Report

Merging #138 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #138   +/-   ##
=======================================
  Coverage   99.50%   99.50%
=======================================
  Files           7        7
  Lines         200      200
  Branches       35       35
=======================================
  Hits          199      199
  Misses          1        1

Impacted Files
Coverage Δ

use-location.js
100.00% <100.00%> (ø)

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 89704a9...91ffb79. Read the comment docs.


Allow Set#intersect? and #disjoint? to accept array argument

Implements [Feature #17838]

Why not go further and allow any Enumerable?
In that case, we simply loop on it and check against the set. For many cases, that would be more efficient than building an intermediary set as proposed here.

Considering I screwed up the implementation and create an intermediate set without doing anything with it, that seems fine with me.

@knu Thank you for your review.  I've pushed a commit that updates the documentation.


Sending a Rang to Tag

Hello.
I want to improve your library for my goal. I'm searching about a way to implement sending Rang to Tag side.
in this method A tag side can print its Rang from each anchor that it blinks with.
right now we don't have it.
We read dwm1000 manuall.If i understand correclty there are some optional ways to do that.
I understood the tag and anchor connect in this way :
Tag : blink
Anchor : Ranging_INIT
Tag : Poll
Anchor: Poll-Ack
Tag: Sending Final Message to anchor
Anchor: have Rang
at this time Anchor calculates and have a Correct Rang then print it but Tag does not have. it does not print anything right now.
in the best solution We should send a Rang String to Tag after "Final Message" from anchor.
I found some lib codes in each side, tag and anchor.step by step what happend.
I could send a packet frame with "0x60" value from anchor side,
but I could not find a way to wait to recieve this packet frame "0x60" in Tag side.
I don't know what "command - code - function"  i should use to wait in Tag side to receive "0x60" packet frame in correct time.
I mean the waiting time for receiving packet in Tag side.(halt time)

Hi NimaaZx,
according to the standard ISO/IEC 24730-62_2013 on which we are based, the final step of the TWR  flow (with embedded TX time) is an Activity control (0x10 , 1 octet) + Activity Code ( 0x00 or 0x01 , 1 octet) + Optional parameters ( new blink rate or Next anchor to perform ranging ,2 octet ), that the anchor sends back to the tag.
According to the standard, the final step isn't usable to report range to the tag.
You Should create a new personal message (and functions) with User Code (from 0x60 to 0x77 and from 0xE0 to 0xF7) that returns the range to the tag. Then the Tag should implement a function to receive the range.
In this scenario, you can't use high-level abstraction functions inside DW1000NgRTLS but you must use the individual functions for every single step of the TWR flow ( transmitTwrShortBlink(), transmitRangingInitiation(), transmitPoll(), and so on ... ) + new personal function.
We haven't implemented this functionality because it isn't a default step in the standard 24730-62

thank you for your response.
We implemented it inside "TransmitRangingConfirm"
there is a packet that we put the range in and when this function Call, it is sent and in tag side we receive it.
I have 3 more questions .
1- For developing and working with dwm1000 module,
what do we do if someone like you don't share his libs and source code with us? you write all libs ? how did you do that? how much time does it take ?
I mean if your source code does not exist what should we do for working with this module?
because I never found the library from decawave and we want to build a industrial product with dwm1000 and esp32.
we need them.
2- we use your souce code and library with some our modifications.
do you set sleep mode for anchors?
because after 10 minutes when a tag is off or out of coverage , when the tag is back , anchors can't see them until we reset anchors manually.
3- do you have any TX/RX mode for anchors or tags coverage or something like said in
TX/RX table in datasheet ?
I see something like AutoTx in your code.
thanks a lot.

Before all, you must read DW1000 user manual and DWM1000 datasheet. It's very important to understand how to develop a driver and libraries for the DWM1000. We spent approximately 1 month only to study these two documents, plus 3 months (more or less) to refactor the Thotro code and add some futures. Our code is compatible with esp8266 but with esp32 we had problems.
The best work that you can do is: take the source code of the driver from Decawave and convert it for the esp32 and esp-idf (driver is written in C).

No. Sleep mode isn't enabled in anchors. Maybe that is a bug in our code or in your modification of the TransmitRangingConfirm function. Remember that the last 2 octets (payload is: 0x10+0x01+NextAnchor) are for specifying the next-anchor to perform ranging.

before reply to this question, I prefer consulting with my friend @an4cr0n

Thank you for your concern.
1- Actually at first We did not find a source code of dwm1000 driver in official site. (can you send me a link please ?)
as well as i think reading module datasheet is a little hard for us. because I think
Electrical Engineering or other knowledges is needed. but we did it.
we studied software engineering. some topics are related to our majors. even writing library with low level language is a challenge.
I mean this indoor positioning is a part of of IoT.
unfortunately IoT is a branch that requires several fields of study(software-hardware-electronic eng)
2- at this time I dont have an access to our code but as I tell you we injected ranging parameter to "TransmitRangingConfirm" function. yes you right, we injected it after "next anchor bits". I mean after last bit you wrote, we injected ranging parameter and sent to the packet to tag side.
I will send the code for you.
for sleep problem we used reset command.
it solves the problem for now.
3- About Tx,Rx / for Extending Rang of dwm1000.
I found this issue of Thotro topic
https://github.com/thotro/arduino-dw1000/issues/245
Do you use this source for extending rang of dwm1000 ?
somebody said using this
writeValueToBytes(txpower, 0x1F1F1F1FL, LEN_TX_POWER);

Drivers are inside DW1000 Application Programming Interface software package.
Yes, IoT is a branch that requires several fields of study.

OK

The range of the dwm1000 is very short because it has a very small antenna. You can try to extend it changing frequency(channel, lower is better), use manual power instead of SmartPower (see the struct of the configuration options). This argument is explained in DW1000's user manual or in an Application note.
Very important is the vertical antenna polarization for the DWM1000 module.


Command for "dispatch add -t qna" must be changed to get sample 14.nlp-with-dispatch to run properly

Per email conversation, currently the command to add the child qna service to dispatch that's shown in tutorial is this "dispatch add -t qna -i "<knowledge-base-id>" -n "<knowledge-base-name>" -k "<your-cognitive-services-subscription-id>"
Change it to add the argument --intentName q_sample-qna, so that the intent name in the bot matches the intent name that automatically gets generated from the dispatch add/dispatch create commands
new command:
"dispatch add -t qna -i "<knowledge-base-id>" -n "<knowledge-base-name>" -k "<your-cognitive-services-subscription-id> --intentName q_sample-qna"

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

ID: f68e3448-ea66-7aab-26d2-37de5f44527c
Version Independent ID: 30c9096c-926c-3b92-6ab0-cc49edf63842
Content: Use multiple LUIS and QnA models - Bot Service
Content Source: articles/v4sdk/bot-builder-tutorial-dispatch.md
Service: bot-service
Sub-service: sdk
GitHub Login: @DeniseMak
Microsoft Alias: v-demak

@DanDev33 - please look into this.

Updated the article in my branch.


Jseldess/manual deployment

@mberhault, finished drafting up a page on manual deployment. It covers prerequisites, recommendations, and steps for starting up an insecure and secure multi-node cluster.

@mberhault, please take another look when you have time.

nice! LGTM

@mberhault, one more commit: Added steps for connecting with client drivers and monitoring with the admin ui. Still look ok?

Yup. LGTM


Revert "[TS migration] Migrate 'SettingsProfileDetails' page to TypeScript"

Reverts Expensify/App#35307 which caused this failure: https://github.com/Expensify/App/issues/37215

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.4.44-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.4.44-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


<EMAIL_ADDRESS>decompress error

Installing 'github' (3.1.0) [64bit] from extras bucket
Loading dl.7z_ from cache.
Checking hash of dl.7z_ ... ok.
Running pre_install script...
ERROR Exit code was 2!
Failed to extract files from C:\Users<user>\scoop\apps\github\3.1.0\extract\GitHubDesktop-3.1.0-full.nupkg.
Log file:
~\scoop\apps\github\3.1.0\extract\7zip.log

@AntonOks are you still having this problem, or this problem solved?


Unsupported message in the Telegram info channel

Hi, I received this message as "unsupported" in our Telegram client based on tdlib:

What message type is that, it looks like all the others in that channel?
BR Florian

Make sure that you use the lastest TDLib version.

@AYMENJD I agree, we are not on latest, but we can only upgrade step by step, and if I know the message type I can put this on the roadmap for one of the next updates.

See the new added features to telegram and try to implement it on your client. There is newly added entities which can cause unspported message on older TDLib versions. Such as custom emojis, spoilers etc.

The message contains custom emoji.


search input in submenu

I have input in submenu, Is it possible hide element when clicked anywhere on the page, but I click inside not close? http://postimg.org/image/51qceiufz/

I can confirm this bug. I have a search input inside a submenu. Clicking in the input closes the submenu.
Here's an isolated test case on CodePen to reproduce the issue.

Thanks for your comments, and especially for the working CodePen which made debugging it super-easy.
You'll need to prevent the click event from the input from propagating to up to its parent elements, which causes the Superclick close event to fire. Here is the code I added to make it work:
$(document).ready(function(){
	// initialise Superclick
	$('.sf-menu').superclick();

	// prevent click events of search field from propagating to Superclick submenu
	$('input:text').on('click', function(e){
		e.stopPropagation();
	});
});

I think this should be done on a case by case basis, not added to Superclick core because I can think of cases where people may want the input click to propagate up to be handled by a form inside a megamenu submenu, for example.
I have forked @mrwweb's CodePen to show this working.

Rock on! This works as advertised. (Note to others, you'll need to use $('input[type="{whatever}"]') if using HTML5 text-like fields).
I'm trying to think if there are any accessibility or other downsides to putting stopPropagation on the input. Any idea? Is there a more specific way to prevent just the hide event?
I think I agree with you that this specific example wouldn't be worth adding as a feature. However, I could imagine some type of option that blanket disables menus closing for any click inside a submenu.

Here is another fork where I have wrapped part of Superclick's click handler in a conditional so it only runs if the  original click event was triggered by a Superclick anchor element. This could be a way of handling it within Superclick, and may not even need an option. Can you think of any negative side effects of this solution?

I changed this:
clickHandler = function (e) {
				var $this = $(this),
					$popUp = $this.siblings(e.data.popUpSelector),
					func;
				if (e.originalEvent.target === e.currentTarget) {
					if ($popUp.length) {
						func = ($popUp.is(':hidden')) ? over : out;
						$.proxy(func, $this.parent('li'))();
						return false;
					}
				}
			}

To this:
clickHandler = function (e) {
				var $this = $(this),
					$popUp = $this.siblings(e.data.popUpSelector),
					func;
				if ($popUp.length) {
					func = ($popUp.is(':hidden')) ? over : out;
					$.proxy(func, $this.parent('li'))();
					return false;
				}
			}

While this works, I just noticed that it actually throws an error. I'll look into it.

Can you think of any negative side effects of this solution?

Sadly I think yes. In existing versions of Superclick and your first CodePen fork, the menu closes on any click outside the submenu. I consider that a feature and even filed #11 on an instance where that doesn't happen. This breaks that behavior which certainly some people wouldn't like. (I'm not sure what I think.)

Good call!

Let's stick with leaving it out of core for now as I need to move on to other stuff. This thread can guide others if they encounter the issue. Thanks again.


Implement Sonarcloud scanning

Implements Sonarcloud scanning with gosec security scan and unit test coverage (E2E test coverage will be figured out at a later time).
ref: https://issues.redhat.com/browse/ACM-8788

/cherry-pick release-3.14


withState() updater does nothing above withPropsWithChange

Hi, I have spent a good part of today going completely insane figuring out why my withState updaters weren't doing anything. I haven't peeked at the source code that might be causing this but this certainly seems like a bug.
Plunkr here: http://plnkr.co/edit/P6Vhq98ShiUJ1L6N0WrE?p=preview
And I made a github repo you can clone and play around with more easily here:
https://github.com/realalexhomer/recompose-with-state-bug
The gist of this issue is that when you have a withState function composed above a withPropsOnChange function, and the withPropsOnChange function maps props, the withState handler will not update state. It will get mapped to props, but invoking it does nothing. I'm guessing the this it is referencing is incorrect.
For the example I invoked it on a handler created with withHandlers, but this can just as easily be invoked in any other callback ( I was originally doing it in apollo callbacks) or in a withStateHandler, etc.
I think the plunkr / repo speak for themselves, please let me know if theres anything else I can do to help get this fixed.
Thanks! 😁

You disable all updates in withPropsOnChange, and override all props there, so no updates are visible since then.
As withPropsOnChange always return cached version.
Its expected behaviour and not a bug. Just return props you need, so cached props will be merged with props.

I understand that from your perspective this would be the expected behavior, but I would be surprised if many of the longterm users of this library would be able to foresee this bug given the code sample I provided. Once you see it in isolation is makes sense (sort of), but based on the documentation this is not something in my opinion a reasonable user of the library would expect.
Would it be possible for us to either
a) document the behavior as a known gotcha of the pattern in withHandlers or withPropsOnChange
or
b) make it impossible for a handler not to lose its reference to the parent component, or at the very least fire a warning when this happens? Or fire a warning if a handler is called using an outdated setState?
I'd be very happy to submit a PR based on your feedback, just please let me know what you think is the solution and if you'd be willing to take another look this 😄
Thanks!!

withPropsOnChange exists 1 or 2 years, and you are the first with this issue ;-)
imo if updates are disabled and you override all the props - what other result it can be ;-)
Im not sure about what warning are you talking, as code above can be used to freeze all the props in some code, so its not bug or any issue.
btw you can improve the docs and if it will be better we will accept PR


[Android] com.BV.LinearGradient.LinearGradientView cannot be cast to android.view.ViewGroup

The following code executes perfectly on iOS, however an error is thrown for Android:
render() {
  return (
      <LinearGradient
        colors={[ ... ]}
        locations={[ ... ]}
      >
        <ScrollView>
          <LinearGradient
            colors={[ ... ]}
            locations={[ ... ]}
          >

            <Component1 />
            <Component2 />
            <Component3 />

           </LinearGradient>
         </ScrollView>
       </LinearGradient>
  );
}

Thoughts?

I originally thought this error was due to the way I'm implementing the component in a nested fashion, however I am receiving the same error when attempting to render an example very similar to the one provided in the readme. Again, only an issue on Android:
    return (
      <LinearGradient colors={['#4c669f', '#3b5998', '#192f6a']}>
        <Text>
          Sign in with Facebook
        </Text>
      </LinearGradient>
    );

I had imported the LinearGradient incorrectly! 😭


FPS++ 60 fps issue

When I enable the FPS++ 60 fps mod the game runs too fast when I move through the tabs in menus. Instead of that I prefer using 60 fps v4 + cutscene fix + DynamicFPS 1.42 (which don't have this bug). The problem is the combo mod that includes these 3 mods is not compatible with the last update 1.1.1 (or at least the name says that). Can you make this combo mod (60 fps v4 + cutscene fix + DynamicFPS 1.42) compatible with the update 1.1.1?
Thanks, I really appreciate the job you're doing.

i'll update it tomorrow, you can't use them separately ?

When I enable the FPS++ 60 fps mod the game runs too fast when I move through the tabs in menus. Instead of that I prefer using 60 fps v4 + cutscene fix + DynamicFPS 1.42 (which don't have this bug). The problem is the combo mod that includes these 3 mods is not compatible with the last update 1.1.1 (or at least the name says that). Can you make this combo mod (60 fps v4 + cutscene fix + DynamicFPS 1.42) compatible with the update 1.1.1?
Thanks, I really appreciate the job you're doing.

i'll update it tomorrow if i think about, you can't use them separately ?

Yeah, that’s what I’m using right now.

In addition to what I've said before 60 fps v4 + cutscene fix + DynamicFPS 1.42 make the game run at 60 fps in cutscenes so is not a perfect solution neither. I think I'll be using FPS++ 30 fps as long as these bugs are not fixed.

In addition to what I've said before 60 fps v4 + cutscene fix + DynamicFPS 1.42 make the game run at 60 fps in cutscenes so is not a perfect solution neither. I think I'll be using FPS++ 30 fps as long as these bugs are not fixed.

Use FPS++ 30fps and press ctrl + U to disable the framerate cap
Enable it back when you get a cinematic, it's the old method and you sure to don't have unespected issues

In addition to what I've said before 60 fps v4 + cutscene fix + DynamicFPS 1.42 make the game run at 60 fps in cutscenes so is not a perfect solution neither. I think I'll be using FPS++ 30 fps as long as these bugs are not fixed.

Use FPS++ 30fps and press ctrl + U to disable the framerate cap Enable it back when you get a cinematic, it's the old method and you sure to don't have unespected issues

If I do that (and I press 0Ctrl+U to unlock the frames) I get the same effect than using FPS++ 60fps: I'm moving through the tabs in the menu too fast, I mean I press L1 once but I move through the tabs 2 or 3 times. I remember this issue in BOTW as well when you played it at 60fps in Cemu (at least at the beginning, maybe it's fixed now).
Currently, the only way to avoid this is weird behavior is playing the game locked at 30 fps.

I can confirm that this 3 mods: 60fps Static + Cutscene fix + DynamycFPS 1.42 included in the last TOTK mod collection (1.6.5) work fine now: cutscenes play at 30 fps automatically and the speed moving through the tabs in menus is finally solved now.


do not merge: Search for applicant by prison number

Context
We currently have the functionality to search (and add) a person by CRN. We received feedback that not all POMs would know how to get this ID, and the "prison number" (aka NOMIS ID, aka nomsNumber on the API) would be more appropriate.
For the simplicity of the MVP we have decided to remove searching by CRN and only enable searching by prison number.
🚨 This PR requires this API PR to merge first https://github.com/ministryofjustice/hmpps-approved-premises-api/pull/1152
The end-to-end test is also ready to go: https://github.com/ministryofjustice/hmpps-community-accommodation-tier-2-e2e/pull/39
Changes in this PR
I'm thinking that as there are several ways of referring to this ID, where the frontend displays it or knows about it we can call it prison number and where the API is responsible it will know it as nomsNumber.
findByCrn in the person service becomes findByPrisonNumber  and we update the client and controller to handle it.
NOTE:

I need to confirm the exact design/copy on the template with UCD
Currently when a person is found from the client we automatically go and create the application. There is an upcoming story to show some the information about the applicant before proceeding to create and save the application.

Screenshots of UI changes
Before

After

Release checklist
As part of our continuous deployment strategy we must ensure that this work is
ready to be released at any point. Before merging to main we must first
confirm:
Pre merge checklist

[ ] Are any changes required to the e2e tests?
[ ] If you've added a new route, have you added a new
auditEvent? (see server/routes/apply.ts for examples)
[ ] Are there environment variables or other infrastructure configuration which needs to be included in this release?
[ ] Are there any data migrations required. Automatic or manual?
[ ] Does this rely on changes being deployed to the CAS API?

Post merge checklist
Once we've merged it will be auto-deployed to the dev environment.
Find the build-and-deploy job in CircleCI.
Should a release fail at any step, you as the author should now lead the work to
fix it as soon as possible.

It looks like there are still a few references to CRNs in parts of the code that there shouldn't be, e.g. the confirm eligibility integration tests are looking for a 'crn' field in the prison number form.

Thanks, thought I'd got rid of them, should be good now.

Also maybe not for this PR but do we still want to surface CRNs in the applications dashboard, or move to prison number?

Good thinking. It looks like the prototype still has CRN on dashboard, I think if it needs changing it can be picked up in another ticket


Session Ends After Initializing Cluster Networking

After attempting to run the second command to initialize cluster networking;
kubectl apply -n kube-system -f \
    "https://cloud.weave.works/k8s/net?k8s-version=$(kubectl version | base64 |tr -d '\n')"

The cluster reaches the Ready state, but soon after disconnects and ends the session;
"Your session has expired and all of your instances have been deleted."

fixed


Assertion with 100-continue

Discussed the #1777 issue with @oknet. We could reuse HttpSM::state_watch_for_client_abort as 100-continue handler.

[approve ci]

/cc @ffcai

It looks like we should land this!

@oknet  is this a 7.1.0 candidate? If so, please change the Milestone to 7.1.0 and add a Backport label.

This doesn't cherry-pick cleanly to 7.1.x branch. it's a small change, but to be by the book, can you please make another PR against the 7.1.x branch, and I'll merge that. Thanks!


web: Add a log select box

Hello @yuindustries, @landism,
Please review the following commits I made in branch nicks/ch1835/links:
d09ddf599c5f7a1aff1182ae7fedacba97b90e46 (2019-03-15 12:36:32 -0400)
web: Add a log select box

most of the pr is yarn.lock updates, don't worry :grimacing:


Sprockets::FileNotFound at / couldn't find file 'my_test_ts_object'   (in /Users/bob/git/myapp/app/assets/javascripts/application.js:89)

Hello,
I'm unable to include a Typescript (1.6) file such as my_test_ts_object.js.ts, when I try to load my app I get:
Sprockets::FileNotFound at / couldn't find file 'my_test_ts_object'   (in /Users/bob/git/myapp/app/assets/javascripts/application.js:89)

What is the preferred way to include Typescript files in the Rails javascript manifest?
Thank you,

Hi @atteeela,
Please look how it was done here: https://github.com/typescript-ruby/typescript-rails-example.


When I run propose for relations, I got stack overflow exception

Run task for Books-v1.csv, go to relations
Try to create a new relation between book and series
Propose it as below:

I suspect that maybe some part of the re-factoring went a bit wrong. Looking into that as well. If that is not the case, I will let @JanVana investigate the underlying querying code.

In the end it was much simpler: infinite recursion caused by omitted argument, leading to a REST API resource method calling itself.
Fixed as of https://github.com/odalic/sti/commit/75e184c9f7fff46c771ac80fc6bc7506b5e190bf .
Relates to #403 .


Test for functional/constructor attributes

Can one of the admins verify this patch?

Jenkins: test this please

Jenkins: Test this please.

Jenkins: add to whitelist

Can virgil or traian review this please?

Who is @u--?  Anyway, please refer to people by their ID, so they get notified: @traiansf @virgil-serbanuta


Compiling oneMKL (CUDA backends) on Windows parsing commands incorrectly

Summary
While the build README says that the CUDA backend is only supported on Linux, I wanted to naively try building it for Windows anyways - the oneAPI sycl compiler on llvm/clang built on Windows while only being tested on Linux, so I expected this to be pretty trivial as well. However, it looks like on the CMake build step, it is parsing some cl compile options as file/directories incorrectly.
Version
Latest develop commit.
Environment
oneMKL works with multiple HW and backend libraries and also depends on the
compiler and build environment. Include
the following information to help reproduce the issue:

HW you use: AMD 5900x, NVIDIA RTX 3090
Backend library version: CUDA Toolkit 11.5
OS name and version: Windows 11 Pro
Compiler version: oneAPI's DPC++ 2021-09 (latest on https://github.com/intel/llvm)
CMake output log: in Observed behavior section.

Steps to reproduce
Following the build guide on the README, but on Windows.
Observed behavior
CMake was able to configure the project (without the functional tests):
C:\Users\valen\Documents\GitHub\oneMKL\build>cmake .. -DCMAKE_C_COMPILER="C:\Users\valen\Documents\GitHub\sycl_workspace\llvm\build\bin\clang.exe" -DENABLE_CUBLAS_BACKEND=True -DENABLE_CURAND_BACKEND=True -DENABLE_MKLCPU_BACKEND=False -DENABLE_MKLGPU_BACKEND=False -DBUILD_FUNCTIONAL_TESTS=False
-- Building for: Visual Studio 17 2022
-- CMAKE_BUILD_TYPE: None, set to Release by default
-- The CXX compiler identification is MSVC 19.30.30706.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.30.30705/bin/Hostx64/x64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- TARGET_DOMAINS: blas;rng
-- Configuring done
-- Generating done
-- Build files have been written to: C:/Users/valen/Documents/GitHub/oneMKL/build

Upon trying to build it however, I get the following output:
[1/3] Building CXX object bin/blas/CMakeFiles/onemkl_blas.dir/blas_loader.cpp.obj
FAILED: bin/blas/CMakeFiles/onemkl_blas.dir/blas_loader.cpp.obj
C:\Users\valen\Documents\GitHub\sycl_workspace\llvm\build\bin\clang++.exe -fsycl /nologo  -IC:/Users/valen/Documents/GitHub/oneMKL/include -IC:/Users/valen/Documents/GitHub/oneMKL/src -IC:/Users/valen/Documents/GitHub/oneMKL/src/include -IC:/Users/valen/Documents/GitHub/oneMKL/build/bin /EHsc -Wno-unused-function -w -DSYCL2020_DISABLE_DEPRECATION_WARNINGS -O3 -DNDEBUG -D_DLL -D_MT -Xclang --dependent-lib=msvcrt -Donemkl_EXPORTS -MD -MT bin/blas/CMakeFiles/onemkl_blas.dir/blas_loader.cpp.obj -MF bin\blas\CMakeFiles\onemkl_blas.dir\blas_loader.cpp.obj.d /Fobin/blas/CMakeFiles/onemkl_blas.dir/blas_loader.cpp.obj -c C:/Users/valen/Documents/GitHub/oneMKL/src/blas/blas_loader.cpp
clang++: error: no such file or directory: '/nologo'
clang++: error: no such file or directory: '/EHsc'
clang++: error: no such file or directory: '/Fobin/blas/CMakeFiles/onemkl_blas.dir/blas_loader.cpp.obj'
[2/3] Building CXX object bin/rng/CMakeFiles/onemkl_rng.dir/rng_loader.cpp.obj
FAILED: bin/rng/CMakeFiles/onemkl_rng.dir/rng_loader.cpp.obj
C:\Users\valen\Documents\GitHub\sycl_workspace\llvm\build\bin\clang++.exe -fsycl /nologo  -IC:/Users/valen/Documents/GitHub/oneMKL/include -IC:/Users/valen/Documents/GitHub/oneMKL/src -IC:/Users/valen/Documents/GitHub/oneMKL/src/include -IC:/Users/valen/Documents/GitHub/oneMKL/build/bin /EHsc -Wno-unused-function -w -DSYCL2020_DISABLE_DEPRECATION_WARNINGS -O3 -DNDEBUG -D_DLL -D_MT -Xclang --dependent-lib=msvcrt -Donemkl_EXPORTS -MD -MT bin/rng/CMakeFiles/onemkl_rng.dir/rng_loader.cpp.obj -MF bin\rng\CMakeFiles\onemkl_rng.dir\rng_loader.cpp.obj.d /Fobin/rng/CMakeFiles/onemkl_rng.dir/rng_loader.cpp.obj -c C:/Users/valen/Documents/GitHub/oneMKL/src/rng/rng_loader.cpp
clang++: error: no such file or directory: '/nologo'
clang++: error: no such file or directory: '/EHsc'
clang++: error: no such file or directory: '/Fobin/rng/CMakeFiles/onemkl_rng.dir/rng_loader.cpp.obj'
ninja: build stopped: subcommand failed.

Expected behavior
For the backend to build cleanly, without any errors.

@valenotary Thanks for your experiments and output logs. As you mentioned (and written in README), CUDA backend is not currently supported on Windows.  Could you please open a feature request using this template?
If you would like to contribute, we also encourage you to submit a PR.

Closing issue, since this is a known limitation.


Vue Query: generates code that doesn't compile for vue-query

What are the steps to reproduce this issue?

Having this swagger "requestBody": {        "content": {          "multipart/form-data": {            "schema": {              "type": "object",              "properties": {                "formFile": {                  "type": "string",                  "format": "binary",                  "nullable": true                }              }            }          }        }      },
Execute orval with vue-query

What happens?
Orval generates code, that doesn't compile:
export const protocolImport = (
    protocolImportBody: MaybeRef<ProtocolImportBody>, options?: AxiosRequestConfig
 ): Promise<AxiosResponse<ImportProtocolsResponse>> => {const formData = new FormData();
if(protocolImportBody.formFile !== undefined && protocolImportBody.formFile !== null) {
 formData.append('formFile', protocolImportBody.formFile)
 }

    protocolImportBody = unref(protocolImportBody);
    return axios.post(
      `/protocol/import`,
      formData,options
    );
  }

What were you expecting to happen?
Code compiles.
Any logs, error output, etc?
error TS2339: Property 'formFile' does not exist on type 'MaybeRef'.
Property 'formFile' does not exist on type 'Ref'.
846 if(protocolImportBody.formFile !== undefined && protocolImportBody.formFile !== null) {
~~~~~~~~
error TS2339: Property 'formFile' does not exist on type 'MaybeRef'.
Property 'formFile' does not exist on type 'Ref'.
846 if(protocolImportBody.formFile !== undefined && protocolImportBody.formFile !== null) {
~~~~~~~~
error TS2339: Property 'formFile' does not exist on type 'MaybeRef'.
Property 'formFile' does not exist on type 'Ref'.
847  formData.append('formFile', protocolImportBody.formFile)
Any other comments?
I guess the issue is that body is unwrapped after appending file to it. Do you have some workaround so that I can have a code that compiles?
What versions are you using?
Operating System: OSX Sonoma 14.5
Package Version: 6.30.2
Browser Version: Version 126.0.6478.57 (Official Build) (arm64)

@pbrzosko , have you found a workaround?
I find myself in a similar situation and found out it is because the variable is not wrapped with unref

Yeah, for now I have overwritten the formBody, If I remember correctly, to be empty just for this one endpoint I have with an error. I don't use this endpoint for now so I could do it like that.

Ok, I found a workaround.
If you use a custom instance of Axios, the generated client uses unref correctly.
You can copy this file from samples/vue-query: https://github.com/anymaniax/orval/blob/833faeb9cc656a2ec87816479a4ebf9321edfc89/samples/vue-query/src/api/mutator/custom-instance.ts
And if you re-run the generation, for me, it fixed those variables accessed before unref is called.
Hope it helps!

Another example:
const queryFn: QueryFunction<Awaited<ReturnType<typeof sampleEntityGetMetadata>>> = ({ signal }) => sampleEntityGetMetadata(unref(id,params), { signal, ...fetchOptions });

There are two errors here:

unref with two parameters where it expects one
and then that status is required in params

Seems like it should simply be:
const queryFn: QueryFunction<Awaited<ReturnType<typeof sampleEntityGetMetadata>>> = ({ signal }) => sampleEntityGetMetadata(unref(id), params, { signal, ...fetchOptions });

When you have a GET method with both a path and query params it causes the unref(id,params) instead of unref(id), unref(params),... issue.
Seems to work fine if it's all query params, or just a path param, but mixing the two produces that error.

@MadManMcIvor this is probably a regression, I just checked v6.22.1 that we're using and it produces correct code for such case where you have both path and query parameters for get request:
export const getApiV1LibraryAuTariffconcessionsInstrumentNo = (
  instrumentNo: MaybeRef<string | undefined | null>,
  params?: MaybeRef<GetApiV1LibraryAuTariffconcessionsInstrumentNoParams>,
  options?: AxiosRequestConfig,
): Promise<AxiosResponse<TariffConcession[]>> => {
  instrumentNo = unref(instrumentNo);
  params = unref(params);
  return axios.default.get(`/api/v1/library/au/tariffconcessions/${encodeURIComponent(String(instrumentNo))}`, {
    ...options,
    params: { ...unref(params), ...options?.params },
  });
};

"/api/v1/library/au/tariffconcessions/{instrumentNo}": {
      "get": {
        "tags": [
          "V1AULibrary"
        ],
        "parameters": [
          {
            "name": "instrumentNo",
            "in": "path",
            "required": true,
            "schema": {
              "type": "string"
            }
          },
          {
            "name": "year",
            "in": "query",
            "schema": {
              "type": "string",
              "default": "2017"
            }
          }
        ],

Probably best to create a new issue for this tho, cheers!

Will do. Thanks, @Maxim-Mazurok!


fixed deprecation for Atom v1.13.0 in Issue #9

i am trying fixing #9
no longer produces an error.

merge please


Vectorize more decals

Attached..

PAX icon already exists in their official logo. We already have the planet vectorized already and we're not going to cover player group icons as the groups themselves have the vector already
Only one of interest is the Pirate one but that is already mentioned in #16 so I'm closing this one :P


feat: support format (png, svg)

Add support for multiple formats such as PNG or svg

Currently not possible in CF workers? (@resvg/wasm)


Syntax error on valid python code

a=[]
a+=0,*[]

Expected expression (near *)

Thanks for the bug report. It appears that the grammar rules for Python changed between 3.8 and 3.9. The above code generates a syntax error with 3.8 but is allowed in 3.9.
Could someone from the pylance team please transfer this issue to pyright?


override default Flask error handlers - not CORS compliant

Need to override the default Flask error handlers when they fire as they are not CORS compliant and portal won't consume them

[x] identify class of errors to override (e.g. from connexion.exceptions import ...)

BadRequestProblem
ValidationError
Unauthorized
Forbidden

[x] add application wide handling of defined errors

deployed in v1.4.0 #15


Benchmark test suite has poor execution when parallelized

Create a SharedMemory buffer to ensure that the large ScoreMatrix class can be shared across processes with copying.
Increase chunk size to reduce the overhead of creating new processes

Addressed with #16


Adding Context Subscription. Improving Variable Subscription while maintaining backward compatibility.

WARNING: ONLY ONE CONTEXT SUBSCRIPTION CAN BE MADE AND VARIABLE SUBSCRIPTIONS NOT SUPPORTED. Only for demonstration.
Hello,
I made a system for context subscription and improved Variable subscription in order to use Dictionary like the python library.
I have also created as a demo for new users that want to use this library and how they can me subscriptions.
The current pull request IS NOT FUNCTIONING due to a problem in HandleRequest.
I changed  trresult.Length = --len ; to  trresult.Length = len - 1; .  That way ONE context subscription can be made for demostration puproses. Change to  trresult.Length = --len ; to see the new improved Variable Subscription system.
I have also created a small sumo network with it's configuration to use with  the demo.
I would've loved it if you figured out the problem with HandleRequest and change it so it can support both Context and Variable subscriptions.
One can check if it is  Variable or Context subscription by testing the high part of the response byte
to be 0x0e for the first and 0x09 for the latter.  (from SUMO documentation https://sumo.dlr.de/wiki/TraCI/Object_Variable_Subscription https://sumo.dlr.de/wiki/TraCI/Object_Context_Subscription)
Thank you,
Chrysostomos Chadjiminas

Thank you for your pull request, unfortunately i don't have much time this week. maybe i have time on the weekend to check it out

Hello,
Did you have any change to review my code?
Thank you

Hello,
I managed to fix the problem.
please check my comments and check the code in HandleResponse in TraCIDataConverter class to see if the change is correct.
I have mad multiple context and variable subscriptions to make sure it works and it seems to be working just fine.
Thank you.

I'm pretty busy right now, but I'll check it asap.

Ok I understand,
Thank you.

Improved VariableSubscriptionEventArgs and ContextSubscriptionEventArgs to have the same response as https://sumo.dlr.de/wiki/TraCI/Object_Variable_Subscription and https://sumo.dlr.de/wiki/TraCI/Object_Context_Subscription so it  will be easy for someone to retrieve the variable count, context domain and object count. Also changed usage example to show this features.

Hello @cchadj01, the code looks good in my opinion.
Are variable subscriptions supported now? The code looks like that, but the above conversation indicates otherwise. If so, how complete is the API coverage? Are all variables covered (as described here)?
As for context subscriptions, is there still the limit for a single subscription?
I propose we merge this now, but try to work towards a more complete version the coming months.
Thanks a lot for the contribution, the API coverage is almost complete now, I believe!
Greets, Menno

Hello,
Multiple context subscriptions are supported. Variable subscriptions are
also working. Variable subscrptions can work along context subscriptions. I
have added a small demo project so new users can test the  subscription
systems and how one can use them. I am currently using this library in my
thesis. I didnt check all the variables (only lane and vehicle context
subscriptions) but since I implemented this functionality as an abstract
class I believe it would work for other variables as well. If a context
subscription would fail it is because I didnt override the right context
domain but I believe i havent done that mistake.
Thank you,
Chadjiminas Chrysostomos
On Wed, May 15, 2019, 11:30 AM Menno van der Woude<EMAIL_ADDRESS>wrote:

Hello @cchadj01 https://github.com/cchadj01, the code looks good in my
opinion.
Are variable subscriptions supported now? The code looks like that, but
the above conversation indicates otherwise. If so, how complete is the API
coverage? Are all variables covered (as described here
https://sumo.dlr.de/wiki/TraCI/Object_Variable_Subscription)?
As for context subscriptions, is there still the limit for a single
subscription?
I propose we merge this now, but try to work towards a more complete
version the coming months.
Thanks a lot for the contribution, the API coverage is almost complete
now, I believe!
Greets, Menno
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CodingConnected/CodingConnected.Traci/pull/15?email_source=notifications&email_token=AFK7IYJB2I2V7TNP6KMY2X3PVPC27A5CNFSM4HAV4NU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVN5ZQA#issuecomment-492559552,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFK7IYOSRHRJRV43D2TPOC3PVPC27ANCNFSM4HAV4NUQ
.

Hi Chadjiminas,
thanks for the prompt reply! fter I posted that initial comment, I reread
the conversation more thouroughly (and edited my post) and then already
merged. Let's keep an eye out for any problem that pops up. Great to have
this in the library now!
Thanks, Menno
On Wed, May 15, 2019 at 10:42 AM Ithil<EMAIL_ADDRESS>wrote:

Hello,
Multiple context subscriptions are supported. Variable subscriptions are
also working. Variable subscrptions can work along context subscriptions. I
have added a small demo project so new users can test the subscription
systems and how one can use them. I am currently using this library in my
thesis. I didnt check all the variables (only lane and vehicle context
subscriptions) but since I implemented this functionality as an abstract
class I believe it would work for other variables as well. If a context
subscription would fail it is because I didnt override the right context
domain but I believe i havent done that mistake.
Thank you,
Chadjiminas Chrysostomos
On Wed, May 15, 2019, 11:30 AM Menno van der Woude <
<EMAIL_ADDRESS>wrote:

Hello @cchadj01 https://github.com/cchadj01, the code looks good in my
opinion.
Are variable subscriptions supported now? The code looks like that, but
the above conversation indicates otherwise. If so, how complete is the
API
coverage? Are all variables covered (as described here
https://sumo.dlr.de/wiki/TraCI/Object_Variable_Subscription)?
As for context subscriptions, is there still the limit for a single
subscription?
I propose we merge this now, but try to work towards a more complete
version the coming months.
Thanks a lot for the contribution, the API coverage is almost complete
now, I believe!
Greets, Menno
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/CodingConnected/CodingConnected.Traci/pull/15?email_source=notifications&email_token=AFK7IYJB2I2V7TNP6KMY2X3PVPC27A5CNFSM4HAV4NU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVN5ZQA#issuecomment-492559552
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AFK7IYOSRHRJRV43D2TPOC3PVPC27ANCNFSM4HAV4NUQ
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/CodingConnected/CodingConnected.Traci/pull/15?email_source=notifications&email_token=AANHGEUN3TRLTUTO5TU7U5DPVPEGZA5CNFSM4HAV4NU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVN6XCA#issuecomment-492563336,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANHGESDEA4TI2JHH5I2VJTPVPEGZANCNFSM4HAV4NUQ
.

--
To ensure safety and privacy, this email will selfdestruct in 5 seconds.
Don't panic!


Payloads and Hyperlinks

Hi Jitesh,
Thanks for the awesome UI kit.
I have few doubts on how to configure payloads for below categories in domain.yml file

Buttons
Images
Quick Replies
Cards Carousels
Dropdown
Is there any documentation available for above payloads ?

Next, how can i display text along with some hyperlinks in response ?
I tried to do like ( - text: Here is the [link] (https://xxx.yyy.net)) , but its not working as expected.
Thanks,
Vinod L

Hey @vinothkumar1097 , thanks. I don't have any documentation for the above query and the response which you are trying to send is in markdown format which my UI doesn't supports.

Thanks Jitesh for the prompt reply.
So i need custom payloads for Buttons, Images, Quick replies and for dropdown. Could you please help on this.
Also for displaying hyperlinks in responses, how can i achieve it.

Hey @vinothkumar1097 , I will be writting a Blog on how to use all those elements, just wait for few days.

Thanks Jitesh. It will be useful for others too.
But for displaying the hyperlinks in responses, is there any way to achieve it ?

Hey @vinothkumar1097 , can't you send the link message as it is from rasa?

Sorry Jitesh for being late to reply. As my hyperlinks contain more than 200 words, it would be not user friendly to display the direct hyperlink response from rasa.
So it would be good if i encode hyperlink in a word (link). So when users click on the word link they would be redirected to the correct hyperlink.


Create Torrent page

Done:

Handle drag-and-drop (gives you an array of W3C File objects) and the Create Torrent menu item (gives you a path to a folder)
Show the user which torrent they're about to create

Left to do:

Change from modal to page
Add options.

We can't have a name option, for now. WebTorrent requires that to be the name of the file or folder we're seeding from
Display Name (DN) attribute
We should have an option to change the announce lits
Private bit
Order of files
etc

Fixes #389

New look
** Basic **

** Advanced **

I think this is ready to go now.
Caveats

The comment field doesn't work yet. See https://github.com/feross/create-torrent/issues/36
You can add and remove trackers, but removing the built-in WebTorrent trackers (the ones that start with wss://) doesn't work. They get re-added automatically by create-torrent because we set the WEBTORRENT_ANNOUNCE global variable. However, we can't just remove that variable because we want webtorrent to use those trackers for existing torrents.
I'm thinking we should add a different mechanism to tell webtorrent which trackers to always use. Global variables are gross anyway.

@feross might want to review

Nice work on this. My only suggestion (and i am not a designer) is about the style. I faced the same problem when i was doing the settings page (PR #426).
There is no definition on styling in the app (except for the gorgeous torrent list and player).
I saw you started using some kind of 'material design' but with a light theme, when the app clearly use a dark one. Now i see you go with the dark theme approach but i found the palette kind of high contrast B/W.
What we need to define:

We go with the 'Material Design' style
An 'Atom-alike' one (like you can see in the screenshots of https://github.com/feross/webtorrent-desktop/pull/426)
A new one, combining styles

If we go with the material design we need to follow it. A couple of days ago google released an example of material design for desktop, using the Chrome Browser as an example.
To be clear,it is not my intention to hijack your PR, nor criticizes your great work, just an attempt to define this thing to allow more people to get involved.

@grunjol I agree. We have a lot of colors, margins, font sizes, and so on that could be cleaned up and made consistent across the app.

I think it would be sweet to get a review from a designer. They could tell us what specifically needs to be fixed.
Then, we could go in and try to refactor and simplify index.css

I think we can do that in a future PR tho

👍


client self-study guide

👷 Deploy Preview for peaceful-meerkat-940447 processing.

Name
Link

🔨 Latest commit
c5daea56262a68a3da5b127a9e562f65b4115443

🔍 Latest deploy log
https://app.netlify.com/sites/peaceful-meerkat-940447/deploys/63109d6124ad6d0008cf6817


All Arguments Attributed To Logged In User

Once logged in, all arguments are attributed to the logged in user

Thanks, it's fixed!


Review of Target of From lane

This PR contains the suggested changes and comments for target from lane. Read the comments and reply to them and if you feel that changes are fine, merge it. It will be automatically updated in our PR to AGV-IIT-KGP.

@shivamvats I'm closing the PR, I've forgotten the context and can't work on it.


fix visium axes ordering for many versions AND cytassist.

this is yet another fix to the ongoing struggle of the visium spaceranger output (See #51 )
Now this has been tested:
tested it on spaceranger versions

[x] 1.1.0
[x] 1.3.0
[x] 2.0.0
[x] 2.0.1

with following type of full res images

[x] tif
[x] jpg
[x] png

AND

[x] Cytassist

🤞 something tell me it's no the last one

Codecov Report

Merging #94 (4266443) into main (09ffb1e) will decrease coverage by 0.05%.
The diff coverage is 25.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
- Coverage   41.44%   41.39%   -0.05%
==========================================
  Files          14       14
  Lines         871      872       +1
==========================================
  Hits          361      361
- Misses        510      511       +1

Files
Coverage Δ

src/spatialdata_io/readers/_utils/_utils.py
27.08% <100.00%> (ø)

src/spatialdata_io/readers/visium.py
23.23% <0.00%> (-0.24%)
:arrow_down:

Can confirm it works with me


Support marking dates before a min and max date as passive.

Implements #33

It looks like this is only to style passive days. Should we also prevent them to be selected? What do you think @mattfurness ?

merged this too

Hi @burakcan,
Sorry for the late replies. If a DayCell has the isPassive prop the handleSelect function will return null.
if (this.props.isPassive) return null;
Cheers,
-Matt.


populate s via JavaScript

The index.html file is getting quite bloated with all these <select>s and <option>s. Refactor the project to populate <select>s via JavaScript. This will get the repository listed as a JavaScript repository again.

<select>s are now created and populated via JavaScript.


[HOLD for payment 2023-05-16] [$1000] GBR and RBR show up at the same time in the LHN

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Request money from userA to userB
As userB notice the GBR indicator in the LHN
Perform an action that will result in an error
Notice that both GBR and RBR show up

Expected Result:
only one of GBR and RBR should show up at the same time
Actual result
There are two GBR and RBR shows up
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @luacmartins
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1682348382322349
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ca926e0b968c9e14
Upwork Job ID:<PHONE_NUMBER>286518784
Last Price Increase: 2023-04-25

Triggered auto assignment to @anmurali (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Proposal
Please re-state the problem that we are trying to solve in this issue.
GBR and RBR show up at the same time in the LHN
What is the root cause of that problem?
We are showing both GBR from OptionRowLHN and RBR from OfflineWithFeedback.
So when a report has both request money and error, we will see both GBR and RBR appear.
I can also reproduce on my end:

What changes do you think we should make in order to solve the problem?
It depends which dot will have higher priority, I think RBR will have higher priority.
So in case we want to fix this problem we should add a condition to show the GBR only when that report there's no error.
So the condition to display the GBR inside OptionRowLHN will look like this:
// only show the GBR when optionItem.allReportErrors is empty.
{_.isEmpty(optionItem.allReportErrors) && optionItem.hasOutstandingIOU && !optionItem.isIOUReportOwner && <Icon src={Expensicons.DotIndicator} fill={colors.green} />}

What alternative solutions did you explore? (Optional)
None

Job added to Upwork: https://www.upwork.com/jobs/~01ca926e0b968c9e14

Current assignee @anmurali is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav (External)

Current assignee @luacmartins is eligible for the External assigner, not assigning anyone new.

Proposal
Please re-state the problem that we are trying to solve in this issue.
GBR and RBR show up at the same time in the LHN
What is the root cause of that problem?
Because this report have both error and OutstandingIOU comment. So both GBR and RBR are show up
What changes do you think we should make in order to solve the problem?
We should add a condition if there is any error we should not display GBR
What alternative solutions did you explore? (Optional)

I am going to tag @Expensify/design @shawnborton to confirm if we need do something about this here?
For me as a user, I am fine seeing both GBR and RBR  (and as others have mentioned earlier), shows there was an error and there are some pending actions.
cc - @anmurali @luacmartins

Yeah, a couple of things here:

we should keep the pin/pencil icon together and not have an indicator in between them
red and green indicators should be the same size (20x20)
we should only show one indicator at a time

Part of me feels like you should only see one indicator at a time, and it should be the RBR indicator. cc @JmillsExpensify @trjExpensify for thoughts there

Yup, agreed. When we introduced Pattern B we said errors (i.e red brick road) would take priority and we wouldn't show both. So in this case, the LHN would have a red dot for the error and once dismissed, it would have a green dot for the outstanding request to settle.
CC: @iwiznia

Agreed

I think @hungvu193's proposal is fine here as it covers the expected result. We can put this on hold until #17163 is merged. I can see we're fixing size there, not sure about conditional rendering of the brick messages.
cc- @luacmartins

@mananjadhav I think we are only addressing size in that PR, so I don't think we need to hold on that. I think we can move forward with @hungvu193's proposal

📣 @hungvu193 You have been assigned to this job by @luacmartins!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

PR is ready (https://github.com/Expensify/App/pull/18137)
@luacmartins  @mananjadhav

PR is deployed to staging. Will be soon deployed to production soon.

I think we need a regression test proposal to avoid having the same issue again.
Regression Test Proposal

Open any chat with say userA
Pin the chat, and verify the Pin icon is showing in the LHN.
Request money from userA
Notice the GBR indicator in the LHN, and the Pin icon is hidden.
Perform an action that will result in an error
Verify that GBR is hidden and only RBR show up
Also verify that GBR and RBR are of the same size 20px.

I am not able to decide if we treat this is as a Feature Request? I can see we added the GBR in this PR https://github.com/Expensify/App/commit/2c0f6dd850e3f5b945286bb8b83906ed4aae2837 but I don't see any regression test/QA related to RBR or any requirements.  @luacmartins @anmurali any thoughts?

@mananjadhav maybe something like this would work?

Open a chat in different windows as two users - userA and userB.
As user B, pin the chat.
Verify the Pin icon is showing in the LHN.
As user A, request money from user B + > Request money
As user B, verify the GBR indicator shows in the LHN and the Pin icon is hidden
As user B, go offline and pay the request elsewhere
As user A, delete the request
As user B, come back online and verify that the pay request resulted in an error
As user B, verify that GBR is hidden and only RBR show up
Verify that GBR and RBR are of the same size

Yes this looks good. Thanks for helping out here.

I remembered that the weekend won't be counting as business day, so I think this issue will be eligible for bonus right @mananjadhav

@strepanier03 just re-added the Bug label since @anmurali is ooo and the payment is overdue. Could you help with that?

@hungvu193 and @mananjadhav - I've hired you in Upwork. Please accept as soon as you can and I'll check throughout the day so I can pay asap when you do. Thanks!

Accepted @strepanier03

Thank you @mananjadhav - I've paid you, ended the contract, and left some feedback.

Accepted @strepanier03 !

Thank you, @hungvu193! I've paid you via Upwork as well.
Closing this out as completed.


draggable_component.dart - no 'Selection.node' constructor

Have been using this as a way to learn angular2 - writing an app to display all of these on desktop and mobile.
In both standalone and in the a2 app, the 'Selection.node' constructor is called on lines 71 and 91, and is coming up undefined/not implemented.

Ok, I forked this with the intent to fix it, but it turns out that the actual constructor is 'Selection.elem' rather than 'Selection.node'.  Simple fix.  There are some other things I think might be worth looking at, so I'll keep the fork up, for now.  I'll relay the info here.  Right now, I just want to get my demo working....


Salts and keys are not set

Hello :)
In a fresh setup out-of-the-box, I notice that salts and keys are not set in /home/site/wwwroot/wp-config.php. This raises some security concerns, to say the least.
Is this intentional, am I missing something here?

Hi!
The out-of-the-box WordPress wp-config includes:
/**#@+
 * Authentication unique keys and salts.
 *
 * Change these to different unique phrases! You can generate these using
 * the {@link https://api.wordpress.org/secret-key/1.1/salt/ WordPress.org secret-key service}.
 *
 * You can change these at any point in time to invalidate all existing cookies.
 * This will force all users to have to log in again.
 *
 * @since 2.6.0
 */
define( 'AUTH_KEY',         'put your unique phrase here' );
define( 'SECURE_AUTH_KEY',  'put your unique phrase here' );
define( 'LOGGED_IN_KEY',    'put your unique phrase here' );
define( 'NONCE_KEY',        'put your unique phrase here' );
define( 'AUTH_SALT',        'put your unique phrase here' );
define( 'SECURE_AUTH_SALT', 'put your unique phrase here' );
define( 'LOGGED_IN_SALT',   'put your unique phrase here' );
define( 'NONCE_SALT',       'put your unique phrase here' );

/**#@-*/

Part of the standard procedure of bootstrapping a new WP instance is updating these phrases, optionally by using the tool provided.
I cannot speak on Microsoft's behalf, but suspect that maintaining this procedure is intentional.

@tersor, thanks for letting us know. We will add this by default.
WordPress uses a built-in random generator to create these keys and salts when they are not explicitly defined in the wp-config file. However, it's a good practice to define your own unique keys and salts manually. This way, you will have greater control over the security of your WordPress site.
You can run the below command on your from SSH console to generate the keys.
wp config shuffle-salts --path=/home/site/wwwroot --allow-root


Translate https://kubernetes.io/docs/setup/independent/install-kubeadm in Korean

This is a Feature Request

What would you like to be added
Translate https://kubernetes.io/docs/setup/independent/install-kubeadm in Korean
Why is this needed
No translation with Translate setup/independent/install-kubeadm
Comments

Pages to Update
https://kubernetes.io/docs/setup/independent/install-kubeadm/

/language ko
/assign

/remove-lifecycle stale


Pydantic or predictbody problems ?

Hello,
i'm fairly new to the platform and the whole ai training procedure, but i had an idea in mind and i wanted to try it out. so i followed the tutorials i could find and the documentation, i correctly isolated the vocals and hashed the audio with Adobe in 9 separate files of less than ten seconds, linked correctly the files folder into the web-ui, but upon clicking on Start Processing, i caught some errors in the CMD that i haven't seen popping up anywhere else, so here goes :
2024-01-01 16:57:39 | ERROR | root | Task exception was never retrieved
future: <Task finished name='n2m4xmw2ave_14' coro=<Queue.process_events() done, defined at C:\Users---\Desktop\Mangio-RVC-v23.7.0\runtime\lib\site-packages\gradio\queueing.py:343> exception=1 validation error for PredictBody
event_id
Field required [type=missing, input_value={'data': ['C:\Users\---...on_hash': 'n2m4xmw2ave'}, input_type=dict]
For further information visit https://errors.pydantic.dev/2.5/v/missing>
Traceback (most recent call last):
File "C:\Users---\Desktop\Mangio-RVC-v23.7.0\runtime\lib\site-packages\gradio\queueing.py", line 347, in process_events
client_awake = await self.gather_event_data(event)
File "C:\Users---\Desktop\Mangio-RVC-v23.7.0\runtime\lib\site-packages\gradio\queueing.py", line 220, in gather_event_data
data, client_awake = await self.get_message(event, timeout=receive_timeout)
File "C:\Users---\Desktop\Mangio-RVC-v23.7.0\runtime\lib\site-packages\gradio\queueing.py", line 456, in get_message
return PredictBody(**data), True
File "C:\Users---\Desktop\Mangio-RVC-v23.7.0\runtime\lib\site-packages\pydantic\main.py", line 164, in init
pydantic_self.pydantic_validator.validate_python(data, self_instance=pydantic_self)
pydantic_core._pydantic_core.ValidationError: 1 validation error for PredictBody
event_id
Field required [type=missing, input_value={'data': ['C:\Users\---...on_hash': 'n2m4xmw2ave'}, input_type=dict]
For further information visit https://errors.pydantic.dev/2.5/v/missing
i have legit no idea what to do and i would be very thankful  for your help
have a nice day and happy new year :)

This one is mangio-fork and we no solve this problem here...
Happy New Year


Feature Request: .string_handle() support for iterator based on Dataset to which .prefetch_to_device() has been applied

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): OSX
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): 1.8.0
Python version: 2.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory: AWS p3.2xlarge instance
Exact command to reproduce:

Request:
As stated in the title, it seems that a Dataset object upon which .prefect_to_device has been applied and an iterator defined from results in an iterator for which .string_handle() gives an error. In my case, my dataset and iterator are defined as
training_dataset_shuffle_batch = tf.data.Dataset.from_tensor_slices(training_data)
training_dataset_shuffle_batch = training_dataset_shuffle_batch.apply(tf.contrib.data.shuffle_and_repeat(buffer_size = dataset_size))
training_dataset_shuffle_batch = training_dataset_shuffle_batch.batch(minibatch_size)
training_dataset_shuffle_batch = training_dataset_shuffle_batch.apply(tf.contrib.data.prefetch_to_device(gpu_names[0]))
training_shuffle_batch_iterator = training_dataset_shuffle_batch.make_initializable_iterator()

and I'd like to use this with a feedable iterator defined as
handle = tf.placeholder(tf.string, shape=[])
iterator = tf.data.Iterator.from_string_handle(handle, training_dataset_shuffle_batch.output_types, training_dataset_shuffle_batch.output_shapes)
next_input_data_element = iterator.get_next()

However,
training_shuffle_batch_handle = sess.run(training_shuffle_batch_iterator.string_handle())
results in an
AttributeError: '_PrefetchToDeviceIterator' object has no attribute 'string_handle'
This unfortunately prevents us from using feedable iterators and simultaneous benefit from prefetching data to the GPU.
Thank you.

Awesome, thank you guys!

Nagging Assignee @rohan100jain: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

The preferred new way of doing prefetching is
ds.apply(prefetching_ops.copy_to_device("/gpu:0")).prefetch(1) [https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/data/python/ops/prefetching_ops.py#L355]
This should give you a regular PrefetchDataset object that you can create a regular iterator out of.
Let me know if that helps.

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 44 days with no activity and the awaiting response label was assigned. Is this still an issue?

We are closing this issue for now due to lack of activity. Please comment if this is still an issue for you. Thanks!


arithmetics should ignore nans created by alignment

Can anybody tell me if there is anybody who benefits from this behaviour? I can't think of any good use cases.
wallet = xarray.DataArray([50, 70], dims=['currency'], coords={'currency': ['EUR', 'USD']})
restaurant_bill = xarray.DataArray([30], dims=['currency'], coords={'currency': ['USD']})
with xarray.set_options(arithmetic_join="outer"):
    print(wallet - restaurant_bill)

<xarray.DataArray (currency: 2)>
array([ nan,  40.])
Coordinates:
  * currency  (currency) object 'EUR' 'USD'

While it is fairly clear why it can be desirable to have nan + not nan = nan as a default in arithmetic when the nan is already present in one of the input arrays, when the nan is introduced as part of an automatic align things become much less intuitive.
Proposal:

add a parameter to xarray.align, fillvalue=numpy.nan, which determines what will appear in the newly created array elements
change __add__, __sub__ etc. to invoke xarray.align(fillvalue=0)
change __mul__, __truediv__ etc. to invoke xarray.align(fillvalue=1)

In theory the setting could be left as an opt-in as set_options(arithmetic_align_fillvalue='neutral'), yet I wonder who would actually want the current behaviour?

Related pure-numpy thread
https://stackoverflow.com/questions/42209838/treat-nan-as-zero-in-numpy-array-summation-except-for-nan-in-all-arrays

This behavior is consistent with the default behavior on pandas, which always does an outer join for arithmetic:
In [6]: wallet.to_series() - restaurant_bill.to_series()
currency
EUR     NaN
USD    40.0
dtype: float64

So I don't think we want to change this in an inconsistent way in xarray.
I do agree that we should have support for an explicit fill value in alignment/reindexing and arithmetic. For consistency with pandas (and elsewhere in xarray), let's call it fill_value.

change __add__, __sub__ etc. to invoke xarray.align(fillvalue=0)
change __mul__, __truediv__ etc. to invoke xarray.align(fillvalue=1)

I can see the logic in using an identity value instead of NaN as a default in arithmetic. One peril of this approach is that it isn't always evident what the right identity is. In fact, according to NumPy:
In [16]: import numpy as np

In [17]: print(np.add.identity)
0

In [18]: print(np.multiply.identity)
1

In [19]: print(np.subtract.identity)
None

In [20]: print(np.divide.identity)
None

Let me give a couple other examples of why I don't think we should use an identity of some sort as the default:

Suppose we are comparing a model to observations. model - observations gives the residuals -- unless the variables are mis-aligned, in which case it would give a residual equal to one of the variables.
Suppose we want to do a weighted average. (observations * weights).sum() gives a sensible answer -- but not if a missing weight defaults to 1 (all the weights put together are supposed to sum to 1!).

Closed by #2876


add sysroot set

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
Cross Compile.
When I exec:
xmake f -p cross --sdk=/opt/riscv-gnu-toolchain -a riscv64 -cvD
configure
{
    arch = riscv64
    plat = cross
    ndk_stdcxx = true
    mode = release
    ccache = true
    clean = true
    kind = static
    target_os = linux
    buildir = build
    host = linux
    sdk = /opt/riscv-gnu-toolchain
}

There is no sysroot, but I need to build it with "sys/socket.h".
Describe alternatives you've considered
No response
Additional context
No response

you can pass it to --ldflags, --cxflags

I only want to change the sysroot when the toolchain is riscv, is there any way to check the toolchain?

I only want to change the sysroot when the toolchain is riscv, is there any way to check the toolchain? Anyway, I hope xmake can support the feature for that there may be a lot third party libs in the sysroot.

you can define custom toolchain and add ldflags/cxflags in toolchain()