Breaking API Change in 1.638?

I'm seeing api calls throwing 500s  starting at 1.638, which is a patch to implement security fixes:
https://wiki.jenkins-ci.org/display/SECURITY/Jenkins+Security+Advisory+2015-11-11
Has anybody else tried this out yet?
I'm doing job creation in a Jenkins unit test:
JenkinsServer jenkins = new JenkinsServer(j.getURL().toURI());
String job11Xml = Resources.toString(Resources.getResource("updated_" + JOB1_CONFIG), Charsets.UTF_8);
jenkins.updateJob(JOB1_NAME, job11Xml);

Sorry to be anal, but can you provide a 'working minimal example'? Maybe a compilable example.java?, along with a specification of your Jenkins configuration showing the configuration options you've gotten set up that demonstrate this change?

Yea I know, its a totally lame bug report.  You're not being anal.  I'll get some more data or close this down myself.


Freshplayerplugin crashes on iqiyi.com

Open a video  on  http://www.iqiyi.com
Close the tab before the video end.

And then the plugin crashed.
I tested other video sites ,like youku.com, and this phenomenon didn't appear.
I installed freshplayer(from git) today.
flash  version : <IP_ADDRESS>5
I'm using Arch Linux 64-bit.
Finally,sorry for my poor English.

Fixed in 7b0c829d9f0a055424ef07eccc65fb395ba4cb82.
Not actually fixed, udp sockets are essential for Flash, but currently there is no implementation in freshwrapper. But at least player on iqiyi.com doesn't hang anymore.
Created #147 to track ppb_udp_socket implementation status.


Reference CSS from index.html outside of the static folder?

I'm totally lost on how to reference CSS files within the /src/assets folder. Why is the path not serving at /assets/app.css?

As is documented in the Vue docs  you use the assets the following way (btw they are not under src)
Asset Resolving Rules

Relative

URLs, e.g. ./assets/logo.png will be interpreted as a module dependency. They will be replaced with an auto-generated URL based on your Webpack output configuration.

Non-prefixed

URLs, e.g. assets/logo.png will be treated the same as the relative URLs and translated into ./assets/logo.png.

URLs

prefixed with ~ are treated as a module request, similar to require('some-module/image.png'). You need to use this prefix if you want to leverage Webpack's module resolving configurations. For example if you have a resolve alias for assets, you need to use <img src="~assets/logo.png"> to ensure that alias is respected.

Root-relative URLs

e.g. /assets/logo.png are not processed at all.


Instruction header file errors for Mac ARM

Mac ARM doesn't seem to support using the xmmintrin.h instructions
using gcc-11
/Users/eart0557/Builds/AxiSEM-3D/SOLVER/src/fenv.cpp:12:10: fatal error: xmmintrin.h: No such file or directory
   12 | #include <xmmintrin.h>
      |          ^~~~~~~~~~~~~

using clang
In file included from /Users/eart0557/Builds/AxiSEM-3D/SOLVER/src/fenv.cpp:12:
In file included from /Library/Developer/CommandLineTools/usr/lib/clang/13.1.6/include/xmmintrin.h:13:
/Library/Developer/CommandLineTools/usr/lib/clang/13.1.6/include/mmintrin.h:33:5: error: use of undeclared identifier '__builtin_ia32_emms'; did you mean '__builtin_isless'?
    __builtin_ia32_emms();

Fixed. Thanks
https://github.com/kuangdai/AxiSEM-3D/commit/053d2174a566ad354d6faaafc6960a6e9defbb62

Add -Dskip_mm_set=true when doing cmake.


substrait error when I run cargo test test_format

What type of bug is this?
Other
What subsystems are affected?
Other
What happened?
➜  new_greptimedb git:(develop) ✗ cargo test test_format
   Compiling common-query v0.3.2 (/home/neo/new_greptimedb/src/common/query)
   Compiling substrait v0.10.0
   Compiling substrait v0.7.5
   Compiling common-datasource v0.3.2 (/home/neo/new_greptimedb/src/common/datasource)
   Compiling common-recordbatch v0.3.2 (/home/neo/new_greptimedb/src/common/recordbatch)
error: failed to run custom build command for `substrait v0.10.0`

Caused by:
  process didn't exit successfully: `/home/neo/new_greptimedb/target/debug/build/substrait-7dd109d702698189/build-script-build` (exit status: 1)
  --- stdout
  cargo:rerun-if-env-changed=FORCE_REBUILD
  cargo:rerun-if-changed=substrait
  cargo:rerun-if-changed=substrait/text/simple_extensions_schema.yaml
  cargo:rerun-if-changed=substrait/proto/substrait/type_expressions.proto
  cargo:rerun-if-changed=substrait/proto/substrait/capabilities.proto
  cargo:rerun-if-changed=substrait/proto/substrait/plan.proto
  cargo:rerun-if-changed=substrait/proto/substrait/function.proto
  cargo:rerun-if-changed=substrait/proto/substrait/extended_expression.proto
  cargo:rerun-if-changed=substrait/proto/substrait/type.proto
  cargo:rerun-if-changed=substrait/proto/substrait/extensions/extensions.proto
  cargo:rerun-if-changed=substrait/proto/substrait/parameterized_types.proto
  cargo:rerun-if-changed=substrait/proto/substrait/algebra.proto

  --- stderr
  Error: Custom { kind: Other, error: "protoc failed: google/protobuf/empty.proto: File not found.\nsubstrait/type.proto:6:1: Import \"google/protobuf/empty.proto\" was not found or had errors.\nsubstrait/type.proto:192:7: \"google.protobuf.Empty\" is not defined.\nsubstrait/type_expressions.proto:6:1: Import \"substrait/type.proto\" was not found or had errors.\nsubstrait/type_expressions.proto:61:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:67:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:73:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:80:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:86:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:97:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:104:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:110:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:15:5: \"Type.Boolean\" is not defined.\nsubstrait/type_expressions.proto:16:5: \"Type.I8\" is not defined.\nsubstrait/type_expressions.proto:17:5: \"Type.I16\" is not defined.\nsubstrait/type_expressions.proto:18:5: \"Type.I32\" is not defined.\nsubstrait/type_expressions.proto:19:5: \"Type.I64\" is not defined.\nsubstrait/type_expressions.proto:20:5: \"Type.FP32\" is not defined.\nsubstrait/type_expressions.proto:21:5: \"Type.FP64\" is not defined.\nsubstrait/type_expressions.proto:22:5: \"Type.String\" is not defined.\nsubstrait/type_expressions.proto:23:5: \"Type.Binary\" is not defined.\nsubstrait/type_expressions.proto:24:5: \"Type.Timestamp\" is not defined.\nsubstrait/type_expressions.proto:25:5: \"Type.Date\" is not defined.\nsubstrait/type_expressions.proto:26:5: \"Type.Time\" is not defined.\nsubstrait/type_expressions.proto:27:5: \"Type.IntervalYear\" is not defined.\nsubstrait/type_expressions.proto:28:5: \"Type.IntervalDay\" is not defined.\nsubstrait/type_expressions.proto:29:5: \"Type.TimestampTZ\" is not defined.\nsubstrait/type_expressions.proto:30:5: \"Type.UUID\" is not defined.\n" }
warning: build failed, waiting for other jobs to finish...
error: failed to run custom build command for `substrait v0.7.5`

Caused by:
  process didn't exit successfully: `/home/neo/new_greptimedb/target/debug/build/substrait-f3b7bf90c67bcfea/build-script-build` (exit status: 1)
  --- stdout
  cargo:rerun-if-env-changed=FORCE_REBUILD
  cargo:rerun-if-changed=substrait
  cargo:rerun-if-changed=substrait/text/simple_extensions_schema.yaml
  cargo:rerun-if-changed=substrait/proto/substrait/type_expressions.proto
  cargo:rerun-if-changed=substrait/proto/substrait/capabilities.proto
  cargo:rerun-if-changed=substrait/proto/substrait/plan.proto
  cargo:rerun-if-changed=substrait/proto/substrait/function.proto
  cargo:rerun-if-changed=substrait/proto/substrait/extended_expression.proto
  cargo:rerun-if-changed=substrait/proto/substrait/type.proto
  cargo:rerun-if-changed=substrait/proto/substrait/extensions/extensions.proto
  cargo:rerun-if-changed=substrait/proto/substrait/parameterized_types.proto
  cargo:rerun-if-changed=substrait/proto/substrait/algebra.proto

  --- stderr
  Error: Custom { kind: Other, error: "protoc failed: google/protobuf/empty.proto: File not found.\nsubstrait/type.proto:6:1: Import \"google/protobuf/empty.proto\" was not found or had errors.\nsubstrait/type.proto:192:7: \"google.protobuf.Empty\" is not defined.\nsubstrait/type_expressions.proto:6:1: Import \"substrait/type.proto\" was not found or had errors.\nsubstrait/type_expressions.proto:61:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:67:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:73:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:80:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:86:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:97:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:104:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:110:5: \"Type.Nullability\" is not defined.\nsubstrait/type_expressions.proto:15:5: \"Type.Boolean\" is not defined.\nsubstrait/type_expressions.proto:16:5: \"Type.I8\" is not defined.\nsubstrait/type_expressions.proto:17:5: \"Type.I16\" is not defined.\nsubstrait/type_expressions.proto:18:5: \"Type.I32\" is not defined.\nsubstrait/type_expressions.proto:19:5: \"Type.I64\" is not defined.\nsubstrait/type_expressions.proto:20:5: \"Type.FP32\" is not defined.\nsubstrait/type_expressions.proto:21:5: \"Type.FP64\" is not defined.\nsubstrait/type_expressions.proto:22:5: \"Type.String\" is not defined.\nsubstrait/type_expressions.proto:23:5: \"Type.Binary\" is not defined.\nsubstrait/type_expressions.proto:24:5: \"Type.Timestamp\" is not defined.\nsubstrait/type_expressions.proto:25:5: \"Type.Date\" is not defined.\nsubstrait/type_expressions.proto:26:5: \"Type.Time\" is not defined.\nsubstrait/type_expressions.proto:27:5: \"Type.IntervalYear\" is not defined.\nsubstrait/type_expressions.proto:28:5: \"Type.IntervalDay\" is not defined.\nsubstrait/type_expressions.proto:29:5: \"Type.TimestampTZ\" is not defined.\nsubstrait/type_expressions.proto:30:5: \"Type.UUID\" is not defined.\n" }

What operating system did you use?
fedora 38
Relevant log output and stack trace
No response
How can we reproduce the bug?
cargo test test_format
my environment

libprotoc 3.19.6
code is latest 8bea85395444e640cfef93cc38ff6445213677c8

Hi @gobraves which protoc version you are using?

Before merging the remote code yesterday, I ran the tests successfully. However, after merging the remote code, I encountered this error today. The last time I updated the code was probably around last Tuesday or so, but I can't recall the exact time.

Before merging the remote code yesterday, I ran the tests successfully. After merging the remote code, I encountered this error today. The last time I updated the code was probably around last Tuesday or so, but I can't recall the exact time.

Can you check the dir where the protoc was installed? You should see a file called empty.proto.
➜  protobuf echo $PWD
/home/weny/.local/include/google/protobuf
➜  protobuf ls
any.proto  compiler          duration.proto  field_mask.proto      struct.proto     type.proto
api.proto  descriptor.proto  empty.proto     source_context.proto  timestamp.proto  wrappers.proto

I don't have protobuf cmd. And I found that I don't have google proto files. So after installing protobuf-devel, cargo test will be ok. But I don't know why it could run before :smile:

Before merging the remote code yesterday, I ran the tests successfully. After merging the remote code, I encountered this error today. The last time I updated the code was probably around last Tuesday or so, but I can't recall the exact time.

Can you check the dir where the protoc was installed? You should see a file called empty.proto.
➜  protobuf echo $PWD
/home/weny/.local/include/google/protobuf
➜  protobuf ls
any.proto  compiler          duration.proto  field_mask.proto      struct.proto     type.proto
api.proto  descriptor.proto  empty.proto     source_context.proto  timestamp.proto  wrappers.proto

I don't have protobuf cmd. And I found that I don't have google proto files. So after installing protobuf-devel, cargo test will be ok. But I don't know why it could run before 😄

Great, Let's close this issue.

Before merging the remote code yesterday, I ran the tests successfully. After merging the remote code, I encountered this error today. The last time I updated the code was probably around last Tuesday or so, but I can't recall the exact time.

Can you check the dir where the protoc was installed? You should see a file called empty.proto.
➜  protobuf echo $PWD
/home/weny/.local/include/google/protobuf
➜  protobuf ls
any.proto  compiler          duration.proto  field_mask.proto      struct.proto     type.proto
api.proto  descriptor.proto  empty.proto     source_context.proto  timestamp.proto  wrappers.proto

I don't have protobuf cmd. And I found that I don't have google proto files. So after installing protobuf-devel, cargo test will be ok. But I don't know why it could run before smile

I figured it out. Previously, I didn't encounter any protobuf-related errors because I ran the tests under the "datasource" crate...


Revisit the animations

Revisit the height animations that used to be part of the project for certain types. http://stackoverflow.com/a/8331169

Not going to animate the height but going to fix the animations coming off the side of the screen. This is going to remove the height option for the plug-in.


SSR server freezes after throwing error.

Both apollo-server and graphql-yoga automatically collect errors thrown in resolver functions and pass a sanitized version of the erros to the front-end. That works perfectly without ssr.
However, when doing this with vue-cli-plugin-ssr, the ssr server freezes when the graphql server thrown errors. Not only yarn ssr:serve but also yarn ssr:start behaves like that, making it quite fragile in production.
I am not sure if this is due to my mis-configuration or something else?

I understand that this is because the graphql server throws error back to the ssr server, so the ssr serve fails to prefetch proper data and cannot do the proper initial rendering. I just wonder what is the proper way the handle this situation. Simply keeping the reject uncaught and letting the ssr server go frozen is clearly not the ideal solution.
Any thoughts about that, guys? ;)

It's simply solved by adding a .catch()  after ApolloSSR.prefetchAll(...)  in entry-server.js like this:
// entry-server.js

...
      Promise.all([
        // Async data
        ...matchedComponents.map(Component => {
          if (Component.asyncData) {
            return Component.asyncData({
              store,
              route: router.currentRoute,
            })
          }
          return null
        }),
        // Apollo prefetch
        ApolloSSR.prefetchAll(apolloProvider, [App, ...matchedComponents], {
          store,
          route: router.currentRoute,
        }).catch(error => console.error(error)), // ***HERE IS THE CATCH***
      ]).then(() => {
        // After all preFetch hooks are resolved, our store is now
        // filled with the state needed to render the app.
        // When we attach the state to the context, and the `template` option
        // is used for the renderer, the state will automatically be
        // serialized and injected into the HTML as `window.__INITIAL_STATE__`.
        Object.assign(context, { state: store.state })

        // Same for Apollo client cache
        Object.assign(context, { apolloState: ApolloSSR.getStates(apolloProvider) })
        return resolve(app)
      })

...


This is more than what is documented here, a shame it does not include 203, possibly the most appropriate where this is an "outage" or "maintenance" page. It would be nice as well to see the 990-998 usages (suggested or otherwise) documented somewhere.

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

ID: 814193da-7d05-d565-7159-fc258450e325
Version Independent ID: 92775277-873d-eec1-ab69-bd4764f67e10
Content: Policy settings for Web Application Firewall in Azure Front Door
Content Source: articles/web-application-firewall/afds/waf-front-door-policy-settings.md
Service: web-application-firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

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

@vhorne
Can you please check and add your comments on this doc update request as applicable.

@mckenzm
I'm going to assign this to the document author so they can take a look at it accordingly

Thanks for your dedication to our documentation. We have created an internal work item in our backlog to resolve this issue. If you determine another possible update to our documentation, please don't hesitate to reach out again.
#please-close


URL goes into external browser on Mac Catalyst.

An interesting (actually weird) issue here, when running on Mac Catalyst, instead of showing the specified URL with the ViewController, it opens externally with the default browser:
https://github.com/stleamist/BetterSafariView/assets/8158163/2009133e-49ca-4559-b360-e3e0559bb1ce

figure this out or still an issue?


How to stop current tween?

First of all  app.tween(object)
Creates a new instance of a tween. You can have multiple concurent tweens. So how to stop a tween before running another one?
Method 1
Save reference to a tween and stop it before calling another one.
var current = app.tween(turtle).to({ x: 100, y: 200 }).loop();

/* later in the code you want to call another tween */

current.stop();
app.tween(turtle).to(...);

Method 2
Use .discard() to kill all other tweens attached to the object.
app.tween(turtle).to({ x: 100, y: 200}).loop();

/* later in the code */

app.tween(turtle).discard().to({ x: 200, y: 300}).loop();

/* discard() killed all previous tweens */

my suggestion:
.destroy to discard and destroy.
.stop to just stop and .resume to resume.


startDebugging method in vscode still return true even when the build fails

Live share extension on the host side is using the startDebugging method to launch a debug configuration remotely. As part of step it will eventually need to invoke a task to build that could fail, the problem is that the API will not report 'false' value during the invocation.
Extract of the code here:
const result = await vscode.debug.startDebugging(folders ? folders[0] : undefined, debugConfiguration);
if (!result) {
// THIS code almost never get invoked....
throw new Error(Failed to launch debug configuration:${debugConfiguration.name});
}
we would like a better mechanism to know what really happen in our launch invocation.
Maybe knowing:

If the build task was cancelled
task failed
Debug configuration is invalid
Failure to launch

Adding a new richer method may be better...

I have exact the same problem with the API vscode.debug.startDebugging.
When was starting a Java debug session with the API vscode.debug.startDebugging, build error occurs. It popped up a message dialog letting me choose whether to "Proceed" or "Abort". This part is provided in extension "Debugger for Java". See: https://github.com/Microsoft/vscode-java-debug/blob/0e86ed2cc7e67f423a25421627dad33a89f4a2a2/src/configurationProvider.ts#L116-L119
After clicking "Abort", resolveDebugConfiguration returns "undefined".
According to the VS Code API , the debug session will not start.

Returning the value 'undefined' prevents the debug session from starting.

But actually vscode.debug.startDebugging still resolves "true", it does not match the API spec:

@return A thenable that resolves when debugging could be successfully started.

duplicate of #52737

After #53874 has been fixed it now becomes clear that there is another issue that prevents build errors from bubbling up to the extension API.

Is it enough to verify that a build error reaches the startDebugging API? Or was there a more specific issue with it?

The startDebugging API was always returning true. To verify:

Have a wrong launch configuration, verify that startDebuggin returns false
Have a prelaunch task that fails and verify that startDebugging returns false if you decided not to continue debugging


Make "Code collapse buttons" also visible at the end

I did my best to look and find if there is no similar issue, but i don't know exactly how this "code collapse button" is called, my apologies if this was already requested
In long methods, it is annoying to have to scroll up to find the collapse code button..
I know there is a shortcut for everything, but i can never remember them, so i prefer having UI guides
In Jetbrains IDEs it is possible, take a look at this screenshot:

You can see the buttons are visible for both begin, and end, as well as a visual guide for the right scope
It would be cool to have same thing in vscode

If you know how the buttons are called please let me know, or edit the title, thanks

/duplicate of #60670 - the commonly used term is 'folding'. Thanks anyway for trying to find similar issues before you posted this.

thanks @gjsjohnmurray


Proposal: Add Array, Dict, Set Literal Syntax

Inspired by Clojure's collection literals, I propose to add support for Array, Dict, and Set literals (analogous to List syntax).
Proposed syntax
Array
array =
  #[1, 2, 3]

Rationale: Looks like a list but with a hash sign.
Dict
dict =
  #{ "key1" : "value1"
   , "key2" : "value2"
   , "key3" : "value3"
   }

Rationale: Looks like Json. In spirit, Dicts and Json serve the same purpose--to store key-value data.
Set
set =
  #{1, 1, 2, 3, 2}

Rationale: While we're at it, why not? I mean... Clojure does it.
In all cases, the main goal of the syntax is to encourage the usage of these datastructures, which have advantages that linked lists don't. Arrays have fast access, setting, and efficient appending to the end. Dicts are perfect for storing key-value data. Sets guarantee that there are no duplicates (therefore, no need to filter out all duplicates from a list).
Currently, if you want to write a DSL in Elm (like elm-html), lists and records are your only tools. The downside of this is that lists may not be the optimal data structure for your DSL.
For example, here is how elm-html would look with arrays and dicts
div
  #[ style #{ "position" : "absolute" , "color" : "red" }
   , id "containerDiv"
   ]
  #[ ul #[]
      #[ li #[] #[text "Hello"]
       , li #[] #[text "World"]
       ]
   ]

I dislike this. More syntax is bad. Specifically, I almost never use sets on the job. Moreover, since our target user is accustomed to using dictionaries for everything, convenient record syntax pushes them to think typed-ly.

It's just sugar--and a good, terse sugar that other languages are already using. I don't see why you couldn't have still have Set., Dict., Array. for backwards compatibility or whatever your stylistic conviction may be. I feel like I'd be more inclined to use other (read: better) data structures if the API was this way. Maybe you can rope in some Clojure/Script users in as well...

Regarding the dict example: four quotes on a line is pretty ugly. Without endorsing the proposal, I think it should be = not :.
I think the more fundamental concern here is, what problem are we trying to solve? If it's "we have all these great data structures but no one uses them," then literals are only part of the problem. You'll also need

the Array etc module to be imported by default (sounds like an uphill battle)
examples that use these data structures on the website, perhaps with comparisons between data structure choices
more reliability in these libraries. Dict and Set will don't treat == in a useful fashion. The array library had some bugs when it came out and I'm not sure if these are all flushed out.

@mgold Yes, that is exactly the problem I'm trying to solve. We have amazing data structures and no one uses them. The reason I looked at Clojure as an example is because, as a Lisp, it would sound a-priori unwise to extend the Lisp syntax to include stuff like curly braces and square brackets. But, that bet paid off as vectors (equivalent to Elm Arrays) are preferred to lists as a means to store sequential data.
As regards the actual Array, Dict, Set implementations, I think it would be worth considering either strengthening these implementations by thorough testing, performance tweaks, etc... Or, we could consider switching to something like Immutable.js or Mori. But that's a discussion for another day I fear.
@dnalot About the target user.  JS people are accustomed to having efficient indexing in collections and never really deal with linked lists. (I don't remember ever using a linked list in JS to be honest) As such, we could consider a stronger form of the proposal specifically to target JS people.
The stronger form
Redefine the square bracket syntax to mean array instead of list (and find some other syntax for lists). Yes, this would break with the ML family. But like @evancz pointed out in his talk, the goal is not to make Haskell or Standard ML more usable and more maintainable. The goal is to make Javascript more usable and more maintainable. Javascript has Arrays. JS people know arrays, not linked lists. We can cater to JS people by having square brackets mean Arrays.
array : Array Int
array =
  [1..10]

We can then figure something out for lists.

I like the premise, we should use more data structures than just lists. I'm not sure this literal syntax is the way to go though. If we expand the literals syntax, I think we should have a general way to do so, not specialise for some stuff that seems nice right now.

I think this is a nice proposal, great work @TheSeamau5! I think it's kind of sad that # is an ugly thing to be putting in so many places, but I think the HTML example is pretty compelling.
My first intuition was the same as @Apanatshka's: "is there something general happening here?" Perhaps not. Maybe that way is a quagmire of things we don't actually care about. Maybe if our core data structures are easy to use, then we are all set.
I also agree with @mgold that = is better. If we ever allow type annotations in expressions, the dict vs set one will become sooooo ambiguous. It's already pretty rough as is though, not sure how plausible it is to parse that efficiently.
The OCaml syntax for arrays is [| 1, 2, 3 |] which I didn't add because it always seemed kind of meh as syntax. I like #[ 1, 2, 3 ] more now that I see them both, but I wonder if we can do a bit better.

So maybe the way forward is to get a small set of example programs that use dicts and sets and lists and arrays and try out variations of this idea til we are satisfied. Perhaps like with the tasks stuff, someone shares a gist, others make variations, we list them all up as an addendum to the initial issue.

What if it is somehow related to modules, kind of like #5 but not:
import Array as A
import Dict as D

div
  A.[ style D.{ "position" = "absolute" , "color" = "red" }
    , id "containerDiv"
    ]
  A.[ ul A.[]
        A.[ li A.[] A.[text "Hello"]
          , li A.[] A.[text "World"]
          ]
    ]

Haha, def looks worse to me! Okay, I'm gonna get back to components stuff now, but again, this seems quite interesting!

Here's one example, from elm-todomvc
Current version with linked lists:
view : Address Action -> Model -> Html
view address model =
    div
      [ class "todomvc-wrapper"
      , style [ ("visibility", "hidden") ]
      ]
      [ section
          [ id "todoapp" ]
          [ lazy2 taskEntry address model.field
          , lazy3 taskList address model.visibility model.tasks
          , lazy3 controls address model.visibility model.tasks
          ]
      , infoFooter
      ]

Current proposal: (colon replaced with equals)
view : Address Action -> Model -> Html
view address model =
    div
     #[ class "todomvc-wrapper"
      , style #{ "visibility" = "hidden" }
      ]
     #[ section
         #[ id "todoapp" ]
         #[ lazy2 taskEntry address model.field
          , lazy3 taskList address model.visibility model.tasks
          , lazy3 controls address model.visibility model.tasks
          ]
      , infoFooter
      ]

Great idea @TheSeamau5!
I think a good way to go for this would be to split it up into separate proposals for "Separate Syntax for Array Literals and List Literals" vs. others. I think the motivation for Array vs. List is pretty strong, because it's so common to want to use literals for those in DSLs.
The use case that comes immediately to mind is elm-html's element functions; I think it's safe to say the primary reason they accept a List of attributes over an Array of them is not that List is the best data structure to hold them (quite the opposite!) but rather because [attr1, attr2, attr3] is the nicest DSL to work with.
Every time I've wanted to construct a Dict or Set so far, it's been by starting with some other variable-length data structure and transforming it, so I'm actually not sure I'd even have used literals for them if they existed! That's not to say my experience generalizes, but rather that I'm not sure there's as strong a case to be made there.

It seems unfortunate to me that we are binding specific implementations of a listy things to syntax. As soon as we want a different 'listy' thing (a skip list, a doubly linked list, etc) we'll run into this problem again.
Might there be a way to create a listy thing and (optionally) assign it's implementation after?

@z5h That's the problem type classes solve, and we're not going there right now. Or, if we decide we need to, this conversation balloons.
@ everyone: Correct me if I'm wrong, but lists are much more performant than arrays. Arrays of less than 32 elements are basically single chunks of memory that must be copied entirely on write, and this is the use case for the vast majority of DSL uses. FP typically uses mapping instead of indexed traversal, because it works so well with immutability.
Or, when you're writing out a literal, you don't really need to index into it. And the HTML library doesn't care what data structure you're using since it just maps over it to get the complete set of attributes, children, or what have you. Sure, you might use indices to construct something dynamically, but then you're no longer writing literals. So I don't agree with Richard (at the moment) that lists are "far from" the best data structure for the HTML library.
Point is, I agree with Evan on code examples.

The module prefix doesn't look so bad to me @evancz. That's the direction I was thinking in too, reminiscent of one of the tasklet macro syntaxes with a record for, in this case, cons and nil or something like that.
You could also try a variation of the Haskell quasi-quoting again like we had for markdown and still do for glsl:
view : Address Action -> Model -> Html
view address model =
    div
     [A| class "todomvc-wrapper"
     , style {D| "visibility" = "hidden" |}
     |]
     [A| section
         [A| id "todoapp" |]
         [A| lazy2 taskEntry address model.field
         , lazy3 taskList address model.visibility model.tasks
         , lazy3 controls address model.visibility model.tasks
         |]
     , infoFooter
     |]

@mgold I didn't give the example of elm-html to claim that arrays were superior to lists. What I mean, is that there a number of cases where they are the better structure.
For example:  Infinite scrolling lists. You need a collection that can store all that information, send actions uniquely to the concerned element, and slice the collection appropriately to only display what is visible. Arrays beat lists on all these fronts.
While you can implement the described behaviors with Arrays today, you end up paying a needless conversion to lists. Why? It's completely unnecessary.
Again, in the particular context of elm-html, I am not advocating the change from lists and tuples to arrays and dicts. What I am pointing out is what @rtfeldman pointed out. The reason elm-html took the routes of lists and tuples is not due to a calm, thoughtful analysis of performance tradeoffs. It is uniquely because lists and tuples have associated literals. Arrays and Dicts do not. Currently, if you want to make a DSL, using Arrays or Dicts or Sets are out of the question, even if they would be ideal data structures for these DSLs.

@Apanatshka Sounds like an interesting idea to find a more general pattern, but also sounds super complex in the short term.
That said, I would be interested in reading a proposal that could explain how these generalized data structures would work and how one would be able to define their own custom data structures.

@evancz's module syntax looks intriguing to me to :+1: Swift features literal convertible protocols for transforming standard types into custom ones.

I think we should not make things implicit. This exists in Haskell for strings as OverloadedStrings, and it's always been super confusing when I use it. Apparently they have a mechanism to do this with lists as well (OverloadedLists). It's possible, there's a formula, but I don't think this is a nice place to be in many ways.
I am generally in agreement with @TheSeamau5 that this is going to get out of control if we try to solve every case, when in fact we have a very specific issue: whether it is elm-webgl or elm-html or Graphics.Element, arrays are faster to use and impose no real costs on people in an ideal world where the syntax is cheap.
Meta Note: To move this forward, share concrete examples. Maybe that is code, maybe that is scenarios that could come up for learners. Point is, concrete arguments or nothing. If you have an idea, share it as a gist with a bunch of examples, similar to how we did things here. We can collect all the ideas and add them in an addendum to the initial issue. If it is about personal opinions on things, it's not gonna help make this decision. I'll try to do better too :)

Are there languages besides Clojure that do this? If so, what syntax do they use?

Sorry to keep cluttering this issue, but I just remembered VLists which @rtfeldman told me about recently. It might be worthwhile to open another issue exploring this, or better yet, for someone to just make it as a community library.

Correct me if I'm wrong, but lists are much more performant than arrays. Arrays of less than 32 elements are basically single chunks of memory that must be copied entirely on write, and this is the use case for the vast majority of DSL uses. FP typically uses mapping instead of indexed traversal, because it works so well with immutability.

Unfortunately the coefficient for linked lists in JS is huge because each Cons cell is a fully-fledged JS Object instead of just a couple of pointers. :crying_cat_face:

Time to benchmark, then.

Just wanted to add I'm with @z5h on this one. Binding to one specific implementation seems too limiting.

I'm in the process of closing down this repo. This idea is cool. I have it in my personal set of "things to consider now". We will revisit it before 1.0

If you want to preserve this, move it to a gist. I will probably delete this repo in a couple days and salvage certain pieces (like the CONTRIBUTING page) for managing other repos and online venues.

Are there languages besides Clojure that do this? If so, what syntax do they use?

It's a bit late but since I saw this issue and nobody replied

Python doesn't have linked list literals (its list type is an array), its collection literals are "array" [1, 2, 3], set {1, 2, 3} and dict {1: 2, 3: 4, 5: 6}
Ruby has array literals ([1, 2, 3] and the perl-ish %w{a b c}), and two syntaxes for hash literals: {"a" => 1, "b" => 2} and {a: 1, b: 2}. In the second syntax keys are always symbols (so it's a shorthand to {:a => 1, :b => 2}
Erlang has list literals [1, 2, 3] and map literals #{a => 2, "a" => 1}, I don't think it has array or set literals
Ruby doesn't have vec or hashmap literals, though it has a vec![1, 2, 3] macro

The "module" option sounds similar to initializer lists (C/C++/C#), maybe desugared at compile-time so dictionaries can be source encoded as pair sequences or as proplists without  needing their own custom syntax?

Just to leave one use case on this zombie: representing JSON-style data.
While representing JSON with strong type systems is generally ... interesting, let's say we manage to define an Elm type for the JSON at hand. If it's not completely trivial, it will probably contain some instances of Dict (e.g. for capturing patternProperties keys, to speak in JSON schema terms). Now if we want to include such data statically in an Elm app (I guess that applies to non-JSON data as well, but that's my use case), what code do we use?
Going via Dict.fromList feels wrong, both w.r.t aesthetics and (potentially) efficiency.


tweaks to compile correctly on linux

Description
/fp:fast is a windows only build command and test.cpp has some characters that I stripped from it.
still does not quite fix the problem, I was able to compile DXC separately on my machine just not sure why it doesn't build within this project ...

[cmake] CMake Error at CMakeLists.txt:33 (project):
[cmake]   Generator
[cmake]
[cmake]     Ninja
[cmake]
[cmake]   does not support platform specification, but platform
[cmake]
[cmake]     x64
[cmake]
[cmake]   was specified.
[cmake]
[cmake]
[cmake] CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
[cmake] CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
[cmake] CMake Error at 3rdparty/dxc/CMakeLists.txt:79 (message):
[cmake]   -- No build type selected, default to Debug
[cmake]
[cmake]   -- Configuring incomplete, errors occurred!
[cmake]
[cmake]   See also
[cmake]   "/home/michaelpollind/projects/Nabla/build/3rdparty/dxc/build/CMakeFiles/CMakeOutput.log".
[cmake]
[cmake]
[cmake]
[cmake]
[cmake] -- Configuring incomplete, errors occurred!
[cmake] See also "/home/michaelpollind/projects/Nabla/build/CMakeFiles/CMakeOutput.log".
[proc] The command: /home/michaelpollind/opt/cmake-3.25.2-linux-x86_64/bin/cmake --no-warn-unused-cli -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -DCMAKE_BUILD_TYPE:STRING=Debug -DCMAKE_C_COMPILER:FILEPATH=/usr/bin/gcc -DCMAKE_CXX_COMPILER:FILEPATH=/usr/bin/g++ -S/home/michaelpollind/projects/Nabla -B/home/michaelpollind/projects/Nabla/build -G Ninja exited with code: 1

this is what I'm currently seeing.

@pollend are you planning on supporting GCC btw?

@pollend are you planning on supporting GCC btw?

Originally I was compiling with gnu.  https://godbolt.org/z/95hfq6j1G

the Other thing you need to do is disable embedded assets, there are some compiling problems when you use it with clang.

tried to compile one of the examples. looks like i'm missing a linking -latomic

pretty happy with these changes everything compiles except for embedded assets. something minor that can be addressed after the fact.
[build]         template<> const std::pair<const uint8_t*, size_t> get_resource<typename NBL_CORE_UNIQUE_STRING_LITERAL_TYPE("nbl/builtin/shader/loader/gltf/uv.frag")>()


Add the applicable solution for resolving macOS issues to the FAQ.md file

Page Link
https://github.com/kurosakishigure/katharsis/blob/canary/FAQ.md
What is the documentation issue?
The documentation is also missing another solution, which involves rebuilding from scratch.
Is there any context that might help us understand?
We should include this solution and guide users to the INSTALL.md file for detailed instructions.

This part of the content isn't important; we can consider marking it with >:
https://github.com/kurosakishigure/katharsis/blob/bf9e26438f4409a295b9d2fe95b923c6dd849b2a/FAQ.md?plain=1#L13


Extra blank page printed

We using printer plugin to print PDF. For some reason it always adding extra one page.
I have used all setting but it not working. Is there any specific setting i need to set in UIPrintInteractionController?
Thanks,
Muthu.S

@smuthu01 which printer(hardware) are you using?

We are using iOS10.x and using air print.
Devices are iPod and iPadmini.
Thanks,
Muthu.S

I set border false then i started to work properly.


How to render specific tile color?

I was curious how this can be done. using Render block seems most ideal but not sure, Is there another prop I can pass in?

Do you mean how to define a color theme or do you want to change the color of a single block?
For the first case this page should answer your question: Color themes.

Do you mean how to define a color theme or do you want to change the color of a single block? For the first case this page should answer your question: Color themes. In the second case renderBlock is probably best, yes.

I meant to change the color of titles randomly. I was also wondering if I could replace the tile with, say, an icon, like marking a tile on the calendar with a checkbox icon.

Yes, for these kind of things renderBlock is your best bet.


Feature: Use eth_sign for hardware wallets connected via onboard.js

Onboard.js fixed their bug for eth_sign for hardware wallets, now we can check if the connected wallet is a HW wallet and fallback to eth_sign
To be updated and merged when onboard.js releases a new version

Travis automatic deployment:
https://pr742--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr742--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr742--safereact.review.gnosisdev.com/app


Update README.md

Summary of the Pull Request
What is this about: Fixing some grammatical mistakes here and there in readme.md.

I think a lot of these are stylistic and some are absolutes like "we'll have all the install instructions" which isn't true. This is why we really want issues opened and a conversation to be had before work is done.


Fix ProgressPlugin for multi compiler caused by ES6 refactoring

What kind of change does this PR introduce?
refactoring bugfix

Did you add tests for your changes?
no

Summary
This PR fixes a ES6 refactoring error caused by 3e22f61bf1ebd04ad052cfb64fd62990a657ec6e

Does this PR introduce a breaking change?
no

Other information
arrow functions does not bind it's own arguments

Thanks

Published as v2.3.3


fix: Increase padding between elements in workflow template creator. …

Fixes #7309
Before:

After:

@simster7

@lstolcman I didn't see any difference in the above picture. Can you highlight it? Please fix the lint  error

A wider gap to easily distinguish which text imput belongs to what item (explained well in https://github.com/argoproj/argo-workflows/issues/7309).

Also, there's no linting errors, jsut some weird item:

@lstolcman thanks for screenshot. Please fix the lint failure.


RFC: User Defined Window Functions

Which issue does this PR close?
https://github.com/apache/arrow-datafusion/issues/5781
Rationale for this change
We would like to allow users to take full advantage of the power of DataFusion's window functions (largely contributed by @ozankabak and @mustafasrepo  👏 )
This PR contains a potential implementation of User Defined Window Functions: (the "Use existing APIs" approach described on https://github.com/apache/arrow-datafusion/issues/5781#issuecomment-1583105449)
I don't intend to merge this specific PR. Instead, if the community likes this basic approach I will break this PR up into pieces and incrementally merge it
What changes are included in this PR?
The new example in this PR shows how this works. Run
cargo run --example simple_udwf

Which produces the following output (where my_average's implementation is defined in simple_udwf.rs as a user defined window function):
+-------+-------+--------------------------+------------------------+---------------------+
| car   | speed | LAG(cars.speed,Int64(1)) | my_average(cars.speed) | time                |
+-------+-------+--------------------------+------------------------+---------------------+
| red   | 20.0  |                          | 20.0                   | 1996-04-12T12:05:03 |
| red   | 20.3  | 20.0                     | 20.15                  | 1996-04-12T12:05:04 |
| red   | 21.4  | 20.3                     | 20.85                  | 1996-04-12T12:05:05 |
| red   | 21.5  | 21.4                     | 21.45                  | 1996-04-12T12:05:06 |
| red   | 19.0  | 21.5                     | 20.25                  | 1996-04-12T12:05:07 |
| red   | 18.0  | 19.0                     | 18.5                   | 1996-04-12T12:05:08 |
| red   | 17.0  | 18.0                     | 17.5                   | 1996-04-12T12:05:09 |
| red   | 7.0   | 17.0                     | 12.0                   | 1996-04-12T12:05:10 |
| red   | 7.1   | 7.0                      | 7.05                   | 1996-04-12T12:05:11 |
| red   | 7.2   | 7.1                      | 7.15                   | 1996-04-12T12:05:12 |
| red   | 3.0   | 7.2                      | 5.1                    | 1996-04-12T12:05:13 |
| red   | 1.0   | 3.0                      | 2.0                    | 1996-04-12T12:05:14 |
| red   | 0.0   | 1.0                      | 0.5                    | 1996-04-12T12:05:15 |
| green | 10.0  |                          | 10.0                   | 1996-04-12T12:05:03 |
| green | 10.3  | 10.0                     | 10.15                  | 1996-04-12T12:05:04 |
| green | 10.4  | 10.3                     | 10.350000000000001     | 1996-04-12T12:05:05 |
| green | 10.5  | 10.4                     | 10.45                  | 1996-04-12T12:05:06 |
| green | 11.0  | 10.5                     | 10.75                  | 1996-04-12T12:05:07 |
| green | 12.0  | 11.0                     | 11.5                   | 1996-04-12T12:05:08 |
| green | 14.0  | 12.0                     | 13.0                   | 1996-04-12T12:05:09 |
| green | 15.0  | 14.0                     | 14.5                   | 1996-04-12T12:05:10 |
| green | 15.1  | 15.0                     | 15.05                  | 1996-04-12T12:05:11 |
| green | 15.2  | 15.1                     | 15.149999999999999     | 1996-04-12T12:05:12 |
| green | 8.0   | 15.2                     | 11.6                   | 1996-04-12T12:05:13 |
| green | 2.0   | 8.0                      | 5.0                    | 1996-04-12T12:05:14 |
+-------+-------+--------------------------+------------------------+---------------------+

Here are the major changes in this PR

Move PartitionEvaluator definition into datafusion_expr (much like the Accumulator trait for AggregateUDFs)
Moved WindowAggState, WindwFrameContext and some related structures to datafusion_expr (so the UDWF did not depend on datafusion-physical-expr
Traitify the built in state so WindowUDF did not depend on datafusion-physical-expr

Open questions:
I think it may be possible to simplify the PartitionEvaluator  to remove the state management which would make the needed changes (the amount of code that needs to be moved to datafusion_expr) smaller. I will try to do this as a separate PR
Outstanding cleanups
I found a place where the optimizer special cases a particular window function which I think I can remove (and I will try to do so as separate PR (https://github.com/apache/arrow-datafusion/pull/6619)
https://github.com/apache/arrow-datafusion/blob/1af846bd8de387ce7a6e61a2008917a7610b9a7b/datafusion/core/src/physical_plan/windows/mod.rs#L254-L257

Thanks for putting this together. @mustafasrepo will take a look and we will discuss in our next meeting, then we will circle back to you with our thoughts.

@stuartcarnie  / @ozankabak / @mustafasrepo  / @doki23  as you have expressed interest in the user defined window functionality I wonder if you have some time to review the proposed API for doing so

Thanks @alamb for this proposal. I have examined this PR. Overall it is very well written, and showcases the usage of the new API. However, I have some concerns about flexibility of WindowUDF and AggregateUDF. Specifically, user can implement multiple different WindowUDFs. However, all implementations are initialized from WindowUDFExpr (There is a similar pattern in AggregateUDF. All AggregateUDFs are initialized from AggregateFunctionExpr).
BuiltInWindowFunctionExpr (WindowUDFExpr implements this trait) contains methods such as  supports_bounded_execution, uses_window_frame, include_rank. These methods effect which methods will be used during evaluator. For instance if user wants to add a evaluator that uses window frame boundaries, user should be able to set flag uses_window_frame for the evaluator (Hence when WindowUDFExpr is implemented, these fields are fixed, and users have no control over them).
Hence I think, we should either support for user to create custom BuiltInWindowFunctionExpr for each new WindowUDF (I think this is hard and more cumbersome on the user side) or we should move all the parameters that effects the decision (such as supports_bounded_execution, uses_window_frame, include_rank) to the evaluator and/or aggregator side. By this way user will have full control on the  custom implementation.
I am working on a new design to solve this problem. @metesynnada and I will discuss it tomorrow, and let you know about the final result.

Hence I think, we should either support for user to create custom BuiltInWindowFunctionExpr for each new WindowUDF (I think this is hard and more cumbersome on the user side) or we should move all the parameters that effects the decision (such as supports_bounded_execution, uses_window_frame, include_rank) to the evaluator and/or aggregator side. By this way user will have full control on the custom implementation.

thank you for the response @mustafasrepo  -- this makes sense to me. I agree it would be better if the type system could help enforce functions that needed to be implemented
Are you thinking something like this?
pub trait Accumulator: Send + Sync + Debug {
    // Required methods
    fn state(&self) -> Result<Vec<ScalarValue, Global>, DataFusionError>;
    fn update_batch(
        &mut self,
        values: &[Arc<dyn Array + 'static>]
    ) -> Result<(), DataFusionError>;
    fn merge_batch(
        &mut self,
        states: &[Arc<dyn Array + 'static>]
    ) -> Result<(), DataFusionError>;
    fn evaluate(&self) -> Result<ScalarValue, DataFusionError>;
    fn size(&self) -> usize;
}

/// A trait for accumualtors that also provide the ability to remove rows from a window incrementally
pub trait RetractableAccumulator: Accumulator {.  // <---- add this trait
    // Provided method
    fn retract_batch(
        &mut self,
        _values: &[Arc<dyn Array + 'static>]
    ) -> Result<(), DataFusionError> { ... }
}

And then adding a separate function to AggregateUDF:
enum AccumulatorImpl {.     // <---- add this enum
  NonRetractable(Box<dyn Accumulator + 'static>
  /// returned if the accumulator supports retractable windows
  Retractable(Box<dyn RetractableAccumulator + 'static>),
}

pub struct AggregateUDF {
    pub name: ...,
    pub signature: ...,
    pub return_type: ...
    // change this signature ----+v
    pub accumulator: Arc<dyn Fn(&DataType) -> Result<AccumulatorImpl>, DataFusionError> + Sync + Send + 'static>,
    pub state_type: ...
}

?
Also related is https://github.com/apache/arrow-datafusion/issues/6611 we found (which is one of the features of the Accumulator that is not yet exposed by AggregateUDF)

I pushed d531d50 that refines the API somewhat and makes it possible to call evaluate and evaluate_with_bounds
My plan for this PR is to

complete the example (mostly done)
write tests for the various window function apis.
Port all documentation changes upstream incremntally

Once we have settled on the final API (pending the feedback from @mustafasrepo and @metesynnada, on https://github.com/apache/arrow-datafusion/pull/6617#issuecomment-1587430709)  I'll then make one or more PRs to master with that API and the examples / tests.

We had a meeting on this today. @mustafasrepo and @metesynnada will update you with their progress soon (tomorrow possibly).

Hence I think, we should either support for user to create custom BuiltInWindowFunctionExpr for each new WindowUDF (I think this is hard and more cumbersome on the user side) or we should move all the parameters that effects the decision (such as supports_bounded_execution, uses_window_frame, include_rank) to the evaluator and/or aggregator side. By this way user will have full control on the custom implementation.

How about moving supports_bounded_execution and uses_window_frame into the evaluator?

In the PR I have implemented my proposals. As a summary, I think on top of current approach, if we combine evaluate and evaluate_stateful fields. User can accomplish desired behavior by setting appropriate flags and implementing corresponding evaluation (either evaluate or evaluate_all) method.
I will open the PR that unify evaluate and evaluate_stateful fields on the main repo once it is ready.

How about moving supports_bounded_execution and uses_window_frame into the evaluator?

I think this is a good idea @doki23, and  I think it is broadly speaking what @mustafasrepo  has proposed in https://github.com/alamb/arrow-datafusion/pull/12

In the https://github.com/alamb/arrow-datafusion/pull/12 I have implemented my proposals.

TLDR I really like the proposal. I left my comments here:  https://github.com/alamb/arrow-datafusion/pull/12#pullrequestreview-1479108753

Update here is that the final PR (that has all the learnings from this PR and various cleanups we did along the way) is ready for review: https://github.com/apache/arrow-datafusion/pull/6703


Add continuous integration

add Travis CI
start Camunda pre-test
stop Camunda post-test

Closed via 6bfebeda32afde20af648c0bd1bc24be4b0b9c30.


'React/RCTBridgeModule.h' file not found

After strictly following the installation steps on the Readme, and then attempting to build, I get this preprocessing error:
'React/RCTBridgeModule.h' file not found

In file included from .../node_modules/@bugfender/rn-bugfender/ios/RNBugfender.m:1:

This react native project is ejected from expo, using Expo SDK v32.0.0, and the latest version of BugfenderSDK.framework

Hi @vananreed,
I see you are using ExpoKit. We created the RN plugin following the official docs for vanilla RN.
Fortunately, in the docs of ExpoKit they mention this issue:

If the library doesn't support CocoaPods, react-native link may fail to include the library's header files. If you encounter build issues locating the <React/*> headers, you may need to manually add Pods/Headers/Public to the Header Search Paths configuration for your native dependency in Xcode. If you're not familiar with Xcode, search Xcode help for "configure build settings" to get an idea of how those work. Header Search Paths is one such build setting. The target you care to configure is the one created by react-native link inside your Xcode project. You'll want to determine the relative path from your library to Pods/Headers/Public.

Could you please follow the steps in the official docs of Expo and let us know how it worked? We might add some lines about this to our docs as well.
I hope it helps!
Side note: the last time we checked, it was not possible to make RNBugfender compatible with Cocoapods due to changes in RN that were happening at that moment. We had in our roadmap to make compatible in a near future.

We were able to resolve this with the above mentioned header search paths setting, and we skipped the installation instruction copied from the iOS project regarding the "Bugfender.swift" snippet.
It's a bit confusing in the RN setup instructions as the RN project ejected from expo is written in Objective-C, so while a bridging header was created and we could import the framework there to make available to Swift, at the end none of it is needed since we're doing all our logging in RN anyway.
Our header search paths on the "RNBugfender.xcodeproj" target look like the following (see screenshot).  We're referencing the React managed via Cocoapods, setup by ExpoKit itself.
"$(SRCROOT)/../../../../ios/Pods/Headers/Public"
"$(SRCROOT)/../../../../ios/Pods/Headers/Public/React"


fix database #241

#241 Database Views Do Not Display
https://github.com/stephenou/fruitionsite/issues/218

Thanks for the fix. Here are further improvements from my end:
A. Light/Dark Mode Toggle
The toggle for light/dark mode was broken. As a fix, we should add the following code snippet after this line
if (!__console.isEnabled) __console.enable();
__console.enable();

B. Reduce Errors in Network Tab
Some errors still crop up in the network tab. Further improvements will be to:

Add an A or CNAME record exp. that points to the domain for your notion site (can be properly documented),
Rewrite the hostname on this line with the function below

function rewriteHostName(hostname) {
  if (hostname.startsWith("exp.")) {
    return hostname.replace(MY_DOMAIN, 'notion.so');
  }

  return 'www.notion.so';
}

Thanks for the fix. Here are further improvements from my end:
A. Light/Dark Mode Toggle The toggle for light/dark mode was broken. As a fix, we should add the following code snippet after this line
if (!__console.isEnabled) __console.enable();

B. Reduce Errors in Network Tab Some errors still crop up in the network tab. Further improvements will be to:
* Add an `A` or `CNAME` record `exp.` that points to the domain for your notion site (can be properly documented),

* Rewrite the hostname on this [line](https://github.com/stephenou/fruitionsite/blob/master/worker.js#L87) with the function below

function rewriteHostName(hostname) {
  if (hostname.startsWith("exp.")) {
    return hostname.replace(MY_DOMAIN, 'notion.so');
  }

  return 'www.notion.so';
}

Thank you for your suggestion. In my rust code, I directly set up localStorage.__console = true;
I prefer to use the subdomain name of notion.site, which only requires one replacement. The disadvantage is that a variable needs to be added, such as stephenou.notion.site like it code
    MY_DOMAIN = "stephenou.notion.site"
    response = new Response(
      body
       .replace(/'${MY_DOMAIN}'/g, NOTION_DOMAIN)

i tried modifying my code by using this https://github.com/stephenou/fruitionsite/pull/243/files, still cant see the databases. can see red lines under page heading and description. is this due to that? can you help me out pl

You saved my life

Hello ! I can't do it myself
I'm a bit of a code whiz but I'd like to be able to do it. Can anyone help me?


There's a very high change to return a NSFW Exception, Is there any option to disable it?

https://github.com/replicate/cog-stable-diffusion/blob/main/predict.py#L118 for NSFW detection

very bad NSFW detection

bad NSFW detection. I'm not an advocate of NSFW images at all, but sometimes it filters out even safe images and that's a pitty because it costs money for nothing. Please add option/param/API to deactivate it. We should leave the decision to the user.


AU selected and no collection selected possible after "select collection dialog"

Testserver: qa-imeji.mpdl.mpg.de
Test date: 28.09.2017
Device: Tablet Nexus 9 Android 5.1.1.
User: not yet logged in
Actions:
log in user -> automatic upload inactive and "selected collection none" -> activate upload -> "Please select a collection" -> "ok" -> "Remote collections setting" (without having selected a collection)
Observation:
Menue -> "Automatic upload" active but no collection selected
Expected result: it is not possible
having AU activated and no collection selected after the info "please select a collection"-> "ok"- dialog  or
leaving menue "select collection" without actually selecting a collection.

Testserver: qa-imeji.mpdl.mpg./rest
Test date: 11.10.2017
Device: Tablet Nexus 9 Android 5.1.1.
User: standard user
Result: ok, if user goes through described process, with leaving "remote collections settings", "automatic upload" becomes deactivated.
but still activated by default, see #105


Changed update_conponents to use a union-find datastructure

I changed update_components to use a fast union-find disjoint set implementation with wait-free and path-compression properties.
I benchmark a dataset with 5M vertices in texrecon at 30% less total runtime than with the old implementation.
I'm happy to work on issues/problems you might find.

Hi,
thanks for your work! I don't see any problems with the licence of Wenzel Jakob's work and all the tests run fine -- will merge it today.


Describe config.yaml syntax more prominently

A significant number of the different configuration options are described in their respective documents only in terms of command-line argument passed to the rke2 binary, significantly burying the lede on being able to configure the server using the same keys and values in config.yaml.
Given that the quickstart install directions lead you to a configuration where changing the server's command line arguments is rather not straightforward,  I'd wager that the config.yaml method is probably far more useful to most people following those directions, and information on how to do this needs to be a lot more prominent than it currently is, buried between several other less-relevant sections.
If nothing else, the section describing the equivalence between command-line arguments and config.yaml keys should be made a lot more prominent and discoverable. But I'd even argue that the config.yaml syntax should probably be the primary way of describing the configuration options themselves, with the command-line syntax only secondary.
For instance, take this paragraph:

To enable Multus, pass multus as the first value to the --cni flag, followed by the name of the plugin you want to use alongside Multus (or none if you will provide your own default plugin). Note that multus must always be in the first position of the list. For example, to use Multus with canal as the default plugin you could specify --cni=multus,canal or --cni=multus --cni=canal.

IMO this would probably be more useful if paragraphs like that were rewritten something along the lines of:

To enable Multus, add multus as the first list entry in the cni config key, followed by the name of the plugin you want to use alongside Multus (or none if you will provide your own default plugin). Note that multus must always be in the first position of the list. For example, to use Multus with canal as the default plugin you could specify:
# /etc/rancher/rke2/config.yaml
---
cni:
- multus
- canal

This can also be specified with command-line arguments, i.e. --cni=multus,canal or --cni=multus --cni=canal.

That paragraph above is actually specifically the one I just spent two days stumped on before finally figuring it out.
Just to describe my own experience here:
I had RKE2 set up and installed from the quickstart guide. I knew that I needed Multus for my use-case, and was able to find the Using Multus section, and therefore "knew" that I somehow needed to get --cni=multus,canal passed as a command-line argument... and then spent the next two days going down dead ends, reinstalling RKE2 half a dozen times, even just reading the RKE2 source code (e.g. to see if there was some undocumented option that would get the https://get.rke2.io installer script to append --cni=multus,canal to the ExecStart line in the rke2-server.service systemd unit file it created). It wasn't until near the end of day 2 combing through the docs and reading every bit on every page that I finally found what I needed with the config.yaml file.

If this is something of interest, I don't mind doing most of the leg-work to add a config file example for each CLI argument example -- but I did want to open the issue here first in case this isn't a change the maintainers would accept.

@AJMansfield Thanks for point this out, this problem is certainly relevant. I'll take a pass at rewording the docs and pushing config yaml values over flags.


coinmarketcap cors issue

XMLHttpRequest cannot load https://api.coinmarketcap.com/v1/ticker/?_dc=1493933013556. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://freewallet.io' is therefore not allowed access.

I added a proxy to call this API server-side and avoid this error now. Thought it might solve my inability to send from freewallet but still unable to.
Maybe contact CMC to see about their CORS same-origin policy.

Tried countertools wallet to test since i think similar APIs used. here is console screenshot. if I have real time to look into it, i'll follow up.

I believe there are 2 issues going on
1.) Coinmarketcap is using a new API and depreciated the old one (CORS issue)
2.) Issue Broadcasting signed transaction via chain.so
The issue is coinmarketcap changed their API without notifying anyone.
The new API is at : https://api.coinmarketcap.com/v1/ticker/bitcoin/
I have fixed the issue on counterpartychain.io to use the new API... just need to update freewallet now
The issue with broadcasting a signed tx will need to be fixed... I might just setup an API on xchain.io to allow pushing signed transactions.
Thanks for reporting these issues, they will get fixed shortly (within next few weeks)

no prob. yeah I mixed two issues here. should have added separate issue.
On Sat, May 6, 2017 at 10:38 AM J-Dog<EMAIL_ADDRESS>wrote:

I believe there are 2 issues going on
1.) Coinmarketcap is using a new API and depreciated the old one (CORS
issue)
2.) Issue Broadcasting signed transaction via chain.so
The issue is coinmarketcap changed their API without notifying anyone.
The new API is at : https://api.coinmarketcap.com/v1/ticker/bitcoin/
I have fixed the issue on counterpartychain.io to use the new API... just
need to update freewallet now
The issue with broadcasting a signed tx will need to be fixed... I might
just setup an API on xchain.io to allow pushing signed transactions.
Thanks for reporting these issues, they will get fixed shortly (within
next few weeks)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jdogresorg/FreeWallet/issues/8#issuecomment-299644018,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAassb7kIYUr22pvNusBgaBCg7a6RD5ks5r3IX-gaJpZM4NRPuq
.

closed with c76a21984abc3282781dab2a3a5d0785fd0cc155 which uses xchain.io/blocktrail.com for most requets, with failover to blockr.io and chain.so

not sure if connected, but for future references, CMC doesn't allow client-side js calls. Check the doc here https://coinmarketcap.com/api/documentation/v1/#section/Quick-Start-Guide


Consider live crypto portfolio updates (1m)

Using binance / kucoin websocket we could constantly update the portfolio message (check max rate) to display profit on coins and current price

We could update it every 1m, but is it really necessary?

Could use
https://www.kucoin.com/_api/currency/v2/prices?base=USD&lang=en_US&targets= for getting all USD worth of listed tickers on KuCoin

This is not scalable and will cause the discord rate limit to be hit


Added Is Symmetric Tree

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Added Is Symmetric Tree
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#6464

@HarshCasper Sir kindly review the pr and let me know if there is any mistake that i need to correct.

@HarshCasper  Sir, I have made changes accordingly , please see to it. Thank you.

@HarshCasper Sir kindly review the pr and let me know if there is any mistake that i need to correct.

Sir mad e changes accordingly.


Upgrade to GHC 8.10.7

I have updated haskell.nix to the latest version, in order to acquire the latest stackage LTS (18.13).
Update stackage LTS to 18.13.
Update from GHC 8.10.5 to 8.10.7.
Regenerate Nix expressions

Issue Number
ADP-1205

I wrote a bit of a log of my experiences for next time: https://github.com/input-output-hk/cardano-wallet/wiki/Notes-from-upgrading-GHC-version


Update and expand README.md as appropriate for a public repository

The readme should provide a description of what it is, its purpose and function, etc. We do not need to provide release instructions. It looks like the readme was written before the repo became public.

We want an overall description of what gist is and how to use it. Pull text from the website.
Don't include bundling documentation and release management - this goes into a separate document.

Could be the same or similar to a revised version of the website page - see issue #369

@sa-bpelakh A quick approach to this would be to copy what I put on the front page of the wiki. It's not optimal, but it's better than what we have now.


Indian number formatting broken

See https://github.com/JavaMoney/jsr354-ri/issues/275

Based on https://github.com/JavaMoney/jsr354-ri-bp/blob/master/src/test/java/org/javamoney/moneta/format/MonetaryFormatsTest.java it seems parsing works, but formatting doesn't.


[Ballista] Support to access remote object store, like HDFS, S3, etc

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

After introducing the object store API, to support to access remote object store for Ballista executors, there are still some gap. For example, as #349 mentioned, it's not able to get file metadata for remote object store.
Describe the solution you'd like

Our workaround is to make the file path self described. For example, a local file path should be file://tmp/..., a hdfs file path should hdfs://localhost:xxx:/tmp/...

@yahoNanJing fyi @seddonm1 and I have been working on https://github.com/datafusion-contrib/datafusion-objectstore-s3. Still early stages but would be great to have more use cases that we could steer development with.

This PR would support this use case: https://github.com/apache/arrow-datafusion/pull/1677

Hi @matthewmturner, thanks for your info. It's really helpful example for creating another independent crate, like datafusion-objectstore-hdfs. However, it would better to avoid new object store registration for cases like providing sql service without any places for users to do manually registration. And it's not a good way to do the registration for each sql or each session, since it's quite stable for the data sources the service provides for.

This PR would support this use case: https://github.com/apache/arrow-datafusion/pull/1677
Thanks @thinkharderdev. I think it's better for the path to be self-described, like s3://hostname:port/xxx/... or hdfs://hostname:port/xxx/.... Then the object store management service can detect which remote object store will be used.

@yahoNanJing there actually already is a hdfs extension here https://github.com/datafusion-contrib/datafusion-hdfs-native.
Understood on your point - in that case what would be your ideal API for accessing S3? I would have thought the service could register the ObjectStore to share with clients and clients could access after registration at service level had been completed.

Hi @matthewmturner and @thinkharderdev, what I'm thinking is to introduce some remote object store extensions as Datafusion features. These extensions will be inclusive crates and are managed independently so that this solution will not introduce additional maintenance effort for Datafusion.
However, this solution is blocked by #1772. After the datasource module is splitted into an independent crate. Then the remote object store extensions will be able to just depend on that crate. In other places, we can setup whether to enable the extension features or not. By this way, we can avoid cyclic dependency.
Further to say, for the default object stores in the ObjectStoreRegistry, maybe better to make it configurable so that we can avoid remote object store registration manually.


Clean up FP6-LLM

Remove original FP6 quantization code (qtorch and C++ bit-packing)
Replace FP32<->FP6 dtype conversion with @vkuzo's implementation for MX dtypes

I also migrate some of my FP32->FP6 rounding test cases to MX custom cast test.

Replace FP32<->FP6 dtype conversion with @vkuzo's implementation for MX dtypes

I have two questions:

Just curious, have you benchmarked the performance of this change?  I have not optimized the mx types for performance yet.
I think it's not ideal to depend on code in prototype folder, if other people need custom_cast.py it might be good to move it to a common place outside of the prototype folder.

@vkuzo

I did benchmarking. IIRC, your implementation is faster than mine for both CPU and GPU (w/ torch.compile). Will benchmark it again and update the result in this comment. (I also did correctness comparison. Your implementation and mine are bit-identical for all FP16 bit patterns - can't test all FP32 bit patterns because it would take too long)
Yea I want to discuss with you about this also. It will be good to move it to a separate file for low bit-width floating point conversion. The FP6-LLM author added support for FP5 E2M2, so I want to support FP5 also (mx doesn't have FP5).

A question for @msaroufim. Is there a guideline when we should or should not decorate a function with @torch.compile? These functions rely on torch.compile to be fast, but there are cold-start (first time is slow) and dynamic shape (can we avoid re-compile on different shapes, in a guaranteed way?) problems. (haven't looked into the problems too much yet, maybe it is not that significant).

So for Windows the main issue is torch.compile() codegenerates triton kernels which hasn't prioritized Windows support. I think for inductor cpu backend this should be less of an issue, I suspect there might be an overly aggressive assert somewhere though.
I would say overall everything should be compilable, the cold start problems is indeed annoying and is actively being worked, there are some broader plans that have been shared though https://dev-discuss.pytorch.org/t/how-to-bring-compile-time-down-to-zero-our-plans-and-direction-may-14th-edition/2089
Regarding dynamic shapes the way I iterate through things is first eliminate graph breaks then recompilations, this has been my goto guide https://github.com/pytorch/pytorch/blob/main/docs/source/torch.compiler_troubleshooting.rst
Also just FYI we removed the requirement to have branches up to date before merge, there was a breaking change in PyTorch that was just reverted so please rebase your changes to get rid of CI flakes


Feature/traefik

Status
Type
Env Vars Change

:white_check_mark: Ready
Feature
No

Description

This PR adds a basic configuration to run vocascan-server with Traefik
Motivation and Context

You are now able to use a reverse proxy with a domain and ssl encryption
Screenshots / GIFs (if appropriate):

Checklist

[x] I have read the CONTRIBUTING document.
[x] I have considered the accessibility of my changes (i.e. did I add proper content descriptions to images, or run my changes with talkback enabled?)
[x] I have documented my code if needed

Resolves

could you please fix the indentation?


Support PR reviews

Hello there,
Thanks for the great project! We're running into a potential good use of github-pullrequest-resource to not only set PR statuses with comments, but also to submit Github reviews with associated in-line comments.
This would allow a more deeply integrated feedback loop when pipelines produce the required information on a line by line, file by file basis. A good example would be a code linter which would produce errors and their files and locations, then used to submit a PR review with in-line comments.
What are your thoughts? Thanks.

This sounds like a new resource.

A review exists only in the context of a pull request. While I’m not opposed to the idea of it being a separate resource, I’d like to understand your particular train of thought.

Originally this resource was to solve the TravisCI behaviour, which I would like to keep. This addition seems like away to avoid an API call from the pipeline maintainer perspective. When you'd already have to convert your output to something the resource could support and then convert into API calls. Remove the middle-person.

I'm trying to make this resource do less. Not just become a converter of one format to an API call. I know it does a bunch of that already, but I'm hesitant to keep adding it in. As it means more error handling, interfaces to extra API calls, etc.
I'm going to close this issue on those thoughts. If you can propose an interface that requires minimal effort on the resource's part and your's (to convert the output of your code linter to something the resource understands) I'm all ears.


Added simple example

Had some difficult figuring this one out, as this is so straight forward and is the simplest use case, maybe should be included at the top of the docs ?

thank you.  What samples you'd like to add would be great


ci: run tests on macos-latest

Using github actions  with macos-latest configuration.
Similar to #34.
Follow-up to #25

Pull Request Test Coverage Report for Build 214285405

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.6%) to 66.56%

Totals

Change from base Build 212807048:
-1.6%

Covered Lines:
830

Relevant Lines:
1247

💛  - Coveralls

Pull Request Test Coverage Report for Build 214285405

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.6%) to 66.56%

Totals

Change from base Build 212807048:
-1.6%

Covered Lines:
830

Relevant Lines:
1247

💛  - Coveralls


Use proper path separators.

Thanks, so it didn't even load the emmylua on linux/ubuntu? I vaguely remember about path but was too lazy to use it

Correct. The path that was being added before: /home/ck/.vscode-server/extensions/ketho.wow-api-0.7.2\EmmyLua\API


Create interface in the active folder

DTDL: Create Interface always places the new file in the root.
Is it possible to create the new file in the current active folder?

Yes, the current behavior is that if there is any active workspace (opened folder), the new interface file will be created in this folder. Otherwise, the command will ask user to select a folder.

I meant the active subfolder inside the workspace. Note how sample2.json is always created in the root folder although the active folder was selected.

Since DTDL extension has been transferred to Ilie's team, add @iluican for awareness.


Retry the polling when guest agent not ready

Closes https://github.com/dmacvicar/terraform-provider-libvirt/issues/1037

What is needed to get this merged and a release cut with it?

@rgl @thequailman do we know what change made this necessary?

It seems to be fine in 0.7.1, I can dig into it later tonight.


Zero or more path query is broken with some datasets

We've just upgraded our software from using Sesame 2.7.15 to rdf4j 2.1.2 and found that property paths (https://www.w3.org/TR/sparql11-property-paths/#path-language) that use * do not work with some datasets. I had trouble building up a dataset from scratch to reproduce the issue (it seemed to work on smaller datasets) so instead took the larger dataset that we did have that was showing the issue and have attached that (after scrambling some IP out of it). There are screen captures showing the steps to reproduce the issue and the dataset and queries.
We have been able to work around the issue by re-writing the query using a UNION clause, but have now found another query that doesn't seem to work, even after doing that. At this point we can't upgrade as we are not confident that the queries will continue to work correctly as the underlying dataset changes.
rdf4j-bug.zip

Hmmm, the tests Jeen added when I was having issues with property paths myself years ago only include +, not *:
https://github.com/eclipse/rdf4j/blob/master/testsuites/sparql/src/main/java/org/eclipse/rdf4j/query/parser/sparql/ComplexSPARQLQueryTest.java#L1469

Oddly, I can reproduce the problem when executing the query from the RDF4J workbench, but not when executing directly from code (on a SailRepository). So it appears the SPARQL engine itself works as expected, but something is going wrong in communicating the query to the engine.

Disregard my previous comment, made a mistake in trying to duplicate the issue. I can reproduce from the workbench as well as directly from code.

It appears to be related to how the query planner optimizes the joins. This is why the problem is hard to reproduce with a smaller dataset.

Fast work guys! I'll pick up 2.1.4 when released and retest. Is it possible to download the current build before then to test it?

You can grab a nightly snapshot build, though keep in mind that that is from our development branch, so it may also include some changes that will not be part of the next patch release.

Hi Jeen,
We've tested the updated jar file and it seems to have fixed the issue! Nice! I noticed that this issue was scheduled for milestone build 2.1.4. Is this going to be an official released package as we were hoping to pick it up when it was closed off, or do we need to wait for 2.2.0 in late January?

I'm way ahead of you. Patch release 2.1.4 is now available for download, this fix is part of it. I'm writing the release announcement e-mail as we speak :)

Nice work! Thanks for getting a package out so quickly!

I know this has been closed but I'm still experiencing tha same issue. It's been driving me mad for some time now. I have a more complex model but I narrowed it down to this.
During execution of my Java code, I print the  query which returns no results when executed via RDF4J. However, if I execute the same query using a third party tool such as Topbraid Composer, I get the desired results. I upgraded to 2.1.4 but it didn't seem to have made a difference for me.
My dataset is not very large, a couple of thousand triples only. It happens when checking for rdfs:subClassOf*.
Any ideas?


Misc org update

@dannys42 We may want to publish a new version (or at least the tag) for this fix.


multiple series with single threshold

when my query was resulting 1 metrics, it was working good with threshold and tooltips and graphs shows good.
now change my query to generate 2 metrics and using regular expression in "Apply to metrics", everything seems normal except few things

I want threshold based on single metric not both metrics as both metrics data is different.
graph is okay to generate both metrics but based on requirement seems better.

Version 0.5.0

You need to create 2 rules for that if definition is different for each

it works thanks, but I was looking if it can have threshold based on ID in a single rule then it will reduce the number of rules and easy to manage


Website Homepage UX Optimization

Navigating the different wallet selections available was rather clunky in that the differing image sizes in the gallery caused the selection buttons and the screen to jump around when cycling through them instead of remaining static and consistent

Here's an example of a screen record of that happening:
https://drive.google.com/file/d/1TWD8qARVSFNtgTjoO0vqRspYcqD-mG3x/view?usp=drivesdk

There's a significantly large extra margin of empty space on the right side of the screen that the user can scroll off page towards that I see has no functional purpose and merely distracts from the intended experience scrolling through the content (that margin of empty space is basically almost half the size of the screen, contains no content and basically shows the main page being cut off)

Here's an example of a screen record of that:
https://drive.google.com/file/d/1TllpvQ5h-UpUlx6YOMNiloH5mgSt4RC1/view?usp=drivesdk

Fixed on the latest version :)


general way to create a config from a pytorch model already created

Hi, excellent contribution, I saw it win a kaggle competition some months ago . I was wondering if there is any general way to create a config file from a pytorch model I have already created?

Hi @MrRobot2211 ,
Could you please be more specific?

Yes I have the pytorch architecture for a given model,, witten down in
code   or saved in a checkpoint, is there a tool to put it in mmdetector's
config file format?
El jue., 12 mar. 2020 a las 12:22, Wenwei Zhang<EMAIL_ADDRESS>escribió:

Hi @MrRobot2211 https://github.com/MrRobot2211 ,
Could you please be more specific?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/open-mmlab/mmdetection/issues/2242#issuecomment-598248205,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFTMVKAO5D6AX45G7M7FVSTRHD437ANCNFSM4LF4HNMA
.

--
φ

There is no general way since we do not know what other architectures will look like. You can just follow existing models to put different components to backbones, heads, etc, and then write config files.


Propose extending core optimizer and create_hooks to support Horovod

In the style of SyncReplicasOptimizer, using Horovod for managing the syncing of gradients requires both wrapping the optimizer and a custom hook, see the example here: https://github.com/uber/horovod
Including the custom hook could be accomplished by overriding train_hooks in T2TModel,
e.g.
import horovod as hvd

class Img2imgTransformerHVD(Img2imgTransformer):

  @staticmethod
  def train_hooks():
    """Add Horovod hook to perform gradiant sync."""
    # Should be added conditionally depending on hparams.

    hvd.init()
    return [hvd.BroadcastGlobalVariablesHook(0)]

But it would be awkward to need to subclass every model you might want to use in this way.
Alternatively, the additional train hook could be added in create_hooks based on whether --use_hvd is specified.
With this approach, use_hvd can also be passed to optimize  in the same manner as use_tpu to determine whether to wrap the optimizer with hvd.DistributedOptimizer.
def optimize(loss, learning_rate, hparams, use_tpu=False, use_hvd=False):
  """Minimize loss."""

  ...

  opt = ConditionalOptimizer(hparams.optimizer, learning_rate, hparams, use_tpu)
  if use_tpu:
    opt = tf.contrib.tpu.CrossShardOptimizer(opt)
  elif use_hvd:
    opt = hvd.DistributedOptimizer(opt)

/cc @rsepassi

I'd be interested in seeing this happen as well.  I'm currently attempting to benchmark the transformer on 32 gpus across 4 machines, ie. 8 gpus each.

@cwbeitel Have you tried to implement it locally?
I just started to experiment with t2t. The code kinda looks confusing as I'm trying to figure out where all to modify to add the horovod functionality.
Any pointers would be helpful.

No but the two approaches I gave above should be good starting points if you want to give it a try!
The question I meant to raise by opening the issue was whether @rsepassi @lukaszkaiser et al. would consider this change to t2t in-scope given I was proposing making a change to the core (second case) instead of something in t2t_usr_dir (first case). If you want to continue making use of t2t improvements into the future it's probably best to incorporate a core change into the main repository instead of having to resolve merges every time you want to be up to date.

Hi @cwbeitel @vsuthichai I have integrated Horovod v0.13.11 with tensor2tensor v1.8.0 (tensorflow v1.10) and managed to run distributed training over RoCE plus GPUDirect RDMA.
However, when I re-run the the same distributed training task using plain old tensor2tensor v1.2.9 (tensorflow v1.4), it surprisingly achieves almost the same performance as the NCCL over RoCE one. The point is I didn't even specify any extra environment variables (NCCL and MPI mca parameters, etc.) like I did to run NCCL over RoCE. I kind of doubt if this is a normal case as I recall that to run tensorflow over RDMA + GDR, there are usually some prerequisites like this and this.
Have you compared the performance difference between the Horovod task and its original tensor2tensor one?
BTW, the tensor2tensor distributed training appears to be really confusing and neither the new document nor the old document  are clear enough, I have to somehow tweak the worker/master parameter to get it work using tensor2tensor v1.2.9 (tensorflow v1.4). Down below is the command I use.
Does this really trigger the distributed async training as expected? If so, does it mean that the gRPC backend of Tensorflow is smart enough to automatically detect the Mellanox NIC and set up the RoCE and GDR all correctly without any manual configuration?   I've shelved
the parameter server for now because I didn't see significant throughput increase compared to Horovod.  However, I did spend over a week trying to get the parameter server technique up and working with sync training, before and after the distributed training documentation updates.  I couldn't get it running successfully, until I changed the job name from master to chief within the cluster environment json and an extra code fix to start tf.train.Server manually for the chief.

@EdwardZhang88 @vsuthichai That's cool you both got distributed training working! I've only done synchronous training not async or using Horovod so can't help there.
One sticking point I ran into with distributed training (that is explicitly mentioned in the docs now) is that all your jobs need to share the same checkpoint dir.
I'm going to close this issue since we haven't heard a response. This discussion would probably be a great fit for the gitter channel or the mailing list!


Use itoa when formatting content-length

And perhaps generate a magic string to format small lengths without any allocation at all, like we do for status codes?

And perhaps generate a magic string to format small lengths without any allocation at all, like we do for status codes?

This perf investigation was done when adding benchmarks.
Usage of itoa / an itoa Buffer pool would also be beneficial for content-length chunks in HTTP/1.1


Add debug pins on signal lines

At least for the next few revisions, debug pins should be put on the signal lines for various parts of the board. While soldering on the MCU pin lead wires is kind of fun and looks cool, really slows down the process and introduces a ton of surface area for error. Remove them once the board is ready for a full test.

I've gone back and forth on this about ten times now and decided it's not worth it given they'll only add signal integrity issues.

The ethernet SPI lines work, those are already tested.
The i2c lines for the indicator lights already work, those are already tested.
The SWD lines "work" (ST did not do a great job implementing them) and a programmer is already designed and will be assembled at the same time as the link board. Plus, worst case, I can always solder to the Oro pad as it gives enough space.
The only debug lines left are the OLED SPI signals, which are NOT rated for 5v line the arduino pumps out, and I don't have any signal translators on hand (I could get some but.. why) but ARE rated for the MCU's voltages (as far as I can understand the very, very poor documentation). If this doesn't work, I'm going to resort to i2c and just... eat the framerate drop, I guess.
All of the power lines and signal lines have THTs associated with them already between the button and the power jack, and the power converter has been tested oodles of times at this point and is known to work.

Other than that, there are no other debug THTs that were marked for population, thus they have all been removed.

The SWD lines "work" (ST did not do a great job implementing them) and a programmer is already designed and will be assembled at the same time as the link board. Plus, worst case, I can always solder to the Oro pad as it gives enough space.

What’s the difference between using SWD line and using the programmer?


[SMALLFIX] User static import for standard test utilities in CapacityTest

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17527/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17528/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17529/
Test PASSed.


create sample level metadata for irtex data

No count data though so maybe not worth it now. closing.


[Figma Refactoring][Badge] Colors & Fonts not aligned

I realised that we have following differences:

Colors don't match (much brighter in Storybook)

There are major accessibility issues at the moment with the colors used on Storybook (white font on warning-3 background e.g. only has a contrast ratio of 1.43:1 and fails). In this case I strongly recommend to adapt the colors according to our Figma.


Get CPU temperature working in Netgear R6300V2 running Merlin 380.70

CPU temperature is not working on Netgear R6300V2 router running Merlin 380.70
https://github.com/Vaskivskyi/asusrouter/blob/b07d47ea5e5a61f64f530d677b82d5ceacbad62e/asusrouter/const.py#L244
^Above does not match what I have from my router's /ajax_coretmp.asp:
curr_coreTmp_2_raw = "44°C";
curr_coreTmp_2 = (curr_coreTmp_2_raw.indexOf("disabled") > 0 ? 0 : curr_coreTmp_2_raw.replace("°C", ""));
curr_coreTmp_5_raw = "47°C";
curr_coreTmp_5 = (curr_coreTmp_5_raw.indexOf("disabled") > 0 ? 0 : curr_coreTmp_5_raw.replace("°C", ""));
curr_coreTmp_cpu = "58";

It's actually curr_coreTmp_cpu rather than curr_cpuTemp
I have not checked whether this applies to: (a) stock Asus, or (b) Merlin specific or (c) customization specifically made for Netgear devices. Although (c) seems highly unlikely.
Thanks for your awesome ha integration anyway!

@Vaskivskyi I wonder if you have time to take a look? Let me know if any help is needed

Hey, @richardzone,
First of all, I am really sorry for the late reply - I didn't see a notification about the opened issue.
Thanks for your feedback! 👍🏻
PR #102 is adding this functionality, so it will be available in the HA integration with the next version. Either by the end of this week or next week.


Azure Stream Analytics auto-created sink table in Azure SQL with "Change Data Capture" has a too-wide key

Azure Stream Analytics needs a better choice for the key column in an auto-created table
I have the following tech stack:

SQL Server on-prem running 2019
Change Data Capture (CDC) is enabled on one table for all columns
Debezium is connecting that to Kafka for transport to AEH
Azure Event Hubs (AEH) is my message broker for all changes
Using Azure Stream Analytics (ASA) to stream those changes to my sink
Azure SQL Server is my sink

Issue layer from the aforementioned list:
Azure Streaming Analytics
Experience
CDC is working beautifully, as is the Debezium+Kafka piece. For the data structure in the sink, I opted to allow for ASA to create the table structure with me providing just the name. It turns out that the key it chose to use is a field called "Schema" with a data type of NVARCHAR(8000); that can be an issue, and would trigger a warning when this type of field is used as the key to a table, as it leaves room for error when the key being inserted is too wide for a clustered or non-clustered index, which allow 900 and 1,700 bytes respectively. My very first record had a key of 4,780 bytes, and the table structure displayed in the schema column is tiny, at just 4 columns. My solution was to drop the index, add a surrogate key as the column which was used was the only unique field, then to recreate the index using my new field.
Proposed idea for a solution
When the table gets auto-created based on a data set, and the key field provided is too wide for a clustered key, to use a surrogate key. I'm simply suggesting what I used for my solution, though you guys may have a better idea on how to mitigate this potential issue.

Hi @eligoldner - Thank you for sharing your feedback! Since this channel is for driving improvements towards MS Docs, could you provide us with a specific URL of the MS Doc that your feedback is most applicable to? That way, we're able to connect you with the right team that is best suited to address your feedback.

It's not a docs issue, it's an issue with an application of yours. If this
is just for docs, can you please direct me to the correct venue for
reporting "features" in MS applications?
Thanks,
Eli
On Tue, Jun 8, 2021 at 12:33 AM Mike Urnun @.***> wrote:

Hi @eligoldner https://github.com/eligoldner - Thank you for sharing
your feedback! Since this channel is for driving improvements towards MS
Docs, could you provide us with a specific URL of the MS Doc that your
feedback is most applicable to? That way, we're able to connect you with
the right team that is best suited to address your feedback.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/76518#issuecomment-856431204,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF2PHTNXYS3SHUF3W2GQYLTRWMQ3ANCNFSM46IOKHOQ
.

@eligoldner Ok, if you make a new post on MS Q&A channel and one of our engineers will review the feature/scenario and loop in the product team if need be.
If you'd like to propose a feature directly to the product team, you can make a new post on Azure UserVoice.

Since we didn't determine any changes to any of MS Docs upon reviewing this feedback, we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

Please stop sending me message Im trying to close my account

From: eligoldner @.>
Sent: Tuesday, June 8, 2021 8:45:50 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Subscribed @.***>
Subject: Re: [MicrosoftDocs/azure-docs] Azure Stream Analytics auto-created sink table in Azure SQL with "Change Data Capture" has a too-wide key (#76518)
It's not a docs issue, it's an issue with an application of yours. If this
is just for docs, can you please direct me to the correct venue for
reporting "features" in MS applications?
Thanks,
Eli
On Tue, Jun 8, 2021 at 12:33 AM Mike Urnun @.***> wrote:

Hi @eligoldner https://github.com/eligoldner - Thank you for sharing
your feedback! Since this channel is for driving improvements towards MS
Docs, could you provide us with a specific URL of the MS Doc that your
feedback is most applicable to? That way, we're able to connect you with
the right team that is best suited to address your feedback.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/76518#issuecomment-856431204,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF2PHTNXYS3SHUF3W2GQYLTRWMQ3ANCNFSM46IOKHOQ
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/76518#issuecomment-856734493, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASH6YZR2I6YDAURVGWRLHE3TRYGH5ANCNFSM46IOKHOQ.