how setlocation?

How do I put my data in it (set loaction ?
and view error
     FirebaseFirestore db = FirebaseFirestore.getInstance();
     CollectionReference myCollection2 = db.collection("xxxx");
     GeoFire geoFire = new GeoFire(myCollection2);

this CollectionReference is database ref

Hey,
So basically geofire doesnt have save capability, thats provided by firebase itself. What you'll need to do (for now at least) is save the document as normal, and in the oncomplete use:
geoFire.setLocation(YOUR_ID, YOUR_LATITUDE, YOUR_LONGITUDE, this);
You'll need the class youre calling setLocation from to implement SetLocationListener. In the onCompleted method you'll either get null if the location was set successfully, or an exception if not.

ok  i am test then replay

this error

Hi,
Please check the docs. You need to give GeoFire your collection reference rather than your database
private FirebaseFirestore db = FirebaseFirestore.getInstance(); private CollectionReference myCollection = db.collection("myCollection"); private GeoFire geoFire = new GeoFire(myCollection);

now ok , but record one value
where lat,lon

g

    FirebaseFirestore db = FirebaseFirestore.getInstance();
    CollectionReference myCollection = db.collection("myCollection");
    GeoFire geoFire= new GeoFire(myCollection);
    geoFire.setLocation("hi", mlocation.getLatitude(), mlocation.getLongitude(),this);

    FirebaseFirestore db = FirebaseFirestore.getInstance();
    CollectionReference myCollection = db.collection("myCollection");
    GeoFire geoFire= new GeoFire(myCollection);
    geoFire.setLocation("hi", mlocation.getLatitude(), mlocation.getLongitude(),this);

So GeoFire wont do that for you. As an example, using Google Firestore guide: https://firebase.google.com/docs/firestore/quickstart
You'd need to do something like this:
`
FirebaseFirestore db = FirebaseFirestore.getInstance();
CollectionReference myCollection = db.collection("myCollection");
GeoFire geoFire= new GeoFire(myCollection);
// Create a new user with a first and last name
Map<String, Object> user = new HashMap<>();
user.put("first", "Ada");
user.put("last", "Lovelace");
user.put("born", 1815);
user.put("latitude", 123.456);
user.put"longitude", -1.23456);
// Add a new document with a generated ID
db.collection("users")
.add(user)
.addOnSuccessListener(new OnSuccessListener() {
@Override
public void onSuccess(DocumentReference documentReference) {
Log.d(TAG, "DocumentSnapshot added with ID: " + documentReference.getId());
geoFire.setLocation(documentRererence.getId(), user.get("latitude"), user.get("longitude"));
}
})
.addOnFailureListener(new OnFailureListener() {
@Override
public void onFailure(@NonNull Exception e) {
Log.w(TAG, "Error adding document", e);
}
});`

Hi,
I've just updated the docs in the readme. Please check those. You'll need to use standard Firestore utilities to store your document first. GeoFire only updates the geoFireLocation field. The google docs are good for explaining how to use Firestore to store documents.
https://firebase.google.com/docs/firestore/quickstart
You'll need to store your document, and then in the onComplete, use geoFire.setLocation() to update the document with the id reference you get back and the latitude and longitude values.

I'm going to close this issue, as I believe you should now have all the information you need.
To summarise:

Save your object using standard Firestore utilities, and in the oncomplete use geoFire.setLocation() to update the document with the geoFireLocation you'll need to perform queries.
use geoFire.query() as shown in the README.

In your examples, geoFire.setLocation needs to go in the onSuccess listener for your user addition. In there, use the ID from the returned document reference, and your latitude and longitude from the user object. Then you will have your latitude and longitude, and the necessary geoFireLocation.

Inside the method appears a line but outside without a line ..

Hi. You are not implementing the listener. As the message tells you, you need to implement SetLocationListener.
Thanks,
Martin

How to implement SetLocationListener.

i implement
public class Find extends AppCompatActivity implements MyTracker.ADLocationListener , SetLocationListener {
and add
@Override
public void onCompleted(Exception exception) {
    Log.w(TAG, "onCompleted ", exception);
}

and same error >>

Okay, let me have a look. If you've implemented everything correctly then it seems there could be an issue. Leave it with me and I'll have a look ASAP.

ok  and thank you alots


Initial Ethereum implementation

Signed-off-by: Graziella<EMAIL_ADDRESS>

Squashed and rebased everything into one commit that contains just the relevant changes - will review and provide feedback shortly.

This looks pretty good - one question:
You use the terms "contract" and "enclave" a lot in here. I know that's what PDO is based around - but does this actually interact with an SGX enclave or "contract" in any way? Or is it just the infrastructure that would be needed for someone else to hook up "enclave" to actually call methods of an actual SGX enclave?
Maybe I'm missing something obvious...
Looking forward to the documentation in a subsequent pull request

The solidity file just holds a “registry” of the available enclaves and contracts— currently there’s no interaction with an actual SGX enclave. It basically acts like a map, linking an enclave’s id with the enclave’s info and a contract’s id with the contract’s info (like which enclaves are permitted to execute it). All this info must be provided by the client when they register an enclave or contract. In that sense it’s just the infrastructure side. As far as I know you can’t make calls to an actual SGX enclave from an Ethereum smart contract.
The only contract interaction is the following: when registering a contract in the contract registry, a call is made to the enclave registry contract to make sure the enclaves that are associated with a contract have already been registered there.
As for the python file, it’s just a demo of registering components in deployed contracts. Basically just a couple of set/get calls to those contracts.

Thanks! Makes sense to me. Will merge.


docs: better image size

v14.0.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Allow availability to be updated in a project

Is your feature request related to a problem? Please describe.
I would like Lagoon CLI to be able to update a Lagoon project for the availability flag.
mutation updateProject {
  updateProject(input: {id: 1238, patch: {availability: HIGH}}) {
    id
  }
}

Describe the solution you'd like
A new option in lagoon update project
Describe alternatives you've considered

Given lagoon doesn't utilise this field currently (and may not in the future), I'd be hesitant to implement a specific command for it?
Future direction for setting and defining workloads needs to be more flexible than the current ENUM.

the cli should not judge what is useful or not, the lagoon api has the field so it should be exposed via the CLI. The CLI is exposing other data that is way less used in the API (like the meta data fields) and they are still in the CLI.
So I vote to add support for it asap and create a documentation in lagoon that explains what the field is used for and what not.


Use PTR constant instead of deprecated ptr() function on registers and peripherals

Command: crate build
Full warning:
warning: use of deprecated associated function `cortex_m::peripheral::MPU::ptr`: Use the associated constant `PTR` instead
   --> board-support/src/lib.rs:140:26
    |
140 |         let mpu = &*MPU::ptr();
    |                          ^^^
    |
    = note: `#[warn(deprecated)]` on by default

warning: `nw-board-support` (lib) generated 1 warning

You might be using nightly rust or a newer version, I don't know why it only happens sometimes, if/when I work on this again I'll check the code for use of deprecated functions.


Citation to POX controller

Hi guys,
Thank you for the POX controller, you have a great work.
We have been using the POX  controller in a network experiment in our research.
We want to cite the POX controller in our research paper.
Currently, our plan is adding footnote "POX Controller : https://github.com/noxrepo/" in one of page in our research paper.
Then, we cite [1], which is the NOX controller paper; just like in [2] to refer to POX controller.
However, others, citing the POX controller as online resource as "POX. http://www.noxrepo.org/pox/about-pox/." like in [3]; in which the domain has been expired.
What is the correct way to cite Pox controller in our research paper?
Best,
hoihei
[1] N. Gude, T. Koponen, J. Pettit, B. Pfaff, M. Casado, N. McKeown, and S. Shenker, "Nox: Towards an operating system for networks, " SIGCOMM Comput. Commun. Rev., vol. 38, no. 3, pp. 105-110, 2008.
[2] M. P. Fernandez, "Comparing OpenFlow Controller Paradigms Scalability: Reactive and Proactive," Advanced Information Networking and Applications (AINA), 2013 IEEE 27th International Conference on, Barcelona, 2013, pp. 1009-1016.
doi: 10.1109/AINA.2013.113
[3] Erickson, David. "The beacon openflow controller." Proceedings of the second ACM SIGCOMM workshop on Hot topics in software defined networking. ACM, 2013.

This is a good question.  noxrepo.org will probably start pointing to a github page soon, but this isn't set in stone and there's no timeline.  So it may make sense to just use the github URL for now.
If you have a chance for revision, e.g., at camera ready time, you could check back to see if the story has changed...


Fix docs of base file helper

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues

Merged. Thanks!

Glad to be of help!


Adding support for EventInvokeConfig

Issue #, if available:
Description of changes:
Adding support for EventInvokeConfig permitted by Lambda API operations in ACK. This PR adds EventInvokeConfig as an inline property for a function.
It is still a WIP PR.


Add the log-probability function

Add the log-probability function, which can be used for plots and for the MCMC inference.

Time estimates: S and M.

Perhaps add #17 to this issue.


Added new image segmentation lab exercise (Car segmentation)

When you added a new program / programs , put a cross on the  OS that you have test it on
(Example: [ X ] Windows  )
[X] Windows
[  ] Linux Ubuntu
[  ] Mac
[  ] Others  (State here -> xxx )
Assign a reviewer to review your code.

pending: issue yet to resolve, crash half way.

FYA @kenghooi-teoh

missing carsegmentation url at config.properties.. dataset not downloaded


yaw change in APM:Copter V3.4-dev

Hi team,
How do I control the yaw of the drone?
I am using APM:Copter V3.4-dev .
I have used https://github.com/dronekit/dronekit-python/blob/master/examples/set_attitude_target/set_attitude_target.py . But set_attitude(yaw_rate = 30, thrust = 0.5, duration = 3) does not work properly.
I need to rotate the drone on its yaw axis. I am using Gazebo simulator

If you only need to control the yaw, you could use the SET_POSITION_TARGET_LOCAL_NED message, setting the type_mask to ignore all the axes except the yaw.


Right click > Save Link as returns error

It seems I messed up something.
For some reason, Right click > Save Link as returns the following error:
      The download cannot be saved because an unknown error occurred.
      Please try again.

Does anybody have the same problem? Or know how to fix this?
FF 59.0.2 on Linux

Hey @theWalkingDuck ... I tested in a nilla 59.0.2 profile (I have a userChrome.css that cleaned up all my context menus). With the user.js applied as it is currently, I had no issues either drag n dropping the link to a folder, or right clicking and saving as. I'm on windows. The test was only for an URL.
I assume this is not a linux issue - eg you have tested it works fine in a new profile. And you tested with all extensions disabled, right?
Are you trying to save a pdf, mp3, jpg? Can you give us a working example?

Did some searching earlier, and it all seems to come down to file type associations, I think. Helps to know what you're trying to download. Here's some possible solution ideas

https://support.mozilla.org/en-US/questions/1022568

I guess it may also depends on what you use for each application type - always open, ask etc. But I would assume "save as" should save to disk, not try to open (i.e why would file associations be important?).
Anyway, I would have a look at Options>General>Applications (which is controlled by mimeTypes.rdf in your root profile - you can close FF, rename it, restart FF and see what happens - this is a text based file btw, so feel free to edit it and have a play)

Also, the RFP pref could affect mimetypes/plugins - eg I know it causes Flash to not load/play (depends on the site's code)

@theWalkingDuck
Running Debian stable with FF 59.0.2 manually installed from Mozilla site

I was experimenting with uMatrix and forgot to reset a rule.
That means Right click > Save Link as can be disabled
either by removing the rule  matrix-off: behind-the-scene true
or by setting it to false matrix-off: behind-the-scene false

Thanks for the info, added an FYI to make the solution stand out more. Still intrigued as to where/how this was happening. If it's behind the scene, then can I assume you were eg in the dashboard right clicking a hyperlink?

@theWalkingDuck  and others, also see https://github.com/uBlockOrigin/uMatrix-issues/issues/5

FYI blocking requests with type Other in Request Control or similar will cause the same error message.

It's all magic to me (I have no idea how any of this stuff works), but here is my default uM

I've never had a problem saving anything (not that I have ever used the right context menu to do so). Guess I've never come across it myself.

other in uM is probably not the same as the Other ResourceType available to webextensions that fe. Request Control uses.

^^ Good point. Not that I meant to imply it was the same. Thanks for clarifying


MET Issues & Misc tasks

Description

[ ] Remove hashes in MET nav for IDN website
[x] Fix squished circles in CARE for Chrome
[ ] Fix modal scrolling
[ ] Fix form validation

[ ] Distribution
[x] WKT

[x] Form unloads on import
[x] Loading example disables bottom buttons
[ ] CARE scores aren't updating for custodian
[x] Auto save to localstorage, remove draft buttons
[x] Change Export to Save As

All of these will be resolved in https://github.com/idn-au/idn-metadata-creator/issues/32


Add access to --embed-thumbnail for audio files

From @philipzae on Jul 11 2016

Similar to the 'write thumbnail to disk' option in the Filesystem tab, an option to embed a thumbnail in the audio file would be most useful and it is supported in m4a files as long as atomicparsley is installed.

Original issue: https://github.com/youtube-dlg/youtube-dl-gui/issues/14

I'm with him. Would be nice to have this option.


also forward --manifest-path to 'cargo metadata'

and then I went on to refactor the argument flag splitting thing a bit, and, uh, that snowballed...

@bors r+

:pushpin: Commit a60475615d91df57907cac77d99f321fc392b96b has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit a60475615d91df57907cac77d99f321fc392b96b with merge 3ffd096c6f071b370939ae4f3e49aa54f9d4855b...

@bors r+ retry

:pushpin: Commit 08e7d945629b32b32c34d5df84d881292e6f0451 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 08e7d945629b32b32c34d5df84d881292e6f0451 with merge 938fdf82c1c8e85771481b8c4d8faa4affdb6546...

:broken_heart: Test failed - checks-actions

Oh wow. Not forwarding --manifest-path has meant we haven't actually properly been checking the benchmarks for whether they run on CI... and of course they don't run on Windows.

@bors r+

:pushpin: Commit 04744a2935ba5c3e01af396ff91e24afba212acb has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 04744a2935ba5c3e01af396ff91e24afba212acb with merge a52244252187b133b3736b37ccb8e39c23992e2a...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing a52244252187b133b3736b37ccb8e39c23992e2a to master...


refactor: toBoolean does more; narrower predicates

the toBoolean function centralizes the logic for coercing a value and allows the predicate functions to be more specific in what they validate rather than them simply being alais names to broad validation.
Closes #122
✅ Pull Request Checklist:

[x] Included link to corresponding GitHub Issue.

[x] Added/updated unit tests and storybook for this change (for bug fixes / features).
[x] Added/updated documentation (for bug fixes / features)
[x] Filled out test instructions.

📝 Test Instructions:

❓ Does this PR introduce a breaking change?

[x] Yes
[ ] No

The function toBoolean no longer will consider specific values - 'no', 'off', 'n' to be "false" or 'on', 'yes', 'y' to be "true". If that is being relied on that assumption will break; no tests show this to be extant.
💬 Other information

I was waiting to update the bench tests till after we agree on a general direction; no need to benchmark something that isn't going to move forward.

Feels like maybe we also want something that is isYes + isTrue + isOn & isNo + isFalse + isOff, like isTruthy & isFalsy?

Feels like maybe we also want something that is isYes + isTrue + isOn & isNo + isFalse + isOff, like isTruthy & isFalsy?

The converter toBoolean is the most liberal and could be combined with the more specific calls to gain those benefits I think.
I don't think it is a good idea to include all of the specific cases - yes + true + no or no + off + false - into a single call.

I decided to sever the relationship between converters/toBoolean and predicates/is-noyes because I think they are not as compatible as might appear.
First thought, for toBoolean there should only two possible return values: true or false; it should never throw an error or return null or undefined. Therefor it needs to be liberal in it's conversion and that leads to the ethos of, "anything, that is something not explicitly false, is true." The other thought I had along these lines is, "if there is a list for true and list for false and we check against those two lists, then anything not in those lists would be an alternate return value." All of this leads me to believe that there is a small set of "false" values and anything outside of that set is not false, or true; at least for the purposes and domain of this type of function.
Second, for the predicates, toBoolean becomes problematic for truthy values because it's liberal criteria conflicts with the more restrictive ethos of is* functions. For instance because toBoolean('no') will evaluate to true then isYes('no') would evaluate to true without a more complicated check than ['y', 'yes', 1, 'true].includes(val). Also, I felt that it would be better to be consistent within is-noyes function implementations as well as limit interdependencies where not prohibitive.

@kalisjoshua will need a changeset

Changeset added.


manager should handle when linodes are created beyond the account limit

When adding multiple Linodes, no attempt is made to restrict the number of Linodes to the account limit.
Linodes up to the limit will be created but no indication is given that the request for additional Linodes was ignored (or dismissed silently).
When the Linode limit has been met, the "Create Linode" button will appear to submit but not proceed nor reveal an error.

Thanks. All error handling on the create page is less than ideal since the implemetation of multiple linode creation, which makes it possible for errors to occur halfway through and makes a bit of a mess of things.

This is not just an issue with adding multiple Linodes. We should at least deal with the case where the user already has three Linodes and they try to add one more. I was confused about this for a while because there are no error responses from the API, they just don't show up at /linode/instances.


Classes and ids autocomplete

Atom should include a feature in which it also includes classes and ids ( that are already named ) in autocomplete. This would be useful for me using CSS since I am a beginner to coding. Thanks!

This belongs on the atom/autocomplete-css repo: please file it there instead, and completely fill out the issue template.


Fix search

Search now requires being logged in + a CSRF token.
This PR adds a CLI flag to provide an authentication cookie (must be obtained by logging in with a browser, in Firefox the cookie can be found in the developer toolbox under the storage tab).
It looks like a randomly generated CSRF token works, so no complicated mechanism is required to obtain one.
Fixes #11.
Fixes #13.

That sounds great! Can you tell how stable this solution will be if you run twint regularly on a daily basis, i.e., how fast will the cookie expire? Will it work when you use the cookie to run twint on a different machine/IP address?

Can you tell how stable this solution will be if you run twint regularly on a daily basis, i.e., how fast will the token/cookie expire?

No idea yet, but we run a twint job every 12 hours on github actions (https://github.com/catgirl-v/cubari/actions), so we'll find out soon enough.

That sounds great! Can you tell how stable this solution will be if you run twint regularly on a daily basis, i.e., how fast will the token/cookie expire? Will it work when you use the token/cookie to run twint on a different machine/IP address?

It's working so far.

Is working for you guys? In my case this error is popping up
"ConnectionError: Access forbidden, try passing --auth-token."

Yes, it's working. I'm gonna need more details to help you. Did you in fact pass a valid authentication cookie as per the op? If so, please post minimum example that reproduces the problem.

Do I need to pass a valid authentication cookie, how so? I just use the changes in this pr and try to execute my previous code the that error message popped up. How can I do what you recommed?

Sounds to me like you didn't read any of the conversation in #13 and here. The error message is very clear, you need an auth token. This is the whole point of this PR: Twitter now requires login to search. Instructions are in the op.

Brilliant solution, works just fine. Thanks.

Sounds to me like you didn't read any of the conversation in #13 and here. The error message is very clear, you need an auth token. This is the whole point of this PR: Twitter now requires login to search. Instructions are in the op.

My bad, I though that csrf_token = random.randbytes(16).hex() was it but I need to replace it with my auth token witch I get from Firefox browser, right? because I did make the change and I'm still having the same error ("ConnectionError: Access forbidden, try passing --auth-token."). Maybe am I doing something wrong? Some help would be nice please :)

No, you don't have to modify the code. Pass the token with the --auth-token flag, or set the TWITTER_AUTH_TOKEN environment variable.
CSRF is unrelated, it's just that both changes were required to actually get it to work.

No, you don't have to modify the code. Pass the token with the --auth-token flag, or set the TWITTER_AUTH_TOKEN environment variable.
CSRF is unrelated, it's just that both changes were required to actually get it to work.

I have my code implemented in AWS Lambda with twint's library as a layer. I update the lib and set the env variable as mentioned but I still having the same error.
[CRITICAL]	2023-05-12T20:53:44.334Z	38205fb9-65a5-41b2-b6ce-377909a1b4e3	twint.run:Twint:Feed:noData'data'
sleeping for 1.0 secs
[CRITICAL]	2023-05-12T20:53:45.425Z	38205fb9-65a5-41b2-b6ce-377909a1b4e3	twint.run:Twint:Feed:noData'data'
sleeping for 8.0 secs
[CRITICAL]	2023-05-12T20:53:53.524Z	38205fb9-65a5-41b2-b6ce-377909a1b4e3	twint.run:Twint:Feed:noData'data'
sleeping for 27.0 secs

Thank you for the fix @9ary, works great! 😃
Tiny request, is it possible to add a wait time to prevent rate limits? I got TokenExpiryException: Rate limit exceeded

For what it's worth, it seems the owner of this repo is inactive, so this PR is unlikely to be merged anytime soon. We've set up a fork at https://github.com/catgirl-v/twint.
@leonardoulloa21 @batmanscode please open issues over there with the code or command line invocation that reproduces your problems. It's not practical to do all development and troubleshooting in a single PR thread.

For what it's worth, it seems the owner of this repo is inactive, so this PR is unlikely to be merged anytime soon. We've set up a fork at https://github.com/catgirl-v/twint.
@leonardoulloa21 @batmanscode please open issues over there with the code or command line invocation that reproduces your problems. It's not practical to do all development and troubleshooting in a single PR thread.

Makes sense, thanks!

I replacd everything on the changes on the py files of my twint but i keep getting the ones below on all of my searches.
module 'random' has no attribute 'randbytes'

I replacd everything on the changes on the py files of my twint but i keep getting the ones below on all of my searches.
module 'random' has no attribute 'randbytes'

You have to use python 3.9 or above. It's mentioned in some of the early comments

Thank. I replaced it to 3.9 but I got 'Access forbidden, try passing --auth-token.' i saw that auth tokens were added. how can i add this to the search function?

Thank. I replaced it to 3.9 but I got 'Access forbidden, try passing --auth-token.' i saw that auth tokens were added. how can i add this to the search function?
i'm very new at this. How do i pass the auth token?

Login to twitter on Firefox -> developer tools -> storage -> Auth token
Then wherever you're running twint, save that as an environment variable called TWITTER_AUTH_TOKEN
It should then run. Good luck

Thank. I replaced it to 3.9 but I got 'Access forbidden, try passing --auth-token.' i saw that auth tokens were added. how can i add this to the search function?
i'm very new at this. How do i pass the auth token?

Login to twitter on Firefox -> developer tools -> storage -> Auth token
Then wherever you're running twint, save that as an environment variable called TWITTER_AUTH_TOKEN
It should then run. Good luck

Thanks it's working now.

Thank. I replaced it to 3.9 but I got 'Access forbidden, try passing --auth-token.' i saw that auth tokens were added. how can i add this to the search function?
i'm very new at this. How do i pass the auth token?

Login to twitter on Firefox -> developer tools -> storage -> Auth token
Then wherever you're running twint, save that as an environment variable called TWITTER_AUTH_TOKEN
It should then run. Good luck

Hey @batmanscode
Would you mind testing my code and tell me if you are getting the same error message?
I'm trying to run it in jupyternotebook and then in AWS Lambda.
`import twint
import os
import nest_asyncio
os.environ["TWITTER_AUTH_TOKEN"] = "my_token"
nest_asyncio.apply()
c = twint.Config()
c.Username  = "BCPComunica"
c.Since="2023-05-21"
c.Limit = 100
twint.run.Search(c)`
Hope you can give a hand!
Thanks in advanced

@JoelBird hopefully this is detailed enough:

go to twitter.com
log in
press F12, the developer toolbox will appear
click the storage tab
on the left, select cookies > https://twitter.com
find the cookie named auth_token
double-click the value and copy it

Hi, thanks for the fix. But for now, using the command line, only the -u parameter works, the search isn't work. Any idea why it isn't. I'm trying to debug it here.

I'm having issues of Rate Limit exceeded? How do i fix this? what should i keep looping to override this?

Thank. I replaced it to 3.9 but I got 'Access forbidden, try passing --auth-token.' i saw that auth tokens were added. how can i add this to the search function?
i'm very new at this. How do i pass the auth token?

Login to twitter on Firefox -> developer tools -> storage -> Auth token
Then wherever you're running twint, save that as an environment variable called TWITTER_AUTH_TOKEN
It should then run. Good luck

Hey @batmanscode
Would you mind testing my code and tell me if you are getting the same error message?
I'm trying to run it in jupyternotebook and then in AWS Lambda.
`import twint
import os
import nest_asyncio
os.environ["TWITTER_AUTH_TOKEN"] = "my_token"
nest_asyncio.apply()
c = twint.Config()
c.Username  = "BCPComunica"
c.Since="2023-05-21"
c.Limit = 100
twint.run.Search(c)`
I'm getting this error:
CRITICAL:root:twint.run:Twint:Feed:noData'data'
sleeping for 1.0 secs
Hope you can give a hand!
Thanks in advanced

I'm not sure, sorry. I'm having the same issue :(

For those that this is working for, would someone be able run through

Thank. I replaced it to 3.9 but I got 'Access forbidden, try passing --auth-token.' i saw that auth tokens were added. how can i add this to the search function?
i'm very new at this. How do i pass the auth token?

Login to twitter on Firefox -> developer tools -> storage -> Auth token
Then wherever you're running twint, save that as an environment variable called TWITTER_AUTH_TOKEN
It should then run. Good luck

Hey @batmanscode
Would you mind testing my code and tell me if you are getting the same error message?
I'm trying to run it in jupyternotebook and then in AWS Lambda.
import twint import os import nest_asyncio os.environ["TWITTER_AUTH_TOKEN"] = "my_token" nest_asyncio.apply() c = twint.Config() c.Username  = "BCPComunica" c.Since="2023-05-21" c.Limit = 100 twint.run.Search(c)
I'm getting this error:
CRITICAL:root:twint.run:Twint:Feed:noData'data'
sleeping for 1.0 secs
Hope you can give a hand!
Thanks in advanced

I'm not sure, sorry. I'm having the same issue :(

Hi all, it this still working for anyone? I'm experiencing the same issue as @leonardoulloa21

It's working fine on my end.
1672684481071976449 2023-06-25 03:13:51 +0800  @kvafelled @kvafelled se debe esperar el plazo de 15 días calendario aproximadamente.
1672663630951878656 2023-06-25 01:51:00 +0800  @kvafelled Hola, @kvafelled 👋 Te contamos que cuando se realiza una cancelación, anulación o reembolso de compra por parte de alguna empresa, estas tienen hasta 15 días (calendario) para proceder con la devolución del dinero a tu cuenta de ahorros. 🤝
...
...
...
@Maverick99210 ¡Hola, @Maverick99210! 👋 Lamentamos el inconveniente generado, por favor, envíanos tu DNI vía DM para poder orientarte de la mejor manera. Esperamos tu mensaje.
1668754764476346368 2023-06-14 06:58:34 +0800  @mendezt_29 Hola @mendezt_29 Envíanos un DM con la captura de pantalla de lo que te aparece y el número de tu DNI. Quedamos atentos.
1668710970741628928 2023-06-14 04:04:33 +0800  @PALICUYA ¡Hola Pilar!👋  Por favor envíanos un inbox con tu DNI y la imagen que te aparece aquí 👉🏻  https://t.co/HE00YFfJez. Quedamos atentos. 🤝
1668690239685267457 2023-06-14 02:42:10 +0800  @vladineitor Elsa, gracias por la información. Estamos reportando lo sucedido al equipo a cargo, para que se pueda hacer las consultas y verificaciones al respecto. Lamentamos mucho la molestia
1668687790408884224 2023-06-14 02:32:26 +0800  @vladineitor Hola, Elsa. Queremos conocer lo ocurrido. Por favor, detállanos vía DM el inconveniente presentado y la ubicación de la Agencia (avenida/calle/número/alguna referencia). Quedamos atentos.
1668631670076116997 2023-06-13 22:49:26 +0800  @RodrigoVinyas ¡Hola, Rodrigo! 👋 Nos importa mucho la experiencia de cada uno de nuestros clientes, agradeceríamos que puedas comunicarte al 01 311 9400 de L-V  de 7:00 a.m. a 5:00 p.m. con nuestra área de Soluciones de Pagos, para solicitar alguna facilidad de pago o un compromiso de pagos.
[!] No more data! Scraping will stop now.
found 0 deleted tweets in this search.

For those that this is working for, would someone be able run through

Thank. I replaced it to 3.9 but I got 'Access forbidden, try passing --auth-token.' i saw that auth tokens were added. how can i add this to the search function?
i'm very new at this. How do i pass the auth token?

Login to twitter on Firefox -> developer tools -> storage -> Auth token
Then wherever you're running twint, save that as an environment variable called TWITTER_AUTH_TOKEN
It should then run. Good luck

Hey @batmanscode
Would you mind testing my code and tell me if you are getting the same error message?
I'm trying to run it in jupyternotebook and then in AWS Lambda.
import twint import os import nest_asyncio os.environ["TWITTER_AUTH_TOKEN"] = "my_token" nest_asyncio.apply() c = twint.Config() c.Username  = "BCPComunica" c.Since="2023-05-21" c.Limit = 100 twint.run.Search(c)
I'm getting this error:
CRITICAL:root:twint.run:Twint:Feed:noData'data'
sleeping for 1.0 secs
Hope you can give a hand!
Thanks in advanced

I'm not sure, sorry. I'm having the same issue :(

Hi all, is this still working for anyone? I'm experiencing the same issue as @leonardoulloa21

It's working fine on my end.
Output:
1672684481071976449 2023-06-25 03:13:51 +0800 @kvafelled @kvafelled se debe esperar el plazo de 15 días calendario aproximadamente.
1672663630951878656 2023-06-25 01:51:00 +0800 @kvafelled Hola, @kvafelled 👋 Te contamos que cuando se realiza una cancelación, anulación o reembolso de compra por parte de alguna empresa, estas tienen hasta 15 días (calendario) para proceder con la devolución del dinero a tu cuenta de ahorros. 🤝
...
...
...
@Maverick99210 ¡Hola, @Maverick99210! 👋 Lamentamos el inconveniente generado, por favor, envíanos tu DNI vía DM para poder orientarte de la mejor manera. Esperamos tu mensaje.
1668754764476346368 2023-06-14 06:58:34 +0800 @mendezt_29 Hola @mendezt_29 Envíanos un DM con la captura de pantalla de lo que te aparece y el número de tu DNI. Quedamos atentos.
1668710970741628928 2023-06-14 04:04:33 +0800 @PALICUYA ¡Hola Pilar!👋 Por favor envíanos un inbox con tu DNI y la imagen que te aparece aquí 👉🏻 https://t.co/HE00YFfJez. Quedamos atentos. 🤝
1668690239685267457 2023-06-14 02:42:10 +0800 @vladineitor Elsa, gracias por la información. Estamos reportando lo sucedido al equipo a cargo, para que se pueda hacer las consultas y verificaciones al respecto. Lamentamos mucho la molestia
1668687790408884224 2023-06-14 02:32:26 +0800 @vladineitor Hola, Elsa. Queremos conocer lo ocurrido. Por favor, detállanos vía DM el inconveniente presentado y la ubicación de la Agencia (avenida/calle/número/alguna referencia). Quedamos atentos.
1668631670076116997 2023-06-13 22:49:26 +0800 @RodrigoVinyas ¡Hola, Rodrigo! 👋 Nos importa mucho la experiencia de cada uno de nuestros clientes, agradeceríamos que puedas comunicarte al 01 311 9400 de L-V de 7:00 a.m. a 5:00 p.m. con nuestra área de Soluciones de Pagos, para solicitar alguna facilidad de pago o un compromiso de pagos.
[!] No more data! Scraping will stop now.
found 0 deleted tweets in this search.

Thanks @corpuzdonn, maybe it's a token issue from my end
I attempted a huge scrape (4 weeks via search terms) and that got rate limited. Maybe that token wasn't valid after that
Have you tried long scrapes? I saw there's a time out parameter but even setting that very high didn't work for me

For those that this is working for, would someone be able run through

Thank. I replaced it to 3.9 but I got 'Access forbidden, try passing --auth-token.' i saw that auth tokens were added. how can i add this to the search function?
i'm very new at this. How do i pass the auth token?

Login to twitter on Firefox -> developer tools -> storage -> Auth token
Then wherever you're running twint, save that as an environment variable called TWITTER_AUTH_TOKEN
It should then run. Good luck

Hey @batmanscode
Would you mind testing my code and tell me if you are getting the same error message?
I'm trying to run it in jupyternotebook and then in AWS Lambda.
import twint import os import nest_asyncio os.environ["TWITTER_AUTH_TOKEN"] = "my_token" nest_asyncio.apply() c = twint.Config() c.Username  = "BCPComunica" c.Since="2023-05-21" c.Limit = 100 twint.run.Search(c)
I'm getting this error:
CRITICAL:root:twint.run:Twint:Feed:noData'data'
sleeping for 1.0 secs
Hope you can give a hand!
Thanks in advanced

I'm not sure, sorry. I'm having the same issue :(

Hi all, is this still working for anyone? I'm experiencing the same issue as @leonardoulloa21

It's working fine on my end.
Output:<PHONE_NUMBER>071976449 2023-06-25 03:13:51 +0800 @kvafelled @kvafelled se debe esperar el plazo de 15 días calendario aproximadamente.<PHONE_NUMBER>951878656 2023-06-25 01:51:00 +0800 @kvafelled Hola, @kvafelled 👋 Te contamos que cuando se realiza una cancelación, anulación o reembolso de compra por parte de alguna empresa, estas tienen hasta 15 días (calendario) para proceder con la devolución del dinero a tu cuenta de ahorros. 🤝 ... ... ... @Maverick99210 ¡Hola, @Maverick99210! 👋 Lamentamos el inconveniente generado, por favor, envíanos tu DNI vía DM para poder orientarte de la mejor manera. Esperamos tu mensaje.<PHONE_NUMBER>476346368 2023-06-14 06:58:34 +0800 @mendezt_29 Hola @mendezt_29 Envíanos un DM con la captura de pantalla de lo que te aparece y el número de tu DNI. Quedamos atentos.<PHONE_NUMBER>741628928 2023-06-14 04:04:33 +0800 @PALICUYA ¡Hola Pilar!👋 Por favor envíanos un inbox con tu DNI y la imagen que te aparece aquí 👉🏻 https://t.co/HE00YFfJez. Quedamos atentos. 🤝<PHONE_NUMBER>685267457 2023-06-14 02:42:10 +0800 @vladineitor Elsa, gracias por la información. Estamos reportando lo sucedido al equipo a cargo, para que se pueda hacer las consultas y verificaciones al respecto. Lamentamos mucho la molestia<PHONE_NUMBER>408884224 2023-06-14 02:32:26 +0800 @vladineitor Hola, Elsa. Queremos conocer lo ocurrido. Por favor, detállanos vía DM el inconveniente presentado y la ubicación de la Agencia (avenida/calle/número/alguna referencia). Quedamos atentos.<PHONE_NUMBER>076116997 2023-06-13 22:49:26 +0800 @RodrigoVinyas ¡Hola, Rodrigo! 👋 Nos importa mucho la experiencia de cada uno de nuestros clientes, agradeceríamos que puedas comunicarte al 01 311 9400 de L-V de 7:00 a.m. a 5:00 p.m. con nuestra área de Soluciones de Pagos, para solicitar alguna facilidad de pago o un compromiso de pagos. [!] No more data! Scraping will stop now. found 0 deleted tweets in this search.

Would you mind packling up your twint library and share it to us, please! I might be doing something wrong because I have just tried it and I got the same result :
CRITICAL:root:twint.run:Twint:Feed:noData'data'
sleeping for 1.0 secs
I don't think that this message is related to the auth token, it has to be something else...
Thanks in advanced for your time @woluxwolu

I am actually getting the following below all of a sudden. Did something change?
CRITICAL:root:twint.get:User:Expecting value: line 1 column 1 (char 0)

Most likely the latest Twitter changes require more API calls to be authenticated. Our scripts broke too but I'm currently on vacation. I'll have a look in a few days.

Has there been any updates.  This means twint will need to be reworked to use the graphql API, which is a lot more work than I'm willing to put in personally.

 This means twint will need to be reworked to use the graphql API, which is a lot more work than I'm willing to put in personally.

I see. It's ok. Will find alternative solutions. Thanks for your hard work!


add more flexible command

Like in GDB, the list command to list source code in current context,and also some short commands for the respective long commands.
really appreciate your work!

Closing - open #58 to deal with this.


Add-on not starting

Add on refusing to start. Initially worked, but now the following error:
[init        ] container is starting...
[cont-env    ] loading container environment variables...
[cont-env    ] APP_NAME: loading...
/init: line 65: can't create /var/tmp/.cont-env-internal: nonexistent directory

Which version? Normal, Edge? Did you try to uninstall and reinstall it again?

Owner

both versions, uninstalled, restarted reinstalled, same error


[Feature Request]: add a faq modal

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Add an FAQ modal to the footer section of the website to provide users with quick access to common questions and answers. This feature will improve user experience by offering immediate support and information.
Use Case
Add an FAQ modal to the footer section of the website to provide users with quick access to common questions and answers. This feature will improve user experience by offering immediate support and information.
Benefits
No response
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

can you assign me this issue.i have worked on this before and im a beginner.


chore(KarmaConf): read CLI browser option to run tests

Hi,
This PR (re)makes it possible to specify a browser when executing a test command (e.g. npm run test-nolint -- --ff for Firefox).
So that it is easier to run tests in local browser (other than PhantomJS).
This CLI argument browser option disappeared when Jake has been removed from scripts.
This PR will make it easier to locally debug to re-enable PR's https://github.com/Leaflet/Leaflet/pull/5845 and https://github.com/Leaflet/Leaflet/pull/5831

mmm I got caught by the same gotcha as in my PR:
https://github.com/ghybs/Leaflet/blob/ecd1c009ce4420f1d96b285343bc42560431eb19/spec/karma.conf.js#L134-L136
=> use either npm test -- -- --browsers Firefox (twice -- flag before --browsers)
or npm run test-nolint -- --broswsers Firefox
…because of the nested npm script, we have to use multiple times the end of args flag "--"
Thanks for the quick review!

https://www.contributor-covenant.org/version/1/3/0/code-of-conduct.html


[WIP] Prototype implementation of queue-per-model scheduler

This is a prototype implementation of the queue-per-model scheduler (design doc). This PR should not be as stands, but should instead be broken up into distinct components that can be merged individually.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/32/
Test FAILed.

jenkins test this please

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/34/
Test FAILed.

test this please

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/35/
Test FAILed.

The effort to implement this is documented in CLIPPER-91.


IE9 compatibility

Hi,
The vaadin plupload plugin shows an "INIT_ERROR" on startup in IE9.  No error in server console but in browser debug console I see this:
455ms   28 (class pl.exsio.plupload.client.PluploadConnector) :
455ms      pl.exsio.plupload.client.PluploadServerRpc.error([{"code":-500,"message":"Init error."}])
456ms      pl.exsio.plupload.client.PluploadServerRpc.destroy([])
If I remember well, the plupload plugin has multiple choices to support "older" browsers without HTML5 support.  Is there a way to choose the underlying technology of the Plupload plugin ?
Regards,
Frank

Hi
Thank You for the interest in the addon. As for IE9 compatibility, You are correct that the plupload library itself can work with environments other than html5 (html4, flash, silverlight). Unfortunately I wasn't able to make the Vaadin Connector to work with them - the file selection window wouldn't show, so I restricted the compatibility to html5 compliant browsers only. If however You'd find a way for this to work with environments other, than html5, that would be great and I surely would welcome a PR for that :)

I think for old browsers will be nice option to add support for HTML4 Runtime.
It will be much better when getting now INIT_ERROR


How to add other login modes in the authorization code mode? For example: email verification code?

The client obtains an authorization code

User authentication: user name and password or email verification code or SMS verification code or Google

OAuth authorization, select the authorization scope

The client site obtains the authorization code

The client site uses the authorization code and client password to obtain the token

Closing as duplicate of gh-534


model_type: 'minicpm-v-v2_5-chat' is not registered.

I do pip install 'ms-swift[llm]' -U and still get an error: model_type: 'minicpm-v-v2_5-chat' is not registered.

git clone https://github.com/modelscope/swift.git
cd swift
pip install -e '.[llm]'

I tried to do that, but get a new error: TypeError: Subscripted generics cannot be used with class and instance checks

git clone https://github.com/modelscope/swift.git cd swift pip install -e '.[llm]'

I tried to do that, but get a new error: TypeError: Subscripted generics cannot be used with class and instance checks

I use the new code and meet the same problem. Even i can't train minicpm-v-v2

I use the new code and meet the same problem. Even i can't train minicpm-v-v2

i solved this by update the transformers version ==4.41.0

I use the new code and meet the same problem. Even i can't train minicpm-v-v2

i solved this by update the transformers version ==4.41.0

Have you solved it? I still seem to get dependency conflicts here.

emmm,is effective for me,maybe other reason? I try to pull the new code and retry the installation steps, then update the version of transformers. I'm not sure which step takes effect

This is a follow-up to #1822 and #100


Night Mode is not working on 44

I did upgrade to 44 yesterday. While HDR does work fine (yay!), Night mode did not apply at all. It does not matter if i disable hdr, reenable night mode or whatever. I flipped switches for like 5 min and no night mode showed up. Another reboot did not help too.
Please make it working again <3

Can confirm - does not work on my ally.

No - enabling that option does not change anything.

Thanks, I will wait for an official fix :)

I have been able to reproduce and narrow it down to gamescope. Both -git and latest -plus have issues. 3.11.51 from the arch repos work for me.
You could try downgrading gamescope. But you'll lose the -plus changes (mostly streaming, resolution fixes and intel support)
I'll see if I can bisect

What’s the status on this?

Bug is still present on 45 unstable, tested on Legion Go.
Still works on 43-1, which I'm running on a GPD Win 4.

It is fixed for me on unstable now. Can you verify?

Installed chimeraOS unstable earlier today on my Legion Go, I had to enable Use Native Color Temperature under the Display settings. Afterwards, Night mode seemed to be working as-expected.


COMP: Consistently propagate CMAKE_MACOSX_RPATH to all projects

With update of the minimum required version of CMake to 3.5.0 in Slicer,
default value for CMAKE_MACOSX_RPATH changed to ON. This commit makes
sure the value passed by the Slicer extension build system is used
consistently.
This commit associated with the use of Slicer >= r26305 [1]
will prevent error like the following:
Error copying file<EMAIL_ADDRESS>to "/.../SPHARM-PDM-build/SPHARM-PDM-install/Slicer.app/Contents/lib/Slicer-4.7/libitkminc2-4.12.1.dylib".

Related Slicer discussions:
https://discourse.slicer.org/t/interpreting-cdash-reporting-extension-is-packaged-despite-build-errors/856
https://discourse.slicer.org/t/extension-build-failure-on-mac-due-to-error-copying-files/935
[1] r26305 COMP: Update ITK and CTK to fix remaining extension packaging issues
http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=26305

Closing. Superseded by #23


Valid action in script is reported as unknown for an automation which uses that script

What version of Spook are you using?
3.1.0
What version of Home Assistant are you using?
2024.11.1 (it was also present in 2024.10.x)
The problem
I get this error:

The automation "🔥 Turn on Kettle" (automation.claire_near_home) has an unknown action: water_heater.set_temperature.

I see two issues with this spook:

The action water_heater.set_temperature shouldn't be unknown, it is a valid action
This automation doesn't use the water_heater.set_temperature action, however it does call a script which contains this action.

Automation YAML (automation.claire_near_home):
alias: 🔥 Turn on Kettle
description: ""
mode: single
triggers:
  - entity_id: person.claire
    zone: zone.near_home
    event: enter
    id: near_home
    trigger: zone
  - type: opened
    device_id: 577747254dfd42d9a9e9ce289f6e36dd
    entity_id: binary_sensor.master_bedroom_door_contact
    domain: binary_sensor
    id: first_coffee
    trigger: device
  - entity_id:
      - sensor.claire_s_s24_ultra_charger_type
    id: first_coffee
    to: none
    trigger: state
conditions:
  - condition: or
    conditions:
      - condition: and
        conditions:
          - condition: trigger
            id: near_home
          - condition: time
            after: "07:30:00"
            before: "19:00:00"
      - condition: and
        conditions:
          - condition: trigger
            id: first_coffee
          - condition: time
            before: "09:00:00"
            after: "07:00:00"
          - type: is_open
            condition: device
            device_id: 577747254dfd42d9a9e9ce289f6e36dd
            entity_id: binary_sensor.master_bedroom_door_contact
            domain: binary_sensor
          - condition: zone
            entity_id: person.claire
            zone: zone.home
          - condition: state
            entity_id: switch.kitchen_kettle
            state: "off"
            for:
              hours: 2
              minutes: 0
              seconds: 0
          - condition: state
            entity_id: sensor.claire_s_s24_ultra_charger_type
            state: none
actions:
  - data: {}
    action: script.turn_kettle_on

Script YAML (script.turn_kettle_on)
alias: Kettle On
sequence:
  - alias: Set Temp based on time of day
    if:
      - condition: time
        after: "00:00:00"
        before: "12:00:00"
    then:
      - metadata: {}
        data:
          temperature: 90
        target:
          entity_id: water_heater.kitchen_kettle
        action: water_heater.set_temperature
      - metadata: {}
        data: {}
        target:
          entity_id: switch.kitchen_kettle_keep_warm
        action: switch.turn_on
    else:
      - metadata: {}
        data:
          temperature: 100
        target:
          entity_id: water_heater.kitchen_kettle
        action: water_heater.set_temperature
  - metadata: {}
    data: {}
    target:
      entity_id: switch.kitchen_kettle
    action: switch.turn_on
mode: restart
icon: mdi:kettle-steam

Anything in the logs? Paste it here!
No response

The action water_heater.set_temperature shouldn't be unknown, it is a valid action

Unfortunately, it seems like the water heater integration isn't loaded, otherwise it would have been a valid action. The reason it raises, is because it is referenced, but not available on your system. Just because the action is documented, doesn't mean it is available on your instance.

This automation doesn't use the water_heater.set_temperature action, however it does call a script which contains this action.

If it isn't used... remove it? 🤷
Closing this issue, as it sounds like it works as intended.
../Frenck

The kettle does use that integration though? So it should be loaded?

What I mean by it doesn't use the water_heater.set_temperature action, it's not directly referenced in the automation configuration.


Fix up tests and add test case for https://bugs.swift.org/browse/SR-3886

@swift-ci Please test

@swift-ci Please test

@dcci OK, the first patch still makes sense though.

@swift-ci Please test

sure, LG.

@slavapestov Presumably yesterday night I was too sleepy and I didn't notice this. "Re-enable some tests that weren’t being run", what lets you think they weren't being run?

Because, to the best of my knowledge, they're run when you build with the Cmake build. And I'm under the impression that darwin and system-darwinare actually synonyms in lit context, but don't quote me on that, I might be wrong.


Fixes

Behavior of game play. New cards should be dealt every round.
As a result of 1, check that we have at least 3 cards for next round.
Removed the extra feature I added, so the program behaves like original

Excellent thank you!


rusticl testing

mesa-libOpenCL was added to davincibox as of #67 and so rusticl is now available as an opt-in option for testing.
ROCm has given me plenty of issues in the past and recently. As of Mesa 24.1.1, rusticl seems to handle DaVinci Resolve fine on my system (though it doesn't handle Fusion too well).
I want to evaluate making rusticl the default for AMD & Intel users, but I need wider testing than just my own results.
Testers, please check the following:
Run DaVinci Resolve with:
Distrobox: distrobox enter -n davincibox -- /usr/bin/run-davinci rusticl
Toolbox: toolbox run -c davincibox /usr/bin/run-davinci rusticl
Nvidia
Check whether running with the above command prevents using Resolve with CUDA as normal
Intel
Davincibox ships the intel-compute-runtime for Intel GPUs. Follow the same testing steps as the AMD section below to test rusticl on Intel.
AMD
Compare your experience with ROCm (the current default) to your experience with using rusticl. Note that when switching between the two, you may get a popup about needing to update your GPU configuration, and sometimes the switch may not seem to work at first. Just re-launch Resolve if the latter case happens.
Please report any notable differences in DaVinci Resolve usage that you think are relevant.
Some examples, including but not limited to:

Timeline performance, especially with VFX added
Rendering performance
Any differences in app stability

in my intel platform it was unusable , but after adding the rusticl it run properly without gpu pop up panel.
i did not test further to see GPU usage etc ... , plz if needed give me a relevent informations "guide" to share the results of the intel platform

As rusticl is now the default for AMD GPUs and intel-compute-runtime seems to work for Intel, I'm closing this issue in favor of more specific issue reports in the future.


STI support: ability to config a model as a leaf

Imagine you have a tree, wich is constructed from Node models and their inheritors Leaf models.
So the Leaf model should not have any descendants, so it would be great to add an option which will remove all the descendant related methods from it.

Feel free to reopen with a pull request!


upgrade pytorchvideo to 0.1.2

What does this PR do?

Fixes # (issue)
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests? [not needed for typos/docs]
[x] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[x] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

hi @tchaton, pls review the PR for pytorchvideo version upgrade.

Sure @ethanwharris, I'll check it today evening

Hi @ethanwharris, the test was failing due to annotation argument is required in ClipSampler. Not sure if this was there in previous version.
I looked at the source code of pytorchvideo but the annotation arg is not being used by any of the implemented Sampler 😀


DSpace 7 community and collection statistics are inaccurate

Describe the bug
Statistics displayed on the "Statistics" page of a community or collection are inaccurate (in my opinion). They seem to only reflect views of the community or collection page itself, rather than counting child communities, collections, items, and bitstreams.
To Reproduce
Steps to reproduce the behavior:

View the "Statistics" page for a community or collection, take note of the UUID
Use Solr to perform a search for statistics records directly belonging to that UUID, for example: id:d09e6100-df60-4280-9ff3-f3fea5ea4e6b. These are views of the community/collection page itself.
Use Solr to perform a search for statistics records belonging to children of that UUID, for example: owningComm:d09e6100-df60-4280-9ff3-f3fea5ea4e6b (or owningColl for a collection).

Expected behavior
The number of "views" of a community or collection should take into account all children of the community or collection, in addition to the community page itself. We already have the owningComm and owningColl fields in Solr managed by DSpace. We can use those to show more accurate statistics than we are currently doing, without much work.
For example, our DSpace repository has years of Solr statistics and currently shows this for the simple id query of a large community:
$ curl 'http://localhost:8983/solr/statistics/select?q=id:d09e6100-df60-4280-9ff3-f3fea5ea4e6b&rows=0'
{
...
  "response":{"numFound":2543,"start":0,"numFoundExact":true,"docs":[]
}}

But really there are many, many more if we include the child communities, collections, item pages, and bitstreams via the owningComm:
$ curl 'http://localhost:8983/solr/statistics/select?q=owningComm:d09e6100-df60-4280-9ff3-f3fea5ea4e6b&rows=0'
{
...
  "response":{"numFound":6461926,"start":0,"numFoundExact":true,"docs":[]
}}

In addition to this, at our institution we disaggregate statistics into "views" and "downloads" using the type field (see DSpace Constants¹):

A "view" is something that is an HTML landing page for a community, collection, or item (types 2, 3, 4)
A "download" is something that is a "view" of a bitstream (type 0) where the bundleName:ORIGINAL

So in Solr I can see this community had:

1872043 "views"
2582247 "downloads"

You can see how we count views and downloads in our standalone dspace-statistics-api indexer.
Related work

#9261

¹ See dspace-api/src/main/java/org/dspace/core/Constants.java

This is a duplicate of #8572.


Adds a rake tasks to soft delete all unissued moves past their date

Includes:

Migration to add deleted_at field to moves
Add default scope to moves to only return non-deleted moves by default
Add service to delete all unissued moves past their date
Add rake task to call the service

Tested on staging :shipit:


fix(dcd): update to OpinionSiteLinear

Solves #1019
Updating base class to OpinionSiteLinear makes code shorter and cleaner, also solves the following bug:
InsanityException caused by unexpected extra line of text in case name cell: "LEAVE OF COURT REQUIRED FOR FILINGS.", which inflated the case_name count by 1

So adding support for all optional fields to OpinionSiteLinear breaks the tests.local.test_ScraperExampleTest because fields are collected now, that were silently ignored before, on other sites such as fla with the disposition field. Will make a separate issue / PR to solve that and tag you when this one is ready

@flooie please check #1028 and the associated PR , to put the getters on OpinionSiteLinear

@grossir I think this needs to be split into two separate PRs.
The tests are failing because the addition of the optional getters is now causing other court tests to fail because they were failing silently before.  Now that you are fixing the silent fail - we should fix all those tests.  Those though should be handled in a separate PR.  Then lets fix DCD

@grossir We need to merge this: https://github.com/freelawproject/juriscraper/pull/1029 before this PR?

@quevon24 please check again, I just separated the fix for dcd.
The missing fields will be collected on the other PR


Fix admonition icons in AsciiDoc documentation

An example text like this:
NOTE: Note

WARNING: Warning

TIP: Tip

CAUTION: Caution

IMPORTANT: Important

Now displays as:

Closes #371

LGTM, will merge this when we're doing 21.0.0 as it'll need the updated guides.


[site request] bittube.co

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.10.26. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[X ] I've verified and I assure that I'm running youtube-dl 2016.10.26

Before submitting an issue make sure you have:

[ X] At least skimmed through README and most notably FAQ and BUGS sections
[X ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[X ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue
Please add bittube.co to supported site
If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2016.10.26
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

https://www.bittube.co/video/1384/hillary-promotes-white-genocide-meme
Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

That's already supported by the generic handler:
$ youtube-dl -v "https://www.bittube.co/video/1384/hillary-promotes-white-genocide-meme"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-v', 'https://www.bittube.co/video/1384/hillary-promotes-white-genocide-meme']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.10.26
[debug] Git HEAD: c725333
[debug] Python version 3.5.2 - Linux-4.8.4-1-ARCH-x86_64-with-arch
[debug] exe versions: ffmpeg 3.1.5, ffprobe 3.1.5, rtmpdump 2.4
[debug] Proxy map: {}
[generic] hillary-promotes-white-genocide-meme: Requesting header
WARNING: Falling back on generic information extractor.
[generic] hillary-promotes-white-genocide-meme: Downloading webpage
[generic] hillary-promotes-white-genocide-meme: Extracting information
[download] Downloading playlist: None
[generic] playlist None: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[debug] Invoking downloader on 'https://www.bittube.co/mobile_src.php?id=1384'
[download] Destination: Hillary Promotes White Genocide Meme-hillary-promotes-white-genocide-meme.mp4
[download] 100% of 4.09MiB in 00:05
[download] Finished downloading playlist: None


Swagger support

Castle Mock should allow the user to import Swagger files which can be used to create and define a REST interface.

Swagger has been implemented and will be part of 1.10 release


Entries in .gitignore are applied to the wrong base directory

Thank you very much for ag!
I have an issue similar to #1318:
Reproduction instructions:
mkdir root
cd root
echo 'root/subdir' > .gitignore
mkdir subdir
echo 'string to look for: shibboleth' > subdir/test.txt

Situation: I can find the string shibboleth with ag if I run ag in the current directory:
$ tree
.
└── subdir
    └── test.txt

2 directories, 1 file

$ ag shibboleth .
subdir/test.txt
1:string to look for: shibboleth

But when I search for it in the same directory specified as ../root, then it does not find the string:
$ ag shibboleth ../root
[1]    93249 exit 1     ag shibboleth ../root

But it should have found the string.
I suspect that the entries in .gitignore are applied relative to the "base directory" that is given to ag as the parameter. The entry root/subdir does not match any files/dirs within the root directory, but it seems that ag is using it relative to the "base directory", so searching within ../root makes ag construct the path ../root/subdir and ignores it.
It's really odd!
Is there anything else I can help with to get this issue solved? Thanks a lot!

Seems I found the same issue described in https://github.com/cielavenir/silversearcher-parent-directory-ignored


PortableImageSupervised.predict is slow with many images

Bug Description (maybe feature enhancement.)
PortableImageSupervised.predict method on the cifar10 test dataset is a bit slower when test dataset size is bigger. The number of images in the dataset is 10000.
Maybe predict should be run on gpu like the code in the url above?
https://github.com/jhfjhfj1/autokeras/compare/master...sammy-yusuke:0e71db8dab4841a577c7943daf4bfdb19acbfc0c
Reproducing Steps

Place all files in the URL below into some dir
https://gist.github.com/sammy-yusuke/bc8aa440d2fdbe3908198fdcf82442e2
Train model

python train_cifar10.py --timelimit-h 0.5 --dataset cifar10

Test model

python test_cifar10.py --training-output-path ~/nas_try/${trained_date}

Expected Behavior
Step 3 should finish in at least 30 min.
Setup Details
Include the details about the versions of:

OS type and version: Description:    Ubuntu 16.04.5 LTS
Python: 3.6.6
autokeras: master
scikit-learn:0.19.1
numpy:1.14.5
keras:2.2.2
scipy:1.1.0
tensorflow:1.10.0
pytorch:0.4.1

Additional context
I was just trying to see all the results of all searched models by autokeras.

@satyakesav Would you please fix this issue?
I think it can be solved by moving this line https://github.com/jhfjhfj1/autokeras/blob/master/autokeras/image/image_supervised.py#L279
to the predict() function.


Any plan to include more product types?

Firstly, thank you very much for the official homebridge plugin from tuya developer, the community really appreciates the hard work.
However, since the current state is only supporting Outlets and Lights, are there any plan to include the covers, pet-feeder, valve, door lock, and all other broad range of products from Tuya? It looks really promising and hopefully it will gain support on more devices the soonest.
If it's getting support on wider range of products, I might really have to consider switching to Tuya Platform on many of my incoming smart home devices.
Thank you.

Thank you for your feedback. We will support nearly 12 categories of Tuya devices in the next version by the end of May. Please look forward to it.

Thank you for your feedback. We will support nearly 12 categories of Tuya devices in the next version by the end of May. Please look forward to it.

Any possibility you could review all of those 12 product types that would be available by the end of may?
Thanks.

It's already on schedule.

We showed you how to expand Accessory in the README, so if you're interested, you can join us.

Any possibility to see blind motor at the end of may?
This plug-in can show also Smart life ?

It's already on schedule.

I think the question was which category of devices will be supported in the 12?

Any update on the new products?

Any possibility to see blind motor at the end of may?

You can already do this with the other Tuya plugins… but I agree - having an officially supported one would be ideal.

Hi @codyc1515 @Aekung @Stetocasa , we have released a new version of Tuya homebridge plugin two days ago, add more products support:

Light: Supports Tuya Wi-Fi light devices.
Outlet: Supports Tuya Wi-Fi Outlet devices.
Smoke Sensor: Supports Tuya smoke sensor devices.
Switch: Supports Tuya switch devices.
Heater: Support Tuya heater devices.
Garage Door: Support Tuya smart garage door devices.
Fan: Support Tuya fan devices.
Air Purifier: Support Tuya air purifier devices.

Please check the github repo and try it. You are welcome to help develop more drivers to support more Tuya PBT products by following this tutorial: https://developer.tuya.com/en/demo/tuyahomebrigeplugin, thanks!

Thanks for quoting, really appreciated.
Unfortunately for me I just have a blind motor in Tuya.
Awaiting future implementation because Tuya web plug-in has an issue with motor status

Will Oil diffusers/Humidifiers like deviceID:<PHONE_NUMBER>f3eb40ba82 (productID: nOtxbOfJNNVSatZD) be supported?
In the IoT Platform is shown as "Other" product type but HomeKit does support this kind of product as Humidifier.
Thanks!

Great to see the expanded category of accessories. I'm still looking forward to the blind and winder cover type.
Hope it would be added in the next version.
PS. Thanks for mentioning about the update.

Hi @bosco90 , thanks for the feedbacks, as the Humidifiers is in our standard instruction set list: https://developer.tuya.com/en/docs/iot/f?id=K9gf45rjel6x5, we will add it to our new product support roadmap. If you are interested, you can follow this tutorial Use and Develop Tuya Homebridge Plugin to develop the plugin driver together.
Hi @Aekung ，we will also consider to support blind and winder cover type too, thanks for the feedbacks!

Hi @bosco90 ,We have already supported the Outlet Accessory with the category name of 'cz' , which supports the power off and on.

Thats great, but both of them doesn't show up in HomeKit, I've checked the log file but this tropic is not appropriate for this, I'll open a new issue with the details.

Hi guys,
Are there any plan for the window coverings type? I wish to switch to this plugin but right now I'm still sticking with Tuya Lan and Tuya Web since it offers blinds and window covering type.
Really looking forward to the support of this category.

Hi, are you planning to support WiFi door sensors in the near future?

Hi, are you planning to support WiFi door sensors in the near future?

Hi @PeterKiwi ,We already support the Door Window(category:mcs) Sensor in the latest version.

Okay thanks.  I'll take another look at my end

Can you'll please provide an ETA for Tuya Door Locks .... These products are hot selling products in the market. Would appreciate some information.

@zlinoliver to include pet-feeder support ?


[BottomNavigationBar] Icons not resizing on iOS

Which platform(s) does your issue occur on?

iOS
emulator and device

Please, provide the following version numbers that your issue occurs with:

CLI: 8.2.3
Cross-platform modules: 8.2.5
-Package.json

{
  "name": "TinaAppV2",
  "main": "./src/main.ts",
  "version": "1.0.0",
  "private": true,
  "dependencies": {
    "@angular/animations": "~13.2.0",
    "@angular/common": "~13.2.0",
    "@angular/compiler": "~13.2.0",
    "@angular/core": "~13.2.0",
    "@angular/forms": "~13.2.0",
    "@angular/platform-browser": "~13.2.0",
    "@angular/platform-browser-dynamic": "~13.2.0",
    "@angular/router": "~13.2.0",
    "@nativescript-community/ui-material-bottomnavigationbar": "^7.0.25",
    "@nativescript-community/ui-pulltorefresh": "^2.5.1",
    "@nativescript/angular": "^13.0.0",
    "@nativescript/core": "~8.2.2",
    "@nativescript/datetimepicker": "^2.1.9",
    "@nativescript/firebase-core": "^2.2.1",
    "@nativescript/firebase-crashlytics": "^2.2.1",
    "@nativescript/firebase-database": "^2.2.1",
    "@nativescript/firebase-messaging": "^2.2.1",
    "@nativescript/firebase-messaging-core": "^2.2.1",
    "@nativescript/geolocation": "^8.0.2",
    "@nativescript/iqkeyboardmanager": "^2.0.0",
    "@nativescript/mlkit-barcode-scanning": "^1.0.5",
    "@nativescript/mlkit-core": "^1.0.5",
    "@nativescript/theme": "~3.0.2",
    "@nstudio/nativescript-checkbox": "^2.0.5",
    "@nstudio/nativescript-floatingactionbutton": "^3.0.4",
    "nativescript-forgm-sound": "^1.1.3",
    "nativescript-ui-chart": "^9.0.2",
    "nativescript-ui-sidedrawer": "^10.0.2",
    "rxjs": "~7.5.0",
    "zone.js": "~0.11.5"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "^13.3.6",
    "@angular/compiler-cli": "~13.2.0",
    "@nativescript/android": "8.2.2",
    "@nativescript/ios": "8.2.3",
    "@nativescript/types": "~8.2.0",
    "@nativescript/webpack": "~5.0.6",
    "@ngtools/webpack": "~13.2.0",
    "typescript": "~4.5.5"
  }
}

Please, tell us how to recreate the issue in as much detail as possible.
I upgraded my project from NativeScript 6 to 8 and I had the bottom navigation component and I saw that I have to use this plugin in order for it to work.
I have a home page where I group all the pages that I need for bottom navigation but the issue is that the icons of the bottom navigation are really small. I use the icons from app_resources folder and I have 3 different size but it seems that it only take the smallest one on iOS. The icons are in the folder app_resources/iOS and it is like icon.png<EMAIL_ADDRESS><EMAIL_ADDRESS>...
The issue doesn't occur in Android

Is there any code involved?
<GridLayout rows="*,auto">
            <GridLayout row="0">
                <StackLayout [visibility]="selectedTab == 0 ? 'visible' : 'collapse'">
                    <ns-dashboard></ns-dashboard>
                </StackLayout>
                <StackLayout [visibility]="selectedTab == 1 ? 'visible' : 'collapse'">
                    <ns-calendar></ns-calendar>
                </StackLayout>
                <StackLayout [visibility]="selectedTab == 2 ? 'visible' : 'collapse'">
                    <ns-leaderboard></ns-leaderboard>
                </StackLayout>
                <StackLayout [visibility]="selectedTab == 3 ? 'visible' : 'collapse'">
                    <ns-supervision *ngIf="checkPerm('Supervision')"></ns-supervision>
                </StackLayout>
            </GridLayout>
            <BottomNavigationBar row="1" activeColor="#ECE0C5" inactiveColor="#525252" backgroundColor="#F4F4F4"
                (tabSelected)="onSelectedIndexChanged($event)" selectedTabIndex="{{selectedTab}}" titleVisibility="Never">
                <BottomNavigationTab icon="res://dashboard">
                </BottomNavigationTab>
                <BottomNavigationTab icon="res://calendar">
                </BottomNavigationTab>
                <BottomNavigationTab icon="res://leaderboard">
                </BottomNavigationTab>
                <BottomNavigationTab *ngIf="checkPerm('Supervision')" icon="res://supervision">
                </BottomNavigationTab>
            </BottomNavigationBar>
        </GridLayout>

@danouche93 can you create a repro sample project? Thanks

@farfromrefug I'm using font icon, but you can see the same issue in this repo; the iOS icons are very small compared to the Android counterpart:

HI,
we should set the font here font = font.withFontSize(30) this solve the problem but the spacing between the icon and the title stay the same, it should be handled also.

            let font = this.style.fontInternal;
            font = font.withFontSize(30)
            is = ImageSource.fromFontIconCodeSync(fontIconCode, font, new Color('white'));


Accept friend

Hello, i dont understand something
'friend'
SteamID of the user
EFriendRelationship
Some activity in your friend list. For example, EFriendRelationship.PendingInvitee means you got a friend invite, EFriendRelationship.None means you got removed. The friends property is updated after this event is emitted.
I've done this :
bot.on('friend', function (SteamID, Relation){
if (Relation == 2){
bot.addFriend(SteamID);
console.log(SteamID + " Accepted")
}
});
Witch auto accept invites, but how do i see those EFriendRelationship.PendingInvitee
EFriendRelationship.None
or What's the sintax, i dont get it

sorry but the https://github.com/seishun/node-steam/blob/master/lib/generated/steam_language.js page is not found

@trader514 [url=https://github.com/SteamRE/SteamKit/blob/master/Resources/SteamLanguage/enums.steamd]Here's the new list[/url]


fixing scale bugs in bilinear (src = 1) and bicubic (dest = 1)

fixes #73 and corner case from #86

cc: @deltheil

@soumith sounds good (no more buffer overflow here and no more division by zero here).
Note: I ran a quick check with address sanitizer - very handy!


BE: Chore: use record classes

What changes did you make? (Give an overview)
Refactors the code to take advantage of records
Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[x] Unit checks

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged

A picture of a cute animal (not mandatory but encouraged)

@Haarolean What is blocking the merge of this change?
Thanks in advance!


Return a stream of accept results from TcpListener::incoming().

Addresses issue #37.

I've left a comment on #37, perhaps we can continue discussion there?


make pass.json from Buffer instead of string, for non english text

Fix for #83
I did check the tests, but some of them failed with and without this change - so guess it's ok to merge

/azp run


GPII-4111: Fix 404 failures

This PR fixes GPII-4111 - low number of 404s we recently observed in our pipeline.
TL;DR
Roughly 0.02% of requests to database would fail in 503 error, which further results in 404 error returned by preferences. Based on my investigation this happens only in case of istio-proxy to istio-proxy traffic, and is related to connection pooling (Istio reuses connections for multiple requests). Istio-proxy on the client side will try to send traffic over a connection that has already been closed, and this results in 503 error upstream connect error or disconnect/reset before headers. reset reason: connection termination.
See https://gist.github.com/stepanstipl/51780eb0755ec9d177b6da9df5baaa61 for more details.
This seems to be caused by known issue in Envoy - https://github.com/envoyproxy/envoy/pull/6578 - and should be fixed in Istio 1.1.8 and higher.
Fix
Until version with the fix is available on GKE, this can be resolved by disabling connection pooling, via DestinationRule with:
trafficPolicy:
  connectionPool:
    http:
      maxRequestsPerConnection: 1

Performance impact
Performance impact of this seems to be on average ~ 3ms higher latency on a -duration=600s -rate=100/s test. Given this is temporary and our current traffic levels & response times, this should not be an issue.
Current:
Latencies     [mean, 50, 95, 99, max]    47.106521ms, 43.92287ms, 87.839265ms, 108.368494ms, 191.956161ms

With connection pooling disabled:
Latencies     [mean, 50, 95, 99, max]    44.300331ms, 41.563617ms, 83.572071ms, 100.518912ms, 1.000213669s

Tests
Without fix applied, duration=600s -rate=100/s test against gpii/_design/views/_view/findPrefsSafeByGpiiKey?key=%22wayne%22&include_docs=true endpoint would result usually in 5-15x 503 errors.
bash-5.0# echo "GET $URLSVC" | ./vegeta attack -duration=600s -rate=100/s  | tee results.bin | ./vegeta report^C
bash-5.0# cat results.bin | ./vegeta report
Requests      [total, rate, throughput]  60000, 100.00, 99.98
Duration      [total, attack, wait]      10m0.016821357s, 9m59.989852387s, 26.96897ms
Latencies     [mean, 50, 95, 99, max]    44.300331ms, 41.563617ms, 83.572071ms, 100.518912ms, 1.000213669s
Bytes In      [total, mean]              49314184, 821.90
Bytes Out     [total, mean]              0, 0.00
Success       [ratio]                    99.99%
Status Codes  [code:count]               200:59992  503:8
Error Set:
503 Service Unavailable

With connection pooling disabled, 10x runs of the same test would finish with 0 errors:
bash-5.0# echo "GET $URLSVC" | ./vegeta attack -duration=600s -rate=100/s  | tee results.bin | ./vegeta report
Requests      [total, rate, throughput]  60000, 100.00, 100.00
Duration      [total, attack, wait]      10m0.025406738s, 9m59.989864885s, 35.541853ms
Latencies     [mean, 50, 95, 99, max]    47.106521ms, 43.92287ms, 87.839265ms, 108.368494ms, 191.956161ms
Bytes In      [total, mean]              49320000, 822.00
Bytes Out     [total, mean]              0, 0.00
Success       [ratio]                    100.00%
Status Codes  [code:count]               200:60000
Error Set:

I have also tried to run rake test_preferences 10x and all the tests finished fine, without any 404s or 503s (but based on the investigation this test is not a reliable indicator and fails roughly only 1 in 10 times in dev environment).

Good job on connection pooling issues investigation!

Performance impact of this seems to be on average ~ 3ms higher latency on a -duration=600s -rate=100/s test

I don't think this extra connection handling overhead is a problem atm, but let's hope fixed Istio is available on GKE soon!
LGTM


Use an intermediate struct in workload endpoint cache

Description
This PR introduces an intermediate struct used in the workload endpoint cache. It will allow us to monitor and synchronize additional wep fields beyond just labels.
This is groundwork for supporting named ports in the policy controller.

Seems like we're using a DeepEqual to compare old/new, and then just (currently) overwriting the label field. Shouldn't the old/new compare be limited to the fields we're updating?

Not sure I understand the question - ff there's any difference between the old / new objects, then we need to reprogram the workload endpoint to the datastore in its entirety.

The code looks like its comparing all fields recursively in WorkloadEndpoint, but on line 54 of pod_converter.go, it's only merging/overwriting Labels field.
My question/concern is whether we're comparing more than we're updating. If we compare all fields, but only update one, we could enter a compare/update loop.

If we compare all fields, but only update one, we could enter a compare/update loop.

Ah, I see.  IIUC this won't happen because we're doing the DeepEqual not on a WorkloadEndpoint struct, but on the more limited WorkloadEndpointData struct.
The intention is that ever field in the WorkloadEndpointData will be both compared and also updated on the WorkloadEndpoint, so we won't hit that loop.

ah, got it.

then LGTM!


Use chain address if identity alias is null

Includes a regression test to assert that chain address gets used if (mocked) identity service responds with alias: null

Closed in favour of making the changes in dscp-identity-service instead

https://github.com/digicatapult/dscp-identity-service/pull/69


SRP publish not triggered on renewal of entry

Problem
When a SRP service got re-added, it was not fed into the Thread otSrpClientAddService() call again.
The function exited early in the function as the srpService reference was not filled in correctly in the lookup loop.
This only happens when it concerns a 'refresh/renewed/duplicate' call to add the SRP service + you only configure 1 service entry.
When more entries were available, this issue is obfuscated by filling in on top of an empty entry later on in a next iteration.
Change overview

Fixed the lookup loop to re-use the cleared out SRP entry on a duplicate.
Break added to avoid unwanted scenario's
typo fix

Testing

Manual test of commissioning flow using CHIP_DEVICE_CONFIG_THREAD_SRP_MAX_SERVICES set to 1

/rebase