Warning: Native component for "FastImageView" does not exist

Thank you for the great work! I cannot wait to see it work on my end (^ ^)/
Here is my basic information:
MacOS Sierra
Expo 20.0.0
React Native 0.47.2
I installed the package following the instruction:

yarn add react-native-fast-image
react-native link

But when I use it, the terminal gives me this warning Warning: Native component for "FastImageView" does not exist, and NO image was shown, only red boxes. But if I change to React Native's Image component, things work as expected.

I wonder if it is because I am using expo? But I tried eject expo, and link again, but the warning persists, and no image was showing up. :(

+1

+1

+1

+1
Expo 20.0.0
React Native 0.47.2
npm 4.6.x

Projects that use react-native link are not compatible with Expo. In order to use a library like this, you must detach your application and use ExpoKit. More information can be found here.

Also, you may want to consider these alternatives:

https://github.com/wcandillon/react-native-img-cache
https://github.com/wcandillon/react-native-static-images

+1

@solababs +1 for what? The error?
This package will not work with Expo. Only an ejected app using ExpoKit.

@pcooney10 Sorry for the inconvenience. I opened this issue mainly because I failed to use the library after ejecting my expo app. Do I need to go to xcode for extra setup? Because I only ran react-native link after ejection.

@pcooney10 im not runnning on expo. im running in emulator and also tested on device. i get this red border and the image doesnt display. I used react-native-image-progress with it too and get the same result. on android tho.

My apologies - disregard my statement then.
I had looked into using this a while ago, and commented my two cents at the time (multiple people were referencing expo in their setup). I got a notification today and didn't read through the entire issue before posting.
I unfortunately do not have much experience with apps ejected from expo, but recall reading something about 'react-native link` not always playing nicely with CocoaPods. There is a brief note on it here: https://docs.expo.io/versions/latest/guides/expokit.html#ios.
This issue / PR might also be helpful (for those looking to use react-native link with ExpoKit projects): https://github.com/facebook/react-native/pull/15460. After a quick take, it seems that this should implemented in the coming version (v22), but I am not certain this will resolve your issue.

+1

+1 😞

What @pcooney10 said is correct, you cannot use this with a non-ejected app. This is because you cannot make native code additions or modifications when using expo.

I don't  use expo but have the same error. However, my app is not ejected - does this mean I should eject my app to use a library? Thanks!

I am not using expo, and using RN version 50.03, and still getting the same error?
i linked it using react-native link, but still not working.

@vitto-moz are you using create-react-native-app? If so, you are using expo.

no, I don't
react-native run-android  (for example)

@vitto-moz Ok. From what I've read, react-native link doesn't always work as expected, so I would maybe use that as your basis when trying to debug your issue. A lot of repos have instructions on how to link manually, so you may need to look through the source and see if all of the dependencies are getting added to your app's build.gradle and AndroidManifest.xml files.

A non-ejected app created using create-react-native-app uses expo.
react-native init myproject will create a plain React Native project.

@vitto-moz i also had the same issue today. my project was build using react-native init.  I agree with @pcooney10 the react-native link didn't work as expected.
This is what i did to fix:
I created a new project and only added react-native-fast-image and successfully linked it using react-native link and then copied all the changes related to react-native-fast-image(MainActivity, ...) in my former project and it worked. I hope this helps.

Seems the react-native link react-native-fast-image missed the step 2 described here https://facebook.github.io/react-native/docs/linking-libraries-ios.html. It worked after doing the step 2 manually.

@peteroid As I'm using pod on my iOS project, I guess I have to link manually too.
Did you add FLAnimatedImage and SDWebimage too?

Manual linking instructions have been added here: https://github.com/DylanVann/react-native-fast-image/blob/master/docs/installation-manual.md

@peteroid thanks , I'm ok with your method!~

Manual linking worked!


NOMINATE Plot: Porting to low resolution context, margins, and other display issues

I wired up the district page to be able to display NOMINATE results. There are at least two or three major issues with it, though:

The axis and margins are quite space intensive and mess up totally if I try to make the plot small
If I modify the exterior margins, the points don't plot properly on the oval overlay--even changing the dimensions of the plot seems to mess up the overlay, which makes me think the large overlay on the Congress page is actually wrong.
Even if it displayed properly. it's hard to parse meaning from it without a sense of temporal ordering (i.e. is my district moving right or left over time) -- and options like drawing arrows don't fit the small space.

So this is a space both to discuss what to do with the District page to display nominate (use a 1d histogram?) AND to discuss robustifying the decoration setup to be size invariant.

Yes, I should make all of the decorations size invariant.   However, I think it would be easier/cooler to make the the red an blue line the that runs through the district table list reflect the 1d nominate score. The rest I think could be found be clicks to the particular member page.

That's... an excellent idea. I will do that.

Can we have the median or at least a thin grey 0 line running through? hard to see what it means for districts that are largely one party.  Any way to more clearly signal it? Perhaps a more glamorous column title?

There are a few reasons why it's difficult to layer a second line in this context but I agree that's a good idea. In the mean time, I've added tooltips for the actual score as you mouse over.
When you say incumbency advantage, what do you mean?

It wasn't important. The solution you've implemented with the boundaries works very nicely.

Related to the above issue of porting the NOMINATE plot to be size independent, the tooltip code has magic constants for margin sizes in it. (Not a nag, Jeff, just wanted to write it here so we know for further followup)

No doubt.  decorate.js really needs to be re-engineered to rely more conventionally on d3 and which should allow more flexibility.   Ultimately, the problem to solve is figuring out how DC maps/places the scatter of points on the SVG canvas.   Right now that is done by inspection.  If it can be done by logic, we are home free.

After at bunch of reengineering (head scratching), the scatterplot is a lot more flexible.  Setting the width and margin in voteCharts.js is all that is needed.
The symbols, axis text and tick marks do not vary with the size of the plot, so better to probably better to leave the margins alone.
The tool tip no longer relies on magic constants.  Note that the height of the plot is determined by the aspect ratio of the ellipse which in turn depends up the value of nomDWeight.  For this reason care must be take not rely on the exact height of the plot.  A
ll-in-all a major upgrade.


Update Symfony to 4.3

This PR updates Symfony to 4.3 including the November 2018 changes so it would be a good idea to copy .env to .env.local before pulling these changes.

Hmm. But the project already is 4.3.2 so there is no really need to update.

Besides I have bad experience with .env.local / dist. We have project on Symfony 4.2.x at work and all this .env didnt wored at all. Only the .env was being loaded so I had to write custom controller to load data depending on setted up APP_ENV.
So I'm not changing .env unless this was typically 4.2 problem.

The November 2018 update includes changes to bin/console and public/index.php that loads a new file (config/bootstrap.php). This update is one that had to be applied manually. If you don't do the steps listed here the application won't load anything but .env.
It was already running 4.3.2 because composer.json used ^4.0. However dependencies changed since 4.0 symfony/lts was deprecated and is no longer used and Symfony's dependencies have changed slightly.
perhaps I made a mistake in saying "Update Symfony to 4.3". What I actually did was update the dependencies to match those of Symfony 4.3 by removing the deprecated ones and updating the requirement to "^4.3".

O, ok.  If so - did You encountered any problems?

Nope. I was able to run the server, build the assets and log into the application with fixture users. Browsed around a bit, etc. I wasn't able to run tests though, because, well... there aren't any yet.

Nice, but in my case this will have to wait. First i need to finish tagging mechanism - so this is definitely for later.
I'm not an TDD so I'm not working on tests.

Thanks. I apologize for taking your time.

Nothing happened. I plan to do this later anyway. That's a useful info for me. I just can't allow myself to break the code and then search for bugs or update without checking it out.


Mutation: Refetch Queries appears to broadcast twice?

Hi,
Great package! Really appreciate your work on this.
I am trying to create an optimistic update within a BLoC (flutter_bloc). Here is what I am doing.

I emit the new state within the bloc to optimistically update the UI.

emit(
        state.copyWith(
          productLists: state.productLists
              .map(
               ...etc
                ),
              )
              .toList(),
        ),
      );

Call a mutation to request update on the server

shelvesRepo.addProductToShelf().mutate(
            ShelfProductPatchModel(
              ...etc
            ),
          );

The addProductToShelf() mutation invalidates the query of the GET request which is providing the data to the UI via Query<type> as usual.

return Mutation<void, ShelfProductPatchModel>(
      refetchQueries: [_shelvesKey],
...etc

I am expecting to have this GET request stream update once and only once - when the API call returns with the updated data.
However, I seem to be getting two broadcasts into the stream, where the first broadcast is of the old (pre mutation) data, and the second is the updated (post mutation data).
So I am getting the following action.

UI updates optimistically on emit of new state.
UI receives data via stream. This is the pre-mutation data. The UI flickers back to original.
UI receives data via stream. This is the post mutation data. The UI flickers forward to again to match the optimistic update.

I can't use onStartMutation in this case because the args to the mutation are not enough for me to fully form the optimistic data models.
Are you able to shed any light on this? Should refetchQueries result in two broadcasts into the stream or is something going wrong here?
Thanks!
Rich

Hi Rich,
If I understand the problem properly, this is working as expected. When a query fetches it emits different states at different times in the life cycle. The first will be the loading state, which holds any existing data, then the success state, which will have the new data.
What I am assuming you are doing is emitting a new state directly from the bloc event before you call "mutate". The data in the "shelves" query is never updated and presumably is still listened to by the bloc; so when the loading state comes through it will update the UI with old data.
Fixing:
The easiest way to fix is to update the query data before you mutate. You don't need to use onStartMutation, that call back isn't doing anything special it is just called before the mutation happens. So what you can do is just update the shelves query manually instead of emitting from the block.
Replace step 1 with the following:
CachedQuery.instance.updateQuery(
  key: _shelvesKey,
  updateFn: (dynamic old) {
     // update the old data here
  },
);

This will trigger the "GET shelves" query to update its data and therefore update the UI optimistically. You will still see two stream broadcasts when refetching but the old data will now be the optimistically updated data.

Hi,
Thanks for the super fast reply. That all makes sense now and I'm glad I'm not going mad with the double broadcasts :)
The above process works perfectly!! I guess I could also check the status of the query in the bloc's emit.forEach() listener and act accordingly (i.e assume that data is old when in this state)?
Thanks again!
Rich

yeah, that would also work but might remove some of the benefits of cached query. It is usually best to treat the query as the source of truth for the api data.

That's true, and directly updating the query data works great within the flutter_bloc architecture as you described.
Thanks


Class method mocks are not working properly.

Code: application.cr file
class Application
  def self.exists?(message)
     "wolrd"
  end
end

Spec: applcation_spec.cr file
describe Application do
   it "should mock the class mthods" do
      allow(Application).to receive(self.exists?("hello")).and_return("world")
      Application.exists?("hello")
   end
end

Throwing following Error:
Error in ./spec/application_spec.cr:12: instantiating
Error in ./spec/application_spec.cr:12: instantiating 'Mocks::Allow(Application:Class)#to(Mocks::Message)'

    allow(Application).to receive(self.exists?("hello")).and_return("world")
                                 ^~

in ./libs/mocks/mocks/allow.cr:16: instantiating 'Mocks::Registry#fetch_method(String)'

      Mocks::Registry
         ^~~~~~~~~~~~

in ./libs/mocks/mocks/registry.cr:24: instantiating 'Hash(String, Mocks::Registry::Method)#fetch(String)'

      methods[method_name] = methods.fetch(method_name) {
                                     ^~~~~

in ./libs/mocks/mocks/registry.cr:24: instantiating 'Hash(String, Mocks::Registry::Method)#fetch(String)'

      methods[method_name] = methods.fetch(method_name) {
                                     ^~~~~

in ./libs/mocks/mocks/registry.cr:25: instantiating 'Mocks::Registry::Method:Class#new()'

        Method.new
               ^~~

instantiating 'Mocks::Registry::Method#initialize()'

in ./libs/mocks/mocks/registry.cr:52: instantiating 'Mocks::Registry::Stubs:Class#new()'

        @stubs = Stubs.new
                       ^~~

instantiating 'Mocks::Registry::Stubs#initialize()'

in ./libs/mocks/mocks/registry.cr:118: can't use Mocks::Registry::Result(T) as generic type argument yet, use a more specific type

        @hash = {} of {ObjectId, Args} => Result
                                          ^~~~~~

@spraveen1112 Thanks for the report! Could you tell which version of Crystal you use here? (0.10.0 ?)

@waterlink : yup, Crystal 0.10.0

@spraveen1112 Everything works correctly for me. It feels like you use way outdated version of mocks. Maybe the only released version at that.
If so, it is my fault for not releasing any new version (I myself use master branch everywhere for mocks).

And don't forget to actually define the mock itself: https://github.com/waterlink/mocks.cr/blob/master/spec/issues/2_spec.cr#L9-L11 - otherwise it is not gonna work, there is no way library can figure this information out with current state of the compiler (and probably never would be able to).

@spraveen1112 I have released 0.2.0, can you try adding version: 0.2.0 to your shard.yml's dependency for mocks and see if it works?

@waterlink: thanks.... its working fine now :+1:


Add start_date field

We're going to use this field as a sort of waitlist. Clubs with "ASAP"
or the current/next month as their start_date will be onboarded
immediately, while everyone else will only be onboarded when the month
they designated comes around.

Currently blocking on @kyleemile to confirm that we have the operations infrastructure to su.pport this change.
From slack

harrison> 1. we need to make sure we have acceptance copy for leaders who do not want to start straight away
harrison> 2. we need to make sure dariana knows not to reach out to clubs that aren't starting immediately

Feel free to assign me when it's ready!

@MaxWofford: Just got word from Kyle that we're ready to go!
Please review/merge at will.


Remote token bucket

@apostolosfilippas @onmyraedar @zer0dss: I'll add more to this over the next few days, but in a nutshell, I've created a new version of TokenBucket (TokenBucketClient) - there's a local version and then the client version that communicates with a FastAPI server (that itself is using the 'local' token bucket). Which one is used is set with a CONFIG / env variable. I'm imagining we'll eventually create the coop version so the cloud run instances can communicate with it. It will also require us protecting some of the routes.


memory issue in method

hey , the method doAnimateCycleWithRects.
the _weak is not needed     //__weak typeof(self) wSelf = self; //weak issue
should be typeof(self) wSelf = self;

(void)doAnimateCycleWithRects:(NSArray *)rects
{
//__weak typeof(self) wSelf = self; //weak issue
typeof(self) wSelf = self;
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.25 * 0.5 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
[wSelf animateRect:rects[0] withDuration:0.25];
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.25 * 0.5 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
[wSelf animateRect:rects[1] withDuration:0.2];
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.2 * 0.5 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
[wSelf animateRect:rects[2] withDuration:0.15];
});
});
});
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t) (0.6 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
[wSelf doAnimateCycleWithRects:rects];
});
}

@shattars3d Accidentally) Are you sure that there is no need for "weak"? How did you come up with it? How did you check that?


Tiered storage device support

We would like to create the first class notion of a tiered storage device implementing the IDevice interface. The device would take an ordered sequence of devices in the tiered hierarchy, e.g., [ LocalStorageDevice("C:..."), LocalStorageDevice("D:..."), AzurePageBlobStorageDevice("...") ]
It would also take in a parameter for commit level, i.e., after up to what level of completion do we return a "commit" for a page write. For example, we may wish to commit (1) as soon as the page is written to local storage, or (2) after the page is written to local storage and cloud storage.
Each tier would be associated with a range of addresses (segments) that are present at that tier. Reads are served by the highest tier that has the data. Each tier other than the last is also probably limited in size, e.g., 200GB for local SSD #1 via configuration. We can delete old segments from a tier by incrementing the valid start address for that segment and deleting the segment when it is safe to do so.

A first stab at the problem yields two questions:

SectorSize for a TieredStorageDevice is not clearly defined when underlying tiers have different sector sizes.
Each IDevice in the system right now keeps track of their own segments, with starting index 0. Additionally, each segment is assumed to be uniform in size. This makes it difficult to implement segments for TieredStorageDevice if segments are allowed to be partitioned across tiers (e.g. half of segment 2 is on disk, and the other half is in the cloud.)

Proceeding with the following assumptions/workarounds

Most, if not every IDevice will have SectorSize of 512. SectorSize also seems unused in the code based right now. I will just make TieredStorageDevice have sector size 512 for now.
I will enforce that segments are not spilled across tiers. This is equivalent to saying that each tier's capacity must be a multiple of their segment sizes. If segment sizes are reasonable, this should not be an issue.


Reuse connection for lambda serverless

Is your feature request related to a problem? Please describe.
I use kafkajs in my serverless application: when request come in It invoke a lambda, create a producer, connect to broker, send messages and then disconnect. That was "fine" for little request, for huge spike of request this is problematic: connection timed out. I'm thinking of reuse a producer connection and keep it a live in memory for a while until lambda is no longer hot.
Something like
const producerCache = {};  // This will be kept in lambda pod memory and survive between requests

export const handler = () => { // This handler is invoke in each request
  let producer = null;
  if (!producerCache[clientId]) {
    const kafka = new Kafka({ clientId });,
    producer = kafka.producer({ ... });
    producerCache[clientId] = producer;
  } else {
    producer = producerCache[clientId]
  }
  await producer.connect(); // I believe calling this multiple time will not cause a problem and only keep 1 connection?
}

Have anyone did this before? Is there any better way?

This is the normal way you keep any object from being GC'd, whether serverless or not. It's the same as if you're connecting to a database or anything else. You can move the await producer.connect into your if (!producerCache[clientId]) { case, since you only need to connect it when you first instantiate it. If the connection drops, it'll reconnect on demand. The connect/disconnect methods are there to signal intent.
You might still have a problem if you don't control the concurrency of your lambda function, since a spike in requests will cause multiple concurrent lambdas to start, each of which will have its own connections to you Kafka cluster, but this is handled by controlling the concurrency of your lambda - nothing specific to KafkaJS.
The only thing I would add is that you should probably add a listener for SIGTERM and disconnect your client when the lambda actually shuts down. Otherwise you'll leave half-open connections on the brokers, which will take some time to clean up. See https://github.com/aws-samples/graceful-shutdown-with-aws-lambda/blob/main/nodejs-demo/hello-world/app.js#L3-L12

Well look like lambda shutdown is not SIGTERM but SIGKILL so nothing can be done here I guess. Thank you for the answer

By default, yes, but if you register an extension the lambda does get sigterm - at least according to the documentation.

For a function with registered external extensions, Lambda supports graceful shutdown. When Lambda service is about to shut down the runtime, it sends a SIGTERM signal to the runtime and then a SHUTDOWN event to each registered external extension. Developers can catch the SIGTERM signal in their lambda functions and clean up resources.

Just want to share that to enable SIGTERM in an AWS lambda apparently you have to have some extension installed. What I understand is that as soon as you are running with an extension you get this extended live cycle mgmt. As I really just wanted to enable SIGTERM handling and nothing more, I went with this receipe:
https://github.com/aws-samples/graceful-shutdown-with-aws-lambda/tree/main/nodejs-demo
In a nutshell:

reference as lambda layer the already existing "Lambda Insights" extension
add an IAM role required for the extension

In terraform:
resource "aws_lambda_function" "my_func" {
  ...
  layers = ["arn:aws:lambda:eu-central-1:580247275435:layer:LambdaInsightsExtension:14"]
  ...

I am wondering if a slimmer no-op extension would do the same trick 🤔

Hi question, do we need to close connection of kafka or lambda function will automatically close it if lambda function died ?


Unable to retrieve details of the existing app my_test_app_name. Please check that the app is a part of the current subscription

Document's example doesn't work.
I followed the instructions using powershell:
az login
You have logged in. Now let us find all the subscriptions to which you have access...
[
  {
    "cloudName": "AzureCloud",
    "id": "<redacted>",
    "isDefault": true,
    "name": "Pay-As-You-Go",
    "state": "Enabled",
    "tenantId": "<redacted>",
    "user": {
      "name": "<redacted>",
      "type": "user"
    }
  }
]
mkdir quickstart
cd quickstart
git clone https://github.com/Azure-Samples/html-docs-hello-world.git
cd html-docs-hello-world
az webapp up --location westeurope --name my_test_app_name --html

Error:

Unable to retrieve details of the existing app my_test_app_name. Please check that the app is a part of the current subscription

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

ID: a825f031-250e-034d-e552-4b0d976d88b1
Version Independent ID: 626d7451-0b8e-051e-e512-0bf90be319a0
Content: QuickStart: Create a static HTML web app - Azure App Service
Content Source: articles/app-service/app-service-web-get-started-html.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

Hi @JeremyTCD Thanks for reaching out. We will review and update as appropriate.

@JeremyTCD, For creating a WebApp using Powershell use:
New-AzWebApp -ResourceGroupName "RGname" -Name "yoursitename" -Location "West US" -AppServicePlan "yourAppserviceplanname"
Reference.  Also, checkout 'az webapp up' and 'az webapp create' difference in cmdlets in this document.
Kindly refer this discussion thread - https://github.com/Azure/azure-cli/issues/9955.

Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply and we will gladly continue the discussion.


Module uses deprecated Dependencies tag

Throws an error in foundry right now, supposedly will cause issues in later versions if left in.

Throws an error in foundry right now, supposedly will cause issues in later versions if left in.

Is this still an issue? I updated to v12 and my players couldn't hear my music until I left a track looping for like 20 minutes.

Throws an error in foundry right now, supposedly will cause issues in later versions if left in.

Is this still an issue? I updated to v12 and my players couldn't hear my music until I left a track looping for like 20 minutes.

Hello, this warning should cause no issue in V12. But I did not check if the Module works in V12 yet as my world all use v11 or v10. I will look into it today.

@Anvillior fixed the module for Foundry V12, should now work properly.

@Anvillior fixed the module for Foundry V12, should now work properly.

Glad to hear it, and thanks for being so fast!


Volumetric calculations taking a ton of time on our server

We just switched over to volumetric instead of radius and we also moved to the threaded calculations (option 3). running a top in linux shows it at 680% cpu usage as compared to 95% or so before switching to this option. We've got the block size set to 24 from 32 as well. In the logs we also frequently see
[advancedRocketry]: Atmosphere calculation at zmaster587.libVulpes.util.HashedBlockPosition@2a49a56c aborted due to oversize queue!
Is there anything we can do to improve performance here besides lowering the block size more or switching back to radius?
Would it be possible in the future to see a configurable polling inerval for the oxygen mechanics, setting it to check far less often would be an acceptable sacrifice in realism vs playability for us for instance.

I believe i can improve the hashing function I'm using for hashed block which may speed things up.  I can also add a config that determines the maximum number of threads it's allowed to use.  The "aborted due to oversize queue" is basically a fancy way of saying "something is trying to recalculate atm on stations very frequently" which could come from something causing a constant block update, or multiple block updates.  Though if an area is already sealed and remains sealed after a block update, it should be a maximum of 6 checks which is lightning quick.  This leads me to beleive whatever it is is happening in an unsealed area.
I really want to avoid polling mechanics because in most cases that'll actually make it run slower and make the server likely to have sudden lag spikes.
Would it be at all possible to run opis or visualvm on the server so we can get an idea of what the situation actually is?

well its 1.10 so no opis unfortunately, I have warmroast but since it runs outside the main thread im not sure if that's going to pick that up, I could move it to the mainthread a bit to profile (option 2) tomorrow though and see what that finds

What i need to know in particular is where "zmaster587.advancedRocketry.util.AtmosphereBlob.addBlock" is being called from frequently.  That should be on the server thread somewhere.  The method itself won't take up much time because it just passes off to another thread.

So I suspect the issue is a player had a block placer/breaker for processing ores triggering repeated recalculations. We ran into a similar issue today when someone decided to us an rftools builder to altar their base while the oxygen vent was active. This resulted in a total server deadlock and since it was chunkloaded it had to be edited out. Here was a screencap of the profiler at the time:
http://imgur.com/a/OgXj2

Alright, hopefully this particular issue would be fixed by a modification
to the hashing function for blockpos.  The algorithm currently used results
in a lot more collisions in certain circumstances than a more improved
algorithm I've been using in another project as of late.  I'll port it
later today and post a build.  If you would be able to test and report back
on performance difference I would greatly appreciate it.  I'll post here
when I push the build

Can you try with this version of libvulpes and see if it helps?
AR 1.1.x builds: http://<IP_ADDRESS>:8080/job/LibVulpes/job/libVulpes/job/unstable1_10/
AR 1.0.x builds: http://<IP_ADDRESS>:8080/job/LibVulpes/job/libVulpes/job/MC1_10/

This does seem to help quite a bit

But does it get timin down to a reasonable value?

it does seem so, similar activity with builders wand etc didnt spike the server tps like it was, thats a bit anecdotal but additionally the profiler showed it as vastly reduce by about 75% of what it was.


Bug: JSON schema 'conditionals' validation and null values not working properly

# values.yaml
credit_card: "test"
billing_address:

// values.json.schema
{
  "$schema": "https://json-schema.org/draft-07/schema#",
  "title": "CNOM Server - Helm chart values JSON schema",
  "type": "object",
  "dependencies": {
    "credit_card": ["billing_address"],
    "billing_address": ["credit_card"]
  },
  "properties": {
    "credit_card": {
      "description": "TODO",
      "type": ["string", "null"]
    },
    "billing_address": {
      "billing_address": "TODO",
      "type": ["string", "null"]
    }
  }
}

With the above code, I would expect it to fail with an error like:

Message: Dependencies for property 'credit_card' failed. Missing required keys: billing_address.
Schema path: #/dependencies

It does show such an error when using the below as seen here https://www.jsonschemavalidator.net/s/IIXnyest:
{
  "credit_card": "test"
}

It does not show the error when using it like seen here https://www.jsonschemavalidator.net/s/wd016qoV:
{
  "credit_card": "test",
  "billing_address": null
}

I would however from a Helm perspective think that the values.yaml as shown here should generate an error. Effectively not passing in the billing_address to the schema validation at all. So Helm should "preprocess" the message and completely remove properties that are null before passing it to the JSON schema valdation. Otherwise the construct of conditionals as described here https://json-schema.org/understanding-json-schema/reference/conditionals.html using dependencies will not work at all (it was split into dependentRequired and dependentSchemas in Draft 2019-09, but that is not of importance for this issue).
I am aware of issues described in https://github.com/helm/helm/issues/10732, but I do not believe this is an issue in the JSON schema validation library, but in Helm itself.
Please also note that this does validate properly, but this is not good practice to omit proper values from values.yaml (even if null by default):
# values.yaml
credit_card:

Output of helm version:
version.BuildInfo{Version:"v3.8.0", GitCommit:"d14138609b01886f544b2025f5000351c9eb092e", GitTreeState:"clean", GoVersion:"go1.17.5"}

I think this comes down to golang's parsing of yaml. If you unmarshal a key with no/an empty value, it defaults the value to null (as you observe). But this means that helm later can not tell if the key was specified with a null value, or was simply empty.
go playground example to demonstrate: https://go.dev/play/p/YzVis6-TYB5
I would have thought that your example would be invalid yaml. I think the follow up here would be to read the yaml spec, understand what the correct behavior is, and potentially follow up with a report on https://github.com/go-yaml/yaml (note: helm strictly uses https://github.com/kubernetes-sigs/yaml, which I think in turn uses go-yaml/yaml)

For your immediate issue, I have seen folk do:
# values.yaml
credit_card: "test"
# billing_address:

i.e. comment out optional values.yaml fields. For which a user can remove the comment if they want to specify.

@gjenkins8 thanks for the great reply, this was also our thinking that it might be the YAML parser and how it behaves.
I found https://github.com/go-yaml/yaml/issues/681 which seems to indicate v3 actually does remove the key. So is Helm using v2 of https://github.com/go-yaml/yam (via https://github.com/kubernetes-sigs/yaml)?

I took a look at the yaml spec. I think Example 7.3 Completely Empty Flow Nodes shows that the above is actually valid yaml (and/or yaml doesn't actually require content for values). Which also means that go-yaml/yaml is behaving correctly (note: I used v3 in my go playground example)
My conclusion is that it isn't possible to do exactly what you want here. If you specify a key without a value in values.yaml, parsing will default to a null value. And it isn't later possible to distinguish between that null and an explicit null later.

I am not particularly fond of commenting out valid properties, at it can be interpreted by users as "this is experimental, do not use this".

Also note, you could comment and/or not include experimental values in the charts values.yaml

I am not particularly fond of commenting out valid properties, at it can be interpreted by users as "this is experimental, do not use this".

Also note, you could comment the experimental status and/or not include experimental values in the charts values.yaml

This is very common practice in helm charts, so there should be no confusion if your values file is well documented.

FWIW, if you have a schema, you can use that one as ground truth for values instead of values.yaml. For instance, you could generate documentation from the schema, and much more easily than from a commented YAML file.
Then, the need to list all admissable values in values.yaml goes away.

Would an issue still not be if someone supplies their own values.yaml via --values where they set a value, but to null? Then using dependencies would still fail. So I am not sure commenting values is the best approach to solve this.

Yes, if a user set it to null, it would delete the empty map and break the chart. This could be caught, as @reitzig suggested, with a json schema. The other option is that the user would break the chart, read the values.yaml as documentation, realize their mistake and fix their own values file.

I do not find commenting out values in the values.yaml file a good approach. It is not as readable and immediately obvious what values can be configured, so even if some charts do that,  it does not mean it is great. It would enforce a stylistic choice onto charts that is implicitly needed to solve using JSON schemas properly in all cases.
In my original post I want to use the dependencies JSON schema keyword. I then deduct there is no logical way to do this when at the same time properly showing what values can be configured in values.yaml (without commenting them out) as of today?

I do not find commenting out values in the values.yaml file a good approach.

That's fine. I don't like tabs instead of spaces. Common practices are usually best to follow, however, when publishing works within a particular idiom. It causes less confusion and reduces the amount of issues filed. I've seen alternatives tried and speak from experience.
Advice out of the way, here's the best I can do to help with your specific request:
The code that checks the values against the schema is here: https://github.com/helm/helm/blob/main/pkg/chartutil/jsonschema.go
Is uses this schema library: https://github.com/xeipuuv/gojsonschema
and this yaml parser: https://github.com/kubernetes-sigs/yaml
If one of those libraries isn't implementing their spec correctly, please file an issue upstream so we can have that fix.

I'm going to go ahead and close this as answered.


Can not install command line tool

Have tried to install Djot.js several times.
I have to run the install script as sudo to get it to work since the admin user doesn't have the right levels of permisions to the /usr/local/lib/node_modules directory where Djot can be installed.
When I type
djot into the terminal it says: Command not found.
I installed Node.js from their website instead of using Homebrew hoping this would give better results.

Did the installation succeed?  Do you see djot in /usr/local/bin?  If so, is /usr/local/bin in your PATH?
Alternatively you can install your global npm packages in your user directory (and not need sudo):
mkdir ~/.npm-packages
npm config set prefix ~/.npm-packages
npm install -g .

djot should then be in ~/.npm-packages/bin, which you'd ned to add to your PATH.

No, I do not see djot in /usr/local/bin even when I ran the installer as sudo. I did not get any errors while installing so wonder what failed and how I can debug this.

Try the approach above for a user directory install.

Ok, now I got Node and NPM working as expected, but the installer fails here again.
´´´
I ran:
npm run build
that returned

@.*** build
tsc && webpack
sh: tsc: command not found

´´´

Ok finally got the tsc installed as well, so now I run the installer which gives this error:
Ran this from the djot repo directory after the npm run build
´´´
npm install -g djot-js
npm ERR! code ENOVERSIONS
npm ERR! No versions available for djot-js
´´´
That's interesting since the build gave me a version number, so wonder what went wrong here.

This is because I unpublished the package (we're thinking of a different name). Try this:
npm install -g https://github.com/jgm/djot.js

I can try to fix npm run build so it doesn't assume a global install of tsc.

Thanks, I fixed it by using your suggestion in a recloned repo directory, and now it finally got installed!
That's fantastic.
Yes, it would be great if the TSC was mentioned if not installed. This would have saved me quite some hours reinstalling stuff. It was just this morning I found that TSC wasn't installed.
Depending on the future of Djot, would it make any sense to make some more userfriendly documentation for the installation of Djot.js?

I think I've now fixed the tsc issue.
Once we get this published on npm it should be a simple npm install -g djot (or whatever the npm name turns out to be).

Cool, I have now experimented for some hours, and i absolutely LOVE Djot!
I really hope this format can be included with Pandoc and other tools. I really see Djot as a great alternative for Markdown in so many cases. Both for people who writes blogs or just want to format their documents before converting them to docx or other known formats.
Djot has a very easy to learn syntax.
I am a Braille reader since I have no eye sight, and Djot makes formating my document so easy. I liked Markdown and used it for years, but I think DJOT may replace this when it is released.
If you are interested I could make some notes in a discussion over at the syntax repo.
Thanks for a great project and for helping me getting this to work!
Sendt fra min Mac Mini via Apple Mail

Den 6. jan. 2023 kl. 21.02 skrev John MacFarlane @.***>:
I think I've now fixed the tsc issue.
Once we get this published on npm it should be a simple npm install -g djot (or whatever the npm name turns out to be).
—
Reply to this email directly, view it on GitHub https://github.com/jgm/djot.js/issues/10#issuecomment-1374066795, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGGLXRQPUGPBGJ63X7GFLOTWRB24LANCNFSM6AAAAAATQAOUA4.
You are receiving this because you authored the thread.

Glad to hear you finally got it to work!


Initial implementation benchmarks for the rune cli

We use the criterion benchmark framework for this task, as the built in rust benchmarks aren't yet available in rust stable.
Each of the benchmarks use a sample size of 10 instead of the default 100.
These benchmarks measure the time taken for:

build - sine rune
inference - sine rune, microspeech, style transfer.

The results on my Linux Ryzen 7 1700 machine look like:

~TODO: Try to enable Github Actions for these benchmarks~ These benchmarks can be run manually before each release, so we aren't adding them to github actions for now

This is a long time coming!!


do something with enums

Reported by<EMAIL_ADDRESS>Aug 25, 2011
manually migrated from Issue 2 - 5 people starred - Type-Defect - Priority-Medium
The JSON has enums described, showing possible values.
Should probably promote to documentation at least, if not generate types or run-time checks on setters.

Closing issue in preparation for auto-migration to Github.


Missing License

Hi, I would like to use windows_task custom_type for a project I am involved in but cannot do so if the license is not specified. Thanks!

Hello @RyanDickson ,
Sorry for the delayed reply I've been away for a few days.
Thank you for pointing out that I have not included any licensing information, I do apologise and will add licensing details, my preference is to go with Unlicense or MIT.
Feel free to update and improve the code, please do push any changes back.
Thanks again.

Added license (MIT) and Rspec tests.


InitialSync not possible due to malformed JSON

So I have an unfortunate problem. While my laptop and desktop can sync just fine with my Synapse instance, the Android client seems to fail on some malformed JSON returned by the server. Even after deleting and redownloading Riot, and signing in again, upon syncing I get a 'Network Error' toast popping up during the Riot.im loading screen.
Using adb logcat, I found the following logs appear:
05-28 10:39:27.889 16335 16349 I im.vector.alpha: Background concurrent copying GC freed 645765(21MB) AllocSpace objects, 1(24KB) LOS objects, 49% free, 19MB/39MB, paused 392us total 213.840ms
05-28 10:39:28.113 16335 16335 D RestAdapterCallback: ## failure(): [syncFromToken] with error Use JsonReader.setLenient(true) to accept malformed JSON at line 1 column 7208507 path $.rooms.join..state.events[5].content.users.@walle303:kek.community
05-28 10:39:28.114 16335 16335 E EventsThread: Sync V2 onNetworkError Use JsonReader.setLenient(true) to accept malformed JSON at line 1 column 7208507 path $.rooms.join..state.events[5].content.users.@walle303:kek.community
05-28 10:39:28.114 16335 16335 E ErrorListener: Network error: Use JsonReader.setLenient(true) to accept malformed JSON at line 1 column 7208507 path $.rooms.join..state.events[5].content.users.@walle303:kek.community
05-28 10:39:28.127 16335 16335 I EventsThread: Waiting a bit before retrying

There's obviously some malformed JSON going on here, though I'm sure how to 1. see what exactly it's tripping on, 2. attempt to move past it.
And from your side I'm not entirely sure what the correct solution is to handle this, though somehow other clients are able to.

this is probably walle303 shoving bigger than maxint timestamps in their events or similar. the way to diagnose is to run a manual initial sync with curl on your account (literally just GET /sync) and the spelunk with jq at the path where android is barfing. or just grep for walle303 membership events to find out dodgy looking ones.

Sounds good, will do thanks.
On 05/28/2018 11:30 AM, Matthew Hodgson wrote:

this is probably walle303 shoving bigger than maxint timestamps in
their events or similar. the way to diagnose is to run a manual
initial sync with curl on your account (literally just GET /sync) and
the spelunk with jq at the path where android is barfing. or just grep
for walle303 membership events to find out dodgy looking ones.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vector-im/riot-android/issues/2303#issuecomment-392487832,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABR7mJa9qbv2hv4tKwHo_MhVHmTVjOl5ks5t29HAgaJpZM4UP98I.

ah.

Yep, left the 1-1 room and now am fine. Perhaps Synapse should filter these? :)


Context menu not keyboard accessible

Bug already reported?

[X] I confirm that I have checked if the bug already has been reported

For which framework/library you are reporting the bug
Angular
Component name
More menu
Description
When navigation with the keyboard to the More menu and press enter, next focus is not i the opened menu. When you use tab, focus moves to the top of the browser (in chrome), through adress bar, to the opened menu and back to the More button. You cannot move focus to the opened menu with arrow keys either.
Steps To Reproduce
Upgrade to version @sebgroup/green-angular": "^2.1.2"
Run application
Navigate using the keyboard to a More menu button
Press enter
Try to navigate to the items in the opened context menu
Current Behaviour
It is not possible to select the items in the opened context menu using the keyboard
Expected Behaviour
It should be possible to navigate using only the keyboard

Hi @anneliev, thank you for reporting this issue!
I'm not able to reproduce it when testing it in Storybook: https://sebgroup.github.io/green/latest/angular/?path=/docs/components-context-menu--custom-template-default
If possible, it would be great if you could send me a link on Teams to where the issue can be reproduced.
For reference, here is how the keyboard interactions are supposed to work:

Enter or key-down when focus is on trigger opens the popover and focus moves to the first option.
Arrow-up and arrow-down to move focus between options while the popover is open.
Tab should close the popover and return focus to the trigger button.

Basically, it should follow the menuitem role recommendations from MDN: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/menuitem_role

Hi,
As you say it works as intended in Storybook, but not when we implement it. You can see it here : https://internalapps.tst.sebank.se/nexus/payments/payment-investigation
Regards, Annelie

Hi, As you say it works as intended in Storybook, but not when we implement it. You can see it here :
Regards, Annelie

Thank you, I will take a look!

@anneliev It seems like this issue originates somewhere else in the app (or shell). Probably something like focus or keyboard event getting intercepted before it reaches the component, and therefore the component can't respond to it.
I will close this issue now, but feel free to re-open, or open a new issue, if you find that there is something wrong with the component itself.


Fixed BeanNoSuchBeanDefinitionException caused by use-default-objectmapper=false & fixed ability to use ObjectMapperConfigurer

Fixed BeanNoSuchBeanDefinitionException thrown by GraphQLWebAutoConfiguration.graphQLObjectMapper when graphql.servlet.use-default-objectmapper is set to false and no bean for ObjectMapperProvider is provided by an application.
This also allows users to specify an ObjectMapperConfigurer bean. Currently, it will never be used because a ObjectMapperProvider bean is essentially required to always exist.

Thanks @bsara !

@oliemansm thank you for the merge!


Clarify that authority is not automatically replicated

A lot of users, (including myself in the past), are unaware that multiplayer authority is not automatically replicated. I've added a note to the docs, and also provided guidance on how to solve the problem.

Thanks!


Add skipForwarding option for Orion

Proposed changes
This PR adds --skipForwarding option for Orion.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[ ] I have signed the CLA
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
N/A

Coverage remained the same at 100.0% when pulling 0aaa05d56f8cfc16a8c356dcc55d47df2f7e0191 on fisuda:feature/orion_skip_forwarding into 8292e88856ff8045a4272c9d5c70056598210f8e on lets-fiware:main.


Add a Gitter chat badge to README.md

hyperoslo/facebook-messenger now has a Chat Room on Gitter
@timkurvers has just created a chat room. You can visit it here: https://gitter.im/hyperoslo/facebook-messenger.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Coverage remained the same at 99.502% when pulling 723cd8bd553a2899aca817674f69529c8ac3b02e on gitter-badger:gitter-badge into 563b77751410bb005b8f60009959c9a7cdc19a90 on hyperoslo:master.

Nice! See you there!

Such Gitter Badger! :grinning:


ENH: Added loggamma entropy method

Reference issue

Part of #17832
What does this implement/fix?

Adds a formula for the loggamma entropy, as well as an asymptotic expansion for larger values.
Additional information

I wasn't able to get mpmath to generate test values with integration, so if anybody else could get it to work, that'd be helpful. There was discussion in previous PRs about using sympy or Wolfram Alpha to help get the asymptotic expansion for entropies. Since the expansion for this method is already very accurate, I decided that wasn't necessary. Attached to this PR is a writeup explaining how I derived the entropy formula and the asymptotic expansion.

loggamma_entropy.pdf

@mdhaber The changes you requested in #18122 are in this PR.

@mdhaber The changes you requested in https://github.com/scipy/scipy/pull/18122 are in this PR.
Thank you! But

There was discussion in previous PRs about using sympy or Wolfram Alpha to help get the asymptotic expansion for entropies. Since the expansion for this method is already very accurate, I decided that wasn't necessary.

I thought I should rephrase my previous comments about this (https://github.com/scipy/scipy/pull/18135#issuecomment-1468869071, https://github.com/scipy/scipy/pull/18094#issuecomment-1467440742): the purpose of using sympy or Wolfram Alpha is not to get more accurate asymptotic expansions. Rather, it is to simplify development, avoid (reasonably accurate but nonetheless) incorrect expansions, and speed up review. You don't need to follow this suggestion, but I wouldn't want it to be dismissed due to a misunderstanding.
For example, instead of undertaking the derivation by hand and preparing the PDF, it may have saved you time to begin with the sympy code you referred to (https://github.com/scipy/scipy/pull/18094#issuecomment-1467440742). By adding the symbol c instead of df and redefinining:
old =  gammaln(c) - c * digamma(c) + c

the script pops out an answer:
-log(c)/2 + log(2)/2 + 1/2 + log(pi)/2 + 1/(6*c) - 1/(90*c**3)

(If you want the `c**-5` term, just add it to the expansion of `gammaln` before executing the code. There is a balance between the accuracy and complexity of adding extra terms, and where to stop is a matter of choice. I got the impression before that you did not want to add terms past $1/c^4$ (https://github.com/scipy/scipy/pull/18131#issuecomment-1469002212), and that's fine. Either way, I don't think it undermines the point of the PR.)

Also, by letting the computer do the symbol manipulation, we avoid the possibility of mistakes in the algebra. Sometimes these mistakes are subtle - we've seen this a few times (https://github.com/scipy/scipy/pull/18094#issuecomment-1467440742, https://github.com/scipy/scipy/pull/18122#issuecomment-1467473461, https://github.com/scipy/scipy/pull/17874#discussion_r1089880639). For instance, the expansion can give good results even when there are mistakes, but results are even better when the math is correct. Moreover, committing implementations of mathematically incorrect formulas can erode confidence in the rest of SciPy, so I prefer to avoid it when possible.
Finally, although I believed that the derivation was correct in this case (because it was better than the default implementation for x as small as ~40), I still need to check it before it can be merged. For me, it's easier to review the line
old =  gammaln(c) - c * digamma(c) + c

than it is to review a two-page PDF, despite its careful preparation.
There may be other reviewers with different preferences, but this is my suggestion to help you get your contributions reviewed efficiently.

Rather, it is to simplify development, avoid (reasonably accurate but nonetheless) incorrect expansions, and speed up review.

Okay, now I understand. I'll consider using computer assistance to validate my expansions, but I'm still going to do them by hand as well, just in case.

I got the impression before that you did not want to add terms past 1 / c^4

That was because the PR in question was for an issue that specifically focused on large values. If you focus too much on adding more terms, the PR stops becoming about large values and more about getting the most accurate asymptotic expansion. I know that's pretty pedantic, but I think it's good to make sure every PR sticks to a well-defined goal.

but I think it's good to make sure every PR sticks to a well-defined goal.

Absolutely. I'm sorry for the misunderstanding, but in that discussion, I didn't ask for there to be more terms. I hoped to clear that up when I wrote:

No, I didn't suggest adding more terms.

I'm sorry for the misunderstanding

No big deal, these things happen. At least we're now on the same page.

This PR fixes the entropy for small and large c:

from mpmath import mp
from scipy import special as sc
import numpy as np
import matplotlib.pyplot as plt

mp.dps = 500

@np.vectorize
def loggamma_entropy_mpmath(c):
    c = mp.mpf(c)
    return float(mp.log(mp.gamma(c)) + c * (mp.one - mp.digamma(c)))

def loggamma_entropy(c):
    return np.where(c < 45,
                    sc.gammaln(c) - c * sc.digamma(c) + c,
                    -0.5 * np.log(c) + (1 / 6) * c ** -1. -(1 / 90) * c ** -3. + (1 / 210) * c ** -5. + 0.5 + 0.5 * np.log(2 * np.pi))

c = np.logspace(-8, 20, 1000)
plt.semilogx(c, stats.loggamma.entropy(c), label = "main", ls='dotted')
plt.semilogx(c, loggamma_entropy(c), label = "PR", ls='dashdot')
plt.semilogx(c, np.array(loggamma_entropy_mpmath(c), np.float64), label = "mpmath", ls='dashed')
plt.legend()
plt.show()

I propose to commit my suggestions to simplify the tests and then merge it as it is a definite improvement.

Ok, I've merged the suggestion made by @dschmitz89.

@mdhaber any updates on this?

I didn't plan to review this further right now. Someone else is welcome to.

I didn't plan to review this further right now. Someone else is welcome to.

@dschmitz89 reviewed this 2 weeks ago and approved the changes, along with the recommendation to merge this PR once I committed his suggestions.

I discussed this a bit with Pamphile and Matt in person and I think the decision was to bump the milestone at this point, or at least I didn't get a clear indication that we were comfortable merging at this point.
I'll bump the milestone and happily merge once folks are convinced we're ready to put this in.

It looks like I missed the ping and this is still waiting for a review from me. I’ll try to get to it today.

Re: https://github.com/scipy/scipy/pull/18156#issuecomment-1474758635, I don't think we need to worry about negative c because loggamma is not defined for negative c.

Failures so far are unrelated (CircleCi, Linux Meson Python Debug & ATLAS)


Dev data is missing sector information on the project overview [6]

Describe the Bug:
Probability (how likely the bug is to happen, scored from 1-5): 5
(For example, probability of 5 is something like "it happens to all users every time they log in."
Probability of 1 "only happens to certain users when a really specific and unlikely path is followed.")
Effect (how bad the bug is when it does happen, scored from 1-5): 2
Steps to reproduce the behaviour:

Edit a project created by the dev script without touching the project overview
Save the revision

There is an error on the project overview form
Screenshots:
Additional information:

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]
Additional context

This card was addressed as part of this card.


Error while running on cloud

This is the error shown in the error  logs when I try to run on the cloud.
File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main "__main__", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code exec code in run_globals File "/root/.local/lib/python2.7/site-packages/luminoth/train.py", line 330, in <module> train()
File "/root/.local/lib/python2.7/site-packages/click/core.py", line 722, in __call__ return self.main(*args, **kwargs) File "/root/.local/lib/python2.7/site-packages/click/core.py", line 697, in main rv = self.invoke(ctx)
File "/root/.local/lib/python2.7/site-packages/click/core.py", line 895, in invoke return ctx.invoke(self.callback, **ctx.params)
File "/root/.local/lib/python2.7/site-packages/click/core.py", line 535, in invoke return callback(*args, **kwargs)
File "/root/.local/lib/python2.7/site-packages/luminoth/train.py", line 296, in train config = get_config(config_files, override_params=override_params)
File "/root/.local/lib/python2.7/site-packages/luminoth/utils/config.py", line 17, in get_config model_base_config = get_base_config(model_class)
File "/root/.local/lib/python2.7/site-packages/luminoth/utils/config.py", line 63, in get_base_config return load_config_files([config_path])
File "/root/.local/lib/python2.7/site-packages/luminoth/utils/config.py", line 38, in load_config_files new_config = EasyDict(yaml.load(f))
File "/usr/local/lib/python2.7/dist-packages/yaml/__init__.py", line 69, in load loader = Loader(stream) File "/usr/local/lib/python2.7/dist-packages/yaml/loader.py", line 34, in __init__ Reader.__init__(self, stream)
File "/usr/local/lib/python2.7/dist-packages/yaml/reader.py", line 85, in __init__ self.determine_encoding()
File "/usr/local/lib/python2.7/dist-packages/yaml/reader.py", line 124, in determine_encoding self.update_raw()
File "/usr/local/lib/python2.7/dist-packages/yaml/reader.py", line 178, in update_raw data = self.stream.read(size)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/lib/io/file_io.py", line 125, in read self._preread_check()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/lib/io/file_io.py", line 85, in _preread_check compat.as_bytes(self.__name), 1024 * 512, status)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/errors_impl.py", line 519, in __exit__ c_api.TF_GetCode(self.status.status))
NotFoundError: /root/.local/lib/python2.7/site-packages/luminoth/models/fasterrcnn/base_config.yml; No such file or directory

After this the logs indicated that a cleanup was finished followed by this error
The replica master 0 exited with a non-zero status of 1. Traceback (most recent call last): [...] File "/root/.local/lib/python2.7/site-packages/click/core.py", line 535, in invoke return callback(*args, **kwargs)
File "/root/.local/lib/python2.7/site-packages/luminoth/train.py", line 296, in train config = get_config(config_files, override_params=override_params)
File "/root/.local/lib/python2.7/site-packages/luminoth/utils/config.py", line 17, in get_config model_base_config = get_base_config(model_class)
File "/root/.local/lib/python2.7/site-packages/luminoth/utils/config.py", line 63, in get_base_config return load_config_files([config_path])
File "/root/.local/lib/python2.7/site-packages/luminoth/utils/config.py", line 38, in load_config_files new_config = EasyDict(yaml.load(f))
File "/usr/local/lib/python2.7/dist-packages/yaml/__init__.py", line 69, in load loader = Loader(stream)
File "/usr/local/lib/python2.7/dist-packages/yaml/loader.py", line 34, in __init__ Reader.__init__(self, stream)
File "/usr/local/lib/python2.7/dist-packages/yaml/reader.py", line 85, in __init__ self.determine_encoding()
File "/usr/local/lib/python2.7/dist-packages/yaml/reader.py", line 124, in determine_encoding self.update_raw()
File "/usr/local/lib/python2.7/dist-packages/yaml/reader.py", line 178, in update_raw data = self.stream.read(size)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/lib/io/file_io.py", line 125, in read self._preread_check()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/lib/io/file_io.py", line 85, in _preread_check compat.as_bytes(self.__name), 1024 * 512, status)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/errors_impl.py", line 519, in __exit__ c_api.TF_GetCode(self.status.status))
NotFoundError: /root/.local/lib/python2.7/site-packages/luminoth/models/fasterrcnn/base_config.yml; No such file or directory To find out more about why your job exited please check the logs: https://console.cloud.google.com/logs/viewer?project=484504151094&resource=ml_job%2Fjob_id%2Ftrain_20181104_131838&advancedFilter=resource.type%3D%22ml_job%22%0Aresource.labels.job_id%3D%22train_20181104_131838%22

I tried running this an additional time, by changing the number of epochs in the fasterrcnn.py file to 100 instead of the default 1000, but that resulted in the same errors.

Related to #231.


All logs are logged to systemd at level "info", regardless of error level

runtime cilium $ journalctl -u cilium --priority=6 | grep debug | wc -l
21931
runtime cilium $ journalctl -u cilium --priority=7 | grep debug | wc -l
21931
runtime cilium $ journalctl -u cilium --priority=7 | grep info | wc -l
1891
runtime cilium $ journalctl -u cilium --priority=6 | grep info | wc -l
1891
runtime cilium $ journalctl -u cilium --priority=6 | grep err | wc -l
13
runtime cilium $ journalctl -u cilium --priority=5 | grep err | wc -l
0

According to the journalctl documentation here:
-p, --priority=
Filter output by message priorities or priority ranges. Takes either a single numeric or textual log level (i.e. between 0/"emerg" and 7/"debug"), or a range of numeric/text log levels in the form FROM..TO. The log levels are the usual syslog log levels as documented in syslog(3), i.e. "emerg" (0), "alert" (1), "crit" (2), "err" (3), "warning" (4), "notice" (5), "info" (6), "debug" (7). If a single log level is specified, all messages with this log level or a lower (hence more important) log level are shown. If a range is specified, all messages within the range are shown, including both the start and the end value of the range. This will add "PRIORITY=" matches for the specified priorities.

I don't know if this is a Cilium issue or a logrus issue.

Reviving this because I'm trawling through old issues , but it looks like this is because Cilium logs to stdout / stderr directly, and not to journald / systemd themselves. See
https://github.com/docker/for-linux/issues/127#issuecomment-337045289
We could perhaps add journald / systemd as a log driver within Cilium logging configuration and then use that in the developer VM systemd files in the repo: https://github.com/wercker/journalhook

@ianvernon completely agree, that looks like the appropriate fix. If someone wants to take up this work, feel free to re-open this ticket and assign to yourself. For now, it's not a high enough priority to schedule in an upcoming release.


fancy p3js.AnimationMixer timeline widget

Hi @romanarust
https://github.com/compas-ITA19/ITA19/blob/c4b10dc183599eed4ed60d922b6ef5922d173bdb/modules/module2/viewer/threejs_viewer.py#L122-L125
I share the 💖for pythreejs and the animation approach is incredible cool and sensible
The UI rendering the animation sequence works but is limited in scope.
So I've looked into whether more polished UI's exist - they do see here, to give you an idea:

So what could be cool if such a widget pops up when rendering a p3js.AnimationMixer instance in a notebook...
Wanna take a stab at this with me?

Wow, so cool! Remembering making animations with AfterEffects ;)
Wanna take a stab at this with me? - Yes; definitely! I will copy and clean up the code into another repo and invite you.

@romanarust wooohooohooo, wonderful, I look fwd to the invite!


FStar.List.Tot.Properties.append_mem is missing SMTPat

Hello!
I just found this very helpful property has its SMTPat commented out:
https://github.com/fstarlang/FStar/blob/master/ulib/FStar.List.Tot.Properties.fst#L191-L199
val append_mem: #t:eqtype ->  l1:list t
              -> l2:list t
              -> a:t
              -> Lemma (requires True)
                       (ensures (mem a (l1@l2) = (mem a l1 || mem a l2)))
                       (* [SMTPat (mem a (l1@l2))] *)
let rec append_mem #t l1 l2 a = match l1 with
  | [] -> ()
  | hd::tl -> append_mem tl l2 a

For want of this pattern, we don't enjoy automated proof for properties such as
let state_concat_has_both (vs vs': list vars) (s:state{state_has s (vs @ vs')})
    : Lemma (state_has s vs /\ state_has s vs')
    = ()

In general, it seems like a useful prop to have in the theory of lists. I assume this was commented out for a reason, but the git blame doesn't turn up anything obvious for me.
Would it be feasible to uncomment this? If not, perhaps a comment can be added explaining why it is omitted.
I'd be pleased to contribute either change, in case that is helpful.
Thanks for the amazing software!

Adding this pattern to append_mem is indeed fairly natural, but putting it in the standard library will make proofs much more expensive for many clients. Every mem a (l1 @ l2) in scope will introduce an SMT case split.
Instead, it may be better to introduce this lemma in scope when needed, rather than globally.
For example, one might write:
let append_mem_auto ()
  : squash
    (∀ (a:eqtype) (x:a) (l1 l2:list a).
       {:pattern mem x (l1 @ l2)}
       mem x (l1 @ l2) = (mem x l1 ‖ mem x l2))
  = introduce ∀ a x l1 l2. _
    with append_mem l1 l2 x

And in your particular proof, you can do something like this:
let test (l1 l2:list int) (x:int { mem x l1 }) =
  append_mem_auto ();
  assert (mem x (l1 @ l2))

We could add append_mem_auto to the list library and document this style of usage.


Swift 2.0 syntax changes

:+1: I will create a Swift-2.0 branch and we can merge there until it goes full release.

https://github.com/daltoniam/JSONJoy-Swift/tree/swift-2.0

Time to merge, please


Not able to refetch valid token

This is not a bug, rather a question. We have received Facebook email, stating we have broken experience on our app. They provided us with use cases which we should test and find where the problem is.
One of those use cases is this:

After user changes facebook password, access token is no longer valid. When trying to log with Facebook after password change logInWithPermissions throws error like this:

How am I supposed to fetch new token? I cannot find suitable method to do this. I found a 'hack' where you would catch the error loginWithPermissions throws and then try again calling it with login behaviour set to 'web_only', but I doubt it would work in every case. Is there official way to do this, I couldn't find it.
I guess the use case is same when token expires.
Thanks!

Update here:
The problem seems to be present only on Android when there is Facebook application installed. If I use behaviour 'web_only' all the time, then it asks user to log in again to facebook account when token is invalid - as it should. Since iOS is using browser, it's working as expected as well.

I think you might be "first" here - I haven't seen this come up yet. If you have something that works for you, it might be a help to others if you added a note about it to the troubleshooting section (like https://github.com/thebergamo/react-native-fbsdk-next/commit/2614385e127455488f154bfef5a6f5d157080085) as well as to the loginWithPermissions section https://github.com/thebergamo/react-native-fbsdk-next#requesting-additional-permissions-with-login-manager

Thanks for the answer. I can make it work using 'web_only' behaviour, but I don't like that approach.
Does this lib handle token expirations and invalid tokens or is that something that sdks do by themselves?
When I get this error, if I go to Facebook app, it says 'Session expired' and it logs me out. After that, this lib works as it should.

Again with the "your first" theme, and my apologies - but I'm not sure? Inspection of the module / testing of the scenarios and documentation of the same would be a huge help as I'm just not sure myself.


Add nanostring section to tutorial notebook

This PR adds a new section, "Analysing Nanostring data in the Scatter Widget" to the tutorial notebook on using napari-spatialdata.
Related tutorial: https://squidpy.readthedocs.io/en/stable/external_tutorials/tutorial_nanostring.html

Codecov Report
Patch coverage: 100.00% and project coverage change: +3.08 :tada:

Comparison is base (5b893f4) 73.44% compared to head (04dcb7b) 76.53%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
+ Coverage   73.44%   76.53%   +3.08%
==========================================
  Files          11       11
  Lines        1322     1338      +16
==========================================
+ Hits          971     1024      +53
+ Misses        351      314      -37

Impacted Files
Coverage Δ

src/napari_spatialdata/_view.py
80.76% <100.00%> (-0.42%)
:arrow_down:

... and 4 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[3.0 Triple]Support Socks5

Support Socks5

Codecov Report

Merging #10000 (1998a55) into 3.0 (bce439e) will decrease coverage by 0.24%.
The diff coverage is 16.66%.

@@             Coverage Diff              @@
##                3.0   #10000      +/-   ##
============================================
- Coverage     65.69%   65.44%   -0.25%
- Complexity      294      317      +23
============================================
  Files          1215     1215
  Lines         53040    53010      -30
  Branches       8034     8005      -29
============================================
- Hits          34842    34694     -148
- Misses        14386    14489     +103
- Partials       3812     3827      +15

Impacted Files
Coverage Δ

...java/org/apache/dubbo/remoting/api/Connection.java
66.40% <16.66%> (-3.27%)
:arrow_down:

...dubbo/remoting/zookeeper/ZookeeperTransporter.java
55.55% <0.00%> (-33.34%)
:arrow_down:

...ng/zookeeper/curator5/Curator5ZookeeperClient.java
43.47% <0.00%> (-28.02%)
:arrow_down:

.../remoting/api/SingleProtocolConnectionManager.java
61.53% <0.00%> (-23.08%)
:arrow_down:

...nt/metadata/ServiceInstanceHostPortCustomizer.java
65.78% <0.00%> (-21.06%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

...e/dubbo/rpc/protocol/tri/transport/WriteQueue.java
64.28% <0.00%> (-9.53%)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
62.79% <0.00%> (-9.31%)
:arrow_down:

...ava/org/apache/dubbo/rpc/proxy/InvocationUtil.java
81.81% <0.00%> (-9.10%)
:arrow_down:

... and 54 more

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 bce439e...1998a55. Read the comment docs.

Some scenarios using socks5 may also need to explain this PR's purpose :)

Some scenarios using socks5 may also need to explain this PR's purpose :)

Let me think, hahaha


How to working with ios 7

Hi! i have problem with app iso 7 . How to use plugin for ios 7 ?
. I hope you help me. Thank you for read.

iOS 7 is not supported, the plugin uses a native class that isn't available on iOS 7


Apple Watch : there is no camera

Hi,
I’m all update
Apple Watch 4
In octopod on iOS it’s ok I’ve the picture of my camera but not on Apple Watch...
There is a circle green but no picture of my camera.
Thanks

Hmm, I wonder if image size is too big and it fails to reach the Apple Watch. Could you try reducing resolution of the camera and see if things start to work? You can revert the change after this experiment. It will help collect info to understand what is going on.
Thanks,
Gaston

I have also the problem. I have reduced the resolution of the camera without success..
I have reinstalled all Apps (iOS and watchOS). I have reinstalled OctoPod extension on Octoprint...
without success.
My widget does not load things automatically. I first have to open the App...
Camera is still not working!!

While adding HLS support to Apple Watch, I found a few use cases that would not render any image on the Apple Watch and the refresh button remains disabled. I'm not sure if this is your use case or not. This would happen if Apple Watch is not able to connect to iOS device to fetch image or if there was an error while sending the image from the phone to the watch.
I fixed those 2 cases in the next release. Let me know if you want to be part of beta testing for the next release.
Thanks,
Gaston


Extension code

The extension Source code is available?

https://github.com/IBM/taxinomitis/tree/master/mlforkids-api/resources

Thank you


[3.0-SNAPSHOT] Automatic Rename not working or saved

Version: 3.0-SNAPSHOT
Build: 2018-06-06 00:21
OS: Mac OS X 10.13.4
JDK: 1.8.0_131 x86_64 Oracle Corporation
What is the actual behaviour?
Under 'Settings' > Movies
Automatic tasks for 'Automatic rename' does not appear to work
nor does the setting get saved out to reload on the next open of the application

What is the expected behaviour?
umm, well, automatic rename should rename the folder and files as well as the setting to be saved for persistant reloading
Steps to reproduce:
refer above and screenshot

should be fixed with https://github.com/tinyMediaManager/tinyMediaManager/commit/ee782fbe2b19b6fb3b51c3cb6caa29488065d7aa


Fixed Issue 180 Pass connection pool and seed_connections as two

This should fix issue 180

Thanks for the fix, could I please ask you to sign our CLA so I can merge this? http://www.elasticsearch.org/contributor-agreement/

Signed the CLA. Thanks.


Increase Footer Link Size

https://github.com/zopaUK/react-components/issues/227

:tada: This PR is included in version 2.11.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add iframe to show good-quality data

It would be nice to embed a nifti viewer in our RTD, to show examples of good quality data.
Technologies to embed niftis include:

https://brainsprite.github.io/ --> looks like this is the winner! 🎉   (and made by Pierre Bellec)
https://github.com/rii-mango/Papaya
https://github.com/FNNDSC/ami
https://brainbrowser.cbrain.mcgill.ca/

or via Python code:

NiftiWidget: https://github.com/jcohenadad/GBM8378/blob/master/lab3-irm/gbm8378-lab3-irm.ipynb

info from Pierre: to generate the html to embed, view_img  from nilearn can be used

@jcohenadad could you please point me to some good quality data so that I could use to implement the viewer ? thanks.

Here is a test, please open the file click_here.html
test_unf1.zip
The only problem that I see is the contrast that is a bit strange. Not sure if we can change the contrast, unless we open an issue on https://brainsprite.github.io/

@jcohenadad could you please point me to some good quality data so that I could use to implement the viewer ? thanks.

in the spine-generic manuscript, see: Figure GOOD_DATA

The only problem that I see is the contrast that is a bit strange. Not sure if we can change the contrast, unless we open an issue on https://brainsprite.github.io/

ah, good point. Let me ask Pierre, maybe he has a quick answer.

my bad, I fixed the contrast. Here is a good example
test_unf1.zip

awesome! how did you fix it?

bsprite = viewer_substitute(cmap='gray', symmetric_cmap=False, black_bg=True,
                         threshold=None, vmax=1500, title="anatomical scan", value=False)

I changed the vmax from 200 to 1500

where should we insert this iframe ?

where should we insert this iframe ?

I'm thinking there --> https://spine-generic.readthedocs.io/en/latest/data-acquisition.html#multi-center-data

Here is an output example for contrast from Figure GOOD_DATA
results_contrasts_good.zip
I don't know how to get rid of the brain behind some images. I checked in the documentation, but I couldn't find the option

I don't know how to get rid of the brain behind some images. I checked in the documentation, but I couldn't find the option

probably that (under bg_img)

I change bg_img=False and it did the trick. Now we have another problem. The images are squished.
See here:
results_no_background.zip

@alexfoias could you open an issue on brainsprite?

https://github.com/brainsprite/brainsprite/issues/98
I opened the issue here: https://github.com/brainsprite/brainsprite/issues/98

@jcohenadad where should I put the scripts for creating the htmls ?

@jcohenadad where should I put the scripts for creating the htmls ?

how long is that? maybe it would be more appropriate in the wiki actually...


chore: update model I/O functions and documentation

Main improvements in this PR:

Refactor:

Use RAVEN functions readYAMLmodel, writeYAMLmodel and exportForGit, instead of the Human-GEM specific importYaml, exportYaml and exportHumanGEM.
Modify increaseHumanGEMVersion, and include a check that Human-GEM.yml and model/tsv files match.
Prepare for Human-GEM.txt to show metNames[comps] instead of metIDs.
Full round of I/O with Human-GEM.yml using RAVEN 2.10.0. Changes in txt (major, see above), xml and mat (both minor) will appear with next release.
Remove duplicate getGenesFromGrRules function, already present in RAVEN.

Documentation:

Update README.md to reflect changes in model I/O functions, more clearly explain the software requirements and the ability to contribute to development without RAVEN.

I hereby confirm that I have:

[x] Tested my code on my own computer for running the model
[x] Selected develop as a target branch
[ ] Any removed reactions and metabolites have been moved to the corresponding deprecated identifier lists

@feiranl Can you review this PR? If you approve, then a lot of conflicts in other PRs can be resolved.

@edkerk, I noticed this PR includes significant changes to the YAML file. I'm concerned that merging it into the develop branch might cause conflicts and complications with other open PRs. If that is not the case, I can review this PR later today.

Almost all YML changes are the removal of empty lines:
      - rxnNotes: ""
      - eccodes: ""
      - references: ""

This should pose very  limited if any complications with any open PRs.

@edkerk Could you check this PR, so that we can merge this. There are around 20 PRs waiting to be approved and 20 PRs needs to be done...

@edkerk Thank you so much for all the hard work yesterday! Please let me know when this PR is ready for review. I can then do a quick check.😊

Ready for review!


Xenko still shows privacy agreement popup even when running <IP_ADDRESS>-dev

Release Type: Official Release/GitHub (please choose appropriate option)
Version: Version number and/or git branch
<IP_ADDRESS>-dev Master
Platform(s): Does the problem occur on Windows, Android...?
Windows 8.1
Describe the bug
Xenko still shows privacy agreement popup even when running a github build for the first time. This occurs when the Xenko Installer version has been removed.
To Reproduce
Build the <IP_ADDRESS> master branch from github
Run Xenko.GameStudio.exe
Expected behavior
An MIT project should not require acceptance of a privacy agreement.
Screenshots
Log and callstacks
Additional context

In the current code, Xenko still send some anonymous usage metrics even when run from GitHub.
Do you think that's something that should be disabled except in official builds? (at the risk of loosing lot of useful statistics for us)

In the current climate where the public is somewhat suspicious of data collection... yes.
Not that big a deal actually but perhaps it could be made opt-in to help Xenko instead of "must accept"
IMHO

@aurodev good point. We should make it opt-in and maybe explain exactly what we are collecting.


Build Instructions for DnxCore branches

Could you please provide build instructions for the DnxCore branches?  I pulled them down and was unable to F5 build or use the batch file to build the project.
We are wanting to evaluate how far that project is progressing. We have a dependency on NEST and need to be able to run on .NET Core. Any instructions would be greatly appreciated.

We were able to get this to work.

Glad you were able to get this to work @grantgoodart :+1:
FYI, the dnx branch has been merged into master and the build.bat file in the root of the repo uses DNX tooling for nearly all targets

Is it on the nuget or myget feed yet @russcam?

The latest packages on the MyGet feed have CI builds that target

.NETFramework,Version=v4.6
.NETFramework,Version=v4.5
.NETPlatform,Version=v5.1

We're working on a getting core clr support in a nuget release :smile:

Hi @russcam are there any updates to getting this into a nuget release?

@shakdoesgithub, there's a Core CLR version of NEST and Elasticsearch.NET on nuget since version 2.0.0-rc1 and it's in all 2.x releases since. It's targeting dotnet5.1


Support TTS and STT beyond selected provider

This PR makes Text to Speech and Speech to Text functionalities independent of the current LLM provider being used for chat completions.
Here's a brief overview of changes:

We now cache all the providers and their models in useModels hook so we can query any provider's model info without an API request. This was necessary since we need to determine if we support TTS very frequently, at a rate that is greater than 1 models query per TTS chunk request.
Created a new hook called useTextToSpeech and moved the logic to determine TTS support, and generating audio from text into it, all based on the providersWithModels info cached in useModels.
The textToSpeech function now uses an OpenAI client configured with one of the available providers that support TTS. The idea is to use the currentProvider if it supports TTS, else find any other providers if they do.
Speech to Text logic also follows a similar logic to create an OpenAI client which is now passed into the SpeechRecognition instance vs hard coding the client information in its trancscribe method.
I feel like isSpeechToTextSupported and getSpeechToTextClient belong in a separate hook outside useModels, but not sure if its necessary.

I don't think this is the best approach, but it worked out of the many I tried.
Here's a working demo:
https://www.loom.com/share/67024f7807254bd6a8ba89fb3e985133?sid=238e6ee8-fcb4-4b34-9b32-14464be3c7af
This closes #721.

So if I'm reading the code right, the logic is "see if current provider has an STT/TTS model, otherwise fallback". I like this. Will test.

So if I'm reading the code right, the logic is "see if current provider has an STT/TTS model, otherwise fallback". I like this.

Exactly!

Cross-provider algo does not seem to work.
I tried this with free provider, ...which has a stt model and it keeps using openai according to network requests in devtools

Cross-provider algo does not seem to work.
I tried voice to text with free provider, ...which has a stt model and it keeps using openai according to network requests in devtools

Hmm,  I guess that's because we only cache the provider info that is in localstorage and free providers are not part of that?

@tarasglek Thanks for finding the issue, just pushed a fix.
Here's a test with a custom provider selected:

My expectation is that when I have a provider selected that offers whisper, it would prefer whisper from that provider. After most recent push it always defaults to free STT, eg the behavior flipped.

My expectation is that when I have a provider selected that offers whisper, it would prefer whisper from that provider. After most recent push it always defaults to free STT, eg the behavior flipped.

@tarasglek This should be fixed by recent changes, where we now include default supported providers in models cache and select the STT model based on STT client's provider.
With Free Provider:

With OpenAI:

works


Add scripts to update ml4bio version

I added simple scripts to use pip to check whether there is a newer version of the ml4bio package available and install it if there is.  I tested the Windows and Linux scripts but not the Mac OS script.
This partially addresses #19.  Before closing #19, I'd like to consider whether we should also try updating the other packages in the conda environment.
I'll document the scripts in the setup guide before merging.

The conda update command to update an existing file did not work with older versions of conda during my testing, so I am keeping that out of this pull request.  I tried
conda update --file conda_env.yml
conda update --name ml4bio --file conda_env.yml --prune

and similar commands.  Therefore, this pull request also closes #19.
I'm going to update documentation and merge.


For issue identified in #35774 (IBMPrivateCloud/roadmap#35774)

Key changes:

Add license files to all images.

/ok-to-test

@msolis-29 did you have a chance to test building the image locally?

@dian-xh did you have a chance to test building the image locally? no

/assign @dian-xh

/retest

/retest

/lgtm


Serious errors sometimes logged at "INFO" level

When a batch task is requested and the disk is full, a log message is written, but it's at "INFO" level so is easily missed if someone's trying to use logs to trigger alerts.
The code in this specific case is: https://github.com/meilisearch/meilisearch/blob/db0cf3b2ed0ca6cf6549510bf373681e83b775dd/index-scheduler/src/lib.rs#L1266 . There may be other places in the code where this can happen.
The error that was logged for me: {"timestamp":"2024-09-07T18:06:37.387047Z","level":"INFO","fields":{"message":"Batch failed No space left on device (os error 28)"},"target":"index_scheduler"}

Hello :wave:
it indeed sounds reasonable to upgrade this to warning or even error level.
Sorry for the belated answer.

Fixed in #4955


Configure and deploy the Minter for payment Layer 1 for the October 2023 Project

Description
Configure and deploy the Minter for payment Layer 1 for the October 2023 Project

Not relevant anymore - closing


Generate a script for i18n prompts

Matching PR for adhearsion/adhearsion-i18n#8
This rake task will generate a script in Markdown format suitable for sharing with the person who will record the prompts.
Todo:

[x] Handle nested values (eg. ivr_section.main_menu)
[x] Let the audio filename attribute override the i18n key, if provided

Coverage increased (+0.01%) to 98.218% when pulling 40167101fb6f4c1f3be518bd0fd68ee2c2e50ffd on feature/i18n_recording_script into 3e6e026151b0d647984755b1c630f7a2a3713d1f on develop.

LGTM

Test failures are unrelated, merging


fix server_version memory management

these fixes repair issue #19 and issue #20
I've stress-tested everything in deep.

Hi @pabloandresm
Thanks for your PR. I've reviewed, so far so good, and am merging now. Will publish a new release soon.
Best,

Hi @pabloandresm
The new MySQL_MariaDB_Generic releases v1.6.1 has just been published. Your contribution is noted in Contributions and Thanks
Please have some more tests to verify the bugs have been squashed and if there is any more bug.
The library have been much better now, thanks to your contributions.
Best Regards,

Release v1.6.1

Fix memory management bugs. Check corrupt heap at MySQL_Connection destructor #19 and malloc server_version result not correctly handled may lead to memory corruption #20
Add support to SAMD21/SAMD51 boards using Fab_SAM_Arduino core
Add support to RP2040 boards using Seeed RP2040 core
Add Packages' Patches for Fab_SAM_Arduino core


Calling connect() without network connectivity makes the socket indefinitely unusable.

If you call connect on a socket when there is no internet/network connectivity, it will forever put the socket in an incorrect state, such that even after network connectivity is re-established, further calls to connect will never do anything. Moreover, the original call to connect (that was made when there was no network) does not eventually succeed either.

I can't reproduce this behavior with the latest code, it is able to reconnect after a failed attempt fine. Perhaps you are using an old version of Starscream?


[UC] Ignore optimizer when UC

PR types

Bug fixes
PR changes

APIs
Description

Whenignore_save_lr_and_optim,unified_checkpoint_config should be empty.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: gongel:x: gongenleigongenlei seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


port forwarding 51820

This is more of a question than a issue. In setting up wiregate it works great on LAN but once I try it from a hotspot or other network it just doesn't work. I was under the impression that port 51820 had to be UDP forwarded and that was all. I looked around for quite a bit and I still cant figure it out.
I currently have wiregate running on a fresh install of dietpi on a Pi 4 on ethernet.

Send a screenshot of the install output.

After performing a fresh installation of DietPi, I switch to local install mode and select Express, AdGuard, and Dark Wire for installation. If the Raspberry Pi shuts down, is there a way to restore everything without having to rerun the installation script?

Forward the given port mappings. Wirehole is set to randomize the portmappings if not set manually. Also you can use the environment variables file to set a consistent deployment.

I apologize for the ongoing issue, but it still doesn't seem to be working correctly. I'm using Portainer to monitor all the exposed ports:

Based on this, I've port-forwarded all of them to the Raspberry Pi:

However, there is still no response. I checked the ports on the Pi using netstat -lnu, which shows the UDP ports, but none of them appear to be listening for connections. I'm unsure of what else to try at this point and am starting to suspect this might be a bug that arises when deploying locally.

Can you provide a bit more detail about your setup. It seems the masterkey config isn't generation properly. The endpoint doesn't match the given port mappings. I'll do some checks and get back to you this weekend.

Yeah, thanks for the continued help BTW, I'm running WireGate on a RPI 4, I did a fresh install of Dietpi and now using wiregate and portrainer to manage it. Everything for WG works great locally but it just doesn't want to connect outside of LAN with all of the open ports forwarded. The pi itself is ethernet connected.

I'll recreate your setup on my local net.

@LegendarySpy Found the issue. Update your endpoint IP to your public IP in the master key. Use the install menu to reset the deployment properly. Update perma fix coming this weekend.

Yeah, that worked. Thank you so much for the help this far 👍


error webcam java.lang.reflect.InvocationTargetException

java.lang.reflect.InvocationTargetException
	at java.awt.EventQueue.invokeAndWait(Unknown Source)
	at java.awt.EventQueue.invokeAndWait(Unknown Source)
	at webcamsdd.appMain.init(appMain.java:78)
	at com.sun.deploy.uitoolkit.impl.awt.AWTAppletAdapter.init(Unknown Source)
	at sun.plugin2.applet.Plugin2Manager$AppletExecutionRunnable.run(Unknown Source)
	at java.lang.Thread.run(Unknown Source)
Caused by: java.lang.ExceptionInInitializerError
	at com.github.sarxos.webcam.Webcam.getDriver(Unknown Source)
	at com.github.sarxos.webcam.Webcam.getDiscoveryService(Unknown Source)
	at com.github.sarxos.webcam.Webcam.getWebcams(Unknown Source)
	at com.github.sarxos.webcam.Webcam.getDefault(Unknown Source)
	at com.github.sarxos.webcam.Webcam.getDefault(Unknown Source)
	at com.github.sarxos.webcam.Webcam.getDefault(Unknown Source)
	at webcamsdd.appMain$1.run(appMain.java:84)
	at java.awt.event.InvocationEvent.dispatch(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$200(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)
Caused by: java.security.AccessControlException: access denied ("java.util.PropertyPermission" "webcam.debug" "read")
	at java.security.AccessControlContext.checkPermission(Unknown Source)
	at java.security.AccessController.checkPermission(Unknown Source)
	at java.lang.SecurityManager.checkPermission(Unknown Source)
	at sun.plugin2.applet.AWTAppletSecurityManager.checkPermission(Unknown Source)
	at java.lang.SecurityManager.checkPropertyAccess(Unknown Source)
	at java.lang.System.getProperty(Unknown Source)
	at com.github.sarxos.webcam.ds.buildin.WebcamDefaultDriver.<clinit>(Unknown Source)

There is no error when running it on my desktop, but when I run it on my webpage this is all I get.
can u help me to solve this error

@abdulrosyid, have you signed your applet?

how to signed my applet ? i use editor netbeans.

You can do that from command line by using jarsigner tool (there are thousands of examples in google) or use Maven (as I did) to bind maven-jarsigner-plugin to your Maven build. The Maven POM to build my applet example (if you decide to use it) can be found here:
https://github.com/sarxos/webcam-capture/blob/master/webcam-capture-examples/webcam-capture-applet/pom.xml
If this is the first time you hear about project called Maven please read more about it because the first approach may be overwhelming.
Generally, you may copy my project and put your sources inside. Netbeans supports Maven very well, but since this is not my IDE of choice (I use Eclipse) I cannot give you details on how to integrate Maven in your Netbeans project. From the console it's enough if you have pom.xml in a directory and you run mvn clean package command there.

okay thx's sarxos, i want to ask again about applet, why must signed applet ?

@abdulrosyid, this is because you have to prove you are you and not some random hacker from the depths of Internet. Without applets signing everyone could do literally everything on a remote computer whenever victim access dedicated web page which is a host of the malicious applet. This is also the reasons why Sun/Oracle dropped self-signed applets and they do not intend to support it any more without a valid RSA signature which can be purchased from a certificate vendors. The price for such signature varies between several bucks up to the hundreds per year. There are some really cheep ones, but they may work only on some very limited set of platforms, so please consider yourself warned when you decide to purchase such certificate. Also, please note that code signing certificate is something different than regular SSL/TLS certificate that are used on a web page.
For further reading please see:
http://mrbool.com/what-is-self-signed-applet/28604
http://docs.oracle.com/javase/7/docs/technotes/guides/jweb/security/rsa_signing.html

thx's sarxos for your explain.