Cannot build build steven_resources crate from clean build

I tried building steven with the current Rust nightly toolchain (2016-06-23) but it keeps failing because the assets aren't copied to the out directory of the crate.
I tried inspecting resources/build.rs and so far I know build_map creates the out array correctly, ie. this resource.rs file is generated as intended:
pub fn get_file(name: &str) -> Option<&'static [u8]> {
    match name {
        "assets/steven/logo/textures.txt" => Some(include_bytes!("../assets/steven/logo/textures.txt")),
        "assets/steven/logo/logo.txt" => Some(include_bytes!("../assets/steven/logo/logo.txt")),
        "assets/steven/textures/environment/clouds.png" => Some(include_bytes!("../assets/steven/textures/environment/clouds.png")),
        "assets/steven/textures/gui/cog.png" => Some(include_bytes!("../assets/steven/textures/gui/cog.png")),
        "assets/steven/models/block/missing_block.json" => Some(include_bytes!("../assets/steven/models/block/missing_block.json")),
        "assets/steven/blockstates/missing_block.json" => Some(include_bytes!("../assets/steven/blockstates/missing_block.json")),
        "assets/minecraft/textures/font/ascii.png" => Some(include_bytes!("../assets/minecraft/textures/font/ascii.png")),
        "assets/minecraft/texts/splashes.txt" => Some(include_bytes!("../assets/minecraft/texts/splashes.txt")),
        _ => None
    }
}

But the files aren't being copied to the build directory in my case. I'm not sure if the resources/build.rs script should be modified to copy the files to the appropriate location, use the correct path (maybe building with absolute paths is an option?) or if something in my environment is not correct.

Apparently a change in rust/cargo broke this, shame. Haven't had the time to work on steven for a while and i'm not sure when I will so this might not get fixed.

It would be so much to me if you could fix it. Whenever you have the time,
I need to get it working as soon as possible. I've kinda promised a friend.
Thanks for replying!
On 2 July 2016 at 01:39, Matthew Collins<EMAIL_ADDRESS>wrote:

Apparently a change in rust/cargo broke this, shame. Haven't had the time
to work on steven for a while and i'm not sure when I will so this might
not get fixed.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Thinkofname/steven/issues/53#issuecomment-229948973,
or mute the thread
https://github.com/notifications/unsubscribe/AQqLIxNrjHawsspIENyV27D480mgpXJkks5qRRibgaJpZM4JDJkp
.


MS/MS display

Basic information

My operating system: Windows 10.0.18363 Build 18363, 64 bit
My MZmine version: MZMine 3.0.0beta

What happened
Excellent new version! Unfortunately I can't get the MS/MS plot display to work (formerly called MS/MS visualizer). I've tried fiddling with all of the settings (axes, options, etc.), file selection, updating Java, etc.
No error appears, it just doesn't fire up any visualizer once I click OK.

Hey dwheeler,
thank you for the bug reprot.
Can you please post the log output? You can find it via the "Show log file" button in the help menu.
Also: What kind of file raw data file are you using? Perhaps it might be good to upload the file somewhere so we can reproduce the error.

Hey, thanks for the report. That's one of the modules I have never used or touched so far.
There are plenty of options for MS/MS visualization but this one seems very different from the rest.
During the change of the interfaces, we created the new visualizer but forgot to add it to the main window.  That's an easy fix.
I suspect this will be in the next patch release 3.0.1-beta.
Keep up the good work and report what you experience.
Best Robin

Oh and updating java does not work because MZmine ships with its own java version. That's what modern java does.

Thank you so much! This was using Thermo RAW files. Again, excellent new version, I really appreciate the wonderful tool you have all made.
Best,
Doug


fix(): infinite scroll & refresher to use Sass variables for icon color

Do not squash
Prefer this to not be squashed since it's technically for 2 different components. :)
Testing is pretty simple: change the value of the variable and see the icon change color. I tested this with different values for the spinners and no value passed.
Fixes #10479 and #10574

Cherry picked into mastah 👍


Mixing shorthand with longform

When you do something like this, Baz displays with 2rem in padding due to the shorthand property in Bar even though it's changed with the longform property.
const Foo = styled('div', {
  paddingLeft: '1rem',
});

//

const Bar = styled('div', {
  padding: '2rem',
});

//

const Baz = styled(Bar, {
  paddingLeft: '1rem',
});

//

I noticed through #5 that the function warnMixedProperties exists, but I can't find that it's used anywhere.
What is best practice to handle this?

Yeah, this is also a problem even with plain inline styles. To be un-opinionated and lightweight as possible I don't really do anything about this in the core.
I think a pretty good pattern is using helper functions that generate longform properties. I've built a collection of these for this purpose: https://lostyle.js.org/
This works nicely if you're using object spread syntax. For example if you're using the padding helper:
const Bar = styled('div', {
  ...padding('2rem'),
  ...padding({left: 1rem'})
});

I think ideally warnMixedProperties is added in NODE_ENV=development but I hadn't gotten around to integrating it yet. But it's there just in case other framework authors who may want to use it. Actually now I think about it, maybe it's a good idea to move it into a separate repo/package since it doesn't really have anything to do with Styletron specifically (just style object literals).
I have some ideas for solving this problem in a more robust, general way (somewhat unrelated to Styletron) but nothing concrete yet. Another problem I'm working on solving is robust Right-To-Left transformation at runtime.

I really like the idea of Strict Style Object Notation. However, it would be a small effort to wrap warnMixedProperties in NODE_ENV=development and something that would ease your workflow as developer, especially when debugging is hard.
That actually reminds me of asking: What is the best way of debugging? Sometime when you type a wrong value to a style property, it just renders as an empty CSS class. At least as shown in Chrome developer tools. And it's not that clear where CSS classes comes from when there's no source maps.

An idea, could this be solved temporary by adding different specificity to rules.
For example
.a { border: 1px solid #fff; }
.b.b { border-bottom: none; }
Specificity is decided by number of dashes.

What is the status of this?
This has been a real problem for us in our project, we use "inline styles" see #72 which makes it more common for a developer to mix short and longhand properties.
It seems that when using Iostyle it will introduce a lot of additional styles, especially for properties like border.
If the goal of this project is to reduce the bytes rendered, do we really accomplish that?
Additionally Styletron will create a class name for each of these properties which will in many cases increase the bytes compared to using a single class name like Aphrodite.
I'm not trying to bash on Styletron, I just want to provoke some thoughts.

With #91 you could add a plugin that parses all shorthand to longform before they injects.

@faddee that will still introduce a lot of additional styles, just like Iostyle.

The same issue crops up for @media conditionals:

const Foo = styled('div', {
  color: 'red',
  '@media (min-width: 600px)' : {
    color: 'blue'
  }
});

const Bar = styled('div', {
  color: 'green',
  '@media (min-width: 600px)' : {
    color: 'blue' // fetched from cache, `color: green` prevails since it is the latest definition.
  }
});

styletron-engine-atomic provides development warnings for mixing shorthand/longform properties. Feel free to re-open if there's more issues. Thanks!


Can't copy file to Azure Blob Storage

What are the other pre-requisites for private agent such that Azure tasks are run properly? I've installed Azure PS (v6.2.1) and AzureRM modules however still can't copy a file to Azure Blob Storage.
This works fine if I use the Hosted VS2017 agent.
Log from the task:
2018-06-11T19:28:23.4792254Z ##[section]Starting: AzureBlob File Copy - README.md
2018-06-11T19:28:23.5140067Z ==============================================================================
2018-06-11T19:28:23.5140740Z Task         : Azure File Copy
2018-06-11T19:28:23.5141068Z Description  : Copy files to Azure blob or VM(s)
2018-06-11T19:28:23.5141434Z Version      : 1.0.115
2018-06-11T19:28:23.5141759Z Author       : Microsoft Corporation
2018-06-11T19:28:23.5142203Z Help         : More Information
2018-06-11T19:28:23.5142640Z ==============================================================================
2018-06-11T19:28:27.1311142Z Added TLS 1.2 in session.
2018-06-11T19:28:27.8562122Z ##[error]Requested registry access is not allowed.
2018-06-11T19:28:27.9075691Z ##[section]Finishing: AzureBlob File Copy - README.md

@kmkumaran
Note that there isn't a requirement to run private agents as admin.

@VinKamat , Can you please share the debug log with us?
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
You can email us the logs to<EMAIL_ADDRESS>

@niadak  Here it is:
Loading module from path 'C:\Program Files\WindowsPowerShell\Modules\AzureRM\6.2.1\AzureRM.psm1'.
Entering Get-SdkVersion.
Leaving Get-SdkVersion.
Leaving Import-FromModulePath.
Leaving Import-AzureModule.
Leaving Initialize-Azure.
Caught exception from task script.
Error record:
Get-ItemProperty : Requested registry access is not allowed.
At C:\agent_work_tasks\AzureFileCopy_eb72cb01-a7e5-427b-a8a1-1b31ccac8a43\1.0.115\ps_modules\VstsAzureHelpers_\ImportFunctions.ps1:188 char:34

$installedApplications = Get-ItemProperty -Path $regKey

CategoryInfo : PermissionDenied: (HKEY_LOCAL_MACH...all\QualysAgent:String) [Get-ItemProperty], Securit yException
FullyQualifiedErrorId : System.Security.SecurityException,Microsoft.PowerShell.Commands.GetItemPropertyCommand
Script stack trace:
at Get-SdkVersion, C:\agent_work_tasks\AzureFileCopy_eb72cb01-a7e5-427b-a8a1-1b31ccac8a43\1.0.115\ps_modules\VstsAzureHelpers_\ImportFunctions.ps1: line 188
at Import-FromModulePath, C:\agent_work_tasks\AzureFileCopy_eb72cb01-a7e5-427b-a8a1-1b31ccac8a43\1.0.115\ps_modules\VstsAzureHelpers_\ImportFunctions.ps1: line 94
at Import-AzureModule, C:\agent_work_tasks\AzureFileCopy_eb72cb01-a7e5-427b-a8a1-1b31ccac8a43\1.0.115\ps_modules\VstsAzureHelpers_\ImportFunctions.ps1: line 40
at Initialize-Azure, C:\agent_work_tasks\AzureFileCopy_eb72cb01-a7e5-427b-a8a1-1b31ccac8a43\1.0.115\ps_modules\VstsAzureHelpers_\VstsAzureHelpers_.psm1: line 51
at , C:\agent_work_tasks\AzureFileCopy_eb72cb01-a7e5-427b-a8a1-1b31ccac8a43\1.0.115\AzureFileCopy.ps1: line 55
at , : line 1
at , : line 22
at , : line 18
at , : line 1
Exception:
System.Security.SecurityException: Requested registry access is not allowed.
at System.ThrowHelper.ThrowSecurityException(ExceptionResource resource)
at Microsoft.Win32.RegistryKey.OpenSubKey(String name, Boolean writable)
at Microsoft.PowerShell.Commands.RegistryWrapper.OpenSubKey(String name, Boolean writable)
at Microsoft.PowerShell.Commands.RegistryProvider.GetRegkeyForPath(String path, Boolean writeAccess)
at Microsoft.PowerShell.Commands.RegistryProvider.GetRegkeyForPathWriteIfError(String path, Boolean writeAccess)
The Zone of the assembly that failed was:
MyComputer
Requested registry access is not allowed.
Processed: ##vso[task.logissue type=error]Requested registry access is not allowed.
Processed: ##vso[task.complete result=Failed]

@VinKamat , It will be great if you can share the complete debug logs with us.
I have tried the scenario with AzurePS V6.2.1 and it is working for me.
You can email us the logs to<EMAIL_ADDRESS>

@niadak sent the logs to the email address, let me know what you find. Thanks.

@VinKamat Thanks for sharing the logs with us. Can you check if the agent is running with non-admin account ? If so, can you configure the agent with admin account and try?

@VinKamat As discussed, the task requires that the agent be configured with credentials that has read access to registry. By default, NT Authority/Network Service account has these permissions. For some reason, NT Authority/Network Service on your VM is not able to that, but you were able to run the task using admin credentials.
You can try and create a new user on your VM and try to configure it to have read access on registry. You can also try to create a new VM to see if you still see the same behaviour, because I tried on Windows server 2016 datacenter VM and it worked for me with default settings.

Closing this issue as this is not a task issue but rather a infra/permissions issue.
@VinKamat Please use our RM_Customer_Queries mail thread if you need further assistance regarding this.


change om-cli to tool-om in pcf-install

change om-cli in install-pcf  *.yml and *.sh files to tool-om to match rest of repository.

@bhudlemeyer Thanks, but could you rebase it to remove your commit of "param ops man install vcenter folder selection" and the revert of that commit?

I will try yes. Sorry accidentally pushed to the same branch when working
on that stuff.
On Mon, Apr 24, 2017 at 2:59 PM krishicks<EMAIL_ADDRESS>wrote:

@bhudlemeyer https://github.com/bhudlemeyer Thanks, but could you
rebase it to remove your commit of "param ops man install vcenter folder
selection" and the revert of that commit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pivotal-cf/pcf-pipelines/pull/22#issuecomment-296834275,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACK4opzo5k0gCw2ge0TPXv19j73yUGfdks5rzRtZgaJpZM4NE5Vn
.

rebase complete.
On Mon, Apr 24, 2017 at 5:20 PM Brandon Hudlemeyer<EMAIL_ADDRESS>wrote:

I will try yes. Sorry accidentally pushed to the same branch when working
on that stuff.
On Mon, Apr 24, 2017 at 2:59 PM krishicks<EMAIL_ADDRESS>wrote:

@bhudlemeyer https://github.com/bhudlemeyer Thanks, but could you
rebase it to remove your commit of "param ops man install vcenter folder
selection" and the revert of that commit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pivotal-cf/pcf-pipelines/pull/22#issuecomment-296834275,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACK4opzo5k0gCw2ge0TPXv19j73yUGfdks5rzRtZgaJpZM4NE5Vn
.


output of theorems should be lazy, but is not, because of alignment

For larger values of maxSize, I have to wait a looong time to see anything, in the output of speculate.
I was hoping that theorems are printed as they are discovered. But I think they are all held until the end - for computing the alignment. (report calls prettyEquations calls table which wants to see all words)
A work-around (I guess) is to call finalEquations directly, but then I have to copy code out of report.
Assuming the above is true (the lazy list of theorems is available),
perhaps Args can get an attribute that says whether to print them at once (but un-aligned), or in the end (and aligned).

For larger values of maxSize, I have to wait a looong time to see anything, in the output of speculate.

The quick workaround I tended to use was to actually call Speculate multiple times, each time increasing the maxSize.  You could use:
./yourprogram-s4 && ./yourprogram -s5 && ./yourprogram -s6 && ./yourprogram -s7

In some cases the runtime magnitude increases by one with each two increments of size, so the earlier runs wouldn't matter to much in the overall runtime.  (This a similar process to how the seminal SmallCheck ends up retesting values of ealier series.)

I was hoping that theorems are printed as they are discovered. But I think they are all held until the end - for computing the alignment. (report calls prettyEquations calls table which wants to see all words)

If I recall correctly, the output is not lazy just because of equation alignment, but actually it is so because Speculate may use later equations to discard redundant earlier ones.  It's been a while, so I could be wrong (-:
Assuming I'm not, in theory, Speculate could have an option where we only discard later (bigger) equations based on earlier (smaller) ones, so output would appear while Speculate tests and thinks.  For that to happen, the reasoning engine has to become lazy.  This would be non-trivial to implement as my reasoning engine is a currently sort of a can of worms.
(some handwavy technical details below)
Now about "Speculate may use later equations to discard redundant earlier ones":
One of the reasons why earlier equations may be discarded is that by default, on the term-rewriting-based reasoning module, Speculate uses a variation of Dershowitz order which is more "lexicographicky" rather than using Knuth-Bendix order, which is more based on term size.
(A technical detail of a technical detail now.)  Though Speculate is not strictly lexicographic.  Leaf terms are actually compared quasi-lexicographically based on the position they appear in the constants list and their arity -- so the order in constants may influence which equations are discarded or how they are displayed.  This was chosen sort of in an ad-hoc manner, to replace weird equations in examples.
Warning: the two above paragraphs are handwavy - I'm not being strict on the terms I use.

Thanks for explanation. Well, then just keep this issue open, as documentation for current behaviour, and reminder for possible enhancement.


Investigate impact of long running DiagnosticSourceEventSource usage

Several of the ASP.NET triggers will like use the Microsoft-Diagnostics-DiagnosticSource event provider to collect ASP.NET events such as Microsoft.AspNetCore/Microsoft.AspNetCore.Hosting.HttpRequestIn.Start and Microsoft.AspNetCore/Microsoft.AspNetCore.Hosting.HttpRequestIn.Stop. Investigate the impact (memory, cpu) of using this provider for long spans of time (entire target process lifetime) for applications that are under high load (many requests).

I was not able to observe any significant impact from a asp.net trigger prototype. Methodology:

Bombardier was used to load test the website
A trigger prototype was written against asp.net provider. See https://github.com/wiktork/diagnostics/tree/dev/wiktork/triggerProto and https://github.com/wiktork/dotnet-monitor/tree/dev/wiktork%2FaspnetTriggerTesting
The trigger action was to simply log the event (using ILogger in the agent)

No significant RPS difference was observed when running with the trigger or without.
No significant impact on cpu/memory usage was observed.

Closing this out, so far the asp.net triggers are meeting perf goals under load.


Unstyled Components: MultiSelect Presets not Compatible

Describe the bug
I have latest PrimeVue 3.47.2 and there are presets for lara theme in the website: _https://tailwind.primevue.org/multiselect/_.
The preset si incompatible many ways, there are props that have wrong names "emptymessage" -> "emptyMessage" or props that are not supported at all eg. "headerCheckboxContainer".
Where I can get right presets for the latest version please?
Thank you and have a nice day
Reproducer
N/A
PrimeVue version
3.47.2
Vue version
3.x
Language
TypeScript
Build / Runtime
Vue CLI App
Browser(s)
N/A
Steps to reproduce the behavior

Install PreimeVue
Set unstyled
Download the presets

Expected behavior
The presets are part of the main source package

Hi, this issue has likely been resolved in the latest updates.
If you're still experiencing the problem, please feel free to reopen the issue or create a new one with more details.


Can't build samples

Today I cloned your repo and now I'm trying to run samples. I don't know why, but gradle can't build them. I'm receiving following errors:
e.g. sample project

/mosby/mvp/src/main/java/com/hannesdorfmann/mosby/mvp/MvpActivity.java
Error:(20, 32) error: cannot find symbol class MosbyActivity
Error:(29, 67) error: cannot find symbol class MosbyActivity
/mosby/mvp/src/main/java/com/hannesdorfmann/mosby/mvp/layout/MvpFrameLayout.java
Error:(7, 19) error: package butterknife does not exist
/mosby/mvp/src/main/java/com/hannesdorfmann/mosby/mvp/layout/MvpLinearLayout.java
Error:(7, 19) error: package butterknife does not exist
/mosby/mvp/src/main/java/com/hannesdorfmann/mosby/mvp/layout/MvpRelativeLayout.java
Error:(7, 19) error: package butterknife does not exist
/mosby/mvp/src/main/java/com/hannesdorfmann/mosby/mvp/lce/LceAnimator.java
...

I'm using Android Studio 1.2 Build 3. Why can't I build them? Can you help me?

Im facing the same problem with Android Studio 1.2
It seems that there are some internal issues with Annotation Processing.
However, this error only occurs on first app start or if you clean / rebuild your project. If you simply press the green Play button to compile and run the sample it should work without any compilation errors.
Let me know if that works on your machine as well.

Thanks. I tried cleaning, rebuilding, syncing gradle and running project in random order but unfortunately it's still not working.
Do you have any other ideas?

Yep, I'm having the same issue

but pressing the "Green Play Button" works, doesn't it?

I installed Android Studio 1.1.0, but still get the same error. I don't know exactly what the problem is (I guess something with annotation processing) but simply pressing the "Play button" to compile and run works fine for me.
Ensure that you have pulled the latest version:
Could you try to compile it from command line:
./gradlew assemble --stacktrace

If you just want to have a look at the sample apps live on your android phone you can download the apk sample files from here:
https://db.tt/ycrCwt1L
https://db.tt/3fVqVdAz
https://db.tt/z85y4fSY

here is a gist of the gradle stacktrace: https://gist.github.com/62ff4714235bdd7aaced.git
It seems to be complaining about the support library but I do have version 22.0.0 installed on my system. In fact I'm using the exact same version on other projects.
My suspicion is that the variables declared on the rootProject with the versions of each library are not being processed correctly by gradle. I'm gonna do some testing and report back my results. BTW I'm using windows @ziem are you also using windows?

Strange. maybe with an clean:
./gradlew clean assemble --stacktrace

Btw. I'm working on mac osx.

I'm using OS X (Mavericks). Pressing green play button doesn't work. I also tried both commands - the same errors occurred.
Thanks for apks ;).

I am getting similar issues.  Gradle build spits out the following errors:

Error:(19, 10) error: package rx does not exist
Error:(20, 29) error: package rx.android.schedulers does not exist
Error:(21, 21) error: package rx.schedulers does not exist
Error:(32, 39) error: cannot find symbol class Observable
where T is a type-variable:
T extends Object declared in class AndroidSchedulerTransformer
Error:(32, 20) error: cannot find symbol class Observable
where T is a type-variable:
T extends Object declared in class AndroidSchedulerTransformer

I am on Android Studio 1.2 Beta 3.  What is going on??

I will strip out the samples of the library project. There seems to be a problem with annotation processing in multiple modules (the sample modules uses annotation porcessing). However, just pressing the "green play button" (and ignoring any previous error messages from console) works just fine on my mac book with android studio 1.2 Beta 3.

@sockeqwe that's strange. I'm also using Mac OS X and pressing green button does nothing. Maybe Android Studio cached something from your previous builds?

@ziem Really strange! I already checked that, but it still works after doing a clean.
Even from commandline with
./gradlew clean assemble --stacktrace it compiles without any problems.

I used other computer (OS: Mavericks, AS: 1.2 Beta) to run mosby. I cloned repo then I opened project using "Open existing Android Studio project" and chose root build.gradle. After syncing/building I pressed play and I got errors similar to @TechFreak stacktrace then I pressed play button again, now I got errors with MosbyActivity. Pressed few times more - the same result. I also tried ./gradlew clean assemble --stacktrace - still the same.

Tried removing the definition of versions for each libraries from the root gradle file, and using the exact values on each project, but it still doesn't work.
I tried checking out latest updates from a Linux pc, and got a different error: https://gist.github.com/dee73a2513a1f7e53617
It seems to be complaining in the :core-dagger1:androidJavadocs task. Is there a way to skip javadocs generation?

Ok, I think I've found the issue and a possible workaround. Using my linux pc I switched from Java 8 to Java 7, I got a lot of warnings about javadocs but the samples built just fine.
This (http://blog.joda.org/2014/02/turning-off-doclint-in-jdk-8-javadoc.html) seems to be the issue, apparently Java 8 is more strict about javadocs than java7, and that's what we are seeing here. I'll try to check back on my windows pc to see if switching to java 7 solves the issue there as well.
I see 2 solutions to make this work on Java 8, first is to fix all of the javadoc warnings (there are a lot) or second is to disable javadoc lint from the gradle scripts. @sockeqwe not sure which one would you prefer? but I can help you either way.
If anyone else can canfirm if switching to java 7 solves their issues it would be great!

OMG! It's true! Javadoc is the problem!
In Java 7 javadoc errors result in warnings. Java8 is more strict and will abort compiling.
@TechFreak  Thank you very much!
I removed javadoc generation temporarly. Could you please pull the master branch and check if that works on your machine?
I tried to disable java doc lint by using
task androidJavadocs(type: Javadoc) {
    failOnError false
    source = android.sourceSets.main.java.srcDirs
    classpath += project.files(android.getBootClasspath().join(File.pathSeparator))
  }

but it doesn't work right now.
However, I will try to find a solution for them (fixing the javadoc line errors is also an option )
What command did you run to get the javadoc lint errors?

@sockeqwe Thanks for the suggestion to failonError false.  That did the trick for me.  Such a convoluted problem with a such a simple fix!  I can continue to use Java8 now.  Have some bitcoin $1 on me @changetip.

Hi @sockeqwe, @IgorGanapolsky sent you a Bitcoin tip worth 4,474 bits ($1.00), and I'm here to deliver it ➔ collect your tip.
Learn more about ChangeTip

@sockeqwe Thanks - last commit solves problem :+1: .

@sockeqwe Thanks! that fixed it, I can build perfectly from my windows box using java 8 and the latest version from master. BTW the mail example is awesome!

@TechFreak @ziem @IgorGanapolsky I apologize for the inconvenience!
@IgorGanapolsky Thanks for the bitcoin :+1:  but I prefer a real beer whenever we will meet us :)
Btw. The mail example is a work in progress and not finisehed yet. I publish a blog post about the mail app example with some tips of how to use mosby in a real world app.

It would be awesome if you could augment the mail sample with scrolling tabs.  That would pretty much complete the gamut of Material Design + MVP pattern + dependency injection + RXJava.

if I find time I will add "profiles" for mail senders, but I'm not sure what the tabs should contain ^^
or
are you asking me about how to implement an activity with tabs in mvp, i.e. the number of tabs and type of fragment the tabs will display are loaded dynamically i.e. from a server?

I am interested in implementing tabs with Toolbar in MVP.  Your second
point...
On Apr 17, 2015 3:58 PM, "Hannes Dorfmann"<EMAIL_ADDRESS>wrote:

if I find time I will add "profiles" for mail senders, but I'm not sure
what the tabs should contain ^^
or
are you asking me about how to implement an activity with tabs in mvp,
i.e. the number of tabs and type of fragment the tabs will display are
loaded dynamically i.e. from a server?
—
Reply to this email directly or view it on GitHub
https://github.com/sockeqwe/mosby/issues/10#issuecomment-94062740.

I cant promise to implement, but I try to find time.
Actually, with mosby it's just an MvpLceActivity, where the "Content" is a LinearLayout containing  Tabbar and ViewPager. The "data" is a list of "tabs" like
class TabsData {
     String name;  // The name displayed in the tabbar
     String fragmentType;  // some kind of mapping from "foo" to "FooFragment"
}

Each Fragment in the ViewPager has it's own MVP thing ... So Activity uses MVP to control the Tabbar PagerAdapter and ViewPager  and each "tab" is a Fragment having it's own MVP thing ...
Hope that helps

It's a start :)  I'm trying to keep it from getting complicated.  Google's
iosched source code implements tabs in a very convoluted way.  So I was
wondering if MVP would simplify this process...
Thanks,
Igor
On Apr 17, 2015 4:05 PM, "Hannes Dorfmann"<EMAIL_ADDRESS>wrote:

I cant promise to implement, but I try to find time.
Actually, with mosby it's just an MvpLceActivity, where the "Content" is a
LinearLayout containing Tabbar and ViewPager. The "data" is a list of
"tabs" like
class TabsData {
String name;  // The name displayed in the tabbar
String fragmentType;  // some kind of mapping from "foo" to "FooFragment"
}
Each Fragment in the ViewPager has it's own MVP thing ... So Activity uses
MVP to control the Tabbar PagerAdapter and ViewPager and each "tab" is a
Fragment having it's own MVP thing ...
Hope that helps
—
Reply to this email directly or view it on GitHub
https://github.com/sockeqwe/mosby/issues/10#issuecomment-94063896.


Android : Error while updating property 'region' of a view managed by : AIRMap

While everything works fine on iOS, I get this error when running on Android :

My MapView component :
<MapView
            ref='map view'
            rotateEnabled={false}
            style={styles.restaurantsMap}
            showsUserLocation={this.state.showsUserLocation}
            region={this.state.region}
            onRegionChangeComplete={this.onRegionChangeComplete}
            onRegionChange={this.onRegionChange}>
            <MapView.Circle
              center={center}
              radius={this.state.radius}
              fillColor='rgba(0, 0, 0, 0.1)'
              strokeColor='#FE3139' />
          </MapView>

I was having the same issue! Debbuging the component I realized that region needs latitudDelta and longitudDelta. Once you add those props everything will work just fine.. At least that worked for me.

I had this issue too, and doing what @DennisMG did resolved the issue.
Tip: Use the Chrome debugging. It will provide you with a very clear error, in this case, it would've told you that region needs latitudeDelta and longitudeDelta as arguments as well.

I'm having the same issue, I added the deltas coordinates but I still get this error:
            <MapView
              region={{
                latitude: 52.3015493,
                longitude: 4.6939769,
                latitudeDelta: 3,
                longitudeDelta: 4,
              }}
              style={styles.map}
              showsUserLocation
              zoomEnabled={false}
              scrollEnabled={false}
              rotateEnabled={false}
            >
              <MapView.Marker
                coordinate={{latitude:52.3015493,
                  longitude:4.6939769}} />
            </MapView>

Ok, I figured it out what was my problem in this case:
If you use showsUserLocation props you need to add the ACCESS_FINE_LOCATION permission into your AndroidManifest.xml.
<uses-permission android:name="android.permission.ACCESS_FINE_LOCATION"/>

That worked for me

@ghamaide are you still getting the error?

Closing this for now, please reopen if that's still an issue

Mr@lucasfeliciano    already i added Still getting error.

In my case, my issue was that the latitude and longitude passed to MapView were typeof string, just did a parseFloat before.

Thanks @tenhsor1 that was my issue too!

In my case, my issue was that the latitude and longitude props passed to MapView were typeof string. Fixed doing a parseFloat before.

I fixed by changing my first declarations from latitude: null to latitude: 0 as null was being read as type string.

In my case, my issue was that the latitude and longitude props passed to MapView were typeof string. Fixed doing a parseFloat before.

you are a life savior !!!!!! @tenhsor1

In my case latitude and longitude were missing from camera.center:
<MapView
  ...
  camera={{
    center: {
      latitude: parseFloat(group?.facility?.latitude),
      longitude: parseFloat(group?.facility?.longitude),
    },
    ...
  }}>

In my case it was a typo:
I used longTitude instead of longitude

While everything works fine on iOS, I get this error when running on Android :

I had the same error using yarn add react-native-maps to display a MapView component
Using expo install react-native-maps fixed things!

I am using expo-cli to see the mapView component. I installed it and just added the below code
import React, { useState } from "react";
import {  StyleSheet,  Text,  View} from "react-native";
import MapView from "react-native-maps";

const Map = () => {

  return (

    <View style={styles.container}>
        <Text style={styles.text}>
            Maps
        </Text>
        <MapView
           initialRegion={{
           latitude: 37.78825,
           longitude: -122.4324,
           latitudeDelta: 0.0922,
           longitudeDelta: 0.0421,
        }}
       />
      </View>
  )
}

const styles = StyleSheet.create({
    container: {
        flex:1,
        justifyContent:'center',
        alignItems:'center'
    },
    text:{
        fontSize:40,
        margin:40,
    }
})
export default Map;

And I still found this error

How to solve this?

Uma solução que achei foi substituir initialRegion por initialCamera desta forma:
      <MapView
        initialCamera={{
          center: { latitude: -4.143009,  longitude: -40.582803 },
          pitch: 0,
          zoom: 18,
          heading: 0,
          altitude: 1000,
        }}
        provider={PROVIDER_GOOGLE}
      >

Uma solução que achei foi substituir initialRegion por initialCamera desta forma:
      <MapView
        initialCamera={{
          center: { latitude: -4.143009,  longitude: -40.582803 },
          pitch: 0,
          zoom: 18,
          heading: 0,
          altitude: 1000,
        }}
        provider={PROVIDER_GOOGLE}
      >

The solution I found for this error was to use a previous version of react native maps, using the command expo doctor --fix-dependencies

Same problem here, but i realized the warning: "Some dependencies are incompatible with the installed expo package version: react-native-maps - expected version: 0.30.2 - actual version installed: 0.31.1"

So.. I've run:
expo doctor --fix-dependencies
et voilá!

you saved my day

While everything works fine on iOS, I get this error when running on Android :

I had the same error using yarn add react-native-maps to display a MapView component
Using expo install react-native-maps fixed things!

you have saved my day

I'm having the same issue, I added the deltas coordinates but I still get this error:
            <MapView
              region={{
                latitude: 52.3015493,
                longitude: 4.6939769,
                latitudeDelta: 3,
                longitudeDelta: 4,
              }}
              style={styles.map}
              showsUserLocation
              zoomEnabled={false}
              scrollEnabled={false}
              rotateEnabled={false}
            >
              <MapView.Marker
                coordinate={{latitude:52.3015493,
                  longitude:4.6939769}} />
            </MapView>

Worked for me, Thanks

While everything works fine on iOS, I get this error when running on Android :

I had the same error using yarn add react-native-maps to display a MapView component
Using expo install react-native-maps fixed things!

This fixed it for me! Thanks a lot!

While everything works fine on iOS, I get this error when running on Android previous it working but now not working

While everything works fine on iOS, I get this error when running on Android previous it working but now not working

I'm having the same issue.

@DevangMstryls Did you find the solution for this issue ?

@Saichand13 @DevangMstryls  guys we are rocking on the same boat, coz I got the same error while using the "map Polyline" if you find any solution then share it. 💯

@logan0900 found a temporary fix change  googlePlayServicesVersion = "+" to  googlePlayServicesVersion = "17.0.0" in android/build.gradle.
Working for me

@Saichand13  Android works fine for me, I got an error in IOS.
here it is.

expo install react-native-maps

Running this command fixed it for me as well!
Cheers.

I got this error in EXPO SDK 48 v1.3.1


Implement NATS configuration setup for authenticated access

[x] Change in NATS configuration as per security point of view (added authentication key to access NATS)

Implementation is done


Stm reduce transmute

Content

This PR aims to reduce or completely remove the use of transmute in multi_sig.rs.
Pre-submit checklist

Branch

[x] Commit sequence broadly makes sense
[x] Key commits have useful messages

PR

[x] No clippy warnings in the CI
[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

Comments
The following are the benchmark results giving the regression between the versions with transmute and serde.  The performance has regressed drastically. So, we should consider another way to prevent the possible problems of using transmute which is also efficient. As we discussed, a helper structure for unsafe codes could be better @iquerejeta.
STM/Blake2b/Key registration/k: 25, m: 150, nr_parties: 300
                        time:   [502.24 ms 502.95 ms 503.73 ms]
                        change: [+26.772% +27.320% +27.940%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Play all lotteries/k: 25, m: 150, nr_parties: 300
                        time:   [908.94 µs 910.58 µs 912.56 µs]
                        change: [+31.221% +31.619% +32.056%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Aggregation/k: 25, m: 150, nr_parties: 300
                        time:   [24.401 ms 24.622 ms 24.895 ms]
                        change: [+26.608% +27.610% +29.137%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Key registration/k: 250, m: 1523, nr_parties: 2000
                        time:   [3.3481 s 3.3779 s 3.4273 s]
                        change: [+23.353% +27.009% +30.851%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Play all lotteries/k: 250, m: 1523, nr_parties: 2000
                        time:   [7.5255 ms 7.5313 ms 7.5375 ms]
                        change: [+30.692% +30.971% +31.228%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Aggregation/k: 250, m: 1523, nr_parties: 2000
                        time:   [238.58 ms 238.82 ms 239.12 ms]
                        change: [+24.642% +24.856% +25.077%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Batch Verification/k: 25, m: 150, nr_parties: 300/batch size: 1
                        time:   [2.9301 ms 2.9354 ms 2.9427 ms]
                        change: [+30.118% +30.369% +30.663%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Batch Verification/k: 25, m: 150, nr_parties: 300/batch size: 10
                        time:   [23.629 ms 23.650 ms 23.674 ms]
                        change: [+31.475% +31.664% +31.845%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Batch Verification/k: 25, m: 150, nr_parties: 300/batch size: 20
                        time:   [46.031 ms 46.192 ms 46.431 ms]
                        change: [+32.254% +32.681% +33.157%] (p = 0.00 < 0.05)
                        Performance has regressed.
STM/Blake2b/Batch Verification/k: 25, m: 150, nr_parties: 300/batch size: 100
                        time:   [230.16 ms 230.38 ms 230.62 ms]
                        change: [+31.979% +32.165% +32.361%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Batch Verification/k: 250, m: 1523, nr_parties: 2000/batch size: 1
                        time:   [4.3937 ms 4.4235 ms 4.4583 ms]
                        change: [+18.011% +19.305% +20.655%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Batch Verification/k: 250, m: 1523, nr_parties: 2000/batch size: 10
                        time:   [45.043 ms 45.313 ms 45.601 ms]
                        change: [+22.106% +23.430% +24.636%] (p = 0.00 < 0.05)
                        Performance has regressed.
STM/Blake2b/Batch Verification/k: 250, m: 1523, nr_parties: 2000/batch size: 20
                        time:   [97.238 ms 100.43 ms 104.30 ms]
                        change: [+44.286% +49.695% +54.132%] (p = 0.00 < 0.05)
                        Performance has regressed.

STM/Blake2b/Batch Verification/k: 250, m: 1523, nr_parties: 2000/batch size: 100
                        time:   [396.87 ms 397.68 ms 398.52 ms]
                        change: [+34.851% +35.222% +35.516%] (p = 0.00 < 0.05)
                        Performance has regressed.

Issue(s)

Closes #532

As mentioned by @curiecrypt, the regression is considerable, and we won't proceed on that path. Doing serialisation/deserialisation, while being safer, has an important hit on performance. In order to minimise the risks of transmute, we will create helper function (to facilitate auditing), and fix to a particular version of blst library to avoid changes under the hood. Closing this in favour of #690 .


getDisplayName API

I have code a little Plugin with LuckPerms API but i need help. How can i read the Displayname (perm on MySQL is: "displayname.Owner"
I can read whit this only GroupName and shows as owner but i configure it as Owner
LuckPermsApi api = LuckPerms.getApi();
User user = api.getUser(p.getUniqueId())
String GroupData = user.getPrimaryGroup();
How can i change that GroupName to Owner or read the DisplayName ?

Take the result of user.getPrimaryGroup and pass that group name to api.getGroup - that will give you a Group object.
You can then use group.getFriendlyName to get its display name.
https://github.com/lucko/LuckPerms/blob/6fc2321fada78fa23b01e0d7b4ef6be72878fbd3/api/src/main/java/me/lucko/luckperms/api/PermissionHolder.java#L73-L88


Fix stalling tests

Trying to pin point the exact commit which stalls the tests.

Not really sure what's happening on the stalling tests, logs show jest can't exit after some failed tests.
Everything I tried here passes the test, maybe it's about the duplicated key in pnpm-lock.yaml?
Let's try merging this, we'll see after renovatebot rebases its PRs.


cifs & nested test stablization

Improve the test code lines for cifs and KVM nested test call.
PR #70

#70

Done


Timeout error while cf push

Please fill out the issue checklist below and provide ALL the requested information.

[ ] I reviewed open and closed github issues that may be related to my problem.
[ ] I tried updating to the latest version of the CF CLI to see if it fixed my problem.
[ ] I attempted to run the command with CF_TRACE=1 to help debug the issue.
[ √] I am reporting a bug that others will be able to reproduce.

Describe the bug and the command you saw an issue with
Provide details on what you were trying to do (and why).

Getting Timeout issue while cf push, failing on uploading code file. This issue is coming intermittently/randomly at any time.Below is the error code.
$ Response 502
$ code 0, Title : , Detail: 502 Bad gateway : Registered endpoint failed to handle request.**

What happened
A clear and concise description of what happen.

Need to push code mutiple time to get success result.

Expected behavior
A clear and concise description of what you expected to happen.

It should not give timeout error, as this is critical bug.

Exact Steps To Reproduce
Steps to reproduce the behavior; include the exact CLI commands and verbose output:

Run cf push <app-name> --strategy rolling
Failing while pushing through azure devops(Windows) , VMware accepted that this issue is related to cli.

Provide more context

platform and shell details ( e.g. Mac OS X 10.11 iTerm)

Azure devops (Windows)

version of the CLI you are running

cf v7.1.0+4c3168f9a

version of the CC API Release you are on

CAPI - 1.95.2

Notes regarding V6 and V7 CLI support:

V6:

Minimum supported version of CF Deployment: v7.0.0 (CAPI Release: 1.74.0 (APIs 2.128.0 and 3.63.0))
Maximum supported version of CF Deployment: v13.4.0 (CAPI Release: 1.94.0 (APIs 2.149.0 and 3.84.0))

V7:

Minimum supported version of CF Deployment: v13.5.0 (CAPI Release: 1.95.0 (APIs 2.150.0 and 3.85.0))

For your reference, a Jira had been opened to the CAPI team. The link is https://pivotal-io.atlassian.net/browse/CC-36. The backgournd and the logs can be found in the Jira too. It is suspected that this issue being in the CLI itself.

Thank you for your submission. If you still run into this issue with the latest CLI v8, please open a new ticket with the necessary information mentioned in our issue template.


using node-pty to support windows builds

node-pty is a fork of pty.js which works on windows machines - tested on both windows and centos - the windows fails because getpass doesn't work on windows.
Some of our windows developers use modules that depend on getpass and this is causing us problems - could you review this pull request and see if you could publish to npm if you agree with it?
Thanks in advance

This doesn't seem unreasonable, but I am curious -- pty.js is only in devDependencies not dependencies, because it's not required to use the library, just to run the unit tests. These unit tests are not shipped in the npm package getpass, so your Windows developers must be checking the getpass module out of git directly? Is this the normal way they're bringing in dependencies?

yes we are doing an npm install for local development as well as on our
build machines so we get the dev dependencies for our own unit tests etc.
i think when i traced what was using getpass it was restify and a few other
modules are using it.
On Apr 21, 2017 1:15 PM, "Alex Wilson"<EMAIL_ADDRESS>wrote:

This doesn't seem unreasonable, but I am curious -- pty.js is only in
devDependencies not dependencies, because it's not required to use the
library, just to run the unit tests. These unit tests are not shipped in
the npm package getpass, so your Windows developers must be checking the
getpass module out of git directly? Is this the normal way they're bringing
in dependencies?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arekinath/node-getpass/pull/3#issuecomment-296282983,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACOKH10IrOpMHp4l5SaPWPzS0w_7H1l1ks5ryQA8gaJpZM4NEmc3
.

@arekinath - we use npm install which by default pulls in all dev dependencies for all the modules we depend on. This causes our windows development machines and build machines to fail because they try to pull down pty.js which has problems on windows currently.
Would you be able to help us out by taking this pull request and publishing a new version of getpass to npm?
Thanks in advance

Hi @jonathanmccracken -- my main concern here at the moment is that node-pty does not appear to support node.js versions prior to 4.x. I can't get it to install on node 0.10 or 0.12 (which are both still supported for our commandline tools that node-getpass was written for).
Can you not use npm install --production or set NODE_ENV to avoid installing devDependencies that you aren't going to use?

@arekinath thanks for looking into this. We have tried using npm i --production but unfortunately we need some dev dependencies some of the packages like node-sass and contentful-export
I am unaware of how to exclude a specific package's dev dependencies without excluding all of them. We'll try to find an alternative solution, but it appears getpass is extremely popular as it pty.ps for many npm modules except just not on windows machines :(
again thanks for looking into this for us

Hmm... since I haven't shipped the tests in the npm package, maybe I should just upload it to npm without any devDependencies at all? That would seem to solve the problem as well. It's kind of annoying to have package.json different in npm to the repository, but in this case it might not be so bad (this library doesn't really get updated very often, since it's so small).

Published 0.1.7 to npm with the devDependencies removed

@arekinath appreciate you pushing this out so quickly
We were also able to solve the original problem with node-gyp by doing the following:
install -g node-gyp
install -g --production windows-build-tools

I figure I can close this PR as this is totally resolved. Thanks again 👍


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.25.7

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.25.6 -> v1.25.7

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.25.7
Compare Source
Bug Fixes

deps: update gomod (#721) (addd7eb)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

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


Build with GitHub Actions

It would be very nice, to have GitHub build the project automatically on commits.
We could hook up some GitHub Actions for that.

This would be nice, but I am not aware of a good way to achieve that.
The main problem is that our integration tests depend on password-protected data that I think we can not share (old TREC runs etc).
I would be not happy with building a jar automatically without performing those tests because I think that they are very helpful and cover important aspects of our evaluation.
We should keep this in mind, as soon as we have a good idea on how we can handle the problem with the tests we can build with github actions.

Closing due to inactivity.


Addition of what seems pretty radom items

In 6.4.0 getting the following added to a lot of elements that I cannot figure out why.
element {
content: '';
display: table;
}
This does not happen in 6.3.0 and have had to revert

this is the same bug as described in #143

Fixed https://github.com/corysimmons/lost/pull/148


[SPARK-32168][SQL] Fix hidden partitioning correctness bug in SQL overwrite

What changes were proposed in this pull request?
When converting an INSERT OVERWRITE query to a v2 overwrite plan, Spark attempts to detect when a dynamic overwrite and a static overwrite will produce the same result so it can use the static overwrite. Spark incorrectly detects when dynamic and static overwrites are equivalent when there are hidden partitions, such as days(ts).
This updates the analyzer rule ResolveInsertInto to always use a dynamic overwrite when the mode is dynamic, and static when the mode is static. This avoids the problem by not trying to determine whether the two plans are equivalent and always using the one that corresponds to the partition overwrite mode.
Why are the changes needed?
This is a correctness bug. If a table has hidden partitions, all of the values for those partitions are dropped instead of dynamically overwriting changed partitions.
This only affects SQL commands (not DataFrameWriter) writing to tables that have hidden partitions. It is also only a problem when the partition overwrite mode is dynamic.
Does this PR introduce any user-facing change?
Yes, it fixes the correctness bug detailed above.
How was this patch tested?

This updates the in-memory table to support a hidden partition transform, days, and adds a test case to DataSourceV2SQLSuite in which the table uses this hidden partition function. This test fails without the fix to ResolveInsertInto.
This updates the test case InsertInto: overwrite - multiple static partitions - dynamic mode in InsertIntoTests. The result of the SQL command is unchanged, but the SQL command will now use a dynamic overwrite so the test now uses dynamicOverwriteTest.

@cloud-fan, @brkyvz, @aokolnychyi, @dbtsai, @dongjoon-hyun, you may be interested in this PR. This fixes a correctness bug in SQL INSERT INTO with v2 tables. It only affects hidden partitioned tables, so the impact is limited. I think we should aim to get this into 3.0.1 if possible.

Thank you for pinging me, @rdblue .

ok to test

BTW, AmpLab Jenkins farm has been out of order since last Friday.

Test build #124925 has started for PR 28993 at commit fd65dd4.

Merged build finished. Test PASSed.

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

Test build #124925 has finished for PR 28993 at commit fd65dd4.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Hi, @rdblue . Could you take a look at the relevant UT failures?

Good catch! LGTM, let's update the failed tests.

The tests were failing because InMemoryTable was ignoring hidden partitions when building partition keys. I implemented the rest of the transforms needed for the tests to fix it, so now we have an implementation of hidden partitioning.

Test build #125112 has started for PR 28993 at commit afef6ce.

Merged build finished. Test PASSed.

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

Thank you for updating, @rdblue .

Test build #125112 has finished for PR 28993 at commit afef6ce.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Retest this please.

Merged build finished. Test FAILed.

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

Test build #125119 has started for PR 28993 at commit afef6ce.

Merged build finished. Test PASSed.

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

Test build #125119 has finished for PR 28993 at commit afef6ce.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

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

Test build #125142 has started for PR 28993 at commit afef6ce.

Merged build finished. Test PASSed.

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

Test build #125142 has finished for PR 28993 at commit afef6ce.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #125193 has started for PR 28993 at commit afef6ce.

Merged build finished. Test PASSed.

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

Test build #125193 has finished for PR 28993 at commit afef6ce.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #125222 has started for PR 28993 at commit afef6ce.

Merged build finished. Test PASSed.

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

Test build #125222 has finished for PR 28993 at commit afef6ce.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #125256 has started for PR 28993 at commit 2efb84c.

Merged build finished. Test PASSed.

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

One question https://github.com/apache/spark/pull/28993#discussion_r451234819. Otherwise, I am okay considering that it's still under development.

Test build #125256 has finished for PR 28993 at commit 2efb84c.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

python/pyspark/mllib/tests/test_streaming_algorithms.py", line 461, in condition
self.assertGreater(errors[1] - errors[-1], 2)
AssertionError: 1.672640157855923 not greater than 2

seems like a flaky test. @huaxingao  can you take a look?

retest this please

Test build #125297 has started for PR 28993 at commit 2efb84c.

Merged build finished. Test PASSed.

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

Test build #125297 has finished for PR 28993 at commit 2efb84c.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #125322 has started for PR 28993 at commit 2efb84c.

Merged build finished. Test PASSed.

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

Test build #125322 has finished for PR 28993 at commit 2efb84c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #125355 has started for PR 28993 at commit 2efb84c.

Merged build finished. Test PASSed.

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

Test build #125355 has finished for PR 28993 at commit 2efb84c.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

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

Test build #125369 has started for PR 28993 at commit 2efb84c.

Test build #125369 has finished for PR 28993 at commit 2efb84c.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #125382 has started for PR 28993 at commit 2efb84c.

Merged build finished. Test PASSed.

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

Test build #125382 has finished for PR 28993 at commit 2efb84c.

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

Thank you, @rdblue and all. Merged to master/3.0.

Merged build finished. Test PASSed.

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

Thanks to everyone that reviewed, and to everyone who helped keep the tests running until they passed!


[openblas] Support feature single-thread

The BLAS library openblas provides a setting make USE_THREAD=0 to build a single-thread library. See here
https://fossies.org/linux/OpenBLAS/USAGE.md
However, if I do vcpkg install openblas, I have no idea how to specify this option. I have read the help of vcpkg install, it does not seem to provide a way to pass through a make build option.
The question can be generalized to: if a package author provides multiple make or cmake options to customize the build, how can vcpkg pass through these options when using vcpkg install?

We can implement this feature through the options OPTIONS /OPTIONS_DEBUG /OPTIONS_RELEASE in vcpkg_configure_cmake or vcpkg_configure_make etc., see documentation.

We can implement this feature through the options OPTIONS /OPTIONS_DEBUG /OPTIONS_RELEASE in vcpkg_configure_cmake or vcpkg_configure_make etc., see documentation.

@JackBoosY @JonLiu1993 Thanks for the reply. I have read the documentation you referred. If I understand correctly, this is a way to specify a build option when you create a port.
But after the port has been created (without a build option), how can I pass an additional build option at install time via vcpkg install?

@TonyXiang8787 For example, see:
https://github.com/microsoft/vcpkg/blob/1d415e8dc3afef0ae85b4d00ea56013b2f26bb46/ports/libiconv/portfile.cmake#L24-L36

@TonyXiang8787 ,Does it still a question for you?

@JonLiu1993  Thanks for the answer. If I understand correctly, I have to modify the portfile.cmake file of openblas to add build options. This file is however part of the GIT version control. So does it mean my version of vcpkg diverges from the main?

I need to confirm some thing:

Openblas provides two related feature: USE_THREAD and USE_SIMPLE_THREADED_LEVEL3, which one do you mean?
Seems that if we enable USE_SIMPLE_THREADED_LEVEL3, we need to enable USE_THREAD, right?

@JackBoosY I am self only familiar with the option USE_THREAD. I set USE_THREAD=0 if I want to build open blad manually in single thread.


Seg fault on Initial Conditions

Error using mingw64 and g++ on windows 10.
on a fresh download of hydrad and a fresh download of mingw (x86_64-8.1.0-release-posix-seh-rt_v6-rev0) to provide gcc and g++ on windows (other versions of mingw also cause the same problem)
mingw placed onto the system path
initial conditions compile and run. Using the default settings from the GUI to run a test simulation.
Outputs initial.amr.gravity and initial.amr.table and encounters a segfault in writing the .amr file.
testing/fiddling finds that the segfault is inside writeAMRfile, in either of the two ppiID loops at lines 156 and 167.
Adjusting the minimum cells, the error occurs at just past 50% of the array allocation.
If one reduces the minimum grid cells and refinement to absurdly low values (4-6 grid cells and refinement 0 or 1) then the .amr file writes but still crashes in writing the subsequent files.
No issues encountered if a uniform grid is requested, even up to 10k cells.
uploading the same files to a linux machine and running through commandline works, so the problem lies with the windows side of the problem.
running Initial_Conditions.bat in the main dir gives the following output.
C:\Users\junverferth\MHD\HYDRAD\HYDRAD-10-11-2022\Initial_Conditions\build_scripts>g++ -O3 -flto -Wall -Wno-unused-variable -Wno-write-strings ../source/main.cpp ../source/ode.cpp ../source/misc.cpp ../../Radiation_Model/source/element.cpp ../../Radiation_Model/source/radiation.cpp ../../Radiation_Model/source/OpticallyThick/OpticallyThickIon.cpp ../../Radiation_Model/source/OpticallyThick/RadiativeRates.cpp ../../Resources/source/gammabeta.cpp ../../Resources/source/fitpoly.cpp ../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp ../../Resources/Utils/regPoly/regpoly.cpp ../../Resources/Utils/regPoly/nrutil.cpp ../../Resources/source/file.cpp -o ../../Initial_Conditions.exe
In member function 'OpenPieceWiseFit',
inlined from '__ct_base ' at ../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp:32:18,
inlined from 'main' at ../source/main.cpp:93:73:
../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp:59:38: warning: argument 1 range [18446744056529682432,<PHONE_NUMBER>3709551608] exceeds maximum object size<PHONE_NUMBER>854775807 [-Walloc-size-larger-than=]
ppfSDCoefficient = (double**)malloc( sizeof(double*) * inumSD );
^
../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp: In function 'main':
C:/Users/junverferth/MHD/x86_64-8.1.0-release-posix-seh-rt_v6-rev0/mingw64/x86_64-w64-mingw32/include/stdlib.h:503:17: note: in a call to allocation function 'malloc' declared here
void __cdecl malloc(size_t _Size);
^
In member function 'OpenPieceWiseFit',
inlined from '__ct_base ' at ../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp:32:18,
inlined from 'main' at ../source/main.cpp:93:73:
../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp:60:33: warning: argument 1 range [18446744056529682432,<PHONE_NUMBER>3709551608] exceeds maximum object size<PHONE_NUMBER>854775807 [-Walloc-size-larger-than=]
ppfSDMinMax = (double*)malloc( sizeof(double*) * inumSD );
^
../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp: In function 'main':
C:/Users/junverferth/MHD/x86_64-8.1.0-release-posix-seh-rt_v6-rev0/mingw64/x86_64-w64-mingw32/include/stdlib.h:503:17: note: in a call to allocation function 'malloc' declared here
void *__cdecl malloc(size_t _Size);
^
C:\Users\junverferth\MHD\HYDRAD\HYDRAD-10-11-2022\Initial_Conditions\build_scripts>cd ../..
C:\Users\junverferth\MHD\HYDRAD\HYDRAD-10-11-2022>Initial_Conditions.exe
Calculating initial hydrostatic conditions...
Peak heating range = 6.2661e-004 -> 6.2806e-004 erg cm^-3 s^-1
Optimum peak heating rate = 6.2748e-004 erg cm^-3 s^-1
Writing initial conditions file...
C:\Users\junverferth\MHD\HYDRAD\HYDRAD-10-11-2022>

Hmmm… Thanks for the feedback, very interesting. I’ve not used mingw64 for g++ under Windows before, preferring cygwin instead – which has not given me any problems like this. Evidently, the Windows 10 version of mingw64 is doing something odd / different with memory allocation.
I spotted one possible issue on Line 154 of Initial_Conditions/source/misc.cpp. Try replacing that line with the following:
Line 154: ppiID = (long**)malloc( sizeof(long*) * iMAX_CELLS );
Since it is allocating memory for an array of pointers to pointers then the byte length of a pointer to the variable type (long int) should be passed to the sizeof() function, rather than the byte length of the variable type itself. I’m not confident it will work, but it’s a first thing to try. Otherwise, a “fix” to at least get up and running under Win 10 is to use cygwin to provide Linux-like functionality.
Cheers,
Steve
From: junverferth @.>
Sent: Wednesday, October 12, 2022 7:00 AM
To: rice-solar-physics/HYDRAD @.>
Cc: Subscribed @.***>
Subject: [rice-solar-physics/HYDRAD] Seg fault on Initial Conditions (Issue #95)
Error using mingw64 and g++ on windows 10.
on a fresh download of hydrad and a fresh download of mingw (x86_64-8.1.0-release-posix-seh-rt_v6-rev0) to provide gcc and g++ on windows (other versions of mingw also cause the same problem)
mingw placed onto the system path
initial conditions compile and run. Using the default settings from the GUI to run a test simulation.
Outputs initial.amr.gravity and initial.amr.table and encounters a segfault in writing the .amr file.
testing/fiddling finds that the segfault is inside writeAMRfile, in either of the two ppiID loops at lines 156 and 167.
Adjusting the minimum cells, the error occurs at just past 50% of the array allocation.
If one reduces the minimum grid cells and refinement to absurdly low values (4-6 grid cells and refinement 0 or 1) then the .amr file writes but still crashes in writing the subsequent files.
No issues encountered if a uniform grid is requested, even up to 10k cells.
uploading the same files to a linux machine and running through commandline works, so the problem lies with the windows side of the problem.
running Initial_Conditions.bat in the main dir gives the following output.
C:\Users\junverferth\MHD\HYDRAD\HYDRAD-10-11-2022\Initial_Conditions\build_scripts>g++ -O3 -flto -Wall -Wno-unused-variable -Wno-write-strings ../source/main.cpp ../source/ode.cpp ../source/misc.cpp ../../Radiation_Model/source/element.cpp ../../Radiation_Model/source/radiation.cpp ../../Radiation_Model/source/OpticallyThick/OpticallyThickIon.cpp ../../Radiation_Model/source/OpticallyThick/RadiativeRates.cpp ../../Resources/source/gammabeta.cpp ../../Resources/source/fitpoly.cpp ../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp ../../Resources/Utils/regPoly/regpoly.cpp ../../Resources/Utils/regPoly/nrutil.cpp ../../Resources/source/file.cpp -o ../../Initial_Conditions.exe
In member function 'OpenPieceWiseFit',
inlined from '__ct_base ' at ../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp:32:18,
inlined from 'main' at ../source/main.cpp:93:73:
../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp:59:38: warning: argument 1 range [18446744056529682432,<PHONE_NUMBER>3709551608] exceeds maximum object size<PHONE_NUMBER>854775807 [-Walloc-size-larger-than=]
ppfSDCoefficient = (double**)malloc( sizeof(double*) * inumSD );
^
../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp: In function 'main':
C:/Users/junverferth/MHD/x86_64-8.1.0-release-posix-seh-rt_v6-rev0/mingw64/x86_64-w64-mingw32/include/stdlib.h:503:17: note: in a call to allocation function 'malloc' declared here
void __cdecl malloc(size_t _Size);
^
In member function 'OpenPieceWiseFit',
inlined from '__ct_base ' at ../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp:32:18,
inlined from 'main' at ../source/main.cpp:93:73:
../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp:60:33: warning: argument 1 range [18446744056529682432,<PHONE_NUMBER>3709551608] exceeds maximum object size<PHONE_NUMBER>854775807 [-Walloc-size-larger-than=]
ppfSDMinMax = (double*)malloc( sizeof(double*) * inumSD );
^
../../Resources/Utils/generatePieceWiseFit/source/piecewisefit.cpp: In function 'main':
C:/Users/junverferth/MHD/x86_64-8.1.0-release-posix-seh-rt_v6-rev0/mingw64/x86_64-w64-mingw32/include/stdlib.h:503:17: note: in a call to allocation function 'malloc' declared here
void *__cdecl malloc(size_t _Size);
^
C:\Users\junverferth\MHD\HYDRAD\HYDRAD-10-11-2022\Initial_Conditions\build_scripts>cd ../..
C:\Users\junverferth\MHD\HYDRAD\HYDRAD-10-11-2022>Initial_Conditions.exe
Calculating initial hydrostatic conditions...
Peak heating range = 6.2661e-004 -> 6.2806e-004 erg cm^-3 s^-1
Optimum peak heating rate = 6.2748e-004 erg cm^-3 s^-1
Writing initial conditions file...
C:\Users\junverferth\MHD\HYDRAD\HYDRAD-10-11-2022>
—
Reply to this email directly, view it on GitHub https://urldefense.com/v3/__https:/github.com/rice-solar-physics/HYDRAD/issues/95__;!!BuQPrrmRaQ!nqvQmUA2IotpDKUOF-hRoQizJT7s3qYKiEIgtckWTWP06qflDfA25P33uhhaWqKTbSpFLx9N_C5lBvCCwNNjFIx5mf5LQ9QGHuk$ , or unsubscribe https://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/ACC6C7WISSLUO3D4TT7H573WC2R47ANCNFSM6AAAAAARDG4HLA__;!!BuQPrrmRaQ!nqvQmUA2IotpDKUOF-hRoQizJT7s3qYKiEIgtckWTWP06qflDfA25P33uhhaWqKTbSpFLx9N_C5lBvCCwNNjFIx5mf5LbwTfAi0$ .
You are receiving this because you are subscribed to this thread.  https://github.com/notifications/beacon/ACC6C7QHHGANMDH4LADVBWDWC2R47A5CNFSM6AAAAAARDG4HLCWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHFHTZPC4.gif Message ID: @.*** @.***> >

Swapping over to Line 154: ppiID = (long**)malloc( sizeof(long*) * iMAX_CELLS );
did fix the issue.
Looking into it in mingw for whatever reason, a long is 4 bytes and the long pointer is 8, instead of both being 8.


FixedHeaderCount ignored when TabablzControl contains only 1 tab

FixedHeaderCount works as advertised with > 1 tab. When there is only 1 tab, that tab is free to move and join another TabablzControl.

Thanks for the spot.  Fixed.  You can still drag the window around with a single tab, but it wont snap into another control.


TypeError: load() takes exactly 1 argument (2 given)

Hello, I set WORKSPACE and pod.workspace according to the tutorial
After I execute:
bazel run @rules_pods//:update_pods -- --src_root $PWD

PIC:

log:
[source-only][~/Documents/bazel/examples/PodToBUILD-master/aa]$ sudo bazel run @rules_pods//:update_pods -- --src_root $PWD                   rvm:
Password:
Starting local Bazel server and connecting to it...
DEBUG: Rule 'rules_pods' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "b523e1c567dce08f6afcde607fb9f62e1c33074891b1fcc1ee647aa2cc2f7e26"
DEBUG: Repository rules_pods instantiated at:
  /Users/mac/Documents/bazel/examples/PodToBUILD-master/aa/WORKSPACE:3:13: in <toplevel>
Repository rule http_archive defined at:
  /private/var/tmp/_bazel_root/65baa7e1beeb8bf9ba32b64053619473/external/bazel_tools/tools/build_defs/repo/http.bzl:353:31: in <toplevel>
INFO: Analyzed target @rules_pods//:update_pods (42 packages loaded, 272 targets configured).
INFO: Found 1 target...
Target @rules_pods//bin:update_pods up-to-date:
  bazel-bin/external/rules_pods/bin/update_pods
INFO: Elapsed time: 30.721s, Critical Path: 0.05s
INFO: 4 processes: 4 internal.
INFO: Build completed successfully, 4 total actions
INFO: Build completed successfully, 4 total actions
Updating pods in /Users/mac/Documents/bazel/examples/PodToBUILD-master/aa
Building PodToBUILD dependencies...
running: make release
skipping build..

Traceback (most recent call last):
  File "/private/var/tmp/_bazel_root/65baa7e1beeb8bf9ba32b64053619473/execroot/__main__/bazel-out/darwin-fastbuild/bin/external/rules_pods/bin/update_pods", line 528, in <module>
    main()
  File "/private/var/tmp/_bazel_root/65baa7e1beeb8bf9ba32b64053619473/execroot/__main__/bazel-out/darwin-fastbuild/bin/external/rules_pods/bin/update_pods", line 523, in main
    load("Pods.WORKSPACE")
  File "/private/var/tmp/_bazel_root/65baa7e1beeb8bf9ba32b64053619473/execroot/__main__/bazel-out/darwin-fastbuild/bin/external/rules_pods/bin/update_pods", line 492, in load
    exec(workspace_str, d, d)
  File "<string>", line 1, in <module>
TypeError: load() takes exactly 1 argument (2 given)

ERROR: TypeError: load() takes exactly 1 argument (2 given)

It's my fault。。。。


change:.travis.yml

The latest version needs to run the script
deno run ./test.ts --allow-all

hhh I wanted to do something together, and it was a small attempt。i am from QQ 😊

thx. but the deno run --allow-all ./test.ts is correct


[Enhancement] Vendor pull from terraform private registries

Describe the Feature
This feature is an enhancement for Atmos Components to be able to fetch terraform modules from private registries based on https.
Expected Behavior
Ability to fetch terraform modules from private terraform registries
Use Case
Many companies use private registries and repositories to:

Decoupled from vendor servers.
Fasten their pipeline builds.
Avoid breaking routine work when problems appear with vendor servers.
Improve security.
etc...

E.g:

Docker
Apt
Rpm
Npm
Maven
Helm
Terraform
etc...

Describe Ideal Solution
Option 1:
After running terraform login <some terraform private registry>, credentials file being created under ~/.terraform.d/credentials.tfrc.json .
If Atmos can look for this path on component initialization, it might solve the problem for any terraform registry use case as long as it work with native terraform registry authentication.
Reference: ==> Terraform Credentials Storage
Option 2 (might be the preferred):
Atmos will fetch the following environment variable in case of private registries TF_TOKEN_cloudposse_jfrog_io=<terraform-private-registry-token> , terraform translate the uri to cloudposse.jfrog.io .
Reference: ==> Environment Variables Credentials
Alternatives Considered
No response
Additional Context
No response

Hi @Amos-85 I'm using vendor pull right now and I'm trying to understand your usecase.
you are talking about the module instantiation source or the URL on the component.yaml that is read by atmos when you run atmos vendor pull?

Hi @jamengual ,
The use case is actually similar to terragrunt tfr protocol .
Currently I'm trying to run atmos vendor pull -c aws/vpc after setup ComponenetVendorConfig as below :
apiVersion: atmos/v1
kind: ComponentVendorConfig
metadata:
  name: vpc-vendor-config
  description: Source and mixins config for vendoring of 'vpc' component
spec:
  source:
    uri: "https://<company-domain>.jfrog.io/artifactory/<terraform-remote-repository>/terraform-aws-modules/terraform-aws-vpc/{{ .Version }}.zip"
    version: v4.0.1
    included_paths:
    - "**/*.tf"
    - "**/*.tfvars"
    - "**/*.md"

but it's failing to fetch the module with error bad response code: 401 because of missing terraform private registry api token
It's fetching the module as expected if I'll use the uri with username:password as following:
uri<EMAIL_ADDRESS>.Version }}.zip"

but it's not secure way to use credentials in configuration files.

I think the simplest solution would be to just copy the (MIT licensed) Terragrunt tfr package from

https://github.com/gruntwork-io/terragrunt/blob/main/terraform/getter.go
https://github.com/gruntwork-io/terragrunt/blob/main/terraform/doc.go
and adapt the names of the used environment variables.


How to disable specific date range?

Hi there,
I'm trying to disable specific dat-range on calendar with shouldDisableDate, but seems like its not possible? Any suggestions should be really appreciated.

Hi,
It looks like you are using material-ui-pickers, I would ask this on stackoverflow .
Looking at their API shouldDisableDate takes a function with the date passed in and you return true or false if you want the date disabled.
Hope that helps.


fix apply config in STORAGE_CLASS_SHARE_MOUNT mode

fix: https://github.com/juicedata/juicefs-csi-driver/issues/1065

Codecov Report
Attention: Patch coverage is 66.66667% with 4 lines in your changes missing coverage. Please review.

Project coverage is 27.87%. Comparing base (ad57e95) to head (83b744e).
Report is 5 commits behind head on master.

Files with missing lines
Patch %
Lines

pkg/config/setting.go
66.66%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1122      +/-   ##
==========================================
- Coverage   27.96%   27.87%   -0.10%
==========================================
  Files          51       51
  Lines        6969     7021      +52
==========================================
+ Hits         1949     1957       +8
- Misses       4814     4858      +44
  Partials      206      206

Flag
Coverage Δ

27.87% <66.66%> (?)

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

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


Android: Set SYSTEM_UI_FLAG_HIDE_NAVIGATION when going to Fullscreen …

…mode
Thanks a lot for making a contribution to SFML! 🙂
Before you create the pull request, we ask you to check the follow boxes. (For small changes not everything needs to ticked, but the more the better!)

[ ] Has this change been discussed on the forum or in an issue before?
[x] Does the code follow the SFML Code Style Guide?
[x] Have you provided some example/test code for your changes?
[ ] If you have additional steps which need to be performed list them as tasks!

Description
Hides the navigation bar on API >= 14. On Android >= API 19 when immersive sticky mode is enabled the navigation bar can be swiped to be shown by the user.
https://developer.android.com/reference/android/view/View#SYSTEM_UI_FLAG_HIDE_NAVIGATION
This PR is related to the issue #
Tasks

[ ] Tested on Linux
[ ] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[x] Tested on Android

How to test this PR?
Describe how to best test these changes. Please provide a minimal, complete and verifiable example if possible, you can use the follow template as a start:
On line 82 on main.cpp of the Android SFML example, change the window constructor to be:
sf::RenderWindow window(screen, "", sf::Style::Fullscreen);

Superseded by #1554


[MRG] Space

This pull request resolves #76, resolves #109.
The pull request is ready to be merged.
The details of the pull request:
Added two constants in constant.py to limit the size of the architecture.
This will resolve the memory error caused by the too large model.

Pull Request Test Coverage Report for Build 504

10 of 11 (90.91%)  changed or added relevant lines in 3 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.05%) to 96.54%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

autokeras/net_transformer.py
4
5
80.0%

Files with Coverage Reduction
New Missed Lines
%

autokeras/bayesian.py
1
97.55%

autokeras/graph.py
2
97.21%

Totals

Change from base Build 497:
-0.05%

Covered Lines:
1646

Relevant Lines:
1705

💛  - Coveralls


inkscape: use latest poppler, use gtkspell2, drop -fpermissive

It seems that inkscape-0.9.4 builds with the latest poppler and
without the -fpermissive workaround.
Motivation for this change
Cleanup.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @7c6f434c (added -fpermissive and poppler = poppler_0_61); no meta.maintainer.

Do we have a working gtkspell2 backend on darwin, or it's better to disable spellcheck on darwin?

Observation: gtkspell2 depends on enchant which has failed on Darwin for exactly a year (OK, a year without a few hours). It has been marked as unsupported on x86_64-darwin. I think as gtkspell2 is an added feature and we used Inkscape fine without it, adding the feature only on Linux platforms first should be fine (and will unbreak ofBorg eval).

@7c6f434c Done.

@GrahamcOfBorg eval

@7c6f434c Fixed another typo.


[FLINK-22017][coordination] Allow BLOCKING result partition to be individually consumable

What is the purpose of the change
In FLINK-22017, we construct a scenario that regions may never be scheduled when there is cross-region blocking edges in the graph. To solve this issue we should allow BLOCKING result partitions be consumable individually. Note that this will result in the scheduling to become execution-vertex-wise instead of stage-wise, with a nice side effect towards better resource utilization. The PipelinedRegionSchedulingStrategy can be simplified along with change to get rid of the correlatedResultPartitions.
There's three main concerns that need to be considered:
1. Since scheduling become execution-vertex-wise instead of stage-wise, we need to make sure the computation complexity in PipelinedRegionSchedulingStrategy won't fall back to O(n^2). We tested it with benchmark and end-to-end tests. Our pull request doesn't introduce significant performance regression.
2. Before this pull request,finishPartitionsAndUpdateConsumers already has the complexity of O(n^2). We intend to optimize it in FLINK-21915. Since each partition will finish individually, this optimization is not valid any more. As we tested in the job with two vertices (parallelism 8k, all-to-all, batch mode), it takes less than five seconds.
3. SchedulingDownstreamTasksInBatchJobBenchmark is modified in accordance with this change. We need to monitor the result of this benchmark.
Brief change log

We can get the ConsumedPartitionGroup that an IntermediateResultPartition or a DefaultResultPartition belongs to
A blocking result partition will be consumable individually once its producer is finished. It doesn't need to wait until all other IntermediateResultPartitions that belong to the same IntermediateResult finish

Verifying this change
This change added tests and can be verified as follows:

Added unit tests for getting ConsumedPartitionGroup from IntermediateResultPartition and DefaultResultPartition
Added unit tests for scheduling pointwise vertices in the batch job
Extended the unit tests that schedule vertices in the graph illustrated in FLINK-22017
Manually verified the change by running a job with two job vertices, their parallelisms are both 8k. Two distribution patterns (pointwise and all-to-all) and two job type (batch and streaming) are involved. All jobs finish correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure


Layout Grid: not responsive to device screen sizes

current way is responsive to pre-assigned, meant if you have 2 columns, it would be 2 columns on both phone and tablet. What I meant here is if you have 320px and 1920px width of 2 devices, it just always display 2 columns. The responsiveness to the screens should be 12 columns on 1920px screens but remaining 2 columns in 320px screen.

Small screen break points
|||
|||
|||

Big screen break points
|||||||||||||
|||||||||||||
|||||||||||||

It is mobile-only framework, and it supports 2-sizes: phone and tablet. You can do column like:

...

It is explained in documentation http://www.idangero.us/framework7/docs/grid.html


feat: add buffering layer to BulkWriter

Porting from node.

Closing. Based this on the wrong branch.


Refactors docker compose file setup

This structure more naturally fits what docker expects. A docker-compose file at the root, with docker files for the respective services in those modules.
I have tested this locally for the prod build
and pushed things and I believe it works for
the other cases too.
TODO:

dockerx build instructions
automating the build & versioning of images with new releases

Changes

move docker files around

How I tested this

locally

Notes
Checklist

[ ] PR has an informative and human-readable title (this will be pulled into the release notes)
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future TODOs are captured in comments
[ ] Project documentation has been updated if adding/changing functionality.

@flavour thoughts?


"Install via github" instructions --> Error: "fatal: Could not read from remote repository."

$ git clone git@github.com:imsweb/ezmomi.git
Cloning into 'ezmomi'...
Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

Try the https endpoint instead:
git clone https://github.com/imsweb/ezmomi.git

I'll actually change the README to this since its preferred.
You'll need an ssh public key added to your github profile in order to use the ssh endpoint per https://help.github.com/articles/which-remote-url-should-i-use/,

Works for me!  Thanks!

You may also want to change the README to include this:
virtualenv --no-site-packages -p python2.7 ezmomi
Without '-p python2.7' the virtualenv command fails on my Mac running OS X 10.11.6

Good info, thanks,  I'll see if -p python2.7 works on Linux and add that flag in, otherwise I'll just make a side note in the README about it.

Works on Linux as well. :+1:  Added to the README install notes.


Per partition producer, replace #11

#11 was targeting master branch, github does not allow change merge target (at least not now).

merging in, create an issue or pr if anyone have anything to complain.


how to run froned pro

I have encountered some problems. Can you give me some help thanks

can you try installing bun https://bun.sh ?


vdk-trino: collect lineage for select/insert and rename table only

Why:
To make lineage collecting more production ready,
some improvements are needed.
What:
In order to reduce the load on the query engine,
only plans for insert/select queries are calculated.
For rename table queries, the plan doesn't give information.
The query is parsed and table names extracted.
Counting the number of rows in the output table before and after
is removed to reduce the burden on the query engine.
How has this been tested:
Tweaked the test_vdk_trino_lineage.py test
to be more comprehensive and cover all scenarios.
What type of change are you making?
Bug fix (non-breaking change which fixes an issue)
or a cosmetic change/minor improvement
Signed-off-by: Philip Alexiev<EMAIL_ADDRESS>

I am not sure if you noticed - the CI tests failed (ci/gitlab/gitlab.com) - https://gitlab.com/vmware-analytics/versatile-data-kit/-/jobs/2176218096

@tozka Thank you for the review and valuable comments.


Dockerfiles are cloning from original repo

Dockerfiles are cloning from original repo and not this forked one.
https://github.com/Py-KMS-Organization/py-kms/blob/master/docker/docker-py3-kms-minimal/Dockerfile#L28
https://github.com/Py-KMS-Organization/py-kms/blob/master/docker/docker-py3-kms/Dockerfile#L28

Shit. I've missed that! I'll be fixing that ASAP. Sorry.

Thanks!


Sorting by Group Value

Is there any way to combine useSortBy and useGroupBy hooks in order to sort by the currently selected group? Currently, I have the grouped value in a separate column where I use the Aggregate function to pull out the groupByVal and display it to the user, like so:
Aggregated: ({ row: { isExpanded,  groupByVal } }) => {       return (         <span>           {isExpanded ? (             <IconKeyboardArrowUp className="expand-collapse" />           ) : (             <IconKeyboardArrowDown className="expand-collapse" />           )}           {groupByVal}         </span>       );     }
However, I'm not sure how I can get out the groupByVal into a way where it is sortable, since the accessor is dynamic based on the table state.

@dsoffiantini I am looking for a way to do the same. I group a column A and column B sums up the integers within that group. If I now sort column B it will consider all the subRows, too. However, this behaviour is slightly confusing for the user who would rather want to sort by the grouped sum.

This is a question, so converting it to discussions. Feel free to use the discord server too!


[zlib] Build error

Package: zlib:arm64-osx -> 1.3
Host Environment

Host: arm64-osx
Compiler: AppleClang <IP_ADDRESS>30022
vcpkg-tool version: 2023-10-18-27de5b69dac4b6fe8259d283cd4011e6d20a84ce
vcpkg-scripts version: 8eb57355a 2023-10-19 (5 days ago)

To Reproduce
vcpkg install
Failure logs
-- Using cached madler-zlib-v1.3.tar.gz.
-- Cleaning sources at /Users/takumihorikawa/Source/Inside/vera/bin/Debug/_deps/vcpkg-src/buildtrees/zlib/src/v1.3-8825ee792f.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /Users/takumihorikawa/Source/Inside/vera/bin/Debug/_deps/vcpkg-src/downloads/madler-zlib-v1.3.tar.gz
-- Applying patch 0001-Prevent-invalid-inclusions-when-HAVE_-is-set-to-0.patch
-- Applying patch 0002-skip-building-examples.patch
-- Applying patch 0003-build-static-or-shared-not-both.patch
-- Applying patch 0004-android-and-mingw-fixes.patch
-- Using source at /Users/takumihorikawa/Source/Inside/vera/bin/Debug/_deps/vcpkg-src/buildtrees/zlib/src/v1.3-8825ee792f.clean
-- Found external ninja('1.10.2').
-- Configuring arm64-osx
-- Building arm64-osx-dbg
-- Building arm64-osx-rel
-- Installing: /Users/takumihorikawa/Source/Inside/vera/bin/Debug/_deps/vcpkg-src/packages/zlib_arm64-osx/share/zlib/vcpkg-cmake-wrapper.cmake
-- Fixing pkgconfig file: /Users/takumihorikawa/Source/Inside/vera/bin/Debug/_deps/vcpkg-src/packages/zlib_arm64-osx/lib/pkgconfig/zlib.pc
CMake Error at scripts/cmake/vcpkg_find_acquire_program.cmake:163 (message):
  Could not find pkg-config.  Please install it via your package manager:

      brew install pkg-config
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:203 (vcpkg_find_acquire_program)
  ports/zlib/portfile.cmake:43 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:168 (include)

Additional context
vcpkg.json
{
  "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg-tool/main/docs/vcpkg.schema.json",
  "dependencies": [
    "libzip"
  ]
}

I have installed pkg-config using brew, but I'm getting this error for some reason. Can anyone identify the cause?

CMake Error at scripts/cmake/vcpkg_find_acquire_program.cmake:163 (message):
  Could not find pkg-config.  Please install it via your package manager:

      brew install pkg-config

I've already executed that command, but I'm still encountering the error.

@T-Horikawa1992, you can try to clone a vcpkg again and run the command brew install pkg-config first and then try again.

@JonLiu1993
Recloning didn't help. Is it an issue that pkg-config is installed in /opt/homebrew instead of /usr/local? The path to pkg-config is correctly set up.

My pkg-config is in the ./usr/local path after installation. You can install it to the ./usr/local path and try it.
./usr/local/bin/pkg-config
./usr/local/Homebrew/Library/Homebrew/shims/mac/super/pkg-config
./usr/local/var/homebrew/linked/pkg-config
./usr/local/opt/pkg-config
./usr/local/Cellar/pkg-config
./usr/local/Cellar/pkg-config/0.29.2_3/bin/pkg-config
./usr/local/Cellar/pkg-config/0.29.2_3/share/doc/pkg-config
./usr/local/share/doc/pkg-config

IIRC it is one location is used for x64, the other for arm64.

@JonLiu1993 @dg0yt
The default installation location for Apple Silicon version of Homebrew is /opt/homebrew. Do you know how to change it to /usr/local? I've been researching it, but haven't succeeded yet. Also, is it possible that because pkg-config is installed in /opt/homebrew, vcpkg can't find it?

Don't move it!
The immediate workaround is to set environment variable PKG_CONFIG.

Can you please post the full path of the arm64 pkg-config?

@dg0yt
oh,sry
I was able to install Homebrew in /usr/local by running Terminal in x86_64 mode.
After that, running brew install pkg-config resolved the vcpkg error.
It seems like it's working under the assumption that it should be in /usr/local.
Is there any potential issue with this solution?

Again, I want to verify the full path of pkg-config so that I can propose a permanent fix.

@dg0yt
This is the full path.
/opt/homebrew/bin/pkg-config

I'm experiencing exactly the same problem the OP has, but, if possible, I would like to not use HomeBrew just only for a single package.
Is it possible to fix it with vcpkg install pkgconf in MacOS as suggested here https://github.com/microsoft/vcpkg/discussions/30365?
I've installed the pkgconf port and it continues failing, but it works if I use the tool installed by the pkgconf port like this:
PKG_CONFIG=$PWD/installed/arm64-osx/tools/pkgconf/pkgconf ./vcpkg install zlib

Shall the pkgconf tool be automatically detected?

Hi @dg0yt
Do you know if what I wrote above about using pkgconf binary is expected to be working?
Thank you! :)

Using pkgconf from vcpkg via the PKG_CONFIG environment variable works for non-windows, and in some ways also for windows.
However, you need to trigger the build explicitly, before it is needed by a port. This may be okay for a single use, or in custom CI scripts, but it is clumsy as a general solution.  (And hopefully pkgconf
never gets a circular dependency on pkgconf.) You need brew as soon as you build the first port based on autotools.