Extend p4rt client to provide an interface for master election

@tomikazi  Going to merge this but if you have any comments please let me know and I address in a separate PR.


UBSan: thousandths should be longs

According to the UBSanitzer that ran during the unit tests, pwg_format_inches kept overflowing. thousandths and the other variables should be longs, not integers.

@michaelrsweet

UBSan caught this in one of your unit tests, so something is bugged either way.

@michaelrsweet The unit tests are tripping this up

@michaelrsweet @zdohnal Are the tests wrong or maybe there is a different bug that isn't the size of "thousandths"?

Will continue to investigate...

OK, so the calculation is for 100', which is 1200 (inches) * 2540 (hundredths of millimeters) = 3048000. Just concerned that long isn't guaranteed to be 64 bits...
I've pushed a different fix that eliminates the overflow:
[master 61f066e52] Fix an integer overflow issue in the PWG media size name code (Issue #668)


Docker Get Started Part 6 refers to discontinued features

File: get-started/part6.md
Hello,
Following the getting started guide, all was good until I got to part 6 and then everything seemed to go off the rails. It seems the guide refers to 'swarm mode' in 'docker cloud' which appear to have been recently discontinued. This is very confusing for novice users such as myself. I feel a 'getting started' guide being even slightly out of date is a disaster, as it is liable to turn off new users from the technology.

Ugh, that's not good.  I've just been going through the Getting Started bit myself, discovering docker-machine in the earlier section is now discontinued as well. :frowning:
Where's the info on swarm mode being discontinued?

Thanks for sharing the issue. This issue appears to be no longer applicable to the current docs. Closing the issue. If the issue still exists, please let us know.


Added Error when using [Focus] while not in development

Proposed change

More logging about Focus App (to avoid questions)
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the [development checklist][dev-checklist]
[ ] The code compiles without warnings (code quality chek)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

Codecov Report

Merging #532 (f8ac0b2) into dev (8c2f9e4) will decrease coverage by 0.13%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev     #532      +/-   ##
==========================================
- Coverage   59.56%   59.42%   -0.14%
==========================================
  Files         138      138
  Lines        5846     5854       +8
  Branches      840      841       +1
==========================================
- Hits         3482     3479       -3
- Misses       2081     2090       +9
- Partials      283      285       +2

Flag
Coverage Δ

unittests
59.42% <0.00%> (-0.14%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...DaemonRunner/DaemonRunner/Service/RunnerService.cs
0.00% <0.00%> (ø)

...rc/Daemon/NetDaemon.Daemon/Daemon/NetDaemonHost.cs
55.15% <0.00%> (-0.44%)
:arrow_down:

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 8c2f9e4...f8ac0b2. Read the comment docs.


Modal - focus ring around 'Close' button is not visible when user opens modal

Describe the bug
Noticed, that the focus ring is not initially visible when a user opens modal.
To Reproduce

Open modal (first example from docs can be used)
Check focus ring around 'Close' button

Expected behavior
Focus ring is visible
Actual behavior
Focus ring is not visible (see att. screencast)
https://github.com/semrush/intergalactic/assets/39348763/6067bb48-6310-4b20-a04f-1f0623ac432e

Hi, the issue was just fixed!
https://github.com/semrush/intergalactic/releases/tag/v15.10.0
Also need to note that focus ring will be visible only if modal is being opened by keyboard interaction


crypto/cipher: HTTPS throughput is poor

Here is a program that downloads the first 1 GiB from a large object full of zeroes stored in Google Cloud Storage, and reports its throughput. It also dumps a CPU profile to /tmp/cpu.pprof.
Running on an n1-standard-4 Google Cloud Engine VM in us-central1-a on Sandy Bridge CPUs looks like this:
% go version
go version devel +d3ffc97 Wed Jul 29 23:50:20 2015 +0000 linux/amd64

% go build -o read tmp/read.go && time ./read
Copied 1.00 GiB in 13.160669712s (77.81 MiB/s)
./read  12.67s user 0.88s system 102% cpu 13.164 total

In other words, it takes 100% CPU to do 78 MiB/s; CPU appears to be the bottleneck here. Compare that to curl:
% time curl -H "Range: bytes=0-1073741823" "https://storage.googleapis.com/jacobsa-standard-us/10g-big-chunks" > /dev/null
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 1024M  100 1024M    0     0   240M      0  0:00:04  0:00:04 --:--:--  240M
curl -H "Range: bytes=0-1073741823"  > /dev/null  1.72s user 1.01s system 64% cpu 4.262 total

Here it appears we've maxed out the network bandwidth available, using 64% CPU. The throughput in terms of walltime is 3x as high. Throughput in terms of bytes per CPU second is about 5x as high.
The CPU profile for the Go program looks like this:
11530ms of 13190ms total (87.41%)
Dropped 61 nodes (cum <= 65.95ms)
Showing top 5 nodes out of 45 (cum >= 330ms)
      flat  flat%   sum%        cum   cum%
    7780ms 58.98% 58.98%     7780ms 58.98%  crypto/cipher.(*gcm).mul
    1930ms 14.63% 73.62%     1930ms 14.63%  crypto/aes.encryptBlockAsm
     900ms  6.82% 80.44%     8670ms 65.73%  crypto/cipher.(*gcm).updateBlocks
     590ms  4.47% 84.91%     3550ms 26.91%  crypto/cipher.(*gcm).counterCrypt
     330ms  2.50% 87.41%      330ms  2.50%  crypto/cipher.fastXORWords

I know that it's possible to do some AES cipher work in hardware on modern Intel CPUs, and presumably that's what curl is doing via OpenSSL. Has this been considered for crypto/cipher in Go?

/cc @agl

@jacobsa crypto/aes already uses AES-NI when possible for the AES block calculations (I believe that's the crypto/aes.encryptBlockAsm in your profile there). The internet tells me that the Galois field multiplication that's at the top of your profile (implemented in Go right now) can be done efficiently using the PCLMULQDQ instruction. Maybe that makes up a large part of the performance gap to OpenSSL.
I can't locate any previous discussion of GCM-specific hardware acceleration in issues or on the mailing lists.

Thanks to the inspiration from @cespare I wrote an implementation based on this Intel whitepaper that's about twice as fast. We could probably do better by incorporating the techniques for parallelizing blocks from that paper, but it's a start.
I'll clean it up so that it works on other architectures and older CPUs, then send a CL.

How does this play with https://groups.google.com/forum/#!msg/golang-codereviews/m5QTnSUZU6c/Jc5yaMyF2_QJ ?

Note that we do have https://go-review.googlesource.com/#/c/10484/ pending for 1.6.

@agl: Awesome, this is where I was hoping to go eventually anyway. I only wish I had known about this before I spent a day on my CL. :-(

For the record, that CL was merged as eedaf9ac8b3a7b91d9a033d902b00eccf185d36f. That appears to help a lot; not sure if it's good enough to call the issue fixed or not.

@jacobsa, can you update this bug with a status update of where we're at, numbers-wise?

@jacobsa that commit eedaf9ac8b3a7b91d9a033d902b00eccf185d36f is not on master. (And CL 13020 is abandoned?)
Also, is your change superseded by CL 10484, which went in a couple of months ago? Or would it still be helpful for GCM-with-a-non-AES-block-cipher? (Is that a thing?)

Sorry, I copied from the wrong tab. I meant efeeee38c9aa30d2bdcb9d150c6e76ada01c5145, which is CL 10484.
@bradfitz: Will do.

For the record, here's what I did to set up a VM and the object:
gcloud compute instances create ubuntu-us --scopes storage-full --image ubuntu-15-04 --machine-type n1-standard-4 --zone us-central1-a --boot-disk-size 500GB
gcloud compute ssh --ssh-flag=-A --zone us-central1-a ubuntu-us
head -c $(numfmt --from iec-i 10Gi) /dev/zero > 10g
gsutil cp 10g gs://jacobsa-standard-us/10g
gsutil acl set public-read gs://jacobsa-standard-us/10g

Curl's throughput varies a decent amount, but this run showing 245 MiB/s seems representative:
% time curl -H "Range: bytes=0-1073741823" "https://storage.googleapis.com/jacobsa-standard-us/10g" > /dev/null
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 1024M  100 1024M    0     0   245M      0  0:00:04  0:00:04 --:--:--  245M
curl -H "Range: bytes=0-1073741823"  > /dev/null  1.48s user 0.79s system 54% cpu 4.184 total

Go 1.5.1 looks like this:
% go version
go version go1.5.1 linux/amd64

% go build foo.go && time ./foo
Copied 1.00 GiB in 12.830507037s (79.81 MiB/s)
./foo  12.19s user 0.84s system 101% cpu 12.833 total

And at tip:
% go version
go version devel +a92543e Thu Oct 29 23:16:27 2015 +0000 linux/amd64

% go build foo.go && time ./foo
Copied 1.00 GiB in 4.116806503s (248.74 MiB/s)
./foo  2.44s user 1.29s system 90% cpu 4.121 total

So looking pretty comparable with curl. Nice work.

I should have translated the output to be comparable with curl. Go 1.5.1 does 80 MiB/CPU-second, and tip does 248 MiB/CPU-second. So about as fast as curl now.

Can this be done for ARM and ARM64?

@gaillard, search for (or create) an ARM-specific bug. This one is closed and we don't reuse issues.

will do thanks


fix: keep this binding inside function screen while formatting it

Summary
Fixes #1055
The thisType was missed when trying to format the the parameters of a function

Test Plan
Added a test

Thanks!


Fix incorrect debug filename for bytecode from multiple *.rb

Fix #1243
With this fix bytecode from multiple *.rb with mrbc -g shows correct filename.
I hope someone to review.
[koji@macbookpro:~/work/mruby]$ cat one.rb
puts "hi from one.rb"

non_defined_func

[koji@macbookpro:~/work/mruby]$ cat two.rb
puts "hi from two.rb"

[koji@macbookpro:~/work/mruby]$ mrbc -g -o prog.mrb one.rb two.rb
[koji@macbookpro:~/work/mruby]$ mruby -b prog.mrb
hi from one.rb
trace:
    [0] one.rb:3
one.rb:3: undefined method 'non_defined_func' for main (NoMethodError)

I confirmed this is fixed.


Consider the addition of f format strings

Python lets you write:
world = 'world'
print('Hello {}!'.format(world))
print('Hello %s!' % world)
print(f'Hello {world}!')

Starlark lets you write the first two, but not the third. Of these, only the third lets you put variables where they occur and thus lets you scan the string with context, instead of hopping back and forth to the end to see what gets spliced in. The recommendation for modern Python is to use f-strings (e.g. https://realpython.com/python-string-formatting/#which-string-formatting-method-should-you-use). Should we add them to Starlark? (And for bonus points, I'd love if we could remove the other two flavours, but I expect we can't.)

Strong +1 from me!

Dupe of https://github.com/bazelbuild/starlark/issues/91 ?

Yep, will subscribe there.


[FIX] adding optional _rec-label to DWI

Pull request to fix issue #1089 this adds _rec-label to dwi.yaml template to match other MRI image modalities.
Also related to comment #105

I think this is fine but from a quick check most other mri datatypes have a short description of the optional entities on the markdown document on top of showing them in the filename templates generated from the schema.
My suggestion would be to do the same here, even if this leads to repetition of pretty much the same paragraphs in different sections of the same page.
I think we can later on figure out a way to "refactor" those repeats (macro?).

Added the short description of the optional entities in the DWI section of the markdown document
@Remi-Gau

@Lestropie don't you work with DWI data as well (judging from your work on https://www.mrtrix.org/)? Can you review (double-check) this? Should we add you to the raw-mri-dwi GitHub team that Chris linked to above?

Unless there is a scanner vendor performing inline motion correction as an integral part of image reconstruction prior to DICOM export, I am opposed to this as it currently stands. The distinction between with & without motion correction should be one between BIDS raw and BIDS derivatives, not of "different reconstructions" within the same dataset.
The only case I've come across thus far of differential reconstruction of raw image data that might be applicable to DWI is with vs. without Siemens prescan normalise. I know other sequences can export data both with & without this step from a single acquisition---I've never tried to do it from a DWI sequence but it's probably possible---so if you really want to explicitly list the rec entity for DWI, that's the only reasonable exemplar I'm aware of.

Should we add you to the raw-mri-dwi GitHub team that Chris linked to above?

Yep fine by me.

Sorry, but I am not sure whether I understand what you are opposed to?
This pull request only lists adding _rec-label for dwi, because

I collaborate with an MR physics group that works on reconstruction algorithms, so I sometimes have data that are reconstructed in different ways
this was inconsistent between dwi and other MRI modalities and seemed a straightforward fix.

Pinging @francopestilli for another DWI view

dear @dorahermes I see the issue here.
We are working on a BIDS BEP for DWI data derivatives (we = mostly @arokem @Lestropie and I).  In our understanding there is a distinction between raw data and derived data. Hence (I think) @Lestropie's opposition to this. This change
Yet, you mention that the other MRI modalities have a similar filed. Can you point me to a couple of examples?
If the goal of _rec-label is to describe a type of reconstruction say linear combination of coils or weighted sum or sum of squares etc, that this seems to be a helpful addition.
I made the following comment at the location where I suggest you make a change to your pull request. I think the comment you made to the requested change, might be the source of confusion.
(1) The _rec-label should not be used for post DICOM operations, such as motion correction.
(2) The label should be used to report the type of reconstruction used only. If in a special case a reconstruction also implements pre-DICOM conversion reconstruction that would be fine.
Perhaps a simpler comment would be all we need here. For example:
"key/value pair can be used to distinguish different reconstruction algorithms (for example one using sum of squares or another using a non-linear combination of the coils)."

Yep: I don't have an issue with the prospective addition of "_rec-XXX" to the DWI section, my problem is specifically with motion correction being the exemplar of such, as that is to my knowledge exclusively a post-processing operation, not a scanner reconstruction operation prior to obtaining DICOM data. Even if there is a group doing motion correction on the scanner, it would nevertheless still not be a good idea to use that as the exemplar, for the same reason. Disambiguating between with & without prescan normalise, or sum-of-squares and SENSE1, would be better.

thanks for explaining, I agree - I changed it to specifically state pre-dicom and added @francopestilli 's example:
The OPTIONAL rec-<label>
key/value pair can be used to distinguish different pre-dicom reconstruction algorithms (for example one using sum of squares or another using a non-linear combination of the coils).

Thanks @dorahermes looks good to me.

Thanks @dorahermes


Add support for escape sequence in string literals

Description
Add support for escape sequence
hi bhai
  bol bhai "mujhe chor do \"mere\" quotes pe";
bye bhai

and unicode character support
hi bhai
  bol bhai "Made with \u{1F48C} by bhailang community";
bye bhai

Domain
Bhai Lang
Type of Contribution
Enhancement
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of bhai-lang.

The output for the first code can be achieved by the following:
hi bhai
  bol bhai 'mujhe chor do "mere" quotes pe';
bye bhai


Issue accessing https://atlassian.github.io/data-center-helm-charts/

I am getting an error when accessing https://atlassian.github.io/data-center-helm-charts/ from my terraform play. I get "This site can’t provide a secure connectionatlassian.github.io sent an invalid response.
ERR_SSL_PROTOCOL_ERROR" when trying to get to that URL from a normal or incognito chrome browser.
What am I doing wrong?

@rhaskins looks like GitHub issue since it's GitHub pages. We'll raise a ticket with them.

@rhaskins unfortunately this is a GitHub issue and we are waiting for a response from the support team.
Thank you for raising this.

Suggestion: mirror this project somewhere else with more reliable service.

OK, I can again access the page. Thank you for your help!


Create a "const" folder and added constants

Create a "const" folder and added constants like contract address in this case

New pr #24 takes care of this


Show the Clipboard/Download Even When Repo Connected

If the editor is in github-connected mode, the clipboard/download buttons are hidden, and the only way to access the keymap is by committing it to the target repository. I think it'd still be useful to be able to access the keymap definition within the editor UI  when connected to a repo, even if it's just to view it it.
When the repository has branch protections enabled, it's worse in that there is no way to get the keymap out of the editor at all (#120 will provide a resolution for this).

It has crossed my mind, for sure. Actually when I first released the clipboard source this functionality existed somewhat accidentally because switching between keymap sources didn't necessarily clear out the old data.
Eventually when live keymap updates become possible I think there will still be value in being able to export the keyboard's version of the keymap to a GitHub repo or any other code-based source, if only for the benefits of revision history rather than leveraging GitHub Actions for firmware builds.
Before that, there are some fine details that would need to be sorted out to make sure that switching between sources works as expected. I think the simplest example is the fact that each source gives you access to potentially a number of keymaps for a number of keyboards, and whatever mechanism "copies" the data between the two needs to be able to verify that there isn't a mismatch. Trying to copy a keymap for a full-sized keyboard layout to a smaller ergo layout or vice versa would be a mess for sure.
Being able to at least spit out the code for the current keyboard state is a safe bet, though I might keep that a little out of the way to avoid confusion with the more typical workflows. I'll update here when I have more thoughts about how I'd like to implement it.

Just a small update: I started to touch on an implementation here and immediately realized that part of the challenge would come from not having the original keymap source client-side (the parsed keymap data doesn't include any nodes that I don't currently manage like conditional layers or even source code comments and all of this is needed to properly merge changes back into the document).
This raises some other questions -- I can update the API endpoint to return the keymap's raw devicetree content along with the parsed data structure, but now that the app is fully capable of parsing and serializing devicetree code within the browser then should any parsing happen on the server side? I probably won't make a change as significant as that any time soon but I still want to give this some more thought before I'll get around to implementing this feature for the GitHub source.

Alright, it's pretty basic but I've implemented a dialog that let's you view keymap changes as they would be applied to the source .keymap file.


Add hubble diagrams and data dicts

Add Hubble data model diagram
Add Hubble data dictionaries

Something went wrong with PR preview build please check

Preview is available here:http://developers-pr670.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr670.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr670.previews.kube001.services.stellar-ops.com


[FEATURE]: refactored embedding layer to decouple with gpc

Describe the feature
In Shardformer, all layers accept process group as the arguments for distributed operations. We should remove the dependency on the gpc.

Closed by @FoolPlayer due to direct push.


Seperate sockets for Eaglew. ontw and Eaglew. tst

Both EagleW instances should be running their own socket, not to pollute eachother.

Don't see why this is an issue of COW. It is solvable in the configuration.


Improve test coverage

We need to add more tests for some stuff that was implemented before the test framework was integrated. Most of the library infrastructure is covered by tests, but not all.
Also many of the more complex image analysis routines do not have tests. They were tested when written, but it is not trivial to write an automated, thorough test for such algorithms.
Finally, we need to set up some form of continuous integration on GitHub, so these tests are run automatically for pull requests.

I will have a look at CI testing. #30 and #31 could have been easily caught if we would have this.

This applies also to the tests for PyDIP (which is already quite OK) and DIPimage (the DIPimage test suite is brand new, see #100, and certainly needs a lot of work to be meaningful).
Note that, for these two components, what needs to be tested is the glue code and the Python or MATLAB native code, not the DIPlib functionality; we don't need to test that twice.


cannot find symbol return locationManager.isLocationEnabled()

🐛 Bug Report

error: cannot find symbol
return locationManager.isLocationEnabled();
^
symbol:   method isLocationEnabled()
location: variable locationManager of type LocationManager
1 error
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':location_permissions:compileDebugJavaWithJavac'.

Compilation failed; see the compiler error output for details.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 33s
Finished with error: Gradle task assembleDebug failed with exit code 1
Version: 2.0.2
Platform:

[ ] :robot: Android

@Oga-Emma we cannot reproduce this error. I looks like you have an outdated environment. Please make sure you update your IDE and the Gradle build tools.
If you keep on having problems please provide us with some more information:

Output of flutter doctor -v;
The complete error output of the compiler.


chore(deps): update common-dev-assets digest to 398aa1f

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
0a577eb -> 398aa1f

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add flag for "spout i" to NOT tenderize display names

cc @mrueschman How does --preserve-case sound to you?
spout import CSV_FILE --preserve-case

or --preserve-casing

Either one is good with me!


Getting command not found ForeignShellFunctionAlias(extra_args=(), funcname='pyenv', shell='bash')

Thanks for resolving the last issue, I was facing. After the latest version and the new system I am getting mentioned error:
xonsh: For full traceback set: $XONSH_SHOW_TRACEBACK = True
xonsh: subprocess mode: command not found: ForeignShellFunctionAlias(extra_args=(), funcname='pyenv', shell='bash')
foreignshellfunctionalias(extra_args=(), funcname='pyenv', shell='bash'): command not found
xonsh: For full traceback set: $XONSH_SHOW_TRACEBACK = True
xonsh: subprocess mode: command not found: ForeignShellFunctionAlias(extra_args=(), funcname='pyenv', shell='bash')
foreignshellfunctionalias(extra_args=(), funcname='pyenv', shell='bash'): command not found
xonsh: For full traceback set: $XONSH_SHOW_TRACEBACK = True
xonsh: subprocess mode: command not found: ForeignShellFunctionAlias(extra_args=(), funcname='pyenv', shell='bash')
foreignshellfunctionalias(extra_args=(), funcname='pyenv', shell='bash'): command not found

Any guesses?
Thanks :)

Do you have pyenv in your $PATH?

in bash? yes

my ~/.xonshrc:
# Thanks: https://github.com/anki-code/awesome-xonshrc
# Cheatsheet: https://github.com/anki-code/xonsh-cheatsheet
# Bash to xonsh: https://xon.sh/bash_to_xsh.html

# 🤔 Is this correct?
source-bash ~/.bashrc

# The SQLite history backend saves command immediately
# unlike JSON backend that save the commands at the end of the session.
$XONSH_HISTORY_BACKEND = 'sqlite'

# What commands are saved to the history list. By default all commands are saved.
# * The option ‘ignoredups’ will not save the command if it matches the previous command.
# * The option `erasedups` will remove all previous commands that matches and updates the command frequency.
#   The minus of `erasedups` is that the history of every session becomes unrepeatable
#   because it will have a lack of the command you repeat in another session.
# Docs: https://xonsh.github.io/envvars.html#histcontrol
$HISTCONTROL='ignoredups'

# Remove front dot in multiline input to make the code copy-pastable.
$MULTILINE_PROMPT=' '

# Avoid typing cd just directory path.
# Docs: https://xonsh.github.io/envvars.html#auto-cd
$AUTO_CD = True

# Globbing files with “*” or “**” will also match dotfiles, or those ‘hidden’ files whose names begin with a literal ‘.’.
# Note! This affects also on rsync and other tools.
$DOTGLOB = True

# Don't clear the screen after quitting a manual page.
$MANPAGER = "less -X"
$LESS = "--ignore-case --quit-if-one-screen --quit-on-intr FRXQ"

# ==================================================================================
#  Xontribs
#  -------------------------
#  Official list - https://xon.sh/xontribs.html
#  Github topic with thumbnails - https://github.com/topics/xontrib
# ==================================================================================

_xontribs = [
    'cmd_done',
    'default_command',
    'xlsd',
    'pyenv',
]
if _xontribs:
    xontrib load @(_xontribs)

# ==================================================================================
# -- Aliases
# ==================================================================================

# cd-ing shortcuts.
aliases['.'] = 'cd ..'
aliases['..'] = 'cd ../..'
aliases['...'] = 'cd ../../..'
aliases['....'] = 'cd ../../../..'

# This is already in .bash_aliases
# List all files: sorted, with colors, directories will be first (Midnight Commander style).
aliases['ll'] = "$LC_COLLATE='C' ls --group-directories-first -lAh --color @($args)"

# Make directory and cd into it.
# Example: md /tmp/my/awesome/dir/will/be/here
aliases['md'] = 'mkdir -p $arg0 && cd $arg0'

# Using rsync instead of cp to get the progress and speed of copying.
aliases['cp'] = ['rsync', '--progress', '--recursive', '--archive']

# Grepping string occurrences recursively starting from current directory.
# Example: cd ~/git/xonsh && greps environ
aliases['greps'] = 'grep -ri'

# Copy output to current clipboard using xclip
# Example: echo hello | clp
aliases['clp'] = 'xclip -sel clip'

# SSH: Suppress "Connection close" message.
aliases['ssh'] = 'ssh -o LogLevel=QUIET'

# Run http server in the current directory.
aliases['http'] = 'python3 -m http.server'

# Open directory in file manager from terminal
aliases['='] = 'xdg-open'

# Activates poetry environnent without writing env dir path
aliases['env-pa'] = 'source @$(poetry env info -p)/bin/activate'

# This is already in .bash_aliases
# git: Beautify git log
# aliases['glog'] = "git log --color --graph --date=format:'%Y-%m-%d %H:%M:%S' --pretty=format:'%Cred%h%Creset -%C(yellow)%d%Creset %s %Cgreen(%ad) %C(bold blue)<%an>%Creset'"

# Generates random password
aliases['gen-pass'] = "openssl rand -base64 20"

# Get external ip of your system
aliases['ex-ip'] = 'curl ipblue.io/ip'

# Get ip of system
aliases['ip'] = 'hostname -I'

# nginx: Check syntax
aliases['nx-syntax'] = 'sudo nginx -t'

# nginx: restart nginx
aliases['nx-re'] = 'sudo systemctl restart nginx'

# Shorthand for checking status of service
aliases['service-status'] = 'sudo systemctl status'

# Download gitignore from https://github.com/github/gitignore by name
# aliases['gignore'] = 'f(){ wget -q<EMAIL_ADDRESS>mv<EMAIL_ADDRESS>.gitignore; unset -f f; }; f'

# Yarn: Add upgrade alias
aliases['yui'] = "yarn upgrade-interactive --latest"

# Node: Remove node_modules related files
aliases['rmnode'] = 'rm -rf node_modules yarn.lock package-lock.json'

No, in xonsh without the xontrib loaded. But it should be, since you load your .bashrc.

Is sourcing the bashrc bad thing?

I don't know, but defining the same aliases in both .bashrc and .xonshrc (in this case in a xontrib) might cause issues.

You can also try to remove the pyenv initialization from your .bashrc and check if it works that way.

I removed sourcing .bashrc and everything is working fine.
Thanks.


Upgrade dependencies of embulk-guess-csv_all_strings which were old unintentionally

Thanks!
Ah, we may want to upgrade them later, but they are tests. Not impacting the result pom.xml.


Gitcoin Bounty

This is the GitHub issue template dedicated for the Gitcoin bounty set up by the XDC Community.  The bounty will have specific requirements the contributor must follow to be considered for the bounty.  Please see specific requirements.
Please follow the instructions if you need clarification or a step by step posted here: https://xdcf.atlassian.net/wiki/spaces/XDOC/pages/edit-v2/6193153
Judging Criteria
All submissions must contain clear and concise information with explicit instruction and images describing each step required to execute the process.  An accompanying video in addition to the written content is highly desired.
All submissions will be put through an industry standard plagiarism detection tool to ensure that all submissions are the work and ideas of the person submitting for the bounty.  If the plagiarism tools flags your submission you will be disqualified from participation in the bounty.
You will be provided with resources (if available) as an aid to help you get started.  The sole act of submitting content for the bounty does not automatically qualify you for the bounty.  Our team will review all submissions and will choose the submission whose content best fits the needs and requirements of the bounty.
Participants that are selected as winners for the bounty will be notified within 7 working days after the bounty has ended.
Description
Amount of Bounty
Foundry Grant. All monies paid will be in XDC.
What will you be creating?
You will be creating how to documentation for the XDC Network. Some documentation has already been created, and you will read through it to ensure it works as intended, reads clearly or if it needs better clarfication, then make those changes. Those items without documentation will be created from scratch.  Documentation will have images that describe or represents what's being done or described in your tutorial.
Details
Name or title of the tutorial

Overview
Here you can write the overview of what the tutorial will be about.

What you will learn
For example: In this tutorial, you will learn how to set up (insert name of tutorial) and use it to build, test and deploy a simple smart contract.
What you will do

Set up (Name of tool(s) you will be setting up)
Create a simple smart contract
Compile contract
Test contract
Deploy contract

Setting up the development environment

Creating a project (with images)
Checking the contract (sample code with images preferred)
Setting up the contract
Compiling the contract
Testing the contract (where is the contract going to be tested...a link will be great)

Deploying a contract on the XDC Network
This section can be used if you are deploying contracts on the network. Otherwise it can be ommited when you create your bounty.
Total Prize of Bounty
Total sum of up to $X,XXX worth of XDC for this event.
Each submission will be paid according to the amount of work submitted. Only one winner will be chosen and announced within 7 working days after bounty has expired.
Further Details
This event will be an ongoing collaboration to create "how to" documentation for the XDC Network. All documentation will be open source and uploaded here:
(insert link for either github, gitbooks, etc)
Resources
These are some of the resources you have available to you where you will find the necessary information to complete your submission.
XDC Remix
https://remix.xinfin.network/#optimize=false&runs=200&evmVersion=null&version=soljson-v0.8.7+commit.e28d00a7.js
Sample of Documentation
(Please add links to documentation here)

This is not a bounty from XDC Community.


Add GBP to currency options

Hi! This is only a simple addition and I don't even know if you're maintaining this any more but I've added GB Pounds as a currency that can be used.
Thanks!

Thanks!


Simplify TestName determination when running tests (no need for Closure)

By moving the defer recoverFromTestPanic into the synchronous text execution loop, we can eliminate the need for using a testNameFunc closure to get the test name. It's known within the loop when executing the test.
Tested that the correct test name is retrieved in case of runs + panics, parallel + synchronous.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b76a0f9) 93.24% compared to head (16ef81e) 93.24%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #360   +/-   ##
=======================================
  Coverage   93.24%   93.24%
=======================================
  Files          32       32
  Lines        1822     1822
=======================================
  Hits         1699     1699
  Misses        100      100
  Partials       23       23

Flag
Coverage Δ

micro
93.24% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

This is great!


Bug Report: Failure due to invalid characters

Describe the bug
When attempting to pull our existing tenant along with all resources and child resources, the pull action fails with the following error.
The specified AzOpsState file contains invalid characters (remove any "[" or "]" characters)! Skipping /home/runner/work/super-octo-bassoon/super-octo-bassoon/root/tenant root group (***)/contoso (contoso)/management (management)/aa_main_master (***)/mgmt/./microsoft.insights_metricalerts-test-datasource [mcaafunctionsv1] failure detected.json

The resourceId of the resource it is failing on is /subscriptions/redacted/resourceGroups/MGMT/providers/microsoft.insights/metricAlerts/Test-DataSource [mcaafunctionsv1] Failure Detected and it is just an alert setup against a function to detect if the function fails execution. I have a few of these but this is the first one AzOps encounters.
Steps to reproduce

Setup Azure Function and configure it to an application insights resource
Enable alert for failed function invocations within the app insights resource
Make sure your AzOps settings are configured to pull resources and child resources
Run AzOps pull

This is currently a known limitation as described in https://github.com/azure/azops/wiki/frequently-asked-questions#subscriptions-or-resources-not-showing-up-in-repository.

@daltondhcp Any comment on my previous note?

@PleaseStopAsking
I've started to encounter the problem on various AzOps installations and since the names are pre-defined I see no other option than excluding for now.


Select image uploads a corrupt file iOS 11

Issue is that if I select an image and post it to an API to save to the server from an iOS device with iOS 11, the file is corrupt and cannot be opened, if I take an image and post to API everything is fine. I do not have the issue on an older device with iOS 9.3.5 and Simulator with iOS 10.1 with the same app version.
Failure to fill out this information will result in this issue being closed. If you post a full stack trace in a bug it will be closed, please post it to http://gist.github.com and then post the link here.
Bug Information
Version Number of Plugin: 3.0.1
Device Tested On: iPod Touch iOS 11.1.2, iPod Touch iOS 9.3.5, iPhone 6S iOS 11.1.2
Simulator Tested On: iPhone 5 iOS
Version of VS: Visual Studio 2017 Enterprise
Version of Xamarin: Xamarin Forms <IP_ADDRESS>42
Versions of other things you are using:
Steps to reproduce the Behavior
Select image from library and upload to an API
Expected Behavior
Be able to view the image once in the final destination folder
Actual Behavior
when moved to final destination error: We can't open this file.
Code snippet
    public ICommand SelectPictureCommand { get { return new Command(PickPhoto); } }

    private async void PickPhoto()
    {
        await CrossMedia.Current.Initialize();
        MediaFile pic = new MediaFile("", null);
        if (!CrossMedia.Current.IsPickPhotoSupported)
        {
            return;
        }
        var filename = DateTime.Now.ToString("yyyyMMddhhmmss") + ".jpg";
        try
        {
            pic = await CrossMedia.Current.PickPhotoAsync(new PickMediaOptions
            {
                CompressionQuality = 100,
                PhotoSize = PhotoSize.MaxWidthHeight
            });
        }
        catch(Exception ex)
        {
            System.Diagnostics.Debug.WriteLine(ex.Message);
            return;
        }
        if (pic == null)
            return;
        Stream imgStream = new MemoryStream();
        pic.GetStream().CopyTo(imgStream);
        ImgStreams.Add(new NCRPics { FileName = filename, Image = pic });
        ImgList.Add(pic.Path);
        SelImage = pic.Path;
    }

Screenshots

Does it display locally on iOS?

I am not sure why you closed this issue, I have found a consistent issue, and after several test, when I take a picture with TakePhoto the file is uploaded without an issue, but when I select the image from the library it ends up corrupted, this is from the same device and the same image, it can only be the the plugin, I am missing something.
But I did notice when I take a picture the file size is about 500kb but when I select the same image it is about 1700kb. I have the compression quality and file size set to the same on both.


Fix animations on icon hover

Context
@DarrachBarneveld correct me if I'm wrong, but I think the original video you showed me had a nice smooth transition?
The timing and stutter of the hover is jarring at the moment:
https://github.com/codu-code/codu/assets/12615742/6393f80b-01c2-4543-b4af-9dfa11d15d6a
Expected Behavior
A nice subtle transition into/and out of colors.

Ah yes, the duration. Will update now


Runtime package with removed page does not install on BC22.0

1. Describe the bug
I have an app that is some years old now. We have declared some pages as obsolete in version 17.0. Now we're starting to clean up those obsolete pieces and therefore I have removed some given page from the app. Being declared as obsolete, the AppSourceCop correctly accepts my change and does not detect it as a breaking change.
The build pipeline ran fine and produced a runtime package. However, I am unable to install this runtime package on service. The error I get is
The metadata object Page xxxx was not found for the compiler with emit version 22004.
      The application might be incompatible with the current compiler. Please unpublish the application that contains the application object Page xxxx and republish it.
      Or, run a technical upgrade of the application database with the 'Force' flag set.

2. To Reproduce

Create an app that has a page that is declared as Obsolete = Pending.
Build this as v1.0.0 and publish it to a first service (service A)
Create a runtime package from v1.0.0 and install it on a second service (service B)
Delete the page in the app and recompile it as v2.0.0
Publish v2.0.0 to service A
Create a new runtime package from v2.0.0
Install this runtime v2.0.0 to service B

3. Expected behavior
I expect the deployment to succeed, because the compilation has also succeeded.
4. Actual behavior
The deployment fails with the error message above.
5. Versions:

AL Language: 11.0.787898
Visual Studio Code: 1.77.3
Business Central: platform 22.0.54512.0, application IT 22.0.54157.54635

I'm having trouble reproducing this. What I have done:

Create an app with a single page, mark it as obsolte pending
Publish v1 of that app to a local BC server
Create v2 of that app with the page deleted
Publish v2 of that app to a local BC server
Get runtime packages of both versions of the app
Publish/Sync/Install v1 of the app on a different BC server
Publish/Sync v2 of the app on the other sever as well
Uninstall v1 of the app
Upgrade the app to v2

All of the steps succeed for me unfortunately. Is there anything different from what you're doing?

Alternatively, can you check the windows event logs for the machine hosting the server where the runtime packages are being hosted on? They might contain a stacktrace that I can work from.

Unfortunately the stack trace on the event log from yesterday is gone. I am currently trying to reproduce it locally with the same packages that I was working with yesterday. It happened on one of our build machines. Not during build, but during deploy to a test service. It's pretty crowded there, so I don't have many details left.
I've just spun up a new fresh container and tried to replicate using the same identical runtime packages. The version that was installed to begin with (22.0.2), the version that broke installation (22.0.3) and the version that reverted my changes and then installed fine (22.0.4). Each one went up without problems.
I think I might have found something though. Let met test this through and I will get back to you.

OK. I have unveiled the arcane. This is a problem on my end and I just got confused by the new error message.
The cause was that on our test service there was a second app which contained a pageextension to the page that I have deleted. This causes my update to fail because - obviously - the base page disappeared.
I am closing this issue.


Moves e2e and perf tests Azure infrastructure to eastus

Description
e2e and perf tests started seeing Azure infra deployments errors a few days ago.
This PR moves the infrastructure to a different region.
It is dependant on new log analytics workspace and storage id in eastus. Once those are created, we need to update the AZURE_DIAG_LOG_ANALYTICS_WORKSPACE_ID and AZURE_DIAG_STORAGE_ID secrets in Git Hub.

Ping @artursouza

What about the storage accounts?

Yeah, that's what I was referring to in the PR description, those need to be created and the secrets in Github need to be updated before this PR can get merged.
I don't have permissions to update the secrets, and I can try creating the storage accounts in Azure, if that's ok with you.

Must merge to verify fix.


One most important feature is this

Other like extracting to component is useless because of impossibility to change template and file extensions. Personally im working with typescript and react-native and by creating index.js with package.json file is not resolvable by typescript.
https://github.com/theshukran/theshukran-react-utils/blob/979df2583e5e98d170781cc9088a2b5cd7627e90/extension/commands.js#L50-L70

implemented in fork https://github.com/CodersAKL/react-toolkit/actions


Only use hard coded ipus_per_replica if layers_per_ipu is default

What does this PR do?
Stops manual overrides of layers_per_ipu throwing errors when ipus_per_replica was not also updated
Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


support N==0

N==0 should work without a problem, it currently doesn't

Note: std::vector uses 1 as the minimum allocation size, even for char.


CXXODM-58 Add support for $where operator in builder.

The builder now supports syntax of the form:
auto results = User::find(mangrove::where("this.prev_score == this.high_score"));
The b_code and b_codewscope types are also supported.

Lgtm


fix(client): forum search link for English challenges missing block name

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

When I was testing #54294 locally, I noticed that the forum search link for English challenges aren't containing block name. I think we forgot to update the function when we renamed English challenges from "Step" to "Task".
This PR fixes the issue, and also adds unit tests to prevent regression.
Testing

Go to /learn/a2-english-for-developers/learn-greetings-in-your-first-day-at-the-office/task-6
Click the "Ask for Help" button
Check the forum search link

https://github.com/freeCodeCamp/freeCodeCamp/pull/54393
https://github.com/freeCodeCamp/freeCodeCamp/pull/54393
https://github.com/freeCodeCamp/freeCodeCamp/pull/54393
https://github.com/freeCodeCamp/freeCodeCamp/pull/54393
https://github.com/freeCodeCamp/freeCodeCamp/pull/54393

****

asdhgkj


Representable docs small update

I have applied @myabc 's suggestions on previous PR (#64) but have forgotten to open the PR.
So here it is!
cc: @apotonick

thanks @abinoam!


Deployment to heroku : BreadcrumbsOnRails detected it won't be overridden.

Hi,
When i deployed to heroku i have this error message.
"BreadcrumbsOnRails detected it won't be overridden. To use methods from this gem you need to call them using bootstrap prefix i.e. add_bootstrap_breadcrumb and render_bootstrap_breadcrumbs."
Can you help?

I tried several times and I can not reproduce the problem, sample here:
https://camaleon-cms-rails.herokuapp.com/


Create Contact form

Created Contact Form and Add Footer in My Portfolio

Added Form tag with three form elements.
Added types and placeholders for each element.
Collaborated with my Coding Partner @MariamOsman11
My Coding Partner styled my footer.

Thank you so much :)


Update dependencies versions

I recently had to re-install this package in a new environment and encountered some issues.
The very old pinned version of astropy was triggering a crash from setuptools which only triggered a cascade of more issues when trying to fix it.
All dependencies' versions were pinned and this leaves very small range to development: I propose here to pin only the core ones (astropy and numpy and leave thew rest to pip - or conda).
I also propose to bump the minimum version of Python from 3.8 to 3.9 as the former will reach end-of-life in about 4 months.
Also fixes #8.
Testing (see #16) should uncover any issue with these changes and only then it will make sense to pin versions.

I very much agree with the concept that Unit testing shall find problematic changes in the external modules and to drop the pinned versions until such issue come up. Following this idea, should we really keep the constrains for astropy and numpy?

I thought that maybe at least pinning astropy might be a good idea, since there are usually breaking changes between major releases (when I say "pin" I also mean simply what I did here, e.g. making sure astropy is recent enough, but preventing an automatic upgrade to the next major release, not really pinning a specific version).
I know numpy 2.0 caused a bit of issues in some other open-source projects, so I thought it might be beneficial to do the same for that dependency.

What is the policy of this project regarding mergin pull requests? is one approval sufficient? should we fix the threshold to 2?

This wasn't agreed upon. A minimal sufficient practice would be if the PR author request review from developers of his / her choice (those can add others if required) and PR is merged following their approval. We can indeed require at least to developers to concur, though.

I think that at least until we have a solid test/benchmarks suite 2 approvals should be required just to make sure...

can someone else additionally review this? I encountered again this issue...


Admin user fixes

Don't allow user to change his role or status
Allow user to change himself
Do not allow passwords to be empty
...

There is room for improvement as a user now always has to update his password, but at least it won't be set to an empty string.

replaced by #492


Hololens Windows UWP App Icon specifications missing from  existing documentation

Describe the issue
When adding the icons into the Player settings in Unity, it is not clear what size icons should be for the Hololens (1 or 2, doesn't matter). Considering that a Hololens app (for now) is rarely "cross platform", it doesn't seem to make sense that one would need all the icons in all the sizes requested.
In addition to this, the region right below the "Icon" settings in Unity says the following: "Not applicable for this platform".
If a developer adds every possible icon shape and scale possible, it is also not clear that using a PNG with an alpha will result in the Windows blue color background being added as a default. It is also not clear if the developer should use white for the icon symbol as a default recommendation or if white is specifically reserved for Microsoft apps.
Feature area
Current documentation in MRTK has no information about the Build/App icons whatsoever. The Unity Documentation has general UWP Windows App Icon information but lacks details specific to the Hololens devices. The Windows UWP App icons documentation seems to focus on Desktop apps and also lacks details for Hololens apps.
Existing doc link
Unity UWP App Icons
Windows UWP App Icons

After I created this issue I found this page Mixed Reality App Packaging which gets closer to what I would expect in terms of documentation. For whatever reason it is not in the first search pages of a well known search engine. I still believe it can be improved, in particular the Table section "Where is this displayed" (N/A). Also it doesn't mention the color topics i mentioned in the issue.

Also I am realizing I might have opened this in the wrong repo... This seems to be the  right place

Moved issue to Mixed Reality Documentation Repo. Related to this issue as well.


Requires update

Hello, not sure if this still active but recently i have this issue:

Same for me here

Thanks to both of you, I'm currently rewriting this theme in order to follow the same conventions as Material GitHub. It should be released and fixed these days.


Letsencrypt : do not check the optional cert.pem

This letsencrypt docker image https://github.com/janeczku/rancher-letsencrypt doesn't create the cert.pem required to pass this test.

Restarted build.

Thank you @fevrierR


Multi-Actions: Add support for Mistral AI

Description
Fixes: https://github.com/dust-tt/dust/issues/5169

Adds support for tool_call_id
Adds support for multiple function calls
Make sure we can interact with 8x22b in Dust apps
Fix our stitching of assistant message before tool message in the case of multiple tool messages

Tested with mistral-small-latest, mistral-large-latest  and open-mixtral-8x22b
Mistral wants as many tool messages (function for us) as there were tool_calls, function_calls for us).
mistral-small-latest does not seem to be able to do multiple tool calls at the same time but the two other do.

TODO:

[x] test with streaming over API
[ ] post-merge/deploy test of a non multi-action mistral assistant

Risk
Medium, could break some pre-existing Mistral interactions but unlikely.
Deploy Plan

deploy core

(streaming working as expected)

I have a single action assistant ready in prod to test post deploy

(pivoting for now on understanding the error condition of multi actions mistral pre this fix which is now well reported in the UI)

Also aligns function_call event implementations. Checked locally that streamed event are aligned when switching models.


"item could not be parsed" on every corrupted weapon

🐞 bug report
Describe the bug

Basically, any item that has a physical damage stat (which are all weapons I believe) will fail to parse.
This is because in src\app\shared\module\poe\service\item\processor\item-damage-processor.service.ts, line 67, the code tries to access weaponPhysicalDamage.value.tier.min, which is usually set in src\app\shared\module\poe\service\item\processor\item-quality-processor.service.ts, line 84+. But the if check in line 13 in item-quality-processor.service.ts skips this code if the item is corrupted, which makes an error occur.
I have no experience with typescript or this codebase at all, but removing the || corrupted should fix this issue, I believe.
Probably the same issue as #684.
🌍  Your Environment

OS:  Windows 10 x64
Version:  latest master

same issue here.
Rarity: Rare
Pandemonium Song
Short Bow
Bow
Physical Damage: 7-20 (augmented)
Elemental Damage: 8-111 (augmented)
Critical Strike Chance: 5.00%
Attacks per Second: 1.50
Requirements:
Level: 50
Dex: 26
Sockets: G-R-B-G-G-G
Item Level: 50
Socketed Gems are supported by Level 10 Faster Projectiles (implicit)
Triggers Level 20 Spectral Spirits when Equipped
+9 to Dexterity
24% increased Physical Damage
Adds 8 to 111 Lightning Damage
+25 to Accuracy Rating
Corrupted

0.6.27 (2020-05-08)

fixed corrupted weapons unable to parse (#684, #688)


Serving "/" leads to miss the first letter of directory

As stated, when I try to run updog from root or if I do updog -d / or updog -d \/, all links in http interface seems to miss their first letter, like /etc becomes <IP_ADDRESS>:9090/tc and so on
To reproduce:

updog -d /
open web interface
see what is a http link to /etc - it would be /tc

The hiccup is in the get_relative_path method:
def get_relative_path(file_path, base_directory):
    return file_path.split(os.path.commonprefix([base_directory, file_path]))[1][1:]

Replacing this function with os.path.relpath seems to fix things.

The hiccup is in the get_relative_path method:
def get_relative_path(file_path, base_directory):
    return file_path.split(os.path.commonprefix([base_directory, file_path]))[1][1:]

Replacing this function with os.path.relpath seems to fix things.

@adeutscher Could you test it on Windows as well? It seems to fix #23 as well

On Windows (Python 3.7.6), everything appears to go alright:

Running python -m updog -d / shares C:\, and subdirectories can be navigated into.
Running python -m updog -d <relative local dir> also works.

@Alveona Also just tested with a specific drive letter, more directly addressing #23.
Sharing with python -m updog -d T:\ successfully shares the T:\ drive on my Windows box and allows me to get into sub-directories.

@adeutscher Sounds great! Waiting for @sc0tfree

Hey @adeutscher and @Alveona - Thanks so much for working through this issue. I’m on a plane home from vacation and will dive in ASAP!

dive in ASAP

How is it going?

On Windows (Python 3.6.8 miniconda virtual env) with updog v1.4,  still same error.
when i want to go into the subfolder, it will miss the first letter of the subfolder's .


Remove goog.json

Should resolve this issue: https://github.com/BranchMetrics/web-branch-deep-linking/issues/393

@yntema completed regression testing against this branch - QA gives 👍


Upgrade to Cake3.

Hi, I'm working on a project that uses this plugin, I'm upgrading to Cakephp 3, so I was wondering if is in your plans to upgrade this plugin?
Thanks.

Every once in a while I think about doing this, but then whatever project I am working on doesn't require it anymore. If you're willing to pay for it, then I can sit down and write the code, otherwise I have no immediate plans that don't require alcohol and boredom.
Sorry.

I've got the code sitting on my machine as I contemplate tacking this task. But I haven't quite committed to it yet.

To be quite honest, if I were to reimplement this, it would be using a combination of Intervention Image and Flysystem.

hmm... those are worth looking into.

Just to note that I've created a plugin for Cake 3 which works similarly to this plugin.
https://github.com/davidyell/CakePHP3-Proffer

What @davidyell said. Though I personally think that route is a dead-end for future development.
Good work to @davidyell though!

becoming enlightened. thanks

Well we did chat about it in irc @josegonzalez ;)

Thank you for replying guys.
On Tue, Jul 28, 2015 at 1:24 PM David Yell<EMAIL_ADDRESS>wrote:

Well we did chat about it in irc @josegonzalez
https://github.com/josegonzalez ;)
—
Reply to this email directly or view it on GitHub
https://github.com/josegonzalez/cakephp-upload/issues/311#issuecomment-125703181
.

@davidyell what api changes are there between this plugin and yours? I know you modeled it pretty close and then decided not to push as a new branch, but it might help people upgrade if they have a "stable" upgrade path of some sort.

The behaviour configuration is the same, except on the thumbnailSizes as I used Imagine, which I kinda regret as it's so complicated.
As for actual api changes, quite a few, as you'd imagine. I've got a path class which deals with all the file naming, folders, etc, and also I've abstracted the ImageTransform into a class.
Otherwise the way the actual behaviour works internally is very similar. I've also added a few shells for certain tasks which I needed, such as regenerating the thumbnails and cleaning up old images.
I've not hit too many bug reports, but fleshing out the functionality a bit is on my todo list before a 1.0.0 release.

On a complete lark, I started rewriting cakephp-upload for 3.x. It is missing almost all of the current feature-set. Here's what I'm planning:

Create a separate, companion plugin that just does file upload validation. Standalone, use it or don't.
Fix issues where {primaryKey} isn't a valid replacement (like on entity creation).
Potentially re-add thumbnailSizes. Honestly, I might just chuck this and show in the docs how to create thumbnails of various sizes.
Test shells
Ensure polymorphic behavior works
Add some goddamn tests and write some docs.

I do plan on dropping a few features, but will document what to do in all cases so that the migration isn't too painful.
One positive change!  Want to upload to S3? Easy-peasy. Same for dropbox. Etc.
There aren't any docs, but here is what currently works (tested with uploading to S3):
$this->addBehavior('Josegonzalez/Upload.Upload', [
    'photo' => [
        'adapter' => function () {
            $client = \Aws\S3\S3Client::factory([
                'credentials' => [
                    'key'    => getenv('AWS_ACCESS_KEY'),
                    'secret' => getenv('AWS_SECRET_KEY'),
                ],
                'region' => getenv('AWS_REGION'),
                'version' => 'latest',
            ]);

            $adapter = new \League\Flysystem\AwsS3v3\AwsS3Adapter(
                $client,
                getenv('AWS_BUCKET')
            );
        },
        'rootDir' => WWW_ROOT,
        'path' => 'files{DS}{model}{DS}{field}{DS}{primaryKey}',
        'fields' => [
            'dir' => 'dir',
        ],
    ],
]);

yay alcohol and boredom!

open pr: https://github.com/josegonzalez/cakephp-upload/pull/314

Damn, this is nice :-) Any way to support you, like Flattr or equivalent?

Hey Jose, I have some free time, do you want any help?

I pushed my latest code. I need to test initialize and beforeMarshal (the latter fails miserably). If you want to make a pr to fix that test/test initialize, that would be dope. I'm shooting for 100% unit test coverage.

All is tested, just needs docs now. To keep the behavior simple, I'm not going to do file validations of any kind for now. That should be a separate plugin.

@mlumbroso I have a gratipay! https://gratipay.com/~josegonzalez/

I would dearly like to see the docs updated so I can start using it. (I'm trying, but I'm not that experienced with CakePHP, and certainly not with 2.x to 3.x upgrading, and guesswork isn't getting me very far :/ )

@vaughany I haven't had much time to work on the branch (other than get it to more or less where I want it to be). Might have some time this weekend. In the meantime, you may want to ping @ADmad on irc :)

Released a 3.x version.
Given that I don't use CakePHP for my day job, I'm only going to maintain that version, and any pull requests must have 100% test coverage and documentation, otherwise I won't merge and will probably close them after a set period of time.
Thanks for waiting for a release.

Thank you. :smiley:


Update QR Code Generator.py

1-Added functionality of saving qr code as an image locally.
2-Reading qr code from live camera source.
3-Reading qr code from local image.

Nice Work

ATTENTION 🚨🚨🚨
Don’t Forget To Follow Me On Github , and Add Your Name In _contributing.md if not found your name in _contributing.md it should be Labelled as SPAM.
STAR 🌟 this REPO for future HacktoberFest.


Error resolving database IP ( using docker )

Hi,
For some reason, gogs don't use the hostnames defined in /etc/hosts while trying to connect to the database.
Here is the error I found in the log files
.s/routers/install.go:63 GlobalInit()] [E] Fail to initialize ORM engine: migrate: sync: dial tcp: lookup dbgogs on <IP_ADDRESS>:53: no such host .
I also noticed that It worked when I installed gogs, but it saved the IP adress in  app.ini , this IP adress can change if I restart my mysql container, so I think that we should always use the hostname.
Thank you

Hi @rahal,
What version of the docker container are you using ?
FYI, in the latest version of it we added the possibility to refere to linked service as localhost using socat link. This way any service linked to the docker container will be available as localhost:service_port from the docker container itself.
If you give me a little bit more information about which image you are using I will try to take a deeper look into this problem.

Thank you @0rax !!
It didn't work with localhost,  then I used <IP_ADDRESS>:port then it worked :)  .
Thank


Using Firefox - Page fails to load when press edit.

Gitea version (or commit ref): 1.12.2
Git version:
Operating system:
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x ] Not relevant

Log gist:

Description
Using Firefox - Page fails to load when press edit.
Using Chrome - Very slow render of page when trying to edit page.
Screenshots

It should be caused by new code editor which will load gzipped 872.88 KB (original 3.71 MB) static files from the server when you first visit the website. But it should be cached after that. When you second visit the site, it should be faster.

It should be caused by new code editor which will load gzipped 872.88 KB (original 3.71 MB) static files from the server when you first visit the website. But it should be cached after that. When you second visit the site, it should be faster.

This seems to fix the issue.


Cannot specify fixed output file names

Version
3.0.0-beta.6
Reproduction link
https://github.com/noscript/vue-cli-nohash
Steps to reproduce
Add vue.config.js:
var ExtractTextPlugin = require("extract-text-webpack-plugin")
module.exports = {
  configureWebpack: config => {
    config.output.filename = "[name].js";
    config.plugins.push(new ExtractTextPlugin({
      filename: 'css/[name].css'
    }));
  }
};

What is expected?
  dist/vendor.js               89.20 kb               31.27 kb
  dist/app.js                  12.54 kb               8.00 kb
  dist/manifest.js             0.83 kb                0.47 kb
  dist/css/app.css             0.42 kb                0.26 kb

What is actually happening?
  dist/vendor.js               89.20 kb               31.27 kb
  dist/app.js                  12.54 kb               8.00 kb
  dist/manifest.js             0.83 kb                0.47 kb
  dist/css/app.3812306e.css    0.42 kb                0.26 kb
  dist/css/app.css             0.42 kb                0.26 kb

I really like the idea of slim vue.config.js to configure only necessary parts but nowadays vue-cli generates projects with chunks/splitting enabled (for cache busting reasons etc), so .js and .css files have a hash in their file names. Due to the nature of the webserver I use I need to have deterministic file names. I managed to get rid of the hashes in .js files but stuck with css.
Using ExtractTextPlugin I can specify the file name but  I still get another css generated with a hash, which is also included in index.html.
P.S. I hope I do not need to bloat vue.config.js with entire output of vue-cli-service inspect to configure that.

Since you are adding an extra ExtractTextPlugin, you end up with two CSS files. You need to use tap('extract-css') to modify the existing ExtractTextPlugin, see the webpack-chain docs.


Create Downgrade.yml

Similar to https://github.com/SciML/SciMLBase.jl/pull/553.
See https://github.com/cjdoris/julia-downgrade-compat-action and also https://github.com/trixi-framework/Trixi.jl/pull/1771.

There is a bug in the GitHub action that fails if a package name contains an underscore (here P4est_jll), see https://github.com/cjdoris/julia-downgrade-compat-action/pull/2.

The GitHub action is fixed now and in principle it works with the newest version. I've set the lower compat bounds to the lowest possible such that after downgrading the tests still pass.
However, it seems that - uses: cjdoris/julia-downgrade-compat-action@v1 doesn't recognize the newest version v1.0.7 of the GitHub action although it's already released. Explicitly stating - uses<EMAIL_ADDRESS>fixes the version, but I'd rather not like to specify the exact version. Usually, @v1 should take the latest release, shouldn't it @ranocha?

Looks like GitHub is doing weird stuff here, see https://devopsjournal.io/blog/2022/10/19/How-GitHub-Actions-versioning-works
The @vXYZ specification makes it look at a tag vXYZ in the repo. The tag v1 in the repo points to an old version, see https://github.com/cjdoris/julia-downgrade-compat-action/tags
The maintainer has to update the tag manually to make it work as you (and I) would expect. Thus, it's fine to use<EMAIL_ADDRESS>explicitly.

Ok, I set the version back to v1.0.7.


Refactor Durable Task Health Check to Verify Each Component

Description

Refactor health check to verify the following components:

Leases Container
TaskHub.json
Control Queues
Work Item Queue
History Table
Instance Table

Related issues
AB#94982
Testing
Unit tests and verify offline

I will also look into integrating a health check into the Durable Task Framework myself


New Saga utils: Part 3 (route tree gen fixes)

Just directly use the generated actions

@keybase/react-hackers this is ready. this does a big cleanup to deprecate the manual actions in actions/route-tree. we should be using route-tree-gen instead


asking

why i get it? can you tell me how to deal it or explain the reason. thankyou!
Building prefix dict from the default dictionary ...
Loading model from cache C:\Users\gongh\AppData\Local\Temp\jieba.cache
Loading model cost 0.678 seconds.
Prefix dict has been built succesfully.
Postagger: Model not loaded!
Traceback (most recent call last):
File "E:/GitHubResourse/open_entity_relation_extraction_master/code/demo/extract_demo.py", line 37, in
words_postag = nlp.postag(lemmas)
File "..\core\nlp.py", line 94, in postag
word = WordUnit(i+1, lemmas[i], postags[i])
IndexError: Index out of range

The problem is that the ltp models are not loaded and they can be downloaded from http://ltp.ai/download.html, select ltp_data_v3.4.0.zip.

hello,i have already download ltp_data_v3.4.0.zip and put it on the folder model but it still show indexerror, that is why? Could you give me some advice?

@Max1110  Thank you for your attention. The question is too general. Can you post the code snippet with problem.

Hello, I also encountered this problem. What is the specific placement method of the model under ../../model? Can you directly put it into the ltp_data_v3.4.0.zip downloaded? Do you need to download the ltp-win-x64-release-zip ? Could you please tell me how to place the file under ../../model？

You only need to unzip ltp_data_v3.4.0.zip to the model

|-- code/
|   |-- bean/
|   |-- core/
|   |-- demo/
|   |-- tool/
|-- data/
|   |-- input_text.txt
|   |-- knowledge_triple.json
|-- model/  # 我已下载解压，该文件夹内都是 xxx.model 文件
|-- resource
|-- requirements.txt
|-- README.md
我已照上面的目录结构做了，但仍然运行失败，而且我注意到下载的东西内包含一个MD5.txt文件，意思是model被加密了是吗。其中 nlp.py 文件加载的词性标注模型路径，我打印出来是这样的 ../../model/pos.model    应该没错吧。作者有什么建议吗，我被这个折磨了好久 @_@#。谢谢。


undefined method 'server' for ActionCable::Channel::ConnectionStub

I'm trying to test successful subscribe with channel ...
and I'm getting this error .. because of calling stream_from ...
     NoMethodError:
       undefined method `server' for #<ActionCable::Channel::ConnectionStub:0x007f84a96be678>
     # /Users/yaman/.rvm/gems/ruby-2.3.1/gems/actioncable-<IP_ADDRESS>/lib/action_cable/channel/streams.rb:85:in `stream_from'
.....

server method is missing from ConnectionStub, I just tried to add it like this:
class ConnectionStub
  attr_reader :transmissions, :identifiers, :subscriptions, :logger

  def initialize(identifiers = {})
    ...
  end

  def server
    ActionCable.server
  end

  ...
end

and it works! .. any idea ?!
my original code is like this:
class ConversationChannel < ApplicationCable::Channel
  def subscribe_to_channel
    stop_all_streams
    stream_from conversation_id
  end
end

Hi @yamanaltereh,
How does your test look like?

@yamanaltereh
One more question: why do you override subscribe_to_channel in your channel class instead of defining subscribed callback? I guess, that's the problem; #subscribe_to_channel is an internal method, you should not patch it.

@palkan Ah, I see .. actually the problem was .. when I use subscribed I get this error:
NoMethodError:
       private method `subscribe_to_channel' called for #<ConversationChannel:0x007fc1940c7418>
       Did you mean?  unsubscribe_from_channel
     # /Users/yaman/.rvm/gems/ruby-2.3.1/gems/action-cable-testing-0.1.0/lib/action_cable/channel/test_case.rb:201:in `subscribe'

after looking at the source code .. I thought it should be subscribe_to_channel ...

@palkan my rspec test is like this:
RSpec.describe ConversationChannel, type: :channel do
...
end

@yamanaltereh It turned out that the problem was with Rails <IP_ADDRESS> calling subscribe_to_channel during the channel's initialization (fixed by myself, btw, in 5.0.1).
Fixed by #3 and released as 0.1.1.

@palkan okay thanks, updated gem to release 0.1.1 .. and it's working now 👍


provide an executable of the toolbox

we are requiring Ruby language to be installed to use this toolbox, which can be a big limitation especially if being used directly by systems.
I suggested packaging the toolbox as an executable, I did it myself by leveraging this tool
https://github.com/pmq20/ruby-packer
and executing this command
rubyc --gem=3scale_toolbox --gem-version=0.7.0 3scale

https://github.com/3scale/3scale_toolbox_packaging/releases/latest
You have installers for

MacOS
Windows
RPM based linux distro (x64) like centos
Deb based linux distro like Ubuntu
Docker images

No dependencies on host O.S. not even ruby to be installed.

There was an issue on windows support https://github.com/3scale/3scale_toolbox/pull/113 and that's why windows package is not available on release 0.7.0. The issue has been fixed and already merged to upstream master.
We will release 0.8.0 soon to include windows package

I consider this issue as closed @lucamaf
Feel free to do that when you checkout available packages and see ruby is not required to be installed for the toolbox to work.

thank you!


Migrate to newer/more maintained jwt library:

Description

github.com/dgrijalva/jwt-go -> github.com/golang-jwt/jwt/v4
because of CVE-2020-26160. See: https://github.com/advisories/GHSA-w73w-5m7g-f7qc
Why is this needed

fixes security advisory. See https://github.com/tinkerbell/pbnj/security/dependabot/go.sum/github.com%2Fdgrijalva%2Fjwt-go/open
Fixes: #
How Has This Been Tested?

Manually tested using make run-server and make evans. Followed https://github.com/tinkerbell/pbnj/blob/main/docs/Authorization.md to test auth.
How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Hey @nshalman and @Raj-Dharwadkar when either of you have some cycles would you be able to review this, please?


Rename-DbaDatabase - skips -DatabaseName for new database and file name ()

Bug Report
DatabaseName omited for database and file renames
General Troubleshooting steps

[X] Verified running latest release of dbatools?
[X] Verified errors are not related to permissions?
[X] Can duplicate in new/fresh PowerShell.exe session?

Steps to Reproduce
PS> Rename-DbaDatabase -SqlInstance SRV1 -Database SAD_test_TTS-21140 -DatabaseName "SAD_test_TTS-21442" -FileName "<DBN>_<FT>" -Preview -ReplaceBefore -Verbose

VERBOSE: [Rename-DbaDatabase][10:55:50] Connecting to SRV1
VERBOSE: Performing the operation "Renaming Database [SAD_test_TTS-21140] to SAD_test_TTS-21442" on target "[SAD_test_TTS-21140]".
VERBOSE: [Get-DbaFile][10:55:50] Connecting to SRV1
VERBOSE: [Get-DbaFile][10:55:50] Adding paths
VERBOSE: [Get-DbaFile][10:55:50] 2 files found.
VERBOSE: Performing the operation "Renaming FileName D:\MSSQL\DATA\sad_test_TTS-21140.mdf to D:\MSSQL\DATA\SAD_test_TTS-21140_ROWS.mdf (on FileGroup PRIMARY)" on target "[SAD_test_TTS-21140]".
VERBOSE: Performing the operation "Renaming FileName D:\MSSQL\DATA\sad_log_test_TTS-21140.ldf to D:\MSSQL\DATA\SAD_test_TTS-21140_LOG.ldf (LOG)" on target "[SAD_test_TTS-21140]".
VERBOSE: [Test-PSRemoting][10:55:50] Testing SRV1
VERBOSE: [Test-PSRemoting][10:55:51] Testing SRV1 | <f:WSManFault xmlns:f="http://schemas.microsoft.com/wbem/wsman/1/wsmanfault" Code="2150858770" Machine="mymachine.domain.me"><f:Message>The client cannot connect to the destination specified in the request. Verify that the service on the destination is running and is accepting requests. Consult the logs and documentation for the WS-Management service running on the destination, most commonly IIS or WinRM. If the destination is the WinRM service, run the following command on the destination to analyze and configure the WinRM service: "winrm quickconfig". </f:Message></f:WSManFault>
VERBOSE: [Resolve-DbaNetworkName][10:55:51] Connecting to SRV1
VERBOSE: [Resolve-DbaNetworkName][10:55:51] IP Address from SRV1is 192.168.*.*
VERBOSE: [Resolve-DbaNetworkName][10:55:51] Getting computer information from SRV1.domain.me
VERBOSE: [Get-DbaCmObject][10:55:51] Configuration loaded | Cache disabled: False
VERBOSE: [Get-DbaCmObject][10:55:51] [sedna.veles-capital.org] Retrieving Management Information
VERBOSE: [Get-DbaCmObject][10:55:51] [sedna.veles-capital.org] Accessing computer using Cim over DCOM
VERBOSE: [Get-DbaCmObject][10:55:51] [sedna.veles-capital.org] Accessing computer using Cim over DCOM - Success!
VERBOSE: [Resolve-DbaNetworkName][10:56:02] Unable to get computer information from sedna
VERBOSE: [Resolve-DbaNetworkName][10:56:02] Resolving SRV1 using .NET.Dns GetHostEntry
VERBOSE: [Test-PSRemoting][10:56:02] Testing SRV1.domain.me
VERBOSE: [Test-PSRemoting][10:56:03] Testing SRV1.domain.me | <f:WSManFault xmlns:f="http://schemas.microsoft.com/wbem/wsman/1/wsmanfault" Code="2150858770" Machine="*"><f:Message>The client cannot connect to the destination specified in the request. Verify that the service on the destination is running and is accepting requests. Consult the logs and documentation for the WS-Management service running on the destination, most commonly IIS or WinRM. If the destination is the WinRM service, run the following command on the destination to analyze and configure the WinRM service: "winrm quickconfig". </f:Message></f:WSManFault>

ComputerName   : SRV1
InstanceName   : INST14
SqlInstance    : SRV1\INST14
Database       : [SAD_test_TTS-21140]
DBN            : {SAD_test_TTS-21140}
FGN            : {}
LGN            : {}
FNN            : {D:\MSSQL\DATA\SAD_test_TTS-21140_LOG.ldf, D:\MSSQL\DATA\SAD_test_TTS-21140_ROWS.mdf}
PendingRenames<EMAIL_ADDRESS>Destination=\\SRV1\D$\MSSQL\DATA\SAD_test_TTS-21140_ROWS.mdf; ComputerName=SRV1}, @{Source=\\SRV1\D$\MSSQL\DATA\sad_log_test_TTS-21140.ldf; Destination=\\SRV1\D$\MSSQL\DATA\SAD_TTS-21140_LOG.ldf; ComputerName=SRV1}}
Status         : PARTIAL

Renamed database
PS> Rename-DbaDatabase -SqlInstance SRV1\INST14 -Database SAD_test_TTS-2114
0 -DatabaseName "SAD_test_TTS-21442"

ComputerName   : SRV1
InstanceName   : INST14
SqlInstance    : SRV1\INST14
Database       : [SAD_test_TTS-21442]
DBN            : {SAD_test_TTS-21140}
FGN            : {}
LGN            : {}
FNN            : {}
PendingRenames :
Status         : FULL

But I think <DBN> must have value from -DatabaseName

huh ? The logic for both is right from my point of view, can you explain better ?

-preview lets you know what would happen in a more easier way than -confirm:$true .  for example, it showed in the first example that you don't have access to the underlying server, so the rename of the physical files would fail.
That being said, I think I got your point... you want the  placeholder to be not the original database name but the renamed one (both with -Preview and without it).
If you confirm that is indeed the issue, I'll look into it ASAP.

<DBN> needs to be renamed one, because there no point to have old name.


How do host process id, host thread id, GPU id and GPU stream id mapped to pid and tid in chrome://tracing?

A heterogeneous computing application usually has computing units of muitiple level, for example, a host process controls a GPU, and may uses several streams on a GPU; or a host process spawns several host threads, each thread controls a GPU and may use several streams on a GPU. However, in chrome://tracing there seems to have only two levels: process and thread. In my experience, rocprof --hip-trace doesn't have a good solution of this problem. Another tool rpt, which is provided in hcc, seems to always map the "quene" number to tid, and the pid is alwayse 1, as the rpt has these code:
    def printJSON(self, file, timeOffset=0):
        tid = self.queue
        file.write('{ "pid":1, "tid":%d, "ts":%d, "dur":%d, "ph":"X", "name":"%s", "args":{"dev.queue.op":"%d.%d.%d", "stop":%d } }' %\
                (tid, self.startTime/1000, (self.stopTime - self.startTime)/1000, self.name, \
                    self.device, self.queue, self.cmdNum, self.stopTime/1000) )
        file.write(',\n')
Maybe the self.device is really the GPU ID, but I don't know what the self.queue really is. Is it GPU stream id? But in rpt it is mapped to host thread id. This map sometimes makes the visualization in chrome://tracing in confusion.

@ipe-zhangyz Closing ticket. Please feel free to re-open ticket if you still need assistance with this ticket. Thanks!


Not recognizing plaintext files in folder

Hi,
first of all thanks for the great plugin. It really helped solve my post-export issues.
I also wanted to use it as a temporary solution to link to a few plaintext files that are in my data structure.
Unfortunately, I have encountered a bug when I try to include all .txt files from a folder.
Most of the time the plugin doesn't recognize any of the files.

I tried changing the file structure, filename and file location, but nothing seems to yield any different results.
Once I managed to get the plugin to recognize only 2 of the 5 txt files, but I haven't been able to reproduce this issue.
As you can see, with JSON, everything works as normal, so I am confident that I am using the plugin correctly.

Thanks again for the nice piece of software, hopefully this feedback proves useful.
Cheers
-Vaisko

I did some testing on this. As it stands right now Godot doesn't consider .txt files to be resources, while .json files actually are a supported resource type.
The reason why it works for some of the .txt files is some internal editor ... creativity ... that somehow makes a .txt file a resource once it is opened at least once in the editor.  I added 3 txt files to the project and initially they were not detected because the plugin checks if the resource is actually visible to the resource loader (it runs a ResourceLoader.exists(filename)). Once I opened one of the text files inside of the editor (by double-clicking it), suddenly this check returned true (probably because there is some editor magic shoehorning a text-file into a temporary resource so the editor can handle them as if they were resources (likely courtesy of this ticket https://github.com/godotengine/godot/issues/21787).
However at runtime this will not work, e.g. you cannot load a txt file with load("res://path/to/some.txt") and therefore also the resource group's load_* methods wouldn't work (that's why the plugin verifies that these resources actually can be seen by the resource loader). So as such, there is not a lot this plugin can do, because .txt files are not officially resources, hence they cannot be in a resource group.
I have added a warning to the code that should at least make people aware of this but ultimately the only fix is for the engine to recognize .txt files as resources.

Thanks for the super quick response.
Adding the warning should do the trick; although I am aware that I probably wasn't using the plugin as intended, I opened this ticket precisely because I couldn't find anything related to this in the documentation/readme.
Thanks again.


bigger Soroban callout in Welcome page

Something went wrong with PR preview build please check

Preview is available here:http://developers-pr282.previews.kube001.services.stellar-ops.com


🌐 Update Chinese translation for docs/tutorial/first-steps.md

as title

It seems this translation was updated at some point, could you please check in a new PR if the changes are still needed? Thanks! 🍰


Docker user directives

Feature or Bugfix

Bugfix

Detail
This PR is to add USER directive to all the Docker files
Relates
https://github.com/awslabs/aws-dataall/issues/864
Security
Please answer the questions below briefly where applicable, or write N/A. Based on
OWASP 10.

Does this PR introduce or modify any input fields or queries - this includes
fetching data from storage outside the application (e.g. a database, an S3 bucket)? No

Is the input sanitized? Yes
What precautions are you taking before deserializing the data you consume? N/A
Is injection prevented by parametrizing queries? N/A
Have you ensured no eval or similar functions are used? N/A

Does this PR introduce any functionality or component that requires authorization? N/A

How have you ensured it respects the existing AuthN/AuthZ mechanisms? N/A
Are you logging failed auth attempts? N/A

Are you using or adding any cryptographic features? N/A

Do you use a standard proven implementations? Yes
Are the used keys controlled by the customer? Where are they stored? N/A

Are you introducing any new policies/roles/users? No

Have you used the least-privilege principle? How? Yes, we are disabling the containers to run as root user. This change will create a system user without root privileges to run the containers.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Left 1 comment but otherwise LGTM! How did you test it?

@dbalintx I have seen your deployment in AWS with the latest changes.

[X] CICD succeeds
[X] frontend views appear as expected
[X] dataset creation succeeds

@dbalintx I have seen your deployment in AWS with the latest changes. I went ahead and tested some items there:

[x]  CICD succeeds
[x]  frontend views appear as expected
[ ]  dataset creation -->  it fails with the following error 'Error: EACCES: permission denied, mkdtemp '/tmp/jsii-kernel-XXXXXX''
[ ]  ML Studio user creation --> same error

@mourya-33 I think you need to provide permissions to /tmp for the new user as these processes (and probably also further ones) are writting to it

@dbalintx I added permissions for tmp directory for all the docker files

Tested the latest changes to this PR in an AWS internet facing deployment:

[x] CICD succeeds
[x] frontend views appear as expected
[ ] dataset creation --> it fails with the following error 'Error: EACCES: permission denied, mkdtemp '/tmp/jsii-kernel-XXXXXX''
[ ] ML Studio user creation --> same error

Getting the same errors as before with permission issues @mourya-33

@noah-paige i updated the permissions for temp dir and tested the dataset and environment creation as well.

@dlpzx is this ready to be merged or more testing is needed?


ADC1_CHANNEL4 Not working in ESP32-H2 (IDFGH-12492)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hai,
I'm using ESP-IDF Master branch and ESP32-H2 Custom board.
I checked the ADC1 Channel1 and Channel 2 using adc one-shot example. It's working fine.
But ADC1 Channel 4 was not working, If I give 3V also I'm getting around 650-700 mV.
I referred the data sheet. this pin having functionality of GPIO5, FSPID, ADC1_CH4, MTDI. Is any additional settings need to use as ADC?
Best regards
Bose

Hai,
I referred the Errata of esp32-h2, I found the reason.


Wrong positioning if popper is child of positioned parent

CodePen demo

https://codepen.io/anon/pen/EXQeKp
Steps to reproduce the problem

Open https://popper.js.org/
Open devtools.
Find in tab Elements this element id="page-wrapper"
Set CSS props position: absolute
A lot of tooltips don't work (especially tooltips with top & bottom position)

What is the expected behavior?
The expected behaviour - all works good and the position of parent element won't crash the tooltips position.
What went wrong?

Any other comments?
Also in this cool lib - https://github.com/atomiks/tippyjs/issues/70
P.S. Your lib also f*cking awesome

I need a reproducible codepen as requested by the issue template you ignored.

@FezVrasta Okay. I will create Codepen demo. Sorry man.

@FezVrasta I add CodePen demo. Please recheck and if all good, reopen the issue. Thanks.

I can't see anything wrong in the codepen demo. The tooltip has more left offset because the absolute body is considered the boundaries element (rather than the default document) and then it adds some extra margin.

https://codepen.io/anon/pen/EXQeKp

@atomiks thks :)

Found that there is also a problem if the popper element is contained in a relatively positioned element. Don't know if it is the same bug or a different bug. Repro here: https://codepen.io/anon/pen/BdLPxw

Is there any neat workaround for this at the moment?

Den 29 aug. 2017 14:55 skrev "maxpaj"<EMAIL_ADDRESS>
Is there any neat workaround for this at the moment?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/FezVrasta/popper.js/issues/331#issuecomment-325654683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYAk9wBThTSAXp1_w0s2LndfzOO_xLSMks5sdAo1gaJpZM4OKNXK
.

Hi,
I can still reproduce the bug using the provided Codepen (https://codepen.io/anon/pen/EXQeKp). Same results in Chrome beta 63, Firefox 56, and Edge 40 (all on Windows 10).
After the page loads, the text is added which pushes the reference element down, but the popper stays in the old location. The location is not changed after scroll or resize.
Here's a gif showing the issue: https://i.imgur.com/squuhji.gif

@JosephusPaye  that is 1.12.4 which has that problem. Change the package in the CodePen settings to 1.12.7, and it's fixed

Cool, thanks!

Unfortunately the latest Popper (1.12.5+) has a bug when updating a popper's position usually when it's near the viewport edges. If the bug is not an issue for you, you can use tippy.standalone.js + the latest Popper.js instead.
Currently, Tippy uses 1.12.4 because of that issue. I'm hoping he is able to fix it soon so it can be updated...

For reference, we are talking about #251