Aerial  constantly asking for permission to access macOS screensaver.  Did macOS 14.1 break Aerial again?

After updating macOS to 14.1, Aerial or Aerial Companion constantly asks for permission to access macOS screensavers.  However, an affirmative response does not stick after reboot.
Did macOS 14.1 break Aerial again?
2021 24" 4-Port M1-iMac
macOS 14.1
Single, built-n monitor
Aerial Companion 1.5.2
Aerial 3.3.4

Hey @RGM1956
Yeah it's one of those things, you need to go to System Settings and explicitly give "Full Disk Access" to Companion to make that disappear.
Allowing when prompted is temporary sadly, Apple doesn't make those choices stick for some reason, which is a real pain.

Think, maybe, it’s intentional?

On Nov 4, 2023, at 7:28 AM, Guillaume Louel @.***> wrote:
Hey @RGM1956 https://github.com/RGM1956
Yeah it's one of those things, you need to go to System Settings and explicitly give "Full Disk Access" to Companion to make that disappear.
Allowing when prompted is temporary sadly, Apple doesn't make those choices stick for some reason, which is a real pain.
—
Reply to this email directly, view it on GitHub https://github.com/JohnCoates/Aerial/issues/1342#issuecomment-1793458234, or unsubscribe https://github.com/notifications/unsubscribe-auth/BCWXRDUEZ74VBKTWTO3QCGTYCZGI7AVCNFSM6AAAAAA65TLYPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTGQ2TQMRTGQ.
You are receiving this because you were mentioned.

And now another new item.  Yes,  gave Aerial Companion Full Disk Access, but despite also setting it to restart (motion wallpaper) at startup, it (now) isn’t.

On Nov 4, 2023, at 7:28 AM, Guillaume Louel @.***> wrote:
Hey @RGM1956 https://github.com/RGM1956
Yeah it's one of those things, you need to go to System Settings and explicitly give "Full Disk Access" to Companion to make that disappear.
Allowing when prompted is temporary sadly, Apple doesn't make those choices stick for some reason, which is a real pain.
—
Reply to this email directly, view it on GitHub https://github.com/JohnCoates/Aerial/issues/1342#issuecomment-1793458234, or unsubscribe https://github.com/notifications/unsubscribe-auth/BCWXRDUEZ74VBKTWTO3QCGTYCZGI7AVCNFSM6AAAAAA65TLYPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTGQ2TQMRTGQ.
You are receiving this because you were mentioned.

Which part doesn't restart ?
Does Aerial Companion show up in your menu bar at reboot, but it doesn't start playback, or does Aerial Companion NOT show up in your menu bar at reboot, and when you launch it manually, playback starts again ?

No, Companion no longer appears in the menu bar even though the app is set to start at login. I had to manually add Companion to Login Items” in order to get it to work. On Nov 4, 2023, at 10:19 AM, Guillaume Louel @.***> wrote:
Which part doesn't restart ?
Does Aerial Companion show up in your menu bar at reboot, but it doesn't start playback, or does Aerial Companion NOT show up in your menu bar at reboot, and when you launch it manually, playback starts again ?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@RGM1956 are both selected as such here ? Not just the bottom one ?
If they are, I'll try to have a look when I can but that might be a few days.

They are

On Nov 7, 2023, at 2:30 AM, Guillaume Louel @.***> wrote:
https://user-images.githubusercontent.com/37544189/281007051-79be1a93-5f4c-4d8d-b62e-009f5a369fc3.jpg
@RGM1956 https://github.com/RGM1956 are both selected as such here ? Not just the bottom one ?
If they are, I'll try to have a look when I can but that might be a few days.
—
Reply to this email directly, view it on GitHub https://github.com/JohnCoates/Aerial/issues/1342#issuecomment-1798232876, or unsubscribe https://github.com/notifications/unsubscribe-auth/BCWXRDSRCLA7YRHWS4BQ733YDIEUVAVCNFSM6AAAAAA65TLYPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJYGIZTEOBXGY.
You are receiving this because you were mentioned.

Try changing "At Startup" to "Manually", then back to "At Startup". It's likely they broke something again with launch agents.

That fixed it.  Thanks,

On Nov 7, 2023, at 3:44 AM, Guillaume Louel @.***> wrote:
Try changing "At Startup" to "Manually", then back to "At Startup". It's likely they broke something again with launch agents.
—
Reply to this email directly, view it on GitHub https://github.com/JohnCoates/Aerial/issues/1342#issuecomment-1798343790, or unsubscribe https://github.com/notifications/unsubscribe-auth/BCWXRDX5KSXAMPFCR7TQC5DYDINJ3AVCNFSM6AAAAAA65TLYPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJYGM2DGNZZGA.
You are receiving this because you were mentioned.


Predicted sample export failed in GUI

Hello,
I am trying to map a list of fastq files to reference mus musculus for my research. My system has 32GB ram and i7 processor. But when i start running i get the following error. I don't have any idea how to solve this. I would be highly obliged if you help me solve this .
I have attached the image for your reference. Thanking you for your time.

Hi Aniruddh,
As you note, when performing an unsupervised analysis with ICGS2 with FASTQ files loaded, the program supports processing FASTQs through the Kallisto workflow to produce TPM quantified gene expression files. Below is an example I just ran in the software using some downsampled paired-end FASTQ files:

If this analysis fails there are different possible reasons. If it immediately fails, it suggests that there is a simply a naming issue with the FASTQ files not being sufficiently recognized by AltAnalyze and communicated to Kallisto. Kallisto here is by default a version of Kallisto we created called Kallisto-splice, that also produced spliced-aligned, genomic coordinate associate BAM and BED files used for possible downstream splicing analyses.
My guess is that AltAnalyze was not properly recognizing which files were either fastq files or read 1 and read 2. Can you indicate an example names of paired-end fastq files? AltAnalyze supports a number of possible fastq file names, including recognizing read1 and read2, but it can fail if it doesn't recognize the names. As shown in the above screen shot, of the files produced in the created ExpressionInput directory, which includes the folder Kallisto and log files for each sample run (run_info.json) and an overall log file (log.txt). If the file names are OK, then the issue is likely in these log files.
Best,
Nathan

Hello Nathan,
Thanks for your quick response. Yeah, it fails immediately after I start the analysis. I have attached the screenshot for my fastq files. The R1  here is read 1 and R2 is read 2. If you can have a look and suggest if I should change the names of fastq files and try or should I try something else?

Thanking you for your time.
Aniruddh

Hello Nathan,
As you can see I have changed the file names as suggested and ran

but the problem is I'm still getting the error and this stops immediately after I run the analysis using UI.

I also did go to the Kallisto folder to check the log files but there is nothing generated.

Can you help??

Hi Aniruddh,
If no files are produced in the Kallisto directory, then Kallisto ends up not being called. In such cases there are two options:

Call AltAnalyze's version of Kallisto directly from the command-line as a test
Try test FASTQ files to make sure something odd about the specific system configuration on your computer is not conflicting with AltAnalyze.

I would suggest #2 first, but before this, I recommend running again, but this time, do not select the ICGS option, denote at least two biological groups and set comparisons in the GUI. When doing so, please send the screen shot of how the samples show up to ensure it is not some hidden character naming that is a problem. Will be also good to see if the same issue occurs. Otherwise, to proceed with number 2, see the below instructions provided with the software:

Input Files for Kallisto-Splice Analysis
The latest version of AltAnalyze introduces a new method to quickly process raw sequencing data (FASTQ files) to directly produce gene expression and alternative splicing estimates, without any additional software on your desktop or laptop computer.
Demo Files
Two zip files with very small FASTQ files for demonstration purposes are available here:
http://altanalyze.org/Data/Hs_GSE45419_FASTQs.zip (344MB - Human downsampled)
http://altanalyze.org/Data/Mm-FASTQ-GSE70245.zip (59MB - Mouse scRNA-Seq)
The Breast Cancer dataset was downsampled from the original fast files as described here: https://www.synapse.org/#!Synapse:syn7286377/files/
The human breast cancer samples correspond to two subtypes (ER-positive and Triple-negative). Unzip these file before proceeding (right click and extract of open and extract to this directory - e.g., WinZip).
Running Kallisto-Splice through the Graphical User Interface

Double-click on the AltAnalyze executable (see Running-AltAnalyze file in the program directory for problems opening).

Install the correct species database when prompted.
a.	For the demo dataset, ensure Homo Sapiens is downloaded. Any version of the database should be compatible (e.g., EnsMart72).

From the main menu in AltAnalyze, select RNA-Seq as the “Select vendor/data type”. Then select the Continue button.

Select the “Process RNA-Seq reads” radio button. Then select the Continue button.

Dataset Location: Enter a dataset name of choice (e.g., Breast_cancer). For the “Select FASTQ files to run in Kallisto” , select the location of the unzipped FASTQ directory. The program will process all FASTQ files in that directory. Select the output directory, which is the folder to save all results and subsequent input files to. Then select the Continue button.

Expression Analysis Parameters: Choose the additional options you want to include or exclude for the pipeline analyses (optional). These include pathway analyses options, which statistical comparison tests to apply for differential expression analysis. If users wish they can select “no” for the option “Perform alternative analysis, which will skip the splicing analyses and process the Kallisto TPM expression file instead of the produced exon-exon junction derived gene RPKM file. When complete, select the Continue button.

Pathway Analysis Options: Here, the user will be prompted to specify the statistical cutoff applied for differential gene and splicing analyses. The adjp indicates an FDR corrected p-value versus a non-corrected p-value.

Alternative Splicing Analysis Options: The default recommended method for splicing analysis is selected (MultiPath-PSI), however, alternative and additional algorithm options are available. When finished, select the “Run Analysis” option.

Groups Designation: Type a label for each FASTQ sample shown (e.g., ERpos, TripleNeg). This will create a “groups.” text file in the output directory folder ExpressionInput. This file will be reloaded when.

Comparisons Designation: Select the experimental and control datasets to compare to (e.g., TripleNeg vs. ERpos). Select “Continue” to run the analysis.

Analysis Progress: A black screen will appear once the analysis has begun. Be patient as the software is performing a series of in-depth analyses, including indexing of the Kallisto transcriptome (run the first time FASTQ files are processed), Kallisto pseudo-alignment to the reference transcriptome, BAM file generation with genome coordinates for all pseudo-aligned reads, gene expression quantification, differential gene expression analysis, QC analysis, network analysis, marker identification, pathway analysis and alternative splicing analysis.

Outputs of Kallisto-Splice
There are a large array of results from this workflow which can be found in the below described folders. Note, a separate PDF file is saved to the root directory describing the files in each of these folders. Please refer to those PDFs for details.

ExpressionInput: This includes all expression estimates for exon-exon junctions, kallisto isofroms and genes as normalized values (TPM and RPKM) and counts. All Kallisto results are saved to the Kallisto_results folder along with the number of percentage of aligned reads.
ExpressionOutput: This folder contains all computed differential gene expression results, primarily found in the DATASET file. The MarkerFinder folder contains the top markers assigned to each sample group (AllGenes_correlations-ReplicateBased.txt file).
DataPlots: This folder contains the majority of saved plots as pdf and png files. Note, that the MarkerFinder folder in this directory contains additional plots. Splicing associated plots will also be saved to the folder AltResults/AlternativeOutput.
AltResults: This directory contains all splicing analysis results. This most important file is “Hs_RNASeq_top_alt_junctions-PSI_EventAnnotation.txt”, which contains all MultiPath-PSI detected splicing events and associated annotations. Statistical comparison results are saved to the “Events-dPSI” folder and splicing graphs to the SashimiPlots folder in the output directory (derived from the BAM files in the output directory.
SashimiPlots: This folder contains the PDF and PNG outputs for genome and exon-exon junction aligned reads associated with example top-significant alternative splicing events. Users can output addition such plots from the Additional Analyses menu option “AltExon Viewer”.
GO-Elite: This folder contains all pathway and gene-set enrichment analysis results. See each folder for the “pruned-results_z-score_elite.txt” (open in Excel or equivalent). Network graphs, heatmaps comparing different comparison groups and optional colored WikiPathways are saved to these directories.

Below is an example command similar to the above analysis:
python AltAnalyze.py --platform "RNASeq" --species Mm —fastq_dir /Users/altanalyze/DemoData/Mm-FASTQ-GSE70245-DownSampled/ --groupdir /Users/altanalyze/DemoData/Mm-FASTQ-GSE70245-DownSampled/groups.Breast_cancer.txt --compdir /Users/altanalyze/DemoData/Mm-FASTQ-GSE70245-DownSampled/comp.Breast_cancer.txt --output /Users/altanalyze/DemoData/Mm-FASTQ-GSE70245-DownSampled/output --expname Breast_cancer --runGOElite yes --returnPathways all


mqtt qos support

Hi,
Wouldn't be nice to have mqtt qos = 1 and/or 2 support?
Looking at the code, it seem to be hardcoded to 0.
Cheers
-Anders

@brocaar I may implement it.
Thoughts?


Review HAWKULAR-67 implementation

Well done Juca! I comment inline:

Dropdown switch: please follow the design specified in hawkular-67 + Manage Organizations link
Spinner loading: use the same approach as Application Servers, for example. The title of the page should appear also only after the spinner.
Create organization: the feedback (toastr) should be success and text: “Organization successfully created.”
Organizations table: since we have a big space, we could list the members as hyperlinks, like:
You (admin), Juraci, Thomas and 1 more
Clicking on one name would go to edit user. Clicking on 1 more would go to the users list screen.
Organization table: tooltip Delete is not showing on the trash icon. Need fix.
When removing an organization, we should have the confirmation dialog like in the urls list. Also, the feedback (toastr) should be success type (not info), and text: “Organization successfully deleted.”
Organization not found: no need for feedback tostr on this page.

Are the changes to the metrics less file intentional? Wouldn't they influence the look and feel of metrics? Apart from that, I'm ok with this PR.

Yes, they are intentional :)

@cardosogabriel looks like this needs to be rebased to master

Merged #512


animation: Refactor AnimatedVariable

Describe your PR, what does it fix/add?
This commit decomposes the AnimatedVariable class into a base class with the common attribute to all variable types and a templated derived type containing strongly typed info on the type being animated.
Access to the typed version is perfomed using the visitor pattern. A utility is provided to build a visitor on the fly using lambdas.
Adding a new type to be animated should just be a matter of adding the typed in the list defined by the ANIMABLE_TYPES macro
The size of the commit is justified by the API change in the AnimatedVariable class. No more vec(), fl() or col() method but a unified value() method.
This PR reduces code duplication in the AnimatedVariable class and in the AnimationManager. Moreover, it improves it's extensibility to other types.
The tradeoff is a bit more complexity with all the templated arguments in AnimatedVariable.cpp but less code duplication and smaller memory footprint for these variables.
Note that the added complexity does't leak to much on the code that uses these variables.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
This can break compatibility as it changes the way to declare and use AnimatedVariables.
Is it ready for merging, or does it need work?
It's ready to merge. I'm running this branch to test it while I'm redaction this PR.

A few comments:
* why split the Base out?
To me, all the code you wrote in the linked snippet (L33-L83) looks like useless garbage

The base class exists so we can store CBaseAnimatedVariable* in the AnimationManager. And  it's also easier to implement the common methods in non-templated class. The benefit of separating then is to isolate common code under the base class but still consider AnimatedVariable<float> and AnimatedVariable<Vector2D> as different types at compile time and not at runtime.
The snippet of code you linked defines which types can be animated (Vector2D, float and CColor). Then it defines a visitor class that have an empty visit method for each possible types. Then the CAnimatedVisitorOverload allows construction of the visitor with lambdas. All this code is used to safely down cast CBaseAnimatedVariable* into CAnimatedVariable<T>& in this snippet : https://github.com/GartoxFR/Hyprland/blob/dccd0f5c1cf17e80673172b83896e9f914b35609/src/managers/AnimationManager.cpp#L116-L143

In summary, the visitor pattern is a bit of a glorified switch statement that uses virtual functions to downcast pointers when needed.

doesn't std have std::visit?
Why isn't that used?
Generally the visit-related code is 100% unreadable to me, and thus I can't accept a pattern like this as I can't maintain it later. I'd rather have just the template and a switch later without the entire visitor stuff.

A few comments:
* why split the Base out?
To me, all the code you wrote in the linked snippet (L33-L83) looks like useless garbage

The base class exists so we can store CBaseAnimatedVariable* in the AnimationManager. And  it's also easier to implement the common methods in non-templated class. The benefit of separating then is to isolate common code under the base class but still consider AnimatedVariable<float> and AnimatedVariable<Vector2D> as different types at compile time and not at runtime.
The snippet of code you linked defines which types can be animated (Vector2D, float and CColor). Then it defines a visitor class that have an empty visit method for each possible types. Then the CAnimatedVisitorOverload allows construction of the visitor with lambdas. All this code is used to safely down cast CBaseAnimatedVariable* into CAnimatedVariable<T>& in this snippet : https://github.com/GartoxFR/Hyprland/blob/dccd0f5c1cf17e80673172b83896e9f914b35609/src/managers/AnimationManager.cpp#L116-L143

In summary, the visitor pattern is a bit of a glorified switch statement that uses virtual functions to downcast pointers when needed.

doesn't std have std::visit?
Why isn't that used?
Generally the visit-related code is 100% unreadable to me, and thus I can't accept a pattern like this as I can't maintain it later. I'd rather have just the template and a switch later without the entire visitor stuff.

std::visit is used for variant so it's a bit different as it doesn't use any Base class.
I proposed this pattern because I find it more readable than a switch as it's closer to pattern marching I can find in other languages but this is just a personal opinion.
I can replace the visitor pattern by adding back an enum variable to tell at runtime which type to cast to. It's basically a matter of using this kind of variables or v-tables to tell which type we have.

I can replace the visitor pattern by adding back an enum variable to tell at runtime which type to cast to. It's basically a matter of using this kind of variables or v-tables to tell which type we have.

please do. Much more readable

please do. Much more readable

Done. This needed some template to associate the types with the enum constants. I hope it's not to unreadable

oops seems like clang-format is unhappy, can you slap the mr with it?

Of course, editing this much files I forgot pressing the format button on some of them. Should be fixed now


Implement 1-to-1 hashing for BLAKE3 module in Miden stdlib

Currently, we can hash using BLAKE3 or Sha256; see here. Both hashes have 64-bytes as input and 32-bytes as output.
However, when we want to compute multiple hashes in a row, e.g. hash(hash(a,b), b)) or hash(hash(a,b)), we need the ability to hash using a 32-byte input as well.

https://github.com/maticnetwork/miden/pull/563


Set Username and Password at Run time

Hi,
Is it possible to define username and password properties at runtime with env variables?
Let me explain furthermore. Here is what I'm trying to achieve:
// https://nuxt.com/docs/api/configuration/nuxt-config
export default defineNuxtConfig({
  runtimeConfig: {
    basicAuthUsername: "hardcoded-username", // Will be replace by NUXT_BASIC_AUTH_USERNAME at runtime
    basicAuthPassword: "hardcoded-password", // Will be replace by NUXT_BASIC_AUTH_PASSWORD at runtime
  },

  basicAuth: {
    users: [
      {
        username: useRuntimeConfig().basicAuthUsername,
        password: useRuntimeConfig().basicAuthPassword,
      },
    ],
  },

});

However, useRuntimeConfig is not defined inside the nuxt.config.ts file.
If I switch to process.env. NUXT_BASIC_AUTH_XXX it is not replaced at run time and keep the value of build time.
Do you know any solution / workaround to this issue?
Thanks for your help !
Thib

Fixed in: fc0a6f2


Adds Vagrant molecule

Adds vagrant molecule to make locally testing with vagrant much easier and cleaner
Requires molecule to be installed

I think builds failed because upstream pushed a commit right in between the deploy and idempotency test. Lucky.

Forgot to mention that Ive disabled lint checks in this PR as was throwing up a number or errors throughout the code which is out of scope of the PR to fix them.
Lint tests can be re-enabled in molecule.yml under their relevant sections
e.g.
lint:
  name: yamllint
  enabled: true

provisioner:
  name: ansible
  log: true
  lint:
    name: ansible-lint
    enabled: true

Can you add a DEVELOPING.md that discusses briefly how to setup and run this test environment?
The .gitignore is pretty unwieldy.
Does molecule/ need to be a top level directory or does it make sense to move it into the tests/ directory?

@lae can do not a problem.
I develop in vscode and its part of my standard .gitignore so I don't commit anything I shouldn't but can be removed.
molecule needs to be as it is currently set as when running tests you run from the root of the role.
e.g. molecule test
All tests reside in the molecule/scenarios/test, in this case a single "default" scenario.
Other scenarios can be added at anytime in the future using molecule init scenario .....

@lae please review DEVELOPING.md to see it its what you want

Commits were cherry-picked and merged.


Upgrades Oracle database libraries to version <IP_ADDRESS>. Adds test under datasource-ucp CDI integration.

This PR updates the Oracle database library versions to <IP_ADDRESS>.
It also ensures that we are using the ojdbc11 family of artifacts, at least in the datasource-ucp CDI integration, which are the ones that should be used with JDK 11+ (or 21, which is the base of Helidon 4). See https://www.oracle.com/database/technologies/appdev/jdbc-downloads.html for more information.
It also adds a test resulting from a very lengthy internal discussion that shows that the Universal Connection Pool does not do anything to reset any underlying connection state when a borrowed connection is checked out, modified, and returned to the pool. This is useful information going forward and we want this test to fail/alert us if UCP addresses this shortcoming in any way.

What about the other areas that ojdbc8-production is used?

Sure, I'll take care of those.

Hmm; might also need to decipher https://www.oracle.com/database/technologies/appdev/jdbc-downloads.html

Hmm; UCP is dying:
21:50:05,662 [INFO] Caused by: oracle.ucp.UniversalConnectionPoolException: Universal Connection Pool internal error: java.lang.NullPointerException: Cannot invoke "Object.hashCode()" because "key" is null
21:50:05,662 [INFO] 	at oracle.ucp.util.UCPErrorHandler.newUniversalConnectionPoolException(UCPErrorHandler.java:379)
21:50:05,662 [INFO] 	at oracle.ucp.util.UCPErrorHandler.throwUniversalConnectionPoolException(UCPErrorHandler.java:69)
21:50:05,662 [INFO] 	at oracle.ucp.util.UCPErrorHandler.throwUniversalConnectionPoolException(UCPErrorHandler.java:92)
21:50:05,662 [INFO] 	at oracle.ucp.util.UCPErrorHandler.throwUniversalConnectionPoolException(UCPErrorHandler.java:165)
21:50:05,662 [INFO] 	at oracle.ucp.common.Core.growBorrowed(Core.java:1316)
21:50:05,662 [INFO] 	at oracle.ucp.common.UniversalConnectionPoolImpl.helpGrowBorrowed(UniversalConnectionPoolImpl.java:407)
21:50:05,662 [INFO] 	at oracle.ucp.common.UniversalConnectionPoolImpl.borrowConnectionWithoutCountingRequests(UniversalConnectionPoolImpl.java:350)
21:50:05,662 [INFO] 	at oracle.ucp.common.UniversalConnectionPoolImpl.borrowConnectionAndValidateHelper(UniversalConnectionPoolImpl.java:206)
21:50:05,662 [INFO] 	at oracle.ucp.common.UniversalConnectionPoolImpl.borrowConnectionAndValidate(UniversalConnectionPoolImpl.java:166)
21:50:05,662 [INFO] 	at oracle.ucp.common.UniversalConnectionPoolImpl.borrowConnection(UniversalConnectionPoolImpl.java:139)
21:50:05,662 [INFO] 	at oracle.ucp.jdbc.JDBCConnectionPool.borrowConnection(JDBCConnectionPool.java:185)
21:50:05,662 [INFO] 	at oracle.ucp.jdbc.PoolDataSourceImpl.getConnection(PoolDataSourceImpl.java:1882)

The NullPointerException bug in UCP appears in version <IP_ADDRESS> and <IP_ADDRESS>. I am therefore editing this PR to upgrade to <IP_ADDRESS> which is the highest version that does not exhibit this behavior.
A workaround, should it become necessary, is to invoke the private PoolDataSource#setServiceName(String) method via reflection and supply it a dummy non-null service name. (This is already handled in the UCPBackedDataSourceExtension if the user has specified a serviceName configuration property.)


Update spectral_cube.py

I think this fixes the ability to pass kwargs to the convolution routine in convolve_to()

Coverage remained the same at 89.017% when pulling 9e8f11eb3ba01bc20f11079c4eae5ed21689c956 on akleroy:master into 37577e898ddcd877520484cdcb6784433a2122f4 on radio-astro-tools:master.

This change now passes kwargs from convfunc to convolve instead of taking the kwargs from convolve_to and passing them directly to convolve.  I think the behavior will end up exactly the same in both cases unless apply_function_parallel_spatial passes additional kwargs.  I don't think it does (or should).
@e-koch or @astrofrog, want to chime in here?  I'm not sure there's any difference, but this is a little subtle and I could be missing something.
@akleroy was this motivated by any observed behavior that was causing a problem?  You didn't mention a specific issue in #630, but if this was causing a problem, we need to add a test so we can prevent a regression.

I think none of the convolution kwargs were being passed (e.g., allow_huge, nan_treatment, etc).
And the kwargs weren't being passed here but inside apply_function_parallel_spatial. The ordering seems to be used elsewhere so this seems fine to me: https://github.com/radio-astro-tools/spectral-cube/blob/190df4216538cd65b11ebb1fb07c261482b4a9e8/spectral_cube/spectral_cube.py#L2589
We should test this for SpectralCube, VaryingResSpectralCube, and their equivalent Dask counterparts, though.

Why wouldn't allow_huge, nan_treatment, etc. get passed in the current code?  convolve is taking **kwargs directly from convolve_to, so if you specified those, they should get passed onto convolve, no?

I thought so, but I guess it's because the function doesn't get called until it's wrapped in apply_function_parallel_spatial. I can reproduce the issues @akleroy was having:
In [3]: outcube1 = cube.convolve_to(Beam(3 * u.arcsec), nan_treatment='fill')
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-3-29a0b370cc7e> in <module>
----> 1 outcube1 = cube.convolve_to(Beam(3 * u.arcsec), nan_treatment='fill')

~/ownCloud/code_development/radio_astro_tools/spectral-cube/spectral_cube/utils.py in wrapper(self, *args, **kwargs)
     47                           PossiblySlowWarning
     48                          )
---> 49         return function(self, *args, **kwargs)
     50     return wrapper
     51

~/ownCloud/code_development/radio_astro_tools/spectral-cube/spectral_cube/spectral_cube.py in convolve_to(self, beam, convolve, update_function, **kwargs)
   3127
   3128         newcube = self.apply_function_parallel_spatial(convfunc,
-> 3129                                                        **kwargs).with_beam(beam)
   3130
   3131         return newcube

~/ownCloud/code_development/radio_astro_tools/spectral-cube/spectral_cube/spectral_cube.py in apply_function_parallel_spatial(self, function, num_cores, verbose, use_memmap, parallel, **kwargs)
   2849                                                   num_cores=num_cores,
   2850                                                   use_memmap=use_memmap,
-> 2851                                                   **kwargs)
   2852
   2853     def apply_function_parallel_spectral(self,

~/ownCloud/code_development/radio_astro_tools/spectral-cube/spectral_cube/spectral_cube.py in _apply_function_parallel_base(self, iteration_data, function, applicator, num_cores, verbose, use_memmap, parallel, memmap_dir, update_function, **kwargs)
   2767                                                               function,
   2768                                                               **kwargs)
-> 2769                                           for arg in iteration_data)
   2770             except ImportError:
   2771                 if num_cores is not None and num_cores > 1:

~/anaconda3/lib/python3.6/site-packages/joblib/parallel.py in __call__(self, iterable)
    981             # remaining jobs.
    982             self._iterating = False
--> 983             if self.dispatch_one_batch(iterator):
    984                 self._iterating = self._original_iterator is not None
    985

~/anaconda3/lib/python3.6/site-packages/joblib/parallel.py in dispatch_one_batch(self, iterator)
    823                 return False
    824             else:
--> 825                 self._dispatch(tasks)
    826                 return True
    827

~/anaconda3/lib/python3.6/site-packages/joblib/parallel.py in _dispatch(self, batch)
    780         with self._lock:
    781             job_idx = len(self._jobs)
--> 782             job = self._backend.apply_async(batch, callback=cb)
    783             # A job can complete so quickly than its callback is
    784             # called before we get here, causing self._jobs to

~/anaconda3/lib/python3.6/site-packages/joblib/_parallel_backends.py in apply_async(self, func, callback)
    180     def apply_async(self, func, callback=None):
    181         """Schedule a func to be run"""
--> 182         result = ImmediateResult(func)
    183         if callback:
    184             callback(result)

~/anaconda3/lib/python3.6/site-packages/joblib/_parallel_backends.py in __init__(self, batch)
    543         # Don't delay the application, to avoid keeping the input
    544         # arguments in memory
--> 545         self.results = batch()
    546
    547     def get(self):

~/anaconda3/lib/python3.6/site-packages/joblib/parallel.py in __call__(self)
    259         with parallel_backend(self._backend):
    260             return [func(*args, **kwargs)
--> 261                     for func, args, kwargs in self.items]
    262
    263     def __len__(self):

~/anaconda3/lib/python3.6/site-packages/joblib/parallel.py in <listcomp>(.0)
    259         with parallel_backend(self._backend):
    260             return [func(*args, **kwargs)
--> 261                     for func, args, kwargs in self.items]
    262
    263     def __len__(self):

~/ownCloud/code_development/radio_astro_tools/spectral-cube/spectral_cube/spectral_cube.py in _apply_spatial_function(arguments, outcube, function, **kwargs)
    161
    162     if np.any(includemask):
--> 163         outcube[ii, :, :] = function(img, **kwargs)
    164     else:
    165         outcube[ii, :, :] = img

TypeError: convfunc() got an unexpected keyword argument 'nan_treatment'

OK, I see - kwargs is getting passed both ways.
I think this PR is a fix, so we should go ahead with it, but the test you've shown is needed too.

Agreed. But we should check if the fix is needed for all the convolve_to functions here, too. I'm happy to do so but I'll be delayed by a couple days.

yes, sorry, I mean we should go ahead w/expanding this PR to cover the other cases

Tests are passing. The failure is from me accidentally pushing a branch to upstream (deleted now).

Thanks @e-koch!  Thanks @akleroy!  Glad to see you're using it.


2.5.0beta 1 : undefined method `set_searchable_param_defaults

When using
source 'https://rubygems.org'

gem 'rake'
gem 'ProMotion', git: 'git@github.com:clearsightstudio/ProMotion.git'
gem 'redpotion'

I got the following
Application Specific Information:
*** Terminating app due to uncaught exception 'NameError', reason: 'undefined method `set_searchable_param_defaults' for module `ProMotion::Table::Searchable' (NameError)
'
terminating with uncaught exception of type NameError
abort() called
CoreSimulator 117.15 - Device: iPad 2 - Runtime: iOS 8.4 (12H141) - DeviceType: iPad 2

Application Specific Backtrace 1:
0   CoreFoundation            0x04516746 __exceptionPreprocess + 182
1   libobjc.A.dylib           0x01729a97 objc_exception_throw + 44
2   MyApp                     0x00be2e06 _ZL10__vm_raisev + 422
3   MyApp                     0x00be2f48 rb_vm_raise + 216
4   MyApp                     0x00adf791 rb_exc_raise + 17
5   MyApp                     0x00ad8ed7 rb_name_error + 135
6   MyApp                     0x00adedb7 rb_print_undef + 343
7   MyApp                     0x00bdaaeb _ZL8vm_aliasmmm + 683
8   MyApp                     0x00bdab67 rb_vm_alias + 23
9   MyApp                     0x00bb07f4 rb_mod_alias_method + 52
10  MyApp                     0x00bd67b4 _ZL20dispatch_rimp_callerPFP11objc_objectS0_P13objc_selectorzEmS1_iPKm + 46452
11  MyApp                     0x00bbcf83 rb_vm_dispatch + 7379
12  MyApp                     0x0010c2bd vm_dispatch + 1373
13  MyApp                     0x005c88c9 rb_scope4 + 105
14  MyApp                     0x005c87dd rb_scope3 + 125
15  MyApp                     0x005c86dd rb_scope2 + 125
16  MyApp                     0x005c85da rb_scope1 + 122
17  MyApp                     0x005cb60f MREP_E9356F4D9958475AA9391DE144536853 + 3183
18  MyApp                     0x001165a9 RubyMotionInit + 5225
19  MyApp                     0x00117b9d main + 109
20  libdyld.dylib             0x064afac9 start + 1

The problem disappear when using
[clip]
gem 'redpotion', git: 'git@github.com:infinitered/redpotion.git'

After a quick search, this makes sense with https://github.com/infinitered/redpotion/blob/v1.4.0/lib/project/pro_motion/data_table_searchable.rb#L6 which as been reworked since 1.4.0
I guess RedPotion will have to bump a release when PM 2.5.0 will be available.
cc @twerth

Yup. This is a redpotion specific issue. We need to release a new version with the fixes in master asap now that CDQ has been fixed to lock motion-yaml to 1.4

Okay, I'll leave this open until RedPotion is fixed.

Any ETA on the fix of this?  Just updated my gems, and it broke an app that I am working on.
Gems that were updated on my system:
Gems updated: ProMotion actionmailer actionpack actionview activejob activesupport activemodel activerecord afmotion autoprefixer-rails aws-sdk aws-sdk-core aws-sdk-resources bootstrap-sass bson bson_ext cocaine command_runner_ng font-awesome-rails fuzzy_match geocoder httpclient mime-types mime-types-data mongoid motion-cocoapods motion-yaml mysql2 netrc nokogiri paperclip power_assert rack-cache rails railties redis redis-activesupport redis-rack-cache redis-store rspec-core rspec-support rspec-expectations rspec-mocks rspec-rails run_loop shoulda-matchers simple_token_authentication slop sprockets unicorn unicorn-worker-killer

any ETA on this? Just updated my gems, and it broke an app I am working on.  Just happened to update RedPotion to 1.4.0.

The ProMotion update to 2.5.0 killed my app as well accordingly. Of course I can stick to 2.4.2; nevertheless ...

That's what I ended up doing is adding the ProMotion gem to my gemfile locked at 2.4.2.  Be nice to get this fix in now that PM 2.5 is available

As per a suggestion from @bmichotte, I just added this to my gemfile:
gem "redpotion", git: "https://github.com/infinitered/redpotion.git"
And then bundled and that fixed it for now.

@bmichotte @Atastor @ch0ke @bmichotte I'm planning on cutting a release today so you can go back to a released gem

Thanks @squidpunch !

RMQ and redpotion released, let me know if this fixes the issue guys!

That fixed it for me. This might be obvious, but I had to bundle update redpotion and then rake.
Thanks again for the quick fix, @squidpunch.

:+1:

:100:


Add optional expected content type to URL

This would be a very useful property for certain URLs, and would help avoid some ugly workarounds.

@apdavison note that this would be best followed by one or two ContentType instances for URLs


docs: Cleanup backticks in version_history

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message:
Additional Description:
Not sure why CI is not picking these up - i thought we added something previously
This PR just cleans up the current backtick issues in version history - i will investigate further why they are being missed
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

related #20506


feat: Update artifacthub-pkg.yml automatically from now on, 2nd try

Description
Add reusable test workflow for the rust policy

Fix: https://github.com/kubewarden/allowed-proc-mount-types-psp-policy/pull/24#issuecomment-1476667228

Everything is fine, besides the "check artifacthub" job failing because artifacthub-pkg.yml has 1 empty line more:
https://github.com/kubewarden/allowed-proc-mount-types-psp-policy/actions/runs/4471541590/jobs/7856585269?pr=27#step:5:98
This I suspect is happening because the GitHub Actions is using kwctl v1.6.0-rc4, meanwhile v1.6.0-rc3 was used locally when running the make targets. Such an annoyance, and not fair to you @geeksambhu.
Merging this! Consider it done. For follow-up PRs, update kwctl to rc4.
Congrats for your first PR merged to the project! :smile:

Tagged in main as v0.1.7.


I140 extending statistics to fill whitespace on home page

Change Summary

Added 2 features in the home page per Figma - (i) Number of Hours and (ii) Distance Commuted

Change Form
Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.

[Y] The pull request title has an issue number
[N] The change works by "Smoke testing" or quick testing
[N] The change has tests
[N] The change has documentation

Other Information
NA
Related Issue

Resolve #140

Looks like you forgot to run prettier.
Run yarn format:check in terminal to fix it :)

Failing as files that need to be deleted will be deleted in #139 when it is merged to main.


emojis: Change emoji picker to category-based.

This now changes the emoji picker to separate all of the emojis
into distinct categories that they can be filtered by.
Thanks to @bkiu for a majority of the work to make this possible! :D

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!

@HarshitOnGitHub can you review this?

@HarshitOnGitHub since I think other folks are pretty busy, would you be up for doing a pass on fixing those issues in this PR and getting the tests passing?  That would be very helpful.

@timabbott Sure! :slightly_smiling_face:

I have opened #5608 for further work on this so this can be closed.

Closing in favor of #5608.


Multiple tasks in one group supported?

I'm trying to deploy an application that consists of two IIS applications. I want them on the same host, so I grouped them. It looks like there is a racing condition going on: One site is created with configuration of one task, but the configuration is swiftly replaced by the other configuration of the other task.
So does the IIS driver support multple win_iis tasks in the same group?

This definitely appears to be a bug from your statement. Mind giving me an example nomad job spec and I can find some time to add it to the list of tests the driver should do and see if I can reproduce.

job "multiple-tasks" {
  datacenters = ["dc1"]
  type = "service"

  group "group" {
    count = 1

    network {
      port "http" {
        static = 80
      }
    }

    restart {
      attempts = 10
      interval = "5m"
      delay = "25s"
      mode = "delay"
    }

    task "one" {
      driver = "win_iis"

      config {
        path = "C:\\inetpub\\wwwroot"

        bindings {
          type = "http"
          hostname = "one.localhost"
          port = "http"
        }
      }
    }

    task "two" {
      driver = "win_iis"

      config {
        path = "C:\\inetpub\\wwwroot2"

        bindings {
          type = "http"
          hostname = "two.localhost"
          port = "http"
        }
      }
    }
  }
}

The job spec above gives me one Application pool and one Web site. Like I said, I briefly get a binding to one.localhost which in turn becomes a binding to two.localhost.


ARRAY : Inverse of an array

💥 Proposal
This is very standard problem, in which we swaps the indices of the array with element.
This type of basic questions can make the users understand of the feel and concept of array.
Have you read the [Contributing Guidelines on Pull Requests]
YES
I will be implementing the problem code in CPP with proper comments and explaination along with test cases.
code will be mine , I can guarantee that.

I am a GSSOC21 participant ,I want to contribute to this issue in c++.please assign this issue to me

please assign this task to me in java under GSSOC'21

Greetings sir/Mam,
I would like to contribute the code for Bubble Sort Algorithm #933 in C++/CPP.
The code is ready. I have added the code , along with proper explanation of the algorithm , test cases , time complexity and space complexity of the same, as per the guidelines.
Kindly assign it to me.
This is regarding GSSOC'21.
Regards,
Hitain Kakkar

/assign

please assign it to me @Harsh Mishra
On Mon, Mar 15, 2021 at 7:05 PM Harsh Mishra @.***>
wrote:

Reopened #3043 https://github.com/TesseractCoding/NeoAlgo/issues/3043.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TesseractCoding/NeoAlgo/issues/3043#event-4458752046,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASZVY2BLFJAAPGCBS7IO7BTTDYEL3ANCNFSM4ZEIVCIQ
.

please assign this issue to me, i would like to work in cpp.


Added unit tests for Step Adapters: map, raw and tee.

Added some basic unit tests for the remaining Step Adapters.

Thanks!


Move default fonts to new crate epaint_default_fonts

This allows license checking tools to omit the OFL and UFL licenses when default_fonts are turned off.
There was some discussion of versioning on the original issue; I have chosen to label this version as 0.28.1 to match the other crates. Happy to adjust the version as needed.

Closes https://github.com/emilk/egui/issues/2321
[X] I have followed the instructions in the PR template

We should also add epaint_default_fonts to https://github.com/emilk/egui/blob/64d5f948e43b4be93e3a4541c87dcce74c948df4/.github/workflows/labels.yml#L32

Ok, comments addressed!

Thanks!


gem error with rbenv

I can use gem in system version but when I change the version to 2.2.0.
>  gem upgrade --system
ERROR:  While executing gem ... (Gem::CommandLineError)
    Unknown command upgrade

> gem install rake
ERROR:  Could not find a valid gem 'rake' (>= 0), here is why:
          Unable to download data from https://rubygems.org/ - no such name (https://api.rubygems.org/specs.4.8.gz)

But with system version, this does not happen.

How do you feel that this is a bug with rbenv, which only switches Ruby versions and isn't responsible for them working?
How did you install Ruby 2.2.0?
What is your gem --version on Ruby 2.2.0?
Does the same error happen every time?

I do not know actually what causes this problem.
rbenv install 2.2.0.
gem --version 2.4.5
Yes, It happens every time

Same here with rbenv install 2.2.2

I have googled the error message; it seems pretty common and smells like a networking issue. It is raised when the DNS entry for a hostname to fetch gemspecs from couldn't be resolved.
I really don't think this is caused by rbenv or Ruby installation in any way. Please try to check your DNS settings and HTTP proxy settings. I'm sorry I couldn't be of more help.


Comparison should be done with table name variable not with string literal

Comparison should be done with table name variable not with string literal

Some more work needed


Can't dump multisession discs

Version
I have latest test version
Describe the bug
"Failed to get (Track 1) (-LBA)" message appears when trying to dump a multisession disc

Max Kids.zip

Try audio command and upload logs, plz.
audio F Max Kids.bin 72 -5000 -1150

Here you go. Dump hasn't even started so I'm uploading what I created LOL
Max Kids.zip

DiscImageCreator_test.zip

deleted: /ms and the related code

It still doesn't work as I expected
Yes BTW: Can Plextor read lead-in in Track AA? Because I feel like not much
Max Kids.zip

Does other multi-session disc also output the same error?


Server basic management

model a server

name
image
owner
unique id

model a user

username
mail (unique reference)
unique id

crud

[x] create
[x] read
[x] update
[x] delete

link user to servers

[x] method to generate invite link from server
[x] invite link adds a user to a server

[x] in server add users added
[x] in user add server linked

[x] leave server

delete server can be done without owner permission and does not remove the links in the users, same think if i delete a user
to solve this problems we need authentication and authorization that will be maked in another task, closing because is just a preliminary step


Unable to change uploads path from storage to public

Hello!
I am trying to change the path of the uploads directory from from storage to public but it does not work.
I've add a this string in the .env file
MEDIA_LIBRARY_LOCAL_PATH=public/uploads/

also, I change filesystem.php from storage_path() to public_path(), but is does not have result
`<?php
return [
/*
|--------------------------------------------------------------------------
| Default Filesystem Disk
|--------------------------------------------------------------------------
|
| Here you may specify the default filesystem disk that should be used
| by the framework. The "local" disk, as well as a variety of cloud
| based disks are available to your application. Just store away!
|
*/

'default' => env('FILESYSTEM_DRIVER', 'local'),

/*
|--------------------------------------------------------------------------
| Default Cloud Filesystem Disk
|--------------------------------------------------------------------------
|
| Many applications store files both locally and in the cloud. For this
| reason, you may specify a default "cloud" driver here. This driver
| will be bound as the Cloud disk implementation in the container.
|
*/

'cloud' => env('FILESYSTEM_CLOUD', 's3'),

/*
|--------------------------------------------------------------------------
| Filesystem Disks
|--------------------------------------------------------------------------
|
| Here you may configure as many filesystem "disks" as you wish, and you
| may even configure multiple disks of the same driver. Defaults have
| been setup for each driver as an example of the required options.
|
| Supported Drivers: "local", "ftp", "sftp", "s3", "rackspace"
|
*/

'disks' => [
    'local' => [
        'driver' => 'local',
        'root' => public_path(),
    ],

    'public' => [
        'driver' => 'local',
        'root' => public_path(),
        'url' => env('APP_URL').'/public',
        'visibility' => 'public',
    ],

    's3' => [
        'driver' => 's3',
        'key' => env('AWS_ACCESS_KEY_ID'),
        'secret' => env('AWS_SECRET_ACCESS_KEY'),
        'region' => env('AWS_DEFAULT_REGION'),
        'bucket' => env('AWS_BUCKET'),
        'url' => env('AWS_URL'),
    ],
],

];
`

What is your 'disk' value set to in the twill.php config?   For me that was the most important change.  You can see how I've customised mine...

'media_library' => [
        'disk' => 'twill_media_library',
        'endpoint_type' => env('MEDIA_LIBRARY_ENDPOINT_TYPE', 'local'),
        'cascade_delete' => env('MEDIA_LIBRARY_CASCADE_DELETE', true),
        'local_path' => env('MEDIA_LIBRARY_LOCAL_PATH','/'),
        'image_service' => env('MEDIA_LIBRARY_IMAGE_SERVICE', 'A17\Twill\Services\MediaLibrary\Glide'),
        'acl' => env('MEDIA_LIBRARY_ACL', 'private'),
        'filesize_limit' => env('MEDIA_LIBRARY_FILESIZE_LIMIT', 50),
        'allowed_extensions' => ['jpg', 'jpeg', 'png'],
        'init_alt_text_from_filename' => true,
    ],
    'glide' => [
    	'base_url' => '',
    	'base_path' => '/storage/',
        'cache_path_prefix' => '/storage/',
        'cache' => '/test'
    ],
    'file_library' => [
        'disk' => 'twill_file_library',
        'endpoint_type' => env('FILE_LIBRARY_ENDPOINT_TYPE', 'local'),
        'cascade_delete'=> env('FILE_LIBRARY_CASCADE_DELETE', true),
        'local_path' => '/',
        'file_service' => env('FILE_LIBRARY_FILE_SERVICE', 'A17\Twill\Services\FileLibrary\Disk'),
        'acl' => env('FILE_LIBRARY_ACL', 'private'),
        'filesize_limit' => env('FILE_LIBRARY_FILESIZE_LIMIT', 50),
        'allowed_extensions' => ['pdf'],
    ],


API proposal

Proposed API from our meeting to not forget it.
(I hope we can make it in the future.)
def test_something():
    """
    save input->output mapping to the file on the first test run
    use the mapping when running for a second time
    (of the whole test, not a particular method)
    """
    requre.record_method("modul.method")

def test_something2():
    """
    same as above but with context manager
    """
    with requere.record_method("modul.method"):
        calls_recorded()
    calls_not_recorded()

def test_object():
    """"
    `module.class` will be replaced by `MySubclass`
    that will record its functionality using the implemented
    methods from `OurRecorderClass`
    """
    requre.record_class("module.class",  MySubclass)

class MySubclass(Origin, OurRecorderClass)
    """
    Implement recording functionality on your own.
    (implement the neccessary methods from OurRecorderClass)
    """
    pass

def test_koji():
    """
    Some usefull replacements/recording can be turned with some function from `requre.helpers` module.
    """
    requere.helpers.record_koji()

please describe the calls what they mean

please describe the calls what they mean

I've only copied the notes from our architecture-meeting. These are only poor usage examples written some weeks ago.
Please, feel free to edit them to fit your needs. (How we want to use this library.)

edited

We can close it, as it is already solved by new decorators for methods and classes.


Use base tag in index.html to allow for relative urls

We send index.html to the browser to bootstrap the app no matter what route within the app the user loads. Hence, to use relative urls we must update the base against which urls in the index page are resolved to use the root path where the app is hosted, not the route from which the page was served.
On the server, we modify the base href appropriately before serving the page by manipulating the page DOM (previously this was done by injecting javascript, but the DOM manipulation is much cleaner). We also fix up the way this is handled on the client-side to be consistent with how the browser itself resolves relative URLs.

@joshmutus

Hey @maffoo can you provide me an example of how this worked before and how this works now? I'm having a little trouble understanding this from the PR description.
For example when I request https://SERVER/registry/Jim/AliceBobRes/Cooldown151230/160222/Bc/ what is happening different than before?

With the URL you provided, nothing is different. The idea here is I might have a single webserver where I want to host two copies of scalabrad-web (say, one running the latest release and one running a dev version for testing). I want to host the release version at the root url path so that people will see that by default when going to, say https://my.server/registry/... but I also want to host the dev version at another path like /dev/ so people can see it by going to https://my.server/dev/registry/.... The change here makes this possible. You would just have to run the dev server with the option --url-prefix=/dev/. (You would also have to run the dev server on a different port and set up a proxy server to forward the paths appropriately, but that is a different story).
The reason this didn't work before is that the index.html page had absolute URLs for things like javascript and html imports, so you could load the initial page from https://my.server/dev/registry/... but it would then load additional resources like javascript from the absolute URLs which would get the versions from the server running on the root path.
Does that help?

@maffoo OK, I understand that, how do I test it? Say I have two builds I'm running off the same server what uh, do I do with gulp to run them both at the same time?

I'm assuming there are commits to follow that deal with handling and labelling the versioning of the dev servers? It looks good. I'm able to bounce back and forth between the two versions of the software. I'm curious how I'll go about deploying my own dev-servers etc, it would be great to go through some together and create some instructions.

LGTM


Allow passing several anchor values to Plot.frame

It would be cool if I could do
Plot.frame({anchor: ["left", "bottom"]})

instead of:
Plot.frame({anchor:"left"}),
Plot.frame({anchor:"bottom"}),

This proposal worries me a bit in the sense that, in most other marks, passing an array of values to an option means that we consider the value to be a channel. In that sense, it might be more consistent if we accepted a string representation ("top-left", "top-bottom", etc.), instead of an array for multiple sides.

Related, I like the following pattern:
  ["left", "bottom"].map((anchor) => Plot.frame({anchor}))

or, a bit more complex:
d3.groups(data, d => d.value > 10).map(
  ([large, data]) => Plot.text(data, {
    x: "value",
    y: "type",
    textAnchor: large ? "start" : "end",
    dx: large ? +4 : -4
  })
)

I agree with Fil. We could allow an anchor of bottom-left, but I don’t think we would allow anchor to be specified as an array.


Call to undefined method Endroid\QrCode\Builder\Builder::create()

I'm trying to make Qr Code of links provided by user,
Method: if (isset($validatedData['Permit_link'])) {
            $qrCodeImage = Builder::create()
                ->writer(new PngWriter())
                ->data($validatedData['Permit_link'])
                ->size(300)
                ->build();

                $fileName = 'QrCode_' . time() . '.png';
                $directory = 'uploads/qrcode_images';

            Storage::disk('public')->put($directory . '/' . $fileName, $qrCodeImage->getString());

            $validatedData['Permit_qr_link'] = $directory . '/' . $fileName;
        }

its working perfectly on localhost but on liver server, its displaying me this error: "Internal Server Error: Call to undefined method Endroid\QrCode\Builder\Builder::create() "

Hi @Saad-Ramzan-Kiani what version are you using to your composer.lock file?

"require": {
"php": "^8.2",
"endroid/qr-code": "^5.0.7",

@Saad-Ramzan-Kiani this is the requirement, what does your lock file say? Also, please look at the Builder.php file on the server and let me know what it shows.

my composer.lock file: ""name": "endroid/qr-code",
"version": "5.0.7",
"source": {
"type": "git",
"url": "https://github.com/endroid/qr-code.git",
"reference": "0cc00f0626b73bc71a1ea17af01387d0ac75e046"
},
"dist": {
"type": "zip",
"url": "https://api.github.com/repos/endroid/qr-code/zipball/0cc00f0626b73bc71a1ea17af01387d0ac75e046",
"reference": "0cc00f0626b73bc71a1ea17af01387d0ac75e046",
"shasum": ""
},
"require": {
"bacon/bacon-qr-code": "^2.0.5",
"php": "^8.1"
},
"conflict": {
"khanamiryan/qrcode-detector-decoder": "^1.0.6"
},
"require-dev": {
"endroid/quality": "dev-main",
"ext-gd": "*",
"khanamiryan/qrcode-detector-decoder": "^1.0.4||^2.0.2",
"setasign/fpdf": "^1.8.2"
},
"suggest": {
"ext-gd": "Enables you to write PNG images",
"khanamiryan/qrcode-detector-decoder": "Enables you to use the image validator",
"roave/security-advisories": "Makes sure package versions with known security issues are not installed",
"setasign/fpdf": "Enables you to use the PDF writer"
},
"type": "library",
"extra": {
"branch-alias": {
"dev-main": "5.x-dev"
}
},
"autoload": {
"psr-4": {
"Endroid\QrCode\": "src/"
}
},
"notification-url": "https://packagist.org/downloads/",
"license": [
"MIT"
],
"authors": [
{
"name": "Jeroen van den Enden",
"email"<EMAIL_ADDRESS>}
],
"description": "Endroid QR Code",
"homepage": "https://github.com/endroid/qr-code",
"keywords": [
"code",
"endroid",
"php",
"qr",
"qrcode"
],
"support": {
"issues": "https://github.com/endroid/qr-code/issues",
"source": "https://github.com/endroid/qr-code/tree/5.0.7"
},
"funding": [
{
"url": "https://github.com/endroid",
"type": "github"
}
],
"time": "2024-03-08T11:24:40+00:00"
},"

Your lock looks fine, that version should have the create method, see https://github.com/endroid/qr-code/blob/0cc00f0626b73bc71a1ea17af01387d0ac75e046/src/Builder/Builder.php#L72. In other words, the remote server seems to have a different version than the one according to your lock file. Please check what methods your live server's Builder.php.
Make sure to check on the live server, not locally of course ;)

For version: 6.0.0, what method is used in place of create() to generate a new QR code?

In 6.x you can simply use the constructor, see https://github.com/endroid/qr-code/blob/master/README.md

to update to latest version :"endroid/qr-code": "^6.0.1",
code: " composer update endroid/qr-code --with-all-dependencies "
Please mention it in README.md file.

I updated to the latest version, then revised my method, and now it's working perfectly.

constructor

Can you please add this to your release notes?? https://github.com/endroid/qr-code/releases
I mean I need to debug on my local server. Trying to find the issue in github issues. In order to know how to upgrade from v5 to v6


[PT Run] Updated themes (dark/light/highcontrast)

Summary of the Pull Request

Added a brush for 'secondary text' (instead of primary text font with a lower opacity)
Minor updates to the Light and Dark themes to reflect the updated secondary text brush
Added a BorderThickness of 1 to visualize the border when a high contrast theme is selected. Transparant for Light and Dark
Updated the high contrast themes, to reflect the default high contrast settings.

PR Checklist

[X] Applies to #4007
[X] CLA signed. If not, go over here and sign the CLA

Hey @niels9001 Can you please separate the border brushes of searchbox and resultlist box, so as to not impact #2988😊

@AnuthaDev That might be an issue, yes. Not sure where we are with that PR, so let's follow up there - I'm happy to make any changes if needed to solve a potential conflict
I saw your comment about breaking ClearType due to (semi)-transparency. What specific brushes are impacted at the moment that you'd recommend to update? And is there any documentation I can check out, wasn't aware of this.

@niels9001  Read the remarks section here:
https://docs.microsoft.com/en-us/dotnet/api/system.windows.media.renderoptions.cleartypehint

If you want to dive really deep:
https://github.com/microsoft/terminal/pull/5277

The PR #2988 is a good idea but it was work done without collaboration.  Niels helped properly get our HC settings correct in this PR.

@AnuthaDev The latest commit changed the transparency to fully opaque colors - let me know if this solves your concern wrt ClearType. Only the 'focus' color (AccentBrush) is still using opacity, we might want to update that as well but maybe in a new PR.

@niels9001 Unfortunately its a bit trickier than just fixing colors, because we are using a dropshadoweffect, We need to apply this fix to get clear text. I think a new issue should be opened for cleartype after this PR is merged.

Opened #4172 for cleartype


Syntax highlighting broke in<EMAIL_ADDRESS>

Prerequisites

[x] Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
[x] Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
[x] Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
[x] Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
[x] Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
All syntax highlighting seems to be broken in Atom 1.32.0.

The syntax seems to have a totally different color or no color at all (white) after the last Atom update.
Steps to Reproduce

Create a file (in my case a JS file)
Write some code
Notice syntax is not working in all cases

Expected behavior: Correct syntax highlighting
Actual behavior: Some peaces of code get the wrong color or no color at al
Reproduces how often: All the time
Versions
atom --version:
Atom    : 1.32.0
Electron: 2.0.9
Chrome  : 61.0.3163.100
Node    : 8.9.3

apm --version:
apm  2.1.2
npm  6.2.0
node 8.9.3 x64
atom 1.32.0
python 2.7.10
git 2.19.1

Additional Information
I'm not sure if this is just related to JS syntax highlighting or that it is way bigger but i have found numerous issues about highlighting issues within your issue tracker. They're mostly related to the latest Atom update.

I second that — same version, same problems.

Syntax highlighting is not broken, but we have changed the default parser implementation, which is why some parts of your code will be highlighted differently. See https://github.com/atom/atom/issues/18307 for more details.
You can switch back to the old parser by unticking Use Tree Sitter Parsers in your core settings, or by running atom.config.set('core.useTreeSitterParsers', false) from the developer console.
If you are seeing specific keywords or syntax in specific languages not highlighted correctly, please open an issue on the relevant language repository (e.g. atom/language-javascript).

Another way to apply @thomasjo  suggestion is by editing the config.cson file.
Atom > config... and then under core add useTreeSitterParsers: false and save.
Then you  get your lovely syntax highlighting back.
Note: this works for me using a mac I have not tried on a Windows or Linux.

It would have been amazing to get some form of notification regarding the "not broken" breaking change to all of our stylesheets and grammar packages. For me it happened in the middle of the work day, and after a fresh re-install, rewriting my stylesheet and a few issue reports, I finally found this thread 🤦♂ .  FYI, It completely breaks the core Go language package. I have made an git issue there but anyone using go should revert the new default setting and turn off auto updates to avoid similar issues in the future

@thomasjo I'm having entire chunks of my code highlight in the wrong color for the past few weeks. If I go and edit each individual line that's incorrectly highlighted, it will fix the highlighting for that specific line. I think this is a legitimate issue.
Highlighting for this chunk of code broke when I added a comment line to the top of the block:

Highlighting corrected for a single line when I added and deleted a space on that line:

I've been experiencing this with JS and Ruby code highlighting. It seems like the parser is not working correctly at random.

@allisonphillips In the next hotfix release (v1.32.2) there have been several further improvements made to those languages in particular. If you are still seeing issues after updating to that version when it comes out please file an issue in the respective repository for the language you are seeing a problem with, if there isn't an issue already filed of course 😉.

@Arcanemagus It seemed like a parser issue, given the timeline, that it's affecting multiple languages, it's resolved by setting useTreeSitterParsers to false, and that other people (including my coworkers) are also having this problem within the same timeline. I'm glad there are improvements being made since it looked like this was dismissed as a non-issue above, and I will wait for the next release and file tickets if it's still broken at that time. Thank you for the update! :)

@allisonphillips We noticed exactly this issue for ERB on master a few days ago and the fix for it will ship in Atom 1.32.2. It's likely that this fix will apply to javascript as well. If you still encounter the issue then please open a new issue.

this was dismissed as a non-issue above,

This issue and what you report are entirely different issues. This issue is about colors being different permanently, See https://github.com/atom/atom/issues/18352#issuecomment-436829384 for a more in depth explanation of this. Your report is about syntax highlighting breaking.


Link for sent message in API

The Rust project currently automatically opens Zulip topics when certain issues are filed/labeled on GitHub -- we would like to post a comment on github linking to the Zulip topic.
However, I've not found a nice way to go from the API for sending messages (which in our case automatically opens a topic for discussion) to actually getting a link to the topic in question.
Right now I'm thinking about trying to manually come up with the URL syntax by escaping and mangling the names of topics and streams (e.g., https://rust-lang.zulipchat.com/#narrow/stream/243200-t-lang.2Fmajor-changes/topic/A.20.60FunctionPointer.60.20trait.20to.20represent.20all.20.60fn.60.20lang-team.2323 could in theory be created with the information the bot has). But that's rather unfortunate and we'd need to precisely replicate the undocumented(?) strategy for mangling Zulip uses.
Opening this from here: https://github.com/zulip/zulip/issues/15290#issuecomment-642939455

Cc @timabbott
I'd be happy to help out with a PR or so if you can point me to the relevant piece of code.
FWIW, I will note that for our purposes we don't need it to be a permalink - we're fine needing to edit it if the stream or topic is renamed, and regardless neither should really happen.

Cool.  There's an ongoing discussion about how to do this; if we did it in the form described here, it'd be a quick change, but there's also discussion of just making a different URL structure that just requires the message ID.

Mind pointing me at that discussion?
I personally think that both would be reasonable and having the message ID based URL is nice for short permalinks but since it's harder can probably wait?

https://chat.zulip.org/#narrow/stream/3-backend/topic/send_message.20API/near/902103
The main concern is that if we're going to provide a much nicer interface where all you need is the message ID, it seems a bit silly to extend the API to serve you the URL for that message.

There's too much going on in the 3.0 release prep process to think through this properly, so I'm demoting it to something we'll look at after 3.0.


what is the correct way of changing depth_scale?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 16

Kernel Version (Linux Only)
4.15.0-34

Platform
PC

SDK Version
2.16.0

Language
C++

Segment
VR}

Issue Description
What is the correct way of setting depth_scale? I have the following code running, but the output puzzles me. See below. I would expect depth_scale_2 be the updated value 0.0001, not the original one (0.001).
int main()
{
rs2::context ctx;
auto list = ctx.query_devices();
if (list.size() == 0) throw std::runtime_error("No device detected. Is it plugged in?");
rs2::pipeline pipe;
rs2::config cfg;
cfg.enable_stream(RS2_STREAM_COLOR, 1920, 1080, RS2_FORMAT_BGR8, 15);
cfg.enable_stream(RS2_STREAM_DEPTH, 1280, 720, RS2_FORMAT_Z16, 15);
rs2::pipeline_profile profile = pipe.start(cfg);
rs2::depth_sensor depth_sensor = profile.get_device().firstrs2::depth_sensor();
auto original_scale = depth_sensor.get_option(RS2_OPTION_DEPTH_UNITS);
std::cout << "original scale: " << original_scale << std::endl;
float depth_scale_1 = get_depth_scale(profile.get_device());
std::cout << "depth_scale_1 = " << depth_scale_1 << std::endl;
depth_sensor.set_option(RS2_OPTION_DEPTH_UNITS, 0.0001);
auto updated_scale = depth_sensor.get_option(RS2_OPTION_DEPTH_UNITS);
std::cout << "updated scale: " << updated_scale << std::endl;
float depth_scale_2 = get_depth_scale(profile.get_device());
std::cout << "depth_scale_2 = " << depth_scale << std::endl;
return EXIT_SUCCESS;
}
output:
original scale: 0.001
depth_scale_1 = 0.001
updated scale: 0.0001
depth_scale_2 = 0.001

[Realsense Customer Engineering Team Comment]
Hi @ywjia,
Check below code. It should be no problem to modify the depth scale.
auto sensor = profile.get_device().firstrs2::depth_sensor();
auto depth_unit = sensor.get_option(RS2_OPTION_DEPTH_UNITS);
auto scale = sensor.get_depth_scale();
printf("[Before]Depth Scale: %f\r\n", scale);
sensor.set_option(RS2_OPTION_DEPTH_UNITS, 0.0001);
depth_unit = sensor.get_option(RS2_OPTION_DEPTH_UNITS);
scale = sensor.get_depth_scale();
printf("[After]Depth Scale: %f\r\n", scale);

Hi @RealSense-Customer-Engineering,
I copy&paste and run your code above,  but depth_scale was not changed, as output below:
[Before]Depth Scale: 0.001000
[After]Depth Scale: 0.001000
Of course, the first line was changed to
auto sensor = profile.get_device().first < rs2::depth_sensor >();
Also, what is the difference between "sensor.get_option(RS2_OPTION_DEPTH_UNITS)"  and " get_depth_scale(profile.get_device())"?
Thanks.

[Realsense Customer Engineering Team Comment]
Hi @ywjia,
"RS2_OPTION_DEPTH_UNITS" and "get_depth_scale" can be used to get the value. But, you can use "RS2_OPTION_DEPTH_UNITS" before stream starts.
So, suggest to set the depth unit first, and then you should be able to get the correct scale value after pipe start. The example code like below:
rs2::context ctx;    // Create librealsense context for managing devices
auto list = ctx.query_devices(); // Get a snapshot of currently connected devices
rs2::device dev = list.front();
auto depthSensor = dev.firstrs2::depth_sensor();
// set depth unit
depthSensor.set_option(RS2_OPTION_DEPTH_UNITS, 0.0001); // 0.0001

thanks. I understood it now. the key is to apply set_option() before starting the pipeline.
issue closed.


Error in building openvdb for blender

good to all, I'm trying to compile openvdb for blender and give me error. Could you help me with that error? Attached the output of the console.
Thank you

Where did you get your libblosc?

Which version of the g++ compiler are you using? Just passing through, but this first error you hit:
_/opt/lib/blosc/lib/libblosc.so: undefined reference to `_cpu_model'
sounds (from a quick google search) like a bug that a few others hit on Ubuntu in similar projects.
https://github.com/Intel-Media-SDK/MediaSDK/pull/21/commits/15db8effca57e8c686e17a25f4223b07a1926721
https://github.com/baldurk/renderdoc/issues/365
https://github.com/cilkplus/clang/issues/19

Closing due to inactivity - @argotan please do re-open if this is still an issue


Merge rewrite branch, splitting into two books

This PR merges the rewrite branch, but instead of replacing the existing contents, it puts both into subdirectories as individual complete mdbooks. A new top-level page contains links to both.
This is still a WIP and it was a bit of a tricky merge so I expect a few things will need figuring out, plus I'd welcome feedback on the presentation.
We should also at least think about URLs and how we'll be breaking any old deep links (though I think mostly links go to the top-level). This also means all currently open PRs will need a bit of rebasing.
Anyway, I mostly wanted to see what the CI makes of it... feedback welcome!
View build output online (until it's merged) here.

r? @therealprof
(rust-highfive has picked a reviewer for you, use r? to override)

As far as I know everything should be fine. I guess it would be good if someone who wasn't directly involved with the rewrite takes a look at it to see if everything is alright though.

I've hosted a copy of the build output here to make it easier to read through.

Maybe we can do a little announcement that we're looking for proof readers on tuesday's meeting?

On discovery/index.html
would it be possible to link to both books in the menu?

@hargoniX, how do you want to handle @Emilgardis's suggestions in this PR? If you're happy with them shall we just merge them here? Are you able to or do I need to?

would it be possible to link to both books in the menu?

I've tried to do this but can't figure out how to make mdbook put an external link in the ToC :/

I think I got all (?), except the section with the stuff you cannot do, since as Emil said its more of a humorous in the future thing.

Shall we get this merged and then tidy up the front page afterwards, if no one has any other suggestions right now?

Let's merge this then! :tada:
bors r+

CI failed :(
@hargoniX ?

cargo doc --features v1 (the command that's failing in CI) works for me locally :(

I changed chapter 5 so it explicitly tells the user to comment out the panic halt and comment in the panic RTT once we need it. This is because before we are doing some debugging etc. so the rtt setup and stuff would just be a little distracting at that point.

bors cancel

Oops, I had a mismatch between the CI names and the bors required checks. Hopefully fixed now.


791 chore(ui): typescript cleanup

Updates many packages in package.json.
This required changing our Supabase server side auth. The pattern follows the latest example here
Also includes lots of typescript fixes (many of these are repetitive changes throughout our test files)

Checking to see if pipeline passes


[PE-D][Tester C] Occasional missing success message when using the edit command

Your browser does not support the video tag.video:https://raw.githubusercontent.com/limyixiang/ped/main/files/dce9b961-2821-4f15-aff9-cd2c8124bcb6.mp4
As shown in the above image, the contact number was successfully updated for 8. alex to be 11111111 from 33333333, without displaying the corresponding success message. In addition, the command input box does not automatically clear itself.
After trying quite a number of times, I managed to replicate the bug, by editing a person's contact to be an existing duplicate contact number in the address book. This causes the success message to not be displayed.

Labels: type.FunctionalityBug severity.High
original: limyixiang/ped#9

Duplicate issue of #259


[flink] support system function truncate for computed column in CDC

Purpose
this close https://github.com/apache/incubator-paimon/issues/1136
Tests
UT & IT
API and Format

Documentation

Hi, @yuzelin , would you help review this PR at your convenience.
Thanks.


Filter and pagination ordering

Description
We should apply the filter before the pagination.

This bug was introduced in v2.1.1 and its fix should be included in the v2.1.2 and the v2.2.1!

Checklist

[ ] Fix.
[ ] Include this fix in the v2.1.2.
[ ] Include this fix in the v2.2.1.
[ ] Update Work in progress section in changelog.

Moved here.


sendResetPassword operation does not await notifier call

Steps to reproduce
Perform a sendResetPassword operation with a notifier sending an email.
Expected behavior
Notifier call should be awaited
Actual behavior
Notifier call is not awaited => see https://github.com/feathers-plus/feathers-authentication-management/blob/master/src/send-reset-pwd.js#L30
System configuration
NodeJS version: 12.16
Operating System: Windows
Module Loader: 3.0

Thanks!


Settings branch - Adding Moderators

I get a couple of errors:

If I select myself as a Moderator from the list: Non-hexadecimal digit found
If I select another node as a Moderator: Odd string length

This has been fixed.


Peppol compliance status and messaging extension

Hi there,
Great library!
Can you advise the current status of compliance with PEPPOL?
We are looking to extend the functionality here to include messaging using something like this implementation here:
https://github.com/everbinding/econnect-psb-php
Happy to collaborate, please let me know how we can get in touch.

Hello @turbo124,
I'm not familiar with this library. Could you explain what's "PSB" and how could it be integrated with einvoicing?

@josemmo
From my research, creating a PEPPOL invoice is only one step in the process. The einvoice then needs to be passed into the PEPPOL network to the recipient.
This is where a library such as the one above could assist in the transport of these invoices.

I see. Is this something exclusive to PEPPOL? The closest EU specification I could find related to the exchange of documents is eDelivery.
I still don't see how could this be integrated into this library, to me it looks like a good candidate for a separate project. Are you interested in any specific feature?


Added "Resolution" section

Added the "Resolution" section with the first troubleshooting steps.

Hi @LuigiBruno, I have recently moved the debugger commands into their own "project" - debuggercmds. And we use relative links to our topics. So the links to !analyze would look like this:
[!analyze](../debuggercmds/-analyze.md)
Let me know if you can update the PR, or I can fix the link after we check in this PR.

I will go ahead and merge this in and fix the link...

!analyze

Hi @DOMARS.
Noted. Thank you.


docs: add pgxpool example

Fixes #119.

Revised the example and fixed some headings (levels were wrong, inconsistent wording etc).


Future proof config

This one’s a doozy, but you can read the commit message for 1d4d0455674773e693f77142e11408adfc9c64ac to get up to speed.
Basically, the Vault config can change, and this cookbook’s vault_config is prescriptive about how that config is handled. Instead, we can just directly convert the node[vault][config] attribute into the necessary JSON.
We do lose some error handling & defaults, but Vault is a software that must be custom configured for one’s environment regardless, so I trust that anyone taking advantage of this cookbook will be able to handle that.

Rebased as per recent merges. Unfortunately, this depends at the moment on #13, though I might be able to pull that out of here if it turns out that PR is unwanted.

I am open to adding an options collector similar to what I just did with the Consul cookbook but I do not want to give up on having specific attributes validated.

@johnbellone a stance I can understand, but it seems as though you will forever be playing catch-up to Vault itself, esp. if the validity of certain values changes (rendering the validation in this cookbook incorrect).
I’m more of the school of thought that simpler is better, and that an operator will read the docs, and suss out any failures on the part of Vault while writing their wrapper cookbook.
Any chance I can convince you to let go of the validations in favor of letting Vault validate its own config at runtime? It has pretty good error reporting on that stuff (as I discovered while working on this).

@johnbellone actually, after using the current incarnation of the cookbook w/ the manage_certificate stuff merged in, I’m quite alright to refactor this PR to follow the model you’ve espoused. Works well, and it’s your cookbook, after all.
I’ll try to get this cleaned up, and add an options collector as the “future-proof” mechanism.

@jeffbyrnes did you get to it by any chance?

@scalp42 I haven’t, and probably won’t get a chance for at least another week. Looking to tackle yourself?


BACKLOG-15757 add release workflow and tweak config file for release

JIRA
https://jira.jahia.org/browse/BACKLOG-15757
Description
Added release workflow and tweaked config file for release

Out of curiosity, why removing auto ?

Out of curiosity, why removing auto ?


Hello! Prompt please: I am in my application that would create a chat room, i using wonderful framework "JSQMessagesViewController". But I have a problem

New issue checklist

[ ] I have read all of the README, documentation, and FAQ.
[ ] I have reviewed the contributing guidelines. Confirmation: ____
[ ] I have searched existing issues and this is not a duplicate.

General information

Library version(s):
iOS version(s):
Devices/Simulators affected:
Reproducible in the demo project? (Yes/No):
Related issues:

Bug report
Expected behavior
Actual behavior
Steps to reproduce
Crash log? Screenshots? Videos? Sample project?

Thanks! :smile:
Please provide all the information in the issue template so we can help!
And don't forget to read our contribution guidelines. :blush:

Hello! I use FireBase database and try to create an interface, as demonstrated in the test video. I have a list of contacts, I want to be able to when you click the "chat" button on the contact cell, move into messenger and obschatsya with this contact. If i click on the next button contact, it has already opened a messenger with the current contact. And I Now all contacts write messages to one messenger
(video below
triper test.zip
)

Hi Jesse!
Please tell me! I use to create chat a framework YSKMessazhesvievKontroller and Firevase.I need to build in such a way that the functionality would be when you click on another cell I moved the conversation with that friend. If I click on the cell next friend, respectively, I turn to the conversation with the second friend. Tell me how can I implement such functionality? Because I have a user and all of his friends wrote in a chat room and all can read and I want to do that all could obschatsya separately… As shown in the screen shots..

2 авг. 2016 г., в 18:00, Jesse Squires<EMAIL_ADDRESS>написал(а):
Closed #1782 https://github.com/jessesquires/JSQMessagesViewController/issues/1782.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jessesquires/JSQMessagesViewController/issues/1782#event-742523420, or mute the thread https://github.com/notifications/unsubscribe-auth/ALBkumwWzP7M-Imvykp6P2ty5D2ZVuXyks5qb1uPgaJpZM4JabID.

I'm sorry, I use frameworks : JSQMessageViewController and FireBase...

2 авг. 2016 г., в 18:00, Jesse Squires<EMAIL_ADDRESS>написал(а):
Closed #1782 https://github.com/jessesquires/JSQMessagesViewController/issues/1782.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jessesquires/JSQMessagesViewController/issues/1782#event-742523420, or mute the thread https://github.com/notifications/unsubscribe-auth/ALBkumwWzP7M-Imvykp6P2ty5D2ZVuXyks5qb1uPgaJpZM4JabID.


Minor optimization of inftrees.c based on profiling

Reorder an if chain to make the most likely branch come first, and provide a compiler hint that two branches are very unlikely.
Results in about 0.25% speedup in my tests.

Baseline e56238cbc4433b71b75f08deac35addb8426fef6 x86-64
   text    data     bss     dec     hex filename
 113433    1376      32  114841   1c099 libz-ng.so.2

 Tool: minigzip   Levels: 1-9
 Runs: 30         Trim worst: 15

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     54.165%      2.975/3.007/3.022/0.014        1.647/1.675/1.690/0.015      264,231,515
 2     43.875%      2.950/2.991/3.006/0.013        0.992/1.008/1.022/0.011      124,277,644
 3     42.382%      2.980/2.997/3.010/0.008        0.800/0.813/0.823/0.007      100,039,412
 4     41.669%      2.730/2.749/2.766/0.012        0.616/0.631/0.643/0.007       78,685,354
 5     41.215%      2.924/2.933/2.946/0.007        0.565/0.580/0.589/0.007       71,342,944
 6     41.036%      2.804/2.823/2.834/0.009        0.461/0.473/0.478/0.005       58,118,026
 7     40.781%      2.723/2.738/2.748/0.008        0.352/0.362/0.371/0.006       44,921,649
 8     40.702%      3.009/3.015/3.021/0.004        0.310/0.317/0.324/0.005       38,429,965
 9     40.698%      2.774/2.793/2.801/0.009        0.253/0.264/0.270/0.005       32,021,745

 avg1  42.947%                        2.894                          0.680
 tot                                390.693                         91.864      812,068,254

PR1006 841d7b918407a4496cbae59161628d3bfda572f4 x86-64
   text    data     bss     dec     hex filename
 113433    1376      32  114841   1c099 libz-ng.so.2

 Tool: minigzip   Levels: 1-9
 Runs: 30         Trim worst: 15

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     54.165%      2.969/3.000/3.016/0.014        1.644/1.668/1.689/0.017      264,231,515
 2     43.875%      2.964/2.980/2.992/0.008        0.996/1.016/1.026/0.009      124,277,644
 3     42.382%      2.972/2.992/3.004/0.011        0.788/0.810/0.818/0.008      100,039,412
 4     41.669%      2.717/2.739/2.751/0.010        0.621/0.632/0.645/0.009       78,685,354
 5     41.215%      2.908/2.925/2.936/0.008        0.547/0.576/0.587/0.010       71,342,944
 6     41.036%      2.816/2.827/2.837/0.007        0.463/0.472/0.480/0.004       58,118,026
 7     40.781%      2.703/2.726/2.737/0.009        0.347/0.363/0.370/0.007       44,921,649
 8     40.702%      2.992/3.010/3.020/0.009        0.296/0.311/0.319/0.006       38,429,965
 9     40.698%      2.782/2.791/2.796/0.004        0.249/0.261/0.266/0.004       32,021,745

 avg1  42.947%                        2.888                          0.679
 tot                                389.859                         91.644      812,068,254

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     54.165%      2.935/2.991/3.006/0.018        1.646/1.669/1.685/0.011      264,231,515
 2     43.875%      2.983/3.002/3.011/0.009        0.986/1.005/1.016/0.009      124,277,644
 3     42.382%      2.999/3.014/3.028/0.010        0.801/0.818/0.825/0.007      100,039,412
 4     41.669%      2.716/2.731/2.741/0.008        0.601/0.630/0.641/0.011       78,685,354
 5     41.215%      2.927/2.964/2.978/0.014        0.567/0.581/0.590/0.008       71,342,944
 6     41.036%      2.791/2.807/2.819/0.008        0.453/0.467/0.477/0.007       58,118,026
 7     40.781%      2.708/2.731/2.742/0.009        0.353/0.367/0.374/0.006       44,921,649
 8     40.702%      2.990/3.003/3.011/0.006        0.303/0.312/0.316/0.004       38,429,965
 9     40.698%      2.774/2.786/2.795/0.007        0.249/0.261/0.269/0.005       32,021,745

 avg1  42.947%                        2.892                          0.679
 tot                                390.427                         91.646      812,068,254

PR1006 9dd34ecb47ba860c6db6f9cd817270641f77e6ec x86-64
    text    data     bss     dec     hex filename
 113433    1376      32  114841   1c099 libz-ng.so.2

 Tool: minigzip   Levels: 1-9
 Runs: 30         Trim worst: 15

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     54.165%      2.975/3.011/3.028/0.015        1.641/1.673/1.688/0.012      264,231,515
 2     43.875%      2.963/2.994/3.008/0.014        0.993/1.005/1.016/0.008      124,277,644
 3     42.382%      2.986/3.014/3.024/0.010        0.781/0.812/0.825/0.011      100,039,412
 4     41.669%      2.717/2.730/2.740/0.008        0.608/0.635/0.648/0.013       78,685,354
 5     41.215%      2.939/2.967/2.977/0.010        0.557/0.580/0.590/0.009       71,342,944
 6     41.036%      2.790/2.812/2.824/0.010        0.460/0.472/0.480/0.006       58,118,026
 7     40.781%      2.711/2.729/2.741/0.011        0.340/0.362/0.370/0.008       44,921,649
 8     40.702%      2.992/3.006/3.012/0.006        0.299/0.312/0.320/0.006       38,429,965
 9     40.698%      2.772/2.789/2.797/0.007        0.249/0.258/0.262/0.003       32,021,745

 avg1  42.947%                        2.895                          0.679
 tot                                390.784                         91.633      812,068,254

0.25% is well within error margin.

@mtl1979 Are you saying you don't think there is an actual performance improvement with this PR?
It seems very consistent here, both in benchmarking and profiling, feel free to perform your own profiling to see how often these are used in workloads you care about.
If you think the improvement is just small, well there is not much to be done about that, this is a simple and minor optimization after all.

@Dead2 I'm not saying that there can't be improvement, I'm just not trusting the number... Most results from benchmarks I have seen are pretty inconsistent as they use only one type of file with predictable contents. Even if you run 70 iterations of test with one file, the error margin is more than if you run 10 or 20 iterations of 7 files. Concatenating multiple files to a larger file is not same as running test with same files non-concatenated.
Basically doing same operation over and over will just let the operating system and processor learn from the data and cache enough data that skew the results.

@mtl1979 Feel free to contribute improvements to benchmarking tools

Baseline 834e7d83eae9ff7bfb1b9769d9f311ea7ca8e746 aarch64
   text    data     bss     dec     hex filename
 105194    1576      16  106786   1a122 libz-ng.so.2

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     54.166%      2.746/2.764/2.780/0.011        0.965/0.993/1.015/0.013       42,618,858
 2     43.879%      2.819/2.851/2.870/0.017        0.563/0.601/0.631/0.022       20,714,938
 3     42.383%      3.713/3.758/3.780/0.020        0.565/0.580/0.592/0.008       20,008,755
 4     41.660%      2.768/2.802/2.818/0.015        0.353/0.372/0.381/0.008       13,111,391
 5     41.216%      3.014/3.031/3.051/0.012        0.365/0.375/0.383/0.005       12,971,746
 6     41.039%      3.505/3.539/3.555/0.014        0.358/0.372/0.384/0.009       12,916,025
 7     40.778%      2.239/2.270/2.284/0.011        0.168/0.175/0.181/0.004        6,416,924
 8     40.704%      2.769/2.800/2.809/0.010        0.168/0.179/0.188/0.007        6,405,244
 9     40.696%      2.983/3.016/3.029/0.013        0.160/0.172/0.180/0.007        6,404,084

 avg1  42.947%                        2.981                          0.424
 tot                                402.450                         57.285      141,567,965

PR #1006 9dd34ecb47ba860c6db6f9cd817270641f77e6ec aarch64
   text    data     bss     dec     hex filename
 105234    1576      16  106826   1a14a libz-ng.so.2

 Level   Comp   Comptime min/avg/max/stddev  Decomptime min/avg/max/stddev  Compressed size
 1     54.166%      2.741/2.775/2.802/0.018        0.947/0.989/1.000/0.015       42,618,858
 2     43.879%      2.785/2.838/2.865/0.024        0.561/0.595/0.611/0.014       20,714,938
 3     42.383%      3.698/3.746/3.779/0.022        0.550/0.577/0.595/0.014       20,008,755
 4     41.660%      2.765/2.796/2.815/0.016        0.365/0.377/0.384/0.006       13,111,391
 5     41.216%      3.006/3.029/3.046/0.014        0.357/0.370/0.382/0.008       12,971,746
 6     41.039%      3.507/3.531/3.540/0.009        0.340/0.363/0.376/0.011       12,916,025
 7     40.778%      2.239/2.268/2.281/0.013        0.164/0.177/0.184/0.007        6,416,924
 8     40.704%      2.789/2.803/2.808/0.005        0.160/0.176/0.184/0.008        6,405,244
 9     40.696%      2.994/3.017/3.029/0.010        0.156/0.172/0.180/0.007        6,404,084

 avg1  42.947%                        2.978                          0.422
 tot                                402.027                         56.930      141,567,965

Compiled code size increased by 40 bytes, likely due to moving the two unlikely blocks of code out of the hot-path of the function.
0.62% faster inflate.


Code refactor & cleanup and possible bug fixes

This will be started < 30.5 > after the rest of the team Finnishes their coding process on 29.5


Remove component-specific models

@hammar - the team at Optio3 noticed that we still had some remnant component-specific models that should be removed. Since we made the decision that we stop at "OpenSensor" and "OpenSetpoint" any further sub-classing gets defined by setting the assetComponent tag = Fan, Damper, Valve, etc.

LGTM.


Update README.md

In the Installing via Composer section:
$ curl -sS https://getcomposer.org/installer | php
results in the following composer file location on OSX: /usr/local/bin/composer.phar
This means the following next step will result in a "command not found" error
$ composer require guzzlehttp/guzzle
Some solutions include:
(1) changing command to composer.phar
(2) renaming / moving composer.phar to composer
(3) creating a link from composer.phar to composer
This request uses the composer.phar name but if this is only for select environments, either of the above or simply a note would be useful.

:-1:


Replaced titangraph dependencies with janusgraph equivalents.

Long live JanusGraph.

I added a commit that introduces more extensive removal of references to titan. Let me know if you approve. I can also squash the commits if necessary.

@DarioAmiri the changes look good to me. Yes please squash commits and I'll merge it in.

@anubhavi25 I squashed the PR commits.


Add first unit test

Current coverage is 0.64% (diff: 100%)

No coverage report found for dev at 1992a91.

Powered by Codecov. Last update 1992a91...83262d5