Setting server IP in ALVRClient works only while running

Setting server IP in ALVRClient while it is not running causes a crash. I will fix this.
Workaround is to start ALVRClient and set the server IP while ALVRClient is running. ALVRClient needs to be restarted for the setting to take effect.

I have a question, no matter what IP I set my client is never seen by the server, obviously I'm doing something wrong...

@lorenzinfantino Have you done all the necessary firewall openings?
The UDP ports 9943 and 9944 must be opened on the server (and all firewalls between client and server), so the client can "connect" there.

@DavisNT Yup, i went into the firewall settings and added rules for those ports, but still nothing happens, maybe i need to do something on the client other than setting the IP?

@lorenzinfantino Nohing else should be done on client (apart from opening it, connecting headset to PC, running the command line on PC, restarting the client).
You can try to troubleshoot the connectivity issue as described here: https://github.com/polygraphene/ALVR/wiki/Troubleshooting#ip-address-is-not-displayed-on-alvrexe

I'll try to troubleshoot later today, but if I connet the oculus to the same wifi the servers sees the client, so I guess my ports are fine and working, as soon as I try to change the wifi on the oculus (staying on the same network, just a different subnet) it disappear. Maybe I'm missing the point of the IP change but to my understanding, and correct me if I'm wrong, it's supposed to make the oculus connect to the server even from a different network...

@lorenzinfantino To be able to connect to ALVR server from a different network both networks must be connected (UDP ports 9943 and 9944 of ALVR server must be reachable by ALVR client). I.e. The ALVR server must be reachable (e.g. pingable) from the network where ALVR client is connected.

Pull request fixing this issue has been submitted https://github.com/JackD83/ALVRClient/pull/2

Fixed in ev7.


Use unix rm to delete file

Fix #15

This would be a dangerous breaking change. Better to just tell users to customize it themselves . There's a guide in the README anyway https://github.com/roginfarrer/vim-dirvish-dovish?tab=readme-ov-file#customize-commands

As @user202729 said, I'm afraid that this is a breaking change, and it'd be better for users to customize to their liking.


fix: remove @prisma/cli dependency

Closes https://github.com/node-casbin/prisma-adapter/issues/22

@Zxilly what does this PR do?

@hsluoyz @prisma/cli has been abandoned by prisma community. It was renamed to prisma.
Seems it also has some side affect which broke CI. I'm working on this.

@Jack-Barry please review.

@nshusr plz review this.

Added some notes for my review of this in the original issue


Using VisualServer.request_frame_drawn_callback within such a callback results in infinite loop

Godot 3.0.2
I am searching for a way to have a function being called once every frame/main loop iteration, but WITHOUT needing a reference to the scene tree, in order to use time slicing on a partially non-thread-safe task.
I've tried a recursive call_deferred, however calls are no longer "deferred" if scheduled within such callbacks, which results in a blocking infinite loop. But I thought it was normal because it's a queue being used behind the scenes and there isn't much reason why a callback wouldn't get executed next.
So I've tried VisualServer.request_frame_drawn_callback, which from its name, means callbacks get executed once a frame is drawn... BUT, the same infinite loop happens if I schedule a callback within such a callback, which this time makes less sense, because no frames have been drawn between the scheduling and the call.
Hence why I open this issue: is this really intented?
If that's not going to change, my only left alternative is to spawn a dummy thread whose only purpose is to call_deferred on the main thread every 16ms, which feels quite dumb...

@Zylann Can't you connect to idle_frame of the MainLoop? The MainLoop is the parent class of SceneTree, and is going to be always present in a running game.

@bojidar-bg but can I access MainLoop in tool mode from outside the tree? (is it a singleton?)

Check VisualServer again, there are a few signals to do this:
ADD_SIGNAL(MethodInfo("frame_pre_draw"));
ADD_SIGNAL(MethodInfo("frame_post_draw"));


Update index.yml

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

advocates/index.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Closing this, as index file has to be generated by automation script


Allow arbitrary index types

I'm using ParadeDB's bm25(https://github.com/paradedb/paradedb)index for search, and am getting this error
Error: invalid index storage parameter:  min_gram: 4

It looks like the error is coming from here
https://github.com/ariga/atlas/blob/5922441c051dd131b7e86227893813d44cbc6222/sql/postgres/inspect.go#L1401
What is the reason to limit the index types?
Can we at least have an option to bypass this check?

Thanks for reporting this, @mingfang.

What is the reason to limit the index types?
Can we at least have an option to bypass this check?

There's no option to bypass this check atm. We currently parse the specific Postgres format we support, as it's used in other stages like diffing, planning and when written as an HCL.
I'm not sure what are the gaps between ParadeDB and our current Postgres driver, but these cases we'll be handled when we add support to it - I don't have ETA for this.
Closing, as this is duplicate of https://github.com/ariga/atlas/issues/1472. I'll update it when the issue is resolved.


Code cleanups and Typo fixes

Thanks - also I've updated the jenkins job - the error was not caused by your patch, but because it used JDK7 instead of JDK8.


(MAINTENANCE) update ticket template

SUGGESTED FONT: DM SANS
INPUT

Upper half of the page - visitor’s badge

Links to graphics:
Regular
VIP
Expo partner: ikas (logo)

Middle of the page - text

Print, fold and insert into a badge holder, available on arrival at the event

Left bottom corner - text

WELCOME TO THE E-COMMERCE BERLIN EXPO!
HOW TO USE YOUR BADGE
To ensure a seamless experience at the event, please follow these instructions for using your badge:

Print this page clearly on A4 paper and bring it with you to the venue.
Upon arrival, head directly to the entrance to be scanned in.
Badge holders and lanyards will be available at the entrance.
If you lose or forget your badge, visit the registration kiosk for a reprint. Please note that not having your badge may result in a delay at the entrance.

STAY CONNECTED!
Follow us on social media and use the official event hashtags when you share. We can't wait to see your posts!
#EcommerceBerlinExpo2024 #EBE2024 #EcommBerlin

Right bottom corner - text

VENUE
STATION Berlin
Luckenwalder Str. 4-6
10963 Berlin, Germany
OPENING TIME
Thursday, 22nd February 2024, 09:00-18:00
REACHING THE EVENT
To avoid traffic, we strongly suggest using public transportation. The closest U-Bahn station, "Gleisdreieck," is available for your convenience.
Alternatively, if you choose to drive, there's a nearby car park, "Parkhaus Gleisdreieck". Parking fees may apply.
CONTACT
<EMAIL_ADDRESS>THANK YOU FOR CHOOSING TO JOIN US!

@az i'd be great to have it ready to be sent at the beginning of next week - preferably 12.02

@az, tickets dispatch should be starting, so could you please let me know when to expect the possibility?

@KarolinaTEH today..... most probably 14:00-15:00 your time

thanks!

done


Add LEARNING.md with basic resources

Just the stuff available at haxe.org so far, but it's a start.

This is great, thanks! Apologies for the delay in getting it merged.


Any example or demo here ? I am getting require is not defined.

I am using this plugin on cordova to setup a gallery app , installed gallery-api and cordova-gallery-access.
Always getting require is not defined if I called function , anyone can help me out ?

Cordova 8.1.1

I think your question is something where stackoverflow.com would be a more suitable forum for. It sounds like your problem isn't with this library, but rather with your general infrastructure and the way you build the application.

yes, sorry about that .
I fixed it , its my application setting issue,  work perfectly now.


ModMail

Don't forget to check it idk if I made the closing part correct (I guess I did, but I didn't tested it as I was too lazy)

U might need to change

As when you dm it rn it will also create a modmail ticket :rofl:

Added that it auto reacts in #requests

yes

Guess u don't want this anymore


Bug

Dear @Senipah , in the last stage of testing for the CSV interface I found a little bug in the method FromCSVString of the VBA Better Array class. Specifically, I was trying to make my project able to handle empty CSV files and records having zero length fields inside them. More details: the error occurs over the UnquoteString function (line 2450). See the attached screenshot
To Reproduce
The test string is the following:
"",[vbCrLf]
,"" [vbCrLf]
Field1,Field2[vbCrLf]

Screenshot

Operating System:

Windows 10 Pro

Host Application:

MS Excel 32 bits

Thanks for the heads-up. closed in 5168a06 (included in release 1.6.4).
Cheers


How to delete uvent in Mac OS X

Hi, I just install uvent in my mac.
But how to uninstall or delete uvent when I don' t need it ?

If you installed it by doing python setup.py install then you'll have to manually delete the files. If you used pip install ., then you pip uninstall uvent.


Version 0.2.1

Codecov Report

Merging #2 (fd6aac9) into master (d236bf6) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #2   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           88        88
  Branches        26        26
=========================================
  Hits            88        88

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 d236bf6...fd6aac9. Read the comment docs.


DragOverlay shadowed preview

Hello and thanks for all the work on the dnd-kit!
I've migrated from an old version to the latest @dnd-kit/core and @dnd-kit/sortable and it fixed some bugs I had with autoScrolling.
What I can't manage to bring back now, is the DragOverlay shadowed preview in a vertical sorted list when Dragging without releasing. It used to display the future location as a shadow, out of the box, but now it just doesn't.
Could a gentle soul point me to the right parameter I am missing or the behavior name of what I am looking for?
Sorry to pollute here, I didn't know where to get this info!

Have you found the solution? @bloodbird

Have you found the solution? @bloodbird

So, when looking back on this project, I don't remember what I did that brought it back. I just see that I've used :
<DragOverlay dropAnimation={null}>
Within dnd context, outside the Droppable but I don't think it helps at all sorry!
I also used on SortableItem :
const style = {
        transform: CSS.Transform.toString(transform),
        transition,
    };

That's the only thing I find but again, that may not have been the solution. I remember that I either randomly got it back without knowing how, or I switched to a different release until I got my fix + the shadow.


koka: Only build the executable

Motivation for this change
Just like for most other haskell programs, we only want the binary, not the haskell libraries and ghc.
This reduces the closure size and avoids "conflicting packages" errors on macos.
Before:
❯ nix path-info -Sh /nix/store/5mshznqzns5lndkv4632m886dwlvkbka-koka-2.1.1/
/nix/store/5mshznqzns5lndkv4632m886dwlvkbka-koka-2.1.1     2.4G

after
❯ nix path-info -Sh /nix/store/vs77d70z834c3hvzz73a87sl5xkb0w65-koka-2.1.1/
/nix/store/vs77d70z834c3hvzz73a87sl5xkb0w65-koka-2.1.1   817.0M

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@ofborg build koka

Thank you!


Remove insert_key parameter

Removed obsolete insert_key parameter from format_on_key.  Re-enabled format line and format brackets, which were broken due to rename of insert key parameter.

:+1:


fix(web): improve project import

Overview

Add teamId in project import  mutation input

What I've done
What I haven't done
How I tested
Which point I want you to review particularly
Memo

Summary by CodeRabbit

New Features

Enhanced project import functionality by allowing imports with both teamId and file.

Bug Fixes

Simplified error handling for project export operations, improving user feedback.

Documentation

Updated GraphQL mutation signatures to reflect new input requirements for project imports.

Implementation is good. I have two questions added and I think we would also require backend changes in this repo to support the new input property.

Implementation is good. I have two questions added and I think we would also require backend changes in this repo to support the new input property.

Implementation is good. I have two questions added and I think we would also require backend changes in this repo to support the new input property.

@mulengawilfred actually the backend already handled this input that why I updated on front-end


'cargo test' fails

At least some of the reason is that 9f78ea9c83b04dbf1b4ee5c11882ab5c878f7f90 did not update test expectations.
Once this is fixed, cargo test should be added to .travis.yml.

cargo test is not expected to work, nor be up to date. It was a churn to add new tests and modify expectations, so I added the make test command.
Agreed that should be better to use a rust harness though.

Then maybe remove the tests that are not expected to pass? They’ll be in the git history when it’s time to bring them back.

I think we need a better story than we currently have for running tests, if only for potential contributors and newcomers to this code base.
Right now, I run make test and get no message about whether the test suite overall passed or had failures. I can't even rely on $?. If I look into .travis.yml, I'll find that finding out if my test run passed effectively relies on git diff'ing the test/expectations directory. But I can't even do make test on master without getting diffs in that directory!
This is not friendly for contributors. We need a much better story here.


JSX: Fix assignment expression parsing in place holders

Fixes #2092

@johnjbarton PTAL
Turned out to be simpler than I anticipated in the bug. The problem is that peekJsxToken treats this as a JsxIdentifier. Fortunately, the content of a jsx placeholder is a normal expression so we can use the normal peek function here.

LGTM, thanks!


LinuxMakeGNU_OpenBLAS_x86 and LinuxMakeLLVM_OpenBLAS_rv64 Public CI failing

Describe the bug
Failures are occurring during Azure Pipelines runs on these steps of public CI as of recent - check any open PR for example of failures:

@keeranroth any idea where these might be coming from?

The exec format error is a problem with QEMU and binfmt. I'll have a look
tomorrow. It's a public holiday today. I'm not sure about the illegal
instructions, but if that is on the riscv build,  it's likely to do with
the qemu setup as well
On Tue, 30 Apr 2024, 23:25 ethanglaser, @.***> wrote:

Describe the bug
Failures are occurring during Azure Pipelines runs on these steps of
public CI as of recent - check any open PR for example of failures:
image.png (view on web)
https://github.com/oneapi-src/oneDAL/assets/42726565/cf00bf0b-b3e9-429e-aa5f-fadf09728dbe
image.png (view on web)
https://github.com/oneapi-src/oneDAL/assets/42726565/2b48a19a-887b-4fa6-8510-0c371a1cbc3b
@keeranroth https://github.com/keeranroth any idea where these might be
coming from?
—
Reply to this email directly, view it on GitHub
https://github.com/oneapi-src/oneDAL/issues/2758, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBVOCXGLXRAOEETOIPAPOUDZAAD3LAVCNFSM6AAAAABHBAPNKSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3TENBXGM2TIMQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

https://github.com/oneapi-src/oneDAL/pull/2762 should fix this. The riscv64 issue was easy enough, as the location of the downloaded QEMU had changed. I suspect that the error on x86_64 is a corrupted OpenBLAS build, as I can't reproduce it locally. Giving a key that hasn't been cached yet should fix things 🤞

Should indeed be fixed by https://github.com/oneapi-src/oneDAL/pull/2762. Closing.


Question: What target platforms are currently supported by Tightlock?

I am scoping Tightlock to be a solution that we might be interested in implementing, but I have only found support for GA4's MP right now. Is there any other platform that the project is currently supporting or plans on supporting in the near future?

Hi Gonzalo, sorry for the late reply. We are working on new targets (we call them "Destinations") and you can see the ones that are currently scoped by looking at the issues marked as Destination.

Hi Gonzalo, sorry for the late reply. We are working on new targets (we call them "Destinations") and you can see the ones that are currently scoped by looking at the issues marked as Destination.

Thanks! I'll take a look!


[mgmt] migrate mgmt package to esm 3

https://github.com/Azure/azure-sdk-for-js/issues/32184
@azure/arm-azurestack
@azure/arm-azurestackhci
@azure/arm-baremetalinfrastructure
@azure/arm-batch
@azure/arm-billing

API change check
API changes are not detected in this pull request.

the ci is failing...


[Question] vpn

I run pogo map on my windows pc.
Without vpn i can connect to it from any public ip. And with my vpn on i cant. Only from the local ip.
Ports are open on my router and vpn service! Is it possible to make it work?
Sry bad english :)

this has nothing to do with this map. you are asking a network topology question, so you should not be asking here
that is normal and expected behavior, so nothing is wrong
try https://ngrok.com/

sry then! ngrok does not work when im on vpn either :(

for the record - i just tested ngrok + VPN and it works fine. something is wrong with your computer or network, but again this is not the place for this discussion


Update search

I tried to update the search functionality. My suggestion is by no means a perfect solution and still has various issues, but it has a few improvements, namely:

split searches, e.g. searching for "treenation" or "tree nation" would now also show the result "tree-nation", which is currently not the case
part searches, e.g. searching for "proton calendar" will now show the appropriate result "proton"

This comes with one major downside. Searching for something like "Time and Date", which is clearly supposed to be www.timeanddate.com will lead to various other results including the word "time" or "date", like the "New York Times".
I still think this could become useful for users, I'd just like to hear your opinion and maybe get some help with fixing this issue.
EDIT: Maybe one solution for the issue of finding unrelated things could be by sorting entries based on similarity to the search term, instead of alphabetically. That way users would get the most relevant result first, while still being able to find similar things.
EDIT2: Just realized, should have made an issue to discuss this instead of a PR, but this was the easiest way to share my code, so please ignore that.

Thanks for the suggestion. I haven't had the time to look into the code, but to be 100% honest I mostly don't use the website search myself and rather just use whatever the browser search capability is, just because it's quicker when you know the hotkey. Since the site is not paged and loads all entries, I think the search makes less sense.

This comes with one major downside. Searching for something like "Time and Date", which is clearly supposed to be www.timeanddate.com will lead to various other results including the word "time" or "date", like the "New York Times".
I still think this could become useful for users, I'd just like to hear your opinion and maybe get some help with fixing this issue.

I see this as a substantial downside. I think this would be much less of an issue if this were to adopt something resembling a closest-distance style algorithm where text which is "closer to" other text by some metric would be given a higher priority, and thus listed earlier. eg. "Time and Date" could show both these results so long as it showed the closer www.timeanddate.com entry before www.nytimes.com. This would, however, be a significant increase in effort compared to the simple filtering search logic we're using currently.
Also, if we are committing to writing search which should follow more specific rules, I might want to add a test suite to ensure we can track the desired behavior and/or any cases which we might not be covering.


Unable to build Unity wrapper

I am trying to build the Unity wrapper based on the instructions given here. Unfortunately running cmake is giving an error as shown in the image.

What should be the correct command?

Hi @lafith  There is an easier method to install the RealSense Unity wrapper.

Download the file Intel.RealSense.unitypackage from the 'Assets' file list at the bottom of the RealSense SDK version listing at the SDK Releases web-page.

https://github.com/IntelRealSense/librealsense/releases

Create a new Unity project or load your existing one.  Then go to the Assets > Import Package > Custom Package menu option and use the file selector to select the Intel.RealSense.unitypackage file that you downloaded to your computer.  It will typically be located in the Downloads folder of your Windows user documents folder if you did not define a custom path when downloading it.

Another way to find the Downloads folder is to left-click on the arrow beside the file download tab at the bottom of your browser screen and select the Open file location menu option.

After you select the Intel.RealSense.unitypackage file, Unity will begin automatically import the RealSense Unity wrapper files into your project.  When a pop-up list of files appears, left-click on the Import button at the bottom tof the panel to complete setup of the wrapper in your Unity project.

To launch the starter project, go to the central-bottom Assets panel of the Unity window and naviate to Assets > RealSenseSDK2.0 > Scenes and double-leftclick on the StartHere scene file to load the starter project.  Once you have the RealSense camera plugged in, left-click on the 'play' triangle at the top of the Unity screen to run the project.

Thank you, that worked for importing. Following is the output on selecting 'UI Image Sample(Depth & Color).

Hi @lafith  Which RealSense camera model do you have plugged in when you run the example, please?  Also, which RealSense SDK version do you have installed?

Camera Model: D435
SDK version : Intel® RealSense™ SDK 2.0 (v2.47.0)

Thanks for the information.  That setup should be fine.  I just wanted to confirm that you were using a RealSense SDK version where Unity 2020 was supported, and you are.
  RealSense programs currently streaming with the camera should be closed before running another RealSense program, because the program that was run first will be blocking access to other applications using the streams that it currently has a 'claim' on.

Yes, that fixed it.


Security: improve secret handling, dont store them directly in environment variables

Storing secrets in environment variables can be problematic as the environment of a process can be visible by other processes.
This is not a major risk, but if another process has been compromised on a machine, secrets will be leaked.
Docker and systemd both have a specific mechanism for handling secrets (docker secrets and systemd credentials ).
The actual fix for this would be to create a separate environment variable fr each secret that is a filepath where the secret will be read from.
So we would add a LIVEBOOK_PASSWORD_PATH env var and either read the env var from the path or directly from the previous env var to keep compatibility.
The change should not be too involved, but would you be open to a PR in that direction ?
Context: im posting this because we are in the process making a systemd service for nixos at the moment.
No matter what you decide on this issue, thank you for making amazing software open source !

This is not a major risk, but if another process has been compromised on a machine, secrets will be leaked.

Can you please expand how a compromised process would be able to read the secrets from another one?

Thank you for the reply.
I understand that env vars given via docker run can leak, but sourced env vars do not. So if you change the entrypoint of your Dockerfile and set env vars there, you should be safe. So anyone should be able to achieve safer deployments with Docker.
Unfortunately I have no systemd experience though to judge there. In a Unix system, are all env vars available to other processes? Or only those set via CLI?

Also please note that we delete the env vars from memory during boot.

It looks like on Linux you can read initial env vars from /proc/<pid>/environ, but if that's also a file, I'm not sure if it helps to read the env var from another file explicitly.
@happysalada also one of your points is kinda against using a file too :) "Avoiding Disk Storage: Storing secrets in configuration files means they're persisted on disk. Even if the file permissions are restricted, there's still a risk associated with having secrets on persistent storage, especially if backups or snapshots are taken."

Avoiding Disk Storage: Storing secrets in configuration files means they're persisted on disk. Even if the file permissions are restricted, there's still a risk associated with having secrets on persistent storage, especially if backups or snapshots are taken.

The solution taken by agenix is to store the unencrypted files in a tmpfs (/run), afaik.

@jonatanklosko thank you for your comment !
The proper mechanism for handling secrets is through docker secrets and kubernetes secrets and systemd credentials.
Both of those use a file like interface, that is why i suggested the fix.
For systemd credentials will give you a file path you can read the secret from, not just a naked value (its the same for docker secrets if i remember correctly ).
The underlying systems though are trying to store the secrets only in memory.

Are we talking about env vars that we use to boot the VM with or “Livebook secrets” which are exposed as env cars to the notebooks? If former, please ignore this message. If the latter, note env vars are emulated by the VM, not OS level env vars, so im not sure the security concerns applies.

Sorry folks, the discussion is pulled into many directions, and it is unclear where to go.
I understand that LIVEBOOK_PASSWORD_PATH is a possible solution to the problem. However, I would like to know:

What is the problem precisely?
How can other users read environment variables?
What are other solutions available? Can any of them be used today? If not, why not? For example, I know custom entry points is enough to hide env vars in Docker.

In order to make a decision, I need to understand the problem statement and all possible alternatives well. If someone can expand on those, let me know. I have already tried ChatGPT, it was not enough.

@josevalim i have to admit, ive never tried any of those hacks and dont know how much leakage there actually is. Ive always taken the recommendation at face value.
I was thinking though, that is the switch from SECRET to SECRET_FILE is all it takes, impact should be minimal no ? Are you concerned about this being a breaking change maybe ?
(Of course there needs to be some more work done on the docker image and the docker compose)
More context: the reason i proposed the change is that in nixos, its just much easier to have secrets as files without shooting yourself in the foot.
On the other hand, we run livebook as a user service which is constrained to the user, so for us its not critical.

Ok let me do more research on this and come back to you @josevalim

Environment variables can be leaked depending on the running ecosystem. Having tokens / secrets mounted as a file path does a couple things here.

It allows for rotations, service tokens for JWT aws auth for example
This can be mounted as a config map / secret / etc for rotation without redeployment.

examples:

Vault
AWS Secrets Manager
K8s

Environment variables can be leaked depending on the running ecosystem.

That's what has been claimed so far but I would like to understand how this would happen in a way that is addressed by files. :) I understand how it can happen in Docker but not yet outside of Docker.

This can be mounted as a config map / secret / etc for rotation without redeployment.

For this to happen, then an additional feature is required: we would need to program Livebook to also check for those files and reload these secrets. Another approach is for us to provide an API to update these secrets, and folks could emit rpc calls into their Livebook instances.

That's what has been claimed so far but I would like to understand how this would happen in a way that is addressed by files. :) I understand how it can happen in Docker but not yet outside of Docker.

So from an attack vector perspective:

usually environment variables are not stored with encryption

This means that if someone isn't using something like fargate or bottle rocket, other containers running, if a compromised container running on the same host, it could jump over and get the secrets
Logging and monitoring tools, if they aren't watched carefully they can pull the environment variables as a diagnostic tool

There are ways to compensate for these controls to reduce risk so I don't nessesarily think it is a must have, just wanted to give the attack vector context.

For this to happen, then an additional feature is required: we would need to program Livebook to also check for those files and reload these secrets. Another approach is for us to provide an API to update these secrets, and folks could emit rpc calls into their Livebook instances.

Maybe something like the setup side that could define hooks to allow for fetching etc.
Right now I just force our pod to restart every X amount of days depending on the rotation of the secret :man_shrugging: so I have ways around it.

Right now I just force our pod to restart every X amount of days depending on the rotation of the secret 🤷♂ so I have ways around it.

And to clarify, which secrets are you setting? LB_* or LIVEBOOK_PASSWORD?

LB_*
On Sun, Nov 5, 2023 at 1:44 AM José Valim @.***> wrote:

Right now I just force our pod to restart every X amount of days depending
on the rotation of the secret 🤷♂ so I have ways around it.
And to clarify, which secrets are you setting? LB_* or LIVEBOOK_PASSWORD?
—
Reply to this email directly, view it on GitHub
https://github.com/livebook-dev/livebook/issues/2315#issuecomment-1793663792,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFIULYXQZNSZ6XOIKBX4XDYC47WBAVCNFSM6AAAAAA6VE6C2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTGY3DGNZZGI
.
You are receiving this because you commented.Message ID:
@.***>

Yeah, in this case it is even better if we provide APIs, because then we can sync across all deployed instances, so I will go ahead and close this for now. :)

@josevalim sorry it took me some time to get to this.
as an aside, anything passed as a cli argument does indeed leak to any other user (ps -eww). Meaning that probably -cookie shouldn't be passed as a cli argument. if env vars are passed as a cli argument, those would leak too. In the same vein, commands typed in the shell end up in the shell history, people might just forget to turn off shell history.
here are the problems I see with storing secrets in environmment variable

It's common to have applications grab the whole environment and print it out for debugging or error reporting. So many secrets get leaked to PagerDuty that they have a well-greased internal process to scrub them from their infrastructure.
Environment variables are passed down to child processes, which allows for unintended access. This breaks the principle of least privilege. Imagine that as part of your application, you call to a third-party tool to perform some action—all of a sudden that third-party tool has access to your environment, and god knows what it will do with it. (I'm guessing here each subprocess called by livebook doesn't share the environment, so this one is okay ? )
When applications crash, it's common for them to store the environment variables in log-files for later debugging. This means plain-text secrets on disk. (I'm guessing here, the environment doesn't get stored in case of a crash of the erlang vm ?)
Passing secrets as env vars, means the user is storing secrets unencrypted on disk and I don't know of any mechanism that "helps" you store them in memory or encrypt them. This seems to not encourage the user to use best practices.

One the other hand

you mentioned is scrubing the secrets from the environment on startup. You mentioned livebook was doing this and this is indeed a common technique that has been used.
accessing environment through /proc has been patched, as of recent, it's not possible for unprivileged users to access the environment of another process through /proc
There isn't a standard for handling secrets, there seems to be so many different softwares that work slightly differently (hashicorp vault, infisical...). In terms of DX, this is not great and complicates the process.

My conclusion is that this seems largely to be fine as of now if you scrub the secrets from environment.

Thinking more about this, the problem is how is the user supposed to store the secret ?
There is no way to encrypt a value passed in the environment. If the environment variable is looking for a value, it means that this forces the user to store the secret unencrypted on disk. I think that is the real problem.
The file solution gives the user the possibility to encrypt the file.

Aren’t we just moving the problem around? Sure, you encrypt the file, but Livebook now needs to decrypt the file, how are you going to pass the key to Livebook? Are we encrypting the file to avoid who from accessing it? Whoever can access the encrypted files will most likely access the key anyway.
I understand the need for rotation but other than that moving to file doesn’t seem to be addressing any additional problem.

The mechanism for encription and decription are made available by docker secrets and systemd credentials. They make sure the secret is accessible only to certain person and stored encrypted.
The only thing to use those is that they require the secret to be read from a file.

Right, but if someone can access an env var, then they can also access a file. So I am really missing the attack vector this is meant to mitigate.

I see your point.
The way i see this is that even if the permissions werent set properly on where you stored the env var, the encryption makes it inaccessible.
Say you stored the env vars in a file but allowed other users to read the contents of those files, encryption would prevent them accessing it.
I think its not a matter of accessing the env var inside livebook but accessing them where they are stored before they are passed to livebook.
Even if the env var is secure inside livebook, if the user stored it on disk unencrypted and messed up the permissions then potentially another user on the machine could read them.

Whatever you set LIVEBOOK_PASSWORD_PATH to, that file won't be encrypted, because Livebook needs to read the contents. From the previous comments my understanding is that you could define a service such that it stores the value in an encrypted file, then on boot it creates a decrypted version of that file perhaps in in-memory file system and passes that as LIVEBOOK_PASSWORD_PATH. The thing that is not clear is why someone reading LIVEBOOK_PASSWORD_PATH won't be able to just read the file, the same way that Livebook does.

I'm more familiar with how things happen on the systemd side, but that is exactly what it does.
It will ensure that what you pass it is encrypted at rest and only decrypted for the particular service you are running (I imagine that docker secrets works the same).
With nix specifically you encrypt things and make sure they are only decrypted for systemd which en turns only decrypts things for the service running.
all in all I realize that this is a bit intense, especially for a service that isn't going to be run with top secret data. I'm happy to leave this issue closed as unplanned.
I'm also grateful for everyone taking the time to discuss this, thank you again!


Move repository to fermitools organization after 1.1 release

Either then, or around the beginning of March.

This is complete.


Library docs

Description
Adds contributing documentation for the Metecho Storybook component library.
Steps to test/reproduce
Please explain how to best reproduce the issue and/or test the changes locally (including the pages/URLs/views/states to review).
REMEMBER: Attach this PR to the Trello card

 Training & Documentation for Handoff

@sanajaved7 As someone with more distance from this component library work it would be great to hear your thoughts on these docs too.


MLP without LayerNorm

Is there currently a way to use MLP without applying the LayerNorm? What would be the best way to implement this? Thanks!

The simplest solution would be to manually construct an MLP out of multiple te.Linears, but this won't be able to do all of the kernel fusions in te.LayerNormMLP.
Long-term, this kind of customization is the purpose of the operation-based API being developed in https://github.com/NVIDIA/TransformerEngine/pull/707:
mlp = te.Sequential(
    te.ops.Linear(...),
    te.ops.GeLU(),
    te.ops.Linear(...),
)


Double attach of behavior in \yii\base\Component::attachBehaviorInternal() BC break

See:
https://github.com/yiisoft/yii2/commit/954c771fdbd867500bc5de8498a3a7453716a74a?diff=split#diff-d45d5e14cbb9739c2e1c63b53ba5e363L669
What steps will reproduce the problem?
Attach behavior in behaviors() to model without key:
/**
     * @inheritdoc
     */
    public function behaviors() {
        return [
            [
                'class' => 'common\behaviors\I18nBehavior',
            ],
        ];
    }

What is the expected result?
Behavior::attach() is called once.
What do you get instead?
Behavior::attach() call twice.
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
5.6

Operating system
ubuntu

There is now a <IP_ADDRESS> release including this fix. https://github.com/yiisoft/yii2/blob/<IP_ADDRESS>/framework/CHANGELOG.md#20112-february-08-2017


fixing #140

Submit JSON SBOM to DT instead of XML to generate license information as well.
The issue is related #140

I hope this PR is no longer required.


com.mysql.jdbc.exceptions.jdbc4.CommunicationsException: Communications link failure

hi
i'm following each step to configure spark recommendation engine in google cloud
below step are completed
created cloud platform project.
created  cluster in dataproc.
created instance in google cloud sql.(first generation)
i have created database and table in mysql  cloud sql.
i try to  find best model through the dataproc i got jdbc connection error
File "/tmp/433d1223-b139-4b58-8749-30bbf193eed6/find_model_collaborative.py", line 59, in
dfRates = sqlContext.read.jdbc(url=jdbcUrl, table='Rating')
File "/usr/lib/spark/python/lib/pyspark.zip/pyspark/sql/readwriter.py", line 475, in jdbc
File "/usr/lib/spark/python/lib/py4j-0.10.4-src.zip/py4j/java_gateway.py", line 1133, in call
File "/usr/lib/spark/python/lib/pyspark.zip/pyspark/sql/utils.py", line 63, in deco
File "/usr/lib/spark/python/lib/py4j-0.10.4-src.zip/py4j/protocol.py", line 319, in get_return_value
**py4j.protocol.Py4JJavaError: An error occurred while calling o44.jdbc.
: com.mysql.jdbc.exceptions.jdbc4.CommunicationsException: Communications link failure**
The last packet sent successfully to the server was 0 milliseconds ago. The driver has not received any packets from the server.
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
at com.mysql.jdbc.Util.handleNewInstance(Util.java:418)
at com.mysql.jdbc.SQLError.createCommunicationsException(SQLError.java:989)
at com.mysql.jdbc.MysqlIO.readPacket(MysqlIO.java:632)
at com.mysql.jdbc.MysqlIO.doHandshake(MysqlIO.java:1016)
at com.mysql.jdbc.ConnectionImpl.coreConnect(ConnectionImpl.java:2194)
at com.mysql.jdbc.ConnectionImpl.connectOneTryOnly(ConnectionImpl.java:2225)
at com.mysql.jdbc.ConnectionImpl.createNewIO(ConnectionImpl.java:2024)
at com.mysql.jdbc.ConnectionImpl.(ConnectionImpl.java:779)
at com.mysql.jdbc.JDBC4Connection.(JDBC4Connection.java:47)
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
at com.mysql.jdbc.Util.handleNewInstance(Util.java:418)
at com.mysql.jdbc.ConnectionImpl.getInstance(ConnectionImpl.java:389)
at com.mysql.jdbc.NonRegisteringDriver.connect(NonRegisteringDriver.java:330)
at org.apache.spark.sql.execution.datasources.jdbc.JdbcUtils$$anonfun$createConnectionFactory$1.apply(JdbcUtils.scala:61)
at org.apache.spark.sql.execution.datasources.jdbc.JdbcUtils$$anonfun$createConnectionFactory$1.apply(JdbcUtils.scala:52)
at org.apache.spark.sql.execution.datasources.jdbc.JDBCRDD$.resolveTable(JDBCRDD.scala:58)
at org.apache.spark.sql.execution.datasources.jdbc.JDBCRelation.(JDBCRelation.scala:113)
at org.apache.spark.sql.execution.datasources.jdbc.JdbcRelationProvider.createRelation(JdbcRelationProvider.scala:47)
at org.apache.spark.sql.execution.datasources.DataSource.resolveRelation(DataSource.scala:306)
at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:178)
at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:146)
at org.apache.spark.sql.DataFrameReader.jdbc(DataFrameReader.scala:193)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at py4j.reflection.MethodInvoker.invoke(MethodInvoker.java:244)
at py4j.reflection.ReflectionEngine.invoke(ReflectionEngine.java:357)
at py4j.Gateway.invoke(Gateway.java:280)
at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)
at py4j.commands.CallCommand.execute(CallCommand.java:79)
at py4j.GatewayConnection.run(GatewayConnection.java:214)
at java.lang.Thread.run(Thread.java:748)
Caused by: java.io.EOFException: Can not read response from server. Expected to read 4 bytes, read 0 bytes before connection was unexpectedly lost.
at com.mysql.jdbc.MysqlIO.readFully(MysqlIO.java:3011)
at com.mysql.jdbc.MysqlIO.readPacket(MysqlIO.java:567)
... 33 more
please help me how to fix this problem

Seeing the same issue, @shashi4414 did you ever resolve this?

Are you using the Dataproc or bdutil track?

I'm using the Dataproc track.  I had a configuration problem, this is no longer an issue for me.

noo
On Dec 20, 2017 8:14 AM, "Daniel Reed"<EMAIL_ADDRESS>wrote:

Seeing the same issue, @shashi4414 https://github.com/shashi4414 did
you ever resolve this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/spark-recommendation-engine/issues/13#issuecomment-352949007,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcaXiA8Fcg98M2BgmP_K5clXQeGUXH_aks5tCHR4gaJpZM4QQ-h2
.

Oopsie, apologies @shashi4414 . Thought you had it fixed as well.
Did you whitelist the IPs from the Dataproc instances so they can access Cloud SQL?

i triend it but issue is not solved.
On Feb 15, 2018 6:54 PM, "Matthieu Mayran"<EMAIL_ADDRESS>wrote:

Oopsie, apologies @shashi4414 https://github.com/shashi4414 . Thought
you had it fixed as well.
Did you whitelist the IPs from the Dataproc instances so they can access
Cloud SQL?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/spark-recommendation-engine/issues/13#issuecomment-365925928,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcaXiMCG4Dvj9_rPPiohUasA0wBzrE86ks5tVDADgaJpZM4QQ-h2
.


Trying to get .ipynb to run

Hello!  Thank you for posting.  As per your blog, I opened up a Paperspace instance and cloned your repo, then ran:
git clone https://github.com/cdpierse/script_buddy_v2.git
pip install -r requirements.txt
It all worked perfectly.
Then I opened script_generation.ipynb.  I tried to run it, and I got this error:
OSError: Model name './storage/models/' was not found in tokenizers model name list (gpt2, gpt2-medium, gpt2-large, gpt2-xl, distilgpt2). We assumed './storage/models/' was a path, a model identifier, or url to a directory containing vocabulary files named ['vocab.json', 'merges.txt'] but couldn't find such vocabulary files at this path or url.

Hey @Tylersuard I know what the issue is here, the model and tokenizer were stored locally at this stage of the process. Thankfully it's all up on the Transformers model hub now. If you make the change below It should all work.
Change
tokenizer = GPT2Tokenizer.from_pretrained(output_dir)
model = GPT2LMHeadModel.from_pretrained(output_dir)

To:
tokenizer = GPT2Tokenizer.from_pretrained("cpierse/gpt2_film_scripts")
model = GPT2LMHeadModel.from_pretrained("cpierse/gpt2_film_scripts")

Hi. I'm running script_generation.ipynb on Google Drive as a Colab notebook. I amended the tokenizer and model lines above as you advise, but when I get to the cell that creates the ScriptData object I get this error:
AttributeError                            Traceback (most recent call last)
<ipython-input-9-64b0d907a510> in <module>()
----> 1 dataset = ScriptData(tokenizer= tokenizer, file_path= FILE_PATH )
      2 script_loader = DataLoader(dataset,batch_size=4,shuffle=True)

/content/language_modelling.py in __init__(self, tokenizer, file_path, block_size, overwrite_cache)
     39
     40         block_size = block_size - (
---> 41             tokenizer.max_len - tokenizer.max_len_single_sentence
     42         )
     43

AttributeError: 'GPT2Tokenizer' object has no attribute 'max_len'

Any help you can provide would be appreciated.

Hi @jkurlandski01, back when I created this notebook I was using transformers with version 2.6.0 it appears that somewhere along the line in version 3.0 and now 4.0 that tokenizer.max_len has been replaced by tokenizer.model_max_length see (here)[https://huggingface.co/transformers/master/main_classes/tokenizer.html] for a description of all the tokenizers default attributes.
If you change that one line in the language_modelling.py to the new attribute name it should do the trick.

Thanks for the quick reply!
Your fix worked, then in a later cell in the Colab notebook (for epoch in range(EPOCHS):...) it crashed in the first epoch with this error:
Your session crashed after using all available RAM. If you are interested in access to high-RAM runtimes, you may want to check out Colab Pro.

I think I'm not running this .ipynb file as expected. Should it work in Google's Colab notebooks? I can import only the .ipynb file into Colaboratory, but have to manually upload the script_buddy_v2 project's files. This doesn't seem right to me. When I tried to run it as a Jupyter notebook locally it just hanged on the import statements. What am I doing wrong?
Again, thanks for your help.


target5 clone

This clone seems to be cloning two target5 elements to the left well instead of one.
$("#target5").clone().appendTo("#left-well");

Closing as a duplicate of #5072. Thanks and happy coding!


Change playing to paused on pause

I found the blinking elapsed time too subtle to indicate the paused state.
This PR introduces transport_playing_format_paused that is displayed instead of transport_playing_format_playing when musikcube is paused (or rather not playing).
Before this PR when musikcube was paused the bottom bar would show:
playing You by Nils Frahm from Screws                                  shuffle

After this PR when musikcube is paused the bottom bar shows:
paused You by Nils Frahm from Screws                                   shuffle

Being unfamiliar with musikcube's codebase I'm uncertain whether the approach I chose is the best and I'd appreciate feedback in how this feature (if it makes sense to add) could be implemented better.

Nice! Looks good to me, merged!

Yay! Thanks for merging 😄


Initial release TODOs

Internally

[ ] pull in boilerplate repo contents
[ ] migrate code
[ ] migrate tests (all existing assertions) using mocha
[ ] ensure tests pass 100%
[ ] replace any grunt.util with grunt-legacy-util
[ ] allow instances to be created instead of the lib being a singleton (prefer Ctor+prototype, but module pattern is ok when module.exports is a function)
[ ] require grunt to be passed-in to the Ctor/factory instead of requireing it
[ ] write API documentation as code comments
[ ] use existing grunt to lint / test / build readme & docs
[ ] add "built with grunt" badge :P
[ ] add missing tests to ensure 100% code coverage

Externally
(In a PR on gruntjs/grunt)

[ ] replace core internal module with this project
[ ] ensure orphaned dependencies are removed

Related:
https://github.com/gruntjs/grunt-next/blob/master/task/index.js
https://github.com/gruntjs/grunt-next/blob/master/runner/index.js


No custom variable available without REACT_APP_ prefix

cross-env version: 6.0.3
node version: 12.13.0
npm (or yarn) version: 6.13.1

What you did:
"start-cz": "cross-env LANG=cz react-scripts start"

What happened:
console.log(process.env);

{NODE_ENV: "development", PUBLIC_URL: ""}

Reproduction repository:
Problem description:
When I run my package.json script I cant get my custom variable, it only works when I use REACT_APP_* variables or NODE_ENV
Suggested solution:
create process env variables for every custom variable.

This is actually how react-scripts works. It's an intentional feature. cross-env is doing things correctly. It's just that react-scripts wont make all env variables available to your frontend code.
Read more: https://create-react-app.dev/docs/adding-custom-environment-variables/
Good luck :)


fix(android): don't set EXTRA_MIME_TYPES when using 'Any' media type on Android

PR Checklist

[x] The PR title follows our guidelines: https://github.com/NativeScript/NativeScript/blob/master/CONTRIBUTING.md#commit-messages.
[ ] There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it.
[ ] All existing tests are passing
[ ] Tests for the changes are included

What is the current behavior?
On Android, even when media type is set to ImagePickerMediaType.Any, it is not possible to select any file except image or video, everything else is greyed out.
What is the new behavior?
You can select any file.
Partly reverts changes made in #290 and #291.

Hi @xuhcc
Can you suggest other types that you would expect to get picked by image picker that are not image or video?

I'm using it for selecting text files (text/plain).

We are trying to have as much similarity between android and iOS as possible and as we are using QBImagePicker on iOS which doesn't support text files having that option only on android would bring a difference between the two. What's more, the plugin's name (imagepicker) does not imply having support for all possible file types and that's why we'd prefer to keep the option "any" only in the images/videos context.
Have you tried nativescript-mediafilepicker, it seems to be similar and supports different types of files and not only images on both platform?

Yes, I tried every plugin available, and none of them suited my needs, so I decided to add support for all media types to nativescript-imagepicker. I don't remember exactly what was wrong with nativescript-mediafilepicker, but probably it was the UI.
As an alternative approach, we can introduce a new option like allowArbitraryFileSelection which will be available only on Android, and leave current behaviour as it is, limiting user to images and videos by default.

I can also move all logic related to EXTRA_MIME_TYPES to a separate method on ImagePicker class to allow easy overriding in subclasses.

Hi @xuhcc
As stated earlier, we prefer to keep the plugin with only image picking capabilities especially given that the iOS library does not support other file types. You are free to try re-evaluate other plugins like the above-mentioned nativescript-mediafilepicker or even fork the nativescript-imagepicker plugin and add changes as you see fit. We will gladly welcome any PRs related to adding new image related capabilities and bug fixes.

@tgpetrov
I use nativescript-imagepicker plugin in an open-source project, so forking is an option of last resort for me because I'm trying to keep maintenance costs at a minimum. Hopefully we can agree on this change: #304


No matches - press enter and continue

Hi guys, thank you for your awesome plugin.
I need, if no results are found to press enter and continue to select data.
(of course, it will add nothing to selection)
Is it possible?
Thank you in advance
M.

For the time being, pressing Esc will close the dropdown.


UX of flipping button

[ ] maybe a marker on which side you are?
[ ] title of the section?

maybe solved by #174 ?

closed by #174


[Feature Request] Implement Readarr as a Service

Description
This may be branching out a bit too much but it would be useful to implement Readarr so users can request books.
Not sure if this is possible.
Desired Behavior
Readarr is implemented and books can be requested.
Additional Context
No response
Code of Conduct

[X] I agree to follow Overseerr's Code of Conduct

Just wanted to check whether there was any progress on this?

i would also like this

Does anyone know of a fork that has this feature or is being implemented in the future?

any updates on this?

+1 For Books/Audio-books!!!


[GUI] Text 'No configured flows' is not vertically centralized

In the flow list screen, the text 'No configured flows' appears at the top of the page, in the other screens (templates, devices) the text is vertically centralized.

Test OK.


Versatile JS/Python Builds

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


Using both file based and api for configuration

Hi! Thanks for reporting this feature/bug/question! Please keep / fill in the relevant info from this template so that we can help you as best as possible.
Type (choose one): Question
NLog version: 4.2.3
Platform: .Net 4.6.1
Current NLog config (xml or C#, if relevant) Both
  <configSections>
    <section name="nlog" type="NLog.Config.ConfigSectionHandler, NLog"/>
  </configSections>
  <nlog throwExceptions="true" internalLogFile="nlog-internal.log">
    <targets>
      <target name="logfile" type="File" fileName="file.log" layout="${date:format=HH\:mm\:ss} ${logger} ${message} ${exception:format=tostring} ${event-context:item=MyValue}" />
    </targets>
    <rules>
      <logger name="*" minlevel="Trace" writeto="logfile"/>
    </rules>
  </nlog>

Is it possible to configure NLog using both a file (app/web.config) and the API? If so, is it possible to provide and example.  What I have put together, doesn't seem to work as nothing is appearing in the console.
Thanks
marc

I have attempted to upload a zip of the solution.  For some reason it is not being accepted. Is there an email address I can send the zip file to?
Thanks
marc

Hey,

Is it possible to configure NLog using both a file (app/web.config) and the API?

Yes it it :)
Let's try first without emailing as I try to keep my e-mail address private.
You need to (re-)configure the configuration programmatically when the XML get's (re)loaded. You can do that with these two events

LogManager.ConfigurationChanged
LogManager.ConfigurationReloaded

See http://nlog-project.org/documentation/v4.0.0/html/Events_T_NLog_LogManager.htm
I'm not sure if you need both.
Too bad if haven't got an example for now. If you can't get above working, let me know!

Hi Julian:
Thank you for your guidance.  Unfortunately the LogManager.ConfigurationChanged and LogManager.ConfigurationReloaded events are being raised too late in the process.
In my little sample app, ConfigurationChanged didn't get raised until program exit.
I did find a solution, after I modify the configuration, I need to call LogFactory.ReconfigExistingLoggers().
After that, everything works as expected.
Here is the little sample code:
class Program
{
    // Define a static logger variable so that it references the logger instanced named "Scribe"
    private static readonly Logger log = LogManager.GetCurrentClassLogger();

    static void Main(string[] args)
    {
        var consoleTarget = new ColoredConsoleTarget();
        consoleTarget.Name = "console";
        log.Factory.Configuration.AddTarget("console", consoleTarget);

        consoleTarget.Layout = @"${date:format=HH\:mm\:ss} ${logger} ${message} ${exception} ${event-context:item=MyValue}";
        var rule1 = new LoggingRule("*", LogLevel.Debug, consoleTarget);
        log.Factory.Configuration.LoggingRules.Add(rule1);
        log.Factory.ReconfigExistingLoggers();

        log.Info("Entering Application.");

        Bar bar = new Bar();
        bar.DoIt();

        log.Info("Exiting application.");

        Console.WriteLine("Press any key to exit ...");
        Console.Read();
    }
}

public class Bar
{
    private static readonly Logger log = LogManager.GetCurrentClassLogger();

    public void DoIt()
    {
        LogEventInfo theEvent = new LogEventInfo(LogLevel.Debug, "", "Oops! I did it again");
        theEvent.LoggerName = "log4netexample.Bar";
        theEvent.Properties.Add("MyValue", "Another really bad song.");
        log.Log(theEvent);

        theEvent = new LogEventInfo(LogLevel.Error, "", "Exception Test");
        theEvent.Exception = new ArgumentException("Some exception");
        log.Log(theEvent);
    }
}

Here is my app config:

Thank you for all your help.
-marc

Thanks !
So this is tricky when auto-reload is enabled. Maybe if auto-reload is enabled, are the events in time.
PS: I enabled syntax highlighting in your post.

@304NotModified yes if I replace log.Factory.Configuration with LogManager.Configuration, it will work also.
I have not tried the Configuration events.
I understand that depending upon my usage scenario, I might need to merge the file based and API based configurations every time the configuration events are fired.
I have not thought though all scenarios yet, I'm working on a prototype and understanding NLog.
Thanks for all the help and guidance.
-marc

I think the events will work, but not in a console as you have only one thread (and there is a delay of ~1000ms)
So it should be something like this, which can be tested in a website
using System;
using NLog;
using NLog.Config;
using NLog.Targets;

class Program
{
    // Define a static logger variable so that it references the logger instanced named "Scribe"
    private static readonly Logger log = LogManager.GetCurrentClassLogger();

    static void Main(string[] args)
    {
        ExtendNLogConfig();

        LogManager.ConfigurationReloaded += LogManager_ConfigurationReloaded;
        LogManager.ConfigurationChanged += LogManager_ConfigurationChanged; ;

        log.Info("Entering Application.");

        Bar bar = new Bar();
        bar.DoIt();

        log.Info("Exiting application.");

        Console.WriteLine("Press any key to exit ...");
        Console.Read();
    }

    private static void LogManager_ConfigurationChanged(object sender, LoggingConfigurationChangedEventArgs e)
    {
        ExtendNLogConfig();
    }

    private static void LogManager_ConfigurationReloaded(object sender, LoggingConfigurationReloadedEventArgs e)
    {
        ExtendNLogConfig();
    }

    /// <summary>
    /// Extend the logging rules in the nlog.config with programmically rules.
    /// </summary>
    private static void ExtendNLogConfig()
    {
        //don't set  LogManager.Configuration because that will overwrite the nlog.config settings
        var consoleTarget = new ColoredConsoleTarget();
        consoleTarget.Name = "console";
        LogManager.Configuration.AddTarget("console", consoleTarget);

        consoleTarget.Layout = @"${date:format=HH\:mm\:ss} ${logger} ${message} ${exception} ${event-context:item=MyValue}";
        var rule1 = new LoggingRule("*", LogLevel.Debug, consoleTarget);
        LogManager.Configuration.LoggingRules.Add(rule1);
        LogManager.ReconfigExistingLoggers();
    }
}

public class Bar
{
    private static readonly Logger log = LogManager.GetCurrentClassLogger();

    public void DoIt()
    {
        LogEventInfo theEvent = new LogEventInfo(LogLevel.Debug, "", "Oops! I did it again");
        theEvent.LoggerName = "log4netexample.Bar";
        theEvent.Properties.Add("MyValue", "Another really bad song.");
        log.Log(theEvent);

        theEvent = new LogEventInfo(LogLevel.Error, "", "Exception Test");
        theEvent.Exception = new ArgumentException("Some exception");
        log.Log(theEvent);
    }
}

@304NotModified Thanks for the example.  I will give it a try.

Thanks for testing!

See also wiki: https://github.com/NLog/NLog/wiki/Combine-XML-config-with-C%23-config


获取群中用户列表获取到的id并不是微信号 无法查找到账号 添加好友失败！

🚨 The issue tracker is not for questions 🚨
If you have a question, please ask it on https://stackoverflow.com/questions/tagged/wechaty
[question]
获取群中用户列表获取到的id并不是微信号 无法查找到账号 添加好友失败！

Thanks for your issue.
This problem is due to the puppet which you used is not supported get room member's WeChat account, it could only get the wxid of room member who is not your friends.
So we could not add room member as friends by puppet wechaty-puppet-padplus or wechaty-puppet-macpro.

@su-chang 我看官方文档写的支持，那我应该怎么处理，才可以添加群里的人呢

Sorry, it is not supported by wechaty-puppet-padplus or wechaty-puppet-macpro.

@su-chang
项目有支持微信添加好友的功能吗

If you can login in web wechat, click me to test.
You could have a try for wechaty-puppet-puppeteer

@su-chang
项目有支持微信添加好友的功能吗

Of course, see : https://wechaty.js.org/api/friendship#friendship-add-contact-hello-promise-less-than-void-greater-than

@su-chang wechat官方网页版的都提示不能登陆

@su-chang

需要发送好友请求的联系人是通过什么获得呢?
还是固定已知的微信号实现添加好友呢?

wechat官方网页版的都提示不能登陆

It means that you can not use wechaty-puppet-puppeteer.

还是固定已知的微信号实现添加好友呢?

What you think is right.

@su-chang  我必须知道别人的微信号 才可以吗 ？

这个通过这个ID是查找不到的    有没有别的处理方式  可以在群里 自动加好友

Of course, just like what you can do in WeChat App. We could add friend only by phone number or WeChat account.
Sorry, add room member as friends is not support now.

@su-chang 感谢您的回答 谢谢您

@vaevas You are welcome.

看完上面的，到底有没有可能获取到群成员的微信号？
还是说仅仅pad和macos的puppet不行？那有哪个puppet可以的没？

@terranc I believe the WXWork, Donut, Rock, and PadLocal are all supporting to get the wxid of the room members.
Learn more about the Wechaty Puppet Services at https://wechaty.js.org/docs/puppet-services/


feature: azure document service deployment

Still work to do to make azure deployments work better in general. Also questions around the default georeplication locations.

Closed in favor of: #174


[Do not merge] Rework tagging sync

This changes the way tags are indexed by Rummager.
Context
Currently, every time we send a document to elasticsearch, the document is first "prepared" by the DocumentPreparer class. That adds some extra information to the document (like popularity) and copies some fields (format to _type). It also looks up the tags for the document in the content-api.
For example, when indexing the payload for
https://www.gov.uk/state-pension-age
it will look up which tags have been added to this artefact:
https://www.gov.uk/api/content/state-pension-age
(At the time of writing, this is the "State Pension" browse page)
Changes

This PR changes the datasource, so that rummager looks up the tags in the publishing-api rather than the contentapi. This has been made possible by work to make publishing-api the accurate source of all publishing-api tags.

It replaces way we index things from the message queue. Currently there's an incomplete implementation (which has been switched off on all environments and currently isn't running), which consumes messages from the publishing-api message queue and indexes the payloads.

This message queue consumer has been changed to treat new messages as a "change notification". Whenever the message queue consumer receives a new message, we simply re-add the existing document to the index, forcing a new tag-lookup by the DocumentPreparer, as described above.
This gives us a guarantee that we always index the latest version of the links. If we were to index the links from the queue directly (like the current implementation does) we would run the risk of indexing stale data if the delivery of the message had been delayed.
The trade-off here is that we have to make an extra request to the publishing-api for data that has already been sent to rummager in the message queue payload.
For review

TagLookup is called by the DocumentPreparer and does the fetching of links from the publishing-api.
The MessageProcessor is responsible for receiving the message from the queue, and reporting back to RabbitMQ. The ChangeNotificationProcessor takes the actual message payload and triggers a re-index of the document.

Trello: https://trello.com/c/RrN7B6xQ
Deploy

[ ] We need to be sure that the links in the publishing-api are accurate (https://trello.com/c/qXmMfTOZ)
[ ] Panopticon changes to remove tagging interface should be deployed (https://trello.com/c/4vU2VrC0)

The general approach here looks good to me. I'm not sure I have a complete understanding of the "change notification" change though: IIRC this minimises race conditions in the case where there are extra publishing api writes between the time a change is first written to publishing api and the time it is written to ES.
I think there is still a (much less likely) race condition within rummager itself since multiple processes consume from the queue and may process messages out of order - @benhyland commented on this on the trello ticket. I assume this will go into a separate PR though?
Also - do we have a separate ticket for sorting out the inconsistency with lead/supporting organisations in the publishing/frontend apps? We should unify this sooner rather than later as it adds extra complexity to everything that touches the tags.

Somebody should pair with @tijmenb on the remainder IMO.

Closing to keep our todo-list clear, see Trello for update:
https://trello.com/c/RrN7B6xQ/593-rework-the-way-rummager-indexes-links

I wonder if an ADR might be useful? Something which expands on the 'why' of this change. Appreciate the commit is a decent summary, but an ADR would call out this reasoning in a place that's arguably more visible/discoverable than the git log.
Might also be an idea to explain the temporary nature of this change, and how we expect it to be replaced by a 'better' message queue based implementation later on.

A few smallish comments from me,  but overall this looks solid. Happy to review again / merge once build's green and remaining questions addressed.


Uninformative error message opening file

Using the petpvc executable, I'm getting an error opening the input file. Even in Debug mode, the error isn't overly informative:
[Error]	Cannot read PET input file: /Users/rich/Documents/Data/MCIR/Output-MCIR/7_AC/AC_g4.nii

The file definitely exists and I can open it with other packages. I haven't checked the source code to figure out what the error is, but would a different message be possible (e.g., "could not locate file" if that were the problem).

petpvc now displays itk exception information when file does not exist or cannot be read.


OSGeoLive 15 Update

Docs

FEATUREREQUEST
Description:

We have to update the GeoStyler installation on the upcoming OSGeoLive 15. This includes updating docs, install scripts, screenshots, etc.
Following parts have to be updated (probably incomplete):

https://github.com/OSGeo/OSGeoLive-doc/blob/master/doc/overview/geostyler_overview.rst
https://github.com/OSGeo/OSGeoLive-doc/blob/master/doc/quickstart/geostyler_quickstart.rst
https://github.com/OSGeo/OSGeoLive-doc/tree/master/doc/images/projects/geostyler
https://github.com/OSGeo/OSGeoLive/blob/master/bin/install_geostyler.sh
transifex

Related PRs:

https://github.com/OSGeo/OSGeoLive-doc/pull/766
https://github.com/OSGeo/OSGeoLive/pull/337

This issue will be closed as soon as the PRs were merged.

@jansule seems like the PRs are merged. Can we close this?

I would like to wait until the translations in transifex were also updated. This ticket serves as a reminder to check and apply possible translations. Afterwards I will close this issue.


obsolete: GO:0043492 ATPase activity, coupled to movement of substances

Not a necessary grouping term.
Has no children, but is labelled not for direct annotation
(note that there are probably other terms   which need the ancestor of this term as a parent)

Hi @ValWood
I'll merge this into GO:0042626 ATPase-coupled transmembrane transporter activity
OK ?

You could. The grouping was possibly established for ATPase motors and ATPase transmembrane transporters because they are both 'transport' but these types of transporters have no commonality except the use ATP for energy.
So, I agree with the merge, but it should probably be 'related' rather than broad.

In fact, if it has no direct annotations, do the merge, but exclude the label as a synonym. It isn't  a helpful synonym (but the ID will be preserved)

Actually, there are 2 annotations and they are both transmembrane transporters so yes go ahead and merge.
Both of these annotations are ISS/O but the with filed is empty. Shouldn't they be rejected anyway?

Those annotations are from GeneDB to the ABCF1 LmjF.03.0160:mRNA gene
@Achchuthan Can you please make sure there is a value in the 'with' field?
Thanks, Pascale

Done here https://github.com/geneontology/go-ontology/pull/19995/commits/cd33db36b43303785ee5f5219acb9332351dfd07


Rename "Disconnect" type to "Connection" type for less confusion

since "Disconnect" does not always disconnect I think it'd be fair to rename it in order to avoid confusion.

Rejected due to massive merge conflict (sync your fork with upstream).


Google fonts

I see your css is depending from Internet as referencing Google fonts.
Could you please fix this by adding needed stuff in local assets or providing sass or css sources.

This depends on how you import semantic.min.css in your application.
Right now, i did this for Demo on https://ng-semantic.herokuapp.com
You can see in source that i am pulling everything from local source now.

Thanks.
Your components are awesome.


Add custom handler for image uploading

This change adds the ability to define a custom image upload handler. It passes the instance of the markdown editor so the dev can still specify progress using MarkdownEditor.ImageUploadProgressed

Thank you for your PR!
Enrico

The new version of the package includes your PR
Thanks
Enrico


add defaulting for awscluster description

Towards: https://github.com/giantswarm/giantswarm/issues/14025
Spec.Cluster.Description : default to Unnamed cluster

Tested on gauss and all is well


[Design] - Applicant Job Browser

Status

[x] Being Researched
[x] Prototyping and Testing
[ ] Data Story
[ ] Ready For Dev

Design Goal
Please explain the primary goals of this design work.
Eventually, as the platform scales, the browse jobs page will have to support a far greater number of jobs. This means that users will need a more efficient way to browse, which includes being able to search by keyword, filter by things like classification, language, location, etc.
Research
Please list/link to all research performed for or referenced in this design work. This can include research you've done in the past. Include research relevant to insights from team members as well. Don't be afraid to ask for references or documents! This section should also incorporate data collected from user testing.

https://emploisfp-psjobs.cfp-psc.gc.ca/psrs-srfp/applicant/page2440?fromMenu=true&toggleLanguage=en
https://ca.indeed.com/jobs?q=&l=Ottawa%2C+ON (competitor, browse jobs)
https://www.monster.ca/jobs/l-ottawa-on (competitor, browse jobs)
Both Indeed and Monster really struggle with information density and whitespace usage - there's no real focal point on their browse pages, and this makes skimming very, very difficult.
https://www.amazon.ca/s?k=teapot&ref=nb_sb_noss_1 (comprehensive filters)
Ambient discussion about government jobs has indicated that clicking on a job, only to discover you don't meet the classification requirements/language profile creates a lot of frustration to users.

GC Employee Feedback (Version 2)

Being able to save filters for later is great.
Recommended jobs are nice (if they work), but could potentially be more visually distinct.

Feedback from Discussion w/ @JasmitaChawla (Version 2)

A way to filter by skills might be helpful.
It might be beneficial to separate classification from level, to help shorten the filter lists.
~There's a feeling that classification might be a more important filter than department, because it's more likely to be consistent or similar across an applicant's history.~
~We can definitely remove max salary.~
~We should add a "search" or "filter" button to make the filtering a decisive action instead of automatic.~

Feedback from Discussion w/ @vid (Version 2)

Load More / Infinite Scrolling has disadvantages, perhaps use pagination instead?

Referencing: https://uxplanet.org/ux-infinite-scrolling-vs-pagination-1030d29376f1
Referencing: https://medium.com/@dalpattapaniya/infinite-scrolling-vs-pagination-f237592f339a
Referencing: https://www.orbitmedia.com/blog/why-we-choose-the-view-more-button-instead-of-pagination/

~Showing number of results would be super helpful.~

Existing Job Summary:

Assumptions
Please list all assumptions made during your design work. This should include behavioural assumptions made by the team that have no direct research references.

Jobs should be easy to skim
Users will benefit from having jobs recommended to them based on their existing skills
Users want to save a default set of filters so that they can browse quickly
Filters should be below recommendations - going off my gut here - show me hyper local job matches first, that I can collapse or dismiss, then show me all jobs
Users will benefit from choosing to show recommended jobs on the homepage instead of "latest jobs"
No one will ever want to filter their list of jobs by a maximum dollar amount.

Important Notes / Copy
This is a space to include important notes for other team members, including things like key copy, questions that need to be asked at a later date, etc.

Do we need more than 3 recommendations? If yes, how many?
Is keyword search actually useful?

Wireframes / Images
Working Prototype: https://joshbeveridge.ca/tc-browse/
Version 1

Version 2

Filters were moved to the left on desktop, because it was a waste of space to keep them on the top.
Personalized filters were condensed into the filter bar, because why replicate things.
Recommendation explanations were generalized, because it didn't make much sense to clutter the UI with repeated info (it's now a single sentence above the recommended jobs).
Minor changes to the layout of condensed job cards do adapt them for more info.
Added a keyword search.

Version 3

Functional search button has been added, and save filters have been moved below with some accompanying text.

A modal was added to confirm the deletion of saved filters.

Number of search results is now displayed.
A bug was fixed where filters hidden on mobile wouldn't show up on desktop if you stretched the viewport (thanks @nicolsonaitken)
Classifications were moved above departments.
Max salary was removed.
A (flexible) number was added to the load more button.

Final Data Story
Use this section to summarize where we started, and how and why we got to the approved design. This section must be complete before the issue can be handed off to development.
TBD

I would like the ability to show/hide this entire filter. If I can save it, I don't really think I need something that will take up almost 1/3 of my screen.

@nic - I'll fix that bug, thanks for pointing it out.
In terms of hiding the filters on desktop - what's the inherent benefit? Content in the right shouldn't be much wider than it already is (the wider the content, the harder it is to skim), so by hiding the filters you'd just be centring what's already there.

@joshdrink Maybe it's just a matter of preference or how people take in information differently. In the image below, I simply find it much easier to read the version on the right versus the version on the left.

Josh, I'd like a filter to enable "old" or "Closed" jobs to be shown as well. In the short to medium term, I see that as the main place we will actually be using these filter: to search old jobs.

Moved to Trello


Configure Validation for Requiring only 1 Teams Owner or 2

Suggestion
Good day friends,
I'd like to remove the validation for a second owner.
However, this seems to have an impact on the workflow behind the scenes as it is expecting an account. Is there any documentation on adjusting the 2nd owner required field?
Thanks in advance for any insights !!!!

Updated the logic and initial default data load for cmbOwners combo boxes.


feat(website): add campaigns page

Please check if the PR fulfills these requirements:

[ ] The commit message follows our code of conduct and contributing guidelines
[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added / updated (for bug fixes / features)

Please include the following information in your pull request:

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Any other information (if needed):

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: grzim:x: DmytroHryshynYou have signed the CLA already but the status is still pending? Let us recheck it.

Closing as stale.


Setting label centered text loses alignment

When I set the text of my label within the viewDidAppear it works fine:

But when I set it within the viewDidLoad the alignment is left from the center as you can see in the image:

@cytryn I've tried reproducing this in the included JVFloatLabeledTextFieldViewController, but it seems to work fine in both cases. Can you share some code that demonstrates the problem?

Here is a bit of a simple code where the problem is happening:

But the thing is that if I call [self customizeAppearance] in viewDidAppear instead of viewDidLoad, it works.

Apparently version 1.0.3 fixed this, but generated a different bug as described in #98

1.0.4 broke this again apparently...
My placeholder are back in the middle in a strange way just like my previous post.

After walking through entire code and playing with it, I found this workaround.
Set the "floatingLabelYPadding" property of the textview to 0.
myTextView.floatingLabelYPadding = 0


Sink should read filters out of the ConfigMap

Related to #237 #222 #221
The sink needs to mount the Filters ConfigMap, and read the filters out of it. Hopefully it can be parse in yaml format. Then we can have a struct that can be used to load the data into. There should be a separate thread that checks if there is a change to the mounted configmap and updates the filter struct.
The filter struct will need to be thread safe. We can use a RWMutex for that.
Acceptance Criteria:

the sink can read the filters out the of the configmap
the sink can update the filter data when the configmap is updated

I'm looking at go-cel right now, but I'm not sure how to get it work with maps that have arbitrary values. I'm falling back to json path for now


LoadVersions only returns the current version

Currently LoadVersions only returns the current version
Updated LoadVersions to collect both targetReferences and sourceReferences and concatenate them, and add the current workplan to the final result. Return all the versions ordered by version.

The initial implementation always assumed the latest version is given, but from there it recursively went all the way back (do-while loop). Your current implementation looks at sources and targets, but only one level.

I restored and fixed the LinQ approach but also found another issue in the process. The model definition of WorkplanReferences.Target => Workplan.SourceReferences is inverted and really confusing.


Update to use metaschema 2.0.1

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[ ] description of changes is added in CHANGELOG.md

Landed in https://github.com/metarhia/metadomain/commit/8a60bb4099b422a0c13d8a072466998191d25986


Remove usage of Node API in UI code

Examples: os, fs, process, path
Ideally don't even attempt this until everything is on named icons.

Since we are bundling AzCopy extension in 1.33.0, this needs to be done at least in AzCopy dialogs.


How to use with react component

Hi,
Before checking this component, I used react-native-video to play my stream file.
My component looks like this:
import React from 'react';
import PropTypes from 'prop-types';
import Video from 'react-native-video';

function AudioPlayer(props) {

  if (props.paused) {
    return null;
  }

  const { source, paused, onLoad, onError } = props;

  return (
    <Video
      source={{ uri: source }}
      volume={1.0}
      muted={false}
      onLoad={onLoad}
      paused={paused}
      onError={onError}
      playInBackground
      playWhenInactive
    />
  );
}

AudioPlayer.propTypes = {
  source: PropTypes.string.isRequired,
  paused: PropTypes.bool.isRequired,
  onLoad: PropTypes.func.isRequired,
  onError: PropTypes.func.isRequired
};

export default AudioPlayer;

Since react-native-video is not working great on android while playing in background, i was looking for this component but I do now understand how I am supposed to use redux to dispatch action to the player.
In my app, my ui component dispatch redux actions like Play, Pause, etc. I have a component over  my audioplayer bind to redux that pass props to my AudioPlayer to update it depending on user action.
My question: How can I accomplish similar lifecycle using this component ? All the events are in the headless task and I do not understant how am i supposed to update the props of my components using this.
I want to keep a similar API for my component as I am going to use react-native-track-player only on android.
Thank you.

As you have mentioned, react-native-video works as a UI component. But due to the fact that we need audio to work in background (with lockscreen controls), react-native-track-player works as an API.
You can update your redux store with new information from through the headless task

How to access store in the headless task ?

You should just be able to import it?  I use Mobx, so I just import store from '../store/PlayerStore'

You also can just pass it to the function like so:
// index
const store = ...
....registerHeadlessTask('TrackPlayer', () => require('event-handler.js').bind(null, store));

// event-handler.js
module.exports = async (store, data) {
    if(event.type == '...') {
        store.dispatch(...);
    }
};

Thanks for the help, it works

Thanks for help @Guichaguri
Its working

Thanks for the help. It works


Update iperf.py

For iperf3 --mss is not works. For iperf3 we need --set-mss.

Hey @vortex610. Thanks for your contribution.
Any changes must be submitted through gerrit, you can find documentation to help you with that here.
Also bugs need to be filed using launchpad

Thank you for contributing to openstack/shaker!
openstack/shaker uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


customerEphemeralKeySecret format does not match expected client secret formatting.

Describe the bug
The error "customerEphemeralKeySecret format does not match expected client secret formatting." is thrown when trying to do initPaymentSheet(). This only happens on iOS, the same code works perfectly on Android. I'm only working with CC, no ApplePay or GooglePay.
Expected behavior
Initialize the payment sheet and proceed to the next step where the payment sheet is displayed through presetPaymentSheet();
Screenshots
This is the error thrown by initPaymentSheet()

Implementation

You  can be sure I'm properly getting and sending customerId, customerEphemeralKeySecret and setupIntentClientSecret. I wan to highlight the fact this same code is working on Android, it only fails on iOS.
I'm using the version 0.19.0 of this plugin. This is the version I need for iOS 12.4 which is the target the project has setup on xCode, so I cannot update the version of this library.
Do you guys know why this happens only on iOS? I can't find anything on the internet. I'm lost.
I'll appreciate your help.

@GustavoLucero , have you found a solution? I'm also encountering the same issue on iOS. Initially, it was functioning correctly, but now I'm suddenly receiving this error. It's working fine on Android.

Are you sure, you are not passing the ephemeralKeyId rather than the ephemeralKey itself? I encountered an issue for that.

If you hit this error, it's almost certainly due to the fact that you are passing the wrong value as your EphemeralKey secret. Many developers mistakenly pass the EphemeralKey id which looks like ephkey_123456 instead of passing the EphemeralKey secret that looks like ek_test_abc123 instead.
On Android, we incorrectly don't validate the value the same way which makes it look like it works.


remove no labelSelectors svc for waypoint

Please provide a description of this PR:
When svc has no label, it will also be added to waypoint
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[*] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hmm.. this is not strictly accurate since a service without selector could have manually added endpoints not selecting the SA we care about. Maybe we should be looking at ServiceInstances instead of re-computing the labels here - this will help when we do ServiceEntries in the future as well

Hmm.. this is not strictly accurate since a service without selector could have manually added endpoints not selecting the SA we care about. Maybe we should be looking at ServiceInstances instead of re-computing the labels here - this will help when we do ServiceEntries in the future as well

Yes, I found some useless configurations in waypoint while testing and found that the code here introduced them, the problem is not serious, just some redundant configurations.

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Known issue: Vivisect: rundll32.exe dll-relative breakpoints fail

rundll32.exe loads the DLL specified in its argument more than once, causing vtrace to add the breakpoint, discard the delayed breakpoint entry, and never re-resolve the breakpoint on subsequent loads. This will require a fix for Vivisect, but is documented here to advertise a resultant limitation of flare-qdb.
I see that vtrace/platforms/base.py removes deferred breakpoints once resolved, as follows; this may be where to start:
    def _activBreakpoints(self):

        """
        Run through the breakpoints and setup
        the ones that are enabled.
        NOTE: This should *not* get called when continuing
        from a fastbreak...
        """

        # Resolve deferred breaks
        for bp in self.deferred:
            addr = bp.resolveAddress(self)
            if addr != None:
                self.deferred.remove(bp)
                self.breakpoints[addr] = bp

        for bp in self.breakpoints.values():
            if bp.isEnabled():
                bp.activate(self)

A work-around: use Python/ctypes as your target. It only loads the DLL once and doesn't fool Vivisect. Example by export name with a stdcall function:
python -c "import ctypes; ctypes.windll.mymalware.MyFunction()"
Example by ordinal with a cdecl function:
python -c "import ctypes; ctypes.cdll.mymalware[2]()"


The implementation of higher-rank types is unsound

I was playing around with higher-rank types and discovered that an inner-quantified type could escape, leading to unsoundness:
let foo f: forall b. (forall a. a -> b) -> b = f ()
let id x: forall a. a -> a = x
let false: forall a. a = foo id // Oops
let some_int: Int = false
some_int // Undefined behaviour

foo id should not typecheck, because forall a. a -> a and forall a. a -> b should not unify.

I might be interested in having a look at the typesystem implementation. Is there some documentation/a paper that describes it somewhere ?

I can dig up the papers/implementations I have looked at tomorrow (and I should link these somewhere in the docs as well). I found the issue in subsumption in any case, just have some fallout left to fix.

Cool, thanks !
In passing, here is another failure case that makes clearer where the flow of typing fails:
let foo f: forall b. (forall a. a -> b) -> b = f ()
let some_int: Int = foo (\a -> if True then a else 42)
some_int

The papers on 1ML is a good base for what the type system is supposed to be https://people.mpi-sws.org/~rossberg/1ml/ (at least at a high level).
Implicit arguments are mainly inspired by Ocamls (WIP) implementation https://github.com/ocamllabs/ocaml-modular-implicits and https://www.cl.cam.ac.uk/~jdy22/papers/modular-implicits.pdf (the paper is pretty light on implementation details).
(and more, need to remember which ones though)


upgrading to 0.39.2 gives Access denied in terragrunt plan

Hi there,
while trying to upgrade from 0.36.6 to 0.39.2, I get the error below:
Error checking if SSE is enabled for AWS S3 bucket my_bucket_name: AccessDenied: Access Denied
Any idea what might fix this?
Cheers

found the problem, additional to the existing IAM policy, I had to add the following:
      "s3:GetEncryptionConfiguration",       "s3:GetBucketPolicy",       "s3:GetBucketPolicyStatus",       "s3:GetBucketPublicAccessBlock",
Closing issue


File tree interactivity enhancements

Check for existing issues

[X] Completed

Is your feature request related to a problem?
I want to be able to perform operations in the file tree with the keyboard.
Describe the solution you'd like
It should be possible to do any of the following with the keyboard

[x] focus the file from the currently active buffer (sort of happens implicitly because the file tree tracks the current active buffer, but that's not necessarily what I want?)
[x] navigate throughout the tree with arrow keys
[ ] create files or folders
[ ] delete files or folders
[ ] rename files or folders

If applicable, add mockups / screenshots to help present your vision of the feature
No response

I think this issue can be closed now, as we have these actions listed here.  We currently don't have the ability to move or delete files / directories in bulk, but there is a new issue for that here:

https://github.com/zed-industries/community/issues/1236


tree: fix newer pylint warnings

Fix a set of new pylint warnings. This starts with assemblers/, devices/, inputs/, and mounts/. More to come.
This is triggered by #1106.

CI on this failed, but auto-merge still merged it? I added pylint-suppressions which old pylint does not know, so we need a suppression for old pylint to not complain about unknown suppressions sigh


feat: Add Sync to Backend in acvm

Description
Problem*
Needed for threading in https://github.com/noir-lang/noir/issues/2196.
Resolves none
Summary*
Add Sync to Backend in acvm
Additional Context

PR Checklist*

[x] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Hey there, thanks for submitting the PR!
Could you elaborate a bit on how you plan on tackling https://github.com/noir-lang/noir/issues/2196 and how would this help execute that plan?

Obviously, it would be iterative for me, as I'm fairly new to the code-base. Pls, see https://github.com/noir-lang/noir/compare/master...skaunov:noir:2196-Run-test-cases-for-nargo-test-in-parallel for the first step I have take. As you can see, the main idea is to feed test cases one-by-one to few thread running in parallel.
It was some time ago, I recollect that I tried to just unfold workspace to see how it would work, so there's mainly pasted code for index part. And after analyzing available options I couldn't see a way to run a test in a new thread without sending a reference to Backend to the thread.
If there's any other preferred way, just show how take it! X)


TF-TRT MNIST test with V2 converter

This PR extends the MNIST test to use the V2 converter. We create a saved model, and convert it both with and without dynamic shape mode.
Tagging @bixia1 for review. Tracker #45481

@bixia1 this is ready for review

@tfeher Can you please resolve conflicts? Thanks!

@gbaned I have resolved the conflicts.


It is possible to have exactly one "self" relationship for a class. Do not gray out relationships if less than two classes

fixed by allowing self relationships and removing the check for two classes before allowing the relationship button to be pressed.


Pytorch to IR AttributeError: 'OrderedDict' object has no attribute 'state_dict'

Platform (like ubuntu 16.04):
Python version:
Source framework with version (like pytorch0.40 with CPU):
Destination framework with version (like tensorflow1.8.0 with CPU):
Pre-trained model path (webpath or webdisk path):
Running scripts:
mmtoir -f pytorch -d inception_v3 --inputShape 3,299,299 -n /**/.torch/models/inception_v3_google-1a9a5a14.pth
show Error Information below:
Traceback (most recent call last):
File "/usr/local/bin/mmtoir", line 11, in
load_entry_point('mmdnn==0.2.3', 'console_scripts', 'mmtoir')()
File "/home/.local/lib/python2.7/site-packages/mmdnn/conversion/_script/convertToIR.py", line 192, in _main
ret = _convert(args)
File "/home/.local/lib/python2.7/site-packages/mmdnn/conversion/_script/convertToIR.py", line 92, in _convert
parser = PytorchParser(args.network, inputshape[0])
File "/home/.local/lib/python2.7/site-packages/mmdnn/conversion/pytorch/pytorch_parser.py", line 81, in init
self.pytorch_graph = PytorchGraph(model)
File "/home/.local/lib/python2.7/site-packages/mmdnn/conversion/pytorch/pytorch_graph.py", line 59, in init
self.state_dict = _unique_state_dict(self.model)
File "/home/.local/lib/python2.7/site-packages/torch/jit/init.py", line 81, in _unique_state_dict
state_dict = module.state_dict(keep_vars=keep_vars)
AttributeError: 'OrderedDict' object has no attribute 'state_dict'
Thanks!

Hi @muzi-8 , please refer to this. Thanks!

@namizzz Thanks for your reply! However, I could't solve it by myself, so can you say it more detailed? I mainly refer to the document PyTorch README

@muzi-8 I've met the very problem you presented in the OP. File "/usr/local/lib/python3.5/dist-packages/torch/jit/init.py", line 259, in _unique_state_dict
state_dict = module.state_dict(keep_vars=keep_vars)
AttributeError: 'collections.OrderedDict' object has no attribute 'state_dict'
How did you solve the problem??

same here with py3.6.4, torch1.1.0, mmdnn 0.2.5.

dear,
Did you solve AttributeError: 'collections.OrderedDict' object has no attribute 'state_dict' ?

Has anyone solved it?

Has anyone solved it?