[5.5] Allow HtmlString as line in notifications MailMessage

Allows for
public function toMail($notifiable)
{
    $line = new HtmlString('Hello world');

    return (new MailMessage)->with($line);
}

Perhaps allow the Htmlable interface instead of the HtmlString implementation?

@sisve sure. fixed, thanks

Should have changed the title to reflect the change, now the change list in the tag is wrong.
https://github.com/laravel/framework/releases/tag/v5.5.27


[Azure.Core] Test Environment Variables

Summary
The focus of these changes is to include additional environment variables in the TestEnvironment to allow for cross-cloud management plane operations,
such as those used by the Azure Messaging client libraries.  This is the first step in enabling cross-cloud support for Live testing in the Event Hubs and Service Bus client libraries, both T1 and T2.
The test resource creation script has been updated to populate these additional variables using the current Azure environment context.   Orthogonal, but included are some tweaks to allow local runs of the TestResources.ps1 script using the -WhatIf flag, which was failing for me before changes were made.
Last Upstream Rebase
Monday, July 27, 4:48pm (EDT)
References and Related Issues

New-TestResources.ps1 should set AZURE_AUTHORITY_HOST environment variable (#757)
Event Hubs: AAD Auth failure when testing against other clouds (#12964)
Event Hub Live Test Change (#13217)

Great change, make sure you change the shared scripts in https://github.com/Azure/azure-sdk-tools/tree/master/eng/common/TestResources first and let them sync.

Thanks!  I wasn't aware of the mirroring.


Update ODP roadmap image

Description
Update ODP roadmap image (in FINOS wiki and website) to reflect the current status of the ODP project.
See: https://www.linkedin.com/posts/
finosfoundation_opendevplatform-compliance-findeliveryaccelerator-activity-6608018939970666497-_8My

Done, https://odp.finos.org/docs/roadmap


#2875 Messages V2: Fix smart filters

[ ] Breaking change? (if so, please describe the impact and migration path for existing application instances)

What changes did you make? (Give an overview)
Fixes #2875
Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[ ] No need to
[x] Manually (please, describe, if necessary)
[x] Unit checks
[ ] Integration checks
[ ] Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged

Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)

@provectus/kafka-qa to be tested within 0.6

@provectus/kafka-qa to be tested within 0.6

will be tested as a whole branch with v2 msgs


load networks.config dynamically

loads the network config dynamically at app start instead of compiling the file statically into the package

@begonaalvarezd Jorge und Lukas told me that the PR has two flaws:

it doesn't work in dev mode. The networks.json is not served but instead it is tried to access it from the local filesystem
the synchronizing with the browser's local store doesn't work.

Do you think it's fixable? :pleading_face: :pray:


Update bits/architecture and better error handling

@pearu ah no, the branch changed but not the config of the CI. Shall I update this here?

(I will fix the notebook with ibis in another PR)


Filtering with multiple format placeholders

I would like to do a LDAP filtering like this:
LDAP_USER_OBJECT_FILTER = '(&(objectclass=Person)(|(mail={0})(sAMAccountName={0})))'

So that a user is able to authenticate using either an email address or a username, and the LDAP-search will look for one of both matches.
But it seems that this is not possible, I get the error:
TypeError: not all arguments converted during string formatting

This might be due to the changes made following issue https://github.com/admiralobvious/flask-simpleldap/issues/4
Is there a way of achieving the desired behaviour that I'm not aware of?

@DavidUnzue Did you ever find a solution or workaround for this issue?

@admiralobvious I ended up not using a flask extension like this one but the python 3 fork of pyldap (https://github.com/pyldap/pyldap).


Damaged item variants

I believe that it was already possible to use damaged item variants as wands if you change the wooden hoe's in the resourcepacks but can I also use this as iron or diamond hoe's. If I try this I get only a purple block.

That sounds like an issue with your resource pack. You can use whatever item variants you want, yes.
You generally have to make the wand indestructible (otherwise it may change appearance if damaged, which could happen via vanilla behavior) but otherwise there's no special trick to it.
Purple+black block means your RP is broken in some way, or you've put an invalid damage value on a tool.


Comments submitted cannot be displayed on my website

Issue Discription:
I use ISSO with Apache2 and Pelican.The ISSO can be displayed on the website normally, but when I submit a comment, nothing happens.

For ISSO server using isso -c isso.cfg run, and when I submit a comment, the ISSO server will receive the messages as below picture:

It looks like it is not written to the sqlite database.

Apache2 reverse proxy config:
<VirtualHost *:80>
	ServerAdmin webmaster@localhost
	ServerName isso.jerrylsu.net

        DocumentRoot /var/www/blog/isso

        ProxyPreserveHost On
        ProxyRequests Off
        ProxyPass / http://localhost:8280/
        ProxyPassReverse / http://localhost:8280/
        <Proxy *>
                      Order Deny,Allow
                      Allow from all
         </Proxy>
         <Directory /var/www/blog/isso>
              Options -Indexes +FollowSymlinks
          </Directory>

	ErrorLog ${APACHE_LOG_DIR}/error.log
	CustomLog ${APACHE_LOG_DIR}/access.log combined
</VirtualHost>

/etc/hosts config:
<IP_ADDRESS>	localhost
<IP_ADDRESS>	http://jerrylsu.net
<IP_ADDRESS>	http://isso.jerrylsu.net
::1	localhost ip6-localhost ip6-loopback
fe00::0	ip6-localnet
ff00::0	ip6-mcastprefix
ff02::1	ip6-allnodes
ff02::2	ip6-allrouters
<IP_ADDRESS>	e7cac75317f0

ISSO server config:

[general]
dbpath = /var/www/blog/isso/comments.db
name = jerry
host = http://jerrylsu.net
notify = stdout
log-file = /var/www/blog/isso/isso.log

[server]
listen = http://localhost:8280
public-endpoint = http://isso.jerrylsu.net/

[moderation]
enabled = true
purge-after = 30d

[smtp]

[markup]
options = strikethrough, superscript, autolink

[guard]
enabled = true
ratelimit = 3
direct-reply = 3
reply-to-self = true
require-author = true
require-email = true

[admin]
enabled = true
password = custom define

[hash]
salt = Eech7co8Ohloopo9Ol6baimi
algorithm = pbkdf2

ISSO with Pelican blog

SITEURL = 'http://www.jerrylsu.net'

ISSO_SITEURL = "http://isso.jerrylsu.net"

base.html

...
{% include 'include/isso_script.html' %}
...

iss_script.html

{% if ISSO_SITEURL %}

{% endif %}

article.html

...
{% include 'include/comments.html' %}
...

comments.html

{% if SITEURL and article.status != "draft" %}

Comments

  {% if ISSO_SITEURL %}

  {% endif %}
  Please enable JavaScript to view the comments.

{% endif %}

Thanks for any support!!!

Seems like it is failing at CORS preflight (OPTIONS sent but no response).
Can you post the entire contents of the browser console? (available by pressing Ctrl-Shift-K)
Also, you have non-www in your isso host config, but your pelican blog SITEURL is with www, is that desired?

Hi @ix5 , this is indeed a problem caused by cross-domain resource sharing(CORS).

I've solved this problem by setting host as you mentioned.
host =
    http://jerrylsu.net
    http://www.jerrylsu.net

Thank you very much!
At the same time thanks to this issue: CORS errors I also got inspiration.


[qa] Add run-qa-checks, upgrade to openwisp-utils 0.5.1, black formatter

Add run-qa-checks, upgrade to openwisp-utils 0.5.1 and reformat code with black.

I'd like to help with this. I am new to your project.

welcome to OpenWISP @morrme . Sure, you can work on it. if you haven't yet seen our docs, you read more about the project from http://openwisp.io/docs/

Go for it @morrme, see how it's done in another module to replicate it here, eg:

https://github.com/openwisp/django-netjsonconfig/blob/master/run-qa-checks & https://github.com/openwisp/django-netjsonconfig/blob/master/.travis.yml#L28

Thank you. I am not familiar with your run-qa-checks. Are you seeking to add the functionality here
https://github.com/openwisp/openwisp-utils/blob/master/run-qa-checks
to your project?


Payment id not checked on secret request receive

Description
In the specs it says it be check if there is a payment with the same hash time lock and payment identifier, but only the hash time lock is checked in the sdk (src/transfers/epics/secret.ts:67. Update the specs, if this check is not necessary or check it.
Copied from the team repo
Acceptance criteria

Tasks

[ ]

It is checked, 9 lines below:
https://github.com/raiden-network/light-client/blob/60903231bc2bcebc366b8431061bb20610ca5579/raiden-ts/src/transfers/epics/secret.ts#L67-L80


Add jacoco for code coverage.

Description

Add jacoco plugin for code coverage.
Fix one ArgLine conflict with surefire.

Signed-off-by: Pan Li<EMAIL_ADDRESS>Related PRs
List related PRs against other branches:

branch
PR

other_pr_production
link

other_pr_master
link

Todos

[ ] Tests
[ ] Documentation

Steps to Test
Steps to test code change

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ff6ed43). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #42   +/-   ##
=========================================
  Coverage          ?   40.93%
  Complexity        ?       90
=========================================
  Files             ?       36
  Lines             ?      535
  Branches          ?       31
=========================================
  Hits              ?      219
  Misses            ?      297
  Partials          ?       19

Continue to review full report at Codecov.

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


IDVA5-1482 New links to clear session when verifying another client

Short description outlining key changes/additions

Added new section to confirmation view with 2 links as per prototype: verify another client and authorised agent account.
New controller with GET method to clear the USER_DATA and CHECK_YOUR_ANSWERS_FLAG session variables to allow user to go through verify journey again.
Redirecting user to correct url.
Unit test class for new controller.

JIRA Ticket Number
IDVA5-1482
Type of change

[ ] Bug fix
[X]  New feature
[ ]  Breaking change
[ ]  Infrastructure change

Pull request checklist

[X]  I have added unit tests for new code that I have added
[ ]  I have added/updated functional tests where appropriate
[X]  The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: Security warnings found!

CI: Security warnings found!

CI: Security warnings found!

CI: Security warnings found!


http-proxy-middleware

With latest version of the library, the import method has changed.  The current doc still explains the v.0.x implementation whereas the latest version is v1.x.
For reference: https://www.npmjs.com/package/http-proxy-middleware

Updated the http-proxy-middle implementation example to the current version of the library.

Closes #8661

Fixed in #8515
@ianschmitz


docs(icons): migrate icons gallery(VIV-1120)

Make vivid icons gallery (https://icons.vivid.vonage.com/) use vivid-3 components by migrating it to vivid-3 docs.
Waiting for https://github.com/Vonage/vivid-3/pull/1169 to be merged

@RichardHelm The empty states are not aligned inside the layout. Maybe it should not be set to default of height=100% ?

@olaf-k Is this more clear now?

@olaf-k Is this more clear now?

it was clear already, my point is that these filters are mutually exclusive, i.e., selecting solid and linear at the same time is allowed although it doesn't make sense. I'm not sure if there are overlaps between the others, too lazy to check, but radios might be more suitable here.

@RichardHelm The empty states are not aligned inside the layout. Maybe it should not be set to default of height=100% ?

Hi @rinaok, firstly, it looks great!
I think you can get them to align with align-self: start; on them.
But I feel like it's a bit of the an abuse of the empty-state to begin with to use it for data display, so maybe you're better served with some custom CSS?
Please note as well that I've updated the empty-state PR to remove the default 48px padding now.

I think you can get them to align with align-self: start; on them.

this will not work.
The align-center is inside the shadow-dom

@olaf-k Is this more clear now?

it was clear already, my point is that these filters are mutually exclusive, i.e., selecting solid and linear at the same time is allowed although it doesn't make sense. I'm not sure if there are overlaps between the others, too lazy to check, but radios might be more suitable here.

@olaf-k Changed it to radio-group

I think you can get them to align with align-self: start; on them.

this will not work. The align-center is inside the shadow-dom

@rachelbt It worked, it looks good now :)

@olaf-k Changed it to radio-group

did you?

DID YOU?

@olaf-k Changed it to radio-group

did you?  DID YOU?

Wasn't my decision 💁🏻♀ But I also think that it looks better with tags


v4: Flex media redux

This PR overhauls the .media component by reducing it to only two required classes, .media and .media-body. Everything else can be accomplished with utility classes for padding, margin, and alignment. This also overhauls the docs to improve copy, add more examples, and improve existing examples with the new markup and utils usage guidelines.
While working on this, I took a look at #20408. Turns out that's a known bug with inline flex items not being counted in IE10-11. I've added a docs warning for that.
As an aside, I'll likely be tackling some flex utils name changes and docs updates in a follow-up PR. We're missing some key flex utils and the class names could use a small change for consistency with our other property-value utils (e.g., d-flex).

Reminder to remove unused media variables.


[9.x] New 'valueKey' method to return only model key value from builder

It's nice being able to do Model::value('id') but what would be even nicer would be to not have to specify the key name at all: Model::valueKey()

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


handling certain categorical features with many distinct values

Thanks for the review @joepvdbogaert let's merge in tomorrow's meeting together

Oops, already merged. Nothing special here though, I guess?

I would prefer to squash before because now we also got two useless messages in the history: https://github.com/MLblog/jads_kaggle/commits/master
but its too much work to fix so i won't let my OCD get in the way of things :P

Ah okay, sorry, I should have checked the commit history I guess. It's useful for me to have this PR merged before I create mine though.

Oh then it's fine you did well!


Dependency using SharedWorker can't fetch it from Dev Server

Describe the bug
I am using a package that creates its own shared worker. I expect that when running in dev mode that the worker script is added to .vite/deps so the package can fetch it.
What happens is the worker script gets bundled into one package js file, making it inaccessible on the server.
Existing Workaround: Copy the worker script into the root of the project. Add the following to your vite config:
optimizeDeps: {
  include: ["sharedWorker"]  //name of your worker script file
},

I tried to reference the worker script as it exists within node modules, but I couldn't get it to retain its specific file name, which the creator of the shared worker needs to fetch it from the server.
The repro is a toy example, I encountered this issue on a project that needs auth to repro.
Related issue: https://github.com/vitejs/vite/issues/15359
Reproduction
https://github.com/andrewDoing/vite-shared-worker-issue-repro
Steps to reproduce
Run npm install
Run npm run dev
Look at the network tab, shared-timer-worker request never finishes because the file isn't in .vite/deps
System Info
System:
    OS: Windows 11 10.0.22631
    CPU: (8) x64 Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz
    Memory: 1.13 GB / 15.60 GB
  Binaries:
    Node: 18.18.2 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.21 - C:\Program Files\nodejs\yarn.CMD
    npm: 8.19.4 - ~\AppData\Roaming\npm\npm.CMD
    pnpm: 8.11.0 - C:\Program Files\nodejs\pnpm.CMD
  Browsers:
    Edge: Chromium (122.0.2365.92)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    @vitejs/plugin-react-swc: ^3.5.0 => 3.6.0
    vite: ^5.1.6 => 5.1.6

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Your package includes the following code.
new SharedWorker('shared-timer-worker.js');

When a relative URL is passed to SharedWorker constructor, it will be resolved to an absolute URL using document.baseURI as a base URL. document.baseURI can be changed by <base> and is not known at build time, thus Vite cannot handle this code.  The package should either (1) allow the user to configure the worker script URL or (2) use new SharedWorker(new URL('./shared-timer-worker.js', import.meta.url)).
An example usage of (1) will be:
import url from 'pkg/worker?url'
import { run } from 'pkg'

run({ workerUrl: url })

Note that for (2), the user would need to set the package to optimizeDeps.exclude (https://github.com/vitejs/vite/issues/11672).
Closing as this is not a bug in Vite.

@sapphi-red I updated the package to use the URL constructor and also a relative path for the worker. These were oversights on my part. Now this example more closely matches my real issue. I would expect with the relative import that vite would know to include the worker script in the .deps directory
The optimizeDeps.exclude worked for this example case, thank you for that. However, in my real use case the package is much larger and has many dependencies of its own, so it takes a long time to load in, and actually doesn't work for some reason. The time is on the order of minutes.
The idea of having URL be configurable is interesting, we may end up doing that. I guess I would have hoped there was another way via a new feature/bugfix in Vite (happy to help write it).
Let me know if I should reopen as a feature request, or if there's something else given the changes.

Then, it's a duplicate of #11672 (related #8427). It would be nice if we can make it work but it needs some idea unless esbuild supports it (https://github.com/vitejs/vite/issues/8427#issuecomment-1589084571). Rolldown might land before that which would fix this as well.


Connect HWINFO64 to the package

Is your feature request related to a problem? Please describe.
On windows 10 and 11, the package returns a cpu temperature that is under the ambient temp. It messes up my stuff.
Describe the solution you'd like
I wish it would be possible to get temperature data from apps like HWINFO64, HWMONITOR, Core Temp, etc in the package
Describe alternatives you've considered
Couldn't find any
Additional context
I have a minecraft server running on a W10 machine. After the cpu reaches a temperature threshold (70C), a discord bot gets launched. That bot updates a channel with information related to the temperature of the cpu, but that temp is always 16.8C. Tried on 2 machines, both ryzen, and it's kinda frustrating knowing that I could create a bot that can do that, but I cant show the correct temperature..

@Luminitza-Noptzii As I am not familiar with HWINFO64, HWMONITOR, ... are there command line options that I can read out??

@Luminitza-Noptzii As I am not familiar with HWINFO64, HWMONITOR, ... are there command line options that I can read out??

@sebhildebrandt Closest I could get for HWINFO64 are forums asking for command-line and a question in this forum which is incomplete
https://www.hwinfo.com/forum/threads/automatic-sensor-logging-via-command-line.8407/
and this for Open Hardware Monitor
https://github.com/openhardwaremonitor/openhardwaremonitor/issues/776

@Luminitza-Noptzii for now I do not see any easy way to integrate HWINFO64 data ... closing it for now but will continue trying to find a better way to solve this issue.


cmd/compile: method expressions are misprinted

On the dev.regabi branch, the error message for this source file:
package p

import "bytes"

var x int = (*bytes.Buffer).Len

mentions *bytes.Buffer.Len. That's incorrect: it should be (*bytes.Buffer).Len, as it appears in the source.
(I suspect this is related to the ir/fmt.go changes in 4629f6a51da5afabbebe9616f65fbfe0675d6039. Though it turns out we're printing it incorrectly even on master, where it prints as bytes.(*Buffer).Len.)
/cc @cuonglm

We've printed the bytes.(*Buffer).Len version since at least Go 1.1.

@ianlancetaylor Thanks, that sounds believable. You aren't suggesting cmd/compile should continue printing it incorrectly though, are you?
In case it wasn't clear, the linker symbols used for pointer-receiver methods (including as they appear in runtime stack traces) are unchanged on dev.regabi; and while I believe the parentheses are superfluous there, I don't intend to remove them either. This issue is only about how method expressions are printed in Go-syntax compiler diagnostics, like error messages and -m output.
go/types correctly prints (*bytes.Buffer).Len in its diagnostics.

cmd/compile now prints diagnostics using Go syntax for method expressions on dev.regabi.

cmd/compile now prints diagnostics using Go syntax for method expressions on dev.regabi.


fix(history): provide location and use named parameters

The API between createURL and parseURL was not symetric. Also it add too much parameters for it to be nice.

It's fast enough :)

Object always wins!

Let's agree to disagree :)

Is this a breaking change technically? Or do you think it's small and fast enough to be a patch?

Since the release was yesterday and is not documented yet, I'd say let's go for a patch and that's it.


Leading upper case.

When going trough the example I found that constructors do not start with upper case.
Could you give an explanation why?
Can type names also start with lowercase?

Yes, there is no required lexical distinction between any sort of binding. In a macro-enabled language, it doesn’t make a whole lot of sense to assign so much meaning to the names of identifiers. By convention, type names and typeclasses still start with uppercase identifiers, but all values start with lowercase letters.


Update drain.sh in activemq-artemis-launch module to use python3

Is your feature request related to a problem? Please describe.
Right now the script is using python2 which is very old.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
Optional. A clear and concise description of any alternative solutions or features you've considered.
Additional context
Optional. Add any other context or screenshots about the feature request here.

fixed by https://github.com/artemiscloud/activemq-artemis-broker-kubernetes-image/pull/82


Customize forms for creating objects

Feature Idea
Customize forms for creating objects (LimitRule, HardwareAsset, etc)

Split into sections
Add form documentation

Added custom create/update forms for limit rules, live in the new testing branch.
SIEM still needs forms for parsers, parse helpers.
Risk mgmt still needs custom forms.
Asset mgmt still needs custom forms.

Added custom forms for everything else. Live in dev/testing branches, will merge into master after some testing.

Now live in master, version 0.2-beta-r4


Request for feedback: Indented template/render literals

For my own purposes I require some sort of improved fstring, consider the following:
Render literal
def some_template(a):
  return ``

echo `{a}

  ``

sugar for
def some_template(a):
  return ["\n
echo ", a, "
\n"]

Render operators
html ``

echo `{a}

  ``

sugar for
html(["\n
echo ", a, "
\n"])

So my questions are:
Are the starlark people willing to break with python for things like this? Also, any feedback or thoughts on the feature idea would be appreciated. I will probably make an attempt at implementing this in my own fork.

created https://github.com/bazelbuild/starlark/issues/72 instead as per the readme.


#51: Ignore files

This PR adds logic to ignore files or folders using a glob pattern with the input exclude, to avoid the changelog to ask for changes if those files are modified.
Introduces the library wildcard-match because glob can only check the file system, the list of modified files is handled in memory since it comes from the GitHub API.
Fixes: https://github.com/zattoo/changelog/issues/51

@z-merge it


James 3797 Moving all source code related to SpamAssassin to module third-party/spamassassin

https://issues.apache.org/jira/browse/JAMES-3797
https://github.com/linagora/james-project/issues/4588

You have a conflict.

https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-1108/7/testReport/
Bunch of failure on some spam matcher... is it related?

Yes, likely!


Hide Hyper icon in MacOS top bar

[x] I am on the latest Hyper.app version

[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: MacOS Catalina

Hyper.app version: 3.0.2

The issue is reproducible in vanilla Hyper.app: yes

Issue
When opening the Hyper app, its icon is displayed in the MacOS top-bar (the bar which is always on the top of the screen containing a menu of the running app and icons of utils apps). As far as I can see, there is currently no way to hide this icon, which is rather a bad idea, cause I don't need a "terminal icon" appearing every time I'm starting using the terminal. Am I missing some config option for that?

Where are you seeing the hyper icon exactly? I'm confused
Can you post a screenshot?

@pablopunk Of course, you can find the screenshot below (the leftmost icon):

That's weird... I have the same Hyper version and also running Catalina and I can't see that. Did you upgrade to the second Beta of Catalina? I didn't yet

@pablopunk Interesting, on both beta1 and now beta2 I download Hyper 3.0.2 and it just says "The application Hyper cannot be opened." Any suggestions?

@evanheckert I'm getting the exact same error. I think it's related to an electron issue in Catalina, but I don't know why homebrew's version does work 🤔
Try brew cask install hyper and see if that one works

@pablopunk That worked great! Thank you kind sir!

You're very welcome 🧐

@pablopunk nope, I did not upgrade yet! But I'd really love to get rid of that icon ;)

@pablopunk After your reply, I realized that this problem might be connected to my installation of Hyper, so I started to change the config and I've found out the case! The plugin 'hyperterm-overlay' adds such an icon to the top bar ... I'm really sorry for taking your time and thank you for the assistance.


Unresolved External Symbol: public: double __cdecl double_conversion::StringToDoubleConverter::StringToDouble(char const *,int,int *)const

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): N/A
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): r1.9
Python version: 3.5
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): Cmake 3.10.2, swigwin 3.0.12, Visual studio 2015 v140
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce:  I followed to CMake tutorial here: https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/cmake TensorFlow CMake build

Describe the problem
Error when compiling a sample program in Visual Studio:

Severity	Code	Description	Project	File	Line	Suppression State
Error	LNK2019	unresolved external symbol "public: double __cdecl double_conversion::StringToDoubleConverter::StringToDouble(char const *,int,int *)const " (?StringToDouble@StringToDoubleConverter@double_conversion@@QEBANPEBDHPEAH@Z) referenced in function "bool __cdecl tensorflow::strings::safe_strtod(char const *,double *)" (?safe_strtod@strings@tensorflow@@YA_NPEBDPEAN@Z)	TensorFlow r1.9	D:\GIT\All\TensorFlow.plugin\tf_core_lib.lib(numbers.obj)	1
Severity	Code	Description	Project	File	Line	Suppression State
Error	LNK2019	unresolved external symbol "public: float __cdecl double_conversion::StringToDoubleConverter::StringToFloat(char const *,int,int *)const " (?StringToFloat@StringToDoubleConverter@double_conversion@@QEBAMPEBDHPEAH@Z) referenced in function "unsigned __int64 __cdecl tensorflow::strings::FloatToBuffer(float,char *)" (?FloatToBuffer@strings@tensorflow@@YA_KMPEAD@Z)	TensorFlow r1.9	D:\GIT\All\TensorFlow.plugin\tf_core_lib.lib(numbers.obj)	1

Is there any additional dependency that I am missing? Check below for the list. Thanks
Source code / logs
Sample program:
#include "stdafx.h"

#include <vector>
#include <eigen/Dense>

#include "matmul.h"
#include "tensorflow/core/public/session.h"
#include "tensorflow/cc/ops/standard_ops.h"

using namespace tensorflow;

int main()
{
	Scope root = `Scope::NewRootScope();
}

Additional Dependencies:
TensorFlow\r1.9\tensorflow\contrib\cmake\build\zlib\install\lib\zlibstatic.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\gif\install\lib\giflib.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\png\install\lib\libpng16_static.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\jpeg\install\lib\libjpeg.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\lmdb\install\lib\lmdb.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\jsoncpp\src\jsoncpp\src\lib_json\Release\jsoncpp.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\farmhash\install\lib\farmhash.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\fft2d\\src\lib\fft2d.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\highwayhash\install\lib\highwayhash.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\nsync\src\nsync\Release\nsync.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\snappy\src\snappy\Release\snappy.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\protobuf\src\protobuf\Release\libprotobuf.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_string_ops.dir\Release\tf_string_ops.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_cc.dir\Release\tf_cc.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_cc_ops.dir\Release\tf_cc_ops.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_cc_framework.dir\Release\tf_cc_framework.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_core_lib.dir\Release\tf_core_lib.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_core_cpu.dir\Release\tf_core_cpu.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_core_direct_session.dir\Release\tf_core_direct_session.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_core_framework.dir\Release\tf_core_framework.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_core_kernels.dir\Release\tf_core_kernels.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_core_ops.dir\Release\tf_core_ops.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\tf_cc_while_loop.dir\Release\tf_cc_while_loop.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\Release\tf_protos_cc.lib
TensorFlow\r1.9\tensorflow\contrib\cmake\build\sqlite\install\lib\sqlite.lib

Solved by including double-conversion.lib

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

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

__cdecl? Are you linking TensorFlow against a system library with a 32-bit ABI?
cc: @martinwicke @gunan https://www.agner.org/optimize/calling_conventions.pdf


Add "reconfig" command to read IP address again

When my network configuration changes and my VM gets a new IP address I would like to reconfigure the landrush DNS server without rebooting the box.
I thought about adding something like vagrant landrush reconfig to read the IP address again and reconfigure the DNS server.
@phinze wdyt?

I'm not using landrush anymore. Feel free to reopen if you still want this.


Update Organization#valid_items - return more than partner_key

Resolves #776
Must merge in conjunction with issue 100 on Partner
Description
In order to complete issue 100 on Partner App, the API::V1::PartnerRequestsController#show method (api/v1/partner_requests/:id) currently returns an array of strings representing the partner_key from the Organization's items.
To allow the partner app to show all the items (including name) when making a partner request, the json should have an id for the item, as well as the actual name.
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Tests are passing
Updated tests

OK let me know when  Partner#100 is done -- marking this one blocked for now

Hey @adam-barton I see that there is a pr on the partner app. Are these ready to go in?

I would double check with Josh just to be sure. He had me mark them as
blocked because there was one more piece that we didn't have time to get
to, but he said he would work on it over the next week.
-Adam
On Wed, Apr 24, 2019 at 5:51 PM Sean Marcia<EMAIL_ADDRESS>wrote:

Hey @adam-barton https://github.com/adam-barton I see that there is a
pr on the partner app. Are these ready to go in?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rubyforgood/diaper/pull/780#issuecomment-486478940,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJDAN3JEUDFY52UD65AE3CDPSD6A7ANCNFSM4HCSFOSA
.

@chaserx Are you able to run point on the partner side and confirm when {this} PR can be merged?

Merging in conjunction with Partner#109


explainability - cases in which only partial info is provided

There are two cases in explainability in which only partial information is provided.
These are cases of no connection in which only the reason of "no-connection" is provided.
These are the cases

Missing router to external address; the reason only partial info is provided here is that without the router it is not known whether NACL tables are relevant, and so we do not know what are the relevant information
Endpoints in two different VPCs with no transit gateway between them. The issue here is development oriented: our code is VPCConfig oriented, and due to the lack of the transit gateway there is no VPCConfig that contains the relevant endpoints.

Item 1 - the reason is not valid #751 - thus once we fix #762 this can be fixed as well


Payment Screen Blank

I'm setting up a system to let users deposit money into their account. When I try to generate the payment screen, it is returning a blank empty box.
You can see the URL here: https://merkd.com/me.wallet.gourl.php?amount=300
And my code is below:
`<?php
global $me;

$amount = (int)$_GET['amount'];

/**
 * @category    Example1 - Pay-Per-Product (single crypto currency in payment box)
 * @package     GoUrl Cryptocurrency Payment API
 * copyright 	(c) 2014-2016 Delta Consultants
 * @crypto      Supported Cryptocoins -	Bitcoin, Litecoin, Paycoin, Dogecoin, Dash, Speedcoin, Reddcoin, Potcoin, Feathercoin, Vertcoin, Vericoin, Peercoin, MonetaryUnit
 * @website     https://gourl.io/bitcoin-payment-gateway-api.html#p1
 * @live_demo   https://gourl.io/lib/examples/pay-per-product.php
 */

	require_once('engine/api/bitcoin/cryptobox.class.php');

	/**** CONFIGURATION VARIABLES ****/

	$userID 		= $me['id'];				// place your registered userID or md5(userID) here (user1, user7, uo43DC, etc).
										// you don't need to use userID for unregistered website visitors
										// if userID is empty, system will autogenerate userID and save in cookies
	$userFormat		= "COOKIE";			// save userID in cookies (or you can use IPADDRESS, SESSION)
	$orderID 		= "invoice00033";	// invoice number - 000383
	$amountUSD		= $amount;				// invoice amount - 2.21 USD
	$period			= "NOEXPIRY";		// one time payment, not expiry
	$def_language	= "en";				// default Payment Box Language
	$public_key		= ""; // from gourl.io
	$private_key	= "";// from gourl.io
	// IMPORTANT: Please read description of options here - https://gourl.io/api-php.html#options

	/********************************/

$options = array(
    "public_key"  => $public_key,         // place your public key from gourl.io
    "private_key" => $private_key,         // place your private key from gourl.io
    "webdev_key"  => "",        // optional, gourl affiliate program key
    "orderID"     => $orderID, // few your users can have the same orderID but combination 'orderID'+'userID' should be unique
    "userID"      => $me['id'],        // optional; place your registered user id here (user1, user2, etc)
    // for example, on premium page you can use for all visitors: orderID="premium" and userID="" (empty)
    // when userID value is empty - system will autogenerate unique identifier for every user and save it in cookies
    "userFormat"  => $userFormat,   // save your user identifier userID in cookies. Available: COOKIE, SESSION, IPADDRESS, MANUAL
    "amount"      => 0,         // amount in cryptocurrency or in USD below
    "amountUSD"   => $amount,         // price is 2 USD; it will convert to cryptocoins amount, using Live Exchange Rates
    // For convert fiat currencies Euro/GBP/etc. to USD, use function convert_currency_live()
    "period"      => $period,  // payment valid period, after 1 day user need to pay again
    "iframeID"    => "",         // optional; when iframeID value is empty - system will autogenerate iframe html payment box id
    "language"    => "EN"       // text on EN - english, FR - french, please contact us and we can add your language
);

// Initialise Bitcoin Payment Class
$box = new Cryptobox ($options);

// Display payment box with custom width = 560 px and big qr code / or successful result
$payment_box = $box->display_cryptobox(true, 560, 230, "border-radius:15px;border:1px solid #eee;padding:3px 6px;margin:10px;",
    "display:inline-block;max-width:580px;padding:15px 20px;border:1px solid #eee;margin:7px;line-height:25px;");

// Log
$message = "";

// A. Process Received Payment
if ($box->is_paid())
{
    $message .= "A. User will see this message during 24 hours after payment has been made!";

    $message .= "
".$box1->amount_paid()." ".$box1->coin_label()."  received
";

    // Your code here to handle a successful cryptocoin payment/captcha verification
    // For example, give user 24 hour access to your member pages
    // ...

    // Please create/use IPN (instant payment notification) function cryptobox_new_payment() for update db records, etc
    // Function cryptobox_new_payment($paymentID = 0, $payment_details = array(), $box_status = "") called every time
    // when a new payment from any user is received.
    // IPN description: https://gourl.io/api-php.html#ipn
}
else $message .= "The payment has not been made yet";

// B. Optional - One-time Process Received Payment
if ($box->is_paid() && !$box->is_processed())
{
    $message .= "B. User will see this message one time after payment has been made!";

    // Your code here - user see successful payment result
    // ...

    // Also you can use $box1->is_confirmed() - return true if payment confirmed
    // Average transaction confirmation time - 10-20min for 6 confirmations

    // Set Payment Status to Processed
    $box->set_status_processed();

    // Optional, cryptobox_reset() will delete cookies/sessions with userID and
    // new cryptobox with new payment amount will be show after page reload.
    // Cryptobox will recognize user as a new one with new generated userID
    // $box1->cryptobox_reset();

    /********************************************************************************************************/
    /**  This IPN function is used every time a new payment from any user is received successfully         **/
    /**  Function receives paymentID - current payment ID (record id in your mysql table crypto_payments), **/
    /**  payment details as array and box_status - 'cryptobox_newrecord' OR 'cryptobox_updated'.           **/
    /**                                                                                **/
    /**  Move this function to the bottom of the file cryptobox.class.php or create a separate file        **/
    /**  More info: https://gourl.io/api-php.html#ipn                                                      **/
    /********************************************************************************************************/
    function cryptobox_new_payment($paymentID = 0, $payment_details = array(), $box_status = "")
    {
        // Your php code here to handle a successful cryptocoin payment/captcha verification
        // for example, send confirmation email to user
        // update user membership, etc - https://gourl.io/api-php.html#ipn

        // .... ....

        return true;
    }

}

?>
<!DOCTYPE html>
    <meta http-equiv='cache-control' content='no-cache'>

<?= $payment_box ?>
<?= $message ?>

`

You need to copy cryptobox.min.js javascript file to your home directory also
https://merkd.com/cryptobox.min.js


Fire event when expanding or collapsing an item

This seem to be already implement we do fire expandevent.


Create dependabot.yml

📝 Description
This is to enable dependabot for py-metadata.

Lint errors are irrelevant and should probably be fixed in another PR.


Remove Travis Docker Build

Converting to docker cloud automated builds, so removing Travis build integrations.
Proposed changes:

Removing automated docker build via Travis to replace with docker cloud automated builds

Status:

[ ] ready for code review
[ ] there are tests for the functionality
[ ] documentation updated
[ ] changelog updated

looks good :+1:


Upgrade to v1.0.0-RC1 of go.opentelemetry.io/otel

Thanks for raising this PR.
Please run make precommit before commit. And, we should also change the go.mod in the example folder.

Just a FYI that I am blocked upgrading to OTEL v1.0.0-RC1 in our repositories waiting on this MR :)

Just a FYI that I am blocked upgrading to OTEL v1.0.0-RC1 in our repositories waiting on this MR :)

Sorry for the late upgrade. I kind of busy recently. I can help with this PR to merge.

Thanks for your contribution! @rcrowe

Just a FYI that I am blocked upgrading to OTEL v1.0.0-RC1 in our repositories waiting on this MR :)

Sorry for the late upgrade. I kind of busy recently. I can help with this PR to merge.

thank you for doing this! Unblocked now and looking good 🎉 . Thanks again.


Mistake mark showing up while using hypen character (-) in Salesforce Apex language

Type: Bug

Create a class in sfdx project.
open a new "switch on" statement within any method
add a hypen within any of the "when" values, e.g.
//script
String foo = 'Test - 1';
switch on foo {
when  'Test - 1'{
//case for "Test - 1"
}
when else {
//default case
}
}
open brace gets marked with red color and the string "Test - 1" is incorrectly colored, suggesting there is a mistake in case having the hyphen character, the codes compiles correctly anyways. This scenario should be replicable with other special characters

VS Code version: Code 1.75.1 (441438abd1ac652551dbe4d408dfcec8a499b8bf, 2023-02-08T21:32:34.589Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8565U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.81GB (1.93GB free)

Process Argv
--crash-reporter-id 43000006-d192-4e86-a6f3-b66d6781f3dc

Screen Reader
no

VM
0%

Extensions (12)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.4.0

forcecode
Joh
4.0.5

salesforce-vscode-slds
sal
1.4.8

salesforcedx-vscode
sal
57.3.0

salesforcedx-vscode-apex
sal
57.3.0

salesforcedx-vscode-apex-debugger
sal
57.3.0

salesforcedx-vscode-apex-replay-debugger
sal
57.3.0

salesforcedx-vscode-core
sal
57.3.0

salesforcedx-vscode-lightning
sal
57.3.0

salesforcedx-vscode-lwc
sal
57.3.0

salesforcedx-vscode-soql
sal
57.3.0

salesforcedx-vscode-visualforce
sal
57.3.0

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.76.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


node folder not showing

As the title suggests, the node folder icon is not being shown.
VSCode 1.44.2
vscode-icons 10.1.1

The node icon only works for node_modules folder name. You can enable that by using the customisation feature.

The node icon only works for node_modules folder name. You can enable that by using the customisation feature.

So I leave it as a suggestion. A folder icon for folders called "node" or "nodejs". It could be the image of a green folder with the nodejs emerald on it.

node could also mean a node in a cluster of server

https://docs.microsoft.com/en-us/mem/configmgr/core/servers/deploy/configure/use-a-sql-server-cluster-for-the-site-database
https://kubernetes.io/docs/concepts/architecture/nodes/

If it is to be then nodejs should be the name of the folder.
This issue should be closed by October 2020 if it doesn't have any traction.

The day has come and this issue shall be closed 🤖


DNN Tracking DLL Dependencies fails when multiple Versions are in Use

Description of bug
Two modules using the same DLL in different versions can easily do so by placing them in a subfolder in the bin and setting the Web.Config rules for version redirects. This works at runtime, but fails on module installs and updates. Here's a real example where 2sxc and DnnSharp run into a problem (see also the issue on 2sxc:

2sxc uses EF 1.1 and the MS Dependency Injection 1.1 and places all this in bin/2sxc
PlantAnApp (DnnSharp) uses EF 2.1 and places all this in bin/PlantAnApp

Because the web.config redirects manage all this, it's not a problem. But on Module-Install and Upgrade it fails. The reason is that DNN tries to help by tracking installed DLLs, but doesn't keep track of the folder. So even though the files don't conflict, DNN checks the DB and assumes a newer version has already been installed (Table: Assemblies):

Note: for historic reasons the EF in 2sxc has the wrong version (it believes it's EF v11 or 12 because that's the 2sxc version). This may lead to more confusion, because actually installing 2sxc after PlantAnApp works, but updating PlantAnApp will fail then. But the core problem is that the version ignores the folder it's in.
Current behavior
This leads to the following problems:

If PlantAnApp is installed first, 2sxc with EF v1.1 wouldn't install, because DNN believes it already has a newer version.
If 2sxc with EF 1.1 is installed first, the entry would say v1.1 and Would allow PlantAnApp to install it as well. That would bump the version to 2.1. After that updating 2sxc with a newer EF (say 1.2) would fail, because DNN believes it already has a newer version.

Expected behavior
DLL Versions in Assemblies should include the folder they are in, as the same DLL can be installed in different versions.
Affected version

[x] All DNN versions as of now up and till v09.09.01

If you're maintaining your own collection of DLLs in its own folder, and managing your own binding redirects, etc., then I'm not sure that you're getting any value out of using the Assembly component, rather than just a simple File component. Are there features of the Assembly component you're looking to keep?

@bdukes I'm not sure what you mean. We're just listing our DLLs in our Dnn Manifest since we assume that well-behaved modules should do this, and I think it's necessary that they get installed within the bin (sub) folder.
What features would Assembly components have? I'm not really aware of any features in this scope.

Using an Assembly component (as opposed to a File component) provides the following features:

Registers the assembly in the Assemblies table (records what version of an assembly each package installed)
Creates a binding redirect for strong-named assemblies
Checks the version of installed assemblies to avoid overwriting newer assemblies with older assemblies

It doesn't seem to me that any of those features are of use to these assemblies that are installed to an alternate folder. Certainly a well-behaved module should declare their shared assemblies that are going into the bin folder using the Assembly component, but assemblies that you're managing and that are not shared probably don't benefit by using the Assembly component.

I just looked at the .dnn file in your latest install package and wanted to respond to the comments 🙂
The File component will report file extensions that are not on the allowed list. However, the ResourceFile component will not.
https://github.com/2sic/2sxc/blob/9ba57100fa532aef41ec96a9bb0f347ac89aeb73/Src/Dnn/ToSic.Sxc.Dnn.Main/DnnPackageBuilder/2sic_2SexyContent.dnn#L266-L275
It looks like this is what you're doing now for pdb files, you could do the same for your bin/2sxc/*.dll files.
https://github.com/2sic/2sxc/blob/9ba57100fa532aef41ec96a9bb0f347ac89aeb73/Src/Dnn/ToSic.Sxc.Dnn.Main/DnnPackageBuilder/2sic_2SexyContent.dnn#L286-L293
Regarding PDB files, it looks like you implemented support for pdb files in the Assembly component in DNN 9.1.1.
https://github.com/2sic/2sxc/blob/9ba57100fa532aef41ec96a9bb0f347ac89aeb73/Src/Dnn/ToSic.Sxc.Dnn.Main/DnnPackageBuilder/2sic_2SexyContent.dnn#L145-L148

I have to say that I agree with @bdukes on this one.
The nested/sub-folder approach is not necessarily an approach that is truly publicized, and requires additional work (config section additions, etc).
I think that the usage of the file component is a better option for these.

@bdukes and @mitchelsellers thanks for your insights.
Note that even though we contributed the .pdb fix we cannot use it ourselves, because 2sxc can still be installed on Dnn 7.4.2.
@bdukes From what I understand this is what would resolve the issue:

We could change our DLLs which are in a subfolder should become ResourceFile dependencies and not Assembly dependencies
DnnSharp should do the same - this would be important for @rezagarrow

I assume this would look like this:
<component type="ResourceFile">
  <resourceFiles>
	<basePath>bin\2sxc</basePath>
	<resourceFile>
	  <name>Microsoft.Extensions.DependencyInjection.Abstractions.dll</name>
          <name>Microsoft.Extensions.DependencyInjection.dll</name>
          <name>Microsoft.Extensions.Logging.Abstractions.dll</name>
          <name>...</name>
	</resourceFile>
  </resourceFiles>
</component>

I think this would be simple to do for both of us, but there is a problem making the change. I assume we should somehow tell the installer that the old Assembly entries should be cleared, even though we're still including the file. So it's a kind of CleanUp but then again the files should stay there.
Do you have a suggestion how we could add this to the manifest and not break new installs or upgrades? I simply have no clue on this. Many thanks!

The Resource File component takes a zip file and unzips it into a given folder:
        <component type="ResourceFile">
          <resourceFiles>
            <basePath>bin/2sxc</basePath>
            <resourceFile>
              <name>2SxcBinResources.zip</name>
            </resourceFile>
          </resourceFiles>
        </component>

(or you can remove the basePath element and include that path in the zip file itself, which is what you're doing for the PDB files).
For clearing out the Assemblies table, I don't think there's a good way other than running your own SQL script.  Something like this:
DELETE a
FROM {databaseOwner}[{objectQualifier}Assemblies] a
JOIN Packages p ON a.PackageID = p.PackageID
WHERE p.Name = '2SexyContent'

@bdukes ok, I was a bit afraid that it would need SQL 😢. Thx for the update.

ok, we can close this for now.


merge from arlac77/template-github-action,arlac77/template-kronos-component

.github/workflows/ci.yml

chore: (jobs.build.steps)

.github/workflows/update_readme_api.yml

chore: (jobs.update_readme_api.steps)
chore(deps): add 14.4.0 remove 14.2.0 (jobs.update_readme_api.steps.with.node-version)

Codecov Report

Merging #974 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #974   +/-   ##
=======================================
  Coverage   92.40%   92.40%
=======================================
  Files          10       10
  Lines        1409     1409
  Branches      160      160
=======================================
  Hits         1302     1302
  Misses        107      107

Continue to review full report at Codecov.

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


Remove implicit ordering params

@bherd-rb This PR is a first step towards making the ordering parameters explicit. This allows the creation of an overloaded empty constructor where the user can either pass ordering parameters or not (in which case the default orderings are used).
This allows a user to create a order book as follows in Java...
FourHeapOrderBook<GoogleStock> orderBook = FourHeapOrderBook.empty();

...and in Scala...
val orderBook = FourHeapOrderBook.empty[GoogleStock]

...there is still a bit of work to do cleaning up the way orderings are defined on the Scala side. In particular I am thinking about pulling the orderings out of the companion objects and into a separate orderings package.  I think this would allow easier use of defined orderings from the Java side (i.e., no messing with MODULE$ syntax; would also allow for easier sub-classing of orderings.

@bherd-rb The changes originally in this PR have made it into develop via other PRs! I am going to leave this branch open and use it to explore some of the ideas we discussed concerning extracting the orderings from the various companion objects into classes (possibly in a separate package) to improve the syntax for Java users.

I am going to close this an open a clean PR for extracting orderings.


Add types for <Html> and <Head> element

Ran into this converting _document to Typescript, I couldn't add the lang attribute.

@huv1k is going to add the tests 👍


Getting error when installing

Hi! This looks great! I'm getting an error when I try to start phoenix:
==> wootheex
Compiling NIF crate :wootheex_nif (native/wootheex_nif)...
could not compile dependency :wootheex, "mix compile" failed. You can recompile this dependency with "mix deps.compile
wootheex", update it with "mix deps.update wootheex" or clean it with "mix deps.clean wootheex"
** (ErlangError) Erlang error: :enoent
    (elixir) lib/system.ex:791: System.cmd("cargo", ["rustc", "--release"], [cd: "/PATH_TO_PROJECT/deps/wootheex/native/wootheex_nif", stderr_to_stdout: true, env: [{"CARGO_TARGET_DIR", "/PATH_TO_PROJECT/_build/test/rustler_crates/wootheex_nif"}], into: %IO.Stream{device: :standard_io, line_or_bytes: :line, raw: false}])
    lib/mix/tasks/compile.rustler.ex:59: Mix.Tasks.Compile.Rustler.compile_crate/1
    (elixir) lib/enum.ex:1336: Enum."-map/2-lists^map/1-0-"/2
    lib/mix/tasks/compile.rustler.ex:15: Mix.Tasks.Compile.Rustler.run/1
    (mix) lib/mix/task.ex:331: Mix.Task.run_task/3
    (mix) lib/mix/tasks/compile.all.ex:73: Mix.Tasks.Compile.All.run_compiler/2
    (mix) lib/mix/tasks/compile.all.ex:53: Mix.Tasks.Compile.All.do_compile/4
    (mix) lib/mix/tasks/compile.all.ex:24: anonymous fn/1 in Mix.Tasks.Compile.All.run/1

I've done the compile, update, and clean steps suggested and no change. Any idea what's going on? Is this something on my end?

You need to install rust compiler (rustc) along with it's cargo package manager. Gotta be something like rust in many distros.
Guess should clear this up in the readme.


Moved grade projections to home page and removed from buckets page

Fixed bug with mapping score to letter grade in the "Grade Projections" table.


"Recreate" update strategy

Is there a way to recreate all clickhouse pods at the same time instead of the current RollingUpdate strategy whenever there are changes. It is taking a very long time for all pods to restart with RollingUpdate.
In case of rollingupdate, is there an option to set maxSurge?

Can you check this pr https://github.com/Altinity/clickhouse-operator/pull/1124?
It should solve your problem.

@dheerajathrey
Since 0.22 the default strategy for big clusters is the following:

Upgrade (re-create) the first shard  (one replica in a time)
The upgrade 50% of remaining shards in parallel (one replica in a time), maximum in 5 parallel threads.
Then upgrade the remaining cluster (one replica in a time), maximum in 5 parallel threads.

This can be changed/tuned in operator configuration, see there: https://github.com/Altinity/clickhouse-operator/blob/221d151a80e57f02ca4a1a421ff3698294553426/config/config.yaml#L207


AspNet WebSocket Transport

Based on #775
Related to #736
As it's not recommended to use Task.Run in ASP.NET Full framework, we are creating a new package (.NET Full Framework 4.7.2) called PuppeteerSharp.AspNetFramework.
This package has a class named AspNetWebSocketTransport which uses HostingEnvironment.QueueBackgroundWorkItem instead of Task.Run.

@Meir017 would you mind taking a look at this guy? I really like to have your feedback here.
I had to change a few things on our build process because we have not a .NET Framework library in our solution.

Not ready to be reviewed, I will create another solution and AppVeyor project because it's breaking my Mac setup

I noticed this was mentioned in the December 18 update but it's not yet merged right?

@Jojoshua we will ship it with v1.11 next week :)

Hey I saw this was merged. I just updated to 1.11.1 but I am getting ....
The type or namespace name 'AspNetWebSocketTransport' could not be found (are you missing a using directive or an assembly reference?)
Any ideas?

@Jojoshua you will need to add this package https://www.nuget.org/packages/PuppeteerSharp.AspNetFramework/

Ahh I missed that. Thanks!


Issue with sending emails

Hello, I'm trying to send an email from Python app using Microsoft Graph SDK. The code that I show below runs well and does not throw any error. However I do not receive any email to the email address indicated in toRecipients.
What is missing?
import os
from msgraphcore import GraphSession
from azure.identity import ClientSecretCredential

scopes = ['user.read']
    graph_credentials = ClientSecretCredential(
      client_id=os.environ["AZURE_CLIENT_ID"],
      client_secret=os.environ[AZURE_CLIENT_SECRET"],
      tenant_id=os.environ[AZURE_TENANT_ID"]
)
graph_session = GraphSession(graph_credentials, scopes)
post_sample(graph_session)

def post_sample(graph_session):
    body = {
        'message': {
            'subject': 'Python SDK test',
            'body': {
                'contentType': 'Text',
                'content': 'Bla-bla-bla.'
            },
            'toRecipients': [{
                'emailAddress': {
                    'address'<EMAIL_ADDRESS>                }
            }]
        }
    }

    # ['mail.send']
    result = graph_session \
        .post('/me/microsoft.graph.sendMail',
              data=json.dumps(body),
              scopes=['.default'],
              headers={'Content-Type': 'application/json'}
              )
    print(result.status_code)

AB#8590

Hi Liana,
Thanks for reporting this issue.

If you are getting a success response code (200/202), check the spam/junk folder of the recipient email. This is a common issue with sending emails from test/demo accounts.
If you are getting a 403 response, check that you are authorizing with the right credentials and client has the appropriate permissions to send mail.

Let us know if the issue persists.


First HuszarR conversion

Past project, data collected and paper published.
Paper: Preconfigured dynamics in the hippocampus are guided by embryonic birthdate and rate of neurogenesis
Data: Buzsaki Globus, HuszarR folder, everything under optotagCA1
Other info:

The data was all recorded from mouse CA1- they are long recordings, many sessions have behavior on a familiar T-shaped maze, there are also animals that were exposed to both familiar and novel environments.
Some of the sessions were recorded with dual sided probes from Diagnostic Biochips, which gave us really good cell yield. One session has up to 480 cells, which is really great for ipsilateral CA1.
Sessions with these high cells yields (we also have more that haven't been spike sorted) might be especially useful for the wider community.

Another unique aspect of the dataset is that it includes information about the embryonic birthdate of pyramidal neurons, which (as it turns out) constrains the activity patterns across brain states (the paper is all about this).
The data is organized according to the embryonic birthdate of the cells, e.g, folder "e15" will have all animals where neurons of embryonic day 15 were targeted. There were 6 such animals (e.g., "e15_13f1"), and the folder of each holds the recording sessions that contributed to the analysis in the paper.
In the supplementary information of the paper (table 1, also attached), I describe what manipulations were done for each animal- it's not always the same.

In my case, all the data is in the folder HuszarR/optotagCA1, and it is primarily CellExplorer and buzcode-related formatting.
I did not include any LFP data, although the ripple files include a single channel of LFP from the middle of the pyramidal layer. This works well for detection of events, but for finer timing-related LFP analysis (e.g., phase of hippocampal theta), this might be insufficient.
Also I did not include any of the raw data in these folders.

I am talking to them about getting the raw data included in this conversion.

Prototyping: /HuszarR/optotagCA1/e13/e13_16f1/e13_16f1_210302/

To-do for Huzsar conversion.
The skeleton of the conversion is on https://github.com/catalystneuro/buzsaki-lab-to-nwb/pull/58

[ ] Cofirm that in the SleepState.states file, the timestamps match with the current event representation in #58
[ ] Add position from Behavior.mat file
[ ] Add trials from Behavior file.
[ ] Modify either the interface or the sorter for CellExplorer to account for the new information. Right now, the sorting interface is added as an ad-hoc interface in #58.
[ ] Check if there is anything to add from the ripples.events.mat. Possible use https://pypi.org/project/ndx-events/ for this.
[ ] Add datetime for the session.
[ ] Wait to see how the recording data will look like. When that is available decide how to include the chanMap.mat information.
[ ] Add metadata for subject, experimenter, etc.
[ ] Run the conversion for all sessions.
[ ] Create stub nwb files and share them with team for analysis.
[ ] Once correct, run the conversion for all the files again and create a requirements.file with pin dependencies for this specific conversion.
[ ] Upload conversion to dandi.

Done and published
Great job both of you @h-mayorquin @garrettmflynn on this rapidly developed conversion, especially given how complex the data streams turned out to be


Support IPFS hosting with hash router

[x] Add env config to control switch between hash router and browser router, so we can set to use hash router when deploy site to IPFS
[x] Config favicon, imgs urls to make them recognized in IPFS hosted environment

BTW, IPFS hosting will have domain https://ipfs.dotreasury.com

A bug: network switch button won't produce hash urls in IPFS hosting environment. @qiyisi


update mypy version to match next-mypy in azure-sdk-for-python

Updating the pinned mypy version to target the version we're bumping up to in azure-sdk-for-python in October. CI seems to be passing so I think it's safe to merge?

I went ahead and updated to 1.6.1, so closing this PR. thanks @kristapratico


Do not create a folder in start menu for typora

For now, typora is placed in a "(default)" folder in start menu. I think placing it in Start -> Programs directly will be better.

This isn’t controlled in the package, but in the installer itself. You should file this issue at https://github.com/typora/typora-issues/.
If you know some installer argument that can change this behavior, please comment and I’ll re-open this issue.

I tried to install it manually with the official installer. It turns out that there is a typora folder actually. But with choco, It becomes "(default)".

I'm able to reproduce this behavior with https://typora.io/windows/typora-update-x64-0621.exe
So this should still be reported to the original author.


package-lock.json

@Balaadithya002, As I can see, you've created a PR without meaningful contributions and with an ambiguous or incorrect description. I kindly request you to avoid submitting spam PRs. If you'd like to contribute, please create a valid issue or work on an existing one, and then submit your PR with real improvements and a clear description.


Holiday object doesn't generate holidays until contains() ran

As shown below holidays won't appear in the object until I check if a date is in the holiday object.
Python 2.7.5 (default, Oct 11 2015, 17:47:16)
[GCC 4.8.3 20140911 (Red Hat 4.8.3-9)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import holidays
>>> print holidays.__version__
0.5
>>> h = holidays.UnitedStates()
>>> h
{}
>>> datetime.datetime.now() in h
False
>>> h
{datetime.date(2016, 11, 24): 'Thanksgiving', datetime.date(2016, 2, 15): "Washington's Birthday", datetime.date(2016, 7, 4): 'Independence Day', datetime.date(2016, 9, 5): 'Labor Day', datetime.date(2016, 12, 25): 'Christmas Day', datetime.date(2016, 1, 1): "New Year's Day", datetime.date(2016, 10, 10): 'Columbus Day', datetime.date(2016, 1, 18): 'Martin Luther King, Jr. Day', datetime.date(2016, 12, 26): 'Christmas Day (Observed)', datetime.date(2016, 11, 11): 'Veterans Day', datetime.date(2016, 5, 30): 'Memorial Day'}

This is on purpose. There is an optional years parameter you can use to force it to generate specific years before checking any dates.
>>> h = holidays.US(years=2016)
>>> h
{datetime.date(2016, 11, 24): 'Thanksgiving', datetime.date(2016, 2, 15): "Washington's Birthday", datetime.date(2016, 7, 4): 'Independence Day', datetime.date(2016, 9, 5): 'Labor Day', datetime.date(2016, 12, 25): 'Christmas Day', datetime.date(2016, 1, 1): "New Year's Day", datetime.date(2016, 10, 10): 'Columbus Day', datetime.date(2016, 1, 18): 'Martin Luther King, Jr. Day', datetime.date(2016, 12, 26): 'Christmas Day (Observed)', datetime.date(2016, 11, 11): 'Veterans Day', datetime.date(2016, 5, 30): 'Memorial Day'}

# or multiple years
>>> h = holidays.US(years[2015, 2016, 2017])

One option would be to default the years argument to the current year but my thinking was as follows:
At the time of creating object h it doesn't know what year(s) you are going to be working with and you wouldn't want to waste cpu cycles generating the current year if you plan on working with another year, or perhaps you initialize the holiday object and never need to use it.
If you think this is not the ideal default I'm open to be persuaded differently.

@ryanss that makes sense. Appreciate your response. No I think your reasoning is valid. Mind if create a Pull Request that updates the docs with an example of initiating the object with the years? As I'm using it to get a list of holidays to exclude from dateutil rrule set.

Sure, better documentation is always helpful.
You may be interested in another one of my projects https://github.com/ryanss/bdateutil.
The main feature is adding a new bday option to relativedelta and BDAILY to rrule. The last example in the top Example Usage section might be what you're looking for.
import holidays
from bdateutil import rrule, BDAILY
from datetime import date

for dt in rrule(BDAILY, count=10, dtstart=date(2015, 12, 23), holidays=holidays.US()):
    print dt

Unfortunately, the holidays keyword is only available in the master branch, not the latest release v0.1, so you would have to clone the repository and python setup.py install instead of installing using easy_install or pip. I will be releasing v0.2 to PyPI sometime in the next week.


feat: Standalone dns proxy

Note: This PR is includes changes from this PR: https://github.com/cilium/cilium/pull/36121
The PR adds the standalone dns proxy and handling the client side of the grpc created in #36121.

Standalone DNS Proxy will start by starting the DNS Proxy with a type StandaloneDNSProxy and creating a trigger that handles the connection to cilium agent.
The controller tries to connect to the cilium agent and create the stream using the SubscribeToDNSPolicies(For receiving the DNS policies) and then start a go routine to listen on the created stream. It retries to use the connection/stream if there are multiple triggers.
SubscribeToDNSPolicies is called as a part of trigger to create a stream to listen for DNS policy rules from cilium agent.
UpdateMappings is used during the callback with the DNS mappings in the NotifyDNSMsg in the DNS proxy. This is used to send the data to cilium agent to configure the policy.

Fixes: https://github.com/cilium/cilium/issues/30984
Related PRs: https://github.com/cilium/cilium/pull/36213, https://github.com/cilium/cilium/pull/36215
CFP: https://github.com/cilium/design-cfps/pull/54

Including David in reviewers per his request.

The specifics of the design are still being discussed in https://github.com/cilium/design-cfps/pull/54. I'm marking this PR as draft, so it doesn't show up in reviewer's queues. Please feel free to mark as ready for review once the discussion on the CfP has converged and the PR updated accordingly.


When the value of the min is equal to a negative number. Then drag one of the handles to 0. the segments change color incorrectly

When the value of the min is equal to a negative number. Then drag one of the handles to 0. the segments change color incorrectly
 sendbox Code
 Codesendbox

New version of the library doesn't have this issue.
https://tanstack.com/ranger/v1
Hence, closing this one.


[FEAT]: Create ICICLE <> Ark bindings

Description
Please provide a clear and concise description of the feature you would like included.
Motivation
Please provide a clear and concise description of the motivation for adding this feature.

@ImmanuelSegol not sure about this issue, is it the same as point 2 in https://github.com/ingonyama-zk/icicle/issues/137?


[AutoRelease] t2-reservations-2022-09-15-23065(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3174
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1024

issue link:https://github.com/Azure/sdk-release-request/issues/3174


Sleeping CDT

This crash happens when you right click on a bed
You get into the bed for less than half a second and then Crash To Desktop
crash log

This is fixed
Can't remember in what commit though


Update thunder

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Build failure seems unrelated, restarting.


fix strong-next notation. \circ to \bullet

Fix in Section 4.4 Rewiting and Reachability the notation for "strong next" symbol. Change it from \circ to \medbullet. In order to do that, I used the package txfonts which has an affect in the appearance of the fonts of the entire document.

Jenkins: test this please


JAVA system capability not detected on windows with only JDK installed

Agent version and platform
Version of your agent? 1.95.3

OS of the machine running the agent? Windows

VSTS type and version
On-Prem TFS
2015 Update 3

What's not working?
The system cabability of 'JAVA' does not get detected if only the JDK is installed.
vsts-agent/src/Misc/layoutbin/powershell/Add-JavaCapabilities.ps1

Expectation
If the JDK is installed, then it also installs the JRE, this should be detected
eg. C:\Program Files (x86)\Java\jdk1.8.0_102\jre

This repo is for the 2.x agent.  You supplied the 1.x for QU3.


Manually creating a Dataset table will not render a table in Dataviz

It seems like using .data(ds) instead of a parseRequest or parseRawData neglects to set the dataType, which pisses off the visualization. If you manually run viz.dataType('categorical') everything works wonderfully.
Here's a fiddle: http://jsfiddle.net/7pn1hb6p/1/

You forgot to uncomment the line that says "uncomment this line for the table to show up"
http://jsfiddle.net/yk1kmbtz/
    table
      .data(ds)
      // This next line needs to be uncommented for the table to show up
      .dataType('categorical')
      .render();

Yeah, that was intentional :)
That dataType line shouldn't be required. It should work without being there.

Ahh in that case, I am curious what the solution is. Because I need to create a dataset manually on the fly too.

Closing this for now- keen-dataviz.js (now inside<EMAIL_ADDRESS>handles this properly and can be loaded on top of earlier versions of keen-js if need be. Sorry for the super late reply!


how about creating one Chinese branch?

I think it very meaningful.

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


Turn off -Wswitch-default

Requiring all switch statements have a default case is not that
sensible. It's better to let the developers omit the default case so
that the compiler can check that a switch on an enum is exhaustive.
This way, when new enum types are added, the developer can lean on the
compiler to catch all the unhandled cases instead of doing a manual
audit.
This commit removes the -Wswitch-default compiler flag and also
codemods the codebase to omit default cases where appropriate.
Note that the unreached return statements are necessary to avoid
warnings. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87951 for
in-depth discussion.
Checklist

[n/a] Language changes are updated in docs/reference_guide.md
[n/a] User-visible and non-trivial changes updated in CHANGELOG.md
[n/a] The new behaviour is covered by tests

Now with this diff:
diff --git a/src/types.cpp b/src/types.cpp
index d1477d64..af2a2f2a 100644
--- a/src/types.cpp
+++ b/src/types.cpp
@@ -222,7 +222,6 @@ std::string probetypeName(ProbeType t)
     case ProbeType::usdt:        return "usdt";        break;
     case ProbeType::tracepoint:  return "tracepoint";  break;
     case ProbeType::profile:     return "profile";     break;
-    case ProbeType::interval:    return "interval";    break;
     case ProbeType::software:    return "software";    break;
     case ProbeType::hardware:    return "hardware";    break;
     case ProbeType::watchpoint:  return "watchpoint";  break;

you get this warning (I had -Werror on for the build below):
/home/daniel/dev/bpftrace/src/types.cpp: In function ‘std::string bpftrace::probetypeName(bpftrace::ProbeType)’:
/home/daniel/dev/bpftrace/src/types.cpp:215:11: error: enumeration value ‘interval’ not handled in switch [-Werror=switch]
  215 |    switch (t)


Corrects syntax for markdown link in instrumentation README

noticed that a link was not created properly in markdown syntax while reading the instrumentation guides for creating new instrumentation

appreciated!


Dependency package rollup-plugin-polyfill-node has a new version available

We have identified a dependency on version 0.8.0 of rollup-plugin-polyfill-node. A new version (0.12.0) is available for upgrade.
Following are the steps to upgrade package dependency.

Understand the breaking changes between the version being used and the version you want to upgrade to.

Identify all packages that take a dependency on this package.

Go to the root folder for each such package (/sdk/service-name/package-name) and update package.json to have the new version.

Run rush update to ensure the new version is pulled in.

Make relevant changes to absorb the breaking changes.

Repeat steps 3 to 5 for each of the packages that have a dependency on this package.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.36768338,Service Bus:0.1396759,Azure.Identity:0.06084747'

already upgraded to 0.12.0


Skip pypy3 tests for psycopg2

OpenTelemetry contrib skips pypy3 env for psycopg2 so we should follow suite.

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


Logging on Retry policy and Temporary add AZURE_LOG_LEVEL to 2 for investigating

Investigate this error:
com.azure.core.exception.HttpResponseException: Status code 429, "{"type":"https://azconfig.io/errors/too-many-requests","title":"The amount of requests sent to the server have surpassed the assigned quota","policy":"TotalRequests","status":429}"

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=119198&view=logs&j=3f0b3bd8-2d88-59b8-ec21-d9a54670d8fb
Add logging info to the RetryPolicy
and temporary turn on AZURE_LOG_LEVEL to 2 for investigation of above 429 too many request error

The changes to tests.yml look OK... Is the goal here to investigate or permanently have the log level set to 2?
If the goal is to investigate we can do investigation out of a branch and playground or use a draft PR.

@mssfang I'm tagging this as do not merge so it doesn't accidentally ship in preview 4

Will not merge this for preview 4 now. As we need to have a proper discussion about the logging in retry.


Input column in chat excel is disabled

The input column in the chat excel is disabled, and can't input any chat.
To Reproduce
Steps to reproduce the behavior:

Go to chat excel and upload an excel file
The chat column is disabled

Hi, @JosnaKurian, it has been fixed in 514, please try again.


Terms of Use

Much like  #13, I notice a lack of terms of use and license. May I have permission to use your api as an endpoint for a discord bot I'm creating? I'll credit the data of course.

Yeah, you can use the API and such. Just added the license, sorry for the long response!


Load and run visual plugin (system) on GUI side

🎉 New feature
Related PR: https://github.com/ignitionrobotics/ign-rendering/pull/533 - Needed for running the shader_param.sdf demo world added in this PR, and also for shader_param_system integration test to pass.
Summary
This PR allows visual plugins to be loaded on the gui side.
Changes include:

adding a VisualPlugin component to store the <plugin> sdf element

this component is sent from server to gui

adding a VisualPlugin gui event for GzSceneManager to notify GuiRunner that a new visual plugin is detected
extending GuiRunner to load and run systems (visual plugin)

this can probably be refactored later to avoid duplicate code with SimulationRunner

adding a SceneUpdate event that systems can connect to on both the server and gui side for rendering
adding a ShaderParam visual plugin (ported from gazebo-classic) to test the whole pipeline

Test it
Run the shader_param.sdf world - this world contains a sphere model with the new ShaderParam system that modifies its appearance using custom shaders. Requires https://github.com/ignitionrobotics/ign-rendering/pull/533
ign gazebo -v 4  -r shader_param.sdf

Checklist

[x] Signed all commits for DCO
[x] Added tests
[x] Added example and/or tutorial
[x] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

I believe this closes https://github.com/ignitionrobotics/ign-gazebo/issues/657

@iche033 - just checked and that this works for Metal with https://github.com/ignitionrobotics/ign-rendering/pull/554 🙂

Here are the metal shaders:
deform_fs.metal.zip
deform_vs.metal.zip

oh nice, looks like you forward ported this to Garden? I can update the shader param system to read metal shaders, e.g. something like:
<shader launguage="glsl">
  <vertex>path_to_vertex_shader.glsl</vertex>
  <fragment>path_to_fragment_shader.glsl</fragment>
</shader>
<shader launguage="metal">
  <vertex>path_to_vertex_shader.metal</vertex>
  <fragment>path_to_fragment_shader.metal</fragment>
</shader>

This just lets user specify multiple shader files and the system will then select the best shader to use depending on the platform. How does that look?

Implemented the above idea in the shader_param_metal branch. I'll create a PR once I can upload metal shaders to Fuel.
I've also created https://github.com/ignitionrobotics/ign-rendering/pull/558 to accomodate cases like https://github.com/ignitionrobotics/ign-rendering/pull/554 where extra params (shader_reflection_pair_hint) are needed for metal but not glsl

oh nice, looks like you forward ported this to Garden?

Yes, just cherry-picked it in. Most of the Metal support needs Garden now because of various ABI breaking interface changes.
Your proposed changes for multi-shader language support and extra param checks look good!

@iche033 Just noticed you've already done all the texture changes in #1310! I should have looked more carefully...

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-03-01-citadel-edifice-fortress/1313/1

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/using-a-custom-hlms-with-ignition-rendering/1315/2


OAuth Server

we have an Oauth server, need to get it production ready.

I think this is done, or as done as it's going to get (for now) we may change what data gets returned, and tweak some things -- @rschenk @awead  thoughts?


runtime/metrics: update documentation w.r.t. stack_sys

What version of Go are you using (go version)?
$ go version
go1.20

Issue
The documentation related to stack memory in MemStats says:
        // StackInuse is bytes in stack spans.
	//
	// In-use stack spans have at least one stack in them. These
	// spans can only be used for other stacks of the same size.
	//
	// There is no StackIdle because unused stack spans are
	// returned to the heap (and hence counted toward HeapIdle).
	StackInuse [uint64](https://pkg.go.dev/builtin#uint64)

	// StackSys is bytes of stack memory obtained from the OS.
	//
	// StackSys is StackInuse, plus any memory obtained directly
	// from the OS for OS thread stacks (which should be minimal).
	StackSys [uint64](https://pkg.go.dev/builtin#uint64)

And, in runtime/metrics, there is also:
/memory/classes/heap/stacks:bytes
	Memory allocated from the heap that is reserved for stack
	space, whether or not it is currently in-use.

/memory/classes/os-stacks:bytes
	Stack memory allocated by the underlying operating system.

But this is at best confusing and at worst incorrect. Quoting @mknyszek from a recent conversation:

if a program is pure Go, we use newosproc0 to allocate the first stack. this accounts to stacks_sys, which funnels into StackSys. all other system stacks are allocated from the same pool as goroutine stacks (basically, StackInuse). 8192 bytes.
if a program is cgo, then we use _cgo_sys_thread_create for the first thread which is the platform C library thread creation. we use this for every new stack, too. we don't control the size of those stacks at all, and we also don't account for them AFAICT. this is how we ensure that calls into C are fine: all system stacks are basically provided by pthreads.

This means:

The behaviour is different between pure Go and cgo programs, which should be mentioned.
Even for pure Go programs, stacks for syscalls are generally allocated from stack_inuse (in cgo programs this is not tracked at all).

This different behaviour should be called out in the docs.

@golang/runtime


I wonder what is the max or min image size in autokeras?

I want to deal a hyperspectral image data, and it's shape with (1800,) (1800, 103) for train data and train label, when I use the autokeras.image_supervised.ImageClassifier() method, I find that the input size must a 4 dimension format, so I reshape the train data with shape (1800, 103, 1, 1), and I got this error:
Traceback (most recent call last):
  File "E:/自主进化/Code/AutoKeras/HSI.py", line 50, in <module>
    clf.fit(train_data, train_label, time_limit=12 * 60 * 60)
  File "C:\Users\lab103\Anaconda3\lib\site-packages\autokeras\image_supervised.py", line 239, in fit
    run_searcher_once(train_data, test_data, self.path, int(time_remain))
  File "C:\Users\lab103\Anaconda3\lib\site-packages\autokeras\image_supervised.py", line 40, in run_searcher_once
    searcher.search(train_data, test_data, timeout)
  File "C:\Users\lab103\Anaconda3\lib\site-packages\autokeras\search.py", line 222, in search
    raise e
  File "C:\Users\lab103\Anaconda3\lib\site-packages\autokeras\search.py", line 204, in search
    metric_value, loss, graph = train_results.get(timeout=remaining_time)[0]
  File "C:\Users\lab103\Anaconda3\lib\multiprocessing\pool.py", line 644, in get
    raise self._value
RuntimeError: Given input size: (64x103x1). Calculated output size: (64x51x0). Output size is too small at c:\programdata\miniconda3\conda-bld\pytorch_1533090623466\work\aten\src\thnn\generic/SpatialDilatedMaxPooling.c:67

Process finished with exit code 1

maybe it caused because the third dimension is too small, so I wonder what's the image size boundary for autokeras when I use it?

If you use PyTorch the input to a ConvNet should be shaped like [batch, channels, height, width].
Keras offers two modes: "channels_last" corresponds to inputs with shape (batch, height, width, channels) while "channels_first" corresponds to inputs with shape (batch, channels, height, width)" https://keras.io/layers/convolutional/
My take on what sets the max size of an image is that your batch of images should fit into your RAM.

which I mean is that if I use a pic with shape 2828, autokeras may normal, but when I take a pic with shape 281, it out put the output size is too small, so I wander what is the max or min pic shape for autokeras, thank you.

We will be supporting any size image in the future.
Currently, we don't have an upper limit.
Only a lower limit which is image still larger than 1 after three pooling layers with stride 2.

the statement "larger than 1 after three pooling layers with stride 2" means the low limit of image dimension is 16 ?  Is there a exact number for this question ? or the low limit could be changed somewhere in the source code ? @jhfjhfj1


estimator for tf serving error '_lower_using_switch_merge'

System information

OS Platform and Distribution :
win10 subsystem Ubuntu 18.04 LTS
TensorFlow version: 2.0
Python version: 3.6

Describe the current behavior
when I use tensorflow_model_server, I get a error:
failed: Internal: Node {{node dnn/zero_fraction/cond}} of type StatelessIf has '_lower_using_switch_merge' attr set but it does not support lowering.

Code to reproduce the issue
import pandas as pd
from sklearn import datasets
import tensorflow as tf
import itertools

COLUMNS = ["crim", "zn", "indus", "nox", "rm", "age",
           "dis", "tax", "ptratio", "medv"]

training_set = pd.read_csv(r'./data/boston/boston_train.csv', skipinitialspace=True, skiprows=1, names=COLUMNS)
test_set = pd.read_csv(r'./data/boston/boston_test.csv', skipinitialspace=True, skiprows=1, names=COLUMNS)
predict_set = pd.read_csv(r'./data/boston/boston_predict.csv', skipinitialspace=True, skiprows=1, names=COLUMNS)

FEATURES = ["crim", "zn", "indus", "nox", "rm",
                 "age", "dis", "tax", "ptratio"]
LABEL = "medv"

feature_cols = [tf.feature_column.numeric_column(k) for k in FEATURES]

estimator = tf.estimator.LinearRegressor(
        feature_columns=feature_cols
)

def get_input_fn(data_set, num_epoch=None, n_batch=128, shuffle=True):
    return tf.compat.v1.estimator.inputs.pandas_input_fn(
        x=pd.DataFrame({k: data_set[k].values for k in FEATURES}),
        y=pd.Series(data_set[LABEL].values),
        batch_size=n_batch,
        num_epochs=num_epoch,
        shuffle=shuffle
    )

estimator.train(input_fn=get_input_fn(training_set, num_epoch=None, n_batch=128, shuffle=False), steps=1000)

feature_spec = tf.feature_column.make_parse_example_spec(feature_cols)
erving_input_receiver_fn = tf.estimator.export.build_parsing_serving_input_receiver_fn(feature_spec)
estimator.export_saved_model(r'./saved/boston', erving_input_receiver_fn)

after that, when I try tf serving:
tensorflow_model_server --rest_api_port=8501 --model_name=saved_model --model_base_path='/mnt/c/Study/tensorflow/saved/boston'

I get this error:
 SavedModel load for tags { serve }; Status: fail. Took 59577 microseconds.
Loading servable: {name: saved_model version:<PHONE_NUMBER>} failed: Internal: Node {{node zero_fraction/total_zero/zero_count/else/_1/zero_fraction/cond}} of type StatelessIf has '_lower_using_switch_merge' attr set but it does not support lowering.

I face the same problem when serving for model from TF2.0 ,  and I found that tensorflow/serving:1.15.0-rc2 (docker image) can fix this problem .
PS: At this time , tensorflow/serving:latest == tensorflow/serving:1.14.0 , so you should try specific tag 1.15.0-rc2.
Hope this help

@siyi8088
This issue is more suitable for TensorFlow Serving repo. Please post it on Serving repo from here. Thanks!

@siyi8088
I am closing the issue here.Please, post the issue in serving repo and we can track there. Thanks!


Add list constructor expression

Description:
Refer: https://ballerina.io/spec/lang/2020R1/#list-constructor-expr

Added with #22936


new(content/en/docs): running Falco in a Kind cluster

Signed-off-by: Leonardo Grasso<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind transalation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation
What this PR does / why we need it:
This PR adds instructions on how to properly run Falco in a Kind cluster.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign radhikapc
You can assign the PR to them by writing /assign @radhikapc in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Minor suggestion. Otherwise great.

@radhikapc Thank you for your suggestion.
Done and also rebased to the current master.

LGTM label has been added.  Git tree hash: 48e4776108de9b9a620576fff7b3dd06087979dc

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido, radhikapc
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido,radhikapc]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


chore: Create LICENSE

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

1.0.0
GitHub release

Your semantic-release bot :package::rocket: