Idle Calibur does not start, shows light flickering (Arch linux)

When trying to start the game, it hangs on the starting screen, light flickering and repeating every 3 seconds. On Windows, this "light" is part of the start menu. Game never starts, and never crashes either.
Software information
Idle Calibur, defaults settings since i can't even start the game
System information

GPU: GTX 1650 GDDR6 (laptop)
Driver: 510.47.03
Wine version: 7.1-GE-2 (Proton)
DXVK version: v1.9.4-async

Apitrace file(s)

Put a link here

Log files

d3d9.log: empty
d3d11.log: Game_d3d11.log
dxgi.log: Game_dxgi.log
proton log: steam-1674890.log

Put a link here

Please do that.

Please do not report bugs here when using an unofficial fork.

fwiw I just installed the game to test and found the following:

the exact same thing happens with wined3d
the main menu is not being rendered with D3D11 at all (or really any D3D from what I can tell)

The game just seems to be stuck in an infinite initialization loop for some reason, doesn't look like our bug.

Sorry, i overstimated my issue-reporting capabilities. You can close this issue if you want, since it seems to be a Proton issue. Thanks for the help anyway!


Create top level error type

The subcommands have ad-hoc errors which aren't printed in color. This should be fixed, either by creating a new error enum, or a struct StringError{message: String} catch-all error.
I'm going to create a new top-level error, Error, retire the Error trait, and create a sub-error for SubcommandErrors.

[x] Rename CompilationError to CompileError
[x] Subject all error printing to verbosity filtering
[x] main run function takes loader as argument so it can return errors that borrow from loaded strings
[x] get rid of eprint
[x] have top-level error type
[x] move subcommand invocation into subcommand
[x] merge all errors but compilation error?
[x] Test Error::{Load, Io} error messages
[x] create ColorDisplay trait which includes color information
[x] remove Color::fmt
[x] Un-bold usage string in ArgumentCountMismatch
[x] Re-color parameters in ArgumentCountMismatch usage string
[x] Harmonize RunError and Error names

Done! Errors are now much better in a bunch of ways.


Improve how variables are passed through to avoid quoting issues

What problem does your feature request solve?

Currently a lot of issues arise from various shells interpreting the variables to be put into the templates. This is confusing, and hard to get right, so should be improved.
Describe the solution you'd like

Ideally nothing between rose edit and CSET should be parsing these values, so we should probably encode them somehow and then send them across directly. Maybe JSON encoding, or something else?
Describe alternatives you've considered

Much pain.

While I hoped to fix this upstream in Cylc, for various reasons that solution is not viable. A good suggestion did come up however: https://github.com/cylc/cylc-flow/issues/6202#issuecomment-2222304635


Prediction script

Can you provide a prediction script for iterating and predicting through 3D volumes?

The issue is referenced here

Passing --inference with the model path worked last time I tried it. See inference() in train.py.


Can't disable watermark

Describe the bug
I can't disable the Zitadel watermark (and social icons in footer)
To Reproduce
Steps to reproduce the behavior:

Go to Settings > Branding
Expand section 'Advanced Behavior"
Section is empty

Expected behavior
I can disable the watermark
Additional context
#4498 contained a description of that bug, but was closed.
Version
v2.9.1 selfhosted with postgres as database

@peintnermax what is the progress of this issue?

Could not reproduce this issue tn the cloud. @adlerhurst Can you take a look if the labelpolicy is correctly returned with a postgres db as this could prevent the toggle to be shown?

I can reproduce this issue locally by starting the zitadel with docker compose.
I have same behaviour on organization and instance settings:

@pr0gr8mm3r @goran-hc
I think we just found out what the problem is, since we could not reproduce it.
Do you have an add-blocker installed?
They sometimes block some inline things what results in not showing everything in the console.

Interesting.. we'll definately fix this classname and try to avoid adv prefixes in future.. Thx for the hint 👍😎

For future people visiting: You need to change this per organization, only changing the instance settings isn't enough. You can switch by clicking the settings button in the top right.


I'm banging my head over this for far too long and figured I would ask for help.
So, I'm trying to add a datepicker-dropdown (or datepicker) to my app (using latest re_com), but that fails with an error thrown in unparse:
ERROR unparse  TypeError: Cannot read property 'call' of null

I tried to add a datepicker like this:
[datepicker
  :model invoice-date
  :on-change #("New Date " (pr-str %))]

invoice-date is (r/atom (minus (now) (days 3)))
The problems seems to happen when a formatter is search in formatter-fn (in cljs-time):
((or formatter-overrides formatters) %) this seems to return nil. The value passed into that anonymous fn seems to be:
["MMMM" "MMMM" nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil]

I can't really figure out what the problem is, I'm sure its some tiny little detail somewhere.
Does it make a difference that my browser's locale is german?
Thanks,
Torsten.
PS: The documentation says :model is optional however if I omit it I run into a deref error.

Does the demo app work okay for you?  If so, then it won't be related to the locale.
I'd start by nudging the demo around so it mimics what you have in your code.

@mike-thompson-day8 The demo app works beautifully for me- no errors in the console and the datepicker shows up nicely.
Thanks for looking at it!

I copied all my project's dependencies into re-com's project.clj and now the demo exhibits the same problem. I'll let you know when I figure out the exact dependency causing this.

@mike-thompson-day8 Got it!
I was using [org.clojure/clojurescript "1.7.145"] in my project, that caused the problem,
As soon as I change the version you use to this one (keeping all my other dependencies) that problem appears in your app.
I did update cljs-time to "0.3.13" and the problem disappeared. The deps on your project are now:
:dependencies
                [[org.clojure/clojure "1.7.0"]
                 [reagent "0.5.1"]
                 [org.clojure/clojurescript "1.7.145"]
                 [com.andrewmcveigh/cljs-time "0.3.13"]]

Would it be ok for you to update these dependencies?

In summary:  if you use [org.clojure/clojurescript "1.7.145"]  then use [com.andrewmcveigh/cljs-time "0.3.13"]
If you provide those deps in your app's project.clj, they will override the ones in re-com. So you don't need to wait for a new re-com release.
Having said that, we'll be doing a new release shortly (within a week or so) and we will include these new dependencies for the demo app.

Thank you, I got it working!
I had this setup before and it still failed, even after a lein clean.
Then I moved the cljs-time dependency below re-com and it started working. I moved it back at the previous position and it continued to work, even after clean.
Thanks for your help,
Torsten.


[Task]: Update BigtableIO to use veneer client under the hood

What needs to happen?
Migrate BigtableIO to use new idiomatic java bigtable client under the hood. Mark the use of BigtableOptions as depreacted .
Issue Priority
Priority: 2 (default / most normal work should be filed as P2)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

done by #24015


Create auto-generated reference docs for the Pipelines API

/assign @vicaire

We now have an API reference generated from the code. It's not continuously built, but it's auto-generated in that the doc emerges from the code comments. I'm going to close this issue.
/close


Image upload: Show preview before upload

I can't upload images along with messages, when I select the image, it just gets uploaded without even showing preview. It would be good to show image preview, option to edit the image, upload along with message.

I don't think this should be our immediate focus. am I right @borisyankov ?

I think this is something that's fairly valuable; in part because it's part of the flow for submitting bug reports.

It can be valuable, but I am not convinced we have come up with a good solution yet.

I think this is a very valuable feature. I don't have any hints when sending images. I will be very confused and think that I have not sent it. Moreover, users have developed a habit, all im apps have a preview.So, can you prioritize this feature?


Fixed bug and Integrate library with dynamic and static framework for compatible device for Swift and Objct

Fixed Target has transitive dependencies when pod lib lint when depend on GoogleMaps  #119
Fixed and created the dynamic library to use in Swift  #86
also, provide use pod 'Google-Maps-iOS-Utils' with submodules
Fixed Cocoapods issue with use_frameworks #23

update Repo to last changed

Provide two new Sample to use Static and Dynamic Library into project directly with user any pods

update pod version to 2.2.0

Hey @dion-loetscher, is this change the best path forward, post v2.6?
The context for this is prior to v2.6 we had a known issue with Swift that was predicated on dropping support for iOS 7.

@domesticmouse @dion-loetscher Can you decide if this approach is acceptable and either reject or merge?  I'm assuming a reason for rejecting would be that you want to just support dynamic frameworks going forward?

Thanks for the hard work in creating this library. iOS 7 is now <0.1% of users and there is a pull request to fix this issue. Any chance of an update?

Any chance of getting these conflicts resolved and the PR pulled? Farshad's fixes for the original issue are critical for us. We are now using his fork of the library. Thanks Farshad!

I'm happy to pull this if it's conflicts get resolved.
Alternatively, if the community wants to nominate an owner I can mark this as deprecated and point people at a community owned repo. I don't have the time or skill to keep this project up to date.

I'm happy to pull this if it's conflicts get resolved.
Alternatively, if the community wants to nominate an owner I can mark this as deprecated and point people at a community owned repo. I don't have the time or skill to keep this project up to date.

Hi Guys What shoud I do to resolved this confliction files

Guys, At Last, I resolved conflicts. Therefore you can merge.
@domesticmouse @google-admin @googlebot

Is there a version of the Google-Maps-iOS-Utils pod that we can use now in the podfile with use_framework? Thanks!

@farshadmb Is there any reason why the conflicts were not resolved and the merged?

@santinucera I have not enough time to update my fork. I will work on it as soon as possible in the next two weeks comes.

Hi @farshadmb, do you have any news of this PR?

@santinucera Actually, I work on it.

@santinucera Here you go.

@santinucera, I have done what you want.

You are my hero

Excellent work! 👏I'm also waiting for this. Now can it be merged? @domesticmouse

@domesticmouse or @jpoehnelt, can you merge this PR please?

@farshadmb Can you split this into multiple prs?

@jpoehnelt, Would you provide more detail, please? What should I do?

Please merge this PR. This is causing issues for a lot of people. I am facing the same issue.

Hi @farshadmb, thanks for contributing. Give me a few days to review/test this since it's a fairly large change. Please provide any testing instructions, if any.

@arriolac, I can explain every question you have and prepare my self for them.


Double-click to load a file?

Is it the case that if you're trying to load a previously saved file, you're supposed to double click or tap?
Just poking around at the UI this seems to be the case, but I can't tell if that's me being incompetent and missing the hit region, or the UI being designed this way.

Egad, i see what's going on now.
If you click on the file name, it always tries to load that file. However, the actual file contents don't /update/ until you click a second time. So, for example:

write something in the buffer
save into a new file using save as..., e.g. as "test1"
make some changes in the buffer
go back to the save as... menu
click on "test1"
ui returns to what looks like your modified code but...
if you click a second time in the buffer, the UI updates to reveal what's happened is your changes are lost, and the old file was actually loaded

This is really counter-intuitive.
Apparently to save something into an existing file, you have to type the file name in again, then it saves it into that file. Just clicking on the file name only loads the file, but jankily so.
To fix this, suggest to change the UI so that:
The file list is in one region of the screen, and above it is the text "Click name to load, [x] to delete"
Then use an  tag to separate into a separate UI area that says "Type the file name to save:"
This helps separate the loading from the saving feature. Furthermore, we need to add an explicit "load" button, which I will open a new issue for shortly.


Updated default values for poolOptions

Coverage remained the same at 99.988% when pulling 1794bc80c4a0a4c80897066dbd8de76ca6734afd on vkedia-patch-2 into ac8967c92035f469c745995af61f261e9c37e427 on master.

Coverage remained the same at 99.988% when pulling 8397f3a90a78e39cb976fe216b52402bbf8b9f98 on vkedia-patch-2 into ac8967c92035f469c745995af61f261e9c37e427 on master.


Two useful patterns: Controllers and Responders - Thoughts?

Hey @zenazn, just wondering your opinion on adding a controller-like handler in goji? I wrote my own that works with goji as a separate package, which I'd need to refine further to make it feel just right, but I find this is very useful once services start growing in size.
The fundamental difference between host most services are written in Go with http.Handler versus a controller is how a request passes through the mux. I believe both should be supported.
If using a controller to handle requests/responses, the controller still adheres to http.Handler, but it will actually instantiate a new controller object with the writer / request as instance variables of the controller and calls to the functions are the "actions" based on the routes specified on the controller itself. Think like in Ruby with rack + sinatra.
 Someone just walked into my office so I have to tend to that now, .. btw, this is similar to how gocraft and gin work.. beego tries to do this, but feels terribly awkward.
There are some pains to do this with Go in a clean way (without reflection), but its totally doable.

I still look forward to seeing your ideas in action, but I'm going to close this issue since there's no longer anything to be done on the Goji side.


Using 3.1.1 and UWP

Describe the bug
In FluentAssertions we have a test project UWP.Specs exercising running FluentAssertions with UWP with <UseDotNetNativeToolchain>true</UseDotNetNativeToolchain> enabled.
With 3.0.4 I can I can run the test Determining_caller_identity_should_not_throw_for_native_programs from the Visual Studio Test Explorer.
With 3.1.1 I get build time and runtime diagnostics:
Build time warning
1>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.12-rel-31116-00\tools\Microsoft.NetNative.targets(809,5): warning : ILTransform : warning ILT0003: Method 'MSTestSettings.ValidateSettings(IMessageLogger)' will always throw an exception due to the missing method 'MSTestSettingsProvider.get_Settings()' in assembly 'Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices'. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.

Runtime error
[Error] An exception occurred while test discoverer 'MSTestDiscoverer' was loading tests. Exception: Method 'MSTestSettingsProvider.get_Settings()' from assembly 'Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices' was not included in compilation, but was referenced in MSTestSettings.ValidateSettings(IMessageLogger). There may have been a missing assembly, or a dependency on a more recent Windows SDK release.

Steps To Reproduce

Checkout https://github.com/fluentassertions/fluentassertions
Change to release mode
Right click on UWP.Specs and select build (it's excluded from the configuration manager)
See that Determining_caller_identity_should_not_throw_for_native_programs succeeds
Edit Tests\UWP.Specs\UWP.Specs.csproj to use 3.1.1
Build UWP.Specs again
See that Determining_caller_identity_should_not_throw_for_native_programs no longer runs.

Expected behavior
Determining_caller_identity_should_not_throw_for_native_programs should be executable
Actual behavior
Cannot execute Determining_caller_identity_should_not_throw_for_native_programs
Additional context
I tried to dig into this and it looks like this addition to MSTest.TestAdapter from #1669
https://github.com/microsoft/testfx/blob/1223683d9b4ee9e2dd784f8315b45a50698281a0/src/Adapter/MSTest.TestAdapter/MSTestSettings.cs#L466-L469
tries to reference code from MSTestAdapter.PlatformServices which is excluded for the UWP target.
https://github.com/microsoft/testfx/blob/1223683d9b4ee9e2dd784f8315b45a50698281a0/src/Adapter/MSTestAdapter.PlatformServices/Services/SettingsProvider.cs#L17-L43

Please @engyebrahim have a look at this issue.

Same issue here

Hi @jnyrup! Sorry for the delay, I will be investigating the issue.


Analyzer throws unexpected compile error when checks least and greatest closures for invariant generic typedef

Dart SDK version: 2.12.0-29.0.dev (dev) (Mon Nov 9 06:42:00 2020 -0800) on "windows_x64"
This issue is similar with #44163, but it's against the analyzer whereas #44163 is about dart.
Here is a source code example:
X captureTypeArgument<X>() => throw "Error";

typedef check<X> = X Function(X);

void main() {
  void f<X>(check<X> Function() g) => g();
  f(() => captureTypeArgument());
}

Seems like analyzer should pass and dart should throws a runtime error here. However, this is not so and compile error appears instead.
Analyzer sample output is:

Analyzing test.dart...
error - Couldn't infer type parameter 'X'.
Tried to infer 'Object?' for 'X' which doesn't work:
Parameter 'g' declared as     'X Function(X) Function()'
but argument is 'Object? Function(Never) Function()'.
Consider passing explicit type argument(s) to the generic.

test.dart:7:3 - could_not_infer
error - The argument type 'Object? Function(Never) Function()' can't be assigned to the parameter type 'Object? Function(Object?) Function()'. - test.dart:7:5 - argument_type_not_assignable
2 errors found.

Seems like this issue should be closed according to the #44163 evaluation.
@eernstg, could you please confirm?

Yes, we get the greatest closure Object? Function(Never) here again, and there is no type argument to f which could make f(() => captureTypeArgument()) type correct. So the analyzer behavior is correct and we should close this issue.
Thanks for spotting this!


A proto with import couldn't be properly processed

With https://github.com/bojand/grpc-inspect/pull/7 grpc-inspect starts to inspect packageName. However, if there exists import "common.proto"; with another namespace, the logic would treat namespace as message then none of the services would be recognized.
Sample proto:
// common.proto
syntax = "proto3";
package common;
message Animal {
  string name = 1;
}

// human.proto
syntax = "proto3";
package human;
message Person {
  string last_name = 1;
}
message MakeHumanRes {
  string last_name = 1;
}
service HumanService {
  rpc MakeHuman() returns (MakeHumanRes);
}

This broke mali as mali just say ~0.3.0 for its dependency.

Fixed in #9.


Cue packages

I saw we can use "strings", "math" packages in cue files.
My question can we use the "time" package as well? If yes, what functions we can use?
What other packages are supported by the cuelang?
Can we please get documentation on it?

The standard library is documented on GoDoc: https://pkg.go.dev/cuelang.org/go/pkg.

The standard library is documented on GoDoc: https://pkg.go.dev/cuelang.org/go/pkg.

Thanks @slewiskelly . Closing this issue.

<EMAIL_ADDRESS>Standard packages in cuelang


encoding/xml: decoding a>b>c decoration syntax should support namespaces

What version of Go are you using (go version)?

$ go version
go version 1.13.5 windows/amd64

Does this issue reproduce with the latest release?
Yes, behaves same on 1.13.7
What operating system and processor architecture are you using (go env)?
Windows/amd64 but this is repeatable at least on linux
go env Output
$ go env

What did you do?
I set up a struct with
I set up this set of structures
type myGetValueResponseEnvelope struct {
	XMLName xml.Name `xml:"http://schemas.xmlsoap.org/soap/envelope/ Envelope"`
	Body    myGetValueResponseBody
}
type myGetValueResponseBody struct {
	XMLName xml.Name `xml:"http://schemas.xmlsoap.org/soap/envelope/ Body"`
	Value   string   `xml:"getValueResponse>getValueReturn"`
}

to decode this XML:
<?xml version="1.0" encoding="utf-8"?>
<soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
   <soapenv:Body>
      <getValueResponse soapenv:encodingStyle="http://schemas.xmlsoap.org/soap/encoding/">
         <getValueReturn xsi:type="xsd:string">69.1</getValueReturn>
      </getValueResponse>
   </soapenv:Body>
</soapenv:Envelope>

but I wanted to eliminate the second object by using a>b>c syntax.   Note though that the soapenv namespace is as far as Body but not farther. So I attempted this
type myGetValueResponseEnvelope struct {
	XMLName xml.Name `xml:"http://schemas.xmlsoap.org/soap/envelope/ Envelope"`
	Value string `xml:"http//schemas.xmlsoap.org/soap/envelope Envelope>getValueResponse>getValueReturn"`
}

but it doesn't SEEM with a>b>c syntax that there's any way to have namespaces in all or (in my case) part of the xpath.
I can just eliminate the namespace requirement on Body and it actually decodes just fine, so this isn't an urgent issue at the moment, but some day I may want to be more selective.  Regardless it seems the answer would be to split on > then ' ' then process it as usual.

Thank you for filing this issue @wz2b and welcome to the Go project!
I am not an XML expert but I shall kindly ping @mvdan @rogpeppe @nigeltao.


Height parameter enforces new line switch

[X] I have read through the manual page (man fzf)
[X] I have the latest version of fzf
[X] I have searched through the existing issues

Info

OS

[X] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[X] zsh
[ ] fish

Problem / Steps to reproduce
Hi there,
First of all, thank you for this tool. Amazingly helpful. Just wanted to report a small inconsistency.
Consider this hacky function on my .zshrc:
function _fuzzy_search_current_word() {
    CURRENTWORD="${LBUFFER/* /}${RBUFFER/ */}"
    # If there is no word entered, do nothing
    [[ -z $CURRENTWORD ]] && return
    # Pass current subtree to fzf
    FZFRESULT=`rg --files --no-ignore-vcs --hidden | fzf --reverse -q $CURRENTWORD`
    # Replace the word with selected result
    BUFFER=`echo "$BUFFER" | sed "s#${CURRENTWORD}#${FZFRESULT}#"`
    zle end-of-line
}
bindkey '^[^M' _fuzzy_search_current_word

Basically what it does is, when I type vim somefuzzystring and trigger this function with Alt-Enter, fzf opens and once I select the result, it replaces "somefuzzystring" with my selection (on the current line).
But when --height parameter is given to fzf, it prints the word on the next line for some reason. None of the options I've tried changed this behavior. Is this expected?

I can't explain why, but adding zle reset-prompt will help in that case.
https://github.com/junegunn/fzf/blob/8b0e1f941a419258b0877d9a5f71373d76472043/shell/key-bindings.zsh#L65


fix(Accordion): An accordion can have multiple levels of nested content. showing error at time of run

Steps to Reproduce

Copy pasted the example of nested accordion in create-react-app app.
Does show an error: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in.

Expected
The Accordion should run without error
Result
React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in.
Testcase
https://react.semantic-ui.com/modules/accordion#accordion-example-nested

small changes in example to make it work
`import React from 'react'
import { Accordion } from 'semantic-ui-react'
const level1Panels = [
{ title: 'Level 1A', content: 'Level 1A Contents' },
{ title: 'Level 1B', content: 'Level 1B Contents' },
]
const Level1Content = (

Welcome to level 1

)
const level2Panels = [
{ title: 'Level 2A', content: 'Level 2A Contents' },
{ title: 'Level 2B', content: 'Level 2B Contents' },
]
const Level2Content = (

Welcome to level 2

)
const rootPanels = [
{ title: 'Level 1', content: Level1Content, key: "1" },
{ title: 'Level 2', content: Level2Content, key: "2" }
]
const AccordionExampleNested = () => (

)
export default AccordionExampleNested`

Changes in the example to run it successfully:-
Need to use Accordion instead of Accordion.Accordion.
And content should only contain a reactNode not an object:
{ title: 'Level 1', content: Level1Content, key: "1" }
instead of
{ title: 'Level 1', content: { content: Level1Content, key: 'content-1' } }

Please check your environment, this example works perfectly on codesandbox. If you still expiriencing this issue, please make a repro on codesandbox.


validator.reset doesn't work for values.

Here is a working example:
https://codesandbox.io/s/y2mv6vpqo1
this.$validator.reset() works for errors, but not for values.

This is intended. VeeValidate does not reset values, you have to do that yourself. Validator.reset only resets the validation state.


Time for a new release?

It's been a while ... 41 changes sine 0.2.1 ... or should packages release against the current state of 'origin/master'

I was hoping to get #133 merged before a 0.3.0 release.

Sounds good!

Hi, @ammen99! Now that it's been merged do you think wf-recorder is ready for another release? I ask because ffmpeg 5.0.0 is getting prepared in Arch Linux and while the latest commit builds fine with it, 0.2.1 does not.

https://github.com/ammen99/wf-recorder/releases/tag/v0.3.0


Use travis to deploy to npm/github releases.

Fixes #492

Wow, that's impressive, npm and GitHub releases with just a few lines of config.


add energy market exchange

from #256

add energy market exchange (oeo-dev-meeting 14)
make fuel market exchange subclass

Sounds reasonable.
Yet, the question is open of what to put into oeo-shared to interlink the modules. Can we do this without moving everything involved to oeo-shared? 😨

If I don't miss anything, we only have to move commodity and its subclasses to oeo-shared. Then we could to the commodity-traded at-market relations in oeo-social and relate fuel to commodity in oeo-physical.

I implemented an equivalent class that connects fuels to the commodity role. It's not at all intuitive, though. Would it be better with an anonymous class? I don't know how that's implementes, actually... @sfluegel05

If you want to use anonymous classes, you just have to insert has role some fuel commodity role whenever you would use fuel commodity. For example fuel market exchange would get the axiom trades some (has role some fuel commodity role).
This way you don't need a class fuel commodity, but the meaning is the same (because we defined fuel commodity as equivalent to has role some fuel commodity role)

I now added the market exchange side, too.

What is still missing is market exchange trades some commodity

What is still missing is market exchange trades some commodity

Not necessarily. The def of 'market exchange role' describes  financial instruments, commodities, or other products, services, or goods as its range.
But the inverse relation 'commodity is traded at market exchange' should be added.


pg:backups info command causes errors

When I run heroku pg:backups info b001 -a my-app or heroku pg:backups info -a my-app I get the following error:
Error submitting error.

 !    Heroku client internal error.
 !    Search for help at: https://help.heroku.com
 !    Or report a bug at: https://github.com/heroku/heroku/issues/new

    Error:       undefined method `-' for nil:NilClass (NoMethodError)
    Command:     heroku pg:backups info b001 -a panorama-rainbow
    Plugins:     heroku-pg-extras
                 heroku-push

    Version:     heroku-toolbelt/3.25.0 (x86_64-darwin10.8.0) ruby/1.9.3

    More information in /Users/jacobevelyn/.heroku/error.log

My local .heroku/error.log file looks like this:
exception class/object expected
/Users/jacobevelyn/.heroku/client/lib/heroku/rollbar.rb:9:in `raise'
/Users/jacobevelyn/.heroku/client/lib/heroku/rollbar.rb:9:in `error'
/Users/jacobevelyn/.heroku/client/lib/heroku/helpers.rb:441:in `styled_error'
/Users/jacobevelyn/.heroku/client/lib/heroku/cli.rb:52:in `rescue in start'
/Users/jacobevelyn/.heroku/client/lib/heroku/cli.rb:35:in `start'
/usr/bin/heroku:24:in `<main>'
Heroku client internal error.
undefined method `-' for nil:NilClass
/Users/jacobevelyn/.heroku/client/lib/heroku/command/pg_backups.rb:227:in `backup_status'
/Users/jacobevelyn/.heroku/client/lib/heroku/command/pg_backups.rb:53:in `backups'
/Users/jacobevelyn/.heroku/client/lib/heroku/command.rb:223:in `run'
/Users/jacobevelyn/.heroku/client/lib/heroku/cli.rb:40:in `start'
/usr/bin/heroku:24:in `<main>'
exception class/object expected
/Users/jacobevelyn/.heroku/client/lib/heroku/rollbar.rb:9:in `raise'
/Users/jacobevelyn/.heroku/client/lib/heroku/rollbar.rb:9:in `error'
/Users/jacobevelyn/.heroku/client/lib/heroku/helpers.rb:441:in `styled_error'
/Users/jacobevelyn/.heroku/client/lib/heroku/cli.rb:52:in `rescue in start'
/Users/jacobevelyn/.heroku/client/lib/heroku/cli.rb:35:in `start'
/usr/bin/heroku:24:in `<main>'
Heroku client internal error.
undefined method `-' for nil:NilClass
/Users/jacobevelyn/.heroku/client/lib/heroku/command/pg_backups.rb:227:in `backup_status'
/Users/jacobevelyn/.heroku/client/lib/heroku/command/pg_backups.rb:53:in `backups'
/Users/jacobevelyn/.heroku/client/lib/heroku/command.rb:223:in `run'
/Users/jacobevelyn/.heroku/client/lib/heroku/cli.rb:40:in `start'
/usr/bin/heroku:24:in `<main>'
...

Let me know if there's anything I can do to help fix this.

@uhoh-itsmaciek any ideas on this one? I put a fix in for the rollbar error (rollbar is rate limited, that's not the real error, but I fixed it in #1408)
the real issue is this:
undefined method `-' for nil:NilClass
/Users/jacobevelyn/.heroku/client/lib/heroku/command/pg_backups.rb:227:in `backup_status'
/Users/jacobevelyn/.heroku/client/lib/heroku/command/pg_backups.rb:53:in `backups'
/Users/jacobevelyn/.heroku/client/lib/heroku/command.rb:223:in `run'
/Users/jacobevelyn/.heroku/client/lib/heroku/cli.rb:40:in `start'
/usr/bin/heroku:24:in `<main>'

Hmm, I haven't seen this. Looking into it.

@JacobEvelyn could you try running heroku plugins:update to see if it fixes the problem? If that doesn't work, maybe try uninstalling that heroku-pg-extras and retrying to see if there is a conflict there?

Ok, sorry about the delay. I got sick and it took me a while to catch up on everything. I think I made some incorrect assumptions in the client code w.r.t. the data available about each backup: I had added some additional info, but older backups did not have it, and this step failed. Rather than requiring a client update, I just filled in the missing info for older backups, so this should be working correctly now. Let me know if you still have problems @JacobEvelyn.

Sorry for the delay. Works great now, thanks!


Sitetransform.css

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description
A subdomain for a minimalist framework
Link to Website
https://sitetransform.github.io

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request


v4 does not work on windows if home folder is on a network drive

likely need to fix this by using a different directory

update: this pr should solve this: https://github.com/heroku/heroku-cli/pull/76
needs to be tested though


Internal Error

when i do /mv it says internal error and in console this is the bug report
java.lang.NullPointerException
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at com.onarandombox.MultiverseCore.MultiverseCore.onCommand(MultiverseCore.java:897)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:45)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:149)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at org.bukkit.craftbukkit.v1_14_R1.CraftServer.dispatchCommand(CraftServer.java:710)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PlayerConnection.handleCommand(PlayerConnection.java:1641)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PlayerConnection.a(PlayerConnection.java:1481)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PacketPlayInChat.a(PacketPlayInChat.java:47)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PacketPlayInChat.a(PacketPlayInChat.java:1)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PlayerConnectionUtils.lambda$0(PlayerConnectionUtils.java:19)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.TickTask.run(SourceFile:18)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.executeTask(SourceFile:144)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.IAsyncTaskHandlerReentrant.executeTask(SourceFile:23)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.executeNext(SourceFile:118)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.MinecraftServer.aX(MinecraftServer.java:909)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.MinecraftServer.executeNext(MinecraftServer.java:902)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.awaitTasks(SourceFile:127)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.MinecraftServer.sleepForTick(MinecraftServer.java:886)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.MinecraftServer.run(MinecraftServer.java:819)
22.08 08:49:27 [Server] INFO [08:49:27] [Server thread/WARN]: 	at java.lang.Thread.run(Thread.java:748)

Version 1.14

This is not the full error. I need to see the rest, especially the part after "Caused By".

22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: java.lang.NullPointerException
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at com.onarandombox.MultiverseCore.MultiverseCore.onCommand(MultiverseCore.java:897)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:45)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:149)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at org.bukkit.craftbukkit.v1_14_R1.CraftServer.dispatchCommand(CraftServer.java:710)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PlayerConnection.handleCommand(PlayerConnection.java:1641)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PlayerConnection.a(PlayerConnection.java:1481)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PacketPlayInChat.a(PacketPlayInChat.java:47)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PacketPlayInChat.a(PacketPlayInChat.java:1)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.PlayerConnectionUtils.lambda$0(PlayerConnectionUtils.java:19)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.TickTask.run(SourceFile:18)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.executeTask(SourceFile:144)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.IAsyncTaskHandlerReentrant.executeTask(SourceFile:23)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.executeNext(SourceFile:118)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.MinecraftServer.aX(MinecraftServer.java:909)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.MinecraftServer.executeNext(MinecraftServer.java:902)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.awaitTasks(SourceFile:127)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.MinecraftServer.sleepForTick(MinecraftServer.java:886)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at net.minecraft.server.v1_14_R1.MinecraftServer.run(MinecraftServer.java:819)
22.08 09:41:56 [Server] INFO [09:41:56] [Server thread/WARN]: 	at java.lang.Thread.run(Thread.java:748)

is this it ?


update illustration for detectors

We recently added yolo and yunet, add their outputs in the illustration

Closed with PR - https://github.com/serengil/deepface/pull/1000


Chequebook breaking change

There is an upcoming breaking change in the cashout status. For more context see:
https://github.com/ethersphere/bee/pull/1402

fyi the PR and api change is already final. I am just waiting with merging until after the next minor release.

Thanks for the heads up! Will do this today.

PR was merged

This was included in our release version, so closing this.


DataArray.indices_min() and DataArray.indices_max() methods

These return dicts of the indices of the minimum or maximum of a DataArray over several dimensions. Inspired by @fujiisoup's work in #1469. With #3871, replaces #1469. Provides a simpler solution to #3160.
Implemented so that
da.isel(da.indices_min(list_of_dim)) == da.min(list_of_dim)
da.isel(da.indices_max(list_of_dim)) == da.max(list_of_dim)

[x] Tests added
[x] Passes isort -rc . && black . && mypy . && flake8
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API

This looks really comprehensive, thank you!
Before doing a really careful review here, I'd like to try to work out the full API design we want. I'll write out some of my thoughts here, but your thoughts would also be very welcome!
Here's my summary of the current situation:

We have two pairs of methods, argmin/argmax and the new idxmin/idxmax for calculating integer positions or coordinate labels of the minimum/maximum along one dimension.
We would like to have a way to do a similar calculation in multiple dimensions, which should return a dictionary of DataArray objects suitable for feeding into isel/sel.

This PR implements the multidimensional equivalent of argmin/argmax. It's fine to save the multidimensional equivalent of idxmin/idxmax for later, but we'll want to copy whatever design we pick here.
My first concern is about the name: it isn't obvious to me whether indices_min refers to integer positions or coordinate labels. I think a name based on argmin/argmax could help, e.g., perhaps argmin_nd or argmin_dict.
Another option would be to overload argmin/argmax to also support return a dictionary, depending on the type of the arguments. If you pass a list of dimensions over which to calculate the min or max locations, you would get a dictionary back:

array.argmin('x') would return a single DataArray
array.argmin(['x', 'y']) would return a dict of DataArray objects with keys {'x', 'y'}
array.argmin(['x']) would also return a dict {'x': ...}

I think I like this last option best but I would be curious what others think! @pydata/xarray any thoughts on this?

@shoyer I think your last option sounds good. Questions:

What should da.argmin() with no arguments do?

Currently returns the flattened index of the global minimum.
I think returning a dict of indices would be much more useful, but it does change existing behaviour (more useful because you can then do da.isel(da.argmin())).
Could anyway do da.argmin(da.dims) to get the dict result, but that's a bit ugly
Could have something like da.argmin(...) - maybe as a temporary workaround while we deprecate the current behaviour of da.argmin()?

If we overload argmin, what's the cleanest way to get the existing argmin() method within the new one? Would something like from .duck_array_ops import argmin as base_argmin work?

Maybe worth noting, at the moment if you try to call argmin(dim=("x", "y")) with multiple dimensions, there's a not-very-helpful exception
>>> array.argmin(dim=("x", "y"))
Traceback (most recent call last):
  File "/home/john/anaconda3/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 61, in _wrapfunc
    return bound(*args, **kwds)
TypeError: 'tuple' object cannot be interpreted as an integer

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/<...>/xarray/xarray/core/common.py", line 46, in wrapped_func
    return self.reduce(func, dim, axis, skipna=skipna, **kwargs)
  File "/<...>/xarray/xarray/core/dataarray.py", line 2288, in reduce
    var = self.variable.reduce(func, dim, axis, keep_attrs, keepdims, **kwargs)
  File "/<...>/xarray/xarray/core/variable.py", line 1579, in reduce
    data = func(input_data, axis=axis, **kwargs)
  File "/<...>/xarray/xarray/core/duck_array_ops.py", line 304, in f
    return func(values, axis=axis, **kwargs)
  File "/<...>/xarray/xarray/core/duck_array_ops.py", line 47, in f
    return wrapped(*args, **kwargs)
  File "<__array_function__ internals>", line 6, in argmin
  File "/<...>/anaconda3/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 1267, in argmin
    return _wrapfunc(a, 'argmin', axis=axis, out=out)
  File "/<...>/anaconda3/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 70, in _wrapfunc
    return _wrapit(obj, method, *args, **kwds)
  File "/<...>/anaconda3/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 47, in _wrapit
    result = getattr(asarray(obj), method)(*args, **kwds)
TypeError: 'tuple' object cannot be interpreted as an integer

Another option would be to overload argmin

+1 for overloading argmin (and later idxmin). IMO we should never have one function for a 1D operation and one for an N-D operation if we can avoid it, everything should be N-dimensional.
I also really like how neat this resultant property is
da.isel(da.argmin(list_of_dim)) == da.min(list_of_dim)

we could even use a hypothesis test to check it...

I think returning a dict of indices would be much more useful, but it does change existing behaviour (more useful because you can then do da.isel(da.argmin())).

Although it's breaking and would require a deprecation cycle, I think this is what we should aim for.

there's a not-very-helpful exception

Yes let's take the time to make that clearer for users - this will be a commonly-used function.

+1 for making argmin/argmax (and idxmin/idxmax`) work, when given multiple dimensions.
According to the current docstring it should already work that way for argmin/'argmax`:
Reduce this DataArray’s data by applying argmin along some dimension(s).
Returns: New DataArray/Dataset object with argmin applied to it's data and the indicated dimension(s) removed
But this behaviour is broken currently (works only for one given dim).
My main concern for changing the API as suggested above is, how should we discern (at least for argmin/argmax), if the user want's to:

reduce over all given dimensions? (current behaviour according to docstring)
reduce along all given dimensions? (suggested behaviour by @johnomotani [please correct if I'm wrong])

how should we discern (at least for argmin/argmax), if the user wants to...

That's a good question @kmuehlbauer, and the distinction probably needs to be clearer in the docs in general.

reduce over all given dimensions?

By this do you mean find the minimum as if the array were first (partially or totally) flattened along the given dims somehow? I'm not sure we provide that kind of behaviour anywhere in the current API.

reduce over all given dimensions?

By this do you mean find the minimum as if the array were first (partially or totally) flattened along the given dims somehow? I'm not sure we provide that kind of behaviour anywhere in the current API.

@TomNicholas Probably I was bit confused by the current docstring. I think I understand now and there should be no problem at all.

@johnomotani FYI: For #3871 (merged) there is #3922 (yet unmerged) to fix dask-handling.

I've updated so the new functionality is provided by argmin() and argmax(), when they are passed a multiple dimensions.

I've added Dataset.argmin() and Dataset.argmax(), which only work for a single dimension, but give informative exceptions for unsupported cases. If anyone has some behaviour that they would like for Dataset.argmin() with multiple dimensions, I'm happy to implement it, but returning something like a dict of Datasets didn't seem nice, and as far as I understand a Dataset cannot contain something like a dict of DataArrays.
I renamed argmin and argmax in ops.py and duck_array_ops.py to _argmin_base and _argmax_base so that I could still use them inside the argmin and argmax methods. Any alternatives, or better suggestions for naming them?
Variable no longer has argmin or argmax methods. Is that a problem? I just updated test_variable.py and test_dask.py to use _argmin_base and _argmax_base.

The rename to _argmin_base is also causing test_aggregation (which is a test of units with pint) to fail, because the numpy function and the Variable  method do not have the same name. Any suggestions how to fix this? It seems tricky because I can't pass either argmin (no Variable method) or _argmin_base (no numpy method) to the test.

These test failures seem to have uncovered a larger issue: overriding a method injected by ops.inject_all_ops_and_reduce_methods(DataArray, priority=60) is tricky. I think there may be a way around it by mangling the name of the injected operation if that method is already defined on the class; I tried this and it nearly works, but I think I need to implement an override for argmin/argmax on Variable, because DataArray.__array_wrap__ uses the Variable version of the method... Work in progress!

Also, feel free to rewrite to avoid using inject_all_ops_and_reduce_methods() for argmin/argmax at all. Method injection is pretty hacky, and generally not worthwhile, e.g., see this note about removing it.

I eventually found that the cause of the errors I was getting was that the argmin and argmax methods did not have an out argument. In order for the methods to be wrapped by numpy (and I guess dask is the same), the call signature of np.argmin() has to be supported, which means axis and out arguments are needed.
@shoyer I've kept the use of injected functions, because removing the injected argmin and argmax completely meant re-implementing handling of skipna and only within _unravel_argminmax, which seemed less nice to me. If/when method injection is refactored, it would be nice to include a mechanism to override the core operation with an explicitly implemented, extended version like argmin/argmax.
I think this PR is ready for review now :smile:

@pydata/xarray - I think this PR is ready to be merged, are there any changes I should make? Thanks!

I eventually found that the cause of the errors I was getting was that the argmin and argmax methods did not have an out argument. In order for the methods to be wrapped by numpy (and I guess dask is the same), the call signature of np.argmin() has to be supported, which means axis and out arguments are needed.

I guess this would happen if somebody calls np.argmin(xarray_object)? Does this happen from inside xarray somewhere, or in external code?
If it's the former, then we should just fix xarray not to do this. If it's only in external code, I would still consider breaking this behavior.
The override mechanism that NumPy uses in argmin/argmax is really old/fragile, and it's not something we ever intended to support. There's no way we could write an argmin/argmax method with an incompatible signature that would work with these functions.
The better way to do overrides in NumPy is probably with __array_function__ now, but I'm not sure we want the trouble...

I notice that some deprecation warnings get issued when running your tests. Please silence these methods, e.g., using pytest.mark.filterwarnings.

I guess this would happen if somebody calls np.argmin(xarray_object)? Does this happen from inside xarray somewhere, or in external code?
If it's the former, then we should just fix xarray not to do this. If it's only in external code, I would still consider breaking this behavior.

test_units.py has a couple of tests that apply numpy functions to xr.Variables, xr.DataArrays and xr.Datasets. Those tests break for argmin and argmax without the out argument. Should I just remove the argmin and argmax function calls? The tests would still call them as methods.

you could mark the function("argmin") / function("argmax") with xfail (or skip?). Then we can decide on what to do with them when we close #3917

I'm going to wait a little while before merging in case anyone else has comment, but otherwise will merge in a day or two (definitely before the next release)

I noticed a few issues while debugging:

calling on a array with missing values raises ValueError: All-NaN slice encountered (try ds = xr.tutorial.open_dataset("rasm"); ds.argmin(dim="y"))
ds.argmin(dim=...) returns a single index (the same result as ds.argmin()) but ds.Tair.argmin(dim=...) returns something different from ds.Tair.argmin(). Is that intentional?

ds.argmin(dim=...) returns a single index (the same result as ds.argmin()) but ds.Tair.argmin(dim=...) returns something different from ds.Tair.argmin(). Is that intentional?

I think this is a bug. dim=... to Dataset.argmin should be an error because dim=... returns a dict of results, and it's not clear how to do that consistently for a Dataset that might have several members with different combinations of dimensions.

well, we'd need to somehow be able to use sel to index every variable in a Dataset differently. While that would probably be possible it would make sel way too complicated, so I agree that an error would be good here.
I think no further changes to the pint tests should be required (except refactoring, but I can do that in a new PR after this one has been merged).

calling on a array with missing values raises ValueError: All-NaN slice encountered (try ds = xr.tutorial.open_dataset("rasm"); ds.argmin(dim="y"))

Some missing values should be OK. In your example though for example
In [27]: ds.Tair.isel(time=0, x=0).values
Out[27]:
array([nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,
       nan, nan, nan, nan, nan, nan, nan, nan, nan, nan])

So there are no values in the array slice that argmin should be applied over, which is an error.
I guess we could add some special handling for this (not sure what though, because we can't set a variable with type int to nan or None), but I think that is a separate issue that would need a new PR.

Thanks @keewis! I think we've addressed all the review comments now.

Thanks @johnomotani . This is a significant contribution.

Thanks @dcherian :smile: You're very welcome!


Segments parent directory not removed on permanent deletion with NFS deep-storage

I have setup a Druid cluster on five KVM (CentOs 6.5-based) on our private Joyent cloud.
The cluster executse the Coordinator, Historical, Overlord, Broker and Realtime node each one on a separate VM, whilst the middleManager is on the same VM of the Overlord (even if configured with druid.indexer.runner.type=remote).
Deep storage is configured on NFS (provided on NetApp appliance).
Using the "old coordinator console", I am able to disable a datasource and then to permanently delete its segments; however parent directories of removed segments are not removed from the deep storage.
As an example, if I have this segment-related file:
/NFS/sso024/2015-08-13T00:00:00.000Z_2015-08-14T00:00:00.000Z/2016-01-13T14:30:02.644Z/0/index.zip
after the deletion I have:
/NFS/sso024/2015-08-13T00:00:00.000Z_2015-08-14T00:00:00.000Z/2016-01-13T14:30:02.644Z/
Notice that killing tasks are marked "SUCCESSFULL" in the Overlord Web UI.

@himanshug do you know if this behavior exists for the HDFS deep storage as well?

Thanks for the detailed report @msspadoni !

@drcrallen  we fixed this for hdfs in https://github.com/druid-io/druid/pull/1659/ , same needs to be done to LocalDataSegmentKiller i guess. will send a PR to fix that.


undefined method `build_settings' for nil:NilClass Error

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Complete output when running fastlane, including the stack trace and command used

I am using the versioning plugin with fastlane, i get an error when it runs that ends my script:
[10:23:51]: -------------------------------------------
[10:23:51]: --- Step: get_version_number_from_plist ---
[10:23:51]: -------------------------------------------
+------------------+------------------------------+
|                  Lane Context                   |
+------------------+------------------------------+
| DEFAULT_PLATFORM | ios                          |
| PLATFORM_NAME    | ios                          |
| LANE_NAME        | ios distribute_to_testflight |
+------------------+------------------------------+
[10:23:52]: undefined method `build_settings' for nil:NilClass

+------+-------------------------------+-------------+
|                  fastlane summary                  |
+------+-------------------------------+-------------+
| Step | Action                        | Time (in s) |
+------+-------------------------------+-------------+
| 1    | Verifying required fastlane   | 0           |
|      | version                       |             |
| 2    | default_platform              | 0           |
| 3    | cocoapods                     | 3           |
| 💥   | get_version_number_from_plis  | 0           |
|      | t                             |             |
+------+-------------------------------+-------------+

[10:23:52]: fastlane finished with errors

Here is the fastfile:

platform :ios do

  before_all do
    cocoapods
  end

  lane :distribute_to_testflight do |options|
    ["Target1", "Target2", "Target3"].each do |scheme_name|

    scheme = scheme_name

    version = get_version_number_from_plist(xcodeproj: "projectTitle.xcodeproj", target: scheme, build_configuration_name: 'Release')

    current_build_number = latest_testflight_build_number(version: version)

    increment_build_number(
      build_number: current_build_number + 1
    )
    testflight_build = current_build_number + 1

      gym(
        scheme: scheme,
        export_method: "app-store",
        configuration: "Release",)

      pilot(
          skip_submission: false,
          distribute_external: false,
          skip_waiting_for_build_processing: true)

    end
end

Any idea what im doing wrong? i want to loop the targets and version them then upload them
Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
10.13.1

Ruby
2.3.3

Bundler?
false

Git
git version 2.14.3 (Apple Git-98)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.13.1 (17B1003)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib

OpenSSL Version
LibreSSL 2.2.7

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.2

System Locale

Variable
Value

LANG
en_GB.UTF-8
✅

LC_ALL

LANGUAGE

fastlane gems

Gem
Version
Update-Status

fastlane
2.68.2
✅ Up-To-Date

Loaded fastlane plugins:

Plugin
Version
Update-Status

fastlane-plugin-versioning
0.3.3
✅ Up-To-Date

Loaded gems

Gem
Version

did_you_mean
1.0.0

slack-notifier
1.5.1

rouge
2.0.7

xcpretty
0.2.8

terminal-notifier
1.8.0

unicode-display_width
1.3.0

terminal-table
1.8.0

plist
3.3.0

public_suffix
2.0.5

addressable
2.5.2

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.6.3

babosa
1.0.2

colored
1.2

highline
1.7.10

commander-fastlane
4.4.5

excon
0.59.0

faraday
0.13.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.0

gh_inspector
1.0.3

mini_magick
4.5.1

multi_json
1.12.2

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
1.0.0

dotenv
2.2.1

bundler
1.16.0

faraday_middleware
0.12.2

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.1

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

logging
2.2.2

jwt
2.1.0

memoist
0.16.0

os
0.9.6

signet
0.8.1

googleauth
0.6.2

httpclient
2.8.3

google-api-client
0.13.6

json
2.1.0

io-console
0.4.5

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.5.3

fastlane-plugin-versioning
0.3.3

generated on: 2017-12-11

The increment_build_number action is a wrapper around agvtool. If you're using AGV versioning, you shouldn't use the versioning plugin. If you're not using AGV, you should not use the built-in increment_build_number, increment_version_number, get_build_number or get_version_number actions (described here https://docs.fastlane.tools/actions/#project). The crash you're getting is coming from versioning, though I'm not sure why.

thanks, i used the plugin because i dont seem to be able to handle the various targets in my project with the default fastlane actions which dont take scheme params, is there a way i can achieve that so i dont need to use the plugin and can get around this error?

It sounds like you want different targets to have different version numbers and build numbers. This is legitimate, but incompatible with Apple Generic Versioning. If you're not familiar with AGV, here's an explanation: https://developer.apple.com/library/content/qa/qa1827/_index.html.
The increment_build_number and increment_version_number actions just call agvtool next-version and agvtool next-marketing-version, so they will do just what that Apple tech note says and set all targets to have the same version.
I think the versioning plugin can probably do just what you want. I'm not very familiar with it. You might get more help if you open an issue in that repo. But I think the exception is being raised here: https://github.com/SiarheiFedartsou/fastlane-plugin-versioning/blob/master/lib/fastlane/plugin/versioning/actions/get_info_plist_path.rb#L39. That means the target is nil, which probably means it's not finding the target by name in the project. That could mean the target name is misspelled or something. But it looks like you're calling the action right.
Instead of increment_build_number (the agvtool wrapper) you probably want to call increment_build_number_in_plist from the versioning plugin.

Im happy for the targets to have the same version and build, they will all always increment together, ill try and debug the versioning issue as you have said, but having no luck so far!

I think AGV might be what you really want, using built-in actions instead of the versioning plugin. You may also have to add a CURRENT_PROJECT_VERSION setting to your project (at the project level) for agvtool. The CURRENT_PROJECT_VERSION needs to be the same as the build number (CFBundleVersion) in all of your Info.plist files. It may be that agvtool doesn't need this, but sometimes it complains.

platform :ios do

  before_all do
    cocoapods
  end

  lane :distribute_to_testflight do |options|
    #
    # --- Update the build number for all targets
    #
    version = get_version_number(xcodeproj: "projectTitle.xcodeproj")
    current_build_number = latest_testflight_build_number(version: version)
    testflight_build = current_build_number + 1
    increment_build_number(
      xcodeproj: "projectTitle.xcodeproj"
      build_number: testflight_build
    )

    #
    # --- Now build each target/scheme
    #
    ["Target1", "Target2", "Target3"].each do |scheme_name|
      scheme = scheme_name

      gym(
        scheme: scheme,
        export_method: "app-store",
        configuration: "Release")

      pilot(
          skip_submission: false,
          distribute_external: false,
          skip_waiting_for_build_processing: true)
    end
end

But I also think instead of 7 lines of code, you can just do this:
    #
    # --- Update the build number for all targets
    #
    increment_build_number xcodeproj: "projectTitle.xcodeproj"

    #
    # --- Now build each target/scheme
    #
    ...


[Moderation] Add User Status Checks  on Endpoints

1/3 PRs for #224, ensuring that a user who has been banned/deleted recently will no longer be able to interact with the site.
The next 2 PRs will be to (1) Update all views for other users so they no longer see the usernames of deleted/banned users and (2) Add support for banning + deleting users (as admin or yourself).

I know we've stuck to lookup tables for Site Roles however I don't imagine the UserStatus field to grow in value. If anyone thinks there is a nonzero chance of this, please let me know and then I can create a separate UserStatus table.

Adding unverified state is definitely useful - I can see us gating certain functionality behind it.
I don't think we'll have any other statuses beyond the 4 mentioned - I think the idea of adding temporal annotations statuses seems to suggest a creation of a UserStatus Log / HIstory Table, which is just a running log of all status changes for a user with the timestamp and potentially a "previous action" column that stores context as to why the status change occured.
So in total, the Table UserStatusHistory would have columns (id, timestamp, user_id, status, explanation).
Mentally, I am modelling the state machine of Users to have 4 states (statuses) that I don't expect to change much but I feel we may introduce a variety of policies for moving between them (edges) often enough to keep them separate. What are your thoughts on implementing this approach?

What are your thoughts on implementing this approach?

I like it in principle, but this view of the world assumes we can model user states as a simple finite state automaton where 1 state = 1 node. But I think there are reasonable semantics we can associate with combinations of states, and having just one state can potentially lose information.
For example, I think it's meaningful to distinguish between deleted and banned and deleted and not banned if a user wishes to have their account reinstated for whatever reason. Likewise I think it's possible to have combinations like banned and not verified (e.g. if we allow limited activity without verification).
Using dates as a truth value is something I haven't done before, but it's a way to get bookkeeping for cheap without much complexity elsewhere in the stack (e.g. with a separate audit log). I do think we should have an audit log, but that's less urgent.
So although it feels clumsy, what do you think about having simple columns like:
# whether the user has a verified email
is_verified
# whether the user is banned
is_banned
# whether the user's account has been deleted (deactivated?)
is_deleted

and managing the state this way?

Ah I see, you make a fair point about having "multiple states at once". Let's stick to the columnwise approach and, like you said, create the audit log of "state changes" later on once the banning/removing functionality gets added.

@epicfaace ^ looks like the docker-setup is failing here -- can you help debug?


OWL-2635: Adding Stars to TopNav

https://jira.uwo.ca/browse/OWL-2635

Is there a sakai.property to enable/disable this feature?

I'm not familiar with adding properties to Sakai, so there is currently no property to control this.


Add elfeed-update-all

Update all feed in a special function so elfeed-sources could make advice to it instead of elfeed-update.
I'm not sure if the function name is well, you could adjust it if necessary, sir ;)

Yes, so I ask you to give new names ;)
How about just separate the methods and just let elfeed-sources works, firstly.


Engine datasource dependency

Minimize direct dependencies on dataSource (HiloFactoryBean, DataBaseDetector and ExecutionQueueRepositoryImpl)
Use @Resource instead of @Autowired
Consider supporting custom dataSource name
Consider these point for other DB related stuff (jdbcTemplate etc)

why is @Resource better if we are not specifying a qualifier?

With @Resource the resolution is done preliminary by name. Besides it's standard javax and recommended by spring.
Suggestion: Pass the ds name through EngineBeanDefinitionParser


Rules of React

Page: id.react.dev/reference/rules
Assignee:

is this issue already picked up by someone? otherwise, i will take it


fix(oop): update variables in ProcessDefinitionRequest

Query Parameters are updated according to documentation of REST API.
http://docs.camunda.org/latest/api-references/rest/#process-definition

Thanks for contributing.
We will review and merge the pull request. @shentschel, the maintainer of the PHP SDK is currently out of office, he will take a look at it when he comes back.
Cheers,
Daniel

HI jvaldesh,
sorry for the long wait time!
I merged this into the api and thank you for your contribution :+1: :)
Cheers,
Stefan


replace old ATS key name to new one.

On iOS project, ATS is disabled for 'localhost'.  But the key for this setting is old and just for early iOS9 beta.  iOS9 release and later uses new one, so I updated it.

@facebook-github-bot shipit


add workflow templates for build and deploy actions

These are taken from an existing repo with steps deleted that wouldn't generally be used everywhere.
For deploy there isn't really anything there, but it does define inputs.
For build there is a Go specific build.
This stripped down version hasn't been tested. I want to try to add it to this repo for testing, but I have to get it merged first.

I also added a generic build workflow with Go specific steps stripped out.

The release workflows are added now.


fix subnetselector label

Signed-off-by: Steven Borrelli<EMAIL_ADDRESS>Description of your changes
Changes eks nodeGroup patch to correctly match subnetIdSelector
Fixes #92
I have:

[x] Read and followed Upbound's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested
Created 2 claims on the same UXP cluster, reviewed node group subnet selection.

/test-examples


Integrationtest for authentication workflow

Current situation
There is no integration test that tests the entire authentication workflow.
Target
The entire authentication workflow is to be tested within an integration test. This includes

Register a new user account
Verify user account
Request reset password
Set new password
login
logout
Access to frontend if not logged in
Access to the backend if not logged in
Access to backend when logged in, but not in the correct user group
Access to the guest area when logged in

Moved to trello.


Sync playground tutorials and update helm index url

What do these changes do?

Related issue number

Fixes

LGTM


Fix to IT tests to match Updated Spark

Also I missed a rename on defaultSparkConf
Spark output for pushdown filtering changed so we change our tests to
match the expected output.

Build against Scala 2.11 finished with failure

Build against Scala 2.10 finished with failure

+1 in general, but looks like it compile stage still failing for src/it/scala/com/datastax/spark/connector/streaming/CheckpointStreamSpec.scala

retest this please

Build against Scala 2.10 finished with failure

Build against Scala 2.10 finished with failure

Build against Scala 2.11 finished with failure

Build against Scala 2.11 finished with failure


APEXCORE-660 Added documentation for custom control tuple support

@tweise Please see.
Currently just opened PR for review. Waiting for any other comments over the renaming of the control tuple: https://github.com/apache/apex-core/pull/482

Would this be better included in the operator developer guide?
@amberarrow please review also

The question whether this should be a separate document is still open. Looks a bit lost in the current structure, seems it should be a section of the operator developer guide. @amberarrow any thoughts on that?

@tweise I think we can add a small section in the operator developer guide and link to the control tuple doc from there.

@tweise @amberarrow Added a section in operator development on control tuple support and linked to that document. Please see.

@bhupeshchawda after changes are done, please rebase.

@tweise Done. Please see.

Done


Fix off by one error in View.expr_node

Noticed that the upper bound was off by one. Essentially the same code also appears on line 199, but there it does correctly include the -1.

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/shape/shapetracker.py             1               1
------------------------------------------------------------
lines added in the tinygrad folder: 0

Can you add a test that would have caught this?

Added a test for expr_node, has a bit of a pytest.mark.parameterize vibe to it. The loop seems out of style compared to the other tests but I need to check different idx Variables against different shape operations somehow.
The change I made to View.expr_node is tested through ShapeTracker.expr_node.
The default argument to ShapeTracker.expr_node is a bit weird as its a string:
def expr_node(self, idx='idx'):     if idx.__class__ is str: idx = Variable(idx, 0, prod(self.shape)-1)
Compared to the View.expr_idx:
def expr_node(self, idx=None) -> Node:     if idx is None: idx = Variable('idx', 0, prod(self.shape)-1)
Ideally expr_idxs and expr_node_mask should really be tested as well.
I'm working out a small cleanup of shapetracker atm, okay if I add those tests there?

Thanks!


Enable users to generate geo factors based on a collection of nodes

This PR adds a constructor to GeometricFactors class. The constructor enables users to generate geometric factors from a GridFunction.

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

@artv3, #1737 is now merged in master -- can you please merge master here and update the new constructor in this PR to be like the updated constructor from #1737?

Just to follow up here, is there anything else I can do?

LGTM

Merged in next for testing...


[FEATURE] Add Blog Page

Detailed description

[ ] Implement the Blog Page as per the new design.

Context
New feature on the community website
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hey @siddhi-244 @darkterminal @vinzvinci
I would like to work on this under GSSoc.
Please add the label GSSoC23.
And assign it to me.

The blog page will be changing to a new design. Please wait for now. I will update you when the new design is complete.

Okay, Sure!

btw I just noticed, there's past issue #283 currently working on this. You can work on other pages for now.

btw I just noticed, there's past issue #283 currently working on this. You can work on other pages for now.

Thank you for bringing that to my attention. I wasn't aware of the past issue #283 related to this matter. I appreciate your guidance, and I'll redirect my efforts towards working on other pages for now. Please let me know if there are any specific areas you'd like me to focus on. I'm eager to contribute effectively to the project.

Yeah sure, there's a lot more to come @s2sharpit! We'll keep you folks updated, just always check our pinned post.

Is this issue still open, if it is can I please get assigned to this under GSSOC'23?

Hey @vinzvinci
As there is no pr from the issue #283 .
Can I work on this.
If yes, can you assign me this issue.

Hey @vinzvinci @siddhi-244 @mkubdev @Panquesito7 @Mustafiz04 @Anmol-Baranwal @darkterminal As there is no pr from the issue #283 . Can I work on this. If yes, can you assign me this issue.

Don't tag everyone. You can tag me or @vinzvinci.
Also, I am not aware about any blog design, can you share that link, where v3 blog design is shown.

Don't tag everyone. You can tag me or @vinzvinci.
Also, I am not aware about any blog design, can you share that link, where v3 blog design is shown.

Hey @Anmol-Baranwal
It's in the figma design

Okay, then others will assign you the work, if the design is completed.

Hey @vinzvinci I have viewed the design of the blog page and would like to work on this issue.

@Kushagra102
Since s2sharpit asked for it first, I am assigning the task to him. Please search for other issues to contribute to. If you have any ideas, feel free to suggest them in an issue, and we will guide you on what you can contribute.

@s2sharpit
I don't have any information regarding the finalization of this design, so I cannot guarantee that the pull request will be merged. Let me ask the same question in the Discord channel.

Hey @vinzvinci
What's your views on it?

@s2sharpit
I have assigned you the issue. Please proceed with it, and don't hesitate to tag me if you require any assistance.

You can work on this now @s2sharpit

Subject: Inquiry about Data Source & Social Media Profile Links for Cards
Dear @vinzvinci @Anmol-Baranwal ,
I am writing to inquire about the data source and social media profile links for the cards in the API. Could you kindly clarify if the card data and associated GitHub, LinkedIn, and Twitter profile links are automatically sourced from an API or if they need to be added manually?
If the data, including social media links, is obtained from an external API, please provide documentation or guidelines for accessing it.
Alternatively, if the data and links are to be added manually, please advise on the recommended method. Could you kindly provide instructions on how to add the GitHub, LinkedIn, and Twitter profile links to the cards? It would be helpful to understand the specific format or structure required for successful integration.
I greatly appreciate your assistance and prompt response.
धन्यवाद (Thank You)

Subject: Inquiry about Data Source & Social Media Profile Links for Cards in Blog Page.
Dear @vinzvinci @Anmol-Baranwal ,
I am writing to inquire about the data source and social media profile links for the cards in the API. Could you kindly clarify if the card data and associated GitHub, LinkedIn, and Twitter profile links are automatically sourced from an API or if they need to be added manually?
If the data, including social media links, is obtained from an external API, please provide documentation or guidelines for accessing it.
Alternatively, if the data and links are to be added manually, please advise on the recommended method. Could you kindly provide instructions on how to add the GitHub, LinkedIn, and Twitter profile links to the cards? It would be helpful to understand the specific format or structure required for successful integration.
I greatly appreciate your assistance and prompt response.
धन्यवाद (Thank You)
Please refer to the provided screenshot for reference:
Figma Link: https://www.figma.com/file/3DBLz1I6bI651lizseSQ0d/WebX-DAO---Website?type=design&node-id=1330%3A2186&mode=design&t=UZviNxVkeeugCtFV-1

As per my knowledge, you cannot retrieve social links from GitHub api, so you need to add it manually. Just leave it empty for now. We will ask you to add the links before merging PR.


Docs to add

Data Services docs:
http://www.ni.com/documentation/en/systemlink-data-services/latest/manual/manual-overview/
Hosting docs:
http://www.ni.com/documentation/en/labview-web-module/3.0/manual/hosting-web-application-on-server/
HTTP Nodes docs:
http://www-preview.ni.com/documentation/en/labview/latest/node-ref/http-nodes/
Host on Systemlink docs:
http://www-preview.ni.com/documentation/en/systemlink/latest/deployment/hosting-ni-web-server/
Creating a custom swif entry:
https://github.com/ni/systemlink-web-interface-template
Github topic:
https://github.com/topics/webvi

closed in favor of #7


Presubmit check on engine -> framework roll queueing for 16 hours

https://github.com/flutter/flutter/pull/86913/checks?check_run_id=3139269614

If possible, it would be preferable to fail the roll and let the autoroller trigger a new one after some max queue time instead of letting a PR queue indefinitely.

Let me try triggering that builder first.

There is a typo in the builder config side. Fix cl: https://flutter-review.googlesource.com/c/infra/+/16060

Builder is triggered, and passed.
Created https://github.com/flutter/flutter/issues/86949 to track the request.
Closing here.


Avoid non-safe pyyaml's yaml.load() in build system

It was reported to the grpc-security mailing list that gRPC is using unsafe YAML functions. Making this public to ease tracking, as it does not seem to have an actual security impact to us. The function appears to only be involved in the build system such that an attack could just as easily modify a .py file. While not a security issue, we should try and clean this up to ease future audits.
yaml.load without a loader is unsafe. A search shows instances of load() without a loader specified https://github.com/grpc/grpc/search?q=yaml.load . It seems we noticed this before in https://github.com/grpc/grpc/pull/18829 but were worried about the changes suggested.
It's been a year and half since it was looked at previously, so it may now be fine to bump the version of pyyaml. But it was also pointed out that earlier versions of pyyaml actually do have the safe_load alternative: https://github.com/yaml/pyyaml/blob/3.12/lib3/yaml/init.py#L88 . It seems doubtful to me that we need the full loader, but I don't actually know how we're using YAML. The full loader is not safe itself (although it aspires to be safe-ish), so using it mainly just signals "yes, this is trusted input," especially if we aren't keeping the pyyaml version up-to-date. It may be good to replace the full loader usages with safe loader as well, from a "why are you even using that YAML feature" perspective.
CC @veblush @jtattermusch

@gnossen  since this is python related, could you please take a look into this?
CC @lidizheng

I think we can do this as a followup for @gnossen  python3 migration.

I upgraded the YAML invocations for the generate project code path at https://github.com/grpc/grpc/pull/24673

I upgraded the YAML invocations for the generate project code path at #24673

But the rest still needs to be updated, right?

Yes. Also, if we use newer Python version, the old YAML loading won't work. So, I guess part of this will be resolved by @gnossen 's Python3 migration.


Support retrieivng mapped/projected data from a document

We should offer a way to return data that is a query within the documents.
My current thoughts on this is to adopt an approach similar to GraphQL. A simple "Queryable" trait that allows a collection to respond to individual field queries, and a "Mutable" trait that allows mutation operations to be written.
When designing the traits, we should consider how a proc-macro could be used to generate the traits automatically by annotating a type with the extra information needed.

As per Discord, a proc-macro that allowed mapping/projecting data would also be able to provide basic View code generation.


Prevent route change in vaadin router in middle of some other task.

I am using vaadin router in LitELement. I have a vaadin upload component on one page. While uploading the content, if by mistakenly I change the route. In that case, I need to show a confirmation popup for the user. So how do I prevent this route change behavior by default and carry on the action depending on the option selected by the user in the confirmation?
Somewhat like onbeforeunload which works only when reloading the page, is there any event or method in vaadin to mimic that kind of feature in route change.
I don't want this feature for all the route changes. This should just be triggered whenever I am in middle of doing something and mistakenly I change the route.

@web-padawan  could you please help me here?

Hi @Krishuthapa, good to hear that you are using vaadin-router. For your use case, I think you can use the onBeforeLeave lifecycle callback. You can find the documentation here. We also have a helper interface BeforeLeaveObserver  to make it typesafe, for example, you can define your view like
class MyViewWithBeforeLeave extends LitElement implements BeforeLeaveObserver {
  onBeforeLeave(
      location: RouterLocation,
      commands: PreventCommands,
      router: Router) {
        if (!window.confirm('Are you sure you want to leave?')) {
          return commands.prevent();
        }
  }
}

@haijian-vaadin  thank you very much for the reply. I will try and see if it works for my use case.

@haijian-vaadin I see the docs mention commands.prevent will abort the navigation and fall back to the last one, will the router location object also update itself to match the navigation/location it's falling back to?


The useSize() demo for resize-observer doesn't capture element's initial size

Describe the bug
The useSize() function in the demo for @react-hook/resize-observer uses the anti-pattern of depending on the ref object provided by React.useRef() in the dependency list of React.useLayoutEffect().  This has the effect of not setting an initial value for the size on the first rendering.  This matters because elements that don't get re-rendered never get a sized.
To Reproduce
Use the size
Expected behavior
Get a size for elements on the first time they are rendered.
Additional context
Depending on the node itself via useState instead of the ref (which is never updated and therefor never calls) is the usual work around
export function useSize(): {
    ref: (node: HTMLDivElement | null) => void
    size: DOMRect | undefined
} {
    const [size, setSize] = useState<DOMRect | undefined>()
    const [node, ref] = useState<HTMLElement | null>(null)

    useLayoutEffect(() => {
        node !== null && setSize(node.getBoundingClientRect())
    }, [node])

    // Where the magic happens
    useResizeObserver(node, (entry) => {
        setSize(entry.contentRect)
    })

    return { ref, size } // this is also a little wierd since ref isn't a ref object but a setter function, but works if using it in the context of <div ref={ref}>
}

https://codesandbox.io/s/react-hookresize-observer-example-ft88x?file=/src/App.js
Fixed in the demo but feel free to submit a PR for the README :) Thanks!


Chess: Move A7 to A5

Please do not change the title. Just click "Submit new issue". You don't need to do anything else :D

move failed.


Make Hugo-Theme-Learn Multilingual.

Hi;
Thanks to port this awesome theme from grav to hugo.
My hope, question is : How hard would be; and If people have interest for it, to make Hugo-Theme-Learn Multilingual ?

Hello !
Did you try https://gohugo.io/tutorials/create-a-multilingual-site/ ?
Never tested it, but by reading the page, I think it would be possible to create multilingual site without modifying the theme.

@matcornic
Didnt try with Hugo-Theme-Learn but with an another theme earlier and few times with Jekyll
and you need to edit the theme by changing all string you considere need to being translated
such as: Title, description, tags, categories, dates, ...

Actually, there are only the following static strings used in the theme:

Built with love from Grav and Hugo
Edit this page
the whole 404 page

All the rest is user content. No dates are displayed by the theme itself.
So, I don't know if it's worth it ^^


bug: get valid datasource mono

Description

Added a method for getting a Valid Datasource Mono. Further Validations can be added and converted here.

Fixes # (issue)

#17096

Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Fix has been tested manually.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/ok-to-test sha=8d3bf34


Data missing when drill into genome browser

I was looking at:
fig|1773.173.peg.4048
And when I tried to look at it on the genome browser this was returned (see screenshot)

And here's another one:

Rebecca,
Did you mean to close this ticket?
Ron
Ron Kenyon
PATRIC Project Manager, patricbrc.org
Project Management Director
Virginia Bioinformatics Institute, Virginia Tech
Ph:<PHONE_NUMBER>
On 4/13/2015 4:17 PM, ARWattam wrote:

Closed #153 https://github.com/PATRIC3/patric3_website/issues/153.
—
Reply to this email directly or view it on GitHub
https://github.com/PATRIC3/patric3_website/issues/153#event-280142999.Web
Bug from
https://github.com/notifications/beacon/ADCnWuqxyJ_fFy8wHiAoxsNEcdYHFxDtks5n_BvHgaJpZM4D_sra.gif

Maulik, this feature belongs to a genome (genome_id:1773.173) below,
http://macleod.vbi.vt.edu:8080/solr/genome/select?q=genome_id:1773.173
but we're missing sequence
http://macleod.vbi.vt.edu:8080/solr/genome_sequence/select?q=genome_id:1773.173
any idea?

Fixed. All bad genomes without genomic sequences were removed and reloaded in the database.


chore: bdd-nh-1574355952 to 0.0.1

chore: Promote bdd-nh-1574355952 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


How to store the state in a struct

Sorry, surely a beginner question, I couldn't find in the doc or in the examples how I should store the state in a struct so multiple instances could be at different states, for example. What is the type of the generated states by the macro? I don't think I should store the sm object, no?

Hey @jeanmanguy,
The sm! macro returns the Machine trait object.
However, in order to validate the state machine at compile time, you get a new trait object back with different associated types each time the machine transitions to a different state.
I haven't used the library in a while, and my Rust is a bit rusty as I've been focused on Go for my day job lately, but I'd say you'd either have to store it as a dyn type in a Box, or you have to use generics to store the machine.
Let me know if that works for you.

I will try that that, thanks

I haven't used the library in a while, and my Rust is a bit rusty as I've been focused on Go for my day job lately, but I'd say you'd either have to store it as a dyn type in a Box, or you have to use generics to store the machine.

Didn't want to necrobump, but Machine isn't trait object-safe because it uses Self (a stand-in for a concrete type) as a return type. Since each transition changes the Machine's type (which is the intention for it to be type-safe), I'm not sure how generics would work. Are there any other alternatives to store the machine? @jeanmanguy did you find anything that worked?

You can use Rust's enum to store every state in one field and then match on it to get different behaviors.

That works. Thanks!

@ArekPiekarz @WinstonHartnett Can you provide a quick example of how to do this?

@tmilker I expanded the example from README to show how the type of the machine changes with each transition and how to store it in a field of another struct (Engine) using enum (generated Lock::Variant):
use sm::sm;

sm! {
    Lock {
        InitialStates { Locked }

        TurnKey {
            Locked => Unlocked
            Unlocked => Locked
        }

        Break {
            Locked, Unlocked => Broken
        }
    }
}

struct Engine
{
    machine: Lock::Variant
}

use Lock::*;
use Lock::Variant::*;

impl Engine
{
    fn handle(mut self, event: Break)
    {
        self.machine = match self.machine {
            InitialLocked(lock) => BrokenByBreak(lock.transition(event)),
            UnlockedByTurnKey(lock) => BrokenByBreak(lock.transition(event)),
            LockedByTurnKey(lock) => BrokenByBreak(lock.transition(event)),
            BrokenByBreak(lock) => BrokenByBreak(lock)
        }
    }
}

fn main() {
    // types on the left written explicitly for educational purpose
    let lock: Machine<Locked,sm::NoneEvent> = Machine::new(Locked);
    let lock: Machine<Unlocked,TurnKey> = lock.transition(TurnKey);

    let engine = Engine{machine: UnlockedByTurnKey(lock)};
    engine.handle(Break);
}

@ArekPiekarz Now that I've had some time to look at your example: There's no way to just store the machine and transition that?
I haven't found any state machine patterns or crates for Rust that don't feel like a hack in some way.


refactor(linter): use run_on_jest_node for existing lint rules

closes https://github.com/oxc-project/oxc/issues/6038

Migrates all simple iterations over Jest nodes to use the run_on_jest_node method. Rules which require more custom data (such as getting nodes in order or storing data about counts, duplicates, etc.) have not been migrated.
Some simple benchmarking shows that this is ~5% faster on vscode when the Jest/Vitest rules are NOT enabled (due to being able to skip them now). And it is up to 8% on vscode when the Jest/Vitest plugins are enabled.
hyperfine -i -N --warmup 3 --runs 30 './oxlint-jest-iter --deny=all --silent ./vscode' './oxlint-main --deny=all --silent ./vscode'
Benchmark 1: ./oxlint-jest-iter --deny=all --silent ./vscode
  Time (mean ± σ):      2.348 s ±  0.104 s    [User: 16.922 s, System: 0.641 s]
  Range (min … max):    2.141 s …  2.544 s    30 runs

Benchmark 2: ./oxlint-main --deny=all --silent ./vscode
  Time (mean ± σ):      2.476 s ±  0.042 s    [User: 17.768 s, System: 0.668 s]
  Range (min … max):    2.430 s …  2.598 s    30 runs

Summary
  ./oxlint-jest-iter --deny=all --silent ./vscode ran
    1.05 ± 0.05 times faster than ./oxlint-main --deny=all --silent ./vscode

hyperfine -i -N --warmup 3 --runs 30 './oxlint-jest-iter --deny=all --jest-plugin --vitest-plugin --silent ./vscode' './oxlint-main --deny=all --jest-plugin --vitest-plugin --silent ./vscode'
Benchmark 1: ./oxlint-jest-iter --deny=all --jest-plugin --vitest-plugin --silent ./vscode
  Time (mean ± σ):      2.728 s ±  0.118 s    [User: 19.782 s, System: 0.786 s]
  Range (min … max):    2.580 s …  3.078 s    30 runs

Benchmark 2: ./oxlint-main --deny=all --jest-plugin --vitest-plugin --silent ./vscode
  Time (mean ± σ):      2.939 s ±  0.051 s    [User: 21.259 s, System: 0.859 s]
  Range (min … max):    2.848 s …  3.064 s    30 runs

Summary
  ./oxlint-jest-iter --deny=all --jest-plugin --vitest-plugin --silent ./vscode ran
    1.08 ± 0.05 times faster than ./oxlint-main --deny=all --jest-plugin --vitest-plugin --silent ./vscode

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6722  👈
#6721
#6720
#6719
#6718
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @camchenry and the rest of your teammates on  Graphite

Nice, performance looks better:

Merge activity

Oct 20, 10:05 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


Migrate snforge to use profiler structs (trace_data.rs)

Which components does the task require to be changed? (think hard pls)
snforge
Description
Should be done when structs in the profiler are stable

Blocked by #1724


fix(pkg,cmd,validate): fixed flatcar builder

What type of PR is this?
/kind bug

Any specific area of the project related to this PR?
/area pkg
What this PR does / why we need it:
Fix a crash in the flatcar builder; moreover, make it actually work.
Basically it is now the same as vanilla/minikube builders, with some small differences. It will now require kernelconfigdata option. That is much more smart than fetching the kernelconfigdata URL only to download it in the template script, considering that we already had a kernelconfigdata option in driverkit.
Moreover, moved all builder structs receivers to pointers, so that we can actually update any struct members in receiver methods.
Finally, actually require kernelconfigdata param for minikube and flatcar too.
This goes together with https://github.com/falcosecurity/kernel-crawler/pull/50.
Which issue(s) this PR fixes:
Flatcar crash + make it working.
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

fix(pkg): fixed flatcar builder crash and implementation

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: e71332b25b1db46773f384a839705bc5a8298f69

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dwindsor, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,dwindsor]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add Compability and Fixes for Middleman 4 & 5

Middleman 4+ support
Fix undefined method `app.escape_html' in Middleman v4.1+, solves #8
Fix undefined method find_all for Middleman::Sitemap::ResourceListContainer in Middleman v5.0.0.rc2+

@statonjr

Hi @westonganger is there a reason you deleted your fork?

I dont remember, sometimes I have cleaned up old forks I have. This PR is still relevant, just seems like this plugin is unmaintained.


Migrate GitHub Actions to Node 20

GitHub will soon require that all Actions run with Node 20
https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/
https://github.com/actions/checkout/releases/tag/v4.0.0
https://github.com/peter-evans/repository-dispatch/releases/tag/v3.0.0
https://github.com/actions/setup-node/releases/tag/v4.0.0
You can see example deprecation warnings in one of our recent runs:
https://github.com/abbvie-external/OmicNavigatorWebApp/actions/runs/8147004420

Thanks for this update @jdblischak.  @bengalengel, @ternst23, @JoeDalen81 take a quick look and if voice any questions/concerns, otherwise let's merge


Convert Dockerfile into a multistage one.

Removes need for builder pattern.

@stephenwithav you can merge.

Done.  Thank you, sir.


Implement generateScalarImplementation for linalg_ext.fft op.

This is a step toward https://github.com/google/iree/issues/6477

Nice to see no changes to ConvertToLoops was needed!

Me too! Thanks for the refactoring!