suggestion: consistent info if something doesn't exist

hello,
at the moment we have two behaviours when a key doesn't exist in this example
	map_cmd := fme.Redis_client.HGetAllMap("missing_key")
	t.Log(map_cmd.Err(), map_cmd.String(), map_cmd.Val())

	ss_cmd := fme.Redis_client.Get("missing_key")
	t.Log(ss_cmd.Err(), ss_cmd.String(), ss_cmd.Val())

results in
	a_test.go:256: <nil> HGETALL missing_key: map[] map[]
	a_test.go:259: redis: nil GET missing_key: redis: nil

So we have .Err() which is "redis: nil" one time and nil the other time.
redis-cli gives us this
<IP_ADDRESS>:6379> HGETALL missing_key
(empty list or set)
<IP_ADDRESS>:6379> GET missing_key
(nil)
<IP_ADDRESS>:6379> SMEMBERS missing_key
(empty list or set)
<IP_ADDRESS>:6379>

so we explicitly know: ok this doesn't exist
It would be nice to have another cmd.Func (bool) called KeyNotFound() (like in sql-queries) additional to the Err(), Val() , ...
So, this wouldn't break anything (current api) and would be nice for all get methods.
Instead of checking if len(map) == 0 {} , etc.. for the types we could just do: if cmd.KeyNotFound() {}
What do you think?
best
m

In my opinion, the current behaviour is correct and intended, as it follows
redis' own response logic.
For example, http://redis.io/commands/get may return nil, while
http://redis.io/commands/hgetall never do.
Returning redis.Nil as an error on some commands - I agree - sometimes
feels a little clumsy, but it's also a sensible compromise. I don't think
it would be right to return Nil on all commands, that would be
counterintuitive and a layer of logic on top of redis itself. A client
should not make such decisions.

Speaking about HGETALL empty response can mean that:

hash does not exist
hash is empty

and we don't know answer for sure, because Redis Server does not tell us.

for the types we could just do: if cmd.KeyNotFound() {}

We could add cmd.IsEmpty(), but it does not look important/popular enough (even now I don't see a convincing use case for it).

Redis does not keep empty hash keys, empty keys are deleted:
$ redis-cli hset hash key value
(integer) 1
$ redis-cli exists hash
(integer) 1
$ redis-cli hdel hash key
(integer) 1
$ redis-cli exists hash
(integer) 0

On 18 December 2015 at 12:01, Vladimir Mihailenco<EMAIL_ADDRESS>wrote:

Speaking about HGETALL empty response can mean that:

hash does not exist
hash is empty

and we don't know answer for sure, because Redis Server does not tell us.
for the types we could just do: if cmd.KeyNotFound() {}
We could add cmd.IsEmpty(), but it does not look important/popular enough
(even now I don't see a convincing use case for it).
—
Reply to this email directly or view it on GitHub
https://github.com/go-redis/redis/issues/220#issuecomment-165763812.


Move back to nightly for torchtune instead of commithash

Dropping thing for now until we get the CI back up
Will spin up again later


[SITE] Implement View single Kit page

Data will be provided based on the kitId from KitsService. Ask @Mastaleru for further details

Please start with UI first

example of how to get the required data in order to populate the view: https://github.com/PharmaLedger-IMI/csc-workspace-main/blob/d9cae1268864d48c6e75e2b25db4d8ae7b77a4d0/csc-services/controllers/impl/KitsControllerImpl.js#L86

Created a new Kit page and added functionality to display the Kit data. PR has been created for it


{dshot,pwm_out}: allow mode_pwm4 for boards with 5 outputs

Required on durandal if camera triggering is enabled.
@mhkabir this fixes your initial issue, still good to add support for the capture pins though.

Any docs impact @bkueng ?

No, this is just a bugfix


MVC action string param null when empty

Describe the bug
I have an action in my MVC project which works with URLS like search?q=xxx. The action definition is:
[HttpGet("search")]
public IActionResult ThreadsJson([FromQuery(Name = "q")] string query)
{
    return Json(listOfStrings.Where(x => x.Contains(q)));
}

This works well in general, but causes an error when the URL is called with an empty q parameter like search?q=. In this case, q is null instead of an empty string causing the .Contains method to fail.
To Reproduce
Steps to reproduce the behavior:

Using Microsoft.AspNetCore.App 2.1.1.
Refer above.

Expected behavior
The q parameter is an empty string.

Thanks for contacting us, @shravan2x.
You can initialize query to the default String.Empty value and that would do it.

@mkArtakMSFT While that's a workable solution, it forces you to put the query param at the end of the list of arguments since it has a default value. This results in bad code where you can't put important arguments in the beginning without adding default values to every argument the function takes.
I ended up just using
if (query == null)
    query = "";

Altogether, I don't intend to reopen this issue since it adds to your backlog, but I believe there has to be a better solution to this problem.

@mkArtakMSFT I've found that defining default empty string values under complex query objects seem to be overridden if the mentioned behavior is performed?
Here's our action definition:
public async Task<ActionResult<IEnumerable<Company>>> SearchCompanies([FromQuery] CompanySearchQuery search)
And here is our query model:
`
public class CompanySearchQuery : ISearchQuery
{
[DefaultValue("")]
public string SearchString { get; set; } = "";
    [DefaultValue("")]
    public string Name { get; set; } = "";

    public Guid? ParentId { get; set; }
}

`
Whenever something comes in like 'searchString=', we end up with null in that field, regardless of whether we have a default value for that property. Is this intended? It's a bit cumbersome to have to add a check for the property being null to our queries whenever we're adding a new parameter to the search model.
We're using .NET Core SDK 3.0.100 preview 8


QGC showing wrong battery percentage when throttle is increased

Hey everyone i am having some issues regarding battery calibration while working with pixhawk. The battery percentage which i am getting on my QGC is right when it is not armed but when i armed the quad and increase its throttle then instead of decreasing the battery percentage it started increasing. For example if my battery is 74 percent remaining, now when i increase its throttle it started increasing to approx 88% at full throttle instead of decreasing.
I am badly stuck up unable to understand and solve the problem. Please help.

As you increase the throttle the voltage of the battery drops. Normally this would cause it to appear that % remaining battery was lower than it actually is. PX4 counters this by scaling the voltage based on the current throttle. If you look in the advanced battery settings in QGroundcontrol there will be a box where you can the voltage drop caused by 100% throttle. It sounds like the value of this parameter may be too high for you. Try reducing it until your % battery remaining is unaffected by changes in throttle.

ok i will try it and let you know about it.
Thanks

@lakshaydang please reopen if still an issue.


use jQuery from their site, not google

moving away from Google to jquery.com - and going to newer jquery in the process.

Little need to run the risk - sysadmin asked not to do it.


Feature | Added new dom() method for parsing HTML and XML

This PR introduces a new method for the SaloonResponse, a new dom() method. This method requires Symfony's DomCrawler package and is not installed by default. This method will allow you to easily parse HTML or XML responses.
Read more about DomCrawler: https://symfony.com/doc/current/components/dom_crawler.html

🔥🎉🚀✅


When SM/FS is unpatched, .3dsx version crashes when changing selection from WiFi Information to another row (GUI)

Yeah I'm working on one final release to solve these issues.
This time it'll check if the functions that require the sm/fs patch, can run successfully, if it doesn't it'll display 0 or NULL.

Fixed with
https://github.com/joel16/3DSident/commit/624c305ba8c5d0059387366f44f64691d33f3e9c
Seems like this service cannot be accessed when running under *hax. Therefore a message saying "This service cannot be accessed in *hax." will be displayed to avoid crashes.


Add DoCallback-Test to ensure that the globalConfig is isolated..

in AppDomain. No other test is affected by the global change.

👍  :-)


neo-gui assest exception

hi
i create an assets at neo private chain but when i distribute it i have this error on reception wallet

any info what problem ??

problem in neo-gui only ,neo-cli just work fine
best to handle exceptions guys ,this is a 4B$ project ,seriously !!

I need the exception message.

problem is here
Saying Path is "null" , there no path var in settings structure

Please send me the exception message.

note :

i am running in private network
error when try parse my registered asset

exception message :
https://prnt.sc/jp728t

It seems that the problem is in the configuration file.
Can you provide me your config.json?

of course :
http://prntscr.com/jpifbl

I know where the problem is. You provide a neo-cli configuration file that cannot be used in neo-gui.

my mistack , i dont know how cli config come to gui folder
i fixed it , thanks bro , keep good work


USB cherry pick series for NCS 2.7.99-CS2

Bring the features that are supposed to be in NCS 2.7.99-CS2

Descoped due to issues that are not present in upmerge branch. No point in trying to figure out the root cause which most likely is fixed in upstream Zephyr (and hence it does not reproduce in upmerge).


3.0 Upgrade Report: Spring MVC and WebFlux URL Matching Changes

What needs to be done
The auto-generated section in the Spring Boot 3 Upgrade Report for the 'Spring MVC and WebFlux URL Matching Changes' section should be improved.
:warning: Please read Spring Boot 3 Upgrade Report Contribution Guideline :warning:
From the Release Notes

As of Spring Framework 6.0, the trailing slash matching configuration option has been deprecated and its default value set to false.
This means that previously, the following controller would match both "GET /some/greeting" and "GET /some/greeting/":

[source, java]
@RestController
public class MyController {
@GetMapping("/some/greeting")
public String greeting {
return "Hello";
}
}
As of https://github.com/spring-projects/spring-framework/issues/28552[this Spring Framework change], "GET /some/greeting/" doesn't match anymore by default.
Developers should instead configure explicit redirects/rewrites through a proxy, a Servlet/web filter, or even declare the additional route explicitly on the controller handler (like @GetMapping("/some/greeting", "/some/greeting/") for more targeted cases.
Until your application fully adapts to this change, you can change the default with the following global configuration:
[source, java]
@Configuration
public class WebConfiguration implements WebMvcConfigurer {
@Override
public void configurePathMatch(PathMatchConfigurer configurer) {
  configurer.setUseTrailingSlashMatch(true);
}

}

The report is defined in components/sbm-recipes-boot-upgrade/src/main/resources/recipes/27_30/report/sbu30-report.yaml
The (generated) test is defined in components/sbm-recipes-boot-upgrade/src/test/java/org/springframework/sbm/boot/upgrade_27_30/report/helper/SpringMVCAndWebFluxURLMatchingChangesReportSectionTest

Hi @fabapp2,
Many thanks for providing an opportunity.
I would like to contribute for this improvement of boot report changes, since am doing this for the first to spring, could you please guide me how/where to start, for the rest of the upgrade report I will pick it up myself.
In fact have signed contributor license agreement here https://cla.pivotal.io/sign/spring/icla#sign-form
Thanks,
Mahi

Thanks for your interest @bsmahi 🚀
I am not very responsive at the moment.
I just realised this has been done with #529.
could you maybe pick another issue for a start?


Add input and output interfaces

This branch should implement the input and output boundary into the use case interactor, as well as create an interface for the data access object which gets implemented
This will ensure we follow the clean architecture model and decouple the the classes from each other

issue resolved. refer to pr #60. Good job Marco 👍


fix return type

Hi @Damoness, can you elaborate why you have proposed this change?

const onSubmit = async () => {
  try {
    const eventId = await CalendarModule.createCalendarEvent(
      'Party',
      'my house'
    );
    console.log(`Created a new event with id ${eventId}`);
  } catch (e) {
    console.error(e);
  }
};

The return type of createCalendarEvent  is a number rather than a array


fixed classIdPrefix

It is the same as https://github.com/jhamlet/svg-react-loader/pull/107. But will be merged, i hope.

@jhamlet Review and merge this or other one, please


PostAgent and XMLRPC - howto?

Hello!
Huginn makes my life easier, but some times I'm confused.
I'd like to use xml-rpc with the PostAgent. Specification here: http://xmlrpc.scripting.com/spec.html
But I can't build the sctucture with multiple parameters with the same name. Huginn doesn't let me do this, it's always a json error.
My agent:
{   "post_url": "http://domain/xmlrpc.php",   "expected_receive_period_in_days": "1",   "content_type": "xml",   "method": "post",   "no_merge": "true",   "xml_root": "methodCall",   "payload": {     "methodName": "wp.newPost",     "params": {       "param": {         "value": "1"       }     }   },   "headers": {},   "emit_events": "false" }
I need more  and multible  but how? What am I doing wrong?
Could you help me?

Hi @nata-lee, I've never actually used true XML-RPC.  (As opposed to ajax, which is usually JSON these days.)  You may want to have Huginn post to a server that logs your requests, to make sure they look right.  http://requestb.in/ could be useful here.

Hi @cantino, thank you for the answer.
Well, I have no problem with XML-RPC then I use curl. And requests and responds are the next step.
I'm confused how to format XML-RPC xmls (and all xmls in general) with huginn interfase and the PostAgent. I tried to form the request as I wrote in the last message, but I can't understand how to get such structure as:
<methodCall>
   <methodName>examples.getStateName</methodName>
   <params>
      <param>
         <value><i4>41</i4></value>
         </param>
      <param>
         <value><user>testname</user></value>
         </param>
      </params>
   </methodCall>

I just need an example.

The following JSON options are pretty close:
{
  "post_url": "http://requestb.in/123456",
  "expected_receive_period_in_days": "1",
  "content_type": "xml",
  "method": "post",
  "payload": {
    "methodName": "examples.getStateName",
    "params": [
      {
        "value": {
          "i4": "41"
        }
      }
    ]
  },
  "headers": {},
  "emit_events": "false",
  "xml_root": "methodCall"
}

When pointing at a requestbin I made, I got this:

@cantino I think the problem is the to_xml serializer can not generate multiple params elements which have the same parent.

Wait, what's the problem?  Isn't might output what @nata-lee is looking for?

@dsander is quite right. I can't get multiple param in the params.
<params>
 <param>123</param>
 <param>456</param>
</params>

is impossible via huginn interface :( Or I don't know how to get it, so I asked my question.
Maybe any workaround? Now I have to use Shell Command Agent insteed of the PostAgent,and it's some kind of unsecure.

I don't think that is possible at the moment, but I am planning to change the PostAgent to allow sending arbitrary data which can be formatted using liquid.

@nata-lee With #1402 merged you will be able generate the XML yourself and setting it in the payload as a string.

I don't think that will ever be possible with the way to_xml works however the workaround to to manually construct the XML structure should work.
Please feel free to reopen if you need further assistance!


fix(k8s-port-forward): delete config if start fails

What does this PR do?
Before we were creating the port forward even if an error was raised in the middle, this is because we are creating it in two phases. (creation, then starting).
If the start phase fail we should delete the port foward, as we are not able to show a failing state, but we handle the error, so we should only display the error
Screenshot / video of UI
https://github.com/user-attachments/assets/bfbd6013-e5b0-454e-8f9c-8c8c30233c26
What issues does this PR fix or reference?
Fixes https://github.com/podman-desktop/podman-desktop/issues/9640
How to test this PR?

[x] Tests are covering the bug fix or the new feature

there are errors in PR checks (before looking at the hoisted issue)

my comment and PR have been ignored

my comment and PR have been ignored

I'm very sorry I will revert tomorrow. I don't know why but I've not seen the comments when I clicked on merged.


Update OLM to bundle

Refactor CSV to use OLM bundle layout

Have this in the work ... please assign.

PR -> https://github.com/citrusframework/yaks/pull/310


Exercise 05 Maybe Class don't understand code behaviour

For this Maybe Exercise 5, the code shown on the left is the one that allows me to compile. I tried to write the 3 lines of code into 1 line, as shown on the right. But when I do so, I can't compile anymore, giving me that error.
Question: Why is there an error if I squeeze the 3 lines of code into 1 line? What does the error "defined in an inaccessible class or interface" exactly mean? I thought about whether it's because the get() function in Maybe class is private, but, if so, won't there be an error for the code written with 3 lines (left part of the picture)?
Thank you!

Prof brought up something similar in class regarding the implementation of flatMap, and he let us go back to think about it. I added a section to Lecture 7 on our Wiki to explain this behaviour:

In the definition of flatMap, you may be tempted to chain the three statements together, as follows:
<R> Maybe<R> flatMap(Function<? super T, ? extends Maybe<? extends R>> mapper) {
   if (this.isEmpty()) {
       return Maybe.<R>empty();
   }
   return Maybe.<R>of(mapper.apply(this.get()).get());
}

However, this will result in a compilation error.
The key observation to make here is that get is a private method defined in Maybe, and mapper.apply(this.get()) returns a value of type ? extends Maybe. We cannot guarantee being able to call get from a subclass of Maybe. Hence, the result of mapper.apply(this.get()) first has to be assigned to a variable of type Maybe, which we can guarantee being able to call get on.

The same applies to the definition of or: calling supplier.get() returns an ? extends Maybe, which you cannot guarantee being able to call get on, hence you have to first assign it to a Maybe.

cuz mapper.apply(this.get()) returns ? extends Maybe<? extends R>, which means not only can it return a Maybe, it can return a child class of Maybe
t can return say a Child<T> that extends Maybe<T>, in which case it cannot access the .get() in Maybe, since its private

Ohhh I see, I understand now.  Thanks for the explanation!


Input Types and Newline/Wrapping

Some comments/suggestions sparked from the gfm mention here: https://github.com/piotrmurach/tty-markdown/pull/16#issuecomment-568202761
While gfm is mostly useful additions on top of standard markdown I don't think I'd choose it as the default since it causes single newline chars in text to inject :br elements into the doc (side note: maybe useful for that test case discussed in #15).  In normal markdown single newlines are just left as newlines in the text and are disregarded as that HTML is rendered on screen.  In the context of the rest of the current tty-markdown implementation this means with gfm mode single newlines within text input become double newlines in output.  The generated kramdown doc shows gfm producing the :br element and also keeping the \n in the following :text element:
Kramdown::Document.new("foo\nbar", input:"Markdown").root.children
=> [<kd:p nil {:location=>1} [<kd:text "foo\nbar" nil {:location=>1}>]>]
Kramdown::Document.new("foo\nbar", input:"GFM").root.children
=> [<kd:p nil {:location=>1} [<kd:text "foo" nil {:location=>1}>, <kd:br nil {:location=>1}>, <kd:text "\nbar" nil {:location=>2}>]>]

Noting this led to trying some larger text-focused docs some of which had wrapped their markdown paragraphs to 80 columns with newlines (which again doesn't impact the HTML rendering).  I think the look and feel of that output would be improved by stripping newlines from :text elements.  This would more closely align the terminal output with the HTML rendered output.  Currently:
Kramdown::Document.new("foo\nbar\nbaz", input:"Markdown").root.children
=> [<kd:p nil {:location=>1} [<kd:text "foo\nbar\nbaz" nil {:location=>1}>]>]
Kramdown::Document.new("foo\nbar\nbaz", input:"Markdown").to_html
=> "
foo\nbar\nbaz
\n"
TTY::Markdown.parse("foo\nbar\nbaz", input:"Markdown")
=> "foo\nbar\nbaz\n"

If space is available that HTML paragraph will render to a single line whereas tty-markdown output is split across 3 lines even with standard markdown input type and a very wide terminal.  Stripping these newlines probably has some consequences to manage with respect to width wrapping and indentation (and peeking at it there's an existing bug in #convert_text where indentation is being added after wrapping is applied which could extend lines beyond the specified width).
It'd be nice to also support disabling the built-in wrapping for long text elements so that modern terminals could do their own wrapping and reflow of full paragraphs.
Suggestion summary:

input: "Markdown" as default, document availability of others
strip newlines from text elements
add option to disable addition of newlines that wrap long text to width

On the general idea behind tty-markdown wrapping feature. It was meant to provide a nice way alternative to man documentation. Specifically, I wanted to have nicely formatted documents limited let's say to 80 characters regardless of terminal width. This is desirable, especially when working on HD wide terminals. Natural wrapping could be an option but wasn't a design goal at the start. The design was aimed at providing easy to read documentation for command-line tools.
As for motivation for the gfm parser. The most important benefit is code formatting. The gfm parser actually handles code samples with newlines. It provides langauge specification as part of parsing behaviour. Many people expect this, that's why it was reported as 'broken' when didn't behave this way. Github flavoured markup for many is the way markdown should work. The kramdown and Markdown formatters break and display code samples as unformatted paragraph of text including the fenced symbols. For example:
template = <<~MSG
```ruby
def test

  puts "testing"

end
```
MSG

TTY::Markdown.parse(input: "Markdown")
# =>
# ```ruby
# def test
#
#   puts "testing"
#
# end
# ```

I consider the above as 'poor' and 'confusing' default. Especially, where the main point of this gem was to provide 'out-of-the-box' experience for formatting code samples. With the GFM parser:
template = <<~MSG
```ruby
def test

  puts "testing"

end
```
MSG

TTY::Markdown.parse(input: "Markdown")
# => 'a colorful nicely formatted langauge specific output
# def test
#
#   puts "testing"
#
# end

As for the br element being added. When I tested the output on my little example I realised that it is actually beneficial to how output looks. Taking your example of foo\nbar. The previous implementation would do this:
TTY::Markdown.parse("foo\nbar")
# =>
# foo
# bar

Which when displayed would have 2 lines that should be visually separated look like they are one paragraph. Considering new implementation:
TTY::Markdown.parse("foo\nbar")
# =>
# foo
#
# bar

I believe this to be a more desirable outcome. Visually two separate lines get visually split into two separate lines in the terminal. Profit. The same goes for many other behaviours like dealing with headers followed immediately by content:
TTY::Markdown.parse("#header\nsome copy")
# =>
# header
# some copy

The same now gets nicely split thanks to br being emitted:
TTY::Markdown.parse("#header\nsome copy")
# =>
# header
#
# some copy

To me, the important bit is the final visual outcome and what may be expected by the users. I don't want people to be thinking about types of parsers but rather things to work out of the box. Inevitably, this will create issues for wrapping plain content as newlines will mean double output but we can handle this scenario when processing text paragraphs etc... For example, we could change convert_br to only inject new lines for certain elements and skip for others. You can check the type of parent element for the br element. Also, we can add metadata to text paragraphs that are wrapped vs regular newlines supplied by the user.
I'd rather have awesome looking code samples and tweak the paragraphs rather than the other way round. I'd happily leave parsing of codeblocks to kramdown. I'm in for the option to disable wrapping, it could be a special value for the width parameter like false or :none.

Another alternative is to extract the code fencing logic out of the GFM parser and add it to the default one in tty-markdown.

I agree that the output and what's expected by users is the important bit.  I maybe didn't do a great job with the foo\nbar example but output not matching expectations is what I was hoping to illustrate:

previously the default output was foo\nbar
with the gfm default switch the new output is foo\n\nbar
as a new user I'd actually expect to see foo bar according to standard markdown rules

I think changing to gfm as default is fine too as long as it's mentioned somewhat prominently in the readme.

So also in the context of gfm I think some of the newlines your examples present are still not what I'd expect.
In the foo\nbar example the complete html output produced by kramdown parsing that in gfm mode looks like:
Kramdown::Document.new("foo\nbar", input:"GFM").to_html
=> "
foo
\nbar
\n"

I'd expect the terminal output to represent this single paragraph containing 1 line break the same way a browser would which is to say on just two lines: foo\nbar\n.
In gfm if you instead used a double newline like foo\n\nbar the html output is significantly different:
Kramdown::Document.new("foo\n\nbar", input:"GFM").to_html
=> "
foo
\n\n
bar
\n"

Here I would expect the terminal output to be foo\n\nbar\n. The blank line makes it clear the surrounding lines are not part of the same single paragraph.
So currently in gfm mode both foo\nbar and foo\n\nbar, which mean different things in gfm, are resulting in identical output from tty-markdown which also isn't what I'd expect.
This is where my previous mention of stripping newlines from :text elements comes in. If you replace newlines plus all surrounding whitespace with a single space then both of these gfm situations and the regular markdown generate the output I'd expect.  The full change would be more complex than just a single gsub in order to manage indent/width features.
For the header example of # header\nsome copy the html output from kramdown is:
Kramdown::Document.new("# header\nsome copy", input:"GFM").to_html
=> "
header
\n
some copy
\n"

Again here I think an empty line between the header and paragraph is a reasonable translation.  I wanted to mention though that in my terminal using the current tty-markdown master branch here I'm getting just header\nsome copy without the blank line your example output included.  This probably needs to be changed at the end of convert_header to append a double newline?

New Summary:

Choosing whatever default flavor is fine as long as it's documented prominently if the default isn't standard markdown.
I think tty-markdown's output should strive to mimic the overall formatting of the HTML output produced by typical markdown converters for whichever flavor(s) it supports.
I suspect stripping newlines out of text elements is the easiest-to-maintain option for getting that portion of tty-markdown's newline output to match the HTML equivalents across the various flavors kramdown will parse.

I resumed work on this gem and I'm close to making a new release. I wanted to answer your points:

Choosing whatever default flavor is fine as long as it's documented prominently if the default isn't standard markdown.

I switched back to the default kramdown parser which is compatible with Markdown and adds a few extra elements like tables etc. I'm not going to expose the ability to change this as the tty-markdown is a whole tool that needs to work consistently out of the box. I completely scrapped the idea of gfm parser. I ported the fenced behaviour, for now, to make the code blocks work as expected.

I think tty-markdown's output should strive to mimic the overall formatting of the HTML output produced by typical markdown converters for whichever flavor(s) it supports.

I'm not sure that this gem should strive to 'mimick' HTML output. There are some significant differences between the use cases of HTML rendering and rendering to the terminal. For example, I cannot let the text wrapping be handled by terminal. That wouldn't' work with how this tool styling works. For example, the blockquote or table formatting 'heavily' relies on the terminal width constraint as well as indentation for each paragraph. It would almost impossible to provide sensible formatting, at least the way I want it to be by having terminal handle the wrapping. By default, the wrapping is set to be automatic and span the entire terminal width.

I suspect stripping newlines out of text elements is the easiest-to-maintain option for getting that portion of tty-markdown's newline output to match the HTML equivalents across the various flavors kramdown will parse.

I don't think this is desirable. If a user provides input where they specifically want to break blockquote element by a newline, I don't think that tty-markdow should modify and strip any tabs or newline characters. The HTML is a different medium in so far as controlling output format goes.
I really appreciate your help and reading this thread was a great reminder as to what needed to change and what I need to focus on.


Dead key and then return key produces double character

Issue Type: Bug
Description:
Some dead keys prints the character doubled (only in Visual Studio Code) when return is pressed next.
Steps to reproduce:

Use macOS
Open Visual Studio Code
Create a new file
Select U.S. software keyboard layout
Press on US hardware layout option + `
Press return

Expected:
The character  ` will be printed followed by a newline: `\n. Like it does when simply using the key `
Without pressing the additional option modifier (because then it is not a dead key). Since the dead key state is terminated by not using one of the keys that combine with the dead key `
It should just print  ` followed by the symbol that could not be combined with  `. This is the normal behavior in any other app, like TextEdit.app.
What happens:
The character ` will be printed two times followed by a newline:  ` ` \n.
Notes:
It does not happen with option + u, there it always prints ¨ just once, even though this is also a dead key. Custom set dead keys do always have this problem.
System:
macOS Big Sur 11.5.2 on MacBook Pro
VS Code version: Code 1.59.1 (3866c3553be8b268c8a7f8c0482c0c0177aa8bfa, 2021-08-19T11:53:52.479Z)
OS version: Darwin x64 20.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz (8 x 3100)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
79, 25, 11

Memory (System)
16.00GB (4.06GB free)

Process Argv
--crash-reporter-id 22e78f52-d1d7-4c5d-8c9a-855d201de122

Screen Reader
no

VM
0%

Extensions: none

I can reproduce this in any <textarea> in Chrome and Edge, so this is an upstream issue.
When pressing option+`, followed by Enter, I get `` instead of `\n.
Please report this upstream at https://bugs.chromium.org/p/chromium/issues/list
https://user-images.githubusercontent.com/5047891/143960520-2537b2f1-3d6a-47dc-9bd9-63f6169c867f.mp4


phonegap-plugin-push stop functioning after install code-push of microsoft

Thanks so much for filing an issue or feature request! Please fill out the following (wherever relevant):
Description
I have added code-push to a cordova project and plugin phonegap-plugin-push stopped functioning properly.
Reproduction
install phonegap-plugin-push and then install cordova-plugin-code-push
Additional Information

cordova-plugin-code-push version:  1.11.17

phonegap-plugin-push version: 2.2.3

List of installed plugins: 25

Cordova version: 7.1.0

android platform version: 6.3.0

iOS/Android/Windows version: android 8

Does this reproduce on a debug build or release build? debug build.

Does this reproduce on a simulator, or only on a physical device? physical device.

Before installing code , when receiving push notification, the log for logcat looks as follows:
01-20 19:17:30.874  8292  8312 D Push_FCMService: onMessage - from:<PHONE_NUMBER>62
01-20 19:17:30.884  8292  8312 D Push_FCMService: normalize extras
01-20 19:17:30.885  8292  8312 D Push_FCMService: key = content-available
01-20 19:17:30.885  8292  8312 D Push_FCMService: replace key content-available with content-available
01-20 19:17:30.885  8292  8312 D Push_FCMService: key = summaryText
01-20 19:17:30.885  8292  8312 D Push_FCMService: replace key summaryText with summaryText
01-20 19:17:30.886  8292  8312 D Push_FCMService: no locale found for key = summaryText, error Value There of type java.lang.String cannot be converted to JSONObject
...

after installing code push, whenever recieving notification, the log related to FCMService stop after one line. and there is no error. push notification does not arrive either.
01-20 19:24:34.329  9183  9202 D Push_FCMService: onMessage - from:<PHONE_NUMBER>62
I will gladly supply more information if needed. Thanks for any help!

Same issue here. This is what adb logcat is reporting:
02-07 12:30:35.538 32568 32730 V Push_Plugin: execute: action=init
02-07 12:30:35.541 32568 32731 V Push_Plugin: execute: data=[{"android":{"icon":"push","iconColor":"#ff0000","senderId":"XXXXXXXXXXXX"},"browser":{},"ios":{"sound":true,"vibration":true,"badge":true},"windows":{}}]
02-07 12:30:35.556 32568 32731 V Push_Plugin: execute: jo={"icon":"push","iconColor":"#ff0000","senderId":"XXXXXXXXXXXX","id":"PushPluginChannel","description":"PhoneGap PushPlugin"}
02-07 12:30:35.560 32568 32731 E Push_Plugin: execute: Got Resources NotFoundException String resource ID #0x0

Cordova 8.1.2
Android 7.1.1
cordova-plugin-code-push 1.11.17
phonegap-plugin-push 2.2.3

Interestingly I was able to get this to work eventually.
I think what mattered was the order of plugins in config.xml. I put phonegap-plugin-push after the appcenter plugin and either because of this or setting targetSdkVersion to 28 (from 27) it started working.

Exact problem with me. Installed code-push and push plugin broke.

Same error here


Closes #228 | Add M3LS dataloader

Please name your PR title and the first line of PR message after the issue it will close. You can use the following examples:
Closes #228
Checkbox

[x] Confirm that this PR is linked to the dataset issue.
[x] Create the dataloader script seacrowd/sea_datasets/{my_dataset}/{my_dataset}.py (please use only lowercase and underscore for dataset folder naming, as mentioned in dataset issue) and its __init__.py within {my_dataset} folder.
[x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _LOCAL, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _SEACROWD_VERSION variables.
[x] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[x] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one SEACrowdConfig for the source schema and one for a seacrowd schema.
[x] Confirm dataloader script works with datasets.load_dataset function.
[x] Confirm that your dataloader script passes the test suite run with python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py or python -m tests.test_seacrowd seacrowd/sea_datasets/<my_dataset>/<my_dataset>.py --subset_id {subset_name_without_source_or_seacrowd_suffix}.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Heads-up for reviewers:

Total data is ~14GB
The _generate_examples requires file reconstruction and validation (due to the nature of scraped data requires more validation), in my machine it only generate around 3-4 examples/s and has ~57K examples in total

pushed-force to remove secrets in earlier commit

Hi @patrickamadeus, I would like to let you know that we plan to finalize the calculation of the open contributions (e.g., dataloader implementations) in 31 hours, so it'd be great if we could wrap up the reviewing and merge this PR before then.
cc: @sabilmakbar

done making changes, pls have a look @fhudi

@sabilmakbar cool. thanks.
The test is currently running, since it is a 15.9GB file, please wait for a while

@sabilmakbar
Other than the above mentioned issue, everything else work fine.
Passed the test and reviewing check-list. 🙏

Pillow library is required.
Shall we add try-except for checking?

Hi @fhudi, I thought PIL was included in SEACrowd reqs (but in fact it isn't). I'll add the try-catch exception for now so that every user can avoid downloading it only to find the error when generating the dataset due to missing PIL lib. Thanks for pointing this


Saving a document adds random characters to my file

Type: Bug
Intermittently, usually after a day or two of having VSCode running without quitting, when I save the document I'm working on, random text is added to the file. This breaks my code and throws all kinds of errors. I can undo the change, but when I attempt to save again, the same text applies.
This tends to happen after adding a new import to a Typescript file (.ts or .tsx).
This has been happening for months now.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


bigger than 51*51?

We have reported the performance of 61x61 kernels. What is your concern here?

Dear henbucuoshanghai,
Due to inactivity, we are closing this. We hope you could circumvent your problem. Feel free to reopen it if needed.


allow setting of DHCP option 7

allow setting of DHCP option 7. Used in Tinkerbell stack to have ipxe send logs to Boots syslog receiver.
Expected Behaviour

Current Behaviour

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Operating System and version (e.g. Linux, Windows, MacOS):

How are you running Tinkerbell? Using Vagrant & VirtualBox, Vagrant & Libvirt, on Packet using Terraform, or give details:

Link to your project or a code example to reproduce issue:

resolved via https://github.com/tinkerbell/dhcp/pull/33


stable-chrome Dockerfile would not build

Missing dependencies.  Can you take a look please?
Thank you,
Ed-

also i have jenkins setup and it alerts me for all errors in build and there have been none, i can not diagnose this without more details than "it doesnt work"

I will re-try and collect logs.
Thanks,
Ed-


add fallout smoke test

What this PR does:
Adds smoke test for Fallout, including the target nosqlbench file
Checklist

[ ] add README
[x] resolve open questions

Test run 493fc5b9 with these two files.


Directory info update reminder email

Sent semi-annually on a staggered basis?
Reminds people what their info currently is and suggests they review and update it, gives link

Originally created by Tom Smyth at 2016-11-06 15:40, migrated from redmine-#5376

Subject changed from Info update reminder email to Directory info update reminder email

By Tom Smyth at 2017-04-28 13:21

Description updated (diff)

By Tom Smyth at 2016-11-06 20:40

Priority changed from 4 to 2

By Tom Smyth at 2019-05-18 13:16


Create a convenience method to enable sandbox mode

Issue Summary
Related to issue https://github.com/sendgrid/sendgrid-nodejs/issues/1128
Currently, here is some sample code to enable sandbox mode:
const sgMail = require('@sendgrid/mail');
sgMail.setApiKey(process.env.SENDGRID_API_KEY);
const msg = {
  to<EMAIL_ADDRESS>  from<EMAIL_ADDRESS>  subject: 'Hello world',
  text: 'Hello plain world!',
  html: '
Hello HTML world!
',
  mailSettings: {
    sandboxMode: {
      enable: true,
    },
  },
};

Implementing a convenience method sgMail.enableSandboxMode() is the ask.

Recommend we close this and just use #1128 to track the work. Issues created by members are not tracked.


docs(v2): i18n doc + polish

Motivation
Write the initial i18n doc
Entry point: https://deploy-preview-4014--docusaurus-2.netlify.app/classic/docs/next/i18n/introduction

Excellent write-up and tutorial, thank you! :pray:
Some suggestions:

In Introduction or Tutotrial, it would be very helpful to include an example project filetree structure for a basic FR/EN site à la this. Doc location is crucial in this framework.

In Use Crowdin or FAQ, regarding CI it may be helpful to share that Crowdin has integrations with Github and other version management sites.

To setup Github + Crowdin integration:

As admin in Crowdin project dashboard, go to "Integrations" tab
Click "Github" and follow the instructions for setup.  Translations made in Crowdin are automatically PRed to Github Repo, and changes to source documents are automatically picked up by Crowdin for re-translation.

You can edit the sync settings

You can also manually trigger a sync by visiting "Integrations" > "Github" > "Sync Now"

Excellent write-up and tutorial, thank you! :pray:
Some suggestions:

In Introduction or Tutotrial, it would be very helpful to include an example project filetree structure for a basic FR/EN site à la this. Doc location is crucial in this framework.

In Use Crowdin or FAQ, regarding CI it may be helpful to share that Crowdin has integrations with Github and other version management sites.

To setup Github + Crowdin integration:

As admin in Crowdin project dashboard, go to "Integrations" tab
Click "Github" and follow the instructions for setup.  Translations made in Crowdin are automatically PRed to Github Repo, and changes to source documents are automatically picked up by Crowdin for re-translation.

You can edit the sync settings

You can also manually trigger a sync by visiting "Integrations" > "Github" > "Sync Now"

P.S. - are there any SEO considerations to be aware of?

P.S. - are there any SEO considerations to be aware of?

Thanks.
Will include a project tree example
For now I just put a simple note about the github integration here: https://deploy-preview-4014--docusaurus-2.netlify.app/classic/docs/next/i18n/crowdin#automate-with-ci
Unfortunately, and after talking with Crowdin support, it does not work if the crowdin.yml is not at the root of the repo (ie website/crowdin.yml does not work, as I use this in the tutorial). Don't really understand why but unfortunately that's the current state 😅 I may need to adapt the tutorial
About SEO, I don't think there's much to be said apart from that I'll add support for hreflang meta headers + an option to disable the headers. Let me know if you think about something else that could be useful.

Thanks.
Will include a project tree example
For now I just put a simple note about the github integration here: https://deploy-preview-4014--docusaurus-2.netlify.app/classic/docs/next/i18n/crowdin#automate-with-ci
Unfortunately, and after talking with Crowdin support, it does not work if the crowdin.yml is not at the root of the repo (ie website/crowdin.yml does not work, as I use this in the tutorial). Don't really understand why but unfortunately that's the current state 😅 I may need to adapt the tutorial
About SEO, I don't think there's much to be said apart from that I'll add support for hreflang meta headers + an option to disable the headers. Let me know if you think about something else that could be useful.

Unfortunately, and after talking with Crowdin support, it does not work if the crowdin.yml is not at the root of the repo

strange! if you'd like a working example, we have github crowdin integration working fine in this repo:
https://github.com/redwoodjs/learn.redwoodjs.com

Unfortunately, and after talking with Crowdin support, it does not work if the crowdin.yml is not at the root of the repo

strange! if you'd like a working example, we have github crowdin integration working fine in this repo:
https://github.com/redwoodjs/learn.redwoodjs.com


Incorrect timestamps (0.5sec off)

The timings of segments/words are sometimes inaccurate. When the attached audio is transcribed (we’re using base.en, but it seems to happen with larger models too), a lot of the segments have start times ~0.5sec after their actual start times. In the example, the word “Like” in “Like before…” should begin at 13.6s but WhisperKit is giving us 14.12s. This is happening for 5 out of the 10 segments in this audio.
I noticed that the segment contains a timing token with an accurate time of 13.6, but it uses 14.12 instead.

WhisperKit.TranscriptionSegment(….start: 14.12, end: 21.7, text: "<|13.04|><|13.60|> Like before, this balloon is still filled with mostly hydrogen. However, this time, about one third of it is oxygen.<|21.28|>”, ..., words: Optional([WhisperKit.WordTiming(word: " Like", tokens: [4525], start: 14.12, end: 14.44, probability: 0.8)

When word timestamps are disabled, the segment gets a start time of 13.04, which doesn't account for all the silence.
out.m4a.zip

Thanks for the report @finnvoor! We started relying on the accuracy of word timestamps in streaming mode too. This is important, so we will triage and address it.

Low-hanging fruits:

Leverage the redundancy in segment and word-level timestamps for consistency checks
Implement median filtering in DTW as in the original implementation even though it didn't have a major impact in our early tests

Quick update, I've identified the issue and am putting together a patch for this now.

@finnvoor Please confirm that this fixes your issue 🙏

@ZachNagengast @atiorh gave it a quick test and the start times seem much more precise, thanks for the quick improvement.
It does seem like this has made the end times of words/segments slightly worse though. Previously, the end times would sometimes include some silence (be too late), but they never seemed to include any of the last word, so were good for splitting after a word/sentence.  Now it seems like it accounts for silence at the end better, but seems to go a bit too far and includes the end of the word. In the same example at ~4s the word "gas" used to end at 4.06, now ends at 3.62, but should end at ~3.8.

We'll continue to test it a bit more today.

I see, good to know, we might be able to improve this with some VAD (shift the end time to the last point that the sound level was past a threshold), but this is also the same endpoint that openai/whisper gives for their word timestamps, so it might be a model issue, or need a bit more massaging to get perfect. There are many such so called "hacks" in the main repo that could be improved.
For detail: the reason it ended that far past the audio previously is because we were including the punctuation token ".", which has non-zero length, as part of the word's end time, the fix removed that time entirely, so it is ending exactly where it things the word "gas" ends, before the punctuation. Next step may be to consider some middle ground where the punctuation counts for some time but not the full token because it's not a spoken word. Open to ideas here too!

Got it, figured eventually we'd run into model limits.  I think in our case I'll try just adding a small offset to the end since it seems pretty consistent, and in general adding silence is better than cutting words. VAD would be really nice but sounds a bit tricky to implement.


python312Packages.soco: 0.30.5 -> 0.30.6

Diff: https://github.com/SoCo/SoCo/compare/refs/tags/v0.30.5...v0.30.6
Changelog: https://github.com/SoCo/SoCo/releases/tag/v0.30.6

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354943

x86_64-linux

:white_check_mark: 21 packages built:

beets
beets-unstable
beets-unstable.dist
beets-unstable.doc
beets-unstable.man
beets.dist
beets.doc
beets.man
home-assistant-component-tests.sonos
python311Packages.soco
python311Packages.soco.dist
python312Packages.soco
python312Packages.soco.dist
quodlibet-full
quodlibet-full.dist
quodlibet-full.doc
quodlibet-xine-full
quodlibet-xine-full.dist
quodlibet-xine-full.doc
soco-cli
soco-cli.dist


tint2 with environment variable err

The first startup was successful, great. but...
If start with [docker start] from the second time onwards, an error will occur.

C:\Windows>docker start -i computer_use
Xvfb is already running on display :1
starting tint2 on display :1 ...
tint2 stderr output:
tint2: Using glib slice allocator (default). Run tint2 with environment variable G_SLICE=always-malloc in case of strange behavior or crashes
tint2: could not open display!
Is it not possible to [docker start]?

@oyamadasan this is currently not supported - docker start re-executes the entrypoint, which attempts to restart the already started startup sequence. feel free to submit a PR if you have a fix here.


Acquisition on all modules impossible in IDE Spyder with Python 3.7.3

But everything works with the Python interpreter

I had the same problem, but for me (on Python 3.6 at least) it worked to switch from the master branch to develop-0.9.3, which is quite ahead in terms of commits


RuntimeError: Only tensors, lists, tuples of tensors, or dictionary of tensors can be output from traced functions

Describe the issue:
When I used NNI to compress the DLA34 network model,
executing ModelSpeedup(model, dummy_input=dummy_input, masks_file='mask_fairmot.pth')
there is a wrong:
"File"/home/user /. Conda/envs/yang_pytorch/lib/python3.7 / site - packages/torch/jit / _trace py ", line 940, in trace_module_force_outplace,RuntimeError: Only tensors, lists, tuples of tensors, or dictionary of tensors can be output from traced functions'.
Some people have proposed that my model returned is a dictionary. I would like to know how to convert my model from dictionary format to Tensors format.
The return of my model is:
def forward(self, x):
x = self.base(x)
x = self.dla_up(x)
y = []
for i in range(self.last_level - self.first_level):
y.append(x[i].clone())
self.ida_up(y, 0, len(y))
output = {}
output['hm'] = self.getattr('hm')(y[-1])
output['wh'] = self.getattr('wh')(y[-1])
output['reg'] = self.getattr('reg')(y[-1])
output['id'] = self.getattr('id')(y[-1])
    return [output]

thanks a lot

I think just return output may work, and you need to check if the values in output are all tensor.


OGC API: misc improvements

More robust lookup of collection document: calling getCollectionInfo on the sample-data-2 fixtures was failing without this logic (no rel=self link is present on collections)
Base documents of the endpoint (root doc, collections) are now queried lazily instead of eagerly at the endpoint creation; this will avoid unhandled promise rejection and offer clearer errors
A new bulkDownloadLinks property was added to collections: these are links of type "enclosure" which do not offer pagination, filter etc. but only a bulk download; the property is a map between the mime type and the url

sorry @ronitjadhav this might generate a few conflicts with #34


CARBONDATA-133 remove show load and delete load command

Fixed #https://issues.apache.org/jira/browse/CARBONDATA-133

pls close this PR

not required since already handled in #899

pls close this PR


Provide indication of copy quality

Reveal in the UI how well the copy reads at a Grade Level, buzzwords and length.  Can be JS.

Some explorations of how we could do this:

Some comments:

Change over time could be really interesting in reinforcing why its important to write well but only if there is a correlation. Who knows, maybe there isn't a correlation.

How many different stats we want might dictate where they might live in the interface.

I kind of like the idea of making the statistics human readable (in sentences).

We could highlight things in red when they go above a certain grade level or a certain wordcount,

What other things do we want to add in here? (mobile vs desktop usage, etc)

https://github.com/btford/write-good
Javascript based 'writing linter'. Beyond the library itself, lots of good resources in here around this topic.


Small docs clarification when escaping in SecretTemplate

This fails:
  template:
    data:
      key: $(.secret.data.my\.key)

It must be surrounded by quotes, this succeeds:
  template:
    data:
      key: '$(.secret.data.my\.key)'

@Samze snippets above are equivalent from yaml perspective. what is the failures you are getting?

Yeah my bad, these are functionally equivalent 👍


LFS-895: Move + button to the bottom of the Admin's Users List and Group List pages

tested, waiting for the final code changes to approve it

Can we improve the look of the Save button while saving is in progress when label font colour is black?

re-tested


Add permission to set Pod finalizers for controller

Error seen:
type: 'Warning' reason: 'InternalError' failed to bind resource to pod: failed to get or create data plane ConfigMap knative-eventing/kafka-source-dispatcher-2: configmaps \"kafka-source-dispatcher-2\" is forbidden: cannot set blockOwnerDeletion if an ownerReference refers to a resource you can't set finalizers on: , <nil>","knative.dev/pod":"kafka-controller-bfcdc9d66-mfkrh"}

Signed-off-by: aavarghese<EMAIL_ADDRESS>This is needed to set blockOwnerDeletion for Pod owner reference.
Part of #1537

Proposed Changes

Release Note

Docs

cc: @pierDipi


Lite Mode

Is is possible to add lite mode? just like specify in this url: https://developers.google.com/maps/documentation/android-api/lite

No.

Why not?

Because Google does not provide the lite mode for iOS

Yes but it will be very useful to have it for android... could you provide any input on how to have this implimented?

Please learn how to folk git repository, then how to write Java code.

I know how to write java... But can you provide some head start?

I recommend you should learn how to develop the cordova plugin.
https://cordova.apache.org/docs/en/7.x/guide/hybrid/plugins/index.html

If you are already know, please follow the steps.
First of all, please fork this repository.
https://help.github.com/articles/fork-a-repo/
Then clone it to your local, and you can modify the code.

Thank you very much I will look into it later.


document use cases in more detail

Signed-off-by: JENNIFER RONDEAU<EMAIL_ADDRESS>Per discussion in #ark-dr on K8s Slack. Plus a couple of links to encourage attention/reading. n.b. this PR is for master -- will update gh-pages and site separately, probably next week. Process is still a bit manual and fiddly.
@ncdc PTAL

Closing in favor of an issue so I can revisit and rework more thoughtfully.


Quickly toggling expansion panel leaves it in undefined state

Bug, feature request, or proposal:
Bug
Current behavior

Reproduction
http://plnkr.co/edit/PaoRWzPQe29y1x7BBnvr?p=preview
Is there anything else we should know?
Maybe similar to https://github.com/angular/material2/issues/4932?

Note that to replicate

Start with the panel open
Quickly close -> open

Another way is to slow down the animation and toggle between accordions.

I think you've mentioned it elsewhere, but here's a plunker showing that it is likely a bug in<EMAIL_ADDRESS>http://plnkr.co/edit/1Lk0pj6B9V1lCFqSlms8?p=preview
Do the same thing to replicate:

Start with panel open
Quickly close -> open

The original issue seems to be fixed with the latest animations updates, but the toggle icon still rotates the wrong way when quickly closing and then opening.

Closing since I'm having a hard time reproducing it anymore and even in that case the issue is on the @angular/animations side.


Migrate tests in mina_base to Alcotest,

It has recently been accepted to write unit tests with Alcotest library rather than with JaneStreet's inline test ppx. Because mina_base still seems to require more testing, let's migrate it first to Alcotest, adding more tests and improving existing ones along the way.

The refernces PR does half of this, we'll put the rest on hold for now.


[Bug] mtree_mutate incorrectly strips root directory prefixes

It looks like #852 causes issues with strip_prefix in mtree_mutate when stripping the root directory.
https://github.com/bazel-contrib/bazel-lib/blob/250c835b5f39f239de3ae2d97b3273d4c7390c5b/lib/private/modify_mtree.awk#L17-L20
This causes a / to be added to the first keyword in the mtree specification.
Steps to reproduce
# BUILD

genrule(
    name = "test.mf",
    outs = ["test.mf.spec"],
    cmd = "printf 'root_dir/ uid=0 gid=0 time=0 mode=0755 type=dir\nroot_dir/child_dir/ uid=0 gid=0 time=0 mode=0755 type=dir' >$@",
)

mtree_mutate(
    name = "test_mutate.mf",
    mtree = ":test.mf",
    strip_prefix = "root_dir",
)

Result:
# test_mutate.mf.spec

uid=0/ gid=0 time=0 mode=0755 type=dir
child_dir/ uid=0 gid=0 time=0 mode=0755 type=dir

The entry should probably be deleted unless package_dir is provided. I'm not very familiar with awk syntax, but another check for whether package_dir is provided and a more complex check on L18 would probably fix this.

Sorry for the delay - @j-eid you could help us out with a fix here if you have time to make a (red) PR that adds the missing test case. Then we just need to add the fix as another commit on that PR.


Automate - Vm reconfigure event passing invalid data.

One line removed from Automate method.
https://bugzilla.redhat.com/show_bug.cgi?id=1322101

@tinaafitz
Please Review

@lfu
Please review

@billfitzgerald0120
Looks good.

@billfitzgerald0120
Looks good.


Pasting username/password fails

You can't paste a username/password at the moment. When you do, it provides an error message that you need to login to paste an image.
Our paste handler is on the window.paste, so it's like the cause of the problem.

Already fixed.


Update TOC.md

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

MicrosoftCollaboratePortal/TOC.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


lock canary version of regalloc2 to 0.9.3

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

They yanked the bad crate. No need for this now.


[Bug] Refresh Error

Cannot see over 8 likes on google chrome. When ever I scroll it throws an error and reverts back to blured/popup.

Could you please try the development version?

perfect


Provide documentation over observability stack and ADR

The following adds documentation over how we provide observability and monitoring of our systems inside Howard. It also provides ADR for the different services that were chosen to enable observability.

also fixed end of files violations that were pushed directly to main.

Wow! Nice docs and great improvement on our observability stack! Well done sir


Add eg-0.7 directory

Supersedes #15 now that e-t 0.5 is released.

@bugadani wins again

What did I do now? :D

A subtle reference to https://github.com/embedded-graphics/examples/pull/15#issuecomment-894657385 ;)


Review protocol 2.1

DON'T MERGE
Review comments on "protocol-2.1".
UNTESTED / UNREVIEWED.

Out of date... need to rerun cppcheck on the new code.
cppcheck --enable=all --force --inconclusive .
and grep for "]".

Transcribe comments into PR#15 and close

Closing -- as have transcribed comments into a direct review of https://github.com/avnet-iotconnect/iotc-c-lib/pull/15


Submitting an invalid bearer token value can cause a 500 in the auth filter.

The auth process will attempt to base64 decode bearer tokens without validation or a try/catch leading to 500s where the error should be in the 4xx range.

Ellie, I didn't look at this but is this a fix that should go into the client?  We have InvalidTokenException... is it not being caught and converted to 400 properly in AuthFilter?

It probably should go into the client.  I will move the ticket there and update.


Place RP convention scenarios in context

Fixes #5831
Thanks @csharpfritz for flagging this problem! :rocket:

@scottaddie Updated.
Reminds me that I still wish for a better solution for documenting the extension methods on WebHost in all of these examples. I have a kernel of a idea forming about it, but I think it's fair to say that we're kind'a busy at the moment to sweat it. I'll make a note, and we can all discuss it in a few weeks/months.

@guardrex Send an email with your ideas when you have some time. I'm interested in what you're thinking.


Mesh joining for unstable Bullet collision compounds.

Motivation and Context
Related to Issue #301
Blocked by: PRs #309, #310. (diff includes these)
This PR addresses remaining physics instabilities when loading hierarchal meshes as Bullet rigid object by introducing optional but default mesh joining for more stable simulation of compound objects.
Mesh joining:
Creating compound shapes as combinations of convex components can be an effective way of introducing macro concavities efficiently into simulation. PR #310 added this functionality by computing convex hulls for each mesh in a loaded collision mesh hierarchy and combining them into a single btCompoundShape object. However, when individual convex hulls overlap significantly, this treatment can result in unstable simulation artifacts.
This is demonstrated by simulation of a compound box shape with 2 convex sub-boxes only slightly offset from one another:

To remedy this, meshes can be joined into one convex hull, losing any concavity for improved stability and efficiency:

The mesh join option is added to the physics object config with default true for stability.
How Has This Been Tested
Local testing.
Simulating random hierarchal objects after mesh joining and COM shift:

Types of changes

[x] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

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

@aclegg3 what's the status of this PR, is it good to land?

Waiting on C++ test coverage for new functions. Functionality is fine. I'll look into this soon.

Codecov Report

Merging #317 into master will increase coverage by 0.02%.
The diff coverage is 69.69%.

@@            Coverage Diff             @@
##           master     #317      +/-   ##
==========================================
+ Coverage   56.89%   56.92%   +0.02%
==========================================
  Files         153      153
  Lines        7067     7090      +23
==========================================
+ Hits         4021     4036      +15
- Misses       3046     3054       +8

Impacted Files
Coverage Δ

src/esp/assets/Attributes.cpp
34.25% <100%> (+2.61%)
:arrow_up:

src/esp/assets/ResourceManager.cpp
58.31% <16.66%> (-0.43%)
:arrow_down:

src/esp/physics/bullet/BulletRigidObject.cpp
55.46% <78.26%> (+0.33%)
:arrow_up:

Continue to review full report at Codecov.

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


Settings seems broken with some not recognise

Changing color variants does not reflect immediately as it used to before the update.
Some settings are not recognized, accent being one - error was thrown.

@OctoD Can you check?

@equinusocio doing that works like a charm. All back to normal! Thank you.

One issue still persists is that changes are not directly reflected. Require restart for such changes to take into affect.

Could you please provide us vsc version and the exact actions you are performing? Thanks in advance.

@awesomenaz @OctoD Changing the accent color require a window reload.

Current vsc version 1.14.2.
The steps utilised as specified by docs.
Options for color scheme as referred above were missing.
Settings at initial point were throwing error as not found.
At the moment after resetting all setings related to material theme from configuration file, it works. But changing colors are still not reflecting immediately.

Such a hemorrhoid, it would be better not to be updated

@awesomenaz We can't avoid refresh the window when the accent color is selected, but we can auto-reload the window without ask the user confirm. About the old settings, we are working on it. :)

@antonreshetov Please provide useful information or you can follow this steps:

Remove this theme
Leave this topic and this repository too.
When leave, remember that we are working for free, for you.

Thank you, @equinusocio was just wondering why some behavior related to the theme was changed, while docs remained the same instead of reflecting the changes.
All in all, at the moment things are working as usual.
Thank you again for the assist!

@awesomenaz What do you mean? The README was update to reflect the new activation system. About accent colors nothing's changed. Next patch the accent selection will not ask for user confirm.

@awesomenaz The package is cool, not cool one, that now to quickly change the theme I need to go to Material Theme: Settings before I just clicked cmd+k+t and select theme

@antonreshetov This was a technical decision. By this way we can have more control over settings, customisations and we can provide more material theme settings easily. By the way, how many time you select the material theme? i think one. Please stop this OT.

@equinusocio What I meant was the changes made to the settings - where it was throwing errors. Once all previous settings removed and changes made, things are working fine. My apology for the misunderstanding.

@awesomenaz The warning and errors are a bug :) removing settings is just a quick fix until we figure what happen. :)

@antonreshetov If something goes wrong just let them know, if the changes are too much, use another theme. Don't have to be rude.

@equinusocio Yes, this matter is solved for now. Have a great day and thank you again.

Peace for everyone, keep on hacking :)

I have Mac OS 10.11.16 - "Material Theme: Settings" doen't work. Setting was wrote in settings.json, but theme doesn't change

@equinusocio I have the same issue as @sd-design. Fresh VS Code install first time using this theme. Nothing changes for me using the Material Theme Settings menu option.

@sd-design @m19 have you tried these steps?
https://github.com/equinusocio/vsc-material-theme/issues/65#issuecomment-317367014

@equinusocio thanks! I just edited my previous post. The issue was not having set workbench.colorTheme to Material Theme.
First time using VS Code, wasn't aware of this.


Fix: Description should not be required for posting flag

Expected: Description can be blank when adding a new feature flag
Actual: 400 bad request, description is required

QA: works


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_8397ec85-6a40-49ce-8a97-980d678fe034/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_8397ec85-6a40-49ce-8a97-980d678fe034.git

Docs Build status updates of commit 46d890a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/VSC-Service-Account/docs-archive-test-source' (branch: Release_Archive_master_2020-05-25-15-12-17) is not authorized.. It looks like this might be the first time you’ve submitted a pull request to this repo. Before you can run builds in a pull request, you have to enable your GitHub account in Docs by visiting https://opportal-sandbox.azurewebsites.net. After you enable your account, close and then reopen this pull request to start another build.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Now y'all can shut up about Lua

rude but true

useless/joke pr, should be closed.

useless/joke pr, should be closed.

the only reason I did this was to fix an issue if people did not switch their forks of linc_luajit

on second thoughts, I'll close this because I made it out of annoyance from the issues

but I hope something could be done to prevent them if people don't switch forks

useless/joke pr, should be closed.

the only reason I did this was to fix an issue if people did not switch their forks of linc_luajit

its not an issue it's haxelib being haxelib, you had to specify that you want to download from the master branch, the command would go like this:
haxelib git linc_luajit https://github.com/superpowers04/linc_luajit master

useless/joke pr, should be closed.

the only reason I did this was to fix an issue if people did not switch their forks of linc_luajit

its not an issue it's haxelib being haxelib, you had to specify that you want to download from the master branch, the command would go like this:
haxelib git linc_luajit https://github.com/superpowers04/linc_luajit master

good to know


Update Rust Nightly Version

Fix a new Rust Nightly version

Auto-generated PR

Pull Request Test Coverage Report for Build 674796202

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.006%) to 82.126%

Files with Coverage Reduction
New Missed Lines
%

operators/src/source/csv.rs
1
87.92%

operators/src/source/ogr_source.rs
1
92.75%

Totals

Change from base Build 657570475:
-0.006%

Covered Lines:
14804

Relevant Lines:
18026

💛  - Coveralls


Add structure function analysis functionality

The task is add the possibility to calculate structure function squared in order to enable simple AGN science calculations.

Added in https://github.com/lincc-frameworks/lsstseries/pull/16


Improve user feedback on configuration mistakes

The only issue I've found with it so far is, the sink can sometimes crash an not recover if you misconfigure the sink.json file.
For example: - i configured it to look for a variable named foo, but forgot to add to the payload from the C# worker.
Crashed the sink. took me a while to debug.

Yeah I thought about that, but I wasn't sure what was the "best" way to handle errors, i.e. one record fails to be handled, what happens if we keep processing further records? I assume we never process that record. How are users expecting this to behave? Where should we report the error? If you know of Kafka Connect power users who would have insight there, that'd be great!


aiven_gcp_vpc_peering_connection get "invalid resource id"

What happened?
When i try to setup vpc peering with aiven_gcp_vpc_peering_connection i got an error "invalid resource id"
i try with aiven_vpc_peering_connection and i didn't have error
you can try with this code
provider version 3.7.0
resource "aiven_project_vpc" "vpc" {

  project      = aiven_project.aiven_project["integration"].project
  cloud_name   = "google-europe-west1"
  network_cidr = "<IP_ADDRESS>/24"

  timeouts {
    create = "5m"
  }
}

resource "aiven_gcp_vpc_peering_connection" "vpc-peering-gcp" {

  vpc_id         = aiven_project_vpc.vpc.id
  gcp_project_id = "my_gcp_project"
  peer_vpc       = "my_gcp_network"

  timeouts {
    create = "10m"
  }
}

debug logs
2022-10-03T16:49:16.687Z [ERROR] provider.terraform-provider-aiven_v3.7.0: Response contains error diagnostic: tf_rpc=PlanResourceChange @caller=github.com/hashicorp/terraform-plugin-go@v0.14.0/tfprotov5/internal/diag/diagnostics.go:55 diagnostic_summary="invalid resource id: " tf_provider_addr=provider tf_resource_type=aiven_gcp_vpc_peering_connection @module=sdk.proto diagnostic_detail= diagnostic_severity=ERROR tf_proto_version=5.3 tf_req_id=85cae062-97dd-2d63-2f41-1db24d24fa5c timestamp=2022-10-03T16:49:16.687Z

What did you expect to happen?
I expect to have the plan working

What else do we need to know?
perhaps link to this issue but not exactly the same behaviour
https://github.com/aiven/terraform-provider-aiven/issues/901

The Issue is close but when the new provider version will be released ?

@Serpentiel @ivan-savciuc
The Issue is close but when the new provider version will be released ?


add robbery minigame

I spent a couple of hours yesterday afternoon and made this microgame. Butano was super easy to use without any previous experience! Maybe I'll improve my submission before the deadline, time permitting.

The idea of the game is to grab all of the pumpkins, but you slow down as your bag gets heavier.

Thanks!

Maybe I'll improve my submission before the deadline, time permitting.

Then I'll wait until deadline before merging.

And please, remember to add the license txt file for all graphics.

And please, remember to add the license txt file for all graphics: https://github.com/evanbowman/microjam23/tree/pumpkin-raid-squashed-commits#microgame-rules

Ok, I added it.

Ready to be merged?


Update MD file on new branch 2017-20-14_03-11-55

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Is it appropriate to set initial_lr to 1 ?

in config.yaml:
optimizer: Adadelta
lr: 1
lr_decay: cosine
step_ratio: 10
step_decay: 5
eps: 1e-6
weight_decay: 1e-4
beta: 0.9
in training,py:
new_lr = 0.5 * (1 + math.cos((current_step + 1 + (current_epoch - 1) * steps) * math.pi / (200 * steps))) * initial_lr
Did you set  set initial_lr to 1 ?
On my own data set, the "eval_ExpRate" fluctuates greatly.

in config.yaml:
optimizer: Adadelta lr: 1 lr_decay: cosine step_ratio: 10 step_decay: 5 eps: 1e-6 weight_decay: 1e-4 beta: 0.9
in training,py: new_lr = 0.5 * (1 + math.cos((current_step + 1 + (current_epoch - 1) * steps) * math.pi / (200 * steps))) * initial_lr
Did you set set initial_lr to 1 ? On my own data set, the "eval_ExpRate" fluctuates greatly.

Yes, the initial_lr is set to 1, since by default the lr of Adadelta optimizer is set to 1. For CROHME and HME100K, this setting can reach the best results in our experiments.  If it doesn't perform well on your own dataset, consider using a smaller lr or changing the optimizer(e.g., Adam).

in config.yaml:
optimizer: Adadelta lr: 1 lr_decay: cosine step_ratio: 10 step_decay: 5 eps: 1e-6 weight_decay: 1e-4 beta: 0.9
in training,py: new_lr = 0.5 * (1 + math.cos((current_step + 1 + (current_epoch - 1) * steps) * math.pi / (200 * steps))) * initial_lr
Did you set set initial_lr to 1 ? On my own data set, the "eval_ExpRate" fluctuates greatly.

Yes, the initial_lr is set to 1, since by default the lr of Adadelta optimizer is set to 1. For CROHME and HME100K, this setting can reach the best results in our experiments. If it doesn't perform well on your own dataset, consider using a smaller lr or changing the optimizer(e.g., Adam).

Thx.


Switch: Set aria-checked value to "true" and "false"

Description
Previous implementation was passing a boolean in as the attribute value.  New implementation passes in a string.
References  #2143
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How Has This Been Tested?
Add e2e checks, update unit tests
https://deploy-preview-2450--moduswebcomponents.netlify.app/?path=/story/user-inputs-switch--medium
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

This is an improvement so I'll get this merged - thanks!
Still things to fix with switches though:

Should really change aria-checked to just checked
labels should match ids set on the input
screen reader support is currently not working at all for this component.


Allow "Exclude" as an alias for "Remove"

Background
MSBuild now allows developers to remove an item statically via the use of "Remove", previously you could only do this within a target.
<ItemGroup>
    <Compile Include="*.cs" Exclude="DoNotCompile.cs" />
    <Compile Remove="AlsoDoNotCompile.cs" />
</ItemGroup>

This is quite powerful and allows you to remove an item statically, without needing to change the original item declaration. For example, a wildcard could be implicitly imported in a props file, and the Remove in a project file.
Problem
The name "Remove" makes sense in the context that there's an existing feature, previously we just allowed it within a target, we now allow it everywhere.
In terms of symmetry of terms however, the name does not make sense. You "add" and "remove" files from a list, or "include" and "exclude" files from a list. You do not "add" and "exclude" files from a list or "include" and "remove" files from a list.
Given existence of "Include" already, naturally developers are going to gravitate towards using "Exclude" by itself (I personally ran into this), only to be told that it's not valid syntax.
Proposal
Based on above, we should allow the use of "Exclude" in the same places that you can "Remove", ie the above example could become:
<ItemGroup>
    <Compile Include="*.cs" Exclude="DoNotCompile.cs" />
    <Compile Exclude="AlsoDoNotCompile.cs" />
</ItemGroup>

Or in a target:
<Target Name="RemoveFiles">
    <Compile Exclude="AlsoDoNotCompile.cs" />
</Target>

This means when Exclude is on the same item as a Include, it excludes items from the Include. When Exclude is by itself, it excludes items from the current list of all items.
Compatibility
Given it has never been valid to use 'Exclude' by itself without an 'Include' - this change should not have an impact on compatibility.
This issue was filed on behalf of feedback by @shanselman.

tag @Sarabeth-Jaffe-Microsoft @rainersigwald @cdmihai and @AndyGerlicher

What if we wanted to support removing all items except the ones that match a pattern?  Currently we don't support this, but if we did I'd expect it to look like this:
<Compile Remove="obj\**\*.*" Exclude="obj\generated\*.cs" />

That would mean to remove all compile items from the obj directory except .cs files in obj\generated.
But if Exclude can be an alias for Remove, then this could start to get confusing.  What do you think?

Very good point @dsplaisted. I'd be for having Exclude scoped down to mean filtering the applicability of the current item operation. And we can have it for all item operations (include, remove, update, keepmetadata, updatemetadata).
If exclude is the only attribute, then it would be interpreted as an exclude for a global Update. See #955

I'm not sure what the right thing to do is.  I think the request coming from @shanselman is reasonable, and is how people would generally expect it to work.  When you think about the details and edge cases though, it starts to get confusing.

If exclude is the only attribute present in the item element, then it would be interpreted as an exclude for a global Update. See #955

This follows naturally from the existing behavior and the changes we are making, but I think it ends up in a place that would be confusing, or at least not the expected behavior for most people.

I'm not sure about backward compatibility but I think it  would be clearer if had "Except" instead of "Exclude"  to mean filtering the applicability of the current item operation. Then we could use "Exclude" instead of "Remove" or allow both
<ItemGroup>
    <Compile Include="*.cs" Except="DoNotCompile.cs" />
    <Compile Exclude="AlsoDoNotCompile.cs" />
</ItemGroup>

<Compile Exclude="obj\**\*.*" Except="obj\generated\*.cs" />

I like @cannn's suggestion. Except seems like a more suitable term for modifying the main item operation. Exclude could then be aliased to Remove as a main item operation. Back-compat would be fine, since we'd bake in the rule that Exclude can only appear besides remove, and then it's aliased to Except.
<Compile Include="*.cs" Except="DoNotCompile.cs" />
<Compile Remove="**/bin/**/*.cs" Except="DoNotRemoveMe.cs" />

<!-- Exclude as an alias to Remove -->
<Compile Exclude="**/bin/**/*.cs" Except="DoNotRemoveMe.cs" />

I agree that Except is a better name for what Exclude does. But I'm not in favor of adding it as an alias now for language-complexity reasons. Under what circumstances should the API choose one over the other when manipulating a project? How would a user familiar with the current rules interpret Except? And what does the error look like if you have
<Compile Remove="**/obj/**" Exclude="**/bin/**/*.cs" Except="DoNotRemoveMe.cs" />

API favor: We'd champion Except as the new Exclude, so we'll change the APIs to use Except instead of today's Exclude.
What if both Exclude and Except appear for an Include operation. We'd issue a warning saying the preferred modified is Except, but we accept both so we merged the two. Or just error.

What if Add=".." could be used instead of Include? Then the meaning of other existing attributes wouldn't need to change and the verbs would be Add, Update and Remove with Include staying around for compatibility.

At this point, I believe folks are used to the way things work and changing this would be a breaking change. Closing.


Add two specs for choosing platform specific gems

This is a follow-up to #5337. It adds two specs for choosing platform specific gems. The first spec succeeds, but the second spec fails currently.
What was the end-user problem that led to this PR?
 The intention behind this tag is to give bundler a chance to select the most suitable gem for the given platform and ruby version. Unfortunately it doesn't work.
This leads to issues for all Windows users whenever a new Ruby version comes out. The intended effect (to make the install of gems easy) turns to the opposite, because the platform specific gem effectively blocks the installation of the "ruby" platform gem.
These issues can be seen at many popular gems with C extensions: ffi #598, nokogiri #1706, pg
What was your diagnosis of the problem?
The first test case doesn't offer a x64-mingw32 gem in the latest version. Bundler therefore selects the latest "ruby" platform gem so that the installation succeeds. This is OK and expected.
The second test is very similar, but offers a x64-mingw32 gem in addition. This additional gem however is limited to ruby versions not matching the running ruby version. Bundler should exclude this gem therefore and should select the "ruby" platform gem as in the first test. However it doesn't resolve to the "ruby" platform gem, but to an older x64-mingw32 gem, which wasn't tagged by a required_ruby_version constraint to that point in time (and will not work with the latest ruby version in practice). That is the current situation with ffi and nokogiri.
The current workaround is to set force_ruby_platform to true, but this has several negative consequences since it's a global and not a per-gem setting. Also it has to be enabled manuelly by each (Windows) user. Insofar it's not a general solution.
What is your fix for the problem, implemented in this PR?
Why did you choose this fix out of the possible options?
I didn't dig deeper into bundlers internals, but my hope is that someone with more knowledge of bundler can solve this issue that way or at least give me a pointer where I could start to fix this.

Any updates here? The problem remains in place after 2 years!
@larskanis @segiddins @indirect

I've implemented these added specs in #7522.
#7522 includes specs added in this pull request.

Closing in favor of #7522 then. Thanks @kou and everybody helping out here!


Support for MacOS

What OS was AIF360 built on?  I'm attempting to run on a Macbook Pro with the latest Mac OS however I am getting errors when attempting to install the module 'cvxpy'.  During the 'pip install cvxpy', I get numerous compilation errors:
  warning: include path for stdlibc++ headers not found; pass '-std=libc++' on the command line to use the libc++ standard library instead [-Wstdlibcxx-not-found]
  In file included from cvxpy/cvxcore/src/cvxcore.cpp:15:
  cvxpy/cvxcore/src/cvxcore.hpp:18:10: fatal error: 'vector' file not found
  #include <vector>
           ^~~~~~~~
  1 warning and 1 error generated.
  error: command 'gcc' failed with exit status 1

  ----------------------------------------
  Failed building wheel for cvxpy

Getting closer.  Found first part of a solution here: https://www.cvxpy.org/install/index.html
Switched from "pip install cvxgrp" to this:
conda install -c conda-forge lapack
conda install -c cvxgrp cvxpy

So now cvxgrp is installed but getting this error:
/anaconda/envs/dlaas_cos/lib/python3.6/site-packages/aif360/algorithms/preprocessing/optim_preproc_helpers/opt_tools.py in <module>()
     15 import numpy as np
     16 import pandas as pd
---> 17 from cvxpy import Problem, Minimize, Variable, sum_entries,\
     18                   mul_elemwise, norm
     19 from logging import info, debug, warn

ImportError: cannot import name 'sum_entries'

What version of CVXPY is used by AIF360?  According to the current v0.4.8 release of CVXPY:
Several CVXPY atoms have been renamed: ...sum_entries is now sum...Due to the name changes, we now strongly recommend against importing CVXPY using the syntax from cvxpy import *.

https://media.readthedocs.org/pdf/cvxpy/stable/cvxpy.pdf

Hi! It looks like your most recent error is due to a newer version of cvxpy. We require 0.4.11. Are you using a virtual python environment to install packages?

Yes.  I'm using conda and used this command. I expect that should have installed the most recent release.
conda install -c cvxgrp cvxpy

As above though, are you sure you're using v0.4.11?  The PDF above says that sum_entries was rename to sum in v0.4.8 and also recommends not using from cvxpy import *.

And I'm guessing that since v0.4.8 was installed by conda install -c cvxgrp cvxpy then v0.4.11 isn't yet available via conda install?

Sorry, I wasn't clear. I meant did you create a new environment like conda create --name aif360 python=3.6.
The latest version of cvxpy is incompatible with our code as you pointed out. It appears that 0.4.11 is not available on conda though. You could try a lower version but we've only tested with 0.4.11. It should be available from through pip though: pip install cvxpy==0.4.11
If you still encounter the gcc error you can try conda install -c cvxgrp libgcc

Actually, the original error may have occurred if you haven't installed the command line tools on that machine before. You can do this by xcode-select --install. This should give you things like gcc, etc.

Starting over again with a fresh environment and these commands:
$xcode-select --install
$conda install -c cvxgrp libgcc

This seems to have worked slightly better.  Now getting this error:
...
In file included from /usr/local/include/stdint.h:59:
    In file included from /usr/local/include/stdint.h:59:
    In file included from /usr/local/include/stdint.h:59:
    In file included from /usr/local/include/stdint.h:59:
    In file included from /usr/local/include/stdint.h:59:
    /usr/local/include/stdint.h:2:10: error: #include nested too deeply
    #include <stddef.h>
             ^
    /usr/local/include/stdint.h:59:11: error: #include nested too deeply
    # include <stdint.h>
              ^
    /usr/local/include/stdint.h:72:11: error: #include nested too deeply
    # include <sys/types.h>
              ^
    /usr/local/include/stdint.h:76:10: error: #include nested too deeply
    #include <limits.h>
             ^
    /usr/local/include/stdint.h:82:11: error: #include nested too deeply
    # include <inttypes.h>
              ^
    5 errors generated.
    error: command 'gcc' failed with exit status

Any ideas?  Can't find any guidance/workarounds in Google.

Also looks like v0.4.* is 2.5 years old.  Seems easiest for all users if AIF360 is updated to the latest version as my guess is most people will hit the same issues as me.
https://pypi.org/project/cvxpy/#history

That's odd. Which command caused that error? I don't have that header file on my machine so it might have been installed by something else. According to https://github.com/SOHU-Co/kafka-node/issues/881, you might have luck just renaming it.
Also, when I installed on a mac, I was able to do pip install cvxpy==0.4.11 with just the xcode gcc installed. I didn't need to install libgcc from cvxgrp.

OK.  Starting over again and just editing the AIF360 file to use the latest version of cvxpy.  Just ran this:
$conda create --name aios python=3.6
$source activate aios
$conda install -c cvxgrp cvxpy

Then I edited the opt_tools.py from this:
from cvxpy import Problem, Minimize, Variable, sum_entries, mul_elemwise, norm

to this
from cvxpy import Problem, Minimize, Variable, norm
import cvxpy as cvx

and replaced sum_entries to cvx.sum and mul_elemwise to cvx.multiply

OK, finally found the solution.  Conda had the older v0.4.9 version of cvxpy so installed it as below and the notebook now works:
$conda create --name aios python=3.6
$source activate aios
$conda install -c cvxgrp cvxpy=0.4.9


Roadmap v0.4

Introduce the basic optimizer module. First of all, we should consider how to design the architecture to support RBO.
And then we need to add more rules to satisfy the basic query requirements, such as: ColumnPruning, PushDownPredicates, LimitPushDown, Constant folding ...
Meanwhile, we need to introduce optimizer regression test framework to verify plan tree
Let's start!

Heuristic Optimizer:

[x] #43

pushdown predicates:

[x] #45
[x] #47

limit pushdown:

[x] #48

column pruning:

[x] #50

constant folding:

[x] #52

Optimizer regression test:

[x] #53

explain support:

[x] #57

This milestone is already done. Let's go to the next step 🎉


Add Travis

Fixes https://github.com/LLK/scratch-render-fonts/issues/5

@rschamp now you're both requesting changes and reviewing. Did that work to request your review?


Roll Flutter Engine from 5da44b9aafdd to e48e34522acd (1 revision)

https://github.com/flutter/engine/compare/5da44b9aafdd...e48e34522acd
2023-06-11<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from E7gwQ9L15jziMUKha... to UxuMYdEq-rI5WqUKJ... (flutter/engine#42744)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from E7gwQ9L15jzi to UxuMYdEq-rI5
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Cannot create testbed in interactive mode

I'm an absolute beginner of pyats. I installed pyats through pip install 'pyats[full]'
I was trying to create the testbed in interactive mode (genie create testbed interactive --output testbed1.yml), but always got below error:
 [root@localhost pyats]# genie create testbed interactive --output testbed1.yml

please make sure pyats.contrib is installed. If not, please do 'pip install pyats.contrib'.

invalid option: interactive

I'm pretty sure that pyats.contrib is installed with version 22.3:
 [root@localhost pyats]# pip freeze | grep pyats.contrib
pyats.contrib==22.3

And I did 'pyats version update' as well, but still no luck.
What should I do?

Can you please run pip list | egrep "ats|genie|unicon" and share the output?

Can you please run pip list | egrep "ats|genie|unicon" and share the output?

Hi Domachad,
[root@localhost ~]# pip list | egrep "ats|genie|unicon"
genie                        21.3.1
genie.libs.clean             21.3
genie.libs.conf              21.3
genie.libs.filetransferutils 21.3
genie.libs.health            21.3
genie.libs.ops               21.3
genie.libs.parser            21.3
genie.libs.sdk               21.3
pyats                        21.3
pyats.aereport               21.3
pyats.aetest                 21.3.1
pyats.async                  21.3
pyats.connections            21.3
pyats.datastructures         21.3
pyats.easypy                 21.3
pyats.kleenex                21.3
pyats.log                    21.3.1
pyats.reporter               21.3
pyats.results                21.3
pyats.tcl                    21.3
pyats.topology               21.3
pyats.utils                  21.3
unicon                       21.3
unicon.plugins               21.3

I don't see pyats.contrib in this list. Are you sure you have it installed?

Thank you domachad. My bad, I forgot that I previously installed pyats in a virtual environment, the output of pip list | egrep "ats|genie|unicon" is not from that venv, issued solved after I activate the venv.


Help for Open-ExcelPackage example typo

The help for Open-ExcelPackage example 2 appears to have an a where there was probably meant to be a ., between Worksheets and "sheet1".

I'll have a pop at this.

Looks like this has been fixed already in the repo?

Looks like it is in the https://github.com/dfinke/ImportExcel/blob/master/en/ImportExcel-help.xml. There is no automated way in the repo to re-generate the help though

I'll take a look at PlatyPS (I think that's what it was called) within the code tree to generate the help. I'll also look to see if there's a simple way of updating the help XML data when the MD files are changed. Have some ideas for this, but not sure how best to integrate with the automation.


Close query on API; Apply latest changes from Prometheus.

https://github.com/prometheus/prometheus/pull/7136#pullrequestreview-395804846

Done!

Thanks to @codesome


How do I include User relationships to response after authenticate

Hi,
My User has_and_belongs_to_many Profiles;
my question is: how do I include User.profiles to sign in callback.
thanks in advanced.

@ricardohonorio have you tried overriding the normal routes in the correct overridescontroller in order to send back a custom response?
Closing this in 7 days if we don't hear back from ya.


Add support for b2clogin hostnames

MSFT Recommends to use b2clogin.com hostnames for B2C applications instead of login.microsoftonline.com.
And given that MSAL.js is unable to validate b2clogin authority. This fix detects if the instance points to b2clogin.com page and set the validateAuthority to false accordingly.
Reference:
https://docs.microsoft.com/en-us/azure/active-directory-b2c/b2clogin
https://docs.microsoft.com/en-us/azure/active-directory/develop/msal-b2c-overview

Hi!!
This solution helped me to solve the problem! It would be great if it could be merged to the project! Any help i could provide for that? :D

Closing