Hg38 VCF file error: Error in UseMethod("select") :    no applicable method for 'select' applied to an object of class "NULL"

Hi everyone,
I am trying to execute a leukemia sample aligned with STAR with hg38, featurecounts (hg38.gtf) and GATK Haplotype caller inputs.
Count header:
ENSG00000160072	1222
ENSG00000279928	87
ENSG00000228037	37
VCF header:
#CHROM	POS	ID	REF	ALT	QUAL	FILTER	INFO	FORMAT	851
chr1	10560	.	C	G	37.32	.	AC=2;AF=1.00;AN=2;DP=2;ExcessHet=0.0000;FS=0.000;MLEAC=1;MLEAF=0.500;MQ=60.00;QD=18.66;SOR=0.693	GT:AD:DP:GQ:PL	1/1:0,2:2:6:49,6,0
chr1	14464	.	A	T	152.14	.	AC=2;AF=1.00;AN=2;DP=4;ExcessHet=0.0000;FS=0.000;MLEAC=2;MLEAF=1.00;MQ=60.00;QD=25.36;SOR=3.258	GT:AD:DP:GQ:PL	1/1:0,4:4:12:166,12,0
The command:
RNAseqCNV_wrapper(config = "C:/Users/neret/Documents/R/win-library/4.1/RNAseqCNV/extdata/config_mock_2.txt", metadata = "C:/Users/neret/Documents/R/win-library/4.1/RNAseqCNV/extdata/metadata_mock_851.csv", snv_format = "vcf", genome_version = "hg38")
And I obtain this answer which I read in the #7 Issue referring but, the programe has been updated for hg38, so I do not understand why is happening this. I guess it is something related to the VCF file:
Error in UseMethod("select") :
no applicable method for 'select' applied to an object of class "NULL"
Thanks in advance,

Hi,
I am glad you are trying out our package!
Unfortunately, the error message is very unspecific, so there are many possibilities which might be causing this. If there is any other information regarding the error, please send it as well!
RNAseqCNV_wrapper generates messages, which show which part of the analysis is running. This message will also help us locate the issue.
Is this happening for all of your files, or only for certain ones?
The two most probable causes are:

Incompatible input - the format of the count file and the vcf file look good at first glance. This does not necessarily mean there is no other issue with them. The metadata and config files might also be incompatible. The fastest way to troubleshoot would be to send me all the input files, and I could analyze what is happening in detail, but I understand, that this might not be possible due to data security.
Versioning problem - although I think this is less likely due to the type of your error, it might be the case, that due to newer versions of some packages, RNAseqCNV does not run as expected. You could try running your analysis in a docker container if you want to test this.

In summary, please send me all the information regarding the error, so we can be more specific in troubleshooting.
Best,
Jan

Dear Honzee,
Thank you very much for your dedication!
By this "RNAseqCNV_wrapper generates messages, which show which part of the analysis is running. This message will also help us locate the issue" you mean this message?
[1] "Analysis initiated"
[1] "Normalization for sample: 851 completed"
[1] "Preparing file with snv information for: 851"
Reading in vcf file..
Extracting depth..
Extracting reference allele and alternative allele depths..
Needed information from vcf extracted
Finished reading vcf
Error in UseMethod("select") :
no applicable method for 'select' applied to an object of class "NULL"
I would be glad to send you the inputs to your email for the troubleshooting.
Thanks a lot for you time and help,
Best regards,
Nere,

I see, thank you for the information. If you are ok with this, you can send me the files to<EMAIL_ADDRESS>

Hi Nere,
I have looked into your files and there was an issue with the name of the sample. RNAseqCNV did not assume, that the file name could be purely numerical (like 851), which subsequently caused an error when creating a name for an element in a list. I corrected this bug and pushed the changes to the repository. You can either download the new version or not use numerical sample names.
Thank you for letting me know about this and sorry for the trouble.
Best,
Jan

Dear Honzee,
I am very glad to have received your email and what's more, checking that
my samples run you program and that I was able to confirm hyperdiploidy.
Thank you for your help and dedication,
Great program,
Best regards,
Nere,
El sáb., 18 mar. 2023 20:09, honzee @.***> escribió:

Reopened #19 https://github.com/honzee/RNAseqCNV/issues/19.
—
Reply to this email directly, view it on GitHub
https://github.com/honzee/RNAseqCNV/issues/19#event-8785605618, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A3SZASX3EPVLE4OAFN5PCZTW4YB7RANCNFSM6AAAAAAVG7YAK4
.
You are receiving this because you authored the thread.Message ID:
@.***>

That makes me quite happy, thank you.
I will be closing the issue now.


Add blog to website

This PR addresses #730

[x] Add page that list all blog posts (auto generated)

[x] Sort posts by date
[x] Style posts list page

[x] Show author of the post
[x] Add page that show all posts from a specific tag
[x] Show tags on the post page
[x] Add page that shows all tags
[ ] Style tags
[ ] link the most recent blog post on the top of the homepage
[ ] Add instructions in the website readme

Blog url structure:
/blog/ Shows all posts
/blog/posts/{postName} Shows the blog post
/blog/tags Shows all tags
/blog/tags/{tagName} Shows all posts tagged with that tag

@sebmck i might finish this tomorrow. anything to add or change?

Nope, looks good to me!


Portal does not check if exit portal is in a scene tree

SUPER helpful example, thank you for creating it!
Quick nitpick: currently, if the exit portal is orphaned (due to user error with dynamic creation, etc), the viewport will remain enabled, and Godot will respond to all the "global_transform" calls with Transform3D.IDENTITY. So the portal will still attempt to render, but will just show whatever's at the origin.
N.B. Godot doesn't check "is_inside_tree()" as part of "is_visible_in_tree()", it only checks whether the node has any ancestors marked as not visible.
While this is definitely a PEBKAC/"well then don't let that happen, dummy" case, the current behavior is not the easiest to debug, and this could be changed with one extra check here: https://github.com/Donitzo/godot-simple-portal-system/blob/main/src/scripts/portal.gd#L65 or here: https://github.com/Donitzo/godot-simple-portal-system/blob/main/src/scripts/portal.gd#L121

Thank you for the feedback.
That's a good suggestion and I have implemented it in the portal class. I also added a check for if the portal belongs to the correct SceneTree during raytracing. I guess it would technically be possible to have multiple scenes with portals if for example using ViewPorts with their own worlds.


feat: synchronous retry API calls

@adamdecaf
Sorry for merging before your approval. This is a blocker for me to complete https://github.com/namecheap/terraform-provider-namecheap/pull/54
Feel free to ask me if you have any questions/suggestions.

No worries @ifree92!


runtime error in port filter

I'm running master with 1f66308
swarm.master_1 | 2015/10/13 22:02:51 http: panic serving <IP_ADDRESS>:47556: runtime error: invalid memory address or nil pointer dereference
swarm.master_1 | goroutine 567 [running]:
swarm.master_1 | net/http.(*conn).serve.func1(0xc82086d6b0, 0x7fbaf36c0058, 0xc82014ac08)
swarm.master_1 |        /usr/local/go/src/net/http/server.go:1287 +0xb5
swarm.master_1 | github.com/docker/swarm/scheduler/filter.(*PortFilter).portAlreadyInUse(0x11a9ae0, 0xc82025ab40, 0x0, 0x0, 0x0, 0x0, 0x0)
swarm.master_1 |        /go/src/github.com/docker/swarm/scheduler/filter/port.go:92 +0xb2
swarm.master_1 | github.com/docker/swarm/scheduler/filter.(*PortFilter).filterBridge(0x11a9ae0, 0xc820fa6380, 0xc82014ac48, 0x1, 0x1, 0x0, 0x0, 0x0, 0x0, 0x0)
swarm.master_1 |        /go/src/github.com/docker/swarm/scheduler/filter/port.go:52 +0x27e
swarm.master_1 | github.com/docker/swarm/scheduler/filter.(*PortFilter).Filter(0x11a9ae0, 0xc820fa6380, 0xc82014ac48, 0x1, 0x1, 0x0, 0x0, 0x0, 0x0, 0x0)
swarm.master_1 |        /go/src/github.com/docker/swarm/scheduler/filter/port.go:28 +0x15a
swarm.master_1 | github.com/docker/swarm/scheduler/filter.ApplyFilters(0xc82007e800, 0x5, 0x8, 0xc820fa6380, 0xc82014ac48, 0x1, 0x1, 0x0, 0x0, 0x0, ...)
swarm.master_1 |        /go/src/github.com/docker/swarm/scheduler/filter/filter.go:61 +0x12d
swarm.master_1 | github.com/docker/swarm/scheduler.(*Scheduler).SelectNodesForContainer(0xc820108360, 0xc82014ac38, 0x1, 0x1, 0xc820fa6380, 0x0, 0x0, 0x0, 0x0, 0x0)
swarm.master_1 |        /go/src/github.com/docker/swarm/scheduler/scheduler.go:37 +0xb3

This failure is triggered by this test in the docker-compose test suite. It fails when compose tried to create two containers with the same port.  The create container call runs in parallel on the client, so both containers are created at approximately the same time. This failure seems to be easily reproducible (every run causes this failure).
You can reproduce the failure with:
parallel docker run -d -p 8888 alpine:edge sleep ::: 2 2

Actually, I think any test that use the compose scale command causes this failure.
I wasn't seeing it when running against master from about a week ago, so I think it's something that was merged recently.

I have the same issue:
2015/10/19 12:39:12 http: panic serving <IP_ADDRESS>:54360: runtime error: invalid memory address or nil pointer dereference
goroutine 22068 [running]:
net/http.(*conn).serve.func1(0xc8211ef3f0, 0x7f6085941f18, 0xc820953080)
/usr/local/go/src/net/http/server.go:1287 +0xb5
github.com/docker/swarm/scheduler/filter.(*PortFilter).portAlreadyInUse(0x11a9b08, 0xc820540780, 0x0, 0x0, 0xc82041f590, 0x4, 0x4)
/go/src/github.com/docker/swarm/scheduler/filter/port.go:92 +0xb2
github.com/docker/swarm/scheduler/filter.(*PortFilter).filterBridge(0x11a9b08, 0xc8215ea700, 0xc82078bb00, 0xa, 0x10, 0x0, 0x0, 0x0, 0x0, 0x0)
/go/src/github.com/docker/swarm/scheduler/filter/port.go:52 +0x27e
github.com/docker/swarm/scheduler/filter.(*PortFilter).Filter(0x11a9b08, 0xc8215ea700, 0xc82078bb00, 0xa, 0x10, 0x0, 0x0, 0x0, 0x0, 0x0)
/go/src/github.com/docker/swarm/scheduler/filter/port.go:28 +0x15a
github.com/docker/swarm/scheduler/filter.ApplyFilters(0xc820123600, 0x5, 0x8, 0xc8215ea700, 0xc82078bb00, 0xa, 0x10, 0x0, 0x0, 0x0, ...)
/go/src/github.com/docker/swarm/scheduler/filter/filter.go:61 +0x12d
github.com/docker/swarm/scheduler.(*Scheduler).SelectNodesForContainer(0xc820148000, 0xc8204e4280, 0xa, 0xa, 0xc8215ea700, 0x0, 0x0, 0x0, 0x0, 0x0)
/go/src/github.com/docker/swarm/scheduler/scheduler.go:37 +0xb3
github.com/docker/swarm/cluster/swarm.(*Cluster).createContainer(0xc820127b00, 0xc8215ea700, 0xc820530f73, 0x1c, 0x0, 0x0, 0x0, 0x0)
/go/src/github.com/docker/swarm/cluster/swarm/cluster.go:143 +0x417
github.com/docker/swarm/cluster/swarm.(*Cluster).CreateContainer(0xc820127b00, 0xc8215ea700, 0xc820530f73, 0x1c, 0x0, 0x0, 0x0)
/go/src/github.com/docker/swarm/cluster/swarm/cluster.go:112 +0x6e
github.com/docker/swarm/api.postContainersCreate(0xc820134b40, 0x7f6085944010, 0xc8211de160, 0xc8202eae00)
/go/src/github.com/docker/swarm/api/handlers.go:392 +0x295
github.com/docker/swarm/api.NewPrimary.func1(0x7f6085944010, 0xc8211de160, 0xc8202eae00)
/go/src/github.com/docker/swarm/api/primary.go:128 +0x300
net/http.HandlerFunc.ServeHTTP(0xc82025c240, 0x7f6085944010, 0xc8211de160, 0xc8202eae00)
/usr/local/go/src/net/http/server.go:1422 +0x3a
github.com/gorilla/mux.(*Router).ServeHTTP(0xc820131b30, 0x7f6085944010, 0xc8211de160, 0xc8202eae00)
/go/src/github.com/docker/swarm/Godeps/_workspace/src/github.com/gorilla/mux/mux.go:98 +0x29e
github.com/docker/swarm/api.(*dispatcher).ServeHTTP(0xc820146300, 0x7f6085944010, 0xc8211de160, 0xc8202eae00)
/go/src/github.com/docker/swarm/api/server.go:32 +0x9f
net/http.serverHandler.ServeHTTP(0xc820330300, 0x7f6085944010, 0xc8211de160, 0xc8202eae00)
/usr/local/go/src/net/http/server.go:1862 +0x19e
net/http.(*conn).serve(0xc8211ef3f0)
/usr/local/go/src/net/http/server.go:1361 +0xbee
created by net/http.(*Server).Serve
/usr/local/go/src/net/http/server.go:1910 +0x3f6

+1

Are you hitting this in the 1.0.0rc1?
+kind/bug
+milestone 1.0.0

swarm version 1.0.0 (fe88189)

@dnephin @agsmorodin could you elaborate steps to reproduce this?
(It would be great if we can reproduce it without docker-compose, anyway)

@chanwit it's easy to reproduce with this:
parallel docker run -d -p 8888 alpine:edge sleep ::: 2 2

I'll add it to the PR description as well

Thanks, we are going to investigate asap

ping @aluzzardi

@dnephin Can you confirm #1314 fixing the crash?

Thanks for spotting @dnephin, this was a really nasty one.

The error doesn't reproduce anymore
Thanks!


Fixed range GRPC check

Signed-off-by: sunwp<EMAIL_ADDRESS>
Description
Fixed the range GRPC check regular expression

Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist:

[ ] Code compiles correctly.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report

Merging #872 (ca1c744) into main (6bbd7e6) will increase coverage by 0.06%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #872      +/-   ##
==========================================
+ Coverage   41.14%   41.21%   +0.06%
==========================================
  Files         102      102
  Lines        8057     8058       +1
==========================================
+ Hits         3315     3321       +6
+ Misses       4330     4326       -4
+ Partials      412      411       -1

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
41.21% <0.00%> (+0.06%)
:arrow_up:

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

Impacted Files
Coverage Δ

client/daemon/peer/piece_manager.go
62.88% <0.00%> (-0.22%)
:arrow_down:

client/daemon/peer/peertask_base.go
45.45% <0.00%> (-1.22%)
:arrow_down:

client/daemon/peer/peertask_stream_callback.go
64.07% <0.00%> (+1.94%)
:arrow_up:

client/daemon/peer/peertask_stream.go
55.96% <0.00%> (+3.05%)
:arrow_up:


breakfix: API Key Generator generates wrong key length

After implementing a working Caddyfile for the authcrunch docker container, with local users (and finding out that it is no longer /settings but /auth/profile/. I tried to generate an API-Key for my static user, which went fine. But when I tried using it with
curl -H 'X-API-Key: <key>' https://localhost/web1 I got 401 Unauthenticated. Looking into the caddy log it stated api key payload is malformed using that string I then found this code which checks if the API-Key is smaller than 72 Characters and in that case return the error.
I was confused on why that happened and checked back with the API-Key I generated and in fact it was only 64 characters long.
I generated a new API-Key and again it was shorter that 72 characters. After some more generate attempts I finally got a key that was 72 characters long and it worked properly.
I also tried to edit the generated API key in the "Add API Key form" to make it 72 characters long, which also worked.
I tried to make one longer than 72 characters and the form returned the following error:

Further test shenanigans later I was sure the key was generated in the browser and found this React file, but I am unable to contribute here, as it is minified js.
Personal Note:
I think instead of fixing the browser based API-Key generator, the key should be generated server side and checked for possible duplicates (this is because adding the same API Key twice will result in the profile page not responding anymore....)

It looks like the UI for the portal is copied on build time from another repository, but it seems that that repository is not publicly available.
Is there a specific reason for that ?


Possibility to provision devices that have OS installed

Hi!
It would be amazing for our possible use case that you could re-write the OS of a device that have OS already installed. Based on my testing of the program it was not possible, please correct me if I'm wrong.

Whether that is easily possible depends on your exact use case.
If you are just using cmprovision to write images, things are easy.
Just connect a USB<->micro USB cable between the cmprovision server, and the CMIO board's "USB slave" port.
And temporarily fit a jumper where it says "disable eMMC boot" on the CMIO board.
It will then boot the provisioning system through USB (rpiboot) instead of Ethernet, and that will also work if there is already an OS installed.
See also p. 14 of the whitepaper: https://pip.raspberrypi.com/categories/685-whitepapers-app-notes/documents/RP-003468-WP/Using-the-Compute-Module-Provisioner.pdf
Things get a lot more difficult if you have scripts configured in cmprovision that need to use USB ports during the provisioning, though.
Which will not work if the USB port is configured in device mode for the USB booting.

Oh wow, I didn't know about that documentation. I think that can cover our use case well!
On another note, I think you could have link to the whitepaper in the README. I atleast found this tool from googling and ended up straight to this github page.

Good point. Added a link to the whitepaper.


Switch from impl to dyn for passing raw window handle trait objects

Should make it a bit easier for interacting with the library due to increased flexibility.
This is mostly due to static dispatching enforces functions interacting with ash-window to also use static dispatching which can be annoying (e.g having an Option<impl HasRawWindowHandle>).
Using dyn HasRawWindowHandle relaxes the constraints a bit due to not having to explicit name the type as sometimes there is none.
Unfortunately, this is still not completely flexible as RawWindowHandle doesn't implement HasRawWindowHandle, so there is no way to get back the abstraction ladder. Using &RawWindowHandle would add the requirement for users to also add raw-window-handle dependency just to have access to HasRawWindowHandle::raw_window_handle ..

Unfortunately, this is still not completely flexible as RawWindowHandle doesn't implement HasRawWindowHandle, so there is no way to get back the abstraction ladder.

Is this something we should fix upstream? Even if not, someone can always define a newtype if they really need to.

Is this something we should fix upstream? Even if not, someone can always define a newtype if they really need to.

Good point with the newtype, even though it feels a bit akward.
Looks like RawWindowHandle had an impl in the past but it got removed ):
https://github.com/rust-windowing/raw-window-handle/issues/35


Music loop PKW

If you restart a course and finish a course just as the music is about to start the loop again it'll continue playing

This being an example. And yes the music stops after another 2 minutes (if you are still in the course you finished in) but I thought it should be mentioned

Fixed in: https://github.com/AsoDesu/IslandUtils/commit/f3768b4d7b134c0d1f962851bf45a86d1ecdb230


Auto-detect graph objects (and guess Backend) from URI

To enable this syntax:
from grand import Graph

g1 = Graph("https://example.com/net.graphml")

g2 = Graph("~/graph.sqlite3")

g3 = Graph("s3://example/my-open-cypher")

Already in a downstream package grandlite. Wondering out loud about upstreaming that feature into this package...


Validation

Hello,
Sorry for dumb question, but is there a way to include validation step after training each epoch? would it just calculate mean_loss on validation data set? how can it be used for  adjusting the weights?
Thank you,
Dimitrius

After some train step or all train step, you can use test_one_image to validate, but there is no validation step during training. And test_one_image only give results for every image, no calculatation


Create CONTRIBUTING.md

Linked to cyberark/community#30

Can you please move the ## Deploying Conjur Master and Followers (*Test and Demo Only*) section from README to CONTRIBUTING?
btw I am really glad about the ability to make this change - having those instructions in the README when they're for test/demo purposes only has been really confusing at times

Addressed


all objects i Nuuk area is misplaced, about 25m towards W

this fixpoint is placed at the outerwall in the Asiaq building:
utm22: y=7116148.640   =464353.901
(lat,lon): 64 10 11.527693 N  51 43 59.680975 W (64.16986880361111  , - 51.73324471527778 )
Who knows the 'story' behind the Nuuk objects. I guess the creator

has used utm22 from official source, Asiaq, but used a wrong transformation to (lat,lon)

Hello and welcome, @bjarnephys.
I'm not sure I understand the issue. This repository is for the iD map editor project, but the screenshot looks like a different application. You can ask more general questions about OpenStreetMap in the forums.

The buildings countours are at the same place as on Bing imagery.
So if they are in the wrong place, then imagery offset may not be properly set.
How did you measure this difference?

I know the position of the reference/fix point; it is placed in the building correctly in the utm22 map I have worked for the official map organization in Greenland, Asiaq, the fixpoint must be correct!
---------------------------------------------------------Bjarne Hansen Mobil:(+299)565535, Suloraq 3-504, 3905 Nuuk, Greenland, DK
Email<EMAIL_ADDRESS>Kontakt mig per facebook, har kontoen:https://www.facebook.com/bjarne.hansen.102Kontakt mig per Skype: 'bjarnephys' -er på alle dage omkring kl 21:15 dansk tid.
Den mandag den 25. maj 2020 20.35.43 GMT-2 skrev maro-21<EMAIL_ADDRESS>

The buildings countours are at the same place as on Bing imagery.
So if they are in the wrong place, then imagery offset may not be properly set.
How did you measure this difference?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I know the position of the reference/fix point; it is placed in the building correctly in the utm22 map. I have worked for the official map organization in Greenland, Asiaq. OpenStreetMap objects must come from an import of objects from Asiaqs map, but something has gone wrong with the projection -I would really like to know, what was done in this process.
---------------------------------------------------------Bjarne Hansen Mobil:(+299)565535, Suloraq 3-504, 3905 Nuuk, Greenland, DK
Email<EMAIL_ADDRESS>Kontakt mig per facebook, har kontoen:https://www.facebook.com/bjarne.hansen.102Kontakt mig per Skype: 'bjarnephys' -er på alle dage omkring kl 21:15 dansk tid.
Den mandag den 25. maj 2020 20.35.43 GMT-2 skrev maro-21<EMAIL_ADDRESS>

The buildings countours are at the same place as on Bing imagery.
So if they are in the wrong place, then imagery offset may not be properly set.
How did you measure this difference?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Decompiler output partly disappears after local var renaming

Describe the bug
When renaming certain local variables, not changing their type or whatnot, it is possible for the decompiler to remove other local variables and omit to decompile parts of the code.
To Reproduce
Analyze the file in the attachments(rename it to 102h.exe, github does not seem to allow exes attachments), go to 0x43f0ce, rename local_10 to whatever and see part of the program disappear.
Expected behavior
Renaming the variable not removing decompiled output.
Environment (please complete the following information):

OS: 5.1.15-gentoo
Java Version: openjdk 11.0.3-gentoo 2019-04-16
Ghidra Version: 9.1_DEV (master at https://github.com/NationalSecurityAgency/ghidra/commit/15ee8040096d41f7e3f3044de950996737edd834 )

Attachments
102h.txt

just adding more info, not familiar enough with those instructions.
Here is the diff in the debug function compilation, looks like the undefined4->float is the root issue.  Its float local_10 in Decompile and undefined4 local_10 in listing, guessing the rename triggers that type change which makes the decompiler act differently on the pcode.
< <symbol name="local_10" typelock="false" namelock="true" readonly="false" cat="-1">
< <typeref name="undefined4"/></symbol>
---
> <symbol name="whatever" typelock="true" namelock="true" readonly="false" cat="-1">
> <typeref name="float"/></symbol>

If the local variable type can come from a parameter or return type, it is usually better to define it there.  Not having looked at the example, that would be my first suggestion.  Entering the type on a local is usually the last thing done to improve a funciton, as setting local types can get in the way of the decompiler data flow analysis.

@emteere the thing with this issue is that there is a discrepancy between the listing and decompiler view, and simply changing the name of the variable, not its type, forces it to assume one type. As such the issue triggers even when trying to leave a type alone, because of its nature.

@GovanifY  I wonder if the missing instructions might have been removed from dead code options.  I'd guess local_10 is actually  float[3].  As float, writes to local_c and local_8 might look like dead code.

Yup, it looks accurate; from what I've gathered this is a vector(3) normalizing function, and in_vec = &x (param_2 = &local_10) wouldn't make a lot of sense if the function was not a float array.

Thanks for posting the example.  I've duplicated the issue.
If you define local_10 as a float[3], then the code does decompile correctly for me in the main branch.  If it doesn't for you, make sure you re-compiled the decompiler.
This is still a bug, we'll look into the root cause.

We figured that out later after having posted this bug report, thanks for looking into the bug!

Extracted function sample.
102h.bytes.txt
102h.xml.txt

I have noticed recently, at least with #874 that in the decompiler's function signature, "custom storage" is checked for some reason. It is only checked in the decompiler's function signature and not in the listings. Not sure if it is related to what is causing the bug or not.

This should no longer be a problem in the master branch, and fixes will be in the 9.2 release.  This is essentially the same problem as #524 . The renaming action forces the decompiler to incorrectly interpret a location as a distinct variable, when it is really a structure field or array element, which then results in faulty alias analysis.


Refactor Spark33XShims to avoid code duplication

Closes #5150
Refector Spark33XShims
Signed-off-by: Chong Gao<EMAIL_ADDRESS>

build

A known issue caused Check Markdown links failed
[✖] https://repo1.maven.org/maven2/com/nvidia/rapids-4-spark-tools_2.12/22.04.0/ → Status: 404

LGTM except a nit as the above.

build


GROOVY-7397 fix

I believe this should fix the issue.

Jira: https://issues.apache.org/jira/browse/GROOVY-7397

Indeed, thanks!


Task/integrate ches/141

Created an emailService for sending emails using CHES API.
See backend .env.example - there's some new env vars needed for local setup. To get the secret values, see the updated backend .env file in 1Password.
Have also added the relevant secrets to our Secrets in Github, so automated tests are now passing in CI.
Note: at the moment MailHog is not set up for github actions, so when the CI tests run I genuinely get emails. Use/abuse this as you wish. (Or you can pull the code locally and change it to your email address and spam yourself instead).

Closing this PR - a new one has been created here: https://github.com/bcgov/cas-registration/pull/1253


Configure Travis build

@umarbhatti1 This repo needs one more version mapped.

@apotterri this repo already have 5 versions mapped
credentials-2.3.12 credentials-2.3.11 credentials-2.3.10 credentials-2.3.9 credentials-2.3.8

Ok, i must have missed one. Thanks.

without agent

with agent


⚠ Cioos National CKAN has degraded performance

In 16f9250, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 6994 ms

Resolved: Cioos National CKAN performance has improved in 7a2aa18 after 29 minutes.


Custom connectors with Direct Query supported by Enterprise On-Premise Data Gateway?

Hey everyone,
I know it's not working natively but at the moment I'm trying to figure out if it is somehow possible to use Direct Query on an Azure MySQL DB in the Power BI Service.
I thought about the idea to enable Direct Query with a custom data connector via an ODBC driver.
There are two articles that let me believe Direct Query custom connectors are not supported by the enterprise on-premise data gateway.

Custom connectors using Direct Query are also not supported yet. This support should arrive with a later update. (26/Jul/2018)

From https://powerbi.microsoft.com/en-us/blog/on-premises-data-gateway-july-update-is-now-available/
and

…with support for both this and Direct Query coming to the Power BI Service via the On Premise Data Gateway - Enterprise coming in Q3. (09/May/2018)

From https://github.com/microsoft/CustomConnectorGallery
And there is no info/release dates out there that I could find, which announced or mentions this feature to be available.
Can anyone provide information about this? Thank you!

This should be possible today, right @cpopell ?

It should be possible today--I believe it says so in more up to date documentation, the gallery should probably be deprecated.


Added "toString"-option, allows Massive to return generated sql without executing it

Added "toString"-option, allows Massive to return generated sql without executing it:
var sql = db.users.save({ firstName:'Peter' }, {toString:true})
console.log(sql) // 'insert into...'

Why?

Allows massive to be used as query builder.
Simple debugging, easily view generated queries.
Makes it possible to create quick transactions:

db.query([
    'BEGIN',
    db.users.save({ name: 'Peter' }, {toString:true}),
    db.users.save({ name: 'OtherGuy' }, {toString:true}),
    'COMMIT'
  ].join(';'), function(...)
})

(I have also created a small wrapper function (db.wrapInTx) that takes unlimited numbers of queries, joins them (as above) and wraps a BEGIN/COMMIT. Didn't include it here)

Closing, doesn't work with all types of commands. Guessing because options aren't passed unaltered to the query runner


Unable to add page title in French

Description
The field for page title cannot be filled when editing in French.
Might be related to previous issue #3786
Steps to reproduce

Go to GC Forms > Edit
Add a new page
Try adding a title
Toggle to translation, see that title isn't available

Details

Browser: Chrome
Operating system: Mac
Environment: Staging
Language: French (editing), French and English interface
Viewport: Desktop

Expected behaviour
Ability to edit immediately in EN/FR in Edit/Translations modes when new page is added.
Screencapture
Private Zenhub Video

Is this referring to the translation tab / page?
Tested and seem I can edit from the "questions" editor
https://github.com/user-attachments/assets/cd3a2101-01d5-4dee-bea2-3458d118139a

Can't seem to add title when on English interface, editing in French?

@anikbrazeau can you try re-testing? I was able to replicate the issue on Friday but now I can't seem to be able to 🤔

@Abi-Nada was able to replicate just now under these conditions:

Starting a new form and creating a new page, in French interface, editing in French from get go

I was also able to duplicate the bug locally
https://github.com/user-attachments/assets/24714c73-412f-431c-be46-e0cd74ff25fe

The bug is related to the language not being set when the title is update. The code defaults to english so only the english title is ever updated. Slowly digging into this bug.

@anikbrazeau when you have a minute, could you try testing a fix for this on environment:
https://ucjrj3npfslcehmz5iyzdepps40ftwlz.lambda-url.ca-central-1.on.aws/


AAD SSO integration with Google Cloud - Logout URL missing

Hello
About heading "Configure Azure AD SSO" -> Step 8 -> Logout URL
For my point of view the logout url "https://login.microsoftonline.com/common/wsfederation?wa=wsignout1.0" is missing.
Would be great if you could have a look at it.
Kind regards
Pascal

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

ID: 3fdf59c4-653e-956e-d675-f7e17ed7d488
Version Independent ID: 57b9c048-a938-6e21-633b-e0aabdd59946
Content: Tutorial: Azure Active Directory single sign-on (SSO) integration with Google Cloud (G Suite) Connector
Content Source: articles/active-directory/saas-apps/google-apps-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@kathodion
Thanks for your feedback! We will investigate and update as appropriate.

@kathodion
Thank you for pointing this out! I've created a PR to update the documentation and it's currently waiting for the author to review and sign off.
I'll go ahead and assign this issue to the author. Please allow some time for the author to review and the changes to reflect.
Thank you for your time and patience!

@JamesTran-MSFT I have sign off on the PR and it should get merged now. Please close this thread once you see the PR is merged.
@kathodion Thanks for your input here and we are updating the doc now.


Move zero backends to DI itself

Idea

In DI

test correctness for backends from primitive (correctness of default differentiation)
test type stability for zero backends (type stability of default differentiation)

In DIT

test correctness for ForwardDiff (correctness of scenarios + testing)
test type stability for zero backends (testing)

DI source

Add zero backends (AutoZeroForward and AutoZeroReverse) alongside backends from primitive

DI tests

Add type stability tests for zero backends
Remove type stability tests for backends from primitive

DIT source

Remove zero backends

DIT tests

Remove some correctness tests for zero backends

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

Project coverage is 0.09%. Comparing base (7c60378) to head (9b0207a).

:exclamation:  There is a different number of reports uploaded between BASE (7c60378) and HEAD (9b0207a). Click for more details.
HEAD has 50 uploads less than BASE

Flag
BASE (7c60378)
HEAD (9b0207a)

DI
41
0

DIT
10
1

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #437       +/-   ##
==========================================
- Coverage   93.63%   0.09%   -93.54%
==========================================
  Files         102      19       -83
  Lines        4885    2058     -2827
==========================================
- Hits         4574       2     -4572
- Misses        311    2056     +1745

Flag
Coverage Δ

DI
?

DIT
0.09% <ø> (-86.24%)
:arrow_down:

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

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


fix typo

Hi,
overlooked this one while blindly c&p the example.
regards,
Roman

This depends on whether you use secret or the plural secrets (both work) in RESOURCETYPES. Since RESOURCETYPES doesn't include any secret objects by default you could also fix this by adjusting your environment variable.


Make SkeletonIk3D node usable

Fixed an issue where "Play IK" would stop playing IK in editor when changing from one node to another. This issue made basically SkeletonIK3D node useless since you couldn't preview how it looked like or you could but it would have involved going back and forth between ik and target node.
Now you can:

preview ik without going back and forth between ik and target node.
turn on multiple SkeletonIK3D nodes.

https://github.com/godotengine/godot/assets/33091666/a59c4c8c-587c-4f03-b4e3-d6268c3a0b39
Fixes: not sure if there's an issue reported, but I was so annoyed by this when making my own game

Looks like this would possibly fix https://github.com/godotengine/godot/issues/72945?

Sure, didn't even realize to test that use case 👍

@SaracenOne now when the node exits tree, it will stop ik playback. Undoing the action won't turn it back on though, not sure if it is necessary to even track.

For future reference, please use more descriptive commit messages and PR titles.

Thanks!

Yeah sorry, the title and description was written a bit out of frustration

Cherry-picked for 4.1.3.


es6 lessons control + enter bug

Challenge Name

http://beta.freecodecamp.com/en/challenges/es6/use-class-syntax-to-define-a-constructor-function
Issue Description

After completing one of the tasks the modal appeared saying I was successful. However, the next button did not work. So I used control + enter which allowed me to continue to the next lesson. From the next lesson without doing anything else you can use control + enter to successfully complete the assignment then skip to the next lesson.

Browser Name, Version: chrome
Operating System: mac
Mobile, Desktop, or Tablet: desktop

I can confirm. It is not only this mentioned challenge. I tried submit button in more beta challenges and it doesn't seem to work.

Browser Name, Version: chrome
Operating System: windows
Mobile, Desktop, or Tablet: desktop

Confirmed as well. Open for takers.

Looking into it.

@dhcodes Can I take up this issue ?

@ajomadlabs looks like @adityaparab is already working on it. You can find other issues by filtering for the help wanted tag. Many are still open or if they are older, then you can probably assume the person who claimed it is not still working on it.

@adityaparab Please update. How far have your work reached?

@vivek-agrawal27 : Take a look at PR above your comment, :)
It is fixed but it needs to be updated as per review comment I received there. I'm on it. Please look into some other issue.

@adityaparab Yes I saw that :) I just wanted to know its latest stage.
btw Great work.


make install failling because antsMalfLabeling.sh is missing

Hi all,
Posting this here as I could not find any related issue.
./Scripts/antsMalfLabeling.sh is missing from master branch.
Consequently, make install fails during installation: script not found whereas listed in Examples/CmakeLists.txt (line 199).
Thanks,

https://github.com/stnava/ANTs/commit/f6e1f0bbc1fd08bef87e365677ce5aea1e801cc1

@ntustison Thanks !


TS server error: "Cannot read property 'modifierFlagsCache' of undefined"

The TS service stopped working, and I saw this output. I don't think those things are necessarily related though, because this 40 minutes after the 10:26 timestamp.
[Info  - 5:30:40 PM] Using tsserver from location: /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js
[Error - 10:26:22 AM] 'occurrences' request failed with error.
Error processing request. Cannot read property 'modifierFlagsCache' of undefined
TypeError: Cannot read property 'modifierFlagsCache' of undefined
    at getModifierFlags (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:7583:17)
    at Object.hasModifier (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:7579:17)
    at getExport (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62017:28)
    at Object.getImportOrExportSymbol (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62000:53)
    at getImportOrExportReferences (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62743:56)
    at getReferencesAtLocation (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62691:17)
    at getReferencesInContainer (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62653:21)
    at getReferencedSymbolsForSymbol (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62405:21)
    at Object.getReferencedSymbolsForNode (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62373:24)
    at Object.getReferenceEntriesForNode (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62197:58)
    at getSemanticDocumentHighlights (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:61135:57)

Thanks. Opened https://github.com/Microsoft/TypeScript/issues/15299 to track this upstream


Pie graph legend item color not coloring when using scriptable for backgroundColor

Hello.
After going through the changelog and stack overflow, I could not find anything related to the issue I am experiencing.
Expected Behavior
Using a function to dynamically return a background color for pie graph data points, should also color the legend items correctly.
Current Behaviour
Legend item's color remains default grey while the sections of the pie graph are colored properly.
Possible Solution
Have not had the chance to go through the source code. Since scriptable is new for pie graph, it's probably something that was missed.
Steps to Reproduce (for bugs)
This codepen shows the results of both types of backgroundColor implementations.
Context
Wanted to have the legend colors match the pie chart sections colors using a function.
Environment

Chart.js version: 2.8.0
Browser name and version: Safari 12.1.1
Link to your project: private (however the code pen is representative of what I am attempting to do)

Has been addressed in a later version.


⚠ Jenkins Plugins Artifactory has degraded performance

In 54d6d7e, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1341 ms

Resolved: Jenkins Plugins Artifactory performance has improved in f54f660 after 25 minutes.


doc: add badges

SSIA

Codecov Report
Patch and project coverage have no change.

Comparison is base (eec9a17) 92.40% compared to head (7163441) 92.40%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master       #1   +/-   ##
=======================================
  Coverage   92.40%   92.40%
=======================================
  Files           5        5
  Lines         474      474
  Branches       74       74
=======================================
  Hits          438      438
  Misses         35       35
  Partials        1        1

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


Add support for LightDismissing the renamer

Summary of the Pull Request
Huh, I guess I missed making the window renamer light-dismissable. This is a oneline fix for that.
Light dismissing is treated as a cancel, not as a commit.
References

Added in #9662

PR Checklist

[x] Closes #9718
[x] I work here
[n/a] Tests added/passed
[n/a] Requires documentation to be updated

Validation Steps Performed
This feels right.

@msftbot merge this in 2 minutes

ah, a conflict!

wait, what? something was conflicted..


Build iOS or Android

What's going on?
Trying to run yarn build-ios. First my error was 'index.js was not found'
I could not find index.js in project, but found App.js, i changed command line params and run yarn build-ios again. But got an error.
First error:
$ react-native bundle --entry-file index.js --platform ios --dev false --bundle-output ios/main.jsbundle --assets-dest ios
                 Welcome to React Native!
                Learn once, write anywhere

error The resource `/Users/tagirhadzhiev/Documents/react-native/PizzaApp/index.js` was not found. Run CLI with --verbose flag for more details.
Error: The resource `/Users/tagirhadzhiev/Documents/react-native/PizzaApp/index.js` was not found.
    at /Users/tagirhadzhiev/Documents/react-native/PizzaApp/node_modules/metro/src/IncrementalBundler.js:157:26
    at gotStat (fs.js:1665:21)
    at FSReqCallback.oncomplete (fs.js:158:21)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Second error:
yarn build-ios
yarn run v1.19.1
$ react-native bundle --entry-file App.js --platform ios --dev false --bundle-output ios/main.jsbundle --assets-dest ios
                 Welcome to React Native!
                Learn once, write anywhere

info Writing bundle output to:, ios/main.jsbundle
error ENOENT: no such file or directory, open 'ios/main.jsbundle'. Run CLI with --verbose flag for more details.
Error: ENOENT: no such file or directory, open 'ios/main.jsbundle'
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Steps to reproduce

npx ignite-cli new PizzaApp
yarn build-ios

App.js from 7.5.0 version

ignite-cli doctor results:
System
  platform           darwin
  arch               x64
  cpu                8 cores      Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
  directory          PizzaApp     /Users/tagirhadzhiev/Documents/react-native/PizzaApp

JavaScript (and globally-installed packages)
  node                 12.14.1      /usr/local/bin/node
  npm                  6.13.4       /usr/local/bin/npm
    expo-cli           4.12.0
    npm                6.13.4
    react-native-cli   2.0.1
  yarn                 1.19.1       /usr/local/bin/yarn
    gatsby-cli         2.10.4

Ignite
  ignite-cli         7.5.0        /Users/tagirhadzhiev/.npm/_npx/77336/bin/ignite
  ignite src         build        /Users/tagirhadzhiev/.npm/_npx/77336/lib/node_modules/ignite-cli/build
/Users/tagirhadzhiev/.npm/_npx/77336/lib/node_modules/ignite-cli/node_modules/gluegun/build/index.js:13
    throw up;
    ^

Error: Command failed: java -version 2>&1

    at ChildProcess.exithandler (child_process.js:295:12)
    at ChildProcess.emit (events.js:223:5)
    at maybeClose (internal/child_process.js:1021:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:283:5) {
  killed: false,
  code: 1,
  signal: null,
  cmd: 'java -version 2>&1',
  stdout: 'The operation couldn’t be completed. Unable to locate a Java Runtime.\n' +
    'Please visit http://www.java.com for information on installing Java.\n' +
    '\n',
  stderr: ''
}

@Aspedm before we delve deeper into this issue, which appears to be an issue with Ignite being unable to complete the creation of a new app due to the inability to find a Java runtime, I recommend going through the Installing React Native on Mac OS course that covers the steps for setting up the Java environment.
This tutorial is 100% free of charge! Infinite Red offers it as a way to help the React Native community get set up with a working development environment and prepared to use Ignite.
Please go through the tutorial, which will help you set up the required Java runtime, and post the results of that process as well as whether you were able to create a new React Native app using Ignite.

npx ignite-cli new PizzaApp --expo does not create android and iOS folder. Is this the expected?


Switch versioning to a different method

Using this method should make updating for linux easier to setup as the verisons will be consistent on builds between Windows and Linux, unless they somehow are build just before and after midnight.
Example version is now 1.0.8837.39476
So we now have to manuall update the first 3 values but that shouldn't be too much effort to handle

Hmm nope doesn't seem to work as good as I hoped 😢


Update post table format version checks in post profile

Closes #3635

Downgrades TT and CFF2 post format 3 FAIL to a WARN
Keeps mandatory presence of OTF CFF post format 3 with FAIL status
Adds post format 2.5 check as FAIL in all build formats
Adds post format 4 check as FAIL in all build formats

TODO

[x] replace CFF format (only) check for post format 3 (https://github.com/googlefonts/fontbakery/pull/3637/commits/ec7a514440a10397706440df474cd35230adfa6e)
[ ] update changelog if this is accepted

Thanks for having a look and merging this Felipe!


5th argument

I want to skip directories started at ".DS_Store". But, the app does not skipped directories started with .DS inside directory.
Example:
./.DS_Store
./Editor/Test.cs
./Plugins/.DS_Store
...
Build:
mono UnityPacker.exe . "Name" no "Assets/" "gitignore,md" ".git,.DS"
Exported:
./Editor/Test.cs
./Plugins/.DS_Store
...

You are right, the directory omission stuff isn't recursive. I'll look into it.


Stops playing after a while in background (android)

It works nice but when streaming in background it stops after some minutes.
Appreciate any help I can get

@sweapp Did you fix it?

@sweapp Did you fix it?

No, after searching for a solution in weeks without any result, I wrote an native android app instead and it's working very nice.

i have the same question , can you help me? thanks

@13627491210 this is still can't work. You can use another library : react-native-track-player


[4656] Rename HPITT service and rake task

Context
Follow on work from https://github.com/DFE-Digital/register-trainee-teachers/pull/2741
I didn't want to rename the files in the same PR so that the diff was preserved for ease of review.
Changes proposed in this pull request
Rename the HPITT rake task and service now that they can be used for any SCITT.
Guidance to review
Decide whether you like the names, then 👍 or 👎 .
Important business

[ ] ~Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?~
[ ] ~Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?~

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 [S] Rename HPITT upload now it's used for general upload too


Fix valuelib_test.bal fail on master

Purpose
note $subject
fix build fail
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

The fix was included in https://github.com/ballerina-platform/ballerina-lang/pull/28506.


[WIP][SPARK-37211][INFRA] Added descriptions and an image to the guide for enabling GitHub Actions in notify_test_workflow.yml

What changes were proposed in this pull request?

Descriptions and an image are added to the guide for enabling GitHub Actions in notify_test_workflow.yml.
Why are the changes needed?

For first-time contributors to know how to enable workflow on their own repositories quicker
Does this PR introduce any user-facing change?

No
How was this patch tested?

To be tested.

do you mind showing the example of the comment?

Test build #144919 has started for PR 34487 at commit c6e030f.

@HyukjinKwon Thanks for the review. I'm currently trying to run the edited workflow in my repo to see if my changes work fine. I'll put the image of the result in How was this patch tested? section later.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49390/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49390/

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

Test build #144924 has started for PR 34487 at commit e6811da.

Test build #144919 has finished for PR 34487 at commit c6e030f.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49395/

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

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49395/

Test build #144928 has started for PR 34487 at commit 2e5985b.

Test build #144928 has finished for PR 34487 at commit 2e5985b.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

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

Test build #144924 has finished for PR 34487 at commit e6811da.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49399/

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

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49399/

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

Merged to master.


Fix #337 - Array/Matrix environments do not trim newlines

This is an attempt to fix #337. With all my testing it appears to work as shown in the images below:
tex
  \begin{bmatrix}
    a & b \\
    c & d \\
  \end{bmatrix}

You should trim at most the last newline. This agrees with LaTeX:
  \begin{bmatrix}
   \\ a & b \\ \\
    c & d \\ \\
  \end{bmatrix}

Though there is one question that I was hoping to clear up. I wasn't able to trigger rowGaps to be anything other than null. I'm not sure how they are supposed to be handled in the case for deleting a row.
I also wasn't exactly sure how I should go about using the screenshooter. I was able to get the docker to run. Am I supposed to run some kind of image diff on the results with what you have on github?
EDIT: Sorry for the duplicate pull request.  I wanted to work on something else so I needed to put these changes into a branch.

@cbreeden thanks for the pull request.  I'll have a look at it tomorrow.
Please sign the CLA, found at found at www.khanacademy.org/r/cla, so we can merge this in.

To use the screenshotter, you'll need to either add a new entry in https://github.com/Khan/KaTeX/blob/master/test/screenshotter/ss_data.yaml or modify an existing entry.  I think it makes sense to create a new one and label it appropriately so that people know that the test is looking for a particular behavior.  After running make screenshots it should generate two new image files which should be added to this pull request.

@kevinbarabash Thanks! I filled out the CLA.  Can we put a hold on this?  I have been spending some time trying to write up a proposal AST which would take out a lot of the TeX logic from the BuildHTML with a few goals:

Setup a foundation for pluggable renderers
Provide a starting point for implementing macros (at least have a working sketch of where they would fit)
Auditing compliance relevant TeX rules found in TeXbook.

I started working on implementing <math atom> commands like: \mathop, \mathrel, etc.. In hopes to be able to implement commands like \stackrel, and I was able to for the most part except that \mathop{=}\limits^{?} wouldn't work primarily due to TeX logic living in BuildHTML.js.  Certainly I could hack it out, but there were other issues involved such as implementing \mathrel commands would essentially require a new mathrelgroup, just like there is a mathordgroup and so on to allow recursive expansion.
The only reason I ask for the hold is that I believe I am almost complete with the proposal.  I just need to finish a two more tasks:

[ ] Review placement for user macro expansions and "hard-coded" expansions (ie: matrix)
[ ] Check to make sure all of the current "dirty-tricks" would translate to the new proposal.

@cbreeden looking forward to your proposal.

Though there is one question that I was hoping to clear up. I wasn't able to trigger rowGaps to be anything other than null. I'm not sure how they are supposed to be handled in the case for deleting a row.

The following works:
\begin{pmatrix}
  a & c \\[1em]
  b & d\\[2em]
  c & e
\end{pmatrix}

I see... extra \\ add spacing after lines (or before if there is one before), but we should be rendering an empty line after the extra space.  @cbreeden It's probably worth updating the matrix screenshot test to include an example that tests this behavior.  Could you ad d a test case to https://github.com/Khan/KaTeX/blob/master/test/screenshotter/ss_data.yaml?  Then run make screenshots to regenerate the screenshot files and commit the changed files.

@cbreeden friendly ping on updating/adding screenshots.

Sorry.  I'll make the screenshots when I get back next week.

@cbreeden This PR recently reached its 1-year anniversary!  It looks like this was almost done -- just needed screenshots.  Let me know if you want help with those.

I cherry picked the change on to master and it doesn't seem to have any effect.
  \begin{bmatrix}
   \\ a & b \\ \\
    c & d \\ \\
  \end{bmatrix}

stills has an extra space.
It turns out that lastRow is a little more complicated that it used to be:

It turns out that lastRow is a little more complicated that it used to be

Indeed.  There seems to be more nesting than I remember.  I needed to add a few more .values and that seemed to do the trick.  I had an issue getting docker working (this is a local issue, not an issue with KaTeX), would it be possible to get someone to do a run for me?

@cbreeden thanks for updating.  I can run the docker.  I'll send you a PR with the screenshots.

Ok not sure what those errors are about.  I'll try getting docker working on my machine again and take a look.

It's odd that it's only failing for Chrome.  I can see why it would change thought:
AccentsText: |
    \begin{array}{l}
    \text{\H{e}}\\
    \text{\H{X}}\\
    \end{array}

@cbreeden I think the easiest thing would be to change the AccentsText to:
AccentsText: |
    \begin{array}{l}
    \text{\H{e}}\\
    \text{\H{X}}
    \end{array}

@cbreeden I'll send you another PR with screenshots for AccentsText.

I was able to get the docker to work again! 🥇
The AccentText failure makes sense.  The HTML should be different, but it should look roughly the same here since we aren't using parameters.  But it would've made a difference if it were embedded in an HTML document for instance.
When I ran docker locally, I had a sporadic failure for
M       test/screenshotter/images/Unicode-chrome.png

The Unicode test was interesting:
Unicode: \begin{matrix}\text{ÀàÇçÉéÏïÖöÛû} \\ \text{БГДЖЗЙЛФЦШЫЮЯ} \\ \text{여보세요} \\ \text{私はバナナです} \end{matrix}

This PR shouldn't affect this test since it doesn't have a trailing empty line, but just to be sure I inspected the HTML output for both with and without this PR and they were identical: https://gist.github.com/cbreeden/72ca53e6089b9d39f2fcc54bffcda3a5

Chrome is a little finicky with these tests.  I wonder if the new headless API for chrome would help here.

I wonder if the new headless API for chrome would help here.

Something we might look into.  I think we'd still want to do it inside a docker though b/c there's lots of variables that affect the rendering.

@cbreeden thanks for PR.


Array filter/map predicate not type checking properly

TS 2.0.3
See the comments and last line in the code below for expected/actual.
{
    type Foo = { foo: true };
    const isFoo = (foo: Foo) => foo.foo;

    const xs = [{ foo: true }, undefined];

    isFoo(xs[1]) // good, errors
    const x = undefined as Foo | undefined;
    isFoo(x) // good, errors
    xs.filter(isFoo) // bad, didn't fail, expected filter arg to fail, should be Foo | undefined
}

This is expected. Consider
declare function f(x: number): any;
declare function g(x?: number): any;
declare function h(x: number | undefined): any;
let i = f;
i = g;
i = f;
i = h;
let k = f;
k = h;
k = g;

Also, see this section of the handbook. type-compatibility

In the example above, I would receive a type error at runtime. Is there any way to enforce this at compile time?

See https://github.com/Microsoft/TypeScript/wiki/FAQ#why-are-function-parameters-bivariant


Errors when running the "digital_ads_market" and "simple_market" sample environments

Expected Behavior

Running the files in Phantom/examples/digital_ads_market and Phantom/examples/simple_market should train the model with Phantom.
Current Behavior

Instead, there are bugs that arise.
When I run ./Phantom/scripts/phantom Phantom/examples/environments/digital_ads_market/digital_ads_market.py train, the error I get is:
Exception: Tried to initialise agent ADV_1's Supertype with default values but failed:
	__init__() missing 1 required positional argument: 'budget'

When I run either ./Phantom/scripts/phantom Phantom/examples/environments/simple_market/example_simple_market.py or ./Phantom/scripts/phantom Phantom/examples/environments/simple_market/example_info_leakage.py, the error I get is:
  File "Phantom/phantom/utils/samplers.py", line 132, in __init__
    assert high > low
AssertionError

Possible Solution

I'm not entirely sure what a possible solution might be, but it seems like there are some constructor inputs that need to be added/changed.
Steps to Reproduce (for bugs)

Download the repo and install the requirements.
From the file that the repo is in, run ./Phantom/scripts/phantom Phantom/examples/environments/digital_ads_market/digital_ads_market.py train.
Try the same with the other commands I mentioned above.

Context

I just downloaded the repo and wanted to get started. Running the "supply_chain" environment worked but the other two example environments did not.
Your Environment

Phantom version used: 2.0.0
Python version used: 3.9.6
Operating System and version (eg. Ubuntu 21.04): MacOS Ventura 13.6

Hi Lee,
Thanks for bringing this to our attention. We have fixed the issues in PR #114 and PR #115 and released a new version 2.1.0 with the changes.


 The "index refreshed" message keeps flashing on the screen with every change, is there a way to turn off informational messages?

Sure thing, it's been added and will be available in the next release :)

Wow! Thank you, that was so fast!  :-)


DEPS.xwalk: Re-roll chromium-crosswalk.

No new commits have been added, and there are no functional changes. The
only difference is that the "Cr-Commit-Position" line in the commit
message for commit "[Backport] Move recently added functions to
V8_USE_EXTERNAL_STARTUP_DATA" has been removed.
chromium-crosswalk commits that are not present upstream must never have
a "Cr-Commit-Position" line, otherwise src/build/util/lastchange.py
will use a commit hash that is not present in Chromium and
DevTools/remote debugging will fail to work as expected.
BUG=XWALK-5139
BUG=XWALK-6208

@mrunalk: fyi

lgtm


missing libxml2 for 2.7.0 error on macOS BigSur

We are using 2.7.0 to upload our test report. It works fine on Windows and Linux. However for macOS BigSur, we always see this failure. Seems it is caused by libxml2 could not be found. But I found even I installed libxlml2 using brew, it still has this problem. Could you help to take a look? Thanks!
    - name: upload-test-report
      if: always()
      uses<EMAIL_ADDRESS>
--log info
2023-05-19T19:02:54.1153590Z       clang -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -arch arm64 -arch x86_64 -g -DCYTHON_CLINE_IN_TRACEBACK=0 -Isrc -Isrc/lxml/includes -I/Library/Frameworks/Python.framework/Versions/3.11/include/python3.11 -c src/lxml/etree.c -o build/temp.macosx-10.9-universal2-cpython-311/src/lxml/etree.o -w -flat_namespace
2023-05-19T19:02:54.1253290Z       In file included from src/lxml/etree.c:96:
2023-05-19T19:02:54.1325510Z       /Library/Frameworks/Python.framework/Versions/3.11/include/python3.11/Python.h:23:12: fatal error: 'stdlib.h' file not found
2023-05-19T19:02:54.1425760Z       #  include <stdlib.h>
2023-05-19T19:02:54.1525970Z                  ^~~~~~~~~~
2023-05-19T19:02:54.1627810Z       1 error generated.
2023-05-19T19:02:54.1712770Z       Compile failed: command '/Library/Developer/CommandLineTools/usr/bin/clang' failed with exit code 1
2023-05-19T19:02:54.1814230Z       creating var
2023-05-19T19:02:54.1876910Z       creating var/folders
2023-05-19T19:02:54.1976620Z       creating var/folders/24
2023-05-19T19:02:54.2076380Z       creating var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn
2023-05-19T19:02:54.2178010Z       creating var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T
2023-05-19T19:02:54.2264560Z       cc -I/usr/include/libxml2 -c /var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/xmlXPathInitubb8a7_k.c -o var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/xmlXPathInitubb8a7_k.o
2023-05-19T19:02:54.2366550Z       /var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/xmlXPathInitubb8a7_k.c:1:10: fatal error: 'libxml/xpath.h' file not found
2023-05-19T19:02:54.2437770Z       #include "libxml/xpath.h"
2023-05-19T19:02:54.2539170Z                ^~~~~~~~~~~~~~~~
2023-05-19T19:02:54.2631400Z       1 error generated.
2023-05-19T19:02:54.2732770Z       *********************************************************************************
2023-05-19T19:02:54.2733210Z Failed to build lxml
2023-05-19T19:02:54.2830320Z       Could not find function xmlCheckVersion in library libxml2. Is libxml2 installed?
2023-05-19T19:02:54.2915900Z       Perhaps try: xcode-select --install
2023-05-19T19:02:54.3012070Z       *********************************************************************************

Which Python version are you using?

I am not sure. We are using this on github workflow. The host is provided by github. We set the OS to BigSur. We did not set or install python. Should be default one fro BigSur.

Are you using
runs-on: macos-11

The full log of the job would be useful to debug.
You could always upload the test result files and have a dependent job that runs-on: ubuntu-latest, which downloads the files and publishes the results. Running on ubuntu, you can use the non-composite action, which starts up much faster and does not install anything.
See the Example workflow YAML: https://github.com/EnricoMi/publish-unit-test-result-action#use-with-matrix-strategy

Building lxml on Github hosted macos BigSur runner works for me: https://github.com/EnricoMi/publish-unit-test-result-action/actions/runs/5066042781/jobs/9095336359#step:7:177

Please try latest release (2.10.0) of this action, it uses latest lxml. Please reopen if there is still an issue.


update kubectl "$" format

Fix #13222

/assign @DanyC97
will try to review it today @Liujingfang1 , thanks for the PR

@Liujingfang1 thank you for working on this PR. you mind resolving the conflict before i can take another look at ?

@DanyC97 I'll resolve the conflicts.

@Liujingfang1 thank you for making the changes. I'm not sure why those failures, i'll try to understand the reason behind it

@Liujingfang1 could you please resolve the conflicts.

/lgtm
/approve

@tengqm much thanks !


Add support for pager duty services

I would love to be able to manage pagerduty services using this tool. Are there any plans for this?

Hi @softprops thanks for the input. We'll take a look at adding that resource type. Are there any other 3rd party resource types you'd like to see (both PagerDuty and non-PagerDuty)?

For completeness I there might need to be a way to configure pd  cw integration
Here's a common usecase at my company when setting up a new service for completeness
in a service template...

setup your service infra as a cf/sam template ecs/lambda/ect.
set up monitoring with cloudwatch metrics
setup topic subscriptions for ok/error states

then separately...

go set up pager duty service for the service to notify oncall engineers grab an cloudwatch integration url

then come back and...

update the topic subscriptions in infra template with the pager duties cw integration url

the end result looking something like
  SnsTopic:
    Type: AWS::SNS::Topic
    Properties:
      # topic definition
      Subscription:
        - Protocol: https
          Endpoint: PAGER_DUTY_INTEGRATION_URL

  Alarm:
    Type: 'AWS::CloudWatch::Alarm'
    Properties:
      # alarm definition
      OKActions:
        - !Ref SnsTopic
      AlarmActions:
        - !Ref SnsTopic

What'd I'd love to be able to do is to set up the all of this in one infra template and not have to go manage pager duty service to get that integration url separately.
  SnsTopic:
    Type: AWS::SNS::Topic
    Properties:
      # topic definition
      Subscription:
        - Protocol: https
          Endpoint: !GetAtt MyPDService.IntegrationURL // or something similar

Thanks for the additional context, we'll explore what's possible with exposing the integration URL for reference in a template. Makes complete sense to want to automate this end-to-end.
Is PagerDuty the only third-party provider you're interested in? Any other ones that you'd like to see?

For us PagerDuty is how aws alarms reach our oncall engineers. We don't have need for others atm.

Excellent!

Also to be clear I’d love to provision of services themselves in my templates. They represent the logical extension of the application. Application foo has a foo pd service with an cw integration whose url I attach to a sns topic subscription attached to a topic associated with ok/error alarms so that when an oncall engineer gets pages in pd there’s a clear mental map for which application is impacted

We're planning to add 2 PagerDuty resources. One for services and one for integrations.

I also wanted to note I did something like this a while ago for a serverless framework plugin (which I no longer use)
one parameter I’d like to have for the cf is service escalation policy. These tend to be shared across services so it doesn’t really fit with the stack but would need to be a service property to make the service useful. The escalation policy attached to the service determines who gets notified
https://github.com/softprops/serverless-oncall/blob/18c35d1afe72e52d83c3636dd75e555226244261/src/index.ts#L92

Thanks for the input @softprops. There will be an EscalationPolicyId property in the Service resource. Is this what you're looking for?
Also the PR for the Service resource has been opened if you care to take a look :)
https://github.com/aws-ia/cloudformation-pagerduty-resource-providers/pull/43/files#diff-d13ae449bc38cff2ee4e8b2245ad2f3340eae2c9bedba6d2c966069b7b51b334

@LucasQChen this looks perfect!

@LucasQChen I saw the pr was approved. Do you know when this might be available for use?

Sounds great 👍

Just checking in to see how things are progressing

Great to hear and thanks!

Sorry to bother folks but I was wondering if there was an update here?

Unfortunately, not yet. Some other emerging work had to be prioritized, but this is still planned. I will circle back with the team to get a new ETA.

No worries. I know the feeling

Looks like this landed in #43

I’m not sure how publishing works. Does a merge mean this is available to the registry now?

I checked but could find it listed under https://us-east-1.console.aws.amazon.com/cloudformation/home?region=us-east-1#/registry/public-extensions?visibility=PUBLIC&type=RESOURCE&category=AWS_TYPES

The PR is merged, but publishing hasn't been done yet. The team started picking this back up (hence the merged the PR), but ran into some issues during the publishing process so they've been working to resolve.

Happy new years folks. Just checking in to see if there's an estimation when when these might be published and to show my support because I'm here to cheer you on

Hi @LucasQChen just checking in on how things are going.

Hey @softprops, our team is still working on this and plan to get this out. There are some test failures that are taking more time to resolve.

Great news! Both the PagerDuty Service and Integration resource are now published on the CloudFormation registry. Apologies for the delays on this and thank you everyone for your patience.

That's great new.
Sadly, I just lost the opportunity to take immediate advantage of feature as most of my company has just been laid off, but I'm going to keep this feature in mind at my next gig. This will definitely streamline the process of wiring up important monitoring notifications for essential services.


[SPARK-6337][Documentation, SQL]Spark 1.3 doc fixes

Can one of the admins verify this patch?

ok to test

LGTM pending Jenkins

Test build #28943 has started for   PR 5112 at commit 2c6aee6.

This patch merges cleanly.

Test build #28943 has finished for   PR 5112 at commit 2c6aee6.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

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

LGTM - @jkbradley I suppose we can merge this and ... keep SPARK-6337 open or is it worth just making a new JIRA should any other doc issues come up? (or just a follow-on PR?) meaning, can we resolve it once this is merged?

Resolving the JIRA sounds fine to me.


Add a language attribute for the User

Is your feature request related to a problem? Please describe.
I want to continue the Language issue in the Frontend and for this reason i need a language attribute from the user.
Describe the solution you'd like
A language attribute to recognize the selected user language ("String")
Additional context
Please do it fast! :)

Solved in https://github.com/KrbDevelopment/Articly-Backend/commit/91a9a25e55f4ca09fc438b2befb6d42bf6461c07


Update ahash 0.7.3->0.8.3 and remove some clippy warnings.

Merged: NB nightly build is still failing for super dooper trait related reasons with hashing.


Update nlu.md

added "yo"

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


Revisit keyword classification

The classification of keywords seems unclear at times.
For example:

pub is a modifier, and opaque type in my view is a two-word declaration keyword. But opaque gets the modifier formatting, while pub gets the declaration keyword formatting.

@jhckragh Could you do your debugging magic again? :)

Here's the semantic token response provided by the Flix language server:
{'id': '3', 'status': 'success', 'result': {'data': [0, 16, 2, 1, 0, 0, 5, 3, 10, 0]}}

This translates to:
[0:16] 'OT' (ENUM)
[0:21] 'Int' (TYPE)

So the classification of pub, opaque and type comes from the VSCode grammar file (lines 92 and 148). 🙂

If you'd like, I can update the grammar file according to the suggestions from @mlutze? 🙂

So whats the actual problem? That they belong to the wrong category or that the theme treats them weirdly or what?

They belong to the wrong category in the grammar file:

pub is set as a declaration keyword instead of a modifier.
opaque is set as a modifier.
type is a keyword by itself (a declaration keyword).

OK, so the solution is to make pub a modifier?

I suggest we make pub a modifier and remove type as keyword and replace it with type alias and opaque type (both as declaration keywords). What do you think? 🙂

I suggest we make pub a modifier and remove type as keyword and replace it with type alias and opaque type (both as declaration keywords). What do you think? 🙂

I think this is the right solution. We should be sure that we can support arbitrary whitespace between the keywords though.
While we're here, are there any other tokens that are miscategorized?

After looking at the grammar file, I don't see any other misclassifications.

Question: As you point out @mlutze, arbitrary whitespace (including comments) can occur between the words in type alias and opaque type. This can be achieved through the begin/end mechanism in the grammar file, by using e.g. "\\b(type)\\b" as the begin pattern and "\\b(alias)\\b" as the end pattern. The result can be seen here:

But with this approach we get the same highlighting effect as with strings: When you type in ", then any code that comes after the cursor will be miscolored until the matching " is typed in. (In our case it would be e.g. opaque having to be matched by type). The effect can be seen here:

Alternatively we could use a single-line pattern like "\\b(type[ \t]+alias)\\b", but then a comment placed in between the two words would cause the highlighting to fail.
Which approach do you guys prefer? Or do you have other ideas for tackling it? 🙂

I don't think we should allow more than just a space between type and alias.

Why can't both type and alias be two keywords?

Why can't both type and alias be two keywords?

That would indeed be the simplest solution here. 🙂 I guess the counterargument is that neither word is syntactically valid on its own; they must occur in combination.

Why can't both type and alias be two keywords?

That would indeed be the simplest solution here. 🙂 I guess the counterargument is that neither word is syntactically valid on its own; they must occur in combination.

Aah who cares. The compiler will check that. I would just make them keywords of the appropriate kind.

Fixed in master. Thanks @jhckragh !


Issues with new version (1.0.0)

Describe the bug
When installing the latest version (1.0.0) it seems to have problems connecting. It was working fine on 0.6.0.
To Reproduce
Steps to reproduce the behaviour:

Install v1.0.0 and configure with your credentials

Expected behaviour
Able to connect and provide sensor data
Version
What version of the integration and what version of Home Assistant are you running?
1.0.0
Debug log
Error 1:
Error while setting up hildebrandglow_dcc platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 225, in _async_refresh
    self.data = await self._async_update_data()
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 370, in _async_update_data
    return await tariff_data(self)
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 229, in tariff_data
    return tariff
UnboundLocalError: local variable 'tariff' referenced before assignment

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 85, in async_setup_entry
    await coordinator.async_config_entry_first_refresh()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 197, in async_config_entry_first_refresh
    raise ex
homeassistant.exceptions.ConfigEntryNotReady: local variable 'tariff' referenced before assignment

Error 2:
Unexpected error fetching tariff data: local variable 'tariff' referenced before assignment
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 225, in _async_refresh
    self.data = await self._async_update_data()
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 370, in _async_update_data
    return await tariff_data(self)
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 229, in tariff_data
    return tariff
UnboundLocalError: local variable 'tariff' referenced before assignment

Error 3:
Unexpected exception: local variable 't' referenced before assignment
Traceback (most recent call last):
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 202, in tariff_data
    tariff = await self.hass.async_add_executor_job(self.resource.get_tariff)
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.10/site-packages/glowmarkt/glowmarkt.py", line 56, in get_tariff
    return self.client.get_tariff(self.id)
  File "/usr/local/lib/python3.10/site-packages/glowmarkt/glowmarkt.py", line 408, in get_tariff
    return t
UnboundLocalError: local variable 't' referenced before assignment

Do you have tariff data available in the Bright app @gregoryduckworth?
If not, you should disable the rate and standing sensors to avoid unnecessarily polling the API for data that isn't there
If you do, then something else is going wrong. Can you try enabling debug logging (as detailed here: https://github.com/HandyHat/ha-hildebrandglow-dcc#debugging) and hopefully that will produce more meaningful information?
I will look into making this error message more helpful :)

Good shout, the bright App has never got my electricity tariff but it does have a gas tariff.
However I’m unable to disable it as the integration is coming back with no entities:

However I’m unable to disable it as the integration is coming back with no entities:

Hello, I have a similar problem with no devices/entities, here's my debug log
2023-01-02 12:38:40.725 DEBUG (MainThread) [custom_components.hildebrandglow_dcc] Successful Post to https://api.glowmarkt.com/api/v0-1/auth
2023-01-02 12:38:40.828 DEBUG (MainThread) [custom_components.hildebrandglow_dcc.sensor] Successful GET to https://api.glowmarkt.com/api/v0-1/virtualentity
2023-01-02 12:38:40.912 DEBUG (MainThread) [custom_components.hildebrandglow_dcc.sensor] Successful GET to https://api.glowmarkt.com/api/v0-1/virtualentity/c80267e8-576f-40f4-adae-dc183fd6d70e/resources
2023-01-02 12:38:41.201 ERROR (MainThread) [custom_components.hildebrandglow_dcc.sensor] Unexpected exception: local variable 't' referenced before assignment
Traceback (most recent call last):
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 202, in tariff_data
    tariff = await self.hass.async_add_executor_job(self.resource.get_tariff)
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.10/site-packages/glowmarkt/glowmarkt.py", line 56, in get_tariff
    return self.client.get_tariff(self.id)
  File "/usr/local/lib/python3.10/site-packages/glowmarkt/glowmarkt.py", line 408, in get_tariff
    return t
UnboundLocalError: local variable 't' referenced before assignment
2023-01-02 12:38:41.209 DEBUG (MainThread) [custom_components.hildebrandglow_dcc.sensor] Successful GET to https://api.glowmarkt.com/api/v0-1/resource/6d1202db-98d4-46fb-b8b8-3ca1fc25bf74/tariff
2023-01-02 12:38:41.209 ERROR (MainThread) [custom_components.hildebrandglow_dcc.sensor] Unexpected error fetching tariff data: local variable 'tariff' referenced before assignment
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 225, in _async_refresh
    self.data = await self._async_update_data()
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 370, in _async_update_data
    return await tariff_data(self)
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 229, in tariff_data
    return tariff
UnboundLocalError: local variable 'tariff' referenced before assignment
2023-01-02 12:38:41.210 DEBUG (MainThread) [custom_components.hildebrandglow_dcc.sensor] Finished fetching tariff data in 0.298 seconds (success: False)
2023-01-02 12:38:41.226 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up hildebrandglow_dcc platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 225, in _async_refresh
    self.data = await self._async_update_data()
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 370, in _async_update_data
    return await tariff_data(self)
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 229, in tariff_data
    return tariff
UnboundLocalError: local variable 'tariff' referenced before assignment

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 85, in async_setup_entry
    await coordinator.async_config_entry_first_refresh()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 197, in async_config_entry_first_refresh
    raise ex
homeassistant.exceptions.ConfigEntryNotReady: local variable 'tariff' referenced before assignment

hi, just to say i have the exact same issue after updating to 1.0.0, i've rolled back to 0.6.0 for now.

Hey all, this should hopefully be fixed in v1.0.1. Please try it out and let me know!

Thanks for you swift response but unfortunately I still don't have devices/entities with v1.01
2023-01-02 14:40:29.216 DEBUG (MainThread) [custom_components.hildebrandglow_dcc] Successful Post to https://api.glowmarkt.com/api/v0-1/auth
2023-01-02 14:40:29.548 DEBUG (MainThread) [custom_components.hildebrandglow_dcc.sensor] Successful GET to https://api.glowmarkt.com/api/v0-1/virtualentity
2023-01-02 14:40:29.907 DEBUG (MainThread) [custom_components.hildebrandglow_dcc.sensor] Successful GET to https://api.glowmarkt.com/api/v0-1/virtualentity/c80267e8-576f-40f4-adae-dc183fd6d70e/resources
2023-01-02 14:40:30.446 ERROR (MainThread) [custom_components.hildebrandglow_dcc.sensor] Unexpected exception: local variable 't' referenced before assignment. Please open an issue.
Traceback (most recent call last):
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 214, in tariff_data
    tariff = await self.hass.async_add_executor_job(self.resource.get_tariff)
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.10/site-packages/glowmarkt/glowmarkt.py", line 56, in get_tariff
    return self.client.get_tariff(self.id)
  File "/usr/local/lib/python3.10/site-packages/glowmarkt/glowmarkt.py", line 408, in get_tariff
    return t
UnboundLocalError: local variable 't' referenced before assignment
2023-01-02 14:40:30.451 DEBUG (MainThread) [custom_components.hildebrandglow_dcc.sensor] Successful GET to https://api.glowmarkt.com/api/v0-1/resource/6d1202db-98d4-46fb-b8b8-3ca1fc25bf74/tariff
2023-01-02 14:40:30.451 ERROR (MainThread) [custom_components.hildebrandglow_dcc.sensor] Unexpected error fetching tariff data: local variable 'tariff' referenced before assignment
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 225, in _async_refresh
    self.data = await self._async_update_data()
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 394, in _async_update_data
    return await tariff_data(self)
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 243, in tariff_data
    if tariff:
UnboundLocalError: local variable 'tariff' referenced before assignment
2023-01-02 14:40:30.455 DEBUG (MainThread) [custom_components.hildebrandglow_dcc.sensor] Finished fetching tariff data in 0.548 seconds (success: False)
2023-01-02 14:40:30.476 WARNING (Thread-2) [websocket] websocket connected
2023-01-02 14:40:30.505 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up hildebrandglow_dcc platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 225, in _async_refresh
    self.data = await self._async_update_data()
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 394, in _async_update_data
    return await tariff_data(self)
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 243, in tariff_data
    if tariff:
UnboundLocalError: local variable 'tariff' referenced before assignment

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/hildebrandglow_dcc/sensor.py", line 85, in async_setup_entry
    await coordinator.async_config_entry_first_refresh()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 197, in async_config_entry_first_refresh
    raise ex
homeassistant.exceptions.ConfigEntryNotReady: local variable 'tariff' referenced before assignment

Ah, it looks like this is actually an issue with the upstream library. I will try to work around it

Hey all, please try v1.0.2 which should actually fix this :)

Reinstalled it just now and things are looking better:

Will keep you posted.

Thought I'd come back and say it all looks good and the improved messaging around tariffs is also a great addition.


Make pip install great again (+fix, -bloat)

Fixed ModuleNotFoundError issue

Removed unused dependencies on setup.py:
This reduced installation time considerably. Regarding the  removed dependencies:

jupyter: Probably needed for launching Jupyter notebook demos, but they aren't included in the pip package, so probably not needed at all.
cython: Probably a dependency for compiling spams? I don't have it in my pip environment but I can run the demos without a problem.

With these changes I can successfully run the Jupyter notebook demos. (Copied the code to Python console manually.)

Thanks for the help. I've added the init file I missed now but I think I'll leave the dependencies so that things will go smoother for those downloading the demo notebooks to run manually

Cool, thanks! can't wait to install this from pypi again!


investigate opportinity to use golang for s3 communication

compare python and golang for s3 communiction

golang uses too many depencies for such a small task


"UnsupportedOperationException" during upload Android file

Environment
-- Java -- :
openjdk 12 2019-03-19
OpenJDK Runtime Environment (build 12+33)
OpenJDK 64-Bit Server VM (build 12+33, mixed mode, sharing)
-- OS -- :
System Version: macOS 10.14.3 (18D109)
Kernel Version: Darwin 18.2.0
-- CLI -- :
crowdin: stable 2.0.28
Reproduce
$ crowdin upload sources --config crowdin.yml
WARNING: sun.reflect.Reflection.getCallerClass is not supported. This will impact performance.
Exception in thread "main" java.lang.ExceptionInInitializerError
       at com.crowdin.cli.commands.Commands.initCli(Commands.java:858)
       at com.crowdin.cli.commands.Commands.initialize(Commands.java:128)
       at com.crowdin.cli.commands.Commands.run(Commands.java:232)
       at com.crowdin.cli.Cli.main(Cli.java:17)
Caused by: java.lang.UnsupportedOperationException: No class provided, and an appropriate one cannot be found.
       at org.apache.logging.log4j.LogManager.callerClass(LogManager.java:555)
       at org.apache.logging.log4j.LogManager.getLogger(LogManager.java:580)
       at org.apache.logging.log4j.LogManager.getLogger(LogManager.java:567)
       at com.crowdin.client.CrowdinApiClientImpl.<clinit>(CrowdinApiClientImpl.java:22)
       ... 4 more

Update
It's seems to be issue of compatibility with my java version. I got it working with
java version "1.8.0_202"
Java(TM) SE Runtime Environment (build 1.8.0_202-b08)
Java HotSpot(TM) 64-Bit Server VM (build 25.202-b08, mixed mode)

As I prefer to use the latest SDK system wide, i'm using jenv to set the java version inside the folder where I use crowdin.

Could it be related to the log4j update on  https://github.com/crowdin/crowdin-cli-2/pull/106/commits/423c2301a1e070694cbda473105210a78a76362f ?  it's the change of v.2.0.28. To confirm I will try to g back t v2.0.27

Hi there @anis-campos !
Our developers are aware of this kind of issue and already working to fix it in the nearest!
Really glad to hear that you've found a workaround for the time being.
We'll let you know as soon as everything is ready.

Hi @anis-campos !
We've released a new version (2.0.29) of our CLI tool. Now it should work just fine with the latest Java 11. Kindly try to update CLI on your end and let me know whether it works fine now

Hi @rohalskyy
Thanks for the work. Here what the result of my testing
Test protocole

set java version with jenv global java_version_here
call java -jar crowdin-cli.jar upload for each release

Results
v2.0.29

[X] java 1.8.0_202 ( java for mac)
[X] java 11.0.2+9 (openjdk)
[X] java 12+33 (openjdk)

v2.0.28

[X] java 1.8.0_202 ( java for mac)
[ ] java 11.0.2+9 (openjdk)
[ ] java 12+33 (openjdk)

v2.0.27

[X] java 1.8.0_202 ( java for mac)
[X] java 11.0.2+9 (openjdk)
[X] java 12+33 (openjdk)

Everything looks fine. Thanks a lot !

This is great news! Thanks for the confirmation!


[BUG] Dupe Glitch

What is the bug?
Players are able to macro /team deposit and /team withdraw on the same key making them run simultaneously. This causes the server to lag and duplicate the money their using with this macro.
Expected Behaviour
Figure out a way to maybe add a wait timer to these commands. So players can't use them but ever "x amount of minutes"
(Just a thought maybe)
How to Reproduce

Bind /team deposit and /team withdraw to same key and set the amount for both.
Spam this key to cause lag and duplicate the money.

Error message
No response
Output of /teama version
4.6.4
Additional Information
This was not found by me, was directed to me before it occurred with my servers.

Can confirm this issue.


Create 0000-01-02-mgarrettdev.md

This is my first change to the project - This is a change on 2/8/2019

Here is my merge on 2/8/2019


Add Infrastructure Libraries to NPSP

This PR brings the following libraries into the NPSP
project: Apex Mocks, Apex Common, Apex Extensions.
The libraries are "vendored," meaning the repo has
its own copy of each library.  When new features are
introduced into the original libraries, an internal
"release" process will be used update the NPSP repo
version of each library, on a schedule determined
by the NPSP engineering teams.
Apex Mocks: https://github.com/apex-enterprise-patterns/fflib-apex-mocks
Run-time mock configuration.  Reduce lines of code needed
to configure various test configurations.  Empower engineers
to write more variations of tests. Enable easier unit-testing.
Dependency of Apex Common.
Apex Common: https://github.com/apex-enterprise-patterns/fflib-apex-common
Base Apex Enterprise Patterns library.  Empowers engineers to
use established patterns and base class methods to support
separation of concerns, consistent techniques for interacting
with platform APIs, querying, security, DML, etc.
Apex Extensions: https://github.com/wimvelzeboer/fflib-apex-extensions
Extensions package for Apex Common.  Includes many features
beyond Apex Common, including many additional domain
methods, dependency injection via configuration, etc.
Critical Changes
Changes
Issues Closed
Community Ideas Delivered
Features Intended for Future Release
Features for Elevate Customers
New Metadata
Deleted Metadata

Just two small naming notes!

Looks like there's small typo for the following directory force-app/infrastructure/apex-extensions/main/default/classes/triggerHanders -> ../triggerHandlers.
Pluralize the child directories of force-app/main to align with naming in the force-app/infrastructure.

Otherwise looks good to me as well! I appreciated the resources and vids in the design doc!

Thanks Prince! I updated the trigger handler folder name but left those in our main directory just to keep the diff smaller.  I don't have any issues with that change, just think it is minor and not necessary right now. Let me know if you feel differently. Thanks for the review!

@force2b Just wanted to ping you that we are close on this one.  Please do let us know if you have any outstanding concerns.  Thank you!


new routine ATTRIB_VAR_CONTEXT

Reported by ansley b manke on 30 Oct 2013 22:44 UTC
For the SHOW VAR/TREE (aka DEPENDENCIES) #2040
Background:
I'm pondering how best to include attribute values into the notion of variable dependencies.  For example in

    let a = var.long_name[   the variable tree should show that 'a' depends on var[d=1](d=1]

).  After that I have to deal with some issues that are peculiar to attributes -- like
    let a = ..history[   where the variable, a, depends on a dataset, rather than a variable.

What I need in order to do this in an orderly way is to get a context (a "cx" index, that is) that contains a description of what "var.long_name[d=1](d=1]
)" is.  In other words I'm wishing that I had a routine "ATTRIB_VAR_CONTEXT" that is analogous to these routines:
> ls -1 */*var_context*.F
ctx/get_aux_var_context.F
ctx/get_var_context.F
ctx/pseudo_var_context.F
ocn/cvar_context.F
stk/const_var_context.F
stk/svar_context.F

All of the logic of such a routine seems to be available inside of IS_ALGEBRA.F, where following line 750 the context "hi_cx" gets populated.   But the needed code is in-line inside of IS_ALGEBRA, rather than in the form of a subroutine that I can call.
Would it make sense for me to delay adding logic about attributes to SHOW VAR/TREE until you could create a routine ATTRIB_VAR_CONTEXT?  Your thoughts?
Note -- if ATTRIB_VAR_CONTEXT existed, it might also be reused in IS_UVAR_GRID.F to reduce the lines of text parsing found there.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2099

Modified by steven.c.hankin on 8 Nov 2013 16:53 UTC