all: builders and TryBots should check documentation for broken links

We occasionally end up with broken or outdated links in documentation (see https://github.com/golang/go/issues/37042#issuecomment-582448476, #27860, #21951, #19244).
Testing for broken links would not be entirely trivial, but nonetheless pretty straightforward to implement: we have an HTML parser in golang.org/x/net/html, and it is easy enough to issue a HEAD request for link targets to see if they resolve. (For links with anchors, we would probably want to also check the Content-Type header and then parse the actual linked HTML to ensure that the anchor exists.)
CC @golang/osp-team

Thanks for opening an issue about this. I agree it's not entirely trivial, but if implemented in a way that doesn't cause too much noise or false positives, I also agree that this would be helpful.
I believe @jayconrod wanted this too. I can't find an existing issue; it may have been a comment in a CL.
Edit: Found it, it was this comment in a CL.

Thinking about this some more: we have control over golang.org, go.dev, and github.com/golang URLs, but not over other URLs. Probably the TryBot test should only fail if the broken link is on one of those sites, but at least the longtest builders should also check external links.

Hello, I'm newbee in goland dev but want to dive in the core code.
I have experience in developing code but not in go.
I would have a look into this issue (as a use case) but my investigations could take time.
Is my contribution acceptable ?

@alain91, the tricky part of this issue will likely be getting the rendered documentation (in order to obtain the links it contains), and indexing that documentation (in order to be able to detect broken relative links). I'm not familiar with that part of the codebase myself, so I don't know how accessible it would be to a newcomer.

I'm only newcomer in go. I have experience with parser, html decoder...

@bcmills I took a crack at implementing something like this. This just walks the filepath, generates documentation html, then calls HEAD on any tag values with an href key. Running against the src directory outputs:
$ godoc_link_validator -check '^golang\.org|go\.dev|github.com/golang$' ./src
2020/03/03 15:05:59 src/cmd/go/internal/lockedfile: http HEAD 'https://golang.org/issue/20803.' status code: 404

Without ignoring external links (just ignoring localhost and example URLs) I get:
$ godoc_link_validator -ignore '^localhost|example|code' ./src
2020/03/03 15:08:21 src/cmd/go: error retrieving 'https://proxy.golang.org,direct': Head "https://proxy.golang.org,direct": dial tcp: lookup proxy.golang.org,direct: no such host
src/cmd/go/internal/lockedfile: http HEAD 'https://golang.org/issue/20803.' status code: 404
src/cmd/internal/obj/arm: http HEAD 'http://infocenter.arm.com/help/topic/com.arm.doc.ihi0040b/IHI0040B_aadwarf.pdf' status code: 404
src/cmd/internal/obj/arm64: http HEAD 'http://infocenter.arm.com/help/topic/com.arm.doc.ecm0665627/abi_sve_aadwarf_100985_0000_00_en.pdf' status code: 404

That proxy link represents the default GOPROXY value (found in docs here), and I'm not sure if its better to just ignore it or to update the doc generator to not include the ",direct" portion in the clickable link.
That issue link (found in docs here) returns a 404 because of a trailing period (created an issue to track this).
Both of the ARM links are actually broken (here and here in docs).
I am not sure if/how this should be included in the trybots since I am not familiar with how exactly they are configured.

I sent go.dev/cl/548059 to check all links inside the go.dev server.
(That is, not checking external links.)
That covers all the issue @bcmills linked in the original report except for checking #fragment.
I think external links are and should be beyond the scope of builders and TryBots.
I will leave this open to track potentially checking #fragment too.


yoga-wasm-web versus webpack

hello there, thanks for the great module. we are trying to integrate this into a webpack build. webpack isn't particularly happy with this module's use of import.meta.url -- it interprets this to be the location at build time, which, for webpack, is not going to be the same as the location at runtime.
have you given any thoughts to this problem? we tried a few things, such as import("./yoga.wasm"), and thus relying on webpack 5's direct support for webpack modules. but this kind of change will require some deeper modifications...
thanks!

update: this is what i finally got working:

use the encoded-uint8array-loader via a webpack rule, but just specifically for yoga.wasm (because other wasm uses in our application work fine with normal webpack 5 schemes)

use a hacked version of node.js that does the following:

import initYoga from "./index.js";
import * as bytes from "./yoga.wasm";
const Yoga = await initYoga(bytes.buffer);
export * from "./yoga/javascript/src_js/generated/YGEnums.js";
export default Yoga;


Ensure proper cleanup of dangling resources in case of failure

In AWS EC2 driver we start creating resources that simply get lost in case
of failure during instance creation.
The worst case scenario is when a SpotInstance price is too low to be fulfilled
before the timeout.
In such case we already created a Key Pair, but we are not going to persist
config.json due to the failure, and we will never be able to delete such key.
Moreover the SpotInstanceRequest will remain in AWS and if it will be fulfilled
later a new instance will be created outside of docker-machine control.
This patch ensures that a proper cleanup is done in case of failure in
amazonec2.Create()
Furthermore docker-machine rm is now able to handle properly AWS machines
failed during creation thanks to this two new conditions:

Key Pair deletion will not fail in case of missing KeyPairName
Instance termination will not fail in case of missing InstanceId

Closes #3555

When will this be merged?

ping @shin-


Feature for situation management

Short Description
Enables to use the situation api of a opentosca runtime within the ui
Proposed Changes

adds a situation tab that enables the following:

creating situations

creating (event) triggers on the situations by combining situations with plans

manual triggering of situations

How to use the view to configure different triggers which can either be started by users or external system via emitting an event:
Add a new situation:

Add a new trigger:

Trigger the change of the previously added situation:

The spacing between the buttons do not seem to fit


Consider limiting the cookies we use for nonce and correlationId to the paths that we use them on

Currently the handle of /, if there are incomplete logins they cause trouble and they stay on the browser cookie cache incrementing the size of each request. We should look into setting their path to the sign-in callback endpoint so that incomplete login flows don't increase the size of requests sent to the server.

@javiercn any chance you want to send a PR for this? @Tratcher believes the code change is quite simple, but it requires a bit of testing.

Yeah, I will talk to @Tratcher and look into it next week.


Support for Amazon schemas

See highsource/jsonix#59.

@brutallyhonest I'm closing the support issue as well then.


Add Perl 5 implementation

Hello Emily,
I've written a Perl 5 implementation, which is similar to the Java and Ruby implementations. The tests require version 0.96 of the Test::More module, which is distributed with Perl 5.14 and better.
Kind regards,
Andreas

@voegelas thank you for this. We had a lot fun on yesterday's Barcelona Perl Mongers meetup about refactoring Perl code.

Thankyou!

Thankyou!


Add basic class and function definition for Millicast SDK

Since millicast sdk still doesn't support typescript, so we need to add the type definitions

[x] Director
[x] Publisher
[x] RTCPeerConnection
[x] Logger

Use this gist for reference https://gist.github.com/dubeyShivank/50b707a6adaa69683b56cd0ebd46ed71

please add the list of the API & Types you need for your other tasks in this issue

@dubeyShivank  please check the code, we can add types anytime.


Morning session: Peer-to-peer code review Module 2 - Week 3

https://github.com/Elerqsousy/to-do-list/blob/main/src/modules/object-class.js#L5
line not necessary

https://github.com/Elerqsousy/to-do-list/blob/main/src/modules/functionality.js
Class for data is not necessary since there is only one data object for each app


--ip_forward doesn't enable IP forwarding for IPv6

It should set /proc/sys/net/ipv6/conf/all/forwarding too

@tomdee It actually enabling it, but only with --fixed-cidr-v6


License updates for cleanup-ruby-usage

This PR was auto generated by the licensed-ci GitHub Action.
It contains updates to cached github/licensed dependency metadata to be merged into cleanup-ruby-usage: (branch, PR).
The licensed-ci action will add comments to this PR with the status of license checks.  Please make any changes needed to fix any status failures on this branch, then merge license updates into your branch.
If updates are unexpected, please check the github/licensed changelog for recent updates.

The licensed-ci GitHub Action updated cached license metadata based on changes to cleanup-ruby-usage: (branch, PR).
licensed status result: FAILED
There are reported status failures, please fix all logged errors before merging.
Checking cached dependency records for licensed
....F....F.F......................
Errors:
* licensed.bundler.concurrent-ruby
  version: 1.1.10, filename: /home/runner/work/licensed/licensed/.licenses/bundler/concurrent-ruby.dep.yml, license: other, allowed: false
    - license needs review: other

* licensed.bundler.ffi
  version: 1.15.5, filename: /home/runner/work/licensed/licensed/.licenses/bundler/ffi.dep.yml, license: other, allowed: false
    - license needs review: other

* licensed.bundler.httpclient
  version: 2.8.3, filename: /home/runner/work/licensed/licensed/.licenses/bundler/httpclient.dep.yml, license: other, allowed: false
    - license needs review: other

34 dependencies checked, 3 errors found.

Licensed found errors during source enumeration.  Please see https://github.com/github/licensed/tree/master/docs/commands/status.md#status-errors-and-resolutions for possible resolutions.

/cc @jonabc


Publish browser agent as npm package

Summary
The current mechanism of using the NewRelic browser agent is to embed the code snippet - shown in the "Configure the agent" section in the README.md file - directly to the website.
An additional option might be to allow website developers to integrate the browser agent by using npm packages. This mechanism has several advantages, e.g. developers can decide when to update and test a new version of the browser agent. Furthermore, type-safety (when using TypeScript) could be enabled.
For the NewRelic node client, this feature is already implemented. See https://github.com/newrelic/node-newrelic/releases and https://www.npmjs.com/package/newrelic
Desired Behavior
I am able to install the NewRelic browser agent with npm.
Possible Solution
Standard npm publish / GitHub release mechanism. See https://github.com/newrelic/node-newrelic/releases and https://www.npmjs.com/package/newrelic
Additional context
none

Thank you for submitting your recommendation @maveOne! We'll need to explore this issue and consider where to prioritize it. There are some infrastructure changes we would need to make to make this happen, including switching to semver which would likely require some cascading changes to our supporting services.  That said, the values you noted are indeed evident and the main roadblocks for us are planning and prioritizing.  Thanks for the feedback and we will keep you posted.

@maveOne I know it's been a while. We have started publishing an NPM package that does basic functionality (manually capture errors) and are planning to get to feature parity soon. Have you by chance tried it out? Would love to get your feedback.


Terraform Language Support

It would be amazing if this editor had syntax highlighting for terraform

I've added this to the list in #272


Mouse misbehaving on old laptop when connected during power up

Hi, thanks for sharing this project, it's really useful! I used it to connect a modern mouse to my old Toshiba Satellite 310CDS running Windows 98.
I noticed the mouse works fine but only if I reset the microcontroller (or reconnect the PS/2 port). If I leave it connected during power-up it moves incorrectly and it randomly clicks around.
I tried flashing the 100Hz build (haven't tried the 200Hz one) but the behavior is the same.
Do you have any suggestions for troubleshooting? Thanks!

Hi, do you have a serial cable? Can you post the debug output from GPIO0?

Hi, yes!
You should be able to see where I reset the pico; after that it started working correctly.
ps2x2pico-2.1
Setting defaults for keyboard
scancodeset set to 2
kb > host aa
ms > host aa
ms > host 00
Blinking keyboard LEDs

HID(1,0,keyboard) mounted
 VID: 046d  PID: c52b
 HID has 1 reports
 HID(1,0,keyboard) registered for reports

HID(1,1,mouse) mounted
 VID: 046d  PID: c52b
 HID has 4 reports
 HID(1,1,mouse) registered for reports

HID(1,2,none) mounted
 VID: 046d  PID: c52b
 HID has 3 reports
 HID(1,2,none) registered for reports
host > ms ff
ms > host fa
ms > host aa
ms > host 00
host > ms f2
ms > host fa
ms > host 00
host > ms ff
ms > host fa
ms > host aa
ms > host 00
keyboard unknown  usage_page: ff00  usage: 02
keyboard unknown  usage_page: ff00  usage: 02
host > ms ff
ms > host fa
ms > host aa
ms > host 00
host > ms ff
ms > host fa
ms > host aa
ms > host 00
host > ms f3
ms > host fa
host > ms c8
ms > host fa
host > ms f3
ms > host fa
host > ms 64
ms > host fa
host > ms f3
ms > host fa
host > ms 50
ms > host fa
host > ms f3
ms > host fa
host > ms 0a
ms > host fa
host > ms e8
ms > host fa
host > ms 03
ms > host fa
host > ms e6
ms > host fa
host > ms f3
ms > host fa
host > ms 28
ms > host fa
host > ms f4
�
ps2x2pico-2.1
Setting defaults for keyboard
scancodeset set to 2
kb > host aa
ms > host aa
ms > host 00
host > ms f5
ms > host fa
host > ms f4
host > ms ff
ms > host fa
Blinking keyboard LEDs
ms > host aa
ms > host 00

HID(1,0,keyboard) mounted
 VID: 046d  PID: c52b
 HID has 1 reports
 HID(1,0,keyboard) registered for reports
host > ms ff
ms > host fa

HID(1,1,mouse) mounted
 VID: 046d  PID: c52b
 HID has 4 reports
 HID(1,1,mouse) registered for reports

HID(1,2,none) mounted
 VID: 046d  PID: c52b
 HID has 3 reports
 HID(1,2,none) registered for reports
ms > host aa
ms > host 00
host > ms f3
ms > host fa
host > ms c8
ms > host fa
host > ms f3
ms > host fa
host > ms 64
ms > host fa
host > ms f3
ms > host fa
host > ms 50
ms > host fa
host > ms ff
ms > host fa
ms > host aa
ms > host 00
host > ms ff
ms > host fa
ms > host aa
ms > host 00
host > ms f3
ms > host fa
host > ms 0a
ms > host fa
host > ms e8
ms > host fa
host > ms 03
ms > host fa
host > ms e6
ms > host fa
host > ms f3
ms > host fa
host > ms 28
ms > host fa
host > ms f4

Thanks! What OS and mouse driver are you running?

Thanks! I'll check that after christmas.


Load additional contrast + allow denoising

This PR addresses the request from the canproco project about the possibility of opening additional contrast within a single FSLeyes window. For example, T2w image overlaid by PSIR image.
Namely, this PR adds:

'-load-other-contrast' flag for loading an additional image to the viewer (currently, only FSLeyes is supported)
-denoise flag for denoising the input image using 'sct_maths -denoise p=1,b=2'.

Flags can be used together or separately.
The PR also contains "hacky" commit for the opening of the already existing PSIR segmentations. Once @mchen1110 finishes her work with these PSIR segmentations, I will remove the commit and merge the PR. - UPDATE: removed in https://github.com/spinalcordtoolbox/manual-correction/pull/8/commits/7da2d5684fb62354dd56cc2d40599d0d6a6e082e
Fixes https://github.com/ivadomed/canproco/issues/20

Thank you for reviewing! --> merging


Multi-tenancy recipe results in infinite open connections

I am trying to use the same recipe provided in the docs for multi-tenancy.
When each tenant has a different database I have to connect to that specific one.
But after a simple findById query the connection does not close. If I keep hitting refresh I keep seeing the connection number go up.

Has anyone got this problem?

The multi-tenancy example isn't a complete example. For example, there is no implementation for getDatabaseForRequest. Without knowing how you implemented that, there's no way for anyone to help you. Could you share the code? You need to cache the knex connections in that function so that you don't create a new one for each request.

getDatabaseForRequest returns a brand new knex object correctly configured. I've already checked a million times. Even if I cache it wouldn't it maintain always 1 connection open at all times for every request?
Basically the same problem

If you always return the same knex instance for the same tenant, then you have tenanats.length connections and not the number of requests. I don't understand your problem.


added ICLR 2022 paper

Just added a new entry

Congratulations on your accepted paper! :)

Thank you! having such a good and easy to use benchmark helped a lot :)


Convert vocabularies to OWL ontologies

In order to use automated ontology documentation tools, such as pyLODE, we need to convert the vocabularies into OWL ontologies.
Converting them to OWL is straightforward since  we simply need to: 1) state that the vocabulary is an owl:Ontology, and 2) change all instances rdf:class to instances of owl:Class.
For example
@prefix rdf:    <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix rdfs:   <http://www.w3.org/2000/01/rdf-schema#> .
@prefix xsd:    <http://www.w3.org/2000/10/XMLSchema#> .
@prefix dc:     <http://purl.org/dc/elements/1.1/> .
@prefix lr:     <http://w3id.org/hefquin/lpg2rdf#> .

<http://w3id.org/hefquin/lpg2rdf>
    dc:creator       "Olaf Hartig" ;
    dc:date          "2023-11-29" ;
    dc:format        "RDF" ;
    dc:identifier    <http://w3id.org/hefquin/lpg2rdf> ;
    dc:title         "Vocabulary for describing LPG-to-RDF configurations"@en ;
    rdfs:label       "Vocabulary for describing LPG-to-RDF configurations"@en ;
    dc:description   "This vocabulary can be used to describe LPG-to-RDF configurations which are the basis of the user-configurable LPG-to-RDF mapping that is defined in the paper titled 'Foundations to Query Labeled Property Graphs using SPARQL*' by Olaf Hartig (2019) and that is implemented in HeFQUIN."@en ;
    rdfs:comment     "This vocabulary can be used to describe LPG-to-RDF configurations which are the basis of the user-configurable LPG-to-RDF mapping that is defined in the paper titled 'Foundations to Query Labeled Property Graphs using SPARQL*' by Olaf Hartig (2019) and that is implemented in HeFQUIN."@en .

## ==== CLASS DECLARATIONS ====

lr:LPGtoRDFConfiguration rdf:type rdfs:Class ;
    rdfs:label      "LPG-to-RDF Configuration"@en ;
    rdfs:comment    "Class of LPG-to-RDF configurations. Instances of this class must have exactly one lr:nodeMapping property, one lr:nodeLabelMapping property, one lr:edgeLabelMapping property, one lr:propertyNameMapping property, and one lr:labelPredicate property."@en .

would become:
@prefix rdf:    <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix rdfs:   <http://www.w3.org/2000/01/rdf-schema#> .
@prefix owl:    <http://www.w3.org/2002/07/owl#> .
@prefix xsd:    <http://www.w3.org/2000/10/XMLSchema#> .
@prefix dc:     <http://purl.org/dc/elements/1.1/> .
@prefix lr:     <http://w3id.org/hefquin/lpg2rdf#> .

<http://w3id.org/hefquin/lpg2rdf>
    a owl:Ontology ;
    dc:creator       "Olaf Hartig" ;
    dc:date          "2023-11-29" ;
    dc:format        "RDF" ;
    dc:identifier    <http://w3id.org/hefquin/lpg2rdf> ;
    dc:title         "Vocabulary for describing LPG-to-RDF configurations"@en ;
    rdfs:label       "Vocabulary for describing LPG-to-RDF configurations"@en ;
    dc:description   "This vocabulary can be used to describe LPG-to-RDF configurations which are the basis of the user-configurable LPG-to-RDF mapping that is defined in the paper titled 'Foundations to Query Labeled Property Graphs using SPARQL*' by Olaf Hartig (2019) and that is implemented in HeFQUIN."@en ;
    rdfs:comment     "This vocabulary can be used to describe LPG-to-RDF configurations which are the basis of the user-configurable LPG-to-RDF mapping that is defined in the paper titled 'Foundations to Query Labeled Property Graphs using SPARQL*' by Olaf Hartig (2019) and that is implemented in HeFQUIN."@en .

## ==== CLASS DECLARATIONS ====

lr:LPGtoRDFConfiguration
    a owl:Class ;
    rdfs:label      "LPG-to-RDF Configuration"@en ;
    rdfs:comment    "Class of LPG-to-RDF configurations. Instances of this class must have exactly one lr:nodeMapping property, one lr:nodeLabelMapping property, one lr:edgeLabelMapping property, one lr:propertyNameMapping property, and one lr:labelPredicate property."@en .

The branch 361-convert-vocabularies-to-owl-ontologies contains the converted ontologies (renamed engineconf.ttl and lpg2rdfconf.ttl) according to the pattern above. See https://liusemweb.github.io/HeFQUIN/vocab/engineconf/latest/index.html and https://liusemweb.github.io/HeFQUIN/vocab/lpg2rdfconf/latest/index.html for examples generated based on these files using pyLODE.
Note: Before merging into main we should update hefquin-vocabs/build.sh and confirm that the generated files are correct. I managed to make the generation work for procECVocab but not for procLPGtoRDF. We also need to update the update the example configuration files.

The branch 361-convert-vocabularies-to-owl-ontologies contains the converted ontologies (renamed engineconf.ttl and lpg2rdfconf.ttl) according to the pattern above.

I have created a (draft) PR from the branch to make it easier (at least for me) to see the differences. See #362

See https://liusemweb.github.io/HeFQUIN/vocab/engineconf/latest/index.html and https://liusemweb.github.io/HeFQUIN/vocab/lpg2rdfconf/latest/index.html for examples generated based on these files using pyLODE.

Thanks. That's very helpful! Two observations about these pyLODE renderings:

The table of contents (TOC) on the RHS of these pages seems to be based on the rdfs:label and looks a bit weird. Perhaps the actual issue here is that I have used rdfs:label wrongly?
The engine conf. vocabulary also contains a number of instances. These are not in the page (at least not in the TOC).


version 1.25.0 can not debug in WSL

Issue Type: Bug
When I update VS code to version 1.25.0
I found that the breakpoint not hit when debugging (I'm using WSL)
I have set "useWSL": true in launch.json and it works well in version 1.24.1
VS Code version: Code 1.25.0 (0f080e5267e829de46638128001aeb7ca2d6d50e, 2018-07-05T13:11:58.697Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6500 CPU @ 3.20GHz (4 x 3192)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.95GB (3.29GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
50%

Extensions (16)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.4.12

javascript-ejs-support
Dig
0.3.2

gitlens
eam
8.4.1

EditorConfig
Edi
0.12.4

php-debug
fel
1.12.3

beautify
Hoo
1.3.2

docthis
joe
0.7.1

vscode-language-pack-zh-hans
MS-
1.25.3

cpptools
ms-
0.17.6

csharp
ms-
1.15.2

debugger-for-chrome
msj
4.7.0

vetur
oct
0.12.5

material-icon-theme
PKi
3.5.1

ayu
tea
0.14.0

vscode-todo-highlight
way
0.5.12

markdown-all-in-one
yzh
1.5.1

@LeoYoung what version of node.js are you using in WSL?
You can find out by running this in Windows:
wsl node --version

I have the same problem, I have version 8.9.4 in WSL.

@weinand thank you for reply
my node version is 8.11.2 and npm version is 6.1.0

Most likely this issue is a duplicate of https://github.com/Microsoft/vscode/issues/53785.
There will be a fix for #53785 in tomorrow's Insider build.
It would be great if you could verify that this fixes the WSL issue as well.

@weinand
I have tried the new Insider build (Version 1.26.0)
It seems that this bug has been fixed, THANKS👍

@LeoYoung great, thanks a lot for verifying this.

I confirm, also works on my end! 👍

duplicate of #53785


I can't grab data.7z and deps.7z files

Hello @Temtaime !
Few weeks ago I was taking a look in this project and now I would like to make some contribution, but unfortunately I can't download the data.7z and deps.7z files anymore.
Just let me know if u are still working in this project or if you have intention to keep working on it, perhaps I could make some contribution!
Thank you

Hello.
Thanks for being interested in the project.
Yes, I'm working on it and will release major commit with new ui in the near future.
You cannot download the files because I'm upgrading the hardware on hosting server.
It will be up in 2-3 days.

Thanks for the information! I'm excited to see this new release!


run cmd "manage.py collectstatic",  Is this bug?

my env:

python -V
Python 3.5.2
pip freeze
Django==1.10rc1
git clone git@github.com:viewflow/django-material.git
cd django-material
python setup.py install

./manage.py collectstatic

You have requested to collect static files at the destination
location as specified in your settings:
/Users/silen/Development/python/myshop/src/static

This will overwrite existing files!
Are you sure you want to do this?
Type 'yes' to continue, or 'no' to cancel: yes
Found another file with the destination path 'admin/js/jquery.init.js'. It will be ignored since only the first encountered file is collected. If this is not what you want, make sure every static file has a unique path.
Found another file with the destination path 'admin/js/prepopulate.js'. It will be ignored since only the first encountered file is collected. If this is not what you want, make sure every static file has a unique path.
Found another file with the destination path 'admin/js/SelectFilter2.js'. It will be ignored since only the first encountered file is collected. If this is not what you want, make sure every static file has a unique path.
Found another file with the destination path 'admin/js/admin/DateTimeShortcuts.js'. It will be ignored since only the first encountered file is collected. If this is not what you want, make sure every static file has a unique path.
Found another file with the destination path 'admin/js/admin/RelatedObjectLookups.js'. It will be ignored since only the first encountered file is collected. If this is not what you want, make sure every static file has a unique path.
0 static files copied to '/Users/silen/Development/python/myshop/src/static', 117 unmodified.

That's common problem for django admin themes project, caused by django recently.
Read this - https://github.com/sehmaschine/django-grappelli/issues/751

thanks


Fix typo in README

Thanks for your fix!

No worries, always happy to contribute even if it was just a small typo!


Get A mirror in vue3

index.ts:173 Refused to load the script 'https://client.crisp.chat/l.js' because it violates the following Content Security Policy directive: "script-src 'self' 'unsafe-inline'". Note that 'script-src-elem' was not explicitly set, so 'script-src' is used as a fallback.
What should I do?

I would recommend adjusting your CSP policies as described here: https://docs.crisp.chat/guides/others/whitelisting-our-systems/crisp-domain-names/


Close / hide icons?

Not sure what the "hide icon" does here:

I clicked it and it seemed like Hidden Wallet closed, I don't see it in windows task manager. However when I re-opened I didn't need the password to access wallet. Haven't tested if I can spend funds without password re-prompt yet though.

Since you are downloading the full blockchain from the creation of the wallet, if you doesn't open your wallet for 1 day, next day you need to download 144MB of data (because there comes 144MB in every day to the blockchain).
HiddenWallet has 3 processes: HiddenWallet.GUI, HiddenWallet.API, tor.
If you hide, indeed the HiddenWallet.GUI will be closed, however HiddenWallet.API and the tor process will continue to run and keep syncing with the network.
Ultimately I hope I will not need the hide icon, because it'll be fast by default, after I implement the block and transaction filtering architechture.


fix : Show download dialog on emulator

Fixes #1868
Changes:

Remove active internet connection check
Remove ShowNotificationSnackbar class which is redundant because snackbar is shown using event bus(DownloadEvent).
Fixed error in getting facebook page id
Created View util method for SwipeRefreshLayout to setRefreshing state.
Update sample api-link

@iamareebjamal @mejariamol Please review

@mejariamol @iamareebjamal Updated please review.

@Shailesh351 Please squeeze your commits into one. It is a good practice to have one commit per PR.


Datetime width improperly calculated

Version: 3.2.1
Grafana version: 10.0
The required width of the datetime type seems to get improperly calculated when inside of a section. This causes the datetime picker to overlap with other sections.

As a bonus feature it would be nice to be able to set the width of a section and the padding between sections.

@steelskillet Thank you for opening the issue.
@asimonok Could you please take a look.

Created a bug in Grafana - https://github.com/grafana/grafana/issues/78466

thanks for passing that over to grafana, didn't even come to mind that it might have been an issue there.

@steelskillet No worries, we will check with Grafana team what can be done.

Grafana team confirmed the issue https://github.com/grafana/grafana/issues/78466.


Set permissions progrmatically on teams for a pipeline

Hello,
I am facing problem in setting permissions programatically to the teams.
I am able to create a pipeline programatically and assign teams to it but by default it give full access to all the added teams. Is there a parameter which I can pass for team permissions. I tried finding one in API documentation but couldn't.
Thanks

Hey @amazingankur! Sorry you're running into trouble with the teams permissions, and sorry for the delayed response to your issue.
There is a permissions parameter, but only if you're using the GraphQL API. You can pass an 'accessLevel' param along with your team id when you're creating a pipeline with GraphQL. Unfortunately our REST API doesn't have the same level of team support.
You can find that param in the GraphQL docs sidebar of the Buildkite GraphQL Explorer under: mutations > PipelineCreate > PipelineCreateInput > PipelineTeamAssignmentInput.


fix(CardLink): add missing css

Description
Add missing css import to CardLink
Changelog
New

import Card styles

@kennylam I'm going to let the tests run through this one to make sure everything is ok, but I approved this already.


Work on the Attribute module

Currently a draft, any comments are welcome!

Oh, and I forgot to cargo fmt. I will do it after, I just didn't want to screw something up.

Next up I will try to document GattServer and stuff related to it. Then I will:

try to split the AttributeData read and write functionality into a separate trait
try to make ServiceBuilder::add_characteristic_internal public, but use the trait I added as a dyn type.

Also, related issues: #98 #35

I changed a lot of stuff, so if anyone has the time, please review! :heart:

I have no idea how the error[E0597]: bat_level does not live long enough error came to be in example/apps/ble_bas_peripheral.rs. I would like some help with this. I don't think that I've yet changed anything in how the AttributeData worked...

I have no idea how the error[E0597]: bat_level does not live long enough error came to be in example/apps/ble_bas_peripheral.rs. I would like some help with this. I don't think that I've yet changed anything in how the AttributeData works...

I encountered similar error before, see #95 . But I'm not sure whether this is your case. There're some complex lifetime calculations.


Create a System Mock Up

Add basic instructions for code tests or screen shots of completed tests to the Wiki page.

Added screenshots of the main script running and resulting commit.

Added information for running the main script.


License?

What license is this code released under?

I'd love to see a license on this too. MIT would be nice but any kind of clarity would do.

Good day! Sorry for the delay! Added an MIT license =)


.cpu() hangs after gather_for_metrics

System Info
This my evaluation loop which is edited from the official deepspeed example.
I'm training on 4 nodes of 4 GPUs each on slurm.

def evaluate(cfg, model, tokenizer, eval_dataloader, accelerator, eval_dataset, logger):
    progress_bar = tqdm(range(len(eval_dataloader)//cfg.training.per_device_eval_batch_size), disable=not accelerator.is_main_process)
    torch.cuda.empty_cache()
    model.eval()
    losses = []
    logger.info("Starting Evaluation")
    samples_seen = 0
    for step, batch in enumerate(eval_dataloader):
        with torch.no_grad():
            outputs = model(**batch)
        loss = outputs.loss
        labels = torch.where(batch['labels'] != -100, batch['labels'], 0)
        preds = torch.where(batch['labels'] != -100, torch.argmax(outputs['logits'], dim=-1), 0)

        logger.info("Gathering Predictions and Labels .. ")
        preds, labels = accelerator.gather_for_metrics(preds), accelerator.gather_for_metrics(labels)
        logger.info(f"Gathered Predictions {type(preds)} on {preds.device} and Labels {type(labels)} on {labels.device}")

        logger.info("Gathering Losses .. ")
        losses.append(accelerator.gather_for_metrics(loss.repeat(cfg.training.per_device_eval_batch_size)))

        logger.info("Moving Predictions and Labels .. ")
        preds, labels = preds.cpu().numpy(), labels.cpu().numpy()

        logger.info("Decoding gathered predictions and labels")
        decoded_labels = tokenizer.batch_decode(labels, skip_special_tokens=True)
        decoded_preds = tokenizer.batch_decode(preds, skip_special_tokens=True)
        logger.info("Done decoding")
        rouge.add_batch(predictions = decoded_preds, references=decoded_labels)
        logger.info(f"Added a batch to metric of {len(decoded_labels)} labels and {len(decoded_predictions)} predicitions")
        del preds, labels
        torch.cuda.empty_cache()

        progress_bar.update(1)

    losses = torch.cat(losses)
    try:
        eval_loss = torch.mean(losses)
        perplexity = math.exp(eval_loss)
        rouge_results = rouge.compute()
    except OverflowError:
        perplexity = float("inf")
    return perplexity, eval_loss, rouge_results['rouge1'], rouge_results['rouge2'], rouge_results['rougeL'], rouge_results['rougeLsum']

After gathering the predictions and labels, the code hangs when i try to move them to cpu or when i call accelerator.wait_for_everyone().
I thought that some process might haven't gathered yet, so i printed print("Process {accelerator.process_index} gathered"). All of the 16 (4 nodes x 4 GPUs) processes has gathered. but it still hangs at both .wait_for_everyone()andpreds, labels = preds.cpu().numpy(), labels.cpu().numpy()``` i tried a variants of .cpu() like to('cpu') but it also hanfs.

### Information

- [ ] The official example scripts
- [X] My own modified scripts

### Tasks

- [X] One of the scripts in the examples/ folder of Accelerate or an officially supported `no_trainer` script in the `examples` folder of the `transformers` repo (such as `run_no_trainer_glue.py`)
- [ ] My own task or dataset (give details below)

### Reproduction

def evaluate(cfg, model, tokenizer, eval_dataloader, accelerator, eval_dataset, logger):
    progress_bar = tqdm(range(len(eval_dataloader)//cfg.training.per_device_eval_batch_size), disable=not accelerator.is_main_process)
    torch.cuda.empty_cache()
    model.eval()
    losses = []
    logger.info("Starting Evaluation")
    samples_seen = 0
    for step, batch in enumerate(eval_dataloader):
        with torch.no_grad():
            outputs = model(**batch)
        loss = outputs.loss
        labels = torch.where(batch['labels'] != -100, batch['labels'], 0)
        preds = torch.where(batch['labels'] != -100, torch.argmax(outputs['logits'], dim=-1), 0)

        logger.info("Gathering Predictions and Labels .. ")
        preds, labels = accelerator.gather_for_metrics(preds), accelerator.gather_for_metrics(labels)
        logger.info(f"Gathered Predictions {type(preds)} on {preds.device} and Labels {type(labels)} on {labels.device}")

        logger.info("Gathering Losses .. ")
        losses.append(accelerator.gather_for_metrics(loss.repeat(cfg.training.per_device_eval_batch_size)))

        logger.info("Moving Predictions and Labels .. ")
        preds, labels = preds.cpu().numpy(), labels.cpu().numpy()

        logger.info("Decoding gathered predictions and labels")
        decoded_labels = tokenizer.batch_decode(labels, skip_special_tokens=True)
        decoded_preds = tokenizer.batch_decode(preds, skip_special_tokens=True)
        logger.info("Done decoding")
        rouge.add_batch(predictions = decoded_preds, references=decoded_labels)
        logger.info(f"Added a batch to metric of {len(decoded_labels)} labels and {len(decoded_predictions)} predicitions")
        del preds, labels
        torch.cuda.empty_cache()

        progress_bar.update(1)

    losses = torch.cat(losses)
    try:
        eval_loss = torch.mean(losses)
        perplexity = math.exp(eval_loss)
        rouge_results = rouge.compute()
    except OverflowError:
        perplexity = float("inf")
    return perplexity, eval_loss, rouge_results['rouge1'], rouge_results['rouge2'], rouge_results['rougeL'], rouge_results['rougeLsum']

### Expected behavior

It should not hang.

Does it hang on "Gathering losses" or "Moving predictions and labels"

"Moving predictions and labels"

Turned out it was my mistake, i didn't know that both preds and labels should of the same size (sequence length) across all GPUs. which is wrong in my case. Solved it via padding


Add dynamic table columns and additional search parameters

Is your feature request related to a problem? Please describe.
Improve the library overview. We need new options for the search bar to search and filter by more fields.
Describe the solution you'd like
Front end:
Search bar

Add new fields and toggle buttons for search / filters
Add new attributes to the searchForm in the components.ts file

Back end:

Add or adjust fields in the existing filter class if necessary

Describe alternatives you've considered
N/A
Additional context
We need new options for the search bar to search and filter by more fields.

Hey Savu, I edited your issue to better reflect the current status as we already implemented the dropdown menu.


Silo patching failing in build_visit

Describe the bug
The patches for fpzip are failing in container builds on Debian and Ubuntu, due to different line endings.
I tried to force the cray path, which calls dos2unix, and uses a different patch function, but the dos2unix command wasn't available.
Desktop

OS and version: Ubuntu, Debian
VisIt Version: 3.4.2
Server info (if applicable):

@markcmiller86 @JustinPrivitera suggestions?

patch --binary perhaps?

patch --binary perhaps?

didn't work.

or maybe we just provide a good tarball with correct line endings

I tried this manually, and it  works.
Had to also modify the build script to use the non-windows-line-endings version of the patch.

well so, does it work if we modify the logic such that dos2unix is never called but the non-windows-line-endings patch is applied?

well so, does it work if we modify the logic such that dos2unix is never called but the non-windows-line-endings patch is applied?

No. Tried that first.  Then I untarred the silo file, changed the fpzip files to unix style by hand, and retarred it. Modified the script, then reran and it worked.

@markcmiller86 what do you think of modifying the silo source tarball used by build_visit to remove those pesky windows line endings?

@cyrush says we can go ahead and modify the tarball.

I will do this now.

In TPL release and repo:


Refactor auth package

General tracking issue for refactoring the app/auth package. Improvements include but are not limited to:

Renaming classes and other symbols to better represent their use case
Restructuring the inheritance hierarchy and using other architectural constructs to more clearly represent the relationships and reuse of logic between classes.
Factoring out common, redundant, or misplaced logic across files.

Added back the tech-debt tag because I think the current state of the auth package is convoluted and hard to understand, which is something I consider to be tech debt. Nick, feel free to reply if you disagree.

Closing this as the issue is relatively subjective, several PRs have been put out for it, and it's been a while since it's been touched.


Proposal: Spans

I'd like to start adding allocation-free or mostly allocation-free alternative methods for what's available.
I've implemented GetRandomBuffer and BinaryToHex methods, by way of example.

@tabrath any follow up, here? The cause of the build failure is the CI pipeline doesn't support C# 7.3

Hi, sorry for the long wait. I really appreciate your effort as this is something I've been thinking about myself. What's your plan for this PR, do span-versions of everything or seperately for each feature in libsodium?
I'll look into the CI issue ASAP.

Well, either way, most likely I'd add them one at a time in a branch then you can merge to a single commit in a PR.
But the examples were to propose an approach (doing them in a partial in an additive file, which is "non-invasive") or you might want to do them in inline with the allocating methods, or remove the allocating methods, or wrap the allocating versions with the non-allocating versions, or create a new library for the non-allocating version... and so on.

Alright, I've fixed the CI to support 7.3 so you can rebase to get the checks passing.
One idea I had was to keep the allocating methods but call the new non-allocating methods internally, effectively removing the old methods but not break the api. What do you think?

Sounds good. Do you want the new and old methods side by side (method
signatures and interop signatures)?
On Sat, Mar 30, 2019 at 11:58 AM Trond Arne Bråthen <
<EMAIL_ADDRESS>wrote:

Alright, I've fixed the CI to support 7.3 so you can rebase to get the
checks passing.
One idea I had was to keep the allocating methods but call the new
non-allocating methods internally, effectively removing the old methods but
not break the api. What do you think?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tabrath/libsodium-core/pull/23#issuecomment-478258981,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAH6SrZe2Y8y_VhWBZzkSRkCQ0cjjNquks5vb4mTgaJpZM4bGwMf
.

--
Daniel Crenna
Conatus Creative Inc.
cell:613.400.4286

Yeah, side-by-side sounds good to me.

@tabrath this is done for Utilities.cs and CryptoHash.cs, if you want to do this piecemeal, those should be ready now.

@tabrath looks like another CI failure on AppVeyor because of a missing file.

Hi, nice work going on here. There's a change is vstests behavior in recent versions causing the test results not to be found. I've updated the master branch to use wildcards when searching for that file, so you might try merging master into your branch (or rebase) and check if that helps.

@tabrath I updated the CI, appveyor passed by Travis only supports .NET Core 2.1 :/

@danielcrenna It's the macOS build that is failing, you might try to update the travis configuration for macOS to use 2.2 as well.

@tabrath yep, my bad. looks like it's back to green now

Hi, @danielcrenna, I'm sorry this has taken so long, but if you'd like, you could resolve those conflicts, then I will merge this pretty soon.

Is there a tracking issue for this PR? It could be great to discuss the possibility of implementing the span/memory based functions with less and controlled allocations.
I can also help in implementing the rest if @danielcrenna can't work on this right now.

I ran out of steam trying to do all of it, though it’s mostly done. I ended
up just using P/Invoke directly for the handful of methods I actually use.
Have at it.
On Mon, Feb 17, 2020 at 11:42 PM Attila Érsek<EMAIL_ADDRESS>wrote:

Is there a tracking issue for this PR? It could be great to discuss the
possibility of implementing the span/memory based functions with less and
controlled allocations.
I can also help in implementing the rest if @danielcrenna
https://github.com/danielcrenna can't work on this right now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabrath/libsodium-core/pull/23?email_source=notifications&email_token=AAA7USTJHFIQGRYHTT2PQ7LRDNRL7A5CNFSM4GY3AMP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMARZXA#issuecomment-587275484,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA7USRLW4XQKUTWX7GXRJTRDNRL7ANCNFSM4GY3AMPQ
.

--
Daniel Crenna
Conatus Creative Inc.
cell:613.400.4286

@tabrath any input from your side?

Hi, it's pretty high up on the priority list, really.  @danielcrenna has done a good effort to build upon. Sad to hear you went "out of steam" to pull it off, and now there's a pile of merge conflicts to take care of before continuing the work.
If you're up to the task @attilaersek, I'd suggest to either help out resolving the current conflicts - then we could merge that to a "feature branch" and continue work from there, whoever might be interested in helping out.
@danielcrenna is that something you'd have time to do? Otherwise I might find time to do it myself in a week or so.
As there's much work left, we might want to do one piece at a time, reviewing and merging that back to the feature branch.

@tabrath @attilaersek I don't think there's that much work left, probably three or four APIs?
I'd be happy to create a new feature branch and PR one feature at a time, at least up to the ones I've already done and made use of, to get the extension methods and other helpers added.

Well, if you could clean this PR up and merge a solid baseline into either master or a new feature branch that could allow me and others join the effort case by case.
@danielcrenna if you don't have time to resolve the conflicts in your branch I can help you in that by opening a pr onto your branch!
I would also like to add some performance benchmarks and same extra tests around spans to make sure the memory is fixed/pinned in all cases.

@danielcrenna : i've opened a PR into your branch that resolves the merge conflicts and solves the build failure. hope that'll be enough to get this one accepted.

i've also done with merging current master to that branch, but i prefer holding that commit back as long as someone agrees to review it properly. i had harsh times with dealing with all these style and linting changes.

can someone help on the build failures? both failures looks strange.

Seems like some dependencies didn't work well together. Added some conditionals for each target framework and dependency and created a PR on your fork @danielcrenna.

Allocation-free or mostly allocation-free alternative methods would be a very nice enhancement. However, since libsodium-core is now in maintenance mode, I don't think this is worth the effort. Going forward, it is recommended to use one of the alternative libsodium bindings.


Is this project still being actively maintained ?

I see that the last commit was almost 1.5 years ago. Looks pretty dead to me unless I am missing something :(
If so, is there an alternative project ?

For anyone who is still interested, I am maintaining a fork at @softwareventures/precise-commits.
Compared to this project, it fixes a bunch of bugs, and adds support for Windows and CRLF line breaks.
I've been using it for a couple of months on my projects and it works well. There are still some pathological cases where it does something weird, but I've never seen it break my code. If you use it on a project where the existing indentation is consistently different to the indentation you configure for prettier, then you are going to have a bad time, but otherwise it is suitable for general use.
You can install it from npm @softwareventures/precise-commits.


Primary energy demand mekko should be updated with new biomass carriers

Now the primary energy demand mekko does not load! Because the queries still refer to old biomass carriers like:

corn
manure
woody biomass
wood

Would anybody like to pick up this issue before the deploy? :)

I can fix this!

This is done right? :)

Yes it is, closing this!


Polish v0.13.2 release notes.

Per conversation with @mrakitin, I am adding a link to the v0.13.2 GitHub milestone. This is a good way to provide developers with a full list of the changes, some of which are too low-level to be of interest in the published release notes for users.
Let's get in the habit of getting our milestones up to date when we release and adding links like this to all future release notes.


PACKAGES. Fix flutter web caching

The prob:
js_helper.dart:2390 Uncaught DeferredLoadException: 'Loading main.dart.js_3.part.js failed: the code with hash 'CJgBkQFYyQJWPw43SMSfjnA6Qpk=' was not loaded.
event log:
 - _loadHunk: main.dart.js_3.part.js
 - download: main.dart.js_3.part.js from https://maintemplate.ci.getcouragenow.org/main.dart.js_3.part.js
 - download success: main.dart.js_3.part.js
 - missing hunk: main.dart.js_3.part.js (CJgBkQFYyQJWPw43SMSfjnA6Qpk=)

At the moment we do this approach:
https://medium.com/flutter-community/caching-in-flutter-for-web-42b3ae0e348f
We do it in the CI.yaml using sed to update the version querystring value.
Its not working because:

does not actually clear cache because it just forces ANOTHER entry point leaving the old.
Ci of adding hash now break flutter web due to flutter web architecture changing

https://github.com/flutter/flutter/issues/51141


Refactoring localization in json files

Localization files are placed in Client/public/locales. Values from localizations are use in pages modules and components. Check it all.


Remove id from top-level notebook structure.

nbformat currently barfs on this notebook as it has an id field set directly on the cell. They're expected under a metadata field, and this notebook already has them set on the metadata, so assuming they're otherwise unused I have removed them.
Here's the error, in case it helps:
E 2022-09-08 20:30:15.962852-070026 57758 __init__.py:80 Notebook JSON is invalid: Additional properties are not allowed ('id' was unexpected)

Failed validating 'additionalProperties' in code_cell:

On instance['cells'][0]:
{'cell_type': 'code',
 'execution_count': 1,
 'id': 'ee22a795',
 'metadata': {'cellView': 'form',
              'execution': {'iopub.execute_input': '2022-07-07T09:27:05.420016Z',
                            'iopub.status.busy': '2022-07-07T09:27:05.419434Z',
                            'iopub.status.idle': '2022-07-07T09:27:05.423857Z',
                            'shell.execute_reply': '2022-07-07T09:27:05.423255Z'},
              'id': '906e07f6e562'},
 'outputs': ['...0 outputs...'],
 'source': '# @title Copyright 2022 The Cirq Developers\n'
           '# Licensed under the...'}

Failed Pytest looks like differences in floating point rounding, I'm guessing unrelated to this change?

@markmcd What did you run to get this error? This passes on the master branch for me.
(cirq_venv) vtomole@vtomole:~/Cirq$ ./check/nbformat
Notebooks are formatted.


Alerts and averages not working consistently

need consistent upper/lower case in JSON and SQL. Fixed.


Adapter layer: two flavors of adapters: optimized for internal vs. app<->backend exchange

Depending on the use case these both can be important, but It's not possible to optimize for both in a single adapter:

Copyless app-internal data exchange, together with direct property access (by storing our own type)
Copyless (type-matched) data exchange between app code and the backend (by storing the backend's own type, wrapped by unified function-style accessors)

Impedance matching can only be done for one (unless the data itself stored inside the adapter is duplicated... :) ).
But then... The internal types aren't even going to be adapters any more, so the interop issue with the backend will just resurface again, anew, like cancer! :)
So... I guess, along with the "real" adapters optimized for app<->backend traffic, I just have to grow my own dictionary types then, with "some swappable conversion capabilities"... :-/
The key insight here is to embrace the differences in the app-facing vs. backend-facing property access (which is basically an inevitable limitation of C++)_: assuming (well: hoping... :) ) the use cases (coding contexts) for the two flavors would already be different enough for the access syntax differences not being too confusing/annoying/distracting (they might even be a natural fit, or even helpful, making things a bit more explicit!) -- design the split (app-optimized vs. backend-optimized) interfaces around that, because that would still allow sharing a substantial chunk of the operations (e.g. with CRTP). Those would, in fact, make up the bulk of the interface.
This way the two flavors, despite having different access APIs, and the app-oriented one not even being a real adapter any more, can still be kept strongly related, evolving coherently!
One remaining concern is how those uniform operations (shared across the different storage & access flavors) can avoid copying... Well, the derived specialized (app vs. backend) interface flavors can still have a common internal funtion-style accessor interface, too, that the shared ops in the CRTP base could use!
Mmm... And, also... To still keep the light, but unoptimized (copying) conversion ability of the app-internal type, it needs an additional mixin/trait ctor + cast op. pair, dispatched by the current backend... Which might imply YET ANOTHER backend access adapter to unify all the different backend types (for our app-internal "dictionary" class), only this one can copy, unlike the backend-oriented "real" adapter... Ugh... This is getting out of hand again... :)

OK, let's start from scratch... :) I do have a vision now, a dual-headed, twin (well, a Siamese then...) adapter pair, with a shared body  of common generic ("standard") operations characteristic to the abstract type (like a vector, rectangle, vertex) itself, and the two "heads" specialized for optimal data exchange betwewn either the app, or the backend (but not both, because I think that's impossible).

-> POC at http://github.com/x1ab/flexivec!


msbuild: always use shared library from dotnet-sdk

Description of changes
This changes the use of .so to use the more generic stdenv.hostPlatform.extensions.sharedLibrary.
This also overrides mono/build/extract_and_copy_hostfxr.sh which uses the internet and is x86 specific.
I also changed the platforms for roslyn, I am unsure why it was Linux only to start with

fixes https://github.com/NixOS/nixpkgs/issues/233570
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

This looks fine, but is there anything existing that handles more complex shared library naming conventions? For example, hostfxr.dll on windows (I'm not sure if this is the actual name on windows, it just seems like most build systems use both a prefix and suffix to handle this)?

This looks fine, but is there anything existing that handles more complex shared library naming conventions? For example, hostfxr.dll on windows (I'm not sure if this is the actual name on windows, it just seems like most build systems use both a prefix and suffix to handle this)?

Can you elaborate? nixpkgs does not in any way support window builds, so it would be useless to add a case for it.

Can you elaborate? nixpkgs does not in any way support window builds, so it would be useless to add a case for it.

I just thought there might be something like CMAKE_SHARED_LIBRARY_PREFIX in nixpkgs, but I had a look and there's not.  Sorry for the noise.


MAINT: Revisions after previous maintenance commit (#83)

Revise #83 and fix the two outstanding issues:

[x] Outdated requirements.txt file when generating docs.
[x] Inconsistent handling of the --output-spaces section, as a result of copy&paste from fMRIPrep.

for some obscure reason, circle has stopped to build my PRs :(

for some obscure reason, circle has stopped to build my PRs :(

That's annoying.
@josephmje : is your PR #81 rebased on current master? Maybe there's a difference between these two PRs that explains why one is tested and the other isn't?

for some obscure reason, circle has stopped to build my PRs :(

That's annoying.
@josephmje : is your PR #81 rebased on current master? Maybe there's a difference between these two PRs that explains why one is tested and the other isn't?

Yes, I rebased earlier this morning.

With the new version of smriprep, many of the output_spaces arguments need to be changed to spaces.

I forgot that the documentation used a different set of package versions. That explains why we were still getting nipype errors.

@josephmje have you rebased to this one? should I just merge and check whether I successfully addressed all issues?

@josephmje have you rebased to this one? should I just merge and check whether I successfully addressed all issues?

I haven't rebased to this one but I did test locally. The pinned versions are working well. I left a comment about the "orig" space. Also, in the call to init_anat_preproc_wf(), the output_spaces argument should be changed to spaces.

Seems like unfollowing and following from my repo did the trick (regarding to CircleCI).


Added colored baseball caps for clippy, updated icon

Since the green color is now also selectable from configuration, I had to introduce an if statement to prevent the combination of green with cats and dogs. I think we need a better way of handling pet types and their available colors soon 😅.
Closes #17

 well that seals it.
:+1:


feat: Threads in side bar

[x] Threads in sidebar.
[x] Styles for complementary sidebar (different width).
[x] Persistent properties.

🤖 Generated by Copilot at 479ab3b
Summary
🛠💾🎨

Refactored several plugins and components to use a local storage store for managing the split view state and other settings, and to improve the UI and code quality. Extracted common styles and functions into modules. Removed unused or unnecessary imports, parameters, and dependencies.

Oh, we're the crew of the SplitViewPlugin
And we work with local storage all day
We refactor and improve the code we write
And we make the UI look nice and bright

Walkthrough

Removed enableComplementarySidebar option from SplitViewPlugin configuration and used local storage store to manage split view state (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link).

A few notes/questions:

Why the random person icon with the random colors? Shouldn't we use the HALO shape?
Is this a standalone thread object, or is it associated with the document that's open?

I assume it's standalone and has a title of "Book Club"?


error message for missing async could suggest fix

given:
foo() async => 42;
main() {
  print(await foo());
}
error message is:
'test.dart': error: line 6 pos 15: ')' expected
  print(await foo());
              ^
it seems like await is not supported. I can imagine this is for backwards compat, await still needs to work as a variable. However it would be super nice to have it suggest the fix, which is add async to main.

dart2js now prints a helpful message:
~/tmp $ dart2js bug.dart
bug.dart:4:13:
Error: Expected ';' after this.
  var foo = await awesome();
            ^^^^^
bug.dart:4:13:
Warning: Cannot resolve 'await'.
Did you mean to add the 'async' marker to 'main'?
  var foo = await awesome();
            ^^^^^
Hint: When run on the command-line, the compiled output might require a preamble file located in:
  <sdk>/lib/_internal/compiler/js_lib/preambles.
Error: Compilation failed.

@mit-mit Why did you change the area to 'Analyzer'? I don't see any description of a problem with analyzer in here, so I'm puzzled.

Quick-fixes are provided by the analyzer, aren't they?
The use of await could provide a quick-fix for adding async if it isn't there already.

This was originally intended to ask that the VM print a more contextual error message.
I don't know if Analyzer currently prints a helpful feedback message, but if not, that should also be helpful :)

I guess I misunderstood what this bug was tracking. Should we keep it to track the VM issue, and then file a new one for the analyzer? I mean, it should be able to suggest a better fix than Undefined name 'await', right

From what I can see, dart2js is the only one giving a good error message.
Both the analyzer as well as the VM complain about the syntax or the undefined name.
For reference here is the VM output:
$ dart /tmp/tt.dart
'file:///tmp/tt.dart': error: line 2 pos 15: ')' expected
  print(await 5);
              ^

Yes, I think it's appropriate to keep this issue for the VM and create a new one for the analyzer.

Moved back to VM, and opened #24264 for Analyzer

It is not the VM's job to analyze errors. That's why I commented above that the Analyzer should issue suggestions, not the VM. Change Florian's example above to print(await -5) and you have a syntactically correct expression that is most likely still wrong.
Even better than burdening the Analyzer or compiler: fix the language.

As Matthias points out the contextual error message for Dart in this case is the fact that there is no getter await.


Introduce Deployment type to eventually replace Environment enum

Not meant to be fully complete, but shows the Misk changes for starting to remove the Environment enum in favor of a Deployment type.
The environment (in plain string form) is still used for things like parsing config, at least for now.

Misk fails builds on compile warnings, including deprecation. I'm planning on disabling this during the rollout.


Rmarkdown inside the VSCode-R cannot render Chinese character in figure/table caption correctly

Describe the bug
The rmarkdown within the VSCode cannot render Chinese character correctly.
(after clicking the knit rmd button, the corresponding pdf renders UTF8 codes instead of correct Chinese character).
However, the same document can be rendered correctly if I open an terminal and type
R
rmarkdown::render("file.rmd")

So I think there's nothing to do with the encoding of my source file. The render() function takes "UTF8" encoding as default option.
To Reproduce
Suppose we have the following rmd file file.rmd:
---
title: "minimal example"
author: "Kobe Bryant"
output: html_document
---

```{r setup, include=FALSE}
knitr::opts_chunk$set(echo = F, warning = F, message = F, eval = T)
```

Chinese character in the main article can be rendered normally.
(正文区的中文可以正确的渲染)

Figure legends cannot be rendered normally.

```{r, fig.cap = "科比"}

plot(1:5, 1:5)

```

// R path for Mac OS X
"r.rterm.mac": "/usr/local/bin/R",

// R command line options (i.e: --vanilla)
"r.rterm.option": ["--no-save", "--no-restore"],

// An optional encoding to pass to R when executing the file, i.e. 'source(FILE, encoding=ENCODING)'
"r.source.encoding": "UTF-8",

// Keeping focus when running
"r.source.focus": "editor",

// Use active terminal for all commands, rather than creating a new R terminal
"r.alwaysUseActiveTerminal": false,

// Use bracketed paste mode
"r.bracketedPaste": true,

// Enable R session watcher
"r.sessionWatcher": true,

// Delay in milliseconds before sending each line to rterm (only applies if r.bracketedPaste is false)
"r.rtermSendDelay": 8,

Expected behavior
The Chinese character within a figure caption can be rendered correctly.
Screenshots
The correct html document (via calling rmarkdown in terminal) should be:

What rendered by VSCode-R extension  knit rmd button (wrong) is:

Environment (please complete the following information):

OS: [macOS 12.0.1, Apple M1 MacBook Air]
VSCode Version: [1.63.2(universal)]
R Version: [4.1.2(aarch64)]
vscode-R version: [2.3.5]

I believe I have the same problem with French accents.
My system uses the following locale in the terminal :
$ locale
LANG="fr_FR.UTF-8"
LC_COLLATE="fr_FR.UTF-8"
LC_CTYPE="fr_FR.UTF-8"
LC_MESSAGES="fr_FR.UTF-8"
LC_MONETARY="fr_FR.UTF-8"
LC_NUMERIC="fr_FR.UTF-8"
LC_TIME="fr_FR.UTF-8"
LC_ALL=

In the interactive R console, I have
> Sys.getlocale()
[1] "fr_FR.UTF-8/fr_FR.UTF-8/fr_FR.UTF-8/C/fr_FR.UTF-8/fr_FR.UTF-8"

But for some reason, vscode-R knit and preview don't inherit this setting, and a R chunk
with Sys.getlocale() will output
[1] "C"

As a workaround, adding Sys.setlocale(category = "LC_ALL", locale = "fr_FR.UTF-8") in the setup chunk of my Rmd
file solved my accent problem.


Reduce redundancy in tsdb for different modules

Desktop (please complete the following information):
OS: ubuntu
Describe
Currently after end-to-end integration of the ping module, almost all the methods and structures are written again with minor changes, like Block and BlockPing. When we will add more modules there will be lot of redudancy. For example, Block, BlockFloodPing, BlockJitter, BlockReqRes etc. There must be a better way to do this.

@Harkishen-Singh one more thing!
There are certain functions such as "append" which does the same thing for all the modules. Is it required to create different functions for different modules?

@ganeshpatro321  @ankitjena, for normal cases (except ping and flood ping), we require the results to be stored in tsdb. But, the default design of tsdb supports storing a single value corresponding to the time scale. But in the case of ping, we were required to store 4 different values corresponding to a single timestamp - min, mean, max, mdev. That's y we went for different functions like BlockPing.
but, as far as other modules are concerned, I don't see any multiple values needed to be stored corresponding to a single timestamp. So, the default design would work.

@ankitjena  we can come with an approach that could be universal for each module and thus reducing the redundancy. That would be really great.

@Harkishen-Singh that's what I was suggesting

@Harkishen-Singh What would be the better approach, any suggestions?

@ankitjena  nothing as of now. If something strikes will share here.

@ankitjena one way we can do this is by storing the complex values as string in datapoint (of type string) field. This way, we can store the value by marshalling it and retrieve it in the same process and store in the respective lower level structure.

Let's have a common blockType struct with a common timestamp and a datapoint,
The datapoint can be of various types since it is different for different modules as per basic Object Modelling

Can embedded struct types help here? Or the same thing?


added stand with Ukraine badge

This may be contentious but I do believe it is something we all agree on - that we should all stand with Ukraine, at least in spirit, and we should show it, whether we are software engineers, scientists, or of any other profession - the events happening now in Ukraine go beyond politics, beyond science, and beyond any national or regional interest - they shake our core concept of humanity and peace in our time, and endanger everything around us, including climate science, and our mission of promoting it. @ESMValGroup/esmvaltool-developmentteam please approve :ukraine:

Dear @valeriupredoi, thanks a lot for initiating it.
That would be great if the community would be ok to acknowledge it.

Thank you for the support you guys are showing! If any of the @ESMValGroup/esmvaltool-coreteam could approve this this then I'll merge; alternatively please express reservations if any
@ledm very nice idea about Twitter, please feel free to do that, from my point of view, that is :+1:
:ukraine:

Will merge this in the evening if nobody has any reservations in the meantime :+1:

excellent, cheers, Manu! :beer:


Problem with publicPath when localhost has URL segment but production server hasn't

I'm using file-loader to load images in background-image in CSS files. I faced difficulty with different URL on my local environment and server.
Locally on my IIS I have a website with the name "platform", so the URL is http://localhost/platform.
The production server is working with root folder so URL on production is http://application-name.com, without "platform" segment.
Now I use a workaround where I check NODE_ENV and add or not "platform" segment:
loader: 'file-loader',
options: {
	name: '[path][name].[ext]',
	publicPath: process.env.NODE_ENV === 'development' ? '/platform/' : '/'
}

This approach is not so reliable, so could somebody help me to specify a universal publicPath?
Previously I made it another way:
loader: 'file-loader',
options: {
	name: '[path][name].[ext]',
	publicPath: (url, resourcePath, context) => {
	  return `../../../${url}`;
	}
}

But now it doesn't work anymore for me because now I need to launch my code in different places of project, and there can be different levels of urls:
http://application-name.com/1/2
http://application-name.com/1/2/3

It looks like you just deleted our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please either update the issue with the template and reopen, or open a new issue.
Don't use relative url, it can break app

Also please use gitter or stackoverflow for questions


[Enhancement] Custom CSS + Option to pass custom class to widgets

Users familiar with CSS would want to overwrite the styling of widgets. A place to write custom CSS and an option to pass the custom class to widgets would be great.

@jsartisan should this instead be achieved through the themification option we are thinking about?

Closing this as it is a duplicate of https://github.com/appsmithorg/appsmith/issues/3095 (moving this issue's description to the other issue.)

@nikosgpet Hi. We've been working on a revamp of our layout and theming engines that should facilitate faster, more predictable, and more consistent styling of the widgets. While that is still work-in-progress the team wants to properly evaluate a common request for custom CSS, that has been asked by the community.
To that end I'd be interested in talking with you, to see what kind of ideas you have.
You can schedule a call with me if you're willing to talk or drop me an email.


Implement wrapper for native go WaitGroup for use in cadence workflow Context

Simple implementation of native go sync.WaitGroup using workflow Selector and workflow Future.  This is a first pass.  Let me know any changes that I should make.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Pull Request Test Coverage Report for Build 277

0 of 41 (0.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 73.809%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

workflow/deterministic_wrappers.go
0
3
0.0%

internal/workflow.go
0
6
0.0%

internal/internal_workflow.go
0
32
0.0%

Totals

Change from base Build 276:
-0.3%

Covered Lines:
8353

Relevant Lines:
11317

💛  - Coveralls

Need to produce some unit tests for coverage test.

Good simplification. I'll make the changes when I'm back down at my workstation... And add unit test.

@mfateev I am experimenting with some irregular native go sync.WaitGroup behavior.  One thing that I noticed is that if the native WaitGroup is blocking and waiting on goroutines that will never finish or waiting on a number of goroutines but Done() is not called the specified number of times to increment the counter back to 0, WaitGroup.Wait() will throw a fatal error:  "fatal error: all goroutines are asleep - deadlock!".  Is this behavior that should be replicated or should the future.Get() just continue to block and wait for a timeout?

It looks like the deadlock detection is done by go runtime, not by WaitGroup implementation. It can be triggered by deadlocks not involving WaitGroup. So I think we don't need to match this functionality in the mapper. The good news is that Cadence support getting stack trace of any workflow which makes troubleshooting deadlocks simpler.

Filed https://github.com/uber-go/cadence-client/issues/803


[1.16.1] Furnace / Blast Furnace provides light when placing it down without anything cooking.

Title^,
Furnace or the Blast Furnace provides light when placing them without anything cooking in them.
Versions:
Minecraft: 1.16.1
Forge: 32.0.69
FastFurnace Version: 4.0.0
GIF with FastFurnace: https://gyazo.com/4a6949c2a5c856ef130590ce3747534e
GIF without FastFurnace: https://gyazo.com/ba3a9c781f2dc88fe5f2f9c96b694473
Edit: Tried this in a clean instance w/ Only FastFurnace & Placebo installed and it still happens.

I can confirm that this is fixed in FastFurnace-1.16.1-4.1.1, thanks.


feat: archivist & attestation source integration

This uses the updated attestation source API from go-witness during
policy verification. This also adds support for storage of attestations
within archivist and using archivist to verify policies.
Signed-off-by: Mikhail Swift<EMAIL_ADDRESS>

This is a draft but all existing tests are working.
Archivist functionality needs to be thoroughly tested.

This requires https://github.com/testifysec/go-witness/pull/10
The go.mod has been updated to point at a commit on the go-witness branch but a go-witness release will need to be cut and this project's go.mod updated once ready.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

23 of 34 (67.65%)  changed or added relevant lines in 2 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 26.768%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/verify.go
21
26
80.77%

cmd/run.go
2
8
25.0%

Files with Coverage Reduction
New Missed Lines
%

cmd/run.go
1
41.94%

cmd/verify.go
4
53.95%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
106

Relevant Lines:
396

💛  - Coveralls

Docs also need to be updated

Will require https://github.com/testifysec/go-witness/pull/21 and a go.mod bump before merging


Customized /etc/realmd.conf being overwritten

Hello!
I am trying to work around this issue, where "realm join" overwrites a hardened password-auth/system-auth with authselect:
https://issues.redhat.com/browse/RHEL-5101
https://access.redhat.com/solutions/5956991
A suggested workaround is to customize /etc/realmd.conf to include these:
[commands]
sssd-enable-logins = /usr/bin/sh -c "/usr/bin/systemctl enable oddjobd.service && /usr/bin/systemctl start oddjobd.service"
sssd-disable-logins = sssd-disable-logins = /bin/true

However, realmd.conf gets overwritten when the template is applied. An alternative would be to edit /usr/lib/realmd/realmd-distro.conf directly, but it would be nice if the role could accommodate customization of realmd.conf.

I would prefer to keep the template based implementation of realmd.conf, rather than moving to use lineinfile or blockinfile.
What if we introduce a parameter like ad_integration_preserve_custom_sssd_profile: true|false
then in the template
{% if ad_integration_preserve_custom_sssd_profile %}
[commands]
sssd-enable-logins = /usr/bin/sh -c "/usr/bin/systemctl enable oddjobd.service && /usr/bin/systemctl start oddjobd.service"
sssd-disable-logins = /bin/true
{% endif %}

@justin-stephenson @jakub-vavra-cz wdyt?

Well realmd.conf is unfortunately a file not owned by realmd package so we can not check if user changed it by rpm -V realmd.
We can stat if it exists and if does not, use the template.
If it exists and does not have our ansible signature we can apply block in file (showing warning that this is not recommended). If  it exist and has our ansible signature we can just silently replace it with template.

Well realmd.conf is unfortunately a file not owned by realmd package so we can not check if user changed it by rpm -V realmd.

But can it be owned by the ad_integration role?  Can we say "if you use the ad_integration role to manage realmd.conf, you cannot be guaranteed to preserve any edits made to realmd.conf outside of the ad_integration role."  Note that most other system roles work the same way - you use the system role to manage the resource, and any changes made outside of the system role will be wiped out by the role.

We can stat if it exists and if does not, use the template. If it exists and does not have our ansible signature we can apply block in file (showing warning that this is not recommended). If it exist and has our ansible signature we can just silently replace it with template.

If we have to code it up in such a way that any/every change made to realmd.conf outside of the role must be preserved, then we are going to make the role very complicated . . .

I would prefer to keep the template based implementation of realmd.conf, rather than moving to use lineinfile or blockinfile. What if we introduce a parameter like ad_integration_preserve_custom_sssd_profile: true|false
then in the template
{% if ad_integration_preserve_custom_sssd_profile %}
[commands]
sssd-enable-logins = /usr/bin/sh -c "/usr/bin/systemctl enable oddjobd.service && /usr/bin/systemctl start oddjobd.service"
sssd-disable-logins = /bin/true
{% endif %}

@justin-stephenson @jakub-vavra-cz wdyt?

These realmd changes are just a workaround, it would be cleaner to have a proper solution to this (something similar to the realm join argument --do-not-touch-config). That said I don't know when that feature will be developed so for now I am fine with this proposed approach, ad_integration_preserve_authselect_profile would be more valid.
I would also add a comment in this template
Remove authselect command from `sssd-enable-logins` to avoid overwriting previous PAM/nsswitch changes, until https://issues.redhat.com/browse/RHEL-5101 is addressed.

Or maybe putting this in the README for the role variable is enough.

I would prefer to keep the template based implementation of realmd.conf, rather than moving to use lineinfile or blockinfile. What if we introduce a parameter like ad_integration_preserve_custom_sssd_profile: true|false
then in the template
{% if ad_integration_preserve_custom_sssd_profile %}
[commands]
sssd-enable-logins = /usr/bin/sh -c "/usr/bin/systemctl enable oddjobd.service && /usr/bin/systemctl start oddjobd.service"
sssd-disable-logins = /bin/true
{% endif %}

@justin-stephenson @jakub-vavra-cz wdyt?

These realmd changes are just a workaround, it would be cleaner to have a proper solution to this (something similar to the realm join argument --do-not-touch-config). That said I don't know when that feature will be developed so for now I am fine with this proposed approach, ad_integration_preserve_authselect_profile would be more valid.
I would also add a comment in this template
Remove authselect command from `sssd-enable-logins` to avoid overwriting previous PAM/nsswitch changes, until https://issues.redhat.com/browse/RHEL-5101 is addressed.

Or maybe putting this in the README for the role variable is enough.

https://github.com/linux-system-roles/ad_integration/pull/79

@jaredmarcuccilli can you test https://github.com/linux-system-roles/ad_integration/pull/79 ?

@jaredmarcuccilli can you test #79 ?

Worked exactly as expected, my PAM files are intact. Thanks for adding this so quickly.

merged #79


Unit Test for img_filter()

Create unit test function : test_img_filter()

should contain 2-3 edge cases
ensure tests cover all branches (i.e. tests cases for all outcomes of an 'if' statement)
write documentation for the unit test function
update code documentation if any changes occur
use pytest package to ensure the code passes the unit test

Added unit test for img_filter with pr #36


Emojis

Hey!
I use markdown-it-emoji it creates a type called emoji so I create custom rule, example:
emoji: (node, children, parent) => {
			return (
				<Text key={node.key} style={styles.emoji} numberOfLines={numberOfLines}>
					{node.content}
				</Text>
			)
		},

What I wanted to apply is a unique style for emojis when there is only emoji (and other non-strings).
Example:
;) ;) ;) -> Only Emoji
;) -> Only Emoji
;) lorem -> Paragraph
;) ipsum -> Paragraph
🔢  -> only Emoji
🔢 lorem -> Paragraph
There is another case that I pass in the markdown is that the emoji already formatted can arrive and I would like it to pass the right rule (type: emoji) and not the paragraph.
I have this:
const renderOnlyEmoji = (key, text) => (
		<Text key={key} style={styles.emoji} numberOfLines={numberOfLines}>
			{text}
		</Text>
	)

	const rules = {
		emoji: (node, children, parent) => {
			if (this.isOnlyEmoji(node.content)) { // <- if I could get the formatted emoji here, it will work
				return renderOnlyEmoji(node.key, node.content)
			}

			return (
				<Text key={node.key} style={styles.text} numberOfLines={numberOfLines}>
					{node.content}
				</Text>
			)
		},
		paragraph: (node, children, parent) => {
			if (this.isOnlyEmoji(message.txt)) {
				return renderOnlyEmoji(node.key, message.txt)
			}

			return (
				<Text key={node.key} style={styles.text} numberOfLines={numberOfLines}>
					{children}
				</Text>
			)
		}
}

...

<Markdown
					rules={rules}
					style={styles}
					mergeStyle={true}
					markdownit={...}
				>
					{message.txt}
				</Markdown>

ping to @iamacup

Sorry buddy this one has completely slipped past me,
can you print out the element hierarchy with the debugPrintTree prop
<Markdown debugPrintTree> ... </Markdown>

Let's check that the plugin is working properly, and what rule name(s) it is assigning the emoji stuff to

hahaha no problem
I use version 4.0.11 it doesn't seem to work, does not seem to work, I will try to update and analyze

Cool, it's working ... I think it's a bit complex to check if each message has only emoji, knowing that a structure is generated below
;) ;) ;)
body
-paragraph
--textgroup
---emoji
---text
---emoji
 ---text
---emoji

so it looks like markdown-it-emoji does not pick up already formatted emoji's - what you will have to do is either see if you can make that library pick them up or you will need to use something like https://github.com/mathiasbynens/emoji-regex and rewrite the paragraph rule to detect >textgroup>text containing an emoji and return your other rule - a bit of a pain i know!

Ah right!
I have this:
isOnlyEmoji(message) {
		if (!message || !message.trim()) {
			return false
		}

		return message.replace(emojiRegex(), '').trim() === ''
	}

Let me have a little play tomorrow, need to hit the sack right now!

Example

The idea is that when there is only emoji it has different styles leaving the emojis with different height when I get formatted I can handle it, when I get it via markdown that's the problem.

Hey @iamacup, any news?

Hey, really sorry about this, a million things came up recently - are you still working on this and still need help?

Hey
I gave a bypass for not finding a smart solution, but soon I must get back to that...

I think i am going to close this one - as it's unlikely any changes will be made to the core library and instead to solve this is going to involve a bunch of either

annoying specific render rules or;
a custom AST parser

which won't ship with RNMD
Good luck!


Merge password and 2fa page on user settings

Will be part of #2271

@lunny tests still fail (login enroll)

Codecov Report

Merging #2695 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2695      +/-   ##
==========================================
+ Coverage   27.12%   27.13%   +<.01%
==========================================
  Files          87       87
  Lines       17176    17172       -4
==========================================
  Hits         4659     4659
+ Misses      11837    11833       -4
  Partials      680      680

Impacted Files
Coverage Δ

routers/user/setting.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@lafriks fixed and rebased.

LGTM

LGTM


Bulk API

This article says "The Salesforce connector is built on top of the Salesforce REST/Bulk API." but this seems ambiguous. Is it on the Bulk API or is it on the standard REST API?
If it is the Bulk API, is it built supporting PK Chunking as per these articles?
https://developer.salesforce.com/docs/atlas.en-us.api_asynch.meta/api_asynch/asynch_api_bulk_query_processing.htm
https://developer.salesforce.com/docs/atlas.en-us.api_asynch.meta/api_asynch/asynch_api_code_curl_walkthrough_pk_chunking.htm
I would expect that it should use PK chunking as this would be more reliable with larger sets of data
If it is, is this an option that we can specify somehow or is it a feature that I should request to the product team? Thanks

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

ID: df56c0c4-ac20-76e5-3f99-7e5ed9c2d726
Version Independent ID: 8b224227-1cf7-6fa7-4bee-24d243509e45
Content: Copy data from and to Salesforce - Azure Data Factory
Content Source: articles/data-factory/connector-salesforce.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

Thank you for your detailed feedback.  We are looking into the issue, and we will respond when we have more information.

@o-o00o-o Just confirmed with ADF team, that "The Salesforce connector is built on top of the Salesforce REST/Bulk API." means the connector is built on top of both REST and Bulk API.  The
connector automatically choose one for better performance.  PK Chunking is used when the connector chooses to go with Bulk API.
Hope this info helps.

Great thanks for this, this is great news. This clarification should be added to the article.  It would also be good to understand both how the connector decides this as well as how we can tell which one was used for each execution.
Get Outlook for Androidhttps://aka.ms/ghei36

From: KranthiPakala-MSFT<EMAIL_ADDRESS>Sent: Wednesday, April 22, 2020 6:56:06 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Brett Gerhardi<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Bulk API (#52945)
@o-o00o-ohttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fo-o00o-o&data=02|01||cbc1b4a518624ce6b4a708d7e681d99e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637231317677666554&sdata=bbo5wIuI7QKjMYs29rpFV0qqEQkytgcL%2BCJfwN15pwQ%3D&reserved=0 Just confirmed with ADF team, that "The Salesforce connector is built on top of the Salesforce REST/Bulk API." means the connector is built on top of both REST and Bulk API. The
connector automatically choose one for better performance. PK Chunking is used when the connector chooses to go with Bulk API.
Hope this info helps.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F52945%23issuecomment-617567306&data=02|01||cbc1b4a518624ce6b4a708d7e681d99e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637231317677676548&sdata=UCVE%2B2vxXdYjUolNIaSZUC8auhV014IxJ8CwZFdF5VI%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADIPBZ3TGNEYPMMW4TNYD53RN2BHNANCNFSM4MNLEALQ&data=02|01||cbc1b4a518624ce6b4a708d7e681d99e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637231317677686544&sdata=Yz3dcjP7ZC5X%2BiXrtRG%2Boz2RHwZqGS%2FV80m5TYlO4QA%3D&reserved=0.

The PR for additional clarification has been created and it will be merged soon. Thanks again for pointing this out, I am closing the issue based on this update, please let us know if there is something else that we can help you with.


README settings for large images

Please, take a look at the following issue created by user:
https://github.com/datahubio/qa/issues/2
Acceptance criteria

[x] images in README rendered correctly with appropriate width (e.g., https://datahub.io/OriHoch/israel-traffic-violations-2012-2016-partial-2017#readme )

Tasks

[x] refactor sass to set max-width=100% of all img tags in /public/sass/_generals.scss

@anuveyatsu i would be a bit more selective and only set the max-width in the readme section of the showcase page. Was there a reason for doing this site wide?

@rufuspollock I don't think we need any img to have width of more than 100% of its wrapping tag. Note that setting max-width wouldn't break image's aspect ratio. But both cases would work.

@anuveyatsu i'd still suggest just fixing for the readme for now ...

FIXED


telink: blobs: Update blobs location.

Now Telink blobs are stored at wiki.telink-semi.cn

The following west manifest projects have been modified in this Pull Request:

Name
Old Revision
New Revision
Diff

hal_telink
https://github.com/telink-semi/hal_telink/commit/36283106d9b0b1d5fb3025a98be5a58007ac3d77 (develop)
https://github.com/telink-semi/hal_telink/pull/95
telink-semi/hal_telink#95/files

Note: This message is automatically posted and updated by the Manifest GitHub Action.


[Question] Plans for 1.14/1.15

Any plans on moving this to newer versions? Forge already has a 1.15 RB, although I'm not sure their API is enough for this mod, but I believe there is nothing preventing a 1.14 version
I really like this mod, so I'd really want to see it moving forward (:

there only one person working on EIO and he doesn't want to port EIO by himself to 1.15.x and future versions likely 1.14.x is being skipped and no backporting either the other devs have there mods first


error CS0012: The type 'JsonSerializerSettings' is defined in an assembly that is not referenced. You must add a reference to assembly 'Newtonsoft.Json'

I tried to just use the .dll's in a example project and I get this error when trying to do a simple test retrieving the last block with
new EthBlockNumberUnityRequest.
error CS0012: The type 'JsonSerializerSettings' is defined in an assembly that is not referenced. You must add a reference to assembly 'Newtonsoft.Json
In the sample project I see you have the package "Newtonsoft Json" but this is not a standard package and it does not show in the manifest so I was not sure how it was imported or from where, also the version included is nuget.newtonsoft-json 2.0.0 which  the unity docs state "This is a package intended for internal Unity Development Projects and as such this package is not supported. Use at your own risk."
as found here<EMAIL_ADDRESS>"com.unity.nuget.newtonsoft-json": "3.0.1" looks to be the suggested version but does not seem compatible .
After adding v3 directly to the manifest, the library showed up in the package manger and I was able to down grade to vs 2 and get it working, is there any plan to upgrade the dlls to use v3, thanks.

same here, it asks for the version <IP_ADDRESS> though.
any fix for that?


MB-16477: Update Initial SIT delivery address shown in edit service item modal

Jira ticket
Summary
This PR updates the edit service item modal to instead show the sitDestinationOriginalAddress in the Initial SIT delivery address column
Verification Steps for the Author
These are to be checked by the author.

[x] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.
Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Run make_server_test to verify all tests pass

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys.
[ ] Have been communicated to #g-database.
[ ] Secure migrations have been tested following the instructions in our docs.

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

Messages

:book:
:link: MB-16477

  Generated by :no_entry_sign: dangerJS against 9910a4cd55c828d8d861b2db9ff3a57efa34e3e5

https://github.com/transcom/mymove/assets/48488699/69b7f27b-7f4d-49ff-986b-c2f0fe65cc05


multiqc fails with: OSError: [Errno 2] No such file or directory

Hi,
multiqc fails with a No such file or directory error, and indeed, the temp directory seems to be missing. The error happened with 2 different samples and I'm using the development version of bcbio. Can it be a filesystem error or is it a problem with bcbio?

[2017-01-19T00:54Z] Run multiqc
[2017-01-19T00:54Z] Traceback (most recent call last):
[2017-01-19T00:54Z]   File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/bin/multiqc", line 4, in
[2017-01-19T00:54Z]     __import__('pkg_resources').run_script('multiqc==1.0.dev0', 'multiqc')
[2017-01-19T00:54Z]   File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/site-packages/setuptools-20.3-py2.7.egg/pkg_resources/__init__.py", line 726, in run_script
[2017-01-19T00:54Z]   File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/site-packages/setuptools-20.3-py2.7.egg/pkg_resources/__init__.py", line 1484, in run_script
[2017-01-19T00:54Z]   File "/pico/work/ELIX_prj6/esebesty/bcbio/data/anaconda/lib/python2.7/site-packages/multiqc-1.0.dev0-py2.7.egg/EGG-INFO/scripts/multiqc", line 31, in
[2017-01-19T00:54Z]     from multiqc import __version__
[2017-01-19T00:54Z]   File "/pico/work/ELIX_prj6/esebesty/bcbio/data/anaconda/lib/python2.7/site-packages/multiqc-1.0.dev0-py2.7.egg/multiqc/__init__.py", line 2, in
[2017-01-19T00:54Z]     from multiqc.utils import config
[2017-01-19T00:54Z]   File "/pico/work/ELIX_prj6/esebesty/bcbio/data/anaconda/lib/python2.7/site-packages/multiqc-1.0.dev0-py2.7.egg/multiqc/utils/config.py", line 31, in
[2017-01-19T00:54Z]     git_hash = subprocess.check_output(['git', 'rev-parse', 'HEAD'], stderr=subprocess.STDOUT)
[2017-01-19T00:54Z]   File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/subprocess.py", line 566, in check_output
[2017-01-19T00:54Z]     process = Popen(stdout=PIPE, *popenargs, **kwargs)
[2017-01-19T00:54Z]   File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/subprocess.py", line 710, in __init__
[2017-01-19T00:54Z]     errread, errwrite)
[2017-01-19T00:54Z]   File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/subprocess.py", line 1335, in _execute_child
[2017-01-19T00:54Z]     raise child_exception
[2017-01-19T00:54Z] OSError: [Errno 2] No such file or directory
[2017-01-19T00:54Z] Uncaught exception occurred
Traceback (most recent call last):
  File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 22, in run
    _do_run(cmd, checks, log_stdout, env=env)
  File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 102, in _do_run
    raise subprocess.CalledProcessError(exitcode, error_msg)
CalledProcessError: Command 'set -o pipefail; export TMPDIR=/pico/home/userexternal/esebesty/scratch/tmp && /pico/home/userexternal/esebesty/work/bcbio/data/anaconda/bin/multiqc -f -l /pico/scratch/userexternal/esebesty/bcbio_run/uuid_batch6597/work/qc/multiqc/list_files.txt -o /pico/home/userexternal/esebesty/scratch/tmp/tmp6rgRE7
Traceback (most recent call last):
  File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/bin/multiqc", line 4, in
    __import__('pkg_resources').run_script('multiqc==1.0.dev0', 'multiqc')
  File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/site-packages/setuptools-20.3-py2.7.egg/pkg_resources/__init__.py", line 726, in run_script
  File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/site-packages/setuptools-20.3-py2.7.egg/pkg_resources/__init__.py", line 1484, in run_script
  File "/pico/work/ELIX_prj6/esebesty/bcbio/data/anaconda/lib/python2.7/site-packages/multiqc-1.0.dev0-py2.7.egg/EGG-INFO/scripts/multiqc", line 31, in
    from multiqc import __version__
  File "/pico/work/ELIX_prj6/esebesty/bcbio/data/anaconda/lib/python2.7/site-packages/multiqc-1.0.dev0-py2.7.egg/multiqc/__init__.py", line 2, in
    from multiqc.utils import config
  File "/pico/work/ELIX_prj6/esebesty/bcbio/data/anaconda/lib/python2.7/site-packages/multiqc-1.0.dev0-py2.7.egg/multiqc/utils/config.py", line 31, in
    git_hash = subprocess.check_output(['git', 'rev-parse', 'HEAD'], stderr=subprocess.STDOUT)
  File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/subprocess.py", line 566, in check_output
    process = Popen(stdout=PIPE, *popenargs, **kwargs)
  File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/lib/python2.7/subprocess.py", line 1335, in _execute_child
    raise child_exception
OSError: [Errno 2] No such file or directory
' returned non-zero exit status 1

Endre;
Apologies about the issue and thank you for the report. MultiQC requires git, and the error message is from not having it in your PATH. bcbio installs this into the anaconda directory and I pushed a fix to the latest development that will inject this into the PATH automatically. If you upgrade to the latest development version and re-run it should work cleanly now. Thanks again.