Debugger crashes with "Extension host terminated unexpectedly"

Issue Type: Bug
Attempting to start debugging on a large coffeescript/javascript project.  This had been working fine until the new javascript engine in latest stable vscode.
I see the following in the javascript console:
/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:266 [Extension Host] Unknown request: evaluate
t.log @ /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:266
/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1670   ERR No application in the Launch Services database matches the input criteria.: Error: No application in the Launch Services database matches the input criteria.
abstractExtensionService.ts:166 Extension host terminated unexpectedly.

VS Code version: Code 1.47.2 (17299e413d5590b14ab0340ea477cdd86ff13daf, 2020-07-15T18:18:50.054Z)
OS version: Darwin x64 19.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
32.00GB (10.30GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Can you please do the following two things?

See if the nightly build fixes the issue for you
If not, collect a trace log using the following instructions:

If you're able to, add "trace": true to your launch.json and reproduce the issue. The location of the log file on your disk will be written to the Debug Console. Share that with us.
⚠ This log file will not contain source code, but will contain file paths. You can drop it into https://microsoft.github.io/vscode-pwa-analyzer/index.html to see what it contains. If you'd rather not share the log publicly, you can email it to<EMAIL_ADDRESS>
/needsMoreInfo

I cannot recreate the issue in the nightly build.  Cheers!


After plugin upgrade, docker has to be restarted to see all the volumes

Error: If plugin is upgraded with container running, docker volume ls does not show all volumes. Docker has to be restarted to see all volumes.
Test Steps:
1. Installed 0.15 vDVS plugin
2. Created two volumes.
3. Ran a container with one of the volumes mounted.
4. While container was running, uninstalled the 0.15 plugin and installed 0.16 plugin.
5. After 0.16 plugin was successfully installed, ```docker volume ls``` does not show both volumes
6. Restarted the docker and could see both the volumes but docker restart caused container to be exited.

Is this behavior expected? Should user be stopping a container to do plugin upgrade?
Logs have been attached at the end

Steps and their output are as follows:

Installed 0.15 plugin

root@photon-machine [ ~ ]# docker plugin install --grant-all-permissions --alias vsphere15 vmware/docker-volume-vsphere:0.15
0.15: Pulling from vmware/docker-volume-vsphere
967f49dfadaf: Download complete
Digest: sha256:64a50830c2fa9befbede6efca3f28d2e2d8b4e595703f1b5984470579656e935
Status: Downloaded newer image for vmware/docker-volume-vsphere:0.15
Installed plugin vmware/docker-volume-vsphere:0.15
root@photon-machine [ ~ ]# docker plugin ls
ID                  NAME                DESCRIPTION                           ENABLED
fe44e3b62f64        vsphere15:latest    VMWare vSphere Docker Volume plugin   true
root@photon-machine [ ~ ]#

Created two volumes

root@photon-machine [ ~ ]# docker volume create -d vsphere15 --name TestVol15
TestVol15
root@photon-machine [ ~ ]# docker volume ls
DRIVER              VOLUME NAME
vsphere15:latest    TestVol15@local2
root@photon-machine [ ~ ]# docker volume create -d vsphere15 --name TestVol15_XX
TestVol15_XX

Mounted one of the volumes to a container

root@photon-machine [ ~ ]# docker run -it --volume-driver=vsphere15:latest -v TestVol15@local2:/vol1 --name ubUpgrade ubuntu
root@459006671aa3:/# echo "Hello" >> /vol1/hello
root@459006671aa3:/# cat vol1/hello
Hello
Hello World
Hello World CAT
root@459006671aa3:/#

While container was running, uninstalled vDVS 0.15 plugin and installed 0.16 version

root@photon-machine [ ~ ]# docker plugin ls
ID                  NAME                DESCRIPTION                           ENABLED
fe44e3b62f64        vsphere15:latest    VMWare vSphere Docker Volume plugin   true
root@photon-machine [ ~ ]# docker plugin rm -f vsphere15:latest
vsphere15:latest
root@photon-machine [ ~ ]# docker plugin ls
ID                  NAME                DESCRIPTION         ENABLED
root@photon-machine [ ~ ]# docker plugin install --grant-all-permissions --alias vsphere vmware/docker-volume-vsphere:latest
latest: Pulling from vmware/docker-volume-vsphere
ad10a6bbce92: Download complete
Digest: sha256:feaa90e4234880fbf1669015e88cd485b2077270219ff84a730b35207bb622d4
Status: Downloaded newer image for vmware/docker-volume-vsphere:latest
Installed plugin vmware/docker-volume-vsphere:latest

Did docker volume ls but do not see all the volumes - volume that was attached is no longer shown

root@photon-machine [ ~ ]# docker plugin ls
ID                  NAME                DESCRIPTION                           ENABLED
db4670541e9d        vsphere:latest      VMWare vSphere Docker Volume plugin   true
root@photon-machine [ ~ ]# docker volume ls
DRIVER              VOLUME NAME
vsphere:latest      TestVol15_XX@local2
root@photon-machine [ ~ ]#

Restarted the docker and then I could see both the volumes

root@photon-machine [ ~ ]# systemctl restart docker
root@photon-machine [ ~ ]# docker volume ls
DRIVER              VOLUME NAME
vsphere:latest      TestVol15@local2
vsphere:latest      TestVol15_XX@local2

Logs:
docker-volume-vsphere.txt
vmdk_ops.log.1.txt
vmdk_ops.txt

@ashahi1 what version of Docker is this? Because docker doesn't even allow disabling a plugin (busy error is reported) if volumes are available (in use or not) that are associated with that plugin. Doubt even rm is feasible when a volume managed by the plugin is in use. Flip side is even if all volumes are removed DOcker will still disallow disabling a plugin and needs to be restarted. Its a bug on the docker side.

Because docker doesn't even allow disabling a plugin (busy error is reported) if volumes are available (in use or not) that are associated with that plugin.

Usage of --force flag allows us to remove the plugin.

As long as the plugin alias (in docker plugin install) is the same there is no issue - plugin driver upgrade scenario. Issue happens only if the alias changes - another plugin driver is registered with a different alias and tries to use the same socket. Checking how this is handled on the docker/moby side. Will update on that shortly.

Tested with 17.07 and don't see any problem. Created a volume with one version of the driver and with a certain alias. Then ran a container with the volume. Then removed the plugin and installed a newer version with a different alias. Then stopped the container (started earlier) and created and new volume and docker is able to show both volumes.
Will verify with 17.03 again.

@ashahi1 am not able to repro this issue at all, this is what I did:
docker plugin install --grant-all-permissions --alias vsphere15 vmware/docker-volume-vsphere:0.15
0.15: Pulling from vmware/docker-volume-vsphere
967f49dfadaf: Download complete
Digest: sha256:64a50830c2fa9befbede6efca3f28d2e2d8b4e595703f1b5984470579656e935
Status: Downloaded newer image for vmware/docker-volume-vsphere:0.15
Installed plugin vmware/docker-volume-vsphere:0.15
docker volume create -d vsphere15 upvol1
upvol1
docker run -it --rm -v upvol1:/mnt busybox
Unable to find image 'busybox:latest' locally
latest: Pulling from library/busybox
9e87eff13613: Pull complete
Digest: sha256:2605a2c4875ce5eb27a9f7403263190cd1af31e48a2044d400320548356251c4
Status: Downloaded newer image for busybox:latest
/ #
docker plugin ls
ID                  NAME                DESCRIPTION                           ENABLED
b401d9c62f12        vsphere15:latest    VMWare vSphere Docker Volume plugin   true
docker plugin rm -f b401d9c62f12
b401d9c62f12
docker plugin install --grant-all-permissions --alias vsphere vmware/docker-volume-vsphere:latest
latest: Pulling from vmware/docker-volume-vsphere
ad10a6bbce92: Download complete
Digest: sha256:feaa90e4234880fbf1669015e88cd485b2077270219ff84a730b35207bb622d4
Status: Downloaded newer image for vmware/docker-volume-vsphere:latest
Installed plugin vmware/docker-volume-vsphere:latest
docker volume inspect upvol1
[]
Error: No such volume: upvol1 <---------- this is expected as the volume is matched against a driver named vsphere15. New driver won't be matched for an existing volume.
docker volume create -d vsphere upvol2
upvol2
docker volume ls
DRIVER              VOLUME NAME
vsphere:latest      upvol1@sharedVmfs-0
vsphere:latest      upvol2@sharedVmfs-0
Pls. let me know if this sequence is what you are seeing and if the steps are matching.

@govint This issue is reproducible. I deployed a fresh setup and I cans till see this issue. I can also share the setup details with you if that helps to debug the issue.

@govint I deployed a new setup and not able to see this issue hence closing it.


Parsing error in formatter

Test case
class test;
  function new(string name = "test");
    L2UCMTLBxLA_reg.get_field_by_name("LA").set(i_hdas_biu_config.i_l2uncache_config[0].base_addr[31:12]);
  endfunction
endclass

Include any options or configuration used.
Actual output
test: Error lex/parsing-ing formatted output.  Please file a bug.
First error: token: "endfunction" at 4:5-15:; problematic formatter output is
class test;
    function new(string name = "test");
        L2UCMTLBxLA_reg.get_field_by_name("LA").set(
    endfunction
endclass

Note that, replacing the identifiers with different names automagically makes it work. So I have no clue what is happening here.

Identifier name change confirmed: if the underscores are removed, the formatting works.
Maybe some issue in the lexer in certain circumstances not accepting underscores in identifiers.
class test;
  function new(string name = "test");
    L2UCMTLBxLA_reg.get_field_by_name("LA").set(ihdasbiuconfig.il2uncacheconfig[0].baseaddr[31:12]);
  endfunction
endclass

Printing the raw tokens however shows that tokens are properly lexed as full tokens, e.g. (#SymbolIdentifier @98-115: "i_hdas_biu_config")
So something else is going on.
 Maybe some issue in the lexer in certain circumstances not accepting underscores in identifiers.

The important change seems to be line length (under/over limit), not an identifier itself.
ok:
class test;
  function new(string name = "test");
    abcdefghijklmno.get_field_by_name("LA").set(Abcdefghijk.i_l2uncache_config[0].base_addr[31:12]);
  endfunction
endclass

failure:
class test;
  function new(string name = "test");
    abcdefghijklmno.get_field_by_name("LA").set(Abcdefghijkl.i_l2uncache_config[0].base_addr[31:12]);
  endfunction
endclass

Logs for the failing case:
[token_partition_tree.cc:786] ReshapeFittingSubpartitions, before:
{ (>>>>[abcdefghijklmno . get_field_by_name ( "LA" ) . set ( Abcdefghijkl . i_l2uncache_config [ 0 ] . base_addr [ 31 : 12 ] ) ;], policy: append-fitting-sub-partitions, (origin: "abcdefghijk...dr[31:12]);"))
  { (>>>>[abcdefghijklmno . get_field_by_name (], policy: fit-else-expand, (origin: "abcdefghijklmno")) }
  { (>>>>>>>>["LA"], policy: fit-else-expand, (origin: ""LA"")) }
  { (>>>>[) . set (], policy: uninitialized, (origin: ")")) }
  { (>>>>>>>>[Abcdefghijkl . i_l2uncache_config [ 0 ] . base_addr [ 31 : 12 ] ) ;], policy: fit-else-expand, (origin: "Abcdefghijk...addr[31:12]")) }
}

(... line wraps search ...)

[token_partition_tree.cc:889] ReshapeFittingSubpartitions, after:
{ (>>>>[abcdefghijklmno . get_field_by_name ( "LA" ) . set ( Abcdefghijkl . i_l2uncache_config [ 0 ] . base_addr [ 31 : 12 ] ) ;], policy: append-fitting-sub-partitions, (origin: "abcdefghijk...dr[31:12]);"))
  { (>>>>[abcdefghijklmno . get_field_by_name ( "LA" ) . set (], policy: fit-else-expand, (origin: "abcdefghijklmno"))
    { (>>>>[abcdefghijklmno . get_field_by_name (], policy: fit-else-expand, (origin: "abcdefghijklmno")) }
    { (>>>>["LA"], policy: fit-else-expand, (origin: ""LA"")) }
    { (>>>>[) . set (], policy: uninitialized, (origin: ")")) }
  }
}

The wrapped line is lost somewhere in ReshapeFittingSubpartitions.

Can I take the AR to resolve this?
How can I get started and where should I look?

@saw235 I assume you know how to build Verible and run it with logging. If not, see Formatter docs (especially Troubleshooting) and the README.
Comment above ReshapeFittingSubpartitions explains briefly what does the function expect.
By looking closer at the log pasted in my previous comment, we can see the partition tree structure the function gets is wrong. It should get partitions with one call, but gets partitions with two calls joined together:
[token_partition_tree.cc:786] ReshapeFittingSubpartitions, before:
{ (>>>>[abcdefghijklmno . get_field_by_name ( "LA" ) . set ( Abcdefghijkl . i_l2uncache_config [ 0 ] . base_addr [ 31 : 12 ] ) ;], policy: append-fitting-sub-partitions, (origin: "abcdefghijk...dr[31:12]);"))
  { (>>>>[abcdefghijklmno . get_field_by_name (], policy: fit-else-expand, (origin: "abcdefghijklmno")) }
  { (>>>>>>>>["LA"], policy: fit-else-expand, (origin: ""LA"")) }
  { (>>>>[) . set (], policy: uninitialized, (origin: ")")) }
  { (>>>>>>>>[Abcdefghijkl . i_l2uncache_config [ 0 ] . base_addr [ 31 : 12 ] ) ;], policy: fit-else-expand, (origin: "Abcdefghijk...addr[31:12]")) }
}

To make it work with ReshapeTokenPartitions there should be two partition trees, one for abcdefghijklmno.get_field_by_name("LA"). and one for set(Abcdefghijk.i_l2uncache_config[0].base_addr[31:12]);.
Since ReshapeFittingSubpartitions is most probably not the culprint, we should look in places where partitions are created and reshaped:

TreeUnwrapper::SetIndentationsAndCreatePartitions
TreeUnwrapper::ReshapeTokenPartitions

Which cases apply to the code from this issue can be found in logs. Look for those two function names and you'll find what nodes they process and what partitions are reshaped. There are partition dumps similar to the one pasted above, so you should be able to find when things gets weird.
After finding out the place, you could try changing partition policy to PartitionPolicyEnum::kFitOnLineElseExpand from kAppendFittingSubPartitions (this is the one handled by ReshapeFittingSubpartitions). I recommend not to try too hard to fix this with kAppendFittingSubPartitions. There probably would be issues with nicely joining its results together.
Feel free to ask here if you have more questions.
Warning: the formatter runs twice internally to verify its results. This results in two almost identical chunks of lines in log output. The second run starts in the middle of the whole log output, so its pretty easy to remove or ignore during analysis.


🐞 [Bug]: too low balance error look too bad in connect your wallet

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
When trying to connect to a wallet with a low balance, I got this error.
It was too long and couldn't access the email or the password input cause the error covered both of them.
Steps To Reproduce

Navigate to the dashboard.
Connect your wallet (a wallet with a low balance).
Logout.
Open connect your wallet dialogue.

which network/s did you face the problem on?
Test
version
5.4.0
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

Relevant log output
.

Tried reproducing the same error with a balance as low as 0.029TFT but didn't get an error. What was the balance at the time of this error?
https://www.loom.com/share/e352021790784feb9430de781def8fc6?sid=ed2be9ec-255d-4da9-9369-68a43c2ce690

verified on devnet
version : 9d671ca
https://github.com/threefoldtech/tfgrid-sdk-ts/assets/110984055/34608729-f021-4bda-8d38-10e7a11d3190
https://app.testlodge.com/a/26076/projects/40893/suites/234374?expand_section=442162#case_4295011


Gradle 8.5: Resolving old artifacts of the project

Expected Behavior
I want to create a configuration that resolves the old artifact of the project.
Current Behavior (optional)
When resolving the configuration, Gradle 8 silently adds the current version artifact of the project into the artifact candidate list. After that, it resolves the latest one and rejects the old one.
For example, the current project version is 2.0.0 and I create a configuration to resolve the nearest old artifacts with range version.
project.configurations.findByName("oldArtifacts") ?: project.configurations.create("oldArtifacts") {
			canBeConsumed = false
			transitive = true
			visible = false
			dependencies.add(
					project.dependencies.create("$project.group:$project.name") {
						version {
							strictly "[0.0.0,2.0.0)"
						}
					}
			)
		}

Even the range excludes the version 2.0.0, Gradle 8 always returns the artifact 2.0.0.
After debugging, I found that these line of code that causes the problem.
        // If the module matches, add the root component into the mix
        if (moduleId.equals(rootModuleId) && !candidates.contains(rootComponent)) {
            candidates = new ArrayList<>(candidates);
            candidates.add(rootComponent);
        }

https://github.com/gradle/gradle/blob/master/platforms/software/dependency-management/src/main/java/org/gradle/api/internal/artifacts/ivyservice/resolveengine/graph/selectors/SelectorStateResolver.java#L71-L74
With Gradle 7 I could cheat it by changing the project group or project name when resolving the configuration but this data is cached with Gradle 8 now.
Context
We need to fetch the old artifacts for revapi to work.
It needs the old artifacts and the new artifacts for comparing and determining the breaking change.
Besides, we also need the old artifacts for analyzing purposes.

Thank you for your interest in Gradle!
This issue will be closed as a duplicate of

#1629

Please add your use case and 👍 to that issue.


1.13 compatibility.

Hey,
Is OldCombatMechanics compatible with 1.13 ? 🗡
Thanks !

Try :smile:
If you have some bugs, report it here...

I tried OCM in a 1.13 server and it seemed at least at first glance to be fully functional. I'm sure there will be some minor bugs we can fix later on, but otherwise it appears to be fully functional.
Plugins, unlike mods, utilize an API that is designed to have as few if any breaking changes per Minecraft update. This means that plugins often work immediately in a new version of Minecraft. 1.13 has by far had the most breaking changes of probably any Minecraft update, as they changed the way blocks' data are handled, and removed integer ID support. However, neither of those things affects a plugin like OCM.

We'll have to change the INK_SACK material constant to INK_SAC though.
Then we will have to move away from the DamageModifier system some day, as md5 is keen on dropping that for over a year now.
Apart from that I think nothing really needs adjustments, besides maybe updating some dependencies (NBT, the packet listener). Both of these are quite compatible with minecraft changes though, so I don't expect that to need much tinkering.

But it could be a problem if we are to keep compatibility with both 1.13+ and 1.9-1.12

The INK sack will need to be hidden behind some small abstraction method (unless you have a nicer idea. Maybe some automatic mapping via an internal config file? Sounds a bit more future proof as the names seem to be kinda fluctuating anyways).
I've seen your posts in the thread about the DamageModifier — are you aware of any updates?
If the dependencies e.g. for NBT need an update, that would be a more interesting challenge, yea :/

The DamageModifier API just seems to be in a permanent state of deprecated, without any real alternative in the Bukkit API. Thus I have simply left it in the code and we should only worry about when it is actually removed, if it ever will be...


Using in Objective-C project.

When using DisplaySwitcher in an Objective-C project the init method on DisplaySwitchLayout is public init(staticCellHeight: CGFloat, nextLayoutStaticCellHeight: CGFloat, layoutState: LayoutState) is not available to Obj-C code. The only init method is initWithCoder.
Using Xcode version 8.0 (8A218a) and macOS 10.12. iOS deployment target 10.0.

Hi @lateAtNight! Thanks for your interest. I've just added compatibility. Please update to 1.0.1


Gælder ikke d365 / BC

Man kan ikke på d365 / BC søge Interaktioner. Den henviser alene til dokumentationen her, men ingen menupunkter.
Mvh Maria

Dokumentoplysninger
⚠ Undlad at redigere denne sektion. Det er påkrævet til docs.microsoft.com ➟ GitHub-problemlinkning.

ID: b44b0359-ebac-60f6-dcb6-628180a15c9a
Version Independent ID: 2fe7b562-278e-7f43-9f68-cfa9e6ba3ce4
Content: Oprette automatisk registrering af interaktioner - Business Central
Content Source: business-central/marketing-auto-record-interactions.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: jswymer

Hej @MariaFrederiksen,
My apologies for getting back to you belatedly. I'm writing in English so that others will also benefit from this information.
Interactions are available in Dynamics 365 Business Central and have been for a long while. If you still cannot find the capabilities, then I suspect that you might not have permission to view those pages. You might want to talk to your administrator about making sure you have the right permissions.
Also, you can get an quick overview of the capabilities that you have access to by using the role explorer. For more information, see https://docs.microsoft.com/en-us/dynamics365/business-central/ui-role-explorer (på dansk: https://docs.microsoft.com/da-dk/dynamics365/business-central/ui-role-explorer)


added release notes

@WardF not sure where to put these, but probably we want them in the release notes for the next release.

merging into 4.6.2.


combination of unary and power operator leads to misleading formatting.

Operating system: Ubuntu 18.04.1 LTS
Python version: Python 3.6.7
Black version: 18.9b0
Does also happen on master: as of 2018-12-20T09:32, yes.
(as per my tweet: https://twitter.com/llanga/status/1075484480436080640 )
The code
-2**8

blacks to
-2 ** 8

I think this is misleading because the grammar reduction rules mean that the code actually parses as -(2**8).
The result of this code, however you format it, is -256, not 256.
I'm not sure what to suggest. Perhaps not altering the space around **?
Though I can see that almost any suggestion will be controversial.

Maybe we should insert parentheses when the unary negation operator applies to more than just a literal (so -(2 ** 8)).

Indeed, assert -16 == -2 ** 4 is looks pretty weird!


Migrating from 1.1.200 to 1.1.300 Custom Endpoints

Is the only way to use custom endpoints by mounting chainlit as a FastApi sub application? The previous approach of importing app from chainlit.server works for my usecase because there was only a need for an additional endpoint, it doesnt make sense now that i have to deploy a whole FastApi application just to have 1 api and make my chainlit a sub app.

@Physium Yes, you'll have to deploy a 'whole' FastAPI application.
As you see in the docs, it'll take all of some 7 lines. https://docs.chainlit.io/integrations/fastapi#fastapi
I am confident, you can manage! ;)

Appreciate your confirmation in this. The example shows how easy it is no doubt, but migrating to this approach from an existing project built around the previous design requires significant effort. Not that it's your problem but yea I wish it was "7 lines".


arg --curl doesn't include body in the output

need to debug and fix this

Todos:

add content-type header to reqwest::Request during postman::request.to_reqwest()
move dbg_as_curl's AsCurl implementation to postman_to_curl.rs and add body as field of AsCurl struct ( would have to take postman::request as an argument in the constructor and extract body from it )
update the Display trait to add body as well to the curl

fixed in 5e708a9633146fcd977e0b601a0ca64a779ff0b0


Debug

debug = True allows for premature breaking. The command line argument --debug also works.

@chengzhuzhang Can you review this?
Try the code with the below cfg file on acme1. With debug=False, it should handle the error for 'MAM' and continue. When debug=True, you should see that the diagnostics stops at the first error. Thanks!
[messed up var]
sets = ["lat_lon"]
case_id = "CERES-EBAF-TOA-v4.0-COSP"
variables = ["LWCF"]
seasons = ["ANN", "MAM", "SON"]
contour_levels = [0, 10, 20, 30, 40, 50, 60, 70, 80]
diff_levels = [-35, -30, -25, -20, -15, -10, -5, -2, 2, 5, 10, 15, 20, 25, 30, 35]

ref_name = "ceres_ebaf_toa_v4.0"
reference_data_path = "/p/cscratch/acme/data/obs_for_acme_diags"
reference_name = "CERES-EBAF Jan 2000-Dec 2015"

test_data_path = "/export/zhang40/test_data/"
test_name = "20170926.FCT2.A_WCYCL1850S.ne30_oECv3.anvil"

backend = "mpl"
results_dir = "bad_var"
debug = False

Have it tested. the -debug parameter works. However, we decided that  the error for debugging should be more verbose, (https://github.com/ACME-Climate/acme_diags/issues/106).

@zshaheen  I tested the latest debug branch and the full error messages are printed, good for debugging.  Commits look good to me. Feel free to merge 👍  and close issue.


webhooks/flock: Add flock integration.

Flock webhook integration.

@eeshangarg I assume you'll do the first review on this.

Yeah I'll review this soon :)
On Mar 4, 2018 6:46 PM, "Tim Abbott"<EMAIL_ADDRESS>wrote:

@eeshangarg https://github.com/eeshangarg I assume you'll do the first
review on this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/8554#issuecomment-370269397, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AG6nb6TQZuybUUcLlY8JbjnX58EBsnLbks5tbGfGgaJpZM4SZcIb
.

@Rishabh570: Also, just as an aside, if you update a PR, make sure you post a comment saying "Updated" or "addressed your feedback". If you simply push changes, GitHub doesn't notify me unless you make a comment! Thanks again! :)

@eeshangarg Okay, so I've updated this :)

Sounds good, merged, thanks @Rishabh570 and @eeshangarg!


add info about getting a sim for prime lite

device IMEI
only need data, no talk/text
typical data usage on release
advantages of unlimited data

currently working on this and figured the best place to display the info would be where it notifies the user that the sim is unavailable


How do I use EndofLine or DefaultEndofLine Enums?

https://github.com/Microsoft/monaco-editor/blob/4930b6aff505b72e966fcfa54ade4e33d6abe3f8/monaco.d.ts#L1279
Really could use some help on how to use endofline and/or defaultendofLine enums in Monaco-Editor. I have the editor working but I can't get the editor to handle carriage returns without having to manually enter a ' \ ' at the end of each of line where I need a carriage return.  Also, spacing is not respected. Has anyone run into this issue? I have text (javascript) entered into editor being saved into a backend database. Any help would be greatly appreciated.

Sorry, I'm not sure what your problem is.
The editor accepts a JavaScript string as the value to display. It will then split this string into lines, looking for \r\n or \r or \n. You can read the string value of the editor via ed.getModel().getValue(). Typically, this call will return the lines, joined by the same line separator that was used when the editor was created.
Hopefully you can further explain what you're trying to do, or perhaps you can validate that everything is fine in your DB layer by using a <textarea> for presenting the text for testing purposes, and getting everything working correctly in your app by using a <textarea>. Once everything is fine in your XHR calls / DB layer / etc. you can replace the front-end <textarea> with a monaco editor.

Thanks for responding to my question. I was able to resolve my issue, see Monaco-Editor Single String From Database
|
|
|
|  |  |
|
|
|
|  |
Monaco-Editor Single String From Database
Been researching for the last couple of days and still no result. I have a back-end database that saves text ent...
|
|
|
Sorry if my question was confusing, I just wasn't entirely able to explain the odd behavior. Anyways, once I was able to replace the new line breaks into an acceptable format, the "div" container was able to properly display the returned string.
On Tuesday, November 7, 2017, 8:31:26 AM PST, Alexandru Dima<EMAIL_ADDRESS>wrote:

Sorry, I'm not sure what your problem is.
The editor accepts a JavaScript string as the value to display. It will then split this string into lines, looking for \r\n or \r or \n. You can read the string value of the editor via ed.getModel().getValue(). Typically, this call will return the lines, joined by the same line separator that was used when the editor was created.
Hopefully you can further explain what you're trying to do, or perhaps you can validate that everything is fine in your DB layer by using a  for presenting the text for testing purposes, and getting everything working correctly in your app by using a . Once everything is fine in your XHR calls / DB layer / etc. you can replace the front-end  with a monaco editor.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Update CHANGELOG for v1.4.3

Motivation
Prepares release for v1.4.3
Description
Updates changelog and prepares v1.4.3 release
Test Plan
n/a
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed
[x] Update changelog as needed

bors merge


fix(tests/NoCredentialsError): set aws creds in settings

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[ ] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Set aws credentials in test.py to avoid moto crashing
How did you test this code?

tested by removing my ~/.aws creds

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

Comparison is base (068253e) 96.08% compared to head (95666bf) 96.08%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3131   +/-   ##
=======================================
  Coverage   96.08%   96.08%
=======================================
  Files        1054     1054
  Lines       31702    31708    +6
=======================================
+ Hits        30462    30468    +6
  Misses       1240     1240

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


LibWeb/Geometry: Implement "other than none" keyword check in DOMMatrix init

This implementation leads to passing more WPT tests. Relevant test imported in this PR.

Hi, thanks for working on this! The change itself looks OK to me, but we try to avoid adding commits that modify work done in previous commits. All the changes made in this PR should be squashed into the first commit, then I think this should be good to go. Feel free to ask on Discord if you need help with this.

Hi, thanks for working on this! The change itself looks OK to me, but we try to avoid adding commits that modify work done in previous commits. All the changes made in this PR should be squashed into the first commit, then I think this should be good to go. Feel free to ask on Discord if you need help with this.

Thanks for the review. I have squashed the commits down to one as requested.

 You should write a proper commit message instead.

 You should write a proper commit message instead.

Commit message has been updated to just the proper original message.


feature: add button to hide single jobs

Hi there,
we're using gitactionboard in our team (in the otto payments space) as well and have the issue that we just have some pipelines that are supposed to stay red for now. To not let those distract you all the time, it would be great to have a way to hide jobs that you are not interested in.
This PR is adding this, implemented via the preferences-store in the frontend, so that every user can decide for themselves which jobs to hide and which to show.
Is this a feature you would be happy to add, by merging this?
I'm also very open to feedback on this, if you think it would make sense, but should be done differently.

Hi @sweiler, can you please add couple of screenshots

Sure, here you go:

Hi @sweiler, Thanks for the screenshots. I have a small questions, what will happen if there is no hidden jobs? Will it still display the 2nd section like 0 hidden jobs?

No, if you have not hidden any jobs, the section including the horizontal line above it will not be shown.

So @sumanmaity1234 , what do you think would it make sense to include this feature like this?

Hi @sweiler, this is an interesting feature and I think it'll be a nice addition to current dashboard.
I'll merge this PR this week once I get some time. Meanwhile I'll review the code and if required I'll comment. I hope that'll be fine with you.

Hi @sumanmaity1234, thanks a lot for your review. I've adapted most of the code accordingly and added two comments to your remarks. Please have a look whenever you are free. If you'd like we could also have a quick chat later today via Otto Teams.

Hi @sumanmaity1234 , great to see you approved the PR :)
Is there anything I need to do before it is able to be merged and released?

Hi @sweiler, I checked the code, I tried to run in local, there is a small issue (I believe) in terms of page refresh.
It takes sometime once I click on the show/hide button, it takes time to hide/show it.
I'll ping you on teams so that we can connect and discuss about it

@sumanmaity1234 Please have a look at my recent changes. Now the responsibility of hiding this is in the generic Dashboard component, but just disabled for the secrets part. Its now working instantaneously, and gets only synced to the preferences in the background.
I'm not sure if it makes sense this way, as now the Dashboard has this hiding feature, although only one of the use-cases needs it. There is a property 'hide-by-key' which is used to enable the feature and specify which key of the underlying content object is used for hiding.

To the "hide-feature in generic Dashboard component" thing: I see only two options here:

As it is now: Dashboard offers generic feature to hide items, but this is only used by one of the two Dashboards
Do not have a generic Dashboard component
Maybe you see another way of extracting the hide logic without having two completely separate dashboard implementations.

To the question of making this optional. I think the question here is does those hide buttons annoy people that do not want to use them. If so, can we maybe style them in a less distracting way? Because I think the feature itself is already optional by design. If you don't click any of those buttons, nothing will change, right?

Great :) Looking forward to show this to the team soon :D

@sumanmaity1234 Fixed the hide button formatting issue now, please have a look :)

@sweiler Changes looks good to me. Will merge this PR this week


Rendering of scene "" did not complete successfully, caught exception: myRenderJob0[FileStream] Error while trying to open file "./projects_render/222/000000_mlt.rgbe": No such file or directory

Hi, yindaz,  thanks for this awesome repo, I'm using it in my project.
I got some error when I run the example demo. My script is as follow. I download the camera info and unzip to projects_camera folder before that.
"python mitsuba_render.py -s ./rendering/projects_serialize/0004dd3cb11e50530676f77b55262d38/main_template_color.xml -c ./rendering/projects_camera/0004dd3cb11e50530676f77b55262d38/room_camera.txt -o ./projects_render/222/ -g ./rendering/projects_camera/0004dd3cb11e50530676f77b55262d38/room_camera_good.txt"
2018-10-17 15:50:55 INFO  main [PluginManager] Loading plugin "plugins/envmap.so" ..
2018-10-17 15:50:55 INFO  main [EnvironmentMap] Loading environment map "HDR_111_Parking_Lot_2_Ref.hdr"
2018-10-17 15:50:55 INFO  main [MIPMap] Mapped MIP map cache file "/home/hzch/usr/PBRS/pbrs/./util_data/HDR_111_Parking_Lot_2_Ref.mip" into memory (100.0 MiB).
2018-10-17 15:50:55 INFO  main [EnvironmentMap] Precomputing data structures for environment map sampling (50.0 MiB)
2018-10-17 15:50:56 INFO  main [EnvironmentMap] Done (took 625 ms)
2018-10-17 15:50:56 INFO  main [PluginManager] Loading plugin "plugins/independent.so" ..
2018-10-17 15:50:56 INFO  main [PluginManager] Loading plugin "plugins/hdrfilm.so" ..
2018-10-17 15:50:56 WARN  main [HDRFilm] The RGBE format only supports componentFormat="float32". Overriding..
2018-10-17 15:50:56 INFO  main [PluginManager] Loading plugin "plugins/gaussian.so" ..
2018-10-17 15:50:56 INFO  main [PluginManager] Loading plugin "plugins/perspective.so" ..
2018-10-17 15:50:56 INFO  main [KDTreeBase] Constructing a SAH kd-tree (244419 primitives) ..
2018-10-17 15:50:56 INFO  main [KDTreeBase] Finished -- took 706 ms.
2018-10-17 15:50:56 INFO  main [PluginManager] Loading plugin "plugins/sphere.so" ..
2018-10-17 15:50:56 INFO  myRenderJob0[MLT] Starting render job (640x480, 4 cores, SSE2 enabled, approx. 1024 mutations/pixel) ..
2018-10-17 15:50:56 INFO  myRenderJob0[PluginManager] Loading plugin "plugins/direct.so" ..
2018-10-17 15:50:56 INFO  myRenderJob0[PluginManager] Loading plugin "plugins/ldsampler.so" ..
2018-10-17 15:50:56 INFO  myRenderJob0[SamplingIntegrator] Starting render job (640x480, 8 samples, 4 cores, SSE2 enabled) ..
Rendering: [++++++++++++++++++++++++++++++++++++++++++++] (47.5s, ETA: 0.0s)
2018-10-17 15:51:44 INFO  myRenderJob0[PathSampler] Integrating luminance values over the image plane (100000 samples)..
2018-10-17 15:51:50 INFO  myRenderJob0[PathSampler] Done -- average luminance value = 0.187713, stddev = 1.948820 (took 6586 ms)
Rendering: [++++++++++++++++++++++++++++++++++++++++++++] (2.6m, ETA: 0.0s)
2018-10-17 15:54:26 INFO  myRenderJob0[RenderJob] Render time: 3.4898m
2018-10-17 15:54:26 INFO  myRenderJob0[HDRFilm] Writing image to "./projects_render/222/000000_mlt.rgbe" ..
2018-10-17 15:54:26 WARN  myRenderJob0[RenderJob] Rendering of scene "" did not complete successfully, caught exception: myRenderJob0[FileStream] Error while trying to open file "./projects_render/222/000000_mlt.rgbe": No such file or directory
2018-10-17 15:54:26 INFO  myRenderJob1[MLT] Starting render job (640x480, 4 cores, SSE2 enabled, approx. 1024 mutations/pixel) ..
2018-10-17 15:54:26 INFO  myRenderJob1[SamplingIntegrator] Starting render job (640x480, 8 samples, 4 cores, SSE2 enabled) ..
Rendering: [++++++++++++++++++++++++++++++++++++++++++++] (52.3s, ETA: 0.0s)
2018-10-17 15:55:18 INFO  myRenderJob1[PathSampler] Integrating luminance values over the image plane (100000 samples)..
2018-10-17 15:55:27 INFO  myRenderJob1[PathSampler] Done -- average luminance value = 0.015386, stddev = 0.233618 (took 8452 ms)
Rendering: [++++++++++++++++++++++++++++++++++++++++++++] (4.0m, ETA: 0.0s)
2018-10-17 15:59:24 INFO  myRenderJob1[RenderJob] Render time: 4.9630m
2018-10-17 15:59:24 INFO  myRenderJob1[HDRFilm] Writing image to "./projects_render/222/000001_mlt.rgbe" ..
2018-10-17 15:59:24 WARN  myRenderJob1[RenderJob] Rendering of scene "" did not complete successfully, caught exception: myRenderJob1[FileStream] Error while trying to open file "./projects_render/222/000001_mlt.rgbe": No such file or directory
2018-10-17 15:59:24 INFO  myRenderJob2[MLT] Starting render job (640x480, 4 cores, SSE2 enabled, approx. 1024 mutations/pixel) ..
2018-10-17 15:59:24 INFO  myRenderJob2[SamplingIntegrator] Starting render job (640x480, 8 samples, 4 cores, SSE2 enabled) ..
Rendering: [++++++++++++++++++++++++++++++++++++++++++++] (52.4s, ETA: 0.0s)
2018-10-17 16:00:16 INFO  myRenderJob2[PathSampler] Integrating luminance values over the image plane (100000 samples)..
2018-10-17 16:00:23 INFO  myRenderJob2[PathSampler] Done -- average luminance value = 0.357498, stddev = 3.087691 (took 7423 ms)

Do you have writing permission, folder path, or sufficient space to save the output? It seems that the output file cannot be written to the disk.

I run the code in root, but still the same error.

I can confirm that the problem is caused by failed file saving. Please make sure the target folder exist also.


Markdown issues: multiple "definitions"; lists with sub-paragraphs

Typemill looks brilliant! I have just begun to "play" with it, and using my own Markdown test file, have noticed a couple of problems:
(1) Definition Lists
Definition lists allow for multiple "definitions",

 elements for a single term,

. Typemill does not recognize this situation, but runs the second definition into the first. Here's some test text:
Apple
: Pomaceous fruit of plants of the genus Malus in the family Rosaceae.
: A brand of computers and high-end tech equipment.

Orange
: The fruit of an evergreen tree of the genus Citrus.
: Valve-driven amplifiers for electric guitar.

In the Babelmark 2 "dingus" (if that's the right term), here are the results:

= Definition list: pandoc 2.9.1; RDiscount 2.1.8; PHP Markdown Extra 1.9.0; MultiMarkdown 5.1.0; MultiMarkdown 6 6.3.0; kramdown 1.2.0
render "acceptably": Blackfriday; showdown (flavor: github) 1.7.4
not recognized: the rest

So most of the implementations do something acceptable with it, and quite a few correctly parse the
...
 list.

(2) Sub-paragraphs in lists
Most Markdown implementations I'm familiar with allow for list items (whether ul or ol) to contain multiple paragraphs within a single
...
 entry. "Continuation" text is usually signalled by leading spaces, like Example 234 in the GFM spec. The results for the "test text", below, show that almost all the renderers it includes handle this in the expected way:
1. Fog everywhere. Fog up the river, where it flows among green aits and meadows; fog down the river, where it rolls defiled among the tiers of shipping and the waterside pollutions of a great (and dirty) city.

  Fog on the Essex marshes, fog on the Kentish heights. Fog lying out on the yards and hovering in the rigging of great ships; fog drooping on the gunwales of barges and small boats.

1. Gas looming through the fog in divers places in the streets, much as the sun may, from the spongey fields, be seen to loom by husbandman and ploughboy. Most of the shops lighted two hours before their time—as the gas seems to know, for it has a haggard and unwilling look.

1. The raw afternoon is rawest, and the dense fog is densest, and the muddy streets are muddiest near that leaden-headed old obstruction, Temple Bar.

  And hard by Temple Bar, in Lincoln’s Inn Hall, at the very heart of the fog, sits the Lord High Chancellor in his High Court of Chancery.

So when rendered, list items 1 and 3 should have two paragraphs, but item 2 only one.

FAILS: Python-Markdown 3.0.1; lunamark 0.4.0; RedCarpet 3.4.0; MultiMarkdown 6 6.3.0; MultiMarkdown 5.1.0
SUCCEEDS: the rest!

What happens in Typemill (I think) is that the "continuation" text is flagged as being illegal code entry, probably because of the leading spaces. This is (IMO!) most unfortunate! This is a valuable aspect of markdown for full-form documentation of the kind that Typemill seems designed to produce.
Anyway! I hope these two cases are clear, although I don't know whether these two are "bugs" or "features" in Typemill. ;)

Hello David,
thank you very much for your tests and feedbacks! I just tested your cases with parsedownExtra and the library parses your tests correctly. As expected, this is a bug in my extension code somewhere. I will try to fix it asap, but I am not sure when I find the time (working very intensively on next typemill-versions right now).

Thanks for the reply. Typemill looks like a wonderful writing platform/CMS, and I'm excited to explore more. Hope you enjoy successful developing!


Correctly handle module version

xstate - this module is painful because its version has @ (at) and / (slash), which breaks literally everything:

deno.land website
deno.land proxy
cdn.deno.land
doc.deno.land

Yet this module is the 5th one on deno.land/x, which is why I spotted this issue.

This PR fixes the website
I will submit a PR to doc.deno.land
I don't know how cdn.deno.land exactly works, but getVersionDeps fails on both escaped URL (https://cdn.deno.land/xstate/versions/xstate%404.16.2/meta/deps_v2.json) and unescaped<EMAIL_ADDRESS>Unfortunately for the proxy, I found oak decodes the URL before passing it to path-to-regexp, so the router can't match the correct version even if we escaped it. I will submit a PR to them as well.

P.S. dealing with all these paths is a nightmare, I think a helper function for creating these URL is needed or there will be more issues in the future.
Close #1718

@kidonng Nice refactoring and fixes!
But can you add test cases of getRepositoryURL (with 3rd arg), parseNameVersion, getBasePath in util/registry_utils.test.ts?

Unfortunately for the proxy, I found oak decodes the URL before passing it to path-to-regexp, so the router can't match the correct version even if we escaped it. I will submit a PR to them as well.

Can you describe a bit more about what happen for which path?

FYI, there's this related issue on the registry https://github.com/denoland/deno_registry2/issues/216

Sorry for not responding for a long time!

@kidonng Nice refactoring and fixes!
But can you add test cases of getRepositoryURL (with 3rd arg), parseNameVersion, getBasePath in util/registry_utils.test.ts?

I will try to find time to add these in the future 😀

Unfortunately for the proxy, I found oak decodes the URL before passing it to path-to-regexp, so the router can't match the correct version even if we escaped it. I will submit a PR to them as well.

Can you describe a bit more about what happen for which path?

The issue is that / is always treated as a path separator, and can't be matched. Will link create a issue there and link it to here, again, if I find time.


fix: Init can use relative imports

Using a relative import solves a discrepancy between Python 2.7.11 and
3.5.1. For some reason, 2.7.11 brings all the module objects down to the
expected graphitesend.* level using an absolute import, while 3.5.1
leaves the objects at the graphitesend.graphitesend.* level. The
relative import will bring everything down to graphitesend.* for
2.7.11 and 3.5.1.

Coverage remained the same at 94.318% when pulling 9dd9241aad673d16474f6a4d18c9a05010bbf42e on e4r7hbug:fix/import into 91cf58beb45a877322b2fe9453258a885ea4f043 on daniellawrence:master.

Coverage remained the same at 94.318% when pulling a96d93c11cb517fd013a17b7e33e51c24b8b03b1 on e4r7hbug:fix/import into 90dbdc85558b66fd2b5019c4d6ea0b5977a5f17f on daniellawrence:master.

Coverage remained the same at 94.318% when pulling ee0041ba8edc8de8d38158e6b8ed49ccbfc4323b on e4r7hbug:fix/import into 90dbdc85558b66fd2b5019c4d6ea0b5977a5f17f on daniellawrence:master.

+1 I confirm this works with python 3.
@daniellawrence Hopefully this could be merged soon.


Blog to introduce Secrets Store Sync Controller

Write a blog to introduce the new Secrets Store Sync Controller.
previous attempt: https://github.com/kubernetes/website/pull/45538
/sig auth

/triage accepted
/assign nilekhc

/language en
/area blog

/assign

/assign


Update Composer dependencies (2021-10-20-00-11)

Visual regression test passed!
View the visual regression test report


Add project openings to website

Closes #1270

@angle943 Take a look at this.
I think instead of creating a separate category we should classify them into existing categories and add a text below the heading

This position is for - ABC App

When no text is added that means the opening is for the TNB team.
What do you think?

Alright, so I should make a category called "Project Proposal Openings" ?

I added the Project Proposals category and a small part in the description which says which proposal the opening is from. I am still working on the bug which @angle943 found, but I can't figure out why that's happening. Any ideas?

@AbhayAysola
I was thinking to add an extra optional field on Opening - projectName.
If this exists you'll know it belongs to a project, so you'll put a highlighted(some styling that is easily noticeable at a glance but not too striking that it diverts the attention) text below the position title - This position is for {projectName}.
All the openings will fall under existing categories, i.e. engineering, design, etc.
I'll check your issue, tonight. At work rn.

There's something weird that happens when you click on UI/UX designer and try navigating away. It persists up at top here.

This is currently occurring as there is duplicated key: position (UI/UX designer) that is being passed to the openings and it interrupts React's reconciliation phase. We should probably change the key to pass openingId instead.
https://github.com/thenewboston-developers/Website/blob/c6226e3d988d7880e58f98e56f17ae96f9e50893/src/containers/Openings/index.tsx#L65-L76

So I added the optional projectName field on opening like @thesanjeevsharma said, and also styled it a little. I changed the key to use openingId instead of position and it fixed the bug. I didn't push that though, because I wasn't sure if it would impact other parts of the page. @jamessspanggg just wanted to make sure if passing openingId won't break anything else.

I didn't push that though, because I wasn't sure if it would impact other parts of the page. @jamessspanggg just wanted to make sure if passing openingId won't break anything else.

It shouldn't break if the openingId is unique, we prolly need a way to structure the openingId such that it will be unique every time someone adds a new opening.

It shouldn't break if the openingId is unique, we prolly need a way to structure the openingId such that it will be unique every time someone adds a new opening.

Ah okay, so we should be good for now I think. We can do something like adding the category or projectName to the beginning of the openingId to make it more unique.

@jamessspanggg @angle943 thoughts?

Would it be better if we provide the url to the corresponding project proposal or website (if any), so that people that are interested can view more details about the project straight away?
Something like:
project: {
    name: string;
    url: string;
}

@jamessspanggg @thesanjeevsharma yeah that would be great.
For the given data:
project: {
   name: string;
   url: string;
}

if the URL is provided, let's have the name of the app be the link, ie:

So I added the anchor tag, let me know what you guys think.

So I added the anchor tag, let me know what you guys think.

please resolve the merge conflicts so that I can review

@AbhayAysola @thesanjeevsharma I've removed the generic span css selectors, but other than that this is solid work. Thanks guys.
@AbhayAysola i've upped the bounty for this issue for the new acceptance criterias

@angle943 okay, cool. Here's my account number<PHONE_NUMBER>de48e4bf2ed2a9b4b839e6b2603018ecdfbf4cffdb2065e01a3ed1 if I'm still going to get paid, since a lot of the work wasn't done by me.

@AbhayAysola yep you'll get paid. Can you post your account number on the issue too? It might help the accountants

@AbhayAysola yep you'll get paid. Can you post your account number on the issue too? It might help the accountants

Sure, will do that. Thanks :)


Invalid arn during validation on new cluster

Using version 3e402be845b5362756594b6cd61d962aab9f84d6 or v0.9.9-rc.3:

Run through the quick start guide that I was working on at https://github.com/c-knowles/kube-aws/blob/5ce4fc461f06a6c95e4e407ae3059411e09ac252/docs/tutorials/quick-start.md.
During validation the following error occurs:

➜ kube-aws validate  --s3-uri=s3://MY_BUCKET_NAME
INFO: generated "credentials/tokens.csv.enc" by encrypting "credentials/tokens.csv"
Error: Failed to initialize cluster driver: failed to read/create encrypted assets: Error encrypting credentials/kubelet-tls-bootstrap-token: NotFoundException: Invalid arn
	status code: 400, request id: 9d4a79de-xxxx-xxxx-xxxx-xxxxxxxxxxxx

Merging https://github.com/kubernetes-incubator/kube-aws/pull/1034 depends on fixing this issue.

Just curious but are you sure your kmsKeyArn is correct?
I'm really unsure why such an error happens.

@mumoshu I've dug into this and found the cause of the problem. It's because when we configure the KMS SDK we take the region from the main config here but we do not perform any validation that the ARN is the same region. So a slight mismatch between my region and ARN lead to this error.
Is it mandatory to have the KMS key in the same region? Since it's an ARN I thought it worked across regions. If it's mandatory somewhere like cluster runtime, I'd like to add extra validation that the ARN contains the stackConfig.Config.Region.


Show skipped tests in gray color

Instead of showing the skipped tests as failed (not passed),
display them separately (using dimmed gray, not green or red).

@afbjorklund Hi 👋 Thank you so much for taking the time to add this update. I really like what you did here. I'll aim for a release this weekend based on your changes. Once again, thank you for your contribution. Have a wonderful weekend! Cheers! 🍻

Thanks!


python311Packages.hahomematic: 2023.10.7 -> 2023.10.8

Diff: https://github.com/danielperna84/hahomematic/compare/refs/tags/2023.10.7...2023.10.8
Changelog: https://github.com/danielperna84/hahomematic/releases/tag/2023.10.8
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 260674 run on x86_64-linux 1

2 packages built:

python311Packages.hahomematic
python311Packages.hahomematic.dist


Added header requests + other success codes

Some APIs return information in the headers.
Server response codes are 200-299 success.

Rejected, but features implemented!
Headers are added to the response result and 200-299 is considered success.


ArchWSL: don't use registry hacks

Change for prevent dangerous registry hacks in ArchWSL script.
lxuid is buggy and should not be used.

@kuinaein @rase any thought on this?

I commented some changes on the original PR.
https://github.com/lukesampson/scoop-extras/pull/1179#issuecomment-416246332
Maybe the installation can be simplified.

Installing with administrator privileges can break the permissions.
I think it is safer to install in user mode.

If installed in a directory to which you do not have full access permissions, you may not be able to start with user permissions.

@r15ch13 @yuk7
Works great for me. Thanks a lot!
The hack is only for data persistence ( https://github.com/lukesampson/scoop-extras/pull/1179#issuecomment-416269681 ) and Arch Linux filesystem is still persisted.

@r15ch13 @kuinaein @Ash258
Sorry for too late
I have pushed the requested changes.
Could you review it?
This change is also required for WSL2 support.

ok.

Did you test it?

Yes. tested

Should I resolve conflicts by following the latest master?

Someone please test this. I am not gonna mess my environments with this.

I've tested this version many times in the past few days.

I used to install ArchWSL by applying this patch to the archwsl.json file.

This commit breaks ArchWSL update to <IP_ADDRESS> for me ...


Double rendering?

For some reason whenever I render a layout into a template, it just gets rendered below all other content on the page. I've posted this on the Meteor forums as well but here is the code:

    ...
    {{> layout}}

<template name="layout">

                        {{> avatar size="large" shape="circle"}}

                {{>Template.dynamic template=content}}

</template>

<template name="usercard">
    ...
</template>

<template name="home">

        {{#each playerslooking}}
            {{> usercard}}
        {{/each}}

</template>

<template name="find_page">

hello

</template>

JS:
FlowRouter.route('/', {
  action: function() {
    BlazeLayout.render("layout", {content: "home"});
  }
});

FlowRouter.route('/find/:_id', {
  name: 'postfind.show',
  action: function() {
    BlazeLayout.render('layout', {content: "find_page"});
  }
});

What happens is that it seems to double render my .full-row  for some reason. This puts all content below the page.
I'm guessing this is user error but im not sure what im doing wrong.
Paint skills:
What Im going for:
http://prntscr.com/br8xh6
whats happening:
http://prntscr.com/br8z8p
Related: http://stackoverflow.com/questions/34299849/blazelayout-rendering-content-to-a-dynamic-template-but-the-content-is-appearin

So I was just going through some more tutorials trying to debug, and ran the followinng:
<template name="mainLayout">

    {{>Template.dynamic template=content}}

</template>

I did not include the mainLayout template anywhere, yet it still showed up. How? If the dynamic templates just popup by themselves, how are you supposed to structure your layout?

Hey, didi you figure it out?? having the same issue with a navbar.


Fix to allow bolding of text that coincides with the end of body.

textToBold that is a suffix of text would be rejected by the end>=text.length test.
end > text.length() is the correct indication that text.indexof gave us a bogus offset.

@ElectricMagic thanks for the PR!  Merged into the 6.4.0 branch


h2d.HtmlText improvements

HtmlText is missing maxWidth
adding <img/> tag might be nice too (with a dynamic function loadImage(url:String):h2d.Tile

Both are now implemented.


Help option

The game should include a basic help entry when running the game without any options or with option like --help.

Sorry, had other issues.


DBZ-4968 Remove server name from source partition

Change summary:

Remove SERVER_PARTITION_KEY from all Partition implementations.
Remove connectorConfig  from the Partition.Provider constructors that used it only to source the server name.
Update Partition unit tests. With the server name removed, for most of the Partition implementations, there are no other properties that would make different Partition instances non-equal.
Remove DBZ-3347 tests for Oracle and SQL Server (single-partition mode) since the record source partition in these cases no longer contains any properties. The SQL Server connector source partition in multi-partition mode still contains the database name, so in this case, the test is still valid.

Backward compatibility considerations
This is a breaking API change.

The connector won't be able to read the offsets produced by its older version. The Kafka Connect connector requests the committed offsets for each source partition, the format of which is being changed.
The users of the connectors will have to produce the offset in the new format during the upgrade. There don't seem to be any standard tools for that (see KAFKA-4107, KIP-199) but there is an article online that describes the principles.

The connector won't be able to consume database history produced by its older version. As of https://github.com/debezium/debezium/pull/2871 (DBZ-2975), during the database history recovery, the messages are filtered by the source partition(s) being processed by the task.
The users of the connectors will have to rewrite the database history during the upgrade and remove the source.server field from each message:
kcat -b <brokers> -C -t <history-topic> -e > history.json
cat history.json | jq 'del(.source.server)' > history-2.0.json
# delete and create <history-topic>
kcat -b <brokers> -P -t <history-topic> < history-2.0.json

TODO:

[ ] Provide an example of producing a new connector offset.
[x] Provide an example of rewriting database history.

@morozov Could you please rebase your PR? https://github.com/debezium/debezium/pull/3373 removed all legacy MySQL related code.

Thanks, @morozov! Given the highly disruptive nature of this change, I'm wondering though whether it actually is worth it?
While we are ready to make breaking changes for 2.0, defaulting to making the upgrade not a drop-in one in each and every case, requiring complex operations like rewriting the history, I'm quite doubtul. After all, that's a non user-facing nuisance, which fixing IMHO isn't worth such disruption of users.
Curious what other folks here think?

Given the highly disruptive nature of this change, I'm wondering though whether it actually is worth it?

That's a fair concern. I didn't realize that this change would affect the database history until I implemented it.
Here are a few considerations:

A similar migration will be required for the SQL Server connectors anyways if the support for single-partition mode is removed (the committed offsets and the database history will have to contain the database name for the connector to recognize them).
As discussed offline database.server.name is part of the API exposed by the connector (topic names, schema names). The fact that it's also imprinted into each database history message an undesired implementation detail. Being an internal topic, the database history topic shouldn't be coupled to the public API.

The current implementation may make certain operational scenarios unnecessarily complex. For instance, there is an SQL Server instance srv1 with databases a and b and a corresponding multi-database connector kctr1. There's also a similar setup srv2 / kctr2. In order to reduce the load on srv1, I want to move a from srv1 to srv2. Besides just copying the database history of a from srv1.history to srv2.history, I also need to alter every message by replacing srv1 with srv2 because it's part of the source metadata. At the same time, it's not used for anything and is redundant.

If there are other, more user-facing changes planned for 2.0 that would require a similar migration anyways, it would be justified to piggyback and make this change as well.

As for intrusiveness, I'd say that the migration process itself is fairly simple but it indeed may be complicated by different operational factors, e.g. the number of connectors to migrate, the size of the history to rewrite, etc.


services.nginx.config not working as described

Issue description
The variable services.nginx.config is described as "Verbatim nginx.conf configuration." I have used this variable for a while without any problems.
However, something changed recently that caused there to be a second "http" block added to the generated /nix/store/...-nginx.conf, which causes the nginx service to fail in the configuration verification invoked in the nginx systemd unit's ExecStartPre field.
Steps to reproduce
Use the nixos "unstable" channel corresponding to Git revision 20f009d56df9a0e5f224761ba48aa4142564753e.
Pass a valid nginx configuration file to services.nginx.config, enable services.nginx, and rebuild the system. systemctl status nginx should demonstrate that the service has failed like this:
nginx[14176]: nginx: [emerg] "http" directive is duplicate in /nix/store/...-nginx.conf:70

Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
16.09pre87999.20f009d (Flounder)

Nix version: (run nix-env --version)
nix-env (Nix) 1.11.2

Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)
"16.09pre87999.20f009d"

You are probably referring to the change from #15862, which on the whole seems like a good idea to me. I assume someone will add an nginx section to the release notes for next release.

I'll have a look later today, do you use any other nginx config options apart from services.nginx.config?

I'll have a look later today, do you use any other nginx config options apart from services.nginx.config?

The only additional variable I had been using was services.nginx.enable.

You are probably referring to the change from #15862, which on the whole seems like a good idea to me. I assume someone will add an nginx section to the release notes for next release.

Having a declarative abstraction over 3rd party software configuration is a great idea but, in general, I think there needs to be an "escape hatch" for users who want to use the 3rd party upstream's configuration format.


Fix (big bug, no fake)

Removed the redundant func in function names, they waste precious bytes (16 of them !) of the pristine sectors of my next gen nvme ssd.

I appreciate your comment about this waste of precious bytes, I'll probably release a new version of my script and make it with less bytes eater function names.


Gpr malloc is all

cc @jboeuf

And I am missing a lot of imbalanced frees...

Thanks for the cleanup. Please make sure to sanitize the realloc calls as well.

That one is ready to go.


maintain process info

Save all process info in process.json file in container root directory. With it, runv  ps can look up and verify each of them.
[root@hypervsock foobar]# cat process.json
[
        {
                "id": "init",
                "pid": 66524,
                "cmd": "sh",
                "create_time": 10904725
        }
]

With it, runv ps works and containerd is able to interpret its result properly.
[hypervsock@containerd]$sudo runv --root /run/containerd/runc/default ps foobar
PROCESS     CMD
70492       sh
70559       top
[hypervsock@containerd]$sudo runv --root /run/containerd/runc/default ps foobar -f json
[70492,70559]
[hypervsock@containerd]$sudo ctr t ps foobar
PID
70492
70559

the output of runc ps is changed. Could you also change it for runv ps?  At least change it to be what containerd expects.

@laijs updated

LGTM


Revisit credential cancellation - UX flow + redesign

Want to cancel credential - revoke it from the Admin Dashboard, even before user collects
Is your feature request related to a problem? Please describe.
When I click on option to revoke credential that has been sent but not claimed, it does not revoke
Describe the solution you'd like
When I click on option to revoke credential that has been sent but not claimed, I expect to be revoked even if:

email not opened
cred not collected
cred not added to wallet

Another option:

for creds that have been sent BUT NOT claimed, they should canceled
stop/invalidate credential vs revoke option
revocation API

Describe alternatives you've considered

Currently, user needs to accept/verify/collect cred in order to revoke - not ideal?

Additional context
Use cases:

bulk revocation: if many creds issued at once and error found, want to revoke all
via csv

Reviewed with Kerri, closing this ticket.


[SPARK-7608] Clean up old state in RDDOperationGraphListener

This is necessary for streaming and long-running Spark applications. @zsxwing @tdas

Merged build triggered.

Merged build started.

Test build #32633 has started for   PR 6125 at commit 33c0843.

You have to add unit tests that check for cleanup.

Merged build triggered.

Merged build started.

Test build #32638 has started for   PR 6125 at commit 8660949.

Test build #32638 has finished for   PR 6125 at commit 8660949.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32638/
Test FAILed.

LGTM

retest this please

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32640/
Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #32643 has started for   PR 6125 at commit 8660949.

Test build #32633 timed out     for PR 6125 at commit 33c0843     after a configured wait of 150m.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32633/
Test FAILed.

Test build #32643 has finished for   PR 6125 at commit 8660949.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32643/
Test PASSed.

Master 1.4


How to keep selected text in the textbox and then keep getting suggestions like Gmail autocomplete?

How to keep selected text in the textbox and keep getting suggestions and then keep the selected text in the textbox again and keep going like this? Something like Gmail autocomlete.

Found an example. https://jsfiddle.net/Rohith_KP/69p3k0ma/
But here, I want to keep texts that are not a part of suggestion and keep going like this example again.


Some Pipelines are not getting ingested from airflow 2.5.1

Affected module
Ingestion Framework
Describe the bug
OM 13.2
OM Airflow 2.3.3
My Airflow 2.5.1
I have added my airflow as source for Pipeline Service.
In connection I choosed 'Postgres Connection' for 'Metadata Database Connection' option, defined host, port, user, password.
Then I added weekly ingestion and deployed ingestion to OM Airflow.
Some DAGs are getting ingested, but some DAGs cause fatal error. Log is attached.
To Reproduce
Described above
Expected behavior
All DAGs should be ingested successfully
Version:

OS: ubuntu 20+
Python version: 3.10
OpenMetadata version: 13.2

Issue:
I'm trying to enable ingestion of lineage metadata from airflow into OMD and I followed your documentation (https://docs.open-metadata.org/connectors/pipeline/airflow/lineage-backend).

Installed openmetadata-ingestion library
Added environment variables for docker file
Defined inlets and outlets using guid tables in OMD

I'm getting following errors "AttributeError: 'BashOperator' object has no attribute '_inlets'. Did you mean: 'inlets'?"
And error appear in 2 scripts "backend.py" and "xlets.py"!
Version:
Python version: 3.10
OpenMetadata version: 13.2.0
Airflow version: 2.5.0

Affected module
Ingestion Framework
Describe the bug
OM 13.2
My Airflow 2.5.0 (self hosted)
I have added my airflow as source for Pipeline Service.
In connection I choosed 'Postgres Connection' for 'Metadata Database Connection' option, defined host, port, user, password.
Then I added daily ingestion and deployed ingestion to Airflow.
Some DAGs are getting ingested, but some DAGs cause fatal error. Log is
Affected module
Ingestion Framework
Describe the bug
OM 13.2
My Airflow 2.5.0 (self hosted)
I have added my airflow as source for Pipeline Service.
In connection I choosed 'Postgres Connection' for 'Metadata Database Connection' option, defined host, port, user, password.
Then I added daily ingestion and deployed ingestion to Airflow.
Some DAGs are getting ingested, but some DAGs cause fatal error. Log is attached.
To Reproduce
Described above
Expected behavior
DAGs should be ingested successfully
Research:
Dags configuration json file is getting created under dag_generated_configs directory
Dags code python file is not getting created under dags folder.
here are the logs
Version:

OS: Amazon Linux 2
Python version: 3.8.15
OpenMetadata version: 13.2
openmetadata-managed-apis:  <IP_ADDRESS>


Add support for indium debugger

This PR includes some refactoring to support easy customization of debuggers as well as an implementation of the indium debugger. (The existing realgud debugger is kept and is still the default)

@scottaj As usual, I'm going to let these PRs sit here for a week or so in case you want to take a look. #57 has a bit of a hack in the test code, but no user-facing changes.
If want to review this PR, 8284785 is probably the best place to look as the other commits are more routine changes


Visual Studio 2022

I really love this theme and It would be amazing if you could port the theme to Visual Studio 2022, it is very easy using Theme Converter for Visual Studio
Thanks in advance!

After a quick glance it looks like you need Windows and a .NET environment which I don't have access to.


[Feature] Add support to link build assets

Clear and concise description of the problem
Currently it is not possible for Woodpecker plugins to integrate with the Ui. So it is for example not possible to download builds (which were saved with the drone s3 plugin, for example) directly from the Ui.
Suggested solution
A simple solution would be to give plugins the ability to store links in a build. These links are then displayed in the Ui. So, as a solution for my example, a download link to the build could be stored.
Alternative
A long term solution would be to allow plugins to integrate with the Ui. But since this is too much effort for now, it should be discussed in a separate feature request.
Additional context
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

should probably be solved by #1006


StratifiedKFold Doesn't work for non-binary data

StratifiedSubject seems to work fine, but the StratifiedKFold from scikit-learn doesn't seem to accomodate continuous data yet.  See issue https://github.com/scikit-learn/scikit-learn/issues/4757.   We should either write our own version or switch out to RandomizedKFold

Fixed.  We have added a KFoldStratified() class that behaves similarly to scikit-learn cross-validation classes.


Change the source layout for integrations

Now that we have better nomenclature for integrations we should change the repo layout to match the product. We should have src/Integrations folder but within it we have a few options:

Make all of the integrations top level in the Integrations folder, both client and hosting integration live side by side.
Try to group integrations by the integration software (e.g. kafka) src/Integrations/Kafka/Aspire.Hosting.Kafka, src/Integrations/Kafka/Aspire.Confluent.Kafka.
More options that I haven't thought about.

It would also be a chance to figure out if we should move the tests as well. Keeping each integration "self contained".
Thoughts welcome.
cc @radical @eerhardt @mitchdenny

Would you want to move the integration aligned playgrounds as well?

We could yea.

I'm looking at this and currently have a file structure like....
integrations
-AWS
--Aspire.Hosting.AWS
---src
---tests
--playground
---AWS (I just kept the names of the playground directories here instead of flattening so there is room to add additional playgrounds per integration by default)
So far this structure seems to scale well to all of the integrations, and puts each integration's code into nice self-contained packages.
What about in the solution though?  Should it mirror this exactly or is it better to go with a structure there like...
Integrations
-AWS
--src
--tests
--playgound
---AWS
I have everything moved around locally at this point and need to start reworking the solution file and paths in csproj files.

@davidfowl I started working on this in a branch.  Take a look and let me know if this is heading in the right direction.  There's still a lot of work to do on it, but I think it's far enough along to get a better sense of all everything we want to shift/better align.

Since this touches so many files would the preference be to do a PR per integration or just shove it all through in one shot?


netbox: 3.7.3 -> 3.7.4

Description of changes

Changes: https://github.com/netbox-community/netbox/releases/tag/v3.7.4
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)

netbox_3_7
netbox-upgrade

made sure NixOS tests are linked to the relevant packages

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

ping @n0emis & @RaitoBezarius


Read username/password from environment variables

In addition to interactively asking for username/password on nanobox login (and consequently on nanobox remote add ...) Nanobox should be able to read credentials from predefined environment variables. I'd suggest:

NANOBOX_USERNAME
NANOBOX_PASSWORD
NANOBOX_ENDPOINT

This is essential for workflows where e.g. deployments should be triggered from a CI server after performing all necessary checks and tests. Usually data like credentials can be stored as encrypted environment variables here which are decrypted and provided during build.

great suggestion. and it should be a quick capture.

Thanks a lot for adding this. :-)

no problem. I actually was able to take it just one step further. if you have the env var's set you shouldnt need to run the 'login' command at all.

@lyondhill Yeah, saw that and this matches my planned CI workflow perfectly. :-)


Issue recovering stringified date||url into generic type

Hey, need some help deserializing a value into a proper generic type
Your JSON dictionary:
{"customType":{
        "value":1490864456.164102
      }
}

Your model:
struct CustomType<T>: Mappable{
    var value: T

    init?(map: Map) {}

    mutating func mapping(map: Map) {
        value <- map["value"]
    }
}

What you did:
let customValueDate = CustomValue<Date>(value: Date())
let customValueURL = CustomValue<URL>(value: URL())
...etc

What you got:
When my generic uses any type which should need a transform (Date, URL...), it serializes well into JSON, but it indeed can't deserialize it inferring that the string representation of my value needs to be transformed back to the desired type of my generic.
I thought about storing along with my value it's stringified type, to be able to switch() it to get back the value correctly typed and transform, but I kind of loose the interest using generics here.
Is there any builtin way to accomplish what I need, like a way to add to the JSON the transform needed to deserialize back the value into the correct type ?
Or maybe I'm just missing something ?
Thanks !
Edit: Tried wrapping up a Date into a Mappable object, then doing like here, but this
var valueJSON: Dictionary<String, AnyObject>?
valueJSON <- map["value"]

if(valueTypeString == String(describing: MappableDate.self)) {
    value = Mapper<MappableDate>().map(JSONObject: valueJSON)
}

triggers "Type of expression is ambiguous without more context" on assigning value
Edit 2: Found a nice way of accomplishing that, will update as soon as I get it clean

I handled this that way:
class CustomValueTransformer<T> {
    static func transform() -> TransformOf<T, Any> {
        return TransformOf<T, Any>(fromJSON: { (value: Any?) -> T? in
            if T.self == Date.self {
                return ISO8601DateTransform().transformFromJSON(value) as! T?
            } else {
                return value as! T?
            }
        }, toJSON: { (value: T?) -> Any? in
            if T.self == Date.self {
                return ISO8601DateTransform().transformToJSON(value as! Date? )
            } else {
                return value as Any?
            }
        })
    }
}

Usage:
value <- (map["value"], CustomValueTransformer.transform())
If someone is interested someday, I could add a configurator were a Transform could be registered to a Type, which'd avoir updating CustomValueTransformer.transform each time a new type is added.
Closing the issue


Update UI.xaml.cs

If Rx = 13.2Gy, crop Lung+5mm instead of 3mm

Adapted pull request to ask user if they want to increase the cropping margin on the lungs rather than just doing it. Also modified code to replace the lungs sparing item in-place rather than appending it to the end. See latest version of beta branch


removeAdditional is not consistent with documentation.

What version of Ajv are you using? Does the issue happen if you use the latest version?
7.0.3
Ajv options object
{ removeAdditional: true }

JSON Schema
{
  type: "object",
  oneOf: [
    {
      properties: {
        foo: { type: "string" }
      },
      required: ["foo"],
      additionalProperties: false
    },
    {
      properties: {
        bar: { type: "integer" }
      },
      required: ["bar"],
      additionalProperties: false
    }
  ]
}

Sample data
{ bar: 1 }

Your code
Please see this codesanbox. Change ajv version from 7 to 6 to compare.
https://codesandbox.io/s/ajv6-oneof-removeadditional-sf9yt?file=/src/App.tsx
Validation result, data AFTER validation, error messages
The data is valid.
What results did you expect?
According to the doc "With the option removeAdditional: true the validation will pass for the object { "foo": "abc"} but will fail for the object {"bar": 1}. It happens because while the first subschema in oneOf is validated, the property bar is removed because it is an additional property according to the standard (because it is not included in properties keyword in the same schema)."
This was true in ajv 6 but  not in ajv 7.
As far as I'm concerned, I believe that that the example in the doc is pretty odd and that the current behavior is correct.
Are you going to resolve the issue?
No, I am working on the removeUnevaluated property and I believe that removeAdditional has many counterintuitive elements and should be deprecated.

I can reproduce - thank you - it needs further investigation.

I can reproduce - thank you - it needs further investigation.

This works as documented with allErrors: true option, the reason it does not work as documented without is because the branch is abandoned before additionalProperties is evaluated (because it fails required as well).
I think this is not the only case when the behaviour of removeAdditional depends on this option...


Autos Window support when using ptvsd 4.x/debugpy debugging

We supported the autos window in legacy debugger
IVsLanguageDebugInfo.GetNameOfLocation
Not sure where to hook this up when using the debug adapter.
Analyzer provided the available expressions with ProximityExpressionsRequest

The interface has to be implemented on the VS language service, and it's VS itself that hooks up the debug session with that method - basically, the debugger reports documents and line numbers in stack frames, and VS is then supposed to look that up on the language service corresponding to the language type for that session and/or document (not sure what the algo is exactly).
If it's not working, there's probably some disconnect wrt language types somewhere. I think it might be due to how we register it as a debuggable language:
https://github.com/microsoft/PTVS/blob/122ea96df125697638bd18211205a4c684639d50/Python/Product/PythonTools/PythonToolsPackage.cs#L99
https://github.com/microsoft/PTVS/blob/1d04f01b7b902a9e1051b4080770b4a27e6e97e7/Common/Product/SharedProject/ProvideDebugLanguageAttribute.cs#L25-L43
Note that under the AD7Metrics key, the language is associated with a debug engine GUID, and it specifies the GUID of our legacy debugger. So it might be that, because the frames come from the built-in debug engine that wraps debug adapters, it has a different engine GUID, and hence doesn't get wired up.
If so, then we should just change this GUID to point at the new debugger, although that will also disable Autos support for the legacy one.

Also note that the implementation of autos support is in our implementation of IVsLanguageDebugInfo.GetProximityExpressions. It currently relies on analyzer functionality that will have to be reimplemented in new language server.

Autos window is no longer empty like it was for a while, but the algorithm that automatically populates it is not as good as what we used to have in ptvsd 3.x.


fix missing millisecond unit for hystrix timeout

Fixes #76

Coverage remained the same at 87.234% when pulling 77a91a4ac69449801a456cae593e009eeb97ffee on flof:master into 93314ae5ecfa40b6fd10373b9ec54089bd061cb8 on gojek:master.


Add Awesome Pentest Cheat Sheets

Added Awesome Pentest Cheat Sheets
URL: https://github.com/ByteSnipers/awesome-pentest-cheat-sheets
Awesome Pentest Cheat Sheets contains a lot of security and pentesting related information. The list has a huge list of very sorted and selected resources, which can save a lot of time to any security oriented people. This list was started 2017 and was improved over the years. The initial awesome pentest cheat sheets collects over 3.8k stars

Thanks for making an Awesome list! 🙌
It looks like you didn't read the guidelines closely enough. I noticed multiple things that are not followed. Try going through the list point for point to ensure you follow it. I spent a lot of time creating the guidelines so I wouldn't have to comment on common mistakes, and rather spend my time improving Awesome.

It appears that many of your items do not have any description. You also need to review at least 2 pull requests before making your own pull request.
Additionally running the linter on your repo, the following errors are given:
  README.md:323:3
  ×    9:1    ToC item "Security Talks and Videos" does not match corresponding heading "🖥 Security Talks and  Videos"                                            remark-lint:awesome-toc
  ×    9:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-security-talks-and-videos                                                            remark-lint:double-link
  ×   11:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-mobile-pentesting                                                                    remark-lint:double-link
  ×   12:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#%EF%B8%8F-cloud-pentesting                                                            remark-lint:double-link
  ×   13:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#active-directory                                                                      remark-lint:double-link
  ×   14:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-pentest-methodology                                                                  remark-lint:double-link
  ×   15:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-privilege-escalation                                                                 remark-lint:double-link
  ×   16:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#%EF%B8%8F-web-pentesting                                                              remark-lint:double-link
  ×   17:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#binary-exploitation                                                                   remark-lint:double-link
  ×   18:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#learning-platforms                                                                    remark-lint:double-link
  ×   19:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#bug-bounty                                                                            remark-lint:double-link
  ×   20:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#tools                                                                                remark-lint:double-link
  ×   21:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/?tab=readme-ov-file#payloads-1                                                         remark-lint:double-link
  ×   22:3    https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/?tab=readme-ov-file#programming                                                        remark-lint:double-link
  ×   60:100  Unexpected reference to undefined definition, expected corresponding definition (pdf) for a link or escaped opening bracket (\[) for regular text  remark-lint:no-undefined-references
  ×  243:3    https://owasp.org/www-project-vulnerable-web-applications-directory/#div-online                                                                    remark-lint:double-link
  ×  251:3    https://owasp.org/www-project-vulnerable-web-applications-directory/#div-offline                                                                   remark-lint:double-link
  ×  323:3    Don’t pad link with inner spaces                                                                                remark-lint:no-inline-padding

  19 errors

For the first error just make your title for the header 🖥 Security Talks and  Videos the same by either removing the emoji or from the header or adding the emoji to the title in the Table of Contents(ToC).
The double-link errors can be easily solved by changing the links to #(Your header here). Example: https://github.com/ByteSnipers/awesome-pentest-cheat-sheets/#-security-talks-and-videos ---> #-security-talks-and-videos.
Then the last issue needs to be fixed too. (no-inline-padding)

I've update the repository, but I still have some issues, what I cannot solve, because I cannot identify why they are issues.
✖   15:1  ToC item "Privilege Escalation" does not match corresponding heading "Web Pentesting"  remark-lint:awesome-toc
✖  243:3  https://owasp.org/www-project-vulnerable-web-applications-directory/#div-online        remark-lint:double-link
✖  251:3  https://owasp.org/www-project-vulnerable-web-applications-directory/#div-offline       remark-lint:double-link
✖  323:3  Don’t pad link with inner spaces                                                       remark-lint:no-inline-padding

I run your repo in linting and can see some errors are still need to be resolved.

This looks good. Can I suggest you create a separate folder for Azure Security Architecture and put everything about it there or maybe you can create a separate awesome list for Azure Security Architecture?

Apart from fixing liniting issues I suggest you add a books sections and books related to book bountry web penetration testing mobile app pentesting....etc

This pull request has a title in the format Add Name of List. It should not contain the word Awesome.

As per the guidelines, the title of the pull request should be "add Pentest Cheat Sheets".


Percentage not showing for roller blind

First off this is fantastic! I was in the middle of reverse engineering this to do exactly what you did - so much better to just have it done.
I just upgraded to HomeAssistant 2024.04 but I don't seem to get any percentages back within the UI of how far down the blind is, is that expected?

Hi!
Awesome! Firstly, Motionblinds Bluetooth motors work only using connections which means Home Assistant must first connect after which it receives a status update that includes information on position, tilt, battery percentage, etc. After the motor is disconnected, the cover in Home Assistant will go back saying the position is unknown.
If you use device.connect() or the connect button in Home Assistant you should receive position information with the position of the cover being adjusted. On the other hand, if you use the slider then I have configured it to not use the position information. The reason for this is that updating the position of the blind to be open when the user has for example moved the slider to close the blind will result in the cover slider moving down due to the position information when the user has just moved it up to open the blind.
You could of course always keep a permanent connection to the blind by constantly calling the connect service (or this could be an additional feature in the future), but this would only really be useful for blinds that are wired or if you want to always charge the blind, otherwise the battery will of course drain very fast.

I just upgraded to HomeAssistant 2024.04 but I don't seem to get any percentages back within the UI of how far down the blind is, is that expected?

Unless you use the connect button, you should get percentages back in the UI as mentioned above, even when you are for example connected to the blind and you pull on the cord. If that is not the case or you find something else that does not work, let me know here by leaving a debug log and I will have a look!

Ah, that's makes sense - I didn't realize you had to connect and disconnect. It seems to be working as expected then.

Good to know! This integration works in the same way the Motionblinds Bluetooth mobile app does, which also requires a connection.
There is still an open PR that will add sensors such as a connection sensor that shows whether Home Assistant is connecting, connected, disconnecting, or disconnected to the motor.


Rust cli github authentication

Proposed Changes
Issue: https://github.com/ockam-network/ockam/issues/1068
Basically you just need to run the following in tools/ockam-hub-cli directory:
cargo run -- auth

PS: It doesn't show the ockam-hub's address, because I didn't have api instructions. If I have, I can implement it :)

Merged the changes in https://github.com/ockam-network/ockam/pull/1403 - thanks!


Update tflite wheel version to 1.13.1

This PR updates tflite wheel version to 1.13.1
I also added alternative way to get tflite module by cloning https://github.com/FrozenGene/tflite.git repo.
Merge it after web-data PR is merged https://github.com/dmlc/web-data/pull/190

@FrozenGene could you review it?

Given that we are not maintaining TFLite. I recommend we keep all the docker builds as binary, give instructions to install tensorflow and host the wheel image somewhere else.

nd host the wheel image somewhere else.

@tqchen  I can replace # Setup tflite from schema in ubuntu_install_tflite.sh with pip3 install tflite-1.13.1-py3-none-any.whl after you merge https://github.com/dmlc/web-data/pull/190 which adds tflite-1.13.1-py3-none-any.whl to 'web-data`

Sorry that I am not being clear.  I think we should build from source in the docker script when possible. As the binary wheel distribution is not really official.

@tqchen currently we use tflite-0.0.1-py3-none-any.whl wheel only in the tutorial
https://docs.tvm.ai/tutorials/frontend/from_tflite.html
This particular wheel version 0.0.1 was generated from schema 1.12
This PR updates tflite wheel to version 1.13.1 - tflite-1.13.1-py3-none-any.whl
In the tutorial we offer two option on how users can install tflite package

prebuilt wheel:
generate TFLite package using flatc

in the docker file we use Option #2 - flatc
If the options listed above are fine with you then can you merge PR https://github.com/dmlc/web-data/pull/190
PR 190 adds tflite-1.13.1-py3-none-any.whl to web-data repo

@FrozenGene What you think if you make new release in you repo FrozenGene/tflite (version 1.13.1)?
You can attach newtflite wheels as the release Assets.
In TVM tutorial we can use wheel URLs pointing to release Assets instead of web-data.
Assets URLs will be smth like:
https://github.com/FrozenGene/tflite/releases/download/v1.13.1/tflite-1.13.1-py2-none-any.whl
https://github.com/FrozenGene/tflite/releases/download/v1.13.1/tflite-1.13.1-py3-none-any.whl
Looks like PR https://github.com/dmlc/web-data/pull/190 will never be merged.

Given that we do not maintain tflite, I personally think it is not a good idea to distribute tflite binaries as part of the project. We could certainly advertise  FrozenGene/tflite, or even better, point instructions to install tflite from the original tf repo

@FrozenGene I created tflite Release 1.13.1 in my forked repo
https://github.com/apivovarov/tflite/releases/tag/v1.13.1
Assets:
https://github.com/apivovarov/tflite/releases/download/v1.13.1/tflite-1.13.1-py2-none-any.whl
https://github.com/apivovarov/tflite/releases/download/v1.13.1/tflite-1.13.1-py3-none-any.whl
Can you do the same in your repo?

Changed web-data/master/tensorflow/tflite/whl wheel links to FrozenGene/tflite/releases/download links

@apivovarov I think you could make a PR on my tflite repo, then I merge it. If there is a need, I could create a release branch after merging your assets wheel files

@apivovarov I have done it. You could do a double check whether it is you want.

Thank you Zhao! The release and assets look good!
@tqchen I updated TFLite tutorial to use wheel files from FrozenGene/tflite/releases/download.
This PR should be ok to merge now.

Thanks @FrozenGene @apivovarov , this PR is now merged