Simplify end-to-end deployment

Impact of the new feature
In order to enable end-to-end deployment procedure developers should be able to setup any WMCore service on their laptop, VM, k8s or any other hardware. Then, with the help of CI/CD pipeline (discussed https://github.com/dmwm/WMCore/pull/10911) they should develop and run their code locally without any deployment procedure (RPM based or otherwise). Finally, they should tag their code and rely on CI/CD pipeline to build a release, run tests, create new image and deploy it on k8s.
To do that, we need to adjust configuration files to avoid hard-coded CMSWEB deployment directory structure and allow developers to use environment variables to define location of auxiliary files for WMCore services. For instance, all config files has the following:
ROOTDIR = __file__.rsplit('/', 3)[0]
...
sys.path.append(path.join(ROOTDIR, 'auth/reqmgr2ms'))
...
sec.key_file = "%s/auth/wmcore-auth/header-auth-key" % ROOTDIR
...
heartbeatMonitor.log_file = '%s/logs/reqmgr2ms/heartbeatMonitor_output-%s-%s.log' % (__file__.rsplit('/', 4)[0],
                                                                                HOST.split('.', 1)[0],
                                                                                time.strftime("%Y%m%d"))
...

This implies pre-defined directory structure is required by the configuration file itself, and there is no way to change it in local setup since all paths are bound to location of configuration file. Moreover, plenty of logic required to have specific paths, directories, files to run the service.
Is your feature request related to a problem? Please describe.
I should be able to setup and run any WMCore service on my laptop without enforcing me to define CMSWEB directory structure environment. For example, if I want to run MS service, I should only define few environment variables required for this service, I should be able to choose whatever flat or nested directory structure I want to use, via local or system wise directories, and I should be able to go. For instance, I may to choose my logs to go to /tmp area, and my auth key file to point to my $HOME/secrets/auth-file, etc.
Describe the solution you'd like
Instead of using pre-define directory structure in config files we should identify common set of environment variables and use them. For instance, the example of config structure above I'll change as following:
# instead of
# ROOTDIR = __file__.rsplit('/', 3)[0]
# rely on env variable
ROOTDIR=os.environ.get('WMCORE_ROOTDIR')
if ROOTDIR == "":
   raise Exception("WMCORE_ROOTDIR is not set")
if not os.path.isdir(ROOTDIR):
   raise Exception("WMCORE_ROOTDIR=%s directory does not exist" % ROOTDIR)
...
# this may stay or relaxed to use another WMCORE_AUTHDIR env
# sys.path.append(path.join(ROOTDIR, 'auth/reqmgr2ms'))
auth_path = os.environ.get("WMCORE_AUTHDIR")
if auth_path and os.path.isdir(auth_path):
   sys.path.append(auth_path)
else:
   raise Exception("WMCORE_AUTHDIR is not set")
if not os.path.isdir(auth_path):
   raise Exception("auth_path=%s does not exist" % auth_path)
...
# instead of hard-coded auth key file in pre-defined location
# sec.key_file = "%s/auth/wmcore-auth/header-auth-key" % ROOTDIR
# we may use
key_file = os.getenv("WMCORE_AUTH_KEY")
if not os.path.isfile(key_file):
    raise Exception("key_file=%s does not exist" % key_file)
...
# instead of hard-coded pre-define location
#heartbeatMonitor.log_file = '%s/logs/reqmgr2ms/heartbeatMonitor_output-%s-%s.log' % (__file__.rsplit('/', 4)[0],
#                                                                                HOST.split('.', 1)[0],
#                                                                                time.strftime("%Y%m%d"))
# we should use
log_file = os.environ.get("WMCORE_HEARTBEAT_LOGDIR")
if log_dir and os.path.isdir(log_dir):
    heartbeatMonitor.log_file = log_dir
else:
   raise Exception("WMCORE_HEARTBEAT_LOGDIR is not set")
if not os.path.isdir(log_dir):
   raise Exception("log_dir=%s does not exist" % log_dir)
...

Such definitions will support both CMSWEB and user defined environment, for CMSWEB we will define all env variable to point to existing dir structure, while for local environment user may define his/her own locations. As such, the config files will provide universal configuration which will work both on CMSWEB and local environments. Moreover, such flexibility and dependence on env variables will significantly simplify CI/CD pipepine and k8s deployment and will allow to decouple deployment of WMCore services from CMSWEB deployment procedure.
Describe alternatives you've considered
The config files should be adjusted to provide all details of configuration logic required by WMCore services. It can be achieved in a different way. For instance, you provide additional shell setup.sh file which will define all environment variables, etc.
Additional context
This proposal will simplify CI/CD pipeline development and local deployment procedure, see https://github.com/dmwm/WMCore/pull/10911

@klannon , @amaltaro , @todor-ivanov , @goughes I consider this very important step towards end-to-end deployment procedure and automation via CI/CD. Please consider it carefully. I tried to provide reason why current configuration significantly constrain our ability to use local setup (and therefore introduced large latencies in dev cycle). For instance, if I need to work on issue A, which only necessary to fix some corner part of WMCore web framework, I should not be forced to install RPMs on VM with cmsweb deployment tools. I should be able to run WMCore service locally, test my feature, tag my change and get my build automatically. I successfully applied such practice in many other services which does not inherit constraints defined by WMCore structure and configuration files. I propose to rethink how configuration files should be structure and first step is to decouple them from CMSWEB pre-defined structure (since this is no longer required within docker image) and will simplify working with local setup.


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Fri Mar 25 2016 21:32:57 GMT-0500 (Central Daylight Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\Jaden\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\Jaden\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #470


Remove excessive trailing whitespace from Region, Territory.

Related to issue #12 but I can't work out how to link a pull request to an issue.


[test] Use target-swift-frontend for x-compile tests. NFC

(cherry picked from commit 139293d6f834b95cbe42aa91c6b4b399131aab75)
rdar://34868792

@swift-ci please smoke test

@swift-ci please test


Allow noscript fallback to be disabled

Fixes https://github.com/jekyll/jekyll-gist/issues/17.
Fixes https://github.com/jekyll/jekyll-gist/issues/26.

@benbalter Great! Just needs a test. Thank you!

Great! Just needs a test

Done (with docs too!)

LGTM. Once CI gives the :green_heart:, we can merge.

@benbalter To fix CI, gist must be defined in your block like output.
let(:gist) { "mattr-/24081a1d93d2898ecf0f" } # choose a different ID if you want :)


fix: Connection Instability with socketTimeout Parameter

This PR is fixing https://github.com/redis/ioredis/issues/1919 issue (When setting the socketTimeout parameter to a non-zero value, the Redis connection becomes unstable after startup)

The problem is that the auth command is sent before DataHandler is created on connection startup. This ruins the correct listener orders--the parser listener shall execute before the other ones.
To avoid this, I replaced on with prependListener to ensure the correct order.

Sure, no problem

I just added the small unit for the changed subscribe logic, but I also noted that we missed commits that add "socketTimeout" to the lib. It happened because the socketTimeout was added after the fork was created

https://github.com/redis/ioredis/commit/673ac77d9d88bd461110da7b4a8b2b98fb45f845
https://github.com/redis/ioredis/commit/558497cba8dc7487c06c7765ddbe12b479bd9b9b

I can propose merging this PR (if you don't mind), and later, we will copy these missed commits to the repo. @mcollina
is it a good way to handle this?

Can you open a separate PR with those two commits? We'll land that before this.

The test does not really cover the use case of the bug. Can you add a test that actually reproduce the loop?

A functional test for socketTimeout was just added
@mcollina, could you check it and provide feedback please?


Confusion over license

Hello,
I am confused about the license for VerneMQ...
The EULA says
"You must not, directly or indirectly: (a) modify, translate, adapt, or otherwise create derivative works or improvements, whether or not patentable, of the Software or any part thereof; (b) reverse engineer, disassemble, decompile, decode, or otherwise attempt to derive or gain access to the source code of the Software or any part thereof; (c) remove, delete, alter, or obscure any trademarks or any copyright, trademark, patent, or other intellectual property or proprietary rights notices provided on or with the Software, including any copy thereof; (d) rent, lease, lend, sell, sublicense, assign, distribute, publish, transfer, or otherwise make available the Software, or any features or functionality of the Software, to any Third Party for any reason; (e) use the Software in violation of any law, regulation, or rule; or (f) use the Software for purposes of competitive analysis of the Software, the development of a competing software product or service, or any other purpose that is to the Octavo Labs’s commercial disadvantage."
However, the source code is open source and published to Github and the EULA directly contradicts the Apache 2 License in the Github repository that states that...
"Redistribution. You may reproduce and distribute copies of the
Work or Derivative Works thereof in any medium, with or without
modifications, and in Source or Object form, provided that You
meet the following conditions:
(a) You must give any other recipients of the Work or
Derivative Works a copy of this License; and
(b) You must cause any modified files to carry prominent notices
stating that You changed the files; and
(c) You must retain, in the Source form of any Derivative Works
that You distribute, all copyright, patent, trademark, and
attribution notices from the Source form of the Work,
excluding those notices that do not pertain to any part of
the Derivative Works; and
(d) If the Work includes a "NOTICE" text file as part of its
distribution, then any Derivative Works that You distribute must
include a readable copy of the attribution notices contained
within such NOTICE file, excluding those notices that do not
pertain to any part of the Derivative Works, in at least one
of the following places: within a NOTICE text file distributed
as part of the Derivative Works; within the Source form or
documentation, if provided along with the Derivative Works; or,
within a display generated by the Derivative Works, if and
wherever such third-party notices normally appear. The contents
of the NOTICE file are for informational purposes only and
do not modify the License. You may add Your own attribution
notices within Derivative Works that You distribute, alongside
or as an addendum to the NOTICE text from the Work, provided
that such additional attribution notices cannot be construed
as modifying the License.
You may add Your own copyright statement to Your modifications and
may provide additional or different license terms and conditions
for use, reproduction, or distribution of Your modifications, or
for any such Derivative Works as a whole, provided Your use,
reproduction, and distribution of the Work otherwise complies with
the conditions stated in this License."
I am massively confused here if I build a SaaS product using VerneMQ the EULA pretty much says I am not allowed to sell it or market it as it, in which case I need to ditch VerneMQ and use a properly open-sourced solution, but the Apache license for VerneMQ suggests I am free to modify it and make it available to others in whichever way as long as I retain the original attribution and document changes as per the above. Please can somebody clarify how this works and which license takes precedent?
Best Regards,
Rob

The EULA is for the compiled Linux packages and our Docker images, starting from Release 1.10.0.
(we don't build those packages 'for free' any more).
If you don't want to use those, you can compile/build everything yourself. Source code is open source, Apache2, don't worry.

Ok, so providing I compile from the source I am all good to assume the EULA doesn't apply to me only the terms of the Apache 2.0 license?
In this case, does anyone in the community want to start packaging a community release of this from the Apache 2.0 licensed source code with me?

Yes, to your first question. Note that you absolutely can build commercial solutions with our compiled packages.
This is a somewhat similar approach to what Chef.io does (obviously in a much bigger context): https://blog.chef.io/chef-software-announces-the-enterprise-automation-stack/

Yes, to your first question. Note that you absolutely can build commercial solutions with our compiled packages.
This is a somewhat similar approach to what Chef.io does (obviously in a much bigger context): https://blog.chef.io/chef-software-announces-the-enterprise-automation-stack/

Could I use the build image from source code in commercial product?
What is the difference between using pre-build image and build it myself? any different features?

You can use everything that is open sourced here, build it and use it commercially.
You can also download & use our pre-compiled packages (with the EULA addition), and use them commercially for a usage fee.

This what I understand so far, but my question again is there any difference that requires a usage fee?

Functionally (MQTT spec etc), no difference currently. The work & optimization we put in. It allows up to keep VerneMQ open source as is. The alternative is offering essential features as closed source only (clustering, persistence), like others do.

@ahmedwahdan to my understanding there is no difference in the functionality I think what is happening here is...

Octavo Labs created VerneMQ and wanted to offer an opensource product OR are obliged to release their code under some opensource license.
Octavo Labs take the opensource project at some point in time and create a package, perform some bug fix maybe optimisation/validation and release the packages.
Octavo Labs require in the EULA if you use said packages for commercial reasons you pay them some fee.

The problem is there are no community packages so you either must:

Pay Octavo Labs if you wish to use their packages for a commercial project.
Build the project from source and validate it works/test it yourself and then Apache 2 license applies and you do not need to pay Octavo anything even for commercial use.

To make it easier for people using the Opensource Apache 2 licensed option what we could do with is:

Creating CI mechanism to package for .deb and .rpm automatically from the repository.
Create CI to deploy it to some gate and test the packages against common LTS operating systems.
Publish the community packages somewhere which have passed the validation.

This way people can then choose either:

Octavo Labs packages which are paid for commercial use.
The community packages which are free and means the user doesn't need to compile from scratch / configure SystemD etc...
Compile from scratch and configure how they wish.

If there are any keen people out there who'd like to help do some community packaging for this it would be great.
I don't mind companies charging for support and consultancy for users of opensource projects but I have to admit I am not keen on companies making the opensource / community release of their product harder to use than the commercial release just for the sake of converting sales. It doesn't really seem in the spirit of the open-source movement.

@robputt796
I got it, and agree with you regarding the point of restricting the packages for sales.
I don't have much experience but I for sure will help in case there is an initiative to create community packages.

@ahmedwahdan which OS do you target? I am currently targeting Ubuntu 20.04 for my project so I'll probably make some deb packages for myself and upload them somewhere if others want to use them, I am hoping you are using some RedHat variant and are thinking of making RPM packages... 📦

l believe my colleague will build a docker image.
Please share with me the details of the package you will build.

l believe my colleague will build a docker image.
Please share with me the details of the package you will build.

Did you build it? Is it published somewhere? I'm asking because I'm about to create one myself, but I don't want to create an unnecessary duplicate.


Log bot; include Heals plz

LOG bot is not reporting Heals after combat, Plz include!

Not in the bot yet, closed.


Can't create instance variable for "country-name"

I'm having a problem when trying to parse a page with a tag og:country-name:
*** NameError Exception: `@country-name' is not allowed as an instance variable name

(example: https://www.kao.co.jp/liese/lineup/sugami/)
Ruby doesn't support hyphen when creating a instance variable, so should we change the code to snake_case tag names before trying to instance?

Hi,
Thanks for taking the time to report this behaviour to me.
I see you came up with a solution on your fork.
Are you willing to create a proper pull request  for your patch along with some unit tests?
We could merge it in the official repository and release a new version of the gem.
Kind regards,

@jcouture Yes, I wasn't sure if "snake_casing" was the desired solution so I decided to fork and make a quick solution to solve temporarily.
But we ended up using other gem as we were having other issues with other website addresses too. (e.g. missing required attributes raise exception)

@wkurosawa, thanks for sharing your experience with OGP.
What could be tweaked in OGP that would allow you to use it instead of the other solution?
Regards,


feat: event but impl

Basic implementation for rabbitMq that supports sending simple messages (bytes payload) into predefined queue.
We using default callbacks that should be replaced with correct ones in next updates (maybe(not sure(meeeh)))
Also until entire "notification" feature is done we could use Mock service that will not require any RabbitMQ connection.  New env variable should be empty.

https://github.com/tikitko/blog-deploy/pull/2 merge only after this one


fix document name uniqueness, errors UI

Hi, I'm from upwork https://www.upwork.com/freelancers/~01e03c5a9f3b993057
Here is the fix for #251 issue. I've decided to make a PR directly without awaiting your acceptance of my proposal.
If this solution works for you, I would be grateful if you leave a positive feedback for my profile
Thank you!

That travis errors don't related with my code. They were before already

That travis errors don't relate with my code. They were before already

Yes don't worry we are going to address that in future. Needless to say thanks for the patch @slava-nikulin .

@slava-nikulin - I tested your patch. What we need is as follows :

Continue to support /docs/name-of-document.
But additionally also have the ability to create /docs/category_name/name-of-document.

We would love to see your PR for supporting multiple slugs. Default slug will be  /docs/name-of-document for all unique documents. Whenever there is a name collision, we will offer  /docs/category-name/name-of-document. We could validate that this is not a unique name and offer to create this slug instead. Feel free to propose a checkbox in the UI admin if needed. This would support #251 comprehensively.

@deekoder that is quite difficult task, not trivial at least, but I handled it. Now user will see popup if document name meets collision, where he can enter slug whatever he wants. Slug format changed to doc-name, cat-name/doc-name
Also admin user now have a possibility to change slug on edit document page

oh nice. Let me test your patch. Thanks.

@slava-nikulin  - Is it possible to resolve this merge conflicts on your end. I had to take another pr to hide/unhide documents. Thanks! Looking fwd to evaluating your fix.

@deekoder it is done

@slava-nikulin - It works perfectly!! There is one small thing that needs to be addressed. When a new document is created, please include the option to specify a slug name.  It shows on editing an existing document but not when creating a new document.I am really looking fwd to accepting your updated patch!

@deekoder, will do. But a little question first: is there any chance for me to be rewarded, since I've found this task on upwork and you're satisfied of fix and features I've made?

@slava-nikulin - We are an open source company that accepts contributions from the community. We do not have rewards per se for code contributions. Actually I am not really sure who set up the task on upwork. You could contact them directly to know further about it. We do recognize contributors and send them Minio swag. I am happy to send you swag :)

Splendid =)


ErrorMessage multiple errors

The Documentation of displaying multiple errors does not seem to work properly when using TypeScript
@ api#ErrorMessage
<ErrorMessage errors={errors} name="multipleErrorInput">
        {({ messages }) =>
          messages &&
          Object.entries(messages).map(([type, message]) => (

{message}

          ))
        }
      </ErrorMessage>

Error:
Property 'children' does not exist on type 'IntrinsicAttributes & { as?: undefined; errors?: DeepMap<inputFields, FieldError>; name: string; message?: string; render?: (data: { ...; }) => ReactNode; } & Pick<...>
solution:
following npm packages documentation
@ npm/hookform
<ErrorMessage
        errors={errors}
        name="multipleErrorInput"
        render={({ messages }) =>
          messages &&
          Object.entries(messages).map(([type, message]) => (

{message}

          ))
        }
      />

I'll investigate whats going on here, and later today I'll see if I can update the example with some TS code and a CodeSandbox. Thank you for bringing this up!

Take a look at the CodeSandbox's created in https://github.com/react-hook-form/documentation/pull/354. I believe they provide a working example of TS and ErrorMessage!

Closing this issue, but feel free to re-open if you have a specific CodeSandbox detailing the error! Thank You!


[Event Request] Codeunit 99000809 - CheckMultiLevelStructure

In function CheckMultiLevelStructure we need a publisher before validation of Quantity
ReqLine3."Demand Date" := PlanningComp."Due Date";

**// ==> NEW
OnBeforeValidateQuantityForPlanningComponent(ReqLine3,PlanningComp);
// <== NEW**

ReqLine3.VALIDATE(Quantity,PlanningComp."Expected Quantity");

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Need to add Package to Rowan Project Changes list

Currently the lists are Type, Class, Selector, Detail.
Don't think it's a must have for alpha.

Moving a method between packages in rowan-hybrid (as of 3.2.12) shows up as an addition plus a removal, which is not very helpful.  This might be worth doing for end 2022.

Package names are shown. Added a few tests also - one for moving a method between packages.


[BE]create new table for skills{id, name of the skills}, add 20 real skills and rename our current table to graduate-skills

Story

As a developer

I Want to create a new table for skills, add 20 real skills and  rename our current table to graduate-skills

So That we can retrieve the data easily.

Acceptance Criteria

Given I am a developer

When I view a database
Then I should see three tables Graduates, Graduate-skills and skills.

Given  I am a developer

When  I view a skills table
Then I should see real programming skills.

// /graduates GET
[{id: 1, name: "Bob", skills: [{id: 1, name: "React"}, {id: 3, "JavaScript"}]}]
// /graduate:id GET
{id: 1, name: "Bob", skills: [{id: 1, name: "React"}, {id: 3, "JavaScript"}, summary: "I have 6 months experience in React"]}
// /skills GET
[{id: 1, name: "React"}, {id: 2, "CSS"}, {id: 3, "JavaScript"}]

Hi Guys my task is done, I modified our current table "skills"  to ==> graduate_skills and I created a new table named "skills" with the data "id" and "skills", with 20 programming language skills.

Hi Franklin, can you push the changes in the branch, I cannot see your changes in the branch.

Table graduate_skills is not ready.

You are right simeon, I forgot that, that is not on the issue label, I will modify that ASAP


kubectl exec not working while proxy configured

Describe the bug
I have few clusters which is only allow proxy IP. When I perform Pod shell. I got
error: unable to upgrade connection: error dialing backend: read tcp :58315->:443: read: connection reset by peer
Terminal will auto-close in 15 seconds ...
To Reproduce
Steps to reproduce the behavior:

configure proxy
go to pod
click Pod shell
See error

Expected behavior
remote exec shell inside Pod
Environment (please complete the following information):

Lens Version: 3.6.4
OS: OSX
Installation method: auto upgrade

same issue in version 3.6.5

Where did you configure the proxy?

I configured proxy in the settings. When I new cluster, I add it.

it working in version 3.6.6.
Thanks


1s limit on reject output

Hi there,
I have a config that I am using that is fairly high throughput using pubsub:
input:
  gcp_pubsub:
    project: myproj
    subscription: mysubscription
    max_outstanding_messages: 40000
    batching:
      count: 500
      period: "5s"
processors:
- type: somethingThatMightFail
output:
  switch:
    retry_until_success: false
    cases:
    - check: errored()
      output:
        reject: |
          project=${! meta("projectId") } error=${! error() }
    - output:
        drop: {}

and sometimes my processor will error out (eg if the database it is talking to is down). When it does error, the error does correctly get applied to the parts and it will fall down into the reject output - but when this happens, I get exactly 1 message/s throughput. I would expect it to nack pubsub and continue along as fast as it can. Where is the 1s coming from here? Note it is still 1s even if I change/remove the batching from the input. I looked at the code for the reject output and its literally just return errors.New() so I assume it has something to do with errors propagating to inputs? I have reproduced this 1s limit in your WASM lab with the reject output.
Is there a better way I can be propagating these nacks while still performing the rest of my work as fast as I can?
Thanks!

Hey @iluminae, this is actually being caused at the input level https://github.com/Jeffail/benthos/blob/master/lib/input/async_reader.go#L168, basically it's a back pressure mechanism that prevents a busy loop when messages that will never succeed are being attempted with an input that doesn't propagate acknowledgements upstream. In your case that mechanism isn't required since we're pushing those failed messages upstream.
It should be possible to disable that mechanism selectively for inputs that support upstream acks and then we'll unlock the throughput for rejected messages.

Sweet, that would feel better for pubsub for sure. Probably will cost me more if im spinning on errors but im ok with that.

Needs a bit of manual testing to go along with the integration test suites but I believe this is fixed with: https://github.com/Jeffail/benthos/commit/e1f2faef07877676563df41c9781efd1dcd761bc
It turns out I was wrong and that there's a general purpose rate limit for all output types. I've removed that and added a rate limiter at the input level that specifically only applies to inputs that do not push nacks upstream, this also has the advantage of rate limits applying to the individual failing message and not to the general output.

There was a small regression but it looks good in local tests now.


CSP violation error in react-error-overlay

The following line produces CSP (Content Security Policy) errors when react-error-overlay wants to display an JS error and having a script-src 'self' CSP policy.
https://github.com/facebook/create-react-app/blob/36ce5786f51f3c793d15178d530d2eb1ae26427a/packages/react-error-overlay/src/index.js#L150
Atm. I use (a version dependent) sha256-IxIMJ2x2Rnqbl5UsK1uoQOZYtdDuuWoUIMsikFzsDwk= CSP policy as a workaround.
The shown error: "Refused to execute inline script because it violates the following Content Security Policy directive ..."
Would it be possible to include the iframeScript via src reference instead of an inline script to solve this issue?
Related: #4822

Hmm... please don't close.


Error: pip install -r requirements.txt

"pip install -r requirements.txt" causes following error.
$ pip install -r requirements.txt
Exception:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/pip/basecommand.py", line 122, in main
    status = self.run(options, args)
  File "/usr/lib/python2.7/dist-packages/pip/commands/install.py", line 262, in run
    for req in parse_requirements(filename, finder=finder, options=options, session=session):
  File "/usr/lib/python2.7/dist-packages/pip/req.py", line 1632, in parse_requirements
    req = InstallRequirement.from_line(line, comes_from, prereleases=getattr(options, "pre", None))
  File "/usr/lib/python2.7/dist-packages/pip/req.py", line 173, in from_line
    return cls(req, comes_from, url=url, prereleases=prereleases)
  File "/usr/lib/python2.7/dist-packages/pip/req.py", line 71, in __init__
    req = pkg_resources.Requirement.parse(req)
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2667, in parse
    reqs = list(parse_requirements(s))
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2605, in parse_requirements
    line, p, specs = scan_list(VERSION,LINE_END,line,p,(1,2),"version spec")
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2583, in scan_list
    "Expected ',' or end-of-list in",line,"at",line[p:]
ValueError: ("Expected ',' or end-of-list in", "py2-ipaddress>=3.4.1 ; python_version < '3.2'", 'at', " ; python_version < '3.2'")

Storing debug log for failure in /root/.pip/pip.log

Environment
$ uname -a
Linux fb2423ed34ab 4.9.12-moby #1 SMP Tue Feb 28 12:11:36 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
$ python --version
Python 2.7.6

seems ur pip does not support the range grammar in requirements.txt
can u try with upgrading it?

Thank you for the answer, but it still doesn't work for me.
# pip install --upgrade pip
Downloading/unpacking pip from https://pypi.python.org/packages/b6/ac/7015eb97dc749283ffdec1c3a88ddb8ae03b8fad0f0e611408f196358da3/pip-9.0.1-py2.py3-none-any.whl#md5=297dbd16ef53bcef0447d245815f5144
  Downloading pip-9.0.1-py2.py3-none-any.whl (1.3MB): 1.3MB downloaded
Installing collected packages: pip
  Found existing installation: pip 1.5.4
    Not uninstalling pip at /usr/lib/python2.7/dist-packages, owned by OS
Successfully installed pip
Cleaning up...

and...
# pip install -r requirements.txt
Exception:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/pip/basecommand.py", line 122, in main
    status = self.run(options, args)
  File "/usr/lib/python2.7/dist-packages/pip/commands/install.py", line 262, in run
    for req in parse_requirements(filename, finder=finder, options=options, session=session):
  File "/usr/lib/python2.7/dist-packages/pip/req.py", line 1632, in parse_requirements
    req = InstallRequirement.from_line(line, comes_from, prereleases=getattr(options, "pre", None))
  File "/usr/lib/python2.7/dist-packages/pip/req.py", line 173, in from_line
    return cls(req, comes_from, url=url, prereleases=prereleases)
  File "/usr/lib/python2.7/dist-packages/pip/req.py", line 71, in __init__
    req = pkg_resources.Requirement.parse(req)
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2667, in parse
    reqs = list(parse_requirements(s))
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2605, in parse_requirements
    line, p, specs = scan_list(VERSION,LINE_END,line,p,(1,2),"version spec")
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2583, in scan_list
    "Expected ',' or end-of-list in",line,"at",line[p:]
ValueError: ("Expected ',' or end-of-list in", "py2-ipaddress>=3.4.1 ; python_version < '3.2'", 'at', " ; python_version < '3.2'")

Storing debug log for failure in /root/.pip/pip.log

can u show ur pip version, i guess ur default pip is still the os owned one, which is old version...
maybe u can use virtualenv to make a clean new env and install python3 there.


Consider replacing unwrap with expect for error handling

Volker made a good point about using expect() instead on unwrap() for handling errors.
Investigate whether all uses of unwrap() are correct and replace with expect() or some other mechanism if not.

The MIT effective rust examples (see: A brief interlude: unwrapping isn't evil) recommend only using unwrap():

in examples and quick and dirty code;
and when panicking indicates a bug in the program.

Clearly the library is currently using unwrap() in many more instances than those above.
Some common instances that require further consideration are drop() methods (i.e. destructors) and string conversion functions.

All unwrap() calls outside of examples and tests changed to either mapping CString errors or expect() calls.


Display bug in order creation box

I'm not sure if it is intended to do so, I know its a small detail :)
when activating market maker option and then switching form buy to sell tab the text in the button change inaccurately.

yes, in advanced mode, there could possibly be some issues like this UI rendering. thanks for reporting. will have a look soon.


RateRequest requires contact information for sender and receiver

Hello Tarek,
I am starting on DHL integration and trying to use your library.
I am currently getting an error 999 with every invocation of RateRequest call.
After combing through the reference documentation from DHL v1.6 I noticed that they mark contact person as "M" which I presume stands for the mandatory field.
Do you know if the contact person fields are mandatory in the API? Can you provide implementation if they are?
Thank you for publishing the library.
Ivan

Hi Ivan,
Thank you for doing the investigative work on this. However, the library is working as expected.
For Rate Queries, there is no need to send the Contact element at all (if you do send it for whatever reason, then the PersonName field becomes mandatory). For using the library though, you need to use the Address class in the MyDHLAPI_REST_Library.Objects.RateQuery namespace for rate queries.
Moreover, the PersonName field is marked as required for the Address class in the MyDHLAPI_REST_Library.Objects.Common namespace which should be used for shipment creation since the entire Contact object is required for shipment creation.


Error code -9 when creating a custom object detection dataset

Hello.
I'm attempting to create a custom data set but get "ERROR: error code -9". I haven't found an explanation for it. The create_train_db_db.log contains this; no mention of the error:
WARNING: log_file config option not found - no log file is being saved
2016-09-10 23:23:27 [INFO ] Created features db for stage train_db in train_db/features

Does anybody know what this error means?
Thanks,
Daryl.

This SO post seems relevant:
http://stackoverflow.com/a/18529453/2404152

Indeed - thanks.


Fix MultiSelect-within-Popover functionality

The implementer of this MultiSelect-within-Popover-with-desired-blur/closing-behavior unfortunately needs to implement a cheeky setTimeout for the functionality to work as expected. This is because the state of the component updates from the onBlur event resolution before the onClick event resolves, resulting in an impossible (at least through my many attempts) situation to achieve the desired effect.

Current dependencies on/for this PR:

main

PR #469   👈

This stack of pull requests is managed by Graphite.


Alexa Media Player support - this doesn't seem to work...

i have alexa_media_player, and these show up in StreamAssist as output media player devices, but they simply don't work. I can see in the logs that tts is generated, but i get no sound out of my alexa_media_player devices. Moreover, the 'targets' are empty:
2024-09-09 13:16:39.891 DEBUG (MainThread) [custom_components.stream_assist.core] event: PipelineEvent(type=<PipelineEventType.INTENT_END: 'intent-end'>, data={'intent_output': {'response': {'speech': {'plain': {'speech': "Captain, I am here to serve. What can I do for you? Please specify what you want.  I can't help you if I don't know what you need!", 'extra_data': None}}, 'card': {}, 'language': '*', 'response_type': 'action_done', 'data': {'targets': [], 'success': [], 'failed': []}}, 'conversation_id': '01J7BWK6652MBP1CG6MPH2H2SC'}}, timestamp='2024-09-09T17:16:39.890914+00:00')

Well, never mind... i can see why this shouldn't work... but probably shouldn't display alexa_media_player targets in the dropdown. Anyhow, i got this to work with broser_mod (my tablets are fire tablets, hence why i was using alexa_media_player). unfortunately, the camera stream is pretty unreliable (IP Camera app apk) on these devices, so although the wake word works at times, the intent is garbage... need to find a lighter weight way of getting reliable audio to the stream from a fire tablet... any suggestions?


DATEDIFF does consider time zone

The docs for DATEDIFF say:

DATEDIFF does not use a time zone offset component of startdate or enddate to calculate the return value.

This is demonstrably untrue. I am in the eastern US, so currently my timezone is UTC-4. On SQL Server 2017, executing SELECT datediff(hour, GETDATE(), SYSDATETIMEOFFSET()) yields 4.
The docs should be fixed to reflect this.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 340056a9-0593-4289-7b72-601ae718b05f
Version Independent ID: 1db8e288-8068-be96-b2f2-c8a2fe5a857a
Content: DATEDIFF (Transact-SQL) - SQL Server
Content Source: docs/t-sql/functions/datediff-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@WilliamAntonRohm Hi William, can you please process this Issue #2394.  Thanks.
Ideally this Issue will not need to be assigned to ms.author MikeRayMSFT.

@rdicroce -- Rich, thank you for your feedback.  I have updated the documentation and it should be live by tomorrow.
I am closing this issue 2394 now.  You are welcome to @ mention me for any followup.


Treat www-eu and www-us equally

AFAICT it does not make sense to only support www-us for the (shared) tlpserver.
Applying this should fix the broken www-eu.a.o URLs.

AFAICT it does not make sense to support www-{eu,us} for the (shared) tlpservers ;
www-{eu,us}.a.o URLs only appeared on generated pages ; those pages are now gone.

I see two problems with dropping www-eu.a.o and www.us.a.o:

www.[eu|us].a.o do not support SSL

Having introduced www-[eu|us] to deal with the SSL issue, it's not right to arbitrarily stop supporting the hostname.

I'm okay with supporting it in the short run.
In the long run, we'll likely figure out a way of not using either of these solutions, and go with a download.a.o instead.

Thanks. That has fixed the www-eu.a.o URL.
Note that it's not just about supporting downloads; the URLs can also be used for accessing the ASF website.


Add CONTRIBUTION.md File for Contribution Guidelines

Hello! @AbhiDiva96 ,
The contributing.md looks is not in a structured manner. Also some sections are missing. I can enhance it in a well structured manner with proper details and some additional sections also that will make the contributors easier to contribute. Please can you assign me this task?
Wanted to do the following changes :-

[ ] Enhance the project structure
[ ]  Add project structure section
[ ]  Add additional method of contributing through GitHub desktop section
[ ]  Add good coding practices section
[ ]  Add the help and support section
[ ]  Add the pull request section
[ ]  Add issue report process section

so please assign this issue to me under gssoc-ext and hacktoberfest

@a-n-u-vanguri
proceed..

Thanks! I will work on it.


[PE-D][Tester D] sdelete does not function properly

sdelete 2 does not work as shown in the screenshot, despite having a module at index 2.

Labels: severity.High type.FunctionalityBug
original: shogun187/ped#10

This bug has been fixed.


[SPARK-14579][SQL]Fix a race condition in StreamExecution.processAllAvailable

What changes were proposed in this pull request?
There is a race condition in StreamExecution.processAllAvailable. Here is an execution order to reproduce it.

Time
Thread 1
MicroBatchThread

1

dataAvailable in constructNextBatch returns false

2
addData(newData)

3
noNewData = false in  processAllAvailable

4

noNewData = true

5
noNewData is true so just return

The root cause is that checking dataAvailable and change noNewData to true is not atomic. This PR puts these two actions into synchronized to make sure they are atomic.
In addition, this PR also has the following changes:

Make committedOffsets and availableOffsets volatile to make sure they can be seen in other threads.
Copy the reference of availableOffsets to a local variable so that sourceStatuses can use a snapshot of availableOffsets.

How was this patch tested?
Existing unit tests.

Test build #55653 has started for PR 12339 at commit a49a46b.

cc @marmbrus

LGTM

Test build #55653 has finished for PR 12339 at commit a49a46b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55653/
Test FAILed.

retest this please

Test build #55659 has started for PR 12339 at commit a49a46b.

Test build #55659 has finished for PR 12339 at commit a49a46b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55659/
Test PASSed.

Thanks, merging to master.


Add cdl adapter

Link the Issue(s) this Pull Request is related to.
https://github.com/AcademySoftwareFoundation/OpenTimelineIO/issues/1366
Fixes #1366
Summarize your change.
This change adds a new adapter, 'cdl', which exports individual .cdl files to a user-provided directory for each clip/timeline event in an OTIO instance.
This change does not change anything in the OTIO core.
Please note: Due to the nature of this adapter exporting multiple files instead of a single file, the provided export method name write_to_file is slightly semantically confusing. The output provided to the method should be the directory (not a file) in which you wish to export the .cdl's to. Because of this, it is also necessary to directly provide the adapter_name to use this adapter (there is no file extension to read):
otio.adapters.write_to_file(timeline, '/your/cdl/output/directory', adapter_name='cdl')
Reference associated tests.
/contrib/opentimelineio_contrib/adapters/tests/test_cdl_adapter.py provides a straightforward test which takes a sample EDL, ingests into OTIO and exports temporary .cdl files via the adapter. Two of these CDL's are read in as samples and checked.

Please close this - will submit with an account covered under the CLA.

Apologies, please close this PR - a duplicate PR has been created with the correct account to cover the CLA:
#1368
https://github.com/AcademySoftwareFoundation/OpenTimelineIO/pull/1368


ConnectionResetError: [Errno 104] Connection reset by peer

Occasionally encounter errors
+ python3 -m vllm.entrypoints.openai.api_server --host xxxxx --port 8003 --served-model-name qwen1.5-72b-chat-int4 --model /home/vllm/model/Qwen1.5-72B-Chat-GPTQ-Int4 --trust-remote-code --tokenizer-mode auto --max-num-batched-tokens 32768 --tensor-parallel-size 4
INFO 02-29 14:28:09 api_server.py:228] args: Namespace(host='xxxxx', port=8003, allow_credentials=False, allowed_origins=['*'], allowed_methods=['*'], allowed_headers=['*'], api_key=None, served_model_name='qwen1.5-72b-chat-int4', lora_modules=None, chat_template=None, response_role='assistant', ssl_keyfile=None, ssl_certfile=None, root_path=None, middleware=[], model='/home/vllm/model/Qwen1.5-72B-Chat-GPTQ-Int4', tokenizer=None, revision=None, code_revision=None, tokenizer_revision=None, tokenizer_mode='auto', trust_remote_code=True, download_dir=None, load_format='auto', dtype='auto', kv_cache_dtype='auto', max_model_len=None, worker_use_ray=False, pipeline_parallel_size=1, tensor_parallel_size=4, max_parallel_loading_workers=None, block_size=16, seed=0, swap_space=4, gpu_memory_utilization=0.9, max_num_batched_tokens=32768, max_num_seqs=256, max_paddings=256, disable_log_stats=False, quantization=None, enforce_eager=False, max_context_len_to_capture=8192, disable_custom_all_reduce=False, enable_lora=False, max_loras=1, max_lora_rank=16, lora_extra_vocab_size=256, lora_dtype='auto', max_cpu_loras=None, device='auto', engine_use_ray=False, disable_log_requests=False, max_log_len=None)
WARNING 02-29 14:28:09 config.py:186] gptq quantization is not fully optimized yet. The speed can be slower than non-quantized models.
INFO 02-29 14:28:09 config.py:421] Custom all-reduce kernels are temporarily disabled due to stability issues. We will re-enable them once the issues are resolved.
2024-02-29 14:28:12,795 INFO worker.py:1724 -- Started a local Ray instance.
INFO 02-29 14:28:15 llm_engine.py:87] Initializing an LLM engine with config: model='/home/vllm/model/Qwen1.5-72B-Chat-GPTQ-Int4', tokenizer='/home/vllm/model/Qwen1.5-72B-Chat-GPTQ-Int4', tokenizer_mode=auto, revision=None, tokenizer_revision=None, trust_remote_code=True, dtype=torch.float16, max_seq_len=32768, download_dir=None, load_format=auto, tensor_parallel_size=4, disable_custom_all_reduce=True, quantization=gptq, enforce_eager=False, kv_cache_dtype=auto, device_config=cuda, seed=0)
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/workspace/vllm/entrypoints/openai/api_server.py", line 236, in <module>
    engine = AsyncLLMEngine.from_engine_args(engine_args)
  File "/workspace/vllm/engine/async_llm_engine.py", line 625, in from_engine_args
    engine = cls(parallel_config.worker_use_ray,
  File "/workspace/vllm/engine/async_llm_engine.py", line 321, in __init__
    self.engine = self._init_engine(*args, **kwargs)
  File "/workspace/vllm/engine/async_llm_engine.py", line 366, in _init_engine
    return engine_class(*args, **kwargs)
  File "/workspace/vllm/engine/llm_engine.py", line 126, in __init__
    self._init_workers_ray(placement_group)
  File "/workspace/vllm/engine/llm_engine.py", line 303, in _init_workers_ray
    self._run_workers("init_model",
  File "/workspace/vllm/engine/llm_engine.py", line 1036, in _run_workers
    driver_worker_output = getattr(self.driver_worker,
  File "/workspace/vllm/worker/worker.py", line 94, in init_model
    init_distributed_environment(self.parallel_config, self.rank,
  File "/workspace/vllm/worker/worker.py", line 275, in init_distributed_environment
    cupy_utils.init_process_group(
  File "/workspace/vllm/model_executor/parallel_utils/cupy_utils.py", line 90, in init_process_group
    _NCCL_BACKEND = NCCLBackendWithBFloat16(world_size, rank, host, port)
  File "/usr/local/lib/python3.10/dist-packages/cupyx/distributed/_nccl_comm.py", line 70, in __init__
    self._init_with_tcp_store(n_devices, rank, host, port)
  File "/usr/local/lib/python3.10/dist-packages/cupyx/distributed/_nccl_comm.py", line 94, in _init_with_tcp_store
    self._store_proxy['nccl_id'] = shifted_nccl_id
  File "/usr/local/lib/python3.10/dist-packages/cupyx/distributed/_store.py", line 148, in __setitem__
    self._send_recv(_store_actions.Set(key, value))
  File "/usr/local/lib/python3.10/dist-packages/cupyx/distributed/_store.py", line 130, in _send_recv
    result_bytes = s.recv(sizeof(
ConnectionResetError: [Errno 104] Connection reset by peer

Hi,
Do you solve this problem? I encounter the same problem too.

Hi,
Do you solve this problem? I encounter the same problem too.

This issue hasn't been resolved, I encounter it occasionally.

I think there's an issue with cupy backend that's used for tensor parallelism.
If you use enforce_eager=True, it is likely resolved (though it will affect performance). Regarding error itself, I think https://github.com/cupy/cupy is probably a better place to report an error.

I think there's an issue with cupy backend that's used for tensor parallelism.
If you use enforce_eager=True, it is likely resolved (though it will affect performance). Regarding error itself, I think https://github.com/cupy/cupy is probably a better place to report an error.

Hi,
Thanks for your suggestion. I will report on cupy. However, since the vllm must run successfully somehow in multi-node environment before release while I fail to deploy every time, I think there must be something that is related to environment hinder the deployment.

I think the cupy backend has been introduced lately for cuda graph (which is disabled by force_eager=True). My guess is this backend is not working well in some environments, but it is pretty difficult to troubleshoot without reproducing the issue. If you can tell me your instance details, I can try repro


(EMAIL) EGA: Tickets - to be sent 27.11

SUBJECT: E-Commerce Germany Awards: Get your Networking Party tickets now at Black Week promotional price
@component('mail::message')

# Hello {{ array_get($profile, "fname") }},

We are thrilled to announce the upcoming return of our exclusive **Networking Party** on February 21st in Berlin, scheduled for one night before the E-commerce Berlin Expo.

We have some exciting news just in time for the **Black Week**.

The Networking Party is an integral part of the much-celebrated [E-Commerce Germany Awards](https://ecommercegermanyawards.com?utm_source=email&utm_medium=email&utm_campaign=exhibitors_system_ega).

As an exhibitor, you have a special opportunity to be part of this prestigious event. What's even better is that during the Black Week, you can secure your gala ticket for an incredible price of only **90 euros** (30 euros off the regular price). Hurry up, as the number of tickets is limited!

@component('mail::button', ['url' => $accountUrlVouchers])
Buy your ticket here
@endcomponent

Don't miss out: the offer is valid only from **November 27th till December 1st (4 pm)**.

Should you have any questions, do not hesitate to contact me.

Kind regards,

{{$footer}}
@endcomponent

@az dzisiejszy mailing ;)


Authentication Issue

No problem committing up until today:

I have not changed anything, using the same token to access my account from SmartGit.

@jean-christophe-manciot which GitHub service are you experience an issue with?
Are you having problems pushing to GitHub?
This repository is for issues related to the services available from a repository's Settings page.

Closing this out since this looks like a Git authentication issue unrelated to the services included in this repository.
Also, please contact support if you are still having issues pushing or pulling from your repositories.


Error in installing HAPROXY from web interface

So after installing HAPROXY-wi manually following your commands,
i tried to install Haproxy on my host, but i got this error !!!!

Maybe it's a bug, I will check

okay, i hope you find a way out
i'll be waiting for your feedback

Wich ansible version do you use?

i have updated my ansible version and now i get this problem

I cannot see text now, show me text

error: Unhandled error: Traceback (most recent call last): File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 554, in update_config_data value, origin = self.get_config_value_and_origin(config, configfile) File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 498, in get_config_value_and_origin value = ensure_type(value, defs[config].get('type'), origin=origin) File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 121, in ensure_type makedirs_safe(value, 0o700) File "/usr/lib/python2.7/site-packages/ansible/utils/path.py", line 90, in makedirs_safe raise AnsibleError("Unable to create local directories(%s): %s" % (to_native(rpath), to_native(e))) AnsibleError: Unable to create local directories(/usr/share/httpd/.ansible/tmp): [Errno 13] Permission denied: '/usr/share/httpd/.ansible' Traceback (most recent call last): File "/usr/bin/ansible-playbook", line 62, in import ansible.constants as C File "/usr/lib/python2.7/site-packages/ansible/constants.py", line 174, in config = ConfigManager() File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 290, in init self.update_config_data() File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 566, in update_config_data raise AnsibleError("Invalid settings supplied for %s: %s\n" % (config, to_native(e)), orig_exc=e) ansible.errors.AnsibleError: Invalid settings supplied for DEFAULT_LOCAL_TMP: Unable to create local directories(/usr/share/httpd/.ansible/tmp): [Errno 13] Permission denied: '/usr/share/httpd/.ansible'

Create folder usr/share/httpd/.ansible with apache owner

after creating the folder with apache owner i got this one
*********************************************************', '', 'TASK [Gathering Facts] *********************************************************', 'fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Invalid/incorrect password: Failed to add the host to the list of known hosts (/usr/share/httpd/.ssh/known_hosts).\r\nPermission denied, please try again.", "unreachable": true}', '', 'PLAY RECAP *********************************************************************', '<IP_ADDRESS> : ok=0 changed=0 unreachable=1 failed=0 skipped=0 rescued=0 ignored=0 ', '', "error: Can't install Haproxy service

Create the folder /usr/share/httpd/.ssh

thank you so much for your help !
it all comes down to this one
['', 'PLAY [<IP_ADDRESS>] *********************************************************', '', 'TASK [Gathering Facts] *********************************************************', 'fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Invalid/incorrect password: Permission denied, please try again.", "unreachable": true}', '', 'PLAY RECAP *********************************************************************', '<IP_ADDRESS> : ok=0 changed=0 unreachable=1 failed=0 skipped=0 rescued=0 ignored=0 ', '', "error: Can't install Haproxy service
and thank you !

Here I think pretty obviously problem

it looks so obvious but its not !
i dont know why it shows this error !
( btw when i added the server, ive done it using password and not ssh key )

On the overview pages shows everything ok?

its like this !

It's seems like everything is ok, for me it's works fine... I cannot say anything use your logs

when you add a server you use ssh credentials or password ?

Both

Thank you i have fixed to error with ssh key !


PHP: testing appsec/misc fixes in tests

There are two failures that have no details and that I couldn't figure out:
============================================ short async test summary info =============================================
 FAILED tests/test_sampling.py::Test_SamplingDecisions::test_sampling_decision_added - 1 fails
 FAILED tests/appsec/test_traces.py::Test_AppSecEventSpanTags::test_custom_span_tags - 1 fails
================================================ 2 failed, 97 passed ================================================

Also requires https://github.com/DataDog/dd-appsec-php/releases/download/v0.1.0/dd-appsec-php-0.1.0-amd64.tar.gz in /binaries, as the repository is not open yet, so it cannot be downloaded.

For the lint, you can do, with python 3
pip install -r requirements.txt
black .

For the lint, you can do, with python 3
pip install -r requirements.txt
black .

done

Checking ...

I've pushed a new commit. Tests are now passing locally.

Tests now pass on CI

@cbeauchesne  Is there anything left for me to do here?

@cataphract so far, I don't think so. On my side, I need to make a change on the versionning system, will be ok today or tomorrow

Ok, I've added a commit that handles the php/AppSec component. In the released decorators, php refers to the tracers (current version 0.67) and php_appsec your component.
Now, several questions :

If there is nothing in binary folder, it's the 0.67 version of the tracers who is installed. 0.67 is hardcoded, is there a way to get the "latest" instead ?
If there is nothing in binary folder, the 0.1 version of the php_appsec component is installed => I have the lock issue.
./utils/scripts/load-binary.sh php_appsec should be working. Basically, it loads the latest build from your master branch and put it binary. For instance, for the tracer, I load it from CircleCI.

Note : turns out I may have introduced some mess on the CI, fixing it.

Ok, I've added a commit that handles the php/AppSec component. In the released decorators, php refers to the tracers (current version 0.67) and php_appsec your component.
Now, several questions :

If there is nothing in binary folder, it's the 0.67 version of the tracers who is installed. 0.67 is hardcoded, is there a way to get the "latest" instead ?

With the latest version of the installer, this is possible. I've added a commit that enabled this.

If there is nothing in binary folder, the 0.1 version of the php_appsec component is installed => I have the lock issue.

This shouldn't happen with the latest version of the install script or with the upcoming 0.2.0 version of ddappsec.

./utils/scripts/load-binary.sh php_appsec should be working. Basically, it loads the latest build from your master branch and put it binary. For instance, for the tracer, I load it from CircleCI.

We currently don't publish snapshots. I'll add this to our todo list.

With the latest version of the installer, this is possible. I've added a commit that enabled thhis

:+1:

We currently don't publish snapshots. I'll add this to our todo list.

ok, we can move forward without it, just ping me once it'll be done. Side question : does the repo be open sourced one day ?

Ok, the last step is the support of DD_APPSEC_WAF_TIMEOUT=5s. Regarding the current setup, the tests will be too flaky without it :(

@cataphract @Anilm3 merging main into this PR is a little bit a pain, I'm splitting it in two parts :

the php_appsec mechanism, to be merged this morning
unskipping test for php_appsec, to be merged once the waf badget can be set threw an env var.


Remove call to createDeviceInfoList.

"This function takes quite a while to run and can cause major slowdown. The d2xx developer documentation explicitly states that functions which rely on the device info list can be called, createDeviceInfoList must be called. A developer following that guide (such as myself) would be thrown off by all the implicit calls to that function."

I think this change may break existing code. Would it be possible to make the change such that existing code will not be affected? One idea would be an optional flag to disable implicit call to createDeviceInfoList

Thanks for the reply! I wasn't sure if this repo was dead or not.
t will break existing code, but only if you aren't following the D2xx programmers guide. To quote from that (excerpt taken from the GetDeviceInfoList description):
"This function should only be called after calling FT_CreateDeviceInfoList. If the devices connected to the
system change, the device info list will not be updated until FT_CreateDeviceInfoList is called again."
I'll tell you how I came across this, and you can tell me what you think. I was developing for windows, and instead of using a COM port to open devices (they change a lot), I wanted to use the serial number programmed into the FTDI chip. The problem was that I had 50+ devices on the system and the call to createDeviceInfoList would take upwards of 10 seconds to return. I was following the programmer's guide. I would call createDeviceInfoList on boot, then at runtime I would getDeviceInfoList and getDeviceInfoDetail to look for the device that matched the serial number I was looking for. I couldn't figure out why that process took over 10 minutes to finish! Then I found the two calls to createDeviceInfoList that didn't match what I expected from following the programmer's guide. So if had 50 devices plugged in, createDeviceInfoList had the possibility of being called 50+ times!
Sorry for the wall of text hahaha. You're the maintainer so it's your call but I felt like this library should reflect what the documentation FTDI gives as closely as possible.
Thanks again!

@snmishra Any thoughts here?

Sorry for taking this long.
Could you try the code in the "nocreate" branch? You'd need to call init and getDeviceInfoDetail with update=False. Will this work for you?
I suspect that most users have few devices on the bus. They would find it convenient to have a builtin call to createDeviceInfoList ensuring the device list is up to date. In fact, I suspect that's why I put it in there originally. Additionally, my change should allow existing code to work as before.

@snmishra your changes look great!
I will still be using my branch for my purposes, but I agree with you that few users will have more than one or two devices.


CI: Add workflow for tox verify

Issue: RELENG-5096

Those inputs are mostly coming from the Gerrit to Platform trigger.
I believe we could remove the ones that are not actually used but noticed that all our workflows have them even when not used.
I think at this point is more for consistency.


feat: dependent required validation trigger when value equals

Sometimes a property becomes required based on another property having a certain value.
This is often the case when a property takes the role of a mode or type, something that would be modeled as an enum in Java.
For example, in JSON patch the operation op chosen directly translates to which other properties must or may be present.
Such triggers can be modeled by extending the dependentRequired semantics.
A group given as groupname=x annotated on a property Y means that if Y has the value x the group is triggered.
Assuming that the group would usually be named after the (enum) value that informs it, for example add=add, it could be allowed to shorten this to =add.

Implemented by #49


AdditionalTools validation & feedback

Summary of the pull request
In Settings | Additional Tools, ensure the toolpath and name are valid, and provide user feedback when registering a tool (via a new timed message dialog in WindowHelper). Also only enable the Unregister button if the user selects at least one tool (and enable unregistering multiple tools at once).
Also, added tooltips to all the fields on the App Details page; suppress the external tools menuflyout if there are no external tools; and fixed the pin/unpin menu icon.
References and relevant issues
Fixed the following:

Closes #3040
Closes #3041
Closes #3042
Closes #3043
Closes #3045
Closes #3047

I edited your description with the magic syntax so GitHub would link everything together. Now when this closes, the issues it fixes will close.


Crash today

I'm not shure if you can make sense out of this.
coredump.log

what were you doing? Is this reproducible? Seems weird.

I'm not shure, i think i was moving a window to a workspace or something.
Sorry

got another one.
I tested something with a youtube video pinned fullscreened moved to another workspace.
coredump.log

Reproduce with:
produce a pinned window,
fullscreen that window,
switch to another workspace that has some windows,
focus the pinned fullscreen window (i do it with a mouse motion (follow mouse).
try to unfullscreen the pinned window.
I hope its reproducbble it on your machine too.

should be fixed with d24f31de514be1c78fcb6a3ad392a01dd7117e3e as pinned windows can no longer be fullscreen.

Reasonable,
thanks!


Fix convert-relax-to-half invalid code (#3099)

Rewrite to do formal closure of relaxed precision on composite and phi instructions contiguous to relaxed computations to "minimize" convert operations.

CI is showing a format error in a file I didn't even change :P

Rebased to latest master and re-pushing


StreetBattle01.tscn doesn't win properly

The scene StreetBattle01.tscn doesn't end after the player has killed all the mechs.

The $HUD node was hidden because it clutters the inspector.
I added code to make it visible in-game.


Midea MF200W100WB-14A Waschmaschine

Hi,
i bought an Midea Washingmashine and it working in the "Smarthome" App From Midea.
Will there be support for this Washingmashine?
Bye
Jacek Jashari

hello! could you please install the plugin and do a discovery? it contains informations for me which is needed to answer this question

do you need help doing this step? I can't proceed with this until I get the information needed

Hi, this is a dump of the discovery of my MF200 washer dryer:

Name
Type
Model
Id
Ver.
Token / Key
Local Ip
Add / Update

midea_db_0155
Unknown
38123703
153931628080922
3
tokentokentoken (dm if needed)
<IP_ADDRESS>
Not supported

I've published v1.1.2-beta.2, which should include the front loading washers' support. Could you please try it out?

It works perfectly- however, during the washing, it does not show the remaining time, but always "0:00".

Oh: and the plugin generates the following error every 10 seconds if the washing machine is off (it is then disconnected from the network):
[8/8/2024, 10:49:08 PM] [homebridge-midea-platform] Socket error:
Error: connect EHOSTUNREACH <IP_ADDRESS>:6444
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1606:16)
[8/8/2024, 10:49:08 PM] [homebridge-midea-platform] Socket closed with error

Thank you so much for your great work on this plugin!

Could you please try it again using the latest beta version?

It mostly works - however, it counts the time down as seconds, not minutes.
When 13 minutes are remaining, home counts down from 13 seconds.

I've released v1.2.0-beta.2, which should contain a fix for this. Could you please check it and write feedback?

Works perfectly! Thank you so much!

Is there any other feature you would want?

No, not really.
If you could get the dishwasher controls in #104 working, this might be a welcome feature for the washers, as well - but it's probably easier now to leave the washers as is for now and extend the functionality later on.

Cool! Let me know if you need something. I just wrote a comment on the other issue. If you have any functionality in your mind, you can do the same process with your model. Just send me that file here or in email, and reopen this issue with the feature you need.


Single length validation with "null" value its not working correctly

validate.single(null, {length: {minimum: 3}})
// undefined

This function returns undefined. The correct should not be a validation error?

The same for an empty string:
validate.single('', {length: {minimum: 3}})

An empty string length is less than 3

Working as intended:

Important! One thing that is a bit unorthodox is that most validators will consider empty values (null, undefined, whitespace only strings etc) valid values. So for example adding a constraint of at least 6 characters will be like saying If the attribute is given it must be at least 6 characters.
You need to use the presence validators as well.


Zero values get ignored when populating data (e.g. 0 precipitation)

Noticed that precipitation always appears on the first day of the forecast, even if rain isn't expected until later in the week. Here's the cause:
https://github.com/Yevgenium/weather-chart-card/blob/0c16ee74a90c8b0902d4a989d883f9d2207331c7/src/main.js#L145-L153
if (d.temperature) resolves to false for zero values (e.g. 0 degrees, 0 precipitation), causing these days to be skipped in the chart data. Need to use something like if (typeof d.temperature !== 'undefined').
Also, it may be best to insert a dummy value in these cases, so that if d.datetime and d.temperature are defined, but d.templow is not defined, the various arrays all keep the same length. Without doing that, the array that's missing a value will have all of its remaining data offset by one day.

Yes, I already noticed that. Will be fixed in the next version. Thanks!

I modified my version in the interim, and came across another edge case: the Dark Sky weather integration returns null when precipitation is 0 (instead of returning a float). That was causing the graph to not show up for me. I had to add a check for null and instead push a 0 into the array :-(

Fixed in v1.0.2. Could you check?

Tested OK with both 0 and null values. Looks good to me!


Wrong build targets

The current build targets are in my opinion wrong:
The macro build targets build multiple binaries with the same name (server, client), which then overwrite each other when installed.
The build targets should build binaries with the same names as those used by the packaged version.
The current https://github.com/netsec-ethz/scion-apps/blob/cc4e1dc18ebaf340cd3d32e381e6900f0879ae0a/Makefile#L33 way made it easy to have something that covered all apps, but is also confusing and does not work for the current state, diverge from the scion-apps build as part of the packages.

Agreed. We should also build to a common bin/ directory at the project root.


Category Filter | Category Filter must have provision to show category based on some condition in dropdown menu.

Prerequisites

[X] I have read the Contributing Guidelines.
[X] I have not leaked any internal/restricted information like screenshots, videos, code snippets, links etc.

Suggestion / feature request
In "[categories]" we have added list of categories but in real time we have use case to show/hide some category based on previous category and its value.
categories = [ Cat1:{ },  Cat2:{ },  Cat3:{ },  Cat4:{ }   ]
So eg. Cat1  has selected value as Cat1-Val  In this case we do not want to show Cat2 and Cat3 in the dropdown list once user click on selecting new category.

Hello @Jayesh26,
You should be able to set the categories property during runtime to adapt it to the current selection. We will not be able to provide custom logic like the one you described since this is too specific. Please feel free to reopen the issue in case the above solution does not work from a technical standpoint.


How to change state without triggering event ?

Hi! I'm making some stuff ajax and want change it back on error and the currently it goes to infinite recursion..

or at least how to change only style?
i can change state with jquery function.

Update: I've got a way to do it, but it was not very easy to find a way. At least add it to documentation or create method.
$this.prop('checked', true).bootstrapToggle('destroy').bootstrapToggle();

@AucT I tried your trick, it doesn't work... :(

@NRJMons
Add class toggleItem to the input

@AucT I found what didn't work. I had forgotten to change the property of the object before destroying it. I should have copy/pasted instead of typing.
I confirm, it works fine 👍

Worked for me too, thanks. I'd like to see something like bootstrapToggle('toggle-no-event') or bootstrapToggle('toggle', false) to suppress on-change events from firing.

Thanks, I've been pulling my hair for the last 30 minutes and your method saved me.

Would like a fix for this as well. Changing the value of a checkbox doesn't fire off events so why does toggle class? Agreed, there needs to be a backwards-compatible change like 'toggle-no-event'

+1

Is it not possible to be able to generate click events on these buttons, or the underlying checkbox? This way, change events won't run.

This is what I get when i use your code:

any idea why?

Update:
how i solved this at my end:
var input; // the checkbox field
var div = $(input).closest('.toggle');
var current_state = ! $(input).prop('checked');

// on error or need to change back to old state
$(input).prop('checked', ! current_state);
// assuming success button is shown on previous state
$(div).removeClass( current_state ? 'btn-success on' : 'btn-danger off').addClass(current_state ? 'btn-danger off' : 'btn-success on');

Bumpety bump bump. This is still an issue :-(

+1024
please make this happen

+1 please

I've run into needing this several times. Any chance of making it a reality?

same problem here, is there any other replacement for this control ? or any other library ?

doesn't work and apparently not going to be i just create my own thing

https://github.com/minhur/bootstrap-toggle/issues/24 works for me


Remove hardcoded version from composer.json

Hey,
You don't need version in composer.json, it mislead. Composer will automatically take version from git tags.
https://getcomposer.org/doc/02-libraries.md#specifying-the-version

BTW, version 1.0.4 is not available on packagist - https://packagist.org/packages/onesky/api-library-php5

Oops...thanks again and will handle this together with your another PR as well

:+1:

@skolodyazhnyy Raised issue to match this PR: https://github.com/onesky/api-library-php5/issues/15

Done at #19


A follow-up for #4324

A follow-up for #4324. Refactors the code a bit.

@DimitrisJim Would you mind taking a look?


Change sw and te tokenizers for Mr.TyDi

Note that we do not have language-specific tokenizers for sw and te, so we use whitespace, which is exactly the same as -pretokenized. Using -language xx is conceptually cleaner, but doesn't change results.

Codecov Report

Merging #1727 (ce4b942) into master (ed88453) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1727   +/-   ##
=========================================
  Coverage     57.58%   57.58%
  Complexity     1032     1032
=========================================
  Files           173      173
  Lines          9781     9781
  Branches       1344     1344
=========================================
  Hits           5632     5632
  Misses         3675     3675
  Partials        474      474

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 ed88453...ce4b942. Read the comment docs.


Feat(web): Switch Select component to v3 design tokens #DS-1447

Description

Additional context

Issue reference

coverage: 78.659%. remained the same
when pulling cb4a355a9fa27f0a5655e05146f4567dfdd57b87 on feat/v3-select
into 6fdb9d211ef0c9e9f6b46e4fffbb055ec4b40a06 on feat/v3-textfield.


False Positive on google-http-client-1.38.0

google-http-client seems to be mistaken with apache:httpclient
dependencyCheck reports CVE-2020-13956 as the vulnerability
google-http-client-1.38.0 uses org.apache.httpcomponents.httpclient-4.5.13, which is not affected by CVE-2020-13956
This is evident from https://repo1.maven.org/maven2/com/google/http-client/google-http-client/1.38.0/google-http-client-1.38.0.pom, which uses https://repo1.maven.org/maven2/com/google/http-client/google-http-client-parent/1.38.0/google-http-client-parent-1.38.0.pom as a parent. There project.httpclient.version is set to 4.5.13.

I just tested this using 6.0.4 and do not see this FP being reported.

I just tested this using 6.0.4 and do not see this FP being reported.

For google-http-client-apache-v2-1.41.0.jar, there seem to be a false positive as well:
{
  "file": "google-http-client-apache-v2-1.41.0.jar",
  "sha1": "0d3edd9b035d8407a359c3395a5a7f1b3d1e534e",
  "description": "Google HTTP Client Library for Java",
  "projectReferences": [
    "Implementation for Alfred:compile",
    "External Alfred deployment:compile"
  ],
  "vulnerabilities": {
    "name": "CVE-2020-13956",
    "severity": "MEDIUM",
    "description": "Apache HttpClient versions prior to version 4.5.13 and 5.0.3 can misinterpret malformed authority component in request URIs passed to the library as java.net.URI object and pick the wrong target host for request execution."
  }
}

But in google-http-client-parent-1.41.0.pom, we have a:
<project.httpclient.version>4.5.13</project.httpclient.version>
<!-- ... -->
<dependency>
  <groupId>org.apache.httpcomponents</groupId>
  <artifactId>httpclient</artifactId>
  <version>${project.httpclient.version}</version>
</dependency>


Update leaflet-geotiff.js

Atm the checking of plotty and GeoTIFF libraries is being done at script load time.
This means a JS error is thrown for NodeJS applications where those libraries are only loaded on an as needed basis instead of a global load for all web pages.
Suggest moving those 2 if statements into the initialize function

Yep it poses an issue.
Since the library does not export any module it can't get included as a node reference. Furthermore it is also not available via npm so may be this issue was not detected before.
I'm including all required references plotty, geotiff and leaflet via npm.


t_ship @destroy

@Destroy not working with t_ship event

Does it happen even with houses?

Does it happen even with houses?

yes doesn't work with houses either

Fix sent

thanks for your helps :)


confusing error message in forward over reverse

using Enzyme

function f_ip(x, tmp)
    tmp .= x ./ 2
    return dot(tmp, x)
end

function f_gradient_deferred!(dx, x, tmp)
    dtmp = make_zero(tmp)
    autodiff_deferred(Reverse, Const(f_ip), Active, Duplicated(x, dx), Duplicated(tmp, dtmp))
    return nothing
end

function f_hvp!(hv, x, v, tmp)
    dx = make_zero(x)
    btmp = make_zero(tmp)
    autodiff(
        Forward,
        f_gradient_deferred!,
        Duplicated(dx, hv),
        Duplicated(x, v),
        Duplicated(tmp, btmp),
    )
    return nothing
end

x = [1.0]
v = [-1.0]
hv = make_zero(v)
tmp = similar(x)

f_hvp!(hv, x, v, tmp)

In this example, all calls to f_ip will fail because dot is not defined (one should do using LinearAlgebra to fix this).  The error message on calling f_hvp! however is quite puzzling
ERROR: AssertionError: Base.allocatedinline(actualRetType) returns false: actualRetType = Any, rettype = Active{Any}
Stacktrace:
  [1]
    @ Enzyme.Compiler ~/.julia/dev/Enzyme/src/compiler.jl:4170
  [2] enzyme!(job::GPUCompiler.CompilerJob{…}, mod::LLVM.Module, primalf::LLVM.Function, TT::Type, mode::Enzyme.API.CDerivativeMode, width::Int64, parallel::Bool, actualRetType::Type, wrap::Bool, modifiedBetween::Tuple{…}, returnPrimal::Bool, expectedTapeType::Type, loweredArgs::Set{…}, boxedArgs::Set{…})
    @ Enzyme.Compiler ~/.julia/dev/Enzyme/src/compiler.jl:4011
  [3] codegen(output::Symbol, job::GPUCompiler.CompilerJob{…}; libraries::Bool, deferred_codegen::Bool, optimize::Bool, toplevel::Bool, strip::Bool, validate::Bool, only_entry::Bool, parent_job::GPUCompiler.CompilerJob{…})
    @ Enzyme.Compiler ~/.julia/dev/Enzyme/src/compiler.jl:6308
  [4] codegen
    @ ~/.julia/dev/Enzyme/src/compiler.jl:5465 [inlined]
  [5] (::GPUCompiler.var"#187#198"{GPUCompiler.CompilerJob{…}, GPUCompiler.CompilerJob{…}})()
    @ GPUCompiler ~/.julia/packages/GPUCompiler/CaR87/src/driver.jl:224
  [6] get!(default::GPUCompiler.var"#187#198"{…}, h::Dict{…}, key::GPUCompiler.CompilerJob{…})

    @ Enzyme.Compiler ~/.julia/dev/Enzyme/src/compiler.jl:7224
 [18] #s2084#19056
    @ ~/.julia/dev/Enzyme/src/compiler.jl:7266 [inlined]
 [19]
    @ Enzyme.Compiler ./none:0
 [20] (::Core.GeneratedFunctionStub)(::UInt64, ::LineNumberNode, ::Any, ::Vararg{Any})
    @ Core ./boot.jl:602
 [21] autodiff
    @ ~/.julia/dev/Enzyme/src/Enzyme.jl:428 [inlined]
 [22] autodiff
    @ ~/.julia/dev/Enzyme/src/Enzyme.jl:344 [inlined]
 [23] autodiff
    @ ~/.julia/dev/Enzyme/src/Enzyme.jl:325 [inlined]
 [24] f_hvp!(hv::Vector{Float64}, x::Vector{Float64}, v::Vector{Float64}, tmp::Vector{Float64})
    @ Main ~/src/scrap.jl:19

It's probably fair to say that anyone not intimately familiar with enzyme internals might have a hard time figuring out why this failed from the stack trace and error message.
Note that if one calls only f_gradient_deferred! one gets
ERROR: Active return values with automatic pullback (differential return value) deduction only supported for floating-like values and not type Any. If mutable memory, please use Duplicated. Otherwise, you can explicitly specify a pullback by using split mode, e.g. autodiff_thunk(ReverseSplitWithPrimal, ...)
Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:35
 [2] default_adjoint
   @ ~/.julia/dev/Enzyme/src/compiler.jl:6651 [inlined]
 [3] autodiff_deferred
   @ ~/.julia/dev/Enzyme/src/Enzyme.jl:467 [inlined]
 [4] f_gradient_deferred!(dx::Vector{Float64}, x::Vector{Float64}, tmp::Vector{Float64})

This seems a lot more reasonable because it's pretty clear that it can't infer the return type of dot so Enzyme thinks the call to autodiff_deferred is invalid for this code, but if there's anyway of showing the user there is an UndefVarError here, that would surely be useful.

hm did you start your split tests from your old PR... I say this because this is one of the abi's that changed and shouldn't happen if you just copy and paste from scratch.
It makes me a bit worried that the new PR to split tests misses some changes to tests or new tests (which is why I suggested to copy and paste from scratch and not rebase)

No, I started it entirely from latest main, I wound up not even touching my old branch.  This is definitely on latest main.

this should be fixed by https://github.com/EnzymeAD/Enzyme.jl/pull/1909
and now you'll just get
julia> f_hvp!(hv, x, v, tmp)
ERROR: UndefVarError: `dot` not defined
Stacktrace:
  [1] f_ip
    @ ./REPL[2]:3 [inlined]
  [2] augmented_julia_f_ip_3182wrap
    @ ./REPL[2]:0
  [3] macro expansion
    @ ~/git/Enzyme.jl/src/compiler.jl:9226 [inlined]
  [4] enzyme_call
    @ ~/git/Enzyme.jl/src/compiler.jl:8792 [inlined]
  [5] AugmentedForwardThunk
    @ ~/git/Enzyme.jl/src/compiler.jl:8629 [inlined]
  [6] autodiff_deferred
    @ ~/git/Enzyme.jl/src/Enzyme.jl:719 [inlined]
  [7] f_gradient_deferred!
    @ ./


Fix --port=0 option to correctly open on random port

Dependent on https://github.com/cloudflare/miniflare/pull/382

Hey so with https://github.com/cloudflare/miniflare/pull/382 merged, is this good to go?

Just waiting on a Miniflare release 😊

Also may want to add this to --experimental-local

@mrbbot pls

https://github.com/cloudflare/wrangler2/pull/2003

hmm, this actually isn't quite ready. It improves things, but it doesn't update the hotkey things, so pressing b will try to open localhost:0 :(


Change hover color of New England and small state label boxes?

I noticed that changing the stateHoverStyles property will change the color of the actual state but not the color of the label box for the smaller northeastern states. Even specifying for the state (stateSpecificHoverStyles) does not work. Does anyone have suggestions for fixing this?

I have a pull request to fix this:
https://github.com/kcaran/us-map
Example here:
http://www.kcaran.com/us-map-example/

Thank you! The line that fixed it by setting the fill property of labelBackingHoverStyles to the same color as stateHoverStyles.


Add "Max inline results per request = 50" in [Bots] category

Add Max inline results per request =  50 inline result, Under Bots category.
The info source (For checking this info).

Thank you so much, I'll add it shortly

Thank you so much, I'll add it shortly

With that, you can too limitatiom for Group Call Title - 64 Characters


Error: "networkId" is required

After all steps, the app not builds, with an error "networkId" is required.
I guess it comes from @pooltogether/bnc-onboard package but could not find any solution...
Any idea how it can be solved?

Solved. Installed dotenv and moved environment variables there.

Having a similar issue, can you provide more details on your solution?


Disallow TextArea resize if cols is specified

A recent fix to the upstream Carbon library caught my eye: https://github.com/carbon-design-system/carbon/pull/13069
It disables the "resize" attribute for TextArea if the cols prop is specified.
The differences in behavior compared to the Svelte implementation:

cols does not have a default value (the Svelte implementation defaults to 50)
if cols is not specified, the textarea has width=100%
if cols is specified, the textarea unsets width and applies the style rule resize=none

If we want to align with the upstream library, this would be a breaking change:
- export let cols = 50;
+ export let cols = undefined;

+ <textarea style:resize={typeof cols === "number" ? "none" : undefined} />

Perhaps we can include this change as part of the v11 upgrade, assuming the flagship library adopted this behaviour in v11.


Bad result

I am running a local notebook and am getting very bad false results.
Example
Example

It also require prompt engineering.
For high resolution image, you may need to resize the image or increase the size of selection_box for outpainting

It also requires prompt engineering. (here I use "Ivan Grohar, impressionist, painting style") For high resolution image, you may need to resize the image or increase the size of selection_box for outpainting

Oh so the program itself can't pick up the style that good?

It also requires prompt engineering. (here I use "Ivan Grohar, impressionist, painting style") For high resolution image, you may need to resize the image or increase the size of selection_box for outpainting

Oh so the program itself can't pick up the style that good without specifying it?

I think the current version of diffusers's stablediffusion pipeline is unable to do that. (Currently there is no explicit constraint to make the filled region looks like other parts.)
My program only provides a handy UI and the initial guess for empty region based on texture synthesis.

It also requires prompt engineering. (here I use "Ivan Grohar, impressionist, painting style") For high resolution image, you may need to resize the image or increase the size of selection_box for outpainting

Oh so the program itself can't pick up the style that good without specifying it?

I think the current version of diffusers's stablediffusion pipeline is unable to do that. (Currently there is no explicit constraint to make the filled region looks like other parts.) My program only provides a handy UI and the initial guess for empty region based on texture synthesis.

Understandable. I just saw your demo and thought it worked that way.

@lkwq007 I am thinking through how some of the current outpainting techniques work. Could this use img2img and treat this as inpainting, using the previous results as a mask for the next image? (If that's already how this works, forgive me).
I've seen pretty good results using the newwer outpainting custom script in AUTOMATIC1111's repo, but the UI for it is pretty inflexible. Link to the script: https://github.com/AUTOMATIC1111/stable-diffusion-webui/blob/84e97a98c5233119d0f444e0a3a0f6391da23677/scripts/outpainting_mk_2.py
Can any of this be used to make these results better?


When running docker, merged-SBoM-deep.json saved inside the container

When running from docker, merged-SBoM-deep.json is created and left in the container. There needs to be an argument passed for an output directory.
`
cgriffin@cgriffin-MBP fetch-sboms % docker run -it
-v ${PWD}/sbom_dir:/code
-e DOCPATH='/code'
-e NAME='electricgull-sbom'
-e MERGETYPE='1'
-e AUTHOR='Cameron Griffin'
-e<EMAIL_ADDRESS>-e DOCNAMESPACE='https://electricgull.com/'
-e FILETYPE=J
docker.io/philipssoftware/spdxmerge:v0.1.0
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
arguments:

DOCPATH      : /code
NAME         : electricgull-sbom
MERGETYPE    : 1
AUTHOR       : Cameron Griffin
EMAIL<EMAIL_ADDRESS>DOCNAMESPACE : https://electricgull.com/
FILETYPE     : J
File merged-SBoM-deep.json is generated

cgriffin@cgriffin-MBP fetch-sboms % find ./ -name "merged-SBoM-deep.json"
cgriffin@cgriffin-MBP fetch-sboms %
cgriffin@cgriffin-MBP fetch-sboms %  docker ps -a |grep spdx
7657298698b5   philipssoftware/spdxmerge:v0.1.0               "/app/entrypoint.sh"     16 seconds ago   Exited (0) 11 seconds ago                                                              bold_franklin
cgriffin@cgriffin-MBP fetch-sboms % docker start bold_franklin; docker exec -it bold_franklin ls
bold_franklin
CHANGELOG.md CONTRIBUTING.md  Dockerfile  LICENSE  README.md  REPO  TAGS  action.yml  entrypoint.sh  launch.json  merged-SBoM-deep.json  requirements-dev.txt	requirements.txt  src  test
`

Added the following code to provide an option for an output path.  I tested both via the command line and built a container. Both seemed to work fine.  I didn't test the github action.
https://github.com/philips-software/SPDXMerge/pull/42

Thanks for the PR. Merged into main in #42


src directory is not found in file path <...>/build.gradle.kts

Describe the bug
This message is true, but we shouldn't log it. We have some special handling of gradle scripts, and it should include legal absence of src directory
Environment information

diktat version: 1.0.0-rc.2
build tool (maven/gradle): gradle

I think under this issue we can also investigate problem with extra warnings from kotlin
Seems, it should be fixed (youtrack, stackoverflow) in kotlin 1.5

I think under this issue we can also investigate problem with extra warnings from kotlin
Seems, it should be fixed (youtrack, stackoverflow) in kotlin 1.5

Yep, I've also found this youtrack ticket, which also states that this issue should be fixed in 1.5.20. However, several projects with diktat, i.e. diktat, save, have another class in the warning:
WARNING: Illegal reflective access by org.jetbrains.kotlin.com.intellij.util.ReflectionUtil (file:/home/runner/.m2/repository/org/jetbrains/kotlin/kotlin-compiler-embeddable/1.5.20/kotlin-compiler-embeddable-1.5.20.jar) to field java.lang.Throwable.backtrace

which is probably another issue


ENT-6018 Teach tom to add packages to package_mapping.json file

Sample PR: https://github.com/cfengine/system-testing/pull/333

commit amended to reflect the fact that masterfiles package must be among hub packages, not agent ones.

PR updated to accept also version, in addition to LTS branch name, so Tom can also add non-LRS 3.16.0 packages to package_mapping file! PR: https://github.com/cfengine/system-testing/pull/346


Animations missing on iPhone/iOS since Bootstrap version 4.1

Since Bootstrap version 4.1 the animations are missing on my iPhone. I tested it on multiple devices with iOS (latest version, iOS 12) and different browsers (Safari and Chrome), but the animations are missing.  Because of this, any modal opens without a fade effect (even though it has the "fade" class), the hamburger menu opens without a slide down effect and so on.
This happens not only on my websites, but also on all of the official examples like https://getbootstrap.com/docs/4.1/examples/carousel/ and when I change the CSS file back to version 4.0 (just CSS, not the JS file), everything works fine like it did before. In accordance with this, the v4.0 examples (e.g. https://getbootstrap.com/docs/4.0/examples/carousel/) are also working fine with all the animations on my iPhones. Only since v4.1 I encountered this problem with missing animations.
On my PC (Windows 10) all the animations are working fine when using the latest version 4.1.3, even when I resize the Chrome or Firefox window just like an iPhones screen size. Is this a known issue and is this an iOS problem? Unfortunately, I have no Android smartphones for a comparison.

This is because when reduce motion is enabled on the OS, animations are disabled.


Local Table of Contents not formatted

Problem
When you use the contents directive to create a local table of contents, the table just formats as a list with no style. (no indents or bullets)
Reproducible Project
https://onap.readthedocs.io/en/latest/submodules/vnfrqts/guidelines.git/docs/vnf_guidelines/vnf_guidelines.html
RST code:
.. contents::
:depth: 3
Error Logs/Results

Expected Results

Environment Info

Python Version: latest
Sphinx Version: latest
RTD Theme Version: latest

@hagopboz Is this still an issue? The reproducible project is no longer accessible.

Closing due to no response.


FI-2820: Update test kit list

Drug Formulary and Plan Net are "DaVinci" instead of "Da Vinci". Otherwise this looks fine!


Support additional refactorings in the analysis server

There is a need to make it easier to add new refactorings, and this is a tracking item for that work.
We need to take into account the refactorings that have been requested by users when designing this support to ensure that all of the reasonable refactorings could potentially be supported. (This is not a promise to support any specific refactoring.)

I believe that the mechanism to make it easier to create new refactorings is complete, so I'm closing this issue.


Possible undefined behavior (std::erase_if)?

Commit: https://github.com/greg7mdp/gtl/commit/f80aefde2b3b1b57c0cbf467f6897561e2f889ad
I think the std::erase_if overload might be undefined behavior.
Maybe it's better to use ADL (https://en.cppreference.com/w/cpp/language/adl) and declare erase_if in the gtl namespace?
Please check it.
Links:
https://en.cppreference.com/w/cpp/language/extending_std
https://stackoverflow.com/a/11599/590688

Thanks @phprus I'll definitely check this (not right now though)

Thanks again @phprus !


Test improvement

This is a test refactoring.
The resource optimism occurs when a test method makes an optimistic assumption that the external resource (e.g., File), utilized by the test method, exists.
As this test creates a file, it is necessary to ensure that any problems caused by the existance (or not) of this file will not affect the result of the test method even without testing what is meant to be tested. In order to do that, we use JUnit's 5 API Assume to make sure that the test file will exist, and if it doesn't just skip the test, since the test goal will not be reached.

I have this problem then across entire test codebase. Where we pass File directly into a decoder. Are you wanting to tackle all of them?

I'm currently working on a research project where we are evaluating a handful of situations like this one across open-source projects. For the criteria applied in our research, my pull request submission is conditioned to a single class for each project and, unfortunately, It won't be possible for me to deal with all the other occurrences throughout the entire project. I would appreciate if you perhaps can answer if do you think this is a real problem and my solution addresses it? There is no problem in closing this PR so that the refactored test class won't be different from the project's standards, but your opinion is very important for my research and I thank you in advance for your time and any further considerations you could give.

@gvma - I took a look and going to decide against this. This feature ("assume") looks better suited for environment checks. Imagine you want to test an edge case with Windows file handling. Since the code is cross platform, that test on a non-Windows environment is pointless.
So by doing an assumption you can skip the test if the environment is non-windows as the test is useless. Looking at your diff, you will skip the test if the file was not created. That is effectively hiding the outcome of a much needed test. If the file fails to load for whatever reason, we will get a nice error. Maybe not pretty, but at least the test will fail. With this patch, the test will be skipped and critical automated tests will not run.
Hope that helps. Thanks for PR though and allowing me to research this.


request: make most/all of the primitive constructor functions into const fn

Allow
const OPEN_PAREN: Just<char, char, Error=Simple<char>> =
  just('(');

and such

I believe this has now been done.


Missing file

Hello,
I believe the following file is missing: RuntimeError: Unable to open shape_predictor_68_face_landmarks.dat
Thanks for the repo btw, awesome work!


Continuation lines for function calls get extra indentation

With fluent-style function calls we might have many-line continuations with chained function calls.  go-mode is adding an additional indent level after every line.
Go-mode's actual formatting:
func Test() {
	foo := Bar().
		Baz().
                        Quxx().
                                Another()
}

Expected (gofmt):
	foo := Bar().
		Baz().
		Quxx().
		Another()

Thanks for the report! Can you make sure it's fixed for you by above change?

@muirdm it works! Thanks for fixing.


chore: Release vodozemac version 0.8.1

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.28%. Comparing base (1a2c003) to head (6cd9d70).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #193   +/-   ##
=======================================
  Coverage   90.28%   90.28%
=======================================
  Files          34       34
  Lines        1905     1905
=======================================
  Hits         1720     1720
  Misses        185      185

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.