Could not find declared class redis

My Vagrantfile calls upon configure_cache.pp, which calls a custom module redis.pp:
class package::redis {
    class { 'redis::install':
        redis_version => '3.2.0',
    }
}

Note: I attempted to install redis via an older suggestion of yours.
However, the following traceback occurs when vagrant runs configure_cache.pp:
...
==> default: Running provisioner: puppet...
==> default: Running Puppet with environment development...
==> default: Error: Evaluation Error: Error while evaluating a Resource Statemen
t, Could not find declared class redis at /tmp/vagrant-puppet/modules-f71316dd46
7cca918424590c4186206a/package/manifests/redis.pp:6:5 on node drupal-demonstrati
on.com
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

I forgot to mention, puppet is provisioning Centos 7.1.

redis::install a private class, use class { 'redis': ... } instead.

@arioch, how do I specify which version of redis to install via your suggested directive:
class { 'redis': ... }

Additionally, how do I tell redis to automatically start up, when the machine boots, or restarted?

In an attempt to perform your suggested implementation, as indicated above, I created an additional temp.pp:
$version = '3.2.0'

class { 'redis':
    package_ensure => $version,
}
contain redis

Next, I proceeded to install your module, and run the above temp.pp:
[root@localhost puppet]# puppet module install arioch-redis
[root@localhost puppet]# puppet apply --modulepath=/vagrant/puppet/modules --confdir=/vagrant/puppet temp.pp
Warning: Setting configtimeout is deprecated.
   (at /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/settings.rb:1142:in `issue_deprecation_warning')
Warning: ModuleLoader: module 'redis' has unresolved dependencies - it will only see those that are resolved. Use 'puppet module list --tree' to see information about modules
Warning: ModuleLoader: module 'redis' has unresolved dependencies - it will only see those that are resolved. Use 'puppet module list --tree' to see information about modules
Warning: ModuleLoader: module 'redis' has unresolved dependencies - it will only see those that are resolved. Use 'puppet module list --tree' to see information about modules
Notice: Compiled catalog for localhost.localdomain in environment production in 0.78 seconds
Error: Could not update: Execution of '/bin/yum -d 0 -e 0 -y install redis-3.2.0' returned 1: Error: Nothing to do
Error: /Stage[main]/Redis::Install/Package[redis]/ensure: change from purged to 3.2.0 failed: Could not update: Execution of '/bin/yum -d 0 -e 0 -y install redis-3.2.0' returned 1: Error: Nothing to do
Notice: /Stage[main]/Redis::Config/File[/etc/redis]: Dependency Package[redis] has failures: true
Warning: /Stage[main]/Redis::Config/File[/etc/redis]: Skipping because of failed dependencies
Notice: /Stage[main]/Redis::Config/File[/etc/redis.conf.puppet]: Dependency Package[redis] has failures: true
Warning: /Stage[main]/Redis::Config/File[/etc/redis.conf.puppet]: Skipping because of failed dependencies
Notice: /Stage[main]/Redis::Config/File[/var/log/redis]: Dependency Package[redis] has failures: true
Warning: /Stage[main]/Redis::Config/File[/var/log/redis]: Skipping because of failed dependencies
Notice: /Stage[main]/Redis::Config/Exec[cp -p /etc/redis.conf.puppet /etc/redis.conf]: Dependency Package[redis] has failures: true
Warning: /Stage[main]/Redis::Config/Exec[cp -p /etc/redis.conf.puppet /etc/redis.conf]: Skipping because of failed dependencies
Notice: /Stage[main]/Redis::Service/Service[redis]: Dependency Package[redis] has failures: true
Warning: /Stage[main]/Redis::Service/Service[redis]: Skipping because of failed dependencies
Notice: /Stage[main]/Redis/Anchor[redis::end]: Dependency Package[redis] has failures: true
Warning: /Stage[main]/Redis/Anchor[redis::end]: Skipping because of failed dependencies
Notice: Applied catalog in 0.81 seconds

I got redis to install via your module:

https://github.com/jeff1evesque/drupal-demonstration/issues/512

https://github.com/jeff1evesque/drupal-demonstration/blob/ca5b8e253e0aa0a1d8d8b3375e37867d9b25be64/puppet/environment/development/modules/package/manifests/redis.pp
https://github.com/jeff1evesque/drupal-demonstration/blob/fcf03f550ba9c1748aa41c703e42f8ace31b3e73/puppet/environment/development/modules/package/lib/facter/redis_version.rb


Logic inside the constructor function

A little bit of concern regarding put logic inside the constructor function, is there any better approach to replace the below code?
constructor(
        private router: Router,
        private http: HttpClient
    ) {
        this.userSubject = new BehaviorSubject<User>(JSON.parse(localStorage.getItem('user')));
        this.user = this.userSubject.asObservable();
    }

I got error on the same line, please take a look
ERROR SyntaxError: Unexpected token u in JSON at position 0     at JSON.parse (<anonymous>)     at new AccountService (account.service.ts:21)     at Object.AccountService_Factory [as factory] (account.service.ts:83)     at R3Injector.hydrate (core.js:17053)     at R3Injector.get (core.js:16803)     at NgModuleRef$1.get (core.js:36454)     at Object.get (core.js:34091)     at getOrCreateInjectable (core.js:5830)     at Module.ɵɵdirectiveInject (core.js:21082)     at NodeInjectorFactory.AppComponent_Factory [as factory] (app.component.ts:7)

Found the solution by using the stringfy method
JSON.stringify();


It is an error to call the default List constructor

The spec was changed. No matter, with nullable type argument, or length.

https://dart-review.googlesource.com/c/sdk/+/140363

The CL fixes two co19 tests, but breaks a lot of other tests, mostly library, some language tests. @leafpetersen @munificent Are we still OK to make this change? What do we do with tests - who will update them? Should I open new issues (with which area?) and approve the bot failures?

@scheglov Go ahead and approve the failures.   File an issue with the list of tests you're approving, area-test.   You can assign it to me, I'll find an owner or fix them myself.


No GridLine for row(0) ?

Hello there ...
unfortunately I'm not able to get gridlines for row(0).
Any hints ?

workaround is to SetRangeBorders for each cell.
:(

If you have cell background color the grid line will not be displayed. You should set borders manually.

So it's a feature not a bug ! ?
:))
THX Jing

Yes it’s by design. But also it sounds good to add an option that can always show girdlines.


Memory leak NavigatorExperimental

Hello,
I'm using NavigatorExperimental and when I'm switching between views on React 0.30.0 iOS 9.3, memory keep rising up when I push a route / pop a route again and again. Is there a tool or something to track memory leaks ? Is this a known issue or is my app abnormally memory consuming ?
Thanks.

I'm seeing the same thing. I'm an iOS developer by day and when profiling this you can see that popped routes are never being dealloc'd. I wasn't sure if this was because I was using redux with NavigatorExperimental, but I was about to start profiling the UIExplorer example before I did a quick search to find this issue. I found another issue where someone mentioned that they were using a method to reset the navigation state in order to allow that retained memory to be released, but that seems to go against the flow of a navigator so I was avoiding that option.

Pretty sure I'm just doing something wrong now (and you might be as well).
All the leaks I'm seeing in popular examples are in CSSNodeListNew and not causing entire VCs to be retained.
Offending area of tiny leaks:
CSSNodeListRef CSSNodeListNew(unsigned int initialCapacity) {
  CSSNodeListRef list = malloc(sizeof(struct CSSNodeList));
  assert(list != NULL);

  list->capacity = initialCapacity;
  list->count = 0;
  list->items = malloc(sizeof(void*) * list->capacity);
  assert(list->items != NULL);

  return list;
}

I retract my previous statement.
When using mapviews (even when using react-native-maps) on iOS, you can create endless mallocs simply by navigating to and from views that have mapview components in them. Memory will just climb and climb without ever being released.

Can I have a look in your App & Navigation Component please ?
I tried reset but it doesn't solve anything. I'm also using the redux module but even without it I've this issue.

@facebook-github-bot label Icebox

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/memory-leak-navigatorexperimental
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.

@facebook-github-bot close


Whitespace in actors_value but not in actor's flipper_id

Hi @jnunemaker, thanks for this really useful gem (also hi @AlexWheeler! 👋)
I noticed that by copy-pasting emails as actors_value in the Flipper UI like<EMAIL_ADDRESS>\n, then it won't let a User with<EMAIL_ADDRESS>as flipper_id in.
The UI also makes it difficult to notice that kind of extra whitespace, so it took a little effort debugging on my side to find the root cause.
Should values stored in Redis always be stripped from leading-trailing whitespace? Should the user's custom Thing#flipper_id be in charge of this logic? Should the UI run String#strip before giving the value to Flipper?
I'd love to help out with this problem but I'm not sure where the responsibility lies between the user, the gem and its various components.
Let me know how I can help!

Great question! I would have the UI/API do the strip. Putting it in flipper
could cause unexpected things for users of flipper in the future.
On Wed, Nov 23, 2016 at 8:20 AM Kevin Bongart<EMAIL_ADDRESS>wrote:

Hi @jnunemaker https://github.com/jnunemaker, thanks for this really
useful gem (also hi @AlexWheeler https://github.com/AlexWheeler! 👋)
I noticed that by copy-pasting emails as actors_value in the Flipper UI
like<EMAIL_ADDRESS>\n, then it won't let a User with<EMAIL_ADDRESS>as
flipper_id in.
The UI also makes it difficult to notice that kind of extra whitespace, so
it took a little effort debugging on my side to find the root cause.
Should values stored in Redis always be stripped from leading-trailing
whitespace? Should the user's custom Thing#flipper_id be in charge of
this logic? Should the UI run String#strip before giving the value to
Flipper?
I'd love to help out with this problem but I'm not sure where the
responsibility lies between the user, the gem and its various components.
Let me know how I can help!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jnunemaker/flipper/issues/204, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAA6xWlpC3vZ7NH7QJX3CT0MgConI2Zks5rBD2igaJpZM4K6kY8
.

Thanks, I also agree it shouldn't be in Flipper. The UI seems the right place, as the API should just save what it's given. I'll take a look today


Set default values properly

Set configured default values correctly with a distinct method.
DMXGateway.set_channels() is flagged as async coroutine and cannot be called
from the constructor of ArtnetLight. Also, send the default values directly
and unconditionally.
Without this, HASS would start up without sending the default values to the
hardware, but reporting them up the stack, so the UI is out of sync.

Thanks for contributing to the code! I'm still getting my head around async so didn't realise couldn't call from constructor.
I did have a specific use case where I didn't want the DMX universe to be plunged into darkness on a HASS reboot if default values wern't set; or if controlling from another desk and HASS rebooted for some reason.
Is it worth adding an optional config parameter 'send_values_on_startup' kind of thing to allow that functionality?

Hmm, not sure. If you don't send the values on startup, and if Art-Net doesn't give you a way to read back the current state of its channels, the values reported up to HASS are definitely wrong. And that's very confusing because you have to toggle each light in order to bring it back in sync.
So I'd rather make sure things are in sync when HASS starts up.

I've added a 'send_levels_on_startup' configuration item which defaults to True meaning I can still override this behaviour to not send levels on startup for my use case; and renamed set_channels to set_channels_async


Two-Bucket has an incorrect test

The number of moves for the 6th test is wrong.
  describe('Measure using bucket one of size 2 and bucket two of size 3', () => {
    xtest('start with bucket one and end with bucket two', () => {
      const twoBucket = new TwoBucket(2, 3, 3, 'one');
      expect(twoBucket.moves()).toEqual(4);
      expect(twoBucket.goalBucket).toEqual('two');
      expect(twoBucket.otherBucket).toEqual(1);
    });
  });

The number of moves should be 2:

fill the start bucket
since the other bucket size == the goal, fill it
there is no move 3.

Following that change, the otherBucket amount should be 2 (the start bucket is full).
I humbly suggest my solution as a reference solution.
This was changed in #1396.

Thanks for the report! We were planning a wider check what's out of sync once the new configlet release lands that allows to also sync instructions.
Here the task list to fix this issue for whoever wants to work in this:

[ ] Use configlet sync to update the tests.toml file.
[ ] Fix the test cases so they match the latest version defined in problem spec.
[ ] Update the example solution (see suggestion above).

@SleeplessByte We had a bit of a discussion about memoizing vs erroring for the result accessors.  Evidently we were both suggesting slightly different directions (in our mentoring with Glenn).  I dislike students "over-engineering" solutions (unless that's declared to be the game).  I think adding a bunch of "throw errors" or "call move magically" is too much given that the tests specify no behavior in a
new TwoBucket(...).goalBucket

scenario.  IE, it's expected one simple doesn't ask for a result until after move has been called.   So, what should happen if someone doesn't do that?

Should it error?
Should it just perform the calculation and return the result?
Perhaps this behavior should be based on whether a prior result was cached or not?
Other options I haven't thought of?

For these reasons I'd say documenting the behavior as returns null or throwing an error is better - but since the tests specify neither I'm not sure either is required for a solid Exercism solution here.  For an actual client or FOSS library obviously you'd want to lock this type of behavior down in a more professional fashion.  So there are good conversations that can happen with students around this.
BUT I'm wondering if we should just change this API to avoid the problem entirely...   One solution would be that a result object is directly generated) and returned by the move method:
buckets = new TwoBucket(...)
bucket.move() => // { goalBucket: "two", otherBucket: 0 }

Now one doesn't even have to scratch their head over the correct behavior.  It's simply impossible to try to work with the result before it's been generated since the result is it's own discrete object.
Thoughts?

I'll think about it for a hot minute, but i like how you've written down your thoughts!


background album art is cut off

Describe the bug
background album art is cut off
To Reproduce
Start Spotify
Screenshots

Desktop Setup

Operating System: Windows 11
Spotify version: <IP_ADDRESS>9
Spicetify version: 2.8.1
Dribbblish version  4.0.0

This is a lyrics-plus problem, you have to disable the background in the lyrics-plus settings

This is a lyrics-plus problem, you have to disable the background in the lyrics-plus settings

It's hard to see, so I took a picture of the lyrics plus window, and no matter what window I go in, the album art is cut off.
-old Dribbblish version (3.0.1)

new Dribbblish version (4.0.0)

This slipped by me since it only happened when friends activity is disabled


Add support for helpers in commonjs scripts

[x] Support for externalHelpers: false
[x] Clean up code
[ ] Test in real-life project

Fixes #1814

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Sven Tschui seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing as it's implemented by https://github.com/swc-project/swc/pull/4025


Avoid upgrade processing deleted broker clusters

This defect didn't produce a functional defect.

@enmasse-ci run tests testcase=SmokeTests

:heavy_exclamation_mark: Build Failed :heavy_exclamation_mark:


Crash at development stage

I'm developing Windows Store application and purchased VideoPlayer license.
I've added call of FormsVideoPlayer.Init(...) to appropriate place in app with actual license key, but now it always crashed with following stack trace:
at Windows.ApplicationModel.Store.CurrentApp.get_AppId()
at Octane.Xam.VideoPlayer.WindowsStore.PlatformFeatures.get_PackageName()
at Octane.Xam.VideoPlayer.Licensing.VideoPlayerLicense.Initialize(String licenseKey)
at Octane.Xam.VideoPlayer.WindowsStore.FormsVideoPlayer.Init(String licenseKey)
.....
and HResult is 0x803F6107
Crash occur when I start my app from Visual Studio 2015 for debugging purposes.
Application target is Windows 8.1 and I'm run it on Windows 10.
Looking to the documentation of CurrentApp I can see that my application must be published in Windows Store to make this API working. But I'm at development stage and can't publish unfinished application!
Any suggestions excluding removal of FormsVideoPlayer.Init(...) call?

Windows 8 has been temperamental. In the interim, I would recommend running in trial mode without specifying the key until you are ready to publish.


Add metrics for total time spent on user logic

executeTimeNs in FullBoltMetrics is quite useful for investigation. However it is not in BoltMetrics. It is added in this PR.
Also, there is no similar metrics in SpoutMetrics and FullSpoutMetrics. A new metric "__next-tuple-time-ns" is added as well.

@nwangtw Is this PR still relevant?

Can we merge this? Or should we close it?

Can we merge this? Or should we close it?

Not motivated enough for me to merge. I am going to drop it.

Not motivated enough for me to merge. I am going to drop it.

I'd be willing to test it if you think it was ready to merge.

I'd be willing to test it if you think it was ready to merge.


Fix link to BicycleGAN paper

The link to the BicycleGAN paper points to the BEGAN paper. This pull request fixes the link to point to the correct paper ( https://arxiv.org/abs/1711.11586 )

Thanks


Questions on smiles str canonicalization when inference

Hi there, thanks for your great work!
I found that in predict.py you directly use the input smiles string rather than a canonical one for inference. Is this intended to be that? If so, will two smiles string for the same molecule lead to different predictions?
Best regards!

Yes, that is the way. We predict a given SMILES as is, but as the model has been trained with augmentations, the differences in predictions should not be large between different SMILES of the same molecule. If they are, you may have an OOD sample. You can also augment the SMILES and do several predictions and take the mean prediction, which will improve your results a bit.
In essence, a model trained with with augmentation can predict canonical SMILES, while a model only trained on canonical SMILES can't really predict non-canonical forms. The canonical trained model are usually also worse on the prediction metrics. As example, see what happens if you predict canonical vs. augmented smiles in a model trained with or without augmentation in this classic preprint: https://arxiv.org/abs/1703.07076


feat: configure ssh proxy jump

Maybe one could now also remove the special cases for github and gitlab? But I am unsure about that.

Also, note that this is untested (the templated version, the concrete config has been tested by me). Please test it before merging.

Indeed now we can remove the special configuration snippets for github and gitlab. I'll remove them on the slurm cluster as soon as this is merged.


Idea: Timestamp index in block order

Keep a timestamp index that will map a timestamp to a blockhash in a way that will keep the block order, and iterating by timestamp would not yield out-of-order blocks.
The timestamp could be defined as the actual block's timestamp or would assume the previous block's timestamp (with an additional second) if the block's timestamp is not greater than the previous timestamp.

This was added in: https://github.com/bitpay/bitcoin/pull/29


Error: no response from server

When trying to do any routing, error pops up. Is brouter-web currently unusable, or is it just me?

Works for me. Still occuring?

Yes, but only on my work PC. Probably due to some changes to the corporate proxy. My mistake, sorry for that.


State, City and Services drop-downs selected value is not fully visible

In Essentials section State, City and Services dropdowns selected value is not fully visible in the Chrome browser. (My browser Version 81.0.4044.122 (Official Build) (64-bit))
OS: Windows 10
Display Resolution: 2160 x 3840 (4k Portrait mode)  or 3840 x 2160 (4k Landscape mode)

Can you give some more details like screen resolution & OS .I was not able to replicate the problem.

@dharmesh-m @Abhinav-Mani I'm able to reproduce the issue. Fixing it.

/claim

Seems irrelevant now after the Essentials redesign


Websocket not allowing to capture all messages if I don't apply handler

Description:
When we connect to websocket without handler, in order to receive all messages. We are getting only one message everytime. We should get all the messages during the interval
Steps to Reproduce:

Connect to websocket without datahandler
Listen to websocket

Actual Result:
I am getting only one message, if I provide manual delay of let us 10 seconds still I am getting the previous message that too only one message
Expected Result:
As soon as I hit Websocket, Websocket delivered multiple messages which I can see using postman but on Karate I am on getting only one message. If we don't provide handler, we should get all the messages being sent by websocket server

@prnbtr09 please follow this process to reopen: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue

@ptrthomas I can't share the real sockets. In general, If I am trying to connect to websocket, I am always getting same message for each listen. What we are expecting is Queue of responses.

@prnbtr09 this is an open source project that expects some level of contribution, I'm sorry that I am not able to help more here. also see this: 12 hours ago

@ptrthomas Thanks for your prompt response. Do we have any way to get all the messages from websocket, instead of getting only one message.

@prnbtr09 I've replied to your comment at the link above already


Add a page for ti-community-owners

Feature Request

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Is your feature request related to a Issue or PR(Optional)
https://github.com/pingcap/tidb-operator/pull/3522#issuecomment-733509452
Describe the feature you'd like:
For now, we directly expose the URL to the collaborators, and we need a page to display these content.
Do you have any ideas on how to implement it?(Optional)
You can refer to https://pingcap.com/developer/sig/tiup/.
And I suggest you use antd's card to display the collaborators.

/type feature

/assign @fireairforce

/assign @ti-community-prow-bot
Assign to bot but @fireairforce will working on this.

/help

I will try this one.

/assign @fireairforce

/unassign @ti-community-prow-bot


Support Queries / Mutations using the WS transport

Is your feature request related to a problem? Please describe.
Only queries and mutations using the HTTP transport are detected and shown via the extension.
Describe the solution you'd like
The extension ideally should also monitor ws connections and detect the queries / mutations made.

are you open to a PR implementing this ?

Absolutely, it's a highly requested feature, I haven't had time to get around to it, but would welcome a pr. ⭐

This is in progress. Will be the next major release.

Closing as Beta feature for subscriptions now added in 2.18.0


Train project tests

@alecgunny I tried to make that signal comparison test we discussed earlier. For the life of me, I can't see why it's not working. I figured I'd put it here to get a second set of eyes on it, and to see if you had any comments on the other tests that I added.
Oh, and it looks like making and un-making changes in validation.py created another large diff - not sure why, but I did check, and nothing is actually different.


bug: can't create SyncTarget yaml for WEC - kubectl-kubestellar-prep_for_syncer: line 184: 27522 Killed: 9 "$bindir/kubectl-kubestellar-syncer_gen" "$stname" --syncer-image "$syncer_image" -o "$output"

Describe the bug
receiving:
kubectl kubestellar prep-for-cluster --imw root:example-imw1 wec5 env=wec5
Current workspace is "root:example-imw1".
--- current directory is /Users/andan02/projects
current ws is root:example-imw1
imw is .
current cwsi is root:example-imw1
edge.kubestellar.io apibinding exists in workspace root:example-imw1
Current workspace is "root:example-imw1".
Current workspace is "root:espw".
/opt/homebrew/bin/kubectl-kubestellar-prep_for_syncer: line 184: 27522 Killed: 9               "$bindir/kubectl-kubestellar-syncer_gen" "$stname" --syncer-image "$syncer_image" -o "$output"
Current workspace is "root:example-imw1".
Current workspace is "root".

Steps To Reproduce

install kubestellar v0.7.0 via helm into openshift
download kubestellar kubectl plugin scripts and put them in path
run 'kubestellar-release' to confirm v0.7.0 for plugins
run 'kubectl kubestellar get-external-kubeconfig -o ~/ks.config'
run 'KUBECONFIG=~/ks.config kubectl ws root'
run 'KUBECONFIG=~/ks.config kubectl kubestellar prep-for-cluster --imw root:example-imw1 wec5 env=wec5'

receive output:
kubectl kubestellar prep-for-cluster --imw root:example-imw1 wec5 env=wec5
Current workspace is "root:example-imw1".
--- current directory is /Users/andan02/projects
current ws is root:example-imw1
imw is .
current cwsi is root:example-imw1
edge.kubestellar.io apibinding exists in workspace root:example-imw1
Current workspace is "root:example-imw1".
Current workspace is "root:espw".
/opt/homebrew/bin/kubectl-kubestellar-prep_for_syncer: line 184: 27522 Killed: 9               "$bindir/kubectl-kubestellar-syncer_gen" "$stname" --syncer-image "$syncer_image" -o "$output"
Current workspace is "root:example-imw1".
Current workspace is "root".

Expected Behavior
expected to receive SyncTarget directions
Additional Context
No response

invalid issue - looks like it was a plugin to core mismatch

Thank you for including the steps to reproduce. That was very helpful. It revealed a bug in the documentation, #1062 .

In the "Steps To Reproduce" section of the opening comment, it looks to me like there is NOT a version mismatch. The kubestellar-release command is looked up on the $PATH of the shell where you issue it; when it prints "v0.7.0" that would tell me there is consistency with a core that is also v0.7.0.


Run mirror tests 3 times before failing

An attempt to fix https://github.com/istio/istio/issues/17496.
Refactor code to avoid code duplication in both tests.

@jwendell: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
ac697feca955a9e90205cc525a8946cf4f533f6f
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


MT.1008 - Require MFA vs. Require authentication strength

Should https://maester.dev/docs/tests/MT.1008/ be failed if a policy exists that requires authentication strength of Phish-resistant MFA for admins?
Or is the assumption both policies are needed?

MT.1008 checks both Authentication Strength as well as the "legacy setting" for MFA.
So it should only fail if both are missing

Appreciate the quick reply, not sure what I'm missing.

I will repo this and let you know.

Cannot reproduce this issue. Please let me know if this is still an issue and we will check together


feat(lightning): using on chain address for channel close

Description

Updates react-native-ldk to include a change where funds from a channel close are sent direct to Bitkit's on chain wallet and no longer require and extra sweeping step.
Also includes a fix preventing some users from completing the restore flow

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (improving code without creating new functionality)

Tests

[ ] Detox test
[ ] Unit test
[x] No test

QA Notes

Funds should show up directly on chain after channel close
Some users stuck on restoring screen should not be able to progress

Actually although this works I think we'd want more work on the UI to avoid confusing the user. Right now it shows a pending card (60m) but at the same time also shows an activity item (10-20m)

Instead of using generateNewReceiveAddress we'll want to grab what the current addressIndex is instead. Since generateNewReceiveAddress will always create a new address regardless of whether or not the previous index was used. This means opening and closing the app a few times will end up pushing the user's addressIndex into the gap limit. I'll make a PR for this shortly.

Can you fix the failing onboarding e2e test?

I was seeing webRelayClient not ready yet in all the PR's? Unless you mean another test?

Instead of using generateNewReceiveAddress we'll want to grab what the current addressIndex is instead. Since generateNewReceiveAddress will always create a new address regardless of whether or not the previous index was used. This means opening and closing the app a few times will end up pushing the user's addressIndex into the gap limit. I'll make a change suggestion for this shortly.

Oh yes right. So originally I had it use getReceiveAddress and I changed it to return IAddress instead of a string but I thought maybe I was breaking something with the store and didn't want to go down that path right before testing session.

Can you fix the failing onboarding e2e test?

I was seeing webRelayClient not ready yet in all the PR's? Unless you mean another test?

Was talking about this:
FAIL e2e/onboarding.e2e.js (109.532 s)
  Onboarding
    ✕ Can pass onboarding correctly (101505 ms)

  ● Onboarding › Can pass onboarding correctly

    expect(received).toBe(expected) // Object.is equality

    Expected: "bitcoin:bcrt1q2zkt93dtl8cxqgtvzxurlu0mu6z6ahnh6sy5uf"
    Received: "bitcoin:bcrt1qjz82ap0k2x6r7zg9jketlthx8whm0dne8f0cvx"

      120 | 		const { label: address2 } = await element(by.id('QRCode')).getAttributes();
      121 |
    > 122 | 		jestExpect(address1).toBe(address2);
          | 		                     ^
      123 |
      124 | 		markComplete('onboarding-1');
      125 | 	});

Will be fixed by what @coreyphillips described above 👍

Oh yes right. So originally I had it use getReceiveAddress and I changed it to return IAddress instead of a string but I thought maybe I was breaking something with the store and didn't want to go down that path right before testing session.

PR created here. Since most of this will be changed once the migration is merged anyway I simply created a new method that is effectively the same as getReceiveAddress except it returns the entire object. Will make it easier to merge the changes in later on.


User profile should be updated/refreshed on removing a resource from My plan or My Saved Resources

From @charmila-c on July 17, 2018 10:34
Copied from original issue: Microsoft/Access2Justice#247

Issue resolved


test for fleek preview purposes

Nothing to see here.

Image optimization came back clean!

Vuepress build was successful!

Check of docs/how-to/lazy-minting.md came back clean!

Check of docs/how-to/minting-on-different-blockchains.md came back clean!


Fixed some things.

Fixed:

Core.java
HelloWorldChatListener.java
HelloWorldCommand.java

pom.xml
-> Missing bukkit dependency
-> Missing spigot repository

I changed it.


lidar bag and imu bag were not recorded on one day?

When I was using lidar dataset 'D',I finded that the bags named 'husky4_lidar_2019-08-20-xxxx.bag' were recorded on Aug 21 2019,while the bag named 'imu.bag' was recorded on Aug 22 2019.those two kinds of sensor massege were not recorded on one day.It's there something wrong?  thank you!
path:         husky4_lidar_2019-08-20-13-33-22_61.bag
version:      2.0
duration:     26.9s
start:        Aug 21 2019 04:33:22.45<PHONE_NUMBER>.45)
end:          Aug 21 2019 04:33:49.30<PHONE_NUMBER>.30)
path:        imu.bag
version:     2.0
duration:    1hr 25:42s (5142s)
start:       Aug 22 2019 03:24:20.01<PHONE_NUMBER>.01)
end:         Aug 22 2019 04:50:02.02<PHONE_NUMBER>.02)

I found the same problem. How did this problem be solved?


CoinMarketCap - Issue with HA and mysql DB

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.55.1
Python release (python3 --version):
Python 3.4.2
Component/platform:
CoinMarketCap
Description of problem:
Getting sqlite3.OperationalError: unable to open database file when trying to setup the CoinMarketCap component.
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:
sensor:
  - platform: coinmarketcap
    currency: ethereum

Add to configuration file
Restart HA and view log

Traceback (if applicable):
2017-10-16 13:40:14 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up platform coinmarketcap
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 164, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=self.hass.loop)
  File "/usr/lib/python3.4/asyncio/tasks.py", line 372, in wait_for
    return fut.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/sensor/coinmarketcap.py", line 50, in setup_platform
    CoinMarketCapData(currency).update()
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/sensor/coinmarketcap.py", line 119, in update
    self.ticker = Market().ticker(self.currency, limit=1)
  File "/srv/homeassistant/lib/python3.4/site-packages/coinmarketcap/core.py", line 54, in ticker
    response = self.__request('ticker/' + currency, params)
  File "/srv/homeassistant/lib/python3.4/site-packages/coinmarketcap/core.py", line 28, in __request
    response_object = self.session.get(self.base_url + endpoint, params = params, timeout = self.request_timeout)
  File "/srv/homeassistant/lib/python3.4/site-packages/coinmarketcap/core.py", line 21, in session
    self._session = requests_cache.core.CachedSession(cache_name='coinmarketcap_cache', backend='sqlite', expire_after=120)
  File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/core.py", line 68, in __init__
    self.cache = backends.create_backend(backend, cache_name, backend_options)
  File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/backends/__init__.py", line 50, in create_backend
    return registry[backend_name](cache_name, **options)
  File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/backends/sqlite.py", line 28, in __init__
    self.responses = DbPickleDict(location + extension, 'responses', fast_save=fast_save)
  File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/backends/storage/dbdict.py", line 58, in __init__
    with self.connection() as con:
  File "/usr/lib/python3.4/contextlib.py", line 59, in __enter__
    return next(self.gen)
  File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/backends/storage/dbdict.py", line 70, in connection
    con = sqlite.connect(self.filename)
sqlite3.OperationalError: unable to open database file

Additional info:

Same problem... started when updated to last version I think.
2017-10-16 02:36:26 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up platform coinmarketcap
Traceback (most recent call last):
File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 164, in _async_setup_platform
SLOW_SETUP_MAX_WAIT, loop=self.hass.loop)
File "/usr/lib/python3.4/asyncio/tasks.py", line 372, in wait_for
return fut.result()
File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
raise self._exception
File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
result = self.fn(*self.args, **self.kwargs)
File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/sensor/coinmarketcap.py", line 50, in setup_platform
CoinMarketCapData(currency).update()
File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/sensor/coinmarketcap.py", line 119, in update
self.ticker = Market().ticker(self.currency, limit=1)
File "/srv/homeassistant/lib/python3.4/site-packages/coinmarketcap/core.py", line 54, in ticker
response = self.__request('ticker/' + currency, params)
File "/srv/homeassistant/lib/python3.4/site-packages/coinmarketcap/core.py", line 28, in __request
response_object = self.session.get(self.base_url + endpoint, params = params, timeout = self.request_timeout)
File "/srv/homeassistant/lib/python3.4/site-packages/coinmarketcap/core.py", line 21, in session
self._session = requests_cache.core.CachedSession(cache_name='coinmarketcap_cache', backend='sqlite', expire_after=120)
File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/core.py", line 68, in init
self.cache = backends.create_backend(backend, cache_name, backend_options)
File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/backends/init.py", line 50, in create_backend
return registry[backend_name](cache_name, **options)
File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/backends/sqlite.py", line 28, in init
self.responses = DbPickleDict(location + extension, 'responses', fast_save=fast_save)
File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/backends/storage/dbdict.py", line 58, in init
with self.connection() as con:
File "/usr/lib/python3.4/contextlib.py", line 59, in enter
return next(self.gen)
File "/srv/homeassistant/lib/python3.4/site-packages/requests_cache/backends/storage/dbdict.py", line 70, in connection
con = sqlite.connect(self.filename)
sqlite3.OperationalError: unable to open database file

Same issue. Home Assistant version 0.57.3.
  - platform: coinmarketcap     currency: bitcoin     display_currency: EUR
2017-11-12 19:16:23 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up platform coinmarketcap Traceback (most recent call last):   File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity_component.py", line 170, in _async_setup_platform     SLOW_SETUP_MAX_WAIT, loop=self.hass.loop)   File "/usr/lib/python3.5/asyncio/tasks.py", line 400, in wait_for     return fut.result()   File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result     raise self._exception   File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run     result = self.fn(*self.args, **self.kwargs)   File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/sensor/coinmarketcap.py", line 55, in setup_platform     CoinMarketCapData(currency, display_currency).update()   File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/sensor/coinmarketcap.py", line 129, in update     from coinmarketcap import Market   File "/srv/homeassistant/lib/python3.5/site-packages/coinmarketcap/__init__.py", line 10, in <module>     from .core import Market   File "/srv/homeassistant/lib/python3.5/site-packages/coinmarketcap/core.py", line 6, in <module>     import requests_cache   File "/srv/homeassistant/lib/python3.5/site-packages/requests_cache/__init__.py", line 28, in <module>     from .core import(   File "/srv/homeassistant/lib/python3.5/site-packages/requests_cache/core.py", line 21, in <module>     ver = tuple(map(int, requests.__version__.split("."))) AttributeError: module 'pip._vendor.requests.__version__' has no attribute 'split'

As above I am getting errors:
- platform: coinmarketcap
    currency: bitcoin
    display_currency: GBP

Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/helpers/entity_component.py", line 171, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=self.hass.loop)
  File "/usr/lib/python3.5/asyncio/tasks.py", line 400, in wait_for
    return fut.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/components/sensor/coinmarketcap.py", line 55, in setup_platform
    CoinMarketCapData(currency, display_currency).update()
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/components/sensor/coinmarketcap.py", line 129, in update
    from coinmarketcap import Market
  File "/config/deps/lib/python3.5/site-packages/coinmarketcap/__init__.py", line 10, in <module>
    from .core import Market
  File "/config/deps/lib/python3.5/site-packages/coinmarketcap/core.py", line 6, in <module>
    import requests_cache
  File "/config/deps/lib/python3.5/site-packages/requests_cache/__init__.py", line 28, in <module>
    from .core import(
  File "/config/deps/lib/python3.5/site-packages/requests_cache/core.py", line 21, in <module>
    ver = tuple(map(int, requests.__version__.split(".")))
AttributeError: module 'pip._vendor.requests.__version__' has no attribute 'split'```

I can confirm I have the same issue as ntalekt

I have the same error since October. I do not think the mysql db is the problem. Usually the sensor creates a sqlite cache db and that does not work. For me permissions are ok, but the file is not created at startup of HA.
Could anyone of the python pros please take a look for a solution of this problem?

This is a problem within the coinmarketcap library. When there's a fix there, we can upgrade home assistant to use the fix. Follow https://github.com/mrsmn/coinmarketcap/issues/23 for updates.

A workaround for now is to change line 21 from backend='sqlite' to backend='memory' in the coinmarketcap package.
By default it should be located at path/to/your/venv/lib/python#.#/site-packages/coinmarketcap/core.py, although I don't know if changing it there is the best idea, as I'm not sure how HASS will handle updates, so beware of that.

Should be fixed in https://github.com/home-assistant/home-assistant/pull/11953.


Flutter  ListView builder is laggy and ruining the user experience in IOS/WEB.

I have been trying the flutter web from past one year and it have made significant improvements but when it comes to lazy loading with the list containing the images it can be improved.
I am using list view builder to load my list, On android device it is buttery smooth. when it comes to IOS/WEB at there is lots of frame drop. App itself is  crashing at times  in IOS. The page become laggy in web.
Please can someone give a solution for this.
Thanks.

Hi @yogesh-xseed
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample and flutter run --verbose logs running the minimal code sample?
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Avoiding auto convert to CRLF when commit

Changing the line breaks for create_db.sh from CRLF to LF to fix an issue during local postgres db creation.

It does not seem to be needed.


Copy directory path shouldn't have line number

Describe the bug
Copy directory path will generate line number, such as https://github.com/<org>/<repo>/blob/<hash>/path/to/dir#L1-L1.
To Reproduce
Steps to reproduce the behavior:

Select any directory
Copy
Paste
The link has current file line number or #L0-L0 if there is no opened file

Expected behavior
Generate https://github.com/<org>/<repo>/blob/<hash>/path/to/dir only

Thanks, that doesn't look right. I'll look to get that fixed this week.


SCV Sandbox refresh no how too

After a refresh of my Salesforce sandbox, I have been unable to complete the reconfiguration of the link to this package.  I have updated all the secret information and created new certs but authentication fails to SF with the error
Failed: Missing Consumer Key Parameter
Using Postman we were able to authenticate and placed the token in the config and functionality was temporarily restored but after 12 hours a 2nd test failed and returned to the same error. What would be the procedure to create a linkage to a new salesforce org with the same AWS set of lamdas?

Re built certs a 3rd time and it finally linked, must have been human error


Hard Coded Tic Tac Toe

A hard coded tic tac toe game in python to illustrate the best algorithm of the game..
can i work on this issue under IWOC 24

Ok go ahead.


Version 4.0.1: Unable to adjust the size of a chart in a dashboard when the chart touches the right edge of the dashboard

Bug description
Unable to adjust the size of a chart in a dashboard when the chart touches the right edge of the dashboard.
The only solution seems to delete the chart in the dashboard and import it again.
How to reproduce the bug

Go to a dashboard
Import a graph
Adjust the width to the maximum on the right size of the dashboard
Impossible to make it smaller then

Screenshots/recordings
No response
Superset version
4.0.1
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[x] I have searched Superset docs and Slack and didn't find a solution to my problem.
[x] I have searched the GitHub issue tracker and didn't find a similar bug report.
[x] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

Yes unable to resize if it is stretched to right most side of dashboard. But if we try to drag from the right bottom corner the drag handler appears. @Kim-frau,  have you got any solution?

@justinpark might have some ideas on this one.


Cassandra_stress and Redis use more VMs than requested machine_type

Running cassandra_stress or redis on a target with 4 VMs produces an error that CPUS are exceeded, limit 8. The limit 8 is because I using a GCS trial which limits to 8. But in any case the benchmark is using more VMs than is configured for the test run.
I ran cassandra_stress and redis successfully on a GCS 1 VM systems, but they fail on 4 and 8.

cassandra_stress doesn't use the the num_vms flag.  It always uses four VMs.  The redis benchmark uses redis_clients + 1 VMs.
You should be able to run either benchmark on n1-standard-1s (total cores of 4 for cassandra_stress and 6 for redis in the default configuration), but you won't have enough cores to run them with n1-standard-4s (total cores of 16 and 24) or n1-standard-8s (total cores of 32 and 48).

Hi Kris
Thanks for the reply. Your info explains why I can't run some benchmarks on some machines, but I don't know where this is documented or how to figure it out.
I am new to the server and vm world. When I look at the google info showing machine types, the n1-standard-1 supplies 1 vCPU or VM. I've read that typically 1 physical core would support 4 VMs.
Where is it documented that for this benchmark case 1 VM allocates or consumes 4 physical cores?
Best Regards,
-Dick
From: Kris Hildrum<EMAIL_ADDRESS>Sent: Thursday, June 16, 2016 5:58 PM
To: GoogleCloudPlatform/PerfKitBenchmarker<EMAIL_ADDRESS>Cc: Vail, Richard<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [GoogleCloudPlatform/PerfKitBenchmarker] Cassandra_stress and Redis use more VMs than requested machine_type (#1016)
cassandra_stress doesn't use the the num_vms flag. It always uses four VMs. The redis benchmark uses redis_clients + 1 VMs.
You should be able to run either benchmark on n1-standard-1s (total cores of 4 for cassandra_stress and 6 for redis in the default configuration), but you won't have enough cores to run them with n1-standard-4s (total cores of 16 and 24) or n1-standard-8s (total cores of 32 and 48).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/1016#issuecomment-226637609, or mute the threadhttps://github.com/notifications/unsubscribe/ATA2xMh8sDYE9o8pqRKY9GCbp3kPLEkDks5qMdT5gaJpZM4I1tJM.

The mapping between machine types and number of cores is on this page.  An n1-standard-k machine type has k virtual cores.
The benchmarks allocate a certain number of VMs.  How many cores total the benchmark uses depends on the machine type you use.  The number of virtual cores you'll need is:
VMs * #cores/VM for your machine type.
Google's accounting is based on the number of virtual cores.   The number of physical cores doesn't need to be part of the calculation at all.
Some benchmarks use a fixed number of VMs, some use a variable number.  You'd have to look at each benchmark to determine that.

Is there more to answer here or is this ok to close?


MacOS TextField does not work if window style is changed

After changing this value to "NO" all TextFields in the app stop working. (When you input anything - nothing happens).

  @override
  Widget build(BuildContext context) {
    return SizedBox(
      width: scale(widget.width),
      child: TextField(
        controller: widget.controller,
      ),
    );
  }

Platform: MacOS, Flutter: the latest from the stable channel

Hey @kirill-21. Can you specify where you're making changes?

Hey @kirill-21. Can you specify where you're making changes?

Triage report
I can reproduce this issue with Master (3.8.0-12.0.pre.68).
Steps to reproduce

Create a new Flutter project
Edit macos/Runner/Base.lproj/MainMenu.xib and change the titled attribute of windowStyleMask to NO. This is a child of the window tag towards the end of the file.
Run the app with the code sample below and notice that keystrokes are not recognized.

Code Sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Flutter Demo',
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: Text(title)),
      body: const Center(
        child: TextField(),
      ),
    );
  }
}

flutter doctor -v (Master)
[!] Flutter (Channel master, 3.8.0-12.0.pre.68, on macOS 13.2 22D49 darwin-arm64, locale en-US)
    • Flutter version 3.8.0-12.0.pre.68 on channel master at /Users/nabeelparkar/fvm/versions/master
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/2.19.1/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nabeelparkar/fvm/versions/master. Consider adding /Users/nabeelparkar/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4b8ad1b007 (86 minutes ago), 2023-02-14 22:29:23 -0800
    • Engine revision d860892528
    • Dart version 3.0.0 (build 3.0.0-233.0.dev)
    • DevTools version 2.21.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nabeelparkar/Library/Android/sdk/
    • Platform android-33, build-tools 33.0.0
    • ANDROID_SDK_ROOT = /Users/nabeelparkar/Library/Android/sdk/
    • Java binary at: /Users/nabeelparkar/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/221.6008.13.2211.9477386/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /Applications/Brave Browser.app/Contents/MacOS/Brave Browser

[✓] Android Studio (version 2021.3)
    • Android Studio at /Users/nabeelparkar/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9123335/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] Android Studio (version 2022.1)
    • Android Studio at /Users/nabeelparkar/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/221.6008.13.2211.9477386/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.75.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.58.0

[✓] Connected device (3 available)
    • Nabeel’s iPhone (mobile) • 00008101-000E48283A20001E • ios            • iOS 16.2 20C65
    • macOS (desktop)          • macos                     • darwin-arm64   • macOS 13.2 22D49 darwin-arm64
    • Chrome (web)             • chrome                    • web-javascript • Brave Browser <IP_ADDRESS>

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

/cc @cbracken

am having this same issue on android when using Go Router on only build apks


Genesis v6 from JSON v5 export and v5 JSON export

This PR covers two related/dependent features seen below as Step 1, 2 and fixes for some Move errors (see commits).
Steps:

v5 JSON export (new feature)
cargo r -p ol-genesis-tools -- --recover /opt/rec.json --snapshot-path /opt/state_ver*
state_ver: https://github.com/OLSF/epoch-archive/tree/main/359/state_ver_76353076.a0ff
Resulting rec.json: https://drive.google.com/file/d/1jaumaKdaGjWExNdUVqNZDvIAQPmppt19/view

Genesis v6 from JSON v5 (new feature)
Create genesis blob using v6 code from JSON v5 export:
cargo r -p ol-genesis-tools -- --fork  --recovery-json-path /opt/rec.json --output-path /opt/genesis_from_recovery.blob
Resulting genesis_from_recovery.blob: https://drive.google.com/file/d/1f2gfISy5AecZIXGQEOgP4yQm3ArpYRO1/view?usp=sharing

Verify the validity of genesis blob by starting a v6 node in test mode with it:
cargo r -p diem-node -- --test --genesis-modules /opt/genesis_from_recovery.blob
Validator logs: https://drive.google.com/file/d/1DCuKe4hY0VwzYS3nMsQie4o5uyt25coo/view?usp=sharing

Two questions spring to mind on this set of changes:

Why is compiled bytecode included in the commit? Is this due to wanting to have binary compatibility across the board and if so, would it not be better instead to collate the files and produce a hash which can be verified, so that the codebase doesn't end up full of binary artifacts?
It is not immediately clear to me where the tests are for the move contracts or rust code in this case. How are you checking that the migration from v5 to v6 behaved as expected?

Two questions spring to mind on this set of changes:

Why is compiled bytecode included in the commit? Is this due to wanting to have binary compatibility across the board and if so, would it not be better instead to collate the files and produce a hash which can be verified, so that the codebase doesn't end up with binary artifacts?

This is legacy from FB. It's a real PITA but many of the tests and builds depend on finding these files. Not something that can be easily fixed, we've tried a number of things.

It is not immediately clear to me where the tests are for the move contracts or rust code in this case. How are you checking that the migration from v5 to v6 behaved as expected (e.g. checking for equality between the accounts from v5 to v6)?

It seems no e2e tests have been done, or they are not documented.

This PR looks correct to me. Although we are missing e2e tests for this feature. Ordinarily we wouldn't merge this, but since we need to do cleaning up on the v6 branches I'll approve for merge.

LGTM


Support compilation on Linux for version 2.0

Tweaked my previous CMakeLists.txt file to work with theoraplayer v2.0
Fixed a few Linux compilation errors
Currently building ogg, vorbis, theora and theoraplayer libraries as well as glut_player demo

@borisblizzard forgot about this pull request, what do you think?

Seems more or less ok. I suggest that -D_UNIX is added and maybe we can remove the enc* files from complication since they aren't actually used. We only compile encoder_disabled.c and enquant.c as far as I can see in our VS setup.

agree. did you check the source changes too? not just the cmake file. maybe we should put the cmake file in a separate folder like we do with xcode and msvc?

I agree with moving cmake into its own directory. This might make it easier to switch things around in the future.
Source files seems ok, just a few additional header includes and a convention fix.

merged and verified, all seems to be ok. cmakeLists is in root actually which is ok, forgot we only checkout the theroaplayer folder as externals.

Thanks for merging this, I't good to have Linux support.
@borisblizzard: I'm just curious as to why you disable encoding in your build of the theora library? Is it simply to keep the size down? I find it helpful to have a theora lib that I can both decode with (using libtheoraplayer) and also encode with, so I always have to manually add the encoding support back in libtheora when I build libtheoraplayer for both Windows and Linux (which is my the enc* files are included in the CMakeLists.txt I submitted). Would you consider adding the encoding support when building libtheora by default in this project, or is there some other reason you are opposed to it?

I think we mostly removed it due to size and the fact that we have only ASM optimizations for decoding. I guess we could add it. Kreso, what do you think? I see no reason not to (unless we're getting compiler errors).

yes, we removed it to reduce size. and theoraplayer isn't designed for encoding purposes. if you need to use encoding and specifically theora, it's best if you use your own compilation environment. We have webM and we support apple's AVFoundation backend, and each of them has encoding done differently. it would not be systematic to include encoding in our theora library I think if it's not designed to be used in theoraplayer.
The lib doesn't include sources of theora on purpose, you can always supply your own theora distribution if you need special functionality.

@kspes: I understand where you're coming from. However, I like to use the same version of the library for both encoding and decoding to ensure what is encoded will be decoded properly (not to mention it is quite inconvenient to manage two versions of the lib). Since I use theoraplayer for decoding and it has the ASM optimizations, I always need to reconfigure the projects to add encoding support to libtheora whenever I rebuild libtheoraplayer so that I can use the same lib for encoding.
I'd like to ask you to reconsider this decision. As a suggestion, for the Windows solution, you could simply add a few more build targets that one could optionally choose if they would like to enable encoding support in theora. Does that sound reasonable?

@borisblizzard I suppose it couldn't hurt to add a new target to libtheora bundled with the project?

Yeah, sure. I'll take care of it on Monday.

Added. Complication, linkage and running seemed fine, haven't tested if the encoder works, though.


uninitialized constant WorldDb::Model::Region (NameError)

Hi @geraldb ,
I'm try to run the rake db:create or rake db:migrate command I'm getting the following error .
Ruby Version is : 2.3.0
Rails Version is : <IP_ADDRESS>
Bundler version is : 1.13.2
Can you suggest how do i get fix this error to get the beer.db and world.db ?
Thanks

@ramthakur1991 Welcome to world.db (and beer.db). Please send (setup/installation) questions to the forum / mailing list. Thanks. Cheers.


Cannot acces WSL from windows service

I want to use WSL to run scripts from my automated build system which is running as a service.  This is the latest version of TeamCity.
The service is logged on as a system user with admin rights.  for the same user, I have logged on on the desktop, and installed Ubuntu-18.
From a command line on the desktop, I can run wsl.exe and bash.exe. further, I can see wsl.exe:
dir c:\windows\system32\wsl.exe
Volume in drive C has no label.
Volume Serial Number is A813-6BF8

Directory of c:\windows\System32

09/17/2019  09:09 PM           142,848 wsl.exe
              1 File(s)        142,848 bytes
              0 Dir(s)  171,728,314,368 bytes free

However, from the service, my command scripts fail, claiming that
'wsl' is not recognized as an internal or external command,
operable program or batch file.

Further, doing the same exact dir command from here gives me:
dir c:\windows\system32\wsl.exe
Volume in drive C has no label.
Volume Serial Number is A813-6BF8

Directory of c:\Windows\System32

File Not Found

What magic is windows using to hide the files like this?  What is the correct way to do this?

This is not a WSL actionable nor was a repro provided, but it can be tag question for a short bit in the event there are takers.

Well, creating windows services isn't exactly trivial, so the repro case is, I suppose, to set up a teamcity build agent as described and have it try to execute wsl commands.
But also understanding how it is possible that the executables are not even visible from the service context would be useful.

But also understanding how it is possible that the executables are not even visible from the service context would be useful.

Sounds like that would be useful. But the likelihood of finding an answer to that question in a WSL bug tracker, contrast a question/discussion forum (possibly a forum related to teamcity, whatever a teamcity may be, or elsewhere) is.... probably small.

File system redirection with WOW64, almost certainly.
https://docs.microsoft.com/en-us/windows/win32/winprog64/file-system-redirector

If so then #223 might hint work-around.

Thanks, the 64/32 bit thing was the clue.  And the redirector is indeed the culprit for why the files weren't visible.
Turns out TeamCity uses 32 bit java to run:  https://teamcity-support.jetbrains.com/hc/en-us/community/posts/360000181539-Upgrade-a-build-agent-to-use-64-bit-java
So, a 32 bit command processor cannot start wsl.  Thanks for looking into this.

C:\Windows\Sysnative\wsl.exe
run WSL on ASP.net server side
Computer Windows
IIS server
ASP.net or Core web project
From web page ASP run on server side: call WSL/SWL2  run linux programm , return result to the web page
Problem after transfer project from IIS express to IIS
WSL.exe have not user iis app pool


add option for scheduler step once per loop for multi-GPU scenery

I find that when running
scheduler.step()
in a loop of
for batch in training_dataloader:
while cause scheduler step number of gpus per loop, is it possible to let it step only once per loop?
(I test it on single node, don't know multi-node scenery)

I can confirm I observe the same behavior on my side. Is this a bug ?

My current workaround is to not apply the prepare method on the scheduler anymore.

There is an argument in Accelerator to disable this (step with optimizer iirc)

As this isn’t a bug, it’s intentional


[Impeller] capture client labels are still allocated in release mode.

After disabling the command.h labels, I can observe the CPU traces still show that the capture client labels are still being allocated.
These calls either need to be if'def'd out, or the labels need to be made into static strings that aren't allocated (and free'd) per contents.

Could make these const string views or const char* and enforce they are not allocated dynamically.


Remove jcenter repository

Description
JCenter has been turned off and should be removed as a referenced repository.
Please Remove any direct dependency on jcenter() within a repositories block in gradle files.
Related: https://github.com/opensearch-project/opensearch-build/issues/1456

@opensearch-project/asynchronous-search can this be triaged and prioritized?

Sure @peternied
We will pick this up.

Looks like this has been resolved, closing


tls: use SSL_in_init() to check for handshake completion

In some cases (e.g. 0-RTT) BoringSSL will report the handshake as
completed by returning "1" from SSL_do_handshake(), even if the
handshake is not in fact complete.
So use "SSL_in_init()" to check for handshake completion.

I added a check to avoid parsing the peer's transport params multiple times.


Draft aesara-base package

Closes #76

Checklist

[X] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[X] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

CI failure was just due to a connection issue. Retrying...

This is unfortunately being blocked by https://github.com/conda-forge/mkl-service-feedstock/issues/24, because this recipe builds for PyPy 3.7, and it's impossible to install mkl-service and PyPy 3.7 simultaneously.

Can we drop pypy 3.7?

I also had that thought. There is a newer migration for pypy 3.8 & 3.9, so there doesn't seem to be much point in continuing to offer 3.7. I will ask on Gitter if there's a procedure for this, so that we don't confuse their migration infrastructure...
https://gitter.im/conda-forge/conda-forge.github.io?at=62c06471b408e830dc7a06c2

I found a selector for pypy which I was previously unaware of. Now I can set pypy3.7 to openblas. :smile:

@conda-forge-admin, please rerender

Cool, all green! :tada:

We should probably discuss the extra stuff to add to aesara-base for aesara. Currently I put:
        - {{ compiler('c') }}  # [not win]
        - cxx-compiler  # [not win]
        - {{ compiler('m2w64_c') }}  # [win]
        - {{ compiler('m2w64_cxx') }}  # [win]
        # Use openblas on Apple M1 or pypy3.7. For everything else, use mkl-service.
        - openblas  # [arm64 or (python_impl == 'pypy' and py==37)]
        - mkl-service  # [not (arm64 or (python_impl == 'pypy' and py==37))]

Is this desired, or do we want something else?

Is this desired, or do we want something else?

That looks good to me!

Todo: the ci infrastructure is not capable of running tests on the cross-compiled osx-arm64 packages, so these should ideally be tested by hand when we are ready. The builds can be downloaded under checks -> Azure pipelines as "artifacts".

@conda-forge-admin, please rerender

Weird!!! The Python executable suddenly can't be found for osx_arm64. Maybe something went wrong after I merged main and rerendered?

According to the diff before and after merging main and rerendering, it looks like I lost something in the CI configuration regarding libblas and liblapack. Probably I should have been rerendering more often. Could this really be the source of the missing python executable though?

Open question: why does cxx-compiler work but compiler('cxx') does not?

I opened two parallel draft PRs #81 and #82 to try to get to the bottom of the missing python executable on osx_arm64. I did this because each PR can run the CI independently.

It turns out that the issue with the failing builds was indeed a bug with the Conda-Forge infrastructure. :sweat_smile:
See on Gitter.

Thanks for your work on this, the meta.yml file is shaping up nicely. So libpython is not required at all then?

@twiecki thanks! And everything is green again! :tada:
It seems that libpython is some sort of Windows-specific package. It's unnecessary to list it as a direct dependency because it occurs in all Windows builds as an indirect dependency. (Unfortunately I'm having trouble determining from what.) It was added in https://github.com/conda-forge/aesara-feedstock/pull/29.
So if we have a particular reason that we need libpython we should probably list it as a direct dependency, but otherwise it seems fine to me as-is.

@conda-forge-admin, please rerender

@twiecki, I'm fairly satisfied with how this looks. Let's get comments from @brandonwillard. If he is satisfied, then let's seek a review from a BLAS expert on Conda-Forge. Perhaps we could ping conda-forge/blas?

Awesome, I agree that this looks much better. Let's wait on @brandonwillard.

It seems that libpython is some sort of Windows-specific package. It's unnecessary to list it as a direct dependency because it occurs in all Windows builds as an indirect dependency.

Could you explain a bit more why it's not needed explicitly?
I just chatted with @brandonwillard and he's fine with the approach. Just need to make sure we didn't break anything with installing the compiler chain and necessary libraries, but I guess the tests ensure that?

Could you explain a bit more why it's not needed explicitly?

Some other package pulls in libpython regardless of whether or not we do it explicitly. Thus in the end it makes no difference, unless whatever it was stops requiring it.
Since you're concerned, let's add it in with a comment. There's no harm in it, so if it increases confidence then let's do it.
The tests only check for errors / warnings on import. The tests do not run for osx-arm64.

Since you're concerned, let's add it in with a comment. There's no harm in it, so if it increases confidence then let's do it.

No, it's fine, just wanted to understand.

Thanks so much @maresb, this really made the package a lot better!

Some other package pulls in libpython regardless of whether or not we do it explicitly. Thus in the end it makes no difference, unless whatever it was stops requiring it.

Since Aesara uses the Python C libraries and headers directly, this is not a sound approach.
Generally, any time a codebase explicitly relies on a library, it should have that library as a direct dependency, even if that dependency is known to be provided by other dependencies; otherwise, one invites avoidable issues caused by changes in said dependencies (e.g. when a dependency removes its dependence on your indirect dependency, sets its own install options for the indirect dependency, etc.)

Since Aesara uses the Python C libraries and headers directly, this is not a sound approach.

Exactly, and because I was unsure whether or not this was a direct dependency I wanted your to review. :smiley:
But no harm, just a bit of extra work for me. We can fix this going forward in #88, and retroactively in #87.

Another important point is that the osx-arm64 packages have, to my knowledge, not been tested. Could someone please test aesara and aesara-base on M1?

Just sampled a model and that went fine.


adding a new import might invalidate/fix any of the existing nodes

This task is automatically imported from the old Task Issue Board and it was originally created by Greg Ociepka.
Original issue is here.

The file does not seem to insert Standard.Table it seems.
That's my header with imports:
from Standard.Base import all
from Standard.Table.Data.Table import Table
import Standard.Visualization
And I have
operator8 = [ Aggregate_Column.Group_By "gender", Aggregate_Column.Count Nothing ]
resulting in Aggregate_Column is unknown error
Discussed in the Week 1 BC challange: https://discord.com/channels/401396655599124480/1060875693199728720/1065594261824864326
Further discussion: https://discord.com/channels/401396655599124480/1071077992090050622
Comments:
@farmaazon please discuss this issue with @ilyabogdanov on how to start (Greg Ociepka - Feb 1, 2023)

closing:
https://discord.com/channels/401396655599124480/1071077992090050622/1072284282170658896

As per the discussion on discord, the specific import of Aggregate_Column is done but we need to check the reexport field and assess how the IDE decides.

Reexports were fixed in #6613


Zooming?

kbn_network is awesome - thanks so much for all the hard work!  Question: is there some way to zoom in/out on a graph, or do you have any plans to add that functionality?

Thanks!!
The zooming works using the Mouse wheel, Do you test it?

I just found it - yes. This thing is awesome.


add Owi

Add Owi, see #1620.

In order to add it, see this comment: https://github.com/analysis-tools-dev/static-analysis/pull/1620#issuecomment-2359171977
I'd appreciate a pull request.  😌


size props for textField

the props size for textField not working
this is the code for 2 textField
the "first name" field with size="small"
<Grid item xs={6}>
              <TextField
                id="outlined-dense"
                label="First name"
                value={this.state.firstName}
                fullWidth
                required
                name="firstName"
                margin="dense"
                variant="outlined"
                onChange={this.handleChangeFirstName}
                helperText={this.state.errorFirstName}
                FormHelperTextProps={{
                  error: true
                }}
                size="small"
              />
            </Grid>
            <Grid item xs={6}>
              <TextField
                id="outlined-dense"
                label="Last Name"
                name="lastName"
                value={this.state.lastName}
                fullWidth
                required
                margin="dense"
                variant="outlined"
                onChange={this.handleChangeLastName}
                helperText={this.state.errorLastName}
                FormHelperTextProps={{
                  error: true
                }}
              />
            </Grid>

size seems to be an alias for margin. Apparently it was added in #18624 to fix an issue in Autocomplete. But this shouldn't require an alias in TextField. This makes the component needlessly complex. I didn't look at the PR at the time because neither title nor labels indicated that this affected other components.

@mohamedtijani Make sure you update to the latest version.
@eps1lon The size and margin have different behavior. The margin prop extends the size prop with an extra margin.

This needs definitely a critical review

margin: (size === 'small' ? 'dense' : undefined) || margin,

-- margin: (size === 'small' ? 'dense' : undefined) || margin,
This looks not ok.
Same applies to the textfield demos. They fail to illustrate the difference between margin and size.

@eps1lon What solution would you propose for v5? What do you think of keeping the size demo and updating the layout demo to use something else than the margin prop (removing the prop).

I need to dive into this first. It isn't clear what problem TextField#size is trying to solve. In any case having a separate size and layout demo makes little sense. These concerns are the same.

@eps1lon There is definitely an overlap. I believe the inputs can be used in a wide variety of contexts. For instance here a small text field in Google Docs to leave a comment:

Benchmarking with Bootstrap is interesting: https://getbootstrap.com/docs/4.4/components/forms/#horizontal-form-label-sizing.
Maybe we still need this margin prop. What if we changed the implementation to have these two props isolated. Each with their dedicated impact? Meaning developers could use

the size prop to change the density of the TextField.
the margin prop as an alternative to Box, Grid, theme.spacing().

the problem is resolved
my previous version is 4.0.2, i upgrade it to the last version

I believe we have solved the issue: https://next.material-ui.com/components/text-fields/#sizes.

I believe we have solved the issue: https://next.material-ui.com/components/text-fields/#sizes.


wlan0 instead of eth0

@EricClaeys
If I recall correctly you pointed me to the place to modify the ethernet dashboard.  I know is is purely cosmetic but is there a way the default can have this changed in the repo?
<
cd /var/www/html/includes
sudo nano -l dashboard_eth0.php
line 65 change wlan0 to eth0

Hi @maserowik ... the file you're looking for is:
/var/www/htm/includes/dashboard_eth0.php
You'll see it around line number 65
Just change wlan0 on that line to eth0.

Hi @maserowik  -- I submitted a PR to fix this, and it was merged.  Thanks for reminding us about that one! :+1:
I'll go ahead and close the ticket.

@maserowik I could have sworn I fixed this after you first pointed it out.    Either way, thanks to @linuxkidd for actually fixing it.
@linuxkidd The LAN and WLAN pages on the GUI display the information correctly but (for me and another user) the "start" and "stop" buttons don't work.  If I manually execute sudo ifdown eth0 or sudo ifup eth0 (or the same for wlan0 they don't work either.  Do they work on your Pi?

Hmm... I'll have to get a different test setup going to play with that...  I've opened a new issue against allsky-portal -- I'll investigate/update there.


[pcl[vtk] or vtk] run time failure

Describe the bug
Program crash after X connection to p�}Td broken (explicit kill or server shutdown).
Host Environment
arm64-linux and x64-linux:
OS: Ubuntu 20.04 (Jetson Xavier NX and WSL)
Compiler: gcc-11 (Ubuntu 11.1.0-1ubuntu1~20.04) 11.1.0
CMake version: cmake version 3.22.0
PCL: 1.12
VTK: 9
To Reproduce
Steps to reproduce the behavior:

install plc and vtk from vcpkg
Compile the code with vscode cmake extension.
The code did compiled but when I run it, some errors occurred.

2Dplot.cpp
#include <iostream>
#include <pcl/io/pcd_io.h>
#include <pcl/point_types.h>
#include <pcl/visualization/cloud_viewer.h>

int main(int argc, char **argv)
{
	pcl::PointCloud<pcl::PointXYZ>::Ptr cloud(new pcl::PointCloud<pcl::PointXYZ>);

	if (pcl::io::loadPCDFile<pcl::PointXYZ>("lidarData.pcd", *cloud) == -1)
	{
		PCL_ERROR("Could not read the pcd file\n");
		return (-1);
	}

	pcl::visualization::CloudViewer viewer("Simple Cloud Viewer");

	viewer.showCloud(cloud);
	while (!viewer.wasStopped())
	{
	}

	return (0);
}

.../bin$ ./2Dplot
... ... ... ...
X connection to p�}Td broken (explicit kill or server shutdown).

[1]+  Exit 1                  ./2Dplot

Full log

2021-11-21 05:01:14.590 (   1.056s) [        5B244700]vtkOpenGLPolyDataMapper:303   WARN| vtkOpenGLPolyDataMapper::GetVertexShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::GetVertexShaderCode instead.
2021-11-21 05:01:14.598 (   1.064s) [        5B244700]vtkOpenGLPolyDataMapper:295   WARN| vtkOpenGLPolyDataMapper::SetVertexShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::SetVertexShaderCode instead.
2021-11-21 05:01:14.598 (   1.064s) [        5B244700]vtkOpenGLPolyDataMapper:333   WARN| vtkOpenGLPolyDataMapper::GetGeometryShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::GetGeometryShaderCode instead.
2021-11-21 05:01:14.598 (   1.064s) [        5B244700]vtkOpenGLPolyDataMapper:325   WARN| vtkOpenGLPolyDataMapper::SetGeometryShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::SetGeometryShaderCode instead.
2021-11-21 05:01:14.598 (   1.064s) [        5B244700]vtkOpenGLPolyDataMapper:318   WARN| vtkOpenGLPolyDataMapper::GetFragmentShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::GetFragmentShaderCode instead.
2021-11-21 05:01:14.598 (   1.064s) [        5B244700]vtkOpenGLPolyDataMapper:310   WARN| vtkOpenGLPolyDataMapper::SetFragmentShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::SetFragmentShaderCode instead.
2021-11-21 05:01:14.598 (   1.064s) [        5B244700]vtkOpenGLPolyDataMapper:303   WARN| vtkOpenGLPolyDataMapper::GetVertexShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::GetVertexShaderCode instead.
2021-11-21 05:01:14.598 (   1.064s) [        5B244700]vtkOpenGLPolyDataMapper:295   WARN| vtkOpenGLPolyDataMapper::SetVertexShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::SetVertexShaderCode instead.
2021-11-21 05:01:14.598 (   1.064s) [        5B244700]vtkOpenGLPolyDataMapper:333   WARN| vtkOpenGLPolyDataMapper::GetGeometryShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::GetGeometryShaderCode instead.
2021-11-21 05:01:14.598 (   1.065s) [        5B244700]vtkOpenGLPolyDataMapper:325   WARN| vtkOpenGLPolyDataMapper::SetGeometryShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::SetGeometryShaderCode instead.
2021-11-21 05:01:14.598 (   1.065s) [        5B244700]vtkOpenGLPolyDataMapper:318   WARN| vtkOpenGLPolyDataMapper::GetFragmentShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::GetFragmentShaderCode instead.
2021-11-21 05:01:14.598 (   1.065s) [        5B244700]vtkOpenGLPolyDataMapper:310   WARN| vtkOpenGLPolyDataMapper::SetFragmentShaderCode was deprecated for VTK 9.0 and will be removed in a future version.  Use vtkOpenGLShaderProperty::SetFragmentShaderCode instead.
X connection to p�}Td broken (explicit kill or server shutdown).

[1]+  Exit 1                  ./2Dplot

The only way to make the code compiled and running is to use VTK8 instead, which is weird.

Old vcpkg.json

{
    "name": "vcpkg",
    "version": "1.0",
    "dependencies": [
        "cuda",
        {
            "name": "vtk",
            "features": [ "cuda", "opengl" ]
        },
        {
            "name": "pcl",
            "features": [ "libusb", "cuda", "vtk", "opengl", "qt", "pcap" ]
        }
    ]
}

New vcpkg.json

{
    "name": "vcpkg",
    "version": "1.0",
    "dependencies": [
        "cuda",
        "proj4",
        "vtk",
        {
            "name": "pcl",
            "features": [ "libusb", "cuda", "vtk", "opengl", "pcap" ]
        }
    ],

    "builtin-baseline": "c9a123a0d27bbd7f68225cc101bba05c1abf2097",
    "overrides": [
        {
            "name": "proj4",
            "version-semver": "6.3.1#3"
        },
        {
            "name": "vtk",
            "version-semver": "8.2.0-13#0"
        }
    ]
}

Expected behavior
Showing graphs, the graph shown below is compiled with vtk8

Thanks for posting this issue. It looks like a upstream bug, Could you report this issue on Upstream: https://github.com/Kitware/VTK?

Close this PR until upstream fixes this.


Provide Kafka hooks for processing and other systems

This will allow better abstraction of processing from LSDC itself.
Requirements:

Kafka cluster must be up and accessible from AMX/FMX - needs investigation
Decisions need to be made on channel names and content that are consistent with DAMA standards
Decisions need to be made on what content is required for processing and whatever else people want to do with these hooks
Development of basic client code that will allow staff to develop the functionality they need.
Development of conda environments for server that include the confluent-kafka conda

May require methods to monitor who is using the channels

@JunAishima there is a kafka cluster running on kafka1.nsls2.bnl.gov, ..., kafka7.nsls2.bnl.gov. It is visible to all beamlines. You will need some new topics for the MX beamlines. Garrett and I know how to add them and we can show you how.

thanks, @jklynch - this work is being delayed for higher priority work. I will talk with you or Garrett to register a topic, I wasn't sure whether I needed to or not!


Support for syntax highlighting of bitvector constants in vim.

This commit makes bitvector constants like 0bv32 look pretty in vim.

@pramodsu LGTM. Thanks.


An attempt at fixing #12168 by only returning the cached data at the child path rather than at the parent

Discussion
This attempts to fix the issue reported here:
https://github.com/firebase/firebase-ios-sdk/issues/12168
Testing
I will be adding tests to this PR.
API Changes

I forgot how to run tests when using SPM, so I haven't had any luck with the testing yet.

cc: @aashishpatil-g


Voicemail button not available when user logged in to Basic client

When a EV enabled user logs in to Skype for Business Baisc client and another user searches for this user and Call to voicemail button is missing ( search > right click > call > voicemail button missing )
suggestion for correction :

we can add granularity of each feature which wont work
Or
Add a generalized statement " some of the advanced EV feature wont work"

Impact  : user cannot leave the voicemail directly

Hi @dipinn26592 can you kindly reference the document you are talking about in this issue?
Thanks for sharing this feedback, we really appreciate it.

@dipinn26592  Thank you very much for oping the issue. @get-itips could help if  reference document would be available. If you find the one please feel free to reopen this issue. Thanks again for taking out some time to open this issue.

https://docs.microsoft.com/en-us/skypeforbusiness/plan-your-deployment/clients-and-devices/desktop-feature-comparison

Hi @dipinn26592 thanks for sharing the referenced document.
I'll take a look at it and if it needs some edits I'll do it. Thank you again


Write full precision of float to config file

As this depends on the patch to the yaml code, consider this proof of concept, rather than fully mergable (although it should work).
This avoids issues when (re)writing config files and drifting floating point numbers due to the required precision to exactly store floating point numbers.
For details see http://www2.open-std.org/JTC1/SC22/WG21/docs/papers/2005/n1822.pdf
I have reported the yaml-cpp bug upstream (https://code.google.com/p/yaml-cpp/issues/detail?id=272)

looks like upstream moved to github.... and the issue might be lost but there is a pull request targeting a similar fix
https://github.com/jbeder/yaml-cpp/pull/470
Kevin

@KevinJW Please double-check with master because changes were recently done where some Ops now hold double values (instead of float values) i.e. #640 for Matrix, etc.

Kevin, no need to recheck this quite yet.  As Patrick wrote, the underlying MatrixOpData is now able to hold doubles and we read double precision from CLF files (and have a unit test for that).  HOWEVER, the YAML load/save is still float based.  So it's very likely that the issue you wrote this PR to address is still present.  Please stay tuned and we'll get back to you on this.

Even if the MatrixOp values moved to double precision, the YAML read/write was not updated.
A pull request will soon pop to complete the MatrixTransform transition to doubles.

The pull request #724 fixes the Matrix precision.

Note that the real bug in the yaml library limits the correct output of float or double - this aspect relates to #690 if we bundle a broken library we'll always be limited to its output capabilities. yes switching to doubles 'fixes' the float precision issues, but from my reading of the code doubles will still break.
For reference the fix to the yaml-cpp https://github.com/jbeder/yaml-cpp/pull/649 is from December 2018, so either we bump the bundled version and deal with any fall out, and we document that older versions will have a slight chance of this occurring.
Kevin

Agree. As you mention, it should be part of the ext library refactoring needed for the ASWF transition (i.e. update all 'core' third-party libraries to their latest version).

OCIO v2 is now using the newest yaml-cpp release by default. Does that solve the issue this PR was intending to fix @KevinJW ?


qt6.qtbase: enable debug info

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Should we do it for all the Qt modules at this point? It would be valuable for Plasma at least.

I don't mind, but people get nervous about enabling debug info across the board, so I only ever PR it where I've actually needed it and tested that it works.

It's definitely fine on all of Qt.


cli currently doesn't compile on master due to clap arg configuration

This is a result of this clap arg taking a char not a &str. See error below. I have a pull request with a fix that I'll submit as well.
   Compiling sqlx-cli v0.1.0-beta.1 (/home/khampton/personal/sqlx/sqlx-cli)
error[E0308]: mismatched types
 --> sqlx-cli/src/opt.rs:8:20
  |
8 |     #[clap(short = "D", long)]
  |                    ^^^ expected `char`, found `&str`

error: aborting due to previous error

Looks like this was fixed in https://github.com/launchbadge/sqlx/pull/686, so the issue can be closed.


Defer timer creation to DeviceSelfManagedIoInit callback

Fixes #41 by avoiding the need for an arbitrary 100ms wait. DeviceSelfManagedIoInit is selected since it's the last callback invoked before the device is operational according to Power-Up Sequence for a Function or Filter Driver

I notice that subscopes are used a lot, like:
Function DoSomething() {
{
           DoThis();

}
{
           DoThat();

}
}
I have some theories, but why is the code like this?
From: Fredrik Orderud @.>
Sent: Thursday, January 25, 2024 12:16 AM
To: forderud/IntelliMouseDriver @.>
Cc: Bryan Allen Young @.>; Review requested @.>
Subject: Re: [forderud/IntelliMouseDriver] Defer timer creation to DeviceSelfManagedIoInit callback (PR #43)
@forderud commented on this pull request.

In TailLight/device.cpp https://github.com/forderud/IntelliMouseDriver/pull/43#discussion_r1465991516 :

@@ -113,30 +147,6 @@ Routine Description:
return status;

return status; is here done if the WmiInitialize call fail. However, this code isn't changed in this PR so you don't need to bother about it.
—
Reply to this email directly, view it on GitHub https://github.com/forderud/IntelliMouseDriver/pull/43#discussion_r1465991516 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGBDV77LNXVCARLA7HA24U3YQIIDPAVCNFSM6AAAAABCHEHJP6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQNBTGA3TMOJZGE .
You are receiving this because your review was requested.  https://github.com/notifications/beacon/AGBDV7Z5HG6ZRDLT3DUOLTTYQIIDPA5CNFSM6AAAAABCHEHJP6WGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTTN3MPX6.gif Message ID: @.*** @.***> >

I notice that subscopes are used a lot, like: Function DoSomething() { { DoThis(); } { DoThat(); } } I have some theories, but why is the code like this?

I like to use subscopes due to the following reasons:

It reduces stack usage for variables that are only used in parts of the function. The stack is more limited in kernel-mode compared to user-space.
It improves readability, since it restricts variable lifetime to only the parts of a function where they're used. You can think of this as a sort of "lightweight function".
Reduced variable lifetime makes it easier to effectively utilize C++ destructors when appropriate.

@ManOnTheMountainTech Any more comments on the changes or are you ok with them?

No further comments. Going forward there is some cleanup work to do.


Add bingo hall preset

Saw this diary post and thought it would be a useful preset to have: https://www.openstreetmap.org/user/UKChris/diary/391755 Used over 100 times in the UK.
name: Bingo Hall (or maybe it should be venue?)
terms: amenity=gambling, gambling=bingo

Okay, I added a Gambling Hall for preset for amenity=gambling. I don't think a specific preset for gambling=bingo is necessary since even the Buzz Bingo brand wouldn't match it since it has gambling=bingo;slot_machines.


Missing example to save and reload a trained model

I installed the requirements and the demo for object detection works fine.
Is there a way how I can save and reload a trained model after the task.fit call?
detector = task.fit(dataset_train,
num_trials=2,
epochs=epochs,
lr=ag.Categorical(5e-4, 1e-4),
ngpus_per_trial=1,
time_limits=time_limits)
I also tried to set a resume to False to initialize the task to predict directly, but training is always started from the beginng.
?

That's good question. We should improve the documentation.
When calling task.fit(...), the training will start. If resume=True, it will start will the existing search history.
If you would like to save/load the trained model, please use detector.save('detector.ag') and detector= Detector .load('detector.ag')

Thanks @zhanghang1989 for your response.
I tried to call the save method after training, but receive an error.
# start training
detector = task.fit(dataset_train,
                    num_trials=2,
                    epochs=epochs,
                    lr=ag.Categorical(5e-4, 1e-4),
                    ngpus_per_trial=1,
                    time_limits=time_limits)

# save model to disk
detector.save("motorbike.ag")

Error message:
/usr/lib/python3/dist-packages/apport/report.py:13: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
import fnmatch, glob, traceback, errno, sys, atexit, locale, imp
Traceback (most recent call last):
File "demo_eval.py", line 24, in
detector.save("motorbike.ag")
File "/home/user/.local/lib/python3.7/site-packages/autogluon/task/object_detection/detector.py", line 154, in save
raise NotImplemented
TypeError: exceptions must derive from BaseException

Thanks for the feedback @rebotnix . We will prioritize the implementation of save/load method.