Add removeMany and keepOnly

These are variants of Dict.diff and Dict.intersect that I find to present a more useful API, given that in Dict.diff and Dict.intersect one argument is a dictionary whose values are irrelevant. See https://github.com/elm-lang/core/issues/518#issuecomment-192820149.
Also, removeMany is a very natural generalisation of the existing function Dict.remove.

Arguably, for removeMany passing the elements to be removed as a list makes more sense. I'll make that change.

Hmm. I find it odd that one of the fns take a Set and another a List. As a user I would find it annoying if, through refactoring or simply calling the wrong function, I would have to change the data type of the input for things to work.
It also seems a little odd that the name of the two functions are so different, when what they do is very similar (one of them keeps values, the other one removes). Would it be better to reflect this similarity by changing the names to keepOnly and removeOnly?

Aren't these concerns just caused by the fact that I put both functions into the same PR? If we look at the two functions as separate proposed additions, is it really strange that they take different input types? Is it really strange that they have names that don't have a common word stem? After all, removeMany shares its word stem with Dict.remove and with Dict.Extra.removeWhen. I don't really see why it should share word stem with the keepOnly function. Also, the functions Dict.intersect and Dict.diff in the core package, which play similar roles, don't have any name similarity. So I don't really get this naming issue.
About refactoring, are there really situations where one would want to switch from one of these functions to the other, more often than switching from one of them to some completely different function?
What I'm really getting at: Let's consider the two functions separately, and maybe I should have made two separate pull requests.
Let's start with removeMany on its own merits. Is the name appropriate for it? And is List comparable -> Dict comparable v -> Dict comparable v the proper type for it? I'd answer yes to both questions, given the existing names Dict.remove and Dict.Extra.removeWhen and given the fact that one really doesn't need the collection of to-be-removed-elements to be a set, since one only needs to process those elements one after another in arbitrary order, and doesn't need to do lookups in that collection.
Plus, if we have
removeMany : List comparable -> Dict comparable v -> Dict comparable v

but want
removeMany' : Set comparable -> Dict comparable v -> Dict comparable v

that's easy (and cheap, with O(n) complexity) to achieve with Set.toList.
Whereas conversely, if we have
removeMany : Set comparable -> Dict comparable v -> Dict comparable v

but want
removeMany' : List comparable -> Dict comparable v -> Dict comparable v

we will have to convert a list to a set, which is not O(n), and we will do this conversion for no good reason, since the removeMany : Set comparable -> ... we will call from removeMany' : List comparable ->  will anyway just do a foldl on the list-converted-to-set. It won't actually use any algorithmic property of sets like efficient lookup.

My concerns remain even if you propose these functions as two separate PRs. I'm trying to see the entire library from a user perspective. Let me try to explain my position better.
What you're essentially proposing is two functions which both takes a sequence of keys and a dict, and then provides you with a filtered version of that dict. Why should this library have two functions that accepts two different data structures for a sequence of keys?
Let's say that removeMany is perfect (and I believe it is, I'm ready to merge it right now), why should keepOnly take a Set when removeMany takes a List? After all, keepOnly can be implemented as (not tested):
keepOnly : List comparable -> Dict comparable a -> Dict comparable a
keepOnly keys dict =
    List.foldl (\k acc -> Dict.update k (always (Dict.get k dict)) acc) Dict.empty keys

The naming of the functions isn't that big of a deal to me. I just think it makes sense that when you have two functions that takes a sequence of keys and gives you a filtered Dict back, that the only difference in their name is the actual difference in what it does (in this case, remove and keep).
Regarding Diff.intersect, Diff.union and the like... I don't like those names either, as they don't mean much to those who haven't learned set theory in school.

Btw, could you also add some simple unit tests?

About unit tests, I'll add something.
About seeing the library from a user perspective, as a whole, that of course makes sense, though I would say that "the whole" should not only comprise of Dict.Extra from this package, but instead of Dict.Extra from this package plus Dict from core (possibly without intersect and diff). What that suggests in terms of naming, I don't know.
About lists vs. set arguments: My reason for having keepOnly take a set is algorithmic. Really much like the comparison of the two ways of having removeMany vs. removeMany' in my previous comment. With the implementation for keepOnly that I proposed, it would be a disadvantage for the user to make the argument a list and only inside keepOnly transform it into a set (or do inefficient lookups in the list). Both users that want to provide a list and users that want to provide a set to removeMany are well served when the provided removeMany takes a list. And both users that want to provide a list and users that want to provide a set to "my implementation" of keepOnly are well served when the provided keepOnly takes a set. In both cases, switching the choice would serve at least some set of users worse.
Now, "your implementation" of keepOnly has different characteristics, and indeed some more in-depth analysis of complexity trade-offs might be in order, and might actually lead to the realization that two different versions of keepOnly should be provided, one for the case when the dictionary is the dominant factor and one for the case when the sequence of to-be-kept elements is the dominant factor? What I'm really saying is: First we have to sort out which algorithm to use (do we want to iterate over the dictionary or over the sequence?), then what is the most useful interface for passing input to that algorithm.

After some more thinking: If those functions were to ever be moved to inside core, then for both keepOnly and removeMany more efficient (than those discussed here, and than those for diff and intersect currently in core) implementations should be possible by exploiting the internal tree representation, provided they eschew list representations. So, in a turn around, to be prepared (API wise) for that eventual case, I would now swing back to making removeMany take a set as first argument. We cannot really get ideal computational complexity in Dict.Extra anyway, due to not having access to internal representations. So the decision should be guided more by API design than by efficiency considerations as I tried above.

Cool. After closer consideration, I'm also fine with the names. Disregard any comment I've made about the naming of the functions.
One last thing for consideration and I swear I'm not trying to be difficult; just trying to cover all bases.
Lists vs Sets for keys:

When retrieving the keys from a Dict using Dict.keys, they are returned as a List. When typing, it's also marginally less to type by creating a List instead of a Set. I would therefore assume that, to a user, handing in a List would be preferred as you wouldn't have to convert it to a Set before using these functions.

It appears to me that the user will rarely want to remove/keep all or more keys in a Dict. Because of that, I believe it would be more efficient to implement these functions as folds over the sequence of keys, rather than using Dict.filter (as the last will iterate over all key-values in the dict).

If nothing else, people will look back at this github issue and see we considered all the options :D

Considering all options is good. :smile:
About the point concerning Dict.keys returning a list: That's something that already occurred to me in https://github.com/elm-lang/core/issues/518#issuecomment-192820365, but as noted there, it seems that Dict.keys will eventually switch to returning a set. Of course, one could still use lists here for now and switch to sets "eventually" as well.
My main reason for sets now would be the documentation value of the type signatures. It really makes no sense to remove a key more than once or to keep a key more than once, but lists will allow to pass duplicate keys. Using sets is raising an explicit flag about this. But I'm fine switching to lists for convenience's sake (when people want to write collection literals).
About the 2. major point, that seems independent of whether to use lists or sets, right? I think your hunch about most frequent uses of these functions is right, and this also factored into my realization this morning that it's not really possible to make complexity based decisions at the moment, because we don't know what the dominant sizes will be (and since we can't make O(n+m) implementations outside core).
So the next steps would be:

decide whether to use lists or sets (both ultimately fine with me)
implement keepOnly as a foldl (List.foldl or Set.foldl, depending on decision above)

?

I agree with all your points.
I wasn't aware that it was planned for Dict.keys to return a Set. With that knowledge, I think it's fine for us to implement these functions using Set for keys.

Okay, I've made the changes.

Merged.
Thank you for high quality commits and discussion :)


Make tests/full.rs no longer use git diff

Currently, tests/full.rs will overwrite the files containing the expected output of cargo-semver with its actual output, then call git diff to see if there are any changes. This has a couple of downsides as discussed in #99.
This PR changes the behavior to do the following instead:

Capture the new output in memory and compare it against the stored version.
If they differ, put the new version into $OUT_DIR for inspection.
To make up for the lost functionality from git diff, suggest a command to view the diff.

This means that tests can now run even if the source directory is read-only, not checked out via git, or if git is not installed at all. It also means that failing tests do not cause a dirty working tree any more and cargo clean as well as git clean -x work as intended.

Looks good to me, works, no complaints. Thanks and congrats on item #100!


Is "override" a correct modifier ?

According to https://docs.microsoft.com/en-us/uwp/midl-3/intro#protected-and-override-methods

A method can be overridden in a derived class. When an instance method declaration includes an override modifier, the method overrides an inherited virtual method with the same signature. Whereas a method declaration ordinarily introduces a new method, an override method declaration specializes an existing inherited virtual method by providing a new implementation of that method.

As I tried it out, "override" doesn't seem to be a valid modifier .

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

ID: 2dacbdf2-172f-dd41-829f-94295cd0c621
Version Independent ID: b511accc-e0b1-9b30-d044-231d5204024d
Content: Introduction to Microsoft Interface Definition Language 3.0 - Windows UWP applications
Content Source: winrt-related-src/midl-3/intro.md
Product: uwp
Technology: winrt-reference
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Thanks, @zh4ui! I'm investigating.

I've updated the docs in line with how we expect the keyword to work. You're right, override isn't a valid keyword; it's overridable.
-Steve


add a simple kafka destination

Adds a simple kafka destination. Lots of room for improvement, but it does work. You can hook up a sqlite append-only source or hello world source to a kafka destination and you'll see the payloads show up as json. There are many things that don't work, e.g. binary columns from sql, but there's a fun happy path to play with. \
TODO:

[ ] Add the frontend component so the UI is more consistent.

🥳


Jenkins's screed PR tester does not behave similarly to the khmer version

Investigation required/requested

@bocajnotnef I think I fixed this, yes?

Yes, we did.
On Thu, Mar 12, 2015 at 7:31 PM Michael R. Crusoe<EMAIL_ADDRESS>wrote:

@bocajnotnef https://github.com/bocajnotnef I think I fixed this, yes?
—
Reply to this email directly or view it on GitHub
https://github.com/ged-lab/khmer/issues/810#issuecomment-78691443.


File routes and dataset thread

Following merged PR #248 but for TEST and next round of release

Replaced by https://github.com/hubmapconsortium/ingest-api/pull/256


[Question]: Support for multi-gpu embedding

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Is there any support for embedding model parallelism across multiple gpus? From what I can tell, both the InstructorEmbedding and HuggingFaceEmbedding models are currently defaulting to cuda:0 (with device not being passed to either during the instantiation in resolve_embed_model -- an easy update). But going a step further, I'm interested in how we could configure model parallelism here.
Thanks

Did someone figure out a way to do this? Current SOTA Embedding models are way to large for a single Consumer-GPU: https://huggingface.co/Salesforce/SFR-Embedding-Mistral

@fabianslife using a 14GB model is definitely not worth it imo. The accuracy increase is minor for such a large computation increase.
I'd recommend using multiple replicas of text-embeddings-interface with some kind of load balancer on top if you want to leverage multi gpus. (I.e. using kubernetes or something similar)


[FEATURE] Display link-name flag even if optional

Is your feature request related to a problem? Please describe.
When starting capability providers or putting link definitions, the link-name flag is "optional" because it can default to the value default. This works well in development environments where the link name is rarely modified, but in more complex scenarios link names are used frequently. The value of default is reasonable as it covers the 80%+ use case, but in the help text for wash we should show --link-name as a flag.
Today, this is what wash ctl link put shows:
~/github.com/cosmonic/demo-scripts ➜ wash ctl link put
error: The following required arguments were not provided:
    <actor-id>
    <provider-id>
    <contract-id>

USAGE:
    wash ctl link put [OPTIONS] <actor-id> <provider-id> <contract-id> [values]...

For more information try --help

Describe the solution you'd like
Ideally, the above message should be:
~/github.com/cosmonic/demo-scripts ➜ wash ctl link put
error: The following required arguments were not provided:
    <actor-id>
    <provider-id>
    <contract-id>

USAGE:
    wash ctl link put --link-name <LINK_NAME> [OPTIONS] <actor-id> <provider-id> <contract-id> [values]...

For more information try --help

The difference here is that the --link-name flag is shown even though it is optional. There should be a clap option to do this.
Describe alternatives you've considered
Alternatively we could remove the default value of default, but this would introduce a jarring breaking change and negatively affect the 80% use case.

@emattiza yeah would love to take a look at what that looks like, I haven't had a chance to look at the clap args quite yet but feel free to open a draft PR and we can collaborate there

Fixed in #254


Bug: onDone not getting called when all child states are final

Description

Fill out "Name" field
Click "Submit" button. There is an error that the Category is missing
Select an option for the "Category" and "Network" dropdowns
Click "Submit" button

Expected result
The states are all in a "final" state. The "onDone" transition should be called and put us into the "Creating" state.
Actual result
We are still in the "Ready" state. All of the child nodes are in the "Submitting" state.
Reproduction
https://codesandbox.io/s/ondone-demo-26sy9n
Additional context
If I ignore step 2 from above, then we transition to the "Creating" state.

Fill out "Name" field
Select an option for the "Category" and "Network" dropdowns
Click "Submit" button

I'm not sure why the addition of the "Submit" button click would cause any issues.
Also, if I add a self-transition in the "Submitting" state (see here), then it transitions to the "Creating" state.
Name: {
  initial: "Ready",
  states: {
    Ready: {
      // ...
        Submitting: {
          type: "final"
+            on: {
+              SUBMIT: {
+                 target: "Submitting"
+              }
+            }
              Error: {}
            },
            // ...
          },

See the discussion here. @davidkpiano I think this should become recursive in v5 (and thus it should deviate from SCXML), what do you think?

This was addressed by https://github.com/statelyai/xstate/pull/4358


Two raw HTML blocks without blank lines between

Given:
The command-line client ships alongside the daemon. Usage information for all commands, including sub-commands, is available via the --help flag.

## textile [<flags>]

Textile is a set of tools and trust-less infrastructure for building censorship resistant and privacy preserving applications

Flag	Description

--help
Show context-sensitive help (also try --help-long and --help-man).

--help-long
Generate long help.

--help-man
Generate a man page.

--completion-bash
Output possible completions for the given args.

--completion-script-bash
Generate completion script for bash.

--completion-script-zsh
Generate completion script for ZSH.

--api="http://<IP_ADDRESS>:40600"
API Address to use

--api-version="v0"
API version to use

--debug
Set the logging level to debug

## textile help [<command>]

Show help.

Argument	Description

[<command>]
Show help on command.

## textile account

Manage a wallet account

The latter two headers are not converted to h elements. If one changes it to:
The command-line client ships alongside the daemon. Usage information for all commands, including sub-commands, is available via the --help flag.

## textile [<flags>]

Textile is a set of tools and trust-less infrastructure for building censorship resistant and privacy preserving applications

Flag	Description

--help
Show context-sensitive help (also try --help-long and --help-man).

--help-long
Generate long help.

--help-man
Generate a man page.

--completion-bash
Output possible completions for the given args.

--completion-script-bash
Generate completion script for bash.

--completion-script-zsh
Generate completion script for ZSH.

--api="http://<IP_ADDRESS>:40600"
API Address to use

--api-version="v0"
API version to use

--debug
Set the logging level to debug

## textile help [<command>]

Show help.

Argument	Description

[<command>]
Show help on command.

## textile account

Manage a wallet account

Then they are.

Thanks for the report. This is a weird one. It appears that the raw HTML parser is failing to find the end of the raw HTML block and consuming the rest of the document. Here is the minimum example I found which exhibits the problem:

Raw HTML

## Header

The above results in this output:

Raw HTML

## Header

Note that this requires all of the following:

The
 must have a
 nested in it.
There must be no blank line between the
 and
.
I tried a
 element instead of a
 and the problem did not persist. I did not try any other elements yet.
The actual contents of the
 seem to not matter, thus they are excluded in the above example.

Given number 2 above, as a workaround, you can add a blank line between each of the your raw HTML block level elements (add a blank line between your
 and
 elements). This should not be surprising as the rules state that each block level raw HTML element must be surrounded by blank lines. However, in practice, all other implementations don't require the blank line so we shouldn't either. This is a bug.

I have a suspicion that this could be related to #780. The result is different, but the tag counting alogarithm is wrong in both cases.

I suspect you are right...not looking forward to unraveling this 😬

This is issue doesn't exist in #803. Of course, that is incomplete, but our efforts might be better utilized by finishing that than by addressing each of these individual edge cases in the existing code base.

That would honestly be my preferred route. Getting a nice clean implementation that is easier to debug would be great.


[gradlew.bat] Task 'install' is ambiguous in root project 'baker-android-refactor'

Similar problem here. Have you figured it out?


error parsing config

My case is not standard. I am storing in config part of POST request, but I really not found nothing better.
dini fail on next config:
key1 = NW_Pacific%2FOverview%2Fvis_ir_background%2Fgms_6_lowcloud&DIR=%2Fnexsat_data%2FPUBLIC%2FCONUS%2Ffocus_regions%2FNW_Pacific%2FOverview%2Fvis_ir_background=

You can pass Ini.Parse[String](target, false) to disable lookups.


[Subject Pages] hide error component

On staging and prod, hide this error message: https://screenshot.googleplex.com/ASLVAEemSrt7uJU

fixed in: https://github.com/datacommonsorg/website/pull/3401


Improved Java wrapper generation

resolves https://github.com/Itseez/opencv_contrib/issues/652
While generating Java JNI wrappers package names with an underscore (_) character where not properly escaped according to https://docs.oracle.com/javase/8/docs/technotes/guides/jni/spec/design.html#resolving_native_method_names (see also: https://github.com/Itseez/opencv_contrib/issues/652).
This fix replaces all the occurrences of _ with _1 resulting in proper JNI method names.

@dharezlak , if you create another PR (copy of https://github.com/Itseez/opencv_contrib/pull/656) to the opencv_contrib using same branch name (patch-1), then both PRs will be built and tested together and we will see how it works.

Will https://github.com/Itseez/opencv_contrib/pull/679 do the trick?

Yes, I've restarted all builds, they use both repositories now: see opencv_merge and opencv_contrib_merge revisions

:+1:


License?

Heyo! Starting work on an open source LN/WN reader for android and would love to use this as part of my backend, what are you guys doing licensing wise? Wouldn't want to use someone's code for my own project without everything being proper. I see you have it listed as MIT but if you guys have personal credits and stuff you'd like included (or just don't want it used) I'll respect that.

Nah just the MIT license with a link is fine. Keep the used libraries mind through.

Great! and I definitely will.


Better changelog

Would it be possible to keep a proper changelog of the changes for each version? Are there any notable changes beyond getting turned into an addon instead of a fork? (also my discord account got disabled [waiting for appeal to bring it back -_-] so if you've pinged me and I haven't responded that's why)

Yes i am very inconsistent in my changelog
I will refine it for future updates starting now


Bot stoping recerve message

Hi,
Thank you fro this lib. I have big problem...
I'm use Bot, ex:
       var Bot = new TelegramBotClient(apiKey);
       Bot.OnMessage += Bot_OnMessage;
       Bot.StartReceiving();
        private static void Bot_OnMessage(object sender, Telegram.Bot.Args.MessageEventArgs e)
        {
            try
            {
                var message = e.Message;
                //some CODE
            }
            catch (Exception ex) {
                Log.logger.Fatal("Error:{0}", ex.Message);
                Log.logger.Fatal("Error:{0}", ex.StackTrace);
            }
            //throw new NotImplementedException();
        }

After 1-3 hours stops receiving messages...
Can I do something wrong?
May need to reconnect, when raise OnReceiveGeneralError or OnReceiveError?
NuGet Package Version:  10.4.0
.NET Version: 4.6
IDE: VS2015
App: Console App and ASP.NET MVC

Are there some errors / exceptions logged?

Error list is empty((

this errors from OnReceiveGeneralError and OnReceiveError
2017-05-24 04:32:46.4355     ERROR    OnOnReceiveError: Invalid response received
2017-05-24 04:32:46.4355     ERROR    OnOnReceiveError:    at Telegram.Bot.TelegramBotClient.<SendWebRequestAsync>d__125`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()
2017-05-24 04:32:46.6416     ERROR    OnOnReceiveError: Invalid response received
2017-05-24 04:32:46.6416     ERROR    OnOnReceiveError:    at Telegram.Bot.TelegramBotClient.<SendWebRequestAsync>d__125`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()
2017-05-24 05:25:20.0010     ERROR    OnReceiveGeneralError: Response status code does not indicate success: 500 (Internal Server Error).
2017-05-24 05:25:20.0156     ERROR    OnReceiveGeneralError:    at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
   at Telegram.Bot.TelegramBotClient.<SendWebRequestAsync>d__125`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()
2017-05-24 05:25:22.6826     ERROR    OnOnReceiveError: Invalid response received
2017-05-24 05:25:22.6826     ERROR    OnOnReceiveError:    at Telegram.Bot.TelegramBotClient.<SendWebRequestAsync>d__125`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()
2017-05-24 05:25:22.8887     ERROR    OnOnReceiveError: Invalid response received
2017-05-24 05:25:22.8887     ERROR    OnOnReceiveError:    at Telegram.Bot.TelegramBotClient.<SendWebRequestAsync>d__125`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()
2017-05-24 05:25:23.0888     ERROR    OnOnReceiveError: Invalid response received
2017-05-24 05:25:23.0888     ERROR    OnOnReceiveError:    at Telegram.Bot.TelegramBotClient.<SendWebRequestAsync>d__125`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()
2017-05-24 05:25:23.2969     ERROR    OnOnReceiveError: Invalid response received
2017-05-24 05:25:23.2969     ERROR    OnOnReceiveError:    at Telegram.Bot.TelegramBotClient.<SendWebRequestAsync>d__125`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()
2017-05-24 05:25:23.4980     ERROR    OnOnReceiveError: Invalid response received
2017-05-24 05:25:23.4980     ERROR    OnOnReceiveError:    at Telegram.Bot.TelegramBotClient.<SendWebRequestAsync>d__125`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()
2017-05-24 05:25:23.7119     ERROR    OnOnReceiveError: Invalid response received
2017-05-24 05:25:23.7119     ERROR    OnOnReceiveError:    at Telegram.Bot.TelegramBotClient.<SendWebRequestAsync>d__125`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()

Same issue.
Cannot recieve messages, Inline Query ,Callback Query.
Exception thrown: 'System.ArgumentOutOfRangeException' in Telegram.Bot.dll System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values.    at Telegram.Bot.Types.Update.get_Type()    at Telegram.Bot.TelegramBotClient.OnUpdateReceived(UpdateEventArgs e)    at Telegram.Bot.TelegramBotClient.<ReceiveAsync>d__54.MoveNext()
Using Telegram.Bot 10.4.0

@MrDevIr if you are using a switch(update.Type) check your default statement.  Most likely since you are using an older version, there is a new update type that you are not handling.  If using ReSharper, and you created the switch statement with that, the case: default is always to throw that exception.

@AugerSoft sometimes Telegram has.. hiccups.  I'd recommend adding this to your on error method:
            {
                Bot.StartReceiving();
            }```

@parabola949 Thanks, I just updated the package to newer (beta) version and the problem got solved

@AugerSoft You are using an old package. Use .Core package instead.
See why on  project's README.


Bug : [Mainnet] Prover aborted tx send due to critical error: failed to get tx into the mempool

Describe the bug
I'm getting this err :
taiko_client_prover_relayer-1  | ERROR[06-18|02:53:14.580] Submit proof error                       blockID=76054 minTier=200 error="aborted tx send due to critical error: failed to get tx into the mempool"
taiko_client_prover_relayer-1  | ERROR[06-18|02:56:38.646] Submit proof error                       blockID=76054 minTier=200 error="aborted tx send due to critical error: failed to get tx into the mempool"
taiko_client_prover_relayer-1  | ERROR[06-18|03:00:02.711] Submit proof error                       blockID=76054 minTier=200 error="aborted tx send due to critical error: failed to get tx into the mempool"
taiko_client_prover_relayer-1  | ERROR[06-18|03:03:26.777] Submit proof error                       blockID=76054 minTier=200 error="aborted tx send due to critical error: failed to get tx into the mempool"
taiko_client_prover_relayer-1  | ERROR[06-18|03:06:50.842] Submit proof error                       blockID=76054 minTier=200 error="aborted tx send due to critical error: failed to get tx into the mempool"

Log is fine yesterday.

Raiko log is good also

Steps to reproduce
Steps to reproduce here.
Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

I can see it's retrying each 3 or 4m . But still fail and it will run out of windows time for submit proof..
If you do retry failed need to reverted that prover transaction.
I can't accept lose 250 Taiko again because weird err on mainnet. that's not acceptable.

The issue was solved after adding more ETH in prover balance..

We can't monitor our prover 24/24 because that weird err with gas bump.
Team need do action after some retry because 1h windows time for submitting proof.
We need better log clarify on issue so we can have better action. I can't even understand why I'm getting that err.

The recommended way is to add a prover balance alert.
And will optimize the logger.


setting to keep cache even if job fails

I use this action for the cache of htmltest, a link checker. Often it'll get through plenty of links before finding a broken one, but because the job fails the cache isn't saved. I'd like for there to be a setting to keep the cache even if the job fails.

@kylrth cache save happens at the end of workflow run. This happens because the directory/file cached can change till the end of run. Saving cache after every step would create a lot of requests which many users won't want.

That's not what I'm asking for. The checkout action has a post run step that runs regardless of the success of the run, as you can see in this example. What I'd like is for there to be a setting where the same behavior can happen for the cache action.

@kylrth actions/checkout runs for cleanup. If cache was allowed to save cache even on job fail then it could create corrupted cache, in most cases. If there is more request for this feature we would work on this.

I'm pretty sure that there already is more than a request for this feature (https://github.com/actions/cache/issues/92).

Thanks @jorgemo-fever for pointing that. We have internal discussions related to these issues.

I guess I'll close this issue then since it's a duplicate of #92. It's a shame this hasn't been added because there is definitely interest.

Hey everyone 👋🏽
Two new actions actions/cache/restore and actions/cache/save are now available with tag v3 to everyone for use. These can now be used to achieve granular control on the restore and save steps in the cache action.
Do try them and give your feedback. We believe these new actions will take care of your use cases. 🙇🏽
The save action can be made to run always by passing always() in the if condition.


Output a warning when using a django-storages backend with overwriting behaviour

Is your proposal related to a problem?
Several of the storage backends provided by django-storages, such as the one for Amazon S3, default to silently overwriting the existing file when uploading a new file with the same filename. This violates the Django storage API (because they just return immediately from get_available_name without actually checking whether the filename is available) and breaks some Wagtail functionality - see https://github.com/wagtail/wagtail/pull/4567#issuecomment-392934164 and #9789.
This is documented at https://docs.wagtail.org/en/stable/advanced_topics/deploying.html#cloud-storage but users are unlikely to find that advice until it's too late.
Describe the solution you'd like
Wagtail should add a Django system check, so that if one of these storage backends is in use, and the relevant setting (AWS_S3_FILE_OVERWRITE for S3, different for others) is set to cause overwriting, then a warning is output on the console on startup.
Describe alternatives you've considered
We could identify and fix all the places in Wagtail that are affected by this, and apply workarounds so that everything still works with overwriting enabled. However, doing this in the test suite is hard (because we'd have to re-run everything with a mock storage backend with the overwriting behaviour); it would require duplicating the checks that a well-behaved storage backend should be doing already; and it undermines Django's aim of providing a common API that works consistently across all backends.

Hi@gasman i in interested in working on this issue. I am new to open source and excited to contribute something meaningful.
Can you assign this issue to me ?

Hi @Nazi-pikachu - we don't usually assign issues, but you're very welcome to work on this!

Thanks @gasman
Can you suggest in which app should i add the relevant check
I am thinking of adding it in admin

@Nazi-pikachu go for it - that makes sense - if it needs to move when your PR is reviewed it should be easy to do.

@lb-  I have made a PR #9823  , Please review it.

Fixed in #9823


Manager listenAddress can't be changed

Stopping manager and starting it again with a different port still shows the old address in swarmctl manager ls. Joining a second manager to that node comes up as reachable but can't be used(probably because the connection is only one way).
After switching back the address:
» swarmctl  cert ls
Error: incorrect cluster state

WARN[0040] sending message to an unrecognized member ID 5e815a6c316dfc82
ERRO[0040] could not find cluster member to query for leader address

@abronan

@aaronlehmann Is this still relevant given the listen/advertise improvements of 1.12?
Also, there's a second issue in this ticket: changing advertised address. Should we log an issue for that? Should we attempt a fix for 1.13 or is that low priority?

I think it is still relevant.
Changing the advertise address is important to support but nontrivial. It probably needs more discussion before we choose a milestone.

Related to #2198.

I recently had this issue with a cluster of Raspberry Pi's I wanted to move somewhat smoothly between networks. I found that adding iptables rules redirecting connections destined to the old manager addresses via NAT to the current addresses would get the swarm online, after which manager nodes can be removed and re-added one by one to update the raft addresses and such.
I kept fiddling around with this and came up with a script to do this automatically, and then proceeded to wrap the whole catastrophy in a Docker container.
It's far from perfect, but it should keep any cluster running on Debian, or a derivative like Raspbian, intact shortly after changing all the IP addresses.
In short, the script runs on all manager nodes, generates one SSH key per manager and distributes those to other nodes using services, relies on Avahi to find the new addresses, adds iptables NAT rules to get the swarm online, uses temporary services as distributed locks to re-join one by one, removes the iptables rules when they're no longer required, proceeds to re-invite any non-privileged workers, and then sits happily watching for swarm changes.
Feel free to fiddle around with it, draw inspiration or whatnot, and if you make improvements or other customizations, submit a pull request and I'll happily include it.
 I know. I happened to have a hammer, and the problem looked like a nail.


You're not on NPM

Your docs said you could install via npm but I ran into an error. I checked the site and you don't show up.
https://www.npmjs.com/search?q=SuggestBox
What am I missing?

well, it's not an npm package. Sorry if docs are misleading.
you should run '''npm install''', if you cloned this repo, using  '''git clone'''
I recommend you to use bower

If you have bower then you should use:
bower install angular-suggestbox
If not, you can just clone the repo like:
git clone https://github.com/Ascarbek/suggestbox
or download as zip, using github's special button for that.
In releases I usually include dist folder where you can find all files you need to use.


cob_driver: 0.7.5-3 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository cob_driver to 0.7.5-3:

upstream repository: https://github.com/ipa320/cob_driver.git
release repository: https://github.com/ipa320/cob_driver-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.10.3
previous version for package: 0.7.4-1

cob_base_drive_chain
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

cob_bms_driver
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

cob_canopen_motor

No changes

cob_driver

No changes

cob_elmo_homing

No changes

cob_generic_can

No changes

cob_light
* Merge pull request #419 <https://github.com/ipa320/cob_driver/issues/419> from benmaidel/fix/topic_multi_color
  [cob_light] fix topic multicolor support
* fix topic multicolor support
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Benjamin Maidel, Felix Messmer, fmessmer

cob_mimic
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

cob_phidget_em_state

No changes

cob_phidget_power_state

No changes

cob_phidgets
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

cob_relayboard
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

cob_scan_unifier
* Merge pull request #420 <https://github.com/ipa320/cob_driver/issues/420> from lindemeier/feature/improve_performance
  performance boost
* performance boost
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer, tsl

cob_sick_lms1xx
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

cob_sick_s300
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

cob_sound
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

cob_undercarriage_ctrl
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

cob_utilities

No changes

cob_voltage_control
* Merge pull request #418 <https://github.com/ipa320/cob_driver/issues/418> from fmessmer/fix_catkin_lint
  fix catkin_lint
* fix catkin_lint
* Contributors: Felix Messmer, fmessmer

laser_scan_densifier

No changes

We are preparing for the final kinetic sync. All releases should be prepared by April 20th and we will have a long soak time until April 26th. Please make sure to test from the testing repository to make sure that there's no regressions after this is released. You can get more info from this thread on ROS Discourse


Osramlightify color tempertaure inverted

Make sure you are running the latest version of Home Assistant before reporting an issue.
Home Assistant release (hass --version):
0.39.3
Python release (python3 --version):
Python 3.4.2
Component/platform:
osramlightify
Description of problem:
The scale when setting the color temperature with color_temp is inverted. E.g. 500 mired is bright white, and 154 mired is yellow. But it should be the other way around. See https://en.wikipedia.org/wiki/Mired
Expected:
The other way around. 154 mired should be bright white and 500 mired should me more yellowish.
Problem-relevant configuration.yaml entries and steps to reproduce:
- platform: osramlightify
  host: <IP_ADDRESS>

I raelized it when I wanted to use the fluxer. I had to invert it. Now it is:
- platform: flux
  lights:
    - light.beamer
  name: Fluxer
  start_time: '7:00'
  stop_time: '23:00'
  start_colortemp: 1900
  sunset_colortemp: 4000
  stop_colortemp: 6000
  brightness: 200
  disable_brightness_adjust: True
  mode: mired

and it works as expected. White in the morning, and yellowish in the evening.
Additional info:
I might find the time to fix it myself, but I wanted to drop it here anyways.

Thank you,  @deisi
Now I understand why my fluxer behaved strangely.
It would be great if you could fix it!

This seems like the LIFX issue that was fixed in #6087.
I do not have Osram lights but I ported the fix anyway: https://github.com/amelchio/home-assistant/tree/osramlightify-colortemp
If someone tests that it works, I can submit it for inclusion.

@amelchio It is working. Now 500 mired is yellow and 154 is white. So plz make a pr ready for merge.

I have noticed the same inverted color temperature behaviour with the LimitlessLED component. Is it possible to apply the same fix to that as well or should I start another thread?

@billybigbass Definitely open a new issue.


diffLines doesn't support ignoring whitespace or leading/following spaces?

I feel like I'm missing something, because it seems like such an obvious feature to include, but there doesn't appear to be a version of diffLines that ignores whitespace and/or leading/following spaces.

No one has reported the need for this yet. Glad to accept a PR if you have such a need!

Sure, I'll see when I can get around to it. :smile:

Fixed by #47


Update tutorial.step_18.html

Angular-material redesign of socially app

Amazing!
will go through it soon and deploy


Is there any way to add array item as [] rather than collection of -

I need to convert JSON array to flow-style array element.
Input json:
{ name: 'bundle', x: ['one', 'two', 'three'] }

Output YAML;
name: 'bundle'
x:
- one
- two
- three

Expected:
name: 'bundle'
x: [one, two, three]

I need to convert JSON array to flow-style array element.

https://github.com/nodeca/js-yaml#safedump-object---options-
try { flowLevel: 1 }, no more options about that.


Messages are not delivered to one of subscribers

Few days ago one of client apps using Ruby client stopped seeing new messages on its subscription. E.g. messages got published fine both by other clients and this one. According to Nats streaming stats (available on port 8222) everything looks fine. However one of Ruby clients does not receive it, like there were no incoming messages.
I've suspected RAFT logs might be corrupt, so I've reinstalled NATS streaming from scratch. But the error persists.
I'm using a cluster of NATS Streaming servers of the latest version (0.22.1). Using latest version of Ruby client (0.2.2). Subscribe to messages from specified date in the past. Subscription is not durable. Tried enabling "deliver_all_available" option, but still got no messages.
Any ideas what should I do? How to debug the issue?

I'm closing the issue. Turned out the bug was on my side.


Is it possible to set the bookmarks bar to display in four or more columns?

For users with many bookmarks, a three-column display is obviously insufficient and may disrupt the page.

Wow, that is a lot of bookmarks indeed! Currently there is no way of customizing the column count without modifying the source directly. Have been hassling with health related stuff for a while, sorry for responding late. I will try to get to it, it would be a nice addition!


ENH: output format geoparquet

This came up in the STAC meeting today. Currently it looks like the supported output formats (available with the f query param) are: 'geojson', 'html', 'json', 'csv', 'geojsonseq', 'ndjson'. I got that list by naively trying https://firenrt.delta-backend.com/collections/public.eis_fire_lf_perimeter_nrt/items?f=geoparquet
It would be neat to add 'geoparquet' as an option.
Not sure if this is the right place to capture the request so feel free to close, just wanted to increase the visibility of that conversation.

thanks for starting the discussion @jsignell 🙏
This is definitely something we could/should support.

yeah exactly! I think /items under just a different f query param. stac-geoparquet will definitely be helpful, but might need some alterations since these aren't STAC objects exactly.

@kylebarron yes, I think it makes senses to enable GeoParquet output for Items first. (we might want collections later but it will be less useful).

It might be easiest to use the existing https://github.com/stac-utils/stac-geoparquet library for this

stac-geoparquet, depends on pandas and geopandas (thus shapely), this would be quite heavy dependencies just to add an output format. I was hopping for a more lightweight solution 🙏

🤯 I don't think this feature is extremely needed right now so we can wait especially if this can help for your library to be ready :-)
FYI: we already have pyproj dependency (via morecantile)
Note: we could still add an heavy dependency and make the whole thing optional if this is really something user/customers want

Definitely agree with making it an optional dependency if we add it.

Yes, definitely would want to add this as optional dependency. We would probably want to implement this starting from a query like SELECT column_a, column_b, ST_ASWKB(geometry_column) FROM mytable rather than going through the geojson that we create and build up the geoparquet from those results which would eliminate the need for shapely or the like.

I'm not too familiar with the tipg internals but happy to help implement this


Bug: GRPC sometimes errors

Describe the bug
Some users are seeing this error:
Failed to load Inbox: Call to function 'XMTP.listConversations' has been rejected.→ Caused by: io.grpc.StatusException: UNAVAILABLE: Unable to resolve host production.xmtp.network

Expected behavior
No response
Steps to reproduce the bug
No response

That's starting to look more like an issue on the nodes. Are you able to reproduce the same error when using grpc.production.xmtp.network?

I did a little digging in to the metrics from AWS, and there are some spikes of errors coming from the load balancer. These are the ELB_5XXs metric over the past week.

Amazon describes that metric like this, which rules out issues on our nodes. It seems like the Amazon Managed LB is failing on its own (maybe through slow autoscaling or an Amazon incident)

The number of HTTP 5XX server error codes that originate from the load balancer. This count does not include any response codes generated by the targets.

The metrics from the node are healthy, but that would be consistent with a load balancer error since the requests never even would make it to the node.

The android repo does use grpc.production.xmtp.network there is nothing in the android repo that actually chomps the grpc portion of the host.... I looked a little into the grpc android library and I think it's trying to treat grpc as the scheme? When I add https it fails with a no scheme error for https.
Fwiw I was able to reproduce this once. But after restarting my computer and updating android studio I haven't been able to. We do know atleast 3 others are experiencing this on there actual device though.


Allow execution of arbitrary commands

For scripting purposes, let's allow execution of arbirtary commands.

:+1:

Is there anything I can do to move this along @cbrunsdon or @jhawthorn?

Looks good by me :+1:


lib*-lib-.o + lib-shlib-*.o make lesser sense

I understand the formal sentiment behind idea to compile object files that go to shared and static libraries differently, but on practical level it's probably an overkill. For following reason. Contemporary systems are likely to employ [variation of] ASLR. Secondly, OpenSSL can get embedded into another shared object, most notably JNI. In both cases it's desired that object modules in static library are compiled with position-independent flag, i.e. as if they were compiled for usage in shared library. In other words it hardly makes sense to compile anything else but position-independent code. But in such case flags would be same for lib*-lib-*.o and lib*-shlib-*.o, and compiling pair would be just waste of resources and time. Of course majority hardly notices the waste, because majority is using fancy processors and difference in build time is tolerable for human observer. (I mean humans are slow.) But it's a waste nevertheless. And it gets really "painful" on less fancy systems, such as those in IoT segment. Either way, bottom line is that original single-object approach arguably was balanced enough. I mean by default build single position-independent object and put it into both static and dynamic libraries. And those who actually require non-position-independent code in static library can achieve the goal with dedicated config.

Just in case. I recall a remark elsewhere about -static link being problematic, i.e. linking application code with -static flag. If this was contributing factor to the lib-shlib split, then one should recognize that problem with -static link runs deeper than just compiler flags. glibc threads don't work at all, binary crashes at startup. Name resolution doesn't work, binary crashes in attempt to resolve name to address. It should be noted though that these are apparently glibc-specific problems, because for example application statically linked with musl doesn't exhibit such problems...

If there is something that is sensible to do differently when putting together static library it's ... engines. It makes sense to compile engines and engine support as if whole thing was configured with no-shared and put these object files into static library. So that application [consciously] linked with static library (or shared library that wraps OpenSSL) would be self-sufficient. Another option is to compile static engine support as if whole thing was configured with no-engine.
On related note. e_padlock is "bit rotten" in shared build. Since -DPADLOCK_ASM is not passed bind_engine does nothing.

@dot-asm, if you have a look at #7159, you'll see that the main driver behind this change isn't quite that, but rather about shared source between different products, and where the different products come with different build settings (inclusion directories, for example).  This may not be terribly important now, but we have a FIPS-related future where this may/will make more sense.

However, we may use the same mechanisms to resolve some things about simultaneous shared and static builds that we have ignored.  I haven't dived into that very much yet, thanks for the reminder.

The focus is rather on lib-shlib, i.e. second prefix, than on the first prefix, such as libcrypto. I appreciate the possibility of same input file being compiled for different contexts, it's just that as it stands now lib-shlib split makes lesser sense. And I'm inclined to assert that just-double-everything-with-lib-shlib won't ever make much sense. It should rather be more nuanced, with just a small subset of object files (at least engines and engines initialization) being compiled differently for static and shared libraries.
On potentially related note. After AIX arrangement, the lib{crypto|sll}_a.a thing, I wonder if it would be appropriate to extend this approach to all platforms (including Windows). I.e. to produce lib{crypto|sll}_a.a and tell users to -lcrypto_a (modulo something else for Windows), if they want to link statically. How is it potentially related? Because first prefix would be different, e.g. libcrypto vs. libcrypto_a;-)

It's true that we could complicate the configuration code to check if the lib and shlib configuration settings differ at all, and only apply the prefix if they do, per object file.  It's gonna require some brain twisting, but should be doable.
As for a rename of the static library, I've been thinking for quite a while that we should do that, because it will give better consistency across all supported platforms.  I would avoid _a and rather go with something like -static or _static (I actually think there's even a precedent for such a naming scheme...  could that even be on Linux?  My memory is a little vague on that subject)

Here is another related "bit rot". Original idea behind subdir targets was to facilitate "rapid prototyping". I mean you edit a file, but don't want to re-link whole thing just yet, so you run for example make crypto/bn. Now make crypto/bn will rebuild libcrypto-lib-*, but not libcrypto-shlib-*. Well, it doesn't really matter [now], because as already established shlib is just redux of lib. But things turn unexpected/weird on Windows. Provided that you configured shared build, nmake compiles only libcrypto-shlib-*, no libcrypto-lib-*. But if you run nmake crypto\bn it builds libcrypto-lib-*...

That's interesting. I will have a look shortly

That's interesting. I will have a look shortly ... #7452

Now if I run make crypto/bn on Linux in no-shared config I get make: *** No rule to make target 'crypto/bn/libcrypto-shlib-bn_add.o', needed by 'crypto/bn'.  Stop. Well, there is certain ambiguity in the assertion, as I haven't actually tested shared config, or looked how things are on Windows now. But either way, I'm still inclined to assert that just-double-everything-with-lib-shlib won't ever make much sense. As already said, "it should rather be more nuanced, with just a small subset of object files (at least engines and engines initialization) being compiled differently for static and shared libraries."

I just discovered that you get a related issue if you nmake crypto with 1.1.0 and 1.1.1, because crypto\dllmain.obj is missing among the dependencies for crypto\ when configured shared.  That's not bad in the same way as missing rules, but still...


Implement "direct NVDIMM" mode bypassing LVM

Driver could be made capable to operate in 2 modes:

LVM-enabled mode (current main mode being developed)
Direct-NVDIMM mode (was one of prototype models before LVM mode was added)
Direct-NVDIMM code stubs have been kept in code just in case if we decide to keep that path.
In that mode, driver will create NVDIMM namespaces and provision block devices based on those, directly.

Cons of direct mode:

leaves fragmentation risk wide open

Pros of direct mode:

LVM layer is bypassed, i.e. less possible error paths, (see #25), simplified start-up
runtime dependency on LVM components will not be needed
Performance hit by LVM avoided (hit is not expected anywhere close to serious though)
namespacemode (see #22, #30) can be set per-volume which is not possible in LVM-mode

We could say in doc: select LVM mode if you are concerned about fragmentation (dynamic create/delete of different volume sizes expected), but you can select direct-nvdimm mode if you only allocate once i.e. fragmented states are not expected.

resolved via merge of direct-nvdimm mode in #133


Attachements plugin with cds typer throws error

When trying to integrate the attachments plugin into our CAP application we get the following errors:

`@cds-models/application/index.ts:414:46 - error TS2344: Type 'Attachments' does not satisfy the constraint 'readonly any[]'.
Type 'Attachments' is missing the following properties from type 'readonly any[]': length, concat, join, slice, and 20 more.
414         attachments?: __.Composition.of.many<_.Attachments>;

and:

@cds-models/index.ts:81:78 - error TS2552: Cannot find name '_AttachmentAspect'. Did you mean '_AttachmentsAspect'?
81 export class Attachments extends _managedAspect(_cuidAspect(_MediaDataAspect(_AttachmentAspect(__.Entity)))) {static drafts: typeof Attachments}`

The project is implemented with typescript and is using cds-typer plugin from sap (https://github.com/cap-js/cds-typer) to define types. After we integrated the attachements plugin via npm add @cap-js/attachments, then integrated it into our model and tryed to deploye it to our sqlite data base. We get the error written above.
We are using the following versions:

cds: 7.9.2
cds-dk: 7.9.3
@cap-js/cds-typer: 0.21.2
@cap-js/attachements: 1.1.1

Hi @Beniseeger,
Can you please share a sample or steps to reproduce the issue?
Best Regards,
Muskan

Hi @Beniseeger,
I have tried replicating the error mentioned in the issue description using the bookshop sample application. After adding the cds-typer and attachments plugin as instructed, I was unable to reproduce the error that was reported.
I have attached the schema.cds and the generated index.ts for reference.
schema.cds.txt
index.ts.txt
Since the error could not be replicated on my end, I am closing this issue for now. Please feel free to provide additional context or steps if the error is still occurring.
Best Regards,
Jeevitha


Add merchants to list

Will upload with new merchants as soon as they are submitted on Reddit.
Closes #472
Closes #473
Closes #483
Closes #485

pls change url to https://www.megachange.is

I don't see Monero listed anywhere on the site.

Monero will be added in 1-2 weeks, our developers in the process of implementing it now
We need to get listed before or at the same time when monero become available for trading on our website.

We need to get listed before or at the same time

I think it would be confusing to users if it was added before.

we don't think so
as monero will be added shortly as i mentioned

I agree, listing this service now would be misleading. referencing #466 comment

Hello,
Could you please explain how long it will take to add the ChangeNow exchanger to the list?

@ChangeNOW-Exchange This PR got a bit delayed since there were a lot of requests from various merchants during last month. Btw I'm not going to add any new one.
@luigi1111  this is good to go

Thank you for your answers.
When are you planning to merge this pull requests?

Since this PR was open for a while, I don't think it will take long (I'm not the one who merge btw)

Thanks for the info, we will be expected.


SkillTreeGUINew.lua causing crash in latest update

As of the latest update, PD2 will crash if GoonMod is enable and you try going into your skills.

We still have inventory problem, well it has been 3 months since it's last update. Probably GoonMOD will be not gonna updated on future. Rest in pepperoni.

Heya, I messed around with the files and it seems not to crash if you remove the files SkillTreeGUI.lua and SkillTreeGUINew.lua from your mod.
Please be aware that I don't know which other potential repercussions it can have on the game while doing this. I suppose the files were modified by JamesWilko for the only purpose of the post-Infamy skill tree outliner, so it should be OK as long as you try not to activate this feature.
About the inventory problem, I think we won't be able to dodge it the same way, unfortunately.

GoonMod overrides Payday 2 functions in order to implement its own features, so when the core Payday 2 code is updated and depends on these functions to be updated as well, the game will crash. However, sometimes GoonMod's override doesn't cause the game to crash and the new features will simply be missing. You could consider this lucky, but this also means many parts of GoonMod haven't been updated in a LONG time because, since they didn't cause a crash, there wasn't much reason to update them. This might not become apparent until one of these features is actually needed in order for the game to work properly and thus causes a crash.
For now I would just recommend removing the mod completely, or commenting out the lines in goonbase.lua that tell the game to use GoonMod's files instead of Payday's own files.

@DarKobalt Thanks for the comment but there's no need to remove/rename SkillTreeGUI.lua. The one causing the crash is SkillTreeGUINew.lua thus that's the only one that needs to be removed/renamed. Doesn't seem to have any potential repercussions, at least not that I've noticed since I removed it when I posted this Issue.


Possibility to display line endings in text area

VSCode Version: 1.5
OS Version: Win 7 x64

At the moment there is only the little statusbar menu which display/change the line ending if the actual file. But sometimes it would be great to see the line endings directly in every line (see atom f.e.) especially when there are mixed line endings (not good, but this happens from time to time).
Greetings
Tonka

Same thing as #24625

@alexandrudima Any progress on this front? Keep having to crack open notepad++ or cat -e to see newlines, which is pretty annoying.

Any update on this ?
And it seems that this issue is looping since there are 2 issues that refer to each other (#24625 <> #12223)

@alexandrudima Any update on this ?

Add "editor.renderWhitespace": "all" to settings.json
It renders most whitespaces, but not newline characters.
It would be very helpful to render the newline characters, with distinct renderings for CR vs LF. This is especially helpful if you have a mixed line ending file and want to see where the issues are.
@bpasero Is there any chance, someone picks this up and completes this?

@sajidali I agree.  I keep having to copy my code into NotePad ++ to see special characters like new lines.  I would like to see editor.renderControlCharacters so it behaves.

Any movement on this case for rendering the line endings?

This issue has been open for almost a year, and seems like very "low hanging fruit" which would benefit a lot of users. Is the team considering implementing it?

@adamralph: perhaps this one is kept unresolved on purpose... there should at least be one or two things that Atom is better at :wink:

@alexandrudima Could you kindly retriage this one? A highly requested feature.

In the mean time, does anyone know of any extensions that implement this feature? Doesn't look like the team are very excited to add this feature anytime soon.

willing to send the person who adds this a snickers bar.
no bamboozles.
I mean, thats a tasty offer

+1, very needed feature!

Gentle reminder that lots of people are eagerly awaiting this.

+1, #26646
If Render Control Characters does not show Newline characters — what control characters are rendered when enabling this setting?
Nearly every other editor I have used with support for rendering control characters does display newlines characters when enabled.

I am desperately awaiting this feature as well.
In the meantime, I made an extension to show line endings, in case anyone wants this feature now : code-eol

@sohamkamani that's pretty cool!
Although I notice that it displays the same character ⬎ regardless of whether the EOL character is LF, CR, or CRLF. The reason I want this feature is to be able to distinguish them. If that's something you'd be willing to implement, I humbly suggest:

LF = ↓ (DOWNWARDS ARROW)
CR = ← (LEFTWARDS ARROW)
CRLF = ↵ (DOWNWARDS ARROW WITH CORNER LEFTWARDS)

These are the characters I've seen used in some other apps. The nice thing about them is that they represent the physical movement of the typewriter head when performing a line feed (LF) or carriage return (CR) and they are additive (downwards arrow + leftwards arrow = downards arrow with corner leftwards).

+1

Good work, just one more request.
@sohamkamani as @adamralph said, we need distinct rendering for each of line break characters.
I have already asked for the same in the above comments, that distinct displayed character is need to notice the mixed line ending issues.
It would be great if the official team can provide this basic feature.

This missing feature is the sole reason I still have Notepad++ installed.

Thanks @adamralph for the suggestion. Just to give you and @sajidali  an update, I have implemented and updated the code-eol extension according to your suggestion.
If there are any more suggestions, please feel free to file them under the extensions repo, since we wouldn't want to pollute this thread.
Of course, what I really hope for, is for this extension to be made obsolete by vs-code having this feature natively! 😄

@sohamkamani that is all kinds of awesome. Thanks!
https://twitter.com/adamralph/status/966378278020894720

When opening a file with mixed line endings, vscode is still showing it as LF or CRLF in the status bar.
After saving, it fixes it, but I think it should show, for example, LF (mixed) (maybe in red) until the file is modified.
This is important so that you know the file still has mixed EOLs, and you might want to fix that.
To reproduce: printf "lf\ncrlf\r\ncr\rnone" > mixed-eol.txt && code mixed-eol.txt
If you open this file in Notepad++ and enable visible EOL, you can see it has mixed endings.
By the way, this might by why code-eol isn't showing them as different EOLs.
Should I post this in a different issue, as a bug?

This missing feature is the sole reason I still have Notepad++ installed.

I have a second reason: Notepad++ opens instantly.

I am now working in a project where I need to have mixed line endings (some files require LF only), and this is pretty difficult to deal with in VScode. :-S  Komodo Edit has the ability to set CR/LF vs LF on specific files.  That would be a fantastic addition, for those that need it.

In vscode version 1.36.1, it seems like the problem still exists. Is there a plan to improve this feature?

It has been almost 4 years since this feature request. I would never imagine this would be such a hard feature to implement. Are there any updates or a timeline for this?

Any plans to implement this?

Please, please address this!  Thank you

To everyone subscribed to this: You'll likely also want to upvote my feature request here, before it's closed by bot: https://github.com/microsoft/vscode/issues/119025
The reason most people want to see line endings is because they want to detect and fix mixed endings, right?
For this, I think that it's more helpful to easily know when a file has any mixed line endings (without having to look at it line by line), with option to auto-fix it on save, than to know where a file has a specific line ending (and having to manually fix it).
Knowing "where" is useful mostly in binary files, but you can currently use hex editor if you're dealing with those. It's also useful if you're working on a text file where you want mixed endings on purpose, like testing how compilers and tools deal with such files, etc., but this is an infrequent corner case.

The reason most people want to see line endings is because they want to detect and fix mixed endings, right?

Can't speak for most people, but mixed line endings aren't always a problem that need fixing. Or at least that's not why I subscribed to this issue. I was writing a test suite that did some stuff with whitespace and wanted to be able to read the whitespace in my source code.

Knowing "where" is useful mostly in binary files, but you can currently use hex editor if you're dealing with those.

"Use " is basically a solution to every feature request on this tracker, but probably not a very satisfying one to the people subscribed.

@masaeedu Sure, I recognize this feature is important too. In fact, the corner case I mentioned is one I'm having to deal with myself, so I'd like to have this as well.
But since the mixed lines case is closely related to this, I thought I'd notify subscribers of a better solution in fixing that particular problem (as complementary to this, not replacement). I wanted to draw attention to that feature request mostly so it doesn't die, not because it would solve the same problems as this.

I need this feature.
I'm manually copying text from a PDF into MARKDOWN, and I need to delete unwanted new lines.
It can't be done automatically, because some new lines have to stay.
If I could see where the newline mark is, it would make my job a lot easier.
Because I have to distinguish between wrapping a line in the editor and a newline.
Disabling line wrapping is also inconvenient, because I have to move the text sideways...

@lukaszpolowczyk out of curiosity, what's stopping you from deleting the new line characters?
You should be able to go to the end of the line or beginning of the next line and delete them.  Just because you can't see the CRLF or LF doesn't mean you can't delete it.

@DougPlumley I'm deleting, but first I need to guess where the newline character is and where the wrapped line is.
In the PDF file, it goes something like this:

Lorem Ipsum is simply dummy text of the printing and typesetting
industry. Lorem Ipsum has been the industry's standard dummy
text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book.
It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged.

And the editor additionally wraps some lines.
The result is that I have two newline characters to delete, one wrap that I don't need to delete, and one newline character that should stay.
In the editor I cannot see where the newline is and where the wrap is.
Do you understand now? :D

🧐

@DougPlumley I'm deleting, but first I need to guess where the newline character is and where the wrapped line is.
In the PDF file, it goes something like this:

Lorem Ipsum is simply dummy text of the printing and typesetting
industry. Lorem Ipsum has been the industry's standard dummy
text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book.
It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged.

And the editor additionally wraps some lines.
The result is that I have two newline characters to delete, two wrappers that I don't need to delete, and one newline character that should stay.
In the editor I cannot see where the newline is and where the wrap is.
Do you understand now? :D

Yes and no I suppose, because you can show line numbers and anything that wraps is going to have the same line number.
You can also select a single line feed and Control + D (Command + D on macOS) to select the next word
You can also use search in the file to find, find/replace new lines.

We're still waiting...

As someone that helps new programmers learn PHP, the fact that there isn't a setting for showing newlines is really unfortunate... for example...
<?php require('thing.php'); ?>
<?php
// code and whatever here
header('Location: /index.php');

this header line does not work because line 1 outputs a newline, but students do not understand that there is a newline character at the end of line 1 that is being output to the response body.
Whereas in Notepad++, the settings can make this mistake perfectly obvious:

Would love this feature as well.
There is clearly interest for it.
Hope we get a kind soul interested in this.
Pretty please. <3

I want it, but just for diff comparison purpose, because very often I don't see a change and usually it's because of line endings.

We're coming up on 8 years since this was proposed. It'd be wild if this bug went a full decade without attention. And it is a bug. If my config is telling me it's rendering "all" whitespace, but there's exceptions, well then that's not "all" whitespace is it? I can't imagine it'd be super difficult to get the settings to stop lying to users. So alternative solution: change the setting to say "most", drop the "all", and just accept that truly rendering all whitespace is just beyond the technical abilities of VSC.

So even Github CoPilot, with the @vscode chat participant, will hallunicate me this:

So this does not work unfortunately:
{
    "editor.renderWhitespace": "all",
    "editor.renderControlCharacters": true
}


AAD authentication fails with az network bastion ssh

If the issue is to do with Azure CLI 2.0 in-particular, create an issue here at Azure/azure-cli

Following the instructions here:
https://docs.microsoft.com/en-us/azure/bastion/connect-native-client-windows
Extension name (the extension in question)
network bastion ssh
Description of issue (in as much detail as possible)
Certificate based authentication fails with bastion to linux vm.
Command
az network bastion ssh --name "" --resource-group "" --target-resource-id "" --auth-type AAD
Errors:
Command group 'network bastion' is in preview and under development. Reference and support levels: https://aka.ms/CLI_refstatus
The command failed with an unexpected error. Here is the traceback:
_check_or_create_public_private_files() missing 1 required positional argument: 'credentials_folder'
Traceback (most recent call last):
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 657, in execute
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 720, in _run_jobs_serially
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 691, in _run_job
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 328, in call
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/network/custom.py", line 7801, in ssh_bastion_host
TypeError: _check_or_create_public_private_files() missing 1 required positional argument: 'credentials_folder'
To open an issue, please run: 'az feedback'

network

this is an issue with changes to ssh and az beyond versions 2.26 and 0.1.5 for ssh.

Any News or temporal horizon when a Solution might rise up on this Issue?

this is tagged in the wrong repo. THis has been fixed in azure-cli as this is not part of cli-extensions yet.

@aavalang
You sure that this is a duplicate? Stacktrace looks completely different to me.
I'm getting exact same Error with CLI 2.31.0 and ssh extension 1.0.0

@alex200789 yes, fix for the mentioned issue is for this as well.
This is fixed with 2.32 I believe.

@perttulaurila Apologies for the late reply. Could you please use the most recent version of AzCLI and test again ? Please let me know if you have any further questions on this. Awaiting your reply.

az version
{
  "azure-cli": "2.34.1",
  "azure-cli-core": "2.34.1",
  "azure-cli-telemetry": "1.0.6",
  "extensions": {
    "ssh": "1.0.0"
  }
}

/var/folders/7t/p2jt5jv57r3f0_2c0l__09vc0000gn/T/aadsshcerttks350cb/id_rsa.pub-aadcert.pub

Exception in thread Thread-1 (_start_tunnel):
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 1009, in _bootstrap_inner
    self.run()
  File<EMAIL_ADDRESS>line 946, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/local/Cellar/azure-cli/2.34.1/libexec/lib/python3.10/site-packages/azure/cli/command_modules/network/custom.py", line 8232, in _start_tunnel
    tunnel_server.start_server()
  File "/usr/local/Cellar/azure-cli/2.34.1/libexec/lib/python3.10/site-packages/azure/cli/command_modules/network/tunnel.py", line 184, in start_server
    self._listen()
  File "/usr/local/Cellar/azure-cli/2.34.1/libexec/lib/python3.10/site-packages/azure/cli/command_modules/network/tunnel.py", line 117, in _listen
    auth_token = self._get_auth_token()
  File "/usr/local/Cellar/azure-cli/2.34.1/libexec/lib/python3.10/site-packages/azure/cli/command_modules/network/tunnel.py", line 104, in _get_auth_token
    raise exp
msrestazure.azure_exceptions.CloudError: Tunneling is disabled

@davidkarlsen  if native client support is not enabled on your Bastion host, you'll receive the error "msrestazure.azure_exceptions.CloudError: Tunneling is disabled". More Info here.
Similar issue is discussed here https://github.com/Azure/azure-cli/issues/20735

@navba-MSFT I had the wrong SKU, it works fine now from a mac box, when I have the Standard SKU for the bastion!

It’s not tied to sku alone, there’s also a feature you have to enable on the bastion in the configuration page
Get Outlook for iOShttps://aka.ms/o0ukef

From: David J. M. Karlsen @.>
Sent: Wednesday, March 23, 2022 5:48:42 AM
To: Azure/azure-cli-extensions @.>
Cc: Aakash Radhakrishnan @.>; Mention @.>
Subject: Re: [Azure/azure-cli-extensions] AAD authentication fails with az network bastion ssh (Issue #4148)
@navba-MSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnavba-MSFT&data=04|01|Aakash.Valangaiman%40microsoft.com|0e78c6b946584d4624fa08da0ccb768f|72f988bf86f141af91ab2d7cd011db47|1|0|637836365267642003|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=pJwMWb8sqOGC5m%2F9GTajMTJ5bYWSa603GoVs7xizRo0%3D&reserved=0 I had the wrong SKU, it works fine now from a mac box, when I have the Standard SKU for the bastion!
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli-extensions%2Fissues%2F4148%23issuecomment-1076337404&data=04|01|Aakash.Valangaiman%40microsoft.com|0e78c6b946584d4624fa08da0ccb768f|72f988bf86f141af91ab2d7cd011db47|1|0|637836365267642003|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=yzD2pRN1v9zH%2BfPFu3ux2jMRawHnzrCkXOLDW%2BqKACo%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANOED6A5VYG47K2B2KYDFUTVBMHKVANCNFSM5ITFGJ7Q&data=04|01|Aakash.Valangaiman%40microsoft.com|0e78c6b946584d4624fa08da0ccb768f|72f988bf86f141af91ab2d7cd011db47|1|0|637836365267642003|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=YNf2vMl2DT1skUHaNqDh584%2FdLvk3zvtHtT%2FalmZFW8%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

It’s not tied to sku alone, there’s also a feature you have to enable on the bastion in the configuration page
Get Outlook for iOShttps://aka.ms/o0ukef

From: David J. M. Karlsen @.>
Sent: Wednesday, March 23, 2022 5:48:42 AM
To: Azure/azure-cli-extensions @.>
Cc: Aakash Radhakrishnan @.>; Mention @.>
Subject: Re: [Azure/azure-cli-extensions] AAD authentication fails with az network bastion ssh (Issue #4148)
@navba-MSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnavba-MSFT&data=04|01|Aakash.Valangaiman%40microsoft.com|0e78c6b946584d4624fa08da0ccb768f|72f988bf86f141af91ab2d7cd011db47|1|0|637836365267642003|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=pJwMWb8sqOGC5m%2F9GTajMTJ5bYWSa603GoVs7xizRo0%3D&reserved=0 I had the wrong SKU, it works fine now from a mac box, when I have the Standard SKU for the bastion!
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli-extensions%2Fissues%2F4148%23issuecomment-1076337404&data=04|01|Aakash.Valangaiman%40microsoft.com|0e78c6b946584d4624fa08da0ccb768f|72f988bf86f141af91ab2d7cd011db47|1|0|637836365267642003|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=yzD2pRN1v9zH%2BfPFu3ux2jMRawHnzrCkXOLDW%2BqKACo%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANOED6A5VYG47K2B2KYDFUTVBMHKVANCNFSM5ITFGJ7Q&data=04|01|Aakash.Valangaiman%40microsoft.com|0e78c6b946584d4624fa08da0ccb768f|72f988bf86f141af91ab2d7cd011db47|1|0|637836365267642003|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=YNf2vMl2DT1skUHaNqDh584%2FdLvk3zvtHtT%2FalmZFW8%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Yes. But in order to make that available you need the right sku

Please make sure the error description specifies both requirements + a help link?
Requirements are:

Standard SKU
checkbox for native support


Images in lists

To convert a list with image, I need to put the raw Markdown in the list item:
{ ul: [
     '![](url "title") bullet 1',
     'bullet 2',
     'bullet 3' ]
 }

It would be better to have the json image syntax:
{ ul: [
     [{img: {source: url, title: "title"}}, "bullet 1],
     'bullet 2',
     'bullet 3' ]
 }

That is currently not supported.

I know that is not currently supported, that is why I opened this issue.
It's a standard in markdown and possible in json.
I think it could be a good enhancement to your lib

You are right. Contributions are welcome.

On 25 Mar 2020, at 07:50, Cédric Delpoux<EMAIL_ADDRESS>wrote:
I know that is not currently supported, that is why I opened this issue.
It's a standard in markdown and possible in json.
I think it could be a good enhancement to your lib
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/IonicaBizau/json2md/issues/55#issuecomment-603820828, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAV3J4YS5VSOTR5YQZ2PTSTRJH42NANCNFSM4KQ4O52A.


sorter: fix the bug that value can be corrupted when accessing sorter

What problem does this PR solve?
Issue Number: close https://github.com/pingcap/tiflow/issues/6367
Issue Number: close https://github.com/pingcap/tiflow/issues/10853
What is changed and how it works?
This PR is extracted from #10899 , which fixes the bug that value can be corrupted when accessing sorter.
It's hard to construct a test case for the change because the value corruption can't be reproduced stable. So we just ignore it.
Check List
Tests

No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/cherry-pick release-6.5-fips

/cherry-pick feature/release-6.5-fips

/cherry-pick release-7.1


Small mistake

Wiki accidentlly says "When running deltaBS, the option -hd specifies the directory that HMMER3 has been installed in and -hp specifies the directory that the Pfam HMMs have been dowloaded to."
Instead of "When running deltaBS, the option -hp specifies the directory that HMMER3 has been installed in and -hd specifies the directory that the Pfam HMMs have been dowloaded to."

Thanks for pointing this out, I've fixed it now.


[FIX] Crash when compound kinematic body enters trigger

I can't say why but if I update compounds after kinematics, the crash goes away.
Fixes #3127
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

Any idea if this change might alter physics behavior?

closing as per @willeastcott


Remove yiisoft/cache dependecy

Seems, Yii Cache don't use in this package.

https://github.com/yiisoft/yii-debug/blob/master/config/params.php#L58

https://github.com/yiisoft/yii-debug/blob/master/config/params.php#L58

This is config. The package itself does not require Yii Cache.
See docs:

Package must not have dependencies in require section of composer.json that are used in config only.


Doesn't work when 2FA is active

When I try and use the Open Homebridge UI option and specify my credentials, I get this error...
Unsuccessful login: Server is unavailable or login/password is incorrect. {"statusCode":412,"message":"2FA Code Required"}
My assumption here is that this is because I have 2FA on my account. Is there a way around this?

@dartiss what 2nd factor do you use?
Can you please briefly describe our auth flow?

Hi,
I'm using the built-in 2FA which makes use of app authentication.
Within the Stream Deck app, I'm providing the server address, user name and password, as per my Homebridge account. That's when I get the above error.

I receive the same response when trying to log in, also when using 2FA via app authentication in Homebridge.

Can you point to the manual on how you configured 2FA in Homebridge?

In Homebridge, you go into User Accounts, then click on Enable 2FA. Then you scan the QRCode in your preferred Authenticator app. This being said, your plugin works ok when 2FA is off.
The only issue I get, is that when I change a switch state into Home or homebridge, it's not reflected in Stream Deck


Any chance to get a WP plugin, or at least a guide?

So the question says it all I think.
It would be great to get this as a WP plugin to place as a widget on post pages.
If no plans for that, can anyone tell me what to keep in mind to add this to WP myself? Possibly as a plugin, and the TOC as a widget.

This is my question exactly, too.  I'm really in need of a TOC like this in WP, but it's quite complicated for me to figure out exactly what code I need to place where.
Would really appreciate a WP plugin, or some help with how to add this to WP myself, as well.
Thanks!


Considerations for total processing time during CNI DEL (given that we have the controller)

If there's API server connectivity problems (or congestion that causes api server throttling), Whereabouts can sit for a long time during CNI DEL causing a pod deletion to take a while.
Could it be possible that we just set some kind of maximum run time for CNI DEL, and then just bail out if we reach it? And then let the controller kick and clean up the address.
I'm not sure what happens with the pod deletion event for the controller in times of api server wonkiness, so, that's another consideration.
#spitballing

AFAIU, there is already a timeout. It is defined here.
Now ... I just don't understand why have this specific value for deletes.
Furthermore, there's a timeout on the k8s client, which can be overriden from the configuration.
@dougbtv I would suggest whatever timeout we have for add / del to be configurable. One size fits all is a disaster waiting to happen for some scenarios.


Wargames no CD patch doesn't work

Hello, I copied the three files from Wargames-fix.zip file in my local folder where the WarGames.exe file is, but it keeps asking me the CD (which I have, but either my CD drive of the CD itself works 50% of the times...). I am attaching the log file here, I think the issue is
17688 09:50:19.784 Utils::WriteMemory::WriteMemory Failed verification for memory write!
17688 09:50:19.769 Starting DxWrapper v1.0.6334.21
17688 09:50:19.769 ASUSTeK COMPUTER INC. SABERTOOTH X79 Rev 1.xx (Desktop)
17688 09:50:19.769 NVIDIA GeForce GTX 1060 6GB
17688 09:50:19.769 Windows 10 64-bit (10.0.19041)
17688 09:50:19.769 "WarGames.exe" (PID:10972)
17688 09:50:19.784 Utils::WriteMemory::WriteMemory Failed verification for memory write!
17688 09:50:19.784 Disabling High DPI Scaling...
17688 09:50:19.784 Loaded library: user32.dll
17688 09:50:19.784 Loaded library: shcore.dll
17688 09:50:19.784 Loading 'ddraw.dll'...
17688 09:50:19.784 Hooking ddraw.dll APIs...
17688 09:50:19.784 Enabling ddraw wrapper
17688 09:50:19.784 Enabling d3d9 wrapper
17688 09:50:19.784 Loading 'd3d9.dll'...
17688 09:50:19.784 DxWrapper loaded!
17688 09:50:19.878 dd_DirectDrawEnumerateA
17688 09:50:19.878 Redirecting 'Direct3DCreate9' ...
17688 09:50:19.925 Creating device m_IDirect3D9Ex::m_IDirect3D9Ex(00A8DB28)
17688 09:50:19.925 dd_DirectDrawCreate
17688 09:50:19.925 dd_DirectDrawCreateEx
17688 09:50:19.925 Redirecting 'DirectDrawCreate' IID_IDirectDraw to --> 'Direct3DCreate9'
17688 09:50:19.925 Creating device m_IDirectDrawX::m_IDirectDrawX(009A3410) converting device from v1 to v9
17688 09:50:19.925 Redirecting 'Direct3DCreate9' ...
17688 09:50:19.925 Creating device m_IDirect3D9Ex::m_IDirect3D9Ex(00A8D888)
17688 09:50:20.065 Creating device m_IDirect3DDevice9Ex::InitDirect3DDevice(00A7F890)
17688 09:50:20.081 m_IDirectDrawX::~m_IDirectDrawX(009A3410) deleting device!
17688 09:50:20.097 m_IDirect3DDevice9Ex::~m_IDirect3DDevice9Ex(00A7F890) deleting device!
17688 09:50:20.097 m_IDirect3D9Ex::~m_IDirect3D9Ex(00A8D888) deleting device!
17688 09:50:20.097 m_IDirectDrawSurfaceX::CleanupSharedEmulatedMemory Deleting 0 emulated surfaces!
17688 09:50:20.097 Redirecting 'DirectDrawCreate' IID_IDirectDraw to --> 'Direct3DCreate9'
17688 09:50:20.097 Creating device m_IDirectDrawX::m_IDirectDrawX(0790DA78) converting device from v1 to v9
17688 09:50:20.097 Redirecting 'Direct3DCreate9' ...
17688 09:50:20.097 Creating device m_IDirect3D9Ex::m_IDirect3D9Ex(079454E8)
17688 09:50:20.175 Creating device m_IDirect3DDevice9Ex::InitDirect3DDevice(00A7FB60)
17688 09:50:20.175 m_IDirectDrawX::~m_IDirectDrawX(0790DA78) deleting device!
17688 09:50:20.190 m_IDirect3DDevice9Ex::~m_IDirect3DDevice9Ex(00A7FB60) deleting device!
17688 09:50:20.190 m_IDirect3D9Ex::~m_IDirect3D9Ex(079454E8) deleting device!
17688 09:50:20.190 m_IDirectDrawSurfaceX::CleanupSharedEmulatedMemory Deleting 0 emulated surfaces!
17688 09:50:20.190 m_IDirect3D9Ex::~m_IDirect3D9Ex(00A8DB28) deleting device!
17688 09:50:24.143 Redirecting 'DirectDrawCreate' IID_IDirectDraw to --> 'Direct3DCreate9'
17688 09:50:24.143 Creating device m_IDirectDrawX::m_IDirectDrawX(0790DB38) converting device from v1 to v9
17688 09:50:24.247 Creating device m_IDirect3DDevice9Ex::InitDirect3DDevice(00A80070)
17688 09:50:24.435 m_IDirect3DDevice9Ex::~m_IDirect3DDevice9Ex(00A80070) deleting device!
17688 09:50:24.933 Creating device m_IDirectDrawClipper::m_IDirectDrawClipper(009AEB60)
17688 09:50:24.933 Creating device m_IDirectDrawSurfaceX::m_IDirectDrawSurfaceX(07A36848) converting device from v1 to v9
17688 09:50:24.933 Creating device m_IDirectDrawSurfaceX::m_IDirectDrawSurfaceX(07A369E8) converting device from v1 to v9
17688 09:50:24.933 Creating device m_IDirectDrawPalette::m_IDirectDrawPalette(0798B3F8)
17688 09:50:24.933 Creating device m_IDirect3DTexture9::m_IDirect3DTexture9(07944988)
17688 09:50:24.934 Creating device m_IDirect3DTexture9::m_IDirect3DTexture9(07944C10)
17688 09:50:24.934 Creating device m_IDirect3DVertexBuffer9::m_IDirect3DVertexBuffer9(07944CA0)
17688 09:50:24.941 Creating device m_IDirect3DTexture9::m_IDirect3DTexture9(07944C70)
17688 09:50:25.297 Creating device m_IDirectDrawSurfaceX::m_IDirectDrawSurfaceX(038228A8) converting device from v1 to v9
17688 09:50:25.344 m_IDirectDrawSurfaceX::~m_IDirectDrawSurfaceX(038228A8) deleting device!
17688 09:50:28.895 m_IDirectDrawSurfaceX::~m_IDirectDrawSurfaceX(03822E50) deleting device!
17688 09:50:29.745 m_IDirectDrawSurfaceX::~m_IDirectDrawSurfaceX(03822E50) deleting device!
17688 09:50:32.375 m_IDirectDrawPalette::~m_IDirectDrawPalette(0798B3F8) deleting device!
17688 09:50:32.375 m_IDirectDrawX::~m_IDirectDrawX(0790DB38) deleting device!
17688 09:50:32.718 m_IDirect3DTexture9::~m_IDirect3DTexture9(07944988) deleting device!
17688 09:50:32.718 m_IDirect3DTexture9::~m_IDirect3DTexture9(07944C70) deleting device!
17688 09:50:32.718 m_IDirect3DTexture9::~m_IDirect3DTexture9(0CBC29A0) deleting device!
17688 09:50:32.718 m_IDirect3DVertexBuffer9::~m_IDirect3DVertexBuffer9(07944CA0) deleting device!
17688 09:50:32.719 m_IDirectDrawSurfaceX::CleanupSharedEmulatedMemory Deleting 0 emulated surfaces!
17688 09:50:32.730 Quiting DxWrapper
17688 09:50:32.731 Unloading libraries...
17688 09:50:32.731 Reseting screen resolution
17688 09:50:32.737 Reseting font smoothing
17688 09:50:32.745 DxWrapper terminated!

It seems like you have a different version of the game.  Can you zip and upload your exe file?

Sure, it is an italian copy of the game, I think I got it with a videogame magazine ages ago...
WarGames.zip

Try using this config file update.  It should work with that version of the game.
dxwrapper.zip

Related to issue #119

Hello, just to inform that I tried the zip from April 8 2022, but the game crashes for me after launching it. If this problem happens only with my version of the game, maybe it is not worth the effort to fix (I can play with the CD or virtualizing the iso).

Ok, try with this one.  If it still crashes then please zip up the wargames.exe file and post it here so I can check it out.
Here is the latest build from today.  I also changed some settings to make it more compatible:  dxwrapper.zip

Nope, please, find below the log:
12208 10:07:08.446 Starting DxWrapper v1.0.6612.21
12208 10:07:08.446 Running from: C:\WarGames\dxwrapper.dll
12208 10:07:08.446 Reading config file: C:\WarGames\dxwrapper.ini
12208 10:07:08.446 ASUSTeK COMPUTER INC.   Are you are using the exe file you sent above?  I tested that one and it worked for me.
Did you follow all the steps about copying the data from the CD ROM to the WarGames folder or are you playing with the CD ROM?
Can you try with this binary file and see if it works?  Make sure to backup your file first.  This is the one I am using:  wargames.zip

Tried with the exe from your last comment and https://github.com/elishacloud/dxwrapper/files/11238039/dxwrapper.zip, not launching, below the log:
9044 18:35:21.408 Starting DxWrapper v1.0.6612.21
9044 18:35:21.408 Running from: C:\WarGames\dxwrapper.dll
9044 18:35:21.408 Reading config file: C:\WarGames\dxwrapper.ini
9044 18:35:21.408 ASUSTeK COMPUTER INC.

Can you try with the latest patch, just released:  https://github.com/elishacloud/dxwrapper/wiki/WarGames

Closing issue.  You can reopen it if you still have an issue.


Can't build with v2.7.0b

target: https://github.com/ValveSoftware/steamvr_unity_plugin/tree/bc86c9dd82c33b432c6d0b63870ea4b96b9e46a7

Build error in SteamVR_Input.cs as the platform of json.dll is only editor selected
Build error because the code to be built refers to Unity Editor

Thanks for the report, give this a try: https://github.com/ValveSoftware/steamvr_unity_plugin/releases/tag/2.7.1RC

I tried 2.7.1RC but the settings in Valve.Newtonsoft.Json.dll have not been modified.

Valve.Newtonsoft.Json.dll after importing 2.7.1RC

Valve.Newtonsoft.Json.dll after importing 2.6.1

Newtonsoft actually shouldn't be included in builds. It is only meant as an editor time thing. We changed the editor only code to #ifdef the unity editor. Give the build a try again and lemme know if you hit the same issue.

The script still uses Valve.Newtonsoft.Json.dll.
Steps to reproduce

Create a unity2019.4.16f1 project
Import https://github.com/ValveSoftware/steamvr_unity_plugin/releases/download/2.7.1RC/steamvr_unity_plugin-2.7.1RC.unitypackage
Open SteamVR Input Window and generate SteamVR Input classes
Add Interactions_Example scene to Build Settings
Build

I have the same problem

I also do. Unity 2020.2, same errors.

Okay give the proper 2.7.1 release a go: https://github.com/ValveSoftware/steamvr_unity_plugin/releases/tag/2.7.1

Okay give the proper 2.7.1 release a go: https://github.com/ValveSoftware/steamvr_unity_plugin/releases/tag/2.7.1

Thank you. It worked fine in 2.7.1.

Thank you. It worked fine in 2.7.1.

Closing as fixed.