Migrate Identity Server 3 to Service Fabric 4

Do we have any steps to migrate ASP.NET Identity server 3 to the ASP.NET Identity server 4 for DotNet Framework >4.6 application?

We don't have any thing specific. I'd suggest starting with our quickstarts, then from that knowledge you'd have to take what you have for IS3 and plan for yourself moving to IS4.

Thanks for the update Allen.
Do we have any quick start for Identity Server 4 with .Net Framework

Yes, consult the docs.

Hi Allen,
There are only blogs that says it supports in .Net Framework but there are no samples or reference to implementation in .Net Framework 4.6
We can't introduce .Net Core to our existing environment as there are some Nugets dependent on .Net Framework. So can you guide me how to implement Identity Server 4 in .Net Framework

So can you guide me how to implement Identity Server 4 in .Net Framework

IdentityServer is just a library that targets netstandard. Just build your ASP.NET Core app targeting .NET framework. Consult Microsoft with how to do that.


Activity is created after the app crashes

The activity is recorded when the app crashes. It is not a big of a deal but during testing you end up with several failed activities in your history that you then have to delete manually.
I wonder if the app can have a method that is called before panic.

I dug around a bit and this is the "normal" behavior of an Activity. If the app crashes the activity is automatically saved by the system. Looking around, beyond attempting try/catch on a couple of things to prevent exceptions from bubbling (which we should do, but it's not a major concern right now) there's no way to have a master error handler. Basically, if an activity is running and your app crashes it will be saved and you don't have control over that.

Closing. Normal behavior of an Activity. Thinking about it, it makes sense to have this activity recorded after a crash. I rather have the data than losing it.


ALMA-165: Add top warning bar message

Implements ALMA-165 - but this should not be merged as-is unless the relevant banner message needs to be deployed.
View for testing.
This PR adds CSS which displays a message in a warning bar at the top of every (real) page, like the home page, search results, and account pages.  It does not directly appear on Primo's "overlay" pages, like the detailed record display and the "more ..." page, though it may be seen in the background beneath those overlays.
Since there's no obvious way to toggle this on & off without changing the code directly, this PR should not be merged as-is unless the re-indexing causes disruption.  Though since merging & deployment are disconnected, I guess it doesn't matter, but we should decide how to handle this long-term.

@ztucker4 I've wrapped the new top warning bar CSS in a block comment to disable it, and am merging so it's available if/when we need it in the future.


CapitalReporttest failing

outputCapitalCitiesReportsN_MoreThanInListTest method in CapitalReportTests is failing with the following error message
outputCapitalCitiesReportsN_MoreThanInListTest on line 101

Closed - issue due to re-running code whilst intellji was still building

Was missing password in db/dockerfile.
Database now works, so you can continue looking into the other errors now generated for your capital city reports

There are still lots of errors on your branch for this, I'm going to restart the capital reports from scratch


[02/01/2023] Isolate backend API src code

Summary
Removed the frontend code and isolated only the backend code in backend_dev branch
List of changes proposed in this PR (pull-request)

Removed frontend code from current branch

@BatoolMM, will you be handling reviewing the PR before merge, or can we continue to handle it?

Hi Richi, I am happy to merge this PR but the backend_dev branch will (in the future) have to be reabased with the main and include files from the front-end (at some point) as we link both front-end and backend together. Do you have in mind a different way or how do you envision this?

ohh, i understand you now, i was thinking the frontend and backend code will remain in different branches forever
I'll revert the commit where I deleted the front end code then

I think it's okay to merge now
@BatoolMM, what do you think

Why is CONTRIBUTING.md is removed?

Why is CONTRIBUTING.md is removed?

I was of the mindset that the branch will strictly be for the backend API, and remember i already made a CONTRIBUTION guide for the backend API here #119


Introduce CustomInputs component

MaxMind allows you to submit values for "custom inputs"
that you have defined yourself. This introduces support
for that cool feature.

Coverage increased (+0.05%) to 98.347% when pulling 3ce6ca27d7215450c8ce1b3800057c56317bb859 on dan-jensen:master into 7cfe7aa9ef5d6d0d4825e8cb410cde287b67aefa on kushniryb:master.

Hey @kushniryb, great suggestions.  Just applied them all 👍


chore: release v0.3.13

:robot: I have created a release beep boop
0.3.13 (2024-04-07)
What's Changed

chore(checksum): log incremental checksums by @coffeebeats in https://github.com/coffeebeats/gdbuild/pull/125

Full Changelog: https://github.com/coffeebeats/gdbuild/compare/v0.3.12...v0.3.13

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/coffeebeats/gdbuild/releases/tag/v0.3.13 :sunflower:


Docker security flaw

Hello.
I was unable to run this app with any other user than root. This is a huge security flaw if this app is accessible from internet.
I think it should be pretty easy to fix and should involve only the dockerfile. If you don t have time to do it, may i look into it and propose a merge request later ? thanks.

Fixed in 1.2.8


Reference to 'this' within class showing as error after update on the dev channel.

I recently updated Flutter on the Dev channel and am now getting an error when referencing the class (as 'this') within the class (but not within the Class constructor).
Now when making references to this within my class displays as an error. This has happened for two classes that were previously showing no error before update.
Same behaviour occurs in Android Studio and VS Code.
The app compiles and runs fine, despite showing an error.

Flutter doctor output:
> The screen cannot be set to the number of lines and columns specified.
> [√] Flutter (Channel dev, v1.13.3, on The screen cannot be set to the number of lines and columns specified.
>
> Microsoft Windows [Version 10.0.18362.535], locale en-AU)
>     • Flutter version 1.13.3 at C:\flutter
>     • Framework revision 4af66e335f (21 hours ago), 2019-12-19 03:53:27 -0500
>     • Engine revision 0f90e6546b
>     • Dart version 2.8.0 (build 2.8.0-dev.0.0 aa6709974d)
>
> [!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
>     • Android SDK at C:\Users\fring\AppData\Local\Android\sdk
>     • Android NDK location not configured (optional; useful for native profiling support)
>     • Platform android-28, build-tools 28.0.3
>     • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
>     • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
>     X Android license status unknown.
>       Try re-installing or updating your Android SDK Manager.
>       See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.
>
> [√] Chrome - develop for the web
>     • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
>
> [√] Android Studio (version 3.5)
>     • Android Studio at C:\Program Files\Android\Android Studio
>     • Flutter plugin version 42.1.1
>     • Dart plugin version 191.8593
>     • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
>
> [√] IntelliJ IDEA Community Edition (version 2017.3)
>     • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2017.3.4
>     • Flutter plugin version 23.2.2
>     • Dart plugin version 173.4700
>
> [√] VS Code, 64-bit edition (version 1.40.2)
>     • VS Code at C:\Program Files\Microsoft VS Code
>     • Flutter extension version 3.7.1
>
> [√] Connected device (2 available)
>     • Chrome     • chrome     • web-javascript • Google Chrome 79.0.3945.88
>     • Web Server • web-server • web-javascript • Flutter Tools
>
> ! Doctor found issues in 1 category.
> Process finished with exit code 0

import 'dart:ui';

import 'package:flutter/material.dart';
import 'package:sescommandcentre/dialogs/addShift.dart';

class Person {

  Person(
      {Key key,
      this.userid,
      this.membernumber,
      this.firstname,
      this.lastname,
      this.phone,
      this.starttime,
      this.finishtime,
      this.statusId,
        this.statusDesc,
      this.groupId,
      this.groupDesc,
      this.unitId,
      this.unitDesc,
      this.userLevelId,
      this.userRoleDesc,
      this.username,
      this.password});

  int userid;
  String shiftid;
  String membernumber;
  String username;
  String password;
  String firstname;
  String lastname;
  String groupDesc;
  int groupId;
  String unitDesc;
  int unitId;
  String district;
  int starttime;
  int finishtime;
  int phone;
  int statusId;
  String statusDesc;
  int userLevelId;
  String userRoleDesc;
  Color color = Colors.yellow;

  Card userSearchCard(BuildContext context) {
    Card userCard = new Card(
      child: Row(
        children: <Widget>[
          Expanded(flex: 2, child: Text(membernumber)),
          Expanded(flex: 3, child: Text(firstname)),
          Expanded(flex: 3, child: Text(lastname)),
          Expanded(
              flex: 2, child: groupDesc != null ? Text(groupDesc) : Text('No group')),
          MaterialButton(
              color: Colors.blue,
              child: Text('Add Shift'),
              onPressed: () {
                var addShift = new AddShift(person: this); //// problem here
                addShift.displayDialog(context);
              }),
          Spacer(
            flex: 4,
          )
        ],
      ),
    );
    return userCard;
  }
}

If I pass a manual reference to the class object as a generic object when calling the class in does not show an error.

Recent flutter upgrade appears to have resolved this.


'@apocentre/bc-ur' in deps

I see @apocentre/bc-ur is in dependencies and its used in src/RegistryItem.ts.
And at the same time, this lib requires outside code to use @apocentre/bc-ur as well to first decode CBOR to pass it then to ur-registry.
This doesn't make much sense, this library should either fully handle CBOR stuff and relieve outside code from this burden OR allow outside code to do CBOR stuff on its own, thus reducing number of dependencies for this lib (zero deps is the target, as dependencies are always a risk in our line of work)

not to mention @apocentre/bc-ur is not very reputable, and has too many dependencies. risky.

Nice point. Already extended bc-ur in this lib.


CAB file checking can be simplified

When checking for CAB files that can potentially have multiple parts, if a CABNAME.HDR exists, that's the only one that needs to be scanned. Otherwise, only CABNAME1.CAB needs to be scanned. This can make things go a lot quicker if you have the flag enabled.

http://www.mediafire.com/file/eq80y20l9cwf48f/DiscImageCreator_test.7z/file

changed: 1st cab or hdr is only extracted

I'll admit I don't fully understand how your code change addresses this. I'll test it out in the latest release and see if it still occurs.


Autoprefixer does not work in Angular 7

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting

Command (mark with an x)
- [ ] new
- [x] build
- [x] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions
Angular CLI: 7.1.3
Node: 10.14.1
OS: win32 x64
Angular:
...

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.11.3
@angular-devkit/core         7.1.3
@angular-devkit/schematics   7.1.3
@schematics/angular          7.1.3
@schematics/update           0.11.3
rxjs                         6.3.3
typescript                   3.1.6

Repro steps

angular7Project.zip
Step1: install node_modules for the above provided project.
Step2: run ng serve
Step3: Open the browser devtools and focus the element with class "sample-div", which has display:grid as applied style.

Desired functionality

The .sample-div should also contain the prefixed style which is display: -ms-grid like in Angular 6.
angular6Project.zip

@alan-agius4 Where should I add
/* autoprefixer grid: on */ ? This is unfamiliar territory for me.

You can try to add this comment in your scss / css files. If you put this comment on the top, it will enable grid for all the file. If you set it in a declaration block, grid will only be available inside this block.


configure max_ejection_time in outlier detection check

While using outlier detection i found out the behaviour of baseEjectionTime , which can extend its value up to maxEjectionTime (which is 5 m default value). what if this 5 minutes value of ejectionTIme is not suitable for a use case .how can i custom configure this maxEjectionTime in destinatioRule  up to 1m?

Currently, there is no maxEjectionTime (Check #30602).
Also by checking the envoy documentation we can see that:

maxEjectionTime is defined as the maximum time that a host is ejected for and  If it is not specified, the default value (300000ms or 300s) or base_ejection_time value is applied, whatever is larger.

I hope this could help you

right @salehsedghpour . i also found that currently there is no such configuration .
Just curious , how this problem is being currently solved ( to lower maxEjectionTime value less than 5 m).

if you want to change maxEjectionTime, you have to do using an Envoy filter


NumCpp NdArray not initialized

The model seems to successfully load, but after the first user input it displays the following with an error related to NdArray:
file: NdArrayIterators.hpp
function: NdArrayConstIterator
Line: 70
Error: NdArray has not been initialized.
This is with the v11x model using CUDA 12 on Windows. It works when using the latest storygen.exe artifact but not the one generated under /examples/storygen/release after running /build.bat. I might just be doing something silly / missing a dependency.

Seems to be related to this issue with NumCpp. There were some commits since then, will try updating NumCpp.

Hi, @qquill ,
Is there any solution for this issue? Thanks.

Hi, @harrisonvanderbyl.
I encountered this problem with environment below when running the storygen.

Ubuntu 22
CUDA version 12.0, NVIDIA driver version 525.60.13
A Geforce GTX 1060 6GB gpu
A model.bin converted from https://huggingface.co/BlinkDL/rwkv-4-raven/blob/main/RWKV-4-Raven-1B5-v12-Eng98%25-Other2%25-20230520-ctx4096.pth

Hi, @harrisonvanderbyl. I encountered this problem with environment below when running the storygen.

Ubuntu 22
CUDA version 12.0, NVIDIA driver version 525.60.13
A Geforce GTX 1060 6GB gpu
A model.bin converted from https://huggingface.co/BlinkDL/rwkv-4-raven/blob/main/RWKV-4-Raven-1B5-v12-Eng98%25-Other2%25-20230520-ctx4096.pth

Hi, @harrisonvanderbyl , @mazhaojia123
In example/storgen, the bash build.sh failed with this issue, but use a simple nvcc command works well.
Try the script below out.
#!/bin/bash
rm release -r
mkdir release
nvcc --std=c++17 ./storygen.cpp ../../include/rwkv/cuda/rwkv.cu -I../../include -o ./release/storygen

This repository is deprecated and only works for v4, you can try my rwkv.cuh to try v5 on cuda/cpu.


reminder of shortcut that could have been used

Display relevant keyboard shortcuts in the bottom corner of the editor window when the user performs actions that could have been done using shortcuts instead of clicking The status bar item would provide a quick reminder of the keyboard shortcut and offer an explanation of its purpose when hovered over.

Create a new status bar item or use existing at the bottom corner of the VS Code window using the window.createStatusBarItem API.
Subscribe to relevant events, such as mouse hover events, to trigger the display of the shortcut information.
Determine the current context and identify actions where a keyboard shortcut could have been used instead of clicking.
Retrieve the relevant keyboard shortcut for the identified action using the VS Code API, specifically the commands.getCommands and keybindings.getKeybindings methods.
Display the shortcut information in the status bar item when the user hovers over it.
Provide an explanation of the keyboard shortcut and its purpose, possibly using tooltips or other means of displaying additional information.

These are all great ideas. I would really hope it also would work on other extensions other than default vscode extensions, and read customized ones as well.


[CNCF] Understand the license allowlist

Parts of the CNCF onboarding issue tracked by this issue
This is the list of all bullet points from the CNCF onboarding issue that this issue will track:

[ ] Understand the license allowlist: https://github.com/cncf/foundation/blob/master/allowed-third-party-license-policy.md#approved-licenses-for-allowlist

Yes. I understand the license allowlist: https://github.com/cncf/foundation/blob/master/allowed-third-party-license-policy.md#approved-licenses-for-allowlist


Reduce latency by avoiding database commit blocking

Database commits currently occur on the critical path, which slows down the process.
Since this operation is typically time-consuming, we should avoid waiting for it when the extractor isn’t syncing. Instead, we can submit and store the future and await with the next message if it is set. This approach allows database commits to run in the background, ideally completing before the next block arrives.
Taking this idea further, we should support storing multiple futures to effectively handle short bursts of larger data volumes (which are likely to happen on L2s).
Outline of required changes
The following outline serves as a conceptual guide to illustrate the proposed changes. It’s not intended to be prescriptive or definitive.
pub struct ProtocolExtractor<G, T> {
    ...
    reorg_buffer: Mutex<ReorgBuffer<BlockUpdateWithCursor<BlockChanges>>>,
    // A queue of pending database tasks
    db_futures: Mutex<VecDeque<JoinHandle<Result<(), StorageError>>>>
}

We can then store the tasks using:

// todo: remove all finished tasks from the db_futures

while let Some(msg) = msgs.next() {
    // Force a database commit if we're not syncing and this is the last block to be
    // sent. Otherwise, wait to accumulate a full batch before
    // committing.
    let force_db_commit = if is_syncing { false } else { msgs.peek().is_none() };

    let fut = tokio::spawn(self.gateway.advance(
        msg.block_update(),
        msg.cursor(),
        force_db_commit,
    ));
    // todo: ensure this doesn't grow to big.
    self.db_futures.push_back(fut)

}

I would like to take this one.

Thanks, assigned it to you @royvardhan


CORE-16203 5.1 Performance integration - FlowMapper using Multi-Source Event Mediator (#4875)

Using Multi-Source Event Mediator instead of State and Event Subscription in FlowMapperWorker.

E2E test run for this PR: https://ci02.dev.r3.com/job/Corda5/view/Full Health 5.1/job/corda5-e2e-tests/job/release%252F5.1/1076/


Error: max_token to be integer

Hello guys,
I just installed, activated and configured the plugin, but when I click on "Generate answer (GPT)" I get this:
"Error occurred. Please check your internet connection and try again."
In the Laravel logs I see this error:
" Type error: Invalid type: expected Tectalic\OpenAi\Models\ChatCompletions\CreateRequest::max_tokens to be of type integer, instead got value 1024, which is string. "
I tried lover numbers (I use gpt3.5) but it's the same.
Is it a bug?
Thanks

Hello,
Go to the module settings page and make sure that the value of Token limit is 1024 and does not contain spaces.
Click save and try again.

Hello,
I tried but it doesn't help.
I'm running PHP 8.1, maybe is something related to that?
Thanks

Il giorno mar 6 giu 2023 alle ore 14:25 AliakseyM @.***>
ha scritto:

Hello,
Go to the module settings page and make sure that the value of Token limit
is 1024 and does not contain spaces.
Click save and try again.
—
Reply to this email directly, view it on GitHub
https://github.com/code-debug228/HostetskiGPT/issues/14#issuecomment-1578667692,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNY4VZBNDM5T3MM4GXCH23XJ4OTHANCNFSM6AAAAAAY4BQNHY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hello,
What FreeScout version are you using?

I am on latest version of FreeScout and modules are updated.
On Wed, Jun 7, 2023, 00:18 Mihail Barinov @.***> wrote:

Hello,
What FreeScout version are you using?
—
Reply to this email directly, view it on GitHub
https://github.com/code-debug228/HostetskiGPT/issues/14#issuecomment-1579529575,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNY4V4IGTTM65E2XSERTETXJ6UBTANCNFSM6AAAAAAY4BQNHY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Unfortunately, I couldn't reproduce this error, but I made an update in the code that might help. Download updated module code and try again.

Thank you, problem solved!!


Edit organization name/slug

What is the correct way to edit an existing organization name/slug? I modify and save the organization name but the slug does not automatically change. Do I need to manually change both together or is there a helper function that keeps these fields in sync?

The short answer is you'll need to update both.
The AutoSlug fields do not update the slug by default unless it's missing. This tends to be a good strategy!


Cube loader should only load cubes, dimensions >= 3

A real DUH. STcube should only load CubeData with dimensions >=3.

@nmearl Is this an issue with IO, or more appropriately an issue with CubeData? Or does it ultimately not matter?


Update modals.dart

In one of our project, we need to use custom image, which was not possible with current icon data holder, so need to change to ImageIcon.
With this, we can use icons, images from asset etc.
Please verify and upgrade your change.

Hey @dshrinath thanks for your PR
There are some conflicts with master, can you fix them?

Hey @dshrinath thanks for your PR
I like the idea but maybe instead of ImageIcon we should use something more customizable, like a Widget.
Can you do this change?

I changed to Widget and renamed it to trailing.

Fix #15


adds Dan Olson's documentary: “The Problem With NFTs”

Quoting @paullouisageneau:

“I think this is by far the most articulate video I've seen about the whole cryptocurrencies and NFTs issue. I like how he addresses the problem as a political one, whereas a lot of skeptics, as technical people, tend to miss that aspect or play it down. Folding Ideas is undoubtedly great even for subjects far from cinematography, the original topic.”

@CynicusRex It has already been added, it's actually the first link in the general category 😉

@CynicusRex  Thanks for the suggestion and indeed as @paullouisageneau mentions it is already in there so won't merge. If you are interested I've started extensive notes from the documentary here https://github.com/rufuspollock/awesome-crypto-critique/blob/main/notes/olson-2022-line-go-up.md and you and @paullouisageneau are welcome to add to there too! (especially a summary of the main points for people who may not watch the full 2h ...)


vectorized fix window function can't handle first row of beyond

Proposed changes
Issue Number: close #16845
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run p0

LGTM

Please also add test case for first_value

LGTM


ES6 to ES5 translation error of  "for of" loops targeting Set objects

In my projet I have a node module dependency delivered in ES6 which is incorrectly translating to ES5 in my output bundle.
Indeed, I have "for of" loops wrongly translated to simple "for" loops, even when the target is a Set object. The problem being that the property length doesn't exist on a set object !
For exemple :
let items = new Set();
for(item of items) { ... }

ouput like this:
let items = new Set();
for(let i=0; i<items.length; i++) {
     // Will never end up in this for loop because "items.length" will always be "undefined"
}

Any idea how to fix this ?

@guins that's typescript, unfortunately we can't do much now. Not at this stage. You should set target ES2017 then it will be transpiled correctly.  Alternatively add a polyfill to fix the issue

@guins can you please confirm if the issue still occurs with the latest v4 alpha?

Closed as no response received.


Use griffe for function signature

An attempt to use griffe for function signatures. This only changes the sig to use griffe at the moment, consequences are not handled yet.
For #249

The CI failure with numpy is due to the infinite recursion in griffe while loading numpy modules. I have created an issue in griffe here: https://github.com/mkdocstrings/griffe/issues/155

Preview failing because, griffe is not added as dependency in main branch's workflow file. Other wise all tests passes.

The CI failure with numpy is due to the infinite recursion in griffe while loading numpy modules. I have created an issue in griffe here: https://github.com/mkdocstrings/griffe/issues/155

This was fixed yesterday and a release was made.

Fixed conflicts.

It also does not seem to be able to extract signature for a number of compiled function in numpy for example, so we may want our own implementation that is based on inspect.

https://github.com/scikit-image/scikit-image/pull/6905 might be of help as well.

Apologies for the late review. I had to do a number of fixes to the rendering before reviewing this as changing fully qualified names to use : broke a number of thing a number of places.
Now your ObjectSignature need to have a way to be exported to Json and reimported. You should be able to modify the take2.py Signature Object to hold those values.

If we go the more generic discussion about the is_<foo>, you can see the inspect has grown them organically, but they are all mutually exclusive nothing can be is_function and is_async_generator (I think). So this point us that most likely we want to store an Enum in the JSON. If you look at what griffe dump give you, you will mostly see "kind":... whcih tend to confirm what we have.

Checking this is not exactly what griffe does (generator async or not are all tagged as function), but may have the "async" label.

Almost there, just trying to fix this:
ValueError: Wrong type at field :: signature.value field of  <class 'papyri.take2.Signature'> : expecting one of typing.Optional[papyri.signature.Signature], got "DataSource(destpath='.')".value field of  <class 'papyri.take2.Signature'> : expecting one of typing.Optional[papyri.signature.Signature], got "DataSource(destpath='.')"


Code comments need to mention the units of time used

Description:

Everywhere we talk about time durations in the code, we need to mention the unit of time used. For example, the cache expiration times in the api-manager.xml file.

by @Nirdesha from https://wso2.org/jira/browse/APIMANAGER-5522

Closing Since the issue has been stale for a long time.


[messaging] Update https-proxy-agent

Packages impacted by this PR

@azure/event-hubs
@azure/service-bus

Issues associated with this PR

https://github.com/Azure/azure-sdk-for-js/security/dependabot/49

Describe the problem that is addressed by this PR
Updates with the latest https-proxy-agent which does not have the vulnerability.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


Implement object deletion in test steps.

What type of PR is this?
/kind feature
What this PR does / why we need it:
This implements object deletion in test steps, as defined in KEP-0008.
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Test harness now supports deleting resources as a part of test steps.

Updated this to also allow matching objects to delete on label instead of just name as this is required for the test that @gkleiman wants to write using this feature.


chore(api): comment newsletter worker to understand the process

:tada: This PR is included in version 1.4.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add assert contains

From https://github.com/microsoft/botbuilder-dotnet/pull/1922

Hey Axel, this in an external PR that has been lingering for a long time.
Can you review this in Python? I've already approved the C# one.


Leetcode Question Number 1281

Leetcode Question Number 1281 solution needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

Assign me


Option to stop text scroll when music paused

I noticed that the scrolling text would catch my eye even when the music wasn't playing.
I think it would be nice if there was an option in the Text Label configuration to stop scrolling when the music isn't playing (with the text either pausing in the current position or jumping to the original position).
Unrelated: I love this project. Thank you @dsafa for all of your hard work!

Thanks, I've got plans for better scrolling options which will include this.

Closing done issues


Remove unused doubled methods

I found a few test doubles that unnecessarily define methods that are not required to make the test pass.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.

These methods can be safely removed from their test doubles without
breaking the test case

👍


Updated PR template - less negatives

Change content and motivation
Removed some Negatives (so that Positives are more plentiful than Negatives). Added "kubenow/docs" to cross repo examples.
GitHub cross-links

Fixes:

Docs:

:-)


Develop Change Healthcare data source (counts signal)

[x] Deliver CHC our user access list by 9 July
[x] Create and send CHC a list of tools & libraries we need installed to their dev environment
[x] Develop the research results pipeline
[x] Get the pipeline statistically certified by CHC
[x] Deploy the research results pipeline
[ ] Develop hookups to transfer research results to COVIDcast receiving

Other tasks as they arise.

We have counts; just chasing down the last few weird bits and then we can write them up as an indicator.

Working on getting this to staging and Automated now 👍
Additionally, the production deployment branch is in place so it will be an easy promotion once we validate in staging.

Not sure what the last bullet point above means, but is this done with #479? (or if further development is desired, should we make separate specific tickets?)

Yep!


Add User, School, Provider models and default scopes

Context
Setting up some of the basic data structures before proceeding with feature work.
I decided it made sense to do this as a separate PR before proceed with logging in as a support user.
Changes proposed in this pull request
No UI changes. The only change to existing functionality is to add the service to the seeded personas.
Guidance to review
Pull and run the migrations and seed the database.
Have a look at the erd in this PR -- they should reflect the agreed approach. https://github.com/DFE-Digital/itt-mentor-services/pull/30
Note: Not everything in the erd is here -- I haven't included the membership model because that is outside of the scope of what I'm working on (support users will have access to all the schools and providers, not through membership). I can add it for completeness if people think it would be useful. Otherwise, happy for someone to do it when it becomes more relevant.
Link to Trello card
https://trello.com/c/SX3aODZS
Things to check

[NA] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[NA] API release notes have been updated if necessary
[NA] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[NA] Required environment variables have been updated or added to the Azure KeyVault

Screenshots

Based on the new support_user_enabled attribute, I have a few questions:

Will a User be allowed multiple roles?
Would we continue with this path to differentiate Provider users vs School users?

Other than support, I don't think there are other roles. We have asked Simon a few times I think about whether or not there will be an 'admin' role -- someone who is able to add other users for schools, for instance. The answer, as I understand it, is that what a user can do will be based on the service / membership, not a specific role. I think SupportUser is unique as a role.
If we think there are going to be other roles, using inheritance and scoping for the support user is probably not the right approach. I thought about having a separate SupportUser model (ie, placements_user has_one placements_support_user). Do you think that would be a more scalable approach?

I believe so. My understanding is that there will be unique support journeys for each service.

 Models and migrations to enable support user journey

Other than support, I don't think there are other roles. We have asked Simon a few times I think about whether or not there will be an 'admin' role -- someone who is able to add other users for schools, for instance. The answer, as I understand it, is that what a user can do will be based on the service / membership, not a specific role. I think SupportUser is unique as a role.
If we think there are going to be other roles, using inheritance and scoping for the support user is probably not the right approach. I thought about having a separate SupportUser model (ie, placements_user has_one placements_support_user). Do you think that would be a more scalable approach?

I was thinking that, yeah. I'm just not sure if it's necessary or not. 🤔 I do think it's more modifiable though. There is a small db performance tradeoff by separating this context into a separate table though. I have seen AdminUser -> User in the past though and it's generally fine with the proper indexes and eager loading.

I'm not sure what you mean. The user is scoped for service and then associated with organisations (that isn't represented here).

I think the question is two-fold. 1) Are we planning to categorise users into School users vs Provider users? 2) If so, will we add an additional boolean attribute to signify those roles?

Other than support, I don't think there are other roles. We have asked Simon a few times I think about whether or not there will be an 'admin' role -- someone who is able to add other users for schools, for instance. The answer, as I understand it, is that what a user can do will be based on the service / membership, not a specific role. I think SupportUser is unique as a role.
If we think there are going to be other roles, using inheritance and scoping for the support user is probably not the right approach. I thought about having a separate SupportUser model (ie, placements_user has_one placements_support_user). Do you think that would be a more scalable approach?

I was thinking that, yeah. I'm just not sure if it's necessary or not. 🤔 I do think it's more modifiable though. There is a small db performance tradeoff by separating this context into a separate table though. I have seen AdminUser -> User in the past though and it's generally fine with the proper indexes and eager loading.

I suppose we could keep it as a bool for now and it would be a straightforward migration to move it to a separate table later if we find it's necessary. Perhaps it is wise not to anticipate complexity that may not actually exist. I'm happy to go either way. @ollietreend do you have a view?

I'm not sure what you mean. The user is scoped for service and then associated with organisations (that isn't represented here).

I think the question is two-fold. 1) Are we planning to categorise users into School users vs Provider users? 2) If so, will we add an additional boolean attribute to signify those roles?

Thanks for the clarification! 1) I don't see where that will be necessary. They will be users for whatever organisation they have a membership to. And since that is a polymorphic association, the school-ness or provider-ness of the will be implied. Does that sound ok?

I suppose we could keep it as a bool for now and it would be a straightforward migration to move it to a separate table later if we find it's necessary. Perhaps it is wise not to anticipate complexity that may not actually exist. I'm happy to go either way. @ollietreend do you have a view?

👍 I'm good with keeping it a boolean for now. I'd also drop the _enabled as @ollietreend suggested.

Thanks for the clarification! I don't see where that will be necessary. They will be users for whatever organisation they have a membership to. And since that is a polymorphic association, the school-ness or provider-ness of the will be implied. Does that sound ok?

👍 Sounds good to me for now. This means that a user can potentially belong to both a School and a Provider.


product.json plotly not showing ber tract bar graphs

@stevengeeky I just tested the latest update to add plotly graphs on product.json. I see the aggregated fiber count graph, but not the per tract bar graph that you've shown me. I thought we were going to display both graphs?

This is fixed


BUG: Compact Window It doesn't work properly.

Description
When you go to the compact field view, the interface window is shrinking very much and cannot be expanded.
Steps To Reproduce

Go to Compact View CTRL+ALT+UP

Steps to fix

The window should be resized when it goes to a compact appearance.

Files Version
<IP_ADDRESS>
Windows Version
Windows 10 Home 22H2 19045.2486
Relevant Assets/Logs

This is the intended behavior and not something we have controll over, compact overlay is not the same as "always on top".


DB error (Image search)

Error when pressing Image Search. No logs were left, so it's not clear what the error was. After the first time it works, it happens when you click Reload UI. After that, it doesn't work.
ERROR:    Exception in ASGI application
Traceback (most recent call last):
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\uvicorn\protocols\http\h11_impl.py", line 429, in run_asgi
result = await app(  # type: ignore[func-returns-value]
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\uvicorn\middleware\proxy_headers.py", line 78, in call
return await self.app(scope, receive, send)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\fastapi\applications.py", line 273, in call
await super().call(scope, receive, send)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\applications.py", line 122, in call
await self.middleware_stack(scope, receive, send)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\errors.py", line 184, in call
raise exc
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\errors.py", line 162, in call
await self.app(scope, receive, _send)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\gzip.py", line 24, in call
await responder(scope, receive, send)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\gzip.py", line 44, in call
await self.app(scope, receive, self.send_with_gzip)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\exceptions.py", line 79, in call
raise exc
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\middleware\exceptions.py", line 68, in call
await self.app(scope, receive, sender)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\fastapi\middleware\asyncexitstack.py", line 21, in call
raise e
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\fastapi\middleware\asyncexitstack.py", line 18, in call
await self.app(scope, receive, send)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\routing.py", line 718, in call
await route.handle(scope, receive, send)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\routing.py", line 276, in handle
await self.app(scope, receive, send)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\starlette\routing.py", line 66, in app
response = await func(request)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\fastapi\routing.py", line 237, in app
raw_response = await run_endpoint_function(
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\venv\lib\site-packages\fastapi\routing.py", line 163, in run_endpoint_function
return await dependant.call(**values)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\extensions\sd-webui-infinite-image-browsing\scripts\api.py", line 349, in match_image_by_tags
ids = [int(x) for x in tag_ids.split(',')]
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\extensions\sd-webui-infinite-image-browsing\scripts\api.py", line 349, in
ids = [int(x) for x in tag_ids.split(',')]
ValueError: invalid literal for int() with base 10: ''
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\extensions\sd-webui-infinite-image-browsing\scripts\api.py", line 328, in get_db_basic_info
img_count = DbImg.count(conn)
File "C:\Users\bluen\Documents\AI\stable-diffusion-webui\extensions\sd-webui-infinite-image-browsing\scripts\db\datamodel.py", line 106, in count
with closing(conn.cursor()) as cur:
sqlite3.ProgrammingError: SQLite objects created in a thread can only be used in that same thread. The object was created in thread id 11760 and this is thread id 25828.

You need to select a tag first before clicking the search button.
You can select a tag by clicking on the tags listed below or by manually typing a tag in the input box.
I forgot to add a restriction for this case.

What about images without tags?

And it doesn't search for anything with tags.

Why are there no tags when using images generated by stable-diffusion-webui?
If you can see the image generation information through the right-click menu, there must be relevant tags.
For example, your model, size, sampler, positive prompt, Lora, and so on will be automatically converted into tags.

Are you run this repo separately   and no pass --sd_webui_config param?

Did you run this repo separately without passing the --sd_webui_config parameter?

yes

Why are there no tags when using images generated by stable-diffusion-webui?
If you can see the image generation information through the right-click menu, there must be relevant tags.
For example, your model, size, sampler, positive prompt, Lora, and so on will be automatically converted into tags.

The ones with and without tags are together.

The image you uploaded is tagged "1girl". However, when hit Search, I don't see anything.

The image you uploaded is tagged "1girl" and ETC.. . However, when hit Search, I don't see anything.

Suddenly found.
It seems that your --sd_webui_config parameter is referring to a directory, rather than a JSON file.
This is the path to my JSON config file: C:\Users\zanllp\Desktop\stable-diffusion-webui\config.json.
this file, In  sd-webui working directory

Did I run it the wrong way?

Now it looks like the parameters are correct.
Did clicking 'generate image for image index' still fail to generate the tag? If it still doesn't work, could you pull the repository and try running it again?

Did I run it the wrong way?

You seem to have missed the step of installing dependencies, haven't you?

Did I run it the wrong way?

You seem to have missed the step of installing dependencies, haven't you?

I already have the requirement.txt installed. The add-on works very well. However, stand alone is still the same.

The UI in dark mode seems to have some issues too. I forgot to check it yesterday.
You can enable the development mode using set APP_ENV=dev && python app.py [extra args].
Errors will be written to the log.log file in the repo root directory.
Please show me the error part of the log afterwards.

This is the log content before using git pull.
After using git pull, there is no log left.
2023-04-27 10:32:19,083 - scripts.logger - ERROR - [WinError 3] ������ ��θ� ã�� �� �����ϴ�: 'C:/Users/bluen/Documents/AI/stable-diffusion-webui-ux/stable-diffusion-webui-ux/outputs'
2023-04-27 10:32:22,759 - scripts.logger - ERROR - [WinError 3] ������ ��θ� ã�� �� �����ϴ�: 'C:/Users/bluen/Documents/AI/stable-diffusion-webui-ux/stable-diffusion-webui-ux/outputs'


Write documentation for programmers on how to release an anthology

Releasing an anthology is not trivial, and of course pushing files to the publication location is potentially risky, so this needs to be detailed and precise.

Done.


Debug fails with AGBPI error:  Class JavaLaunchHelper is implemented in both...

AGPBI: {"kind":"error","text":"objc[1413]: Class JavaLaunchHelper is implemented in both /Library/Java/JavaVirtualMachines/jdk1.8.0_91.jdk/Contents/Home/bin/java and /Library/Java/JavaVirtualMachines/jdk1.8.0_91.jdk/Contents/Home/jre/lib/libinstrument.dylib. One of the two will be used. Which one is undefined.","sources":[{}]

Attempting to downgrade to JDK 1.7.0_25 to unblock running a build.

Got this working with 1.7, but I think the trick was running a "Clean build" locally. I've added 1.8 again and seems to work ok (and 1.7 was making Android Studio look fuzzy on my machine)


Add helm diff functionality to fluxctl

I propose adding helm diff functionality to fluxctl to:

Locally test out HelmRelease changes before committing to origin
Show diffs in Flux HelmRelease repository pull requests before merging

Currently this functionality can be hacked by copying values portion of HelmRelease to a separate file and running helm diff plugin accordingly.

Duplicate of #1864


Sensitivity

Hey,
I didn't know where to put suggestions (the link on the project site for "feedback" didn't lead me anywhere to leave feedback honestly, or I am blind) so I'm putting it here.
I am using this plugin now and it works great. The only thing I am missing is a better sensitivity. There is an old plugin that did a great job at filtering/sorting results based on the input:
https://github.com/jakiestfu/Mention.js#sensitivity
This worked much better in terms of ordering the results. Maybe your plugin can do this already but I couldn't find any option for it. Without it, I am left with either correct input (case sensitive) which isn't very practicable with names or no ordering of the results ("A" will return every name that contains an "a" or "A" but doesn't weigh the results).
If you don't think this would be a great addition, never mind :)

You can use the filter.casesensitve option

I can but as I said, it does not help a lot.
You either set it to insensitive: everything will be suggested that works with the input string.
Or you set it to sensitive: only case matching strings will be suggested.
The idea of the sensitivity is the probable order of results being returned, taking the case and position of characters into account.
Case sensitivity doesn't solve this with true and false values.
When working with usernames you don't want to force anyone to use correct upper and lower case when looking up users but you also want results that look more matching to be on top of the suggestions list.
But since you haven't read the doc which was directly linked to the section that explains it I will just go for a modification myself.

You're right, my bad, I directly concluded that you meant about case sensitivity. Anyways, feel free to submit a PR or lemme know how it goes for you.


Use buildkit for docker builds

This switches Docker builds to use build kit which is less prone to error than legacy docker-in-docker building. It also switches back to using the upstream docker image again now that it has been updated to a version that supports alpine 3.14+

Codecov Report

Merging #23415 (cdbf64d) into main (8cadd51) will decrease coverage by 0.50%.
The diff coverage is 27.65%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main   #23415      +/-   ##
==========================================
- Coverage   47.61%   47.11%   -0.50%
==========================================
  Files        1147     1149       +2
  Lines      151225   151446     +221
==========================================
- Hits        72002    71360     -642
- Misses      70709    71650     +941
+ Partials     8514     8436      -78

Impacted Files
Coverage Δ

cmd/admin.go
0.00% <0.00%> (ø)

models/user/user.go
58.86% <0.00%> (-1.87%)
:arrow_down:

modules/context/access_log.go
0.00% <0.00%> (ø)

modules/git/command.go
86.12% <ø> (+2.04%)
:arrow_up:

modules/repository/push.go
45.45% <ø> (-10.11%)
:arrow_down:

routers/api/v1/admin/hooks.go
0.00% <0.00%> (ø)

routers/api/v1/org/hook.go
0.00% <0.00%> (ø)

routers/api/v1/user/hook.go
0.00% <0.00%> (ø)

routers/web/admin/auths.go
43.44% <0.00%> (-3.81%)
:arrow_down:

routers/web/org/projects.go
0.76% <0.00%> (-0.01%)
:arrow_down:

... and 30 more

... and 35 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Code formatting settings in IDE's

Since different developers are working on the project, the code formatting in IDE's should be standardized.
IDE's:

Eclipse
IntelliJ
vscode (?)

A common method is to use the Eclipse formatting scheme, which can be imported into an IntelliJ project.
https://github.com/redhat-developer/vscode-java/blob/master/formatters/eclipse-formatter.xml
https://blog.jetbrains.com/idea/2014/01/intellij-idea-13-importing-code-formatter-settings-from-eclipse/

Great, this could be KBA Issue.

I would like to solve this KBA


Fixed bugs for implementing using recycle

server store$ using take(1) is created in its own driver

@domagojk  - After npm run build-server && node dist/server and curl'ing, I get:

Error Cannot read property 'store$' of undefined

Are you seeing the same?

@leebenson that is strange, it works fine for me
I tried on my branch and I also tried cloning your branch again
git clone https://github.com/leebenson/reactnow --depth 1 -b recyclejs --single-branch
and it worked as expected 😕

ah, I think I know why

recyclejs            ^1.0.0  →   ^1.1.2

package.json needs updating... will try again

Perhaps you didn't update recycle to latest version (1.1.2) ?

works now 😄


Icon still no working on darwin (OSX) with electron 1.2.2

As issue #300, again with the last version of electron (1.2.2) Icon didn't work on darwin.
If I explore the package the icns file call atom.icns instead electron.icns

Which version of electron-packager are you using?

7.0.4

What CLI arguments are you passing? Alternatively, if you are using the API, what parameters are
you passing to the packager() function?

use API packager function

What version of Electron are you building with?

1.2.2.

What is the host platform are you running electron-packager on?

darwin OSX el capitan

What target platform(s)/architecture(s) are you building for?

all , but the bug still on darwin

Is there a stack trace in the error message you're seeing?

(node:37556) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.

Please provide either a failing testcase or detailed steps to reproduce your problem.

Compile with icon parameters

I can't reproduce this. I just built my app with Electron 1.2.3, and in Foo.app/Contents/Info.plist, the value of CFBundleIconFile is electron.icns, and Foo/Contents/Resources/electron.icns exists as the file I passed in as the icon. Perhaps you did not save the .icns file with the correct image sizes?

hi @malept Thanks you !
The problem is that I have installed an old version of electron-packager :(...
The last version was installed with -g arguments but my compile script still grab the local one with an old version :S
Thanks a lot !
Elia


Vagrant File Update

updated vagrant file for centos.

@RaghavBabu can you make these changes for the ubuntu and playground vagrantfile as well?


GPT-4 does not exist

After set up I am getting the following
    "error": {
        "message": "The model: `gpt-4` does not exist",
        "type": "invalid_request_error",
        "param": null,
        "code": "model_not_found"
    }
}

If I search for gpt-4 in the project I see a couple of references. Anyone getting the same problem?

Sadly, i realised while GPT-4 is being actively rolled out by openAI to a large % of folks - its not everyone.
This uses GPT-4 for openAI, so you will need API access for it

Model seems to be listed as an "option" in several places.
What's the right way to adjust so we can use gpt-3.5?

I believe gpt-4 roll out is completed, so will be closing this issue.
PS: didn't allow the usage of gpt3.5 due to how low quality / often out of context will be hit


Smoke test failure

Details: https://github.com/microsoft/vscode-jupyter/runs/1343321593?check_suite_focus=true

The attached logs are not related to the failure.

@joyceerhl are you investigating this? The latest failures look related to the random bytes stuff. If you're not actively investigating, I'm happy to look at it.

Sorry just saw this @rchiodo. I've been investigating the smoke tests for the last 2-3 days (among other things) but wouldn't mind extra eyes on these.
The random bytes smoke test is failing on Windows, and I can repro the failure locally. But I'm pretty sure it's only a smoke test setup issue because the actual VSIX itself works fine. For some reason node can't load the ffi-bindings.node module (complains about it not being a valid win32 application). My latest theory I'm pursuing is that the smoke tests are running with the wrong node.exe.
There are also macos smoke test failures. Random bytes test passes there but the other two tests (running cells in notebook and IW) fail. There are errors about the interpreter not having ipykernel installed which seems weird because ipykernel is deliberately installed as part of the smoke test requirements.

I'll look at the mac test failures.

Validated.


Cleanup of README and travis.yml

Signed-off-by: Martin Mihálek<EMAIL_ADDRESS>

Changes Unknown when pulling 8be92f0e6988ffb41e1f54fbb5fd908a7a098899 on docs_cleanup into ** on master**.


lib.externals - lodash

Type of Issue
[ ] Bug Report
[ ] Feature Request
[*] I'm stuck

Description
BUILD ERROR
'forEach' is not exported by node_modules\lodash\index.js
Error: 'forEach' is not exported by node_modules\lodash\index.js
....
How To Reproduce
in a component.ts use the following to import:
import * as _ from 'lodash';

  ngOnInit() {
    const x = [];
    _.forEach(x, (row: any) => {
      console.log(row);
    });
}

Expected Behaviour
in package.json some like this should work?
  "ngPackage": {
    "lib": {
      "entryFile": "public_api.ts",
      "externals": {
        "_": "lodash"
      }
    }
  }

Version Information
ng-packagr: v1.5.0
node: v6.9.3
@angular: v4.4.4

I can't figure out how to setup the lib.externals for this..  tried lots of variations no luck..
Studied #129 , no example seems to help for this case.

 These externals need improvement.
Your confit should be:
“lodash”: “something”
Lodash because that is the import path in the typescript statement. Something needs to be replaced with the UMD module id. The only reliable way if figuring it out is looking at  dist JavaScript code. Suspicion is that it’s maybe “” the underscore for lodash because you’re using lodash as “.foo()” when it’s registered on the global scope.
Cheers, David

On 10. Nov 2017, at 01:44, joshpitkin<EMAIL_ADDRESS>wrote:
Type of Issue
[ ] Bug Report
[ ] Feature Request
[*] I'm stuck
Description
BUILD ERROR
'forEach' is not exported by node_modules\lodash\index.js
Error: 'forEach' is not exported by node_modules\lodash\index.js
....
How To Reproduce
in a component.ts use the following to import:
import * as _ from 'lodash';
ngOnInit() {
const x = [];
_.forEach(x, (row: any) => {
console.log(row);
});
}
Expected Behaviour
in package.json some like this should work?
"ngPackage": {
"lib": {
"entryFile": "public_api.ts",
"externals": {
"_": "lodash"
}
}
}
Version Information
ng-packagr: v1.5.0
node: v6.9.3
@angular: v4.4.4
I can't figure out how to setup the lib.externals for this.. tried lots of variations no luck..
Studied #193 , no example seems to help for this case.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

So this is strange.  I discovered the lodash error was actually coming from a lodash import within another library, not my own.  The error was quite misleading as adding
"lodash": "_"
(and a variety of other combinations) to externals did nothing to it..
When I finally found the culprit library and added it to the externals it everything cleared up.
Yeah, looking forward to something better for externals if you figure it out..
Thanks for the quick reply!

After talking to Filipe Silva, he suggested something like "dynamic externals" in rollup. If you or someone else knows about this, please let me know.


The allOf sections in these objects is breaking the swagger-codegen generated clients, so…

… remove them, but keep the objects
Signed-off-by: Samuel Dacanay<EMAIL_ADDRESS>
What this PR does / why we need it: The branch name was a bit premature, but essentially the "allOf" logic in the swagger.yaml was affecting the ability of a python swagger codegen client to successfully be used (it resulted in some ImportErrors when the client got imported)
This PR replaces those blocks with the properties of the "inherited" definition. We don't get the allOf simplicity any more, but it allows the clients to still work and allows us to keep the definitions themselves, albeit with a little less reuse.

@dakaneye, I would like us to move in the direction of more reuse. It makes the APIs easier to read and consistent across the board (for instance image definition is slightly different depending on the route although it doesn't have to be). With the upgrade to swagger 3.0 we should be able to use oneOf, anyOf and other keywords for combining schemas with builtin validation. I understand that this could be a pain with codegen client, is there anyway we could patch the codegen to work correctly?

@zburstein your changes are failing CI. Please revisit when you get time.

@Vijay-P Turns out that my commits and one of Sam's commits in this branch are already present in master so when I rebased the branch, there is only one commit remaining. @dakaneye I just want to confirm that this PR/commit is still relevant and should still be merged given that the PR is in a pretty different state from what it was when it was first opened.


If noCalendar is set, the wipes will remain

Due to the influence of the open class pseudo-element, the wipe remains.

https://jsfiddle.net/yogarasu/f5xa4jLm/3/
Your Environment

flatpickr version used: v4.6.13
Browser name and version: Chrome 103.0.5060.134（Official Build）(arm64）
OS and version: Mac OS 12.5

The wipe is given because of the pseudo element of the open class, but if the name open is accepted at face value, is it better not to give the open class when nocalendar is true in this process?
I'm uncertain, so I gave a issue for the time being.
https://github.com/flatpickr/flatpickr/blob/master/src/index.ts#L1934


Create exception hierarchy based on status code classes

Currently, there is no way to catch all exceptions for a specific class of status codes.
A client might e.g. want to retry requests based on the status code class. It might e.g. make sense to retry a request returning a 5xx server error, but not to retry the request for a 4xx client error (that would fail again anyway with the same parameters).
Such a hierarchy would also allow us to better process status codes that don't have a specific exception for.
Suggestions for a hierarchy:

Base - allow catching all Grac-related exceptions

ClientException - already exists, keep for compatibility, currently inherits from StandardError

ClientError (4xx)

BadRequest (400)
Forbidden (403)
NotFound (404)
Conflict (409)

ServerError (5xx)

ServiceError - we already have this and don't want to break applications catching these (might have to add it above ServerError)

InternalServerError (500)

RequestFailed - already exists, currently inherits from StandardError

ServiceTimeout - already exists

InvalidContent - already exists, currently inherits from StandardError

We'll have to extend this hierarchy when adding more status codes (see #1). We don't explicitly deal with redirects yet, so nothing on 3xx codes here.
Still missing:

Handling unknown 2xx and 3xx status codes

After a bit of experience with the current error handling in Grac, we found another problem: Currently, the same exception is raised when both an error response is valid JSON and when it's any other format, like an HTML page from a load balancer indicating a 503.
That makes error handling harder - instead of only checking the error message in the response body, we also have to check that the response body could be parsed as JSON at all, so our error handling often looks like this:
begin
  client.path('/v1/foo').post(foo: 'bar')
rescue Grac::Exception::ClientException => e
  unless e.body.kind_of?(Hash)
    raise e
  end
  case e.body['code']
  when 'invalid_parameters'
      # [...]
  else
    raise e
  end
end

We're going to solve this by raising a different exception that doesn't inherit from ClientException. We'll name it ErrorWithInvalidContent and it's going to inherit directly from StandardError.
With this, there's no need to catch the ErrorWithInvalidContent, we can treat it as a common exception and let our standard exception handling deal with it.
With this change, the code above will look like this (the type check for .body is no longer necessary):
begin
  client.path('/v1/foo').post(foo: 'bar')
rescue Grac::Exception::ClientException => e
  case e.body['code']
  when 'invalid_parameters'
      # [...]
  else
    raise e
  end
end

I have no objections


Can't open PR - Object reference not set to an instance of an object.

Error window pops up with these messages, steps to reproduce:

gh repos
select PR menu item

Flow Launcher version: 1.11.0
OS Version: Microsoft Windows NT 10.0.22000.0
IntPtr Length: 8
x64: True

Python Path: C:\Users\OMG\AppData\Local\Programs\Python\Python311\pythonw.exe
Node Path: C:\Program Files\nodejs\node.exe
Date: 01/05/2023 22:41:25
Exception:
GitHub Exception:
Websites: https://github.com/JohnTheGr8/Flow.Plugin.Github
Author: Ioannis G. (@JohnTheGr8)
Version: 1.2.2
Flow.Launcher.Core.ExternalPlugins.FlowPluginException: Object reference not set to an instance of an object.
 ---> System.NullReferenceException: Object reference not set to an instance of an object.
   at <StartupCode$Flow-Plugin-Github>.$GithubPlugin.clo@71-14.GenerateNext(IEnumerable`1& next) in C:\projects\flow-plugin-github\src\GithubPlugin.fs:line 0
   at Microsoft.FSharp.Core.CompilerServices.GeneratedSequenceBase`1.MoveNextImpl() in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\seqcore.fs:line 371
   at Microsoft.FSharp.Collections.SeqModule.ToList[T](IEnumerable`1 source)
   at <StartupCode$Flow-Plugin-Github>.$GithubPlugin.func2@1(GithubPlugin this, FSharpChoice`2 _arg4) in C:\projects\flow-plugin-github\src\unknown:line 134
   at <StartupCode$Flow-Plugin-Github>.$GithubPlugin.-ctor@131-2.Invoke(FSharpAsync`1 x) in C:\projects\flow-plugin-github\src\GithubPlugin.fs:line 131
   at Flow.Plugin.Github.GithubPlugin.Flow.Launcher.Plugin.IPlugin.Query(Query query) in C:\projects\flow-plugin-github\src\GithubPlugin.fs:line 148
   at System.Threading.Tasks.Task`1.InnerInvoke()
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread)
--- End of stack trace from previous location ---
   at Flow.Launcher.Core.Plugin.PluginManager.<>c__DisplayClass22_0.<<QueryForPluginAsync>b__0>d.MoveNext() in C:\projects\flow-launcher\Flow.Launcher.Core\Plugin\PluginManager.cs:line 191
--- End of stack trace from previous location ---
   at Flow.Launcher.Infrastructure.Stopwatch.DebugAsync(String message, Func`1 action) in C:\projects\flow-launcher\Flow.Launcher.Infrastructure\Stopwatch.cs:line 40
   at Flow.Launcher.Core.Plugin.PluginManager.QueryForPluginAsync(PluginPair pair, Query query, CancellationToken token) in C:\projects\flow-launcher\Flow.Launcher.Core\Plugin\PluginManager.cs:line 213
   --- End of inner exception stack trace ---
   at Flow.Launcher.Core.Plugin.PluginManager.QueryForPluginAsync(PluginPair pair, Query query, CancellationToken token) in C:\projects\flow-launcher\Flow.Launcher.Core\Plugin\PluginManager.cs:line 213
   at Flow.Launcher.ViewModel.MainViewModel.<>c__DisplayClass135_0.<<QueryResults>g__QueryTask|2>d.MoveNext() in C:\projects\flow-launcher\Flow.Launcher\ViewModel\MainViewModel.cs:line 821
--- End of stack trace from previous location ---
   at Flow.Launcher.ViewModel.MainViewModel.QueryResults() in C:\projects\flow-launcher\Flow.Launcher\ViewModel\MainViewModel.cs:line 822
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_0(Object state)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

Could you please share the exact queries that cause an exception for you? I tried to reproduce with no luck...


Add simple script to traversed defined pipelines and output their config.

Per my note today, here is the script used to analyze and output configuration for pipelines declared in the project.

Thanks @Aerlinger !

I have no objection to your adding these files.
On Tue, Jun 30, 2020 at 10:44 AM Anthony<EMAIL_ADDRESS>wrote:

@Aerlinger commented on this pull request.
In .gitignore
https://github.com/open-covid-19/data/pull/202#discussion_r447868015:

\ No newline at end of file
+.ipynb_checkpoints/
+.idea/

These are files which are added by my local environment by IntelliJ IDEA
and Python virtualenv, which aren't necessarily in my direct control, so I
add them to .gitignore to prevent them from being tracked by git.
I could also update my global .gitignore if it's an issue, but it's pretty
common to see paths like .idea/ in a project .gitignore.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/open-covid-19/data/pull/202#discussion_r447868015,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABUM5R7TEXN77EJ6EIZLWOTRZIP7BANCNFSM4OLWNGCA
.


changed php version during phar creation

I changed the PHP version from 7.4 to 7.1 to solve this issue: #242
In my opinion, this can create future problems, I don't know at the moment if the option target-php-version is enough or we have to create a phar version for each PHP version.

Codecov Report

Merging #245 (45c2ab8) into main (755e6e4) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #245   +/-   ##
=========================================
  Coverage     93.10%   93.10%
  Complexity      324      324
=========================================
  Files            54       54
  Lines           841      841
=========================================
  Hits            783      783
  Misses           58       58

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 755e6e4...45c2ab8. Read the comment docs.

Do you think it might be worth creating some tests?

Yes @fain182, but you need to create a phar file and try it with different containers with different PHP versions.

Makes sense to me as we should use the lowest PHP version we support in composer.json
IIRC that was the case at the beginning, did we change it for a reason? 🤔


leetcode 32

Fixes #{Insert your issue number here}
32

My code sample is in :

[🗸 ] CPP
[ ] Java

Screenshot of working code sample  :

Checklist

[🗸 ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[🗸 ] My code sample is formatted and works correctly

Please update your issue comment appropriately, also add one solution at once.

done

Please update your issue comment appropriately, also add one solution at once.

please update now


⚠ AGEG website (prod) has degraded performance

In dc293ac, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 1720 ms

Resolved: AGEG website (prod) performance has improved in 32564c9.


Cluster toolbar move

Depends on https://github.com/mareklibra/facet-lib/pull/255
Move the cluster list toolbar actions above the table
To align to OCM cluster list the botton toolbar is removed in clusters list and actions are put into clusters table toolbar.
Add 'refresh' button to cluster list toolbar

Cool! Mind adding a screenshot?


Can't reproduce the result of torch_vct

Thanks for your excellent implementation!
I tried to train VCT on 2 RTX 4090, a total of 100k step used. Initially I follow the training config mentioned in README but fail to reproduce.Then I adjust the learning rate and distortion lambda for several groups, PSNR/bpp is finally 35/0.11
Would you share the checkpoint?

Hello @cccleon, I'm going to CC @desi-ivanova for running the code while I check in on the status of releasing the checkpoint for you.

Hi @cccleon -- I don't think 2x4090s and 100K steps will get you there, unfortunately. The checkpoint @mmuckley is referring to was trained for 400K steps on 32 V100 (32GB) cards (as described in the readme).

@desi-ivanova @mmuckley
Thanks for your reply!
Looks like I should lower my learning rate and increase my total steps.I'll keep trying~


Copied firmware protocol from Wrangler codebase

@carl-anders would you be willing to confirm that you're ok with me adding this code on your behalf, and that you are agreeing to license it as dually licensed under the LICENSE-MIT and LICENSE-APACHE files that are in this repo?

Ok to copy. I agree to the licenses.


[HIPIFY] Incorrect HIP generated from CUDA for DNN based YOLO Algorithm

Hi,
I have been trying to run Hipify-clang on a YOLO algorithm written using CUDA. However, hipify-clang does not correctly generate a HIP version of the CUDA file. Below are some of the errors I am facing
Some kernels are unable to convert keywords in CUDA:
/u/p/r/preyesh/private/darknet_tiny_preyesh/src/activation_kernels.hip.cu:151:39: error: use of undeclared identifier 'gridDim'
int id = (blockIdx.x + blockIdx.y*gridDim.x) * blockDim.x + threadIdx.x;
        But gridDim is a keyword in CUDA and error should not be flagged.

   2. Does not recognize #define

       <command line>:1:11: note: expanded from here
       #define X 1
                         ^

   3. For kernels like batch_norm, we fixed the typecasting issues by replacing the below code
        l.scales = calloc(c, sizeof(float)); with
        l.scales = (float *)calloc(c, sizeof(float));

However, even after these changes hipify-clang does not generate the correct code -- the .hip file which is generated is either empty or exactly the same as the .cu file. We are using roc-1.6.x branch of hipify-clang as that is the version compatible with the gem5 simulator.
Could you please tell us if we are missing something because of which were are not able to generate the .hip files correctly?
Thank you,
Rohan Mahapatra

Could you please provide the source CUDA file and your command line?

Hi,
I used the following source files (i used other .cu files present in the same directory as well)
https://github.com/pjreddie/darknet/blob/master/src/activation_kernels.cu
And the command was as mentioned in https://github.com/ROCm-Developer-Tools/HIP/tree/roc-1.6.x/hipify-clang. I added my local paths where every it was required in the below command to include the source files and CUDA installation path.
./hipify-clang -print-stats sort_kernel.cu -- -x cuda -I/srv/git/HIP/include -I/usr/local/cuda-7.5/include -DX=1
Thanks,
Rohan

Thanks!
First of all, please refer to the Readme.md from the master branch and clone sources from it as well. 2. hipify-clang is HIP independent tool now, so you may build it separately from HIP, and the include path to HIP (-I/srv/git/HIP/include) is not needed anymore.
Cuda 7.5, well ok... Could you please run hipify-clang -version and provide an output?

Hi @rohanmahapatra,
I've just checked your issue on my side, but only on Windows (with pthread.h port for Windows). I've tested it with CUDA 7.0, 7.5, 8.0, and 9.0. All is ok, except for one thing: #include "cuda.h", which is not CUDA's, but darknet's. So I changed the name to cuda_darknet.h. I'll think about avoiding such conflicts with CUDA headers.
hipify-clang was built from sources taken from the 'master' branch and based on LLVM 6.0.1. Nonetheless, I believe It should work on Linux as well. Tomorrow I'll let you know.
Here is my command line:
hipify-clang d:\GIT\darknet\src\activation_kernels.cu --cuda-path="c:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v7.5" -DCUDA_VERSION=7500 -Id:\GIT\darknet\include -Id:\GIT\darknet\src -DGPU -D_TIMESPEC_DEFINED -print-stats -o-dir=.
Here is a hipified result:
activation_kernels.cu.zip

Hi @rohanmahapatra,
Linux command line:
./hipify-clang ~/git/darknet/src/activation_kernels.cu -I/home/emankov/git/darknet/include -I/home/emankov/git/darknet/src -print-stats -DCUDA_VERSION=8000 -DGPU
With cuda.h to cuda_darknet.h workaround.
If you have multiple CUDA versions installed, --cuda-path should be specified.
So please check it is working on your side and also let me know of any other issues with hipification.

Sorry for the delay -
Running hipify-clang -version outputs
$ hipify-clang -version
LLVM (http://llvm.org/):
LLVM version 3.8.0
Optimized build.
Default target: x86_64-unknown-linux-gnu
Host CPU: (unknown)
I will download the master branch and run the test and let you know

Well, LLVM 3.8.0... Mmm, ok. Let's test with the oldest LLVM 3.8.0.

Ping

Hello, I am there.

What about building hipify-clang from scratch (better based on llvm+clang 6.0.1 release)? hipify-clang as I said a HIP/Rocm independent tool.

Hi,
I tried building hipify-clang from scratch with the below command and used the versions you had asked me to use.
cmake -DHIPIFY_CLANG_TESTS=1  -DCMAKE_BUILD_TYPE=Release  -DCMAKE_INSTALL_PREFIX=../dist  -DCMAKE_PREFIX_PATH=../../clang+llvm-6.0.1-x86_64-linux-gnu-ubuntu-16.04/  -DCUDA_TOOLKIT_ROOT_DIR=/p/hal/private/libs/cuda-8.0/ ..
Output is below. This was not as per what is present in the readme.
-- HIP Platform: nvcc
-- HIP Compiler: hcc
-- HIP will be installed in: /nobackup/hipify-clang/HIP/dist
-- Configuring done
-- Generating done
-- Build files have been written to: /nobackup/hipify-clang/HIP/hipify-clang
Thanks,
Rohan

Output is below. This was not as per what is present in the readme.

You're right, cause you built HIP. Please refer to Readme to build hipify-clang only. As for the building HIP, by default, it doesn't build hipify-clang at all.
And if you want to build hipify-clang with testing support, setting -DHIPIFY_CLANG_TESTS=1 only is insufficient. -DCUDA_DNN_ROOT_DIR and -DLLVM_EXTERNAL_LIT should be specified as well.

ping

I did use the README to build hipify-clang. Problem was that the output was not like what was expected.

Thank you,
Rohan

Sure, no problem, it's your ticket.

Hi,
Sorry for the delay.
I installed LLVM + Clang 6.0.1 with CUDA 8 and followed the read me steps to build Hipify-clang in testing mode. However, when I use the below command as you had told earlier, I get one error as shown below and the output does not match to what is displayed in the README.
cmake -DHIPIFY_CLANG_TESTS=1  -DCMAKE_BUILD_TYPE=Release  -DCMAKE_INSTALL_PREFIX=../dist  -DCMAKE_PREFIX_PATH=../../llvm-6.0.1.src/dist/  -DCUDA_TOOLKIT_ROOT_DIR=/p/hal/private/libs/cuda-8.0  -DCUDA_DNN_ROOT_DIR=/p/hal/private/libs/cuda-8.0/  -DLLVM_EXTERNAL_LIT=../../llvm-6.0.1.src/build/bin/llvm-lit  ..
-- Could NOT find LibXml2 (missing:  LIBXML2_LIBRARIES LIBXML2_INCLUDE_DIR)
-- Native target architecture is X86
-- Threads enabled.
-- Doxygen disabled.
-- Go bindings disabled.
-- OCaml bindings disabled, need ctypes >=0.4.
-- Could NOT find Python module pygments
-- Could NOT find Python module pygments.lexers.c_cpp
-- Found Python module yaml
-- LLVM host triple: x86_64-unknown-linux-gnu
-- LLVM default target triple: x86_64-unknown-linux-gnu
-- Building with -fPIC
-- Constructing LLVMBuild project information
-- Linker detection: GNU ld
-- Targeting AArch64
-- Targeting AMDGPU
-- Targeting ARM
-- Targeting BPF
-- Targeting Hexagon
-- Targeting Lanai
-- Targeting Mips
-- Targeting MSP430
-- Targeting NVPTX
-- Targeting PowerPC
-- Targeting Sparc
-- Targeting SystemZ
-- Targeting X86
-- Targeting XCore
-- Configuring done
-- Generating done
-- Build files have been written to: /u/m/a/mahapatra/private/gpu_project/hipify/llvm-6.0.1.src/build
[mahapatra@hylia] (98)$ make test-hipify
make: *** No rule to make target 'test-hipify'.  Stop.
[mahapatra@hylia] (99)$
Also, I installed lit using the steps provided in the README. Should I point to LIT inside LLVM or in my /usr/local/bin/

It looks like you ran cmake for llvm...

Yes, I followed the README from the master branch. I was supposed to follow the README right?
cmake
-DCMAKE_INSTALL_PREFIX=../dist
-DCMAKE_BUILD_TYPE=Release
..

So you ran cmake command intended for hipify-clang in llvm subdir, right? Well, what did you want to happen?
Building LLVM is building LLVM. Building hipify-clang is building hipify-clang. Hipify-clang depends on LLVM binaries, which might be used from packages or built from sources. Hipify-clang with testing support needs LLVM built from sources, distributive LLVM packages are insufficien in that case - it is mentioned in Readme.
Please, provide where is it written in Readme to run cmake in llvm tree for building hipify-clang (with or without testing support).

Thank you, I got it working and could match your .hip which you generated. I will let you know in case I come across some issues.

Ok


DPL-nnn Add Chromium Visium library to be Limber-Bespoke - Chromium – xxxxx  templates

Chromium Visium library exists but needs to be added to Limber-Bespoke - Chromium – xxxxx  templates
At the moment Bespoke have to manually proceed through Limber and when released the library type is missing from the batch data in MLWH
From srl
"Chromium Visium is a valid library type, we (i.e. NPG) rely on the library_type being correct to determine the appropriate downstream analysis"
Who are the primary contacts for this story
Lynda D - ld20 (CellGen SSR)
Referenced in RT730498

Library type added here: https://github.com/sanger/sequencescape/issues/2508

In other words, its associated with the bespoke PCR templates, not bespoke chromium.
Will also need to be added to the filters of the appropriate chromium pipelines.


Front Door Rule - Upper & Lower Case File Names within Storage Blob

Is there a rule set that can be applied to Azure Front Door to manage upper and lower case file names within Storage Blob? It seems that file names are case-sensitive and must be an exact match.
For example:
Uppercase in the Request URL:

URL Request: /file-name.pdf
Storage Blob File Name: File-Name.pdf

Lowercase in the Request URL:

URL Request: /File-Name.pdf
Storage Blob File Name: file-name.pdf

In both cases, you would receive the following error message:
<Error>   BlobNotFound   <Message>The specified blob does not exist.    RequestId:4b4a080f-601e-005e-4847-2323232323    Time:2024-08-22T03:55:49.6483070Z</Message> </Error>
Is there a way to address this issue so that Azure Front Door can handle case discrepancies, such as when the request URL is uppercase but the Storage Blob file name is lowercase, or vice versa? Additionally, what should be done when the file name is in all uppercase, e.g., FILE-NAME.pdf?

@KloudstackHosting Azure Front Door doesn't support changing case for an incoming URL request but only allows pattern match and replace type of functions.
However, for achieving case insensitive querying of wordpress media, you can explore WordPress extensions that convert any new media files to lowercase before uploading (This can be done by adding a simple hook to 'wp_handle_upload_prefilter' wordpress filter) and updating existing media file URLs to lowercase.

So this approach would only work if the images had been uploaded to WordPress by the media upload or placed under the uploads folder. But not if images had been uploaded to Azure blob, it would not apply the conversion.

If you're looking at converting all URLs to lowercase, then uploading to WordPress by media file upload will automatically upload it to Azure blob storage as well.
In general, uploading to Azure blob directly is not recommended since it won't be picked up by WordPress as a media file even though it is present in the blob storage. This is because WordPress also needs to a reference to the media file in the database which wouldn't happen in this case.

Closing this since you can achieve case insensitive Blob Storage URLs by adding a WordPress hook to convert media files to lowercase name.


Question: possibility to connect 'locations'

Let's asume you have the following setup:

Location 1,2,3 should not be able to talk to each other
A client from Location 4 can connect/talk to one client from location 1-3
Location 4 should get the status if a client from location 1-3 is connected (in this example location 2 is offline)

That operation is not supported, and not something I'd consider worth supporting.
This VPN is designed specifically to allow multiple locations to cross-talk.  That's pretty much implicit because of the way we setup the routing - we route the whole shared network-range to all peers.  You could cripple connections by setting up routes as /32s, but that would get hard to manage.
I'd be tempted to say use firewalling, but that's also fragile for the same reason.
In short: The software cannot be setup to handle only partial connectivity, and I think the same is true for most of the simple alternatives such as tinc, etc.

PS.  Great use of an image though, that made it 100% obvious what you meant :)

Thank for your fast reply!
Actually we got a commercial solution but it costs too much and the performance is very bad.

PS. Great use of an image though, that made it 100% obvious what you meant :)

Made it with mermaid in vscode. But theres also an online editor. see: https://mermaidjs.github.io/mermaid-live-editor


Fix generate np

Re-opens the PR #858 with a rebase.

A bunch of the tests were written incorrectly and should never have been passing to begin with. I'm going to fix them in this PR so at least they pass, but someone later should go back through and figure out what was intended to happen.

Except for this crash, everything passes terminate called after throwing an instance of 'std::bad_alloc'

It looks like it's failing for both tf 1.12 builds but none of the other builds

Rebase broke the history.
The diff still looks somewhat unhappy because I had to merge the two methods together, but it looks somewhat nicer now.

Again this issue "terminate called after throwing an instance of 'std::bad_alloc". Going to retry.

The test is now getting killed, but everything is passing.
I don't know that much about travis do debug this one. Any suggestions @npapernot @goodfeli?

I restarted the test to see if it passes. Travis sometimes encounters issues reporting back the result of tests

It failed again. Interestingly it is consistently just 2.7 on TF 1.8 that is failing here.

Looks like the test commented out was the faulty one

Okay so this is very confusing. By removing a test that I never called, the tests pass. Let me put those lines back in and see if it works again.

I am 90% sure this is just a travis issue. I installed TF 1.8 on Python 2.7 and I can run the tests successfully.
If it's okay with you @npapernot @goodfeli I'm going to comment out this test again so we can merge this commit, and then investigate further.

I have removed some duplicated work that was going on in the tests. This should allow the tests to pass, but in the future we will need to find a way to make the tests faster or this will happen again.

Thanks, hopefully we will make progress towards migrating to CI with GPUs

This broke generate...

What does it break?

The new assert only works if eps is numpy/tensorflow. If it's a float, the new assert causes an AttributeError while trying to read its dtype.


Are you out of maintenance?

This is a very good product, but some functions seem to be unavailable, for example:auto-layout

Glad you like it, however I have been too busy to maintain it. I'm open for contributions however :)

Nice.... I love this


Customize frontend routes

Is there any way to drop the /plugin part of a route exposed by a plugin?
Suppose I have a plugin called About. Can I have the frontend use /about/something/list?

Yes, you can do it by routes_main.txt
Sample:
scope "client", module: "plugins/tusqpones/client", as: "client" do
resources "promotions", controller: "promotions", path: "promotions"  do
end

end
This will accept your route and use your controllers like this:
http://localhost/client
http://localhost/client/promotions
http://localhost/client/promotions/....
Regards!

Thanks! I figured I could use the Rails routing parameters and methods.
On a sidenote, why do these files have a .txt extension instead of a.rb?

Because ruby throws an error with:
"not exist method get for ...."
for this simple sample:
get "index" => :index
This is because it is not wrapped with any routes.draw

Is it not possible for camaleon plugins to use a routes.draw like Rails engines?

Yes, I am thinking in that for new versions!

:+1:


operations_research_constraint_solverPINVOKE

The type initializer for 'Google.OrTools.ConstraintSolver.operations_research_constraint_solverPINVOKE' threw an exception.
System.TypeInitializationException: The type initializer for 'SWIGExceptionHelper' threw an exception. ---> System.DllNotFoundException: Unable to load DLL 'Google.OrTools.runtime.win-x64.dll': The specified module could not be found. (Exception from HRESULT: 0x8007007E)
I had this exception and search all same problem and my solution was:
The problem was that the nuget packages was in user folder, in my case it's a service who executes the dll, I changed de user in the service and work
C:\Users"user".nuget\packages\google.ortools.runtime.win-x64\7.5.7466

OR-Tools is splitted in, at least, two package a runtime platform dependent package providing the C++ and swig wrapper library, and a "pure" .Net package depending of this package.
Please take a look at https://github.com/Mizux/dotnet-native if you want to understand how it is working on a simpler project...


Sync eng/common directory with azure-sdk-tools for PR 3457

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/3457  See eng/common workflow

/azp run python - storage - tests, python - keyvault - tests, python - eventhub - tests

/check-enforcer override


4Ch PRO

Hello,
I just wanted to check if this integration will work for 4Ch Pro version 3.4.1 ?
I have following config on my home assistant but I don't see any entities pulled from cloud.
sonoff:
username<EMAIL_ADDRESS>password: 123456
mode: local
reload: always  # update device list every time HA starts
thanks for your response.

Edit: after a while when I switched on the relays on ewelink app the entities appeared in Home assistant, sorry for the question..

I've noticed, every time I restart Home assistant server, I have to switch on one channel from ewelink app before the switches on HA will be enabled, anyone facing similar issue. Appreciate your advise.
thanks.

Remove line mode local from config


Licence for models

Thank you for all your work on these models!
We are wondering what the license is of the pretrained models. We are building a free online drawing tool to integrate and use together many models in one place, and would love to include some of these.

@aman-tiwari , there is an MIT license associated with repository!


[Feature]: Make networkId parameter optional

Is your request related to a problem?
We would like to migrate to Onboard but our application is multi-chain and generally defaults to using the chain that the user's wallet is already connected to.  Tying the application to a single networkId and requesting that the user switches chains upon connecting their wallet may lead to a poor UX by requiring an unnecessary extra step.
Feature Description
Allow the networkId parameter to be optional when instantiating onboard and do not make a switch chain RPC call when networkId is omitted.
Apologies if this is already possible and thanks in advance!
Alternative Solutions
No response
Anything else?
No response

+1 I also would like this feature. I was thinking in proposing it, but you have done it first. :)

@morozj01 @imsys In Onboard v2 you provide an array of networks that your dapp supports. I believe that would address the issue you are experiencing? If so, we suggest waiting for the O2 release as this feature will be there by default.
Below is an example from the README:
import Onboard from '@bn-onboard/core'
import injectedModule from '@bn-onboard/injected-wallets'

const ETH_MAINNET_RPC = `https://mainnet.infura.io/v3/${INFURA_KEY}`
const ETH_RINKEBY_RPC = `https://rinkeby.infura.io/v3/${INFURA_KEY}`
const MATIC_MAINNET_RPC = 'https://matic-mainnet.chainstacklabs.com'

const injected = injectedModule()

const onboard = Onboard({
  wallets: [injected],
  chains: [
    {
      id: '0x1',
      token: 'ETH',
      label: 'Ethereum Mainnet',
      rpcUrl: ETH_MAINNET_RPC
    },
    {
      id: '0x4',
      token: 'rETH',
      label: 'Ethereum Rinkeby Testnet',
      rpcUrl: ETH_RINKEBY_RPC
    },
    {
      id: '0x89',
      token: 'MATIC',
      label: 'Matic Mainnet',
      rpcUrl: MATIC_MAINNET_RPC
    }
  ],
  appMetadata: {
    name: 'Token Swap',
    icon: myIcon, // svg string icon
    description: 'Swap tokens for other tokens',
    recommendedInjectedWallets: [
      { name: 'MetaMask', url: 'https://metamask.io' },
      { name: 'Coinbase', url: 'https://wallet.coinbase.com/' }
    ]
  }
  i18n: {
    en: {
      connect: {
        selectingWallet: {
          header: 'custom text header'
        }
      }
    }
  }
})

@morozj01 I was looking for something similar. I put in the required networkId as 1 (Ethereum), though our application also supports 137 (Polygon).
I would prefer to listen to the user's wallet's network and display "Invalid Network" if they are connected to an unsupported network. For v1 Onboard, I just did the following to bypass the network check:
const onboard = Onboard({
  ...
  networkId: 1,
  walletCheck: [{checkName: "connect"}]
})


Hasone tests and improvements, take 2

@pxpm I've created this PR to help you continue working on https://github.com/Laravel-Backpack/CRUD/pull/3322
Hope it helps.

A new inspection was created.

Should we merge this in https://github.com/Laravel-Backpack/CRUD/pull/3250?
And test everything together?

The inspection completed: 9 new issues, 14 updated code elements

@tabacitu thanks for your effort.
After clicking some buttons in Fork I endup merging everything in #3250 so I think this branch is even with the other.
For that I am closing this one to avoid confusion.
Thanks again


Lots of ortholog missing gene warnings on startup

It's okay if genes are missing, we don't have all taxa, but it's just too many warnings. Maybe reduce the level to debug or something.

It's not as bad now, so maybe revisit this in a patch release.


Autotile bug with walls (3*3 bitmask)

Godot version:
3.1.stable.official
Issue description:
Wrong behavior, when autotiles being placed close to corners:

Bitmask:

Expected:

Minimal reproduction project:
autotile_bug.zip

Duplicate of #17063.


Consistent 'checksum mismatch' errors when trying to install certain MRI versions

I'm using ruby-build v20170201, I've used rbenv and ruby-build in production for several years now. Recently, attempts to install new-to-us versions of Ruby (2.0.0-p353, 2.4.0) have consistently failed with checksum mismatch errors:
owen@mybox:~$ rbenv install 2.4.0
Downloading ruby-2.4.0.tar.gz...
-> http://dqw8nmjcqpjn7.cloudfront.net/440bbbdc49d08d3650f340dccb35986d9399177ad69a204def56e5d3954600cf
-> https://cache.ruby-lang.org/pub/ruby/2.4/ruby-2.4.0.tar.bz2

BUILD FAILED

Inspect or clean up the working tree at /tmp/ruby-build.20170317212156.18771
Results logged to /tmp/ruby-build.20170317212156.18771.log

Last 10 log lines:

checksum mismatch: ruby-2.4.0.tar.gz (file is corrupt)
expected 440bbbdc49d08d3650f340dccb35986d9399177ad69a204def56e5d3954600cf, got 98f29161860fe1b6c65396717847a358

curl: (33) HTTP server doesn't seem to support byte ranges. Cannot resume.

checksum mismatch: ruby-2.4.0.tar.gz (file is corrupt)
expected 440bbbdc49d08d3650f340dccb35986d9399177ad69a204def56e5d3954600cf, got 98f29161860fe1b6c65396717847a358

First, I notice that Cloudfront is redirecting us to a tar.bz2 file, but it seems that ruby-build is saving it as a tar.gz. So:
owen@mybox:~$ file /tmp/ruby-build.20170317212410.18947/ruby-2.4.0.tar.gz
/tmp/ruby-build.20170317212410.18947/ruby-2.4.0.tar.gz: bzip2 compressed data, block size = 900k

So if I just short-circuit the ruby-build checksum code, it still fails to install because of that assumption. But as for the checksumming, I looked through the ruby-build code and I see that it may use a couple of different methods to do that. This is an Ubuntu Trusty system, I'm having the same problem on Precise hosts as well. So ruby-build thinks the checksum for this file should be 440bbbdc49d08d3650f340dccb35986d9399177ad69a204def56e5d3954600cf. But:
owen@mybox:~$ md5sum /tmp/ruby-build.20170317212410.18947/ruby-2.4.0.tar.gz
98f29161860fe1b6c65396717847a358  /tmp/ruby-build.20170317212410.18947/ruby-2.4.0.tar.gz
owen@mybox:~$ openssl md5 /tmp/ruby-build.20170317212410.18947/ruby-2.4.0.tar.gz
MD5(/tmp/ruby-build.20170317212410.18947/ruby-2.4.0.tar.gz)= 98f29161860fe1b6c65396717847a358

Both md5sum and openssl agree about the checksum of what we downloaded, so my sense is that this is not a problem with my local tools.

Oh, interesting... the systems I'm using are Amazon EC2 instances... I just realized that the ruby-build definitions don't specify a Cloudfront URL, so perhaps Amazon is redirecting my request for me, and this is causing breakage.

Closing as stale


small addition of tags and categories + home page

add loc for the home page with current date and tags/categories if configured in the _config.xml

Coverage remained the same at 100.0% when pulling cb5dfebe0feef8c4f4db972037d257b04d2f0647 on achauvinhameau:evol-tags-and-cats into a45bedd6b7c9d2f8b82bbc9702bd1ab358114ddb on hexojs:master.

Could you please make the travis ci happier? 😄

Coverage remained the same at 100.0% when pulling 64cf5027ade752f6bc2906e45af4d8b77a659576 on achauvinhameau:evol-tags-and-cats into a45bedd6b7c9d2f8b82bbc9702bd1ab358114ddb on hexojs:master.

the failling test in travis is linked to a change I've made for the root url (adding date). Let you check wether you want to keep it.

I see. Could you please also fix the test file and add test cases for your changes? This will make the future development easier. 😄

Coverage remained the same at 100.0% when pulling e7e657206cc3df0ff3251e031fe6d6f9fc8cf4c5 on achauvinhameau:evol-tags-and-cats into a45bedd6b7c9d2f8b82bbc9702bd1ab358114ddb on hexojs:master.

Coverage remained the same at 100.0% when pulling f76a8107f57bf279aaca49b89cf5fd66297e4c25 on achauvinhameau:evol-tags-and-cats into a45bedd6b7c9d2f8b82bbc9702bd1ab358114ddb on hexojs:master.

Coverage remained the same at 100.0% when pulling 3a87811fcff70d799c5c58871cba1c71613fcdd5 on achauvinhameau:evol-tags-and-cats into a45bedd6b7c9d2f8b82bbc9702bd1ab358114ddb on hexojs:master.

Thank you. LGTM.

Adding loc for the home page is nice,  is there anyone who would click merge button ?

什麼時候才能增加 tags 和 category 啊

Rebased.

@curbengh Ops. Seems that it needs to be rebased again.

Rebased. Updated sNow to be date only.

Approved.