visualRecognition. detectFaces method doesn't recognise key locally

According to the visualRecognition documentation there is a method called detectFaces which takes in similar parameters as the .classify() method.
const fs = require('fs'),
    VisualRecognitionV3 = require('watson-developer-cloud/visual-recognition/v3'),
    visualRecognition = new VisualRecognitionV3({
        api_key: '[MY_REAL_API_KEY]',
        version: '2016-05-20'
    });

const params = {
  images_file: fs.createReadStream('./peter.jpg')
};

visualRecognition.detectFaces(params, (err, res) => {
  if (err) {
    console.log(err);
  } else {
    console.log(JSON.stringify(res, null, 2));
  }
});

The classify works good when I send over a picture and returns JSON alright, when I try to use the .detectFaces() with the same parameters I get an error:
~/Git/watson-developer-cloud  node watsonImageRecognition.js
{
  "status": "ERROR",
  "statusInfo": "invalid-api-key-permissions"
}

Am I missing something in the parameters to run this locally?
I tried exporting in the environment variable VISUAL_RECOGNITION_API_KEY=[MY_REAL_API_KEY]
but that did nothing also. It works fine with classify() just the detectFaces does nothing and asks for the API_KEY... any thoughts?

Ok, after a bit of investigation I think the problem is not with the module code but with the api route the module is calling. A CURL to the same url does not recognise the the api_key parameter in the call. This may be a problem with the route itself then

Hi Peter, are you having any trouble with this demo in particular or are you just having a problem with the visual recognition node SDK? If you are not having any issues with this Visual Recognition demo please post your issue here https://github.com/watson-developer-cloud/node-sdk. Thank you!

Hey Kyle, turns out the API was not playing ball when I testing my app. The facial-recognition route was just not taking in the API_Key, but it looks like it has been sorted out and now it's returning some fine JSON results. Thanks for reaching out.


Creates a TextInput component

Why:

There's a few components that are common enough that we can define
ourselves, namely related with forms. Moreover that will eventually give
us the chance to create form wrappers that abstract over repeated
complexity of implementing said forms.
Related to msaraiva/surface#32

This change addresses the need by:

Creating a new namespace Form

Adding a very simple TextInput component which tries to mimic
Phoenix.HTML.Form. There are a couple of issues with different
levels of priority that could be addressed before finalizing this:

There is no way to pass a "blob" of properties along. This would be
needed, because we can't/don't want to set a property in the component
for every property the html element supports. (msaraiva/surface#2)
It would be nice to be able to only set an attribute if the
property is non empty. This is particularly interesting for non
required properties. For example, if no class attribute is passed,
it still sets class="".

This might be a moot problem after number 1 is solved, as we can
simply not set class as property.

@msaraiva This is just an example to understand if this is what you had in mind. Also, dynamic props would really help here. Not sure if that's something to be worked on before hand.

Hi @zamith!
Thanks a lot for working on this ❤

There is no way to pass a "blob" of properties along. This would be
needed, because we can't/don't want to set a property in the component
for every property the html element supports. (#2)

For the <TextInput> and friends we just want to wrap the existing Phoenix functions so in our case we have text_input(form, field, opts \\ []) where, according to the docs, "All given options are forwarded to the underlying input"
So we could have something like:
defmodule Surface.Components.Form.TextInput do
  use Surface.Component

  import Phoenix.HTML.Form

  property form, :string
  property field, :string
  property value, :string, default: ""
  property class, :css_class
  property opts, :keyword, default: []

  context get form, from: Form, as: :context_form
  context get field, from: Field, as: :context_field

  def render(assigns) do
    form = assigns[:form] || assigns[:contex_form]
    field = assigns[:field] || assigns[:contex_field]

    ~H"""
    {{
      text_input(
        String.to_atom(form),
        field,
        [value: @value, class: @class] ++ @opts
      )
    }}
    """
  end
end

A couple of remarks:

We don't have a type :keyword yet, however, it's just a matter of adding it to this list
The golden rule to decide if we should create a specific property for an option is:

Do we need to customize it? e.g., add synctatic sugar like we do for css classes), if so, we create it, otherwise, we just forward as it is to the phoenix helper.

It would be nice to be able to only set an attribute if the
property is non empty. This is particularly interesting for non
required properties. For example, if no class attribute is passed,
it still sets class="".

I totally agree. We do this already for boolean attributes. We should extend this behaviour when working with regular HTML tags. Feel free to open an issue for it so we can address it later.

@msaraiva updated with the opts and added some tests.
I didn't add context yet because we still don't have form and field as existing components. This works as a standalone, albeit without any context of what surrounds it.
My idea was to merge these building blocks and then work on bringing them together, but I can add a form component in this PR if you think that's the way to go.

I didn't add context yet because we still don't have form and field as existing components. This works as a standalone, albeit without any context of what surrounds it.
My idea was to merge these building blocks and then work on bringing them together, but I can add a form component in this PR if you think that's the way to go.

No problem. We can add <Form> next.

I realize that the way I'm testing adds yet another warning about an alias not being used, although it is being used but Elixir can't know about it. Is this a problem? How do we get rid of it?

Can you send more details? What alias and what message?

Ah sorry, since it happens with other aliases as well, I assumed you were aware.
warning: unused alias TextInput
  test/components/form/text_input_test.exs:4

Which I actually need, otherwise I get:
warning: cannot render <TextInput> (module TextInput could not be loaded)
  code:1: Surface.Components.Form.TextInputTest.TestLiveView_16419.render/1

I don't know enough about Elixir's compiler to know for sure, but I believe it can't understand that the alias is being used in the test, because it is only evaluated later.

because it is only evaluated later on the Code.eval on render_live

That's right. The compiler has no idea it will be used at runtime. So you have 2 options:

Remove the alias and use the full name:

<Surface.Components.Form.TextInput form="user" ...>

Use the alias after defining it:

alias Surface.Components.Form.TextInput; TextInput

I would probably go with #2 but feel free to go with #1 if you prefer :)

#2 is fine. Will seem a bit weird for other people, maybe. But it seems to be the least intrusive.

Removed all of the compiler errors if that's ok. :)

warning: unused alias TextInput
test/components/form/text_input_test.exs:4

I also ran into this recently and got by it by telling alias to not warn.
alias Surface.Components.Form.TextInput, warn: false

Not sure I like this either but it might be more clear than
alias Surface.Components.Form.TextInput; TextInput

@mathewdgardner I wasn't aware of warn: false. I think it's better since it makes our intention explicit. Thanks!
@zamith feel free to add @doc to each property. There's no need to get fancy, just a short description is fine. You can even copy/paste from input_text docs if you want. After that, I think we can merge it 🎉

@zamith thanks a lot! ❤


How do I configure tests to run in babel stage=0

I've modified cfg/dev.js to support stage 0 features of babel:
config.module.loaders.push({
  test: /\.(js|jsx)$/,
  loader: 'react-hot!babel-loader?stage=0',
  include: path.join(__dirname, '/../src')
});

then I added a stage 0 feature such as class properties to src/components.Main.js
myProp=42;
render(){
...
}

https://github.com/jeffmo/es-class-static-properties-and-fields
hit npm start, and everything works fine.
I've noticed  cfg/test.js has two babel loaders (the second is for test coverage?)
I tried to add stage=0 for the first one and for both as a query string, using the two options that are suggested here (query string and query property):
https://github.com/babel/babel-loader/tree/v5.3.2#options
either way, running npm test fails with syntax error where the stage 0 code is.
any idea how to solve this?
Thanks!

@yakirn fwiw,there is a related issue targeting the update from babel 5 to 6 in the repository that features the template used in this generator. not sure if they are related, but it might be interesting either way weblogixx/react-webpack-template/issues/11.

Hi @yakirn,
currently we are still using babel 5, so there may be some stuff that is currently not working.
I currently wait for https://github.com/deepsweet/isparta-loader to update the isparta dependency for coverage reports. If the update is done, it should work as intended.

@weblogixx @sthzg Thank you for your answers.
So, if I understand correctly, isparta-loader is just about code coverage?
if removing isparta will make my tests work without coverage, it is good enough for me, at least until a real fix/update will be available. can you instruct me on how to do this?
Thanks again

@yakirn,
yes, it should work when you disable the isparta-loading and update to babel 6. I have all parts ready but unfortunately still have to wait for isparta-loader to be updated. If this is not happening in the nearer future, I will have to disable it (or fork the project, the change that needs to be done is just updating the package.json).

@yakirn: The generator was just updated to use babel 6. It should work with the new version without problems. Note: Only newly created projects (after updating the generator) will be build with babel 6.

Ok, I managed to run the tests after I disabled isparta.
@weblogixx fyi, I updated the generator and installed in a new dir, and got this error while trying to run npm test:
ERROR in ./src/components/Main.js
Module build failed: ReferenceError: [BABEL] src/components/Main.js: Unknown option: babelrc.presets
    at Logger.error (node_modules/isparta/node_modules/babel-core/lib/transformation/file/logger.js:58:11)

this time I didn't change anything. it's the pure generator's code. npm start works.
but I guess thats a different issue. should I close this one?

@yakin:
Yes, the new loaders author has choosen to "update" his isparta loader with the old isparta config for versions 0.2.x. So it ran when it was 0.2.2, but no more for 0.2.3 :(. A new version of the template will be available in a couple of minutes, the dependency is fixed then. Just update your generator in about half an hour to make it work correctly.

Think that should be done with the update. Closing this. If there are any other errors, feel free to open a new ticket.


⚠ GitLab Server has degraded performance

In 04c7b96, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6613 ms

Resolved: GitLab Server performance has improved in 592e86d.


Update RSpec guidelines to place happy path before edge cases

This allows someone reading the spec for the first time to understand
what the single responsibility of the subject under test is, before
reading about the edge cases below.

:+1:


Supporting for user functions, fixes #140

This allows the registration of functions via JavaScript that can be called in queries. I have a little left to do to handle errors, but I wanted to check that this is on the right track before continuing. I've not worked with C++ in a long time and this is the first that I've worked with v8, so I did my best.
It takes the approach of calling on the Node.js thread, so this would certainly have a huge impact on performance (though in some cases that may not matter). The suggestion in #140 to create a new v8 isolate and transfer the function there seems like a far better way to make this work, but is way beyond what I'd be able to do.
This seems like a start at least.

So the last thing that I wanted to figure out in this is how to know when a function call has produced an exception.

heads up @wbyoung: I disabled travis builds for pull requests on this repo because the OS X builds were hanging and blocking the queue for other builds. Sorry about the hassle. /cc @rclark

@kkaefer I had trouble enabling C++11 features to get std::future/std::promise working, so I just stuck with uv_cond and am now ensuring that the work has been completed to properly handle spurious wakeups.
Each function call is now being addd to a queue & the queue is being processed on the main thread. I'm not sure whether this is necessary or not. The SQLite3 docs don't really specify how callbacks are invoked with regards to threading. I'm not sure whether SQLite3 will make multiple simultaneous calls to a callback registered with sqlite3_create_function, but for some reason I seem to think that it wouldn't. The current implementation should work regardless & be future proof.
The failures with iojs on Linux seem to be addressed, so I assume that was related to the improper use of uv_async_init and/or not handling the spurious wakeups.
I'd still be interested in any advice on how to know when a function call has produced an exception.

@springmeyer thanks for the heads up. It's hard to test all those configurations, but I probably shouldn't have pushed each of those commits either. I didn't really realize until later that each build was taking about an hour to complete.

@wbyoung nothing to worry about on your end - it was just tough timing with travis being slow.

@springmeyer let me know if you're able to re-enable at any point or any other ideas to test all configurations. I can't enable Travis on my own fork because of the use of sudo.

A faster version of this with some issues can be found at wbyoung/node-sqlite3 user-functions-isolate that creates a new v8 isolate.

@wbyoung interesting; what implications does the use of isolates have? Can the user provided functions still access other parts of the script?

@kkaefer as of right now, that branch won't even let you require anything or even use console from within a user provided function because node::CreateEnvironment has not been called. So you only have pure javascript. If I have some time, I can dig a little deeper and try to make that work, but it may be a bit challenging to get right as I'm still feeling quite unfamiliar with the inner workings of Node & v8.
Anyway, even if it it were loading the node environment, v8 stipulates that you can't access objects from a different isolate. The function has been stringified and copied to the new isolate, so the following would produce a ReferenceError: outofscope is not defined:
var outofscope = 0;
db.registerFunction('MY_FUNC', function(value) {
  return value + outofscope;
});

I'm torn between which approach seems better.
dispatch to default loop
✓ Function executes in expected environment. Closures & scoping work as expected.
✗ Slower. When a user registers a custom function, I don't think they'd expect that it could run hundreds of times & interrupt the default loop that should be focusing on more important things (perhaps processing incoming HTTP requests).
v8 isolate
✓ Faster.
✗ The function executes in an unexpected environment. This could have consequences that wouldn't necessarily be obvious or clear when looking at the code. For instance, requiring modules would re-load them since you'd have a new require cache. Basically, it'd be like that one function was executing in its own little world.

@kkaefer an update on isolates…
wbyoung/node-sqlie3@f8cd018 creates an isolate for a user defined function. It does not load the node environment.
wbyoung/node-sqlite3@79756a4 actually loads the node environment (and loads of functions from a separate file). At this point, I'm pretty convinced that a separate v8 isolate is a bad idea.

Without loading the node environment, you're pretty limited in what you can do (pure JS only, no node stuff).
When you load the node environment, you'll need to bring all the assumptions that node brings with it. Correct me if I'm wrong, but I believe Node.js makes the assumption that all JS code run will run on a single thread. To avoid contention with the uv_default_loop, we could create a separate uv_loop_t and start it running in a separate thread. The node environment setup would allow that. But then we'd have to dispatch all work to that loop so we didn't break assumptions for how Node.js was architected (pure v8 would allow JS execution from multiple threads w/ locks, though). At that point, the performance would be similar to simply dispatching to the uv_default_loop, but with the advantage that it wouldn't be competing with activity in the the main node isolate. I don't think that's a convincing enough reason to pursue this.

The point to drive home is that if what you want is performance, you should create a user defined function via a native extension. Most who are looking for performance are probably going to look to another database anyway. If you're not looking for performance, then it's okay to take the performance hit that comes with dispatching to the default loop & executing the function in the original node isolate. Plus, there'd be no weird scoping issues.
At this point, I'm hoping that this pull request may be sufficient to be incorporated into the project.
Thanks for the tips so far!

Thanks for all the exploratory work; very insightful and informative.
As for using isolates, what we could do is the following: Instead of using the libuv work pool, all SQlite functions could be run in a custom thread that we spawn. This thread could also run a uv_loop_t, and load a node environment. Instead of registering custom functions in code, the user specifies a root entry file/module that is getting loaded in that thread, and all exported JS functions are available as SQLite functions. This means that users can't modify the custom SQLite functions on the fly from the original node file, but that may not be a common use case anyway.

var db = new sqlite3.Database(':memory:', done);
db.loadEnvironment('./functions.js');

db.all('SELECT FOO() AS txt', function(err, rows) {
    console.warn(rows);
});

functions.js:
exports.FOO = function() {
    return 'bar';
};

wbyoung/node-sqlite3@c24fc5e loads all functions from a single file.

@kkaefer by running all SQLite functions, you mean that all queries would be run on a single thread rather than being run as they are right now (in a threadpool via uv_queue_work)?
Wouldn't that approach mean that all queries would potentially block each other?
Currently, as I'm understanding the code, all sqlite_* operations are within a Work_<Type> function that's initiated via a call to uv_queue_work and then the result is processed (and callbacks made) via the uv_default_loop in a Work_After<Type>. Since uv_queue_work uses a threadpool, none of the operations will block one another (up to the thread limit imposed by libuv). This seems consistent with your latest comment as well, @kkaefer, so I hope I've understood correctly.

Ramblings/Thoughts
The way that I'm reading the SQLite3 docs, it seems that it has a few different thread modes. If my understanding above is correct, then I'm also led to believe that this library simply assumes that the SQLite library it's linking to has at least multi-thread if not serialized (the default). What I'm getting at is that adding the ability to switch modes into one operating only in a single thread could allow node-sqlite3 to also work with SQLite libraries that are operating in single-thread mode.
So the following would be how the library worked:

Operate in multi-threaded mode (using uv_queue_work)
Switch to single-threaded mode when sqlite3_threadsafe returns false.
Switch to single-threaded mode when using db.loadEnvironment()

I'm not convinced that the tradeoffs are worth it. With the pull request as it is (dispatching user functions to the default loop), only queries that are using user defined functions are affected. Running all queries on a single thread would affect all queries even if you want to add a user function that only runs occasionally.
Also, the SQLite3 docs doesn't actually specify on which thread user defined function callbacks are made. I think it's safe to assume that it's the same thread as the one on which the statement was executed, but I just wanted to throw that out there. Please correct me if I've overlooked something in the docs.
I actually hope that I'm wrong on all of this and that there's a way to make it work in its own isolate. Regardless, the back and forth with you, @kkaefer has given me a far better understanding of Node & v8, so thanks!

A single SQLite database can never be used from more than one thread at a time (unless in serialized mode, but that one is, well, serialized), so giving each Database handle its own thread wouldn't slow down performance. In fact, the current code may produce SQLITE_BUSY errors quite often if used incorrectly (we're technically violating SQLite's requirement about not using the same connection in more than one thread). When you call .serialize(), we're manually serializing all requests to a database connection, and this should be the default mode anyway. In reality, I doubt that this module will become slower when assigning each database handle its own thread.

@kkaefer ok, I was somehow confused into thinking that multiple queries could run at once for any given connection, but the SQLite3 docs are clear that's not possible.
Just to be sure we're on the same page (since this could be a lot of work to get working), the process of opening a connection (Database::New or Database::Work_BeginOpen) would start the worker thread & loop. Then all methods would be refactored to run their initial work Work_<Type> on that thread / in that loop.
I'll give it a shot if I have some time.

Hi Guys, When can we expect this to be merged in master ?
@wbyoung  i tried your branch user-functions, but it gives me error Object #<Database> has no method 'registerFunction' , am i missing something ?

@suchitpuri try using revision 3db73b9.
I haven't had time to work on this and am not sure if/when I will again.

Hi @kkaefer, I'm curious what is the main blocking point for this work? I would really like to see the user-function feature added to node's sqlite. I could take over the code if @wbyoung doesn't have time to finish it.

Hi,
any update on this?
Regards

@SamuelBrucksch I re-did most of the work in this branch in #1200. Still waiting for a review though.

@kewde any way this merge can be closed? #1200 that replaced it was closed out by its maintainer due to code fixes being unmanageable due to it being outdated. Thanks


美好生活

Describe the bug
A clear and concise description of what the bug is.
Provide more information
Running on EKS, AKS, GKE, Minikube, Rancher, OpenShift? Number of Nodes? CNI?
To Reproduce
Steps to reproduce the behavior:

Run kubeshark <command> ...
Click on '...'
Scroll down to '...'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Logs
Upload logs:

Run the kubeshark command with --set dump-logs=true (e.g kubeshark tap --set dump-logs=true)
Try to reproduce the issue
CTRL+C on terminal tab which runs kubeshark
Upload the logs zip file from ~/.kubeshark/kubeshark_logs_**.zip

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. macOS]
Web Browser: [e.g. Google Chrome]

Additional context
Add any other context about the problem here.

秀儿，是你吗

Closing because of inactivity and incomplete issue description.


spanner DatabaseAdminClient::CreateBackup() is erroneously labelled as non-idempotent

DatabaseAdminClient::CreateBackup() is treated as non-idempotent, but it will return StatusCode::kAlreadyExists given an existing backup_id, so transient errors could indeed be retried without risk.
For what it's worth, DatabaseAdminClient::RestoreDatabase() seems to fall into the same category.
DatabaseAdminClient::CreateDatabase() isn't documented either way, but it is also treated as non-idempotent.

Note that changing an operation from non-idempotent to idempotent means that callers should be prepared to see "already exists" returns from create operations, and "does not exist" returns from delete operations, due to retries after attempts that actually completed but failed to report an OK status all the way back.

This is related to https://github.com/googleapis/google-cloud-cpp-spanner/issues/512
Does Spanner guarantee something about CreateBackup wrt kAlreadyExists? I thought this function was funny in that it seemed like it should be idempotent, but there was something weird on the server side that made it not actually idempotent. Sorry I don't recall more.

Does Spanner guarantee something about CreateBackup wrt kAlreadyExists?

This is only how I imagine it must work, but I'll say yes ... just like open(O_CREAT | O_EXCL) does with EEXIST.

I thought this function was funny in that it seemed like it should be idempotent, but there was something weird on the server side that made it not actually idempotent.

I'll claim that it is not actually idempotent ... repeated applications do not have the same affect.  The question here, given the guarantee above, it whether the library should retry transient errors, and then expect callers to treat kAlreadyExists as success.
My latest thought on that is "no, it should not", as the retry would be indistinguishable from a racing CreateBackup, and now that the requested backup has attributes like version_time and encryption_config, treating kAlreadyExists as success would be wrong (at least without then verifying those attributes).
That is, I now disagree with the "transient errors could indeed be retried without risk" in the problem statement, and I think this should be closed.

Does Spanner guarantee something about CreateBackup wrt kAlreadyExists?

This is only how I imagine it must work, but I'll say yes ... just like open(O_CREAT | O_EXCL) does with EEXIST.

I thought this function was funny in that it seemed like it should be idempotent, but there was something weird on the server side that made it not actually idempotent.

I'll claim that it is not actually idempotent ... repeated applications do not have the same affect.

The only definition of idempotent that I found in a reputable book [1] is "A set of replicated messages is idempotent if the effect of receiving more than one copy of a message is the same as receiving only a single copy" [emphasis mine].  Could you provide an example where multiple replicated copies of CreateBackup() do not have the same effect as a single request?

My latest thought on that is "no, it should not", as the retry would be indistinguishable from a racing CreateBackup,

The archetype (at least in my mind) of an idempotent message is "set the value of X to Y" (where X is an existing record).  If one has to consider racing messages to "set the value of X to Y1"  vs. "set the value of X to Y2" then idempotence would not be a very useful concept.

and now that the requested backup has attributes like version_time and encryption_config, treating kAlreadyExists as success would be wrong (at least without then verifying those attributes).

I think that analysis is not quite right.  Or I completely misunderstood something.

That is, I now disagree with the "transient errors could indeed be retried without risk" in the problem statement, and I think this should be closed.

I disagree, but maybe you had something in mind that I am missing.

Ping.  What are the next steps here?

An example of an operation that could have been retried but was not.  I do agree that if retried this could have returned kNotFound, and the test would need to be prepared for such a return value.
https://pantheon.corp.google.com/cloud-build/builds;region=us-east1/3962c5fb-bb4f-42f5-a792-d3c997875b36;step=3?project=cloud-cpp-testing-resources
2021-09-23T08:11:55.004918381Z [DEBUG] <140600243849280> DropDatabase() << database: "projects/cloud-cpp-testing-resources/instances/test-instance-mr-0/databases/db-2021-09-23-ak88fh-gfwigt8uq"  (./google/cloud/internal/log_wrapper.h:73)
2021-09-23T08:11:55.005916111Z [DEBUG] <140600243849280> DropDatabase() >> status=GOAWAY received [UNAVAILABLE] (./google/cloud/internal/log_wrapper.h:75)

Close it for now.


useMachine logs warning about reading initial state from uninitialized service

Description
In 4.7.0-rc4 an warning is logged to the console when calling useMachine.
Expected Result
No warning.
Actual Result
Warning: Attempted to read initial state from uninitialized service 'toggle'. Make sure the service is started first.
Reproduction
https://codesandbox.io/s/xstate-react-template-yo5ui?expanddevtools=1&fontsize=14
Additional context
Occurs on 4.7.0-rc4 but did not on 4.6.7.

@davidkpiano is this warning even useful there? It just proxies to machine.initialState which is pure.

is this warning even useful there?

It is not, I'm going to be reworking this so that the warning no longer occurs.


Improve stubs of classes that have some method copied by create_proxy_methods

In most cases a type only superclass can avoid repeating the definitions in one or more classes.
Examples: Session|ScopedSession|AsyncSession; Connection|AsyncConnection

@CaselIT since I would be happy to see this in the future, I'm open to open a PR for discussion.
Let me try to summarize my current understand with an example:
scoped_session copies a few methods from Session. Therefore we should extract these methods in a super class and use it in both scoped_session and Session?

Thanks for the interest, @kasium. I've already started working on this locally, I'll try to open a PR shortly.
I'll tag you once I've opened it

Oh great. Thanks a lot 😄

Added #149  @kasium

great, thanks


Disallow users from deleting themselves

We didn't display this in the web UI, and we shouldn't have the API
loophole if we're serious about folks not being able to disappear on
their own.
See #621

I wanted to point out that self-deletion is possible by an administrator as an edge case, I'm not certain that we care about at this time but it is a possibility.

Corner case even if we closed the direct edge case: admin creates another account, promotes that other account to admin, deletes the original account. Superusers are slippery ;)


Is there a way to make Serde generate camelCase fields by default?

I am aware that I can use #[serde(rename_all = "camelCase")] but I don't want to do this for all structs. Besides there are dynamic/anonymous Value objects that I am using.
camelCase is one of the most widely used cases and it will help greatly for us (and everyone) when replacing their backends from other technologies (especially from ones like Spring Boot that use camelCase by default)
In short, is there a way to make Serde generate camelCase fields by default?

up.

+1 up.

You could convert the data from camelCase to snake_case before deserializing it with serde.
Or you could use a macro that adds a serde(rename_all = "camelCase") attribute and then delegates to serde's derives.


No license file

Would it be possible to add a LICENSE file to this repo?

This is presently blocking some work downstream: https://github.com/conda-forge/texlab-feedstock/pull/10
Happy to PR something, if @Vurich would like to propose the copyright string...

The license is specified in the Cargo.toml as MIT, although most of the code I write nowadays is under the Unlicense so if you need it to be relicensed I can do that. Is the file necessary if the license is specified as metadata?

Thanks for the super quick response!
A number of "distro"-style package managers don't really care much if the
wording of license XYZ doesn't require distributing the text of the
license... But packagers still have to "tick the box" when code gets
vendored/linked. It would super help if cargo "enforced" this, but can't
fix that now!
For texlab, which recently adopted crunchy, that's 100 or so crates, and
I'm down to just this package and one other.
So: no relicensing needed, just a clarification, in the repo, of the rights
a user (or in my case, redistributor!) has (or had, whenever the last
version went out).

Ok sure, I’ll do that. I actually didn’t mean to close this issue, I’ll reopen it until I get around to adding the license file (which will just be whenever I’m next at my laptop).

Does this solve the issue? https://github.com/eira-fransham/crunchy/pull/7

Thanks, that solves the most important part for me - the only thing that's technically still missing is publishing a new version of this crate to crates.io so the change actually propagates to users of the crate. I can work around that for my use case, but other people my not be able to do that and would rely on a new version to be published.


[Auto-generated] Check & update dependencies (pyproject.toml)

Update dependencies (pyproject.toml)
Automatically created PR based on the 'CI - Check pyproject.toml dependencies' workflow in SINTEF/ci-cd.

Codecov Report

Merging #108 (991b402) into ci/dependency-updates (2e0929c) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                  Coverage Diff                   @@
##           ci/dependency-updates     #108   +/-   ##
======================================================
  Coverage                  53.69%   53.69%
======================================================
  Files                          3        3
  Lines                        311      311
======================================================
  Hits                         167      167
  Misses                       144      144

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Update Long String Overflow Display

Description
Updates long string display to overflow. Currently the limit is set to be 40ch.
For reference, this is how it looks currently, for 40ch and for 80ch.
Previous:

40ch

80ch

References
https://github.com/XpressAI/xircuits/pull/190
Pull Request Type

[ ] Xircuits Core (Jupyterlab Related changes)
[x] Xircuits Canvas (Custom RD Related changes)
[ ] Xircuits Component Library
[ ] Xircuits Project Template
[ ] Testing Automation
[ ] Documentation
[ ] Others (Please Specify)

Type of Change

[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Tests

Drag in a string component. Give it a long text, maybe from here. Rate the aesthetic quality.
Run the workflow. Ensure that it can perform its function correctly.

Tested on?

[x] Windows
[ ] Linux Ubuntu
[ ] Centos
[ ] Mac
[ ] Others  (State here -> xxx )

Maybe we could add a scroll bar for the literal string component in future


error when running react-native link

Command link unrecognized. Make sure that you have run npm install and that you are inside a react-native project.'

tried to run react-native init on the directory but it would overide existing node packages
tried to run react-native init on a separate new folder and it is incompatible with npm version 10.24.1
i have npm install react-native-cli -g for node version 10.24.1
i have also tried the entire setup again with using yarn as a separate attempt

hi @dnaicker , can you try installing it with the following command after you have created your react-native app
npm i git://github.com/ayanworks/ARNIMA.git#master

hi @dnaicker , can you try installing it with the following command after you have created your react-native app
npm i git://github.com/ayanworks/ARNIMA.git#master

Thank you Sairanjit, its been a while since I have visited the repository, I am sure perhaps your suggestion will help the next person receive clarity


Specify ci.yaml workflow trigger paths

Description
Workflow ci.yaml only makes sense on changes to a client or a runtime code. Otherwise there should not be an expensive workflow run and a protective guard against a patch.
Required for https://github.com/Cerebellum-Network/blockchain-node/pull/461.

Branch protection rules don't let this pass. See also: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/collaborating-on-repositories-with-code-quality-features/troubleshooting-required-status-checks#handling-skipped-but-required-checks.


NOTEPAD EVENT

Describe your problem here.
Is there a way to make an event that opens a notepad? Like on the D-SIDES mod. If there is, could you pass me the script? it would really help me
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

@WilliamGaming12 Look at the sonic.exe 2.0 source code and go in GameOveSubstate.hx look for the function youFuckedUp

@WilliamGaming12 Look at the sonic.exe 2.0 source code and go in GameOveSubstate.hx look for the function youFuckedUp

I mean that it oppens the aplication, not  that it's made in the game, like the mighty.zip mod

io.open


Can't set hover from within react context

I'm trying to customize this button element to do a background color change on a "hover". Can't figure out how to do it. Nothing I put into this getChildContext function seems to have the effect I'm looking for. Here's what I'm trying:
getChildContext: ->
rebass:
Button:
backgroundColor: 'blue'
  Button:hover:
    backgroundColor: 'green'

I don't really understand the last section in the README.md that mentions hover. Am I supposed to stick that somewhere in a separate .css file? I thought the whole point of rebass and JS CSS modules was that I wouldn't need to have special rules in CSS files anymore. (?)

The way things are implemented currently (and since there's no :hover state with inline styles), there's no way to set hover state with the configuration object. There are several ways to handle CSS within your own setup, but I think that using CSS for things like hover and focus states is the best way to go with the technology available right now. I'll try to clarify that better in the docs.
It's also worth noting that these are all stateless components, so it would require a higher-order container component to control hover state with mouse events.

This really cripples Rebass, imho. I disagree that a JS CSS module should require a mixture of JS and CSS files, that is a really bad fragmentation and basically destroys my incentive to use Rebass at all.

It's usually a tiny amount of code and a limitation with inline styles, since they are pseudo-selectors.  I consider it a big win when the only css you have is something like this:
.Input input:focus {
  outline: none;
  box-shadow: 0 0 0 2px #00749d;
}

Problem is that is breaks deterministic ordering and logical separation of code, which are the main two reasons I wanted to use JS CSS modules, as those are the two features that can't be handled properly when using .css files. I'm looking at switching to a different JS CSS module, or just writing my own now, that will not require the use of .css or .sass files.

Hmmmm, I'm able to override the default styles using !important in a scss file, but I'm having a hard time setting those variables using contexts, can't seem to get any effect here, maybe because I'm using coffeescript and there is no "static" keywork possibly. I should just be able to stick a Button xml element into the render response right? Or is there some setting of state I need to do or something to get those Button variables to pass to the childContext or something? I looked at the demo code but the mechanism there is more complex. Here is an example very simple page, where I'm trying to set the backgroundColor of a button to green, but it's not working, maybe you can help me figure out why?
@cjsx React.DOM
@View = React.createClass
displayName: 'View'
childContextTypes:
rebass: React.PropTypes.object
getChildContext: ->
rebass:
Button:
backgroundColor: 'green'
render: ->
 <div className='topLevelView'>
   <Button
     rounded={true}
     pill={true}
   >button</Button>
 </div>


feat: prop to disable recently used wallets

Resolves PAR-13

~This PR introduces a disableRecentWallets prop to completely disable the re-organisation of recently used wallets on desktop and mobile modals.~
~Discussion is still ongoing to see if there's potentially a better solution. for this reason this PR is in Draft mode.~
~Follow the discussion here: https://github.com/rainbow-me/rainbowkit/discussions/625#discussioncomment-3625668~

UPDATE:
This PR implements "solution C" as discussed here.

solution C: disable "recent group" on desktop, but re-order the wallet list
basically means desktop having the same behaviour as mobile, where most recently used wallets are first in the list

Before

After

I noticed the accent color text is hidden when the wallet is selected since we use the accent color as the background:

We'll probably need to set it to accentColorForeground when selected?

Is it expected that the entire group of the most recently used wallet moves to the top of the list? My concern with this is that it could push other groups off the screen (especially as we add more wallets), and those groups could also contain other recently used wallets.
e.g. in this screenshot, "Other" has taken precedence over "Popular":

Yeah, I agree with @markdalgleish on the ordering of wallet groups. Apart from that, I am pretty happy with this PR!

@markdalgleish good catch, i missed that!

about the group ordering, the idea there was to really push the most recently used wallet to the top, and that includes the group.
but i can see how re-ordering the group is kinda weird. let me make some changes to only re-order wallets, but keep groups in place.

Nice one! 🎉


⚠ SceneTime has degraded performance

In ad8fb8d, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2324 ms

Resolved: SceneTime performance has improved in ea7830d after 8 minutes.


chore(actions): upgrade to checkout action to v3

Upgrade checkout action to v3 which uses Node 16, following Node 12 deprecation.
Besides the Node version upgrade, the new action version doesn't seem to bring any breaking changes:
https://github.com/actions/checkout/releases/tag/v3.0.0
Closes #1259

@fsmiamoto Thanks!


missing static/ folder in project root

I've created my site in blogdown using:
blogdown::new_site(theme = "hugo-apero/hugo-apero", format = "toml", sample = FALSE)
I am confused that it creates the static/ folder in /themes/hugo-apero/static. That is to say, there is no static/ folder in my project root. I've tried to create a static folder in the project root and move the landing page image (image /themes/hugo-apero/static/img/revoir.jpg) there; however, I don't know how to reroute Hugo to look for the image in this newly created static/ folder.

I find this confusing as well, I created a static folder in the root path, but it doesn't seem to be recognised while serving the website, is there any kind of references about this in apéro I can check, or is there a change I'm not aware of?

I am confused that it creates the static/ folder in /themes/hugo-apero/static.

This folder is part of the theme. All of Hugo works by overriding things in the themes/ older with project root folders. You don't need to re-route anything.

I find this confusing as well, I created a static folder in the root path, but it doesn't seem to be recognised while serving the website, is there any kind of references about this in apéro I can check, or is there a change I'm not aware of?

This works for me on the docs site and all my sites too? https://github.com/hugo-apero/hugo-apero-docs/tree/main/static/img
https://github.com/rbind/apreshill/tree/main/static
https://github.com/rstudio-education/iyo-rstudio-global/tree/main/static
Happy to update the docs, but not sure what would have made a difference here? https://hugo-apero-docs.netlify.app/start/site-config/#apéro-variables

I was unfortunately never able to figure out how to correctly set up a static folder in the root of the hugo-apero website created using the blogdown::new_site function. The way that I avoided the problem was to pull the starter repo for the "Introduce Yourself Online" workshop and begin to edit that.  To recreate the issue one can do the following:

Execute blogdown::new_site(theme = "hugo-apero/hugo-apero", format = "toml", sample = FALSE)
Create a static/ directory in the root with a subdirectory img/
Copy the contents of the original img directory to the newly created folder: cp -R /themes/hugo-apero/static/img/ static/img/
Add the image foo.jpg into  /static/img/
Edit the /content/_index.md to have the image img/foo.jpg (instead of img/revoir.jpg). Save
blogdown::serve_site()
If you follow those steps, you will see that the landing page has a broken image icon.  I hope that is somewhat helpful.

Thank you so much @vrbiki! Unfortunately I cannot reproduce. When I follow these steps I see:

FYI you can have blogdown create the empty folders for you in the project root:

At the command line: blogdown::new_site(theme = "hugo-apero/hugo-apero", format = "toml", sample = FALSE, empty_dirss = TRUE)

I'm not sure if asking this is appropriate or not, but

This works for me on the docs site and all my sites- if you fix it, will you let me know what the issue was?
https://github.com/hugo-apero/hugo-apero-docs/tree/main/static/
https://github.com/rbind/apreshill/tree/main/static
https://github.com/rstudio-education/iyo-rstudio-global/tree/main/static
I want to be responsive here, but not sure what would have made a difference in the docs? https://hugo-apero-docs.netlify.app/start/site-config/#apéro-variables
PRs for improvements to the docs are definitely welcome!

Thank you for your reply, I noticed in your personal website you have a pdf folder in your static, I originally wanted to do is to link a menu tab(resume) directly to a pdf file, and placed that in a static folder I created in root directory, I specified the file path in the config.toml, but I'm not sure if I need to create a directory for that menu tab in the content folder, and what arguments the _index.md file should have if yes.

Closing since not able to repro bug, and no response from original author. Happy to reopen discussion in a new issue for suggestions on improving the docs 📝


brunch b -p error : property 'content' of null

Description
Hello,
I am new here, and new in dev, also i am french so sorry for my bad english.
I work on a project with brunch, and i have a problem with stylesheets/joinTo files.
I want to have 3 differents CSS output files, so I write the following thing in my brunch-config.coffee
    stylesheets:
      joinTo:
        'css/file1.css': [/^mysite\/less\/header\// ]
        'css/file2.css': [/^mysite\/less\/content\// ]
        'css/file3.css': [/^mysite\/less\/footer\// ]

But it doesn't work (see below)
Expected behavior
Have 3 output css files
Actual behavior
When i used brunch b -p, i get the following error :
19 Aug 16:53:07 - error: Cannot read property 'content' of null
Stack trace was suppressed. Run with `LOGGY_STACKS=true` to see the trace.

So i do LOGGY_STACKS=true brunch b –p
And I get this :
/home/dev/myproject/node_modules/graceful-fs/polyfills.js:18
  chdir.call(process, d)
        ^

Error: ENOENT: no such file or directory, uv_chdir
  at Error (native)
  at process.chdir (/home/dev/myproject/node_modules/graceful-fs/polyfills.js:18:9)
  at Promise.all.then.setProp.constructor.application.loadConfig.then.then.initCompilation.initWatcher.chokidar.watch.on.on.absPath.on.compile.assetErrors.forEach.Promise.all.then.then.watch (/home/dev/myproject/node_modules/brunch/lib/watch.js:326:15)
  at exports.new.start (/home/dev/myproject/node_modules/brunch/lib/index.js:43:10)
  at Command.listener (/home/dev/myproject/node_modules/commander/index.js:301:8)
  at emitTwo (events.js:87:13)
  at Command.emit (events.js:172:7)
  at Command.parseArgs (/home/dev/myproject/node_modules/commander/index.js:615:12)
  at Command.parse (/home/dev/myproject/node_modules/commander/index.js:458:21)
  at Object.keys.forEach.exports.run (/home/dev/myproject/node_modules/brunch/lib/commands.js:86:11)
  at /home/dev/.nvm/versions/node/v4.2.2/lib/node_modules/brunch/bin/brunch:25:7
  at FSReqWrap.oncomplete (fs.js:82:15)

Environment

Brunch: 2.4.2
Node: 4.2.2
NPM: 3.0.0
Operating system: Windows 7

package.json contents
{
  "name": "tiiproject",
  "version": "0.1.0",
  "private": true,
  "devDependencies": {
    "brunch": "2.4.2",
    "css-brunch": "^1.7.0",
    "javascript-brunch": "^1.7.1",
    "uglify-js-brunch": "^1.7.8"
  },
  "dependencies": {
    "bless-brunch": "^1.6.1",
    "bootstrap-less": "^3.3.8",
    "extend": "^3.0.0",
    "fastclick": "^1.0.6",
    "font-awesome": "^4.5.0",
    "hammerjs": "^2.0.8",
    "inherits": "^2.0.1",
    "jquery": "^2.2.1",
    "jquery-hammerjs": "^2.0.0",
    "jquery-ui": "1.10.5",
    "less-brunch": "^1.8.1",
    "node-smushit": "^0.5.1",
    "swfobject": "^2.2.1"
  }
}

brunch config contents
exports.config =
  paths:
    public: 'web'
    watched: [
      'mysite/less',
    ]
  npm:
    enabled: true,
  files:
    javascripts:
      joinTo:
        'js/layout.js': [
          /^mysite\/js\/layout\//,
          /^mysite\/js\/player\//
        ]
    stylesheets:
      joinTo:
        'css/file1.css': [/^mysite\/less\/header\// ]
        'css/file2.css': [/^mysite\/less\/content\// ]
        'css/file3.css': [/^mysite\/less\/footer\// ]
  plugins:
    imageoptimizer:
      smushit: true
      path: 'img'
    bless:
      compress: false
      cacheBuster: true
  watcher:
    usePolling: true

Thanks for your help

Hey @thenewbii, could you please update Brunch to 2.8.2 and try again?

Also please try npm install github:brunch/brunch.

@shvaikalesh, @paulmillr : Thank's for your help, I update Brunch to 2.8.2 and it work :)
I'm curious, do you know what was the reason of this problem ?

Hello,
Finally update BRUNCH to 2.8.2 not solve the problem.
After hours of research, the problem comes of the bless-brunch plugin.
This plugin requires that the output files are named app.css, so the solution I find is this :
stylesheets:
      joinTo:
        'css/header/app.css': [/^mysite\/less\/header\// ]
        'css/content/app.css': [/^mysite\/less\/content\// ]
        'css/footer/app.css': [/^mysite\/less\/footer\// ]

By doing this it worked perfectly.
I'll send an email to the author of bless-brunch to see if a change is possible.
Thanks all for your help,


restclient require fix

the gemspec requires ruby-bitly just to get the version.  ruby-bitly.rb in turn requires restclient before it may have actually been installed.  This essentially makes this gem unbundleable on a system or gemset that doesn't already have restclient in it.
This fix smooths it all out.

Thanks!

Anytime! Thanks for putting this together.
On Fri, Nov 11, 2011 at 1:28 PM, rafaeldx7 <
<EMAIL_ADDRESS>
Thanks!

Reply to this email directly or view it on GitHub:
https://github.com/rafaeldx7/ruby-bitly/pull/4#issuecomment-2711471


Introduce Macro module and Musical Keyboard input handler

The current state is sort of messy, but it works. After merging this, the plan is to have the framework go in the direction of having things globally register themselves. i.e. globalThis.jamtools.registerModule
The API passed into these functions will be React "hooks-like", and lean into declaring things programmatically with function calls, rather than having classes implement some specific interface


support for murmur3

Added support for murmur3 hashing.
Uses new dependency murmurhash3js.
Since output is a 32bit number, had to create a generic function to convert from number into a 32 bit buffer.
Also, since input can only be a string, converting from buffer if needed.
murmur3 can use a seed, but to get parity with go-ipfs, leaving it to the default of 0.

Depends on https://github.com/multiformats/js-multihash/pull/30

released js-multihash :)

@diasdavid Thanks! Dep upgraded. Merge away if you will. :)

curious what murmur is going to be used for?

@jbenet it's the hashing function for HAMT.


site parameters from DB instead of telescope file

    The example files we have in `data/layout` have the long and lat. However, yesterday we said we should change that to be the actual site name and then read the long/lat from the DB for that site.

Originally posted by @orelgueta in https://github.com/gammasim/gammasim-tools/pull/429#discussion_r1043230431
Based on discussions with Orel, we would like to change the way we get the site information and pass it to LayoutArray. Now we get it from the telescope file but we want to get it from the DB.
The fact that we pass now also de db_config to LayoutArray (#429) will help implement this!
We need to first add all the information needed to the DB and after that to change the header of the files to not include unnecessary information.
This is planned to be implemented together in the beginning of next year.

Technically the information necessary is already in the DB since we have EPSG, longitude and latitude. What we don't have is the easting and northing centres, but I assume that those can be derived from the info we have. Is that correct @GernotMaier?

In #429 @GernotMaier pointed out it would be better to keep the information in the header, which I have no objection. But in this case, I would include in the header the site location as well (either 'North' or 'South').
For now, the header does not contain a direct indication of the site (label, i.e. either 'North' or 'South') except from a piece of text in the table.meta["description"]. Note that I am not talking about the site information such as lat, lon..
Given that, what would be the best way to get the site information (including 'North' or 'South' labels)?
1 - include the site label in the header and get the site info and label from there?
2 - get the site information from the DB and match it with the info from the header to extract the site label?
Sorry if that is not so clear. If thats the case I can try to explain with examples

This issue is essentially the same as #436 now I think. I would keep both open now cause there's good discussion in both. I will answer here in a similar way as I did there.
I think we should allow both options. If the file contains all the necessary info, then we don't retrieve anything from the DB. If it doesn't, we retrieve the missing info from the DB, assuming the file at least lists the requested site (North/South). We can also add a consistency check (i.e., the altitude in the file fits the provided site, but that might be a bit of overkill).


Create CodeQL action

Default config, seems to work local in local

Adding this will give more options for security

Thanks!


why the customView didn't show?

- (UIView *)customViewForEmptyDataSet:(UIScrollView *)scrollView
{
    UIView *view = [UIView new];
    view.frame = CGRectMake(300, 100, 200, 200);
    view.backgroundColor = [UIColor greenColor];

    return view;
}

the view is can't see.how to set customView，from top replace center。

Seems like the constraints for contentView inside UIScrollView+EmptyDataSet.h are incomplete. The code on 920 line sets centerX, centerY and Width but no height for contentView that actually contains custom view.
NSLayoutConstraint *centerXConstraint = [self equallyRelatedConstraintWithView:self.contentView attribute:NSLayoutAttributeCenterX];
NSLayoutConstraint *centerYConstraint = [self equallyRelatedConstraintWithView:self.contentView attribute:NSLayoutAttributeCenterY];

[self addConstraint:centerXConstraint];
[self addConstraint:centerYConstraint];
[self addConstraints:[NSLayoutConstraint constraintsWithVisualFormat:@"H:|[contentView]|" options:0 metrics:nil views:@{@"contentView": self.contentView}]];

// When a custom offset is available, we adjust the vertical constraints' constants
if (self.verticalOffset != 0 && self.constraints.count > 0) {
    centerYConstraint.constant = self.verticalOffset;
}

This means that contentView  height is 0.
I'd propose get rid of that centre constrains and change them to something like that all in one with vertical offset
[self addConstraints:[NSLayoutConstraint constraintsWithVisualFormat:@"H:|[contentView]|" options:0 metrics:nil views:@{@"contentView": self.contentView}]];
[self addConstraints:[NSLayoutConstraint constraintsWithVisualFormat:[NSString stringWithFormat: @"V:|-%f-[contentView]|",self.verticalOffset] options:0 metrics:nil views:@{@"contentView": self.contentView}]];

Changing the constraints as suggested fixes this one case but breaks all other use cases of the library. Since the view that is shown participates in auto layout you can instead create a UIView subclass and have it return a custom intrinsicContentSize.

Does someone have a sample project on how to create a custom view? It will be greatly appreciated :)

Project no, but here's example of implementation
And how the view is initialised.
You should just get rid of extra constarints
open func customView(forEmptyDataSet scrollView: UIScrollView!) -> UIView? {
    let emptyView : EmptyStateBaseView = EmptyStateBaseView.view(delegate: nil, viewSize: self.emptyStateViewSize)
    emptyView.emptyImageView.image = UIImage.init(named:"favorites_emptystate")
    emptyView.emptyTitleLabel.text = LS("my_zone_empty_favorites_title")
    emptyView.emptySubtitleLabelText = LS("my_zone_empty_favorites_sub_title")
    emptyView.emptySubtitleLabel.text = LS("my_zone_empty_favorites_sub_title")
    emptyView.containerWidthLayoutConstaint.constant = self.traitCollection.horizontalSizeClass != .compact ? 380 : scrollView.size.width - 40.0
    return emptyView

}

On Jan 18, 2017, at 10:40, Jayven N<EMAIL_ADDRESS>wrote:
Does someone have a sample project on how to create a custom view? It will be greatly appreciated :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I just want to return a UIView that fills my collection view with a background of black color

func customView(forEmptyDataSet scrollView: UIScrollView!) -> UIView! {
    let view = UIView()
    view.frame = CGRect(x: 0, y: 0, width: scrollView.bounds.width, height: scrollView.bounds.height)
    scrollView.addSubview(view)
    return nil
}

CustomVeiw must add height constriant,frame dosen't work

CustomVeiw must add height constriant,frame dosen't work


The documentation does not work exactly for YAML based pipeline tasks

Document says:
Search for Project Collection Build Service. By default, this identity can read from the repo but cannot push any changes back to it. Grant permissions needed for the Git commands you want to run. Typically you'll want to grant
Instead it should say:
Search for Project Collection Build Service. Choose the service typically named Project Collection Build Service (OrganizationName). For XAML builds, use Project Collection Build Service Accounts, which is a legacy security group. By default, this identity can read from the repo but cannot push any changes back to it. Grant permissions needed for the Git commands you want to run. Typically you'll want to grant

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

ID: 7eb008b3-663d-0698-9f39-65cbfde34dcf
Version Independent ID: ddcd9666-1517-866c-4cee-f79fccaf0663
Content: Run Git commands in a script - Azure Pipelines
Content Source: docs/pipelines/scripts/git-commands.md
Product: devops
Technology: devops-cicd-scripts
GitHub Login: @juliakm
Microsoft Alias: jukullam

this PR #9930  has fixed this issue

this PR #9930  has fixed this issue


Fix grammar in object's short description

Please always provide a description that explains the need for this PR. It involves minor grammar changes as your first contribution.
/lgtm

/approve


'axisTitleData' not defined when using LineChartData

Hi, here is my code. 'axisTitleData' not defined as well as 'FlAxisTitleData' and 'AxisTitle'.
  LineChartData sampleData1() {
    return LineChartData(

      gridData: const FlGridData(
        show: false,
      ),
      axisTitleData: FlAxisTitleData(
          leftTitle:
              AxisTitle(showTitle: true, titleText: 'Return', margin: 4)),

Hi,
which version are you using?

I am using 0.6.0. I changed to 0.6.2 and it works perfectly now. So silly that I did not check before. Thank you!

No problem.
Cheers!

Reopening this issue because I just updated to v0.55.0 which was using 0.36.3 earlier, and now LineChartData doesn't have an axisTitleData: named parameter anymore. The documentation still refers to it, including the source code file's documentation comments. But the parameter has been removed, it seems in this commit: https://github.com/imaNNeoFighT/fl_chart/commit/43fe43aae5d686da476c0f86bf7f2be37cd4edd5
Can you please document what we need to do to adapt? Thanks.


Update mlir-aie

The start of updates for AIE.device, AIETargetModel and AIE2.

update mlir-aie to 97bb9cc
Generate AIE.device op during air-to-aie
Add device option to air-to-aie
Generate AIE2 locks in air-to-aie dma lowering
...

We should make sure this version bump also contains the update for objectFifo symbolic names (as recent PRs rely on it). I will work on getting that merged into mlir-aie as a priority.

We should make sure this version bump also contains the update for objectFifo symbolic names (as recent PRs rely on it). I will work on getting that merged into mlir-aie as a priority.

Sure, once https://github.com/Xilinx/mlir-aie/issues/290 is resolved we can update freely

The PR for objectFifo symbolic names is now merged into mlir-aie main: commit hash 136d8de22e5978761152ed9bc4fd686823b39dd6.

It is currently passing against https://github.com/Xilinx/mlir-aie/pull/496, I'll update the hash once that is merged.


Fix coverage report being in ephemeral path.

Codecov Report
Base: 19.74% // Head: 25.25% // Increases project coverage by +5.50% :tada:

Coverage data is based on head (badb21f) compared to base (868f9a4).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           stable     #746      +/-   ##
==========================================
+ Coverage   19.74%   25.25%   +5.50%
==========================================
  Files          70       70
  Lines        5829     5833       +4
  Branches      314      313       -1
==========================================
+ Hits         1151     1473     +322
+ Misses       4632     4298     -334
- Partials       46       62      +16

Impacted Files
Coverage Δ

library/virtual_machine/evaluator.cpp
7.92% <0.00%> (-0.08%)
:arrow_down:

library/grammar/expr.hpp
23.17% <0.00%> (-2.44%)
:arrow_down:

library/virtual_machine/process_context.cpp
22.42% <0.00%> (+2.80%)
:arrow_up:

library/object/number/compare.cpp
6.71% <0.00%> (+4.69%)
:arrow_up:

library/object/number/left_shift.cpp
6.45% <0.00%> (+6.45%)
:arrow_up:

library/object/number/div.cpp
10.90% <0.00%> (+6.66%)
:arrow_up:

library/object/number/and.cpp
7.97% <0.00%> (+7.97%)
:arrow_up:

library/object/number/mod.cpp
9.23% <0.00%> (+9.23%)
:arrow_up:

library/object/number/gcd.cpp
10.00% <0.00%> (+10.00%)
:arrow_up:

library/object/number/floor_div.cpp
10.43% <0.00%> (+10.43%)
:arrow_up:

... and 10 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Remootio hostname instead of IP address?

I have two remootios (two different gates) and I'd like to be able to set configuration using a local hostname like "maingate.local" and "backgate.local" instead of an IP address that can change. Any ideas how to do this?
Thanks!

@ccorcos, my Remootio devices seem to be hardcoded with the hostnames in this format: remootio_.local
I found the device serial number on the Bluetooth configuration page in the Remootio iPhone app. An alternate way is to use the Discovery app on your iOS device and look for the _remootio._tcp service.
With the hostname in hand, I changed the IP address to the hostname in the homebridge configuration, and it worked for me.

Hmm. When I run arp -a, I'm seeing two remootios with the same remootio.local address.
❯❯❯ arp -a
chets-mbp.attlocal.net (<IP_ADDRESS>) at f4:d4:88:79:9a:d7 on en0 ifscope permanent [ethernet]
remootio.attlocal.net (<IP_ADDRESS>) at 8c:4b:14:78:ca:98 on en0 ifscope [ethernet]
remootio.attlocal.net (<IP_ADDRESS>) at e8:31:cd:66:a2:40 on en0 ifscope [ethernet]

The IP address is reachable
❯❯❯ ssh<EMAIL_ADDRESS>ssh: connect to host <IP_ADDRESS> port 22: Connection refused

But I can't get the local address to work...
❯❯❯ ssh<EMAIL_ADDRESS>❯❯❯ ssh<EMAIL_ADDRESS>❯❯❯ ssh<EMAIL_ADDRESS>

I think you have the wrong serial number. the first number 46 should be 4b:
remootio.attlocal.net (<IP_ADDRESS>) at 8c:4b:14:78:ca:98 on en0 ifscope [ethernet]
❯❯❯ ssh<EMAIL_ADDRESS>

Ah you're right! The image-to-text on my iPhone parsed this bonkers font wrong!
Thanks for the help.

❯❯❯ ssh<EMAIL_ADDRESS>ssh: connect to host remootio_8c4b1478ca98ecrsqjlj.local port 22: Connection refused

The config seems unhappy with a local address though.

That seems like a bug/enhancement to the form.
If you go directly into the config editor, you can bypass the form check and make it work.

Fixed in release 1.3.2

Thank you so much for being so responsive! 😄 💯


iTerm.sh no longer working with iTerm2 Build 2.9.2x

With the new release of iTerm2 v.3, Open Terminal Here... command is not working any more. Any solution?

I just ran into this as well.

This is a duplicate of #89, please see the discussion there.


Error when loadDocument in pdfController

I find an error message when try to load new document. This is the error message:

The getter 'pageNumber' was called on null.
Receiver: null
Tried calling: pageNumber

it is fixed by downloading the recently git hub source.


_parsedUrl.url is undefined - version 1.0.0-next.3

Hi thanks for the great library!
This line is throwing TypeError: Cannot read property 'substring' of undefined when running custom middleware.
https://github.com/lukeed/polka/blob/602086f1f4f1a6103e0589b04b69f8b4d934f0fc/packages/polka/index.mjs#L30
I believe its caused by this line
https://github.com/lukeed/polka/blob/602086f1f4f1a6103e0589b04b69f8b4d934f0fc/packages/polka/index.mjs#L36
As far as I can tell, the latest version of @polka/url doesn't set the url on the _parsedUrl. Is this a bug? Or am I missing something?

Hey, thanks!
Do you mind showing me how you're using it? I'm using this version and haven't run into this error, so I'm a bit puzzled haha
Thanks

I'm just trying to simplify the code a little, I'll throw it up shortly, and it'll hopefully be easier for you to detect what I'm doing wrong haha
I did switch that line req.url = req._parsedUrl.href; and it seems to be working fine now

Okay, heres a simplified version that seems to be throwing this error.
const polka = require("polka");

const app = polka();

const mock = (req, res) => {
  res.end();
};

const middleware = (req, res, next) => {
  next();
};

app.use("/one", middleware);
app.use("/one/:id", middleware);
app.get("/one/:id", mock);

app.listen(3000, err => {
  if (err) {
    throw err;
  }
  console.log(`> Running on localhost:3000`);
});

You were right, thank you! 🙌

Now included in<EMAIL_ADDRESS>– much appreciated


feat: Add hero section in index.html

Hey @michaella23 I would like to add hero section in out index.html page

[x] Background Image
[x] Feature Text
[x] Site Description
[x] CTA Button

This sounds fantastic! Thanks @rabin-thami. 🙂

@michaella23  i just did a PR

@rabin-thami @michaella23 small order of business: the closing  tag seems to be missing for the hero section in /index.html

Wow. Good catch @BastienWinant ! Sorry if that was creating issues for you.


Multiprocessing problems on Windows

Hey there!
Some days ago I receive a awesome version of stl-to-voxel from @youngkiu that makes the time of convertion decrease more than 10 times.
The version have some new libraries like ray, psutil, etc.
All works very well on Linux and Mac OS X, but when I finished the setup on Windows and take a test, I receive this message:

x: -149.99998474121094 ~<PHONE_NUMBER>1757812
y: -150.00006103515625 ~<PHONE_NUMBER>8242188
z: -150.0 ~<PHONE_NUMBER>5878906
x: 0.0 ~<PHONE_NUMBER>1839716
y: 0.0 ~ 511.0
z: 0.0 ~<PHONE_NUMBER>1839716
Traceback (most recent call last):
File "C:\Users\cogit\Downloads\StlToVoxel_new\stltovoxel.py", line 104, in
doExport(args.input, args.output, args.resolution, args.pad)
File "C:\Users\cogit\Downloads\StlToVoxel_new\stltovoxel.py", line 22, in doExport
vol, bounding_box = slice.meshToPlane(mesh, bounding_box, pad)
File "C:\Users\cogit\Downloads\StlToVoxel_new\slice.py", line 11, in meshToPlane
ray.init(num_cpus=num_cpus)
File "C:\Users\cogit\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\ray_private\client_mode_hook.py", line 47, in wrapper
return func(*args, **kwargs)
File "C:\Users\cogit\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\ray\worker.py", line 703, in init
ray_params=ray_params)
File "C:\Users\cogit\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\ray\node.py", line 227, in init
self.start_head_processes()
File "C:\Users\cogit\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\ray\node.py", line 875, in start_head_processes
self.start_monitor()
File "C:\Users\cogit\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\ray\node.py", line 843, in start_monitor
backup_count=self.backup_count)
File "C:\Users\cogit\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\ray_private\services.py", line 1918, in start_monitor
fate_share=fate_share)
File "C:\Users\cogit\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\ray_private\services.py", line 611, in start_ray_process
ray._private.utils.set_kill_child_on_death_win32(process)
File "C:\Users\cogit\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.7_qbz5n2kfra8p0\LocalCache\local-packages\Python37\site-packages\ray_private\utils.py", line 758, in set_kill_child_on_death_win32
"AssignProcessToJobObject() failed")
OSError: [Errno 0] AssignProcessToJobObject() failed

I read and try some solutions on web, but I had not sucess. Is it possible to solve?
A big hug!

In the branch based on the Commit: 1376d0c1d4a697851f54b2c43ee2c56721ced813 revision implemented in ray, I confirmed that there is a problem in the windows os.

Fail: windows - https://github.com/youngkiu/stl-to-voxel/actions/runs/844661796
Pass: ubuntu, macos - https://github.com/youngkiu/stl-to-voxel/actions/runs/844666178

In Commit: d6abeadabb855c2b48ec7d523c2ef8ba6568aae5 revision, @cpederkoff has already changed the library for parallel processing from ray to multiprocessing.
So, I additionally added github actions to work on all os.
https://github.com/youngkiu/stl-to-voxel/actions/runs/844714034, I have confirmed that the latest code works fine on all os.
After pull/29 merge, I will make a pull request again.

Dear @youngkiu
Thank you again for the effort to make the code better.
I downloaded the new version and run a test on Windows.
In the old version, without ray or multiprocessing the voxel was generated in 521 secods.
In the new version: 51 seconds!
OBS.: Included the time to convert to DICOM.
Now Ill test this news code in Mac and Linux. Thank you very much!


Highlight plugin prefix in plugin artifactId

This is a draft to try the rendering of #143

Is this what you had in mind @hboutemy ?

I have also tried with strong rendering without being able to choose one winner 😇

wow, I love it: yes, that was my intent!
I can't choose the winner either :)

I've gone with the green one.
At worst it can be change with:
mvn org.codehaus.mojo:buildplan-maven-plugin:2.1.1-SNAPSHOT:list -Dstyle.mojo=bold

WDYT @hboutemy ?


Multibit not send bitcoin

Hello friends, attach screen

Can you verify that you have read this article:
https://multibit.org/en/help/v0.5/help_support_network.html

I 2 ISP connection with different routers, this ports is open.
I could solve my wallet exporting to Blockchain but Multibit not send.

Can you verify that your computer clock is correct, please. Also to properly check your network connection you'll need to review #554.

Closing due to inactivity.


fix get/post example code

ping @evancz this should be a no-brainer :)

Thanks. I am starting to roll out @process-bot who will talk about the process behind issues, so hopefully that'll show how things like Elm 0.17 may block other things and why this tradeoff is valuable. In any case, sorry for the delay on this!

That sounds very reasonable, but at the same time, having small typos like this one not fixed for so long is a big deal for people trying to learn the language. Anyway, thanks for finally getting this in!


Force AttributeBase Alignment to 8-byte

On 32-bit ARM, if an AvailableAttr was getting 4-byte aligned, it would trigger a LLVM assert that the PointerIntPair wasn't aligned enough. Forcing 8-byte alignment for AttributeBase addresses the problem. However, the assert is mentioning it wants 2 bits (ParserResult also only asks for 2), so it's a bit more complicated here than simply too many bits being asked for.
If you are using the custom new operator on a child class of AttributeBase, you get the implementation on AttributeBase, which appears to force alignment to be that of AttributeBase. This is problematic if any child class requires more strict alignment than AttributeBase in general. It looks like PointerIntPair uses NumLowBitsAvailable to mask the pointer, and LLVM was catching that AvailableAttr was getting aligned less strictly than its type requires due to the behavior in new.
It's probably better to fix the underlying issue in the new operator to be able to allocate on the alignment of the type, but as an unblocking measure, it is simpler to ensure that AttributeBase has an alignment at least as strict as its child classes.

Makes sense
@swift-ci please smoke test

⛵


Potodo is not working on Windows

"NotImplementedError: "Windows" is currently not supported."

Since I work with this.. "OS", I'll give it a try

@Mindiell you should only get this error if running potodo in interactive mode (-i)

Nope, got it from the import of simple_term_emu even if I'm not in interactive mode (or is this the default mode ?).
After some research, got it because simple_term_emu is based on termios which seems to be "only available for those Unix versions that support POSIX termios style tty I/O control configured during installation."...

Hmm ok I see the problem. I'll add a fix soon

But imports in code are ugly I think.

When they fix things, I feel it's acceptable.
It's probably less ugly when used in the right place, typically right after an if is_interactive may not be the best place, but as a first line of the only function needing it feels OK to me.
I typically enjoy having import argparse in the only function using it, to avoid importing it when it's not needed (when used as a library).
So, same here, if imported in the only function using it, when the program is called without -i it won't be imported. It's probably a few milliseconds faster not to import it, so it's not only a gain for Windows users, it becomes a gain for everyone.

@JulienPalard got time to issue a fix ?

Not today, gotta sleep, but I hope so.

@Mindiell can you try updating potodo with
pip install git+https://github.com/JulienPalard/potodo.git@windows-compatibility


ParquetDataConverter and refactoring DataConverter

FYI @RasmusOrsoe

Yes, agreed! I think that extracting everything from the I3 files is separate from the format to which we subsequently save the data, right? Because I remember all data also being saved to HDF5. That is why I have created #214, which could then be used as the sole extractor, that takes everything from the I3 files and dumps it in a parquet file (or SQLite database, for that matter). Does that agree with what you have in mind? :)

Yes, agreed! I think that extracting everything from the I3 files is separate from the format to which we subsequently save the data, right? Because I remember all data also being saved to HDF5. That is why I have created #214, which could then be used as the sole extractor, that takes everything from the I3 files and dumps it in a parquet file (or SQLite database, for that matter). Does that agree with what you have in mind? :)

Almost. I think it would be very challenging to make the SQLite format extend to whatever is in the files.

Hi @RasmusOrsoe,
I have finally prepared the PR for review. It's a massive one, so I think it could make sense for us to go over it in 1:1/in-person. The most substantial changes are:

Rewrite of DataConverter class
Rewrite of SQLiteDataConverter class to properly subclass DataConverter
Addition of ParquetDataConverter class
Addition of Dataset class
Rewrite of SQLiteDataset class to subclass Dataset
Addition of ParquetDataset class
Restructuring graphnet.data module to make it easier to navigate
Addition of example to compare the two sets of DataConverter/Dataset classes
Unit tests for both "backend" file formats


[markdown] Tabbing when writing a word replaces it with an HTML element

VSCode Version: 1.6.0
OS Version: Windows 7

Steps to Reproduce:

Create a mardown file.
Type a word, like test
Press TAB. It replaces test with <test></test>. Why so?

This seems to be coming from emmet. After disabling emmet.triggerExpansionOnTab, I no longer see this completion.

Ok, I believe a workaround is to add the setting:
"emmet.excludeLanguages": ["markdown"]

This is that triggered for markdown files in the first place because the parent syntax of a markdown document is html. Since you can have any html content inside of markdown files, I believe this behavior is intended.


Pleases provide guidance on how to embed scriptedforms in iframe

Hi Team,
Many thanks for the work and I found scriptedforms particular useful in presenting outputs. I'd greatly appreciate any advices on how to embed the local instance of scriptedforms in a webpage so it could be presented to my colleagues.
I have been running into "Content Security Violation" error, although using the same setup I have no problems embedding a local Jupyter Notebook instance. The following is my configuration (so far NOT working), once again, grateful to any comments.
Flask app to embed the output of scriptedforms via iframe
Modified scriptedforms.py
    def start(self):
    ┆   load_jupyter_server_extension(self)
    ┆   # Add remote access
    ┆   # self.config_file = u"/home/d594400/.jupyter/jupyter_notebook_config.py"  # Doesn't work
    ┆   # self.port = 9999  # CANNOT change port
    ┆   # self.ip = u"<IP_ADDRESS>"  # IP must be localhost, CANNOT change IP
    ┆   self.allow_remote_access = True
    ┆   self.token = ""
    ┆   self.password = ""
    ┆   self.tornado_settings = {
    ┆   ┆   "headers": {
    ┆   ┆   ┆   "Content-Security-Policy": "frame-ancestors self http://localhost:8888 http://localhost:60010 http://<IP_ADDRESS>:8888 http://<IP_ADDRESS>:60010; report-uri /api/security/csp-report"
    ┆   } }
    ┆   super(ScriptedForms, self).start()

flask app
from flask import Flask
app = Flask(__name__)

@app.route('/')
def homepage():
    return """

Hello world!

    """
if __name__ == '__main__':
    app.run(host="<IP_ADDRESS>", port=60010, debug=True, use_reloader=True)

Then ran into error while visiting the local flask page:
[W 10:32:02.140 ScriptedForms] Content security violation:
{"csp-report":{"document-uri":"http://localhost:8888/scriptedforms/use/scripted.md?","referrer":","violated-directive":"frame-ancestors","effective-directive":"frame-ancestors","original-policy":"frame-ancestors
'self'; report-uri /api/security/csp-report","disposition":"enforce","blocked-uri":"http://localhost:8888/scriptedforms/use/scripted.md?","status-code":0,"script-sample":""}}

Thanks and regards,
Peter

Hi Peter,
Thanks for reaching out, and thanks for the interest in ScriptedForms.
ScriptedForms installed itself as a jupyter notebook server extension. This means that if you start a standard notebook server and travel to http://IPADDRESS/scriptedforms/use/scripted.md it will resolve to the form.
So if you want to use custom server configuration, just use the notebook server :).
Do be aware that anyone who can access that page can remotely execute any python code, command line code, as well as code from any other kernel you have installed.


Missing attributes to Template DTO objects

When using Get-BetaNotificationTemplates in the PowerShell SDK, it returns additional attributes slackTemplate and teamsTemplate, which always seem to be null by the way. ConvertFrom-BetaJsonToTemplateDto does not account for these today I guess because they are not defined in the TemplateDto schemas, so the error below is thrown if you try to convert templates as they are exported out of a tenant in order to push them to another.

https://github.com/sailpoint-oss/api-specs/pull/53

Thank you for reporting this @patrickboston . I have opened a PR to add an additional check in our schema validator tool which will surface any more instances of additional properties in SailPoint's APIs. This tool will fix the two properties you mentioned here, and it may find more for us to fix. Once we find and fix the missing properties, I will close out this ticket.

This issue has been mentioned on SailPoint Developer Community Forum. There might be relevant details there:
https://developer.sailpoint.com/discuss/t/official-powershell-identitynow-sdk-unofficial-quickstart-guide/9160/8


Access to props

Hope to have a way to access props to get attributes such as :

width of the panel
height of the panel
eventBus.

When you say evenBus, what do you mean by that?

I want to use enentBus from PanelProps interface to make use of 'shared crosshair'
https://grafana.com/docs/grafana/latest/packages_api/data/panelprops/


Added License to the top of cli.py

Added the license that should be in every file to the cli.py.

Thanks!


Function for regressing/correlating multiple fields?

I came across this StackOverflow thread about applying linear regression across multi-dimensional arrays and there were some very efficient, xarray-based solutions suggested to the OP which leveraged vectorized operations. I was wondering if there was any interest in adding a function like this to future releases of xarray? It seems like a very neat and useful functionality to have, and not just for meteorological applications!
https://stackoverflow.com/questions/52108417/how-to-apply-linear-regression-to-every-pixel-in-a-large-multi-dimensional-array
Most answers draw from this blog post:
https://hrishichandanpurkar.blogspot.com/2017/09/vectorized-functions-for-correlation.html

Very much so! Unfortunately we've not managed to merge anything yet; check out.
https://github.com/pydata/xarray/issues/1115, https://github.com/pydata/xarray/pull/2652
We'd be very keen to merge a minimum-viable-product and then iterate from there. If you'd be interested in contributing towards this that would be great!

I'll take a look at them!

@max-sixty Just had a peruse through a few of the relevant issues, do we know what the status of [#3550 ] is? It seems like @r-beer was pretty close on this, right?

Yes that's so close! I'm sad that didn't make it in.
@r-beer if you see this: do you have a one-sentence update? Might you come back to this area or should someone else try to land it?

Hi @max-sixty, just coming back to this now. It seems @r-beer isn't available...do you know roughly how far away his PR was from completion? I'm getting a little bit lost trying to follow #3550 sorry!
Was the main todo to avoid the drop=True after broadcasting? Is there any idea about what to do instead?

In a fit of covid-induced insanity, I've decided to have a crack at finishing up #3550 ! I'm playing around with the changes made by @r-beer at the moment, but I'm finding the tests quite confusing - I think they're wrong? But maybe someone could help me out with this?
Here's something from test_computation.py in #3550
def test_cov(da_a, da_b, dim):
    def pandas_cov(ts1, ts2):
        """Ensure the ts are aligned and missing values ignored"""
        ts1, ts2 = xr.align(ts1, ts2)
        valid_values = ts1.notnull() & ts2.notnull()

        ts1 = ts1.where(valid_values, drop=True)
        ts2 = ts2.where(valid_values, drop=True)

        return ts1.to_series().cov(ts2.to_series())

    expected = pandas_cov(da_a, da_b)
    actual = xr.cov(da_a, da_b, dim)

    assert_allclose(actual, expected)

What I don't understand is, why would we expect the Pandas covariance or correlation functions to return anything remotely like the output of xr.cov()? The line ts1.to_series().cov(ts2.to_series()) always produces a scalar value, whereas in most reasonable use cases xr.cov(da_a, da_b, dim) would be producing a matrix of values (eg. the pixel-wise correlation in time between two DataArrays).
I wasn't sure whether to open a PR for this or not? I'm working on it but would require some help to set up some appropriate tests...


CNI errors when frequently adding and removing deployments

Is this a request for help?:

Is this an ISSUE or FEATURE REQUEST? (choose one): Issue

Which release version?: 1.0.27

Which component (CNI/IPAM/CNM/CNS): CNI

Which Operating System (Linux/Windows): Windows Server 2019

For Linux: Include Distro and kernel version using "uname -a"

For windows: provide output of "$(Get-ItemProperty -Path "C:\windows\system32\hal.dll").VersionInfo.FileVersion"

Which Orchestrator and version (e.g. Kubernetes, Docker) Kubernetes

What happened: We have a PR build environment where we frequently add and remove entire deployments many times throughout the day.  We are using AKS-Engine 1.15.4.  Several times a day we will see that multiple pods are unable to talk to other pods causing them to crash.  After that runs for a while the node will end up crashing.  The constant building up and tearing down of the system seems to exercise the CNI plugin ability to assign IP addresses.  The kubelet.err logs contain the following:
I0128 16:16:56.012105 18216 fake_cpu_manager.go:45] [fake cpumanager] RemoveContainer (container id: f5fd17ba23f7fc983f1dbbbaac64c8d89a0745402228e24ffe403a1a00aef245)
I0128 16:16:56.173003 18216 fake_cpu_manager.go:40] [fake cpumanager] AddContainer (pod: feedadmin-68644b5684-rbp79, container: feedadmin, container id: ae7406228e2e0a56083cd15be347c193acfbaa41551aca831aab3e1e829d7bfa)
E0128 16:16:56.952164 18216 cni.go:331] Error adding oncallprtest-19207_loi-669888c8d7-nbfl4/60efc16f1dd2414a62dbaae1862c2ffcdb3ac329f39b546b911a8a81ed6e05fa to network azure-vnet/azure: invalid character '{' after top-level value
E0128 16:16:56.952164 18216 cni_windows.go:50] error while adding to cni network: invalid character '{' after top-level value
W0128 16:16:56.952164 18216 docker_sandbox.go:384] failed to read pod IP from plugin/docker: NetworkPlugin cni failed on the status hook for pod "loi-669888c8d7-nbfl4_oncallprtest-19207": invalid character '{' after top-level value

What you expected to happen:  Be able to spin up and tear down these environments many times a day without experiencing CNI errors.

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know:

Can you use latest CNI v1.0.33 and let us know?


[Bug]: Can't login as a customer while developing theme app extension

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
Extension
Expected behavior
While developing app blocks, for a theme app extension, I'd expect to be able to mimic a signed-in customer, from within the localhost development store.
Actual behavior
While running npm run dev and attempting to sign in through the local storefront, <IP_ADDRESS>:9292, it redirects to the myshopify-domain when submitting the "I'm not a robot"
Verbose output
DEBUG GET https://MYSHOPIFY-DOMAIN.myshopify.com/admin/api/unstable/themes/THEME-ID/assets.json with X-Request-Id: 450b7f34-c2a6-4807-b1fe-60e0696c0103
┃ DEBUG [BulkJob #3380] job request: size=4, bytesize=15635                                                                                36%
┃ DEBUG PUT https://MYSHOPIFY-DOMAIN.myshopify.com/admin/api/unstable/themes/THEME-ID/assets/bulk.json with X-Request-Id: f94a258b-4da8-441a-ae17-a03e5650b135
┃ DEBUG [BulkJob #3380] bulk rest_request time: 1257.497ms
┃ DEBUG [BulkJob #3380] job response: http_status=207
┃ DEBUG [BulkJob #3380] bulk item success (item=sections/header-group.json)
┃ DEBUG [BulkJob #3380] bulk item success (item=templates/cart.json)                                                                               52%
┃ DEBUG [BulkJob #3380] bulk item success (item=templates/index.json)                                                                              68%
┃ DEBUG [BulkJob #3380] bulk item success (item=templates/product.json)                                                                            84%
┃ DEBUG [Bulk] shutdown, remaining_items=0                                                                                100%
┃ DEBUG [BulkJob #3400] job request: size=1, bytesize=5588
┃ DEBUG PUT https://MYSHOPIFY-DOMAIN.myshopify.com/admin/api/unstable/themes/THEME-ID/assets/bulk.json with X-Request-Id: 40f56a50-7889-4474-b7e3-56e2c783b809
┃ DEBUG [BulkJob #3400] bulk rest_request time:<PHONE_NUMBER>000001ms
┃ DEBUG [BulkJob #3400] job response: http_status=207
┃ DEBUG [BulkJob #3400] bulk item success (item=config/settings_data.json)
┃ DEBUG [Bulk] shutdown, remaining_items=0                                                                                100%
┃                                                                                100%
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ (11.48s) ━━
DEBUG POST https://partners.shopify.com/api/cli/graphql with X-Request-Id: 7a2d32ff-6d3f-4e34-9466-b5f7475bcae8
DEBUG {"data"=>{"app"=>{"id"=>"APP_ID", "title"=>"Local", "apiKey"=>"API_KEY", "apiSecretKeys"=>[{"secret"=>"API_SECRET_KEY"}], "appType"=>"public", "extensionRegistrations"=>[{"id"=>"EXTENTION_ID", "type"=>"THEME_APP_EXTENSION", "uuid"=>"EXTENSTION_UUID", "title"=>"App Extension", "draftVersion"=>{"registrationId"=>"EXTENTION_ID", "context"=>nil, "lastUserInteractionAt"=>"2023-03-15T06:55:50-04:00", "location"=>"https://partners.shopify.com/PARTNER_ID/apps/APP_ID/extensions/theme_app_extension/EXTENTION_ID", "validationErrors"=>[], "id"=>"11303401", "uuid"=>"b4868e18-905e-481b-a2dc-e60d897ad8b8", "versionTag"=>"0.0.0"]}
┏━━ Viewing extension… ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
┃ Enable your theme app extension:
┃ https://partners.shopify.com/PARTNER_ID/apps/APP_ID/extensions/theme_app_extension/EXTENTION_ID
┃
┃ Setup your theme app extension in the host theme:
┃ https://MYSHOPIFY-DOMAIN.myshopify.com/admin/themes/THEME-ID/editor
┃
┃ Preview your theme app extension:
┃ http://<IP_ADDRESS>:9292
┃
┃ (Use Ctrl-C to stop)
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ (0.0s) ━━
[2023-03-15 13:00:18] INFO  WEBrick 1.7.0
2023-03-15 12:00:18 | extensions | 2023-03-15T12:00:18.991Z: Closing locale file watching for extension with ID d2be0839-c021-47a9-8149-f425859b4b20

Reproduction steps

npm run dev
Open the link to preview theme app extension
Head to /account
Attempt to sign in

Operating System
MacOS Ventura 13.2.1
Shopify CLI version (check your project's package.json if you're not sure)
3.44.1
Shell
zsh
Node version (run node -v if you're not sure)
18.14.2
What language and version are you using in your application?
3.0.4

@Shopify/theme-developer-tools

I'd say this is still relevant

I'm having the same issue.. when I click on the "I'm not a robot" it goes to the live site.. and I need to test customer account features locally..
anyone is able to do that? or it's not possible?


Changed the data type for the Feature Class parameter for all three scripts

I started from scratch re-applied these changes to the binary esri2open.tbx file in the TopJSON commit (SHA:2400d16efc).
Changed the data type for the Feature Class parameter for all three scripts from Feature Layer to Feature Class to accomodate Network Dataset members (and possibly others).
The Feature Class parameter data type was originally set to 'Feature Layer'. Although it's not clear from the documentation (see link below), the Feature Layer data type excludes complex Geodatabase feature classes, such as the Simple Junction Feautures that are part of a Network Dataset, even though ArcPy treats them as simple Point features for the operations performed by these tools.
To allow the export of Network Dataset participants I changed the Feature Class parameter data type to 'Feature Class'.
Geoprocessing Data Types document: http://resources.arcgis.com/en/help/main/10.1/01m1/pdf/Geoprocessing_data_types.pdf

:+1: thanks.


Training of Qwen2

Hi, EAGLE team. I want to train my  EAGLE-Qwen2 model on  Chinese-ShareGPT dataset, to generate train data, I modified the code in ge_data_all_llama3.py, I only changed sep = "<|eot_id|><|start_header_id|>assistant<|end_header_id|>\n\n" to  sep = "<|im_end|>\n<|im_start|>assistant\n" and sep2="<|eot_id|><|start_header_id|>user<|end_header_id|>" to sep2="<|im_end|>\n<|im_start|>user" . But when I train the model, I encounter exploding gradient. Is there anything I need to take care when I generate train data of Qwen2?

You can refer to eagle/ge_data/ge_data_all_qwen2.py.


[LUIS] luis models list gives 404 if app id is not specified

This is as designed - The endpoint should handle erroneous values for the appId and versionId


Implement better tracking of mod MultiplayerTypes for users

Currently MPTypes are shown for individual mods with a VoidPlugin class, however detection requires reading the mod description or the VoidManager mod info page for every mod.
To improve this:

Mods without a VoidPlugin class should be listed with an easy marker to detect they are Unspecified
An overall MPType should be displayed, for easy detection of whether the current mod list can join vanilla games
A list should of mods incompatible with vanilla clients should be shown

All of this could be done in the F5 menu, handled through a mix of color codes and symbols, as well as a new area for these displays.

As of update 1.0.6, Mods without a VoidManager class are now listed in the F5 menu, The overall MPType is provided, MPTypes have an explanation provided, and the mod names in the F5 menu are color coded based on their MPType.
This is a little different from the initial idea, but should cover the necessities.


[WIP] Sign beta versions (bug 1160593)

I'm attempting to get this to:

Sign new beta files with the prelim cert
Add them to the prelim queue

I'm getting hung up on the tests. I'll keep trying to work through it but something somewhere is changing statuses on me and I can't find it. Maybe a signal somewhere that isn't clear.

The bug requirements changed quite a bit so closing this.


[hotfix] Return correct value for images (array with meta)

Finally green lights. This one and #355 took waaay more effort than I expected. 😓

Made the requested changes, and I think it's a bit cleaner now! 👍

🎊 🎉

Gihub says "1 check was pending", but it was finished with green lights: https://travis-ci.org/bolt/four/builds/508871291


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-08-17)
⚠ BREAKING CHANGES

broken 'go list' report on some systems (requires nvim 0.10.0) (#167)

Bug Fixes

broken 'go list' report on some systems (requires nvim 0.10.0) (#167) (26d937f)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


Fix watch (KVsToFeatureMap) to respect the namespace

depeds on #70 to pass in CI

@promiseofcake can you review this change ?

Sorry for the delay on this! The code change looks good to me - would it be possible to add a test for the change in functionality?

I could give it a try, but seem like none of the tests does those:

configure anything other then default
Save Info (sha and timestamp) into the mock storage, and not flags

again my go is a bit rust ;)

@promiseofcake :+1:


DOMEventEmitter

Either this class constructor needs to become its own project and then be referenced as a dependency, or an existing library which can do the same thing needs to be considered.

Fixed in 8b3f2fcb689228d765455b40b94cfffbb1813fbf