any possibility to include a cache .json with a schema during pod launch?

So I have a quarkus pod that is starting after my infinispan pod is successfully up.
I would like infinispan to have a specific cache schema already in place when its created / restarted / whatever.
Is there a property in chart I can use to supply a json structure / file to do this?

@makdeniss By Cache schema do you mean a cache instance, e.g. distributed-cache, replicated-cache, that will be available on server startup?
If so, it's possible to define these in the values.yaml under deploy.infinispan. For example:
deploy:
  ...
  infinispan:
    cacheContainer:
      # [USER] Add cache, template, and counter configuration.
      name: default
      # [USER] Specify `security: null` to disable security authorization.
      security:
        authorization: {}
      transport:
        cluster: ${infinispan.cluster.name:cluster}
        node-name: ${infinispan.node.name:}
        stack: kubernetes
      # Define one or more caches
      caches:
        mycacheone:
          distributedCache:
            mode: "ASYNC"
        mycachetwo:
          distributedCache:
            mode: "SYNC"
    server:
      ...

@ryanemerson thanks for the reply. And what about protobuf schemas? is it possible to pre-create these through chart on pod startup?

@makdeniss Proto schemas can be registered with an infinispan server on startup by adding a .jar containing SerializationContextInitializer implementations to the server's server/lib directory.
In the helm charts the easiest way to do this is to provide the artifacts via deploy.container.libraries, e.g. deploy.container.libraries=org.postgresql:postgresql:42.3.1. This property allows artifacts to be downloaded via maven, http or ftp.


stable/mongodb Error: release mongodb failed: Ingress.extensions "mongodb" is invalid

Describe the bug
Error: release mongodb failed: Ingress.extensions "mongodb" is invalid: spec.rules[0].host: Invalid value: "map[name:mongodb.local path:/]": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

Version of Helm and Kubernetes:
helm version --tiller-namespace amq-streams
Client: &version.Version{SemVer:"v2.16.0", GitCommit:"e13bc94621d4ef666270cfbe734aaabf342a49bb", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.16.0", GitCommit:"e13bc94621d4ef666270cfbe734aaabf342a49bb", GitTreeState:"clean"}

kubectl version
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.7", GitCommit:"8fca2ec50a6133511b771a11559e24191b1aa2b4", GitTreeState:"clean", BuildDate:"2019-09-18T14:47:22Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.6+2e5ed54", GitCommit:"2e5ed54", GitTreeState:"clean", BuildDate:"2019-10-10T22:04:13Z", GoVersion:"go1.12.8", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/mongodb
What happened:
While installing stable/mongodb with ingress.enabled = true , encountering the following error message
Error: release mongodb failed: Ingress.extensions "mongodb" is invalid: spec.rules[0].host: Invalid value: "map[name:mongodb.local path:/]": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is 'a-z0-9?(.a-z0-9?)*')
What you expected to happen:
As per the documentation, mongodb service should be installed and must be exposed (after setting up nginx-ingress
How to reproduce it (as minimally and precisely as possible):
values.yaml
mongodbRootPassword: admin
mongodbUsername: admin
mongodbPassword: admin
mongodbDatabase: twitter_stream
service.name: mongodb
persistence.storageClass: gp2
persistence.size: 1G
ingress:
  enabled: true
  annotations:
    kubernetes.io/ingress.class: nginx
    nginx.ingress.kubernetes.io/rewrite-target: /
  hosts:
  - name: mongodb.local
    path: '/'

helm install --name mongodb -f values.yaml  stable/mongodb --tiller-namespace amq-streams

**Full Debug Log **
$ helm install --name mongodb -f values_mongodb.yaml  stable/mongodb --tiller-namespace amq-streams --debug
[debug] Created tunnel using local port: '58185'

[debug] SERVER: "<IP_ADDRESS>:58185"

[debug] Original chart version: ""
[debug] Fetched stable/mongodb to /Users/karasing/.helm/cache/archive/mongodb-5.16.1.tgz

[debug] CHART PATH: /Users/karasing/.helm/cache/archive/mongodb-5.16.1.tgz

Error: release mongodb failed: Ingress.extensions "mongodb" is invalid: spec.rules[0].host: Invalid value: "map[name:mongodb.local path:/]": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
$

Hi @ksingh7 ,
I am not able to reproduce. Could you share your values.yml file?
By how the yaml file is rendered, it seems that it is not breaking the line - name: monodb.local.
Could make sure your indentation is right and you don't have any special character in that line ?
I'd suggest to delete both lines and rewrite them by hand (not copying and pasting).


How to "track" async calls?

I have a series of links I am trying to analyze the redirection for. a simple 301 redirect is not working so using playwright
I know I can do page.url to get the final url in a hook, but I need a way to track original URL and final URL for each link i have.
How can I pass/store this metadata in the hooks?

Hi @b-sai , thx for trying crawl4ai. You can store custom metadata as kwargs when triggering your hooks, and then retrieve or modify them inside the hook callback. For example, you can include something like original_url=... in the execute_hook() call before navigation, and then read or update the final URL in after_goto or on_execution_started hooks. The hooks support arbitrary keyword arguments, so you can pass a dictionary or extra parameters containing the original URL.
For instance:
async def before_goto_hook(page, context=None, **kwargs):
    # kwargs might contain original_url and session_id etc.
    original_url = kwargs.get("original_url")
    # Store original_url somewhere if needed, or print
    print(f"Original URL: {original_url}")

async def after_goto_hook(page, context=None, **kwargs):
    original_url = kwargs.get("original_url")
    final_url = page.url
    print(f"Original URL: {original_url}, Final URL: {final_url}")
    # You can return these values or store them globally

crawler_strategy.set_hook('before_goto', before_goto_hook)
crawler_strategy.set_hook('after_goto', after_goto_hook)

# When calling your crawl method:
await crawler_strategy.execute_hook('before_goto', page, context=context, original_url="http://example.com")
await page.goto("http://example.com")
await crawler_strategy.execute_hook('after_goto', page, context=context, original_url="http://example.com")

By doing this, you are free to pass original_url or any other metadata you need through the execute_hook() calls. Each hook gets the kwargs so you can store and retrieve the needed information across the hooks.
Another approach is using sessions_id. Use session_id to maintain state for each URL:
# Pass metadata through session IDs
session_id = await crawler.create_session()
result = await crawler.arun(
    url=original_url,
    session_id=session_id,
    before_goto=lambda page: store_original_url(page, original_url),
    after_goto=lambda page: store_final_url(page, page.url)
)

Hopefully this provide the help you need.


python312Packages.tencentcloud-sdk-python: 3.0.1128 -> 3.0.1129

Diff: https://github.com/TencentCloud/tencentcloud-sdk-python/compare/refs/tags/3.0.1128...3.0.1129
Changelog: https://github.com/TencentCloud/tencentcloud-sdk-python/blob/3.0.1129/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 304194 run on x86_64-linux 1

4 packages built:

python311Packages.tencentcloud-sdk-python
python311Packages.tencentcloud-sdk-python.dist
python312Packages.tencentcloud-sdk-python
python312Packages.tencentcloud-sdk-python.dist


RUM-6945: Add Tab fragment in sample application for Session Replay

What does this PR do?
With current version of SR, we don't correctly show the background of the bottom navigation view tabs,
Thus we need a showcase screen in sample app to test if the tabs can show correctly in SR.
Motivation
RUM-6945
Demo

### Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

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

Project coverage is 70.34%. Comparing base (72455be) to head (c6a353a).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2356      +/-   ##
===========================================
- Coverage    70.35%   70.34%   -0.01%
===========================================
  Files          741      741
  Lines        27645    27645
  Branches      4624     4624
===========================================
- Hits         19448    19446       -2
+ Misses        6910     6904       -6
- Partials      1287     1295       +8

see 31 files with indirect coverage changes


Adding Gemfile.lock to .gitignore

This change is

LGTM.
nit: how come your files never have a trailing line break? it makes github unhappy.

Fixed. Should I make sure all files have a trailing line break going forward. Didn't know github cared.

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Configure a managed instance as a replication distributor. Doc Error

In this tutorial, you learn how to:
Configure a managed instance as a replication publisher and distributor.
Configure a managed instance as a replication distributor.
The 'Configure a managed instance as a replication distributor.' part should be replication subscriber.

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

ID: d7cd1942-8ac7-eae9-4ac8-eb716ecd7b7b
Version Independent ID: 559730f0-71fd-b16c-ecac-56954d5f2f64
Content: Configure replication between managed instances - Azure SQL Managed Instance
Content Source: articles/azure-sql/managed-instance/replication-between-two-instances-configure-tutorial.md
Service: sql-managed-instance
Sub-service: data-movement
GitHub Login: @MashaMSFT
Microsoft Alias: ferno

Thank you for your feedback! We will review and update as appropriate.

Hi @nonokangwei , thank you so much for your feedback! Thanks for a catching this typo :) I have processed a change to fix it that should be live later this week.
I'll close out this Git Issue for the time being but please feel free to comment should you need anything further.
Thanks again for improving our docs and I hope you have a wonderful rest of your day!
Masha from the SQL Docs team
#please-close


Preview Loaded and then Process Exited

Hi can't reliably see xaml preview, the view shown for just a moment and then display warning process exited

And i checked the output window, this is shown :
10:12:41.477 [Error]  Connection error
System.IO.IOException: Unable to read data from the transport connection: An existing connection was forcibly closed by the remote host. ---> System.Net.Sockets.SocketException: An existing connection was forcibly closed by the remote host
at System.Net.Sockets.Socket.EndReceive(IAsyncResult asyncResult)
at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult)
--- End of inner exception stack trace ---
at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult)
at System.IO.Stream.<>c.b__43_1(Stream stream, IAsyncResult asyncResult)
at System.Threading.Tasks.TaskFactory1.FromAsyncTrimPromise1.Complete(TInstance thisRef, Func`3 endMethod, IAsyncResult asyncResult, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Avalonia.Remote.Protocol.BsonStreamTransportConnection.d__15.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Avalonia.Remote.Protocol.BsonStreamTransportConnection.d__16.MoveNext()
already done, clean, rebuild and the problem persist, what should i do?.

Most likely a crash after XAML was loaded. Those aren't really handled by the previewer.
Try adding
while(!Debugger.IsAttached)
   Thread.Sleep(100);

to you control's constructor and attaching debugger to the previewer process from another Visual Studio instance (the same workflow as with debugging issues with WPF's previewer basically)

Okay, will try and look into it much further, thanks.


How to handle error?

Describe the feature
When mounting multiple storage, if one of the mountpoint throw an error, it breaks all.
Let's say I mount a fs storage and http storage. If http is down, I can't get only the fs storage. storage.getKeys() throw an error.
How to prevent this and obtain the fs storage anyway?
Additional information

[ ] Would you be willing to help implement this feature?

In fact, if the driver doesn't throw an error, calling storage.getKeys() return well the others mounted storages.
I tried to comment the error here for the gitlab driver and it works.
Should we throw an error ?


Add components RFC

Proposes the initial high-level structure of the project and the layout
of the source code.
Signed-off-by: Shawn T. Amundson<EMAIL_ADDRESS>

We're 12+ months on this PR. Stale or re-read or ❓

We're 12+ months on this PR. Stale or re-read or ❓

I'm planning to update this RFC soon (prior to a Grid 0.1 release), because it would be good in context of the roadmap - https://grid.hyperledger.org/community/roadmap.html.

Portions of this are probably still relevant to what made it into Grid 0.1, but since I didn't push this through before that release, it does lose some value as an RFC. Will close and we can use it for reference if desired. Otherwise, we can just cover these topics in the Grid documentation directly.

Portions of this are probably still relevant to what made it into Grid 0.1, but since I didn't push this through before that release, it does lose some value as an RFC. Will close and we can use it for reference if desired. Otherwise, we can just cover these topics in the Grid documentation directly.


Move parser to a separate package

To begin the process of compartmentalizing this project, we should move the parser into its own npm package. The package should provide an interface that consumes yarn data and outputs an ast (theoretically standardized by #18), then we can simply include that package in this project.
We should wait until after #17 to do this

Gonna call this unimportant for now but something to consider in the future


Ability to return modified file content for each read

I am testing an application what calculates machine usage by reading the /proc/stat file, I can easily mock it but the problem I ran into is that the app needs modified values for each request (its read in a thread in a 1 sec interval), I could not find an easy way to control what is being returned for each reads besides calling SetContent in another thread on the FakeFile. Any other way to do that? like a passing a callback or something similar?
Thanks for the help

I think there is nothing that that would help you with this in pyfakefs, though I think this is a little out of scope for the library. You may have to think about your testing strategy:

do you really want to use multi-threading in unit tests (note that we do not aim at thread-safety), or could you just split the testing into smaller parts that do not need it?
can you just mock out the timer to be able to modify the files in the mock out? that could also decrease the test time as you don't have to wait a full second for each change
if you want to do something like an integration test, pyfakefs may not be the best choice - in that case it is better to use something like a RAM disk that can be cleared after each test

Just my 2 cents - without knowing your application it is difficult to get more specific.

@mrbean-bremen thank you for your answer, unfortunately this is unit level, the method is part of an snmp library and how it is written the whole file reading and calculating logic is in the run method of a Thread subclass. I do not really want to refactor it now for testing, but indeed that is maybe the best solution.
I hoped stubbing consecutive reads is part of the library but maybe in the future can be part of it?

I may not understand that correctly - what do mean by "stubbing consecutive reads"?
 Pyfakefs is just about replacing the real fs with the fake fs. Can you elaborate a little about the feature you want to see?

thanks, sorry this is my first time to use pyfakefs so maybe everything is there. I do not know how can I stub out or what exactly I need to stub.
My usecase is, I have a class CpuStats which has a static function getIdle() what returns with a number. The function itself is implementing a moving average with an alternate approach, so it needs the previous value and the current to do the calculation. It is implemented in a way that it reads the /proc/stat in a thread independently from my unit test (async). If the content of the file does not change  I will get ZeroDivisionError (as in real life the jiffies are always increased in that file), because it uses the differences in the jiffies in the calculation. My naive approach was use a function what returns the file content for each call so I can make sure its always valid:
def generate_cpu_stat():
   # store some state here (amount of calls, test data etc)
   return 'cpu  %s %s %s %s 106698 0 3117 0 0 0' % (USR, NICE, SYS, IDLE)
file = fs.CreateFile('/proc/stat', content=generate_cpu_stat)

I tried to monkey patch FakeFile#contents but it does not seem to work, it seems only the initial content is being returned ( I tried subclassing FakeFile and setting that class in my instance, or overriding the FakeFile.contents in the test script, as its a property I cannot override the instance attribute itself)

monkey patching byte_contents also wont work as it will set it for FakeDirectory as well FakeDirectory#GetEntry calls contents what is calling byte_contents

Thanks - I will have another look at this in the evening.

Ok, the question here is what you want to test in your unit tests.
If you want to test the algorithms used in your static function, I would split that out and provide it with the data as a parameter - then you can run it in a loop, which provides the data you want (without a thread, and without a delay). Your function then has only to read the data, pass it to the algorithm and return the result.
If you want to test reading the data, you can test that separately, but then again, I'm only guessing...
Depending on what you really want to test, you may not need pyfakefs at all - as sorry as I would be about this ;)

I can understand, its a legacy code what I do not really want to change, I will think about it, thanks for your help

As an aside, in case you haven't read it - some time ago I read Michael Feather's "Working effectively with legacy code". It has a lot of patterns how to unit test legacy code, and at least for me it was very helpful.

will check, thanks!

@kalidasya - can this be closed, or do you need more help?

can be closed


[BUG] Set-PnPMicrosoft365GroupSettings is returning type conversion error

Reporting an Issue or Missing Feature
When calling Set-PnPMicrosoft365GroupSettings -Identity $groupSettingId -Values @{"AllowToAddGuests"="true"} -Group $groupId I get the below error:
Set-PnPMicrosoft365GroupSettings: Cannot implicitly convert type 'void' to 'object'.
I have verified both the $groupSettingId and $groupId values and they exist.
In fact, I could create the group setting on the same $groupId using New-PnPMicrosoft365GroupSettings just a couple of minutes earlier, but with -Values @{"AllowToAddGuests"="true"}.
Besides using the id's for the group and group setting, I have also tried using the entire group object as well as the group setting object with no difference.
Workaround right now is to use Remove-PnPMicrosoft365GroupSettings -Identity $groupSetting.Id -Group $groupId -Connection $conn which works fine, after which the original New-PnPMicrosoft365GroupSettings can be run but with the new Values hash table.
Expected behavior
That the group setting to be updated.
Actual behavior
Receiving the error: Set-PnPMicrosoft365GroupSettings: Cannot implicitly convert type 'void' to 'object'
Steps to reproduce behavior
Run cmdlet
What is the version of the Cmdlet module you are running?
2.12.0
Which operating system/environment are you running PnP PowerShell on?

[x] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

Pull request opened at https://github.com/pnp/powershell/pull/4274

Hello @mblennegard , this issue has been fixed and it will be available in tomorrow's nightly builds. Thanks for reporting.

I reproduced the issue and I'm preparing a PR to fix it

That was quick work of that one.
Awesome!


[Bug]: Temperature broken on OpenAI LLM

Bug Description
Setting the temperature on OpenAI LLM is broken.
When I set any float value using the temperature arg or the default value, it is converted to an int.
The bug is here in openai.py line 42: temperature: int = Field(description="The tempature to use during generation.")
The pydantic type should be float in the class definition.
Version
0.8.8
Steps to Reproduce
from llama_index.llms import OpenAI
llm = OpenAI(temperature=0.5)
print(llm)
Relevant Logs/Tracbacks
callback_manager=<llama_index.callbacks.base.CallbackManager object at 0x10101c350> model='gpt-3.5-turbo' temperature=0 max_tokens=None additional_kwargs={} max_retries=10

huh, could have sworn I fixed this. Will make a PR shortly

Thanks Logan!


[5.1] Make an alias for count(*) in order to access it from order by clause

This commit helps you to sort query results based on count of OneToMany relation.
Example:
Comment model
// app\comment.php
public function post(){
  return $this->belongsTo('App\Post');
}

This is Post model
// app\post.php
public function comments(){
  return $this->hasMany('App\Comment');
}

// get top posts based on count of comments using ORM
public function scopeTopPosts($query){
  return $query->whereHas('comments', function($query) {
    $query->latest('aggregate');
  });
}

This would cause an issue if someone had a field named aggregate in their schema wouldn't it? Perhaps it would be a good idea to namespace it somehow?

@tomschlick is right. In addition, that "" character is specific to MySQL. We would need to use the query grammar's wrap` function to wrap it in the proper quote characters for the database system being used. Also if you want to make these changes could you submit this to master branch?


The following assertion was thrown while handling a gesture: Assertion failed: file:///D:/flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.18.1/lib/audio_cache.dart:49:15 !kIsWeb "AudioCache is not available for Flutter Web."

When the exception was thrown, this was the stack:
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/internal/js_dev_runtime/private/ddc_runtime/errors.dart 236:49  throw
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 29:3    assertFailed
packages/audioplayers/audio_cache.dart 49:15                                                                               new
packages/xylophone/main.dart 17:32
packages/flutter/src/material/ink_well.dart 991:14                                                                         [_handleTap]
...
Handler: "onTap"
Recognizer: TapGestureRecognizer#63710
debugOwner: GestureDetector
state: possible
won arena
finalPosition: Offset(537.3, 282.0)
finalLocalPosition: Offset(59.3, 9.3)
button: 1
sent tap down

Like the error says, AudioCache is not available on web. Check the readme for more info.
And please, follow the template when opening issues.

Like the error says, AudioCache is not available on web. Check the readme for more info.
And please, follow the template when opening issues.


Does not work with ATS 1.49

Using v<IP_ADDRESS>
I get a warning when trying to load a save from ATS 1.49.
Savefile version 74 is NOT TESTED.

Why was this closed? Is there a new version that makes this work with 1.49? I'm confused.

New version of Game with new features. Most probably it won't work.
And new version of TS SET was tested and will be released today.


fix: makes args optional on get members and invitations fns

Description

makes args optional on get members and invitations fns

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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

v1.27.1
GitHub release

Your semantic-release bot :package::rocket:


Eq 1114 make question skip conditions consistent with groups

What is the context of this PR?
Allow for Questions to be skipped with an array of possible (or) conditions in line with those used for Groups and Blocks.
Allow for the summary page to check if a question has been skipped by metadata values.\
How to review
The test_skip_condition_question schema has been created with the new style skip array. On the first page you can select Yes to either/both of the skip questions. If yes is selected on either then the second question will not be present on the next page. You can also set your user_id to "Skip" and that will skip the second question based on metadata.

I think it would be good to get together with @adoublebarrel tomorrow to review the changes to ensure we don't lose any of the escaping fixes he recently added.

@dwyeradam squash?


[C-2042] Improve Slack bot installation warning

Describe the problem
Currently, after users connect Slack integration, we ask to also connect the Slack bot as a required step to fetch data from Slack channels. We have reports of users missing/not acknowledging this step, and we can assume that the current warning message (banner in Integrations page) is not prominent enough.
Describe the improvement
Change the current warning banner on top of Integrations page with a dialog, including a CTA redirecting users to Docs page. The dialog is prompted when users return from Slack auth page and is only dismissed when users hit the CTA button (link opens in the tab).

From SyncLinear.com | C-2042

Couldn't we add a screen where somebody adds<EMAIL_ADDRESS>to their Slack? Would be cool to make it more visual

@nunoeufrasio

@jonathimer when you mention a screen would it be some visuals inside the modal?
To be honest I think having a CTA that leads to a page with the instructions should do the trick since it's a connection occurring outside our app and slack's instructions should be pretty clear

Yes! Thats fantastic. 🙌


Fix links to ziglang.org/documentation

It looks like the recent autodoc update got rid of the prefix A; for API docs, so the links from zig.guide were all landing on a page saying "Declaration not found." This PR updates the links to the new format.

Thanks!
Merged in da4a7d723a2967190d03ef89499d55a790d5075f (had to fix a conflict).


Add support for V8 debugger in v8eval

Take 3. May the force be with you, pull request.

Thank you for your contribution and the quick fix of build errors.


Duplicate Themes on Import

Frodo CLI version
Provide output of frodo -v
cli: v0.15.1
lib: v0.13.2-0
node: v16.15.1
Describe the issue
When doing an import of either journey or theme into Identity Cloud, existing Themes are not over-written.  Rather, a duplicate theme is created.

The underlying issue is reported and tracked here: FRAAS-13391
Closing as there is nothing for Frodo to do.


install.py scipy issue on rpi3b+

Hello.  I'm getting this error on a Raspberry Pi 3b+.  I did a manual build of scipy and still had issues.  This was the original error from install.py
/_lib/_uarray
running build_clib
customize UnixCCompiler
customize UnixCCompiler using build_clib
building 'mach' library
error: library mach has Fortran sources but no Fortran compiler found
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for scipy
Failed to build scipy
ERROR: Could not build wheels for scipy, which is required to install pyproject.toml-based projects
================== Completed Installing Numpy, Scipy, PyAudio, PyQtgraph ==================

the issue was with port audio.  I had to increase swap and manually build it, then install pyaudio 1.5.2 and then it worked
I also installed it on an raspberry pi 4 and had the same problem

Hi there rekk2! Im having the same error as you and im using a RPi3b+, would you be more accurate on what you did to solve the issue? Thanks in advance

I think i used this guide to increase the swap
https://arcanesciencelab.wordpress.com/2019/08/03/increasing-swapfile-space-on-the-raspberry-pi-4/
and then install version 1.5.2 of pyaudio because the others didnt work
On Thu, Jul 21, 2022, 5:13 PM rox4H @.***> wrote:

Hi there rekk2! Im having the same error as you and im using a RPi3b+,
would you be more accurate on what you did to solve the issue? Thanks in
advance
—
Reply to this email directly, view it on GitHub
https://github.com/naztronaut/dancyPi-audio-reactive-led/issues/63#issuecomment-1191941188,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVRLUKAAZES5ETIXWOQRHDVVG4PZANCNFSM5UYUUGSQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you for answering so quickly! I have increased the swap but I can't see how to download version 1.5.2 of pyaudio, according to https://people.csail.mit.edu/hubert/pyaudio/ the newest version is 0.2.12. Could it be that you are referring to version 1.5.2 of scipy? Thanks again!

Yes i forget exactly what i did.  Something only worked with version 1.5.2
On Thu, Jul 21, 2022, 6:11 PM rox4H @.***> wrote:

Thank you for answering so quickly! I have increased the swap but I can't
see how to download version 1.5.2 of pyaudio, according to
https://people.csail.mit.edu/hubert/pyaudio/ the newest version is
0.2.12. Could it be that you are referring to version 1.5.2 of scipy?
Thanks again!
—
Reply to this email directly, view it on GitHub
https://github.com/naztronaut/dancyPi-audio-reactive-led/issues/63#issuecomment-1191984541,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHVRLUI7IX5MDZDVEPUX5VDVVHDIBANCNFSM5UYUUGSQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


kitchen can't start classicui-debian-85

When creating all kitchen's with kitchen create the following error happens.
kitchen create classicui-debian-85
-----> Starting Kitchen (v1.14.2)
-----> Creating <classicui-debian-85>...
       Bringing machine 'default' up with 'virtualbox' provider...
       ==> default: Checking if box 'bento/debian-8.5' is up to date...
       ==> default: Fixed port collision for 80 => 8080. Now on port 2250.
       Vagrant found a port collision for the specified port and virtual machine.
       While this port was marked to be auto-corrected, the ports in the
       auto-correction range are all also used.

       VM: default
       Forwarded port: 443 => 8443
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: 1 actions failed.
>>>>>>     Failed to complete #create action: [Expected process to exit with [0], but received '1'
---- Begin output of vagrant up --no-provision --provider virtualbox ----
STDOUT: Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'bento/debian-8.5' is up to date...
==> default: Fixed port collision for 80 => 8080. Now on port 2250.
STDERR: Vagrant found a port collision for the specified port and virtual machine.
While this port was marked to be auto-corrected, the ports in the
auto-correction range are all also used.

VM: default
Forwarded port: 443 => 8443
---- End output of vagrant up --no-provision --provider virtualbox ----
Ran vagrant up --no-provision --provider virtualbox returned 1] on classicui-debian-85
>>>>>> ----------------------
>>>>>> Please see .kitchen/logs/kitchen.log for more details
>>>>>> Also try running `kitchen diagnose --all` for configuration

@dpnl87 Looks like vagrant was unable to resolve the port conflicts.
However, updated .kitchen.yml file to define custom ports.

@dpnl87 Please reopen if still facing the issue. Closing.


Format descriptions

This branch

adds links to some types and structs,
formats texts,
modifies descriptions in some Panics sections to clarify what users must specify.

@kampersanda Could you approve again?


Config option to revert back to old lighting

Hello there.
I have used your mod for a couple of weeks now and I have really enjoyed seeing the progression and updates it has been getting. However, I noticed that in a recent update (possibly the 1.1.0 update) you changed the planet lighting direction and strength because of another previous issue request. If its possible, I was wondering if you could add a config option to allow players to change the lighting settings back to what they were previously before the update. (Or maybe an option to have lights on each side of the ship? As one side is quite dark now)
I appreciate all the work you do for this mod and I'm sorry to bug you with this silly change. Myself and others would be very grateful if you could do this. Thank you!

ay its fine ites expected people might prefer one over the other i can easily do that just one thing before i publish the update how would u like that config to be saved?
saved on your profile or on your computer so you don't have to redo it every time

Saving it on the profile works just fine, simply having the option to have change the config would be amazing! Thank you so much.

allrighty good to know.
just letting you know i am still working on it. im just getting some werid errors so im fixing that first

allrighty good to know.
just letting you know i am still working on it. im just getting some werid errors so im fixing that first

added option in 1.2.0


Fix: carousel flickering when going fast to left side

Fix for #144

@pauliusguzas this can be tested with feat version 1.1.0-feat.recyclingleft.0


CLI: Interface for Stealify

compile, run, debug

accepts stdio pipe via args - (uses fd 1 and 2) at the end or --inspector-pipe (uses fd 3 and 4)
accepts path to a Stealify Supported file with a Stealify Supported Extension
accepts here doc with stealify via putting - as the end args syntax also uses -f to specifie a file

Advanced Concepts
Nix heredoc
echo -e 'FROM busybox\nRUN echo "hello world"' | docker build -

// Equals too
docker build -<<EOF
FROM busybox
RUN echo "hello world"
EOF

Windows bat heredoc support
replace your prog for example with more in the following example.
(@echo.bla
@echo.bla
) | yourprog.exe

> out.txt (
    @echo.bla
    @echo.bla
    ...
)

Should get implemented as Resolver Loader at its core supporting our Stealify Stream Implementation. As that is the best we can come up with to declare builds.


Include timing information (and more!) in log

Right now the logs are pretty sparse.
We should include at least what services got called, how long they ran, and some sense of the size of the components in and out.

Timing is now represented in the logs


Fix Ok button not working on Prompts

Fixes #329
The ok button on the edit room name prompt wasn't doing anything.

I'm going to fix this in a different way, by wrapping form around the button

@benawad I can make the changes now

I just did it


Refactor BIP21 (onchain QR code) encoding and add tests

Don't duplicate URI constructing code in node.bitcoind and node.xpub, move that to node.bip21. Code is based on the old code I wrote for JoinMarket (but we are only interested in encoding, not decoding). Test vectors are from BIP21 doc.

Rebased to have GitHub Actions CI run, fixed found flake8 issues (remaining ones are covered in #107).

Also added type hints here.


To-do list

[x] Fix Alien::gdal installation
[x] Use tagged version of biodiverse-utils
[x] Verify that location of Biodiverse scripts is added to the PATH

Alien::gdal was updated to r1.27

Verify that location of Biodiverse scripts is added to the PATH

Environmental variable PERL5LIB should be specified, e.g.:
export PERL5LIB="/biodiverse/lib"

Added in 4b581085.


Playback segment error - following Single Analysis.

Describe the bug
When you are finished analyzing (single analysis), and you have got your results - you can click on each call event and listen to it. Unfortunately, the calling segment does not line up with the call time, and you hear a totally different time segment.
To Reproduce
Steps to reproduce the behavior:
For the GUI:

In Single Analysis, load a WAV file
Go to the results from "Analyze"
Click on each time segment to listen to it
Error - a different time segment plays. Verified using Audacity to check the segment.

Expected behavior
The incorrect segment plays, causing confusion.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Mac OS
Version 15.1

Yes, you are right, there is a formatting error, thanks for reporting!

Just released a new version that should include the fix.

Hi – maybe you can help me with some recommendations on analyzing a short recording? It’s from the Isiola region in Kenya (northern Kenya), and you can easily hear a red-chested Cuckoo (~2000 hz) and a ring-necked dove (800 Hz).  I used species by location (37E, 0N). I have attached the recording. I used Wildlife Acoustics Micro.
When I run Birdnet analyzer, the cuckoo gets easily picked up (high confidence) but the ring-necked dove does not (only at the very end). Any suggestions on how to tweak the settings to both get picked up? Is it because the dove is calling at the same time as the cuckoo (and is a little softer)? Maybe using the min and max bandpass frequency would help?
[there are a few other species – like rattling cisticola (30s – 5000 hz), diderik cuckoo at 7000 Hz 0 – and other species  - but softer.]
Also, please explain a little what changing sensitivity does?
Thanks for your help!
Tim
Cheers
Tim
From: Tim Boucher @.>
Date: Tuesday, December 17, 2024 at 8:03 AM
To: kahst/BirdNET-Analyzer @.>
Subject: Re: [kahst/BirdNET-Analyzer] Playback segment error - following Single Analysis. (Issue #528)
When will the new/corrected version be posted?
Thanks!
Cheers
Tim
From: Josef Haupt @.>
Date: Tuesday, December 17, 2024 at 8:01 AM
To: kahst/BirdNET-Analyzer @.>
Cc: Tim Boucher @.>, Author @.>
Subject: Re: [kahst/BirdNET-Analyzer] Playback segment error - following Single Analysis. (Issue #528)
Closed #528https://github.com/kahst/BirdNET-Analyzer/issues/528 as completed via #530https://github.com/kahst/BirdNET-Analyzer/pull/530.
—
Reply to this email directly, view it on GitHubhttps://github.com/kahst/BirdNET-Analyzer/issues/528#event-15685746114, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AL42XWVPDX5ZEXMNCEA2I7D2GAOBVAVCNFSM6AAAAABTXAG3UOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGY4DKNZUGYYTCNA.
You are receiving this because you authored the thread.Message ID: @.***>


[Question] Version Sets for Supporting Independent API Teams

Release version
APIOPS 5.1.4
Question Details
We want to target specific API's, Products using APIOPS pipelines. At the moment there are no way to specify related to them version sets. Couple of the questions:
What will be impact if we push from Dev to higher environments all the version sets without some of the related API's?
Is there a way to restrict to specific versions sets related to the API that we target? It is guid in the repo that we find hard to target.
Expected behavior
Publish only specific versions sets related to API
Actual behavior
All versions sets are published.
Reproduction Steps
Ryn APIOPS 5.1.4 Extractor and publisher.

Any update on the question?

We've added support for this in our latest v6 release. If you filter by API, version sets should also respect that filter.


How to camelCase relationships

Our relationships are being returned as dashed (kebab) style but devour doesn't appear to camelCase them as it typically does for attributes. Therefore, we're having to define our models like this:
jsonApi.define('itinerary', {
  'conversation-group': {
    jsonApi: 'hasOne',
    type: 'conversation-groups',
  },
})

Any suggestions how we could define it like this instead?
jsonApi.define('itinerary', {
  conversationGroup: {
    jsonApi: 'hasOne',
    type: 'conversation-groups',
  },
})

Thanks!

Hi guys! Great library!
I'm having the same issue, is there any plan to fix this?

@luisgabrielroldan In the meantime, you can get around this by defining your own middleware to transform the keys

@luisgabrielroldan, @axsuul
I noticed there is a PR for this now: https://github.com/twg/devour/pull/174
It would be nice to get that merged :)


TestAPI Module Request

Some testing classes can be consolidated and shared across different modules in NB.
We can gather the most commonly used test APIs in a TestAPIs module.
This issue has come from the need to re-use the ProcessInvoker and ProcessResult classes in different modules. (see issue #1084 )

The nb5-proof module is currently disabled in nb521 pom.xml
@MikeYaacoubStax is this a problem?


fix(ENG-878): introduce husky and commit lint

Description

Introduce Husky
Introduce Conventional Commit Lint

Issue(s)
ENG-878
How to test

Make a commit, or run:

 ~/src/oak/oak-terraform-actions  ENG-878                                                                                   22.6.0  12:52:48
❯ echo "lol this is wrong" | commitlint
⧗   input: lol this is wrong
✖   subject may not be empty [subject-empty]
✖   type may not be empty [type-empty]

✖   found 2 problems, 0 warnings
ⓘ   Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint

Relies on #3 being merged to fix issues surrounding the linting of the yml files themselves.

Are we planning on adding semantic release too? Or was that the question around versioning?

Good question! I'll bang in another task for that to pick up rather than that as not sure how it's done just yet.


Replace "œ" by "oe" in restrict-filename mode

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.05.10. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.05.10

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2016.05.10
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue required an account credentials please provide them or explain how one can obtain them.

Hi,
Can you replace "œ" by "oe" in restrict-filename mode :
$ youtube-dl --version
2016.05.10
$ youtube-dl -f 18 -e IYGUF2d5Azc
Connaitre le cœur du père pour gérer la famille (Shora KUETU - 03/03/2015)

The command line youtube-dl -f 18 --get-filename --restrict-filename IYGUF2d5Azc would return :
Connaitre_le_coeur_du_pere_pour_gerer_la_famille_Shora_KUETU_-_03_03_2015__18__IYGUF2d5Azc.mp4

instead of :
Connaitre_le_c_ur_du_pere_pour_gerer_la_famille_Shora_KUETU_-_03_03_2015__18__IYGUF2d5Azc.mp4

This is current mapping:
ACCENT_CHARS = dict(zip('ÂÃÄÀÁÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞßàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿ',
                        itertools.chain('AAAAAA', ['AE'], 'CEEEEIIIIDNOOOOOOUUUUYP', ['ss'],
                                        'aaaaaa', ['ae'], 'ceeeeiiiionoooooouuuuypy')))

Is there anything missing besides œ? I hope to include all possible characters this time.

Let me check on my keyboard layout (which is French Alternative) ...

Ok, it seems fine but you can also add it's capital counterpart : Œ

I hope to include all possible characters this time.

there are a lot of Accented Characters most of them are not widely used.
https://en.wikipedia.org/wiki/List_of_Unicode_characters

This table does not list replacements for all characters, in which native speakers' help is necessary.

Thanks @sebma for the information. The fix will be included in the next version.

Thank you for you reactivity.

looks like there are lots of libraries specialized in the transformation of strings, some of them extend the rule for other languages and even for symbols.
examples:
https://github.com/cocur/slugify/blob/master/src/RuleProvider/DefaultRuleProvider.php
https://github.com/dodo/node-slug/blob/master/slug.js

Seems great, but both use MIT license.


corrected the spelling of directly.

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

Line 384: [Error] Error happened in build.entrypoint.ps1. Please build your target branch first and then reopen this pull request to do another round of verification. If the issue still happens, open a ticket in http://SiteHelp and include the following error message for our team to troubleshoot: Method invocation failed because [System.Collections.Hashtable] does not contain a method named 'Kill'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@dkthedon thanks for catching and fixing that. There was an issue with build validation here. I created a new PR with the same fix and merged it https://github.com/NuGet/docs.microsoft.com-nuget/pull/968


cmd ref: improvements to params

You may disregard these recommendations if you used the Edit on GitHub button from dvc.org to improve a doc in place.

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.
Please choose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏

The intention is that you get to the info. on how to create params faster (dvc run -p). I think that is more important than the other important explanations I moved under. There's really no other doc that covers param creation except perhaps a passage in Get Started.

and the example in dvc run

But those two paragraphs look strange in "benefits and workflow" section

May be we can just remove that sub-section

I removed it. I also reorganized paragraphs again (closer to the original).
I feel like this description is too long though. But that's kind of out of scope right now.

Oops sorry just after I said the scope was limited I went ahead and updated this whole doc for 1.0!


[Harvester] rancher-integration suggests docker run as way to install Rancher

Summary:
Ref: https://docs.harvesterhci.io/v1.0/rancher/rancher-integration/
Installing Rancher on a cluster in a docker container like this is VERY bad practice and leads to failed and lost environments and easily avoidable customer tickets which can sometimes have no path of recovery for the cluster.  This is not supported by us and should suggest good practices of NOT running everything in a docker container, perhaps referring to the methods in the Rancher docs (where they are actually good practice.  Even as a quick start, this is bad practice and if it must be the method in this doc, we should also make it clear that his is ONLY a quick start method and should be avoided in anything that is meant to live for any length of time outside of a quick test.
Details:
Install Rancher on K8s without further external encapsulation.  The k8s environment should NOT be ephemeral!

Closing as this was transferred to the Harvest Docs already.


no module named malmopython

When execute pig_chase_baseline.py
OpenCV found, setting as default backend.
Traceback (most recent call last):
  File "pig_chase_baseline.py", line 38, in <module>
    from environment import PigChaseEnvironment, PigChaseSymbolicStateBuilder
  File "/home/tyler/Software/Malmo/malmo-challenge/ai_challenge/pig_chase/environment.py", line 27, in <module>
    from MalmoPython import MissionSpec
ImportError: No module named MalmoPython

But nothing happened for malmo examples.

Ah, you may need to add MalmoPython to your PYTHONPATH -  or copy (eg) MalmoPython.so into the folder where you are running pig_chase_baseline.py.
Let me know if this works and I'll update the installation instructions accordingly.

copy (eg) MalmoPython.so into the folder where you are running pig_chase_baseline.py.

Thank you Davey. After copy the file, it works.
The other problem is that sometimes the python file cannort find the server, even though I have already started the servers with appropriate ports.

Hi @onlytailei,
thank you for reporting back - I'm glad the proposed solution worked for you. We will update the documentation to reflect this.
For the server problems - could you create a new issue? Please include details on the platform / setup, and any error / output you receive to help us replicate and resolve this.


Move files under "font" folder to "fonts"

I think it's necessary to move "font" folder to "fonts".
Since the most font icon files located under "fonts" folder, it would create 2 similar folder ("font" and "fonts") when using them along with Materialize.

+1 had to create an extra gulp task for it.

fixed in a1ca0a997efb692a27fcfa85a7cc5277b24c0e57


Add optional constructor to change the result-destination

Problem
It is not possible to specify the result-destination for ServiceX Submits. It always defaults to object-store
Solution to #215
Approach
Added a new argument to the ServiceX constructor. Use this in the invocation of ServiceX Transform

Looks great and I think is ready to merge!


Connect to self-hosted memos server

How do I connect this to sync with my memos server?
App looks great. Thanks for this.

@cfredericksen I'll release the version on the weekend haha

Merging the PR?

@cfredericksen  This merge request is slightly problematic and is being processed.

How can I use the Windows client with my self-hosted environment?

@cfredericksen This merge request is slightly problematic and is being processed.

any updates since May?


issue for send config insecureAuth ?

Hi
I wish to send insecureAuth, but dont get how to do this ?
Connection error! MySQL server is requesting the old and insecure pre-4.1 auth mechanism. Upgrade the user password or use the {insecureAuth: true} option.

In the new version (2.2.3) I added a checkbox that allows for insecureAuth.
I hope this helps you at work :)


What if something is a callback/async function AND a Promise

Working with request-promise ;
e.g.
var rp = require('request-promise');
var Brakes = require('brakes');

var options = {
  uri: 'http://some-json-api.com/',
  json: true
};

var brake = new Brakes(request, { timeOut: 1000 });

return brake.exec(options)
  .then( response => {
    return processResponse(response);
  })
  .catch(e => {
    return processError(e.statusCode);
  });

request-promise wraps the normal request object, which is a function that has a callback parameter, and makes it return a promise. Wrapping it causes the behaviour to change because Circuit prefers async callback functions ; instead of the usual behaviour with these options which is for response to be an object representing the processed JSON response, and for errors to end up in .catch, all responses including errors end up in the .then() block because Circuit uses the default promise callback style instead of request-promise's Promise wrapping.
Not sure the best way to resolve this but maybe an option to force the Promise treatment instead of using the callback style? Not a big thing to write code to work around this (and ditch request-promise in the process, since we're not using it's features).

Ok, workaround :
var rpshield = (options) => request(options);

var brake = new Brakes(rpshield, { timeOut: 1000 });

Hey @awilkins, I have used request-promise in some projects, and I have generally found that it is best to directly use or wrap request.  Here is an example of how I wrap it in a promise for use in brakes.
function _requestWrapper(requestOpts) {
  return new Promise((resolve, reject) => {
    request(requestOpts, (err, response) => {
      if (err) {
        return reject(err);
      }
      // check status codes
      const errored = [500, 501, 502, 503, 504, 505].indexOf(response.statusCode) > -1;
      if (errored) {
        return reject(new InvalidResponseError(
          requestOpts.url,
          response.statusCode,
          response.body));
      }
      return resolve(response);
    });
  });
}

I am following this solution (using the _requestWrapper function proposed above, plus a brake:
const brake = new Brakes(_requestWrapper, {
                                          statInterval: 2500,
                                          waitThreshold: 100,
                                          circuitDuration: 15000,
                                          timeout: 10000
                                        }),
      options = {
          uri: 'my_service_uri
      };

return brake.exec(options)
      .then(res => {
          return <something made with my response>;
      })
      .catch(() => {throw new ServiceUnavailable('The microservice is not available');});

However, I cannot figure out how to make this work. Suppose that the service I am requesting things to is currently down. I would like to:

make the request to the service indicated by the URI
wait just a second (and see that the brake somehow keeps performing requests to such service)
start the service indicated by the URI
see that the brake stood in place and waited for the service to start before timeout, returning correctly the response

This actually does not happen: the ServiceUnavailable error is thrown instantly, without waiting for anything. So maybe I am misinterpreting wrongly the documentation (since it is not largely commented).
I would expect your brake to try for a while to perform the request, and then, if the request goes well before timeout, to return correctly the response, otherwise (if the timeout passes), to throw a ServiceUnavailable error. How can I achieve this, please?
Thanks.

@Eleanore The problem you are trying to solve there is not a problem circuit breakers were meant to solve. If a request fails instantly, the circuit breaker will not delay the returning of request until the service is ready. It will also not perform retries for you. The purpose of the circuit breaker is to fail fast and take pressure off downstream services when they are faltering. If you want the request to retry you will either have to write the logic yourself or use a lib like (node-request-retry)[https://github.com/FGRibreau/node-request-retry].
I apologize for any confusion.

I am going to close this issue. If you have any other questions, please open a new issue.

I do think the implementation should be changed, because current implementation with checking "hasCallback" will be error-prone.

Make it explicit, sending in via options:

new Brakes({ fn: function(){}, timeout: 10000 })

new Brakes({ promise: Promise.resolve(), timeout: 10000 })

Make the check inversed, and check if "main/func" is a promise instead of a function with named arguments. e.g. switch here

new Brakes(Promise.resolve(), {timeout: 10000 })

I tested options 2, but the test-suite blew up.
Want a PR for either of these?

Hey @sveisvei,
Thanks for the feedback, but there is a problem with both of your examples and the suggestion to check if the passed function is a promise instead of doing the check to see if it is a callback. In your examples you use Promise.resolve() which returns a pre-resolved promise. However, the concept of brakes requires you to pass in a function that returns a promise (i.e. (opts) => Promise.resolve()). This is a subtle but important distinction. It has to be a function because you may pass in different options to each request. Moreover, and with the lack of return typing in javascript, it is near impossible to do static analysis to determine if a function returns a promise.
These are all valid promise functions in js:
() => Promise.resolve();
() => new Promise(resolve => resolve());
() => {
  const deferred = Q.deferred;
 deferred.resolve();
 return deferred.promise;
}

You can't realistically determine if a function returns a promise until after you have executed it, whereas with callback functions you can perform analysis on the function signature. If the analysis of the function signature is inadequate, that is something we can and should improve, but I don't see how we can replace it with promise detection.
However, after looking at your examples, I think there is an improvement that could be made. Instead of relying solely on the callback detection in hasCallback we can add an option that forces brakes to treat the first argument  as a callback-function.
That would look something like:
const brake = new Brakes((foo, bar) => bar(), {isFunction: true});

In that example if the isFunction flag is set to true, it forces brakes to treat the first argument as a callback instead of a promise. I would be excited and eager to see a contribution that makes that change 👍
-Alex Wolden


use latest ember-cli-moment-shim

you should merge this in dude. I got errors installing. without it.

Thank you!
@jasonmit would you like to have commit rights for this project?

@claudiocro sounds good, willing to help out

@jasonmit the invitation is on the way.
I just introduced https://github.com/semantic-release/semantic-release to this project, so every push to master branch will trigger a version (create changelog, create a github tag, publish to npm).
I would like to use feature branches to push changes to master so we can review each other. Is this ok for you? If needed we also can open a development branch where we work on and whenever we want a version to be release we just merge the development branch to master.

you guys rock.

@claudiocro Sounds good to me, I usually subscribe to feature branches unless it's a quick no-CI (doc change).


⚠ Bofrost.fr has degraded performance

In 1152d3e, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1277 ms

Resolved: Bofrost.fr performance has improved in e734a80.


overhaul the way credentials are managed

An obnoxious number of changes, any of which you're free to veto or tell me to modify. The basic idea behind all of this was to make it possible to save the encrypted credentials to a file for later use so a user wouldn't need to either (a) store their router password as plaintext in a cronjob or, at the every least, their bash history or (b) always be prompted for a password. Note that I still need to test the longevity of the credentials, because if they expire after a few hours (or even a few days), then they're not nearly as useful for this idea, and I'll need to figure something else out (or settle for my unencrypted router password stored in crontab, sigh).
Anyways, the major changes are:

Allow for credentials to be saved to and loaded from a JSON file
Set a default for the admin username (to uh... admin)
Allow the user to either pass the password in as an argument or input it at a password prompt
Move everything into a class and instantiate that class (this really wasn't entirely necessary, I'm just a nerd)

If you want some of these changes but not all, feel free to copy+paste, or even ask me to go make a less-drastically-changed branch.

Just for fun, I just tested using credentials I saved last night (just under 18 hours ago) and the reboot executed properly - so the 15-minute expiration reported by the gateway is definitely a lie (I’m wondering if that may be cookie expiration rather than nonce expiration).
In light of that, I think I’ll wait to change how I’m handling credentials until I’ve seen how long the actual expiration window is - which unfortunately can only be tested over time.

Alright, credentials appear to expire after 24hrs, so this method definitely won't work. I'll go ahead and update this to only save the hashed password and username, then integrate the other changes you've made into the class structure - unless you'd prefer to not have the class structure, in which case I'll nuke this branch and make a new branch based on your current work but with credential saving/loading added into the workflow.

Thanks for confirming the expiration, @BenKesselring. I'd like to use a class-based structure - hoping to abstract away certain things to utility classes like the authentication helper methods (sha256, sha256url, etc).

That works for me! I'll get on that credentials update and working your latest updates into the class later today - the credentials change should actually be the easiest part (basically just combining the fetch_credentials() and login() functions, and moving the hashing to the password and username in the constructor).
I'll need to see if I can somehow merge in your current main branch and then move stuff around, because I'd prefer to keep the commit history that gives you credit for all the recent programmatic changes.


Reflect-metadata broken in 1.6.2: TS2306: File 'node_modules/reflect-metadata/reflect-metadata.d.ts' is not a module.

To reproduce:

Update to Typescript v1.6.2 and reflect-metadata v0.1.1

Compile this:
///
import 'reflect-metadata';

Expected: compiles just fine
Actual: compile error:
index.ts(3,8): error TS2306: File 'node_modules/reflect-metadata/reflect-metadata.d.ts' is not a module.

I created a git repo to demonstrate:
git clone https://github.com/mattyork/TypescriptReflectMetadataBug.git
cd TypescriptReflectMetadataBug
npm install
./node_modules/.bin/tsc

If I delete the typings field in package.json this error goes away. What is this typings field for and why is it causing this bug?

we either need to remove the typings field, or make it an empty module, e.g.
interface reflect_metadata {}
export = reflect_metadata;

I'll have to remove "typings", as making it an external module will prevent the library from introducing globals.

I removed "typings" from the reflect-metadata polyfill and published a new version to npm.

/// <reference path="node_modules/reflect-metadata/reflect-metadata.d.ts" />

What is the way to modify tsconfig.json to accomplish the same effect?

Include the file in the "files": [] property.


The value of the 'Tag Query' field is inconsistent between original and split blob containers when applying tag query in the original blob container

Storage Explorer Version: 1.35.0-dev (99)
Build Number: 20240719.6
Branch: main
Platform/OS: Windows11/Windows 10/Linux Ubuntu 22.04/MacOS Sonoma 14.5(Apple M1 Pro)
Architecture: arm64/x64/x64/x64
How Found: Exploratory testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.
Open one blob container -> Upload a blob -> Add a tag for the blob (Don't use the default tag).
Open view options panel -> Apply the tag query.
Split the blob container.
Go to the split blob container -> Open view options panel.
Check whether the value of the 'Tag Query' field is consistent between original and split blob containers.

Expected Experience
The value of the 'Tag Query' field is consistent.
Actual Experience
The value of the 'Tag Query' field is inconsistent.

I consider this as By Design. The point of splitting tabs is so you can look at or run separate queries on the same container side by side. I don't see any reason why tab queries should mirror each other across split tabs.

@craxal ,Thanks for your updates!, but this issue doesn't reproduce for 'Filter by prefix' field.

And in the split tab, the address bar and the 'Tag Query' field is inconsistent. Could you help to take a look?


Hungarian translate update & link to me on About page

Please resolve the conflicts in cards-hu.coffee first.

I'll make a new pull request.


deploy kcp inside a container with kcp's offical image

Goal
This script is for creating an environment for developer use where kcp is running in a container (using kcp's official image). To avoid confusion, ./ckcp/gitops.sh is renamed to ./ckcp/openshift_dev_setup.sh
Pre-requisites
Before you execute the script

You need have an Openshift 4.x Cluster(Not support Kubernetes)
You need to install kubectl
You need to install argocd

Run ./ckcp/openshift_dev_setup.sh
You can run the openshift_dev_setup.sh script with or without parameters as below:
$ export KUBECONFIG=<Location of kubeconfig>
$ ./openshift_dev_setup.sh
    #installs openshift-gitops in a host cluster, install kcp in a pod (a.k.a ckcp)
    #and register the kcp cluster to the ArgoCD instance

$ ./openshift_dev_setup.sh -a pipelines
    #installs openshift-gitops + ckcp + tekton-pipeline + pipelines controller

Rely on #68

H @XinRedhat can you please do the 2 modifications mentioned above, squash the commits and I should then be able to merge. Thanks!

Hi @XinRedhat I think that my points have been addressed. One thing still:

this PR has: gitops/openshift-certmanager.yaml
#68 has the same file under: gitops/ckcp/openshift/base/operators/cert-manager.yaml

This will need to be aligned.
In my mind nothing of ckcp should be under gitops as long as it is used for development environment only. We can look at that separately.

Hi @XinRedhat I think that my points have been addressed. One thing still:

this PR has: gitops/openshift-certmanager.yaml
Securely access kcp running in a pod #68 has the same file under: gitops/ckcp/openshift/base/operators/cert-manager.yaml

This will need to be aligned.
In my mind nothing of ckcp should be under gitops as long as it is used for development environment only. We can look at that separately.

Hi @XinRedhat I think that my points have been addressed. One thing still:

this PR has: gitops/openshift-certmanager.yaml
Securely access kcp running in a pod #68 has the same file under: gitops/ckcp/openshift/base/operators/cert-manager.yaml

This will need to be aligned.
In my mind nothing of ckcp should be under gitops as long as it is used for development environment only. We can look at that separately.
RAMESSESII2
@fgiloux sure, I got your points. Discussed with Satyam, he will make change to deploy operators as an argo-cd application from gitops/argocd-apps/ which references the openshift-operators from gitops/openshift-operators/

@XinRedhat Thanks for clarifying. #68 needs to merge first by any way.

@XinRedhat Please rebase, solve the conflicts, and squash the commits. We should then be able to merge the PR.

Have an agreement from all the reviewers. Commits squashed. Merging the PR.


Azure Active Directory authentication

Merging #547 and #546 to fully fix #446

Hi @paulmey, as your original changes were merged first, I rebased #547 on top of them. So far I have not tried to add in your new tests from this PR while waiting for confirmation that the revisions I'm proposing are on the right track. I am happy to do the work to bring them over, though! I think I have broken the merge for this PR after rebasing.

Feel free to bring over the tests, this branch will stay around. Aborting this PR.


Issues in the "Using Email Address as the Username" page

The below phrase says you need to restart only if you change the password. But, you need to restart the server anyway.

If you changed the password of the admin user to something other than 'admin', start the WSO2 IS server using the -Dsetup parameter as seen in the command below.
I tried the same steps and it worked only after restarting. IS team members, please advice.

"Supper" tenant occurrences in the below sentence should be changed to "Super"

You can configure email user name without enabling EnableEmailUserName property, then users can login to both supper tenant and tenant using email and non-email user names. But supper tenant users should always use @carbon.super at the end of user names.

That tip is specifically about starting the server with -Dsetup if the admin password has been changed. But either way, step 6 in this doc says to restart the server so it is given as a mandatory step anyway.

This has been corrected by @ShirazAzad in [1].

[1] https://docs.wso2.com/display/IS550/Using+Email+Address+as+the+Username


[otlLib] Use ValueRecord.getEffectiveFormat() to produce smaller lookups

Continuing from https://github.com/fonttools/fonttools/pull/2275

@anthrotype if this looks good to you I can resolve conflicts.

I figured why we didn't merge, because tests are failing. I'm not going to chase how to fix this as it's not trivial...


:sparkles: making flask_cors & kaleido optional

Closes #125

Codecov Report

Merging #177 (5a6a305) into main (80f8a06) will decrease coverage by 0.21%.
The diff coverage is 72.72%.

:exclamation: Current head 5a6a305 differs from pull request most recent head b52a3c0. Consider uploading reports for the commit b52a3c0 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #177      +/-   ##
==========================================
- Coverage   97.31%   97.11%   -0.21%
==========================================
  Files          12       12
  Lines         894      900       +6
==========================================
+ Hits          870      874       +4
- Misses         24       26       +2

Impacted Files
Coverage Δ

...tly_resampler/figure_resampler/figure_resampler.py
89.16% <72.72%> (-1.19%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Poetry picks 0.2.1.post1 for kaleido, however this version is only for arm32 - https://github.com/plotly/Kaleido/issues/156
Locking kaleido to 0.2.1 until a new version is released


License?

Hello, I was wondering if a License file could be added to your project. If you need some help on choosing a License please visit TL;DR Legal here or for a more guided explanation see ChooseALicense
I know there's a comment within the source that says You are free to modify and redistribute this code without restriction. though it's not really a standard practice for licensing your code. Below are some helpful links for licensing your code:

Licensing a repository
Adding a license to a repository

If you want a license close to what you have, might I suggest the Unlicense or MIT licenses. Let me know if you have any questions.
Disclaimer: I am not a lawyer by the way, just someone who advocates for the open source and free software ecosystem and just happened to come across your buggy YouTube videos. (Great work by the way.)

Thank you, I have just added a license file! Best regards.


[chiptest] Network-independent RPC server shutdown

Problem
Terminating app-register RPC server with the usage RPC call itself might cause troubles in case when the network connection is not reliable (e.g. no route to the server). When this will happen, the test runner will not terminate properly (in my case it happens always):
$ ./scripts/tests/run_test_suite.py --target-glob "DL_LockUnlock" run
2022-03-08 11:52:34.880 INFO    Ensuring /run is privately accessible
2022-03-08 11:52:35.043 INFO    Waiting for IPv6 DaD to complete (no tentative addresses)
2022-03-08 11:52:36.764 INFO    No more tentative addresses
2022-03-08 11:52:36.765 INFO    Each test will be executed 1 times
2022-03-08 11:52:36.765 INFO    Starting iteration 1
2022-03-08 11:52:39.482 INFO    DL_LockUnlock        - Completed in 2.72 seconds         <- here it stuck until RPC call timeout
^C
Aborted!
^CException ignored in: <module 'threading' from '/usr/lib/python3.8/threading.py'>
Traceback (most recent call last):
  File "/usr/lib/python3.8/threading.py", line 1388, in _shutdown
    lock.acquire()
KeyboardInterrupt:

Change overview

This commit simplifies the shutdown process by using dedicated shutdown method.

Testing
Now command ./scripts/tests/run_test_suite.py --target-glob "DL_LockUnlock" run exits properly (no Ctrl-C required).

@vivien-apple ?

I'd be glad if that PR could be merged, because, somehow, in my setup this "not clean" tests shutdown occurs all the time...

Fast tracking given this has had enough time for review.

run_test_suite.py is generally created to run local unit tests, in which case the network should always be quite reliable (either localhost or containers)

OK, so I can explain my setup. I'm running tests locally on Ubuntu 20.04 (not in the docker container). I'm running it as follows:
source scripts/activate.sh
./scripts/tests/run_test_suite.py --target "TestSystemCommands" run   # only one test, but when running all test the issue is the same

output (after the line "TestSystemCommands   - Completed in 3.55 seconds" it hangs for ~60 seconds (network timeout)):
2022-03-22 08:51:14.220 INFO    Ensuring /run is privately accessible
2022-03-22 08:51:14.444 INFO    Waiting for IPv6 DaD to complete (no tentative addresses)
2022-03-22 08:51:16.363 INFO    No more tentative addresses
2022-03-22 08:51:16.365 INFO    Each test will be executed 1 times
2022-03-22 08:51:16.366 INFO    Starting iteration 1
<IP_ADDRESS> - - [22/Mar/2022 08:51:18] "POST / HTTP/1.1" 200 -
<IP_ADDRESS> - - [22/Mar/2022 08:51:18] "POST / HTTP/1.1" 200 -
<IP_ADDRESS> - - [22/Mar/2022 08:51:18] "POST / HTTP/1.1" 200 -
<IP_ADDRESS> - - [22/Mar/2022 08:51:18] "POST / HTTP/1.1" 200 -
2022-03-22 08:51:19.919 INFO    TestSystemCommands   - Completed in 3.55 seconds
Traceback (most recent call last):
  File "./scripts/tests/run_test_suite.py", line 241, in <module>
    main()
  File "/home/a.bokowy/connectedhomeip/.environment/pigweed-venv/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/home/a.bokowy/connectedhomeip/.environment/pigweed-venv/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/a.bokowy/connectedhomeip/.environment/pigweed-venv/lib/python3.8/site-packages/click/core.py", line 1289, in invoke
    rv.append(sub_ctx.command.invoke(sub_ctx))
  File "/home/a.bokowy/connectedhomeip/.environment/pigweed-venv/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/a.bokowy/connectedhomeip/.environment/pigweed-venv/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/a.bokowy/connectedhomeip/.environment/pigweed-venv/lib/python3.8/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "./scripts/tests/run_test_suite.py", line 224, in cmd_run
    apps_register.uninit()
  File "/home/a.bokowy/connectedhomeip/scripts/tests/chiptest/accessories.py", line 36, in uninit
    self.__stopXMLRPCServer()
  File "/home/a.bokowy/connectedhomeip/scripts/tests/chiptest/accessories.py", line 133, in __stopXMLRPCServer
    client.ping()
  File "/usr/lib/python3.8/xmlrpc/client.py", line 1109, in __call__
    return self.__send(self.__name, args)
  File "/usr/lib/python3.8/xmlrpc/client.py", line 1450, in __request
    response = self.__transport.request(
  File "/usr/lib/python3.8/xmlrpc/client.py", line 1153, in request
    return self.single_request(host, handler, request_body, verbose)
  File "/usr/lib/python3.8/xmlrpc/client.py", line 1165, in single_request
    http_conn = self.send_request(host, handler, request_body, verbose)
  File "/usr/lib/python3.8/xmlrpc/client.py", line 1278, in send_request
    self.send_content(connection, request_body)
  File "/usr/lib/python3.8/xmlrpc/client.py", line 1308, in send_content
    connection.endheaders(request_body)
  File "/usr/lib/python3.8/http/client.py", line 1251, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.8/http/client.py", line 1011, in _send_output
    self.send(msg)
  File "/usr/lib/python3.8/http/client.py", line 951, in send
    self.connect()
  File "/usr/lib/python3.8/http/client.py", line 922, in connect
    self.sock = self._create_connection(
  File "/usr/lib/python3.8/socket.py", line 808, in create_connection
    raise err
  File "/usr/lib/python3.8/socket.py", line 796, in create_connection
    sock.connect(sa)
TimeoutError: [Errno 110] Connection timed out

After the change proposed in this PR, there should be no implications other than server termination will be handled "locally" (it will not rely on the "ping" RPC, which IMHO is a bad design, just because of the network component required for server termination). This "ping" RPC was used only during the shutdown, so I've removed it as well. For this explanation I've selected "TestSystemCommands" test on purpose. It uses the RPC server for performing the test (RPCs: stop, start, reboot, factoryReset), so you can see that the test application is able to communicate with the server. However, during test shutdown the test framework can not connect to it's own server thread.... So, I've assumed that there has to be something wrong with the network/routing setup. Hence, I've called it "when the network is not reliable". But maybe it is indeed not the best wording for this issue.


ClickHouse Operator not able to watch ClickHouseOperatorConfiguration Resources

Hi operator team,
We recently just upgraded our operator to 0.17 using the latest install bundle yaml and we noticed this warning log in the operator:
W1026 16:29:43.261016       1 reflector.go:436] pkg/client/informers/externalversions/factory.go:117: watch of *v1.ClickHouseOperatorConfiguration ended with: an error on the server ("unable to decode an event from the watch stream: unable to decode watch event: no kind \"ClickHouseOperatorConfiguration\" is registered for version \"clickhouse.altinity.com/v1\" in scheme \"pkg/client/clientset/versioned/scheme/register.go:30\"") has prevented the request from succeeding

Our operator seems to be functioning normally but unsure if this is cause for concern?
I've noticed in the source code, the api doesn't seem to be registered for the ClickHouseOperatorConfiguration type in this function:
https://github.com/Altinity/clickhouse-operator/blob/9731c23e49b32688ae13e68156d6037be9e0ce5d/pkg/apis/clickhouse.altinity.com/v1/register.go#L48-L58
Thanks for the help

@mcgrawia thanks for notification! Next operator version is under development ATM and this issue would be fixed. Please wait for operator 0.18.0 to come.

Ok will do. Thanks @sunsingerus!

Version 0.18.0 of the operator is about to be released, which has this issue fixed. It is stable enough, so feel free to test.


Upvote and Downvotes fixed

Issue that this pull request solves
Closes: #1004
Proposed changes
The followings are added

[x] Upvote and downvote added
[x] Each user can add only one votes per question.

Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue

Screenshots

Other information
Any other information that is important to this pull request

Hey @Kajol-Kumari
Please review it and merge it....
The above failure doesn't affect any functionality
Please upgrade the level from level1 to level2 If possible
Don't forget to add FRONTEND_URL in Backend ENV


Java buildpack, convert to online buildpack, secrets.yml location

Fixes #18  - Buildpack is smarter about where to find secrets.yml
Fixes #28 - Secrets retrieval script not sourced when used with Java Buildpack
Fixes #27 - Buildpack works correctly as an online buildpack
Also just generally added lots of statements about what is happening in the supply script so it is more verbose.

Hello Conjur folks,
After much much much digging, I believe this code is pretty close to complete and addresses the reality of the V2 buildpacks which are messy. When the V3 come out, things should get better.
It address the items that are listed in the description.
I'm WIDE open to comments, suggestions, redirects.  I was unable to get the ./test.sh script to work locally on my laptop, so I did not update any of the cucumber tests, so I probably broke those.
But I did test very extensively via our CloudFoundry install with a pretty recent Java Build pack.
It will also need to tested pretty heavily against other non-Java buildpacks.
I want to clear this code is not what we want todo, it breaks a contract about not modifying files inside the /home/vcap/app directory, but at this point, you don't have a choice with the Java Buildpack.
https://github.com/cloudfoundry/java-buildpack/issues/563#issuecomment-452437417
Also I left some code that I believe was not valid any longer until one of you all get a chance to look (at the bottom of the supply script)
Looking forward to hearing what you all think.

Here is the output of my build, I could not figure out how to get the go mod download to be quiet, oh well.
   Downloaded build artifacts cache (173B)
   Downloaded app package (14.2M)
   [cyberark-conjur-buildpack]: supplying
   [cyberark-conjur-buildpack]: looking for secrets.yml
    - secrets.yml file found in /tmp/app/BOOT-INF/classes/secrets.yml copy to /tmp/app/secrets.yml
   [cyberark-conjur-buildpack]: looking for cyberark-conjur in VCAP_SERVICES
    - found it
   [cyberark-conjur-buildpack]: download go
   -----> Download go 1.11.5
   /tmp/contents580847861/deps/0 ~
   [cyberark-conjur-buildpack]: download the conjur-env binary dependencies
   go: finding gopkg.in/yaml.v2 v2.2.2
   go: finding golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a
   go: finding github.com/sirupsen/logrus v1.3.0
   go: finding github.com/bgentry/go-netrc v0.0.0-20140422174119-9fd32a8b3d3d
   go: finding github.com/cyberark/conjur-api-go v0.5.2
   go: finding github.com/konsorten/go-windows-terminal-sequences v1.0.1
   go: finding github.com/mitchellh/go-homedir v1.1.0
   go: finding github.com/cyberark/summon v0.6.10
   go: finding golang.org/x/crypto v0.0.0-20190211182817-74369b46fc67
   go: finding gopkg.in/yaml.v1 v1.0.0-20140924161607-9f9df34309c0
   go: finding gopkg.in/yaml.v2 v2.2.1
   go: finding github.com/jtolds/gls v4.2.1+incompatible
   go: finding github.com/sirupsen/logrus v1.0.5
   go: finding github.com/smartystreets/assertions v0.0.0-20170925172151-0b37b35ec743
   go: finding github.com/davecgh/go-spew v1.1.1
   go: finding github.com/gopherjs/gopherjs v0.0.0-20181103185306-d547d1d9531e
   go: finding github.com/kr/pretty v0.1.0
   go: finding github.com/smartystreets/goconvey v0.0.0-20181108003508-044398e4856c
   go: finding github.com/pmezard/go-difflib v1.0.0
   go: finding github.com/codegangsta/cli v1.20.0
   go: finding github.com/gopherjs/gopherjs v0.0.0-20180202210947-296de816d4fe
   go: finding golang.org/x/net v0.0.0-20181201002055-351d144fa1fc
   go: finding gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405
   go: finding github.com/stretchr/objx v0.1.1
   go: finding golang.org/x/sys v0.0.0-20180709060233-1b2967e3c290
   go: finding github.com/smartystreets/assertions v0.0.0-20180927180507-b2de0cb4f26d
   go: finding github.com/kr/text v0.1.0
   go: finding github.com/mitchellh/go-homedir v1.0.0
   go: finding golang.org/x/crypto v0.0.0-20180904163835-0709b304e793
   go: finding golang.org/x/crypto v0.0.0-20180621125126-a49355c7e3f8
   go: finding gopkg.in/check.v1 v1.0.0-20180628173108-788fd7840127
   go: finding golang.org/x/sys v0.0.0-20180905080454-ebe1bf3edb33
   go: finding github.com/smartystreets/goconvey v0.0.0-20170602164621-9e8dc3f972df
   go: finding github.com/stretchr/testify v1.2.2
   go: finding github.com/kr/pty v1.1.1
   [cyberark-conjur-buildpack]: building the conjur-env binary.
   /tmp/buildpackdownloads/4b7ef17596bf42a98752c9d73a5e2ada/conjur-env/conjur-env: ELF 64-bit LSB  executable, x86-64, version 1 (SYSV), statically linked, not stripped
   [cyberark-conjur-buildpack]: copy the conjur-env binary from the buildpack dir to the dependencies/index directory
   ‘/tmp/buildpackdownloads/4b7ef17596bf42a98752c9d73a5e2ada/conjur-env/conjur-env’ -> ‘/tmp/contents580847861/deps/0/vendor/conjur-env’
   ~
   /tmp/contents580847861/deps/0 ~
   [cyberark-conjur-buildpack]: setup the dependencies/index/profile.d script so the conjur app starts automatically
   [cyberark-conjur-buildpack]:  - because of missing feature in Java Buildpack, also copy /home/vcap/app/.profile.d/
   ‘./profile.d/0001_retrieve-secrets.sh’ -> ‘/tmp/app/.profile.d/0001_retrieve-secrets.sh’
   ~
   -----> Java Buildpack v4.19 (offline) | https://github.com/cloudfoundry/java-buildpack.git#3f4eee2```

@sgnn7 Totally agree.  Yes, definitely needs a Squash-Merge.  I was struggling on how to test the "online" part of the online buildpack without pushing things up to github.com.  So the history itself is not very helpful of my branch.  If you all know a better way to test that in the future, I'd love to hear it.

hey @sgnn7 just read your comment again, to be clear, are you asking me todo the sqush-merge or will you all do that if the PR is accepted?  If you are asking me todo that, if you will point me in the right direction to do that. I'm not sure how todo that since I've already opened the PR.  I use the git cli and only the github.com for opening PR, Issues, and such.

We can do it (squash merge) if the PR is accepted. Also, we'll work on getting this project to build in gitlab so you can see it.
Since this is a solid contribution, could you please sign our contributor agreement and send the signed version to<EMAIL_ADDRESS>Please feel free to get in touch if you have any questions about that process at all! We'll get on reviewing the PR itself as soon as possible, but just so you know we've got a lot of vacation happening right now so it may take a few days :)
Thanks again for this! A quick read through of it looks great!

@izgeri I have started the internal process here at Home Depot to get that document signed.  I'm just a lowly peon so I can't sign it myself haha Once I get it signed, I'll get it back to you all.

@izgeri Also per the timeline, there is ZERO hurry, because... um... I can use the THD fork. 😄 But at some point, I'd rather use the official release, but we are just in testing phase.  So just sometime in early May would be awesome. And if I can help make local testing more visible or easy let me know.  I finally reverted to caveman testing.  I just hammered on it until it works, but I'm worried about what functionality I broke.

Sounds good! I'll keep you posted - hopefully I'll at least have an update about the GitLab pipeline so you can see the build soon.

@cadethacker I updated master to have a gitlab pipeline - could you rebase to master so that you can pick up the .gitlab-ci.yml and run the build in gitlab?

@izgeri I (with the help of friends :D) rebased and pushed it back to the homedepot fork.  Take a look and let me know if that is what you intended.

I will defer to @micahlee on updating the CHANGLOG.md file if that is ok with you all.

The build is currently green, fyi. You can check it out (CyberArk only,  sorry! Adding to our list to improve.) here.

@izgeri CA signed by my VP and emailed to you all.  It covers any PR from Home Depot.

We received it, thank you!

@izgeri no worries, just glad to help.  You all just guide me.  This was a pretty invasive change, so I'd be pretty impress if I didn't break something :D

@cadethacker Thanks again, this is solid PR with some great improvements!
I've gone through the scenarios we outlined above and everything works great.
I have three commits in a branch here that we should incorporate:
https://github.com/cyberark/cloudfoundry-conjur-buildpack/compare/pr/29
These commits make the following changes:

Only build the conjur-env binary when it doesn't already exist (to account for the offline buildpack scenario).
Update our CI environment to better reflect the expected app staging environment.
Prevent running the retrieve secrets script multiple times in a single app startup.
Update the CHANGELOG and README with the PR changes.

Let me know what you think. I'm happy to commit these directly to your PR, or you're welcome to cherry pick or incorporate them as you see fit.
After these are in, we should have a green CI build and be ready to merge!

@micahlee Please go ahead and commit those.  I view my original commit as the beginning of the conversation. Ultimately i'm a guest in your house :D so update as you see fit.    I'm super happy with how this is progressing.

@micahlee I have merged in your pull request over to the Home Depot fork, so it is now part of this Pull Request.


tweak a bunch of logging and config

make overall logs less verbose and chatty
1 minute computer check-in interval
3 minute snapshot interval
remove CaptureLogger as it has same functionality as buffer logger
add a WithPrefix to the Logger interface so sub-services can have different prefixes

just that TestQueryHistory thing is failing which is ticketed


Test: Added example test

added example test for stringutils.RuneFromCharSet

Hey man, I create this PR, i could not find any other stringutils methods to write example test case, wdyt?

Hey, thanks for contribution. This is good first start :). Basically, you added example for sub-package and it's also good thing. The best would be to add example to utility methods from steps.go file.
Goal here is to show any developer how real world usages of library can looks like.
Steps required to properly use library are:

Initialize APIContext. We can use one of 2 constructors, easier path is to use this one
Now, we have plenty of methods to work on from steps.go file.

Let's egzamine save method, which allows user to save any data into scenario cache. Let's say, we would like to save as HUMAN_AVG_TEMP, value 36.6.
So example logic may look like this (i write it from hand, so may contain errors):
ac := NewDefaultAPIContext(false, "")
if err := ac.Save("HUMAN_AVG_TEMP",  36.6); err != nil {
     //log error somehow
}

temp, err := ac.Cache.GetSaved("HUMAN_AVG_TEMP")
if err != nil {
     //log error somehow
}

fmt.Println(temp)
// Output: [36.6]

Each method from steps.go returns error interface. The logic of examples may trigger error path on method (return not nil error) and print error string (so we can write our assertion. for example: // Output: [97], as you did). There are similar examples in standard go library.
Tell me whether you want to try to add another examples, or you just end on this.

I fixed go vet, I will try to add another examples as you mention


Bi ags small pr

Small version of pull request:

[x] Function Form_from_force_newton
[x] Jacobian Matrix calculation
[x] Constraints Class
[x] One example only
[ ] Manage Fixity only at X and Fixity only at Y
[ ] Nullspace

Reduced version @brgcode

Hi Tom, I think this is ready


Ported DH code from crypticl.

I've tested it on SBCL but it's ANSI CL so the test should work anywhere.

Hi.
There is already a pull request (PR #71) containing functions for Diffie-Hellman key exchanges for multiplicative groups of integers modulo a prime (elgamal) and for some elliptic curves (curve25519 and curve448).
Also, the functions for modular exponentiation and random integer generation are already in the math.lisp file.

Hello,
there does not seem to be an API for performing a Diffie-Hellman key-exchange (as used in TLS in the ServerKeyExchange message and ClientKeyExchange message). If ironclad does indeed support this, I would gladly use the its implementation because that it what I am after anyway. I may also be needing Ironclad's ECC implementation for DHE and ECDSA soon. It seems additions have been made to ironclad but the documentation has not been updated yet, because I ported the DH code from crypticl thinking ironclad does not have support for DH key exchange.

The current release of ironclad doesn't have the DH key exchange code because PR #71 has not been merged into the main branch of the code yet.
In PR #71, the function to perform DH key exchange with integers modulo a prime is in the elgamal.lisp file because the format of the key pairs for DH key exchange and Elgamal encryption is exactly the same (but we could create a new key pair class just for DH if that is confusing).

Ah, I see. math.lisp contains all that's needed anyway to implement it anyway, no need to import code from another project. I may make a pull request with the DH key exchange API (this time using code already in ironclad) if and when #71 is merged.


bf-Screen overflow

System Information

Mozilla Firefox v74.0

Windows 10 Enterprise

Describe the bug
Whenever we start typing something on the editor, the screen overflows and we see vertical and horizontal scrollbars. This issue is not encountered in the development run but only in deployment.

To Reproduce
Steps to reproduce the behavior:
1. Go to 'https://deploy-preview-25--blissful-mcnulty-82f253.netlify.com/lesson/chapter-01'
2. Type something on the editor
3. The screen will overflow'

-Expected behavior
The screen should not overflow and it should completely fit in the browser screen.

I am not able to reproduce the bug
System Information

OS: Mac
Browser: chrome
Possible to share loom link for it ?

I am not able to reproduce the bug
System Information
* OS: Mac

* Browser: chrome

Possible to share loom link for it ?

Here is the loom link : https://www.loom.com/share/dfa50340dea34703b58d62ce953d2704

seems like the issue is with the firefox browser support, for x CSS feature: different browser/viewport screenshots

Possible solution: turning auto-complete suggestion off

seems like the issue is with the firefox browser support, for x CSS feature: different browser/viewport screenshots

Let's ask Abhinav as well to test this.

Possible solution: turning auto-complete suggestion off

Got you!

Cannot replicate the issue on Mozilla Firefox v75.0
Is this solved @bhaskarSingh ?

@dhruv10  yes


"cannot fine the Placeholder op that is an input"

my code:
@tf.function()
def infer(input):
output=model(input)
gpt2 = infer.get_concrete_function(tf.TensorSpec(shape=(None,None),dtype=tf.int64,name='input'))
''''
error occur here, the error message is the tittle
''''
gpt2_function = convert_variables_to_constants_v2(gpt2)

Hi @Coder-nlper ,
I don't have your code to reproduce. In addition, it seems to me that this is a TensorFlow problem instead of a bug in my demo examples.
Best,
Lei

thank you!your demo is ok~it is a tensorflow2.1 bug~


feat: trigger download dialog, when downloading

When clicking on a download link, trigger a download dialog, instead of opening the content inline.

Found two related JIRAs, more about the feature itself, less about the opening of the download dialog:

https://issues.redhat.com/browse/TC-284
https://issues.redhat.com/browse/TC-287


Incorporate supply/demand of basins in allocation

Fixes https://github.com/Deltares/Ribasim/issues/564.

Superseded by https://github.com/Deltares/Ribasim/pull/1082.