installing RDO packages breaks OSP installs

ok: [localhost] => {"changed": false, "msg": "", "rc": 0, "results": ["https://rdoproject.org/repos/rdo-release.rpm: Nothing to do"]}

nInstall  4 Packages (+3 Dependent packages)\nUpgrade  3 Packages (+2 Dependent packages)\n\nTotal size: 1.7 M\nDownloading packages:\nRunning transaction check\nRunning transaction test\n"]}
msg:

Transaction check error:
  file /usr/lib/python2.7/site-packages/keystoneauth1/__init__.py from install of python2-keystoneauth1-2.4.1-1.el7.noarch conflicts with file from package python-keystoneauth1-1.1.0-4.el7ost.noarch

Must allow for either.

This check fails on the infra host:
https://github.com/redhat-openstack/openshift-on-openstack/blob/master/templates/var/lib/ansible/playbooks/registry.yml#L18-L21
  - name: Setup RDO repositories
    yum: name=https://rdoproject.org/repos/rdo-release.rpm state=present
    when: deployment_type == 'origin' and rpm_check.rc != 0
when: ansible_first_run | default(false) | bool

that "when" seems to succeed on the infra host, does the infra host not have a deployment_type?

bump.  Any chance I could get a little help on this?  It's blocking my release.

Sorry for the delay. Could you please give this PR a try : https://github.com/redhat-openstack/openshift-on-openstack/pull/157 ?

It fails.
TASK: [Setup RDO repositories] ************************************************
<localhost> ESTABLISH CONNECTION FOR USER: cloud-user
<localhost> REMOTE_MODULE yum name=https://rdoproject.org/repos/rdo-release.rpm state=present
<localhost> EXEC ssh -C -tt -vvv -o ControlMaster=auto -o ControlPersist=60s -o ControlPath="/root/.ansible/cp/ansible-ssh-%h-%p-%r" -o StrictHostKeyChecking=no -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o User=cloud-user -o ConnectTimeout=10 localhost /bin/sh -c 'mkdir -p $HOME/.ansible/tmp/ansible-tmp-1466630446.45-92239858464923 && echo $HOME/.ansible/tmp/ansible-tmp-1466630446.45-92239858464923'
<localhost> PUT /tmp/tmpJMKyLS TO /home/cloud-user/.ansible/tmp/ansible-tmp-1466630446.45-92239858464923/yum
<localhost> EXEC ssh -C -tt -vvv -o ControlMaster=auto -o ControlPersist=60s -o ControlPath="/root/.ansible/cp/ansible-ssh-%h-%p-%r" -o StrictHostKeyChecking=no -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o User=cloud-user -o ConnectTimeout=10 localhost /bin/sh -c 'sudo -k && sudo -H -S -p "[sudo via ansible, key=oxpuzwsftdembxeeerswewwmzwmvokbo] password: " -u root /bin/sh -c '"'"'echo BECOME-SUCCESS-oxpuzwsftdembxeeerswewwmzwmvokbo; LANG=C LC_CTYPE=C /usr/bin/python -tt /home/cloud-user/.ansible/tmp/ansible-tmp-1466630446.45-92239858464923/yum; rm -rf /home/cloud-user/.ansible/tmp/ansible-tmp-1466630446.45-92239858464923/ >/dev/null 2>&1'"'"''
changed: [localhost] => {"changed": true, "msg": "", "rc": 0, "results": ["Loaded plugins: search-disabled-repos\nExamining /var/tmp/yum-root-8lP9oQ/rdo-release.rpm: rdo-release-mitaka-2.noarch\nMarking /var/tmp/yum-root-8lP9oQ/rdo-release.rpm to be installed\nResolving Dependencies\n--> Running transaction check\n---> Package rdo-release.noarch 0:mitaka-2 will be installed\n--> Finished Dependency Resolution\n\nDependencies Resolved\n\n================================================================================\n Package             Arch           Version          Repository            Size\n================================================================================\nInstalling:\n rdo-release         noarch         mitaka-2         /rdo-release         1.4 k\n\nTransaction Summary\n================================================================================\nInstall  1 Package\n\nTotal size: 1.4 k\nInstalled size: 1.4 k\nDownloading packages:\nRunning transaction check\nRunning transaction test\nTransaction test succeeded\nRunning transaction\n  Installing : rdo-release-mitaka-2.noarch                                  1/1 \n  Verifying  : rdo-release-mitaka-2.noarch                                  1/1 \n\nInstalled:\n  rdo-release.noarch 0:mitaka-2                                                 \n\nComplete!\n"]}

Fixed by https://github.com/redhat-openstack/openshift-on-openstack/pull/157


Should we add winrt::Object to simplify some use cases?

namespace winrt
{
    using Object = winrt::Windows::Foundation::IInspectable;
}

enables
    auto map = winrt::multi_threaded_observable_map<winrt::hstring, winrt::Object>().
        as<winrt::IMap<winrt::hstring, winrt::Object>>();

instead of
    auto map = winrt::multi_threaded_observable_map<winrt::hstring,
        winrt::Windows::Foundation::IInspectable>().
        as<winrt::IMap<winrt::hstring, winrt::Windows::Foundation::IInspectable>>();

This could conflict with somebody using a namespace named Object. Feels like something that should be defined in the project's source code, not cppwinrt. Personally, I do namespace wf = winrt::Windows::Foundation; so I can type wf::IInspectable.

Yep, adding such "helper" types/aliases is not recommended. You can always add the type alias in your own projects/libraries e.g. WIL. If anything, C++/WinRT needs to be trimmed down further to only include code generated from metadata.

This could conflict with somebody using a namespace named Object. Feels like something that should be defined in the project's source code, not cppwinrt. Personally, I do namespace wf = winrt::Windows::Foundation; so I can type wf::IInspectable.

Note that when you're inside a runtime class, you can directly use IInspectable instead of winrt::Windows::Foundation as a type alias exists, even without using Windows::Foundation.


Hello, I'm interested in contributing to this project by offering a translation to the changelog, readme .md files, let me know if interested.

/cc @SteveL-MSFT @joeyaiello

@LuigiBaute this would be great!  Thanks for the offer!  What languages would you support?

@SteveL-MSFT Spanish if thats OK with you guys I'll start asap! :+1:

Just waiting on @joeyaiello to define how we should separate out different languages.


A question about experimental result

How to compute the number behind mean accuracy? is that standard deviation or others?
And what's the number of epoches in the datasets other than IMDB-B？
Thank you very much!

Our code can directly compute it. Yes, it is standard deviation.
What do you mean by "the number of epoches in the datasets"?

I want to know the setting of the number of epochs for each dataset.
Because I set the epochs=1000, but can't reproduce your result

I just can reproduce the result of IMDB-B

You can find all the commands in script.
For IMDB-B, we set epoch=100.


Add Annotations setting context manager

This will allow us to more easily set Annotations for all Artifacts initiated within the block, including those generated by Producers. Ex:
with Graph(...) as g:
    with Annotation.include(<Annotation instances>): # probably better name
        g.artifacts.a = A(...)
        g.artifacts.b = compute(a=a)

--
Something similar for Statistics and Thresholds would be interesting, but may be a bit trickier (eg: a Statistic may not work for all types, etc).

We can likely use the arti.context and track a stack of Annotations. On Artifact initialization (or rather, in a pydantic field "validator"), we'll concatenate those explicitly passed with those in the stack.


add publication's controller and some tests

I have added publication's controllers and some tests.
Plz check it.

OK. I agree to merge it


Feature: Route lines across x OR y axis with 90 degree joints

Is it possible to create a type of policy or feature which would force lines to be either vertical or horizontal and add 90 degree joints as necessary to reach a linked object? Similar to the default behavior of draw.io / lucidchart.
Something like this:

Instead of this:

Here are the draw.io pathfinding/line structure options for reference:

I realize this could be accomplished by the user manually long-pressing the line to add a joint but the application I am building shouldn't require the user to manually route, if possible. How hard would this be to do?

It would be really nice to have a policy inside the package that could be able to make these kind of connections. However, it's not an easy task to compute a path of the lines in all cases. And I can't think of an easy way how to add it to the package.
It's still possible to achieve the required behaviour. I recommend you to compute the position of the points after creating the connection between two components. And compute the same while moving any component.

I created a code sample just for one of the cases (x-y-x). Please try to replace these functions in the example and play with it a little. Hopefully it will help you to make all the cases to work.
  @override
  onComponentScaleUpdate(componentId, details) {
    Offset positionDelta = details.localFocalPoint - lastFocalPoint;
    canvasWriter.model.moveComponent(componentId, positionDelta);
    final c1 = canvasReader.model.getComponent(componentId);
    for (final connection in c1.connections) {
      final c2 = canvasReader.model.getComponent(connection.otherComponentId);
      late final Offset p1;
      late final Offset p2;
      if (connection is ConnectionOut) {
        p1 = c1.position + c1.getPointOnComponent(Alignment.center);
        p2 = c2.position + c2.getPointOnComponent(Alignment.center);
      } else {
        p2 = c1.position + c1.getPointOnComponent(Alignment.center);
        p1 = c2.position + c2.getPointOnComponent(Alignment.center);
      }

      canvasWriter.model.setLinkMiddlePointPosition(connection.connectionId, Offset((p1.dx + p2.dx) / 2, p1.dy), 1);
      canvasWriter.model.setLinkMiddlePointPosition(connection.connectionId, Offset((p1.dx + p2.dx) / 2, p2.dy), 2);
    }
    canvasWriter.model.updateComponentLinks(componentId);
    lastFocalPoint = details.localFocalPoint;
  }

// This function tests if it's possible to connect the components and if yes, connects them
  bool connectComponents(String? sourceComponentId, String? targetComponentId) {
    if (sourceComponentId == null || targetComponentId == null) {
      return false;
    }
    // tests if the ids are not same (the same component)
    if (sourceComponentId == targetComponentId) {
      return false;
    }
    // tests if the connection between two components already exists (one way)
    if (canvasReader.model
        .getComponent(sourceComponentId)
        .connections
        .any((connection) => (connection is ConnectionOut) && (connection.otherComponentId == targetComponentId))) {
      return false;
    }

    // This connects two components (creates a link between), you can define the design of the link with LinkStyle.
    String linkId = canvasWriter.model.connectTwoComponents(
      sourceComponentId: sourceComponentId,
      targetComponentId: targetComponentId,
      linkStyle: LinkStyle(
        arrowType: ArrowType.pointedArrow,
        lineWidth: 1.5,
        backArrowType: ArrowType.centerCircle,
      ),
    );
    final c1 = canvasReader.model.getComponent(sourceComponentId);
    final c2 = canvasReader.model.getComponent(targetComponentId);
    final p1 = c1.position + c1.getPointOnComponent(Alignment.center);
    final p2 = c2.position + c2.getPointOnComponent(Alignment.center);

    canvasWriter.model.insertLinkMiddlePoint(linkId, Offset((p1.dx + p2.dx) / 2, p1.dy), 1);
    canvasWriter.model.insertLinkMiddlePoint(linkId, Offset((p1.dx + p2.dx) / 2, p2.dy), 2);

    canvasWriter.model.updateLink(linkId);

    return true;
  }

That actually turned out to be perfect for what I needed. Thank you for the really quick help with this!


add comment

Hi @SlainTroyard , please make a PR to the dev branch. If this is for APR-COMP, please do include it in the PR name. For an initial commit, do make a placeholder driver


fixed Maven warnings, updated plugin versions

This PR fixes Maven warnings (e.g. missing groupId) and updates several plugin versions.
We will also submit a separate PR which adds support for batch files in component/pom.xml and native/pom.xml (i.e <executable>${basedir}/make_component.sh</executable>) when executing mvn clean package on Microsoft Windows (see my fork).

I sent the OCA to oracle-ca_us -(at)- oracle -(dot)- com.

Partially redundant with #51 and #52; the three PRs should be merge in one step.


Users can invite other users

As a user, I may want to pull in other people into my project. Yes, they can easily click the "JOIN" button, but in some cases it would be nice to do this myself. In particular, if we create a new project page linked to a challenge, and want to quickly pull in the team.
Perhaps this is easiest to do through an server-sent e-mail: the user puts in their team-mates username or e-mail, and they get an invitation link to join the project.
This would build on the administrator feature added in #351
Proposed by a participant of the UPU Hackathon

Obviously, admins should also be able to use this feature. In fact, having a "share link" to invite people to join a project (creating an account, if necessary, on their way) would be extremely helpful.
The main thing standing in the way, is a smart redirect from the login controller.

This is already possible.


[#165245540] Remove cflinuxfs2

What
Cflinuxfs2 (trusty) is due to go out of support at the end of the month. This removes the cflinuxfs2 stack our deployment.
Before Merging

Ensure all apps are running cflinuxfs3 using the stack auditor
Delete the cflinuxfs2 buildpacks
Delete the the cflinuxfs2 stack

How to review

Deploy to your dev env

Who can review
Not me

I don't think the commit that touches config/buildpacks.yml is necessary - this file is just a cache of the current buildpack manifests (maybe shouldn't be in version control in the first place), and it's auto-generated by ./scripts/update_buildpacks.sh. I'll drop the commit in a force-push.

My previous comment was incorrect, as confirmed by trying to run this down my dev environment.
config/buildpacks.yml is actually used by the pipeline to work out which buildpacks it needs to upload. If it still has the cflinuxfs2 ones then they'll fail to upload once the stack is removed.
I've pushed another commit up to remove the cflinuxfs2 buildpacks from the config.


[content] contributor changelog & news

can we pull in all of the changes that we are proposing and those that are landing in a contributor changelog - could use k-dev for this automation? could be a 'news' section and that's one of the sections. in a news area, could also pull in the chairs/tls email, any changes that are made to sigs.yaml (new groups, chairs, etc.).

/remove-lifecycle stale

I'm not sure how much we can automate with this. It's very difficult to automatically pick things that are 'news worthy'.  As an example for picking up sigs.yaml changes, you could use git log, but the content itself would simply be the git commit messages and you'd get a mix of things like:
Update steering meeting time to Monday 11am PT
Add myself to the CoCC
Update reference-docs repo location

/remove-lifecycle stale

could we automate it to the point were its a stream of things that someone can approve/reject/modify? and then it would save time manually looking at all of those places to curate something.

ok then ill type up a list of things that someone will need to manually check on a weekly basis to pull in for curation


Create  ortarad.txt

An Ort (school network) high-school in Arad, Israel.
School Website (The website is written in Hebrew): http://arad.ort.org.il/
School E-Mail Format (A different subdomain): [Generation]-[First Name][1-3 Letters of Last<EMAIL_ADDRESS>My School E-Mail<EMAIL_ADDRESS>

@SirCodalot I am sorry for the delay with processing your request. It is now merged. Thank you!


Clean up

Use new deal.II functionality
Fix typos
Prefix private variables with an underscore (up to now, that was only partially done)
Replace BOOST_FOREACH by a regular for loop

Codecov Report

Merging #257 into master will increase coverage by <.01%.
The diff coverage is 99.09%.

@@            Coverage Diff             @@
##           master     #257      +/-   ##
==========================================
+ Coverage   94.99%   94.99%   +<.01%
==========================================
  Files          72       72
  Lines        5115     5116       +1
==========================================
+ Hits         4859     4860       +1
  Misses        256      256

Impacted Files
Coverage Δ

cpp/source/deal.II/post_processor.h
90% <ø> (ø)
:arrow_up:

cpp/source/deal.II/electrochemical_physics.h
100% <ø> (ø)
:arrow_up:

cpp/source/deal.II/supercapacitor.h
100% <ø> (ø)
:arrow_up:

cpp/source/utils.templates.h
100% <100%> (ø)
:arrow_up:

...source/deal.II/electrochemical_physics.templates.h
98.62% <100%> (ø)
:arrow_up:

cpp/source/deal.II/post_processor.templates.h
99.59% <100%> (ø)
:arrow_up:

cpp/source/deal.II/supercapacitor.templates.h
85.21% <98.79%> (+0.05%)
:arrow_up:


How to change the indent size?

it's set to 4, but only seems to indent 2 no matter what number is there.

Hello saradeakin, thanks for reporting this issue.
I can't reproduce the same issue on my side. Also, I tested a lot of different combinations with the following settings:
VS Code settings
"editor.tabSize": 4,
"editor.insertSpaces": true,

Sass formatter
"sassFormat.indent": 4,

And everything works fine on my side.
Can you try to disable all other extensions in VS Code, and leave only Sass formatter as active? Also, please double check above settings on your side.
Thanks


Fix attempt - save public list order changes on publish

Fixes
Editing item order of public(not edited) list, in the publish form, doesn't save on publish.
Reason
unSavedEdits weren't checked for in collection params creation.

Thanks!


Bind custom command in vscode extension

I am building an extension using typescript where I need to have a custom logic before save command is executed.
 context.subscriptions.push(
    vscode.commands.registerCommand("workbench.action.files.save", () => {
       /*
          1. Execute custom logic
          2. save the file
       */
    })
  );

Also there is a save command in the menu bar. How can I hook these commands to my extension.ts so that my custom logic gets executed?

How does this relate to VS Code samples? You should ask questions like this at StackOverflow or VS Code Dev Slack.
You do not want to override the native save command. You probably just need to use workspace.onWillSaveTextDocument.


nullpointerexception on signinwithemailandpassword Firebase Authentication

Welcome to FirebaseUI and thanks for submitting an issue!
Please take a look at open issues, as well as resolved issues, to see if your issue is either already being addressed, or has been solved by someone else.
If not, please feel free to fill in the following info so we can help faster!
Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a GitHub issue.
For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device:google nexus 5 _____
Android OS version: _____
Google Play Services version: _____
Firebase/Play Services SDK version: _____
FirebaseUI version: ____

Step 3: Describe the problem:
i've been getting null pointer exception error and i'm unable to solve this. help please
cant figure out whats wrong .
Steps to reproduce:

Observed Results:

What happened?  This could be a description, logcat output, etc.

Expected Results:

What did you expect to happen?

Relevant Code:
My code is as follows:
 private EditText AdminID;
    private EditText Password;
    private Button Login;
    Administrator info;
    String Adminid;
    String Pass;
    private ProgressDialog verifylogin;
    private FirebaseAuth mAuth;
    @Override
    protected void onCreate(Bundle savedInstanceState)
    {
        super.onCreate(savedInstanceState);
        requestWindowFeature(Window.FEATURE_NO_TITLE);
        setContentView(R.layout.activity_login);
        mAuth = FirebaseAuth.getInstance();
        refids();
        verifylogin = new ProgressDialog(this);
        info = new Administrator();
        Login.setOnClickListener(this);
    }
    @Override
    public void onStart() {
        super.onStart();
        // Check if user is signed in (non-null) and update UI accordingly.
        FirebaseUser currentUser = mAuth.getCurrentUser();
    }

    @Override
    public void onClick(View view)
    {
        validate();

    }

    public void validate()
    {

        Adminid = AdminID.getText().toString();
        Pass=Password.getText().toString();

       if(Pass.isEmpty() || Pass.contains(" "))
       {
           Password.setError("Invalid Password");
       }
       else if(Adminid.isEmpty() || Adminid.contains(" ") || !Adminid.contains("@"))
       {
           AdminID.setError("Invalid Admin ID");
       }
       else
       {
           validateUser();
       }
    }

    public void refids()
    {
        AdminID = findViewById(R.id.Admin_ID);
        Password = findViewById(R.id.Password);
        Login = findViewById(R.id.Login);
    }

    private void validateUser()
    {
        verifylogin.setMessage("Verifying, Please Wait");
        verifylogin.show();
            mAuth.createUserWithEmailAndPassword(Adminid, Pass)
                    .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
                        @Override
                        public void onComplete(@NonNull Task<AuthResult> task) {
                            if (task.isSuccessful()) {
                                // Sign in success, update UI with the signed-in user's information
                                Log.d(TAG, "signInWithEmail:success");
                                FirebaseUser user = mAuth.getCurrentUser();
                            } else {
                                // If sign in fails, display a message to the user.
                                Log.w(TAG, "signInWithEmail:failure", task.getException());
                                Toast.makeText(LoginActivity.this, "Authentication failed.",
                                        Toast.LENGTH_SHORT).show();
                            }

                        }
                    });
        }

@gulzi619 can you show the stack trace of the exception?  It's hard to debug from just posting all of your code without knowing what happens and what you expect to happen.

Since this issue did not follow the template and there has been no response in ~8 days, closing it.  @gulzi619 I would recommend finding help on StackOverflow!


Cannot see Lookup Rollup Summary Custom Metadata Type to add to change

When I go to Setup > Platform Tools > Custom Code > Custom Metadata Types, I see Lookup Rollup Summary installed by the installed package Declarative Lookup Rollup Summaries Tool.
However, when I try find this component to add it to my Change Set, it is not listed.  Other Custom Metadata Types are listed but not LRS.  I can also see all of the DLRS Apex Triggers, Apex Classes, Custom Fields.
Is there something I'm missing when it comes to migrating DLRS config using change sets?

I've run into the same issue. Would love some help!

FYI, CustomMetadata entries only show up in Change Sets for non-managed packages. Change your changeset option to this instead:

FYI, CustomMetadata entries only show up in Change Sets for non-managed packages. Change your changeset option to this instead:

So , I was missing something. :-)  Found it.  This makes perfect sense now.  Thank you @cumulus-robert !!


Issue with generating data on table snwd_so_inv_item

Hello,
I'm working on this tutorial:
https://github.com/sap-tutorials/Tutorials/issues/3210
And I can not generate data on table snwd_so_inv_item, I search a little and the closer I got was to run transaction: STC01 and then Tasklist: SAP_BASIS_EPM_OIA_CONFIG
I figured out the parameters but I would need confirmation on what to put in "Recereate Master Data", I don't know if it's specting a Yes o Y o No or N or else. Anyway if i leave it blank task list runs ok, but still no data on table snwd_so_inv_item.
I'm stuck there, thanks for any input.

@castrogeneris The link you have provided does not point to a tutorial. Kindly add the correct link. Thank you.

@castrogeneris The link you have provided does not point to a tutorial. Kindly add the correct link. Thank you.

I have found the solution!
Thanks for the correction. This is the link of the tutorial:
https://developers.sap.com/tutorials/abap-display-data-queries.html
The shown transaction and parameters helped me:


⚠ Bofrost.lu has degraded performance

In cf68b56, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1041 ms

Resolved: Bofrost.lu performance has improved in 3ed4b8e after 8 minutes.


Updated RandomSecret to handle KV Secret Engine V2. CRD Updated with …

…isKVSecretsEngineV2 field to indicate KV Secret Engine V1 or V2 for creating with proper payload data type.
Closes Issue #22

LGTM


Issues in getting started guide

The portable command to build RISE V2G with maven should be mvn package, not maven package like in the wiki entry under https://wiki.eclipse.org/RISE_V2G.
Btw: Since the repo https://github.com/eclipse/risev2g.git is obsolete, the clone instruction should be updated to the new github repository.

As long as this getting started guide is used by both repositories, I cannot update the github repo information. I added a note in the README file in this repository.

I am currently working on a video-based documentation for RISE V2G. Should be available by end of October. As soon as it's available, I will change the link to the documentation, including a getting started guide for Maven.

The updated documentation is now available as a video-based online course: https://courses.v2g-clarity.com/p/risev2g-basics


Make navigator mode use bank names, not numbers

How does this interact with navigator mode and page up/page down

Navigator mode used bank names, but was not intelligent about it, and did lot of computations.
Refactored to not rely on these


Admin enforced permissions

Add to the app settings a setting that the application administrator can select the available permissions that an organization can give to it's teams. These settings will then form what permissions an organization admin can assign.

added 3m of time spent

marked this issue as related to #20


Enable selection of Elasticsearch instance types.

Issue #, if available:
#599
Description of changes:
My attempt at adding independent selection of ES instance types.  I couldn't get the dev/test environment to build, so this is untested but hopefully helpful.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @dthvt
This change has been incorporated in release v5.5.0
Thanks for your contribution


Add Butterknife view bindings to MainActivity class

Before you start refactoring, evaluate if it is necessary to add Butterknife view bindings.

[ ] Add Espresso UI tests

[ ] Refactor tests

Have checked and it seems possible to add view bindings to this class.


[FEAT] Split up DSN config

Is your feature request related to a problem? Please describe.
✅ Trying to deploy to AWS App Runner and it accepts secrets as env.
🚫 However, it does not allow the composition of secret values from JSON, which is not RDS stores the password in Secrets Manager. Some other AWS services do offer this option, but not AWS App Runner. Thus, I cannot get the JSON secret into DSN shape.
Describe the solution you'd like
Split up DSN into separate environment variables. Ideally just follow what PostgreSQL already accepts:
https://www.postgresql.org/docs/current/libpq-envars.html
Describe alternatives you've considered
Create a solution that:

Read the original secret.
Shape the secret from JSON into DSN.
Create another secret with DSN.
Maintains secret sync during secret rotation.

Additional context

Deploying using IaC (AWS CDK).
https://docs.aws.amazon.com/apprunner/latest/dg/env-variable-manage.html

Yes, that could certainly be another option to consider is to make a custom container image that adapts the secret value to the DSN format.


Page is not readable as focus is in the wrong place using VO in iOS

When you get to the 'COVID Alert is off' page, after selecting 'Don't Allow' from the Notifications popup window, the focus is at the top on the signal strength.
If I swipe up to read the page, it only reads what's at the very top (time, strength, Testflight, etc.) and doesn't read the page at all.
Tested on iPhone XR 13.5.1

This is because of the testflight demo menu.


Crash when entering a lot of content in the Japanese Microsoft IME

Windows Terminal version
OpenConsole 8a34a0e59ac41bc805c308fb82bc7683e583d45d
Windows build number
10.0.19041.1415
Other Software
No response
Steps to reproduce

Install the Japanese language pack.
Open a WSL bash shell in a recent version of OpenConsole.
Execute read -n 10000.
Switch the Japanese IME to Hiragana mode.
Hold down the a key for a while.
You should see a bunch of あ glyphs being written out.
If you keep the a pressed for long enough, it'll eventually crash.

Expected Behavior
It shouldn't crash.
Actual Behavior
When run from within the Visual Studio debugger, it fails with the following stack trace:
OpenConsole.exe!ConsoleImeInfo::_WriteConversionArea(const std::_Vector_const_iterator<std::_Vector_val<std::_Simple_types<OutputCell>>> begin, const std::_Vector_const_iterator<std::_Vector_val<std::_Simple_types<OutputCell>>> end, _COORD & pos, const Microsoft::Console::Types::Viewport view, SCREEN_INFORMATION & screenInfo) Line 366	C++
OpenConsole.exe!ConsoleImeInfo::_WriteUndeterminedChars(const std::basic_string_view<wchar_t,std::char_traits<wchar_t>> text, const gsl::span<unsigned char const ,-1> attributes, const gsl::span<unsigned short const ,-1> colorArray) Line 444	C++
OpenConsole.exe!ConsoleImeInfo::WriteCompMessage(const std::basic_string_view<wchar_t,std::char_traits<wchar_t>> text, const gsl::span<unsigned char const ,-1> attributes, const gsl::span<unsigned short const ,-1> colorArray) Line 74	C++
OpenConsole.exe!ImeComposeData(std::basic_string_view<wchar_t,std::char_traits<wchar_t>> text, gsl::span<unsigned char const ,-1> attributes, gsl::span<unsigned short const ,-1> colorArray) Line 141	C++
OpenConsole.exe!CConversionArea::DrawComposition(const std::basic_string_view<wchar_t,std::char_traits<wchar_t>> CompStr, const std::vector<TF_DISPLAYATTRIBUTE,std::allocator<TF_DISPLAYATTRIBUTE>> & DisplayAttributes, const unsigned long CompCursorPos) Line 51	C++
OpenConsole.exe!CEditSessionUpdateCompositionString::_MakeCompositionString(unsigned long ec, ITfRange * FullTextRange, int bInWriteSession, CicCategoryMgr * pCicCatMgr, CicDisplayAttributeMgr * pCicDispAttr) Line 988	C++
OpenConsole.exe!CEditSessionUpdateCompositionString::UpdateCompositionString(unsigned long ec) Line 849	C++
OpenConsole.exe!CEditSessionObject::DoEditSession(unsigned long ec) Line 64	C++
[External Code]
OpenConsole.exe!ConsoleInputThreadProcWin32(void * __formal) Line 1057	C++

But the actual problem is probably deeper than that. If I allow the debugger to continue, I get an additional two stack entries implying the crash to be somewhere in the OutputCellIterator constructor.
[Inline Frame] OpenConsole.exe!OutputCellIterator::{ctor}(const gsl::span<OutputCell const ,-1>) Line 149	C++
[Inline Frame] OpenConsole.exe!ConversionAreaInfo::WriteText(const std::vector<OutputCell,std::allocator<OutputCell>> &) Line 116	C++

As a wild I guess, I suspect it might have something to do with the lineVec vector being zero length (or so it appeared at the time of the crash).

I don't know if this is the right solution, but I can prevent the crash if I add a check like this:
if (lineEnd <= lineBegin)
{
    return lineEnd;
}

before the lineVec is constructed here:
https://github.com/microsoft/terminal/blob/91b454ac95b4f8404f2d7c2cb6b13f5c3205001d/src/host/conimeinfo.cpp#L356-L357
It's also worth mentioning that I couldn't reproduce this in my inbox conhost, so it might be a regression.

Ah heck, I tried reproing this Friday and couldn't get a repro. Now, for whatever reason, the internal build I'm on is seemingly refusing to download the Japanese typing package 😑 We might be stuck here for the time being.
How long is long enough for "If you keep the a pressed for long enough, it'll eventually crash."? IIRC I had filled up an entire viewport of output, without it crashing.
I'd be suspicious of when this actually regressed - we haven't touched conhost's IME code in a LONG time... I'd bet this repros quite a way back in the past, or broke because of some other interaction outside ConsoleImeInfo::_WriteConversionArea

I also bet that the check would work. We could probably also get rid of the FAILFAST a few lines up, on 346, and just deal with it. That wouldn't create a new conversion area in this weird edge case where we've backed up before the start of the conversion, because the...
We've gotten the line we're working on
We've gotten the last cell of that line
we've determined the last cell was wide, so we want to start 1 column earlier
we now find that the end is before the start, so we can't actually create a new vector with length=-1
That's a weird edge case. The actual width of the buffer might matter here - might need to be an odd/even number thing, might explain why I couldn't trivially repro Friday.

We might also just want to move that failfast down, after the decrement to lineEnd. Seems like everything in the IME just uses exceptions for flow control anyways. Er, I suppose it shouldn't be a failfast, it could just be a THROW_IF or something. Maybe both accomplish the same thing.

I just checked now and I can still reproduce the crash. At the time of the crash I can see five full lines and then another four characters on the sixth line with a windows size of 80x24. It seems like it can vary though.

I suspect you might be using a debug build, so for me a lot of that stack would be inlined. I think you're still getting the crash in essentially the same place.


Progress bar status bug

Issue Type: Bug
In a notebook, the progress bar status at the top keeps indicating that the cell is running but it is not.

VS Code version: Code - Insiders 1.64.0-insider (Universal) (99305570d885bce929ba8f24d90f2c3ee54c5c2f, 2022-01-03T05:15:53.904Z)
OS version: Darwin x64 20.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
32.00GB (12.13GB free)

Process Argv
--crash-reporter-id 6a5f1067-1364-4707-a779-b0b66087d6de

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

gitlens
eam
11.7.0

copilot
Git
1.7.4421

vscode-pull-request-github
Git
0.35.2021120909

git-graph
mhu
1.30.0

python
ms-
2022.0.1664492081-dev

vscode-pylance
ms-
2022.1.1-pre.1

jupyter
ms-
2022.1.1001614873

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.4

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythontb:30258533
vsc_aa:30263845
pythonvspyt551:30291412
pythonptprofiler:30281269
vshan820:30294714
pythondataviewer:30285072
vscod805:30301674
pythonvspyt200:30323110
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30404738
vscop804:30404766
vscop940:30404999
vsclayoutctrt:30405800
py55gd98cf:30408622
vscexrecpromp3t2:30407759
azactmsal:30409287

Might have been fixed in the past week, but do you have steps to reproduce this?

I cannot reproduce the issue anymore, seems to be fixed.
On Fri, 14 Jan 2022, 8:36 pm Rob Lourens, @.***> wrote:

Might have been fixed in the past week, but do you have steps to reproduce
this?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/140269#issuecomment-1013416542,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE5NYIKZFWFTDNSNS6UJ3VTUWB3KJANCNFSM5LOSAIBA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks, I am still making changes in this area so let me know if you see it again.


[Intellisense] JSON schema works for a few seconds then stops.

VSCode Version: 1.16.0
OS Version: windows 7

Steps to Reproduce:

Create workspace JSON schema
Hit Ctrl+space to get intellisense
Json intellisense works for a couple seconds that gets over written.

Reproduces without extensions: Yes
Here's a sample of my json schema in the settings.json file inside of the .vscode folder.

Here's a gif of my experience:

As you can see, when I reload the window and hit ctrl+space, the intellisense works for a little bit then returns to its default state. Very weird, never had this happen before.
Any help would be appreciated!
Thanks,
Manuel Yang

@mannyyang The proposals you see seem to be all textual proposal, so it doesn't seem like the schema got used at all. And yes, it strange that the proposals get resorted
Can you by chance paste the content of the settings?

Hey @aeschli, thanks for responding. From the gif above, it seems as though the schema does gets parsed and consumed in the first few seconds, then for some reason, is removed. The settings just has a json-schema property and is pretty big so I put it in a pastebin: https://pastebin.com/Pg2HUaPP.
This schema was working before so I'm clueless on how to debug this.

I fixed the issue. There was a bug with resolving a top level $ref in schemas in workspace settings.
As a workaround, don't use $ref at top level, or copy the schema definition to the user settings, or in a file in the workspace. Or use the next insiders build.

Hey @aeschli,
Just wanted to say thanks for the updates and fix! Just wanted to provide an extra verification on this. Glad there's smart developers actively supporting this great software.
👍👍👍👍👍

@mannyyang You are welcome, thanks for using VSCode and thanks for your help!


Can we change predict_len if we use TTM model ?

Pretty good model! There are two models available (512-96 and 1024-96). The predict_len is set to 96, can we change it?

Hi @JieNi4 thanks for your interest.
You can set prediction_filter_length to a value less than prediction_length and the model will only provide that many outputs. The loss will also be calculated based on prediction_filter_length; during fine tuning you can tune the existing head based on this shorter output.

Thanks! Can we add multi-variance into zero-shot or fine-tuning(few-show) process? I have seen it's possible in your paper. Thank you very much for your reply.

Hi @JieNi4 yes -- if you fine-tune you can freeze the backbone, then enable channel mixing in the decoder and tune.
Some code snippets:
finetune_model = TinyTimeMixerForPrediction.from_pretrained(
    "ibm-granite/granite-timeseries-ttm-v1",
    num_input_channels=<num channels for your dataset>,
    decoder_mode="mix_channel",
)

# freeze parameters in the model backbone
for param in finetune_model.backbone.parameters():
    param.requires_grad = False

# define trainer
<standard training arguments, trainer definition>

trainer.train()

Since the forecast_length of TTM is set to 96, how to get the full zero shot evaluation results of different forecast lengths?


demo too lag

32G RAM memory  i5-10500 CPU。。。

I met the same problem. Have any solutions to speed the demo?


Is there a reason why render was made a function rather than a class?

It makes it so much harder to override anything, I want to extend the base functionality to add things like server driven modals, preserving browser url on demand and other things and I find myself needing to do a lot of workarounds due to not being able to inherit from render and extend its functionality so I was wondering if there is a rationale behind this as I'm planning to fork this project just to make the render function into a class.

Hey @IslamNwishy, I know it's been a while since you opened this issue, but if you're still using Inertia and running into difficulties here, I'd be happy to chat about what you're running into!


Setting up postgres database and setting up fastApi

Basic setup in supebase that will help to work with postgres
Set up basic fastapi authentication apis for testing purposes [[need to update response params]

Postgres database using SupaBase is setup and integrated. FastAPI environment installed in development machine.


[backport] Enable optimizer model setup with instantiation

This PR backports #3488

Should we backport this PR?
This is a feature addition, and can change the recommended way of implementing a fundamental part of user's code.
I think it's better to have a trial period.

OK, let's postpone this after the next major version up.


Replace querystring with URLSearchParams

The querystring API is considered legacy and it's recommended to use URLSearchParams instead. I'm getting a warning when building the SvelteKit codebase with Rollup and would like to get rid of the warning
https://github.com/lukeed/polka/blob/52d5902dd11f2fb5962a439ef47799c83ba397ac/packages/polka/index.js#L3

In polka@next, querystring.parse isn't used.
It's only used in<EMAIL_ADDRESS>and you can easily just pass the result into new URLSearchParams directly.
// example
req.query = querystring.parse('foo=bar&bar&baz=');
//=> [Object: null prototype] { 'foo': 'bar', bar: '', baz: '' }

// override
req.query = new URLSearchParams(req.query);
//=> URLSearchParams { '?foo' => 'bar', 'bar' => '', 'baz' => '' }

Hmm, I'm not sure that's right. When I run pnpm install from the kit root directory I see it used
File node_modules/.pnpm/@polka+url<EMAIL_ADDRESS>Line 36:
				query = qs.parse(search.substring(1));

Looks like it's coming from here:
https://github.com/lukeed/polka/blob/e0e131dd28b03e26652a96b9a1428c017209e187/packages/url/index.js#L36

Oh right, sorry was thinking about polka itself.
Sorry, definitely not changing that. Same suggestion from above still applies.

Upcoming framework might be more appealing as a base for something like SK, btw. It's meant to handle different environments seamlessly, which is a good fit for a multi-adapter consumer

Would you accept a PR to change it or is there a reason to avoid updating it?
The suggestion won't work for me because Kit isn't calling @polka/url directly, so there's no easy way to avoid having that line of code being called.
I also wonder if there's a possibility of changing the way that it's imported. Could we updated it so that instead of:
import * as qs from 'querystring';
...
query = qs.parse(search.substring(1));

It could instead do:
import { parse } from 'querystring';

I haven't tested, but think that might be what Rollup is complaining about. I don't know if there's some ESM/CJS compatibility thing that might break though.

It wouldn't make any difference to Rollup.
My suggestion still applies. Just make the change at the first point of contact outside of the Polka handler, so like inside your first .use() middleware.
Not and can't change it because Polka is meant to work with the Express ecosystem, and req.query is always a plain object. Changing that is breaking, not to mention it's slower and decodes slightly differently iirc.

I don't understand how changing anything inside adapter-node would stop polka/url from being included in the SvelteKit code
I wasn't suggesting to change the API. I'd convert back from a URLSearchParam to a pojo

No thank you


Congrats ! Suggestion interoperability / peaks grouping

Hi Peakonly community !
Thanks a lot for this amazing tool ! Data processing is difficult and very much user/parameter/tool dependent. With peakonly we can hope for simplified/fast and comprehensive LC-MS feature detection.
Currently, results are not easily interoperable with other metabolomics tools.
Would it be possible to have an mzTab-M export for the results ?
https://pubs.acs.org/doi/abs/10.1021/acs.analchem.8b04310
Also it would make sense to find a way to integrate tools such as CAMERA or RAMClust to group peaks belonging to one compound, in order to reduce the data complexity and make the results usable.
https://pubs.acs.org/doi/abs/10.1021/ac202450g
https://www.mdpi.com/2218-1989/6/4/37/htm
This could be done with ADAP potentially or even OpenMS FeatureFinderMetabolomics tools.
https://pubs.acs.org/doi/abs/10.1021/acs.analchem.6b02222
Thanks again,
Louis

Actually, maybe the best/easiest solution would be using XNet, a parameter-less clustering algorithm made by one of the leading group in the field:
https://github.com/optimusmoose/XNet
https://pubs.acs.org/doi/pdf/10.1021/acs.jproteome.9b00068

Instead of mzTab-M (or along with), it could be a good idea to use the OpenMS .featureXML for the output. Then it makes possible to use any of their tool to build complex and versatile pipelines.

Thank you very much for your suggestion. We will definitely add the possibility to export results in different formats.

Also it would make sense to find a way to integrate tools such as CAMERA or RAMClust to group peaks belonging to one compound, in order to reduce the data complexity and make the results usable.

In fact, we already have an implementation of an algorithm partially similar to RAMClust.  Unfortunately, we cannot guarantee that it would be integrated very soon. We still have a lot of work to do before it (currently working on a very interesting approach for peak matching between samples, which probably could even increase the final "precision" and "recall" of the raw data processing), but we'll try to do it as soon as possible.

Actually, maybe the best/easiest solution would be using XNet, a parameter-less clustering algorithm made by one of the leading group in the field:
https://github.com/optimusmoose/XNet
https://pubs.acs.org/doi/pdf/10.1021/acs.jproteome.9b00068

It does sound like an interesting solution. We'll experiment with this approach and probably we'll also find a way to integrate it. Thanks again for the profound articles.

That sounds great ! I am happy to test what ever you developed on some data I am familiar with so I can provide additional feedback.


routing: allow runtime updates to mission control config

This PR adds the ability to get and set mission control's config via rpc, allowing for experimentation with these values without a restart.
Since we already have locking in mission control, we can just acquire the lock and update the cfg.

Just from a high level look, I'm wondering if we can do this without introducing race conditions?
Even though we get/set the config behind a mutex, MissionControl itself still reads the config without acquiring the lock first.

Just from a high level look, I'm wondering if we can do this without introducing race conditions?
Even though we get/set the config behind a mutex, MissionControl itself still reads the config without acquiring the lock first.

MissionControl itself still reads the config without acquiring the lock first.

I thought I'd checked all of these, but looks like ReportPaymentSuccess and ReportPaymentFailure don't acquire the lock for adding results to the store. Although they do acquire it later to update MC, so wondering whether than can be moved up. Otherwise the other calls to MC all use the lock so it shouldn't be too much of an issue?

MissionControl itself still reads the config without acquiring the lock first.

I thought I'd checked all of these, but looks like ReportPaymentSuccess and ReportPaymentFailure don't acquire the lock for adding results to the store. Although they do acquire it later to update MC, so wondering whether than can be moved up. Otherwise the other calls to MC all use the lock so it shouldn't be too much of an issue?

@carlaKC I was thinking about places like this: https://github.com/lightningnetwork/lnd/blob/d7ecde2972bfb949c6df717d53ebfbb3283de90b/routing/probability_estimator.go#L82
One alternative to swapping out the mission control config, is to create a new MissionControl with the new config and swap that out instead.

@carlaKC I was thinking about places like this: https://github.com/lightningnetwork/lnd/blob/d7ecde2972bfb949c6df717d53ebfbb3283de90b/routing/probability_estimator.go#L82
One alternative to swapping out the mission control config, is to create a new MissionControl with the new config and swap that out instead.

Perhaps it is worth documenting in the proto that the runtime settings will only be in effect until the next restart.
An alternative would be to hot-reload the config file, either automatically or via a cli command (similar to systemctl daemon-reload). That would at least keep the config in one place. It also opens up the possibility to hot-reload other config settings without adding rpcs for all of them (which aren't really useful other than for experimentation).

Perhaps it is worth documenting in the proto that the runtime settings will only be in effect until the next restart.
An alternative would be to hot-reload the config file, either automatically or via a cli command (similar to systemctl daemon-reload). That would at least keep the config in one place. It also opens up the possibility to hot-reload other config settings without adding rpcs for all of them (which aren't really useful other than for experimentation).

I think just adding to the doc that it won't be persisted is good enough for now?

I think just adding to the doc that it won't be persisted is good enough for now?

Perhaps it is worth documenting in the proto that the runtime settings will only be in effect until the next restart.

Indeed, will add some comments on the rpc making a note of it.

An alternative would be to hot-reload the config file, either automatically or via a cli command (similar to systemctl daemon-reload). That would at least keep the config in one place. It also opens up the possibility to hot-reload other config settings without adding rpcs for all of them (which aren't really useful other than for experimentation).

I think this would be a nice feature to have, but likely a larger project since not everybody runs lnd with a config file. Although it's kicking the can down the road a bit, I'd like to go ahead with adding this rpc because mission control is a pretty underused feature IMO, and surfacing it will help people experiment with it. If we find ourselves exposing other config rpcs, then it's time to talk hot reload.

Perhaps it is worth documenting in the proto that the runtime settings will only be in effect until the next restart.

Indeed, will add some comments on the rpc making a note of it.

An alternative would be to hot-reload the config file, either automatically or via a cli command (similar to systemctl daemon-reload). That would at least keep the config in one place. It also opens up the possibility to hot-reload other config settings without adding rpcs for all of them (which aren't really useful other than for experimentation).

I think this would be a nice feature to have, but likely a larger project since not everybody runs lnd with a config file. Although it's kicking the can down the road a bit, I'd like to go ahead with adding this rpc because mission control is a pretty underused feature IMO, and surfacing it will help people experiment with it. If we find ourselves exposing other config rpcs, then it's time to talk hot reload.

My only real comment is that since we can change the mission control config during route calculation

I don't think that there's a clean way to do this, without restructuring the way payment sessions use mission control quite significantly, which I'm not sure is worth the effort.

It's very unlikely as a single user but may be likely as service provider

If we do run into problems here, I think there could be an option to provide specific MC params for an individual payment, so that you can specify exactly what you want for that route. WDYT?

    The amount of time mission control will take to restore a penalized node
    or channel back to 50% success probability, expressed as a unix timestamp
    in seconds. Setting this value to a higher value will penalize failures for
    longer, making mission control less likely to route through nodes and
    channels that we have previously recorded failures for.
    */
    uint64 half_life_seconds = 1;

If the value is just the number of seconds, why is it described as a unix timestamp?

If the value is just the number of seconds, why is it described as a unix timestamp?

Shouldn't be, fix in #5078.


Session.run with fetches = list apparently works incorrectly

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): The problem manifests in a standard tutorial and I wrote custom code to better understand it.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): ('v1.3.0-rc2-20-g0787eee', '1.3.0')
Python version: Python 2.7.12 (default, Nov 19 2016, 06:48:10) [GCC 5.4.0 20160609] on linux2
Bazel version (if compiling from source):
CUDA/cuDNN version: no GPU
GPU model and memory: none

Describe the problem
The TensorFlow Mechanics 101 tutorial code outputs value of the same Op, loss, in two ways: summary and console output.  The two values do not match.
The code is located here.
To make the problem clear, I changed the console output format on line 305 to 8 decimals:
  cat(sprintf('Step %d: loss = %.8f (%.3f sec)\n',

I compared the console output for loss with the results of the summary Op (the op is defined on lines 120 and 262, and run on line 308).  I obtained the results of the summary op from the csv file downloaded from tensorboard.
The console output is close to the csv file but different, often in the third decimal; they should be identical since the op is the same on the same step.
I do not copy the output here since it is unstable, changes from one run to another.  (I have not been able to stabilize it with use_session_with_seed or with tf$set_random_seed.)
I changed lines 297-298 from
values <- sess$run(list(train_op, loss), feed_dict = feed_dict)
loss_value <- values[[2]]

to a supposedly equivalent code:
sess$run(train_op, feed_dict = feed_dict)
loss_value = sess$run(loss, feed_dict = feed_dict)

After that the console output agreed fully with summary data obtained from csv.
I take it that sess$run(list(... works incorrectly.
I do not know whether the problem is in TensorFlow or in the R interface to it.

values <- sess$run(list(train_op, loss), feed_dict = feed_dict)
loss_value <- values[[2]]

is not equivalent to
sess$run(train_op, feed_dict = feed_dict)
loss_value = sess$run(loss, feed_dict = feed_dict)

This has been discussed many times, e.g. #12863 ,#10860

Thanks!
I see that if the fetches argument in Session.run is a list then its elements are executed in the random order.
I think it is worthwhile to mention this in the manual.
Thus, there is no bug.
However, I want to make this thread a suggestion to improve the above manual.

I'm not sure that "random order" makes sense to add to the manual because it's an implementation detail. IE, maybe today it's random order, and tomorrow it becomes left to right. Leaving it unspecified gives freedom to change implementation around without violating API guarantees
Currently the order is determined by this part in executor.cc. It schedules ready ops simultaneously which means the order is determined by how fast computations run for each fetch

I think it is important to add to the manual that the fetches are run in "unspecified order, possibly in parallel" to dispel the natural assumption that they are run in the order they appear in the list.
In this way you guys will have less threads to answer and close, and we the beginners will have to spend less time breaking our heads figuring out why our code does not work.
In fact, I remember checking that manual a few times before submitting a question to stackoverflow but the idea that the fetches may run not in the specified order, did not come to my mind.

This problem has bitten my butt badly at least twice in the past.
The worst part is that session.run(ops) does evaluate them in order they are provided on tensorflow, but not in tensorflow-gpu.
@yaroslavvb - A warning to user about undefined order in documentation would only be fair. If at some point order will become defined, then that can documentation can be updated. Until then you will have people scratching their heads whenever they test code along the lines of
_, actual_weights = session.run([train_op, weights_op])
assert expected_weights == actual_weights

that works as expected with tensorflow, but not with tensorflow-gpu.

@PuchatekwSzortach do you want to send a PR for this? Since you are the user, you are the best positioned to find the best place to encounter this in documentation.

@yaroslavvb Alright, I'll send a PR :)

@yaroslavvb https://github.com/tensorflow/tensorflow/pull/17727

Sorry to post on this old issue again; however, shouldn't the information about the fetches execution order being undefined be put higher up in the documentation? Currently, it is in the "Returns" subsection, but there is an "args" subsection which seems to make more sense (to me at least).
Should I open another issue for this?

It makes more sense to me that properties of return values are described in return section. I suppose it's best to ask others and see what's the majority's opinion.


Ministry User Welcome Page/ Navigation

As a (Ministry User) I want (To be directed to the Welcome Page after I have successfully signed in and been authenticated) So That (I can navigate in SIMS)
MVP Dashboard is:

[ ]  A Welcome Message
(big text) Welcome to StudentAid BC
(Normal text) ."We are here to support students and institutions"
"A picture of a smiling Student" (TBD)
"Capability to provide Message Board Banner"
General SABC news (to staff). Information about Upcoming programs, applications dates, new system enhancements could be updated here by staff
Make it a bit pretty
Tasks
Use a Form.io form for Content. Form Name:" AEST Dashboard"
[ ] Navigation along the left to Students and Institutions

Acceptance Criteria

[ ] Given (A Ministry User), When (has successfully logged in, Then (they will arrive at the SIMS Welcome Page)

Notes:

[ ] Mirror the student/institution welcome pages and change the text

Draft sketch of welcome page with navigation


Applied fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Coverage decreased (-0.1%) to 81.046% when pulling 1f3839aaa6cd39a8d4894c711ffd765c223987ac on analysis-8n2ZeG into 5d7815fb8a96fff590283bc4780cf4af5a9b4ac5 on master.


json-spec-openapi-<IP_ADDRESS> not compatible with lens-5.3

Compatibility with this new release of lens is required for GHC 9.10 support. See

https://github.com/commercialhaskell/stackage/issues/7408

BTW you can now get notifications on Hackage when you don't accept new versions of dependencies, see discourse

Thanks for the ping! I plan to get to this early this week (just didn't quite make it last week when I did a lot of similar updates).

No problem, there really isn't any rush. I am just pinging because lens-5.3 has been out for a couple of months now. So there are just a few packages left.  It would be nice to not have to churn through the removal/addition process when Stackage Nightly upgrades.

https://hackage.haskell.org/package/json-spec-openapi-<IP_ADDRESS>

Seems to work, thank you!


Setting the container option

I'd like to use bootstrap-datepicker’s container option in order to inject the picker somewhere other than the <body> element.
Doe anyone know a way to accomplish this?
https://bootstrap-datepicker.readthedocs.org/en/latest/options.html#container

The container option isn't supported by the add-on yet, but it's very easy to implement.
Also I've noticed a couple of new options of bootstrap-datepicker that should be supported as well. Will try to implement everything ASAP. Please stay tuned :wink:

Just merged pull request from @minimalweb which adds widgetContainer option. Please use this option to set container parameter on bootstrap-datepicker widget.
This feature isn't documented yet, but update for README and demo page is coming soon.

Awesome!


fix #6682: Add RetinaNet configuration using ResNet18 as the backbone.

Motivation
Add RetinaNet configuration using ResNet18 as the backbone.
Modification

configs/retinanet/retinanet_r18_fpn_1x_coco.py

retinanet_r18_fpn_1x_coco 的权重和训练日志
链接: https://pan.baidu.com/s/10LqXDVzssKnpw_nkRyE4fg 提取码: 88ns

Hi @CCODING04 ,
We do not need to add a base config in the base/models directory. You can directly add only one config for resnet18. Meanwhile, you only need to modify the keys in the backbone and leave others unchanged.

@CCODING04 I think this configuration is ok, can you fill  the performance into the PR message?

@CCODING04 I think this configuration is ok, can you fill the performance into the PR message?

@hhaAndroid
2022-03-13 05:38:02,727 - mmdet - INFO - Saving checkpoint at 12 epochs
2022-03-13 05:42:12,399 - mmdet - INFO - Evaluating bbox...
2022-03-13 05:43:05,056 - mmdet - INFO -
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.324
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=1000 ] = 0.501
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=1000 ] = 0.346
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.173
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.352
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.428
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.512
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=300 ] = 0.512
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=1000 ] = 0.512
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.311
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.550
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.662

2022-03-13 05:43:05,799 - mmdet - INFO - Exp name: retinanet_r18_fpn_1x_coco.py
2022-03-13 05:43:05,799 - mmdet - INFO - Epoch(val) [12][5000]  bbox_mAP: 0.3240, bbox_mAP_50: 0.5010, bbox_mAP_75: 0.3460, bbox_mAP_s: 0.1730, bbox_mAP_m: 0.3520, bbox_mAP_l: 0.4280, bbox_mAP_copypaste: 0.324 0.501 0.346 0.173 0.352 0.428

@CCODING04 I think this configuration is ok, can you fill the performance into the PR message?

@hhaAndroid
2022-03-13 05:38:02,727 - mmdet - INFO - Saving checkpoint at 12 epochs
2022-03-13 05:42:12,399 - mmdet - INFO - Evaluating bbox...
2022-03-13 05:43:05,056 - mmdet - INFO -
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.324
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=1000 ] = 0.501
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=1000 ] = 0.346
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.173
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.352
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.428
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.512
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=300 ] = 0.512
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=1000 ] = 0.512
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.311
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.550
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.662

2022-03-13 05:43:05,799 - mmdet - INFO - Exp name: retinanet_r18_fpn_1x_coco.py
2022-03-13 05:43:05,799 - mmdet - INFO - Epoch(val) [12][5000]  bbox_mAP: 0.3240, bbox_mAP_50: 0.5010, bbox_mAP_75: 0.3460, bbox_mAP_s: 0.1730, bbox_mAP_m: 0.3520, bbox_mAP_l: 0.4280, bbox_mAP_copypaste: 0.324 0.501 0.346 0.173 0.352 0.428

@ZwwWayne Are the results as expected？

Hi @CCODING04 , We do not need to add a base config in the base/models directory. You can directly add only one config for resnet18. Meanwhile, you only need to modify the keys in the backbone and leave others unchanged.

Hi, @ZwwWayne . I have fixed it, Update our retinanet_r18_fpn_1x_coco.py using only one file. Plz check it. Should I retrain it later?

No, we do not need to do that.

@ZwwWayne
I  changed the config and update readme.md and yml of RetinaNet.
Meanwhile, I got some problem cuz 2080ti I used. When I try to train it in bach_size = 2,  I got nan after several epochs. But if I switch it to batch_size = 8, seems all right and everything works fine.
Is there something mistake?

@ZwwWayne I changed the config and update readme.md and yml of RetinaNet. Meanwhile, I got some problem cuz 2080ti I used. When I try to train it in bach_size = 2, I got nan after several epochs. But if I switch it to batch_size = 8, seems all right and everything works fine. Is there something mistake?

You might need to modify the learning rate. And it is a little bit unstable when batch size is small.

You might need to modify the learning rate. And it is a little bit unstable when batch size is small.
@ZwwWayne
Okay, I will try it tommorrow.

Hi @CCODING04 ,
Thanks for your kind PR. Could you use pre-commit to fix the lint issue of the code?
We plan to merge this PR before next release.

@ZwwWayne
Okay, I will pre-commit it ASAP. Sry for replying it so late.

I suggest we keep the config to be 8x2 setting. We can help benchmark. We could also add a note to indicate the performance of 1x8.

I suggest we keep the config to be 8x2 setting. We can help benchmark. We could also add a note to indicate the performance of 1x8.

@ZwwWayne
Ok, that's perfect, I have no ability to benchmark 8X2 config. So what should I do? Should modify config as 8X2  and commit it?

I suggest we keep the config to be 8x2 setting. We can help benchmark. We could also add a note to indicate the performance of 1x8.

@ZwwWayne Ok, that's perfect, I have no ability to benchmark 8X2 config. So what should I do? Should modify config as 8X2 and commit it?
You can submit two configs, we will help to run the experiment.

@CCODING04 We can keep two configurations retinanet_r18_fpn_1x8_1x_coco.py and retinanet_r18_fpn_1x_coco.py

@CCODING04 We can keep two configurations retinanet_r18_fpn_1x8_1x_coco.py and retinanet_r18_fpn_1x_coco.py

@hhaAndroid
In Slack, weiwei told me to update back 8x2 and I delete 1x8 config.
And I pushed it again,

@CCODING04 We can keep two configurations retinanet_r18_fpn_1x8_1x_coco.py and retinanet_r18_fpn_1x_coco.py

@hhaAndroid In Slack, weiwei told me to update back 8x2 and I delete 1x8 config. And I pushed it again,

ok

@CCODING04 I ran your two configurations directly and the results are as follows：
configs/retinanet/retinanet_r18_fpn_1x8_1x_coco.py
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.317
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=1000 ] = 0.499
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=1000 ] = 0.332
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.164
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.348
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.423
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.504
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=300 ] = 0.504
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=1000 ] = 0.504
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.305
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.539
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.659

But I remember you are running 32.4, have the parameters been modified?
configs/retinanet/retinanet_r18_fpn_1x_coco.py
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.317
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=1000 ] = 0.496
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=1000 ] = 0.334
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.162
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.342
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.430
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.502
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=300 ] = 0.502
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=1000 ] = 0.502
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.306
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.533
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.661

@hhaAndroid
Yes, it's 31.7 when lr set 0.005, I got the same metrics too. And I got 32.4 cuz I tried to set lr eq 0.0025.

@hhaAndroid Yes, it's 31.7 when lr set 0.005, I got the same metrics too. And I got 32.4 cuz I tried to set lr eq 0.0025.

So, You can update the lr.

@hhaAndroid Yes, it's 31.7 when lr set 0.005, I got the same metrics too. And I got 32.4 cuz I tried to set lr eq 0.0025.

So, You can update the lr.

Yes, I update lr with better performance.

- Name: retinanet_r18_fpn_1x8_1x_coco
  In Collection: Please fill in Collection name
  Config: configs/retinanet/retinanet_r18_fpn_1x8_1x_coco.py
  Metadata:
    Training Memory (GB): 5.0
    Epochs: 12
  Results: []
  Weights: https://download.openmmlab.com/mmdetection/v2.0/retinanet/retinanet_r18_fpn_1x8_1x_coco/retinanet_r18_fpn_1x8_1x_coco_20220407_171255-4ea310d7.pth
  log: https://download.openmmlab.com/mmdetection/v2.0/retinanet/retinanet_r18_fpn_1x8_1x_coco//retinanet_r18_fpn_1x8_1x_coco_20220407_171255.log.json

- Name: retinanet_r18_fpn_1x_coco
  In Collection: Please fill in Collection name
  Config: configs/retinanet/retinanet_r18_fpn_1x_coco.py
  Metadata:
    Training Memory (GB): 1.7
    Epochs: 12
  Results: []
  Weights: https://download.openmmlab.com/mmdetection/v2.0/retinanet/retinanet_r18_fpn_1x_coco/retinanet_r18_fpn_1x_coco_20220407_171055-614fd399.pth
   log: https://download.openmmlab.com/mmdetection/v2.0/retinanet/retinanet_r18_fpn_1x_coco/retinanet_r18_fpn_1x_coco_20220407_171055.log.json

@CCODING04 Please update the config/readme/metafile

@CCODING04 Please update the config/readme/metafile

Ok~

We should also update the config of auto_scale_lr

We should also update the config of auto_scale_lr, see #7482

Support auto_scale_lr already


List scrolls up when hovering over last item

See  http://jsfiddle.net/TjuZN/1/
All options except the last one are disabled.
The list scrolls back up to the top of the list when you hover over the last item in the list. Making it hard to select.

Can validate that this isn't fixed.


Client Crash 1.12.2

https://pastebin.com/DY4hXCMG
Forge Version: 2768
Minecraft Version: 1.12.2
Crashes when Minecraft loads MineMenu during the Preinitialization Phase.

Are you using just Forge on the client, or Sponge Forge?

I am using just Forge on the client
Sponge Forge for the Server; Server loads fine
last working version of MineMenu for me is "MineMenu-1.12.2-1.6.4-universal"
I could of sworn it the latest set of updates were working after 1.6.4 were working on my client; but it is possible I saw the server crash first and disregarded the client's status.

No it's cause of a change I made in the last build, to make it work in my dev environment. It works on the latest versions of Forge just fine. But if I remember correctly, Sponge Forge requires you to use the recommended version of Forge, right?

Correct, Sponge uses the Recommended Version of Forge (<IP_ADDRESS>68)

I'll release a new version that works properly on older builds then.
Forge really needs to come out with a new recommended build, <IP_ADDRESS>68 is ancient.
I'll upload it to CurseForge in a few minutes, please let me know if it works for you :)

Works perfectly; Thank you for your time and effort! <3


Feat: Update the environment variables for nextjs

What feature?
The documentation of environment variables provided in .env.example is outdated and some variable names have been renamed in actual code. This issue is to update and set the correct environment variables for Appwrite and Nextjs.
Add screenshots

In .env.example

In the codebase

Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

Assigned this issue to you


core: always inject the nobadger build tag

Its simplicity making me worried I missed something

What if a user wants to opt out of this and actually include badger?
What if they specified this flag themselves, will it break from having it twice?

What if a user wants to opt out of this and actually include badger?

This build-tag only affects the github.com/smallstep/nosql library. It'll only be an issue if a user wants to use the github.com/smallstep/nosql with badger, which I have no interest in supporting. Badger is nothing but trouble.

What if they specified this flag themselves, will it break from having it twice?

It'll work without issues. The compiler does not complain.

Maybe we add an XCADDY_BADGER=1 or something?

I don't like exposing the internal implementation this way. It feels like breaking the abstraction. It isn't something they should be aware of.


java.lang.ClassCastException: java.lang.Integer cannot be cast to java.lang.Long

Getting this exception while passing the trackingInterval as integer
Can you pls update this lib based on the solution provided in the below link
https://github.com/dotintent/FlutterBleLib/commit/4069052715e5e9e17c831f597f55fdeb62548f4c

Thanks for reporting, I am reviewing this plugin today to fix open issues where possible.

Fixed here: dd1ad315d5d208f217f9e53e0de47ae4d74d5ad5

Should be fixed in #40


Version 1.1

I pushed a v1.1 branch earlier, which will contain work towards that eventual release. This issue is just to detail the features going into that release and coordinate.
The branch has support for datasets, which will allow us to support the impending Fall of Oriath beta alongside the current version of the game.
One change that might seem odd initially is the use of ES6 Javascript over TypeScript. The syntax is pretty much identical once you strip out the type annotations, so it has taken very little work to actually switch over. I don't think it's worth constantly fussing with the type system in TypeScript with a project of this size.
A list of things that I'm working towards for the 1.1 release would be:

Leverage the context of a block within both the Linter and Completion providers. For example, a block with a Class restriction should only consider BaseTypes of that class as valid values.
Offloading the initial processing of a filter to a separate NodeJS process (via a Task). There are several filters right now that take 2-3 seconds to process. If that work is done in the main process, then Atom will lock up until the processing has finished. The groundwork is already in the v1.1 branch, but the code that processes item filters needs to be reworked slightly for various reasons.
Implementation of a color picker, which can be opened by clicking a color decoration, selecting a particular completion suggestion, or using a command. Etch is probably what I'll be using to build views like this.
Completion suggestions for all unique items, with the inserted text being the base itself. Uniques have been added to the JSON data for base types in the 1.1 branch already.
Icons for completion suggestions. Not per item, but per category. Unique item suggestions might have a specific icon, with maps having their own as well.

I still want to provide some means of previewing a filter within the editor, as well as support for color schemes. The ultimate goal here is to have color scheme modifications applied instantly to a filter preview pane. I don't know if it makes it into 1.1, since I'll be busy later this month.

This ended up being a revamp of most of the package internals, while sticking with TypeScript as the primary language. The revamp dealt with a lot of activation/deactivation issues that were caused by Atom's odd initialization order. I also went through with the Task implementation, so at worst (when dealing with item filters containing 10+ thousand lines) Atom will have mousewheel delay for ~100 milliseconds instead of locking up for several seconds.
The only things that we're really missing are:

additional linter messages
commands
tooltips
support for color pickers

I don't think we should implement our own color picker, instead just adding support for the various color code formats. That allows the user to use whatever color picker they want, whether its an Atom package or not.
Note that 1.1.0 has changes to the dependencies, so you'll need to do an npm install once again if you aren't using the version from the Atom package manager.


Introduce metrics with micrometer. (v0_10)

Integrate micrometer
Support JMX (io.digdag domain)
Customized by Guice module with SPI
Targets of metrics as follows.

API endpoints
WorkflowExecutor
Agent and OperatorManager
Database

@muga I have not yet implemented test and document, but could you check roughly the code?
Following image is sample JMX metrics (not all).
Currently around 150 metrics are created.

@muga Thanks for your comment.
I will try to fix the PR with your comments.

@muga Could you confirm again after CI pass. I updated as follows.

DigdagEmbed set empty DigagaMetrics as default
So remove injection of DigdagaMetrics from CLI and tests.
Add comment to ErrorReporter class on deprecation in future.

I didn't fix your following comment.

Curious in ConfigEvalEngine#eval related metrics. Not only elapsed time but 2 arguments size as well :-)
I think that calculating the size of argument may affect to performance if argument is too big and it may also cause abort because of OutOfMemory.
If you have any good idea to calculate the arguments size safely and effectively, please give me advice.

@yoyama thanks! i didn't expect that they were done on this PR, but great job 👍

I think that calculating the size of argument may affect to performance if argument is too big and it may also cause abort because of OutOfMemory. If you have any good idea to calculate the arguments size safely and effectively, please give me advice.

No I don't have good idea, sorry. Your comment makes sense. we need to avoid the bottleneck for taking metrics. Please ignore my comment.
Good to go? or wait for https://github.com/treasure-data/digdag/pull/1146#discussion_r309019182 Both of them are fine to me.


Arrays, is this intentional behavior

Should be my last tidbit :)
I saw your XML sample for arrays was like so:
<Root>
  <Element1>Value</Element1>
  <Review>stuff</Review>
  <Review>stuff</Review>
</Root>

I'm ashamed to say I didn't even know that was possible, I always nested arrays inside a parent tag lol
My XML is as follows:
<Root>
  <Element1>Value</Element1>
  <Computers>
    <Computer>
      <Name>Machine1</Name>
    </Computer>
    <Computer>
      <Name>Machine2</Name>
    </Computer>
  </Computers>
</Root>

Am I correct in understanding that I'll need 3 total classes:
@XMLElem({name: 'Root'})
export class RootElement {
  @XMLChildElem({name: 'Element1', type: ()=>String})
  element1: string

  @XMLChildElem({name: 'Computers', type: ()=>ComputersElem})
  computers: ComputersElem
}

@XmlElem({name: 'Computers'})
export class ComputersElem {
  @XMLChildElem({name: 'Computer', type: ()=>ComputerElem, array: true})
  computers: ComputerElem[]
}

@XmlElem({name: 'Computer'})
export class ComputerElem {
  @XmlChildElem({name: 'Name', type: ()=>String})
  name: string
}

Then I would access a single computer like so
rootElemInstance.computers.computers[0]

Do I understand that correctly?  I would love to get it so its like so:
rootElemInstance.computers[0]

But the XML structure needs preserved.  I've tried a few things and I don't seem to be able to get it to work and I'm not sure if it's lack of my own knowledge in Typescript or just how it is supposed to work.  I'm not asking you to change anything, just wanting confirmation that I'm not missing some cool trick of the library.
I did try something weird which almost worked but then items was a nested array..
@XmlElem({ name: 'Computers' })
class ComputersElem<T> implements Iterable<T>, ArrayLike<T> {
  @XmlChildElem({ name: 'Computer', type: () => ComputerElem, array: true })
  private items: T[] = []

  constructor(items?: T[]) {
    console.log('Constructor', this.items)
    if (items) {
      this.items = items
    }
  }
  // constructor(d?: NetworkV1) {
  //   Object.assign(this, d || {})
  // }

  get length(): number {
    return this.items.length
  }

  [index: number]: T

  push(...items: T[]): number {
    return this.items.push(...items)
  }

  pop(): T | undefined {
    return this.items.pop()
  }

  *[Symbol.iterator](): Iterator<T> {
    for (const item of this.items) {
      yield item
    }
  }
}

so ComputersElemInstance[0] is an array and that array contains the arary of Computer objects.

I looked and looked and the three class method is the best way, and is fine :)  Thank you again for the awesome library!

Hi! Sorry for late response. Glad you asked, actually this is something that i myself was not happy about, and it indeed is confusing. This post will help others too, so thanks for bringing this up :)
Yes, this is intentional behaviour, and unfortunately, the best one out there that is able to cover all (or almost all) of the rich and wide variety features of XML, like comments, attributes, arrays without a wrapper tag (surely a strange thing, but Amazon S3 API uses them a lot, and btw I came up with this library at our company where we implement an S3 API compatible storage).
And also this is the way that GoLangs XML implementation works, so i decided to not reinvent the wheel, and just take the approach that proved itself already for many years in Go's community.
So here is an example:
<article>
	<comments offset="0" take="10">
	  <comment lang="en">comment 1</comment>
	  <comment lang="it">comment 2</comment>
	</comments>
</article>

As you see, the container tag comments has attributes offset and take. And tag comment has lang attr. So the only way to extract all of those fields is to declare three classes:
class Article {...}
class Comments {
	@XmlAttribute() offset: string,
	@XmlAttribute() take: string,
	@XmlChilElem({ type: ...,  array: true }) comments: Comment[]
}
class Comment {
	@XmlAttribute() lang: string,
	@XmlChardata() content: string,
}


Grammar correction

Changed text to read "I'm a newbie, but I trust I'll become an expert!!". The current version appears to be sample text so this should read better on the site!

Much better :)


Draw to bitmap?

Some graphics systems allow for drawing into (offscreen) bitmaps.  We should investigate whether this can be done with vpd-gl.
If it can, then we'd need to add a VDU command to create such an offscreen bitmap for drawing, and another command to re-target drawing commands to that bitmap.
It might be possible to do something similar to this by just using a double-buffered mode, drawing to the (offscreen) buffer, and copying from that buffer into a bitmap (see #26).  Once the area has been copied the screen area can be overwritten.

implementing this in vdp-gl is tricky
the VDP now has an implementation of "capture screen to bitmap" - as noted this could be used in a double-buffered mode to capture bitmaps purely off-screen

implementing "draw to bitmap" requires a reworking of vdp-gl, splitting its drawing system from VGA signal generation - see AgonConsole8/vdp-gl#12
if/when that work is done it should be practical to implement drawing to a bitmap


Migrate error handling from xerrors to fmt.Errorf

Migrating error handling from the xerrors package to the Go standard library (fmt.Errorf & errors.New).
Resolves #325

I'm guessing CI failure is due to edits to generated code. We can revert those files.

Thank you @anorth. I've reverted edits to the generated code.

I tried to rebase and resolve the conflicts in GH UI, but looks like I might have got something wrong.
There's a giant PR #348 that we should probably wait for.

I recommend writing down a script to do it.
sed -i -e 's/xerrors\.Errorf/fmt.Errorf/' -e 's/xerrors\.New/errors.New/' **/*.go I think should work.

We merged this in #455 as we had a low spot in active code changes.


Escaped xml in generated C# code for generic types

If an operation parameter type contains escapable xml characters such as in generic types like List<string> then the handlebars template will escape these characters giving the non-compilable List<string> whenever the parameter type config value is surrounded by two braces {{ParameterType}} instead of three {{{ParameterType}}}. This only occurs in the minority of cases and is corrected by: https://github.com/Avanade/Beef/pull/78

Pull request accepted; many thanks 👍
However, with the exception of the Web API Controller parameter default. This fix created other issues with the likes of Create and Update where the value was defaulting to = default where the value was not nullable. If a parameter should be defaulted for a controller then the parameter default code-gen configuration should be used; for example: <Parameter Name="Names" Type="List<string>" Default="default" />.
@damiankennedy2012, if you are having an issue with the defaulting are you able to provide further context/example so I can replicate?
Published Beef.CodeGen.Core NuGet package version 4.1.6.

Cheers for accepting that.
Yeah, nah, I wasn't sure about the default change either. What I did was add to GetTransactions endpoint in Cdr.Banking.xml like this:
<Operation Name="GetTransactions" Text="Get transaction for account" OperationType="GetColl" WebApiRoute="{accountId}/transactions" PagingArgs="true" DataCosmosPartitionKey="accountId">
    <!-- Note usage of ValidatorFluent which will inject the code as-is into the validation logic; being a common validator 'Validators.Account' that will perform the authorization check. -->
    <Parameter Name="AccountId" Type="string" ValidatorFluent="Common(Validators.AccountId)" WebApiFrom="FromRoute" IsMandatory="true" />
    <Parameter Name="Args" Type="TransactionArgs" Validator="TransactionArgsValidator" />
    <Parameter Name="SomethingName" Type="List<string>" />
  </Operation>

So it might be because I added it as the last parameter that on regen the extra controller method parameter without a default was added after the existing default controller parameters which then caused a syntax error. Will use Default="default" 👍


Few locations are not found in large map, although they work properly in small maps

Describe the bug
Although following 2 locations are working fine in respective State map but gives PointNotFoundError in North-America map.

46.0838,-109.729 (Montana)
38.0299,-81.235 (West virginia)

Learning from discussion on forum (https://discuss.graphhopper.com/t/connection-between-locations-not-found/3477/7)
I tried both following ways:

increased prepare.min_network_size to 20000 and re prepared the cache. Still no route
decreased prepare.min_network_size to 0 so that no networks are removed in PreparingSubNetwork. Still no route.

To Reproduce
Steps to reproduce the behavior:

checkout master branch of GraphHopper
download montana.pbf (https://download.geofabrik.de/north-america/us/montana.html)
run ./graphhopper.sh -a web -i montana.pbf -o montana-gh
Search for route from any nearby point in montana to 46.0838,-109.729 using car. It will give successful route.
Now, download north_america.pbf (https://download.geofabrik.de/north-america.html)
run ./graphhopper.sh -a web -i north_america.pbf -o north_america-gh
Search for route from any nearby point in montana to 46.0838,-109.729 using car. It will give error: PointNotFound for 46.0838,-109.729

Expected behavior
With change in prepare.min_network_size to 0, it should be able to find route to 46.0838,-109.729 in North America map.
Screenshots & Logs
It works fine in graphhopper.com/maps:
https://graphhopper.com/maps/?point=46.0838%2C-108.729&point=46.0838%2C-109.729&locale=en-us&vehicle=car&weighting=fastest&elevation=true&turn_costs=true&use_miles=false&layer=Omniscale

A difference for different maps is expected for very large distances to the road network and how the LocationIndex currently works. Have a look into your locations: they are ~2km away from the nearest road. At some point we have to throw an exception and currently this distance is large but not strictly defined. Is there a use case to enforce a stricter handling?
You could enforce always the same behaviour via adapting LocationIndexTree.prepareAlgo and make it independent on the map boundaries. This method has a simple heuristic to adapt the "cells" the location index uses to the map size. Please note that changing it might have arbitrary bad effects like a very slow lookup, an unnecessary large LocationIndex, memory waste or similar.

@karussell thanks for responding.
In LocationIndexTree.prepareAlgo, I see cells are restricted to 9. I am understanding cells are map tiles around the location. Let me know if my understanding is correct. Could you explain what would it mean if I increase cells to say 12.
Also, there is mention of radius till which it searched for closest nodes. How can I change this radius? Is it P_RADIUS in GraphEdgeIdFinder class?


Support for solid-js

Can you support solid-js which has been gaining a lot of interest lately.

Hi @oSethoum,
Thanks for your question. Let’s see if solidJs devs are interested in this.
React with a —> 👍 if you want this feature.

Hi just wanted to share that I was able to get Embla working very easily with Solid by following the vanilla JS steps.
Only thing I did different was add the JS code inside Solid's onMount() callback.

Hi @oSethoum,
Thanks for your question. Let’s see if SolidJs devs are interested in this.
React with a —> 👍 if you want this feature.

We need Solid.js support 🥺

Merged thanks to @LiamMartens great contribution. embla-carousel-solid will be released with v8.0.0-rc19 soon 🎉 !

@LiamMartens, @BoilingSoup, @oSethoum, @jassix - I just released the SolidJs wrapper with v8.0.0-rc19. Thanks a ton @LiamMartens ⭐!
Here's the get started guide.
There's still a lot to do. We need to add SolidJs code examples under all API pages. For example, here. Feel free to add these if you want to contribute.
Best,
David


Include file names in error response from package file validation

We had a customer issue today that was difficult to diagnose because the error response received from Cosmos was the following:
{
  "data": null,
  "message": "OneAnd(com.mesosphere.cosmos.PackageFileSchemaMismatch,List())",
  "type": "NelErrors"
}

Which comes from UniversePackageCache.readPackageFiles. The customer was using a modified Universe repo and it turned out that they had dropped the required packagingVersion field from the package.json of Chronos.
Deliverable: the error response generation for the NelErrors type should be made more robust, so that the details of all errors it contains are included in the response. We should also avoid serializing the data structure directly in the message field, and instead put the details in data where they can be parsed.

Additional requirement from #341 -- each nested error in the array under data should be rendered in exactly the same way as it would appear in a response body by itself.

@cruhland I think we can close this issue, no?

@jsancio yes, this is specifically about improving how NelErrors is rendered in responses. It turns out that NelErrors is no longer used after we got rid of UniversePackageCache, so I've created a separate issue to delete it (#426). I'll close this one.


System.IndexOutOfRangeException: 'Index was outside the bounds of the array.'

Hello EveryOne
I want to test my onnx model but I git this Error
my models
Models File

public class TestModel1: YoloModel
{
public override int Width { get; set; } = 640;
public override int Height { get; set; } = 640;
public override int Depth { get; set; } = 3;
    public override int Dimensions { get; set; } = 6;

    public override int[] Strides { get; set; } = new int[] { 8, 16, 32, 64 };

    public override int[][][] Anchors { get; set; } = new int[][][]
    {
        new int[][] { new int[] { 009, 011 }, new int[] { 021, 019 }, new int[] { 017, 041 } },
        new int[][] { new int[] { 043, 032 }, new int[] { 039, 070 }, new int[] { 086, 064 } },
        new int[][] { new int[] { 065, 131 }, new int[] { 134, 130 }, new int[] { 120, 265 } },
        new int[][] { new int[] { 282, 180 }, new int[] { 247, 354 }, new int[] { 512, 387 } }
    };

    public override int[] Shapes { get; set; } = new int[] { 80, 40, 20, 10 };

    public override float Confidence { get; set; } = 0.20f;
    public override float MulConfidence { get; set; } = 0.25f;
    public override float Overlap { get; set; } = 0.45f;

    public override string[] Outputs { get; set; } = new[] { "output" };

    public override List<YoloLabel> Labels { get; set; } = new List<YoloLabel>()
    {

        new YoloLabel { Id = 1, Name = "helmet" }
    };

    public override bool UseDetect { get; set; } = true;

    public TestModel1()
    {

    }
}

I ran into the same problem using my own training results.

I get this problem all the time.
But I found the cause of the problem and came up with a solution.
The key cause of the problem:
Wrong dimension set in model file.

So how do we get the dimensions right?
1st:
Open this website: https://netron.app/
Open your xxx.onnx file in this page
2nd
Find the output result like this:

The output array :[1,25200,7] in my sample
3th
Run the Yolo5NetApp,Set a breakpoint at the point where the program reported an error.
Get run results

The output array's length is 277200 in my sample
4th
The right dimensions  = 277022/25200 = 11
5th
Modify your dimensions in model file ,you can run this app.
Last
There may be a better solution!


fix(tessera): updated tessera version error in quorum-all-in-one

Describe the bug
Tessera version is updated in #2240 in the dockerfile of quorum-all-in-one to fix the vulnerabilities. However,  after the update, when running the docker image of quorum-all-in-one, tessera is having an issue. Versions lower than 21.1.7 works with the older version that is being used which is 21.1.1. The tessera was updated to 22.1.7 which has drastic changes which will be needed to be updated on our end as well in order for tessera to work for quorum-all-in-one.
To Reproduce
[process]
DOCKER_BUILDKIT=1 docker build ./tools/docker/quorum-all-in-one/ -t qaio
docker run --rm qaio
[error]
No such file or directory.
Expected behavior
Tessera will run when quorum-all-in-one docker image is ran. Quorum-all-in-one will run successfully as well.
Additional context
Related to #2240 (merged) and #2274 (fixed but on hold for tessera update)

Hello @petermetz , can you assign to this me? Thank you.

Hello @petermetz , can you assign to this me? Thank you.

@ruzell22 Done, thank you!


How to notify IoT Hub of a completed file upload?

I am uploading a file blob using the sample code - https://github.com/Azure/azure-iot-sdk-python/blob/master/device/samples/iothub_client_file_upload.py
But how can I notify IoT Hub of a completed file upload like this?
https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-devguide-file-upload#notify-iot-hub-of-a-completed-file-upload

Closing now... :)

@openedhardware, thank you for your contribution to our open-sourced project! Please help us improve by filling out this 2-minute customer satisfaction survey


Filtering overlapping CNVs and SNVs/indels

Need a way to do compound het/homozygous filtering that would identify genes for which the two variants in question are an overlapping CNV and a SNV/indel. Perhaps inputting CNVs using a BED+ file?
A number of considerations (brainstorming here, might be missing some possibilities):
compound het:
duplication CNV + het for an SNV/indel
deletion CNV + het for an SNV/indel (non-overlapping with the CNV -- maybe when CNV covers only part of the gene?)
homozygous recessive:
het for a deletion CNV + "homozygous" (would be called this way although really hemizygous) for an SNV/indel

Is this just checking if adjacent lines in the output from the comp_het tool overlap?

No. This would be doing comp_het filtering combining CNVs from an alternate data source (presumably a BED file?) and variants from a vcf.

I notice that currently, the comp_het_id is not unique to a comphet pair since it includes the variant id. if it were unique to a pair, you could do something roughly like:
bedtools groupby -i comp_hets.bed -c 1,2,3 -g $comp_het_id_col -ops first,min,max \
    | bedtools intersect -a - -b $CNV

if I understand correctly.

No, I meant to look for compound het pairs where one allele IS the CNV and the other allele is a SNP/small indel from the vcf.

Some examples:

http://www.ncbi.nlm.nih.gov/pmc/articles/PMC4104741/
http://www.nature.com/nature/journal/vaop/ncurrent/fig_tab/nature13394_SF7.html
Imagine a person who has both one of the described heterozygous carrier CNVs and then a normal small  mutation from the other parent:
http://genome.cshlp.org/content/early/2013/05/16/gr.156075.113.abstract


Can we get a list of all the default config options and values?

I was wondering if we could get a list of all the default config options and values for the config file.
For example, I'd like to know if there was a default option for changing the name of the "static" folder into a custom name. Current documentation only shows the options like contentdir, layoutdir etc.
It would be great if we could have all the options available to us in a single place.

There's an option called StaticDir, default static.
And yes, there should be a list -- and it should be a "hugo command" -- "hugo config"? Sorted list?

For now, you can look at the code:
Here is the latest config:
https://github.com/spf13/hugo/blob/master/commands/hugo.go#L113-L141
Here is version 0.12 (latest release version):
https://github.com/spf13/hugo/blob/v0.12/commands/hugo.go#L102-L123