object is possibly undefined for imported object

From @unional on January 9, 2017 19:57

VSCode Version: 1.8.1

// a.ts

export interface Boo {
   z: string
}

export interface Foo {
  x: string
  y?: Boo
}

const value: Foo = { x: '1' }
const { x, y } = value
export { x, y } // yes, `y` can be undefined

// b.ts
import { x, y } from './a'

function do() {
  if (y) {
    console.log(y.z) // object is possibly 'undefined'
  }
}

It seems like tsc cannot determine the type of y being Boo | undefined. The code completion doesn't show either.
Copied from original issue: Microsoft/vscode#18339

This does not look VSCode specific. I'm also able to repo it using TS 2.1.4 and the latest 2.2 with the following tsconfig.json:
{
    "compilerOptions": {
        "strictNullChecks": true
    }
}

Is there/can there be some syntax for things that can be undefined? I have this:
categories?: Array<ProductCategory> = undefined
Usage:
product.categories = await Promise.all(product.categories.map(async (category) => await this.SaveCategory(category)))
Could usage be product.categories?.map?
I'm expecting at some point it gets set to an array (hydrating from server). The reason is I have other stuff like shipping_taxable?: string = undefined. I don't want that to be shipping_taxable: string = ''. Or id: number = 0. If it's not hydrated and happens to be undefined, I want it to error out, not have an irrelevant default value. It's also useful for checking what changed.

You can use product.categories!.map(... to indicate that you believe the value to be non-null/undefined

Oh, and the other thing you can do is write
categories: Array<ProductCategory> = undefined!;

so that you don't need ! at the use sites. Sort of a trade-off. I would do this method if the field were initialized "early" in the container's lifetime, and the other approach if it were more common to find the field uninitialized at a later point.

Just checking out old issues; this one was fixed by #16658.


Fixed a small typo

Fixed a typo in the Boilerplate wizard where "bing" was used in place of "being"

Thanks, will add your name as a contributor.


fix: Fix HLS lazy-loading with DRM

Now that DRM info is only parsed after playback has begun, the manifest parser needs to communicate back to the other components that new DRM info is available after lazy-loading a playlist.
Closes #4622

New test cases to follow.

Tested locally and it works fine

@joeyparrish This fix does not interfere with the development of https://github.com/shaka-project/shaka-player/issues/917, does it?
Note: the stream of the issue has EXT-X-SESSION-KEY

@joeyparrish This fix does not interfere with the development of #917, does it?

I don't think it will.

Some additional expectations added to existing tests.

Incremental code coverage: 84.38%

The coverage is low, @joeyparrish do you want increase it?

Incremental code coverage: 84.38%

The coverage is low, @joeyparrish do you want increase it?

The test changes just added newDrmInfo checks at the level of the HLS parser, but the callback for newDrmInfo inside Player is uncovered.  In fact, basically none of the callbacks in the manifest parser's playerInterface are covered by tests.  I think this is because when we run playback tests that exercise Player, we're using a fake manifest parser that doesn't use that interface at all.
The callbacks in the ManifestParser playerInterface are mostly very very simple, just delegating to one private Player method.  newDrmInfo is slightly more complex, in that it has two calls and three null checks, but it's still basically just delegating to processDrmInfos_, which is covered.
So I think it's good enough under the circumstances.


Template - history not throwing back results

Not sure whether this is affecting DMPTool too could I check with you @briri? But it has been reported and I checked on our end for DMPonline, that when the admin goes into the template > own templates> click on actions > click on history>
and user tries to click on:
2
Last
or
View all

It throws back just an empty page:

Not sure about the DMPonline codebase @magdalenadrafiova. I just checked and this is working in the new DMPRoadmap version v3.0.6 so you may just need to upgrade.

@raycarrick-ed @dsisu I think this now fixed on DCC DMPOnline


Differences in required test baseline files between stage 1 and stage 2 builds

The release of 6.0.116 exposed an issue in the tests that make use of baseline files. That release caused a need to update the MissingXmlDoc.txt baseline file to include a new entry. But updating that file then caused the corresponding VerifyTargetingPacksHaveDoc test to fail in the stage 2 build (internal link) because the XML file was not missing in that build. So there's a difference in the output of the stage 1 and stage 2 build such that a single baseline file cannot describe the expected output of both scenarios.
This is expected to occur for other baseline files such as SDK diff and poison leaks. We should have a general solution that can apply to all these baseline file types.
One potential solution for this is to have two baseline files: one for the stage 1 build and the other for the stage 2 build. The pipeline would indicate which one to use via an environment variable. The baseline for the stage 2 build could be optionally such that if it does not exist, the test falls back to the stage 1 baseline.

@mthalman - Is this issue still relevant?

XmlDocTests were removed from all supported branches (https://github.com/dotnet/source-build/issues/3031)
PoisonTests baseline is intended to be empty.  Only reason for entries is temporary and we should not be releasing with leaks
SdkContentTests now only run against stage 1 legs

#2 and #3 can affect servicing branches but IIRC they haven't been problematic.  It was really #1 that prompted this issue.

The only known occurrence of this issue currently is SDK content diff in servicing. It's fine to ignore that for servicing.


Us back 04 401 get plug types route

created a route that can show all plug types (GET /api/plugTypes/)
integrated images into the backend and made sure the front could access them

Un commentaire sur le nommage de branche et la précision des commits : c'est top.


Tiny spello a -> are

As title says.

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

Thanks!


[W5][W15-2] Goh Yin Hao

Added sortedlist command.
List entries sorted by their names alphabetically.
Able to use other commands (e.g. delete, view) based on sorted list.

Good job for implementing the SortedList command! However, remember to use present tense for the commit message: Add new command

Comments are very well-written, helped me to follow your code.

@gohyinhao Close the PR after reading, thanks!


Write Logistic Regression documentation

Description
Add Logistic Regression and Optimizers chapters to oneDAL documentation

@Mergifyio copy rls/2024.0.1-rls


Add ability to sort the snippets.

There should be the ability to sort the snippets by different metrics.

Duplicate of #23.
TL;DR: We currently order by popularity, soon we will add ordering alphabetically, too, allowing you to switch between orders.

Would be nice to do by difficulty as well.


SAHP loglike is nan

2024-08-16 17:01:52,531 - config.py[pid:15636;line:33:build_from_yaml_file] - CRITICAL: Load pipeline config class RunnerConfig
2024-08-16 17:01:52,599 - runner_config.py[pid:15636;line:164:update_config] - CRITICAL: train model SAHP using GPU with torch backend
2024-08-16 17:01:52,602 - runner_config.py[pid:15636;line:36:init] - INFO: Save the config to ./checkpoints/15636_15008_240816-170152\SAHP_train_output.yaml
2024-08-16 17:01:52,602 - base_runner.py[pid:15636;line:170:save_log] - INFO: Save the log to ./checkpoints/15636_15008_240816-170152\log
2024-08-16 17:01:53.020743: I tensorflow/core/util/port.cc:153] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0.
2024-08-16 17:01:53.681071: I tensorflow/core/util/port.cc:153] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0.
WARNING:tensorflow:From C:\Users\siat\anaconda3\envs\pytorch_gpu\lib\site-packages\easy_tpp\utils\tf_utils.py:11: The name tf.disable_v2_behavior is deprecated. Please use tf.compat.v1.disable_v2_behavior instead.
WARNING:tensorflow:From C:\Users\siat\anaconda3\envs\pytorch_gpu\lib\site-packages\tensorflow\python\compat\v2_compat.py:98: disable_resource_variables (from tensorflow.python.ops.resource_variables_toggle) is deprecated and will be removed in a future version.
Instructions for updating:
non-resource variables are not supported in the long term
2024-08-16 17:01:56,720 - tpp_runner.py[pid:15636;line:60:_init_model] - INFO: Num of model parameters 6579
2024-08-16 17:01:57,165 - base_runner.py[pid:15636;line:92:train] - INFO: Data 'retweet' loaded...
2024-08-16 17:01:57,165 - base_runner.py[pid:15636;line:97:train] - INFO: Start SAHP training...
2024-08-16 17:01:59,117 - tpp_runner.py[pid:15636;line:96:_train_model] - INFO: [ Epoch 0 (train) ]: train loglike is nan, num_events is 360731
2024-08-16 17:01:59,340 - tpp_runner.py[pid:15636;line:107:_train_model] - INFO: [ Epoch 0 (valid) ]:  valid loglike is nan, num_events is 61288, acc is 0.42432450071792194, rmse is 5.000000000109721
2024-08-16 17:01:59,549 - tpp_runner.py[pid:15636;line:122:_train_model] - INFO: [ Epoch 0 (test) ]: test loglike is nan, num_events is 59634, acc is 0.42220880705637726, rmse is 5.00000000016465
2024-08-16 17:01:59,549 - tpp_runner.py[pid:15636;line:124:_train_model] - CRITICAL: current best loglike on valid set is -179769313486231570814527423731704356798070567525844996598917476803157260780028538760589558632766878171540458953514382464234321326889464182768467546703537516986049910576551282076245490090389328944075868508455133942304583236903222948165808559332123348274797826204144723168738177180919299881250404026184124858368.0000 (updated at epoch-NeverUpdated)

let me have a look. you are using tf or torch?

let me have a look. you are using tf or torch?

pytorch

Hi,
This is due to the very large values in delta_time, which causes the exponential term explodes.
https://github.com/ant-research/EasyTemporalPointProcess/blob/main/easy_tpp/model/torch_model/torch_sahp.py#L81
A temp fix is to use torch.clip:
states = torch.matmul(encode_state, mu) + (                 torch.matmul(encode_state, eta) - torch.matmul(encode_state, mu)) * torch.exp(             -torch.matmul(encode_state, gamma) * torch.clip(duration_t, max=10))
A better fix is i will clean up the data again and drop those event points with extremely large delta time intervals from the previous one, e.g., make the sequence break at that timestamp.

Hi,
I have committed the code in the master branch and i will clean the data again shortly.


Inconsistency in Allure report when switched the state of useCucumberStepReporter

Hi ,
I am using the below versions:
wdio-cucumber-framework@ 2.2.6.
wdio-allure-reporter@ 0.7.0
cucumber @4.2.1.
[1] My allure reports are totally different if I dont delete the old allure-results folder. I reckon its picking the last retained result data.. Is it an expected behavior? shouldnt that consider the data specific to the corresponding run? Kindly confirm?
[2] I have four feature file and 26 Scenarios [this count incude scenario outline.. if SO execution is accounted as individual , then total of 37 scenario]
The SUITES and FEATURES BY STORIES window in the OVERVIEW tab both displays the name of the feature files . Is this the right information?
[3] Previously I used useCucumberStepReporter:false and the SUITE window displayed 37 scenario in total. Because of a bug in webdriver.cucumber 2.0 version I set them as true as advised. But now I am seeing 4 items total. However this 37 is not reflected on the top Allure report summary graph as well. Does that mean webdriver wont account scenario-outline as multiple test scenarios??? Kindly confirm
[4] The above option does not show any ENVIRONMENT values. If I have the option of useCucumberStepReporter:false it displays the Capabalities and SPEC files in it. I am missing those information when I set them as true. How to fix that.
[5] With useCucumberStepReporter:true , I see FeatureNames in Suites and Behavior TAB menu. When I expand them I can see the corresponding scenario names. But I can t  see the TEST STEPS in it. How to configure that?
[5] Also, the TREND section is always blank. If I retain the old allure-results or reports folder, it creates lot other issues. but in either case, the TREND part is empty itself
Let me know if I am missing any information you are looking for. Thanks in advance.

Ignore first appearance of point [5] I can see the TEST STEPS in the Overview section.

[1] and [6] - not related to reporter. Ask in main allure repo.

[2] Is this the right information?

Nope. It displays feature name form head of feature file "Feature: Login feature"

[3] - check scenarion names. they must be unique.
[4] - yes it is. you may add it by yourself

Ask question in Gitter support channel https://gitter.im/webdriverio/webdriverio
Please don't create issues with questions here. Thanks

I do not have access to Gitter channel inside my corporate network. Hence raising the questions here. Do we have an alternative option to access gitter channel behind corporate network?


Uploader, stderr vs stdout in ERT context

All output from uploader is currently sent to stderr, stdout is (usually?) empty. Ideally, only errors should be sent to stderr which should be empty when no errors.
Investigate why this happens, and fix. Ideal outcome is:

Regular info is printed to stdout.
If errors, they are printed to stderr.
If no errors, stderr is empty.

Looks like Python logger defaults to stderr. This can be solved by including stream in BasicConfig, e.g.
logging.basicConfig(stream=sys.stdout)
Looks like other workflow jobs are generally only logging to stdout. Assume that entries will go into stderr if job fails.


Provide improved treatment for "Last Infectious Date"

We expect we will need to improve this. Planned for v3 is a fixed org-wide setting for a number of days after "Earliest Infectious Date"

Dup of https://github.com/covid19risk/permission-portal/issues/69


Call set_precomputed_quantities! in step!, remove from callbacks

This PR adds a call to set_precomputed_quantities! in step! and removes all of them from the callbacks. This PR cherry-picked the first commit in #2175, but is somewhat separate in that this PR will potentially be behavior changing.
Once this PR is merged (assuming there are no changes, or they are negligible), we should be able to follow through with closing out #2015.

bors r+


[Packager] Autoload for packager

Is it possible to write autoload for packager? I have app structure where every file has only one class that it is exporting with same name. I was able to write autoload for webpack using:
require.context to iterate over all files and creating classes from each file, I don't have huge dependencies and can manage them manually and I am adding all classes to global (bad, but i don't care).
Is it possible to do something similar with packager?
I need this, so within my app files I can use all needed classes without explicitly importing them.
Here my Autoloader for webpack.
'use strict';

import superagent from 'superagent';
import Immutable from 'immutable';
import React from 'react';
import Router from 'react-router';
import Bootstrap from 'react-bootstrap';
import RouterBootstrap from 'react-router-bootstrap';
import keyMirror from 'react/lib/keyMirror';
import merge from 'merge';
import Md5 from 'md5';
import {EventEmitter} from 'events';
import {Dispatcher} from 'flux';

class Autoload {
  constructor() {
    this.context = require.context('./../', true, /\.js$/);
  }

  addClass(className, source)  {
    if (className in window) {
      throw `Class ${className} already defined`;
    }
    window[className] = source;
  }

  loadClasses(classNames, classToFile) {
    classNames.forEach(className => {
      const fileName = classToFile.get(className);
      const source = this.context(fileName);
      this.addClass(className, source);
    });
  }

  loadAll() {
    // Load libraries
    this.addClass('Immutable', Immutable);
    this.addClass('React', React);
    this.addClass('Router', Router);
    this.addClass('Bootstrap', Bootstrap);
    this.addClass('RouterBootstrap', RouterBootstrap);
    this.addClass('superagent', superagent);
    this.addClass('keyMirror', keyMirror);
    this.addClass('merge', merge);
    this.addClass('Md5', Md5);
    this.addClass('EventEmitter', EventEmitter);
    this.addClass('Dispatcher', Dispatcher);
    // Load our code
    var allFiles = Immutable.List(this.context.keys());
    var classToFile = Immutable.Map();
    allFiles.forEach(fileName => {
      const className = /.*\/(.*)\.js$/.exec(fileName)[1];
      classToFile = classToFile.set(className, fileName);
    });
    // TODO: clowntown
    const priorityClasses = Immutable.List([
      'AppContext',
      'AbstractPage',
      'AbstractStore',
      'RecordStatus',
      'UserRecord',
      'StatusRecord',
      'SubmissionRecord'
    ]);
    this.loadClasses(priorityClasses, classToFile);
    priorityClasses.forEach(className => classToFile = classToFile.delete(className));
    this.loadClasses(Immutable.List(classToFile.keys()), classToFile);
  }
}
new Autoload().loadAll();

Maybe I'm misunderstanding, it seems like you just want to have your modules loaded and exposed globally automatically? Could you not create a single file, require that in your main app, and do something like this to make everything attached to the global object?

Once #1497 lands you'll be able to provide your own implementation of the JS source code transformer. So in a custom transformer you could add the require calls to each file, similar to what you've done in the webpack plugin.

@brentvatne - that's what I do now, but it is kind of inconvenience to modify this file every time I add or change name of the file/class.

Closing due to inactivity


Serviceinfoids word creates ambiguity.

Link : https://github.com/secure-device-onboard/pri-fidoiot/blame/master/demo/README.md
Line No 141: Can we mention the table name(OWNER_SERVICEINFO) and corredsponding header names(SVI_ID) instead of serviceinfoids or can it be renamed as serviceinfo ids.

No longer valid after README refactor related to ServiceInfo changes.


Decimal numeric literals in Scala sources should not be highlighted

Originally described in now closed #101:
I find the current highlighting a bit too much aggressive. In some cases it highlights things which are not colors at all.
Decimal numeric literals in Scala sources:

  val rng = new scala.util.Random(123456)
  val seeds = (0 until 10).map(_ => rng.nextInt(1000000))

I would like to have hex colors highlighted  only when starting with 0x or #.

Note: it seems when the option "Detect hex colors" is not enabled, plain numbers without 0x or # are not highlighted. Perhaps the option could be made more explicit, like "Detect hex color without prefix", or inverted "Detect hex colors only when prefixed with 0x or #".

I don't plan on releasing an update just for wording. Closing this as this is irrelevant.


💸 Sponsorship tiers [due Nov 9]

The Commonhaus Foundation relies on the generous support of our sponsors.
Working closely with advice from our @commonhaus/advisory-board, we’ve established a set of sponsorship tiers to recognize our supporters. In keeping with our theme of transparency and collaboration, we will manage sponsorship details within the foundation repository. Specifically, the SPONSORS.yaml file will list our sponsoring companies along with their display and description information, similar to how we handle projects with PROJECTS.yaml.
I’ve also put the machinery in place to generate two webpages from this content (currently placeholder content, not yet indexed):

Monetary Sponsors: This page will highlight sponsors who provide financial support.
In-Kind Sponsors: This page will credit those who provide infrastructure, software, and other essential services. The focus will be on the services they provide, rather than a company profile.

In terms of character limits for each tier, I’ve kept it concise:

Silver: Essentially the length of a tweet (240 characters).
Gold: Double that length (480 characters).
In-kind: A tweet. I would like this page to focus on the services provided.

I’d like to gather feedback from everyone, though this is primarily a council decision, as it affects the foundation’s operations. Input on the wording, layout, and structure is very welcome before we proceed to finalize the content.
voting group: @commonhaus/cf-council
Do one of the following:

Approve the PR or react with 👍 (:+1:) if it looks good to you
Review with Comments or react with 👀 (:eyes:) if you're "ok" with it (it may not be your favorite)
If you think it needs discussion or revision

Create a review, add your comments and require changes
Use the +- button to make a suggestion (instead of just adding a comment).

[!NOTE]
This PR also consolidates linting/validation-related code into the .github directory, and updates YAML validation to include checking sponsor descriptions against character count limits (excluding html links, punctuation, and whitespace).

vote::result
3 councilors are on-board + other positive responses. Will start here, and revise later as needed.


how to use this library in nougat

As we know that in android we cannot address direct path to any file. so how to use this library in android Nougat.
I want to select a video file from device and upload to server it is working below Nougat well. but in Nougat it does not returns the real path of file.
so how to use this in Nougat?
please notify soon

Can you give me logs? Also, can you try the sample application and see if it works?

I was using a privious version of library, Now it's working thanks.

We're using 1.1.31 and are get feedbacks that it doesn't work in Nougat. Did you test it in Nougat? Should we use a more advanced build?

Do you have any logs.
On Aug 10, 2017 4:32 PM, "Queskr"<EMAIL_ADDRESS>wrote:

We're using 1.1.31 and are get feedbacks that it doesn't work in Nougat.
Did you test it in Nougat? Should we use a more advanced build?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/coomar2841/android-multipicker-library/issues/87#issuecomment-321520717,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOdKcxfQjXi6EN-FDS84_SEKfa86BDPks5sWuNjgaJpZM4MOiwa
.


country_select with nil name still render a name

I have a stripe powered form, most of the fields are presented without name as expected. country_select however insert empty brackets for its name, causing it to get pulled up.
<%= country_select nil, nil, { selected: 'US' }, { 'data-stripe' => 'address_country', autocomplete: 'billing country', class: 'country form-control', required: true } %>

<select data-stripe="address_country" autocomplete="billing country" class="country form-control" required="required" name="[]" id="_" data-bv-field="[]"></select>

This seems to represent a pretty good use case for bringing back access to the helper that produces just the option tags or the options array, at least.
For now, I'd recommend you not use country_select and rely on generating the options from the countries gem.

Here's how you can get around this right now and still use country_select
<%= country_select nil, nil, { selected: 'US' }, { 'data-stripe' => 'address_country', autocomplete: 'billing country', class: 'country form-control', required: true, id: nil, name: nil } %>

I've just added name: nil and id: nil

I think this should be fixed in a future version, but it's going to take some time to dig into why it's generating [] for name.


Update hibernate-validator to 7.0.2.Final

Updates org.hibernate:hibernate-validator from 6.2.2.Final to 7.0.2.Final.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.hibernate", artifactId = "hibernate-validator" } ]

labels: library-update, commit-count:1

Superseded by #1415.


🔨 Rename DynamicBufferModule to DynamicBufferMixin

What is the motivation for this task?
The DynamicBufferModule class, which is used to allow dynamically changing the shape of tensors registered as buffers, essentially behaves as a mixin and should be named as such.
Describe the solution you'd like
Rename DynamicBufferModule to DynamicBufferMixin.
Additional context
We could consider moving the mixins to their own submodule under models.components

also add some unit tests

Hii @djdameln
I would like to take up this issue.
Also, I would love to know get any advice and insights from your side before starting.
Thanks:)

Hi @kamlesh-ops, @danylo-boiko just created PR https://github.com/openvinotoolkit/anomalib/pull/1776 to address this issue

Ohh no problem. @samet-akcay
Btw let me know if I can still contribute in any way. I'll be looking for other issues till then
cheers:)

sure, thanks for your interest. There are bunch of Good First Issues there. I'll continue to label more.


Using Blaze Enchanters for crafting

Describe the your request
Hello,
My suggestion would be to add the possibility of crafting with the blaze enchanter via datapacks in the same style as Apotheosis's infusion crafts.
For example:
{
"type": "create_enchantment_industry:enchanting",
"ingredients": [
{ "guide": "minecraft:nether_star" },
{ "item": "minecraft:golden_apple" },
{ "fluid": "create_enchantment_industry:experience",
"amount": 200 },
{ "time": 40 }
],
"results": [
{ "item": "minecraft:enchanted_golden_apple"  }
]
}
This would be very practical for automating crafts with experience without using a simple Spout, visually it would be much classier ^^

This should be added, but it's not a high priority at the moment

Current version of this mod had it.  Why is this still open?


people.get("me")  no longer works

when using the ciscospark nodejs lib people.get("me") no longer works.  This has broken the botkit library.
The problem is in the @ciscospark module.
https://github.com/ciscospark/spark-js-sdk/blob/master/packages/node_modules/%40ciscospark/plugin-people/src/people-batcher.js#L69
Should be:
resource: people/${ids}
Also the handleHttpSuccess function should check to see if there are items...because the get("me") will not return an array of items.

@marchfederico didn't realize this was even possible! I've added specific support for it in PR #678


Remove permit_join from options

@danielwelch I opened this issue as a discussion.
My recommendation would be to:

Remove permit_join from options
Always make permit_join false
Allow changing of permit_join only through mqtt

This way it is possible to support implementation of an input boolean (https://www.home-assistant.io/components/input_boolean/) to enable/disable joining.
My proposed approach:

Create an input boolean that defaults to false
Create an automation that reads the zigbee2mqtt/bridge/state to reset the input boolean to false
Create an automation to send mqtt on zigbee2mqtt/bridge/config/permit_join when pressing this input boolean (as toggle true/false)

Note: If you have recorder enabled and allow permit_join to be received from @Koenkk on zigbee2mqtt/bridge/state we could get around removing permit_join from options.

Documentation for allowing permit_join through UI is complete in https://github.com/danielwelch/hassio-zigbee2mqtt/pull/21 bundled together with the pm2 feature.
Sample log messages:
2018-5-28 21:57:24 INFO Zigbee: allowing new devices to join.
2018-5-28 21:58:20 INFO Zigbee: disabling joining new devices.

UI:

Let's leave permit_join in for now, defaulting to false, and once we can look at removing the option altogether once this is battle-tested

I will close this issue. Will open a new one once we have a few stable releases.


New package: DuckStudio.FufuTools version 1.3.1-1

Pull request has been created with komac v2.3.0 :rocket:

Resolves #155966

Microsoft Reviewers: Open in CodeFlow

I'm the author of the Fufu Tools.

emm... I believe there is nothing wrong with my installation package, and I can implement silent installation in the terminal.
Commands:
.\Fufu_Tools_Setup.v.1.3.1.exe /verysilent /suppressmsgboxes

The download link has been updated and the link mentioned in this PR is not valid.

@Luna-Grace We can update this package later

Still can't pass check？

Maybe it should wait for a while due to a possible security issue

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-06-01 06:09:58.110 [FAIL] Installer failed security check. Url: https://github.com/DuckDuckStudio/Fufu_Tools/releases/download/v1.3.1-1/Fufu_Tools_Setup.v1.3.1-1.exe Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 882.)

@wingetbot run

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-06-03 16:39:05.424 [FAIL] Installer failed security check. Url: https://github.com/DuckDuckStudio/Fufu_Tools/releases/download/v1.3.1-1/Fufu_Tools_Setup.v1.3.1-1.exe Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 883.)

Still can't pass check？

Unfortunately, some packages take a while. Some might get installed into a VM and watched, to make sure there is no "time release malware" in it.

@wingetbot run

@wingetbot run

I submitted it to the Microsoft Security Intelligence website, but there still seems to be a problem.
Of the files in question, two are about file lookup, a category of programs that allow users to enter folders and scan folders and get associated file names to feedback to users, without networking.  (The newly written search program actually passed the check)
The other check results don't give a specific path (there are multiple files with the same name), so I can't tell which file it is.
I still think the program is safe because I wrote it myself, and most of the tools don't require networking (including all of the faulty tools).
This is really weird.

我将其提交到了_Microsoft Security Intelligence_网站上，但似乎仍有问题。
出现问题的文件中，有两个是关于文件查找的，这个类别的程序是让用户输入文件夹后对文件夹进行扫描并获取相关文件名反馈给用户，全程无需联网。（新写的_查找程序竟然_通过检查了）
其他的检查结果未给出具体指向路径（有多个同名文件），故我无法判断具体是哪个文件。
我依旧认为程序是安全的，因为他是我自己写的，并且大部分工具无需联网（包括所有出现问题的工具）。
这真的很奇怪。

@wingetbot run

/azp run

https://github.com/microsoft/winget-pkgs/pull/156423#issuecomment-2159426718

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-06-11 01:10:17.274 [FAIL] Installer failed security check. Url: https://github.com/DuckDuckStudio/Fufu_Tools/releases/download/v1.3.1-1/Fufu_Tools_Setup.v1.3.1-1.exe Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
(Automated response - build 884.)

I submitted it to the Microsoft Security Intelligence website, but there still seems to be a problem. Of the files in question, two are about file lookup, a category of programs that allow users to enter folders and scan folders and get associated file names to feedback to users, without networking. (The newly written search program actually passed the check) The other check results don't give a specific path (there are multiple files with the same name), so I can't tell which file it is. I still think the program is safe because I wrote it myself, and most of the tools don't require networking (including all of the faulty tools). This is really weird.

我将其提交到了 Microsoft Security Intelligence 网站上，但似乎仍有问题。 出现问题的文件中，有两个是关于文件查找的，这个类别的程序是让用户输入文件夹后对文件夹进行扫描并获取相关文件名反馈给用户，全程无需联网。（ 新写的 查找程序竟然 通过检查了 ） 其他的检查结果未给出具体指向路径（有多个同名文件），故我无法判断具体是哪个文件。 我依旧认为程序是安全的，因为他是我自己写的，并且大部分工具无需联网（包括所有出现问题的工具）。 这真的很奇怪。

Thanks for your patience with the process. I didn't realize so much of the "other side" of a Defender scan was so open.
Do these have icons - were the icons generated by a website? A newer attack vector is appending virus data after image data, which hides it from the developer.

@wingetbot run

Replying to stephengillie's comment:
Thank you for your comment and sorry for any issue caused. I wanted to provide some clarification regarding the icons used in my programs:

The icons for my installer and each tool are designed by me.
The installer icon is specified in the following configuration:

[Setup]
SetupIconFile=D:\鸭子家园\Gitee\芙芙工具箱（Windows小工具集）发行版\v1.3.1-1\本地\Dev-Files\Fufu_Tools_Setup_ico.ico

The icons for each tool are set using the following command:

pyinstaller [--noconsole] --onefile -i "{icon_path}" --distpath={output_dir} {file_path}

The packaging of each program is managed by a Python script I wrote, which can be found here: Pack script
The program icons are also available in the repository at: ico.ico
Related files in the packaging process: Dev-Files(v1.3.1-1)
I hope this clears up how the icons are added to my programs.
Thank you for your understanding.

I may not have fully understood your question, and I am not clear on which icon you are referring to (which website are you mentioning?).
And I don't really understand "the other side."...
Some parts of this reply were translated by ChatGPT, and the translation may not be entirely accurate.

@wingetbot run

@wingetbot run

/azp run

I wanted to provide some clarification regarding the icons used in my programs:
The icons for my installer and each tool are designed by me.
The installer icon is specified in the following configuration:

Thanks for the thorough description of how you create and package icons into your application. It's much more advanced than my process, and avoids the issue I described.

I think PR #156134 is better...

Emmm... This version seem has some issue (DuckDuckStudio/Fufu_Tools #46)...
刚在VM中测试发现的，马上跑去修...

[!WARNING]
If you use silent installation, you must start the program in administrator mode.
(Issue DuckDuckStudio/Fufu_Tools #46)

I'm the author of the Fufu Tools.

Emm... I believe there is nothing wrong with my installation package, and I can implement silent installation in the terminal. Commands:
.\Fufu_Tools_Setup.v.1.3.2.exe /verysilent /suppressmsgboxes

The complete process:
Video

我会重新创建一个拉取请求

#157822


fix(db) support for cassandra 4

Summary
Kong 014.0 fails to start when backed by Datastax Enterprise 6.X with error
could not load routes: [Invalid] table schema_columns does not exist
DSE 6.X reports 4.0.X as the release_version in system.local table.  The system schema components used by Kong in 3 are the same in 4 so changes were made such that schema selection based on cassandra major_version assumes future versions use the same schema.
Issues resolved
Fix #3837

Thanks! Manually merged with a few minor tweaks (updated a missing location).


Disable sparse build on Windows by default

Cherry-pick the change to disable sparse functionality on Windows for the ROCm 6.1. This can still be built from source on Windows if the appropriate libraries are available, but it's not going to be in the AMD-provided build.
Ticket: SWDEV-435846

Should this not target release/rocm-rel-6.1 instead of release-staging/rocm-rel-6.1?

I'm not sure. I suppose I would imagine that it would be merged to release-staging/rocm-rel-6.1 and then fast-forwarded into release/rocm-rel-6.1? Maybe I need to review the procedures again...

This change fixes the build, but the Windows tests seems to be crashing intermittently (both in this branch and on develop).

Fix for https://ontrack-internal.amd.com/browse/SWDEV-435846


how can i get the mask of the generated text ?

each loop i return the mask，all the loop finished，how can i get the mask of the final_img ？ i do it by bitwise_or. but i find the mask have different values ?  the value is related to the shadow.
i want to konw how can i get the binary_mask of the generated text?

May be this is helpful?

each loop i return the mask，all the loop finished，how can i get the mask of the final_img ？ i do it by bitwise_or. but i find the mask have different values ? the value is related to the shadow.
i want to konw how can i get the binary_mask of the generated text?

each loop i return the mask，all the loop finished，how can i get the mask of the final_img ？ i do it by bitwise_or. but i find the mask have different values ? the value is related to the shadow.
i want to konw how can i get the binary_mask of the generated text?

Have you solved it? This problem has bothered me for a long time


Page throws timeout exception only in headless mode

Steps to reproduce
Tell us about your environment:

Puppeteer version<EMAIL_ADDRESS>Platform / OS version: macOS High Sierra 10.13.5
URLs (if applicable): https://www.michaelkors.eu/en_PL/cotton-polo-shirt/_/R-CU85FGVC93?color=2199
Node.js version: v6.9.0

What steps will reproduce the problem?
  const browser = await puppeteer.launch({
    headless: true,
  });

  const page = await browser.newPage();
  await page.goto("https://www.michaelkors.eu/en_PL/cotton-polo-shirt/_/R-CU85FGVC93?color=2199", {timeout: 30000});
  console.log(await page.content());

What is the expected result?
Promise does not throw "Navigation Timeout Exceeded" no matter if using headless or non-headless mode.
What happens instead?
That url throws Promise throws "Navigation Timeout Exceeded" but only in headless mode.
Similar to:
https://github.com/GoogleChrome/puppeteer/issues/1922

@aslushnikov where to write DEBUG=*session node a.js? In powershell it is giving error
`DEBUG=*session : The term 'DEBUG=*session' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling
of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1

DEBUG=*session node a.js

  + CategoryInfo          : ObjectNotFound: (DEBUG=*session:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException`

using a Windows machine


Process crashes while connecting to Database on Node v11.15.0

Hi,
All worked fine on Windows with Node version v10.14.2.
I did fresh install of Node v11.15.0 on Linux box and tested project.
When I try to open DB connection, process exists with below failure:
error Command failed with signal "SIGSEGV".
Is it possible it is specific to Node version I am using.
Anyone tested 2.0 with above exact version or seeing similar issues?

Quick update:
I tested again on Linux with different Node versions.
It works well with 8.16.0 and performance improvement is almost 50% [at least for my scenario]
But I could not get it working on 10.x or 11.x on Linux.
Either error or crash in these versions - at least for me.

I run mine on Linux (RHEL) at Node v 10.16. On your fresh Linux boxes do you have all your unixODBC stuff set up (drivers in odbcinst.ini, DSNs in odbc.ini)?

Yes. On linux box, I have ODBC is installed and I use MS SQL ODBC driver version 13:
/libmsodbcsql-13.1.so.6.0
I can use version 17 which is more latest if that helps.
But with older Node version it worked fine even with 13 - so did not bother.
I have quick question: We are bit pressed to improve performance and certainly new version is much better mainly for large data. Do you know of any quick workaround or API in older version 1.4.6 that we can tweak to just get performance improvement part - while we wait for 2.0 to stabilize for production use - as we need to complete more testing on 2.0?

I tried with Windows 10, Node.js 12.4.0, and using the MS SQL ODBC driver version 13. Seemed to work find in that case. I will keep trying different configurations and see if I can recreate the SIGSEGV. Is there any other information it gives? (Usually not with a segfault, but though I'd try).
As for your question, the answer is unfortunately no. 2.0 began as a rewrite of how data is retrieved to improve performance, migrating from using SQLGetData to SQLBindColumns. Unfortunately this is an involved process, and not something where you can just change a few dozen lines, more like a few hundred. Another improvement may have come from switching from NAN to N-API as API for building Node.js addons in C/C++. N-API is ABI stable, meaning you should be able to take your odbc.node file in the build/ directory as compiled in Node.js 8 and use it in Node.js 10, 11, 12, etc. Again, the switch from NAN to N-API requires you to alter hundreds of lines of code.
The only v1.x improvement I could think of would be to take out the SimpleQueue class from the JavaScript library portion. I think this class exists to slow down queries so they don't hammer the database, so removing it (and rewriting the code in the Database class) might make things noticeably faster. It also might break the whole package!

Thank you. appreciate getting back on this. As of now I am able to set to 8.16.0 node version and get things going. Will retest higher versions once we have stable runs in 8.16.0 as needed.

I'm experiencing the same issue.
@markdirish - I have a minimal-case docker config that can replicate this against SQL Server 2016. Would it be useful to you if I posted that somewhere?

@erayd
Yeah, I still haven't been able to replicate this, so a docker config would be helpful in tracking it down. Thanks!

@markdirish
Here you go - minimal test case. To replicate against SQL Server 2016:

Edit the database definition in etc/odbc.ini to point to a real database
Edit the username and password in app/main.js to be a real account
make container
make run

If you want to investigate inside the container, I've also added a make shell target for you.
segfault-demo.tar.gz
Please let me know if there's anything I can try on my end that would be useful.

I also cannot get this working on Node 10 but have it working fine with Node 8
I am using ODBC Driver 13 & Node 10.16.3
I am getting a different error "Segmentation fault (core dumped)". Is there a way I could get a deeper insight into what is actually failing?

You can build with 'DEBUG' in the defines, and it should show the last thing that happened before the segfault which can be helpful. I've noticed a few SEGFAULTs when running this package as well, mostly dealing with closing connections during pooling.
I'm hoping to push out v 2.2.0 sometime next week to make the package more stable.

Hey @markdirish,
Thanks for the great Lib. I had been using this for quite a while without issues running on node 8.x on both windows (development) and docker in Linux for quite a while.
We recently upgraded to nodeJS 12.3 and I used your 2.2.2 release which is the latest right now. In windows the library runs just fine no matter what query I throw at it, however in Linux (docker) .. it throws the SEGFAULT (SIGSEGV) while executing some queries only.
Steps to reproduce:

Use Connection and not Pool
query the statement given below
the error throws at line#71 in Connection.js which reads something like return this.odbcConnection.query(sq....

The query:
select a.tbcreator as \"schema\", a.tbcreator as \"database\", a.tbname as table_name, b.type as table_type, a.name as column_name, a.default as column_default, a.coltype as type_name, a.length as character_maximum_length, a.scale as decimal_digits, a.nulls as is_nullable, a.colno as ordinal_position from sysibm.syscolumns a inner join sysibm.systables b on a.tbname = b.name where b.type in ('T', 'V') and a.tbcreator not in ('SYSIBM', 'SYSCAT', 'SYSSTAT', 'SYSIBMADM', 'SYSTOOLS') and b.creator not in ('SYSIBM', 'SYSCAT', 'SYSSTAT', 'SYSIBMADM', 'SYSTOOLS', 'SYSPUBLIC') limit 1

Note that a simple query select blah from table seems to work just fine.
The above stated query also works just fine on my windows laptop running windows 10.
The docker container's root image is node:12.13.0, I do add a lot of stuff on top though.

@markdirish after some more digging.. the fault seems to arise at this statement at line#720 in odbc_connection.cpp c++ file:
data->sqlReturnCode = odbcConnectionObject->RetrieveResultSet(data);

P.S. Digging = adding lots of print statements :)

I've squashed a lot of process crashes in the last few years, and I am going to assume that this issue has been fixed as part of that (since I haven't received similar issues recently). Also, since Node 8 and 10 are out of support, and 12 will soon be (this month), I'm hoping the issue no longer persists in Node 14+


Fix build image script

When I execute php build_image.php, displayed the error.
montage: unable to open image 'img-': No such file or directory @ error/blob.c/OpenBlob/3146.
montage: no decode delegate for this image format `' @ error/constitute.c/ReadImage/509.
Something went wrong

So I changed to pass to list file.

the build script works fine on ubuntu, which is my target build machine. i'm not planning to merge this PR


Video_player plugin. Add HTTP headers support like in Image.network

Use case
My application is using video_player to show videos from a web site. Authentication is required to access media files.
With Image.network I can use headers argument and pass cookies or token. But VideoPlayerController.network does not have this ability
Proposal
Add support of passing arbitrary HTTP headers to VideoPlayerControler.network constructor.

Duplicate of #16466 :)
@Olegas note the issue blocking $16466 should not prevent using cookies, just other types of headers. I am going to work on a PR for that.

Closing as duplicate of https://github.com/flutter/flutter/issues/16466.


BREAKING(collections): remove groupBy()

To be merged for v0.211.0.

This change made me greatly confused while exploring @std/collections.
groupBy is the main reason why I'm reaching out to lodash-like libraries, so not being able to find it at first, and then having to discover it was removed (without a trace of explanation) was a bit frustrating.

Apologies, @av. Yes, we should've included relevant info. groupBy() was replaced by the JS-native Object.groupBy().

Thanks for getting back to me. I figured it out, just wanted to leave some feedback for the team. Thank you!


Integration with monolog reporting errors of all levels

Greetings to you all.
I have a problem with integration with monolog. It logs all levels of errors and changing the configuration doesn't help.
Repro steps:

Followed this tutorial to the letter: https://github.com/getsentry/sentry-symfony#installation
Special attention to this

// monolog.yaml

sentry:
  type: service
  id: sentry.monolog.handler
  level: error

Added this to my test controller:

public function testMonolog(LoggerInterface $logger)
{
  $logger->error('test');
  return $this->json('ok');
}

Expected behavior
To receive Error with message 'test'
Real behavior
Received following errors:

Populated the TokenStorage with an anonymous Token.
Matched route "{route}".
Error 'test'

This does not seem to be a problem with the bundle, but with the underlying base SDK. I'll move the issue there.


Installing .NET Core 1.0.1 Windows Server Hosting fails with error 0x80072f76

I'm installed the .NET Core 1.0.1 Windows Server Hosting on a VM, but it keeps failing with:
0x80072f76 - Unspecified error.
I've inlcuded the install log file.
dd_DotNetCoreWinSvrHosting__20160928152049.txt

I think that's the VC++ redist. Try a manual install from https://www.microsoft.com/en-us/download/details.aspx?id=53840 and run the installer again.

Yes, that fixes it.
This was on a Windows Server 2012 R2 VM. Shouldn't the installer at least tell the user this is a required dependency and offer a link where to download it from?

@joeloff will prob be able to say. This comes up from time-to-time, mostly when the machine doesn't have an Internet connection. The installer was fetching the VC++ redist from the Internet, so that would go sideways with no connection.
I currently have this listed in the Publish to IIS doc's Common Errors section, but I don't have it in the ANCM doc at https://docs.asp.net/en/latest/hosting/aspnet-core-module.html ... at least not yet.
May I ask: Which docs did you consult, if any, before running the installer? That may give us a clue where a note on this would make the most sense (besides, as you pointed out, from the installer itself).

The VC redist requirement is a 3rd level dependency that comes from the CLI installer. The ideal fix is that the CLI that ships with the server bundle should have an embedded copy of the vc redist. I'll raise this again.

@joeloff ... then I'll hold off on sweating a doc update anywhere. I'll keep an eye out for an issue where you're working on bundling it.

@GuardRex: The thing is, the VM does have a working internet connection. So if the installer wanted to download the VC++ redist, it should have been able to.
I got instructions on this package from https://docs.asp.net/en/latest/publishing/iis.html
That page also links to the page you mentioned and I viewed that as well.
I found the page through googling (https://www.google.nl/search?q=install+asp.net+core+iis&oq=install+asp.net+core+iis&aqs=chrome..69i57.3463j0j4&sourceid=chrome&ie=UTF-8)

Just noticed @joeloff comment. Which makes it kind of ironic, since I didn't even want to install the full Core framework, just the IIS handler. I'm creating a self-contained application, so I was looking for a download of just the ACNM. I did found how to do this, by providing a command-line parameter to the installer, but a separate download is much more easy to discover in my opinion.

@GuardRex It won't be a change we'll make. It will have to come from the CLI owners, but I'll raise the issue with them again. The alternative is that we simply add it to the hosting bundle, which might be simpler.

@joeloff Ok, I'll keep an eye out either way so I'll know if I need to beef-up those two docs.
@comburo

the VM does have a working internet connection

Interesting. There is another failure scenario at play then.
Thanks for letting me know on the docs you visited. I have it in the Common Errors section but not in the ANCM doc, and the language of a failure may have changed since I wrote that bit.
I'm going to wait and see how @joeloff fares with getting it bundled before making a doc change. I think that would lick this once and for all.

but a separate download is much more easy to discover in my opinion

I heard from them that at some point that they were going to make the ANCM-only install a better experience than a command-line parameter ... checkbox or something. That was mentioned a while ago tho ... I think @DamianEdwards mentioned it in a Community Standup a couple of months ago.

There was an issue recently with the redist download link embedded in the .NET Core 1.0.1 installer. I'm told they're rebuilding the installer. https://github.com/dotnet/cli/issues/4190

The CLI was rebuilt and we're rebuilding the server hosting bundle. The CLI installer was using an FWLINK to locate the VC redist. When VS shipped it's latest update (3.5), the FWLINK was updated so on clean installs the CLI will fail because it's expecting an older copy of the VC redist and the file hash no longer matches.
The fix only addresses the file hash issue, the VC redist is still an external download that runs when installing on a machine without the redist

Closing as "external".

@joeloff I have two error exceptions in the log now for when the VC++ isn't installed.
My testing shows 0x80072efd ... but @comburo shows 0x80072f76 in this issue.
Should I list them both in the common errors entry of Pub IIS?

I have Windows Server 2012 x64 (no R2, no SP), i tried to install Windows Server Hosting 1.0.5&1.1.2, but i received "Error 0x80072ee7: Process returned error: 0x2ee7". I have installed vc_redist.exe already, being administrator on that machine. It seems to install AspNetCoreModule, but not runtime. Any ideas?

@IonRobu The runtime install in the bundle will attempt to detect the VC redist. There was an issue with a VC redist release around the time of VS2015 Update 3 RC where the registry keys it wrote were absent, causing the runtime to think that vc_redist wasn't present and then attempting to install an older version of it. That is most likely the reason for the failure.
Two asks

There should be a log from the vc_redist that the runtime tried to install in your %temp% folder. Could you maybe attach that.
Can you confirm what the version of the vc_redist is that you have installed? You should be able to find it in ARP (From the start menu, just launch appwiz.cpl to open the Programs & Features console)

@joeloff You're right, i pre-installed vc_redist 14.0.23026, but runtime 1.0.5-1.1.2 requires 14.0.24215 version. Because was not there, it tried to download it during install -> i have no internet on that machine -> setup failed. Questions: 1. is it possible to embed vc_redist in runtime setup? or 2. is it possible to display explicit error messages when setup fails?
Thanks!

@IonRobu In 2.0.0 we've removed the vc redist dependency, but for 1.0.0 there's unfortunately not much we can do unless we ship a rebuild (that's very unlikely). The big problem with surfacing a decent error message is that it's coming from a nested bundle (the redist is nested into the runtime which in turn is nested into the server host).
The error you received is from the runtime trying to go online to download the redist. You're only recourse would be to download a copy of the redist and install that prior to running the server host.
However I will bring this up with the runtime owners in case there is another 1.x release. The other option would be to include the redist in the server host and simply install it before the runtime. I'll open an issue for the hosting bundle so that we can include the redist if we ship another 1.x release.

Oh, but this is enough, to eliminate vc_redist dependency! It's a good thing! So, this effort, to solve the problem in 1.x, probably has no sense, since 2.0 is knocking on the door.

I am still facing the problem when i try to install ANCM 2.0 bundle on Winserver 2016 which has ANCM 1.1.0 installed.


feat: Prevent multiple dotnet restore executions

This approach runs the dotnet tool restore command in the SourceGenerator project because it's a single-target-framework project. The command runs sequentially for MudBlazor and MudBlazor.Docs project.
Description

Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

The build is same for pipeline as for local.
You are not clearing your nuget cache that's why its working locally

The build is same for pipeline as for local. You are not clearing your nuget cache that's why its working locally

I'm clearing my NuGet cache locally. I can reproduce the pipeline behavior by running dotnet run --configuration release --project "src/MudBlazor.Docs.Compiler/MudBlazor.Docs.Compiler.csproj" without building the solution before. For me, it looks like the dotnet run compiler command doesn't depend on the dotnet build --configuration Release command.

The build is same for pipeline as for local. You are not clearing your nuget cache that's why its working locally

I'm clearing my NuGet cache locally. I can reproduce the pipeline behavior by running dotnet run --configuration release --project "src/MudBlazor.Docs.Compiler/MudBlazor.Docs.Compiler.csproj" without building the solution before. For me, it looks like the dotnet run compiler command doesn't depend on the dotnet build --configuration Release command.

Ok you probably have a cached version on the Docs.Compiler.
Try a dotnet clean as well as delete your nuget cache.

Also note MSBuild is not sequential. It will build as many projects concurrently as it can. The dependency tree of the project references will also dictate build order within that concurrency. You can't rely on one command being executed before another across projects.

With sequentially I meant that there are restore commands and they will run in the order as defined.
I thought this approach is a nice kind of workaround. 😆

Try removing the bin folder for the Docs.Compiler.
For performance we use a compiled version if it exists (see the csproj)
If it doesn't exist we use the project and it has to compile first  which will make the build process is different.
The pipeline does nothing different to the local build it is exactly the same process.
The difference is you local build has build artifacts that already exist.
If you want to be really sure you are running the same build as the pipeline you need to go from a git clone.  But you should be able to recreate the error locally by deleting any previous build output.

Thanks for trying.

The command runs sequentially.

Also note MSBuild is not sequential. It will build as many projects concurrently as it can. The dependency tree of the project references will also dictate build order within that concurrency. You can't rely on one command being executed before another across projects.

With sequentially I meant that there are restore commands and they will run in the order as defined.
I thought this approach is a nice kind of workaround. 😆

If you can get it to work that would be great thank you for trying.

@xC0dex Can you post your command for deleting the nuget cache.
For me I always reproduce the build error after deleting the nuget cache directory

@xC0dex Can you post your command for deleting the nuget cache. For me I always reproduce the build error after deleting the nuget cache directory

I'm doing the following:

Cleaning the solution
Deleting the excubo.webcompiler folder in the .nuget directory manually
Deleting the AppData\Local\NuGet\v3-cache HTTP cache manually
Build the solution

This build was successful on my local machine.

@xC0dex OK thanks for that it still fails on my machine which matches what the build server is doing.  I am just about to post a PR which switches to a MSbuild task as discussed on discord

However, I noticed that sometimes a new error occurs. The error moved from the dotnet tool restore to the dotnet webcompiler command and no longer effects the NuGet file, but the generated MudBlazor.js or MudBlazor.css.
 A compilation step encountered an exception The process cannot access the file 'C:\Users\justi\git\github\MudBlazor\s
  rc\MudBlazor\wwwroot\MudBlazor.min.js' because it is being used by another process. with stacktrace:
     at Microsoft.Win32.SafeHandles.SafeFileHandle.CreateFile(String fullPath, FileMode mode, FileAccess access, FileSh
  are share, FileOptions options)
     at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare sh
  are, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
     at System.IO.File.OpenHandle(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options,
  Int64 preallocationSize)
     at System.IO.File.WriteAllBytes(String path, Byte[] bytes)
     at WebCompiler.Compile.Compiler.ReplaceIfNewer(String output_file, Byte[] new_content) in /home/runner/work/WebCom
  piler/WebCompiler/WebCompiler/Compile/Compiler.cs:line 30
     at WebCompiler.Compile.Place.Compile(List`1 file_sequence) in /home/runner/work/WebCompiler/WebCompiler/WebCompile
  r/Compile/Place.cs:line 49
     at WebCompiler.Compile.CompilationStep.With(Compiler compiler) in /home/runner/work/WebCompiler/WebCompiler/WebCom
  piler/Compile/CompilationStep.cs:line 22

So even if the dotnet tool restore works, the command execution fails sometimes.

@xC0dex OK thanks for that it still fails on my machine which matches what the build server is doing. I am just about to post a PR which switches to a MSbuild task as discussed on discord

Do you mean this task? BeforeTargets="DispatchToInnerBuilds"

@xC0dex OK thanks for that it still fails on my machine which matches what the build server is doing. I am just about to post a PR which switches to a MSbuild task as discussed on discord

Do you mean this task? BeforeTargets="DispatchToInnerBuilds"

No using your PR

I appreciate your time on this. Perhaps we could work together over here #7933. I don't think the dotnet tool issue if worth pursuing and the team already decided we were going to an MSBuild task solution as stated on discord.

You're welcome. It's always fun to contribute here!  😄 Ups, maybe I missed that discussion.

@xC0dex I looked back and its not clear I said that in the open forums.  Sorry.  In any case dotnet tool has been so painful that we will be very happy to see it go :-).

In fact running anything command line can be very unpredictable when executed from a larger multi-project solution


Registration success feedback

Regarding the registration feature for the prototype, there is no prompt, nor any response for the successful creation of the account, as such, we are sometimes confused if we have created the account at all.

Should be fixed by https://github.com/parnikkapore/quiknotes-fe/commit/4d345f3586f12547a26a81306782036536d5e2e4


Fix vulnerability of Docker image

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


Invalid kerberos patches, works now and with a test.

Need Apache license headers for all source files.
Need to add the new unit test class to GitblitSuite.

Done


Sleep mode

I now have the anemometer running according to the instructions. So far everything is ok. The only thing is that the Wemos D1 goes into sleep mode after a certain time and thus no longer sends any data to the mqtt broker. How did you solve that. The simplest would be who a time until wakup or that it is activated by moving the anemometer. Do you have a solution?

It should send at least every 10 seconds a message to the MQTT broker, this should be enough to prevent disconnecting from wlan or other things. A sleep mode is not in the origin code implemented. Maybe is this problem a power related issue with insufficient voltage after some time.


BUG: optimize: using integrality prevents linprog from accepting a list for c

Describe your issue.
If integrality is used in linprog, then c must be a numpy array.  Otherwise an exception is raised because c does not have a shape attribute.
The fix is probably as simple as passing c through np.asarray() first.
Reproducing Code Example
In [35]: from scipy.optimize import linprog

In [36]: linprog([1, 1], A_ub=[[-1, 0]], b_ub=[-2.5], integrality=[1, 1])
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In [36], line 1
----> 1 linprog([1, 1], A_ub=[[-1, 0]], b_ub=[-2.5], integrality=[1, 1])

File ~/repos/git/forks/scipy/build-install/lib/python3.10/site-packages/scipy/optimize/_linprog.py:623, in linprog(c, A_ub, b_ub, A_eq, b_eq, bounds, method, callback, options, x0, integrality)
    621     warn(warning_message, OptimizeWarning)
    622 elif np.any(integrality):
--> 623     integrality = np.broadcast_to(integrality, c.shape)
    625 lp = _LPProblem(c, A_ub, b_ub, A_eq, b_eq, bounds, x0, integrality)
    626 lp, solver_options = _parse_linprog(lp, options, meth)

AttributeError: 'list' object has no attribute 'shape'

Error message
See above.

SciPy/NumPy/Python version information
1.10.0.dev0+2184.5057eff 1.24.0.dev0+1048.g939910660 sys.version_info(major=3, minor=10, micro=8, releaselevel='final', serial=0)

PR welcome.

@nboudrie and I will be taking on this issue.

@jgborch @nboudrie Did you plan to submit a PR for this? SciPy 1.10 is branching soon, and this really should be fixed in time for that since it's as simple as changing c.shape in:
https://github.com/scipy/scipy/blob/3365ee6f479ab5a758ed60e4d9c5ce5ee14b1c7e/scipy/optimize/_linprog.py#L623
to np.shape(c).

Yes I will do that. Sorry about the delay, had to catch up on work and
school before the holiday.
On Fri, Nov 25, 2022 at 2:51 PM Matt Haberland @.***>
wrote:

@jgborch https://github.com/jgborch @nboudrie
https://github.com/nboudrie Did you plan to submit a PR for this? SciPy
1.10 is branching soon, and this really should be fixed in time for that
since it's as simple as changing c.shape in:
https://github.com/scipy/scipy/blob/3365ee6f479ab5a758ed60e4d9c5ce5ee14b1c7e/scipy/optimize/_linprog.py#L623
to np.shape(c).
—
Reply to this email directly, view it on GitHub
https://github.com/scipy/scipy/issues/17380#issuecomment-1327840242, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APFWB2FTFHDPHDFN3ODKK6DWKEKD7ANCNFSM6AAAAAAR3YGAQA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Noah Boudrie
College of Engineering | 2023
University of Michigan
@.*** |<PHONE_NUMBER>

This was fixed by https://github.com/scipy/scipy/pull/17513.


Kryo v5 support

Do you plan to support Kryo v5? Main readme says that you support only v2, v3, v4 versions. Is it very hard to implement v5 support?

Any progress with unmodifable collection support for Kryo5?

Kryo 5 can be used without problems. The Readme just doesn't reflect that.

Could you add an example on how to do that exactly?
We ended up reimplementing all the unmodifable collection serializers for Kryo 5.

@paplorinc: What kind of issue did you have? I'm using Kryo 5 and simply call:
	UnmodifiableCollectionsSerializer.registerSerializers(kryo);
	SynchronizedCollectionsSerializer.registerSerializers(kryo);

The latest version of this project already depends on an old Kryo 5 RC so this works fine for me.

@magro: Any chance you could bump the Kryo dependency to the latest release and update the readme?

We depend on both version of Kryo, so we're getting:

OK, that's bad. I guess this wasn't taken into account when the separate Kryo 5 artifact was created. This will be tough to fix.
Since unmodifable collections are part of the JDK, we could add built-in support for them to Kryo. Some of the serializers where already moved over to Kryo in the past. I don't know why these weren't included.

It seems that has been discussed a couple of times in the past:

https://github.com/EsotericSoftware/kryo/issues/151
https://github.com/EsotericSoftware/kryo/issues/604

Nate was against including any "unsafe" serializers, i.e. serializers depending on internal state of the objects not available through public APIs.

Are there any updates on this?

I hope this is done soon! thanks!


fluentPython

确认完毕，唯一申请人为哈尔滨工业大学。
在 2020-08-16 04:10:37，"Ovee Wahed"<EMAIL_ADDRESS>写道：

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


chore: social named export

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
The focus of this PR is to enhance the social exports in the thirdweb package by adding specific profiles and updating paths in the package.json.
Detailed summary

Added specific social profiles to social.ts
Updated paths for social exports in package.json

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#4466  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @joaquim-verges and the rest of your teammates on  Graphite


Re-enable pylint, lint and docs checks on TravisCI

Sadly Scrutinizer gets stuck too many times or fails with errors unrelated to our changes :/

With this, can you look into why coverage.io metrics stopped showing up? :P

@lakshmi-kannan later / separate PR


Fix is_aggregate property for Case fields

So Case's criterions were not taken into account when evaluation a Case field's is_aggregate property, which imo is incorrect.

Coverage remained the same at ?% when pulling e805d9e05c2217b29e6b565e1d0833901fde7800 on x8lucas8x:fix-is-aggregate-property-for-case-fields into cb28b93e9c1159e490c3559e8467c7cfebb0ec5a on kayak:master.


MonarchMixerLayer

Hello,
I've come across an algorithm in the paper that appears to be designed for the M2 layer, with the intention of replacing both the Attention and MLP layers (specifically the nn.Linear part of the latter).
However, upon examining the monarch_mixer_sequence_mixer.py script, I noticed that it uses Hyena filters, and I couldn't find any implementation of this M2 layer algorithm in the code.
I might be missing something, but I wanted to clarify if it's necessary to substitute the Hyena filters with the M2 layer.
Thank you for your assistance with this project!
P.S.: I'm currently working with image data.

Great question!
If you look at section 5.1, we use the Monarchs to implement long convolutions in conjunction with gating for a lot of the backbones. (also see this image from the blog).
I think with image data, you might actually be fine without the Hyena stuff - it's more important for language. In older experiments, we do see higher performance on ImageNet with the gating and the Hyena kernels. On CIFAR, the performance is about the same with/without the gating.


docs: clarify defer in counter app tutorial

The docs suggested App::run() is deferred, but that's not quite right. The result is returned after the terminal is restored so errors are properly displayed. Updated the wording to make this clearer.

Thanks for the PR :)


phototonic: 1.7.1 -> 2.1

Motivation for this change
update with bug fixes
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

cc maintainer @pSub

Success on x86_64-linux (full log)
Attempted: phototonic
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/fd47yahmiysm4b02xn31lk0i0iccwhpz-phototonic-2.1
shrinking /nix/store/fd47yahmiysm4b02xn31lk0i0iccwhpz-phototonic-2.1/bin/phototonic
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/fd47yahmiysm4b02xn31lk0i0iccwhpz-phototonic-2.1/bin
patching script interpreter paths in /nix/store/fd47yahmiysm4b02xn31lk0i0iccwhpz-phototonic-2.1
checking for references to /build in /nix/store/fd47yahmiysm4b02xn31lk0i0iccwhpz-phototonic-2.1...
postPatchMkspecs
postPatchMkspecs
/nix/store/fd47yahmiysm4b02xn31lk0i0iccwhpz-phototonic-2.1


[BUG] server hang waiting for http request response

Mage version
0.9.63
Describe the bug
In local enviroment without network connetion.
use pip installed version.
when start the server, it's stucked at "http request sent; waiting for response"
To reproduce

mage init xxx
mage start xxx

Expected behavior
No response
Screenshots
Operating system

Centos

Additional context
No response

After ^c, it stops here
Process Process-2:
Traceback (most recent call last):
File "/anaconda/anaconda3-2022.05/lib/python3.9/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/anaconda/anaconda3-2022.05/lib/python3.9/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "site-packages/mage_ai/orchestration/db/process.py", line 15, in start_session_and_run
results = target(*args)
File "/site-packages/mage_ai/server/scheduler_manager.py", line 50, in run_scheduler
LoopTimeTrigger().start()
File "/site-packages/mage_ai/orchestration/triggers/loop_time_trigger.py", line 18, in start
time.sleep(sleep_time)
KeyboardInterrupt

The Mage server might not work without internet connection now. We'll fix that.

hey @woday-dev, is this a fresh install of mage? do you have any environment variables set or any other customizations? i'm able to get a Mage server running with no internet connection

Another user got this error when running Mage without internet in docker:

hey @woday-dev, is this a fresh install of mage? do you have any environment variables set or any other customizations? i'm able to get a Mage server running with no internet connection

Hi @dy46 , yes it's a fresh install of mage via pip, virtualenv is used and no special setting by intention. Great to know it can work under no internet connetion, maybe you have internet connection the first several startup?


Update custom claims with refresh token

Im having the same issue as in this issue over in the old repo: https://github.com/tymondesigns/jwt-auth/issues/891
The short of it is, I have a custom claim in my user model:
 public function getJWTCustomClaims()
    {
        return [
            'email_verified_at' => $this->email_verified_at,
        ];
    }

However the getJWTCustomClaims() function is never called when a token is refreshed.
Associated PR from old repo: https://github.com/tymondesigns/jwt-auth/pull/1619
I have tried getting it running quickly by using the above PR as a reference but I am missing the authenticate() function in this repo so its not a quick fix unfortunately.

Provide your model code, please.
Thanks.

How can we add custom claims with for refresh token
Earlier
$newToken = JWTAuth::fromUser($user, [
                           'exp' => Carbon::now()->addMinutes(config('jwt.ttl'))->timestamp,
                           'remember' => 0
                    ]);

We need
something like
auth()->claims( [
                         'exp' => Carbon::now()->addMinutes(config('jwt.ttl'))->timestamp,
                         'remember' => 0
                  ])->refresh()


Pip error during CellProfiler Install

On Popos, the conda env create -f nf1_cellpainting_env.yml produces an error during the pip installation of the CellProfiler as shown in the attached screenshot:

It looks like the solution to their error was making sure that this command was run on the computer before creating the environment:
sudo apt-get install libmysqlclient-dev

Having this installed will fix this issue with the mysqlclient error.
I am not sure if this issue happens on Mac as it looks like it worked on my end, but if this issue pops up again on Mac, then this issue can be reopened.


Issue 49 regression fix

Not sure how the tests were passing, but the kwarg for this issue was referencing the same name as another option. I fixed the lookup

Thanks. We should look at the test a little closer now too.

Coverage remained the same at 97.462% when pulling 3df80229a6079f2106facf1d48cb03887d90eb4a on ckirby:issue-49 into 4aa2cb24cfa5aaf2cc187e1ad8eecbcccb5dccaa on california-civic-data-coalition:master.

I actually did, but since non of the changes I put in caused an error I
didn't push a test change through.
I'm getting on a plane, but will take a deeper look tonight or tomorrow
On Oct 17, 2017 1:14 PM, "Ben Welsh"<EMAIL_ADDRESS>wrote:

Thanks. We should look at the test a little closer now too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/california-civic-data-coalition/django-postgres-copy/pull/63#issuecomment-337290049,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFeHv0QW4HOJJvP6UkQv8iyLwAzKtd3ks5stOBzgaJpZM4P8dVf
.


no such file or directory:'./models/config.pickle'

I ran into this problem, after vocabulary biult, dataset built, cnn built, rnn built, the problem occurs,
no such file or directory:'./models/config.pickle'

same issue

same issue

any solution?

Cheng Fu<EMAIL_ADDRESS>周六下午9:58写道：

same issue
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DeepRNN/image_captioning/issues/20#issuecomment-399723021,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjZjYgpRJXALUCLPYVeeWKZh28G5OdNhks5t_vJNgaJpZM4THrn5
.

Try this:
'.//models/config.pickle' instead of './models/config.pickle'

Jockwang<EMAIL_ADDRESS>周五上午1:50写道：

same issue
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DeepRNN/image_captioning/issues/20#issuecomment-423421054,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjZjYsV_NMlx5-KL4WHGaDNutP88b2vFks5udH4fgaJpZM4THrn5
.

Try this:
'.//models/config.pickle' instead of './models/config.pickle'

jjneojiajun<EMAIL_ADDRESS>周二上午1:59写道：

any solution?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DeepRNN/image_captioning/issues/20#issuecomment-424216958,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjZjYgfTNpcsK-GVV3LXi3WNnqzl8pYlks5uecZJgaJpZM4THrn5
.

Try this:
'.//models/config.pickle' instead of './models/config.pickle'

same issue


[Feature] DMSQL client plugin

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
support  DMSQL  database.
Use case
enhance client to trace.
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

First of all, you should make sure the title is correct and complete when you submit anything to a community.
DMSQL is a commercial product of IBM. It is not licensed for us to do so.

Sorry, DMSQL is a domestic database，Damon database management system is a high-performance database management system with completely independent intellectual property rights launched by Damon, which is called DM for short，In recent years, domestic databases have been used more and more widely, which is a trend

It is also a commercial product.
Nothing different from license perspective.
There is nothing about a trend in this discussion.

Okay! Thank you for your communication which has benefited me a lot.


Bug: Tab dropped into wrong position when moved between windows

Description
When dragging a tab within the same window, a placeholder for the target position appears in the tab bar. When the tab is dropped, it appears in the placeholder position. When dragging a tab across windows, the placeholder still appears, but the tab always appears at the far left of the tab bar.
Steps To Reproduce
.
https://github.com/files-community/Files/assets/68013339/c5deb38c-0fb5-4ab7-9976-88dc4720a874
Requirements
The tab should appear in the position indicated by the placeholder.
Files Version
<IP_ADDRESS> Preview
Windows Version
10.0
Log File
.

Thanks for the report

A fix will be included in the next update


allow php 7.2

Coverage remained the same at 100.0% when pulling df67f4da70c9ca37c1d7b65c384336dc33b38279 on php_72 into 2c76f87166788ea495796b6adf6ccc8381a04994 on master.

Coverage remained the same at 100.0% when pulling df67f4da70c9ca37c1d7b65c384336dc33b38279 on php_72 into 2c76f87166788ea495796b6adf6ccc8381a04994 on master.

Should we tag 0.2 or are we good enough with this for 1.0?
On Nov 27, 2017 18:42, "Alexander Miertsch"<EMAIL_ADDRESS>wrote:

Merged #1 https://github.com/prooph/php-cs-fixer-config/pull/1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prooph/php-cs-fixer-config/pull/1#event-1358961006,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYEvDWsXwmwrGxC-F8rEtJ8S6WRY1wfks5s6pHegaJpZM4QqcPo
.


Bash completion improvements

[ ] Add support for HTTP methods
[ ] Add support for URLs: perhaps complete known hosts like ping or ssh and known URLs from various sources
[ ] Add support for choice values (e.g. -s <tab>, --style=<tab>, --auth-type<tab>, etc.)
[ ] Update the Homebrew formula to install the completion support

// #330 #326

I'll look into this over next few days

This looks quite interesting: https://github.com/kislyuk/argcomplete
So does the way Django does it for manage.py.

Yes. I was looking into argcomplete previously. Couple of issues I found were that requirement of a new package argcomplete. Also, it requires additional setup like activation of global python autocomplete. Wasn't entirely sure whether to go that way. Django way looks more interesting.
I understand that writing a different bash script for autocompletion will add an overhead in case options change, so better to include it in python.
I'll see which one to dig deep into.


Add retries for 50x errors on Terra data table API calls

┆Issue is synchronized with this Jira Story

This should use requests.Session objects to manage retries. A side effect is Session objects use connection pooling, which may improve performance.


Create command to unpackage .pbiviz file

We have to run the javascript in a custom visualization through a security audit before it can be used by employees.
Can a command be added to unpackage the .pbiviz file into it's original folder / file structure?

we'll consider this for our next semester.


Get current file hierarchy (outline)?

On VSCode API, I believe there is no way to get the file structure hierarchy (the outline), correct?
For example, let's say I have a markdown file opened, I want to get the structure, token, parsed result in any form of the document, so I can pass the data to my extension program (and render on the Treeview for example).
I googled some and found a issue on Github such as this, or SO posts such as this, according to my research I could use something like vscode.DocumentSymbol.children, but this code doesn't show an output. (I could expect this since I just tried to run to see if it works by chance)
So is it possible to implement this feature for the API? Thanks.

Maybe vscode.executeDocumentSymbolProvider (see https://code.visualstudio.com/api/references/commands#commands)

The API commands allow you access the model that VS Code UX uses, but the syntax tree of the language in question. VS Code doesn't help with detailed language/syntax information for usage outside of the UC. If you need that you have to see if the corresponding extension exposes an API or use your own parser


Add soversion to libraries

Previously the libs were just "libtensorflow.so" without any version.
This adds the version to the library and adds the appropriate symlinks
to the full name from the base and soversion.
Signed-off-by: Jason Zaman<EMAIL_ADDRESS>DO NOT MERGE THIS YET. I'm opening this to get a full test run. It works to build on my machine so far.

Nagging Reviewer @asimshankar: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

@perfinion : Thanks for the PR, but could you elaborate a bit on this? When building from source we'll always only have a single shared library, not a versioned one. So perhaps what you want is for this versioning to be done in the binary distributions rather than by changes to the BUILD rules?
Or am I missing something?

@asimshankar Sorry, the background is at https://groups.google.com/a/tensorflow.org/forum/#!topic/build/0FjOGrinf5k I should have linked that in the beginning.
basically the version needs to be put in the .so file when building. right now the soname is just "libtensorflow.so" so it cant be versioned later. the soname needs to be set and also requires the symlinks so the other bits of the build work.

Nagging Assignee @rmlarsen: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@asimshankar, @perfinion mentioned the other day that this naming setup was working for everything but Java. Could you take another look at it? (@perfinion, can you describe what you're doing again, like you did in the SIG Build meeting?)

A summary of the reasons for this change:
Distros are quite strict about versioning the libraries properly so that binary compatibility is easy. Whenever a library makes a breaking change the verison in the library is supposed to change accordingly, then other binaries will link to those specific versions. Lets take nsync as a specific example.
$ ls -al /usr/lib64/libnsync*
lrwxrwxrwx. 1 root root    13 Sep 21 01:58 /usr/lib64/libnsync.so -> libnsync.so.1*
lrwxrwxrwx. 1 root root    18 Sep 21 01:58 /usr/lib64/libnsync.so.1 -> libnsync.so.1.20.1*
-rwxr-xr-x. 1 root root 42856 Sep 21 01:58 /usr/lib64/libnsync.so.1.20.1*
lrwxrwxrwx. 1 root root    17 Sep 21 01:58 /usr/lib64/libnsync_cpp.so -> libnsync_cpp.so.1*
lrwxrwxrwx. 1 root root    22 Sep 21 01:58 /usr/lib64/libnsync_cpp.so.1 -> libnsync_cpp.so.1.20.1*
-rwxr-xr-x. 1 root root 51144 Sep 21 01:58 /usr/lib64/libnsync_cpp.so.1.20.1*
$ scanelf -ByF '%S %p' /usr/lib64/libnsync*1.20.1
libnsync.so.1 /usr/lib64/libnsync.so.1.20.1
libnsync_cpp.so.1 /usr/lib64/libnsync_cpp.so.1.20.1

when building a program that uses nsync, you'd build with gcc foo.c -lnsync then the compiler would look for libnsync.so, follow the symlink and link with that, then it will take the soname that is embedded in the library and put that in the NEEDED. So now when nsync releases a 2.0.0 version the lib will update the version and libnsync.so.2.0.0 will exist and the symlinks will point to that instead. libnsync.so.1 will still link to libnsync.so.1.20.1. This means both libnsync.so.1 and libnsync.so.2 can be installed at the same time and any existing binaries that link with the old one will keep working just fine and new compiled binaries will use the new one. More info from debian is here: https://www.debian.org/doc/debian-policy/ch-sharedlibs.html but the idea is exactly the same on all distros.
The linking is like this, notice the .1 that its using.
$ ldd /usr/lib64/libtensorflow_framework.so
[ ... snip ... ]
	libnsync_cpp.so.1 => /usr/lib64/libnsync_cpp.so.1 (0x00007f7e751e2000)

Bazel does not really understand these versioning things so I've used a bunch of genrules to make the symlinks and they work well for all the C++ and go stuff. java is the bit that isn't linking quite right to them. or rather, it is linking to them properly, but later fails because it needs the lib itself and the symlink but only has one.

Hi there, I looked at this a couple of days ago and couldn't reproduce the failures. I'll take another shot at it again today.

@sjamesr hey, thanks for looking into this! yeah, I rebased and tried recently too and the java tests pass on my machine. I'm not sure what changed to make it work now tho. The tests still appear to be failing on some of the builders (but not all, so it clearly sometimes works?). I've been playing around with building with the remote execution in a container to see if that'll help find the error but not gotten that going yet. Do let me know if you find how to repro the issues :)

I managed to get the java tests to fail using RBE. The targets all build properly, just the tests fail at runtime. Hopefully i'll figure out how to make them run now.

@angersson @sjamesr It should pass java and go tests now. RBE is a lot stricter so I could track things down. Basically added tf_binary_additional_srcs() to data= and a few places like that so It was properly in the deps. The symlink is not a proper library according to bazel so it can't be put in srcs= or deps= like normal but data= followed the same pattern as several other libs.
I'm seeing a failure of the tests in //tensorflow/lite/testing/... but those appear turned off in at least some of the CI bots so not sure if they are known bad?

The mac builds fail weirdly, I'm not sure why. The linker commandline contains -l:libtensorflow_framework.so but it should be -ltensorflow_framework. It's probably looking for a file named lib:libtensorflow_framework.so.1? I'm not sure what rule is adding that tho. I tried a few updates
I tried adding tf_binary_additional_srcs() in a few more places but its not making a difference?
Once this is figured out, I'll re-test and remove the random hacks before its merged.
The error is for mac is:
ld: library not found for -l:libtensorflow_framework.so.1
clang: error: linker command failed with exit code 1 (use -v to see invocation)```

@klimek Can you take a look at this when you have a chance? Linux builds pass on my machine and remotely. one of the windows builders passes too but MacOS is failing hard. I tried on a mac locally and changing tf_binary_additional_srcs() to ["//tensorflow:libtensorflow_framework.so"] (ie without the .1 suffix) made it build but it still fails on the CI. (look at the previous force-pushed change for the diff)
We really have to fix the library versioning, lots of distros dont like un-versioned libs and I definitely want this in soon because TF2.0 is starting to take shape.
@meteorcloudy you made some changes to tf_cc_shared_object recently, can you take a look too?

@r4nt

@perfinion Looks good, but I'm not quite sure how versioned dynamic library work on macOS.

@meteorcloudy yeah, I'd found that page, OSX works basically the same way as linux just with different filename order, the version before the .dylib. It would be possible to disable the versioning on OSX but I dont like that idea unless we really really have to.
I could repro the -l:libtensorflow_framework.so.1 breakage on my mac. It probably has something to do with libtensorflow.so.1 being a symlink? if i change things to just .so without the .1 then I can get it to build on my mac but the CI bots go crazy.

Tests finally all pass. Looks like some of the GPU builders are failing but I cant see the details.
To fix the macos builds I had to turn on the per_os libs for libtensorflow_framework too. -l:libtensorflow_framework.so.1 is not allowed on mac, it only allows -l:libtensorflow_framework.1.dylib.
@r4nt @meteorcloudy @gunan @angersson Can you review it again and see if its ready to merge?

I made a tf_shared_library_deps() to make the go change cleaner. It returns the os-specific name for libtensorflow.so and all the version symlinks. looks like the tests are okay so should be all good to go.


Get session data from users

When you use the facebook conncector or any other webchat with Rasa Core,
how are different users handled?  What if two users use the chatbot at same time. Does Rasa handles those two seperately? I think hast something to do with this sender_id?
But how do I use Rasa core to extract for every user (distinguished by sender_id?) the messages, intents, etc?
I want to use my Rasa chatbot via facebook and want to extract this data for testing.
Thank you!

if you take a look at the documentation,
GET /conversations
List the sender ids of all the running conversations.
this basically looks into the tracker for all the sender ids, then for each of them you can get tracker with
GET /conversations/(str: sender_id)/tracker
Retrieves the current tracker state for the conversation with sender_id. This includes the set slots as well as the latest message and all previous events.
depends on what you are using as a tracker( is it Redis? ) or InMemory? You can create a trigger for redis to pull data out of the tracker if there is an event and save it to your persistent storage.
You can also add your own tracker store ( RabbitMQ or even Kafka- though one might argue)
p.s sender id for Facebook is the messenger-id. I think It is linked to the user session in Facebook(not sure)

Thanks! I get a feeling. But I don't know how to use this (within python or  java script?). How do I use Redis? What exactly is it? There is nothing in the docs about that. Right now I have my bot in python on my local machine. Do I have to do something in python to use those Redis trackers?
Sorry for those naive questions...
But those endpoint calls for the tracker state are nice. That is what I want.

Redis is a Message Queque where the tracker is stored temporarily for the period of the conversation( you can even configure it). this allows the tracker to be persistent for a user.
You have a library in python called redis, you can check how Rasa did it.
You can write a script to extract data from the redis queue or use the API.
If you are not too familiar with python, use the API. extract data on a a periodic basis. Redis also has API for other languages as well. Just connect to the Redis server  and look at the API, how to retrieve the objects.

Thanks! I am rather familiar with python. So, this way is ok. Can you show me where Rasa uses Redis?

A snippet from tracker_store.py
class RedisTrackerStore(TrackerStore):

    def keys(self):
        pass

    def __init__(self, domain, host='localhost',
                 port=6379, db=0, password=None):

        import redis
        self.red = redis.StrictRedis(host=host, port=port, db=db,
                                     password=password)
        super(RedisTrackerStore, self).__init__(domain)

    def save(self, tracker, timeout=None):
        serialised_tracker = self.serialise_tracker(tracker)
        self.red.set(tracker.sender_id, serialised_tracker, ex=timeout)

    def retrieve(self, sender_id):
        stored = self.red.get(sender_id)
        if stored is not None:
            return self.deserialise_tracker(sender_id, stored)
        else:
            return None

The way i understand this is RedisTrackerStore is available by default within RASA. However, we have to implement the Tracker Endpoint? Is that right? Can some one clarify for me please?

Well there is a tracker endpoint that can be called via Http as mentioned above if you are running rasa core as a server based on the implementation provided by Rasa already.
You have a choice on where you want to store your tracker, could a Redis or InMemory(by default). So when you load your Agent, you have to provide the tracker_store if it is not InMemory. So all you need to do when you load your agent in server.py , you have to provide the tracjer
Rasa has implementation of Redis and InMemory. You can add your own tracker store as well(implemetation could be in the tracker_store.py( Could be Mongo, Couchbase) keep in mind Rasa is serializing the tracker object ( basically creating pickle file in python and storing it in redis)
There is already a tracker API over http that allows you to read the tracker object at any time and even update it with your custom information asynchronously of the ongoing chat for a user. Really powerful if you want to trigger reminders.
Hope this helps

@souvikg10 What is actually the sense of server.py when you you use an GUI interface like facebook or any other interface. I don'tnt see that any connector (slack, fb) uses the endpoints from the server.py for getting the parsed message?! I.e where does
https://github.com/RasaHQ/rasa_core/blob/master/rasa_core/channels/facebook.py
requests the parsed message endpoint?
My custom GUI also doesn't use server.py. So, what do I need to do when I want to Redis Tracker store?

When you are using a connector pre defined, you are using run.py where you are loading the agent with a particular model, it is there you should pass the tracker_store
I am not sure how you are using your custom GUI( did you use your own Flask server to run up?) Our GUI is not directly connected to Rasa UI as we have an intermediate gateway with Botkit which runs on node js, we use Rasa for model inference in this case.

I use:
https://github.com/scalableminds/chatroom/blob/master/rasa_utils/bot_server_channel.py
from bot_server_channel import BotServerInputChannel

# Creating the Interpreter and Agent
def load_agent(): ...

# Creating the server
def main_server():
    agent = load_agent()
    channel = BotServerInputChannel(agent)
    agent.handle_channel(channel)

main_server()

and it uses flask.
So, it uses InMemory in line 119?
 tracker = self.agent.tracker_store.get_or_create_tracker(cid)
What I have to change for Redis?

@souvikg10 So, it seems they don't use the HTTP API for the NLU? I guess I would extract Intents and ENtities like running locally?

https://github.com/scalableminds/chatroom/blob/master/rasa_utils/bot.py
In this code, they are loading the agent with the models that are stored locally.
agent = Agent.load(cmdline_args.core, cmdline_args.nlu) (line 59)
You can pass the tracker store there
Agent.load(model_directory, interpreter,
tracker_store=tracker_store,
)

@ctrado18 indeed, it is upto your how you want to use the interpreter. We have separated it since it seems Rasa NLU can also be used without necessarily needing the core and vice versa ( case - When you want rasa NLU to extract entities from tweets let's say or you have a click bot that is simply based on core without needing the NLU)

So, I guess I just use like  tracker.latest_message.parse_data["intent"]["name"] in bot_server_channel.py  to extract the data?

https://github.com/scalableminds/chatroom/blob/master/rasa_utils/bot.py
In this code, they are loading the agent with the models that are stored locally.
agent = Agent.load(cmdline_args.core, cmdline_args.nlu) (line 59)
You can pass the tracker store there
Agent.load(model_directory, interpreter,
tracker_store=tracker_store,
)

Hi @souvikg10 , I encountered a similar issue. But chatroom updated their code. I cannot find the exactly line as you mentioned. I want to use InMemory tracker though. Thanks.