failed (exitcode: -9) local_rank: 0 (pid: 1760809) of binary: /home/xxx/miniconda3/envs/swift/bin/python

Describe the bug
Hello, we encountered an error when fine-tuning GLM-4V-9B model. Our fine-tuning command is:
NPROC_PER_NODE=2 \
CUDA_VISIBLE_DEVICES=0,1 \
swift sft \
    --model_type glm4v-9b-chat \
    --model_id_or_path ../../models/ZhipuAI/glm-4v-9b \
    --dataset ../../data/substation/sft/train.json \
    --val_dataset ../../data/substation/sft/test.json \
    --num_train_epochs 3 \
    --sft_type lora \
    --ddp_find_unused_parameters true \
    --eval_steps 150 \
    --save_steps 1 \
    --output_dir ../../experiments/glm-4v-9b-sft \
    --deepspeed zero3-offload

when steps=1, the program will save the model, and then the following error will appear:
{'loss': 6.52929688, 'acc': 0.03333334, 'grad_norm': 23.93275214, 'learning_rate': 0.0, 'memory(GiB)': 10.31, 'train_speed(iter/s)': 0.01689, 'epoch': 0.01, 'global_step': 1}
Train:   0%|▎                                                                                | 1/531 [00:56<8:16:19, 56.19s/it]W0622 19:47:13.438884<PHONE_NUMBER>14144 torch/distributed/elastic/multiprocessing/api.py:851] Sending process 1760810 closing signal SIGTERM
E0622 19:47:17.565751<PHONE_NUMBER>14144 torch/distributed/elastic/multiprocessing/api.py:826] failed (exitcode: -9) local_rank: 0 (pid: 1760809) of binary: /home/xxx/miniconda3/envs/swift/bin/python
Traceback (most recent call last):
  File "/home/xxx/miniconda3/envs/swift/bin/torchrun", line 8, in <module>
    sys.exit(main())
  File "/home/xxx/miniconda3/envs/swift/lib/python3.9/site-packages/torch/distributed/elastic/multiprocessing/errors/__init__.py", line 347, in wrapper
    return f(*args, **kwargs)
  File "/home/xxx/miniconda3/envs/swift/lib/python3.9/site-packages/torch/distributed/run.py", line 879, in main
    run(args)
  File "/home/xxx/miniconda3/envs/swift/lib/python3.9/site-packages/torch/distributed/run.py", line 870, in run
    elastic_launch(
  File "/home/xxx/miniconda3/envs/swift/lib/python3.9/site-packages/torch/distributed/launcher/api.py", line 132, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/home/xxx/miniconda3/envs/swift/lib/python3.9/site-packages/torch/distributed/launcher/api.py", line 263, in launch_agent
    raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
============================================================
/home/xxx/miniconda3/envs/swift/lib/python3.9/site-packages/swift/cli/sft.py FAILED
------------------------------------------------------------
Failures:
  <NO_OTHER_FAILURES>
------------------------------------------------------------
Root Cause (first observed failure):
[0]:
  time      : 2024-06-22_19:47:13
  host      : industai-Super-Server
  rank      : 0 (local_rank: 0)
  exitcode  : -9 (pid: 1760809)
  error_file: <N/A>
  traceback : Signal 9 (SIGKILL) received by PID 1760809
============================================================

How to solve this problem?
Your hardware and system info

OS: Ubuntu 20.04.6 LT
CPU: Intel® Xeon(R) CPU E5-2678 v3 @ 2.50GHz × 24
GPU: NVIDIA GeForce RTX 3060 x 2
torch: 2.3.1

Additional context

Sending process 1760810 closing signal SIGTERM
Can you check your memory status?

same problem here, how do you solve this issue? thanks

Please check your memory status,
I really do think the process is killed by the system because of OOM

Please check your memory status, I really do think the process is killed by the system because of OOM

Thanks! After double check, the killed problem I have is different from [zengzwww]'s problem. Mine is SIGFPE, it's because of cuda version. I use H20 for training, and only cuda 12.3 or above version support H20.


[DNM] more tests to run in native

[DNM] More coming, this lonely one is just to check codecov
The tests/crypto/sha256 can also be run in the posix arch

Codecov Report

Merging #5984 into master will increase coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5984      +/-   ##
==========================================
+ Coverage   52.51%   52.66%   +0.14%
==========================================
  Files         406      408       +2
  Lines       39682    39821     +139
  Branches     7715     7716       +1
==========================================
+ Hits        20838    20970     +132
- Misses      15663    15671       +8
+ Partials     3181     3180       -1

Impacted Files
Coverage Δ

tests/crypto/sha256/src/sha256.c
91.91% <0%> (ø)

tests/crypto/sha256/src/main.c
66.66% <0%> (ø)

ext/lib/crypto/tinycrypt/source/sha256.c
90.69% <0%> (+5.81%)
: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 ce9a5a2...1cb4fbc. Read the comment docs.


Fix base_endpoints.rst User Activate table

Correct syntax error that was preventing the table from rendering properly.

thanks


Problem: http_response is volatile

This may prevent some optimizations.
Solution: make it immutable
In my naive benchmarking, the performance gain was minimal (if any). Yet, perhaps, it's best to keep it immutable as it may still be better.

This is already so since bd18432ab0e2d99bf6459205684e066b5ae0e8a0


OnClickListener for child item

Hey, great library! Was trying it out, but I can't seem to set an OnClick listener other than on an AbstractAdapterItem. How can I do that inside my recycler or the base adapter?

I sort of found a place to hook a listener, on onUpdateViews of an AbstractAdapterItem, but I had to make my classes inner classes so I could access the application context.
A more elegant solution would be better, though I think I'm missing something that you've done.

You can set set an OnClick listener at AbstractAdapterItem onBindViews(final View root).The purpose of this library is to make a different type of item can be really different listener,so can not set only one listener to deal with all item click operation.

Thanks, I had tried it there but it didn't work. I'm using onUpdateView.
Thanks again for working in the library, saved me a weekend!


fix fix do not set per_fragment_instance_idx

Proposed changes
add case  to test  pipelineX union

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


merge package from arlac77/npm-package-template

package.json

chore(devDependencies): update rollup:^0.59.3 from template
chore(devDependencies): add rollup-plugin-babel:^3.0.4 from template
chore(devDependencies): add babel-preset-env:^1.6.1 from template

:tada: This PR is included in version 6.0.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Enhance Coin Selection

Perhaps we can use the Nami Implementation as a basis for this?
https://github.com/berry-pool/nami/blob/main/src/lib/coinSelection.js

https://github.com/cardano-foundation/CIPs/issues/232

@nicolasayotte thanks for commenting on this.
The largestFirst/smallestFirst coinselection were implemented provisionally, and more advanced algorithms will be added to the library in the coming months (a project I'm currently working is having similar issues).
Note to all those reading this: the coinselection interface is a mere function type, so custom algorithms can easily be used with the Helios library (eg. wrapping whatever already exists in other libraries)


MC crash

crash-2022-11-07_18.53.26-client.txt
I guess my mc crashed due to errors, provided by moreweaponry

Is on Fabric or Quilt? As this is typically a common error for Quilt users. Also, is this on a modpack (whether it be custom or an established one)? Again, pretty common error for users who are making/using a quilt modpack.

It's Fabric :(

And is it a modpack? Compatibility issues can happen depending on the case. You should also provide whether or not this is on startup or if it happens at some point while actually playing.

I think I may have found the issue, let me know if I'm wrong though. The version that you're using (v2.2.5) uses Fabric API version 0.60.0 for 1.19.2, while you're currently using 0.60.0. I recommend updating More Weaponry to the latest version, which uses 0.64.0, however, if your mods limit you to loader version 1.14.9 (the most recent More Weaponry version uses loader 1.14.10), then I might suggest removing More Weaponry until you are no longer limited by this. Thank you for your patience with this issue and let me know if you have any more issues/concerns!

Side question: Is the mod 'Deeper And Darker' in the modpack you are using? If so, remove and try again, as it seems to be incompatible with my other mod 'Boleatte', so most likely will be incompatible with 'More Wepaonry' as well.

Alright soooo: I used 0.64.0 (and now there is 0.66.0) so thats not the case, buuuut I do have DAD in my modpack, but not in my life (I had to). So I guess that answers my issue. Though I would think that since Boleatte has its own dimension and stuff (I really have no clue of modding), it maybe is incompatible in that sense? Unless Weaponry and Boleatte change code in the same areas (which (not my) DAD also does).
Btw I need Boleatte to be more popular for Aternos to accept the mod. I like harsh air 🥶
Oh and curseforge is shitty rn and my "no valid data" comes from auth servers being down.


Add support for spring?

I am thinking invoker should support spring (https://github.com/jonleighton/spring). I have not worked out details yet but I am thinking of a flag in config file which will enable invoker to start spring server on each supported application.

Unfortunately, yeah. Related documentation:

... If you run a different sub command (e.g. rails server) then spring will automatically pass it through to the underlying rails executable (without the speed-up).

@swrobel @halilim thanks for the update.
@gnufied i think this issue can be closed.

closing this one. lets revisit if spring supports rails server.


#3292. Scala on windows broken

Remove usage of pathing jar as it breaks scala evaluator.
Minor fixes: removed unused imports, added files separator. Set addDynamicDir property to outDir as it's done in groovy in DynamicClassLaoderSimple.

:+1:


warnings due to tune test?

In the latest PR, I saw that running the test in the CI caused a lot of warnings to be thrown related to the test_tune.py.
Below is an excerpt and here is the complete version (seeRun Tests). Do you know why this is happening @zzhaozhe-profolio or @ZachariahPang ? Thanks!
tests/test_tune.py::TestTune::test_tune_run
  /home/runner/.cache/pypoetry/virtualenvs/bliss-UE3m6-Ul-py3.8/lib/python3.8/site-packages/aiohttp/payload_streamer.py:57: DeprecationWarning: "@coroutine" decorator is deprecated since Python 3.8, use "async def" instead
    def write(self, writer):

tests/test_tune.py::TestTune::test_tune_run
  /home/runner/.cache/pypoetry/virtualenvs/bliss-UE3m6-Ul-py3.8/lib/python3.8/site-packages/aiohttp/payload_streamer.py:68: DeprecationWarning: "@coroutine" decorator is deprecated since Python 3.8, use "async def" instead
    def write(self, writer):

tests/test_tune.py::TestTune::test_tune_run
  /home/runner/.cache/pypoetry/virtualenvs/bliss-UE3m6-Ul-py3.8/lib/python3.8/site-packages/_pytest/threadexception.py:75: PytestUnhandledThreadExceptionWarning: Exception in thread Thread-4

  Traceback (most recent call last):

We probably need to update aiohttp to fix this: https://github.com/aio-libs/aiohttp/issues/4842

We probably need to update aiohttp to fix this: https://github.com/aio-libs/aiohttp/issues/4842

Maybe raised by some recently updated package because I didn't see those in #180 as in https://github.com/prob-ml/bliss/runs/1724460865. I'll check that in detail later.

Maybe raised by some recently updated package because I didn't see those in #180 as in https://github.com/prob-ml/bliss/runs/1724460865. I'll check that in detail later.


Pin sent2vec

Description
Fixes #356
Checklist

[x] Added link to Github issue or Trello card
[ ] Added tests

Codecov Report

Merging #357 (f7fc029) into main (c588894) will decrease coverage by 0.16%.
The diff coverage is n/a.

:exclamation: Current head f7fc029 differs from pull request most recent head 09a2153. Consider uploading reports for the commit 09a2153 to get more accurate results

@@            Coverage Diff             @@
##             main     #357      +/-   ##
==========================================
- Coverage   27.54%   27.37%   -0.17%
==========================================
  Files          43       43
  Lines        2596     2590       -6
==========================================
- Hits          715      709       -6
  Misses       1881     1881

Impacted Files
Coverage Δ

wellcomeml/ml/frequency_vectorizer.py
35.84% <0.00%> (-2.34%)
:arrow_down:

wellcomeml/ml/transformers_tokenizer.py
21.42% <0.00%> (-0.93%)
:arrow_down:

wellcomeml/ml/keras_vectorizer.py
22.42% <0.00%> (-0.72%)
:arrow_down:

wellcomeml/ml/bert_vectorizer.py
92.72% <0.00%> (-0.26%)
: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 c588894...09a2153. Read the comment docs.

The coverage bot is messed up, but this will bring us back to 85%


epiphany: 3.34.1 -> 3.34.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/epiphany/versions.
meta.description for epiphany is: '"WebKit based web browser for GNOME"'.
meta.homepage for epiphany is: '"https://wiki.gnome.org/Apps/Epiphany"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.34.2 with grep in /nix/store/mjh3z0z6sdqnpbaba6jhpcknbxa1vdij-epiphany-3.34.2

directory tree listing: https://gist.github.com/19b3eabab577c9f420e4afb2d002b12c

du listing: https://gist.github.com/0ef588ef9e9be753e282266dc3952099

Rebuild report (if merged into master) (click to expand)

7 total rebuild path(s)
3 package rebuild(s)
3 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
epiphany
gnome3.epiphany
pantheon.elementary-gsettings-schemas

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mjh3z0z6sdqnpbaba6jhpcknbxa1vdij-epiphany-3.34.2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A epiphany https://github.com/r-ryantm/nixpkgs/archive/afec08b05efc8057dcd35135fbd2840e31433dcb.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mjh3z0z6sdqnpbaba6jhpcknbxa1vdij-epiphany-3.34.2
ls -la /nix/store/mjh3z0z6sdqnpbaba6jhpcknbxa1vdij-epiphany-3.34.2/bin

cc @lethalman @jtojnar @hedning @worldofpeace for testing.

https://ftp.gnome.org/pub/GNOME/sources/epiphany/3.34/epiphany-3.34.2.news

Seems to work.


is that razorpay php sdk support auth 2.0 .. can i make calls using access token instead of using basic auth token?

Steps to reproduce the behavior
is that razorpay php sdk support auth 2.0 .. can i make calls using access token instead of using basic auth token?
Expected behavior
its able to make calls using oauth2.0
Actual behavior
i am not able to use auth2.0 using php sdk.  not able to make a call using accessToken
Code snippets
No response
Php version
php v7.4
Library version
razorpay.php v2.8.4
Additional Information
i want to use oauth2.0 using php razorpay sdk.  is that possible to use oauth2.0 ?

please review it @ankitdas13


Best Practice: Securing the Context Cache

Is there a best practice for securing the context cache?
Example:
I just had a scenario where I had to wipe and reload my computer.
I restored the %userprofile%.Azure directory after the computer was wiped and, after re-enabling the autocontextsave, was able to log into my Azure account without reauthenticating.
While this was convenient for me, this tells me that anyone with my .Azure directory can impersonate me if they copy it to their own location until such a time that my credentials change.
An seemingly innocuous example:

A server administrator with access to the file server hosting my redirected profile with access to the could copy my cached credentials to their own profile

This opportunity might be taken if the server admin and Azure admin roles are separated, and the server admin doesn't have rights to Azure
Potentially, there is no way for me to know that my cached credentials have been lifted without reviewing object access audit logs on the server

This also has the potential to be overlooked, as the server administrator might be tasked with manipulating files and folders as part of their responsibilities

Azure might have no way to secure against this or effectively audit this access, as the source IP address might be the same or, at the very least, authorized

What would the documented best practices be for securing and/or auditing the Azure Context cache, and what is the best practice actions if the context cache is compromised?

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

ID: 337dc474-8869-7ca2-6d27-8804add430e1
Version Independent ID: f641b6fa-c346-ad1d-65a6-793a4bedcbdf
Content: Azure contexts and sign-in credentials
Content Source: docs-conceptual/azps-3.2.0/context-persistence.md
GitHub Login: @sptramer
Microsoft Alias: sttramer

@aydeisen Thank you for your feedback! I have assigned an engineer who can help with the next steps.

Hi @aydeisen,
Great question! The reality is that many stored credentials are going to be vulnerable to this sort of attack, particularly if they don't have a password required to access them specifically. Browser cookies have the same weakness. It's possible to copy the cookie file from one machine to another and access all the sites through them. The same thing happens if you use SSH without a password protecting your private key.
Similar to being able to wipe cookies between browser sessions, you can clear the context between uses. Disable-AzContextAutosave disables this default behavior. You could also store the context in an encrypted folder that the server admin does not have access to and import it with Import-AzContext -Path other-context.json.
Making sure that your user has the minimum required permissions and using resource locks to prevent edits and/or deletes where appropriate will help to mitigate any possible damage if your account is compromised. Azure does attempt to detect risky sign-in behavior and it can be setup to alert admins when this happens. As you noted though, this scenario wouldn't necessarily be caught by this sort of monitoring as the access would be similar to legitimate traffic.
If someone has gotten access this way, the Azure Activity Log has detailed information about what has happened to each resource, who made the change, and how it was accomplished. That will allow you to find out what was done and reverse as much as possible. Depending on the data type, the support team may also be able to assist with recovering any data that was deleted and you are unable to retrieve. What support can assist with is going to be situational- I can't make any guarantees about what is and is not possible.
Revoking access once the issue is detected is possible. Changing your password is always a good precaution but isn't required in the case where only a token is compromised.
When authenticating to AAD, you actually receive two tokens- your access token and a refresh token. Once the access token expires the refresh token is used to retrieve a new one without having to sign in again. The access token, which is what is used to actually modify your account, cannot be revoked, so the unauthorized party would have control of your account as long as that lasts. However, for that exact reason, these tokens aren't good for very long. The default setting is for each access token to last for only an hour. This can be configured to as little as 10 minutes, but the shorter this window is, the more often your system has to go back to Azure AD for a new token which impacts performance.
The refresh token can be revoked, and this is where it would be possible to cut off access. While it doesn't stop the person from immediately using your access token, it would stop them from being issued a new one. If you used a password login to generate the token, then changing your password invalidates the refresh token. If you used another method, like Windows Hello, then changing the password does not help, but the refresh token can still be revoked via Powershell.
If you suspect that your token has been compromised then you should run the Revoke-AzureADSignedInUserAllRefreshToken command to invalidate all the refresh tokens attached to your account. This includes app tokens like your Powershell context as well as web cookies. If there is any suspicion that your account is not the only one compromised in this way, which is very possible in the scenario you describe, then the account admin can revoke all refresh tokens for the organization using the Revoke-AzureADUserAllRefreshToken.


App API ID is invalid

Hi all,
I have create an app in https://my.telegram.org/apps, I have given, App Title, Short name, URL, Platform as Desktop and description.
I have got api_id, api_hash etc, When i am trying to implement in code, i am getting API_ID_INVALID, I have tried with different API_ID but same result. Searched hard no luck.

Any help highly appreciated.
Thanks in advance.

Does the connection work with these keys?
Seems like telegram carelessly rejects a lot of keys

Its working fine with these links

What methods are you calling?

Here is my code::
const { MTProto } = require('telegram-mtproto')
const phone = {
num : '*******',
code: ''
}
const config = {
id : ,
hash: '**********************'
}
const api = {
layer          : 57,
initConnection : 0x69796de9,
api_id         : ******
}
const server = {
dev: true //We will connect to the test server.
}           //Any empty configurations fields can just not be specified
const client = MTProto({ server, api })
async function connect(){
const { phone_code_hash } = await client('auth.sendCode', {
phone_number  : phone.num,
current_number: false,
api_id        : config.id,
api_hash      : config.hash
})
const { user } = await client('auth.signIn', {                      ---> Getting Error in this line (This method)
phone_number   : phone.num,
phone_code_hash: phone_code_hash,
phone_code     : phone.code
})
console.log('signed as ', user)
}
connect()

Well, I think that telegram just rejects new keys

Any suggestion of solving this problem?

@javeedrahman just use working keys (i.g. from webogram) :new_moon_with_face:

Thank you :)


Add Linux support

[x] #47
[x] #50
[x] Test compiling in WSL;
[x] Test compiling in Github Codespaces;
[x] #46;

As it is, this project have problems working on both Linux and windows at the same time. One of the reasons is that windows Powershell uses back-slashes "" and Linux use forward-slashes "/" by default. This feature might need a careful look to workout.

The simplest way to support different OS is just adding #ifdef checks for _WIN32, __linux__, etc.
With PR #53, this project supports and should work on Windows, Linux and WSL. It might work on Mac as well, but it wasn't tested on MacOS yet.


Refactored library

Hi!
First of all thank you for your great work on this lib. 🙂
Just to let you know that I have forked your library and I have added some features that I missed, plus a full documentation. It is breaking change from your API but not so hard to migrate. I think it is easier to use and it directly extends NodeJS stream Transform class.
I have added some tests too in order to improve stability and quality.
It is still not perfect but already used in production, maybe you could get advantage of it or even pick up some ideas. And I am not against new testers and contributors!
Cheers! 😉

Okay, have a nice day.


Add secret-agent cflags; resolve linter warnings

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

Comparison is base (65abc0c) 20.40% compared to head (a8429ac) 20.40%.

Files
Patch %
Lines

pkg/model/config.go
0.00%
1 Missing :warning:

pkg/model/http_server.go
0.00%
1 Missing :warning:

pkg/model/restore_request.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   20.40%   20.40%
=======================================
  Files          28       28
  Lines        1142     1142
=======================================
  Hits          233      233
  Misses        890      890
  Partials       19       19

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


cv2.optflow module not found

Expected behaviour
cv2.optflow.DualTVL1OpticalFlow_create()
Actual behaviour
cv2.optflow module not found
Steps to reproduce

cv2.optflow.DualTVL1OpticalFlow_create()
win10 , conda env
win10(e.g. x64)
<IP_ADDRESS>

Issue submission checklist

[ ] This is not a generic OpenCV usage question (looking for help for coding, other usage questions, homework etc.)

[ ] I have read the README of this repository and understand that this repository provides only an automated build toolchain for OpenCV Python packages (there is no actual OpenCV code here)

[ ] The issue is related to the build scripts in this repository, to the pre-built binaries or is a feature request (such as "please enable this additional dependency")

[ ] I'm using the latest version of opencv-python

Cannot reproduce:

pip install opencv-contrib-python

>>> import cv2
>>> cv2.optflow.DualTVL1OpticalFlow_create()
<optflow_DualTVL1OpticalFlow 0000018A18DD84D0>

Please read the installation guide https://github.com/skvark/opencv-python#installation-and-usage


⚠ HopeAlley Ecards has degraded performance

In ef1a486, HopeAlley Ecards (https://hopealley.com/ecards) experienced degraded performance:

HTTP code: 200
Response time: 7030 ms

Resolved: HopeAlley Ecards performance has improved in 418e74d after 6 minutes.


Coderwall form separate from preferences

is this intentional? seems weird from user standpoint, may be combine so you dont have to submit 2 separate forms

@mth4saurabh - What is the status of your above (Dec 17, 2014) comment?

This issue will probably be impacted by https://coderwall.com's parent company Assembly shutting down on December 6, 2015. Reference: https://news.ycombinator.com/item?id=10555710

This issue can be closed, as Coderwall was removed in PR #1702.


python310Packages.lifelines: 0.27.8 -> 0.28.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.lifelines is: Survival analysis in Python
meta.homepage for python310Packages.lifelines is: https://lifelines.readthedocs.io
meta.changelog for python310Packages.lifelines is: https://github.com/CamDavidsonPilon/lifelines/blob/v0.28.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.28.0 with grep in /nix/store/9m7ir8zs5wgqxkpvq1wnr778bd6v11j4-python3.10-lifelines-0.28.0
found 0.28.0 in filename of file in /nix/store/9m7ir8zs5wgqxkpvq1wnr778bd6v11j4-python3.10-lifelines-0.28.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.lifelines
python311Packages.lifelines

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9m7ir8zs5wgqxkpvq1wnr778bd6v11j4-python3.10-lifelines-0.28.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.lifelines https://github.com/r-ryantm/nixpkgs/archive/b7186fb1e496c2cd3e74a8a3bc00d18aaaacb894.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b7186fb1e496c2cd3e74a8a3bc00d18aaaacb894#python310Packages.lifelines

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9m7ir8zs5wgqxkpvq1wnr778bd6v11j4-python3.10-lifelines-0.28.0
ls -la /nix/store/9m7ir8zs5wgqxkpvq1wnr778bd6v11j4-python3.10-lifelines-0.28.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.lifelines
python311Packages.lifelines

Maintainer pings
cc @swflint for testing.

Why is marshmallow-sqlalchemy involved here?

Why is marshmallow-sqlalchemy involved here?

I must have pushed the wrong branch.

That makes sense.  Provided build & tests pass on Darwin, I'll approve.


CuPy support

We should test with numpy-groupies. CuPy provides bincount
For engine="flox" we could implement using scatter_add, scatter_max, scatter_min which are all apparently equivalent to numpy.ufunc.at
We'd have to avoid factorizing with Pandas unfortunately and use np.searchsorted or np.digitize; or use CuDF?

Turns out scatter_* are a repacement for ufunc.at not ufunc.reduceat. The former does not take an axis kwarg so we can't use it for now.
cupy 12 might have more ufunc support, so let's wait and see

Hi, CuPy maintainer here! We've created a dedicated tracker for ufunc features: https://github.com/cupy/cupy/issues/7082
Do you need reduceat for specific ufuncs or all ufuncs?

Thanks! Sounds it aligns with our v12 target 😄
cc/ @asi1024


Corrections in the email_settings view to record SMTPUser field and others. Fixes #3

The SMTPUser field is not writing correctly to the settings table. As a result, the SMTPTimeout Key is being duplicated in the database.
<input type="number" name="SMTPTimeout" class="form-control" value="<?= old('SMTPTimeout', setting('Email', 'SMTPTimeout')) ?>">
Corrected SELECT TAGS with double quotes for better formatting.

Awesome. Thanks so much for contributing! Look forward to more as this grows into the toddler stages lol.


User does not exist error

User does not exist
┌──(code㉿DESKTOP)-[~/ETHena/src/python/GUI]
└─$

I'm getting this error despite editing the apiKeys.go file
package Utils

// Please replace <NAME>, <KEY_ID> and <SECRET> with your respective values
func InitialiseKeys() {
	ApiKeys = map[string]([]string){
		"RandomName": []string{
			"xxxxxxxxxxxxx",
			"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx_Jcdis",
		},
	}
}

Where do I get the name from? can it be a random name or is it the name that displays in Luno dashboard when you're logged in?
I have tried using;
the name in the dashboard
the label of the keyid

Hi,
Thanks for bringing up this problem, to my understanding, you have entered correctly a name, API key and API secret within the NAME, KEY_ID and SECRET spaces outlined and you have tried to run ETHena using the GUI but have been stopped by a 'user does not exit error'
Solution
Please try entering the name which you have entered within the  space into the 'Name' section of the GUI so the system can locate which user's key and secret you are trying to access.
Please let me know if there is follow up error or if this fixes the issue.

Solution
Please try entering the name which you have entered within the space into the 'Name' section of the GUI so the system can locate which user's key and secret you are trying to access.
Please let me know if there is follow up error or if this fixes the issue.

This solved my issue! thanks


Provide animation or place to animate while transitioning from one page to another

Hi,
First of all this is the one full fledge routing library I found. Having all the thing. Thanks for great work. Only part I feel is it will be great if there is a way to inject animation for transitioning from page to page.
It is just a case of beautification but a good to have to compete other part like react and angular.
Thanks for great work. :)

I'll think about how to do that. In principle, what a URL change generates (in this scheme) is a list of actions, so you could theoretically make those actions perform an animation ... but perhaps there is something the library can do to make that easier.

@rgrempel thanks :). Will looking forward to it. For me (while building SPA) first blocker was good url library and your library just blast that blocker. I'm back to elm. :+1:

fyi - I have seen some work by @etaque on this . I think he's also going to consolidate his router solution with the transition module.

yes, the hard work is being done by elm-transit (my router is already doing transitions, but it doesn't work with hashes yet).

@etaque and @rgrempel  is it possible that we can mix match both libraries. May be end up with one or two mature routing library. Like React-Router for react.

I thought about this a bit more, and I'm pretty sure that it is just a question of getting your location2action (or, now, location2messages) function to return some actions (or messages) that will kick off an animation.
To illustrate, I've updated one of the examples to do that ... when you click on the squares to spin them, you can then use the back / forward buttons, and the URL transition will now be animated (just like the original spin was). Here are the changes:
https://github.com/rgrempel/elm-route-url/commit/1618b1685e8981f3d48ddf8ae92f0657beca1287
But, this isn't really anything that elm-route-url can facilitate, since it has no idea what it means to animate your particular state change -- you just have to return messages that actually trigger an animation, whatever that means in your context.

@rgrempel - I think an interesting challenge occurs when one wants to implement a transition where e.g. one page slides out of view and a new page slides in, side-by-side. That means both pages are momentarily visible together at the same time, which means they both have to be rendered, which means they both must be in the model (I guess?), which isn't really desirable. Do you know of an elegant solution?
(I understand this is quite out of scope for this issue but still relevant).

I suppose the Elm-ish solution, as usual, is essentially to model the problem.
So, suppose you have a Page which represents one of several virtual pages, and you've got an update and view function that works with that Model -- so, the normal thing, you might say.
Now, if you want to animate a transition to the next Page, I suppose you'll have to add something like a Maybe AnimationState to the Page. And, the AnimationState would be composed of something like a Page -- that is, the next page -- and something that tracks the animation progress. And your update method would need a modification to kick off the animation (and progress it). And your view method would need to consult the AnimationState to know whether to draw just one Page or perhaps two side-by-side (at some stage of the animation, as tracked within the AnimationState).
Or something like that ... that's all a little vague, of course, but I think that would basically be the idea.
Now, you could imagine a generic AnimationState model type, where, provided with the update, view et. al. functions for the model type, it would wrap them in some way to provide update and view functions that generically handle drawing the various stages of an animation. That seems possible to me, without thinking it through very hard -- there may well be some pitfalls that don't occur to me.


Error connecting Spotify to application: Invalid redirect URI

I have been trying to get this to work for awhile now, and haven't had any luck. Tried reaching out to Netlify support but my email wasn't found in order to be able to submit for support.
Video of what's happening:

Didn't realize a similar issue had been brought to attention. Closing.


Druid Order Explorer not giving the Level 1 feat of the order

https://2e.aonprd.com/Feats.aspx?ID=319
You have learned the secrets of another druidic order, passing whatever rites of initiation that order requires and gaining access to its secrets. Choose an order other than your own. You gain a 1st-level feat that lists that order as a prerequisite, and you are now a member of that order for the purpose of meeting feat prerequisites.  You don’t gain any of the other benefits of the order you chose.
Bard Basic Muse Whisper's already do something similar, without checking for the selected muse you take with the feat, if possible this could be automated too.

With the character in question, do they have the druid class or archetype?

Archetype


PatternFly version 5 update

PatternFly version 5 update for react-component-groups.

@wise-king-sullyman Updated with review comments


experiment on undirected graph

Hi:
Thank you for your great work!
Now I'm trying to run your code on a unfeatured, undirected graph with a feature vector generated from Deepwalk. But the performance is really poor. I'm wondering if you can please provide some ideas on how should I set up a feature vector? Or how could I improve the performace?
Thanks!

If I understand you correctly, you generate DeepWalk features separately for each graph. Our matching procedure is based on finding correspondences based on locally similar neighborhoods (which is not given in case of DeepWalk features as inputs). Instead of using those features, you can use the torch_geometric.transforms.OneHotDegree transform. This should help the neural network to match nodes based on their structural similarities.


Modal standardization

Standardizing the modal template. Version two: getting one modal to display properly (not duplicate) with multiple renders on a single screen. Test as admin: http://<IP_ADDRESS>:3000/en/assignments/2/annotation_categories

@david-yz-liu Just pushed the code that I used for styling the download modal for Annotation_Categories.   To test: http://<IP_ADDRESS>:3000/en/assignments/2/annotation_categories then click on 'download' in the top right corner. I implemented the modal using a common partial and reuse of the content_for tags in app/views/layouts/_modal_dialog.html. I will probably divide the code for the upload and download partials as you suggested, but it would be nice to reuse the variable names set in Annotation_Categories/index.html

Commit e1f97d8: Third and fourth version of modals
Testing: http://<IP_ADDRESS>:3000/en/assignments/2/annotation_categories#
1.0 Click on 'Download' => should see the new modal
2.0 Click on 'Upload' => should see the upload modal with just the CSV content.
2.1 Click on 'Upload' then click the radio button for 'YML' => Should see YML content

Commit 099177d: Modified Graders screen.
Testing: http://<IP_ADDRESS>:3000/en/assignments/2/graders#
1.0 Click on 'Download' => should see new modal

Commit e4c8b46: Trying to resolve display issue.
Currently working in annotation_categories.
To test: http://<IP_ADDRESS>:3000/en/assignments/2/annotation_categories

In views/annotation_categories/index.html: swap which modal is rendered first - line 45 and 48 respectively


Issues with Kubernetes

Hi,
Not able to see the sonar-scanner binary while using the image in Kubernetes environment with jnlp slaves.
the directory /usr/local/bin/ doesn't have the binary or the link.
The Jenkinsfile looks like the one below.
pipeline {
agent none
stages {
stage('SQTest') {
agent {
kubernetes {
label 'sample-app'
defaultContainer 'jnlp'
yamlFile 'cloudprovider.yaml'
}
}
steps {
sh "sonar-scanner -Dproject.settings=testInputs/sonar-project.properties"
}
}
}
}
The cloudprovider.yaml has the image newtmitch/sonar-scanner:alpine and command['cat']
Any suggestions?
Thanks

Sorry, I'm not really familiar enough with Jenkins to make suggestions on how to properly run Sonar in K8S with a Jenkins pipeline.

Here is some example of Declarative Pipeline without k8s we use. You can see the entrypoint hack that is a little bit pain in the ***, most of Docker images have a docker-entrypoint.sh that does nothing as entrypoint, not the cli itself.
        stage('Sonar') {
            when {
		        expression { BRANCH_NAME ==~ /(master|develop.*|feature.*|V-[0-9]+\.[0-9]+)/ }
            }
            agent {
                docker {
                    reuseNode true
                    image 'newtmitch/sonar-scanner:3.3'
                    args '-e CURRENT_WORKSPACE=${WORKSPACE} --entrypoint='
                }
            }
            steps {
                sh 'TAG=$(${CURRENT_WORKSPACE}/jenkins/nginx/getPackageVersion.sh ${CURRENT_WORKSPACE})'
                sh '/usr/local/bin/sonar-scanner \
                    -Dsonar.projectVersion=${TAG} \
                    -Dsonar.branch.name=${BRANCH_NAME} \
                    -Dsonar.projectBaseDir=/${CURRENT_WORKSPACE} \
                    -Dsonar.host.url=http://vm-integration-chronos.cloud-espace.si.c-s.fr:9000/'
            }
        }

Jenkins runs the agent.docker.image with the cat cli as args to blocks the image when it boots so we absolutly need to remove the entrypoint. Then Jenkins runs each commands inside the container with full control.
As Jenkins is already in a docker, I need to clone the CURRENT_WORKSPACE var but that's just another layer of complexity not related to this issue. The TAG is computed inside the container using some shell to show we can do everything we want
source : https://github.com/RegardsOss/regards-frontend

thanks @serut. I just recently refactored the project a bit and moved the Dockerfile away from the ENTRYPOINT approach and instead went back to a purely CMD-based launch. I have seen the docker-entrypoint.sh approach as well and I haven't done that here yet. I ran across another approach that did this by having the entrypoint script deduce the way to start the underlying binary depending on how it was launched, but I haven't gotten to the point of doing that quite yet.
You might take a look at the images I pushed yesterday and see if those are a bit more copacetic to your needs without the entrypoint hack, see if that helps.
@shankarinece if you can try the new images as well, see if this issue goes away for you then.

I can't test, if I'm right your image v4 is for a recent Sonar, and the sonar v3 used the previous one. (a Java compatibily if I remember). So I can't test the image v4 new CMD with my existing sonar :(

Ahh, I see. I can build an image the most recent version of 3.x with a CMD-based approach if that helps?

It would helps me to test it 👍


new synonyms: increased/decreased femur length

Please add as synonyms:
increased femur length to [MP:0004348] long femur
decreased femur length to [MP:0003109] short femur
May also want to add similar synonyms to other bone terms (fibula, tibia, radius, ulna, etc)

Done for femur, ulna, radius, humerus, fibula, tibia, incisors, outer ears, gestation, hair, limb, mandible, maxilla, nasal bone, zygomatic bone, malleus, nails, rib, sternum, snout, toenails, sperm flagellum, vertebral body, vertebral column


Add Prettier, Addresses #3

Prettier is added in this pr.
Also pre commits and stuff are added, so now devs will almost ship prettiered code.wq

added prettier and formatted
added precommit hooks

aight ima merge


Add toString to transform related classes to make debugging easier

OK, I've already triggered the following builds for you:

QuickFeedback build

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Google Summer of Code: Floating PV

This issue gives an overview of the plan and progress of my GSoC  project.
The title of my project is "Floating Solar - Enabling modeling of floating PV with pvlib", where I will be adding floating PV temperature models in the pvlib python’s library. The official project description can be found here, and my mentors during this project will be @AdamRJensen and @kandersolar. My detailed plan is the following:
Calculation of ambient parameters for floating PV:

Albedo for water bodies (non-ocean) as described here
Is relevant to #1227

Module temperature for floating PV:

Gallery example of the expansion of the Faiman expression for floating PV (using the methodologies from here and here)

Addition of the Lindholm model (described here)

Addition of the Rahaman thermal model (described here)

As the project develops, I plan to link these tasks to individual issues and pull requests.

Please don't hesitate to give any feedback! Since this is the first time I am contributing to pvlib (or any other open-source software, tbh), I am really looking forward to learning as much as possible from all of you!

It looks like there is already a plan to add a floating.py module (as mentioned in #1937).

My GSoC is completed, thus this issue is closed. Thanks to everyone for their assistance!

Thanks @IoannisSifnaios for the great contributions!


Max count for profile post downloads

incorporate max_count

incorporate max_count

This would unfortunately change Instaloader's behavior in a backward-incompatible manner, since an invocation with --count and a mix of hashtags/feed/location and a profile would now behave differently in that the max count now also affects the profile and not only hashtags/feed/location.
Also, it is quite trivial for the user to only download a certain count of posts of a profile, e.g. by calling download_post() for each post of an islice()'d profile.get_posts(). There is no need to include that in the CLI.


Multi Tenancy

Whist Roles (and default user role) are now set up with Devise user Model as Enum integer, hence doing away with necessity for the likes of 3rd party Gems like Rolify etc, Multi Tenancy is not yet done as many 3rd party Multi Tenancy Gems cause problems with the signup process and in Network tab devtools (turbo stops working)  hence has been left out until satisfactory solution has been found.

Moved to other theme/repo using DHH Kolides hand rolled solution, no multi tenancy in this (standard) version, but adding UUIDs for users/services tables etc in another branch of this one.


Matlab Online instance provisioning fails

Across multiple test accounts, Matlab Online instance provisioning fails.

Open Matlab Online from the Hydroshare home page (after login)
Authorize
Observe error message

@Castronova is this still an issue or has it been addressed?

This should be in the MATLAB GitHub issue tracker. It has since been resolved, closing.


Introspection: Give advice if introspection result does not include any relations

Some database schemas do not include foreign keys (e.g. MySQL with engine MyISAM which does not support it, PlanetScale which disables it to enable online schema migrations, or older/suboptimal schemas that just doesn't have them for whatever reason). This will lead to very poor experience for the developer when using Prisma Client and Prisma in general (e.g. Studio).
We might be able to help the user out with a link to a good documentation article that explains the problem and offers help to work around the problem and improve the schema and/or database. We "just" need to recognize the situation after Introspection, and output the advice.

Potentially, this could leverage a new getDatabaseInfo utility in packages/internals that exposes (at least) the provider name and version. We can then pattern match on the flavor of the database provider we're using to log additional "help" messages to the user, e.g. when introspecting MySQL [MyISAM] (but I read elsewhere that we thought about only supporting InnoDB) and PlanetScale.


Guest can remove item from the cart.

Guest can delete an item/adjust quantity this quantity is released back into the general inventory
Acceptance Criteria

[ ] A guest can reduce the amount of an item that is in their order.
[ ] When they do so that quantity is released back into the available invnetory

Hey  @bglusman this is done I believe no?

Yup, though not from the cart tab only from the food/meal Rab it came from

Er, tab, on phone :-)


Magic RPC Error: [-32603] Internal error: User denied account access.

✅ Prerequisites

[x] Did you perform a cursory search of open issues? Is this bug already reported elsewhere?
[x] Are you running the latest SDK version?
[x] Are you reporting to the correct repository (magic-sdk)?

🐛 Description
I build a website based on nextjs and use magic link as the wallet solution.it works fine when user sign in with email ,but when connect metamask, when i can magic.wallet.getInfo, i got an error Magic RPC Error: [-32603] Internal error: User denied account access.
🧩 Steps to Reproduce

use magic link with metamask

🤔 Expected behavior
call magic.wallet.getInfo success
😮 Actual behavior
failed to call magic.wallet.getInfo and get an error Magic RPC Error: [-32603] Internal error: User denied account access.
💻 Code Sample
[If possible, please provide a code repository, gist, code snippet or sample files to reproduce the issue.]
🌎 Environment

Software
Version(s)

magic-sdk
^18.0.0

Browser
chrome

yarn
v1.22.18

Operating System
mac

Same here on a Nuxt 3 project.
We tested with Brave, Chrome ...etc same error only using Metamask.

Hey folks, we're getting the same error with Solana Devnet, do you have any idea about this?

Thank you for bringing this to our attention, it seems magic.wallet.getInfo() is returning the user's metadata, which couldn't be done on users who connect via 3rd party wallets.
We'll look into this and update here as soon as possible.

Hi, we have this issue too, any update on it?

@refex We should have an update by the end of month.

@puterhimself We're unfortunately not able to get to this today. We will have an update by end of the month.

@undefinedlol @refex @puterhimself @jdnichollsc @copostic
We just pushed out a fix to this issue. When you call getInfo() while connected with MetaMask the return value will be the user's wallet type, which in this case would be MetaMask.
When connected with MetaMask, doing the following
const walletType = await magic.wallet.getInfo();
console.log(walletType);

will return: {"walletType":"metamask"}

Closing this issue as it is now resolved. Please let us know if you believe it is not resolved and we will reopen it. Thank you!

hey everyone, sorry to keep dragging this on, but we tried this fix and it doesn't work for the BSC network even though its just changing the RPC URL and the chainID, is this intended or a bug?
please help me out @am-hernandez

@am-hernandez did you try with a Solana Wallet? 😅

@devops-prmsnls @jdnichollsc
This would be a separate issue, which you should feel free to create.
Magic Connect does not support Solana or BSC, only Ethereum, Optimism, Polygon, and Flow.  You can find an updated list of supported chains for Magic Connect here.


makepbcore- initial refactor of mediainfo to PBCore/IFI_technical CSV & MP3/AIFF/WAV support - fixes #266

This is an attempt to re-factor the technical metadata extraction so that it does not use the PBCore2 output. This mediainfo output will continue to change, so it's not a good source for scripting/transformation.
This patch uses the modern XML mediainfo schema instead, but there are still a few fields that use PBCore2XML.
This also has the perk of allowing us to update mediainfo on our computers as we are currently using an older version,prior to the PBCore2 updates.
Another note - it's worth looking at the PBCOre2 updates as some of them make sense for us to adopt. Things like Format Profile etc..

Areas where the PBCore2 XML output are still being used are:

Timecode parsing - I'm not updating this now as there are already many workarounds in the current scripts based on general Timecode inconsistencies.
date modified - mediainfo XML does not use ISO8601 with the T and the Z.
Audio - this should be altered prior to merging.

In order to test this:
cd ~/ifigit/ifiscripts
git pull
git checkout pbcore-update

to return to master, run update.py or git checkout master to return to the master branch.
As the goal of this update is to be able to use the latest version of mediainfo,it is recommeded that :

This should be tested on a machine not usually used for accessioning, eg. mac mini
mediainfo should be updated to the latest version
the testing will prove successful if this script produces the same output as the current implementation on es1/es2 with the old versions of mediainfo.

This PR has now scope crawled heavily in order to allow makepbcore to support WAV/AIFF/MP3.

Ok, so I performed the tests with:

current master and mediainfo 18.03
and
this branch and mediainfo 18.08
using an accessioned DV package, an FFV1 package, an XDCAM EX package and a concat package.

In a general sense, everything is improved. We no longer rely as much on the PBCore output. Things like compression_mode, library_names, scan-order to DV, Version 4 instead of Version 4/2 for format_version are all now showing up at last.
The only downside is with MPEG-2 and probably anything with a level/profile. Currently, instead of Main@High showing up (Profile@Level), we are just getting Profile.
This should be an easy fix as seeing as we are not doing MPEG-2 again any time soon, I think this should not block this PR.
Another note is that format_version for Matroska now says just 4, whereas before it said Version 4/Version 2.
I am going to merge this tomorrow as some of the extra metadata will be very useful for accessioning rawcooked material.

Hi Michael,
This is mostly trying to protect against future changes to the XML output
that I convert to CSV. Originally i made the mistake of harvesting a lot of
the pbcore2 output for our database records,thinking that this would be a
fixed output. The output has changed a lot in the last six months,so I'm
hoping that the mediainfo XML schema will stay the same.im now thinking
that this might not be the smartest either,and perhaps ffprobe could be a
bit more fixed.
I think I need to introduce unittests ASAP,as Travis would catch all this
stuff..

Reference Number
Donor
Edited By
Date Created
Date Last Modified
Film Or Tape
Date Of Donation
Accession Number
Habitat
backup_habitat
TTape Origin
Type Of Deposit
Depositor Reference
Master Viewing
Language Version
Condition Rating
Companion Elements
EditedNew
FIO
CollectionTitle
Created By
instantiationIdentif
instantDate_other
instantDate_type
instantiationDate_mo
instantiationStandar
instantMediaty
instantFileSize_byte
instantFileSize_gigs
instantTimeStart
instantDataRate
instantTracks
instantColors
instantLanguage
instantAltMo
essenceTrackEncodvid
essenceFrameRate
essenceTrackSampling
essenceBitDepth_vid
essenceFrameSize
essenceAspectRatio
essenceTrackEncod_au
essenceBitDepth_au
instantiationDuratio
instantiationChanCon
PixelAspectRatio
FrameCount
ColorSpace
ChromaSubsampling
ScanType
Interlacement
Compression_Mode
colour_primaries
transfer_characteris
matrix_coefficients
pix_fmt
audio_fmt
audio_codecid
video_codecid
video_codec_version
video_codec_profile
channels
colour_range
format_version
TimeCode_FirstFrame
TimeCode_Source
app_company_name
app_name
app_version
library_name
library_version
reproduction_creator
reproduction_reason
dig_object_descrip

AF12345

Kieran O'Leary

Digital AV Object

aaa1234

Preservation Object

Kieran O'Leary
In

Kieran O'Leary
fb12ce9d-abfb-44bd-9eda-31b9edbee864
n/a
n/a
2018-11-13T12:13:01Z
Matroska
Moving Image
528920716
0.493
n/a
2256.73
1 video track|1 attachment track
n/a
n/a
n/a
FFV1
24
n/a
12
2160x1702
5:4
n/a
n/a
00:00:01:22
n/a
1
45
RGB
n/a
Progressive
n/a
Lossless
n/a
n/a
n/a
gbrp12le

n/a
V_MS/VFW/FOURCC / FFV1
3.4
n/a
n/a
n/a
4
n/a
n/a
n/a
n/a
n/a
Lavf58.12.100|Lavc58.18.100 ffv1
n/a

Matroska

Just for reference, this is what a rawcooked file looks like when run through makepbcore.py with this merge.

Thanks for these details Kieran! From what I understand PBCore also has an update coming, but I haven't seen anything about a time frame.
We are going to try out RAW cooked in the next couple of months, so thanks for sharing that output!

Yeah I'm very curious on the pbcore updates. My hope is that key metadata
elements that are currently just annotations end up becoming core elements..
And rawcooked is lovely so far, very easy to check the authenticity of the
process via scripting!

Cool! I can say that the rest of the world is grateful for early adopters/funders :)


Function declaration colors do not match

In the preview, the functions are colored with a bold yellow that looks quite nice.  But after installation I get a blue color that matches the assignment operator.  Everything else seems to align.

As an aside, I'm a big fan of your Monokai for Notebook.  Is there a straightforward way to port it to Lab?

Yeah the screenshot is slightly out of date. I do believe that the yellow is still the color of class names.
One option would be to clone the repo, edit the jupyterlab_darkside_ui/style/variables.css file, and re-install from source (see here for more installation details). Changing the hex code on line 343  to #f9c367 will get you the yellow function names you see in the screenshot.
As far as porting the Monokai theme to JupyterLab, I wouldn't say it's trivial but also not too difficult, just a bit time consuming. My advice would be to clone my Legos UI repo and use that as a template. You should be able to get most of the way there by replacing colors in the jupyterlab_legos_ui/style/variables.css file with corresponding ones from Monokai style sheet in the jupyter-themes repo.
There's instructions in the Legos UI README: Develop section on how to monitor UI changes in real-time which makes for a much more pleasant experience.
Anyway, best of luck!

Closing this due to inactivity.


Pre-fill some inputs

I have some inputs that I'd like to pre-fill when a user is logged in like the user's CLIENT_TOEKN.

But I can't figure out how to do it.

Could you please post what you can of your swagger.rb initializer?
Also, did you take a look at this part of the documentation covering tokens like this?  Was there a part of that which did not cover your use case?  You could auto-fill the token at the top of the page so that users don't have to input it at the individual call level.

@aschuster3
swagger.rb
GrapeSwaggerRails.options.url      = '/api/1/swagger_doc'
GrapeSwaggerRails.options.app_url  = GetEnv.fetch('APP_URL')
GrapeSwaggerRails.options.app_name = 'My API Documentation'
GrapeSwaggerRails.options.api_key_name = 'CLIENT_TOKEN'
GrapeSwaggerRails.options.api_key_type = 'query'
GrapeSwaggerRails.options.before_filter do |request|
  GrapeSwaggerRails.options.api_key_default_value = 'test_token'
end

I don't actually want to auto-fill the token at the top of the page. I think it's counter intuitive for users and i'm confident they won't notice it.

That's all good, unfortunately this gem doesn't provide easy means to drop default values as it should ideally be handled in the grape-swagger gem.
Off the top of my head, the two options I can come up with are to use default values as defined by the Open API spec and following the rules from grape-swagger, or you can find a means to inject by adding a JavaScript file that fetches the token and finds the locations to insert it via the DOM manually.  Do either of those seem like plausible solutions?

@aschuster3 The first option makes sense. Thanks!


Usability issue with large images on start page

A user in chat highlighted (sic) a problem with the accessibility of the metasmoke start page.
Chat: https://chat.stackoverflow.com/transcript/message/57723558#57723558
Their suggestion is to set the background to "canvas" instead of hardcoding white. I have not yet explored whether that is something we can easily do in Rails.

Tangentially, neither https://www.w3.org/WAI/tutorials/images/ nor https://developer.mozilla.org/en-US/docs/Web/Accessibility/Understanding_Colors_and_Luminance discuss this particular usability issue.

To be fair, W3 accessibility guidelines are pretty bad in general. They seem to think you can test for screenreaders by checking for the presence of aria tag attributes and the absence of keyboard traps, without actually trying it on a screenreader; or even just knock your mouse away and try it with just the keyboard.


Drag and sort

yo, bro, I want to make a wish for a feature: Drag and sort
In this way, I can more easily see the icon effect between folders and folders.

you mean like the "templater" plugin that moves up and down?

Yes, or like the way of rearranging in obsidian→Appearance→Ribbon menu, that would also work.

Yes, or like the way of rearranging in obsidian→Appearance→Ribbon menu, that would also work.

Oh, thank you, my friend. I'll take your suggestion under advisement.

you are welcome


sorting by kind and nested stories

Issue:
What I did
sorting nested stories when activating: sortStoriesByKind: false
How to test
setOptions({sortStoriesByKind: false});

make sure that now both the root list of stories and the nested ones are ordered. At the moment just the root stories where sorted.
["bug", "maintenance"]

We should add some tests for this function

@dioxmio There's lots of information in https://github.com/storybooks/storybook/blob/master/CONTRIBUTING.md
In short: you need to run yarn before running the unittests

@ndelangen awesome thanks. I have added the test however the build on Danger is failing. Could you check? All the rest is good and green.
cheers

Sorry @igor-dv that should do it 😄


How do I lazily initialize components

What will this enable for the user?
As a user of the application, I want to add a placeholder for a component that requires a long backend task. A chart that requires a long SQL query ...
Recipe tags (optional):
Related tags (optional):

In #62


[Feature] Control Cursor with Keyboard

Hi! This is pretty much the best open-source pixel software that I've seen. I used to use Paint.NET, but now I have this with more features for animation, and it's open-source! I do miss one feature from PDN, though: moving the cursor with the arrow keys. For whatever reason, having tactile feedback for the pixels seems to help, such as when feeling out a stroke to see how long it should be. My most ideal version of this would be, when your cursor is in the Piskel window, pressing an arrow key moves it one "pixel width", where a "pixel width" is however many screen pixels each "canvas pixel" is. So it moves more pixels over per key-press when zoomed in, and pressing the right arrow 3 times with the mouse held down always makes three pixels no matter the zoom level. Having it be always-on would mean that you can use keyboard movements for all tools, such as pen, stroke, etc. Also, this wasn't in Paint.NET, but having the pixel your keyboard cursor is on light up would be helpful, just as it is when using the mouse. Thanks!

@P-Star7 Hey can you confirm if this proposal is similar or a duplicate of issue #711 ? If it is please close this issue to maintain the repo clean. Also I think it's worth pasting over your comment on how the tool show work over there and tag Julian so he can read it. Thanks :)

Good point @Jose-Moreno , both issues should probably be merged and as you said we should keep @P-Star7 's comment in the other thread to make sure the information is not lost!

Closing as duplicate of #711


moduleResolution: bundler diverges from bundlers following node semantics in regards to default import

Bug Report
🔎 Search Terms
moduleResolution bundler node webpack default namespace
🕗 Version & Regression Information

This is the behavior in every version I tried: 5.0.4 (the only one with moduleResolution: bundler

⏯ Playground Link
Repro case can be found here
🙁 Actual behavior
When resolving a .d.mts file that proxies to .d.ts in a CJS package with moduleResolution: bundler, TypeScript complains with an error like:

Property 'default' does not exist on type 'string'.ts(2339)

The reason that it complains about it is that TypeScript is resolving default import as if the .d.ts in this CJS package could export the "real" default export. webpack actually follows semantics coined by node and it loads the namespace object as the default export of that "cjs file".
So the type-level reality diverges there from runtime. At runtime, with webpack - it works exactly like it would work in node. TypeScript thinks that the intention was to load module.exports.default there though.
🙂 Expected behavior
I'm not exactly sure what's the expected behavior here because "resolving" like a bundler is quite under-specified and bundlers are not quite consistent when it comes to this. Even when we check this very same repro, we can see that I had to use defaultIsModuleExports: true in Rollup for it to behave the same. By default, Rollup behaves in the same way as TypeScript does.
cc @andrewbranch

Isn’t this controlled by allowSyntheticDefaultImports?

Yeah, I’m aware of this, but of course it’s not a moduleResolution issue. Really, it’s something that module should handle. I believe behavior differs between different bundlers, which is, in technical terms, a huge bummer. What’s really bad, IMO, is that last I checked, Webpack does the Node behavior for .{m,c}js files, but not the equivalent TS extensions, which violates a pretty core assumption we have about the substitutability of declaration files, TS files, and JS files. This is already on my radar to investigate soon.

I forked https://sokra.github.io/interop-test to https://andrewbranch.github.io/interop-test, removed a bunch of import and export variations that weren’t super interesting to me (there are still a ton), and added .ts/.mts cases to esbuild and Webpack. I need to add similar TS cases to Rollup, and add Parcel and Bun. But it shows basically what we’ve established in this issue:

Both esbuild and Webpack give a Node-ESM-like interop treatment to default imports in .mjs files (that is, it always synthesizes a default; it doesn’t care about whether the target module has a __esModule marker).
esbuild extends this same treatment to .mts files since it understands TypeScript out of the box. Webpack does not.

At first, it struck me as odd that these bundlers voluntarily adopted interop restrictions that Node tried very hard to avoid, but couldn’t for technical reasons. But doing so is the best way for them to handle code written for Node—it’s a cross-compatibility strategy. I think it would be best to push all bundlers toward doing this, if they’re not already. But since TypeScript isn’t yet set up to model this behavior (without also adopting nodenext module resolution), it seems inevitable that if we do, it will be toggleable with compiler options somehow. So even if we can’t get bundlers to be consistent between each other on whether they apply this Node-compat behavior, users will be able to configure TypeScript for whatever their bundler does.
However, it’s not possible for us to model bundlers like Webpack that apply the Node-compat behavior to .mjs files, but not to .mts files. That’s something we’ll need to push for across the bundler/TS-runtime ecosystem (I’m assuming Webpack isn’t the only one that will need changes) before any TS option can work.

However, it’s not possible for us to model bundlers like Webpack that apply the Node-compat behavior to .mjs files, but not to .mts files. That’s something we’ll need to push for across the bundler/TS-runtime ecosystem (I’m assuming Webpack isn’t the only one that will need changes) before any TS option can work.

I recall that you mentioned knowing about this issue for quite some time. Is there some tracking issue about this in webpack? Did their team comment on this anyhow in the past?

I don’t know; I’m in the process of figuring out exactly what to propose, and plan to talk to Sean soon. I haven’t mentioned it earlier because it was something I noticed in passing while focusing on module resolution, and needed to double check that I wasn’t just holding it wrong 🥴

@Andarist do you remember how you originally encountered this issue, i.e. what libraries were involved?

@andrewbranch likely it comes from me and @emmatown trying to implement our library building scheme that is meant to avoid dual package hazard and to provide consistent module shape across formats. We ended up emitting an extra proxy files to work around it. Notice the _default vs default dance introduced here: https://github.com/preconstruct/preconstruct/pull/546

@andrewbranch what ever came of this change as well as this webpack issue?

Our current opinion is that the prevalence of people having problems with this doesn’t merit additional confusing configuration.


[FEAT] Add methods to enable SAF-oSL

[x] Added get_project_tree method to Project class. The method returns the list of all the nodes in the project starting from root. Each item of the list is a dictionary containing the uid of the node and the level in the system hierachy.
[x] Extended find_node_by_uid method to enable search throughout the entire project tree.
[x] Added control method to Node class to control the node execution, i.e. restart, stop, stop_gently, reset and shutdown.

@rfahlberg, @PanekOndrej and @PipKat I'd like to merge this pull request. Unit tests are missing but I am working on it. I can add them in a separate user story if it is okay for you.


PayTab version 2

Is pod 'PayTabs', '~> 4.3.1'
Connected to Pay tab v2 portal :  https://merchant.paytabs.com/merchant/login
As still these SDK ask for Secret key and Region.
While there is no Secret key in new portal.
Also the documentation link here is not redirecting to proper place.

Yes, it is connected to PT2, you can use your old credentials of PT1 or get it by navigating to the "INTEGRATION MAPPING" under the "Developers" section.


function tiglWingGetMAC runs infinitely long for wings with winglets/multi-element sections

From<EMAIL_ADDRESS>on April 11, 2014 16:22:57
Version: TIGL 2.1.1, input file: CPACS 2.1.
The function tiglWingGetMAC runs infinitely long and does not return values for configurations with winglets. Possible causes:

wing definition contains multiple elements per section
winglet is rotated 90 or -90 deg with respect to wing tip

See the zip file for two cpacs versions, one with winglet and one in which the winglet is commented out. The one with winglet produces the problem.
Attachment: wing_withAndWithoutWinglet.zip
Original issue: http://code.google.com/p/tigl/issues/detail?id=105

I am closing this as outdated. Feel free to re-open this issue if it is still relevant.


Using timers instead of Time.now in Mailbox and EventedMailbox

Reading the code and found an easy todo. I hope it's well implemented, if not, I'll correct.

This is already implemented, and with the new Timers api.


Why is Polygon.hull() result not counterclockwise ?

The test below shows that the expected result is clockwise, not counterclockwise as mention in the documentation.
https://github.com/d3/d3-polygon/blob/1bfcdc8b63578b936c785639fa73cec566713c45/test/hull-test.js#L12
Is it due to the fact that the y-axis direction of SVGs is going down the screen, and hence inverting some common computation results ?

Yes; D3 uses the default coordinate system of Canvas and SVG, which is +x pointing right and +y pointing down.

Fine. Thanks for your clarification.
I will refactor the d3-weighted-voronoi (and subsequently d3-voronoi-map/d3-voronoi-treemap) plugin in order to fulfill this convention.

Does it make any practical difference? Serious question.
I implemented the algorithm linked in the readme and it gives the same points, but goes in a different direction after the first point, still when I draw the hull, everything seems normal.
What are use-cases where the actual point ordering matters?


[JOSS Review] Doc Improvements

Looking at the docs, I think there are some small improvements I would suggest to help users find information:

[ ] The docs page at https://automl.github.io/amltk/latest/ seems to miss some of the essential information provided in the README (e.g. installation instructions, minimal working example). I would suggest to mirror the readme into, e.g., the Home part of the docs or at least point to the readme for those instructions.
[ ] Please mention in the README and/or contribution guideline that GitHub issues should be used for support inquiries, bug reports etc. (This is the case, right?)
[ ] (optional) For me, the distinction between guides and examples in the docs is not immediately clear, maybe there are better names? Similarly the reference also looks a lot like the API. I don't know if you use some automated to make sure they stay synchronized? Otherwise, it seems that this may quickly lead to one of the two becoming out-of-date.
[ ] From an ML standpoint, I think, it would be great if you could provide an overview of frameworks you can interface with. I think this is partially done here https://github.com/automl/amltk/?tab=readme-ov-file#installation but I am not sure if it is exhaustive? Maybe reformulate to clarify if this the full list of supported frameworks?

(opened as part of JOSS Review https://github.com/openjournals/joss-reviews/issues/6367 )

Hi @gomezzz,
Thanks for the feedback! The docs definitely need some refresh in love but to answer some of the points:

Yup I think some quick install makes sense on the docs homepage.
Definitely should mention github as the source of issues/feedback, will do.
I agree there's too much and I was looking at Textual as inspiration:

API: API doc autogenerated
Reference: A hand-written quick summary, i.e. like a cheatsheet
Guide: A walkthrough to actually learn about how things work
Examples (Admittedly lacking): Copy-and-pastable code with some minor commentary


Feature: Add an indication of if the hurdle was bet in the 'hurdle' column.

perhaps a colored background or a 🟢 / 🔴  icon

+0 from me. i understand the want to do this, but because our datasource can be a little unreliable (mainly est. votes remaining) indicating with green could indicate a success or a prediction that may not actually be the case. sorta similar to why we closed https://github.com/alex/nyt-2020-election-scraper/pull/107

That makes sense, added https://github.com/alex/nyt-2020-election-scraper/pull/172 in case anyone wants it, though

Per discussion, closing this, since it seems too much like "calling it."


Demangle function names after importing MAP file

I guess it's going to miss type info (not present in MAP files), but at least names could be demangled and committed. The complete demangled prototype could be set as a comment maybe. For simple functions not requiring new, unknown types, the full proto could be committed.

As a counterpoint to MAP... I eliminated the demangling and laying down of types from the PE Loader because it did not belong there and because it caused issues with getting more accurate information from the PDB.  The exported mangled symbols did not have as much accuracy as the PDB and thus blocked the more accurate information from getting laid down, and possibly caused other analysis not to run.
Some exported mangled symbols were accurate in terms of usage, but not in terms of memory layout.  For instance, a global array had a mangled symbol associated with it as a pointer to the underlying type.  PDB wanted to lay down the array, but couldn't because the pointer was put down by the PE loader.
This all speaks to the notion that analysis needs to be well thought out, as there can always fallout from what seems to be a logical improvement.
Whoever looks at this closer should consider that badness like this might also be seen for MAP and for analysis in general.


Remove Analysis Threads

This was requested by Chole, so I assume TheImmortal wants this done too.
http://prntscr.com/g4oh5w

This is a misunderstanding.


Refactor deployment modules

Changes:

Uses tfvars: makes it easier to manage deployments to multiple environments.
Removes duplicated configuration between the govuk-test and application deployment modules.
Task definitions won't be created by the govuk module, so we'll need to run additional terraform applys to bring up the apps.

Note: This Pull Request is most easily reviewed commit by commit.
New directory structure:
├── apps
│   ├── common.tf
│   ├── outputs.tf
│   ├── content-store
│   │   ├── common.tf -> ../common.tf
│   │   ├── main.tf
│   │   ├── outputs.tf -> ../outputs.tf
│   │   └── variables.tf -> ../variables.tf
│   └── variables.tf
├── govuk-test
│   ├── README.md
│   ├── main.tf
│   └── variables.tf
└── variables
    └── test
        ├── apps.tfvars
        ├── common.tfvars
        └── infrastructure.tfvars

This introduces a new directory: terraform/deployments/variables
which will contain the variable definitions (.tfvar) files that should
be called when applying terraform for an app task definition or for
the infrastructure.
The new workflows:

Applying the infrastructure$ cd terraform/deployments/govuk-test
$ terraform apply -var-file=../variables/test/common.tfvars \
   -var-file=../variables/test/infrastructure.tfvars

Applying the application$ cd terraform/deployments/apps/frontend
$ terraform apply -var-file=../../variables/test/common.tfvars \
   -var-file=../../variables/test/apps.tfvars \
   -var=image_tag=release_123

The variables/test/common.tfvars file contains variables common to the test
environment deployments.
Should we want to create a new environment we would need to duplicate these
tfvars files for another environment. When we add extra environments we may
find that we can introduce a cross-environment common.tfvars file.
Story: https://trello.com/c/2N5OLEHg/266-refactor-terraform-deployments

@fredericfran-gds thank you! I have added two documents

Adding an app
Applying Terraform

Please let me know what you think.


Publishing controller testing and javadocs

i renamed PublishingController to PublishingManager just for the sake of keeping naming conventions, created java docs for the class, and wrote some very basic unit tests for publishing.

Ran the tests, all passed.


Create OLM bundle integration-test

What is missing?
We do not verify that the bundle we create can be deployed with OLM at all stages (it's done manually before release). This needs to be ensured so that we can push the output correctly to operatorHub.
Also, fix CassandraTask alm-example:
WARN[0000] Warning: Value control.k8ssandra.io/v1alpha1, Kind=CassandraTask: provided API should have an example annotation

Why do we need it?
We want to automate the output to OperatorHub, the release process shouldn't be manual and we want to ensure that our operator conforms to the current bundle requirements.

Closing to try to fix the sync with Unito that didn't seem to happen.


Default preemption behavior

https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/
Regarding this line:

If there is no PriorityClass with globalDefault set, the priority of Pods with no priorityClassName is zero.

I think it's worth adding some documentation about the default preempting behavior too -- being that the default priority will preempt pods with a negative priority (i.e. if it was expressed as a PriorityClass, the setting would be PreemptionPolicy: PreemptLowerPriority). Something like:

If there is no PriorityClass with globalDefault set, the priority of Pods with no priorityClassName is zero, and will preempt lower priority pods (being those with a negative priority).

/language en
/kind feature
/triage accepted
/help
We could link to https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/#non-preempting-priority-class to highlight that if you want Pods to have non-preempting behavior, you must explicitly configure this and assign those Pods to the PriorityClass you declared.

/remove-lifecycle stale

I'm trying to understand if clarification is needed here or not.
I feel like this suggested text already seems obvious, to me at least, and a this information is provided elsewhere on the page already:
and will preempt lower priority pods (being those with a negative priority).

That being said, if is decided that clarification would be helpful here, how about something like this?

A pod's priority is used by the scheduler to determine its position in the scheduling queue, and along with preemptionPolicy, determines whether it can preempt, or be preempted, when there are not enough node resources available to satisfy all pod requests.

/retitle Clarify default preemption behavior
I'm also not sure if further clarification is needed.
/remove-help
/priority backlog
If this rots out, so be it.

@WilliamDenniss if you open a pull request to make the improvement in https://github.com/kubernetes/website/issues/30887#issuecomment-1165870514, we'll review it.


logging for zync

We should think through and implement how zync should do logging and change the various printf's to log calls.  This will be used from orchestration and automation so these logs will be useful events in operational logging systems.

Some details on specific things one user wanted to see implemented here:

Is it possible to add some log when new data is receive from kafka in continuous mode? Actually data from kafka is write to zed lake but in my Zync container console (docker) I see nothing. I want to see something like old zync : number of row receive, commit id... something to show me that zync is really running and processing.


turbo-frame with data-turbo-permanent for load once behaviour

Hi,
I'm trying to load a turbo-frame[src] once on first page load by wrapping it in a div[data-turbo-permanent].

  <turbo-frame id="user-nav" src="user_nav.html">
    Sign In
  </turbo-frame>

The turbo-frame is currently being requested on each Turbo-intercepted click. The desired behaviour is to load the frame once and then not load it on subsequent Turbo navigation (i.e. not see any user_nav.html fetch requests), in the same way that data-turbo-permanent works with regular elements.
Without the wrapping data-turbo-permanent div Turbo makes 2 requests to user_nav.html on each visit. This might be a separate issue because I would expect only one request to be made in this scenario.
With the wrapping element this increases to 3 user_nav.html requests on the second Turbo click (screenshot).

Example repo: https://github.com/stevehodgkiss/turbo-frame-permanent
Thanks!

It seems you're both providing content in the frame (i.e. Sign In) and also providing a src url. Generally speaking, these are two ways to do the same thing: provide content. In most cases, you should not provide a src attribute, unless you want to do things like lazy-loading.
Could it be that you're looking for something like the following?
<turbo-frame id="user-nav">
    Sign In
</turbo-frame>

I tried your example using the previous beta of Turbo (https://cdn.skypack.dev/<EMAIL_ADDRESS>and that does seem to eliminate the problem.

I tried your example using the previous beta of Turbo (https://cdn.skypack.dev/<EMAIL_ADDRESS>and that does seem to eliminate the problem.

True, 7.0.0-beta.3 does eliminate the multiple XHR request issue present in the latest version.

PS. I may have been distracted by your href-less link in the example, and it could be that you want to do lazy-loading after all.

Yeah the goal is to render fully cacheable html pages and lazily load personalised fragments of html dynamically. Initial content inside the turbo-frame would contain a signed-out version, with the src attribute linking to an endpoint returning personalised partials if the user is signed in. Then when navigating with Turbo, these elements would need to be marked as data-turbo-permanent to prevent each page visit from requesting the same content again. We have this functionality implemented with a Stimulus controller & fetch currently, but I think it would be a great fit for turbo frames.

Hello! Not sure if this will help you but I had a similar problem that I managed to solve just using fragments. In my application I want to be able to cache the whole page in a CDN but still have parts of the navigation dependent on the user session. I initially solved this using a fragment like below that would insert the user navigation based on the session.
<turbo-frame id="user_nav" src="user_nav.html"></turbo-frame>

This worked fine but I noticed that since Turbo caches the pages between visits the dynamic part of the navigation would flicker since it was there in the cached page and then removed before fetched again and updated. To resolve this I first made the whole frame (or a wrapper like you) permanent but the problem with this approach was that when loggin in/out the content wouldn't change unless forcing a page refresh.
To get around this I added a "placeholder" inside of the frame with the permanent attribute and a matching placeholder in the user_nav.html, something like below.
<turbo-frame id="user_nav" src="user_nav.html">

</turbo-frame>

<turbo-stream action="replace" target="user_nav">
  <template>
    ...
  </template>
</turbo-stream>

This will make Turbo keep the user-nav permanent when navigating the page but after each navigation do a new request to user_nav.html and replace the fragment with the actual user state.


Update + edit Config File topics in Sys Admin Guide

Change title to clarify that this is reference info
Rewrite intro for clarity and style
Add links to config procedures in the sysadmin guide
Clarify that the log and path config files are "non-component" files
and reorg to put them after the component files
Change title and edit the log config file topic for clarity and style

Signed-off-by: Anne Chenette<EMAIL_ADDRESS>

Closing PR because it is a duplicate of PR #1882 (which has a corrected branch name).


Missing prefix

One of the examples contains this:
builder.Services.Configure<Settings>(builder.Configuration.GetSection("TestApp:Settings"));
Other configuration providers would not prefix settings with the name of the app. Or: If I ran this locally and wanted the same configuration section loaded from my appsettings.json file, what would the setting look like there?
With the keyvault configuration provider there is an example on how to prefix configuration values. It would be a good idea to do something similar here.

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

ID: 2984100d-7d78-2115-40f4-4b9478a5a5bc
Version Independent ID: 24ac6adc-6779-b847-ce81-c9d36e4b067b
Content: Quickstart for Azure App Configuration with ASP.NET Core
Content Source: articles/azure-app-configuration/quickstart-aspnet-core-app.md
Service: azure-app-configuration
GitHub Login: @zhenlan
Microsoft Alias: zhenlwa

Hi @vRune4, a prefix is like a namespace. It's useful to segment the configuration of different apps or different components within the same app, but you can remove it by calling TrimKeyPrefix when loading keys into your app. For example,
builder.Configuration.AddAzureAppConfiguration(options =>
{
    options.Connect(connectionString)
           // Load all keys that start with `TestApp:` and have no label
           .Select("TestApp:*", LabelFilter.Null)
           // Trim the key prefix if exists before adding a key to the configuration
           .TrimKeyPrefix("TestApp:");
});

@zhenlan Thanks for quick responses.
@vRune4 We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Add back the observations/Atm dir. in diags sync

This is needed for atmosphere diags (obviously).

Testing
I used this to sync atmospheric diags on LCRC and to Compy and Cori.

Thank you for the fix and syncing the data!


Document methods for creating data pipeline interface objects

Also clarify in client package that it is a low-level one.

Thanks Dani, both are done.


Trying to dev: Missing "Demo.vue"

I've forked the project and am trying to run yarn dev but getting the following error:
7:30:04 PM [vite] Internal server error: Failed to resolve import "./Demo.vue" from "src/main.ts". Does the file exist?
  Plugin: vite:import-analysis
  File: /Users/tarwin/all/development/repos/polaris-vue/src/main.ts
  1  |  import { createApp } from "vue";
  2  |  import Demo from "./Demo.vue";
     |                    ^
  3  |  import PolarisVue from "./polaris-vue";
  4  |  createApp(Demo).use(PolarisVue).mount("#app");

There doesn't seem to be a Demo.vue anywhere in the repo. Am I missing something?

Hi there,
Demo.vue is just the file to test components when you dev the lib.
We added that file to gitignore.
You can create an empty one, it will work.


Recent refactor broke rangeArea plots

Type of change

[X] Bug fix (non-breaking change which fixes an issue)

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] My changes generate no new warnings
[X] New and existing unit tests pass locally with my changes
[X] My branch is up to date with any changes from the main branch

Thanks @rosco54 for the early catch before release!


Upgrade to v0.43.1; Change Postgres connection string;

Hey, I updated the Postgres connection string according to the metabase docs because the current schema didn't work for me.
Also bumped up the Metabase version to v0.43.1
If there is anything else that needs changing, please let me know.

@m-wrzr thanks for contributing !
However, that just the example value, there's many format for connection URI and you always can edit this value to suite your deployment.
So this PR making no sense

I know that it's an example, but if you adapt the current connection string<EMAIL_ADDRESS>it won't work, because they changed the format to use user/password as a suffix host:5432/metabase?user=<username>&password=<password>.
At least for me formatting the connection string as in the values.yml didn't work.

I doubt that URI format is changed. Instead,  it just your postgres deployment that did not support the user@password kind of URI.
Do you have any clue of the old URI format is deprecated ?

https://www.metabase.com/docs/latest/operations-guide/configuring-application-database.html#postgres
https://discourse.metabase.com/t/metabase-not-connecting-to-postgres-database/18855/3

Seem fair 👍


docs(README): when patching global context, patch TextDecoder and TextEncoder too

After following the README on patching the global context, node-fetch fails to me in jest jsdom environment on node v16.16.0 with following error:
TextDecoder is not defined

      Cause: ReferenceError: TextDecoder is not defined
      at Response.TextDecoder (node_modules/node-fetch/src/body.js:159:14)
      at call (node_modules/@babel/runtime/helpers/regeneratorRuntime.js:44:17)
      at Generator.tryCatch (node_modules/@babel/runtime/helpers/regeneratorRuntime.js:125:22)
      at Generator._invoke [as next] (node_modules/@babel/runtime/helpers/regeneratorRuntime.js:69:21)
      at asyncGeneratorStep (node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
      at asyncGeneratorStep (node_modules/@babel/runtime/helpers/asyncToGenerator.js:22:9)

That's probably because node-fetch in https://github.com/node-fetch/node-fetch/blob/main/src/body.js#L159 refers to TextDecoder using a global reference, which doesn't exist in my environment.
Changes
In recommended script in README that patches the global context, add TextDecoder and TextEncoder to the global context too.
I also removed a few unnecessary imports from it that weren't used at all.
Additional information

[x] I updated readme

Since this is just readme and no breaking changes, then i would now have suggested another approach.
using conditional lazy top level import() and maybe Nullish coalescing assignment (??=) in order to not load it in vain
So maybe something like this:
// fetch-polyfill.js
globalThis.fetch || await import('node-fetch').then(mod => {
  globalThis.fetch = mod.default
  globalThis.Blob ??= mod.Blob
  globalThis.File ??= mod.File
  globalThis.FormData ??= mod.FormData
  globalThis.Headers = mod.Headers
  globalThis.Request = mod.Request
  globalThis.Response = mod.Response
})

globalThis.TextDecoder || import('node:util').then(util => {
  globalThis.TextDecoder ??= util.TextDecoder
  globalThis.TextEncoder ??= util.TextEncoder
})

// index.js
(globalThis.TextDecoder && globalThis.fetch) || await import('./fetch-polyfill.js')