Data race in Transceiver_Mid test

Your environment.

Version: 4f362ee

Steps to reproduce
$ go test ./ -run Transceiver_Mid -v -race -count=10

Log
=== RUN   TestPeerConnection_Transceiver_Mid
==================
WARNING: DATA RACE
Read at 0x00c00037c410 by goroutine 70:
  github.com/pion/webrtc/v2.(*PeerConnection).startRTPSenders()
      /pion/webrtc/peerconnection.go:973 +0x149
  github.com/pion/webrtc/v2.(*PeerConnection).startRenegotation.func1()
      /pion/webrtc/peerconnection.go:1724 +0x9bc

Previous write at 0x00c00037c410 by goroutine 102:
  github.com/pion/webrtc/v2.(*RTPTransceiver).setSendingTrack()
      /pion/webrtc/rtptransceiver.go:96 +0x7a
  github.com/pion/webrtc/v2.(*PeerConnection).AddTrack()
      /pion/webrtc/peerconnection.go:1196 +0x20c
  github.com/pion/webrtc/v2.TestPeerConnection_Transceiver_Mid()
      /pion/webrtc/peerconnection_renegotation_test.go:297 +0xfa5
  testing.tRunner()
      /usr/local/go/1.14/libexec/src/testing/testing.go:992 +0x1eb

Goroutine 70 (running) created at:
  github.com/pion/webrtc/v2.(*PeerConnection).startRenegotation()
      /pion/webrtc/peerconnection.go:1691 +0x7e
  github.com/pion/webrtc/v2.(*PeerConnection).SetRemoteDescription()
      /pion/webrtc/peerconnection.go:822 +0xb6b
  github.com/pion/webrtc/v2.TestPeerConnection_Transceiver_Mid()
      /pion/webrtc/peerconnection_renegotation_test.go:292 +0xe66
  testing.tRunner()
      /usr/local/go/1.14/libexec/src/testing/testing.go:992 +0x1eb

Goroutine 102 (running) created at:
  testing.(*T).Run()
      /usr/local/go/1.14/libexec/src/testing/testing.go:1043 +0x660
  testing.runTests.func1()
      /usr/local/go/1.14/libexec/src/testing/testing.go:1285 +0xa6
  testing.tRunner()
      /usr/local/go/1.14/libexec/src/testing/testing.go:992 +0x1eb
  testing.runTests()
      /usr/local/go/1.14/libexec/src/testing/testing.go:1283 +0x527
  testing.(*M).Run()
      /usr/local/go/1.14/libexec/src/testing/testing.go:1200 +0x2ff
  main.main()
      _testmain.go:338 +0x223
==================
    TestPeerConnection_Transceiver_Mid: testing.go:906: race detected during execution of test
--- FAIL: TestPeerConnection_Transceiver_Mid (0.43s)
    : testing.go:906: race detected during execution of test

Unable to reproduce after #1170 was merged.

@jeremija It should be fixed by #1170 since we skip starting a not negotiated sender. But the test should wait for negotiation to be finished before continuing on another negotiation. A possible solution is the one used in TestPeerConnection_Start_Right_Receiver:
	pcOffer.negotiationLock.Lock()
	pcAnswer.negotiationLock.Lock()
	pcOffer.negotiationLock.Unlock()
	pcAnswer.negotiationLock.Unlock()

Basically wait for negotiation to be completed before trying another negotiation.
This is related to a broader issue I'd like to open on defining/improving or documenting how pion/webrtc is resistant to concurrent calls. Currently there could be various issues if some peerConnection functions like the negotiation ones are concurrently called and I'm avoiding this by serializing calls to the peerConnection on the caller side. But this behavior should be changed or documented .

That's why I figured so I closed the issue. I restarted the test on your PR and it passed.
I can still create a PR with the negotiationLock.
It would be awesome if we could make pion more thread-safe so I won't have to resort to things like this: https://github.com/peer-calls/peer-calls/blob/4698405bf049e3293bece9375d9b3a763b9b6249/server/wrtcnegotiator.go#L84-L87

It would be awesome if we could make pion more thread-safe so we won't have to resort to workarounds like this: https://github.com/peer-calls/peer-calls/blob/4698405bf049e3293bece9375d9b3a763b9b6249/server/wrtcnegotiator.go#L84-L87

I think we should open a dedicated issue since there're different point in the code not really thread safe. Regarding the negotiation calls that needs synchronization this is also more complex since it shouldn't be permitted to call them in the wrong order (I remember that the webrtc specs should define how to handle such kind of states).


Not sure if this is an issue or not.... :(

Like the title says.... [DEVICE INFO] Model A1687 iPhone 6s Plus 128GB on iOS 12.2 (16E227)

log_file.txt
it jailbreaks and everything seems to be working fine to the best of my knowledge..... but just seeing the word FAILURE grinds my gears lol

if you think this is an error, please provide device and ios version in your issue aswell.

shouldn't be an issue as that thing is only used when searching for kernel offsets to exploit and patch s(o you can actually jailbreak), but it appears to be able to find those offsets anyway so you should be good to go.
Sent with GitHawk

Updated with device info and thanks for
The info lol7344 did you happen to look at the log by chance? Any idea what’s up with the few debs not installing apt 7 lib/key etc thanks again


upgrade packages

Yarn package for uport-connect is failing in Travis

some intermittent error during build?

Yup, broken for everyone, the internet is horrified
https://twitter.com/search?q=registry.yarnpkg.com&src=typd

Yay, good times!


isa.camera.hlc7

Hi,
I have isa.camera.hlc7 Camera and in HomeKit I can see just ON/OFF button and no live stream video - it is OK or I am missing something else?
Thx for lead the right direction.
Here is my config:
{
"devices": [
{
"name": "Mímo",
"ip": "<IP_ADDRESS>",
"token": "",
"deviceId": "1104686365",
"model": "isa.camera.hlc7",
"pollingInterval": 10,
"propertyChunkSize": 5,
"deepDebugLog": false,
"silentLog": false,
"deviceEnabled": true,
"micloud": {
"username": "",
"password": "",
"country": "de",
"forceMiCloud": true,
"timeout": 100
},
"buzzerControl": true,
"ledControl": true,
"childLockControl": true,
"modeControl": true
}
],
"platform": "miot"
}

There is currently no live stream support for cameras... That would need to be first implemented by someone who has knowledge of that.


Edits to Feed Posts not being saved

I was working on a Feed Post, and edited the title. I hit Save and then Done, but my edits had no effect on the post whatsoever. Please fix this bug as it's a pain to try to edit a post to no avail.

I think the issue here might have to do with caching, and it's still a thing.
Basically if you change something - avatar, bio, a blog post, a game description, etc., it can take a while (or several page refreshes) for the changes to become visible to you and others, which can often be a confusing experience. Posting new content isn’t affected, I would presume that’s because looking up freshly posted content makes a cache miss.


Add tasks to redirect manual sections to their parent manuals

Added the ability to page over Rummager results, as we know that there are manuals
with over 1000 sections (and at least one with > 5000)
Added two rake tasks: redirect_hmrc_section_to_parent_manual and
redirect_all_sections_to_parent_manual.  At present we don't have any need
to redirect sections to other manuals or manuals to other manuals.
The `redirect_all_sections_to_parent_manual' task outputs the source and
destination slugs which could be handy if something went awry.
Added a new model PublishingAPIRedirectedSectionToParentManual in the same
vein as PublishingAPIRedirectedSection.  There seems to be enough difference
in the validations (for example) to keep it separate.
Added a SectionRetriever class to encapsulate the construction and execution
of Rummager queries, and allow construction of result items as required so we
can construct both the Redirected and Removed section types from it.
I tried shoehorning it into PublishingAPIManual, but it didn't seem to fit
very nicely.  We don't need it for anything that PublishingAPIManual does at
the moment, so it seemed wrong to add complexity there.
Also removed some duplication from the remove/redirect rake tasks
https://trello.com/c/Wu6gbYDj/464-unpublish-pages-from-hmrc-manuals

I've rejigged it a bit in review of your comments @davidbasalla , what do you think?

I like the new class, this approach seems good! However I'd prefer not to use the yield approach as I find it makes the code harder to decipher (possibly contentious personal opinion so happy to chat about it). Maybe instead of passing in a constructor (?) in the block, the SectionRetriever could become SectionSlugRetriever and just return an array of section slugs? So building the sections in the rake task would become:
    sections = SectionSlugRetriever.new(manual_slug).sections_from_rummager.map do |section_slug|
      PublishingAPIRedirectedSectionToParentManual.new(manual_slug, section_slug)
    end

Not 100% sure if that's better, but maybe easier to understand. Let's have a chat though!

https://github.com/alphagov/hmrc-manuals-api/blob/redirect-all-sections/lib/tasks/redirect_sections.rake#L13 will need to be updated to use the new TaskHelper

All looks good, have fixed the last two comments while Simon's inoperational.
Merging!


Instructions to install Ninja updated.

One sentence was not syntactically correct, I have replaced it to make the installation process clearer.

Thanks for putting up with me ⛵

@swift-ci please smoke test


[REQUEST] Is there a way to update the virtual cockpit car skins

I have updated the vehicles in my skins on the MIB2 unit fine. However, the virtual cockpit still uses the old skins. Is there a way to update the look of those skins too?
For instance. I have modified one of the images for the MIB2 skins to this, id like the same on the virtual cockpit if possible.

Not yet!


forbid-prop-types: forbid contextTypes, childContextTypes

This PR adds support for checking contextTypes and childContextTypes for forbidden prop types. I also added support for checking getters, too, since I have seen them supported elsewhere.
Closes #1513

I'll merge this tomorrow if no objections.


[#7] [UI] As a user, I can register a new account with email and password

Closes #7
What happened 👀
A register feature with email and password
Insight 📝

Create a registration form.
Layout for users and authenticated users.

Proof Of Work 📹

We can adjust the desktop version and don't stretch fields for all screen width since usually desktop forms has fixed width (you can check login/register forms for popular services).
For that you can use different col values for desktop and for mobile: https://getbootstrap.com/docs/4.4/layout/grid/#grid-options (col-xl-)
What do you think?

Thanks for the suggestion 🙏 . I also use modal bootstrap to update the form layout.

Should we write some system test for the Login feature @vnntsu

@andyduong1920 nice catch, let me add it 🙏

@andyduong1920 Added registration feature test in d59ccd0


"Metrics are only available for models build using a model database" when using custom SBML files

I have a custom set of SBML files for the organisms in my samples.
The input DataFrame looks like this:
> df.head()
    sample_id   id  abundance                          file
0  sample1  taxa1   0.364398  /data/model.taxa1.xml.gz
1  sample1  taxa2   0.029670  /data/model.taxa2.xml.gz
2  sample1  taxa3   0.016877  /data/model.taxa3.xml.gz
...

Following the micom documentation for building community models, I execute build as follows:
from micom.workflows import build
manifest = build( taxonomy = df , model_db = None , out_folder = '/data/out', cutoff = 1e-2 , threads = 8 , solver = 'cplex')

Note that model_db argument is None because there is a file column in the input DataFrame, as instructed by the documentation for build.
However, the above build command generates the following error:
Running ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━   0% -:--:--
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/usr/lib/python3.8/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/usr/local/lib/python3.8/dist-packages/micom/workflows/build.py", line 28, in build_and_save
    metrics = com.build_metrics.to_frame().T
  File "/usr/local/lib/python3.8/dist-packages/micom/community.py", line 652, in build_metrics
    raise ValueError(
ValueError: Metrics are only available for models build using a model database :(
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/ubuntu/work/20210423_micom/src/test.py", line 13, in <module>
    manifest = build( taxonomy = indf , model_db = None , out_folder = outdir , cutoff = 1e-2 , threads = 8 , solver = 'cplex')
  File "/usr/local/lib/python3.8/dist-packages/micom/workflows/build.py", line 87, in build
    res = workflow(build_and_save, args, threads)
  File "/usr/local/lib/python3.8/dist-packages/micom/workflows/core.py", line 41, in workflow
    results = list(it)
  File "/usr/local/lib/python3.8/dist-packages/rich/progress.py", line 148, in track
    yield from progress.track(
  File "/usr/local/lib/python3.8/dist-packages/rich/progress.py", line 669, in track
    for value in sequence:
  File "/usr/lib/python3.8/multiprocessing/pool.py", line 868, in next
    raise value
ValueError: Metrics are only available for models build using a model database :(

Here is some system information:
$ python3  -c "import micom; micom.show_versions()"

System Information
==================
OS                  Linux
OS-release 5.4.0-1030-aws
Python              3.8.5

Package Versions
================
cobra        0.21.0
jinja2       2.10.1
micom        0.22.5
pip          20.0.2
scikit-learn 0.24.1
scipy         1.6.1
setuptools   45.2.0
symengine     0.7.2
wheel        0.34.2

My bad, build is only tested with a model database. I'll get to fixing that. For larger data sets I would still recommend building a model database as this will speed up the build step by quite a bit.

No worries, thank you for looking into this so quickly!
Yes, the workaround you suggested is working...

Now fixed in version 0.22.6: `pip install "micom>=0.22.6".


Update conftest.py

Type Hints: Added type hints to all functions for better clarity.

Docstrings: Added or improved docstrings to explain the purpose of each function and fixture.

Unused Imports: Ensure that all imports are necessary; remove any that are not used in the file. (This step requires checking the rest of the file or project to confirm usage.)

These changes should make the code more readable and maintainable. If you have specific changes in mind, feel free to let me know

please merge it if everything is ok


[Automation] Generate Fluent Lite from Swagger recoveryservices#package-2024-04

[Automation] Generate Fluent Lite from Swagger recoveryservices#package-2024-04

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.resourcemanager:azure-resourcemanager-recoveryservices


Attributes - disable required attribute warning message

When a Required attribute is disabled (turned Off), a triangle warning icon appears to its left. Clicking the icon displays the warning message: "This attribute is mandatory...".
To close the message, must click away / outside of it. Clicking the icon or warning message itself should toggle it closed.

There is a weird jump.... See the video. At the end, after I click on the white tooltip, the whole table jumps by about half row upwards. Odd....
https://user-images.githubusercontent.com/7923190/141517729-a4b56670-9126-4008-852f-d5ac5e915fb3.mp4


Skip GCC's __va_list fields on AArch64

Thanks @hbr-det ! Sorry for the huge delay. Will create a new release with this shortly.


DirectLine domain one endpoint websocket inside - standard endpoint

Hi,
Example from https://microsoft.github.io/BotFramework-WebChat/01.getting-started/g.hybrid-react-npm/
Code from g.hybrid-react-npm\packages\chat-component\src\ChatComponent.js
DirectLine call with parameters
useMemo(async () => {
  const domain = 'myDomain/v3/directline';
  const res = await fetch(`${domain}/tokens/generate`, {
      method: 'POST',
      headers: {
        Authorization: 'Bearer AAA',
      }
 });

  const { token } = await res.json();

  setDirectLine(createDirectLine({ token, domain }));
}, []);

method generate works fine:
Request URL: myDomain/v3/directline/tokens/generate
Request Method: POST
Status Code: 200 OK
Remote Address: myDomain
Referrer Policy: no-referrer-when-downgrade

method conversations works fine:
Request URL: myDomain/v3/directline/conversations
Request Method: POST
Status Code: 201 Created
Remote Address: myDomain
Referrer Policy: no-referrer-when-downgrade

websocket works different way:
Request URL: wss://directline.botframework.com/v3/directline/conversations/HfNJdMeadL6DBcmWQlWAbw-h/stream?watermark=1
Request Method: GET
Status Code: 101 Switching Protocols

If I turn off websocket (webSocket: true) - all works fine but I prefer use socket.
My clients has no access to internet, and my solution will connect to nginx myDomain proxy.
Regards,
Vitaliy

The Web Socket URL is a value from the /v3/directline/conversation call:
{
  "conversationId": "K4X7vWGMB...",
  "token": "ew0KICAiYWxnIjogIlJ...",
  "expires_in": 3600,
  "streamUrl": "wss://directline.botframework.com/v3/directline/conversations/K4X7vWGMB.../stream?watermark=-&t=ew0KICAiYWxnIjogIlJ...",
  "referenceGrammarId": "12345678-1234-5678-abcd-12345678abcd"
}

For client without Internet access, you can also consider using Direct Line App Service Extensions, which is targeting this type of users.

Hi @compulim
Direct Line App service Extension.
First of all - thank you for your advise - I really appreciate it.
Let me detail explain my plan.
I have got standard
1 QnA service
2 Published bot from QnA service (without any changes)
3 One intranet page call DirectLine with  domain = 'myDomain/v3/directline
4 One nginx proxy server with rules to redirect myDomain to Azure bot service
Just only one my local page with changes, all others from the box. This mean, upgrade for new version, bug fixing, support all this services I have got from MS.
Now, if I'll use Direct Line App service Extension
Yours suggestion
If I'll use it - I should place on-prem one more server. In case with proxy - I have got only proxy server and all others (support, traffic, and so on) I have got without changes from Azure. In my case it is - QnA + publish bot (without any changes) and one page for users. In your suggestion I should modify bot and more more ....
Additional not too geed describe how to setup this functionality from begin to end. May be you have additional resources where I can read and try your solution?
Lets return back to question ....
If I pass my domain to DirectLine class - why I have got connection to external socket in response?
Thanks
Regards,
Vitaliy


Underscore coming as spaces in terminal

Issue Type: Bug
The underscore "_" is not being displayed in the terminal within VScode.
eg: If i typed "my_channel", it would end up as "my channel" in the terminal. This is not an issue in the ubuntu built in terminal. All underscores are shown as spaces.


0.5.0

Multithreading
SHA256 Checksum verification
SQLite DB creation idempotent and refactored to allow us to make changes without breaking users existing projects
TQDM progress bars
Changed some function signatures which breaks backwards compatibility for some section of users
Better print output
GDAL flag GDAL_NUM_THREADS to use ALL_CPUS, which makes negligible difference in my testing with build_vrt but does utilize slightly more CPU%
Fixed bug in reporting of total time taken for build_vrt

Issue #16 reports failing downloads
The checksum verification should allow for more reliable reporting of corrupted downloads and there are now clearer print statements on when fetch_tiles should be rerun due to a failed download for any reason


Protected against devices without TouchID

Line https://github.com/jarden-digital/react-native-pincode/blob/master/src/PinCodeEnter.tsx#L141
should be protected agains TouchID beeing undefined on devices without such capabilities, to prevent a JS exception when accessing TouchID.

Yep sure. I'll fix this in next version. Thanks!

Done :)


Don't change code page on Windows Server < 1709

Chcp isn't fully supported for cmd on versions < 1709

looks good. make sure we port this over to M144 branch.

FYI, it was failing on Linux ARM builds because it was struggling with the Microsoft.Win32 import, so I put that import in a Windows only block. It looks like that fixed the problem.


Ensure a simple way to integrate with "reactive-streams"

The Project Reactor is one of implementations of org.reactivestreams:reactive-streams.

Fixed with related PR


support DART - new regularization = dropout trees during learning

There is nice article about dropout from neural nets, applied to gradient boosting:
http://arxiv.org/pdf/1505.01866.pdf
It is about drop out some trees during learning process and rescale weight of trees because of that...
It can help to accuracy...
Thanx for consideration...

This should not be too hard to add by adding a new gradient booster, or extending current gbtree.

I would suggest:

extending gbtree
new parameter 'dart_dropout', 'dart_p' or something in that direction (float, range [0,1], default: 0)

thoughts?

I've created a prototype.(not tested yet)
https://github.com/marugari/xgboost/tree/prototype_dart

@marugari It would be great if you can run some benchmarks and see the performance, we can look into bring it back to main repo

it improves performance slightly.
https://github.com/marugari/xgboost/tree/prototype_dart/tests/dart

This sounds interesting. @marugari Can we also refactored the code a bit?
Since I guess most part of dart could reuse the code of GBTree, and only prediction function needs to be replace. Let us consider use inheritance or a common base code, so the code is cleaner and easier to maintain.
Then I am happy to take a review and bring it to xgboost main repo

@tqchen Sure.
I also think Dart shold inherit from GBTree but could not fix duplicate symbols. :cry:

If there is not too much code, we can put two classes both in gbm.cc which should solve the problem

@tqchen If it is agreeable, I can refact the code.
The following methods are defined in Dart.

Configure
Load
Save
DoBoost
Predict
CommitModel
Pred
DropTrees
NormalizeTrees

Sounds good.

How's this?
https://github.com/marugari/xgboost/commit/d4e3a6b4cee1259be5245ea78ad601aef6b3ca2a

great work @marugari. thank you for bringing DART in.

@marugari  This looks good, can you

rebase your commits so the history of the old dart.cc is removed, another simpler way might be start from a clean fork
Add a few comments on what dart is, and the algorithms
Open a pull request and I will do more detailed code review in the PR
It is likely that you can reuse a few functions, like SaveModel, or change the parent function to isolate the common parts and further reduce the code.
To prevent the codebase from getting too large, let us avoid ipynb in the main repo. Use markdown and python scripts instead.  You can always put ipynb to https://github.com/dmlc/web-data
Please write an markdown introduction in documents on how dart can be used.

Thanks for the great effort

It has been merged. https://github.com/dmlc/xgboost/commit/949d1e3027727c3238e90de0cc35a666c64fb728
The tutorial is under construction.
https://github.com/marugari/xgboost/blob/prototype_dart/doc/tutorials/dart.md

great:)

@marugari any updates on english version of guest blogpost?

Updated.
https://github.com/marugari/xgboost/blob/prototype_dart/doc/tutorials/dart.md

looks nice,

it would be great if you add a few brief sentences to describe DART (like abstract in publication) in the intro.
Please send the PR to the dmlc blog

Reference format of other posts to add tags
https://github.com/dmlc/dmlc.github.io/tree/master/_posts
Add bio of yourself in the end
Add reference link to xgboost in the posts
Add a reference to your japanese version of blog

Separately, let us send the same post to https://github.com/dmlc/xgboost/tree/master/doc/tutorials

Can I use MathJax in dmlc.github.io?

seems not, maybe use images for formula for safety

Link to the post http://dmlc.ml/xgboost/2016/07/02/support-dropout-on-xgboost.html
Thanks for the great effort!
One last thing, let us PR a copy of tutorial to https://github.com/dmlc/xgboost/tree/master/doc/tutorials so it can benefit users who look into the document for recipes

I'm fixing formulas.

param = {'booster': 'dart',
'max_depth': 5, 'learning_rate': 0.1,
'objective': 'multi:softmax', 'silent': True,
'sample_type': 'uniform',
'normalize_type': 'tree',
'rate_drop': 0.1,
'skip_drop': 0.5}
num_round = 50
bst = xgb.train(param, xg_train, num_round)
Traceback (most recent call last):
File "", line 15, in
bst = xgb.train(param, xg_train, num_round)
File "//anaconda/lib/python2.7/site-packages/xgboost/training.py", line 121, in train
bst.update(dtrain, i, obj)
File "//anaconda/lib/python2.7/site-packages/xgboost/core.py", line 694, in update
_check_call(_LIB.XGBoosterUpdateOneIter(self.handle, iteration, dtrain.handle))
File "//anaconda/lib/python2.7/site-packages/xgboost/core.py", line 97, in _check_call
raise XGBoostError(_LIB.XGBGetLastError())
XGBoostError: unknown booster type: dart
Isnt it implemented in XGBoost ? I don't get it

Last version release does not support DART.
Please refer to gbtree.cc(or gbtree-inl.hpp).


Scaffold project

Defines CI infrastructure and scaffolds an empty solution

@pascalberger Any thoughts on using my "nickname" in the license and nuspec etc.?

@janniksam Sure updated it accordingly. Once this is merged I can add the code for the empty addin without the business logic / test cases.


Update advance search Twitter documentation

We currently do not document examples of advanced search in the swagger documentation or developer documentation so people understand the full functionality of the twitter search API.
Here are some examples of advanced search queries:
curl -X 'POST' \
  'http://localhost:8080/api/v1/data/twitter/tweets/recent' \
  -H 'accept: application/json' \
  -H 'Content-Type: application/json' \
  -d '{
  "query": "Masa Moon (from:getmasafi) until:2022-01-01 since:2006-01-01",
  "count": 10
}'

curl -X 'POST' \
  'http://localhost:8080/api/v1/data/twitter/tweets/recent' \
  -H 'accept: application/json' \
  -H 'Content-Type: application/json' \
  -d '{
  "query": "(Masa OR CoinList)(from:getmasafi) until:2022-01-01 since:2006-01-01",
  "count": 10
}'

These developer documentation can be found here: https://help.x.com/en/using-x/x-advanced-search

Closing in favor of #399


e2e: remove duplicated volumes entries

This commit

removes duplicated volume entry for /var/lib/nri-resmgr in the deployment and also reverts having multiple volume entries under the same name. resmgrdata volume already has /var/lib/nri-resmgr
in the previous commit, we had decoupled two hostPath based volumes to have their own unique name because kustomize based deployment could not unmarshal it. But we have a single volumeMount to include two hostPaths /sys and /host, which then get mounted into /host/sys. As such, I've reverted /sys and /host volumes to be as before.

Tests summary:
PASS topology-aware/n4c16/test00-basic-placement
PASS topology-aware/n4c16/test01-always-fits
PASS topology-aware/n4c16/test02-shrink-and-grow-shared
PASS topology-aware/n4c16/test03-simple-affinity
PASS topology-aware/n4c16/test04-available-resources
PASS topology-aware/n4c16/test05-reserved-resources
PASS topology-aware/n4c16/test06-fuzz
PASS topology-aware/n4c16/test07-mixed-allocations
PASS topology-aware/n4c16/test09-container-exit
PASS topology-aware/n4c16/test10-additional-reserved-namespaces
PASS topology-aware/n4c16/test11-reserved-cpu-annotations

/cc @jukkar @klihub

@fmuyassarov I think #15 is touching the same deployment files and leaving the same errors there. It would be better to fix these problems there in a single go (and close this PR).

@fmuyassarov I think #15 is touching the same deployment files and leaving the same errors there. It would be better to fix these problems there in a single go (and close this PR).

+1
/close

I think #15 is touching the same deployment files and leaving the same errors there. It would be better to fix these problems there in a single go (and close this PR).

No, this is just fine. I can apply this first and then remove my revert

Lets merge this one first, and then I can remove my revert in #15 as it was suppose to be temporary measure anyway.


Rename ethermintd.service.template to cronosd.service.template

Change to match the docs here

Closed because it's a duplicate of https://github.com/crypto-org-chain/cronos-docs/pull/72


Redux Boiler Plate Killers

Went for a mix of what we talked about:
First I added redux-actions to eliminate a lot of the boiler plate associated with action creators etc.
But it's still too much boiler plate, so I implemented a custom utility createSetters that creates a setter action creator for every key in your initialState
So if you have:
const INITIAL_STATE = {
  currentlyPlaying: null,
  youtubeResults: [],
}

You would get:
set.currentlyPlaying('Drake')
> { type: 'uniplayer/currentlyPlaying', payload: 'Drake' }

There's a little more documentation in the code and I'm happy to explain anything that's still weird (this is definitely my most functional style PR, so don't feel shy to ask what something does)
Oh and I also added a state update library called shades so instead of:
return {
  ...state,
  search: action.payload,
}

We can do:
return set('search')(action.payload)(state)

This has other advantages I allude to in the PR but you'll also see more later.

Handled in #31


HomePod & Volume accessories created even if not configured

Hi,
I use your great plugin to play announcements on my HomePod minis using http get method (shortcuts), as such I don't configure radios or audioFiles section in my config. Even there's no config entries, the plugin (I believe) creates HomeKit accessories that show up when adding other accessories.
In my config, I have 3 separate platforms with separate bridge username/port entries (for 3 HomePod minis), in my HK add accessory I end up w 6 un-needed/un-used accessories (3x HomePod radio, 3x Volume).
Would it be possible to only create those HK acc if they are configured ? Thanks.
Example from my config:
        {
            "homepodId": "idA",
            "serialNumber": "snA",
            "mediaPath": "/home/pi/media",
            "httpPort": 1234,
            "enableVolumeControl": false,
            "verboseMode": false,
            "_bridge": {
                "username": "DE:AD:BE:EF:DE:AD",
                "port": 33333
            },
            "platform": "HomepodRadioPlatform"
        }

Hi, @aboulfad - that's strange, will investigate...

Thank you, plz lmk if you need any logs or additional data.

Hi - could you please provide more logs, I've double checked the code and it should not create any accessories unless these are specified in the config...

Hi, here's some additional logs after restarting HB. I can further enable -D & collect those verbose logs. I have replaced some "sensitive" info (ports, Mac address,...) in both files. Those 3 volume acc + radio acc do show up in HomeKit when trying to add a new accessory. I have tried deleting them in HB "Unpair Bridges / Cameras / TVs / External Accessories" but they come back.
homebridge.log.txt
config-hp.json

Could this be the culprit ? https://github.com/petro-kushchak/homebridge-homepod-radio/blob/884851e65309d4075e267b4e375b557078d0ca67/src/platformConfig.ts#L68

yes, looks like this is backward compatibility issue - its not needed anymore, will release new version shortly

could you please try to install this alpha version: @petro-kushchak/homebridge-homepod-radio v3.0.5
Should have this issue fixed

can you please 😄 post the code in GH (commit, tag, branch) ? I have a local install of your plugin that I modified and is rubbing so I'd need to merge your changes to test them.

yep, this is the commit:
https://github.com/petro-kushchak/homebridge-homepod-radio/commit/4a1492dfac56c79f0fe447f8b8dd8ef23e4198eb

Thanks for the commit, I cherry-picked it and the Radio acc doesn't get created, great ! but we still have the Volume acc that are being created although not configured. I couldn't fully figure it out, but this code https://github.com/petro-kushchak/homebridge-homepod-radio/blob/884851e65309d4075e267b4e375b557078d0ca67/src/platform.ts#L70 should run & return & have a log entry which I can't find.

@aboulfad - good point! Changing it to false by default - this plugin got too many config entries, getting hard to test everything :)

The pseudo code could be as follows:
1- If a config entry exists, read it
2- If no config entry exists, set the default to false
I don’t understand why that line doesn’t work but the following works based on the above pseudo code:
this.enableVolumeControl = config.enableVolumeControl;
If (this.enableVolumeControl === undefined) {
this.enableVolumeControl = false;
}
Oh gosh after staring at it for some time I get it, enableVolumeControl is a Boolean, if the config value is false and it’s or’ed ( || )with true, the result will ALWAYS be true!!!

correct, so make sense to default it to false... in your case you can even comment out piece which adds volume control accessory (comment out this.addHomepodVolumeAccessory() inside platform.ts)

Thanks, I added my above code that handles correctly the boolean config value. This way I can still enable a vol acc in the future. Closing this issue, thanks for your support : )


Change to check last return value of BN_CTX_get

To make it consistent in the code base.
Based on #3676 and other discussions, this PR makes all checks of return value of BN_CTX_get to the 'check-last-one' style.

Updated @mattcaswell

Ping?

Ping @openssl/committers


[WIP] Make HTTP/2 default transport for HTTP client and server

$subject
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

This will be done after 1.0 hence closing


Structured metadata in the deed pages wrongly describe the CC-BY license alone

Problem
The RDFa about attribute in the deed.html template contains the hardcoded value http://creativecommons.org/licenses/by/4.0/:
https://github.com/creativecommons/cc.engine/blob/d0b93aea9920f070509569667574a6928b782241/cc/engine/templates/macros_templates/deed.html#L48
Consequence
The consequence is that the structured metadata for each license asserts those statements about <http://creativecommons.org/licenses/by/4.0/>, regardless of which actual license page is served. Thus, these statements are mostly wrong.
Solution
All that should be needed is to use  {{ license.uri }} in that place instead.
Context
At the National Library of Sweden we are consuming more and more linked data automatically. We would like to link to the CC licenses in our union catalogue, and to support that we need to show their metadata (identifiers, titles etc.) in our editing and presentation interfaces. Our system can cache copies of linked metadata verbatim, but due to this issue, right now we have to tweak this on our side. We would like assist to resolve this issue if possible.

Thank you for the quick and qualitative fix and follow-up!


Rearrange clientList array

actually when eliminating an item, so a window id, from the clientList array, it is replaced by an id with value 0. This is wrong, because then it updates the _NET_CLIENT_LIST with zeroes.
Write a CONSUMER that replace the window id to be eliminated from it, shifting to left the other windows id.
Function prototype in C code:
void CsEliminateWindowFromClientList(xcb_window_t clientList[], int arraySize, xcb_window_t deleteWindow);

It has become:
BOOL FnRemoveWindowFromWindowsArray(xcb_window_t windows[], int arraySize, xcb_window_t windowToRemoce);
In Trasformers.{h,m}


Ability to select or restrict OSM file used

Sometimes user should only have the option to edit roads or buildings. Need to be able to hide other OSM files from the survey and user.

Do we want to restrict based off of tags or what OSM XML file we are using or both?

It would be good to have a button in the UI to zoom to the extent of a given OSM file so we know where it is.

When unchecking a given data set (OSM File), we need to take that data out of the model. I am evaluating the pros and cons of either
a) iterating through what is in the spatial index and removing elements
b) rebuilding the spatial index

Note, this does not include the new "Control Panel" UI I was brainstorming. This adds another button on the top of the app. Consider this temporary.


chore: update text and doc url for git disconnect modal

Description
chore: update text and doc url
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manually

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/ok-to-test sha=544b5ee


Validate uniqueness of listed modules with varying names

Description
Follow up from https://github.com/kyma-project/lifecycle-manager/issues/1242
We validate uniqueness of module names in .spec.modules based on the specified name. However, since we support multiple alternatives for specifying a module name, the same module can still be added multiple times when using the alternative names.
E.g. template operator can be added twice by specifying
spec:
  modules:
  - name: template-operator
  - name: kyma-project.io/module/template-operator

Click here for examplary KymaCR as a result
  Name:         default
  Namespace:    kyma-system
  Labels:       operator.kyma-project.io/watched-by=lifecycle-manager
  Annotations:  operator.kyma-project.io/owned-by: kcp-system/kyma-sample
  API Version:  operator.kyma-project.io/v1beta2
  Kind:         Kyma
  Metadata:
    Creation Timestamp:  2024-01-19T14:56:17Z
    Generation:          10
    Resource Version:    12605
    UID:                 7a30e000-8d9d-451a-afa1-b46e605336bf
  Spec:
    Channel:  regular
    Modules:
      Custom Resource Policy:  CreateAndDelete
      Name:                    kyma-project.io/module/template-operator
      Custom Resource Policy:  CreateAndDelete
      Name:                    template-operator
  Status:
    Active Channel:  regular
    Conditions:
      Last Transition Time:  2024-01-22T09:16:13Z
      Message:               all modules are in ready state
      Observed Generation:   1
      Reason:                Ready
      Status:                True
      Type:                  Modules
      Last Transition Time:  2024-01-22T09:16:13Z
      Message:               module templates are synchronized
      Observed Generation:   1
      Reason:                Ready
      Status:                True
      Type:                  ModuleCatalog
      Last Transition Time:  2024-01-22T09:16:13Z
      Message:               skrwebhook is synchronized
      Observed Generation:   1
      Reason:                Ready
      Status:                True
      Type:                  SKRWebhook
    Last Operation:
      Last Update Time:  2024-01-22T09:16:13Z
      Operation:         kyma is ready
    Modules:
      Channel:  regular
      Fqdn:     kyma-project.io/module/template-operator
      Manifest:
        API Version:  operator.kyma-project.io/v1beta2
        Kind:         Manifest
        Metadata:
          Generation:  1
          Name:        kyma-sample-template-operator-1125230537
          Namespace:   kcp-system
      Name:            kyma-project.io/module/template-operator
      Resource:
        API Version:  operator.kyma-project.io/v1alpha1
        Kind:         Sample
        Metadata:
          Name:       sample-yaml
          Namespace:  kyma-system
      State:          Ready
      Template:
        API Version:  operator.kyma-project.io/v1beta2
        Kind:         ModuleTemplate
        Metadata:
          Generation:  2
          Name:        template-operator-regular
          Namespace:   kcp-system
      Version:         0.1.0
      Channel:         regular
      Fqdn:            kyma-project.io/module/template-operator
      Manifest:
        API Version:  operator.kyma-project.io/v1beta2
        Kind:         Manifest
        Metadata:
          Generation:  1
          Name:        kyma-sample-template-operator-2235966007
          Namespace:   kcp-system
      Name:            template-operator
      Resource:
        API Version:  operator.kyma-project.io/v1alpha1
        Kind:         Sample
        Metadata:
          Name:       sample-yaml
          Namespace:  kyma-system
      State:          Ready
      Template:
        API Version:  operator.kyma-project.io/v1beta2
        Kind:         ModuleTemplate
        Metadata:
          Generation:  2
          Name:        template-operator-regular
          Namespace:   kcp-system
      Version:         0.1.0
    State:             Ready
  Events:              <none>

We need to consider handling/preventing this case.
Reasons
Adding the same module twice is not explicitly supported so we should prevent this from being possible.
Acceptance Criteria

[ ] When adding the same module using alternative names twice to the module list

[ ] Then the API server refuses to update the KymaCR
[ ] Then I get an error message similar to the one when adding it twice under the same way

Feature Testing
No response
Testing approach
No response
Attachments
No response

We should opt to drop the support for different ways of referencing the modules in Kyma CR, this check can be dropped.


get with content_file hangs forever in master

Firefox 40.0.3, MacOS 10.10.5, MozRepl-0.06 perl module, MozRepl::RemoteObject 0.39, MozRepl <IP_ADDRESS>-signed plugin
use WWW::Mechanize::Firefox;
my $ua = WWW::Mechanize::Firefox->new( repl => '<IP_ADDRESS>:42042' ) || die;
my $resp = $ua->get('http://www.example.com/', ':content_file' => '/tmp/example.html');
print STDOUT "get returned $resp->code()\n";
The file is downloaded, but the get method never returns - it just keeps busy waiting forever.

Just tested with WWW::Mechanize::Firefox 0.78 and Firefox 35.0.1 (with a fresh profile with nothing but MozRepl) and I see the same thing. :-(

I think since Firefox 38 or so, they changed the events so that many tests broke. I couldn't find documentation on what changed and how to fix things. If you can test with a Firefox version (well) below 38 and things still break for you, I'll investigate again.


[opentitantool] Add JTAG I/O using OpenOCD

@timothytrippel this is work in progress
To run the test:
bazel test --test_output=all //sw/device/tests:test_openocd

Left a few more comments, but I think this is ready to be converted to non-draft once they are addressed, then we can get this merged and iterate on it to add the desired commands / enhancements. Nice work @pamaury !

Sorry for all the force push mess!
I've added the read and write register commands for the LC controller and tested them a bit in the sw/host/tests/chip/jtag/src/main.rs test.
I've left reading/writing memory and halt/resume as todo!()s for now since those only work on the RISC-V TAP, but I thought it was worth including the OpenOCD commands for a future implementer.
I've hopefully addressed the last round of review comments too, but @pamaury will have to resolve if they're done.


GPU support for run.c

Someone should add cuda support to run.c, I'm thinking of trying it out on the weekend.

I think somoeone with a gpu can try to set this up using OpenMP #pragma omp target directive. It supports CUDA and Apple GPU of I recall correctly. At the same time is really simple and easy to use so it will not bloat the code and make it complex. Sadly, i have only an AMD GPU so can't dig into this.
A nice reference here https://enccs.github.io/openmp-gpu/target/

Wow! Using OpenMP with GPU would be a very interesting thing.

i have only an AMD GPU so can't dig into this

An RTX 6000 costs just USD 0.50 per hour on Lambda Labs

You guys can also check #159

@kroggen Yes, OpenMP is amazing in my books ;]. With a single line of directive you can more or less have a GPU port. It is maybe not the most optimal but it's definitely easy. I think I might still give it a go and see what happens.

I have tried to do this using OpenMP here: #166 . Open to suggestions

Wow this was quick! I'll take a look with a fresh mind tomorrow.


If I delete a build, the build details is not cleared.

I will raise a pull request with a fix.

I will raise a pull request with a fix.


index: diff: reraise DataIndexTreeError if not with_unknown

We now have index.onerror to ignore that if needed.
Related https://github.com/iterative/dvc/issues/9733

Codecov Report
Patch coverage has no change and project coverage change: -0.04% :warning:

Comparison is base (f66462d) 62.18% compared to head (6f08698) 62.15%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #424      +/-   ##
==========================================
- Coverage   62.18%   62.15%   -0.04%
==========================================
  Files          58       58
  Lines        3895     3897       +2
  Branches      680      681       +1
==========================================
  Hits         2422     2422
- Misses       1338     1340       +2
  Partials      135      135

Files Changed
Coverage Δ

src/dvc_data/index/diff.py
83.01% <0.00%> (-1.06%)
:arrow_down:

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

What is with_unknown?

@dberenbaum It is a flag we use in dvc data status to make diff also return to us directories that are missing cache, meaning that the diff for those is "unknown". I know, confusing name, but wasn't able to come up with anything better on the spot back then.

Actually no need for this right now, i'll send a different PR.


Fix Question and Answer UI

PR Fixes:

1 Improves the UI of the Questions List
2 Implements new UI for Answers

Resolves #1046
Improved UI
Questions List

Question Page

Answers List

Video :
https://github.com/user-attachments/assets/33db7a36-f890-4031-8fb1-c35a95cc1088
Few Things to mention :

Previously we have been using a single PostCard Component for rendering both Questions as well as Answers. I am created two new components QuestionPost and AnswerPost to render them differently.
Also, for some reason (which I am unaware of) we are using both Pages Router and App Router of Nextjs in the codebase. However, I  implemented the components in both the routing pages.tsx files.

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

@hkirat @devsargam , if you guys can have a look at this PR. I hope you like the UI.
( This PR was having a merge conflict, which I resolved. The linting issue keeps happening due to some extra brackets for the regex in src/actions/payoutMethods/schema.ts. )

great one, your comment answer section looks cool.
Are you still working on it ?
Can I improvise it a bit in my PR #1123  ?

You should not have yarn.lock or package-lock.json

@devsargam , Let me fix the issue and make a new PR. Do you approve the new UI design ?

@devsargam I have made a new PR #1124


Added an explanatory description to the README file

I played around with the assets and like then so I decided to give more description in the REDME file. I hope it is useful for both new and old users.

Hello!
The text sounds too gpt-y to me. Was it generated?
I don't think it captures the goals of this repo, it's ok as it is.

No It was not generated.
On Thu, 7 Mar 2024 at 16:48, roger_s @.***> wrote:

Hello!
The text sounds too gpt-y to me. Was it generated?
I don't think it captures the goals of this repo, it's ok as it is.
—
Reply to this email directly, view it on GitHub
https://github.com/funDAOmental/lootunderworld-assets/pull/5#issuecomment-1983806319,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMJDUZT2DCDAPZBFOCVGZL3YXCD4TAVCNFSM6AAAAABEKW2266VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBTHAYDMMZRHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Support Elasticsearch 6.3

https://www.elastic.co/blog/doubling-down-on-open

Would it be better to skip 6.2 and go to 6.3? My team is telling me that 6.3 has a lot of functionality that would make their lives easier
As a side note, my team is already looking at 7.

Sure. once we add that we move to 6.3 directly.

This is now a blocker for our adoption for ES backends

Initial PR for container creation: https://github.com/kubedb/elasticsearch/pull/142

https://github.com/kubedb/elasticsearch/pull/143

any news about support for ES 6.3?


State/optimalisation fetch all markets liquidations

Fixed / integrated the fetchAllMarkets func. also fixed issues around the borrow | lend | liquidation page.

@BowTiedHeron this push contains the latest version of release/redesign


added chapter for neural network nutshell

this depends on https://github.com/slds-lmu/lecture_i2ml/pull/1148 for the lecture slides

zu 1): https://github.com/slds-lmu/lecture_i2ml/pull/1164


chore(main): release 0.1.0

:robot: I have created a release beep boop
0.1.0 (2024-02-13)
Features

Add another line (a0a8c4e)
Add another line (a2afa56)

Bug Fixes

Add test4 (910e429)
Add test4 (5522318)
Add test5 (fc5f241)
Added line (0dd71cf)
Added line (4aca032)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/daniel-mizuki/leetcode-solutions/releases/tag/v0.1.0 :sunflower:


Fine-grained identification of MSVC CXX version in traits.hpp

More fine-grained solution for #475 without poisoning downstream compile directives.
Tested against Microsoft Visual Studio Community 2019 Version 16.9.6 with CMAKE_CXX_STANDARD set to 11, 14, 17, and 20.

Sorry for not noticing this easy fix, I focus all my efforts on libc++ these days. Can you rebase onto develop and re-push? I'll merge this.


[ALLUXIO-3045] Add external minicluster

https://alluxio.atlassian.net/browse/ALLUXIO-3045
This PR adds the external minicluster and applies it to JournalShutdownIntegrationTest to replace a couple of our codebase's flakiest tests.
Features of the external cluster:

Can be configured with any number of masters and workers, as well as any Alluxio properties.
Cluster uses a temporary directory as its work directory. An alluxio-site.properties is written to workdir/conf, the underfs is workdir/underFSStorage, the worker ramdisks are workdir/ramdisk${worker_id}, and the journal folder is workdir/journal. Master logs are written to logs-master${master_id}, and worker logs are written to logs-worker${worker_id}
The cluster supports a saveWorkDir method which will create a tarball of worker state and store it in target/artifacts, where Jenkins will temporarily archive it. This will be invaluable for debugging failing tests.

Efforts taken to avoid leaking external processes:

Register a JVM shutdown hook to destroy the external processes on JVM exit
If a test uses the external cluster as a @Rule, the cluster will automatically get destroyed when the test ends.
Tests which use the external cluster directly should use try ... finally to ensure that they call cluster.destroy() when they are done.
External processes run in a wrapper which will cause them to terminate after 20 minutes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17318/
Test PASSed.

Merged build finished. Test PASSed.


Hamburger menu navigation pane background doesn't update with theme change

With Template10 defaulting the Light theme now, I noticed that when you switch to the Dark theme the Hamburger menu navigation pane background isn't updated. It stays with the light theme background.

This is an error in the template, not in the library. Resolved. Will update Extension.


Bug: diarization model parsing problem

What happened?
A bug happened!
Steps to reproduce
New installation, diarization not working
What OS are you seeing the problem on?
No response
Relevant log output
options: {
  "path": "/Users/usr/Downloads/Untitled notebook (1).wav",
  "lang": "en",
  "verbose": false,
  "n_threads": 4,
  "init_prompt": "",
  "temperature": 0.4,
  "translate": null,
  "max_text_ctx": null,
  "word_timestamps": false,
  "max_sentence_len": 1
}

Caused by:
    Failed to create ONNX Runtime session: Load model from /Users/usr/Library/Application Support/github.com.thewh1teagle.vibe/segmentation-3.0.onnx failed:Protobuf parsing failed.

    Location:
        /Users/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/pyannote-rs-0.2.7/src/session.rs:7:19

Location:
    core/src/transcribe.rs:167:108
App Version: vibe 2.5.4
Commit Hash: 0639a81cf382add6d51908098fafa2be0e72dc00
Arch: aarch64
Platform: macos
Kernel Version: 14.6.1
OS: macos
OS Version: 14.6.1
Cuda Version: n/a
Models: ggml-medium.bin
Default Model: ggml-medium.bin"
Cargo features:

CPU feature detection is not supported on this architecture.

https://github.com/microsoft/onnxruntime/issues/13117
Dup
https://github.com/thewh1teagle/vibe/issues/270


Backport add before_execution_proc support to Requests

Would you consider backporting this feature written by @ab as the Stripe gem cannot use version 2x of RestClient? It would really help us out with making OAuth requests.
https://github.com/rest-client/rest-client/commit/58c61be229d77744d2d3199620da1974e244292d

Opened a PR https://github.com/rest-client/rest-client/pull/478

Belatedly, thanks, but the stripe gem no longer uses rest-client, and I'm not inclined to backport features to old rest-client releases.


ID-1: Migrate and adapt forest key management crates

This is the first PR in the effort of implementing identity and key management in the agent so there is no need to import/export keys directly on subnet peer implementations. That way the agent will be able to sign and send message to a subnet without having to import a key on the peer they are connected to.
This PR migrates and adapts forest's key management code so it can be used as a base for the agent.

I feel it might be better to move these code into a separate crate in the repo and import in agent?


Changes to the dndRootElement prop do not update the Tree

If you mount <Tree dndRootElement={dndRootElement.current} /> when dndRootElement.current is null, changes will not be picked up when dndRootElement.current start to refer to a real element.

Maybe someone finds this useful. I don't know whether I did it optimally, but I wrote a custom hook to timeout rendering of children before the container is mounted.
import { useEffect, useState } from 'react';

const useIsParentMounted = (): boolean => {
  const [isParentMounted, setIsParentMounted] = useState(false);

  useEffect(() => {
    // Delay rendering of the child
    const timeout = setTimeout(() => {
      setIsParentMounted(true);
    }, 0); // Ensures the parent is mounted first
    return () => clearTimeout(timeout);
  }, []);

  return isParentMounted;
};

export default useIsParentMounted;

Now the <Tree /> can be rendered only after the parent container is mounted and has a non-null reference:
const containerRef = useRef<any>(null);
const isParentMounted = useIsParentMounted();
return (
  <div ref={containerRef}>
    {isParentMounted && <Tree
      dndRootElement={containerRef.current}
      // ...
      />}
  </div>
  );


'Choose Columns' does not show table extension fields in List pages

If table extension has fields with CaptionClass property, these fields appear in Choose Columns with blank captions.
The issue is reproducible in Dynamics 365 Business Central W1 13.2 (13.0.26413) and earlier versions.
Steps to repro:

Create simple table extension with a field which uses CaptionClass property:

tableextension 99999 CustomerTestFields extends Customer
{
    fields
    {
        field(50001; "My Field"; Code[10])
        {
            CaptionClass = '1,1,1'; // Global Dim 1 code as example. Custom captions will not work either.
        }
    }
}

Create page extension for list page:

pageextension 99999 CustomerListTestFields extends "Customer List"
{
    layout
    {
        addlast(Control1)
        {

            field("My Field"; "My Field")
            {
                ApplicationArea = All;
                Visible = true;
            }
        }
    }
}

After installing the extension, the column is shown correctly on the customer list page but when you try to Choose Columns to rearrange columns, the system shows the field as blank line at the end of the columns list:

Note, for Card pages customize page works fine and the field is correctly shown on a fastab and can be rearranged by user.

It works fine in the RTC client version 14.0.32280.


[2.5] Support for UAE in AKS

Forward port for https://github.com/rancher/rancher/issues/28341

I installed the latest v2.5-db350a6 and master-5ebc964, I am not seeing "UAE xxx" in the region list to pick from in UI. I am seeing in the commit the new region coded is been added, however, I am not able to see those changes in UI.

Verified on master-head, commit a31eecf and 2.5-head, commit e2e530c
I am seeing inconsistent "UAE" region support between master-head and 2.5-head.
On the master-head branch, it listed two regions, 1. "UAE North" and 2. "UAE Central. but 2.5-head has only "UAE North",

Reopening to fix the issue.

@sadiapoddar UAE central was specifically removed from 2.5-head in this commit because it could not be verified before the release. Its still in master head and will need to be validated before the next major release. @codyrancher can you confirm this is the plan?

@westlywright that is the plan.

Will be tracking UAE Central validation in this issue - https://github.com/rancher/rancher/issues/30278

Verified, 2.5-head, v2.5-ef23766

The UI dropdown menu in AKS cluster creation has the "UAE North" region listed as the location to pick from
Created an AKS cluster in "UAE North" and it looks good.


AFC support

Hi, I am just beginning to learn about USBMUX and all the stuff surrounding iPhone to Mac/PC communication, so some associations might be escaping me right now. With that said, I wonder if this library can be used to communicate with the AFC file system service?
I am interested in syncing photos and videos back and forth between a USB connected iPhone and a Mac or a PC programmatically. My understanding is that libmobiledevice would be the right tool to do jus that as it wraps usbmux and then provides APIs for various services like AFC.
I have checked the TODO section in the README but it makes no mention of AFC. Is that just because it is not on the roadmap or is there another approach that would be needed for it?
Also, can the implemented idevicebackup2 lib be used to sync photos and videos selectively? I don't think so based on inspecting its API but asking just in case.

Update: I have found https://github.com/mceSystems/libijs which has AFC support so I decided to use that. I'm still interested to see if you plan on supporting it, but since I have a solution now I am going to close the issue.

This library is an incomplete binding of the libimobiledevice. I'm not using this library anymore so I'm not sure if the original library supports it.


Missing compile time errors when running tests in strong mode

The following tests when run in strong nnbd mode are expected to produce a compile time error but instead either pass or produce runtime errors.
dartk-strong-linux-release-x64:language/nnbd/static_errors/not_assigned_local_initializer_test/11 is new and failed (RuntimeError, expected CompileTimeError)
dartk-strong-linux-release-x64:lib/mirrors/metadata_allowed_values_test/13 is new and failed (Pass, expected CompileTimeError)
dartk-strong-linux-release-x64:lib/mirrors/metadata_allowed_values_test/14 is new and failed (Pass, expected CompileTimeError)

dartk-strong-linux-release-x64:language/nnbd/static_errors/not_assigned_local_initializer_test/11 is fixed https://dart-review.googlesource.com/c/sdk/+/140403
The other failures are not NNBD related.


Sync phone calls migration and query examples with Grakn's latest master

@vmax given that you'll be writing tests for and bazelising the examples, I'm adding you as the reviewer of this PR.
The changes are being made on the (newly created) development branch, having it synced with the latest commit on Grakn Core and Clients Java, Node.js and Python.
This PR modifies the migration and query examples for the phone calls KG to:

reflect the latest changes made in the usage of clients
reflect the changes in the Graql syntax
modify the path and reading of the (raw) data files to have it consistent with how they'll be presented in the documentation

Note on the correctness of the changes: The Node.js and Python examples have been (manually) tested with latest Grakn and Clients (at the time of this writing). The Java examples have not been tested with the latest Grakn Core and Client Java and may require more updates as they get tested.

@vmax The examples are now identical to their counterparts in docs/development (which pass all the docs tests)


mWeb - Workspace - "only visible to ..." Notifier overflows with long names

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:

Login to staging new dot
Go to workspaces and add a user with a long name as a member
Go to the new room created with that member
Observe the "only visible to ..." Notifier overflows

Expected Result:
Notifier shouldn't over flow
Actual Result:
Notifier overflows
Workaround:
Unknown
Platforms:

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

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

Version Number: <IP_ADDRESS>
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

https://github.com/Expensify/App/assets/93399543/be110abf-38e2-4a5d-b558-ea35ebcfe2a7
Expensify/Expensify Issue URL:
Issue reported by: @sofoniasN
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692907694493579
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
The "only visible to ..." notice overflows with long names on small screens.
What is the root cause of that problem?
The number of lines is set to 1 for DisplayNames, so the text does not wrap.
What changes do you think we should make in order to solve the problem?
Change the numberOfLines prop to 0.
What alternative solutions did you explore? (Optional)
None.

@kbecciv it doesn't look likethe video in the OP is the one for this bug, can you double check?

Go to the new room created with that member

@SofoniasN, for the above, I'm not being sent to a room created with that member once I add them, is it the #announce or #admins room? Does it need to be a brand new user?  Thx

@abekkala  I'm off this week, can you please keep 👀 on this then I'll snag it back on Monday?  Thx

@mallenexpensify Let me check everything

Updated the video in GH report
https://github.com/Expensify/App/assets/93399543/ac408003-922d-44ce-8a95-268162ff53f3

Thanks @kbecciv , thanks @abekkala for 👀 too, I've unassigned you.
Was able to reproduce, added external label.

@akinwale's proposal looks like it has the correct RCA and the solutions LGTM! I think we might need design eyes on this to decide between the solutions. Personal preference is for the ellipsis option.
:ribbon::eyes::ribbon: C+ reviewed

Yeah I agree, I think solution 2 is more in line with our design principles. Do you mind confirming, @shawnborton or @dannymcclain? Thank you!

I'm good with solution 2 (truncating at one line). It seems like the simplest solution and we can always revisit if for some reason it causes any problems for users (can't imagine it would, just sayin').

Great! Go for it, @akinwale

@jjcoffee My PR is ready for review. Thanks.

Yup, agree with Danny here - let's do a 1 line trunk

The PR that introduced the bug has been identified. Link to the PR: N/A - the relevant styling has been missing since the whisper message was added
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
Determine if we should create a regression test for this bug. No - not particularly impactful.
If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again. N/A

Issue reporter: @SofoniasN paid $50 via Upwork
Contributor: @akinwale paid $750 via Upwork
Contributor+: @jjcoffee paid $750 via Upwork.
@jjcoffee please err on the side of creating regression tests because our QA team has design-related and monthly test checklists they run so they can likely add this to one of those. I created this one.
https://github.com/Expensify/Expensify/issues/321363


Change RegExp of mnemonic

Issue Type: Feature Request
Related issue #52576
Can you change the RegExp of mnemonic in the custom menu?
Because, some strings in transifex are not suitable for the custom menu (e.g. (&T), not (&&T)).
Native menu supports both styles, also 41 strings adopt the single & style. So, I think it might be a good idea to support the (&T) style

VS Code version: Code - Insiders 1.25.0-insider (07d89abf1522a67f0e8105770505077f454ab021, 2018-06-28T05:19:16.448Z)
OS version: Windows_NT x64 10.0.17134

moving to august to decide if this is necessary as it looks like strings have been updated to support the more explicit && style


Elevation is reset when changing screen orientation

Hi, is this problem fixed?

Not yet.
https://stackoverflow.com/questions/3542333/how-to-prevent-custom-views-from-losing-state-across-screen-orientation-changes/3542895#3542895
I tried the solution provided by this question, but for some reason my view restores the state twice, so the first one, which is the correct elevation, gets overridden.
I'm probably going to look this over next week.

Ok.

@TCQQ Giving a fast look into the code, I found the problem and fixed it.

Thankyou.

It's my pleasure :)


WordPress settings can't be queried?

It appears there is some sort of regression with settings from WordPress.
Trying to query:
{
  wp {
    generalSettings {
      title
    }
  }
}

Doesn't work. It says that a databaseId and uri are required input for querying the wp field.

Also having this issue

Gatsby
"gatsby-source-wordpress-experimental": "^6.0.0"

Wordpress
 "wp-graphql/wp-graphql-acf": "^0.4.0",
 "wp-graphql/wp-graphql": "1.0.5",

Also having this issue

Gatsby
"gatsby-source-wordpress-experimental": "^6.0.0"

Wordpress
 "wp-graphql/wp-graphql-acf": "^0.4.0",
 "wp-graphql/wp-graphql": "1.0.5",


applyModifier('unkown')  returns internal exception

This relates to my work on #1120: I just noticed that when calling applyModifier('unkown') on a query builder, a seemingly internal  ModifierNotFoundError is thrown without any error message set on it. Is this intentional? Shall I address it in the upcoming PR for #1120?

I just noticed that the problem goes further: ModifierNotFoundError.modiferName is undefined instead of the actual unknown modifier in those situations. I also addressed this in #1122


a error

Describe the bug
A clear and concise description of what the bug is.
Version
What version of the lol-bot are you running?
Screenshots
If applicable, add screenshots to help explain your problem.
Log Files
If applicable, add the log file containing the error messages from your issue
Additional context
Add any other context about the problem here.

how to solve ,THX

This should be fixed in the latest release.


Problem with installing/running on Ubuntu 18.04

I have several problems with installing/running WebViz on Ubuntu 18.04

I had to install nodejs and npm. I followed the description on
https://wiki.ubuntuusers.de/Node.js/
When installing npm, I got the error, that dependencies are missing that cannot be resolved (nodejs-dev node-gyp libssl1.0-dev)
So I followed the hints on https://askubuntu.com/questions/1088662/npm-depends-node-gyp-0-10-9-but-it-is-not-going-to-be-installed
and did
sudo apt-get install nodejs-dev node-gyp libssl1.0-dev
But this command deinstalled many of the libraries in my system especially my ros development environment. I am not able to work with the ros tools anymore.

After installing nodejs and npm, I followed the guidelines in the github readme.
I did:
npm run bootstrap.
I remember that there were 2 red lines in the output, that  dependencies could not be resolved, but don't remember exactly which.

When starting webviz, the browser open and I got the message that there is no access to cruise-control/button. So I changed in the package directory and installed cruise-control/button separately. But when starting webviz, there was the next message, that popper.js is missing. So I installed all packages in the folder packages separately. Is this correct? Do I have to install them by myself separately?

After installing all sub-packages, starting WebViz (http://localhost:8080/try/), only a black background appears. In the console I get the error:
ERROR in ./packages/webviz-core/src/components/withDragDropContext.js
Module not found: Error: Can't resolve 'react-dnd-html5-backend' in '/home/q456941/webviz/packages/webviz-core/src/components'
@ ./packages/webviz-core/src/components/withDragDropContext.js 10:51-85
@ ./packages/webviz-core/src/components/Root.js
@ ./packages/webviz-core/src/loadWebviz.js
@ ./packages/webviz-core/src/index.js

How to proceed? How to install/run WebViz on my computer (ubuntu 18.04) correctly?

When building:
npm run build
I get the same error as above.
Here is the corresponding npm-debug.log
1 verbose cli [ '/usr/bin/node', '/usr/bin/npm', 'run', 'build' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prebuild', 'build', 'postbuild' ]
5 info lifecycle root@~prebuild: root@
6 silly lifecycle root@~prebuild: no script for prebuild, continuing
7 info lifecycle root@~build: root@
8 verbose lifecycle root@~build: unsafe-perm in lifecycle true
9 verbose lifecycle root@~build: PATH: /usr/share/npm/bin/node-gyp-bin:/home/q456941/webviz/node_modules/.bin:/opt/ros/melodic/bin:/home/q456941/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/usr/bin:/home/q456941/llvm-project/build/bin
10 verbose lifecycle root@~build: CWD: /home/q456941/webviz
11 silly lifecycle root@~build: Args: [ '-c', 'lerna run build && webpack' ]
12 silly lifecycle root@~build: Returned: code: 2  signal: null
13 info lifecycle root@~build: Failed to exec build script
14 verbose stack Error: root@ build: lerna run build && webpack
14 verbose stack Exit status 2
14 verbose stack     at EventEmitter. (/usr/share/npm/lib/utils/lifecycle.js:232:16)
14 verbose stack     at emitTwo (events.js:126:13)
14 verbose stack     at EventEmitter.emit (events.js:214:7)
14 verbose stack     at ChildProcess. (/usr/share/npm/lib/utils/spawn.js:24:14)
14 verbose stack     at emitTwo (events.js:126:13)
14 verbose stack     at ChildProcess.emit (events.js:214:7)
14 verbose stack     at maybeClose (internal/child_process.js:925:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)
15 verbose pkgid root@
16 verbose cwd /home/q456941/webviz
17 error Linux 4.15.0-55-generic
18 error argv "/usr/bin/node" "/usr/bin/npm" "run" "build"
19 error node v8.10.0
20 error npm  v3.5.2
21 error code ELIFECYCLE
22 error root@ build: lerna run build && webpack
22 error Exit status 2
23 error Failed at the root@ build script 'lerna run build && webpack'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the root package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     lerna run build && webpack
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs root
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls root
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

Hey @fkeidel, could you try running npm run build before npm run docs? Basically the code is referring some internal packages. We forgot to put this step in the README, and it'll be updated in #177.  Let me know if you still have issue running the app.

@jtbandes Did you solve this issue?

@vivekrathi14 this was probably closed due to inactivity-- are you running into the same issue?

@troygibb Yes I am facing the same issue. Same problem with npm as per above points 1 & 2. Initially I was able to do  sudo npm run bootstrap (I am in webwiz directory while running this). However I got errors with npm run build. I wasn't able to fix them. After doing this when I ran my ros project almost all packages like gazebo, rviz were uninstalled (similar to 1) & later doing npm run bootstrap gives error. Also sudo npm doesn't work. I checked whereis node and it did show /usr/bin/node.
I didn't do anything else worrying about installing ros packages again.
I am using Ubuntu 18.04, ROS melodic.

As an aside, you can avoid using sudo by installing and managing Node versions using nvm.
That's what I'm doing on Ubuntu 18.04.
There are other ways of installing Node locally (non-sudo) but I've settled on nvm for now.


Integrate with busy-signal

busy-signal is a base Atom package that allows it's consumers to indicate to the user that something is happening. It has an easy to use API.
The main benefit of using it is that it'll make your own codebase cleaner while providing a unified place for users to see if something is happening.
Edit: Here's a bittersome discussion on the same topic that happened on linter repo

It's funny. We had the same idea. I'm actually working on something similar right now: https://atom.io/packages/busy
It's not ready yet, but I'll be looking to use that instead

Any particular reason of having two packages? We could both collaborate on one

Not really no. I had just thought about for a few weeks and wanted to follow through :)

Great, I've given you push access to the busy-signal package, which also means publish access on APM. Lets collaborate on just one :)

@noseglid @steelbrain So what happened here in the end?
If I have build and linter installed (as many probably do) I have all this extra stuff clogging up the status bar…

Same issue, bit distracting...

I'm quite sorry for doing this, but since there are still two different packages, and I need my working environment as clean as possible, I created two bridge packages, which pretend to be busy or busy-signal, but actually just call the other one's API.

atom-busy-unifier-ng for whom wants only @noseglid busy
atom-busy-unifier-sb for whom wants only @steelbrain busy-signal

I hope they soon won't be useful anymore, and will be replaced by a single busy package :)
But for now, they should be useful to resolve the clogging problem...
They should work well, but I didn't have enough time to test them thoroughly, so any feedback or bug report is welcomed :)

It's been almost a year. Do we really need two packages and two icons in the status bar? Please, guys, just join your forces, I think users mostly don't which one to use, but having both is inconvenient (and @rapgenic's workaround is not a real solution).

I actually prefer to have two icons.
busy-signal to show processes that end automatically
and busy to show long pending processes that may require human intervention to end


Bot stops repying per intent on RASA Shell

Rasa version: Rasa 1.2.9
Python version: 3.6.9
Operating system : Ubuntu 16.04
I've just recently began exploring Rasa after having used Dialogflow for a long while. Once I get a better understanding of it's more intimate components, I may customize the models on the pipeline with my own embeddings or of the sort for better classification in Spanish.
However, having tested it on different installations of rasa using the english sample demo chatbot that's generated as a project when running rasa init --no-prompt, I'm finding that every time such sample replies only a limited number of times for every intent triggered when trying the bot on the shell.
Normally and from my experience using other services and providers, the bot replies every time the intent is triggered, and stories should be able to be repeated instead of triggered only once, which doesn't happen in this case, along with other unusual behavior, like triggering the wrong intents or even replying with the same intent to completely nonsensical strings.
Example 1

Example 2

I happen to get these warnings as well any time I make use of rasa, even when just getting the version.
I don't know what to make of them, so I'll attach them in case it could be relevant to this.
/home/user/rasaenv/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:526: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_qint8 = np.dtype([("qint8", np.int8, 1)]) /home/user/rasaenv/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:527: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_quint8 = np.dtype([("quint8", np.uint8, 1)]) /home/user/rasaenv/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:528: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_qint16 = np.dtype([("qint16", np.int16, 1)]) /home/user/rasaenv/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:529: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_quint16 = np.dtype([("quint16", np.uint16, 1)]) /home/user/rasaenv/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:530: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_qint32 = np.dtype([("qint32", np.int32, 1)]) /home/user/rasaenv/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:535: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   np_resource = np.dtype([("resource", np.ubyte, 1)])
Now as to the contents of the config, nlu, stories and domain files, they're the same than the samples anyone can get and see for themselves, but I include them here anyway to outline how little there is for me to make changes to see if the issue is on any of them.
config.yml:

domain.yml:

stories.md:

since it is presenting this behavior right out of the box, I don't see much else to try before fixing this issue.

Thanks for the issue, @btotharye will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

hello @JECasillas this is actually not a bug but more how the product and stories work.  Rasa uses stories to train the bot the type of conversation flow.  So if it already answered a flow or such it wouldn't respond back again.  You can read more about this at https://rasa.com/docs/rasa/core/stories/
If you have further questions about this I would open up a post on the https://forum.rasa.com/ and we can help there.
I'm going to close out this issue for now since its not a real bug.
Thanks


[HOLD for 30973] Task - When returning from the assignee page back to the task page, the compose focus is lost

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

Version Number: <IP_ADDRESS>
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Launch app
Tap on a report
Tap icon near compose
Select Assign task
Enter title
Tap next
Tap confirm task
Tap on Assignee
Tap app's back button and return to task page

Expected Result:
When returning from the assignee page to the task page, the compose focus should not be lost.
Actual Result:
When returning from the assignee page back to the task page, the compose focus is lost
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/186192cd-b248-4a29-869f-b91e6afc22e8
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ad405e0798cfa725
Upwork Job ID:<PHONE_NUMBER>074606080
Last Price Increase: 2023-11-10

Proposal
Please re-state the problem that we are trying to solve in this issue.
Task - When returning from the assignee page back to the task page, the compose focus is lost
What is the root cause of that problem?
When the user navigates back the focus event is not called.
What changes do you think we should make in order to solve the problem?
We should use the hook useFocusEffect when using navigates back, something like:
useFocusEffect(() => {
        if (_.has(props.session, 'authToken')) {
            // Pop the concierge loading page before opening the concierge report.
            Navigation.isNavigationReady().then(() => {
                Navigation.goBack(ROUTES.HOME);
                Report.navigateToConciergeChat();
            });
        } else {
            Navigation.navigate();
        }
    });

Proposal
Updated
Just in case the PR doesn't fix this (or if they want to use something like this in that PR) I've updated my proposal with an actual use of what I said.

Alright, holding for https://github.com/Expensify/App/pull/30973, as that will fix the issue

PR is still open, so not overdue

PR is still open

Deployed to production yesterday! Adjusting title for the 7-day hold 👍

@tienifr bump on the above ^^^ Thanks!

@tienifr bump please!

@parasharrajat any update on the test? Thanks!

Sorry, was OoO — resetting the title now!

Price is correct 👍

Not overdue

Issue not reproducible during KI retests. (First week)

Issue not reproducible during KI retests. (Second week)

I also can't recreated. @kbecciv we're not able to recreate this, so going to close 👍


R-50 VG weights give worse performance than Imagenet weights

I tried to plug in the R-50 pretrained model weights into a torchvision resnet50 model using a simple pattern matching script
However this does not seem to improve performance as expected and in fact performs worse than image net weights for image size 448 x 448 in an MCAN type model (custom implementation - where I replace LSTM with a BERT and dont use Self attention on top of language features)
Am I using the weights wrong?

Did you try the released features directly for MCAN? Did you do it on VQA or other task? That can help debugging. However, we are not supposed to debug for you, so expect slow response here.

No I generated grid features during training and trained it end to end. Yes it was on VQA. Yes I understand. I just wanted to know if directly loading your pretrained model's weights into torchvision resnet50 model's state dict would work as a csutom grid feature extractor?

We haven't tried directly using it in to torchvision models but it would be interesting to know (I guess it should show some improvements over image-net pre-trained ones). one thing you can try (to debug) is to extract the grid features using this code base (use the 448x448 size) and using torchvision, and see if there is a difference. If so, there is a bug.

oh ok yeah makes sense. I'll try it out and get back to you then I guess

Another factor that may cause the discrepancy is the pixel mean and std values. The values used by Detectron2 (and hence this repo) are different from torchvision. https://github.com/facebookresearch/detectron2/blob/master/detectron2/config/defaults.py#L35-L42

oh interesting. I'll try this out too thanks!

Any updates? I am thinking we can upload it to pytorch so that one can directly get the weights like ImageNet pre-trained ones.

No, I haven't been able to work on it for a while... nor will I for some time in the future I guess

Okay, then closing this for now. You can comment once you have a clear signal.


self.avoidingViewUsesAutoLayout = avoidingView != nil ? avoidingView!.superview!.constraints.count > 0 : false CRASH

I get a nil error.
fatal error: unexpectedly found nil while unwrapping an Optional value

Seems like your avoidingViews superview is nil - has it been removed from the view hierarchy?
Does changing the line to this fix it?
self.avoidingViewUsesAutoLayout = (avoidingView != nil && avoidingView!.superview != nil) ? avoidingView!.superview!.constraints.count > 0 : false


kured: remove extra quotes from annotation

Why is this PR needed?
Remove extra quotes from annotation kured lock annotation. Before this
patch, the annotation created would look like:
weave.works/kured-node-lock: '''{"nodeID":"manual"}'''

With this patch, the annotation looks like:
weave.works/kured-node-lock: '{"nodeID":"manual"}'

/cc @SUSE/caasp-qa
/cc @troytop
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

I just backported this to release-caasp-4.1.0 branch


Windows Terminal can only be installed in user scope

Brief description of your issue
Hello,
Currently, Windows Terminal can ONLY be installed in user scope.  That means:

If I run the command "winget install -e --id Microsoft.WindowsTerminal", only current user can use the application
If I run the command "winget install --scope machine -e --id Microsoft.WindowsTerminal", a red error message is returned and the application is NOT installed

Windows Terminal is the only application I find to have this behaviour.  Please tell me how to install it in machine scope, if possible.
Thanks
Steps to reproduce
Run the command "winget install --scope machine -e --id Microsoft.WindowsTerminal"
Expected behavior
Application is installed in machine scope
Actual behavior
RED error message and the application is not installed
Environment
Windows Package Manager v1.1.13405
Copyright (c) Microsoft Corporation. Tous droits réservés.

Windows: Windows.Desktop v10.0.19044.1566
Package: Microsoft.DesktopAppInstaller v1.16.13405.0

This is the intended behavior and applies to all APPX/MSIX apps, not just for Windows Terminal.
All MSIX apps are installed only for the current user.
For system-wide MSIX app deployment, you can take a look at https://docs.microsoft.com/en-us/powershell/module/dism/add-appxprovisionedpackage?view=windowsserver2022-ps.


Change the way edm CMake mode handles branches of local dependencies.

Up until now edm always tried to checkout the branch specified in the dependency for local dependencies in the workspace unless the repository has uncommited changes.
This leads to very surprising behavior, you could start working on a feature in a local dependency in the workspace and everything just worked as expected - until you committed your changes to a new branch.
If this lead to the new branch beeing "clean" (no uncommitted changes present) edm would just change the branch of the local dependency back to the original branch (usually main).
This change adresses this issue by never changing a branch in CMake mode if the local dependency already exists.
Signed-off-by: Kai-Uwe Hermann<EMAIL_ADDRESS>

As we will change a bit of the everest-dev-enviromnet structure, I merge the commit and do follow-up fixes then


[MM-20603] Migrate components/team_settings to TypeScript

Summary

In this PR:

Migrated: components/team_settings/index.js --> index.ts
Migrated: components/team_settings/team_settings.jsx --> team_settings.tsx

Ticket Link

Fixes https://github.com/mattermost/mattermost-server/issues/13471

Hello @djanda97,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

I get the following error in my IDE when hovering over GeneralTab in team_settings.tsx:
Type '{ team: Record<string, unknown>; activeSection: string; updateSection: () => void; closeModal: () => void; collapseModal: () => void; }' is missing the following properties from type 'InferPropsInner<Pick<{ updateSection: Validator<(...args: any[]) => any>; team: Validator<object>; activeSection: Validator<string>; closeModal: Validator<(...args: any[]) => any>; collapseModal: Validator<...>; maxFileSize: Validator<...>; actions: Validator<...>; canInviteTeamMembers: Validator<...>; }, RequiredKe...': maxFileSize, canInviteTeamMembers, actions
My guess for why this error is appearing is that components/team_general_tab requires more props than are being supplied to GeneralTab in team_settings.
Any advice for how to remedy this?

@devinbinnie Since the consumed component is relatively large (800+ LOC), I opted to just remove isRequired from a few of the props in team_general_tab.jsx.

/update-branch


How can I pad images using mx.nd.pad?

I tried to use mx.nd.pad to pad images decoded by mx.image.imdecode.
Is there any way to pad the images?

reshape to (1, C, H, W)?

MXNet supports and is more efficient with channels_first format only.
Resolving.
For usability question, please ask at - https://discuss.mxnet.io/


[BUG] Multi-word configuration properties for match queries don't work

Describe the bug
Setting MatchQueryConfig.maxExpansions doesn't work
To Reproduce
Steps to reproduce the behavior:
Configure a match query with maxExpansions = 100 (any number works) and execute the query. An error "[match] query does not support [maxExpansions]" is returned because the client passes "maxExpansions": 100 as part of the JSON sent to the OpenSearch server
Expected behavior
"max_expansions": 100 should be passed to the OpenSearch server
Your context
latest version 1.99.14 on Ubuntu 22.04
Will you be able to help with a pull request?
Sure, I can create a PR
Overall I think the default for the JsonDsl constructor should be namingConvention = PropertyNamingConvention.ConvertToSnakeCase. That's the naming convention for OpenSearch after all. Naming things camelCase should be the exception, not the rule

right, I'll have a look at that one. Thanks for reporting

Wow, this was broken in a lot of places. I changed the naming convention to default to asIs at some point without realizing I had to fix this all over the DSL. Fixed by #28

Thanks but why was the default switched to "asIs"? Look at the many places where you need to override it and potential future extensions will have to be careful to override as well. Why not default to snake case when this is the default for OpenSearch/ElasticSearch?


Permission does not exist, when trying add/edit BREAD

Laravel Version: 5.5.*
Voyager Version: 1.1
PHP Version: 7.0
Database Driver & Version:

Description:
After Fresh installation of Voyager with dummy data, when i try to add a new BREAD or edit any existing BREAD, like categories, pages etc.  i keep getting the error below even though the user is the admin and all the permissions are checked

Steps To Reproduce:

Installation of Voyager
Try creating a BREAD for an existing table
Got the error above

Upgrading to 1.1 requires upgrading steps, but I still get this error after doing so.

There is one very important step here which get overlooked sometimes @chaossme but @ArditMata says its a fresh install.

@ArditMata, please verify that your database includes a permissions table with a row for browse_bread.  If not, please make sure you've followed the install steps completely (including migrations/seeders).
VoyagerBreadController:114 is in the edit method(ref), which means the BREAD was created and you're trying to access the edit form.

@fletch3555 I created a new project and and strictly followed the installation steps. The previous error is fixed, but now i cannot delete any BREAD. The delete button just doesn't seem to do anything

This is known and already fixed.
How did you fix it now?

same problem.
v1.1.1

@samanmohamadi Fresh install or upgrade?
If upgrade, please follow my comment

removing all old migrations and a clean install fixed the issue.

Still getting this error, went through all the steps mentioned above

Got it working finally!
Needed at add edit the PermissionsTableSeeder and add in
$keys = [
'browse_admin',
'browse_bread',
'browse_database',
'browse_media',
'browse_compass',
];

@tangod0wn that's mentioned in the upgrade guide


”Slătioara”, ”Coasta Cerbului”, ”Gorunești” are listed in the drop-down menu

Description:
When user writes ”Slătioara”, ”Coasta Cerbului”, ”Gorunești” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Slătioara"

Expected results
The location appears in the drop-down menu as "Sat Slătioara, VÂLCEA (Slătioara)".

Step 3
Write in the search bar "Coasta Cerbului"

Expected results
The location appears in the drop-down menu as "Sat Coasta Cerbului, VÂLCEA (Slătioara)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Gorunești"

Expected results
The location appears in the drop-down menu as "Sat Gorunești, VÂLCEA (Slătioara)".


Add toggle logic to the popover

#80 might be as simple as this. Heads up that I haven't tested this!
This also happens to be my first time working in Swift: it might be totally wrong. 😄

Thanks a lot! Will include it in the next release


Allow collapsing & expanding individual comment replies, programmatically and through the UI

Our users would like the ability to collapse replies from earlier code review rounds, to save vertical space taken up by the comment thread widget and focus on the latest replies. While this is less of an issue for short, one-line replies, more complex replies, e.g. containing example code snippets etc. can quickly take up significant parts of the screen. When these replies happened earlier in the conversation, their contents will generally be known to participants or not even be relevant / up-to-date anymore, but they still take up space. Note that addressing #150824 by setting a maximum width for comments to help readability (wrapping e.g. after 100 characters or similar) would likely increase the number of situations in which this becomes relevant.
We think this would probably take the form of a collapsibleState property on vscode.Comment similar to the existing one on vscode.CommentThread, keeping in mind that this should not only affect initial expansion state of the comment, see #167253.
In the UI, expanding and collapsing could be available through a rotating expansion chevron.

If folks still want this then I will re-open!


Various Small UI Fixes

###v1

This PR gives a couple of minor UI fixes or improvements outlined:
Mirage handler for export policy was created
Lists edit errors for direct JSON advanced edit mode for compliance policy in the action bar and prevents saving
Adds tooltip for purge key and handler on mid tier to provide wikilink in configs
Right now our backend only has one validation for dataset health which causes it to be confusing for people because the breakdown is just 100%. Removing graph component for situations where # of validations <= 1.

ember test results:
1..499
# tests 499
# pass  492
# skip  7
# fail  0

# ok

Codecov Report

Merging #1397 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1397      +/-   ##
==========================================
- Coverage   60.67%   60.64%   -0.03%
==========================================
  Files         181      181
  Lines        3987     3987
  Branches      914      914
==========================================
- Hits         2419     2418       -1
- Misses       1568     1569       +1

Impacted Files
Coverage Δ

wherehows-web/app/constants/datasets/compliance.ts
100% <100%> (ø)
:arrow_up:

wherehows-web/app/components/purge-policy.ts
96.29% <0%> (-3.71%)
:arrow_down:

Continue to review full report at Codecov.

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

Coverage remained the same at 22.752% when pulling bee6e317101f8ccb5147af0a729319fb21fff371 on cptran777:advanced-compliance-edit-fix into cd94e5af6fc574323f3f25ed0c2a490cd5f8aaae on linkedin:master.


Sphere Online Judge (SPOJ) Website Clone

Project title
Clone of Sphere Online Judge (SPOJ) Website
Description
SPOJ is a very famous coding platform where we can learn different types of coding problems
Stack:

[x] Html
[x] CSS
[x] JavaScript

/assign


Dev

The codecov check failed because the %coverage decreased, but that's just since I just decreased the number of covered lines, I didn't increase the number of uncovered lines