New Code Example: Pre-training BERT on NSP+MLM using Hugging Face Transformers

This code example shows how to pre-train BERT using Hugging Face transformers on NSP + MLM tasks. Additionally, it also shows best practices for Hugging Face Datasets.
@fchollet apologies for the duplicate PR on this, but due to shared machines between teammates, the original PR #966 messed up a bit with GitHub usernames pushing to the code and Google needing CLA. This has all the changes mentioned and is ready for merging!

Thank You so much! Also, I understand you are busy so no problem at all! Hope you could review some of my other existing PRs which are already under review whenever you get the time!

@Sreyan88 hello Sreyan! I'd like to thank you on behalf of 🤗 folks for this example!
I see in the example you're trying to push to keras-io organization, however, you're not a member of it. I'd like to help you out in building a demo for that as well, we have a Keras working group on our discord where we put Keras examples on https://huggingface.co/keras-io. You can join us in hf.co/join/discord 🙂


Browser tests are now failing with "fs.createReadStream is not a function"

Even seemingly innocuous tests, like the level1 and level2 tests, are now triggering this. Not sure if this was one of the recent commits, or some dependency update (browserify?), or what... Help appreciated.

Caused by https://github.com/tmpvar/jsdom/commit/1773d704b819f76b609a447345dfc01e96a5d474, hmm. @nicolashenry any ideas? Once you get the environment set up, you do npm run test-browser to reproduce. You didn't change a lot of things in resource-loader.js, so I am not sure why this would start breaking...

Oh sorry, I forgot those tests, I will take a look as soon as possible.

I think I found part of it. Some of our test fixtures were referencing non-existant script files. Somehow your work (maybe the changes to HTMLScriptElement?) un-hid that bug.
I think I have a fix; will push it soon.

This should be all taken care of now :)

Oh ok, nice :)


[WIP] Restrict DropdownButton generic to not allow nullable type T

Currently, if you try to set DropdownButton's generic type to a nullable value, it causes issues at runtime for tests (see this comment for a reproducible sample). This PR simply restricts T to be an extension of Object, which prevents it from being a nullable type. I think that this is needed because the null-safe implementation of DropdownButton needs to be backwards compatible for Flutter apps that have not migrated to null safety yet, but we also want those who have migrated to null-safe applications to not be confused by DropdownButton's behavior.
May address https://github.com/flutter/flutter/issues/74389.
Edit: Also, not sure how this can be tested
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

So, null should no longer be a valid option? Or would DropdownMenuItem.value and the parameter for DropdownButton.onChanged still be nullable?

This was mostly an experiment because I was unclear about what was causing the problems described in the issue. It seems like it might be a problem with the Dart VM itself, so let's wait and see what the Dart folks say

Closing the PR for now


Chapter 8 Block Data

Populate chapter 8 challenge lesson 2

https://www.figma.com/file/LqjK3Tpvd9KJ4buFArCJBQ/Saving-Satoshi?type=design&node-id=7329-18881&mode=design&t=pswCTplFjcGtpKxW-0

Closed by #919


🚀 Feature Request: kv-asset-handler granular caching

Describe the solution
We should be able to add specific caching rules for specific patterns.

This should become obsolete once Assets for Workers lands later this quarter, no?

kv-asset-handler (and so Workers Sites) is not the recommended way to host assets. Either we should use Pages or wait for the new Workers + Assets to land. Closing this as we won't have capacity or desire to add this to kv-asset-handler.


UI SDK - Forgot password - "Go back to login"  button element has undefined id

Created by: Mikuláš Mascautanu Aug 13 2021 at 11:09 CET
Updated: Jul 18 2022 at 11:30 CET
ClickUp ID: IK-1356
On the Forgot password page, the "Go back to login"  button has undefined id.
Just a guess, maybe the issue is a missing id prop in here:

Commented by: @Mikuláš Mascautanu Oct 13 2021 at 13:24 CET
@dusan-kovacik-profiq Is this version deployed somewhere? I don't have access to indyKite's github anymore so I cannot run the branch locally. (I could probably ask to get the permissions again)


Implement DNSStubListenerExtra for resolved.conf

Untested now, but something I need in my setup and having a PR makes testing easier.

OracleLinux 7  is just messed up in acceptance tests. I think we should drop it.


Performance differences Vs. multiple pointclouds

Hello!
I wondered, how does your method compare to processing 256 separate tiles and loading them all in the viewer together? I wondered, did you try that and experience loading issues (certain tiles not showing up)?

Answering my own question, for reference, have a 120 billion point cloud. I made separate pointclouds of about 1 billion points each and loaded them all in the same viewer, offset from eachother. The result was a significant stalling (5-10 seconds) of all menu operations. Toggling EDL, changing splat quality, all that.
The 3D view, however, was not terribly impacted. double clicking a new pivot center causes a stall, but otherwise navigation is as smooth as normal.


dbeaver/dbeaver#26613 Fix json format for binary column exporting

After discusion with @serge-rider, i decided to not extract it to common


The NBTAPI Plugin gets shaded

Hey there.
Your plugin uses the maven shade plugin, but didn't mark dependencies as provided, causing them to get shaded.
Currently, this results in the RyseInventory plugin overwriting parts of the NBTAPI plugin, potentially breaking both and anything depending on them entirely. This issue also extends to many compile-time annotations that get shaded into the final jar.
Either shade the API(with a relocation), or mark it as provided(in this case the plugin.yml is missing its declaration).
For more info: https://github.com/tr7zw/Item-NBT-API/wiki/Using-Maven

Hey.
Thanks for the message. Unfortunately I didn't notice it myself.
The API should now be shaded with a relocation in update 1.4.0.
This issue is not closed directly. If you still encounter errors, feel free to post them here. Otherwise I will close the issue on August 14.

Oh you mean
instead of this:

de.tr7zw
item-nbt-api-plugin
2.10.0

I'll take that:

de.tr7zw
item-nbt-api
2.10.0

Or am I possibly misunderstanding you right now?
Am Di., 16. Aug. 2022 um 17:02 Uhr schrieb tr7zw @.***>:

Am on holidays so cant check it directly, but I think it's still wrong.
With shading you need to use the item-nbt-api, not item-nbt-api-plugin.
—
Reply to this email directly, view it on GitHub
https://github.com/Rysefoxx/RyseInventory/issues/6#issuecomment-1216761227,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOEYYL4VXP5SH24FIO5EZBLVZOUOZANCNFSM56J3QVCQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Oh you mean
instead of this:

de.tr7zw
item-nbt-api-plugin
2.10.0

I'll take that:

de.tr7zw
item-nbt-api
2.10.0

Or am I possibly misunderstanding you right now?
Am Di., 16. Aug. 2022 um 17:02 Uhr schrieb tr7zw @.***>:

Am on holidays so cant check it directly, but I think it's still wrong.
With shading you need to use the item-nbt-api, not item-nbt-api-plugin.
—
Reply to this email directly, view it on GitHub
https://github.com/Rysefoxx/RyseInventory/issues/6#issuecomment-1216761227,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOEYYL4VXP5SH24FIO5EZBLVZOUOZANCNFSM56J3QVCQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Yea exactly. The -plugin dependency is the plugin itself, only used when having it as a normal dependency in the plugin.yml.


Remove ProjectExtensionsTaskRegistrationAction, apply a plugin instead

That way, no need to check for .kts files presence.
Builds not using gsk won’t suffer the performance penalty anymore.
Builds using gsk will benefit a small performance gain.
See #320

With this approach, a multi-project build with its root project using a Groovy build script and subprojects using Kotlin build scripts, the root project won't get the gskGenerateAccessors task.

This PR doesn't implement the exact same behaviour as before. In particular, a multi-project build with a root Groovy project and a Kotlin sub-project should still expose gskGenerateAccessors.

Yep, let's discuss this today.

@bamboo could you have another look?


[FYI] New homepage and logo

FYI only, posting this here only in order to reach a broader audience.
The React project is currently in the process of updating and redesigning its homepage and getting a new logo. If you want to participate, check out the following two issues:
https://github.com/reactphp/website/issues/8
https://github.com/reactphp/website/issues/10

Hi!
Thanks for the news, I was wondering if the project was still alive because the last commit is more than 1 year old...
Event driven and async code is the current hype with javascript, and i really love see it coming to PHP...
Please provide a project status...

@jason69 yes it is, however all development is now done on all the components own repositories, you can find those at: https://github.com/reactphp

@jason69 Very good comment! Hype with JS is just unbelievable that I begin to wonder how Linux kernel works w/o JS or async-everything stuff. :)
This project is just great, combining PHP's intuitive nature with async event loop, which can be useful for a certain tasks (not for all).

New logo is now in place via https://github.com/reactphp/branding/pull/3 :tada:
New homepage is now live via https://github.com/reactphp/website/pull/11 :tada:
See also http://reactphp.org


RetryableError is not retried

I just got an error saying

Error creating machine: Error in driver during machine creation: network/InterfacesClient:CreateOrUpdate 429 Failure responding to request -- Original Error: azure: Service returned an error. Code="RetryableError" Message="A retryable error occured." Status=429

does swagger spec actually specify retryable errors and do we make use of any retry logic in the SDK. I have been getting TLS handshake and network timeouts from Network RP all the time in the past week and it immediately fails.
Maybe we could make use of some retries in case of connectivity issues even if we don't have retryable codes specified in swagger specs.

@ahmetalpbalkan There is no default retry for errors today. The 429 you received ("too many requests") suggests you may have been thrzottled. A retry under, in that case, would require a moderate delay. Retrying general errors, say HTTP 500, is a bit risky. We can consider adding the support.

It's very unlikely that I got throttled (doesn't throttling a different status code?). I tend to think HTTP 500s are meant to be retried on the user and with a reasonable backoff. Unless we have such retries (especially in operation status checks via polling), the entire operation might seem like failed yet it actually would succeed.
I think we should consider reducing flakiness somehow. Based on my experience in the past week, RPs aren't reliable at all and they drop connections or return such errors all the time.

@ahmetalpbalkan I'm happy to adjust the ARM packages based on observed behavior. Let's chat and come up with what we should do.

this won't happen when we poll correctly, so closing.


Minor improvements to smoothing rules

Codecov Report
Base: 90.21% // Head: 90.43% // Increases project coverage by +0.21% :tada:

Coverage data is based on head (c6792aa) compared to base (d08156a).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #70      +/-   ##
==========================================
+ Coverage   90.21%   90.43%   +0.21%
==========================================
  Files          12       12
  Lines         552      554       +2
==========================================
+ Hits          498      501       +3
+ Misses         54       53       -1

Impacted Files
Coverage Δ

src/smoothing.jl
100.00% <100.00%> (ø)

src/discrete_randomness.jl
94.00% <0.00%> (-0.12%)
:arrow_down:

src/backends/dict.jl
91.80% <0.00%> (+1.48%)
:arrow_up:

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

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


"scm.providerCountBadge" not working

VSCode Version: 1.50.1
OS Version: Ubuntu 20.04

The "scm.providerCountBadge" setting no longer works (none of the 3 option).
If i remember correctly, this is responsible for the number on the right. But even if i set it to "hidden", it's still visible.
Currently it acts like it's always on "visible".

Does this issue occur when all extensions are disabled?: Yes

The scm.providerCountBadge only affects the repositories in the Source Control view, not the Source Control Repositories view. This is because we'd like all actions to be aligned in the Source Control Repositories view and this would break alignment. The setting description also makes this clear:

These headers only appear when there is more than one provider.

The scm.providerCountBadge only affects the repositories in the Source Control view, not the Source Control Repositories view. This is because we'd like all actions to be aligned in the Source Control Repositories view and this would break alignment. The setting description also makes this clear:

These headers only appear when there is more than one provider.

@joaomoreno
scm.providerCountBadge doesn't work in Source Control view either.
https://user-images.githubusercontent.com/20318608/106373486-ec9e6a00-63b4-11eb-9ddc-dfb25f344c4a.mp4
And I have tested the PR I have made, it only affect Source Control view

@joaomoreno
scm.providerCountBadge doesn't work in Source Control view either.
https://user-images.githubusercontent.com/20318608/106373486-ec9e6a00-63b4-11eb-9ddc-dfb25f344c4a.mp4
And I have tested the PR I have made, it only affect Source Control view

@joaomoreno Hi could you relook this issue?

@susiwen8 This is about the count badge on Source Control Provider headers, not Source Control Resource Groups. To see the providers in the Source Control view, please enable scm.alwaysShowRepositories, or open more than one repository.

@susiwen8 This is about the count badge on Source Control Provider headers, not Source Control Resource Groups. To see the providers in the Source Control view, please enable scm.alwaysShowRepositories, or open more than one repository.

Got it,  thanks


sampling delegation

I'm still testing and documenting this.

Thanks for taking a look, Damien. I'm going to shine this up into a real non-draft pull request after Christmas. Then we can revisit it, and focus on the dd-trace-cpp branch that hidden in here, too.

Thanks for the review, Damien.
I'm going to make the configuration directive change you recommended: on|off directives require the on|off parameter.
As for whether to support delegation in auth subrequests, I am starting to think it might be better to remove the feature. You make a good point that once we add it, it is difficult to remove. I'll think more about this, and possibly modify the code along with the configuration changes before merging.

I'm starting to see a flaky test:
Verify that when `datadog_trace_locations` is `on`, the span ... ok

======================================================================
FAIL: test_in_http (cases.configuration.test_configuration.TestConfiguration)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/david/src/nginx-datadog/test/cases/configuration/test_configuration.py", line 62, in test_in_http
    self.assertEqual(config, expected)
AssertionError: {'col[206 chars]tp://bogus:1234/v0.7/config', 'request_timeout[624 chars]12]'} != {'col[206 chars]tp://localhost:8126/v0.7/config', 'request_tim[676 chars]12]'}
  {'collector': {'config': {'event_scheduler': {'type': 'datadog::nginx::NgxEventScheduler'},
                            'flush_interval_milliseconds': 2000,
                            'http_client': {'type': 'datadog::tracing::Curl'},
-                           'remote_configuration_url': 'http://bogus:1234/v0.7/config',
?                                                               ^ ^^    ^^

+                           'remote_configuration_url': 'http://localhost:8126/v0.7/config',
?                                                               ^ ^^^^^ + +  ^

                            'request_timeout_milliseconds': 2000,
                            'shutdown_timeout_milliseconds': 2000,
-                           'telemetry_url': 'http://bogus:1234/telemetry/proxy/api/v2/apmtelemetry',
?                                                    ^ ^^    ^^

+                           'telemetry_url': 'http://localhost:8126/telemetry/proxy/api/v2/apmtelemetry',
?                                                    ^ ^^^^^ + +  ^

+                           'traces_url': 'http://localhost:8126/v0.4/traces',
-                           'traces_url': 'http://bogus:1234/v0.4/traces'},
?                            -------

+                           'url': 'http://bogus:1234/v0.4/traces'},
                 'type': 'datadog::tracing::DatadogAgent'},
   'defaults': {'environment': 'fooment',
                'service': 'foosvc',
                'service_type': 'web'},
   'environment_variables': {},
   'extraction_styles': ['B3', 'Datadog'],
   'injection_styles': ['B3', 'Datadog'],
-  'runtime_id': 'a52051f2-2e3f-4762-b453-f90912b4cbd4',
+  'runtime_id': '262e8005-a704-49a3-a03a-772ee0d874ac',
   'span_sampler': {'rules': []},
   'tags_header_size': 512,
   'trace_sampler': {'max_per_second': 200.0, 'rules': []},
   'version': '[dd-trace-cpp version v0.1.12]'}

----------------------------------------------------------------------
Ran 72 tests in 82.856s

FAILED (failures=1)
make: *** [Makefile:73: test] Error 1
david@ein:~/src/nginx-datadog$

It looks like once in a while, the request is served by an older or newer instance of nginx, as opposed to the one under test. The error goes away when you run just the one test, because then there is only once instance.

Ah, it's not flaky, it's just broken (I think). When I tried to run the particular test, I ran the wrong one.
It fails reliably now:
$ make test TEST_ARGS='--verbose cases.configuration.test_configuration.TestConfiguration'

Probably has to do with the change in the configuration JSON since features have been added to dd-trace-cpp.

I revisited the question of whether
datadog_delegate_sampling;

should instead be
datadog_delegate_sampling on;

and I've decided to keep the zero-parameter version.
I suspect that the question "what is the default?" will be less common than "what does it want as an argument?"
I could be wrong, but I find the minimalism of datadog_delegate_sampling; compelling.
@dmehala


Catalina support

modify most MsgId

can you add the missing semi colon so the build works?

add catalina、mojave、highsierra support file

This works like a charm

Discovering devices, characteristics, services works but reading characteristic hangs. Any luck with reading?

line 5 should be changed from
var debug         = require('debug')('mojave-bindings');
to
var debug         = require('debug')('catalina-bindings');

Mystery solved. The magic number for reading is 110.
I did pull request to pull request: https://github.com/lzever/noble/pull/1 :)
Here is the fork with the fix:
https://github.com/gad0lin/noble

Please forward change to downstream fork:
https://github.com/noble/noble/issues/923


chore: refactor csi-driver, use filesystem enum

I suggest you rename chore: refactor to just refactor:

bors merge


Add bindings for Cairo.Region

This also adds some additional wrapper code for Cairo.RectangleInt in order to implement the unit tests (e.g. constructing a rectangle, comparing equality, and field accessors) - this is the main thing I'd like to discuss since it's a more general topic on the generator side of things!
The extra level of indirection to access the fields (RectangleInt vs RectangleIntData) feels kind of awkward - my current changes have some rough code that attempts to hide this. I feel like it would be more natural if the generator could just directly emit a struct type here, since on the C side this is a plain struct rather than an opaque / handle type
I think it might also make something like cairo_region_create_rectangles() simpler to wrap as well since the memory layout would match
The old cairo bindings from Mono / GtkSharp also made this a struct type, so changing from value to reference semantics would also make it tricky to port existing code correctly

Should wait until #622 is fixed to rely on improved struct generation.

I've rebased this on top of the latest changes - the RectangleInt struct is much more usable now without any manual bindings 👍
The two awkward things I encountered are:

Constructing it requires new RectangleInt(Internal.RectangleIntManagedHandle.Create()) for now, as was noted in the release notes
Equality comparison doesn't work, I think because it includes the Handle member. So code which could have been written as  rect_copy.Should().Be(rect_init) requires comparing all the members explicitly. Mostly just an annoyance for writing unit tests :)

Ah good point! I will add some equality check to the generator which just compares the pointers.


Rajout des pre-commit hooks
What is the new behavior?

[ ] Yes
[x] No

Other information

Nice job, on aime ca comme ca.


[INLONG-2856][Manager] Support multi-source and multi-sink in one stream

Title Name: [INLONG-2856][Manager] Support multi-source and multi-sink in one stream
Fixes #2856
Motivation
Modifications
1 change table involved
2 support multi-source in one stream
3 support multi-sink in one stream
Verifying this change

[X] Make sure that the change passes the CI checks.

Documentation

Does this pull request introduce a new feature? (yes / no)

It is recommended to set your IDE's single-line code length to 120 characters.


xSQLServerAlwaysOnAvailabilityGroupDatabaseMembership: New Resource

Details of the scenario you try and problem that is occurring:
Manage the databases that are members of availability groups
xSQLServerAlwaysOnAvailabilityGroupDatabaseMembership
{
	[DatabaseName = [string[]]] # Array of database names to add to the availability group. Wildcards are supported.
	SQLServer = [string]
	SQLInstanceName = [string]
	AvailabilityGroup = [string]
	[BackupPath = [string]] # UNC path that is accessible by both the primary and secondary instances. This is required if the seeding mode of the Availability Group Replica is Manual. This is always required for SQL 2012 and 2014.
	[Ensure = [string]{ Absent | Present }] # Defaults to Present
	[MatchDatabaseOwner = [switch]{ $true }] # Ensures the database owner on the secondary matches the owner on the primary. PsDscRunAsCredential must have impersonate rights to run as the database owner in order to perform the database restore. The database owner must exist on all instances.
}

The DSC configuration that is using the resource (as detailed as possible):
n/a
Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
SQL Server 2012, 2014, 2016
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
dev

I plan on working on this next. I'm thinking of writing this as a class-based resource. Any comments or concerns before I get rolling?

No comments, looks good to me. And class-based should be all good (don't know if we have to tweak AppVeyor testing maybe). Go for it. 😄

One thing I've identified that will be significantly different is the help file. As it stands now, we do comment-based help in every resource. With the class-based resource I need to create module-based help using .help files.
Quite frankly, we should consider doing this for the project rather than using comment-based help so the end user can utilize the help without cracking open the script.

Comment-based help is requirement for the .mof resources to align with the guideline and HQRM. If we can get as close as possible to that with a class-based resource that would be good.
This is new territory, don't know if any module has a class-based resource yet. So we just have to do what we think is best and go from there :smile:
I did not know about the .help files. I can't find any documentation on it, do you have a link?
Also it seems it could be comment-based help for class-based resource in the (distant) future:
https://github.com/PowerShell/PowerShell issue 2225
Regarding doing this for the the entire project, I don't know. That would mean yet another place to have the parameter description for a .mof resource.
Currently there are in the comment-based help, in the .schema.mof file, and in the README.md. README.md hopefully will be removed soon when we can start using the wiki, like xNetworking and SharePointDsc.

Makes sense. It just seems odd that we're writing help in a location that is not helpful for the end user to get to. Maybe I'm missing something too.
Here's the link for the .help files.
https://msdn.microsoft.com/en-us/library/dd878343(v=vs.85).aspx
I'm all about getting rid of duplicated effort! I'd love to see the help removed from the psm1 files, especially since the help is already in the schema.mof files. That would make the .psm1 files much smaller and much more manageable. :)

Yes, I agree that adding the same information in two (and currently three) locations are to many. I could write a issue in DscResources to query about the guideline of comment-based help.

Thanks @johlju , I appreciate you raising the question!

Apparently a class-based resource cannot be mixed with a MOF-based resource. :( I'm starting to convert it to  MOF-based resource now.

Oh! :/ How do you mean? Because of the module as a whole is MOF its not possible to compile a configuration?
That is to bad, I was looking forward having the first class based resource :)

I can't get the module to recognize a class-based resource exists until I remove all of the MOF based resources. I didn't even try to compile a configuration because it didn't see the resource when I did a Get-DscResource.

Okay! That doesn't seem good, then we can never have class-based resources unless creating a new module. :/
@kwirkykat and @mbreakey3 is this the case? Or is there something missing to get mof and class-based to work together?

Yes, this is the case. Currently you can't have both mof and class-based resources in the same module.

I'm pretty sure you can have both in the same module. We were considering adding WindowsPackageCab to xPSDesiredStateConfiguration as a class-based resource. We didn't because that would require xPSDSC to require WMF 5.0.
@randomnote1 What have you tried? Did you add the class-based resource to the 'DscResourcesToExport' field in the module manifest? If that field does not exist in the module manifest, you will need to add it for the class-based resources.

My mistake, yes, you should be able to have them in the same module. The main issue we've faced with adding class-based resources to our current modules is their dependency on WMF 5.0 and the fact that they require a different testing format than what we currently use.

@mbreakey3 @johlju @randomnote1
@nitinbps just tried out having a module with both a mof-based resource and a class-based resource and it worked. That was with WMF 5.1 and it worked for both Get-DscResource and compiling a configuration.
There are a couple problems that you may be running into:

You have to have the DSCResourcesToExport field in your manifest with the value as a list of the names of all your class-based resources to export.
You must be running at least WMF 5.0. The DSCResourcesToExport field will cause a parsing error on WMF 4 or older. The module manifest should also be updated to require at least WMF 5.0 because of this.
Your mof-based resources must reside in a folder called 'DscResources' within your module. This is how they are discovered. (Your class-based resources can also reside in this folder. This should not cause discovery issues since they do not have mof files.)

@kwirkykat , you made my day!
I was able to get the class-based resource to return using Get-DscResource -xSQLServer, but only if I ran Import-Module xSQLServer first. This is likely because the module has to load the SMO assemblies first.
The other thing I found is that I have to reference the .psm1 file the class is in directly using the RootModule property. This was the missing piece. Did you have similar findings?
Here's the psd1 in question:
https://github.com/randomnote1/xSQLServer/blob/xSQLServerAlwaysOnAvailabilityGroupDatabaseMembership/xSQLServer.psd1

I was able to get Get-DscResource -xSQLServer to display the class-based resource if I remove all explicit references to types defined in the SQL SMO objects. This makes sense technically, but I believe it makes the code a little less clean. More to come.

@randomnote1 Great that you have gotten further with this!
It would have been nice if using namespace could actually load the assembly too, but that .
http://info.sapien.com/index.php/scripting/scripting-classes/import-powershell-classes-from-modules
But could you use using module xSQLServer to load the SQL SMO objects? When the RequiredAssemblies property has the correct assemblies? Just guessing here.

Yeah, it's a shame that using namespace didn't do the trick. It looks like using module also has the same limitation where it doesn't load the assemblies when running Get-DscResource.
I have a workaround in place that will check the object type in the method body rather than at the parameter level. It's not pretty, but it allows everything to work as expected.

@randomnote1 I see in the current proposed resource in PR #593 you have Ensure parameter as three different values; Present, Absent and Exactly. This is entirely different of how Ensure is used all over the DSC Resource Kit. And and in help.txt file you say about the value Exactly of the Ensure parameter "The availability group member database(s) are added and removed until they match the defined databases.".
Ensure parameter should be used, in this case, if a database should be present or absent from the availability group. Ensure is always used over DSC Resource Kit as Present = Create/Add, Absent = Delete/Remove. Adding another value to this parameter would be, I think, confusing.
I would rather see three parameters DatabaseName, DatabaseNameToInclude and DatabaseNameToExclude. Each parameter can take one or more databases (including wildcards). Where DatabaseName cannot be used at the same time as DatabaseNameToInclude and DatabaseNameToExclude.
When DatabaseName is set to one or more database names then the same logic is applied as the current proposed Exactly value of Ensure parameter. That is "only these, an no other databases".
The other to parameters DatabaseNameToInclude and DatabaseNameToExclude will add and/or remove databases from the availability group.
In this case the Ensure parameter might not be needed at all.

Sounds like a plan to me. I'll work on re-factoring that. With any luck it won't require too much in code change :)

Awesome! I continue the review in the meanwhile.


Bad indentation when run-tests-coverage.sh

$ ./.github/run-tests-coverage.sh
* Running in regular mode. Use "-v" or "--verbose" option for verbose output.
* Moving current path to: /workspaces/QiiTask
* Current path is: /workspaces/QiiTask

- Static analysys: Scanning all the packages

  Success! All Go vet static analysis passed.

- Unit test: Testing all the packages
        ok      github.com/Qithub-BOT/QiiTask   0.004s  coverage: 100.0% of statements
        ok      github.com/Qithub-BOT/QiiTask/cmdlist   0.009s  coverage: 100.0% of statements
        ok      github.com/Qithub-BOT/QiiTask/cmdsort   0.004s  coverage: 100.0% of statements
        ok      github.com/Qithub-BOT/QiiTask/config    0.093s  coverage: 100.0% of statements
        --- FAIL: ExampleFmtAsAbout (0.00s)
        got:
        About:
        A longer description that spans multiple
        lines and likely
        contains examples and usage of using your command.
        want:
        About:
        A longer description that spans multiple
        lines and likely
        contains examples and usage of using your command.
        FAIL
        coverage: 100.0% of statements
        FAIL    github.com/Qithub-BOT/QiiTask/utils     0.004s
        FAIL

The above got: contents and want: contents contain spaces before each line. But it seems to be trimmed.

It is missing the IFS= to clear the white spaces list.
 function indentStdIn() {
     indent="\t"
-    while read -r line; do
+    while IFS= read -r line; do
         echo -e "${indent}${line}"
     done
     echo
 }

Wrong indent

https://github.com/KEINOS/Hello-Cobra/blob/85c5321804d80974c59e84b871c8e04e9b8e9111/.github/run-tests-coverage.sh#L23-L30

Right indent

https://github.com/KEINOS/Hello-Cobra/blob/85c5321804d80974c59e84b871c8e04e9b8e9111/.github/run-tests-merge.sh#L26-L32


New version: uazo.cromite version 126.0.6478.127

Created by 🥟 Dumplings in workflow run #2401.
Logs
Updated: 126.0.6478.115 → 126.0.6478.127
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.
Submitting WinGet manifests
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:
(Automated response - build 886.)


adds support for multiple worker classes, activeJob wrapped classes

When I saw that my workers were all showing up in the 'Workers Table' as 'ActiveJob::QueueAdapters::SidekiqAdapter::JobWrapper', I set out to see how Sidekiq supported wrapped job classes. Realizing that they instantiate a Sidekiq::Job using the payload from Redis, that feature was easy to implement.
However, once done, I realized that there's a bug in the code that saves the wrong hash for the job history to redis. Using more than one job class, the worker process would crash when trying to write the 2nd job's status to the 'sidekiq:history:#{date}' hash.
I updated the middleware_test suite to check these new cases, but please give me any feedback if you'd like to see these changes implemented differently.
Cheers!
p.s. thank you for writing the best history plugin for Sidekiq, ever.

Sorry, it's my google summer of code project and therefore I can't merge others changes before the date of delivery of the project :sweat:
But any way, thank you so much for contribution! :green_heart:
@nessur also I want to change hash serialization to native redis hash soon (#27 issue).

Hello, I fixed AJ bugs in #33. Thanks for contribution :tada:


Reference error: BraintreePlugin not defined

In my ionic2 provider file I declared BraintreePlugin but I get a reference error https://github.com/kolawoletech/ionic2-marketcloud/blob/ionic2-marketcloud/src/providers/cart-service.ts
Is it a typings issue?
What is going on here?

On what platform? The plugin doesn't work in browser, only on ios and android platforms. Can you copy the exact error?

Runtime Error
Error in ./OrderPage class OrderPage - caused by: BraintreePlugin is not defined
Stack
ReferenceError: BraintreePlugin is not defined
at CartService.createOrder (http://localhost:8100/build/main.js:57261:21)
at OrderPage.processOrder (http://localhost:8100/build/main.js:57050:40)
at CompiledTemplate.proxyViewClass.View_OrderPage0.handleEvent_74 (/AppModule/OrderPage/component.ngfactory.js:1065:34)
at CompiledTemplate.proxyViewClass. (http://localhost:8100/build/main.js:95230:37)
at HTMLButtonElement. (http://localhost:8100/build/main.js:37046:36)
at t.invokeTask (http://localhost:8100/build/polyfills.js:3:9723)
at Object.onInvokeTask (http://localhost:8100/build/main.js:35117:37)
at t.invokeTask (http://localhost:8100/build/polyfills.js:3:9659)
at e.runTask (http://localhost:8100/build/polyfills.js:3:7083)
at HTMLButtonElement.invoke (http://localhost:8100/build/polyfills.js:3:10836)
Ionic Framework: 2.0.0-rc.5
Ionic Native: 2.2.11
Ionic App Scripts: 1.0.0
Angular Core: 2.2.1
Angular Compiler CLI: 2.2.1
Node: 6.9.4
OS Platform: Windows 10
Navigator Platform: Win32
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36

Android or ios?

I plan on deploying on both, But for now it is an android issues so Android, Yes

I've no idea, you have the declare var BraintreePlugin: any; declaration, that should work.
Please check that you see braintree-plugin.js in Chrome remote debugger (Should be under Sources/plugins/cordova-plugin-braintree)
I suggest you to update Ionic framework to 2.1.0. For me with the followin ionic components it works:
Ionic Framework Version: 2.1.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.3

With the braintree plugin was in the right place and everything seemed to work in the background up to the DropIn UI which never pops, Any ideas?

i have tried many solutions, token initiliase and is set but dropinUI never pops

Can you upload an example project?

I am getting this when running the ionic 2 up in the browser too:
Error: Uncaught (in promise): ReferenceError: BraintreePlugin is not defined
ReferenceError: BraintreePlugin is not defined
Is it typings? How are they added?
I have declare var BraintreePlugin: any;
near the top of my component and then try to initialize with braintree in the constructor

I was also getting the same error for my cordova Android App.
but this issue resolved for me when I tried to initilize my BrainTree Plugin code inside the
$$(document).on('deviceready', function() { // your brainTreePlugin code });


Option to Hide/Disable/Change VS Code logo from Top left Coner for Windows.

in mac system the vscode logo is not there, in windows changing theme the only thing that seemd out of place is the logo, i asked on stack overflow and they told me to go fullscreen.
It would nice addition to be able to change/hide the logo as per will or maybe change it according to themes.

comparison of Mac and windows vscode.


indent-rules: add indentation rules for brackets and if/for/while/swi…

…tch/try/trap
Added a bunch of indentation rules and updated the test file to show case how the new indentation rules work

@dmille56 Done now

Cool thanks!


Robot mBot2

Can the Open Roberta Lab team add the mBot2 robot to the software?
Thanks...

This is already in work, will be pubished asap if we have solved the problem of uploading programs to the robot.

when is it operational?  this spring?
A segunda, 2/05/2022, 11:18, Beate Jost @.***> escreveu:

This is already in work, will be pubished asap if we have solved the
problem of uploading programs to the robot.
—
Reply to this email directly, view it on GitHub
https://github.com/OpenRoberta/openroberta-lab/issues/1289#issuecomment-1114697746,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY54G3WNF2JSCT5NSCZ6SXTVH6TYXANCNFSM5USWUDBA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Dava muito jeito.


Add simple echo example

usage:
sever: echo.exe server -i server.json -s echosvc client: echo.exe client -i client.json -s echosvc
Here's the server/client example in action:

I like it. I do think some high level comments wouldn't hurt. And I'm just a little queasy about the executable name of "echo.exe".

Oh that's actually feedback is didn't even occur to me. Any ideas that wouldn't make you queasy?  I wanted a single executable to keep it easy... I had it called ziti-echo at first but it looked stuttery...

Any ideas that wouldn't make you queasy?

Nothing great comes to mind. simple-echo doesn't feel terrible though.


Spectral initialization for 1D line

Hi Leland, I was playing around with embedding a 1D line and noticed that the spectral initialization does not behave like I think it should.
Here is a reproducible example:
n = 10000
X = np.zeros((n,3))
X[:,0] = np.arange(n)

from sklearn.manifold import SpectralEmbedding
S = SpectralEmbedding(n_components=2, n_neighbors=15).fit_transform(X)
Z = UMAP().fit_transform(X)

plt.figure(figsize=(6,3))
plt.subplot(121)
plt.scatter(Z[:,0], Z[:,1], s=1, c=np.arange(n))
plt.subplot(122)
plt.scatter(S[:,0], S[:,1], s=1, c=np.arange(n))

The spectral embedding looks like a parabola, without any overlaps. But the UMAP result looks like a parabola folded in two. I ran it with small n_epochs and then it's even clearer that it is initialized with the parabola folded in two.
Why would that be?

An error in the UMAP version of spectral embedding perhaps?
On Thu, Feb 13, 2020 at 4:37 AM Dmitry Kobak<EMAIL_ADDRESS>wrote:

Hi Leland, I was playing around with embedding a 1D line and noticed that
the spectral initialization does not behave like I think it should.
Here is a reproducible example:
n = 10000
X = np.zeros((n,3))
X[:,0] = np.arange(n)
from sklearn.manifold import SpectralEmbedding
S = SpectralEmbedding(n_components=2, n_neighbors=15).fit_transform(X)
Z = UMAP().fit_transform(X)
plt.figure(figsize=(6,3))
plt.subplot(121)
plt.scatter(Z[:,0], Z[:,1], s=1, c=np.arange(n))
plt.subplot(122)
plt.scatter(S[:,0], S[:,1], s=1, c=np.arange(n))
[image: index]
https://user-images.githubusercontent.com/8970231/74421054-b5a21200-4e4c-11ea-9383-f57385dc82fc.png
The spectral embedding looks like a parabola, without any overlaps. But
the UMAP result looks like a parabola folded in two. I ran it with small
n_epochs and then it's even clearer that it is initialized with the
parabola folded in two.
Why would that be?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lmcinnes/umap/issues/360?email_source=notifications&email_token=AC3IUBNSMY6O2VNMFNTVGDDRCUIFJA5CNFSM4KUOJWPKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4INGNOJQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC3IUBLDQNBGTDQ53FYW5QDRCUIFJANCNFSM4KUOJWPA
.

That's what I am suspecting. Is this the relevant function? https://github.com/lmcinnes/umap/blob/master/umap/spectral.py#L213

@dkobak, yes that would be the function. For the example data you provide, it's using the scipy.sparse.linalg.eigsh code path.
This is probably an initialization issue (and specifically probably a convergence of the eigenvector calculation): the spectral initialization in uwot, which uses the same graph Laplacian but uses RSpectra to find the eigenvectors, correctly initializes the data to a parabola. Conversely, the Laplacian Eigenmap initialization does a pretty bad job, even though the only difference is the choice of the graph Laplacian.

Hi @jlmelville.

This is probably an initialization issue (and specifically probably a convergence of the eigenvector calculation): the spectral initialization in uwot, which uses the same graph Laplacian but uses RSpectra to find the eigenvectors, correctly initializes the data to a parabola.

Do you think it's possible? scipy.sparse.linalg.eigsh uses ARPACK to compute the eigenvectors and as far as I know Spectra is a C++ reimplementation of ARPACK routines. ARPACK is very well tested, so I'd be surprised if it returns nonconverged results. Also, for this 1D line example, the Laplacian should be so well behaved that I wouldn't expect any tricky convergence problems.
To be honest, I rather suspected some bug/problem in the UMAP code around the eigsh. But I didn't investigate it.

Conversely, the Laplacian Eigenmap initialization does a pretty bad job, even though the only difference is the choice of the graph Laplacian.

Not sure what you mean here. My code snippet above uses https://scikit-learn.org/stable/modules/generated/sklearn.manifold.SpectralEmbedding.html which is Laplacian Eigenmaps, and it yields a parabola shape.

In my Laplacian Eigenmaps example, I had tol = 1e-4 and I get a bad result.

Hmm. Sklearn SpectralEmbedding uses ARPACK eigsh by default, with tol=0.0 (which also is default in eigsh): https://github.com/scikit-learn/scikit-learn/blob/b194674c4/sklearn/manifold/_spectral_embedding.py#L179
Leland's code calls eigsh with tol=1e-4: https://github.com/lmcinnes/umap/blob/master/umap/spectral.py#L267
One could change the params of the eigsh call in spectral.py and see if it makes the difference in my toy example.

Based on my experiences with RSpectra in uwot, I am sure that decreasing the tol parameter would fix the issue.

@jlmelville Interestingly, adding a small amount of Gaussian noise to the simulated data makes the problem go away. I wonder if it somehow can make eigsh converge faster.

Based on my experiences with RSpectra in uwot, I am sure that decreasing the tol parameter would fix the issue.

You are right. Pavlin and me now found out the same in the linked thread at openTSNE. Decreasing the tolerance to zero does indeed fix this issue can make the runtime a lot slower. So it's not clear that it would be advantageous by default. I guess I will close this issue then.


Add option to email summary stats when finished

This resolves #10.

Coverage increased (+2.6%) to 38.55% when pulling 6908af50506055f4ada0462e665ae5de2ba45203 on improvement/add_option_to_email_summary_stats into 15095fc03413302583e5584bb0bdbb228f48a5fb on develop.


allow 'statx' in the sandbox

Linux 4.11 added statx, which it seems libc now uses in preference to some of
the older stat calls.
Reported-by: Adrian Bunk<EMAIL_ADDRESS>Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1004035

Reproduced the issue on a Debian unstable 32-bit x86 VM and confirmed this PR solves it.


ADD: List of Celestial Bodies

Purpose: Display a list of celestial bodies with details like name, distance from Earth, and mining potential.
Features:
Sort by distance, mineral composition, or economic potential.
Clicking an item brings up detailed information
@Devanik21 please assign me this task

@shubhagarwal1 , how will u implement it?


meta.yaml updates

add pc494 as maint
correct dependencies for versions
bump build number

Checklist

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

@conda-forge-admin, please rerender


Some users with large review caches report input lag during reviews

Some users are reporting that since the updates WaniKani made a month ago they have seen a significant increase in input lag that they have pinpointed to the Heatmap script (which uses Review Cache and thus has components that run during reviews). The following comment in the Heatmap thread uses a profiler to find where the bottleneck is:
https://community.wanikani.com/t/userscript-wanikani-heatmap/34947/2209

PR here https://github.com/Kumirei/Userscripts/pull/23


Workflow/tool/service file should be in the browser url

Feature Request
When viewing files in the Dockstore UI, the current file should be in the url. The tab is, but the file isn't.
I wanted to have a slide linking to the primary descriptor for this service, but I can only link as deep as the tab, which just always defaults to the first source file.
Ideally the url should be something like, in this case, https://dockstore.org/services/github.com/coverbeck/xenahub:v0.5?tab=files&type=primaryDescriptor.
If the type were source, then a name query parameter would also be required.
BTW, why does Primary Descriptor page for services have a dropdown? Can there be more than one? If not, that shouldn't be a dropdown.
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Issue Number: DOCK-1103

BTW, why does Primary Descriptor page for services have a dropdown? Can there be more than one? If not, that shouldn't be a dropdown.

Might be a hold-over from multiple languages (i.e. multiple primaries when there is more than one language), but not relevant for services


Slice bounds out of range from decode function and Header is invalid error

Hi, we are trying to make multiple select queries in parallel to retrieve data from kdb.
query:-  select open:first price, high:max price, low:min price, close:last price, volume:sum volume by sym from trade where time within time1 time 2
we are encountering an error : "Slice bounds out of range" from Decode function.
panic: slice bounds out of range
goroutine 106 [running]:
bufio.(*Reader).Read(0xc000049380, 0xc00112d328, 0x8, 0x8, 0xc00f669c08, 0x441cd3, 0x8)
C:/Go/src/bufio/bufio.go:234 +0x3ce
io.ReadAtLeast(0xe339e0, 0xc000049380, 0xc00112d328, 0x8, 0x8, 0x8, 0x5, 0xc000dc7560, 0x0)
C:/Go/src/io/io.go:310 +0x8f
io.ReadFull(...)
C:/Go/src/io/io.go:329
encoding/binary.Read(0xe339e0, 0xc000049380, 0xe4a140, 0x1487ec0, 0xbd22c0, 0xc00112d2f8, 0x0, 0xc00f669e28)
github.com/sv/kdbgo.Decode(0xc000049380, 0xc00011c270, 0xc00011c201, 0xc000621120, 0x0)
This error occurs only when there is huge data for processing.
We don't see this error when there is only one select query.
Could some one please look into this ? and recommend a fix.
Environment
kdb windows 32bit
go version go1.12.7 windows/amd64
OS: windows

i am closing this issue, since it is a duplicate of https://github.com/sv/kdbgo/issues/5 . I have created individual connection for each go routine and its fine now.


Revert "refactoring jp.co.yahoo.yosegi.message.design"

Reverts yahoojapan/yosegi#16
remove refactoring

LGTM


After I stopped and started machine, new IP did not show up in rancher

Rancher Version: master 11/10
Steps to Reproduce:

Stop the machine on host side (went to google console and stopped)
Wait for host to disconnect on Rancher side
Start machine (notice it gets a new IP)
Wait for host to reconnect

Results: Host still has old IP showing on rancher UI
Expected: Should show new IP

Tested with rancher-server version - v1.2.0-pre4-rc14
Added "AWS host" as docker-machine host.
Deployed few containers.
Power off the host from AWS console.
Wait for the host to go to "Disconnected" state.
Power on the host.
Once the host gets to "Active" state , we still see the same Ip address for the host in the UI.
Cross host networking is broken for the container in this service and the health-check container is stuck in "Initializing" state.


cemetech-expert pips are broken

Actual:

Expected:
Show cemetech expert image where the pips go

Probably won't fix this, löng bar is probably fine judging by user testing (me). If other people complain, I'll work on it.


Documenting and testing wavefd-simulation-class #137

I don't want to be hard with python but it seams its documentation features for abstract/interfaces/derived class or methods lack functionality.  For documenting methods that are going to be used by derived classes I am now copying on all classes with equal methods. Python doesn't check if your method is not complying with the mother method it s overriding.
I wrote few test cases after watching the software carpentry lessons.
I will push more stuff as they come. It takes time a long time to understand someone-else's code without or with very few comments!!! @leouieda luckily I am learning some things.
Copying from yours 137
I need to reference this most recent ipython notebook usage procedures and the old one

[x] Make tests for new code  (I've done some ... working one more and maybe some doctests)
[x] Create/update doc-strings (working on ... very very tough not being my code! )
[x] Update requirements (I think it's okay now)
[x] Code follows PEP8 style conventions (working on ..)
[x] Code and docs have been spell-checked (for the end...)
[x] Changelog entry
[x] Documentation builds properly (builds properly but need more...)
[x] All tests pass (so far passed ... working on more!)
[ ] Can be merged

Progress updates:
Updated (20/01/2015):
Porting my scalar code and cookbook.
Soon I will try to refactor all old cookbook code to get rid of everything old.
Need to do:

[x] Re-factor scalar + dumping region
[x] Re-factor scalar cookbook (do not re-factor the others) implicit optimizing plotting and animation code.
[x] add padding support again needed for dumping region (I'll leave it for future for sh and psv)
[x] Fix doc strings (don't forget development guide)
[ ] Tests coverage acceptance
[ ] Update installation requirements : h5py; all IPython notebook dependencies (pip recommended for latest);  animate requires libavcodec-extra-54 and libav-tools (In Ubuntu 14.04 LTS)

About documenting classes:

Base class abstract methods should not have docstrings.

@leouieda after a lot of work to understand your code I still could not understand everything.  hahaha still missing documentation.
Ive written some more tests.
My old Scalar class is totally documented.
Sources are fully documented.
Well I have I fixed at least (I guess) most of the docstrings.
But I haven't finished reviewing the documentation for the other classes.
I haven't fixed the cookbooks or get rid of all old code.
Your help needed
What I need from you is to take I look on the way I am going with the docs. I haven't finished but...
I suggest that the common methods of the simulation classes should be on wavefd docstring in two topics: Simulation Storage Access and Simulation (rich) display.
Doing so we would not document the methods on those topics on the body of the classes, leaving the classes cleaner. My worry is that the module wavefd is getting really huge.

@eusoubrasileiro thank you so much for helping with this! The semester is finally over at the university so I might have a chance to look at this next week.

although I've installed numba when I run make test I am getting the following error:
======================================================================
ERROR: gravmag.tesseroid numba and pure python implementations give same result
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/andre/PycharmProjects/fatiando/test/test_gravmag_tesseroid.py", line 89, in test_numba_vs_python
    nb = func(lon, lat, height, model, engine='numba')
  File "/home/andre/PycharmProjects/fatiando/fatiando/gravmag/tesseroid.py", line 383, in potential
    ratio=ratio, engine=engine, njobs=njobs, pool=pool)
  File "/home/andre/PycharmProjects/fatiando/fatiando/gravmag/tesseroid.py", line 277, in _dispatcher
    field])
  File "/home/andre/PycharmProjects/fatiando/fatiando/gravmag/tesseroid.py", line 301, in _forward_model
    module = _get_engine(engine)
  File "/home/andre/PycharmProjects/fatiando/fatiando/gravmag/tesseroid.py", line 224, in _get_engine
    module = _tesseroid_numba
NameError: global name '_tesseroid_numba' is not defined

----------------------------------------------------------------------
Ran 174 tests in 3504.121s

FAILED (errors=1)
make: *** [test] Error 1

am I missing something...?

Are you sure that you can import numba?
On Apr 26, 2015 10:10 PM, "André"<EMAIL_ADDRESS>wrote:

although I've installed numba when I run make test I am getting the
following error:
======================================================================
ERROR: gravmag.tesseroid numba and pure python implementations give same result
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
self.test(*self.arg)
File "/home/andre/PycharmProjects/fatiando/test/test_gravmag_tesseroid.py", line 89, in test_numba_vs_python
nb = func(lon, lat, height, model, engine='numba')
File "/home/andre/PycharmProjects/fatiando/fatiando/gravmag/tesseroid.py", line 383, in potential
ratio=ratio, engine=engine, njobs=njobs, pool=pool)
File "/home/andre/PycharmProjects/fatiando/fatiando/gravmag/tesseroid.py", line 277, in _dispatcher
field])
File "/home/andre/PycharmProjects/fatiando/fatiando/gravmag/tesseroid.py", line 301, in _forward_model
module = _get_engine(engine)
File "/home/andre/PycharmProjects/fatiando/fatiando/gravmag/tesseroid.py", line 224, in _get_engine
module = _tesseroid_numba
NameError: global name '_tesseroid_numba' is not defined

Ran 174 tests in 3504.121s
FAILED (errors=1)
make: *** [test] Error 1
am I missing something...?
—
Reply to this email directly or view it on GitHub
https://github.com/fatiando/fatiando/pull/153#issuecomment-96455435.

no I am not sure! ... I am fighting to install numba on ubuntu...

@leouieda  now numba working. Though I don't know what has broke AppVeyor.
About the docs, although I like the new style I miss something.
Since, for example, wavefd.py has too many classes you get lost when you come to the documentation and search for a specific class. I took a look in the Sphinx documentation but I will try more, something like a summary would be very handy.
Despite of that IPython widgets is something really nice but according to doctest it is subject to changes in future, just to let you know.
When you have a time take a look on what I wrote above, I still need help on documenting it. Above all else this simulation classes are really amazing in this new format!

Wow, why are there so many different commits in this branch? Things probably got a bit messed up with all the recent PRs. I'm running a merge on your branch here to try to make things a bit cleaner and resolve all the merge conflicts (there are many).
I've made a PR from the branch I've used to resolve the conflicts on to yours. You can merge it to make things up-to-date with master.

OK, I don't like how we are doing this with so many different branches. This is creating a lot of headaches!
We should probably just both commit to the same branch. What do you think? Would you be comfortable pushing things to the main repo? Just need to be extra careful not to mess up master. We would also need to coordinate our development so that we don't break each others code (we'd need to talk more, which is not a bad things :facepunch:).

OK, once you merge the PR with the conflict resolutions the diffs should be more sane. I've had a look at the code and I think we can merge this and do some more later. There is still a lot of repeated code and I agree that the module is way too big. We can safely delete the old code because no one would use it once the classes are done, so that removes some of the volume.

Well, that was quick. Don't you have a job?

We should probably focus first in simplifying the code. We can make the proper documentation after the code is settled then. What do you think?

haah I do have I job but I just entered on github to see what was going on. Just coincidence heh lucklily

calling you


segfault with ubuntu 12.04

Hi guys,
I get a segfault after 2 minutes I run any command at the shell. Even a 'ls' causes that.
Feb  4 10:20:35 venus1553624 kernel: [  268.932535] zsh[18525]: segfault at 0 ip b759d5e2 sp bfe9f490 error 6 in libc-2.15.so[b750f000+1a4000]
Any hint?

What version of zsh are you running? What's your .zshrc, .profile files contain. Does it happen without oh-my-zsh enabled (using a blank zshrc file)?

It may also be a bug in the glibc library provided by Ubuntu, see if you have any pending updates to install


the problem of long sequences

Bug description
When I use esmfold, adding the "-- cpu offload" parameter does not solve the problem of long sequences. My GPU is A100 32GB. Please help me. The error is reported as follows:
22/11/10 13:56:33 | INFO | root | Reading sequences from hipAB.fasta
22/11/10 13:56:33 | INFO | root | Loaded 5 sequences from hipAB.fasta
22/11/10 13:56:33 | INFO | root | Loading model
22/11/10 13:57:23 | INFO | torch.distributed.nn.jit.instantiator | Created a temporary directory at /tmp/tmp4ekwnkqe
22/11/10 13:57:23 | INFO | torch.distributed.nn.jit.instantiator | Writing /tmp/tmp4ekwnkqe/_remote_module_non_scriptable.py
22/11/10 13:57:23 | INFO | torch.distributed.distributed_c10d | Added key: store_based_barrier_key:1 to store for rank: 0
22/11/10 13:57:23 | INFO | torch.distributed.distributed_c10d | Rank 0: Completed store-based barrier for key:store_based_barrier_key:1 with 1 nodes.
22/11/10 13:57:26 | INFO | root | Starting Predictions
22/11/10 13:59:12 | INFO | root | Predicted structure for hipAB-750 with length 765, pLDDT 51.2, pTM 0.264 in 105.6s. 1 / 5 completed.
22/11/10 13:59:15 | INFO | root | Failed (CUDA out of memory) on sequence hipAB-900 of length 918.
22/11/10 13:59:17 | INFO | root | Failed (CUDA out of memory) on sequence hipAB-1050 of length 1071.
22/11/10 13:59:19 | INFO | root | Failed (CUDA out of memory) on sequence hipAB-1200 of length 1224.
22/11/10 13:59:20 | INFO | root | Failed (CUDA out of memory) on sequence hipAB-1350 of length 1377.
/home/houj21/miniconda3/envs/esm/lib/python3.7/site-packages/torch/distributed/fsdp/fully_sharded_data_parallel.py:930: UserWarning: Module is put on CPU and will thus have flattening and sharding run on CPU, which is less efficient than on GPU. We recommend passing in device_id argument which will enable FSDP to put module on GPU device, module must also be on GPU device to work with sync_module_states=True flag which requires GPU communication.
"Module is put on CPU and will thus have flattening and sharding"

Try something like --chunk-size 16?


update rustler to support OTP 24

There are some compilation warning because the authors of rustler deprecated macros. I think we can update the rust code after they release the final version of these macros in the 0.22.
Right now this library fails to compile on OTP 24

@InoMurko hey. pr wasn't ready to merge. Rustler team released a stable version. So I'll try to change nif because they changed almost every macro.
I should have made it draft


Rework setSIMDExtension to compile with partial SIMD enabled

There are existing options to enable both Arm native and SIMDe-based intrinsics implementations, however if only one of these is enabled then compilation for Arm targets currently fails.
Fix this by adjusting the existing #ifdef guards and adding new ones to cover the previously failing option combinations. Also stop checking REAL_TARGET_ARM since it does not distinguish between which SIMD targets are enabled.

Thank you for attempting to clean up the compilation mess.
I just looked into it and realized that the ARM-SIMD code cannot work without having TARGET_SIMD_X86 enabled, because it explicitly calls some X86-SIMD implementations (xCalcHAD*_SSE for non-square block sizes). This is something that came in after the first NEON implementations, and I did not know this when I said it was supposed to work in my comment on #453. I checked back with colleagues and we can define that TARGET_SIMD_X86 is a requirement for TARGET_SIMD_ARM.
So, even with your changes applied, the code does not compile when TARGET_SIMD_ARM is enabled, but TARGET_SIMD_X86 is not. Maybe we should explicitly catch that in the configure step.

Thank you for attempting to clean up the compilation mess.
I just looked into it and realized that the ARM-SIMD code cannot work without having TARGET_SIMD_X86 enabled, because it explicitly calls some X86-SIMD implementations (xCalcHAD*_SSE for non-square block sizes). This is something that came in after the first NEON implementations, and I did not know this when I said it was supposed to work in my comment on #453. I checked back with colleagues and we can define that TARGET_SIMD_X86 is a requirement for TARGET_SIMD_ARM.
So, even with your changes applied, the code does not compile when TARGET_SIMD_ARM is enabled, but TARGET_SIMD_X86 is not. Maybe we should explicitly catch that in the configure step.

Thanks for pointing out the problems with the xCalcHAD functions, I now realise that CMake cache variables meant that my existing build directories wasn't actually picking up my attempts to recompile with the SIMDe kernels disabled.
I think it is still possible to avoid the dependency between the two options: we can simply disable the RdCost optimisations if TARGET_SIMD_X86 is not enabled. This is similar to how we already require AArch64 for these kernels to be used. I've put up a patch to that effect (and swapped the existing __ARM_ARCH >= 8 check for REAL_TARGET_AARCH64 now that we have it). Let me know what you think about this approach!
I also have a patch to remove the AArch64-only requirement for the RdCost kernels such that it also compiles on 32-bit Arm platforms, but I will put that up as a separate PR once this is merged so it doesn't conflict.


New feature to handle WAR files as input

Known limitations of processing WAR files

The WAR file is unzipped into temp folder directory, this takes some time on bigger files
There is no DeleteOnExit of that temp folder directory. Do this manually by now!
All embedded JAR files needs to be unzipped and a final ZIP file is create, this takes some times due to file system I/O

Thank you @psytester, I know it has been a while since you submitted this.


Deprecate module and repository

With Warehouse Next coming up soon, I think would probably make sense to deprecate this module.

We should hold on this until we finish in the main repo.

We should hold on this until we finish in the main repo.


Files still open in tabs even when tabs are disabled

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I just recently started using Atom, and immediately noticed the really long startup times. So, I started disabling things I didn't need, and slowly re-enabling things that I later saw as useful.
One of the first things disabled were tabs. I want to just use it as a text editor and don't really care about workspaces, or tabs. I'd already hacked sublime to have each file open in a new window and I'd intended to get the same behaviour now.
Steps to Reproduce

Start with a fresh installation
Disable the tabs package
Open some files
Re-enable the tabs package

Expected behaviour: Whilst the tabs package is disabled each file should open in a separate window.
Actual behaviour: The contents of the existing window (appear to) get replaced with the last one opened. Turning on the tabs package, though, reveals that every file has secretly been opened in that same window.
Reproduces how often: 100%
Versions
Atom version is 1.26.1 x64
(can't run atom or apm in the command line, sorry)
OS is Windows 10 x64 build 1803.
Additional Information
There are also other problems, unrelated, but I think the installer is borked. I had to dig the atom.exe out of my appdata folder (why is it there of all place!?) to create a desktop link.

Tabs are simply a visualization of having multiple files open in the same window pane (called "pane items" in the Atom API). Without the tabs visualization, it is still possible to open multiple files in the same pane and switch between them using the standard keys. This is by design.

@lee-dohm Well that's silly. How are you supposed to know they're open if you can't see them?
I still maintain that it shouldn't keep opening files in the same window, or at least give us some option to have them open in a new instance of Atom.

The fuzzy-finder package allows you to list all open files and switch between them using Cmd+B on macOS and Ctrl+B on other platforms. We have no plans to add the feature you're requesting, so this is going to remain closed.

@lee-dohm Ah, fair enough. The description for fuzzy-finder was just "Open an editor to a file in the project with cmd-t." do I disabled it, along with about 40 other packages.
I will either get used to it having tabs, or reinstall sublime.


Any chance...

...this could be done as a standard solution, with program.cs, startup.cs etc. so can be run directly from Visual Studio?
Many thanks.

Hi ASKemp,
Please look at the following link for demo solution. Demo Solution
Thanks,
Abdul


Octicons: need a "Info" icon that has full background

Just tracking as bug here that the adoption of the tasks status item needs the previous info icon we had that has full white background.
@misolori that actually makes me wonder if we should revisit some of the octicons that are partially transparent? E.g. in general I think they do not look very good in our status bar, for example:

Instead of adding new icons we could simply fix those that are not fully with a background.

We could do that, however one thing to consider is other extensions (like LiveShare) use certain icons in their custom viewlet (like the sharing & people icons). We’d just need to be sure to communicate that with them as they use those same icons in VS (I believe).
At the minimum we could replace the info and smile icons. Planning on pushing those changes tomorrow.

Yeah good point, these icons are used in different contexts (even quick pick). Alternatively, we could:

identify the icons that do not fully have a background
introduce variants that have a full background
rewrite any octicon in the statusbar to use that variant to enforce a consistent look down there
do not change the look in other places

I guess this depends on how much such a change would break existing UX. We should maybe do an experiment how a solid-background icon looks like in quick pick and custom views compared to the current ones and then decide.

I just noticed in Insiders this info icon looks weird. I don't know if it falls under this issue (which talks about the BG), but it seems to be too close to the number to its left, and also seems larger than the others.

@DanTup yeah thanks for posting the screenshot, this issue is about fixing exactly that. The margins should be equal to make this look good.


Roomsettings: Make blacklist cmd perms toggleable

I don't like this. I have considered it but decided a 1 year ip ban is too
much power for roommods

This should also never be a large concern.  If you need to use blacklists
so much the ROs can't handle it, you have different problems as a room.

I also don't like the idea of roommods being able to mass ban a room.

If you need to use blacklists so much the ROs can't handle it, you have different problems as a room.

Is it a matter of "this room doesn't have active ROs," or a matter of "this room has active ROs and trusts their roommods with blacklists"?
I think it's the later.

I also don't like the idea of roommods being able to mass ban a room.

... they can already do that with /roomban

too much power for roommods

Uh, well, before this was a thing server-side, how it worked was ROs would have roommod bots, and they would usually give roommods+ access to that bot's blacklist command.  I don't see how allowing them to use the server-side blacklist command is much different than a bot blacklist command.
I understand that it is IP based, and I also understand what uncommon offenses should be blacklisted for.  If a staff or roommods also understands this, and the RO recognizes that fact, then I see no reason for that room to not allow roommods to use this command.
If someone is unfairly blacklisted or a roommod fucks up, then that room will handle that specific instance accordingly, as with all other forms of room punishment.
maybe there's just something I'm not getting here, or maybe I don't really see what's wrong here, but in terms of actual power, I don't see this as a huge policy change.
As a matter a fact, some rooms specifically aren't using the server-side blacklist system BECAUSE it doesn't allow roommods to use it.
I think our mission should be to eliminate bots from being needed for all forms of moderating, and this is a good final step towards that goal.

I think it's the later.

No it's the former. Most rooms have (and should) have <25 people on the blacklist. An active RO should be able to deal with that by themselves even.

... they can already do that with /roomban, in theory

In practice this makes it a lot more convenient, and harder to stop midway.

Uh, well, before this was a thing server-side, how it worked was ROs would have roommod bots, and they would usually give roommods+ access to that bot's blacklist command. I don't see how allowing them to use the server-side blacklist command is much different than a bot blacklist command.

The bot blacklists ONLY worked on names and could not even check the IP of users. It also didn't work ALL the time, only when the user attempts to join the room. The serverside blacklists are actually the longest punishment based on IP ON THE SIM. Granted, they're from a single room, but I would very much prefer to only have this available to people who have directly received the necessary trust from staff.

I understand that it is IP based, and I also understand what rarer offenses should be blacklisted for. If a staff or roommods also understands this, and the RO recognizes that fact, then I see no reason for that room to not allow roommods to use this command.

Again, if they understand how uncommon it should be, there should be even less reason to allow more people to use it.

As a matter a fact, some rooms specifically aren't using the server-side blacklist system BECAUSE it doesn't allow roommods to use it, and are still using bot-based blacklist systems.

Then they can do it. This feature is only there to provide an option. Rooms can choose themselves whether they use the options given to them. The key is to provide the right options to the right people, and I don't think that is the case here.

I think our mission should be to eliminate bots from having a purpose of moderating of all forms for rooms, and this is a good final step towards that goal imo.

I disagree that bots should not be allowed to moderate at all. There are things a bot provides more flexibility for and allows to fit a room's specific needs more, as they are sometimes better done externally.

(Also probably worthy to note: After discussing this with several users in higher global/room policy positions on PS, there overall seems to be a want for this change)

I've talked with multiple ROs that are strongly against this change. I also would like to note that some of the decisions are also made from a technical perspective, and trying to meet as many demands as possible while still trying to leave little margin for errors.
I hope this clears up my perspective more.

In practice this makes it a lot more convenient, and harder to stop midway.

The bot blacklists ONLY worked on names and could not even check the IP of users.

Well, roombans are IP based... so, not really.

Again, if they understand how uncommon it should be, there should be even less reason to allow more people to use it.

Some larger rooms use blacklists pretty regularly, namely Lobby and Wi-Fi, afaik
(and there's probably more this would be applicable to, too)

Then they (ROs) can do it.

Some rooms don't have extremely active ROs in terms of actual room visits/day.

This feature is only there to provide an option.

So would this roomsettings option... ?_?

I disagree that bots should not be allowed to moderate at all.

Never did I say that they shouldn't be allowed to moderate, but that they shouldn't have to.

I've talked with multiple ROs that are strongly against this change.

Then those ROs can keep this setting turned off, and nothing will change for that room's policy.  But with this logic, don't let one room's policy interfere with that of another's... :|

I mean, if you have a roommod mass banning users in a room, then you have more pressing matters to be concerned about...

This is still a concerning matter considering various mods have gone renegade before. ROs on the other hand undergo an intensive vetting process that is (and should be) more intensive than for mods.

Well, roombans are IP based... so, not really. The only actual difference is the duration.

Yes but someone has to join the room on the blacklisted name unlike the serverside blacklists.

Some larger rooms use blacklists pretty regularly, namely Lobby and Wi-Fi, afaik
(and there's probably more this would be applicable to, too)

Zarel and I agreed that the Wi-Fi room was using the blacklists wrong, and should even keep a bot for blacklists for scamming (this is actually the case as of today)
[19:51:24] Zarel: again, tell them to keep the bot for trade blacklists (relevant quote)

Some rooms don't have extremely active ROs in terms of actual room visits/day.

In the meantime, a user could be roombanned, which could be extended to a blacklist when a RO does come online.

Never did I say that they shouldn't be allowed to moderate, but that they shouldn't have to.

And they don't have to.

Then those ROs can keep this setting turned off, and nothing will change for that room's policy. But with this logic, don't let one room's policy interfere with that of another's... :|

It changes how blacklists work sitewide and established permissions. By implementing this feature you are effectively allowing ROs to pass on RO features to mods bypassing the public RO approval thread.

This is still a concerning matter considering various mods have gone renegade before. ROs on the other hand undergo an intensive vetting process that is (and should be) more intensive than for mods.

Just as easily as a rouge roommod could go by roombanning everyone, they could also blacklist, and both log, so both are just as easy to reverse by that logic.

Yes but someone has to join the room on the blacklisted name unlike the serverside blacklists.

I mean, once they get roombanned (by a bot blacklist or in general), they get IP banned (from that room).

Zarel and I agreed that the Wi-Fi room was using the blacklists wrong, and should even keep a bot for blacklists for scamming (this is actually the case as of today)

Wi-Fi isn't the only room that has a large blacklist.  Is that to say all rooms that have a large blacklist use it wrong? I think that might be a stretch, if so...

In the meantime, a user could be roombanned, which could be extended to a blacklist when a RO does come online.

My point still remains:

I would argue that because it's harder to find an active RO on at any given time, that someone who might of deserved a blacklist might only get away with a roomban in it's place because of the hassle of remembering to tell a RO to blacklist them.

It changes how blacklists work sitewide and established permissions. By implementing this feature you are effectively allowing ROs to pass on RO features to mods bypassing the public RO approval thread.

It doesn't change "established permissions," as this won't change anything by default.
What determines that this is a RO feature, but yet a bot blacklist isn't?  That sounds like room-policy territory to me.

Just as easily as a rouge roommod could go by roombanning everyone, they could also blacklist, and both log, so both are just as easy to reverse by that logic.

I think this entire line of conversation is getting incredibly petty and blown out of proportion.

I mean, once they get roombanned (by a bot blacklist or in general), they get IP banned (from that room).

This still needs to happen. If the troll is using a different username on the same IP (which isn't hard to do), he can enter the room with no problems.

Wi-Fi isn't the only room that has a large blacklist. Is that to say all rooms that have a large blacklist use it wrong? I think that might be a stretch, if so...

You were listing specific examples so I decided to elaborate on one of them given I'm roomstaff in the room in question.

My point still remains:

No it doesn't. The roomban can be extended into a blacklist with ease, and nowhere did I suggest people could "get away with a roomban". If roommods aren't competent enough to remember to tell a RO to blacklist, should we be trusting them with the blacklist permission itself?

It doesn't change "established permissions," as this won't change anything by default.

There is a permission established since the PR for roomblacklists. There were ample opportunities to voice your concerns then (the PR was open for literally months), but you didn't do that before a command (with a permission) was already implemented on PS.

What determines that this is a RO feature, but yet a bot blacklist isn't? That sounds like room-policy territory to me.

As I've said before, the severity of the punishment and the necessary trust in a RO.

I think this entire line of conversation is getting incredibly petty and blown out of proportion.

??
Perhaps I misunderstood what you meant then, could you elaborate?

This still needs to happen. If the troll is using a different username on the same IP (which isn't hard to do), he can enter the room with no problems.

Blacklists aren't particularly much harder to evade, though.  I also fail to see how this ties in at all.

You were listing specific examples so I decided to elaborate on one of them given I'm roomstaff in the room in question.

But my point is is that those aren't the only rooms that this might be helpful for, that those were just some of them... :|

No it doesn't. The roomban can be extended into a blacklist with ease, and nowhere did I suggest people could "get away with a roomban". If roommods aren't competent enough to remember to tell a RO to blacklist, should we be trusting them with the blacklist permission itself?

Yes, they can be extended "with ease."  Does this mean that every single mod remembers to pass on names who should be on the blacklist in every single instance? No.  Does this make them a bad mod? I personally don't think so, but you're entitled to your own opinion here.

There is a permission established since the PR for roomblacklists. There were ample opportunities to voice your concerns then (the PR was open for literally months), but you didn't do that before a command (with a permission) was already implemented on PS.

As I said on that PR, that entire PR was merged pre-maturely, and I could say that the "lack of expression on that PR" for this was a consequence of that.
Refs:

For what it's worth, I think big refactors like this should take a considerable amount of time. I feel as though this instance was rushed in the way that there were still a lot of things not really addressed and it all kind of had a "thrown together" vibe to it when all was said and done.

Blacklists aren't particularly much harder to evade, though. I also fail to see how this ties in at all.

You need to change your IP, which a lot of (mostly younger) users cannot do.

But my point is is that those aren't the only rooms that this might be helpful for, that those were just some of them... :|

Where do I imply otherwise, I just responded to your example with an example.

Yes, they can be extended "with ease." Does this mean that every single mod remembers to pass on names who should be on the blacklist in every single instance? No. Does this make them a bad mod? I personally don't think so, but you're entitled to your own opinion here.

If someone should be blacklisted, there is usually a policy discussion within the room, since blacklists are not a punishment like a lock or ban, but intended for long-term problem users in the room who should not be allowed in the room at all (definition of a blacklist in this context). A RO should always be involved with this discussion, and it seems only logical to me that they are the one making the decisions.

As I said on that PR, that entire PR was merged pre-maturely, and I could say that the "lack of expression on that PR" for this was a consequence of that.

There were a whole lot of replies on that PR, and not a single one about the permission. The time it took for the PR is definitely enough to bring up these concerns in my opinion.
Also, can you provide a source and context for that quote?

You need to change your IP, which a lot of (mostly younger) users cannot do.

Roombans were IP based, they were before this refactor and still are... (still failing to see relevancy with this though)

If someone should be blacklisted, there is usually a policy discussion within the room, since blacklists are not a punishment like a lock or ban, but intended for long-term problem users in the room who should not be allowed in the room at all (definition of a blacklist in this context). A RO should always be involved with this discussion, and it seems only logical to me that they are the one making the decisions.

Yes, and different rooms obviously feel differently about this.

There were a whole lot of replies on that PR, and not a single one about the permission. The time it took for the PR is definitely enough to bring up these concerns in my opinion.

I would like to agree with this, but there were so many things that weren't addressed that before I looked at it closer on a more serious note, I was waiting for it to get into, what I considered, its final stadges... of which, never happened.

Also, can you provide a source and context for that quote?

My latest comments on: https://github.com/Zarel/Pokemon-Showdown/pull/2545

Roombans were IP based, they were before this refactor and still are... (still failing to see relevancy with this though)

My point is that evading blacklists wasn't IP based. I'm tired of repeating myself.

Yes, and different rooms obviously feel differently about this.

I'm not saying what my opinion was, but rather the idea of the entire feature to begin with.

Strong no. Mods can roomban until ROs come around.

I ran into a similar situation to this while hosting BoTTT, so I guess I should mention it.  At one point I was forced to add a regex variant of autobans to it because people were evading autobans on server resets and taking advantage of the easy name recognition by keeping a pattern of names. I couldn't trust room mods with the command like I could with regular autobans though because like with server blacklists, few room mods knew what they were doing to avoid autobanning far more users than intented. Likewise, few room mods in the majority of rooms now are capable of seeing user IPs to avoid blacklisting an entire school because of one user, for example. I can't trust room mods with blacklists for the same reason I only trusted a few people with regex autobans; they have no way to be able to see just how many different users they're blacklisting from a room.

they aren't guaranteed to be able to gauge what effect blacklisting someone would have.

Well, not all ROs are global mod+ that they can see IPs to gauge that, either.

Well, not all ROs are global mod+ that they can see IPs to gauge that, either.

I don't see how this helpa either of our arguments.

So, I take it I should close this since among the people who have commented, the general consensus is that this shouldn't be a thing.
If I am wrong in this assumption, then someone can feel free to reopen this.


Incorrect Username and Password Error

Running into an issue when I provide Username and password from commandline. It uses local auth even though the creds are provided using commandline.
PsMapExec -Method SMB -Username <user> -Password <pass> -Domain avengershq.local -Targets All
Since, it is taking local creds that's the reason I am getting invalid auth error. Please have a look at traces screenshot. Also, my computer is not part of the domain.

@raghavmishra please try the latest commint in the main branch. This should now (hopefully) address the issue.

Sure, I will try it.

It is working. Thank you.


Adopt pytest-order into pytest-dev

That is a tracking issue around for https://github.com/mrbean-bremen/pytest-order/issues/4
@pytest-dev/core please comment on this.

My answer in https://github.com/ftobia/pytest-ordering/issues/32#issuecomment-715257657 still stands.
cc @ftobia

Sure. I only wanted to have a centralized place to track adoptions.

@nicoddemus - I would like to go ahead with the transfer, if that is ok. I haven't got a response from the maintainer of the original package so far. What do you think?

Sure, let's go ahead then, we have left enough time for the original maintainer to reply. 👍

Thanks! I just subscribed to pytest-dev and wrote a respctive mail, is there anything else to do for me at the moment?

Let's follow proper protocol and wait for others to 👍, should take a day max.

Looks good, please go ahead!

@nicoddemus - thanks, I will just wait for a go or no-go then.

Please go ahead! The "proper" protocol is for people to respond to the mailing list, but of course responding here in the issue also works. 👍

Thanks 😃. So... if I understand this correctly, I shall transfer the ownership to a pytest-dev admin, e.g. to you. Is this correct, or is there something else to do?

That's it! I will then accept the transfer, and add you as maintainer under the new location. 👍

Transfer complete! Any lingering issue we can open new issues in the new repository location itself. 👍
Thanks everyone!


Issues with the Bunny DNS API

@dennismuench
Coming from here:
https://github.com/caddy-dns/bunny/issues/2
I've sorted the empty DNS zone and now I can better see where the issue is coming
{
   "level":"error",
   "ts":1734093679.7992978,
   "logger":"tls.obtain",
   "msg":"could not get certificate from issuer",
   "identifier":"*.mysite.com",
   "issuer":"acme-v02.api.letsencrypt.org-directory",
   "error":"[*.mysite.com] solving challenges: presenting for challenge: adding temporary record for zone \"_acme-challenge.mysite.com.\": zone not found: _acme-challenge.mysite.com (order=https://acme-staging-v02.api.letsencrypt.org/acme/order/158696983/21239318474) (ca=https://acme-staging-v02.api.letsencrypt.org/directory)"
}
{
   "level":"error",
   "ts":1734093679.7995076,
   "logger":"tls.obtain",
   "msg":"will retry",
   "error":"[*.mysite.com] Obtain: [*.mysite.com] solving challenges: presenting for challenge: adding temporary record for zone \"_acme-challenge.mysite.com.\": zone not found: _acme-challenge.mysite.com (order=https://acme-staging-v02.api.letsencrypt.org/acme/order/158696983/21239318474) (ca=https://acme-staging-v02.api.letsencrypt.org/directory)",
   "attempt":2,
   "retrying_in":120,
   "elapsed":62.522038877,
   "max_duration":2592000
}
{
   "level":"info",
   "ts":1734093681.371488,
   "logger":"tls.obtain",
   "msg":"obtaining certificate",
   "identifier":"mysite.com"
}
{
   "level":"info",
   "ts":1734093681.3735995,
   "logger":"tls",
   "msg":"using ACME account",
   "account_id":"https://acme-staging-v02.api.letsencrypt.org/acme/acct/158696983",
   "account_contact":[

   ]
}
{
   "level":"info",
   "ts":1734093682.2132306,
   "logger":"tls.acme_client",
   "msg":"trying to solve challenge",
   "identifier":"mysite.com",
   "challenge_type":"http-01",
   "ca":"https://acme-staging-v02.api.letsencrypt.org/directory"
}

It appears that it cannot create the _acme-challenge. I wonder if something has changed in the Bunny DNS API. I think that this belongs to libdns-bunny.

After talking with the Bunny representatives, they got to this conclusion
https://caddy.community/t/issues-with-bunny-dns-module/27224/6
Judging by this, it seems that there are some issues with how Caddy handles the certificate creation, but I don't really believe that the problem is here in the libdns, still since Caddy uses a generic mechanism to generate the certificates and bunny is failing, this plugin should handle it.
I check other solutions like nginx with swag and they work perfectly for certs gen, so basically I not sure how we could handle this but I'm putting some ideas here

zone not found: _acme-challenge.mysite.com implies that Caddy/CertMagic is getting an SOA record for _acme-challenge.mysite.com instead of the registered domain, mysite.com. This could be an issue with the local DNS resolver(s) or with Bunny's API/record management (though that seems less likely).
CertMagic's DNS solver works well for over 60 DNS providers, so it's either a bug in this repository or in the environment it's being run in.

zone not found: _acme-challenge.mysite.com implies that Caddy/CertMagic is getting an SOA record for _acme-challenge.mysite.com instead of the registered domain, mysite.com. This could be an issue with the local DNS resolver(s) or with Bunny's API/record management (though that seems less likely).
CertMagic's DNS solver works well for over 60 DNS providers, so it's either a bug in this repository or in the environment it's being run in.

I dont understand what can be going on. I simply run two Caddy instances with caddy latest docker image being built with Bunny DNS thing
The problem is that we are missing so many debugging that its impossible
And I have 0 knowledge of Go.

@SirLouen

What it’s OMEGA weird, is that I used another domain, and it worked well I even tried to delete the certificates twice, and it worked again. I don’t really understand what Caddy is finding in my domain that decides to ask for the _acme-challenge while in the other domain it doesnt. The only difference between the two domains is that the one that doesnt work, has a number.

Running tests with my own domains on Bunny, I cannot see any new behavior creating or reading all possible record types. You say it continues to work on one domain but stopped working on the other and the only difference is a number in the domain name. I don't know if you wrote it somewhere but can you maybe tell me those two domain names? And for both you trying to issue a wildcard certificate right?
You can DM them to me if those domains are for some reason private for now.

Running tests with my own domains on Bunny, I cannot see any new behavior creating or reading all possible record types. You say it continues to work on one domain but stopped working on the other and the only difference is a number in the domain name. I don't know if you wrote it somewhere but can you maybe tell me those two domain names? And for both you trying to issue a wildcard certificate right?
You can DM them to me if those domains are for some reason private for now.

Are you testing with a single domain or a wildcard?

Yep, that's a good plan -- I don't maintain this repo (nor use Bunny myself) so I would direct that q toward @dennismuench :+1:


No print button

Describe the bug

Hello, so there is no print button when photo taken even with the Show print true. If you know how to fix it, thank you !
IMG_20240823_183321.jpg
To Reproduce
Take a photo and try to print it

Raspberry Pi Bookworm 64bit
Chromium
Version : the last one

Hi, you can check in the share section of the config
Number Direct Access Buttons
Number of buttons directly accessible in the gallery. Remaining items are available in the more-menu.
Does this fit with the no of buttons ?
If Yes it cloud be a browser cache problem.

Yes could be cache or after update there is no new default share option configured. Go to the share actions and ensure there is one.
Also ensure the global switch share-enable is active in that section.


Map condition doesn't work when SegmentList is followed by Segment of same entity type

If you have the following data:
...
PER*CX*PROVIDER SERVICES*TE*321321321*UR*www.default.org
PER*BL*PROVIDER SERVICES*TE*321321321*UR*www.default.org
PER*BL*PROVIDER SERVICES*TE*321321321*UR*www.default.org
PER*BL*PROVIDER SERVICES*TE*321321321*UR*www.default.org
PER*IC*PROVIDER SERVICES*TE*321321321*UR*www.default.org

...
...and a map that looks like this...
...
Segment(x => x.BusinessContactInformation, 4,
x => x.Condition = parser => parser.Where(p => p.ContactFunctionCode.IsEqualTo("CX")));
Segment(x => x.TechnicalContactInformation, 5,
x => x.IsRequired().Condition = parser => parser.Where(p => p.ContactFunctionCode.IsEqualTo("BL")));
Segment(x => x.PayerWebsite, 6,
x => x.Condition = parser => parser.Where(p => p.ContactFunctionCode.IsEqualTo("IC")));
...

...then the expectation is that I should be able to match on all PER segments that have a ContactFunctionCode of "BL" and return a SegmentList on the TechnicalContactInformation property. This works but when it is followed by another segment of the same entity type (e.g. PER) with different condition (e.g. ContactFunctionCode = "IC") it will return that segment as part of the previous segment list. This is similar to issue #58 but for segments of the same type coming after a segment list.
I have a failing unit test,  Layout835Tests.Verify_can_parse_PayerWebsite_when_multiple_TechnicalContactInformation_segments_present

This is also the case when you have only 3 in a row, 1 BusinessContactInformation = CX, 1 TechnicalContactInformation = BL, and 1 PayerWebsite = IC