text:
Failed to load the Remote-FTP package

File.properties is not a function
Hide Stack Trace
TypeError: File.properties is not a function
at /Users/14681820/.atom/packages/Remote-FTP/lib/file.js:10:8
at Object. (/Users/14681820/.atom/packages/Remote-FTP/lib/file.js:58:2)
at Module._compile (/Applications/Atom.app/Contents/Resources/app.asar/src/native-compile-cache.js:103:30)
at Object.defineProperty.value [as .js] (/Applications/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:208:21)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:313:12)
at Module.require (module.js:366:17)
at require (/Applications/Atom.app/Contents/Resources/app.asar/src/native-compile-cache.js:50:27)
at Object. (/Users/14681820/.atom/packages/Remote-FTP/lib/directory.js:4:10)
at Module._compile (/Applications/Atom.app/Contents/Resources/app.asar/src/native-compile-cache.js:103:30)
at Object.defineProperty.value [as .js] (/Applications/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:208:21)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:313:12)
at Module.require (module.js:366:17)
at require (/Applications/Atom.app/Contents/Resources/app.asar/src/native-compile-cache.js:50:27)
at Object. (/Users/14681820/.atom/packages/remote-ftp/lib/client.js:9:60)
at Module._compile (/Applications/Atom.app/Contents/Resources/app.asar/src/native-compile-cache.js:103:30)
at Object.defineProperty.value [as .js] (/Applications/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:208:21)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:313:12)
at Module.require (module.js:366:17)
at require (/Applications/Atom.app/Contents/Resources/app.asar/src/native-compile-cache.js:50:27)
at Object. (/Users/14681820/.atom/packages/remote-ftp/lib/remote-ftp.js:4:24)
at Module._compile (/Applications/Atom.app/Contents/Resources/app.asar/src/native-compile-cache.js:103:30)
at Object.defineProperty.value [as .js] (/Applications/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:208:21)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:313:12)
at Module.require (module.js:366:17)
at require (/Applications/Atom.app/Contents/Resources/app.asar/src/native-compile-cache.js:50:27)
at Package.module.exports.Package.requireMainModule (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:715:27)

Please update to the newest update (0.10.5) in a couple of minutes and reopen if the issue persists

Thank you!

authors: avalanchedesigns jpxd

repo: mgrenier/remote-ftp


text:
Lack of documentation

Configuration file : node-ftp, TLS, ssh2
How to
Shortcut

Would be highly appreciated, because right now I don't know how to use your package.

@buschtoens I added a Getting started in the readme that should get you up and ready.

Thank ya,
The problem is, I already tried exactly that. But nothing seems to happen.
I use Windows 8.1 64bit.

Can you elaborate on what step does nothing?

Remote-FTP not present in Packages : try disabling/enabling the plugin
Sidebar not toggling : try reload the editor with CTRL+ALT+R
Connect does nothing : if your using FTP, open the console CTRL+ALT+I and search the FTP log for error

@buschtoens might be related to #21

Sorry for not responding sooner. Will look into that this evening.
Am 22.01.2015 19:17 schrieb "Michael Grenier" notifications@github.com:

@buschtoens https://github.com/buschtoens might be related to #21
https://github.com/mgrenier/remote-ftp/issues/21
—
Reply to this email directly or view it on GitHub
https://github.com/mgrenier/remote-ftp/issues/7#issuecomment-71069557.

@mgrenier
In regards to

Configuration file : node-ftp, TLS, ssh2

How about a section in showing both config types and we can borrow the descriptions of node-ftp and ssh2's respective options and inline them as comments and edit as needed. For example:
SFTP Configuration Options
{
    "protocol": "sftp",
    "host": "example.com", // string - Hostname or IP address of the server. Default: 'localhost'
    "port": 22, // integer - Port number of the server. Default: 22
    "user": "user", // string - Username for authentication. Default: (none)
    "pass": "pass", // string - Password for password-based user authentication. Default: (none)
    "remote": "/",
    "agent": "", // string - Path to ssh-agent's UNIX socket for ssh-agent-based user authentication. Windows users: set to 'pageant' for authenticating with Pageant or (actual) path to a cygwin "UNIX socket." Default: (none)
    "privatekey": "", // mixed - Buffer or string that contains a private key for either key-based or hostbased user authentication (OpenSSH format). Default: (none)
    "passphrase": "", // string - For an encrypted private key, this is the passphrase used to decrypt it. Default: (none)
    "hosthash": "", // string - 'md5' or 'sha1'. The host's key is hashed using this method and passed to the hostVerifier function. Default: (none)
    "ignorehost": true,
    "connTimeout": 10000, // integer - How long (in milliseconds) to wait for the SSH handshake to complete. Default: 10000
    "keepalive": 10000 // integer - How often (in milliseconds) to send SSH-level keepalive packets to the server (in a similar way as OpenSSH's ServerAliveInterval config option). Set to 0 to disable. Default: 10000
}

FTP & FTPS Configuration Options
{
    "protocol": "ftp",
    "host": "example.com", // string - The hostname or IP address of the FTP server. Default: 'localhost'
    "port": 21, // integer - The port of the FTP server. Default: 21
    "user": "user", // string - Username for authentication. Default: 'anonymous'
    "pass": "pass", // string - Password for authentication. Default: 'anonymous@'
    "remote": "/",
    "secure": false, // mixed - Set to true for both control and data connection encryption, 'control' for control connection encryption only, or 'implicit' for implicitly encrypted control connection (this mode is deprecated in modern times, but usually uses port 990) Default: false
    "secureOptions": null, // object - Additional options to be passed to tls.connect(). Default: (null) see http://nodejs.org/api/tls.html#tls_tls_connect_options_callback
    "connTimeout": 10000, // integer - How long (in milliseconds) to wait for the control connection to be established. Default: 10000
    "pasvTimeout": 10000, // integer - How long (in milliseconds) to wait for a PASV data connection to be established. Default: 10000
    "keepalive": 10000 // integer - How often (in milliseconds) to send a 'dummy' (NOOP) command to keep the connection alive. Default: 10000
}

privatekey should probably note that the string should be a file path to the private key...  I know it should have been my first choice, but I encoded the private key file with \n and put that in... it was after I determined that it really is supposed to be a path.

The README says Buffer or string that contains a private key for either key-based or hostbased user authentication (OpenSSH format)
It should have worked if you had left out the \ns

But Wiki > Settings says:

key: set privatekey to the path to your private key, set passphrase if key is encrypted

Funny...
We use the mscdex/ssh2 module for SFTP. The keyparser used there expects a buffer or a string in the OpenSSH format
BUT: We seem to support both path and key if I understand this line correctly: https://github.com/mgrenier/remote-ftp/blob/96bfd7d9a46d7b2b15ac32362c129c78c0b57a2e/lib/client.js#L170

@jpxd no, line means that if self.info.privatekey  is set then read file that it targets to. so remote-ftp  supports only path

Ah you`re right. I'll update it in the README

source: gharchive/issue

authors: buschtoens hatzopoulos jpxd mgrenier tracker1 wasikuss

license: mit

license_type: permissive

license_source: bigquery


text:
Add helper to display item count information

Similar to what will_paginate has:

I always found this type of information handy and have included it in my Rails projects.  I didn't see this option available for this plugin (sorry, not sure if that is the right term) and added a simple version of the functionality myself.
Would you accept a PR for this?

You provide the %Scrivener.Page{} which has that information easily accessible so I'm not following whats to be gained by offering a helper to do that? Additionally, the copy would need to be swappable and I wouldn't want to introduce a gettext dependency. What would your helper do exactly?

What would your helper do exactly?

The bottom part, "Displaying x 1-20, of 59 in total"

You provide the %Scrivener.Page{} which has that information easily accessible

It provides page_number, page_size, total_entries, total_pages, right?  Does it provide the indexes of the entries on the current page?
Here is the code if it's helpful for anyone else.  I'm new to elixir so I'm probably doing something fundamentally wrong and I need to cover some edge cases:
 def pagination_info(page, type) do
  total_entries = page.total_entries

  first_index = find_first_index(page)
  if first_index <= total_entries do
    content_tag :p, class: "pagination-info" do
      pagination_info_content(page, type)
    end
  end
end

defp pagination_info_content(page, type) do
  page_size = page.page_size
  total_entries = page.total_entries
  first_index = find_first_index(page)
  last_index = find_last_index(page)

  if page_size >= total_entries do
    "Displaying all #{type}"
  else
    start = "Displaying "
    indexes =
      if page_size > 1 and first_index != last_index do
        "#{first_index} - #{last_index}"
      else
        first_index
      end

    indexes = content_tag :strong, indexes
    total_entries = content_tag(:strong, total_entries)

    [start, type, " ", indexes, " of ", total_entries, " in total"]
  end
end

defp find_first_index(page) do
  (page.page_number - 1) * page.page_size + 1
end

defp find_last_index(page) do
  page_number = page.page_number
  page_size = page.page_size
  total_entries = page.total_entries

  index = if page_size > 1, do: page_number * page_size
  cond do
    index > total_entries -> total_entries
    true -> index
  end
end

Use like so:
<%= pagination_info @page, "Widgets" %>

I think this should be a helper in your application since that text you have in there will need to be configurable and internationalized w/ gettext. If you really feel it something others could use, feel free to create your own package on hex.

authors: dsandstrom mgwidmann


text:
🛑 Traffic Dashboard Mediahuis Nederland is down

In 4a0417d, Traffic Dashboard Mediahuis Nederland (https://traffic-tmg.mediahuis.nl/api/system/status) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Traffic Dashboard Mediahuis Nederland is back up in bb57574.

repo: mh-data-science/upptime

license: MIT

license_source: github-api


text:
🛑 Traffic Dashboard Mediahuis Nederland is down

In cc04830, Traffic Dashboard Mediahuis Nederland (https://traffic-tmg.mediahuis.nl/api/system/status) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Traffic Dashboard Mediahuis Nederland is back up in abf96da.


text:
Customice Output directory for html file

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Problem ist, that when using a new version it is necessary to rename the directory or to adjust the path to the favorits html file in Browsers where this file is set as default page.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Customice the place where the html file has to be stored. So it might only be necessary to respecify it when upgrading to a newer version. But Browser links can be stay the same.
This should be done by adding an optional flag. If this flag has never been used, use the default place to store the file like in Version 1.00. If flag was set once store html by default in this location, so it is necessary to sotre the location for reuse. If flag is set another time update and save to the new location.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

ToDo:

[ ] Store Path information in .env file
[ ] Implement functionality to give a path where to store html file


text:
Add Cake.Scripting.Abstractions

Shoudl include interfaces, and base entitites...

Closed via #7


text:
Restructure

Rename Cake.Scripting.Core --> Cake.Scripting
Create Cake.Scripting.Abstractions
Create Cake.Scripting.Transport

@mholo65 Any issues that should be closed?

@patriksvensson hehe ofc! Thanks for noticing ;)

source: gharchive/pull-request


text:
Replace environment variables in matchers as appropriate

I'd like to be able to use environment variables within my l4 matchers, e.g.:
"match": [
    {
        "tls": {
            "sni": [
                "{env.DOMAIN}"
            ]
        }
    },
    {
        "http": [
            {
                "host": [
                    "{env.DOMAIN}"
                ]
            }
        ]
    }
],

Unfortunately, it doesn't seem like l4 uses the replacer to do this yet. Would be great if this was possible!

This should be pretty straightforward, I think. Anyone want to submit a patch?

@tls_sni_env tls sni {env.DOMAIN} is now possible after https://github.com/caddyserver/caddy/pull/6480 is merged.

authors: avirut mholt vnxme


text:
Caddy down without any useful information

(Are you asking for help with using Caddy? Please use our forum instead: https://forum.caddyserver.com. If you are filing a bug report, please answer the following questions. If your issue is not a bug report, you do not need to use this template. Either way, please consider donating if we've helped you. Thanks!)
1. What version of Caddy are you running (caddy -version)?
0.9.3
2. What are you trying to do?
nothing, it was usually running without problem
here's the caddy.service systemd file:
[Unit]
Description=Caddy HTTP/2 web server
Documentation=https://caddyserver.com/docs
After=network-online.target
Wants=network-online.target systemd-networkd-wait-online.service

[Service]
#Restart=on-failure

; User and group the process will run as.
User=x
Group=users

; Letsencrypt-issued certificates will be written to this directory.
Environment=HOME=/home/x/.caddy

; Always set "-root" to something safe in case it gets forgotten in the Caddyfile.
ExecStart=/usr/bin/caddy -log stdout -agree=true -conf=/home/x/web/Caddyfile -root=/home/x/web/public
ExecReload=/bin/kill -USR1 $MAINPID

; Limit the number of file descriptors; see `man systemd.exec` for more limit settings.
LimitNOFILE=1048576
; Unmodified caddy is not expected to use more than that.
LimitNPROC=64

; Use private /tmp and /var/tmp, which are discarded after caddy stops.
#PrivateTmp=true
; Use a minimal /dev
#PrivateDevices=true
; Hide /home, /root, and /run/user. Nobody will steal your SSH-keys.
#ProtectHome=true
; Make /usr, /boot, /etc and possibly some more folders read-only.
ProtectSystem=full
; … except /etc/ssl/caddy, because we want Letsencrypt-certificates there.
;   This merely retains r/w access rights, it does not add any new. Must still be writable on the host!
ReadWriteDirectories=/home/x/.caddy

; The following additional security directives only work with systemd v229 or later.
; They further retrict privileges that can be gained by caddy. Uncomment if you like.
; Note that you may have to add capabilities required by any plugins in use.
;CapabilityBoundingSet=CAP_NET_BIND_SERVICE
;AmbientCapabilities=CAP_NET_BIND_SERVICE
;NoNewPrivileges=true

[Install]
WantedBy=multi-user.target

and the Caddyfile:
x.com, pub.x.com, http://pub.test.x.com {
        root /home/x/web/public
        proxy / 127.0.0.1:8083 {
                proxy_header Host {host}
                proxy_header X-Forwarded {remote}
                proxy_header X-Real-IP {remote}
                proxy_header X-Forwarded-Proto {scheme}
        }
}

sub.x.com, http://sub.test.x.com {
        root /home/x/web/public
        proxy / 127.0.0.1:8088 {
                proxy_header Host {host}
                proxy_header X-Forwarded {remote}
                proxy_header X-Real-IP {remote}
                proxy_header X-Forwarded-Proto {scheme}
        }
}

and the journalctl -r output (down around 08:40, i restart the service at 10:15):
Dec 15 10:21:07 x caddy[22654]: 2016/12/15 10:21:07 [INFO] www.rechizitelemele.ro - No such site at :80 (Remote: 137.116.232.1$
Dec 15 10:18:13 x caddy[22654]: 2016/12/15 10:18:13 [INFO] www.badabum.ro - No such site at :80 (Remote: 104.41.208.86, Refere$
Dec 15 10:15:13 x caddy[22654]: http://sub.x.com
Dec 15 10:15:13 x caddy[22654]: http://pub.x.com
Dec 15 10:15:13 x caddy[22654]: http://x.com
Dec 15 10:15:13 x caddy[22654]: http://sub.test.x.com
Dec 15 10:15:13 x caddy[22654]: http://pub.test.x.com
Dec 15 10:15:13 x caddy[22654]: https://sub.x.com
Dec 15 10:15:13 x caddy[22654]: https://pub.x.com
Dec 15 10:15:13 x caddy[22654]: https://x.com
Dec 15 10:15:13 x caddy[22654]: WARNING: proxy_header is deprecated and will be removed soon; use header_upstream instead.
Dec 15 10:15:13 x caddy[22654]: Activating privacy features... done.
Dec 15 10:15:12 x systemd[1]: Started Caddy HTTP/2 web server.
Dec 15 08:38:31 x caddy[308]: 2016/12/15 08:38:31 [INFO] www.dpap.ro - No such site at :80 (Remote: 104.41.208.86, Referer: )
Dec 15 08:35:57 x caddy[308]: 2016/12/15 08:35:57 [INFO] www.b-mall.ro - No such site at :80 (Remote: 137.116.232.123, Referer$
Dec 15 08:34:22 x caddy[308]: 2016/12/15 08:34:22 [INFO] www.terrasport.ro - No such site at :80 (Remote: 104.41.208.86, Refer$
Dec 15 08:33:03 x caddy[308]: 2016/12/15 08:33:03 [INFO] www.flanco.ro - No such site at :80 (Remote: 104.41.208.86, Referer: )
Dec 15 08:31:01 x caddy[308]: 2016/12/15 08:31:01 [INFO] www.clickshop.ro - No such site at :80 (Remote: 104.41.208.86, Refere$
Dec 15 08:29:10 x caddy[308]: 2016/12/15 08:29:10 [INFO] www.elara.ie - No such site at :80 (Remote: 104.41.208.86, Referer: )
Dec 15 08:29:07 x caddy[308]: 2016/12/15 08:29:07 [INFO] www.okian.ro - No such site at :80 (Remote: 137.116.232.123, Referer:$
Dec 15 08:25:12 x caddy[308]: 2016/12/15 08:25:12 [INFO] www.veloteca.ro - No such site at :80 (Remote: 104.41.208.86, Referer$
Dec 15 08:23:06 x caddy[308]: 2016/12/15 08:23:06 [INFO] www.pcfun.ro - No such site at :80 (Remote: 104.41.208.86, Referer: )
Dec 15 08:22:17 x caddy[308]: 2016/12/15 08:22:17 [INFO] www.okian.ro - No such site at :80 (Remote: 104.41.208.86, Referer: )
Dec 15 08:16:49 x caddy[308]: 2016/12/15 08:16:49 [INFO] Done checking OCSP staples
Dec 15 08:16:49 x caddy[308]: 2016/12/15 08:16:49 [INFO] Scanning for stale OCSP staples
Dec 15 08:16:49 x caddy[308]: 2016/12/15 08:16:49 [INFO] Done checking certificates
Dec 15 08:16:49 x caddy[308]: 2016/12/15 08:16:49 [INFO] Scanning for expiring certificates

Does it happen if you don't use systemd? Users have reported problems with it in the past.

Not yet tested, umm.. this only happened once (5 days since started), never happened before

Has it happened since you tried it without systemd? Caddy doesn't take itself down...

Hello @kokizzu,
any updates?

no, haven't down since then

Excellent. It was probably a systemd issue then... closing. ;)

authors: elcore kokizzu mholt

repo: mholt/caddy


text:
Make characteristic poly default for LFSRs

Fixes #293

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (d1c65db) 76.39% compared to head (359a7fd) 77.65%.

Files
Patch %
Lines

src/sdr/_sequence/_lfsr.py
81.63%
9 Missing :warning:

Additional details and impacted files
@@                Coverage Diff                @@
##           release/0.0.x     #294      +/-   ##
=================================================
+ Coverage          76.39%   77.65%   +1.26%
=================================================
  Files                 56       56
  Lines               3292     3290       -2
=================================================
+ Hits                2515     2555      +40
+ Misses               777      735      -42

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: codecov-commenter mhostetter

repo: mhostetter/sdr


text:
add UK band charts

Description
Add UK band charts.
Fixes #66
Type of change
Resource update
How has this been tested?
N/A
Checklist

[x] Issue exists for PR
[ ] Code reviewed by the author
[ ] Code documented (comments or other documentation)
[ ] Changes tested
[ ] All tests pass (see DEVELOPING.md, if it exists)
[x] CHANGELOG.md updated if needed
[ ] Informative commit messages
[ ] Descriptive PR title

#66

Images have been compressed with pngquant.

Thanks for contributing these! We'll make a release once the bulgarian charts are merged too.

repo: miaowware/qrm-resources


text:
browsersync don't work with connect-php

// ### Server tasks

// server w/ php
gulp.task('connect', function() {
  connect.server({
    hostname: 'localhost',
    bin: '/usr/bin/php',
    ini: '/etc/php5/cli/php.ini',
    base: './dist',
    keepalive: true,
  }, function (){
    browserSync.init({
      proxy: "localhost:8000",
      open: false
    });
  });
});

// server w/out php
gulp.task('connect-no-php', function() {
  browserSync.init({
    server: "./dist",
    open: false
  });
});

without connect-php browserSync.reload() and browserSync.stream() works fine. when i run connect task, it just don't want to refresh browser or inject styles, until i refresh page manually.
and...it happens only when i run linux. on windows there is no problem.
what it can be?

what do you get from which php?

/usr/bin/php

I had that problem just right now on a windows installation and figured out that browser-sync does not connect with a raw php file as it doesn't find the body tag to append some javascript. It worked fine with a html file.
So, I just added
<script async="" src="/browser-sync/browser-sync-client.2.11.1.js"></script>

on the bottom of my simple php file. And that worked for me.

For now, @buecklers response should be followed: https://github.com/micahblu/gulp-connect-php/issues/20#issuecomment-173598269.

just add
echo "<body></body>";
at the start of php script
its working for me.

authors: ashishsciit buecklers gosteev grmartin micahblu mjrev

repo: micahblu/gulp-connect-php


text:
Time sensitivity

We don't have very long to archive all these projects!  However, we have a bit more time than I previously thought.   https://github.com/LLK/scratch-www/pull/6824 says that progress on the token auth cannot proceed until file uploads are fixed.  Let's keep archiving, everyone!

I've got 80,000 to 100,000 - if you archive any more make sure to start from 100,001 or higher.

Awesome, will do!

repo: micahlt/sbarchive


text:
GitHub Actions

Travis is dead.  Long live GitHub.  Credit to @cylammarco.

Working on it. Deployment pending on PR #89, #92.

authors: cylammarco michaelJwilson

repo: michaelJwilson/desihigh


text:
v0.5.3

Display modes !
Mac compat

fixes #61  and fixes #53

fixes #53  also but github didn't understand when it was in prose

Codecov Report

Merging #62 (825f3a8) into master (b35fcb4) will decrease coverage by 0.60%.
The diff coverage is 10.00%.

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
- Coverage   61.35%   60.75%   -0.61%
==========================================
  Files          25       25
  Lines        2484     2423      -61
==========================================
- Hits         1524     1472      -52
+ Misses        960      951       -9

Impacted Files
Coverage Δ

src/display.rs
0.00% <0.00%> (ø)

src/main.rs
9.41% <5.55%> (+1.47%)
:arrow_up:

src/launcher.rs
73.14% <75.00%> (-1.86%)
:arrow_down:

src/interpreters/Python3_jupyter.rs
47.68% <100.00%> (+0.86%)
: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 b35fcb4...825f3a8. Read the comment docs.

authors: codecov-commenter michaelb

repo: michaelb/sniprun


text:
Issue with geoWithin()

I keep running into this issue: "PHPUnit_Framework_MockObject_RuntimeException: Cannot mock Doctrine\MongoDB\Query\Builder::geoWithin() because a class or interface used in the signature is not loaded". I originally tried just mocking the document manager directly and hit the issue as well. I thought your module might resolve this for me, but I am still getting it. I'm not sure what the issue is other then I am sure I am missing this class.

@bsacks99 Did you install the jmikola/geojson Composer package? https://github.com/michaelmoussa/doctrine-qbmocker/blob/master/composer.json#L24
I didn't explicitly require it because some people may only be using this library for ORM, but it's listed in the suggest and in the readme for ODM.
Running composer require jmikola/geojson should take care of this.

Thanks for the quick response Michael! I'll install the dependency and update the issue when resolved.

Closing old issue with no further response. Please re-open if this is still an issue.

repo: michaelmoussa/doctrine-qbmocker


text:
Memory leak.

Version: 2.10
Original reporter: hao huang
I just use MxmlLoadFile(File),and then mxmlDelete(TopNode),there will be a leak.
I found that the  node would be not released.And some node like value would be not released,but ,node like  can be released.This is wired.

Same as issue #183

repo: michaelrsweet/mxml


text:
Automatize db migration and seed for core schema and data

It must be automatized for every tenant db, or at least with a user action from front-end

@xtya Can you help us to find a suitable way that make sense?

repo: michaelsogos/primebrick-core


text:
lineSplit

I have a piece of code that I've needed recently in two unrelated projects, so I thought I'd see if you would be interested in adding it to streaming-bytestring. It's a function that splits up a stream after every nth newline. In the PR for lineSplit, documentation is included that explains this better and demonstrates it.
My uses for this have been (1) chunking syslog messages and (2) bulk inserts into elasticsearch. Both of these are newline-delimited protocols where I have needed to break stream into 1000-line chunks.

Any particular reason for not splitting the lines up then chunking the resultant Stream? Just to avoid the extra steps?

Basically, yes. I wanted to be able to reuse the original newline characters that were present instead of reconstructing something equivalent to the original chunk. This should ultimately translate to fewer memcpy calls when the bytestring is consumed. I'm also able to use Data.ByteString.count, which helps make things faster when the line count spans multiple chunks.

authors: andrewthad ivan-m

repo: michaelt/streaming-bytestring


text:
Suggestion for active tab highlighting

Hello:
This is the only place where I could contact you.
Thank you for the high contrast UI & syntax themes.
Since the tabs for each file are of ALMOST same color,
it is hard to see which is active since the active tab is only slightly darker than
the inactive tabs.
The LCD screen angle can make the contrast between active and inactive tabs much poorer.
It would be a good idea to swap the tab background color and tab title text color
to indicate an active tab, almost similar to browser tab highlighting.
For dark theme, you now have white text and black (or off-black) tab background color for all tabs.
So with the suggestion, you should have an active tab of
black text and white tab background color.
Or if it is easier, just use a different high contrast color text on the active tab.
Thank you.

Thank you.  I will try to fix this as soon as possible.
Michael

From: Hoe-Phuan Ng [notifications@github.com]
Sent: Thursday, June 11, 2015 7:05 AM
To: michardy/contrast-ui
Subject: [contrast-ui] Suggestion for active tab highlighting (#1)
Hello:
This is the only place where I could contact you.
Thank you for the high contrast UI & syntax themes.
Since the tabs for each file are of ALMOST same color,
it is hard to see which is active since the active tab is only slightly darker than
the inactive tabs.
The LCD screen angle can make the contrast between active and inactive tabs much poorer.
It would be a good idea to swap the tab background color and tab title text color
to indicate an active tab, almost similar to browser tab highlighting.
For dark theme, you now have white text and black (or off-black) tab background color for all tabs.
So with the suggestion, you should have an active tab of
black text and white tab background color.
Or if it is easier, just use a different high contrast color text on the active tab.
Thank you.
—
Reply to this email directly or view it on GitHubhttps://github.com/michardy/contrast-ui/issues/1.

Than you.
When I look at settings view, the blue font is good, but could use a little better contrast.
If you can make the blue a little lighter ( e.g. bright light blue?), it will have better
contrast against the black background.
Do you have a high contrast theme for Vim?
I am also new to vim.... :-)
Thanks again.

Is this an improvement?:

The "your keymap file" blue font on black background is very hard to read.
Here is black background color comparison from:
http://www.w3schools.com/tags/ref_color_tryit.asp?color=black
After picking the font color, say "LightCoral", from above link, you
append "color=LightCoral" to the URL like so in a new browser page:
http://www.w3schools.com/tags/ref_color_tryit.asp?color=LightCoral
Comparing the browser tab views would give you a good idea of contrast for the chosen color
for white on chosen color and chosen color on black.
Looks like LightCoral works well with black background and white font on LightCoral background,
Also RoyalBlue may be good too:
http://www.w3schools.com/tags/ref_color_tryit.asp?color=RoyalBlue
Also it looks like RoyalBlue works well with black background and white font on RoyalBlue background,
I also think Salmon color has good contrast for white on slamon and salmon on black:
http://www.w3schools.com/tags/ref_color_tryit.asp?color=Salmon
If you tell me which source file line to make changes on which, I can also test it for you.
Thanks.

Here are some other suggestions for the "low contrast" areas:

Color of line numbers ( vertical gutter ) and comments are gray-white ob black background.
For comments, consider changing it to say yellow on black or some other higher contrast combination.

I understand the line number  gets brighter when a cursor is on the line.
Consider changing it to a better contrast combination and still allow for
a highlighted line number when cursor is in focus.
Thank you.

I fixed the link colors and made the settings page background darker:

I also made comments stand out more in contrast-syntax:

I could not find how to lighten the text in the gutter but I will look.

Thanks.
I suggest you should consider Salmon for the link.
The royal blue did not change / improve the contrast as much as I hoped,
as least for my eyes.
See screenshot below.

Thank you again for your generous time.

authors: FriendOfFatBeagle michardy

repo: michardy/contrast-ui


text:
Aquatone Report HTML page takes too long to load big outputs.

Using a aquatone with nmap is great and everything is working very good. The only issue that i'm getting is when loading big outputs, for example when getting about 60.000 Successful Screenshots the HTML Aquatone Report page can take long time to load all the content on one page.
it be possible to output the HTML result with multiples pages ? For example page 1, page 2 page 3 etc etc.
Maybe only the option to output in multiple pages.
Amazing app.
Thanks.

I understand that adding multiple pages on the output is very difficult because the current output join domains by similarity. Maybe there's a way around this. It be nice to have this like the ex aquatone. but I understand that this may not be possible.

repo: michenriksen/aquatone


text:
Websocket Server Only Allows Single checkQueueUpdatesInterval

GMJ requires that servers allow multiple persistent sessions at the same time. Currently, the websocket server only supports checking for queue updates in the latest session connection made. We need it to check for queue updates on every session. I would like to also remove its dependency on the socket connection, by extracting the code for this interval into its own module, seperating it from the io.on("connection", ...) module.

Resolved with this commit


text:
Error build the precompiled library

Steps to reproduce the issue
git clone -b humble https://github.com/micro-ROS/micro_ros_arduino
cd micro_ros_arduino
docker pull microros/micro_ros_static_library_builder:humble
docker run -it --rm -v $(pwd):/project --env MICROROS_LIBRARY_FOLDER=extras microros/micro_ros_static_library_builder:humble

Finished <<< stereo_msgs [2.99s]
--- stderr: visualization_msgs
CMake Warning at /uros_ws/firmware/mcu_ws/install/share/rcutils/cmake/ament_cmake_export_libraries-extras.cmake:116 (message):
Package 'rcutils' exports library 'dl' which couldn't be found
Call Stack (most recent call first):
/uros_ws/firmware/mcu_ws/install/share/rcutils/cmake/rcutilsConfig.cmake:41 (include)
/uros_ws/firmware/mcu_ws/install/share/rosidl_runtime_c/cmake/ament_cmake_export_dependencies-extras.cmake:21 (find_package)
/uros_ws/firmware/mcu_ws/install/share/rosidl_runtime_c/cmake/rosidl_runtime_cConfig.cmake:41 (include)
/uros_ws/firmware/mcu_ws/install/share/builtin_interfaces/cmake/ament_cmake_export_dependencies-extras.cmake:21 (find_package)
/uros_ws/firmware/mcu_ws/install/share/builtin_interfaces/cmake/builtin_interfacesConfig.cmake:41 (include)
CMakeLists.txt:14 (find_package)

Finished <<< visualization_msgs [7.79s]
Starting >>> common_interfaces
Finished <<< common_interfaces [0.69s]
Summary: 69 packages finished [1min 35s]
50 packages had stderr output: action_msgs actionlib_msgs builtin_interfaces composition_interfaces control_msgs diagnostic_msgs example_interfaces geometry_msgs libyaml_vendor lifecycle_msgs micro_ros_msgs micro_ros_utilities microxrcedds_client nav_msgs rcl rcl_action rcl_interfaces rcl_lifecycle rcl_logging_interface rcl_logging_noop rclc rclc_lifecycle rclc_parameter rcutils rmw rmw_implementation rmw_microxrcedds ros2trace rosgraph_msgs rosidl_cli rosidl_runtime_c rosidl_typesupport_c rosidl_typesupport_microxrcedds_c sensor_msgs sensor_msgs_py shape_msgs statistics_msgs std_msgs std_srvs stereo_msgs test_msgs test_tracetools_launch tf2_msgs tracetools_launch tracetools_read tracetools_test tracetools_trace trajectory_msgs unique_identifier_msgs visualization_msgs
Crosscompiled environment: cleaning path

I thank you in advance for your answer.

There is no error on your log, just a warning that is not relevant for this build system.

Additional info:
The build process starts a loop and the library is never built.
During the build process I get the following error:

@LuisYordano, I am also having the rsync error problem. Please keep the thread updated with any solutions you may find.

The rsync errors are just verbose and it actually does not affect the build process.
You can just ignore those errors.


text:
[BRD-151] - User menu UI

Refactor of user menu UI as described by designers

Pull Request Test Coverage Report for Build 728918658

11 of 11 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 726043579:
0.0%

Covered Lines:
184

Relevant Lines:
184

💛  - Coveralls

Pull Request Test Coverage Report for Build 729025122
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

11 of 11 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 726043579:
0.0%

Covered Lines:
184

Relevant Lines:
184

💛  - Coveralls

authors: coveralls ilteoood

repo: micro-lc/micro-lc


text:
Replace hitchip1006

Several data sets from CuratedMetagenomicData can suitably replace hitchip1006.

The checks are failing. Can you fix that? Merging is possible after the tests go through.

authors: RiboRings antagomir

repo: microbiome/benchmarking


text:
Model Stan's Bioscales elements/nutrients terms here

Were originally requested in MIxS: https://github.com/GenomicsStandardsConsortium/mixs/issues?q=is%3Aissue+author%3AStantonMartin
Some have been implemented in MIxS already. Questions remain for the others.
Copy of GA lab Data 11_17_2020 (1).xlsx illustrates some additional new terms required for Bioscales
CC @StantonMartin

carbon terms for Bioscales (plus errors) #7

@turbomam reported this morning that this issue is done. Closing issue.

authors: ssarrafan turbomam

repo: microbiomedata/nmdc-schema


text:
Upgrade to MICROPYVERSION = 1.12

Could you please take into account the new version of Micropython and build either a ESP8266 and ESP32 version of Microhomie?

I will, MicroPython has changed on how to build firmware with included freezed modules. I have to look at this and make some tests.

Updated to MicroPython 1.12 in development branch. I'll release a Microhomie v2.3 alpha firmware release for the esp8622 the next days.

v2.3.0 alpha1 is released. You may want to use the non ota version for now. OTA is not very well documented in the MicroPython docs. I'll add a section how to use OTA to the Microhomie documentation later.

Thanks, going to test beta.1

Did you enable littlefs? Also, I have not tried building 1.12, do you have any tips or suggestions on building. I know about the frozen modules, but if you ran into any other issues I would appreciate the feedback.
Thanks for all your work on microhomie!!!

I have not looked into littlefs and compiled MicroPython with a clean repo. I just disabled btree support via the environment variable.
If littlefs is enabled in the repository by default it should be enabled too in the beta firmware.
The only issue compiling MicroPython was that the openesp tool, does not detect bash in version 5.

authors: dmartinpro palmtreefrb rroemhild

repo: microhomie/microhomie


text:
Doesnt seem to be responsive as a twitter card

Prerequisites

[ x ] I'm using the last version.
[ x ] My node version is the same as declared as package.json.

Subject of the issue
I love this, but it doesnt seem to be responsive or follow the sizing for a twitter card:

Steps to reproduce
check the image above.
Also, this is how im using it:
  const headline =
    "This is my blog where I write about stuff im working on and things i find interesting";
  const caption = "My articles, tutorials and thoughts. Under one roof";
  const domain = "richardhaines.dev";
  const backgroundImage = "/diamonds.png";

  const api = "https://i.microlink.io/";
  const cardUrl = `https://cards.microlink.io/?preset=richhaines&headline=${headline}&caption=${caption}&domain=${domain}&backgroundImage=${backgroundImage}`;
  const image = `${api}${encodeURIComponent(cardUrl)}`;

Expected behaviour
Id expect it to be responsive and fit the twitter cards
Actual behaviour
It seems to be zoomed in

It depends on the template constraints and that template was created by you 😛
The image dimensions are constant over time. You can verify how the screenshot will be taken, you can see it removing /editor from the URL:
https://cards.microlink.io/?preset=richhaines
If your use case is to use them as Twitter Card images, then you need to take into account the Twitter Card image dimension restrictions in order to well-design the card that fits on the Twitter preview.
For an example, see how pedro preview fits perfect

It depends on the template constraints and that template was created by you 😛
The image dimensions are constant over time. You can verify how the screenshot will be taken, you can see it removing /editor from the URL:
https://cards.microlink.io/?preset=richhaines
If your use case is to use them as Twitter Card images, then you need to take into account the Twitter Card image dimension restrictions in order to well-design the card that fits on the Twitter preview.
For an example, see how pedro preview fits perfect

Thanks for the quick response! Right ok, so i have to set the height and width of the card so that it matches what the twitter crad will be?

Im closing this as it appears that its an issue with my code as opposed to the service

authors: Kikobeats molebox

repo: microlinkhq/cards


text:
Baggage items are not being put in scope when extracted

Hi,
We send baggage items as headers on Kafka records. When we extract the span in the consumer and put the span in scope with tracer.withSpan(extractedSpan), the bagage items are not available.
I created a test project with a failing test. In this test:

Add a key to the baggage in the current context
Use a propagator to inject the context in a map
Validate the bagage item is in the map
Use a propagator to extract the context
Set the extracted span in scope
Validate that the bagage item is present

I'm using the OpenTelemetry bridge and you find the test project here:
https://github.com/bdeneuter/baggage-test

We were discussing this issue on Slack. I'm posting the conversation here for keeping track:
https://micrometer-metrics.slack.com/archives/C662HUJC9/p1669849341671159
Marcin Grzejszczak
Check this https://micrometer.io/docs/tracing#_micrometer_tracing_baggage_api
Marcin Grzejszczak
For samples try https://github.com/micrometer-metrics/micrometer-samples/tree/main/baggage-producer and https://github.com/micrometer-metrics/micrometer-samples/tree/main/baggage-consumer
Bart De Neuter
Thx @Marcin Grzejszczak
The baggage API is a confusing API. The baggage is present on the span that has been extracted with a propagator. But the baggage doesn't seem to be set and passed to the child spans?
This succeeds:
try (var context = tracer.withSpan(extractedSpan)) {
    assertThat(tracer.getBaggage(ROOT_SERVICE).get(extractedSpan.context())).isEqualTo("testApp");
}

So if I give the context from the extracted span, the baggage is returned.
But when I don't pass the extracted span context explicitly, the baggage is not returned. Which is weird because the extractedSpan is in scope with tracer.withSpan(extractedSpan) . Why does this fail?
try (var context = tracer.withSpan(extractedSpan)) {
   try(var baggage = tracer.getBaggage(ROOT_SERVICE).makeCurrent()) {
       assertThat(tracer.getBaggage(ROOT_SERVICE).get()).isEqualTo("testApp");
   }
}

It is also not very clear from the documentation when you need to use baggage.makeCurrent() and when not. Is it only for getting a value? Is it needed when you pass the context? It seems it is not needed to set a value? (edited)
Marcin Grzejszczak
The baggage is passed to child spans. Which versions are you using? What tracer are you using?
Bart De Neuter
I'm using version 1.0.0 and use the OpenTelemetry bridge
Bart De Neuter
If I put the extracted span in scope and use the propagator to inject the current context in a map, the baggage from the extracted span is not present
Bart De Neuter
So this fails:
try (var context = tracer.withSpan(extractedSpan)) {
    var map = new HashMap<String, String>();
    propagator.inject(tracer.currentTraceContext().context(), map, (carrier, key, value) -> carrier.put(key, value));
    assertThat(map).containsKey(ROOT_SERVICE);
}

Marcin Grzejszczak
let me check that , I mean we have tests that prove that the baggage gets propagated so I have no idea what's going on here. I'll take the baggage example from micrometer-samples and play around with it
Marcin Grzejszczak
where does the extractedSpan take the baggage from? Did you set it manually? Is it coming from a controller?
Marcin Grzejszczak
Have you provided the baggage as a property like this? (edited)
Marcin Grzejszczak
management:
  tracing:
    sampling:
      probability: 1.0
    baggage:
      remote-fields:
        - mybaggage
        - myremotefield

Bart De Neuter
No, I have set the baggage in code with:
tracer.createBaggage(ROOT_STARTED_AT, String.valueOf(Instant.now().toEpochMilli()));
tracer.createBaggage(ROOT_RESOURCE, record.topic());
tracer.createBaggage(ROOT_SERVICE, serviceName);

As I understand correctly, the property  remote-fields will set the key as is on the header. The rest of the baggage items are still put on the Kafka record.
I created a test project with a failing test. Should I create a bug ticket so it can be investigated?
Marcin Grzejszczak
you must define the baggage as a remote-field entry as presented above
Marcin Grzejszczak
sure create the project, create a ticket and I will look into that
:+1:
Bart De Neuter
We tried it but didn't change anything
Marcin Grzejszczak
I think you should try putting those baggage entries in scope
Marcin Grzejszczak
cause especially with OTel stuff needs to be put in scope
Bart De Neuter
Thx for the help. I have created the bug issue here: https://github.com/micrometer-metrics/tracing/issues/122
:+1:
Marcin Grzejszczak
Maybe we need to improve the API a bit  but I know what the problem is. Also it might be that you caught a bug - I'll look into that.
@Test
	void injectAndExtractKeepsTheBaggage() {
		// GIVEN
		var carrier = new HashMap<String, String>();

		var span = tracer.nextSpan().start();
		try(var spanInScope = tracer.withSpan(span)) {
			tracer.createBaggage(KEY_1, VALUE_1);

			// WHEN
			propagator.inject(tracer.currentTraceContext().context(), carrier, HashMap::put);

			// THEN
			assertThat(carrier.get(KEY_1)).isEqualTo(VALUE_1);
		}

		// WHEN
		var extractedSpan = propagator.extract(carrier, HashMap::get).start();

		// THEN
		try(var spanInScope = tracer.withSpan(extractedSpan)) {
			assertThat(tracer.getBaggage(KEY_1).get(extractedSpan.context())).isEqualTo(VALUE_1);
			try(var baggageInScope = tracer.getBaggage(KEY_1).makeCurrent()) {
				assertThat(baggageInScope.get()).isEqualTo(VALUE_1);
			}
		}
	}

so this test works until the last assertion. Instead of calling tracer.getBaggage(KEY_1).makeCurrent() you should call tracer.getBaggage(extractedSpan.context(), KEY_1) . If possible always pass the trace context explicitly (in this case the one from extractedSpan).
Bart De Neuter
I think it should work on the tracer without the need to pass the context of the extracted span. In our production code we would consume from Kafka, set the span in scope and call other services. These other services can cause events that will be published again on Kafka. But in those services, we don't have the reference anymore to the extracted span. So a propagator is being used to inject the baggage from the current scope into the carrier. With the current implementation, we loose the baggage
Bart De Neuter
I added a test to show what I mean:
@Test
	void fakeKafkaExample() {
		// GIVEN
		var span = tracer.nextSpan().start();
		try(var spanInScope = tracer.withSpan(span)) {
			tracer.createBaggage(KEY_1, VALUE_1);
			kafka.publish("event 1");
		}

		// WHEN
		kafka.consume(message -> {

			// THEN
			assertThat(message).isEqualTo("event 1");
			try(var baggage = tracer.getBaggage(KEY_1).makeCurrent()) {
				assertThat(baggage.get()).isEqualTo(VALUE_1);
			}

			// WHEN
			kafka.publish("event 2");

			// THEN
			kafka.consume(message2 -> {
				assertThat(message2).isEqualTo("event 2");
				try(var baggage = tracer.getBaggage(KEY_1).makeCurrent()) {
					assertThat(baggage.get()).isEqualTo(VALUE_1);
				}
			});

		});
	}

Marcin Grzejszczak
Sure, so I think the bug is about us not reading the current context properly. So if you take the current context from the bean and manually pass it to getBaggage then it should work fine  (edited)
Bart De Neuter
I added a test to validate if baggage is passed to child spans and this test passes. So it seems it is a bug when the baggage is extracted and not placed correctly on the extracted span. The baggage is there because you can get the baggage when using the context of the extracted span directly. But it is not set correctly because when the span is put in scope, the baggage is not on the tracer.

repo: micrometer-metrics/tracing


text:
Could you please provide an update on when the next release is planned?

Issue description
I'm waiting some fixes

see: https://github.com/micronaut-projects/micronaut-core/discussions/10527
https://micronaut.io/micronaut-roadmap/

We can probably release a minor version already, but you will need to reference it directly.

@dstepanov it'll be great thank you

authors: dstepanov ramilS sdelamo


text:
Do not declare parameters by reference if their values are never changed.

Title
Do not declare parameters by reference if their values are never changed.
Description
Unintentional value changes might propagate. In addition, it might lead other developers to think that value changes are intended.
Reason for the rule
Defining a parameter as passed by reference and not actually changing it may lead to an unexpected variable value if the variable is later changed in the method. In addition, other developers may think that the value is changed in the method, even though that is not true.
Bad code sample 1 - developer's error - forgot to remove var property of MyList in Method1, after redesigning Method2
procedure Method1(var MyB: boolean)
begin
    Method2(MyB);
end;

procedure Method2(MyC: boolean)
var
    someVar: boolean;
begin
    someVar := MyC;
end;

Good code sample 1
procedure Method1(MyB: boolean)
begin
    Method2(MyB);
end;

procedure Method2(MyC: boolean)
var
    someVar: boolean;
begin
    someVar := MyC;
end;

Bad code sample 2 - developer's error
procedure Method1(var InvtValue: array[5] of Integer)
var
    MyInt: Integer;
begin
    MyInt := InvtValue[0];
end;

Good code sample 2
procedure Method1(InvtValue: array[5] of Integer)
var
    MyInt: Integer;
begin
    MyInt := InvtValue[0];
end;

Good and bad practices for fixing the rule
Remove not needed var property from the parameter declaration.
Remarks
Variables of the following types are analyzed: Integer, BigInteger, Boolean, Byte, Char, Code, Date, DateTime, Decimal, Time, and Text. Also lists and arrays of above types are analyzed - these types should be changed with appropriate methods if they are passed by reference.
Event subscribers, Integration Events and ConfirmHander are not analyzed also.

Isn't this already released? ...albeit with subsequent bugs opened about false positives, e.g.

https://github.com/microsoft/AL/issues/6184
https://github.com/microsoft/AL/issues/6179
https://github.com/microsoft/AL/issues/6143
https://github.com/microsoft/AL/issues/6211

Isn't this already released? ...albeit with subsequent bugs opened about false positives, e.g.

https://github.com/microsoft/AL/issues/6184
https://github.com/microsoft/AL/issues/6179
https://github.com/microsoft/AL/issues/6143
https://github.com/microsoft/AL/issues/6211

@PaulIvan could you verify?

@PaulIvan could you verify?

authors: MarcHansenMicrosoft atoader dtkb

repo: microsoft/AL


text:
Add FactBox Area with PageExtension

Hi,
We're trying to add a FactBox to "Sales Lines" Page (Id 516 for old NAV developers who always talk about objects by their numbers 😄).
Unfortunately, the FactBox area doesn't exists on this page, and we can't add it.
Or maybe I missed something...
Can we have the possibility to add an area (FactBox or other) to an existing Page using a PageExtension ?
Have a nice day !

So, e.g., addfirst(factboxes) does not work, and tells you that no such area exists?

Version is 16.0.11240.12076-be. I missed to include this info.
Indeed, it works. Even if this Area is not present in the page.
I don't know why I wasn't able to do this before opening this issue (mistyping, lack of concentration).
Thanks for your help.

authors: dtkb legrlion


text:
[Access Request] codeunit 6501 "Item Tracking Data Collection" RetreiveTempGlobalEntrySummary

Is it possible to get a procedure in codeunit 6501 "Item Tracking Data Collection" to retrieve TempGlobalEntrySummary after calling procedure RetrieveLookupData? Or a new procedure also called RetrieveLookupData with an extra parameter?
Example of usage:
ItemTrackingDataCollection.RetrieveLookupData(TrackingSpecification, true);
ItemTrackingDataCollection.GetTempGlobalEntrySummary(TempGlobalEntrySummary);

New procedure ex 1:
procedure GetTempGlobalEntrySummary(var TempGlobalEntrySummary2: Record "Entry Summary" temporary)
begin
    TempGlobalEntrySummary2.Copy(TempGlobalEntrySummary, true);
end;

New procedure ex 2:
procedure RetrieveLookupData(var TempGlobalEntrySummary2: Record "Entry Summary" temporary; var TempTrackingSpecification: Record "Tracking Specification" temporary; FullDataSet: Boolean)
begin
    RetrieveLookupData(TempTrackingSpecification, FullDataSet);
    TempGlobalEntrySummary2.Copy(TempGlobalEntrySummary, true);
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

authors: JesperSchulz auzhhv

repo: microsoft/ALAppExtensions


text:
[Event Request] table 295 "Reminder Header" - OnAfterSetReminderTextFilter

We would like to add a new event OnAfterSetReminderTextFilter in the following 2 procedures:
Procedure 1:
local procedure InsertBeginTexts(ReminderHeader: Record "Reminder Header")
var
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeInsertBeginTexts(ReminderHeader, IsHandled);
    if IsHandled then
        exit;

    ReminderLevel.SetRange("Reminder Terms Code", ReminderHeader."Reminder Terms Code");
    ReminderLevel.SetRange("No.", 1, ReminderHeader."Reminder Level");
    if ReminderLevel.FindLast then begin
        ReminderText.Reset();
        ReminderText.SetRange("Reminder Terms Code", ReminderHeader."Reminder Terms Code");
        ReminderText.SetRange("Reminder Level", ReminderLevel."No.");
        ReminderText.SetRange(Position, ReminderText.Position::Beginning);
        OnAfterSetReminderTextFilter(ReminderText, ReminderHeader); //A108F0065.Event

        ReminderLine.Reset();
        ReminderLine.SetRange("Reminder No.", ReminderHeader."No.");
        ReminderLine."Reminder No." := ReminderHeader."No.";
        if ReminderLine.Find('-') then begin
            LineSpacing := ReminderLine."Line No." div (ReminderText.Count + 2);
            if LineSpacing = 0 then
                Error(Text004);
        end else
            LineSpacing := 10000;
        NextLineNo := 0;
        InsertTextLines(ReminderHeader);
    end;
end;

Procedure 2:
local procedure InsertEndTexts(ReminderHeader: Record "Reminder Header")
var
    ReminderLine2: Record "Reminder Line";
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeInsertEndTexts(ReminderHeader, IsHandled);
    if IsHandled then
        exit;

    ReminderLevel.SetRange("Reminder Terms Code", ReminderHeader."Reminder Terms Code");
    ReminderLevel.SetRange("No.", 1, ReminderHeader."Reminder Level");
    if ReminderLevel.FindLast then begin
        ReminderText.SetRange(
          "Reminder Terms Code", ReminderHeader."Reminder Terms Code");
        ReminderText.SetRange("Reminder Level", ReminderLevel."No.");
        ReminderText.SetRange(Position, ReminderText.Position::Ending);
        OnAfterSetReminderTextFilter(ReminderText, ReminderHeader); //A108F0065.Event
        ReminderLine.Reset();
        ReminderLine.SetRange("Reminder No.", ReminderHeader."No.");
        ReminderLine.SetFilter(
          "Line Type", '%1|%2|%3',
          ReminderLine."Line Type"::"Reminder Line",
          ReminderLine."Line Type"::"Additional Fee",
          ReminderLine."Line Type"::Rounding);
        ReminderLine.SetRange("Line Type"); //A108F0065.n
        if ReminderLine.FindLast then
            NextLineNo := ReminderLine."Line No."
        else
            NextLineNo := 0;
        ReminderLine.SetRange("Line Type");
        ReminderLine2 := ReminderLine;
        ReminderLine2.CopyFilters(ReminderLine);
        ReminderLine2.SetFilter("Line Type", '<>%1', ReminderLine2."Line Type"::"Line Fee");
        ReminderLine2.SetRange("Line Type"); //A108F0065.n
        if ReminderLine2.Next <> 0 then begin
            LineSpacing :=
              (ReminderLine2."Line No." - ReminderLine."Line No.") div
              (ReminderText.Count + 2);
            if LineSpacing = 0 then
                Error(Text004);
        end else
            LineSpacing := 10000;
        InsertTextLines(ReminderHeader);
    end;
end;

Event Signature:
[IntegrationEvent(false, false)]
local procedure OnAfterSetReminderTextFilter(var ReminderText: Record "Reminder Text"; ReminderHeader: Record "Reminder Header")
begin
    //A108F0065.Event
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


text:
[Event Request] codeunit 12101 "Withholding - Contribution" - OnWithholdLineFilter

In codeunit 12101 "Withholding - Contribution" please add the following event:
    procedure WithholdLineFilter(var WithholdCodeLine: Record "Withhold Code Line"; WithholdCode: Code[20]; ValidityDate: Date)
    // New variable
    var
        IsHandled: Boolean;
    // New variable
    begin
        if ValidityDate = 0D then
            ValidityDate := WorkDate;

        WithholdCodeLine.Reset();
        WithholdCodeLine.SetRange("Withhold Code", WithholdCode);
        WithholdCodeLine.SetRange("Starting Date", 0D, ValidityDate);

        // New event
        OnWithholdLineFilter(WithholdCodeLine, WithholdCode, ValidityDate, IsHandled);
        if IsHandled then
            exit;
        // New event
        if WithholdCodeLine.FindLast() then
            WithholdCodeLine.SetRange("Starting Date", WithholdCodeLine."Starting Date")
        else
            Error(WithholdCodeLinesNotSpecifiedErr, WithholdCode);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnWithholdLineFilter(var WithholdCodeLine: Record "Withhold Code Line"; WithholdCode: Code[20]; ValidityDate: Date; var IsHandled: Boolean)
    begin
    end;

Thank you

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

authors: JesperSchulz Vittorio-Panigoni


text:
[Event Request] Codeunit 97 "Blanket Purch. Order to Order" - trigger OnRun - OnBeforeUpdatePurchOrderLineLineDiscountPercent

Please add an Event OnBeforeUpdatePurchOrderLineLineDiscountPercent in trigger OnRun in Codeunit 97 "Blanket Purch. Order to Order"
[Event Request] Codeunit 97 - Blanket Purch. Order to Order - trigger OnRun
    var
        Vend: Record Vendor;
        PurchCommentLine: Record "Purch. Comment Line";
        PrepmtMgt: Codeunit "Prepayment Mgt.";
        RecordLinkManagement: Codeunit "Record Link Management";
        PurchCalcDiscByType: Codeunit "Purch - Calc Disc. By Type";
        ShouldRedistributeInvoiceAmount: Boolean;
        IsHandled: Boolean;
    begin
        OnBeforeRun(Rec);

        TestField("Document Type", "Document Type"::"Blanket Order");
        ShouldRedistributeInvoiceAmount := PurchCalcDiscByType.ShouldRedistributeInvoiceDiscountAmount(Rec);
...
...
                    if (PurchOrderLine."No." <> '') and (PurchOrderLine.Type <> PurchOrderLine.Type::" ") then begin
                        PurchOrderLine.Amount := 0;
                        PurchOrderLine."Amount Including VAT" := 0;
                        PurchOrderLineValidateQuantity(PurchOrderLine, PurchBlanketOrderLine);
                        if PurchBlanketOrderLine."Expected Receipt Date" <> 0D then
                            PurchOrderLine.Validate("Expected Receipt Date", PurchBlanketOrderLine."Expected Receipt Date")
                        else
                            PurchOrderLine.Validate("Order Date", PurchOrderHeader."Order Date");
                        UpdatePurchOrderLineDirectUnitCost();

                        IsHandled := false;
                        OnBeforeUpdatePurchOrderLineLineDiscountPercent(PurchOrderLine, PurchBlanketOrderLine, PurchOrderHeader, IsHandled); <-- NEW EVENT
                        if not IsHandled then

                            PurchOrderLine.Validate("Line Discount %", PurchBlanketOrderLine."Line Discount %");

[IntegrationEvent(false, false)]
local procedure OnBeforeUpdatePurchOrderLineLineDiscountPercent(var PurchOrderLine: Record "Purchase Line"; PurchBlanketOrderLine: Record "Purchase Line"; PurchOrderHeader: Record "Purchase Header"; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 42698.


text:
Guided Experience Item entry still exist after unpublishing our app

We have an assisted setup page, which is works fine, but when we have uninstall and unpublish our app, a guided experience item is still exist.
Should I remove this entry somehow? How can I do this? Is there an event for this somewhere?
It's a bit annoying, because when we have reinstall our app, the assisted setup seems to be completed.

There is no best practice to remove this entry or I'm asking on a wrong place?

As I see, there is no option to subscribe on uninstalling apps. Please vote on this if you also need this option.
https://experience.dynamics.com/ideas/idea/?ideaid=2ee9e0ed-6cc5-e911-b083-0003ff68ac6b

Hi @HegedusZsolt
As you say, there is no OnUninstall trigger where you could place such code.
Your only option for now is to handle potentially existing Guided Experience Items at install, removing the old ones before adding new.
Thanks.

authors: HegedusZsolt sorenfriisalexandersen


text:
[BUG] on Format function (with setting 9)

https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-format-property#standard-xml-and-string-formats
Hoping that I interpret the documentation correctly, the function Format(Value, 9) formats as value to be used in an XML document.
Simplified the functions, where I expected a different result.
trigger OnOpenPage()
var
    MessageLbl: Label 'The formatted value of ''%1'' is ''%2''.';
begin
    Message(StrSubstNo(MessageLbl, WorkDate(), Format(WorkDate(), 9)));
end;

Result
Value of pop-up message

Expected
The formatted value of '21/11/22' is '2022-11-21'.

Outcome
The formatted value of '21/11/22' is ' 21/11/22'.

trigger OnOpenPage()
var
    MessageLbl: Label 'The formatted value of ''%1'' is ''%2''.';
    myDecimal: Decimal;
begin
    myDecimal := 1234.56;
    Message(StrSubstNo(MessageLbl, myDecimal, Format(myDecimal, 9)));
end;

Result
Value of pop-up message

Expected
The formatted value of '1.234,56' is '1234.56'.

Result
The formatted value of '1.234,56' is ' 1.234,56'.

(I've noticed also that in both cases a space is added before the value)
version: BE Business Central 21.1
application: 21.1.48363.48789
platform: 21.0.48357.49378

The way Format() works is a platform/AL issue, to some extent (since it is the content of a message) maybe a client issue, but not a System app issue. Hence, this is definitely the wrong repository.
You could try to move it to https://github.com/Microsoft/AL/issues instead, but no guarantee that it will be accepted there.

@NKarolak, you're probably right!
I'm also convinced that it's the Format() function causing this problem. The message here is to simplify the issue.
It seems I can't transfer this issue. I'll wait the response here, hoping that this issue will be transferred or otherwise I'll create a new issue on the right repository.

We cannot process non extensibility related bugs here, the only way to get it fixed is to report vis technical support. I can transfer this one to AL (platform area) but without any promises it will be considered or fixed.

authors: AlexanderYakunin Arthurvdv NKarolak


text:
[Event Request] - Event signature change OnBeforeHandleNonRecurringLine in Codeunit 23 "Item Jnl.-Post Batch"

Please, consider changing the signature of OnPostOutputOnBeforePostItem event in Codeunit 23 "Item Jnl.-Post Batch" from:
    [IntegrationEvent(false, false)]
    local procedure OnBeforeHandleNonRecurringLine(var ItemJournalLine: Record "Item Journal Line"; var IsHandled: Boolean)     begin
    end;

to:
    [IntegrationEvent(false, false)]
    local procedure OnBeforeHandleNonRecurringLine(var ItemJournalLine: Record "Item Journal Line"**; var OldEntryType: Enum "Item Ledger Entry Type"**; var IsHandled: Boolean)
    begin
    end;

and call it accordingly inside the "HandleNonRecurringLine" procedure:
    local procedure HandleNonRecurringLine(var ItemJnlLine: Record "Item Journal Line"; OldEntryType: Enum "Item Ledger Entry Type")
    var
        ItemJnlLine2: Record "Item Journal Line";
        ItemJnlLine3: Record "Item Journal Line";
        IncrBatchName: Boolean;
        IsHandled: Boolean;
    begin
        IsHandled := false;

        OnBeforeHandleNonRecurringLine(ItemJnlLine, **OldEntryType**, IsHandled);

        if IsHandled then
            exit;

        with ItemJnlLine do begin
            ItemJnlLine2.CopyFilters(ItemJnlLine);
            ItemJnlLine2.SetFilter("Item No.", '<>%1', '');
...
...

Thank you!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52463.

authors: JesperSchulz oldmat


text:
[Event Parameter] codeunit 5611 "Calculate Normal Depreciation" - OnAfterCalcFinalDeprAmount #20442

In #20442, a new parameter to the event was added. But the call of the event was mistaken. NumberOfDays2 was asked to use but NumberOfDays was used instead. Would it be possible to change it?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10853.

authors: JesperSchulz RadoArvay


text:
Master Data Management - Cannot Debug - cannot make it work for some scenarios

🕮 Describe the issue
I am using Master Data extension to sync master data to multiple companies. Its rather difficult to investigate why things dont work because, we cannot debug the extension. Its not downloaded in the main app symbols
We cannot debug as the symbols are not included in the BASE, SYSTEM, APP packages. Thus no self service to fix it .
Can you help me or assist me to someone so I can resolve this issue?
🔧 To Reproduce
Steps to reproduce the behavior:
Download symbols and see if the table 7231 "Master Data Mgt. Coupling" is included in the app files . IT is not.
📣 Expected behavior
I would like to be able to debug it and fix it, perhaps improve logging?

Actually I can download it in a separate package sorry, closing this


text:
[Event Request] Codeunit 760 Trailing Sales Orders Mgt.

The following events are required for this object:
    [IntegrationEvent(false, false)]
    local procedure OnDrillDownBeforeRunPage(SalesHeader: Record "Sales Header"; var IsHandled: Boolean)
    begin
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeGetSalesOrderAmount(TrailingSalesOrderQry: Query "Trailing Sales Order Qry"; TrailingSalesOrdersSetup: Record "Trailing Sales Orders Setup"; Status: Option; FromDate: Date; ToDate: Date; var Result: Decimal; var IsHandled: Boolean)
    begin
    end;

    procedure DrillDown(var BusChartBuf: Record "Business Chart Buffer")
    var
        SalesHeader: Record "Sales Header";
        ToDate: Date;
        Measure: Integer;
        IsHandled: Boolean;
    begin
        Measure := BusChartBuf."Drill-Down Measure Index";
        if (Measure < 0) or (Measure > 3) then
            exit;
        TrailingSalesOrdersSetup.Get(UserId);
        SalesHeader.SetRange("Document Type", SalesHeader."Document Type"::Order);
        if TrailingSalesOrdersSetup."Show Orders" = TrailingSalesOrdersSetup."Show Orders"::"Delayed Orders" then
            SalesHeader.SetFilter("Shipment Date", '<%1', TrailingSalesOrdersSetup.GetStartDate);
        if Evaluate(SalesHeader.Status, BusChartBuf.GetMeasureValueString(Measure), 9) then
            SalesHeader.SetRange(Status, SalesHeader.Status);

        ToDate := BusChartBuf.GetXValueAsDate(BusChartBuf."Drill-Down X Index");
        SalesHeader.SetRange("Document Date", 0D, ToDate);
        //EVENT REQUEST
        IsHandled := false;
        OnDrillDownBeforeRunPage(SalesHeader, IsHandled);
        if not IsHandled then
            PAGE.Run(PAGE::"Sales Order List", SalesHeader);
        //EVENT REQUEST
    end;

    local procedure GetSalesOrderAmount(Status: Option; FromDate: Date; ToDate: Date) Result: Decimal
    var
        CurrExchRate: Record "Currency Exchange Rate";
        TrailingSalesOrderQry: Query "Trailing Sales Order Qry";
        Amount: Decimal;
        TotalAmount: Decimal;
        IsHandled: Boolean;
    begin
        //EVENT REQUEST
        IsHandled := false;
        OnBeforeGetSalesOrderAmount(TrailingSalesOrderQry, TrailingSalesOrdersSetup, Status, FromDate, ToDate, Result, IsHandled);
        if IsHandled then
            exit(Result);
        //EVENT REQUEST

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 12399.

authors: JesperSchulz SimonasDargis


text:
[Event Request] Codeunit 80 - f(x) FinalizePosting

Hi!
We need an additional event before the CreateOutboxSalesInvTrans/CreateOutboxCrMemoTrans in f(x) FinalizePosting in the Sales-Post codeunit. We need the IsHandled pattern to be included.

Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

authors: andersnordmann bc-ghost


text:
[Adaptive Cards - Schema Explorer] - Screen Reader is conveying the incorrect role as 'Link' for 'View' button.

Target Platforms
Other
SDK Version
1.5
Application Name
Adaptive Cards
Problem Description
Test Environment:
OS: Version 22H2 (OS Build 25300.1000)
URL: https://adaptivecards.io/explorer/
Browser: Edge Dev Version 111.0.1661.36 (Official build) dev (64-bit)
Screen Reader: Narrator
Repro Steps:

Turn on the Narrator using 'ctrl+win+enter' key.
Open the above URL in edge dev browser.
Navigate to the 'View' button under 'Publish Adaptive Card schema' and observe the issue.

Actual Result:
Incorrect role as 'Link' is defined for the 'View' button. It is announcing as 'View' Link.
Expected Result:
Correct role should be defined for the 'View' option and screen reader should convey the correct role information as 'View' button.
User Impact:
It will impact the screen reader user if the incorrect role as link is announced by the screen reader as user will not able to understand the actual functionality of the element.
MAS Reference:
https://www.w3.org/WAI/WCAG21/Understanding/name-role-value.html
“Have feedback to share on Bugs? Please help fill Trusted Tester Bug Feedback (office.com) “
Screenshots

https://user-images.githubusercontent.com/95913748/224330457-59cefc40-afe3-41d1-a9d4-9c45902eb794.mp4
Card JSON
NA

Sample Code Language
No response
Sample Code
No response

Hi @anna-dingler,
As this issue is still repro, so could you please let us know is there any future plan to fix this issue?

repo: microsoft/AdaptiveCards


text:
question about "number of registers"

For the reg_count method, I have a doubt
The comments for Step1 are as follows:
     // Step 1. Probe for the most number of registers we can use by activating
     // different numbers of threads to try to use up all the register resources.
     // In case of register spill the kernel latency would surge.
     //
     // The group size was kept 1 to minimize the impact of scheduling.

But the code of Step1 shows that the currently measured nreg_max is the maximum number of registers available to the kernel when the number of threads remains 1
uint32_t nreg = NREG_MIN;
for (; nreg <= NREG_MAX; nreg += NREG_STEP) {
         double time = bench(1, 1, nreg);
         //....
}

In addition, according to the following sections, we can know that nreg_max will not exceed 512. According to the description in the paper, I guess that nreg_max is 181 measured on Adreno 640.
  const uint32_t NREG_MIN = env.cfg_num("NRegMin", 1);
  const uint32_t NREG_MAX = env.cfg_num("NRegMax", 512);
  const uint32_t NREG_STEP = env.cfg_num("NRegStep", 1);

My question is: Is the purpose of step1 to probe the maximum number of available registers on a single thread - nreg_max
If yes, I think the comments of step1 is not quite correct and will lead to misunderstanding.
And I want to know that on Adreno 640, since there is a 384 x 181 shared 32bit-register pool, why the maximum number of available registers for a single thread is 181, is it related to the allocation strategy of the pool?
If not, is the value of nreg_max 384 x 181?

    "RegCount": {
        "Done": true,
        "FullRegConcurWorkgroupCount": 12,
        "HalfRegConcurWorkgroupCount": 24,
        "RegCount": 183,
        "RegType": "Pooled"
    },

According to adreno640/Google_Pixel_4/ArchProbeReport.json, Does this mean "It shows a 24×183 shared 32b-register pool."？

Thanks for your comment. If I understand it correctly you are confused why the pooled registers are not shared among all threads (any physical ALU has access to all register) but, rather, per physical ALU (any physical ALU only has access to its own 181 regsiters).
The number 384 is the number of all physical ALUs on chip, and there are 3 SMs contributing to this number so the registers doesn't share across SM boundary. They are isolated compute units on chip. So for each of the 384/3=128 physical ALUs there are 181 registers available (aka the value RegCount) at most. The illustration in the figure is somehow confusing and I would apologize about that.

Other things you might want to know:

In addition, according to the following sections, we can know that nreg_max will not exceed 512.

It's just a random number so that the tests don't take too much time to run.

According to adreno640/Google_Pixel_4/ArchProbeReport.json, Does this mean "It shows a 24×183 shared 32b-register pool."？

FullRegConcurWorkgroupCount and HalfRegConcurWorkgroupCount are only used to decide whether register allocation has an impact on workgroup scheduling. If yes, the number of concurrent threads decreases as the number of register occupation increases, i.e., the registerd are pooled, shared among concurrent warps. Otherwise, using more or less registers wouldn't change the number of concurrent warps. Please refer to my paper for more detail.

First of all thank you very much for your answer！
At the same time, I would like to thank you all for your contributions in this field, which gave me the opportunity to learn more about mobile GPUs.
This paper and ArchProbe are of great significance. I will continue to study this and hope to have the opportunity to communicate with you in the future.
Have a good day!

Thank you for your appreciation. g day. :)

authors: PENGUINLIONG i-Taozi

repo: microsoft/ArchProbe


text:
Test new app ID

### Progress
- [x] Public Azure
- [x] Azure China
- [ ] Azure US Government

Testing done for public Azure in 1.30.0. Resume the work in 1.31.0.

We have shipped Azure China with the new app ID in 1.33.0. We will ship 1.34.0 with the new app ID for public Azure. Work for Azure US Government will be moved to 1.35.0 and later.

repo: microsoft/AzureStorageExplorer


text:
Review Compatability With HTTPS Azurite Steps

Review our compatability with HTTP Azurite steps: https://github.com/Azure/Azurite/blob/main/README.md#https-setup
Can we give Azurite team easier steps for users via System Proxy? If yes, do we need to implement this: https://www.electronjs.org/docs/latest/api/app#event-certificate-error

Related to: https://github.com/microsoft/AzureStorageExplorer/issues/7471

Old issue. Might revisit if we get user feedback.

authors: MRayermannMSFT craxal


text:
It is better to remove the 'Delimiter' option from the 'Export Messages' dialog

Storage Explorer Version: 1.37.0-dev (103)
Build Number: 20241123.6
Branch: main
Platform/OS: Windows11/Windows11/Linux Ubuntu 22.04/MacOS Sequoia 15.0(Apple M1 Pro)
Architecture: x64/arm64/x64/arm64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Queues.
Create a queue -> Click 'Export'.
Observe the 'Export Messages' dialog.

Expected Experience
It is better to remove the 'Delimiter' option.
Actual Experience
There is a 'Delimiter' option.

Additional Context
We tried exporting the message using different delimiters and the exported content remained the same.

@v-xianya Then it seems like the answer is not to remove to delimiter but to actually use the selected delimiter.

@v-xianya Sorry, let me rephrase. Export currently only outputs one column (the message), so the delimiter is effectively unused.
However, we should not remove it for a few reasons:

Export should be symmetrical to import, where it is used (even if import effectively ignores all but the first column).
At some future date, we may import/export more columns, such a message's expiration. Already handling delimiters now will make that feature easier later.

Got it. thank you for the clarification.😊

authors: craxal v-xianya


text:
Redacting nested props (retry)

Resolves #2836
What is being addressed
Following the move on the base workspace to conditional properties via allOf, the client secret was no longer being redacted as it was assumed to be in the main template properties and the input property was assumed to be a flat list with no nesting. This fixes that and supports conditional properties in the template as well as nested properties.

/test-extended

/test-destroy-env

/test-destroy-env

/test-extended

/test-destroy-env

repo: microsoft/AzureTRE


text:
Use of undeclared identifiers [posix_memalign, free, memset] in bitnet-lut-kernels.h Ubuntu ARM

System Info
(bitnet-cpp) ubuntu@REDACTED:~/BitNet$ uname -a
Linux ip-REDACTED REDACTED-aws #17-Ubuntu SMP Wed Sep  4 11:15:30 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux
(bitnet-cpp) ubuntu@redacted:~/BitNet$ cmake --version
cmake version 3.28.3

CMake suite maintained and supported by Kitware (kitware.com/cmake).
(bitnet-cpp) ubuntu@redacted:~/BitNet$ which clang
/usr/bin/clang
(bitnet-cpp) ubuntu@redacted:~/BitNet$ clang --version
Ubuntu clang version 18.1.8 (++20240731025043+3b5b5c1ec4a3-1~exp1~20240731145144.92)
Target: aarch64-unknown-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
(bitnet-cpp) ubuntu@redacted:~/BitNet$ python --version
Python 3.9.20
(bitnet-cpp) ubuntu@redacted:~/BitNet$ conda --version
conda 24.9.0

AWS System Info
This is an ARM / Graviton instance (r7g.4xlarge)
Command
python setup_env.py --hf-repo HF1BitLLM/Llama3-8B-1.58-100B-tokens -q i2_s

## Errors
python setup_env.py --hf-repo HF1BitLLM/Llama3-8B-1.58-100B-tokens -q i2_s
INFO:root:Compiling the code using CMake.
ERROR:root:Error occurred while running command: Command '['cmake', '--build', 'build', '--config', 'Release']' returned non-zero exit status 2., check details in logs/compile.log

Task that failed
[  5%] Building CXX object 3rdparty/llama.cpp/ggml/src/CMakeFiles/ggml.dir/__/__/__/__/src/ggml-bitnet-lut.cpp.o

Grepping logs/compile.log
cat logs/compile.log | grep error

/home/ubuntu/BitNet/3rdparty/llama.cpp/ggml/src/../../../../include/bitnet-lut-kernels.h:12:5: error: use of undeclared identifier 'posix_memalign'
/home/ubuntu/BitNet/3rdparty/llama.cpp/ggml/src/../../../../include/bitnet-lut-kernels.h:20:5: error: use of undeclared identifier 'free'
/home/ubuntu/BitNet/3rdparty/llama.cpp/ggml/src/../../../../include/bitnet-lut-kernels.h:289:5: error: use of undeclared identifier 'memset'
/home/ubuntu/BitNet/3rdparty/llama.cpp/ggml/src/../../../../include/bitnet-lut-kernels.h:423:5: error: use of undeclared identifier 'memset'
/home/ubuntu/BitNet/3rdparty/llama.cpp/ggml/src/../../../../include/bitnet-lut-kernels.h:541:5: error: use of undeclared identifier 'memset'
/home/ubuntu/BitNet/3rdparty/llama.cpp/ggml/src/../../../../include/bitnet-lut-kernels.h:675:5: error: use of undeclared identifier 'memset'

gmake[2]: *** [3rdparty/llama.cpp/ggml/src/CMakeFiles/ggml.dir/build.make:146: 3rdparty/llama.cpp/ggml/src/CMakeFiles/ggml.dir/__/__/__/__/src/ggml-bitnet-lut.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:764: 3rdparty/llama.cpp/ggml/src/CMakeFiles/ggml.dir/all] Error 2
gmake: *** [Makefile:136: all] Error 2

Full logs attached
compile.log
## Ask
I am not sure why these system libraries are missing. I have used the suggested installer in the README.md for clang. Please let me know if you bumped into similar errors.

realized this is a duplicate of https://github.com/microsoft/BitNet/issues/74

repo: microsoft/BitNet


text:
[10/30 Hack] 'pwsh' is not recognized as an internal or external command

When creating the Azure Resources from the Settings > Publish page, the command that I need to run is "cd C:/code/BotFramework-Composer/SampleBots/WeatherBot2
pwsh ./Scripts/create.ps1 -name WeatherBot2 -environment Production -location WestUS2 -appPassword 'A12345678912345*'".
When I run it I get this error

Put some link to install the right powershell in the dialog as a prerequisite.

Closing as by design

authors: Andrea-Orimoto cwhitten

repo: microsoft/BotFramework-Composer


text:
Create QnA Maker knowledge base by importing URL issue

Describe the bug
Was unable to create a QnA Maker knowledge base by importing URL.
To Reproduce
Steps to reproduce the behavior:

Create QnA KB from template
Select QnA from Composer menu
Select Create new KB
Enter KB name and URL
Select View KB and URL was not loaded.
Enter QnA KB authoring key and Start the bot. The KB can be published to QnA portal with success, but the content showed completely different as the imported URL should.
Test in Emulator with example question cannot get the expected answer.

Screenshots

Additional texts

Testing URL: https://docs.microsoft.com/en-us/azure/cognitive-services/qnamaker/troubleshooting
It might be an issue from the QnA side, which needs investigation.

Hi @zxyanliu, extracting QnA from url requires good HTML document format, if url source is web page.
I noticed the URL https://docs.microsoft.com/en-us/azure/cognitive-services/qnamaker/troubleshooting
has some embed tab, even show/hide by JS code which is not be able to extract in bf-lu parser.
This whole page is extracted to one QnA pair:

if you ask "Troubleshooting for QnA Maker", it would answer with:

that is expected.

Hi @zhixzhan Thanks for your response.
FYI. This is the URL we provided in the documentation and it worked for 1.2.1 and previous versions.
https://docs.microsoft.com/en-us/azure/cognitive-services/qnamaker/troubleshooting
If this URL was not supported, can you please provide specifications of what type of URL are supported? Can you elaborate a bit more on "it must a static HTML page with good semantic format. e.g."?

Hi @zhixzhan
Yes, it is the issue with the testing URL. I tried this troubleshooting URL in QnA portal separately and failed to load the questions properly. I will use another URL such as https://download.microsoft.com/download/2/9/B/29B20383-302C-4517-A006-B0186F04BE28/surface-pro-4-user-guide-EN.pdf in the docs. Closing this issue for now because it is not a Composer issue after initial investigation.

authors: zhixzhan zxyanliu


text:
feat: Add preparatory work before connecting remote skill

Description
Add preparatory work before connecting remote skill

Task Item
closes #7205
Screenshots

@alanlong9278 I pulled down your branch, and I'm trying to run through the scenario on my local build. On the 'Create a publishing profile' step, I am not able to select a publishing profile - there are no available options. Can you please take a look.


text:
[MAS4.1.2] Name of Audio clip section is not properly defined.

Actual Result:
Name of Audio Clip section is not properly defined. So, Screen reader is reading 'Audio' twice for Audio clip present on the page.
Expected Result:
Name of Audio Clip section should be properly defined. So, Screen reader should read 'Audio' once only with proper description for Audio clip present on the page.
Repro Steps:

Open URL https://microsoft.github.io/BotFramework-WebChat/full-bundle/ in Edge browser.
Notice 'Upload File' and 'Send' button icons present in bottom of screen.
Type 'accessibility' in the input control at the bottom of the screen and hit enter (or click on send button).
Page will upload.
Open Narrator (Ctrl+Win. key+Enter)
Navigate to audio clip by using Tab key.
Observe that narrator reads audio twice when focus lands on the audio clip present on the page.

User Impact:
Screen reader users may get confuse if same content is being read more than once.
MAS Reference:
MAS4.1.2 - Name,Role,Value
Test Environment:
OS: Windows 10
OS Build: 17134.345
OS Version: 1803
Browser: Edge
Tool: Narrator
Attachment:
Usable_reading same value 2 times.zip

Closing this bug as part of internal Bugs review.

#HCL;#Accessibility;#HCL_BotFramework_WebChat;A11yMAS;WCAG4.1.2

repo: microsoft/BotFramework-WebChat


text:
Image Customizer: Add logic to create .ssh dir if not exists

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary
in Trident, the .ssh doesnt pre-exist, so add logic to create ssh dir if not exists, also fix a bug in if condition
Change Log
in Trident, the .ssh doesnt preexist, so add logic to create ssh dir if not exists, also fix a bug in if condition
Does this affect the toolchain?
NO
Test Methodology
test emu invocation in trident in local dev, and in the log, the .ssh was created and key was copied into .ssh/authorized_keys, and i can ssh into the vm with the private key on my local machine

an you provide more details about the issue you're trying to fix? This looks like a potential misconfiguration of a package expecting that directory to be present.

as the image customizer supports adding users and setting up sshkey access for the user, there could be a case that the .ssh dir does not exist in the OS, in our case, we have Trident project which invoke MIC binary to do in-OS customization, and .ssh dir  does not exist in the OS beforehand, so we create it before copying over the sshkeys. in this case, i dont think there is misconfiguration of a package.

repo: microsoft/CBL-Mariner


text:
Collect the virtual directory information in HC

Reason:
There are some additional checks that we can add to verify that we are configured correctly for the virtual directories. For now, let's just collect them all and store them in a property.
This will add some time to the data collection process because we aren't using the -ADPropertiesOnly parameter. We could add this switch to the collection if we need to. However, we will lose the ability to rely on some properties of the output.
Validation:
Lab tested and Pester tested

Need to add in Pester testing for this as well.

repo: microsoft/CSS-Exchange


text:
Questions regarding pre-training

Thank you for the open-source code and detailed documentation of the experiments. I had a few questions about pre-training, which I can't seem to find in the paper or appendix. Could you please help?

How long did pre-training take on the 80 V100 GPUs?

The learning rate schedule in the appendix suggests that pre-training has a total of 200,000 steps and fine-tuning has 100,000 steps, but I'm not sure (for one, I don't think the number of fine-tuning steps would be the same for all downstream tasks)

What is the temporal resolution of the initial states of ERA5 used in training and evaluation?
Are there any results on ERA5 performance without pre-training (at all)?

Thanks.

Hi, thanks for showing your interest in ClimaX.

It took 2-3 days.
You're right about the scheduler. We used the same number of finetuning epochs (50) for all tasks, but since different tasks have different numbers of data points, the finetuning steps should be different. You can follow this schedule: linear warmup for 5 epochs and cosine decay for 45 epochs.
What do you mean by temporal resolution? We use only 1 step in the input, which means it has a shape of V x H x W
We ran some experiments in the early stage of the project, and the takeaway is ViT overfits heavily without any pretraining. You need to use very heavy dropout rates and/or weight decays to mitigate this.

Thanks for the response! Regarding the temporal resolution, I am interested in the frequency of input assimilations or initial states included in the training set (sampled from the hourly ERA5 data). For instance, FourCastNet and GraphCast use assimilations at 00, 06, 12, and 18H UTC, resulting in four training samples per day. On the other hand, Pangu-Weather employs hourly initial states. Could you provided details on how often ERA5 data is sampled for ClimaX's training set?

Hi @itsnamgyu! Thanks for your question. One key difference in inference setup for ClimaX vs other methods you mentioned is that we condition on $\Delta t$ in the future we want to make predictions for and can therefore make a prediction at any hour in a single forward pass vs doing rollouts in other methods.
Specifically https://github.com/microsoft/ClimaX/blob/6d5d354ffb4b91bb684f430b98e8f6f8af7c7f7c/src/climax/global_forecast/datamodule.py#L44
allows us to control the maximum prediction range we want to finetune our model on. We used the available hourly ERA5 data here.

@itsnamgyu I assume your question has been answered. Feel free to open it again if you have more questions.

Oh, yes, I was asking about this information We used the available hourly ERA5 data here. Thanks for the help :)

authors: itsnamgyu rejuvyesh tung-nd

repo: microsoft/ClimaX


text:
Add Mask R-CNN

Description

Add Mask R-CNN notebook.
This PR was created for figuring out whether a PR from a fork won't trigger all testing pipelines.  Since this PR triggered all testing pipeline, it now replaces https://github.com/microsoft/ComputerVision/pull/363 to be the main PR for Mask R-CNN notebook.
Related Issues

Checklist:

[ ] I have followed the contribution guidelines and code style for this project.
[X] I have added tests covering my contributions.
[X] I have updated the documentation accordingly.
[X] This PR is being made to staging and not master

@simonzhaoms Great to see this PR progressing well!! I will do a code review also on Monday, but don't expect much additional comments to what JS already pointed out.

@PatrickBue Please comment on the new PR https://github.com/microsoft/ComputerVision/pull/389

@simonzhaoms @gramhagen The notebook looks great and exactly what I had in mind. The user can optionally provide ground truth masks, and the rest of the code still looks very similar to the "default" case in the 01 notebook where no mask was provided. Really like the visualizations also of the ground truth and of just running the pre-trained classifier without any training.
Am going to do a review of the code next..

@PatrickBue did you mean to tag me here?

So, made it through most of the code. I spend 1.5 hours and it's getting difficult to focus. I saw at least one place (maybe more) which might be breaking the hyperdrive notebook.

@simonzhaoms Thanks for making some of the edits already! I now went through all the scripts (except for the tests scripts) and left more comments.
The code looks pretty good already, and my comments are mostly small items. However there are many of them which is due to this PR being massive.
I'd recommend to first address the comments, then to ask for me to look at the code again, and only afterwards to fix the tests again (that should make you life easier).

authors: PatrickBue gramhagen simonzhaoms

repo: microsoft/ComputerVision


text:
CsWin32 analyzer slows Visual Studio 2019 editor down to a crawl

Actual behavior
As soon as I add a reference to Microsoft.Windows.CsWin32 to my project, the Visual Studio 2019 editor slows down to a crawl to the point of barely being usable anymore. The editor lags behing as I type, autocomplete takes several seconds (3~10) to pop up. Code color coding and error underlining is delayed by the same amount.
The moment I remove the reference and hit save, it's back to normal again and editing, autocomplete and code coloring and error underlining is again instantaneous.
Expected behavior
I'd expect the code generator/analyzer to perform better. I see no reason it should be so slow.
Repro steps

NativeMethods.txt content:

// Misc

FILE_ACCESS_FLAGS
STATFLAG
STGTY

// General windowing

SetWindowsHookEx
GetCurrentThreadId
CallNextHookEx
GetClientRect
RegisterClassEx
CreateWindowEx
ShowWindow
UpdateWindow
DefWindowProc
ValidateRect
LoadCursor
WM_PAINT
WM_SIZE

// WIC

IWICImagingFactory
IWICBitmapSource

// DirectWrite

DWriteCreateFactory
IDWriteFactory

// Direct2D

D2D1CreateFactory
ID2D1Factory
ID2D1RenderTarget
D2D1_RENDER_TARGET_PROPERTIES
D2D1_HWND_RENDER_TARGET_PROPERTIES
D2D_RECT_F
D2D1_ROUNDED_RECT
D2D1_ELLIPSE
D2D1_FILL_MODE
D2D_MATRIX_3X2_F
D2D1_STROKE_STYLE_PROPERTIES
ID2D1RectangleGeometry
CWPSTRUCT

NativeMethods.json content (if present):

{
  "$schema": "https://aka.ms/CsWin32.schema.json",
  "public": false,
  "allowMarshaling": true
}

Any of your own code that should be shared?

My solution consists of 74 .NET 5 c# projects
Context

CsWin32 version: 0.1.422-beta
Win32Metadata version: 10.0.19041.5-preview.138
Target Framework: .NET 5
LangVersion: n/a

On closer inspection I suspect that the SourceGenerator's Execute is called on every code change in the editor, which would explain the slow down.
My suggestion would be to cache the compilationUnits obtained at line 173 of SourceGenerator.cs and only regenerate them when either the contents of NativeMethods.txt or NativeMethods.json changes (Likely just checking the modified time would be sufficient).

This issue is caused by failing to reuse cached outputs when the input hasn't changed. If you prefer, you can use SoftReference<T> instead of the Large Object Heap for avoiding premature collection of the cached value.

@AArnott ?

@sharwell Where is this reusing outputs documented? I had no idea that was a thing.

This will largely be a dupe of #38

This issue is caused by failing to reuse cached outputs when the input hasn't changed. If you prefer, you can use SoftReference<T> instead of the Large Object Heap for avoiding premature collection of the cached value.

Not just that, it could be contributed with the usage of YamlDotNet in Docs class which is used in Generator to return the docs in the generated code. I am filing an issue in the metadata repository to embed the docs in the winmd files in a way that all languages can extract && make use of them without too much effort or performance issues.

Thanks, although we didn't really need that extra issue. I'm doing the work of moving the docs to the win32metadata repo as part of the issues we already had. :)

Several perf fixes have been made, Can you try 0.1.505-beta or later (from our CI feed) and let us know how it's looking?
We are not caching the actual generated output yet. We could potentially do that, but I'm not sure how effective it would be because in fact the inputs span more than just NativeMethods files. Because we allow you to define or reference your own types for ours, any change you make (manipulate references or code changes) could potentially alter the generated output.

@BasTossings you may want to review this comment, as apparently a VS extension has been known to interact negatively with source generators and needs a fix.
I'm going to close this issue as fixed given the perf work we've already done. But I'll open a new issue to track future perf opportunities as well.

authors: AArnott AraHaan BasTossings bastos80 sharwell