Add Object.freeze, Object.isFrozen and Object.validateIsFrozen.

Hi,
Here is an attempt to provide immutable values to wren by simulating a freeze API to implement some ideas from #1156.
All regular object should be covered, via the interception of ObjInstance write access.
Implementation is not perfect:

It is opt-in for foreign and primitives, and should be manually checked.
Due to implementation detail static member variables do not responds to the their class being frozen.
Fn/Closure are immutable with real up-values. Like class static member variables, accessing global up-values do not responds to their function being frozen.

Edit: Updated to handle/test functions.

Well, it certainly seems promising.
It would be an important feather in Wren's cap if we had a decent story on immutability which didn't cost too much.
I think freeze (courtesy of JS) is a good word to use - it's not too long and does what it says on the tin.
Presumably Object.validateIsFrozen is just there to test the internal API - we shouldn't need it should this idea get to the production stage.
However, there might be a case for adding an Object.unfreeze method even though it probably wouldn't be used much. I'm thinking here of scenarios where you have a global list (say) which you can freeze and then be safe in the knowledge that some complicated method or function can access it but not mutate it. You might then want to unfreeze it so you can change a few elements, refreeze it and call the method/function again.
Another point which occurs to me is that the List.sort method couldn't be applied to a frozen list but, as that's implemented entirely in Wren, we'd need to add a manual check for that.
I'm not sure how important it is to be able to freeze the static fields of a class or to freeze a function/fiber object. If there isn't any easy way to do this, perhaps we should just exempt them from the immutability API.

Object.validateIsFrozen() is there in an attempt to provide a universal error. Eg: List.sort (which I forgot, and probably others) should use it to validate early before it enters the C List.swap(_,_) function.
I can imagine there are scenarios that would require to Object.unfreeze(), but I think I want to avoid it unless there is a requirement. Security wise, I think I would want to add a key/lock mechanism (on the wren side only). Thought it would be quite heavy, since it would mean creating a key Object (which would also need to be discussed).
var res = createResource()
var key = res.freeze()
// use the frozen resource
res.unfreeze(key)
// mutate the resource
key = res.freeze()
// use the frozen resource again

To limit the number of symbol we could use Object.isFrozen=(_) instead, but that doesn't play nice with a key/lock mechanism. That said it should be rare enough so we can accept the oddity?
var res = createResource()
var key = res.isFrozen = true
// use the frozen resource
res.isFrozen = key
// mutate the resource
key = res.isFrozen = true
// use the frozen resource again

I'm not keen on Object.isFrozen=(_) as it's not very intuitive. If we need it, I'd prefer to stick with Object.validateIsFrozen() or perhaps something a little shorter such as Object.checkIsFrozen().
Looking at the C side and list_addCore in particular, I'm wondering whether it's really necessary for that function to call VALIDATE_VALUE_IS_NOT_FROZEN(args[0]) as that's only going to be called by the compiler when compiling list literals and I don't think it will be possible for the list to be frozen at that stage.
Turning now to what's currently implemented for the List class on the Wren side, would it be feasible to move part of the implementation of the addAll(other) method to the C side so VALIDATE_VALUE_IS_NOT_FROZEN(args[0]) is only called once rather than for each item added? This could only be done if other were aList which is a simple check. For other sequences, we'd have to iterate through them and add items individually as we do now.
The * operator calls that method and the + operator doesn't at present but could be changed so that it does if other were a List. Otherwise we could leave their implementations on the Wren side.

I'm currently testing another possible API that is more promising.
class Object {
  foreign freeze() // Eternal freeze
  foreign isFrozen
  foreign setFrozen(isFrozen) // Reversible freeze (`true` is the default secret)
  foreign setFrozen(isFrozen, secret) // Reversible freeze with user secret
}

If addAll is proven to be a bottleneck in practice, we may need to add a specialized addAll_ that expect natives. But in deeps, it is not directly related to this change.

FWIW, Object.freeze() is not reversible in JS though that doesn't mean we can't make it so in Wren as there are clearly scenarios where it would be useful.

Updated to the new API with secret.

Its an interesting idea, though I have two questions:

What was the rationale behind choosing a Ruby style frozen variable approach, rather than introducing a keyword like const or val that handles variable mutability at compiler level?
The approach seems to add 2 new field to the object header struct Obj, is there any performance implication of this? Or put it another way, how expensive is it to add fields to object header? Currently fields are only stored in struct ObjInstance, what if the fields array is moved to struct Obj rather than struct ObjInstance? Will it be a significant slowdown?

Introduction of const keyword requires compiler to be more intelligent
or requires additional symbols to route non const to const by default.

It has an impact, but there is room for it so at least it as no memory
impact.


Add "base_edid_changeable" to Input

Add "base_edid_changeable" to Input

I made this property required, so in case of Input without EDID support it's always false.

I wonder whether this should be explicitly documented as implying that PUT or DELETE /inputs/{inputId}/edid/base return 405 Method Not Allowed?

@garethsb, thank you, it's a good point.
I also removed

There is no Base EDID at the initial state.

because I see nothing in the spec text that prevents an Input to have a default read-only Base EDID.

I disagree with removing that sentence "There is no Base EDID at the initial state.". You are changing the specification and IS-11 will never be completed if we keep on changing the specification.
You cannot remove that sentence. If there is no base EDID it is the manufacturer EDID that is presented to the source device. "If Base EDID is not set, Effective EDID is built on the basis of a default Base EDID defined for the Input by the manufacturer."

I would not use the name base_edid_changeable but base_edid_support to better match edid_support that we already have. This is just a name so I'm ok with any name as along as it does not imply changing the actual behavior implied by the specification.

@alabou, hm, let me think about it. In case of an Input that doesn't allow changing its Base EDID, it still uses some Base EDID under the hood. This Base EDID can be fetched via /edid/effective when Active Constraints are empty.
Does it make sense to return this Base EDID from /edid/base of such Input? Maybe, no. We can keep the removed sentence, rename the property to base_edid_support and prescribe GET /edid/base to return 204 if we consider Base EDID as a resource controlled by the user and not changed internally.
But in this case, I would replace "a default Base EDID" in "If Base EDID is not set, Effective EDID is built on the basis of a default Base EDID defined for the Input by the manufacturer." with something to distinct the manufacturer EDID and Base EDID.

I would be ok with a modification to "If Base EDID is not set, Effective EDID is built on the basis of a default Base EDID defined for the Input by the manufacturer." for "If Base EDID is not set, Effective EDID is built on the basis of a default EDID defined for the Input by the manufacturer."
"GET /edid/base to return 204" it is already the case in the RAML if there is no Base EDID set.

Added a commit removing the citing of RAML but preserving the hint itself.


Added GL_SRGB8 texture format

This doesn't require that a runtime implement that, though, right? it just handles it in case they do?

This doesn't require that a runtime implement that, though, right? it just handles it in case they do?

This doesn't require that a runtime implement that, though, right? it just handles it in case they do?

Yes, exactly.

This doesn't require that a runtime implement that, though, right? it just handles it in case they do?

Yes, exactly.

I'll merge as soon as I hear from @Wallbraker that the macro usage is right.

I'll merge as soon as I hear from @Wallbraker that the macro usage is right.

I'll merge as soon as I hear from @Wallbraker that the macro usage is right.

@rpavlik I didn't write that code, or completely swapped that out of my head. But it sounded like it was right from Ed.

I'll merge as soon as I hear from @Wallbraker that the macro usage is right.

@rpavlik I didn't write that code, or completely swapped that out of my head. But it sounded like it was right from Ed.


Feature: Add utility function to read server-side resolver from another resolver

The load resolver can be called from within another resolver by accessing it through the route config.
export const routeMeta: RouteMeta = {
  resolve: {
    data: async(route) => {
      // call server load resolver for this route from another resolver
      const data = await route.routeConfig?.resolve?.['load']?.(route);

      return { ...data };
    }
  }
}

We can add a utility function for this named getLoadResolver and use it as getLoadResolver(route).
import { getLoadResolver } from '@analogjs/router';

export const routeMeta: RouteMeta = {
  resolve: {
    data: async(route) => {
      // call server load resolver for this route from another resolver
      const data = await getLoadResolver(route);

      return { ...data };
    }
  }
}

Originally posted by @brandonroberts in https://github.com/analogjs/analog/issues/1039#issuecomment-2069965764

Hello @brandonroberts, can you assigned me to this task please ?

@Tenessy yep, thanks!

Closed by https://github.com/analogjs/analog/pull/1144  and released in 1.5.0


ThresholdedReLU crashes when the input is a list

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.8.0
Python version: 3.7.12
Bazel version (if compiling from source): N/A
GPU model and memory: N/A
Exact command to reproduce: https://colab.research.google.com/drive/144FOk8RO-Ew_eBtGZlCmsUL6k0cEAlUO?usp=sharing

Describe the problem.
keras.layers.ThresholdedReLU fails to accept a list input by reporting the following error:
[/usr/local/lib/python3.7/dist-packages/keras/layers/advanced_activations.py](https://localhost:8080/#) in call(self, inputs)
    262
    263   def call(self, inputs):
--> 264     theta = tf.cast(self.theta, inputs.dtype)
    265     return inputs * tf.cast(tf.greater(inputs, theta), inputs.dtype)
    266

AttributeError: Exception encountered when calling layer "thresholded_re_lu_1" (type ThresholdedReLU).

'list' object has no attribute 'dtype'

Call arguments received:
  • inputs=['tf.Tensor(shape=(None, 1, 10), dtype=float32)', 'tf.Tensor(shape=(None, 1, 10), dtype=float32)', 'tf.Tensor(shape=(None, 1, 10), dtype=float32)']

In contrast, keras.layers.ReLU and keras.layers.LeakyReLU can accept the list input.
Describe the current behavior.
keras.layers.ThresholdedReLU crashes when the input is a list
Describe the expected behavior.
ThresholdedReLU can accept the list input.
Contributing.

Do you want to contribute a PR? (yes/no):
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing):

After comparing the code between ThresholedReLU and ReLU, I think the reason is that ReLU directly use the backend implementation: keras/layers/activation/relu.py#L96 while ThresholdedReLU implements by itself: keras/layers/activation/thresholded_relu.py#L63. Not sure why does such an implementation inconsistency exist, but I think we can do something similar in the thresholded_relu.py#L61-63 like backend.relu does:
def call(self, inputs):
    dtype = getattr(inputs, 'dtype', floatx())
    theta = tf.cast(self.theta, dtype)
    return inputs * tf.cast(tf.greater(inputs, theta), dtype)

Of course, we can also directly use the backend.relu for the implementation of ThresholdedReLU like ReLU and LeakyReLU do.
Standalone code to reproduce the issue.
You can access this link or run the following code:
import keras
x = keras.layers.Input(shape=(1,10))
y = keras.layers.ThresholdedReLU()([x,x,x])
model = keras.models.Model(x,y)
model.summary()

@maybeLee,
keras.layers.ThresholdedReLU accepts input_shape as tuple of integers.
import keras
x = keras.layers.Input(shape=(1,10))
y = keras.layers.ThresholdedReLU()((x))
model = keras.models.Model(x,y)
model.summary()

Output
Model: "model_1"
_________________________________________________________________
 Layer (type)                Output Shape              Param #
=================================================================
 input_10 (InputLayer)       [(None, 1, 10)]           0

 thresholded_re_lu_11 (Thres  (None, 1, 10)            0
 holdedReLU)

=================================================================
Total params: 0
Trainable params: 0
Non-trainable params: 0

@gadagashwini,
Thanks for your reply! I think the problem in this issue is that: ThresholdedReLU fails to accept a list of tensor as input while similar layers such as ReLU and LeakyReLU can accept a list of tensor as input:
Sending a list of tensor to ReLU:
import keras
x = keras.layers.Input(shape=(1,10))
y = keras.layers.ReLU()([x,x,x])
model = keras.models.Model(x,y)
model.summary()

__________________________________________________________________________________________________
 Layer (type)                   Output Shape         Param #     Connected to
==================================================================================================
 input_17 (InputLayer)          [(None, 1, 10)]      0           []

 re_lu_7 (ReLU)                 (3, None, 1, 10)     0           ['input_17[0][0]',
                                                                  'input_17[0][0]',
                                                                  'input_17[0][0]']

==================================================================================================
Total params: 0
Trainable params: 0
Non-trainable params: 0
__________________________________________________________________________________________________

Sending a list of tensor to ThresholdedReLU:
import keras
x = keras.layers.Input(shape=(1,10))
y = keras.layers.ThresholdedReLU()([x,x,x])
model = keras.models.Model(x,y)
model.summary()

[/usr/local/lib/python3.7/dist-packages/keras/layers/advanced_activations.py](https://localhost:8080/#) in call(self, inputs)
    262
    263   def call(self, inputs):
--> 264     theta = tf.cast(self.theta, inputs.dtype)
    265     return inputs * tf.cast(tf.greater(inputs, theta), inputs.dtype)
    266

AttributeError: Exception encountered when calling layer "thresholded_re_lu_7" (type ThresholdedReLU).

'list' object has no attribute 'dtype'

Call arguments received:
  • inputs=['tf.Tensor(shape=(None, 1, 10), dtype=float32)', 'tf.Tensor(shape=(None, 1, 10), dtype=float32)', 'tf.Tensor(shape=(None, 1, 10), dtype=float32)']

Hi, I temporarily close this issue since a pr thread has already been created


Add profiling pprof flag

Description
Adds a --prof flag, which enables the pprof server for continuous profiling. When the --prof flag is provided, the application starts a pprof server that listens on port 9080.
Changes:

Added the --prof flag
Run profiler server pprof

Types of changes

New feature (non-breaking change which adds functionality)

Testing
Requires testing: Yes
Did you write tests??: No
Documentation
If this requires a documentation update, did you add one? No

@wojciechos do we need both --pprof and --metrics flags? if not then we should replace metrics with pprof.

@wojciechos do we need both --pprof and --metrics flags? if not then we should replace metrics with pprof.

I would like to keep both the --pprof and --metrics flags, as they serve different purposes. The profiler is mainly intended for our internal/developer usage, so maybe we should even hide the --pprof flag from the public-facing cli. What do you think?


Is Fedex SmartPost supported?

Our fulfillment center ships some items via FedEx SmartPost. I don't see that option available. Is it named something else or is it not supported?
irb(main):010:0> Spree::Calculator::Shipping::Fedex::
Display all 221 possibilities? (y or n)

Spree::Calculator::Shipping::Fedex::Base
Spree::Calculator::Shipping::Fedex::ExpressSaver
Spree::Calculator::Shipping::Fedex::FirstOvernight
Spree::Calculator::Shipping::Fedex::Ground
Spree::Calculator::Shipping::Fedex::GroundHomeDelivery
Spree::Calculator::Shipping::Fedex::InternationalEconomy
Spree::Calculator::Shipping::Fedex::InternationalEconomyFreight
Spree::Calculator::Shipping::Fedex::InternationalFirst
Spree::Calculator::Shipping::Fedex::InternationalGround
Spree::Calculator::Shipping::Fedex::InternationalPriority
Spree::Calculator::Shipping::Fedex::InternationalPriorityFreight
Spree::Calculator::Shipping::Fedex::InternationalPrioritySaturdayDelivery
Spree::Calculator::Shipping::Fedex::OneDayFreight
Spree::Calculator::Shipping::Fedex::OneDayFreightSaturdayDelivery
Spree::Calculator::Shipping::Fedex::PriorityOvernight
Spree::Calculator::Shipping::Fedex::PriorityOvernightSaturdayDelivery
Spree::Calculator::Shipping::Fedex::Saver
Spree::Calculator::Shipping::Fedex::StandardOvernight
Spree::Calculator::Shipping::Fedex::ThreeDayFreight
Spree::Calculator::Shipping::Fedex::ThreeDayFreightSaturdayDelivery
Spree::Calculator::Shipping::Fedex::TwoDay
Spree::Calculator::Shipping::Fedex::TwoDayFreight
Spree::Calculator::Shipping::Fedex::TwoDayFreightSaturdayDelivery
Spree::Calculator::Shipping::Fedex::TwoDaySaturdayDelivery

If it is not supported, what would it take to get it supported?

@MisinformedDNA I think you have the wrong repo.


Add Semantic Similarity with KerasNLP tutorial

Issue : https://github.com/keras-team/keras-nlp/issues/726
Kaggle Tutorial Link
This PR continues the previous PR #1253
This PR ports Keras-io's existing Semantic Similarity with BERT to KerasNLP. Now the tutorial is pure Keras + tensorflow. It doesn't depend on Tensorflow at all.

@mattdangerw & @fchollet Please review.
I have addressed all the comments from the previous one. The example text turned out a lot better with the help of ChatGPT.

Did you catch this comment from the last PR? #1253 (comment) We should try to have the triangular rate play out over the entirely of the training run, so I may have messed up my initial suggestion there.

I totally missed it

Will update .

@fchollet The PR is ready to merge.
Could you please review?

Hey @fchollet, when I run the autogen script, at later stage it starts to train the model, and my computer keeps crashing there :(

Hey @fchollet, when I run the autogen script, at later stage it starts to train the model, and my computer keeps crashing there :(

Are you able to produce a rendered notebook by some other means? In that case you could manually edit the md and ipynb files using your rendered notebook (see what the other files look like for some examples...)

@shivance I have added the autogenerated files, but there was one fairly large bug here--the training, validation and test sets were are loading the same data!
I have fixed that, but the hyperparameters are now somewhat off. None of the improvements to the learning rate or optimizer are actually boosting performance. Roberta still boosts things. This is probably something we should look at.

OK! Tweaked learning rates and update the commentary, this should be ready to go!

Thank you for the contribution! LGTM


Sign for Antony Bailey and Cloudponents

Signed as Director of Cloudponents Ltd.

Added in https://github.com/opentffoundation/manifesto/pull/903


Add elasticsearch_jvm_uptime gauge

I discovered that the the JVM uptime (readily available in the http://localhost:9200/_nodes/stats output) wasn't included in the output returned by the exporter. This PR adds it.
Tested locally with an ES 7.10.2 instance.

Ping @sysadmind, would you min reviewing this? (unsure about what the process is in this repo, it seems to have a number of open PRs at the moment)

Thanks for the merge @sysadmind. :pray: I intended to ping you that I had fixed & rebased this, but you seem to have noticed it anyway. :slightly_smiling_face: :+1:


uni-app client + client APIs

Description
Due to some specific reasons (such as constraints of the execution environment), I need to use my own encapsulated request library instead of public libraries like fetch or axios. Does hey-api allow self-encapsulation and replacement? How should I proceed with this attempt?

Hey @normal-coder, can you describe what your constraints are? Are you able to download any dependencies at all? If yes, you can bundle those clients in your output using client.bundle = true in the next release. If not, that complicates the situation somewhat. Would you be able to provide an example code you image you'd need to write to make this work?

Hey @normal-coder, can you describe what your constraints are? Are you able to download any dependencies at all? If yes, you can bundle those clients in your output using client.bundle = true in the next release. If not, that complicates the situation somewhat. Would you be able to provide an example code you image you'd need to write to make this work?

I am currently using uni-app to Cross end development with a technology stack of TypeScript+Vue+uni-app. When the code is compiled into a cross end application (such as an iOS/Android APP), the WebView or base runtime of the APP does not provide DOM operations, nor does it support fetch requests. uni-app provides an encapsulated request method: uni. request()  (document address: https://en.uniapp.dcloud.io/api/request/request.html ), which used for compatibility with requests from different ends.
If I am unable to use the encapsulated request library on my own. It means I need to modify the code from codegen project or find other solutions to integrate OpenAPI. This is not conducive to the maintenance of the project.
Therefore, in conclusion:

Perhaps hey-api can consider supporting developers to self-encapsulate the request library, so that the encapsulation of the request library can satisfy all or part of the implementation of hey-api through inheritance or implementation?
Perhaps hey-api can use the request library of uni-app as a type of client? The developer base for this cross end development solution is estimated to be in the millions. I understand that it has great value in helping hey-api achieve better development

@normal-coder can you show me how to download uni-app? Is it on npm? I'd like to understand how much usage it has.

Is the request interface a part of uni-app or is it downloaded separately?
Have there been any breaking changes to the uni-app request interface in the past 12 months?

Both 1 and 2 are possible solutions, not sure yet which one would be better long-term. If I provided 1, how much effort would it be for you to implement the request?

@normal-coder can you show me how to download uni-app? Is it on npm? I'd like to understand how much usage it has.

Is the request interface a part of uni-app or is it downloaded separately?
Have there been any breaking changes to the uni-app request interface in the past 12 months?

Both 1 and 2 are possible solutions, not sure yet which one would be better long-term. If I provided 1, how much effort would it be for you to implement the request?

Currently, uniapp supports initializing an empty scaffold using the CLI method. They invented uni_modules, a modular way similar to node_modules to distribute their own module packages, so the data of NPM is not their full amount. For NPM downloads, you can try to pay attention to the data of these packages: https://npmtrends.com/@dcloudio/uni-app-vs-@dcloudio/uni-app-plus-vs-@dcloudio/uni-h5-vs-@dcloudio/uni-mp-qq-vs-@dcloudio/uni-mp-weixin

The part about 'uni_modules' is their own defined module extension, aimed at providing free or paid extension services through the distribution of module packages. This section can refer to: https://ext.dcloud.net.cn/ Their plugin market.

The uni.request() method hasn't had a major change (e.g. BreakChange) in at least 4 years that I know of, and it's also a wrapper that's been made available through the uni-app runtime after it's been wrapped for different platforms. See https://github.com/dcloudio/uni-app/blob/next/packages/uni-api/src/protocols/network/request.ts

To quickly create a UNIAPP project using the CLI method, you can refer to this article: https://en.uniapp.dcloud.io/quickstart-cli.html

Due to the need to compile their APP Runtime using their self-developed IDE HBuilder (you can understand it as similar to compiling iOS applications requiring XCode), you can refer to this article https://en.uniapp.dcloud.io/quickstart-hx.html , import the above CLI into HBuilderX, and try running it on the simulator (if you are using macOS, you can download XCode and use iOS Simulator, if you are using Windows, perhaps you need an Android emulator like Genymotion. Note: Android debugging does not require an AOSP development environment).

If we provide solution 1, due to the implementation mechanism of HeyAPI, I am not quite sure if you will define the methods I need to redefine in your configuration file, or if you will provide a unified encapsulation layer. I will try to adapt to this encapsulation layer (similar to adapting axios or other HTTP request libraries). I understand that this may just be nesting dolls. If there is a specific example to help HeyAPI users understand or try, perhaps it won't be too complicated.
If we try solution 2, perhaps HeyAPI will be favored by many axios enthusiasts, just like adapting axios. As a network request library that supports OpenAPI, it will be favored by a large number of uni-app developers. As far as I know, there is currently no good choice for a network request library that supports OpenAPI and is adapted based on uni.request. In most cases, developers can only choose to encapsulate it themselves and complete the API interfaces one by one in the logic.

I think both options are cool. 😎

Thank you for the context @normal-coder. What are your alternatives at the moment? Are you post-processing the generated code? This feature is not a priority right now so it might take a while to implement, but want to make sure you're not blocked

Thank you for the context @normal-coder. What are your alternatives at the moment? Are you post-processing the generated code? This feature is not a priority right now so it might take a while to implement, but want to make sure you're not blocked

Thanks for your response. Currently, I have not made any modifications to the generated code. I maintain a package @gitee/typescript-sdk-v5 at https://gitee.com/sdk/typescript-sdk-v5. Three months ago, I migrated it from openapi-typescript-codegen to hey-api, but it did not work well with uni-app.
In the future, maybe I will try to find more code-generator alternatives compatible with uni-app for OAS, or abandon this approach and choose to encapsulate OpenAPI calls on my own. (BTW, openapi-typescript-codegen and hey-api are the most elegant TypeScript code-generators for OAS that I have seen, and it is highly unlikely that I will find better ones 🤣). I will continue to monitor the support for this feature. If it is implemented, perhaps I can also promote it to the uni-app team and the developer of uni-app at github.com/hey-api/openapi-ts 😉

@normal-coder can you provide more details around what works differently now that you migrated from openapi-typescript-codegen? The goal is to allow a seamless migration, so if it worked before, I am hoping it's possible it would work with this package too.
Regarding the uni-app, can you find out if they'd be willing to have a section on codegen in their documentation like this one in FastAPI https://fastapi.tiangolo.com/advanced/generate-clients/? That would be one way to make sure this is shipped waaay faster. Thank you for the kind words btw!

@normal-coder can you provide more details around what works differently now that you migrated from openapi-typescript-codegen? The goal is to allow a seamless migration, so if it worked before, I am hoping it's possible it would work with this package too.
Regarding the uni-app, can you find out if they'd be willing to have a section on codegen in their documentation like this one in FastAPI https://fastapi.tiangolo.com/advanced/generate-clients/? That would be one way to make sure this is shipped waaay faster. Thank you for the kind words btw!

Originally, I used the CodeGen project for my H5 environment, but as I had the demand to develop an app, the existing implementation encountered obstacles. Initially, I could generate an SDK based on the CodeGen solution. I could directly call methods in the code without the need to encapsulate requests separately for each API, which was very convenient. However, if I were to abandon this approach, I would need to integrate each API separately (indeed, in terms of coding, this is a repetitive and tedious process of refinement).
I found two projects on Github that make axios-adapter for uni-app. Perhaps based on this, hey-api could more easily complete the integration of the uni-app client. If you are interested, we can discuss it together.

https://github.com/uni-helper/axios-adapter
https://github.com/zxfd/uniapp-axios-adapter

In addition, I asked them if they could add an article to their community at https://ask.dcloud.net.cn/ to provide relevant information.

如果是使用uniapp, 不支持 uniapp.request 方法，requestLibPath参数可以满足你的需求，支持自定义request函数，request函数里面实现uniapp.request封装，我们的项目就是这么干的 https://github.com/openapi-ui/openapi-ts-request?tab=readme-ov-file#参数

如果是使用uniapp, 不支持 uniapp.request 方法，requestLibPath参数可以满足你的需求，支持自定义request函数，request函数里面实现uniapp.request封装，我们的项目就是这么干的 https://github.com/openapi-ui/openapi-ts-request?tab=readme-ov-file#参数

有意思的建议，我会尝试一下

如果是使用uniapp, 不支持 uniapp.request 方法，requestLibPath参数可以满足你的需求，支持自定义request函数，request函数里面实现uniapp.request封装，我们的项目就是这么干的 https://github.com/openapi-ui/openapi-ts-request?tab=readme-ov-file#参数

有意思的建议，我会尝试一下

没问题，嘿嘿，🤔


Adding the beginnings of a binary16 (half-precision) floating-point implementation

This is work towards https://github.com/dotnet/corefxlab/issues/2635 and starts with a basic implementation for Half.
The type exposes the same interfaces (modulo IConvertible and ISpanFormattable) as System.Single and System.Double. For ToString or Parse, the implementations currently throw NotImplementedException, since the actual implementation would be a small variant on the existing NumberToString and NumberToDouble/Single implementations.
The various operators (such as the casting operators as well as +, -, *, /, and %) still need to be implemented.

I plan on it, will likely do so this weekend.

It only took three months, but I have added some tests mirroring the existing ones we have for Single/Double.
This should be good for an initial merge; and then work can continue progressing as time is available.

@ahsonkhan, @danmosemsft. Is this good to merge?


Clean up bool conversions and calls to fromOption and functions duplicating BuckleScript std lib

Removed bool-Js.boolean conversion artifacts (calls to UtilsRN.optBoolToOptJsBoolean which became redundant).
Removed a lot of calls to Js.Undefined.fromOption
Removed references to functions in utilsRN.re duplicating BuckleScript standard library.
This PR includes changes from two previous PRs.

Awesome stuff! I'll run it on my app and report back today.
On Wed, 5 Sep 2018 at 09:51, Wojtek Czekalski<EMAIL_ADDRESS>wrote:

@wokalski approved this pull request.
This looks fantastic to me. Could someone else give it another sweep?
It'd be cool to run this branch against some production app to verify. cc
@arnarthor https://github.com/arnarthor @grabbou
https://github.com/grabbou @knowbody https://github.com/knowbody
@Trancever https://github.com/Trancever
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reasonml-community/bs-react-native/pull/235#pullrequestreview-152372360,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADoDp-RlVthMbPTFktQhH5EedVE3Es6Pks5uX4J-gaJpZM4WWJtf
.

Compiles w/o any errors on my branch. Boolean conversion support announced here and support for Js.Undefined from Option was in the documentation already.


Request: add cardiologie-tonkin.com to the filter

Evidence: https://stackoverflow.com/questions/295579/fastest-way-to-determine-if-an-integers-square-root-is-an-integer
Original: http://cardiologie-tonkin.com/questions/295579/fastest-way-to-determine-if-an-integers-square-root-is-an-integer

As the website seems to be down, I won't add it. I'm adding a tag on this issue, to be able to find this domain if it is back online.


Feature request: add "latest" to "to version"

combined with #25 it would allow me to have a fixed URL that would work as an upgrade guide, no matter from which version an existing user is coming, it would always look like this: https://octoclairvoyant.vercel.app/probot/probot/v9.0.0...latest, or maybe https://octoclairvoyant.vercel.app/probot/probot/v9.0.0 for short?

That's interesting, never thought about picking "latest" no matter what is that one. For being able to do that I need to implement first the other feature I'm planning: paginate versions. This is necessary in case there are more than one version pages between that v.9.0.0 and latest one. This one will take little bit longer then, but definitely on my list!

What would you consider a “page”?

Sorry, I didn't explain myself really well. With page I meant paginating the versions. At the moment I'm requesting only page 1 with first 100 versions so if the "from" one is previous than that, I wouldn't be able to retrieve it properly with the current mechanism.

@octokit/rest comes with https://github.com/octokit/plugin-paginate-rest.js, so yo can just do
const releases = await octokit.paginate(octokit.repos.listReleases, { owner, repo, per_page: 100 })

see https://octokit.github.io/rest.js/v18#pagination

Oh wow, that's brilliant! I had the pagination bit from octokit/rest pending, but this is really useful. I'm gonna give it a try, this will make the implementation I needed really simple 🚀

Octokit pagination works like a champ, I got it included in the app in 5 minutes! I'm working on shareable links for #25 and then I'll come back to this one for adding the "latest" repository version.

Implemented on #28, you have there a link to give it a try!

This is how the example you mentioned would look: https://octoclairvoyant-git-issue-26.belco.vercel.app/?repo=probot%2Fprobot&from=v9.0.0&to=latest

awesome, this is very cool! Thanks for implementing it Mario!

@gr2m This was the perfect excuse to go back to this side project! Thanks for your ideas and help with pagination. Deploying on production now 🚀


Fix error during running on nvptx with cuda9

This PR fix a error caused by changing libdevice file name in CUDA9. I locally confirmed to pass unit tests on CUDA9.
reference:
https://devtalk.nvidia.com/default/topic/1025985/fedora-25-cuda-9-missing-libdevice-files/?offset=1

@tqchen thank you for your review. I addressed it and locally confirmed to pass tests in cuda9 again because of the mistake included in earlier my commit.

Thanks for the contribution! this is now merged


[v0.2.x] GHA CI

Backport moving CI to GHA

r? @therealprof
(rust-highfive has picked a reviewer for you, use r? to override)

:warning: Warning :warning:

Pull requests are usually filed against the master branch for this repo, but this one is against v0.2.x. Please double check that you specified the right target!


[station-score-if-badge] Add Station GroupOS voting strategy for members with specific badge

This PR adds the station-score-if-badge strategy. The strategy returns a score for members of a Station GroupOS group that have a specific badge.
This strategy will be used by Mangrove DAO's multistakeholder governance.

Hi @espendk One PR should contain changes of one strategy

Ok, done.
As mentioned on #1318 I'm not sure how to submit multiple PR's with different, interdependent strategies. But this one should be good now.


Re-create analysis for S Lawndale

Pull data for S Lawndale for summer and fall (PurpleAir #113494, #113474, #5774)
Compare to existing AirNow data

Issues faced:

ValueError: Expected object or value on 1.1-purpleair-data-extraction while trying to read in using sensor_df = pd.read_json("data/pa_sensors.json")


EDA

With the following plots:

Hits to events histogram
Distribution of track momentum
Heatmap for the track position


fix: Allow PyVista HTML jupyter backend

Codecov Report
Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 60.53%. Comparing base (f12e515) to head (af23383).

Files
Patch %
Lines

...on_interface/backends/pyvista/pyvista_interface.py
66.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
+ Coverage   60.46%   60.53%   +0.07%
==========================================
  Files          24       24
  Lines         817      821       +4
==========================================
+ Hits          494      497       +3
- Misses        323      324       +1

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


"Failed encoding.Errors: no output file created" when output is a rtmp url

Sample command:
encoding_options = {custom: %w(-c copy -f flv)}
transcoding_options = {input_options: ['-re']}

FFMPEG::Movie.new(path_to_video)
    .transcode(stream_url, encoding_options, transcoding_options)

That gets translated to
ffmpeg -re -i video.mp4 -c copy -f flv 'rtmp://localhost/live'
The above raises error on https://github.com/streamio/streamio-ffmpeg/blob/master/lib/ffmpeg/transcoder.rb#L120 because of https://github.com/streamio/streamio-ffmpeg/blob/master/lib/ffmpeg/transcoder.rb#L111 (File.exists?('rtmp://....') = false)
Any ideea on how it is best to go about this issue ? Probably a simple validate unless rtmp_url? check is sufficient for my case, but the File.exists? is only valid for local output paths (which i don't know if is enough for ffmpeg since output can be other types of urls).

@pirvudoru I am sorry I missed this one. If it's not too late, I might suggest an option similar to validate. Validate disables existence validation on the output file. A new option named, say, validate_input_stream, when false (default), might skip the validation. This approach might be more generically applicable for other use cases.


[Docs] Docker Standalone Cluster Instructions on CLI vs UI

Bug Report
Other options on the cluster create for standalone clusters instruct users to complete via the UI. Docker does not.
Expected Behavior
Uniformity of options on the cluster create that addresses all options for install via the UI.

This issue was related to our deprecated and removed standalone cluster feature. Things have been greatly simplified now with our move to unmanaged clusters. I don't think this requires any action now, so I am going to close it out. Please feel free to reopen or file a new issue if there is still something missing.


Error building flutter web --no-sound-null-safety

Steps to Reproduce
Sorry  to bother. I know realize that it was a faulty HD, with corrupted files.

Run flutter create bug.
Update the files as follows: ...
...
I'm building my web app, with flutter --non-sound-null-safety param, and i'm getting the errors below

Expected results:
The app being build, or an significant error message
Actual results:
The error below:
call flutter build web --no-sound-null-safety
Running "flutter pub get" in theheroesbook...                       7.9s
Target dart2js failed: Exception:
../../../SDKs/flutter/bin/cache/flutter_web_sdk/lib/_engine/engine/bitmap_canvas.dart:615:7:
Warning: Operand of null-aware operation '!' has type 'CanvasRenderingContext2D' which excludes null.

'CanvasRenderingContext2D' is from 'dart:html'.
ctx!.fillText(line.displayText!, x, y);
^
../../../SDKs/flutter/bin/cache/flutter_web_sdk/lib/_engine/engine/bitmap_canvas.dart:630:9:
Warning: Operand of null-aware operation '!' has type 'CanvasRenderingContext2D' which excludes null.
'CanvasRenderingContext2D' is from 'dart:html'.
ctx!.fillText(char, x, y);
^
../../../SDKs/flutter/packages/flutter/lib/src/widgets/basic.dart:5750:14:
Error: The method 'debugGetOpenHandleStackTraces' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
image?.debugGetOpenHandleStackTraces()?.isNotEmpty ?? true,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/widgets/basic.dart:5755:21:
Error: The method 'clone' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
image: image?.clone(),
^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/widgets/basic.dart:5777:14:
Error: The method 'debugGetOpenHandleStackTraces' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
image?.debugGetOpenHandleStackTraces()?.isNotEmpty ?? true,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/widgets/basic.dart:5782:24:
Error: The method 'clone' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
..image = image?.clone()
^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/rendering/image.dart:93:50:
Error: The method 'isCloneOf' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
if (value != null && _image != null && value.isCloneOf(_image!)) {
^^^^^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/painting/decoration_image.dart:440:11:
Error: The method 'debugGetOpenHandleStackTraces' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
image.debugGetOpenHandleStackTraces()?.isNotEmpty ?? true,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/painting/image_stream.dart:51:20:
Error: The method 'clone' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
image: image.clone(),
^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/painting/image_stream.dart:89:24:
Error: The method 'isCloneOf' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
return other.image.isCloneOf(image)
^^^^^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/painting/image_stream.dart:120:19:
Error: The method 'debugGetOpenHandleStackTraces' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
assert((image.debugGetOpenHandleStackTraces()?.length ?? 1) > 0);
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/painting/image_stream.dart:877:34:
Error: The method 'clone' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
image: _nextFrame!.image.clone(),
^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/painting/image_stream.dart:932:34:
Error: The method 'clone' isn't defined for the class 'Image'.
'Image' is from 'dart:ui'.
image: _nextFrame!.image.clone(),
^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/semantics/semantics.dart:3654:47:
Error: Getter not found: 'isSlider'.
bool get isSlider => _hasFlag(SemanticsFlag.isSlider);
^^^^^^^^
../../../SDKs/flutter/packages/flutter/lib/src/semantics/semantics.dart:3656:28:
Error: Getter not found: 'isSlider'.
_setFlag(SemanticsFlag.isSlider, value);
^^^^^^^^
Error: Compilation failed.

Compiling lib\main.dart for the Web...                             80.5s
Exception: Failed to compile application for the Web.

Logs

[√] Flutter (Channel beta, 1.24.0-10.2.pre, on Microsoft Windows [Version 10.0.19041.630], locale
    en-US)
    • Flutter version 1.24.0-10.2.pre at d:\Users\Diego\Programming\SDKs\flutter
    • Framework revision 022b333a08 (12 days ago), 2020-11-18 11:35:09 -0800
    • Engine revision 07c1eed46b
    • Dart version 2.12.0 (build 2.12.0-29.10.beta)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at D:\Users\Diego\Programming\SDKs\android-sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = D:\Users\Diego\Programming\SDKs\android-sdk
    • ANDROID_SDK_ROOT = D:\Users\Diego\Programming\SDKs\android-sdk
    • Java binary at: C:\Program Files\Java\jdk-11.0.9\bin\java
    • Java version Java(TM) SE Runtime Environment 18.9 (build 11.0.9+7-LTS)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[!] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    X Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[√] IntelliJ IDEA Community Edition (version 2020.2)
    • IntelliJ at C:\Users\Diego\AppData\Local\JetBrains\Toolbox\apps\IDEA-C\ch-0\202.8194.7
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart

[√] VS Code (version 1.51.0)
    • VS Code at C:\Users\Diego\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.16.0

[√] Connected device (4 available)
    • MotoG3 (mobile)  •<PHONE_NUMBER> • android-arm    • Android 9 (API 28)
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 87.0.4280.66
    • Edge (web)       • edge       • web-javascript • Microsoft Edge 86.0.622.61

! Doctor found issues in 1 category.

You have not included any steps on how to reproduce the problem

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Require minimum versions on CMakeLists template.

The code generated is now designed for Fast CDR >= 2.0 and Fast DDS >= 2.12, so the generated CMakeLists.txt should require those versions.

Please, increase cmake_minimum_required to 3.22 as fastcdr and fastdds.
cmake_minimum_required(VERSION 3.22)

404ab20


Feature/: Integrate with amazon cognito

Codecov Report

Merging #16 (e792d64) into develop (54d2df8) will decrease coverage by 12.73%.
The diff coverage is 0.00%.

@@              Coverage Diff               @@
##             develop      #16       +/-   ##
==============================================
- Coverage      98.34%   85.61%   -12.74%
  Complexity        79       79
==============================================
  Files             13       17        +4
  Lines            121      139       +18
  Branches           8        8
==============================================
  Hits             119      119
- Misses             0       18       +18
  Partials           2        2

Impacted Files
Coverage Δ

...va/org/wcdevs/blog/core/rest/config/AwsConfig.java
0.00% <0.00%> (ø)

...g/wcdevs/blog/core/rest/config/CoreProperties.java
0.00% <0.00%> (ø)

...g/wcdevs/blog/core/rest/config/SecurityConfig.java
0.00% <0.00%> (ø)

...va/org/wcdevs/blog/core/rest/config/WebConfig.java
0.00% <0.00%> (ø)

...a/org/wcdevs/blog/core/rest/util/ErrorMessage.java
100.00% <ø> (ø)

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 54d2df8...e792d64. Read the comment docs.


⚠ Geosearch has degraded performance

In 279a468, Geosearch ($SITE_URL_GEOSEARCH) experienced degraded performance:

HTTP code: 200
Response time: 4105 ms

Resolved: Geosearch performance has improved in c0f5646 after 6 minutes.


update kfp-tekton handson

for kubeflow v1.5, it uses kfp-tekton v1.1.1.
and its tekton version is 0.30.0. update docummentation
to align with these versions.
Signed-off-by: Yihong Wang<EMAIL_ADDRESS>

thanks @yhwang
/lgtm


Switch to bionic on Travis

Due to LetsEncrypt certificate validation errors with the xenial image.
https://techcrunch.com/2021/09/21/lets-encrypt-root-expiry/
https://community.letsencrypt.org/t/help-thread-for-dst-root-ca-x3-expiration-september-2021/149190/36

I'd be ok with moving to bionic til it's the default.

After this is merged, are you planning on doing a blaster-outer?  If so, we have to do all of the gems too I think.

If this pull request is based on another pull request that has not been marked for backport, add the appropriate labels to the other pull request. Otherwise, please create a new pull request direct to the kasparov branch in order to resolve this.
Conflict details:
diff --cc .travis.yml
index 779dd15078,ab61112b61..0000000000
--- a/.travis.yml
+++ b/.travis.yml
@@@ -1,7 -1,9 +1,8 @@@
  ---
  language: ruby
+ dist: bionic
  rvm:
  - 2.6.6
 -- 2.7.2
  cache: bundler
  env:
    global:
@@@ -11,19 -13,25 +12,33 @@@
    - PARALLEL_TEST_PROCESSORS=2
    matrix:
    - TEST_SUITE=vmdb PARALLEL=true
 -  - TEST_SUITE=security
 +  - TEST_SUITE=brakeman
  matrix:
    fast_finish: true
++<<<<<<< HEAD
++=======
+   jobs:
+   - group: edge
+     virt: lxd
+     arch: arm64-graviton2
+     rvm: 2.6.6
+     env:
+     - RUBY_GC_HEAP_GROWTH_MAX_SLOTS=300000
+     - RUBY_GC_HEAP_INIT_SLOTS=600000
+     - RUBY_GC_HEAP_GROWTH_FACTOR=1.25
+     - PARALLEL_TEST_PROCESSORS=2
+     - TEST_SUITE=vmdb PARALLEL=true
++>>>>>>> 31aeded52b... Merge pull request #21472 from bdunne/certs
  addons:
    postgresql: '10'
 -before_install: source ${TRAVIS_BUILD_DIR}/bin/ci/before_install.sh
 -before_script: source ${TRAVIS_BUILD_DIR}/bin/ci/before_script.sh
 -after_failure: source ${TRAVIS_BUILD_DIR}/bin/ci/after_failure.sh
 +before_install:
 +- source ${TRAVIS_BUILD_DIR}/tools/ci/before_install.sh
 +before_script:
 +- bundle exec rake test:$TEST_SUITE:setup
 +script:
 +- bundle exec rake test:$TEST_SUITE
 +after_failure:
 +- source ${TRAVIS_BUILD_DIR}/tools/ci/after_failure.sh
  notifications:
    webhooks:
      urls:

kasparov backport details
commit 0bbeff1936c48c03ae0e1d9b96ace4102ea6a66f (HEAD -> kasparov, upstream/kasparov)
Author: Jason Frey<EMAIL_ADDRESS>Date:   Thu Sep 30 16:40:10 2021 -0400

    Merge pull request #21472 from bdunne/certs

    Switch to bionic on Travis

    (cherry picked from commit 31aeded52b6e0ac0209522216dee10cfaed68ca2)


KinD's Sysbox support

The goal here is to allow KinD tool to be able to create K8s clusters running over Sysbox containers. Sysbox would offer the following benefits over KinD's default runtime (runc):

Security: Sysbox would allow secure containers to be utilized as K8s-nodes -- currently KinD can only run over 'privileged' containers.
Simplicity: KinD's Dockerfile images could be drastically simplified as Sysbox absorbs most of the complexity required to allow K8s execution.
Flexibility: Sysbox imposes no restrictions on the docker images to utilize, so users would have more flexibility to define their own (customized) K8s-node images, including inner images corresponding to K8s components and/or applications.

The following issue has been filed in KinD's project to track this effort: kubernetes-sigs/kind#1772.

See this comment regarding the feasibility of realizing the simplicity and flexibility benefits: https://github.com/nestybox/sysbox/issues/70#issuecomment-692200331


Switch to Workspaces

This is going to be a big one.
@lasryaric @fontanierh not expecting you to review it but you can maybe have a look at the new Authenticator object and how it works (basiscally no more user anywhere).
It's here: https://github.com/dust-tt/dust/blob/62d32b9f05ce30102a50e0c1a91cb8c948cc1f6b/front/lib/auth.ts

This is ready for merge.
Deployment procedure is actually straightforward.

run init.sh (index update)
For each production instance

Deploy core
Build and deploy front


fix: make paths relative to the owner for RunProtocStep

gen-proto step fails when run from a separate project because the build system try to find the files in the current directory.
This PR addresses that issue by making paths relative to the source root.

duplicate of #46 , but your interest and PR is extremely appreciated. Thank you for your work and time,


I'm noticing some windows specific imports

Any way you can add Mac and Linux support?

I think that is possibe, we could just check what is the system, will add tommorow since i have no time today

I am against cross platform support, since we did that in timerx and that was always a headache for me.  Yea, the package is useable on macos. Just dont call the functions that only work on windows like the window blur

I am against cross platform support, since we did that in timerx and that was always a headache for me. So if you really want that implemented, it has to be done by sumeshir or lg

I can handle the Mac part for cross platform stuff


nixos/systemd: drop warnings to improve evaluation time

On my machine those warnings took 40% of the evaluation time.
A none very scientific test changed the time a noop nixos-rebuild switch takes from 26s to 21s.
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

IMO this is fine.

What happens if you move it into a let-block at the beginning of the file?

:-1: I think these warnings are useful. We shouldn't be dropping warnings that alert users of footguns, just because Nix Evaluation is slow.
@SuperSandro2000 can you check if the proposal from @Profpatsch helps?

just because Nix Evaluation is slow.

They are not just slow, they use almost 40% of the evaluation time when not rebuilding any changes.
I'll post the next flamegraph I have.

What happens if you move it into a let-block at the beginning of the file?

I just tried doing that but it references local variables so that does not work.

Thats the old flamegraph if you want to explore it
nix-function-calls.zip

Did you even test this? There are no checkboxes in the PR

Did you even test this? There are no checkboxes in the PR

As you probably know, I never tick those boxes and I must have tested this or where else should I've got those numbers?

That doesn't work because the let in only uses local variables.

Not in favor of this for the same reasons as @flokli — people who want this kind of thing should rather invest in faster evaluation time IMHO.


Generalize access logging (rebased)

Refactor access logging internals from oxyproxy.go to accesslog/log.go, allowing for sharing most of the code with other proxy types.
Note that the (marked internal) http.Request has been removed from the access log message. This means that it is also removed from the json formatted output.

Rebased to current master. Envoy integration depends on this, so please review :-)


Update yolov8 example to use Keras Core

@ianstenbit @fchollet

@ianstenbit  Also, given script doesn't work with jax backend ...

@ianstenbit Any updates?

@ianstenbit Any updates?

Can you elaborate on the issues you were seeing with the JAX backend?

Can you elaborate on the issues you were seeing with the JAX backend?

Getting error while using augmentation layers
    ValueError: Only input tensors may be passed as positional arguments. The following argument value should be passed as a keyword argument: tf.RaggedTensor(values=tf.RaggedTensor(values=Tensor("RaggedFromVariant/RaggedTensorFromVariant:2", shape=(None,), dtype=float32), row_splits=Tensor("RaggedFromVariant/RaggedTensorFromVariant:1", shape=(None,), dtype=int64)), row_splits=Tensor("RaggedFromVariant/RaggedTensorFromVariant:0", shape=(5,), dtype=int64)) (of type <class 'tensorflow.python.ops.ragged.ragged_tensor.RaggedTensor'>)

colab : https://colab.research.google.com/drive/1qtAk8GRF1Nt_9TtkkLMPBkqUUiBA3bB2?usp=sharing

Can you elaborate on the issues you were seeing with the JAX backend?

Getting error while using augmentation layers
    ValueError: Only input tensors may be passed as positional arguments. The following argument value should be passed as a keyword argument: tf.RaggedTensor(values=tf.RaggedTensor(values=Tensor("RaggedFromVariant/RaggedTensorFromVariant:2", shape=(None,), dtype=float32), row_splits=Tensor("RaggedFromVariant/RaggedTensorFromVariant:1", shape=(None,), dtype=int64)), row_splits=Tensor("RaggedFromVariant/RaggedTensorFromVariant:0", shape=(5,), dtype=int64)) (of type <class 'tensorflow.python.ops.ragged.ragged_tensor.RaggedTensor'>)

colab : https://colab.research.google.com/drive/1qtAk8GRF1Nt_9TtkkLMPBkqUUiBA3bB2?usp=sharing

Ahh I see the issue.
For preprocessing, using keras.Sequential is problematic here because that's a Keras Core model that expects JAX inputs.
Instead, try using a keras_cv.layers.Augmenter (same API, just swap the symbol)

@ianstenbit now its working with keras_cv.layers.Augmenter and also its working on kaggle, I will push a commit in some time updating code and docs ..

@ianstenbit Now you can review it ...

I think @jbischof has approval powers here -- we'll also need to render the changes to the guide @IMvision12 (to update the MD and ipynb files)

Once @jbischof and @fchollet approve these changes then I can add the generated files

@jbischof Any updates?

@IMvision12
I am testing your Yolo colab code, your keras-core version; https://colab.research.google.com/drive/1qtAk8GRF1Nt_9TtkkLMPBkqUUiBA3bB2?usp=sharing
And for training(yolo.fit)
I removed .take(2) in your code;
yolo.fit(
train_ds,
steps_per_epoch=2,
validation_data=val_ds.take(2),
epochs=3,
callbacks=[coco_metrics_callback],
)
also, I modified epochs to 100 as follows, but the loss does not improve,
yolo.fit(
train_ds,
steps_per_epoch=2,
validation_data=val_ds,
epochs=100,
callbacks=[coco_metrics_callback],
)
I think I am making a salient mistake! also, do I need to remove /take from the previous line(val_ds.take(2)) in the following;
coco_metrics_callback = keras_cv.callbacks.PyCOCOCallback(
val_ds.take(2), bounding_box_format="xyxy"
)
Thanks

Hi @jbischof, can you please review the merge of the above issue?

@IMvision12
I ran it again in Colab using T4 machine, it has this issue on Model creation part;
Code:
backbone = keras_cv.models.YOLOV8Backbone.from_preset(
"yolo_v8_s_backbone_coco"  # We will use yolov8 small backbone with coco weights
)
Error:
AttributeError                            Traceback (most recent call last)
 in <cell line: 1>()
----> 1 backbone = keras_cv.models.YOLOV8Backbone.from_preset(
2     "yolo_v8_s_backbone_coco"  # We will use yolov8 small backbone with coco weights
3 )
4 frames
/usr/local/lib/python3.10/dist-packages/keras_cv/models/utils.py in get_tensor_input_name(tensor)
23         return tensor._keras_history.operation.name
24     else:
---> 25         return tensor.node.layer.name
26
27
AttributeError: 'KerasTensor' object has no attribute 'node'

P.S. : @ianstenbit and @fchollet, could you please take a look at KerasCV? It needs more attention.

@Paryavi I have opened a issue https://github.com/keras-team/keras-io/pull/813

@IMvision12
It seems that issue 813 was resolved.
Can we fix the error I shared in this thread by modifying your YoloV8 example to Keras 3?


Calculate cost using tokens that will actually be sent

The in-chat token count & cost seem to sum up all the tokens in the current chat history, even if the token count exceeds the maximum token count for the model.
If the in-chat cost is the estimated cost for current submission, it should not include the tokens that will not be sent as part of the next request.
The total cost should be calculated like this as well.

This is correct, the cost is miscalculated but it is only noticeable in long conversations. If the owner could fix this bug it would be great, as it gave me a very big surprise to see the counter.


Return PermissionDenied status instead of InvalidArgument in case access to resources of not permitted project

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Nice catch!
/approve


Combining mergePackageProject with nestPackage depends on timing

Hi,
I have a multi-project build where one project uses mergePackageProject of another’s packageCompose which uses nestPackage.
This either works fine or it results, depending on timing, in the following failure:
$ ./gradlew packageCompose
> Task :package:packageCompose FAILED
Build failure
Execution failed for task ':package:packageCompose'.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':package:packageCompose'.
> Should not be called

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 776ms
4 actionable tasks: 1 executed, 3 up-to-date

Only happens when parallel builds are enabled and when the task with nestPackage finishes after the one with mergePackageProject.
I have prepared a repo with a reproducible example. I use Thread.sleep(10 * 1000) as a stand-in for a long running dependency.
Removing either nestPackage or mergePackageProject will make the build always succeed.

interesting scenario. totally amazing that you prepared an example which reproduces an issue.
I will use it tomorrow to find a fix for that. thanks!

you are speeding me up :)
if you downgrade Gradle in your example from 6.3 to 6.2.1. it will work. I haven't tested yet 6.3 :)
looks like something in Gradle changed and there is a chance that it is retrogression of Gradle itself.
however, I will try to fix it or find corresponding issues in the Gradle project. There was issues in the past looking similar and in the end, after upgrading Gradle issue was gone in GAP.

Oh, I didn’t make the connection because I had updated gradle before updating GAP in my project, the problem only manifests itself sometimes and only immeadiately after a ./gradlew clean so I didn’t catch it at first…

don't worry ;)
reported to Gradle - https://github.com/gradle/gradle/issues/12677

That line is causing issue probably. And seems that it could be easily improved. I've got an idea. Tomorrow I will check and let you know.
https://github.com/Cognifide/gradle-aem-plugin/blob/master/src/main/kotlin/com/cognifide/gradle/aem/pkg/tasks/PackageCompose.kt#L154

the order of evaluation is a problem;
you could have it fixed without upgrading GAP by writing:
		packageCompose {
			dependsOn(":package:packageCompose")
			mergePackageProject(":package")

		}

to be released in 12.0.0-beta


[improvement](External Catalog) Remove unnecessary conjuncts handling in External Catalog

In the previous FileScanNode, some parts that used conjuncts for predicate conversion were placed in the init phase. However, for the Nereids planner, pushing the filter down to the scan happens in the Translator, which means that the ScanNode can only get the complete conjuncts in the finalized phase. Therefore, in this PR, I have removed all conjuncts variables in External for the Nereids planner. They no longer need to store conjuncts themselves or add them to the ScanNode. Instead, all places in the ScanNode that use conjuncts should be moved to the finalized phase.

Yes, such as: test_hive_partition, test_hive_default_partition


Add JSON Schema for Point Cloud extension

A JSON schema for the Point Cloud extension is missing and should be added.

I’m having a go - but feel free to get in first :D

We forgot to close this. There's now a JSON Schema for the PC extension.


Problem on getMediaByTags - stdClass::$authenticated

Hello there,
i am using latest version of this Lib, but when i want to scarp by tag i am getting the following error:
Undefined property: stdClass::$authenticated
instagram-php-scraper/src/InstagramScraper/Instagram.php on line 1225.
My code:
            $instagram = \InstagramScraper\Instagram::withCredentials('username', 'password'); $instagram->login(); $medias = $instagram->getMediasByTag('trbvls', 10);
What might be the problem?

My fault... i used the login() method, but i dont need that cause i am using withCredentials() before.

My fault... i used the login() method, but i dont need that cause i am using withCredentials() before.

Sry for asking, but if you use withCreditentials(), you don't should use login() at all?

My fault... i used the login() method, but i dont need that cause i am using withCredentials() before.

Worked for me.

im still facing same issue is there quick fix?

it does working fine on my local version but when im using it online its showing me erro
Undefined property: stdClass::$authenticated


io.github.brevilo.jolm.Utils$OlmException: BAD_MESSAGE_MAC

Hey @brevilo, happens this error occasionally in a unit test, and it only seems to happen in unit tests.

It happens less often than #4 .

When reporting issues please at least include at the jOlm and JVM versions used and also the actual stacktrace, not just the last line.
Thanks

Hey @brevilo, happens this error occasionally in a unit test, and it only seems to happen in unit tests.

Does this also happen when you annotate your unit test with @TestInstance(TestInstance.Lifecycle.PER_CLASS)? The https://github.com/brevilo/jolm/issues/4#issuecomment-968356340 for some background info.

It should be the same problem as #4.


CONAN_PIP_COMMAND has no effect in docker conteiner

Conan Package Tools Version: 0.30.4
Operating System: Linux

Steps to reproduce

Set the following environment variables:
CONAN_USE_DOCKER=True
CONAN_DOCKER_IMAGE_SKIP_UPDATE=False
CONAN_PIP_COMMAND=pip3
CONAN_DOCKER_IMAGE=image where only pip3 is available
Start CPT on Linux

Actual result:
CPT fails with the "pip: not found" error
Expected result:
pip command specified in CONAN_PIP_COMMAND variable is used
Additional info
"pip install" command is hard-coded in this method:
https://github.com/conan-io/conan-package-tools/blob/1699024a011b479d12c854c947b86e55e0aff0c2/cpt/runner.py#L212

Why do you need it? All Conan docker images are configured to run pip as default, also it's used pyenv to install any python/pip version.

I need it because I use my own containers and I don't want to change them. I'm trying to configure CPT instead. But, unfortunately, some CPT settings have no effect inside the container.
The documentation says about support of specific set of containers. So it's probably not a bug. But it would be great to have additional build configuration tools supported in Docker.

The problem is that I can't change CONAN_PIP_COMMAND to be executed in the container, otherwise, it will break most users which run a custom pip oh host, and the regular one in the container. Why don't you use conan docker images instead? You can create you docker images based on conan images too. Otherwise, I recommend using alias command to reach pip as your custom pip command.

BTW the CONAN_PIP_COMMAND has been created only to solve problem under the host: https://github.com/conan-io/conan-package-tools/issues/147

I see that backward compatibility can be an issue. If you decide to add this feature in the future, probably you can add bool variable CONAN_DOCKER_USE_PIP_COMMAND to control this behavior.


coverage report wrong info

current coverage report doesn't pre-compile the source files correctly, and report wrong information and wrong line numbers.

fixed in dev.

@Havunen I can confirm that is not fixed. The issue sticks deeper then this. You need to remove pretty and post loaders etc. So still issues with coverage.
I know this because I fixed this in boily. You can eventually copy that solution.

ok I will look into that tommorow.

You can see here how it's fixe: https://github.com/Kflash/boily/blob/master/config/webpack.karma.config.js#L10-L47

@trueadm @Havunen Sad news! This issue can't be fixed at all!! There are some major conflicts between webpack and Karma. So any tried on a solution will fail :(
I experienced this as well on the Boily repo. Also the solution I thought worked, failed now.
A solution on the problem is coming soon, but not in the way we wanted it.

ok, meanwhile I think the current coverage is good enough. Even if line numbers are wrong in report you can see what is covered and what not. closing.

Sure thing. If you know what is on line 145 in index.js when the index.js in reality have 30 lines. And what is covered is totaly wrong if you look at the coveralls.io webpage.
Sure thing, then this wouldn't need a fix.
As @trueadm knows, there was and is issues with wrong lines reported on the coveralls.io as well.

@trueadm I just ran Inferno through a fixed coverage that works, and Inferno are actually 86% covered. Not 93% as reported.

looking into HTML report you see branches correctly ( I think )

but in case you find better way for Boily maybe you can just add it to Inferno also?

@Havunen Boily v. 3.0.0 just got released. Everything solved there. So just copy&paste

Ok good! I will look into that after events task.

In boily changes are made for travis.yml that is important you copy over. Just read the Q&A in the readme.

@Havunen @trueadm I just ran coverage check, and I still get wrong coverage report. Is this going to be fixed?

Related issue
https://github.com/webpack/karma-webpack/issues/13

@Havunen @trueadm No longer an issue. Fixed in the w branch.


import Document with tesseract OCR enabled on M2 Mac arm64 is giving DllNotFoundException for libleptonica-1.82.0.dylib

Describe the bug
Uploading pdf documents for long term memory under Documents tab fails on M2 Macbook pro arm64 platform with following error:
fail: Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware[1]
     An unhandled exception has occurred while executing the request.
     System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
      ---> System.DllNotFoundException: Failed to find library "libleptonica-1.82.0.dylib" for platform x64.
        at InteropDotNet.LibraryLoader.LoadLibrary(String fileName, String platformName)
        at InteropRuntimeImplementer.LeptonicaApiSignaturesInstance.LeptonicaApiSignaturesImplementation..ctor(LibraryLoader loader)
        at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
        at System.Reflection.ConstructorInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)
        --- End of inner exception stack trace ---
        at System.Reflection.ConstructorInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)
        at System.Reflection.RuntimeConstructorInfo.Invoke(BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
        at System.RuntimeType.CreateInstanceImpl(BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture)
        at InteropDotNet.InteropRuntimeImplementer.CreateInstance[T]()
        at Tesseract.Interop.LeptonicaApi.Initialize()
        at Tesseract.Interop.TessApi.Initialize()
        at Tesseract.Interop.TessApi.get_Native()
        at Tesseract.TesseractEngine..ctor(String datapath, String language, EngineMode engineMode, IEnumerable`1 configFiles, IDictionary`2 initialOptions, Boolean setOnlyNonDebugVariables)
        at Tesseract.TesseractEngine..ctor(String datapath, String language, EngineMode engineMode)
        at CopilotChat.WebApi.Extensions.CopilotChatServiceExtensions.<>c__DisplayClass4_0.<AddPersistentOcrSupport>b__1(IServiceProvider sp) in /Users/ag568e/personal/mlcode/chat-copilot/webapi/Extensions/ServiceExtensions.cs:line 137
        at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitFactory(FactoryCallSite factoryCallSite, RuntimeResolverContext context)
        at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitRootCache(ServiceCallSite callSite, RuntimeResolverContext context)
        at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSite(ServiceCallSite callSite, TArgument argument)
        at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.Resolve(ServiceCallSite callSite, ServiceProviderEngineScope scope)
        at Microsoft.Extensions.DependencyInjection.ServiceProvider.CreateServiceAccessor(Type serviceType)
        at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd(TKey key, Func`2 valueFactory)
        at Microsoft.Extensions.DependencyInjection.ServiceProvider.GetService(Type serviceType, ServiceProviderEngineScope serviceProviderEngineScope)
        at Microsoft.Extensions.DependencyInjection.ActivatorUtilities.GetService(IServiceProvider sp, Type type, Type requiredBy, Boolean isDefaultParameterRequired)
        at lambda_method84(Closure, IServiceProvider, Object[])
        at Microsoft.AspNetCore.Mvc.Controllers.ControllerFactoryProvider.<>c__DisplayClass6_0.<CreateControllerFactory>g__CreateController|0(ControllerContext controllerContext)
        at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
        at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.InvokeInnerFilterAsync()
     --- End of stack trace from previous location ---
        at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeFilterPipelineAsync>g__Awaited|20_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
        at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeAsync>g__Awaited|17_0(ResourceInvoker invoker, Task task, IDisposable scope)
        at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeAsync>g__Awaited|17_0(ResourceInvoker invoker, Task task, IDisposable scope)
        at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
        at Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext)
        at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
        at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
        at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
        at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddlewareImpl.Invoke(HttpContext context)

To Reproduce
Steps to reproduce the behavior:

Go to 'Documents' tab
Click on 'upload' button and Select a pdf file to upload
The upload fails
See error logs in terminal

Expected behavior
A document upload should work without error
Screenshots

Platform

OS: [Mac M2 chipset arm64 platform]
Source: [Tesseract Initialization]

Additional context
Add any other context about the problem here.

Self-assigning to test on MAC. Will add more details after testing

Hey @gotoashishgupta, this is an issue with tesseract in Linux / MacOS, where its native dependencies (libleptonica in this case) aren't included for any platforms other than Windows, so you have to manually compile and link the library within the Chat Copilot webapi.
You can try installing libleptonica-dev, copying the *.so file to webapi/bin/Debug/net6.0/x64 folder, and renaming it to leptonica-1.82.0.dll.
I found these threads most helpful in providing guidance:

https://stackoverflow.com/questions/69813484/error-could-not-load-file-or-assembly-tesseract-on-ubuntu-using-net-core-3-1/71066165#71066165
https://github.com/sirfz/tesserocr/issues/268

Closing this thread as this issue is related to Leptonica Library. As the library is a third-party library, it is out of the scope for further support in this forum. Please continue with the thread linked above!

@teresaqhoang leptonica-1.82.0.dll is already present at webapi/bin/Debug/net6.0/x64, however I still see the bug

@gotoashishgupta Did you find a solution to this problem? I'm currently experiencing the same issue.

this is a little strange. In .NET 8 when I run the console application through visual studio, everything works fine, but if I run the .exe file directly, I get the error "Exception has been thrown by the target of an invocation. ## Failed to find library " leptonica-1.82.0.dll" for platform x64."
There is the x86 and x64 folder with the appropriate DLLS, but I still get the error. I even tried moving the dlls to the system32 context, but the error remains.
When we compile the application, the x86 and x64 folder is also copied to the program root as shown in the image below, but the API cannot be loaded, as the system does not identify the existence of the DLL

Did anyone have a solution for this? I'm facing the same problem on MacOS Intel.

I solved this at the time for windows. If it helps, this is the link to the solution.
,
https://stackoverflow.com/questions/78105308/leptonica-1-82-0-not-found-in-net-runtime

Did anyone have a solution for this? I'm facing the same problem like Lucas on MacOS Intel.

https://github.com/charlesw/tesseract/blob/master/docs/Get running on macOS.md
If you use an Apple silicon chip the homebrew path isn’t /usr/local but /opt/homebrew. Also for me it asked for 1.82.0 for libleptonica instead of 1.80.0 and version 5.0 (50) instead of 4.1 which is used here for tesseract.

Hey I managed to get things working with homebrew on apple silicon by adding the following to my csproj:
  <Target Name="link_deps" AfterTargets="AfterBuild">
  <Exec Command="ln -sf /opt/homebrew/lib/libleptonica.dylib $(OutDir)x64/libleptonica-1.82.0.dylib"/>
  <Exec Command="ln -sf /opt/homebrew/lib/libtesseract.dylib $(OutDir)x64/libtesseract50.dylib"/>
  </Target>

It would appear there have been some naming convention changes since the docs were written.


Site test infrastructure

Background
Exokit essentially reimplements all of WebGL.
Problem
We have bugs and regressions with various sites as our code changes, which it does frequently! Some of these are soft-crashes in JS (throws), some is hardcrashes in the native bindings (SEGV, hangs), and some is incorrect rendering.
Solution
We could automate checking a list of sites against a build of Exokit in our CI environments on Appveyor and Travis CI.
In particular it seems it would be straightforward to check that some sites load and run to completion -- the harder part of this is differential rendering verification, which we might want to punt on till later.
One wrinkle is that we might want to make this work for all three OSes we support (Windows, OSX, Linux), so we might need three harnesses. Also, since Exokit is a graphical app that requires either OpenGL or an emulation of it (e.g. mesa on Linux), we'll need to make sure the CI environment runs such support.
This could be broken up into separate issues if this proves to be too much to chew at once 🍖.
Getting started
The three.js examples are a goldmine of simple but thorough test suites.
The ratio is high but not all of them examples load and render. However, those that do should continue to do so and can be whitelisted for verification.

Does this qualify as unit tests?


이정안님 mentoring meetin 30, 2024-02-05

오후 11시, google meet

https://github.com/fkdl0048/Mentoring-Project/issues/116


Mobile- Coming soon processing until i click another page

Mobile- Coming soon processing until i click another page


DimensionChooser should persist last value to preference

Currently, DimensionChooserModel has built in support for persisting history to a preference via its historyPreference config, but no method for saving the last used value.
Propose we offer a unified preference config that saves both aspects in a structured way. You can optionally toggle off either if you're not interested in persisting it.

https://github.com/exhi/hoist-react/pull/1236
https://github.com/exhi/toolbox/pull/219


Please, update dependencies

Because every version of dash_flags depends on flutter_svg ^1.1.6 and ***** depends on flutter_svg ^2.0.0+1, dash_flags is forbidden.
So, because *** depends on dash_flags ^0.0.3, version solving failed.
pub get failed

Upgraded and officially released to pub.dev 💙
Thanks for helping out 🙏🏻💙


Update RAC+JSONAble.swift to RAC 4

I want to use SignalProducer instead of RACSignal in RAC+JSONAble.swift but it gives me error "Cannot invoke 'Failure' with an argument list of type (NSError)". Anyone has idea about this?
extension SignalProducer  {
/// Get given JSONified data, pass back objects
func mapToObject(classType: JSONAble.Type) -> SignalProducer<JSONAble, E> {
func resultFromJSON(object: [String: AnyObject], classType: JSONAble.Type) -> JSONAble {
return classType.fromJSON(object)
}
        return attemptMap {
           object -> Result<JSONAble, E> in
           if let dict = object as? [String: AnyObject] {
              return .Success(resultFromJSON(dict, classType: classType))
          }

          return .Failure(NSError(domain: MoyaErrorDomain, code: MoyaErrorCode.Data.rawValue, userInfo: nil))
    }
}

We've just finished the conversion from RAC to RxSwift, https://github.com/artsy/eidolon/compare/master...develop in Eidolon.
Which means that that file has been completely re-written too, you might want to go and ask this question to the ReactiveCocoa team or on Stack Overflow.


Retrieving unknown variant via GET returns AssertionError

Just tested the python wrapper and noticed that when querying for a single variant that is not in any database, a 404 response is returned, thus breaking the 200 assertion in the _get method.
import myvariant
mv = myvariant.MyVariantInfo()
mv.getvariant('chr17:c.3748G>T') # unknown variant

If I attempt to use mv.getvariants('chr17:c.3748G>T') (which uses POST), I get a response like so:
In [7]: mv.getvariants(["chr7:g.140453133Tmv.getvariants('chr17:c.3748G>T')
querying 1-1...done.
Out[7]: [{u'notfound': True, u'query': u'chr17:c.3748G>T'}]

The API actually returns 404 on this request, thus makes the assert break on line 195 of __init__.py
So, while the python script should account for these scenarios (tests should account for these situations too), I think it might be something the API should be doing differently.

Thanks for your feedback. I agree, the more friendly return from an unknown variant could be just a "None" instead of raising exception. Will make that change shortly.

Fixed in 6cfff54, now return None from getvariant method when the input id does not exist.

Great simple fix. The API still returns the 404 (which makes sense) but the python wrapper handles it in a more "friendly" manner.


async_setup_platforms will fail in vs 2023.3

Logger: homeassistant.helpers.frame
Source: helpers/frame.py:77
First occurred: 6:53:37 AM (1 occurrences)
Last logged: 6:53:37 AM
Detected integration that called async_setup_platforms instead of awaiting async_forward_entry_setups; this will fail in version 2023.3. Please report issue to the custom integration author for vivint using this method at custom_components/vivint/init.py, line 129: hass.config_entries.async_setup_platforms(entry, PLATFORMS)

Fixed in 2023.2.1


hi tried it but error on checkout conf..im using windows 7

Hello, can you please screenshot the error for me? I need more information in order to help.

im actaully using python 3.0

Hmm, well for starters, the script was made for Python 2.X, so I don't believe Python 3 will work. If you still get the error after using Python 2.X, let me know.

Closing this issue until I know that this issue still occurs on Python 2.X.


Adds a reset method with an optional completionBlock

When a user signs out of the app, we call -reset. However, if we immediately call -identify: with a new id that value is subsequently overwritten since -reset dispatches its work asynchronously.
Usage:
[[Mixpanel sharedInstance] resetWithCompletion:^{
    [[Mixpanel sharedInstance] identify:@"some id"];
}];

Closing this stale PR.


Re-order the section, Assignment, to make more fluent

In Assignment,

Each of the above kinds of variables will be explained later on."

But the section of "Local variables"(4.3) and "Global variables"(4.4) are before "Assignment"(4.5).
After confirmed by @jhass on IRC, this patch is going to move the Assignment section to above Local/Global variables.

Thanks!


[BUG]: install.sh fails due to 403

Version
0.5.5
OS
Linux
Describe the bug
Manual installation instruction fails:
$ curl -sSL https://raw.githubusercontent.com/version-fox/vfox/main/install.sh | bash
Failed to get the latest version. Please check your network connection and try again.

This is due to the curl call to get the version (NOTE: This will also fail the earlier check if only wget available):
https://github.com/version-fox/vfox/blob/89689740184e2e5c26377107441028002ffb373d/install.sh#L16-L23
The failure is due to 403 status:
$ curl -fsSL "https://api.github.com/repos/version-fox/vfox/releases/latest"
curl: (22) The requested URL returned error: 403

You should not rely on user session to install. I tried to use vfox in a container on WSL2.

I installed this way instead:
curl -fsSL https://github.com/version-fox/vfox/releases/download/v0.5.5/vfox_0.5.5_linux_x86_64.tar.gz \
  | tar -xz -C /usr/bin/local --strip-components=1 vfox_0.5.5_linux_x86_64/vfox

If you improve your GH release process, this could be simpler.

Don't use version in asset name.
Don't include directory in archive (or at least not one with version

Then it would look like this:
curl -fsSL https://github.com/version-fox/vfox/releases/download/v0.5.5/vfox_linux_x86_64.tar.gz \
  | tar -xz -C /usr/bin/local vfox

Now only the version tag is in the URL. This allows to also get the latest version released:
curl -fsSL https://github.com/version-fox/vfox/releases/latest/download/vfox_linux_x86_64.tar.gz \
  | tar -xz -C /usr/bin/local vfox

This might because github api has rate limit,
we can add some tips like: Error: Request https://api.github.com/repos/xxxxx failed with status code: 403 (you may be rate limited)

we can add some tips

If you modify your release process, you will not need to use api.github.com, except for getting a version. Just delay this line:
https://github.com/version-fox/vfox/blob/89689740184e2e5c26377107441028002ffb373d/install.sh#L23
Show that after the binary is downloaded and extracted and now you have the version
$ vfox --version
vfox version 0.5.5

All of this can be simplified:
https://github.com/version-fox/vfox/blob/89689740184e2e5c26377107441028002ffb373d/install.sh#L46-L70
Example:
  sudo mkdir -p /usr/local/bin
  if [ $? -ne 0 ]; then
    echo 'Failed to create /usr/local/bin directory. Please check your sudo permissions and try again.'
    exit 1
  fi

  local VFOX_LATEST_RELEASE_URL="https://github.com/version-fox/vfox/releases/latest/download/vfox_${OS_TYPE}_${ARCH_TYPE}.tar.gz"
  curl -fsSL "${VFOX_LATEST_RELEASE_URL}" | sudo tar -vxz -C /usr/local/bin vfox
  if [ $? -ne 0 ]; then
    echo 'Failed to extract vfox to /usr/local/bin. Please check your sudo permissions and try again.'
    exit 1
  fi

  echo "$(vfox --version) installed successfully!"

No need for rm or mv, curl will pipe archive into tar command which extracts and moves for you.
To make this work you will need to publish to GH Releases with changes I mentioned:

Don't use version in asset name.
Don't include directory in archive (or at least not one with version


Move panel to statusbar - as icon who show popup panel

Recently introduced VSCode to move between the sidebar panel and the bottom panel.
The roadmap includes plans to move the panel to a separate window and various other panel layouts, e.g. side by side.
I would like to add to this, the proposal to move the panel to the statusbar - as icon who show a popup panel:

Reason:
Such a popup panel gives a greater sense of lightness compared to a collapsible / expandable sidebar panel whose bar always takes up space, even when the panel is collapsed - This fits in with the roadmap foundations for VSCode.
Additionally: It enables faster access to a specific, currently deeper hidden panels.

Why this way?
After all, it is in line with the roadmap.
Why isn't it added to the Backlog immediately?

@sbatten
Quote from the VSCode roadmap:

Workbench layout
Support for detachable workbench parts is our most upvoted feature request which due to architectural issues is challenging to implement. We will explore how we can work around this limitation. This investigation will focus on detaching terminals (2nd most upvoted feature request) and editors.
:runner: Allows to move views between the panel and sidebar.
Support a more flexible workbench layout such as allow sidebars on the left and the right. For example, you could have your outline on the right and the file explorer on the left.

You can see that this is the direction.
Popup Panels are the icing on the cake.

状态栏按钮可能不是一个好主意，因为状态栏很矮小，而且对于win10来说，只要鼠标稍微移过一点，就有可能触发系统任务栏的悬浮预览弹窗，而这个弹窗可能会正好将你需要点击的按钮遮蔽。
所以在我看来，更好的主意是：为一些可弹出的菜单和窗口制作相应的指令（vscode command），然后用户就可以用快捷键或者状态栏按钮调用这些指令（参考adam's toolbar）。

@KnIfER  After all, the status bar has buttons.
My suggestion would be status bar buttons that bring up a popup with a panel.
The button itself has buttons, after all!


[DOCS] Improve Readme file with animation and add Link to LICENSE File

Issue Description
The Readme file is not as good, I'll put animation in readme file with MIT LICENSE Link to the LICENSE file.
Additional Context
No response
Screenshots or Examples (if applicable)

Proposed Solution (optional)
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

@heizshubham this issue was assigned before, you can check it.
So i am closing this issue.
Please Understand.


Move container comparison logic to a separate function

As part of random improvements idea: move out some part of compareStatefulSetWith with cyclomatic complexity 23 to a separate function to simplify it.

:+1:

:-1:

:+1:

Given that this bit of code here already uses reflect anyway, we could consider using it even more aggressively: https://stackoverflow.com/questions/18930910/golang-access-struct-property-by-name
e.g.
	comparedFields := []string {"Name", "Image", "Ports", "Env", "EnvFrom"}
	comparators := map[string]interface{}{
		"Resources": compareResources,
	}

	func getField(v interface{}, field string) interface{} {
		return reflect.Indirect(reflect.ValueOf(v)).FieldByName(field)
	}

	for index, containerA := range setA.Spec.Template.Spec.Containers {
		containerB := setB.Spec.Template.Spec.Containers[index]
		for _, field := range comparedFields {
			fieldA := getField(a, field)
			fieldB := getField(b, field)
			comparator := comparators[field]
			if comparator != nil {
				comparator = reflect.DeepEqual
			}
			if (fieldA == nil) != (fieldB == nil) || !comparator(fieldA, fieldB) {
				needsRollUpdate = true
				reasons = append(reasons, fmt.Sprintf("new statefulset's container %d field %s doesn't match the current one", index, field))
			}
		}
	}

(NB: I didn't test this and I've never used this library so I have no idea if it works, but as far as I can tell it should probably work fine with only minor adjustments (maybe changing the parameter types of compareResources and possibly adding some casts inside or somesuch).  And it's easy to extend, clean, short, fits reasonably in the main function, and we're already relying on dynamic type information from the reflection module anyway, so introducing this stringy field access is no big deal IMHO)


Split button e2e

Please provide a link to the associated issue.
Closes SAP/fundamental-ngx#
improved code according to comments
Please provide a brief summary of this pull request.
Please check whether the PR fulfills the following requirements

[ ] the commit message follows the guideline:
https://github.com/SAP/fundamental-ngx/blob/main/CONTRIBUTING.md
[ ] tests for the changes that have been done
[ ] all items on the PR Review Checklist are addressed :
https://github.com/SAP/fundamental-ngx/wiki/PR-Review-Checklist
[ ] Run npm run build-pack-library and test in external application

Documentation checklist:

[ ] update README.md
[ ] Breaking Changes Wiki
[ ] Documentation Examples
[ ] Stackblitz works for all examples

Improved according to the comments at https://github.com/maxym-valor/fundamental-ngx/pull/1 from Sean Moore