failed to run hieroglyph-test on Mac

Hello,
I tried to run hieroglyph-test on my Mac, but failed to build slides.
I wonder if this is a bug or due to system setting problems.
I tested sphynx-test and it worked fine.
Thanks,
- jongwoo lim
jwlim@jwlim-mbp-r ~/Documents/slides/hieroglyph-test $ make slides
sphinx-build -b slides -d _build/doctrees   . _build/slides
Running Sphinx v1.3b3
making output directory...
loading pickled environment... not yet created
loading intersphinx inventory from https://docs.python.org/objects.inv...
building [mo]: targets for 0 po files that are out of date
building [slides]: targets for 1 source files that are out of date
updating environment: 1 added, 0 changed, 0 removed
reading sources... [100%] index
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
writing output... [100%] index
Exception occurred:
  File "/usr/local/lib/python2.7/site-packages/Jinja2-2.7.2-py2.7.egg/jinja2/utils.py", line 388, in __getitem__
    rv = self._mapping[key]
TypeError: unhashable type: 'dict'
The full traceback has been saved in /var/folders/87/fhfl43hd0tj05wlchl06jyh80000gn/T/sphinx-err-QpwjJa.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!
make: *** [slides] Error 1

jwlim@jwlim-mbp-r ~/Documents/slides/hieroglyph-test $ cat /var/folders/87/fhfl43hd0tj05wlchl06jyh80000gn/T/sphinx-err-QpwjJa.log
# Sphinx version: 1.3b3
# Python version: 2.7.9
# Docutils version: 0.11 release
# Jinja2 version: 2.7.2
# Last messages:
#
#   looking for now-outdated files...
#   none found
#   pickling environment...
#   done
#   checking consistency...
#   done
#   preparing documents...
#   done
#   writing output... [100%] index
# Loaded extensions:
#   sphinx.ext.intersphinx (1.3b3) from /usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/ext/intersphinx.pyc
#   sphinx.ext.doctest (1.3b3) from /usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/ext/doctest.pyc
#   alabaster (0.7.1) from /usr/local/lib/python2.7/site-packages/alabaster-0.7.1-py2.7.egg/alabaster/__init__.pyc
#   sphinx.ext.coverage (1.3b3) from /usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/ext/coverage.pyc
#   sphinx.ext.pngmath (1.3b3) from /usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/ext/pngmath.pyc
#   sphinx.ext.todo (1.3b3) from /usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/ext/todo.pyc
#   sphinx.ext.ifconfig (1.3b3) from /usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/ext/ifconfig.pyc
#   hieroglyph (unknown version) from /usr/local/lib/python2.7/site-packages/hieroglyph-0.6.5-py2.7.egg/hieroglyph/__init__.pyc
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/cmdline.py", line 246, in main
    app.build(opts.force_all, filenames)
  File "/usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/application.py", line 263, in build
    self.builder.build_update()
  File "/usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/builders/__init__.py", line 237, in build_update
    'out of date' % len(to_build))
  File "/usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/builders/__init__.py", line 308, in build
    self.write(docnames, list(updated_docnames), method)
  File "/usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/builders/__init__.py", line 346, in write
    self._write_serial(sorted(docnames), warnings)
  File "/usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/builders/__init__.py", line 354, in _write_serial
    self.write_doc(docname, doctree)
  File "/usr/local/lib/python2.7/site-packages/hieroglyph-0.6.5-py2.7.egg/hieroglyph/builder.py", line 101, in write_doc
    result = super(AbstractSlideBuilder, self).write_doc(docname, doctree)
  File "/usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/builders/html.py", line 441, in write_doc
    self.handle_page(docname, ctx, event_arg=doctree)
  File "/usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/builders/html.py", line 780, in handle_page
    output = self.templates.render(templatename, ctx)
  File "/usr/local/lib/python2.7/site-packages/Sphinx-1.3b3-py2.7.egg/sphinx/jinja2glue.py", line 133, in render
    return self.environment.get_template(template).render(context)
  File "/usr/local/lib/python2.7/site-packages/Jinja2-2.7.2-py2.7.egg/jinja2/environment.py", line 791, in get_template
    return self._load_template(name, self.make_globals(globals))
  File "/usr/local/lib/python2.7/site-packages/Jinja2-2.7.2-py2.7.egg/jinja2/environment.py", line 761, in _load_template
    template = self.cache.get(name)
  File "/usr/local/lib/python2.7/site-packages/Jinja2-2.7.2-py2.7.egg/jinja2/utils.py", line 339, in get
    return self[key]
  File "/usr/local/lib/python2.7/site-packages/Jinja2-2.7.2-py2.7.egg/jinja2/utils.py", line 388, in __getitem__
    rv = self._mapping[key]
TypeError: unhashable type: 'dict'
jwlim@jwlim-mbp-r ~/Documents/slides/hieroglyph-test $

It looks like it's raising an exception during template rendering; a couple questions/suggestions:

What is slide_theme set to in conf.py?
It looks like you're on the development branch of Hieroglyph; can you try updating to the latest development version and see if it still does this? What if you install the last release (0.6.5)?
Is this a test project/presentation you created to play with Hieroglyph? Are you able to share the conf.py and source files?
You mentioned that Sphinx-test works; do you mean that it works when you run make html?

I installed hieroglyph using easyinstall. Is there a chance that the development version is installed through this?
I am running the 0.6.5 version of hieroglyph.
jwlim@jwlim-mbp-r ~/Documents/slides/hieroglyph-test $ easy_install hieroglyph
Searching for hieroglyph
Best match: hieroglyph 0.6.5
Processing hieroglyph-0.6.5-py2.7.egg
hieroglyph 0.6.5 is already the active version in easy-install.pth
Installing hieroglyph-quickstart script to /usr/local/bin

Using /usr/local/lib/python2.7/site-packages/hieroglyph-0.6.5-py2.7.egg
Processing dependencies for hieroglyph
Finished processing dependencies for hieroglyph

What is slide_theme set to in conf.py?

html_theme = 'alabaster'
...
# -- Hieroglyph Slide Configuration ------------

extensions += [
    'hieroglyph',
]

slide_theme = 'single-level'
slide_levels = 3

I also tried slides and it didn't work.

Is this a test project/presentation you created to play with Hieroglyph? Are you able to share the conf.py and source files?

I did not make any changes to conf.py after running hieroglyph-test.
Thanks a lot for the support!

jongwoo lim

Thanks for the details.
Hieroglyph 0.6.5 has a known incompatibility with Sphinx 1.3, which was just released. See https://github.com/nyergler/hieroglyph/issues/84 for details.
I'm working on getting a new release out that will fix this, but until then you can work around it by forcing Sphinx to use the 1.2 series:
$ easy_install sphinx~=1.2.0

I'll also add a notice to the README momentarily.
Sorry for the trouble, thanks for reporting this!

Thanks for the quick answer!
It solved the problem, and I look forward to the updated version of
hieroglyph soon. :)

jongwoo lim

On Mon, Mar 16, 2015 at 11:59 AM, Nathan Yergler<EMAIL_ADDRESS>wrote:

Closed #88 https://github.com/nyergler/hieroglyph/issues/88.
—
Reply to this email directly or view it on GitHub
https://github.com/nyergler/hieroglyph/issues/88#event-254953805.


no conn, hlsvp or url_encoded_fmt_stream_map information found in video info

youtube-dl --verbose 'http://www.youtube.com/watch?v=a6aXv9oDHc8'
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'http://www.youtube.com/watch?v=a6aXv9oDHc8']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.04.17
[debug] Python version 2.7.6 - Linux-3.13.0-49-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] exe versions: ffmpeg 2.5.2, ffprobe 2.5.2
[debug] Proxy map: {}
[youtube] a6aXv9oDHc8: Downloading webpage
[youtube] a6aXv9oDHc8: Downloading video info webpage
[youtube] a6aXv9oDHc8: Extracting video information
ERROR: no conn, hlsvp or url_encoded_fmt_stream_map information found in video info; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "./youtube-dl/youtube_dl/extractor/youtube.py", line 883, in _real_extract
    raise ValueError('No stream_map present')  # caught below
ValueError: No stream_map present
Traceback (most recent call last):
  File "./youtube-dl/youtube_dl/YoutubeDL.py", line 651, in extract_info
    ie_result = ie.extract(url)
  File "./youtube-dl/youtube_dl/extractor/common.py", line 275, in extract
    return self._real_extract(url)
  File "./youtube-dl/youtube_dl/extractor/youtube.py", line 1111, in _real_extract
    raise ExtractorError('no conn, hlsvp or url_encoded_fmt_stream_map information found in video info')
ExtractorError: no conn, hlsvp or url_encoded_fmt_stream_map information found in video info; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

This broadcast is not yet started.


install.sh cd-ing glitch when nvm is already installed

Reproduction: run the following so that the second time, install.sh detect that nvm is already installed.
$ NVM_DIR=nvm-dir ./install.sh
$ NVM_DIR=nvm-dir ./install.sh

This happens:
=> nvm is already installed in ./nvm-dir, trying to update using git
=> bash: line 72: cd: ./nvm-dir: No such file or directory
bash: line 73: cd: ./nvm-dir: No such file or directory

This happens because line 62 has already cd-ed to nvm-dir.
EDIT: a quick fix would be for line 62 to pushd and then popd immediately afterwards.

Hmm, I'm not able to reproduce that. In what directory are you running install.sh from?
I do see the logical issue in the code - that should be trivial to fix.

I'm running it in /Users/cspotcode/Desktop/nvm-test/.  I have git installed, so the install script is using git instead of wget or curl; maybe that's the difference between my system and yours?

Pretty sure I have git installed ;-)
So you've downloaded install.sh to that dir, and you're pointing NVM_DIR to another empty/nonexistent dir?

Here's a full reproduction case.  I'm running Mac OSX 10.10.5.
cd ~/Desktop
mkdir issue1137
cd issue1137
mkdir custom-nvm-dir
touch profile
curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.31.2/install.sh | NVM_DIR=custom-nvm-dir PROFILE=../profile bash
curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.31.2/install.sh | NVM_DIR=custom-nvm-dir PROFILE=../profile bash

Thanks! When I run
cd ~/Desktop
mkdir -p issue1137
cd issue1137
mkdir -p custom-nvm-dir
touch profile
curl -o- https://raw.githubusercontent.com/creationix/nvm/master/install.sh | NVM_DIR=custom-nvm-dir PROFILE=../profile bash
curl -o- https://raw.githubusercontent.com/creationix/nvm/master/install.sh | NVM_DIR=custom-nvm-dir PROFILE=../profile bash

I do still see bash: line 85: cd: custom-nvm-dir: No such file or directory

K, should be all fixed in latest master, which will be in the next release.


Add translation key/value for debug

For debug purposes, should be possible to add, maybe with the translation-tool, a button that toggles the translation label and the actual translation
{
  message: 'hello'
}

<p>{{ $t('message') }}</p>

with a toggle on should show message instead of hello

all components are moved to another package that I'm working on, so I'm going to close this ticket here and working on this one https://github.com/MatteoGabriele/vue-i18n-utils/issues/1


Fix Dynamic types

Summary

Dynamic accepts T as a prop, but this is wrong, we should infer the prop based on component.
How did you test this change?

The original implementation is based on solid-headless
Example:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.889%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1255

Relevant Lines:
1339

💛  - Coveralls


Wrapper for OCINlsGetInfo()

Hello,
I would like to have a wrapper around OCINlsGetInfo() inside the ocilib-Library.

Hi,
This will be added to OCILIB in version V4.8.0 (as this will result in a new API function) soon.
Regards,
Vincent


Query condition should be 'country' instead of 'region'

https://github.com/firebase/snippets-web/blob/08888388e7cf300369f20a2046457c7e957f0010/firestore/test.firestore.js#L811-L811
should be: citiesRef.where('country', 'in', ['USA', 'Japan']);

https://github.com/firebase/snippets-web/blob/08888388e7cf300369f20a2046457c7e957f0010/firestore/test.firestore.js#L811-L811
seharusnya :citiesRef.where('country', 'in', ['USA', 'Japan']);

https://github.com/firebase/snippets-web/blob/08888388e7cf300369f20a2046457c7e957f0010/firestore/test.firestore.js#L811-L811
seharusnya :citiesRef.where('country', 'in', ['USA', 'Japan']);

This was fixe a while ago, must have forgotten to close this issue:
https://github.com/firebase/snippets-web/blob/b27016fa3e86cfffd7fef7b3ee6fb34f5dbf9759/firestore/test.firestore.js#L887

This was fixe a while ago, must have forgotten to close this issue:
https://github.com/firebase/snippets-web/blob/b27016fa3e86cfffd7fef7b3ee6fb34f5dbf9759/firestore/test.firestore.js#L887


$scope.$broadcast('init-map') troubleshooting help please

Hello Allen,
I have slightly modified your Example
https://rawgit.com/allenhwkim/angularjs-google-maps/master/testapp/map_lazy_init.html
to
http://<IP_ADDRESS>:9050/testapp/custom/map_lazy_init_mine.html
Entire and much more details documented here
https://docs.google.com/document/d/1z4a7G_ZNqG2V5hPb_ZO3b3oTteKswQheKYQiiZBFwrk/
Also raised in Stackoverflow.
https://stackoverflow.com/questions/31215685/debug-scope-broadcastinit-map-in-ngmap-angularjs-google-map-troubleshoot
Can you please help me on this troubleshoot.
Thanks and Regards,
Raj

TL; DR;
Your example fires init-map event when document is loaded.
angular.element(document).ready(function () {
    console.log('Hello World');
 	$scope.clickMe(5);
    console.log('Hello World2 ');
});

It seems all look fine to me. what's the problem is with your example.

Hi Allen,
Thanks for the reply ..
Inside that clickMe(5) ,
All 4 options are not firing the close loop, so that the {{zoom}} get the value $scope.$broadcast('init-map') , $scope.$emit('init-map') , $rootScope.$broadcast('init-map') , $rootScope.$emit('init-map')
You can see that in
http://<IP_ADDRESS>:9050/seed/#/app/configure-svp
And for the working example,
http://<IP_ADDRESS>:9050/testapp/custom/map_lazy_init_mine.html
this  http://<IP_ADDRESS>:9050/testapp/app/scripts/services/attr2_options.js - Line 125 watches the {{zoom}} variable and gets updated.. But it is not the case with configure-svp.
   Can you please review the difference in both console.log details.

Thanks and Regards,
Raj
https://docs.google.com/document/d/1z4a7G_ZNqG2V5hPb_ZO3b3oTteKswQheKYQiiZBFwrk/edit#heading=h.l0fofjpdmhc1

Please make the problem short and simple.
Honestly I do not read all your document and find out what problem  is . I only read your html  code without other library and find what is not working
Thus send me the example code without 3rd party library and let me know what is expected and What is not working

Hello allen,
As per one of my friend's suggestion , In the clickMe, wrapped up with few ms delay (when making REST service call, there will be some realtime work will happen, so this $timeout hack is fine)  and $scope.$emit and it Worked.. So, updated the google doc as well closing this issue.

$scope.clickMe = function(zoomValue){
console.log("Click Me");
$scope.zoom = zoomValue;
$timeout(function() {
$scope.$emit('init-map');
}, 100);
};

Thanks and Regards,
rAj


exit code 0 on slimerjs

If I have a failing test and run it with phantomjs, I get exit code 1:
$ phantomjs mocha-phantomjs-core.js test/index.html
…
  7 passing (52ms)
  1 failing
…
$ echo $?
1

When I run the same tests with slimerjs, I get exit code 0:
$ slimerjs mocha-phantomjs-core.js test/index.html
…
  7 passing (52ms)
  1 failing
…
$ echo $?
0

I have not tested explicitly, but this should break many tools, e.g. travis.
Note that I tested with slimerjs 0.9 due to #25.

Yeah you should be using slimerjs 0.9.6 to ensure you include laurentj/slimerjs#78 otherwise the exit code will always be 0.

slimerjs exit codes were always 0 before that change, which landed in 0.9.6. Make sure you are using exactly 0.9.6. I do need to take another look at slimerjs now that it has an alpha of 1.0.0

Yes, I am using exactly 0.9.6.


Add long running performance benchmark feature

Travis CI benchmarks seem to be very volatile and we really could use something more accurate for permanent record.  I'm suggesting adding a --long-run flag to the PBench (Pinch Bench) suite to run much longer and have that used only on Travis CI master branch merges.
We will of course need to make sure that it won't time out on Travis CI if there are any time limitations (of which I'm not sure).
Thinking more about this I believe I should also have it work with an environment flag and only have one CI runner perform this action under allowed failures.  It should be a Linux only system as that will allow other PRs to start there tests even if it's still running.

The long run numbers here are very close to what I get on my machine!
https://travis-ci.org/danielpclark/faster_path/jobs/353433323

I updated the README with the new numbers


Would be great to be able to generate addresses from only the xpub

Great to see interest in this. Thanks for submitting an issue.
See https://github.com/iancoleman/bip39/issues/40 - BIP32 root key with xpub throws error and does not generate addresses
Since this is a duplicate I'll close it and add any progress to the original issue.


move ghc-iserv to lib/bin

For validation, the testsuite looks for ghc-iserv in lib/bin. This is causing near 200 validation failures.
@angerman @snowleopard

@angerman What do you think? I'm on Windows, so I don't think I even build iserv on my machine.

the location is _build/stage1/bin/ghc-iserv on my Linux machine by the way

This needs to be parameterized. Especially as this will likely move someplace into utils/iserv.../ soon.

In general, we should try not to bend the layout for the testsuite, but adapt the testsuite to the new layout. Again this may need to be parameterized so that the old (make), and new (shake) based buildsystems both work.

@angerman See my comment here: https://github.com/snowleopard/hadrian/issues/592#issuecomment-388612287. I don't think I've ever seen any explanation for the rationale behind moving things to bin. If this is a useful feature, there should be a corresponding issue/PR that takes care of testsuite as well.

@snowleopard @angerman Current make system does not seem to take the location of ghc-iserv as an argument. I am looking further into this matter, but as of now, path to every other dependency can be specified as an argument (compiler, check-ppr, hp2ps, hsc etc), but not ghc-iserv.

I agree with @angerman that we ideally want this to be configurable. It should not be too hard to add an option for pointing to the iserv binary. I would however understand it if you decide to move the binary under lib/bin for now, so that @chitrak7 can make progress on the validation task. I might in fact look into this myself, I think I know what needs to be done in the testsuite driver.
@chitrak7 Do you think you could create a (GHC Trac) ticket with a summary of the problems you get when ghc-iserv is not under lib/bin/, with instructions to reproduce (just hadrian validate?) and assign it to me (I'm alpmestan on trac) ? Thanks!

Just an update: iserv has been moved in GHC and I made the necessary changes in Hadrian in https://github.com/snowleopard/hadrian/commit/8dba6748e84c632c3c0302620c3e62c9fcc3e948.

I think we can close it now. The issue has been taken care of!!


build error: $ go build api.go:4:2: no Go files in /usr/local/go/src/database

build success in docker environment, but build failed outside docker.
the following is the steps build api outside docker:
DJdeMacBook-Pro:api dj$ go get ./vendor/database
go install: no install location for directory /Users/dj/WorkSpace/GitSpace/10_19/goDR_12_16/GoDoRP/api/vendor/database outside GOPATH
	For more details see: 'go help gopath'
DJdeMacBook-Pro:api dj$ go get .
api.go:4:2: no Go files in /usr/local/go/src/database
DJdeMacBook-Pro:api dj$ go build
api.go:4:2: no Go files in /usr/local/go/src/database
DJdeMacBook-Pro:api dj$ go version
go version go1.9.2 darwin/amd64
DJdeMacBook-Pro:api dj$

`DJdeMacBook-Pro:api dj$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/dj/WorkSpace/GitSpace/10_19/goDR_12_16/GoDoRP"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/dz/n0nt7d3924g6h5436gfc6xsc0000gn/T/go-build606782250=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
DJdeMacBook-Pro:api dj$ `

golang version is 1.9.2;
I guess this error is related with vendor fearture, but I don't know how to fix.
can you help me for this issue?

Hi @fish2016. I have not contributed to this repo in a while, but it looks like your go path is not set up correctly. Have you successfully started up other golang projects?
Also, one the features of this project is that you could develop without needing to set up go.
Lastly, go is now working on an official package manager tool , so I would now use that instead of a vendor folder.


chore(main): delete unnecessary configs

Types of changes
What kind of change does this PR introduce? (check at least one)

[ ] Breaking change
[ ] Bugfix
[ ] Feature
[ ] Code style update
[x] Refactoring
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

Description

Issue Number: N/A

Codecov Report

Merging #128 into develop will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           develop   #128   +/-   ##
======================================
  Coverage      100%   100%
======================================
  Files           12     12
  Lines          120    120
  Branches        25     21    -4
======================================
  Hits           120    120

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 8aa1423...c416fc9. Read the comment docs.


Support for secure management of OAuth tokens for Slack notification

What would you like to be added:
With the release of PipeCD v0.44.0, OAuth tokens can now be used for Slack notification settings.
Slack OAuth token (oauthToken: xxxx) should be treated as confidential information.
In the Piped config, we would like to read external encrypted information and send notifications to any desired Slack channel.
For example, we can read files mounted from Kubernetes secrets, such as spec.git.sshKeyFile or spec.pipedKeyFile.
Why is this needed:
Slack token should be securely managed as sensitive elements.
However, Piped config does not provide a direct scheme to read Kubernetes resources such as environment variables or ExternalSecret.
Therefore, developers must include token information for Slack notifications in plain text within the data fields of the Piped config manifest.

@GotoRen  @kentakozuka @khanhtc1202
I'd like to work on this issue
I've seen that OAuthToken is also stored as string w/o encode base64
should we change it to OAuthTokenData w/ base64 and addition OAuthTokenFile as we had spec.git.sshKeyFile or spec.pipedKeyFile

@hungran that is great, but I think we should make the current field deprecated first, and imtroduce the field name with suffix Data as you said. We can remove the deprecated field later.

@hungran
I think it's a great idea !!
As @khanhtc1202  mentions, it's best to do this after deprecating an existing field.

Hi, 100% agreed that we should make the currrent field as deprecate (oauthToken)
I just have no insight for implementing that, could you give me some suggestions? @khanhtc1202

Hi, I read discussion and I'm agree with your saying.
I have done this implementation. So if you are confused about the implementation, I will implement it for you.

@sivchari, @hungran already sent a PR #4525 as a fix for this. Feel free to review; that would be a great help 😄

oh I might missed the comment above, feel free to reject my PR and do it :)

@sivchari, are you okay to take over this? 👀

@khanhtc1202
Sorry to rate response. I've reviewed #4525 . Please final review.

Thank you @sivchari !!


Landscape mode

We tested Agora flutter SDK in the demo app in landscape mode with two different scenarios and we got this result.
How can we solve that first scenario?

If the capturer device and the player device both capture video different facing camera (front camera and back camera).
it shows the rotated video in the player device

If the capturer device and the player device both capture video same facing camera (front camera or back camera).
it shows the perfect result in the player device.

just unlock the screen orientation and test again, then share the screenshot pls.

It's working when the unlock screen orientation. but what about when the screen is a lock orientation?
can you suggest?

If you lock the screen orientation, just work on the locked orientation is expect.


chore: cleanup solution and fix unresolved symbols in nuget pkg

@Mezo-hx I've slightly cleaned up the solution, packed it as a nuget package and tested locally - all symbols are resolved as one would expect.
Also:

retargeted library to net8.0
cleaned up .gitignore by removing unnecessary commented sections
added package icon, since it's referenced in .csproj file

MonoGame.ImGuiNet.1.1.0.nupkg.zip
ready to publish nuget pkg

Gotcha!

@Mezo-hx when can we expect a new package version being published?

@Mezo-hx when can we expect a new package version being published?

Probably not anytime soon, exactly. I have decided to drop the NuGet package, but if there are a few people who would like it back, then I can try to find time to update it. I can't really be certain now as I've been pretty busy these days, which is also the reason why the library hasn't been updating.

Ah, gotcha. To save you some time, I can prepare a PR with github actions configured to pack and deploy nuget to the feed automatically

Also, if you'd share what you'd like to include in potential future updates - would be great. Possibly someone from the community would be able to work on it

Ah, gotcha. To save you some time, I can prepare a PR with github actions configured to pack and deploy nuget to the feed automatically

Yeah, that'll help a lot!

Also, if you'd share what you'd like to include in potential future updates - would be great. Possibly someone from the community would be able to work on it

I'll probably be able to set up a Trello page for that, hopefully today since it's the weekend.


Cannot use SQL Server with Identity

We can not create the Identity tables on our SQL Server, however, using localdb works fine.
We are using the Package Manager Console to migrate and update.
If you are seeing an exception, include the full exceptions details (message and stack trace).
Exception message:
Failed executing DbCommand (1ms) [Parameters=[], CommandType='Text', CommandTimeout='30']
DROP INDEX [UserNameIndex] ON [AspNetUsers];

Stack trace:
fail: Microsoft.EntityFrameworkCore.Database.Command[200102]
      Failed executing DbCommand (1ms) [Parameters=[], CommandType='Text', CommandTimeout='30']
      DROP INDEX [UserNameIndex] ON [AspNetUsers];
System.Data.SqlClient.SqlException (0x80131904): Cannot drop the index 'AspNetUsers.UserNameIndex', because it does not exist or you do not have permission.
   at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.SqlCommand.RunExecuteNonQueryTds(String methodName, Boolean async, Int32 timeout, Boolean asyncWrite)
   at System.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean asyncWrite, String methodName)
   at System.Data.SqlClient.SqlCommand.ExecuteNonQuery()
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.Execute(IRelationalConnection connection, DbCommandMethod executeMethod, IReadOnlyDictionary`2 parameterValues)
ClientConnectionId:3ec80fda-53a6-4c66-bdf4-c50e1c67e3dc
Error Number:3701,State:6,Class:11
System.Data.SqlClient.SqlException (0x80131904): Cannot drop the index 'AspNetUsers.UserNameIndex', because it does not exist or you do not have permission.
   at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.SqlCommand.RunExecuteNonQueryTds(String methodName, Boolean async, Int32 timeout, Boolean asyncWrite)
   at System.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean asyncWrite, String methodName)
   at System.Data.SqlClient.SqlCommand.ExecuteNonQuery()
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.Execute(IRelationalConnection connection, DbCommandMethod executeMethod, IReadOnlyDictionary`2 parameterValues)
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.ExecuteNonQuery(IRelationalConnection connection, IReadOnlyDictionary`2 parameterValues)
   at Microsoft.EntityFrameworkCore.Migrations.MigrationCommand.ExecuteNonQuery(IRelationalConnection connection, IReadOnlyDictionary`2 parameterValues)
   at Microsoft.EntityFrameworkCore.Migrations.Internal.MigrationCommandExecutor.ExecuteNonQuery(IEnumerable`1 migrationCommands, IRelationalConnection connection)
   at Microsoft.EntityFrameworkCore.Migrations.Internal.Migrator.Migrate(String targetMigration)
   at Microsoft.EntityFrameworkCore.Design.Internal.MigrationsOperations.UpdateDatabase(String targetMigration, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.UpdateDatabase.<>c__DisplayClass0_1.<.ctor>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)
ClientConnectionId:3ec80fda-53a6-4c66-bdf4-c50e1c67e3dc
Error Number:3701,State:6,Class:11
Cannot drop the index 'AspNetUsers.UserNameIndex', because it does not exist or you do not have permission.

Further technical details
EF Core version: 2.0
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10
IDE: Visual Studio Enterprise 2017

@shammelburg Is the schema on your localdb instance different to the one on your other SQL Server instance? Has the SQL Server schema been modified by other Migrations or manual changes since it was created?

@ajcvickers The schema is the same. Nothing has been modified.

@shammelburg That's very surprising. I was expecting that the schema would be different and the main thrust of the investigation here would be figuring how they got to be different and what to do about it. How did you determine the schema was the same in both cases? Do they both contain the UserNameIndex or both not contain the UserNameIndex?

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


[#581] Removed session_preference

Removed session_preference code per issue https://github.com/PyCon/pycon/issues/581.
Tested this in a local vagrant environment and it worked.

should this include a migration to remove the sessions_preference column?

Hm, it's not critical, but seeing as how call for proposals is over I don't know it matters much?

I started work on this myself (having forgotten this pull request was here!) back in 1c2faefd2e6f5e4ab0bdfbc7649247176b539d77, but now that I've re-read this pull request I have just committed f8e9d3 which finishes out the change by removing the choices constant, which I had forgotten to do but which was done correctly here in this pull request — thanks!


load_resource doesn't find nested resource when association is polymorphic

Steps to reproduce
Given these models:
Polymorphic model Attachment
class Attachment < ApplicationRecord
  belongs_to :attachable, polymorphic: true, optional: true
end

Model Milestone
class Milestone < ApplicationRecord
  has_many :files, class_name: 'Attachment', as: :attachable
end

And controller FilesController
class FilesController < ApplicationController
  load_resource :milestone
  load_and_authorize_resource :file, through: :milestone

  def destroy
    @file.destroy

    head :no_content
  end
end

Expected behavior
It should find milestone's file and assign it to @file variable
Actual behavior
Instance variable @file is nil
System configuration
Rails 5.2.0:
Ruby 2.4.3:
CanCanCan 2.0

It wasn't a bug, but you need to specify option parent: false for load_and_authorize_resource :file, through: :milestone like this
load_and_authorize_resource :file, parent: false, through: :milestone


[Feature] List incompatible mods

It would be awesome if there were a command to list any mods that are not compatible with the current acceptable-game-versions. After updating to 1.19.1 I am trying to figure out which mods are still pending updates and the only easy way right now is for me to simply run the game and see which mods cause it to crash or go to each mod page manually.
Thanks!

I agree, something like this would be helpful.  You kinda can figure it out by upgrading the MC version and then running packwiz update --all which will prompt you with a list of all the mods it'll update.  So anything it doesn't list would likely be incompatible.

Further clarification: not only will the --all option list mods it'll upgrade, but it'll list the mods that (1) it can't update likely because it's a direct download url; and (2) failed to find a valid version.  The latter (2) are the mods that were not found to match the acceptable-game-versions.
You can then proceed with the update by pressing Y or cancel by pressing n.

--all option list mods it'll upgrade, but it'll list the mods that (1) it can't update likely because it's a direct download url; and (2) failed to find a valid version. The latter (2) are the mods that were not found to match the acceptable-game-versions.
You can then proceed with the update by pressing Y or cancel by pressing

Many mods support 1.19 and 1.19.1 so they have already been updated in my modpack (which supported 1.19 and I am updating to support 1.19.1). This means that running packwiz update --all doesn't list those mods even though they do support 1.19.1 because they have already been updated to a version that supports 1.19.1.

Sure, I get that.  I thought you were looking for incompatible mods.  So I thought the mods that failed the update --all with "failed to find a valid version" might be sufficient.

See also: https://github.com/packwiz/packwiz/issues/40


BUILD.vbs not working

I notice that it opens a new window but the new window instantly closes itself and nothing happens. I have everything I need currently installed on my system.

Can you open cmd and type python --version

Closing due to inactivity and lack of proofs. Reopen if necessary

Can you open cmd and type python --version and send the screenshot.

the version is 3.11.0

Can you open cmd and type python --version and send the screenshot.

the version is 3.11.0

Can you download the version I have told in the settings, also if the build.VBS file is not working, open the cmd in that dir and type python builder.py (its what the build.VBS does)


Align USA banner to new SR width

FRONTEND PULL REQUEST
Related Issue

#5976

Changes Proposed

align USA header to SR header
a follow up PR will address the alert

Testing

How should reviewers verify this PR? deployed on dev 6

Screenshots / Demos

before

after

I noticed that when the alert is active, the SimpleReport banner logo loses its top margin and squishes into the USA banner:

I see your notes for this PR say that a follow up PR will address the alert, so this may already be on your radar, but I just wanted to call it out.


Double allocation without reason

as!aType.slice.asImage allocates without reason, bacause asImage allocates data anyway. asImage is uses data too. In addition, it should be something like toImage.

So, it should be just as!aType.asImage

Also, to match the Slice API, I'd like to rename the dcv.core.image.Image.asType to as. I'll deal with this in the same PR - deprecate Image.asType in favor of Image.as.

Probably it should be asSlice.

Probably it should be asSlice.

There's already Image.sliced. Image.asType (or Image.as) would change underlying data to targeted type.

There's already Image.sliced. Image.asType (or Image.as) would change underlying data to targeted type.

OK

But would be this as be lazy?

Image has no range interface, so I believe having lazy as doesn't give us much. Remember, this is only a convenience if for some reason we'd like to convert image data to another type (change bit depth) without any further processing. For extensive processing needs, we should slice it and work with Slice object.

Do we really need Image type? Why?

As said in the description of the module in docs it is designed mainly to help with image I/O, but also to hold additional image metadata. Since it's data type is defined in runtime, it allows reading of unknown image format. Since Slice format is statically defined, we would have to expect certain image format when reading it, and if read image is not of expected format, we'd have to convert it. Also, Image contains additional metadata, e.g. color format (HSV, YUV, RGB etc.). And, in future Image should hold EXIF metadata.
Pipeline in DCV should be:
Image = dcv.core.image.Image
Slice = mir.ndslice.slice.Slice

LoadImage(path) --> dcv.core.image.Image
InspectAndAdoptImageFormat(Image) --> Slice
Processing(Slice) --> Slice
PackSliceToImage(Slice) --> Image
SaveImage(Image, path)

Long story short, we need image container with runtime defined data type, and additional image related metadata.

This is scripting language idioms. They are not good for D.
If you have processing, then you work with one, two, maximum three formats for processing.
They should have their own CT instantiations because performance reasons.
Then, when you want to save something, you can just call a function which accepts Slice, Metadata, and optionally RT/CT format.
The last one issue os reading. Yes, when we read something, the image format is unknown. But, as was said above, only beforehand image types are interesting. So, a user or library should define mapping, for example:

RT image type1 -> Alg1
RT image type2 -> Alg1
RT image type3 -> Alg2
Other RT image -> Error

It is not possible to eliminate this mapping. But rather hiding it in different classes implementations it is better how have an explicit way to do it and library helpers if required.
Please avoid any usage of classes (except already existing D libs, which can be replaced in future). Even async I/O can be performed without classes. D users like it because they are familiar with OOP. But this is bad practice for D. Structural programming is proper way to move forward with D.

I understand everything you're saying, and I have tried to design the API without the Image as intermediate class at first, but didn't like the way it turned out. If you're willing to make a proposition how this API should look like, please do.
Created separate issue about this, see #64

Thanks, will do


Enable win integration tests

This PR will enable integration tests also on Windows. Also, this PR unifies pipeline step conditions between different operating systems.
Fixes #398

It feels that we don't need Unpack cache step at least on Windows.
Also tested pre-installed minikube and kind on Windows, but they fail because docker inside the vm is capable to run only windows containers.
Tests are now green, @jakolehm PTAL again.


Missing class definition: XamlMetaDataProvider

The sample code makes references to a XamlMetaDataProvider class (note without a leading I) -- in app.base.h and in app.manifest -- but never defines it.  I presume it should derive from winrt::Windows::UI::Xaml::Markup::IXamlMetadataProvider

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

ID: f62891dc-cad0-0ea2-66c3-a5b58d9e3f9a
Version Independent ID: 911ab9f0-1c11-b2bf-9ecf-9fbed29df211
Content: Host a custom UWP control in a C++ Win32 app using the XAML Hosting API
Content Source: hub/apps/desktop/modernize/host-custom-control-with-xaml-islands-cpp.md
Product: uwp
Technology: apps
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Thanks for your feedback. You are correct that the walkthrough code doesn't define the XamlMetaDataProvider struct. Instead, when you build the project, this struct is defined automatically in a generated header file named XamlMetaDataProvider.h. You do not need to define XamlMetaDataProvider directly in your code.

Thanks for your reply.  Building in CMake, without the behind-the-scenes magic of a typical Visual Studio project / solution, has been a battle.  The midl, mdmerge, and cppwinrt tools are not well documented for that use case.
I have obtained the NuGet packages for Microsoft.Windows.CppWinRT.2.0.200729.8, Microsoft.UI.Xaml.2.4.3, and Microsoft.Toolkit.Win32.UI.XamlApplication.6.1.2.  I have experimented with the tool command-line options and quirks, "projected" the SDK and my custom classes into C++/WinRT v2, generated the files, compiled and linked without errors.
Now debugging, I'm trying to resolve a REGDB_E_CLASSNOTREGISTERED exception in the construction of Microsoft.Toolkit.Win32.UI.XamlHost.XamlApplication .  I've following the advice at https://docs.microsoft.com/en-us/windows/uwp/cpp-and-winrt-apis/faq#why-am-i-getting-a-class-not-registered-exception and https://docs.microsoft.com/en-us/windows/uwp/cpp-and-winrt-apis/author-apis#opt-in-to-uniform-construction-and-direct-implementation-access but no joy yet.

It sounds like you have questions about the functionality of the Microsoft.Toolkit.Win32.UI.XamlHost.XamlApplication class that goes beyond the scenarios covered in the docs. This repo is intended mainly for reporting and addressing bugs in the docs, not for assistance with technical questions of this nature. If you haven't done so already, I recommend that you post your questions to the Microsoft.Toolkit.Win32 repo. This is the open source repo where these APIs are developed, and the engineering team that builds these APIs watches over this repo.


Question: What makes a folder a project?

I can't see an option to fzf find files in a "plain" folder.
Great plugin by the way.

yeah i agree with you, sometimes you want to work in a project that isn't necessarity a gitepo or maybe it's just 1 folder of a big git repo project... it would be nice to have the ability to open up the folder structure...
this could even replace nerdtree for me if it did it.

Added FzfPreviewDirectoryFiles command. #26
Require ripgrep

Awesome, thank you!


Decimal formatter

I ran into an issue where I started using Decimal objects but my views were using a variant of number_format, so I've written a simple Formatter class for Decimal which I figured I should offer back to the project. I originally made it a method directly on Decimal, but it got ugly pretty fast and then I remembered that there are countries in the world that aren't Australia!
I tried to find a flexible middle-ground between the clumsiness of number_format and the incredible completeness of NumberFormatter (which would be way too much work to replicate). Hopefully you find this a useful addition.
// basic usage:
$f = new \Litipk\BigNumbers\Formatter();
echo $f->format("1234567.89");
echo $f->format(\Litipk\BigNumbers\Decimal::create("1234567.89"));
// "1,234,567.89"

// customisable:
$f = new \Litipk\BigNumbers\Formatter([
    'thousandsMark'=>"'",
    'decimalMark'=>',',
    'padDecimal'=>false,
    'tpl'=>'YEAH! {num}{sign} !!!',
    'sign'=>' -',
    'scale'=>5,
]);
echo $f->format(\Litipk\BigNumbers\Decimal::create("-1234567.89"));
// "YEAH! 1'234'567,89 - !!!"

I added a simple registry so you can store formats as you please for easy access:
Formatter::register('aud', ['scale'=>2, 'tpl'=>'AUD ${sign}{num}']);
Formatter::formatAs('aud', '1.234');
Formatter::formatAs('aud', \Litipk\BigNumbers\Decimal::create('1.234'));
// "AUD $1.23"

There's some syntactic sugar too, i.e. you can also call Decimal->format($options) and Decimal->formatAs($registered), but all it does is delegate to the formatter.

Don't really know why scrutinizer is failing - it just says "Task Failed" and provides no details. Am I doing something wrong?

I'll take a look this sunday , surely it's a minor defect given your previous pull requests.

I agree entirely with the dangers inherent in global state, but it can be convenient in certain situations when used with some discipline. I think it should always be a supplement to an instance-based approach rather than a replacement for it, and my examples above titled // basic usage and // customisable show that working with the instances directly is definitely a first-class citizen and the static stuff merely syntactic sugar.
The convenience is worthwhile though, at least in my eyes. The intention was to offer something halfway between the instance method (new Formatter(...))->format() and the PHP intl extension's new \NumberFormatter($locale, $style): the former can potentially be too verbose and repetitive if you have only a few specific formats application-wide (like my present use-case), the latter is way way way outside the scope of what we can reasonably provide (awesome as it is) ;)
There's really not that much separating these two methods conceptually:
// PHP's intl:
$formatter = new \NumberFormatter("en-US", \NumberFormatter::CURRENCY);

// BigNumbers:
$formatter = \Litipk\BigNumbers\Formatter::registered("USD");

Both return a formatter which is ready to go based on a statically configured dataset. The difference is that in the case of intl the data is provided by the extension, but in the case of the Bignumber formatter, it's provided by your own calls to register(...). (n.b. the Formatter::registered() method in the above example is missing from the pull request at the moment)
I'm not married to the static stuff though. I don't mind at all if you'd prefer it wasn't in there and will gladly remove it if it raises the chances that an instance-only Formatter is included, this is just a bit of explanation about why I put it in there.

hello. this is quite an interesting feature. do you plan to merge it in the near future? thanks

Hello @shabbyrobe . After all this time, I've been thinking and I definitely want to merge your pull request. The are only two points to be "fixed":

This branch should be "rebased" on the master branch to ensure a zero conflicts merge.
The last commit, the one fixing the truncation behaviour in the comp function should be sent in another pull request because it's not strictly related with the formater.

Thank you for your effort, I apologize for the excessive waiting time.


ValueError when slicing with empty masked arrays

Version of Awkward Array
1.8.0rc3
Description and code to reproduce
Please consider the following:
import awkward as ak
array = ak.Array([[5.,6.],[7.]])
masked = array.mask[[[True, True], [True]]]
empty = masked[[False, False]]

Now the following works:
array[ak.argsort(array)]
masked[ak.argsort(masked)]

But this doesn't (giving ValueError: only arrays of integers or booleans may be used as a slice)
empty[ak.argsort(empty)]

I see two inconsistencies here:

ak.argsort(masked) returns an array of values type int, while ak.argsort(empty) returns an array of values type float. In both cases it should be int.
Slicing an array that doesn't have a mask with an array of values type float works, but doing the same with a masked array raises an error.

Hi @jrueb, thanks for the report!
Could you elaborate on your last point? Namely:

Slicing an array that doesn't have a mask with an array of values type float works, but doing the same with a masked array raises an error.

I assume you've seen this during your debugging, but if I take your statement to the simplest case of a floating point index array, then I can't reproduce it.
RE ak.argsort(empty), this is an artefact of returning shallow_copy where the length is 0:
https://github.com/scikit-hep/awkward-1.0/blob/aaded1ccee9bbe24048d084a2141fe410a576ff7/src/libawkward/array/ListOffsetArray.cpp#L1920
@jpivarski this is a policy issue that I'm not clear on. We have the following behaviour for len(this) == 0 in v1 and v2, for these layouts:

ak.layout.ListOffsetArray - shallow_copy()
ak.layout.RegularArray-<Array [] type='0 * int64'>`
ak._v2.contents.ListOffsetArray - <NumpyArray dtype='int64' len='0'>[]</NumpyArray>
ak._v2.contents.RegularArray - <NumpyArray dtype='int64' len='0'>[]</NumpyArray>

So, currently only ak.layout.ListOffsetArray tries to preserve the structure of the array. Personally I think this makes the most sense - I would intuitively expect the argsort result to have the same number of dimensions as the array upon which it operates. This would mean changing v2, and v1 ak.layout.RegularArray. Do you agree here?

I assume you've seen this during your debugging, but if I take your statement to the simplest case of a floating point index array, then I can't reproduce it.
I just noticed it's a bit more specific than I thought. What I saw was that

empty2 = array[[False, False]]
empty2[ak.argsort(empty2)]

works. empty2 has value type float and so does ak.argsort(empty2). From the error message "only arrays of integers or booleans may be used as a slice" I would expect this not to work. However, looking at this now, it only happens with one dimensional empty array slices. So this might also be meant as a corner case, where it's fine not to raise an error?

I would intuitively expect the argsort result to have the same number of dimensions as the array upon which it operates. This would mean changing v2, and v1 ak.layout.RegularArray. Do you agree here?

@agoose77, you're right: ak.argsort (and ak._v2.argsort) should return an array with the same dimensions as the input, even if it's empty. Especially now that we're requiring the output Form of every operation to be independent of any buffer values or shapes, so that a Dask DAG can be examined independently of any concrete arrays. Short-circuits like
https://github.com/scikit-hep/awkward-1.0/blob/aaded1ccee9bbe24048d084a2141fe410a576ff7/src/libawkward/array/ListOffsetArray.cpp#L1912-L1921
or even
https://github.com/scikit-hep/awkward-1.0/blob/5e49baf2f20fb52a66aaa121d521932f5980cd27/src/awkward/_v2/contents/listoffsetarray.py#L1030-L1045
might result in different Forms for length 0 arrays than any other length, and when building a Dask DAG, we don't know if the length will be 0. (Even if self._offsets.length returns UnknownLength, any comparison operator on this returns false, so the DAG-building step wouldn't enter into the if statement, but the real array might.)

empty2 has value type float and so does ak.argsort(empty2). From the error message "only arrays of integers or booleans may be used as a slice" I would expect this not to work.

@jrueb, what might be confusing is the distinction between an array with zero length and an EmptyArray, which has zero length and unknown type.
>>> array = ak.Array(["This", "is", "not", "empty", "or", "unknown", "type."])
>>> array[[]]
<Array [] type='0 * string'>

>>> ak.Array([])
<Array [] type='0 * unknown'>

If you get an array of type 0 * float64 by slicing, you shouldn't be able to use that as a slicer because it has type floating point—it's unclear how you intend for it to work as a slicer.
If you have an array of type 0 * unknown, the type it gets can be determined from how it's used.
>>> ak.Array(["To", "be", "sliced."])[unknown_empty]
<Array [] type='0 * string'>

That might be why you got a different result in two cases. Be sure to check the .type!

@jpivarski I think this is the issue I was thinking of!

Thanks, @agoose77, this is the same issue as #1386, and it's fixed by the same PR #1387.
I just verified that the initial problem is gone:
>>> import awkward as ak
>>> array = ak.Array([[5.,6.],[7.]])
>>> masked = array.mask[[[True, True], [True]]]
>>> empty = masked[[False, False]]
>>> array[ak.argsort(array)]
<Array [[5, 6], [7]] type='2 * var * float64'>
>>> masked[ak.argsort(masked)]
<Array [[5, 6], [7]] type='2 * var * ?float64'>
>>> empty[ak.argsort(empty)]
<Array [] type='0 * option[var * ?float64]'>

(Also for v2.)
So I'll mark this as one of the issues that will get closed by the PR.


Revisit options.override

Now that every editor input has a editorId, I think we should revisit the options.override thing in untyped inputs and rather make it top level. Because setting the editor id should not be a optional thing but required in most cases. I.e. something like: editorService.openEditor({ editorId: 'someEditor', resource: <some URI>}).
We can still allow to select to pick an editor somehow, but I feel that override should just move out of the options.
I find this especially strange because in the IEditorInput#matches method we now have to probe onto options.override.

Yeah good point, we possibly can only do this when no-one is using typed inputs anymore for opening because otherwise you wouldn't be able to open a typed editor with another editor.

Just trying to revive or close out currently open debt items. Looks like we have a few cases of typed editor inputs being used for opening still and we should see what needs to be done to enable this case or we decide it's not worth it as the editor lifecycle is clean enough?
https://github.com/microsoft/vscode/blob/1df1845c5b0de85989676893b16b212a5908bda1/src/vs/workbench/browser/parts/editor/editorCommands.ts#L603
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/contrib/debug/browser/breakpointsView.ts#L166
https://github.com/microsoft/vscode/blob/146049f09c70cf6e6e573c1f5fdcafbb980cf907/src/vs/workbench/contrib/debug/browser/debugEditorActions.ts#L165
https://github.com/microsoft/vscode/blob/8aebc83324fd6d541a92c5bc525321c4d11079f5/src/vs/workbench/contrib/debug/common/debugModel.ts#L462
https://github.com/microsoft/vscode/blob/ec9fc48cafd47af4adcd3ce8f03b4f23dc4d9253/src/vs/workbench/contrib/extensions/browser/abstractRuntimeExtensionsEditor.ts#L489
https://github.com/microsoft/vscode/blob/ec9fc48cafd47af4adcd3ce8f03b4f23dc4d9253/src/vs/workbench/contrib/extensions/browser/extensionsWorkbenchService.ts#L872
https://github.com/microsoft/vscode/blob/4636c22e55a7d4ac49740257d72d2de060f5809a/src/vs/workbench/contrib/extensions/electron-sandbox/extensionProfileService.ts#L61
https://github.com/microsoft/vscode/blob/62de8dc3f78b8bef1eb62deeb093164a8d76c254/src/vs/workbench/contrib/extensions/electron-sandbox/extensionsAutoProfiler.ts#L199
https://github.com/microsoft/vscode/blob/1d111bf898550f33a02c0dd43f01a26428dca19f/src/vs/workbench/contrib/interactive/browser/interactive.contribution.ts#L357
https://github.com/microsoft/vscode/blob/1d111bf898550f33a02c0dd43f01a26428dca19f/src/vs/workbench/contrib/interactive/browser/interactive.contribution.ts#L395
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/contrib/output/browser/output.contribution.ts#L206
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/contrib/output/browser/output.contribution.ts#L272
https://github.com/microsoft/vscode/blob/776bd3d6d02dac6877a47772ad2bc72c4dcf37b0/src/vs/workbench/contrib/performance/browser/performance.contribution.ts#L54
https://github.com/microsoft/vscode/blob/0e8fcebfbc9f5ecb1e7db0750fc3eb5a9a8cad69/src/vs/workbench/contrib/search/browser/anythingQuickAccess.ts#L964
https://github.com/microsoft/vscode/blob/8d956244a57c2386fb1e9da57dd7b0c5a9055c71/src/vs/workbench/contrib/terminal/browser/terminalEditorService.ts#L197
https://github.com/microsoft/vscode/blob/8d956244a57c2386fb1e9da57dd7b0c5a9055c71/src/vs/workbench/contrib/terminal/browser/terminalEditorService.ts#L326
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/contrib/webviewPanel/browser/webviewWorkbenchService.ts#L231
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/contrib/webviewPanel/browser/webviewWorkbenchService.ts#L248
https://github.com/microsoft/vscode/blob/204d0247033ab7d2f6d1949ae6fd6d87e46a91b3/src/vs/workbench/contrib/welcomeGettingStarted/browser/gettingStarted.contribution.ts#L77
https://github.com/microsoft/vscode/blob/204d0247033ab7d2f6d1949ae6fd6d87e46a91b3/src/vs/workbench/contrib/welcomeGettingStarted/browser/gettingStarted.contribution.ts#L84
https://github.com/microsoft/vscode/blob/204d0247033ab7d2f6d1949ae6fd6d87e46a91b3/src/vs/workbench/contrib/welcomeGettingStarted/browser/gettingStarted.contribution.ts#L86
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/contrib/welcomePage/browser/welcomePage.ts#L105
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/contrib/welcomePage/browser/welcomePage.ts#L153
https://github.com/microsoft/vscode/blob/afb3140beb3d60393cbb70d02270e91a613e113b/src/vs/workbench/contrib/welcomeWalkthrough/browser/editor/editorWalkThrough.ts#L45
https://github.com/microsoft/vscode/blob/bf1c9e93913714c85b0db6f2e2690b0bddbab7e4/src/vs/workbench/contrib/workspace/browser/workspace.contribution.ts#L669
https://github.com/microsoft/vscode/blob/6827a04f445c9f4434760cdd7817f0c47d1e9f45/src/vs/workbench/services/history/browser/historyService.ts#L1860
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/services/preferences/browser/preferencesService.ts#L237
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/services/preferences/browser/preferencesService.ts#L319
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/services/preferences/browser/preferencesService.ts#L350
https://github.com/microsoft/vscode/blob/4a130c40ed876644ed8af2943809d08221375408/src/vs/workbench/services/preferences/browser/preferencesService.ts#L358
To say there's a lot of cases is an understatement. Some are easier to fix than others, such as ones which have the group and can just use group.openEditor or ones which we could probably have adopt the resolver just as exclusive editors for their specified id. The other benefit of this is these editors aren't benefiting from the locking which you implemented due to the dependency on the resolver service for the editor list. I still believe webviews will be the hardest to fix though as they use a group identifier number and a webview editor cannot currently be easily registered or represented as an untyped input


Can hatbitat support importing URDF models

Can hatbitat support importing URDF models

Hey @zsl13734413436, Habitat does support URDF. See the tests starting here for example code.


About the pred.py file

I want to predict , so I use the file.
Add the code for this file
 model_dir = "../mypath/checkpoint"                            vocab_file = "../mypath/vocab"                              pred = Prediction(model_dir, vocab_file)                              content = "2014年，51信用卡管家跟宜信等P2P公司合作，推出线上信贷产品“瞬时贷”，其是一种纯在线操作的信贷模式。51信用卡管家创始人孙海涛说，51目前每天放贷1000万，预计2015年，自营产品加上瞬>时贷，放贷额度将远超30亿"       print(pred.predict(content))
But it give a error ValueError: The last dimension of the inputs to Denseshould be defined. FoundNone.
So look forward to  your answer.
Thx

I guss shor of  input_y in the file, but I cant correct it.

It's my fault, After changed model structure, I didn't modify the pred.py, Now, I fixed it, please, pull code again.


Tag new release (0.16.3)

Purpose

To-do

Content

[ ] I have read and checked the items on the review checklist.

bors r+

bors r-

Let's merge this https://github.com/CliMA/ClimaAtmos.jl/pull/2319 first!

bors r+

bors r-

bors r+


Canvas keeps redirecting when canvas domain is set

When I set subdomain in canvas config, the canvas-ui works fine but the /canvas keeps redirecting to /subdomain/stats instead of  /stats.
Example if subdomain is blog.example.test then it keeps redirecting to https://blog.example.test/blog.example.test/stats

First, you should add the CANVAS_DOMAIN variable to your .env file.
Second, is your subdomain defined as a fully qualified URL? (ie. http:// or https://)

My subdomain is not fully qualified URL.

If you use a fully qualified URL in the CANVAS_DOMAIN variable, does the issue resolve?

The bug is from this line of code.
    /**
     * Return the configured public path url, prioritizing a subdomain.
     *
     * @return string
     */
    public static function basePath(): string
    {
        return config('canvas.domain') ?? '/'.config('canvas.path');
    }

I fixed it by changing tweaking it a bit.

    /**
     * Return the configured public path url, prioritizing a subdomain.
     *
     * @return string
     */
    public static function basePath(): string
    {
        return config('canvas.domain') ? '/' : '/'.config('canvas.path');
    }`

Now above fix works with the assumption that on sub domain theirs no need for config('canvas.path') and hence set to null.
But after further testing, I found out that config('canvas.domain') is not necessary in this function
    /**
     * Return the configured public path url, prioritizing a subdomain.
     *
     * @return string
     */
    public static function basePath(): string
    {
        return config('canvas.domain') ?? '/'.config('canvas.path');
    }

Therefore having this will work absolutely fine.

 public static function basePath(): string
    {
        return '/'.config('canvas.path');
    }

Also I think that the docs should reflect that config('canvas.path') can be set to null if you don't want a prefix to your canvas route.

Thanks for providing that. I'd be more than happy to review a PR if you'd like to submit one, I think it'd be a helpful change.

Alright then. I'll do that.


Issue 175 | Fix test type naming

Summary
Fix test type naming
Details and comments
Closes #175

LGTM thanks! 🚀


WebCord doesn't stop running

Aknowledgements

[X] I have checked that there's no other issue describing the same or
similar problem that I currently have, regardless if it has been
closed or open.

[X] I can confirm that this is not an issue with the Discord website,
but it is a problem specific to the WebCord itself. I have tested
if this bug occurs on Chromium/Chrome or any other Chromium-based
browser that uses unpatched/upstream Chromium engine.

[X] I have tried running the build from the master branch and it does
not have any fixes implemented according to my issue.

[ ] My issue describes one of the unstable and/or not fully implemented
features.

[x] I have found a workaround to mitigate or temporarily fix this issue
in affected releases (please write it in Additional context section
below).

Operating System / Platform
🐧 Linux
Operating system architecture
x64 (64-bit Intel/AMD)
Electron version
v19.0.5
Application version
v.3.3.0 (AUR git)
Bug description
Hi! Since the latest commits I cannot make the application stop running. Pressing Ctrl + Q or clicking to "Quit" from the tray icon or through the menu itself just closes the app to the tray.
I have to terminate the processes from the System Monitor in order to stop it running.
Additional context
No response

Sorry, I just realized that my AUR helper wasn't catching the latest commits where it was resolved. Closing the issue!


chore(demo): use prefixed $modalInstance

Never realized the demo used the modal too - LGTM

easy to skip because that is only triggered for the custom build or download modal.


Feature in "View PDF" for drawing tool

Hi, can you add a rubber for delete the writings?
But instead of deleting all, just a part (similar to Samsung notes)
Thanks

For future reference: Please remember to provide a more descriptive title since it's not really possible to understand what the issue is about from the title alone.
Furthermore, please actually fill out the ISSUE_TEMPLATE properly (rather than ignoring/deleting it) such that all relevant information is available.

Yes, please provide all requested information and elaborate a bit more on what kind of feature you mean, because from the existing text I can't infer what kind of feature you're specifically asking for. Do you perhaps have screenshots of what Samsung Notes apparently offers?

Sorry
I want to add to the drawing button another option like a rubber for deleting the writing notes if i commit a mistake

As you can see in the image, the re are 3 options
1 -> pen
2 -> highlighter
3 -> rubber
for examen i draw something in a normal note

and if i choose the rubber, i can delete only one part

sorry because i did with my phone and maybe is not the best option


non header pin loction

Hi,
I believe the location of the non-header pins relative to pin 1 of the 40-pin header is incorrect.
These additional headers are on the exact opposite side of the zero.
(just looking at my zero and the test pads look mirrored)
Cheers

Thank you for confirming - did not occur to me they were mirrored. But now
that you mentioned it: Yeah, that explains it.
On Thu, Dec 20, 2018 at 7:28 PM offtherails2010 & L.O.G (Lazy Old Geek !)
Collaborations<EMAIL_ADDRESS>wrote:

FACEPALM !
I thought it was my head playing games with me, after checking this
against my pi zero over a hundred darn times all Test-Pads are indeed on
the WRONG side (Mirrored)
But apart from that they are 100% correct
WARNING; ALL test pads need to be mirrored before use
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vitalibr/rpi-zero-eagle/issues/2#issuecomment-449109889,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMdGoRU8eFapiRqtwGF8hc9XX4D4xt5Eks5u6-TWgaJpZM4X99C5
.

Apologies, i just also realised that this was for the Raspberry Pi Zero version 1.2
I hope you dont mind that ive modified your library-part to be the Version 1.3 Pi-Zero - You're most welcome to have a copy too, im not sure how to upload it to you though ???
Ive included the PP-Test-Point-Pads that will ONLY Show on the bottom of the Pi-Zero-v1.3 as pads not plated-through-hole (PTH) as im making up a USB-Hub for the Pi, a humble 4-port and 7-port powered hubs that will supply enough power to the pi, USB peripherals and anything else connected to the USB-Powered-Hubs through-which i will be including some extra features like being able to switch off the voltage regulator that supplies power to the hub and also maybe wifi in a future version
But for now, just making two powered USB Hubs for stand-alone or raspberry-pi projects.


Tempus 460

This PR consists code of following issues:-

Tempus-460 (Create Metadata sidebar and homescreen entry)
Tempus-461 (Create Metadata UI service screen)

Merged the two issues as both are dependent.

LGTM


[Request-for-external] Table 10750 "SII History" - CreateNewRequest

Can you remove [Scope('OnPrem')] from procedure CreateNewRequest in table 10750 "SII History"?

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.


Admin console - User list is not showing users details

Admin console
Click on user list card and showing as Something went wrong while fetching users

Re-tested the defect is resolved - now user list shows list of user name and user role

Closing this defect as it has been test validated successfully


Base for unit tests for the network service

Added the base for unit tests for the network service

@ckhatri did you get round to taking a look at @j-a-m-l's suggestions? 😄

Closing due to lack of a response.


Use with mongodb database

Can I use this with mongoDB or alternatively use my existing SQLite database to convert to mondodB?

Typo I mean mongoDB

This doesn't interact with DBs on its own at all. It just helps you define a RESTful API.

Are you having any issues using mongodb with flask-restful? I was able to use it with pymongo

Hi @ueg1990, Could you give example of usage? Is it just flask with pymongo? Because I also see mongo engine Thanks

here is a link to where i used pymongo with flask-restful. It may not be that easy to follow...let me know if i can help u in anyway: https://github.com/wearhacks/hackathon_hotline/blob/master/hotline/db/db_mongo.py


Enable building with cargo profiles

Hello, thank you for processing my recent PR's!
so my previous PR only supports --profile with --no-build. this PR enables to actually build the profile passed on to the CLI options.
e.g. you can do for instance 'cargo deb -p my_package --manifest-path applications/my_app/Cargo.toml --profile RelWithDebInfo --no-strip'
The other guy from the issue list will probably be happy to get it as well.
PS; you probably already know, but there is quite some technical debt in the build flags support.. i shouldnt really need to explicitly pass --no-strip if my cargo profile states that i want debug symbols

Thanks


Provide Diamond with --jobs, not "--processes"

Description
When investigating the inconsistent --processes tag, I realised that run_diamond.sh was not written in a robust way, and "processes" is not a good description of its function.
Specifically, in this command (pulled out the parts not relevant to parallelization):
ls ${DB_PATH}/nr*.dmnd | parallel -j ${PROCESSES}
diamond blastx --quiet -d ${DB_PATH}/nr.{%}.dmnd \
--threads ${THREADS} -o ${OUTPUT}.{%}.tsv

PROCESSES is not really "processes", it's the number of jobs we tell parallel to run at once
we use the {%} replacement string, which is the job number, to decide which database we use in --d... I think they way we've set this up we need to literally always be counting from 1-6. (which is what screen.py was doing, but not run_pipeline.sh). Based on some local testing of parallel, if we were running with only 2 processes, then the outputs would get (I think? not sure what diamond does with pre-existing output files) overwritten

I have changed this to set the number of jobs to run in parallel based on the threads and CPUs (unless a number is supplied) and made some other parts of run_diamond a bit more robust, then propagated those changes.
Issues:
Addresses #10, though we'll need to change the wiki once this is released
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected) (well, it's a breaking change to run_pipeline, but not to commec screen, which is the preferred interface)

Thanks for raising that, I hadn't caught the possible weird problem with integer division. I think (more or less by accident) this will be fine in this case, because GNU parallel interprets --jobs 0 as "run as many jobs in parallel as possible" according to the docs.
In the case where threads is 1, we aren't concerned about needing to manage the number of jobs because we're not multithreeading, so -j 0 --use-cpus-instead-of-cores is the behaviour you want.


🥽 Add visionOS platform support

This PR adds support for the visionOS platform.
This PR is stacked on top of #114 and #95 and will not be merged until those PRs land and official support is launched for visionOS (in 4.9?).
Uses 4.8.0-visionos-beta.1 releases of both DittoSwiftPackage and DittoSwiftTools

Obviated by #124


Tasks not chaining properly

I have a BFTask that is returning false to [ob isKindOfClass:[BFTask class]] and yet returns @"BFTask" for NSStringFromClass([obj class]).
Here is a sample project: git@github.com:krusek/Bolts-error.git

https://github.com/krusek/Bolts-error

Closing this one as a duplicate of #111
Let's continue discussion there.


Don't attempt sync when a machine is not being used

Consider a user whose computer does not sleep (or does not for a long time) where VS Code is opened. I think right now regardless of focus or inactivity polling will still occur.

Actioning this could also help with this scenario:
The user has 2 machines opened, installs C# on one, waits 10 minutes and then marks to not sync. Ideally the second machine would never have installed the C# extension.

Not sure if it is intended but I like my machine gets synced automatically. If user does not want that then he/she should pause the sync.
I think this was brought up because of the scenario - user wants to try out an extension which is getting installed on all machines automatically. IMO it is a different issue than this which has to solved differently.

Closing it as this might not be relevant anymore. Please try out latest state and file a new issue if needed.


validate representation of CrunchBase IPOs

Could you please validate this representation of CrunchBase IPOs in FIBO?
The simplest representation is 1 node, this one in FIBO is 25 nodes:

I made a gist that describes the fields and shows the turtle model.
Thanks in advance! We at Ontotext plan to write a blog post about this experience

Please find below some quick comments from me - @ElisaKendall may give you a more comprehensive review:

I understand that the current ttl is a kind of stub to be filled with the actual data, e.g., '(money_raised)'^^xsd:decimal will be replaced with the actual value - currently these stubs confuse Protege when it saves the file.
I would make <cb/agent/(org_uuid)/issuer> an instance of Offeror (as well as of Issuer).
<cb/exchange/(stock_exchange_symbol)> as an instance of both Exchange and RegistrationAuthority looks wrong given their definitions.
4.Strictly speaking, the instances of fibo-fnd-acc-cur:Currency, e.g., <cb/currency/(share_price_currency_code)>, introduce potentially new currencies. I do not think you need them - in the proper version of the ttl file you should reuse the current instances thereof.
We recommend using unique IRI fragments even across naming spaces (see: https://github.com/edmcouncil/fibo/blob/master/ONTOLOGY_GUIDE.md#general-naming-and-labeling-conventions) your instances of Currency do not follow this recommendation.
We also recommend having rdfs:labels for all resources - some of yours do not have them, e.g., <cb/exchange/(stock_exchange_symbol)/code>.

Finally, could you publish in the gist some examples of the actual IPO data?

Thanks @mereolog !

1: Stub used by my rdf2rml tool to generate transformations (R2RML, TARQL CONSTRUCT, OntoRefine UPDATE).
I understand Protege may get confused because the parenthesized string is not a valid integer, but I hope it can print a warning and still load the data?
2: added fibo-fbc-pas-fpas:Offeror. But then what else should I add to satisfy this?

         rdfs:subClassOf  [ rdf:type            owl:Restriction ;
                           owl:onProperty      fibo-fnd-pty-rl:isPlayedBy ;
                           owl:someValuesFrom  [ rdf:type            owl:Restriction ;
                                                 owl:onProperty      fibo-fnd-pty-pty:isAPartyTo ;
                                                 owl:someValuesFrom  fibo-fbc-pas-fpas:Offering]]

3: I now added <cb/ipo/(uuid)/ticker> a fibo-sec-sec-id:ListedSecurityIdentifier: do you agree? Then there is:

fibo-sec-sec-id:ListedSecurityIdentifier
        rdfs:subClassOf  [ rdf:type            owl:Restriction ;
                           owl:onProperty      fibo-fbc-fct-ra:isRegisteredBy ;
                           owl:someValuesFrom  fibo-fbc-fct-mkt:Exchange] ;

The range of fibo-fbc-fct-ra:isRegisteredBy is union ( fibo-fbc-fct-ra:RegistrationAuthority fibo-fbc-fct-ra:Registrar ): which of these 2 classes you think is more appropriate?
4: Yes, it says there Better reconcile to fibo-fnd-acc-4217:ISO4217-CodeSet. But for this to happen:

I must check that CB currency codes are all from ISO 4271
I must use an UPDATE to link eg "USD" to <ISO4217-CurrencyCodes/USDollar>, cannot use a CSV->RDF transform.

5: do you mean that local names of individuals should be unique? That's not very practical for source data that includes only codes but not names (as are CB currencies)

It would be more data-engineer friendly if the Currency individual URLs use codes instead of names, just like CurrencyIdentifier individuals do

6: ok, added
7: added ipos-sample.csv

Updated the model image and the gist:

8 (TODO): Currently I use monetary nodes in "national currency" vs in USD with URLs like this:

<cb/ipo/(uuid)/pricePerShare> vs <cb/ipo/(uuid)/pricePerShareUsd>

But many IPOs are of US companies, so we can save on nodes if we use URLs like this.
The difference is subtle but crucial: it will effectively merge the nodes:

<cb/ipo/(uuid)/pricePerShare/(share_price_currency_code)> vs <cb/ipo/(uuid)/pricePerShare/USD>

As for:

2: added fibo-fbc-pas-fpas:Offeror. But then what else should I add to satisfy this?

         rdfs:subClassOf  [ rdf:type            owl:Restriction ;
                           owl:onProperty      fibo-fnd-pty-rl:isPlayedBy ;
                           owl:someValuesFrom  [ rdf:type            owl:Restriction ;
                                                 owl:onProperty      fibo-fnd-pty-pty:isAPartyTo ;
                                                 owl:someValuesFrom  fibo-fbc-pas-fpas:Offering]]

These two triples should do the trick:
<cb/agent/(org_uuid)/issuer> fibo-fnd-pty-rl:isPlayedBy <cb/agent/(org_uuid)>.
<cb/agent/(org_uuid)> fibo-fnd-pty-pty:isAPartyTo <cb/ipo/(uuid)/offering>

As for the rest of your questions, you should contact get in touch with @ElisaKendall .

@VladimirAlexiev
For 3, for ticker symbol, your individuals should be of type ListedSecurityIdentifier and TickerSymbol (both are in the SecuritiesIdentification ontology). Ticker symbols are reassignable, and are specific to the exchange, though.
Related to that, we don't have a reference ontology with all of the ticker symbols for any exchange (since there are over 7K exchanges), but we do have the latest ISO MIC codes as of the end of June (Q2 2022) - these are either operating level or market segment level codes.  The class, MarketIdentifier captures both cases, but the codes themselves are in the FBC/FunctionalEntities/MarketsIndividuals ontology and they require the BusinessCentersIndividuals ontology (which says where they are located per whatever they reported to ISO). The only thing is that you need to know whether it is an operating level or segment level market.  For IPOs in the US, for issuance of shares of stock, it is typically either the NYSE (Exchange - fibo-fbc-fct-mkti;Exchange-XNYS, MIC - fibo-fbc-fct-mkti;MIC-XNYS), or the Nasdaq (Exchange - fibo-fbc-fct-mkti;Exchange-XNAS, MIC - fibo-fbc-fct-mkti;MIC-XNAS).
A given IPO will be in a specific market, such as the NYSE or Nasdaq in the US, so for the Crunchbase IPO you should be able to get all of those details from the exchange individual, e.g., fibo-fbc-fct-mkti;Exchange-XNAS, which will even tell you which municipality or in some cases FpML business center that the exchange is located in.
For isRegisteredBy, the difference is whether it is registered by the official RA or by someone that they have delegated that role to. Typically for a security, the exchange is the registration authority. So in the individual you create for the exchange, add a triple saying that it is the registration authority and you are done.
So for currency codes, what about using a property chain for hasCurrency --> isIdentifiedBy?  You could then use the resulting code rather than the name for matching?  We did that at Wells Fargo for a similar case, no problem.
One other thing - you may be able to map at least some of the details to openFIGI (https://www.openfigi.com/) if you have the FIGI for the security. At least some of their symbology data is publicly available.  And, some of the data related to the legal entity should be mappable to Open Corporates (https://opencorporates.com/companies?jurisdiction_code=&q=Crunchbase&utf8=✓), likely the Delaware corporation is the one doing the IPO, but you would have to confirm through the offering statement.
Hope this helps - I'll keep checking back, and thanks @mereolog  for your help with the other questions!
Elisa

Hi @ElisaKendall, thanks for the feedback!

3a: fibo-sec-sec-id:ListedSecurityIdentifier wants fibo-fbc-fct-ra:isRegisteredBy some fibo-fbc-fct-mkt:Exchange.

fibo-fbc-fct-ra:isRegisteredBy has range union ( fibo-fbc-fct-ra:RegistrationAuthority fibo-fbc-fct-ra:Registrar )
fibo-fbc-fct-mkt:Exchange is a subclass of neither.
So as you said, I'll add a second type fibo-fbc-fct-ra:RegistrationAuthority

9: "there are over 7K exchanges... have the latest ISO MIC codes.. operating level or market segment level codes":
I know, I added https://www.wikidata.org/wiki/Property:P7534 and matched all 1947 values (as of Apr 2020) to WD, see https://www.wikidata.org/wiki/Property_talk:P7534. Is it really 7k values as of today? I sincerely hope not :-)
10: "for the Crunchbase IPO you should be able to get all of those details from the exchange individual".

Unfortunately CB Exchanges use ambiguous codes, eg 'mse' could be Metropolitan or Mongolian. But we added them to WD so now we can use WD queries to match between MIC and CB exchanges. Out of 164 CB exchange codes:
1 pair uses the same code: https://w.wiki/5aYS
8 have two codes: https://w.wiki/5aYP (which is not a crime)
18 are not in MIC: https://w.wiki/5aYM (I reduced this to 13)

8: done
11: "what about using a property chain for hasCurrency --> isIdentifiedBy":
How will I express this in RDF? Prop chains can be used as a shortcut to navigate quickly through nodes. But I need those nodes when making RDF?
12: OpenFigi: well, a search for Tencent Holdings returns 18k securities, which use different (not MIC) exchange codes, so I don't think we'll  go there just yet.

Do you see any problems on the relations, or I got them right?

BTW this mapping was done on request by @peio.
His comment: "Both impressive and scary" ;-)

@VladimirAlexiev
For 9 - yes, over 7K individuals, but I think it's really half that for the number of exchanges because we have both the exchanges and the mic codes represented :).  The reference data is up to date as of June 2022 now.  Currency codes too - we updated those recently.
There are example property chains in the FND/Parties/Parties ontology for a bunch of relationships that we use for linking counterparties up.  If you follow the pattern there, you should end up with something like
<owl:ObjectProperty rdf:about="<your prefix>hasCurrencyCode">
	<rdfs:label>has currency code</rdfs:label>
	<owl:propertyChainAxiom rdf:parseType="Collection">
		<rdf:Description rdf:about="&fibo-fnd-acc-cur;hasCurrency">
		</rdf:Description>
		<rdf:Description rdf:about="&lcc-lr;iisIentifiedBy">
		</rdf:Description>
	</owl:propertyChainAxiom>
	<skos:definition>relates something to the relevant currency code for the currency in question</skos:definition>
</owl:ObjectProperty>

We have all of the currencies and currency codes already in FIBO, so you may not need any additional nodes to do this. They are in the ISO 4217 ontology, but I'm assuming that you already knew that.
This is getting much closer to what I would expect.  We have some examples in SEC/Equities/EquitiesExampleIndividuals that provide some examples for alphabet and apple shares that might be useful for comparison if you haven't already looked there. They include properties for apple stock on multiple exchanges, and include that the listing is in USD so maybe those examples can provide some additional ideas.  Let me know if you have other questions, and good luck!

@ElisaKendall @mereolog

11: I don't think I want to use custom:hasCurrencyCode in my data. I want to use the standard property hasCurrency
13: I know about fibo-fnd-acc-4217:ISO4217-CodeSet but it's a bit hard to use: https://github.com/edmcouncil/fibo/issues/1816
I've looked at some of the examples (Apple). I think I missed a few dates, will add them now

Cheers!


migration: introduce primitives for below-raft migrations

PR: https://github.com/cockroachdb/cockroach/pull/58088
From release notes:

Cluster version upgrades, as initiated by SET CLUSTER SETTING version = <major>-<minor>, now perform internal maintenance duties that will delay how long it takes for the command to complete. The delay is proportional to the amount of data currently stored in the cluster. The cluster will also experience a small amount of additional load during this period while the upgrade is being finalized. The ideas here follow from our original prototype in #57445. [#58088][#58088] {% comment %}doc{% endcomment %}

@taroface, I'll take this since I'm in the process of updating the v21.1 upgrade docs.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #516 (a1255c0) into main (d3038af) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #516   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


does Json sdk cares about nfc nfd utf8 issue?

used case:
json composed on windows by json parser and contains ü character
same json file parsed on mac
In mac there is DFD and in windows - DFC utf8 encoding for letters like ü etc..
Is json toolkit cares about it?
Thanks

This library supports UTF-8.


Rename LLMonitor Callback Handler

Following #14574, I've added LunaryCallbackHandler in langchain_community and kept LLMonitorCallbackHandler as an alias, in order to ensure backward compability.
Thanks!
@hwchase17

Hi @hwchase17, could you please check and confirm if the pull request can now be merged?

Hi @baskaryan pinging you also to see if there is something else to do to get this merged :) Thanks

Hi @hwchase17, I've made the changes you've suggested

hmm seems like some merge conflicts that need to be resolve via command line?

Hi @hwchase17, I've resolved the conflicts.

Changes a bunch of unrelated files (including CI) - feel free to clean up and reopen!


Artefact, module and folder names update

As per email thread today with @vassyz and @qzagarese as well as others;

Update artefact names (increment version number for next maven release).
Rename maven modules, folders and references.

@qzagarese merged, release when appropriate/convenient for you.

@davidgoate @vassyz deployed new version with updated naming convention to sonatype: https://oss.sonatype.org/#nexus-search;quick~yoti
Should synchronise to central by the end of the day.
Thank you David!


feat(tabs)!: migrate tokens

Description
This PR updates the Tabs component to use the new tokens package.
It also adds additional stories to Storybook. Since the components do not currently have an overarching parent directory it moved the Tabs to the bottom, this should be changed in a separate PR. But this one does contain a few changes to the base styling for Storybook to allow for nested stories.
Issues Addressed

#1526

To-do list

[x] If my change impacts other components, I have tested to make sure they don't break.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have tested these changes in Windows High Contrast mode.
[x] I have updated any relevant storybook stories and templates.
[x] If my change(s) include visual change(s), a designer has reviewed and approved those changes.

[x] This pull request is ready to merge.

@jawinn Yeah my thought was that we could do the Storybook organization in another PR so I can get that up! I'll want Patrick to check that to make sure that it won't totally break Chromatic.
And I was torn on the amount of stories as well. I felt like they were different enough that since we're running VRTs, we needed to have a visual representation of each of the ways represented in the docs so that they were all tested for regressions... it's a bit much but I didn't want to give up on testing Compact, Emphasized and Quiet with Chromatic because there were too many options. @pfulton what do you think?

Released a new beta<EMAIL_ADDRESS>

Published a new beta<EMAIL_ADDRESS>


chore: update marksman to version 2022-12-28

https://github.com/artempyanykh/marksman/releases/tag/2022-12-28

Needs to merge with main.


progwrp TLS features

How do I use progrwrp's TLS features.
I see exported:

_TLSInit_DllMain_ProcessAttach@4
_TLSInit_DllMain_ThreadAttach@4

well you need to link it via its .lib file. And use look at my patch for Windows XP that show where he is attaching the dll to processes by searching through this file for "TLSInit_DllMain_ThreadAttach" and "TLSInit_DllMain_ProcessAttach" > https://github.com/Alex313031/thorium-legacy/blob/main/patches/winxp-vista-support_thorium.patch

So if I understand correctly, the functions must be called from the context of the target dll's dllmain, with the HMODULE as sole parameter.
So this means I have to build dll with the lib
OR
hook dllmain before it happens.
Alternative to this is baking a dummy IAT import of dummy dll into application.

I recently decided to update the method of calling the TLS code to support third-party proprietary DLLs. This means that the exported TLS functions are no longer used, and instead, calls to progwrp's LoadLibrary functions will invoke the TLS process attach function, as well as InitializeCriticalSectionEx calls in DllMainCRTStartup as they are only made in DLL_PROCESS_ATTACH.
FlsGetValue calls are always done in DLL_THREAD_ATTACH in DllMainCRTStartup so TLS thread attach is done there, as well as in progwrp's DllMain when a new thread is created.
The TLS presence in InitializeCriticalSectionEx and FlsGetValue is there to safeguard against TLS accesses in new threads that occur before even reaching the "real" DllMain. If the CRT is not used and it's certain that no TLS accesses are made until DLL loading is complete, only LoadLibrary will have to be called from progwrp and everything else should remain the same. Otherwise, progwrp should be linked against the loaded DLL.

Ah, so I'm patching the IAT of the program that dynloads the offending library, yes?

Yes, to bring in progwrp's LoadLibrary*, yes.

@pachuco @win32ss OK three things.

What's IAT mean.
What's TLS mean? I could not find anything related to it outside of its TCP/IP meaning. What is "implicit tls initalization" versus I'm assuming the opposite that non NT 5.x oses use?
Why do the (now old), main attachment functions have an underscore and @4 at the end of them?

 I have more Linux expertise and its a miracle I am even able to make the Windows programs that I do lol. Most of my coding in Thorium has been for higher level stuff like controlling features, disabling annoying stuff, and adding flags, rather than this lower level win32 API stuff. Need to read up more so I can become a better Windows nerd.
Interesting to see the inner workings of library usage in Windows and how it differs from the way application-level shared libraries (i.e. .so files) are used and can be imported on Linux.

1. What's IAT mean.

2. What's TLS mean? I could not find anything related to it outside of its TCP/IP meaning. What is "implicit tls initalization" versus I'm assuming the opposite that non NT 5.x oses use?

Before understanding IAT, we need to know what a pe file is and what a dll file is:
The full name of the PE file is Portable Executable,Common EXE, DLL, OCX, SYS, and COM are all PE files. PE files are program files on the Microsoft Windows operating system (may be executed indirectly, such as DLL).
DLL (Dynamic Link Library) file is a dynamic link library file, also known as "application extension", which is a software file type. In Windows, many applications are not a complete executable file. They are divided into some relatively independent dynamic link libraries, that is, DLL files, which are placed in the system. When we execute a program, the corresponding DLL file will be called. One application can use multiple DLL files, and one DLL file may also be used by different applications. Such DLL files are called shared DLL files.
IAT (Import Address Table) Since imported functions are functions that are called by the program but whose execution code is not in the program, the codes of these functions are located in one or more DLLs. When the PE file is loaded into the memory, the Windows loader Load the DLL and associate the instruction to call the imported function with the actual address of the function (dynamic connection). This operation requires the import table to be completed. The import address table indicates the actual address of the function.
Thread Local Storage

.dll is like an .so library on linux(but PE/COFF instead of ELF).
IAT is section in PE that tells OS loader what functions the .exe/.dll needs from other .dlls, the Imports.
you also have an EAT to tell OS loader what function our .dll puts out, the Exports.
besides EAT/IAT stuff, a .dll can call LoadLibraryA/W() or related functions(equiv to dlopen() on linux). This is where TLS fails on windows XP.
TLS can be done in two ways in windows. Implicitly via information in the PE header, or explicitly via some winapi functions. Implicit TLS is what fails with LoadLibrary() on windows XP.
_ and @4 stuff is name mangling relating to calling convention of functions(I think CDECL). the @4 tells how much space on stack all params take, in bytes(a single parameter, pointer, 4 bytes on x86).

Heave read this. And want to say some adjustment.
@num is __stdcall convention.
Moreover -- all the WinAPI for x86 is __stdcall by default.
But why API functions has no such names in DLL funcs ? because they use .DEF file (or separate linker directives), which renames exported functions to be more readable and to be cross-programming language compatible.
For both functions TLSInit_DllMain_Process/ThreadAttach this techinque jush was not used, so names are given as they generated by compiler.


[Bug] multiccd option not a valid attribute of option

Tried multiccd for fun and got an error on both the MuJoCo app (2.1.5) and PyMJCF from dm_control:
AttributeError: Line 12: error while parsing element <option>: 'multiccd' is not a valid attribute for <option>. # PyMJCF.
XML Error: Schema violation: unrecognized attribute: 'multiccd'. # App.

Apologies, I realize this should have been under the flag attribute.
<option>
  <flag multiccd="enable"/>
</option>

Actually, while we're here, there is a bug (fixed internally, to be pushed soon): The multiCCD disable/enable button in simulate does not work.