Extension causes high cpu load

Issue Type: Performance
Extension Name: vscodeintellicode
Extension Version: 1.1.3
OS Version: Linux x64 4.15.0-29deepin-generic
VSCode version: 1.31.1

:warning: Make sure to attach this file from your home-directory: /home/dragonf/VisualStudioExptTeam.vscodeintellicode-unresponsive.cpuprofile.txt :warning:
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Thanks for the report! Closing as a duplicate of #60. This should be fixed soon with version 1.1.4 of the extension.


watch and trace command line parse error with space in the quotation such as "{params, resultObj}"

When execute command such as [watch test.Class test "{params[0], throwExp}" -e], then will failed.
The "{params[0], throwExp}"  with a space cased.

Now delete the space will be ok.

fixed in v1.0.7


Add a PoE Sigmoid breaking point util

Some maximum values of stake for the Sigmoid function.
Breaking point is inversely correlated with the p exponent.
$ cargo run --example poe_sigmoid_breaking
...

Sigmoid function breaking checks:
Sigmoid(p=0.68, s=0.00003)(stake=32320000, engagement=50000) broke.
Sigmoid(p=0.55, s=0.00005)(stake=1924550000, engagement=50000) broke.
Sigmoid(p=0.57, s=0.00002)(stake=909000000, engagement=50000) broke.
Sigmoid(p=0.72, s=0.000001)(stake=12370000, engagement=50000) broke.
Sigmoid(p=0.59, s=0.00003)(stake=451740000, engagement=50000) broke.
Sigmoid(p=0.62, s=0.00001)(stake=172230000, engagement=50000) broke.
Sigmoid(p=0.741, s=0.000001)(stake=7790000, engagement=50000) broke.
Sigmoid(p=0.54, s=0.00005)(stake=2859310000, engagement=50000) broke.
Sigmoid(p=0.56, s=0.00002)(stake=1313830000, engagement=50000) broke.
Sigmoid(p=0.707, s=0.000001)(stake=16700000, engagement=50000) broke.
$

As long as there are not more than 7M TGD in stake, the original Sigmoid works. That value increases for lower exponents.

Thanks.
As an addendum, here's a fitting of the breaking point of the sigmoid function as a function of the exponent p:


Bug/#680

Fix for bug #680 that includes a step for #329. Because I'm VERY clumsy with git. The #329 part is safe to PR.

not sure what the relation is to #329 which is discussing zoom levels. The solution for that is to ditch android view scroller and maintain our own pixel x,y offsets of the world. I'm fix with the change, but it's only tangentially related to the zoom issue

@spyhunter99 there's definitely no connection between #329 and #680. The thing is that I was working on #329 yesterday, spotted #680 this morning and decided to quickly fix it. The initial idea was to start from scratch for #680. And when I saw that the #329 step was included too in the PR, my first reaction as an adult was to yell. Then I failed to have a clean #680-only PR. What I committed in #329 is atomic, works and fixes another bug.

haha no worries

@spyhunter99 Is it correct, that only commit 1b2af6e got merged (bug #680) and not cd618a6 (step for feature #329)?
@monsieurtanuki Does the commit cd618a6 (step for feature #329) actually repair the map rotation issue (now around map center instead of upper left corner) when tilting the device? This has been bugging me for some time now too and I was happy that you seemed to do something about it and eager to test, but no change probably due to not having been merged..?

@secaw That's partially correct: the specified commit cd618a6 repairs the map rotation issue, but only in the StarterMapActivity demo. If you want to test it in another activity, you need to add some code, basically:

save the map center latitude and longitude in prefs inside onPause (instead of PREFS_SCROLL_X) (I recommend using OpenStreetMapConstants.PREFS_LATITUDE_STRING and storing as a String, because double cannot be saved in prefs and float are not accurate enough)
load the map center latitude and longitude from prefs inside onActivityCreated, and call new method MapView.setInitCenter

The commit is safe to merge btw. But it was very remotely connected to #680, and not even that close to #329 either!

@monsieurtanuki Thanks for clarification and the proposal. Unfortunately I'm overriding orientation config changes in my map activity, so I can't do it that way, but I get the idea - will figure it out. I migrated from Google maps V1 which rotated around the map center automatically..

@secaw i'm not sure, i merged all the open PRs. If https://github.com/osmdroid/osmdroid/commit/cd618a6251498fea4d3493c780373b14fd60b133 wasn't on a PR it probably wasn't merged.
Related: there was a related branch for ticket #155 that someone opened a PR for. Unfortunately there was no sample code. Basically the idea was to offset my location to be closer towards on the bottom of the screen and still support heading/course up on the map. It only partially worked and was never merged.

@monsieurtanuki @spyhunter99
Got it! This little trick with the new MapView.setInitCenter does it for me. Rotates nicely around the map center - perfect!
@Override
public void onConfigurationChanged(Configuration newConfig) {
  super.onConfigurationChanged(newConfig);
  if(newConfig.orientation != map_orientation) {
    map_orientation = newConfig.orientation;
    mapView.setInitCenter(mapView.getMapCenter());
  }
}


youtube-dl: 2016.04.19 -> 2<IP_ADDRESS>

Motivation for this change
Updating to new version.
Things done

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

[ ] NixOS
[x] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thank you


RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [41, 4, 4]], which is output 0 of AsStridedBackward0, is at version 1; expected version 0 instead.

Hi, thanks for the excellent work. However, during test as proposed  "python3 main.py test-dataset/chess/seq-01.mkv --name test --pose_graph", I encountered a runtime error. It seems untraceable even when I set detect_anomally to "True". Has anyone encountered the same problem? Here are the outputs in the shell.
=>Load models
  Loading weights: weights/models/midas_v21-f6b98070.pt
=>Reconstruct keyframes
    0%|                    | 0/1200 [00:00<?Traceback (most recent call last):
  File "main.py", line 249, in <module>
    vd.process_keyframes()
  File "main.py", line 82, in process_keyframes
    keyframes = self.sfm.reconstruct_images(
  File "/home/pc/GCVD-release/sfm.py", line 113, in reconstruct_images
    loss['full'].backward()
  File "/home/pc/.conda/envs/GCVD/lib/python3.8/site-packages/torch/_tensor.py", line 396, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
  File "/home/pc/.conda/envs/GCVD/lib/python3.8/site-packages/torch/autograd/__init__.py", line 173, in backward
    Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [41, 4, 4]], which is output 0 of AsStridedBackward0, is at version 1; expected version 0 instead. Hint: the backtrace further above shows the operation that failed to compute its gradient. The variable in question was changed in there or anywhere later. Good luck!
    0%|                    | 0/1200 [00:01<?

Seems inverse mat function in torch 1.12 version appear to be inplace op. (Maybe not right  :! )

However，the pose optimization result seems not right

I had the same problem ,How did you solve it?

I had the same problem ,How did you solve it?


Start testing UTF-8 handling.

create some files with UTF-8 encoded names
add a simple test to see if we process them correctly
templatize the manifest golden tests.

Future plans:

Integration tests with pkg_tar/zip/...
pkg_mkdir of UTF-8 string

Tests fail on Windows in a way that looks like a Bazel problem.
(04:43:10) ERROR: C:/b/bk-windows-fzgx/bazel/rules-pkg/pkg/tests/testdata/utf8/BUILD:18:10: //tests/testdata/utf8:files: invalid label '4-?' in element 3 of attribute 'srcs' in 'filegroup' rule: invalid target name '4-?': target names may not contain '?'

On hold until #393 is merged.


Form feedback on FS.Collection filter error

For example, if a maxSize filter fails it would be nice to automatically show feedback to the user.

Yes this would be really helpful.


Better invocation header

The invocation header
https://github.com/FStarLang/karamel/blob/b76942a3fa36c016a7fb1da96ce924c71964488a/lib/Output.ml#L60-L62
leads to unnecessary noise because it uses absolute paths for the invocation. How about making this relative to stay the same across different machines?
Further, the F* version is <unknown> when a release (non-git) version is used. Instead of trying to get it from git directly, something like fstar.exe --version | grep commit | sed 's/commit=\(.*\)/\1/' should work.

karamel itself has an option called -header where you can pass something custom -- eurydice doesn't expose this option, would it help?

Yeah that's an option as well. I think cleaning up the default would be nice as well, but exposing the -header option through eurydice we could set something less noisy ourselves.

https://github.com/AeneasVerif/eurydice/pull/43 should fix this


Sort Hub search results by number of projects using the plugin

As a way to naturally give proper prioritization of widely used and successful plugins on the Hub, we can sort search results output based on the "Projects (last 3 months)" count for each.
At least in most cases, this sort would naturally bring the best result to the top of the results - without having to build a complex algorithm.
Since taps and targets almost always have more results than utilities, files, and transformers, this would almost certainly solve this one as well:

https://github.com/meltano/hub/issues/1057

Search is hard - I worry if we weight it too much by output then legitimate matches would be pushed down too far. Hard to know w/o digging in deeper though. (I spent too much time in a past life fine tuning elasticsearch with customized tokens and stems 🤪 )

Duplicate of https://github.com/meltano/hub/issues/862


Authentication state is showing "logged out" after restart of Podman Desktop

The extension claims to be logged out after a restart.  I'd love us to find a way to improve on that.  There may be ways to store/cache the refresh token or to check whether we're logged into registry.redhat.io and do a status check of subscription-manager inside the VM.  There may be other options as well.

Ignore until https://github.com/redhat-developer/podman-desktop-redhat-account-ext/pull/35 has merged.
depends on https://github.com/containers/podman-desktop/issues/6044.
Cc: @dgolovin @benoitf @jeffmaury @slemeur

This issue also introduces another unwanted behavior. If I am successfully logged into registry.redhat.io after first authentication using SSO (which is expected?), I am not logged out of if I "sign off" from the SSO. I am still logged into registry.
After application restart, I am still logged into registry, but I logged off from the SSO (no matter if I wanted or not) -> I log into SSO again, but this time new registry record appears.

@dgolovin is the secure-storage feature tracked somewhere in Podman Desktop? Can we link it here?

secure storage has been implemented there https://github.com/containers/podman-desktop/pull/6422 and provided to extensions there https://github.com/containers/podman-desktop/pull/6423 so I think @dgolovin now you can store the token there

Thanks, @benoitf !  We're getting really close :)


Paradiso and PyMKL

Awesome - glad this is getting used.
I'd prefer an import from pyMKL rather than this duplication of code.  There are still a number of improvements I need to make to the paradiso wrapper and it would be easier if everything was in one place.  I know extra external dependancies are annoying but I can get pip setup if that helps.

If it was on pip and versioned then I would absolutely use it. I will close this and revert the changes.
Having trouble with getting travis to trigger. Not sure if anyone has any ideas there?

It is possible that Travis did not pick up on the transfer to the simpeg organization. I think I will transfer it back and fork instead. maybe that will help?
Thanks @dwfmarchant for offering to get this on pip, that would be super helpful!


remove extends Promise<T> from RedditContent

Reddit content should be the body of a promise, not an extension of a promise. This addresses #221 and enables
const user = await snoowrap.getUser()

Whoops, missclick - ignore that close and reopen.

It seems like this would solve some issues but not all. For example, snoowrap makes use of proxies which makes it possible to fetch items without having to await or use promises explicitly.
Your proposal means this will throw an error
export function multireddits (username: string): Promise<MultiReddit[]> {
  return r.getUser(username).getMultireddits();
}
export function user (username: string): RedditUser {
  return r.getUser(username);
}

I for one would welcome a breaking change that removed all the proxy logic in favor of explicit fetch semantics. IMO too much hand-waving magic leads to issues like this one and also makes it really confusing as to whether or not any given property reference will result in async behavior. Namely, will the following result in a network call or not?
function someUser(): RedditUser;

async function printName() {
  const user = someUser();
  const name = await user.name;
  console.log(name);
}

There's no way of knowing without looking to see if someUser() performs a fetch, which means that every access to every RedditContent object has to be treated as if it's async, even if it's not. This feels to me like an anti-pattern. I would much prefer if lazy loading were either removed entirely or split into a wrapper type like below or some other way of making it explicitly clear at the type level whether or not the object is fetched.
// Note: This is a very naive solution that I have not tested at all. It's just meant to be a proof-of-concept
interface PartialComment {
  id: string
  fetch(): Comment
}

This reply got much larger (both in length and scope) than I was expected when I started. If you want I will split it off into it's own issue, and I am willing to start work on it if anyone else would like it.

In hindsight, if I were recreating the library from scratch I might get rid of the proxy logic too. As a bit of context, when this library was originally created, most platforms didn't support async/await. So in order to "chain" calls, it would have been necessary to do something like
r.getUser('foo').getSubmissions().then(fetchedUser => {
    return fetchedUser.posts[0].fetch().then(fetchedPost => {
        return fetchedPost.comments.fetchAll().then(fetchedComments => {
            // ...
        });
    });
})

(It's also possible to do this without so much nesting, but that's beside the point.)
Promises, and monadic promise chains, were fairly difficult to use correctly before async functions were widespread. For example, forgetting any of the returns in the code block above might cause errors to be silently ignored. This would particularly be an issue for this library, given that writing reddit bots is many peoples' first introduction to any kind of programming. It seemed like asking those people to understand monadic error handling would have been a heavy lift. This is now less of a problem since people can use async/await.

There's also a separate issue in that reddit's API was very inconsistent (and AFAIK, this is still the case). In many places, it only returns a subset of the properties of any given object, and it returns a different set of properties in different places. (In general, this would make the type signatures pretty complex.)
One of the goals of this library was to paper over the idiosyncracies of the API and allow people to write high-level code (say, "just get me this user's karma, I don't care really whether it's included in the initial API response or somewhere else"). This design is well-suited to the goals of many bot authors, and I think overall it gets rid of a lot of complexity. However, it might be less well suited to users who care about the specifics of when network requests are required.
I originally created this library over 5 years ago, and since then my taste in API/language design has moved much more towards the "prefer robustness over magic" end of the spectrum. I would now definitely agree with you that "getters sometimes trigger network requests" is an antipattern. That said, I'm not sure that dumping all of the complexity of "which fields reddit provides at which times" into type definitions for the user would be a good solution either.
If you want to disable the "magic network requests" feature at runtime, you could use Snoowrap#config({proxies: false}). However, that probably doesn't help with type definitions.
If we were going to do a breaking change, maybe it would be feasible to just replace every property with a method that returns a Promise (which sometimes immediately fulfills). Now that async/await is common, an API like that might actually be tenable to work with, and it would simplify the type definitions. (However, (await (await (await user.name()).submissions()).comments())[0] is still kind of a mouthful, and doing a breaking change like that at this point would be very disruptive -- effectively requiring every piece of code using snoowrap to be rewritten -- so I'm not sure it would be worthwhile at this point.)

@not-an-aardvark that makes perfect sense. My personal design taste has shifted wildly over the years as well. I have used {proxies: false} in my bots, but unfortunately that doesn't help when writing in modern TypeScript, as #221 can attest.
The way I see it, there are a few options:

Leave everything the way it is

pros: really easy
cons: the only way around #221 is to use // @ts-ignore (or some variant thereof)

Do something like what @bluecomm42 suggested -- rip apart the whole system and put it back together keeping sync and async firmly seperated

pros: it would absolutely fix the issue at hand
cons: it would also be a very major change, likely requiring a nearly full rewrite, not to mention rewriting every bot  using it if they want to stay up-to-date

Make it all async, like you suggested

pros: this would also fix the problem
cons: it would also be a major change, possibly even more work than option 2

Just start over

pros: clean slate, learn from all the mistakes / new apis / etc.
cons: the most work of all the options, but also the most rewarding

I don't think there is a "right" option, but in terms of lesser-evils I am actually leaning towards the fourth option. So much so that I have actually started work on a potential replacement (plus lockdown has me bored out of my mind, so it's something to do). Trying to retrofit these sorts of designs into a codebase as old and battle-tested as this one is hard. Having a clean slate every once in a while is probably a good idea. Plus this way we can keep the nice magic chainable version around for those who want it while simultaneously offering a lower-level, more robust interface for those who prefer it.

The most difficult thing here isn't necessarily the work itself, but the lack of contributors. Rewriting the async functionality has relatively little gain for the amount of work needed to make it work, not to mention warranting a major version bump.
It will most likely be fixed when someone gets annoyed enough that they'll spend a weekend rewriting it

Yeah I agree. That's why I think options 1 and 4 above are better than 2 or 3. Plus the rewrite I'm doing is going to be fully in TypeScript from the start, so no more grafting definitions into a legacy codebase.

It sounds like the consensus is just add ts-ignore / ts-expect-error as anything else would be too much work. @franklinharvey I'd say go ahead and close it.
That being said, development on snoots is well underway. It's nearing v1.0.0-dev.1 and I'd love for people to join the discussion on how to shape it! (This will be the last time I mention it unprompted here, I promise. 😁)


Canonical serialization

Resolves #1

The test-map branch now contains a nested. You should add a test for the recursive case.


Upgrade sumo_db dep to 0.7.3

I did my own fork and did the update to start using sumo_db_mysql with count_by/3. Maybe you can find some hints of what was required to get the test working:
https://github.com/Svampen/sumo_db_mysql/tree/upgrade_sumo_db_0.7.3


[FLINK-17516] [e2e] Exactly-once verification E2E

This PR adds an E2E test that verifies exactly-once semantics with failure recovery.
It is based on #109 so that this new E2E test is also run in Travis.

Please see the class level docs of ExactlyOnceVerificationModule and ExactlyOnceE2E on the specifics of the app used for verification, and the verification scenario.
Change log

8321db2 is a refactoring of StatefulFunctionsAppContainers. While extending that class for extra functionality required by this new E2E, it was obvious that the class is growing to big and bundling too many responsibilities (test runtime functionality, and pre-test configuration). This commit refactors the class using the builder pattern.
fcb0782 to 5ff971f is enabling checkpointing in apps run by StatefulFunctionsAppContainers.
82867ea Adds a utility method to restart specific workers at test runtime.
8a5e5ca Adds the function modules / app to be used for verification
302db5b The actual E2E test verification scenario.

Verifying
The Travis build should pass.
In the exposed master logs, you should see that the job fails due to a lost worker, is recovered and restarts.

It seems like the tests are failing because the temporary checkpoint directories for E2E tests cannot be deleted. Mostly a permission issue in the Travis build environment.

@igalshilman I've addressed all comments except from the one about using Docker named volumes. Please let me know what you think, and if there are no further objections, I'll merge this PR. Thank you!


MariaDB Database goes up & down, up & down

The database is super flaky, it dies and restarts over & over.
I've created two or three of the Wordpress on Linux Web App instances and all behave the same.
This means you can't really use the service at all!
Instances are running in West Europe, on a S1 plan
Logs look like this
Digest: sha256:bed0cf5a9fd13e0053b45fb206ec839728bfa3f3fa1a08f5725538e3fded4a5f
Status: Downloaded newer image for appsvc/apps:wordpress
2017-04-14T09:10:54.966743189Z  * Starting MariaDB database server mysqld
2017-04-14T09:10:58.095089766Z    ...done.
wordpress: Pulling from appsvc/apps
Digest: sha256:bed0cf5a9fd13e0053b45fb206ec839728bfa3f3fa1a08f5725538e3fded4a5f
Status: Image is up to date for appsvc/apps:wordpress
2017-04-14T09:33:46.525490347Z $connectstr_dbhost = '<IP_ADDRESS>';
2017-04-14T09:33:46.593063517Z  * Starting MariaDB database server mysqld
2017-04-14T09:34:17.207564656Z    ...fail!
wordpress: Pulling from appsvc/apps
Digest: sha256:bed0cf5a9fd13e0053b45fb206ec839728bfa3f3fa1a08f5725538e3fded4a5f
Status: Image is up to date for appsvc/apps:wordpress
2017-04-14T09:43:34.886610762Z $connectstr_dbhost = '<IP_ADDRESS>';
2017-04-14T09:43:34.931979535Z  * Starting MariaDB database server mysqld
2017-04-14T09:43:38.021119734Z    ...done.
wordpress: Pulling from appsvc/apps
Digest: sha256:bed0cf5a9fd13e0053b45fb206ec839728bfa3f3fa1a08f5725538e3fded4a5f
Status: Image is up to date for appsvc/apps:wordpress
2017-04-14T09:56:06.955670300Z $connectstr_dbhost = '<IP_ADDRESS>';
2017-04-14T09:56:07.143725080Z  * Starting MariaDB database server mysqld
2017-04-14T09:56:39.111825579Z    ...fail!
wordpress: Pulling from appsvc/apps
Digest: sha256:bed0cf5a9fd13e0053b45fb206ec839728bfa3f3fa1a08f5725538e3fded4a5f
Status: Image is up to date for appsvc/apps:wordpress
2017-04-14T09:57:37.226469298Z $connectstr_dbhost = '<IP_ADDRESS>';
2017-04-14T09:57:37.338712792Z  * Starting MariaDB database server mysqld
2017-04-14T09:57:41.066693630Z    ...done.
wordpress: Pulling from appsvc/apps
Digest: sha256:bed0cf5a9fd13e0053b45fb206ec839728bfa3f3fa1a08f5725538e3fded4a5f
Status: Image is up to date for appsvc/apps:wordpress
2017-04-14T10:05:36.783143896Z $connectstr_dbhost = '<IP_ADDRESS>';
2017-04-14T10:05:36.932770171Z  * Starting MariaDB database server mysqld
2017-04-14T10:06:08.660148791Z    ...fail!
wordpress: Pulling from appsvc/apps
Digest: sha256:bed0cf5a9fd13e0053b45fb206ec839728bfa3f3fa1a08f5725538e3fded4a5f
Status: Image is up to date for appsvc/apps:wordpress
2017-04-14T10:13:07.360744132Z $connectstr_dbhost = '<IP_ADDRESS>';
2017-04-14T10:13:07.541001115Z  * Starting MariaDB database server mysqld
2017-04-14T10:13:11.588512040Z    ...done.
wordpress: Pulling from appsvc/apps
Digest: sha256:bed0cf5a9fd13e0053b45fb206ec839728bfa3f3fa1a08f5725538e3fded4a5f
Status: Image is up to date for appsvc/apps:wordpress
2017-04-14T10:16:48.530868065Z $connectstr_dbhost = '<IP_ADDRESS>';
2017-04-14T10:16:48.668271531Z  * Starting MariaDB database server mysqld
2017-04-14T10:17:20.291817053Z    ...fail!
wordpress: Pulling from appsvc/apps
Digest: sha256:bed0cf5a9fd13e0053b45fb206ec839728bfa3f3fa1a08f5725538e3fded4a5f
Status: Image is up to date for appsvc/apps:wordpress
2017-04-14T10:22:09.727332858Z $connectstr_dbhost = '<IP_ADDRESS>';
2017-04-14T10:22:09.909137414Z  * Starting MariaDB database server mysqld
2017-04-14T10:22:14.277322997Z    ...done.

Further testing with app plan in both "West US" and "West Europe", and swapping between B1/B2 and S1/S2 tiers
The problem only seems to happen when on a S1 or B1 tier - could be a memory problem?
Speculating that 1GB of memory isn't enough and MariaDB falls over?

Now Azure Database for MySQL has been released I would suggest getting rid of MariaDB and local redis and replace the Dockerfile so it works only with the PaaS services?
Current implementation is cool but does not scale so it will only be useful for testing purposes. Moving to the PaaS services would work for production.

@benc-uk We have made some imrpovements with a new base image https://github.com/Azure-App-Service/apps/tree/master/apache-php-mysql/0.1 which allows u to host any PHP/MySQL app on Linux web app.


cmake: use GNU install dirs (should work on Windows too)

This should fix the installed location of s2geography.dll on Windows (in the bin subdirectory instead of lib).

Thanks!
(I copied the configuration from s2geometry, which seems to work on linux / macos / windows for the conda packages).


Snapshot using a react native ListView fails with unique "key" error

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Adding a snapshot test using a ListView fails with the following error:
console.error node_modules/react-native/Libraries/Core/ExceptionsManager.js:71
      Warning: Each child in an array or iterator should have a unique "key" prop.
      Check the render method of `ScrollViewMock`.
      It was passed a child from ListViewMock. See https://fb.me/react-warning-keys for more information.

Code snippet:
export class App extends Component {
  state = {
    dataSource: new ListView.DataSource({ rowHasChanged: (r1, r2) => r1 !== r2 }),
  };

  componentWillReceiveProps({ results }) {
    this.setState({
      dataSource: this.state.dataSource.cloneWithRows(results),
    });
  }

  renderHeader = () => (
    <Header />
  );

  renderRow = resultInfo => (
    <ResultCell resultInfo={resultInfo} />
  )

  renderFooter = () => {
    const { results } = this.props;
    if (results.length >= results.totalItems) {
      return null;
    }

    return <ActivityIndicator />;
  }

  render() {
    return (
      <ListView
        dataSource={this.state.dataSource}
        enableEmptySections
        renderHeader={this.renderHeader}
        onScroll={Keyboard.dismiss}
        renderRow={this.renderRow}
        renderFooter={this.renderFooter}
      />
    );
  }
}

Test:
test('renders correctly', () => {
  const tree = renderer.create(
    <App results={[]}  />
  ).toJSON();
  expect(tree).toMatchSnapshot();
});

jest: 19.0.2
yarn: 0.21.3
os: macOS

This is a problem with the mock inside of react-native, not a Jest issue. Please file an issue there or send a PR.

@cpojer do you know if anyone filed an issue or a PR? This is an issue with the latest and I would love to run jest on my react-native project.

@lyndsey-ferguson I submitted a PR https://github.com/facebook/react-native/pull/12767
Not much attention from RN though.

Sigh. I guess I just have to ignore the error then.

I'm seeing this issue as well with React Native 0.47.1. Is there a way to fix it?


Feature request: required prop & focus indicator

It would be great if basic required validation was added. E.g:
<FileUploader required={isRequired} />

add required as a prop
apply required prop to input field
change input css from display: none to display: block
add input css opacity: 0;
add input css position: absolute;
add input css pointer-events: none;

The CSS changes mean:

Native browser validation prompts display when required is set to true.
The dropzone can receive keyboard focus and have a visible focus indicator (for accessibility, see WCAG 2.4.7)

fixed in version 2.3.9


ArgoCD UI does not show the generated Terraform resource

I am trying the demo terraform deployment using the instructions provided in the docs. While everything works as expected with the Terraform resource being reconciled by the tf-controller, the ArgoCD UI does not show me any of the (Kustomize) generated resources which in this case would be the resources created by the GitRepo object and the Terrafrom resource.
This is my view

I see the Kustomize resource icon which means Argo recognizes the flux objects but not the terraform resource type.

Turned out to be a caching issue with the git branch. I moved to a new branch and things worked just fine. Closing this out.


Inconsistent canonicalization of FSim angles

Description of the issue
FSimGate and PhasedFSimGate disagree on whether or not theta and phi angles that are used by __init__ and are exposed as properties are canonicalized (by projecting them into [-pi, +pi]) or not.
How to reproduce the issue
In [1]: import cirq, numpy as np

In [2]: for t in (-3/2, -1/2, 1/2, 3/2):
   ...:     theta = np.pi * t
   ...:     g = cirq.FSimGate(theta, 0)
   ...:     print(theta, g.theta)
   ...:
-4.71238898038469 -4.71238898038469
-1.5707963267948966 -1.5707963267948966
1.5707963267948966 1.5707963267948966
4.71238898038469 4.71238898038469

In [3]: for t in (-3/2, -1/2, 1/2, 3/2):
   ...:     theta = np.pi * t
   ...:     g = cirq.PhasedFSimGate(theta, 0)
   ...:     print(theta, g.theta)
   ...:
-4.71238898038469 1.5707963267948966
-1.5707963267948966 -1.5707963267948966
1.5707963267948966 1.5707963267948966
4.71238898038469 -1.5707963267948966

Cirq version
0.10.0.dev

The behavior of canonicalizing on initialization is different from other gates:

In cirq.PhasedXZGate, canonicalization is in the [equality check]((https://github.com/quantumlib/Cirq/blob/d967317cdb0f954e14907f69aaae70fd1f76ff6f/cirq-core/cirq/ops/phased_x_z_gate.py#L105) and construction from matrix.

cirq.EigenGate also only canonicalizes in the equality check.

Should we make this consistent across Cirq gates so as to not surprise users?

@verult Do you have an example of where this becomes an issue? Regarding consistency, I would consider things to be inconsistent if the same parameter (eg: exponent) is treated differently in similar gates (eg: X/Y/Z).
So, for example, if X ** 2.5 implicitly became X ** 0.5 because we canonicalized the exponent during construction, but didn't do so for Z ** 2.5.
FSim gate has different parameters (theta, phi) and till the time it's consistent with what happens in PhasedFSimGate, I don't see much issue here. Also, changing behavior of EigenGate will be a large breaking change, which I'd prefer to avoid unless we have a good reason to do so.

Not a major issue for me. For context, I discovered this while implementing PhasedXZGateFamily using FSimGateFamily as an example, and a check like this won't work for the PhasedXZGate because exponents aren't canonicalized on initialization. But we could easily fix this by instantiating a dummy PhasedXZGate with 0 angles and do a gate equality check.

Regarding consistency, I would consider things to be inconsistent if the same parameter (eg: exponent) is treated differently in similar gates (eg: X/Y/Z).
Both exponents and FSimGate angles are derived from angles at the end of the day, and are canonicalized based on angles. So to me keeping their canonicalization process consistent makes sense to me, but I'm not a user so I don't have a strong preference :)

Agree that changing EigenGate would be big. If we do want to do this, changing FSimGate and PhasedFSimGate might be easier.

Discussed at Cirq sync: an argument could be made for this as a cleanup item, but leading into 1.0 there isn't a strong enough motivation to justify directing time towards it.


add_virtual_server_subclient does not support chained content rules

I need to create a new VMware subclient with cvpysdk, and the method add_virtual_server_subclient does not support creating a content rule like this:

When I add two tags as content, it will be created as two separated rules:

storage_policy = 'storage_disk'

vmContent = [              {'allOrAnyChildren': True,
                             'displayName': '744',
                             'equalsOrNotEquals': True,
                             'name': '744',
                             'path': '',
                             'type': VSAObjects.Tag
                            },
                            {'allOrAnyChildren': True,
                             'displayName': 'CV-VM-1d-185days',
                             'equalsOrNotEquals': True,
                             'name': 'CV-VM-1d-185days',
                             'path': '',
                             'type': VSAObjects.Tag}
            ]
results = comm.clients.get('vc01.example.dk').agents.get('virtual server').instances.get('vmware').backupsets.get('defaultBackupSet').subclients.add_virtual_server_subclient('Test1',vmContent,storage_policy=storage_policy,description="Auto generate subclient")

It is not possible to add content with  'children'. It would be very nice to have this supported.

Please try this with latest changes on master branch


Implement MCORR, DCORR, Mantel (Bear)

[ ] DoD: produce relevant parts of fig. 2

@junhaobearxiong are you also going to implement graph testing now?

@jovo I’m planning to implement the core functionalities like permutation tests and power computation first. But if graph testing is something we want in the package (does graspy plan to implement similar things?), I will also implement it afterwards.

graspy will implement related things.
there is a nice intersection of graspy and mgc at testing whether graphs
are independent...
On Mon, Oct 1, 2018 at 9:16 AM Junhao Xiong<EMAIL_ADDRESS>wrote:

@jovo https://github.com/jovo I’m planning to implement the core
functionalities like permutation tests and power computation first. But if
graph testing is something we want in the package (does graspy plan to
implement similar things?), I will also implement it afterwards.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NeuroDataDesign/mgcpy/issues/13#issuecomment-425903122,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACjcnWtjh_aIDUnYk_5OlQbuG1hTLluks5ughVggaJpZM4WfrDE
.

--
the glass is all full:  half water, half air.
neurodata.io


Bug?: Optional Tuple Values

I'm in a situation where I'd like to send an optional number of arguments in an array:
const schema = tuple([
	string(),
	number().optional()
])

type ZodType = z.infer< typeof schema >

// Conversion of type '[string]' to type '[string, number | undefined]' may be a mistake
// because neither type sufficiently overlaps with the other.
const ar = [ "foo" ] as ZodType

Is it possible to infer to [ string, number? ] instead of [ string, number | undefined ] ?

it's not exactly the same, but this might work for your use case:
const schema = z.union(
  [
    z.tuple([
      z.string(),
    ]),
    z.tuple([
      z.string(),
      z.number()
    ]),
  ]
);

type ZodType = z.infer< typeof schema >;
// type ZodType = [string] | [string, number]

This is perfect thanks @ealmansi
As a workaround, is it possible to treat a tuple as an object with index properties? Ex:
object({
	0: string(),
	1: number().optional()
})

For your solution, my actual tuple schema is more complex so copying/pasting it is a little painful in terms of staying dry. Is it possible to clone a tuple (or any schema) and then omit the index? For example:
const schema = tuple([
	string(),
	number()
])
const combinedSchema = union([
	schema,
	schema.omit({2: true})
])

Thanks for your help!

@stockson looks like your workaround works just fine:
const argsSchema = z.object({
  0: z.string(),
  1: z.number().optional()
});
type Args = z.infer<typeof argsSchema>;
const args1: Args = ["str"];
const args2: Args = ["str", 23];

otherwise you can remove duplication in the schemas with a bit of boilerplate:
const arg1 = z.string();
const arg2 = z.number();
const arg3 = z.date();
const args = z.union([
  z.tuple([arg1]),
  z.tuple([arg1, arg2]),
  z.tuple([arg1, arg2, arg3]),
]);
type Args = z.infer<typeof args>;

Working as intended, thanks everyone for the helpful examples and discussion.


Update Cortex with the latest engine file structure.

Description
We will likely use only one instruction binary file.
Per https://github.com/janhq/cortex.llamacpp/pull/176

Deprecated, will create more relevant issue in the future


Add Use cases to DG

Give a few representative use cases (textual form) that need multiple steps to complete. e.g. Adding a tag to a person (assume the user needs to find the person first)

From the 2103t website, it mentions only need to add few representative use cases. Hence, 8 use cases are added only.


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter tvprogram for common errors and appropiate suggestions to keep this adapter up to date.
Please see the result of the check below.
ioBroker.tvprogram
   -

ERRORS:

[ ] :heavy_exclamation_mark: [E162] js-controller 3.3.22 listed as dependency but 4.0.24 is required as minimum, 5.0.19 is recommended. Please update dependency at io-package.json.
[ ] :heavy_exclamation_mark: [E204] Versions "1.1.3-alpha.1, 1.1.3-alpha.0, 1.1.1" listed at common.news at io-package.json do not exist at NPM. Please remove from news section.

WARNINGS:

[ ] :eyes: [E145] Missing mandatory translation into de of some "common.news" in io-package.json.
[ ] :eyes: [W040] "keywords" within package.json should contain "ioBroker"
[ ] :eyes: [W105] Missing suggested translation into uk of "common.titleLang" in io-package.json.
[ ] :eyes: [W109] Missing suggested translation into uk of "common.desc" in io-package.json.
[ ] :eyes: [W145] Missing suggested translation into ru,pt,nl,fr,it,es,pl,uk,zh-cn of some "common.news" in io-package.json.
[ ] :eyes: [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.
[ ] :eyes: [W184] "common.title" is deprecated and replaced by "common.titleLang". Please remove from io-package.json.
[ ] :eyes: [W202] Version of package.json (1.1.3-alpha.1) doesn't match latest version on NPM (1.1.2)

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be considered as a suggestion and be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence

i have to wait till vis-2 >=v2.10 is released and available in the repo


Implement top-level Node or "Host"

This is what brings it all together into a nice usable format.
See: https://github.com/libp2p/go-libp2p/tree/master/examples/libp2p-host

For this one, it is definitely better to take a page from the js-libp2p book instead of the go-libp2p book.
In js-libp2p, a Node is a class that lets you instantiate libp2p Nodes with your own libp2p recipe. Read more at the README here https://github.com/libp2p/js-libp2p#bundles

In the case of rust-libp2p, I guess that the host library would simply provide an implementation of the Transport trait (TCP+Secio+Multiplex+p2p-circuit in the future).
If it makes sense, this library could use conditional compilation to automatically use TCP on regular operating systems, and websockets on emscripten.

I think that it makes sense to have a library that automatically uses TCP on desktop and websockets on emscripten, and handles making progress on futures (ie. wraps around tokio).
However if the only concern is usability, then I think that the libp2p-swarm library is already simple and clear enough.

Let's close this issue.


Pajantom

Really minor one here and I don't put you at fault at all (since the Pokemon just released yesterday) but Pajantom is a CAP mon that is finished, and it's something I'd like to see added!
Have some art: https://i.imgur.com/PUOV3wp.png

Thanks, I'll do it in a few hours. Do you have the type for me?

Dragon/Ghost :)

Dragon primary? (It matters)

Yep, Dragon's the primary type.

Looks like it's working for adfe453f145af937


@datasets feature branch

Public RFC: Datasets on Workflow SDK
This is the first commit.  The next Pull Request will be for plugins see Plugin Design appendix in the Public RFC
Status:

[x] @sruthinagalla and I have discussed plugin design in Public RFC: Datasets on Workflow SDK, see link and link
[x] When plugin design is implemented https://github.com/zillow/datasets/ will have some default plugins (ex: parquet-dataset-plugin, and spark-dataset-plugin, hive-dataset-plugin can be installed via pip install zdatasets[spark])

[x]  Newer plugins can be other python modules such as https://github.com/zillow/kafka-dataset-plugin

We (@Peilun-Li , @smatzana) have opted to not split out the read and write of the kafka-dataset-plugin, so shared code can be reused.  This of course can be revised!

[x] I believe that we can name the project and python module dataset yet in pypi name it zdataset

Milestones:

This week:

[x] RFC for plugins has been agreed (see :point_up:)
[x] Plugin PR by end of week so https://github.com/zillow/kafka-dataset-plugin

In meantime @sruthinagalla  isn't blocked because can author def create_topic(topic_name) -> KafkaProducer and def write_topic(producer: KafkaProducer, df: pd.DataFrame)

Next week:

RFC and implementation for supported types within our AIP paved path plugins (example https://api.docs.feast.dev/python/_modules/feast/value_type.html) thank you 🙏  @ville !!

Looking into using Spark Pandas to do all the Pandas --> Parquet type checking, along with an integration test utility function that consumer Flow projects can use to ensure that Pandas --> Parquet --> AVRO --> Pandas and also Pandas --> AVRO --> Parquet --> Pandas will work for the data going through the dataset.write()

Address issue of Flow tagging/@project(branches)/namespace: Can a tagged experiment overwrite the Flow default or another? thank you 🙏  @ville !!

Thank you for the feedback @alexlatchford !


Implement 11.6 inch 960x640 resolution black/red, GDEH116Z91

Sample code
ATC take: https://github.com/atc1441/E-Paper_Pricetags/blob/main/GxEPD2_modded/src/epd3c/GxEPD2_1160c.cpp

esptool.py v3.0-dev
Generated /home/martin/esp/projects/cale-idf/build/cale-epd.bin
Replace /home/martin with your own path to home. Here are the bin files:
atc.zip
Project build complete. To flash, run this command:
/home/martin/.espressif/python_env/idf4.2_py3.8_env/bin/python ../../esp-idf/components/esptool_py/esptool/esptool.py -p (PORT) -b 460800 --before default_reset --after hard_reset --chip esp32  write_flash --flash_mode dio --flash_size detect --flash_freq 40m 0x1000 build/bootloader/bootloader.bin 0x8000 build/partition_table/partition-table.bin 0x10000 build/cale-epd.bin
or run 'idf.py -p (PORT) flash'

Tested the firmware but it does not boot,
the watchdog gets triggered

Mh strange. Watchdog usually does trigger after boot, and usually you get the alert and keeps on working.
Tell me when you have time and I will setup the toolchain for you in Ubuntu

Same here looping after frequency: 4000000
CalEPD component version 0.9.8
Gdeh116Z91() constructor injects IO and extends Adafruit_GFX(960,640) Pix Buffer[76800]

Available heap after Epd bootstrap:30768
I (370) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
CalEPD version: 0.9.8
Fonts-demo. Free heap: 143108 (After epaper instantiation)
DRAM     : 143108
Gdeh116Z91::init(debug:1)
MOSI: 23 CLK: 18
SPI_CS: 32 DC: 27 RST: 26 BUSY: 35

EpdSPI started at frequency: 4000000
E (10380) task_wdt: Task watchdog got triggered. The following tasks did not reset the watchdog in time:
E (10380) task_wdt:  - IDLE (CPU 0)
E (10380) task_wdt: Tasks currently running:
E (10380) task_wdt: CPU 0: main
E (10380) task_wdt: CPU 1: IDLE
E (10380) task_wdt: Print CPU 0 (current core) backtrace

Backtrace:0x400D8E00:0x3FFB0810 0x400823AD:0x3FFB0830 0x400D5732:0x3FFDAD60 0x400D5A03:0x3FFDAD80 0x400D54D3:0x3FFDADA0 0x400E9954:0x3FFDAEB0 0x40087CED:0x3FFDAED0
0x400d8e00: task_wdt_isr at /Users/adrien/Dev/esp/esp-idf/components/esp_common/src/task_wdt.c:187

0x400823ad: _xt_lowint1 at /Users/adrien/Dev/esp/esp-idf/components/freertos/port/xtensa/xtensa_vectors.S:1105

0x400d5732: Gdeh116Z91::fillScreen(unsigned short) at /Users/adrien/Dev/epaper/cale-idf/build/../components/CalEPD/models/color/gdeh116Z91.cpp:39

0x400d5a03: Gdeh116Z91::init(bool) at /Users/adrien/Dev/epaper/cale-idf/build/../components/CalEPD/models/color/gdeh116Z91.cpp:27

0x400d54d3: app_main at /Users/adrien/Dev/epaper/cale-idf/build/../main/demos/demo-fonts.cpp:65

0x400e9954: main_task at /Users/adrien/Dev/esp/esp-idf/components/freertos/port/port_common.c:133 (discriminator 2)

0x40087ced: vPortTaskWrapper at /Users/adrien/Dev/esp/esp-idf/components/freertos/port/xtensa/port.c:168

E (10380) task_wdt: Print CPU 1 backtrace

Thanks Adrien,
I don't have this epaper Model to test that is the problem. So I guess the best move here would be to install ESP-IDF to @atc1441
and I can then connect and debug remotely.
Other than that, if you find what it is, feel free to make a Merge request and I will merge it so it's available for the rest. Thanks a lot for trying CALE-IDF is the first time I make a component for the Espressif Framework

Other than that @atc1441 this called my attention:
Available heap after Epd bootstrap:30768  -> That is around 30 Kb Heap. Can be that the 9606402 is eating all the RAM?
GFX  RAM Usage (Black,red):
960*640/8  = 76800  also 76 Kb * 2 = 152 Kb approx. to handle GFX. That's a bit too much. Maybe that's a reason why it hangs? We are just eating with our huge GFX array all the available RAM in ESP32?
Maybe it's totally different. Should be since you got it working :)

@martinberlin  couldnt you test the bare .bin on your system till it runs, even without the display it should boot for you. Just thinking about that

Hey I discovered what it was, the fillscreen() Method was using a uint16 unsigned, that holds max 65535, that generated a never ending dead loop since:
for (uint16_t x = 0; x < sizeof(_buffer); x++)
x was never going to arrive to 76800 that is the buffer size per color in this display. @adrien3d can you try it?
If it works for you then I will generate a new bin for Aaron. But just tell me one afternoon and I install you IDF, since you will need it to test the touch epaper I'm sending you soon.

Ok, so the good new is that it is actually doing something (although I noticed I had to connect the FPC the other way around). But it just shows a noisy picture.

With my code (just a straightforward implementation of atc's): https://github.com/adrien3d/gdeh116Z91-test/blob/master/main/epaper_main.c, I made it "working" by putting 0xF7 instead of atc 0xC7 for register 0x22.

I am trying few things (more inspired by good-display code) and will keep you updated in case of progress.

Great but at least does something. Let's wait for @atc1441 feedback for sure we will get it working soon :)
Feel free to modify the commands and check them with Aaron example. It's very possible that I forgot something or there is a command missing. I'm also not sure if it's better to do a software reset or a hardware reset (With the RST pin)
Anyways I'm a bit disabled since I can compile it on an ESP32 but I don't have any test epaper so it's hard to debug it.

Hey @atc1441 and @adrien3d any interest to close this one?
If it does not work just point me where to get one of this or send me one. But I would like to either implement it to the end or discard it if there is no interest and close this issue

Hey @atc1441 and @adrien3d any interest to close this one?
If it does not work just point me where to get one of this or send me one. But I would like to either implement it to the end or discard it if there is no interest and close this issue

So far it can be closed from my side so you dont habe the opened issue here, when the time has come i will try it out :)

So far it can be closed from my side so you dont habe the opened issue here, when the time has come i will try it out :)

No problem leaving it on hold will close at the end of this month. Thanks

No problem leaving it on hold will close at the end of this month. Thanks


Upgrade Linux Kernel for flatcar-3033 from 5.10.190 to 5.10.191

Subject says it all.

See instead https://github.com/flatcar/scripts/pull/1087


Improve error message for not meeting minimum

Better error message.
Before:
Error: Options JSON failed validation
Found: 128
Path: service.mem

Please create a JSON file with the appropriate options, and pass the /path/to/file as an --options argument.

Seeing this, I might think that it thinks the value is missing but I check my config and it's there so I would be puzzled about what the problem is.
After:
Found: 128
Required minimum: 256
Path: service.mem

Please create a JSON file with the appropriate options, and pass the /path/to/file as an --options argument.

This makes it clear what the actual problem is.

Can one of the admins verify this patch?

thanks for the pr @msabramo. LGTM
ok to test


Updating reference to WinUI 2.4 release webpage

Description

Updating the table to point to WinUI 2.4 release notes.
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

/azp run


tests: settings: Fix coverity issue CID 203623

Mask the value passed to memset() to avoid an error from Coverity.
Fixes #18687.
Signed-off-by: Carles Cufi<EMAIL_ADDRESS>

Fine with the change, but something is wrong with the number of the issue that is fixed

oops, fixed.


Configuring incomplete, errors occurred!----IOS  complie

os：10.11.1
xcode：7.1-beta
cmake：2.8.11
openssl：1.0.2a
-- Toolchain using default iOS SDK: /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS9.1.sdk
-- Could NOT find PkgConfig (missing:  PKG_CONFIG_EXECUTABLE)
-- FreeRDP GIT
-- Git Revision n/a
-- Looking for pthread_mutex_timedlock in pthread
-- Looking for pthread_mutex_timedlock in pthread - not found
-- Looking for include file sys/eventfd.h
-- Looking for include file sys/eventfd.h - not found
-- Looking for include file sys/eventfd.h
-- Looking for include file sys/eventfd.h - not found
-- Looking for include file sys/timerfd.h
-- Looking for include file sys/timerfd.h - not found
-- Looking for include file poll.h
-- Looking for include file poll.h - not found
-- Looking for ceill
-- Looking for ceill - not found
-- Finding recommended feature libsystemd for systemd journal appender (allows to export wLog to systemd journal)
--     Disable feature libsystemd using "-DWITH_LIBSYSTEMD=OFF"
-- Could NOT find PkgConfig (missing:  PKG_CONFIG_EXECUTABLE)
-- Could NOT find libsystemd (missing:  LIBSYSTEMD_LIBRARY LIBSYSTEMD_INCLUDE_DIR)
CMake Warning at cmake/FindFeature.cmake:46 (message):
feature libsystemd was requested but could not be found! ON / FALSE
Call Stack (most recent call first):
CMakeLists.txt:623 (find_feature)
-- Skipping disabled feature X11 for X11 (X11 client and server)
-- Skipping disabled feature Wayland for Wayland (Wayland client)
-- Skipping disabled feature DirectFB for DirectFB (DirectFB client)
-- Finding required feature ZLIB for compression (data compression)
CMake Error at /Applications/CMake.app/Contents/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:108 (message):
Could NOT find ZLIB (missing: ZLIB_LIBRARY) (found version "1.2.5")
Call Stack (most recent call first):
/Applications/CMake.app/Contents/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:315 (_FPHSA_FAILURE_MESSAGE)
/Applications/CMake.app/Contents/share/cmake-2.8/Modules/FindZLIB.cmake:85 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
cmake/FindFeature.cmake:16 (find_package)
CMakeLists.txt:639 (find_feature)
-- Configuring incomplete, errors occurred!
I don't know why。please help me

You need to specify the pkg- config path.(run cmake with the additional parameter  -DPKG_CONFIG_EXECUTABLE="your pkg-config path").

@allenwen1 anything better with @cloudecker suggestion ?

@allenwen1 you using el captain,  el captain has issues with cmake, Apple removed some header files... try to downgrade your osx.. to yosemite which works../

Apple is shipping some libraries in a new format (tbd).
They can not be found by older versions of cmake, but recent releases should cover support for those.
Can you check your version of cmake in use with cmake --version?
From what I have found versions > 3.3.1 should work.

Closing due to lack of feedback. Feel free to reopen if the issue still persists.

I have this problem, too
os: 10.12.6
xcode:8.3.3
cmake:3.9.1
openssl:1.0.2L


Modularize canopy model to have subcomponents; run model

Purpose
#152 Adds in the big leaf model update equations and hooks up with the existing plant hydraulics model. lays the foundation for more complex canopy RT, stomatal conductance, and photosynthesis models.
Content

Adds canopy model prognostic and aux state vectors and types
adds an update aux function and a canopy_surface_fluxes function which adjusts the MOST-computed surface fluxes for canopy conductance.
Changes the interface to the plant hydraulics standalone model - you now create a canopy model with just a plant hydraulics component using model = CanopyModel{FT}(plant_hydraulics) @gagnelandmanna
adds unit code tests
updates docs

Review checklist
I have:

followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
checked that this PR does not duplicate an open PR.

In the Content, I have included

relevant unit tests, and integration tests,
appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

-->

[x] I have read and checked the items on the review checklist.

@gagnelandmanna I made a catchall issue for the points you raised: https://github.com/CliMA/ClimaLSM.jl/issues/174

bors r+


Move complete repository

Hi,
Is there a way to move the complete repository to another place?
Thanks!

Can you give us an example of what you are trying to achieve?

All my bower folder is on my .gitignore. The files I need I get them with gulp or I move them with bower-installer. But now I need a repository with some files and I want it to put it on another folder, so I can upload it with git. Not sure if it is now better explained, I hope so.
Thanks

I would suggest changing your folder structure or your gitignore file, I don't think that's related to bower-install at all. Maybe check out SO gitignore


dirty_bcache: fix deadlock between bg goroutine and full channel

If we block trying to send on decreasedBytes (while holding the lock),
we could get into a deadlock with the permissions-granting goroutine
which both gets the lock and drains the decreasedBytes channel.

lgtm

CI failure is known flake.  Merging.


fix storybook (pattern library) tabs data

fixes #9092

no idea why the UI tests are failing from this change - looks like a timeout issue.
 FAIL  client/tests/integration/groups.test.js (7.551 s)
  ● Groups › axe

    thrown: "Exceeded timeout of 5000 ms for a test.
    Use jest.setTimeout(newTimeout) to increase the timeout value, if this is a long-running test."

       8 |   });
       9 |
    > 10 |   it('axe', async () => {
         |   ^
      11 |     await expect(page).toPassAxeTests({
      12 |       exclude: '.skiplink, .sidebar__collapse-toggle, #wagtail-sidebar',
      13 |     });

      at groups.test.js:10:3
      at Object.<anonymous> (groups.test.js:1:26)

 PASS  client/tests/integration/users.test.js

Test Suites: 1 failed, 1 skipped, 4 passed, 5 of 6 total
Tests:       1 failed, 4 skipped, 12 passed, 17 total
Snapshots:   0 total
Time:        26.808 s
Ran all test suites.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>test:integration: `./client/tests/integration/node_modules/.bin/jest --config ./client/tests/integration/jest.config.js "--runInBand" "--reporters=default" "--reporters=jest-junit"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test:integration script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

npm ERR!
Note @thibaudcolas  we will need this in before we start looking at moving the Tabs to Stimulus controllers + would be good to have our storybook pattern library working again for tabs.

I'll rebase this and probably just merge in - it's a small change and this is blocking us testing some other storybook items.

Thanks @laymonage  - appreciate the quick review.
The other context is that Thibaud is getting the storybook functional again + getting it to be set up in the CI so hopefully this will add to the test cases once working again.
https://github.com/wagtail/wagtail/pull/10089


feat(core): concurrent stat in list

Description

resolve: #3575

Additional Notes

I tested, and the entry order remains the same whether concurrent is greater than 1 or concurrent is equal to 1.
Regarding the current implementation, I'm not sure whether this will affect the performance when concurrent is equal to 1. If it does, I need some suggestions on the best practices for making changes. 🙏

Not sure if the test failure is related to the PR.

Hi, this PR will need to wait for https://github.com/apache/incubator-opendal/pull/3593 which changes the internal API of list.

Hi, @morristai, we are going to start a new release recently. Would you like to include this PR inside it?

Hi, @morristai, we are going to start a new release recently. Would you like to include this PR inside it?

Give me a day or two; I will merge the master changes.


Optional COMPOSE text

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
6.506
Problem you are trying to solve
Consider removing or adding an setting to make the "COMPOSE" text beside the pencil in the bottom right optional. Using the dark theme it draws my attention with a big white button. The pencil icon alone would be enough.
Suggested solution
Consider removing or making the COMPOSE text optional. Personally i wouldn't even want the pencil icon but the pencil icon only as opposed to the pencil and COMPOSE would be a good compromise
Screenshots / Drawings / Technical details
No response

See Settings → General settings → Display → Show floating compose button


Provide a method to change the default registry as the original OperatorFilterer also supports this now.

https://github.com/ProjectOpenSea/operator-filter-registry/issues/42

Hmm, 2100 extra gas.
I do agree with @dievardump on the thread there.
If you do know any projects who will use closedsea and need the registry to be configurable, let me know. May consider if the demand is strong enough.

Okay, so the general consensus is:

Way to much gas to remove constant
No practical purpose in doing this

I'll go ahead and close the issue. Feel free to resurrect if need be. Nice library btw.


Cannot create a second flask application

Hi,
I believe that due to recent changes around the Blueprint.doc decorator it's not possible to have 2 flask applications created at the same time. See below a minimal example. Tests should pass with flask-rest-api 0.10 but the test test_app fails with 0.11.

            # Prepend Blueprint name to endpoint
            endpoint = '.'.join((self.name, endpoint))

            # Format operations documentation in OpenAPI structure
            # Tag all operations with Blueprint name
            # Merge manual doc
            for key, (auto_doc, manual_doc) in doc.items():
                self._prepare_doc(auto_doc, spec.openapi_version)
>               auto_doc['tags'] = [self.name]
E               TypeError: 'str' object does not support item assignment

../flask-rest-api/flask_rest_api/blueprint.py:161: TypeError

Hi.
This use case was never supported. Api keeps a reference to the app object set in init_app and may access it anytime. Calling init_app twice breaks this link.
In practice, most of the accesses to api._app occur at init, so perhaps this could be mitigated. This would mean remove the _app attribute and pass the app as parameters to init methods. And that's assuming we don't need access to the app at runtime some day.
I'm not totally opposed to supporting this, but can you please elaborate on your use case?
Generally, I run the app after creation, so I can't create it twice from the same script:
    app = create_app()
    app.run()  # Stuck here until the end of times

The issue you're seeing happens because Blueprint._docs is mutated in register_views_in_doc.
It could be solved by deep copying:
        for endpoint, doc in self._docs.items():
            # doc is a dict of documentation per method for the endpoint
            # {'get': documentation, 'post': documentation,...}
            from copy import deepcopy
            doc = deepcopy(doc)

But maybe I ought to pass manual and auto docs in a different way because the way it is done currently is a bit ugly.
By the way, the comment about doc structure is a bit outdated.
And you're not using 0.11.0 but an unreleased revision in master branch. The test pass with 0.11.0. (But the use case is still broken due to reasons explained above.)

I think your test should pass with the latest commit.
This also fixes the case of a Blueprint being registered several times, which is not officially supported, and untested, as I never thought of that case before.
Regarding the "multiple app" use case, I have no definitive opinion. I had a look and it is not obvious. It might work. If it does, it will require breaking changes.
Let's keep this open until we figure that out.

Thanks! Let me know if you decide to move forward with the "multiple app" use case. I will be happy to help.
Cheers!

@svidela, I gave it a shot: #28. Feedback welcome.

I'm gonna put this on hold for the moment.
As I wrote in https://github.com/Nobatek/flask-rest-api/pull/28#issuecomment-442988345, this change is fundamental and I'd rather be provided a convincing real-life use case before committing to supporting the feature.
Hint: In my tests, I generally recreate the whole app at each test. Maybe it's overkill.

Closing this because thanks to https://github.com/Nobatek/flask-rest-api/issues/38#issuecomment-495527421 and below it shouldn't be needed.
Anyone needing this, please comment here.


TF226001: Unable to create the detail nodes for factory QueueDetails.

Agent version and platform
Version of your agent? Latest (download yesterday)

OS of the machine running the agent? Windows

VSTS type and version
VisualStudio.com

What's not working?
Error in the title whenever I try to trigger a build.
Started since server reboot.
VSTS seems to think all is fine through it's options but we have a CI + gated build on check in and VS reports the above error.
Tried reinstalling the build service ... seems to have made no difference.

After the 17 hour wait from the rest of yesterday then overnight the two builds I have stacked are still sat there ... queues are showing "idle" going on to one of the builds to look into more detail just shows the message "Waiting for an agent to be requested".
I have a funny feeling this is not a problem with the build agent but VSTS itself and have no idea how to debug a service hosted by Microsoft.

Send me an email.  I will get someone to look at the backend.  my alias is bryanmac.  Send me your account name in the email as well.

Hi Bryan, thank you :) ... one question .... how do I get your email address? (sorry not a github user at all really)

my alias is bryanmac.  I work for Microsoft.

Ok sent ... sorry about that ... probably the heat in here!

This ended up being a service side issue


Fixed "jc fail" instructions not working properly and updated README.md

I noticed that the times where "jc fail" asm instructions are called are not behaving as expected. They need to be "call fail" in order for the stack to be set up properly to print the fail character. This can be reproduced by changing the jc to jnc and running in qemu.
I added a label in the boot.s that just calls the rust "fail" function and it seems to work as expected now.
I also updated the readme and fixed some error codes that were not unique.

I addressed the recommended changes in this new commit. I can squash into one commit if you would like.


(BOLT-331) Add helper to generate an inventory from PuppetDB

This adds the bolt-inventory-pdb command which can be used to populate a
"template" inventory file with nodes based on PuppetDB queries.
The input to the command is basically an inventory file, except groups
should have a "query" field instead of a "nodes" field. The command will
read the input, execute each of the queries against PuppetDB, and print
a complete inventory with the "nodes" field populated for each group.
This command supports both token and cert auth to PuppetDB, and will
read the puppetdb.conf from the puppet query command if it's present.

CLA signed by all contributors.

I pushed some updates in a commit that I'll squash in after it's reviewed.
• It works again!
• --trace will print backtraces on error
• --configfile specifies the config file to load
• the input file is now a positional argument rather than a flag
• --output specifies where to write the output, defaulting to STDOUT
• The default token is always used if present and another token wasn't specified (if you're using cert auth, supplying the token anyway won't hurt)

Just need to fix up tests now.

Updated the tests and appeased rubocop (which found a bug!)


Proximity Forest

thread to discuss the proximity forest implementation. Contributors

George Oastler (UEA)
Jason Lines (UEA)
Francois Petitjean (Monash)
Ahmed Shifas (Monash)

Implemented in #57, #106


Need the page on NUC models support

There are lots of different NUC models.
Some are known to work (were used for testing), some are known to not work, and some are unknown
We need the NUC support page with the status, and for the devices that are not working ideally with the explanation what specifically is wrong with them
 Front conversations

We need the NUC support page with the status

Where is this page?

@srlowe With the issue being 3 years, first someone might need to go through the linked front conversation to find if this is really something missing from the docs now. Then, I think from this issue it seems like the page never existed and was needed to be made in the first place.


[Bug] The block that the Task fails to send needs to be cleared

Code of Conduct

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

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
When a Task sends a Block to the ShuffleServer, it is placed in the DataPusher asynchronous thread to send it, but if the Task fails before the block is finished sending, the block to be sent is not cleaned.
Affects Version(s)
0.8.0
Uniffle Server Log Output
2023-08-07 10:41:16,838 INFO  [Executor task launch worker for task 0.0 in stage 2.0 (TID 10)] writer.RssShuffleWriter (RssShuffleWriter.java:checkBlockSendResult(273)) - Wait 327 blocks sent to shuffle server
2023-08-07 10:41:16,838 INFO  [Executor task launch worker for task 1.0 in stage 2.0 (TID 11)] writer.RssShuffleWriter (RssShuffleWriter.java:checkBlockSendResult(273)) - Wait 350 blocks sent to shuffle server

Uniffle Engine Log Output
No response
Uniffle Server Configurations
No response
Uniffle Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

@jerqi @zuston @advancedxy Take a hard look at this problem. Are there any good solutions.

Maybe we should add FailureCallback to clean the blocks. You can see https://github.com/apache/incubator-uniffle/blob/dbb9890a20da8d013abc0b333d8e0ccbeac10c75/client-spark/spark2/src/main/java/org/apache/spark/shuffle/reader/RssShuffleReader.java#L144


Thinning doesn't work on ConvTranspose2d

Thinning doesn't work on ConvTranspose2d. I tried replacing the instances of
assert isinstance(layer, torch.nn.modules.Conv2d)
by
def isconv2d(layer):
    return isinstance(layer, torch.nn.modules.Conv2d) or isinstance(layer, torch.nn.modules.ConvTranspose2d)
assert isconv2d(layers[layer_name])

in thinning.py, but that's not enough as the input channels of ConvTranspose2d remain unchanged whether the previous layer was a ConvTranspose2d or a Conv2d.

Hi @trougnouf
We have not integrated support for torch.nn.modules.ConvTranspose2d as of yet.
Cheers
Neta

Hi,
Closing due to staleness.


Switch to PyTorch

There are many issues raised here because of the use of tensorflow and I also struggle to get it to work. Hence I switched from using the TF StyleGAN repo to using https://github.com/NVlabs/stylegan2-ada-pytorch. This should make the usage of this repo much easier. It works quite nicely in my tests and it might be faster (the NVIDIA people claim the pytorch version is faster, I did not benchmark it).
The only issue I see is that the wikiart model is not compatible with the PyTorch repo - at least it throws an error when trying to convert it from TF to torch. I "solved" this by just defaulting back to the TF repo when using wikiart. This is not beautiful, but it should work.  So, if anyone can test if wikiart still works in this PR with TF or knows how to convert it to torch that would be great.

Hi @NotNANtoN, thanks for this work ! I wanted to try to implement it myself but I saw you did it first :)
Which script did you use to convert the pre-trained TF models to PyTorch ? This one looks promising but that's maybe the one you used: https://github.com/rosinality/stylegan2-pytorch#convert-weight-from-official-checkpoints

Hi @julienbeisel! I used https://github.com/NVlabs/stylegan2-ada-pytorch/blob/main/legacy.py. We could try out to use the rosinality weight conversion for the wikiart weights (or all weights where the legacy.py loading does not work).

Hi @julienbeisel! I used https://github.com/NVlabs/stylegan2-ada-pytorch/blob/main/legacy.py. We could try out to use the rosinality weight conversion for the wikiart weights (or all weights where the legacy.py loading does not work).

Ok thanks! I will try to work on it :)
I forked the repo to try to make it work on my laptop. I will make some comments if I think some parts can be improved, I can also make a branch and do a PR later if you want!

Thanks for you feedback! Please do a PR on this PR ;)
It would be best to just convert all TF models to Torch, but I'm not sure if this is easily doable for conditional GANs.

Alright I will submit a PR once it's done and I'll try to convert the models. I will also try to fix the comments on the PR.

@NotNANtoN I spent some time working on it and I really couldn't figure out how to convert these models for PyTorch (nothing is working). I guess the only way to make it work would be to re-train them but it's a lot of work...

@julienbeisel I assume it's simply not possible with some models unless we know their exact architecture. Maybe you can just make a PR with your changes and then we'll just use PyTorch wherever possible and for some models TF?
At least this is working and some pytorch is better than none imo. Have you tried the batch_size argument? When I tried it it just slowed things down, which I don't really understand. But I'll look into it

Major update: increased speed massively by a factor of 5-7. The generation for a 90 seconds long piece of music now takes 4.30 minutes for 60fps, whereas this was at over 20 minutes for 43 fps before.

the objective of this branch is to allow stylegan TF models work with pytorch? I already have model generated with stylegan2-ada-pytorch. Is there a readymade solution for that use case?

@MoemaMike The objective is to completely switch to PyTorch. All pytorch models trained using the NVIDIA-pytorch repositories work with this branch.
There are some TF models that cannot be converted to pytorch atm, in these cases the library just loads them as tensorflow.
Maybe @mikaelalafriz can review and/or merge this branch soonish.

ok, thanks, looking forward to trying it out on Colab

unfortunately，my progress sometimes have been killed
how to solve it well

and Setting up PyTorch plugin "upfirdn2d_plugin"... Failed warning

Your progress has been killed because your RAM is full. Either get more RAM or some immediate saves need to be made in the code.
As for the plugin: either get compatible cuda drivers with your pytorch version of nvcc "nvidia-cuda-toolkit" or you can add a "return False" in approxiamtely line 27 in the file upfird2nd.py in stylegan2/torch_utils/ops to disable the initialization of the pluging.
Not really beautiful, I know.

Can't get this going on wildlife or my own pytorch trained models
ValueError                                Traceback (most recent call last)

<ipython-input-9-2b5a7926efc3> in <module>()
      8 L.hallucinate(file_name = 'x.mp4',
---> 9               fps = 60)
     10
     11 files.download("x.mp4")

6 frames

/usr/local/lib/python3.7/dist-packages/PIL/Image.py in frombytes(self, data, decoder_name, *args)
    798
    799         if s[0] >= 0:
--> 800             raise ValueError("not enough image data")
    801         if s[1] != 0:
    802             raise ValueError("cannot decode image data")

ValueError: not enough image data

This happens when the batch_size are set to 1 (error in line 600, main.py). Try increasing the batch size.

Can't get this going on wildlife or my own pytorch trained models
ValueError                                Traceback (most recent call last)

<ipython-input-9-2b5a7926efc3> in <module>()
      8 L.hallucinate(file_name = 'x.mp4',
---> 9               fps = 60)
     10
     11 files.download("x.mp4")

6 frames

/usr/local/lib/python3.7/dist-packages/PIL/Image.py in frombytes(self, data, decoder_name, *args)
    798
    799         if s[0] >= 0:
--> 800             raise ValueError("not enough image data")
    801         if s[1] != 0:
    802             raise ValueError("cannot decode image data")

ValueError: not enough image data

This happens when the batch_size are set to 1 (error in line 600, main.py). Try increasing the batch size.

I also ran into the same issue, I can confirm increasing the batch size resolves it.

I've tried everything possible and for the past five days I haven't been able to get this (both the original lucidsonicdreams and @NotNANtoN 's clmr_clip branch) to work on A100 and V100 GPUs. Does anyone here have some suggestions to finally get this to work? I'm seriously out of ideas to try and I'm really confused as to why it's so hard to get to work on Ampere architecture. It works well on Google Colab and on 1070 Ti, but impossible to install on RTX 3090, V100, A100.
I've followed this link step-by-step: https://developer.nvidia.com/blog/accelerating-tensorflow-on-a100-gpus/
I've also tried nvcr.io/nvidia/tensorflow:20.06-tf1-py3 and nvidia/cuda:10.2-cudnn7-devel-ubuntu18.04 Docker images and they also haven't been helpful.

I've tried everything possible and for the past five days I haven't been able to get this (both the original lucidsonicdreams and @NotNANtoN 's clmr_clip branch) to work on A100 and V100 GPUs. Does anyone here have some suggestions to finally get this to work? I'm seriously out of ideas to try and I'm really confused as to why it's so hard to get to work on Ampere architecture. It works well on Google Colab and on 1070 Ti, but impossible to install on RTX 3090, V100, A100.
I've followed this link step-by-step: https://developer.nvidia.com/blog/accelerating-tensorflow-on-a100-gpus/
I've also tried nvcr.io/nvidia/tensorflow:20.06-tf1-py3 and nvidia/cuda:10.2-cudnn7-devel-ubuntu18.04 Docker images and they also haven't been helpful.

Frustrating. I currently have a working setup on my RTX 3090 running on Ubuntu 20.04.02. Il add info about my environment here, maybe it can point you in the right direction.
Python 3.7
PyTorch:
torch                         1.8.1+cu111 torchaudio                    0.8.1 torchvision                   0.9.1+cu111
Nvidia:
NVIDIA-SMI 460.91.03    Driver Version: 460.91.03    CUDA Version: 11.2

Isn't wikiart fundamentally a VQGAN model? That's trained with pytorch out of the box.
config: http://eaidata.bmk.sh/data/Wikiart_16384/wikiart_f16_16384_8145600.yaml
checkpoint: http://eaidata.bmk.sh/data/Wikiart_16384/wikiart_f16_16384_8145600.ckpt

How to use it please? I go into NotNANtoN 's repo and see the introdunction is yet pip install lucidsonicdreams, but it cannot run with pytorch model yet. What should I do? My test environment is Colab. Thanks so much.


indicate whether/when the ELT last ran successfully in the Pipelines UI

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1408
Originally created by @dmor on 2019-12-09 19:27:31

On a call with a customer today, we were going to back to see if the ELT had finished and it was unclear from the UI. "I can't really see whether it ran or now".
We know that going to Analyze shows you data if the ELT worked, or an error if it didn't, but the user shouldn't need to go to the next step in the process just to be sent backwards again
Desired behavior:

Have a "last run" timestamp indicating the last time the ELT successfully completed for each pipeline in the list
This should work without needing to open the logs

View 3 previous comments from the original issue on GitLab


chore: integrate i18n and phrase

References DM-796
Motivation and context
It should be a rare exception but we will have some components that we would want to share between projects that require translations (e.g. error pages)
Before
Hardcoded translations in ts file
After
Proper i18n setup with our shared package
How to test
check out the error page, should work as before

@lkappeler I did some testing related to your comment:
Components have their own translation provider (self-sustained)

this PR
https://github.com/smg-automotive/listings-web/pull/222

Having 2 Translation providers does not seem to be an issue. It renders as expected on dynamic and static pages. I haven't discovered any errors on the console.
The bundle size isn't really increased significantly (also see the comparison to the second approach).
Components exports dictionaries, the project merges them

https://github.com/smg-automotive/components-pkg/pull/336
https://github.com/smg-automotive/listings-web/pull/221

It works fine, requires some changes in all the consuming projects and I have some concerns regarding coupling and key conflicts (we would probably want to scope the components translations but that means the consumer needs to be aware of the implementation).
The bundle size is slightly smaller by 0.08KB (the impact should be bigger for components with more translations).

with exported dictionaries: 267.79KB
self-sustained: 267.87KB

I was wondering, how many translations do we expect for the header/footer since we were planning to pass the items from the projects, no?

Based on the PoC I would expect the translations for the Header around 3-5kB per language. (I added some dummy paths to the links and minified the config without visibility options which came out at about 3.5kB) Which will probably result in an additional 15kB loaded for the languages we don't currently render.
I saw that we are currently not loading translations dynamically within the project which will make it have less of an effect. I made very quick tests and we can shave 2-3kB already by dynamically loading the translations on the PDP.
I would expect the effect to grow over time if we add more translations either to the component or the project.
That being stayed I really like the self-contained approach from an implementation perspective. Thinking that we are talking roughly 5% of the page bundle we could shave off with the much more complex approach I would tend to not go fro that at the moment and keep it in mind when we decide to optimise later on.
I would take a look at dynamically loading the project dictionaries as I expect it to be rather straight forward and will save off a lot once we have a full page.


Gitkey Error

import 'package:Prism/gitkey.dart';
^
lib/data/pexels/provider/pexelsWithoutProvider.dart:5:8: Error: Error when reading 'lib/gitkey.dart': Sistem belirtilen dosyayı bulamıyor.
import 'package:Prism/gitkey.dart';

Please refer to the contributing guidelines before making any changes, it has all the instructions required for gitkey, as well as other stuff which is required for the application. I will be closing the issue, but if you have any other issues, feel free to reopen this one.


CLI option and CANAgent server arguments

Hi Jose,

I am now reading the CLI Class, and wanting to add an option, a CAN interface, eg. "can0", With I understanding (correct me if I am wrong), from the command line, user's selection will be parsed and then passed as argument when calling the corresponding server.  Would you help, where would be best place and how to add in the CLI code, please? My apologies for my request as I am uncertain of the thing I add will work or not. Elaborate please? I would like to conform to the existing CLI style.

In CLI.hpp, the lines that calls CANAgent are :
void launch_server()
{
server_.reset(new eprosima::uxr::CANAgent(port_, common_opts_.middleware_opt_.get_kind()));
if (server_->start())
{

To allow the new CAN transport to work with least disruption, what would you suggest to use/replace with  when calling the CANAgent server. FYI, the only argument that CAN transport needs is the interface, can be string, for instance  "can0" or "can1" ? However, I can take numeric value, 0 or 1 from which I can add string "can" to it. My worries are, if I change/add new arguments that the existing application algorithms may be catastrophic.  Another example, when receiving a message, CAN-transport does not have addr, port. However, I can fill the addr and the port respectively with CANID, CAN-message-Length, but will it cause the other algorithms to fail.
eg, uint32_t addr = client_addr.sin_addr.s_addr;
uint16_t port = client_addr.sin_port;
input_packet.source = CANEndPoint(addr, port);  <----  is a replica of IPv3EndPoint for time being.
Appreciate your suggestion.
Hear from you soon.
Thanks in advance.
vi

Hello,
Sorry for the late reply, but this week the ROS World takes place and we are quite busy.
Regarding this, I saw you already included some code in the uxr/agent/utils/ArgumentParser.hpp file. This is our custom CLI implementation and will be the only one that remains in the future. The plan is to delete everything related with CLI11 library.
I checked your code, and saw you are including a port parameter in the CanArgs class. This is not mandatory. Only parameters that are common to every AgentKind and check are the ones present in the CommonArgs class. Therefore, your CanAgent could only have a string parameter, namely the interface. Check for instance class PseudoTerminalArgs. Argument class is a template with two specializations, one for numeric values and another for string values, so you can play with that.
Hope this helps a little bit. Please write down any other question and I will answer as soon as I have more free time (Friday morning, probably. ROS World is tomorrow :smiley: )
PS: I haven't digged enough into your implementation of the CAN agent using CANSocket, since I am not familiar with it. But I promise I will, since it seems to be an interesting approach on how to face CAN connections.

Closing since inactive for 4 months.
Just FYI and for anyone else, this should not be necessary from now on. We have developed a whole new approach for custom transports besides the ones that we already provide as built-in within the default MicroXRCEAgent executable. You can have a look at the provided example here.


Update SDK nuget references.

No functional change other than absorb breaking changes from SDK:

Use new LogFactory method for per-table sharding.
binder class got moved

:ship:

pushed


Create firewall_setup

# How to configure the firewall on Ubuntu to allow SSH and block everything else?

How to configure the firewall on Ubuntu to allow SSH and block everything else?


GitHub Pages don't redeploy

I noticed that even though the plugin got updated, the https://ultrabug.github.io/mkdocs-static-i18n/ page wasn't rebuilt.
The last build is from mkdocs-material version 7 and it lacks sitemaps etc.
So maybe redeploy it with updated theme / plugin version. ✌

Done for version 1.0.0
I shall make this automatic using GH actions indeed
Thanks


Crash Report 117399862

Link:
https://wowsims.github.io/wotlk/warlock/#eJztUz1LA0EQ3Zl8r0rWIHiGRHQrEQKXOw60SrSyNn/AKoi/QKy0EUmjgogGi2BpJQhqFD8QLYIoIUrEaCGRg2CTCAoiRNy9K5VAIHa+Ypg3b+ftsMzSkBcYDMEYTMI0ZODNsQ14CpAHqACMkgAOQXDL5XUxb/AQqJ+Zh6gcdHP3xRm+7iP1sfdH5I6SSNtZRWjlMHccW8y8QyUVtjU/y1ZQ+Qhzt9BkH2XVMiobAQ6EdrA9Ia6EuFOIRFiePKFt4mfXwjLTa7XZd8yLwmqIO6SJky1lZVw8lvHkQMaXI6QeVhfmn2FBi7uyuGwiJ5t4hXQNyT9+RUBttmPGI3dnAWCKUhJ0C+JSIAUJtQEiqqprqq6r0ahuRKOaITLd0DTJIoZhnTF+tpVB8d6byG5NVB4usa9ewIG5G1S/ClgBZw3a0kBKQNJI+DP6rOlmk/FOe86ueE96XSIXG7Yr+Vh/0kIxpu1gX0JxDf7xYtTm4q2wmc2dt8SnGdhvdlSNj9j8IzYB4/9fqQGa/0rfbBx/yQ==
RNG Seed:<PHONE_NUMBER>
runtime error: invalid memory address or nil pointer dereference
Stack Trace:
goroutine 12 [running]:
runtime/debug.Stack()
/opt/hostedtoolcache/go/1.18.10/x64/src/runtime/debug/stack.go:24 +0x6
github.com/wowsims/wotlk/sim/core.runSim.func1()
/home/runner/work/wotlk/wotlk/sim/core/sim.go:59 +0x10
panic({0xef060, 0x1348080})
/opt/hostedtoolcache/go/1.18.10/x64/src/runtime/panic.go:838 +0x29
github.com/wowsims/wotlk/sim/warlock.(*Warlock).tryUseGCD(0x17de000, 0x2015080)
/home/runner/work/wotlk/wotlk/sim/warlock/rotations.go:558 +0x1e7
github.com/wowsims/wotlk/sim/warlock.(*Warlock).OnGCDReady(0x17de000, 0x2015080)
/home/runner/work/wotlk/wotlk/sim/warlock/rotations.go:284 +0x16
github.com/wowsims/wotlk/sim/core.(*Character).initialize.func1(0x2015080)
/home/runner/work/wotlk/wotlk/sim/core/character.go:384 +0xe
github.com/wowsims/wotlk/sim/core.(*Simulation).runOnce(0x2015080)
/home/runner/work/wotlk/wotlk/sim/core/sim.go:355 +0x1f
github.com

I'm just now getting into contributing to the project, but when I attempt to replay your scenario the immediate issue I see is that the rotation options in the settings tab have a few missing options (ie, only corruption is selected, no other rotation spells, and there's no curse selected).
Did you change anything in the settings panel? Or did you just open the warlock sim and select/import your gear and hit Simulate?


Wire up domain replication queue cleanup coroutine

The domain replication queue has a function to cleanup the applied messages which is not being invoked. This change is to invoke the function from the domain replication queue.
The queue maintains a map of the ack levels of each cluster. The domain replication queue will clean up all the messages before the minimum ack level among all clusters.

Coverage increased (+0.2%) to 67.924% when pulling f6b60771ceb14901b27b3486e5e50e232661ee53 on domain_queue_cleanup into d69591532cdc305033dd62d19f9ff3f83a3c5364 on master.


Allow spaces and quotes for Custom Args

Allow spaces and quotes for Custom Args:
In worker.py, if you change this line (58) from args += self.cargs.split() toargs += shlex.split(self.cargs) Then we can do this:
--split-chapters -o "chapter:C:\Users\Guy\Downloads\yt-dlp-Downloads\%(title)s\%(section_number)03d. %(section_title)s.%(ext)s"
Adding quotes and spaces in the names. Because as of now, this is not possible.
Also add import shlex
Thank you!

Done, e081b588527f4c8fef9ddaac11d1c00b919a9608.
About contributor thanks for your offer to help. I appreciate it, but I prefer to keep contributions through pull requests. I also want to maintain a simplistic approach, so I’m avoiding adding too many features. you could always fork and add the features you want.


Change log for latest release

Preparing 0.1.4 release

Includes package documentation

Great, just check the docs for getting started are accurate.  E.g. for installing from PYPI is no longer "in the near future", and remove duplication from the readme.

@ajoubertza / @kmadisa , Ready for release.


Update documentation to account for addition of new sideloading commands to Yo Office (see https://github.com/OfficeDev/generator-office/commit/f95809fc1c2b8490cbedb3827d0d1754ad3d2238)

-Add new help topic for sideloading add-ins via "npm run sideload"

Link to new help topic off existing create-a-network-shared-folder-catalog-for-task-pane-and-content-add-ins.md

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/testing/create-a-network-shared-folder-catalog-for-task-pane-and-content-add-ins.md
:white_check_mark:Succeeded

docs/testing/sideload-office-addin-using-sideload-command.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Addressed all comments and submitting new iteration

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/testing/create-a-network-shared-folder-catalog-for-task-pane-and-content-add-ins.md
:white_check_mark:Succeeded

docs/testing/sideload-office-addin-using-sideload-command.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/testing/create-a-network-shared-folder-catalog-for-task-pane-and-content-add-ins.md
:white_check_mark:Succeeded

docs/testing/sideload-office-addin-using-sideload-command.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


.net creating app

Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/create
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
Error message: Could not execute because the application was not found or a compatible .NET SDK is not installed.
Possible reasons for this include:

You intended to execute a .NET program:
The application 'new' does not exist.
You intended to execute a .NET SDK command:
It was not possible to find any installed .NET SDKs.
Install a .NET SDK from:
https://aka.ms/dotnet-download

Tried solutions: 1) reinstalling the .net sdk.
2) changing the directory

what do you get running dotnet --info @Aditya-0110


Can this be installed by npm?

Just published it to npm:
https://www.npmjs.com/package/css-scoper

Thanks to @thomaspark , this is really useful!


[experiment] patch with patch files (re-resolve)

What does this PR try to resolve?
This is an alternative of https://github.com/rust-lang/cargo/pull/13779.
Most parts of the design are the same. The differences are:

patch entry no longer requires an exact version requirement, e.g. =1.2.3.
Instead, only after a few rounds of dependency resolutions can a patch entry know which package version it is going to patch.
One patch is able to match multiple versions, if applicable.
Patching package.version field to a SemVer incompat version is allowed but kinda useless. The metadata of patched package, for example v2.0.0-to-v1.0.0, is different from the metadata of the real v1.0.0. Thus rustc things they are different versions.

How should we test and review this PR?
This is built upon #13779 with two new commits.
While this works, I don't think the code change is in a good state to merge. The design (hacking on a new SourceKind) could be inherently wrong.
The ugliest part is code that handles the reuse of existing patched in lockfile edd3172c2da0f274367a6fdeecf9916619e7bb75. There are rooms for cleanup but I don't believe it'll be significantly better.
Additional information
Doc hasn't got updated.

:umbrella: The latest upstream changes (presumably #14026) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably #14234) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably #13947) made this pull request unmergeable. Please resolve the merge conflicts.

See https://github.com/rust-lang/cargo/pull/13779#issuecomment-2278138457.


Build with go v1.19

Related Issue:  stolostron/backlog#<ISSUE_NUMBER>
Description of changes

/retest

/retest all

/test all


New search dialog is quite thin on small devices

The new Read The Docs search adoption seems much simpler, however, the layout and styling needs some work.
Especially how thin it is on small devices.

Something like width: min(95vw, 55rem); should be good for the .content but I am not sure if we can easily control these styles as it's in a custom component.
Also, I am not sure if the styles sass/_component_search.scss are still used, we should probably delete or clean up that file if it's not being used. Depending on how we want to tackle the responsibility of this theme to cover custom search still.
Relates to https://github.com/wagtail/sphinx-wagtail-theme/pull/299

Good call about the different searching approaches.
I've not seen a bug on readthedocs add-ons that's the same as this but here's two similar ones.
https://github.com/readthedocs/addons/issues/419
https://github.com/readthedocs/addons/issues/292
I'll look at making a seperate bug on that repo soon.

Issue raised. https://github.com/readthedocs/addons/issues/423


Can't download it, don't know why?

Describe the bug
Can't download it, don't know why?
Desktop (please complete the following information):
Windows 10

Browser and Version:
Nothing can be downloaded by all browsers

To Reproduce

Expected behavior

Screenshots

Additional context

The data node is online again, please give another try


Betting on different than needed

Using experimental strategy. It's saying "Betting on Bear Bet" but instead of that it placed bet on the Bull. It happens not everytime but sometimes. Seems like needed to fix that.
And also it will be great if we can set to skip 1-2 rounds or some semi-manual mode to allow/skip bet. Sometimes you can clearly see that price going opposite side.

I don't know how you predict but it would be great to check price difference between rounds: i.e. if difference is low (like 0.05%-0.1%) and our bet is wrong then next bet will be the same because it's a sign of that trend changes.


fix(label): Truncate label replacements based on codepoint count

This helps ensure that when a string is truncated it is not done in the middle of a utf8 multi-byte sequence.
This doesn't 100% correspond to user-perceived characters, but it should be pretty close in most cases.
I think this might fix issue #393.

Thank you


Correctly load icons of panels with whitespaces in their name

In the pictures below, pay attention to Tool Properties icon.
Before:

After:

Basically, plugin name was not sanitized for whitespaces before encoding as URL for resource_retriever (CURL wrapper).

Linux
Linux-aarch64
Linux-rhel
Windows

https://github.com/Mergifyio backport jazzy humble iron