update maintainers file for parsing

update standard for dockerfiles so we can clean up dockers and parse
into one repo
see https://github.com/docker/opensource/pull/35
https://github.com/docker/docker/pull/18321
Signed-off-by: Jessica Frazelle<EMAIL_ADDRESS>

it works https://github.com/docker/opensource/pull/35/files#diff-bb5907b965b501e1856ccd79d2c4f642R34

this was just a test y'all can update yoself now

reopening to save you all the trouble and @thajeztah is working on PRs to the other subcomponents

@jfrazelle Thanks lots for updating this for us!  LGTM, although it looks like it could use a rebase.

Oh, that's the GitHub thingy because of protected branches :disappointed:

the button is green for me?

@jfrazelle you're "owner" so you're allowed to merge even if it's "out of date"

oh :innocent:

ok i fixed the setting

LGTM (but not a maintainer here)

@jfrazelle Out of curiosity, what is docker/is-mergable?

oh i have no idea sorry but looks cool
On Thu, Dec 3, 2015 at 4:41 PM, Ying Li<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle Out of curiosity, what is
docker/is-mergable?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/notary/pull/333#issuecomment-161838236.

other projects have started merginig these cough cough

thanks hehehe guilt trip was strong :innocent:
On Fri, Dec 4, 2015 at 9:24 AM, Ying Li<EMAIL_ADDRESS>wrote:

Merged #333 https://github.com/docker/notary/pull/333.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/notary/pull/333#event-482874067.

:D

also check it outtttt
https://github.com/docker/opensource/blob/add-maintainers-file-parsing/MAINTAINERS#L244
On Fri, Dec 4, 2015 at 9:27 AM, Jessie Frazelle<EMAIL_ADDRESS>wrote:

thanks hehehe guilt trip was strong :innocent:
On Fri, Dec 4, 2015 at 9:24 AM, Ying Li<EMAIL_ADDRESS>wrote:

Merged #333 https://github.com/docker/notary/pull/333.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/notary/pull/333#event-482874067.

nice!


Future: fix cram tests

This improves and fixes the cram tests.
tests/pip-compile.t is  still failing, but that is because of missing pip-sync etc.

I've redesigned everything, making all tests useless (see #152). We should add them back in, but it requires a little bit of thought and organization, so not just yet!

@nvie
Great news! (although you could have announced it before maybe, or was it spontaneous? :))
Anyway, why don't you think that the current cram test suite could be used until there is something better?
At the very least this would show up any regressions/changed caused by the rewrite, too.

Hey Daniel. Yeah, sorry for this to be a little sudden. It was indeed rather spontaneous. I was looking at these incoming pull requests from last week, and felt like they were all hacks that were necessary due to some design choices made in the past that weren't the best way forward. (This is not to say the authors were putting hacks together! Instead, the core wasn't simple enough to support many use cases in the first place.)
Then I just thought "well, screw this, I'll try to rewrite it from scratch right now".
I feel that sometimes an issue with spending free time on open source projects is that some work simply needs to be done that requires a burst of attention, rather than a few hours here and there.

Thanks for your reply, I totally agree.
However, I still think that the existing cram tests would be still useful (when fixed / adjusted) for now.


UG Images/Tabling QA

Add tables/update images with arrows, etc for clarity

Update images/description for filter command

UG changes to be done:

Use "you" language when referring to user
Revamp edit command
Explain how multiple tags are helpful in addc
Give better explanation for editc tag removal
Give better examples for addt interval (Explain use of the weekly/daily/monthly tags first, then explain how u can customize using the interval integers)
Give better reasons/descriptions of feature beyond 1 sentence/1liners
Add a para at the start breaking down the UG: how it is split into contact vs task features, how they fit together, how tips/graphics help them understand
Shift view command to before unassign and after assign
I think in the sample Ui.png we can include annotations that show red task = overdue, yellow = upcoming, etc
Update headers of each command to fit: locate persons by name: findc -> Find contacts by name
Change persons to contacts

++ Add annotations onto diagrams


Merge llvm branch to master

Opening PR to run the CI

Apart from the changes regarding the LLVM backend the llvm branch includes the following changes that can be ported to master of NMODL:

[ ] Addition of ==, += and -= operators in ast_common.hpp
[ ] Move any check related to CodegenInfo members from CodegenCVisitor to CodegenInfo (see src/codegen/codegen_c_visitor.cpp).  Important is setting up the vector of codegen_int and float variables
[ ] print_channel_iteration_loop in src/codegen/codegen_c_visitor.cpp
[ ] Move various utility functions and struct declaration from codegen_c_visitor.hpp to codegen_info.hpp to be used by multiple codegen visitors (to be moved to CodegenCppVisitor in current master
[ ] Move visitor execution from main.cpp to CodegenDriver. main instantiates with the CLI options the CodegenDriver and executed the correct CodegenVisitor
[ ] Changes in codegen_helper_visitor.{cpp,hpp} to accomodate changes in CodegenInfo and CodegenCVisitor
[ ] Additional utility functions in src/codegen/codegen_info.{cpp,hpp}
[ ] Additional naming constexpr variables in src/codegen/codegen_naming.hpp
[ ] See whether Codegen* language constructs in src/language/codegen.yaml are useful
[ ] Add Expression documentation to master from src/language/nmodl.yaml
[ ] Add to_string function in Ast nodes
[ ] Change in src/visitors/inline_visitor.cpp to instantiate VarName instead of Name

Note that the LLVM API used was based on LLVM 11.
Also to merge this PR there are numerous conflicts needed to be handled due to the separation of CodegenCVisitor in master to CodenCppVisitor, CodegenCorenrnCppVisitor, CodegenNeuronCppVisitor, etc.


Various upstream cherry-picks

Return proper gRPC converted errors to be handled by client.
Use correct wait context for StopPodSandbox, StopContainer, and ExecSync when waiting for the container to stop.

@katiewasnothere - PTAL


Watch through etcdv2 HTTP API, any way to get all continuous modifications with one request?

As title, for example:
Watch at index 100,  at etcd server  modifications at index 110 and 120 are related,  so this time http response is info about index 110.
Is there any way to get all infos of index 110 and 120 with only one http watch request?

v2 store does not support historical data.
So if you watch on a key, for every write, previous value will be overwritten.
You have to track those manually.
I would try v3 API.

@gyuho  thanks
https://coreos.com/etcd/docs/latest/v2/api.html
It mentioned "etcd only keeps the responses of the most recent 1000 events across all etcd keys"
If all related modifications are all in recent 1000 event, I think there is some way to get them through one HTTP request.

@2225377fjs Try client.WatcherOptions{AfterIndex: x}?


Default ordering for list of orthologs to be phylogenetic

As discussed on the 11/16 call

It was not clear from the gene pair-wise ortholog file how to interpret the phylogeny. This aspect needs more specifications and will not get done by the Dec 6th meeting, but will be added to the enhancement list for the future.

Right, Paul will have to point you at the ordering (without branch lengths is good enough for the simple list, though these will be needed for drawing a tree). Here is (some of) what I have derived from PANTHER as of today. (apologies for not using latin, these are shorter to type) which we should use not because it is necessarily "right", but it would make things consistent (and can be easily changed as needed).
human
chimp
macaque
mouse
rat
cow
pig
horse
cat
dog
opossum
anole
frog
fugu
zebrafish
lancelet
sea squirt
sea urchin
fruitfly
mosquito
water flea
trichoplax
dictyostelium
rice
stiff brome
grape
thale cress (arabidopsis)
soybean
poplar
moss
etc... (smaller and smaller things)

For what it's worth, a worm needs to be slipped in there. Not sure where exactly, but definitely before the plants!
http://www.treefam.org/browse#tabview=tab2

Closing old prototype-related issues that are not relevant to 0.3.0+. Please re-open if this is incorrect! Thanks.


use random port numbers in integration tests

This helps prevent “address already in use” errors that occur from time to time when running the tests with the untilItFails option.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #559   +/-   ##
=======================================
  Coverage   90.79%   90.79%
=======================================
  Files          79       79
  Lines        5407     5407
=======================================
  Hits         4909     4909
  Misses        318      318
  Partials      180      180

Continue to review full report at Codecov.

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

I'm working on a complete rewrite of the UDP Proxy, which allows binding to port 0.
That's why I removed you as a reviewer here, but I guess Github didn't notify you about that...

Superseeded by #560.


Add SPIFFE ID to sentry workload certs

This PR is part 1 of the epic #813 and does the following:

Adds strong SPIFFE based IDs to workload certs issued by Sentry
Sentry issued workload certs are now valid SPIFFE SVIDs
Updates tests

Every workload cert now carries a SPIFFE ID containing the trust domain, namespace and ID of the app as a SAN extension in the x509 certificate.
This can now be used by the Dapr sidecar to validate the caller identity and perform access list authorization.
In addition, a nit cleanup was made to change the order of the identity elements from ServiceAccount:Namespace to Namespace:ServiceAccount to match better with the way it's done in Kubernetes.

/cc @vinayada1

/ok-to-test

/ok-to-test

@youngbupark tests were failing due to this error:
Failed Platform.addApps(), cannot find dapr sidecar in pod httpmetrics-6b68845f5d-p7b6b2020/09/16 00:13:27 Running teardown...
FAIL	github.com/dapr/dapr/tests/e2e/metrics	4.029s

/ok-to-test

Want to see if the error is persistent.

Ok, this is intermittent. Still needs a triage to see if this was a one off, pre-existing failure, or an intermittent bug introduced by this change.

@vinayada1 SPIFFE change in sentry is merged.


Proposal: Introduce the notion of a middleware stack

Currently, the type of a collection of middlewares is Vec<Arc<dyn Middleware<Data> + Send + Sync>>. I think it would make sense to introduce a proper MiddlewareStack type that itself implements Middleware. This allows better sharing of middleware instances and the way to address them, for example for debugging.

I'd like to take this to start contributing to tide. Seems pretty simple:
impl<Data> Iterator<Item=&Middleware<Data>> for MiddlewareStack<Data> {
    // ...
}

impl<Data> FromIterator<Item=Box<Middleware<Data>> for MiddlewareStack<Data> {
    // ...
}

impl<Data> Middleware<Data> for MiddlewareStack<Data> {
    fn handle(/* ... */) -> FutureObj<'a, Response> {
        // Apply internal middlewares in order..
    }
}

Anything I'm missing?

Is this still valid? If yes, is anyone working on this? If not, I'd like to start working on this.
@RoGryza
@skade

Sorry for the delay, I hacked something together that works but I've introduced some allocations that could be eliminated and never got time to work on it again. I'll submit a PR once I get home from work in order to ask for help

This is worth revisiting now that #156 has landed -- anybody want to take it back up?

I can take a look if no one else is already working on it.

I am taking a look at this and came up with the following
pub struct MiddlewareStack<State> {
    middlewares: Vec<Arc<dyn Middleware<State> + Send + Sync>>,
}

impl<State> MiddlewareStack<State> {
    pub fn new() -> MiddlewareStack<State> {
        MiddlewareStack {
            middlewares: Vec::new(),
        }
    }

    pub fn push(&mut self, m: impl Middleware<State>) {
        self.middlewares.push(Arc::new(m));
    }
}

impl<State: 'static> Middleware<State> for MiddlewareStack<State> {
    fn handle<'a>(&'a self, cx: Context<State>, next: Next<'a, State>) -> BoxFuture<'a, Response> {
        if let Some((head, tail)) = self.middlewares.split_first() {
            let next = Next {
                endpoint: next.endpoint,
                next_middleware: tail, // todo: append middlewares from next
            };
            head.handle(cx, next)
        } else {
            next.run(cx)
        }
    }
}

But I got in trouble with the incompatible types of MiddlewareStack.middlewares and Next.next_middlewares
Are there some strong bounds on the type of Next.next_middlewares that I should keep into account or it is possible to work on that?

I don't know of any bounds that next would need. I'm not even sure if the Stack itself needs the 'static bound. (an external user could apply it)

Thanks @skade, I cleaned up my previous attempt.

/// Middleware stack.
pub struct MiddlewareStack<State> {
    stack: Vec<Arc<dyn Middleware<State>>>,
}

impl<State> MiddlewareStack<State> {
    /// Create an empty stack of middlewares.
    pub fn new() -> MiddlewareStack<State> {
        MiddlewareStack {
            stack: Vec::new(),
        }
    }

    /// Add a middleware to the stack.
    pub fn push(&mut self, m: impl Middleware<State>) {
        self.stack.push(Arc::new(m));
    }
}

impl<State: 'static> Middleware<State> for MiddlewareStack<State> {
    fn handle<'a>(&'a self, cx: Context<State>, next: Next<'a, State>) -> BoxFuture<'a, Response> {
        if let Some((last, others)) = self.stack.split_last() {
            last.handle(cx, Next::new(next.endpoint, others))
        } else {
            next.run(cx)
        }
    }
}

I am not sure if the middleware stack should behave like a stack as the name suggests or like a queue (respectively split_last and split_first). Maybe I can start a pull request and collect some comments/reviews?

As i see it we have three options how to approach this:

Add another field To Next next: Option<Arc<Next>> and in the MiddlewareStack' create new Nextthat warp the original Next, and in theNext::run` function call this next if it exist .
Improvement to 1 :Change Next to be trait instead of struct, this will allow to create a special Next in `MiddlewareStack' without interfering with the Current Next Implementation. But this might present a problem letting users too much control.
Completely inverse the solution and instead of MiddlewareStack'  impl Middleware' we should let MiddlewareStack' expose a vec\slice of Middlewarethat will be added to the server(add a function that accept vec\slice ofMiddleware`).

I Tried 1 but got stuck because i am lacking knowledge and experience to work good enough with generic and lifetime.
But after second thought i think that 3 is better.


this select block is useless since panicChan is not buffered, please remove it.

https://github.com/TimothyYe/godns/blob/d3f82cffe63c759f4da7ca3a0749126d552cdc15/godns.go#L64

@georgetso I'm also thinking about to remove panicCount, panicChan and related codes. If panic occurs, just let the program exit, since upstart/systemd/docker will restart it automatically.

removed.


Mcedit Save error

I have a minecraft pocket edition world which I have editied with mcedit in the past. I've frequently had to use mcedit's "relight" feature to fix lighting glitches. however, recently every time I try to save I get this:

I don't have much experience with code or anything and I really have no idea what the problem is. I kind of assume that it has something to do with stone brick since 98 is it's value, but other than that i'm clueless. Any help is appreciated.

I'm running into the exact same error but its not the stone bricks cause my edits were with smooth stone. Also I know this unrelated to the topic but are you a map maker and if so were can I download them?

Question, by any chance do both of these worlds have nethers? and if they do does it work if you try a world with no nether?

Can you send MCEdit log entries, please?
This will help us to track the exact cause of the problem.

Try the latest build, we (Lachal) has added code to handle missing entities on pocket.

Missing entities are handled correct for me in the log.
Thanks :)


Add --android option to ./mach package

https://github.com/servo/servo/pull/11210 recently updated ./mach package to make it a first-class Mach command and add support for
simple Linux packaging as well (in the form of tarballs). Android used
to be the only packaging target supported, so a naked ./mach package
sufficed, but now we need to explicitly build the Android packaging.
Another step for servo/servo#10339.

This change is

@bors-servo r+

:pushpin: Commit de85289 has been approved by larsbergstrom

:hourglass: Testing commit de85289 with merge 8cf5982...


Fix typo

Doc link: https://docs.microsoft.com/en-us/aspnet/core/security/anti-request-forgery#configuring-antiforgery

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@gldraphael  thanks!


OnMath scrollback ordering problem?

The following commands in IRC:
<`lm`> 5*90
<^> 450
<`lm`> !quo 2,1

produce the following quote:
<^> 450
<`lm`> 5*90

which is not as it ought to be.
--EDIT--
However, something similar:
<`lm`> !hb
<^> HAPPY BIRTHDAY SHEPARD
<`lm`> !quo 2,1

produces output as expected:
<`lm`> !hb
<^> HAPPY BIRTHDAY SHEPARD

---EDIT---
Perhaps there is an issue with how OnMath responses are placed into scrollback?

I just glanced at onMath, as it's been a while, and it just uses reply for output like this. I'd 👉  at scrollback instead.

Duplicate of #131


[Fatal Error] :1:3: The markup in the document preceding the root element must be well-formed. in scala compile

I have a scala project which used to work just fine, but suddenly out of work yesterday...
I have tried with invalidate Intellij cache, delete the project and reclone again, restart my laptop, reinstall the sbt plugin, none of them worked...
Weird thing is my other scala sbt project worked just fine...
I use Intellij: IntelliJ IDEA 2021.3.2 (Community Edition) Build #IC-213.6777.52
my local sbt is 1.3.2
In my project the build.properties is：sbt.version=1.5.0
scala version is 2.12.15
java version is Zulu version 11.0.13
scala plugin for Intellij: 2021.3.22
It seems generating sbt-structure.xml failed... Not sure about the exact error...
/Library/Java/JavaVirtualMachines/zulu-11.jdk/Contents/Home/bin/java -Djline.terminal=jline.UnsupportedTerminal -Dsbt.log.noformat=true -Dfile.encoding=UTF-8 -Xms8G -Xmx8G -Xss20M -XX:MaxMetaspaceSize=1G -XX:ReservedCodeCacheSize=500M -XX:+TieredCompilation -Dhttp.nonProxyHosts=<IP_ADDRESS>|localhost -Xmx8192M -Xms512M -Xmx1536M -Xss1M -XX:+CMSClassUnloadingEnabled -Didea.managed=true -Dfile.encoding=UTF-8 -Didea.installation.dir=/Users/username/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/213.6777.52/IntelliJ IDEA CE.app/Contents -jar /Users/username/.sbt/sbt/bin/sbt-launch.jar
[info] welcome to sbt 1.5.0 (Azul Systems, Inc. Java 11.0.13)
[info] loading settings for project global-plugins from credentials.sbt,plugins.sbt ...
[info] loading global plugins from /Users/username/.sbt/1.0/plugins
[info] loading settings for project index-builder-build from plugins.sbt ...
[info] loading project definition from /Users/username/IdeaProjects/index-builder/project
[info] loading settings for project root from build.sbt,release.sbt ...
[info] resolving key references (10450 settings) ...
[info] set current project to index-builder (in build file:/Users/username/IdeaProjects/index-builder/)
[info] sbt server started at local:///Users/username/.sbt/1.0/server/abaf94f632f9aa72be1a/sock
[info] started sbt server
sbt:index-builder>
;set root.scala.collection.Seq(historyPath := None,shellPrompt := { _ => "" },SettingKey[root.scala.Option[root.sbt.File]]
("sbtStructureOutputFile")
in root.sbt.Global := root.scala.Some(root.sbt.file("/private/var/folders/v9/vbxmj8w54bn_9s_258z0z34c0000gq/T/sbt-structure.xml")),SettingKeyroot.java.lang.String
in root.sbt.Global := "download, resolveClassifiers")
[info] Defining Global / sbtStructureOptions, Global / sbtStructureOutputFile and 1 others.
[info] The new values will be used by cleanKeepGlobs
[info] 	Run last for details.
[info] Reapplying settings...
[info] set current project to index-builder (in build file:/Users/username/IdeaProjects/index-builder/)
[info] Applying State transformations org.jetbrains.sbt.CreateTasks from /Users/username/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/213.6777.52/IntelliJ IDEA CE.app.plugins/Scala/repo/org/jetbrains/scala/sbt-structure-extractor_2.12_1.0/2021.3.0/sbt-structure-extractor-2021.3.0.jar
[info] Reapplying settings...
[info] set current project to index-builder (in build file:/Users/username/IdeaProjects/index-builder/)
[warn] sbt 0.13 shell syntax is deprecated; use slash syntax instead: Global / dumpStructure
[Fatal Error] :1:3: The markup in the document preceding the root element must be well-formed.
[Fatal Error] :1:3: The markup in the document preceding the root element must be well-formed.
[error] stack trace is suppressed; run 'last standaloneIndexer / update' for the full output
[error] stack trace is suppressed; run 'last bulkIndexer / update' for the full output
[error] stack trace is suppressed; run 'last standaloneIndexer / ssExtractDependencies' for the full output
[error] stack trace is suppressed; run 'last docs / ssExtractDependencies' for the full output
[error] stack trace is suppressed; run 'last bulkIndexer / ssExtractDependencies' for the full output
[error] stack trace is suppressed; run 'last standaloneIntegrationTest / ssExtractDependencies' for the full output
[error] (standaloneIndexer / update) org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 3; The markup in the document preceding the root element must be well-formed.
[error] (bulkIndexer / update) org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 3; The markup in the document preceding the root element must be well-formed.
[error] (standaloneIndexer / ssExtractDependencies) org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 3; The markup in the document preceding the root element must be well-formed.
[error] (docs / ssExtractDependencies) org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 3; The markup in the document preceding the root element must be well-formed.
[error] (bulkIndexer / ssExtractDependencies) org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 3; The markup in the document preceding the root element must be well-formed.
[error] (standaloneIntegrationTest / ssExtractDependencies) org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 3; The markup in the document preceding the root element must be well-formed.
[error] Total time: 2 s, completed Jul 6, 2023, 12:24:30 PM
[info] shutting down sbt server

[error] (standaloneIndexer / update) org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 3; The markup in the document preceding the root element must be well-formed.

This error indicates that sbt failed at parsing an ill-formed XML or POM file while resolving the dependencies. Unfortunately it does not indicate which file. You can try to use a clean Coursier cache to see if it re-downloads correct dependencies. Or you can try to bisect your dependencies to identify which one fails.

I sort the file by timestamp and find one corrupted xml file...by removing this folder the issue was gone!!! thank you so much!!!


issues when building tools and tests

When linking the v3d_info tool, the linker complains with unresolved externals.
Patching  -Wl,--no-as-needed -ldl -pthread manually into build/tools/CMakeFiles/v3d_info.dir/link.txt  solves the issue.
When building the tests, TestData.h is missing.
(using a raspi running bullseye)

What are the externals that are not found?

When building Tests from add_subdirectory(test) in the CMakefile.txt,   test.cpp and TestExecutions.cpp miss TestData.h

Sorry, you asked for the unresolved externals. I didnt note them. pthread_create as far as I remember and dlopen to dlsym.

Just cloned the repository and compiled again, but now everything compiles fine. Might eventually come from me compiling VC4CL prior to VC4CLStdLib and VC4C.

Just confirmed. TestData.h is in the VC4C/test directory. Also the unresolved externals are gone. Was def the wrong way to start with VC4CL. Up and running now:
clinfo -l
Platform #0: OpenCL for the Raspberry Pi VideoCore IV GPU
Device #0: VideoCore IV GPU
Thanks a lot and sorry for the inconvenience!


chore(marketing): add KM tracker (v2)

Deploy preview ready!
Built with commit 5abedbb3b18ff23bf6f75d71b67ac30bf982829f
https://deploy-preview-2065--algolia-instantsearch.netlify.com


Chore: Update link with utm code

What changed? Why?

updated the link on the docs basenames launch modal

Notes to reviewers
How has it been tested?
locally

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Cache output directory

Resolves #39.
Figured this is worth a try. We can revert it later if it's not working out.
Regarding your earlier request to use repo branches for PRs for easier edits, I don't think that's necessary with the default "Allow edits by maintainers" settings.
Try adding a newline on line 12 .travis.yml for this PR.
Edit: Give this a shot with a file on the next PR. I'm going to merge this now to check if caching works.

Testing this with #42's PR.
The make buildAll step is now 4 seconds. It was originally 26 seconds without caching and the logs were pretty noisy.
Overall build time reduced from 1:18 to 0:58. Not a major gain yet, but it will likely become more significant as the number of recipes and packages increases.

Yeah, that gain will become important as more recipes are added to this repo.
If we ever have CI problems in the future with code that appears to be fine, let's remember to delete the cache and see whether that fixes the problem.


[TF2.0.0rc0] TPU update

When will be updated the nightly build for TPUs in TF2.0.0rc0 ?
Thank you

@nsantavas ,
Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Use of startTimeMillis for findTraces API , when es-aliases are used

Requirement - what kind of business use case are you trying to solve?
   Improve the query performance in Jaeger, for Elasticsearch storage, when --es.use-aliases is set to true

Problem - what in Jaeger blocks you from solving the requirement?
  Currently, when we use aliases for Elasticsearch storage, the findTraces API  queries all indices present under jaeger-span-read alias (irrespective of time range we give in UI). Due to this, when the data set is huge, significant amount of time is being used for querying unnecessary shards which are not in the given time range.

Proposal - what do you suggest to solve the problem or improve the existing situation?
The findTraces API use the `startTime` field for querying, which is a `long` field. Elasticsearch in built has a feature, to skip shards before querying, when the query is a range query on `date` field.

https://discuss.elastic.co/t/timeline-query-on-timestamped-indices/129328/2 is the related discussion for the same.

  Hence modifying the findTraces API , to use `startTimeMillis` field(which is already present in the data we store) which is a `date` type field, can help in skipping unnecessary shards hence improving the query performance

Any open questions to address

@sivatarunp that's an interesting find; did you get a chance to test this out to see how much of an improvement this change would make to the FindTraces query?

@albertteoh . We have couple of things here.

Changing the timefield to startTimeMillis.
Adding a time range query with startTimeMillis field in the msearch query used to fetch individual trace data, in the findTraces page.
We observed a huge difference, in query times. It's more predominantly seen, when we have the hot and warm nodes setup. Same query got timed out without the above changes

@sivatarunp are you able to quantify the query time improvement by providing some numbers from your tests?

@albertteoh Here is a panel I could build from grafana community dashboards. The current query took more than 4m and even timed out, where as with above changes results were under 1 min, for the same data set

Thanks for that, @sivatarunp. Your proposal sounds reasonable to me. It's not entirely clear to me why the startTime field to hold the microsecond unix epoch was created, perhaps for higher precision for sorting ES results and internal use. @pavolloffay?
Are you able to provide a contribution from the change you have tested?

Looks like this was fixed already.


Map returned undefined

Description
It says to report an issue, so i'm reporting one :sweat_smile: .
What should I provide?
My current map setup

const polandBounds = {
  east: 26.5,
  west: 11.5,
  north: 56.5,
  south: 47,
};

const handleSetBounds = (): void => {
  if (!map) return;

  map.fitBounds(getMapBounds(window.google.maps, places));

  if (places.length < 2 && places.length > 0) {
    map.fitBounds(getMapBounds(window.google.maps, places));
    map.setZoom(14);
  } else if (places.length === 0) {
    map.setZoom(0);
    map.fitBounds(polandBounds);
  }
};

useEffect(() => {
  if (!map) return;
  handleSetBounds();

  // eslint-disable-next-line react-hooks/exhaustive-deps
}, [map]);

<Map
  mapId={XXX}
  style={{
    width: '100%',
    height: 600,
  }}
  fullscreenControl={false}
  streetViewControl={false}
  mapTypeControl={false}
  clickableIcons={false}
  restriction={{
    latLngBounds: polandBounds,
    strictBounds: true,
  }}
  defaultBounds={polandBounds}
  onProjectionChanged={() => {
    if (!map) return;
    handleSetBounds();
  }}
>

Steps to Reproduce
none rn
Environment

Library version<EMAIL_ADDRESS>"^0.10.0",
Google maps version: weekly
Browser and Version: 124.0.6367.78
OS: Linux

Logs
No response

Hmm, apparently it's trying to dispatch the first map-events before the map is ready. See here: https://github.com/visgl/react-google-maps/blob/904b918427da071477ed4bb8c2c65006b35dff88/src/components/map/use-map-events.ts#L103-L109
That might have something to do with using defaultBounds instead of center/zoom. Can you try if it works when you specify defaultCenter/defaultZoom?

Hmm, apparently it's trying to dispatch the first map-events before the map is ready. See here:
https://github.com/visgl/react-google-maps/blob/904b918427da071477ed4bb8c2c65006b35dff88/src/components/map/use-map-events.ts#L103-L109
That might have something to do with using defaultBounds instead of center/zoom. Can you try if it works when you specify defaultCenter/defaultZoom?

For defaultCenter  OR defaultZoom only:
index.modern.mjs:689 <Map> component is missing configuration. You have to provide zoom and center (via the `zoom`/`defaultZoom` and `center`/`defaultCenter` props) or specify the region to show using `defaultBounds`

for  defaultCenter and defaultZoom={0} OR defaultCenter and defaultZoom={0} and defaultBounds  I'm NOT getting an error :)
for  defaultCenter and defaultBounds OR defaultZoom and defaultBounds
index.modern.mjs:361 [createEvent] at least one of the values from the map returned undefined. This is not expected to happen. Please report an issue at ...

In summary: when I'm using defaultBounds i should always provide defaultZoom and defaultCenter? :sweat_smile:

that seems to be an error.
It should be either defaultBounds OR ( center/defaultCenter AND zoom/defaultZoom). So when one of center/zoom is specified it also expects the other one, but that isn't actually needed when defaultBounds is also present.
But the error disappearing when using center/zoom verifies that I need to test with just defaultBounds to reproduce this.
I think I will add another event-type for the onProjectionChanged event that doesn't include the camera-parameters and instead only contains the projection object. That should do the trick in this case.

That was easier than thought, I think this should already fix the error you are seeing: #346


use model on the props

expect
h(custom, { props: {model: this.object}})
actually
h(custom, { model: this.object})

vue-tsx-support is not a transpiler.
https://github.com/vuejs/jsx is the right place to file transpilation issue.


sort and sort_by methods of Hash class

There is no sorting method in Hash class. How should we handle this? Or, will this be included in the future?

While Hash is insertion ordered, depending on that is usually a smell IMO. Are you sure you don't want an Array({A, B}) actually?

I would sort a hash by its values like hash.sort_by { |_, v| v.size } but I could not do as your said.

I didn't say that you can sort a Hash, I said that wanting to usually hints that you're using the wrong data structure.

I understood. And, it would be better to use the structure you mentioned above. Thanks for help.

Hash#{sort, sort_by}  are convenience methods for .to_a.sort and might be nice (they exist in ruby-land anyway, but I suppose forcing the call to .to_a might clarify things hmm)

For future reference,
to get a sorted array of pairs:
hash.to_a.sort_by { |key, value| key }

I don't want to revive an old corpse, but are there technical reasons why Hash#sort_by is not in the standard lib ?
@jhass : are there any reading materials I could work with to find alternate data structures to bypass this need ? Because I find myself using @oprypin one liners so many times...

I don't think there's much reading material because it's largely based on opinionated philosophy. So it's definitely not a technical reason.
Crystal often favors a little bit more explicitness in Ruby with the goal of reducing surprises. Here, I think it's easy to argue, that a method on Hash turns it into an Array, rather than changing the internal ordering a Hash has, can be a bit surprising offhand. In Ruby this happens because Hash is Enumerable and sort_by is an Enumerable method and the key thing to understand here is that in terms of Enumerable a Hash is a list of pairs and that's what sort_by operates on. But internalizing this connection, is quite some cognitive overhead to ask for.
So in Crystal so far we preferred to make this conversion a little bit more explicit, and the code a little bit more intention revealing. It literally says there, I want an ordered list of pairs, ordered by the following criteria.
Reiterating my previous point, personally I had very few cases where I ended up wanting to have the features of a map structure, that is constant time lookup of a value by some unique key, and the features of a list structure, that is a defined ordering between some values at the same time. In fact this feature combination is so seldom that many standard libraries don't offer any possibility for it, or at least do not provide it in the standard map time but rather a special one. Anyways, the few cases where I personally encountered this desire turned out to be quite indicative of me not having found the right datastructure for my problem yet, often simply realizing I didn't actually need one of the two features I talked about above and thus making the resulting program easier to reason about by using the other structure. This is what I meant when I said this is a code smell to me, it's an indication to me that I should be revisiting something.
Personally I wouldn't even use the term "one-liner" for the expression above, which I tend to use for common transformations on some data structure that usually have a much higher complexity than a call to to_a :)
So, why does that little to_a bother you so much? :) Maybe your data should never be a Hash in the first place?

Thanks for this extremely elaborate answer :-)

Personally I wouldn't even use the term "one-liner" for the expression above

Haha ! Yes, you're right. Replacing one line of code with one another line, is not a oneliner :-p
Regarding the sort_by issue, IMO it's just synthetic sugar, mental load reduction and a gift to ruby newcomers.
It's not a big issue if this is not in the standard lib, but the cost of adding it is minimal.
I fell in love with Crystal because of its extreme readability. Thanks from borrowing from Ruby syntax and "encouraging" to declare types it makes reading code almost like reading natural language.

I said that wanting to usually hints that you're using the wrong data structure.
So, why does that little to_a bother you so much? :) Maybe your data should never be a Hash in the first place?

Ok, you have my full attention. I'm a junior developer, and using the key => value structure of Hash seems so obvious and useful to me for many tasks.
For example attributing a score value to an Array or Set of values then picking the 3 most scored ones, what would be the most clean/efficient way to do this without using Hash and a sort_by like method ?
This is an old but recurrent example of using Hash for this problem

I think my concern here is that .to_a.sort_by introduces a conversion to an Array of Tuples first, then calls sort_by on that intermediate Array.  A more direct sort_by method wouldn't need that intermediate Array and would instead work with .each directly.  I'd have to experiment to see if that is an actual performance problem, but it does feel like, at a minimum, you'd have an extra copy of the collection in memory at once.
I think the confusion arises because sort_by in Crystal is only defined on Array (and Slice?), but in Ruby it's defined on Enumerable.  Was there a specific reason that sort_by was not defined on Enumerable in Crystal?

@Fryguy I just gave what I think the reason is above. Also I don't see the extra copy, sort algorithms need an array to operate, so when you chain sort_by! onto to_a, there's no extra. Hash#sort_bys implementation would very likely literally end up as to_a.sort_by!.
@rmarronnier

For example attributing a score value to an Array or Set of values then picking the 3 most scored ones, what would be the most clean/efficient way to do this without using Hash and a sort_by like method ?

It's generally hard to answer this in the abstract because it just depends so much on what you're doing. So take the following as responses to the example rather than me saying it's like this in general and there's no single exception :)
So, in the example
https://github.com/cadmiumcr/language_detector/blob/4020ac783e5962ff7910c17fe34d5c6d081b94cd/src/cadmium/language_detector/language_detector.cr#L34-L39
Here the return value could stay as Array({String, Int32}), the only usage of this is passing it to get_distances which just iterates it:
https://github.com/cadmiumcr/language_detector/blob/4020ac783e5962ff7910c17fe34d5c6d081b94cd/src/cadmium/language_detector/language_detector.cr#L84-L88
get_distances too has no reason to convert it back into a hash, its result is soley passed to
https://github.com/cadmiumcr/language_detector/blob/4020ac783e5962ff7910c17fe34d5c6d081b94cd/src/cadmium/language_detector/language_detector.cr#L56-L62
which does not make any use of any hash features (min = !distances.values[0..].empty? ? distances.values[0] : 1 -> min = distances.dig(0, 1) || 1).  The pattern continues, the only usage of the result of normalize is
https://github.com/cadmiumcr/language_detector/blob/4020ac783e5962ff7910c17fe34d5c6d081b94cd/src/cadmium/language_detector/language_detector.cr#L43
which could just be .dig(0, 0) or [0][0].
In summary there's a lot of unneeded hashes in this example, getting rid of them would in my opinion make the program simpler (No question of "Why's this a hash?") and objectively faster due to less allocations.
If you want to regain some readability after, it probably makes sense to turn the pairs into a small struct like
record Score language : String, distance : Int32

That should have the same performance as a Tuple and suddenly you know what all these "keys" and "values" actually represent!

Thanks @jhass for this fantastic and extensive answer !
I'll use #dig with Array(Tuple) now :-)
Also thanks for pointing out the record class : it's better than aliases or less overkill than a Struct.

No question of "Why's this a hash?"
Now I realize, the reason I used hashes was because it works (and looks some_hash[key]) the same as a js object and the json output would be easier to work with on the frontend side of my web work.

json output would be easier to work with on the frontend side of my web work.

That's a totally legit reason to convert it into a hash, I would just delay it as long as there's no benefit, in other words separate the presentation logic from the business logic :) Just keep in mind the JSON spec does not explicitly say that the order of keys in an JSON object is of semantic value, so that happening in any JSON encoder or decoder would be an implementation detail and something I wouldn't rely on when designing an JSON API :)

It might be interesting to create a "ruby compatibility" shard...or maybe one already exists?

Going to bump this once again as i find myself wanting it every now and then. The main use cases I run into are using a hash to store types keyed by their "name".
For example Routes within a RouteCollection. The collection also has the concept of priorities which can be specified when a route is added. After all the routes are added and you want to iterate over them, they should be iterated in order based on their priority/insert order. Currently I'm needing to convert the hash to an array of tuples, do the sort, clear the hash, then re-insert everything back into it. I'm of course keeping track when the collection is in a sorted state and only need to this once (or when new routes are added), but still feels like there should be an easier way...
I'm also open to alternative implementations, but a hash seems to make the most sense so that the routes can be accessed by name versus needing to use like Array#find.

So maybe you're hoping for something like a TreeMap in java or something
like that?
On Tue, Nov 16, 2021 at 6:50 PM George Dietrich @.***>
wrote:

Going to bump this once again as i find myself wanting it every now and
then. The main use cases I run into are using a hash to store types keyed
by their "name".
For example Routes within a RouteCollection. The collection also has the
concept of priorities which can be specified when a route is added. After
all the routes are added and you want to iterate over them, they should be
iterated in order based on their priority/insert order. Currently I'm
needing to convert the hash to an array of tuples, do the sort, clear the
hash, then re-insert everything back into it. I'm of course keeping track
when the collection is in a sorted state and only need to this once (or
when new routes are added), but still feels like there should be an easier
way...
I'm also open to alternative implementations, but a hash seems to make the
most sense so that the routes can be accessed by name versus needing to use
like Array#find.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/crystal-lang/crystal/issues/1915#issuecomment-971071224,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAADBUESEHODCC24UFXS6RTUMMC5LANCNFSM4BVK6F5Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Also I'm not sure on the implementation in Crystal, but in many hashing implementations (in e.g. other languages), it's a requirement that keys can be sorted/compared. set/map in the C++ stdlib, for example. I am using timestamps myself. Whilst it may indicate that I would be better using an array, in this case there is a definite order to my data. I tag values to the timestamp key, and when I want to output them, doing so in chronological order is what I need. Perhaps rather than sorting, an ordered "visitor" would be better - i.e. an equivalent of each or map that guarantees to yield in order of the key.

Do other languages have hashes that allow a "sort by" key for duplicate
entries?  In that case I guess they wouldn't return values by insertion
order but by this "other" order?
On Mon, Jul 4, 2022 at 6:46 PM stellarpower @.***>
wrote:

Also I'm not sure on the implementation in Crystal, but in many hashing
implementations (in e.g. other languages), it's a requirement that keys can
be sorted/compared. set/map in the C++ stdlib, for example. I am using
timestamps myself. Whilst it may indicate that I would be better using an
array, in this case there is a definite order to my data. I tag values to
the timestamp key, and when I want to output them, doing so in
chronological order is what I need. Perhaps rather than sorting, an ordered
"visitor" would be better - i.e. an equivalent of each or map that
guarantees to yield in order of the key.
—
Reply to this email directly, view it on GitHub
https://github.com/crystal-lang/crystal/issues/1915#issuecomment-1174481214,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAADBUCRKNN57MHHBDHSIYLVSOAW5ANCNFSM4BVK6F5Q
.
You are receiving this because you commented.Message ID:
@.***>

@stellarpower I think this would be a use case for a different data structure with intrinsic ordering. Implementation of such a sorted map type would be more efficient based on a tree instead of a hash table.
To summarize, there are many different concepts of semantics for map types. And there are many possible implementations with specific tradeoffs for different use case.


No tests discovered

Description
I converted a project to the new .csproj format recently as a part of an effort to port to .NET Core, and after I did that and got everything building, the vstest runner stopped being able to find tests in the project.  The strangest thing to me is that I can run individual tests in VS2017, but when I try to run with "Run all tests", the runner fails to find any.
The Xunit runner does work, however.  When I run dotnet xunit, it finds all the tests and I get the expected behavior.
Steps to reproduce
I'm not entirely sure what I did to get in this state, but the project is accessible here:
https://github.com/aggieben/dbup/tree/release/4.0.0
Access is open.  Please git clone https://github.com/aggieben/dbup; cd dbup; git checkout release/4.0.0 to get the project(s) in question.
Expected behavior
VS2017 test runner should show discovered tests in test explorer, and when "run all tests" is selected, it should run the tests.  Similarly, dotnet test should execute all the tests for both targets (netcoreapp1.0 and net46).
As mentioned above, the Xunit runner does produce the expected results.
Actual behavior
When running with dotnet test, this is the output:
C:\Users\ben\proj\oss\DbUp\src\dbup-tests [release/4.0.0 ≡]
λ dotnet test
Build started, please wait...
Engine\SqlScript.cs(15,33): warning CS0169: The field 'SqlScript.name' is never used [C:\Users\ben\proj\oss\DbUp\src\dbup-core\dbup-core.csproj]
Engine\SqlScript.cs(15,33): warning CS0169: The field 'SqlScript.name' is never used [C:\Users\ben\proj\oss\DbUp\src\dbup-core\dbup-core.csproj]
Helpers\NullJournalTests.cs(21,33): warning CS0618: 'SqlServerExtensions.SqlDatabase(SupportedDatabases, Func<IDbConnection>, string)' is obsolete: 'Pass connection string instead, then use .WithTransaction() and .WithTransactionPerScript() to manage connection behaviour' [C:\Users\ben\proj\oss\DbUp\src\dbup-tests\dbup-tests.csproj]
DeployChangesBuilderTests.cs(21,33): warning CS0618: 'SqlServerExtensions.SqlDatabase(SupportedDatabases, Func<IDbConnection>, string)' is obsolete: 'Pass connection string instead, then use .WithTransaction() and .WithTransactionPerScript() to manage connection behaviour' [C:\Users\ben\proj\oss\DbUp\src\dbup-tests\dbup-tests.csproj]
DeployChangesBuilderTests.cs(41,33): warning CS0618: 'SqlServerExtensions.SqlDatabase(SupportedDatabases, Func<IDbConnection>)' is obsolete: 'Pass connection string instead, then use .WithTransaction() and .WithTransactionPerScript() to manage connection behaviour' [C:\Users\ben\proj\oss\DbUp\src\dbup-tests\dbup-tests.csproj]
DeployChangesBuilderTests.cs(61,33): warning CS0618: 'SqlServerExtensions.SqlDatabase(SupportedDatabases, Func<IDbConnection>)' is obsolete: 'Pass connection string instead, then use .WithTransaction() and .WithTransactionPerScript() to manage connection behaviour' [C:\Users\ben\proj\oss\DbUp\src\dbup-tests\dbup-tests.csproj]
DeployChangesBuilderTests.cs(84,37): warning CS0618: 'SqlServerExtensions.SqlDatabase(SupportedDatabases, Func<IDbConnection>)' is obsolete: 'Pass connection string instead, then use .WithTransaction() and .WithTransactionPerScript() to manage connection behaviour' [C:\Users\ben\proj\oss\DbUp\src\dbup-tests\dbup-tests.csproj]
Build completed.

Test run for C:\Users\ben\proj\oss\DbUp\src\dbup-tests\bin\x64\Debug\netcoreapp1.0\dbup-tests.dll(.NETCoreApp,Version=v1.0)
Microsoft (R) Test Execution Command Line Tool Version <IP_ADDRESS>
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
No test is available in C:\Users\ben\proj\oss\DbUp\src\dbup-tests\bin\x64\Debug\netcoreapp1.0\dbup-tests.dll. Make sure that installed test discoverers & executors, platform & framework version settings are appropriate and try again.

Test run for C:\Users\ben\proj\oss\DbUp\src\dbup-tests\bin\x64\Debug\net46\dbup-tests.dll(.NETFramework,Version=v4.6)
Microsoft (R) Test Execution Command Line Tool Version <IP_ADDRESS>
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
[xUnit.net 00:00:00.9516916]   Discovering: dbup-tests
[xUnit.net 00:00:01.2040971]   Discovered:  dbup-tests
[xUnit.net 00:00:01.2348628] dbup-tests: Catastrophic failure: System.ArgumentException: An item with the same key has already been added.
   at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Xunit.Runner.VisualStudio.TestAdapter.VsTestRunner.RunTestsInAssembly(IRunContext runContext, IFrameworkHandle frameworkHandle, LoggerHelper logger, TestPlatformContext testPlatformContext, IMessageSinkWithTypes reporterMessageHandler, AssemblyRunInfo runInfo)
No test is available in C:\Users\ben\proj\oss\DbUp\src\dbup-tests\bin\x64\Debug\net46\dbup-tests.dll. Make sure that installed test discoverers & executors, platform & framework version settings are appropriate and try again.

Environment
C:\Users\ben\proj\oss\DbUp\src\dbup-tests [release/4.0.0 ≡]
λ dotnet --info
.NET Command Line Tools (1.0.3)

Product Information:
 Version:            1.0.3
 Commit SHA-1 hash:  37224c9917

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.15063
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\1.0.3

Test runner-related packages from the .csproj:
<ItemGroup>
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.0.0-preview-20170106-08" />
    <PackageReference Include="xunit" Version="2.3.0-beta1-build3642" />
    <PackageReference Include="xunit.runner.visualstudio" Version="2.3.0-beta1-build1309" />
    <PackageReference Include="Shouldly" Version="2.8.1" />
    <PackageReference Include="TestStack.BDDfy" Version="4.3.1" />
    <!--<PackageReference Include="xunit.core" Version="2.2.0-beta2-build3300" />-->
    <DotNetCliToolReference Include="dotnet-xunit" Version="2.3.0-beta1-build3642"/>
  </ItemGroup>

This looks likely to be a bug in xUnit.net, not VSTest. There is a dictionary in our VS adapter which maps unique IDs to test cases that is probably being tripped up here. The question will be to figure out how you ended up with two test cases with the same unique ID,

I see that from the output of the runner against the net46 target output, but do you think that's the case for the netcoreapp1.0 target as well?

Until I can poke around inside and see what's going on, I don't know. It could be that VSTest is handling the two scenarios differently (hiding the bug in one, exposing it in the other, but not running any tests in either case).

@aggieben Please use latest version of package Microsoft.NET.Test.Sdk which is 15.3.0-preview-20170502-03
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.3.0-preview-20170502-03" />
With new package you will see that error message which adapter is sending is coming to console.
with latest version you will get following output:
PS C:\Enlistment\Customer\dbup\src\dbup-tests> dotnet test
Build started, please wait...
Build completed.

Test run for C:\Enlistment\Customer\dbup\src\dbup-tests\bin\Debug\netcoreapp1.0\dbup-tests.dll(.NETCoreApp,Version=v1.0)
Microsoft (R) Test Execution Command Line Tool Version 15.3.0-preview-20170502-03
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
[xUnit.net 00:00:01.7501199]   Discovering: dbup-tests
[xUnit.net 00:00:02.1956541]   Discovered:  dbup-tests
[xUnit.net 00:00:02.3184782] dbup-tests: Catastrophic failure: System.ArgumentException: An item with the same key has already been added. Key: 9b4bc40fde591750f3097c5f4b22a610ad581cda
   at System.ThrowHelper.ThrowAddingDuplicateWithKeyArgumentException(Object key)
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Xunit.Runner.VisualStudio.TestAdapter.VsTestRunner.RunTestsInAssembly(IRunContext runContext, IFrameworkHandle frameworkHandle, LoggerHelper logger, TestPlatformContext testPlatformContext, IMessageSinkWithTypes reporterMessageHandler, AssemblyRunInfo runInfo)
No test is available in C:\Enlistment\Customer\dbup\src\dbup-tests\bin\Debug\netcoreapp1.0\dbup-tests.dll. Make sure that test discoverer & executors are registered and platform & framework version settings are appropriate and try again.

Test run for C:\Enlistment\Customer\dbup\src\dbup-tests\bin\Debug\net46\dbup-tests.dll(.NETFramework,Version=v4.6)
Microsoft (R) Test Execution Command Line Tool Version 15.3.0-preview-20170502-03
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
[xUnit.net 00:00:00.9311691]   Discovering: dbup-tests
[xUnit.net 00:00:01.2564212]   Discovered:  dbup-tests
[xUnit.net 00:00:01.3457598] dbup-tests: Catastrophic failure: System.ArgumentException: An item with the same key has already been added.
   at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at System.Collections.Generic.Dictionary`2.Add(TKey key, TValue value)
   at Xunit.Runner.VisualStudio.TestAdapter.VsTestRunner.RunTestsInAssembly(IRunContext runContext, IFrameworkHandle frameworkHandle, LoggerHelper logger, TestPlatformContext testPlatformContext, IMessageSinkWithTypes reporterMessageHandler, AssemblyRunInfo runInfo)
No test is available in C:\Enlistment\Customer\dbup\src\dbup-tests\bin\Debug\net46\dbup-tests.dll. Make sure that test discoverer & executors are registered and platform & framework version settings are appropriate and try again.

Only run all scenario ( From both commandline and visual studio) is busted here.
I am closing this bug as this is a bug in xunit adapter and @bradwilson is tracking it in xunit repo.

I am running into this exact issue except I am using the latest versions of everything. Is this still an outstanding issue with xunit or was it determined to be vstest?

@MarkHerhold Issue was determined as xunit adapter's.  Can you share repro project, logs(https://github.com/Microsoft/vstest-docs/blob/master/docs/diagnose.md) and explicitly provide the versions of (dotnet --info, VS, xunit and Microsoft.NET.Test.Sdk)?

@smadala The strange thing is that the tests work just fine when run from VS Community Edition. Luckily, the project is open source and brand new, so that should make things simple.

Locally, I'm running VS Community Edition 2017 v14.4.5 (new install) with .NET Framework v4.7.02556 (came with new VS install).
xunit v2.3.1 defined here
Microsoft.NET.Test.Sdk v15.5.0 defined here

dotnet --info (local)
PS D:\badgeup-dotnet-core-client> dotnet --info
.NET Command Line Tools (2.0.3)

Product Information:
 Version:            2.0.3
 Commit SHA-1 hash:  12f0c7efcc

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.16299
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.0.3\

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.3
  Build    : a9190d4a75f4a982ae4b4fa8d1a24526566c69df

I also have the project running in AppVeyor with the same failures. Here's a link to the repo.

@MarkHerhold I couldn't repro it. See the below image. Share all log.*.txt files.

@smadala I see one thing you did differently is you CC'd to the Test project directory. Could you try running the tests at the directory above that?

Running tests for sln, not yet supported from vstest(https://github.com/Microsoft/vstest/issues/1129). I tried running from above directory. See the below image. Did you face issue with dotnet test \path\to\BadgeUpClient.Tests.csproj, If so share log.txt and log.host.*.txt too.

@smadala Ah, that's the issue! Thanks so much for your help! 😀


connection to retry find_servers after failure after redirect

Change connection retry logic to find_servers after a failure after a redirect.

Will add new branch and pull request.


Ionic info doesn't report global package cordova

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[X] 3.x
I'm submitting a ...  (check one with "x")
[X] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Ionic info doesn't report global package cordova
Expected behavior:
Expect to see cordova version
Steps to reproduce:
running ionic info in a ionic project directory, but it doesn't report global packages.
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (/usr/local/lib/node_modules)

    @ionic/cli-utils  : 1.9.2
    ionic (Ionic CLI) : 3.9.2

local packages:

    @ionic/app-scripts : 2.1.4
    Ionic Framework    : ionic-angular 3.6.0

System:

    Node : v7.3.0
    npm  : 4.2.0
    OS   : macOS Sierra

Hello! Thank you for opening an issue with us!
Have in installed any platforms in your project?
If I just do an ionic start myApp blank, let it run, cd myApp and then do ionic info, I get basically what you get.
But if I then do ionic cordova platform add ios I will then get way more info. Like this:
cli packages: (/usr/local/lib/node_modules)

    @ionic/cli-utils  : 1.9.2
    ionic (Ionic CLI) : 3.9.2

global packages:

    Cordova CLI : 7.0.1

local packages:

    @ionic/app-scripts : 2.1.3
    Cordova Platforms  : ios 4.4.0
    Ionic Framework    : ionic-angular 3.6.0

System:

    ios-deploy : 1.9.1
    ios-sim    : 6.0.0
    Node       : v8.3.0
    npm        : 5.3.0
    OS         : macOS Sierra
    Xcode      : Xcode 8.3.3 Build version 8E3004b

Based on the output you have, though, I would say you haven't actually added any platforms yet.
Thank you for using Ionic

The platforms were added before I updated ionic. I will re-add the platforms and see.
it seems that moving from cordova 6.5 and ionic 3.4 to ionic 3.6, there seems to be change in command such ionic prepare has become ionic cordova prepare.
Where can I review all the changes in commands?

I usually just do ionic --help from the command line. It is terse but very useful, IMO. If I need more info, I then hit the docs.
There is also Change Log Entry that may be useful for this purpose.
Hope that helps. Let me know how things go after you re-add the platforms. If you still don't have the right info from ionic info I will move this over to the ionic-cli repo.

I can see global packages after I re-add the platforms. thanks


add enumerate_possibilities cpu function

Overview
Addressing issue (#2). Adds a function given a mesa and hand which finds all possible ways to add to 15 using exactly one card from the hand and any cards from the mesa.
Checklist

[x] Have you run the test suite? (Run python -m unittest discover in the project's root directory)
[x] Have you run pylint and fixed any linting errors (Run pylint quince in the project's root directory)

quince/cpu.py does not have tests written for it in test/test_components.py, so it was skipped. I've included a basic docstring test which can be added to the test file.

Sorry for taking so long to get back to you! I'll have a look at this over the next couple of days!


phase out libpysal geometries

Moving forward we need to consider phasing out all functionality that requires (and even uses) native PySAL geometries to improve efficiency and interoperability within the ecosystem. These were necessary in the past, but are now a bottleneck for performance.

I am all in favour of this. I would even say we should deprecate libpysal.cg across the ecosystem.
Do you have an idea of a timeline for this? It may make sense to schedule a call and discuss overlaps with momepy. Since we'll be likely refactoring both in some way, it makes sense to do it once and in a super compatible way.

I started updating the Refactor spaghetti document again. Take a look and let me know how you're thinking about it.
Maybe we can also loop in @tomalrussell and @HTenkanen? Anyone else you can think of?

Hi @jGaboardi - I'd be happy to be looped in to discussions, though I'm not familiar with spaghetti internals.
The (gdf_nodes, gdf_edges) tuple or small wrapper object seems like it's the de-facto standard for spatial networks with explicit geometries (and optional attributes) for nodes and edges. You suggest it in your note, snkit.network.Network is a minimal wrapper, and pyrosm.OSM.get_network returns a tuple of gdfs.
I'm not sure if this is in scope for this issue, but there are various adaptors to graph/network representations:

momepy handles gdf<>nx conversion, including making a graph directly from LineStrings
osmnx works with networkx.MultiDiGraph and has helpers to/from gdfs osmnx.utils_graph.graph_from_gdfs
pandana has pandana.network.Network without explicit edge geometries or other node/edge attributes, but accepts node_x, node_y, edge_from, edge_to as pandas.Series and edge_weights as pandas.DataFrame.
pyrosm has adaptors to igraph, networkx, pandana,  in pyrosm.pyrosm.OSM.to_graph

@tomalrussell awesome!

... though I'm not familiar with spaghetti internals

The spaghetti internals are based on the archaic pysal.network. A complete is what we have been tiptoeing around for more than a year now.
A top priority for me (and other PySAL devs) is integration with current (segregation-@knaaptime) and proposed (momepy-@martinfleis) PySAL submodules that would benefit from a unified network representation, which spaghetti could built off of.

The (gdf_nodes, gdf_edges) tuple or small wrapper object seems like it's the de-facto standard for spatial networks with explicit geometries (and optional attributes) for nodes and edges

I don't necessarily agree with this statement. Probably the most common is the OSMnx graph, which is a networkx.MultiGraph with shapely geometries attached to individual nodes and edges in the graph object. momepy is using exactly the same model, just a bit more relaxed. Both can return (gdf_nodes, gdf_edges) but only because that is the most natural way how to export Graph to GeoDataFrame(s). None of them uses it to represent the Graph itself and do the computation. To my knowledge, pyrosm does the same thing in osm.to_graph(nodes, edges, graph_type="networkx").
When creating a representation of a spatial graph, we are doing so because we want to exploit graph theory in our analysis. I think that to maximise the potential, we should have GeoDataFrames on one side of the equation and a pure Graph object on the other. Trying to combine both, finding a middle ground where you have Graph but also vectorized GeometryArray would lead to unforeseen limitations I am afraid.
We need to make sure that we can efficiently convert from GeoDataFrames to networkx, igraph or cuGraph and back and leverage the power of geopandas on one side and the power of graph libraries on the other. Creating yet another representation that would lead to the sequence GeoDataFrame <conversion> spatial graph <conversion> proper graph isn't a direction I'd like to explore.
The main conceptual issue I can see is the different logic of DataFrame and Graph objects. While the former is designed to be an array of objects and the most efficient ways of using it are vectorized array-based operations, Graph is different and in principle atomized to individual nodes and edges. That is why merging both to a single spatial graph object needs to inherently limit either geometry-based operations or graph-based operations. And I don't think we want to embed such a limitation into the very core design of it.

Both can return (gdf_nodes, gdf_edges) but only because that is the most natural way how to export Graph to GeoDataFrame(s). None of them uses it to represent the Graph itself and do the computation.

Yes, I guess I was thinking more about exchange than graph computation - dataframes work well for the edge-list representation of a graph, which networkx, igraph, cuGraph and others can accept or produce.
I think it's reasonable to imagine an object that holds a reference to both a graph (maybe an igraph or a networkx graph) and a spatial index (maybe a pygeos STRTree or a scipy kdtree) over the graph geometries. Pandana does something like this with its Network with a kdtree and delegating down to cython/C++ for the graph implementation in cyaccess.
On the other hand, OSMnx builds indexes on demand in osmnx.distance. NetworkX does similarly internally, like building the graph adjacency matrix when an algorithm needs to do some linear algebra.
The core trade-off might be around deciding what internal data structures to maintain, and what to build only on demand - that's probably driven by what operations the library needs to support and how it's used. Is this list any use as a start? -

geometry manipulation (precision, CRS, simplification, merging)
geometry queries/predicates (nearest, intersects, contains...)
graph traversal (anything where you visit neighbours - BFS, DFS, Dijkstra, components...)
creation, conversion (from/to objects, files)
mutability (add/remove nodes and edges, change attributes or geometries)

I'm still a bit unsure about scope here (trying to add useful things to the conversation without having much background 🙂 ) - for example, the refactor could be more incremental if it was at first limited to replacing use of pysal.cg objects with shapely geometries, then perhaps reworking parts of spaghetti.Network to lean on networkx.MultiDiGraph or similar.

trying to add useful things to the conversation without having much background

That is why I wanted to have a call. I also don't know much about spaghetti, I've never used it in production (sorry @jGaboardi  😄).

That is why I wanted to have a call

Yeah, I have been super busy with work, etc. I will try to set something up soon-ish. I really want to keep the ball rolling on this and not let it stop again.

As a first step to this I am thinking that simply printing out a deprecation warning on initialization that basically states libpysal geometries will no longer be accepted as inputs starting in v2.0.0. Next, I'll get to work on refactoring all the guts that rely on libpysal geometry objects, which is almost everything. Once this is all worked out (which will take some time) I will release a v2.0.0 and continue with maintenance while a schema for total refactoring is being decided on. This total refactoring will be released as v3.0.0.
@martinfleis As an overall strategy (especially the first step), would you agree this an OK plan? If not, any advice on how else to proceed?

@jGaboardi That sounds good to me.

Next, I'll get to work on refactoring all the guts that rely on libpysal geometry objects

On that we should have a chat before you start drafting it.


Create Content creator

📝 Description
Add a brief description.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

Dear @harshita1611 , thank you very much for your contribution. Please attach the link of your AI Agent in your file and reply to this PR. Looking forward to your update.

I just discovered an impressive AI Agent called Content Creator. Check it: https://illa.ai/ai-agent/ILAfx4p1C7vW/detail


This connector no longer works with the lastest version of Airbyte

Hi!
I was setting up a new machine to host airbyte for the purpose of syncing Pscale -> Big Query. I have another machine doing the same thing that I spun up in December and it worked great. However, it looks like the latest version of Airbyte is no longer compatible with the code in this repo.
Two things to note:

The instructions for setting up a self hosted Airbyte instance is also out of dock. You will get an error for a file that does not exist. I manually added that file to get past the composer up error I was getting.

After setting up this repo as the src for a customer Airbyte Source, it will fail and give you this splash screen:

It looks like it could be related to this change
Would love to help contribute on a solution, but don't have the time to dig in this week. Wanted to take the time to make you guys aware because I love Pscale and really want this to work!
Thanks

Thanks for reporting this @cwidafor, I'm looking into it now.


chore: Update the javadoc plugin to 3.2.0

Fixes #139

@stephaniewang526 Can you add this to your release of 0.6.0 ?
@chingor13 FYI
@BenWhitehead FYI

@tbpg Take a look.


External link in a picture

I have create a table on the front page with 4 logos (images) and I am tryng to add a link, so people can click on them. However unsuccessfully.

The pictures look ok on the website with that code, but is there any way to introduce a link on them?

I tried with the following and several combinations, but does not work:
[http://www.epsanet.org](/images/epsa.png)
![http://www.epsanet.org](/images/epsa.png)

I do not use this repo, however does the following work?
[![title](picture.png)](http://link.com/)
Appears to work in these comments. Try clicking the picture below:

Yep! it works. Many thanks for your help :)

I do not use this repo, however does the following work?
[![title](picture.png)](http://link.com/)
Appears to work in these comments. Try clicking the picture below:

Hi, I wonder is there a way to change the size and position of figures on the webpage through coding in the markdown file, like blog_post? Thanks!


MOB-1367] - Do not use IterableAPI.xxx static methods in InAppTests.

Hmm, I'm getting an error in testAutoShowInAppMultipleOverride(), saying Thread 1: Exception: "API violation - multiple calls made to -[XCTestExpectation fulfill] for track in app open."

The error message is misleading. Had to change some other tests.


Make initializer for UnboxError public

This PR makes the initializer for UnboxError public so that consumers can create their own custom UnboxError if desired.
This is a quick fix solution to #136, which I imagine will actually suffice for many/most consumers.

Thoughts on this?

Hey @JRG-Developer, sorry for taking a long time to review this. I actually fail to see how this solves the issue at hand, since that issue is about not being able to get the specific error that occured anymore (because the error API was heavily simplified in Unbox 2.2) - at least that's how I understand it. Not sure I see any usecases where you'd want to manually create an UnboxError instance, but feel free to correct me if I'm wrong 😄

Yeah we actually encountered this recently. It was really nice previously where we could just say throw UnboxError.invalidValue(someValue, "jsonKey") in an initialiser. Now all these calls had to be replaced with our own custom errors which just felt a little overkill.
It would be nice if we could just piggyback off the great set of Unbox errors in these instances were unboxing failed with trivial errors already supported privately by Unbox

@JohnSundell
The issue is two fold:

Being able to determine the specific error type... this isn't solved by this PR.

However, many apps (ours included) may not care about the type, just that something went wrong during unboxing... this leads to point (2).

Being able to throw a custom "unboxing failed" error. As @danielgalasko mentioned, it was previously nice to be able to throw a custom UnboxError to indicate "something went wrong during unboxing."

While this could be done by throwing a different, custom Error, this would lead to checking for yet another type of Error, resulting in more complex code.
It's simpler to just allow consumers to create their own custom UnboxError, and AFAICT, there isn't really any downside to doing such. 😄

@danielgalasko Yeah, I'm planning to bring back the previously public specific errors (that are now called UnboxPathError). My intention was to make the API simpler, but it backfired a bit 😅

@JRG-Developer Alright, I see your point. Makes sense, let's merge this! 👍


Software Restriction Policies (SRP) alternative for normal users

As SRP is no longer supported since 1803 i wonder what we normal users can use if it would removed someday - hopefully not.
As AppLocker or Windows Defender Application Control isn't a alternative for normal, non-company users, nor a option for home / pro users

@beerisgood. just now I seen the software restriction policy is available in 1909. Under security settings in GPO

@RAJU2529 sure. For now. And i'm use it without problems.
But as i say as you can read in my posted link, SRP is abandoned.
Of course it works but it wouldn't get any support/ updates anymore. I also wouldn't surprised if it got removed, even if i don't whish that!

@beerisgood. Thanks for your reply.  You seeking any another way to block the software with new blocker.
Last week I came to know Aaron blocker is used for alternative to app blocker

@RAJU2529 any info about that? Can't find something "Aaron blocker"

@beerisgood. Sorry I mentioned the wrong name, actual name is "AaronLocker"

@RAJU2529
AaronLocker works on all supported versions of Windows that can provide AppLocker.
AppLocker is included with enterprise-level editions of Windows.
So sadly no solution for this issue here as it's for normal users, which doesn't have a enterprise Windows.

@beerisgood.  Currently, i am using Enterprise edition of windows 10 1909 64bit.

@RAJU2529 that's fine but no option for us normal end users which can't get a enterprise edition

@beerisgood. I will look in College desktop computers running in windows Pro edition

Thank you for the feedback and discussion. It does not look like we have any documentation issues here, so I'm going to close this one out with kind regards.

@denisebmsft Please re-open. Issue is not fixed.
@RAJU2529 Any updates?

@beerisgood : When you say "Issue is not fixed.", which page do we need to update and correct to solve this documentation issue?

@beerisgood : When you say "Issue is not fixed.", which page do we need to update and correct to solve this documentation issue?

Please read my first post
This issue isn't about changing docs, it's just a question.
The current state is that Microsoft restrict Enduser (Home + Pro edition) security features like SRP (this issue) but only provide newer/ alternatives for higher editions, like Enterprise.
It's good and important that SRP is still present and i realy hope that don't change unless a real alternative is possible for home user.
So i was just looking here if any other methods exist, but doesn't looks like.
If you know any @illfated or you, @RAJU2529 please post.
But if no other solutions exist, i agree with close this issue.
Thanks for reading

Thank you for pointing out the details I had missed. I am fairly sure you need to look for answers in the community forums.

@beerisgood . I came to know the software naame  "Appsamvid_ from C-DAC  india, produced this software, which used for  whitelisting the  available  .exe  files and  java files
Already installed executable  files can be whitelisted  for  executions  on windows baaed pc.
You can try above  software .
For more information visit official site
https://cdac.in/index.aspx?id=cs_eps_appsamvid

@beerisgood . I came to know the software naame "Appsamvid_ from C-DAC india, produced this software, which used for whitelisting the available .exe files and java files
Already installed executable files can be whitelisted for executions on windows baaed pc.
You can try above software .
For more information visit official site
https://cdac.in/index.aspx?id=cs_eps_appsamvid

Sadly a third-party solution and even not downloadable over HTTPS.
Thanks anyway

@beerisgood . download in android mobile , then copy to pc , then, install

@beerisgood . download in android mobile , then copy to pc , then, install

What? Why should i do that?
Also that doesn't change the fact that this program isn't available over HTTPS. Only the website is.
The download go over unencrypted HTTP site which is highly not recommend for security!

@beerisgood .That software is developed by the team of cyber security of officers from the government of india.
If you having doubt , then dont download

@RAJU2529 : it is recommended to avoid referencing non-Microsoft sources, even on these issue ticket pages.
The only acceptable non-Microsoft material and links is direct partners of Microsoft.
Non-Microsoft material and discussions thereof belongs to forum pages, not these issue tickets.

@illfated . Ok fine .I will take care of that


Add LFO-style routing system to envelopes

We can route LFOs to all sorts of different things. It would be nice if we could do the same for envelopes, for example to make a gradual tremolo effect.
(Also, I accidentally created the issue before I finished the title. Is there some way to remove the title change from the history?)

Cool! I'll take a look, then. I should branch off develop, right?

Yes, thanks for asking. We're using Develop to push to App Store. So, a separate branch off of Develop would be preferable. 👍

Hm, I'm having trouble getting it to build. I installed the dependencies and replaced the bundle identifier, and I'm still getting some errors.

No such module 'ChimpKit'
the [CP] Embed Pods Frameworks PhaseScriptExecution command is failing
Embedded binary is not signed with the same certificate as the parent app (seems to be the OneSignalNotificationServiceExtension, may resolve after fixing the next one)
on the OneSignalNotificationServiceExtension, the linker is failing because of framework not found Pods_OneSignalNotificationServiceExtension.

I looked in Pods/ and ChimpKit is there, it just looks like Xcode isn't seeing it.
Also, under the Inter-App Audio capability it says "your account does not have sufficient permissions to modify containers". What does that mean, and do I have to be an administrator?

Never mind, I didn't know I was supposed to use AudioKitSynthOne .xcworkspace instead of .xcodeproj. Perhaps we should add something about that to the readme for first-time CocoaPods users.

I didn't know I was supposed to use AudioKitSynthOne .xcworkspace instead of .xcodeproj. Perhaps we should put something about that in the readme. Also, it turns out I had a problem with my build products path. I fixed both issues and it's building just fine now.

I went to make space in the ENV panel for the routing settings and discovered that the background is an image.
I found https://github.com/AudioKit/AudioKitGraphics, which looks like it might contain what I need, but I can't open any of the files there since I don't have PaintCode. Does anyone happen to have a .psd, .afphoto, .svg, or similar file of the ENV panel background I could use? Thanks.

The Swift part and interface is rather trivial. And, is something I can make right away if you get the hard part working: writing the c/c++ code and modifing the kernel to allow envelope modulation per voice (which may need to be adapted custom for every parameter/destination). I’d recommend you just make some simple UI controls as a PoC to test the real work - writing the C++. If you get that working, circle back. 👍

Per voice...you mean independently for every oscillator + the FM system? I have absolutely no experience in signal processing. (: At the moment what I really want is to modify the LFO with an envelope, would that be any easier?

Only you can answer that question. I don’t know your skill level. Look at the code and decide.

I tried to add some parameters to the synth system in S1DSPKernel.hpp and S1Parameter.h (increasing S1ParameterCount accordingly), but it's now crashing on me. Any idea why that could be?


Document Object missing Open method

This page indicates that the Document Object has an Open method. However, the page it links to makes no mention of an Open method (in fact, searching for "open" yields 0 results).

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

ID: 994db147-1d84-2af7-e6f6-f01a1491ea4c
Version Independent ID: ccc98f6b-25cc-7d51-4894-d9e31879189d
Content: Word JavaScript API requirement sets
Content Source: docs/reference/requirement-sets/word-api-requirement-sets.md
Product: office 365
GitHub Login: @o365devx
Microsoft Alias: o365devx

@mscaitlina we're still investigating but preliminary info I've uncovered seems to indicate that this is an error in the Word JavaScript API requirement sets article. That is:

The open() method is not available on the Document object.
Rather, the open() method is available on the DocumentCreated object.

We're still investigating but I wanted to share this info with you now in the hopes that it will unblock you. I'll provide additional info here when there's definitive info to share.

The article has been updated to reflect that the open() method is available on the DocumentCreated object; thank you for bringing this issue to our attention.


quarkus.container-image.image not working as expected on OpenShift, wrong imagestream.yaml

Extension version: 0.0.6
When I deploy an application through Quarkus-helm extension over OpenShift with the following configuration
Application.properties
quarkus.openshift.route.expose=true
quarkus.container-image.image=quay.io/quarkusqeteam/ts-quarkus-helm-simple-app:latest
quarkus.helm.name=my-chart
quarkus.container-image.builder=docker

the generated imagestream.yaml is poing to docker.io instead of quay.io
Workaround:
Application.properties
quarkus.openshift.route.expose=true
quarkus.helm.name=my-chart
quarkus.container-image.builder=docker

quarkus.container-image.registry=quay.io
quarkus.container-image.group=quarkusqeteam
quarkus.container-image.name=minimum-quarkus-helm
quarkus.container-image.tag=v1

I can reproduce the issue, but it's a Quarkus issue, not a Quarkus Helm issue.
For the record, what this extension is doing is copying the generated resources by the Quarkus Kubernetes/Openshift extensions into the folder helm and mapping some keys into the values.yaml.
Please, report the issue in Quarkus. I'm closing this.

This issue should now be fixed using Quarkus 2.13.2.Final.


OrderingMethodFinder does not respect the source queryable item type

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Hot Chocolate
Describe the bug
When HotChocolate tries to apply sorting at query like:
IQueryable<Foo> queryable = repo.GetItemsQueryable();
var items = queryable
  .OrderBy(x => x.CreateDate)
  .Select(x => new Bar
    {
      SomeProperty = x.SomeProperty
    });

it fails with an exception
No generic method 'ThenBy' on type 'System.Linq.Queryable' is compatible with the supplied type arguments and arguments. No type arguments should be provided if the method is non-generic.

This is because the original OrderBy was applied to type Foo, and then HotChocolate tries to apply ThenBy to type Bar.
Steps to reproduce
See the above description.
Relevant log output
No response
Additional Context?
The issue can be fixed by adding a type check to OrderingMethodFinder like:
Spoiler: dirty code snippet example below
        public static bool OrderMethodExists(Expression expression)
        {
           var queryableItemType = expression.Type.GenericTypeArguments.Single();

            var visitor = new OrderingMethodFinder(queryableItemType);
            visitor.Visit(expression);
            return visitor._orderingMethodFound;
        }

......
        protected override Expression VisitMethodCall(MethodCallExpression node)
        {
            var name = node.Method.Name;

            if (node.Method.DeclaringType == typeof(Queryable) &&
                node.Type.GenericTypeArguments.Single() == _queryableItemType &&
                (name.StartsWith(nameof(Queryable.OrderBy), StringComparison.Ordinal) ||
                name.StartsWith(nameof(Queryable.ThenBy), StringComparison.Ordinal)))
            {
                _orderingMethodFound = true;
            }
            return base.VisitMethodCall(node);
        }

Version
12.15.2

I'm seeing the same issue using EFCore when the generated query has added ordering internally (usually when using join/include).  If I return the result with .OrderBy(x => true) which returns an IOrderedQueryable, the query no longer fails.


Clipboard extension doesn't work since update to ubuntu 21.10

In ubuntu 20.04 I didn't have problem. After dist-upgrade from 20.04 to 21.10, clipboard extension didn't work. I try after restart but didn't work. I try too uninstall/install extension but same issue.
I already use with gpaste and it works when I used it standalone. I try with copyq too but doesn't work too.
Distro ubuntu 21.10 - gnome 40
Log when I trigger keyword to open clipboard addon on ulauncher
com.github.friday.ulauncher-clipboard | 2022-01-05 18:50:57,280 | DEBUG | ulauncher.api.client.Client: on_message() | Incoming event KeywordQueryEvent
Traceback (most recent call last):
  File "<string>", line 1, in <module>
AttributeError: 'NoneType' object has no attribute 'get_filename'
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/ulauncher/api/client/Client.py", line 54, in on_message
    self.extension.trigger_event(event)
  File "/usr/lib/python3/dist-packages/ulauncher/api/client/Extension.py", line 52, in trigger_event
    action = listener.on_event(event, self)
  File "/home/rmaud/.local/share/ulauncher/extensions/com.github.friday.ulauncher-clipboard/main.py", line 89, in on_event
    icon = getThemeIcon('edit-paste', 32)
  File "/home/rmaud/.local/share/ulauncher/extensions/com.github.friday.ulauncher-clipboard/lib.py", line 40, in getThemeIcon
    return execGet(sys.executable, '-c', "import gi; gi.require_version('Gtk', '3.0'); from gi.repository import Gtk; print({})".format(getIconCode))
  File "/home/rmaud/.local/share/ulauncher/extensions/com.github.friday.ulauncher-clipboard/lib.py", line 21, in execGet
    return subprocess.check_output(list(args)).rstrip().decode('utf-8')
  File "/usr/lib/python3.9/subprocess.py", line 424, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/usr/lib/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/usr/bin/python3', '-c', "import gi; gi.require_version('Gtk', '3.0'); from gi.repository import Gtk; print(Gtk.IconTheme.get_default().lookup_icon('edit-paste', 32, 0).get_filename())"]' returned non-zero exit status 1.

Ask me if you have more questions

You probably need to install an icon theme. And this whole workaround won't be needed in the next major Ulauncher version.

Should be fixed now


Minor tweaks to cache-expiration-docs

R: @jeffposnick @addyosmani
@jeffposnick is the updateTimestamp() useful in CacheExpiration class? If so, is there something more useful we can include in the docs?

updateTimestap() is needed for folks who choose not to use the Plugin and instead use the CacheExpiration class directly, i.e. https://github.com/GoogleChrome/workbox-microsite/blob/master/src/_examples/workbox-cache-expiration/sw.js#L15
I can add a commit to your PR with a better explanation.


[IOTDB-3938] Avoid serializing schema in wal for better write performance of standalone version

Coverage decreased (-0.008%) to 41.992% when pulling a6783f8d141377d8a5e7da63ee61808b0c52121d on write_preformance_wal into 544c60ff877ecc0c7e7b2924814572adfacdc228 on master.


Menus should only show top level options on mobile

https://docs.microsoft.com/en-gb/dotnet/csharp/
On the above page - the sidebar menu on the desktop site is hierarchical.
On mobile however, all menu items are listed in a single list. That's hundreds of items on this page - which makes the menu pretty difficult to use. It would be great if just the current hierarchy level was shown on mobile.
This problem was seen on Chrome 63.0.3239.73 on iPhone, but I can reproduce in Chrome's device simulator.

Moved issue to https://github.com/MicrosoftDocs/feedback/issues/219


Proposal: Make Red-Black tree in std.rb generic

At the moment, the implementation of red-black trees in the standard library are not generic and need to be embedded in other structs in order to work as intended.
https://github.com/ziglang/zig/blob/a2bb246db4c2bb88f402215d5db79a535dbff4b6/lib/std/rb.zig#L16-L27
This is in contrast to other data structures in the standard library such as the linked lists, which are generic:
https://github.com/ziglang/zig/blob/a2bb246db4c2bb88f402215d5db79a535dbff4b6/lib/std/linked_list.zig#L15
https://github.com/ziglang/zig/blob/a2bb246db4c2bb88f402215d5db79a535dbff4b6/lib/std/linked_list.zig#L164
https://github.com/ziglang/zig/blob/a2bb246db4c2bb88f402215d5db79a535dbff4b6/lib/std/atomic/queue.zig#L10
https://github.com/ziglang/zig/blob/a2bb246db4c2bb88f402215d5db79a535dbff4b6/lib/std/atomic/stack.zig#L8
Are there any downsides to making the RBTree Node and Tree data structures generic like the other pointer-oriented data structures?

@joachimschmidt557 https://github.com/pedromsilvapt/zig-stack-vm/blob/master/src/rbt.zig


Translating emoji results in: JSONDecodeError

from googletrans import Translator
translator = Translator()
translator.translate('this does not work 😭')

Will result in:
*** json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

I can see it results in a 403
Similar problem, now closed:
https://github.com/terryyin/translate-python/issues/27

Try this  PR https://github.com/ssut/py-googletrans/pull/134.

Works! But the emoji is not included in the translation. Don't know if it gets removed on Google's side?

Emoji in translation  is  Schrodinger's cat . Sometimes include, but sometimes not.  🤷♂

Same for me trying to use batch translation with emojis:
translator.translate(['hola','adios', '💚⚽'], src='es', dest='en')
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)


chrome will remove DOMXXX event listeners on July 30

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

Running Vitest Browser 2.0 beta 5 on tresjs test I can see a lot of warnings from chrome in the console about relying on DOMXXX event listeners:

The example can be found here: https://github.com/Tresjs/tres/pull/715
Describe the solution you'd like

Here the link https://chromestatus.com/feature/5083947249172480 .
Describe alternatives you've considered

None
Additional context

NA

Hi @userquin
VTU listens to all possible DOM events (see dom-events.ts), so I think this warning is expected, and probably not a big deal (if you don't use these events yourself in your application).
Maybe we can remove them in a future VTU version to avoid the warnings and hope that this won't break tests out there that rely on these.
If you're willing to, a PR would be most welcome.

This message is spammed in every Cypress component test, unluckily
More a nuisance than a real problem

Why is VTU listining for those events in the first place?
Just to allow devs to listen to them in their tests?
Or are they actually used by the library?

Yes, VTU listens to all possible events to record them and allow developers to check them with wrapper.emitted() (see vueWrapper.ts)
https://github.com/vuejs/test-utils/blob/582d7d7bc7d86babbdd3c13de2f7150b80686e84/src/vueWrapper.ts#L200-L209
This code was probably the same in VTU v1. I started helping with the maintenance after this was written so I don't know why this design was picked.
Anyway, I think removing the culprit events from dom-events.ts should be enough. This means developers won't be able to check them with emitted() but as these events are getting deprecated, I suppose it's fine.


Can I upgrade from 1.1.38 to 1.2.0

According to the Changelog, rocksdb has been revert to previous version in 1.2.0.
Will it break anything if I upgrade from 1.1.38?

yup, the minor version wouldn't have the breaking change, so it's ok to upgrade the 1.1.x to the latest.

yup, the minor version wouldn't have the breaking change, so it's ok to upgrade the 1.1.x to the latest.

Got it. Many thanks!

Got it. Many thanks!


Allow to stop unwinding at given address

Bug: http://b/236121587
Test: Run on sample and verify to get complete callstacks on __wine_syscall_dispatcher

Also one high-level question for the whole solution. We have that mechanism to filter out callstacks based on the majority of the outermost frame of a thread. How will this work with it?

Thanks for the review. So here is the general picture:
On OrbitApp:

We hardcode __wine_syscall_dispatcher + it's module ntdll.so as functions were we want to stop unwinding.
We use ModuleManager::GetModulesByFilename to retrieve the function infos for the functions that match our hardcoding and store them in the capture options.
In OnCaptureComplete, we also send the "set of functions" to stop unwinding at to the FilterCallstacks function, such that the minority functions do not get filtered out when they are in this set.

In capture.proto and CaptureClient:

We add a new proto message for the functions to stop unwinding at to capture proto and transform the function infos from app into this proto.

In TracerImpl:

We forward the functions to stop unwinding at to LibunwindstackUnwinder (which transforms those into the std::map) as well as to UprobesUnwindingVisitor

In LibunwindstackUnwinder: We ensure Unwind gets called according to the changes in this PR.
In UprobesUnwidingVisitor:

If the callstack has size 1, it is currently getting filtered out. We relax that check in case we are in a function to stop unwinding at.