Why you didn't implement the library as a recognizer instead a replacement of the activity handler?
This approach could be more generic compatible with composer and potentiallty usable  with other connectors than Teams.

Our team is not affiliated with the Composer. My understanding of Composer is that it is a solution for a code-less bot building experience, which is not the goal of this repo.
The focus of this repo (aside from the AI aspect) is for Teams-specific features (e.g. Message Extensions), so the code will build on top of Teams bot development. The part of the repo that is for Teams features won't be applicable to other channels without extra work from the bot developer. I'm taking a stab at the features of the recognizer that you are looking for:

Text recognition / Regex handling to route to the correct response / handler.
This is already available via app.message(..., which can use regular expressions or strings to identify the action for the app, without needing to take on Composer verbiage or even use AI to help with detection. That would be the equivalent of Composer's Recognizer (without LUIS)

Having the ability to implement LUIS in the app
app.ai.action(.... is the OpenAI equivalent to LUIS detection, but the models available for OpenAI are more robust than LUIS with less configuration needed. (Intents, entities, or their equivalents are all handled in prompt.txt, and OpenAI models don't require different apps to be created per human spoken/written language).

As far as I recall, Composer is the only connector/channel that abstracts out the Activity Handler concept, and therefore is not a prerequisite for other channels to take advantage of the AI section of this codebase.
The Activity Handler isn't unique to Teams, and therefore still builds on top of 'universal' Bot Builder concepts. As you might guess, with a focus on OpenAI, there is no immediate need for this repo to make use of LUIS language understanding, since our prompts for the assistant / AI can naturally detect 'intents' using its language model and prompt. I can't speak for the future as to whether or not LUIS will be supported by us in the future, but at the moment it is not on our roadmap.

I suggest to use recognizer as are more generic and can be used in a more generic way as, in my mind , activity handler should be full feature from bot point if view , eg support all type of activity as events. The scenario is the one you highliged: an alternative to Luis easier to use and more robust. I have an other question scenario like intents with parameters and subsequent request if all parameters are not provide how are handle with your framework? With a new message with a context? No use of activedialogs etc ?

From: Corina @.>
Sent: Monday, June 5, 2023 8:47:27 PM
To: microsoft/teams-ai @.>
Cc: Ugo Castellani @.>; Author @.>
Subject: Re: [microsoft/teams-ai] [Feature Request]:  (Issue #170)
Our team is not affiliated with the Composer. My understanding of Composer is that it is a solution for a code-less bot building experience, which is not the goal of this repo.
The focus of this repo (aside from the AI aspect) is for Teams-specific features (e.g. Message Extensions), so the code will build on top of Teams bot development. The part of the repo that is for Teams features won't be applicable to other channels without extra work from the bot developer. I'm taking a stab at the features of the recognizer that you are looking for:

Text recognition / Regex handling to route to the correct response / handler.
This is already available via app.message(..., which can use regular expressions or strings to identify the action for the app, without needing to take on Composer verbiage or even use AI to help with detection. That would be the equivalent of Composer's Recognizer (without LUIS)

Having the ability to implement LUIS in the app
app.ai.action(.... is the OpenAI equivalent to LUIS detection, but the models available for OpenAI are more robust than LUIS with less configuration needed. (Intents, entities, or their equivalents are all handled in prompt.txt, and OpenAI models don't require different apps to be created per human spoken/written language).

As far as I recall, Composer is the only connector/channel that abstracts out the Activity Handler concept, and therefore is not a prerequisite for other channels to take advantage of the AI section of this codebase.
The Activity Handler isn't unique to Teams, and therefore still builds on top of 'universal' Bot Builder concepts. As you might guess, with a focus on OpenAI, there is no immediate need for this repo to make use of LUIS language understanding, since our prompts for the assistant / AI can naturally detect 'intents' using its language model and prompt. I can't speak for the future as to whether or not LUIS will be supported by us in the future, but at the moment it is not on our roadmap.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/teams-ai/issues/170#issuecomment-1577292070, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA77TTXJ27SREDY4327DPTDXJYST7ANCNFSM6AAAAAAYZGTNQ4.
You are receiving this because you authored the thread.Message ID: @.***>

@guru98 I'd recommend taking a look at the documentation and samples to get a better idea of what capabilities the repo has. In particular I think the samples may help with understanding concepts of learning to 'speak' with an AI agent/assistant. Those samples are categorized under 04.____ in the samples. :)

@corinagum many thanks, anyway. It was just a suggestion. I wrote bots for the last 5 years and I have a sort of confidence with assistants and agents. As you can immagine i used other tecnology to recognize intents and handle in the bot. As you mentioned in a previous post use of openai instead of language understanding could have better results as use framework like semantic kernel and planners allow very interesting and powerful scenarious. My only concern is that replace of activity handler in the bot framework isn't a great idea as suppose that a specific bot have to be used for one channel only. Also an activity handler replacement shoud manage all possible type of activity as the original activity handler does (like events). I think that teams-ai is a great idea but you shoud be more accurate in the architectural choices ..., probably use a custom recognizer could be not the best solution  but be invasive in a "standard" architecture is not better. Thanks for your time to answer to my question.


Feature Request: Assign Levels for firing.

Currently the database will log for every event level except for panic, as seen here:
func (hook *Hook) Levels() []logrus.Level { 	return []logrus.Level{ 		logrus.FatalLevel, 		logrus.ErrorLevel, 		logrus.WarnLevel, 		logrus.InfoLevel, 		logrus.DebugLevel, 	} }
It would be ideal if this method can be provided externally with something like the following:
func (h *Hook) init() {     h.Levels = func (hook *Hook) Levels() []logrus.Level { 	return []logrus.Level{ 		logrus.FatalLevel, 		logrus.ErrorLevel, 		logrus.WarnLevel, 		logrus.InfoLevel, 		logrus.DebugLevel, 	}   } }
Then in my caller I can update Levels with something like:
hook := pglogrus.NewAsyncHook(db.DB, map[string]interface{}{}) 	hook.Levels = func() []logrus.Level { 		topLevel := int(settings.Log.DbLogLevel) + 1 		return logrus.AllLevels[0:topLevel] 	} 	l.AddHook(hook)
Alternatively it would be good if we can assign levels at declaration to a slice, and that could be used during execution. Something like the following:
type Hook struct { 	...         LogLevels []Level }

Thanks for submitting this. We never implemented that, because we never had the need. But I like the idea, and will accept any merge request with tests :)


SEO optimization

Currently we use one title for all pages.
We need to make seo better by adding dynamic seo

I found this doc : https://nextjs.org/docs/pages/api-reference/components/head, hope it can help

@ken999999999999999  Many Thanks. Will get started working on this later

[ ] Relocate 'use client' to react components
[ ] Store metadata in config/site.ts for easier management
[ ] Add metadata using generateMetadata
[ ] Create dynamic metadata content from page params
[ ] Build sitemap? (I haven't done this before, idk how to do it / if it's necessary, will do some research later)
[ ] Any other business

Ref:
Next.js:
Next.js / App Router / Building Your Application / Optimizing / Metadata
Similar websites:
Refer Me
Tentative metadata (please, comment & advice!) :)
I get Chinese version from /components/navigation-menu/base.tsx and try to translate them to English.

Route
Title
Description

/
Referalah | 海外港人搵 Referral 平台
海外港人搵Referral平台。 A job referral platform for overseas Hongkongers. Connect, refer and get referred to support the Hong Kong community across the globe.

/about
理念 About Us | Referalah
海外搵工好多時都需要人脈，有人推薦先有面試，而外地愈嚟愈多港人，但缺乏相關文化同平台。呢個平台喺俾大家搵翻同聲同氣嘅，無論你係藍領白領，都希望大家互相幫忙。

/auth
登入 / 註冊 Sign In / Sign Up | Referalah
透過魔術連結登入或註冊。無須密碼。 Sign in or sign up with magic link. Password is not required.

/post/create
貼街招 Stick a job posting | Referalah
你公司請人又想搵香港人？網上睇到份工又想人幫手？係到貼個街招，等人聯絡你。 Is your company hiring and looking for folks from Hong Kong? Stick a job posting here to get contacted.

/post/referee
人搵工 Job searching | Referalah
係到搵有乜人需要幫手。 Lend a helping hand to those who need a referral for a job posting.

/post/referer
工搵人 Job hiring | Referalah
係到搵有乜工搵人推薦。 Look for job postings and get referred.

/privacy-policy
隱私權政策 Private Policy | Referalah
海外港人搵Referral平台。 A job referral platform for overseas Hongkongers. Connect, refer and get referred to support the Hong Kong community across the globe.

/profile/[userUuid]
{{ user.username }} 用戶檔案 User Profile | Referalah
{{ user.description }}

/referee
受薦人 Referees | Referalah
想搵人材？係到睇下有冇合適嘅人啦！ Recommend an amazing talent from Hong Kong to join your company!

/referer
推薦人 Referers | Referalah
想搵人推薦你入去？係到搵下啦！ Meet someone to bring you closer to your dream company!

/terms-and-conditions
條款及細則 Terms and Conditions | Referalah
海外港人搵Referral平台。 A job referral platform for overseas Hongkongers. Connect, refer and get referred to support the Hong Kong community across the globe.

(default)
Referalah | 海外港人搵 Referral 平台
海外港人搵Referral平台。 A job referral platform for overseas Hongkongers. Connect, refer and get referred to support the Hong Kong community across the globe.

@511234  工搵人  will be removed

@511234 btw 原來有公司做呢啲野 不過要課金

@511234 工搵人 will be removed

ok! Haven't come back for a few weeks, need to catch up with new code. :-)

@511234 工搵人 will be removed

ok! Haven't come back for a few weeks, need to catch up with new code. :-)

Nvm take your time. Happy coding

Ahh should be 人搵工. 人搵工 already removed sor.

Please add your self into contributors page

Done


[FEATURE] Multiple Calendars

Hi there, is it possible to add a feature to have multiple calendars "swappable"? My campaign setting has multiple moons and there are currently (up to 4) two calendars based on two of the moons.
Both Calendars have the following in common

There are 8 days a week
There are 7 months a year
There are 314 days in a year

The differences is that the month names are different and the day distribution to them is also different so say if you are 93 days into the year you are on Month 3, Day 15 but on the other calendar you are Month 3 Day 9
I am basically asking if its possible to have a drop down menu on the "floating widget" to "hot-swap" between the two calendars. Which basically takes its current day/year place and shifts it over to the other calendar.

Support for multiple calendars is something I am currently working on so this will be coming!


python version of evaluator block

Create a more flexible eval() block that ships with the python blocks:

See https://github.com/pothosware/PothosBlocks/blob/master/event/Evaluator.cpp

WIP: https://github.com/pothosware/PothosPython/tree/wip/evaluator


When using disableBodyScroll, set stored scrollTo value to the initia…

…l scrolled position, as well as upon subsequent calls to init.
Hi Gromo, didn't find any information on the best way to submit proposed changes. If you'd prefer that i open up an issue or something else instead let me know.
I have the use case where i programatically scroll to a position initially. however when i mousewheel with disableBodyScroll: true, the mousewheel, assumes that initial scroll is 0, instead of the actual value and scrolls there.
thxs!

How do you change scroll position? Do you use jQuery scrollTop/scrollLeft? Can you prepare example with wrong behavior on jsFiddle.net using original jquery.scrollbar.js file?

here's a jsfiddle:
https://jsfiddle.net/tjqgxkd9/3/
When you attempt to use the mousewheel, if you mousewheel down, you are scrolled to the top, if you mousewheel up, it doesn't scroll.

apologies, try https://jsfiddle.net/tjqgxkd9/5/

Pretty interesting. If you use mousewheel over scrollbar it works correctly. Idea was that disableBodyScroll handles mousewheel and trigger the same event for scrollbars. I will look at the code in next few days

something like this?

https://jsfiddle.net/tjqgxkd9/6/ - quick hack

But much better if you set scrollTop offset first and only then initialize scrollbar


Bug:{product-long} is not  working on local

Bug:{product-long} is not  working on local

Duplicate of https://github.com/redhat-developer/app-services-guides/issues/94


Unauthorized; error code: 401 when using schema registry basic auth

Issue: when using confluent cloud hosted schema registry users are getting the following error:
Caused by: io.confluent.kafka.schemaregistry.client.rest.exceptions.RestClientException: Unauthorized; error code: 401

The reason for this is an old version used:
https://github.com/wepay/kafka-connect-bigquery/blob/1.2.0/build.gradle#L15
    ioConfluentVersion = '5.1.1'

In the above version we didn't have the basic auth to the Schema registry, with this in mind the REST call will drop the authentication method and we will forever get error code 401.
To resolve this, you can recompile the code using the README steps, with simply using 5.3.1 in the file mentioned above.
To fix this we should PR a higher version to the above code.

https://github.com/wepay/kafka-connect-bigquery/pull/209

@MosheBlumbergX this can be closed now, right?

Hey guys, is this resolved?
We use a hosted schema registry that supports basic auth, which we're passing in through the URL, but we're seeing
io.confluent.kafka.schemaregistry.client.rest.exceptions.RestClientException: Unexpected character ('<' (code 60)): expected a valid value (number, String, array, object, 'true', 'false' or 'null')

@alexhwoods it's difficult to debug just based on that message. Could you supply a complete log and your connector config?

{
    "name": "alex-clay-testing-1",
    "connector.class": "com.wepay.kafka.connect.bigquery.BigQuerySinkConnector",
    "topics": "outbound-orders.public.sales_order_request",
    "topicsToTables": "\\.=$2",
    "project": "alex-clay-test-1",
    "datasets": ".*=testing1",
    "schemaRetriever": "com.wepay.kafka.connect.bigquery.schemaregistry.schemaretriever.SchemaRegistrySchemaRetriever",
    "credentials": "<service-account-key>",
    "sanitizeTopics": "true",
    "autoCreateTables": "true",
    "schemaRegistryLocation": "<url-to-schema-registry>",
    "autoUpdateSchemas": "true",
    "keySource": "JSON"
}

And when we hit the REST endpoint checking the status of the connector, we get:
curl <kafka-connect-rest-url>/connectors/alex-clay-testing-1/status

{
    "name": "alex-clay-testing-1",
    "connector": {
        "state": "FAILED",
        "worker_id": "dev-gcloud-us-east4-kafka-6.aiven.local:8083",
        "trace": "org.apache.kafka.connect.errors.ConnectException: Exception while fetching latest schema metadata for topic=outbound-orders.public.sales_order_request, subject=outbound-orders.public.sales_order_request-value\n\tat com.wepay.kafka.connect.bigquery.schemaregistry.schemaretriever.SchemaRegistrySchemaRetriever.retrieveSchema(SchemaRegistrySchemaRetriever.java:72)\n\tat com.wepay.kafka.connect.bigquery.SchemaManager.createTable(SchemaManager.java:48)\n\tat com.wepay.kafka.connect.bigquery.BigQuerySinkConnector.ensureExistingTables(BigQuerySinkConnector.java:117)\n\tat com.wepay.kafka.connect.bigquery.BigQuerySinkConnector.ensureExistingTables(BigQuerySinkConnector.java:140)\n\tat com.wepay.kafka.connect.bigquery.BigQuerySinkConnector.start(BigQuerySinkConnector.java:159)\n\tat org.apache.kafka.connect.runtime.WorkerConnector.doStart(WorkerConnector.java:110)\n\tat org.apache.kafka.connect.runtime.WorkerConnector.start(WorkerConnector.java:135)\n\tat org.apache.kafka.connect.runtime.WorkerConnector.transitionTo(WorkerConnector.java:195)\n\tat org.apache.kafka.connect.runtime.Worker.startConnector(Worker.java:257)\n\tat org.apache.kafka.connect.runtime.distributed.DistributedHerder.startConnector(DistributedHerder.java:1183)\n\tat org.apache.kafka.connect.runtime.distributed.DistributedHerder.processConnectorConfigUpdates(DistributedHerder.java:541)\n\tat org.apache.kafka.connect.runtime.distributed.DistributedHerder.tick(DistributedHerder.java:388)\n\tat org.apache.kafka.connect.runtime.distributed.DistributedHerder.run(DistributedHerder.java:282)\n\tat java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)\n\tat java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)\n\tat java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)\n\tat java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)\n\tat java.base/java.lang.Thread.run(Thread.java:834)\nCaused by: io.confluent.kafka.schemaregistry.client.rest.exceptions.RestClientException: Unexpected character ('<' (code 60)): expected a valid value (number, String, array, object, 'true', 'false' or 'null')\n at [Source: sun.net.www.protocol.http.HttpURLConnection$HttpInputStream@4a049f1a; line: 1, column: 2]; error code: 50005\n\tat io.confluent.kafka.schemaregistry.client.rest.RestService.sendHttpRequest(RestService.java:209)\n\tat io.confluent.kafka.schemaregistry.client.rest.RestService.httpRequest(RestService.java:235)\n\tat io.confluent.kafka.schemaregistry.client.rest.RestService.getLatestVersion(RestService.java:444)\n\tat io.confluent.kafka.schemaregistry.client.rest.RestService.getLatestVersion(RestService.java:436)\n\tat io.confluent.kafka.schemaregistry.client.CachedSchemaRegistryClient.getLatestSchemaMetadata(CachedSchemaRegistryClient.java:209)\n\tat com.wepay.kafka.connect.bigquery.schemaregistry.schemaretriever.SchemaRegistrySchemaRetriever.retrieveSchema(SchemaRegistrySchemaRetriever.java:68)\n\t... 17 more\n"
    },
    "tasks": [],
    "type": "sink"
}

the key line there being the above error message

And with our schema registry, when you just do
curl <url>/subjects/<subject>/versions

with the basic auth in the URL it works fine, which is why I'm confused

We got this working by digging into the code. There are undocumented config options that made it work
"schemaRegistryClient.basic.auth.credentials.source": "USER_INFO",
"schemaRegistryClient.basic.auth.user.info": "username:password",


Delete button from context action menu is not working

Describe the bug
Delete circuit element option is not working from the action menu
To Reproduce
Steps to reproduce the behavior:

Go to 'Circuitverse'
Place an element
Right click and click Delete

Expected behavior
Element should be removed
Screenshots

Looks like this issue is due the fact that the yellow area inside the element are considered as selection area for an element
this issue is prominent with RGB LED matrix

Hey @anasaijaz, I want to work on this issue can you please assign this issue to me?

The delete Button is working fine for me.
@gr455 @pulkit-30 can you please check?

@anasaijaz all working fine for me, is there any specific event or browser for reproducing this issue...?
https://user-images.githubusercontent.com/76155456/216747360-f2673b0e-1cde-4672-9e2e-a9b7af4c1ba9.mov


no method matching /(::_MM{4,Int}, ::Int)

I'm running into the following problem when the @avx loop does some division by integers
using LoopVectorization
function int_bug(τ)
  M,N = size(τ)
  @avx for t = 1:N, j = 1:M
      τ[j, t] = ((j - 1) / (M - 1))
  end
  return nothing
end

τ = randn(16,16)

julia> int_bug(τ)
ERROR: MethodError: no method matching /(::VectorizationBase._MM{4,Int64}, ::Int64)
Closest candidates are:
  /(::BigInt, ::Union{Int16, Int32, Int64, Int8, UInt16, UInt32, UInt64, UInt8}) at gmp.jl:540
  /(::BigFloat, ::Union{Int16, Int32, Int64, Int8}) at mpfr.jl:425
  /(::Missing, ::Number) at missing.jl:115
  ...
Stacktrace:
 [1] macro expansion at /home/fredrikb/.julia/packages/LoopVectorization/MKHJJ/src/reconstruct_loopset.jl:469 [inlined]
 [2] _avx_!(::Val{(0, 0, 0)}, ::Type{Tuple{:j,:j,LoopVectorization.OperationStruct(0x0000000000000002, 0x0000000000000000, 0x0000000000000000, 0x0000000000000000, LoopVectorization.loopvalue, 0x00, 0x01),:LoopVectorization,:LOOPCONSTANTINSTRUCTION,LoopVectorization.OperationStruct(0x0000000000000000, 0x0000000000000000, 0x0000000000000000, 0x0000000000000000, LoopVectorization.constant, 0x00, 0x02),:LoopVectorization,:-,LoopVectorization.OperationStruct(0x0000000000000002, 0x0000000000000000, 0x0000000000000000, 0x0000000000000102, LoopVectorization.compute, 0x00, 0x03),:LoopVectorization,:LOOPCONSTANTINSTRUCTION,LoopVectorization.OperationStruct(0x0000000000000000, 0x0000000000000000, 0x0000000000000000, 0x0000000000000000, LoopVectorization.constant, 0x00, 0x04),:LoopVectorization,:-,LoopVectorization.OperationStruct(0x0000000000000000, 0x0000000000000000, 0x0000000000000000, 0x0000000000000402, LoopVectorization.compute, 0x00, 0x05),:LoopVectorization,:/,LoopVectorization.OperationStruct(0x0000000000000002, 0x0000000000000000, 0x0000000000000000, 0x0000000000000305, LoopVectorization.compute, 0x00, 0x06),:LoopVectorization,:setindex!,LoopVectorization.OperationStruct(0x0000000000000021, 0x0000000000000000, 0x0000000000000000, 0x0000000000000006, LoopVectorization.memstore, 0x01, 0x07)}}, ::Type{Tuple{LoopVectorization.ArrayRefStruct{:τ,Symbol("##vptr##_τ")}(0x0000000000000101, 0x0000000000000201, 0x0000000000000000)}}, ::Type{Tuple{0,Tuple{},Tuple{4},Tuple{(2, 1)},Tuple{},Tuple{},Tuple{}}}, ::Type{Tuple{:t,:j}}, ::Tuple{VectorizationBase.StaticLowerUnitRange{1},VectorizationBase.StaticLowerUnitRange{1}}, ::VectorizationBase.PackedStridedPointer{Float64,1}, ::Int64) at /home/fredrikb/.julia/packages/LoopVectorization/MKHJJ/src/reconstruct_loopset.jl:469
 [3] int_bug(::Array{Float64,2}) at /home/fredrikb/Desktop/debug/gdtw.jl:48
 [4] top-level scope at none:1

I can trick it to run using the following
function int_bug(τ)
  M,N = size(τ)
  Mf = Float64(M)
  @avx for t = 1:N, j = 1:M
      τ[j, t] = (1.0*j - 1.0) / (Mf - 1.0)
  end
  return nothing
end

but without the 1.0*j it does not work. Putting jf = Float64(j) as the first line in the loop does not work either.

Thanks for the report.
The _MM is still used sometimes to represent vectorized loop induction variables.
When we vectorize a loop (lets say by 4), then we increment j by 4 on each iteration.
The _MM type lazily acts like a vector, so that j can represent [1,2,3,4], [5,6,7,8], meaning to load from these values, etc.
julia> using SIMDPirates, VectorizationBase

julia> A = rand(20,20);

julia> A[5:8,5:8]
4×4 Array{Float64,2}:
 0.245402  0.651708  0.792535   0.568214
 0.707808  0.849284  0.0797999  0.698233
 0.846294  0.184514  0.849272   0.108233
 0.8059    0.756412  0.0511856  0.280004

julia> vload(stridedpointer(A), (4,4)) # zero based indexing
0.24540208602645652

julia> vload(stridedpointer(A), (_MM{4}(4),4)) # zero based indexing
SVec{4,Float64}<0.24540208602645652, 0.7078084447023407, 0.8462935942011729, 0.8058998588985846>

julia> vload(stridedpointer(A), (4,_MM{4}(4))) # zero based indexing
SVec{4,Float64}<0.24540208602645652, 0.651707919664472, 0.792534905530137, 0.5682137289640192>

julia> vload(stridedpointer(A), (_MM{4}(4),_MM{4}(4))) # zero based indexing
SVec{4,Float64}<0.24540208602645652, 0.8492843398829972, 0.8492715875423127, 0.2800041445479129>

By being lazy, loading from (_MM{4}(4),4) can be efficient, since it "knows" that these are contiguous.
By multiplying by 1.0, it has to abandon the lazy representation, because [5,6,7,8]*x wont lead to unit increments in general.
I added a few missing / methods.


Correction for raw affiliation Sorbonne Université, CNRS, LIP6, 75005, Paris, France

Correction needed for raw affiliation Sorbonne Université, CNRS, LIP6, 75005, Paris, France
raw_affiliation_name: Sorbonne Université, CNRS, LIP6, 75005, Paris, France
new_rors: 02en5vm52;02feahw73;05krcen59
previous_rors: 02en5vm52;02feahw73
works_examples: W4390410588
contact: d093eb3e883e76d4b3276ae45d1b2f56:54a9fe0149c547f88873e5a975 @ sorbonne-universite.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


Disable concurrency limiter

Adds a MiskFeature flag to disable the ConcurrencyLimiterInterceptor dynamically
see: https://docs.google.com/document/d/15IQxBccR0MlfoxfTT9j51NaCxSvhAHsA37wAN-IMdhs/edit#heading=h.mpnrnewt2hso
Note: The majority of the changes are to add bindings to every TestModule to support the feature flag

This is a big breaking change. It will break so many tests. I think we need to think through alternatives ways on how to introduce the feature flag that mitigate the impact of users.
Maybe MiskConcurrencyLimiterEnabledFeature should be an interface and have a simpler implementation by default?

When you say it breaks so many tests, you are talking about all the changes for the unit tests? I'm not sure how to simplify that as each test has a TestModule and a binding is required for each of them


Scale Onboarding = Ansible host file

Bug exists on page
https://www.azurecitadel.com/arc/servers-hack/scale/#additional-vms
Describe the bug
"Create an Ansible hosts file" is not explicit enough, had members of the team create their own Ansible files (which I don't think is the intent of the hack?)
To Reproduce
NA
Expected behavior
"Update the Terraform variables to create an Ansible hosts file"
or
Explicitly set the variable in terraform.tfvars so attendees are aware
or
Does this affect the initial challenges if this is default to true?

Ansible pulled out of those pages. Closing.


Installation doc tweaks

Made the following improvements:

Remove some extra whitespace.
The AWS instructions mention "Tag instance." as one of the steps, but do not give further instruction. Clarify that there is nothing specific for the administrator to do here.
Correct a filename to use the code markup.

Thanks for these proposals @robhaswell . This PR now has merge conflicts. I will close this and ask that it goes through the JIRA workflow, or resubmit when it can be merged and explain how you'd like it dealt with.


Integrating rustdoc and mdbook content

Integrating rustdoc and mdbook.....
Currently the include directive does not work for rustdoc comments.  There is an open issue in mdbook https://github.com/rust-lang/mdBook/issues/1356. there is an open issue in cargo https://github.com/rust-lang/cargo/issues/739.
This is how safer_ffi goes about things:
https://github.com/getditto/safer_ffi/blob/5d19ae7/scripts/build_docs.sh
This is how materialize do it:
https://github.com/MaterializeInc/materialize/blob/main/bin/doc
https://github.com/MaterializeInc/materialize/blob/main/misc/doc/crates.jq
What does Goose want to do?
Perhaps its worth waiting for https://github.com/rust-lang/rust/pull/78837
At which point we can share markdown files, by doing this in the source files?

Cross-referencing the documentation will certainly be handy, but is not essential at this time.
As the rustdoc is hosted at docs.rs, perhaps we can simply add valid links from the mdBook to the hosted rustdocs? (And once the mdBook is hosted in Github Pages, we can also include links back?) I understand it's preferable to be able to build and use all the documentation locally, but IMO until that's better supported it's likely not worth the headache.


Plotting "Observed vs. Predicted" or "Predicted vs. Observed"

PO (Predicted x, Observed y) is incorrect, and you should plot OP (Observed y, Predicted x), according to https://doi.org/10.1016/j.ecolmodel.2008.05.006

Thanks for commenting here - I agree, it is often reasonable to assume model has error and observations are correct. Depends on the situation.
In the case of "validating my new product against existing 'standards'", maybe assuming that the new product has errors is reasonable (although, of course, historical standards also have errors).


Laravel 5.7 Call to a member function getBody() on null

    "graham-campbell/digitalocean": "^5.1",
    "guzzlehttp/guzzle": "^6.3",
    "kriswallsmith/buzz": "^0.17.2"

when calling any function for example DigitalOcean::size()->getAll();
Issue with C:\xampp\htdocs\front\myadmin\vendor\toin0u\digitalocean-v2\src\Adapter\GuzzleHttpAdapter.php
Line 132:  $body = (string) $this->response->getBody();
Any ideas

Not sure. I think there is an issue tracking this on the upstream repo. If not, please create one.

We did, however, I updated my composer files since I created this ticket, there was your update there too, and that has solved the issue


turso db shell starts in a weird state (not ready to use)

Steps to reproduce:

Start a new terminal session;
Run turso db shell db-name;
Once it's connected, typing .quit for example won't work, one have to press enter first and only then the prompt becomes available;
Quit and run turso db shell db-name again;
Now pressing enter simply shows ^M -- if you delete it and press ctrl-D it'll kill the process BUT if you leave the ^M there and press ctrl-D then the prompt becomes available.

After pressing enter for a subsequent connection on the same terminal session

And then after pressing ctrl-D right after that

MacOS 13.2.1
iTerm2 3.4.19
turso version v0.55.0

I've never seen this behavior, and I always use the same general setup (macos/iterm).

How did you install the CLI?
Upgrade to CLI 0.55.1?
Which shell are you using (bash, zsh)?
Have you tried resetting all of your shell configuration temporarily to see if that might be triggering something?
Is there anything in addition that might be going on here that you haven't mentioned?

Now that I think about it, there was a brief period of time where shell was hanging consistently.  I think it was fixed in 0.55.1.  Glad you got unblocked!


Deserializer: Give client ownership of events

Disclaimer: I am new to Rust and maybe I misunderstand something.
Looking at the client libraries (yaml.rs in this project itself, de::Loader in serde-yaml), I realized that strings need to be cloned unnecessarily, as the EventReceiver interface does not pass ownership of the event.
Having implemented a small test client of serde-yaml myself, I realized that each string scalar was created three times on the heap: Once in yaml-rust, once in serde-yaml, and once in my client. If the EventReceiver passed ownership of the Event, one such allocation could be avoided.
I prepared an implementation myself to see if it works. If you want, I can prepare a pull request. But I would like to have your opinion on that before...

I agree, it should hand over the event by value. We would welcome a PR!

When opening this issue I assumed that this project was a push parser.
However, after separating parser and event emitter while implementing PR #75, I realized that it is in reality a pull parser that is encapsulated in a push parser. The latter only validates the state machine model of the former.
Now I am not sure about the underlying reasons, but in my understanding, pull parsers are much more client friendly. I think it would be easy to change the parser to a pull parser.  If there is some interest in that I could propose a pull request once I have a bit of time.


tuxedo-keyboard module isn't loaded after install

Describe the bug
After adding linuxPackages.tuxedo-keyboard to /etc/nixos/configuration.nix and restarting the kernel modules aren't loaded.
Steps To Reproduce
Steps to reproduce the behavior:

Add linuxPackages.tuxedo-keyboard to /etc/nixos/configuration.nix
nixos-rebuild switch
reboot

Expected behavior
I should be able to see the installed modules in lsmod, however none are loaded.
Screenshots
-
Additional context
$ uname -a
Linux nixos 5.10.88 #1-NixOS SMP Wed Dec 22 08:31:00 UTC 2021 x86_64 GNU/Linux

$ nix-store --query --requisites /run/current-system | cut -d- -f2- | sort | uniq | grep tuxedo
tuxedo-keyboard-5.10.88-3.0.8

$ nix-store -q --references /run/current-system/sw | grep tuxedo
/nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8

$ find /nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8
/nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8
/nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8/lib
/nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8/lib/modules
/nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8/lib/modules/5.10.88
/nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8/lib/modules/5.10.88/clevo_acpi.ko
/nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8/lib/modules/5.10.88/clevo_wmi.ko
/nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8/lib/modules/5.10.88/tuxedo_io.ko
/nix/store/hqpj454zzics2vvbywsg3x9w2270wzpp-tuxedo-keyboard-5.10.88-3.0.8/lib/modules/5.10.88/tuxedo_keyboard.ko

$ sudo modprobe tuxedo_keyboard
modprobe: FATAL: Module tuxedo_keyboard not found in directory /run/booted-system/kernel-modules/lib/modules/5.10.88

Notify maintainers
@blanky0230 (@blitz ?)
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.88, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.16`
 - channels(root): `"nixos-21.11.334814.04bd2d1a470"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Can you share your nixos configuration, specifically the bits regarding tuxedo keyboard and anything regarding the kernel?

Here's the entire thing
# Edit this configuration file to define what should be installed on
# your system.  Help is available in the configuration.nix(5) man page
# and in the NixOS manual (accessible by running ‘nixos-help’).

{ config, lib, pkgs, ... }:

{
  imports =
    [ # Include the results of the hardware scan.
      ./hardware-configuration.nix
    ];

  # Use the systemd-boot EFI boot loader.
  boot.loader.systemd-boot.enable = true;
  boot.loader.efi.canTouchEfiVariables = true;
  boot.loader.grub = {
    efiInstallAsRemovable = true;
    enableCryptodisk = true;
    useOSProber = true;
  };
  boot.loader.efi.efiSysMountPoint = "/boot/efi";
  boot.initrd.preFailCommands =
    "echo dev mapper\n" +
    "ls -l /dev/mapper\n" +
    "echo dev UUID\n" +
    "ls -l /dev/disk/by-uuid\n"
  ;

  # networking.hostName = "nixos"; # Define your hostname.
  # networking.wireless.enable = true;  # Enables wireless support via wpa_supplicant.
  networking.networkmanager.enable = true;

  # Set your time zone.
  time.timeZone = "Europe/Amsterdam";

  # The global useDHCP flag is deprecated, therefore explicitly set to false here.
  # Per-interface useDHCP will be mandatory in the future, so this generated config
  # replicates the default behaviour.
  networking.useDHCP = false;
  networking.interfaces.enp2s0.useDHCP = true;
  networking.interfaces.wlo1.useDHCP = true;

  # Configure network proxy if necessary
  # networking.proxy.default = "http://user:password@proxy:port/";
  # networking.proxy.noProxy = "<IP_ADDRESS>,localhost,internal.domain";

  # Select internationalisation properties.
  # i18n.defaultLocale = "en_US.UTF-8";
  # console = {
  #   font = "Lat2-Terminus16";
  #   keyMap = "us";
  # };

  # Enable the X11 windowing system.
  services.xserver.enable = true;

  # Enable the Plasma 5 Desktop Environment.
  services.xserver.displayManager.sddm.enable = true;
  services.xserver.displayManager.autoLogin.enable = true;
  services.xserver.displayManager.autoLogin.user = "michael";
  services.xserver.desktopManager.plasma5.enable = true;

  # Configure keymap in X11
  # services.xserver.layout = "us";
  # services.xserver.xkbOptions = "eurosign:e";

  # Enable CUPS to print documents.
  # services.printing.enable = true;

  # Enable sound.
  sound.enable = true;
  hardware.pulseaudio.enable = true;

  # Enable touchpad support (enabled default in most desktopManager).
  # services.xserver.libinput.enable = true;

  # Define a user account. Don't forget to set a password with ‘passwd’.
  users.users.michael = {
    isNormalUser = true;
    home = "/home/michael";
    extraGroups = [
        "wheel"
        "networkmanager"
        "docker"
    ]; # Enable ‘sudo’ for the user.
  };

  # List packages installed in system profile. To search, run:
  # $ nix search wget
  environment.systemPackages = with pkgs; [
    git
    vim # Do not forget to add an editor to edit configuration.nix! The Nano editor is also installed by default.
    wget
    firefox
    chromium
    kate
    keepassxc
    libsForQt5.plasma-nm
    libsForQt5.kwalletmanager
    linuxPackages.tuxedo-keyboard
    jetbrains.pycharm-professional
    element-desktop
    docker-compose
    syncthingtray
    xautomation
    xbindkeys
    xbindkeys-config
    # hardware packages
    lm_sensors
    linuxPackages.tuxedo-keyboard
  ];
  nixpkgs.config.allowUnfreePredicate = pkg: builtins.elem (lib.getName pkg) [
    "pycharm-professional"
  ];

  virtualisation.docker.enable = true;

  # Some programs need SUID wrappers, can be configured further or are
  # started in user sessions.
  # programs.mtr.enable = true;
  # programs.gnupg.agent = {
  #   enable = true;
  #   enableSSHSupport = true;
  # };

  # List services that you want to enable:

  # Enable the OpenSSH daemon.
  # services.openssh.enable = true;

  # Open ports in the firewall.
  # networking.firewall.allowedTCPPorts = [ ... ];
  # networking.firewall.allowedUDPPorts = [ ... ];
  # Or disable the firewall altogether.
  # networking.firewall.enable = false;

  # This value determines the NixOS release from which the default
  # settings for stateful data, like file locations and database versions
  # on your system were taken. It‘s perfectly fine and recommended to leave
  # this value at the release version of the first install of this system.
  # Before changing this value read the documentation for this option
  # (e.g. man configuration.nix or on https://nixos.org/nixos/options.html).
  system.stateVersion = "21.11"; # Did you read the comment?

}

and hardware-configuration
# Do not modify this file!  It was generated by ‘nixos-generate-config’
# and may be overwritten by future invocations.  Please make changes
# to /etc/nixos/configuration.nix instead.
{ config, lib, pkgs, modulesPath, ... }:

{
  imports =
    [ (modulesPath + "/installer/scan/not-detected.nix")
    ];

  boot.initrd.availableKernelModules = [ "nvme" "xhci_pci" "usb_storage" "sd_mod" "rtsx_pci_sdmmc" ];
  boot.initrd.kernelModules = [ ];
  boot.initrd.supportedFilesystems = [ "btrfs" ];
  boot.kernelModules = [ "kvm-amd" ];
  boot.extraModulePackages = [ ];

  # Root
  boot.initrd.luks.devices."cr_root".device = "/dev/disk/by-uuid/addf4189-1770-4575-ae52-4a46d51eebdf";
  fileSystems."/".device = "/dev/mapper/cr_root";
  #fileSystems."/".device = "/dev/disk/by-uuid/b862b21d-cf43-469f-95d7-ae51098bfb9f";

  fileSystems."/boot/efi/".device = "/dev/disk/by-uuid/6FB6-5F64" ;
  fileSystems."/boot/efi/".mountPoint = "/boot/efi";

  # Home
  boot.initrd.luks.devices."cr_home".device = "/dev/disk/by-uuid/69f8d66f-45d1-4047-8161-841dc704128a";
  fileSystems."/home".device = "/dev/mapper/cr_home";
  #fileSystems."/home".device = "/dev/disk/by-uuid/0175d2f8-aa4a-487c-bbc4-990624a59c1e";

  # Swap
  boot.initrd.luks.devices."cr_swap".device = "/dev/disk/by-uuid/863d9154-aeca-4e01-bc10-532cd2c59517";

  swapDevices = [
        { device = "/dev/mapper/cr_swap" ; }
        #{ device = "/dev/disk/by-uuid/ea99d871-f581-47ce-bb23-4065799cf803" ; }
  ];

}

Hi there :)
There's a module for the tuxedo-keyboard module as well. Did you try to enable that instead of adding the package?
add  hardware.tuxedo-keyboard.enable = true;  to your configuration.nix.

OK, I'll try that. Thanks for the quick responses.
Could you update https://github.com/NixOS/nixpkgs/blob/nixos-21.11/pkgs/os-specific/linux/tuxedo-keyboard/default.nix to mention the option either in a comment or the description?

hardware.tuxedo-keyboard.enable = true; without explicitly adding the package worked.
A comment in the nix file or (preferably) in the description would be great.
Thanks for the responses!

@MichaelAkvo We've updated the description: https://github.com/NixOS/nixpkgs/blob/master/pkgs/os-specific/linux/tuxedo-keyboard/default.nix#L28
Please re-open, if this is not what you were looking for.


'use strict' at the top of functions is removed by transpiler

exports = {
  includes(searchString, index) {
    'use strict';
    const str = this.toString(); // throws TypeError if this is null
    // ...
  }
}

transpiles to
string = {includes: function(searchString, index) {
  var str = this.toString(); // returns '[object Window]' if this is null
  // ...
}};

which, unfortunately, has different behavior.  We should take care to retain any 'use strict' in function declarations.

The compiler doesn't track strick mode per function. goog module and es6 modules are always strict. So in those cases it isn't necessary.

Relevant discussions:
https://code.google.com/p/closure-compiler/issues/detail?id=69
https://github.com/google/closure-compiler/issues/997

Gotcha - my output language was non-strict, so it was dropping the 'use strict' at the file level.  Setting the output language to ES5_STRICT fixes the issue.


Fix issue 260

Fix implicit conversion changes signedness and higher order bits are zeroes after implicit conversion issues. Most probably due to upgrade to Catalina.

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "fix-issue-260" git@github.com:amaumene/hyperkit.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@djs55 @rn ptal

looks like a possible duplicate of https://github.com/moby/hyperkit/pull/261

Thanks for your work on this and sorry for the delay. A fix was merged in #270 (along with some CI fixes which were unrelated)


Custom data config

Thanks for the great work! I am evaluating the dynamic object removal using our custom data by a Velodyne HDL 32 (the ground is around -2.2 meters in the LiDAR frame). But the results of dufomap and erasor (use default setting) seem not as good as the KITTI one in the example, can I know if any parameters I can tune? The data includes the pcd and the evaluation output is provided for reference. Thanks a lot!
The data: video:

The existing results,
Dufomap:
numerous vehicle cloud is not removed

missing data on the sign, and also some ground points

Thanks for sharing, I checked, it looks like you forgot to save the pose in the VIEWPOINT field: https://kth-rpl.github.io/DynamicMap_Benchmark/data/#a-header-example
And here is an example way to save:
https://github.com/Kin-Zhang/simple_ndt_slam/blob/e12840886641b364580f9b5820164b8b849d7899/tools/src/extract_pcdFbag_tf.cpp#L195-L203
      // set the viewpoint -> CHECK PCD VIEWPOINT FIELD
      pcl_cloud->sensor_origin_ = Eigen::Vector4f(pose[0], pose[1], pose[2], 0.0);
      pcl_cloud->sensor_orientation_ = Eigen::Quaternionf(pose[6], pose[3], pose[4], pose[5]); // w, x, y, z

      // save the pcd
      std::ostringstream tmp_filename;
      tmp_filename << save_pcd_folder << "/pcd/" << std::setfill('0') <<
      std::setw(6) << count << ".pcd"; std::string pcd_file =
      tmp_filename.str(); pcl::io::savePCDFileBinary(pcd_file, *pcl_cloud);

Thanks for your reply! I will re-implement the Pcd write part and evaluate again.

Dear Dr. Zhang,
Thanks for your reply. The results are impressive after I regenerate the pcd and put it into testing again using Dufomap. However, some areas especially on the opposite road are not good. Is it possible for you to provide some hits on any parameter I could further turn (default config of Dufomap is used)? The update data is provided for your reference. Thank you very much for your time!
The update PCD viewpoint using the SLAM pose,

The car near the sensor is removed

For the cars on the opposite road of the vehicle sensor platform has remained

I checked again and believe this is at the end of the frame, and since the DUFOMap strategy is to build the void(free) map, but the space you show and I pointed out, we don't have enough time/dense info to say that.
And this is one limitation we already mentioned in the paper Sec V-E:

Thanks for your reply!

No worry. Let me know if you have any further questions.

Dear Dr. Zhang, thanks again for your kind support. We will consider involving clustering or integrating with learning-based detection before the fine refinement based on your pipeline. The unexpected dynamic objects are quite challenging for large-scale mapping, especially in dense urban such as Hong Kong. Hopefully, we could have a chance to collaborate on dynamic object removal in the future.


Sync: Auth

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

develop

#4141   👈

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @malmstein and the rest of your teammates on  Graphite

Thanks for the review @cmonfortep . I’ve addressed the comments and push them, can you take a look?


Support IncludeFilter and walk RawInclude#filters

Current coverage is 55.17% (diff: 25.00%)

No coverage report found for master at dc4af9e.

Powered by Codecov. Last update dc4af9e...3cdb246


Upload multiple files as single POST request is not working??

Hi I have issue uploading multiple images with form text!!
Problem is that when I send for an example 3 images it duplicates my post text inside database because it POST's form 3 times unfortunatly function uploadMultiple isn't working as it should
uploadMultiple send all items in array as single POST REQUEST but it isn't working??
Dropzone in jQuery works fine it sends all 3 files to server in single POST REQUEST
Other Dropzone options like autoProcessQueue, parallelUploads ..etc work fine and Dropzone isn't sending anything to server until I press submit button
  // Disabling autoDiscover, otherwise Dropzone will try to attach twice.
  Dropzone.autoDiscover = false;
  // or disable for specific dropzone:
  // Dropzone.options.myDropzone = false;

  $(function() {
  // Now that the DOM is fully loaded, create the dropzone, and setup the
  // event listeners

    var myDropzone = new Dropzone('#drophere', {
        autoProcessQueue: false,
	uploadMultiple: true,
        parallelUploads: 100,
        maxFiles: 100,
    });

    $('#btnUpload').on('click', function () {
        myDropzone.processQueue();
    });

  })

When I replicate required options to my script multipleUpload doesn't work

<template>
  <div id="app">

    <input type="text" v-model="postContent" class="form-controll">

    <dropzone id="myVueDropzone"
              ref="createPost"
              url="api/posts"
              :useFontAwesome="true"
              v-on:vdropzone-success="showSuccess"
              v-bind:dropzone-options="dropzoneOptions"
              v-bind:use-custom-dropzone-options="true">
        <!-- Optional parameters if any! -->
        <input type="hidden" name="post" v-model="postContent">
    </dropzone>

    <button type="submit" class="btn btn-primary" @click="submitData()">

  </div>
</template>

<script>
  import Dropzone from 'vue2-dropzone'

  export default {
    name: 'MainApp',
    components: {
      Dropzone
    },
    data () {
      return {

        postContent: '',

        dropzoneOptions: {
          autoProcessQueue: false,
          uploadMultiple: true,
          parallelUploads: 100,
          maxFiles: 100,
        }

      }
    },
    methods: {
      'showSuccess': function (file) {
        console.log('A file was successfully uploaded')
      },
      'submitData': function () {
        this.$refs.createPost.processQueue()
      }
    }
  }
</script>

@atheleron It's strange! Surely look into it.

khbg jhn


2022.03.23  Edge AI _ dragon

2022.03.23  Edge AI _ dragon

Sorry for the late review 😢

Impressive as always! Stacking my AI knowledge thanks to you


Add check for isDestroying?

In my application I have a list of sorted items shown on a page, each of which as a "delete button" that corresponds to an action that called deleteRecord on the associated model item. This seems to work most of the time, but there appears to be a race condition wherein I can start to drag an item just after clicking the delete button. In that case, Ember throws an error when ember-sortable tries to set a property on said item. Am I doing something wrong here? It seems that this could be easily addressed by either wrapping these lines in a try/catch or first checking to see if this.get('isDestroying') is true.
Stack trace
ember.debug.js:17634 Error: Assertion Failed: calling set on destroyed object: <my-app@component:sortable-item::ember2743>.isDropping = [object Boolean]
    at assert (ember.debug.js:5700)
    at Object.assert (ember.debug.js:17372)
    at Object.set (ember.debug.js:22036)
    at Class.set (ember.debug.js:34722)
    at Class._complete (sortable-item.js:675)
    at Object.join (ember.debug.js:770)
    at Function.run.join (ember.debug.js:22289)
    at ember.debug.js:22352
    at tryCatch (ember.debug.js:58528)
    at invokeCallback (ember.debug.js:58543)

Relevant lines from sortable-item.js
/**
  @method _complete
  @private
*/
_complete: function _complete() {
  (0, _emberInvokeAction.invokeAction)(this, 'onDragStop', this.get('model'));
  this.set('isDropping', false);
  this.set('wasDropped', true);
  this._tellGroup('commit');
}

Hmm....this is definitely an interesting problem - I'll definitely look into this further 😄

Thanks. For now I've moved my "drag handles" further away from the button so that it is at least harder to do.

I'm running into a similar error. I'm using a computed property which is used to build the sortable items that I loop over with each. It depends on an array from my model, which I update when an item is moved, resulting in the computed property to be recomputed and update the view.
When I drag an item I will usually get the error. In my case it's coming from the _drag function when setting y. Occasionally it will not throw the error.
Assertion Failed: calling set on destroyed object: <app@component:sortable-item::ember2447>.y = 226


4.0.1 dropped support for Node 18

peggy itself requires Node >=18, but 4.0.1 added a dependency on @peggyjs/from-mem in https://github.com/peggyjs/peggy/pull/480, which requires Node >=20.8, effectively introducing a breaking change.

Nod, will fix.  Parts of from-mem require 20.8, but not all of it.  It does a runtime check and throws an error, so I think it's safe to change the min version on from-mem back to 18.

See https://github.com/peggyjs/from-mem/pull/8

Fixed in 4.0.2


Add metaparameters to simtools

At the moment we do not define any metaparameters in simtools. They are really useful and absolutely necessary, so it should be added.

Could you point towards the documentation of the metaparameter? Are they used later in ctapipe or why are they needed?


Recipes for camel-quarkus

Descripition:
PR introduces a recipe - org.apache.camel.quarkus.update.CamelQuarkusMigrationRecipe - which takes care of the whole camel-quarkus migration process. This main recipe registeres all other recipes via the constructor and doNext method. If the migrated project contains any camel-quarkus dependency, the child recipes are executed. There are 4 types of camel recipes so far:
Java - recipes migrating java classes. Abstract parent of those recipes detects existence of camel-quarkus import and does nothing if there is no such import
Pom - maven recipes taking care of removed extensions (comments helping users to find replacement are created if possible)
Properties - simple recipe
Yaml - some transformation for YAML DSL feature from Camel
All migration cases are covered by JUnit tests.
Important features:

There are 2 performance optimizations implemented. None of the CQ recipe migrates the code if there is no CQ dependency in the pom. Java recipes are skipped, if the is no import of ‘org.apache.camel’ in the class.
We upgraded java in this project to have access to String blocks, which are heavily used in the tests.
PR currently does not contain gradle recipes
All recipes are grouped together into ‘CamelQuarkusMigrationRecipe`
Support of maven surefire plugin was added and the test can be executed via mvn test

Opened questions:

How to cover the upgrade process for more versions with the Java recipes avoiding compilation conflicts? (Example from the readme.md: Recipes applied for a project in version 2.7.0.Final updating to 3.1.0.Final (currentVersion=2.7, targetVersion=3.1):2.9.yaml, 3alpha.yaml and 3.1.yaml. Java recipes used in 3alpha.yaml requires camel dependencies 3.18.7 for both tests (and ideally also recipes). But java recipes from 3.1.yaml requires Camel dependencies 4.x. Having both versions 3.x and 4.x on the classpath is not possible.
Which java version should be used by this project? Because j15 brings String blocks, which are very helpful in writing junit tests, we upgraded Java. Would this be acceptable?
Is it possible to upgrade/use a more recent version of openrewrite? while developing a new recipe we realized it would be nice if we can use the most recent version since it provides many new features which will always be helpful.

Limitations:

PR does not contain any Gradle recipes.

What do you think @gsmet , @maxandersen, @ia3andy

How to cover the upgrade process for more versions with the Java recipes avoiding compilation conflicts? (Example from the readme.md: Recipes applied for a project in version 2.7.0.Final updating to 3.1.0.Final (currentVersion=2.7, targetVersion=3.1):2.9.yaml, 3alpha.yaml and 3.1.yaml. Java recipes used in 3alpha.yaml requires camel dependencies 3.18.7 for both tests (and ideally also recipes). But java recipes from 3.1.yaml requires Camel dependencies 4.x. Having both versions 3.x and 4.x on the classpath is not possible.

My understanding is that your problem is with tests? If so, I think you should have different projects that you upgrade in separate modules.
TBH, we haven't given more thoughts about the testing of the updates. I think ideally, we should have source projects, and a target project, that we could regenerate. @ia3andy did something like that for the codestarts and it's working pretty well.

Which java version should be used by this project? Because j15 brings String blocks, which are very helpful in writing junit tests, we upgraded Java. Would this be acceptable?

As mentioned in the review, maybe let's separate the projects? Also let's make sure the release target of the update jar is 11?

Is it possible to upgrade/use a more recent version of openrewrite? while developing a new recipe we realized it would be nice if we can use the most recent version since it provides many new features which will always be helpful.

Unfortunately not. They removed a feature that is critical to us. We discussed with the OpenRewrite team and we hope at some point they will reintroduce it.
We might put in place some workaround at some point but it would be a lot less practical that what we're doing right now.

Hi @gsmet ,
before there is a correct solution using recipe/test separation similarly to codestarts. would it be acceptable following temporary solution?

quarkus-updates (this PR) won't contain any camel related tests and would use J11 (if possible I'd like to keep a simple test verifying that no recipe is executed if there is no camel dependency)
new project (ideally in camel github repo) would be introduced. This project would depend on quarkus-upgrades and would contain all tests (using j17 and camel dependencies)

Once the (final/general) separation is prepared, the tests from the temporary project would be refactored to use the final solution. This approach would allow to keep recipes + coverage even before the separation is merged (which could take some time, from the glance at the code in codestarts)

@gsmet we fixed all  suggestions as asked. Currently the majority of tests is removed from thisPR

tests will be placed in a different project as a temporary solution (until proper split between recipes & tests is proposed - which may take some time)
the only existing tests takes verifies, that a recipe is not executed in a project with no camel-quarkus dependency.
Java is kept on version 11. No camel dependencies are present in the pom.xml.

Is this solution ok?

Result of the discussion on upstream chat: https://quarkusio.zulipchat.com/#narrow/stream/187030-users/topic/Quarkus.20update.20tool.20.28non-core.29
All tests covering camel-quarkus migration recipes (and skipped execution in non-camel-quarkus projects) are located in recipes-unit-tests module, which uses Java 17.
I see a limitation of this solution:
• current separation does not allow to have a different dependencies for a different stream of migration (which could be solved by separation into test modules for each stream if needed)
I think that PR could be merged (and squashed into 1 commit - I kept separated commits to make reviewing easier).
@ia3andy FYI

Result of the discussion on upstream chat: https://quarkusio.zulipchat.com/#narrow/stream/187030-users/topic/Quarkus.20update.20tool.20.28non-core.29
All tests covering camel-quarkus migration recipes (and skipped execution in non-camel-quarkus projects) are located in recipes-unit-tests module, which uses Java 17.
I see a limitation of this solution:
• current separation does not allow to have a different dependencies for a different stream of migration (which could be solved by separation into test modules for each stream if needed)
I think that PR could be merged (and squashed into 1 commit - I kept separated commits to make reviewing easier)
@ia3andy FYI

@JiriOndrusek since we might have other extensions contributing java recipe, it would make sense to have a common package naming. something like io.quarkus.updates.camel .

I agree, I'll refactor the PR

I refactored the PR to use a package name io.quarkus.updates.camel30  (I added suffix 30, as the same is used in the quarkus/core30 and it clearly shows, that the recipe is for vesion 3.0

@ia3andy thanks for the hints above. Now it is all fixed.

@ia3andy thanks for the hints above. Now it is all fixed.

I still see package org.apache.camel.quarkus.update; maybe you missed a commit?

@ia3andy can you please take a look now? Thanks

@gsmet could you check? I still see the org.apache packages

@ia3andy I think we are good now, could you please take a look ? thanks!

Thanks @svkcemk for the help! I was on PTO.
I checked the current status of the PR and there is no package org.apache.camel.quarkus.update.
(I thought that I fixed it 2 weeks ago, I might indeed skipped some commits.)
@ia3andy I'm sorry for spending so much time on this PR, but it should be OK now.

Thanks @svkcemk for the help! I was on PTO. I checked the current status of the PR and there is no package org.apache.camel.quarkus.update. (I thought that I fixed it 2 weeks ago, I might indeed skipped some commits.) @ia3andy I'm sorry for spending so much time on this PR, but it should be OK now.

We are waiting for a change in Quarkus core to allow having yaml extension recipes. @svkcemk is on it.

https://github.com/quarkusio/quarkus/pull/35688 is the dependent PR

@ia3andy, @maxandersen  I updated (and tested locally) the PR to reflect changes from https://github.com/quarkusio/quarkus/pull/35910.
All succeeded, therefore this PR can be merged.

Problem is fixed.
the only work to be done is to replace simple java recipes with the yaml ones

@JiriOndrusek @svkcemk You did an amazing job in all this, thanks for your patience, it was a long journey, but I think having a way standardized way for extensions to provide their recipes is awesome!

@ia3andy I update PR to reference a yaml recipe from the test.
As I thought, all the migrations contains some kind of logic a condition which can not be simulated by easy yaml recipes. Therefore I can not convert any java recipe to yaml.
Here is a list with the majority of upgrades + explanation why it can not be done

Removed label on @UriEndpoint as you should use category instead.
annotation attribute value replced with the constant

Removed all asyncCallback methods on ProducerTemplate. Use asyncSend or asyncRequest instead.
addedcomment with suggestion

Removed org.apache.camel.spi.OnCamelContextStart. Use org.apache.camel.spi.OnCamelContextStarting instead.
probably not, need to add new interface and remove old one

Removed org.apache.camel.spi.OnCamelContextStop. Use org.apache.camel.spi.OnCamelContextStopping instead.
probably not, need to add new interface and remove old one

Decoupled the org.apache.camel.ExtendedCamelContext from the org.apache.camel.CamelContext.
complicated replaces of context calls, with comment exlanation

Replaced adapt() from org.apache.camel.CamelContext with getCamelContextExtension
change of '  context.adapt(ModelCamelContext.class)->((ModelCamelContext)context).' with some corner cases

Decoupled the org.apache.camel.ExtendedExchange from the org.apache.camel.Exchange.
replace + type cast of (exchange.adapt(ExtendedExchange.class) -> exchange.getExchangeExtension())

Replaced adapt() from org.apache.camel.ExtendedExchange with getExchangeExtension
replace of methods + with several corner cases

Exchange failure handling status has moved from being a property defined as ExchangePropertyKey.FAILURE_HANDLED to a member of the ExtendedExchange, accessible via isFailureHandled()method.
continuation of above case

Removed Discard and DiscardOldest from org.apache.camel.util.concurrent.ThreadPoolRejectedPolicy.
suggestion in comment

Removed org.apache.camel.builder.SimpleBuilder. Was mostly used internally in Camel with the Java DSL in some situations.
suggestion in comment

Moved org.apache.camel.support.IntrospectionSupport to camel-core-engine for internal use only. End users should use org.apache.camel.spi.BeanInspection instead.
change of impor, BUT not a package, only one class

Removed archetypeCatalogAsXml method from org.apache.camel.catalog.CamelCatalog.
suggestion in comment

The method configure from the interface org.apache.camel.main.Listener was removed
suggestion in comment

The type for dumpRoutes on CamelContext has changed from boolean to String to allow specifying either xml or yaml.
change of the parameter type, comment with explanatiom

Hi @ia3andy  I refactored 3 places to use yaml instead of java recipes (properties, poms & change oftype)
For the rest of the recipes it wasn't possible to use yaml, because their functionality contains handing of the parameters or more complicated stuff. There was 1 place which should be pissible to refactor, but the recipe registered in the yaml didn't work (I spent several hours to make it work), so I added a comment with TODO to this line.
From my POV the one jave recipe with todo should be reported in here (or camel-quarkus) fir further investigation, I see an option, that the recipe is not meant for the use in yaml. As it is not documented and the code of the recipe looks differently then the one which works (changeType - works vs changePackage - does not work)


Basic working example for calling via grpc proto client.

Hi, It will be extremely helpful If one can provide a very basic example of consuming weedfs proto services using grpc client.

this is a good standalone example https://github.com/chrislusf/seaweedfs/blob/master/unmaintained/diff_volume_servers/diff_volume_servers.go


next: SSO get config handler

In SSO plugin, sso/config

In SSO plugin, sso/config is defined as
def register_config_lambda(allowed_callback_urls):
    @skygear.op('sso/config')
    def sso_config_lambda(**kwargs):
        return {
            'authorized_urls': allowed_callback_urls
        }

Should we change it to sso/allowed_callback_urls?

We used a generic name sso/config for this endpoint, it is because we want to give room for extension in future. We can put more information into this api, but currently the only config that we have are allowed_callback_urls.

@carmenlau @Steven-Chan Note that SSO handlers response is in following format: {'result': '...'} to align other auth handlers. Not {'authorized_urls': '...'} or {'auth_url': '....'}.


[BUG] Draw a bitmap decoded from SKCodec performance is poor on UWP

Description

Draw a bitmap decoded from SKCodec performance is poor on UWP, it is much slower than .net core and .net framework.
Code

var file = await StorageFile.GetFileFromApplicationUriAsync(new Uri("ms-appx:///test.png"));
using (var stream = await file.OpenStreamForReadAsync())
{
    using (var bitmap = new SKBitmap(960, 720))
    {
        using (var canvas = new SKCanvas(bitmap))
        {
            using (var codec = SKCodec.Create(stream))
            {
                using (var decodeBitmap = new SKBitmap(codec.Info))
                {
                    var pointer = decodeBitmap.GetPixels();
                    codec.GetPixels(decodeBitmap.Info, pointer);

                    var dest = new SKRect(0, 0, 960, 720);

                    var stopwatch = new Stopwatch();
                    stopwatch.Start();

                    canvas.DrawBitmap(decodeBitmap, dest);

                    stopwatch.Stop();

                    await new MessageDialog($"cost {stopwatch.ElapsedMilliseconds} ms").ShowAsync();
                }
            }
        }
    }
}

Expected Behavior

The performance should be equals to run on .net core or .net framework.
Actual Behavior

It cost 200~300 ms on my computer and the .net core and .net framework version, only takes <10 ms.
Basic Information

Version with issue:  2.80.2
Last known good version:  2.80.2
IDE:  Visual Studio
Platform Target Frameworks:

UWP:  18362
Windows Classic:  Windows 10

Reproduction Link

https://github.com/h82258652/SKCodecUwpPerformanceRepo

I found an interesting thing. I download the 2.80.2 nuget package, extra it and then let the UWP project reference the .net standard 2.0 dll (also copy the libSkiaSharp.dll), the performance equals to run on .net core or .net framework.


Fixed #250 - unignore files feature

Coverage decreased (-0.3%) to 52.756% when pulling f95acff78fb54731aeedc53a617194ee31fd5971 on 250-unignore-files into 2776206b13a1345a62fa48eef63b5bb71fdad53e on master.


Environment variable is not correctly selected after updating to v2020.3.0

Describe the bug
When trying to enter an environment variable the variable has an error and it has to be manually updated.
To Reproduce

Try to enter a variable in the URL
Press (Control + Space bar) (mac)
Press Enter or select from the dropdown list
See error

Expected behavior
The variable to be added without need to manually fix it.
Screenshots

Desktop (please complete the following information):

OS: macOS 10.15.4
Installation Method: update and also download and replace
App Version 2020.3.0

I'm getting this on Ubuntu 20.04 as well

They also no longer work in the documentation!
Variables inserted with the previous versions are displayed, but can no longer be added.
The variable window opens but when you select it, the corsor jumps up. All entries of the filter in the window appear in the documentation field.

Same here. I have to type {{base_url}} in order to make it work.

Related to #2377

Reopening this, as it is not related to #2377. We're looking into a fix at the moment.

Thanks for that. I tested it in version 2020.3.3  and it works like a charm 😄

The problem with the variables in the documentation has been resolved. thank you

Am 11.07.2020 um 00:53 schrieb Fransy<EMAIL_ADDRESS>Thanks for that. I tested it in version 2020.3.3 and it works like a charm 😄
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/Kong/insomnia/issues/2378#issuecomment-656928649, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYOHWPUNBP7D7MND62MMO3R26LV5ANCNFSM4OWOY7WA.


Social Media logins not functional yet

I tested facebook login it doesn't seem to be working. It was working at one point. Jim and Bryan were having issues with it.

Facebook, Google+, and LinkedIn working

Disagree.
This is the issue I get with Facebook:

I'll test it out

I'm also seeing that result on both:
http://kubernetes-master.cloudapp.net/
and
http://www.ardentmc18f.com/

LinkedIn appears to be working on both sites.

Google+ is working on Kubernetes-master, but not ardentmc18f

Try clearing your cache, it works...
https://www.youtube.com/watch?v=n31GBOJU-Ng&feature=youtu.be

It didn't work for me. Sorry.

Nor for me.
Remember that we're not developing this for ourselves, but for users and evaluators whom we won't require to refresh their cache.
At tho point, I think we should consider scrapping tho capability unless we think we very close to a real solution that doesn't require a workaround.
Don't take it personally. We just have a working solution with custom logins and I think I'd rather see better design implemented and a more personalized experience  (ie, alerts based on foods user cares about).
Jim
Sent from my Windows Phone

From: Elise<EMAIL_ADDRESS>Sent: 6/26/2015 9:45 PM
To<EMAIL_ADDRESS>Cc: Jim<EMAIL_ADDRESS>Subject: Re: [openFDA-Prototype] Social Media logins not functional yet (#32)
It didn't work for me. Sorry.

Reply to this email directly or view it on GitHub:
https://github.com/ArdentMC/openFDA-Prototype/issues/32#issuecomment-115934999

So we definitely did have a mongo problem, I can confirm. The new configuration should fix the google+ social media problem.
I can confirm that google+ and linkedin are now working for me. Maybe we only need to off facebook?

Approval from Facebook still pending.  If not received by tomorrow I will remove Facebook link.

also add redirects for noshalert.com

Facebook approval received and noshalert.com has been added for all 3 social media login options.  Other users please verify before closing.

This is working just waiting to bring back the icons and test them out that way and then we can close this issue for good

Tested this morning again with the new application and its working I am closing this out.


numbered bullets appear as circle

Hi,
Numbered bullets and footnotes appear as circles in this theme. I tried messing around with the post.css but it did not help. Please can you let me know how can I make the theme to differentiate between un-ordered and ordered lists.
eg:

AAA
BBB
CCC

is displayed as

AAA
BBB
CCC

Hey @pubmania ,
Thanks for letting me know - I'll take a look tonight and see if I can sort it out :)
Kate

Thanks :)

Hi just wondering if you had a chance to look at this and if you could guide me on how I can resolve this problem.

Hi @pubmania
Sorry about the delay - this issue should be all fixed now :)
Kate

Thanks. I can confirm that the issue is now fixed.


add orders endpoint to update shipment status

This PR adds the method to update_shipment_status for Amazon orders.

Hi Lars, can you please add tests. Thank you!


Mouses needs a proper burning sprite

sounds like a fun way to learn how to do things, let me see what i can do

It's done, I'll do some tests and submit a pr once I'm off work

Fixed by #26759


Implement scrollback on DMs

There is currently no way to get scrollback on DMs. Can this be implemented? Thanks!

@hloeung is working on this!

Ready to go! https://github.com/42wim/matterircd/pull/388


Fix the Relay Transform visit order

This is intended to fix an issue in Relay transform if the following use case ever happens:

the upstream pulse.ADD contains {_id: 1}, which is derived inside Relay as {_id: 11} to the out.add, and cached as lut[1] = {_id: 11}
later, the upstream sends in {_id: 1} inside both pulse.ADD and pulse.REM.

In the current implementation, the pulse.ADD is visited first, which creates a new derive, {_id: 21} to out.add, and overwrites lut[1] = {_id: 21}. Thereafter the pulse.REM is visited, which push the newly derived {_id: 21} into the out.rem as well. The previously derived {_id: 11} got lost in the process.
The expected behavior should be visiting the pulse.REM first, which push {_id: 11} into out.rem, and then visit the pulse.ADD, which push {_id: 21} into out.add.
This is somehow similar to an earlier PR 20. As a general principle, it seems to be safe to always visit REM before ADD within the same transform.

Thanks for another valuable bug fix!


Migrate 'FeatureSelectionWidgetUI' component from makeStyles to styled-component

Description
This PR includes a refactorization of RangeWidgetUI component, all classes from makestyles are been converted to styled components
Shortcut: 297037
Type of change

Refactor

Acceptance
The code resultant must returns same styles component visualization as the same before changes

Overview a general visualization from v1.x to v2.x

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 71.351%

Totals

Change from base Build<PHONE_NUMBER>:
0.007%

Covered Lines:
1950

Relevant Lines:
2514

💛  - Coveralls


shim/rollkit: go-da via gRPC

This changeset migrates the rollkit demo to go-da, the more modern version
of the DA interface for Rollkit.
One of the most important changes is that rollkit doesn't embed the adapters anymore.
Instead, it communicates with the adapter through gRPC protocol. Instead of yet another
adapter, this implements gRPC dock directly, which kind of challenges the current
architecture.
Notable changes:

demo/rollkit was regenerated from the tutorial once again. That turned out to be
easier than manually porting changes. I ported init-local.sh though.
removed adapter/sugondat since it is not longer needed
since it looks like we are going away from the docker workflow, I did not port
the Dockerfile to the new code. I left the docker-compose.yml though, just removed
the gm service.
the JSON-RPC was pushed to the sovereign dock. Now serve subcommand will launch
only the specified dock instead of all of them. Each dock has its own subcommand.
ikura-shim serve rollkit is now used for running rollkit.
now dock subcommands get their own options. E.g. rollkit doc now can accept the default
namespace in case requests from rollkit without namespace (they do atm).

There are some things that were left for follow ups:

https://github.com/thrumdev/blobs/issues/258
https://github.com/thrumdev/blobs/issues/259

The CI is flaky.


Run user seeds to PRNGs through SHA256 to unbias

This addresses #1170 by running user input to the PRNG seed function through SHA256. As far as I can tell this is at least .. plausibly recommended practice (it's hard to find discussion of "how to use ChaCha correctly as a PRNG" that is not primarily concerned with applications involving secrets and therefore strong entropy).
I would greatly appreciate input from @kwantam or someone who has better knowledge about CSPRNGs or, specifically, properties-of-ChaCha than myself. In particular I'd like to know how close to correct this comment I just added is.
Fixes #1170

(Perhaps @nano-o knows something about this subject?)


feat(ActionIconGroup): support size prop

💻 变更类型 | Change Type

[x] ✨ feat
[ ] 🐛 fix
[ ] ♻ refactor
[ ] 💄 style
[ ] 🔨 chore
[ ] 📝 docs

🔀 变更说明 | Description of Change
feat(ActionIconGroup): support size prop

📝 补充信息 | Additional Information


Update x/exp to newest version

This also upgraded x/tools.
The new x/exp SortFunc returns an int instead of a bool.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


client.positions() returns incomplete data

to be clear, this is not a defect of iBind
client.positions() uses the API call /portfolio/{accountId}/positions/{pageId}
client.positions() does not always return all data fields. Instead of the expected 47 fields, it sometimes returns only 41 or 21 fields.
Sometimes means: it can be incomplete for a sequence of 10 calls, or only once in 20 calls. There is no visible pattern.
Although it is not a defect of iBind, you may be still impacted by this behaviour, depending on what data fields you plan to use.
You cannot assume all data is delivered but you need to have checks in place.
I will observe this for some time and likely create a ticket at IB. If somebody else has observed this as well or found a logic or reason, please comment here.

That's true, it can happen sometimes, I've noticed it too. There's a solution I'm thinking about but not sure if I can share it here publicly. Can you drop me a quick email? You'll find it in the README of this repo.

ok, thanks. Let's see if IB provides a stability fix or a better solution in the future.


Streaming validation

Opening a PR for @quadrophobiac's work

The streaming validator is typified by the specs added in context "with a single row"and context context "validation with multiple lines: "
Both of these spec contexts test various ways that the Validator_Stream object can be initialised and then it's various methods can be used to validate a CSV
the def validate method has been preserved in the interest of creating a validator that would be backwards compatible with the old code base, see line 43
Alternatively as the specs demonstrate each of the validate_metadata report_line_breaks and parse_contents methods can be invoked independently to generate errors and warnings
Both parse_contents and validate_metadata can be invoked with enumerable behaviour which was the aim with a method that could take a CSV line by line and report errors
see streaming_validator_spec.rb:130 and streaming_validator_spec.rb:157

Pretty awesome ! It this included this feature ? https://github.com/theodi/csvlint.rb/issues/143

Yup, that's the plan!

@pezholio Awesome ! Do we have an ETA of when this will be merged in master and the gem version updated ?

I'm working on it as we speak. Hoping to get it done for the end of the week :+1:

@pezholio Awesome ! Can't wait.

These tests are now failing on a single feature features/validation_errors.feature:126 - error of :stray_quote returned instead of :line_breaks
This hinges on logic
      if !@csv_options[:row_sep].kind_of?(Symbol) && type == :unclosed_quote && !@input.match(@csv_options[:row_sep])
        build_linebreak_error

within validate.rb https://github.com/theodi/csvlint.rb/blob/19dc4244dabd90aa4789f08be42d3edd59546668/lib/csvlint/validate.rb#L290
In the case of the feature this fails because type is reported as unclosed_quote
changes to this logic (to recognise :stray_quote) will adversely affect https://github.com/theodi/csvlint.rb/blob/19dc4244dabd90aa4789f08be42d3edd59546668/spec/validator_spec.rb#L248
The string which the exception builder receives is "\"Foo\",\"Bar\",\"Baz\"\r\"Biff\",\"Baff\",\"Boff\"\r\"Qux\",\"Teaspoon\",\"Doge\"" - no obviously unclosed quote

\o/ :+1: +:100:


Java client

Gonna close this one out given the staleness. We'll probably need Java support at some point in the future and this can be the reference PR.


Use copy_nonoverlapping instead of slice::copy_from

In doing so, we eliminate a potential panic path. Although I was not able to observe panic paths emitted in toy examples, they might be emitted in complex examples in which the optimizer is low on gas. Regardless, I expect this change will ease our future adoption of call-graph analysis techniques of potential panic paths.
Ref https://github.com/google/zerocopy/issues/200#issuecomment-2181544061

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@e12c1e9). Learn more about missing BASE report.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1448   +/-   ##
=======================================
  Coverage        ?   87.46%
=======================================
  Files           ?       15
  Lines           ?     5170
  Branches        ?        0
=======================================
  Hits            ?     4522
  Misses          ?      648
  Partials        ?        0

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


*: Update manifests, scripts and docs to use 4.3

Updates all manifests and images to reference 4.3
Closes #952
/hold Waiting to make sure that we no longer need the 4.2 manifest bundles in the repo

/retest

Need to wait for https://github.com/openshift/release/pull/5205 to merge cause this uses 4.3 image tags in quay that aren't yet mirrored yet.

/retest

/retest


Bump to version 0.0.46

Hmm...
bower patternfly                                       resolution Unsuitable resolution declared for patternfly: ~3.26.1
bower                                                   ECONFLICT Unable to find suitable version for patternfly
make: *** [install] Error 1
The command "make build" failed and exited with 2 during .

?


Guide users after account creation

The new account creation functionality in the Lab is great. When users reach the end of that flow, hopefully they'll continue to explore the Lab's other features, like building a transaction. I'd love to see a CTA button appear after the account is funded with something like "Build a transaction" (or whatever we want them to do after funding the account). What do folks think? @mozzadrella maybe this will be impacted by user testing, and maybe we can figure out what the logical next step would be from that. cc @irisli

As we add tutorials, I think we should be careful about turning this tool into a tutorial.
I'm fine with adding tutorials specifically about the lab, but I'm wary about adding tutorials that are primarily about learning the capabilities of the stellar network:  I think doing so will add friction to using the tool and we would be better served through a more dedicated developer tutorial.
This distinction I'm trying to express is subtle, so let me try to explain further with some examples.  Let me define two groups:  tutorials about the lab (good), and tutorials about stellar (bad).
good:  "learn how to craft transactions using the transaction builder"
bad: "learn what transactions are and how sending them causes changes to the ledger"
good: "learn how to add signatures to a transaction envelope using the signer tool"
bad: "learn about private keys"
good: "learn how to use the endpoint explorer to load your account's current details"
bad: "learn how signature thresholds work"
As you can hopefully see, the "good" tutorials are all centered on learning something specific about how the lab operates and how you might use it to help your stellar development experience, rather than getting to know some feature of the stellar network. Note: I'm fine with linking off to additional resources to help teach people about our network's capability, I am simply wary about bogging down the tool with too much narrative or other guided experiences.

I'd also like to point out some early motivations we agreed to around the creation of the laboratory.  Specifically, we decided that the laboratory is first a tool to help developers that are integrating with the stellar network and secondly it is a way to teach developers about the capabilities of the network through exploration.  The lab isn't meant to be a single experience to be completed(i.e. a tutorial), but rather something that a stellar developer will use daily...  at this point I do and it is a huge boon.  I would be sad if we compromise its utility to improve our tutorials.

All of that said, I don't have any specific issue with the addition being proposed in this issue.

I agree that we should be careful about adding too much tutorial pieces to generic tools. It adds to the thought load and can distract users following a tutorial that goes a different direction as the build in tutorial in the laboratory.

The account creator has the ability to create accounts and the next step could be many different things. By adding a next steps item to a generic tool, it can mislead those following another tutorial that does not follow the same route as the one we build into the tool.

Closing this as it's very old. This is something we'll keep in mind as we plan and implement a larger overhaul at some point in the future.


Fix 1.2 ci log collection

Coverage remained the same at 33.484% when pulling 29f21df687d0cea9a930deb881d709ab12208258 on jmontleon:fix-1.2-ci-log-collection into 7b1c035ab71d4553c0e909513ba89a8e43538017 on openshift:release-1.2.


Type checking

Add type checking (e.g. condition of while must be a boolean)

This additionally includes scope resolution (does this name refer to a global or a local or a builtin function, etc)


[Feature Request] Make link insensitive to chapters in the link

Threw an ObjectNotFound; CommandNotFoundException when i included the chapter part after the link, worked without.

fixed in https://github.com/Aivean/royalroad-downloader/releases/tag/v2.6.1


[drift_postgres] Postgres implicit type casts not working in raw queries

Similar to #2981 this explicit casting can break valid Postgres queries.
For instance, passing a String to insert a INT8 column is valid in the postgres driver, but when forcing it to be always Type.text it will complain.
Should we let the values go through to the postgres package with unspecified typing?

I think this is working as intended for explicit variables added by drift. For others stumbling across this (e.g. for custom queries), a workaround is to use variables with a TypedValue from the postgres package that has its type set to Type.undefined. These values won't be modified by drift at all and can be used to signal that no explicit type is known.
(if it turns out that drift would benefit from not using explicit types as well, I'm happy to revisit this or perhaps make this an opt-in flag)


Feature/2620/improve the navbar

Targets #2620
Description
Improved the nav dropdown UI
Checklist:

[ ] I have performed a self-review of my own code.

Plus the CI checks are failing

Please fix the CI checks as well?


Cannot archive project

Hi,
Please help me out, I use your lib in my project, when I build directly to my device, it works fine. Then I archive project, it show these errors.
Please check and tell me soon.
Thank you very much

Ya, I still don't know why, but I removed it from cocoapod, get swift file and add manually in my project.
It worked.

hello @gauuni . i just do removing from cocoapods and add it manually, but the error is still occured (still same as your screnshoot "apple link mach-o")
can you help me how you import it manually. here is my config. thanks. hope you help me please..

What did you add in your project? I only added SlideMenuController.swift file, and didn't add anything to embedded binaries

i add entire repo in my project like the alamofire manual step said. hehehe.
but if you only add just a single file "SlideMenuController.swift" file, how can you implement "import SlideMenuControllerSwift". it will asking for references and can by done by importing "SlideMenuControllerSwift.framework.

When you add it manually, needn't "import SlideMenuControllerSwift". Now it's just a swift class, not framework anymore. Do you know that?

Emm. so we work with class only. not as framework. okay, I 'll try. I thinks i am understand. thanks @gauuni  :)

You're welcome. Hope can help you ressolve your problem

thanks so much @gauuni . my problem is solved :)

@ahsyarif008 I added whole repo to my project as well but still cannot get it to compile for archive

@ahsyarif008 please help

Solved ✔✔


Great library. Just one little comment

Thanks for the great library. Enjoy it a lot. Just one minor thing, in the example,
title={"DialogInput 1"}
can be simply as:
title="DialogInput 1"

Is this better in some other way than eye clear syntax ? Asking if I am missing something ... nothing else.. thanks

Just eye clearing. haha


Clarify how to integrate ranger into the OrbitAI app

This story is part of epic Evaluate the ranger library for batch training with Random Forest.
This story is a prerequisite to Integrate ranger into the OrbitAI app.

Discussion Points

We can keep it super simple and just make it a binary that is triggered on demand via an NMF app action/command.
The binary would process the training data csv file.
We need to define what the interface is between the NMF app and the binary to allow for n-dimension training.
We would run the binary after manually checking that the training data log file has diverse enough data (i.e. for both 0 and 1 labels).
We should have a flag in the .properties file to indicate whether the app should log training data into a csv file. No need to persist data if we don't intended to use it.
We need to think of how we implement inference mode.

We can keep it super simple and just make it a binary that is triggered on demand.
The binary would process the the training data log file.
We would run the binary after checking that the training data log file has diverse enough data (i.e. for both 0 and 1 labels).

@TanguySoto:

For training: https://github.com/georgeslabreche/opssat-orbitai/issues/51
For prediction: https://github.com/georgeslabreche/opssat-orbitai/issues/52

Let's discuss so we can close this issue?

@TanguySoto refer to the new properties file:
https://github.com/georgeslabreche/opssat-orbitai/blob/main/nmf/space-app/conf/orbit_ai_v2.properties


feat(kit): new component ProgressCircle

PR Checklist
Please check if your PR fulfills the following requirements:

[X] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[X] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?

Closes #
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[X] No

Other information

@waterplea
I made small changes in styles.
Can I merge PR ?


Change pip3 to pipx

Description
Following on discussion in https://github.com/dockstore/dockstore/pull/5958 and at the suggestion of @mr-c we now suggest using pipx over pip3 (it manages virtual environments for the user).
Review Instructions
Successfully install cwltool using the instructions described in the onboarding.
Issue
https://ucsc-cgl.atlassian.net/browse/DOCK-2539
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[ ] Check that your code compiles by running npm run build
[ ] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[ ] If this is the first time you're submitting a PR or even if you just need a refresher, consider reviewing our style guide
[ ] Do not bypass Angular sanitization (bypassSecurityTrustHtml, etc.), or justify why you need to do so
[ ] If displaying markdown, use the markdown-wrapper component, which does extra sanitization
[ ] Do not use cookies, although this may change in the future
[ ] Run npm audit and ensure you are not introducing new vulnerabilities
[ ] Do due diligence on new 3rd party libraries, checking for CVEs
[ ] Don't allow user-uploaded images to be served from the Dockstore domain
[ ] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.
[ ] Check whether this PR disables tests. If it legitimately needs to disable a test, create a new ticket to re-enable it in a specific milestone.

Tested to work with ubuntu 20.04, 22.04, and 24.04. Mac is the odd one out but can install via pipx install cwltool as mentioned at the bottom.

comment as above, pipx runpip cwltool install -r requirements.txt seems redundant , but I probably am missing something

comment as above, pipx runpip cwltool install -r requirements.txt seems redundant , but I probably am missing something

We could probably do without it, then the requirements.txt provided by the server would be purely information only.


Omit schema name in sql preview after changing table properties

Is your feature request related to a problem? Please describe.
When using the table editor to change some properties of a table, (adding attribute, changing attribute name, etc.) the created sql command always includes the schema name. A lot of time, I just want to copy those SQL commands and add them to the Database version control. But then I always need to first remove the schema names (obviously it already happened that I was
tired and forgot to do so).

Describe the solution you'd like
Provide an option to disable the adding of the schema name

Thank you for the suggestion.

Unfortunately, the task is not trivial. The text that comes into the ViewSQLDialog can no longer be changed. We need to use a different entity.

Unfortunately, the task is not trivial. The text that comes into the ViewSQLDialog can no longer be changed. We need to use a different entity.


Scala Stream Collector: emit CloudWatch metric for count(records) per shard

Currently the SSC doesn't actually know which shard each record ends up in - rather than specifying a shard directly, it specifies a partition key which Kinesis then hashes to a 128-bit integer in the keyspace. Each shard gets all the records assigned to a particular range in the keyspace.
We could override this hashing by explicitly providing a value using the ExplicitHashKey parameter (http://docs.aws.amazon.com/kinesis/latest/APIReference/API_PutRecords.html)

Hmm - I think let's leave as-is...

Pushing back...


Mapper doesn't work when using concurrency token as Timestamp/Rowversion

Usually the token is mapped to a byte[] and the error looks like the folowing. If there is a workaround, please share it.
Message:
System.InvalidOperationException : Can't construct byte[]. It does not have a parameterless constructor or a concrete type specified.
Stack Trace:
ClassTypeOptions.Construct(Expression context)
ListMapperFactory.Create(TypeMap typeMap)
MapperFactory.CreateMapper(TypeMap typeMap)
MapperFactory.CreateMemberMappers(TypeMap typeMap, Func2 filter)     EntityListMapperFactory.Create(TypeMap typeMap)     MapperFactory.CreateMapper(TypeMap typeMap)     MapperFactory.CreateMemberMappers(TypeMap typeMap, Func2 filter)
EntityListMapperFactory.Create(TypeMap typeMap)
MapperFactory.CreateMapper(TypeMap typeMap)
MapperFactory.CreateMemberMappers(TypeMap typeMap, Func2 filter)     EntityRootMapperFactory.Create(TypeMap typeMap)     Mapper.<Map>b__6_0[TSource,TTarget](ICacheEntry entry)     CacheExtensions.GetOrCreate[TItem](IMemoryCache cache, Object key, Func2 factory)
Mapper.Map[TSource,TTarget](TSource source, TTarget target, IMapperContext context)
lambda_method(Closure , Object , Object , IMapperContext )
Mapper.Map(Object source, Type sourceType, Object target, Type targetType, IMapperContext context)
DbContextExtensions.Map[TEntity](DbContext dbContext, Object entityOrDTO, MapperParameters parameters)

Hi @ricabrica, thanks for writing
That error message means that byte[] is not being taken as a primitive, so that, the library handles it as an object.
On the other hand, primitives are copied 'as-is'.
It can be fixed by adding the type to primitives:
 .UseDetached(options => options.Primitives.Add(typeof(byte[])))
Do you think I should add it by default?

Hi @leonardoporro,
byte[] is specific to SQL Server. if your library is SQL agnostic, than let the byte[] out. UseDetached is a good  and a viable option. Thanks for your info.

Ok, I'll add more tests about arrays later.
Please let me know if it works...

Collections of primitives are now processed as primitives.


[WIP] TensorFlow Studio prototype

For now:

Added placeholder for TF Studio backend server.
To add index page.

Coming:

Config loading / writing (.studio, ~/.studio.rc).
Example plugin with handlers and frontend.

Datasets plugin
Estimators plugin

FYI @danmane @martinwicke @terrytangyuan

Closing this for now, will do make another PR in few weeks after tf.learn is solid.


[ci:component:github.com/gardener/ext-authz-server:0.3.0->0.4.0]

How to categorize this PR?

/area TODO
/kind TODO
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

/lgtm
/kind enhancement
/approve


Set to private property FieldValidationResult.key

Right now, we have:
export class FieldValidationResult {
  key: string;
  type: string;
  succeeded: boolean;
  errorMessage: string;

  constructor() {
    this.key = '';
    this.type = '';
    this.succeeded = false;
    this.errorMessage = '';
  }
}

When a user has to create validation function that resolve a FieldValidationResult, he has to provide an empty key property:
e.g:
./requiredValidation.ts
export const required => (vm, value) : FieldValidationResult => {
  return {
    key: '',
    type : 'REQUIRED',
    succeeded : (value && value.length > 0),
    errorMessage : 'Mandatory field',
  }
}

I think key property should be a private property because it's only used internally

I'll work on this.

The key property is actually assigned by _validationEngine, to solve that usage case we only have to set key as optional.