Adds a new tenant resolver using request origin

Introduces a new tenant resolver using the request origin. This tenant resolver is lightweight and the perfect addition for an SPA that uses AJAX requests to a Laravel backend. By using the request origin for resolving, requests can be made without additional config on the frontend to include paths or extra request data.

I'll take a second look at this if more people request the feature. Closing for now as there doesn't seem to be interest in it. Thanks for the PR either way


help to run code

Hi
I am a beginner in the python language, when I run the code I get this error, knowing that the data has been added ( data {test[0 , 1] , train[0]} , I don't know if miss something
The code which I run : python train.py
And this is the output :

Thanks

You must keep your data in a sub-directory, under the name of the class
You have probably added your "0" class instances to "./data/train/", instead of "./data/train/0"
At least that's how I could reproduce your exact error... When creating a sub directory "0" as "./data/train/0" and moving all files into it I couldn't see any errors whatsoever

You must keep your data in a sub-directory, under the name of the class You have probably added your "0" class instances to "./data/train/", instead of "./data/train/0"
At least that's how I could reproduce your exact error... When creating a sub directory "0" as "./data/train/0" and moving all files into it I couldn't see any errors whatsoever

Hi eladmeir
This is project folder


Merged citations aren't sorted in increasing order

Description
At least when using the IEEE style, merged citations are not sorted in increasing order, unless that happens to be the first time they are cited. This means one might end up with something ugly like [48], [49], [35], [50] instead of the more compact [35], [48]-[50].
Reproduction URL
No response
Operating system
No response
Typst version

[X] I am using the latest version of Typst

Looks like sorting citations by citation number has not been implemented yet. The key code is in csl/sort.rs:
pub fn sort<T: EntryLike>(
        &self,
        cites: &mut [CitationItem<T>],
        sort: Option<&Sort>,
        term_locale: Option<&LocaleCode>,
    ) {
        if let Some(sort) = sort {
            cites.sort_by(|a, b| {
                let mut ordering = Ordering::Equal;
                for key in &sort.keys {
                    ordering = self.cmp_entries(a, 0, b, 0, key, term_locale);
                    if ordering != Ordering::Equal {
                        break;
                    }
                }
                ordering
            });
        }
    }

cmp_entries is fed 0's for the citation numbers, rather than collecting them from the items. Item sorting within a citation never appears to happen within BibliographyDriver::finish, nor could it using the current sort function. I imagine sort may need to be adjusted to wire the citation numbers in after they are generated with the citation_number closure.


Local Conditioning on F0 Working (Kind of)

Here is my branch:
https://github.com/ibab/tensorflow-wavenet/compare/master...dannybtran:local-conditioning
Here is the audio:
https://soundcloud.com/user-763760918/wavenet-localconditioning-on-f0-of-sin-waves
I'm not sure why there is so much static during the transitions. A couple of guesses are:

Not enough training data.
The training data was "pasted" together resulting in very unrealistic "crisp" boundaries between the sin waves.
Not enough steps. I only did 750
Label sample frequency is too low (it's on the order of 10hz (10 labels/sec)). Deep Voice mentions their phoneme frequency was 260hz.

Appreciate any feedback.

Couldn't you generate training data programmatically for something as simple as this? Maybe it's the fact that there are only a few transitions for it to be able to train on?
I think 750 isn't nearly enough to assess the code
10hz would result in an overlap of at least 1600 sample, relative to the upsampling convolution, so this can definetly be a cause of why the transitions sound weird. I think the net might be undecided on which note to play. More training examples might make it develop a "threshold"

That sounds pretty good. What parameters did you use? Regarding the noisy transitions, from our experiments, given enough speech training data, the model produces smooth transitions between vowels when tested on very sharp (artificial) F0 transitions.

I couldn't get your branch to work unfortunatly, I believe the correct way to perform LC is to condition per timestep for a receptive field. so sample at x-n * lc at x-n
Anyone have working LC?

@dannybtran Thanks so much for making this issue and explaining what you did. It inspired me to do the same experiment with my own wavenet implementation in pytorch.
Training data consisted of sequences of random frequencies, and conditioning was created by bucketing the frequencies, and passing into the model which used an embedding to convert them into a vector that was added to each layer before the activation. Code here, feedback welcome! https://colab.research.google.com/drive/1yCW66z7bsSECFp0lboL0Tznml9BsR02B.
It feels awesome to get local conditioning working, even with a toy problem of basic frequency sequences. I'm hoping to figure out how to map linguistic features to a conditioning vector for TTS next.
It would also be interesting to try using conditioning for other kinds of audio problems, like mapping midi -> wav. Midi files contain discrete events as well as alignment/duration info, and the MAESTRO dataset has pairs of recorded midi and wav files. I think it would be straightforward to use midi events as conditioning for a wavenet to synthesize wav.


Crash while running custom scenario

I built PacketRusher and was able to test free5GC successfully but while trying custom scenario (without making any modification to scenarios/sample/sample.go) it crashes. Following are the steps:

build custom scenario wasm using scenarios/build.sh
from $PACKET_RUSHER_HOME execute:
sudo ./packetrusher custom-scenario --scenario ./scenarios/sample.go.wasm

Following are the logs:
INFO[0000] Selecting <IP_ADDRESS> for host <IP_ADDRESS> as AMF's IP address
INFO[0000] Selecting <IP_ADDRESS> for host <IP_ADDRESS> as gNodeB's N3/Data IP address
INFO[0000] Selecting <IP_ADDRESS> for host <IP_ADDRESS> as gNodeB's N2/Control IP address
INFO[0000] Loaded config at: /home/user1/code/PacketRusher/config/config.yml
INFO[0000] [GNB] SCTP/NGAP service is running
INFO[0000] [GNB] Initiating NG Setup Request
INFO[0000] [GNB][SCTP] Receive message in 0 stream
INFO[0000] [GNB][NGAP] Receive NG Setup Response
INFO[0000] [GNB][AMF] AMF Name: AMF
INFO[0000] [GNB][AMF] State of AMF: Active
INFO[0000] [GNB][AMF] Capacity of AMF: 255
INFO[0000] [GNB][AMF] PLMNs Identities Supported by AMF -- mcc: 315 mnc:010
INFO[0000] [GNB][AMF] List of AMF slices Supported by AMF -- sst:01 sd:010203
INFO[0000] [GNB][AMF] List of AMF slices Supported by AMF -- sst:01 sd:112233
INFO[0001] [GNB] Received incoming connection from new UE
INFO[0001] [GNB] UE MCC: 315 MNC: 010
FATA[0001] func[env.pduSessionRequest] param[1] is unsupported: uint8
I even tried changing argument type of pduSessionRequest in scenarios/sample/sample.go

from func pduSessionRequest(uint32, uint32) to func pduSessionRequest(uint32, uint8)
for pduSessionId:=uint8(1); pduSessionId<2; pduSessionId++ {
fmt.Println("Hi, I'm an UE that wants to request PDU Session id: ", pduSessionId)
pduSessionRequest(ueId, pduSessionId)
}

but again same result.
I am not sure, if I am missing something here or is it a bug?

OS: Ubuntu 20.04 [5.15.0-92-generic]
Commit Id: 2bbcc24fa2ebf14335d6eec8737291f2205b91a1

Hi @himanshu3,
Thanks a lot for testing the custom scenario WebAssembly feature!
The project has evolved quite a lot since I've started working on this feature and I had not the time to work on it, it's very likely it has not kept with the latest changes.
While I fix this bug, I would recommend you to write your custom scenario by editing the PacketRusher's main scenario directly in PacketRusher's Go source here: https://github.com/HewlettPackard/PacketRusher/blob/32a08fa9fb2d83b654628b5187a0244a66b737b2/internal/common/tools/tools.go#L112
I hope it helps!
Cheers,
Valentin

Thanks for pointing to PacketRusher/internal/common/tools/tools.go


Documentation small typos

Coverage remained the same at 100.0% when pulling 269d0d95a8eb792a15bd35ae2f66b62242658ee6 on Ajimi:patch-1 into 78664b831c730b40ae56ea441acb88a114c4c347 on google:master.

Coverage remained the same at 100.0% when pulling d0c98ccfe480ae1c527f7b3d8c1f4380adb71ff8 on Ajimi:patch-1 into 78664b831c730b40ae56ea441acb88a114c4c347 on google:master.

Coverage remained the same at 100.0% when pulling d0c98ccfe480ae1c527f7b3d8c1f4380adb71ff8 on Ajimi:patch-1 into 78664b831c730b40ae56ea441acb88a114c4c347 on google:master.

Coverage remained the same at 100.0% when pulling 14dcb202a76e70b35d6b6c783341818294e5f496 on Ajimi:patch-1 into ae349ee6cda5da4e14faa0699a0c5b0c70350974 on google:master.


Update react-konva.d.ts

Missing Transformer in react-konva.d.ts. Mandatory if used with typescript

Thanks!

Thanks! I don't know how it works, but can you also update it in npm?
Thanks!

Hey @lavrton, Can you please look at npm update? thanks for this!

It is in npm.

Thanks!


Improvements for Removing Unused Oauth2 Tokens from the AM Database

Description:
We are currently removing unused Oauth2 tokens from the AM database using a script [1].
In this script for removing the unused tokens form the database we only delete token entries which have expired/inactive or revoked tokens keeping the latest entry for each combination of consumerKey + user + scopes. The reason for keeping the latest combination is to use refresh grant.
This improvement is to remove any existing invalidated token entries at the time of new token generation from the code level. This way, there wouldn't be a need to have a separate removal script. If the operation could be expensive for use cases with high token generation frequency, it can be provided as a switchable feature. This can be improved to remove tokens which have expired refresh tokens as well.
[1]https://docs.wso2.com/display/AM200/Removing+Unused+Tokens+from+the+Database#57d4816ce2d2427eb93f2bc3a2b2b074
Affected Product Version:
APIM2.0.0

This has been fixed in APIM 2.6.0 [1]
[1] https://docs.wso2.com/display/AM260/Removing+Unused+Tokens+from+the+Database


M2 MacBook keyboard, trackpad do not work when booting from USB

I have a boot stub and nixos rootfs on the USB. The bootstub chainloads to an EFI partition on the M2's internal NVMe. Booting works fine; however, from Uboot on the M2's trackpad and keyboard stop working and I have to use an external keyboard and mouse. Any ideas why?
With the understanding that expert mode is for experimental use, I would also like to point out that running the installer in expert mode when installing onto an external drive fails out here: https://github.com/AsahiLinux/asahi-installer/blob/e4d8bb3524fd004abac7468aac275f33d5c94c56/src/osinstall.py#L123

I'm sorry for asking, have you compile the kernel with the correct config (i.e. drivers for keyboard and trackpad)?

@angelbirth valid concern, but I do not think that is the issue. I have an additional boot stub located on the internal NVMe, that chainloads to the same EFI partition, which then uses the partition on my external drive containing the NixOS rootfs. So when I boot from this stub, it uses the same EFI and same rootfs as when I boot from the boot stub on the external drive. The strange thing with this internal boot stub is that the keyboard and trackpad work from Uboot on, including with NixOS.
In summary, the keyboard and trackpad work for the same EFI and  rootfs if you use a boot stub on the internal NVMe.

which macOS version(s) are used for the boot stubs? which m1n1, u-boot and kernel versions are in play? for u-boot, which exact git sha1? If u-boot is not asahi-v2023.07.02-2 and one of the stubs is macOS 13.5 based keyboard and trackpad will not work.

@jannau From what you pointed out, it is likely that I am using an outdated EFI. When using the Asahi Installer (expert mode) to put the boot stub on the drive, I chose the 13.5 macOS version. On the other hand, the boot stub installed with the regular installer put a 12.4 boot stub and EFI partitions on my internal NVMe. So it is likely that when I combine the 13.5 boot stub with the 12.4 EFI, things are not working.
So, how would you recommend correcting the EFI to be up to date? From the documentation , it would seem that I need to build the default branches of m1n1 and u-boot, and then concatenate them together. I assume I can leave out the device tree since I plan to use NixOS?  Then I figure I would replace the boot.bin on my mounted EFI with this custom m1n1.bin -- is this process correct or am I missing something?

Resolution: the issue was with https://github.com/AsahiLinux/PKGBUILDs/blob/main/uboot-asahi/PKGBUILD#L4 being set for an old version. Once I built U-boot locally following the documentation referenced above, and re-concatenated m1n1 with the dtbs and uboot, I was able to use the keyboard and trackpad successfully.


deploy: add api keys to environment

Add pikespeak and posthog api keys to environment variables accessible by the deploy script.

@frol Would you be able to add POSTHOG_API_KEY to secrets? So that it can be picked up here: https://github.com/NEAR-DevHub/neardevhub-treasury-dashboard/pull/56/files#diff-98111090fa7859bada9e04429085e258a309c153883e7d71f12df62827c00530R21


Replace ValidationError::schema with custom errors

Hi there, bump into here via hacktoberfest mark.
Honestly most likely it doesn't address JSONPointer correctly in errors.
And I was not sure which type of error better use for things like as64 error or regex syntax error but I did used ValidationError::format.
Note: there might be an issue with cargo fmt.
closes #235
Thank you for the library.

Thank you @zhiburt !
I am happy to see your contribution :) I'll try to review this in the next few days :) Could you, please:

Run cargo fmt
Remove the unused function
Add a changelog entry

Honestly most likely it doesn't address JSONPointer correctly in errors.

I think it is alright, in any case, it is an improvement over the status quo

And I was not sure which type of error better use for things like as64 error or regex syntax error but I did used

That's alright :)

During the compilation phase, schema_path means a path within the JSON Schema meta schemas

So its essentially pointers to this map?
https://github.com/Stranger6667/jsonschema-rs/blob/aaadd99b2cc376d9018759ecedbbe68c0700d2ab/jsonschema/src/compilation/options.rs#L28-L74
I am curious if we could reuse self.schema_path in structures like this as schema_path arg in error.
https://github.com/Stranger6667/jsonschema-rs/blob/fba763fd4fed01ae8073c6a0c9204cb8c7080933/jsonschema/src/keywords/pattern.rs#L19-L23

So its essentially pointers to this map?

Yep, I'd say that they point to some locations within the values of this map.

I am curious if we could reuse self.schema_path in structures like this as schema_path arg in error.

In the majority of the cases, these schema_path values are already used by the meta-schema validators. If the meta-schema is not capable to catch the bug, then these manual error-handling scenarios should catch the problem.
Of course, we may try to keep the meta-schema context manually and trace all the paths during the compilation. However, it will be hard to do as we'll need to do it differently for different JSON Schema drafts, so I assume it will be easier to tweak the meta-schema instead - it happened before and was caused by an actual bug in meta schemas.
Hope I helps :)

I am curious if we could reuse self.schema_path in structures like this as schema_path arg in error.

In the majority of the cases, these schema_path values are already used by the meta-schema validators. If the meta-schema is not capable to catch the bug, then these manual error-handling scenarios should catch the problem.

I guess I did make a typo.
I meant to say maybe we could use this self.schema_path in errors.
Like this
ValidationError::format(self.schema_path, context.clone().into_pointer(), ...)

You've spend quite a bit of time to review it.


Some typos in the Dark Squirrel text

Thank you for bringing this to my attention. I'll fix it for the next version of the plugin.


Stability fixes for yaml parsing and other config changes

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

Comparison is base (3bf466d) 78.50% compared to head (83cc390) 78.16%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1670      +/-   ##
==========================================
- Coverage   78.50%   78.16%   -0.34%
==========================================
  Files         281      282       +1
  Lines        7973     8175     +202
==========================================
+ Hits         6259     6390     +131
- Misses       1338     1394      +56
- Partials      376      391      +15

Files
Coverage Δ

pkg/iac-providers/cft/v1/load-file.go
83.67% <100.00%> (ø)

...r/iac-providers/cft/config/api-gateway-rest-api.go
100.00% <ø> (ø)

...pper/iac-providers/cft/config/api-gateway-stage.go
90.62% <100.00%> (+0.30%)
:arrow_up:

...pper/iac-providers/cft/config/api-gatewayv2-api.go
0.00% <ø> (ø)

...er/iac-providers/cft/config/api-gatewayv2-stage.go
100.00% <ø> (ø)

...iac-providers/cft/config/app-autoscaling-policy.go
100.00% <ø> (ø)

...kg/mapper/iac-providers/cft/config/appmesh-mesh.go
100.00% <ø> (ø)

...apper/iac-providers/cft/config/athena-workgroup.go
0.00% <ø> (ø)

...ers/cft/config/autoscaling-launch-configuration.go
87.09% <ø> (ø)

...kg/mapper/iac-providers/cft/config/backup-vault.go
100.00% <ø> (ø)

... and 75 more


Fix code health failure

Summary of the Pull Request
Fixes code health failure since 66b9b9d6f1b600d1b25aa2339fa368113c45952b

References

PR Checklist

[ ] Closes #xxx
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

@msftbot merge this in -1 minutes


Rocket Chat - no for support local RocketChat server

Hi,
In my company, we are about switch between Skype and RocketChat, which is stored locally and available from local network. While I'm trying to add Rocket.Chat plugin to Franz, it returns: "The URL you've entered is not valid". Is there any way to work it around?
Br,
Patrick

i get the same thing

Someone already share a workaround (I tested and it works) but I think that in any case we need a correction of that bug
https://github.com/RocketChat/Rocket.Chat/issues/6115

Also need a fix for that!

I have submitted a fix in Pull Request #231

+1 for getting that pull request in, I'd greatly appreciate being able to consolidate all my chat clients at work :)

+1


Missing parameters

Hi,
I'm running the script as laid out in the vignette and three of the flow parameters don't want to run.
I create the md object using:
file_name<-as.character(pData(fs)$name) sample_id<- as.character(pData(fs)$name) condition<- c("Tube1","Tube2","Tube1","Tube2","Tube1","Tube2","Tube1","Tube2","Tube1","Tube2","Tube1","Tube2") patient_id<- c("Donor_1","Donor_1","Donor_2","Donor_2","Donor_3","Donor_3","Donor_4","Donor_4","Donor_5","Donor_5","Donor_6","Donor_6") md<-data.frame(cbind(file_name, sample_id, condition, patient_id)) head(md)
I then make the panel using:
fcs_colname <-colnames(fs) antigen <- c("FSC-A","FSC-H","SSC-A","SSC-H",              "CD196", "CD25",               "CD16", "CCR10", "CD62L", "CD4",              "HLADR", "CD8", "CD45RA", "CD19", "CD95", "CD194", "CD3IgD",              "PD1", "CD161", "CD28", "CD57", "CD38", "va72TCR", "CD127", "CD27",              "gdTCR", "CD56", "CD185", "CD183", "CCR7",              "Time") marker_class <- c(rep("none",4),rep("type",26),"none") panel<- data.frame(cbind(fcs_colname, antigen, marker_class)) head(panel)
The output from head(panel) is:
       fcs_colname antigen marker_class 1             FSC-A   FSC-A         none 2             FSC-H   FSC-H         none 3             SSC-A   SSC-A         none 4             SSC-H   SSC-H         none 5      FJComp-APC-A   CD196         type 6 FJComp-APC-R700-A    CD25         type
The output of all(panel$fcs_colname %in% colnames(fs)) is TRUE
The sce is made with:
sce <- prepData(fs, panel, md, features = panel$fcs_colname,cofactor = 150)
sce@int_colData contains the FSC, SSC and the three missing parameters (CD196, gdTCR and CD56)
Any ideas what I am doing wrong?

Try running with FACS=TRUE - that should solve it.


[Docs]: Add a CONTRIBUTING.md.

I was trying to just make a small PR, and help this project, but there was no setup guide, nor any instructions on how the project works. Or just a 1-2-3 for running the development server.
That could definitely be helpful both for me and for any future people, wanting to contribute :D

Thanks for the suggestion! This would certainly help, I'll try to collect some information. I guess especially the internals could use some explaining, as vane has it's own config and resource pack generator which might be confusing at first.
Building should actually be as simple as just running ./gradlew build. I then manually copy the jar files to my testserver in the server start script. There is ./gradlew runServer, but as you apparently already noticed, this seems to have issues right now.


11960 Edit UI - Create new Nature of Business component for conversion filing

Issue #: /bcgov/entity#11960
Description of changes:

[x] Nature of business will have a text box instead of NAICS code lookup,
[x] Max 300 characters and validation.
[x] No validation for empty text field
[x] Shows "(Not Entered)" for empty text
[x] Review and Confirm shows the equivalent result.

Comment:

New ticket will be created to handle unexpected NAICS format with missing code for Edit UI (change filing) in the NatureOfBusiness component in bcrs shared components repository: /bcgov/entity#12310

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

e filed with and without a NAICS code/description

I was testing the API for conversion. Apparently, the backend doesn't accept data from conversion filing. There is 400 bad request to the /fee/SP/CONVERSION endpoint, and the post call to /businesses/FM1000018/filings endpoint is unauthorized with 401.

I was testing the API for conversion. Apparently, the backend doesn't accept data from conversion filing. There is 400 bad request to the /fee/SP/CONVERSION endpoint, and the post call to /businesses/FM1000018/filings endpoint is unauthorized with 401.

Yes, this is incomplete code that will be implemented in 11954 and 11889.

I was testing the API for conversion. Apparently, the backend doesn't accept data from conversion filing. There is 400 bad request to the /fee/SP/CONVERSION endpoint, and the post call to /businesses/FM1000018/filings endpoint is unauthorized with 401.

Yes, this is incomplete code that will be implemented in 11954 and 11889.

In that case, I think the frontend is good to go. The new naics data goes to naicsDescription.

I didn't get into this super deep but it looks good to me.
Did you verify that the change can be filed with and without a NAICS code/description?
Please add some Test Notes to your ticket to inform Riyaz what scenarios need testing (and that you have tested yourself, obviously).

Thanks. I left a comment in the ticket and mentioned Yui and Riyaz.


Show ui loading indicator while logging in

Today, when you press the login button - there is no UI feedback i.e. loading indicator that processes are firing in the background (pulling in txs, balance, price, etc), which looks like it is stuck on the UI.
A loading indicator or a toast similar to when you send a transaction that shows the login process is in progress will be helpful. Right now there is a couple of seconds delay to that process.

DAMN SON!

flow of consciousness


Update platforms

Description
Expand platform support and deprecate old platforms
Issues Resolved
#76
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Tests are still failing for other reasons, but this gives us working Fedora module support, probably working FreeBSD/Suse support, and tests that pass on newer Debian systems. It should probably get merged now

Oh it should also be noted that this gives us Amazon 2016 support and removes support for old Debian/Ubuntu systems that are EOL

Updating to note that this won't fix 16.04.  It's a start, but that work needs to happen elsewhere.  This will mostly clean old old platform support, fix Fedora, and add Suse


Crash when scrolling through workshop

Please use the below as a template when making your report.
Describe the bug
Was trying to figure out what having local leader does, and opened the workshop menu, scrolled through a few options and was looking in the "power" area and it just crashed.
Is it already a known issue?
Did not see one
Steps to reproduce
It crashed earlier when I was looking through the workshop menu but I have the "Scientist" perk instead of "Local Leader." There was a rad storm going on too.
crash-2024-02-24-08-47-54.log
Environment
In the crash report
Contact:
Discord: Starkmens

crash-starkmens-AUTOSCAN.md
Do you have anything overwriting the "workshop script"

Not that I am aware of, I haven't added anything.

This is probably due to some type of missing item. Is there a certain item you scroll over that causes this?

I scrolled over the power section both times. I didn't go into it though. Ill try and recreate with the latest update and let you know.

@Starkmens  Issue was related to place everywhere. Has been resolved in new update.


Add focus hook

Introduce a less manual way of capturing focus for keyboard events.

A manual enable hook + insertion in to an array sorted by temporal Z is the best option, also provision for focus trapping (for pop ups and what not)


Update Composer dependencies (2023-06-13-00-09)

Visual regression test passed!
View the visual regression test report


Cant install from npm due to canvas

I want to use this package badly because I'm having the same issue it tries to resolve but I cant download it because I cant install canvas using node 12. How are you able to install?

@CodeByAlex hi, what is the error message?

Hey @Ghustavh97 - this is the error I'm seeing:
npm ERR<EMAIL_ADDRESS>install: node-pre-gyp install --fallback-to-build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.

@CodeByAlex try npm i node-pre-gyp then reinstalling the package again.

Its unfortunately still giving errors:
Failed to execute<EMAIL_ADDRESS><EMAIL_ADDRESS>configure --fallback-to-build --module=...../node_modules/canvas/build/Release/canvas.node --module_name=canvas --module_path=...../Style/node_modules/canvas/build/Release --napi_version=5 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v72' (1)
Ive tried what you suggested, removing node modules, removing package lock, and installed canvas specifically. Nothing seems to work. Im assuming it works for you - what version of node/npm are you using? Do you have any other suggestions?

@CodeByAlex
My local machine:
OS: Ubuntu 18.04.4 LTS
node : v12.16.1
npm: 6.14.4
I have also added an osx test build and it seems to be passing. Here. its running:
OS: mac os 10.15.4
node:  v12.16.3
npm: 6.14.4
nvm  :0.35.3
Have you also tried installing node-pre-gyp globally? npm i node-pre-gyp -g

Some more suggestions:

Try yarn instead of npm
This
Could be a network issue

@CodeByAlex so I think I managed to reproduce your error
Failed to execute '/usr/local/bin/node /usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --module=/var/www/html/test-npm/node_modules/canvas/build/Release/canvas.node --module_name=canvas --module_path=/var/www/html/test-npm/node_modules/canvas/build/Release --napi_version=5 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v72' (1)
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.

This is after I tricked my network drivers by pointing them to localhost. im 99% sure its a network issue.
Would be nice if you could provide the full error log as well so that I could confirm this.

My apologies - this was a network issue. Thanks for looking into this


Mastering limiter boosting quiet sounds

(Based on #95, feel free to dupe. I didn't see a way to reopen #95.)
I hooked up the AUDIO_ENGINE_FLAGS::AudioEngine_UseMasteringLimiter. It is doing a good job of preventing clipping for loud sounds, but it also has an unexpected (and unwanted) effect: it seems to be boosting the volume of quiet sounds.
I have a weapon sound effect which starts loud and has a soft trail-off finish. When using the mastering limiter, the initial loud sound is the same volume as without the limiter (this sound doesn't clip even without the limiter), but the trail-off finish is louder. I expected the limiter to only make loud sounds quieter, not soft sounds louder.
I've noticed this with other sounds too. I have a consistently quiet wind sound which is boosted to be too loud when the limiter is on. I suspect something isn't configured properly, but I have no idea where to look. A Bing search hasn't given me any clues,--just people lamenting how little documentation there is about XAudio. :)
I'm using XAudio2_7.dll and XAPOFX1_5.dll.
I put together this a little demo project which demonstrates the issue: https://1drv.ms/u/s!AhmO0u3HJ7RHjoFUNArr6hFd9K_Epg
The demo runs the wind sound and a sci-fi weapon firing sound. It starts with the limiter off. Clicking the mouse toggles the limiter. When the limiter is on, the wind is louder and the firing sound gets a little quieter.

The first thing to try is to build using XAudio2_8 (or 2.9) and see if it repros.

I just tested them, and I get the same behavior with 2.8 and 2.9.

I'll pass this report along to the audio team, but it's not likely an issue in the DirectX Tool Kit itself.

Thanks, Chuck. If the issue is with DirectX, is there a public place where I can track the issue?


updated with 2023-09-01 CRP

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Compute


CON-4751 Added index in s_plugin_connect_items

Coverage decreased (-0.03%) to 34.427% when pulling f5f52799f26402ff76dc647716074406ffd81c61 on CON-4751 into 58f3c0eb770ded514cf851f97c6d2918e7c5cd88 on master.

@SimonBaeumer I changed CategoryExtractorTest.Removed fifth argument $this->db, because AutoCategoryResolver requires only 4 arguments.

Coverage decreased (-0.06%) to 34.397% when pulling 1e61c6cd11f4efaeaf1631caed54e9a43ca877de on CON-4751 into 58f3c0eb770ded514cf851f97c6d2918e7c5cd88 on master.

Coverage decreased (-0.06%) to 34.397% when pulling 52f7cea83ce2e2fa50ac35cf0d1378cc594277a2 on CON-4751 into 58f3c0eb770ded514cf851f97c6d2918e7c5cd88 on master.

Coverage decreased (-0.07%) to 34.388% when pulling 30e232cd5ef7f4ef393bddf0a964282db5271c6c on CON-4751 into 58f3c0eb770ded514cf851f97c6d2918e7c5cd88 on master.

Coverage decreased (-0.07%) to 34.388% when pulling 30e232cd5ef7f4ef393bddf0a964282db5271c6c on CON-4751 into 58f3c0eb770ded514cf851f97c6d2918e7c5cd88 on master.

Coverage decreased (-0.07%) to 34.388% when pulling 30e232cd5ef7f4ef393bddf0a964282db5271c6c on CON-4751 into 58f3c0eb770ded514cf851f97c6d2918e7c5cd88 on master.


Fix a bunch of files with pylint disabled

CATEGORY
Choose one

[ ] Bug Fix
[ ] Enhancement (new features, refinement)
[X] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

Re-enable pylint on a number of files with pylint globally disabled.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

No UI changes.
TEST PLAN

[ ] Run lint
[ ] Run tests

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@john-bodley

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@003e98c). Click here to learn what that means.
The diff coverage is 69.79%.

@@            Coverage Diff            @@
##             master    #8743   +/-   ##
=========================================
  Coverage          ?   65.85%
=========================================
  Files             ?      483
  Lines             ?    24123
  Branches          ?     2767
=========================================
  Hits              ?    15886
  Misses            ?     8061
  Partials          ?      176

Impacted Files
Coverage Δ

superset/exceptions.py
100% <ø> (ø)

superset/translations/utils.py
87.5% <0%> (ø)

superset/jinja_context.py
74.73% <100%> (ø)

superset/views/log/views.py
100% <100%> (ø)

superset/views/schedules.py
58.03% <40.9%> (ø)

superset/views/annotations.py
79.06% <50%> (ø)

superset/sql_lab.py
77.57% <68.75%> (ø)

superset/views/base.py
72.77% <86.36%> (ø)

superset/sql_parse.py
96.12% <89.47%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 003e98c...a84b8b0. Read the comment docs.


Setup machinery to build package and controller binary images

Adds initial setup to build and publish provider images. Does not yet include tf plugins in controller image.

@kasey should we go ahead and get this merged so we can have CI on subsequent branches where I enable building in the plugins to the controller image?

@kasey should we go ahead and get this merged so we can have CI on subsequent PRs where I enable building in the plugins to the controller image? I can setup some integration tests as well.

sgtm


Define Product model

Define a model in Flask named "Product" with 3 attributes: id (Integer, Primary Key), name (String), and expiration_date (Date).

Hint:

To define a model, create a class, name it Product, and pass "db.Model" into the parameter.
Use db.Column to create the 3 attributes.
Make sure to set 'id' as the primary key.
Ensure 'name' and 'expiration_date' cannot be null.
Ensure 'expiration_date' is typed Date.

Issue has been completed and can be closed


[BUG] The data in the "So Far Today:" section is not updated after coming the time of restarting the claim cycle

Description: The data in the "So Far Today:" section is not updated after coming the time of restarting the claim cycle.
Steps to reproduce:

Open https://wallet.gooddollar.org/
Go to the Claim page
Wait when the time of restarting the claim cycle is came
Pay attention to the data in the "So Far Today:" section

Expected result:
The data in the "So Far Today:" section is updated after coming the time of restarting the claim cycle
Actual result:
The data in the "So Far Today:" section is not updated after coming the time of restarting the claim cycle
The user is new: No
Environment:  Prod 1.13.3
Devices list: Desktop // Google Chrome 87.0.4280.88
Additional info:
Email<EMAIL_ADDRESS>pass: Tester_88888
Video: https://www.dropbox.com/s/v7i1yo0bcxxl4y5/The data in the So far today section is not update.mp4?dl=0

@iLystopad again does it update automatically after a few seconds or it doesnt update at all?
if it does update then it is ok only if it doesnt update at all then its a bug

@sirpy In that case it is updated after more than 20 min

@iLystopad you dont need to open the dashboard.
also if you are no "focused" in the wallet window the stats wont be updated.
the expected behaviour is that after you claim stay on the claim screen and the stats should be updated  within 10 seconds.

@iLystopad you dont need to open the dashboard.
also if you are no "focused" in the wallet window the stats wont be updated.
the expected behaviour is that after you claim stay on the claim screen and the stats should be updated  within 10 seconds.

@sirpy The stats are not updated in the "So far today" section more than 10 sec after the claim
Checked on the Prod v.1.12.5
Device: Desktop // Windows 10 // Google Chrome v 87
Video: https://www.dropbox.com/s/x724938ch9q9idg/The stats are not updated in the so far today section.mp4?dl=0

@sirpy The stats are not updated in the "So far today" section more than 10 sec after the claim
Checked on the Prod v.1.12.5
Device: Desktop // Windows 10 // Google Chrome v 87
Video: https://www.dropbox.com/s/x724938ch9q9idg/The stats are not updated in the so far today section.mp4?dl=0

please test it on goodqa

please test it on goodqa

@sirpy The stat is updated in the "So far today" section less than 10 sec after the claim
Checked on the QA v.1.15.0-0
Device: Desktop // Windows 10 // Google Chrome v 87
Video:https://www.dropbox.com/s/ak8bhxyqgw0lxvs/The stats is updeted immediatly in the so far today section on QA environment.mp4?dl=0

@sirpy The stat is updated in the "So far today" section less than 10 sec after the claim
Checked on the QA v.1.15.0-0
Device: Desktop // Windows 10 // Google Chrome v 87
Video:https://www.dropbox.com/s/ak8bhxyqgw0lxvs/The stats is updeted immediatly in the so far today section on QA environment.mp4?dl=0

so this can be closed for now. or tested  on production on next release

so this can be closed for now. or tested  on production on next release

@sirpy I will test it on prod on the next release

@sirpy I will test it on prod on the next release

@sirpy The stats are not updated in the "So far today" section more than 10 sec after the claim
Note: The stats in the "So far today" section is not updated after 40 minutes from claiming and time of restarting the claim cycle
Checked on the Prod v.1.21.0 // QA 1.21.0-3
Device: Desktop // Windows 10 // Google Chrome v 87
Video: https://www.dropbox.com/s/7clkiliiis8xvtz/So far today section is not updated on PROD and QA.mp4?dl=0

@sirpy The stats are not updated in the "So far today" section more than 10 sec after the claim
Note: The stats in the "So far today" section is not updated after 40 minutes from claiming and time of restarting the claim cycle
Checked on the Prod v.1.21.0 // QA 1.21.0-3
Device: Desktop // Windows 10 // Google Chrome v 87
Video: https://www.dropbox.com/s/7clkiliiis8xvtz/So far today section is not updated on PROD and QA.mp4?dl=0

@iLystopad
the amounts in so far today are in "thousands" so they will  not change frequently because you need 1000 users to claim until they change.
i think we can also this issuee

@iLystopad
the amounts in so far today are in "thousands" so they will  not change frequently because you need 1000 users to claim until they change.
i think we can also this issuee


fix(dates): [DateTimeUtils] Use short duration formats.

Luxon uses short duration formats for toRelative method. However our designers have their own convention about formatting those dates.
This PR includes small changes for those conventions. These changes based on following document https://www.notion.so/superdispatch/Displaying-Time-ab37baa8caeb48abb1f614081d854f6a

@doniyor2109 it will work only when users has English locale.

@umidbekk Maybe we will always use English main locale?

@doniyor2109 I'm not saying that you should not merge it, just heads up


drivers: i2c_dw: Device Tree related cleanups

These two patches take a similar approach as #31274 to organise all PCIe-based I2C nodes under a PCIe bus, as well as to eliminate the hardcoded instance count assumption in the driver in favour of a Kconfig option (normally set by the SoC). Similar to #31274 we can't unfortunately take advantage of DT_INST_FOREACH_STATUS_OKAY() due to the complexities of the per-instance IRQ configuration function.

Not related specifically to this PR, but any reason that the PCIe controller isn't under the SoC node?

Not related specifically to this PR, but any reason that the PCIe controller isn't under the SoC node?

Not related specifically to this PR, but any reason that the PCIe controller isn't under the SoC node?

@galak I suppose mainly because I didn't like the deep nesting it creates, but it might indeed better describe the physical reality. Is there any practical significance either way?

Not related specifically to this PR, but any reason that the PCIe controller isn't under the SoC node?

@galak I suppose mainly because I didn't like the deep nesting it creates, but it might indeed better describe the physical reality. Is there any practical significance either way?

Do we need to add the vt-d node in this PR?

@galak I'm not sure why that'd be related, however yes (probably in a separate PR) I suppose the vt-d nodes should also be listed under the PCIe bus since they're PCIe devices. @tbursztyka FYI

Do we need to add the vt-d node in this PR?

@galak I'm not sure why that'd be related, however yes (probably in a separate PR) I suppose the vt-d nodes should also be listed under the PCIe bus since they're PCIe devices. @tbursztyka FYI

@galak I suppose mainly because I didn't like the deep nesting it creates, but it might indeed better describe the physical reality. Is there any practical significance either way?

At some point there would be in theory.  If you think about getting the proper relationships w/regards to parent & children for order to initialize / de-init things it will mater (for something like power mgmt).

@galak I suppose mainly because I didn't like the deep nesting it creates, but it might indeed better describe the physical reality. Is there any practical significance either way?

At some point there would be in theory.  If you think about getting the proper relationships w/regards to parent & children for order to initialize / de-init things it will mater (for something like power mgmt).

If the pcie0 infrastructure is part of the SoC then I agree it belongs under /soc, not in this PR.  If it's offloaded to a separate IO chip as I think is true on some x86-type devices, it probably doesn't.

If the pcie0 infrastructure is part of the SoC then I agree it belongs under /soc, not in this PR.  If it's offloaded to a separate IO chip as I think is true on some x86-type devices, it probably doesn't.

@galak I'm not sure why that'd be related, however yes (probably in a separate PR) I suppose the vt-d nodes should also be listed under the PCIe bus since they're PCIe devices. @tbursztyka FYI

vt-d is not a PCIe device. It's only found via ACPI.
Anyway, the vt-d node must stay in apollo lake's dtsi.

@galak I'm not sure why that'd be related, however yes (probably in a separate PR) I suppose the vt-d nodes should also be listed under the PCIe bus since they're PCIe devices. @tbursztyka FYI

vt-d is not a PCIe device. It's only found via ACPI.
Anyway, the vt-d node must stay in apollo lake's dtsi.

vt-d is not a PCIe device. It's only found via ACPI.
Anyway, the vt-d node must stay in apollo lake's dtsi.

Its fine if its needed, it was more I was thinking this was the first case of it getting introduced.

vt-d is not a PCIe device. It's only found via ACPI.
Anyway, the vt-d node must stay in apollo lake's dtsi.

Its fine if its needed, it was more I was thinking this was the first case of it getting introduced.


Add support for<EMAIL_ADDRESS>

This PR adds support for bitcoind version 0.14.1.

bump?

@yangwao sorry for the lack of feedback, but something's fishy. If you take a look at https://travis-ci.org/seegno/docker-bitcoind/jobs/240583779#L1140, the hashes don't match. Let me try and find the reason for this before merging 0.14.1.

@ruimarinho see. Looks like here is shasum hardcoded hmm https://github.com/seegno/docker-bitcoind/pull/25/files#diff-7e3c530571e5d1977f61accf9a838000R12

@yangwao exactly, that's what ensures that the file hasn't been tampered with. I'm switching to the .tar.gz which is published on the bitcoin.org website as I can read the hash from the SHA256SUM signed file.

@pedrobranco, deprecating this PR in favour of https://github.com/seegno/docker-bitcoind/pull/29 which brings additional changes to the alpine-based Dockerfile.


Ability to explicitely lock the repo

Output of restic version
restic 0.11.0 compiled with go1.15.3 on linux/amd64
What should restic do differently? Which functionality do you think we should add?
It would be nice to be able to explicit require an exclusive lock of the repository.
The lock code being already there, this just need a new command line option to call it.
What are you trying to do? What problem would this solve?
I need to sync my local repo (restic-rest) to an external source.
This will be done via rclone to an S3 Glacier (but that could be anything rclone supports), and I need to ensure the repo is not being written into while doing the sync to avoid weird behavior (like inconsistent pack, index, snapshots... see https://forum.restic.net/t/directly-copying-restic-data-between-repositories/2876/6)
As for a BDD backup, my plan is:
Lock repo => LVM snapshot => release lock => rclone sync the snapshot
Did restic help you today? Did it make you happy in any way?
Restic is really great for day to day use, but restore speed is pretty slow.
Big repo with lots of files seems difficult to handle too (mail servers) but still works.

Closing this as a duplicate of #2166.

Restic is really great for day to day use, but restore speed is pretty slow.

To which restic version does that apply and what do you mean by "slow"? Since restic 0.10.0 the restore speed should be reasonable. Last time I've restored a server  with lots of small files from a local SFTP server, restic was restoring at around 100MB/s (at which point the SFTP connection was the bottleneck).

Closing this as a duplicate of #2166.

Restic is really great for day to day use, but restore speed is pretty slow.

To which restic version does that apply and what do you mean by "slow"? Since restic 0.10.0 the restore speed should be reasonable. Last time I've restored a server  with lots of small files from a local SFTP server, restic was restoring at around 100MB/s (at which point the SFTP connection was the bottleneck).


TeachDesk shows error because Auto Updater

I fixed this issue by running this command
...\student-group-assignment\TeachDesk> npm install update-electron-app --save

I fixed this issue by running this command
...\student-group-assignment\TeachDesk> npm install update-electron-app --save


Ctrl+TAB for walk through view tabs

It would be convenient for me to walk through tabs using ctrl+tab and ctrl+shift+tab hotkeys.
Not only for me, I hope.

Thanks any way for your work

I'll close this as you already raised this before in #686


Window does not remember size and position

Reproducibility: always
Steps:

Launch Swing Wallet.
Resize the window and move it to a random position.
Quit Swing Wallet.
Launch Swing Wallet.

What happened:
Window size and position are reset to default.
Expected result:
Window size and position are the ones set in step 2.

This project appears to be abandoned. Retracting.


config is not saved to hue.ini

After installing, I can change conf, but it's not saved to hue.ini file and not working.
Am I supposed to get the content of pseudo-distributed conf and save it to hue.ini?

Hi zyfo2:
What‘s the version you are using for hue? if the version is 3.11.0+,the conf of pseudo-distributed can be used.

@zyfo2 All ini files under $HUE_HOME/desktop/conf are treated equally.
At least that is what it says in the files.

Thanks for help. @EsharEditor @cricket007  I'm using 3.12.0.
I'm fairly new to ambari and hue. I didn't see the pseudo content saved in the setup so just wanna make sure that I need to add it.
After adding
File(os.path.join(params.hue_conf_dir, 'hue.ini'),
owner=params.hue_user,
group=params.hue_group,
content=InlineTemplate(params.hue_pseudodistributed_content),
mode=0644,
) in setup_hue.py
and make a few fixs in this pseudo conf, it works.


VS code app freezes/unresponsive

VS Code Version: Version: 1.94.2
Commit: 384ff7382de624fb94dbaf6da11977bba1ecd427
Date: 2024-10-09T16:08:44.566Z (1 wk ago)
Electron: 30.5.1
ElectronBuildId: 10262041
Chromium: 124.0.6367.243
Node.js: 20.16.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 24.0.0
OS Version: Mac OS 15.0.1

Steps to Reproduce:

during ssh, using jupyter notebooks (10MB size, with several plots in it).
copy the image output from notebook to local
after several times, the app would freeze.

It never happened before the VS code update. Same usage as before. Sometimes it recovers itself, most of the time I have to restart it.

it happens again today - I start to believe it is related to the action of copy large image output to local.

Thanks for the report.  For performance and stability issues, especially when intermittent, it is often helpful for us to have a profile.  We have some docs here on how to capture that both on the remote and local hosts https://github.com/microsoft/vscode-remote-release/wiki/Remote-SSH-troubleshooting#performance-issues

We appreciate you taking the time to open an issue. There's lot of variables in play here (with many outside of our control).  If you're able to capture a profile via the link above, please do share


Withdrawal complete notifications not being sent in production

Description

Withdrawal completed notifications are not being sent in production.
The testnet link is hardcoded in the code so even if a notification was sent it would be a link to the testnet.

Staging notification

Moved to https://github.com/ResearchHub/issues/issues/24


artisan:optimize task of laravel recipe Fail with Laravel 5.8.17 with Docker

As of Laravel 5.6, artisan optimize should no longer be executed, but task artisan:optimize executes it in versions prior to 5.5 and in the same or after 5.7. I think the second condition should not be, since doing it with a 5.8 fails.
#####################################################################
I'm working on first LTS release of Deployer.
Please, consider supporting Deployer development on Patreon page:
https://www.patreon.com/deployer
#####################################################################

Q
A

Issue Type
Bug

Deployer Version
6.2.0

Local Machine OS
Ubuntu 18.04.2 LTS

Remote Machine OS
Ubuntu 18.04.2 LTS

Docker Version
18.09.2, build 6247962

Docker Compose Version
1.24.0, build 0aa59064

PHP Container
digitalocean.com/php

Description
If you're reporting a bug, please include following information
Steps to reproduce
Content of deploy.php
task('artisan:optimize', function () {
    $deprecatedVersion = 5.5;
    $readdedInVersion = 5.7;
    $currentVersion = get('laravel_version');
    writeln('Laravel version: ' . $currentVersion);
    if (
        version_compare($currentVersion, $deprecatedVersion, '<') ||
        version_compare($currentVersion, $readdedInVersion, '>=')
    ) {
        writeln('RUN optimize');
        run('{{bin/php}} {{deploy_path}}/current/artisan optimize');
    }
});

Output log
➤ Executing task artisan:optimize
Laravel version: 5.8.17
RUN optimize
[laraveldocker.ltwtest.com] > cd /var/www/current/ && docker-compose exec -T app php /var/www/current/artisan optimize
[laraveldocker.ltwtest.com] < Configuration cache cleared!
[laraveldocker.ltwtest.com] < Configuration cached successfully!
[laraveldocker.ltwtest.com] < Route cache cleared!
[laraveldocker.ltwtest.com] < In Route.php line 917:
[laraveldocker.ltwtest.com] <   Unable to prepare route [api/user] for serialization. Uses Closure.
➤ Executing task deploy:failed
• done on [laraveldocker.ltwtest.com]
✔ Ok [0ms]
➤ Executing task deploy:unlock
[laraveldocker.ltwtest.com] > rm -f /var/www/.dep/deploy.lock
• done on [laraveldocker.ltwtest.com]
✔ Ok [96ms]
In Client.php line 99:
[Deployer\Exception\RuntimeException (1)]
The command "cd /var/www/current/ && docker-compose exec -T app php /var/www/current/artisan optimize" failed.
Exit Code: 1 (General error)
Host Name: laraveldocker.ltwtest.com
================
Configuration cache cleared!
Configuration cached successfully!
Route cache cleared!
In Route.php line 917:
Unable to prepare route [api/user] for serialization. Uses Closure.

Exception trace:
Deployer\Ssh\Client->run() at phar:///usr/local/bin/dep/src/functions.php:304
Deployer\run() at /home/xavier/Documents/Projects/laravel-docker/deployer/recipe/laravel_docker.php:125
Deployer\Deployer::Deployer{closure}() at n/a:n/a
call_user_func() at phar:///usr/local/bin/dep/src/Task/Task.php:105
Deployer\Task\Task->run() at phar:///usr/local/bin/dep/src/Executor/SeriesExecutor.php:63
Deployer\Executor\SeriesExecutor->run() at phar:///usr/local/bin/dep/src/Console/TaskCommand.php:142
Deployer\Console\TaskCommand->execute() at phar:///usr/local/bin/dep/vendor/symfony/console/Command/Command.php:252
Symfony\Component\Console\Command\Command->run() at phar:///usr/local/bin/dep/vendor/symfony/console/Application.php:946
Symfony\Component\Console\Application->doRunCommand() at phar:///usr/local/bin/dep/src/Console/Application.php:133
Deployer\Console\Application->doRunCommand() at phar:///usr/local/bin/dep/vendor/symfony/console/Application.php:248
Symfony\Component\Console\Application->doRun() at phar:///usr/local/bin/dep/vendor/symfony/console/Application.php:148
Symfony\Component\Console\Application->run() at phar:///usr/local/bin/dep/src/Deployer.php:331
Deployer\Deployer::run() at phar:///usr/local/bin/dep/bin/dep:119
require() at /usr/local/bin/dep:4
deploy [-p|--parallel] [-l|--limit LIMIT] [--no-hooks] [--log LOG] [--roles ROLES] [--hosts HOSTS] [-o|--option OPTION] [--] []

Same error here, laravel v5.7.*

It isn't the deployer error. The "artisan:optimize" task calls "route:cache" command to save all application routes to one file at Laravel bootstrap/cache directory.
The default routes/api.php defines a closure like this.
Route::middleware('auth:api')->get('/user', function (Request $request) {
    return $request->user();
});

Closure could not be serialized, that is why the exception is thrown with the message "Unable to prepare route [api/user] for serialization. Uses Closure.". You have to remove or replace all route Closures.

This is a deployer error. The current Laravel recipe by default calls the 'artisan:optimize' task - a command Laravel no longer recommends and out of the box doesn't work with (Hence the route closure errors).
If you are using with a laravel version after 5.7 you have to override the deployer included recipe to avoid running the deprecated "artisan:optimize" task. Based on my experience in the last hour this is confusing to realise, as the errors highlight the issue as the routes file, not that deployer is running a deprecated command.
The easy fix would be to remove the dead "artisan:optimize" task from the default recipe, but unclear how that will affect existing 5.x users who can use the optimize command?
Alternatively is there a way to run the task only on appropriate versions of Laravel? Seems silly to have a default recipe that actively doesn't work with the default setup of the current version of Laravel.

Let's  make it optional in recipe.
Also I'm thinking to moving away of common recipes for frameworks. It should be just templates.

@hennell-git just ran into this as well. How do you "override the deployer included recipe" as you mentioned?

Like this:
task('artisan:optimize', function () {});

There are route caching docs for Laravel > https://laravel.com/docs/5.8/controllers#route-caching
First important thing there is all about closures..
Deployer can't resolve issues defined by the software/framework itself..

Fixed in master. Please verify.

Tried to help out and verify just using a new default laravel app and deploying to a local folder.

Confirmed a default laravel app (7.9.2) & deployer default laravel recipe still breaks in 6.8.0
Swapped to deployer master branch to check fix, and can't get a working setup to even test with. localhost()->stage() doesn't exist anymore, tried to follow src changes to guess how you set a stage name now, but all your commits are labeled things like 🐳🦄🐍 and touch everything, so gave up trying to work it out. 🤷♂

Based on the Laravel.php recipe change (commit message "🍯" ) I would guess it isn't fixed:
task('artisan:optimize', artisan('optimize', ['min' => 5.7]));
should probably be:
task('artisan:optimize', artisan('optimize', ['**max**' => 5.7]));
as that's the last version optimise works on, but unverified as master has other issues.

Yes, master currently difficult to follow. I'm going to squash everything and wrtie better changelog.
Also I'm testing via deploying latest laravel. Also now is best time to get thinks fixed.
host(...)->set('labels', ['stage' => 'prod']);

<?php
namespace Deployer;
//putenv('DONT_TRACK=ಠ_ಠ');

require 'recipe/laravel.php';

// Config

set('application', 'blog');
set('deploy_path', function () {
    return '~/' . currentHost()->alias();
});
set('repository', 'git@gitlab.com:medv/blog.git');

add('shared_files', []);
add('shared_dirs', []);
add('writable_dirs', []);

// Hosts

set('identity_file', '~/.ssh/id_rsa');
set('sudo_pass', 'insure tableful hostile overcast pontific super');
set('labels', ['app' => 'web']);

host('prod')->set('hostname', '<IP_ADDRESS>');
host('beta')->set('hostname', '<IP_ADDRESS>');
host('test')->set('hostname', '<IP_ADDRESS>');
host('monitoring')->set('hostname', '<IP_ADDRESS>')->set('labels', ['app' => 'monit']);

// Tasks
task('deploy')->select('app=web');
task('build', function () {
    run('cd {{release_path}} && npm run build');
});

task('upload', function () {
    upload('/Users/anton/dev/blog', '~/' . time());
});

task('download', function () {
    download('~', '/Users/anton/dev/blog/downloaded');
});

// If deploy fails automatically unlock.
after('deploy:failed', 'deploy:unlock');

task('uptime', function () {
    run('uptime -p');
})->once()->select('host=monitoring');

before('deploy', 'uptime');

task('deploy:symlink')->limit(1);

task('reload:nginx', function () {
    run('sudo nginx -s reload');
});

task('test', function () {
    run('echo {{deploy_path}}');
    on(Deployer::get()->hosts, function () {
        set('foo', run('echo {{deploy_path}}'));
    });
})->once();

task('date', function () {
    run('while true; do date; sleep 1; done', ['idle_timeout' => 2]);
});

task('config', function () {
    var_dump(currentHost()->getConfig());
});


chore: splitting sample into single files

@gguuss I do not know what this means:
[14:44:58][ERROR] Found unpermitted build inputs

@munkhuushmgl I don't think the lint issue is yours.  @kurtisvg  is doing some stuff

What's going on? We always expect some kind of message for PR's. Especially for something this big.
Why are the copyrights all 2017 and for Google Inc. rather than 2020 Google LLC ?

Sorry about the confusion, This was part of my clean up tasks. I will update the message

No description provided.


[LIGHTNODE] Upgrade the Purchase Landing Page UI to address conversion failures

As a user, when I land on the https://www.palomachain.com/purchase/ I want to see compelling reasons for me to purchase.
NOTE: We want to focus on the middle panel which has most purchase consideration activity.

Sale ending time is not highlighted and appears small and non-distinct. Fix this so that the Sale End is pronounced.
The remaining nodes number 144/884 is confusing. Users say they don't know how much remains. Separate the numbers and show the number of nodes left at the slot. users do not need to know how many total nodes in the slot
Implied FDV has been a big win. Most usres want to buy LOWER FDV before it rises. Show the IMPLIED FDV as linkd to the days left. For example: SALE ENDS WITH IMPLIED FDV of $16.98MM in 28 days 10 hrs 13 min
There is no Information for the Number of GRAINs in the middle panel. 50,000 Per NODE.
Create a design version with the left panel removed.

@taariq Version with Left Panel removed: https://www.figma.com/design/yC7zuL9bFx9yvPd2aBalX6/Paloma-Light-Sale-Node?node-id=3513-610&t=xjpPc2snhwxExvcT-4
Version with updated data: https://www.figma.com/design/yC7zuL9bFx9yvPd2aBalX6/Paloma-Light-Sale-Node?node-id=3513-610&t=xjpPc2snhwxExvcT-4

Thanks @joycehermosilla. .let's discuss. We are seeking  a radical redesign, not a continuation of what is not working.

No customer seems to care about the software features in the purchase consideration
Customers seem to care about FDV and Amounts left on the slot.


Can't run hoc on SVN checked out repo (win 8.1 x64)

i installed ruby, gems, ruby dev kit on win 8.1 X64. I installed hoc via CMD and it was successful. I checked out SVN repo and opened cmd and went to checked out repo directory and ran hoc. I get this error please help and let me know how you generate graph.
C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/hoc-0.6/lib/hoc/svn.rb:39:in ``': No such file or directory - diffstat -V (Errno::ENOENT)

from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/hoc-0.6/lib/hoc/svn.rb:39:in `hits'

from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/hoc-0.6/lib/hoc.rb:54:in `report'

from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/hoc-0.6/bin/hoc:61:in `'

from C:/Ruby22-x64/bin/hoc:23:in `load'

from C:/Ruby22-x64/bin/hoc:23:in `

I'm facing issues with both svn and git repo, while running HoC. help!!!!

@kbhavik sorry for delay. I never tested it on Windows. Let me try...

@yegor256 Thanks for looking into it. I was asked by Boss to create reports for new lines of code per project. While browsing i came across your blog about HoC and I planned to represent my reports with HoC. However it's not working although I also used svn2git for conversion of few SVN repos to GiT. I will really appreciate if you could help me out with HoC and guide me how to generate reports/Graphs using HoC. If possible representing authors who contribute or edited the code.

@kbhavik I will try to make it working on Windows, but I'm not sure I will succeed. Give me a few hours...

@yegor256 Take your time. I hope it works.

@kbhavik I managed to reproduce this problem in integration test: https://ci.appveyor.com/project/yegor256/hoc/build/52 Will try to fix or document it now

@rultor release, tag is 0.7

@kbhavik try version 0.7. It will work without problems with Git repository. For SVN repository you will have to install diffstat, see: http://sourceforge.net/projects/mingw/files/MSYS/Extension/diffstat/

@rultor release, tag is 0.7

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.7

@kbhavik @yegor256 Oops, I failed. You can see the full log here (spent 19min)
        svn co file://$(pwd)/base repo
        cd repo
        echo 'Hello, world!' > test.txt
        svn add test.txt
        svn ci -m 'first commit'
        echo 'Bye!' > test.txt
        svn ci -m 'second commit'
        svn rm test.txt
        svn ci -m 'third commit'
        svn up
      """
    When I run bin/hoc with "-f int -d repo" # features/step_definitions/steps.rb:57
    Then Exit code is zero                   # features/step_definitions/steps.rb:69

  Scenario: Real SVN repo                     # features/svn_basics.feature:24
    Given It is Unix                          # features/step_definitions/steps.rb:41
    Given I run bash:                         # features/step_definitions/steps.rb:49
      """
      svn co --non-interactive http://svn.apache.org/repos/asf/httpd/mod_ftp/tags/0.9.6
      """
    When I run bin/hoc with "-f int -d 0.9.6" # features/step_definitions/steps.rb:57
    Then Exit code is zero                    # features/step_definitions/steps.rb:69

Feature: Using .hoc config file
  As a source code writer I want to be able to
  call HOC as a command line tool and configure
  it via .hoc configuration file

  Scenario: Simple hoc counting      # features/uses_config.feature:6
    Given It is Unix                 # features/step_definitions/steps.rb:41
    Given I run bash:                # features/step_definitions/steps.rb:49
      """
      echo '--help' > .hoc
      """
    When I run bin/hoc with "-f int" # features/step_definitions/steps.rb:57
    Then Exit code is zero           # features/step_definitions/steps.rb:69
    And Stdout contains "[options]"  # features/step_definitions/steps.rb:63

8 scenarios (8 passed)
33 steps (33 passed)
13m8.508s
Coverage (45.83%) is below the expected minimum coverage (100.00%).
Coverage report generated for Cucumber Features to /home/r/repo/coverage. 22 / 48 LOC (45.83%) covered.
\u001b[33m[Coveralls] Outside the Travis environment, not sending data.\u001b[0m
659
end.sh: + ps -p 22659
end.sh: + exit 0
end.sh: + '[' '!' -e pid ']'
end.sh: ++ cat pid
end.sh: + pid=22659
end.sh: + ps -p 22659
end.sh: + '[' '!' -e cid ']'
end.sh: ++ cat cid
end.sh: + cid=a78a0af7479eb2d6785842bbd809820ad002168bfaea6da4f8cdd37e4a1b06a8
end.sh: + docker ps -qa --no-trunc
end.sh: + grep --quiet a78a0af7479eb2d6785842bbd809820ad002168bfaea6da4f8cdd37e4a1b06a8
end.sh: + echo 'container a78a0af7479eb2d6785842bbd809820ad002168bfaea6da4f8cdd37e4a1b06a8 is dead'
end.sh: container a78a0af7479eb2d6785842bbd809820ad002168bfaea6da4f8cdd37e4a1b06a8 is dead
end.sh: + exit 1

@rultor release, tag is 0.7

@rultor release, tag is 0.7

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.7

@yegor256 Done! FYI, the full log is here (took me 4min)

Hey! HoC on git is working. I installed diffstat using MinGW installer but on svn repo i get this error:

@yegor256 more issue on svn:

@kbhavik I managed to reproduce the problem in https://ci.appveyor.com/project/yegor256/hoc/build/61 Will try to fix it now

@rultor release, tag is 0.7.1

@rultor release, tag is 0.7.1

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.7.1

@yegor256 Done! FYI, the full log is here (took me 4min)

@kbhavik try version 0.7.1, I believe it will work without problems, with SVN

@yegor256 Thank you for modifying the code and taking pains to resolve the issue just for a single guy! It's working now.. Hats off to you.. Can you please tell me how to generate a graph to show loc and hoc by every author. Like you represented in your blog.

@yegor256 Kindly advice..

@kbhavik hm.. that's a good question. I built this graph manually, collecting data not via this hoc Ruby gem, but directly from Git. It wasn't easy :) I wrote a small software just to build this small graph for the blog post.

@kbhavik can you please close this ticket, if everything works as you expected. And thanks for using this tool :)

Thanks for helping!


Cannot see full error on prior revision

Output of helm version:
Client: &version.Version{SemVer:"v2.6.1", GitCommit:"bbc1f71dc03afc5f00c6ac84b9308f8ecb4f39ac", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.6.2", GitCommit:"be3ae4ea91b2960be98c07e8f73754e67e87963c", GitTreeState:"clean"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.3", GitCommit:"d2835416544f298c919e2ead3be3d0864b52323b", GitTreeState:"clean", BuildDate:"2018-02-07T12:22:21Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.3", GitCommit:"2bba0127d85d5a46ab4b778548be28623b32d0b0", GitTreeState:"clean", BuildDate:"2018-05-21T09:05:37Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

I know I'm on an old version, so feel free to close this if it is resolved. However, I didn't see any related issues, so I'm guessing this hasn't been mentioned yet.
I have a release that failed, but a subsequent upgrade succeeded. history truncates the error message and neither status nor get seem to include it:
[rodencor@karminac tenant]$ helm --tiller-namespace ipaas history       tenant-upgrade-test-1530219075
REVISION        UPDATED                         STATUS          CHART           DESCRIPTION
1               Thu Jun 28 16:51:27 2018        SUPERSEDED      tenant-1.7.2    Install complete
2               Thu Jun 28 16:54:19 2018        SUPERSEDED      tenant-1.8.6    Upgrade "tenant-upgrade-test-1530219075" failed: Deployme...
3               Thu Jun 28 16:54:25 2018        DEPLOYED        tenant-1.8.6    Upgrade complete
[rodencor@karminac tenant]$ helm --tiller-namespace ipaas status tenant-upgrade-test-1530219075 | grep failed
[rodencor@karminac tenant]$ helm --tiller-namespace ipaas get tenant-upgrade-test-1530219075 | grep failed

Note that the history line literally ends with .... If it is going to be truncated, could it at least be incuded in full in the status output?
In most cases you'd ideally see the error response the first time, but I can think of a few scenarios where that isn't viable:

Debugging after an automated test fails
Figuring out why a release failed after a long-runnning helm connection was killed but the operation proceeded (#3015)
In a devops role, where a team is asking you to help debug their failed deployment and you need to go back and look at the history.

this was fixed in https://github.com/kubernetes/helm/pull/2836 IIRC. Can you try with a newer version of helm and confirm?

Don't have ready access to a newer server, but we're updating soon. #2836 sounds like the fix, I'll reopen if not.


update model workspace CTA to point to pre-existing dashboard(s) instead of "create new dashboard"

closes #1622
This PR aims to do the minimum amount necessary to update the CTA on the modeling page. It's part of a larger set of changes involved for #1620. This PR does the following:

Implements a tweak to the main Button.svelte component, needed for additional changes in #1620
if the model already has a dashboard associated with it, replace the Create Dashboard button with Go to Dashboard
if the model has a few dashboards, the CTA will actually drop down a menu.
adds a new Forward.svelte icon from our figma icon set

Code looks good.
Experience-wise, I think it'd be good to retain a CTA to create a new dashboard, alongside the CTA to go to an existing one. Scott says it's a common pattern for our customers to have multiple dashboards for one model (where often fwiw each dashboard is a different subset of measures/dimensions). I know it's technically possible to still create two dashboards, but it's not obvious.

It's a good point re: multiple dashboards on one model, but I think this will be the way we meet the underlying need going forward. There are many other ways to solve this issue that don't look like child dashboards, including RBAC and other affordances that aren't in scope for Rill Developer. Given that this is not a one-way door I would prefer we merge as-is and we can readdress later. Thanks for the review @ericpgreen2!


Update flux version to v0.50.2

This upgrades the flux version to v0.50.2.
The secret service, which is used for alerts, is not included. The
to() function is also still not included.

Sorry this is just a cherry pick of the 1.7 version. I didn't write a lot of the code as it was just synced over from 2.0.

oh, oops.


Add .IfNullOrWhiteSpace() .IfNullOrEmpty() for string properties

Currently, if we have an object with a nullable string property
record Person(string? Name);

We need 2 checks to do this:
person.Throw()
    .IfNull(p => p.Name)
    .IfEmpty(p => p.Name!);

Instead, this would be more fluent and performant:
person.Throw()
    .IfNullOrEmpty(p => p.Name);

person.Throw()
    .IfNullOrWhiteSpace(p => p.Name);

I would love to tackle this issue. I'll start working on it later, hope to have a PR ready within a day. 🚀


Cog for code snippet links

I thought that it would be pretty cool if Discord could show the contents of a snippet link instead of just the link to the snippet, so I made a bot for that.
However, since adding third-party bots to the Python server was out of the question, I decided to add that functionality to the official server bot instead.

@dolphingarlic I don't think that this should good feature. Sending something every time when someone posts GitHub link is not a good idea. And all PRs have to have approved the linked issue, what this PR definitely don't have. Make issue first, then staff/core devs will discuss it and then decide will this come to bot or not.

Sorry, I didn't know that I should've created an issue first. (I don't think CONTRIBUTING.md mentioned that).
I think that printing out snippets would be quite useful since it means that people wouldn't have to open the links each time anymore. Sending embeds with details about repos basically just improves what Discord already does with repo links but with more details.

@dolphingarlic I don't think that this should good feature. Sending something every time when someone posts GitHub link is not a good idea. And all PRs have to have approved the linked issue, what this PR definitely don't have. Make issue first, then staff/core devs will discuss it and then decide will this come to bot or not.

I think this was discussed in the #meta channel over at PyDis and there was some merit in the linked idea. Personally I think it is useful, especially when getting help in any of the help channels because you either copy paste your code standalone or link to it which would mean for the other person to switch back and forth.

Sorry, I didn't know that I should've created an issue first. (I don't think CONTRIBUTING.md mentioned that).

It's not mentioned because it's only a hard requirement for Seasonalbot. It's generally a good idea to make an issue first so you don't "waste" your time working on something that might not be merged, but since this seems to be something you've made for other reasons already it's a bit of a moo point.
I'm a bit meh on the repo link prettification, but the snippet formatting is deinitely interesting. We'll have to get some more opinions before we review formally, so please stand by. Is this something you've discussed in the server previously?

Yes - I've discussed it once in #meta a while ago

I can remove the repo link prettification if you'd like, but here's an example of when the snippet link feature is very useful:

Here, seeing just the single line linked in the snippet instead of having to open the link in a new tab is much more convenient. It would make debugging a lot faster.

As Discord already embeds a link preview by default for GitHub-links, I'm not sure if I'm in favor of having the bot respond with a second embed containing some details about the repository. If we were to have "repository information" feature, I think it would be better as an explicitly invoked command. I'm just not sure whether or not such a command fits the scope of our community utility bot. (I really mean: "not sure"; it's not a euphemism for "I don't think it fits".)
I do like the snippet feature, although I'm a bit concerned of "surprising" vertical space that's get inserted in a conversation. Maybe we could limit it to snippets that are only a few lines long? I wouldn't be surprised if people pasting a link to a snippet of code don't expect the bot to respond with a 15-line code snippet in the middle of a conversation. Alternatively, this could also have a command interface, which ensures that it's explicitly invoked.

If this were to be included, I'd prefer it to be a command or limited to maybe 3 lines without directly invoking it instead of looking up links in messages. Most github permalinks that I've seen get posted in discussion channels like python-language where this behavior wouldn't be as desired
While the repo information is nice I don't think the additional info like stars is particularly useful in most situations

One more thing: When we do this, this should get filtering too. Maybe someone wants to write a message that gets deleted by filters, but then add this GitHub and post a link. And however, this should have some text filter.

So since nobody seems to like the repo widget prettification feature, I removed that. To address the issue of people not wanting the bot to interrupt their conversations, I added the option for the author of the message to react with an ❌ emoji to delete the bot's message.

I've made all the changes requested besides for the regex ones. (Sorry I neglected this PR - I've had exams the past few weeks)

For future changes, please don't push them all in a single commit. It makes it hard for reviewers. See point № 4 in our contributing guidelines https://github.com/python-discord/bot/blob/master/CONTRIBUTING.md

Hi @dolphingarlic, are you still interested in working on this?

Hi, I'm still interested in working on this. I've just been overwhelmed by finals/IOI/IMO lately and haven't found the time or energy to address the reviews :grimacing:
I'll definitely be able to work on this in a few (~3) weeks though

Ok, that is fine, no worries. Just making sure the PR hasn't been abandoned.

Also, should I add tests for this cog? If so, how?

Also, should I add tests for this cog? If so, how?

No, you don't have to.

Coverage decreased (-0.3%) to 56.26% when pulling<PHONE_NUMBER>03c2e7680b92dff309d60ce83837c1 on dolphingarlic:master into 345ee39e7cc5449e563817c4f30895638c66c206 on python-discord:master.

Do you think I should just copy what I did for BitBucket query parameters for the other regexes? Since GitHub doesn't support straight-up ? characters in the URL, I don't think this will break anything

Do you think I should just copy what I did for BitBucket query parameters for the other regexes? Since GitHub doesn't support straight-up ? characters in the URL, I don't think this will break anything

Do you think I should just copy what I did for BitBucket query parameters for the other regexes? Since GitHub doesn't support straight-up ? characters in the URL, I don't think this will break anything

Yup, that sounds fine. I also think it's safe.

Do you think I should just copy what I did for BitBucket query parameters for the other regexes? Since GitHub doesn't support straight-up ? characters in the URL, I don't think this will break anything

Yup, that sounds fine. I also think it's safe.

Truncating might be a bit tricky when there are multiple snippet links in a single message. Do you think I should do something like truncate all code blocks by ceil((len(message) - 2000) / (number of code blocks)) chars if needed? Also, what about the number of lines allowed per message?

Truncating might be a bit tricky when there are multiple snippet links in a single message. Do you think I should do something like truncate all code blocks by ceil((len(message) - 2000) / (number of code blocks)) chars if needed? Also, what about the number of lines allowed per message?

I imagine that trying to evenly distribute truncation would result in all code blocks being too heavily truncated to still be meaningful. A simpler strategy of truncating the first blocks and omitting the rest sounds fine.
Line limit should still apply to the final line count and will truncate at line boundaries. The limit could be raised to 20. Make sure you don't include "bare" file paths that lack code blocks because they got truncated. In other words, if there isn't enough space for the entire file path and at least some of the code block, then omit the file path too.
I think what I'm describing is pretty similar to the extant truncation logic for the snekbox eval command. It's worth investigating how much of that code can be shared between the two features.

I imagine that trying to evenly distribute truncation would result in all code blocks being too heavily truncated to still be meaningful. A simpler strategy of truncating the first blocks and omitting the rest sounds fine.
Line limit should still apply to the final line count and will truncate at line boundaries. The limit could be raised to 20. Make sure you don't include "bare" file paths that lack code blocks because they got truncated. In other words, if there isn't enough space for the entire file path and at least some of the code block, then omit the file path too.
I think what I'm describing is pretty similar to the extant truncation logic for the snekbox eval command. It's worth investigating how much of that code can be shared between the two features.

@dolphingarlic Greetings. Will you be finishing up this PR?

I'm still waiting for my code to be reviewed again. Sorry about the delay

Thanks for the quick reply! I'll poke some people for reviews.


Templates containing root view controllers subclassing LayoutViewController steal the refresh command

I guess this is technically "normal behaviour", as Layout is not really made to have multiple LayoutViewControllers  at the same time. But perhaps we could either make an exception for UITabBarController or UINavigationController, or perhaps output a warning in the console when a template contains a root LayoutViewController?

I'd guess this is fixed now that the reload handler is a singleton? Let me know if you're still seeing it.


Provide a handler which is performed at the very end of project loading

I want to run a script that change the terminal window title (actually, the terminal tab title) after project finish loading.
I have the set -g set-titles on setting in tmux.conf. This setting allows tmux to automatically change terminal title by the name of the currently started session. If I hook my script on the handlers which is available in tmuxinator at present, tmux rewrites its result, so the task can't be accomplished.
I propose to add a project hook which is performed at the very end of project loading, so that you can perform actions the result of which couldn't be rewritten by any following actions in the process of project loading.
I'd call the hook on_project_ready.

@Shamaoke hello! I contributed the "hooks" feature a while back so let me jump in. In not entirely sure what you propose is feasible. I remember that due to how tmuxinator works there were some limitations to what hooks is possible to implement.
May I ask you to try implementing a stub versions of it to test if it suite your use case?
You may do that without touching any code: dump the template that tmuxinator runs when starting a project with tmuxinator debug <project> and do manual edits to achieve your desired result. You can run the resulting script as a normal shell script.
This way we can agree on this feature behaviour before any code edits. What do you think?

@endorama I've followed your suggestion and tried to edit the generated script inserting my hook script in different parts of it. Unfortunately, the hook script didn't work as I expected wherever I put it in. Apparently, the desired feature either can't be implemented or its implementation will be too involved, so the case can be closed. Thank you for your attention to it.

No worries! I remember there was something fishy around that feature, as I would love to use it too 😅
Thanks for trying though, highly appreciated.
(I can't close this so I'll leave that to a maintainer stopping by; thanks!)


big heightmaps (WIP)

The goal of this PR is to change our heightmaps to have the same area as cubes. This cuts the amount of heightmaps and heightmap nodes to 1/4 and thus should improve performance. Additionally, by storing larger pieces of the heightmap at once, compression should benefit as well.
This PR is still WIP.

Closing this PR as it's stale.


Stored Course History Into The Database

Offer History Updater
Completed

Changed the structure of the file stored.
Imported a chromedriver to the project so that selenium will use a specific chrome.
Created a offer history updater class that updates the course offer history in the mongoDB.
Utilize functions from webreg_scraper to gather the course information.

@anthopark nice job strengthening your program!


support actions e.g. run tool

e.g. The mel command videoPlayer; can be run in Python
import maya.mel
maya.mel.eval("videoPlayer;")

https://github.com/hannesdelbeke/Maya-Video-Player
command/action to open the window

make it so you can right click on plugget qt, and click open window, and open docs

existing actions also could use custom input.
e.g. action install requriements. could be fed specific requirements string input.
action install addon could be fed, enable on startup bool


Multiple github tokens

Description
It would be wonderful if you can add multiple github tokens, for other github spaces.
For example:
use token X for : https://github.com/google/...
use token Y for : https://github.com/github/...
Example URLs
Every page

I assume you have different accounts that have access to different repos/orgs, right?

Yes right

Many reasons not to do it unfortunately:

niche
requires extra UI to match tokens to a specific repo/org
your workflow requires you to constantly log in and out of GitHub, which is a bad base to work on IMHO. I'd suggest using multiple profiles instead, on which you can use different tokens

Sorry to bump this, but:

requires extra UI to match tokens to a specific repo/org

Can this be done based on the currently-logged in user instead?

your workflow requires you to constantly log in and out of GitHub

With Firefox's containers this is no longer the case.

I'd suggest using multiple profiles instead, on which you can use different tokens

This is unfortunately more cumbersome because it requires you to maintain multiple windows, whereas with my current setup I can just have tabs on personal/work containers.
It would be nice if this was reconsidered :)

It would be nice to have, but I have zero intention to write the whole component for the options page and logic to pick the right token after a page load.
I’d be happy to accept a complete PR for this but it’s likely too much work. Anything related to this should not interfere with the simple single-token usage, not in performance nor in UX of the options page.
For example I’d imagine a simple “add tokens for multiple users” button to duplicate the whole token verification widget as is. The username is not required because it can be fetched via API. This call can be cached forever via webext-storage-cache
Related: A small part of this logic would be useful to this as well

https://github.com/refined-github/refined-github/issues/7384

Thanks for the response! I'll see if I can spend some time working on it.


rename master->main

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@b587210). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #238   +/-   ##
=======================================
  Coverage        ?   59.29%
=======================================
  Files           ?       95
  Lines           ?    11715
  Branches        ?        0
=======================================
  Hits            ?     6946
  Misses          ?     4769
  Partials        ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b587210...821b407. Read the comment docs.


Room navigation

Closes #20
@whoa-algebraic/jake

Important to note if you test it in Unity. When switching rooms, the camera does not snap to the player character's new location. This is an issue that'll be addressed soon and is detailed here #19


Add Music

Add some music to the game like I did with stores.

Use some stuff from stores, because I think I did it well there.


red font for inline-errors as provided by ink

just to distinguish errors from correct results as provided by atom-ink's Result class (https://github.com/JunoLab/atom-ink/blob/master/docs/results.md)

Thanks for the pull request!


HasValue for AddressComponent vs Uint256Component

HasValue(AddressComponent, { value: playerEntityId }) expects playerEntityId to be checksummed
HasValue(Uint256Component, { value: playerEntityId }) expects playerEntityId to be not checksummed
What matters here is the component schema on the contract side, and how network parses it.
At least for v2 we should normalize it

What component type are you using on the client? I am not seeing an address one here: https://github.com/latticexyz/mud/blob/main/packages/std-client/src/components/index.ts

Entity one (custom). Though for this issue it doesn't really matter, defineComponent isn't the one checksumming addresses
import { defineComponent, Metadata, Type, World } from "@latticexyz/recs";

const Entity = {
  value: Type.Entity,
} as const;

export function defineEntityComponent<M extends Metadata>(
  world: World,
  options?: { id?: string; metadata?: M; indexed?: boolean }
) {
  return defineComponent<typeof Entity, M>(world, Entity, options);
}

closing as outdated


[WIP] refactor sqllab run_query

This PR cleans up the sqllab code and decomposes some really long functions into understandable bits.

Codecov Report

Merging #5852 into master will decrease coverage by 9.47%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5852      +/-   ##
==========================================
- Coverage   63.75%   54.27%   -9.48%
==========================================
  Files         374      328      -46
  Lines       23320    14019    -9301
  Branches     2608     2608
==========================================
- Hits        14867     7609    -7258
+ Misses       8440     6397    -2043
  Partials       13       13

Impacted Files
Coverage Δ

superset/connectors/druid/__init__.py

superset/views/datasource.py

superset/models/sql_lab.py

superset/forms.py

superset/connectors/druid/views.py

superset/db_engine_specs.py

superset/translations/utils.py

superset/models/__init__.py

superset/exceptions.py

superset/connectors/sqla/views.py

... and 34 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 73db918...bf188ff. Read the comment docs.


bulk upserts

it would be great for @hackclub/sb2 if i could upsert an array of objects

completed with v0.3 [T]/


Permissions

Are permissions a new feature? I see they are in the master branch and can be set using https://github.com/eclecticiq/OpenTAXII/blob/master/opentaxii/auth/manager.py but the update_account() method is not present in the latest version in pip.

You have to pull directly from source to get the latest permissions feature. #103 is already open for this specific reason. I think the project might have gone stale, but it still works pretty damn well.

No problem - not really an issue but wanted to clarify. Spent a couple hours figuring out the source on github then thought i was crazy when i went digging through my local copy ;)


3.2 Transition from Target legacy APIs to Adobe I/O > Equivalent Methods terminology for methods

Clarification needed for column headings, i.e. "Legacy API Method" vs "New API Method" should be more accurately descriptive, i.e.
Legacy Test & Target vs Adobe I/O (or other), as the "New" ones link to docs that have been around for some time. May be confusing.

Issue refers to src/pages/implement/server-side/transition-from-target-legacy-apis.md

Changed "Legacy API" to "Classic API" (shorter than "Legacy Test & Target"). Changed "new API" to "Adobe I/O API."
Open to suggestions for different replacement terms, but closing for now.

Need to updates references to "Adobe I/O" to either "Adobe Developer Console" (product) or "Adobe Developer Platform" (overall platform). In most cases, better to use product name.


ci(smoke-tests): parallelize smoke tests on CI

The type of this PR is: CI
This PR solves WP-111
Description
Following the work done in https://github.com/artsy/integrity/pull/368, this parallelizes our smoke tests in Force in an effort to bring the smoke test runner down by 3min.
cc @artsy/grow-devs

👏👏👏


To avoid PHPUnit deprecations, add the 'static' keyword to the method

[x] Added or updated tests
[ ] Documented user facing changes

Resolves #1816
Changes
To avoid deprecations in PHPUnit 10, the 'static' keyword has been added to the method.
Breaking changes
None.

This has been addressed in the following commit by phpstan.
https://github.com/phpstan/phpstan-src/commit/ea2670a086f0f699b92dd6b2535504771054498c

よくやった!

Thank you for your work!


[Functionnal tests] Test on navigators

It would be interesting to do a first overview of how the website currently behave on every browser:

safari
opera
chrome
firefox
IE11
Microsoft Edge

And write here a report of potential bugs, unwanted events. Verify if sass applies correctly everywhere and the events fire properly.
@gaeVG would you be interested in doing this little study?

I found a bug in the reference form: it is not possible to remove a selected theme on Firefox, the problem does not occur on a Chromium browser

Could you create a bug ticket related to that?

I found a bug in the reference form: it is not possible to remove a selected theme on Firefox, the problem does not occur on a Chromium browser

Could you create a bug ticket related to that?

I was going to put this comment in the 88, wouldn't that be more relevant? Would you still like a new ticket?

I found a bug in the reference form: it is not possible to remove a selected theme on Firefox, the problem does not occur on a Chromium browser

Could you create a bug ticket related to that?

I was going to put this comment in the 88, wouldn't that be more relevant? Would you still like a new ticket?

To me, ticket 88 is just a testing ticket. If a bug is found while testing, it should be subject to the creation of a new bug ticket that we can solve later on, no?

It is not wrong 😁i'm going to it !

It is currently impossible to test the application on Internet Explorer, the only result I can get is a blank page and this error :

I think the problem could be solved once we set up webpack because the version should not be compatible with the JS interpreter of IE. That said, it's not a top priority to make the site compatible with this particular browser either...

I checked all the browsers quoted in the descriptions, and apart from the problem detected with IE, everything seems to function properly on the different media.
I think we can close this ticket

After creating a ticket regarding IE (even though webpack may solve the issue, then we'll close the IE ticket, otherwise we'll still have to deal with it and try and have a runnable app even without all the features available), yes you can close this one!


purple mode

Misc comparisons:

@sarahframe Not a huge deal, but is it possible to use a different icon? The checkbox icon make it look like the option is selected when it isn't 😅

@sarahframe Not a huge deal, but is it possible to use a different icon? The checkbox icon make it look like the option is selected when it isn't 😅

That does make it look confusing. It's not possible to change the icon on a model field. We could override the wagtail template but that is more work so will add it to the bottom of my list for now

💯 I have been so confused by this exact checkbox issue on other wagtail sites recently.

I reckon that gradient would look real nice with a bit of noise @liamjohnston...
https://css-tricks.com/grainy-gradients/


Create kura-datatransport component

This component should be a preferred way of using MQTT client from Kura.

I see the component is already developed,
I can manage doc if you want ?

If you could add the docs, that we be great. Otherwise I'll do it.
Thanks!


Safe/invalid email address generator

It would be nice to have a safe/invalid email address generator<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>etc. That is to ensure that generated email really does not exist.

I would suggest using regex or<EMAIL_ADDRESS>if you have a specific email domain that you want to use.


Fix illegal String format

See Java documentation on Formatter syntax; Java will throw an error when it encounters "%-3.3" as a String format.

Hi.
I am happy to see that you have chosen this project, and hope you find it useful.
Thanks for the fix.
Can you, however, change it as follows:

Work on master branch. Not on release branch (so you have to close this pull request, and add a new pull request on master).
The mistake is only the omitted "f". Can you please just replace %-3.3 with %-3.3f  (just add "f") ?

Thanks,
Asher

Oops, sure!
PS: Yes, I really like what you've done Token/Label polymorphism.


How to validate the selected elements only?

Is there any way to validate the elements having a specific selector in a form?
For example,
I'd like to validate the items having class "submit-opt" in a form.  I could find "ignore" option in jQuery Validation plug-in documentation, but not sure how to do reversely.
I tried to use $form.find(".submit-opt").valid(), but there happened a JS error
Uncaught TypeError: Cannot read property 'nodeName' of undefined
Thanks.

`ignore: ":not(.submit-opt)" should do the trick. Otherwise, a post on the forum or StackOverflow should help.


fix: update engine to allow disabling timer batch

What type of PR is this?
/kind bug
What this PR does / why we need it:

Allow setting TIMER_BATCH to ZERO to prevent the engine from fetching timers
Update seeds to be run after any process have been run.

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation e.g., usage docs, etc.:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sort Social links

Hi there,
If i don't sort these links then it changes everytime i deploy the site.

I will have to skip this change as it is a personal preference.