devices about react-native run-android command

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
I connect my PC with usb, and I also have an Android simulator on my computer. Then I run react-native run-android command. Sometimes programs are on  Android simulator and sometimes on my phone.
React Native version:
System:
OS: Windows 10 10.0.18362
CPU: (4) x64 Intel(R) Core(TM) i5-6300HQ CPU @ 2.30GHz
Memory: 7.94 GB / 15.80 GB
Binaries:
Node: 10.16.3 - C:\Program Files\nodejs\node.EXE
Yarn: 1.19.1 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.9.0 - C:\Program Files\nodejs\npm.CMD
Watchman: Not Found
SDKs:
Android SDK:
API Levels: 28, 29
Build Tools: 28.0.3, 29.0.2
System Images: android-28 | Google Play Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: Version  <IP_ADDRESS> AI-191.8<IP_ADDRESS>00203
Languages:
Java: 1.8.0_221
Python: 3.7.4
npmPackages:
@react-native-community/cli: Not Found
react: 16.11.0 => 16.11.0
react-native: 0.62.2 => 0.62.2
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce

connect my phone and PC with usb
run react-native run-android command

Expected Results
I want to run program on my phone not Android simulator on my PC, and I don't want to uninstall Android simulator. Is there any option to select a device
Snack, code example, screenshot, or link to a repository:
Please provide a Snack (https://snack.expo.io/), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
You may provide a screenshot of the application if you think it is relevant to your bug report.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

this related to Android Debug Bridge (aka adb).
try this :
$ adb devices

this command should list of all connect devices and simulator such as :
List of devices attached
emulator-5554 device
7f1c864e  device

for specify  device :
adb -s 7f1c864e shell

see also this
use npx react-native run-android  insted of  react-native run-android

The React Native CLI supports running your app on a specific emulator or physical device. From the instructions,
--deviceId [string]
builds your app and starts it on a specific device/simulator with the given device id (listed by running "adb devices" on the command line).

...you should be able to run react-native run-android --deviceId [string] where [string] is the id of your device as obtained by running adb devices.

Ok guys. I want to run this command on my Android phone that i use. Will my phone data get wiped? how does it work ?


x2goclient: Use nx-libs instead of nxproxy

nxproxy was removed in favour of nx-libs, causing an eval failure in gcofborg (and possibly Hydra as well?) This should probably be reverted (or at least the nxproxy name should be removed instead of using lib.warn,) but this package also needs to refer to the nx-libs instead.
No real testing has been done here aside from running the executable -- I don't know the software.
See also #55723
Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Ah, it's only because the name only exists as an alias now, but this callPackage still depended on it. I think this should suffice to fix it. :)

ofborg checks evaluation without aliases, either keep the warning in all-packages.nix or fix the remaining references.

Oh ignore me, this was the fix. 😄

@srhb thanks a lot for fixing that!


Connection stops working due to failed MTU discovery

Hi,
did you have or heard of any issues with connections no longer working at some point? Some of our testers have the issue that at some point during the connection they no longer receive any data on their end. The other direction, i.e. the data the client sends to the server, still works though.
So far I nailed it down to the LiteNetLib part by putting debug logging to the place where we call NetPeer.Send and on the other end to the NetworkReceiveEvent. As long as data is transmitted normally we get the log output on both ends just fine with the same size of packages but as soon as it stops working only the server side send logs show, the client side never fires the NetworkReceiveEvent.
We only use SendOptions.ReliableOrdered for all our data. Also watching the NetworkErrorEvent but that's not fired either.
Was wondering... when not calling PollEvents for a moment (like 20 seconds) could that kill the connection? I suppose it should handle all the connection keep alive stuff and data reception on its own in the background threads and PollEvents is just to actually get the events handled by the caller?
Of course could the issue still be caused by our software but I'm currently out of ideas.
Library version: release version (<IP_ADDRESS>)
Framework: Unity
OS: Windows
Regards,
Chris

Well, I don't have any idea if the source of the issue is the same, but from a high level view this is not about disconnects at all :)
There's neither a manual disconnect happening nor do I get any kind of automatic disconnects like from timeouts. It's simply stopping to receive the packets on the client side while the other direction on the same LiteNetLib connection still works fine.

@Alloc86

Was wondering... when not calling PollEvents for a moment (like 20 seconds) could that kill the connection?

No)

I suppose it should handle all the connection keep alive stuff and data reception on its own in the background threads and PollEvents is just to actually get the events handled by the caller?

You are right.
Well in <IP_ADDRESS> some code parts (and architecture) works bad. I think there is some error in reliable channel (and possibly hard to detect). Can you try please 0.8 branch version? I rewrited some bad parts of reliable channel in 0.8. I will make 0.8 release soon. And if all works correct and you want to stay on 0.7.x branch i can backport this fixes into 0.7 branch.

Which Unity version are you running on? It might be a bug in the Unity if you are using .NET 4.6 (it's still unstable and in some cases Socket.ReceiveFrom may fail and stop receiving any data. Switching to .NET 3.5 should solve this problem, just as a temporary solution.

@nxrighthere I had that problem with #228 on .NET Core 2.1 both client and server.

Unity 2018.2.1 with the .NET 3.5 framework.
I'll give 0.8 a try and report back :)

Yes, this is definitely a bug in the library.

@Alloc86 thanks!

Unfortunately updating to the latest master didn't solve the issue. Also added a debug log to the NetManager.ReceiveFromPeer method to make sure it's not us for some reason stopping to call PollEvents. Not seeing the incoming packets there either though.
Will get a Wireshark snapshot today and see if the packets even arrive at his system.
(Btw, when I did the same thing on my end (working connection) to test the Wireshark setup, I noticed that I receive quite a lot of the packets two times)

@Alloc86 seems that reliable channel become broken at some moment. Maybe because of some invalid data (maybe some packets corrupted and without checksum).

Yeah, now I suppose the question is why? ;)

Oh, btw, I see didn't specify that clearly in the initial post: The point that (at least this one user I'm testing with) gets the issue is always the same, i.e. it's always the same last successfully received package.
And I can connect to the same server just fine and receive all the data, so it does not seem to be a generic issue with fragmented messages or anything like that, but of course it could still be related to the specific binary content of the messages or something like that as that varies a little bit between different users.

PS: I can give you access to the current game builds that show the issue, but no idea if that would help you without our source to actually debug it.

Hm, the captured data isn't as readable as I hoped it was. Shows a lot of "Fragmented IP Protocol" entries (even on the sender side ... would've assumed that this should not be the case with LiteNetLib sending packets smaller than the MTU?) which seem to be part of the actual UDP traffic so it's harder to read the data in there.

@Alloc86 have you tried LiteNetLib debug logging (compile the client with DEBUG, DEBUG_MESSAGES)? Or try to examine the state of ReliableChannel after the problem - how many _receivedPackets, _pendingPackets, _outgoingAcks it has, etc.

@xmedeko Issue with looking at stuff is that I have to log it out on the client at appropriate events as I can't debug on the tester's machine (and none of our devs had this issue yet). But I will definitely add the script defines to the next build :)

@Alloc86 this defines (DEBUG_MESSAGES) reduces performance alot. Better replace some places with DebugWriteForce. I will add some checks to reliable channel to see if it become broken at some point.

Also, if you may log some state by a Timer tick only, e.g. log a state of ReliableChannel in 1, or 5 min interval only. And instruct your testers to wait after the problem occurs and then send you the logs.
@RevenantX the logging performance is the problem of #227. If you have proper designed logging, you may log warnings e.g. if the channel buffer is full etc. Then you may switch on logging just for Warn and Error levels without big performance penalty.

@Alloc86 @xmedeko i added discord chat https://discord.gg/FATFPdy

Seems like RevenantX was correct, it was an MTU issue that he fixed in a commit today. Thanks a lot :)

@Alloc86 thanks for debugging)

Seems like it's not completely ok right now. With the latest code we get the issue again. Have disabled the MTUs 1500 and 2304 again for the next build to try again.

@Alloc86 what platform?

Unity 2018.2.1, .NET 3.5, Windows 64 bit players on both sides.

I am not so strong in networking, but is it possible to detect the drop by fragmentation by some ICMP? https://en.wikipedia.org/wiki/Path_MTU_Discovery

Well, seems like it works if the MTU is restricted to 1492 in our tests. Of course there could be situations where even that fails. Seems like the MTU detection code isn't fully working yet?

@Alloc86 MTU less than 1492 is very rare (on some very old hardware)

@RevenantX nope, just found out that Google Cloud uses MTU 1460 and other ISP may do it as well.

Yeah, I think VPNs also lower the MTU (at least pretty sure they did in the past) as they basically have to encapsulate complete UDP packets into their own UDP packets.

@xmedeko @Alloc86 when DontFragment enabled packets that more than MTU should not be received on other side. And client (and or server) must get socket error - MessageTooLong. Seems that in some rare cases packets will be fragmented anyway (in strange way).

And can you catch that socket error and lower the MTU?

@xmedeko MTU is minimal on start and increases by one step in PossibleMtu in first seconds. And if it receive that error - it stops.

I mean can you catch this error when sending data packets? Then to lower the MTU or trigger the MTU rediscovery?

@xmedeko i can catch this errors (and there already code for this). But in @Alloc86 case there is no any errors. It just corrupts packet.

@Alloc86 any news?

MTU issues? Wowsers that brings me back. This was a known issue inside linksys and dlink routers. What you might be seeing is a drop from a router that loses the packet, creating a hang up or losing part of the data would be my guess.
Stick at 1492 as you have mentioned, possibly 1490.

@RevenantX Sorry, just now saw your message when looking if there were any tickets for connection timeouts.
We've been running with the possible MTU sizes array only containing 576 and 1400 to be on the safe side. Think there's been no real issues for the most part.
Though we still do get timeouts that I can see no reason for, but I don't think it's the MTU issue with those values. Most of them seem to happen at random points during gameplay and (at least in some situations, not sure if it applies to all of them) it instantly kicks all connected users at that point. No issues reconnecting afterwards.

@Alloc86 well this all bugs must be fixed in 0.8.3 (from our dialogue on discord we determined that you used old unstable version) so i close this. Reopen if you find this bugs in 0.8.3


fix docs.rs build

I did this wrong, which means the docs.rs build couldn't complete.
https://docs.rs/crate/fundsp/0.5.0/builds

technically this is not needed as there are no features anymore, but I guess this is future proofing

Thanks!


Feature/documentation pile up

Mainly documentation that explains queue pile-ups and a few minor improvements to previous documentation.

@rtholmes Thanks for the feedback. Merging now.


Do not truncate opening day

Questions
Answers

Description?
Do not truncate opening day

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes #https://github.com/PrestaShop/PrestaShop/issues/25138

Sponsor company

How to test?

Hi @Hlavtox !
I checked the code, indeed it can be merged without QA. Thanks for the PR!

Same here, thanks @Robin-Fischer-PS!


gevent compatibility

Hi, I want to make grpc client and server code to work with gevent, is grpc compatible with gevent?

@zj8487 did you get gRPC(python) working with gevent.monkeypatch? Could you please share the work-around.

@vinays
it is sad, it is not work with gevent.
i do it as follow:
a thread(not monkey patched) to deal with the grpc and send or receive message from a message queue which was listen or loop in a gevent to process the request or response.

grpc make async io well， why do you want to make grpc client and server code to work with gevent? they do the same thing.

Thanks @zj8487
@mayflaver in our case 'grpc' and 'gevent' are transitive dependency of gcloud-python and gunicorn respectively in our legacy app. If I was writing an app from scratch I would agree with you, but in this case these dependencies are not enforced by me. :(
To best of my knowledge--gunicorn with gevent as worker_class monkey patches a lot of standard library functions including threading. This conflicts with the use of "futures" in grpc and hangs the process.
Since, gunicorn(w/ gevent) is quite commonly used in the industry and anyone who wanna use gcloud-python(or other grpc based libs) in their app will likely run in to the same issue. I am open to any suggestions that might help solve this issue without changing the dependencies too much. Maybe gRPC and gunicorn teams should talk and figure out a solution?

Reopening as this seems to warrant investigating.

@soltanmm any resolution on this problem? It will be really help if we can get a rough timeline for it.

@vinays There's no timeline for it at present (and without in-depth gevent knowledge we're not exactly leaping into action on it), but as there appears to be mounting community pressure I guess I'll take a quick look tomorrow. Feel free to proactively provide a minimal reproduction; it might speed things along.

@vinays We have a hard time justifying interoperability with a library that seems to be unable to work with standard library elements in Python 3 because it literally changes the standard library, and incompletely so.
That said, it's worth documenting this, and I don't know much about gunicorn. What's wrong with the other options listed here for your use case?

@vinays Actually, this issue comment might be of relevance to you: https://github.com/benoitc/gunicorn/issues/927#issuecomment-102176547
It appears that this problem arises with Django as well and what the gunicorn folks end up doing might be the fix you need.

Thanks @soltanmm for looking into the issue.
@mickgarvey @grourk might have more insight into why we did not choose other options in the list. From what I know, since our app is IO bound 'sync' worker does not scale well because of blocking IO.
@soltanmm did your investigation conclude a root cause of the issue? Do you know what part of standard library patching was causing the hang in Python 2.7? Would excluding 'thread' from patch_all, fix the issue? I can test that myself, but I am looking for any side-effects you might know and most likely won't surface itself in a simple test. Any other information from your investigation might also help.

@vinays I'll have to guiltily admit that after reading about what gevent monkeypatching actually is and checking the docs on known hangs interacting with library elements we appear to use, I face-palmed and stopped before attempting a reproduction. I'll take a look again on Monday and get some first-hand results if that'll allow everyone to sleep a little better.
I have a strong feeling it's just going to be the usual, "Green-threads cannot be cooperative if they block on something that isn't cooperative," sort of deal, though (where 'something that isn't cooperative' could be anything the gevent folks missed while monkeypatching [which I have a feeling includes concurrent.futures] and every blocking gRPC core operation).

It's basically that. gRPC Python uses multiple threads for a server (e.g. one for the server receiving messages, one to pass the call data along to the application). These trip over each other when they don't cooperatively release polling to gevent (which at least one of them can't do because it's in gRPC core code and the core is obviously not based on gevent).
Short of getting in-depth knowledge of gevent (and I'm not a gevent dev, so, nope), I can't come up with much more than saying, "Excluding thread from monkey-patching seems to work."
I'm not going to close this as it's possible that there exists a way to force the use of standard Python threads despite monkey patching where they are necessary, or that some other hacky-solution-against-a-hacky-solution exists, but we will not attempt to support gevent's monkeypatching for the near term.

I'm still coming up to speed on what's being asked here, but from what I've gathered so far, I suspect it will be a lot: one of the core design assumptions of gRPC Python is that a reasonable number of threads will always be available to be devoted to blocking operations such as C extension calls and calls into application code. Rolling back that assumption and moving forward with a different structure are going to be substantial work, if they're something that we ever do at all.
I feel like I'm still missing quite a bit. What's the minimum sample program you could write that you would expect to work but that doesn't?

@soltanmm thanks for the explanation, it resonates with my understanding.
@nathanielmanistaatgoogle if your questions was pointed at me, I can write a sample script later today to reproduce the problem.

@vinays: yes, a code sample would be delightful - something that is enough to illustrate a use of gRPC Python, how you expected gRPC Python to behave, and how gRPC Python actually behaved.

@nathanielmanistaatgoogle here is some sample code to illustrate the issue. Let me know if you have any questions.

Just another use-case we have been running into. We are using graphql (https://github.com/graphql-python/graphene) as our public API and grpc for internal communication.
The graphql library supports running resolving requests in parallel using gevent but the problem is, that any grpc calls we are making (in different resolve methods) are run serially, which now is the cause of quite a few performance issues.
I have just started to investigate, to find any potential solutions or workarounds, but good to see there are other people with similar issues.

@nathanielmanistaatgoogle Thoughts on gevent/gevent#786? We could leave an unsupported (private, but documented) hook for the underlying thread pool executor somewhere in the gRPC implementation and allow users to swap it out post-import/pre-absolutely-anything-else if they want to at their own risk.

Better yet: we could do that and attach warnings (like the deprecation warnings) to it in terminal emboldened firetruck-red all-caps font that the users have entered monkey territory and shouldn't be surprised if their hats get stolen and everything hurts twice as much.

Thank you @jamadden for your investigation.
Use of the same thread, with program control passing from application to gRPC Python to the gRPC Core and then to a blocking system call is an explicit part of gRPC’s design, is the basis for many of the performance improvements that we’ll be making in the next few months, and would be very hard to change. In particular major alterations would have to be made to the Core to enable its completion queues to be used in the manner gevent would want to use them. I’ve spoken with @ctiller about it and this work may be scheduled engineering effort if it is forecast to be valuable enough to be prioritized above other goals.
There may be a certain recipe for monkeypatching that happens to work today but we wouldn’t be able to offer proper support (with test coverage and a promise not to break in the future) for anything based around selectively patching thread pools used internally within gRPC Python. In particular we have near-term plans to scale back how widely we use thread pools. If you want today to do something based on selectively patching it may work fantastically for you and then break in a future release. I would celebrate your bravery and wish you fun.

Retitled this since the investigation is over.
Compatibility with gevent might happen, and if it ever does it will be following changes to the C Core.

@nathanielmanistaatgoogle
+1 for Compatibility with gevent.
one more sugesstion(may be not suit talk here.): really hope pure python version or non threads version.
grpc core with number of threads model that make it not convenient to work with the other frame like django uwsgi, etc. because it is hard to control the grpc client's life cycle.
until now i am searching how to use grpc python client in django with uwsgi make it can support reload, and make the server can exit correctly.   :(

A relevant thread on the grpc.io mailing list.

It is possible to get this working using gevent 1.2a (aka GitHub master.)
Use this concurrent.futures compatible ThreadPoolExecutor implementation from gevent http://www.gevent.org/gevent.threadpool.html#gevent.threadpool.ThreadPoolExecutor
And set gevent.monkey.patch_all(thread=False).
At least it doesn't hang anymore when you do this :)

If I understand the documentation correctly, you don't need to pass thread=False.

A version of concurrent.futures.ThreadPoolExecutor that always uses native threads, even when threading is monkey-patched.

Just kicked off a community process for addressing this enhancement request; please participate!

@nathanielmanistaatgoogle @soltanmm With gevent,
File "/Users/srinath.gs/.virtualenvs/vajra/lib/python2.7/site-packages/grpc/_channel.py", line 670, in channel_spin
    event = state.completion_queue.poll()
  File "src/python/grpcio/grpc/_cython/_cygrpc/completion_queue.pyx.pxi", line 105, in grpc._cython.cygrpc.CompletionQueue.poll (src/python/grpcio/grpc/_cython/cygrpc.c:9612)
KeyboardInterrupt

I got a hang on a service that returns a stream as a response. The possible culprit is a busy while loop in the completion_queue.poll function.
.proto definition
  rpc Read(ReadFileRequest) returns (stream ReadFileResponse);

  def read(self, chunk=1024):
        rfreq = file_pb2.ReadFileRequest(file=self.f, chunk=chunk)
        for resp in self.stub.Read(rfreq, self.timeout):
            print resp

  print "reading"
  f.read()

execution does not get past the print message

@nathanielmanistaatgoogle Thanks for the work. In what release version or milestone is this filed under please?

@charyorde: our current thinking is that we may be able to address it before the end of the calendar year.

Thank you @nathanielmanistaatgoogle

gevent v1.2.2 with grpcio==1.4.0 hangs in RawGreenlet switch
I did not look into this issue, just rollback grpcio to v1.1.3 and it worked fine.
think soneone should know this.

@scorpioWalker: I think this is something you should create a new issue about. This one issue here is more integrating grpc with gevent. That's different from simply being able to run next to it.

Google Datastore with Gunicorn blocks forever in app engine flexible.
I am not sure if it's because of this issue.
gevent==1.2.2
google-api-core==0.1.1
google-auth==1.2.1
google-cloud-core==0.28.0
google-cloud-datastore==1.4.0
google-gax==0.15.16
google-resumable-media==0.3.1
googleapis-common-protos==1.5.3
greenlet==0.4.12
grpcio==1.7.0
gunicorn==19.7.1

Hello! Would love to hear the status of this!

We could use this as well, spawning processes is a waste of memory and cpu for heavy-I/O bound tasks...

hi @kpayson64 & devs,
I understand this issue is difficult or doesn't get any priority but would you pls communicate any concrete progress on this?
The issue is open for +2 years. It's totally fine if this doesn't get solved/delivered but if that's the case can we pls close this issue so ppl can move on to an alternative?

@kpayson64 I saw in your repo that you are still working on this. Thanks! Does it seem to work?

@smetj
@codesuki
Sorry about the delay.  There were a few high priority issues the popped up that I needed to address, and I was on vacation for a week so this got pushed back a bit.
This is very much at the top of my stack right now.  I'm currently tackling some lingering test failures, but the bulk of this seems to be working.

Sounds great! Thanks :) I hope we can get our gRPC working with locust soon.

Super excited this is moving forward! Thanks for the feedback ...
Can't wait starting to use Google cloud pubsub with Gevent + Python!

is aiohttp a viable alternative?

Same! I am excited to try this out.

Would be great to get https://github.com/grpc/grpc/pull/14561 merged in

With #14561 in, you should now be able to use gevent.
Note that you have to explicitly monkey patch Python, as well as enabling the gevent loop for Python (Before doing anything with gRPC)
from gevent import monkey
monkey.patch_all()

import grpc.experimental.gevent as grpc_gevent
grpc_gevent.init_gevent()

This will be included in the 1.11 release.
I'm closing this master issue, and separate issues can be created with any issues encountered using gevent.

For those who want asyncio support, can #7910 be reopened or should we open a new issue?

Or is #6046 that?

Finally. Testing away!

@kpayson64 thanks, great job!

@mitar
Feel free to open a new issue.  I'd expect that we need to add some new surface APIs for asyncio support, so its probably a good idea to have a separate issue.

I opened #14876. Based on existing gist (linked in the issue) it seems this could be easily done.

The official documentation needs to be corrected. It links to this issue and says:

Warning: Because using the gevent worker is known to cause issues with gRPC, we recommend that you do NOT use gevent workers if you're using the Google Cloud Client Library, which uses gRPC. Using gevent workers with the Google Cloud Client Library will cause requests to hang.

https://cloud.google.com/appengine/docs/flexible/python/runtime


tests: reorganize

About this change - What it does

tests: replace constants with fixtures

Why this way
CONSTANT_TEST_RSA_PUBLIC_KEY and CONSTANT_TEST_RSA_PRIVATE_KEY are
not special enough to make them constants and require separate file for
them. Replace them with fixtures for consistency.

Codecov Report

Merging #132 (b3600b7) into main (c3e5427) will increase coverage by 1.12%.
Report is 16 commits behind head on main.
The diff coverage is 82.81%.

@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
+ Coverage   65.25%   66.37%   +1.12%
==========================================
  Files          31       31
  Lines        3606     3810     +204
==========================================
+ Hits         2353     2529     +176
- Misses       1253     1281      +28

Files Changed
Coverage Δ

rohmu/object_storage/local.py
86.20% <77.41%> (-3.33%)
:arrow_down:

rohmu/object_storage/s3.py
65.67% <78.18%> (+1.54%)
:arrow_up:

rohmu/util.py
74.07% <86.02%> (+25.23%)
:arrow_up:

rohmu/errors.py
100.00% <100.00%> (ø)

rohmu/object_storage/base.py
78.57% <100.00%> (+2.38%)
:arrow_up:

rohmu/typing.py
61.53% <100.00%> (+0.82%)
:arrow_up:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Modified SDK for Quickstart Playground - Chat Style

Summary of the pull request
Introduces SDK changes needed for the Chat Style integration of Quickstart Playground
References and relevant issues
Detailed description of the pull request / Additional comments

Does not modify the existing Quickstart playground provider and creates a new one that we will then modify
Adds QuickStartChatStyleResult response object

Validation steps performed
PR checklist

[ ] Closes #xxx
[ ] Tests added/passed
[ ] Documentation updated

Do you have a link to the API review?

Do you have a link to the API review?

Good question! It is in the works. I had just sent mail out to Hart about this


[change request] hosts lists arrangement

Steven @StevenBlack is it possible(on your end) when all the hosts lists are arranged into one a line to be inserted at the beginning and the end of each individual lists.For example:
# KADhosts Start
domain.com
domain.com
domain.com
# KADhosts End
It would really help me remove things that i don't need(as we discussed yesterday with you),i managed to do it now but if you change the arrangement it will be a mess on my end
P.S. If it's too much trouble maybe just something to indicate the end will be sufficient also.

Just let me understand ... You do not use the whitelist system ?

@funilrys  i use only the ready hosts file from here and fetching it to my router's dnsmasq. If i have to do any modifications on the file i'm doing it with script loaded in my router,it's like portable Linux OS which has limited editing tools.My request here is custom but i'm pretty sure it would benefit others Linux users as well.Hope this answers your question.

Okay understood only have to code that then :joy:

@funilrys  see #634 he gave some examples there,whatever is the easiest,just something to indicate where the lists starts and where it ends and to be permanent so i don't have to worry that if the arrangement changes i'd wipe out half of the entries there 😦

Understood :+1:

@funilrys  see #641 as to why(just more info).Also checking on progress,not rushing or anything,juuust checking 😉


fix capz example to make use of cluster-values

What does this PR do?
(Please set a descriptive PR title. Use this space for additional explanations.)
What does it look like?
(If this is more than a textual change, a screenshot can help in some cases to speed up the review process.)
Any background context you can provide?
with CAPZ based clusters with the new kube-proxy-less mode, the correct baseDomain is required. This value is stored in the cluster-values configmap and needs to be referenced. The change is already implemented in kubectl-gs
What is needed from the reviewers?
Have you maintained the front matter?
(Please bump the last_review_date in case this qualifies as a review for an entire page. Provide user_questions which should be answered in the page. Provide a meaningful description.)

I suggest we close this PR in favor of https://github.com/giantswarm/docs/pull/1836, since that one deletes the article altogether in order to merge all provider instructions into one article.


Show function parameter names on autocompletion

I don't really know if this issue belongs here, but since I like a lot how WebStorm autocompletes code, I think it would be a great idea to implement a code hint that shows functions parameter names. Here's a gif example showing how WebStorm handles this:

This feature helps a lot when I don't know exactly how many arguments a function is supposed to handle.

This issue was moved to atom/autocomplete-plus#455


support relative package paths

go version
go version go1.11.2 darwin/amd64

when i run gocode -s -debug
2018/11/23 01:23:44 gbimporter: no package found for ./db: can't find import: "/Users/xxxx/go/go_fetch-back-car/db"
2018/11/23 01:23:44 gbimporter: no package found for ./router: can't find import: "/Users/xxxx/go/go_fetch-back-car/router"

Q:
custom relative path package doesn't support, but the import package is true, and path /Users/xxxx/go/go_fetch-back-car/router is exist
In my project
main.go
import (
	"fmt"

	"./db"
	"./router"
	"github.com/gin-gonic/gin"
)

tree
.
├── README.md
├── ctx
├── db
├── errorer
├── handle
├── main.go
├── router
├── test_test.go
└── utils

following your tree, it looks like all the files are in one package. this means that you do not have to be import them. furthermore, I'm not sure if "./somepkg" would work at all.

@frzifus
db files are new package, isn't package main, so need import
package db

import (
	"database/sql"
	_ "github.com/go-sql-driver/mysql"
	"log"
)

var SqlDB *sql.DB

@frzifus oh, goland support it , but I like vscode, so I hope it can be solve

Maybe this post will help you.

@frzifus I try it, but the result is the same, I can import it, but don't autocompletion

@shaodahong 我之前和你有过同样的问题，切换nsf版本的gocode 并设置gocode autobuild 解决。我的go version1.10.4 并没有尝试其他的go版本 gocode问题

google translate tells me that... @MagnusHuan
 I had the same problem with you before, switching the nsf version of gocode and setting gocode autobuild to solve. My go version 1.10.4 did not try other go versions gocode problem

now I resolve it with go mod

google translate tells me that... @MagnusHuan
 I had the same problem with you before, switching the nsf version of gocode and setting gocode autobuild to solve. My go version 1.10.4 did not try other go versions gocode problem
btw i don't see any relative path included there.

my fault, it's relative path

Closing this issue. For a better development experience with Go modules, I would suggest using gopls.


md5sum on download page don't seem to match

when downloaded through the link in README.md or from the one on the downloads page. At least for deb.

Where are you getting the md5 from? I don't recall publishing any md5sums

oh, on the downloads page, now I see, you can open an issue on the docs pages here: https://github.com/influxdb/influxdb.com


Picking

It should be possible to do CesiumJS-style picking of 3D Tiles. Applications should be able to determine which tileset, which tile, and which feature in the tile were picked. They should also be able to access properties of each feature.

Fixed in #545.


Fixed deprecated body.scroll

document.body.scrollTop and document.body.scrollLeft are deprecated and have been now partially
removed from Chrome. They always return 0. (StacksOverflow - ScrollTop deprecated)
document.documentElement.scrollTop contains the correct value.
Closes #11

Thanks @keombre. Fix works! Would be nice if @iaarchiver merge this PR and publish the extension.
Meanwhile for others, attaching the MinimalScrollbar.zip of the PR which you can install manually.
Extract the zip and follow the instructions on how to update the plugin (make sure you remove the old version before you try)

Nice to hear, that the plugin works, BUT!! updating any plugin in Chrome from simple .ZIP package doesn't work the same as a native extension.
IF you want to add the updated extension (from zip, on Windows) you need to repack it to .CRX, install GPO tools for Chrome, add extension and finally white-list it, to enable it (without popups).
I understand, why devs at Google made it that way, however, it makes our job (as independent devs) unimaginably harder :(

Unfortunately, it appears that the original developer has gone the way of the dodo. :cry:
The project is MIT-licensed, so if anyone's up for taking over and maintaining this project, I think you're perfectly fine to do so. Personally, I'm going to stop using this extension and use "Overlay Scrollbars" in chrome://flags. It looks pretty similar and means I won't have to use an abandoned extension. c:

Will this be merged @iaarchiver? 🤔 Or is there someone knowledgeable to fork and release it, maybe @keombre?

I've just learned about chrome://flags/#overlay-scrollbars, thanks @Snuggle so much! I've switched immediately, therefore I won't maintain this, sorry @LeBaux.

No problem! ❤
It's almost identical and is supported within chrome. c:

@keombre I have come from the future, looking for this. No doubt this is a cool project. It needs to be updated, as Overlay Scrollbars are officially dead as of November 2020. Please update this app.


Where I download specific version of Pyoncord?

Like I want 223.5 version apk.

Download Bunny Manager and unlock dev menu (on about menu tap 10 times manager's version) then go to dev menu and type "223205" on discord version field

Open bunny manager settings
Go to about
Tap the version number a few times
Go back
Open developer options
Set discord version to 223205 for 223.05.  If you intend to use the old ui, try 215024 (215.24 stable) instead


Support deeplinking to pipelines, search, etc

Each view in the application should be shareable via a link (to a certain extend, excluding the things that are "unique" to your own session, such as favourites (#9), one-clicks (#10), etc).
We can use the History API for this.
This would be the list of interesting deeplinks:

[x] /pipelines/<pipeline_id> #/task/<task_id> – direct modal window
[ ] #/task/<task_id>?field1=<value>&... – modal window with pre-filled fields
[x] ?search=<query> – direct search result
[ ] ?filter=<regular|favourites|one-click> – filtered lists (see #11)
[ ] ?search=<query>&filter=... – combination of the last two

Deeplinking to a pipeline should never automatically run the pipeline, even if all fields are pre-filled.
The URL should instantly update as you type in form fields (e.g. search or pipeline variable fields), so that you can instantly copy/paste the links without any extra work.

I've changed the above list into a checklist. This is now partially implemented, and the plumbing is there to implement the rest soon as well.

Deep linking to search queries is implemented in 038cb8adf85411394c4035b942a253132dcca009.

The query parameter updates live as you type, so they can be easily shared with others.

Deep linking to tasks with pre-filled variable values is implemented in 2e70f033f8c9b54b71a2db42f3b04d320d8dd049.

Closing this. The infrastructure is there, and I added the last one (deep-linking to favourites, etc) in their respective issues.


ClrProxy documentation

The documentation on ClrProxy was outdated but I was able to figure out how use it. The following is an example.
using YantraJS.Core;
using YantraJS.Core.Clr;

var synchronizationContext = new SynchronizationContext();
var context = new JSContext(synchronizationContext);

var calculator = new Calculator();
context["calculator"] = ClrProxy.From(calculator);

var a0 = await context.ExecuteAsync("calculator.add(1,1)");                 // a0 = {2} JSNumber
var a1 = await context.ExecuteAsync("await calculator.addAsync(1,1);");     // a1 = {2} JSNumber

public class Calculator
{
    public int Add(int x, int y)
    {
        return x + y;
    }

    public Task<int> AddAsync(int x, int y)
    {
        return Task.FromResult(x + y);
    }
}

@adiamante ClrProxy might be removed or will be changed, the correct way to supply Clr object would be to implement IJavaScriptObject or inheirt JavaScriptObject class. You can control what to export by using JSExport attribute. ClrProxy exports everything. I will be updating documentation soon.
Also using signature JSValue Method(in Arguments a) will be faster compared to using Clr types as arguments as there will be extra checks to convert JSValue to clr types. You can check this unit test on how to use it.
https://github.com/yantrajs/yantra/blob/main/YantraJS.Core.Tests/ClrObjects/CustomObject.cs

@ackava Thanks. That was informative


Ipcc color update

Please confirm that this PR has done the following:

[x] Tests Added
[ ] Documentation Added
[x] Description in RELEASE_NOTES.md Added

Description of PR
The IPCC TSU has updated their preferred SSP/RCP color pallete for AR6. It now exists in a public repo here. This PR updates our color mappings to match this and includes new "SSP only" colors as well.
Probably best for at least @znicholls to take a look and confirm one or two of the translations with their own info. @danielhuppmann happy for you to give the a-ok as well if you want =)

I considered putting in a pr, but thought that this happens so
infrequently, it may just not be worth it (other than having a single point
of reference)
On Sun, Dec 29, 2019, 15:13 Zeb Nicholls<EMAIL_ADDRESS>wrote:

@znicholls approved this pull request.
lgtm, I had this hope that the colours would be made machine readable or
put in a package (IPCC-WG1/colormaps#2
https://github.com/IPCC-WG1/colormaps/issues/2) but doesn't seem to be
the case yet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IAMconsortium/pyam/pull/314?email_source=notifications&email_token=AAKUAEITSPRFJ24ORSAVPNLQ3EHFXA5CNFSM4KBBXNQKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCQK5MLA#pullrequestreview-336975404,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKUAEOZFCXXNXQGFBJI4J3Q3EHFXANCNFSM4KBBXNQA
.

should be g2g @danielhuppmann


GH-1467 Make the plugin directory configurable

Since plugins are technically a part of the code, not the configuration, it should be possible to separate them from the workdir.

Thanks!


High GPU memory on simple import

When importing craftax functions such as the one below, my GPU memory immediately fills up for a large part. E.g. when I run the single line below
from craftax.craftax_env import make_craftax_env_from_name

and then check the gpu memory, I get that 60GB is already taken up on a H100 machine:

Is this normal? And if so, what's causing this huge memory allocation?
Thanks a lot for your help!

It is default JAX behaviour to preallocate 75% of GPU memory.
This behaviour can be controlled as detailed here https://jax.readthedocs.io/en/latest/gpu_memory_allocation.html

I had no idea, thanks for the pointer!


404

all links to samples in Batching section return 404 error https://docs.microsoft.com/en-us/azure/storage/common/storage-samples-dotnet?toc=%2Fazure%2Fstorage%2Fqueues%2Ftoc.json#batching

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

ID: 78f784ff-2a15-ab1d-4fba-88d4cd49759d
Version Independent ID: 6df17b4f-2d9c-3d1e-96a3-f209b14b7e2b
Content: Azure Storage samples using .NET
Content Source: articles/storage/common/storage-samples-dotnet.md
Service: storage
Sub-service: common
GitHub Login: @mhopkins-msft
Microsoft Alias: mhopkins

@alexvolchetsky Thanks for your question. We are checking on this and will respond to you soon.

@alexvolchetsky Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Hi @alexvolchetsky
I've fixed these broken links. The topic will be published later today.
Thanks for reporting this!
~ Mark
#please-close


Add Apache License notice statement

Signed-off-by: kiaye<EMAIL_ADDRESS>
Ⅰ. Describe what this PR does
Add Apache License notice statement at pkg/ddc/jindo/operations/base.go
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Didn't add any test cases.
Ⅳ. Describe how to verify it
Just take a look.
Ⅴ. Special notes for reviews
None

收到！


Corrected Readme file order and versioning.

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Readme file was not properly generating SDK files due the ordering and versioning. I have updated the readme to correct this error on my behalf.
Can this PR be a sprint candidate?

It appears that there is no meaningful change to the 2 json files. Maybe you can unstage them?


Add beacon build args

Scope of changes
This adds the rest of the beacon UI build args so that the variables are accessible in the production container.
Fixes SC-13580
Type of change

[ ] new feature
[ ] bug fix
[ ] documentation
[ ] testing
[ ] technical debt
[x] docker

Acceptance criteria
All required variables are present and the github secrets have also been added.
Author checklist

[ ] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[ ]  Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[ ]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.
[ ] Are there any TODOs in this PR that should be turned into stories?

@pdeziel I removed my suggestions - it looks like we already have REACT_APP_ANALYTICS_ID and REACT_APP_SENTRY_DSN so I propose we stick with those for now.


Clarification request on iApp and FAST template compatibility

Can someone explain this statement:
Warning
We strongly recommend not using both FAST AND iApp templates together as these templating solutions are incompatible with each other. Using both FAST and iApps is likely to create configuration and source-of-truth conflicts, resulting in an undesirable end-state.
Source: https://clouddocs.f5.com/products/extensions/f5-appsvcs-templates/latest/userguide/getting-started.html#getting-started
Not all iApp templates have been replaced and F5 customers will have a reason to run both concurrently. If there is a reason to be cautious missing iApp and FAST templates it would be go to know to aid F5 customers migration from iApps to FAST.

Hi Glen, thanks for the conversation. We will follow up internally as discussed and close this issue.


If time_units is "generations" should we allow generation_time to be set?

The value won't be used, so maybe it should be an error to specify it? On the other hand, it could be useful information that might have been used in inference or something?
The reference parser currently throws an error.

I think it should be allowed, for the reason you mentioned. Although if we allow that, you could reasonably ask why we don't have a mutation rate or recombination map (doi?) fields as well, as those could also have been parameters used to infer the model.

I think it should be allowed, for the reason you mentioned. Although if we allow that, you could reasonably ask why we don't have a mutation rate or recombination map (doi?) fields as well, as those could also have been parameters used to infer the model.

OK, let's allow it then. It's a grey area and I doubt it'll end up leading to significant misunderstandings in the future.
To avoid feature creep, I think we should keep Demes about parameters that aren't about the genome - so, we don't ever talk about mutation rate, etc. See #44.

OK, let's allow it then. It's a grey area and I doubt it'll end up leading to significant misunderstandings in the future.
To avoid feature creep, I think we should keep Demes about parameters that aren't about the genome - so, we don't ever talk about mutation rate, etc. See #44.


Drivin back

Is there a button to let drive back the vehicles?
Currently it seems, they are not to be able to drive back.

Sry, found the key, it's shift ;-).

Sry, found the key, it's shift ;-).


1454 Bypass virtual swapchain under certain conditions

Fixes #1454: Replay to use original swapchain under certain conditions, instead of virtual swapchain.

CI gfxreconstruct build queued with queue ID 156938.

CI gfxreconstruct build # 3918 running.

CI gfxreconstruct build # 3918 failed.

Note this was previously #1462, feedback from that PR should be addressed in this PR.


Deeplinking to Apps does not work with mobile

This functionality is great, but it does not work for Teams mobile.

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

ID: 45541337-ecd7-5cf6-cbe4-3f0d2f39aa79
Version Independent ID: 6b4203f0-2844-06c9-eaa3-9cc03e6f76c0
Content: Create deep links - Teams
Content Source: msteams-platform/concepts/build-and-test/deep-links.md
Product: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

@figgsoftware - Could you Please share what you are trying to deeplinking and which mobile you are using ios or android?

Ok thanks.
We will check and redirect it accordingly.

@figgsoftware - The issue is fixed now. Please verify.

We are closing the issue for now. Please feel free to re-open if needed.

@Prasad-MSFT Thx for the update. Still not working for me. Is it a special version of the Android and iOS Teams app that does support it? I don't have any pending updates in Google Play for the Teams app.

@figgsoftware - We will reverify it from our end and let you know.

Thx @v-chetsh ! Here is the deep link to our app that we are testing with: https://teams.microsoft.com/l/app/d3d1be68-066c-4967-a74b-9edcf902dcfb

@figgsoftware - We have tested on Android 11 . Its working fine at our end. For IOS we are checking with the engineering team. We will let you know once we have any update.

@figgsoftware - We have informed Engineering team about the IOS issue and they are currently working on the fix. We will inform you once we have further update.

@figgsoftware - We have tested below deeplink on IOS 16.2 and Teams version 4.22.1 and its working.
https://teams.microsoft.com/l/app/d3d1be68-066c-4967-a74b-9edcf902dcfb
Could you please test this and confirm?


Object Detection on KITTI db - error code -11 / probably learning rate issue?

Hi,
My final aim is to train my own network that is able to find specific objects on microscopy images, but first I would like to see and learn the workflow on the KITTI db. I followed everything as it is here: https://github.com/NVIDIA/DIGITS/tree/master/examples/object-detection
But when I run this setup, at random epoch it stops with error -11. I have a 4GB GTX 960m GPU, so I set the Batch size to 2 and the accumulation to 5.
The strange fact is that the training can be done easily even with 50 epochs if I set the learning rate to 0,01, but as it is expected the mAP stays on 0 (no converging / cant recognize the cars). When I set back to 0,0001 (or even 0,001), than the error comes again.
From the following output files (error at 10th and 16th epoch), first my thought was that there is something wrong with CV, but how can the training be done without any problem when the LR is higher? This makes me feel, that everything is fine with CV...
caffe_output_epoch10.txt
caffe_output_epoch16.txt
I stuck here since two weeks, I would appreciate some comments or tips, what or how to improve the training process.

Hi, I do not know meaning of error 11 but lets check a few things. Did you set input images' resolutions to 1248x384? Also, do you have one (ie, car) or more than one class in label files? Did you change original class names? (Car, pedestrian,  Van, etc...)

Hi @ontheway16 and thank You for your answer!

Did you set input images' resolutions to 1248x384?

Yes, when I created the dataset. (screenshot attached)

do you have one (ie, car) or more than one class in label files?

Yes, it is from the original KITTI dataset. (see the attached file for example)
000004.txt

Did you change original class names? (Car, pedestrian, Van, etc...)

No, ofc I did not changed them.

I am also attaching a screenshot about the details of my training setup:

Everything seems in place.. Did you try other examples, like object recognition etc? (mnist dataset). Are they working without issues?
Randomly occuring errors also brings hardware related issues into mind.. In case of suspicion, you can monitor temp/GPU_load with $nvidia-settings
Thats all I can think of, at the moment.

Did you try other examples, like object recognition etc? (mnist dataset)

Yes, I tried Classification on mnist and worked perfectly!

I was monitoring everything meanwhile, the GPU temp was between 60-65°C and the utilization between 90-100%.
But I am pointing back again, I tried 2 times with 0,01 LR and 50 epochs - and no errors occurred.

As a last resort, try set batch & accumulation to 1 and run again.

That was also my first idea when I got these errors, but didn't worked.
But today, just double check (and I was also pretty curious about this), set them to 1 and started 3 training (not at the same time, but with exactly the same setup) and all of them failed at the same epoch with the same error message:
Iteration 6368 (4.56143 iter/s, 174.507s/796 iter), loss = 26.9224
Train net output #0: loss_bbox = 6.43325 (* 2 = 12.8665 loss)
Train net output #1: loss_coverage = 14.0558 (* 1 = 14.0558 loss)
Iteration 6368, lr = 0.0001
Snapshotting to binary proto file snapshot_iter_6373.caffemodel
Snapshotting solver state to binary proto file snapshot_iter_6373.solverstate
Iteration 6373, Testing net (#0)
Ignoring source layer train_data
Ignoring source layer train_label
Ignoring source layer train_transform

So I think I was wrong and there is no "random" crash, but there should be an explicable reason for this.

I still don't know what was the problem. But I reinstalled my system and now I installed DIGITS in some other way than before and everything works perfectly. Thanks for your help!


Improve taiga client according to specs

Implement taiga as a singleton design, Fix caching and any needed enhancements

Done
branch: taiga_latest_specs
PR: #42

We found an issue and we open an issue in taiga-back official repo while creating project templates
https://github.com/kaleidos-ventures/taiga-back/issues/60


Make urls work locally as well as on gh-pages

If you deploy to a page you need to add '/{Project Name}/' to the _config.yml
However then all the asset linking breaks for testing locally. From what I can tell from the Jekyll docs you can get around this but I haven't quite figured out a good fix as I'm new to jekyll and gulp.
This issue seems to affect your personal blog as well it would be nice if both of these rendered correctly

http://aneverendingdream.com/
http://vincentchan.github.io/aneverendingdream/

+1 to this. It works great locally but assumes you are pushing to an organisation page not a project page, and so the asset paths all break

I know that this issue is pretty old, but Can anyone help me about adding this?? I'm not able to figure out in Jekyll 3

+1

Guys I resolved this problem many months ago, What I did was update the config.yml the permalink section with:
# Permalinks
permalink:           /:title/
baseurl:             ''
baseurl_posts_img:   '/assets/images/hero/'
baseurl_featured_img: '/assets/images/hero/'
baseurl_featured_thumbnail: '/assets/images/thumbnail/'
baseurl_javascripts: '/assets/javascripts/'
icon:                /assets/images/icon.jpg

Also I updated _includes/menu.html the go back button because was broken after update the permalink with:

      {{ site.name }}

Twitter

LinkedIn

Quora

Dribbble

Kiva

Here is my repo and currently working on jekyll 3.5.
I hope help you.


Cbo 1699 spike hearings sort order v2

What
Ticket
board ticket description
Why
How

TODO (wip)

[ ] item 1
[X] item 2

@kmahern also, if you make it a PR against my branch rather than master that will clear out my changes and just display your commits - needs rebasing though as mentioned

@jsugarman, I have removed all references to sort_order and replaced them with column and direction.

No blockers per-se on the functional code side of things, although i think the controller should probably not know about defaults for hearings sorting since we have classes that can handle and isolate that.
The naming of the attributes could also be misleading in the context of a prosecution_case - prosecution_case.column and prosecution_case.direction do not really state there intention (i.e. hearing sort column and  direction)
Couple of factory related suggestions, but not a blocker
On the specs I think there is a lot of repetition that might be handleable by shared examples, plus my other concerns regarding what some tests are basing success on - example

@jsugarman I think I have made the changes you suggest and have created shared_examples_for_hearings_sorters.rb to attempt to reduce repetition.


Add institution scenario

Implement scenario when user creates new institution.

[ ] tab must to display NEW label
[ ] user can create new institution

At first it is not working, while I tried to use different instances of useArrayForm() with same name in InstitutionsTabsList and institutionsList
const {
    fields: institutions,
    append: appendInstitution,
  } = useFieldArray({
    name: `institutions`,
  });

The problem was that InstitutionList not seeing freshly created institutions among fields: institutions. This problem described in docs https://react-hook-form.com/docs/usefieldarray#:~:text=should not have multiple useFieldArray with the same name
to fix this I passed append and fields from InstitutionsList to InstitutionsTabsList avoiding call extra useFieldArray() inside of InstitutionsTabsList

[ ] But, now i wonder if it is correct to call another useFieldArray() for asset list inside of InstitutionContainer

const arrayName = `${institutionName}.assets`;
  const {
    fields: assets,
    remove,
    append,
  } = useFieldArray({
    name: arrayName,
  });

Maybe I need to pass fields from InstitutionList and filter em like fields.institutions[id].assets
It might also affected #47 when isDirty state did not work as expected

Since  useFieldArray append is async, i wonder how is it possible to get updated field right after calling append
function handleInstitutionCreate({ append, setSelectedIndex, fields }) {
  append({
    name: "",
    country: "",
    assets: [
      {
        amount: "",
        currency: "",
        isEarning: false,
        description: "",
      },
    ],
  });
  // how to get updated fields list here?
  const selectedIndex = fields.length;
  setSelectedIndex(selectedIndex);
}

Temporarily not updating StoryBook, because to implement components, which rely on context, need to write tons of decorator stuff. Do not know how to deal with it.
Demo of scenario:
https://github.com/skorphil/nextjs-form/assets/6762009/6c4b2108-ea64-4352-836a-2637ff0b4413


Displace read-only adapter field with an interactive adapter chip.

Enhancement Short Description
Displace read-only adapter field with an interactive adapter chip.
Current Behavior
Context:
When Meshery connects to one of its adapters, users see those adapters represented as a chiclet in the Dashboard view (see screenshot). Users also see menu items representing each adapter in the navigation menu on the lefthand side.

Enhancement Long Description
When a user clicks on one of those menu items (on an adapter), they are navigated to that adapter's specific page or the management page the adapter URL is displayed as a read-only field (see "current" screenshot).
This enhancement is to display the adapter as a chip (the same as are on the Dashboard). When this chip is clicked, it should invoke the same connectivity test that is performed when an adapter chip is clicked on the Dashboard. See the mockup below:
Mockup

@Nikhil-Ladha is a great resource here should you have questions @ashiscs


Doesn't work if a course has no levels

It works fine if a course has at least 2 levels, but when it has no levels, this does not work (the text box remains empty).

For courses with a single level you can just select the words and copy them. I'd be happy to accept a PR for this, though.


Contact form enhanced

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #(issue no.)
Description
I changed the style of contact us page a littile

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
Before

After

Checklist:

[ ] I have made this change from my own.
[ ] I have taken help from some online resources.
[ ] 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 tested the changes thoroughly before submitting this pull request.
[ ] I have provided relevant issue numbers and screenshots after making the changes.

@PriyaGhosal Please check my PR

@Son7c

This picture is very small.Please see the conflicts . Resolve the conflicts . Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Have Visual Studio handle linking to mbedTLS.lib internally

Description
This PR changes the way the mbedTLS.lib is linked to the projects in the Visual Studio solution.
The new way is more idiomatic and solves the linking problem for the x64 target architecture reported in #1347 .
Status
READY
Requires Backporting
This issue seems to have been present for some time. Should be considered to be backported to any older branches that are still maintained.
Migrations
NO
Additional comments
None
Todos
None
Steps to test or reproduce
Load the fresh solution in Visual Studio (tested in 2017 Community edition), and build the Debug Win32 and x64 targets.

Stopper: @mpg points out that this changes automatically-generated files, but not the files that generates them. So the changes would be overwritten, at the latest, when this is relased (by bump_version.sh).

I've added a new commit to update the template. Please review the new commit.
After running ./scripts/generate_visualc_files.pl, no changes to application vcproj files are found: the output of the script matches @k-stachowiak's changes.

Since we have the objective test that ./scripts/generate_visualc_files.pl now leaves the files unchanged and we want to merge this ASAP, I'm going ahead and merging this without waiting for @mpg to re-review.


[BUG] directory: path.resolve(__dirname, "locales")

// i18n options
const i18n = new TelegrafI18n({
  defaultLanguage: "ru",
  useSession: true,
  directory: path.resolve(__dirname, "locales")
});

when try to start bot it says:
Cannot convert undefined or null to object - it is related with the directory attribute of i18n config object. Who knows what is it and how can I resolve  this issue?

Hey @weijinnx I had similar issue with Typescript. I solved it by adding"resolveJsonModule": true, to my tsconfig.json Not sure if it helps, but still. Make sure that you actually can access locales folder and files in it.

@dmbaranov thanks! It helps. Do you know can I load locales, not from 1 folder? I want to load locales from a few directories.

Well, not sure. According to the source code, it accepts the string as a path to the directory and loads all the files from there. What you can try is, create a locales (or whatever) directory and put symlinks there which are pointing to the different folders.

Even though I don't have such a case, it would be interesting to see your solution :)


Teach options to know about a Path type

At the moment, the type we use for Path values is str. We use paths in many options, and we'd like to handle them specially in user interfaces (so we can, for instance, do path completion on tab).
We should teach optmanager to know about a Path type, which is treated just like a string. Ergonomically, I'd like user to still be able to say options.path = "value" rather than have to say options.path = types.Path("value"), so it would be good to automatically handle type promotion.

I'm working on it

I am also working on this. Could a maintainer tell me if #2209 is a step in the right direction? (I am also thinking of integrating some common os.path functions in Path)


fixed various errors in the patches

the following patches have errors in them and can't be applied, I fixed the whitespace errors , but wasn't able to solve 3 of them  as im not that versatile in C, still learning :) ...
attachx (fixed)
git apply patches/dwm-attachx-6.2.diff
patches/dwm-attachx-6.2.diff:73: trailing whitespace.

patches/dwm-attachx-6.2.diff:95: trailing whitespace.

warning: 2 lines add whitespace errors.

fancybar (fixed)
git apply patches/dwm-fancybar-6.2.diff
patches/dwm-fancybar-6.2.diff:89: space before tab in indent.
                }
warning: 1 line adds whitespace errors.

pertag-configurable (fixed)
git apply patches/dwm-pertag-configurable-6.2.diff
patches/dwm-pertag-configurable-6.2.diff:191: trailing whitespace.

patches/dwm-pertag-configurable-6.2.diff:220: space before tab in indent.
                        selmon->tagset[selmon->seltags] = arg->ui & TAGMASK;
warning: 2 lines add whitespace errors.

savefloats-configurable (fixed)
git apply patches/dwm-savefloats-configurable-6.2.diff
patches/dwm-savefloats-configurable-6.2.diff:57: space before tab in indent.
                                resize(c, nx, ny, c->w, c->h, 1);
patches/dwm-savefloats-configurable-6.2.diff:72: space before tab in indent.
                                resize(c, c->x, c->y, nw, nh, 1);
warning: 2 lines add whitespace errors.

systray (fixed)
git apply patches/dwm-systray-6.2.diff patches/dwm-systray-6.2.diff:686: space before tab in indent.         x -= w; warning: 1 line adds whitespace errors.
systray-on-top-of-alpha (fixed)
git apply patches/dwm-systray-on-top-of-alpha-6.2.diff patches/dwm-systray-on-top-of-alpha-6.2.diff:658: space before tab in indent.         x -= w; error: patch failed: dwm.c:59 error: dwm.c: patch does not apply
vanitygaps (fixed)
git apply patches/dwm-vanitygaps-6.2.diff
patches/dwm-vanitygaps-6.2.diff:580: trailing whitespace.

warning: 1 line adds whitespace errors.

the following I wasn't able to fix:
zoomswap-pertag-configurable
git apply patches/dwm-zoomswap-pertag-configurable-6.2.diff
patches/dwm-zoomswap-pertag-configurable-6.2.diff:84: trailing whitespace.

error: patch failed: config.def.h:7
error: config.def.h: patch does not apply

--
even after deleting whitespace
error: patch failed: config.def.h:7
error: config.def.h: patch does not apply

tagswapmon
git apply patches/dwm-tagswapmon-6.2.diff
error: corrupt patch at line 105

tagallmon
git apply patches/dwm-tagallmon-6.2.diff
error: corrupt patch at line 76

autostart
git apply patches/dwm-autostart-6.2.diff
error: corrupt patch at line 41

So, most of these are warnings, not errors and they are actually applied correctly. In your diff you are also removing leading spaces added by git diff (indentation for the +/- prefix indicating what lines have changed).
I'll have a look at the patches that failed.

sure mate, not a pro at this :) just tried your diffs and got the following results so I tried to clean up and reported it to you :)

Hi @kronikpillow,
so tagswapmon, tagallmon and autostart patches were all bad. Not sure why, but I must have done something wrong when I prepared the patches originally - I have corrected these now.
The dwm-zoomswap-pertag-configurable-6.2.diff actually works, but it is meant to be added on top of the pertag patch (it doesn't include that). This is perhaps not obvious from the file name.
I corrected the whitespace warnings from the other patches you mentioned. Thanks a lot for reporting!
Thanks,
-Stein


[CARBONDATA-1730] Support skip.header.line.count option for Hive Table in Carbon

Support skip.header.line.count option for Hive Table in Carbon
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1470/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1889/

i just change the class it extends,which i think is the best way to by-pass having to modify spark source code and adding more functions to carbon.
Changing is necessary,why treat it as confiling files?

add to whitelist

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1633/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/2024/

Build Failed with Spark 2.2.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/955/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2184/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3057/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4301/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5189/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3979/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4472/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7127/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5903/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/55/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8804/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/737/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/559/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2184/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10259/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3566/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/82/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/81/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1083/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/81/


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3376

Questions ?
Please refer to the Backport tool documentation


tcsh hangs after one command

When I start the tcsh shell (open cmd window, type "bash", type "tcsh"), I can enter 1 command, and it executes, but it hangs before the command prompt is shown again.  Ctrl-C doesn't do anything, I need to close the window and start over.

It looks like this was fixed internally recently, I installed tcsh and ran of bunch of commands and nothing hung for me.  I'm going to close this one out but if you ran any special commands go ahead and leave a comment and I'll try them out to make sure they're fixed.
Thanks for reporting the issue!

"ran any special commands" == "ls" :-)
Is there a way we could get updates separate from the normal Windows update process?  Given the fast moving nature of this, I can see a lot of "fixed internally recently" answers, because we're always behind.

Definitely ran ls :).  Also did apt-get, cat, echo, htop, etc.
For now the only way to get updates to the Windows Subsystem for Linux is through the normal Windows fighting process (which I agree is on the slow side).


launcher: don't take a password from them

As a follow-up to #597, I'd prefer if we didn't even ask the user for a password for their GCE instance.
I think the instance can generate its own unique password and set the instance metadata (https://godoc.org/google.golang.org/api/compute/v1#InstancesSetMetadataCall) on first boot. Then in the camlistore.org/launch/ web UI we can just say "for your password, go to [instance page URL] and find it in your metadata list"
To get even fancier, we can add a handler on the instance which does some oauth stuff and checks whether the remote google user is the owner of the instance somehow. That's more like 0.10 material, though.
For 0.9 I'd prefer to not even take their password.
But I suppose we can for now if we have to. Or we can at least update the wording to say that it's optional and we don't retain it, and if they don't trust us, they should update it later?

ACK. I'll ping you about it after I'm done with dock.go to discuss our time
frame. I should have a bit of time this afternoon.
(on phone)
On Dec 27, 2015 05:48, "Brad Fitzpatrick"<EMAIL_ADDRESS>wrote:

As a follow-up to #597
https://github.com/camlistore/camlistore/issues/597, I'd prefer if we
didn't even ask the user for a password for their GCE instance.
I think the instance can generate its own unique password and set the
instance metadata (
https://godoc.org/google.golang.org/api/compute/v1#InstancesSetMetadataCall)
on first boot. Then in the camlistore.org/launch/ web UI we can just say
"for your password, go to [instance page URL] and find it in your metadata
list"
To get even fancier, we can add a handler on the instance which does some
oauth stuff and checks whether the remote google user is the owner of the
instance somehow. That's more like 0.10 material, though.
For 0.9 I'd prefer to not even take their password.
But I suppose we can for now if we have to. Or we can at least update the
wording to say that it's optional and we don't retain it, and if they don't
trust us, they should update it later?
—
Reply to this email directly or view it on GitHub
https://github.com/camlistore/camlistore/issues/656.

Moving this to 0.10. I updated the wording for 0.9. In 0.10 with automatic TLS, we'll remove all the secret generation from the launcher, including this.

I'm not sure I understand that last comment about automatic TLS. Do you mean that we should get issue #239 done first? And that we should use Let's Encrypt to generate a client cert (dunno if that's possible?), and that by default we don't use user/pass auth anymore for the instance created by the launcher?
Ah, or are you saying that the plan for that issue is not changed, but you're mentioning server-side cert generation as another thing they (or us) don't have to provide anymore ? (thanks to LE).

I'm going to work on the first plan to start with: let the instance gen the password, and tell them where to find it.

https://camlistore-review.googlesource.com/6008

Not closing because I haven't done the fancier (optional?) oauth part yet.


make: Move systemd service generation to "install" target

Hi, thanks for developing!
Would you be amenable to moving the systemd service generation lines (https://github.com/rvaiya/keyd/blob/f0b5f122e0d0772f3fedd5bb6295212b0bb18174/Makefile#L37C1-L38C1) down to the "install" target? Running make with PREFIX is usually the install target, not the regular building targets.

This should be done in the latest commit.

Thank you!


Expire auth code after 10 minutes

Description:
When a user successfully logs in, they get an authorization code that they can exchange for a pair of access/refresh tokens. This PR makes sure the auth codes expire after 10 minutes as per OAuth 4.2.1
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

LGTM


Cannot get other devices to connect to databox deployed on ubuntu

Hey, I am testing some video sharing application using databox, most of of my prev development was done on macOS and phones were able to connect normally to the databox, but when I tried to deploy the databox on ubuntu everything is working fine on host itself but other devices are unable to connect to the databox, and nmap shows the databox ports as filtered rather than opened.
I have read an earlier issue mentioning a problem with ubuntu on entreprise network but my deployment is done on personal one.

Can you verify that the docker network is correctly configured on your Linux box, by running a stand-alone container with a network service (just a standard web server or something simple)? Then accessing this from a different device?
This will show whether it is a databox or docker/Linux config problem.
Cheers
Tosh


MenuItem close when has children

Problem description
I'm trying to use on AppBar with the IconMenu and MenuItem. This works for the first children in the
tree. But when I'm trying to use children of a children this wont work.
This work
MenuItem
     MenuItem

This wont work
MenuItem
     MenuItem
           MenuItem

I'm trying to figure out what's happen but until now I doesn't have success. If someone can point me to where I can fix this I can work and make a PR
Example of code
http://www.webpackbin.com/4JN_dLi0-
Versions

Material-UI: 0.16.0
React: 15.3.2
Browser: Chrome/Firefox/Safari

Found the same problem. Not sure if this is a bug or it is by design, as Material Design suggests limiting nesting one level deep.
https://material.google.com/components/menus.html#menus-menu-items

Menu nesting
Menu items can reveal nested submenus. Ideally, limit nesting to one level deep, as it can be difficult to navigate multiple nested submenus.

This also happens when using a regular Menu, as soon as you reach the second level in Main Menu > First Level (popover) > Second Level (popover) and click a MenuItem, it makes the First Level disappear and closes the whole thing.
Doesn't seem possible at all to nest more than one level in IconMenu and more that two levels in Menu.

@jmansor yes I saw that. But it says "Ideally, limit nesting to one level deep". So I believe we can have a second/third level, of course this is not recommended but in some cases this can be necessary

I believe this and #4887 are related

Dupe of #4887


Update get_table_chunks to return only parquet files with given _series_ids if present in filter

As part of the prototyping for making single series queries fast, one idea is to filter down the parquet files that get presented to the query engine based on the series ids in the query. This requires #24866 to be wired up first.
Given a query that is looking for series ids, make chunks only for the Parquet files that have those series. One of the tests we'll want to run here is given N parquet files in the set, say that only 5 of those files have data relevant to the query. What is the performance of the query look like with only those 5 files, or those 5 + 10, 20, 30, 40, 50 additional files that contain data, but are not relevant to the query itself.

no longer relevant


Sidebar items clickable when hidden

I guess these links are still clickable because the ul is display: flex:

I guess there is no issue, there are clickable when the sidebar is shown, and in your example, their container has width: 0 because the sidebar is toggled be hidden


Fixes for retain cycle and SwiftUI runtime warning

Retain cycle fix -  0f2bd35
Create state stream on next run loop to address SwiftUI runtime warning - a52d2a2

Note: This will also address #5.

With this update, some of our SwiftUI workarounds such as BecauseTextEditor should no longer be necessary.
What BecauseTextEditor was designed to address
LEFT - Plain TextEditor using Combiner 0.0.6 (This PR)
RIGHT - Plain TextEditor using Combiner 0.0.3 (Which required BecauseTextEditor workaround to address)
https://user-images.githubusercontent.com/5950250/193140694-80007f2e-1d0b-4c88-8d6b-d5110868fd3f.mov


Fix broken links in Markdown files

This pull request fixes broken links in Markdown files
This is looking like part 1 of 2, not sure I want to have it in a giant pull
Reviewed

[x] cert/ManualSteps.md
[x] CODE_OF_CONDUCT.md
[x] deliver/Reference.md
[x] fastlane/docs/Android.md
[x] fastlane/docs/Appfile.md
[x] fastlane/docs/Bamboo.md
[x] fastlane/docs/CodeSigning.md
[x] fastlane/docs/Gitignore.md
[x] fastlane/docs/Jenkins.md
[x] fastlane/docs/Platforms.md

Fixed

[x] CONTRIBUTING.md
[x] deliver/Deliverfile.md
[x] deliver/MigrationGuide.md
[x] fastlane/docs/Advanced.md
[x] fastlane/docs/FAQs.md
[x] fastlane/docs/Guide.md
[x] fastlane/docs/README.md

@ohwutup please review :smile:

hey @ReadmeCritic, would you mind squashing these before we merge?

closing this for a squashed version: https://github.com/fastlane/fastlane/pull/3618

@ohwutup thanks, sorry for the trouble
Will try to squash/consolidate my next contribution


Screenshots On Portal

Please update the screenshots in the article as none of them are relevant anymore. All the options on the Azure portal have also changed. So, kindly keep the articles updated for the public to view and follow.
[Enter feedback here]

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

ID: 2ac8cd29-26cd-47da-5b3e-d58ba7c4a20c
Version Independent ID: ae9587c8-0822-f5af-98ef-5386c4f17e58
Content: Active Directory Federation Services in Azure
Content Source: WindowsServerDocs/identity/ad-fs/deployment/how-to-connect-fed-azure-adfs.md
Product: windows-server
Technology: ad-fs
GitHub Login: @billmath
Microsoft Alias: billmath

Thank you for bringing this to our attention. The appearance of the UI has changed, but the process described in the article hasn't, so the screenshots are still relevant. However, we can add updating the screenshots to our current backlog.


Schema validation ignores fields that are not required

Schema validation currently ignores fields that are set to not required, as if the field was not specified.
However the comments explain the required field being set to false as allowing patch requests to
function properly.
For example if I have a field called planName and I have the required field set to false,
then it should mean that it is not a violation to submit a PATCH request without the planName field in the body. However if I do send the planName field and it does not match the schema then there should be a violation.
Using the above example currently sending the planName not formatted in the way the schema expects is  met with no errors, when set to "required: false".

You're right. The schema validator does ignore fields that are not required. However, when the isPartialObj parameter is set to true, the function will treat the input object as a PATCH object. All fields are optional, but if you include any of the fields it has to satisfy the schema. Once the isPartialObj parameter is set to true, the validator doesn't care about the required flag anymore---as all fields are now treated as optional.
For your example, since the request is a PATCH, you send in a planName which is one of the field names of the planSchema, the validator must validate this planName regardless of the requirement because if it didn't, it would patch the object with the invalid planName to the existing object.
In short, my intention when I implemented the schema validator to work with PATCH requests is that "All fields are optional, and if a field is included in the request body, it must satisfy the schema."

Thank you for the clarification, with that in mind I realize there is not a bug here. I am closing this issue now.


Application stopped - Unity - Play Games Services

Hello, I'm developing an Android game with Unity, it already had PGS, IAP and AdMob configured, so I installed the Analytics and Cloud Messaging packages from Firebase. Since then the game lost the icon and when the PGS tries to authenticate the application to work, someone could help me with this problem.
Sorry for my English.
Ths

@GilVieira I don't think we can diagnose Unity issues here, this quickstart is for Android applications built in Java.
I'd recommend emailing our personalized support channel who can help you troubleshoot.


TaskBuilder: update default name

Codecov Report

Merging #314 (1b47c7e) into master (07ed9ff) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #314   +/-   ##
=========================================
  Coverage     61.78%   61.78%
  Complexity      448      448
=========================================
  Files            90       90
  Lines          1748     1748
  Branches        184      184
=========================================
  Hits           1080     1080
  Misses          613      613
  Partials         55       55

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 07ed9ff...1b47c7e. Read the comment docs.


Add Product Feedback survey link to welcome page

Links to https://aka.ms/AzureSQLSurvey

Pull Request Test Coverage Report for Build 647190247

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 43.179%

Totals

Change from base Build 647184364:
0.0%

Covered Lines:
25305

Relevant Lines:
53446

💛  - Coveralls


Invalid textDocument field in build/publishDiagnostics notification

Mill build server sends build/publishDiagnostics notification with invalid textDocument field, when publishing information about deprecation. It should contain an uri of a text document but is the build target uri instead. This message should most probably just be send via build/logMessage.
[Trace - 02:45:10 pm] Received notification 'build/publishDiagnostics'
Params: {
  "textDocument": {
    "uri": "file:///Users/kmarek/Desktop/Developer/Tests/scala3-mill-example-project/examples"
  },
  "buildTarget": {
    "uri": "file:///Users/kmarek/Desktop/Developer/Tests/scala3-mill-example-project/examples"
  },
  "diagnostics": [
    {
      "range": {
        "start": {
          "line": 0,
          "character": 0
        },
        "end": {
          "line": 0,
          "character": 0
        }
      },
      "severity": 2,
      "code": "-1",
      "source": "mill",
      "message": "there were 3 deprecation warnings; re-run with -deprecation for details"
    }
  ],
  "reset": false
}

mill-version: 0.11.6-24-0770d3

Thank you @kasiaMarek for opening this issue. I agree with you. Since the compiler message in question does not belong to a single source file, it's not actionable (in the sense of a published diagnostics), hence we should instead just push a logMessage.
I opened PR #2979 to address this.


Include attributes that have changed along with the model

When using wisper listeners in async mode, the model no longer has what attributes have changed which is useful for only firing certain events if a particular attribute has changed.
One option would be to include the changed attributes along with the model so that the listener can inspect old values and perform logic as needed.
ex:
def after_update_broadcast
  broadcast(:after_update, self)
  broadcast("update_#{broadcast_model_name_key}_successful", self, self.previous_changes)
end

This is a good idea. I was also thinking we should change the first argument to self.id (only for the second broadcast) as well since self will not work for async unless you are using ActiveJob which has proper support for deserializing to ActiveRecord.
It will be a breaking change, but since we are still version 0.x it should be okay.

FWIW, there is a new gem that includes this functionality out of the box if anyone is interested in pursuing this further.  It allows for background job listeners to work as expected with activerecord events.
https://github.com/betterup/wisper-activerecord-publisher

Did this got implemented?

I don't think so, other than the gem linked to above, which I haven't tried myself. But you can do this with plain wisper events. Call broadcast from a ActiveRecord hook and include the model id and the previous_changes in the payload. You could wrap this up in a module if you need the functionality for multiple models. Hope this helps.


[All habits] The user can see all custom habits instead of only their own

Environment: Windows 11,
Google Chrome Version 111.0.5563.147 (64-bit)
Mozilla Firefox Version 114.0 (64-bit)
Microsoft Edge Version 114.0.1823.41 (Official build) (64-bit)
Reproducible: Always
Build found: 15.06.2023
Preconditions:
A minimum of three custom habits are created: 1 belonging to a user, 1 belonging to a user's friend, and 1 belonging to a user who is not among friends.
Steps to reproduce

Open Green City https://www.greencity.social/#/greenCity
Log in.
Click on "My habits" tab.
Scroll to the end of the habits list.

Actual result:
The user can see both standard habits and all custom habits created on the 'All habits' page.
Expected result:
The user can see only standard habits,  habits of user's friends, and their own created habits on the 'All habits' page.
User story and test case links:
"User story #1000

Fixed


Multiple data-target not working

Hi guys,
I'm not sure if this is an issue or not but I found a problem between the version 3.2.0 and version 3.3.6.
With exactly the same code, a dropdown-menu with overlay is working on version 3.2.0 but not working on the latest 3.3.6. Example:

Bootstrap version 3.3.6 on https://www.cheveux-naturels.fr/index2.php, when you click on the top horizontal Menu, the menu is opening but not the overlay.

Bootstrap version 3.2.0 on https://www.cheveux-naturels.fr/, when you click on the top horizontal Menu, bot menu and overlay are opening.

The html code of both pages are exactly the same. The problem is that on the 3.3.6 version, only the second (#MenuMob1) data-target parameter is executed:
data-target="#close1,#menuMob1"
I have done some testing with class instead of id, but I have the same problem.
Thanks in advance for your help.
Regards.
Skaia

Please post JS Bins or JS Fiddles that demonstrate the problem. We don't debug full production webpages.

Hey there!
We're automatically closing this issue since the original poster (or another commenter) hasn't yet responded to the question or request made to them 14 days ago. We therefore assume that the user has lost interest or resolved the problem on their own. Closed issues that remain inactive for a long period may get automatically locked.
Don't worry though; if this is in error, let us know with a comment and we'll be happy to reopen the issue.
Thanks!
(Please note that this is an automated comment.)

Fairly certain this is still an issue.  Looks like data-target  is only returning the first result from the selector now.
collapse.js line 314 is just throwing away any other results from the given selector.
      value: function _getTargetFromElement(element) {           var selector = _Util['default'].getSelectorFromElement(element);           return selector ? $(selector)[0] : null;         }

@aholtventuretech This bug is regarding Bootstrap v3. Your code refers to Bootstrap v4.
(IMHO, we probably ought to change it to only accept IDs instead of arbitrary selectors)


great project! any play port to ios/android platform?

great project! any play port to ios/android platform?

@kindlejiang I have trouble to understand your use case here. Are you trying to host a C++ REST Server on and iOS/Android device?


Populate needed software for lab provisioning

As a comment on this issue, please provide what software hackers will need already installed on their lab computers (if nothing is needed, state so).

Visual Studio Code
Python

Python Packages:
FastAPI
Uvicorn
Control:
Github Desktop (If teams want to use it) (not necessary)

Does the Python version matter?

Latest version of each software.
FastAPI and Uvicorn both require 3.7 to ensure full backward compatibility with whatever we're using.

This did not show up in discord but pydantic also needs to be installed.

Submitted


feature request: download progress

There is no feedback about the progress of files being downloaded, this means that on slow networks it appears to stall when large files are being fetched.

I realise this should be in the upstream project not this one


Update README.md

added the attractive YouTube thumbnail instead of the only link text

@sofia100 please refrain from making changes without raising an issue and discussing with a maintainer. This is invalid because the embed tag will add a complete youtube player instead of just the image. Please read our contribution document before contributing.


Vickel: Watch thehackangels.com

Vickel requests the watch of the watch_keyword thehackangels\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

thehackangels\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Nick in Charcoal HQ


Update gh CLI steps in adding-a-new-ssh-key-to-your-github-account.md

Why:
The GitHub CLI does not allow you to add an ssh key unless you are authenticated with your GitHub account. The gh auth login command not only authenticates you but also gives prompts to add your ssh key to your account. This PR amends the GitHub CLI portion with the gh auth login path. This may be useful to someone coming to this page for the first time who has not already installed the gh CLI and will do so as part of this flow and will then need to authenticate.
Closes #11721

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@michelleN
Thanks so much for opening a PR! I'll get this triaged for review :zap:


[WIP] Refactoring base images

@alexandrosm IMO, the blob entity you want is stack_version. Basically, I think stack_version (https://github.com/resin-io-library/base-images/pull/298/commits/28db079e25022f70265f6d0a745c90b6dd8b04f7#diff-553e99faf730145755967335765dce0fR124) and the original blob (https://github.com/resin-io-library/base-images/pull/298/commits/28db079e25022f70265f6d0a745c90b6dd8b04f7#diff-553e99faf730145755967335765dce0fR124) are the same and they can't represent a binary object. I think what I have now can be called blob_of_stack_supports_arch_is_compiled_for_libc.
If we want to find all "Python 2.7" blobs, just look for all blobs connect with stack_version 10, for example.

@nghiant2710 in this way we can't know if a blob is missing from a stack or not. We're trusting the dataset maker to do it right. But even if we do it right, if a stack has multiple blobs, we have no way of differentiating them. If the python stack is composed of the python executable and the pip executable (for instance), we don't know which is which unless we parse the url.


Change message to not tell people to go to localhost

As a possible improvement, you guys could suggest a possible IP:
    local_ip=$(ip addr | grep 'state UP' -A2 | tail -n1 | awk '{print $2}' | cut -f1  -d'/')
   ...
    echo "  You can now visit http://$local_ip:3000 and bootstrap the cluster with the dashboard"

Or even the default interface ip address. I am good either way, it's just an informative message.

Done