Multi tcb info fields validation with derived context

Hi @jhand2
This commit has :

MultiTcb information has field-wise validation (apart from checking structure)

Verify by deriving more child contexts in non-simulation mode and simulation mode.

Check VendorInfo  by validating against target locality
Check Type   by passing Tci type to derived child context
Check Fwids[n].hashAlg  by validating against expected hash alg OID SHA384/ SHA256 based on DPE profile
Check Fwid[0].Digest  by passing a TCI value to child context and checking if reflected in TCI_CURRENT of child TCB info
Check Fwid[1].Digest  by calculating cumulative TCI and cross verification

Aside from above checks, have moved helper functions that operate of certificate and cert extensions to certs.go for better readabilty of certifyKey.go. The helper functions will just return errors and caller must handle/log errors using t.Error()

Check if multitcb info fwidarray length is 1 when extendtci is not supported by DPE profile.
Specification contents

{
	Vendor: "",              // Optional and VendorDefined
	Model: "",               // Optional and VendorDefined
	Version: "",             // Omitted
	SVN: 0,                  // Omitted
	Layer: 0,                // Omitted
	Index: 0,                // Omitted
	Fwids: [
		{{Digest : [], HashAlg:OID},   // TCI_CURRENT
                {Digest : [], HashAlg:OID}],   // TCI_CUMULATIVE  omitted when extend_tci is not supported
	]
	Flags: NotConfigured (0), 	// Omitted
	VendorInfo: [0,0,0,0], 		// TARGET_LOCALITY parameter
	Type: [0,0,0,0]}	       // Vendor defined / input (tci_type) provided by a client to DeriveChild
}

@jhand2 Can this be merged?


[fi] Added new Finnish translations

Replaced Finnish form validation placeholders with translations

A problem of copy-paste ?


⚠ Bofrost.de has degraded performance

In 0e75e76, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2021 ms

Resolved: Bofrost.de performance has improved in e916d7e after 48 minutes.


Lane Detector Denseline network's output

Hi everyone,
I'm a student at Scuola Superiore Sant'Anna in Pisa, Italy.
I'm trying to understand the output of the Lane Detector Denseline caffe model, in the perception module (https://github.com/ApolloAuto/apollo/tree/master/modules/perception/production/data/perception/camera/models/lane_detector/denseline).
After the last layer (the conv_out layer), the network outputs a 9 x 96 x 96 tensor.
Could anyone explain me what is it and how I can visualize the results?

@JerrySciangula The 9-channel feature maps are further postprocessed to get the final results. For details you may refer to https://github.com/ApolloAuto/apollo/tree/master/modules/perception/camera/lib/lane/postprocessor/denseline

Ok thank you @lishaohua01


beta({mean: 0.39, stdev: 0.1}) #497 (no tests or examples)

tiny PR. @Hazelfire is this what you had in mind? Closes #497

Surely this is pretty easy to test? mean(beta({mean: 0.39, stdev: 0.1})) == 0.39?

Mean looks good, stdev maybe questionable?
One thing to do is to throw errors for invalid inputs, which are easy to get in this case. (Beta is much more limted than other dists; mean and stdev are limited.)

One thing to do is to throw errors for invalid inputs, which are easy to get in this case.

Some light algebra shows that the Beta.make validator catches invalid mean and stdev.

What end user behavior from your screenshots do you find undesirable?

The main thing is that the stdev just seems off. Either we're calculating stdev wrong elsewhere, there's something iffy with the toSampleSet, or the function isn't working correctly.

Math seems right, but I would add the validity checks sooner so that we get errors which are a bit more informative than "Beta distribution parameters must be positive" when inputting alternative parametrization schemes.

Great, it seems safer now. Linear from stdev = ~0.01 to ~0.47, which I assume is expected.


Update compliance.rst

Confirm UI steps against 5.18 UI and fixed minor formatting/typos.

Preparing to merge v5.18 doc branch into master. Please open a PR into master branch for any additional edits / changes.


feat: support splitting declarations

The solution just borrows some code from let transformer.
Some basic tests are added.
CONS: it doesn't process inline comments perfectly as there exists various types of inline comment usage.
Related to #158.

Thanks for the PR.
I've added some comments.
I think the comments support is fine for now - the common cases are covered. I'll have to take a deeper look into the comments handling in general - there are lots of edge-cases that aren't covered in various transforms.
My main concern is the name of this transform. I think split-declaration is too generic as there are other declarations in JavaScript besides VariableDeclaration. Perhaps instead split-var although it also deals with let and const. Another thing is that the names of other transforms are more declarative in nature, but "split" sounds more procedural. So perhaps instead of asking Lebab to split variables, we'd ask it to ensure there are multiple variables are declared with multiple declarations (not concerning ourselves whether it needs to split the declaration or do whatever else to achieve that). My choice of name would therefore be multi-var (which would also align in with one-var rule in ESLint).

Thanks for your careful comment.
As almost a newbie to some programming paradigm(declarative programming), your guidance is helpful.
Agreed with your concern.

There is one inconvenient equipment I found.
The registry equipment is not central.Then, once you create a new transform, you have to modify several places to make it work.

Thanks for the changes.

I think you misunderstood my point about multi-line strings - there's no need to have additional tests to ensure we properly handle the newlines (this is largely handled by Recast), rather I wanted to split the code to multiple lines for readability purposes.
Also just one little sanity test would have been enough - there's very little benefit from this test anyway.

I've corrected these things, added some docs and pushed to master.

Regarding the inconvenience of adding new transform...
This is indeed in need of some improvement. It's been somewhat of a pain. I've created an issue for that: #162


Add sastodeal.com

Nepal's own E-commerce.

Added description. You can accept it now.

Thank you 👍


Too many files error

I tried using the tool on my project, and get the following error message many times:
Error: too many open files

My node_modules is giant, and the ulimit workaround just isn't enough. Would an exclusion option be a good way to handle this?

@coltonmorris Interesting. We now set the ulimit automatically at startup to be the biggest possible. Is it possible to check the version of iBazel you're running?

0.5.0.
I've also tried manually setting it.

@coltonmorris In your case, setting ulimit -n to the hard limit (which is what ibazel does automatically) may not have been sufficient to cover the number of files in your node_modules. For example, on my Ubuntu system, the default soft and hard limits are 1024 and 4096, respectively, so a non-root user can't set it any higher than 4096. Did you try changing the hard limit? On Linux, this just requires editing /etc/security/limits.conf as root; if you're using a Mac you should look for instructions that are specific to your macOS version.

@coltonmorris I assume that you're using rules_typescript. If not I would need to find out how you're using this tool. If you are, can you try using the new "idiomatic" npm dependencies version rather than self managed? I think that is where they want things to end up, and it means that there are several orders of magnitude fewer files to watch and it might just work. Along with some other changes made recently, you might have more fun using iBazel now.

We now watch directories instead of files so this shouldn't be a problem any more unless you have a pathological codebase. Please reopen if you have more issues


Error: Cask 'omnigraffle' is unreadable: uninitialized constant Livecheck::Formula

cu: e1aadd4c97203c0de552f0c0e00baee47604e2c7
brew: a4b4b9094054de3f2c93ad40814b51031949ab6e
brew cu pin omnigraffle against {{HOMEBREW}}/Library/Taps/homebrew/homebrew-cask/Casks/o/omnigraffle.rb
All casks are currently unpinnable.

I assume this was a temporary issue as it works fine for me.
~ ❯ brew cu unpin skype
Unpinned: skype
~ ❯ brew cu pin skype
Pinned: skype in version <IP_ADDRESS>

In case it's still broken, let us know here.


Add support to test gpu jaxlib nightly in CI instead of prebuilt jax/jaxlib

Replace the prebuilt jax/jaxlib container with a container built with
an OSS dockerfile cuda 11.4/cudnn 8.2.4
Add additional flags to make NCCL work with bare ethernet and
pass them to srun command
Modify job name, deadline to make it more appropriate to run on
the cluster

LGTM. You have linter failures in the _test.py file.

Ah I see, I just added this code from distributed_test.py but didn't know why it's needed. Is it needed for this test to run?
absltest.main(testLoader=jtu.JaxTestLoader())

@yashk2810 could you please help in merging this? I'll then schedule a run to see if it works

Yeah, merging process started. It should get merged soon.

Does it usually take so much time?

No, its pretty fast but I am just waiting for an LGTM from a googler and since its Friday, I guess many of get are done for the week. Even my CLs are pending (lol).

It's done. Thanks!


Alpha 2

Just a note, converting a tuple to and from a list is very fast as it is an internal BEAM instruction.  :-)
Could simplify some of the tuple code.  :-)
Also for Ord, have you thought of having a default that just falls back to normal BEAM Term ordering?  That way users can override it for their custom types but by default it uses the BEAM's standard method?


[stable/graylog] Fix multiple TCP NotePort support

fixes #14007
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@gschanuel please sign-off your commit

Hi.. i'm really new to PRs, what should I do now?


[Feat] Improve repo discoverability

The repo is not easily discoverable at this point thus making it hard for developers to use the components hosted here.
Suggested solution
Include all components hosted here in the core repository storybook.

We could explore storybook composition for this one
I quickly tried it out and it works, well, kinda:

More investigation is necessary:

potential issues with different storybook versions ( contrib is on 7,  core on 6 - this will change https://github.com/microsoft/fluentui/issues/30858)
need to double check if docs mode is supported with composition ( otherwise we cannot use it in order to maintain same UX )
aggregate contrib storybooks under 1 common SB in order to be able to use 1 deploy link for all contrib storybooks

Added another option to the list:
Option 1: Create a static page(s) in public-docsite-v9
This option involves creating static pages that list all packages with short descriptions and links to standalone Storybooks.
Pros:

Simplicity: This approach is straightforward and easy to implement. It involves creating static page(s) that list all packages with short descriptions and links to standalone Storybooks.
Low Maintenance: Once the static pages are created, they require minimal maintenance. Updates to individual Storybooks do not necessitate changes to the static pages.
Separation of Concerns: Each package maintains its own Storybook, reducing the risk of conflicts and making it easier to manage individual Storybooks.

Cons:

User Experience: Users need to navigate away from the main site to view the Storybooks, which can be inconvenient and disrupt the user experience.
Inconsistent Look and Feel: Since each package maintains its own Storybook, there might be inconsistencies in the visual appearance and user interface across different Storybooks.
Scalability: As the number of packages grows, maintaining and updating the static pages can become cumbersome.

Option 2: Storybook composition
This approach composes stories for all fluentui-contrib packages into one website (Storybook in docs only mode, re-using the same Storybook presets and plugins from the core).
Pros:

Unified Experience: Composing stories for all fluentui-contrib packages into one website provides a unified and consistent user experience.
Reusability: Re-using the same Storybook presets and plugins from the core ensures a similar visual appearance and functionality across all packages.
Centralized Management: Managing all stories in one place simplifies Storybook updates and maintenance.

Cons:

Complexity: This approach is more complex to implement and time consuming, as it requires updating existing stories and changing current storybook setup. Also for unified experience we need to copy-paste or publish or re-use Storybook presets and plugins from public-docsite-v9

Option 3: Publish stories packages from contrib and consume those packages within core public-docsite-v9
Pros:

Modularity: Publishing stories as packages allows for modular and reusable components. Each package can be independently updated and maintained.
Integration: Consuming these packages within the core public-docsite-v9 ensures a consistent look and feel while leveraging the existing infrastructure.
Scalability: This approach scales well as new packages can be easily added without significant changes to the core site.

Cons:

Initial Setup: Setting up the infrastructure to publish and consume story packages requires initial effort and configuration.
Dependency Management: Similar to the second option, ensuring that all packages are compatible with the core site can be complex.

option 1 won't scale / nor will fix existing issues with contrib "docs state of the art"
option 3 is unnecessary "javascript golfing" without any real benefit ATM
option 2 is what we want

current state of contrib "docs" is beyond ideal, inconsistent, no "documentation context" and scattered  around various urls, not automatically exposed to public and the list goes on

example of current "docs" for chat component
we don't wanna expose something like this as a part of official core library

contrib docs should follow the same high quality standard of documentation
we should aim to get something like https://react.fluentui-contrib.dev/ working future-wise
once we have application for "docs" setup in contrib, it's matter of 1 time configuration in core public-docsite-v9 application via storybook composition which will propagate without no further need of maintenance

ty


[Bug] [kafka source] checkpoint 设置不生效

Search before asking

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

What happened
kafka source 配置HDFS 的checkpoint检查点，不生效。

程序中断，在重新启动，插入mysql[primary_key]参数不设置的情况下，会重复，，看日志，offset会重复消费，从0开始。hdfs并没有记录相关检查点

我确信有写入权限
SeaTunnel Version
dev分支：2.3.2-SNAPSHOT
SeaTunnel Config
env {
    parallelism = 5
    job.mode = "STREAMING"
    job.name = "DZFP_DZDZ_DZDZ_FJXX_PTFP"
    checkpoint.interval = "60000"
    #checkpoint.data-uri = "hdfs://localhost:8020/flink-checkpoints"
}

source {
    Kafka {
        result_table_name = "DZFP_DZDZ_DZDZ_FJXX_PTFP1"
        topic = "DZFP_DZDZ_DZDZ_FJXX_PTFP"
        bootstrap.servers = "localhost:9200"
        commit_on_checkpoint = "true"
        format_error_handle_way = "skip"
        schema = {
            fields {
                 message =  {
                    data = "map<string,string>"
                 }
            }
        }
        #semantics = EXACTLY_ONCE
        start_mode = "earliest"
        #start_mode.offsets = {
        #     0 = 0
        #     1 = 0
        #     2 = 0
        #}
        kafka.config = {
            auto.offset.reset = "earliest"
            enable.auto.commit = "true"
            # max.poll.interval.ms = 30000000
            #max.partition.fetch.bytes = "5242880"
            #session.timeout.ms = "30000"
            request.timeout.ms = "60000"
            buffer.memory = "33554432"
            #client.id = client_1
            max.poll.records = "100000"

        }
    }
}
transform {
    sql{
        source_table_name = "DZFP_DZDZ_DZDZ_FJXX_PTFP1"
        result_table_name = "DZFP_DZDZ_DZDZ_FJXX_PTFP2"
        # qdmx自定义UDF函数
        query = "select qdmx(message,'xsfdzdh') as xsfdzdh,qdmx(message,'gf_qxswjg_dm') as gf_qxswjg_dm,qdmx(message,'sblx_dm') as sblx_dm,qdmx(message,'gmfdzdh') as gmfdzdh,qdmx(message,'hzxxb_bh') as hzxxb_bh,qdmx(message,'mac') as mac,qdmx(message,'sjczbs') as sjczbs,qdmx(message,'xsfyhzh') as xsfyhzh,qdmx(message,'cxswjg_dm') as cxswjg_dm,qdmx(message,'fpbb_dm') as fpbb_dm,qdmx(message,'skm') as skm,qdmx(message,'kpjh') as kpjh,qdmx(message,'kpr') as kpr,qdmx(message,'ydk') as ydk,qdmx(message,'jsswjg_dm') as jsswjg_dm,qdmx(message,'ydfpbz') as ydfpbz,qdmx(message,'skr') as skr,qdmx(message,'fpqm') as fpqm,qdmx(message,'xf_qxswjg_dm') as xf_qxswjg_dm,qdmx(message,'jsr_mc') as jsr_mc,qdmx(message,'xdjg') as xdjg,qdmx(message,'tslsh') as tslsh,qdmx(message,'fpdm') as fpdm,qdmx(message,'cslx_dm') as cslx_dm,qdmx(message,'sjtbsj') as sjtbsj,qdmx(message,'fpdmhm') as fpdmhm,qdmx(message,'fhr') as fhr,qdmx(message,'gf_sjswjg_dm') as gf_sjswjg_dm,qdmx(message,'fphm') as fphm,qdmx(message,'jsr_dm') as jsr_dm,qdmx(message,'zbxlh') as zbxlh,qdmx(message,'bs_nsrsbh') as bs_nsrsbh,qdmx(message,'gf_dsswjg_dm') as gf_dsswjg_dm,qdmx(message,'bdbz') as bdbz,qdmx(message,'xf_sjswjg_dm') as xf_sjswjg_dm,qdmx(message,'data_category') as data_category,qdmx(message,'gmfyhzh') as gmfyhzh,qdmx(message,'swjg_dm') as swjg_dm,qdmx(message,'sjbb') as sjbb,qdmx(message,'tspz_dm') as tspz_dm,qdmx(message,'wspzh') as wspzh,qdmx(message,'sbbh') as sbbh,qdmx(message,'yqbz') as yqbz,qdmx(message,'ip') as ip,qdmx(message,'kpyf') as kpyf,qdmx(message,'xf_dsswjg_dm') as xf_dsswjg_dm,qdmx(message,'sslkjly') as sslkjly from DZFP_DZDZ_DZDZ_FJXX_PTFP1"
    }
}
sink {
    #Console {
    #    source_table_name = "DZFP_DZDZ_DZDZ_FJXX_PTFP2"
    #}
     jdbc {
        source_table_name = "DZFP_DZDZ_DZDZ_FJXX_PTFP2"
        url = "jdbc:mysql://localhost:3306/sjjcb_dw?serverTimezone=GMT%2b8&rewriteBatchedStatements=true"
        driver = "com.mysql.cj.jdbc.Driver"
        user = "user"
        password = "pwd"
        batch_size = "200000"
        database = "dbname"
        table = "tablename"
        generate_sink_sql = true
        #is_exactly_once = "true"
        #xa_data_source_class_name = "com.mysql.cj.jdbc.MysqlXADataSource"
        #primary_keys = ["fpdmhm"]
    }
}

Running Command
./bin/seatunnel.sh --config config/qdmx_config/DZFP_DZDZ_DZDZ_FJXX_PTFP.conf -e local

Error Exception
no

Flink or Spark Version
no
Java or Scala Version
No response
Screenshots
no
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Can you change the checkpoint storage to local?


Add whiteboard inpainting demo

This is a whiteboard inpainting demo.
@snosov1

@DmitriySidnev do we have a suitable publicly available video which we can refer to in demo description? I think we need this as demo work with quite specific scenario (person in front of whiteboard).

@vladimir-dudnik, I can only provide link (youtube) to a suitable public video.

@eizamaliev, could you please approve this PR if you do not have any more comments?

Jenkins: test this please


UseSwaggerUi configuration overload not available

Hi,
I'm using 6.0.0-beta902 with net452. The UseSwaggerUi extension method does not accept any configuration.
In my project.json i have the following package: "Swashbuckle": "6.0.0-beta902",
When i try this, it fails as the extension method is missing:
app.UseSwaggerUi(c =>
{
c.SwaggerEndpoint("/swagger/v1/swagger.json", "V1 Docs");
c.EnabledValidator();
c.BooleanValues(new object[] { 0, 1 });
c.DocExpansion("full");
c.SupportedSubmitMethods(new[] { "get", "post", "put", "patch" });
c.InjectOnCompleteJavaScript("/ext/custom-script.js");
c.InjectStylesheet("/ext/custom-stylesheet.css");
});
This is the extension only method, that is available:
public static class SwaggerUiBuilderExtensions
{
public static IApplicationBuilder UseSwaggerUi(this IApplicationBuilder app, string baseRoute = "swagger/ui", string swaggerUrl = "/swagger/v1/swagger.json");
}
What am i missing?

Update:
I've clones the sources and added Swashbucke, Swashbuckle.Swagger, Swashbuckle.SwaggerGen and Swashbuckle.SwaggerUi as project references.
Looks like it's enough to add "net452": {}, to the frameworks section in project json of each project.

any update on this?

I had the same issue but spotted it at the bottom of the readme on the main repository.  That code is only in the preview versions on the authors MyGet repository that he provides.  The Standard Nuget repository does not have the preview version that match the code. It would be good to see the preview versions making their way to standard nuget more often or 2 copies of the samples as i wasted a day before i worked it out as well

Any ETA on when we will see a updated nuget package ?

Available with latest pre-release. Note the package rename to "Swashbuckle.AspNetCore.1.0.0-rc1"


TouchableOpacity is slow when dispatch is used

I am using react-redux with redux-toolkit
import { setLanguage } from "../../redux/main";
   <TouchableOpacity
        onPress={() => {
          dispatch(setLanguage("English"));
        }}
   >

Above code is working fine and it is updating the state globally for me, but problem is TouchableOpacity or pressable or TouchableHighlight is delayed by 500 ms to 2 seconds.
I tried delayPressIn={0} with no result.
if i remove dispatch(setLanguage("English")) , Button click is working fast.

I would recommend testing it with other press-like component like  for example. If you see the same performance issue it is almost certainly problem in your app and not react-native issue.

+1

@vishnuc
Hi bro,
Me and your code is the same How to solve your question? I've the same issue. Please help me.

have the same issue, and dont know how to fix it


Census layer: establish color schemes, classifications

Summary
Establish polygon color scheme/s for the census layer/s. Maya has some experience with this via an ArcGIS web map, so let's start from there.
Questions

Will the symbology be based on a verbatim value (e.g. number of speakers) or an on-the-fly calc like percentage (e.g. number of speakers out of total number of speakers)?

Which classification to use? The JS color lib will likely be chroma.js, which supports four class modes, so if at all possible it seems logical to shoot for one of those in the ArcGIS web map:

It supports the modes equidistant (e), quantile (q), logarithmic (l), and k-means (k).

Can we use the same classification across the board, as in all languages and all granularities (tract and PUMA), or will that not sufficiently communicate patterns in the data?

Strategies

In the ArcGIS web map, use the light basemap similar to our MB one. This should increase the chances that the colors will look good in the final map.
If it helps, this is the full example that the final code approach will likely be based on. It includes Chroma, MB, and the Census SDK.

@fiddleHeads let me know if any of this doesn't make sense and/or you don't want to do it. I think there will be a lot of questions that come out of it due to a variety of scenarios, so let's just take it one step at a time.

Also no hurry as I probably won't touch the census stuff for a bit!

Sounds good. Will let you know what I come up with.

@fiddleHeads @rperlin-ela
this issue kind of got sucked into #113 (probably my bad there) so we can refer to any scheme-related details in that one but continue the scheme convo here.
Demo?
i pushed up a functioning but very-WIP deploy using:

census api perma-joined to mb-lookup
MB Boundaries tracts layer

Shhhh
i'm aware:

lang comm dot points don't show "above" tracts
color scheme classes: not super-meaningful
no PUMA
no legend
"rate of change" not intuitive. mostly just to demo the ease of MB's interpolate options

that list could get huge so use your judgment on what you feel is "ready" to comment on (we know the drill by now!).
Questions
Some overlap mentioned in #135 already, so i guess my question is just am i on the right track with this? the census goals have been reeeeeally broad so far and i'm struggling with what to communicate and offer to the user in the UI.

also, and not to open another can of stats worms, but i install the deceptively named simple statistics lib, so we'll need to answer the relevant things from above and #135 before i know what to do with that.

am i on the right track with this? the census goals have been reeeeeally broad so far and i'm struggling with what to communicate and offer to the user in the UI.

I apologize for the broadness but I think this is really on track. This is really the dream right here— showing people via ELA data that Chinatown, where the census records "Chinese", is much more diverse

So now I just want to make sure people see this, and that it's intuitive to get to this place. I'm not totally sure how to do that, but hopefully the way we ultimately present the "Spatial" panel will help, and fwiw I'll include something in Help. my question is whether there might be one or two more places in the UI where we can nudge people, like (with this example) in "China" under Categories or under "Chinese" in details.

that list could get huge so use your judgment on what you feel is "ready" to comment on (we know the drill by now!).

maybe I should break this out as a separate issue, or it's something you're aware of, but when I zoom in or out and then try to move my way around the map, it now seems to go back out to that default zoom level right away
also curious if you tried "Age5p_Only_English_ACS_13_17"

I apologize for the broadness but I think this is really on track.

no worries, it's a broad topic, but good to know i'm on the right track!

one or two more places in the UI where we can nudge people, like (with this example) in "China" under Categories

typo perhaps? Countries i think? if so, possibly, but the config for this would have to live somewhere, and same for anything that's not in the data already, e.g. allowing user to set the census language at /Explore/Language/Chinese (Mandarin) to Chinese. I could simply do a check (at that particular Explore level anyway) to see if "Chinese (Mandarin)" includes any of the languages available in the Spatial "Show by" dropdown. this sounds way too easy though, and I doubt they'd fall so neatly into the list!
if i'm wrong on that though, would you settle for these spots as the two UI places to set Census language?

/Explore/Language/LanguageName
/Details/ID (see below)

or under "Chinese" in details.

so a new badge/btn/link then, correct? that could work, even sans addl config, if we add a new column like "Census Language" or something. this would actually be far easier on my end (and probably yours) than the Explore-level since that needs more config.
you'd just need to make sure the value for those cells matches the column name in whatever tracts/PUMA layer/s we end up using. oh and actually if we can't get them into one layer, then we'd probably need another column like "Census source" or "Census layer" which would have values of:

puma
tract
nothing/empty

maybe I should break this out as a separate issue, or it's something you're aware of, but when I zoom in or out and then try to move my way around the map, it now seems to go back out to that default zoom level right away

i think i know what you're talking about but i can't replicate it at the moment. presumably has something to do with how i'm setting the symbology for the tracts. are you seeing it consistently and in all scenarios or just after playing with the Spatial Show by dropdown?

maybe I should break this out as a separate issue

don't worry about it, i'm ok with including as part of the current WIP.

also curious if you tried "Age5p_Only_English_ACS_13_17"

i had it in but took it out as it seemed like a different meaning than the others:

is that not the case? if so then the API will have to be hit again to get those values and re-joined to the lookup table.

Latest
also i pushed up some potential improvements to the color scheme classifications: https://deploy-preview-134--languagemapping.netlify.app/spatial . i'm definitely out of my element here, but the patterns look more evident i think.
oh the push may have also fixed the layer draw order issue where tracts were on top of points.
Census popups
i don't think we've answered the "popups for census?" question yet but locally i tried it in the same manner or very similar to how we do it for County/Neighb (via lookup table) and there's quite a lag. i realize popups aren't as useful for census, or at least the tract ID isn't, but it'd be nice to click and get the value at least, just not at the expense of waiting for it.
this would be another issue likely resolved by an uploaded non-Boundaries tracts layer as the values would be baked into each tract and the code would not have to potentially loop over 2000+ records to find the match- by comparison the Neighb lookup has a mere 272 records compared to tracts' 2k, so the lag is not a huge surprise.
it's also possible there's a lag on Neigh/County popups as well, but it's less evident since i don't show it until after the zoom/pan when the feature is clicked. we decided not to zoom to tracts though in order to save a considerable amount of data/space/perf for behavior that seems minimally useful and potentially annoying in the context of small features like tracts.
User luxuries
i know PUMA is not in the dropdowns but based on my latest push are we leaning towards answers on any of these yet for this dropdown?

should user even be allowed this luxury or will they trigger a "this is why we can't have nice things"?
if no luxury, is there a catch-all classification* of the three?
or is it super useful for some and not at all for others meaning we'd have to config it on a per-language basis?

IMO we should just pick one that's sufficient for all. they all show similar patterns, and config-less is always going to be simpler.
if we wanted to go super performant, i could have the natural breaks ready for each class before the layer ever sees the light of day rather than recalculating each time dropdown is changed. this would not be tied to the dataset of course, but if we're hurting for perf at some point, just throwing it out there.

for Item 2, it's not actually a classification- they're all using ckmeans which is evidently a step up from Natural Breaks (again out of my element and not even pretending to understand any of this, ha!). the dropdown actually refers to these in case that helps with clarification of the 1-3 above.

i don't think we've answered the "popups for census?" question yet but locally i tried it in the same manner or very similar to how we do it for County/Neighb (via lookup table) and there's quite a lag. i realize popups aren't as useful for census, or at least the tract ID isn't, but it'd be nice to click and get the value at least, just not at the expense of waiting for it.

In agreement, a simple popup could be nice for a value but only worth it if it's simple and painless

IMO we should just pick one that's sufficient for all. they all show similar patterns, and config-less is always going to be simpler.

I agree— I'm not a stats person, but I notice that Exponential here is giving us the most to look at. If we're just choosing one, do we even need a catch-all word and the whole "Rate of Change" dropdown and all that? If we need a disclaimer or a blurb somewhere that's fine, but I don't think people will expect these options. Btw, not share if we ever shared an old inspiration— this preusmably used PUMA data and does something a little different from what we're doing, but people really like it: https://www.jillhubley.com/project/nyclanguages/

typo perhaps? Countries i think?

Yes!

so a new badge/btn/link then, correct? that could work, even sans addl config, if we add a new column like "Census Language" or something. this would actually be far easier on my end (and probably yours) than the Explore-level since that needs more config.

Sounds great

you'd just need to make sure the value for those cells matches the column name in whatever tracts/PUMA layer/s we end up using. oh and actually if we can't get them into one layer, then we'd probably need another column like "Census source" or "Census layer" which would have values of:

puma
tract
nothing/empty

Ok, I'm ready to move on this whenever

i think i know what you're talking about but i can't replicate it at the moment. presumably has something to do with how i'm setting the symbology for the tracts. are you seeing it consistently and in all scenarios or just after playing with the Spatial Show by dropdown?

I think I'm seeing it pretty consistently in some form in all scenarios but it's a little hard to specify what zoom level it goes back out to when I start pinching— not always the same, maybe the thing it was just on before I started zooming in/out

also curious if you tried "Age5p_Only_English_ACS_13_17"

is that not the case? if so then the API will have to be hit again to get those values and re-joined to the lookup table.

I agree that it's a little different, but maybe we can nod to that by calling it "English (Only)". Maybe it'll feel wrong when we see it, but I do think it's likely to be important and interesting, and in any case these are not all strictly comparable... "French" is including Haitian Creole and "German" is including Yiddish and "Russian" is including "Polish" and "other Slavic" which is not at all intuitive for people and we're going to have account for — both wherever else in the UI and I think also in the dropdown list, where maybe it should be "French/Haitian Creole", "German/Yiddish", "Russian/Polish/Other Slavic".

If we're just choosing one, do we even need a catch-all word and the whole "Rate of Change" dropdown and all that?

nope don't think so. the dropdown i guess was just 75% internal demo so the coloring type options could be played with, and 25% "maaaaybe user would want this?".

If we need a disclaimer or a blurb somewhere that's fine

won't need one if we remove the dropdown, but a blurb via an ℹ btn near the soon-to-be legend for green gradient would be a good spot to indicate the stats method, data sources, difference b/t puma and tracts, etc. this can definitely be hidden by default.
or better approach (still using "i" btn): in the About or Help guide, wherever makes sense, you should learn how to set id values on the blocks in WP. that way I can just link directly to that section (e.g. https://map.languagemapping.org/about#census-data) rather than creating and updating it in the code.
Maya and i set that up for some of the About links but i think some of it may have gotten jostled around in the shuffle as the links no longer work, e.g. https://map.languagemapping.org/about#legal does not take you to the Legal section.
creating those id's is definitely a candidate for the Big Manual, it's kind of a freebie for content mgmt as it keeps it current and requires minimal effort on my end. i don't have WP but i imagine stuff like that is in their docs, and Maya may recall how to do it as well.

Ok, I'm ready to move on this whenever

we to dial in the data stuff first. i'll keep that going in email, i just haven't replied yet.
there's nothing stopping you from adding two new cols in the interim tho:

Census Field
Census Layer, which will either be puma or tract. keep them lowercase for simplicity.

we could probably get clever with just the first column and some kind of prefix before the language name, but that sounds like a recipe for trouble.
if you start on that, please update Data Schema sheet as well.

I think I'm seeing it pretty consistently in some form in all scenarios but it's a little hard to specify what zoom level it goes back out to when I start pinching— not always the same, maybe the thing it was just on before I started zooming in/out

so this is just on mobile, or desk too?

I agree that it's a little different, but maybe we can nod to that by calling it "English (Only)". Maybe it'll feel wrong when we see it, but I do think it's likely to be important and interesting

ok. it's a lot of steps to make all this happen so let's use what we have until the data stuff settles in the email thread.

and in any case these are not all strictly comparable... "French" is including Haitian Creole and "German" is including Yiddish and "Russian" is including "Polish" and "other Slavic" which is not at all intuitive for people and we're going to have account for — both wherever else in the UI and I think also in the dropdown list, where maybe it should be "French/Haitian Creole", "German/Yiddish", "Russian/Polish/Other Slavic".

ha, yeah i figured it wouldn't be as easy as just the one-word name, i think that seals the fate of the two new data cols then!

won't need one if we remove the dropdown, but a blurb via an ℹ btn near the soon-to-be legend for green gradient would be a good spot to indicate the stats method, data sources, difference b/t puma and tracts, etc. this can definitely be hidden by default.

Yes! With this I would say let's be maximally simple otherwise

or better approach (still using "i" btn): in the About or Help guide, wherever makes sense, you should learn how to set id values on the blocks in WP. that way I can just link directly to that section (e.g. https://map.languagemapping.org/about#census-data) rather than creating and updating it in the code.
Maya and i set that up for some of the About links but i think some of it may have gotten jostled around in the shuffle as the links no longer work, e.g. https://map.languagemapping.org/about#legal does not take you to the Legal section.

Ok, will try again at some point, may need help

there's nothing stopping you from adding two new cols in the interim tho:

Census Field
Census Layer, which will either be puma or tract. keep them lowercase for simplicity.

Ok, maybe I should wait, but just want to be sure I understand, I may not yet. For example, for Chinese (tract-level example), under "Census Field" would it be "Chinese" and under "Census Layer" it would be "Age5p_Chinese_ACS_13_17"? And for "Italian" (PUMA-level example) "Census Field" would be "Italian" and the "Census Layer" would be "F1000_Italian"? Likewise exactly the same for Sicilian if we want it linked to the Italian PUMA layer?
What about for a case like "German" (which should be "German/Yiddish" at the tract level, but just "German" at the PUMA)?

if you start on that, please update Data Schema sheet as well.

Started

I think I'm seeing it pretty consistently in some form in all scenarios but it's a little hard to specify what zoom level it goes back out to when I start pinching— not always the same, maybe the thing it was just on before I started zooming in/out

so this is just on mobile, or desk too?

I have the issue on desktop too, when I zoom in and then click once in preparation for dragging myself and looking around

Yes! With this I would say let's be maximally simple otherwise. fwiw, I see the logic of a single dropdown with tracts and PUMAs as long as there's some little thing differentiating them (like an asterisk or divider with name within the dropdown)

so single dropdown would be your preference over two dropdowns then?

Ok, maybe I should wait

yeah let's hold off for a sec. this is either going one of two directions and i don't want to steer you in the wrong one.

For example, for Chinese (tract-level example), under "Census Field" would it be "Chinese" and under "Census Layer" it would be "Age5p_Chinese_ACS_13_17"?

no, sorry if poor explanation on my part:
Census Field: whatever the field name is for Chinese in the uploaded MB tracts tileset. (very TBD at this point)

Census Layer: "tract" (without quotes)

And for "Italian" (PUMA-level example) "Census Field" would be "Italian" and the "Census Layer" would be "F1000_Italian"?
Likewise exactly the same for Sicilian if we want it linked to the Italian PUMA layer?

Census Field: whatever the field name is for Italian in the uploaded MB tracts tileset. (very TBD at this point)

Census Layer: "puma" (without quotes)

What about for a case like "German" (which should be "German/Yiddish" at the tract level, but just "German" at the PUMA)?

Uh oh! I thought you said we only use one or the other, not both? I was under the impression "tract-level if it's available, otherwise PUMA". is that not the case?

Also, this is probably worth quickly putting to the group, but what would be the best way for me to indicate to you the PUMA layers I don't think we need? My own instinct is not to show a few dozen of the ones that have such low numbers that I'm not sure they indicate anything (Pennsylvnia Dutch, Kiowa etc). I realize that might seem arbitrary, but I think a lot of those are actively misleading, and we're curating here anyway so probably should lay our cards on the table.

Totally. I think this will just be a matter of coordinating with Maya and deciding what goes into the uploaded tileset. The way i have it set now is simply grabbing all the fields from it and populating the dropdown with them, so it would just be a matter of excluding the unwanted fields from the upload. Some of them I may have to exclude manually but best-case scenario, I don't have any involvement/config and i just take all the fields entirety verbatim.

I have the issue on desktop too, when I zoom in and then click once in preparation for dragging myself and looking around

yeesh, ok. probably a simple fix but i'm going to hold off until the other stuff is in place so just sit tight until then.

Thanks, all very helpful. I can see it either way, but personally might lean towards a single dropdrown that has the following features:
-somehow differentiates tract ones from PUMA ones (divider in the dropdown? possibly asterisk?)
-somehow but differently indicates problematic categories (possibly asterisk)
-foregrounds reliable ones and backgrounds unreliables (pending team discussion)
As for our original idea, "tract-level if it's available, otherwise PUMA”, is that just up to me and Maya in terms of coordinating with what she uploads? I think it’s basically fine if we rename a few of the tract ones as I mentioned (e.g. German > German/Yiddish) and then still keep puma ones like “German” and “Yiddish”, because each is showing something different. I don’t see this eliminating any of the tract ones currently there.

On Nov 19, 2020, at 2:52 PM, Jason Lampel<EMAIL_ADDRESS>wrote:
Yes! With this I would say let's be maximally simple otherwise. fwiw, I see the logic of a single dropdown with tracts and PUMAs as long as there's some little thing differentiating them (like an asterisk or divider with name within the dropdown)
so single dropdown would be your preference over two dropdowns then?
Ok, maybe I should wait
yeah let's hold off for a sec. this is either going one of two directions and i don't want to steer you in the wrong one.
For example, for Chinese (tract-level example), under "Census Field" would it be "Chinese" and under "Census Layer" it would be "Age5p_Chinese_ACS_13_17"?
no, sorry if poor explanation on my part:
Census Field: whatever the field name is for Chinese in the uploaded MB tracts tileset. (very TBD at this point)
Census Layer: "tract" (without quotes)
And for "Italian" (PUMA-level example) "Census Field" would be "Italian" and the "Census Layer" would be "F1000_Italian"?
Likewise exactly the same for Sicilian if we want it linked to the Italian PUMA layer?
Census Field: whatever the field name is for Italian in the uploaded MB tracts tileset. (very TBD at this point)
Census Layer: "puma" (without quotes)
What about for a case like "German" (which should be "German/Yiddish" at the tract level, but just "German" at the PUMA)?
Uh oh! I thought you said we only use one or the other, not both? I was under the impression "tract-level if it's available, otherwise PUMA". is that not the case?
Also, this is probably worth quickly putting to the group, but what would be the best way for me to indicate to you the PUMA layers I don't think we need? My own instinct is not to show a few dozen of the ones that have such low numbers that I'm not sure they indicate anything (Pennsylvnia Dutch, Kiowa etc). I realize that might seem arbitrary, but I think a lot of those are actively misleading, and we're curating here anyway so probably should lay our cards on the table.
Totally. I think this will just be a matter of coordinating with Maya and deciding what goes into the uploaded tileset. The way i have it set now is simply grabbing all the fields from it and populating the dropdown with them, so it would just be a matter of excluding the unwanted fields from the upload. Some of them I may have to exclude manually but best-case scenario, I don't have any involvement/config and i just take all the fields entirety verbatim.
I have the issue on desktop too, when I zoom in and then click once in preparation for dragging myself and looking around
yeesh, ok. probably a simple fix but i'm going to hold off until the other stuff is in place so just sit tight until then.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/issues/115#issuecomment-730599912, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5GYN54AHN2SHZWP4GTSQVZPHANCNFSM4SQ7JRDQ.

something like this?

assuming column order is retained, and i don't see why it wouldn't be, that seems doable.

As for our original idea, "tract-level if it's available, otherwise PUMA”, is that just up to me and Maya in terms of coordinating with what she uploads?

yep. that redundancy is fine across tract/puma, even if the names are the same in each tileset. accidental bonus of using two tilesets. 💫  the only thing you can't do in this setup is have both puma and tract in the language config sheet for a given language. at least for now you'll have to stick with one or the other, and continue with our "tract if available, otherwise puma" approach.

Exactly, and I take it that Maya and I can somehow determine the ordering within “by tract” and “by puma"? I’m thinking just to do it by total NYC speaker population, e.g. English, Spanish, Chinese etc. so whatever’s easiest in terms of doing it in Maya’s layer or on your end. The ones that are clear and by tract (e.g. Vietnamese) won’t need the PUMA layer.

On Nov 20, 2020, at 12:51 PM, Jason Lampel<EMAIL_ADDRESS>wrote:
something like this?
https://user-images.githubusercontent.com/4974087/99832293-8d52ec80-2b1d-11eb-937a-953b4534d109.png
assuming column order is retained, and i don't see why it wouldn't be, that seems doable.
As for our original idea, "tract-level if it's available, otherwise PUMA”, is that just up to me and Maya in terms of coordinating with what she uploads?
yep. that redundancy is fine across tract/puma, even if the names are the same in each tileset. accidental bonus of using two tilesets. 💫 the only thing you can't do in this setup is have both puma and tract in the language config sheet for a given language. at least for now you'll have to stick with one or the other, and continue with our "tract if available, otherwise puma" approach.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/issues/115#issuecomment-731314452, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5DOEHNZGWHTO7LAV7DSQ2UALANCNFSM4SQ7JRDQ.

Exactly, and I take it that Maya and I can somehow determine the ordering within “by tract” and “by puma"?

yeah just change the column order in the sheets she gave us.

I’m thinking just to do it by total NYC speaker population, e.g. English, Spanish, Chinese etc.

you're breaking away from your alphabetized pattern? :) i'd prefer alphabetical as a user, but that's me.

so whatever’s easiest in terms of doing it in Maya’s layer or on your end.

...or your end. much much easier to just change the column order, so go for it.


JoySerializationManager.cpp & JoySerializationManager.h Missing from this Repo

Hello, I noticed that two files mentioned in your Gamasutra article are no long in this repository.  Are they perhaps available elsewhere?  Thanks!
p.s. looks like they were originally in master/ue4-source/Source/Private/Core/Serialization

Looks like they were removed a while ago here.
https://github.com/trentpolack/joymachine-public/commit/b9f6aeb2ff16b32a25e8e68928dadf51ce86451b
Apparently they were broken or out of date or something?
During my investigation that lead me to finding that gamasutra article and the json serializationManager I did find another that is still active and up to date.
https://github.com/slowburn-dev/DataConfig which is open source but has a 'premium wrapper' for it here https://www.unrealengine.com/marketplace/en-US/product/dataconfig-json-asset
no affiliation. Just a random data driven enthusiast that is disappointed in UE default offerings.


Update advanced.rst

Made some edits that included some of a .jx that will be used in the workflow, clarified other parts of the tutorial, and fixed a couple grammatical errors.

has conflicts with updated version


Beacons/iBeacons Plugin

Feat, Feature
Create a angular service for the Cordova "proximity" / beacons / iBeacon plugin.

https://github.com/petermetz/cordova-plugin-ibeacon
https://github.com/petermetz/cordova-proximity-quickstart

There is a pull request "Add $cordovaBeacon plugin" for that already

Oh nice! I searched the issues but didn't search the PRs.


Update OpenTelemetry Dependencies to latest

Update OpenTelemetry Dependencies to latest

I am closing this since the pipeline picks the latest, and on this re-run we now have some v0.90.0 deps. I will create a replacement PR with the update-dep script and version set to 0.89.0 as opposed to latest used by the pipeline.


Successive calls to item.SetTitle causes panic

Hi!
I had a quick question about 03c78c0eeffdba02e75ddec29d7942770ca1e9b0. After updating, successive calls to item.SetTitle causes my application to panic on this line since the channel has already been closed:
panic: close of closed channel

goroutine 29 [running]:
fyne.io/systray.(*MenuItem).update(0x140000a6440)
	fyne.io/systray@v1.10.1-0.20240608091126-03c78c0eeffd/systray.go:273 +0x6c
fyne.io/systray.(*MenuItem).SetTitle(...)
	fyne.io/systray@v1.10.1-0.20240608091126-03c78c0eeffd/systray.go:199
github.com/gabe565/nightscout-menu-bar/internal/tray.(*Tray).onReady(0x140000c2000)
	github.com/gabe565/nightscout-menu-bar/internal/tray/systray.go:131 +0x354
fyne.io/systray.Register.func2()
	fyne.io/systray@v1.10.1-0.20240608091126-03c78c0eeffd/systray.go:112 +0x38
created by fyne.io/systray.Register in goroutine 1
	fyne.io/systray@v1.10.1-0.20240608091126-03c78c0eeffd/systray.go:110 +0xb8
exit status 2

My application updates the title every minute or so. Is item.SetTitle not intended to be used this way?
Thank you!

SetTitle should be callable any number of times. It looks like the channel being closed is in subtly wrong scope.

Should be fixed now, sorry for that

No problem. Thank you for the quick fix! Love all of the work you do


add functionality for play mode button

Adds functionality for each mode with continue, stop, repeat page, and repeat selection.

@ChristopherRankin: I am sorry it has taken so long to get to this.  Do I need to fix issues or would you have time to look at things?
I noticed that when the mode is stop at the end of the page, the play/pause button still displays that it is playing (i.e. the pause button is showing).


pyserial: fix build on Darwin

Motivation for this change
Disables hanging tests to fix the build
py2
py3
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

I think the doInstallCheck thing was probably a mistake. I thought that the failing tests were install checks. Does pyserial still build if you remove it?

Ok nice. That was my bad.

Thanks! :)
On Wed, Jul 25, 2018, 13:38 Matthew Bauer<EMAIL_ADDRESS>wrote:

Merged #44073 https://github.com/NixOS/nixpkgs/pull/44073.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/44073#event-1753323154, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADWHbf3yAykiRVpAfHBtQdMb07CZbbP-ks5uKLsigaJpZM4VfWZl
.

--
Luke Adams


Fix x86 build

I will verify that this still works on arm64, and then will merge if so - just to be clear, it was a duplicate symbols error?

I will verify that this still works on arm64, and then will merge if so - just to be clear, it was a duplicate symbols error?

Yes it was.


What License OFL or Apache?

ping @davelab6
repo currently uses Roboto's Apache license.

Apache better as an exception
On Jul 27, 2017 6:05 AM, "Marc Foley"<EMAIL_ADDRESS>wrote:

ping @davelab6 https://github.com/davelab6
repo currently uses Roboto's Apache license.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vahanhovh/mardoto/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP9ywdeZepOqgmpfTxZ1fJxD_rLY_YWks5sSGD2gaJpZM4OlEHk
.

Thank you.
Can I still keep our usual copyright string in these fonts,Copyright 2017 The Mardoto Project Authors (https://github.com/vahanhovh/mardoto)?

Yes
On Jul 27, 2017 9:07 AM, "Marc Foley"<EMAIL_ADDRESS>wrote:

Thank you.
Can I still keep our usual copyright string in these fonts,Copyright 2017
The Mardoto Project Authors (https://github.com/vahanhovh/mardoto)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vahanhovh/mardoto/issues/3#issuecomment-318356240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP9y5jeFMbBr3eDq4xoiW-sruWAKFmaks5sSIt-gaJpZM4OlEHk
.


Source in .NET Standard

Would you find it beneficial to complement, or convert, the source from .NET Framework to .NET Standard?
I would happily provide you with a PR if you'd like.

At the moment it's not a priority for us to convert the package to .NET Standard, although its a nice idea.

Just tried to open the solution on my mac and had errors and spotted this issue. Would get my upvote :)


Benches: Move Plane::clone into setup step for more accurate results

I noticed that the benches include p.clone() in the closure passed to b.iter which means that the clone() is part of the benchmark. Benchmarking p.clone() shows that it takes >50% of the runtime of unrelated benches.
The iter_batched{_ref} functions allow running a setup step which is not included in the measurement[^1]:
Some numbers from my machine:
new bench to show the impact of cloning:
plane clone             time:   [5.9355 µs 6.0196 µs 6.1117 µs]

And the benches which used clone() before:
plane pad               time:   [9.2417 µs 9.3874 µs 9.5638 µs]
plane copy_from_raw_u8 8-bit
                        time:   [13.513 µs 13.915 µs 14.305 µs]
plane copy_from_raw_u8 10-bit
                        time:   [21.879 µs 22.466 µs 23.078 µs]

now:
plane pad               time:   [3.0828 µs 3.1744 µs 3.2823 µs]
plane copy_from_raw_u8 8-bit
                        time:   [6.0329 µs 6.2267 µs 6.4289 µs]
plane copy_from_raw_u8 10-bit
                        time:   [6.6135 µs 6.7775 µs 6.9574 µs]

Note that the 10-bit bench seems to be impacted relatively more by this.
[^1]: This technically comes at a runtime cost/"measurement overhead", but the criterion documentation claims that it's usually less than one nanosecond, which is negligible here.

Looks good thank you


MVP部署包 执行时出现错误

Issue Type
Bug
Source
binary
Secretflow Version
MVP docker
OS Platform and Distribution
centos7
Python version
MVP docker
Bazel version
No response
GCC/Compiler version
No response
What happend and What you expected to happen.
2023-07-11T15:45:27+08:00 INFO the jobId=gktz, taskId=gktz-egtepevc-node-3 start ...
2023-07-11T15:45:31+08:00 INFO the jobId=gktz, taskId=gktz-egtepevc-node-3 failed: party bob failed msg: container[secretflow] terminated state reason "Error", message: "Traceback (most recent call last):
  File \"/usr/local/lib/python3.8/runpy.py\", line 185, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File \"/usr/local/lib/python3.8/runpy.py\", line 111, in _get_module_details
    __import__(pkg_name)
  File \"/usr/local/lib/python3.8/site-packages/secretflow/__init__.py\", line 16, in <module>
    from . import (
  File \"/usr/local/lib/python3.8/site-packages/secretflow/data/__init__.py\", line 15, in <module>
    from . import horizontal, vertical
  File \"/usr/local/lib/python3.8/site-packages/secretflow/data/horizontal/__init__.py\", line 15, in <module>
    from .dataframe import HDataFrame
  File \"/usr/local/lib/python3.8/site-packages/secretflow/data/horizontal/dataframe.py\", line 21, in <module>
    from secretflow.data.base import DataFrameBase, Partition
  File \"/usr/local/lib/python3.8/site-packages/secretflow/data/base.py\", line 20, in <module>
    from jax.tree_util import tree_map
  File \"/usr/local/lib/python3.8/site-packages/jax/__init__.py\", line 35, in <module>
    from jax import config as _config_module
  File \"/usr/local/lib/python3.8/site-packages/jax/config.py\", line 17, in <module>
    from jax._src.config import config  # noqa: F401
  File \"/usr/local/lib/python3.8/site-packages/jax/_src/config.py\", line 24, in <module>
    from jax._src import lib
  File \"/usr/local/lib/python3.8/site-packages/jax/_src/lib/__init__.py\", line 84, in <module>
    cpu_feature_guard.check_cpu_features()
RuntimeError: This version of jaxlib was built using AVX instructions, which your CPU and/or operating system do not support. You may be able work around this issue by building jaxlib from source.
"
2023-07-11T15:45:31+08:00 INFO the jobId=gktz, taskId=gktz-egtepevc-node-3 failed: The remaining non-failed parties counts 1 is less than the task success threshold
MVP部署 后 在联合圈人模版中 点击全部执行 时 隐私求交组件报错

Reproduction code to reproduce the issue.
MVP部署 后在联合圈人模版中 点击全部执行时 隐私求交组件报错

Hi @littlegirlpppp

Hi @littlegirlpppp
请注意log中：
This version of jaxlib was built using AVX instructions

请确保你的cpu支持AVX指令集，感谢

啊 但是我们 集团用的 都是服务器可能老了  只能更换服务器吗

@littlegirlpppp 哈喽，这个问题解决了吗？可以提供下之前的系统信息和CPU信息吗？


Support internvl2 chat template

InternVL2 defines 3 types of chat templates.
https://huggingface.co/OpenGVLab/InternVL2-26B/blob/main/conversation.py#L333
Among the currently released models, 2B/8B/26B adopt internlm2-chat while 4B uses phi3-chat.
We should consider them all.

Please fix the UT error

@zhulinJulia24 may update TC

Hi, I feel that it may not be necessary to implement these three chat templates separately. For InternVL2-2B/8B/26B, we can simply use the chat template of internlm2-chat. The main difference is the system prompt.
I used the following code to run the InternVL2 model, and it seems to work well based on my tests.
from lmdeploy import pipeline, TurbomindEngineConfig, ChatTemplateConfig
from lmdeploy.vl import load_image

model = 'OpenGVLab/InternVL2-2B'
system_prompt = '我是书生·万象，英文名是InternVL，是由上海人工智能实验室及多家合作单位联合开发的多模态基础模型。人工智能实验室致力于原始技术创新，开源开放，共享共创，推动科技进步和产业发展。'
image = load_image('https://raw.githubusercontent.com/open-mmlab/mmdeploy/main/tests/data/tiger.jpeg')
chat_template_config = ChatTemplateConfig('internlm2-chat')
chat_template_config.meta_instruction = system_prompt
pipe = pipeline(model, chat_template_config=chat_template_config,
                backend_config=TurbomindEngineConfig(session_len=8192))
response = pipe(('describe this image', image))
print(response.text)

For InternVL2-4B, can the Phi3 chat template be used directly? I see there is an implementation here.

Hi, @czczup thanks for the suggestion.
But I am concerned that it may not be user-friendly, as users are required to be aware of setting the chat template.


feat: added rate limit

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues
#326

http://localhost/api/user

http://localhost/api/user

Too Many Requests

👍


Can't it be used in recyclerView with PagerSnapHelper?

Can't it be used in recyclerView with PagerSnapHelper?

I don't know. Have you tried it?

I have tried.But it can't set click and long click.

If you are reporting a problem, please use the issue template. You deleted it when opening this.

Okay,thanks.


Connection to storage

Unloading the document (as in #339) requires to control a "storage mutex". This is described in more detail here. The storage mutex also appeared on a couple of other occasions.
As the AngleSharp.Core mission is to provide the basis for anything advanced an interface-based implementation (just describe a general interface and use it in the code if an implementation is available / registered) seems to be the right way to include it.

After investigating this one carefully I've decided to omit it from AngleSharp. The reason is that this is rather an implementation detail and that any resources (e.g., cookies) that may be manipulated from multiple contexts need to be working on their own. So this one is left for individual components.


run-with-cover should hook to vm.runInThisContext and vm.hookCreateScript as well

RequireJS uses vm.runInThisContext to evaluate the AMD modules when running on node.js. I think the cover command should add hooks to runInThisContext and createScript automatically or at least have an easy way to set it.
Since transformFn isn't exposed to other modules I ended up editing istanbul/lib/hook.js directly. Not the most elegant solution but worked on my scenario (loading specs and source code with RequireJS):
diff --git a/node_modules/istanbul/lib/command/common/run-with-cover.js b/node_modules/istanbul/lib/command/common/run-with-cover.js
index a9e6e88..b43726d 100644
--- a/node_modules/istanbul/lib/command/common/run-with-cover.js
+++ b/node_modules/istanbul/lib/command/common/run-with-cover.js
@@ -1,3 +1,6 @@
+// this file was edited by Miller Medeiros to support RequireJS on node.js
+// changes are marked with [mm]
+
 /*
  Copyright (c) 2012, Yahoo! Inc.  All rights reserved.
  Copyrights licensed under the New BSD License. See the accompanying LICENSE file for terms.
@@ -124,6 +127,11 @@ function run(args, commandName, enableHooks, callback) {
                 if (opts['self-test']) {
                     hook.unloadRequireCache(matchFn);
                 }
+
+                // ---------- added [mm] -------------------- //
+                hook.hookRunInThisContext(matchFn, transformer, hookOpts);
+                // ------------------------------------------ //
+
                 hook.hookRequire(matchFn, transformer, hookOpts);
                 process.once('exit', function () {
                     var file = path.resolve(reportingDir, 'coverage.json'),
@@ -161,3 +169,4 @@ module.exports = {
     run: run,
     usage: usage
 };
+
diff --git a/node_modules/istanbul/lib/hook.js b/node_modules/istanbul/lib/hook.js
index a22a30a..f5a28f6 100644
--- a/node_modules/istanbul/lib/hook.js
+++ b/node_modules/istanbul/lib/hook.js
@@ -1,3 +1,7 @@
+// this file was edited by Miller Medeiros to support RequireJS on node.js
+// changes are marked with [mm]
+
+
 /*
  Copyright (c) 2012, Yahoo! Inc.  All rights reserved.
  Copyrights licensed under the New BSD License. See the accompanying LICENSE file for terms.
@@ -130,6 +134,21 @@ function hookCreateScript(matcher, transformer, opts) {
     };
 }

+// ======== edited [mm] ======== //
+
+var originalRunInThisContext = vm.runInThisContext;
+
+function hookRunInThisContext(matcher, transformer, opts) {
+    opts = opts || {};
+    var fn = transformFn(matcher, transformer, opts.verbose);
+    vm.runInThisContext = function (code, file) {
+        var ret = fn(code, file);
+        return originalRunInThisContext(ret.code, file);
+    };
+}
+
+// ========================== //
+
 /**
  * unhooks vm.createScript, restoring it to its original state.
  * @method unhookCreateScript
@@ -142,6 +161,7 @@ function unhookCreateScript() {
 module.exports = {
     hookRequire: hookRequire,
     unhookRequire: unhookRequire,
+    hookRunInThisContext : hookRunInThisContext, // added [mm]
     hookCreateScript: hookCreateScript,
     unhookCreateScript: unhookCreateScript,
     unloadRequireCache: unloadRequireCache

Not doing a pull request since I'm focused on other tasks and don't have time to write the tests. Just a FYI that other hooks might be important on different scenarios.
I have it working on the amd-utils travis build, so it can be used as a reference, see tests/runner.js and tests/spec folder for details about my tests: https://github.com/millermedeiros/amd-utils/blob/master/tests/ (source code and tests are written in AMD and are running on node.js and browser).
Cheers.

A piece of late feedback: It works fine with require('vm').runInThisContext(code, ...), but not when using the vm.Script constructor:
var vm = require('vm');
new vm.Script(code, ...).runInThisContext();


Cannot find closure

I'm having trouble with the build.py step in the build process described here...
https://github.com/carlosperate/ardublockly/wiki/Building-Ardublockly
When I run build.py, I receive the following message...
Error: Closure not found.  Read this:
https://developers.google.com/blockly/hacking/closure
The build.py file is looking for
('path = ', '..\closure-library\closure\bin')
('filename = ', 'calcdeps.py')
Which does not exist in the closure library that I downloaded (20210601).  I see that most of the activity on this project was from 2016, was this file deleted in later version of the closure library or is there a step that I missed to create this file?

I was not properly following the instructions.
git submodule update --init --recursive
in the build instructions
https://github.com/carlosperate/ardublockly/wiki/Building-Ardublockly
pulls in the needed files.


Correct formatting input data for SimpleRNNClassification

I have a quick question about the right formatting of input for the decoders.SimpleRNNClassification (and similar classes). In the docstring I read:
X_train: numpy 3d array of shape [n_samples,n_time_bins,n_neurons]
    This is the neural data.
    See example file for an example of how to format the neural data correctly

But in your usage example notebook (central_concepts_in_ML_for_decoding.ipynb) the input X_train is formatted as [n_samples,n_neurons,n_time_bins].
Which one is correct?

It is correct in the docstring.
Thanks for bringing this up - I realize that this is an error in that example notebook that we will need to fix. In the example notebook, you can just use np.swapaxes to put the data in the correct format for the SimpleRNNClassification model. This will give you better accuracy than in the current notebook with the error.

Thanks, great.


Fixup prompt output when using readline

Fixes https://github.com/upwork/php-upwork/issues/15

Hello @reedy ,
we truly appreciate your help and will get back to this misspellings, i.e. pull requests, right before we will release a new version of the library.
Thank you once again!

Fixed in https://github.com/upwork/php-upwork/commit/fe34f2dab8b475c4f32b8df44b8bfbd40729cd49
Thank you for the contribution!


Add upgrade handler for storehost

Storehost simply reports itself as GOING_DOWN to the controller
and just waits for the default upgrade timeout which is 10s.
By this time, the hope is that the controller would have notified
the respective input hosts to start draining.

Coverage decreased (-0.2%) to 68.511% when pulling 536461192b9d32dcc9463840a1d6c928d4cb2fe8 on store_upgrade into c901fe08dbc68e047fe6567a6d69e98e14b39e13 on master.

Changes Unknown when pulling ef3e94f7dff9ef3bfefd8fa7e1f50f245ec8870c on store_upgrade into ** on master**.


[Dreamview Bug] can not started dreamview again after is stopped

when dreamview is stopped. the dreamview cannot be started again.
System information

OS Platform and Distribution: Ubuntu 18.04:
**Apollo installed from source **:
**Apollo version 8.0:
Docker version apolloauto/apollo:dev-x86_64-18.04-20220803_1505:

Steps to reproduce the issue:
./scripts/bootstrap.sh stop
./scripts/bootstrap.sh start
Supporting materials (screenshots, command lines, code/script snippets):

I experienced a similar issue when running Dreamview for an extended period of time. I could not figure out where the issue was coming from and had 2 workarounds: 1) periodically restart Dreamview, and 2) avoid communication with Dreamview over the socket, because it seemed like the socket server is having issues.

hi！You can see if the dream view process still exists, if it exists, kill the process and restart.


Pass back JSON marshaling error

It can be difficult to debug these errors without this information.

Hi, can this be merged?


[HOLD for payment 2023-06-21] [$1000] Thread - Crash when opening the thread image

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

Action Performed:

Go to https://staging.new.expensify.com/
Login with any account
Go to any chat
Send an image to the user
Click on "Reply in thread" after sending the image
Click to preview the image

Expected Result:
Image opens
Actual Result:
App is crashed
Workaround:
Unknown
Platforms:

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

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/75058993-6d5b-4d0e-b622-68cecb2d5941
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal team and @priya-zha
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b43bec189a9704e2
Upwork Job ID:<PHONE_NUMBER>380827136
Last Price Increase: 2023-06-06

Super easy to recreate. Moving on.

@tienifr Thanks for the proposal. Your RCA is correct and the solution looks good to me. Let's pass report instead of reportID.
:ribbon: :eyes: :ribbon: C+ reviewed
cc @NikkiWines

Friendly bump @NikkiWines for your review so we can keep good pace on moving this forward 🙌

PR has been merged so we're just waiting on deploys. All good for now!

@zanyrenney - I'm leaving for sabbatical this evening and will not be available to pay this on the 21st. Thank you for handling it!

Sure thing! Awaiting payment still, not overdue.

Looks like the contributors weren't assigned initially, so assigning them now.

@zanyrenney submitted the proposal. Thanks.

Applied!

I've hired everyone in Upworks. Once the offers are accepted, I'll process the payments.

Accepted!

All payments have been sent including the speed bonus.


Refactors and extend graph building

Type of the feature

[ ] New function in sqduipy.im?
[x] New function in squidpy.gr?
[ ] New function in squidpy.pl?
[ ] Change an existing functionality, such as default behavior?
[ ] Other?

Description
It would be nice to refactor the way the spatial graph is computed in order to generalize to other settings (e.g. grids). Follows up from #314 .
The way it's structured now is the following:

if coord_type = CoordType.VISIUM, it builds a graph from hexagonal grid
if coord_type = CoordType.GENERIC, it build a graph either by using the radius, a knn search or (from #322 ) a Delaunay triangulation.

It would be good to change behaviour with the following:

if coord_type = CoordType.GRID (replace VISIUM), then builds a regular grid with either 4 (old ST data and dbit-seq) or 6 (visium) neighbors. Potentially allow for 8 (although don't have use case).
coord_type = CoordType.GENERIC, then use an additional mode argument (?) to allow either knn, radius or the new delaunay. I'm not sure how this could be best set up but I guess re arguments exposed we can think about it down the line.

Additional points:

Distances should be consistently passed imho. This is not done currently for visium. Although maybe redundant/not needed, I think it would still be useful and would improve consistency.
The argument neigh_correct is currently in used but since it's not exposed, maybe it can be removed?
Is there any graph building approach that we havent' considered and that it would be useful for users? Currently we have knn-search, regular grid, radius, delaunay. Pinging @MxMstrmn @hspitzer @AnnaChristina @michalk8 for ideas...

As discussed on MM @Koncopd , would you mind taking a look at this? Thank you!

closed with #340


Neighboring hostnames known to Windows are not known to WSL

In Command Prompt I can run ping laptop and get a response from my laptop using it’s unqualified hostname “laptop”. Under WSL, I get the error unknown host laptop. I can get to my laptop using it’s fully qualified domain name from DNS, but not using it’s search domain, mDNS name (.local),  hostname, or other nearby-machine discovery. This is very inconvenient.

Adding your laptop to /etc/hosts would give you the desired effect.

@Aeyoun - if you do "sudo rm /etc/hosts" we will automatically regenerate one with your machine's hostname the next time you launch bash.

This has nothing to do with either hosts files.
The desired effect is working local network client discovery using NBNS and NBNS or mDNS so I can reach other clients on my network the same way I can from Command Prompt and PowerShell (plus bash on Linux or Mac).

Thanks @Aeyoun.
Some of the confusion here is that on our internal network your scenario appears to be working.  This just makes it slightly harder to track down.
Any chance you can either post or send me an strace of a failing command? Hopefully that will help us track it down.

strace ping hippaforalkus strace: Test for PTRACE_O_TRACESYSGOOD failed, giving up using this feature. execve("/bin/ping", ["ping", "hippaforalkus"], [/* 15 vars */]) = -1 EPERM (Operation not permitted) write(2, "strace: exec: Operation not perm"..., 38strace: exec: Operation not permitted ) = 38 exit_group(1)                           = ? +++ exited with 1 +++
strace ping microsoft.com gives the same error, yet ping microsoft.com actually works.
SSH is probably more interesting:
ssh -p ## d@hippaforalkus
https://gist.github.com/Aeyoun/b317c35247d2800c0611a66a1d917233
ping hippaforalkus, and mounting smb from \hippaforalkus works in Windows on the same machine, and from other Linux and Macs on the same network. WSL seem to only work with domains that are returned from DNS.

WSL doesn't yet support mDNS (as far as I know).  This is covered by #384 .
I'm not really familiar with NBNS under Linux, but I would kind of expect it to fail for similar reasons.
@russalex -- I would expect Dynamic DNS to work, and I would expect Microsoft's network to be using it.  Could that be what's confounding your tests?

Thanks @aseering, I did not see #384.  I have confirmed that mDNS and the dbus dependency called out are on our backlog.
As for my testing, I am not certain how MSFT has set up their corporate network, but I would not be surprised if it was Dynamic DNS.

Related to

https://github.com/Microsoft/BashOnWindows/issues/149
https://github.com/Microsoft/BashOnWindows/issues/731

No, @MadLittleMods. Those issues are entirely unrelated.

Here is a strace of the mdns-scan debug utility:
https://gist.github.com/Aeyoun/87acc829c6e83f9f3cf00719b72fb862

FWIW I got the same result trying to ssh into a raspberry pi on my network - can ping host, qualified host and IP from cmd and cygwin (and osx terminal); but can only ping and ssh to the IP address in WSL.

@cheshrkat  - While it doesn't solve the underlying problem, adding your local hostnames to /etc/hosts does let you connect to them by name until mdns or wins are working.

@rodrymbo thanks for the workaround.

@rodrymbo sure but that completely defeats the point of using mDNS and it isn't updated dynamically if for example the ipv6 address changes via PE rotation. WSL doesn't support mDNS but Windows 10 does, so this is pretty weird. DNS-SD is much more than just mapping ip addresses to names, there's a whole service discovery bit for things that makes it a lot more convenient.

@nimish - Well, workarounds are like that. They let you work for a while until the bigger problem can be fixed. We already know what the bigger problem is; the question is how can we test other parts of the system while that problem gets resolved.
From my perspective, it looks like WSL is passing all networking off to Windows ... except DNS, which the applications running in WSL get from whatever ends up in /etc/resolv.conf or whatever. In order to get things working properly, I'd expect one way to get all of the networking going through Windows is to fix the Windows resolver functionality so it can act as a resolver for WSL, on localhost. That would incorporate all the neat tricks (and legacy workarounds) that Windows knows about, and in real time. Or maybe the Devs are going to implement a full network stack of some sort. Who knows. All that's clear is that WSL DNS is handled separately now from the networking, which causes all sorts of problems with LAN and VPN and other kinds of work we want to do. (Or maybe someone can write a Windows app that pretends to be a resolver, passing results back to WSL when - and only when - something in WSL makes the request.)
This area is where we are supposed to report bugs. It turns out this is a feature or backlog or something, the Devs know about  it, nobody likes that it is broken. To express an opinion about what its priority should be, go to the UserVoice area.
To get some work done while we are waiting, add the names to /etc/hosts and work away. Or maybe stick with Cygwin or a VM. If you know of a better workaround for while we wait for an official resolution (!), I for one would look forward to learning about it. I'm always up for a good workaround.

The workaround doesn't actually work for me since DHCP tends to mess with the ip assignments ;)
In any case #384 is caused by #376, so until then I'll just switch to using putty or something.

@nimish -- Sorry to hear that you have no control over the DHCP server in your testing environment, and its lease times are set so short that assigned IPv4 addresses change so many times a day. :-p But I agree, it will be good to have socket and dbus support added to WSL.

@da2x - Apologize for the delay in responding to this thread and thanks for the feedback! Is this related to #384, i.e essentially avahi-daemon (the popular mDNS implementation on Ubuntu) does not work in WSL? I tried starting the avahi-daemon on WSL and it failed to start. I can see it failing on few socket options such as SO_REUSEPORT, IP_PKTINFO, IPV6_MULTICAST_HOPS and IPV6_UNICAST_HOPS.

@sunilmut Yes if avahi-daemon works that would be a big step as its the key component that's failing

This seems to be resolved as of Creators Update. (Ubuntu 16.04.2 LTS) I don't know if it's directly related, but avahi-daemon installs and runs without complaint, as well.

fyi, see the update

Marking this dupe of #384.

This should be fixed in insider build 16273

I am marking this issue closed. If it persists after upgrading to 16273, please let us know and we can reopen it.

I continue to have this issue as of 17134

I still have this issue as well. Looking at the logs, this seems to be related to th IP_ADD_MEMBERSHIP errors that were also reported on this issue: https://github.com/Microsoft/WSL/issues/3365


[Program]: Write a Swift program to find sum of gp series

Description
Write a Swift program to find sum of gp series
A geometric progression (GP) is a sequence of numbers where each term after the first is found by multiplying the previous one by a fixed, non-zero number called the common ratio. The nth term of a GP is given by the formula an = a1rn-1 where a1 is the first term and r is the common ratio.
Input  : 2, 3, 3
Output : 26

How to contribute

Save the solution in FindSumOfGpSeries.swift file
Add FindSumOfGpSeries.swift file in program/find-sum-of-gp-series folder

!assign


Changes to better house-keep the operator (MULTI-90, MULTI-91)

Ensure that the operator only watches it's own namespace.
When the version is updated on the status of the install resource (kubefed-resource), make sure that the reconcile loop doesn't run again.

/lgtm


Add basic support for a list of repos as input

@aaronsteers  This is a draft PR for feedback, as I'm not familiar enough with the SDK to make optimal use of it :)
I've added basic support for passing a list of repos to fetch data for instead of passing a search query. It only works for the repositories stream now, and I sort of abuse the search/repositories API endpoint by using its repo:xxx query mode, which returns exactly the repo passed by name (docs).
A few questions:

I didn't find a way to do an "XOR" sort of declaration in jsonschema for the config (to force having searches XOR repositories). There seems to be a oneOf in JSON Schema itself, but I don't know if it makes more sense to try and implement that in typing.py or just raise an exception for invalid config in tap-github itself.
in test_tap.py I try to validate that I'm getting the correct number of data records, but I suspect you might have a helper function for that purpose. I'm not sure what the best pattern for testing the output of a tap is.
The test I added actually fetches records from the github API. While I think it provide some level of safety against possible future changes in the API (because the tests would break in such a case), it also makes the tests brittle because of rate limiting (I got bitten!) or whatever networking problem. It might make more sense to use mock API responses for faster tests and less variability though.
the response content is slightly different between the repos/{org}/{repo} and search/repositories endpoints, with the former adding a few more fields, mainly about the organisation holding the repo. I don't know if it makes sense to have 2 different schemas here, based on what the config params were?

Happy to hear your thoughts on these points before I add support for other streams.

Related to issue #5

@laurentS - Do you mind updating from main now that #6 is merged? Would love to see the CI tests before merging.

it's not perfect, but the "tests" are passing. I has to tweak them a bit when checking the number of records produced, because the stream selection wasn't working (as discussed on slack), despite upgrading to the latest sdk you released yesterday. The tap's output looks ok though. What do you suggest as a method for testing a bit more than just "it runs without crashing"?

Sounds good to me. Thanks for merging so quickly, much appreciated!


metadata: Keywords should be stored as comma-separated values

https://packaging.python.org/en/latest/specifications/core-metadata/#keywords
Keywords are being joined with  spaces in sdist PKG-INFO and wheel METADATA files. I'm not sure this is scikit-build-core's fault. Maybe the root of the issue is from pyproject_metadata? In any case, scikit-build-core may be interested in adding a workaround.

https://github.com/FFY00/python-pyproject-metadata/pull/61. A new release of pyproject-metadata is needed.

Should be fixed via pypa/pyproject-metadata#61.


Dialogs of modules remain on screen if module is deleted.

Describe the bug
Dialogs of modules remain on screen if module is deleted.
Application: Brain Simulator
Module/Network: Any module with a dialog.
To Reproduce
Steps to reproduce the behavior:

Create a module that has a dialog, like Module2DSim.
Open the dialog.
Right-click the module and delete it.
Notice the dialog needs to be closed separately.

Expected behavior
I expect all traces of the module to disappear on delete.
Even synapses leading into or out of the module (is this even used / possible?)
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10 Pro  20H2
Version 2021-04-20 master

Add any other context about the problem here.

Problem fixed.


[Bug]: select-card ignores translations for select entity state/option

Requirements

[X] I have updated Mushroom to the latest available version
[X] I did a search to see if there is a similar issue or if a pull request is open.

Current Behavior
If a translation of the select entity state is available, the card ignores this translation.

This state is translatable in Lokalise:

Expected Behavior
The card should display the translation, not the raw state like the core card does.

Steps To Reproduce
No response
Context
YAML state

Environment
- Browser: Firefox/iOS companion app
- HA Version: 2023.4.6
- Mushroom: 2.7.0

Anything else?
No response

Duplicate


chore(deps): update common-dev-assets digest to ff3f81f

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
8405107 -> ff3f81f

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 2.6.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


compute_and_store_features() got an unexpected keyword argument 'augment_fn'

hi; when i use mini_librispeech dataset and change dataset path, the error coming...

pip list:
python                  3.7
torch                     1.6.0+cu101
torchaudio            0.6.0
torchvision            0.7.0+cu101
lhotse                    0.1.1
k2                          0.1

what are the dev-clean-3 and train-clean-5 here? can you try to run with the original script to see if you get the same error? I mean, just clone the repo and don't change any code in it.

Maybe it was a change in lhotse interface and we need to change our code.
On Wed, Nov 18, 2020 at 6:30 PM Haowen Qiu<EMAIL_ADDRESS>wrote:

what are the dev-clean-3 and train-clean-5 here? can you try to run with
the original script to see if you get the same error? I mean, just clone
the repo and don't change any code in it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/snowfall/issues/22#issuecomment-729587466, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO5AHMNHY27LQY5MW7DSQOO5XANCNFSM4TZXTWQQ
.

Not sure, will check

OK, it should be augmenter with pip package lhotse 0.1.1,
CutSet.compute_and_store_features.__code__.co_varnames
('self', 'extractor', 'storage', 'augmenter', 'executor', 'mix_eagerly', 'futures', 'cut', 'cut_set')

but in master branch of lhotse, it now is augment_fn
>>> lhotse.CutSet.compute_and_store_features.__code__.co_varnames
('self', 'extractor', 'storage', 'augment_fn', 'executor', 'mix_eagerly', 'futures', 'cut', 'cut_set')

https://github.com/lhotse-speech/lhotse/blob/2167c774383b815dc0929e9cc6bb7d26eec0fccb/lhotse/cut.py#L325
@pzelasko can you update pip package of lhotse?
@lvchigo for now, you can just do
git clone https://github.com/lhotse-speech/lhotse
cd lhotse
pip install -e '.[dev]'

to install lhotse from local src (don't forget uninstall your previous version)

OK I’ll release a new Lhotse version. Was waiting to accumulate some changes first. BTW Did you guys figure out how to auto-upload nightly builds to pip?

Wiadomość napisana przez Haowen Qiu<EMAIL_ADDRESS>w dniu 11/18/20, o godz. 06:38:

OK, it should be augmenter with pip package lhotse 0.1.1,
CutSet.compute_and_store_features.code.co_varnames
('self', 'extractor', 'storage', 'augmenter', 'executor', 'mix_eagerly', 'futures', 'cut', 'cut_set')
but in master branch of lhotse, it now is augment_fn

lhotse.CutSet.compute_and_store_features.code.co_varnames
('self', 'extractor', 'storage', 'augment_fn', 'executor', 'mix_eagerly', 'futures', 'cut', 'cut_set')

https://github.com/lhotse-speech/lhotse/blob/2167c774383b815dc0929e9cc6bb7d26eec0fccb/lhotse/cut.py#L325
@pzelasko can you update pip package of lhotse?
@lvchigo for now, you can just do
git clone https://github.com/lhotse-speech/lhotse
cd lhotse
pip install -e '.[dev]'
to install lhotse from local src (don't forget uninstall your previous version)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I had something that might be usefulset up here:
https://github.com/sequitur-g2p/sequitur-g2p/blob/master/.travis.yml
(it's not automatic daily, it needs a tag, but you could tag automatically,
I guess)
y.
On Wed, Nov 18, 2020 at 7:50 AM Piotr Żelasko<EMAIL_ADDRESS>wrote:

OK I’ll release a new Lhotse version. Was waiting to accumulate some
changes first. BTW Did you guys figure out how to auto-upload nightly
builds to pip?

Wiadomość napisana przez Haowen Qiu<EMAIL_ADDRESS>w dniu
11/18/20, o godz. 06:38:

OK, it should be augmenter with pip package lhotse 0.1.1,
CutSet.compute_and_store_features.code.co_varnames
('self', 'extractor', 'storage', 'augmenter', 'executor', 'mix_eagerly',
'futures', 'cut', 'cut_set')
but in master branch of lhotse, it now is augment_fn

lhotse.CutSet.compute_and_store_features.code.co_varnames
('self', 'extractor', 'storage', 'augment_fn', 'executor',
'mix_eagerly', 'futures', 'cut', 'cut_set')

https://github.com/lhotse-speech/lhotse/blob/2167c774383b815dc0929e9cc6bb7d26eec0fccb/lhotse/cut.py#L325

@pzelasko can you update pip package of lhotse?
@lvchigo for now, you can just do
git clone https://github.com/lhotse-speech/lhotse
cd lhotse
pip install -e '.[dev]'
to install lhotse from local src (don't forget uninstall your previous
version)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/snowfall/issues/22#issuecomment-729656031, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUKYX74PGF6WBA7RF4F3U3SQO7H5ANCNFSM4TZXTWQQ
.

thank you guys... it's worked...

uninstall lhotse 0.1.1

install lhotse 0.2.1

final pip list:
python 3.7
torch 1.6.0+cu101
torchaudio 0.6.0
torchvision 0.7.0+cu101
lhotse 0.2.1
k2 0.1

RE automatic builds that upload to pip, I think @csukuangfj might be able to comment.
I believe there we have some kind of automated build but currently only uploads artifacts to github, not pip which would require password.

GitHub actions support uploading wheels to PyPI. As for the password, it can be saved in the settings of your account and be accessed in GitHub actions as environment variables. Others can only see the name of the environment variables,  but not their values.  So the username and password can still be kept as secrets.
GitHub actions can be triggered by some events, e.g., on the creation of a tag, on a new pull request, on a merge into a specific branch, at some specified time points per day etc.
Sent from myMail for iOS
Thursday, 19 November 2020, 11:20 +0800 from<EMAIL_ADDRESS><EMAIL_ADDRESS>
RE automatic builds that upload to pip, I think  @csukuangfj might be able to comment.
I believe there we have some kind of automated build but currently only uploads artifacts to github, not pip which would require password.
—
You are receiving this because you were mentioned.
Reply to this email directly,  view it on GitHub , or  unsubscribe .

Thanks! I’ll check it out at some point and set it up.

Wiadomość napisana przez Fangjun Kuang<EMAIL_ADDRESS>w dniu 11/18/20, o godz. 22:45:

GitHub actions support uploading wheels to PyPI. As for the password, it can be saved in the settings of your account and be accessed in GitHub actions as environment variables. Others can only see the name of the environment variables,  but not their values.  So the username and password can still be kept as secrets.
GitHub actions can be triggered by some events, e.g., on the creation of a tag, on a new pull request, on a merge into a specific branch, at some specified time points per day etc.
Sent from myMail for iOS
Thursday, 19 November 2020, 11:20 +0800 from<EMAIL_ADDRESS><EMAIL_ADDRESS>
RE automatic builds that upload to pip, I think @csukuangfj might be able to comment.
I believe there we have some kind of automated build but currently only uploads artifacts to github, not pip which would require password.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub , or unsubscribe .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


feat: endpoint to delete invites (superadmin)

Description
Create an API endpoint to allow superadmins delete invites.
This endpoint will validate the invite id and then delete the invite from the database.

The changes include:

Endpoint Addition: Added DELETE /api/v1/invite/{invite_id} for deleting invites.
Error Handling: Implemented checks for invalid invitation id and unauthorized access.
Testing: Added pytest cases to ensure functionality and edge cases are covered.

Related Issue (Link to issue ticket)
Issue Ticket
Motivation and Context
This feature is essential to enable superadmins delete invites from the database.
How Has This Been Tested?

Unit Tests: Added tests to validate the deletion of invites from the database, authorization failures and invalid request body.
Test Environment: Tests were run using a mocked database instance to avoid interference with the production environment.

Tests include:

Deleting valid invitations.
Handling invalid invalid id parameter
Handling unauthorized access.

Screenshots:
Successful Request
Invalid id parameter
Forbidden Request
Types of changes

[x]  New feature (non-breaking change which adds functionality)

Checklist:

[x]  My code follows the code style of this project.
[x]  My change requires a change to the documentation.
[x]  I have updated the documentation accordingly.
[x]  I have read the CONTRIBUTING document.
[x]  I have added tests to cover my changes.
[x]  All new and existing tests passed.

I have no idea why delete request is returning 200 though

If there's a response data, then a 200 is appropriate.
If you'd rather there be no response data, I can change to a 204


Create logos for Gazorator and initial readme

Where dafuq did I get grazorator from?! My brain is broken. Let me fix this PR up.

@bjorkstromm was there a reason that this wasn't merged in?
These logos look amazing!

@gep13 just me missing that @khalidabuhakmeh had fixed the PR feedback 😄
Thanks again @khalidabuhakmeh for the awesome logos and README!


One alternative way to make it much faster would be to serialize the function body (what if it contains closure?) and execute the function in a multiprocess fashion (e.g. from a child process). This is quite tricky in general.

Another crazy idea is that we can draw the plot lazily by storing the Tensor arguments and the function body (serialized, cannot contain closures, maybe) into summary eventfiles and then executing the plot code from the tensorboard process. A custom tensorboard extension will be needed to make it work.


[EMB-347] fix(register): let CAS redirect to ORCID

Purpose
We need to bounce off CAS first before we go to ORCID.
Summary of Changes
Go to CAS with redirectOrcid=true instead of ORCID directly.
Side Effects
none
Feature Flags
ember_auth_register
QA Notes
ORCID sign up should full work now.
Ticket
https://openscience.atlassian.net/browse/EMB-347
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Pull Request Test Coverage Report for Build 1941

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

Totals

Change from base Build 1937:
0.003%

Covered Lines:
3786

Relevant Lines:
5674

💛  - Coveralls


Add support for Boolean OptionSet button type

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Other (updates to documentation, formatting, etc.)

Description
Added AppElement Reference to UCI for BooleanFieldContainer and True / False buttons.
Added check for button control to WebClient.cs
Issues addressed
If you have a Boolean field displaying as Buttons, EasyRepro fails to set the value
All submissions:

[x] My code follows the code style of this project.
[ ] Do existing samples that are effected by this change still run?
[ ] I have added samples for new functionality.
[ ] I raise detailed error messages when possible.
[x] My code does not rely on labels that have the option to be hidden.

Which browsers was this tested on?

[x] Chrome
[ ] Firefox
[ ] IE
[ ] Edge

 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.:x: dylanhaskins sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Save generated plots

Hi,
I have added the option to save the generated plots based on the suggestions in Issue #14. Here are the changes I have made.
1. Add argument fname to the plotting functions
If fname is specified then simply calls plt.savefig(fname) to save the plot result.
2. Add argument save_dir to visual_evaluation() function
If save_dir is specified then saves all generated plots into this directory. Each image has a fixed name in this case.
Hope this is useful.

@vnykr Looks great!
Could you make develop the target? That's the branch where it should be merged first, since I'm following a gitflow working method. Thanks!

@Baukebrenninkmeijer My apologies for targeting the wrong branch, this is my first time making a pull request!
I think I have made the correct changes, please let me know if there is anything else.