Add all WMRuntime logs to the job logArchive; improve logging

Fixes #8423
Changes are:

replaced all print statements by logging for the runtime code (well, most of it), including the stage out code
updated some file handling with context managers
updated the wmagent job log name from jobLog.PID.log to wmagentJob.log
added the wmagentJob.log and the condor wrapper logs to the logArchive tarball (NOTE: those logs will be incomplete since the job is still running/logging). I checked a before/after my changes and the size difference seems to be around ~5KB per tarball
added a few 9999? exit codes to the exceptions list

Only issue is, for some reason most of the runtime logging is going to the wmgentJob.log AND the condor stderr wrapper log(?) If someone has a hint, I'm all ears.
TODO: we might get rid of the scramOutput.log files in favor of the wmagentJob.log (?) they usually have no content at all.

I can see how condor.stderr gets all the log as well. If not we should propagate wmagentJob.log to agent and parse that to post in wmstats.

I'm probably missing something, because this ad
ad['TransferOutput'] = "Report.%i.pkl,wmagentJob.log" % job["retry_count"]
which makes HTCondor to create those files comma separated and transfer them back ON_EXIT, is actually creating ONE file only Report.0.pkl,wmagentJob.log
when it's supposed to create TWO files, Report.0.pkl and wmagentJob.log
@bbockelm am I missing something?

Looking at the CRABServer code, they use this code
classad.quote(','.join(outputFiles))
which makes the job classad in condor look like
TransferOutput = "\"Report.0.pkl,wmagentJob.log\""
which creates files named as "Report.0.pkl and wmagentJob.log"...

test this please

test this please

@amaltaro - do you have an example of the generated ClassAd in the schedd log?  Particularly, what's in the ClassAd for the case where you try this:
ad['TransferOutput'] = "Report.%i.pkl,wmagentJob.log" % job["retry_count"]

That looks like it'd be the correct behavior, assuming ad is of type classads.ClassAd (and not a dictionary...).

@bbockelm Brian, I patched one of the testbed agents once again and assigned a new workflow:
775.9  1  493 /data/srv/wmagent/v1.1.10.pre5/install/wmagent/JobCreator/JobCache/amaltaro_MonteCarlo_LHE_Jan2018_Validation_180205_221509_5156/Production/JobCollection_504_0/job_1784/condor.775.9.log T2_CH_CERN
cmst1@vocms0262:/data/srv/wmagent/current $ condor_q -af:h TransferOutput
TransferOutput
Report.0.pkl,wmagentJob.log
...
cmst1@vocms0262:/data/srv/wmagent/current $ condor_q -l 775.9 | grep TransferOutput
TransferOutput = "Report.0.pkl,wmagentJob.log"

Let me know if this log above or the condor id is not enough for you.
And yes, we convert ads from python dict to classAd type:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/BossAir/Plugins/SimpleCondorPlugin.py#L641

Sigh... I see both logs back in the schedd now. I have no clue of what happened in my previous tests, because this was the first one I tried out and it did not work! Anyways, I'm updating this patch and then we can follow up with additional tests during the agent validation.

@ticoann please review again and merge it if it's fine.

I confirm this TransferOutput is still doing weird stuff, likely buggy...
This is the code put in place in this PR:
ad['TransferOutput'] = "Report.%i.pkl,wmagentJob.log" % job["retry_count"]
thus, condor is supposed to:
a) create these 2 files once the payload starts (I noticed a small delay btw)
b) transfer them back to the schedd, once the job has completed.
Looking at their creation (inside the worker node), this was the file created (as can be seen in every single logArchive tarball now):
../../Report.0.pkl,wmagentJob.log
while the files copied back are just fine (and there are two :)):
Job_4/wmagentJob.log
Job_4/Report.0.pkl

@bbockelm Brian, does it smell like a condor bug to you? Or is it that we missed a subtle detail?

@amaltaro - I think your step (a) is wrong above.  HTCondor doesn't create the files once the payload starts.  It appears that this is something custom in WMCore's bootstrap script:
https://github.com/dmwm/WMCore/blob/b1270ddd45e88f80e671c027224b31ac8c37f40f/etc/submit.sh#L61-L76
Probably that needs to get updated?  I guess that WMCore has never encountered jobs with more than one output file before?

I was pretty sure I had read somewhere that those files would be automatically created by HTCondor...
Yes, I think you're write, we need to update the submit.sh to properly handle TransferOutput, good catch!

I think what you remember is that the schedd will create the output files by default on the submit side (particularly when the command-line-client is used).  This was added about a decade back to give people an early heads-up if they tried to write the output to, for example, an AFS path that the schedd has no permissions to write into.


Create Pricing Card 2 black

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

Follow figma design and to make the button use Button Design System
Make the component responsive too.
After building the component please raise a PR with a screenshot of the component and add the component in Pricing Components/src/pricing-card-2.html and rename the file to pricing-card-2-black.html
If you need to use any icon please use it from Hero Icons
Good luck.

@RavindraP04 I want to work on this issue. Can you please assign it to me??

Can i contribute to this Under JWOC

@RavindraP04 , made a PR for this. Can you please look into this...


Add version status page

Changes

Add statuses of Jobs in CI for docker images of all unity versions (MVP)

Checklist

[x] Read the contribution guide and accept the code of conduct
[x] Readme (updated or not needed)
[ ] Tests (added, updated or not needed)

📝Changed routes:

/docs/docker/versions

(view full size)

Commit 23832895a59187dea46e443d5dd4d2f3f6228ae1 (https://website-4dyrwpo7p-game-ci.vercel.app).

Would it maybe make sense to flip the list around, so the latest Unity Version is on top? I assume that more people just show up to verify what the latest supported version is.
(hope this is fine here, didn't large enough for a separate issue)

Excellent suggestion. Thank you @jgollenz.
I'm planning to do this and some more UX related things in the near future. Stay tuned!
(always feel free to create an issue even for small issues, as it's most transparent)


"A Tour Of The Dart Language" refers to "%" as "the multiplicative operator"

At least twice, I read "the multiplicative operator %"...surely that should be "the modulo operator"?

There are several multiplicative operators (Section 16.28 in the spec v1.11 lists them: *, /, %, ~/). What this says is essentially "the operator % (which is by the way one of the multiplicative operators)..", and the main purpose would be to indicate how it fits into the grammar, e.g., with respect to precedence. So it's not so much of a contradiction as it sounds at first. ;-)

Ah..  If I read it as [one of] the multiplicative operator[s], that does
indeed make sense.  Thanks for the learning moment! :-)
On Tue, Mar 22, 2016 at 10:41 AM, Erik Ernst<EMAIL_ADDRESS>wrote:

Closed #26052 https://github.com/dart-lang/sdk/issues/26052.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/sdk/issues/26052#event-598348811

Thanks for being proactive!


BUG - NullReferenceException on SilkierQuartz.Controllers.JobsController.AdditionalData

This took a little more finagling than the samples but I was able to get it working and the UI coming up, but I'm on .NET Core 3.1 and that seems to be par for the course on a lot of packages ;)
The issue I was running into was anytime I would click on Jobs or Triggers there'd be a NullReferenceException thrown out of System.Runtime.CompilerServices.AsyncStateMachineBox. Digging into this I was able to trace it to:

 This exception was originally thrown at this call stack:
    SilkierQuartz.Controllers.JobsController.AdditionalData()
    System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() in ExceptionDispatchInfo.cs
    System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task) in TaskAwaiter.cs
    System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task) in TaskAwaiter.cs
    System.Runtime.CompilerServices.TaskAwaiter<TResult>.GetResult() in TaskAwaiter.cs
    Microsoft.AspNetCore.Mvc.Infrastructure.ActionMethodExecutor.TaskOfIActionResultExecutor.Execute(Microsoft.AspNetCore.Mvc.Infrastructure.IActionResultTypeMapper, Microsoft.Extensions.Internal.ObjectMethodExecutor, object, object[]) in ActionMethodExecutor.cs

specifically this line:
var history = await Scheduler.Context.GetExecutionHistoryStore().FilterLastOfEveryJob(10);
That call to GetExecutionHistory I suspect is the culprit returning null. By default the history plugin is not enabled, but once I was able to figure out how to add it in my startup, the NullReference went away. If anyone else comes across this, here's what I did:
Installed these packages:
Quartz.Plugins
SilkierQuartz.Plugins.RecentHistory
Then you register the history plugin using the properties of the Quartz.SchedulerBuilder and the error vanishes.
 public async Task<IScheduler> GetScheduler(string connString) {
    System.Collections.Specialized.NameValueCollection nvc = new System.Collections.Specialized.NameValueCollection();
    nvc.Add("quartz.plugin.recentHistory.type", "Quartz.Plugins.RecentHistory.ExecutionHistoryPlugin, Quartz.Plugins.RecentHistory");
    nvc.Add("quartz.plugin.recentHistory.storeType", "Quartz.Plugins.RecentHistory.Impl.InProcExecutionHistoryStore, Quartz.Plugins.RecentHistory");
    var schedBuilder = SchedulerBuilder.Create();
    schedBuilder.Properties.Add(nvc);
    schedBuilder.UsePersistentStore(x => {
	 x.UseProperties = true;
	 x.UseSqlServer(connString);
	 x.UseJsonSerializer();
     });

    var sched = await schedBuilder.BuildScheduler();

    await sched.Start();

    return sched;
}

I cannot reproduce this question

fixed


Support multiple flows

Currently, the specification of a system (i.e. YAML) supports multiple flows but the editor / presentation only support one flow.
The current workaround is to duplicate a system to create another flow.
Instead, it would be useful to CRUD flows in a same system to present multiple use cases.

By multiple flows, what exactly do you mean?

@Stan-Stani: If you look at the JSON of this system (view > json editor), you will see:
{
  "flows": [
    {
      "title": "flow 1",
      "steps": [
        {
          "keyframe": 0,
          "from": "foo",
          "to": "bar"
        }
      ]
    },
    {
      "title": "flow 2",
      "steps": [
        {
          "keyframe": 0,
          "from": "bar",
          "to": "foo"
        }
      ]
    }
  ]
}

So it is currently possible to define multiple flows for the same system but there is nothing in the UI to CRUD those multiple flows. Right now, there is a lot of hardcoded code that only works for one flow.
The idea of supporting multiple flows is to create a system once, and then show multiple use cases of that system. For example, here's a use case and here's another use case. Those are 2 completely independent graphs however. It would have been nice that I give you 2 links that looks like:
https://gg-charts.com#file=<X>&flow=0
https://gg-charts.com#file=<X>&flow=1

I'm pivoting this project to a general diagramming tool so there is no need for this feature anymore. Sorry!

@jodigiordano Would you mind explaining your decision to remove the animation of the flows? Just trying to figure out why that feature was removed. General diagramming tools don't need animations? :) 🧐


[Bug 🐞] Failed to fetch dependencies

Hi @bundlejs, thanks for your great tool. Today I checked my small UI library - wrap of Material UI --> But I have the error like the image below. I hope you can check that!

Link: https://bundlejs.com/?q=%40onemind%2Fui

@harrytran998 Unfortunately, this doesn't seem to be something I can fix. It seems due to the high number of packages and the even higher number of network requests, browsers are basically giving up. I'll try to optimize the fetch process but I'm not sure I can actually fix this problem.

After a bit of research I've determined I can't fix this, at least not while bundlejs is running in a browser. I'll look into other alternative solutions to fix this.

Thanks for your quick response. I think your solution is valid for now(call HTTP in browser). So I will try to pre-bundle MUI. But if you find something interesting alternative solutions, pls let's me know 😆 🙏.

@harrytran998 It's now fixed, give it a try https://bundlejs.com/?q=@onemind/ui&bundle

Please let me know if you run into this problem again

Thanks for your work on this 😆  I checked it and everything ok! How do you handle this without call HTTP like before?

I made the package fetching algorithm more specific than before, allowing it to avoid fetching packages that could just be ignored or treeshaken away

thanks for your hard work <3


Issue#22

v0.1.6

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@68dbf51). Click here to learn what that means.
The diff coverage is 10.12%.

@@            Coverage Diff            @@
##             master      #24   +/-   ##
=========================================
  Coverage          ?   64.92%
  Complexity        ?     1563
=========================================
  Files             ?       80
  Lines             ?     4069
  Branches          ?      398
=========================================
  Hits              ?     2642
  Misses            ?     1206
  Partials          ?      221

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/jsonvalues/JsObj.java
69.01% <ø> (ø)
41 <0> (?)

src/main/java/jsonvalues/Json.java
41.46% <ø> (ø)
84 <0> (?)

src/main/java/jsonvalues/JsArray.java
51.68% <ø> (ø)
47 <0> (?)

src/main/java/jsonvalues/Position.java
77.77% <ø> (ø)
4 <0> (?)

src/main/java/jsonvalues/InternalError.java
0% <0%> (ø)
0 <0> (?)

...c/main/java/jsonvalues/OpFilterMutableArrKeys.java
0% <0%> (ø)
0 <0> (?)

src/main/java/jsonvalues/TryObj.java
64.28% <0%> (ø)
9 <0> (?)

src/main/java/jsonvalues/OpMapMutableArrKeys.java
0% <0%> (ø)
0 <0> (?)

...main/java/jsonvalues/OpFilterImmutableArrKeys.java
81.81% <0%> (ø)
7 <0> (?)

src/main/java/jsonvalues/JsElem.java
62.5% <0%> (ø)
48 <0> (?)

... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 68dbf51...3e06919. Read the comment docs.


params of request not support multi tags

type Request struct {
	Name string `json:"name" form:"name" path:"name"`
}

when pass params on url
curl -i http://localhost:8888/users/id/1?name=1
and then get error: "field name is not set"

Don't use the same name in path, form and json.


Device uuid

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[X] iOS
[X] Android
[] tvOS
[] macOS
[] .NET Standard
UWP IS NOT SUPPORTED DURING THE BETA PERIOD - ONLY PRs ARE WELCOME FOR NOW

Version of OS(s) listed above with issue
iOS 12.1.4
Android 8.0
Version of Library
6.2.3
Expected Behaviour
I want to have one phone connect to another phone.
To do that I show a QR Code on phone A with the bluetooth device uuid and scan that with phone B so they can find each other and connect.
I'd like to know how I can find the uuid on phone A that is listed in the scan result of the Adapter.Scan() on phone B. So something like CrossBleAdapter.Current.DeviceUuid
Actual Behavior
If applicable, please include entire exception - type, message, stacktrace
Code of Conduct

No Library Update Requests
No Timeline Requests
Please read through the documentation and FAQs
If you have a complaint about the docs, FIX IT - DO NOT FILE IT HERE!  I won't guarantee everything is always 100% up-to-date since I am just 1 person.
Please be respectful of my time. I put a lot of effort into my libraries.  I try to support the issue requests within a day.  I'm not psychic, so please take the time to fill out the above form as complete as possible

There isn’t such a thing.  You can advertise a custom service uuid and scan for that though

Do you mean this?
BluetoothServer = await BluetoohAdapter.CreateGattServer();
Plugin.BluetoothLE.Server.IGattService service = BluetoothServer.CreateService(guidService, true);
And then scan with a ScanConfig?
ScanConfig sc = new ScanConfig() {
ServiceUuids = new List() { searchForServiceUuid },
ScanType = BleScanType.Background,
AndroidUseScanBatching = true
};
Because I tried that and it didn't find the other phone.

It looks right.  I can't formulate a guess without reproducible case.
Please note, this isn't a support channel.  I do offer consultancy if you need help getting this going.


problem of settings sync on different OS

VS Code Version: 1.65.2
OS Version: macos 12.2

using "settings sync" plugin to sync settings in different os, config with absolute path is not working well in diff os:
  // golang
  "go.goroot": "D:\\go\\anzhuang",
  "go.gopath": "D:\\go\\go_modules",

it's not work in macos.
how to distinguish configs with diff os?

using "settings sync" plugin

Are you using the Settings Sync extension?


Win

Describe the bug
A clear and concise description of what the bug is.
WinError 2 issue
To Reproduce
Steps to reproduce the behavior.
start the .exe
Expected behavior
A clear and concise description of what you expected to happen.
I expected no erros
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Windows]
Version [e.g. Pro 64 bits]
Windows 11 Pro 64 bits
Additional context
Add any other context about the problem here.
client.log

The title was supposed to be ''WinError 2 '', my bad

Hi,
This problem is due to the system's language according to my diagnosis. Please try using Windows in English. If this is not the case, please let me know more details.
Thanks for reporting it.

Oh alright, i'll change the language that's fine to me. I'll reply again if i come across the issue again in English

Yep, sadly, the same issue; the same WinError2, just in English. I'm aware that Windows still retains some of the old language it was installed on, but i'm not sure if that's the issue anymore.

Please try the v0.5.1 version. maybe it will work for you as an alternative.

Sadly that doesn't work aswell

Try using a different browser. If you still get the error then let us know.


Update usbip to <IP_ADDRESS>

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

Update usbip to <IP_ADDRESS> to match the kernel.
~Also add it to the entangled specs list. usbip is built with kernel sources, so it should match the current kernel version.~
Change Log

~Add usbip to check_entangled_specs.py with the kernel.~
Upgrade usbip to <IP_ADDRESS>
Fix supporting script for generating source tar

Does this affect the toolchain?

NO
Test Methodology

Local build
https://dev.azure.com/mariner-org/mariner/_build/results?buildId=531922&view=results
Testing path is  unclear

Dropping from entanglement, shouldn't be needed as the tool updates incredibly infrequently.


Website: E-Mail subscription does not work

After entering an email on Gitpromote and pressing OKAY doesn't do anything. Please fix it. Thank you!

Actually I haven't updated the code. The site is actually running on an another URL. Because of few issues I haven't updated the code. I assure you that the site will run perfectly fine in a week at https://gitpromote.appspot.com . I will also update the code on GitHub within a week.


Unrelated to repo

How did you make that cool ANSI text art in vscode (in the screenshot) 😃

neofetch 🙂
apt install neofetch -y

Thank you!

Thank you!

np~


Qh add lb to docker compose

This replicates the DAMS load balancer for replicating the gateway and dbsync.  One problem I see with this, is that the nature of the way data comes in is such that I fear we would end up with both dbsyncs always trying to update the same document.  We need to test how often these conflicts occur, and make sure version checking makes sure this doesn't lead to bad data.
On the other hand, as we move to three indexes (work,grant,expert) more work will be done on other indexes and different records in that case.

We are waiting for the Kubernetes Implementation for this.


UI stopped animating during heavy operations

I'm implementing a search function in my Flutter app. The search function loads a data file in the local storage and walk through the data according to the search query. It usually takes 1 - 5 seconds to execute the search. I want to display a LinearProgressIndicator to indicate that the search is in progress. However the progress bar is frozen during the search operation.
Here's the actual code.
// this method is called upon tapping the search button
Future _executeSearchQuery() async {
  if (_searchQuery.queryString == null || _searchQuery.queryString.trim().isEmpty) return;
  if (_searchQuery.books == null || _searchQuery.books.length <= 0) return;
  setState(() {
    // this state enables the progress bar
    _isSearching = true;
  });
  _thisScaffoldKey.currentState.showSnackBar(new SnackBar(
    content: new Text('Searching...'),
    backgroundColor: Theme.of(context).primaryColor,
  ));

  _results = await SearchEngine.search(_data, _searchQuery); // the heavy operation that takes 1-5 sec
  setState(() {
    // disable the progress bar
    _isSearching = false;
  });
}

Can you post the code for build() function as well?

If SearchEngine.search runs in the UI thread (isolate) and does not yield frequently there is no time for other ui tasks like animation. So this is expected behavior.
You can for example use https://docs.flutter.io/flutter/foundation/compute.html to execute code in a different isolate, but there are some limitations. Currently plugins can only be used from the UI isolate.

@Kartik1607 async is concurrency on the same thread. If some code waits for async execution to complete, like a network request, it registers a callback which will be added to the event queue and execution yields. If the even (network request completed) is emitted and no other sync code is executed the callback is taken from the queue and executed (.then(...) or await in Dart).
So there can only be one sync line of code be executed at a time.
An isolate is a new thread with its own event queue that can run in parallel with code in the main isolate (if there are enough cores to actually run code in parallel on your system, but that's usually the case nowadays)
In Dart communication between isolates happens about message passing, so there can be no race conditions caused by 2 or more threads accessing a variable at the same time, but this also means that one isolate can work with stale data if you do not design synchronization carefully.

@StaymanHou, are you using any of the sync variants of the file APIs in that method?  If you could post the source of it that would help to identify the issue.

@Kartik1607 , here's the code for the build function
@override
Widget build(BuildContext context) {
  return new Scaffold(
    key: _thisScaffoldKey,
    appBar: new AppBar(
      title: new Form(
        child: new TextFormField(
          decoration: new InputDecoration(
            hintText: 'Search ...',
            suffixIcon: new IconButton(
              icon: const Icon(Icons.clear),
              onPressed: () {
                _searchFieldController.clear();
                setState(() {
                  _searchQuery.queryString = '';
                  _results = null;
                });
              }
            )
          ),
          autofocus: true,
          controller: _searchFieldController,
          onFieldSubmitted: (String value) {
            _searchQuery.queryString = value;
            _executeSearchQuery(); // this is the heavy search operation
          },
        )
      ),
      actions: <Widget>[
        new SizedBox(
          width: 70.0,
          child: new FittedBox(
            fit: BoxFit.scaleDown,
            child: new FlatButton(padding: EdgeInsets.zero, onPressed: _showTranslationDialog, child: new Text('$_currentTranslationShortName  ▼')),
          ),
        ),
        new IconButton(
            icon: Icon(Icons.filter_list),
            color: _searchQuery.books.length == 0 ? null : Theme.of(context).accentColor,
            tooltip: 'Filter',
            onPressed: _showFilterDialog
        )
      ],
      bottom: _isSearching == false ? null : new PreferredSize(
        child: new LinearProgressIndicator(), // this is the progress bar that freezes
        preferredSize: new Size.fromHeight(0.1),
      ),
    ),
    body: _buildSearchResultBody()
  );
}

@jonahwilliams, yes, I'm using some of the sync variants of the file APIs in that method. It's hard to post the code of the search method. However, I can put part of the underlining random file access class.
MyFileStream(String filepath) {
  File file = new File(filepath);
  _fis = file.openSync();
  _pos = 0;
}

// ...

int get currentPosition => _pos;
int get size => _fis.lengthSync();

// ...

Uint8List read(int nbytes) {
  _pos += nbytes;
  return _fis.readSync(nbytes);
}

// ...

void seek(int position) {
  _fis.setPositionSync(position);
  _pos = position;
}

// ...

void close() {
  _fis.closeSync();
}

@zoechi, here's the flutter doctor output:
[✓] Flutter (Channel beta, v0.5.1, on Linux, locale en_US.UTF-8)
    • Flutter version 0.5.1 at /home/stayman/flutter
    • Framework revision c7ea3ca377 (5 weeks ago), 2018-05-29 21:07:33 +0200
    • Engine revision 1ed25ca7b7
    • Dart version 2.0.0-dev.58.0.flutter-f981f09760

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /home/stayman/Android/Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /home/stayman/Downloads/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
    • All Android licenses accepted.

[✓] Android Studio (version 3.0)
    • Android Studio at /home/stayman/Downloads/android-studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)

[✓] IntelliJ IDEA Community Edition (version 2018.1)
    • IntelliJ at /home/stayman/Downloads/idea-IC-181.4203.550
    • Flutter plugin version 23.2.3
    • Dart plugin version 181.4445.29

[!] Connected devices
    ! No devices available

! Doctor found issues in 1 category.

@StaymanHou those sync API will entirely block the UI while they are executing, and if you have multiple calls to them it makes sense that it will freeze your app.  They are really only designed to make command like scripts easier to write and should generally never be used on a Flutter app.  We should definitely do something so that users are aware of this

You could also run those operations in an isolate like @zoechi suggested

Oh, I see. That make sense. Thanks for looking into the issue.

@StaymanHou There is a convenience function in the flutter foundation library, just import flutter/foundation.dart.  https://docs.flutter.io/flutter/foundation/compute.html

I encountered a similar issue.
In my app, I need to make a computation (specifically, an inference of AI, using tflite_flutter package) which takes several seconds. The method is async. Meanwhile, I want to inform the user by a circular progress indicator, but the UI freezes until the computation finishes, thus progress indicator doesn't animate at all.

You can for example use https://docs.flutter.io/flutter/foundation/compute.html to execute code in a different isolate, but there are some limitations. Currently plugins can only be used from the UI isolate.

Is this still the case? I mean, can I run a heavy computation using a plugin on a separate isolate? If not, could you suggest a workaround? All I need is informing the user with an animation that computation is continuing.


avx512vl: implement mm_abs_{mask,maskz}_{epi32,epi64}

I think this is a duplicate of #556, with a slightly different name?  Closing, but if it was intentional feel free to reopen.
BTW, you can edit the title of PRs.  Next to the title, all the way on the right, there is an "Edit" button.

I think this is a duplicate of #556, with a slightly different name? Closing, but if it was intentional feel free to reopen.
BTW, you can edit the title of PRs. Next to the title, all the way on the right, there is an "Edit" button.

These are different functions, the previous do not include for epi32 and epi64.


upgrade argon2-ffi

upgrade to 1.1.1.
Hopefully this resolves the warning about Buffers.

Blocked by cjlarose/argon2-ffi#38

Fixed in 1.1.2, upgrade is in master branch. Will check deprecation warning later.


LAWS-823-Alter-cpu-memory-and-timeout

Restore Load Balancer timout value
Increase Memory and CPU values

This is required as tests are showing that cpu is fully utilized on
startup causing the Load Balancer checks to fail. Will evaluate and
request load testing to finalise.

This approach is no longer required


update samples/admin

What this PR does:

update samples/admin by deleting old project and adding yuque document link
add return statement after handling method config put event

Codecov Report

Merging #208 (b63e352) into develop (ffc58b6) will increase coverage by 0.99%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop     #208      +/-   ##
===========================================
+ Coverage    39.35%   40.34%   +0.99%
===========================================
  Files           39       39
  Lines         1987     2357     +370
===========================================
+ Hits           782      951     +169
- Misses        1107     1304     +197
- Partials        98      102       +4

Impacted Files
Coverage Δ

pkg/config/api_config.go
2.47% <0.00%> (-0.02%)
:arrow_down:

pkg/filter/recovery/recovery.go
69.23% <0.00%> (-8.55%)
:arrow_down:

pkg/filter/logger/logger.go
66.66% <0.00%> (-8.34%)
:arrow_down:

pkg/common/yaml/yaml.go
86.66% <0.00%> (-4.25%)
:arrow_down:

pkg/filter/timeout/timeout.go
90.62% <0.00%> (-1.97%)
:arrow_down:

pkg/filter/authority/authority.go
79.31% <0.00%> (-1.65%)
:arrow_down:

pkg/service/api/discovery_service.go
41.79% <0.00%> (-0.52%)
:arrow_down:

pkg/router/route.go
78.12% <0.00%> (-0.19%)
:arrow_down:

pkg/pixiu/http.go
100.00% <0.00%> (ø)

... and 30 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ffc58b6...b63e352. Read the comment docs.


Admin CLI: skip all replication tasks for particular workflow in worker or replicatorQueue processor

This can be a generic way to mitigate issue when our worker is stuck at processing some problematic workflow

not needed as discussed in standy


[TASK] Implement logic/UI that allows for delegating one's tokens to an operator

The UI contains:

A select button that allows users to select an operator to delegate the asset to.

An operator selection card that shows the list of operators in the system for users to select.

A select button that allows users to select an asset to delegate to an operator.
An amount input that allows users to enter a delegate amount.

Screenshot

"Estimated undelegate delay"


Add coveralls support and badging

I don't think the rule spec runs through Travis right now, but I'm not sure why - it has a .travis.yml.

We're going to have to get @jmarin or @ericspry involved to get this repo added into Travis. Not sure why it wasn't added, or that no one really noticed, when we got these all setup initially.

For cfpb/hmda-pilot#294


Static pull

[ ] Make sure pull strips metadata keyword away

Thanks for point out Dilyan! Will take a look at it!

Thanks @dilyand. It has been migrated to https://github.com/snowplow-incubator/igluctl/pull/7


Address ARC feedback for RC3

Address ARC feedback for RC3 (Issue #14)
Fixes: https://github.com/riscv-non-isa/riscv-acpi-rimt/issues/14
Closes: https://github.com/riscv-non-isa/riscv-acpi-rimt/issues/14

Looks good to me.


How does the app still run if vendor.browser.ts was removed?

Just wondering, how is it possible that the app compiles and runs still even though all the @angular/core etc stuff from vendor.browser.ts no longer gets built by webpack, since that file is now missing?
Was there a change to angular or webpack which makes those imports obsolete?

@IAMtheIAM It doesn't really have to be in vendor.browser.ts to be part of the webpack process.
The vendor file was the place to put all vendors that you want to have put in the vendors chunk (bundle).
Without it webpack will still load all the vendor files since they are referenced (import X from Y) inside modules, so each reference is a place for webpack to add that vendor to the bundle.
I think now this can be done without an vendor file, but with a "catch-all" chunk that will be used for that purpose - CommonsChunkPlugin
It was commented out 2+ months ago, don't know why.

@shlomiassaf Thanks for the explanations. I thought the vendors files would be loaded from the import X from Y statements, glad you confirmed.


Improved Location, MapOptions & Marker.

Changes:

Added Location attributes to receive a most complete location update from Android and iOS.

Attributes: time (in millisecond), speed (meter/second), bearing/course (degree)  altitude (meter), horizontalAccuracy (meter), verticalAccuracy (meter).

Added MapOptions attributes to improve configuration of the map visualized.

Attributes: showMyLocationButton (bool), showCompassButton (bool), hideToolbar (bool)

Fix: Fixed problem in iOS that avoided the correct display of certain mapview "uisettings". In short, some setting was being set before initializing the GMSMapView.
Added draggable Attribute for Marker class and listeners for drag events (Drag, DragStarted, DragEnd).
Added custom icon support for Marker class.
Added marker rotation support.
Camera position update to support bearing and tilt.

Hi @matthewtsmith any chance to merge it?


Fix build issue

Type of change

Enhancement / new feature
Refactoring

Description
Reduce version of React and Typescript due to build failed on master.
https://github.com/EnMasseProject/enmasse/pull/4912

Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

I have run the tests for #4912 locally, and they pass with the versions of typescript and react that are currently on master.  The build failures on #4912 do not seem related to these versions.

I looked at the failure from #4912 (which I assumed was where the failing build was being reported from)
12:45:45.491 [main] INFO io.enmasse.iot.tenant.impl.AbstractTenantService - Failed to validate trust anchor
java.lang.NullPointerException: text

It doesn't seem related to the typescript / react versions, to me. Is this the build failure you are trying to fix? (Currently unit tests are passing on master).

@vbusch This doesn't seem to be causing the build issue as this exception is there in successful builds as well.

The unit tests seem to be passing on master currently.
I believe this commit is for #4912
Do you know what is in the newer minor versions that would be an issue for that PR?

These package version causing issue "react": "^16.13.1", "typescript": "3.8.3"

@ajaypratap003 Just question don't you need for this specific react and typescript version some newer system wide dependency like npm or something like that? :)


[tcgc] do not replace operation with @override when listOperationsInOperationGroup

fix: https://github.com/Azure/typespec-azure/issues/1857

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-client-generator-core

Show changes
@azure-tools/typespec-client-generator-core - feature ✏

add isPagedResultModel helper function

@azure-tools/typespec-client-generator-core - feature ✏

add disableUsageAccessPropagationToBase to support language that does not generate base model

@azure-tools/typespec-client-generator-core - fix ✏

do not replace operation with @override when listOperationsInOperationGroup

@azure-tools/typespec-client-generator-core - fix ✏

do not add protocol usage for protocol method's response type

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


files().list(q="name contains 'substr'") only matches when the substr at the beginning of name or after a space

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Please run down the following list and make sure you've tried the usual "quick fixes":

Search the issues already opened: https://github.com/googleapis/google-api-python-client/issues
Search StackOverflow: https://stackoverflow.com/questions/tagged/google-cloud-platform+python

If you are still having issues, please be sure to include as much information as possible:
Environment details

OS type and version: Darwin Kernel Version 21.2.0
Python version: Python 3.9.10
pip version: pip 22.0.3 from /usr/local/lib/python3.9/site-packages/pip (python 3.9)
google-api-python-client version:

Name: google-api-python-client
Version: 2.36.0
Summary: Google API Client Library for Python
Home-page: https://github.com/googleapis/google-api-python-client/
Author: Google LLC
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /usr/local/lib/python3.9/site-packages
Requires: google-api-core, google-auth, google-auth-httplib2, httplib2, uritemplate
Required-by:

Steps to reproduce

crate a folder named SMITH_JOHN TAYLER_DAVID
try to list the folder with service.files().list(q=f"mimeType = 'application/vnd.google-apps.folder' and name contains 'JOHN'")
It will not show the folder. It only shows when using SMITH or TAYLOR

Code example
import os
import yaml
from google.auth.transport.requests import Request
from google.oauth2.credentials import Credentials
from google_auth_oauthlib.flow import InstalledAppFlow
from googleapiclient.discovery import build

creds = None
scopes = ["https://www.googleapis.com/auth/drive"]

if os.path.exists("token.json"):
     creds = Credentials.from_authorized_user_file("token.json", scopes)
if not creds or not creds.valid:
    if creds and creds.expired and creds.refresh_token:
        creds.refresh(Request())
    else:
        flow = InstalledAppFlow.from_client_secrets_file("credentials.json", scopes)
        creds = flow.run_local_server(port=0)

with open("token.json", "w", encoding="utf-8") as token:
    token.write(creds.to_json())

service = build("drive", "v3", credentials=creds)

response = service.files().list(q=f"mimeType = 'application/vnd.google-apps.folder' and name contains 'SMITH'",
                                spaces = 'drive',
                                fields='nextPageToken, files(id, name)',
                                orderBy = 'name').execute()

Stack trace
# example

Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

Hi @michaelxwang,
Please could you try out a similar request in the 'Try this API' interface on this page and check whether the issue still exists? This helps to confirm if this is a client library issue or a API issue. If it is an API issue, we'll redirect you to the API support channel so you can get better quality answers. For the Drive API please see https://developers.google.com/drive/api/v3/support.

I also ran up against this same issue today, and I can confirm that the issue still exists, and does seem to be an API issue.  (I get the same unexpected null result when searching via the python client, or from the "try this API" page linked above.)

I created a ticket over here: https://issuetracker.google.com/issues/225213315

@odoublewen I created a bug, they say it is not a bug. Then I created a feature request: https://issuetracker.google.com/issues/220326928 , please star this feature request. Thanks.

Ah I see.  Thanks @michaelxwang , good to have that info and the reference here.
@parthea I think you can close this issue


License needed

As an open source project, we need to attach a proper license to this project.
Typically this is done by adding a LICENSE file in the root directory and attach the licensen declaration in every files.
I suggest we apply Apache2 to the project.
How do you think?

yeah,  Apache2.0 license is really a good choice,  i will add it to our project.


Issue: Handle hosts file limitations

Describe the bug
We should take care about hosts file limitations as described in https://documents.uow.edu.au/~blane/netapp/ontap/nag/networking/concept/c_oc_netw_maintaining_host_file_limits.html
Particular attention to points

1: Maximum line size is 256.
3: Maximum domain name length is 256.
4: Maximum search domains limit is 6. The total number of characters for all search domains cannot exceed 256.

To Reproduce
Steps to reproduce the behavior:
If you add more than 6 hostnames to a line, other hostnames are not recognized by the system
Expected behavior
The 7th hostname should automatically create a new line

We need to handle 2 cases here:

A HostsFile is parsed that contains a line with more than 256 chars or more than 6 hostnames
AddHostsFileLine is called on an existing IP Address that already renders with more than 256 chars or already has 6 hostnames

6 Hostnames limitation handled #23


Peek References: double click no longer takes me to reference

Steps to Reproduce:

peek references
double click on an entry in the tree

=> :bug: you no longer navigate to that reference
Sounds a lot like https://github.com/Microsoft/vscode/issues/69284 which also shows broken double-click behaviour.
//cc @joaomoreno

@joaomoreno It seems that the tree doesn't give me a selection change for the double click but that it does things on single click already


Why not coming ngrok link?

Turn ON WiFi Hotspot and Mobile Data connection together. It will work.

I want to use it in Kali Linux

Turn ON WiFi Hotspot and Mobile Data connection together. It will work.

I want to use it in Kali Linux. But the "ngrok" link is not coming to Kali Linux. What can I do now? Do you have any suggestions or solutions for this?

Not workable for Linux.

Not workable for Linux.

That's true. But I came up with a new method. It's working on Linux. And thanks.


[0.57.1][IOS] Nested Image will overlap the text if lineHeight is smaller than the image height.

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
React Native Environment Info:
System:
OS: macOS High Sierra 10.13.4
CPU: x64 Intel(R) Core(TM) i7-4578U CPU @ 3.00GHz
Memory: 17.04 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 8.9.3 - /usr/local/bin/node
Yarn: 1.9.2 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
Android SDK:
Build Tools: 23.0.1, 23.0.2, 23.0.3, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.1, 26.0.2, 26.0.3, 27.0.0, 27.0.2, 27.0.3
API Levels: 23, 25, 26, 27
IDEs:
Android Studio: 3.0 AI-171.4443003
Xcode: 9.4.1/9F2000 - /usr/bin/xcodebuild
npmPackages:
react: ^16.5.0 => 16.5.0
react-native: ^0.57.1 => 0.57.1
npmGlobalPackages:
create-react-native-app: 1.0.0
react-native-cli: 2.0.1
Description
Expected Result - lineHeight should be adjusted to image height because image height is actually larger than the lineHeight value.

Actual Result - lineHeight is still 20 even the image height is larger than its value. So the image is overlap the text.

This problem actually occurred after rewriting the Text component.
The old version of Text component actually implement the correct logic which can be see in :
    if (heightOfTallestSubview > lineHeight) {
      lineHeight = ceilf(heightOfTallestSubview);

https://github.com/facebook/react-native/blob/486dbe4e8f4e3710c258e22af00b2aa7155ef2f3/Libraries/Text/RCTShadowText.m
Reproducible Demo
import {Image, StyleSheet, Text, View} from 'react-native';
export default class App extends Component {
  render() {
    return (
      <View style={styles.container}>
        <Text style={styles.text}>
<PHONE_NUMBER>123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890
          <Image
            resizeMode="contain"
            source={{uri: 'http://via.placeholder.com/65x94'}}
            style={{width: 65, height: 94}}
          />
        </Text>
      </View>
    );
  }
}
const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#F5FCFF',
  },
  text: {
    backgroundColor: 'pink',
    lineHeight: 20,
    flexWrap: 'wrap'
  }
});

0.56.0 has same issue.

Any workaround ? It is really important to me....


Class active inside slider messes navigation up

Expected behavior
Slider should go to the next slide
Current behavior
Slider jumps to the first element with active class found, even though it is not a slide.
It implies wrong initialization, and also active class is not removed from the previous active slide.
Possible solution
The quickest fix is to change the class active to active-slide (for example). It will avoid all conflicts if slides have their own active class.
Steps to reproduce
Add any element with active class inside a slider.
Here's a codepen : https://codepen.io/anon/pen/jLeajN
Context
I'm using a slider with a form on each slide, and I noticed an issue with the active class dynamically added to a text-field label. When firing prev or next functions, the slider jumps to a wrong slide because of the false active index of the label.
Environment
Version used : 0.99 of Materialize (though the code of slider hasn't changed in newer versions)
Browser : Firefox 54 (64-bit)
OS : Linux Mint 18 Sarah Xfce 64-bit

works for me if I put "active" on list element, which makes more sense anyway...

Slide 4

Made active check more specific.
Fixed in 05e2cfc1


Prueba

Codecov Report

Merging #59 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop      #59   +/-   ##
==========================================
  Coverage      63.37%   63.37%
  Complexity       121      121
==========================================
  Files             39       39
  Lines            385      385
  Branches          21       21
==========================================
  Hits             244      244
  Misses           130      130
  Partials          11       11

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46d73ae...78174af. Read the comment docs.


Typo in README.md

Hi,
Thank you for this, this is awesome.
Two minor things:

In README.md, an example sshConfig is given with a property called passphrase. That should be password to work properly.

And from NPM there is no direct link to this Git-repo, that would be nice too!

Hi Reggino,
thanks for the tip about the missing npm information, i'll fix this in the next minor.
The config property "passphrase" is correct as far as i can see.
You should use passphrase only if you are providing a key created with a passphrase.
If you use username / password for authentication you never need a passphrase.

Thanks @agebrock, you are completely right. I am not familiar with mscdex/ssh2 and i made an assumption based on the example set me on the wrong foot here.

Np i guess the 1.0.0 description is even worse, you can still use username and password ;)


Exchange Redeem Feature

Pull Request Test Coverage Report for Build 4528

9 of 36 (25.0%)  changed or added relevant lines in 3 files are covered.
597 unchanged lines in 28 files lost coverage.
Overall coverage decreased (-70.8%) to 29.076%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contracts/core/extensions/CoreModuleInteraction.sol
0
10
0.0%

contracts/core/modules/ExchangeRedeemModule.sol
0
17
0.0%

Files with Coverage Reduction
New Missed Lines
%

contracts/core/extensions/CoreOperationState.sol
3
33.33%

contracts/core/exchange-wrappers/ZeroExExchangeWrapper.sol
3
86.84%

contracts/core/tokens/SetToken.sol
3
72.22%

contracts/core/tokens/rebalancing-libraries/StandardPlaceBidLibrary.sol
4
0.0%

contracts/core/exchange-wrappers/KyberNetworkWrapper.sol
4
82.35%

contracts/lib/CommonMath.sol
6
9.09%

contracts/core/extensions/CoreModuleInteraction.sol
7
28.0%

contracts/core/modules/RebalancingTokenIssuanceModule.sol
10
0.0%

contracts/lib/Authorizable.sol
11
29.63%

contracts/lib/TimeLockUpgrade.sol
11
18.18%

Totals

Change from base Build 4512:
-70.8%

Covered Lines:
305

Relevant Lines:
929

💛  - Coveralls


Google -> TSV migration dashboard

We need script(s) that can iteratively document the progress of converting the MIxS Google sheets into schemasheets TSVs, with no duplications or contradictions (except in the case of slot usages)
Ideally it should be easy for a non-developer to run
From

MIxS6 packages - Final_clean
MIxS6 Core- Final_clean

to

schemasheets/tsv_in/MIxS_6_term_updates_classdefs.tsv
schemasheets/tsv_in/MIxS_6_term_updates_global_partial_slotdefs.tsv
schemasheets/tsv_in/MIxS_6_term_updates_slot_assignments_and_usages.tsv

Questions:

input? preferably the TSVs.
output?

[ ] Make sure that each class and term/slot is defined once and only once.
[x] Every class mentioned in schemasheets/tsv_in/MIxS_6_term_updates_slot_assignments_and_usages.tsv should appear in schemasheets/tsv_in/MIxS_6_term_updates_classdefs.tsv
[x] Every class mentioned in schemasheets/tsv_in/MIxS_6_term_updates_classdefs.tsv should appear inschemasheets/tsv_in/MIxS_6_term_updates_slot_assignments_and_usages.tsv
[x] Every term/slot mentioned in schemasheets/tsv_in/MIxS_6_term_updates_slot_assignments_and_usages.tsv should appear in schemasheets/tsv_in/MIxS_6_term_updates_global_partial_slotdefs.tsv
[x] Every term/slot mentioned in schemasheets/tsv_in/MIxS_6_term_updates_classdefs.tsv should appear inschemasheets/tsv_in/MIxS_6_term_updates_global_partial_slotdefs.tsv
[ ] we should identify the root cause of the multiple term definitions in MIxS6 packages - Final_clean

One way to access these reports is though a web API with a Swagger interface. It's IP address isn't stable right now, but can be obtained from Current NMDC utilities API address Google Doc

[] we should identify the root cause of the multiple term definitions in MIxS6 packages - Final_clean

I'm almost certain it is because we made this sheet manually. Unless @only1chunts disagrees, I think we can check that box.

Assigning myself to complete the task of making sure each term or class has only one definition. I'll seek help as needed.

We need script(s) that can iteratively document the progress of converting the MIxS Google sheets into schemasheets TSVs, with no duplications or contradictions (except in the case of slot usages)

My understanding from our last call was that I would edit the TSV sheets so that we don't need to run the google sheet to TSV conversion ever again, right? However, I think this issue is about writing scripts to confirm that my manual edits to the TSVs accurately reflect (with corrections) the content of the google sheets. Am I correct in that understanding?

I agree with everything Ramona has added here, i.e.

definitions were a mistake caused by manual editing of sheets
moving forwards TSV sheets should be being used for editing, not G-Docs

Some form of documentation of changes being made between the released v6 and the new v7 (or 6.x) needs to be maintained, whether that is via scripts or manually done I dont mind. (As its a one-off process its probably easier to just be a manual thing maybe?). For future versions, it would be nice to have scripts that compare new to old to give a list of the changes, but presumably thats much easier when both versions are in the same format!

Not a concern anymore after transition to MIxS v6.2.


Shape drawing problem

Circle border

common.Circle{ 	BorderWidth: 1, 	BorderColor: color.Black, }
The border drawing is abnormal. Fill color cannot be transparent. The same goes for running demos/shapes.
Circle Arc
common.Circle{ 	Arc: 180 }

Text
When there are Chinese characters in the string.
index out of range ... common.(*textShader).generateBufferContent

Just been looking through the files for this and I think the issue is coming from lines 273-305 of engo/common/render_shaders_shapes.go
Looks like the circle is being drawn full size in the right place for the border, then drawn again but offset left and up by border width, rather than in the same position but 2*borderwidth smaller.

I found the offset by putting it in the same position as a rectangle of the same width/height to see which part of the circle was wrong.

Just managed to fix the circle issue.

Transparency isn't working with borders, it hides the main layer but the border layers still fills that space too. The border code needs tweaking to only occupy the area around the rest of the shape.

Cool beans. The issue with the other two is a bit more challenging sadly. One problem is the number of triangles I'm making the circles out of is 300, which doesn't divide by 180 nicely, so the problem is you'll never get nice flat stuff like that this way. I don't like that, so I think upping the triangles to 360 would be a good idea. That way every whole angle gets a triangle.
Drawing a transparent circle isn't going to be possible the way I've done it. I'm actually drawing two circles, the border and then the normal circle on top of it. If you make the normal circle transparent, you'll see the border circle underneath! So we're  going to have to either decide transparent circles with borders aren't supported, or draw them using another method.

Okay didn't realise you were using triangles, to be honest, interesting! But yeah I was seeing the issue with the two circles... shame it won't create nice edges, because it would be great if we could just get it to draw between the edge and edge-borderwidth rather than edge and center.


Add unit test for getAllDaos

Hi, i wrote unit test for getAllDaos in RelationDaoFactoryTest, it's related for #4 issue

Looks good, thank you for your contribution.


Indentation "bug" at "Lightweight Charts/Dataviz"

Something happened where all the following sections are indented as children under this section

thanks!


"Load my notes" loads artifact characters and three blank titled notes

IO::openJournalFromFile() loads a created note with the prefix "T" in the title. It also loads a "D" character inside the first blank note.
Content of journal.uj:
##T test
##B 23
##D

@vhexs Fixing

Fixed in commit #15ef17a, closing issue


Change MOST Lordaeron building

https://www.hiveworkshop.com/threads/lordaeron-buildings.341580/

the buildings i DO NOT WANT changed are: The farms, and the altar

CLARIFICATION

Onlychange: main buildings, the marketplace, and the magic caster building

actually, change the grpyhon avairy and workshop too, they look nice

Done


Add transient map support for the gateway client

Hi Jim,  there's no need for this addition.  Sending transient data is already supported by specifying WithTransient() as an option on the contract.CreateTransaction() method.  See the example here: https://pkg.go.dev/github.com/hyperledger/fabric-sdk-go/pkg/gateway#WithTransient


AntiAffinity prevents containers from starting on single node

I am using minikube with only a single node. Because the statefulset has an anti-affinity rule, only the first container will start. Simply changing this from anti-affinity to affinity fixes the problem and allows all three containers to start.
      affinity:
#        podAntiAffinity:
        podAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            - labelSelector:
                matchExpressions:
                  - key: app
                    operator: In
                    values:
                      - consul
              topologyKey: kubernetes.io/hostname

Yes, This configuration is designed for a production like configuration. I'll consider adding a note to the tutorial.


Variant layers on component output

Turn of variant layers on component output by deafualt to get correct functionality

Closed by #130


make controls programmatically controllable

probably one of the most requested features for controls, for instance orbit:
orbit.pan(x, y)
orbit.zoom(level)
orbit.rotate(delta, phi)
currently controls are sealed and can only be controlled by pointer input, which also creates a problem bc once you have controls, nothing else is allowed to move the camera bc it would cancel movement out.

This is something I've been trying to tinker with, but I haven't figured out how to make play nice with damping.

at work we had this once, damping and all. i remember that i simply copied the code thats in the generic rotate/pan/zoom functions and made it generic. it was using the same damping settings.

I see that there was already a PR that you commented on that somewhat did this.
I suspected that we were to rip out the math from the update method, but is that still the case? I ask this because the following logic still has a dependency on the update method, and I'm not sure if that's the intention.
this.pan = function ( deltaX, deltaY ) {
  pan( deltaX, deltaY );
  scope.update();
};

here are the two i made years ago: https://gist.github.com/drcmda/6be8a22b824ce29922509c0483f0f1b3
the controls are outdated for sure and as i wrote in that issue, im not a mathematician - there may be better ways. looking at it it's mostly happening in _automate and i didn't understand the concept of lerping back then, it could be easier

This is now merged via #65 I think

why was this closed? I'm somewhat new to threejs, but how does #65 implement programmatic panning for example? is there a way to do this with just the azimuthal and polar angles?

panning is not implemented just polar and azimuth. you can add it though as another pr, it would make sense to have it.


Remove M2Crypto dependency

We had some trouble with SWIG in our production environment (heroku).
I started wondering if it'd be easier to replace python-saml's use of M2Crypto with another (non-swig) crypto implementation.
After a little looking, I was unable to find anything in need of replacing. I asked someone more familiar with crypto APIs, and they couldn't find where python-saml was using M2Crypto either.
I removed M2Crypto from install_requires. We've now got a working SAML flow with no apparent error.
Would you be willing to release a new version without this dependency?

Coverage remained the same at 97.9% when pulling 8b98a272fc26c64102f1b22d8bffec29f519e8f2 on mobilepaks:without_m2 into c44971e331c0acb3d9b0d74b3ff8c1fffb762012 on onelogin:master.

@keturn , take a look on this version of python-saml compatible with python3
https://github.com/pitbulk/python3-saml
We plan to add this to the official OneLogin reposiroty soon.


add support for Alibaba Cloud KMS Secret Manager

with current version only the followings are supported:

'ACSCurrenit' for version stage is the default value and recommented.
AccessKeyId + AccessKeySecret or STS (assume a provided role name using the provided AccessKeyId and AccessKeySecret). ECS_RAM_ROLE and Kube2Ram are not supported yet.
secret value as plaintext - parsing secret value as a json object not yet support

I have tested locally and in Alibaba Cloud K8s cluster as well. Please review and merge

@silasbw Could you please kindly review the change.

ready for review @Flydiverny


Update contributing guide for newcomers

Summary

This PR updates the contributor documentation to make it more clear what to do step by step, and utilize existing tools.

I think this is stale now, sorry we lost track of it.


Atom Crash Problem (Segmentation fault)

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

all command give me segmentation fault :
atom --version (display nothing without error)
atom --safe
atom --clear-window-state
this works well :

apm links
/home/hhlp/.atom/dev/packages (0)
└── (no links)
/home/hhlp/.atom/packages (0)
└── (no links)

in /home/hhlp/.atom/packages I have 37 packages
I delete config file to start fresh and nothing happenned same behaviour
Description
I have two problem :
1.- Segmentation fault

/usr/bin/atom: línea 157: 13874 Segmentation fault      (`core' generado) nohup "$ATOM_PATH" --executed-from="$(pwd)" --pid=$$ "$@" > "$ATOM_HOME/nohup.out" 2>&1

2.- when install from flathub doesn't find .atom directory where my configuration are store (works well but without any configuration)
[Description of the issue]
Steps to Reproduce

Install
Execute
Crash

Expected behavior: [What you expect to happen]
work as usual
Actual behavior: [What actually happens]
problem in startup
Reproduces how often: [What percentage of the time does it reproduce?]
always
Versions
You can get this information from copy and pasting the output of atom --version and apm --version from the command line. Also, please include the OS and what version of the OS you're running.
I don't have problem in Fedora 28 now in Fedora 29 I've got this -> 64 bits
Linux localhost-localdomain 4.18.14-300.fc29.x86_64 #1 SMP Mon Oct 15 13:13:16 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

atom --version (doesn't work)

using the .rpm package from atom.io

rpm -qa atom
atom-1.31.2-0.1.x86_64

apm --version (works well)
apm  2.1.1
npm  6.2.0
node 8.9.3 x64
atom 1.31.2
python 2.7.15
git 2.19.1

Additional Information
Check if your GPU is causing the problem
atom --disable-gpu
atom --enable-gpu-rasterization --force-gpu-rasterization
atom --safe --enable-gpu-rasterization --force-gpu-rasterization

/usr/bin/atom: línea 157: 15421 Segmentation fault      (`core' generado) nohup "$ATOM_PATH" --executed-from="$(pwd)" --pid=$$ "$@" > "$ATOM_HOME/nohup.out" 2>&1

Using ubuntu 18.10, I got the same segfault when attempting to launch atom. using the .deb package from atom.io

Thanks for the report!  As @solsticedhiver mentioned, this was an issue with the version of glibc.  The problem was resolved in https://github.com/atom/atom/pull/18012 and the fix is available in 1.32, currently in beta.

2.- when install from flathub doesn't find .atom directory where my configuration are store (works well but without any configuration)

The only official installation are documented in the Flight Manual.  If there's a problem installing from Flaathub, you would need to reach out to them.
Going to go ahead and close but if you still have a problem in the beta @hhlp, please let us know.

Just wanted to mention that 1.32.0 stable has been released :v:


Orca (1.2.1, release) seems to ignore tickmode = array, and tickvals

If I use the attached plotly JSON and feed it to Orca, I would expect "tickmode" and "tickvals" to get applied and have x axis labels at 82M, 100M, 200M, etc.
But I get the default format ("100M" plus small single digits at the subtick positions) instead.
Feeding the same to the same Plotly JS code in a normal browser produces the expected output.
plotly-orca-failed-example-tickvals.zip

This also applies to online plotly.js.

@jensb Can you post an image of the "normal" expected output you got in a browser. Over here I get the same result in Orca and Chromium v69.
Codepen: https://codepen.io/antoinerg/pen/abbNMxj

plotly-data-anon.zip
Sorry uploading image directly doesn't seem to work.
This is what I get when I patch Plotly so that tickvals and tickmode are respected correctly. both in Browser and in Orca.


initial draft of a kebab pattern and a card pattern

Addresses Kebab #22
Summary of changes:

updated dropdown pattern to include parameters for:

displaying dropdown as a kebab
displaying dropdown button on the right

included two variations of a card pattern

one with separate components for the card header and the card footer. This allows us to build card variations that use other components as the card contents
one single component that includes parameters for included specific elements of the card. Card contents would have to be specified as html (e.g. not reusing other components

@andresgalante @srambach -- Let me know if you have any suggestions. I realize there is a lot going on in this one, but there's a lot I'd like to get your feedback on.
I still want to include different variations of the card as currently represented on the patternfly site, but I'm not sure what direction to take the card (as a split out component where we use a card top component, other componenets for the card contents, and a card bottom component, or not). See my comments about this above.
I also wanted you to see the kebab in the context of the card, and not just by itself.
Some questions I have are:

Which variation of the card component should I use?
What layouts of cards should I include?
Where should styles for the kebab go? I included the same css in two different files, one for icons and one for dropdowns (i'm not sure where to add the dropdown scss file though). I will remove the updates from the one they don't belong in.

Thanks!

@jgiardino A challenge we are having with the kebab on pf3 is the tiny click zone. We should fix it in pf4

Oh, yes. Thank you for reminding me!

I made changes based on our discussion this morning about using partial blocks to include additional card elements. I didn't include an example of a card using other components as contents yet, but the partial block is defined as part of the card.
I also created separate files showing different variations of the card. Let me know if there are other variations I should include or any variation I should remove.
Some questions/issues that still need to be addressed:

Kebab styling - Kebab styling still needs to be updated to enlarge the clickable area (or is this something that would be covered by Sarah's implementation of the icon button?)
checkbox styling -

my biggest issue with this is how to make the clickable area larger. I made the width and height bigger which is clickable, but there's no way to position the checkbox within this area. I'm wondering if we should include the checkbox inside an empty label that's much larger than the checkbox
I included classes  .pf-card--multiselect__block and .pf-card--multiselect__header to define padding inside these elements when checkboxes are included, so that the contents in these elements do not overlap the checkbox. But I feel like this is a case where coding guidelines clash with ease of implementation. For example, with this implementation, we're telling developers "If you want multiselection, then add this class here, and that class there" when really as a developer it's easier to just add a class to the parent element, then add the checkbox

Headers - I noticed the styles for the headers changed (I think the line height was reduced). Currently the styling of headers relative to the card seems off, but I'm not sure if the header styles are final or not, and also am curious what your thoughts are.

@andres, @srambach
I made updates to the PR to remove card variations that use the checkbox and the kebab, until those patterns are finalized.
I also included a utility class to the icon to add margin-bottom.
@andresgalante please check the css file for components and make sure the updates are what you expected.
Thanks!

Lets merge it, I've created a story to continue with it: https://patternfly.atlassian.net/browse/PTNFLY-1961
thanks @jgiardino you are awesome 😄


adding 'items' object to mock_subscription

Improving 'subscription' mocked object

please, add a confirmation test

there you go @alexmamonchik

thank you very much!


feat: Observable.defer

This moves the implementation to BaseObservable. We don't need extra error checking here since we already handle errors in the subscriber function.
Note as of now we cannot defer a function returning IClassicObservable. While it feels a little icky it may make sense in this library for Observable.from to assume a non-symbolObservable, non-iterable argument with a subscribe method to be an IClassicObservable.

LGTM


Enable human readable timestamp in dmesg.

From dmesg(1):
-T, --ctime
          Print human-readable timestamps.

          Be aware that the timestamp could be inaccurate!  The time
          source used for the logs is not updated after system
          SUSPEND/RESUME.

DCOS_OSS-1020 Make dmesg output timestamps in diagnostic bundle human readable.

@janisz The summary of the DC/OS JIRA ticket you reference in the description doesn't match the name in JIRA. It should be "Make dmesg output timestamps in diagnostic bundle human readable"

It looks like this is failing the integration tests: test_dcos_diagnostics.test_dcos_diagnostics_bundle_download_and_extract:
https://github.com/dcos/dcos/blob/de1cc8abc03eddea81c2795c06cdfcee9c7d235d/packages/dcos-integration-test/extra/test_dcos_diagnostics.py#L481
It may just mean that a change to the test is required (since the output changed, you may need to update the test to account for timestamps now being present in the output)

Actually, I triaged this a bit, and it looks like the filename generated has changed from '<IP_ADDRESS>_master/dmesg-0.output.gz' to '<IP_ADDRESS>_master/dmesg_-T-0.output.gz' because of the flag you added. Maybe we should chat tomorrow to make sure this makes sense or not and how to adjust for it.

Just verified this is expected by looking at other files generated from commands in the config, and I think this name change is OK. You just need to update the expected filename in the integration test.

@surdy Are you OK with a name change in the diagnostics bundle from '<IP_ADDRESS>_master/dmesg-0.output.gz' to '<IP_ADDRESS>_master/dmesg_-T-0.output.gz in order to support adding timestamps to the dmesg output?

Closing as a duplicate of #3148


transparent window outline

having a transparent window outline will still allow to resize windows wile making it look a bit more slick and modern.
Also the outline should be set to 0 in fullscreen mode to avoid showing gaps.

Of the top of my head, one way to implement this would be to have JWM set the border of a window to be transparent and the child window (the contents) to be opaque.  JWM currently supports transparency through a compositing manager such as xcompmgr, etc., but it should be possible to get this effect if such an application were used with JWM with fairly little effort.
The hardest part would deciding what the configuration would look like.

maybe something like this

< Width>2
< Outline>transparent or < Outline>clear or
< Opacity>outline:0.0, window:1.0

Iirc X11 has some way to copy color from parent, so translucent windows and effects should require a compmgr, but transparent border/trays/etc... shouldn't necessarily.
Adding a "None" color as in xpm, would allow tray buttons to resemble desktop icons or even desktop widgets now that mouse buttons are configurable.

Unfortunately, as far as I know, there is no way to do real transparency without compositing.  That's why some terminal programs (like aterm) show the background when transparency is enabled.  Of course, even doing that requires a property to be set on the root window.  For a tray, that might not be so bad since there is likely nothing behind it.  I remember seeing this trick used a lot in AfterStep.  Of course, maybe that's good enough?

@technosaurus  I have noticed xpm with None doesn't always work as intended.  With the Active TaskList the none is set to the background of the Tray rather than the ActiveTasklist... at least in an FLTK program I wrote that sets the icon via X11/xpm.h
Though, now that I think about it, this could be a JWM TaskList bug, perhaps...

@joewing i am using compton with jwm works very well.


Fix 366

Linked Issue(s)
Closes #366
Acceptance Criteria fulfillment

[ ] Task 1
[ ] Task 2
[ ] Task 3

Proposed changes (including videos or screenshots)

Further comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

There is a PullRequestsTableMini as well, do I need to change that as well?

@e-for-eshaan  Can you please look into these front-end changes ?

Hey @gajrajgchouhan, thanks for this PR.
The DeploymentInsightsOverlay (the one that opens when you click on the Deployment Frequency card) utilises PullRequestsTableMini. You should remove the redundant copy from there as well.

Hi @e-for-eshaan , I have done this.


Properly respect markdown in select choices, refactor

See 186542
Also removed a function we weren't using
cross https://github.com/dimagi/javarosa/pull/194

"Pull requests that have a failing status can’t be merged from a phone."


add sqlmodel support

I think it plays out fairly well with fastapi, given it's, well, the same author :)
Example tutorial: https://testdriven.io/blog/fastapi-sqlmodel/
I'm happy to help if you are up for it.

@adamjanas
@hkorzeniewski
@LukaszS0110
@maksmatch
@oleg2001-1
@piotek8
Nice first Open Source issue, first come first served ;)


Better error message and sanity check for signed URL generation.

Tests are failing because the firecloud library is broken on python3.10: https://github.com/DataBiosphere/terra-notebook-utils/issues/390

@mbaumann-broad Changes made.

@mbaumann-broad Thanks for the reviews!


multiple-outputs.sh: Install static libraries into $static->$dev->$out

Description of changes
During preFixup, static libraries (lib/*.a) are now installed to the $static output if it exists, or the $dev output if that exists, or just to $out as usual.
This will make it easier to prevent static libs making it into system closures.
Related issues:
https://github.com/NixOS/nixpkgs/issues/164141
https://github.com/NixOS/nixpkgs/issues/224533
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

I'm trying to figure out why /nix/store/...x86_64-unknown-linux-musl-stage-static-gcc-12.2.0.drv fails to build when I do nix-build ./lib/tests/release.nix
Part of my debugging was to try building these derivations on the staging branch (which I'm based on) and then diffing the results to see what's different.
But when I instantiate the tests on the staging branch, gcc-12.2.0 isn't even part of the closure. Why would that even happen?
Steps to repro:
[rynn@rynn:~/sources/luabee-nixpkgs]$ git checkout staging
Updating files: 100% (37520/37520), done.
Switched to branch 'staging'
Your branch is up to date with 'origin/staging'.

[rynn@rynn:~/sources/luabee-nixpkgs]$ git pull
Already up to date.

[rynn@rynn:~/sources/luabee-nixpkgs]$ nix show-derivation -r `nix-instantiate ./lib/tests/release.nix` | grep gcc-12.2.0.drv
warning: you did not specify '--add-root'; the result might be removed by the garbage collector

[rynn@rynn:~/sources/luabee-nixpkgs]$ git checkout luiabee-statics
Updating files: 100% (37520/37520), done.
Switched to branch 'luiabee-statics'
Your branch is up to date with 'origin/luiabee-statics'.

[rynn@rynn:~/sources/luabee-nixpkgs]$ git merge staging
Already up to date.

[rynn@rynn:~/sources/luabee-nixpkgs]$ nix show-derivation -r `nix-instantiate ./lib/tests/release.nix` | grep gcc-12.2.0.drv
warning: you did not specify '--add-root'; the result might be removed by the garbage collector
      "/nix/store/qixsm0r3rrpi1hjj5i381vwm0b9wp4hj-gcc-12.2.0.drv": [
      "/nix/store/ral4xq9i42473a8vwm51xci6gf13rlg3-x86_64-unknown-linux-musl-stage-final-gcc-12.2.0.drv": [
      ...

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/cuda-team-roadmap-and-call-for-sponsors/29495/1

Hi @Luabee -- the CUDA maintainer team sees this as a very useful feature for automatically splitting out static libraries from CUDA redistributable. A number of the redistributables we use include static and dynamic libraries and approach a gigabyte in size. Being able to split out static copies of the library would allow us to shave hundreds of megabytes!
What are your thoughts on the status of this PR? Do you see additional places for testing or improvement? I'd love to help if I can!

Hi @Luabee -- the CUDA maintainer team sees this as a very useful feature for automatically splitting out static libraries from CUDA redistributable. A number of the redistributables we use include static and dynamic libraries and approach a gigabyte in size. Being able to split out static copies of the library would allow us to shave hundreds of megabytes!
What are your thoughts on the status of this PR? Do you see additional places for testing or improvement? I'd love to help if I can!

At this point all the functional changes are in place. I'm just blocked on a confusing build error in an instance of gcc which I'm not familiar with. x86_64-unknown-linux-musl-stage-final-gcc-12.2.0.drv Is failing to build with a strange error related to libatomic? See here:

I believe this is a highly desirable change. I have some nix-built container images that require JAX, TensorFlow and Torch all in one image. The increased size from the cuda libraries doesn't just add to the image size, but in my case more painfully to the single-threaded compression time for the images/layers.
This PR would not only reduce the size of these kinds of images size by several gigabytes, but also cut down on compression time.
One small issue I could imagine popping up after this is that some compilers require certain static archives in their default configuration to "run". So we might end up with some cuda/sycl/hip/c++ compiler packages no longer working out of the box. Fixing such issues (if they even occur at all) seems rather simple though since in those rare cases the relevant archives can just be added back to the package explicitly.

I won't be able to work on this at least through the rest of the year, so someone else would need to take it up if they want it merged


Remove celery ping health check due to unreliability

🛠 Changelog

See title.

🧪 Test plan

Automated tests

🚧 Dependencies

Inform Platta that the health-check can be re-enabled after this hits production

🎫 Tickets

TILA-3702

/ff


Integrate more informations (events, file count, ...)

I think there are a more informations, which could be usefull:

count of the classes loaded (or files)
session / auth
list of the fired events

their order
who catched the event

Hi guys! I wrote module for that! https://github.com/snapshotpl/ZfSnapEventDebugger

All included now....thanks to @moderndeveloperllc
See https://github.com/zendframework/ZendDeveloperTools/pull/142


Fix corporatedomains whois date format with slashes

Some of the responses return the creation/expiration date in a format that is parsed incorrectly through parse_time. This occurs specifically for corporatedomains in some instances where the date format is returned as "MM/DD/YYYY", but is interpreted as "DD/MM/YYYY" so these dates need to be parsed separately.

@njng can you explain how you generated the fixture with the dates with slashes? I performed a live whois query, but what I see is:
Domain Name: corporatedomains.com
Registry Domain ID: 2546326_DOMAIN_COM-VRSN
Registrar WHOIS Server: whois.corporatedomains.com
Registrar URL: www.cscprotectsbrands.com
Updated Date: 2018-10-11T05:49:21Z
Creation Date: 1997-10-16T04:00:00Z
Registrar Registration Expiration Date: 2019-10-15T04:00:00Z
Registrar: CSC CORPORATE DOMAINS, INC.
Registrar IANA ID: 299
Registrar Abuse Contact Email<EMAIL_ADDRESS>Registrar Abuse Contact Phone: +1.8887802723
Domain Status: clientTransferProhibited http://www.icann.org/epp#clientTransferProhibited
Domain Status: serverDeleteProhibited http://www.icann.org/epp#serverDeleteProhibited
Domain Status: serverTransferProhibited http://www.icann.org/epp#serverTransferProhibited
Domain Status: serverUpdateProhibited http://www.icann.org/epp#serverUpdateProhibited
Registry Registrant ID:
Registrant Name: Domain Administrator
Registrant Organization: CSC Corporate Domains, Inc.
Registrant Street: 251 Little Falls Drive
Registrant City: Wilmington
Registrant State/Province: DE
Registrant Postal Code: 19808
Registrant Country: US
Registrant Phone: +1.3026365400
Registrant Phone Ext:
Registrant Fax: +1.3026365454
Registrant Fax Ext:
Registrant Email<EMAIL_ADDRESS>Registry Admin ID:
Admin Name: Domain Administrator
Admin Organization: CSC Corporate Domains, Inc.
Admin Street: 251 Little Falls Drive
Admin City: Wilmington
Admin State/Province: DE
Admin Postal Code: 19808
Admin Country: US
Admin Phone: +1.3026365400
Admin Phone Ext:
Admin Fax: +1.3026365454
Admin Fax Ext:
Admin Email<EMAIL_ADDRESS>Registry Tech ID:
Tech Name: DNS Administrator
Tech Organization: CSC Corporate Domains, Inc.
Tech Street: 251 Little Falls Drive
Tech City: Wilmington
Tech State/Province: DE
Tech Postal Code: 19808
Tech Country: US
Tech Phone: +1.3026365400
Tech Phone Ext:
Tech Fax: +1.3026365454
Tech Fax Ext:
Tech Email<EMAIL_ADDRESS>Name Server: pdns1.cscdns.net
Name Server: pdns2.cscdns.net
DNSSEC: unsigned
URL of the ICANN WHOIS Data Problem Reporting System: http://wdprs.internic.net/
>>> Last update of WHOIS database: 2018-10-11T05:49:21Z <<<

For more information on Whois status codes, please visit https://icann.org/epp

Corporation Service Company(c) (CSC)  The Trusted Partner of More than 50% of the 100 Best Global Brands.

Contact us to learn more about our enterprise solutions for Global Domain Name Registration and Management, Trademark Research and Watching, Brand, Logo and Auction Monitoring, as well SSL Certificate Services and DNS Hosting.

NOTICE: You are not authorized to access or query our WHOIS database through the use of high-volume, automated, electronic processes or for the purpose or purposes of using the data in any manner that violates these terms of use. The Data in the CSC WHOIS database is provided by CSC for information purposes only, and to assist persons in obtaining information about or related to a domain name registration record. CSC does not guarantee its accuracy. By submitting a WHOIS query, you agree to abide by the following terms of use: you agree that you may use this Data only for lawful purposes and that under no circumstances will you use this Data to: (1) allow, enable, or otherwise support the transmission of mass unsolicited, commercial advertising or solicitations via direct mail, e-mail, telephone, or facsimile; or (2) enable high volume, automated, electronic processes that apply to CSC (or its computer systems). CSC reserves the right to terminate your access to the WHOIS database in its sole discretion for any violations by you of these terms of use. CSC reserves the right to modify these terms at any time.

Register your domain name at http://www.cscglobal.com

I can't find any domain that returns dates in MM/DD/YYYY


Instrument module:  Just store camera geometries by camera id, not telescope id

This will drastically reduce time and memory usage.
Give the telescope table a column camera id and then just create tables for each camera, not for each telescope.

I don't think so: only in the MC the cameras are identical. In reality, each camera will be different.
For the MC production, already now some of the cameras are different (as e.g. the quantum efficiency is the base value + a random value calculated using the expected uncertainty)

This bring up a point, however - the reason we break down the instrument description into multiple sub-classes/tables is for exactly this use case.
The CameraGeometry is probably fixed for all cameras (at least for a single camera version and type), but other parts of the instrument model change (like the QE, etc) and those should be in a different object/table.  That was the point of the ObsConfig data model : separate things that change from things that don't change often, and group things that change roughly at the same time, so they can be identified by a unique key.
Max is correct: we should be able to load a single CameraGeometryDescription per type, but still load a different CameraDetectorDescription (or whatever you want to call it) that contains the QE's, etc per camera.  In an event display you only need the fixed geometry, but in the MC you need all the info.
That implies that you need to break up the CameraDescription into at least these two pieces

Yes, I just meant Geometry not values that might be different for each camera.


Revert "Use go 1.16"

Reverts offen/offen#559
This seems to break on non-Linux builds: https://app.circleci.com/pipelines/github/offen/offen/3175/workflows/3b51f00d-b045-436e-ab7d-0a4d67532b89/jobs/23392/parallel-runs/0/steps/0-104

Upstream issue: https://github.com/techknowlogick/xgo/issues/100


Add a note about what a task is to the readme

When I first read the readme I didn't understand the concept of a "task".

Much appreciated.


Upgrade dependencies

Coverage decreased (-0.03%) to 83.759% when pulling cae233d0840fc3ad1e2d3a2da1810c7e07a9265b on upgrade-dependencies into d10c4e4fdde2e345d2694ce430e61345f8c34db0 on master.


fix typo

in regular English it would not be capitalised
(unless it is some special stylistic tool?)

Oh yeah thanks, it would be capitalized in German, that's probably why I spelled it wrong


[Bug]: Option for preemption_mode uses underscore (_) instead of dash (-)

Your current environment
Irrelevant, the error is easily seen in the code. It is seen in version 0.5.0.post1, in the file vllm.engine.arg_utils:576
🐛 Describe the bug
In the latest release (V0.5.0.post1), the CLI argument for preemption_mode parameter of EngineArgs is --preemption_mode which goes against the typical convention of replacing _ with - in command-line parameters. This won't affect any other code if fixed to --preemption-mode, because argparse converts any dashes to underscores in the namespace returned by parser.parse_args(). However the current situation causes problems when I want to convert an EngineArgs instance to set of command-line flags (say in order to launch a vllm server programmatically via subprocess)

Thanks for reporting this @maharjun! I've posted a fix here https://github.com/vllm-project/vllm/pull/5688


Lazy load plotting libraries

Pull request
Please confirm that this pull request has done the following:

[x] Tests added
[x] Documentation added (where applicable)
[x] Example added (either to an existing notebook or as a new notebook, where applicable)
[x] Description in CHANGELOG.rst added

@znicholls Do you have any strong feelings against lazy importing the plotting libraries? They add >1s of import time which is very obvious when running small python scripts without any plotting. This in combination with #208 reduces scmdata's import time to ~300ms down from >2s.
This pattern is used in other libraries to avoid always importing matplotlib.pyplot (https://github.com/pydata/xarray/blob/7379923de756a2bcc59044d548f8ab7a68b91d4e/xarray/plot/dataarray_plot.py#L979).
FYI: There is an interesting pattern to allow importing units for type checking only by optionally importing if  typing.TYPE_CHECKING is True https://github.com/pydata/xarray/blob/7379923de756a2bcc59044d548f8ab7a68b91d4e/xarray/plot/dataarray_plot.py#L979

Looks good.
Yep type checking stuff is a good pattern I often forget about too

Cool I'll clean this up

@znicholls No major changes on clean up so I'll merge this and do a minor release unless I hear a scream

Waiting for https://github.com/hgrecco/pint-pandas/issues/145 to be resolved before a release

Waiting for https://github.com/hgrecco/pint-pandas/issues/145 to be resolved before a release

Can you tell what pint has changed and why everything has now exploded?

Can you tell what pint has changed and why everything has now exploded?

From a look at the changelog there was a refactor into "Facets" (https://github.com/hgrecco/pint/issues/1466). I haven't dived into this. It broke something in primap related to its unit registry so we might have to check openscm_units (https://github.com/hgrecco/pint/issues/1631)

MIght be better to pin to <0.2 for the next release and then loosen once we do some testing

Also works
On Wed, Oct 26, 2022 at 12:02 PM Jared Lewis @.***>
wrote:

MIght be better to pin to <0.2 for the next release and then loosen once
we do some testing
—
Reply to this email directly, view it on GitHub
https://github.com/openscm/scmdata/pull/209#issuecomment-1291302096, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFUH5G2A6TKNWEGY2ZZJVYLWFB7I7ANCNFSM6AAAAAARMWMUWY
.
You are receiving this because you were mentioned.Message ID:
@.***>


chore: Add dropbear dependency for the ssh connection

I am going to remove the dropbear package from the base TDX image by default and therefore since this is needed for the bob use-case, I am adding it here instead.
dropbear is not always necessary for each tdx based image and that's why we are removing it from the default builds

lgtm!


Fix AkairoHandler#add

Currently, when calling readdirRecursive, it returns an array including the full filepath of each file. Then, AkairoHandler attempts to compare the file name included (ex. Eval.js) to the full file path (ex. C:\\Users\\Owner\\Desktop\\Development\\Slice\\src\\Commands\\Owner\\Eval.js). This means it will never find the file called Eval.js unless we were to include the full path or compare it to the basename of the file, in which this PR does.
Example eval to test.
sa$eval const path = require('path');
const files = this.handler.constructor.readdirRecursive(this.handler.directory);
files.find(f => {
  return path.parse(f).base == 'Eval.js';
});

path.basename

@appellation Actually, no, .base includes the file extension aswell which is what we're comparing.

@FireController1847 actually, no.  Check out https://nodejs.org/api/path.html#path_path_basename_path_ext

Ah, I see your point.


(temporary) login captcha (via commands)

How about some login captcha that could be temporarily enabled when users do some mass join via VPN/Tor and spam via login messages or chat?
Basic idea: After sending the login form data the websocket server could answer with some link/token to a captcha, which has to be solved before the actual join.

No sorry, too complicated, we'll just have to deal with it. Blocking Tor is an option, but many people wouldn't like that.

What kind of excuse is "too complicated"?

Sure that might not help hack.chat gain any popularity awards, but I don't think it was ever going to. There's so few of us who truly appreciate it.

The codebase is meant to be easy to modify and install for people that want to hack onto it. A few forks have been made for this reason. It has a few dozen other installations I'm aware because users like the simplicity with minimal dependencies. Feel free to fork it though and host your own. You could take off with it.

There could be a difference between Hack.Chat you are hosting and Hack.Chat on github

yep you could hack onto it yourself


Addition of opening applications via keyboard shortcuts

Description of the new feature / enhancement
Add the ability to input a keyboard shortcut and that open a specific application on the host machine. This could be an addition to the Keyboard Manager application or a new application all together. There would have to be a check so that remapping key Windows shortcuts like Win + L cannot be changed which I think the current version of keyboard manager does.
Scenario when this would be used?
It is already being used to open the PowerToys Run application with Alt + Space. For power users who do not use the mouse all that often, being able to press Win + Enter to open the Terminal or Alt + B to open default browser would make using the system even faster.
Supporting information
Please see the Linux Tiling Window Manager community where this is the primary way of interacting with the system. A standalone system for this is sxhkd.
Currently the only option to do this is to use an external scripting software Auto Hotkey but this feels like an addition to the PowerToys suite.

/dup #5668


Makes "Create shader node" dialog non-exclusive (visual shaders)

This was changed for all dialogs by @reduz in https://github.com/godotengine/godot/pull/39993, but I think the old behavior from 3.2 is better for this particular dialog. This just allows it to close up when the user clicks outside the dialog which makes it more comfortable to use.

This is great. I was trying your other pr and I had to close the dialog many times.


Make buffered console output available to testResultsProcessor

Do you want to request a feature or report a bug?
feature
What is the current behavior?
A testResultsProcessor is given access to a lot of meaningful data but one thing not yet available is the console output from each test case.
What is the expected behavior?
It would be very helpful to provide the buffered console output to test results processors like jest-junit to include console output for each <testcase> (see https://github.com/palmerj3/jest-junit/issues/45 for more details).

TestResult already have a console property.
However it seems that it is null unless jest us started with --verbose=false (see https://github.com/facebook/jest/issues/5232)

I believe this issue still represents a desired feature to implement.

Yeah, PR welcome here 🙂