Allow user override of Operation type

This would allow users to provide their own types in models to be used as response types (basically just prevents it from being generated)
Should we remove the error from the signature, or add an exception for that lint error ?

Thank you for this PR!
I think it is problem what generate same operation type.  I don’t understand why users can provide thir own types. Please more explanation for me ! I wanna be understand your opinion.

Lets say we have the following:
type User {
  id: ID!
  name: String!
  email: String!
}

query GetUsers() {
  getUsers {
    id
    name
  }
}

The schema would generate the User type:
type User struct {
    Id string `json:"id"`
    Name string `json:"name"`
    Email string `json:"email"`
}

And the query would generate the following operation struct:
type GetUsers struct {
  GetUsers []*struct {
    Id string `json:"id"`
    Name string `json:"name"`
 } `json:"getUsers"`
}

While that does the job, lets say in the rest of my codebase i need to pass around the User struct, i would have to have some code to transform the unnamed struct to the User type => a lot of boilerplate.
That's where letting the user provide the struct is useful. By adding to my codebase:
type GetUsers struct {
  GetUsers []*User `json:"getUsers"`
}

And the relevant type map in the config, that allows me to have the generated client.GetUsers function to return some struct that contains an []User which is way easier to work with.

I think that method is not good in GraphQL because it also fetches unnecessary fields. You should use a Query and a client that represent the appropriate use case. If you want to get all the fields of User, in any case, I think you need to define all the fields and make it Fragment.

You dont have to fetch all fields, but using in your codebase an unamed struct is a pain
In my example above, User is defined with id, name, email, but I only request 2 of those fields, and imo that's fine to have the last field being at its zero value.
Giving more flexibility, especially for a library can only be a good thing.

@Yamashou Any thoughts ?

I think the meaning is different between the zero value and the absence of the field.

It is, i m not saying that it is the same, using the generated model is a corner case, but what if I just dont want to use an anonymous struct because the result needs to be passed around ? Giving flexbility to the user to control that would be a good idea imo, just like gqlgen does.
Lmk if you want that into gqlgenc, otherwise feel free to close that PR, I will be maintaining my own fork as I need that feature, and I m sure other people will.


problems with reading blackrock nsx files

Hi all,
I'm trying to read BlackRock nsx data and I do run into several problems.

When reading a ns2 file which is only 100MB long, then it takes unreasonably long time to parse the header. The code I use is following:

from neo.io.blackrockio import BlackrockIO
reader = BlackrockIO(filename=file_name, verbose=True)

If I use

from neo.rawio import BlackrockRawIO
reader = BlackrockRawIO(filename=file_name)
reader.get_analogsignal_chunk(i_stop=300)

the I get an error

AttributeError: 'BlackrockRawIO' object has no attribute '_several_channel_groups'

There is also this option
from neo.io import OldBlackrockIO as BlackrockIO
but the differences to (1) are not clear. Should this option still be used?

In general I have the impression that reading BlackRock files is not that well documented. Do you have any nice examples which work also for larger files?
Thanks for your help.

Hi @jsv13!
Parsing the header indeed shouldn't take very long for such a small file. Can you give more details on

the Blackrock file format version
the neo version

or even provide the ns2 file for testing purposes?
If you plan to use the neo structure for handling your data, then using the approach 1) is the correct one. Approach 2) gives you fast access to the raw data, but does not provide any utilities (eg. quantity handling...) around that.  Regarding your approach 3) There is still an old (non-raw) implementation of the BlackrockIO present for compatibility reasons until the current implementation covers all functionality of the old implementation. You can also try out the OldBlackrockIO, but be aware this has very different parameters and is slower than the BlackrockIO.

Hi.
BlackrockIO do not only oarse header but also load all data in memory and by default scale everything to units so it is quite heavy.
In actual master there is a lazy load option,
reader = BlackrockIO(filename=file_name, lazy=True)
Samuel

neo version:  0.7 and now I pulled also 0.8.0.dev (same problem)
blackrock file format version: 2.3

I see that most of the time is spent in  __read_nsx_dataheader_variant_b
reader = BlackrockIO(filename=file_name, lazy=True) gives an error with 0.8.0.dev:

TypeError: init() got an unexpected keyword argument 'lazy'

Unfortunately I cannot share ns2 file, it's human data, would be difficult from regulatory and data protection point of view

Oups my bad.
reader = BlackrockIO(filename=file_name)
reader.read_block(lazy=True)

And you are right, reader = BlackrockIO(filename=file_name) should do only the parse_header and should be fast.
I am sorry I awsered to quickly.
Maybe it is due to the event and spike file that are really big.
Because to get all possible units the file must entierily read at parse_header stage.
What append if you move the nev file somewhere else ?

still the same if I move the nev file away. To me it seems the problem to be the following:
In the  __read_nsx_dataheader_variant_b method  the offset is increased  only by 9 bytes at a time, because data_size is zero  ( see offset = data_header[index]['offset_to_data_block'] + data_size ).
So, it goes through a 100MB file 9 bytes at a time, this doesn't look correct, does it?

Maybe the number of channels was not recognized correctly? Can you also check self.__nsx_basic_header[nsx_nb]['channel_count'] and dh['nb_data_points'] in the same method?

self.__nsx_basic_header[nsx_nb]['channel_count'] = 128  (correct)
dh['nb_data_points'] = 0  . I don't know what this should be, but I guess this causes the problem

Yes, it looks like the number of datapoints is either not recognized correctly, or all your data pakets are empty. Do you get dh['nb_data_points]==0for all data pakets, i.e. all iterations of thewhile` loop?

yes, for all iterations

That's puzzling. Are you sure the data was recorded correctly? Eg. can you replay it using the Blackrock system used for recording or visualized it using the NeuroExplorer tool? @samuelgarcia: Do you have any more ideas regarding this?

well, I can open the file with NPMK without a problem. I will check neo with other files as well and report back.

This is very strange.
@jsv13 : would you have any possibility to record a new on the same system with same configuration but wihtout patient, so we could have the same buggy file ?
We could so include it in our test file.
Without the file, I guess that you are the only one able to debug this...
@JuliaSprenger I went to the code of __read_nsx_dataheader_variant_b. I guess we could avoid the loop with a memmap no ?

@samuelgarcia: Yes the loop can be avoided by incorporating the __read_nsx_dataheader directly into the __read_nsx_dataheader_variant_b if you assume the numer of data points per paket is constant. But I think this would make the code less modular and readable. If you have a suggestion for a readable version without the loop, I am happy to review your PR :)

@samuelgarcia : This may not happen any time soon, but yes I could try. So far, I've had this problem only with one file.

@jsv13 unfortunately I think we have to close this issue as unfixable for now. If you can provide us with a test file that exhibits the problem, we can reopen the ticket.


chore(deps): update google-github-actions/release-please-action action to v3.7.4

This PR contains the following updates:

Package
Type
Update
Change

google-github-actions/release-please-action
action
patch
v3.7.3 -> v3.7.4

Release Notes

google-github-actions/release-please-action
v3.7.4
Compare Source
Bug Fixes

bump release-please from 15.1.2 to 15.7.0 (#714) (19b65cc)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Fixes code example in README

testSwizzleAddition does not exist in the context of the example code.
Instead of creating testSwizzleAddition I decided to inline the value as I feel this makes matching the output with the example code easier 😄

Thank you!!


Missing implementations on Android side.

Some methods on the plugin side throw errors because they are not implemented in Android, such as activate wifi, isEnabled.
Except for these methods, disconnect, getSSID, connect, prefixConnect, secureConnect, and securePrefixConnect, others are not implemented as mentioned above.
Also, I am getting gradle errors, current kotlin version is 1.7.10, what should I do?
What went wrong:
The Android Gradle plugin supports only Kotlin Gradle plugin version 1.5.20 and higher.
The following dependencies do not satisfy the required version:
project ':plugin_wifi_connect' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.50

Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BU�LD FAILED in 4s

This needs to be fixed, please.

Hi, we need to check this, because we use this package only for iOS, so we have no idea that android has some problems.
thanks for repporting us.

The package configuration for latest flutter is OK.
The only issue I see updated is:
compileSdkVersion 33   minSdkVersion 19
Seems the lib was build with older kotlin version, or some bugs in build tools.
I would also suggest building the plugin with updated pubspec.yaml:
`environment:
sdk: '>=3.1.5 <4.0.0'
flutter: '>=3.3.0'
dependencies:
flutter:
sdk: flutter
plugin_platform_interface: ^2.0.2
dev_dependencies:
flutter_test:
sdk: flutter
flutter_lints: ^2.0.0`

@lukasz-bialek thanks for the support, this year 2023 I didn't have time to fix the problems in the package due to work, but now in 2024 I'll pay more attention


Add cargo example for Rust project

This PR adds example for Cargo, which is an official build tool for Rust toolchain. In addtion to local target build directory, ~/.cargo/registry (packages registry) and ~/.cargo/git (packages index) were necessary to be cached for reusing cached build directory.
I already tried this on my project kiro-editor and cargo test was improved as follows.

Before (without cache): job

Linux: 1m26s (1m32s in total)
macOS: 41s (1m4s in total)

After (with cache): job

Linux: 13s (33s in total)
macOS: 18s (35s in total)

6.6x faster on Linux.

I prefer to add the hash of the lock file to the key. What do you think?
key: ${{ runner.os }}-cargo-registry-${{ hashFiles(format('{0}{1}', github.workspace, '/Cargo.lock')) }}

@imbsky
Cargo handles outdated library. For example, when we update some dependency and Cargo.lock is changed, cargo test checks local dependencies and rebuilds packages which were updated. So if I'm correct, we don't need to care and can use cached directory always.

I see. I have never used Rust so I didn't know that, so I only checked some existing CI providers example. Opinions from people like you who know Rust well are very helpful! Thank you!

Oh, I'm glad if my suggestion helped 😁

The example has been moved to examples.md. See #13.

Thanks. I moved the example to examples.md and rebased this branch onto the latest master.

Looks good to me.

@rhysd can you resolve your merge conflicts and we will get this mrged today.
Thanks.

@chrispat I did.


How to secure MongoDB in Linux

NOTE: All 1st-time contributors should know Topic approval and PR submission does NOT guarantee your Topic/Article will be published. Our team of Peer Reviewers and Content Moderators will review all PRs that come in to make sure they adhere to the standard of quality we expect from the EngEd community.
We expect all community memebers to go through provided examples, resources, and previously published material before submitting content. As a rule of thumb, please only submit articles (pull requests) that are complete, formatted correctly, and include a fully polished article (ready to be published and error free).
All revisions and edits should be completed on your own forked repo (as to not take up room in the queue). Any PR that is submitted incomplete (meaning not ready to be published as is) will be closed. A PR being closed does NOT mean the article can not be published (or fixed) it simply means that the edits, revisions, and fixes will happen outside of the queue.
Topic Suggestion
Securing MongoDB
Pre-submission advice
By following all our pre-submission advice and reviewing our Resources folder, you will maximise your chances of your topic being approved.
We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner.
Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles that will help you propose and write a successful article.
Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
Writing sample(s):
Include any links or writing samples - to help our team better gauge your writing quality.
Proposal Submission
Proposed title of article
Securing MongoDB
Proposed article introduction
MongoDB is a document database that is widely utilized in modern web applications. MongoDB is not by default safe. You're going to have a difficult time if you install MongoDB and launch it without configuring it for authentication. People can browse, write, delete, or modify data on your server without ever needing to log in or authenticate. Securing the database is simple and may be completed in a few steps. In this article am going to show you how to secure your MongoDB using some installation commands.
Key takeaways
By the end of this tutorial the learner will be able to:

secure his/her MongoDB during use.
create an administrative users and enable authentication.

Article quality
very unique since we are going to use ubuntu linux as the server
References
N/A
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Good topic @benardali - Approved.


Add bandit to supported linters

For #2726

[x ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x ] Title summarizes what is changing
[ ] Has a news entry file (remember to thank yourself!)
[x ] Unit tests & system/integration tests are added/updated
[x ] ~Any new/changed dependencies in package.json are pinned (e.g. "1.2.3", not "^1.2.3" for the specified version)~
[x ] ~package-lock.json has been regenerated by running npm install (if dependencies have changed)~

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

The order of the linters in various data structures was kind of random. Should I alphabetize or standardize this?

@brettcannon /cc

@DonJayamanne, thanks for the super fast review and feedback! will make those changes.

@DonJayamanne Made all the changes. Added a fix as well for a bug in Bandit's message templating, documented here: https://github.com/PyCQA/bandit/issues/371 and fixed on Bandit master

@demus
Please could you resolve the code merge issue.
I'd like to merge this in today.

@DonJayamanne should be all set


replit.web using usernames as keys

From what I can see, the replit.web thingy uses usernames as keys for its database. The problem with that is that usernames can change on replit so if someone were to change their username, they would lose all their data on any sites which used this package. If replauth gives you user ids then you should probably use them instead so there is no chance of users losing data due to changing their username.

Interesting idea, however what if you wanted to get the usernames back somehow? You would have to store the ids as keys and the usernames in the values. It just adds a lot of complexity. Since changing usernames is quite rare, I think the impact is tolerable.


zh-trans: add docs/concepts/overview/working-with-objects/names.md

zh-trans: add docs/concepts/overview/working-with-objects/names.md

/assign @chenopis


wen i minimizes my chat screen and open safari browser than i will see that chat disconnected if i return after a minute.

Wrong repository. Probably this issue is related to this repo: https://github.com/livechat/react-livechat/issues


Spec: Are implementations free to use their own hash code calculation

Feature Request

Summary
Are implementations free to use their own hash code calculation
Feature Description
I suppose this is under the rubric of a specification.

@taqtiqa-mark, is this still relevant? Could you please explain this feature request in more detail?

Possibly.  I'm beginning working on a Rust port, and while I haven't got to the point of implementing hash related code, I do believe hash calculations in Rust are different.  I suppose I'm asking whether the hash calculations across languages need to be the same?
Does that make sense?

No, the hashCode implementations do not have to be identical. They are neither used as fingerprints nor for serialization. In Java, it is good practice to define hashCode when equals is defined. This is the main reason why there is a hashCode implementation for histograms.


projectile 0.14.0

Created with brew bump-formula-pr.

dash-emacs now installs in elisp instead of in share/"emacs/site-lisp/dash".  Feel free to change to elisp everywhere while you're at it.

@dunn PR refreshed

👍


Redirect unauthorized users to login/signup and return to attempted secure route upon authentication

Issue Description:
Currently, there is a problem when an unauthorized user attempts to access a secure route. Here’s a breakdown of the issue and proposed solution:

Problem:

When an unauthorized user tries to access a secure route:

They either encounter a blank screen or
They are redirected to the login page without any context of the original route.

Proposed Solution:

If an unauthorized user tries to access a secure route:

They should be redirected to the login or signup page.
After successful authentication (login/signup):

The user should be redirected back to the originally intended secure route.
This can be done by passing the intended route through URL parameters

Benefits:

Improves user experience by making sure they don’t lose track of the page they were trying to access.
Ensures a smooth flow between authentication and accessing secure content.

Hi @vishalverma9572, I want to contribute to this, can you please assign this to me?

Hey, can you assign this issue to me?

Hey, @vishalverma9572 , I also want you to assign this issue to me...

@Rahulagg13 @Vikas350 , this issue is almost resolved. so you guys can work on other issues!


[BUG] ClusterLoadFallbackPolicy is not strictness when a shuffle with big partitions to register

[BUG] ClusterLoadFallbackPolicy is not strictness when a shuffle with big partitions to register
What changes were proposed in this pull request?
If a shuffle with 5w partitions to register, and cluster has 3w but the slots usage is not in high load status, 5w allocation slots will failed.
Why are the changes needed?
This change will avoid this circumstance that a shuffle that it will cost too much slots over clusters' continues process of RSS handling
What are the items that need reviewer attention?
Related issues.
https://github.com/alibaba/RemoteShuffleService/issues/17
Related pull requests.
How was this patch tested?
/cc @waitinfuture @FMX
/assign @wangshengjie

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: FMX:x: zhangxudong1:x: waitinfuturezhangxudong1 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

LGTM


Content not announced with VoiceOver and some announced from screen behind

VoiceOver is not announcing the "close button". The focus on the top seems to be on the COVID Alert image in the background. Double tapping on this closes the COVID Alert. See screenshot 1.

After this, the focus goes to content behind the screen and it announces those items from the behind the screen. User is not able to access other items on this screen with VoiceOver. The screenshot 2 has how the focus with VoiceOver is for content in the back.

iOS 13.5.1 on iPhone 7

Linked to this issue upstream https://github.com/CovidShield/mobile/issues/39

https://github.com/cds-snc/covid-shield-mobile/issues/391 issue duplicated.


fix: App> trouble with building on windows 10

fix: App> trouble with building on windows 10

Ah so here the idea is that you need to copy the tpl file to a real file: https://github.com/plotly/react-chart-editor#development-setup


Add RandomSquareGrid image perturber

Rebased on updated master

Added docstring and comments
Changed the s parameter to be a tuple, requiring the user to specify the cell size along both dimensions. Also changed the class name to a more generic RandomGrid since it is not explicitly square now
If you used a cell size of 1, the max shift was 0 which caused an error to be thrown by the random number generator. I added logic for this

Is mentioning RISE and/or including its license in the docstring something we should do here? The logic is similar.

Is mentioning RISE and/or including its license in the docstring something we should do here? The logic is similar.

We could mention that this impl is based on RISE and exists to address the changing-"square"-size aspect of the original RISE implementation. I think it is important to capture where we came from (RISE) and why we are here (fixed pixel rectangles).

LGTM, thanks!


Finally tasks should be triggered in case of missing results

Expected Behavior
I have a finally task that consumes results of previous tasks. The documentation states:

finally tasks are guaranteed to be executed in parallel after all PipelineTasks under tasks have completed regardless of success or error.

The task producing the result fails with a nonzero exit code. I expect that the finally task gets triggered.
Actual Behavior
Finally task is not run instead it is waiting forever for results. It is fine if the expected results are missing (and set to null). But the finally task should always be triggered.
Steps to Reproduce the Problem

Apply the following definition:

apiVersion: tekton.dev/v1beta1
kind: Task
metadata:
  name: generator-task
spec:
  results:
  - name: my_result
  steps:
  - name: make-result
    image: busybox
    script: |
      echo -n "foo" | tee $(results.my_result.path)
      exit 1
---
apiVersion: tekton.dev/v1beta1
kind: Task
metadata:
  name: finally-task
spec:
  params:
  - description: value of result from previous task
    name: former_result
  steps:
  - name: use-result
    image: busybox
    script: |
        echo -n "$(params.former_result)"

---
apiVersion: tekton.dev/v1beta1
kind: Pipeline
metadata:
  name: resulttest
spec:
  finally:
  - name: notification
    params:
    - name: former_result
      value: $(tasks.generator.results.my_result)
    taskRef:
      name: finally-task
  tasks:
  - name: generator
    taskRef:
      name: generator-task

Create a PipelineRun for pipeline resulttest
The finally task for this pipeline never gets executed.

Additional Info
There is a related section in the documentation about results after error but this is not helpful. Setting onError: continue completely hides the error which is not what I want.
Not sure but may be related to TEP-0058. Potentially another use case to be covered?

Kubernetes version:
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.1",
GitCommit:"632ed300f2c34f6d6d15ca4cef3d3c7073412212", GitTreeState:"clean", BuildDate:"2021-08-19T15:38:26Z", GoVersion:"go1.16.6", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.6", GitCommit:"d921bc6d1810da51177fbd0ed61dc811c5228097", GitTreeState:"clean", BuildDate:"2021-10-27T17:44:26Z", GoVersion:"go1.16.9", Compiler:"gc", Platform:"linux/amd64"}

Tekton Pipeline version:
Output of tkn version or kubectl get pods -n tekton-pipelines -l app=tekton-pipelines-controller -o=jsonpath='{.items[0].metadata.labels.version}'
Client version: 0.17.0
Pipeline version: v0.28.2
Dashboard version: v0.21.0

thank you @jensh007 for this report 👍
It is designed such that the finally task is not executed in case of a missing result:
Q. What happens to Pipeline when the dependent task boskos-acquire either failed or not executed, and the task result leased-resource is not initialized?
A. The finally Task boskos-release is attempted and included in the list of skippedTasks since the task result leased-resource is not initialized. Pipeline controller logs this validation failure including param name leased-resource of the finally task boskos-release with the result reference leased-resource and result producing task boskos-acquire. Pipeline continues executing rest of the final tasks and exits with completion.
We are looking at this proposal to help solve your use case:
https://github.com/tektoncd/community/blob/main/teps/0048-task-results-without-results.md
This proposal was demonstrated at the API WG: https://docs.google.com/document/d/17PodAxG8hV351fBhSu7Y_OIPhGTVgj6OJ2lPphYYRpU/edit#heading=h.qq31rcli4y3f
The demo is available here at the mark of 16:30.
Havent been able to work on it since then.

Thanks for sharing the proposal. I understand that it may be designed like this but I consider this not to be very practical. I assume in many cases finally tasks are used to e.g. send notifications about build status etc. In case something fails (and thus a result may be missing) you always want to get the notification and not a silent death... So hope to see the enhancements soon :)

And  one more comment: You at least should adapt the documentation (see citation above). The statement that "finally tasks are guaranteed to be executed regardless of of success or error" is wrong if this is by design.

 There was no error or other indicator that there was something wrong with the results.

TEP-0103 is addressing this missing piece where the pipelineRun reports the reason why a task was skipped.
@Yablargo what is your use case? What is the expectation, run the finally task anyways (with missing results)?

@pritidesai I actually post results, but since the task fails ( I fail it intentionally, the QC failed.), I can't read the results I posted, when I get to the finally task.
In a simplified example, I run npm audit, see a critical vuln has been caught, and want to write the bit of JSON on the findings summary to /tekton/results/whatever and pick that up later in my finally task to post it back to the originating merge request, but the results I just pushed don't exist and my task gets skipped.

I see, thanks @Yablargo for the details! The pipelineRun does not publish results when a task fails and that's the reason finally task is not executed.
We have designed finally such that pipelineRun must skip running finally in case of missing results to avoid any kind of accidental execution.
I think, this use case is more suitable for https://github.com/tektoncd/pipeline/issues/3749 where pipelineRun must publish results (if they were initialized) in case of a failure.
Do you vote for https://github.com/tektoncd/pipeline/issues/3749? wdyt?

I noticed you have do vote for it 😄  https://github.com/tektoncd/pipeline/issues/3749#issuecomment-1112351402

Yep, Thanks! For right now I upload everything at the end of each TaskRun and pull it back down in the Finally (well, I was already uploading anyways, I just was hoping to use /results to save some calls and task params).
It works, but it would save some complexity to be able to just poll the /results.

this is not fixed yet 🙃
/remove-lifecycle rotten

still not fixed and very annoying
/remove-lifecycle stale

Still exists on latest pipeline release: v0.44.0

it still exists.
/remove-lifecycle rotten

This is still incredibly important, and if I am being honest this sort of thing is why I have moved on from tekton.

I am very sorry to hear that! Yup, I can understand, we are working towards making this project more resilient. At the same time, any kind of contribution is highly appreciated.
We have implemented a feature to allow producing a task result from a failed task:
https://github.com/tektoncd/pipeline/pull/6510
This is available since https://github.com/tektoncd/pipeline/releases/tag/v0.48.0. Please let me know if this does not work for you or looking for something more. Thanks!

The use case mentioned in the description ⬆ is now supported: https://github.com/tektoncd/pipeline/blob/main/test/task_results_from_failed_tasks_test.go


Add in support for sorting arrays with structs in sort_array

This is a part of #4900 but is only a small part of it. CUDF already supported everything so it was only a few more mins to implement this patch with proper tests than it was to verify that it was working manually.

build

build

@jlowe I updated the docs. I forgot that buildall does not do that for us anymore, and I have to manually build with the default version to make it happen

forgot that buildall does not do that for us anymore, and I have to manually build with the default version to make it happen

we could change CI to push the result of antrun:run@update_config_docs to the PR branch instead of having it complain that the developer forgot to run it for the default version


added some effect to the image in signup page

Added some effect to the image in SigUp page. The existing image does not have any effect with this.
Now effect added to the image in signup page which will give user better experience while using the website.

Closes #766

Screenshot
The Screenshot is added while hover over the image:

Kindly add relevant labels into it

solve conflicts.


Review AC VCBS-053 to see if we can reduce the number of node required for the test

Due to the load on Jenkins when running repeatedly and often networks with a large number of nodes we need to review the AC code VCBS-053
Tasks:
@Vegaklaus

[ ] See if we can reduce the over all number of nodes required for this test from 10 to a lower number but still keeping the test useful/relevant
[ ] Assess the criticality of this test so that we can write the tests in a priority order (where possible)

Additional work to be captured in other issues:
@gordsport to plan

[ ] Identify a set of test cases we want to run on a large network (due to the overhead this will run less often that the current QA runs)

Closing as we should do this as described.

https://github.com/vegaprotocol/system-tests/issues/1940

The above issue and linked (partially) blocking issue have been raised to cover this.


check capabilities for cert-manager changes

Relies on https://github.com/rancher/helm-unittest/pull/5

Relates to #23424


Don't Merge this yet

Tell me if you like to double space after periods?

https://en.wikipedia.org/wiki/Sentence_spacing
I guess it is different in French


Allow reaffirming predictions

When a forecaster gets to a question they have forecasted before, and before they've moved any of the slider, the predict button should not be disabled but say "Reaffirm" instead.
Applies to all question types.

Ready for testing on prod

lgtm


Switching between herpesvirus datasets

When switching between herpesvirus datasets, no tree is automatically loaded and all datasets are listed instead. This is because of the way Nextstrain handles the datasets.

This is a known Nextstrain issue and is related to the following issues:
https://github.com/nextstrain/auspice/issues/697
https://github.com/nextstrain/auspice/issues/698


Clarify max byte error message.

Hi there.
Great plugin.  This is just a small change to make the over byte message a little clearer, and bring it in line with the language used by AMP.  See: https://www.ampproject.org/docs/design/responsive_amp#validate-your-styles-and-layout

Thank you!
I am sorry to confirm late.
I gratefully merged it.


Fix for WFLY-12751, Move base layers to the full galleon feature pack

Issue: https://issues.jboss.org/browse/WFLY-12751
Simply moved layers from test FP to main FP. Removed un-needed dependency on test FP.
I have issues running DistributableTestCase locally (even with your branch without my changes). The tests that fail are testGracefulServeOnShutdown and testGracefulServeOnUndeploy. Test gest stucks for very long time.

@jfdenise I pushed https://github.com/wildfly/wildfly/compare/master...bstansberry:WFLY-12751 with this commit in it.
I see the test problem you are talking about locally, and it seems it impacts my base branch, not just your commit. I rebased and repushed that branch a while ago and it looks like https://ci.wildfly.org/viewLog.html?buildId=174135&buildTypeId=WF_PullRequest_GalleonLinuxJdk11&tab=buildLog is hanging in that test.


Preprocessify can't get environment vars

Using preprocessify it  seems that It can't see environment variables.
If I do
$ VAR="foobar" gulp

where I put an @echo it returns "undefined".
It only works if I use context inside the call to processify.

Preprocessify doesn't do anything with environment variables.  It uses the context object to define preprocess variables.  If you want to use environment variables you'll have to copy the environment variables into the context object manually.  Or use it in combination with something like konfig (see the readme).

If you haven't come across this: https://github.com/motdotla/dotenv, check it out.

It's strange because preprocess itself uses environment variables to populate the annotations inside your code during the build. I'll check the dotenv solution.


Added destroy method to remove event listeners

I don't expect this to go anywhere as this repo seems dead, but we are still dependent on this library in a React project so I figured I would keep a paper trail

Since these charts register event listeners and never remove them we were finding bugs when navigating to other pages and resizing
In this PR I've only updated the 2 charts we use

My formatter completely changed the files, so if this ever gets reviewed I can update properly (there is no .prettierrc)

We're currently maintaining a fork, and using the local .tgz file to continue with this library.

@microsoft-github-policy-service agree


Signing session takes 20+ seconds on Node v21

After updating the test matrix to include Node v21 I noticed signing takes ages, causing tests to fail. Maybe this is something to do with the fact I'm still using CryptoJS? I'm not sure...
I think for now we should look into moving over to the built-in crypto functions. Might improve performance and should have the added benefit of reducing the package size

Also worth noting, crypto-js was discontinued as of October 2023. I'm not an expert on crypto stuff, but this issue might be useful for migrating over to the node:crypto equivalents of crypto-js functionality.

Yes it's been at the back of my mind for a while and I've been thinking about how to approach it. I definitely want to move over to the standard crypto library very soon
I wasn't aware of this issue. Some useful info in there that'll come in handy. Thanks! If you (or anyone else) find any more useful guides or discussions feel free to drop them here. It'll make my life a lot easier when we eventually do make the transition 😁

I'm using React Native with Hermes and I was experiencing 5+ second waits on an iPhone 16.
I'm using the buffer polyfill. I did some profiling on both buffer and crypto-js and neither seemed to be the problem. Rather it seems it is jsbn. Replacing its BigInteger with the in-built BigInt gives me pretty good performance (around 6 times faster). I still need to iron out some problems (the sign step fails due to a type error). But it seems to demonstrate that jsbn is the culprit.
See the debug branch of my fork.
I know using BigInt may not work for everyone. If not, would it be possible to make the BigInteger library configurable?

I'm using the buffer polyfill. I did some profiling on both buffer and crypto-js and neither seemed to be the problem. Rather it seems it is jsbn. Replacing its BigInteger with the in-built BigInt gives me pretty good performance (around 6 times faster).

Ah good spot! Thats really useful to know

I know using BigInt may not work for everyone. If not, would it be possible to make the BigInteger library configurable?

Maybe yeah.. I'm thinking we could create a layer of abstraction for large numbers, then swap out the underlying library based on config. Wouldn't need to install any new dependencies since BigInt is built-in. Shouldn't cause any interference for existing users
I'm a bit stuck on how we'd specify that config. Our library's all functions though so we're limited in what we can do. We could use environment variable, a package level config object, more function params... I'll give it some thought
Should be do-able though. I'll put together a solution soon. Feel free to open a PR if you feel like it, always happy to collaborate

Hey, thanks for the thoughtful response and for considering
configurability! Here’s my take:

My initial thought is to provide a higher-order function, something
like createCognitoSrpHelper(config):
CognitoSrpHelper. This would avoid configuring the library itself and
stays aligned with the functional paradigm, which I think is important to
preserve.
I’d be very happy to use an instance-based approach since I don’t have
many integration points, so it’s easy for me to adopt.
To keep the current "import-and-use" style (and avoid breaking the
existing API), you could internally create a default instance and have the
existing functions delegate to that instance. This way, users who don’t
need customization won’t be impacted, while others can benefit from the
flexibility.
Let me know your thoughts! I’d be happy to help out with a refactor if
needed.
On Mon, 25 Nov 2024 at 23:06, Simon McAllister @.***>
wrote:

I'm using the buffer polyfill. I did some profiling on both buffer and
crypto-js and neither seemed to be the problem. Rather it seems it is jsbn.
Replacing its BigInteger with the in-built BigInt gives me pretty good
performance (around 6 times faster).
Ah good spot! Thats really useful to know
I know using BigInt may not work for everyone. If not, would it be
possible to make the BigInteger library configurable?
Maybe yeah.. I'm thinking we could create a layer of abstraction for large
numbers, then swap out the underlying library based on config. Wouldn't
need to install any new dependencies since BigInt is built-in. Shouldn't
cause any interference for existing users
I'm a bit stuck on how we'd specify that config. Our library's all
functions though so we're limited in what we can do. We could use
environment variable, a package level config object, more function
params... I'll give it some thought
Should be do-able though. I'll put together a solution soon. Feel free to
open a PR if you feel like it, always happy to collaborate
—
Reply to this email directly, view it on GitHub
https://github.com/simonmcallister0210/cognito-srp-helper/issues/33#issuecomment-2499213685,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADLLNGA36WSJ5LZIALLJ7T2COUNVAVCNFSM6AAAAABMAW5EZWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJZGIYTGNRYGU
.
You are receiving this because you commented.Message ID:
@.***>


Adds PropertySource to UmbracoPropertyAttribute

This is a fix for issue #184.
Added code to allow you to select a property source when using UmbracoPropertyAttribute so that you can define the order in which properties are checked between the instance properties and properties in the umbraco properties collection. This can be set per property using the PropertySource parameter of the UmbracoPropertyAttribute, or globally by setting Ditto.DefaultPropertySource in an app startup handler. If using either InstanceThenUmbracoProperties (the default) or UmbracoThenInstanceProperties and the site is in debug mode, and ditto detects that the property requested hides a property in the other source, then a warning log message will be logged to the log file giving instructions to either not use reserved aliases, or to use PropertySource to override the behavior if required.

Seems like a very reasonable solution. I like it #h5yr

Looks great to me. Love that you've standardized the property flags and handled the indexer also. Very robust!

Agreed, this is fantastic. I was thinking one feature I might expect would be to set at class level using UmbracoProperties attribute the PropertySource for a specific class. This would be different from at property level or the global default.
What does everyone thing? Separate ticket or a no goer?

@jamiepollock not a bad idea, and pretty easy to implement. Give me a minute :)

Ok, maybe 6 minutes :)

Great work is still great work, chap. Bingo bango! I feel like this pretty much complete now :) #h5yr!

Just gonna do a unit test around the UmbracoProperties stuff, then yea, I'm happy to merge in. Thanks for all the feedback everyone.


feat: Add div mod hint

Hi @zmalatrax,
I've made significant progress on the DivMod hint implementation, including updates to the hintName.ts, hintSchema.ts, hintHandler.ts. The core logic is nearly complete, but a few final adjustments are still needed in divMod.ts. Also, the verified tag is working according with the contribution guidelines.
Next steps:

Create integration tests to validate the overall behavior.

I'll be focusing on these remaining tasks.

Update 📌
Hi @zmalatrax

Implemented and tested the DivMod hint.

All related tests for DivMod have passed successfully.

Here are the test results:

Please review the changes and let me know if any additional modifications are needed, also can you explain me a little bit the integration test with Cairo I think it is the only I need to complete it 🫡

@coxmars Looks like your typescript files aren't well formatted, we're using Trunk, I'd recommend you installing their extension if you're on VSCode or Neovim
Otherwise, you can add their CLI  and run trunk fmt.

Thanks for the feedback, I will apply the requested changes asap 🫡

About the integration test, you need to add a Cairo program divmod.cairo in the cairo_programs/cairo/hints folder that makes use of the DivMod hint.
It should be a simple program focused on this hint, such as
fn main() {
    let _ = 17_u32 / 12_u32;
    let _ = 17_u32 % 12_u32;
}

Hi @zmalatrax I'm trying to execute make compile and I got this error:

I already have Bun, Poetry, I ran poetry install for dependencies and I downloaded cairo-lang too and also Python 3.10 so do you have an idea what is the problem?

Hi @zmalatrax I was able to solve some problems running the integration test haha, it took me a while but I got it done, unit test and integration test ready plus the feedback given, as you can see below it is working so I will open this draft PR to "Ready for review" 🫡

Hi @zmalatrax I already changed the condition as you said and @enitrat I added a new edge case test, if you need something else to be improved just tell me, thanks 🫡

Hi @zmalatrax I applied the changes and add a few more tests ✅

Hi @zmalatrax, I already applied the improvement, thanks 🫡


How can this be modified to get utterance time, word order, word start / stop time ?

How can this be modified to get utterance time, word order, word start / stop time ?
Something similar in scope to what Dragon might give you :

http://www.nuance.com/naturallyspeaking/dss/dtd/dss-idxv2.dtd
<BODY>
  <RENDERING>
    <TRN Type="IC">
      <INP>
        <REF WRD_Id="1">at</REF>
      </INP>
      <OUT>At</OUT>
    </TRN>
    <SPC> </SPC><REF WRD_Id="2">the</REF><SPC> </SPC><REF WRD_Id="3">register</REF>
    <TRN Type="UNK">
      <INP>
        <REF WRD_Id="4">,\guessed</REF>
      </INP>
      <OUT>,</OUT>
    </TRN>
    <SPC> </SPC><REF WRD_Id="5">but</REF><SPC> </SPC><REF WRD_Id="6">there</REF><SPC> </SPC><REF WRD_Id="7">were</REF>
    <SPC> </SPC><REF WRD_Id="8">no</REF><SPC> </SPC><REF WRD_Id="9">criminal</REF><SPC> </SPC>
    <REF WRD_Id="10">penalties</REF><SPC> </SPC><REF WRD_Id="11">at</REF><SPC> </SPC><REF WRD_Id="12">that</REF>
    <SPC> </SPC><REF WRD_Id="13">point</REF><SPC> </SPC><REF WRD_Id="14">was</REF><SPC> </SPC>
.......
    <UTT Id="16" Start="53.570" End="58.880">
      <WRD Id="127" Start="53.570" End="54.130" C="893" SRT="had" />
      <WRD Id="128" Start="54.130" End="54.410" C="885" SRT="been" />
      <WRD Id="129" Start="54.410" End="55.330" C="918" SRT="convicted" />
      <WRD Id="130" Start="55.330" End="55.450" C="848" SRT="of" />
      <WRD Id="131" Start="55.450" End="56.230" C="784" SRT="expectancies" />
      <WRD Id="132" Start="56.230" End="56.710" C="823" SRT="at" />
      <WRD Id="133" Start="56.710" End="57.070" C="908" SRT="four" />
      <WRD Id="134" Start="57.070" End="57.970" C="955" SRT="nineteen" />
      <WRD Id="135" Start="57.970" End="58.290" C="951" SRT="ninety" />
      <WRD Id="136" Start="58.290" End="58.880" C="951" SRT="six" />
    </UTT>

I understand this has not been implemented and would be interested in doing so, but I need a bit of guidance...like what classes to research.
I have been able to get it to debug, so I feel like I have a shot.  The challenge to Kaldi is the learning curve is a brick wall, so am not sure where to focus.

This is already supported. You have to set the property word-boundary-file (as outcommented in https://github.com/alumae/kaldi-gstreamer-server/blob/master/sample_english_nnet2.yaml). Then the JSON encoding of the final result will include word start and end times, something like:
{
   "status":0,
   "segment-start":0.0,
   "segment-length":6.12,
   "total-length":6.12,
   "result":{
      "hypotheses":[
         {
            "transcript":"one two three four five six seven eight.",
            "confidence":10000000000.0,
            "likelihood":153.665,
            "word-alignment":[
               {
                  "start":1.18,
                  "length":0.43,
                  "word":"one",
                  "confidence":1.0
               },
               {
                  "start":1.65,
                  "length":0.29,
                  "word":"two",
                  "confidence":0.989745
               },
               {
                  "start":1.97,
                  "length":0.4,
                  "word":"three",
                  "confidence":1.0
               },
               {
                  "start":2.37,
                  "length":0.53,
                  "word":"four",
                  "confidence":1.0
               },
               {
                  "start":2.9,
                  "length":0.39,
                  "word":"five",
                  "confidence":1.0
               },
               {
                  "start":3.29,
                  "length":0.4,
                  "word":"six",
                  "confidence":1.0
               },
               {
                  "start":3.69,
                  "length":0.43,
                  "word":"seven",
                  "confidence":1.0
               },
               {
                  "start":4.28,
                  "length":0.24,
                  "word":"eight",
                  "confidence":0.991182
               }
            ]
         }
      ],
      "final":true
   },
   "segment":0,
   "id":"e887d790-b321-47ae-ae7b-13276b1b3fcd"
}

I am unable to get this to work.  I am using /client/dynamic/recognize
Is there anything I need to do besides uncomment out the "word-boundary-file" ?

The extended results are only available through the the websocket-based interface.

Can you offer some advice on how to implement this.  Any advice would be
appreciated!
On Thu, Oct 5, 2017 at 2:32 AM, Tanel Alumäe<EMAIL_ADDRESS>wrote:

The extended results are only available through the the websocket-based
interface.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alumae/kaldi-gstreamer-server/issues/95#issuecomment-334371572,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGPc5qKdKCeJqvSg7vFsmEWPXwgQ-Funks5spHf6gaJpZM4Pi-UP
.


Uncaught TypeError: class heritage events_1.EventEmitter is not an object or null

Hi, im getting this error and its crashing my react app.
I wrapped the client inside a hook:
import {FuturesClient} from "bitget-api";
import {supabase} from "../supabase-client";
import {useAuth} from "../providers/user-provider";

export const useBitgetClient = (): FuturesClient => {
    const {user} = useAuth();
    const {data, error} = supabase
        .from("bitget_api")
        .select(`api_secret, api_phrase, api_key`)
        .eq("user_id", user?.id)
        .single();

    if (!data) {
        throw new Error("BitgetClient data not available");
    }

    const clientOptions = {
        apiKey: data.api_key,
        apiSecret: data.api_secret,
        apiPass: data.api_phrase
    };

    const bitgetClient = new FuturesClient(clientOptions);

    return bitgetClient;
};

And using it inside components like this:
 const history = useBitgetClient()
        .getRecentTrades("BTCUSDT_UMCBL")
        .then((r) => console.log("getHistoricTrades", r));

    console.log("history", history);

When I start the app the app now crashes. After doing some research it turns out there's an issue with Vite which does automatically polyfill node.js modules.
From the vite website:
Module externalized for browser compatibility
When you use a Node.js module in the browser, Vite will output the following warning.
Module "fs" has been externalized for browser compatibility. Cannot access "fs.readFile" in client code.

This is because Vite does not automatically polyfill Node.js modules.
We recommend avoiding Node.js modules for browser code to reduce the bundle size, although you can add polyfills manually. If the module is imported from a third-party library (that's meant to be used in the browser), it's advised to report the issue to the respective library.

Is there something else I can do until this has been improved? I'm kinda stuck for now not being able to access Bitget data.
Thanks

Hi @TYRANTHOSAURUS
There are two key dependencies that I'm aware of that have problems in the browser, since they're node modules:

crypto (used for the hmac signature function as part of the request authentication process)
events (EventEmitter is used to bubble up websocket events via the WS client)

Both have polyfills available, although I'm not familiar with Vite so can't provide specific guidance. I know that in typescript projects configured to build for frontend environments, you can install and then provide the polyfills in the tsconfig:
https://github.com/tiagosiebler/bybit-api#browser-usage
I'm not sure if the same method is enough with vite or if there's another way to correctly configure it, but hopefully this points you in the right direction! If this helps, can you let us know how you ended up configuring vite to correctly work with the polyfills? I'd love to document this better for others. Alternatively, if you still can't get it working after fighting it some more, please provide a very very minimal example project using this module and I can try to help.

Hi @tiagosiebler ,
Thanks for the feedback,
In vite config pre-4.0 you could do something similar but that's not working for me anymore so I eventually found this package which did work.
Check out this package: https://github.com/davidmyersdev/vite-plugin-node-polyfills
Thanks!

Very useful, thank you! Would you mind sharing an example for others, on what you added to your vite.config when adding this package? Thanks!

I got mine from the documentation:
import { resolve } from 'node:path'
import react from '@vitejs/plugin-react'
import { defineConfig } from 'vite'
import { nodePolyfills } from '../src/index'

export default defineConfig({
  plugins: [
    react(),
    nodePolyfills({
      include: ['fs', 'path', 'buffer'], // Add to include to import specific protocols
      exclude: ['fs'], // Add to exclude while leaving include empty to enable all then exclude.
      globals: {
        process: 'build',
      },
      protocolImports: true,
    }),
  ],
  root: resolve(__dirname),
})


karate.callSingle() should be able to cache results even across test runs

problem statement: when re-running tests in "local dev" mode, and when the tests depend on a common "auth" flow that typically builds an Authorization header - I have to wait for this step to complete even when having to debug  some trivial step deep in my test-suite
EDIT: most of the time, the auth token will be valid for a period of time like 15 minutes or 30 minutes - so it is wasteful to generate a new one for every re-run, it may involve one or two HTTP calls, and in some cases create a fresh "throw away" user-account in your SSO store
of course if you use the VS Code debugger - that solves this problem well - but here is an alternate proposal - which is to enhance the existing karate.callSingle() functionality to do the following:

cache the results (as JSON) in a temporary file (in addition to the in-memory Map)
when invoked and Map cache hit "misses", check if file exists
if file updated time is within a threshold, return the JSON from the file

I was initially thinking to add a third argument to karate.callSingle() but figured adding a new configure key for e.g. callSingleCache which gives us better flexibility and a more elegant way to switch this on or off depending on karate.env. for example:
    * configure callSingleCache = { minutes: 15 }

and
  if (karate.env == 'local') {
    karate.configure('callSingleCache', { minutes: 30, dir: 'target' });
  }

Since there can be potentially multiple karate.callSingle() instances, we will auto-gen a file-name, but user can choose the "temp" dir, which can default to target.

0.9.6 released


Wordpress HTTPS: Too many redirects

Something strange is happening when I try to enable HTTPS for Wordpress.
I installed all needed dependencies on a fresh FreeBSD installation. Compiled Caddy from source.
I installed Wordpress and then I tried to change site url from HTTP to HTTPS, which yields in a "too many redirects" error in the browser.
I have the most basic Caddyfile possible, which doesn't do any redirects. And in Wordpress both SITE_URL and home url are set to the HTTPS url of my site.
http://freebsd.denbeke.be {
	root site
	fastcgi / <IP_ADDRESS>:9000 php
	log access.log

}

https://freebsd.denbeke.be {
	root site
	tls server.crt server.key
	fastcgi / <IP_ADDRESS>:9000 php
	log access.log
}

In the past I successfully switched from HTTP to HTTPS on other Wordpress sites, but here I don't understand what's going on.
Something I'm doing wrong with the Caddy configuration? Or is Wordpress just messing things up?

Odd. I can't think of any reason why Caddy would be initiating any redirects with that configuration. This doesn't appear to be an isolated occurrence. Maybe fiddle with the WordPress config a little bit?
Pinging @thomas4019 - I know it's been a while, but you ran Caddy with WordPress. Did you ever encounter redirect loops?

I also suspect Wordpress somehow. But since I'm not really experienced in Caddy I wanted to ask it here.
The redirect loop looks like this:
--2015-08-03 19:46:00--  https://freebsd.denbeke.be/wp-login.php
Resolving freebsd.denbeke.be (freebsd.denbeke.be)... <IP_ADDRESS>
Connecting to freebsd.denbeke.be (freebsd.denbeke.be)|<IP_ADDRESS>|:443... connected.
WARNING: cannot verify freebsd.denbeke.be's certificate, issued by ‘/C=BE/ST=Some-State/O=Internet Widgits Pty Ltd/CN=freebsd.denbeke.be’:
  Self-signed certificate encountered.
HTTP request sent, awaiting response... 302 Found
Location: https://freebsd.denbeke.be/wp-login.php [following]
--2015-08-03 19:46:00--  https://freebsd.denbeke.be/wp-login.php
Reusing existing connection to freebsd.denbeke.be:443.
HTTP request sent, awaiting response... 302 Found
Location: https://freebsd.denbeke.be/wp-login.php [following]
--2015-08-03 19:46:00--  https://freebsd.denbeke.be/wp-login.php
Reusing existing connection to freebsd.denbeke.be:443.
HTTP request sent, awaiting response... 302 Found
Location: https://freebsd.denbeke.be/wp-login.php [following]
--2015-08-03 19:46:00--  https://freebsd.denbeke.be/wp-login.php
Reusing existing connection to freebsd.denbeke.be:443.
HTTP request sent, awaiting response... 302 Found
Location: https://freebsd.denbeke.be/wp-login.php [following]

I have copied the is_ssl() function from Wordpress hoping this could reveal the error. Created the following test file:
/**
 * Determine if SSL is used.
 *
 * @since 2.6.0
 *
 * @return bool True if SSL, false if not used.
 */
function is_ssl() {
        if ( isset($_SERVER['HTTPS']) ) {
                if ( 'on' == strtolower($_SERVER['HTTPS']) )
                        return true;
                if ( '1' == $_SERVER['HTTPS'] )
                        return true;
        } elseif ( isset($_SERVER['SERVER_PORT']) && ( '443' == $_SERVER['SERVER_PORT'] ) ) {
                return true;
        }
        return false;
}

// Some tests...
header("Content-Type:text/plain");

echo 'is_ssl(): '; var_dump(is_ssl());

echo '$_SERVER[\'HTTPS\']: '; var_dump($_SERVER['HTTPS']);

echo '$_SERVER[\'SERVER_PORT\']: '; var_dump($_SERVER['SERVER_PORT']);

Nginx
Using Nginx and PHP-FPM on another server this yields (of course for an HTTPS request):
is_ssl(): bool(true)
$_SERVER['HTTPS']: string(2) "on"
$_SERVER['SERVER_PORT']: string(3) "443"

Caddy
Running the same script using Caddy and PHP-FPM it yields this:
is_ssl(): bool(false)
$_SERVER['HTTPS']:

Notice:  Undefined index: HTTPS in /usr/home/mathias/site/test.php on line 26

NULL
$_SERVER['SERVER_PORT']: string(0) ""

This seems to be causing the redirect, since Wordpress will think an HTTP request is sent, which doesn't match the HTTPS in the canonical domain.

Holy moly, you figured it out. Thanks! I think the latest commit should do the trick: 8d81ae88dacaf94d0fc70268cb03050a9e3e6bef
Let me know if it doesn't. Should go out on the next release (or you can compile from source to try it now).

I just recompiled from source, looks all good!
is_ssl(): bool(true)
$_SERVER['HTTPS']: string(2) "on"
$_SERVER['SERVER_PORT']: string(0) ""

And Wordpress works perfectly now!
Thank you!

@DenBeke
In your Caddy file is <IP_ADDRESS>:9000 an apache server virtualname ?

@bigtan
<IP_ADDRESS>:9000 is PHP-FPM.


bump mongoid version to 8

mongoid tree isn't using anything version relveant that would break so it's safe to use with mongoid 8.x

@neongrau Thanks for the PR! I released v2.2.0 based on your changes.

Thank you for reacting so quick!


Fix issue with getting stuck in search/cmdline/qna

Fixes https://github.com/newsboat/newsboat/issues/2336 and simplifies stfl usage (more code duplication in stfl files but less hard-to-read stfl code stored in c++ strings and less keymap indirection)

Coverage: 59.468% (-0.06%) from 59.529% when pulling 7ab6fb2c2798b3b714b90a680315a2f7ba48e8c0 on dennisschagt:fix/stuck-in-qna-on-confirm-exit into 56d9e9337346b6035e9b8d977d29a1bdad4fd15c on newsboat:master.

Thanks!
For the changelog: this effectively means that the user can leave any Q&A with Escape, regardless of key bindings.

For the changelog: this effectively means that the user can leave any Q&A with Escape, regardless of key bindings.

I don't think that is a change, it was already configured that way via the inline stfl setup: input[qnainput] on_ESC:cancel-qna <...>
However, I think the new approach makes it a lot clearer that that is happening.


Updating @angular/core, @angular/material, @angular/fire and @angular/cli

Updating to the latest versions of @angular/core, @angular/material, @angular/fire and @angular/cli.
Changing some imports and modules to match the new versions

Sorry for the delay on this, @mandarini! It looks like a bunch of extra lines were changed as part of some eslint autoformatting. Can you please remove those changes so it is a bit easier to tell what the changes specific to this PR are?
Once you do, I'll take a quick look and then merge this PR.


Atomikos Support

I am migrating from EhCache 2 to EhCache 3. I do identified Bitronix is only supported JTA implementation. Do you plan to add support for Atomikos ?

Please use the mailing list for such question.
Bitronix is used during our integration test to validate our XA implementation. However, we do support any implementation. So Atomikos should work perfectly.


[WIP] split and join

Implements a first draft of split and join of notebook cells.
It only works for (neigboring) code cells. Although there is nothing in the code that should prevent joining a code and a markdown cell setText on a markdown cell does not have any affect. For the same reason splitting of markdown cells does not work. Additionally, that a markdown cell is rendered when it loses focus is not what I'd expect when splitting it.
Modifying the content of cells is bad. For example, when splitting a cell, we should just be able to delete some content rather than resetting it. Also, we get content as a string of joined lines only to split it again after joining it with other content. Also, I'm not sure that the funtionality even lives at the right layer.

really good work and the issues raised are valid. Ideally the logic can implemented all in coreActions but currently we can't due to following limitations in the core

Modifying cell content: right now we will create a text buffer when doing search for every cell and attach TextModel to the cell once it's visible. We should probably expose an ITextBuffer like interface to support rich text editing other than simple set
Layering: putting the split/join actions in either the NotebookEditor or NotebookViewModel are both fine. Using the latter is better as we then only touch the viewModel/textModel code and the UI will be updated properly automatically. One missing feature is merging undo points. Deleting a cell and then inserting one are two separate undo points now.

We need to add pushUndoStop API like what Monaco Editor does, to allow contribs control how operations should be merged
Join Cells is a bit difficult as cell operation and cell content change mixed together.

If we are going to implement this feature in an extension, the missing APIs are:

Shortcut to get TextEditor for a notebook cell if it exist. Currently extensions have to go through all editors and check if their resource URI matches. (but it's not necessarily a bad idea)
Reveal/Focus cell API. After a cell is split or joined, extensions may need to set selections correctly.

For joining two cells, the text editor for the first cell should not change and there should only be a modelContentChange event emitted.

@rebornix Based on our discussion, I reworked the PR so that we now work directly with lines, i.e. string[]. I added a todo in the code where we'd need to make sure we don't have to join the line contents again. This is a bit more complicated but avoids copying content unnecessarily. It also allows to work with positions directly.

Pls merge if you feels that's good enough for now before you rework the cell text buffers.

@kieferrm thanks, it looks great. I'll fix the conflicts and merge.


Created Longest Common Subsequence.cpp

C++ program to find out the length of the longest common subsequence using dynamic programming.

Well Done! Thanks for the contribution. ❤


Add owner information in GetObjectACL and add Context

This PR add the object owner information in the ObjectInfo returned by GetObjectACL.
And add the context to GetObjectACL by creating a method GetObjectACLWithContext

@pierre-emmanuelJ, it would be better to do a git mv get-object-acl.go get-object-acl-context.go to preserve the history better rather than add and remove files. GetObjectACL() function needs to be replaced with GetObjectACLWithContext in the functional tests and examples/s3/getobjectacl.go

Thank you, I'll do it ASAP

Thank you, I'll do it ASAP
Hello, I'll fix the comment of @poornas  today.
Thank you

Is it okay for you? 😃


Add support for Xiu server

This adds support for getting stats from the Xiu server.
Description by Xiu author:

Xiu is a simple,high performance and secure live media server written in pure Rust, it now supports popular live protocols like RTMP[cluster]/RTSP/WebRTC[Whip/Whep]/HLS/HTTP-FLV.

According to the documentation, the HTTP stats API currently only supports RTMP. It seems there are plans to support other protocols in the api in the future.

Hi @datagutt ,
Looks good to me, thanks for adding the stream server!


Move OTel wrapper py code into instrument script

This PR hopes to move some of the logic from otel-wrapper.py into otel-instrument so that we have only 1 file used for Python Lambda.
This was just an initial investigation by me, and requires much more rigorous testing before we should implement these changes.
Creating this as a draft until I can come back to work on it.

Looks like our suspicious were true 😕 We were able to fix the dependency path, but its true that logging doesn't work to show the errors.
The script mysterious ends at this line:
entry_point.load()().configure()  # type: ignore

Specifically at the configure() call. However, it ends with no log information as to what the problem is. It also doesn't seem to raise an Exception, it just ends...
Even if I remove the configuration, this line also ends without any trace
conflict = get_dist_dependency_conflicts(entry_point.dist)

Closing this PR because there doesn't seem to be an obvious solution for now.

The current code follows my mental model - after system() it's another process, so python code executed in this script wouldn't affect the "real" code that is run after system(). So the most that the instrument script is set up environment variables, or modify the command line (which it doesn't seem to currently d0>
An improvement that would probably make this more clear, and would make this more consistent with other languages is to convert otel-instrument into a shell script.
https://github.com/open-telemetry/opentelemetry-lambda/blob/main/java/layer-wrapper/scripts/otel-handler#L6
Mucking with env variables has native syntax in shell which can make things simpler, but more importantly also makes clear that this is just a script and not something we would want to run arbitrary python code in.


[BI-1176] Create Germplasm Template Mapping

Create default germplasm import mapping based on the template
Associate default germplasm import mapping with germplasm import functionality

Linked to PR bi-api/BI-1176

I just wouldn't merge this until the api side is merged. Also, do you know if there are any updates that will need to be done to the table view based on what Chris did and is there a card for that?

There's a reason why this is tentatively in draft at the moment. Probably should have a discussion with @ctucker3 on Monday to get on the same page.

I just wouldn't merge this until the api side is merged. Also, do you know if there are any updates that will need to be done to the table view based on what Chris did and is there a card for that?

There's a reason why this is tentatively in draft at the moment. Probably should have a discussion with @ctucker3 on Monday to get on the same page.

According to Chris it should be good to go!


Update laravel example

Description
Update laravel example to use new client
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Tests pass
[ ] Created/updated tests
[ ] Extended the documentation

Codecov Report

Merging #107 (dbafa7b) into main (9e21bed) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head dbafa7b differs from pull request most recent head 21d8533. Consider uploading reports for the commit 21d8533 to get more accurate results

@@            Coverage Diff            @@
##               main     #107   +/-   ##
=========================================
  Coverage     77.05%   77.05%
  Complexity      540      540
=========================================
  Files            64       64
  Lines          1748     1748
=========================================
  Hits           1347     1347
  Misses          401      401

Continue to review full report at Codecov.

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


Neither the built .java/.class files nor the compiled LibKML.jar file are included in the distribution

What steps will reproduce the problem?
1.
2.
3.

What is the expected output? What do you see instead?

Please use labels and text to provide additional information.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 12 Jun 2009 at 1:34

check new build and reopen this if need needed.


Automatic PR from RestAPI-PR1659

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1659

Codecov Report

Merging #1427 into master will decrease coverage by 0.01%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##           master    #1427      +/-   ##
==========================================
- Coverage   52.61%   52.59%   -0.02%
==========================================
  Files        3344     3344
  Lines       85048    85073      +25
==========================================
+ Hits        44746    44747       +1
- Misses      40302    40326      +24

Impacted Files
Coverage Δ

...ons/network_interface_load_balancers_operations.py
37.5% <ø> (ø)
:arrow_up:

...etwork/v2017_08_01/operations/usages_operations.py
93.75% <ø> (ø)
:arrow_up:

...ork/azure/mgmt/network/v2017_08_01/models/usage.py
100% <100%> (ø)
:arrow_up:

...t/network/v2017_08_01/network_management_client.py
96.63% <100%> (-0.03%)
:arrow_down:

.../operations/virtual_network_gateways_operations.py
16.83% <4%> (-1.16%)
:arrow_down:

Continue to review full report at Codecov.

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

No content


Error due to Existing Log Group Resource

Hi, thank you for new release. I have another question.
I recently attempted to deploy the latest release, but unfortunately, I encountered a new error that persisted across multiple attempts. Despite starting the deployment process from scratch on several occasions, I consistently faced the same issue. However, surprisingly, I could access my REDCap without fixing the error.
✖  Errors
   EC2Server ROLLBACK_COMPLETE
   deleteStackSfnLogGroup: Resource handler returned message: "Resource of type 'AWS::Logs::LogGroup' with identifier '{"/properties/LogGroupName":"/aws/vendedlogs/states/deleteStackSfnLogGroup"}' already exists." (RequestToken: 0ab09a58-75d6-a275-89f6-3c888622940c, HandlerErrorCode: AlreadyExists)

After deleting '/aws/vendedlogs/states/deleteStackSfnLogGroup', the error message disappeared, but I'm not sure it's OK.
My question is, what is the cause of the error?
Thank you in advance for your time and effort.
P.S. In the section '5.1.1 Use redcap as a subdomain' within the README.md file, should the value for hostInRoute53 be set to 'acme.com' rather than true?

The EC2Server stack is an optional feature that you can safely delete and re-create again. It's there to execute REDCap on a temporary EC2 instance and do long running requests (import projects for example). I think what is happening is that the new code version was trying to deploy a new resource for this. Your solution of deleting the log was correct, but I would also recommend two things:

Only enable the EC2Stack in your stage.ts file when you need it, so generally this should be disabled.   // ec2ServerStack: { .... }

You can try to delete  only the EC2stack from CloudFormation (if exists) to make sure future deployments will not fail. Although, when its deployed, it should be automatically be deleted after the configured time.

I will keep an eye for this next week, we might be able to improve this log issue.
Regarding the domain:
I think you case is to setup redcap at this domain redcap.tomotsuguseki.com using an existing registered domain. In the README 5.1.1 is the correct section. However, the acme.com is a fake domain for demo purposes.
For explanation purposes, we will assume we own the acme.com domain and our intention is to deploy REDCap at the redcap.acme.com.

So your setting should be:
const prod: RedCapConfig = {
  ...baseOptions,
  hostInRoute53: true,
  domain: 'tomotsuguseki.com',
  subdomain: 'redcap',

Thank you for your reply during weekend.
I commented out the ec2ServerStack, destroyed, and re-deployed the REDCap for simplicity, instead of updating existing environment. The deployment succeeded, but I have a new problem.
Regarding the domain
During my first attempt, hostInRoute53: 'tomotsuguseki.com' worked better than hostInRoute53: true.
This time, hostInRoute53: true can deploy the REDCap without error, but I couldn't access the REDCap through 'redcap.tomotsuguseki.com' although the status is active.
Thank you for advance for your help at your convenience.

You're welcome.
I suspect there might be something from the previous deploy interfering with the domain configuration. I'll help you fix it, please check the following:

In your AWS Console / Route 53 there should be now 2 HostedZones now. One for tomotsuguseki.com and a new one for redcap.tomotsuguseki.com

In redcap.tomotsuguseki.com Zone, you should have only 2 records NS and SOA types.

In the tomotsuguseki.com Zone please check:
3.1 There should be one A record with value to the xxxxxx.awsapprunner.com.
3.2 There's one record name with redcap.tomotsuguseki.com of Type NS. The value of this record should be the same as the NS record in the previous hosted zone (redcap.tomotsuguseki.com).

Let's confirm this first. Thanks.

I'm sorry, I've updated my previous comment, I made a mistake.

This is what we are deploying for reference:
https://docs.aws.amazon.com/Route53/latest/DeveloperGuide/dns-routing-traffic-for-subdomains.html
Section Create a hosted zone for the subdomain, and create records in the new hosted zone

Thank you for your instruction. I checked accordingly:

There are two Hosted Zones as you said.
Four values in the record (name: tomotsuguseki.com, type: NS) at tomotsuguseki.com, and another one (name: redcap.tomotsuguseki.com, type: NS) at redcap.tomotsuguseki.com were different each other.

Is it OK?

I see... thank you for checking.

There are two Hosted Zones as you said.

This is good.

Four values in the record (name: tomotsuguseki.com, type: NS) at tomotsuguseki.com, and another one (name: redcap.tomotsuguseki.com, type: NS) at redcap.tomotsuguseki.com were different each other.

Yes, they should be different.
HostedZone: tomotsuguseki.com
tomotsuguseki.com	nameserver = ns-1519.awsdns-61.org.
tomotsuguseki.com	nameserver = ns-1645.awsdns-13.co.uk.
tomotsuguseki.com	nameserver = ns-24.awsdns-03.com.
tomotsuguseki.com	nameserver = ns-657.awsdns-18.net.

redcap.tomotsuguseki.com	nameserver = ns-xxxx.awsdns-xxxx.org.
redcap.tomotsuguseki.com	nameserver = ns-xxxx.awsdns-xxxx.co.uk.
redcap.tomotsuguseki.com	nameserver = ns-xxxx.awsdns-xxxx.com.
redcap.tomotsuguseki.com	nameserver = ns-xxxx.awsdns-xxxx.net.

In the second hosted zone redcap.tomotsuguseki.com created by the deployer:
HostedZone: redcap.tomotsuguseki.com
These nameserver should match the one in your tomotsuguseki.com HostedZone.
redcap.tomotsuguseki.com	nameserver = ns-xxxx.awsdns-xxxx.org.
redcap.tomotsuguseki.com	nameserver = ns-xxxx.awsdns-xxxx.co.uk.
redcap.tomotsuguseki.com	nameserver = ns-xxxx.awsdns-xxxx.com.
redcap.tomotsuguseki.com	nameserver = ns-xxxx.awsdns-xxxx.net.

Thank you for your confirmation.
Should I wait for next update?

No, this should work on this version.
Can you please confirm that the NS servers from the HostedZone: redcap.tomotsuguseki.com are the same for in the HostedZone: tomotsuguseki.com for redcap.tomotsuguseki.com  record?

No, there isn't the record named redcap.tomotsuguseki.com in the  HostedZone: tomotsuguseki.com.

OK, this is the problem then. Give me a few minutes to investigate.

I think I found the issue. Investigating the code and the README there's a difference for this case.
Can you try deploying with this:
const prod: RedCapConfig = {
  ...baseOptions,
  hostInRoute53: 'tomotsuguseki.com',
  domain: 'tomotsuguseki.com',
  subdomain: 'redcap',

If there's any error that the zone exist, you can delete the HostedZone redcap.tomotsuguseki.com  and try again.

Thank you for your help.
After the change, I could access my REDCap on redcap.tomotsuguseki.com without problem.


Create New Skeleton "skeleton-tf-deployment"

💡 Summary
Consider creating a new skeleton repository that either descends from cisagov/skeleton-generic or from this repo. This would focus on Terraform deployments instead of Terraform modules.
Motivation and context
We have been using this skeleton for anything Terraform related and it has worked, but the workflows for Terraform modules and Terraform deployments have core differences. Creating a new skeleton-tf-deployment repository would allow Terraform deployment related standards to be represented in a skeleton instead of needing to look across existing deployment repositories to figure out what files need to be added (at minimum) as well as how things look across such projects in our organization.
Implementation notes
Ideally once this new skeleton is created we would, over time, migrate any Terraform deployment repositories to descend from this new skeleton. Then any future Terraform deployments would start from the new skeleton instead of this repository.
Acceptance criteria

[ ] New repository is created
[ ] Dev team signs off on the Initial Commits Pull Request
[ ] (OPTIONAL) Existing Terraform deployment repositories are migrated to use this new skeleton

The handling of default tags by the actual Terraform module and the example code in #49 highlights another place where cisagov/skeleton-tf-module and the new skeleton repository that @mcdonnnj is proposing would differ.  I was a little meh about the need for a separate repo, but this has pushed me toward heartier support of this addition.

Updated the proposed skeleton name to mirror the language used by Terraform as was pointed out to me by @jsf9k 🍻


add_tcm_intro.py

FYI, In the package, to add Turner Classic movies intros and outros to your movies as extras, the file is add_tcm.py but in your README document https://github.com/ghomasHudson/Jellyfin-Auto-Collections/blob/master/README.md, you call for add_tcm_intro.py.

FYI, In the package, to add Turner Classic movies intros and outros to your movies as extras, the file is add_tcm.py but in your README document https://github.com/ghomasHudson/Jellyfin-Auto-Collections/blob/master/README.md, you call for add_tcm_intro.py.

Thanks for the heads up - I've moved the intros functionality into a separate repo: https://github.com/ghomasHudson/Jellyfin-Movie-Intros


chore: Add noxfile to run pytest across multiple python versions

This PR adds a configuration script for Nox.
Nox is useful mainly when you finish working on some bigger feature and you want to be sure it works in all supported versions of Python.
In order to use Nox, just install it:
pip install nox

And run it:
nox

Nox will create 4 independent environments - this takes quite awhile, so I've set Nox to reuse already built environments, which means if you change something inside an environment, you need to manually remove it from .nox/ directory.
As the next step, we should run all Python versions tests inside Github Actions.
Waiting for #34

Codecov Report

Merging #40 (8f2ef8f) into master (7220d5b) will increase coverage by 24.57%.
The diff coverage is 74.41%.

@@             Coverage Diff             @@
##           master      #40       +/-   ##
===========================================
+ Coverage   52.19%   76.77%   +24.57%
===========================================
  Files          43       43
  Lines        6246     6320       +74
===========================================
+ Hits         3260     4852     +1592
+ Misses       2986     1468     -1518

Impacted Files
Coverage Δ

src/evotorch/neuroevolution/gymne.py
79.53% <ø> (+54.44%)
:arrow_up:

src/evotorch/neuroevolution/neproblem.py
92.30% <ø> (+57.14%)
:arrow_up:

src/evotorch/neuroevolution/supervisedne.py
90.47% <ø> (+55.55%)
:arrow_up:

src/evotorch/neuroevolution/vecgymne.py
79.70% <ø> (+58.80%)
:arrow_up:

src/evotorch/tools/__init__.py
100.00% <ø> (ø)

src/evotorch/tools/misc.py
85.78% <29.16%> (+3.74%)
:arrow_up:

src/evotorch/algorithms/ga.py
63.21% <66.66%> (+43.21%)
:arrow_up:

src/evotorch/core.py
69.27% <68.18%> (+7.68%)
:arrow_up:

src/evotorch/neuroevolution/net/parser.py
94.78% <75.00%> (+43.45%)
:arrow_up:

src/evotorch/decorators.py
100.00% <100.00%> (ø)

... and 30 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


TypeError: Cannot assign to read-only property 'expression' in DMN Editor

An unexpected error occurred while using the DMN Editor in the IBM BAMOE Developer Tools extension for VS Code. The editor encountered a TypeError with the message:
Cannot assign to read-only property 'expression' of object '#'.
I was try to call Java method using BKM then coming this error.
Steps to Reproduce:

Open the DMN Editor in the IBM BAMOE Developer Tools extension (version 9.1.1).
Drag and drop a BKM and try call java method.

{
"name": "TypeError",
"message": "Cannot assign to read only property 'expression' of object '#'",
"stack": "TypeError: Cannot assign to read only property 'expression' of object '#'
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:5440336
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:5970779
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4734017
at Object.produce (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4734169)
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4734001
at i (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4592868)
at n.setState (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:6184981)
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:5970728
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:5440304
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:5380018
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:5356256
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:5356305
at https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:5354635
at Rl (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4389775)
at t.unstable_runWithPriority (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4548828)
at $o (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4330628)
at Il (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4389243)
at Be (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4401448)
at Kt (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/kisha/.vscode/extensions/ibm.bamoe-developer-tools-9.1.1/dist/webview/NewDmnEditorEnvelopeApp.js:2:4309411)"
}
Environment:
Extension Version: 9.1.1
VS Code Version: 1.95.3
Operating System: Windows 11

Hi @kkumarDMS. This repository tracks issues for Apache KIE, not IBM BAMOE. Please refer to https://github.com/IBM/bamoe/issues.


Add @rst/@endrst commands to Doxygen

Description
These are just aliases for @verbatim that make it easier to embed restructured text in Doxygen comments.  This is, for example, useful for adding references to other non-doxygen parts of the documentation.
See the change in the README on how to use it.
How I Tested
By using it in a local documentation build.

Thanks for the work!


Add files via upload

@ShinyaHir First stage of verification passed, official domain is jss.edu.jo as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@ShinyaHir Thank you for your patience while waiting for a review and processing your request. Pull request merged.


XPath query in config.yaml

This pull request continues implementation of XPath queries in the config.yaml file.
There were a few edge cases that generated errors when using adv_xml_search, for example, trying to fetch an attribute value rather than a text node. I've added a series of try/except statements to check what kind of node was returned. adv_xml_search was also modified to take direct Xpath queries, instead of friendly comma-separated input.
Three unit tests were added to test_utilities.py:
test_adv_xml_search_abs()
test_adv_xml_search_rel()
test_adv_xml_search_attr()

XPath queries are included for two metadata fields in the example config.yaml:
- BioSampleBioProjectAccession: XPATH, //Links/Link[@target='bioproject']/@label
- NucleotideBioSampleAccession: XPATH, //GBXref[GBXref_dbname/text() = 'BioSample']/GBXref_id

These changes seem cover most of the use-cases/XML I'm encountering right now. I wonder though if you have alternative ideas for implementation, or additional cases to be considered?

Hiya, great work on refining the implementation! I have a couple suggestions on the implementation that I will leave as reviewer comments on the PR. Let me know what you think.

I'm going to merge this PR for now. I think the changes are super useful and tests have good coverage.
Well done!


PCRE library - to maintainer version

There are two major versions of the PCRE library. The current version, PCRE2, released in 2015, is now at version 10.39.
The older, but still widely deployed PCRE library, originally released in 1997, is at version 8.45. This version of PCRE is now at end of life, and is no longer being actively maintained. Version 8.45 is expected to be the final release of the older PCRE library, and new projects should use PCRE2 instead.
I think it can be easily compiled to wasm library based on https://github.com/leyao-daily/pcre2 and we may modify ModSecurity configuration to accept this update.

Based on this comment, PCRE is still a needed dependency even if we enable PCRE2. So, to make it work, I think we have to build pcre2 and provide it to modsec with a proper configuration on top of what we already have. I would suggest doing tries on top of https://github.com/intel/modsecurity-wasm-filter/pull/24, just because of the updated emsdk (if we feel that that PR is going to be okay).

Relevant: https://github.com/SpiderLabs/ModSecurity/issues/2750

v3.0.9 resolved the above issue.


Finding music to download

Having trouble finding lot's of music in download

Please try our new frostwire-android-release-2.4.0-b681-plus.apk -  Release sep 3. 2021 at 4:20pm EST

Having trouble finding music to download it only list one or two song's from the group I'm looking up and I tried that new FrostWire and the something there as well


[POC] add keep-full flag to allow preservation of an entire vendor and subvendor dir

I found a need to preserve an entire top-level vendored project and all of its subvendor tree.
I was vendoring some kubernetes code-gen libraries for my types.  The normal path for making it work is to require k8s.io/code-generator (github.com/kubernetes/code-generator) and then to call a script in that repo. (Usage here if you're interested: https://github.com/openshift/kube-projects/blob/master/hack/update-generated.sh)
I've opened this pull to demonstrate one way to handle the preservation for this case.  In general, vendored  repos can include their own vendor trees which need different handling than the flat vendor case.

@deads2k Thanks for this PR. If I got this right you are vendoring a tool, so it won't appear as a required package and will be removed. I tried to document this in https://github.com/sgotti/glide-vc#vendoring-additional-tools where the proposed solution is to use the glide lock file.
Another solution is the one I proposed here: https://github.com/sgotti/glide-vc/issues/27#issuecomment-268636794
Instead this patch uses a generic approach to keep all the files matching the provided glob. I'm not against it but I was trying to avoid it if other solution were possible (i.e. I'd like to avoid users ending up using it instead of understanding the real reason of the removed packages).
What do you think about the above solutions?


net/frr: Route-Map match Prefix-List

Closes #985
I have to add something to the volt but I dont want to break #996

@mimugmail you if it happens, no problem, I can merge or rebase to get it fixed again.

I'd leave it as match2 and count up for more like community match support @elizandropacheco already metioned, so when everyone's fine, merge ..

@mimugmail I don't think this will cause a merge conflict but I wouldn't be mad at you because of a simple change which needs to be done. I just don't like it if the base branch gets refactored while there is a merge request open which just leads to uselessly burned time because all the conflicts need to be resolved and this may be as much work as writing it again.

Then you are free to approve so we can go on :)

@fichtner perhaps this one too when you find the time. :) Thanks <3

Merged, thanks!

Already available? How do I test? I have the dev option enabled.

# opnsense-code plugins && cd /usr/plugins/net/frr && make upgrade

P.S.: Don't use the CARP option, it's not finished yet.

One only fix is that the SET field should not be mandatory. It is currently requiring padding (even if a blank space).


Qbeast with embedded shaded delta

Description
Qbeast did not work on Databricks because there was a conflict between the open-source delta and the Databricks' one. In this PR, we shaded all dependencies that we need to run Qbeast without relying on the provided delta.

A clear, concise description of the changes proposed in the pull request
I've created a new subproject, QbeastShadeDelta, that generates an assembly with the required shaded dependencies.

Type of change
Describe the change you're making: how it affects the API, user experience...

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

Checklist:
Here is the list of things you should do before submitting this pull request:

[x] New feature / bug fixed
[ ] Add comments to the code
[ ] Change the documentation
[ ] Add tests
[ ] Dependent changes are merged

How Has This Been Tested? (Optional)

Hey guys! I think this is stopped. How we are proceeding? Can we close it for now or can it be merged? @eavilaes

Hey guys! I think this is stopped. How we are proceeding? Can we close it for now or can it be merged? @eavilaes

We didn't manage to make it work. I think we can close it for now, as it has been idle for a long time.


Batch 2 Story 6 Technology 2

Vertical - Technology
Design - https://www.figma.com/file/VqbZX9o1rmf4X1qDpKiEfh/Stories---Template-Batch-2?node-id=7332%3A3347
Asset Folder - https://drive.google.com/drive/folders/1PNXwzutdOeoJ5ZdSl4Q0CnvzlMboNEmY

Story Link: https://google-webstories.rt.gw/web-stories/batch-2-story-6-technology-2/


Tablayout without viewpager

hello how can i create custom layout without viewpager.
I need to create fixed tablayout with image and text but i dont want viewpager for the same how can I achieve it using this library.

Hi, @ashishbhandari
I'm sorry late reply.
I’ve been busy lately at work :P
It does not support without ViewPager.


Broken link in readme

The readme contains this:

Guide Documents: https://github.com/samchon/typescript-json/wiki

But that link just redirects to https://github.com/samchon/typescript-json. Should this be removed? Is the wiki private?

Sorry for late process. I spent too much time on v3.3 update.
The guide documents would be written in this month.


Test additional python versions

Check for support in older 2.7 versions as well as python 3

Tested and confirmed working with 2.7.6 and 3.4.3.


Create solution.py

Python Solution for https://www.hackerrank.com/challenges/minimum-absolute-difference-in-an-array/problem

Also add the README link

@adityabisoi  Should I add the readme link of the problem in Algorithms Section or should I create a separate Greedy section in readme and then add it?

Just add the solution

Done. Now merge this pull request and #179 .