Bridge request for current time

Bridge request
For testing of readers, it would be awesome to have a bridge, which gives out the current time and date, to see, if and when the reader updates the Feed.

General information

Host URI for the bridge (i.e. https://github.com/RSS-Bridge/rss-bridge):

Which of the following parameters do you expect?

[X] Title (datetime)
[ ] URI (link to the original article)
[ ] Author
[X] Timestamp
[X] Content (datetime)
[ ] Enclosures (pictures, videos, etc...)
[ ] Categories (categories, tags, etc...)

Options

[ ] Limit number of returned items

Default limit: 5

[ ] Load full articles

Cache articles (articles are stored in a local cache on first request): no

[ ] Balance requests (RSS-Bridge uses cached versions to reduce bandwith usage)

Timeout (default = 5 minutes, max = 24 hours): 5 minutes

Using RSS-Bridge is overkill. You can make simple php script for that.

I don't know, how to do that, so I created one anyways...
#2181


Conflict with mappings.

Hello,
I have the following mapping in my vimrc (among others, basically I exchange keys around as I use a french bépo keyboard)
noremap n l
noremap k n
noremap N L
noremap K N
And I can't figure out how to make the plugin to work with these. Any ideas?

To override it, you can put your mapping in ~/.vim/after/plugin and read :h after-directory

To override it, you can put your mapping in ~/.vim/after/plugin and read :h after-directory

It works. Thank you very much. I didn’t know about the "after-directory" functionality.


improvements

include the list of unknown barcodse inside the HTML
include the PNG in the HTML and therefore not outside
include ccs and js on a website

this is done in v1.0.3 (last one is a sequana issue)


How to send base64 encoded image data using POST request?

Hi Guys, How can any one let me know how to send base64 encode image data using post request?
I am using this I am not sure this is the correct way to use it or not.
$response = $this->client->post('URL',[
            'multipart' => [
                'name'     => 'image',
                'contents' => ['image'=>$base64]
            ]
      ]);

This throws exception of Type error: Argument 2 passed to GuzzleHttp\Psr7\MultipartStream::addElement() must be of the type array, string given
Or Do I need to use form_params?
$response = $this->client->post('URL',[
            'form_params'=>['image' => $base64]
      ]);

@SebTM  That is throwing an error of  Invalid resource type: array

Can you give us some more information what you are trying to do? What software/url are you calling? Can you link to documentation?

@SebTM  Any idea how I can achieve this?

Hey @Dhavalptel,
sorry didn't had time before to check it again. Here is a working example:
`
post($uriBase,[
    'headers'=>['Authorization' => 'Bearer '.$token],
    'multipart' => [
        [
            'content-type' => 'test/plain',
            'contents' => $base64,
            'name' => 'file',
        ],
    ],
]);
echo $response->getBody()->getContents();
`

> Result: {"Accepted":false,"Error":"NoFaceFound"}

@SebTM  Thanks Man, That is working I was wondering, I have already converted the image to base64 then why do I need to multipart it again?

Seems wired for me, too - also that file as name is just random picked and working. The documentation is really not that good here.

Yes that sucks, Anyway thanks man, I am closing this

You are welcome! :)


Can someone elaborate in this issue?

https://stackoverflow.com/questions/49047247/angular-5-leaflet-real-time-integration-update-markers-issue

Closing this as it is being discussed on stack overflow.


CSV parser with first 1kB block and wrong encoding causes unexpected behavior

Hello,
we validate user's CSV. One of is the encoding validation of a line.
Our validation method looks somehow like this
    def valid_and_parse_lines
      line_number = 2 # count start after headers

      loop do
        row = csv.gets
        break unless row
        line = line_initialization(row)
        add_error(line_number, line.errors.full_messages) if line.invalid?
      rescue ArgumentError => exception
        raise if exception.message != 'invalid byte sequence in UTF-8'
        add_error(line_number, 'This line contains invalid UTF-8 characters')
      ensure
        line_number += 1
      end

      csv.rewind
    end

The CSV is loaded this way
@csv ||= ::CSV.new(file || content, encoding: Encoding::UTF_8, headers: true)

What is the issue? If there is some ones of the lines of the first 1kB block (used by stdlib's CSV parser for some detection) contain some invalid lines it leads some lines are skipped.
Here is an example:
labels_invalid_iso8859-1_encoding.csv.zip
We always open the files in UTF-8, this CSV contains two lines with illegal character in ISO-8859-1. The second and fourth lines (after the header) contain the illegal character. In this case, only the second lines is detected, fourth one is unfortunately skipped. There is no problem with this if there are other invalid lines after the first 1kB block.
It seems the bug relates to these lines in csv.rb

https://github.com/ruby/ruby/blob/ruby_2_5/lib/csv.rb#L2046
https://github.com/ruby/ruby/blob/ruby_2_5/lib/csv.rb#L2039

Can you show a runnable Ruby script to reproduce this case?

@kou
Here is the demo
require 'csv'

class CSVDemo
  attr_reader :errors

  def initialize(file)
    @file   = File.open(file, 'r', encoding: Encoding::UTF_8)
    @errors = []
  end

  def csv
    @_csv ||= CSV.new(file, encoding: Encoding::UTF_8, headers: true)
  end

  def validate!
    line_number = 2

    loop do
      row = csv.gets
      break unless row
      # here can be other code which can cause different ArgumentError
    rescue ArgumentError => exception
      raise if exception.message != 'invalid byte sequence in UTF-8'
      errors << "The line #{line_number} contains invalid UTF-8 characters"
    ensure
      line_number += 1
    end
  end

  private

  attr_reader :file
  attr_writer :error
end

demo = CSVDemo.new('labels_invalid_iso8859-1_encoding.csv')
demo.validate!

puts demo.errors

# Returns:
# The line 2 contains invalid UTF-8 characters
#
# Expected:
# The line 2 contains invalid UTF-8 characters
# The line 4 contains invalid UTF-8 characters

labels_invalid_iso8859-1_encoding.csv is attached in https://github.com/ruby/csv/issues/25#issue-299878501

Thanks. I've fixed.


How to use this in SvelteKit?

That's it. My use case is for embedding Github gist for a tech blog. Width is pretty much set at 100%, it's the height that's tricky. Thank you.

@hi-brylle it's good form to post what you already tried and what exactly is not working.

I elected to simply use a syntax highlighting library for my use case of showing code snippets . I no longer need Github gist embedding, but thank you for replying. Closing this now.


refactoring partition functions

There are currently too many and not well organized functions for partition functions in moldb.py.
The new api, api.py, should have these functions in systematic and consistent ways.

At minimum, naming rules sho8uld be refined.
moldb.py
MdbExomol

QT_interp(T): interpolated value of Q(T)
qr_interp(T): interpolated value of q(T) = Q(T)/Q(Tref)

MdbHit

QT_iso_interp(idx, T):  interpolated value of Q(isotope, T)
qr_iso_interp(idx, T):  interpolated value of of q(isotope, T) = Q(isotope, T)/Q(isotope, Tref)
Qr_HAPI(Tarr): Qr = partition function ratio array [N_Tarr x N_iso], N_Tarr = len(Tarr), N_iso = len(self.uniqiso), using QT_iso_interp,
Qr_line_HAPI(T): Qr (actually, qr because T-array has 1 element T) for all of the line using Qr_HAPI
Qr_line_HAPI_jax(T): Qr_line_HAPI(T)` for JAX
Qr_layer_HAPI(Tarr):  Qr for layers and lines using Qr_HAPI(Tarr) [N_Tarr x Nlines]


Jupyter tutorial

I added a short tutorial in the jupyter notebook to explain some of the stuff which I didn't at first understand.
I think it could be further expanded in future.

Coverage remained the same at 99.473% when pulling 33dbbad964db0f4eb354dafee94eda29f3af9eb6 on jupyter into 196b44ee6024e2566e6dfb795a905d2197bc41e4 on master.


elastalert-test-rule  - TypeError: can't multiply sequence by non-int of type 'str'

Just upgraded from v0.1.28 to 0.1.33, and now failing.
Traceback (most recent call last):
  File "/usr/local/bin/elastalert-test-rule", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/site-packages/elastalert/test_rule.py", line 421, in main
    test_instance.run_rule_test()
  File "/usr/local/lib/python2.7/site-packages/elastalert/test_rule.py", line 406, in run_rule_test
    data_file.write(json.dumps([doc['_source'] for doc in hits], indent='    '))
  File "/usr/local/lib/python2.7/json/__init__.py", line 251, in dumps
    sort_keys=sort_keys, **kw).encode(obj)
  File "/usr/local/lib/python2.7/json/encoder.py", line 209, in encode
    chunks = list(chunks)
  File "/usr/local/lib/python2.7/json/encoder.py", line 431, in _iterencode
    for chunk in _iterencode_list(o, _current_indent_level):
  File "/usr/local/lib/python2.7/json/encoder.py", line 300, in _iterencode_list
    newline_indent = '\n' + (' ' * (_indent * _current_indent_level))
TypeError: can't multiply sequence by non-int of type 'str'
Exited with code 1

I think it was probably caused by switching from simplejson to json
https://github.com/Yelp/elastalert/commit/86988c78a45911c66112b6afbca67fcd543cb645

Ohhh, I see what's going on.  They are passing indent as a string, instead of a number

json is stdlib and since elastalert doesn't support python2.6 it doesn't
provide any speed benefits.  It also looks like (?) it was only being used in
a test?

beyond that, yeah was just trying to eliminate a dependency that required a C toolchain when the stdlib would suffice.  looks like I missed that indent usage though (it works in python3!).


Confusing command line example when setting configuration

The example command on the page:

dotnet run --MyKey "Using --" --Position:Title=Cmd-- --Position:Name=Cmd--Rick

Is confusing, the -- at the end of each value, and in the middle of the Position:Name value is really odd for an example, it is likely to confuse most, yet the example comes with no explanation as to what it is showing, and then even more frustrating is just below it, the previous two examples have explanations of what they are doing.
The example seems to be trying to demonstrate too much, it could easily be split into three examples like so:

dotnet run --MyKey "MyValue"
dotnet run --Logging:LogLevel:Default=Warning
dotnet run --Command=Cmd-- --AlternativeCommand=Cmd--Run

With an explanation of whet each is showing.
At the moment it feels a bit like a guessing game.

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

ID: 0eb4d762-3910-2c57-8741-3306c15537b6
Version Independent ID: ba2254a0-ea8e-9b28-e94b-edd0fc58c043
Content: Configuration in ASP.NET Core
Content Source: aspnetcore/fundamentals/configuration/index.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

See also #24277

This has been marked as needs more info, but you've not said what info you don't get?


setup ci / docker (and binaries)

similar to fortio's but maybe simplified/sharing some stuff

trying goreleaser


Munin: ability to use plugins from munin-contrib

Munin has a wide range of plugins at https://github.com/munin-monitoring/contrib, we should support using them.
I'm particularly interested into raid monitoring at the moment.

I have a WIP for munin 2.999, and rework for munin service (munin 3 will contain nice webui standalone server). Should I publish this WIP?

Sure! I'd prefer to use stable version but let's see if something can be extracted :)

@danbst did you get somewhere with this?

(triage) can we close this in favor of the PR?

A workable stopgap until full contrib support can be implemented might be:

add a services.munin-node.extraPluginConfig option that's appended to munin-plugin.conf, in the same manner as services.munin-node.extraConfig
add a services.munin-node.extraPlugins option that lists additional plugin files to be symlinked into /etc/munin/plugins; at the moment it just blindly accepts whatever munin-node-configure outputs
add a services.munin-node.disabledPlugins to replace the existing hard-coded disablement of diskstats (which may not even be needed anymore, but if it is we can make that the default value)

This lets users disable plugins at will (useful since munin-node-configure tends to be very aggressive in what it enables), and add new plugins by manually downloading them and referencing them from extraPlugins -- which isn't as nice as having all the contrib plugins available as part of the munin build, but is an improvement on the status quo.

I started work on this. It's more complicated than it looks, and I ran into some other issues I want to fix while I'm in there, but it's coming along nicely.

@florianjacob #51980

As #51980 was merged, this is resolved now.

IMO this isn't fully resolved until munin-contrib is also packaged so users don't have to manually download it and fish individual plugins out of it for use with extraPlugins -- this should be pretty simple, I just haven't had the bandwidth to do it.

Makes sense, I thought those were delivered with the main package already.

No, only the core munin plugins are (contrib is a separate and much more eclectic repo) and I was in a hurry and already had munin-contrib checked out in /usr/src, so I added a feature to load arbitrary munin plugins, pointed that at my local checkout and concluded that was good enough for the moment.
Once munin-contrib is packaged it means that stuff like this:
extraAutoPlugins = [
  /usr/src/munin-contrib/plugins/zfs
];

gets replaced with this:
extraAutoPlugins = [
  ${pkgs.munin-contrib}/plugins/zfs
];

while still letting users who have written their own plugins outside contrib reference them directly in extraPlugins and extraAutoPlugins.
I don't think munin-contrib needs any sort of installation procedure, it's just fetch+unpack, so I may see if I can get it packaged soon. It should be an easy review so hopefully it doesn't languish unmerged for too long and then we can close this.

This is still relevant.


Optimize "foo in [...]" expressions

Replace foo in [...] with foo in {...} if possible.

Done in 7772449d673443427c47e31e22979595a382409f


Misc code optimizations

Information

JIRA story QPPCT-875.

Checklist

[X] All JUnit tests pass (mvn clean verify).
[X] New unit tests written to cover new functionality.
[X] Added and updated JavaDocs for non-test classes and methods.
[X] No local design debt. Do you feel that something is "ugly" after your changes?
[X] Updated documentation (README.md, etc.) depending if the changes require it.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 Node.java#L217: Use the "equals" method if value comparison was intended.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 Node.java#L217: Use the "equals" method if value comparison was intended.


Fix parsing caret ranges (cleanup trailing space)

@piotrooo thanks for the quick fix on the additional issue with whitespace!


Dependency issue during installation.

Getting the following error during installation process:
!pip install --upgrade tensorflow-model-remediation
!pip install --upgrade fairness-indicators
ERROR: Cannot install fairness-indicators==0.1.1, fairness-indicators==0.1.2, fairness-indicators==0.23.0, fairness-indicators==0.23.1, fairness-indicators==0.24.0, fairness-indicators==0.25.0, fairness-indicators==0.26.0, fairness-indicators==0.26.1, fairness-indicators==0.27.0, fairness-indicators==0.28.0, fairness-indicators==0.29.0, fairness-indicators==0.30.0 and fairness-indicators==0.30.1 because these package versions have conflicting dependencies.
The conflict is caused by:
fairness-indicators 0.30.1 depends on tensorflow-data-validation<0.31 and >=0.30
fairness-indicators 0.30.0 depends on tensorflow-data-validation<0.31 and >=0.30
fairness-indicators 0.29.0 depends on tensorflow-data-validation<0.30 and >=0.29
fairness-indicators 0.28.0 depends on tensorflow-data-validation<0.29 and >=0.28
fairness-indicators 0.27.0 depends on tensorflow-data-validation<0.28 and >=0.27
fairness-indicators 0.26.1 depends on tensorflow-data-validation<0.27 and >=0.26.1
fairness-indicators 0.26.0 depends on tensorflow-data-validation<0.27 and >=0.26
fairness-indicators 0.25.0 depends on tensorflow-data-validation<0.26 and >=0.25
fairness-indicators 0.24.0 depends on tensorflow-data-validation<0.25 and >=0.24
fairness-indicators 0.23.1 depends on tensorflow-data-validation<0.24 and >=0.23
fairness-indicators 0.23.0 depends on tensorflow-data-validation<0.24 and >=0.23
fairness-indicators 0.1.2 depends on tensorflow-data-validation<1 and >=0.15.0
fairness-indicators 0.1.1 depends on tensorflow-data-validation<1 and >=0.15.0
To fix this you could try to:

loosen the range of package versions you've specified
remove package versions to allow pip attempt to solve the dependency conflict

Were you able to solve it? I have the same problem and I can't run anything :(


activityIndicator disappear when any background UI changes

when we show the activity indicator and perform any ui changes on view then activity indicator get disappear

@amolpokale1: Could you elaborate on the issue and use case? Either screenshots, gifs, snippets would be helpful.

when I land on my screen I request web_service_call_1 and show the activity indicator, On success of web_service_call_1  response I added new view on my UIView and request web_service_call_2, when I get the response from web_service_call_2 then I hide the activity indicator
Expected - activity indicator should display for my both request
Reality -  activity indicator not displayed some time

I'm using delegate methods of NVActivityIndicatorViewable, I found a issue while using this in multiple ViewController
I have this in parentView controller and it worked fine, but in the child VC not working and is I call that child VC from someOther parent VC who's not conforming to this ,than it works.
weird.

I'm facing exactly the same issue.
Any solution so far?

Mine fix was to stop the animation before starting again.
Sent from my iPhone

On 25-Dec-2017, at 9:19 PM, Marco Nascimento<EMAIL_ADDRESS>wrote:
I'm facing exactly the same issue.
Any solution so far?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Stop activity indicator while starting new one.
There is another issue is you are creating multiple instance of activity indicator and never remove it before using it again.

@rehannali: Could you create a new issue for the other issue you mentioned above?

Clean up stalled issues.


Replace heap allocator's block headers with bitmap

                                               Benchmark |                              Rss |                             Perf

-------- | -------- | --------
./sunspider-1.0.2/3d-cube.js |    164 ->    132 (19.512%) |  2.094 ->   1.94 (7.354%) |
./sunspider-1.0.2/access-binary-trees.js |     96 ->     88 (8.333%) |  1.857 ->  1.653 (10.986%) |
./sunspider-1.0.2/access-fannkuch.js |     44 ->     44 (0.000%) |  5.197 ->  5.079 (2.271%) |
./sunspider-1.0.2/access-nbody.js |     64 ->     64 (0.000%) |  2.574 ->  2.526 (1.865%) |
./sunspider-1.0.2/bitops-3bit-bits-in-byte.js |     36 ->     36 (0.000%) |   1.59 ->  1.568 (1.384%) |
./sunspider-1.0.2/bitops-bits-in-byte.js |     36 ->     36 (0.000%) |    2.1 ->  2.038 (2.952%) |
./sunspider-1.0.2/bitops-bitwise-and.js |     28 ->     28 (0.000%) |  2.003 ->  2.013 (-0.499%) |
./sunspider-1.0.2/controlflow-recursive.js |    224 ->    212 (5.357%) |  1.991 ->  1.695 (14.867%) |
./sunspider-1.0.2/date-format-xparb.js |    112 ->     96 (14.286%) |  1.971 ->  1.791 (9.132%) |
./sunspider-1.0.2/math-cordic.js |     44 ->     40 (9.091%) |  2.448 ->  2.387 (2.492%) |
./sunspider-1.0.2/math-partial-sums.js |     40 ->     40 (0.000%) |  1.423 ->  1.431 (-0.562%) |
./sunspider-1.0.2/math-spectral-norm.js |     52 ->     48 (7.692%) |  1.744 ->  1.621 (7.053%) |
./sunspider-1.0.2/string-fasta.js |     52 ->     52 (0.000%) | 15.675 -> 15.469 (1.314%) |
Geometric mean |                          5.1594% |                           4.776%

LGTM

Good to me.
I'll push.


(SE) v1.8.0 Spack-stack update for upcoming atlas changes

This will happen in September. spack-stack v1.8.0 is not released yet so we can't currently test it.
See details here:
https://github.com/orgs/JCSDA-internal/discussions/147

In the month or two ahead, we'll start requiring atlas 0.38 in JEDI. This brings in various improvements and bugfixes, but of particular interest are fixes to atlas's geometric code used in the generic diffusion operator, fixes to atlas's interpolators, and updates to the atlas GPU support.

Here's what the transition will look like:

The next spack-stack release v1.8.0 will include atlas 0.38.1. The targeted release date for this spack-stack is late August or early Sept. At this stage, JEDI should be usable with atlas versions 0.35 to 0.38, and no immediate updates to software envs or code branches will be needed.

A couple weeks later, we will bump the required atlas version for JEDI to 0.38. The target date is mid September. At this stage, JEDI users will need to update to the latest spack-stack v1.8.0 or compile their own updated atlas.

Quick note on this:
Spack-stack release 1.8 switches to Python 3.11, this impacts all the Python relevant modulefiles *.lua scripts as we have to update the Python version via this entry:
local pythondir = "python3.10"

to
local pythondir = "python3.11"

So that following works:
prepend_path("PYTHONPATH", pathJoin(base,"lib",pythondir,"site-packages"))

Update from JCSDA on timelines:

goal to release the next spack-stack (v1.8.0) is now mid-September
goal to require atlas version 0.38 would therefore be at the end of September


Update Cloudflare D1 example for Miniflare v2.12

It looks like Cloudflare recently refactored the integration between their Wrangler and Miniflare D1 tools.
I think the change was introduced in Miniflare 2.12.0 here: https://github.com/cloudflare/miniflare/pull/480
This change fixed my entry-node.ts for testing/development mode using Miniflare v2.12.1:
https://github.com/patdx/edge-cms/commit/9c40e1aaaed835e3915aa4b83165b12ae0abf5a6#diff-92da841f22f071103fd6839de1b596fc8cac0bb9679580f8a36a2ddacc052f6e
Also, here is an example of the new usage inside Wrangler CLI itself:
https://github.com/cloudflare/workers-sdk/blob/2b641765975d98e6e04342533fa088f51a96acab/packages/wrangler/src/d1/execute.tsx#L245

Updated, thank you!


No Mqtt pings on ESP32 with 2.8.

I recompiled my long running code on a new install of ide & libraries and it disconnects from my mosquito 2 instance on ubuntu due to timeout (according to broker logs) .  The loop function is being called a few hundred times per second, and I tried moving it to a section of code that only executes at 10hz with no change.   Added debug code in the loop function and it executes the connected section and properly detects the ping is due and there is currently no ping outstanding.  It builds the ping buffer calls write and updates the pingoutstanding and last activity values.   But the packet never ends up on the wire for some reason (wireshark does not lie :).
Reverting the code to 2.7 performs properly with no changes to the application or network stack and libraries, and pings on the wire as expected.
Will try to investigate further as time permits but just a heads up if you are having disconnects on low traffic sessions due to timeout there seems to be something lurking in 2.8 at this point.

Hi there, has anyone had any success fixing this in 2.8? Wondering if I should move back to 2.7 still or if we're good on 2.8. Thanks!


Put manual setup steps behind an expander

null
Microsoft Reviewers: Open in CodeFlow

Related to #561 I assume? @stmoy FYI


TypeError in /clan/popz-war-clan-y822upj

Error in ClashLeaders
TypeError in /clan/popz-war-clan-y822upj
Cannot read property 'field' of undefined
View on Bugsnag
Stacktrace
webpack:///js/store/index.js:136 -

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
TypeError in /clan/popz-war-clan-y822upj


README doesn't explain what the project is

From the CAPI meeting, I noticed the link to this project and followed it, but I'm still clueless as to what's here after digesting the README.
It would be helpful if the term "MaaS" were explicitly defined once at the top of the README.md file. Also, please include a sentence or two explaining specifically what the project is, such as "Spectro Cloud provider for the Canonical Metal-as-a-Service interface." Links to more info are also helpful.

Thanks for the recommendation. @mboersma


Document layout choice

Another problem to tackle is the choice of what layout we want to use, I am fond of the scrartcl document class from the KOMA-script package, but of course we should decide all together.
(Sorry if I am opening multiple issues, but this should be how things are handled right? One post per issue)

I believe we should have readability as priority since we are gonna use it for the exam. I personally do not know the differences between the different layouts. If you reckon scrartcl allow for nice readable documents then we should go for it.

I think it should suit us well then, scrartcl changes the page layout such that the paper's space is used more efficiently, for example, (the text was generated through lipsum):
This is vector.tex with the article layout.
vectorsArticle.pdf
This, on the other hand, is the same file with the scrartcl layout.
vectors.pdf

then definitely scrartcl


Debian support and optional Java dependency

Please support debian (wheezy) explicitely.
Keep up the good work!

Debian support has been included for some time, and the java mysql connector is now resolved. FYI Atlassian does not recommend using MySQL with stash any more. See https://confluence.atlassian.com/display/STASH/Connecting+Stash+to+MySQL


Improve the getting-started experience on GKE

https://www.kubeflow.org/docs/started/getting-started-gke/
Some initial thoughts:

Reorganize the page so the Kubeflow Deployment UI option is up front. After UI setup, tell users what the Kubeflow URL looks like. Also provide a "what's next" immediately after the UI deployment. Users don't need info about the command-line deployment if deploying via UI, so separate out the bits about the kfctl command and put them all later in the page.

Tell people up front that the deployment name must be between 4-20 characters long. This restriction appears via an error message in the Kubeflow Deployment UI, but by that stage people have already created their OAuth credentials based on deployment name.

Add a screenshot of the deployment UI.

Give more explanation of the fields in the  Kubeflow Deployment UI, in particular tying back to the OAuth info

For "what's next" at the end of the page, point users to some way of using Kubeflow rather than customizing/troubleshooting

/assign


Submitting a new prompt moved the QnA set to Production.

Package Name: @azure/cognitiveservices-qnamaker
Package Version: ^3.0.0
Operating system: Windows 10
[x] nodejs

version: 10.3.0

[x] browser

name/version: Chrome/78.0.3904.87

[x] typescript

version: 2.4.2

Is the bug related to documentation in
It's not a documentation issue

Describe the bug
When I download my QnA maker knowledgebase, 2 issues happen:

Not all the data is there. When I go to qnamaker.ai and go to my knowledgebase, I see 3 of my qna sets. When I download it using the nodejs sdk (documentation), I see 2 of my sets. It seems like qnamaker.ai shows both testing and production qna sets, but in the SDK, I can only choose one.
When I add a new follow-up prompt using the SDK, the qna set gets moved from testing to production, and the follow-up prompt is NOT even added.

To Reproduce
Steps to reproduce the behavior:

Create a client:

const creds = new msRest.ApiKeyCredentials({
    inHeader: { 'Ocp-Apim-Subscription-Key': kb.endpointKey }
});
const client = new qnamaker.QnAMakerClient(creds, kb.host);
const knowledgeBase = new qnamaker.Knowledgebase(client);
const downloaded = await knowledgeBase.download(kb.knowledgeBaseId, "Test");
const qnaSets = downloaded.qnaDocuments[0].questions;

Compare with the number of qna sets in qnamaker.ai. It has more than what was retrieved. Probably because qnamaker.ai is showing both testing and production sets.
Using the above code, add a new followup prompt:

/** @type {import('@azure/cognitiveservices-qnamaker/esm/models').PromptDTO} */
const newPrompt = {
    displayText: newPromptText,
    qnaId: qnaSet.id
};

/** @type {import('@azure/cognitiveservices-qnamaker/esm/models').UpdateKbOperationDTO} */
const updatePayload = {
    update: {
        qnaList: [{
            id: qnaSet.id,
            context: {
                promptsToAdd: [newPrompt]
            }
        }],
    },
};
await knowledgeBase.knowledgeBase.update(knowledgeBase.details.id, updatePayload);

Run the code in step 1 and notice that the qna set is gone... probably moved to production.
Expected behavior
For all the information to be returned from the client SDK, and for the qna set NOT to be moved to production.

On qnamaker.api, they add a query parameter: ?getFlattened=true. And this returns all the results. Is this possible using the SDK?

Thanks for reporting @AskYous
@ChrisHMSFT Can you loop in the right person from the QnA team to answer the above questions?

I found out some more on the issue. The sdk does not bring all the results. It only brings, perhaps, page 1 of the results. When I would update the prompts of a qna set, it would push that qna set to the end of the list, and the SDK only returns page 1 of the list. I have also received the same results the api console.
In qnamaker.ai, they add a query parameter to their URL: ?getFlattened=true. That will return the entire knowledgebase, and not just page 1 of it.

Issue is not reproable with @azure/cognitiveservices-qnamaker version 3.1.0.
The user is unable to add prompt because he/she is trying to add the prompt to same question.

Thanks @nerajput1607
@AskYous Closing this issue as per the findings above from @nerajput1607.
Please log a new issue if you see any problems with the latest version of the library


Add a license?

Hi,
I'd like to use this package but I don't see a license. Do you mind adding one?

No problem! Added the MIT license


fixAPI 'variant.getAssemble()' is obsolete and has been replaced with…

API 'variant.getAssemble()' is obsolete and has been replaced with 'variant.getAssembleProvider()'
修复了这个警告，希望同时升级gralde版本。

既然都修复了，为什么还不更新一版依赖库呢


ND4j: Building from source updates .java files

Issue Description
After building Nd4J with:
mvn install -DskipTests

git status indicates more files have been updated than were edited:
git status
On branch ra_nd4j_append
Changes not staged for commit:
    (use "git add <file>..." to update what will be committed)
    (use "git checkout -- <file>..." to discard changes in working directory)

    modified:   nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/evaluation/classification/Evaluation.java
    modified:   nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/ops/impl/layers/recurrent/config/SRUCellConfiguration.java
    modified:   nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/ops/impl/layers/recurrent/config/SRUConfiguration.java
    modified:   nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/factory/Nd4j.java
    modified:   nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/learning/AdaDeltaUpdater.java
    modified:   nd4j-backends/nd4j-backend-impls/nd4j-native/src/main/java/org/nd4j/nativeblas/Nd4jCpu.java

no changes added to commit (use "git add" and/or "git commit -a")

The expected behavior is to see only the file changed that has been editied. Nd4j.java
Version Information
Win10 command line.

Nd4jCpu.java is expected
The others are not (and I don't see those when building on windows).
At a guess, it's probably character encoding issues.
Post the git diff in a gist and I'll take a look...

Only Nd4jCpu and Nd4jCuda are uodated. Others can't be updated unless you have some formatting plugin activated in the background.

Here  is a gist with commands, maven log and git diff.

So that's literally encoding issue...

-       *    2: row of biases with twice length [1 × 4K]
+       *    2: row of biases with twice length [1 �? 4K]

Yep, just character encoding, easy to fix. I've got a branch open ATM about to merge, I'll take care of it now (will be on skymind fork until merged back to master)

Fix merged to SkymindIO/deeplearning4j master - https://github.com/SkymindIO/deeplearning4j/pull/49


Releases

Any chance @roikles, that you can introduce releases / tags for versioning?

Once its out of alpha, sure ;)


Configure JIM parameters with web UI

How can I change JIM params with web UI? Or maybe can I set it as Docker container params?

@tyndyll any update?


"System.Data.SqlClient.SqlException: Syntax error near ... in the full-text search condition ..." when using special characters

I expected to be able to perform a full text search using special characters.
Here is an example unit test that fails for each of these special characters, but there could be more.
        [TestMethod]
        public void Contains_UsingSpecialCharacters_Succeeds()
        {
            foreach (string specialCharacter in new[]
            {
                "\"",
                "'",
                ")",
                "(",
                "%"
            })
            {
                using (var db = new MyDbContext())
                {
                    db.Database.Log = Console.WriteLine;
                    var text = FullTextSearchModelUtil.Contains(specialCharacter);
                    var query = db.TestModel
                        .Where(c => c.Name.Contains(text))
                        .ToList();
                }
            }
        }

Any work-arounds or recommendations?
Thank you!

If I enter a string like '[Something]'. I get an error saying

Syntax error near '[' in the full-text search condition '(~[something])

Same issue with a string containing commas

Syntax error near ',' in the full-text search condition '(KGBV,Vinchiya,Jasdan)

@icnocop @chsword  were you able to find a workaround for this?

@shyamal890
 '[Something]' do not work with contains(it do not work in the T-SQL also), I think in this case, maybe try freetext

I also found the same error with these special characters:
Underscore (_)
Tilde (~)
Caret Symbol (^)


Remove comments in minification

Fix #33

@mxstbr could we pass this optimization ?

@kopax it's still missing some fixes to not mangle URLs, but I'll fix that soon


Typo "**～Distributor>**dialog"→"～Distributor> dialog"

https://docs.microsoft.com/en-us/sql/relational-databases/replication/security/view-and-modify-replication-security-settings?view=sql-server-ver15

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks.

Looks good; thanks.


HTTP Client - Unable to send query param that is enclosed within curly braces

I have a use case where my query param contains text within curly braces. For example:
http://localhost:8080/sample?query={"name":"abcd"}
which, when encoded should look like
http://localhost:8080/sample?query=%7B%22name%22:%22abcd%22%7D
To specify the HTTP client for this particular request,
<citrus-http:client id="sample_client" request-method="GET" request-url="http://localhost:8080/sample?query={"name":"abcd"}"/>
I get java.lang.IllegalArgumentException: Not enough variable values available to expand '"name"'
When I give it as
<citrus-http:client id="sample_client" request-method="GET" request-url="http://localhost:8080/sample?query=%7B%22name%22:%22abcd%22%7D"/>
all % characters are escaped to %25 while performing the request. So it'll look like
http://localhost:8080/sample?query=%257B%2522name%2522:%2522abcd%2522%257D
Please suggest a solution for this. Thanks!

The strict answer is that you should never have curly braces in your URL as they are very unsafe for several reasons. I had a look at the IllegalArgumentException that you have reported. This exception comes from Spring trying to interpret the curly braces as URI path variables.
So this is way after Citrus has constructed the URI and passed it to the Http client implementation of Spring' RestTemplate. So I guess you have to find a way to escape the curly braces so that Spring won't interpret them as path variables.
However I would recommend to not use the curly braces in your URL.


Implement Beep() API

Add a Beep() method to the Screen interface. On *nix systems, this
writes the bell character (0x07) to the tty. On Windows, we call the
MessageBeep syscall.
Fixes: #2

I'll add some tests.

So I tried this out on Windows, but nothing seems to happen.  This is on Windows 10.

I'm pretty sure that MessageBeep needs to have a specific argument which might not match what is being passed.

Hm, I ran this in a Windows 10 VM and it played an alert sound for me. I don't really use or know much about Windows, do you have any ideas?
According to https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-messagebeep, 0xFFFFFFFF seems reasonable, but I also don't know what someone familiar with Windows would expect to hear.

@gdamore would you be able to test a few of the other beep options listed here? Specifically,

0xFFFFFFFF
0x00000000L
0x00000010L
0x00000020L
0x00000030L
0x00000040L

As I mentioned, the demo program included in the PR does make an alert sound on my Windows 10 VM. I'm not sure why you're not hearing anything, but if one of these other options does work for you I'd be happy to retest it and amend my patch.
In any case, let me know how you'd like to proceed. Thanks!


Why does hive_limits::max exist at all?

Following on from issue #21... actually I don't really understand the goals of reshape and block_capacity_limits in general.
Is hive_limits.max really supposed to be a user-specified upper limit on the size of a block? What if the user sets it to something unaesthetic like 37, is the vendor allowed to increase that to a nice sane 64, or must we respect the 37?
Or what if the user of a plf::hive<short> sets the max block size to 4? With your implementation's current group layout, this allocates a memory block of size (4*2 + 5*1) = 13. That's pretty much guaranteed to waste 20% more memory than if the vendor quietly rounded the max block size up to 5, producing a memory block of size (5*2 + 6*1) = 16.
Setting a min block size makes sense — too-small blocks can hurt performance. Setting a max block size seems counterproductive. Thoughts?

Because some people will want to limit the size of blocks to fit a particular cache size, and that will vary based on element size.
Also they might work out that for their particular usage pattern, a given block size leads to the minimal number of allocations/deallocations vs wasted space.
It is probably the number one thing people have repeatedly thanked me for in terms of the container design as a whole.

Because some people will want to limit the size of blocks to fit a
particular cache size, and that will vary based on element size.
Also they might work out that for their particular usage pattern, a
given block size leads to the minimal number of
allocations/deallocations vs wasted space.
It is probably the number one thing people have repeatedly thanked me
for in terms of the container design as a whole.
On 3/05/2022 8:00 am, Quuxplusone wrote:

Following on from issue #21
https://github.com/mattreecebentley/plf_hive/issues/21... actually I
don't really understand the goals of |reshape| and
|block_capacity_limits| in general.
Is |hive_limits.max| really supposed to be a user-specified /upper
limit/ on the size of a block? What if the user sets it to something
unaesthetic like |37|, is the vendor allowed to increase that to a nice
sane |64|, or must we respect the |37|?
Or what if the user of a |plf::hive| sets the max block size to
|4|? With your implementation's current group layout, this allocates a
memory block of size |(42 + 51) = 13|. That's pretty much guaranteed
to waste 20% more memory than if the vendor quietly rounded the max
block size up to |5|, producing a memory block of size |(52 + 61) = 16|.
Setting a min block size makes sense — too-small blocks can hurt
performance. Setting a /max/ block size seems counterproductive. Thoughts?
—
Reply to this email directly, view it on GitHub
https://github.com/mattreecebentley/plf_hive/issues/22, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABE4FIULE7MAUZSCZWFN5O3VIAX45ANCNFSM5U46JJNA.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


setup: add under construction notice

Coverage remained the same at 81.866% when pulling cbe8f72ab7701bcdd17d91002db3f56157a5500e on under-construction into 53e1d4ee27cf7c67df361ff24b8f14784fae37ee on master.


Various issues with the POD

Your POD under 'Subroutines' often uses the phrase 'Put error if check isn't ok.'. The meaning is unclear to me:
(a) Does it output to STDOUT or STDERR?
(b) Does it halt the program at the 1st error? The examples suggest it does.
(c) Can the user capture to error message? If no, why not?
(d) Can the use force the program to continue after an error? If not, why not?
(e) Why do the sub return undef? They could return the error or say, 0 to indicate no error.

@ronsavage
(a) Errors are handled by Error::Pure and this has the possibility to handle errors via setting.  Mostly to STDERR, but it's possible to STDOUT without die at the end.
(b) Yes, that's the main intent.
(c) Normally via eval.
(d) see (c)
(e) hm. I don't think about return value. The intent is to pass only. Without check. (Example: https://metacpan.org/dist/Data-HTML-Element/source/Element/A.pm#L38)


Subdomain Takeover via Tumblr

Service name
Tumblr
Fingerprint
A source domain has a DNS entry that points to Tumblr, however no active blog is associated with the domain.
DNS Record: CNAME domains.tumblr.com.
HTTP Response Status: 404 Not Found
HTTP Response Body: Whatever you were looking for doesn't currently exist at this address
Verification:
curl -s -N http://$SOURCE_DOMAIN_NAME | grep -E -q "Whatever you were looking for doesn't currently exist at this address" && echo "Subdomain takeover may be possible" || echo "Subdomain takeover is not possible"
Takeover Steps
Domains with CNAME to Tumblr are vulnerable to subdomain takeover.
Step-by-step process:

Log in to Tumblr account (MUST validate email address)
Go to Tumblr Account drop down
Click Edit Appearance
Click on the pencil icon next to your username
Select Use a custom domain
Set custom domain to source domain name
Click on Test Domain (Should return It's good!)
Click on Save

Some reports on H1, for Tumblr blog takeovers:
https://hackerone.com/reports/113869
https://hackerone.com/reports/221631
Documentation
Tumblr Custom Domains
https://www.tumblr.com/docs/en/custom_domains

thanks

This takeover has changed, to point to a custom domain the dns record needs to point to domains.tumblr.com.
As seen here:

So, if the CNAME is different to domains.tumblr.com (probably old deployments) the target domain is not vulnerable.

if the CNAME is different to domains.tumblr.com

i have CNAME pointing to domains.tumblr.com but still shows exactly the same error, what could be the problem??

devs, the same on my
https://tumblr.alexdolbun.com
the same error, what could be the problem???)))

i understand, done. disabled proxy on cloudflare on https://tumblr.alexdolbun.com/ and this error is gone 🦄

OVERPEY

same problem

Hi,
I had originally submitted the A Record based detection for this in #9619931. As far as has been made clear in the most up-to-date documentation provided by Tumblr, the A Record of <IP_ADDRESS> is valid for apex domain names only. When dealing with a domain name that has more than two levels, i.e., a non-apex domain name - e.g., evil.com.au, evil.co.in, or mysite.evil.net - a CNAME Record pointing to the domains.tumblr.com host is required instead.
This issue can be closed once the current information for Tumblr subdomain takeovers has been qualified to account for both apex and non-apex domain case.

Important thing to check before proceeding with the takeover:

CNAME should be domains.tumblr.com (If example.com have CNAME like this domains.tumblr.com) then takeover is possible for Tumblr else not.

This no longer appears possible as of 9 June 2023. See this reference and this reference. According to the references, custom domains must be purchased through Tumblr's own domain service.

On web, we’ve launched support for purchasing custom domains for your blogs directly through Tumblr. Existing custom domains linked to blogs will still work, but going forward, custom domains must be purchased through Tumblr. We’re still working on a domain transfer flow, more to come!

Legacy custom domains are domains registered outside of Tumblr that were connected to a Tumblr blog before we introduced Tumblr Domains. Rest assured that your legacy custom domains will remain the home address of your blog until you disable the "Use custom domain" toggle in blog settings under "Custom Theme". It’s important to note that once your legacy custom domain is disconnected, you will not be able to reconnect it to your Tumblr blog.


License

Hi, I'm packaging this in my private NixOS user repository and I'm wondering what the license for this card is.
Can you clarify?

I added a license file.


Migrating to Flutter Embedding V2 Correctly. Also updated the old _launchURL function

The thing is the package always open links in browsers, but I want it to open them like WebViews. So here we go :")

LGTM 👍

Thanks for the PR @OmarYehiaDev.
Looks good to me as well, I will merge this to main branch but will be included in my next release which is expected around mid of this month.


story: Technology Set Up

Set up all required technology on each team member's machine

Test project story: setup


Can we add Github "Discussion" tab?

Can we add Github "Discussion" tab?
Discord is not good because one cannot search for previous discussions.

Discussions are being closed soon in the Storybook monorepo, they are not activated on the svelte repo and I don't want to follow another channel here.
I am closing this for now


[tests-only][full-ci] run core acceptance tests with the Graph API

Description
Run core API acceptance tests using the Graph API for user/group provisioning
Related

https://github.com/owncloud/ocis/issues/3167
https://github.com/owncloud/core/pull/39927

ci failed with:
curl: (22) The requested URL returned error: 502 Bad Gateway
Server on https://ocis-server:9200 is down or not working correctly.

ocis log:
{"level":"debug","service":"proxy","policy":"ocis","prefix":"/status.php","path":"/status.php","routeType":"prefix","time":"2022-04-12T11:34:30Z","message":"director found"}
{"level":"debug","service":"idm","scope":"ldap","time":"2022-04-12T11:34:31Z","message":"http: proxy error: unsupported protocol scheme \"\""}
{"level":"info","service":"proxy","proto":"HTTP/1.1","request":"da7b97d95be4/USecxB3iyy-000001","remote-addr":"<IP_ADDRESS>:44818","method":"GET","status":502,"path":"/status.php","duration":101.529124,"bytes":0,"time":"2022-04-12T11:34:31Z","message":"access-log"}
could not select ocdav service from the registry: not found

cc @rhafer

:boom: Acceptance test cs3ApiTests-ocis failed. Further test are cancelled...

@saw-jan will look on this pR from now.

@rhafer Could you review it?

Few things need to be fixed in testhelpers: https://github.com/owncloud/core/pull/39987

TUS upload tests still fail with cURL error 7: Failed to connect to localhost port 9200 error
https://drone.owncloud.com/owncloud/ocis/10786/69/6

TUS upload tests still fail with cURL error 7: Failed to connect to localhost port 9200 error https://drone.owncloud.com/owncloud/ocis/10786/69/6

Weird. I had this running yesterday. Will take a look.

depends on PR https://github.com/owncloud/core/pull/39987.


[Bug]: Horizontal card image display

Prerequisites

[X] I have searched the backlog for duplicate or closed feature requests

Your test device
Samsung Galaxy A51 5G
OS Version
Android 11
App version
119
Describe the issue

Unable to repeat, but there is an issue with the height of the image in this horizontal card.
Expected Behavior
The image should be cropped to square format.

Duplicate of #445

Thanks Stephen, you were unlucky because this one is very difficult to reproduce and happens in rare cases 😄


Add custom Dockerfile workflow example

As per #101.

Hm, actually the Docker support doesn't quite do what I want. I want to add the file /usr/lib64/mysql57/libmysqlclient.so.1020 to my zipfile (in the root directory). It's accessible in the Docker image, but Pip (obviously) doesn't copy it to /var/task during pip install....
Current build process:

Build custom image with pip dependencies
Run the image with a bind mount to .serverless/ to /var/task, and command pip install ...

After running pip install, I want to copy some files from the image to .serverless/requirements. Does this plugin have a supported way to do so?

I've updated the docs to handle both cases:

Adding build-time OS dependencies
Adding run-time OS dependencies

If there's a better way to implement 2 than "copy them into your repo" I'm all ears. Couldn't figure it out myself though :(


Update AddCommand

Resolves #23.
Known issues:

AddCommand does not enforce unique NRICs right now. This is due to its implementation breaking EditCommand implementation and test cases. To prevent risks in regression, it is better to implement the NRIC enforcement after EditCommand has been implemented and merged. Enforcement will be listed for implementation under #67.

Valid concern for the NRIC enforcement and implementation of Optional for appointments.
Test cases have good coverage. LGTM. 👍


web

so literate programming became known to knuth as a web. I agree.
In particular, when trying to decide where to place code, it often goes in multiple places. For example, in a CRUD, we have form -> server -> db -> page  so does one put the code for the db in with the db code or with the particular sequence. It becomes an issue when one is trying to deal with changes on either end. So maybe the solutions is both. Have a graph of relations, one relation being the flow, one being the function, near kin, etc.  And so a web of stuff.

Thinking the term pieceful programming might be applicable, to emphasize the pieces and get away from the narrative.


FIX PR #93

PR #93 only needed some lint cleaned.
resolves #93
All commits by fnickels have been preserved.

Thanks @isaacnboyd and @fnickels!


Add ability to accept an OMIS quote

This:

adds accepted_on/by fields to the Quote model
adds the methods needed to accept a quote

quote.accept(by)
order.accept_quote(by) which calls the former

validates that the order is in awaiting quote acceptance status
validates that the quote has not been cancelled

The actual endpoint that accepts the quote is not included as the public facing application is not ready yet.

Codecov Report

Merging #472 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop    #472      +/-   ##
==========================================
+ Coverage    96.29%   96.3%   +<.01%
==========================================
  Files          108     108
  Lines         3350    3354       +4
  Branches       305     305
==========================================
+ Hits          3226    3230       +4
  Misses          85      85
  Partials        39      39

Impacted Files
Coverage Δ

datahub/omis/quote/serializers.py
100% <100%> (ø)
: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 3e75d5a...8ff73a6. Read the comment docs.


Attr.style missing from AttrEngine

I added it just with a string parameter. It's probably better that specialized version are provided as extensions by the implementations.

Ok. I couldn't even see that though. I will review. Unless you mean you've
just added it now.. In which case thanks!
On Mon, 1 Mar 2021 at 13:53, Alfonso Garcia-Caro<EMAIL_ADDRESS>wrote:

I added it just with a string parameter. It's probably better that
specialized version are provided as extensions by the implementations.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alfonsogarciacaro/Feliz.Engine/issues/14#issuecomment-787965109,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACFV3KLX5YKZVOYSKWSGZTTBOL5NANCNFSM4YLNVELA
.


Use Ember.warn

I would like to silence the warnings the library throw on some tests. So Ember.warn should be used instead of Ember.logger.warn. That way the warnings can be silenced in tests by a configuration like this:
Ember.Debug.registerWarnHandler((message, options, next) => {
  let ignoredWarnings=[
    'ember-intl-no-locale-set',
    'ember-intl-missing-translation'];
  if(ignoredWarnings.includes(options.id)) { return; }

  return next(message, options);
});

Thanks!  The test failures don't look to be related.

Versioned as 2.4.0


Training on Single GPU gives NCCL error

Describe the bug
I replaced the MLPs in my model using FMoETransformerMLP and was trying to train the model on single GPU with 4 experts and top-2 routing, defined like this:
self.mlp = FMoETransformerMLP(
            num_expert=num_experts,
            d_model=dim,
            d_hidden=int(dim * mlp_ratio),
            activation=act_layer,
            expert_rank=os.environ.get("RANK", 0),
            world_size=num_gpus,
            top_k=top_k,
            gate=gate
        )

When it starts to train I receive this error:
Trident-MS:1203254:1203254 [0] misc/argcheck.cc:30 NCCL WARN CommUserRank : comm argument is NULL
Trident-MS:1203254:1203254 [0] NCCL INFO init.cc:1577 -> 4

Trident-MS:1203254:1203254 [0] misc/argcheck.cc:30 NCCL WARN Send : comm argument is NULL
Trident-MS:1203254:1203254 [0] NCCL INFO enqueue.cc:1442 -> 4
NCCL Error at /home/aghinassi/Desktop/fastmoe/cuda/global_exchange.cpp:16 value 4

Expected Behavior
I expected the model to train without bringing additional modifications to the training loop. I only changed the architecture maybe I need to do some adjustment or disable NCCL on single gpu?
Platform

Device: NVIDIA RTX3090
OS: Ubuntu 22.04
CUDA version: 12.1
NCCL version: 2.14.3
PyTorch version: 2.0.1+cu117

Recompiled with USE_NCCL=0


Convenient way to make 'basic_json' accept 'QString' as an key type as well?

I want to use Qt's QString type with the excellent nlohmann::json.
This works very well by providing the functions to_json and from_json as described in the readme:
nljson.h
#include <QString>
#include <QByteArray>
#include <vector>
#include "nlohmann/fifo_map.hpp"
#include "nlohmann/json.hpp"

// This is a workaround solution for the "nlohmann::json" library to preserve
// the insertion order of JSON objects instead of sorting them by name.
// It is based on a custom map implementation by Niels Lohmann itself
// (nlohmann::fifo_map). The actual solution (by GitHub user "Daniel599")
// can be found here: https://github.com/nlohmann/json/issues/485#issuecomment-333652309
template<class K, class V, class dummyCompare, class A>
using nljson_fifo_map = nlohmann::fifo_map<K, V, nlohmann::fifo_map_compare<K>, A>;
using nljson = nlohmann::basic_json<nljson_fifo_map>;

QT_BEGIN_NAMESPACE
void to_json(nljson &j, const QString &qstr);
void from_json(const nljson j, QString &qstr);

void to_json(nljson &j, const QByteArray &qba);
void from_json(nljson &j, QByteArray &qba);
QT_END_NAMESPACE

nljson.cpp
#include "qnljson.h"

#include <QString>
#include <QByteArray>
#include <string>
#include "nlohmann/json.hpp"

QT_BEGIN_NAMESPACE

void to_json(nljson &j, const QString &qstr)
{
    j = nljson { qstr.toStdString() };
}

void from_json(const nljson j, QString &qstr)
{
    if (j.type() == qnljson::value_t::string)
        qstr = QString::fromStdString(j.get<std::string>());
    else
        qstr = QString::fromStdString(j.dump());
}

void to_json(nljson &j, const QByteArray &qba)
{
    j = qnljson { qba.toStdString() };
}

void from_json(nljson &j, QByteArray &qba)
{
    qba = QByteArray::fromStdString(j.get<std::string>());
}

QT_END_NAMESPACE

However, I would find it very beneficial if it were possible to use QString as a key data type as well:
myfile.cpp
// {...}
QString foo = "abc";
nljson bar { "a", foo };
QString baz = "a";

qDebug() << bar.at(baz).get<QString>(); // Error: no viable conversion from 'QString' to 'int'

qDebug() << bar.at(baz.toStdString()).get<QString>(); // This works

Is there a convenient way to make nlohmann::json accept QString as a key type without this explicit conversion (baz.toStdString())?

I haven't tested this but how about overriding the StringType from std::string to QString?
using my_json_type = nlohmann::basic_json<std::map, std::vector, QString>;

my_json_type json;

QString key = "hello";
json[key] = "world";

I am not sure if QString provides all APIs required by the library though.

Unfortunately when I tried that:
// {...}
template<class K, class V, class dummyCompare, class A>
using nljson_fifo_map = nlohmann::fifo_map<K, V, nlohmann::fifo_map_compare<K>, A>;
//using NLJson = nlohmann::basic_json<nljson_fifo_map>;
using NLJson = nlohmann::basic_json<nljson_fifo_map, std::vector, QString>;
// {...}

the result was this compilation error (MSVC 2015 64-Bit):
C2338: The C++ Standard doesn't provide a hash for this type.

so at least the hash-thing is missing.
Are there any chances to provide this hash?

I am also afraid that if I use QString directly as the common string type, there may be errors (QString is UTF-16 internally).
The Readme states about string encodings:

Note the library only supports UTF-8. When you store strings with different encodings in the library, calling dump() may throw an exception unless json::error_handler_t::replace or json::error_handler_t::ignore are used as error handlers.

Are there any chances to provide this hash?

This solution might work.
I gave QString a try and there are some missing APIs (e.g. like c_str()). Considering these, the issue you pointed out about QString use of UTF-16, and the previous discussion at https://github.com/nlohmann/json/issues/274 I'm afraid that there is probably not a straightforward solution use QString as std::string at least for now. I can imagine two options to proceed:

Implement a class that provides the minimal API that this library needs for a StringType and also support implicit conversion from QString. Use this class in instantiation of basic_json template instead as StringType instead of QString.

Leave the default nlohmann::json as it is (with std::string as StringType) and do all conversions from QString to std::string manually.

Option 2 is a faster and less hassle solution if you can minimize the instances where you have to modify/query the JSON object. But if there are many places that you have to do this conversion, Option 1 might worth trying as it encapsulates all those conversions in one place and would be easier to maintain.

I agree with https://github.com/nlohmann/json/issues/1640#issuecomment-502490542. The library only has superficial support for other string types, especially if they do not have the API of std::basic_string. The approaches described in https://github.com/nlohmann/json/issues/1640#issuecomment-502490542 are the only options as far as I see.


Errors when using it in Electron

Hello, I have tried using the extension by following the instructions https://www.electronjs.org/docs/latest/tutorial/devtools-extension and seems like there's an issue with it.
session.defaultSession.loadExtension(
  join(
    homedir(),
    '/Library/Application Support/Google/Chrome Beta/Default/Extensions/kmcfjchnmmaeeagadbhoofajiopoceel/<IP_ADDRESS>_0'
  )
);

The output
[20866:0628/125538.597770:ERROR:extensions_browser_client.cc(62)] Extension Error:
  OTR:     false
  Level:   2
  Source:  chrome-extension://kmcfjchnmmaeeagadbhoofajiopoceel/assets/background.ts-9e9ef213.js
  Message: Uncaught TypeError: Cannot read properties of undefined (reading 'addListener')
  ID:      kmcfjchnmmaeeagadbhoofajiopoceel
  Type:    RuntimeError
  Context: chrome-extension://kmcfjchnmmaeeagadbhoofajiopoceel/assets/background.ts-9e9ef213.js
  Stack Trace:
    {
      Line:     1
      Column:   1
      URL:      chrome-extension://kmcfjchnmmaeeagadbhoofajiopoceel/assets/background.ts-9e9ef213.js
      Function: (anonymous function)
    }
[20866:0628/125538.597966:ERROR:extensions_browser_client.cc(62)] Extension Error:
  OTR:     false
  Level:   1
  Source:  manifest.json
  Message: Service worker registration failed. Status code: 15
  ID:      kmcfjchnmmaeeagadbhoofajiopoceel
  Type:    ManifestError
dom ready
add /etc/hosts
[20866:0628/125539.352692:ERROR:CONSOLE(2)] "Electron renderer.bundle.js script failed to run", source: node:electron/js2c/renderer_init (2)
[20866:0628/125539.352717:ERROR:CONSOLE(2)] "TypeError: object null is not iterable (cannot read property Symbol(Symbol.iterator))", source: node:electron/js2c/renderer_init (2)

Same issue when following the Usage section of Electron DevTools Installer

Would it be possible to prepare a reproduction or guide me to some working electron + solid template? I haven't used electron so I'm not sure if what could I do to help with setting up the devtools there.
Usually people use the overlay package instead of the extension on tauri/electoron.


Gantt Groups - Error: [$rootScope:inprog] $digest already in progress

I'm consistently seeing Error: [$rootScope:inprog] $digest already in progress in the groups plugin, both within my own code as well as on the demo site. The $digest calls seem to be originating from https://github.com/angular-gantt/angular-gantt/blob/master/src/plugins/groups/group.controller.js#L29 and https://github.com/angular-gantt/angular-gantt/blob/master/src/plugins/groups/group.controller.js#L36 Where $scope.$$phase is not accurately reporting that a digest is in progress.
I've seen others use if (!$scope.$$phase && !$scope.$root.$$phase) to accomplish the same thing (see http://stackoverflow.com/a/16301044) and after changing these two lines to include the root scope check, I didn't see any more $digest errors. I'd be happy to create a pull request if you feel that this is an acceptable fix.

I'll do thanks for reporting


.xcovignore does not allow wildcard

Hej,
if I add a file name with a wildcard like this:
- *View.swift
no file listed in .xcovignore is not ignored.
Is it possible to use wildcards?
cheers,
steven

Hey :wave:
That's not supported but it's interesting. I will support the matching of that kind of pattern for the next release :+1:

That's awesome!

Closing this thanks to you @stevenreinisch 👍


[GLUTEN-6951][CORE][CH] Move CustomerExpressionTransformer to CH backend

CustomerExpressionTransformer was added to core module in https://github.com/apache/incubator-gluten/pull/1937 however is not used by Velox backend for long time. Propose to move its code and planner rules to CH backend.
Changes:

Drop support of config option a.g.s.c.extended.expressions.transformer from Velox backend
Move code related to CustomerExpressionTransformer  from core to CH backend
Essential code refactors

@liujiayi771 @Yohahaha @ulysses-you @kecookier @zhli1142015 (and all Velox backend users/developers)
Please let me know if you rely on the feature. Otherwise we'll remove then.

@zzcclp What do you think about this? Thanks.

We are not rely on this functionality for now.

+1

let me move test to backends clickhouse first.

+1

please move spark32/src/test/scala/org/apache/spark/sql/extension/CustomerExpressionTransformer.scala, spark32/src/test/scala/org/apache/spark/sql/extension/GlutenCustomerExpressionTransformerSuite.scala to CH backend module too.

please move spark32/src/test/scala/org/apache/spark/sql/extension/CustomerExpressionTransformer.scala, spark32/src/test/scala/org/apache/spark/sql/extension/GlutenCustomerExpressionTransformerSuite.scala to CH backend module too.

I am moving it. It caused Velox's test failure.

@zzcclp Thanks!


Get rid of bem-core dependency

Fixes gh-60

there's still https://github.com/bem-contrib/stat-counters/blob/master/common.blocks/google-analytics/google-analytics.browser.js so we should decide what to do with it

there's still https://github.com/bem-contrib/stat-counters/blob/master/common.blocks/google-analytics/google-analytics.browser.js so we should decide what to do with it

I propose to don't care.

I propose to don't care.

let's add some notes to readme at least?

Can we make 2 levels for bem-core and for vanilla js?

@zxqfox do we really need bem library with vanilla js? can't users of vanilla version use API without any additional library?

let's add some notes to readme at least?

I've updated


Update Polish translate

Below I add updated Polish translate.
pl.json

Thank you, i have included the update with v3.1.1


Add support for<EMAIL_ADDRESS>

Fix #45

Perfect, thanks!

\o/
Thank you, this is the simplest and best organized container for bitcoin full node.
Is so simple that prefer to use it as way to show people how to run full nodes

Thanks @victorperin! If you have more ideas for other improvements, I'm all ears.


issue with popup marker not showing on iOS9

Hello,
I am facing an issue with popup marker not showing, it show sometimes after 5 to 10 clicks.
ionic : 3
angular 6
cordova 8
ios 9
this.map = L.map(this.mapId, {
            center: [this.center.latitude, this.center.longitude],
            zoom: this.zoom,
            minZoom: 2,
            attributionControl: true,
            tap: false,
            dragging: false
});

    const mainLayer = L.tileLayer('https://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png');

    mainLayer.addTo(this.map);

    this.markers.forEach((point) => {
        const icon = L.icon({
            iconUrl: point.icon ? `assets/images/markers/${point.icon}` : 'assets/images/markers/marker-icon-grey.png',
            shadowUrl: 'assets/images/markers/marker-shadow.png',
            iconSize: [25, 41],
            iconAnchor: [12, 41],
            popupAnchor: [1, -34],
            shadowSize: [41, 41]
        });

      const marker = L.marker([point.longitude, point.latitude], {icon}).addTo(this.map);
      marker.bindPopup('Marker  showed');
   });

You may want to check out this issue and try this solution.

Is it displayed if you open it directly?
marker.bindPopup('Marker  showed').openPopup();

@hamdipro Would you like to fill out the bug report template for this issue?

This is possibly related to the WebKit bug that caused https://github.com/Leaflet/Leaflet/issues/8068.


Unable to Start Snapshot Debugger - Attach Snapshot Debugger failed: System.Net.WebException: The remote server returned an error: (401) Unauthorized

Unable to Start Snapshot Debugger - Attach Snapshot Debugger failed: System.Net.WebException: The remote server returned an error: (401) Unauthorized.
at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
at System.Threading.Tasks.TaskFactory1.FromAsyncCoreLogic(IAsyncResult iar, Func2 endFunction, Action1 endAction, Task1 promise, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.ProductionDebug.Utilities.Communication.AzureRequest.<GetResponseInternal>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.VisualStudio.ProductionDebug.Utilities.Communication.AzureRequest.<GetResponseInternal>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.ProductionDebug.Utilities.Communication.AzureRequest.<GetResponse>d__1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.ProductionDebug.Utilities.Communication.AzureRMCommunicationClient.<<GetAzureAppServiceAsync>b__99_0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.ProductionDebug.Utilities.Communication.AzureRMCommunicationClient.<ExecuteWithRetryTokenRefresh>d__101`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.ProductionDebug.Utilities.Communication.AzureRMCommunicationClient.<GetAzureAppServiceAsync>d__99.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.ProductionDebug.Utilities.Communication.AzureRMCommunicationClient.<InitializeSessionConfig>d__42.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Microsoft.VisualStudio.ProductionDebug.DebugSession.ProductionDebugSession.<StartSessionAsync>d__41.MoveNext()

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

ID: d0557fd8-48c5-bbee-04b9-27baeb8858e6
Version Independent ID: 2b397dd4-cfd5-78a8-9256-d1fb981b30bf
Content: Troubleshooting snapshot debugging - Visual Studio
Content Source: docs/debugger/debug-live-azure-apps-troubleshooting.md
Product: visual-studio-windows
Technology: vs-ide-debug
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@MegoCs Thanks for your feedback. It looks like you're having an issue with the product itself, rather than an issue with the documentation. Here are a few options where you might consider asking your question:
https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/
As well, you might also consider using the Report a Problem tool to report the issue.


[RUNTIME] Add function name to PackedFunc and TypedPackedFunc

This PR adds a name field to both PackedFunc and TypedPackedFunc. This name defaults to <anonymous>, but I've also added code that sets the name for TVM_REGISTER_GLOBAL().set_body() and TVM_REGISTER_GLOBAL().set_body_typed(). With this change, the error messages for incorrect number of arguments is now a lot more clear: it prints the function name in the error message.
I'm not sure if the approach I am using here makes sense. I'd like feedback on whether or not this is the correct direction to go.
@tqchen @jroesch @junrushao1994

Adding function name to packed functions definitely make sense to me. I wonder if we can go a bit further:

On C++ side, enhance TVM_REGISTER_GLOBAL to set the name of the packed func
On python side, enhance register_func to set of the name of the packed func too
What about the rust side?

This PR changes TVM_REGISTER_GLOBAL so that is sets the name of the packed func. I can do the same on the python side.

Shall we accomplish this more automatically by modifying the TVM_REGISTER_GLOBAL macro itself?

In file runtime/registry.h, Right now the macro is:
#define TVM_REGISTER_GLOBAL(OpName) \
  TVM_STR_CONCAT(TVM_FUNC_REG_VAR_DEF, __COUNTER__) = ::tvm::runtime::Registry::Register(OpName)

I think we can change the runtime::Registry::set_body method, adding f.name_ = ... to it

That's what I do here: https://github.com/apache/tvm/pull/7108/files#diff-0fe12f71cc9718f4bf446609180e9c13ca870f0a1ecf080f330b7af8743a0d43R61

Oh nice! I didn't notice that :-)

BTW, can you add a unit test? I think you can add it to the existing gtest.

Thanks @tkonolige . First of all, I think it is very useful to be able to support additional meta data like names when the user can provid eit.
On the other hand, attaching a String that defaults to a concrete value would mean additional overhead for PackedFunc creation, and may be prohibitive if we want to create PackedFunc on the fly(with a goal to gain std::function level speed).
It would be good put more thoughts into this and think about how to improve  the impl in general. As a first step, it certainly does not hurt to make the name Optional and default to nullptr, which reduces the string construction cost.
We can alternatively think about the future path of bringing in PackedFunc as a sub-class of Object, and potentially sub-class PackedFunc. So the default one remains un-named, while there is a named subclass.
Given it is quite a core change, an RFC would be great.

Thanks @Tristan Konolige for the packed func naming PR.
It was on my mind for a while but didn't get to think about all the design tradeoffs. An RFC that quickly polls dev community would be great, specifically there are a few design spectrums(feel free to include these in the rfc):
Naming convention:
N0: name_hint: might be  better name since it does not uniquely identify things
N1: name
What is the default string:
S0: anonymous
S1: use a special sub-class of PackedFunc(need to move PackedFunc to object system)
S2: always attach Optional since the ref object is relatively cheap, default to nullptr
API choices to set the name
A0: Allow mutation, add setter API
A1: Only allow immutable construction, might affect how we construct from C caallback
A2: force to use name_hint, allow idempotent setting, if the name_hint is nullptr, allow set, otherwise allow set only if the new name is same as the old one. Error out if mismatch.

closing in favor of the alternative capturing approach

closing in favor of the alternative capturing approach


New test format in hjson

This commit changes the Hjson syntax to support a list of check tests,
each of which can pass or fail early and prevent subsequent tests from
being evaluated. This simplifies the syntax for commands that have
simple prerequisites such as “is this app even installed?”
An early fail state also helps avoid running checks with sudo privy
unnecessarily.
QA: Did lots of manual testing, running the app with debug print mode
enabled and carefully reviewing output. Also confirmed that tests work
correct when Google Chrome is installed or not installed, now that
Chrome-related checks are now making use of this change.

This PR satisfies completely:

https://github.com/kristovatlas/osx-config-check/issues/108
https://github.com/kristovatlas/osx-config-check/issues/107
https://github.com/kristovatlas/osx-config-check/issues/41

This partially satisfies:

https://github.com/kristovatlas/osx-config-check/issues/85

Some of these changes were originally proposed in:

https://github.com/kristovatlas/osx-config-check/issues/77


adios oclif

Removes oclif in favour of Yargs

No more tsnode and tslint dependencies in our published package
No more 2000 dependencies to install
Faster in loading because there's no auto mechanism of understanding if it's javascript or typescript project
Remove hacks in tsconfig that was causing sometimes VSCode to go crazy yelling you to change your job
Supports argument validation (if port is a string you get an error message and we do not have to validate it on our own)
Supports the involve flag, which is going to be really useful for validation

Closes #461

I'm all in for using yargs.
I used it on a number of occasions and it never disappointed me.
It might not be the most feature-complete tool on the market yet it does come with everything you need to implement a robust command-line interface. We don't need more than that, do we?

Re auto lookup commands in the directory - can we please not do this? This sounds like oclif, and is too much magic IMHO. It also breaks certain common patterns, like bundling.
What it's "solving" is just us having to add a few lines to the cli/src/index.ts file right?

@marbemac I'm all for it! I intentionally tried to stay close as possible to the old oclif mechanism but if you're ok getting rid of it — sure!

Is it meant to show this output?

Same for subcommands

Some of our param descriptions have full stops and some dont, lets pick a convention and stick to it

Loading a non-existant description document (nope.yaml) causes a whole lot of unexpected output.

1 & 2: This is part of the yargs output and it's intentional, it's telling that the positional and required arg is missing.
3: good point, I'll add the period everywhere
4. This has nothing to do with oclif/yargs — you can see the thrown exception is from the file loader which is going to die soon anyway

1 & 2: can we get rid of it? People pretty regularly type in the subcommand to get help on how the thing works, i dont think we want to shout at them?

could is that #407 and if so could you add notes about this in there? if not could we make an issue to track?

I found more odd things its tough to know whats broken now and what was already broken, I guess I should make new issues for all new things?

1 & 2: removed in 1dd0f4a
4. Issue updated.

I found more odd things its tough to know whats broken now and what was already broken, I guess I should make new issues for all new things?

Just drop the bombs as issues and I'll go through them.

@philsturgeon Can you give it another try when you've got some time?


wllvm: 1.2.8 -> 1.2.9

Motivation for this change
Things done

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

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

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

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 113937 run on x86_64-darwin 1

1 package built:

wllvm

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

wllvm:

Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
You have the following options to fix this:

Tell pytest(CheckHook) where to find the tests included in the package
Check if the GitHub Repo contains tests but they are not shipped with Pypi. If so please switch to fetchFromGitHub.
If the Packages does not contain any tests add 'doCheck = false;' and a pythonImportsCheck.


License Administrator

Is this role just for Office 365 licenses or all applications?

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

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Administrator role permissions in Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/directory-assign-admin-roles.md
Service: active-directory
GitHub Login: @curtand
Microsoft Alias: curtand

@DeanGross The License administrator role is specifically for managing licenses from Office 365 and Azure AD portal. User can manage Intune, office 365, Azure AD and EMS licenses when assigned to this role.

@DeanGross I am checking in to see if the above answer was helpful. Let me know if you have any other questions.

@DeanGross We have not heard from you in a while.We will now proceed to close this thread. If you have further questions, please tag me in the comments and I will gladly continue the conversation.


syscall: Setrlimit behavior changed on Go 1.12 on macos

We've noticed a while back that when setting the file descriptor allowance on macos, Go restricts it to a lower value than requested:
package main

import (
	"fmt"
	"syscall"
)

func main() {
	// Get the current file descriptor limit
	var limit syscall.Rlimit
	if err := syscall.Getrlimit(syscall.RLIMIT_NOFILE, &limit); err != nil {
		panic(err)
	}
	// Try to update the limit to the max allowance
	limit.Cur = limit.Max
	if err := syscall.Setrlimit(syscall.RLIMIT_NOFILE, &limit); err != nil {
		panic(err)
	}
	// Try to get the limit again and see where it's at
	if err := syscall.Getrlimit(syscall.RLIMIT_NOFILE, &limit); err != nil {
		panic(err)
	}
	fmt.Println(limit.Cur)
}

The above code on Linux works as expected, setting the limit to the max returned by the syscall package.
On macos with Go 1.11.5 (and prior) it silently set it to the minimum of limit.Max and 10240. We had to patch our code to explicitly check the actual value set instead of relying on no-error-returned since we falsely alloted more fds to databases than available.
On macos with Go 1.12, our code again started to blow up, now returning invalid argument errors instead of the silent capping previously.
I'd probably say both behaviors are bad, but the new one is a lot worse than the previous. Now it only tells us it's invalid, but unless you know the magic number defined in the your own kernel version, there's no way for user code to figure out how many fds are actually available.

CC @randall77

Yes, this is unfortunate.  We switched to using the implementation of getrlimit in libSystem.dylib instead of raw system calls, and that's the behavior we get.  In particular, the new implementation lies to us about the maximum value available.
I had to modify the TestRlimit with code like this:
	if runtime.GOOS == "darwin" && set.Cur > 10240 {
		// The max file limit is 10240, even though
		// the max returned by Getrlimit is 1<<63-1.
		// This is OPEN_MAX in sys/syslimits.h.
		set.Cur = 10240
	}

I suppose we could make our implementation of Getrlimit incorporate this fix internally.  But my view of the syscall package is that you get what the OS provides, warts and all. And one day Apple may fix this issue (or change OPEN_MAX) and then any patch we have is obsolete.
You can always binary search for the largest allowable value. Yuck.

Note that this is not just a Go problem - C people run into similar issues:
https://postfix-devel.postfix.narkive.com/kVL334x7/mac-os-x-and-setrlimit-2

Ah yes, I wasn't particularly complaining that Go is doing something wrong (given the restrictions), just trying to figure out what the best solution is. I guess my main complaint wasn't that Go works one way or the other, rather that it changed it behavior, breaking existing code.
We can fix it to handle both cases (i.e. hard code that magic number), just would like to know which solution will Go stay with in the end so we don't have to do a 3rd fixup round for Go 1.12.1 :)

Going forward on Darwin Go is always going to call the C function.

Found that on Go 1.11.5, this code:
func init() {
	raiseFdLimit = func() {
		var rLimit syscall.Rlimit
		err := syscall.Getrlimit(syscall.RLIMIT_NOFILE, &rLimit)
		if err != nil {
			log.Fatal("Error Getting Rlimit ", err)
		}

		if rLimit.Cur < rLimit.Max {
			rLimit.Cur = rLimit.Max
			err = syscall.Setrlimit(syscall.RLIMIT_NOFILE, &rLimit)
			if err != nil {
				log.Warn("Error Setting Rlimit:", err)
			}
		}

		err = syscall.Getrlimit(syscall.RLIMIT_NOFILE, &rLimit)
		log.Info("File descriptor limit is:", rLimit.Cur)
	}
}

reports File descriptor limit is: 24576. On Go 1.12 is reports invalid argument. So I've added the workaround from above (https://github.com/golang/go/issues/30401#issuecomment-467530109) but instead of 10240, I've used 24576 and that returns the behavior of my server compiled with Go 1.12 to the same as it was on Go 1.11.5.

@jeffallen What Mac OS are you running?
I can't get ask for more than 10240 on Sierra (10.12.6) without getting an invalid argument error.

Here is an excellent explanation: https://unix.stackexchange.com/questions/350776/setting-os-x-macos-bsd-maxfile
It says this could be a problem on FreeBSD and OpenBSD as well.

For what it's worth;
On my Mojave 10.14.4, my tests tell me that I can set values in Setrlimit <= the value of limit as defined in /Library/LaunchDaemons/limit.maxfiles.plist (I assume that  10240 is the default?).
This can be set for the current session with something ala:
sudo launchctl limit maxfiles 100 200000

In the above example, I get invalid argument for any argument  to Setrlimit> 100.
If I run Setrlimit as sudo, I can use any value.

This issue was originally filed under the Go1.12.1 milestone with NeedsInvestigation and release-blocker issues. Many point releases have been made since then, and this issue was continuously moved to the next point milestone.
Since there's no fix yet and it's not blocking the 1.12 point releases, I'll move this to Go 1.13 milestone so we can look at it again for 1.13, and determine if it should block that release or not.


Is it possible to cache .gif files?

Markwon version: 4.6.2

when i did this,
// init Coil image loader
ImageLoader.Builder(this)
  .componentRegistry {
      if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) {
          add(ImageDecoderDecoder(this@MyApplication))
      } else {
          add(GifDecoder())
      }
  }
  .build()

val coilImagesPlugin = CoilImagesPlugin.create(context, context.imageLoader)

Markwon.builder(context)
    .usePlugins(coilImagesPlugin)
    .build()

all image files (like .png or .jpg ) display as expected,
but .gif files are not playing (seems like just showing one of its frames).
then I try this,
val imagesPlugin = ImagesPlugin.create { plugin ->
    plugin.addMediaDecoder(GifMediaDecoder.create())
}

Markwon.builder(context)
    .usePlugins(imagesPlugin)
    .build()

all images are showing as expected ( .gif files are playing),
but they are not cached with the implementation.
so I tried to combine them together like this,
Markwon.builder(context)
    .usePlugins(coilImagesPlugin)
    .usePlugins(imagesPlugin)
    .build()

the result is, all image files display without cache.
I wonder that is it possible to play .gif files and show .png / .jpg files with cache?

after tried to start() manually in onSuccess , .gif files work fine and cache is available with Coil.

@gcobc12677 would you mind sharing how you achieved this? I'm building a new image request and applying some transformations, at which point did you utilise start


Create single restaurant Description page

Steps to reproduce

Go to any city page -> Restaurants
The UI looks like this

Expected behaviour

Open up a new activity on clicking any card. That card should have the restaurant details received from the API

Endpoint to use
get-restaurant/<int:restaurant_id>
https://github.com/project-travel-mate/server/issues/110

Hi @Swati4star how should look the new screen? (RestaurantDetailsActivity)

@valericaplesu We don't have any UI mockups for this. Create a basic UI from the information received.
This is the sample info we will receive
{
    "restaurant_id": 4,
    "restaurant_name": "China 1",
    "restaurant_url": "https://www.zomato.com/dummy/china-1-1-aaaa?utm_source=api_basic_user&utm_medium=api&utm_campaign=v2.1",
    "restaurant_address": "G 05, Morya Landmark 1, Off Link Road, Andheri Lokhandwala, Andheri West, Mumbai, AAAA, Dummy",
    "restaurant_longitude": 28.88,
    "restaurant_latitude": 73.99,
    "avg_cost_2": 400,
    "price_range": 4,
    "currency": "",
    "featured_image": "",
    "user_aggregate_rating": 0,
    "user_rating_votes": 0,
    "is_delivering_now": false,
    "has_table_booking": false,
    "cuisines": "Chinese, Indian",
    "phone_numbers": ""
}

Add scrollable image at top (similar to city info page)
then, restaurant name & add more info below that.

@Swati4star got it. I'm working on it

@Swati4star can I use Kotlin? it surely simplify things

Can we use java only for this? We don't want combination of both in a single project

Can we use java only for this? We don't want combination of both in a single project

That's the beauty of Kotlin, it works perfectly with Java and the other way around. And definitely we can improve a lot especially with the help of async(). I'll do the pull request with it, and if you do not agree I can convert it back to Java :(

Sure. Go ahead with Kotlin :)


Replace multiprocessing with loky

This PR replaces multiprocessing.Pool with loky.get_reusable_executor. The advantage of this is that processes that could have been killed by an out of memory problem will now cause the program to fail instead of letting it hang indefinitely. This has been affecting a lot of users of OpenDroneMap that are trying to process large datasets. #190
List of changes:

Added loky to the directory tree. Pulling in the library with pip install loky would remove the need for this addition, but there's no way to disable a warning on OSX that notifies the user that "semaphore are broken on OSX". This is a warning that can be safely dismissed (see https://github.com/tomMoral/loky/issues/103) in our case. I made a fork of the library and uncommented the warning (https://github.com/tomMoral/loky/compare/master...pierotofy:master), but didn't want to add my personal repository in the list of pip requirements. Perhaps the better way to go would be for mapillary to make a fork and then install this dependency via github. Using warnings.catch_warnings is not possible to intercept and dismiss the warning (see https://bugs.python.org/issue6647). If I need to change this, please let me know. Also, if we don't care about the warning messages, we could just install from directly from pip, OSX users will see the warning until we upgrade to Python 3.
Replaced Pool with get_reusable_executor.
Moved logging.basicConfig into a separate module (log.setup()); the reason for this is that get_reusable_executor needs logging.basicConfig to be set within the forked process, otherwise all logging activity disappears from the console.
Added calls to log.setup() at the top of each function that's designed to be parallel and outputs some logs.

Thanks for doing this, @pierotofy.
I think it is better not to include loky code in OpenSfM. It would be a mess to maintain. Installing from your fork in github would be better but I think we can skip that too and just live with the warning.
Let's use default loky.

Sounds good, thank you for reviewing these changes so quickly @paulinus !


Add create_compute_pipelines to device

Looks good, thanks!.

That was quick! Would you mind pushing a new version in the next days on crates.io?

Yes I'll do it later today when I am on my dev machine again. I could also give you repository rights if that would make it easier for you.

Thanks! There is no need for access rights imo, most of the groundwork for the low level vulkan backend is done. Therefore I probably don't need a new version in the near future.

@msiglreith done

Great! Just noticed that 0.15.6 was already published a week ago. Should this rather be 0.15.7?

You are right it didn't even publish. cargo-release is a bit buggy. I just manually published it now.

Thanks, it works 🎉


app.oauth.authenticate middleware behavior

app.get('/secret', app.oauth.authenticate(), function(req, res) {
  // Will require a valid access_token.
  res.send('Secret area');
});

I've implemented something similar to the above, it seems that the handler always get called regardless of the validity of the Bearer token?

My getAccessToken handler always invokes the callback by supplying a dummy user object.


Jobs not created when using V2 config

Describe the bug
When using the V2 config and validating against https://ibis4example.ibissource.org/rest/ibisdoc/ibisdoc.xsd, creating a job is done using a  element in the Configuration/Module. When loading the config, the Job is not created, because the <job> needs to be a child of <scheduler>, but that element is not created.
Reporter
Ricardo
Category
Use following Tags to categorize your bug report

Frank Framework

To Reproduce
Steps to reproduce the behavior:

Load the attached configuration
Check the Scheduler page in the console
Don't see the job
Open the Show configurations page
See the job without parent scheduler element

Expected behavior
To have the job being created correctly. This can be done by either:

create the scheduler-element as part of canonicalize.xsl
remove the need for the scheduler-element completely (keep for backwards compatibility)
fix this in the new upcoming XSD

Screenshots
N/A
Environment:

Application server (f.e. Tomcat/WebSphere/JBOSS/Jetty): Tomcat
Browser [e.g. chrome, safari]: N/A
IAF Version [e.g. 7.5 RC3, 7.6]: 7.6-20201217.135711

Additional context
ConfigurationSchedule.txt

Waiting for new Frank!Doc (Martijn is working on it)

Waiting for new Frank!Doc (Martijn is working on it)


@ references break inline secrets if they are inside one

It seems like if I reference an item/compendium/scene/journal inside a secret block, the secret block does not work.
ex:
<EMAIL_ADDRESS>

Noted. Will look into it later, but recursive Enrichers might be an issue

It looks like this is due to how v10 handles TextEnrichers.  I am able to replicate the issue when using the short code, but manually copying the HTML into the secret block does work.
Unfortunately there isn't anything that can be done at this time.