Instantiating FamilySearch instance makes a lot of remote calls

When I new up a FamilySearch object, I've noticed that it makes a lot of calls. Can we defer these until we need them?
For example:
https://sandbox.familysearch.org/.well-known/collection
https://sandbox.familysearch.org/platform/collections
https://sandbox.familysearch.org/platform/collections/tree
https://sandbox.familysearch.org/platform/collections/records

Good point. It's setting up the hypermedia, but that definitely is something to work on.

Cool.
I'm starting work on a web app, so i'm looking at this through the lens of lots of different users using this simultaneously. This would mean it would be worth rethinking how thread safe this library is.
If newing up an instance is cheap, I'd probably just create a new instance per request.

True. I guess my initial thought was to have it "all ready to go", but if it's used on a Web server, with a potential of thousands of logged-in users a day, that's potentially thousands of wasted calls.

Yep. Assuming webservers are using a thread-per-request model, we could consider keeping slowly/rarely/never changing stuff separate (i.e. cache it once in memory) from the state that's likely to change per request (like a given user's access token).
A couple of options to solve the short term problem:

You could have an  init() method that would allow the caller to decide when to make those expense remote calls.
You could lazy load that data the first time a caller needs to use them

Or do both :-).

I was leaning more towards the 2nd.
We would definitely need the well-known/.well-known/collection, as it contains the login endpoints, though...
I'll try to get this "fixed" by the RootsTech Hack-a-Thon...

OK. I've gotten another Discovery module that trims it to half of its original size, while adding a new function to work on minimizing unnecessary calls, but completely breaks the internals of the SDK that's hypermedia-based.
I'll be adding it to another branch in the meantime.

Got a temporary solution in place.


Type inference not working when assigning somewhere in the function chain

Using TypeScript 1.6 in Visual Studio 2015 RTM, the following code does not provide syntax highlighting:
var x;

function init() {
    x = {
        color: "blue",
        height: "50"
    };
}

init();
x.???

Is this something on TypeScript's roadmap to solve?
Or what is the best way to refactor? Ideally I would have intelligent support in Visual Studio that lifts the assignment towards the variable declaration, e.g.:
var x: { color; height };
function init() {
    x = {
        color: "blue",
        height: "50"
    }
}

init();
x.color = ...

We only infer from the initialization of a variable, not from other assignments. There is no plan to change that.
I would write the code this way given what's declared:

function init() {
    return {
        color: "blue",
        height: "50"
    }
}

var x = init();
x.color = ...

Is this the same as:

Is line 11 an assignment and not an initialization? I suppose the property on line 8 gets initialized as null or something?
I know I could use constructor(private http: HttpClient) {, but I think that I prefer the more explicit declaration of the property.
I guess this means that I have to assign a type to the property in addition to the constructor argument.

@glen-84 yes, you need to specify the type on http itself.
You can also use parameter properties instead:
export class Index {
    // Notice the use of 'public' before 'http', which automatically
    // declares and initializes a property named 'http'.
    constructor(public http: HttpClient) { }

    async activate() {
        try {
            let x = await this.http.fetch("https://cors-text.appspot.com/test");
            console.log(x);
        }
        catch (e) {
            console.log(e);
        }
    }
}

@DanielRosenwasser Thanks, that's what I suspected.


encoding/json Unmarshal does not return error  on ambigues input

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.8 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/sarath/go"
GORACE=""
GOROOT="/home/sarath/goroot"
GOTOOLDIR="/home/sarath/goroot/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build040972726=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
What did you do?

Created two structs with one field same in both and embedded both to a new struct
Eg:
type A struct{ X int }
type B struct{ X int }
type C struct {
       A
   	B
}

Tried to assign C.X , it failed as expected with expected error  ambiguous selector c.X

Tried to unmarshal a JSON to C .It did not assign value to X but did not return any error ,which is unexpected

Here is the playground link : https://play.golang.org/p/SUlPSC2y3F
What did you expect to see?
Expected error to return on  json.Unmarshal
What did you see instead?
No error returned and no field assignments either

Another interesting effect is that json.Marshal() of the C type in OPs playground example outputs nothing, and no error is returned.
Playground: https://play.golang.org/p/REonDIbRan
Is this the expected behaviour?

There seems to at least be some documentation missing. Leaving for @rsc.

Kindly requesting your input here @rsc.

I'm not sure if I see a bug here. It seems to me that json.Unmarshal is working as documented.
Have you read this part of the encoding/json docs?

The Go visibility rules for struct fields are amended for JSON when deciding which field to marshal or unmarshal. If there are multiple fields at the same level, and that level is the least nested (and would therefore be the nesting level selected by the usual Go rules), the following extra rules apply:

Of those fields, if any are JSON-tagged, only tagged fields are considered, even if there are multiple untagged fields that would otherwise conflict.

If there is exactly one field (tagged or not according to the first rule), that is selected.

Otherwise there are multiple fields, and all are ignored; no error occurs.

In the original sample, what's happening is that there are multiple JSON-tagged fields. Therefore, the last rule applies. All multiple fields are ignored, no error occurs.
Do you see anything wrong with my analysis?


No way to specify explicit binding for vertex shader inputs

I found the -EB flag which translates the binding/location of samplers, constant buffers and fragment shader outputs.
However GLSL also allows layout(location = #) on vertex shader inputs.
There should be a way to map HLSL input semantics to GLSL binding locations.
For example:
POSITION   0
NORMAL     1
TEXCOORD0  2
etc.
This map could be passed in Xsc::ShaderInput for the library.
(Or appropriate command line flags for xsc.exe eg: -STEXCOORD0=2)

Yes, it's an open issue, but it's not my current main focus,
since there are still problems with the I/O semantics translation.
E.g. when a struct is used as input and output.
If you have a good solution for this issue, let me know.
Is it an urgent demand for you?

You don't have to do this urgently I just thought I create this issue as a reminder.
I'll check how hard is it to implement it myself.
I need this in a hobby project I try to use XShaderCompiler in.

Completed with 2487ebf5a032c7d6f4f5f8ce24c4f4f4c4f17085


Structure filter: 'current value > use as filter' breaks filtering in some cases

chem 1.8.3, 1.8.2, 1.8.0
Steps to reproduce:

Open a table, e.g. SPGI or PubChem_compound_text_covid-19.csv
Open filters and make sure some structure column is added to the filter panel
Add another view with the same table (e.g. clone current view)
Open filters and make sure same structure column is added to the filter panel in the 2nd view
Right click some structure > Current value > use as filter

Expected results: filter is applied, filters are synchronised in two open views.
Actual results: Looks like filtering process never ends, some rows are appearing/disappearing indefinitely:

In some cases filtering does not start until switching to another view. After that behaviour is the same as described above.
Not reproduced on chem 1.7.2

Chem 1.8.4


support virtual properties through dedicated schema

This provides support for virtual properties, even with select filtering on. Not crazy about the API but the types need to be specified somewhere. I considered adding a boolean virtual property to the existing json schema props but I don't think that's compliant with the spec.
p.s. Sorry about the two other erroneous pull requests - in a bit of a rush.

@nasushkov I feel this might be a good one to start on once you get some time. I realized after submitting it that I had possibly accidentally resolved #51 with it (or gotten close). But I think this will also be a good jumping off point for the discussion of "using objection model properties as GraphQL configurations - yay or nay?". It seems like it might be more palatable if we pick a single property (maybe graphqlConfig or something) and put everything in there.

Hi @timhuff do you want to provide a fine-grained configuration with exact fields in the graphqlConfig or something more high level like:
graphqlConfig: {    useSchema: true,    useRelations: true,    useVirtualProps: false, }
I think by default the schema should be generated with all options set to true and if you want to fine-tune your generated schema, you can configure the output with this extra field.

@timhuff I think we can merge this for now and get back to a more general solution in the future

@nasushkov Yea, I was thinking more along the lines of the high level example. But then the high level one would also have a virtualProps property (which I think could replace useVirtualProps). So something like graphqlConfig: { useSchema: true, useRelations: true, virtualProps: {fullName: {type: "string"}}}


[REQUEST] Limiting cache

What is the feature you're proposing?
As of right now, caching has no limit.
I feel like there should be a way to set a limit for each type of cache, preferably in the Client instance.
Something like:
bot = interactions.Client(
    ...,
    cache_limits=interactions.CacheLimits(
        messages=100,
        guilds=50,
        ...
    )
)

This is the best thing that I can think of off the top of my head.
Another way is to do what dis-snek did do something with kwargs and have something like this:
bot = interactions.Client(
    ...,
    message_cache=100,
    guild_cache=50,
    ...
)

Additional Information

[ ] My feature request is related to an existing Issue.

Issue (if referenceable):

Garbage collection is a pretty bashed topic in terms of data in a library being handled. For the most part, the cache is only on-demand so it has to be done through a listened event, or by creating something through the Web API. I'm not sure if we particularly need limits because while yes, they do help with memory size, it's also hard locking how big a cache can be.
I think TTLing unused data in the cache would be a better approach, not limiting how big the cache can be. However, TTLing information would need to have some work done through how the data is accessed. If we want to reliably check this, we'd have to either force users checking the cache to use the current view() method, or using the attrs module which I have been looking into.

Ok I see
What I was thinking was to remove the first item(s) in the OrderedDict when the cache hit the limit.
I don't know what TTLing is, but I like the current method of using get(), but view() could work I guess.

Ok I see What I was thinking was to remove the first item(s) in the OrderedDict when the cache hit the limit. I don't know what TTLing is, but I like the current method of using get(), but view() could work I guess.

TTL (Time to live), something like store a timestamp when it was stored / accessed and either have a job/task that checks these dates in intervals to invalidate / remove from cache. Or check each entry everytime the cache is accessed. Hope that helps :)

Ohh I see, thanks!

Proposal may be amended to #909 , please make a comment referencing there.

I'd say while #909 mostly fixes it, it may still be useful having something like this for messages, as they can grow more or less unbounded in most servers since many messages are never deleted.

Ideally for me, you would be able to configure the cache size (with a reasonable default) and use the last accessed time as above or a combination of number of times accessed and last accessed time to determine what to purge when the cache limit is hit.
So when adding an item to the cache,

If cache is full, then find least used item and remove
Add new item to cache

This way you don't need to have another task responsible for pruning the cache and you don't unnecessarily remove items from the cache if there is no reason to.
Thanks

Are you able to share what the most frequently cached objects are? I'm guessing it's between guilds and interactions.

I don't explicitly cache anything myself and I'm not sure how to see the full caching of interactions / guilds. My bots are in 1000+ servers so I'm assuming it's the interactions.
If you know how I can view it let me know and I'll share it.

You can get access to the cache by doing client._http.cache. You can do cache[<type>] to get to the Storage of that type, or cache.storages to get a dict of type->Storage.
Once you have the storage, you can do len(storage.values) to see how many items are cached of that type.
Code example:
cache = client._http.cache
for type, storage in cache.storages.items():
  print(f"{type}: {len(storage.values)}")

It looks like <class 'interactions.api.models.message.Message'> is constantly growing and I'm assuming that's causing the memory issues.
starting:
CACHE CHECK
-------------------
<class 'interactions.api.models.guild.Guild'>: 1086
<class 'interactions.api.models.channel.Channel'>: 41631
<class 'interactions.api.models.role.Role'>: 28583
<class 'interactions.api.models.member.Member'>: 1153
<class 'interactions.api.models.channel.Thread'>: 160
<class 'interactions.api.models.message.Message'>: 596
<class 'interactions.api.models.gw.MessageReaction'>: 0

30 mins later:
<class 'interactions.api.models.guild.Guild'>: 1087
<class 'interactions.api.models.channel.Channel'>: 41657
<class 'interactions.api.models.role.Role'>: 28596
<class 'interactions.api.models.member.Member'>: 1158
<class 'interactions.api.models.channel.Thread'>: 161
<class 'interactions.api.models.message.Message'>: 3361
<class 'interactions.api.models.gw.MessageReaction'>: 0
<class 'interactions.api.models.gw.ChannelPins'>: 0
<class 'interactions.api.models.gw.GuildJoinRequest'>: 0
<class 'interactions.api.models.gw.GuildStickers'>: 0
<class 'interactions.api.models.gw.GuildMember'>: 1

@Dillonzer How many times are you working with roles? Are you doing anything with them, or is this purely from what's being cached? I'm astounded to see nearly 30,000 objects cached for them. That's insane!

@i0bs I don't do anything with roles. For context, all this bot does is takes input from the user and spits out embeds, w/ buttons to cycle through the different embeds created via the bot,  from my API based on the users query.  (ex, they put in a card name and it spits out information about those cards).

Interesting. The cache is not intelligent enough to determine what is used the least. After testing, the cache can contain "duped" objects of the same ID (using this as our basis) while values have changed.
Making that work would additionally prove to be a challenge as every object must perform a cache introspection to determine if it exists, and which fields to update. Rewriting the entire object will be extremely memory inefficient, so that would leave us to changing specific attributes.. which now goes into slotting and DictSerializerMixin hassles. TTL seems the best way to go. I don't think LRU or FIFO/FILO-based caching would be efficient here.

For clarification, I'm also trying to use interactions.py for a company contract and I'm running into a plethora of caching problems. Most of them relate to "duped" objects, but the biggest problem by far has been the inability to control storage sizes per cacheable object. There's no way to performantly scale memory usage, I can't opt-out of things being cached without removing Gateway intents either.

Closing this as the base issue has been resolved.


[HUDI-25] Optimize HoodieInputFormat.listStatus for faster Hive Incremental queries

Summary:

listStatus() now classifies inputPaths into incremental, non incremental and non hoodie paths.
Process each of the input paths separately
Incremental queries leverage HoodieCommitMetadata to get partitions that are affected and only lists those partitions as opposed to listing all partitions

Made 1 pass and left some comments.

@bhasudha left 1 comment but rest looks good to me. This is a pretty significant change, could you come up with a test/rollout/rollback plan.

Will do!

I was able to successfully cross verify the query results between the current HoodieInputFormat and this new HoodieInputFormat for few Uber production tables using spark. I ran different snapshot queries on MOR tables that has count(*), group by's,  joins etc. The query latencies were also comparable.
For Incremental queries I can't test it yet, without changing the jar in Hive MetaStore. I will be doing that next. My plan is to have that tested in staging and then gradually rolling it to production.
@n3nash @vinothchandar ^^

@bhasudha this looks good overall. We are currently stabilzing master . Wil merge once we are in calmer waters

@bhasudha Could you please rebase to master and merge it as it is ready?

I was able to run tests using spark.sql() against some of the production tables (both MOR and COW types). I used  --conf spark.sql.hive.convertMetastoreParquet=false so Hive serDe is used instead. Below is a flavor of queries that I tested. The results match between pre-fix and post-fix hudi-spark-bundle jars.
Snapshot queries
simple count:
spark.sql("select count(*) from tableA where datestr = '2019-12-10'").show()
non-hudi hudi datasets join:
spark.sql("select m.col1 as colA, t.col2 as colB from table1 m left join table2 as t on t._row_key = m.col1 and t.datestr >= '2016-01-01' join table3 c on m.col4 = c.col5 where c.col6 = 'XYZ'").show()
non-hudi non-hudi datasets join:
spark.sql("select o.col1, count(distinct e.col2) from tableA o join tableB e on o.id = e.id where to_date(e.col1) >= date_sub(current_date, 10) or to_date(e.col3) >= date_sub(current_date, 10) group by 1 order by 2 desc").show()
hudi hudi datasets join:
spark.sql("select t.id, count(t.load) as total_count FROM tableT t LEFT JOIN tableO o on t.id = o.id AND o.datestr > '2019-12-28' AND NOT o.isactive WHERE t.datestr > '2019-12-28' AND NOT t.isactive group by 1 order by 1,2 desc").show()
group by, order and rank:
spark.sql("select * from ( select , rank() over ( partition by rg order by total_items desc ) as row_number from ( select rg, usr, count() as total_items from tableA where date(datestr) >= date('2019-10-11') and date(datestr) < date('2019-10-16') and event = 'complete' and  SUBSTRING_INDEX(rg, '.',1) = 'adhoc' group by 1,2 order by 1, count(*) desc ) ) where row_number <= 1").show()
Incremental queries
spark.sql("select name, count(*) from  tableA where event_status = 'complete'  and _hoodie_commit_time > '20200101235440' group by 1").show()

Thanks for the update @bhasudha and welcome back :) .. Will make a final pass and then merge.

Tests are passing and review commends are addressed. Merging this code in.


Dynamically loading js files

Hi,
I'm currently testing frontend development in general with rust and kind of
hit a use case that might be very common but currently is not working as
expected.
What I try to achieve is that the framework is loading some JS files and it
gets loaded. What I got working is to load the JS file with the leptos_meta
crate into the head section of the index.html, but things like event
handler for dropdowns for example are not registered. (I try to use flowbite
with tailwind CSS here)
I am kind of confused, is this not a general use case to load JS files on page
load that depend on the DOM? How can I solve this problem? Any information
that I found was about how to call into JS functions from rust, which I don't
even want to do in this scenario.
Is there any solution to this? How are other people using frameworks like
bootstrap, flowbite etc. with leptos?

Can you provide any more information about what you're trying to do and what isn't working, with a code example? It's not really possible to answer your question without more information than the problem that event listeners aren't registered.

Having the following example:
main.rs
use leptos::*;
use leptos_meta::*;

#[component]
pub fn Root(cx: Scope) -> impl IntoView {
    provide_meta_context(cx);
    view! { cx,

                          <svg aria-hidden="true" class="w-6 h-6 text-gray-400 transition duration-75 dark:text-gray-400 group-hover:text-gray-900 dark:group-hover:text-white" fill="currentColor" viewBox="0 0 20 20" xmlns="http://www.w3.org/2000/svg"><path d="M2 10a8 8 0 018-8v8h8a8 8 0 11-16 0z"></path><path d="M12 2.252A8.014 8.014 0 0117.748 8H12V2.252z"></path></svg>
                        "Overview"

                      <button type="button" class="flex items-center p-2 w-full text-base font-normal text-gray-900 rounded-lg transition duration-75 group hover:bg-gray-100 dark:text-white dark:hover:bg-gray-700" aria-controls="dropdown-pages" data-collapse-toggle="dropdown-pages">
                          "Pages"
                      </button>

                              "Settings"

                              "Kanban"

                              "Calendar"

                      <button type="button" class="flex items-center p-2 w-full text-base font-normal text-gray-900 rounded-lg transition duration-75 group hover:bg-gray-100 dark:text-white dark:hover:bg-gray-700" aria-controls="dropdown-sales" data-collapse-toggle="dropdown-sales">
                          "Sales"
                      </button>

                                  "Products"

                                  "Billing"

                                  "Invoice"

                          "Messages"

                              6

                      <button type="button" class="flex items-center p-2 w-full text-base font-normal text-gray-900 rounded-lg transition duration-75 group hover:bg-gray-100 dark:text-white dark:hover:bg-gray-700" aria-controls="dropdown-authentication" data-collapse-toggle="dropdown-authentication">
                          "Authentication"
                      </button>

                            "Sign In"

                            "Sign Up"

                            "Forgot Password"

    }
}

fn main() {
    console_error_panic_hook::set_once();
    _ = console_log::init_with_level(log::Level::Debug);
    console_error_panic_hook::set_once();
    mount_to_body(|cx| view! { cx,  <Root/> })
}

index.html using trunk
<!DOCTYPE html>

    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <title>Testing</title>
    <link rel="stylesheet" href="/app.css"/>
    <link data-trunk rel="rust" data-wasm-opt="z" />
    <link data-trunk rel="copy-file" href="node_modules/flowbite/dist/flowbite.js" />
    <base data-trunk-public-url />

I then get the following in the browser:

But the dropdowns don't work because the event handlers from flowbite.js are missing. It works if I try the exact same component in a static index.html file with the js file beside it.
I can see that the script is correctly added:

And I also can open the script by clicking on the path, so the path is also correct.
I hope I could get my use cases clearer.

If I had to guess I'd assume that the JavaScript is loading before the WebAssembly has rendered your interface, so it doesn't find any drop downs that already exist. Is there a way to call your JS library so it only looks for drop downs when you tell it to, rather than when it loads? This would also be necessary when integrating with a JS front end framework I'd think.

I already tried the same in yew, and my guess was the same, the DOM is not fully rendered by the WebAssembly and therefore no event handlers get attached correctly. I think it is not a good practice to change the library to work with my setup. For one package this might work, but imagine dozens of packages that you have to patch, updates would get a nightmare I assume.
Another solution would be to write a custom JavaScript loader that checks if the DOM is already fully generated (if someone knows how to do this in rust with wasm_bindgen or integrate it even in leptos_meta feel free to join the discussion and share some ideas here).
My hope wit leptos was the Script function, but unfortunately the behavior is quite the same. What I wanted to get clear in my issue: this will be the same for every JavaScript code that depend on the DOM in some way
IMO this is a huge shortcoming for frontend frameworks in rust, don't get me wrong, I like rust (somewhat a member of rust evangelism strike force) but this is not very appealing for frontend development. There are millions of NPM packages there, and we cripple ourselves by making it that difficult, maybe even impossible, to use.
At this point I'll have to admit that my experience with WebAssembly is not sufficient to even tackle this problem, I wouldn't even know where to start, but I hope someone comes up with an idea to solve this in a really simple, robust and elegant way, so frontends get much easier with rust.
Maybe my comment is a bit harsh and I want to end it on a positive note:
Leptos is really fun to use. It might need some more documentation (or examples, maybe I'll open a PR for an example using trunk like I did in my testing). If I'd compare it to yew, for example, I think it's much simpler without loosing expressiveness. So keep up the good work and I try to contribute where I can :grinning:

I actually don't think the real issue has anything to do with JavaScript vs. Rust/Wasm. A library that assumes that everything it needs to do has already been rendered in the DOM will be wrong if you're pairing it with any front-end framework, unless there's a custom-built integration, or you use the framework to render the script tag to load it. To be clear, IMO this is an issue with flowbite's design (i.e., using an IIFE to check once for DOM elements immediately when it's run rather than exposing a function that you can call when you're ready to set it up)
I was able to get your example working by just using a

This seems to work in Chrome, but not in Firefox. I am not sure why. The dropdown menus work properly in Chrome.
It seems like maybe there's an issue with the


feat(field): add hide option

What kind of change does this PR introduce? feature
What is the new behavior (if this is a feature change)?
http://docs.angular-formly.com/docs/field-configuration-object#hide-boolean

please add an example in the demo app too. Thanks

done

Current coverage is 93.26% (diff: 75.00%)

Merging #324 into master will decrease coverage by 0.11%

@@             master       #324   diff @@
==========================================
  Files            16         16
  Lines           726        728     +2
  Methods         141        141
  Messages          0          0
  Branches        187        187
==========================================
+ Hits            678        679     +1
- Misses           48         49     +1
  Partials          0          0

Powered by Codecov. Last update 0a0b42c...c1386a8


US-2072 EOA Wallet - remove Smart Wallet Address from the account screen

@TravellerOnTheRun, I still see the changes in the 2072 branch, both testnet and mainnet:
https://github.com/rsksmart/rif-wallet/assets/114427184/eb999097-ac6b-464d-8949-1fd99d71f943

@TravellerOnTheRun, I still see the changes in the 2072 branch, both testnet and mainnet:
20240109_112200.mp4

Because you're using RelayWallet, switch to EOA by setting .env USE_RELAY=false and refresh your IOS build with npx react-native-clean-project


Conduits prefer empty slot to one with item

When moving items conduits place item in first slot they can, which results in placing items that are already in the inventory in another slot. Can it be changed to check for filled slot first?

no, we cannot change that, because...
...we already changed it 2 months ago.
Will be in version 2.3.

OK :(


Applying plugins when DOM elements aren't static

Consider the following scenario:
http://hedgerh.github.io/fileupload-demo/
A template is generated on the fly when the user selects which option they want to use.  For the case of the drag n drop plugin, it needs a target element to designate the drop area, but that target does not exist statically on the page.
So if we used something like:
Transloadit
  .use(dragdrop, { target: 'dragarea' })
  .use(instagram)
  .use(dropbox)

it's not going to work.  We'll need to apply the drag drop plugin to the element every time it regenerates.
Thoughts?
Repo link:
http://github.com/hedgerh/fileupload-demo

We provide some callback/event thing that is fired when the element regenerates or appears on the page?

Are you saying that we could provide a method for them to call when the element reappears?

Yes.

Can you show an example of how you think it could work?

I'm starting to think a better pattern for using the client may be:

Import transload-client
Import plugins that you wish to use
Interact directly with the plugin methods
submit file data to core
transforms are applied, directed by the core
data sent to server

In this flow, the user has direct control over every plugin and is going to make it easier to deal with use cases.  The .use() syntax gives us an inversion-of-control pattern, where its job is to suddenly handle things like this.
Also, I hope I'm not bugging you guys with this stuff, but I think this is really important to consider.  I think it'd be a good idea to walk through a couple of use cases that are going to rely on these plugins to make sure this actually makes sense.

I'm not entirely clear on what the third party plugins will do exactly.  Will the instagram/dropbox/google drive plugins provide specific file browsers for each platform?  In that case, these should be considered as more component based plugins that would also benefit from this pattern.

Superseded by todo:
core: Decouple rendering from Plugins and try to make Uppy work with React (add basic example) to remain aware of possible issues (@hedgerh), look at https://github.com/akiran/react-slick


42 rent stablized three mainteinance

Rent Stablisation and Three or More Mainteinances
Based on the new spreadsheet from HPD for new rent stablisation and rework the three maintenance indicator.
Functions
So the two functions for the two indicator and both take in geography as an input to produce the results. This work is really similar to the other housing security work. The only difference is where the column name change happens to accommodates the specificities of the source data.
##PUMA_helpers
small changes to the puma helpers function to handle null cases in puma column
Review
Unfortunately this results do diverge quite significantly from the completed works from @SashaWeinstein based on the borough level results in that branch. I do want observe while the results are off by some factor between 2 to 3 but the trend in neighborhood seems to hold in Sasha's results

Sam just sent me the denominator files. Working on adding them now.
2017 HVS Denominator.xlsx

I would just assume my results are wrong? Is it worth digging into?

We're not short of work so leave it for now, but it will be something we'll likely troubleshoot together once it's confirmed that we will need to switch data sources from ACS to HVS and/or more than one year of data will need to be included in EDDT.

Ok neat. I know I said I was interested to see if my results were correct but at this point I'm more interested in getting the EDDT deliverables out the door and working through a new data product

Sorry for keep adding to this work. I realize the column needs reordering and also change the function name to something more explicit. It should be ready for final review now.
Turns about in order to incorporate the other worksheet with denominator there is significant new work added and refactoring for the functions. Here are what changes were made.
extract_geography_dataframe
since I could not come up with a way to pre-join the denominator sheet and indicator sheet together before extract the results for different geography level. I decided that I would need to create two dataframes from the two sheets and rejoined them together. Another thing to note is the denom flag makes sure the right columns to drop depending processing which dataframe since denominator do not have Percent SE.
indicator functions
Now the two indcator functions simply rejoin the denom and indicator dataframes created and then start doing the final column change. No reordering for the columns at the moment but wonder if that is needed or this work.
@SashaWeinstein @AmandaDoyle @mbh329

It appears that the three or more maintenance deficiencies get it's denom info from the "2017 HVS Occupied Units" tab in the "2017 HVS Denominator.xlsx" file. That sheet doesn't say anything about maintenance deficiencies, it's tab names and headers talk about occupied units. Are sure this is the right source data for denoms of this indicator?

@td928 @SashaWeinstein the data matrix mentions that occupied units should be the denominator

I'd like to take a look at this again on monday with a fresh set of eyes

Theres a couple other issues going on, did the cv and moe columns get mixed up?

hey sorry I didn't mean to push b80ef4e, undid changes with second commit

Okay so no generalizable function was written because trying to handle both the reordering for denom and indicator itself it challenging with different measurements with the existing Frankenstein approach to the creating indicator in the first place.


⚠ ClamAV has degraded performance

In abe9194, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 87 ms

Resolved: ClamAV performance has improved in 0dbfa76 after 5 minutes.


Hdf5EventSource performance improvements and skip empty events, fixes #1916

This yields a factor of 5 speed improvement for a large, merged gamma file

@kosack it removes the reliance on these index tables. So https://github.com/cta-observatory/ctapipe/pull/1898 is not strictly needed anymore

But we might still want to merge #1898, since these tables might be handy for reading chunks of events with the TableLoader and they don't take a lot of space


Tooling fixes

Return empty slice when no tags are found.
Added module version string validation.
Added optional arg 'initial module version' to versioner tool.

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] The PR targets the latest branch.
[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] Apache v2 license headers are included in each file.

Can one of the admins verify this patch?


update count_occuerence method

use default method to get the occurrence of the value in a list. this method automatically handle all the condition of the value.

Description

Resolves #(issue number)
What does your PR belong to?

[ ] Website
[ ] Snippets
[ ] General / Things regarding the repository (like CI Integration)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (non-breaking improvement of a snippet)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have checked that the changes are working properly
[ ] I have checked that there isn't any PR doing the same
[ ] I have read the CONTRIBUTING document.

but python method used there are really too bad.

Please read the contribution guidelines for submitting and updating snippets.


Need a power up to parse a column in a table and transform the data

I run a query like: select useraction.name FROM usersession
And I get data like: [loading of page /platform/, loading of page /marketecture-master, loading of page /platform/digital-business-analytics/]. The data elements are delimited by a coma.
I would like the PU to take that character string and return the last element like: loading of page /platform/digital-business-analytics/

Need use case


Range Calculator

Hey @Srijita-Mandal, I would like to open this issue under JWOC. I wish  to add a calculator which will calculate the range of a population/stream by

initially sorting the array
then applying this formula:
Range(arr) = Max(arr) - Min(arr)

Kindly assign 😀

@ritika728 go for it.


Healthcheck and packaging

Please when are we going to see external dns in rancher

@lavvy Please follow this feature to see when it would be released in Rancher. https://github.com/rancher/rancher/issues/1675


FDS-357 November Components

Move Flex component into Cascara from POC and integrate it into a couple components internally

Fails

:no_entry_sign:
Please add a '### Snapshots' section to explain the reason we are changing snapshots.

Warnings

:warning:
Please assign someone to merge this PR, and optionally include people who should review.

Messages

:book:

packages/cascara/src/atoms/Button/Button.fixture.js: This fixture has been changed.

:book:

packages/cascara/src/ui/List/List.fixture.js: This fixture has been changed.

  Generated by :no_entry_sign: dangerJS against 25fc6e665f0c2f0e2e1ee80c5ef61d6b1d36596f


Fix test: Add missing setEditorSplitWidth function to WindowTools

Not sure what broke here, I was able to verifiy functionality and pull the code from the
published 1.0.0 version, but the method never shows in git.
Searched commit history with "git grep "setEditorSplitWidth" $(git rev-list --all)" and can not
find a committed version

🟢 Green light


Required ARIA parent role not present: listbox

Bug Report
Description

@juliacalandro once reported this as a bug, looks like this accessibility issue still persists with the terra-list. Here is the link for the issue that @juliacalandro has logged.
I have updated our project terra-list version to use the latest version which is 2.30.0. If we use any version above 2.6.0 we get the below accessibility error.
"description": "Ensures elements with an ARIA role that require parent roles are contained by them",
"help": "Certain ARIA roles must be contained by particular parents",
"helpUrl": "https://dequeuniversity.com/rules/axe/2.6/aria-required-parent?application=axeAPI",
"id": "aria-required-parent",
..................
"nodes": [
{
"all": [],
"any": [
{
"data": [
"listbox"
],
"id": "aria-required-parent",
"impact": "critical",
"message": "Required ARIA parent role not present: listbox",
"relatedNodes": []
}
],
"failureSummary": "Fix any of the following:\n Required ARIA parent role not present: listbox"

Steps to Reproduce

Install the aXe extension for Chrome.
Run a project using terra-list so that it can be opened and inspected in the browser.
Inspect the page and navigate to where the List component is being used.
Verify that ul tag does not have a role set but its child li tag s do.
Navigate to the aXe tab and run an inspection on the page.
Verify that an accessibility error is thrown with a description matching the error description above.
Add role="listbox" as an attribute to each offending  component and run the aXe inspection
again.
Verify that the accessibility issue doesn't exist anymore.

Additional Context / Screenshots

Expected Behavior

I expect the accessibility issue mentioned above to no longer be thrown by the terra-list component.
Possible Solution

Add missing role="listbox" attribute to  within terra-list component.
Environment

Component Name and Version: terra-list 2.30.0
Browser Name and Version:
Node/npm Version: [e.g. Node 8/npm 5]
Webpack Version:
Operating System and version (desktop or mobile):

@ Mentions

We are currently refactoring the list component to be a controlled component where the app developer will be responsible for managing the selection state of lists. With this, the <List /> componet will render a <ul> without any aria attributes by default.
If you add selectable list items within the <List> component we will be recommending adding tha aria-role="listbox" to the <List> component. With moving the component to a controlled component, we'll no longer know if the list is selectable, only if the list items are selectable, so the developers will need to add this attribute themselves to the <List> component.
We hope the doc helps clarify this, but we are also relying on the aXe error you mentioned to help notify developers if they make a selectable list but forget to add the attribute. We also recommend all UI being built with terra components be run through wdio tests using our aXe service so accessibility issues that can be programmatically identified are caught and corrected before the code is shipped to production.

@bjankord adding the role to the selectable List solved my issue. Yea doc will definitely help developers to solve this issue or else it will be really difficult to figure this out because I think we don't add aria-role explicitly for any other terra-components, so thank you for providing those links.


Strongswan missing curl support - OCSP/CRL Validation not working

Dear Opnsense Team,
May I Propose the implement the two following settings:

Compile Strongswan with curl support (--enable-curl) to be able to fetch CRL& query OCSP responder ?  At the moment it seems not to be the case.

Add a CA section into the configuration, (I know this is possible with adding this into /usr/local/etc/ipsec.opnsense.d) .

Implement a x509 certificate validation process would be a great improvement.
Thank you.
Kind Regards.

@fdhuart something went wrong with your feature request, you opened a PR instead containing our commits. please use our template in https://github.com/opnsense/core/issues to open a FR


Update workaround for copy and paste

This works on my machine. Can someone confirm?

Yep, this is how it works on my machine. IIRC, Bash uses the same shell host as command prompt, but doesn't have the ctrl + c etc keys for compatibility.
Anyway, thanks for the addition!

An alternative is to use the wsltty project to connect to WSL using mintty. With mintty (or putty/ssh client), ctrl+insert and shift+insert work fine.


Update semagglutidequantity.json

This is a slightly temporary fix for #2939 . I have included all the possible variations of tabs/caps, this might be slightly overkill and there are nicer ways of fixing but this is most time efficient I think!

this looks fine, although we might come a cropper again if liquid specials start appearing.
One alternative may be to have
bnf_code LIKE '0601023AW%' --Semaglutide
AND
bnf_name  LIKE '% inj %'

nb @brianmackenna I think it might not be your change that's the problem with CI, will look into it when I can

rebased to add the ansible CI fix

@brianmackenna just thinking about this again - the code won't work, as you'll need bnf_code NOT LIKE...AND rather than OR
I've amended the code to just include %inj% which seems to work in tests.

@inglesp can you merge this please - @brianmackenna and I are both happy.


How to use the model in inference mode

Hello,
how does one load the model for execution only (not training) to watch the trained behavior?
This is related to #71

If you load the agent but run it in eval mode, I think that will do what you want (combined to #71).

How is the eval mode used? Setting the training steps to 0 causes a division by zero exception.

That sounds like a bug -- I think probably just missing a check that if training_steps = 0, we shouldn't run the train phase. We already do this for the eval phase (when eval_steps = 0).

For the eval mode, are all checkpoints files necessary?
Like the checkpoints that store states can get really large.

You should only need the last checkpoint -- the code keeps a few (3) around, in case the most recent checkpoint is corrupted.

I mean these kind of checkpoint files:
add_count_ckpt.19.gz
ckpt.19
invalid_range_ckpt.19.gz
sentinel_checkpoint_complete.19
$store$_action_ckpt.19.gz
$store$_observation_ckpt.19.gz
$store$_reward_ckpt.19.gz
sum_tree_ckpt.19.gz
tf_ckpt-19.data-00000-of-00001
tf_ckpt-19.index
tf_ckpt-19.meta

Are all relevant for the eval mode? Especially the $store$_observation_ckpt can comprise many gigabytes.

The tf_ckpt* files are for restoring the network parameters, so you
definitely want those.
The other files are for restoring the replay buffer to the state it was at
when the checkpoint was saved.
If you're only doing inference for evaluation (i.e. not sampling batches
from the replay buffer to learn from) then you don't need those files.
On Mon, Apr 8, 2019, 12:21 AM Marco Pleines<EMAIL_ADDRESS>wrote:

I mean these kind of checkpoint files:
add_count_ckpt.19.gz
ckpt.19
invalid_range_ckpt.19.gz
sentinel_checkpoint_complete.19
$store$_action_ckpt.19.gz
$store$_observation_ckpt.19.gz
$store$_reward_ckpt.19.gz
sum_tree_ckpt.19.gz
tf_ckpt-19.data-00000-of-00001
tf_ckpt-19.index
tf_ckpt-19.meta
Are all relevant for the eval mode? Especially the
$store$_observation_ckpt can comprise many gigabytes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/dopamine/issues/95#issuecomment-480678215, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATYhMfe3XwytyJiin-OoW0mQbsIklc8Mks5vesPbgaJpZM4cQ3m6
.


INFRA-HOST: add cpu gm for otel

Relevant information
This is adding the cpu golden metric definition for hosts monitored by the opentelemetry collector.
Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

 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.


Take out "project box" on start page and move version info to bottom

The start page has project info that is already covered in the code repo. Please take out the box and only keep "version information". Move version info to the middle on the bottom of the page.

@mariobehling  what enhancement do you need here can you elaborate? I'll try to solve this issue today itself.

@nalawadepiyush This is what the objective is

Please take out the box and only keep "version information". Move version info to the middle on the bottom of the page.


Error while changing view visibility

va.lang.IllegalThreadStateException: Thread already started
at java.lang.Thread.checkNotStarted(Thread.java:849)
at java.lang.Thread.start(Thread.java:1059)
at cn.nekocode.camerafilter.CameraRenderer.onSurfaceTextureAvailable(CameraRenderer.java:121)

Thanks for your feedback. I will fix it later.

I try to fix it in this commit.
Please have a try. If it doesn't work for you, please reopen this issue and give me more information about your problem.

好吧，谢谢了有空试下，当时项目紧迫，我就又还原成以前的margic camera了。 对了不知道您的可以修改是否为前置或者后置摄像头。


" azure-arm.image: IMAGE_STATE_UNDEPLOYABLE''  error. Sysprep workaround needs to be updated.

The workaround 'Workaround for issue: https://github.com/actions/runner-images/discussions/6251' needs to  be updated, since tag  "ExcludeMdeAutoProvisioning": "True"  is not applied correctly. This causes sysprep to fail (see workaround)

I will work on an update of the workaround to fix this.

Just created a pull request with code to fix this issue.


Smarthome feature under Alexa Routine node has stopped working

exactly what the title says. Instead of getting all my devices, scenes and so on, I'm getting this  "???". Is anyone else having this issue?

Ok, it came back to normal by itself


post live demo directly in PR when build succeeds

if it succeeds, we'll see it below this comment

Codecov Report

Merging #809 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #809   +/-   ##
=======================================
  Coverage   99.76%   99.76%
=======================================
  Files         270      270
  Lines        6928     6928
  Branches     1686     1686
=======================================
  Hits         6912     6912
  Misses         16       16

Continue to review full report at Codecov.

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

it did noooooooot work......

it worksssssssssssssssssss !!!!!!!!!!


Android / Unity Engine Support?

I am looking for a high-performance wasm vm or interpreter that cab run modules at runtime using C# Unity scripts. My target devices are Android-based headsets. From the description, it sounds like WasmerSharp is more for server machines running the main Windows/Linux/Mac OSes, so this wouldn't work for my use case. Would you confirm whether WasmerSharp might actually be workable? If so, then how would I proceed?

I should look into upgrading Wasmer, although I do not know if this is what he is looking for?
KT, you want to run WASM code statically on the target device?

I wanted to run wasm in a C# host program, sort of in the way you can just wasm in a js host program in a web browser.

Got it - yes, so I need to upgrade WasmerSharp to bundle a newer version of Wasmer to solve this.

Hey @migueldeicaza , just wanted to know if there is any plans to update Wasmer anytime soon. I can help with the effort, this seems potentially very interesting for many use-cases.


[SPARK-30735][CORE] New functionality for partitionBy to enhance performance when writing data with partitionBy

What changes were proposed in this pull request?

Adding new boolean parameter to partitionBy method in DataFrameWriter.
  @scala.annotation.varargs
  def partitionBy(needRepartition: Boolean, colNames: String*): DataFrameWriter[T] = {
    this.partitioningColumns = Option(colNames)
    if(needRepartition) needRepartitionBasedColumns = true
    this
  }

If partitionBy(true, cols) is specified, the needRepartitionBasedColumns parameter sets true. Based on this parameter, when writing data with saveToV1Source(), the repartition method based on columns is called before writing data.
  private def saveToV1Source(): Unit = {
    partitioningColumns.foreach { columns =>
      extraOptions += (DataSourceUtils.PARTITIONING_COLUMNS_KEY ->
        DataSourceUtils.encodePartitioningColumns(columns))
    }

    // Code path for data source v1.
    var dfx = df
    if(needRepartitionBasedColumns) {
      dfx = df.repartition(partitioningColumns.toArray.flatten.map(df.col): _*)
    }

    runCommand(dfx.sparkSession, "save") {
      DataSource(
        sparkSession = dfx.sparkSession,
        className = source,
        partitionColumns = partitioningColumns.getOrElse(Nil),
        options = extraOptions.toMap).planForWriting(mode, dfx.logicalPlan)
    }
  }

Why are the changes needed?

This new functionality is possible to help many Spark users who use partitionBy when writing data with DataFrame.
Does this PR introduce any user-facing change?

Basically not. This new functionality enables user to specify additional boolean parameter for partitionBy. When specifying this parameter such as partitionby(true, cols), the repartition method based on specified columns is internally called and is expected to enhance performance and to decrease memory usage when writing.
How was this patch tested?

Not yet. However, I confirmed that it works by actual testing with DataFrameReader and DataFrameWriter

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Why don't we just explicitly repartition? it doesn't look particularly worth adding another method with the parameter.


Allow module to be used with PS-Desktop and .NET 4.7.2

Currently the module is set to support PS Core 6.0+ (since the Certificate functions are .NET Core App 2.1 only).
But it's also okay to run it with .NET 4.7.2 since, the relevant functions have been made available there also.
The psd1 should be changed to allow that.

Closed by 6cc6b2d


Unable to resolve product type 'com.apple.product-type.application.watchapp2' for platform 'macosx' (in target 'watch' from project 'Runner')

Mac_staging ios_app_with_extensions_test failing consistently  since 2021-01-08 11:51 AM (PST) | 2021-01-08 1:14 PM (PST)
[ios_app_with_extensions_test] [STDOUT] Executing: /opt/s/w/ir/cache/flutter sdk/bin/flutter build ios --no-codesign --release in /opt/s/w/ir/tmp/t/ios_app_with_extensions_testSeNnIw/app_with_extensions with environment {}
[ios_app_with_extensions_test] [STDOUT] stdout: Downloading ios tools...                                         1,860ms
[ios_app_with_extensions_test] [STDOUT] stdout: Downloading ios-profile tools...                                 1,133ms
[ios_app_with_extensions_test] [STDOUT] stdout: Downloading ios-release tools...                                    6.3s
[ios_app_with_extensions_test] [STDOUT] stdout: Running "flutter pub get" in app_with_extensions...                420ms
[ios_app_with_extensions_test] [STDOUT] stdout: Warning: Building for device with codesigning disabled. You will have to manually codesign before deploying to device.
[ios_app_with_extensions_test] [STDOUT] stdout: Building io.flutter.extensionTest for device (ios-release)...
[ios_app_with_extensions_test] [STDOUT] stdout: Running pod install...                                              6.1s
[ios_app_with_extensions_test] [STDOUT] stdout: Watch companion app found. Adjusting build settings.
[ios_app_with_extensions_test] [STDOUT] stdout: Running Xcode build...
[ios_app_with_extensions_test] [STDOUT] stdout: Xcode build done.                                            4.0s
[ios_app_with_extensions_test] [STDOUT] stdout: Failed to build iOS app
[ios_app_with_extensions_test] [STDOUT] stdout: Error output from Xcode build:
[ios_app_with_extensions_test] [STDOUT] stdout: ↳
[ios_app_with_extensions_test] [STDOUT] stdout:     ** BUILD FAILED **
[ios_app_with_extensions_test] [STDOUT] stdout:
[ios_app_with_extensions_test] [STDOUT] stdout:
[ios_app_with_extensions_test] [STDOUT] stdout: Xcode's output:
[ios_app_with_extensions_test] [STDOUT] stdout: ↳
[ios_app_with_extensions_test] [STDOUT] stdout:     note: Using new build system
[ios_app_with_extensions_test] [STDOUT] stdout:     note: Building targets in parallel
[ios_app_with_extensions_test] [STDOUT] stdout:     note: Planning build
[ios_app_with_extensions_test] [STDOUT] stdout:     note: Constructing build description
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to resolve product type 'com.apple.product-type.application.watchapp2' for platform 'macosx' (in target 'watch' from project 'Runner')
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to find sdk 'watchos' (in target 'watch' from project 'Runner')
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to resolve product type 'com.apple.product-type.application.watchapp2' for platform 'macosx' (in target 'watch' from project 'Runner')
[ios_app_with_extensions_test] [STDOUT] stdout:     warning: duplicate output file '' on task: RegisterExecutionPolicyException /opt/s/w/ir/tmp/t/ios_app_with_extensions_testSeNnIw/app_with_extensions/build/ios/Release (in target 'watch' from project 'Runner')
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to resolve product type 'com.apple.product-type.watchkit2-extension' for platform 'macosx' (in target 'watch Extension' from project 'Runner')
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to find sdk 'watchos' (in target 'watch Extension' from project 'Runner')
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to resolve product type 'com.apple.product-type.watchkit2-extension' for platform 'macosx' (in target 'watch Extension' from project 'Runner')
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to find sdk 'watchos' (in target 'EFQRCode' from project 'Pods')
[ios_app_with_extensions_test] [STDOUT] stdout:     warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.3.99. (in target 'Pods-Runner' from project 'Pods')
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to find sdk 'watchos' (in target 'Pods-watch Extension' from project 'Pods')
[ios_app_with_extensions_test] [STDOUT] stdout:     error: unable to find sdk 'watchos' (in target 'swift_qrcodejs' from project 'Pods')
[ios_app_with_extensions_test] [STDOUT] stdout:     warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.3.99. (in target 'Runner' from project 'Runner')
[ios_app_with_extensions_test] [STDOUT] stdout:     warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.3.99. (in target 'Flutter' from project 'Pods')
[ios_app_with_extensions_test] [STDOUT] stdout:     warning: The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.3.99. (in target 'device_info' from project 'Pods')
[ios_app_with_extensions_test] [STDOUT] stdout:
[ios_app_with_extensions_test] [STDOUT] stderr: Encountered error while building for device.
[ios_app_with_extensions_test] [STDOUT] "/opt/s/w/ir/cache/flutter sdk/bin/flutter" exit code: 1

https://ci.chromium.org/p/flutter/builders/prod/Mac_staging ios_app_with_extensions_test?limit=200

First failing build:
https://ci.chromium.org/p/flutter/builders/prod/Mac_staging ios_app_with_extensions_test/136
\cc @xster @jmagman

First failing build:
https://ci.chromium.org/p/flutter/builders/prod/Mac_staging ios_app_with_extensions_test/136
\cc @xster @jmagman

That test requires an Xcode build with a watch SDK, does it have that?

That test requires an Xcode build with a watch SDK, does it have that?

https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_staging ios_app_with_extensions_test/135/overview passed at 1:14PM and the next build https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_staging ios_app_with_extensions_test/136/overview failed at 3:35PM
The timing might correspond to https://flutter.googlesource.com/recipes/+/79afa0c6218114f077438233ff926c3207350507 which merged at 1:36PM, but I don't see where that would change Xcode versions?
https://github.com/flutter/infra/pull/325 merged at 4:50PM and I don't think that is the same config as the staging one?

https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_staging ios_app_with_extensions_test/135/overview passed at 1:14PM and the next build https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_staging ios_app_with_extensions_test/136/overview failed at 3:35PM
The timing might correspond to https://flutter.googlesource.com/recipes/+/79afa0c6218114f077438233ff926c3207350507 which merged at 1:36PM, but I don't see where that would change Xcode versions?
https://github.com/flutter/infra/pull/325 merged at 4:50PM and I don't think that is the same config as the staging one?

That explains why all the devicelab staging tests were passing consistently and now they are all failing.
Before https://flutter.googlesource.com/recipes/+/79afa0c6218114f077438233ff926c3207350507 the step running the test was being marked as noop and it was always green. After we landed that fix the tests started running again and failing.

That explains why all the devicelab staging tests were passing consistently and now they are all failing.
Before https://flutter.googlesource.com/recipes/+/79afa0c6218114f077438233ff926c3207350507 the step running the test was being marked as noop and it was always green. After we landed that fix the tests started running again and failing.

@jmagman watch os 7.2 sdk is included in the bundle but the test fails with the same error.

@jmagman watch os 7.2 sdk is included in the bundle but the test fails with the same error.

Test are now passing the problem was xcode did have the watchos sdk but the mac_toolchain tool was explicitly removing AppleTV and WatchOS sdks because they are not required for chromium. I'll update the instructions to build a new version of the mac_toolchain that does not remove watchos sdk.

Test are now passing the problem was xcode did have the watchos sdk but the mac_toolchain tool was explicitly removing AppleTV and WatchOS sdks because they are not required for chromium. I'll update the instructions to build a new version of the mac_toolchain that does not remove watchos sdk.


Attempts to improve macro

After reading your case study about "Function epilogue" I made some attempts at improving the code that should be generated. My goal was to generate code that doesn't need to modify the function body that the user provided (to change self to __self).
This rust code:
pub struct S<'a>(&'a i32);
pub type Arg1<'a> = &'a ();
pub type Arg2 = ();
pub type Ret<'a> = &'a i32;

impl<'a> S<'a> {
    #[no_panic::no_panic]
    pub fn f(&mut self, a: Arg1, mut b: Arg2) -> Ret {
        &self.0
    }
}

generates the following function with version "0.1.11"  of the no-panic crate according to cargo-expand:
impl<'a> S<'a> {
    #[inline]
    pub fn f(&mut self, mut __arg1: Arg1, mut __arg2: Arg2) -> Ret {
        struct __NoPanic;
        extern "C" {
            #[link_name = "\n\nERROR[no-panic]: detected panic in function `f`\n"]
            fn trigger() -> !;
        }
        impl core::ops::Drop for __NoPanic {
            fn drop(&mut self) {
                unsafe {
                    trigger();
                }
            }
        }
        let __guard = __NoPanic;
        let __result = (move || -> Ret {
            let __self = self;
            let a = __arg1;
            let mut b = __arg2;
            {
                &__self.0
            }
        })();
        core::mem::forget(__guard);
        __result
    }
}

Other macro expansions
Here are the different macro expansions I could come up with. In all of them I structured the code so that any macro names or types were in their own scope and couldn't be mentioned from user code.
Rebind self to itself

Con: makes the binding to self mutable when it shouldn't be.

impl<'a> S<'a> {
    pub fn f(mut self: &mut Self, a: Arg1, b: Arg2) -> Ret {
        let original = move || {
            // Rebind all the arguments:
            self = {
                let _self = self;
                _self
            };
            let a = a;
            let mut b = b;

            &self.0
        };

        {
            struct Guard;
            impl core::ops::Drop for Guard {
                fn drop(&mut self) {
                    // Do the thing
                }
            }
            let guard = Guard;

            let value = original();

            core::mem::forget(guard);
            value
        }
    }
}

Rebind arguments in a branch that never occurs

Con: Silences the unused_variables warning for the a and b arguments.

impl<'a> S<'a> {
    pub fn f(&mut self, a: Arg1, mut b: Arg2) -> Ret {
        let original = move || {
            // Take ownership of all the arguments:
            if false {
                loop {}

                #[allow(unreachable_code, path_statements)]
                {
                    self;
                    a;
                    b;
                }
            }

            // insert original code here:
            &self.0
        };

        {
            struct Guard;
            impl core::ops::Drop for Guard {
                fn drop(&mut self) {
                    // Do the thing
                }
            }
            let guard = Guard;

            let value = original();

            core::mem::forget(guard);
            value
        }
    }
}

Rebind arguments after closure return

Con: Silences the unused_variables warning for the a and b arguments.
Con: Can't silence unreachable_code warning for code after return.

impl<'a> S<'a> {
    pub fn f(&mut self, a: Arg1, mut b: Arg2) -> Ret {
        let original = move || {
            return {
                // insert original code here:
                &self.0
            };
            // Take ownership of all the arguments:
            #[allow(unreachable_code, path_statements)]
            {
                self;
                a;
                b;
            };
        };

        {
            struct Guard;
            impl core::ops::Drop for Guard {
                fn drop(&mut self) {
                    // Do the thing
                }
            }
            let guard = Guard;

            let value = original();

            core::mem::forget(guard);
            value
        }
    }
}

Rebind self in a branch that never occurs and shadow the other arguments

Pro: Preserves warnings on unused arguments (this warning doesn't seem to affect the self argument).

impl<'a> S<'a> {
    pub fn f(&mut self, a: Arg1, b: Arg2) -> Ret {
        let original = move || {
            // Take ownership of all the arguments:
            if false {
                loop {}

                #[allow(unreachable_code)]
                {
                    let _self = self;
                }
            }
            let a = a;
            let mut b = b;

            // insert original code here:
            &self.0
        };

        {
            struct Guard;
            impl core::ops::Drop for Guard {
                fn drop(&mut self) {
                    // Do the thing
                }
            }
            let guard = Guard;

            let value = original();

            core::mem::forget(guard);
            value
        }
    }
}

Thanks! Moving self in unreachable code is great.


Fixed generating utterances for built-in intents

Coverage remained the same at 100.0% when pulling 38f5e41293c12c31948e6f3fa64473712d08d887 on fix-speechAssets-generation into 4e63c24f0ce52cb4167456354608848dde7a0957 on master.

https://github.com/Accenture/alexia/issues/42 solved


How to access plotlyjs()

I was trying to change the height of the plot window inside pluto, but I reached nowhere. However, I saw that I can use plotlyjs(size = (700, 400)) to do it in PlotlyJS.jl.
How can I access this function through PlutoPlotly? Or better, how can I change the size of the plotting window, increase the red arrow length?

I can not find a plotlyjs function in the PlotlyJS package, are you sure @kichappa that you are not using Plots.jl with the plotlyjs backend?
In PlutoPlotly as well as in vanilla PlotlyJS you control the size (width and height) of the plot with the width and height keyword arguments that you pass to the Layout object you pass as second argument to the plot function.
So for example:
plot(scatter(;y = rand(10)), Layout(;height = 600))

Closing as this is not PlutoPlotly related and no further feedback was given by @kichappa to the proposed solution


Null not handled

Use case: reset to null a field of an entity
Issue: info.archinnov.achilles.internals.statements.PreparedStatementGenerator#generateUpdate filter null values instead of allowing to set Unset.Value so there is no way to reset the value until you set a custom "null" value and info.archinnov.achilles.internals.statements.BoundValuesWrapper#bindForUpdate should not filter null values but use UNSET probably?
Edit: confirming than removing both filter(value != null)  (BTW Objects::nonNull is usable ;)) then as a user you can reset a value to null instead of keeping the old value.

Indeed manager().crud().update() is not meant to be used to update values with null. Updating values with null means delete them.
If you want to delete some columns, why don't you use 👍

either manager.dsl().delete()...
or manager.crud().insert()....withInsertStrategy(InsertStrategy.ALL_FIELDS)

When using InsertStrategy.ALL_FIELDS, Achilles with bind all values, null included so you have the desired behavior

Hmm, if I use delete then an "update" (standard CRUD semantic) means delete+insert which sounds not really usable (and cassandra-driver doesn't imply that, achilles wrapping does), if I use insert I will get a duplicate (relatively to the "key"), so none of these solutions actually work.
Patching achilles I got it working not ignoring null fields so I think it makes sense to have it as a feature (think it means having an UpdateStrategy or just use a single CrudStrategy enum for that).
Here is the pseudo java code matching this ticket to ensure we speak of the same use case:
Model m = new Model();
m.setId(...);
m.setDisplayName("foo");
achillesDto.insert(m); // crud.insert().execute()

m.setDisplayName(null);
achillesDto.update(m); // crud.update().execute()

if I use insert I will get a duplicate (relatively to the "key"), so none of these solutions actually work --> wrong, new values will override old value
You don't understand insert semantics of Cassandra
Model m = new Model();
m.setId(...);
m.setDisplayName("foo");
m.setXXX("xxx");
achillesDto.insert(m); // crud.insert().execute()

m.setDisplayName(null);
achillesDto.insert(m); // crud.insert()..withInsertStrategy(ALL_FIELDS).execute()

The 2nd insert will:

delete the existing foo value (inserting null == delete)
the column XXX will be re-inserted again with same value xxx

Thought so but if i do:
@Test
public void tmp() {
    final Role role = new Role();
    role.setName("1");
    role.setUuid("1");
    repository.insert(role);
    role.setName("2");
    repository.insert(role);
    System.out.println(repository.count()); // select count(*) from role
}

count == 2 so the delete/insert is not reliable, or did i miss a flag?

What's the schema ? And what does repository.insert() ? It's not Achilles API here

schema:
CREATE TABLE IF NOT EXISTS test.role(
	uuid text,
	name text,
	PRIMARY KEY(uuid));

the repo just does:
roleManager.crud().insert(role).execute()

Normally the select count(*) from role should return 1 because the uuid column has not been changed between the 2 inserts. Can you do just a truncate role just before executing the test to be sure there is not side-effects from other tests ?

the test setup creates a dedicated cassandra instance in a temporary folder so it really starts from a plain empty cassandra

Just enable DML logging to see the CQL statements generated by Achilles and the bound values

forget it, very no luck case. Just found we have a default role so the 1+1 is not insert+insert of the test but (default)+(test).
Sorry for the noise. We can close this one and a big thanks for the help and patience.


Fix rubocop

Thanks for fixing this! Could you maybe write a test to verify it's parsing diagnostics correctly? There are insturctions for doing so in CONTRIBUTING.md

I tried to add tests, but this is a bit tricky as we'd need to install bundler, run bundle install before really executing bundle exec rubocop...
Is it possible to configure a linter/formatter partially? If so we could change the cmd/args to not use bundler by default. 🤷

I tried to add tests, but this is a bit tricky as we'd need to install bundler, run bundle install before really executing bundle exec rubocop...

You can just put the table in the config, change the cmd, test that it's ok, and put the string back. That should ensure the config at least works.

Is it possible to configure a linter/formatter partially? If so we could change the cmd/args to not use bundler by default. 🤷

One could require the parse field from the repo I guess🤔, but it's up to you, if you think more people use rubocop not installed by bundler feel free to change that.

I decided to not use guard.nvim anymore. Thanks for the plugin, but it's not for me.


Fix ambiguous calls to type_id() preventing compilation on 1.35-nightly.

Just ran into this after compiling a project on nightly that integrates Rhai.

Yeah we found the same thing, but this is a superior way of doing it.

Thanks!


error: undefined function date_select/3 in v0.11.4

Is it just my env or version 0.11.4 already removed the form functions to the new library, even though that one is not in hex yet?
ssl_verify_fun 1.1.7
  statistics 0.6.3
  surface 0.11.4
  sweet_xml 0.7.4
  swoosh 1.16.4
  table_rex 4.0.0

error: undefined function datetime_select/3 (expected Surface.Components.Form.DateTimeSelect to define such a function or for it to be imported, but none are available)
    │
 89 │     ~F[{datetime_select(@form, @field, @opts)}]
    │     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    │
    └─ lib/surface/components/form/datetime_select.ex:89: Surface.Components.Form.DateTimeSelect.render/1

== Compilation error in file lib/surface/components/form/datetime_select.ex ==
** (CompileError) lib/surface/components/form/datetime_select.ex: cannot compile module Surface.Components.Form.DateTimeSelect (errors have been logged)

    error: undefined function date_select/3 (expected Surface.Components.Form.DateSelect to define such a function or for it to be imported, but none are available)
    │
 77 │     ~F[{date_select(@form, @field, @opts)}]
    │     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    │
    └─ lib/surface/components/form/date_select.ex:77: Surface.Components.Form.DateSelect.render/1

Form removal is just on main. v0.11.4 still has Form components (https://hexdocs.pm/surface/0.11.4/Surface.Components.Form.html) datetime_select if from phoenix_html (https://hexdocs.pm/phoenix_html/3.3.3/Phoenix.HTML.Form.html#datetime_select/3)

How do I get started with surface? it is throwing the same error when I initialize a new project

@noozo datetime_select and date_select are functions from phoenix_html ~> 3.3 that were extracted to phoenix_html_helpers on phoenix_html 4.0.0
Not sure how you didn't see an error like:
Because surface >= 0.11.2 and < 0.12.0 depends on phoenix_html ~> 3.3.1 and your app depends on phoenix_html 4.0.0, surface >= 0.11.2 and < 0.12.0 is forbidden.
So, because your app depends on surface 0.11.4, version solving failed.

:surface 0.12.0 has being released removing the direct dependency on :phoenix_html because the form components were extracted. If you still need the old Form components you can check :surface_form_helpers.

How do I get started with surface? it is throwing the same error when I initialize a new project

@sarat1669, the best way is to follow Surface's Getting Started guide. If you have any questions please reach us on Elixir's slack #surface channel, if you need an invite you can get one here.
Please note that mix surface.init is compatible with phx_new 1.7.10 and to create a new project you'll need to install phx_new with mix archive.install hex phx_new 1.7.10, it may work on newer versions but you'll need to manually patch some files.


Add all missing imports quick fix

Feature Request
For a python file:
datetime
argparse

Have a single quick fix that adds all missing imports:
import argparse
import datetime

datetime
argparse

Currently you have to trigger auto imports for each symbol individually

+1


⚠ Bofrost.es has degraded performance

In d2e8e77, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1534 ms

Resolved: Bofrost.es performance has improved in c802a84 after 6 minutes.


Clean Up Imports and Settings Files

Hey Y'all,
This is mostly a cleanup PR for some imports and the settings.py files littering the repo.

Rename the top-level, non-django settings.py file to environment.py. It's clearly not a django file and shouldn't be confused with one. Now we just have settings.py, local_settings.py and local_settings.example.py under /cwod_site.
Remove symlinks to the top level settings.py (now environment.py) file in the solr, ingest, and parser directories. I think the original environment had python path issues so they used symlinks to fake it. They're not needed, we can import directly from environment.py
Get rid of import * from the files I modified that import environment ... devil be gone! Also bring imports to pep8 and remove unused imports.
Cleaned up several constant imports from settings/environment that were undefined.  I don't know what the values should be for all these variables but I have made educated guesses. These include: SOLR_DOC_PATH, DB_PATH, DB_PARAMS, and BIOGUIDE_LOOKUP_PATH. These variables are all unrelated to each other and refer to separate data sources. DB_PATH points to an unknown sqllite database. DB_PARAMS is used to connect to the Django MySQL directly instead of using the ORM. BIOGUIDE_LOOKUP_PATH points to a file with pipe-separated bio data about legislators. Lastly, SOLR_DOC_PATH seems to point to a directory where debug output (I think) is placed before writing to solr in the ingester.
Clean up solr lib DB imports, now included in environment.py. Some of these variables, as mentioned in 4), were previously undefined and point to data we don't have in the repo e.g. DB_PATH and BIOGUIDE_LOOKUP_PATH. BIOGUIDE_LOOKUP_PATH doesn't look like it's strictly required (it seems to be used as a backup source for data). DB_PATH on the other hand is used to connect to a sqllite database that is mentioned no where in the code base other than when we try to connect to it in solr/lib.py:volume_lookup(...). This is still a mystery to me. Since we don't have data about these two variables I have set them to /dev/null for now.

@will-horning @rmangi @manny

Just pushed a commit which actually fixes the issue with being able to import from the toplevel settings.py/environment.py which led someone to create the symlinks. I thought I had fixed and I hadn't, still couldn't import. I had to add /cwod to the python path. I thought it would already be importable but I guess not.

Also added export to Vagrantfile for export DJANGO_MODULE_SETTINGS=cwod_site.settings

LGTM

Note: merging this now, you may need to re-provision your VM!


GP Repo needs to include Range Dome Analysis tools

Add Range Dome analysis tools

This has been done in the following pull request: https://github.com/Esri/solutions-geoprocessing-toolbox/pull/460. @mfunk please verify.

@topowright  per #460 these tools need a lot of work to get them into the repo. I don't have an exhaustive list of stuff to do as I don't have a copy for 10.4.1 (yet). When you're back in the office we'll have to meet for an hour or so and make it work.


[FEATURE] Add confirmation modal

Description
When a user clicks to log out, there should be a modal asking if they truly want to log out.
This is so that if they accidentally click the logout button, it asks if they are sure they want to log out.
Screenshots
No response

I'll handle this @Dun-sin

@Dun-sin, I can work on this!

Hey @Dun-sin  you can assign me this issue , i will solve this.


Bugfix: avoid double invocation, add exception handling

The double invocation may be caused by the wrapped android library.
I added a flag that will indicate if the callback had already been invoked as safeguard, furthermore wrapped it in a try catch clause to avoid the app crashing

@prscX could you have a look at this PR please?

@prscX could you please have a look at the open PRs?

@prscX could you please have a look at the open PRs?

@danielwinkler: I have made the fix using another approach.
Could you please try latest release V1.0.3. Let me know if still does not work.
Thanks
</ Pranav >

@danielwinkler: I have made the fix using another approach.
Could you please try latest release V1.0.3. Let me know if still does not work.
Thanks
</ Pranav >

Looks nice, i will let you know if I still encounter a problem
Thanks!

Looks nice, i will let you know if I still encounter a problem
Thanks!


Error running  the docker image

Greetings,
I'm trying to build the docker image but it is taking longer time to build the image of the LongQC and it automatically shuts down like nothing was run. Even if I manage to run it, I see an empty longqc text file in the minimapcoverage folder. I'm not sure if this is the way it should work. But, once I try running docker with the built image. It says the longqc:latest is not found. I suppose the longqc is not building properly. Additionally, it also gives me an the below error.
Error response from daemon: pull access denied for longqc, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

Hi @PavithraV0223,
Thanks for your interests in our app.
Looks like your building failed due to some reason. Did you download the Dockerfile and then build the image against it?
wget https://raw.githubusercontent.com/yfukasawa/LongQC/master/Dockerfile
Yoshinori

Self-follow-ups. Docker limits computational resources, and it kills the process internally if the resource such as RAM is not sufficient. If this is the case, simple workaround is increasing the resources (e.g., more RAMs, CPU cores, etc.)

After monitoring, it seems that the issue has been resolved.
As a result, I will proceed to close it.


How to get other claims besides sub, preferred_username and name?

I tried to map the picture claim in the profile scope but it wasn't getting saved. Only the sub and username properties. So, I checked the console and found this:
steam_1     | [17:00:21 Debug] IdentityServer4.ResponseHandling.UserInfoResponseGenerator
steam_1     | Requested claim types: sub name family_name given_name middle_name nickname preferred_username profile picture website gender birthdate zoneinfo locale updated_at
steam_1     |
steam_1     | [17:00:21 Debug] IdentityServer4.ResponseHandling.UserInfoResponseGenerator
steam_1     | Scopes in access token: openid profile
steam_1     |
steam_1     | [17:00:21 Information] IdentityServer4.ResponseHandling.UserInfoResponseGenerator
steam_1     | Profile service returned the following claim types: sub preferred_username name

Apparently the picture and other claims are being requested but only sub, preferred_username and name are being returned.
Is it possible to get those claims?

Valve does not expose the other claims, hence it is not possible.

What if we use https://api.steampowered.com/ISteamUser/GetPlayerSummaries/v0002/?key=&steamids to manually insert the other claims?
It would give us the picture, given_name, nickname and maybe others claims. I could do it on my app but it would be great if it was just managed by the provider.

Sadly I do not have the time to work on that, but I would happily accept contributions for it.


After randomizing gates Grid Seq Gates on/off work weird

You need to double-click to turn them on again.
Off still works as expected, and once you turned them on they also work as expected.
W10  JW v1.0.12
Greetings Peter.
Also mentioned here : https://www.facebook.com/groups/vcvrack/?post_id=608543173139164&comment_id=612123172781164

ya i looked into it a little and fixed one thing after my latest release but i don’t think its all the way fix yet - also apparently been there a while #12 🤣

Yeh, it's an oldie but a goodie (had not seen #12 )

ok just pushed up a fix - let me know if you can build or need a mac build

should be fixed - submitting to library


Initial doc changes

I've worked over primarily the 'Concepts' page, for starters.  Also changed some of the header styling to something I think works better.
Might have gotten carried away.
Look it over and we can discuss.
Do you have a git Project set up for this?
thanx,
rickb

@rickbsgu thank you for the changes I think they are great. I will merge and only remove some of the styling changes as it will be part of a bigger adjustment which we can discuss.

Hey, Leo -
Sounds good.
The styling changes were ad-hoc just to provoke some thought.  They would
have to be done better, especially the outdents.  I need to study nextjs to
understand how it handles responsiveness.
I'll merge back and tackle the other pages content.
My goal is to understand the package.  Figure helping out with the docs
would be a good way to do that.
Cheers,
rickb
On Mon, Dec 27, 2021 at 11:05 AM Leo Farias @.***>
wrote:

Merged #28 https://github.com/leoafarias/mix/pull/28 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/leoafarias/mix/pull/28#event-5818730648, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABHNQVDO5BV66XZZAV3TTTUTCTFHANCNFSM5KZKP22Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


Peter update compare

fix aws aurora options

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9b8896963369d3d2864df70392fb7d897f697159/
Edited pages:

v20.1/cockroachdb-in-comparison.md
v20.2/cockroachdb-in-comparison.md

@anagio, do you need a review on this?

Preview looked good to me
On Sat, Oct 3, 2020, 12:55 PM Jesse Seldess<EMAIL_ADDRESS>wrote:

@anagio https://github.com/anagio, do you need a review on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cockroachdb/docs/pull/8522#issuecomment-703132574,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIFTQJVWZ7GWQP4Y44XLALSI5JRJANCNFSM4SATHOOA
.

LGTM

@anagio, I'll merge this.


feat/web-socket-client

This PR is for creating web-socket client

[x] new utils for web socket
[ ] add new option for web socket client.
[ ] add new client template to clientFactory.
[ ] a function to check/modify the url for web-socket (e.g: rpc.osmosis.zone => wss://rpc.osmosis.zone/websocket)
[ ] open/close connection function
[ ] a function that return message from server ( as Json )

Good morning, @Zetazzz! What do you think about the functions that I listed above? Do you think they are sufficient for now?

@Zetazzz that's great! Let's rock it 😆

@Zetazzz that's great! Let's rock it 😆

And there's one more thing, can you describe some stories when user's using our generated code, hopefully those user stories can cover most of the cases, if something's not sure yet, we can put a ? after it. Like:
users wanna use pre-generated events based on some files?
users wanna see there're several pre-set ws instances for every service(link), and users can access those instances by namespace.
For each service:
users also wanna subscribe queries by themselves.
users also wanna unsubscribe queries by themselves.
users also wanna unsubscribe all

@Zetazzz  I think we'll have to look at where we get all kind of events from the chain. I think that for each chain there will be (ofc base cosmos events) and then we'll have other chain custom module/event. creating a list of those events for each chain would be really useful for users I think. Also your suggest on each services is really good, I think we'll go with it first.

@Zetazzz hey, I did some research on events and here is what I found. we'll have two source for getting list of events https://docs.cosmos.network/v0.46/core/events.html

I think we should get all the typed events working first and then working on event.proto later on. wdyt @Zetazzz ?

@hoangdv2429 Hi, I just checked in generator codes, I also fixed export in ast package, please make sure to get and merge the latest change! Thank you very much!

Close this for now, since Websocket exists in cosmjs.


API URL can now be re-configured (bug 1136830)

This also makes settings initialization lazy which
solves a lot of other subtle issues I've seen, mostly
relating to the adapter.
@muffinresearch r?
The Fireplace adapter needs fixing before this code is in use by Fireplace: https://bugzilla.mozilla.org/show_bug.cgi?id=1136843

r+wc assuming there's not a more direct way of doing this that might mean there's less chance of missing the calls to settings.initialize();

See my last comment the log code needs tweaking - also would be good to have a test that covers this.

I think this is r- presently, it needs a bit of a re-think. It's not working in practice.

Some more info - probably requires a coffee before reading :)
Commenting out the opt.reset if block got this working.
Another issue I had is that I was running fxpay.configure before fxpay.init and I wasn't passing the baseApiUrl to the fxpay.init. This resulted in the confguration happening twice.
I double checked that this bug still happens without the fxpay.configure call if the right baseApiUrl is passed to fxpay.init and it does still occur.
However, I think we could use a warning re: running fxpay.configure before fxpay.init.

Ok the latest version is working nicely for me. See http://webfighter.muffinresear.ch
r+wc


Collab

Hey there,
I found your tool by chance while searching for pomodoro or something on crates.io.
I'm currently building https://github.com/pace-rs/pace because I felt my use case for tracking work and break times is not fulfilled by a lot of tools out there. I see that you also spend a lot of time thinking about time management techniques and I like what you did with work-break as it inspired me for some ideas for pace.

Doing the same work should be discouraged. Working together should be applauded.

Based on that, I wanted to ask if you feel like joining the cause and help to implement pace. :) There is a discord link in the repository, if you want to discuss further, would be happy to see you there.
Cheers,
Simon

Hey, @simonsan ! Thanks for reaching out. My use case working full-time is completely fulfilled with work-break, so I'm not inspired to work on something else now.


Error after upgrade to Statamic 5.x

With the latest version of Iconify and updated to the latest version of Statamic I get an error using the addon:
 [internal function]: Statamic\\Tags\\Tags->__call('index', Array)

I can't really reproduce this. Could you send more info or the full stack trace? What are you doing exactly to trigger this error?

Here is the trace: https://flareapp.io/share/qm1gQpz7
I only upgraded to Statamic 5, and then the frontend page, which shows an icon, throws this error. In the backend, it is working okay.

This should be fixed in v1.3.0. Thanks for reporting!

It seemed the same error. Didn't take a good look at it. Thanks for the quick fix!


added section on how to update TZIDs to match the timezones from the IANA db

created a section with info on how to update TZIDs to match timezones from the IANA tz db.

Looks like this got a bit stale.  I'm going to go ahead and close this.  Please reopen if you're still interested in working on this.


CSS fix, should use background shorthand, instead of background-image

Please merge. This was not fun to figure out the hard way.

I know but I have no pc right now, skiing first ;)


Video recording issues (choppy output)

Is there any way to fix a video screen recording that's choppy? I am recording my screen on a Note 3 and no matter what bitrate or resolution I use, although the quality is perfect, there is choppiness when recording gameplay such as Clash of Clans. Any suggestions for fixes that I could possibly do on my device's end?

Unfortunately it sounds like your device doesn't have CPU cycles to do both recording and a potentially performance heavy game


Nissix plugin scope-packages on svg2react-icon

Hi, I'm Nissix, the automated PR bot!
Wix is moving its internal packages to the @wix scope (but still in the internal registry). Publishing new unscoped internal packages is no longer allowed, and all existing packages are moving to @wix. This means changing package names, and also all usages of those packages to their @wix scope version.
This PR is an automatic codemod that moves all of your packages to @wix scope, and changes any usages (pacakge.json, imports, requires, etc..) to their @wix version.

:bangbang:
This codemod is best-effort! Meaning it may not have found and fixed all usages, but it did change your package.jsons. So go over the changes carefully and test this version carefully

If you want to know why we don't support publishing unscoped to the internal registry, check out this article on Dependency Confusion
If you are unsure, need help or have questions, reach us at #wix-scope-migration
Error Log:
npx: installed 234 in 6.468s
Output Log:
Migrating package "svg2react-icon" in .
Migration from non scope to @wix/scoped packages

/tmp/3a61047a2b4c586305dd54e4e9a3bc99

rename package.json dependencies/dev/bundled/peer/optional, jest & eslintConfig
package.json

replace import/require in js/ts files
wallaby.js

Already merged


generate-rpcwrapper config file flag clarification

Is your feature request related to a problem? Please describe.

I'm always frustrated when I use cli and do not know for sure from the help what argument should be passed.
NAME:
   neo-go contract generate-rpcwrapper - Generate RPC wrapper to use for data reads

USAGE:
   neo-go contract generate-rpcwrapper --manifest <file.json> --out <file.go> [--hash <hash>] [--config <config>]

OPTIONS:
   --config value, -c value    Configuration file to use
   --manifest value, -m value  Read contract manifest (*.manifest.json) file
   --out value, -o value       Output of the compiled wrapper
   --hash value                Smart-contract hash. If not passed, the wrapper will be designed for dynamic hash usage
   --help, -h                  show help

Why does passing --config  yield different results than not using the flag, although this isn't mentioned in the description. Also, I do not know if this config file is the same as it is in neo-go contract compile where at least an extension is mentioned (Configuration input file (*.yml), but the flag is the same)
Describe the solution you'd like

Example or more details about flags in the command description.
Describe alternatives you've considered

do nothing with it. be more attentive and read docs and workshop repo.

Long-term with NEP-25 implemented we will drop this configuration file. Short-term, if we can quickly make it fail because of unknown fields --- that'd be the best solution.


seem in complete the folder.. where is the sprite / icon ?

icon not included. sprite css also.. i prefer not to link to cdn everytime.

This plugin requires emojione, so you must include emojione/lib/emojione.min.js and emojione/assets/sprites/emojione.sprite.css in your HTML.
If you include these files from CDN, the image/sprites will use CDN, too.
You can download the emojione and include it from your own host.
If you want to use images instead of the sprite,  you don't need to include sprite.css, but must set the path to the images, like this:
$(document).ready(function() {
	emojione.imagePathPNG = 'basedir/to/emojione/assets/png/';
});

thanks for info..

I think I understand what you mean.
From v1.0.1 you will not need to load emojione, only emojioneArea.

I see that you are still using emojione.js on your demo, but this is not anymore required right?
only this two files:

and optionally emojione.sprite.css if sprite image is used?
Thanks

Can not having dependency on CDN be the default configuration?  Or at least expose an option in emojioneArea to switch between emojione CDN and local hosting?

For turn off CDN of emojione, (ref https://github.com/mervick/emojionearea/issues/80)
jQuery(document).ready(function() {
    emojione.imagePathPNG = 'wherever/you/want/assets/png/';
    emojione.imagePathSVG = 'wherever/you/want/assets/svg/';
    $("#example1").emojioneArea(
        {
            useInternalCDN: false
        }
    );
});


Facts need to contain more info about the user

Is your feature request related to a problem? Please describe.

Describe the solution you'd like
I want facts to contain actually useful information such as
Age, city of living, marriage status, health, friends names, preferences, allergies. Imagine an FBI report on a person - that's exactly how it should be.

This is just prompt work, you don't need to change UI

done


[Improve] Improve streampark-spark module based on [3.3 Methods Rule]

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
[Improve] Improve streampark-spark module based on 3.3 Methods Rule
Usage Scenario
No response
Related issues
#2986
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

N.A


Improving the Design of the Cards and the Specific Calculators

What's the style idea?
I will improve the cards witha a backgroung effect and a drop shadow. I will also design specific calculators page which  seems to be very normal . I will make them designer
Please Assign this issue to me & to add Gssoc , level1 labels
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

hii @Rakesh9100 ,
I'm GSSoC'24 contributor , I have experienced in HTML, CSS, JS, and react.js , so I want to work on this issue,
so, kindly requested to you to assign me the issue.

Do you have any supporting screenshot @jasjeev013


dpl: don't touch fixed cells inside groups

Fixes https://github.com/The-OpenROAD-Project/OpenROAD/issues/5204

Looks good, do you see any other places that need the same?

This was the only place with the missing check in DPL. I inspected the code for possible causes of the issue, and the other places already checked if the Cell is a fixed inst.