Latest  USB release (2381) not working

Installed the latest USB release 2381. The boot up screen shows fine but no response from the keyboard accepted.
Back to release 2197. All seem to work. So looks like there is an issue with the latest USB release.
Cheers,
Levent

It displayed all the way to the prompt (including build # and Wozmon char).
By the way, the link you included does not contain the USB  firmware serial check disabled.

FYI- the latest SD card version doesn't work for me. I don't have a working USB hub and didn't try the USB build.
With the SD version, I do get the splash screen and the wozmon prompt but no keyboard input. The previous SD version works correctly with the same keyboard/hardware.
I'm using a rev A board if that matters.

It shouldn't do. It's odd as I haven't touched the USB code in ages, or the SDCard code. I will put up a release that has no serial code in it (the main change) and see if that's any better.

Okay release 11 is being built now. This has two changes ; one disables the serial code and the other is to reduce the allocated memory for graphics objects. Previously this has caused boards to crash and the compiler gives no warning.
Unfortunately if I can't replicate the problem I can only fix it by removing code until it works. Which is a pain, sorry.

USB Release 11 (version 2448)  works for me.

Okay if release 11 worked can you try release 12. This has the serial code put back in, but accessed differently. You have to type / [ENTER] at the command line to start it manually.  It also does nothing with the serial code until it's actually needed.
I'm pretty sure the serial code is the problem. Why, I don't know. I suspect it's some timing or interplay between the USB system and the UART system.
If it all seems to work, can you try / and let me know what, if anything happens.

Release 12 booted up  into BASIC OK. Entering  '/' displays 'Serial link enabled.'. After that the system hangs. No response from the keyboard.

Found a USB/TTL serial cable for more testing.
Release 12 (2470) for SD booted into basic. As with leventp when I enter '/' it displays the serial link message and then does not accept any more input from the keyboard. It does not display anything on the serial port either.
However, I found that on release 10 (2381) the USB keyboard does work IF a the serial port is active! If the serial port is not connected then they keyboard does not work.
If I boot 2381 with the serial port connected and then remove the serial connection then the usb keyboard will stop working as soon as the serial is removed.

That's what I expected. I had to do it in a bit of a rush, so I didn't add the ESC out of it. It needs some work on timing out.  It works well enough, the problems occur when it gets out of sync. Rather than timing out and exiting it is waiting for packet completion, I should let it time out once a packet has started and reject the incomplete packet and error.

So far, the Buck Rogers release working fine.
Just a question. I see 'CASE' and 'ENDCASE' tokens  defined. Are they implemented?
Cheers,
Levent

I've closed this as I've had no further reports and the broken ones seem to work. It may still have consequential issues, the serial port might not work maybe, but if that's an issue it can be reopened.


Trailing whitespace in file

We have noticed the yaml file gains trailing whitespace when using the CLI.
If a dev then opens it in an IDE that automatically removes trailing whitespace we see diffs.

Should we make sure the output has no trailing whitespace?

@hlascelles I think it should be fine to fix this. If you want to make a PR to remove this and make sure tests still pass, I can take a look. Not sure what is causing the random spaces lol

Duplicate of https://github.com/pivotal/LicenseFinder/issues/802


Reference generated types in schema

Is your feature request related to a problem? Please describe.
I'm writing a custom lambda resolver for a field which is basically making a mock connection. We've keyed our dynamo DB in a way that connections are thus far not able to take advantage of. Since this is basically just a connection with a bit of extra functionality, I'd like to still have filter, sortDirection, our timestamp, etc.
When defining the inputs of the function, I want to be able to say:
type Taco
  @model
{
  customConnection(filter: ModelDataPointFilterInput, sortDirection: ModelSortDirection): @function
}

etc. That is, I want to be able to reference the types that will be eventually generated.
Describe the solution you'd like
I'd like a syntax to mark a type as "should be generated." Maybe $? gql-compile will fail if these types do not get generated.
Describe alternatives you've considered
The alternative is a lot of copy and paste / renaming from the generated schema.

Hello @isaiahtaylor
If I'm understanding this you would like the types ModelDataPointFilterInput and ModelSortDirection beforehand? If so this types are created as after evaluating the before schema. If you are looking to add them into the schema you would need to add them in the before schema, then the transformer would read these are overwrites and not recreate the types.

@SwaySway this is certainly a valid path in simple scenarios (copy-pasting the types into the before schema), but with more complicated types it would be nice to be able to reference types that other connections depend on eventually.

@isaiahtaylor These inputs are mainly used against a DynamoDB backed datasource. Perhaps another potential enhancement to discuss with the team would be creating a connection against a specified DynamoDB table? I assume these arguments (filter and direction) are mainly used to generate a payload to DynamoDB in the lambda function?

Pasting is fine for us for now. Fine with closing this.

I'd also be interested in a solution for this!
For now, we're also copy and pasting, but re-using would certainly be nicer. For us, it would be nice to re-use generated input types in a Query that resolves from a Lambda function like so:

type LocationScore {
    noise: Float
    shopping: Float
    view: Float
}

[...]

type Query {
  locationScoreCalculations(locationScore: LocationScoreInput!): LocationScore @function(name: "locationScoreCalculations-${env}")
}


CSOM: Create password protected sharing links

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Expected or Desired Behavior
CSOM should contain an option to create a password protected sharing link. Currently I cannot find that option within the SDK. We currently have that option in the UI, is it also possible to create them through CSOM/API's? Or will this be available in the near future?
https://www.microsoft.com/nl-nl/microsoft-365/roadmap?filters=&searchterms=27018
Observed Behavior
Currently cannot find an option to create a sharing link which is password protected.

That capability is just now rolling out to the service and doesn't appear to be present in the REST API / CSOM API ATM... this is a good resource to use to track changes int he APIs: https://s-kainet.github.io/sp-rest-explorer

@andrewconnell : Thanks for your reply. I was hoping for a API first strategy here :wink:. But let's hope it will be available soon.


Possible known cleartext issue

Due to using zip as format before encrypting, there is a possible issue with "known cleartext attack" towards the zip's header.
While in theory you first must figure out which part is the actual "first" part, there still is the issue that this attack may help weaken the encryption of at least the first part.
There are two possible solutions to mitigate this issue:

Figure out the exact length of the zip header bytes and remove them before encrypting (and add them when decrypting again)
Remove zip as layer and instead implement a custom method of gathering the file(s) and directory(ies).

Zip header definition:
https://pkware.cachefly.net/webdocs/casestudies/APPNOTE.TXT 4.3

Because zip headers are hardly dependent on the system the command was executed on, its not a good solution to try to replace/adjust it.
Implement an own packer has shown more complex than initially expected (32 bit system support makes it hard). Therefor this has been postponed till all the necessary things are figured out.
Therefor finally i decided to implement a random length random byte padding which is prepended to the []byte zip data. Therefor an attacker would have to either massively brute to figure the position, or alternative use very complex adjusted frequency analysis to nail it down before brute forcing.
It is a very unlikely scenario anyway, tho still this method should mitigate it quite well and longterm as soon we got our custom packers its good enaugh.


'endedAT' not defined

After pulling the latest set of commits I found a couple things that are preventing compilation.
First is the int definitions for the microsteps are disparate from the pinmode and the rest of the sketch (My bad, cus that came from my PR):
int MicroStep1Pin = 34; // Stepper MS1
int Microstep2Pin = 35; // Stepper MS2
int Microstep3Pin = 32; // Stepper MS3

  pinMode(MicroStep1Pin, OUTPUT);	//set Microstep 1 config
  pinMode(MicroStep2Pin, OUTPUT);	//set Microstep 2 config
  pinMode(MicroStep3Pin, OUTPUT);	//set Microstep 3 config

Notice the capitalization is different.
Second is at line 331:
void stopRoll(){
    // write current position to EEprom
    writeData(EEcurrStep,endedAT);
    if (debugPrint ==true){
       Serial.println("Stop");
    }
    digitalWrite(EnablePin, LOW);
    digitalWrite(DirPin, HIGH);
    delay(1000);
    digitalWrite(EnablePin, HIGH);
}

something is funky with this syntax cus it can't be compiled due to endedAT not being defined. That's one I'm not sure how to fix while still preserving the EEPROM additions

All solved


fixes roles/docker/vars/ubuntu-bionic.yml points to xenial

This pr fixes issue #3387.

Updated CLA

Please sign the CLA

@woopstar done!

ci check this

/lgtm
/approve

please run CI for the OS concerned before merging.


remove tests dir in favor of bundle test repo

tests were moved to https://github.com/canonical/bundle-kubeflow
CI is configured on https://github.com/canonical/bundle-kubeflow as well. Thus, tests in current repo redundant and obsolete

@phoevos done


Unable to get Login page ...

I am Unable to get Login page while running Traccar on Netbeans.  Getting 404 page not found error.
I have downloaded "traccar-master" and loaded it into Netbeans, as directed in the guide https://www.traccar.org/build-in-netbeans/ and when trying accessing it on browser at http://localhost:8082/ then getting 404 - Not found error.....
I followed the guide on https://www.traccar.org/build-in-netbeans/  Still stuck.

Sound like you haven't checked out traccar-web repository.

Thank you very much for you kind reply.
Oh yeah, I got it now, but the problem now arriving is I am not getting Information from android Traccar Client, its telling, "Send failed" in the status window,  Actually I wanna know what exactly server address should I put there, I put <IP_ADDRESS>, <IP_ADDRESS>:8082, both failed

Are both on the same network? It seems like you are using local IP address.
Also, make sure that you are using correct port for the Client. It should be 5055 by default.

I am running Traccar from Netbeans, and tracking my Cell Phone (By having Traccar Client android app on it).  What Server Address should I give now?
Thank you

If you want to connect your local server, you should give the IP address of your local server.

Okay, here I am in a LAN of many PC's, Suppose the global IP of my LAN is 182.75.xxx.xx , then I tried using this IP in Traccar Client, but the same error came, i.e. : "Send failed"

Please follow this guide:
https://www.traccar.org/troubleshooting/

Thank you very much got it !


[Bug]: HTTP Requests charts should not show floating point values

Is there an existing issue for this?

[X] I have searched the existing issues

Deploy type
Installing Dashboard directly (eg. v2.12.0, commit xyz, branch name)
Version
f/mserving-metrics
Current Behavior

Graph shows floating point values as above
Expected Behavior
Graph values should be integers
Steps To Reproduce
No response
Workaround (if any)
No response
What browsers are you seeing the problem on?
No response
Anything else
The values are to do with how prometheus presents time series data, we will need to use a round or floor function for the query.

Th dashboard has been tested out here and it's prometheus that's giving us floating point values. This has been brought up with the model serving team. I'm blocked waiting for a solution for this.


Logger throws exception if config information isn't found

A proper error should be sent to stderr and the application should exit if information on the logger couldn't be found.

Yeah, I got that myself a few times.. Could we define an error?

I'll hammer it out in a bit.

So, this error is closed, right?

Correct.  It throws an error to the console buffer explaining the problem and exits.


Aspire container for RabbitMq not ready, crashing other Aspire managed apps using Wolverine/RabbitMq

Describe the bug
I've been trying to use Aspire (Preview 3) to implement my architecture, using RabbitMq via Wolverine to get things communicating appropriately, I'm aware of the open #635 that seems to be working out how to use the Aspire client services, however in my case I'm purely using host to provide the necessary RabbitMq.  Within the Aspire launched applications (web-server, API, consumers etc) I want to use pure RabbitMq via Wolverine, e.g. not the IConnection etc that the Aspire client libraries add.  However, I'm having startup issues with the client crashing immediately.
To Reproduce
Steps to reproduce the behaviour:

Add to the Aspire host:

var messageQueue = builder.AddRabbitMQ("command-queue");

Add to an Aspire invoked application

builder.Host.UseWolverine(options => {
  var connectionString = builder.Configuration.GetConnectionString("command-queue")
                           ?? throw new InvalidOperationException();
  options.UseRabbitMq(new Uri(connectionString))
         .UseSenderConnectionOnly();
});

Starting of client crashes with the following exception:

024-02-23T17:58:52.4817620 fail: Wolverine.Runtime.WolverineRuntime[0]
      Failed to start the Wolverine messaging
      RabbitMQ.Client.Exceptions.BrokerUnreachableException: None of the specified endpoints were reachable
       ---> System.IO.IOException: connection.start was never received, likely due to a network timeout
         at RabbitMQ.Client.Framing.Impl.Connection.StartAndTune()
         at RabbitMQ.Client.Framing.Impl.Connection.Open(Boolean insist)
         at RabbitMQ.Client.Framing.Impl.Connection..ctor(IConnectionFactory factory, Boolean insist, IFrameHandler frameHandler, String clientProvidedName)
         at RabbitMQ.Client.Framing.Impl.Connection..ctor(IConnectionFactory factory, Boolean insist, IFrameHandler frameHandler, ArrayPool`1 memoryPool, String clientProvidedName)
         at RabbitMQ.Client.Framing.Impl.AutorecoveringConnection.Init(IFrameHandler fh)
         at RabbitMQ.Client.Framing.Impl.AutorecoveringConnection.Init(IEndpointResolver endp
oints)
         at RabbitMQ.Client.ConnectionFactory.CreateConnection(IEndpointResolver endpointResolver, String clientProvidedName)
         --- End of inner exception stack trace ---
         at RabbitMQ.Client.ConnectionFactory.CreateConnection(IEndpointResolver endpointResolver, String clientProvidedName)
         at RabbitMQ.Client.ConnectionFactory.CreateConnection(String clientProvidedName)
         at RabbitMQ.Client.ConnectionFactory.CreateConnection()
         at Wolverine.RabbitMQ.Internal.RabbitMqTransport.BuildConnection()
         at Wolverine.RabbitMQ.Internal.RabbitMqTransport.ConnectAsync(IWolverineRuntime runtime)
         at Wolverine.Transports.BrokerTransport`1.InitializeAsync(IWolverineRuntime runtime)
         at Wolverine.Runtime.WolverineRuntime.startMessagingTransportsAsync()
         at Wolverine.Runtime.WolverineRuntime.StartAsync(CancellationToken cancellationToken)

This can be overcome by something hacky such as the following at the end of the program.cs for a minimal API :
// Map routes

Console.WriteLine("Waiting for 5 seconds before starting the application");
await Task.Delay(5000);
Console.WriteLine("Starting the application after 5 seconds to allow RabbitMQ to start");

app.Run();
}

Expected behaviour
Initialisation code waits for the RabbitMq container to start.  (configurable timeout or resilience pattern?).

Wolverine doesn't yet support Aspire, here's where we're tracking that work: https://github.com/JasperFx/wolverine/issues/635
And no, I don't have a perfect idea of when that will be done, but it's on the radar.

The WaitFor() coming in 9.0 and will solve the originally raised issue, but until then you can use David Fowler's prototype.
I have a complete, working solution that extends the PingPongWithRabbitMq example with (among other things) the Aspire integration.


Generate ID for CSL reference

When a new reference is posted through the UI, the backend should assign an ID for it.
Request failed with status code 400
Error updating database. Cause: org.postgresql.util.PSQLException: ERROR: null value in column "id" violates not-null constraint Detail: Failing row contains (null, 3, null, null, null, null, null, , 2019-06-20 10:18:36.556514, 101, 2019-06-20 10:18:36.556514, 101). ### The error may involve org.col.db.mapper.ReferenceMapper.create-Inline ### The error occurred while setting parameters ### SQL: INSERT INTO reference_3 ( id, dataset_key, sector_key, verbatim_key, csl, citation, year, created_by, modified_by , created, modified) VALUES ( ?, ?, ?, ?, ?::jsonb, ?, ?, ?, ? , now(), now()) ### Cause: org.postgresql.util.PSQLException: ERROR: null value in column "id" violates not-null constraint Detail: Failing row contains (null, 3, null, null, null, null, null, , 2019-06-20 10:18:36.556514, 101, 2019-06-20 10:18:36.556514, 101).
HTTP method: POST
https://api-dev.col.plus/dataset/3/reference
Body:
{"type":"incollection","title":"Cortinarius","author":[{"family":"Niskanen","given":"Tuula"},{"family":"Jeppesen","given":"Thomas S."},{"family":"Brandrud","given":"Tor Erik"},{"family":"Frøslev","given":"Tobias"}],"container-title":"Funga Nordica","editor":[{"family":"Knudsen","given":"Henning"},{"family":"Vesterholt","given":"Jan"}],"edition":"3","volume":"2","publisher":"Nordsvamp","publisher-place":"Copenhagen","URL":"","ISBN":"978-87-983961-3-0","issued":{"date-parts":["2016","05","01"]},"accessed":"","pages":"697-789"}

yes indeed


Description of database::Database::insert_index is not accurate

The docs for insert_index says:

Inserts an index in a naive way, if it already exists, will throw an Err

This is not accurate. When an index already exists, the DesignCreated::results field will be "exists" and no error will be thrown. Because of this behavior, the ensure_index method on Database doesn't really have any purpose.
In my mind, the primary purpose of a method like ensure_index is to run in a 'migration' job before an application depending on couch_rs is launched. A method like this would ensure that the database in question has the indexes that are expected by the application code.
I propose the following changes:

Update the docs for insert_index to reflect its actual behavior.
Add a doctest that will validate this behavior
Update the arguments to insert_index to match the types::index::Index struct, to make it more compatible with the get_indexes method.
Create an enum for the IndexType as defined in the CouchDB docs
Make a new ensure_index function that will take a vector of Index and either validate that they exist in the database or create them.

I can't appear to link this in Development, but this is the working branch of changes: https://github.com/kingledion/couch-rs/tree/improve-indexing
As of this post, I'm about 50% done with creating tests. There is a doctest for insert_index, but I'd like to also add a test for ensure_index (not sure if doctest or unit test yet...I'm currently testing this function in another application, locally).

thanks. can you please create a PR for this? so far your comments make sense to me (I have not looked at the low level details, but definitely we welcome PRs).

Here is PR: https://github.com/mibes/couch-rs/pull/27
I completely removed the ensure_index function because it just didn't have any purpose any more. The idea of having it validate an entire list of Index against the database is pretty trivial to implement (its just a for loop through a Vec calling insert_index on each entry) so not sure it is worth having a specific function for it.
The only last thing to validate is to ensure that my new IndexType enum serializes and deserializes properly.

That is a good point. This PR will return that function with its signature, while making it interact correctly with the new insert_index function: https://github.com/mibes/couch-rs/pull/31

Merged the PR and published the 0.9.1 release. Thanks for your contributions!


wrong text for pragma

This text is wrong.

Unlike __pragma, _Pragma allows you to put pragma directives into a macro definition. The string literal should be what you would otherwise put following a #pragma statement. For example

The second term should be #pragma and read...

Unlike #pragma, _Pragma allows you to put pragma directives into a macro definition. The string literal should be what you would otherwise put following a #pragma statement. For example

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

ID: ce8f0a4b-8eeb-158f-3995-66fa674e67d1
Version Independent ID: d9e7ec65-7238-a9fb-6ab3-dc71144e3a34
Content: Pragma directives and the __pragma and _Pragma keywords
Content Source: docs/preprocessor/pragma-directives-and-the-pragma-keyword.md
Product: visual-cpp
Technology: cpp-language
GitHub Login: @corob-msft
Microsoft Alias: corob

@diablodale
Good catch, and thanks for the report. We've since published a fix.


unbound[698]: segfault at 0 ip 00007f0df4ea5229 sp 00007ffd4b6b5a50 error 4 in libc.so.6[7f0df4e28000+195000] likely on CPU 1 (core 1, socket 0)

Hello.
I built unbound 1.19.0 with ./configure --disable-shared --disable-flto --with-ssl=/opt/ssl3 --sysconfdir=/etc --enable-tfo-client --enable-tfo-server --with-libnghttp2 on Ubuntu 22.04.3 with Kernel 6.5.0-15-generic #15~22.04.1-Ubuntu using OpenSSL 3.2.1 previously built and available in /opt/ssl3.
Configure and build was OK, I was also able to start and use unbound.
After a while (tens of minutes) the pid disappear and unbound crash with Segmentation Fault:
[ 2209.791010] unbound[698]: segfault at 0 ip 00007f0df4ea5229 sp 00007ffd4b6b5a50 error 4 in libc.so.6[7f0df4e28000+195000] likely on CPU 1 (core 1, socket 0)
[ 2209.791028] Code: 00 42 0f b7 0c 62 66 85 c9 0f 84 e9 fe ff ff 49 8d 7c 24 10 48 8b 04 fa a8 0f 0f 85 b1 01 00 00 48 89 c6 83 e9 01 48 c1 ee 0c <48> 33 30 48 89 34 fa 66 42 89 0c 62 48 c7 40 08 00 00 00 00 48 83

So it is unusable.
So I have tried to build it again but using the OpenSSL library included in the OS, the version is OpenSSL 3.0.2 15 Mar 2022 (Library: OpenSSL 3.0.2 15 Mar 2022). I configured it with ./configure --disable-shared --disable-flto --with-ssl --sysconfdir=/etc --enable-tfo-client --enable-tfo-server --with-libnghttp2.
Until now no issues and unbound is still alive and working.
So the issue seems to be related when building unbound using  OpenSSL 3.2.x.
g++ and gcc version are 12.3.0.

This issue is releated to https://github.com/NLnetLabs/unbound/issues/1063#issuecomment-2105593022
I close it.


Remove triggers to not force a replace on manifest change

What does this PR do?
Modify the docker-compose command to include a hash of the content manifest as an environment variable.
This is a workaround to stop using Triggers that will create a replacement (docker-compose down + docker-compose up) to avoid some flakiness we have when this happen. We now let docker-compose handle the update.
Which scenarios this will impact?
docker
Motivation
Fix flakiness
Additional Notes

Environments triggers an Update? So we did not need to have envVars in triggers?


purgeCompensation

Hello,
I'm have trouble understanding exactly what the purge method is doing.  From my read of the code, apps that are started after the fetchCFApps call is completed are discovered are are reported as stopped.  Is the goal here only to report usage an applications we "knew" about when the bridge started?

Hi,
The purge compensation is meant to be executed after billing epoch start/creation. The epoch creation removes all usage events for stopped apps from the database, leaving trace only for the started ones.
The epoch start is done with purge API call. Since "there is the potential race condition if apps are currently being started, stopped, or scaled", we compare the state of the CC and the usage events to check if there are misplaced usage events.
For every app that is reported as started but not present in the CC we fire "stop usage" report.
Regards,
Hristo Iliev

Thanks, that helps.


Support Mac IL2CPP package install

IL2CPP is also supported on macOS standalone builds. Package path is UnitySetup-Mac-IL2CPP-Support-for-Editor-$Version.

I noticed we also don't install the mono Mac components either


no NDK arm-linux-androideabi-gcc on $PATH at (eval 10) line 124.

@vgorloff
Thanks for your job.
When running "node main.js bootstrap", I met error here:
cd /Users/kylewong/Codes/Swift/swift-everywhere-toolchain/ToolChain/Sources/ssl && \
   ANDROID_NDK=/usr/local/ndk/21.3.6528147 PATH=/usr/local/ndk/21.3.6528147/toolchains/llvm/prebuilt/darwin-x86_64/bin:$PATH \
   ./Configure \
   -D__ANDROID_API__=21 \
   --prefix=/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/ToolChain/Install/darwin-armv7a/ssl \
   android-arm
Configuring OpenSSL version 1.1.1d (0x1010104fL) for android-arm
Using os-specific seed configuration

Failure!  build file wasn't produced.
Please read INSTALL and associated NOTES files.  You may also have to look over
your available compiler tool chain or change your configuration.

no NDK arm-linux-androideabi-gcc on $PATH at (eval 10) line 124.
Execution of command is failed:
cd /Users/kylewong/Codes/Swift/swift-everywhere-toolchain/ToolChain/Sources/ssl && \
   ANDROID_NDK=/usr/local/ndk/21.3.6528147 PATH=/usr/local/ndk/21.3.6528147/toolchains/llvm/prebuilt/darwin-x86_64/bin:$PATH \
   ./Configure \
   -D__ANDROID_API__=21 \
   --prefix=/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/ToolChain/Install/darwin-armv7a/ssl \
   android-arm

If error was due Memory, CPU, or Disk peak resource usage (i.e. missed file while file exists),
then try to run previous command again. Build process will perform "configure" step again,
but most of compilation steps will be skipped.

/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/lib/Tool.js:82
      throw error;
      ^

Error: Command failed: cd /Users/kylewong/Codes/Swift/swift-everywhere-toolchain/ToolChain/Sources/ssl && \
   ANDROID_NDK=/usr/local/ndk/21.3.6528147 PATH=/usr/local/ndk/21.3.6528147/toolchains/llvm/prebuilt/darwin-x86_64/bin:$PATH \
   ./Configure \
   -D__ANDROID_API__=21 \
   --prefix=/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/ToolChain/Install/darwin-armv7a/ssl \
   android-arm
    at checkExecSyncError (child_process.js:630:11)
    at Object.execSync (child_process.js:666:15)
    at SSLBuilder.execute (/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/lib/Tool.js:72:10)
    at SSLBuilder.executeCommands (/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/lib/Tool.js:89:10)
    at SSLBuilder.executeConfigure (/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/lib/Builders/SSLBuilder.js:62:10)
    at SSLBuilder.configure (/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/lib/Builder.js:79:10)
    at SSLBuilder.make (/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/lib/Builder.js:96:10)
    at SSLBuilder.runAction (/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/lib/Builder.js:62:12)
    at /Users/kylewong/Codes/Swift/swift-everywhere-toolchain/Automation.js:123:57
    at Array.forEach (<anonymous>) {
  status: 2,
  signal: null,
  output: [ null, null, null ],
  pid: 22798,
  stdout: null,
  stderr: null
}

arm-linux-androideabi-gcc indeed doesn't exist in the ndk.
kylewong@KyleWongs-MacBook-Pro swift-everywhere-toolchain % ls ~/Library/Android/sdk/ndk/21.3.6528147/toolchains/llvm/prebuilt/darwin-x86_64/bin | grep gcc

How to resolve this issue?

Seems you got this case:

One can engage clang by adjusting PATH to cover same NDK's clang. Just keep in mind that if you miss it, Configure will try to use gcc...

Source: https://github.com/openssl/openssl/blob/master/NOTES-Android.md
That is seems why Configure trying to use arm-linux-androideabi-gcc.

Try to comment line -D__ANDROID_API__=${ndk.api} in file lib/Builders/SSLBuilder.js. Maybe it will help.
Before:
  executeConfigure() {
    this.clean();

    var ndk = new NDK();
    // Seems `-D__ANDROID_API__` not needed. See<EMAIL_ADDRESS>    var cmd = `
    ${this.options}
    ./Configure
    // -D__ANDROID_API__=${ndk.api}
    --prefix=${this.paths.installs}
    `;
    if (this.arch.name == Archs.arm.name) {
      cmd = `${cmd} android-arm`;
    } else if (this.arch.name == Archs.arm64.name) {
      cmd = `${cmd} android-arm64`;
    } else if (this.arch.name == Archs.x86.name) {
      cmd = `${cmd} android-x86`;
    } else if (this.arch.name == Archs.x86_64.name) {
      cmd = `${cmd} android-x86_64`;
    }
    this.executeCommands(`cd ${this.paths.sources} && ${cmd}`);
  }

Then execute: node main.js ssl:make arch:armv7a

btw. Similar issue: https://github.com/openssl/openssl/issues/11192
Maybe you have environment variable "CC" set somewhere?

btw: Another similar issue: https://github.com/openssl/openssl/issues/8941

Unfortunately, neither of it works.

OK. What command perl configdata.pm --dump returns?
cd ToolChain/Sources/ssl/
perl configdata.pm --dump

OK. What command perl configdata.pm --dump returns?
cd ToolChain/Sources/ssl/
perl configdata.pm --dump

kylewong@KyleWongs-MacBook-Pro ssl % perl configdata.pm --dump
Can't open perl script "configdata.pm": No such file or directory
kylewong@KyleWongs-MacBook-Pro ssl % pwd
/Users/kylewong/Codes/Swift/swift-everywhere-toolchain/ToolChain/Sources/ssl

Hm. The file configdata.pm even was not generated. Super strange.
Well. If the commands below not working, then it is an issue of SSL. Seems would be more effective to ask them as well: https://github.com/openssl/openssl/issues.
cd ToolChain/Sources/ssl
git clean --quiet -f -X
git clean --quiet -f -x -d
export ANDROID_NDK_HOME=/usr/local/ndk/21.3.6528147
export ANDROID_NDK_ROOT=/usr/local/ndk/21.3.6528147
PATH=$ANDROID_NDK_ROOT/toolchains/llvm/prebuilt/darwin-x86_64/bin:$ANDROID_NDK_ROOT/toolchains/aarch64-linux-android-4.9/prebuilt/darwin-x86_64/bin:$PATH
./Configure android-arm64 -D__ANDROID_API__=29
make SHLIB_VERSION_NUMBER= SHLIB_EXT=.so

Build commands taken from: https://github.com/openssl/openssl/blob/master/NOTES-Android.md

I ran into the same issue locally and I found out that it only happens when /usr/local/ndk/21.3.6528147 is a symlink. Not sure why, as far as I know it should be possible to add symlinks to $PATH but in that case it doesn't work.
PATH=/usr/local/ndk/21.3.6528147/toolchains/llvm/prebuilt/darwin-x86_64/bin:$PATH
   ./Configure android-arm -D__ANDROID_API__=21

# Failure!  build file wasn't produced.
# Please read INSTALL and associated NOTES files.  You may also have to look over
# your available compiler tool chain or change your configuration.

# no NDK arm-linux-androideabi-gcc on $PATH at (eval 10) line 124.

# configures successfully

PATH=/Users/username/Library/Android/sdk/ndk/21.3.6528147/toolchains/llvm/prebuilt/darwin-x86_64/bin:$PATH \
   ./Configure -D__ANDROID_API__=21 android-arm

I used this commit to workaround it locally. I can open a pr if this makes sense for you as well.

I ran into the same issue locally and I found out that it only happens when /usr/local/ndk/21.3.6528147 is a symlink. Not sure why, as far as I know it should be possible to add symlinks to $PATH but in that case it doesn't work.
PATH=/usr/local/ndk/21.3.6528147/toolchains/llvm/prebuilt/darwin-x86_64/bin:$PATH
   ./Configure android-arm -D__ANDROID_API__=21

# Failure!  build file wasn't produced.
# Please read INSTALL and associated NOTES files.  You may also have to look over
# your available compiler tool chain or change your configuration.

# no NDK arm-linux-androideabi-gcc on $PATH at (eval 10) line 124.

# configures successfully

PATH=/Users/username/Library/Android/sdk/ndk/21.3.6528147/toolchains/llvm/prebuilt/darwin-x86_64/bin:$PATH \
   ./Configure -D__ANDROID_API__=21 android-arm

I used this commit to workaround it locally. I can open a pr if this makes sense for you as well.
@vgorloff

Interesting. Thank you for the figuring out this issue!
The whole idea of NDK symbolic link – is to have "similar path on user side" in "umbrella header" when setting up  glibc.modulemap (e.g. /.../swift-android-toolchain/usr/lib/swift/android/armv7/glibc.modulemap). But of cause symbolic link not really needed during build time.
I will try to use your @michaelknoch commit to make two separate variables for NDK path: one will be used during build time (non symlinked) and another will be used when packing toolchain for distribution.


MINT CR1-2022 (Issue #76)

@ConanHoye I see your point.  I've reverted the TSS to it's original wording.


Support 'Run from Package' as deployment source

Not all OS/Plan types are supported today, but its in progress.
See here for discussion: https://github.com/Azure/app-service-announcements-discussions/issues/32

Linux support was added here: https://github.com/Microsoft/vscode-azurefunctions/pull/574
Only thing left to do is default Windows apps to use run from package since it just GA'ed


Added functionality to save the qr code as an image (.png) file

As mentioned in the title, one can now, with the -f option, create a .png/.PNG file containing the QR code.
Running the command- $ qrify -f img.png google.com will create img.png in the current directory which has a qr code
img.png-

note: this uses an api different to the one that generates the code. The api information is available at http://goqr.me/api/
Pull Request Label:

[ ] Bug
[ ] New feature
[x] Enhancement
[ ] New component
[ ] Typo

Pull Request Checklist:

[x] Have you followed the guidelines for contributing?
[x] Have you checked that there aren't other open pull requests for the same fix or component?
[ ] Have you ran the tests locally with bats tests?

I forgot to mention that this has not been tested with httpie and fetch, only with wget and curl

So basically just remove the file validation part and modify the options section, is that right?

I believe so, make those changes and if it looks good Ill merge otherwise I will modify the script and push it to your fork and then it should be good to merge :+1:

I fixed a few things but nothing to major, thanks for your contribution :tada:


ent-go: init 0.10.0

ent-go: init 0.10.0
Note: I added myself as a maintainer at PR #156599 which is not merged yet!
After merging that PR a rebase is necessary and it will work just fine.

@ofborg eval


Update google support library to latest 26.0.1

@dlew Could you create update for the library?

Thanks for the PR, but there's no need to push out an update for RxLifecycle just for a transitive dependency update. If you include 26.0.1 in your builds, RxLifecycle will use it as well.


Cannot add types: Date, Date when attempting to do date math

Describe the bug
unable to perform simple date math such as col("date") + '1y'
To Reproduce
df = daft.from_pydict({
   'date' : ['2021-01-01', '2021-01-02', '2021-01-03'],
}).select(daft.col("date").cast(daft.DataType.date()))

df.select(daft.col("date") + '1y').collect()

---------------------------------------------------------------------------
DaftCoreException                         Traceback (most recent call last)
Cell In[5], line 8
      2 df = daft.from_pydict({
      3    'date' : ['2021-01-01', '2021-01-02', '2021-01-03'],
      4 })
      6 df = df.select(daft.col("date").cast(daft.DataType.date()))
----> 8 df.select(daft.col("date") + '1y').collect()
File ~/Development/Daft/daft/api_annotations.py:26, in DataframePublicAPI.<locals>._wrap(*args, **kwargs)
     24 type_check_function(func, *args, **kwargs)
     25 timed_method = time_df_method(func)
---> 26 return timed_method(*args, **kwargs)
File ~/Development/Daft/daft/analytics.py:198, in time_df_method.<locals>.tracked_method(*args, **kwargs)
    195 @functools.wraps(method)
    196 def tracked_method(*args, **kwargs):
    197     if _ANALYTICS_CLIENT is None:
--> 198         return method(*args, **kwargs)
    200     start = time.time()
    201     try:
File ~/Development/Daft/daft/dataframe/dataframe.py:1233, in DataFrame.select(self, *columns)
   1205 """Creates a new DataFrame from the provided expressions, similar to a SQL ``SELECT``
   1206
   1207 Examples:
   (...)
   1230     DataFrame: new DataFrame that will select the passed in columns
   1231 """
   1232 assert len(columns) > 0
-> 1233 builder = self._builder.select(self.__column_input_to_expression(columns))
   1234 return DataFrame(builder)
File ~/Development/Daft/daft/logical/builder.py:150, in LogicalPlanBuilder.select(self, to_select)
    145 def select(
    146     self,
    147     to_select: list[Expression],
    148 ) -> LogicalPlanBuilder:
    149     to_select_pyexprs = [expr._expr for expr in to_select]
--> 150     builder = self._builder.select(to_select_pyexprs)
    151     return LogicalPlanBuilder(builder)
DaftCoreException: DaftError::External Unable to create logical plan node.
Due to: DaftError::TypeError Cannot add types: Date, Date

Expected behavior
a dataframe with the output of
{
   'date' : ['2022-01-01', '2022-01-02', '2022-01-03'],
}

Component(s)
Expressions
Additional context
No response

So, there's two issues here.

The error message is wrong. It should actually be Due to: DaftError::TypeError Cannot add types: Date, Utf8 The error message is printing the field NAME, which happens to be Date and very confusing.
We currently don't do inference on the string to parse it as a duration.

Update: This issue itself is resolved, but @universalmind303 is working on inference on string as Duration


Period input being swallowed

First of all, incredibly  impressed at how well this works.
Was wondering, has anyone run into an issue inputting a "period" into a form? For instance, I've tried to enter my email address which uses periods as delimiters and the only way that it seems to work is by using windowskey + period (I'm on ubuntu)

What an odd bug. :joy: Not sure how I missed that!
Stranger is that tty/input_event_test.cc seems to detect . just fine, so whatever way it's being sent to Chromium isn't what it expects.


Missing dashes in subject and genre tags

What
We seem to have lost the dashes between words in subject and genre tags.
I would expect that subjects and genres render the punctuation.
Where
https://wellcomecollection.org/works/d5zws93r

Browser: Chrome, Safari

@alexwlchan Long shot, but can this have been caused by any recent trimming work? I know you were more concentrating on trailing full stops.

Let me have a look, it’s certainly possible I messed up the joining character.

Long shot, but can this have been caused by any recent trimming work?

The data in the API has all the dashes, so I don’t think I caused this issue: https://api.wellcomecollection.org/catalogue/v2/works/d5zws93r?include=genres%2Csubjects

Okay, I'm less convinced that this isn't an API issue – testing locally, with the front-end rolled back to code from about a fortnight ago, I still don't see the dashes.

@jennpb this happened as a regression as we have applied the rules for contributors (to not have separators) universally across tags on the works page.
There's a PR in fr this


Add statistics time

Like a browser, it can count the waterfall flow and length of visits.

That is a very cool idea! We already have a HttpSession object that behaves like a browsers (i.e. keeps session, cookies etc), it should be easy to add something like that!

Trying to define the data to capture

request path
response status
request size
response size
response time
offset time from the session start

Trying to define the data to capture

The sky is the limit 😄
As for the timings, I'd go either with absolute timestamps (from which one can infer the duration, of course) or some kind of ticks for each individual step. Though, maybe splitting by the following actions could be a good idea

DNS lookup (if applicable)
Connection time for TCP (at least up until 2.0)
SSL handshake (if applicable)
Sending the request
Receiving the response

The data points you mentioned, @igr, look good to me, maybe also the request method.
@OnePieceMan, what do you think?


[BUG] Deployment fails when reaching [stage 06] - ControlPlaneAddOnsNotReady

Describe the bug
Deployment error when reaching stage 06
code:
ControlPlaneAddOnsNotReady
message:
Pods not in Running status: coredns-547dd8b568-lrswn,coredns-autoscaler-6fb889cdfc-twmtf,metrics-server-7d59848cc6-jkd59,metrics-server-7d59848cc6-sl2tq,tunnelfront-69c958dc9-gndmc,tunnelfront-7f8f87f77c-wjqrn
Also kubectl get nodes from jumpbox returns 'No resources found'
To Reproduce
Follow steps in AKS-Secure-Baseline-PrivateCluster/Bicep
Expected behavior
AKS cluster should deploy correctly
kubectl get nodes should return 3 nodes

An example event grabbed from kubectl cluster-info dump:
says: no nodes available to schedule pods
{
            "metadata": {
                "name": "azure-policy-7b67f765f-kshzq.16ea12e114a5bda8",
                "namespace": "kube-system",
                "uid": "f1fbb7b9-03c8-4833-aab0-1cef01cd6758",
                "resourceVersion": "8471",
                "creationTimestamp": "2022-04-28T13:36:11Z"
            },
            "involvedObject": {
                "kind": "Pod",
                "namespace": "kube-system",
                "name": "azure-policy-7b67f765f-kshzq",
                "uid": "42cd254f-24f3-4088-9fb1-19f386dd1af9",
                "apiVersion": "v1",
                "resourceVersion": "648"
            },
            "reason": "FailedScheduling",
            "message": "no nodes available to schedule pods",
            "source": {
                "component": "default-scheduler"
            },
            "firstTimestamp": "2022-04-28T13:36:11Z",
            "lastTimestamp": "2022-04-28T14:11:34Z",
            "count": 35,
            "type": "Warning",
            "eventTime": null,
            "reportingComponent": "",
            "reportingInstance": ""
        }

I thnk I know what the problem was. I suspect that I incorrectly edited parameters and set the wrong IP address for firewall and dns on the spoke VNet so I'm trying to redeploy now. That would explain why the Node VMs and VMSS has no outbound connectivity


Sign-in & Sign-up completely Styled #310 from heet-kumar

I have done some necessary changes to style the Sign-in and Sign-up pages of the Athavani project.
Also align the position of the Sign-in and Sign-up page and also made fully responsive as well.

Adjusted the color contrast of the page.
Button and Input tag styling.
Styled the header part of the Sign in and Sign up templates and many-more updates are done.

@heet-kumar thanks for your efforts.

@heet-kumar thanks for your efforts.
But always try to add reference to the issue .

ok sure thanks for the appreciation.


Feat/add container insights

Update CDK versions
Silence deprecated warnings
Update Node NVM and NPM versions
Fix vault to run locally
Add container insights to ECS cluster
Standardize the ops.yml versions

@klebertertulino We're going to have to resolve some minor conflicts here once #28 merges.


Window handles are empty

Hi,
I am launching Teams application through winapp driver, after some time of launching I am trying to get the Application's window handles but window handles are returning as empty, If I do the same immediately after launching Teams, I am able to get the Window handles. Please help me to resolve this issue.
Thank you.

Karthikeyan021298 I suggest recreating Teams session in this case by process name

Are you telling to reinitiate Winapp driver  session ?

Teams application has a splash screen, please create another session with the opened window read wiki.

@anunay1 and @Shakevg  I am trying to use driver.launchApp() if windowHandles is empty, I am getting the following Error with no stack trace,
An unknown error occurred in the remote end while processing the command. (WARNING: The server did not provide any stacktrace information)
Command duration or timeout: 0 milliseconds
Build info: version: 'unknown', revision: 'unknown', time: 'unknown'
System info: host: 'DESKTOP-OSAQ11O', ip: '<IP_ADDRESS>', os.name: 'Windows 10', os.arch: 'amd64', os.version: '10.0', java.version: '1.8.0_275'
Driver info: io.appium.java_client.windows.WindowsDriver
Capabilities {app: C:\Users\admin\AppData\Loca..., javascriptEnabled: true, platform: WINDOWS, platformName: Windows}
Session ID: 823A6224-C5FE-4A1E-8C5E-182060EA1EFD

launchApp() holds good for android and ios apps not for windows.
Have you read the wiki?

I took the below screenshot as reference and it was mentioned as work around for splash screen

I am running the Teams application in Windows machine  @anunay1

Which language binding are you using?

I am using JAVA 8

check

Do I need to get the window id  and set the capability while reinvoking a session caps.setCapability("appTopLevelWindow", MAWinHandleHex); @anunay1

Yes


Memory usage in container is 2 orders of magnitude higher for slapd.

When running slapd in a docker container the memory usages increases with 700M (give or take, seen in htop). When the same installation is run on a vm it will only consume around 2M. Is there a way to find out what is causing this huge difference in memory consumption?
Even if the memory reported is not entirely accurate, I cannot run two containers with slapd on a vm that has 1G of ram. The second container just stops and errors out because it can not allocate the required memory.
$ sudo docker run -d ldap
a41c2fd...
$ sudo docker run -it --rm ldap slapd -d 4; echo $?
...
54243799 ch_calloc of 1048576 elems of 704 bytes failed
slapd: ch_malloc.c:107: ch_calloc: Assertion `0' failed.
255

Running just the container sudo docker run -it --rm ldap /bin/bash uses barely any memory (as expected). Someone else seemed to have a similar issue (#6861), but I'm not sure.
To reproduce, dockerfile and install script for archlinux: https://gist.github.com/xcJeroen/3e2b2f2eebdfcb85cef0
In case it's important the host os for the docker container is an archlinux as well.
$ uname -a
Linux sarchy 3.16.3-1-ARCH #1 SMP PREEMPT Wed Sep 17 21:54:13 CEST 2014 x86_64 GNU/Linux

I also tried to run a couple of other openldap containers that I could find on the docker registry, and they all seem to have the same problem.

looks like this was resolved, ping me here if otherwise

Not fixed, but is superseded by #4717 - open files                      (-n) 524288.

:+1:  Same resolution here.
I experienced the same issue with my own slapd container (memory usage in the 500+ MB range), and resolved it by changing my Dockerfile to CMD ulimit -n 1024 && slapd -d1.  Now the slapd container is at around 25MB.
This is on the hypriot/wheezy Raspberry Pi image with Docker 1.6.0.


[AGENT] Boxing queue entries to reduce memory

Phenomenon and reproduction steps
Large amout of memory allocated by queues
Root cause and solution
Queue allocates size_of() * len memory.
Use Box when size of T is large
Impactions
N/A
Test method
Check memory
Affected branch(es)

main

Checklist

[ ] Dependencies update required
[ ] Common bug (similar problem in other repo)

demo上运行一小时占用250M左右


REQ: option for appcenter release-react to exclude bundling assets (only js)

Hi I know that it is possible to bundle without assets using the vanilla command. But as per the docs the preferred method to deploy for react-native is through react-release (and I fully agree).
Can we have an option please to only bundle JS and exclude assets, this is helpful for making smaller deployment packages when you know the assets have not changed.
Thanks,
Matthew

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

ID: a2c2bd47-97e3-abc8-fbdd-305f0c631827
Version Independent ID: 318718e7-a38a-7cc5-01a4-7ce0153c4398
Content: Releasing CodePush updates using the App Center CLI - Visual Studio App Center
Content Source: docs/distribution/codepush/cli.md
Service: vs-appcenter
GitHub Login: @Zakeelm
Microsoft Alias: zakeelm

@mattslight I'm not sure if that's possible. Let me have the PM take a look and respond.

Thank you - really appreciate it. And please keep up the good work.

Hey there @mattslight
I'm not sure I quite understand the scenario you're trying to account for, can you clarify?
From my understanding, the CodePush server performs file-based diffing, which ultimately means only updated and new files are sent to client device.

Well the react-release command seems to bundle JS and assets in my case. The payload is about 4300Kb.
The payload size is fairly constant +/- 50kb and never seems to matter whether I do small or bigger changes (only ever to JS files).
The command I use is this: appcenter codepush release-react -a ACCOUNT/PROJECT -d Staging --output-dir ./build

Checking the build/codepush folder confirms the following:-
build/CodePush  tree -d                                                                           10s
.
└── assets
    ├── assets
    │   └── img
    │       ├── intro
    │       └── patterns
    ├── js
    │   └── lib
    │       └── react-native-zss-rich-text-editor
    │           ├── img
    │           └── src
    └── node_modules
        ├── react-native-datepicker
        ├── react-native-elements
        │   └── src
        │       └── rating
        │           └── images
        ├── react-native-video-controls
        │   └── assets
        │       └── img
        └── react-navigation
            └── src
                └── views
                    └── assets

23 directories, 76 files

@mattslight but we should be doing a delta comparison and only sending the changed assets. have you confirmed that all that data is being sent to the device?

I do understand and +1 on what @jwargo said. The diff should account for this and only send the changed assets. I do have a follow up question, are each of your CodePush releases proceeded by a binary release?

@mattslight but we should be doing a delta comparison and only sending the changed assets. have you confirmed that all that data is being sent to the device?

I subscribed to CodePush progress updates as per:
`CodePush.sync( {}, this.codePushStatusDidChange.bind(this), this.codePushDownloadDidProgress.bind(this)
codePushDownloadDidProgress(progress) {
this.setState({ progress })
}`
And then in render
 if (this.state.progress) {       progressView = (         <Text           style={{             textAlign: 'center',             fontFamily: FontFamily.regular,             fontSize: FontSize.xs,             color: Color.placeholderTextColor           }}         >           {Math.round(parseInt(this.state.progress.receivedBytes) / 1024)} of{' '}           {Math.round(parseInt(this.state.progress.totalBytes) / 1024)} Kb received         </Text>       )
That's how I came to know the size and I can 'watch' it downloading 5000Kb approx. I haven't used any network monitoring tools to actually confirm it, but I assume the CodePush progress methods are accurate.

From what I can tell there is no 'delta' comparison. CodePush deployment payload is as per the tree I sent above. As you can see it is sending assets as part of the payload.
The appcenter codepush release-react is actually using the following command to generate the payload bundle:
 bundle --assets-dest build/CodePush --bundle-output build/CodePush/main.jsbundle --dev false --entry-file index.js --platform ios --sourcemap-output build/CodePush/main.jsbundle.map
I can run this manually to reduce the bundle size without the --assets-dest

@mattslight but to @Zakeelm's question, did you deploy a new binary before this?

@jwargo Deployed new binary means Xcode build and appstore publish?

yep, or deploy via USB to a local device.

I don't always deploy a new binary (but isn't that the point of CodePush?). What's your line of thinking?

@mattslight If you deploy a new binary with an older set of assets, then CodePush is going to deploy its whole bundle on sync.

@jwargo Sorry, I don't follow can you elaborate please.
Am I to understand from what you are saying that I should deploy a new binary to reduce the CodePush payload or that I shouldn't deploy a new binary?

@mattslight shouldn't.

Right - gotcha. So I generally only release a new binary if I link a new native library. If it's just JS updates then I codepush. The assets generally never change (app only has a few fonts, svgs and pngs).

@jwargo Does 5000 kb for incremental CodePush releases sound a lot to you? What payload sizes should I be expecting if a handful of lines of JS change from push to push?

@mattslight sorry, not an expert - I just own the docs. I do know that CodePush shouldn't be sending the whole package every time, should only be sending the changed assets. so, if you're getting everything, then we need to figure out why (whether it's your fault or ours :-)). At this point, the docs isn't the right place to be having this conversation. Can you please open App Center and click the blue chat button in the lower-right corner and start a conversation with our dedicated support team?  They'll help you and we'll see if this is a bug or something else. That OK?

Sure no problem, I appreciate your help. Btw this really wasn't the conversation I wanted to have lol.
I came here because it says in the docs if I have a request for a feature I should raise it here. I was asking for a feature for appcenter release-react to have an option to exclude automatically adding the assets to the CodePush update.
Thanks for your time.


fixed typo

fixed typo,weigth-->weight

Codecov Report

Merging #25 into master will decrease coverage by 0.27%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
- Coverage   79.85%   79.57%   -0.28%
==========================================
  Files          11       11
  Lines        1087     1087
==========================================
- Hits          868      865       -3
- Misses        159      163       +4
+ Partials       60       59       -1

Impacted Files
Coverage Δ

naming/naming.go
90% <100%> (ø)
:arrow_up:

naming/client.go
73.38% <0%> (-0.78%)
:arrow_down:

Continue to review full report at Codecov.

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


[zh]kubernetes-api/workload-resources/priority-class-v1/ does not has chinese version

Problem:
do not have chinese docs: https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/priority-class-v1/
Proposed Solution:
add chinese docs in website
Comments
/language zh
/assign

/triage accepted

could I take this task in this weekend? I want to apply for a member next week :)

@windsonsea: this task has already assigned to me, so do not translate it.

@windsonsea: this task has already assigned to me, so do not translate it.

ok, got it, thanks.


Mysql replication rebased

please test this

this is missing updates to .fixtures so unit tests cannot pass

please test this

please test this

please test this


Configurable buffer length for Console Windows

From<EMAIL_ADDRESS>on 2010-10-27T14:57:57Z
There appears to be an arbitrary buffer length of 300 lines.  I'd like to see this be a user configurable option.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=431

From pierre.raybaut on 2010-10-27T13:09:45Z
It is already a configurable option :)
Preferences > Console > Display tab
Summary: Configurable buffer length for Console Windows
Status: Invalid
Labels: -Type-Defect Type-Enhancement

From<EMAIL_ADDRESS>on 2010-10-27T13:13:56Z
Oh, good call.  :)  I forgot I found it in there yesterday, heh.  Need more coffee!


Add support for Project/repository creation

Steps to test/reproduce

Ensure the Metecho GitHub App has all required permissions as explained in CONTRIBUTING.rst

Projects

The endpoint /api/projects/ now handles POST requests. Projects will be created immediately on Metecho and a job will be queued to create the GH repo.
After the job completes you should see a new team and repository on the selected GH organization. The user that created the Project will be added as maintainer of the team, and any selected collaborators will be added as team members.
If the job fails, both the local Metecho project and GH repo will be deleted.

Project Dependencies

Log into the admin and visit the new "Project Dependencies" section
Visit /api/dependencies/ to browse all objects. This is used by the frontend to populate the Dependencies checkboxes when the user creates a new Project

Organizations

Log into the admin and visit the new "GitHub Organizations" section
From here you'll be able to add/edit GH organizations that should appear to users when creating projects
Ensure each organization has the Metecho GH app installed

Some additional support endpoints are also provided.

The organization's members can be fetched on demand from GitHub by visiting /api/organizations/:id/members.
Each Metecho user has a list of all organizations they are a member of stored in their organizations field. You can refresh this list by POSTing to /api/user/refresh_orgs.
Repo name availability can be checked by POSTing to /api/organizations/:id/check_repo_name. Useful to display an indicator to the user as they type the new name they want to use.
Organizations can check if the Metecho GitHub App has been installed correctly by POSTing to /api/organizations/:id/check_app_installation/.

Show me
Provide screenshots/animated gifs/videos if necessary.

  CircleCI says:
ImportError: cannot import name 'init_from_context' from 'cumulusci.cli.project' (/root/metecho/venv/lib/python3.9/site-packages/cumulusci/cli/project.py)
In the fork init_from_context does exist. Could you run the tests locally to check? I can't reproduce the problem on my machine.

Could you run the tests locally to check? I can't reproduce the problem on my machine.

@jerivas Passing for me locally too. I wonder if CI is too aggressively caching the dependencies, and not actually installing the forked version?

Requirement already satisfied (use --upgrade to upgrade): cumulusci==3.43.0 from https://github.com/oddbird/CumulusCI/...

 As an admin, I want to identify a "template repo" to autopopulate files for newly-created Metecho Projects.

I'm using the full-access gh app from Keybase, but maybe it has something to do with the permissions of that app?

Determined in Slack this was due to the GH App not having access to all repos. Need to discuss with SFDO

 I want newly-created Metecho Projects to autopopulate with GitHub configurations.

I would expect my description I entered in the Create Project modal to show up at the "About" in GitHub.

@jgerigmeyer could you try description=project.description in this line? If not I can try later, just working on another branch right now

@stacyk Thanks! Responses below:

1. I was not in the list of collaborators, so I think I assumed I would automatically be one since I made the project. Is that not the case?  I didn't add anyone else so on the final confirmation step, it doesn't show anything there where I see Collaborators in the mockup and screenshot David posted.

That is correct. They requested after last week's demo that we remove the current user from the list, but you will automatically be given access to the repo anyway (since you created it). This likely won't be confusing -- you're selecting other people to collaborate, not yourself. (No need to do any additional setup in the admin for this. Other users will show up in the list if they have access to the GH org.)

2. On the confirmation screen in the mockup and the screenshot David posted, the project name is in the title of the modal. Here's what I see on the summary page:

Yes, I made that change this morning, and was planning to demo it and see what others think. I was finding that often I completely missed the name in the header, and expected it to be there with the other info I entered in the body of the modal. So what you're seeing currently is my suggestion. (And you'll see I made some other changes especially to that summary page, where we have patterns elsewhere in Metecho that I thought would maybe be appropriate. The mockup for this hasn't been updated since a few rounds of changing functionality, so I did not stick with it exactly. I'm happy for suggested changes from you or @SondraE or whomever.)

3. I assume we aren't doing dependencies yet? Or did I create a new project within an organization that doesn't have dependencies? Probably was another stop I missed in the admin settings.

We don't show the dependencies page unless there are some, and you need to add them manually via the admin: /admin/api/projectdependency/add/

4. Not in the mockup, and it's probably not something we need to fix or override, but I wonder why the first tooltip is on one line but the last one wraps:

Yes, I wondered about this too. If you or @dvdherron has a way to fix this, I'm for it!

5. @dvdherron not sure if there's much we can do, but maybe we can remove the right margin from the select an organization field and add a left margin to the re-sync since it is never left aligned anyway?

I'll defer to you and @dvdherron here. I don't love the changing width of the org dropdown, but hopefully most users won't see the error message anyway.

6. Probably not new to this PR, but for some reason on hover over the Select all checkbox in the GitHub Collaborators table, the background of the table cell changes to white.

Yeah, looks like that's also true for the existing "Add or Remove Collaborators" modal table.

7. `Save` in `Save & Next` feels a little misleading since I accidentally clicked outside of the modal a few times and when I reopen it, nothing is saved. Maybe nothing we can change it to if that is a standard SF uses.

Good call. I went ahead and disabled closing the modal on click-outside (it still closes on ESC or the "close" btn). I made this change for all modals in Metecho: https://github.com/oddbird/Metecho/commit/c745850a9a8f50c62d62d9e691af041e0ec4d0fd

8. I think this has always been a Metecho / GitHub thing, but I think I would expect my description I entered in the Create Project modal to show up at the "About" in GitHub.

Fixed in: https://github.com/oddbird/Metecho/commit/c019937defe88e35745e219191f6dd46ab084aae

9. I am guessing we went with `Create` instead of `Create Project` to save space on the modal button confirmation screen?  (Same reasoning to not include cancel since the X and anywhere else cancels it?)

Correct. We can go back to "Create Project" if you prefer that.

@stacyk

4. Not in the mockup, and it's probably not something we need to fix or override, but I wonder why the first tooltip is on one line but the last one wraps:
   ![Screen Shot 2022-05-25 at 1 58 42 PM](https://user-images.githubusercontent.com/1581694/170347494-9871c5bf-884a-4a90-9793-33ea44b61789.jpg)

I can't figure this one out. As far as I can tell we are only giving each of these a label and an id. I even tested adding an extra word to that label and it still stacked vertically.

5. @dvdherron not sure if there's much we can do, but maybe we can remove the right margin from the select an organization field and add a left margin to the re-sync since it is never left aligned anyway?
   ![Screen Shot 2022-05-25 at 12 21 52 PM](https://user-images.githubusercontent.com/1581694/170348491-74c810af-97e6-4f79-a06b-c63a64970770.jpg)

Fixed in https://github.com/oddbird/Metecho/pull/1376/commits/6730b9b8ac95cde69ccf2bdd0646ff6beac516b3

@stacyk

4. Not in the mockup, and it's probably not something we need to fix or override, but I wonder why the first tooltip is on one line but the last one wraps:
   ![Screen Shot 2022-05-25 at 1 58 42 PM](https://user-images.githubusercontent.com/1581694/170347494-9871c5bf-884a-4a90-9793-33ea44b61789.jpg)

I can't figure this one out. As far as I can tell we are only giving each of these a label and an id. I even tested adding an extra word to that label and it still stacked vertically.

@dvdherron We could try playing with the tooltipPosition prop -- maybe trying overflowBoundaryElement? Or override with our own styles, if needed.

Going to respond one at a time, and possibly out of order

1. I was not in the list of collaborators, so I think I assumed I would automatically be one since I made the project. Is that not the case?  I didn't add anyone else so on the final confirmation step, it doesn't show anything there where I see Collaborators in the mockup and screenshot David posted.

That is correct. They requested after last week's demo that we remove the current user from the list, but you will automatically be given access to the repo anyway (since you created it). This likely won't be confusing -- you're selecting other people to collaborate, not yourself. (No need to do any additional setup in the admin for this. Other users will show up in the list if they have access to the GH org.)

@jgerigmeyer  👍

3. I assume we aren't doing dependencies yet? Or did I create a new project within an organization that doesn't have dependencies? Probably was another stop I missed in the admin settings.

We don't show the dependencies page unless there are some, and you need to add them manually via the admin: /admin/api/projectdependency/add/

👍

6. Probably not new to this PR, but for some reason on hover over the Select all checkbox in the GitHub Collaborators table, the background of the table cell changes to white.

Yeah, looks like that's also true for the existing "Add or Remove Collaborators" modal table.

@dvdherron This is what is causing the white bg on that cell.
```
.slds-th__action:focus, .slds-th__action:hover{
…
  background-color:white;
}

Maybe we can target our own align-checkboxes class (or add a new class):

.align-checkboxes .slds-th__action:focus, .slds-th__action:hover{
background-color: inherit;
}

> > ```

Save in Save & Next feels a little misleading since I accidentally clicked outside of the modal a few times and when I reopen it, nothing is saved. Maybe nothing we can change it to if that is a standard SF uses.

Good call. I went ahead and disabled closing the modal on click-outside (it still closes on ESC or the "close" btn). I made this change for all modals in Metecho: c745850

👍

I think this has always been a Metecho / GitHub thing, but I think I would expect my description I entered in the Create Project modal to show up at the "About" in GitHub.

Fixed in: c019937

Great!

I am guessing we went with Create instead of Create Project to save space on the modal button confirmation screen?  (Same reasoning to not include cancel since the X and anywhere else cancels it?)

Correct. We can go back to "Create Project" if you prefer that.

We can keep Create.

Probably not new to this PR, but for some reason on hover over the Select all checkbox in the GitHub Collaborators table, the background of the table cell changes to white.

Yeah, looks like that's also true for the existing "Add or Remove Collaborators" modal table.

@dvdherron This is what is causing the white bg on that cell.
.slds-th__action:focus, .slds-th__action:hover{
…
  background-color:white;
}

Maybe we can target our own align-checkboxes class (or add a new class):
.align-checkboxes .slds-th__action:focus, .slds-th__action:hover {
  background-color: inherit;
}

Is it strange that outside of the pointer we've removed any visible hover change @stacyk ?

11. Probably not new to this PR, but for some reason on hover over the Select all checkbox in the GitHub Collaborators table, the background of the table cell changes to white.

Yeah, looks like that's also true for the existing "Add or Remove Collaborators" modal table.

> @dvdherron This is what is causing the white bg on that cell.
.slds-th__action:focus, .slds-th__action:hover{
…
  background-color:white;
}

Maybe we can target our own align-checkboxes class (or add a new class):
>
.align-checkboxes .slds-th__action:focus, .slds-th__action:hover {
  background-color: inherit;
}

Is it strange that outside of the pointer we've removed any visible hover change @stacyk ?

@dvdherron I don't think it is strange. Checkboxes (and their labels) don't have any default hover states so we aren't breaking anything/taking an accessibility feature away. Many people will add their own styles to the checkbox labels to add additional visual cues which I think is a good thing for all users, but it is an enhancement. There is a focus style, so that is helpful.

@dvdherron We could try playing with the tooltipPosition prop -- maybe trying overflowBoundaryElement? Or override with our own styles, if needed.

This seems to keep that last tooltip's text on one line:

@jgerigmeyer should we merge this? (the coveralls failure seems like a fluke)


is there no ToS violation?

Thanks for your work on this! However it feels like it would be against the GitHub ToS. Is it actually allowed?

People from GitHub are actually recommending this to customers. I cannot speak for GitHub, but so far there were no issues if you use this tool to fix CI pipeline issues.
If you use this tool to do malicious things, then this is obviously against GitHub ToS.


[$1000] Chat - Hover color is not displayed even on move of cursor on changing windows and back

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

Action Performed:

Open the app
Open any report
Hover on any message
Use CMD+TAB to change window and use it again to get back to app
Move cursor over the same message and observe that no hover color is displayed

Expected Result:
App should display hover color on message when we move cursor over that message
Actual Result:
App does not display hover color on message when we hover on the message, change window and again visit the app and move the cursor on that message
Workaround:
Unknown
Platforms:

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

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

Version Number: v1.3.79-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/66cc5bd5-a1fb-401b-a2cf-21f7491d0fc3
https://github.com/Expensify/App/assets/115492554/f82cd807-4a18-46fa-85fe-11283bbf610c
Expensify/Expensify Issue URL:
Issue reported by: @Dhanashree-Sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1696667037989909
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~017f7a0e22541d49f4
Upwork Job ID:<PHONE_NUMBER>431768576
Last Price Increase: 2023-10-30

Proposal
Please re-state the problem that we are trying to solve in this issue.
When switching to a different window, the hover effect is lost.
And the hover effect is not working untill we do mouse out and mouse enter again.
What is the root cause of that problem?
In the Hoverable component (src/components/Hoverable/index.js),
we're removing the hover effect when document is losing visibility.
But we're not setting the hover back when the document is gaining the visibility back.
What changes do you think we should make in order to solve the problem?
We need to add a mouse over event handler in addition to the other handlers here: https://github.com/Expensify/App/blob/3007017c1848a46cc70faf1618957f1289859788/src/components/Hoverable/index.js#L157-L170
On mouse over, we need to set is hovered to true.
We need to do this in a way that it doesn't trigger additional re-renders. I see we already have a class property called isHoveredRef and a method called setIsHovered on the component. We can use this to efficiently set the hovered state. We can call setIsHovered(true) function only when isHoveredRef is false.
What alternative solutions did you explore? (Optional)
NA

I have found the issue and have a fix for it.
You just need to handle additional (or replace existing) mouse event in one of your components.

@0xmiroslav mind reviewing proposals? 🙇♂

Proposal
Please re-state the problem that we are trying to solve in this issue.
Hover color is not displayed even on move of cursor on changing windows and back
What is the root cause of that problem?
Because we don't have function to control Mouse Movement.
What changes do you think we should make in order to solve the problem?
We can add onMouseMove event after onMouseLeave event.
onMouseMove: (el) => {
                if (this.isHoveredRef) {
                    return
                }
                this.setIsHovered(true);
                if (_.isFunction(child.props.onMouseMove)) {
                    child.props.onMouseMove(el);
                }
},

https://github.com/Expensify/App/blob/8d42ad3221927f5bf0e83b6d0e8c90038d03f4e5/src/components/Hoverable/index.js#L164-L171
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Hover color is not displayed even on move of cursor on changing windows and back
What is the root cause of that problem?
Because we don't have function to control Mouse Movement.
What changes do you think we should make in order to solve the problem?
We need to add onMouseMove event to
https://github.com/Expensify/App/blob/8d42ad3221927f5bf0e83b6d0e8c90038d03f4e5/src/components/Hoverable/index.js#L142
onMouseMove: (el) => {
                if (_.isFunction(child.props.onMouseMove)) {
                    child.props.onMouseMove(el);
                }
                if (this.isHoveredRef) {
                    return
                }
                this.setIsHovered(true);
},

What alternative solutions did you explore? (Optional)

reviewing tomorrow AM!

@dylanexpensify a gentle reminder to review the Proposals here :)

@0xmiroslav, just checked. My proposal above works even when the mouse is not moved:
https://github.com/Expensify/App/assets/140621077/620491e7-c7a8-42e2-a780-211b6ac85025
I'm not touching the mouse while recording the screen

hello @0xmiroslav Do you check my proposal?

bump @0xmiroslav ^ 🙇♂

Hello, @dylanexpensify How to think about my proposal

@dylanexpensify
If we want to change hover status when only back and move on that, then we can like this.
Need to update checkHover function
checkHover(e) {
        if (!this.wrapperView) {
            return;
        }

        if (this.wrapperView.contains(e.target)) {
            if (!this.state.isHovered) {this.setIsHovered(true);}
            return;
        }

        this.setIsHovered(false);
 }

https://github.com/Expensify/App/blob/2ff8c496c98773b3e95a949e2ea82356e35460c9/src/components/Hoverable/index.js#L108

Doubling!

Proposal
Please re-state the problem that we are trying to solve in this issue.
Hover color is not displayed even on move of cursor on changing windows and back
What is the root cause of that problem?
We don't have controller of mouse move.
What changes do you think we should make in order to solve the problem?
We need to add mouth move event
onMouseMove: enableHoveredOnMouseMove,

https://github.com/Expensify/App/blob/e99c297332c738475cfe5701e0a00ba0b5d58029/src/components/Hoverable/index.js#L213
And need to update code
if (shouldHandleScroll && isScrolling.current && hovered === isHovered) {

https://github.com/Expensify/App/blob/e99c297332c738475cfe5701e0a00ba0b5d58029/src/components/Hoverable/index.js#L69
What alternative solutions did you explore? (Optional)
https://github.com/Expensify/App/assets/65789015/4e085b63-7c07-49d4-8c02-0ac196794e53

@0xmiroslav we have proposals to review! 🙇

internal application now 😢

Nice, looking forward to the update @0xmiroslav!

updating today

Bumping this @0xmiroslav, any thoughts?

Nice, @0xmiroslav to confirm!

bump @0xmiroslav 🙇♂

bump @0xmiroslav

bump @0xmiroslav

@0xmiroslav update?

Not overdue, update expected today

@0xmiroslav please give an update or we can re-assign my friend

Bumped in Slack

Hey @Krishna2323, sorry for the delay. I agree and will assign now.

@jjcoffee reassigning, please take over as C+. If you don't have bandwidth, unassign yourself. Thanks

Thanks! Top of my list for review tomorrow.

Ah just noticed, @0xmiroslav already completed the review. I think it's fair that they get paid for that! cc @mallenexpensify

ah.. thanks, I think I was looking for 'merged' vs 'completed review', appreciate the headsup, if no additional work is needed on the PR, @0xmiroslav will be fully compensated.

Issue not reproducible during KI retests. (First week)

@Krishna2323 can you add video to confirm?

today!

@mallenexpensify, is @0xmiroslav still active on this? If not the who will complete the checklist 🤔

I just checked them off. THis wasn't a regression and is too minor for a regression test IMO

Payment summary:

Contributor+: @0xmiroslav $1000
Contributor: @Krishna2323 $1000
Reporter: @dhanashree-sawant $50

Thanks @dylanexpensify, I have accepted the offer.

done!


Bump the crates-io crate to 0.2

@bors: r+

:pushpin: Commit 5d87c4f has been approved by alexcrichton

:hourglass: Testing commit 5d87c4f with merge 81debdd...

r? @wycats
(rust_highfive has picked a reviewer for you, use r? to override)

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


Quicksoil should not form around clouds

I'll fix this when I add Gravitite ore (just waiting on the texture).

Closed as Fixed from testing a bit tonight, feel free to make new issue if reoccurs.

Closed as Fixed from testing a bit tonight, feel free to make new issue if reoccurs.


Document or hint status bar buttons

I was trying to play my list randomly instead of looping as preset, but the icons are not informative (I understand it's partly a VS Code issue). I ended up reading the controller's source code to figure it out:
https://github.com/nondanee/vsc-netease-music/blob/e955783f8571bcc87ed5054601de6e7c9610d54e/controller.js#L48
It'll be helpful for us users to show hints upon hovering over the buttons, or have their functionalities documented somewhere.

Ok, I will add tooltips for buttons


React.Owin

React.Owin
I've created Owin middleware for serving transformed JSX files. It's built on top core React library and Owin / Katana StaticFileMiddleware.
I know it's a big pull request, but since I already written it to use in my project, I thought I could share the code.
Why Owin?
Owin compatible code begins to take lot of attention in .NET. WebAPI 2 can be treated just as Owin middleware, and MVC 6 will be completely based on Owin.
I'm developing a web app in react.js, hosted on Owin-based service. To take advantage of server-side JSX transformation, I created middleware that uses React.Net to serve JSX files compiled to JavaScript.
Structure
I tried to retain the structure and standards of existing React.NET code: every file has a Facebook boilerplate, project libraries have links to shared assembly info files and there's included sample project that uses Microsoft.Owin.SelfHost.
The most important classes are:

JsxFileMiddleware which is wrapper around Owin StaticFileMiddleware, but uses it's own FileSystem.
JsxFileSystem - a decorator class around IFileSystem - that transforms the JSX file when requested.

Tests
I didn't include any tests for the code - it's hard for me to come up with something. Cache implementation is rather straightforward usage of System.Runtime.Caching, middleware itself configures built-in StaticFileMiddleware, and JsxFileSystem (which handles the transformation) is almost entirely based on IJsxTransformer.

Thanks for this!
Please ensure you're using the same assembly settings as the rest of ReactJS.NET. Notably, "warnings as errors" should be enabled for release builds.
Also please don't add jQuery to any examples. Ideally I'd like if you reused the same sample code from React.Sample.Mvc4 for consistency.

Thanks for all your effort to do this review! I hope I covered all your comments.
I used the sample code from React.Sample.Mvc, however I had to change it bit to adjust to Owin environment - it loads initial comments asynchronously.

Thanks for working on this! I really appreciate it :)

I've merged your changes! It would be awesome to get some documentation on this feature (just a Markdown file in site/jekyll/guides) but that's totally optional and I can write something if you don't get around to it. Just a basic explanation of what's supported and how to use it :)
Again, thanks! :D

Is there a nuget package with React.Owin?

@ilyapalkin Good point! I just noticed there's no NuGet package for React.Owin. Hahaha. Just created a config for it, it will be available with the next ReactJS.NET release (which should be soon), or you could use the version off the build server once it picks up the change.

NuGet package is up now: https://www.nuget.org/packages/React.Owin/


Add scope down statement support

what
AWS highly recommends safeguarding against HTTP request floods; they advise implementing two rate limiting rules for web traffic. The initial rule employs AWS WAF's rate-based rules to automatically block IP addresses of malicious actors if the number of requests in a 5-minute sliding window surpasses a predefined threshold. The second rule focuses on a more detailed approach, allowing for targeted blocking. This is achieved by using scope_down_statements to refine the criteria for blocking, enhancing the precision of the defense mechanism.
why

Adds scope_down_statements with byte matching support

references

AWS WAF – Rate-based rules
Rate-based rule request rate limiting behavior
Terraform Resource

/terratest


Smaller refactors

Looks good 🍏
Thanks for making the pytest change, didn't even realize that was a builtin.


Gas estimation - out of spec behavior?

I also want to report a bug? regarding the fee estimation. The response I am getting indicates a higher GasEstimateGasPremium than GasEstimateFeeCap. According to the v0.5.0 release note, GasFeeCap should be larger than GasPremium.

A sender will never be charged more than GasFeeCap * GasLimit.
A miner will typically earn GasPremium * GasLimit as a reward.

request
curl --location --request POST '{{node-url}}/construction/metadata' \
--header 'Content-Type: application/json' \
--data-raw '{
    "network_identifier": {
        "blockchain": "Filecoin",
        "network": "testnetnet"
    },
    "options": {
        "idSender": "t1d5l22aypjpzvtjbjsalw3gauwdf2edeqn6h2lui"
    }
}'

response
{
    "metadata": {
        "chainID": "testnetnet",
        "gasFeeCap": "119",
        "gasLimit": "10000000000",
        "gasPremium": "122201",
        "nonce": 1
    }
}

Originally posted by @qiwu7 in https://github.com/Zondax/rosetta-filecoin-proxy/issues/50#issuecomment-692970592

This is already fixed on master branch


Equip ring of dueling from bank

Hey there I saw that your runecrafting utilities plugin has the equip binding necklace from bank option
is it possible for there to be an option for ring of dueling as well? This would be huge as it's targeted for lava running.

This is a planned feature for 1.3.2 waiting for current PR to be accepted by RuneLite team

Oh perfect, thank you! Hopefully it gets accepted and pushed.

Hey!
Love the plugin, any update on the ring of dueling update?


Reconfigure JS build pipeline

we've switched to a non-vue, TypeScript-based setup on PPA and need to port that config over to MEP. probably better to do this sooner rather than later, although the PPA config allows normal JS to coexist with TypeScript just fine.

bumping this to 2 because it entails:

uninstalling vue-related dependencies & configs
porting over PPA webpack config
refactoring the existing s&co js to a typescript component (the main menu)
testing the new main menu component


fa-icon is not a helper

Hello,
After installing with ember install ember-font-awesome I get the following error:
Error: Compile Error: fa-icon is not a helper

I'm using ember-cli 2.12.2 and node 4.7.0. ember-font-awesome ver 3.0.5.
Would very much appreciate if you can point me to what might be the problem, please.
Thanks for the awesome addon, btw :)

apologies, I was defining fa-icon inside the wrong element/tag.


Hey Whay don't have support for installed softwares....

I've create another script file for the same. If you want i can send it over and you can put a new version.

@artemis15 ... if you have something in mind or even some sample code, would be great to have a look at it.

hey, I'm thinking to become contributer to the discovery of installed software. Do let me know if that's okay with you.

@artemis15 hello Akash, thank you for your feature request. Back then last year I also spend some time on trying to provide a solution here. I guess we should clarify some questions:
What exactly is "installed software" - for Windows and Mac OS this might be clear (I have a solution in place already) but on linux this is not that clear. Here we have to handle quite a lot package managers to get valid lists across different distributions.
What exactly are you thinking of?

Hey,
Wishing you a very happy new year.
Yes win and Mac can be done but there is a catch with win and Mac that you
need admin access to get installed softwares.
Regarding Linux, I've some scripts that I'm using to get data( I've been
using it for last 1.5 yrs and it's giving good result.)
Probably, we can spend some time and discuss on it.
--
Cheers,
Ar
On Thu, Dec 31, 2020, 3:14 PM Sebastian Hildebrandt <
<EMAIL_ADDRESS>wrote:

@artemis15 https://github.com/artemis15 hello Akash, thank you for your
feature request. Back then last year I also spend some time on trying to
provide a solution here. I guess we should clarify some questions:
What exactly is "installed software" - for Windows and Mac OS this might
be clear (I have a solution in place already) but on linux this is not that
clear. Here we have to handle quite a lot package managers to get valid
lists across different distributions.
What exactly are you thinking of?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sebhildebrandt/systeminformation/issues/296#issuecomment-752905195,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB57KEGCKH6K5WEPRVVGFDTSXRBXFANCNFSM4JODQBFA
.

Hey,
Wishing you a very happy new year.
Yes win and Mac can be done but there is a catch with win and Mac that you
need admin access to get installed softwares.
Regarding Linux, I've some scripts that I'm using to get data( I've been
using it for last 1.5 yrs and it's giving good result.)
Probably, we can spend some time and discuss on it.
--
Cheers,
Ar
On Thu, Dec 31, 2020, 3:14 PM Sebastian Hildebrandt <
<EMAIL_ADDRESS>wrote:

@artemis15 https://github.com/artemis15 hello Akash, thank you for your
feature request. Back then last year I also spend some time on trying to
provide a solution here. I guess we should clarify some questions:
What exactly is "installed software" - for Windows and Mac OS this might
be clear (I have a solution in place already) but on linux this is not that
clear. Here we have to handle quite a lot package managers to get valid
lists across different distributions.
What exactly are you thinking of?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sebhildebrandt/systeminformation/issues/296#issuecomment-752905195,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB57KEGCKH6K5WEPRVVGFDTSXRBXFANCNFSM4JODQBFA
.

@artemis15 I will add some initial code for this in version 5 which I am already working on. If you have partial scripts e.g. for linux, maybe you want to create a gist on your side and provide a link? I will then think of how this can be integrated to systeminformation.

@artemis15 I will add some initial code for this in version 5 which I am already working on. If you have partial scripts e.g. for linux, maybe you want to create a gist on your side and provide a link? I will then think of how this can be integrated to systeminformation.

Sure..will do that

Sure..will do that

@artemis15 thank you so much. I am sure, this will be a quite nice enhancement for the package!

@artemis15 thank you so much. I am sure, this will be a quite nice enhancement for the package!

@artemis15 thank you so much. I am sure, this will be a quite nice enhancement for the package!

@artemis15 thank you so much. I am sure, this will be a quite nice enhancement for the package!

yes it will be
On Wed, Jan 6, 2021 at 1:04 PM Sebastian Hildebrandt <
<EMAIL_ADDRESS>wrote:

@artemis15 https://github.com/artemis15 thank you so much. I am sure,
this will be a quite nice enhancement for the package!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sebhildebrandt/systeminformation/issues/296#issuecomment-755136930,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB57KEDNFDHBRZERBTHC3IDSYQG7JANCNFSM4JODQBFA
.

--
Akash Rajput
Tel: (+91)<PHONE_NUMBER>
<EMAIL_ADDRESS>skype: akash.rajput15

yes it will be
On Wed, Jan 6, 2021 at 1:04 PM Sebastian Hildebrandt <
<EMAIL_ADDRESS>wrote:

@artemis15 https://github.com/artemis15 thank you so much. I am sure,
this will be a quite nice enhancement for the package!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sebhildebrandt/systeminformation/issues/296#issuecomment-755136930,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB57KEDNFDHBRZERBTHC3IDSYQG7JANCNFSM4JODQBFA
.

--
Akash Rajput
Tel: (+91)<PHONE_NUMBER>
<EMAIL_ADDRESS>skype: akash.rajput15


Fixed #24833 - Changed references check to look on expressions instead of aggregates

This is to make to make Case expressions work with .exclude().
https://code.djangoproject.com/ticket/24833

To me it seems the commit message's ticket number is wrong. Othwerise LGTM.

@akaariai my bad, fixed it, thanks

merged in 469f1e362bb9670b174b37da9edd4631aff7badb, thanks!


Unable to sign up with new phone number

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Y
Reproducible in production?: Y
Found when executing PR: https://github.com/Expensify/App/pull/44515
Email or phone of affected tester (no customers):<PHONE_NUMBER>4
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Open www.staging.new.expensify.com
Enter new phone number using country code without a + to create a new account
Tap join

Expected Result:
New account is created
Actual Result:
Join button loads briefly then nothing happens
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/22e20581-a4d5-4afc-baa1-8213c935eb29
https://github.com/Expensify/App/assets/115492554/d95ac9e4-26dd-45b1-89ab-931405ad5e37

View all open jobs on GitHub

Tried on 9.0.2-0, entering "12025643894" doesn't allow to open signup modal. Error msg stop from proceeding signup.
It is only reproducible on staging. Maybe any regression raise this issue?

Asked if it's a regression here.

The bug is actually for VoIP numbers, we are not allowed to enter those, I guess this has been discussed previously as well, we block use of VoIP numbers

@allgandalf  Then creating a new account by VoIP num shouldn't be allowed!

Closing as not a problem, people won't generally be signing up with VoIP numbers.


version of caffe being benchmarked

Hi,
for Caffe mentioned in readme's chart below, may I know which Caffe branch it is referring to?

I think it is not Nvidia's experimental/fp16 Caffe branch  mentioned in Tx1 white paper.
This is because above graph shows for Googlenet batch1@fp16, above 51fps is 1.6x faster than 33fps in table below extracted from Tx1 white paper

Thank you.

The caffe branch was experimental/fp16 and max clocks. There had been JetPack/cuDNN/OS updates since initial Jetson TX1 launch.

Also, see issue https://github.com/dusty-nv/jetson-inference/issues/9 for updated nvcaffe install instructions for JetPack 2.3 / Ubuntu 16.04.  Thanks!

Thanks!


Firebase Testlab Tests taking too long on framework presubmits

I'm keeping an eye on the engine->fw roll pr: https://github.com/flutter/flutter/pull/118686. Most of the pending tests are firebase tests, that are taking > 1hr, previous runs have finished in < 35 mins on average.

link: https://ci.chromium.org/p/flutter/builders/try/Linux firebase_release_smoke_test

The Nexus run is pretty fast:

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8791614617499498721/+/u/test_execution/gcloud_firebase__2_/execution_details
But the Pixel 3/5/Motorola XT1650 run is really slow:

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8791614617499498721/+/u/test_execution/gcloud_firebase/execution_details
One device was pending for 23 minutes, maybe a capacity issue with one of those models?
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8791614617499498721/+/u/test_execution/gcloud_firebase/stdout

@sealesj infra on-call--is there some way to see if this is a Firebase Test Lab capacity issue?  There's nothing listed at https://status.firebase.google.com.   And if so, which model, and is it temporary or does a different model/SDK need to be picked?

This is blocking rolls, and the engine roll is almost 30 commits behind, so I will raise this to a P0.

Issue has been mitigated, moving to P3 to land the final fix.

@godofredoc is there a fix for this? The last comment suggests as much. Just wanted to follow up.

No, firebaselab is still working on the fix. I believe the best course of action is to select a new device that already has enough capacity in firebaselab

Ah okay so this is reliant on the firebase team?

That is correct!

Obsolete. The test has been running without a issue: https://ci.chromium.org/p/flutter/builders/try/Linux firebase_release_smoke_test?limit=100


Allow async / Future in the GraphQLConsumer builder

Is your feature request related to a problem? Please describe.
Yes -- I am using the GraphQLConsumer .. but the builder code won't let me use async so
when I try to await in the code...
Here is a code snippet :
   child: ScopedModelDescendant<MainModel>(
        builder: (BuildContext context, Widget child, MainModel model) {
      return GraphQLConsumer(builder: (GraphQLClient client) **async** {
        tasks = await _getTasks(model, client);
          return ColumnBuilder(
            itemCount: tasks.length,
            itemBuilder: (context, index) {

..
Throws an error :
[dart] The argument type '(BuildContext, Widget, MainModel) → Future' can't be assigned to the parameter type '(BuildContext, Widget, MainModel) → Widget'.
Describe the solution you'd like
Allow for a Future return so I can use async in my builder method?
A clear and concise description of what you want to happen.
I really have no idea...

@sjmcdowall not sure if you can use async build functions in Flutter at all. Anyway, have a look at the FutureBuilder class I think it might does what you want.

@HofmannZ -- Yep, that is exactly what I would up using and it worked pretty well too!  Thank you!


all-cabal-hashes: update snapshot to Hackage at 2018-09-12T08:26:27Z

(cherry picked from commit ee6ecb0eafd65637140c898ae563a2d2c0f530d2)
Motivation for this change
Things done

[ ] 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"
[ ] 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)
[x] Fits CONTRIBUTING.md.

Thanks!


Added test coverage for django.contrib.sitemaps.ping_google

As discovered whilst reporting #27523, it had no coverage.

merged in 10d49b96e667d3936c169e2bead2cf93748451af, thanks!